Investigate running lodestar on Nimbus's insecura - the "long ranged attacked" pyrmont fork

Nimbus team created a long range attack scenario on Pyrmont network as described here: https://notes.status.im/nimbus-insecura-network?view
tl-dr: They rewrote pyrnmont chain from epoch 60000 for 12k validators way back by using these validators (leaking the other 100k something validators on this alternate chain) so that this alternate chain finally finalized after exiting the majority "non attacking  validators".
Now these node/validators are serving alternate chain. We want to test out the wss sync related issues using lodestar.
Check test following scenarios:

[x] Scenario 1: sync on the attacking chain using a compromised checkpoint and see further finalization: see comment
[x] Scenario 2: sync from the non attacked checkpoint (<= epoch 60k) to the attacking chain using the compromised bootnodes UPDATE: couldn't sync  see  comment
[x] Scenario 3: syncing the prater on forward sync from blank db see comment

[x] Sync  started - OPINION : sync shouldn't have started
[x] validate if it sync to the present on the attacking chain

UPDATE: following two scenarios can't be tested because insecura is (invalid) continuation of the abondoned pyrmont chain, so no checkpoint exists on pyrmont which is not part of insecura to test the following two points:
- [ ] Scenario 4: invalidate the sync on the attacking chain with compromised bootnodes but using a valid forward checkpoint (using the forward checkpoint PR) https://github.com/ChainSafe/lodestar/pull/3589
- [ ] Scenario 5: Invalidate the backfill sync on the attacking chain with a wss sync but a valid checkpoint in past from the normal chain

UPDATE: Nimbus's wss endpoint seems to be loaded/stuck, not moving past fetching the state from their provided sync url :
bash-5.1# ./lodestar beacon --network pyrmont --eth1.enabled false --rootDir /data/insecura --weakSubjectivitySyncLatest --weakSubjectivityServerUrl http://insecura.nimbus.team
Jan-22 10:37:05.729 []                 info: Lodestar version=v0.26.0/master/+465/f9cb593d (git), network=pyrmont
Jan-22 10:37:05.741 [DB]               info: Connected to LevelDB database name=/data/insecura/chain-db
Jan-22 10:37:05.742 []                 info: Fetching weak subjectivity state from http://insecura.nimbus.team/eth/v2/debug/beacon/states/finalized

^CJan-22 10:37:41.851 []                 info: Stopping gracefully, use Ctrl+C again to force process exit
bash-5.1# ./lodestar beacon --network pyrmont --eth1.enabled false --rootDir /data/insecura --weakSubjectivitySyncLatest --weakSubjectivityServerUrl http://insecura.nimbus.team --network.discv5.bootEnrs enr:-LK4QGQl-6vfK7JdE8zPVzk9rMrXI1myBuy9xFZfA4JgEJm3ZScaoyyfPy3t6X57tY2G7gLK9zMzSkuFI1Hf0kRv7r4Bh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQNmNsAV4a5TWnfEArZrHUim553RymwmAbpAZxc3RoP1S4N0Y3CCIyiDdWRwgiMo enr:-LK4QKbOD5MlwM_BE7uVLwpRIwhlsFxiiOyReRhUBc7jgzblf8EXG5MccDxuRChfRqthkSo4wwC_ICECy2u8QVjx1esBh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQOV-szjqex-IuG7NeNcIhp6jvc8j-hsFnNi14a9FTYGToN0Y3CCIyqDdWRwgiMq enr:-LK4QNymCG1DTf2rrVqzz9yTwU3-lE8_qrufLtUmlGU7nlsib4QNEpr_Csm_hfAjnjYL7uDcjTRkLLti9lwZH_UxnaQBh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQMZ5KZLn-wqfc1FBWmeIn2lqsMmeUPI7nCerJQuBrOzLIN0Y3CCIymDdWRwgiMp
Jan-22 10:38:53.208 []                 info: Lodestar version=v0.26.0/master/+465/f9cb593d (git), network=pyrmont
Jan-22 10:38:53.219 [DB]               info: Connected to LevelDB database name=/data/insecura/chain-db
Jan-22 10:38:53.221 []                 info: Fetching weak subjectivity state from http://insecura.nimbus.team/eth/v2/debug/beacon/states/finalized

UPDATE: synced on the attacking/alternate/invalid chain using compromised wss checkpoint and compromised bootnodes serving invalid chain
an-22 12:19:13.001 []                 info: Synced - slot: 3096095 - head: 3096095 0x04ee…2461 - finalized: 0xa3c0…d03e:96748 - peers: 2
Jan-22 12:19:25.001 []                 info: Synced - slot: 3096096 (skipped 1) - head: 3096095 0x04ee…2461 - finalized: 0x7994…ffb9:96749 - peers: 2
Jan-22 12:19:37.001 []                 info: Synced - slot: 3096097 - head: 3096097 0xc9a6…f93b - finalized: 0x7994…ffb9:96749 - peers: 2

UPDATE: Error while trying to sync from a finalized checkpoint < epoch 60,000 (a correct checkpoint) but from the attacking chain as the checkpoint way behind the current clock epoch
bash-5.1# ./lodestar beacon --network pyrmont --metrics.enabled  --eth1.enabled false --rootDir /data/insecura1 --network.discv5.bootEnrs enr:-LK4QGQl-6vfK7JdE8zPVzk9rMrXI1myBuy9xFZfA4JgEJm3ZScaoyyfPy3t6X57tY2G7gLK9zMzSkuFI1Hf0kRv7r4Bh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQNmNsAV4a5TWnfEArZrHUim553RymwmAbpAZxc3RoP1S4N0Y3CCIyiDdWRwgiMo enr:-LK4QKbOD5MlwM_BE7uVLwpRIwhlsFxiiOyReRhUBc7jgzblf8EXG5MccDxuRChfRqthkSo4wwC_ICECy2u8QVjx1esBh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQOV-szjqex-IuG7NeNcIhp6jvc8j-hsFnNi14a9FTYGToN0Y3CCIyqDdWRwgiMq enr:-LK4QNymCG1DTf2rrVqzz9yTwU3-lE8_qrufLtUmlGU7nlsib4QNEpr_Csm_hfAjnjYL7uDcjTRkLLti9lwZH_UxnaQBh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQMZ5KZLn-wqfc1FBWmeIn2lqsMmeUPI7nCerJQuBrOzLIN0Y3CCIymDdWRwgiMp --weakSubjectivityServerUrl http://insecura.nimbus.team --weakSubjectivitySyncLatest --weakSubjectivityCheckpoint 0x1c8fc8f25001641b25a3572837962684def277fa8f525a86c349ccd8fd2d818b:59998
Jan-24 10:00:09.470 []                 info: Lodestar version=v0.26.0/g11tech/wssurlversion/+469/e6f2585a (git), network=pyrmont
Jan-24 10:00:09.612 [DB]               info: Connected to LevelDB database name=/data/insecura1/chain-db
Jan-24 10:00:10.834 []                 info: Fetching weak subjectivity state weakSubjectivityServerUrl=http://insecura.nimbus.team
 ✖ Error: Fetched weak subjectivity checkpoint not within weak subjectivity period.
    at initAndVerifyWeakSubjectivityState (/usr/app/packages/cli/src/cmds/beacon/initBeaconState.ts:58:11)
    at initBeaconState (/usr/app/packages/cli/src/cmds/beacon/initBeaconState.ts:115:12)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Object.beaconHandler [as handler] (/usr/app/packages/cli/src/cmds/beacon/handler.ts:75:41)

lodestar started syncing on from blank db without using checkpoint sync...
./lodestar beacon --network pyrmont --metrics.enabled  --eth1.enabled false --rootDir /data/insecura --network.discv5.bootEnrs enr:-LK4QGQl-6vfK7JdE8zPVzk9rMrXI1myBuy9xFZfA4JgEJm3ZScaoyyfPy3t6X57tY2G7gLK9zMzSkuFI1Hf0kRv7r4Bh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQNmNsAV4a5TWnfEArZrHUim553RymwmAbpAZxc3RoP1S4N0Y3CCIyiDdWRwgiMo enr:-LK4QKbOD5MlwM_BE7uVLwpRIwhlsFxiiOyReRhUBc7jgzblf8EXG5MccDxuRChfRqthkSo4wwC_ICECy2u8QVjx1esBh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQOV-szjqex-IuG7NeNcIhp6jvc8j-hsFnNi14a9FTYGToN0Y3CCIyqDdWRwgiMq enr:-LK4QNymCG1DTf2rrVqzz9yTwU3-lE8_qrufLtUmlGU7nlsib4QNEpr_Csm_hfAjnjYL7uDcjTRkLLti9lwZH_UxnaQBh2F0dG5ldHOIAAAAAAAAAACEZXRoMpB0ZesLAQAgCf__________gmlkgnY0gmlwhEEVxC2Jc2VjcDI1NmsxoQMZ5KZLn-wqfc1FBWmeIn2lqsMmeUPI7nCerJQuBrOzLIN0Y3CCIymDdWRwgiMp
Jan-24 10:48:33.411 []                 info: Lodestar version=v0.26.0/g11tech/wssurlversion/+469/e6f2585a (git), network=pyrmont
Jan-24 10:48:33.419 [DB]               info: Connected to LevelDB database name=/data/insecura/chain-db
Jan-24 10:48:39.611 []                 info: Initializing beacon state from anchor state slot=0, epoch=0, stateRoot=0x2bb257ca66d05a047a65fe43a5f457b674de445d917cca029efb09b3ba4758c4
Jan-24 10:48:43.480 [SYNC]             info: BackfillSync - initializing from Checkpoint root=0x6a89af5df908893eedbed10ba4c13fc13d5653ce57db637e3bfded73a987bb87, epoch=0, slot=0
Jan-24 10:48:43.488 [METRICS]          info: Starting metrics HTTP server port=8008
(node:7245) MaxListenersExceededWarning: Possible EventEmitter memory leak detec

