[WB-9234] Raise when Table size exceeds limit

Fixes https://wandb.atlassian.net/browse/WB-9234
Description
Currently if a user logs a dataset that is more than 200k rows long as a wandb.Table, we (after warning, but not raising an exception) only log the first 200k rows to W&B. This can lead to unexpected behavior in the frontend where after a run is over a user finds that only part of their dataset is logged. This PR updates the behavior to throw an exception in this case, and gives users a pointer for how they can override the default 200k row limit.
Testing
Unit test

@tssweeney do we have any access to sdk settings/env vars in that block? we could condition the failures on an env var like TABLES_EXIT_ON_FAILURE or something

This looks correct, but I am a bit concerned about actually doing this. Are we sure we want to crash the user script in these cases? I feel like we have heard many times from customers that we should be soft warning on things like this. What if this is in a big training script and they lose their work? Can we get a firm approval from @cvphelps ? Carey: the implication here is that if a user logs a table > 200k rows - their job will crash.

We have been leaning towards not crashing in these instances, instead we should warn (ideally inline, in the footer, and in the UI) and we have a strict mode (not publicized and possibly not even used yet) which can turn warnings into errors (WANDB_STRICT=true and associated settings)

Thanks for the comment Jeff, we had the same discussion here: https://weightsandbiases.slack.com/archives/C032F05V5HS/p1649869538076629
I think for this particular case, the warning is a good start and this user is asking specifically for a crash/strict mode. I agree with not crashing long running training jobs but tables are also during dataset ETL jobs and logging partial data should be a catastrophic failure since it affects the integrity of the dataset.
I'm in favor of this change as long as we rely on best practices to toggle the fail hard mode.

Should we just tell the user to try WANDB_STRICT=true?

I like the general approach of WANDB_STRICT=true, im just worried that there are warning messages that are spammy. Do you have any sense of how common warning messages show up? Can we introduce a stronger error state and a setting to make that fail hard?

@jlzhao27 @raubitsj @dmitryduev @tssweeney @cvphelps
I re-implemented this using settings (thanks for help @dmitryduev), now there is a sdk setting you can set to opt-in to. If you opt-in, it will raise an exception if the length of the table exceeds MAX_ARTIFACT_ROWS. We should also update the documentation to tell people to modify MAX_ARTIFACT_ROWS instead of MAX_ROWS which is currently there
So we can tell JnJ to just do
WANDB_TABLE_RAISE_ON_MAX_ROW_LIMIT_EXCEEDED=True
and they will get the behavior they want

Thanks @dannygoldstein. In general this LGTM, but I'd like to hear your thoughts on the following before giving it my final thumbs-up.

This issue seems super specific to a particular scenario, and we're not sure that the way to control the SDK behavior in this case (and similar) is by introducing a super-specific knob in the settings. Instead, how about displaying a warning by default and raising an error only if wandb.run.settings.strict is True?

@dmitryduev See @jlzhao27's comment above. Right now the default behavior is to warn, and we preserve that here. But we worry that there may be other warnings that may trigger crashing behavior to be too aggressive if we enable WANDB_STRICT=true. This way we can ensure that we will only crash on this one specific warning, which is what the customer wants.


Microsoft.DataFactory API version 2017-09-01-preview contains an enum with type "object"

This causes Autorest.Bicep to throw a FATAL error as it's unable to process the enum field:
2021-05-26T14:56:31.1457134Z FATAL: Error: Enum types of 'object' and format 'undefined' are not supported. Correct your input (StoredProcedureParameterType).
2021-05-26T14:56:31.1472861Z Failure during batch task - {"tag":"microsoft.datafactory-2017-09-01-preview"} -- Error: Plugin modelerfour reported failure..
2021-05-26T14:56:31.1477521Z   Error: Plugin modelerfour reported failure.

This looks like an accidental authoring error:
https://github.com/Azure/azure-rest-api-specs/blob/1b0ed8edd58bb7c9ade9a27430759527bd4eec8e/specification/datafactory/resource-manager/Microsoft.DataFactory/preview/2017-09-01-preview/entityTypes/Pipeline.json#L1149-L1165
In the stable version, it has been corrected to "string":
https://github.com/Azure/azure-rest-api-specs/blob/1b0ed8edd58bb7c9ade9a27430759527bd4eec8e/specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/entityTypes/Pipeline.json#L3822-L3838

@anthony-c-martin Stable version is the one we are currently using, and we don't invest in the old version any more.

Answered question and close it for now since no more response.

Reopening - this hasn't been fixed.

@shawnxzq I've started a PR (#15578) to fix this - would you be able to help get it merged?
@alex-frankel FYI.

@anthony-c-martin I don't have permission to merge as I am not part of the swagger reviewer team, @leni-msft help on this, thx!

@shawnxzq Sure, will do.


intersection types behaving unexpectedly when used inside types vs classes

consider this example vs this one
The only thing changing between the two is that in the first case I use type and in the second I use class. For some reason the intersection inside the type throws an error. The same intersection inside a class (or even an interface) doesn't throw. I believe this is a bug

Another case
type A = { foo: string };
type B = { bar: string };

type Template<TA, TB> = {
  x: TA & TB,
  y: TA
};

const t: Template<A, B> = {
  x: { foo: 'a', bar: 'b' },
  y: { foo: 'c' }
}

declare function f<TA, TB>(template: Template<TA, TB>): void;

f(t)

