Parse troublesome Yaml with new workaround in workaround for underying parser bug

What's changed?
Prevent IndexOutOfBoundsException with new workaround for issues in the underlying YamlParser events.
What's your motivation?
Received reports about failures with a particular Yaml snippet; with this exact snippet you hit the new conditional.
Even if you change strings in the Yaml snippet, you will not hit the new conditional; Fun times to be had tracing that down.
Have you considered any alternatives or workarounds?
Ideally we report this upstream and see it resolved there.
Any additional context
The earlier workaround was introduced in these two commits to work around bugs with the underlying Yaml parser we use.

https://github.com/openrewrite/rewrite/pull/4178
https://github.com/openrewrite/rewrite/pull/4187

@pdulich you might enjoy this one; a very specific Yaml snippet managed to trigger a bug in the workaround for the parser bug you discovered earlier. Those parser events sure seem off at times.

Thanks for the report. I already know the cause.
There are many other problems with the parser. Therefore, I'm completely rewriting it.
In the meantime I can push a bugfix for this issue.

Thanks a lot! This PR already adds a minimal workaround that might cause the least amount of conflicts; are you ok with me merging this in the meantime or do you see another quick option?

Oh haha. Sorry. I'm at work and only had a quick look on your comment. I didn't realize that this is a PR and not an issue.
If I remember correctly, the real bugfix is to replace getIndex() by getPointer().

Ah perfect, trying that now; passed locally with just that replacement, thanks!

There are many other problems with the parser. Therefore, I'm completely rewriting it.

Could you tell us a bit more about your plans here? It can at times be challenging to review large sweeping changes, as much as the effort you'd put in might be appreciated. Just want to be sure we don't end up with a PR that's hard to see through.


User creation should be atomic

Currently if the server or database connection is dropped between User.objects.create_user(**validated_data) and user.save(update_fields=['is_active']) - there will be an account created with is_active=True - this shouldn't be allowed.

Coverage decreased (-1.4%) to 95.543% when pulling 5438421f3204cb2e0696c3e9b6d96f0851c76f89 on pik:user-creation into c4d26bdf4496bfd32cb2719fd24620a754a1c8fd on sunscrapers:master.

Coverage decreased (-1.4%) to 95.543% when pulling 5438421f3204cb2e0696c3e9b6d96f0851c76f89 on pik:user-creation into c4d26bdf4496bfd32cb2719fd24620a754a1c8fd on sunscrapers:master.

Coverage increased (+0.05%) to 96.961% when pulling 6d00dc04d9ff01ddafa6e9548b7bb7d7b13ee014 on pik:user-creation into c4d26bdf4496bfd32cb2719fd24620a754a1c8fd on sunscrapers:master.

Coverage increased (+0.05%) to 96.961% when pulling 6d00dc04d9ff01ddafa6e9548b7bb7d7b13ee014 on pik:user-creation into c4d26bdf4496bfd32cb2719fd24620a754a1c8fd on sunscrapers:master.

True! Thanks a lot for that!


DiskRP sdk changes for api version 2020-05-01

build failed. Please make it sure build succeeded locally and all playback tests pass

@hyonholee Hi Hyonho I've been seeing compute PRs going into different branches recently. For example #12509 is targeting compute-2020-06 while this one is for master.
What's your team's plan for PR reviewing? I assume every compute PR will be reviewed just by you and go to your branch, and eventually a big PR will be created from compute to master. Is that so?

/azp run

@isra-fel this PR is for DiskRP.   This is separate from Compute-2020-06, which contains CRP API updates.

/azp run 9

/azp run 0,1,2,3,4,5,6,7,8


Hyper is unresponsive when Bash is passed interactive argument

[ x] I am on the latest Hyper.app version
[ x] I have searched the issues of this repo and believe that this is not a duplicate
OS version and name: Ubuntu 16.04
Hyper.app version: 1.4.8
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/jonathanbell/ec301f638c79196878e955485f132305
Relevant information from devtools:
total init time<PHONE_NUMBER>0000003

Uncaught TypeError: Cannot read property 'substr' of null
    at Object.t.colors.crackRGB (bundle.js:47)
    at Object.t.colors.mix (bundle.js:47)
    at r.TextAttributes.syncColors (bundle.js:47)
    at r.TextAttributes.setDefaults (bundle.js:47)
    at r.VT.CSI.m (bundle.js:47)
    at r.VT.dispatch (bundle.js:47)
    at r.VT.parseCSI_ (bundle.js:47)
    at r.VT.interpret (bundle.js:47)
    at r.Terminal.interpret (bundle.js:47)
    at l.forEach (bundle.js:41)

The issue is reproducible in vanilla Hyper.app: Is Vanilla
Issue
When using  /bin/bash as my shell interactively (shellArgs: ["-i"]) Hyper is extremely unresponsive. Typing into the window does nothing. Occasionally, when Hyper is launched it will work after 5 minutes or so but it still has a very slow response to input. I would like to use Bash interactively because I want to source my .bashrc file when Hyper is launched. The file is sourced when Hyper launches but the terminal is not usable (cannot type inside the terminal window).

Hey there, please try again with the new Hyper v2 release and open a new issue if the problem stills exists. Thank you!


Add BRIN support for spoint and sbox?

Would incorporating this PR be useful to anyone? It was fully working but never merged. It seems a waste to not retain this functionality, and it could be a real benefit when dealing with very large tables with millions of points.
See https://github.com/akorotkov/pgsphere/pull/8 and this comment.

Hi, I see @vitcpp already opened a PR here (will leave comments there eventually). I'm happy to help if needed.

@gbroccolo Thank you for the patch! I preserved your authorship of the patch with some code adjustments. I believe, you don't mind if the patch will be applied in this repo. We also appreciate it if you will open some new PRs in this repo that seems to be the only alive repo of the pgSphere project.

Hi @vitcpp fine. Will try to open new PRs, each one for each spherical geometry, any sooner. I will probably start with the circular type, as requested by @esabol.


Missing defineCustomElements

With @stacks/connect:4.2.2 calling showConnect results in a TypeError: connectUi.defineCustomElements is not a function.
Although defineCustomElements is imported by Connect it doesn't seem to be exported by connect-ui:
https://github.com/blockstack/ux/blob/128ffa28d47468e001c137d5f1e18bc1458fe5f2/packages/connect-ui/src/index.ts#L1
Possibly related: https://github.com/blockstack/ux/pull/582#issuecomment-688027120

@hstove @yknl thoughts here?

Still happening with @stacks/connect:4.3.0

I'm personally finding that showBlockstackConnect works for me instead of showConnect with @stacks/auth": "^1.0.0-beta.13".

@markmhx @stacks/auth re-exports all exports from @stacks/connect where showBlockstackConnect is just an alias for showConnect, so it is surprising if you get a different outcome using showBlockstackConnect :
https://github.com/blockstack/ux/blob/585a731847dd505cc0c1508ad78f2c50f06d8ff2/packages/connect/src/ui.ts#L46

Calling showConnect worked as expected after monkeypatching defineCustomElements in @stacks/connect-ui by setting it to the same from the generated @stacks/connect-ui/dist/custom-elements.
This suggest that the fix is changing the source path of the import to something like:
import { defineCustomElements } from 'connect-ui/dist/custom-elements';

I'm finding that showConnect works in this implementation:
import { AppConfig, UserSession, showConnect } from '@stacks/auth';

const appConfig = new AppConfig(['store_write', 'publish_data']);

export const userSession = new UserSession({ appConfig });
export const storage = new Storage({ userSession });

export function authenticate() {
  showConnect({
    appDetails: {
      name: 'Todos',
      icon: window.location.origin + '/logo.svg',
    },
    redirectTo: '/',
    finished: () => {
      window.location.reload();
    },
    userSession: userSession,
  });
}

View source

I'm finding that showConnect works in this implementation

I also have showConnect working for an app using another build environment. Surprisingly, given the missing import documented above.
But it doesn't work when building with Google Closure, with showConnect failing due to a missing connectUi.defineCustomElements - just as expected from it not being imported.

Hmm I'm not sure. @hstove or @yknl may have ideas about that behavior you're seeing.

@njordhov is this still an issue for you? If so, please re-open.


zero vram

Have tested three games: No Man's Sky, Titan Quest, Far Cry.  All fail to run.  Wine reports insufficient vram found.  System is intel 8400 cpu with 16GB RAM, 6GB nVidia 1060 using latest proprietary driver in Kubuntu 18:04.  The games cannot initialize an opengl window.
<EMAIL_ADDRESS>

Can you try closing GOG Galaxy, running the following in a terminal, and then attempt the games again:
sudo snap connect gig-galaxy-wine:hardware-observe

This might allow the appropriate permissions for the games to detect the amount of VRAM on the system.

Issue was resolved with nVidia driver 396.45.
Thanks!


Add tick to MultiProgress to complement join

Changed joining field of MultiProgress from AtomicBool to Mutex.
Because the internal join_impl can now be called multiple times
I figured it's better to use a Mutex as it automatically unlocks
preventing panics "bricking" the progress bar.
Because MultiProgress can contain multiple child progress bars I feel
like the meaning of a single tick isn't really clear. Just a single
state update (self.rx.recv()) seems too small. Because of that I
allow the user to configure how long do they want a tick to last
adding greater flexibility.
Made is_done public to let the user know whether they still need
to invoke tick

Fixes #33, seems like fixes #125

I am in need of a fix for #125 asap.  Will this make it into a new release soon?

@dan-da If you want to you can modify your Cargo.toml to use my PR instead of the published version. To do this you need to replace indicatif = "0.14.0" with  indicatif = { git="https://github.com/mibac138/indicatif", branch="mpb-tick" } and it should work. I'd appreciate any feedback you might have if you do use this code.

@mibac138 hey thanks.  I just tried with your branch, but nothing displays, same as with 0.14.0.  I am just creating a MultiProgress with 3 ProgressBar+style and then calling set_position on each, in a single thread  Any chance you can provide a working single-threaded example?
Also, I tried to call MultiProgress::tick() just to see if it helps, but it requires a progress::TickTimeLimit and indacitif::progress is private.

@dan-da I'm not sure what is it that doesn't work for you but I've written a simple example below:
use indicatif::{MultiProgress, ProgressBar, TickTimeLimit};
use rand::Rng;

fn main() {
    let mpb = MultiProgress::new();
    let pb1 = mpb.add(ProgressBar::new(30));
    let pb2 = mpb.add(ProgressBar::new(30));

    let mut rng = rand::thread_rng();
    for _ in 0..10 {
        thread::sleep(Duration::from_secs(1));
        pb1.set_position(rng.gen_range(0, 30));
        pb2.set_position(rng.gen_range(0, 30));
        mpb.tick(TickTimeLimit::Indefinite);
    }
}

It is working great now.  Thx for the example.