CLOSING issue as Scenario 4,5 not possible to check, see description for the reasons.

Thank you so much for doing this tests! Love to see the security protections in action

Thank you so much for doing this tests! Love to see the security protections in action

@dapplion  Should we apply weak subjectivity timeperiod checks on start from blank DB as well (Scenario 3 in the description)? which we don't do right now. This means one would not be able to start from a blank DB without providing a checkpoint.

Should we apply weak subjectivity timeperiod checks on start from blank DB as well (Scenario 3 in the description)? which we don't do right now. This means one would not be able to start from a blank DB without providing a checkpoint.

What do other clients do?


taphold not firing

I'm trying to work with taphold function, as given in doc
https://framework7.io/docs/tap-hold.html
below is my screen shot of my application, when user long press on slots, i need to call a relevant function for cancelling the slot booking.

Below is HTML for rendering slots (using angualrjs )

Below is js script for firing event.

I've also referred https://github.com/nolimits4web/Framework7/issues/405
but I couldn't find any use.
Kindly could any one assist me in firing the event.

Hi
You should add taphold property to true at app initialization
Then remove app variable from event handler
Finally try to bind this event on page container or on document
Hope this help you

@masmanidis , I have added taphold property to true.
Below in my F7 configuration.

So
Use $$(document).on(“taphold”,“.list-item”, function(){});
or $$(page.container).on(“taphold”,“.list-item”, function(){});
[This is a suggestion I've got from f7 forum!]

@masmanidis 👍  temporary I have got response on taphold
using
$$(document).on(“taphold”,“.list-item”, function(){});
but will debug and update with any consequences arises.

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Create pnpm-debug.log file on fail

I created a custom logger for this feature (please review it as well @zkochan/logger). If it is fine, I'll transfer it to the pnpm organization.
The main idea is to use a logger which emits events with the logging info. The file logger records all the different logging levels. The progress logger listens only the progress type.
Also I substituted the debug usages with @zkochan/logger usages. But I did not remove debug, it listens now for debug logging events.
Example output: https://gist.github.com/zkochan/272e4c60eb3248373c63af9bc2db8924

I am merging it because I have to create a new pull request based on these changes.
But I'll be more then happy to make updates if someone will have comments.

Looks good

Just one question: why do we log only progress events?

We don't log progress events. They are not useful.

I mean the progress events that log download percentage

ok, got it. thanks


separated out UIListLayout and UIGridStyleLayout

Coverage increased (+0.02%) to 97.183% when pulling 7417b1d1168ef83c1917f5f7d5b978ce768644aa on aportner:add-ui-grid-style-layout into ac0a04fed063d14390815b48683b733a7706c7c8 on LPGhatguy:master.


Github listing as JavaScript when it should be c#

Hi,
The programming language listed for the repo below says JavaScript but its actually C# - can this be changed please?
https://github.com/asudbury/spectrum
thanks
adrian.

When you filed this, there should have been a list of steps to help you resolve this issue yourself.
What happened to them?

i read those articles - didn't make a lot of sense!
its not that important - please close this issue if it cant easily be resolved.
I love github and im not trying to be awkward!
thanks
adrian.

i read those articles - didn't make a lot of sense!

Could you detail what you think doesn't make much sense? That would help us improve our templates and guidelines for future contributors.
Now, given your issue, I'd recommend reading My repository is detected as the wrong language in the README.

Hi,
First of all i didn't know what the 'language stats bar' was! i do know after googling it :-)
I think if you had included some sort of graphic it in the article to explain where to find it - it might be obvious to you, it might be obvious to a lot of people but it wasn't to me.
Following the article you provided a link for - it mentions moving files to a vendor - i cant do that.
I had assumed it would be a github setting to override the language - i know now that not the case.
You can close this issue.
thanks
adrian.

Thanks for the suggestion. I'll look into it!

If you see files that you didn't write in the search results, consider moving the files into one of the paths for vendored code, or use the manual overrides feature to ignore them.

You might want to pay closer attention to the third phrase of this sentence ;)
You do have more option than moving files to a vendor directory: https://github.com/github/linguist#overrides.


fix: disable microphone button, show only on chatgpt and anthropic website

I accidentally had pushed the Chrome extension to the web store before it was ready and wanted to get this fix out as soon as possible!
For now, I'm disabling injecting the microphone button in all websites EXCEPT chatgpt.com and claude.ai. This will fix many of the errors people are having as it was unintentional functionality.
Please let me know if people are interested in restoring the button on websites other than chatgpt.com and claude.ai. I will create adapters for those desired websites.
closes

#121
#120
#123
#118
#114

check out Dark Reader. There you can decide on which page it should be active. Maybe combined with setting if per default on or off. :)
Thanks for an asome application.


No plugins shown as configured; still running subfont

Something is very broken somewhere. Despite having disabled subfont everywhere, it's still running in my pipeline.

If a plugin is running even when it is not a part of your netlify configuration file, I think this is an issue that belongs in https://github.com/netlify/build/issues


Resolve redundant-host issue with www.k8s.dev

kubernetes.dev has a number of domain "aliases" that are redirected to the primary site, but  www.k8s.dev isn't redirected, which analytics reports as a redundant-host issue.
$ curl -i https://www.k8s.dev 2>/dev/null | grep ^H | head -1
HTTP/2 200

In contrast, notice how k8s.dev is appropriately redirected:
$ curl -i https://k8s.dev 2>/dev/null | grep '^[Hl]' | head -2
HTTP/2 301
location: https://www.kubernetes.dev/

/cc @nate-double-u

/remove-lifecycle stale


Feature/add additional jvm params at runtime through an optional wrapper.conf file

The location for the wrapper conf is currently in $SERVICE_HOME/bin. Not convinced on the location but that can be changed later.
Note - we may not actually need this as $JAVA_OPTS etc. can be set manually to add additional args. Leaving this here for archival purposes - in case we need to revive it later.
NB: I didn't add support for this in the windows startup scripts yet.

Going to decline this since I think this is covered by JVM_ARGS settings that override default parameters anyway.


partial-payment-page

Implemented the ui for partial-payment page
Created a sub-menu route in the side-bar
issue #31

@MeisterJustice  resolved

Is this connected in the sidebar? cos it's not showing up when i click partial payment @Khadeeejah

@Khadeeejah you added partial payment as a sub menu for transfers. Is that how it's supposed to be?

@MeisterJustice, I'm not sure I understand. Which deployment preview are you looking at?
This is the most recent and final one - https://lazerpay-documentation-v2-8o58xtxqn-lazerpay.vercel.app/

https://lazerpay-documentation-v2-8o58xtxqn-lazerpay.vercel.app/

cool. When you are here https://lazerpay-documentation-v2-8o58xtxqn-lazerpay.vercel.app/docs/payments/get-started and you click on partial payment, to doesn't take you to the page you worked on

@MeisterJustice  kindly check the latest deployment,  here https://lazerpay-documentation-v2-bgncxs69g-lazerpay.vercel.app/docs/payments/get-started
i have fixed the routing


fix(es/parser): Fix nested generics

Description:
Related issue:

Closes #8209

I think we have to re-apply https://github.com/swc-project/swc/pull/7439

Closing in favor of https://github.com/swc-project/swc/pull/8607


Attribute helpers

Almost every custom derive implementation will need to work with attributes.
I think working with them is really unergonomic right now though.
It often requires nested matches of 3 levels, which is quite cumbersome.
I'd really love some helper functions that make finding and identifying attributes easier.
If maintainers are ok with the idea, I'd start working on a PR.

I would prefer for this to be handled by other crates, for example darling. There are a lot of design tradeoffs around how to handle attributes and different crates may be able to optimize for different use cases. This is especially true now that https://github.com/rust-lang/rust/issues/34981 intends to allow arbitrary tokens inside of attributes.


Cant find Dyson local and Dyson Cloud in Integration

Hey mate,
i have install both Dyson Cloud and Dyson Local in my Home Assistant
and restart it.
after that, i still cant find anything via searching Dyson in my Configuration -> Integrations

Anything i missed?

sorry mate,
I can find those now. But when i try to add them, error messages pops up
it says that this is not support to config via UI

yeah I get the same for both Dyson Cloud and Dyson Local. I've installed HACS and added the custom repository and downloaded both the dyson cloud and dyson local custom integration. They now show up in my Add Integrations list:

but when I try to set either up I get the following error:

Error
This integration does not support configuration via the UI. If you followed this link from the Home Assistant website, make sure you run the latest version of Home Assistant.

I'm running Home Assistant Core 2022.8.5 on HAOS 8.2.

now i can see the pop up but still having the error while log in via Dyson Cloud or enter the SSID and WIFI password via Dyson Local

I got this error because I was still logged onto the dyson cloud via my app.  I logged off the all and was then able to login via HA

I got this error because I was still logged onto the dyson cloud via my app. I logged off the app and was then able to login via HA

Hey mate,
does that mean only one of HA and App can connect via cloud?

There was a discussion in that but I don't think there was a conclusion.  My own opinion is yes, only one cloud connection. I used the HA cloud integration just to get the settings for the local integration.  I then removed the HA cloud as I didn't need both so now have HA local and my phone app connected to the cloud.

There was a discussion in that but I don't think there was a conclusion. My own opinion is yes, only one cloud connection. I used the HA cloud integration just to get the settings for the local integration. I then removed the HA cloud as I didn't need both so now have HA local and my phone app connected to the cloud.

When you mention HA Local, are u talking about the Dyson Local?  but i tried to use dyson Local to connect it but still failed.