error
 41: f(t)
     ^^^^ function call
 34: const t: Template<A, B> = {
                          ^ property `bar`. Property not found in
 34: const t: Template<A, B> = {
                       ^ object type

Oddly switching the order of the intersection fix both mine and @itajaja's case
type Template<TA, TB> = {
  x: TB & TA,
  y: TA
};

Thanks fo wighing in @gcanti. This is a minimum reproduction of the error, in my real case, which is slightly more convoluted, switching the intersection order didn't help.

@gcanti this also helps:
type Template<TA, TB> = {
  +x: (a: TA & TB) => void,
  +y: (a: TA) => void
}

Life would be easier if properties were covariant by default

This might be related to: This is very likely related to: https://github.com/facebook/flow/issues/2411

Interestingly, #2411 is fixed, but this intersection issue still seems to be present.


Extensions search box not centered nicely in viewlet

I suggest to align the paddings and margins with git and search

#8169


Value generation from specs

This is one of the more interesting features of clojure.spec. Are methods like exercise on the roadmap?
On a related note, what are your thoughts on testcheck-js and how it compares to js.spec?

Yes, it's definitely a cool feature of clojure.spec. I would like to have it in js.spec as well, by using testcheck-js if possible. The "roadmap" is basically this part of the README, so once I have the regex support finished I would look into generators.

Oh, I saw that but for some reason thought that referred to function* ES2015 generators 🤦😅 Sounds good!


Remove namespace lifetimes + support owned namespace strings

Breaking. Don't merge until 2.0.0!
Description
This PR

removes lifetime parameters from collection types when they relate to the lifetime of the "namespace" string slice; the reasoning is that said string slices are always 'static anyway,
introduces new_generic: a non-const constructor to most collection types that allows passing an owned String as the namespace; the idea is to allow storage keys calculated at runtime.

Decisions
I went with the const fn new vs fn new_generic constructors to allow for flexibility while still having an API that allows constant initialization with static strings. Admittedly, I didn't think about it for horribly long - if you think you have a better design in mind, you probably do. LMK!
Should new_generic be renamed to new_dynamic?

introduces new_generic: a non-const constructor to most collection types that allows passing an owned String as the namespace; the idea is to allow storage keys calculated at runtime.

This worked before this PR as well by using a shorter-than-static lifetime &'a str. See e.g. https://github.com/noislabs/nois-contracts/blob/v0.13.8/contracts/nois-gateway/src/state/drand_jobs/drand_jobs1.rs#L48-L54. Great we can keep that.

This worked before this PR as well by using a shorter-than-static lifetime &'a str. See e.g. https://github.com/noislabs/nois-contracts/blob/v0.13.8/contracts/nois-gateway/src/state/drand_jobs/drand_jobs1.rs#L48-L54. Great we can keep that.

Oh, huh. Except now it'll only work with an owned String. I wonder if that's fine. I could imagination a situation where it's worse for performance.

I wonder if that's fine. I could imagination a situation where it's worse for performance.

I think it's fine. The String has to be constructed dynamically anyways and then it can be passed into Ns without a clone even. And at the end of the day this is calling into storage, which is more expensive than the string allocations here.

To remove some confusion and improve the API, I changed new_dyn constructors to accept items by the new Namespace trait. The Ns type is no longer exported, so it's clear it's internal. The trait is exposed with a doc string attached.

I wonder if that's fine. I could imagination a situation where it's worse for performance.

I think it's fine. The String has to be constructed dynamically anyways and then it can be passed into Ns without a clone even. And at the end of the day this is calling into storage, which is more expensive than the string allocations here.

Yes, but IMO one of the nice things of the current impl is precisely that it can work fine with references (as well as with owned values).
I think it would be a pity to lose that.


_zunit_run_testfile:21: tests: assignment to invalid subscript range

$ cat tests/test_cover.zunit
#!/usr/bin/env zunit

@test "cover prints help with -h option" {
  run cover -h
  assert $state equals 0
}

$ zunit tests/test_cover.zunit
Launching ZUnit                                            ZUnit: 0.8.2
ZSH:   zsh 5.3.1 (x86_64-debian-linux-gnu)
_zunit_run_testfile:21: tests: assignment to invalid subscript range

OK, seems to be because I used double-quotes for my test name instead of single-quotes. Is it mentioned in the README/docs?

@pawamoy it's not - I wasn't aware it was an issue. It should accept both so I'll look into a fix

Can confirm this also occurs on my own tests when swapping ' for "

I think it's caused by this line:
https://github.com/zunit-zsh/zunit/blob/e58c49a7b2558344a2c8614b2c04bc9d150c9fb4/src/commands/run.zsh#L259
You are using \' to get the test name boundaries. When using double-quotes instead, the test name is empty, and then later at
https://github.com/zunit-zsh/zunit/blob/e58c49a7b2558344a2c8614b2c04bc9d150c9fb4/src/commands/run.zsh#L271
...you are trying to assign a value at index 0, when arrays start at 1 in Zsh.
Maybe a simple fix would be to use
testname="${line[(( ${line[(i)[\'\"]]}+1 )),(( ${line[(I)[\'\"]]}-1 ))]}"

instead? Not sure about the escaping, or if [] really are regex ranges.


Status task to check, debug and fix

Task Management Status

To-Do List [add_todo; complete_todo; ask_todo][Status: Revisioned]
Sending Mail [mailing; mail_recipient; mail_content][Status: Revisioned]
Weather News [inform_weather][Status: Revisioned]
Mood and Greeting [greet; goodbye; mood_great; mood_unhappy; bot_challenge][Status: Revisioned]
Check Timing [give_time] [Status: Completed]
Default Responses [nlu_fallback][Status: Revisioned]
Bot Functionalities recap [bot_funcionalities][Status: Revisioned]

Priority Scale

Sending Mail
To-Do List
Default Responses
Mood and Greeting
Weather News
Check Timing

Important
Status "Completed" is only for the vocal and textual testing completed. Check both before setting this status.

@giovanniPi997, Weather News handlers didn't work.


SMTP AUTH LOGIN no response after inputting the username and password

Hi there,
My MailRise instance is not providing any response after authenticating. Usually it would provide a response code of 235 if the authentication was successful. Or 535 response code if the authentication data was incorrect. But my instance does not provide any response after entering the base64 encoded username and password. I'm not exactly sure if this is a bug with the MailRise software or an issue with my configuration.
DEMO
root@shell-demo:/# telnet [domain_name] 587
Trying <IP_ADDRESS>...
Connected to [domain_name].
Escape character is '^]'.
220 [domain_name] Mailrise 1.3.0
EHLO [domain_name]
250-[domain_name]
250-SIZE 33554432
250-8BITMIME
250-SMTPUTF8
250-AUTH LOGIN PLAIN
250 HELP
AUTH LOGIN
334 VXNlciBOYW1lAA==
dXNlcm5hbWUK
334 UGFzc3dvcmQA
cGFzc3dvcmQK
[No Response]

LOGS
/home/mailrise/.local/lib/python3.11/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
  warn("Requiring AUTH while not requiring TLS "
[2024-01-18 20:34:55] WARNING:mail.log:auth_required == True but auth_require_tls == False
[2024-01-18 20:40:16] WARNING:mail.log:auth_required == True but auth_require_tls == False

mailrise.conf
mailrise.conf: |
    configs:
      basic_assistant:
        urls:
          - hasio://HOST/ACCESS_TOKEN
    # figure out how to map roles directly to user email address
    #import_code: /script/router.py
    listen:
      port: 587
    tls:
      mode: "off"
      #mode: starttls
      #certfile: /etc/ssl/tls.crt
      #keyfile: /etc/ssl/tls.key
    smtp:
      auth:
        basic:
          username: password
      hostname: [domain_name]

For your reference, I am using MailRise 1.3.0 in a Kubernetes container. I've also tried it on the stable edition and I am getting the same issue. I tried it on the latest edition and it seems to be getting a different bug found in issue #98. Could you check if this is a bug with the software?
Thank you,
Andre S.

After doing some debugging it seems that AUTH PLAIN works with the correct username and password. However, using an incorrect username and password does not provide any response.

Thanks for the test case! I’ll investigate and report back. The authentication code has a lot of moving parts, so I want to have a proper test framework in place before I make any further changes.


Dg branch fix

Codecov Report

Merging #189 (705dfb2) into master (bdffb8f) will decrease coverage by 1.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #189      +/-   ##
============================================
- Coverage     66.01%   64.90%   -1.12%
- Complexity      811      842      +31
============================================
  Files           134      137       +3
  Lines          2660     2855     +195
  Branches        265      293      +28
============================================
+ Hits           1756     1853      +97
- Misses          806      897      +91
- Partials         98      105       +7

Impacted Files
Coverage Δ

...n/java/seedu/rc4hdb/model/util/SampleDataUtil.java
5.50% <0.00%> (-13.25%)
:arrow_down:

...n/java/seedu/rc4hdb/logic/parser/Rc4hdbParser.java
95.55% <0.00%> (-4.45%)
:arrow_down:

...logic/commands/residentcommands/RemoveCommand.java
89.65% <0.00%> (ø)

...main/java/seedu/rc4hdb/logic/parser/Specifier.java
100.00% <0.00%> (ø)

...gic/parser/commandparsers/RemoveCommandParser.java
93.75% <0.00%> (ø)

...gic/parser/commandparsers/FilterCommandParser.java
94.11% <0.00%> (+0.36%)
:arrow_up:

...logic/commands/residentcommands/FilterCommand.java
87.87% <0.00%> (+2.87%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Nextjs: permissions and environment not passed to function

It appears that neither the args.permissions nor args.environment members are passed to the underlying function during creation.  Those members are explicitly set when creating the defaultFunctionProps without reference to any passed in args.
See https://github.com/sst/ion/issues/213#issuecomment-2049592634 for workaround.
Related: https://github.com/sst/ion/issues/213

+1 experiencing this issue. Thanks for providing the workaround

Should be fixed in v0.0.324.
Feel free to reopen if the issue persists.


ForgeOps-4394 secret manager v2

Cherry-pick @dnitsch's earlier PR to update secret-agent to work with secret manager v2 but create env var for AWS region so the secretmanager client can read the configured value in the secretagentconfiguration

I just noticed this commit also commits the kind binary. Was that on purpose?

Good spot @pearj , I'll get this removed


PHP: bad License in package.xml

package.xml states
<license uri="http://www.php.net/license">PHP</license>
LICENSE file is "Apache-2.0"
Please fix package.xml for consistency

Notice: it will be nice to have the package.xml file stored in git repository so we can open PR for it
Also nice a have changelog inside the package.xml
so useful information will be found at https://pecl.php.net/package-changelog.php?package=pinpoint_php

@remicollet
Got it, I will change them.

@remicollet
Can you help me , how to upload windows .dll into pecl.php.net ?
https://github.com/eeliu/pinpoint-c-agent/actions/runs/10141196543#artifacts

For pecl, only a upload button for these files

But I want to release these .dll as redis

Can you help me , how to upload windows .dll into pecl.php.net ?

Sorry, no, I don't know anything about Windows
Perhaps @cmb69 may help

@remicollet
Thanks all the same

See https://news-web.php.net/php.pecl.dev/17597.

@cmb69  Thanks
But my problems is how to upload these compiled binary package into  pecl.php.net like phpredis dll.

I already got an account.

But my problems is how to upload these compiled binary package into pecl.php.net like phpredis dll.

You are not supposed to do this. Instead, make that package buildable on Windows first. While there is a config.m4, there is no config.w32, which is a first requirement to build the extension on Windows.

Ah, I've just noticed that there is a config.w32 in the PECL package (but apparently that is missing from this repo), and did a quick test build without any issues. Therefore I requested https://github.com/php/php-windows-builder/issues/3#issuecomment-2260279730, so that the DLLs can be built, and uploaded.

The DLLs are available now, thanka to @shivammathur.

@cmb69 Thanks

@eeliu We are doing PECL builds right now, so when you add a new release, the DLLs will be added. Feel free to reach out if there is a delay.


use timeout on cell disconnect

hello,
i've discovered on latest code that if i pull some cell cable(simulate something broken/failed module/etc.) that end module disable relays,but victron keeps going on latest values delivered(since last cell module just stops transmitting),is it possible to add some timeout check and kill load/charging? killing load/charging works after reconnect,when fail is delivered on first message after interrupt
thanks,
pb

Hi,
Which version are you using, v0.9 from the website or the beta (v1.2) via kwindem's, PackageManager? V0.9 does not support any active Victron control, v1.2 does but is in beta.

Hello,latest beta...i'm not a programmer myself,but i can understand what it does,so when the communication stops,the latest module stops,so this needs to be in code,as the last module won't send the problem,so this will need bigger code change probably?

Hi,
Thanks for letting me know. I will do some tests and fix it (if not already done) in the new coming beta.

Hi Korenchkin,
We fixed this in the last update. Please let me know if it fixed your issue.

hello,
so,i tested it thoroughly,here is what happens after breaking cell communication:
stopping charging works,after several 10's of seconds mppt goes from bms managed to something like 'keep' (sorry,translating from czech),which stops charge,but i think this is possibly only side-effect,there is really no other option to use in bms software than timeout(for example 10s after last message?to be on a safe side?) to set max charge current and max discharge current to 0(or allow to charge and allow to discharge =no,which presumably sets currents to 0)
after reconnecting the cell back first message gets to ccgx and multiplus goes down(because during this time allow discharge=no message gets there)...then mppt starts to charge slowly and multiplus starts again
(semi-island,i have line,but i chose to keep it off to really stay away from even very small feed-in,crappy laws here)
tested latest 1.3~8, DVCC ON, 24V 8cell LFP 3.4V (dvcc works well,charging to something ~27.4V to balance for a minute,then backing to ~26.88V)
thanks

Hi,
Thanks for testing.
The 30 second timeout is correct. This is done on purpose: we discovered that a USB instance can disconnect from time to time, maybe because of EMF pulse. It takes about 15 seconds to restart the USB code, so we want to be sure the Manager does not stop the Victron meanwhile. That is why the timeout is 30 seconds. Don't worry, in 30 seconds the battery will not be over or undercharged.
We found a small bug which can make the Manager crash (and restart), will fix it directly.
Thanks for testing!

hello,thanks for the fix,maybe i wasn't waiting long enough for shutdown to trigger,i had time today to test different conditions and after disconnecting cell it took around 2 minutes before multiplus went offline,cerbo already in alarm conditions and multipluses were still online,it took some time...
after reconnecting alarm cleared itself and multipluses started,so this works beautifull
//* another possible issue starts here *//
now i have to test bulk/float charging,in some not too past version i had to restart cerbo for charging to start,because bms was holding batteries hostage(another day it would not charge to full,also this might be connected to soc calculation only,voltage was holding to 26.88 if i remember correctly on 24v system,but soc was dropping each day to around 50%,after 7 days it shifted max voltage to something 27.4,not sure exactly,and this led firmware on bms to set soc to 100%,but charge didn't look like 50% of a battery capacity,i believe part of the problem is too low current,below 4amps on 2 turns of wire through dual current sensor)
i'm putting it here in case somebody has some idea,when/if i can precisely replicate what's wrong,i will make new issue to keep this clean
thanks,
k.

Hi,
When the battery cells are full and balanced, the BMS will lower the max charge voltage to 40mV below Vbalance, so the cells will not keep balancing all the time for nothing. So if your Vbalance is 3.4V, the code will keep the cells at about 3.36V per cell (max). In the early version, this is reset every 7 days or when disbalance is detected. In the new code, it is reduced to 4 days. So every 4 days, the BMS will let the cells balance again.
If you notice there is still an issue, for example that you see that balancing is needed within 4 days because you charge and discharge a lot, then please let us know.


Add a Gitter chat badge to README.md

octolab/Cleaner now has a Chat Room on Gitter
@kamilsk has just created a chat room. You can visit it here: https://gitter.im/octolab/Cleaner.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

already exists


netboxcommunity/netbox:latest throwing error Plugin netbox_bgp requires NetBox maximum version 3.3.99.

NetBox version
3.4 (Docker Container)
Describe the bug
netbox-docker-netbox-1               | django.core.exceptions.ImproperlyConfigured: Plugin netbox_bgp requires NetBox maximum version 3.3.99.
Looks like when pulling the netbox_bgp plugin from docker.io (netboxcommunity/netbox:latest) its erroring out with the above.
To Reproduce
Expected behavior
Would expect the netbox-bgp image on netboxcommunity/netbox:latest to be compatible with the latest netbox image?
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Super noob here, may be user error. Any help is appreciated.

Have run into this as well on a upgrade on a non docker install.
Last version that seems to work or be supported it v 3.3.10

Think I found the culprit on LINE 15 of
/opt/netbox/venv/lib/python3.10/site-packages/netbox_bgp/init.py
max_version = '3.3.99'

There's already a PR #126 including a netbox version bump. We're all waiting for it to be merged :fist:

Hoping this comes soon!


#26 portuguese translaction

Requesting the Pull with Portuguese Translaction.
Thank you @ScreamingHawk

Thanks very much! I really can't do translations without the help of open source contributors like yourself. This is really appreciated.


operator opendatahub-operator (2.9.0)

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Are you familiar with our operator pipeline?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Your submission should not

[ ] Add more than one operator bundle per PR
[ ] Modify any operator
[ ] Rename an operator
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/pipeline restart community-hosted-pipeline

by looking at th error this seems has 2.9 already included in the index, which i cannot think of why.
i will run the test once again but will need someone to help if this is the case.

@Allda will you be able to take a look this PR and identify is the problem is the index image already contain a 2.9 bundle in it? Thanks!

Closing this, will create another PR with no version annotation


Analyzer doesn't complain about generic properties

The analyzer doesn't produce any errors about the following code:
T get value<T> => null;

However, this is invalid. The VM and dart2js both fail to parse it, and DDC complains that T is undefined.

Oof, the analyzer now reports this issue, but it reports 9 issues. :/  I think there is another recovery issue flagged for such top-level members, but I'm not sure, so I'll leave this open for now.

The recovery has improved, and there is no recovery issue when this getter is included in a class. I'll close it as the root issue is resolved.


EntityList not returning correct URL values using convenience methods

The GetURL method should honor the key parameter instead of always returning the hard coded "self" entry. This will correct the convenience method calls GetNextURL, GetSelfURL, GetLastURL.

Warning: this code fix will break the ListAll method in the AbstractEntity class. In addition, fixing the inner loop bug in the ListAll method will still cause crashes until the GetList method in the Connection class parses the passed in "Next" URL correctly instead of simply appending the BaseUrl() results to the url parameter.

@MartinJSoles Sorry for sitting on this for a year. Thank you for the thoughtful PR. LGTM!


Error: 'win32-x64' binaries cannot be used on the 'linux-x64' platform.

Hi, I'm using windows 10 with docker. But I keep getting the following error:
Error: 'win32-x64' binaries cannot be used on the 'linux-x64' platform. Please remove the 'node_modules/sharp/vendor' directory and run 'npm install'.

I removed node_modules/sharp/vendor, then npm install. However the issue is still there.
Please I need help

looking for a solution as well.
can't dockerify strapi, because the host computer is on windows and default solution for dockerization suggest copy-paste package-lock.json file


Please use rust-crypto to ease compiling on Windows

Thanks for the library! As the title mentions I am having problems compiling openssl with rust. rust-crypto crate has HMAC implementation so is this possible?

Documentation states that: "Rust-Crypto has not been thoroughly audited for correctness, so any use where security is important is not recommended at this time." So I think this issue can be closed with valid concerns. :)

Thanks for the report! It is indeed a bummer how difficult it is to get OpenSSL working on Windows. I wonder if perhaps though we could move to the ring project? That I believe is more vetted in the sense that it's just BoringSSL's code?

By the way, I got it working. The solution, for now, is installing OpenSSL 1.0.* Because libraries look for libeay32.dll & libssl32.dll .
I don't have much experience regarding crypto. So I will listen where this conversation goes :)

Ok! In that case I'm going to close this for now. Thanks for the report though!


Greenkeeper/sinon 8.0.1

Pull Request Test Coverage Report for Build 1036

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.644%

Totals

Change from base Build 596:
0.0%

Covered Lines:
635

Relevant Lines:
660

💛  - Coveralls


possibility for multiclass semantic segmentation

Hi,
I would like to ask if the code is easily to be modified for multiclass semantic segmentation?
please advise,
thanks

I think it is easy. You only need to change the seg head of the model to support multi-class seg and modify the training pipeline.
However, I don't suggest you to do som because this code is only experimental code for research of promptable seg.
A better way is to modify a SOTA multi-class semantic seg code (e.g. STU-Net，nnU-Net), and then you only need to change the model.
Hope this can help you.

Hi @blueyo0 ,
I am trying to do exactly that, I want to try to finetune the SAM-Med3D model in the nnUNet framework (v2). I am a beginner at this and I was wondering if you could tell me which are the key parts that I should introduce to finetune one of the checkpoints (e.g. SAM-Med3D-turbo).
Thank you.
Best,
Sergio

Hi, currently, SAM-Med3D only supports single-modal inputs. If you expect to support promptable seg pipeline in nnUNet, you may need to implement your own trainer and the related inference functions.


server端如何使用springcloudconfig作为配置中心呢？

Why you need it?
Is your feature request related to a problem? Please describe in details
How it could be?
A clear and concise description of what you want to happen. You can explain more about input of the feature, and output of it.
Other related information
Add any other context or screenshots about the feature request here.

暂时还不支持，这个有计划，估计是在1.6版本


Inline Picklist menu with a scrollbar becomes blank on mouseover [Windows][IE11]

Environment: Windows 8.1, IE11 (not reproducible in Windows 7)
Repro steps:

Go to Popover playground: https://react.lightningdesignsystem.com/components/popovers/
Add a Picklist with isInline={true} with 6 or more items to a popover content

you want an inline picklist with a vertical scrollbar in the menu - see an example below

Open the popover, open the picklistm mouse over an item
Observe that menu content is shifted. It appears that its original offsets are doubled.

<Picklist
	className="slds-m-bottom--large"
	label="Lead Source"
	onSelect={(option) => { console.log('selected: ', option.label); }}
	options={[
			{ label: 'Third Party Program', value: 'A0' },
			{ label: 'Cold Call', value: 'B0' },
			{ label: 'LinkedIn', value: 'C0' },
			{ label: 'Direct Mail', value: 'D0' },
			{ label: 'Third Party Program', value: 'A01' },
			{ label: 'Cold Call', value: 'B01' },
			{ label: 'LinkedIn', value: 'C01' },
			{ label: 'Direct Mail', value: 'D01' },
			{ label: 'Other', value: 'E0' }
	]}
	placeholder="Select Lead Source"
	value="B0"
	isInline={true}
/>

This also affects Filter component:

Closing. Feel free to re-open for combobox.

@interactivellama, could you please let me know which release version contains the fix of the issue? Thanks!

Combobox was added in v0.6.23. Picklist has been deprecated.

@interactivellama , I went to the combo playground with Windows 8 IE 11: https://react.lightningdesignsystem.com/components/comboboxes/
None of the combo boxes shows :( (see screenshot)
I will reopen this bug for that.

IE11 doesn't have object.assign if you look in the console for those examples. You may want to bundle an assign polyfill if you support IE11. I'm looking into the scroll issue in Edge.


Docs: Readme: Instruct to navigate into project directory

Current installation instruction does not ask to navigate into the project directory before npm install. Newcomers to npm may fail to run the  project because of that.
Fix this issue by adding a new instruction to navigate into the project  directory.

Thanks a lot for contributing.
Give us a star if you like.


Questioning the value of OpenAPI schemas in this repo

The OpenAPI schemas in this repo somehow provide the OpenAPI representation according to optimade-python-tools at the moment of the releases of the specification.
Currently this is quite out of date with the versions at https://github.com/Materials-Consortia/schemas, and in fact, requiring the OpenAPI schemas to be correct on release is quite an additional burden for optimade-python-tools. (see https://github.com/Materials-Consortia/optimade-python-tools/pull/1427 and the many PRs before it that show my attempts to keep 1.1 compatibility whilst generating the 1.2 schema).
I will make a PR here with the latest draft 1.2 OpenAPI schema from optimade-python-tools, but I would suggest in future that we remove ./schemas from this repository and instead version them more explicitly in the schemas repo (and link to them from here). I will try to remember to bring this up next meeting...

AFAIR, this is somewhat of a chicken-and-egg problem, arising due to the fact that OpenAPI schemas are generated from optimade-python-tools source. I myself see the value of OpenAPI schemas and would prefer having them alongside the specification as means to validate responses, generate UIs and even to dispel some specification ambiguity (hopefully). However, I have no idea how to make their development easier. I like the staging branch approach, but I get why it is a burden.

Would you be against adding a link in the specification for an authoritative source for the OpenAPI schemas? (i.e. schemas.optimade.org or https://github.com/Materials-Consortia/schema, we could even automatically release them on Zenodo and provide DOIs).

I liked having specification and schemas in the same repository under the same version number, but I guess separating them wouldn't be too much harm. Would it be possible to retain the same version numbering scheme? Otherwise it might be difficult to track the correspondence.
Out of plain curiosity, how does the split help to reduce the burden?

I understand that we are talking about quite a bit of work, but, are we not expected to provide valid schemas "from day one" when we release a new version? I'd rather take more generic schemas, even down to just bare-bones validating essentially just that the response is on jsonapi form, than making releases with no schemas at all.

Sure, and there will be a valid schema for 1.2 from day one, but our current approach leaves us with no space for schema-only bug fixes. Maybe this isn't critical, but I can foresee that the optimade-python-tools schema will be buggier this time around given the additional spec changes. As you suggest, it is only currently a lot of work because the schema requires the entire server to be set up with all possible features --- I wouldn't be against replacing the ones we currently release with more generic versions in the spec repo (then the schemas.optimade.org can give our best attempt at a full schema).

With that said -- my plan here was that we were meant to create property definitions for all data entries we have standardized so far. After that it should be fairly straightforward to auto-generate proper OpenAPI schemas right from what we have in the specification repo without going via optimade-python-tools. (In fact, I'd have done it already if not the post-corona wave of lesser sicknesses kept me from doing anything but the most time critical work, yada, yada excuses.)

I get lost on whether I mentioned this in the other issue thread, but if you have not yet started this (hope you are feeling better, of course!) then what we have in https://github.com/Materials-Consortia/optimade-python-tools/pull/1425 might be a helpful starting point. I will be generating the new-style /info/structures response for the standardized fields soon too so we can see if we agree :sweat_smile:

What time plan are we on for the v1.2 release? The above still seems doable with reasonable effort if I put some time towards it.

As I posted in my other comment, it's up to us (did we confirm that you and I @rartino who are officially "release managers" a few months ago?), but I guess if we want to do this work first then a release candidate soon (now, or just after next meeting?) with a real release (or further rc) after our first 2023 meeting could be a sensible timeline?

Sure, and there will be a valid schema for 1.2 from day one, but our current approach leaves us with no space for schema-only bug fixes.

I don't think we should be afraid to fix schema-only bugs with point releases. It shouldn't be cumbersome - these releases would just replace the schema files and nothing else. Given that we keep track of schemas on https://shemas.optimade.org by OPTIMADE version number, I think we actually need them to have their own version numbers.
It has also been argued that our schemas help clarify what we mean in the specification text and thus are an integral part of the specification. This view further establishes that schema bugs are specification bugs that motivate new point releases.

but I guess if we want to do this work first then a release candidate soon (now, or just after next meeting?)

Since you mention we may want to discuss next meeting what actually goes into the release; I think the earliest we can have a release candidate is right after that meeting. I'll attempt to at least start looking at this before that.

what we have in https://github.com/Materials-Consortia/optimade-python-tools/pull/1425 might be a helpful starting point.

Maybe I am misunderstanding, but I really think it is far more work associated with supporting property definitions in optimade-python-tools than it is to turn our current chapter 8 into property definitions and write a script that inserts them into otherwise barebones  json-schema for OPTIMADE responses.
On that note, what makes the most sense for creating those barebones schemas? Use the current optimade-python-tools generated ones, or just grab http://jsonapi.org/schema and manually edit it to add the OPTIMADE conventions?

Closed by #502 and related PRs -- at some point we may catch up and generate OpenAPI schemas for future versions, but this is not a priority.


Adding support for MobileViTV2 model

Model description
Hi,
I would love to use MobileViTV2 in my application. I am definitely not an expert, but it seems that its architecture is pretty similar to MobileViT, so adding it seems fairly straightforward to me.
Laszlo
Prerequisites

[X] The model is supported in Transformers (i.e., listed here)
[ ] The model can be exported to ONNX with Optimum (i.e., listed here)

Additional information
No response
Your contribution
I experimented with this model on my fork here: https://github.com/KLaci/transformers.js/commit/e1e02b1d5876c1ffe4cadb53a01d592efee623a3
I can submit a PR too if needed.

Hi there 👋 Looks like the ONNX export isn't as simple as I originally thought (see here). Is this something you'd be able to look into? :)

Okay I might have got it working. Feel free to open a PR for your fork.

Example code:
import { pipeline } from '@xenova/transformers';

const url = 'https://huggingface.co/datasets/Xenova/transformers.js-docs/resolve/main/tiger.jpg';
const classifier = await pipeline('image-classification', 'Xenova/mobilevitv2-1.0-imagenet1k-256', {
    quantized: false,
});
const output = await classifier(url);
// [{ label: 'tiger, Panthera tigris', score: 0.6491137742996216 }]


Console freezing

Issue Type: Performance Issue
When I'm using VS Code console, and start typing on that, ie git branch, git status it freez, and the word comes later
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:30:02.420Z)
OS version: Darwin x64 20.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 1, 2

Memory (System)
16.00GB (1.74GB free)

Process Argv
--crash-reporter-id 44547d9d-b8bf-42f4-be92-cf8cf3f4d7e2

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
   10	   180	   375	code main
    1	    66	   602	   gpu-process
    0	    16	   605	   utility
    0	   229	   606	   window (index.js — talent-db-frontend)
    0	    82	   610	     extensionHost
    0	    66	   612	       electron_node tsserver.js
    0	   721	   613	       electron_node tsserver.js
    0	    16	   622	         electron_node typingsInstaller.js typesMap.js
    0	   147	   623	       electron_node eslintServer.js
    0	    16	   691	       /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper (Renderer).app/Contents/MacOS/Code Helper (Renderer) /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/json-language-features/server/dist/node/jsonServerMain --node-ipc --clientProcessId=610
    0	    16	   611	     watcherService
    0	     0	  1022	     /bin/bash -l
   87	    66	   621	   shared-process
    0	     0	  3432	     /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    2	    66	  3430	   window (Issue Reporter)

Workspace Info
|  Window (index.js — talent-db-frontend)
|    Folder (talent-db-frontend): 2068 files
|      File types: js(1363) svg(122) jsx(60) less(53) png(29) json(29) md(27)
|                  ts(10) jpg(4) yml(2)
|      Conf files: package.json(2) jsconfig.json(1) tsconfig.json(1);

Extensions (1)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.1.14

A/B Experiments
vsliv368:30146709
vsreu685:30147344
openlogontheside:30221877
python763:30178808
python383:30185418
vspyt653:30237719
vspor879:30202332
vspor708:30202333
vspor363:30204092
python504:30227505
vswsl492:30211401
wsl2promptcf:30224613
unusedpromptcf:30224611
folderexplorer:30224614
openfilemenu:30224647
pythonvsded773cf:30236630
vsjup459:30237187

please try "terminal.integrated.rendererType": "dom"

still has some freezing, looks much better now. I will continue to work on this way.
If I have same problem again, I'll come back
Thank

it is getting worse, whenever I try to use the integrated terminal, it freeze after I type one word, and then I have to wait until the word comes

I have the same exact problem.
My system is a macbook pro 16 with mac OS Big Sur.

I have the same exact problem.
My system is a macbook pro 16 with mac OS Big Sur.

seems to be this issue here: https://github.com/microsoft/vscode/issues/105446

seems to be this issue here: https://github.com/microsoft/vscode/issues/105446

Duplicate of #105446

Duplicate of #105446


Enable additional stats in NYCTLCUtils.java

When we originally worked on the demo, the intent was to calculate a variety of statistics for the fields contained within the data.  Unfortunately, a few of the stats that we were calculating were throwing exceptions so we had to limit the number of stats that are enabled.
For this ticket, go ahead and enable stats for each field where appropriate, and ensure that everything runs without exceptions.  Specifically, I think we had trouble with the histogram statistic for certain fields.  I think the integer that was passed in was out of bounds in some cases.  We just need to handle that case.

Additionally, if there are any other stats that you think might be useful, go ahead and create/add them.  Also, if you see any room for improvement with the current method of generating stats, feel free to make changes.

Sounds like you have this wrapped up with the latest pull request.


Attribute Seravo/wordpress

Even though this is not strictly a fork, devgeniem/wp-project contains at least the test code from Seravo/wordpress, and thus should attribute it according to the MIT licence.
Pull request adds the original LICENCE

You're right, I added LICENSE tests/rspec/LICENSE.md folder. This is taken from seravo/wordpress. So then your license is bundled with your software.
Seravo development stack has been great inspiration for this, do we have other files in this project we should mention?

Only things I can find that should be attributed to Seravo/wordpress in addition to the rspec tests are the nginx directory in the root and your ingenious wp-load.php fix in htdocs. :)
https://github.com/Seravo/wordpress/blob/master/htdocs/wp-load.php


Add missing error handling for failed http calls that stops metrics from being populated ever again

Proposed changes
As described in https://github.com/splunk/splunk-connect-for-kubernetes/issues/204
we notice that when running Splunk in Azure Kubernetes Service some metrics stop showing up in Splunk after few hours and never show again unless pod with metric aggregator pod is restarted.
This PR tries to fix that.
We noticed that problem is in this plugin, in case of http call faliures, the timers:
https://github.com/splunk/fluent-plugin-k8s-metrics-agg/blob/cea5b8dc343b7a213ccb70de0bd26cb6cbc95c7e/lib/fluent/plugin/in_kubernetes_metrics_aggregator.rb#L160-L162
are detached:
2019-07-12 09:45:21 +0000 [error]: #0 Unexpected error raised. Stopping the timer. title=:resource_usage_scraper error_class=RestClient::Exceptions::OpenTimeout error="Timed out connecting to server"

and never run again after single network or service blip.
Hence I've added timeouts to web mocks to reproduce the issue then added error handling for methods that do http calls. I have no previous knowledge of Ruby, Fluentd so test most likely can be done better.
Types of changes
What types of changes does your code introduce?

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have read the CLA
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

I have updated the tests to assert all of the general metrics in a single test so that the setup was not run for each one, as this makes the tests a lot faster.
Added new tests to check the handling of the request timeouts when calling each of the Kubernetes endpoints. These tests work by counting specific metrics emitted from the various timers with the expectation that the number emitted will be less where that timer experienced a request timeouts.

@KrzysztofKowalczyk and @chapmonkey  are you collaborating on this one?

@KrzysztofKowalczyk and @chapmonkey are you collaborating on this one?

yes

yes

@chaitanyaphalak there are more tests actually, but multiple assertions where put in single test case.
Have a look at https://github.com/splunk/fluent-plugin-k8s-metrics-agg/pull/33/files#diff-4a76a9058d904a64c36088c1aee454efR105-R155
you can see that existing assertions have been brought together and new tests have been created.
Previously there have been multiple test cases with separate setup with data mocks for 3 nodes. This setup takes a lot of time and actually most of it is never really used in test. When I removed two nodes not a single test failed. Tests also does not assert actual values but just their existence ignoring its origin (hence if scrapper would only load data from first node but not the others the test would not fail).
Additional a very slow setup was recreated to assert every single small metric name which made those test to run for 10 minutes for very basic validation.
Hence joining of the existing assertions and some new test cases have been introduced.

@KrzysztofKowalczyk thanks for providing more context. I will do a followup review again. Did you get a chance to test this in your setup? Is it addressing your issues?

Is it expected for there to be a release of https://hub.docker.com/r/splunk/k8s-metrics-aggr

yes @cwebbtw

When can we expect release there ?


[BUG] "ManagedIdentityCredential authentication failed: '<' on local debug

Describe the bug
In Visual Studio without environment variables it fails to authenticate me under correct credentials, unless I set new VisualStudioCredential() (therefore skipping 2 environment and managed). From my knowledge I don't have anywhere set environment variables, so I have no idea on parsing what value it failing...
Expected behavior
Use correct credentials from Visual Studio
Actual behavior (include Exception or Stack Trace)
Exception:
"ManagedIdentityCredential authentication failed: '<' is an invalid start of a value. LineNumber: 0 | BytePositionInLine: 0.
See the troubleshooting guide for more information. https://aka.ms/azsdk/net/identity/managedidentitycredential/troubleshoot"
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Use Account A for VS license
Use account B for Azure debug connection (selected from Options)

public KeyVaultSecretsService()
{
    var keyVaultURL = ConfigurationManager.AppSettings["YourAppSettingsKey"];
    this.client = new SecretClient(new Uri(keyVaultURL), new DefaultAzureCredential() // new VisualStudioCredential() - for replacement during local tests
         );
}
public string GetSecretByName(string secretName)
{
    var secret = this.client.GetSecret(secretName);

    return secret.Value.Value;
}

no environment variables try to retrieve secret

Environment:

Name and version of the Library package used: Azure.Identity 1.5.0, Azure.Security.KeyVault.Secrets 4.2.0
Hosting platform or OS and .NET runtime version Win 10 :NET framework 4.7.2
IDE and version : Visual Studio 16.11.7

Thank you for your feedback.  Tagging and routing to the team members best able to assist.

The json parsing error sounds like a dupe of https://github.com/Azure/azure-sdk-for-net/issues/24158.
The reason this error is occurring locally is because there is something on your local network that is responding on the same non-routeable IP address that Managed Identity is expected to listen on.
https://github.com/Azure/azure-sdk-for-net/blob/40daa335d466fb5284552ca10e6f7854e5e80cf2/sdk/identity/Azure.Identity/src/ImdsManagedIdentitySource.cs#L17-L18

@christothes  Is it planned in case of the invalid response move to the next auth method?
Or it is already there, just not working due to #24158 ?

If there is a service listening on the IMDS endpoint, the credential chain will stop there because the assumption is that this endpoint will only respond in an Azure environment.

hmm, this happens in the Amazon workspace VM of our clients, so here I'm not sure if this Amazon or some network settings of our client... anyway something returns same  404 (as directly from the internet) but also with the HTML body:
<?xml version="1.0" encoding="iso-8859-1"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
		 "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

  <title>404 - Not Found</title>

404 - Not Found

then we have this error, probably because the client try to parse the response body despite 404.
During the test outside this VM, where no body returned on 404 this doesn't happen and this works as expected.
@christothes is it possible to ignore the body if it's 404?

During the test outside this VM, where no body returned on 404 this doesn't happen and this works as expected.
@christothes is it possible to ignore the body if it's 404?

Do you happen to know if the client is actually getting a 404 response, or if that is some kind of proxy returning a 404 content response? Could you reproduce it with logging enabled on the Credential to see how the endpoint is responding?
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/identity/Azure.Identity/README.md#logging

The server which responding with 404 seems to belong to AWS infrastructure- EC2ws, with enabled diagnostic logging the picture is the same, 5 404 responses then this error in the code on getting the secret.

Since this is running in AWS and you never want ManagedIdentity, would it make sense to just disable that with the ExcludeManagedIdentityCredential option property? This will make things faster also, since the client doesn't need to probe the endpoint on initialization.
https://github.com/Azure/azure-sdk-for-net/blob/53f08c606b2e5d0540781f1d44bb0e8189563326/sdk/identity/Azure.Identity/src/DefaultAzureCredentialOptions.cs#L74

well, no, the target is actually Azure, so this will be needed at the end, this bug just complicates the local debug from VM which runs in AWS because I have to do changes for the debug. I checked #24158 one more time, and if response parse will happen after a check on 200 my problem wouldn't exist, so yes this is definitely duplication.

One possible workaround is to set the IMDS_ENDPOINT environment variable to some other non-routeable endpoint address where there is nothing listening. That will allow the IMDS endpoint check to fail as expected.

Thx, I tried to set in the project settings => Debug => Environment Variables to <IP_ADDRESS>, where definitely will be no body, unfortunately this didn't help.

Thx, I tried to set in the project settings => Debug => Environment Variables to <IP_ADDRESS>, where definitely will be no body, unfortunately this didn't help.

Isn't that the one IP that is guaranteed to get a response from the current machine? 😄

yes, but the idea was, that the rest of IPs could go via DNS => receive 404 with the body, and <IP_ADDRESS> return just nothing, because there is no HTTP server running locally, but the result is the same. Seems this setting somehow didn't apply, am I missing something?

Could you provide the exception, stack, and relevant logging you get with the <IP_ADDRESS> host?

The exception is exactly the same as above in the bug description, I assume that env variable just didn't apply somehow

The exception is exactly the same as above in the bug description, I assume that env variable just didn't apply somehow

The logging would confirm - would you mind providing the logging output relevant to the credential discovery?

sorry for the long response, long story short - I didn't understand how to make this logging work.  I would just wait for https://github.com/Azure/azure-sdk-for-net/issues/24158 to be solved.

Assuming you wanted to log output to the console, it should look like this:
public KeyVaultSecretsService()
{
    using AzureEventSourceListener listener = AzureEventSourceListener.CreateConsoleLogger();
    DefaultAzureCredentialOptions options = new DefaultAzureCredentialOptions()
    {
        Diagnostics =
        {
            LoggedHeaderNames = { "x-ms-request-id" },
            LoggedQueryParameters = { "api-version" },
            IsLoggingContentEnabled = true
        }
    };
    var keyVaultURL = ConfigurationManager.AppSettings["YourAppSettingsKey"];
    // Note: use of options here
    this.client = new SecretClient(new Uri(keyVaultURL), new DefaultAzureCredential(options) // new VisualStudioCredential() - for replacement during local tests
         );
}
public string GetSecretByName(string secretName)
{
    var secret = this.client.GetSecret(secretName);

    return secret.Value.Value;
}


Add option to use a local llm, such as Ollama

It should be as simple as
From the users side:

install ollama https://ollama.com/
run ollama to load a model, (eg ollama run llama)

from your code side:

update UChatGPTSetting.dfm to include ollama in the list
if the settings is set to ollama, then update the edt_Url to 'http://localhost://11434/v1/chat/completions'
Allow the user to specify their model or query ollama to see what models are currently present.

Hi, an offline solution has been my concern since the beginning, but I was waiting for a simple, powerful,  easy-to-install and use one.
I'm currently playing with https://opencopilot.so.
Still, your suggestion seems promising, probably I will add yours first.
Thank you.


Implement hierarchy helper functions

Adds the #186 enhancement

Hello @raamcosta, not sure how merging PR's work for you so sorry for tagging you for no reason if it's unnecessary..
This PR feels up to standard for me in terms of its contents, feel free to approve or reject it! There's probably a more elegant way of doing the same thing but hey, I'm learning the internals of your library (which I find really interesting by the way!) :)
Thanks in advance

Hello @raamcosta, not sure how merging PR's work for you so sorry for tagging you for no reason if it's unnecessary..
This PR feels up to standard for me in terms of its contents, feel free to approve or reject it! There's probably a more elegant way of doing the same thing but hey, I'm learning the internals of your library (which I find really interesting by the way!) :)
Thanks in advance

Of course! Thank you so much for this PR 🙏
And sorry I haven’t given any feedback yet. Reason is I’ve been quite busy lately so I’ve given priority to new issues and fixes I needed to do.
I will eventually check this and provide all the feedback 🙂
Thank you also for checking the code and liking it. There is quite a bit of messy parts in there that I wish I had time to clean up 😅

@MaxMichel2 sorry I still haven’t merged this, it is not forgotten.
The thing now is that I am doing quite a few changes which can probably bring new approaches to do this, so I’d rather wait for that and then recheck this PR. Thank you again 🙏


:memo: cleanup docs

PR

Adds a link to the the example folder for the oeprator
Removes the /docs/operator href which 404s

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
216

Relevant Lines:
240

💛  - Coveralls


feat: Initial Bun server with react-email example

I'm sorry for my misclick. 😳

Haha thanks @magalhaespaulo ;)


Enforce required RequestBody

Fixes #878
Fixes #1317

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-4 of 56 (92.86%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 92.329%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

connexion/operations/swagger2.py
20
21
95.24%

connexion/validators/form_data.py
7
8
87.5%

connexion/validators/json.py
10
12
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
3274

Relevant Lines:
3546

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

52 of 56 (92.86%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 92.329%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

connexion/operations/swagger2.py
20
21
95.24%

connexion/validators/form_data.py
7
8
87.5%

connexion/validators/json.py
10
12
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
3274

Relevant Lines:
3546

💛  - Coveralls


devspace build missing "--dependency" flag

Is your feature request related to a problem?
I am having trouble requiring devspace to rebuild a particular dependency.
Which solution do you suggest?
Add --depedency flag to devspace build.
Which alternative solutions exist?
Good question. As far as I can tell, --force-build also isn't working for dependencies.
The only option so far is that I go to a copy of the dependency's repo and rebuild there -- which means I have to manually make sure the config is appropriate.
Additional context

/kind feature

@shaunc thanks for opening this issue! Seems like there are two things here:

--force-build when used with devspace dev and devspace deploy should actually also force rebuilding the dependency images. Is that not working for you? I just tested it and it works for me, it is rebuilding the images if the flag is provided
yes a --dependency flag for devspace build is a good idea, we can do that!

Force build is not working in a complex situation for me, but I don't have a reproduction. Perhaps treat this as a report for #2 -- will open a new ticket with a reproduction for #1 if it persists.

Implemented as part of v4.13


discord link

Hi, whats the discord link? Nowhere for me to find it to get sponge jar and support.

Discord is private. Buy the plugin on spigot if you want a discord link.

i want to buy

so can i get the discord link

I need GDHooks, but there is no Discord link anywhere. This must be the wordt documented plugin ever.


Plans In Progress: Elapsed time should tick every second regardless of polling interval

Currently, the elapsed time only changes every 15 seconds (or whenever the polling returns). It should tick once per second to stay in sync with the actual elapsed time.

I'm grabbing this alongside https://github.com/ManageIQ/miq_v2v_ui_plugin/issues/353 because there is some blocked time on it and I want to make some contributions.


Exception when migration runs on multiple nodes against empty schema

What version of Flyway are you using?
3.2.1
What database are you using (type & version)?
Oracle DB  10.x
What operating system are you using?
Windows and RHEL
What did you do?
I ran the migration against an empty database at the same time from two different nodes. (I am using Spring Boot which runs flyway on startup)
What did you expect to see?
As the FAQs say, it should be okay to run the migration from multiple nodes, since there is appropriate locking applied.
My explanation for the exception below is, that the schema_version table is used for locking, but if two nodes try to create this table at the same time, there is no locking yet.
In other words,  the MetaDataTableImpl.createIfNotExists seems not to be atomic.
It would be nice if flyway handles this case more gracefully, since the exception below lets the migration fail.
Instead, there could be something like a retry in this case or the exception could simply be ignored if a second check (in the exception handler) shows the table is already there.
What did you see instead?
SQL State : 42000
Error Code : 955
Message : ORA-00955: name is already used by an existing object
Line : 17
Statement : CREATE TABLE "INT_USER"."schema_version" (
"version_rank" INT NOT NULL,
"installed_rank" INT NOT NULL,
"version" VARCHAR2(50) NOT NULL,
"description" VARCHAR2(200) NOT NULL,
"type" VARCHAR2(20) NOT NULL,
"script" VARCHAR2(1000) NOT NULL,
"checksum" INT,
"installed_by" VARCHAR2(100) NOT NULL,
"installed_on" TIMESTAMP DEFAULT CURRENT_TIMESTAMP NOT NULL,
"execution_time" INT NOT NULL,
"success" NUMBER(1) NOT NULL
)
at org.flywaydb.core.internal.dbsupport.SqlScript.execute(SqlScript.java:117)
at org.flywaydb.core.internal.metadatatable.MetaDataTableImpl.createIfNotExists(MetaDataTableImpl.java:93)
at org.flywaydb.core.internal.metadatatable.MetaDataTableImpl.lock(MetaDataTableImpl.java:100)
at org.flywaydb.core.internal.command.DbMigrate$2.doInTransaction(DbMigrate.java:158)
at org.flywaydb.core.internal.command.DbMigrate$2.doInTransaction(DbMigrate.java:156)
at org.flywaydb.core.internal.util.jdbc.TransactionTemplate.execute(TransactionTemplate.java:72)
at org.flywaydb.core.internal.command.DbMigrate.migrate(DbMigrate.java:156)
at org.flywaydb.core.Flyway$1.execute(Flyway.java:1059)
at org.flywaydb.core.Flyway$1.execute(Flyway.java:1006)
at org.flywaydb.core.Flyway.execute(Flyway.java:1418)
at org.flywaydb.core.Flyway.migrate(Flyway.java:1006)

we have same issue..
+1 to fix

Same here. It happened on production with us yesterday. Someone looking into this?

Duplicate of #1067


cc-wrapper: fix wrapper tests and building for darwin on staging-next

Motivation for this change
Fix darwin builds and the cc-wrapper-libcxx tests for staging-next. Opening as draft since I am not sure the first commit is the correct fix for this.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/cc @NixOS/darwin-maintainers I guess?

Polluting the wrappers with this type of logic doesn't seem ideal. Allowing this to be set in the environment and inheriting that from the stdenv seems like a nicer approach. Some similar things are done like that here.
https://github.com/NixOS/nixpkgs/blob/8946ff8d71313fbe9634e242149f1e3fb6aa824b/pkgs/stdenv/darwin/default.nix#L36-L38

I strongly agree, that is why I opened it as a draft. What do you think is the correct approach here, setting NIX_DYNAMIC_LINKER to darwin and checking for that? Or do we need another variable for clarity?

CC @Mic92. I'm curious what exactly want wrong with #91293/#92089/#94582 and #106908.
#106925 in particular was supposed to help with this exact issue?

I would change setDynamicLinker=1 to NIX_SET_DYNAMIC_LINKER=${NIX_SET_DYNAMIC_LINKER:-1}.
@Ericson2314 That's just the test, the stdenv is still broken.

I would change setDynamicLinker=1 to NIX_SET_DYNAMIC_LINKER=${NIX_SET_DYNAMIC_LINKER:-1}.

Okay, I just realized that my first fixup was also broken… it's late here so I'll be going to bed, if this hasn't been resolved by tomorrow I'll work on this some more.

Closing this, since the two changes have been reverted in staging-next.


v4.1.0 reported as malicious by VirusTotal

The latest build's executable flags as malicious "by 13 vendors," as linked on the release page. I'm assuming all is well given that this was just built by github actions, but given that the results are prominently linked right on the release page, this might put off some folks if the results remain linked or aren't noted on the release page.

Hi,
thanks for bringing this up. I did notice that some VT scanners flagged it as malicious, but I guess I thought no one will ever be interested in downloading it 🥲. I added a message to release like you suggested. Executable is 100% safe as it only contains Python code and full tilt boards that are also provided under release and are not themselves flagged as malicious. I guess you probably found my repo while searching for Space Cadet board, in that case I suggest to check out the improved version in the repository linked in README, as the version here is the original win 95 version.
Regards


server can't find celeborn-master-1.celeborn-master-svc: NXDOMAIN

Hello，I want to  consult about RSS on K8s.
when I run helm install celeborn-helm helm -n rss
** server can't find celeborn-master-1.celeborn-master-svc: NXDOMAIN

waiting for master
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Name:   celeborn-master-0.celeborn-master-svc.rss.svc.cluster.local
Address: <IP_ADDRESS>

Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

** server can't find celeborn-master-1.celeborn-master-svc: NXDOMAIN

waiting for master
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Name:   celeborn-master-0.celeborn-master-svc.rss.svc.cluster.local
Address: <IP_ADDRESS>

Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

** server can't find celeborn-master-1.celeborn-master-svc: NXDOMAIN

Your pasted log will be printed until all master nodes are launched, then the issue here is why celeborn-master-1 and celeborn-master-2 pods launch failed.

 Maybe network?

Are there logs on the failed Pod?


fix FZ editor bug in multi monitor scenario

Summary of the Pull Request
Fix for crashing bug when resizing a zone in multi monitor configuration.

References
The bug was introduced in this commit https://github.com/microsoft/PowerToys/commit/c8039828fa301783aca4588c8e2c2091403c92d1

Detailed Description of the Pull Request / Additional comments
The bug effects only multi monitor case where a monitor WorkArea coordinates may be different from {0,0}.
Since the zone rect.X and rect.Y coordinates are relative to the current monitor, the boundary check should be made using X=0 and Y=0 as coordinates of the current monitor regardless the absolute value of WorkArea.X and WorkArea.Y.

Validation Steps Performed
Manual tests.
Verified the fix works when the taskbar is on the left edge of the screen.

@TheMrJukes I'm going ahead and merge this one, in case let me know if there was anything to change. Thanks.

@enricogior Rough ETA for next release? :)

Hi @iKlsR,
we hope to release it this week, but we are still fixing some issues in our build/sign/release automated process, so don't shoot us if we don't make it this week ;)

Looks good, sorry I didn't get to this earlier.


Please mark _bazel_codegen deprecated

We won't be publishing this package anymore.
cc @kevmoo

Done. It is marked discontinued.


Doc.2

Codecov Report

Merging #18 (25a4f3e) into master (75463f2) will increase coverage by 0.00%.
The diff coverage is 73.91%.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   52.95%   52.96%
=======================================
  Files           8        8
  Lines        1318     1316    -2
=======================================
- Hits          698      697    -1
+ Misses        503      502    -1
  Partials      117      117

Flag
Coverage Δ

unittests
52.96% <73.91%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/config/config.go
75.00% <ø> (ø)

internal/matrix/matrix.go
25.83% <ø> (ø)

internal/replication/replication.go
50.40% <ø> (ø)

ipfs.go
65.51% <ø> (ø)

mobile/discovery.go
31.03% <0.00%> (ø)

util.go
60.37% <0.00%> (ø)

mobile/datahop.go
62.81% <83.33%> (+0.19%)
:arrow_up:

internal/repo/repo.go
56.64% <87.50%> (-0.25%)
:arrow_down:

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 75463f2...25a4f3e. Read the comment docs.


Attribute Service Client Configurable Timeout

Description
Like the PR here, this change is to make client timeout for the Attribute Service configurable.
Testing
Deployed and tested the application locally. Scenarios:

Missing config: Default timeout is used.
Timeout configured: Configured value is used.

Checklist:

[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

@aaron-steinfeld Same changes what we did for Gateway service. Please review once you're back. Thanks :)

@suddendust Can you take care of snyk failure? For no upgrade lib, can you update the .snyk files?


opontifex (Twitch streamer)

I've been a streamer for 3 years, I'm not a bot. It's incredible how easy this is to verify and how they replicate this false information.

yes you can stream and still abuse the IRC chat system.
https://twitchinsights.net/bots
Twitch Username	Last Seen In x Live Channels	Last Seen At
opontifex	151	Fri, 26 Jan 2024 22:48:57 GMT
you get on the list automatically when your username is seen in over 150 channels within 24 hours
its a fairly hard thing to do as a "normal" user of the website.  think about it
you must have used something which put your username in those channels

Yes, I have many friends, I visit and am a sub in several. I'm a very good bot then

right.  ill remove you but if you end up back on the list then consider not using the software which is in so many channels simultaneously...
https://github.com/arrowgent/Twitchtv-Bots-List/commit/1a92c86dc5254934e4a724964008d312a14de0df

thank you very much!


Need help to pass an email address in frequency rule as an argument

hi,
i would need your help to find out a way how to pass an email address as an argument rather than hard coding it in the rule yaml file . As per the business requirement, I have email address stored in a csv file. Now I have written a python to fetch the email address. but need to know how to have that integrated with my alert such that the email address is fetched from the csv file.

This isn't really possible right now. If you are able to write Python you could create an alerter module, subclass EmailAlerter, and then add your code to fetch the email from CSV and save it into rule['email'] before calling the superclass alert method.

Hi,
What we have done is below.

We have created a frequency type rule and have kept the alerter method as email.
We have written a python code which will look up  the csv based on a match and fetch the email address.
Now we are trying to explore the feature email_from_field so that we can just put the field name there which will contain the email address. But the email _from_field does not seem to work because if casting issue.
4.The field supplied in the email_from_field value is basically the output of the python code.

We have kept the python code by creating a myenhancement.py .

Additionally can you please let me know as to why the email_from_field property is not working with rule type frequency.
I used it following the doc below
https://github.com/Yelp/elastalert/blob/master/docs/source/ruletypes.rst

The field supplied in the email_from_field value is basically the output of the python code.
What's the exact format?
It needs to be either a string or a list of strings, with or without domain.
You could also solve this using a enhancement. That's basically arbitrary python code you can run on every match, and that code can modify things like recipient email address. It should be pretty simple to code.


feat: Update Cloud Pak for Watson AIOps to 3.7.2

Is your feature request related to a problem? Please describe.
The 3.7.1 release was released on 05/31, so it is time to upgrade this repository.

https://www.ibm.com/docs/en/cloud-paks/cloud-pak-watson-aiops/3.7.2?topic=notes-whats-new

Event Manager:
https://www.ibm.com/docs/en/noi/1.6.8?topic=platform-starter-installation-cli
Describe the solution you'd like
Update the subscription channels and reflect whatever changes are required for the release (e.g. NOI spec.version field)
Describe alternatives you've considered
None, the requirement is pretty narrow.
Additional context
N/A

:tada: This issue has been resolved in version 0.22.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Merge essential content from neardevgov.org into neardevhub.org

Problem:
Currently, we have two websites: https://neardevgov.org (our old identity) and https://neardevhub.org. This is causing confusion for users, as they are not sure which website is most up-to-date. We need to migrate all of the content from the old website.
User Story:
As a user on DevHub, I want to find the relevant information in one place so that I do not have to keep track of multiple sources.
Acceptance Criteria
Blog Posts

[ ] All blog posts are moved to DevHub
[ ] Blog posts must be organized within the appropriate communities
[ ] Replace mentions of DevGov and DevGigs with DevHub

Social Links

[ ] Add social links on DevHub (Twitter, Telegram, YouTube)

Newsletter

[ ] Add NEWSWEEK newsletter signup

Other

[ ] Redirect https://www.neardevgov.org/ to http://neardevhub.org/
[ ] Cancel previous hosting plan

Hi @ori-near, could you please assign this to me. Thanks.


License

Hello,
I would like to use your implementation in my project for academic purposes.
Could you tell me what the license is?

请问一下许可证是什么呢？


getting image metadata

Hi!
Awesome library!! Super fast and really easy to use.
I have a quick question, I been looking through the documentation, there doesnt seem to be a way to fetch image width/height. Maybe I am missing something, but I was looking for something like this
https://hexdocs.pm/elxvips/Elxvips.html#get_image_sizes/1
but obv your lib is much more extensive and support more operations. Could you point me to the right direction to how to get a image max height/width and other meta information?

Hi,
Thanks for your kind words :)
Image metadata is not available yet, I overlooked it I guess. I will try to add it soon.

Added support for image headers. Please check v0.2.0


Add codeclimate config to ignore non-coverage paths

Ignore the catalog and test directories specifically (should improve overall coverage).

https://docs.codeclimate.com/docs/excluding-files-and-folders

closed by #240


Improper Syntax Validation - Jedi - SyntaxError: cannot delete starred - Python39/lib/logging/init.py - Line 1034

Jedi is a static analysis tool for Python that is typically used in IDEs/editors plugins (VS Code).  This line of code is falsely marked as having a syntax error.
https://github.com/python/cpython/blob/1f2e3088f3c097b5bde69bbd63dfcd0852d31984/Lib/logging/init.py#L1034
Environment data
Version: 1.56.0 (user setup)
Commit: cfa2e218100323074ac1948c885448fdf4de2a7f
Date: 2021-05-04T22:09:06.405Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
Python 3.9.4
ms-python.python: v2021.4.765268190
.venv
Python39/lib/logging/init.py
jedi
Expected behaviour
Pass/Validate Syntax at Line
https://github.com/python/cpython/blob/1f2e3088f3c097b5bde69bbd63dfcd0852d31984/Lib/logging/init.py#L1034
del t, v, tb
Actual behaviour
See Picture Here
https://bugs.python.org/file50021/PythonError.jpg
Documented Originally Described Here https://bugs.python.org/issue44064
{
"resource": "/C:/Users/kathl/AppData/Local/Programs/Python/Python39/lib/logging/init.py",
"owner": "generated_diagnostic_collection_name#0",
"severity": 8,
"message": "SyntaxError: cannot delete starred",
"source": "jedi",
"startLineNumber": 1030,
"startColumn": 17,
"endLineNumber": 1030,
"endColumn": 29
}
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Use the Logger (example)
logging.info('I am going to cause a syntax error in VS Code using jedi')

Logs
Python39/lib/logging/init.py
{
"resource": "/C:/Users/kathl/AppData/Local/Programs/Python/Python39/lib/logging/init.py",
"owner": "generated_diagnostic_collection_name#0",
"severity": 8,
"message": "SyntaxError: cannot delete starred",
"source": "jedi",
"startLineNumber": 1030,
"startColumn": 17,
"endLineNumber": 1030,
"endColumn": 29
}

This should be filed on jedi, I have filed this there https://github.com/davidhalter/jedi/issues/1777

parso should be upgraded to 0.8.2. After that everything should work :)
(see also my comment in davidhalter/jedi#1777)


Master

Codecov Report

Merging #674 into develop will increase coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #674      +/-   ##
===========================================
+ Coverage    88.83%   88.87%   +0.03%
===========================================
  Files           64       64
  Lines         3635     3639       +4
  Branches       667      667
===========================================
+ Hits          3229     3234       +5
  Misses         309      309
+ Partials        97       96       -1


tests/subsys/fs/fat_fs_api - test_fat_mount results into assertion failure on Arduino_101 - FS init failed (-19)

The test tests/subsys/fs/fat_fs_api results in Assertion failure in test_fat_mount sub-test case.
Arch: x86
Board: arduino_101
Zephyr commit: c738834854849adad705dee38770aef61bd1b4a2.
Error Console Log:
***** Booting Zephyr OS v1.12.0-284-gc738834 *****
Running test suite fat_fs_basic_test
===================================================================
starting test - test_fat_mount
ASSERTION FAIL [(res == FR_OK)] @ zephyr/subsys/fs/fat_fs.c:333:
        FS init failed (-19)

Steps to reproduce:
cd zephyr/tests/subsys/fs/fat_fs_api/
rm -rf build && mkdir build && cd build
cmake -D BOARD=arduino_101 ../
make BOARD=arduino_101 flash

Check the console log.

Additional information:
As per "git bisect", the commit resulting into this test failure is 2fe51996db89c3e941134178b50c2c6982f9b5c7 by @tbursztyka.
By reverting the mentioned commit, test could pass as expected.

@pswarnak  can you update the status? is it same as before?

Hi @ramakrishnapallala,
The test still fails on ARDUINO_101/x86 with the same error signature:
***** delaying boot 1000ms (per build configuration) *****
***** Booting Zephyr OS v1.9.0-rc1-6413-gef93589 (delayed boot 1000ms) *****
Running test suite fat_fs_basic_test
===================================================================
starting test - test_fat_mount
ASSERTION FAIL [(res == FR_OK)] @ /home/pswarnak/workspace/1.13_zephyr/subsys/fs/fat_fs.c:333:
        FS init failed (-19)

@tbursztyka can you look into this issue ?

@SavinayDharmappa can you please take a look at this while @ramakrishnapallala and @tbursztyka are out?

https://github.com/zephyrproject-rtos/zephyr/pull/9714


wrong highlighting of php string which contains php tag

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.68.1
OS Version: MacOS monterey 12.4

Steps to Reproduce:

create a php file
type following contents

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.69.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I upgrade my VSCode to the latest version (1.69.2) and the bug is still there.

Sorry, it seems that the bug is caused by an extension "EJS Language Support" ......
I only considered extensions about PHP before


UnboundLocalError: local variable 'req' referenced before assignment

events.js:161
throw er; // Unhandled 'error' event
^
Error: write EPIPE
at exports._errnoException (util.js:1028:11)
at WriteWrap.afterWrite [as oncomplete] (net.js:804:14)
events.js:161
throw er; // Unhandled 'error' event
^
Error: write EPIPE
at exports._errnoException (util.js:1028:11)
at WriteWrap.afterWrite [as oncomplete] (net.js:804:14)
events.js:161
throw er; // Unhandled 'error' event
^
Error: write EPIPE
at exports._errnoException (util.js:1028:11)
at WriteWrap.afterWrite [as oncomplete] (net.js:804:14)

What is the problem? What caused the issue? Share some code.

I using python called pageres-cli,the Main code:
from multiprocessing.dummy import Pool as ThreadPool
pool = ThreadPool(10)
results = pool.map(pageres_call_function, urls)
pool.close()
pool.join()

This isn't getting us anywhere to be honest. If you want us to take a look at it, create the simplest example that fails on how you would call pageres from python.

cat urls.txt|xargs pageres

events.js:161
throw er; // Unhandled 'error' event
^
Error: spawn /usr/local/bin/phantomjs EAGAIN
at exports._errnoException (util.js:1028:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:359:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Module.runMain (module.js:607:11)
at run (bootstrap_node.js:422:7)
at startup (bootstrap_node.js:143:9)
at bootstrap_node.js:537:3

Odd, that works perfectly fine for me.
http://www.google.com

$ cat urls.txt | xargs pageres
✔ Generated 1 screenshot from 1 url and 1 size

Try reinstalling/upgrading pageres first.
And please, for the 2nd time, add some explanation to your posts. The more info we have, the better we can help you but it has to come from both sides.

@xl7dev, are you still experiencing this issue?


[BUG] Download files does not work from Exchange Admin Centre.

To Reproduce

Browse to portal.azure.com.
Connect to Cloud Shell (icon top right).
Upload a file to Cloud Shell (eg, sample.txt) - Manage Files | Upload file
Run the command download sample.txt (popup shows file with link and clicking the link downloads the file. This also works for Manage Files | Download file).
Browse admin.exchange.microsoft.com.
Run the command download sample.txt (popup shows file with link and clicking the link shows a new tab but file does not download).
Select Manage Files | Download file, type in sample.txt, click the Download button. Download File popup displays with link but clicking the link has the same result as step 6.
As a bonus try to click Manage files, Open File Share (nothing happens).

Commands you ran
download sample.txt
Observed Behavior

All download options work in the Azure portal and Shell.
All download options fail when using Cloud Shell in the Exchange Admin Centre.

Note: File upload works in both the Azure Portal and Exchange Admin Centre.
Include error output here
No error messages, all appears to work but the file download never initiates.
Expected behavior
I expect the downloading files using the Cloud Shell to work in the same manner regardless of the portal being used.
Is this specific to Cloud Shell?
Yes
Interface information
https://admin.exchange.microsoft.com - Fail
https://shell.azure.com - Success
https://portal.azure.com - Success

The Microsoft 365 team have a case open on this under tracking id<PHONE_NUMBER>000129.

Hi @kiwiant, thank you for reporting this issue.  Platforms supporting Cloud Shell outside of Azure Portal control some of the UX code, and this is likely a discrepancy between the Portal and Admin Center UX handling.  Will reach out to the Admin Center team to ensure they are aware.


[cherrypick-dev][Fix][Dag] Timeout alert.

Purpose of the pull request
#8047

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #8049 (b7e375a) into dev (6774827) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head b7e375a differs from pull request most recent head 1861a96. Consider uploading reports for the commit 1861a96 to get more accurate results

@@             Coverage Diff              @@
##                dev    #8049      +/-   ##
============================================
- Coverage     41.33%   41.31%   -0.03%
+ Complexity     3730     3727       -3
============================================
  Files           637      637
  Lines         27129    27129
  Branches       3083     3083
============================================
- Hits          11215    11209       -6
- Misses        14838    14841       +3
- Partials       1076     1079       +3

Impacted Files
Coverage Δ

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (-2.00%)
:arrow_down:

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 6774827...1861a96. Read the comment docs.


Customize the content of the announcement and bring back the previous version of the Chinese announcement

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Example: I'm always frustrated when...
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional Information
Add any other context or screenshots about the feature request here.
Add Customize the content of the announcement and bring back the previous version of the Chinese announcement

Not sure what you mean here, feel free to elaborate more and we can reopen the issue if needed.


AttributeError: function 'SetsConsoleMode' not found

I tried to run the test according to the README, but it gave me the error shown in the picture and I could not find any function or class named "SetsConsoleMode" in the source file.

My version of python is 3.10.12

I found the problem, it was the colored library's attr function that was causing the error


'docs/img/city_map.jpeg' not found during rendering

Calling render on "CVRP-v1" doesn't work because the docs/img/city_map.jpeg file cannot be opened:
https://github.com/instadeepai/jumanji/blob/823830e411b6e54da84985018555637d37c6bd8d/jumanji/environments/routing/tsp/viewer.py#L137
Perhaps a way of downloading that file in a cache could solve the issue?

Hi, thank you for raising this issue! Will look into this shortly.

Cool!
FYI it's part of this PR
https://github.com/pytorch/rl/pull/2129
which enables rendering of envs using from_pixels=True in torchrl
Sort of personal request: It'd be nice to have a way to ask for headless rendering / rgb_arrays in a consistent way (from my 10K feet high view it seems envs have a different way of dealing with rendering). Something like make(envname, rgb_array=True) or render(rgb_array=True).
For torchrl, we prefer always getting arrays of uint8 and deliver them to the users as part of their env output and leave them deal with them as they want (if the user wants to render smth they can just call plt.imshow(data["pixels"]))
Having the human rendering by default usually introduces some consequent overhead.
The way the above PR deals with this is to render on screen, record the content of the plt image in a buffer, convert that buffer to a tensor (since there is no generic way of getting a rgb).

Sorry it took so long @vmoens but this should be fixed now :smile:


JSON Decode Error

@shiyuechengineer, running into a minor issue that I think we fixed in the legacy meraki.py library.  Or maybe I re-wrote it for a project, I can't remember.
Basically if someone mistypes an API key you get a series of errors but the most relevant is from the requests library.  Because the dashboard still returns a web page you get a 200 HTTP response but invalid JSON from requests response.json() (basically the dashboard response is HTML but the requests library expects json)
File "/site-packages/meraki/rest_session.py", line 126, in request response.json()
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 2 column 1 (char 1)
Can we trap for this error? I trap for it often to give a custom user message.  I see it when people are using my tools based on your library but either use the wrong API key or mistype it.
If you can think of a work around without monkey patching (which I don't want to do), please let me know.  It's easy to re-create, just use any invalid API key.

Also, that error should be a trapped by either except ValueError: or more specifically json.decoder.JSONDecodeError:  but neither seem to work - still testing.  I think it has something to do with dependencies in imported modules and "During handling of the above exception, another exception occurred:" multiple errors.

Hey Zach, that JSON error should be trapped under this block if it was a 200 response with invalid JSON returned. However, if the original issue was an invalid API key, then the response in V0 should be a 404 and in V1 401 with message "Invalid API key".

Zach, anything else you need here before closing this issue? Thanks.

Shyue, Sorry I missed this.  The json.decoder.JSONDecodeError: doesn't catch the exception b/c the API returns HTTP content when an API key is bad.


Horizontal scroll not working on Windows

It seems that pressing shift key while scrolling the mouse wheel has no effect on Windows, contents of the canvas move up or down, instead of left or right. On Mac it works as expected.
I was able to reproduce this with the latest versions of Chrome and Edge on Windows 10.

Thank you for the report. I've opened a draft PR with the fix here: https://github.com/statelyai/xstate-viz/pull/230

Just tried the fix and it works great!

@a-uz could you confirm that you are using an actual mouse? I have some doubts about this fix: https://github.com/statelyai/xstate-viz/pull/230#discussion_r698792935

@Andarist Yes, I'm using an actual mouse.
I did some more testing and with the current production version on Windows trackpad interactions work just fine, but shift key + mouse wheel doesn't. With the PR version shift key + mouse wheel works, but shift key + trackpad is now strange: I can only drag up and down to pan left or right and there's no way to pan up or down. TBH, I'm not even sure how it should be.

I tried to use shift key + trackpad on Figma and Draw.io. On Figma's canvas holding shift key and dragging up or down pans left or right, dragging left or right pans up or down. On Draw.io's canvas holding shift key and dragging in all directions pans left or right.

@a-uz I've pushed out some changes, could you give this a spin? I would also appreciate it if you could rephrase your Figma & Draw.io research because I'm unsure if I got this right. Mentioning the exact combination of shift, wheel/trackpad, physical direction and the resulting pan direction for all of those cases would help me a lot.

@Andarist Now the mouse + shift key combo doesn't work anymore and trackpad + shift key works the same as without the shift key.
Dragging on Figma's canvas with trackpad + shift key inverts the direction: dragging vertically pans horizontally and dragging horizontally pans vertically. Dragging on Draw.io's canvas with trackpad + shift key pans horizontally regardless of drag direction.
I guess I would prefer the visualizer to have Figma's approach by inverting the direction when shift key is pressed.

Ok, thank you for your feedback - gonna take another look at this tomorrow. I'm a little bit surprised though that shift+mouse doesn't work - maybe some kind of a silly mistake in the adjustments I've made.

Not sure if this is related (or even a feature at the moment), but dragging to scroll (with middle mouse held) does nothing on windows/chrome as well. With mouse there seems no way to pan around. I tried a variety of mouse buttons and modifier keys without success. Fortunately using a touchpad with multitouch works fine, and I do have one available.

dragging to scroll (with middle mouse held) does nothing on windows/chrome as well.

Could you describe in more detail what dragging to scroll means in this context?

With mouse there seems no way to pan around. I tried a variety of mouse buttons and modifier keys without success

You should be able to hold Space key and then drag around the canvas with your left mouse button.

Functionality wise pressing mouse wheel and dragging == holding Space and dragging. Usability wise it doesn't require a keyboard.

Ok, I see - expect a PR fixing that tomorrow 😉

Space does indeed work, and like @a-uz said, it would be quite convenient if middle-mouse-drag had the same behavior as well.
Thanks for the quick response that even made it in time for a meeting 😁

I don't quite have access to a mouse with a middle button - could you check if this works correctly for you: https://xstate-viz-git-andarist-wheel-panning-statelyai.vercel.app/viz ?

It doesn't seem to work correctly. Here's a short video:
https://user-images.githubusercontent.com/6834140/133439111-dc0faf16-aa7b-400c-8309-c7f78d6f3f6d.mp4

Would you be able to record a replay using https://www.replay.io/ ? That is an amazing product that has just shipped today and would allow me to debug everything that happens during your session. Please only remember to log out before starting to record because this actually records everything.

I already tried that, but there seems to be a bug with the recorder itself https://app.replay.io/recording/5362dc4e-23b1-47de-88c0-ffb7756b98d9

@a-uz thanks for the attempt to record this. I've filed an issue in Replay here: https://github.com/RecordReplay/devtools/issues/3626 . If you have any additional context that you could share there it would be awesome.
As to the issue here - I will try to simulate this on my own to see if I can quickly find anything that is obviously broken.


FixExpandableTables

Summary
This simply fixes an issue found in IQE testing.
Each table row must be wrapped in a tBody element to be targetable by the IQE component, this pattern is also recommended by patternfly.
Additional Changes:

Fixed a bug around clearing the snapshotErrataFilter from empty state.

Testing steps

/retest

/retest

/retest