hmm, I do have one display prob.  I'm not sure if it's a problem with the lib, or my usage of it.   So I have two progress bars with {elapsed_precise} in the template, and one bar (A) is updated more frequently with a new position (via set_position()) than the other (B).  After each update, which occurs every 500 milliseconds, tick() is called on the MultiProgress that contains both, with a 50ms Duration.  The issue is that the elapsed seconds in A sometimes update before B.  Usually they are in sync, each second, but not always.   I'm not sure if TickTimeLimit::Timeout is supposed to work like ProgressBar::enable_steady_tick()?  I just want the elapsed time to update in unison for both bars each time.
If above is not clear, here's a simplified version of my update func, which is invoked from a long-running process.
// pass bars vec in separately because MultiProgress doesn't seem to have a getter...?
fn update_progress_bars(m: &MultiProgress, bars: &[ProgressBar], status: &Status) {
bars[1].set_position(status.file_bytes_written);            // value changes more often
bars[2].set_position(status.transfer_bytes_written);     // value changes less often

// trying to get elapsed time clocks updating secs in unison.
m.tick(TickTimeLimit::Timeout(Duration::from_millis(50)) );

thread::sleep(Duration::from_millis(500));   // simulate lengthy work done for 1/2 second.

}

@dan-da I believe this might be due to the refresh rate of your draw target. Try changing how you created MultiProgress from presumably MultiProgress::new() to MultiProgress::with_draw_target(ProgressDrawTarget::stderr_nohz()). This issue is caused by how the MultiProgress works compared to ProgressBar. Regular progress bars when updating have immediate access to the latest progress state, so when they are drawn everything is based on the latest state. However, MultiProgress internally uses mpsc::channel to communicate with progress bars and it's implemented so that every update of a child progress bar is a single draw which makes it so that if there are two progress bars which need drawing there will be two separate draw calls and the default draw target will throttle those draws.
This also means there isn't a direct replacement for enable_steady_tick in MultiProgress. TickTimeLimit::Timeout and Deadline boil down to the same thing but are divided to improve ergonomics similar to how mpsc::channel has both recv_timeout and recv_deadline.

hi, yeah I've tried with stderr_nohz(), and I also tried creating a thread that calls MultiProgress::tick() every 10ms.   Neither approach causes the elapsed_precise clock to tick each second as desired.
Here is a test case that demonstrates the issue, with a for loop that is updating the progress bars every 2 seconds, and the clock also updates every 2 secs.
use indicatif::{MultiProgress, ProgressBar, TickTimeLimit, ProgressStyle, ProgressDrawTarget};
use rand::Rng;
use std::thread;
use std::time::Duration;

fn main() {
    let sty = ProgressStyle::default_bar()
        .template("{spinner:.green} [{elapsed_precise}] [{bar:40.cyan/blue}] {bytes}/{total_bytes} ({bytes_per_sec}, {eta})  File")
        .progress_chars("##-");

    let mpb = MultiProgress::with_draw_target(ProgressDrawTarget::stderr_nohz());
    let pb1 = mpb.add(ProgressBar::new(30));
    let pb2 = mpb.add(ProgressBar::new(30));

    pb1.set_style(sty.clone());
    pb2.set_style(sty);

    // To use thread, uncomment this and comment mbp.tick() at bottom.
/*
    let _ = thread::spawn(move || {
        loop {
            mpb.tick(TickTimeLimit::Indefinite);
            thread::sleep(Duration::from_millis(10));
        }
    });
*/

    let mut rng = rand::thread_rng();
    for _ in 0..50 {
        thread::sleep(Duration::from_millis(2000));
        pb1.set_position(rng.gen_range(0, 30));
        pb2.set_position(rng.gen_range(0, 30));

        mpb.tick(TickTimeLimit::Indefinite);
    }
}

Closed without comment on the test case I posted?

This pull is working for me for single threaded MultiProgress. Is there anything blocking it being merged? elapsed_precise is working https://github.com/etrombly/dropcutter

I don't think this is necessary with current main, but if still is feel free to open a new issue/PR.


Pledge to drop 2.7 support

The https://python3statement.org/ pledge to drop support for Python2.7 when it hits end of life on January 1, 2020 should probably be taken up by this project. We could also add a test to check for any Py2 specific stuff and create a new error saying it is not longer receiving security updates and should be avoided.
Happy to make the PR on the py3statement project on behalf of bandit if this is something the maintainers are interested in.

I agree, it would be good to indicate future dropping of Py27.  However for https://python3statement.org/, I would expect them to only take PRs on such statements from maintainers of Bandit.
In any case, we should make changes in Bandit to note the upcoming removal.  README and docs definitely.  Maybe even a runtime message.

Probably the best thing would be to just note the 1.x series will be the last to support Py2.7 and make the changes in 2.0 to be 3 only

1.6.3 is out on PyPI which includes drop of Py2.7 Thanks!


Missing publish file.

After install from npm and trying to generate definition, have this problem.
[JSDOC] FATAL: Unable to load template: Cannot find module '.../node_modules/tsd-jsdoc/publish'

Duplicate of #48


Version 0.8.20 of shims not in Maven

Describe the bug
Attempting to update the RoaringBitmap dependency in our leiningen project to version 0.8.20, but I get an error saying that version 0.8.20 of the shims can not be found. Looking at some past issues here, I see that there have been problems pushing to Maven Central, but I thought I should bring this to attention in case it is an oversight.
Java version: openjdk 14.0.1

Thanks for the report. I do verify that it is not there:
https://repo1.maven.org/maven2/org/roaringbitmap/shims/
As I explained elsewhere, there is not much I can do. The files were synced. They have been uploaded as far as I can tell.
I am afraid that Maven Central is an obsolete and not-very-serious system. There is no trace, no transparency. It may as well be managed manually by a secretary behind some desk in some remote island.
I don't understand why the Java community tolerates something like that.
I'll remove the bug label. It may be a bug, but the bug is upstream. It is not with us. We did the release.

Please wait a few days. It can take days or weeks for the artifacts to show up.

It looks like 0.8.20 landed today.


Made UI of navbar

@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
3, because the PR includes both CSS and JSX changes across multiple files, introducing new components and styles. The complexity is moderate, requiring a detailed review of styling consistency, responsiveness, and code structure.

 🧪 Relevant tests
No

 🔍 Possible issues

Inconsistent Styling: The use of hard-coded colors and sizes may lead to inconsistencies. Consider using CSS variables for colors and sizes to ensure consistency and ease of maintenance.

Lack of Responsiveness: The CSS does not include media queries for different screen sizes. This might cause layout issues on smaller screens.
 🔒 Security concerns
No

 Code feedback:
relevant fileClient/Client/src/components/LandingPage.css
suggestion

Consider using CSS variables for colors, font sizes, and spacings to improve maintainability and consistency across your application. For example, --primary-color: rgb(84,121,247); can be used for all primary color usages. [important]

relevant linecolor:rgb(84,121,247);
relevant fileClient/Client/src/components/LandingPage.css
suggestion

For better accessibility and user experience, ensure that interactive elements have a focus state defined. This can be achieved by adding :focus alongside your :hover states. [important]