Yes - I mean HA dyson local ... Initially I had issues with installing the two integrations but a fix was made and I was then able to proceed.
I use HACS to download both the dyson cloud then dyson local packages.  I then used the documentation to do the following:
step 1 was to add the HA Dyson cloud integration (logging out of the dyson app on my phone ahead of doing this)
step 2 was to configure and start the HA dyson cloud integration - this provided the data to be able to do step 3
step 3 was to configure and start the HA Dyson local integration
step 4 was to remove the HA dyson cloud integration (I still have it in HACS but the integration has been removed from settings>integrations)
step 5 was to log back onto my dyson app on the phone.
I can now use HA to see, via the dsyon local integration, the info from the device and also control the device.

yeah I get the same for both Dyson Cloud and Dyson Local. I've installed HACS and added the custom repository and downloaded both the dyson cloud and dyson local custom integration. They now show up in my Add Integrations list:

but when I try to set either up I get the following error:

Error
This integration does not support configuration via the UI. If you followed this link from the Home Assistant website, make sure you run the latest version of Home Assistant.

I'm running Home Assistant Core 2022.8.5 on HAOS 8.2.

same issue here, anybody figured away to make it work?

Yes - I mean HA dyson local ... Initially I had issues with installing the two integrations but a fix was made and I was then able to proceed.
I use HACS to download both the dyson cloud then dyson local packages. I then used the documentation to do the following:
step 1 was to add the HA Dyson cloud integration (logging out of the dyson app on my phone ahead of doing this) step 2 was to configure and start the HA dyson cloud integration - this provided the data to be able to do step 3 step 3 was to configure and start the HA Dyson local integration step 4 was to remove the HA dyson cloud integration (I still have it in HACS but the integration has been removed from settings>integrations) step 5 was to log back onto my dyson app on the phone.
I can now use HA to see, via the dsyon local integration, the info from the device and also control the device.

How did you get you configure the HA Dyson Cloud integration, did you use the UI or the config file?  If you use date config can you post an example?


docs(README): Remove binaries

Neither Kent nor I can get pkg to compile this thing anymore, and the old binary locations aren't accessible anymore. Let's remove the option to avoid confusion until we solve it.

Codecov Report

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

@@          Coverage Diff          @@
##           master    #89   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines         373    373
=====================================
  Hits          373    373

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 9e4a42a...e0f7549. Read the comment docs.


set periodEnd to the next month if subscription cancelled at perio…

This one appears to be included in #3094, so I'll close this one and review over there.


Add UTF-8 charset meta to html-gen.ts

Fixes issue #43, allows for UTF-8 characters to be dislayed, e.g. emojis 🤯

Thanks I'll merge this. Unfortunately, I don't have the time at the moment to set up a release. But hopefully one should be on it's way sometime next week along with a bunch of other changes.

Nice @beelux , thanks for the pull request.


Introducing a new option: maxOccupationTime

maxWaitTime is another form of timeout that covers some use-cases that timeout option might not be useful.
In case of timeout, Async-lock would throw an error, and then the queue will lose all of its waiting items.
maxWaitTime is a timeout for every single item and if the function exceeds the time, we release the lock and let the next task acquire the lock without canceling the unfinished task.
In collaboration with @ThePiz, we suggest this new option as we are using Async-lock in our project & this feature is essential for our use case.

Hi @abozaralizadeh
What is your use case exactly?  Releasing locks without cancelling the task sounds like it makes everything you do from that point onward would be incorrect/unpredictable?   What guarantees would the user still have after that point?
There is one likely bug in your code, namely that you never cancel the timer. I think it should be canceled wherever the lock is released for another reason than your wait timeout.
If you want this integrated, please also create unit tests (preferably better ones than the existing ones, e.g. use sinon)  and update README.md to explain your new feature.
If that would be too much work for you, you might consider just forking the repository and making it work for your project.  You can directly reference repositories in your package.json, so you don't actually need it to be published.

Hi @rogierschouten
Thank you for your comment.
Locks can be used not only for protecting critical sections but also in other use-cases such as avoiding race conditions or guaranteeing arbitrary ordering. In our case, the Async-lock helped us to implement a fairly ordered message delivery system using unordered Ack messages. the lock would wait for the Ack message to unlock the next message, but in case of losing the Ack, we implemented this new option to unlock the queue anyway after exceeding the max occupation time.
We are already using our fork in our project but I was sure that this option might be useful also for others so I decided to suggest the current PR.
After reading your comment I decided to generalize the option and raise an error to give the handling part to the programmer to decide. it might be 'not canceling the task' and just releasing the lock as in our use-case or canceling it and do whatever is needed to be done (although some jobs can’t be simply undone/canceled)
As you asked I also added related unit test and modified the README.md file.
Thank you

That is starting to look really good!  I still think that you need to cancel the timer in more places?  Now, when the lock is released, the timer still runs.  Maybe in the done function?
Please tell me if I'm wrong, I am not really into this code.

Thanks! actually, I believe that right now it should be just fine. every time the lock is released naturally before the timer,
1. If there is another task in the queue, it will clear the timer here:
			if (occupationTimer) {
				clearTimeout(occupationTimer);
			}

and set the new timer quickly afterward.
			occupationTimer = setTimeout(function () {...});

2. If the task is the last one in the queue, the queue will be deleted, so when the timer due time arrives and runs it's function, the first condition if (!!self.queues[key]) will be false, so basically nothing happens:
	occupationTimer = setTimeout(function () {
		if (!!self.queues[key]) {
			done(false, new Error('Maximum occupation time is exceeded'));
		}
	}, maxOccupationTime);

Finally, in case of error due to 'Maximum occupation time is exceeded', it's the programmer's job to handle the error and release the lock as described in the previous comment.

f the task is the last one in the queue, the queue will be deleted, so when the timer due time arrives and runs it's function, the first condition if (!!self.queues[key]) will be false, so basically nothing happens:
Yeah that is what I meant, even though it is guarded, it is a timer that runs unnecessarily.  Let me know if you still want to fix it or not, I will merge the PR otherwise.

I see, I just moved the reset part to the very beginning of done and that should be more accurate as it will reset the timer every time after release/error. I think this is better than the previous commit as it solves both unnecessary timer in the last item and also reset the timer in any other case of error. Thanks

Thank you @abozaralizadeh and @ThePiz
Published as v1.3.0.    I forgot to add both of you as contributors so I did that in the next commit after the release.


[WIP] Offloading and Multiple devices quantization/inference

This pr will add supports to quantize model and inference on quantized model using multiple devices, and choose to offload to cpu when model is too big to fit into gpus.
And tanks for qwopqwop200's efforts, gpt2 type models are also supported.

At this stage, quantize model on multiple gpus (requires the hole model is in gpu, no layer dispatched to cpu or disk) is supported, and one can also use the old way to first load model at cpu then quantize on gpu layer by layer.

At this stage, CPU offload when doing quantization is supported, so now a user can quantize 20B model with 16GB cpu memory and 24GB gpu memory

At this stage, thanks to qwopqwop200's efforts, gpt2 type models quantization and inference are supported, and some bugs that only occur when using pipeline to do inference with CPU offloading are also fixed by him.
However there is still a device mismatch problem if one use pipeline to do inference on multiple gpus, but none of those problems happen if using model.generate.

At this stage, the device mismatch bug when directly inference after quantization (not save-and-reload) on multiple devices is solved.

At this stage, one can use large batch size to speedup quantization


Add Secp256k1 support

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] Commit messages follow the conventional commits spec
[x] If this is a code change: I have written unit tests.
[x] If this changes code in a published package: I have run pnpm changeset to create a changeset JSON document appropriate for this change.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Adds support for secp256k1 curve
Test Plan

All tests pass
Related issues/PRs

https://github.com/near/near-api-js/pull/985
https://github.com/near/near-api-js/issues/1039
https://github.com/near/near-api-js/issues/440

I gave it a quick look, do we have any test against the chain using secp256k1? I think the current tests only look for internal consistency (i.e. what we signed, can be checked) but not necessarily for external consistency (what we signed, the chain can understand)
Making a small transfer after creating the account should be sufficient to prove correctness

@race-of-sloths 🦥

@gtsonevv Thank you for your contribution! Your pull request is now a part of the Race of Sloths!
Are you going to win race-of-sloths this month? If so, you should speed up! Run, Sloth, run!

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called

@race-of-sloths score 5

🌟 Score recorded!
@andy-haynes, thank you for scoring this pull request in the Race of Sloths!

@gtsonevv would you mind rebasing this please? It should just be your work + release ahead of you in history.

@andy-haynes rebase onto master?

✅ PR is finalized!

You have received 50 Sloth points for this contribution


style(props-table): add spacing between prop descriptions

Closes #709
📝 Description
Update spacing between props in the PropsTable so that the name of each prop did not appear as if it was a part of the previous prop description.
⛳ Current behavior (updates)
Current table (light mode):

🚀 New behavior
Updated table: (light mode):

Thanks for the quick fix!


Use the swedish Setelin logo in main view

Description
Assets
TBA

@miladf please upload the asset here

@leosa83 https://www.dropbox.com/sh/ekoyuqi2tkggg6b/AAC9fEPlyPGUBq66ELxlZoy-a/Logos/SetelinSweden.svg?dl=0


Enhancement Request: Enable Legacy Mode for Python REPL in Versions Below 3.8

Issue Description
I've come across a challenge with the VSCode Python extension where the "python.experiments.optOutFrom": ["pythonREPLSmartSend"] setting in the {project_name}/.vscode/settings.json doesn't seem to be taking effect. Could it be that this particular setting is only recognized in user.settings?
Would it be possible to extend the functionality so that {project_name}/.vscode/settings.json can also utilize the "python.experiments.optOutFrom": ["pythonREPLSmartSend"] setting? I believe it would be highly beneficial for project-specific configurations.
For developers working with legacy Python projects, it would be highly beneficial if the Python extension could automatically switch to legacy interaction mode when operating with Python versions below 3.8.
This would ensure compatibility and a seamless development experience across various Python environments.
Benefits

Consistent development experience across multiple Python versions.
Assistance in maintaining and running legacy Python code within VSCode.
Reduced need for manual toggling between different REPL modes.

Thank you for taking the time to look into this issue. (´▽`ʃ♡ƪ)

Environment
版本: 1.84.2 (user setup)
認可: 1a5daa3a0231a0fbba4f14db7ec463cf99d7768e
日期: 2023-11-09T10:51:52.184Z
Electron: 25.9.2
ElectronBuildId: 24603566
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Python Extension Version: 2023.20
Python Version: 3.7 (Python REPL error), 3.10 (OK)

Hello @we684123! Thank you for filing this request.
I have actually added settings to turn off this new "Smart Send" behavior. It should be in insiders right now, and will be on stable soon. Meanwhile you can get a work around by explicitly highlighting/selecting lines of code you wish to run and then pressing shift+enter or pressing run selection/line.

Thank you @anthonykim1 and VScode-python team, i got it~
v2023.21.13261010 is work


Adding upgrade ubuntu libraries step to dockerfile

Purpose
fixes: https://github.com/wso2/apk/issues/1824

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (75ce98d) 29.24% compared to head (c8c7398) 22.09%.
Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1825      +/-   ##
==========================================
- Coverage   29.24%   22.09%   -7.16%
==========================================
  Files         198       55     -143
  Lines       20893    10084   -10809
  Branches     5163        0    -5163
==========================================
- Hits         6111     2228    -3883
+ Misses      14651     7725    -6926
  Partials      131      131

Flag
Coverage Δ

adapter
25.56% <ø> (?)

admin-domain-service
?

backoffice-domain-service
?

common-controller
1.11% <ø> (ø)

devportal-domain-service
?

idp-domain-service
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 209 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Old container id's are referenced instead of the new ones, (exited container entities) was this fixed?

@MichelFR This seems very much related to https://github.com/MichelFR/MqDockerUp/issues/8
So I am seeing old container entities (portainer and frigate) with their generated docker names in exited status, rather then showing as running. This used to run fine, but I guess stuff happened in between then and now.
I'll try and remove these and see if it gets populated/updated again, but this seems like a bug.
I tried removing them from the MQTT broker, stopping mqdockerup, but they keep coming back and I can't remove them from HA.

@MichelFR Any idea on this? This is really stopping me from using this. Happy to provide logs/screenshots where needed.
Many thanks,

This shows that all the autogenerated names by docker are exited, but mqdocker uses them for monitoring the containers. Whereas eclipse, or mosquito, if you click on them they use the current containers.
So - not sure what's going on.


[DMS-200] Run sample data bulk load

Descriptor bulk load works. Some of the other sample data load correctly, but some fail due to the problem described in DMS-235.

Is the E2E action still running for some reason?

Is the E2E action still running for some reason?

"Error: The operation was canceled."
Looks like a transient GitHub error. I'm re-running the job now.


Clarity around what applications targetted release applies to

Hi,
Could you add some clarity please around how targeted releases affects teams and OneDrive, is it just the web version of these products that first release updates? i.e. the client version of OneDrive, its updates are linked to GPO and update ring you set, however what about the MS Teams client though as that always updates on a more regular basis? Ive seen a few discussion threads asking the same so its not crystal clear. The important section on this page is where this could be added rather than just say M365 I think this could be expanded
thanks

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dd43f78a-7c74-5c56-c1f0-24deb342455f
Version Independent ID: f7ed9828-2580-1b50-d126-003d222abb5e
Content: Set up the Standard or Targeted release options - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/release-options-in-office-365.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

Hello @obhayes ,
This article is about update channels for services. You can read about feature updates for client apps here.
Thank you

@MaratMussabekov  Thank you very much for the contribution and updating the documentation with PR. @obhayes Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Removal request: vmagent

Issue description
The vmagent package provides a subset of the programs provided by the victoriametrics package.
This package should be removed (and aliased to victoriametrics).

Correct me if I'm wrong, but it seems as if vmagent could be useful without the whole victoriametrics package. Maybe someone wants that without also having vmbackup, vmrestore and so on?

Correct me if I'm wrong, but it seems as if vmagent could be useful without the whole victoriametrics package.

So could all of the other programs in the package. :D
If removal is not the best option, is there a way to provide all or some of the programs as individual packages without maintaining versions in multiple places?

cc @yorickvP @ivan @NULLx76

i played a bit around with the code. Would something like that fulfill the goal? https://github.com/Shawn8901/nixpkgs/commit/a2728a5acb7a6f262c133e53323c662cf776255c
vmagent as client tool could then be exposed like that https://github.com/Shawn8901/nixpkgs/commit/833c8f829741f28691559aa2d7adeb6933acc512
If yes i could create a pr for that

That looks good to me! :)

That looks good to me as well 👍

Cool, that looks like a reasonable solution! :)
I'll leave this issue open to be closed by merging the PR.

As in the issue was some response and the did not attract any deeper looks yet, does some from here feel comfortable in peeking at the linked o3r?

Sorry, I haven't been checking my GitHub notifications since. The merged solution looks good to me. Especially leaving the vmalert program in the victoriametrics package. :)

FYI: I' ve submitted a PR #350737 to add more options and  enhance the utility of the VictoriaMetrics's NixOS module. Any reviews would be appreciated.

Please restrain replies to PRs to topics that are directly relevant for the people involved in that specific PR such as issues the PR caused or follow-ups.
If you want a review, please ping the relevant people in your PR instead.

@Atemu Understood, comment deleted.


Service Worker fetch call returns response status 0

Describe the bug
I am experimenting with the service worker implementation from the SvelteKit documentation. I am finding that when the service worker fetches URLs on subdomains or other 3rd-party domains, the response status reported by the call to fetch is 0 (zero). This is not happening for resources that are hosted in the same domain as the service worker.
I have verified that the Chrome Network Tools shows that the requests are going through fine and returning full payloads as I would have expected with response status 200. This seems to be an issue specific to the response object coming from the call to fetch. This impacts my ability to properly detect when to cache resources from subdomains (eg. data files from AWS S3 buckets).
I am happy to work on creating a reproduction of the issue, but before I do that I wanted to make sure that this wasn't simply a documentation issue.
Any suggestions helpful.
Reproduction
Will provide one if requested.
Logs
No response
System Info
System:
    OS: Linux 5.19 KDE neon 5.27 5.27
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
    Memory: 5.73 GB / 15.34 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 16.18.0 - ~/apps/node-v16.18.0-linux-x64/bin/node
    npm: 9.3.0 - ~/apps/node-v16.18.0-linux-x64/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 110.0.5481.177
    Firefox: 110.0
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.0.0
    @sveltejs/adapter-netlify: ^2.0.3 => 2.0.5
    @sveltejs/kit: ^1.0.0 => 1.8.3
    svelte: ^3.44.0 => 3.55.1
    vite: ^4.0.1 => 4.1.4

Severity
serious, but I can work around it
Additional Information
No response

Seems like this is expected behavior relating to CORS. See this Stack Overflow response.

Upon further thought, there is the open question of why the request succeeded (as reported by the Chrome dev tools) but didn't supply the expected response status.
A CORS issue would have rejected the entire request, yet it went through fine.

This post helps explain the reasoning behind opaque responses.


Fixed policy  categories in migration 203 to 204

This PR fixes categories in migration seq 203 to 204
Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/retest

Please fill out the test section of the description about how you tested it.  Screenshots of effective policies would be helpful here too.
@charmik-redhat do you anticipate a migration for the node cve time PR?  I'm assuming that one isn't necessary as the column will default to false.  But I wanted to make sure since I would rather that migration go first if one is needed.

/retest


Fix invalid DEFVAL in SpatialEditorGizmo::add_unscaled_billboard

Likely added by mistake in #19487.
Should fix #19543.

@neikeq Since the Mono build is so sensitive to any binding mistake, we really need to improve the workflow in this regard, to either:

get compilation errors when bindings are invalid
or get clearer error messages at runtime

get compilation errors when bindings are invalid

Not possible, bindings happen at runtime. However, I've planned for a long time to add a test for the ClassDB bindings. So ideally, if you made changes to the bindings, you run this test before opening a PR and it would report any anomalies. It's in my TODOs list.

or get clearer error messages at runtime

I will have to check if this is possible.

However, I've planned for a long time to add a test for the ClassDB bindings. So ideally, if you made changes to the bindings, you run this test before opening a PR and it would report any anomalies. It's in my TODOs list.

That would be nice, we could have it run on the CI with the server/headless binary to catch those at build time.

That would definitely be welcome. I try to double check the bindings every time I make a PR, but it seems I am not that good at it.


[$250] Split - Group chat does not show bill creator name in LHN preview after refreshing the page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.60-5
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to FAB > Request money > Manual
Create a split bill with two users
Once landed in group chat, note that group chat in LHN shows the bill creator name in the preview
Refresh the page

Expected Result:
The group chat will continue to show the bill creator name in LHN preview after refreshing the page
Actual Result:
The group chat does not show the bill creator name in LHN preview after refreshing the page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/f9407c5a-6c4d-4aee-808a-ce4639cb42c2
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0126a3e7c5e2856078
Upwork Job ID:<PHONE_NUMBER>133911552
Last Price Increase: 2024-04-14

@kadiealexander  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
The LHN subtitle shows the sender name when creating the split bill, but then it disappears.
What is the root cause of that problem?
When we refresh/wait for a moment, we will get the correct response from the BE and the sender name disappears is already correct because we don't want to show the sender name if it's coming from ourself.
https://github.com/Expensify/App/blob/2995f2c79751d5096919eb98a26e0bd7b8533658/src/libs/OptionsListUtils.ts#L529-L533
The logic to show the sender name is, by taking the report lastActorAccountID and getting the user data from the ID.
https://github.com/Expensify/App/blob/2995f2c79751d5096919eb98a26e0bd7b8533658/src/libs/SidebarUtils.ts#L297
However, when we create a split bill, the lastActorAccountID is 0, so the user detail is empty.
https://github.com/Expensify/App/blob/2995f2c79751d5096919eb98a26e0bd7b8533658/src/libs/ReportUtils.ts#L3781
If it's empty, we will get it from the last report action person data which contains the user display name.
https://github.com/Expensify/App/blob/2995f2c79751d5096919eb98a26e0bd7b8533658/src/libs/SidebarUtils.ts#L299-L307
Because 0 is not the same as the current user ID, the current user display name is shown as the sender.
What changes do you think we should make in order to solve the problem?

Set the lastActorAccountID optimistically when creating a split bill (and revert it when fails). We did this when adding a message/task too.
We might need to do this for other money request too.
What alternative solutions did you explore? (Optional)
Don't show the display name when the lastActorAccountID is 0.

Reviewing proposals

I have some doubts about the "Expected results" being correct here. If we say that the frontend is right, and the split creator name shouldn't appear in the report's last message in the LHN from the perspective of the creator, this means that the last message of the report is different for each participant, and I think we may want to avoid that.
@marcaaron can I get your eyes here since I think you know about groups. 🙏

I agree 1000%.
With Group Chats, we did not really change much about how this works on the frontend side of things though - just replaced the Group DM with a Group Chat so I'm not sure where we went wrong or if this was just always awkward.
Could probably take this to #vip-split room and loop in whoever is working on Splits now. cc @arielgreen @dubielzyk-expensify and @youssef-lr? Can we see about rolling this in as a polish item for this project?

Seems like a minor bug. I agree that it'd be nice to fix it though, but fine with this as a polish item

@kadiealexander Based on the discussions above, can you help to update the Expected results section of this issue?

@kadiealexander Based on the discussions above, can you help to update the Expected results section of this issue?

@kadiealexander Friendly bump!

Or @bernhardoj would you like to update your proposal according to @aldo-expensify 's https://github.com/Expensify/App/issues/39647#issuecomment-2057456493 ?

Updated!

@bernhardoj Thanks for the update!
Based on the above discussions, yes, it makes more sense to Don't show the display name when the lastActorAccountID is 0.
So, @bernhardoj 's alternative solution looks good.
@aldo-expensify All yours.
🎀👀🎀 C+ reviewed

@aldo-expensify Should we assign @bernhardoj  to move this issue forward?

It is not clear to me from the conversation which option we want, so I asked here https://expensify.slack.com/archives/C05RECHFBEW/p1713833726038629 to confirm

Sorry @eh2077 @bernhardoj for the back and forward. The conclusion from the slack conversation is:
The bill creator name should appear in the last message, and the front end should take care of that. The backend will send the last message with the bill creator in it.
I think this code is related to that:
https://github.com/Expensify/App/blob/42d4ed7b72dc979df430a2a7eac4ba30a3eac790/src/libs/OptionsListUtils.ts#L711-L713
I think this was what @bernhardoj  original solution was trying to achieve, correct?

@aldo-expensify Thanks for driving clarity.
@bernhardoj Can you take some time to update proposal based on the expected result^ ?

I just noticed a mistake in the expected results I typed above. The backend WON'T send the bill creator as part of the last message:

My proposal has both behaviors, showing or not showing the sender name if it's the current user. If I understand correctly from @aldo-expensify comment, we always want to show the sender's name, so it should be the 2nd alternative.

@bernhardoj Thanks for the update. Yeah, the 2nd alternative makes sense to me.
@aldo-expensify All yours.
🎀👀🎀 C+ reviewed

PR is ready
cc: @eh2077

The fix has hit production https://github.com/Expensify/App/pull/41483#issuecomment-2101804136

Sorry guys, this one slipped through the cracks!
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] @eh2077] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@eh2077] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@eh2077] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@eh2077] Determine if we should create a regression test for this bug.
[ ] [@eh2077] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@kadiealexander] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Payouts due:

[ ] Contributor: $250 @bernhardoj
[ ] Contributor+: $250 @eh2077

Upwork job is here.

BugZero Checklis

[x] [@eh2077] The PR that introduced the bug has been identified. Link to the PR: I can't find the specific PR caused this issue. I think this case is not handled when we added the feature.
[x] [@eh2077] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
[x] [@eh2077] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] [@eh2077] Determine if we should create a regression test for this bug. This bug was reported by QA team and it's an edge case, so I think we don't need a regression test for it.


File system scheme 's3' not implemented

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.10.0
Custom Code
No
OS Platform and Distribution
Windows11
Mobile device
No response
Python version
3.8.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
An error occurred when I ran tensorboard using s3 as logdir

tensorboard --logdir=s3://mnist/log --port=6123

I find this method is the cause of the problem
tf.io.gfile.exists("s3://minst/log/xxxxxx")

### Standalone code to reproduce the issue

import os

os.environ.setdefault("S3_ENDPOINT", "https://xxxx")
os.environ.setdefault("AWS_ACCESS_KEY_ID", "xxx")
os.environ.setdefault("AWS_SECRET_ACCESS_KEY", "xxxx")
os.environ.setdefault("AWS_REGION", "us-east-1")

import tensorflow as tf
import tensorflow_io as tfio

gfile = tf.io.gfile.GFile("s3://minst/log/xxxxxx")
print(gfile)
# ↑ ↑ ↑ ---------it's ok

v = tf.io.read_file('s3://minst/log/xxxxx')
print(v)
# ↑ ↑ ↑ ---------it's ok

tf.io.gfile.exists("s3://minst/log/xxxxxx")
# ↑ ↑ ↑ ---------error

tensorboard.run_main()
# ↑ ↑ ↑ ---------error

os.system("tensorboard --logdir=s3://mnist/log --port=6123")
# ↑ ↑ ↑ ---------error

# will got same error

### Relevant log output

Traceback (most recent call last):
  File "D:\Users\xiaojunnuo\Desktop\tensorboard\venv\lib\site-packages\tensorflow\python\lib\io\file_io.py", line 288, in file_exists_v2
    _pywrap_file_io.FileExists(compat.path_to_bytes(path))
tensorflow.python.framework.errors_impl.UnimplementedError: File system scheme 's3' not implemented (file: 's3://minst/log/xxxxxxxxxxxxxxxx')

tf.io.gfile.exists("s3://minst/log/xxxxxx")
https://github.com/tensorflow/tensorflow/blob/359c3cdfc5fabac82b3c70b3b6de2b0a8c16874f/tensorflow/python/lib/io/file_io.py#L270-L272
os.system("tensorboard --logdir=s3://mnist/log --port=6123")
What TB version are you using? Check https://github.com/tensorflow/tensorboard/issues/767 as it is not too old.

tensorboard is the latest
import tensorflow_io as tfio
import tensorflow as tf
tf.io.gfile.exists("s3://minst/log/xxxxx")

This code has nothing to do with tensorboard , But it made an error

2022-11-10 09:32:39.992444: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll
Traceback (most recent call last):
  File "D:\Users\xiaojunnuo\Desktop\tensorboard\venv\lib\site-packages\tensorflow\python\lib\io\file_io.py", line 268, in file_exists_v2
    _pywrap_file_io.FileExists(compat.path_to_bytes(path))
tensorflow.python.framework.errors_impl.UnimplementedError: File system scheme 's3' not implemented (file: 's3://minst/log/xxxxxxxxx')
python-BaseException

Looks like it should be a problem with tensorboard, it shouldn't use tf.io.gfile.exists, I'll go to the tensorboar to report this problem.
thanks .

I am asking about the version cause it seems similar to https://github.com/tensorflow/tensorboard/issues/5480

My tensorboard version is 2.10.1
I see that the tensorboard source code is actually using tf.io.gfile.exists
https://github.com/tensorflow/tensorboard/blob/53e6d9ba3c2b1fb749725acb02e154d5387d9217/tensorboard/backend/event_processing/io_wrapper.py#L195

Do you have also an updated tensorflow-io version installed in the venv?

tensorflow_io==0.27.0 , The problem is still there

My s3 backend is minio, I don't know if it matters

I remember we had a thread on the forum:
https://discuss.tensorflow.org/t/writing-tensorboard-logs-to-s3-bucket-using-keras-callbacks/9960

Same issue over here.  It seems to me that S3 tensorboard support has never worked in recent history, courtesy of the issue @greper describes.

If it is a TB issue I think you could post your case at https://github.com/tensorflow/tensorboard/issues/4255

If 'tf.io.gfile.exists' does not support s3, then what interface can determine whether s3 files and directories exist

If 'tf.io.gfile.exists' does not support s3, then what interface can determine whether s3 files and directories exist

S3 is implemented in the IO repo so it is better that you ask there on in the forum

I have submitted an issue to io , thank you for your reply

@greper Thank you for the update!
Could you move this issue to closed status as it will be addressed in the IO repo. Thank you!


ABAPGit use survey

Do you use ABAPGit at your company? I'd like to add a list of real-world users, but I don't want it to start with a single company. If you are using ABAPGit in your ABAP development process and it's OK with you to have your company listed in the ABAPGit project, can you chime in here and once we have 5-10 users I'll submit a pull request?

Count Yelcho Systems Consulting in ( that’s me ) and these other customers of mine - Sword Holdings and Inchcape Australia. I also have 2 other customers that wouldn’t want to be named that use it too.

Of course: RheinEnergie AG, Cologne, Germany (currently only "leaching") + SE38 IT-Engineering, Neuss, Germany (contributing)

We have used it for Queensland health however only in offline mode right now. But have big plans to use it for ci together with pipelines and selenium testing

At emineo we use abapGit online repositories for product development. Other scenarios are offline repositories for custom development and migration projects.

We are using ABAPGit for our ABAP SDK for Azure. So far it has been used at a few customers and in quite a few codejams.

I have mostly been in the UI5 fiori world for several years and have preaching git there until it was built into webide. (unfortunately one client was still using SVN but source control at least )  Recently I have a little more say in Architecture and will be doing an ABAPGit poc for people in the abap stack with of course abapgit.

Yes we use ABAPGit at [www.pmconseil.com](Progress Management), mainly for internal tools that we reused on every projects.

Tricktresor uses abapGit ;)

Yes. some teams on SAP IBSO

We use abapGit at CQSE for development - not yet the full online experience with feature branches etc., but we are moving towards such a development model.

We use abapGit at objective partner for backup of coding and for version control but without feature branches etc.

I use ABAPGit most of the time for my opensource solutions. Talking about clients, adoption here in Brazil is still a bit slow, but I've already implemented it along with ABAPLint and ABAPOpenChecks

Should mention that all of my customers use abapGit - they just don't all know it 😉

Haufe Group using abapGit for installing ABAP open source and version control. We are also setting up abapGit to sync our internal frameworks over the different system landscapes.

My company (percept ltd) is using ABAPGit for ABAP developments for its customers.
A use case is mostly backup of coding and also deploying developed ABAP objects to SAP systems.

We started within the last weeks with abapGit. It’s a greate tool.

We use abapGit for decentralized ABAP development powered by Docker + OpenStack in SAP Labs Czech Republic

Still have the vision to use this as 12h bachup of our Z-Code.
Issue are cipherSuites somehow SAPserver <-> GitLab ... can't get it working.

@TimoJohn use abapGitServer? This is of cause not a off site/system backup, but you will get the visibility as to what is happening in the system.
Closing this issue, list can be edited via pull requests, https://docs.abapgit.org/other-where-used.html


Fix group merging and linking

WIP, do not merge. Other plugins will need changes for this to work.

I definitely think the GUI could use some improvements to change how linked groups are visualised, so I can make some improvements there as well.

After discussion with Orinnari, I think the best approach here is to remove merging as the implementation is simply too complex, and it can be wholly replaced by group linking. So, I will change this PR to just fix linking and remove merging.

Mulling this over, am I mistaken in thinking that the subgroup/linking issue could be [largely] resolved by banning the concept of subgroup-subgroups altogether? You could impose such a restriction fairly straightforwardly and efficiently both in-code and in-database by asserting that the parent's parent is null.
The only issue I can foresee, though I am fairly tired, is attempting to link a parent-group to another group, eg:
• Ashelor

• Ashelor-Public // wants to link to "Ashelor"
  ├──Ashelor-Gov
  └──Ashelor-Roads

I'd therefore propose that the former parent ("Ashelor-Public") become a sibling-subgroup, like so:
• Ashelor
  ├──Ashelor-Public
  ├──Ashelor-Gov
  └──Ashelor-Roads

You can then have APIs to change a group's parent, or to 'emancipate' a subgroup from its parent.


Dev

Still in draft form - but the tests pass. Created an option -c (compare) based on the description of the node version of bunyan. Currently implemented for a few of the easy base JSON fields. Constrained to the comparison operators, rather than "any javascript" the node version allows.  Intend on expanding to include the JSON in the "extra" field as well.
Not sure the effort to create types for the comparison is necessary - as that will not be possible for the free-form JSON in the extra field. Might need to simplify this down to just String comparisons.

Finally found the time to have a look - it looks quite promising!
A few comments:

The parsing of the comparison condition should happen once, at the beginning, not every time you need to check if a line matches the condition;
As you noted, we need something more dynamic to handle the extra keys, which is where the real fun begins;
Does the JS version of bunyan allow chaining of comparisons? I.e. x == 2 && y == 3?

Thanks for the feedback!
The JS version of bunyan allows an “arbitrary JS expression” for a filter. One of the advantages of a dynamic language! I started with a simple one-field comparison to get started, a chain of comparisons would be a logical next step – but I don’t know if defining a “language” for something more complicated than that would be worth the effort.
I have an idea on how to tackle the other items you mention – which will be a good Rust-programming exercise for me.
Mike.
From: Luca Palmieri @.>
Sent: Wednesday, November 24, 2021 5:49 AM
To: LukeMathWalker/bunyan @.>
Cc: Michael Hentges @.>; Author @.>
Subject: Re: [LukeMathWalker/bunyan] Dev (PR #5)
Finally found the time to have a look - it looks quite promising!
A few comments:

The parsing of the comparison condition should happen once, at the beginning, not every time you need to check if a line matches the condition;
As you noted, we need something more dynamic to handle the extra keys, which is where the real fun begins;
Does the JS version of bunyan allow chaining of comparisons? I.e. x == 2 && y == 3?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/LukeMathWalker/bunyan/pull/5#issuecomment-977802611 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGYPYBRSOIWYXEWQSU4WDUNTGKPANCNFSM5G2L6XPQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/ABGGYP63UXD4RAHFO45QGIDUNTGKPA5CNFSM5G2L6XP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHJEBK4Y.gif


Current stock shows no number for 0 stock

Build Number: 2.1.0
Description:
Current stock shows no number for 0 stock
Reproducible: Yes
Reproduction Steps: Look at current stock

Comments: Something passive aggressive about testing

Fixed in https://github.com/sussol/mobile/pull/580


wrapGAppsHook: Correct wrapProgram invocations

This change fixes several defects in the way wrapGAppsHook selected
the executable to wrap.
Previously, it would wrap any top-level files in the target /bin and
/libexec directories, including directories and non-executable
files.  In addition, it failed to wrap files in subdirectories.
Now, it uses find to iterate over these directory hierarchies,
selecting only executable files for wrapping.
Motivation for this change
Wrapping directories inside libexec broke emacs.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

This breaks at least mate-terminal and claws-mail:
nix-build -A claws-mail
find: '/nix/store/vxpdp4m7gpp0053myvq6pbf128igqp90-claws-mail-3.15.0/libexec': No such file or directory
Wrapping program /nix/store/vxpdp4m7gpp0053myvq6pbf128igqp90-claws-mail-3.15.0/bin/claws-mail
builder for ‘/nix/store/0a4gh5jkdba3wlb9hmbpbj5xmh3hr40g-claws-mail-3.15.0.drv’ failed with exit code 1
error: build of ‘/nix/store/0a4gh5jkdba3wlb9hmbpbj5xmh3hr40g-claws-mail-3.15.0.drv’ failed

nix-build -A mate.mate-terminal
 /nix/store/d56snr32kff2l1941jknkpf076h6mbwf-coreutils-8.27/bin/mkdir -p '/nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/share/mate-about'
 /nix/store/d56snr32kff2l1941jknkpf076h6mbwf-coreutils-8.27/bin/install -c -m 644 mate-version.xml '/nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/share/mate-about'
make[2]: Leaving directory '/tmp/nix-build-mate-desktop-1.18.0.drv-0/mate-desktop-1.18.0'
make[1]: Leaving directory '/tmp/nix-build-mate-desktop-1.18.0.drv-0/mate-desktop-1.18.0'
glibPreFixupPhase
hicolorPreFixupPhase
post-installation fixup
Removing /nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/share/gtk-doc
rmdir: failed to remove '/nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/share': Directory not empty
find: '/nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/libexec': No such file or directory
Wrapping program /nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/bin/mate-color-select
Wrapping program /nix/store/j6aw5qx8w2nb78lqcdjlr16m23xdv7a3-mate-desktop-1.18.0/bin/mate-about
builder for ‘/nix/store/y0641rgmdz6y1fmbg83pi6y0akikkvxk-mate-desktop-1.18.0.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/zdwiv0z0h3gc0qnmf1zw6a0n45raf5l9-mate-terminal-1.18.0.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/zdwiv0z0h3gc0qnmf1zw6a0n45raf5l9-mate-terminal-1.18.0.drv’ failed

Yeah, I see what happens. The patch passes those paths as arguments to find without verifying first that they actually exist. 😞 I'll revert in a minute ...

OK, I've worked up a fix, and tested it on claws-mail, mate.mate-desktop, and emacs (the original broken package that motivated the PR).  All three programs build and run, and all of the applications of wrapProgram look correct.
I think this PR needs to be reopened in order for the new commits to appear here, or I can open a new PR.

alarm-clock-applet seem to be broken by this PR as well. https://hydra.nixos.org/build/52065714
glibPreFixupPhase
post-installation fixup
find: '/nix/store/bmbzpmhlh458h5lwdl9x5bv7rlz2yw4f-alarm-clock-applet-0.3.4/libexec': No such file or directory
Wrapping program /nix/store/bmbzpmhlh458h5lwdl9x5bv7rlz2yw4f-alarm-clock-applet-0.3.4/bin/alarm-clock-applet

builder for ‘/nix/store/ylfyibdk8wbayn03234wa3yfjh736l6r-alarm-clock-applet-0.3.4.drv’ failed with exit code 1

Oh, nevermind. It is fixed now. https://hydra.nixos.org/build/52082092


extension development host: console log should also go in Extensions output and log

Create a new extension with the following code:

'use strict';
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
    let disposable = vscode.commands.registerCommand('extension.sayHello', () => {
        console.log('Hello')
    });

    context.subscriptions.push(disposable);
}

launch a extension development host
run the 'Hello World' command
open the Output view, Extensions tab: empty, Log (Extension Host): empty
The output is printed in debug console of the outer VSCode where the extension development host as launched, but I think it should also appear in the extension development host itself.

I think the Extension Host output channel just shows logs from our EH code and it probably doesn't make sense to mix extension output in there, but we could add a new output channel for that. I think it would end up including logs from other extensions too (same as the debug console)

I think this is out of scope, extensions can/will be able to explicitly log if they want a log. Ping @jrieken in case you disagree.


[ docs] add leaflet-sidebar-v2 to docs/plugins.md

leaflet-sidebar-v2 is a fork of sidebar-v2, but is more actively developed & has new features.

Nice!


[feature] Fix MultipleRelationshipsParser flatten_relationships()  [PLAT-1068]

Purpose
Allows to-one and to-many relationships to be sent in the same POST request.
Changes
Modifies JSONAPIMultipleRelationshipParser's flatten_relationships() to not overwrite to-one relationships when to-many relationships are present. Previously, a post request to create a node containing the following relationships
{
    'region': {
        'data': {
            'type': 'regions',
            'id': 'us-1'
        }
    },
    'affiliated_institutions': {
        'data': [
            {
                'type': 'institutions',
                'id': 'cos'
            }, {
                'type': 'institutions',
                'id': 'ljaf'
            }
        ]
    }
}

would be flattened to
[{'affiliated_institutions': 'cos'}, {'affiliated_institutions': 'ljaf'}] # sadly, no region

This PR modifies flatten_relationships so the above payload is flattened to
{'region': 'us-1', 'affiliated_institutions': ['cos', 'ljaf']}

QA Notes
Prior to this fix, creating a project from the dashboard (if your user had affiliated institutions) would fail to set the storage region correctly. Now, the storage region should be set properly when affiliated institutions are also present.
Side Effects
flatten_relationships() will now always return a dict. Previously, it could return a dict or a list, depending on what the relationships argument contained. This feels like an improvement.
Ticket
https://openscience.atlassian.net/browse/PLAT-1068

Do you know why the base JSONAPIParser doesn't handle multiple relationships? I find it surprising, and wonder if it would make sense to move the "handle multiple" logic into JSONAPIParser.flatten_relationships, and maybe remove JSONAPIMultipleRelationshipsParser altogether?

@aaxelb, I don't know why JSONAPIParser doesn't handle multiple relationships, but moving the multiple logic and deleting JSONAPIMultipleRelationshipsParser doesn't quite work.
Branch here: https://github.com/CenterForOpenScience/osf.io/compare/feature/storage-i18n...caseyrollins:feature/parsers
TCI here: https://travis-ci.org/caseyrollins/osf.io/jobs/422719343
It's probably worth investigating further (as well as investigating why we allow application/json content_types and why we have unused parsers, like JSONAPIOnetoOneRelationshipParser...), but I don't think it needs to be handled in this PR.


Add support for dialogs

This commit allows using the library over Dialogs and DialogFragments. In the DialogFragment documentation is specified the following

Note that dialogs are generally autonomous entities -- they are their own window, receiving their own input events, and often deciding on their own when to disappear (by receiving a back key event or the user clicking on a button).

If we want Spotlight to work over dialogs, we need to use dialog.window rather than activity.window

as of version 2.0.0, you can optionally set a viewGroup to show a spotlight on with setContainer. 👍

For anyone who might come across this, in v2.0 you can set the container to dialog.window?.decorView as ViewGroup in order to get the full dimensions of the screen and make proper calculations


Update js-packages.csv for deprecated packages

To unblock #6351 for broken links on deprecated packages

I fixed these with https://github.com/Azure/azure-sdk/pull/6343.


Request: terraform plan output the total price

Once coupon codes are supported, it'll be nice to confirm they're applied before the order is placed.

I don't think it's possible. dominos_order can have a property like price but it would only exist after creating the resource (that is, placing the order). I don't see how this request could be implemented but maybe @ndmckinley has an idea.

Ooh, I do actually!  :)  There's a special API called price_order you can hit to get the price out.  I could add a price_only boolean to the order object, which wouldn't really create the order, just get the price.  How's that?

Awesome! Would it be possible to add a hook to the end of a terraform plan instead? Makes more sense to me for it to be in plan since an apply should attempt to change state imo

I don't know, actually.  I think no matter what I do, the plan output will say price: "" -> "<COMPUTED>", right?  Can you point to any other provider that does what you're looking for?  Maybe I misunderstand.

Oh bummer. No, I was just wondering and I've never seen anything like it haha Thanks for looking into it!

Hey can someone take a look at #7?  I'm actually in Ireland right now and I can't hit dominos API from here, so I could use some help testing.

@ndmckinley Looking at the code from my phone, shouldn't this price only field also have something like IgnoreChanges on it? Just to make sure a different price reported via API won't trigger a new resource (therefore, a new order). The goal is, once a pizza is order, the order will never happen again.

Heh, technically yes, if this were a real, good provider.  It only gets updated during Create, though, so it won't cause changes.


YouTube Video Start at Time

I've tried a few different variations, but I can't get it to play the video at that specific time, always starts from the beginning.
Any ideas? Great plugin, by the way 👍

Looks like for embeds, only ?start=<seconds> works, eg.
Link


correct comparison with pokeshell

Hey @talwat I'm glad you did a rewrite of pokeget in rust! (A bit sad to see the big sprites to go tho...)
I saw the comparison with pokeshell and wanted to correct it. Because pokeshell uses timg, it is pixel perfect, any blurriness you would see would be when the pokemon you want to display don't fit in your current terminal size, so timg/chafa scales the image to fit. pokeget on the other hand doesn't even handle this case and floods the terminal output.
I thought this was quite unfair to pokeshell and corrected the comparison blurb with pokeshell.
btw if you do see something that is blurry (not due to resizing) please let me know/open an issue.

Nonetheless, I will remove the blurriness part. Although, if you could in exchange re-add pokeget to the comparison table on pokeshell's README that would be great.

The commit log goes back to early 2022.
See: https://github.com/acxz/pokeshell/commit/3263acb547018a797390fa02e25c107cd0a96a9c#diff-ae70f23979342abb8be85b943302b8c32473107b7da2e64f1b91f75f979140c7R65 where timg was first used (Feb 27, 2022)
There was a period of time, where I moved the display to chafa instead of timg, but timg is pixel perfect. With chafa I did some workarounds (--scale 8) for pixel display but those still cause a minute amount of blurriness. See: https://github.com/hpjansson/chafa/discussions/70#discussioncomment-11065855
While chafa is good enough imo, I recently added timg back in, in addition to chafa to let the user decide.

Nonetheless, I will remove the blurriness part. Although, if you could in exchange re-add pokeget to the comparison table on pokeshell's README that would be great.

Of course! That was my next step, running some timing runs with hyperfine. I'll post on this thread when I get in on the readme so you can take a look over it.

Also what does "heavier" mean in this context? number of dependencies? size of files?
we already mention that pokeshell is slower, but it is considerably smaller in file size. It does require more dependencies tho. I would prefer if you use specific language here instead of "heavier" which is pretty vague.

More dependencies is inherently heavier software. TIMG has a lot of features, and as a result is a heavier program. Not to mention ImageMagick, which is absolutely enormous because of all of it's various dependencies.

btw added pokeget-rs to the comparison table: https://github.com/acxz/pokeshell/commit/2aecdbee4e00e15c45ed3095c4ce6f7b807b8a86
pokeget is blazin' fast! Real good job here man!


Adding annotations to kubernetes service

We have EKS installed with latest terraform-aws-eks module version.
And we are using Datadog  for monitoring everything including this EKS cluster.
For enabling API server metrics in Datadog additional annotations to kubernetes service are needed.
The question is whether it's possible to manage these annotations with terraform-aws-eks module?
How terraform-aws-eks next apply or cluster upgrade will influence on such custom annotations?

Outside of the aws-auth configMap (which is slated to be removed soon), this module does not interact with any of the internal resources within the cluster. I would suggest taking a look at the Kubernetes Terraform provider to see if you can add the annotations with it


how to use plugin

hello, I'm newbie in Cordoba and want to add the plugin in my apps.
I have already installed the plugin but not as used or step forward. help!!

Two tutorials for you to follow:
http://macdonst.github.io/push-workshop-eu/
http://macdonst.github.io/push-workshop/

thanks


chore(FDS-141) Test danger action

Make danger file not check for description comments for changes on develop/master
New Component Checklist

[x] Includes unit tests for ALL required FDS use cases
[x] Does not mute any top level API props in React
[x] Includes an explanation for any changes to Jest snapshots (should be a PR tag automatically added if this happens)
[x] Component uses React.forwardRef() when a single DOM node is returned, AND it makes React-sense to access that DOM node

Warnings

:warning:
Please assign someone to merge this PR, and optionally include people who should review.

  Generated by :no_entry_sign: dangerJS against b693477d0f170d6cd52d2cacd202c25ce3c0faf9


mac_info.IsValidFolderPath() always returns False?

Has the behavior of mac_info.IsValidFolderPath() changed?
IsValidFolderPath() always returns False whether I give it the correct path that exists in the disk image or the wrong path.
As a result, the CFURLCACHE and CHROME plugins will not work properly.
I have modified CFURLCACHE plugin to test it like below:

The result:
MAIN.CFURLCACHE-INFO-/Users/macforensics/Library/Caches/ is NOT valid.
MAIN.CFURLCACHE-INFO-/Users/macforensics/Library/Caches/ZZZZ is NOT valid.
MAIN.CFURLCACHE-INFO-/Users/Guest/Library/Caches/ is NOT valid.
MAIN.CFURLCACHE-INFO-/Users/Guest/Library/Caches/ZZZZ is NOT valid.

I confirmed that cache folders are existed.

Fixed now.


feature/lit-2363-patch-fix-capacitydelegationauthsig-shouldnt-be-required-for

Description
The compatibility check functionality is designed to validate parameters for specific network and function combinations before proceeding with operations that depend on these parameters. This ensures that all required parameters are present and correct for the operation to succeed, preventing runtime errors and improving system reliability.
This PR should also resolve issues like these:
https://github.com/LIT-Protocol/getlit/pull/12#issuecomment-1919730708
https://github.com/LIT-Protocol/domain-wallet/pull/54
Functionality
To add a new compatibility check, use the compatibilityChecks.set method with a key combining the function name and network name, and provide a callback function that implements the validation logic.
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] unit tests
[x] e2e tests

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Closing this for now because if the wallet has capacity credit you don't really need to delegate the credit to the pkp, but we can re-open this when we need more compatibility checks for different networks

Closing this for now because if the wallet has capacity credit you don't really need to delegate the credit to the pkp, but we can re-open this when we need more compatibility checks for different networks.
We can just merge this instead: #337

Sounds good. we can keep this branch i like the idea of using unique keys for each network to check for an implementation. We could bring this into the v4 upgrade as a core utility method.


Link explaining how to install the update goes to the wrong page

If you click on the link "Install a software update for SharePoint Server 2013", it takes you to this page:
https://docs.microsoft.com/en-us/sharepoint/upgrade-and-update/install-a-software-update
which only applies to SharePoint Server 2016.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c2a90935-fb89-c75b-f0c3-683c45dbaad4
Version Independent ID: a01cb7f5-097f-629d-ded6-bf1362de329a
Content: Deploy software updates for SharePoint 2013
Content Source: SharePoint/SharePointServer/upgrade-and-update/deploy-software-updates-for-sharepoint-2013.md
Product: sharepoint-server-itpro
GitHub Login: @Techwriter40
Microsoft Alias: kirks

@AndreaBarr I'll take this one.

@repton-infinity looking at this, while the specific article applies to 2016 and does have some 16-specific paths in some of the commands, the procedure is identical to 2013. There are even 2013-only references, such as the UPSS :)
@Techwriter40 how should this best be handled? Most of the information here is accurate for both versions, just some very minor detail differences.

@repton-infinity  Thank you for submitting this feedback.
From our understanding the issue has been resolved. If you feel it hasn't been resolved please re-open this issue.

@officedocsbot close


Steps to deploy in a machine as locally?

??

hi,
if you are asking about how to run livekit-server locally, here are the docs for that: https://docs.livekit.io/guides/getting-started#starting-with-docker
for general questions, the livekit community on slack is active here

https://example.livekit.io/#/
i want to deploy this demo site on my local machine

the code for it is in the example folder: https://github.com/livekit/livekit-react/tree/master/example

the code for it is in the example folder: https://github.com/livekit/livekit-react/tree/master/example

facing error:-
`Failed to compile.
../dist/index.modern.js 358:88
Module parse failed: Unexpected token (358:88)
File was processed with these loaders:

<EMAIL_ADDRESS>../node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|
|   if (enableVideo) {

const enabled = !((camPub === null || camPub === void 0 ? void 0 : camPub.isMuted) ?? true);

|     videoButton = React__default.createElement(VideoSelectButton, {
|       isEnabled: enabled,
`

the code for it is in the example folder: https://github.com/livekit/livekit-react/tree/master/example

Thanks.....working fine now

the code for it is in the example folder: https://github.com/livekit/livekit-react/tree/master/example

hey, where can i change the name, uid which is set default like tony_stark, stark_tower. ??

https://docs.livekit.io/guides/access-tokens


To directly access the Data property from an API response

Hi,I want To directly access the "Data"property from an API response where the response is structured as:
{
  "status": 0,
  "code": 0,
  "errorMsg": "aaa",
  "Data": {}
}

Before :
        const result= await api.order.freightSeachFreightPageList()
        expect(result.data.Data.Rows.length > 0, "xxxxxxxxx").toBeTruthy();

Expected:
        const result= await api.order.freightSeachFreightPageList()
        expect(result.Rows.length > 0, "xxxxxxxxx").toBeTruthy();

thx.

--unwrap-response-data

thx


HADOOP-17965. Fix documentation build failure using JDK 7 on branch-2.10.

https://issues.apache.org/jira/browse/HADOOP-17965
mvn site by JDK 7 fails due to error related to spotbugs which does not support Java 7.
$ ./start-build-env.sh
ubuntu@2a660195e110:~/hadoop$ mvn clean install -DskipTests
ubuntu@2a660195e110:~/hadoop$ mvn site
...(snip)
[INFO] Apache Hadoop YARN Site ............................ SUCCESS [  2.138 s]
[INFO] Apache Hadoop YARN UI .............................. SUCCESS [  0.044 s]
[INFO] Apache Hadoop YARN Project ......................... FAILURE [  0.700 s]
...(snip)
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.5:site (default-site) on project hadoop-yarn-project: Execution default-site of goal org.apache.maven.plugins:maven-site-plugin:3.5:site failed: An API incompatibility was encountered while executing org.apache.maven.plugins:maven-site-plugin:3.5:site: java.lang.UnsupportedClassVersionError: org/codehaus/mojo/spotbugs/SpotbugsViolationCheckMojo : Unsupported major.minor version 52.0

Since we are using JDK 7 in the Dockerfile, dev-support/bin/create-release is affected too.
The cause is obsolete  entries of pom.xml for already resolved MSITE-443.
We can safely remove the  entry. mvn site generates same contents without the entry. spotbugsXml.xml are generated regardless of it (on mvn compile spotbugs:spotbugs using JDK 8).

./dev-support/bin/create-release invoking mvn site inside worked with the patch.
$ ./dev-support/bin/create-release --docker --dockercache
...
****************************************************************************
                          Maven Build and Install
****************************************************************************

$ /usr/bin/mvn -Dmaven.repo.local=/maven install -Pdist,src,yarn-ui -DskipTests -Dtar > /build/source/patchprocess/mvn_install.log 2>&1
$ /usr/bin/mvn -Dmaven.repo.local=/maven install site site:stage -DskipTests -Pdist,src,releasedocs,docs > /build/source/patchprocess/mvn_site.log 2>&1
...
Congratulations, you have successfully built the release
artifacts for Apache Hadoop 2.10.2-SNAPSHOT

The artifacts for this run are available at /home/ubuntu/srcs/hadoop/target/artifacts:
$ ls -1 /home/ubuntu/srcs/hadoop/target/artifacts
CHANGES.md
RELEASENOTES.md
hadoop-2.10.2-SNAPSHOT-rat.txt
hadoop-2.10.2-SNAPSHOT-site.tar.gz
hadoop-2.10.2-SNAPSHOT-src.tar.gz
hadoop-2.10.2-SNAPSHOT.tar.gz


Expose Extensions::from_http to user

This exposes Extensions::from_http to allow user to write more efficient tower's Service instead of using Interceptor as it doesn't allow to customize Response

As http::Extensions has already been an external type, it sounds good to exposing this API.


Support EPSG:4326 projection

Hi maplibre!
It would be awesome if maplibre-gl-native would support non-mertcator projections, mainly EPSG:4326, it seems like it was a long-existing request to the mapbox-gl-js but not until recently they decided to work on it, ref https://github.com/mapbox/mapbox-gl-js/issues/3184#issuecomment-831385021
relates also to https://github.com/maplibre/maplibre-gl-js/issues/166 https://github.com/maplibre/maplibre-gl-js/issues/168

Hello
I use MapLibre Native for Android and MapLibe GL GS for desktop projects. My geoserver is Geoserver. My current maps in web-mercator projection (3856). MapLibre works great!
Now my customer would like to use EPSG:4326 projection. I don't have enough experience to modify MapLibre itself, but I would like to fund non-staff developers to implement the issue. Is it possible? How can I do that?

@remennikovge Wet Dog Weather has taken on a big project like this. You could try reaching out to them if you are serious about it. https://www.wetdogweather.com/blog/maplibre-commitment
I expect it to be be fairly expensive. It is on the roadmap of MapLibre, however. And we do want to support this effort with our own financial reserves: https://maplibre.org/roadmap/non-mercator-projection/


Feature Request - combination Deny vNet peering cross subscription & Deny-VNET-Peering-To-Non-Approved-VNETs

szenario:

HubToSpoke Peering done by: Deny-VNET-Peering-To-Non-Approved-VNETs => fine
subscription intneral peering: Deny vNet peering cross subscription => fine

but if we've got an HubToSpoke Subscription with internal vnet peering. This won't work with these to Policy.
Suggestion:

Option A) Deny-VNET-Peering-To-Non-Approved-VNETs

add an option to allow inner subscription peering

Option B) Deny vNet peering cross subscription

add an excludelist of vnet ids which are ok to do cross subscriptions

or both ;-)

Tracking in ab#35840

https://github.com/Azure/Enterprise-Scale/pull/1731

This is complete and has been merged.


[Streaming] Merge streaming branch into master

Description
This PR will merge a branch off of 4.6-preview-streaming that has been updated with master.
Specific Changes

Adds streaming logic back into BotFrameworkAdapter
Combines TranscriptLoggerMiddleware bugfix in the 4.6 branch with code in master

The clonedActivity.id will be overridden with the corresponding ResourceResponse.id if it exists for sent activities.
Fix a 4.6 branch test that was failing with the code in master

Pull Request Test Coverage Report for Build 93688

141 of 170 (82.94%)  changed or added relevant lines in 13 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.8%) to 85.829%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botframework-streaming/src/webSocket/webSocketClient.ts
5
6
83.33%

libraries/botframework-streaming/src/webSocket/nodeWebSocket.ts
28
31
90.32%

libraries/botbuilder/src/botFrameworkAdapter.ts
84
109
77.06%

Files with Coverage Reduction
New Missed Lines
%

libraries/botframework-streaming/src/webSocket/webSocketServer.ts
7
60.61%

Totals

Change from base Build 93648:
0.8%

Covered Lines:
8073

Relevant Lines:
9095

💛  - Coveralls

Tested against simple scenarios with BridgeBot WebSockets and NamedPipes, then tested with DirectLine Speech, all passed.