relevant line.dropdown-content button:hover {
relevant fileClient/Client/src/components/LandingPage.jsx
suggestion

Use a mapping function for rendering navigation links to reduce redundancy and improve maintainability. This can be done by defining an array of link objects and mapping over them to render <Link> components. [medium]

relevant lineHome
relevant fileClient/Client/src/components/LandingPage.jsx
suggestion

Consider extracting the dropdown menu into its own component to improve readability and reusability of your code. This will also make it easier to manage state and events related to the dropdown. [medium]

relevant line

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


Download all albums, add album name to file name

Download all albums recursively if new config option Is set
Add album name to filename.
I'm using photoview to provide online gallery. It uses only filename to identify photos. So if one photo is in multiple albums there's problem. Adding album name to filename solves it.

There's no 100% coverage in unit tests - it requires mocking data for all Smart Albums, I do not know how to do it.

Wouldn't fixing just #128 solve your problem? Every album (on disk, in its folder) will have unique file name. Per https://github.com/photoview/photoview#main-features, it looks like directories (folders) are mapped to albums in the app. No need to rename the file names with prepended album names.

No, it doesn't work. I had to add album name to work it.
For example, I have two albums:

Vacation 2022
Printed photos

I have same photo in both albums. In master version it will be in different folders, but same filenames.
In photoview photo will be visible only in one folder. Which one? First scanned.
Adding to database happens here. Removing check origURL == nil "fixes" problem, but creates new - every scan adds all photos to database again.
Alternatively I was thinking to add config option to provide custom filename.


Select UK English

Hello !
The login form is now forced in english.
BUT :

After loging in, we're back to the awesome French world again, so we'll have to handle lang selection here too... :'(
Couldn't find a way to get rid of my 2 waitForTimeout

Update 06/07/2021 00:54 :

Got the 2nd Select working, and the video uploading process completes. But now the video links I get in return are bad :

Seems unrelated to my changes, it shows an incomplete link when the processing isn't done yet

Update 06/07/2021 01:09 :

Fixed it with a simple do While
So it's now working on my french chromium.
The way I implemented is likely not french dependant, so it may work for other languages as well.

This change is

Thanks for the PR


Beware of any potential delay between reading bits

This isn't really so much an issue as hope to save someone else the time I've spent debugging this—in trying to debug things, I'd added statements that ran between bits and took long enough that subsequent bits were being lost.  In particular, String construction takes milliseconds, whereas the individual bits can be on the order of 20 microseconds long/apart.
Perhaps worth a note in the readme, but with a better explanation than I gave?

I can confirm that adding String inside receivedData function produces read errors.
How can we extract information from the read card, if adding String inside the receivedData function produces errors  ?

Depending on the sender, each bit sent to the wiegand reader takes ~1ms.
This is quite slow compared with other communications protocols, but it does require the microcontroller to be able to quickly process the event.
And since the receivedData callback is often called from an interrupt or from flush(), which is inside a noInterrupts() block, the microcontroller is unresponsive during the call.
Therefore receivedData() must be very fast.
For example / test purposes it is OK to print it to a serial port, but on complex applications it should probably copy the buffer somewhere else and do any complex processing asynchronously.
In particular, yes, using arduino's String() is quite slow :sweat_smile:

Therefore receivedData() must be very fast. For example / test purposes it is OK to print it to a serial port, but on complex applications it should probably copy the buffer somewhere else and do any complex processing asynchronously.
In particular, yes, using arduino's String() is quite slow 😅

How would that look ? Because right now I am having issues when not all cards are read.
I have the code structured like this:
void receivedData(uint8_t *data, uint8_t bits, const char *message)
{
    uint8_t bytes = (bits + 7) / 8;
    // Wiegand 26, first and last bits are for control, remaining 24 bits
    if (bits == 24)
    {
        wiegandFacilityCode = String(data[0] >> 4, 16) + String(data[0] & 0xF, 16);
        wiegandCardNumber = "";

        for (int i = 1; i < bytes; i++)
        {
            wiegandCardNumber += String(data[i] >> 4, 16);
            wiegandCardNumber += String(data[i] & 0xF, 16);
        }
        didReadWiegandCard = true;
    }
}

I am building wiegandCardNumber  which is a String and then set didReadWiegandCard flag to true which I'll check inside  readWiegandRoutine().
void readWiegandRoutine()
{
    noInterrupts();
    wiegand.flush();
    interrupts();

    readWiegandCard();
}

readWiegandRoutine() is inside void loop() among other routines:
void readWiegandCard()
{
    if (didReadWiegandCard)
    {
        didReadWiegandCard = false;

        wiegand_msg += "Card read: ";
        wiegand_msg += "24bits / HEX: ";
        // Print value in HEX
        String hexa = wiegandFacilityCode + wiegandCardNumber;
        hexa.toUpperCase();
        wiegand_msg += hexa + " / DEC: " + String(fromHexToDec(hexa));
        wiegand_msg += " / Facility Code: " + fromHexToDec(wiegandFacilityCode);
        wiegand_msg += " / Card Number: " + fromHexToDec(wiegandCardNumber);

        logOutput(wiegand_msg);
        wiegand_msg = "";

        wiegandID = fromHexToDec(wiegandCardNumber);
        while (wiegandID.length() < 10)
        {
            wiegandID = String(0) + wiegandID;
        }

        uint8_t buffer[40];
        String foobar = wiegandID;
        wiegandID = "Metrici multicontroller, key=" + wiegandID;
        wiegandID.toCharArray((char *)buffer, 40);
    }
}


Add Playground subscription to cleanup script

It's currently only running against Test Resources in Azure Cloud.

@danieljurek I have a cleanup script for the playground subscription written that we should get into your cleanup pipeline probably.

Done.


[Refactor] Mmlu subgroups and weight avg

Further splits MMLU into 4 subcategories instead of just aggregating over all subtasks.
Add weighted averaging and stderr.

Output
hf (pretrained=EleutherAI/pythia-2.8b), limit: None, num_fewshot: None, batch_size: 1
|                  Tasks                   |Version|Filter|Metric |  Value   |   |Stderr|
|------------------------------------------|-------|------|-------|---------:|---|-----:|
|boolq                                     |Yaml   |none  |acc    |    0.6465|±  |0.0084|
|                                          |       |      |samples| 3270.0000|   |      |
|mmlu                                      |N/A    |none  |acc    |    0.2522|±  |0.4029|
|                                          |       |      |samples|14042.0000|   |      |
|-mmlu_humanities                          |N/A    |none  |acc    |    0.2349|±  |0.1431|
|--mmlu_formal_logic                       |Yaml   |none  |acc    |    0.2857|±  |0.0404|
|--mmlu_high_school_european_history       |Yaml   |none  |acc    |    0.2485|±  |0.0337|
|--mmlu_high_school_us_history             |Yaml   |none  |acc    |    0.2353|±  |0.0298|
|--mmlu_high_school_world_history          |Yaml   |none  |acc    |    0.2616|±  |0.0286|
|--mmlu_international_law                  |Yaml   |none  |acc    |    0.1983|±  |0.0364|
|--mmlu_jurisprudence                      |Yaml   |none  |acc    |    0.2407|±  |0.0413|
|--mmlu_logical_fallacies                  |Yaml   |none  |acc    |    0.1963|±  |0.0312|
|--mmlu_moral_disputes                     |Yaml   |none  |acc    |    0.2254|±  |0.0225|
|--mmlu_moral_scenarios                    |Yaml   |none  |acc    |    0.2425|±  |0.0143|
|--mmlu_philosophy                         |Yaml   |none  |acc    |    0.1961|±  |0.0226|
|--mmlu_prehistory                         |Yaml   |none  |acc    |    0.2716|±  |0.0247|
|--mmlu_professional_law                   |Yaml   |none  |acc    |    0.2288|±  |0.0107|
|--mmlu_world_religions                    |Yaml   |none  |acc    |    0.2398|±  |0.0327|
|-mmlu_other                               |N/A    |none  |acc    |    0.2803|±  |0.1703|
|--mmlu_business_ethics                    |Yaml   |none  |acc    |    0.2200|±  |0.0416|
|--mmlu_clinical_knowledge                 |Yaml   |none  |acc    |    0.2566|±  |0.0269|
|--mmlu_college_medicine                   |Yaml   |none  |acc    |    0.2717|±  |0.0339|
|--mmlu_global_facts                       |Yaml   |none  |acc    |    0.3000|±  |0.0461|
|--mmlu_human_aging                        |Yaml   |none  |acc    |    0.2960|±  |0.0306|
|--mmlu_management                         |Yaml   |none  |acc    |    0.2524|±  |0.0430|
|--mmlu_marketing                          |Yaml   |none  |acc    |    0.2735|±  |0.0292|
|--mmlu_medical_genetics                   |Yaml   |none  |acc    |    0.2700|±  |0.0446|
|--mmlu_miscellaneous                      |Yaml   |none  |acc    |    0.2771|±  |0.0160|
|--mmlu_nutrition                          |Yaml   |none  |acc    |    0.2288|±  |0.0241|
|--mmlu_professional_accounting            |Yaml   |none  |acc    |    0.2695|±  |0.0265|
|--mmlu_professional_medicine              |Yaml   |none  |acc    |    0.4118|±  |0.0299|
|--mmlu_virology                           |Yaml   |none  |acc    |    0.2771|±  |0.0348|
|-mmlu_social_sciences                     |N/A    |none  |acc    |    0.2398|±  |0.1615|
|--mmlu_econometrics                       |Yaml   |none  |acc    |    0.2193|±  |0.0389|
|--mmlu_high_school_geography              |Yaml   |none  |acc    |    0.1970|±  |0.0283|
|--mmlu_high_school_government_and_politics|Yaml   |none  |acc    |    0.2487|±  |0.0312|
|--mmlu_high_school_macroeconomics         |Yaml   |none  |acc    |    0.2128|±  |0.0208|
|--mmlu_high_school_microeconomics         |Yaml   |none  |acc    |    0.2353|±  |0.0276|
|--mmlu_high_school_psychology             |Yaml   |none  |acc    |    0.2734|±  |0.0191|
|--mmlu_human_sexuality                    |Yaml   |none  |acc    |    0.2137|±  |0.0360|
|--mmlu_professional_psychology            |Yaml   |none  |acc    |    0.2500|±  |0.0175|
|--mmlu_public_relations                   |Yaml   |none  |acc    |    0.3636|±  |0.0461|
|--mmlu_security_studies                   |Yaml   |none  |acc    |    0.1918|±  |0.0252|
|--mmlu_sociology                          |Yaml   |none  |acc    |    0.2289|±  |0.0297|
|--mmlu_us_foreign_policy                  |Yaml   |none  |acc    |    0.2400|±  |0.0429|
|-mmlu_stem                                |N/A    |none  |acc    |    0.2623|±  |0.1835|
|--mmlu_abstract_algebra                   |Yaml   |none  |acc    |    0.3000|±  |0.0461|
|--mmlu_anatomy                            |Yaml   |none  |acc    |    0.2667|±  |0.0382|
|--mmlu_astronomy                          |Yaml   |none  |acc    |    0.2434|±  |0.0349|
|--mmlu_college_biology                    |Yaml   |none  |acc    |    0.2847|±  |0.0377|
|--mmlu_college_chemistry                  |Yaml   |none  |acc    |    0.2300|±  |0.0423|
|--mmlu_college_computer_science           |Yaml   |none  |acc    |    0.2800|±  |0.0451|
|--mmlu_college_mathematics                |Yaml   |none  |acc    |    0.2700|±  |0.0446|
|--mmlu_college_physics                    |Yaml   |none  |acc    |    0.3431|±  |0.0472|
|--mmlu_computer_security                  |Yaml   |none  |acc    |    0.2900|±  |0.0456|
|--mmlu_conceptual_physics                 |Yaml   |none  |acc    |    0.2979|±  |0.0299|
|--mmlu_electrical_engineering             |Yaml   |none  |acc    |    0.2069|±  |0.0338|
|--mmlu_elementary_mathematics             |Yaml   |none  |acc    |    0.2619|±  |0.0226|
|--mmlu_high_school_biology                |Yaml   |none  |acc    |    0.2581|±  |0.0249|
|--mmlu_high_school_chemistry              |Yaml   |none  |acc    |    0.2512|±  |0.0305|
|--mmlu_high_school_computer_science       |Yaml   |none  |acc    |    0.2400|±  |0.0429|
|--mmlu_high_school_mathematics            |Yaml   |none  |acc    |    0.2630|±  |0.0268|
|--mmlu_high_school_physics                |Yaml   |none  |acc    |    0.2517|±  |0.0354|
|--mmlu_high_school_statistics             |Yaml   |none  |acc    |    0.2269|±  |0.0286|
|--mmlu_machine_learning                   |Yaml   |none  |acc    |    0.2589|±  |0.0416|

|       Groups        |Version|Filter|Metric |  Value   |   |Stderr|
|---------------------|-------|------|-------|---------:|---|-----:|
|mmlu                 |N/A    |none  |acc    |    0.2522|±  |0.4029|
|                     |       |      |samples|14042.0000|   |      |
|-mmlu_humanities     |N/A    |none  |acc    |    0.2349|±  |0.1431|
|-mmlu_other          |N/A    |none  |acc    |    0.2803|±  |0.1703|
|-mmlu_social_sciences|N/A    |none  |acc    |    0.2398|±  |0.1615|
|-mmlu_stem           |N/A    |none  |acc    |    0.2623|±  |0.1835|

The print-out is very ugly. I get ASCII-art tables aren't the most aesthetic thing in the world, but I suspect we can do better than this. I think it would look a lot better if we replaced the - with   and dropped the leading mmlu_ from each component task, for example.
I also think that the "main results table" should show the grouped scores not the score breakdown, as the grouped scores is typically the thing we actually want.

The dash is suppose to denote the hierarchy of the task (if it's part of a group).
The group table at the bottom was so for easier to inspect since it was printed last it would be easier than having to scroll up. But it's also not an issue if you think we should put it above, my preference is that having printed after the full task table would make it easier to inspect.
As for the naming, it's understandable that the 'mmlu' prefix might be too crowded, but it's a by-product of how each task has to be uniquely identified. For the default mmlu tasks, i suppose we can remove the prefix.

@StellaAthena, simpler table presentation through the addition of task_alias and group_alias that a user can enter to have that presented instead of the task_name that has to be unique across all tasks in lm-eval.
hf (pretrained=EleutherAI/pythia-2.8b), limit: None, num_fewshot: None, batch_size: 1
|                Tasks                 |Version|Filter|Metric |  Value   |   |Stderr|
|--------------------------------------|-------|------|-------|---------:|---|-----:|
|mmlu                                  |N/A    |none  |acc    |    0.2522|±  |0.0407|
|                                      |       |      |samples|14042.0000|   |      |
| -humanities                          |N/A    |none  |acc    |    0.2349|±  |0.0289|
|  -formal_logic                       |Yaml   |none  |acc    |    0.2857|±  |0.0404|
|  -high_school_european_history       |Yaml   |none  |acc    |    0.2485|±  |0.0337|
|  -high_school_us_history             |Yaml   |none  |acc    |    0.2353|±  |0.0298|
|  -high_school_world_history          |Yaml   |none  |acc    |    0.2616|±  |0.0286|
|  -international_law                  |Yaml   |none  |acc    |    0.1983|±  |0.0364|
|  -jurisprudence                      |Yaml   |none  |acc    |    0.2407|±  |0.0413|
|  -logical_fallacies                  |Yaml   |none  |acc    |    0.1963|±  |0.0312|
|  -moral_disputes                     |Yaml   |none  |acc    |    0.2254|±  |0.0225|
|  -moral_scenarios                    |Yaml   |none  |acc    |    0.2425|±  |0.0143|
|  -philosophy                         |Yaml   |none  |acc    |    0.1961|±  |0.0226|
|  -prehistory                         |Yaml   |none  |acc    |    0.2716|±  |0.0247|
|  -professional_law                   |Yaml   |none  |acc    |    0.2288|±  |0.0107|
|  -world_religions                    |Yaml   |none  |acc    |    0.2398|±  |0.0327|
| -other                               |N/A    |none  |acc    |    0.2803|±  |0.0490|
|  -business_ethics                    |Yaml   |none  |acc    |    0.2200|±  |0.0416|
|  -clinical_knowledge                 |Yaml   |none  |acc    |    0.2566|±  |0.0269|
|  -college_medicine                   |Yaml   |none  |acc    |    0.2717|±  |0.0339|
|  -global_facts                       |Yaml   |none  |acc    |    0.3000|±  |0.0461|
|  -human_aging                        |Yaml   |none  |acc    |    0.2960|±  |0.0306|
|  -management                         |Yaml   |none  |acc    |    0.2524|±  |0.0430|
|  -marketing                          |Yaml   |none  |acc    |    0.2735|±  |0.0292|
|  -medical_genetics                   |Yaml   |none  |acc    |    0.2700|±  |0.0446|
|  -miscellaneous                      |Yaml   |none  |acc    |    0.2771|±  |0.0160|
|  -nutrition                          |Yaml   |none  |acc    |    0.2288|±  |0.0241|
|  -professional_accounting            |Yaml   |none  |acc    |    0.2695|±  |0.0265|
|  -professional_medicine              |Yaml   |none  |acc    |    0.4118|±  |0.0299|
|  -virology                           |Yaml   |none  |acc    |    0.2771|±  |0.0348|
| -social_sciences                     |N/A    |none  |acc    |    0.2398|±  |0.0390|
|  -econometrics                       |Yaml   |none  |acc    |    0.2193|±  |0.0389|
|  -high_school_geography              |Yaml   |none  |acc    |    0.1970|±  |0.0283|
|  -high_school_government_and_politics|Yaml   |none  |acc    |    0.2487|±  |0.0312|
|  -high_school_macroeconomics         |Yaml   |none  |acc    |    0.2128|±  |0.0208|
|  -high_school_microeconomics         |Yaml   |none  |acc    |    0.2353|±  |0.0276|
|  -high_school_psychology             |Yaml   |none  |acc    |    0.2734|±  |0.0191|
|  -human_sexuality                    |Yaml   |none  |acc    |    0.2137|±  |0.0360|
|  -professional_psychology            |Yaml   |none  |acc    |    0.2500|±  |0.0175|
|  -public_relations                   |Yaml   |none  |acc    |    0.3636|±  |0.0461|
|  -security_studies                   |Yaml   |none  |acc    |    0.1918|±  |0.0252|
|  -sociology                          |Yaml   |none  |acc    |    0.2289|±  |0.0297|
|  -us_foreign_policy                  |Yaml   |none  |acc    |    0.2400|±  |0.0429|
| -stem                                |N/A    |none  |acc    |    0.2623|±  |0.0412|
|  -abstract_algebra                   |Yaml   |none  |acc    |    0.3000|±  |0.0461|
|  -anatomy                            |Yaml   |none  |acc    |    0.2667|±  |0.0382|
|  -astronomy                          |Yaml   |none  |acc    |    0.2434|±  |0.0349|
|  -college_biology                    |Yaml   |none  |acc    |    0.2847|±  |0.0377|
|  -college_chemistry                  |Yaml   |none  |acc    |    0.2300|±  |0.0423|
|  -college_computer_science           |Yaml   |none  |acc    |    0.2800|±  |0.0451|
|  -college_mathematics                |Yaml   |none  |acc    |    0.2700|±  |0.0446|
|  -college_physics                    |Yaml   |none  |acc    |    0.3431|±  |0.0472|
|  -computer_security                  |Yaml   |none  |acc    |    0.2900|±  |0.0456|
|  -conceptual_physics                 |Yaml   |none  |acc    |    0.2979|±  |0.0299|
|  -electrical_engineering             |Yaml   |none  |acc    |    0.2069|±  |0.0338|
|  -elementary_mathematics             |Yaml   |none  |acc    |    0.2619|±  |0.0226|
|  -high_school_biology                |Yaml   |none  |acc    |    0.2581|±  |0.0249|
|  -high_school_chemistry              |Yaml   |none  |acc    |    0.2512|±  |0.0305|
|  -high_school_computer_science       |Yaml   |none  |acc    |    0.2400|±  |0.0429|
|  -high_school_mathematics            |Yaml   |none  |acc    |    0.2630|±  |0.0268|
|  -high_school_physics                |Yaml   |none  |acc    |    0.2517|±  |0.0354|
|  -high_school_statistics             |Yaml   |none  |acc    |    0.2269|±  |0.0286|
|  -machine_learning                   |Yaml   |none  |acc    |    0.2589|±  |0.0416|

|     Groups      |Version|Filter|Metric |  Value   |   |Stderr|
|-----------------|-------|------|-------|---------:|---|-----:|
|mmlu             |N/A    |none  |acc    |    0.2522|±  |0.0407|
|                 |       |      |samples|14042.0000|   |      |
| -humanities     |N/A    |none  |acc    |    0.2349|±  |0.0289|
| -other          |N/A    |none  |acc    |    0.2803|±  |0.0490|
| -social_sciences|N/A    |none  |acc    |    0.2398|±  |0.0390|
| -stem           |N/A    |none  |acc    |    0.2623|±  |0.0412|

Looks good to me!

how to use this pr to evaluate all mmlu tasks? any one provide an example command line for me? I have updated to latest version.

You should be able to run via simply python -m lm_eval --model hf --model_args ... --tasks mmlu!

What are the weights on each subject for the weighted average?

They're weighted based on number of samples.

@lintangsutawika in the latest code, this avg score seems gone.  Any guidance on the latest way to get the mmlu avg score?

Could tell me more what the issue might be? Also maybe in a new issue would be better.

opened an issue: https://github.com/EleutherAI/lm-evaluation-harness/issues/1415

Just fixed in #1414 !


Add autoscalers for dnsmasq and kubedns

By default kubedns and dnsmasq scale when installed.
Dnsmasq is no longer a daemonset, but replicationcluster.
Minimum replicas is two (to enable rolling updates).

ci check this

is it optional ?

is it optional ?
No. However, it is quite light (requests only 10mb memory and 0.02 cores). I think keeping it on isn't a big intrusion.
it adds an extra moving part and autoscaller is in early dev.
It's an officially suggested addon: https://github.com/kubernetes/kubernetes/tree/master/cluster/addons/dns-horizontal-autoscaler
Also, this project has been in dev since June 2016 https://github.com/kubernetes-incubator/cluster-proportional-autoscaler/ . It's not exactly brand new and really brings value for those who want to go into production.

Also I think dnsmasq as daemonset was intentional, cc @bogdando ?
dnsmasq as daemonset predates @bodando and myself. The intention here is to create a sensible number of dnsmasq instances because 1 per node is clearly too many and adds little value.


CSV as the default Export Profile

With #74 Implemented, CSV can now the default export profile.

Done. Will be in 0.3.2.


This item may not have a label readable by screen readers.

Godot version
3.5.1
System information
Windows 10
Issue description
Hi,
I have 4 accessibility warnings in Google Play Console "Pre-launch report".

Element path: <package_id>:id/godot_fragment_container/FrameLayout[1]/EditText[3]
Recommendation: This item may not have a label readable by screen readers.

Element path: <package_id>:id/godot_fragment_container/FrameLayout[1]/GodotEditText[1]
Recommendation: This item may not have a label readable by screen readers.

Element path: <package_id>:id/godot_fragment_container/FrameLayout[1]/GodotView[2]
Recommendation: This item may not have a label readable by screen readers.

Element path: <package_id>:id/godot_fragment_container/FrameLayout[1]/View[1]
Recommendation: This item may not have a label readable by screen readers.

I tried changing the first page and testing it again, but the warnings still appear. Do you have any ideas? Anyone else having this problem?
Steps to reproduce
Export your app for Android -> upload to Google Play Console -> see "Pre-launch report"
Minimal reproduction project
Do not have. Will try to create if needed.

Godot UI doesn't support screen readers, so probably nothing can be done here.
Adding support for 4.x is in progress - https://github.com/godotengine/godot/pull/76829, but AccessKit library currently does not support Android, iOS and Web, so it will work only on desktop platforms.

@bruvzg I see. So these warnings is a "normal behaviour" for now, right? Should I just ignore them?

Should I just ignore them?

Yes.

Should I just ignore them?

Yes.

Ok, thanks :)

I guess this is resolved then?

but how to ignore warnings? my app is not yet published and i dont see any option to ignore the warning and proceed with testing


Use thrust to generate the coarse of amgx_pgm

This pull request eliminate the usage of spgemm in coarse grid generation of amgx_pgm
After pull request, the transposed is only needed by the weight matrix computation.
If the matrix is hermitian, the transpose can also be ignored.

rebase!

@pratikvn there are some result. The performance is not so good, and the generation should be improved more in the future. This version is mainly to avoid the SpGEMM calls such that we can implement it in distributed version more easily.
L-shape
original: 82 80 80
updated: 65 65 64
beam
original: 80 80 74
updated: 74 72 72

@yhmtsai, thanks for checking. I think that is still a significant improvement.


SFDX commands using PollingClient throws "CLIError: timed out"

Summary:
I'm currently on sfdx-cli/7.206.6 darwin-x64 node-v18.15.0
While using PollingClient (salesforce/core) module, the command times out in most of the newer SFDX versions.
Although the other operation continues to work, but this error displays on the terminal.
CLIError: timed out
at Object.error (/Users/s***/.local/share/sfdx/client/7.206.6-140cac7/node_modules/@oclif/core/lib/errors/index.js:27:15)
at /Users/s***/.local/share/sfdx/client/7.206.6-140cac7/node_modules/@oclif/core/lib/cli-ux/index.js:29:62
at flush (/Users/s***/.local/share/sfdx/client/7.206.6-140cac7/node_modules/@oclif/core/lib/cli-ux/index.js:121:9) {
oclif: { exit: 2 },
code: undefined
}
It is working fine for v7.179.0 but not for v7.192.0
Not sure about the versions in between this range and when it actually stopped working.
Steps To Reproduce:
TBD
Expected result
It should not time-out.
Actual result
For newer versions, it times out in under a minute.
Additional information: N/A
OS and version: N/A

Hello @SakshiV19 👋 It looks like you didn't include the full Salesforce CLI version information in your issue. Please provide the output of version --verbose --json for the CLI you're using (sf or sfdx).
A few more things to check:

Make sure you've provided detailed steps to reproduce your issue.

A repository that clearly demonstrates the bug is ideal.

Make sure you've installed the latest version of Salesforce CLI. (docs)

Better yet, try the rc or nightly versions. (docs)

Try running the doctor command to diagnose common issues.

Search GitHub for existing related issues.

Thank you!

Information provided above.

@SakshiV19 - do you have steps to reproduce this?  Lots of commands rely on PollingClient and if this was a widespread issue we'd have a fire on our hands.  E.g., deploying metadata and creating scratch orgs both rely on PollingClient.  This is likely specific to something in your environment.

We are also using this while deploying metadata and then we poll for the results of the deployment job in our custom CLI solution. This is where PollingClient is being used.
Although the operation continues to poll even after getting the below error, but every time the wait parameter is used, this error is displayed on the screen :
CLIError: timed out
at Object.error (/Users/s***/.local/share/sfdx/client/7.206.6-140cac7/node_modules/@oclif/core/lib/errors/index.js:27:15)
at /Users/s***/.local/share/sfdx/client/7.206.6-140cac7/node_modules/@oclif/core/lib/cli-ux/index.js:29:62
at flush (/Users/s***/.local/share/sfdx/client/7.206.6-140cac7/node_modules/@oclif/core/lib/cli-ux/index.js:121:9) {
oclif: { exit: 2 },
code: undefined
}

is there some code you can share of your implementation?

Hi @mshanemc My implementation of poll()  doesn't have try, catch yet, which might be a reason for this error. I am working on the same. I will get back in case it doesn't work. Thank you!


wrong filename in content-disposition when using res.sendFile()

here is my code:
var path = require('path');
var fs = require('fs');
var app = require('express')();
var port = 4001;
app.get('/:filename', function(req, res) {
  var filePath = path.join(__dirname, 'code.jpg');
  res.setHeader("Content-Type","image/jpeg");
  res.setHeader("Content-Dispositon","attachment; filename=" + "test.jpeg");
  res.sendFile(filePath);
});

app.listen(port, () => {
	console.log(`server start at ${port}`);
}

when i type url like: http:// my machine ip:4001/test on browser address area, it will show the image .
but when I right click the image through my mouse, and choose save as image item, it doesn't show the correct name defined in content-disposition.

as the picture show, the response header content-diposition's filename is test.jpeg, but the download dialog show other name.
how should I solve the problem?

Hi @zhuyujiess, since the header is correctly being sent to the client, I am pretty sure this is not an express issue.  According to the documentation on MDN, it looks like you might need to wrap the filename in quotes.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Disposition
If you need any more help, and it is not an express specific issue (like this), please try to ask on Stack Overflow or in the express Gitter before posting here, as this is more for bug reports or feature development.  Thanks!

Your example code above
app.get('/:filename', function(req, res) {
  var filePath = path.join(__dirname, 'code.jpg');
  res.setHeader("Content-Type","image/jpeg");
  res.setHeader("Content-Dispositon","attachment; filename=" + "test.jpeg");
  res.sendFile(filePath);
});

Can be entirely replaced with the res.download API which will also property construct the Content-Disposition header for you:
app.get('/:filename', function(req, res) {
  var filePath = path.join(__dirname, 'code.jpg');
  res.download(filePath, 'test.jpeg');
});

Just as an aside, when you set "Content-Disposition" in the header, there was a typo.


Spanner: Dead session is reused in php-cli

Hi, we got infinite loop when using cloud-spanner client in php-cli.
When client keeps running (e.g., job worker running in php-cli) and server close a session, "session not found" error is thrown by CacheSessionPool.
Client retries the connection when this error is thrown, but the dead session is reused since Database class keeps $this->session although session is dead.
Because Database keeps $this-session, client loops retrying forever.
Expected Behavior
When "session not found" error is thrown, Database should discard $this->session and get new session.
Environment

OS: alpine 6.4.0 (Docker)
PHP version: 7.2.16
Package name and version:

google/cloud-spanner: v1.11.5

Reproduction
Code example
    while (true) {
        try {
            $row = $db->execute('SELECT "Hello, world"')->rows()->current();
            printf('%s' . PHP_EOL, json_encode($row));
        } catch (\Throwable $ex) {
            printf('error: %s'.PHP_EOL, $ex->getMessage());
        }
        sleep(1);
    }

In case server close sessions when running this script, the situation is reproduced.
Steps to reproduce

Run this script

$ php reproduce.php

Delete server session

$ gcloud spanner databases sessions list --database=xxxx --instance=xxxxx --uri | xargs -n1 gcloud spanner databases sessions delete

Output
["Hello, world"]
["Hello, world"]
["Hello, world"]
["Hello, world"]
error: {
    "message": "Session not found: ....
error: {
    "message": "Session not found: ....
error: {
    "message": "Session not found: ....

Error Message
Google\Cloud\Core\Exception\NotFoundException: {
    "message": "Session not found: projects\/[project_name]\/instances\/[instance_name]\/databases\/[db_name]\/sessions\/AJSwhASM95I1gTIBDN9kruMXIY8gb-B3A6Cuk7WU6prwFNvWu5oIFaEt3zS1Ug",
    "code": 5,
    "status": "NOT_FOUND",
    "details": [
        {
            "@type": "google.rpc.resourceinfo-bin",
            "resourceType": "type.googleapis.com\/google.spanner.v1.Session",
            "resourceName": "projects\/[project_name]\/instances\/[instance_name]\/databases\/[db_name]\/sessions\/AJSwhASM95I1gTIBDN9kruMXIY8gb-B3A6Cuk7WU6prwFNvWu5oIFaEt3zS1Ug",
            "owner": "",
            "description": "Session does not exist."
        },
        {
            "@type": "grpc-status-details-bin",
            "data": "<Unknown Binary Data>"
        },
        {
            "@type": "grpc-server-stats-bin",
            "data": "<Unknown Binary Data>"
        }
    ]
} in /src/vendor/google/cloud-core/src/GrpcRequestWrapper.php on line 263

Hi @RY-2718,
How often does the loop repeat? Spanner sessions are automatically closed when they have been idle for one hour. One workaround for your issue is to issue a SELECT 1 query periodically to ensure the session remains alive between iterations of your worker.
We'll discuss your report and get back to you!

@jdpedrie I'm observing this issue from times to times saying "session not found" for 2 days now. The session is not idle for 1 hour as you say, maybe 5 seconds at most. Any idea?

Thank you for the report. Is this script executed with/without a session pool? I'm a little apprehensive of the idea of automatically pulling a new session if the one currently tied to the database becomes expired. While highly unlikely, it seems possible this may open us up to a case where we run into a infinite loop where we keep trying to fetch a new session. I'm currently leaning towards handling this in userland code like so:
use Google\Cloud\Core\Exception\NotFoundException;

while (true) {
    try {
        $row = $db->execute('SELECT "Hello, world"')->rows()->current();
        printf('%s' . PHP_EOL, json_encode($row));
    } catch (NotFoundException $ex) {
        printf('Session not found, releasing so a fresh one can be acquired on the next loop.' . PHP_EOL);
        $db->close();
    } catch (\Throwable $ex) {
        printf('error: %s'.PHP_EOL, $ex->getMessage());
    }
    sleep(1);
}

WDYT?

@RY-2718 do you have any thoughts on @dwsupplee's previous comment?

We stopped seeing this error a while ago.
Perhaps it was a bug on the server end?

Glad to hear! If it pops up again, let us know.


Fix Allow Unsafe Code on ARM in SessionManagerUniversal.UI

To resolve part of #21
Thanks @jwittner for pointing it out

Looks good to me, I'll leave it here until tomorrow in case anyone has any qualms, then pull it in.


Releases closure-library to bower.js

You're going to have to be more specific. What exactly is this issue? I'm assuming bower is http://bower.io/?

Yes, I means http://bower.io/search/
I want to install closure library with
bower install closure-library
or something alike

Sorry, I don't think this is something we want to maintain at this time. Even if we did, it would never be prioritized as there's so much else higher in the priority list.


problem with VocabAugmentor

There seems to be a problem with the VocabAugumentor class.
When I run:
new_tokens = augmentor.get_new_tokens(ft_corpus_train)
I get this error:
TypeError: Can’t convert <tokenizers.trainers.PreTrainedTokenizerFast object at 0x7f8641325570> to Sequence
Solution: I resolved this problem by switching orders of arguments on line 91 in vocab_augmentor.py. (this was a suggested fix for the above mentioned error, from a quick google search)
From:
self.rust_tokenizer.train(self.trainer, train_files)
to:
self.rust_tokenizer.train(train_files, self.trainer)
After this change program runs as expected.
I didn't look into this problem - as the quick fix solution worked - but you may want to have a look if there are no bugs related to that.

I had a similar issue, it fixed my problem :)

My issue was: TypeError: Can't convert <tokenizers.trainers.WordPieceTrainer object at 0x17ceef250> to Sequence

You will need to download class VocabAugmentor(BaseEstimator): from SBERT website and change
From:
self.rust_tokenizer.train(self.trainer, train_files)
to:
self.rust_tokenizer.train(train_files, self.trainer)


Update quick start doc with current state of affairs:

Description

Update quick start doc with current state of affairs, as the quick start didn't work anymore with the latest changes.
Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

@mergify requeue


Add valid and expires dates to metadata

Not at all a priority, but when you're doing the next bit of maintenance to the code, it would be great if the metadata included the date that the chart becomes valid / active, and the date that it expires. Something like:
valid|2024-10-31
expires|2024-11-28

Another fixed value that would be useful to have in the metadata is the scheme. Chartmaker seems to use the TMS scheme so:
scheme|tms

Added valid date, expire date, and scheme fields to the metadata table.

P.S. - now updating the docker image.


Better handle not having a config file

Fixes: #1406
Signed-off-by: Spencer Krum<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
When there is a config file that can't be read and a command line config set, that can create a seg fault. Instead of supporting the use case where the user doesn't have a config file, force the user to provide a config file and stop supporting a no-config-file case.
Which issue(s) this PR fixes:

Fixes # 1406
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Users who run falco without a config file will be unable to do that any more
Developers may need to adjust their processes

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign mstemm after the PR has been reviewed.
You can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: a2b6bc20469674478f2d0fc3575f8b15ddeff8d1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.27.0

Updated the release notes indicating that this is a breaking change.


[Errno 13] Permission denied

Hi guys, i get this issue when trying to using dictionaries with the node. I've looking for a fix but can't find any solution. Btw thank you so much for all you work guys, you're precious <3
2024-09-25 17:39:33,027 - root - ERROR - Traceback (most recent call last):
  File "D:\AImatrix\Data\Packages\ComfyUI\execution.py", line 323, in execute
    output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "D:\AImatrix\Data\Packages\ComfyUI\execution.py", line 198, in get_output_data
    return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "D:\AImatrix\Data\Packages\ComfyUI\execution.py", line 169, in _map_node_over_list
    process_inputs(input_dict, i)
  File "D:\AImatrix\Data\Packages\ComfyUI\execution.py", line 158, in process_inputs
    results.append(getattr(obj, func)(**inputs))
  File "D:\AImatrix\Data\Packages\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 11077, in load_file
    file_list = self.TextFileLoader(file_path, label)
  File "D:\AImatrix\Data\Packages\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 11100, in __init__
    self.load_file(file_path)
  File "D:\AImatrix\Data\Packages\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 11111, in load_file
    with open(file_path, 'r', encoding="utf-8", newline='\n') as file:
```PermissionError: [Errno 13] Permission denied: 'D:\\AImatrix\\Data\\Packages\\ComfyUI\\dictionaries'
2024-09-25 17:39:33,028 - root - INFO - Prompt executed in 0.01 seconds

Does python have write access to the file? Do you have the file open by another ComfyUI or process?

Does python have write access to the file? Do you have the file open by another ComfyUI or process? The error means it's either open already a locked by that process, or the user executing Python doesn't have read/write to the location.

I don't know, I used dictionaries a few months ago, but not anymore, I don't understand why. But that was before comfyui v2, maybe that's where it comes from, how can I check if write access is enabled?

I have also encountered the same problem. I have tried to open and modify file permissions through administrator privileges, but still cannot solve it. Please find a solution. Thank you！！！

That won't solve the problem, as the process you are running from (ConfyUI) is the one lacking permissions


docs-bug(Portal): Provide examples on how to use @Input/@Ouput

Documentation Feedback
Right now, they are no example showing how to use @Input() or @Output() for a component attached using the portal cdk.

I don't quite understand what the example would be. Inputs and outputs should work in the exact same way as for non-portal components.

You can find more information here:
https://stackoverflow.com/questions/47469844/angular-cdk-how-to-set-inputs-in-a-componentportal

For me the last answer seams very interesting, especially as there is a lot of hidden potential.
The point I don't really understand is why the typecasting is needed.
setInputs(portalOutletRef: CdkPortalOutletAttachedRef) {
portalOutletRef = portalOutletRef as ComponentRef<BaseAuditView>;

portalOutletRef.setInput('prop1', this.prop1);
portalOutletRef.setInput('prop2', this.prop2);

With the new setInput option I would expect that Portal could provide an easy way to utilize this function.


Integration with Unity

@devtrends
Would you be interested to merge your libraries with official Unity Container organization and take ownership of webapi and mvs projects?
I have very limited knowledge of both technologies and would really appreciate help with both.

I guess not

Sorry I missed this. I am not opposed to a merge but I am ridiculously busy right now so would not have the time to take ownership. My projects are very little code and are not going to be updated to .NET Core so they currently require minimal work on my part - generally only when Unity introduces a breaking change ;-)


Brandon's chapter contribution

Here is my chapter in content/06. This is the first time I am using GitHub, so I hope I am not messing things up ;)

This is a super beautiful and well-written chapter. Thank you, @brandonleighpanos!
Here are some (minor) comments:

I think we can get rid of the .DS_Store files. There are in HelioML/content/06/ and also in HelioML/content/06/01/.
Please link to your ApJ paper and Github repository (with the Zenodo DOI) in the intro.md file. You can follow the example here.
Please do the same at the beginning of your notebook -- maybe in the paragraph that says "This chapter draws heavily..." (to give people maximual opportunity to cite you 😃).
I know we've been a little inconsistent with formatting, but I think we will use everyone's first and last name for the chapter authors -- so can you please change B. Panos to Brandon Panos? I will also add your name to the title page as well.
Do you mind linking some of the URLs to text (just for it to look a little cleaner)? For example:

Previously: For the details of this version, we refer the reader to the source code https://scikit-learn.org/stable/modules/generated/sklearn.cluster.MiniBatchKMeans.html
Instead: For the details of this version, we refer the reader to the scikit-learn source code.

I tried to sort out the references in the HelioML book (by including a bibliography using jekyll-scholar) but it is messed up right now and I don't want to hold up this PR. Can you please open an issue on the HelioML repo saying that the references in your book are incorrect and should be fixed? That way I will keep track of this task.


Wrong recipient, payload audience != requiredAudience

I have integrated Google sign in and getting Token from it, now when i vertify the token, i always get this error. I tried many times but nothing works, and i always get this error
Error: Wrong recipient, payload audience != requiredAudience
    at OAuth2Client.verifySignedJwtWithCerts (/home/XXX/node_modules/google-auth-library/lib/auth/oauth2client.js:499:23)
    at OAuth2Client.<anonymous> (/home/XXX/node_modules/google-auth-library/lib/auth/oauth2client.js:361:31)
    at /home/XXX/node_modules/google-auth-library/lib/auth/oauth2client.js:407:13
    at Request._callback (/home/XXX/node_modules/google-auth-library/lib/transporters.js:113:17)
    at Request.self.callback (/home/XXX/node_modules/request/request.js:186:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (/home/XXX/node_modules/request/request.js:1163:10)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)
    at IncomingMessage.<anonymous> (/home/XXX/node_modules/request/request.js:1085:12)
    at Object.onceWrapper (events.js:313:30)
    at emitNone (events.js:111:20)
    at IncomingMessage.emit (events.js:208:7)
    at endReadableNT (_stream_readable.js:1056:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)

I Followed this document => https://developers.google.com/identity/sign-in/web/backend-auth
the Code i am using to verify
var GoogleAuth = require('google-auth-library');
var auth = new GoogleAuth;
var client = new auth.OAuth2(CLIENT_ID, '', '');
client.verifyIdToken(
    token,
    CLIENT_ID,
    // Or, if multiple clients access the backend:
    //[CLIENT_ID_1, CLIENT_ID_2, CLIENT_ID_3],
    function(e, login) {
      var payload = login.getPayload();
      var userid = payload['sub'];
      // If request specified a G Suite domain:
      //var domain = payload['hd'];
    });

Greetings!  Could you take a look at this sample, and see if anything pops out?
https://github.com/google/google-auth-library-nodejs/pull/219/files#diff-62502f095646cbe2651801cf13671479
In your code - where are you getting the value token from?  Could you share that?
In taking a look at your error message - we're able to parse the token, we're pulling out the fields in the payload, and the audience field just doesn't seem to match.  What kind of a keyfile are you using to acquire the audience / CLIENT_ID?  Is it using service account credentials, or OAuth2?


up(linter): add rule fixer

Adds a new RuleFixer struct that gets passed to ctx.diagnostic_with_fix.
This PR is half proposal and half implementation. I'd love thoughts and feedback on its design (including whether or not it adds enough benefit to make it worth merging this PR).

Definitely a better API!


New Feature Request to Group Applications

Can you add a feature where we can group applications together in a window without creating virtual desktop?
Virtual desktops slow down my laptop so I would like you to implement a feature like Groupy. It would enhance the productivity and help in keeping organized.
For example, I am playing games and have opened discord. Also, at the same time I am working something else like MS Word and MS Powerpoint and need to research on the Edge at the same time. It would be very helpful If I could create a new Group where I could put discord and game in and another group where I could put word, ppt, edge in. I would be able to switch between the two with ease and keep free from clutter.
Hope to this feature implemented :)
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@KaavayGupta
if you are asking for just tabbing, we already have https://github.com/microsoft/PowerToys/issues/10
If you are asking for something more, consider that Microsoft is committed to improve the Virtual Desktop experience and PowerToys will not try to duplicate it.
For the performance issue you have with Virtual Desktop, I suggest you send your feedback using the Windows 10 Feedback Hub application.

Hi, thanks for the reply. Yes, tabs is a feature that I wanted. But I can't
find it in the PowerToys though. I have version 0.31.2. Is it available to
only some selected devices?
On Fri, Feb 26, 2021 at 3:52 PM Enrico Giordani<EMAIL_ADDRESS>wrote:

@KaavayGupta https://github.com/KaavayGupta
if you are asking for just tabbing, we already have #10
https://github.com/microsoft/PowerToys/issues/10
If you are asking for something more, consider that Microsoft is committed
to improve the Virtual Desktop experience and PowerToys will not try to
duplicate it.
For the performance issue you have with Virtual Desktop, I suggest you
send your feedback using the Windows 10 Feedback Hub application.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/9909#issuecomment-786555617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCNJZDMBIPRTRO7ZWDTGCTTA5Y5BANCNFSM4YIBKJUA
.

@KaavayGupta
I meant we already have an issue tracking the tabbing feature https://github.com/microsoft/PowerToys/issues/10
The feature is not implemented in PowerToys.
Closing as duplicate of https://github.com/microsoft/PowerToys/issues/10


Investigate the impact of HashiCorp license MPL -> BUSL

This issue is for tracking the usage of HashiCorp Go packages and software products in the terraform-provider-carvel project.
The CNCF is tracking the impact of the HashiCorp license change in https://github.com/cncf/foundation/issues/617 and they're collecting the list of affected projects in https://github.com/cncf/foundation/issues/619

References:

CNCF Policy for allowed third party licenses
CNCF Source Available recommendations

It looks like we do have these libraries that we are importing from HashiCorp

Due to changes made by Hashicorp to the license of some of their products, I would like to provide some recommendations to the Reviews/approvers of all the tools so they do not impact Carvel.

If a dependabot PR bumps any Hashicorp libraries, we should not merge it. (We can discuss bumping some of these libraries in the future if we are sure no changes will happen to their licenses.)
If a PR unrelated to Hashicorp bumps some Hashicorp library, check if the version of the Hashicorp library is the Apache License; if it is, we can go ahead and merge the PR.
If a PR unrelated to Hashicorp bumps some Hashicorp library, check if the version of the Hashicorp library is the Apache License, and if it is NOT, the PR should NOT be merged. We should bring this up in the community meeting to see if any mitigation factor can be done or if we should move to a different library.
Every PR that bumps dependencies in kapp-controller, especially sops and helm, should be vetted using the above. steps

These are our initial recommendations, but we will bring this up again during the next community meeting, and we could have a more in-depth conversation. In the meantime, feel free to add any thoughts on this issue.


[Enhancement] Added ARIA specification of First and Last option highlight

Addresses https://github.com/paypal/downshift/issues/589
What:
Added highlight on first and last options when user opens the dropdown via Up/Down Arrows.
Why:
According to ARIA, if dropdown is collapsed and focus is on the text input:

Arrow Down should open menu and highlight on option.
Arrow Up should open menu and highlight last option.

How:
Modified moveHighlightedIndex in dropdown.js, specifically the else branch. There I open the menu, and then check if the opening action was made by Arrow up or down. I also check if there are any items in the dropdown.
If yes, I call setHighlightedIndex with either itemCount - 1 or 0, depending on the value of the event type (arrow up or arrow down), in order to comply to ARIA specifications.
I also modified some unit tests and end-to-end tests in order to reflect the changes. Also added 2 more end-to-end tests that test only this new behaviour.
Checklist:

[ ] Documentation
[ ] Tests
[ ] Ready to be merged

[ ] Added myself to contributors table

Oh, and looks like some cypress tests are failing on CI: https://travis-ci.org/paypal/downshift/builds/437712230

Thank you for the quick feedback.
Those tests are the ones I added, will take a look and find out what the problem is.
Also will address the other comments and come back to you once the PR is polished.

I'm going to manually merge this because I'm doing weird things. Standby...

Sweet. Hopefully I can get this released in the major release this afternoon.

Let's keep it open until it's actually released 👍

This has been released in 3.0.0!


Spark is unable to write to Elasticsearch

Hello,
We are  using  SG 1.7  .  authorized users are able to create indexes manually .
We  have spark jobs that will create indexes every day and  those jobs are failing by not able to create indexes. we are getting below exception. Is this the issue with search Guard?
[RemoteTransportException[[es-test-master][inet[/hostname:9300]][indices:admin/create]]; nested: RuntimeException[Unauthenticated request (SEARCHGUARD_UNAUTH_REQ) for action indices:admin/create]; ]]
at com.xxx.spark.ElasticsearchClientLib.ESClient.call(ESClient.java:136)
at com.xxx.spark.ElasticsearchClientLib.ESClient.call(ESClient.java:17)
at org.apache.spark.api.java.JavaRDDLike$$anonfun$foreachPartitionAsync$1.apply(JavaRDDLike.scala:678)
at org.apache.spark.api.java.JavaRDDLike$$anonfun$foreachPartitionAsync$1.apply(JavaRDDLike.scala:678)
at org.apache.spark.SparkContext$$anonfun$37.apply(SparkContext.scala:1992)
at org.apache.spark.SparkContext$$anonfun$37.apply(SparkContext.scala:1992)
at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:66)
at org.apache.spark.scheduler.Task.run(Task.scala:89)
at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:214)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

Auto close this issue because it refers to Search Guard 1 which is decommissioned now. For details pls. see https://github.com/floragunncom/search-guard-attic/wiki . If this issue is still valid for Search Guard 2 (for elasticsearch 2.x) feel free to reopen it (or to file a new one). Pls. do not longer report issues for Search Guard 1 (for elasticsearch 1.x). We sincerely apologize for the inconvenience this may have caused. Thanks for contributing!


log 50% of GitHub rate limit

It's no good to find out about throttling after the fact. We need to be notified well in advance so we can plan for it. We should log errors to Sentry when we hit 50%, 80%, and 95% of our rate limit.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Closing in light of our decision to shut down Gratipay.
Thank you all for a great run, and I'm sorry it didn't work out! 😞  💃


Error when calling an exposedFunction from within page.evaluateOnNewDocument

Steps to reproduce
Tell us about your environment:

Puppeteer version: 3.3.0
Platform / OS version: Mac OS
Node.js version: 14.0.0

What steps will reproduce the problem?
Calling an exposed function immediately inside evaluateOnNewDocument causes a JSON parsing error.
Please include code that reproduces the issue.
page.exposeFunction('foo',function(bar){ console.log(bar) })
page.evaluateOnNewDocument(function(){ window.foo('bar') })

What is the expected result?
"bar" is emitted from stdout
What happens instead?
An error is thrown when Page._onBindingCalled attempts to parse the value passed to the exposed function.
SyntaxError: Unexpected token b in JSON at position 1
    at JSON.parse (<anonymous>)
    at Page._onBindingCalled (/****/node_modules/puppeteer/lib/Page.js:405:42)
    at CDPSession.<anonymous> (/****/node_modules/puppeteer/lib/Page.js:126:60)

Note that awaiting page.exposeFunction seems to prevent this error, but I'm not sure if this is guaranteed.
await page.exposeFunction('foo',function(bar){ console.log(bar) })
page.evaluateOnNewDocument(function(){ window.foo('bar') })

Also note that deferring the call to the exposed function does not trigger the error.
page.exposeFunction('foo',function(bar){ console.log(bar) })
page.evaluateOnNewDocument(function(){
  setTimeout(window.foo('bar'),0)
})

Hi! Try to add a line with/* istanbul ignore next */ before page.evaluateOnNewDocument

Hi! Try to add a line with/* istanbul ignore next */ before page.evaluateOnNewDocument

I don't see how this is related to this particular issue.

Also getting this

Also at 4.0.0

Can confirm at 4.0.0 that awaiting it resolves it, but it's weird when doing one-time page.exposeFunction + page.evaluate


Discussion: User Profiles

Never implemented. Users need a place to update their password and other information. They can also provide historical data, such as archived projects, past notifications listed and other data associated with the user.
Profiles could be accessible by other users if there is value in that. They should also be more than just a form to edit user information.
This is a perfect example of not creating a CRUD page for the user edit form. No user should be allowed to change their role through this interface. Updating a password should require confirmation.
Suggestions

Implement
Careful attention paid to not allowing updates of off-limit values.

ooh ooh time to build a profile state (maybe off that "edit profile", though maybe rename, in the user dropdown or on profile icon click).
I don't completely follow the users introducing historical data aspect, how is that related to a user profile?  OR does the envisioned extend of user profiles extend beyond basic information about a user, rather a hub to manage/represent the extent of a users interaction with the JF instance (manage if you have the proper permissions, represent if you are specific user)?

So we'll stand up a new "profile" state, make the access point through the welcome dropdown (though the wording of which will be renamed from "Edit Profile" to just "Profile").  As with the most other pages, the profile landing state will display all relevant user information.  If there is a distinct dicatomoy in information desire to be displayed, maybe have an "Edit" button for each zone rahter than a catch all on the top.
I don't completely follow proposition of allowing users to contribute historical data, this concept doesn't feel related to a user profile.  Or does the envisioned concept of user profiles extend beyond basic information about a user (name email pass etc), rather a hub to manage/represent the extent of a users interaction with the JF instance (manage if you have the proper permissions, represent if you are common user)?  Within the ability to add users to a group already exists as a substate int he "admin" state.


Gcloud: exec: python: not found

Describe the bug
I am on the latest lens version from homebrew (5.3.4) and currently when I try to connect to my GKE cluster I get:
E0124 22:37:06.401361 32006 proxy_server.go:147] Error while proxying request: error executing access token command "/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud config config-helper --format=json": err=exit status 127 output= stderr=/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 69: readlink: command not found
/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 83: dirname: command not found
/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 69: readlink: command not found
/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 162: exec: python: not found

error executing access token command "/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud config config-helper --format=json": err=exit status 127 output= stderr=/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 69: readlink: command not found
/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 83: dirname: command not found
/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 69: readlink: command not found
/usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud: line 162: exec: python: not found

I checked that python is in my path, and I can happily run  /usr/local/Caskroom/google-cloud-sdk/latest/google-cloud-sdk/bin/gcloud config config-helper --format=json and I see the config in json format on my terminal.
Does anyone have any ideas what is going wrong and what else I should be checking?
Environment:

Lens Version: 5.3.4
OS: [e.g. OSX] macOS Monterry
Installation method (e.g. snap or AppImage in Linux): brew

All the best,
Dario

I am also experiencing the same issue. Can someone please provide instructions as to how to downgrade? I wasn't having this problem until I installed 5.3.4 and Lens is now unusable for me.
Environment:
Lens Version: 5.3.4
OS: [e.g. OSX] macOS Monterry
Installation method (e.g. snap or AppImage in Linux): .dmg file installer

Same issue here:
Environment:
Lens Version: 5.3.4
OS: macOS Monterey 12.0.1
Installation method :.dmg file installer

I am no longer seeing this issue having upgraded to Lens: 5.4.0-latest.20220301.1.

So far works for me as well, I will wait couple more days that it does not come up again and then will close the ticket

Closing this ticket as it seems fixed in latest version, thanks everyone!

I am still having this issue on 5.4.1-latest.20220304.1
E0304 18:51:24.710725 65951 proxy_server.go:147] Error while proxying request: error executing access token command "/google-cloud-sdk/bin/gcloud config config-helper --format=json": err=exit status 127 output= stderr=google-cloud-sdk/bin/gcloud: line 70: readlink: command not found
/google-cloud-sdk/bin/gcloud: line 84: dirname: command not found
/google-cloud-sdk/bin/gcloud: line 70: readlink: command not found
/google-cloud-sdk/bin/gcloud: line 190: exec: python: not found

@narner90 and others who might have the same issues:

Sometimes due to zsh environment not fully loaded when restarting the system, and Lens runs on startup (MacOS). You need to force quit Lens from System monitor and start again.

@xgenvn that fix does not work for me. I still get that error on 5.4.6-latest.20220428.1 after force quitting.

Same issue
readlink: command not found
line 192: exec: python: not found
line 84: dirname: command not found

and so on Lens: 5.5.3-latest.20220602.2

This really needs to be re-opened. I encounter this issue now every time a new Lens version is available, which basically makes every upgrade a forced upgrade.
Considering what's just happened over here, I think the devs should be taking a long hard look at this.

yep totally, its basically unusable now, just randomly disconnects

Same here with version 6.0.2-latest.20220908.1

I use fish, my workaround is use the default bash, and when i use the terminal the first thing i do is type fish to start a session.
With the default bash Lens can use gcloud binaries properly.
I think it should work with zsh and so on. Hope it helps.


Expose current state of the client & make scalar configurable

Summary:

expose current state of the client
make scalar configurable

Thanks for reviewing 👍


(Question) returning existing angular frontend

Hi, I would like to start using this framework for a small project im working on. I already have an angular frontend. I want to serve this angular app from pippo on the base url e.g.
GET("/", (routeContext) -> routeContext.send("index.html")); (with the js and css files)
I've looked at the documentation but I don't seem to understand.
Can someone help me?
Thanks in advance

Take a look at angular integration demo [1]. If you don't understand leave a comment
[1] https://github.com/decebals/pippo-demo/tree/master/pippo-demo-crudng

I looked at the demo. But the angular app in that demo is not what im looking for. I don't want to use the template language and angular at the same time and reder it server side, I just want it to serve a single html file with all it's dependencies (js & css) without having to use webjarsAt etc.

First step is to take a look at Static files [1] section from documentation.
The approach with webJarsAt is optional of course. You can use the classic approach with put all static artifacts in a folder (default the folder is public) and split these artifacts in folders: css and js.
➤ tree src/main/resources/public
src/main/resources/public
├── css
│      └── style.css
├── fonts
└── js
└── app.js
Then in your Application (for example in onInit method) or directly from Pippo you must "mount" the "public" folder with the call of method addPublicResourceRoute.
After this, if you don't want to use the publicAt directive (and implicit a template engine in your project)  you must check if your resource is seen by Pippo as public resource.
So, variant with publicAt:
<head>
    <link href="${publicAt('css/style.css')}" rel="stylesheet">
</head>

Variant without publicAt:
<head>
    <link href="public/css/style.css" rel="stylesheet">
</head>

Now it's time to serve your static html. A possible solution can be:
// send 'public/welcome.html' as response
GET("/", (routeContext) -> {
    try {
        String html = IoUtils.toString(getClass().getResourceAsStream("/public/welcome.html"));
        routeContext.send(html);
    } catch (IOException e) {
        throw new PippoRuntimeException(e);
    }
});

It's also possible to serve static file without "/public" prefix in the URL request. For this, instead of
addPublicResourceRoute()

you must use
addPublicResourceRoute("/")

If you need to serve only some artifact types( for example css and js) from root ("/") and the rest from public ("/public") please read [2].
Pippo is very versatile, you have the freedom to chose your way to resolve a specific problem.
Did what I say make sense for you?
[1] http://www.pippo.ro/doc/static-files.html
[2] https://groups.google.com/d/msg/pippo-java/P9XlmtrmaxA/h7ttPxltAwAJ

This does answer my question!
Thank you very much!


WIP: Refactor config and related stuff

Fixes #3466.
What technically changed:

Use pydantic to create a new ManimConfig class, providing built-in type hints, validation. This eliminates the need to define properties for each parameter.
Remove the need to have a default.cfg file where defaults were stored. Instead, default values are set on the ManimConfig class directly (making it easier to keep in sync).
Fortunately, most of the existing config options were standard types (meaning supported out of the box by Pydantic). ManimColor was the only exception, so a custom annotation was created (see docs for more details; TLDR the ManimColor class in untouched and not tied to Pydantic in any way, the custom annotation simply provides info about how to validate/parse input, and how JSON schema should be generated).

What functionally changed:

Decouple the config model from file parsing. To do so, an abstract ConfigProvider class was added, exposing a get_config method. Each subclass of ConfigProvider is responsible for parsing a specific config format (for now, only CfgProvider is implemented and provides an instance of ManimConfig parsed from .cfg files).
In the future, users will be able to use other file formats such as TOML (the new standard in Python). Unlike cfg, TOML is well-specified and easier to parse because of the built-in types it supports. Thanks to the use of Pydantic, a JSON schema can be generated and used in IDEs to provide autocomplete when writing configuration files.
The structure of the ManimConfig class has slightly changed. Mainly:

Before
After

ffmeg_loglevel
ffmpeg.loglevel

ffmpeg_executable
ffmpeg.executable

verbosity
logging.verbosity

media_embed
jupyter.media_embed

media_width
jupyter.media_width

properties with deprecation warnings where added to avoid any breaking changes.

the CLI_CTX section is now made available in the config, through the cli_formatting attribute. cloup context settings are made accessible through the cli_formatting.context_settings property.

TODO

[x] Implement tex_template / tex_template_file
[ ] Implement custom_folders. Needs investigation, confusing? Still required?
[ ] Set default values to input_file / output_file (only set through the CLI)
[x] Implement get_dir
[ ] Implement ManimFrame
[ ] Parse CLI args (that is re-implement digest_args)
[x] Implement support for tex_template/tex_template_file: Might need some refactor to the related tex classes.
[ ] Choose a serialization format for ManimColor
[ ] Docstrings
[ ] Extensive testing
[ ] Rename CLI switch to CLI flag in docstrings?

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Heyo, are there any updates as to how the progress here is going? I'm interested in helping out here :)


Annotation parser does not have correct value for clusterName

When I run 1.0-beta.1 with autodiscovered subnets, I get this error:
I0627 01:24:38.019996       1 event.go:218] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"redacted", Name:"redacted", UID:"redacted", APIVersion:"extensions", ResourceVersion:"14402657", FieldPath:""}): type: 'Warning' reason: 'ERROR' Error parsing annotations: Retrieval of subnets failed to resolve 2 qualified subnets. Subnets must contain the tag:kubernetes.io/cluster/ tag with a value of shared and the tag:kubernetes.io/role/alb-ingress tag signifying it should be used for ALBs Additionally, there must be at least 2 subnets with unique availability zones as required by ALBs. Either tag subnets to meet this requirement or use the subnets annotation on the ingress resource to explicitly call out what subnets to use for ALB creation. The subnets that did resolve were []. (cache hit)

(Note the ... must contain the tag:kubernetes.io/cluster/ tag with ... part)
Given how that log line is formatted, I believe that means that the ValidatingAnnotationFactory has clusterName = "". It looks like NewValidatingAnnotationFactory is called in NewALBController, which is (necessarily) called before ConfigureFlags. Since ConfigureFlags is responsible for populating the clusterName variable, this means the ValidatingAnnotationFactory gets an empty string. If this is correct, then I believe this behavior is a regression introduced with #418 .
I'm not terribly familiar with Go so it's possible I'm missing something with mutable references or something, but then I'm really not sure why I'm getting the above behavior (which is a regression in my deployment).

I just confirmed the same behavior in 1.0-alpha.9; 1.0-alpha.3 works correctly.

I confirm the same problem as well. The 1.0-alpha.3 works fine, the  1.0-beta.1 does not find the subnets wuith the same (correct) Tags. The new version works only if I explicitely define the subnets in pod deployment manifest yaml file.

can you guys give me an example of how to reproduce it? we use a common set of security groups and i haven't had to leverage the tagging logic (nor did i write it) so i am not fluent on how it should be working

nevermind, dont need the example. the problem is the annotationFactory is created before  pflag.Parse() is called, so its created with an empty cluster name. @r0fls has a fix in the oven


Extend Channel fields to include NeuroML file + metadata

We want bioparameters associated with ion channels as well as metadata indicating level of certainty to appear in PyOpenWorm so that they can be read in the dashboard.
Examples of bioparameters being built into a model can be found here and here
'E_rev', cond_density (g_des), v_half_a, k_a, T_a, should be the initial parameters, with two metadata parameters for each.
Need to reconcile the metadata parameters from NeuroML with the PyOpenWorm concept of Evidence.  For now, let's use the Evidence mechanism as the way of incorporating the two metadata parameters into the Ion Channel.  We may need help from @mwatts15 to understand how we can use evidence in a per-parameter way to achieve this.
One example of how this might be is E_rev's value of -70 mV is at 'BlindGuess' and 0.1 certainty, while v_half_a is at 30 mV with 'BlindGuess' at 0.3 certainty.  We want the evidence to tell us that the v_half_a value came from running the ChannelWorm fitter and the E_rev came from Stephen Larson's personal best guess.  Other values could come from published papers with DOIs.

@shubhsingh594 Any luck with this so far?  Talking to @VahidGh today, it may be better for us to be storing the NeuroML file that represents the ion channel model directly, rather than breaking it down into its separate parameters and storing each of them explicitly in PyOpenWorm.   Main reason is that different ion channel files are likely to have different numbers of relevant parameters and may be a pain for data representation.  Let me know what you think about this?

@slarson , so should we just have link to the corresponding NeuroML files?
or at some point we should parse the file, get the available relevant data and show that on the dashboard.

Let's settle on importing the full text of the channel.nml file, along with the metadata parameters into PyOpenWorm.  Then we'll have a reader parse the channel.nml file on the output side.  This is going to be easier because the data model for a channel.nml file is complex and we don't want to reproduce it wholesale in PyOpenWorm.

@tarelli Made a good suggestion -- can we point at a URL with the nml files instead of the full text?

Update on this -- not extending the field to include the data, just the URL.


left ideas for the next cohort to start on

Description
This PR updates the handoff file to help the next cohort better understand where our cohort left off. Includes notes about features to add, bugs to fix, testing that needs to be done, refactoring ideas, etc.
Co-authors/Collaborators for this PR:
Ahmed Serag @ahmedseragcodes
Kelly  Jerrell @kcjerrell
Ethan Edmond @Ethan-Edmond
Type of change

Documentation and Information for the upcoming cohort
[ ] This change requires a documentation update

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

Reviewers, please let me know if I accidentally left you off as a co-author/collaborator or if you have any new changes you would like to include in this file

Ok, Thanks for doing this Austin! Nice job! Approved.

Thanks for your help!

I removed the over-eager stuff about my redux revamp and fixed a couple of the links I put in.


New version: Sony.XperiaCompanion version <IP_ADDRESS>410161222

Created by 🥟 Dumplings in workflow run #9130.

Log
Updated: <IP_ADDRESS>409180710 → <IP_ADDRESS>410161222
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
The manifests for the last version <IP_ADDRESS>409180710 will be removed. Reason: No installer URL is changed compared with the last state while the version is updated
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@stephengillie Could you merge this PR?


Only allow nested to reduce if it is already nested

Found a bug where strings including Foo {Bar} Baz would parse forever because they nested check would drop below 0 on the }

Also fixed float parsing. Since number parsing was done first, and the number regex also matches the float regex, it was breaking on any float.
You can also modify the regex by adding \b(?!.\d+)


Build: Update Cloud Auth, Beta and GA versions.

Codecov Report

Merging #1374 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1374   +/-   ##
=========================================
  Coverage     84.14%   84.14%
  Complexity     1439     1439
=========================================
  Files           230      230
  Lines          6725     6725
  Branches        617      617
=========================================
  Hits           5659     5659
  Misses          890      890
  Partials        176      176

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 676ad16...bcf55be. Read the comment docs.