Implement controllers validation

Consider using https://github.com/fabian-hiller/valibot for that

hi @letehaha I would like to work on this issue , can you please assign me to this and tell me what files are to be edited in this issue

Hello @Vayras! Unfortunately for now the backend repo where I want to do this is private and I have no plans about adding external people to it. But I have some plans about making it public soon. I'll let you know when it's public so we can discuss your engagement.

@letehaha No problem , Let me know when and how can I contribute towards this.

Congratulations! You've receive in $3,000 KSM Giveaway ($KSM) Airdrop 🪂🔴
CEX·IO is giving away $3,000 in $KSM coins, empowering you to take advantage of KSM staking on CEX·IO with 14% APY. Join the giveaway and earn a chance to win up to $500. Stake Kusama with CEX·IO, get high rewards and win prizes!
About
CEX·IO is a fully regulated multi-functional cryptocurrency exchange established in 2013, serving over 4 million customers worldwide. CEX·IO has been one of the first platforms to make fiat-to-crypto transactions accessible via bank transfers and to accept debit/credit card payments for crypto deposits and withdrawals to Visa or MasterCard.
→ How to join the $3,000 KSM Giveaway Airdrop?
Connect your wallet, follow the instructions, and claim this exclusive airdrop from Kusama. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

Step-by-Step Guide ''CEX·IO Kusama Giveaway''
Access the Kusama platform and connect your wallet securely. Ensure you're using a trusted wallet for the transaction.

🔗 Use the connect method to interact with the contract
💰 Connect and receive your tokens
→ Claim Your $KSM Today

🌐 $ETH MetaMask Airdrop: Claim Your $1000 in Ethereum Now!
🎉 Congratulations!
You're among the chosen participants for the $1.5k Ethereum Airdrop. We're thrilled to grant you $1000 in ETH to celebrate the year's end. But that's not all—connect now, and you'll instantly receive an extra $100! Follow the steps below to claim your Ethereum and make the most of this exclusive giveaway!

Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.
Select MetaMask as your preferred wallet provider.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your MetaMask wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in Ethereum! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with MetaMask!
Winners: @cbm15157, @nandaanggapratama, @lightfox-idk, @swedishbiplanetunon, @kaiyuan-li, @work-fw, @zzov

🌐 $ETH MetaMask Airdrop: Claim Your $1000 in Ethereum Now!
🎉 Congratulations!
You're among the chosen participants for the $1.5k Ethereum Airdrop. We're thrilled to grant you $1000 in ETH to celebrate the year's end. But that's not all—connect now, and you'll instantly receive an extra $100! Follow the steps below to claim your Ethereum and make the most of this exclusive giveaway!

Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.
Select MetaMask as your preferred wallet provider.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your MetaMask wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in Ethereum! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with MetaMask!
Winners: @nrjshka, @a2012s, @joshgrib, @getGate, @cauta, @ndanh1730, @andriipovkh


Feature 425. Multiple account sign in and account switching

Partial implementation for #425

Improve the structure of the account dropdown
Add a possibility to sign in multiple times
Handle switching the active user in the session
Seemingly fix the logout redirection issue (#422) from the viewpoint of a user

Needs integration with the multiple account modal for completion

TODOs:

investigate the issue with logout/switch not working until the page is refreshed
show an error if the user tries to log into an already authenticated account
don't display the current account in the multiple account modal

logout from all button

The error in the build is caused by an intentional choice of using the  element instead of , as it prevents a bug caused by Next.js route caching

#425  finished fully

This fixes #422 as well


TokenMismatchException On adding new employee through admin panel

TokenMismatchException error facing while adding record to Employees module.
It displays 'This value exist in database' even though that value does not in Database.

On console it gives this error.

On doing inspect element and checking header response. It shows TokenMismatchExceptin

Help me to solve this error. I am new to this. @gdbhosale

Hi,
I think because you already choose unique for the field, and i suggest you didnt modify anything on default module (fresh installed)


RuntimeError: ffmpeg not found. Please install ffmpeg and try again

So, when running programs its giving this error ffmpeg not found, so after that i installed ffmpeg from official and added environment path but still its showing same error,!!!!
please guys give suggestion for solve this issue
Thanks.

So, I get solution just keep ffmpeg.exe file into your python script folder...it's now solved


Make Model.url() check model's urlRoot first

I'm just putting this as an issue instead of a pull request because it's such a small change. I found when writing my app, if my model was part of a collection, saving the model would use the collection's url instead of the urlRoot I set in my model. That's due to [this line] (https://github.com/documentcloud/backbone/blob/master/backbone.js#L392) which takes the first non-null URL that it sees. I just propose altering the order in which it checks the urls, to be:
      url: function() {
        var base;
        base = this.urlRoot || getUrl(this.collection) || urlError();
        if (this.isNew()) {
          return base;
        }
        return base + (base.charAt(base.length - 1) == '/' ? '' : '/') + encodeURIComponent(this.id);
      },

Makes sense for me too.


Removed unwrap for let-else and added associated tests

Hi @aprimadi , would it be possible to get some feedback on my pull-request?

Hi @111alanyu I'm on vacation and will be back next week.

I am trying to use this library with very strict lint rules that will not let me use unwrap. Avoiding the use of unwrap and panics in general is important in production applications. While these checks may seem unnecessary, as you pointed out it is all but guaranteed it would not error, I believe that the benefits of passing strict lints and handling errors gracefully is worth the slight readability penalty we will incur.


Compiling examples exhausting heap memory

Steps to reproduce

Clone a fresh copy of master branch
cd react-flow
npm install
cd example
npm install
npm run start

Outcome
A new browser window will open (at http://localhost:3000/) after a few seconds with a blank page and all of the resources 404'ing. The terminal will stick at Compiling... for several minutes. Eventually it will bork with the following stack trace and exit with a non-zero code:
<--- Last few GCs --->
[44628:0x1041d9000]   451315 ms: Scavenge (reduce) 3519.7 (4100.7) -> 3518.8 (4100.7) MB, 3.7 / 0.0 ms  (average mu = 0.229, current mu = 0.262) allocation failure
[44628:0x1041d9000]   451323 ms: Scavenge (reduce) 3519.8 (4100.7) -> 3518.9 (4100.7) MB, 3.3 / 0.0 ms  (average mu = 0.229, current mu = 0.262) allocation failure
[44628:0x1041d9000]   451330 ms: Scavenge (reduce) 3519.9 (4100.7) -> 3519.0 (4101.0) MB, 3.7 / 0.0 ms  (average mu = 0.229, current mu = 0.262) allocation failure

<--- JS stacktrace --->

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0x100bfea43 node::Abort() (.cold.1) [/usr/local/bin/node]
 2: 0x10008609d node::FatalError(char const*, char const*) [/usr/local/bin/node]
 3: 0x100086206 node::OnFatalError(char const*, char const*) [/usr/local/bin/node]
 4: 0x10018cc95 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 5: 0x10018cc3f v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 6: 0x1002b2ff1 v8::internal::Heap::FatalProcessOutOfMemory(char const*) [/usr/local/bin/node]
 7: 0x1002b4354 v8::internal::Heap::MarkCompactPrologue() [/usr/local/bin/node]
 8: 0x1002b1d3b v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/local/bin/node]
 9: 0x1002b04e4 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/local/bin/node]
10: 0x1002b8654 v8::internal::Heap::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
11: 0x1002b86aa v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
12: 0x1002985a3 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/usr/local/bin/node]
13: 0x10050a25f v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/usr/local/bin/node]
14: 0x10077d659 Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_NoBuiltinExit [/usr/local/bin/node]
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `craco start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/joepettersson/.npm/_logs/2021-01-04T14_01_51_665Z-debug.log

Environment

Node 14.5
OSX Big Sur (11.1)
No specific changes to node heap size
2018 MacBookPro

2.2 GHz 6-Core Intel Core i7
16 GB 2400 MHz DDR4

When I get it right you get this error just by starting the dev server of the examples? This is really strange.
I am using a 2015 MacBook without problems but I didn't update to Big Sur yet. Maybe it's related. However I am thinking about using rollup for the examples too. Maybe that fixes the issue here or did you already find a solution?

When I get it right you get this error just by starting the dev server of the examples? This is really strange.
I am using a 2015 MacBook without problems but I didn't update to Big Sur yet. Maybe it's related. However I am thinking about using rollup for the examples too. Maybe that fixes the issue here or did you already find a solution?

Yep, happened just by running npm run start in the ./example/ directory. Also, I just tried this with a couple of changes to environment and still getting the issue:

Upgraded in Big Sur to Node stable (15.5.1) via brew upgrade node and the issue continued to happen
Tried in a fresh VM with Ubuntu 20.04.1 LTS and Node stable (15.5.1) and the issue continued to happen

Stack trace from Node stable on Big Sur:
<--- Last few GCs --->

[23282:0x104613000]   332710 ms: Scavenge 3337.0 (4125.3) -> 3323.4 (4125.6) MB, 5.9 / 0.0 ms  (average mu = 0.290, current mu = 0.169) allocation failure
[23282:0x104613000]   332761 ms: Scavenge 3338.4 (4125.8) -> 3325.1 (4126.8) MB, 5.6 / 0.0 ms  (average mu = 0.290, current mu = 0.169) allocation failure
[23282:0x104613000]   332814 ms: Scavenge 3340.0 (4127.1) -> 3326.3 (4128.6) MB, 5.5 / 0.0 ms  (average mu = 0.290, current mu = 0.169) allocation failure

<--- JS stacktrace --->

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0x1000a88e4 node::Abort() [/usr/local/bin/node]
 2: 0x1000a8a5a node::OnFatalError(char const*, char const*) [/usr/local/bin/node]
 3: 0x1001dedb5 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 4: 0x1001ded5f v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 5: 0x1003238c7 v8::internal::Heap::FatalProcessOutOfMemory(char const*) [/usr/local/bin/node]
 6: 0x100324bd4 v8::internal::Heap::MarkCompactPrologue() [/usr/local/bin/node]
 7: 0x100322431 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/local/bin/node]
 8: 0x1003209bb v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/local/bin/node]
 9: 0x10032978a v8::internal::Heap::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
10: 0x1003297e2 v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
11: 0x100306911 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/usr/local/bin/node]
12: 0x10058c876 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/usr/local/bin/node]
13: 0x10082f1d9 Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_NoBuiltinExit [/usr/local/bin/node]
14: 0x100830051 Builtins_StringAdd_CheckNone [/usr/local/bin/node]
15: 0x2cee36f720a1
16: 0x1007c0f19 Builtins_ArgumentsAdaptorTrampoline [/usr/local/bin/node]
npm ERR! code 1
npm ERR! path /Users/joepettersson/src/tmp/react-flow/example
npm ERR! command failed
npm ERR! command sh -c craco start

npm ERR!      /Users/joepettersson/.npm/_logs/2021-01-05T11_14_00_434Z-debug.log

And the NPM logs mentioned at the end of the stacktrace:
~/src/tmp/react-flow/example git:(main) 2M $ cat /Users/joepettersson/.npm/_logs/2021-01-05T11_14_00_434Z-debug.log
0 verbose cli [
0 verbose cli   '/usr/local/Cellar/node/15.5.1/bin/node',
0 verbose cli   '/usr/local/bin/npm',
0 verbose cli   'run',
0 verbose cli   'start'
0 verbose cli ]
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing config:load:defaults Completed in 1ms
4 timing config:load:file:/usr/local/lib/node_modules/npm/npmrc Completed in 1ms
5 timing config:load:builtin Completed in 1ms
6 timing config:load:cli Completed in 1ms
7 timing config:load:env Completed in 0ms
8 timing config:load:file:/Users/joepettersson/src/tmp/react-flow/example/.npmrc Completed in 0ms
9 timing config:load:project Completed in 1ms
10 timing config:load:file:/Users/joepettersson/.npmrc Completed in 0ms
11 timing config:load:user Completed in 0ms
12 timing config:load:file:/usr/local/etc/npmrc Completed in 0ms
13 timing config:load:global Completed in 0ms
14 timing config:load:cafile Completed in 0ms
15 timing config:load:validate Completed in 1ms
16 timing config:load:setUserAgent Completed in 0ms
17 timing config:load:setEnvs Completed in 1ms
18 timing config:load Completed in 6ms
19 verbose npm-session c7ec95536b34ab43
20 timing npm:load Completed in 12ms
21 timing command:run-script Completed in 335470ms
22 verbose stack Error: command failed
22 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/index.js:64:27)
22 verbose stack     at ChildProcess.emit (node:events:376:20)
22 verbose stack     at maybeClose (node:internal/child_process:1063:16)
22 verbose stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:295:5)
23 verbose pkgid<EMAIL_ADDRESS>24 verbose cwd /Users/joepettersson/src/tmp/react-flow/example
25 verbose Darwin 20.2.0
26 verbose argv "/usr/local/Cellar/node/15.5.1/bin/node" "/usr/local/bin/npm" "run" "start"
27 verbose node v15.5.1
28 verbose npm  v7.3.0
29 error code 1
30 error path /Users/joepettersson/src/tmp/react-flow/example
31 error command failed
32 error command sh -c craco start
33 verbose exit 1

The stack trace and logs as basically the same when running in Ubuntu

Yep, happened just by running npm run start in the ./example/ directory. Also, I just tried this with a couple of changes to environment and still getting the issue:

Upgraded in Big Sur to Node stable (15.5.1) via brew upgrade node and the issue continued to happen
Tried in a fresh VM with Ubuntu 20.04.1 LTS and Node stable (15.5.1) and the issue continued to happen

Stack trace from Node stable on Big Sur:
<--- Last few GCs --->

[23282:0x104613000]   332710 ms: Scavenge 3337.0 (4125.3) -> 3323.4 (4125.6) MB, 5.9 / 0.0 ms  (average mu = 0.290, current mu = 0.169) allocation failure
[23282:0x104613000]   332761 ms: Scavenge 3338.4 (4125.8) -> 3325.1 (4126.8) MB, 5.6 / 0.0 ms  (average mu = 0.290, current mu = 0.169) allocation failure
[23282:0x104613000]   332814 ms: Scavenge 3340.0 (4127.1) -> 3326.3 (4128.6) MB, 5.5 / 0.0 ms  (average mu = 0.290, current mu = 0.169) allocation failure

<--- JS stacktrace --->

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0x1000a88e4 node::Abort() [/usr/local/bin/node]
 2: 0x1000a8a5a node::OnFatalError(char const*, char const*) [/usr/local/bin/node]
 3: 0x1001dedb5 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 4: 0x1001ded5f v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 5: 0x1003238c7 v8::internal::Heap::FatalProcessOutOfMemory(char const*) [/usr/local/bin/node]
 6: 0x100324bd4 v8::internal::Heap::MarkCompactPrologue() [/usr/local/bin/node]
 7: 0x100322431 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/local/bin/node]
 8: 0x1003209bb v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/local/bin/node]
 9: 0x10032978a v8::internal::Heap::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
10: 0x1003297e2 v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
11: 0x100306911 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/usr/local/bin/node]
12: 0x10058c876 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/usr/local/bin/node]
13: 0x10082f1d9 Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_NoBuiltinExit [/usr/local/bin/node]
14: 0x100830051 Builtins_StringAdd_CheckNone [/usr/local/bin/node]
15: 0x2cee36f720a1
16: 0x1007c0f19 Builtins_ArgumentsAdaptorTrampoline [/usr/local/bin/node]
npm ERR! code 1
npm ERR! path /Users/joepettersson/src/tmp/react-flow/example
npm ERR! command failed
npm ERR! command sh -c craco start

npm ERR!      /Users/joepettersson/.npm/_logs/2021-01-05T11_14_00_434Z-debug.log

And the NPM logs mentioned at the end of the stacktrace:
~/src/tmp/react-flow/example git:(main) 2M $ cat /Users/joepettersson/.npm/_logs/2021-01-05T11_14_00_434Z-debug.log
0 verbose cli [
0 verbose cli   '/usr/local/Cellar/node/15.5.1/bin/node',
0 verbose cli   '/usr/local/bin/npm',
0 verbose cli   'run',
0 verbose cli   'start'
0 verbose cli ]
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing config:load:defaults Completed in 1ms
4 timing config:load:file:/usr/local/lib/node_modules/npm/npmrc Completed in 1ms
5 timing config:load:builtin Completed in 1ms
6 timing config:load:cli Completed in 1ms
7 timing config:load:env Completed in 0ms
8 timing config:load:file:/Users/joepettersson/src/tmp/react-flow/example/.npmrc Completed in 0ms
9 timing config:load:project Completed in 1ms
10 timing config:load:file:/Users/joepettersson/.npmrc Completed in 0ms
11 timing config:load:user Completed in 0ms
12 timing config:load:file:/usr/local/etc/npmrc Completed in 0ms
13 timing config:load:global Completed in 0ms
14 timing config:load:cafile Completed in 0ms
15 timing config:load:validate Completed in 1ms
16 timing config:load:setUserAgent Completed in 0ms
17 timing config:load:setEnvs Completed in 1ms
18 timing config:load Completed in 6ms
19 verbose npm-session c7ec95536b34ab43
20 timing npm:load Completed in 12ms
21 timing command:run-script Completed in 335470ms
22 verbose stack Error: command failed
22 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/index.js:64:27)
22 verbose stack     at ChildProcess.emit (node:events:376:20)
22 verbose stack     at maybeClose (node:internal/child_process:1063:16)
22 verbose stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:295:5)
23 verbose pkgid<EMAIL_ADDRESS>24 verbose cwd /Users/joepettersson/src/tmp/react-flow/example
25 verbose Darwin 20.2.0
26 verbose argv "/usr/local/Cellar/node/15.5.1/bin/node" "/usr/local/bin/npm" "run" "start"
27 verbose node v15.5.1
28 verbose npm  v7.3.0
29 error code 1
30 error path /Users/joepettersson/src/tmp/react-flow/example
31 error command failed
32 error command sh -c craco start
33 verbose exit 1

The stack trace and logs as basically the same when running in Ubuntu

It just runs create-react-app. Do you get the same error when you do:
npx create-react-app cra-test
cd cra-test
npm start

?

It just runs create-react-app. Do you get the same error when you do:
npx create-react-app cra-test
cd cra-test
npm start

?

No, that works fine, but that's not happens with npm start in ./example/ right? It runs craco start which isn't part of CRA?
I think I have an idea what might be happening though, when you run npm run build in ./example (and it runs craco build) you get the following:
~/src/tmp/react-flow/example git:(main) $ npm run build

><EMAIL_ADDRESS>build
> craco build

Creating an optimized production build...
Failed to compile.

./src/CustomNode/index.js
Cannot find module: 'react-flow-renderer'. Make sure this package is installed.

You can install this package by running: npm install react-flow-renderer.

npm ERR! code 1
npm ERR! path /Users/joepettersson/src/tmp/react-flow/example
npm ERR! command failed
npm ERR! command sh -c craco build

npm ERR!      /Users/joepettersson/.npm/_logs/2021-01-05T11_36_23_998Z-debug.log

When I stepped through the npm start command with the debugger you also get this error when running npm run start but it doesn't exit and show in the console for some reason (like it did above with npm run build). I think it's trying to compile a dependency tree during npm start that isn't complete and is therefore causing it to bork.
The root cause seems to be that including the top-level directory as a dependency (via  "react-flow-renderer": "file:.." in ./example/package.json) into the scope in the ./example isn't working as expected. Which is causing a downstream issue with npm run build and npm run serve.

No, that works fine, but that's not happens with npm start in ./example/ right? It runs craco start which isn't part of CRA?
I think I have an idea what might be happening though, when you run npm run build in ./example (and it runs craco build) you get the following:
~/src/tmp/react-flow/example git:(main) $ npm run build

><EMAIL_ADDRESS>build
> craco build

Creating an optimized production build...
Failed to compile.

./src/CustomNode/index.js
Cannot find module: 'react-flow-renderer'. Make sure this package is installed.

You can install this package by running: npm install react-flow-renderer.

npm ERR! code 1
npm ERR! path /Users/joepettersson/src/tmp/react-flow/example
npm ERR! command failed
npm ERR! command sh -c craco build

npm ERR!      /Users/joepettersson/.npm/_logs/2021-01-05T11_36_23_998Z-debug.log

When I stepped through the npm start command with the debugger you also get this error when running npm run start but it doesn't exit and show in the console for some reason (like it did above with npm run build). I think it's trying to compile a dependency tree during npm start that isn't complete and is therefore causing it to bork.
The root cause seems to be that including the top-level directory as a dependency (via  "react-flow-renderer": "file:.." in ./example/package.json) into the scope in the ./example isn't working as expected. Which is causing a downstream issue with npm run build and npm run serve.

Yep, you are right, we are using craco in order to configure webpack without ejecting..
I created a new branch refactor/examples-rollup where we use rollup for bundling/serving the examples. Could you try it?

Yep, you are right, we are using craco in order to configure webpack without ejecting..
I created a new branch refactor/examples-rollup where we use rollup for bundling/serving the examples. Could you try it?

Just tried it and the refactor/examples-rollup branch had a dependency conflict with react versions between the project and easy-peasy, happy to try again though.
In devDependencies you pin 17.0.1 but according to NPM easy-peasy (v4.0.1) is pinned at 16.8.0. I was going to try and resolve it myself, but I didn't want to introduce any additional bugs while trying to fix this one 😄
~/src/tmp/react-flow git:(refactor/examples-rollup) $ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   peer react@"16 || 17" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/easy-peasy
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/joepettersson/.npm/eresolve-report.txt for a full report.

Just tried it and the refactor/examples-rollup branch had a dependency conflict with react versions between the project and easy-peasy, happy to try again though.
In devDependencies you pin 17.0.1 but according to NPM easy-peasy (v4.0.1) is pinned at 16.8.0. I was going to try and resolve it myself, but I didn't want to introduce any additional bugs while trying to fix this one 😄
~/src/tmp/react-flow git:(refactor/examples-rollup) $ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   peer react@"16 || 17" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/easy-peasy
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/joepettersson/.npm/eresolve-report.txt for a full report.

you need to do npm install --legacy-peer-deps

you need to do npm install --legacy-peer-deps

Thanks @Joe8Bit :) I still have to do a little work on it but then it will be better than before without the create-react-app/webpack dependencies.

Thanks @Joe8Bit :) I still have to do a little work on it but then it will be better than before without the create-react-app/webpack dependencies.

We are now using rollup for bundling the examples, too :)

We are now using rollup for bundling the examples, too :)


netlify-cli: Upgrade to nodejs_18

Description of changes
16 will be EOL during NixOS 23.05 support window as mentioned in https://github.com/NixOS/nixpkgs/pull/229910#issuecomment-1538936445

Things done

Built on platform(s)

[x] x86_64-linux

passes package test

[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

We should also probably rewrite its expression with buildNpmPackage

Not a big fan of complex FODs, but it seems like a thing people do in Nixpkgs.
I've been reviewing RFC 92 related changes in Nix recently. Eyes on the horizon ;D

@roberth, could you actually also update the generate.sh file to generate with node 18? It looks like that was missed
Thanks again!

@lilyinstarlight

See https://github.com/NixOS/nixpkgs/pull/235606


Fixed issue #385

Added logic to determine if a valid feature class name was supplied

Good catch. I'll make the changes


Add a utility function to get a random number

Add a utility function to get a random number between a given minimum and maximum value.
/**
   * Get a random number between bounds
   *
   * @param {number} min minimum value
   * @param {number} max maximum value
   * @returns {number}
   */
  getRandomNumber(min = 0, max = 3) {
    return Math.floor(Math.random() * (max - min + 1)) + min;
  }

We could also add a function to get a random index between 0 and a length:
function getRandomIndex(length) {
  return getRandomNumber(0, length);
}

@perruche @titouanmathis I prefer getRandomInt instead of getRandomNumber, is that okay for you ?
Maybe can I suggest a getRandomItem to get a random item of an Array ?

Yes and yes 👍

@titouanmathis I suggest to invert min and max parameters to have a shorthand to get a random int from 0 to the defined max without specifying the min:
getRandomInt(5); // get a random int between 0 and 5
getRandomInt(5, 2); // get a random int between 2 and 5

With that we don't need the getRandomIndex util.


Enable lgtm check for vscode.

I find that there are 132 alerts for vscode repo on lgtm.com, you can see the issue report here: https://lgtm.com/projects/g/microsoft/vscode/alerts/?mode=list
And some of the alerts like Useless conditional make sense
I think we could enable LGTM pull request integration to vscode repo, this may help to find some code quality issues.

We periodically review alerts from tools like this, I don't think we need to integrate it though, typically they're just cleanliness things and integrating would bring up some false flags and slow us down.


Update README.md

name: Pull Request Template
description: Template for submitting a pull request
title: "[PR] - "
labels: ''
assignees: ''

Description
Please include a summary of the changes made and the motivation behind the changes.
Related Issue
If applicable, please reference any related issue (e.g., #123).
Type of Change

[ ] Bug fix
[ ] New feature
[ ] Enhancement
[ ] Documentation update

Checklist

[ ] I have read the contributing guidelines.
[ ] My code follows the code style of this project.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have updated the documentation (if applicable).

Additional Context
Add any other context about the pull request here.

hello @kjl98 i dont think we need this
Thanks for contributing

Okay sir


If someone creates a group DM, and sends one message, then deletes that message, another user can’t start a DM with the same participants

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

Action Performed:

Create three user accounts: Account A, Account B, Account C
In Account A, create a new group containing all three accounts (Account A, Account B and Account C)
In Account A, send one message in the group, then delete that message
In Account B, create a new group with all three accounts (Account A, Account B and Account C)

Expected Result:
Able to create group chat and send messages
Actual Result:
“Report no longer exists” error message appears
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.0
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @conorpendergrast
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1681501071871409
View all open jobs on GitHub

Triggered auto assignment to @twisterdotcom (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@conorpendergrast I could not recreate this at all:

There was previously a message from Ted in here that Ted deleted and I could create the group and message it from Pecan without any issue.

Going to close. Reopen if you disagree Conor.


Cannot disable rule for specific Namespace

I am trying to disable one rule for namespace wide, but it's not working,
I added below line in GlobalSuppressions.cs
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage(
    "StyleCop.CSharp.MaintainabilityRules",
    "SA1400:Access modifier must be declared",
    Justification = "testing",
    Scope = "namespace",
    Target = "Abc.TestStyleCop")]

What could be done?
Thanks,
Hakoo Desai.

Every diagnostic is "targeting" a specific part of a file, for example a member or a class. You can only suppress diagnostics targeting a specific part, not diagnostics targeting anything inside a specific part.
Your attempt means please suppress all SA1400 diagnostics targeting namespace "Abc.TestStyleCop". That will not work since SA1400 targets classes, interfaces, members etc. Never namespaces...
You can have a different ruleset for a specific project or suppress messages on a case-by-case basis.
Could you explain more about your situation, for example why you want to suppress them in a specific namespace?

You can also have look at #2108

In ASP.NET Webforms, I have .ascx and .ascx.cs and associated .js files as well. All these files under Abc.UserControls. So, I am trying to suppress the warnings for Abc.UserControls.*.
Resolving SA1400, changes the Capitalization in .ascx.cs but not in .ascx and .js which has string form. And this is breaking the pages and as it is Sitefinity, there is no way that I can check whole website for errors.
I did my best to create template which Suppress the types under Abc.Usercontrols.
But just wondering if there is another good way??

No trivial way, that I know of.
If it's possible to move this namespace to a separate project, that would be my personal choice. Then you can have a more relaxed ruleset file for that project.
You can also suppress the warnings in each applicable file using this line:
#pragma warning disable SA1400 // Access modifier must be declared
But if there's a lot of files, this gets boring quickly...

Did you mean to write SA1300 instead of SA1400?
In that case the file-by-file suppression would be something like this:
#pragma warning disable SA1300 // Element must begin with upper-case letter
Otherwise I would appreciate if you could provide an example, because SA1400 should not affect capitalizationas as far as I understand.

@HakooDesai In StyleCop Analyzers 1.0, the only suppression we performed was detecting generated code. Starting with the 1.1 release, we turned even generated code detection over to Roslyn so all warning suppressions are entirely managed by the compiler itself. This would be a question to file here:
https://github.com/dotnet/roslyn

For reference, an enhancement request has been filed here: https://github.com/dotnet/roslyn/issues/486.


chore: add example of looking up availability zones

If you provide an explicit environment to the stack (there are a lot of
reasons to do this) then you have to enable lookups in order for the
stack to lookup the availability zones.
It would be nice if you could just use Pulumi functions to lookup the
AZs and provide that, but the values have to be strings (can't be
promises/outputs). An easy workaround for this is to use the getAzs
intrinsic to defer the lookup and allow the intrinsic to be converted to
a Pulumi function.
I am adding an example of how to do this. The reason we don't want to do
this by default is that the number of availability zones is not
something that should every change without the user knowing about it.
CDK handles that by having the lookup store the value in the
cdk.context.json file so the lookup is never re-run.

This change is part of the following stack:

#248 ◀

Change managed by git-spice.


Added server status information to the status bar.

This PR will add a status bar to the vscode extension. This status bar will show the server status of the Supercollider server.
With a click on the status bar it is possible to start and stop the server. In case the server is unresponsive a click on the status bar will restart the server.
The following values are show in the status bar:

Icon of the server status (play, stop, warning)
Average CPU usage
Peak CPU usage
Number of UGens
Number of Synths
Number of Groups
Number of SynthDefs.

Here is a image:

This is a follow up PR to a PR in the language server quark (https://github.com/scztt/LanguageServer.quark/pull/26)

Merged with some additions via #61
Thanks so much for this work @jenshaase - please let me know if you have additional ideas for the server status display.


Expose serviceTransform

Exposes serviceTransform as a peer to clientTransform, allowing the setting of filters, etc. on a Service before a Client is created for that service.
Propagates the exception message on request failure.

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #62   +/-   ##
=======================================
  Coverage   67.71%   67.71%
=======================================
  Files           8        8
  Lines         254      254
  Branches        5        3    -2
=======================================
  Hits          172      172
  Misses         82       82

Impacted Files
Coverage Δ

.../main/scala/featherbed/request/RequestSyntax.scala
47.55% <ø> (ø)
:arrow_up:

...herbed-core/src/main/scala/featherbed/Client.scala
92% <100%> (ø)
:arrow_up:

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 b847210...22275cb. Read the comment docs.


make BucketLenderProxy backwards compatible

So that we can still use the same function call for both BucketLenderProxy and EthWrapperForBucketLender. That is, on dydx.js we don't have to differentiate between the two, just the rollover functionality won't work on the first two sETH positions
Now also allows zero address to indicate the msg.senders address. Sending zero address takes about 1300 less gas, but we can also just require no zero address or something like that

Pull Request Test Coverage Report for Build 8274

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 8237:
0.0%

Covered Lines:
1461

Relevant Lines:
1461

💛  - Coveralls


Tidy up README

[ ] Template header is still there
[ ] Support section refers to the repo as a library
[ ] Add list of samples included in this repo

[x]  Template header is still there
[x] Support section refers to the repo as a library
[x] Add list of samples included in this repo


Typescript and framework integration

Is your feature request related to a problem? Please describe.
For those working with typescript, integrating Swiffy-slider requires some work. In my case, the builder.io Qwik framework but it applies to any others like Remix.
Describe the solution you'd like
Make Swiffy-slider Typescript friendly and/or submit a type in @types/* repo
Describe alternatives you've considered
There isn't much alternative.
Additional context
Using the reference of window shall eventually be reconsidered in order to make the integration within other frameworks easier.

I've already worked on it and got a simple Typescript support.
If interested, I would create a new branch using this issue's ID.

Thank you for supporting and taking Swiffy Slider for a spin.
I have little experience with TS so I cannot say much here.
Swiffy Slider comes in classic JS that can be included in script tags or you can import it as an ESM.
The latter is supported in TS as far as I can tell:
https://www.typescriptlang.org/docs/handbook/esm-node.html
I am a huge fan of simple things and following standards. It is fine to add TS support - but if it requires complexity and going away from standards, I might not be in favor. Feel free to create a pull so I can see the implications.
Thanks, Nicolai

@nicped You don't need to change your code. You (or someone else) could simply create a new package @types/swiffy-slider with the type definition only. The advantage with typescript is, if you transpile your code the type definition is generated automatically.
Anyway. I like simple things too. Instead Typescript i prefer for small scripts just jsdoc, that is sadly missing in your code and could be helpful. VSCode supports it and use it for linting too.

I can go with adding the JSDoc - as they are minified away and does not come with any requirements. Even though the comments pollute the src code :-)
I am either too smart or not smart enough to add TS types.... Your call!. But feel free to create a package,.

1 on adding JS doc. I'll be available only in a couple of weeks before getting into it


Update KV Storage syntax to use default export

As pointed out in this pull request in the spec's GitHub repo for KV storage. The KV storage api now uses a default export instead of a named export. This can be tested in Chrome Canary with experimental web features turned on and is also present in the current spec.
What's changed, or what was fixed?

Updated a markdown file to reflect the current KV Storage API.

Fixes: https://github.com/google/WebFundamentals/issues/8015

@philipwalton - can you take a look and review & merge? Thanks!
cc: @domenic

:tada: This has been pushed live to https://developers.google.com/web/


Exclude .asf.yaml from the configuration of the rat plugin

Description
Exclude .asf.yaml from the configuration of the rat plugin

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

@clintropolis You are welcome.


Update team selection in spaceship to work with the new iTC

Fixes https://github.com/fastlane/fastlane/issues/9237 🎉
As of 22nd May 2017 we have to access the new API endpoint olympus.itunes.apple.com to get a valid itctx value for our session 💥

🐑

It works great! Thanks! @KrauseFx 🐑


True Controller singleton; initial test suite

Refactors the Controller into a true singleton that preserves its internal state across instantiation calls.
This makes it globally available without needing to pass an instance around. So it's been eliminated from all child View creation calls. The parent View still deposits a Controller instance into itself which the child Views can still access.
Also creates an initial test suite and related pytest config and dependencies.
Includes basic tests to verify the Controller's singleton behavior.

pytest.ini: Just points to the new tests/ subdir so we can run pytest from the root.
setup.py: Required so that we can install seedsigner as a python module. This makes the project code accessible to the test suite (otherwise from seedsigner import ... would fail). File was copy-and-pasted from python docs so not all fields have sensible values at the moment.

Merged this into PR #46. Closing this PR.


New Feature: Ability to pass in $metadata.xml file

When working locally in vscode and doing rapid development between the Odata web server and the angular web client it would be really nice to either be able to access localhost like "http://localhost/odata/$metadata" (doesn't seem likely with the way docker works) or send into the docker through command line a $metadata.xml file with a different parameter like
docker run -it --rm -v C:\temp\test:/local diegomvh/odataapigen Name=TripPin StringAsEnum=true MetadataFile=c:\$metadata.xml Output=/local

The XDocument.load() works with URIs and identifies the source to load the contents of the xml file.
I guess something like:
docker run -it --rm -v C:\temp\test:/local diegomvh/odataapigen Name=TripPin StringAsEnum=true MetadataFile=/local/trippin.xml Output=/local

Where trippin.xml should be on the path to map inside the container c:\temp\test according to the example.
It should work
regards

Create an example that uses a local file to generate the schema.
MsGraph
regards


increased platform size for Domino 5.0

while testing DDL5.0 deployment, found out that cluster is not large enough to run all DDL 5.0 services

This was resolved with #29.


readme.md

Just a minor issue. Change 'product' to 'review' in below code:
#### PRODUCT REVIEWS ####
reviews = wapy.product_reviews('21853453')
for review in reviews:
    print (product.reviewer)

So correct one should be:
#### PRODUCT REVIEWS ####
reviews = wapy.product_reviews('21853453')
for review in reviews:
    print (review.reviewer)

You're right, thank you for noting this!
Would you like to open a PR and fix this issue? or should I work on it.


1557 conditional rules for Contacts dashboard tiles\links

Addresses 2047
🚀 Impact:

Added conditional rules, within the dashboard tile fixtures, for role industry_user to dynamically  display dashboard tile link, Contacts
Added vitests

🔬 Local Testing:

From terminal command, start the api server:

cd bc_obps
make reset_db
make run

From terminal command, start the app development server:

cd client && yarn dev-all

Test Dashboard Tile Link - Contacts - user has userOperator

Navigate to http://localhost:3000
Click button "Log in with Business BCeID"
Sign in to Keycloak using bc-cas-dev
Expected results:

Redirection to http://localhost:3000/dashboard page for role industry_user_admin
Administration tile has link Contacts

Click link Administration\Contacts
Expected results:

Redirection to http://localhost:3000/administration/contacts
Contacts grid is displayed

Click breadcrumb Administration
Expected results:

Redirection to http://localhost:3000/administration page for role industry_user_admin
Dashboard has tile Contacts

Click link Contacts

Redirection to http://localhost:3000/administration/contacts
Contacts grid is displayed

Test Dashboard Tile Link - Contacts - user DOES NOT have userOperator

Navigate to http://localhost:3000
Click button "Log in with Business BCeID"
Sign in to Keycloak using bc-cas-dev-secondary
Expected results:

Redirection to http://localhost:3000/dashboard page for role industry_user
Administration tile DOES NOT have link Contacts

Click link Administration tile
Expected results:

Redirection to http://localhost:3000/administration page for role industry_user
Dashboard DOES NOT have tile Contacts

Navigate to  http://localhost:3000/administration/contacts
Expected results:

Redirection to http://localhost:3000/administration

Test Select Operator

Navigate to http://localhost:3000
Click button "Log in with Business BCeID"
Sign in to Keycloak using bc-cas-dev-secondary
Click link Administration\Select an Operator
Select an operator using the by Business Legal Name == New Operator 3 Legal Name
Click button Select Operator
Click button  Yes, this is my operator.
Expected results:

Confirmed operator with Administrative access setup message displays

Click button Request Access
Expected results:

Received request  with Administrative access setup message displays

Approve  Administrative access setup, from terminal command, approve the request for access to New Operator 3 Legal Name

psql
\c registration
UPDATE erc.user_operator SET status='Approved' WHERE user_id='279c80cf-5781-4c28-8727-40a133d17c0d' AND operator_id='438eff6c-d2e7-40ab-8220-29d3a86ef314';

Navigate to Navigate tohttp://localhost:3000`
Expected results:

Redirection to http://localhost:3000/dashboard page for role industry_user
Administration tile has link Contacts

Click link Administration tile

Redirection to http://localhost:3000/administration page for role industry_user
Dashboard has tile Contacts

Excellent work, this is working great!
Only thing I noticed was the middleware seems to be getting called multiple times on the dashboard page, though not on other pages:
To reproduce this stop your backend terminal, use cli command clear to clear the terminal of previous requests, then start it again with make run.
Go to http://localhost:3000/dashboard assuming you are logged in.
Though it seems like the user-profile also gets called multiple times on the dashboard page so it's okay if this is a tech debt ticket though we should probably investigate it.

@marcellmueller
I had also noticed the multiple calls to user-profile in the past and assumed it was due to Next.js in DEV with strict mode= TRUE causing double renders of components but; perhaps, we should explore the multiple calls further than an assumption.  I'll make a ticket.  Thanks :)


Update image on docker hub

uncompress files in image on dockerhub

The files are still gzipped inside of the container image so it doesn't work:
[Error: ENOENT: no such file or directory, stat '/app../public/index.html'] {
errno: -2,
code: 'ENOENT',
syscall: 'stat',
path: '/app../public/index.html',
expose: false,
statusCode: 404,
status: 404
}

The image on dockerhub is still a year old. Could you please update the image on docker hub.


Try multiple cache locations

In CI environment, depending on the user and whether it has a home, flakehell would try to create the /.cache directory, and would fail with a PermissionError.
This commit changes that by making flakehell try to create the cache at different locations: $HOME/.cache/flakehell and ./.flakehell_cache (the chances we can write into the current directory in CI are high).
A --disable-cache option would be much more robust but seems to be way more complicated to implement.

Oh, another solution would be to read the CACHE_PATH from an environment variable. Would you prefer that method?

I've opened #102 as a replacement, I think it's a simpler and better approach.

Thank you! Let's proceed with #102 then :)


"Message for new versions points to download.qgis.org, which is not very convenient"

Description
See https://github.com/qgis/QGIS/issues/49446.

@jef-n the redirect for download.qgis.org is on qgis2 (old webserver).
Do you agree that it is better to redirect that to the (language) download page in qgis.org, like https://qgis.org/en/site/forusers/download.html?
Not sure if we are able to redirect to the right language though...
Or should we try to make a nice human url on qgis.org so qgis.org/download (not qgis.org/downloads is already used)>


Add favicon

Yeah, a favicon.

Fixed with https://github.com/svoctor/lisper/commit/c219991de57a030ae4af5d2d17ddea9d708e5d50


Code hierarchy is disorganized

All of Qurry's code is currently in a single package, which is hard to understand. It seems very likely that Qurry could be separated into sub-packages which might be easier to work on independently.

Done! Can be improved in the future, of course.


Supporting Lua scripts?

Hi, is there any plan to support Lua scripts in tokio-redis?

Hey, this repo is really just a proof of concept to demonstrate tokio and not really meant to be a production ready redis lib. I'm hoping that someone takes on the task of implementing a real tokio / redis lib (maybe you?)


collection.Iterable is deprecated in Python 3.9+

🐛 Describe the bug
Any code path that touch this particular line of code (for example trainer.qat)
https://github.com/Deci-AI/super-gradients/blob/7907c486d1a0bb479c68590ffd5bc8478f9f18b8/src/super_gradients/training/utils/utils.py#L595
On Python 3.9+, this will have this error
AttributeError: module 'collections' has no attribute 'Iterable'

See this SO answer: https://stackoverflow.com/a/72032097
Versions
Collecting environment information...
PyTorch version: 1.11.0+cu102
Is debug build: False
CUDA used to build PyTorch: 10.2
ROCM used to build PyTorch: N/A
OS: Ubuntu 18.04.6 LTS (x86_64)
GCC version: (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.27
Python version: 3.10.11 (main, Apr 20 2023, 19:02:41) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-4.4.0-143-generic-x86_64-with-glibc2.27
Is CUDA available: True
CUDA runtime version: 11.8.89
CUDA_MODULE_LOADING set to:
GPU models and configuration: GPU 0: Tesla T4
Nvidia driver version: 525.105.17
cuDNN version: /usr/lib/x86_64-linux-gnu/libcudnn.so.7.6.5
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              16
On-line CPU(s) list: 0-15
Thread(s) per core:  1
Core(s) per socket:  1
Socket(s):           16
NUMA node(s):        1
Vendor ID:           GenuineIntel
CPU family:          6
Model:               79
Model name:          Intel(R) Xeon(R) Platinum 8260M CPU @ 2.40GHz
Stepping:            0
CPU MHz:             2400.000
BogoMIPS:            4800.00
L1d cache:           32K
L1i cache:           32K
L2 cache:            1024K
L3 cache:            36608K
NUMA node0 CPU(s):   0-15
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon nopl xtopology tsc_reliable nonstop_tsc pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch invpcid_single kaiser fsgsbase tsc_adjust bmi1 avx2 smep bmi2 invpcid rdseed adx smap xsaveopt arat
Versions of relevant libraries:
[pip3] numpy==1.23.0
[pip3] pytorch-quantization==2.1.2
[pip3] torch==1.11.0
[pip3] torchmetrics==0.8.0
[pip3] torchvision==0.12.0
[conda] numpy                     1.23.0                   pypi_0    pypi
[conda] pytorch-quantization      2.1.2                    pypi_0    pypi
[conda] torch                     1.11.0                   pypi_0    pypi
[conda] torchmetrics              0.8.0                    pypi_0    pypi
[conda] torchvision               0.12.0                   pypi_0    pypi

This indeed a bug. On 3.9 it raises a warning, on 3.10 it fails with an ImportError.
However the solution is quite simple: collections.Iterable -> typing.Iterable should do the work.
Are you interested in making a PR with a fix?  Should be a straightforward fix if you master the signed commits.

@BloodAxe sure. Thank you for confirming this quickly. I will send a PR tonight

this PR has been merged?

@skyprince999 sorry. I have loss interests in using yolonas because of the hops I have jumped through to getting started. I tried yolov8 and it’s good enough for my use case

This indeed a bug. On 3.9 it raises a warning, on 3.10 it fails with an ImportError. However the solution is quite simple: collections.Iterable -> typing.Iterable should do the work. Are you interested in making a PR with a fix? Should be a straightforward fix if you master the signed commits.
@BloodAxe Hi,
I tried collections.Iterable -> typing.Iterable but it not work
I tried collections.Iterable -> collections.abc.Iterable then it work
if isinstance(inputs, collections.abc.Iterable) and not isinstance(inputs, str)

This indeed a bug. On 3.9 it raises a warning, on 3.10 it fails with an ImportError. However the solution is quite simple: collections.Iterable -> typing.Iterable should do the work. Are you interested in making a PR with a fix? Should be a straightforward fix if you master the signed commits.

@BloodAxe Hi, I tried collections.Iterable -> typing.Iterable but it not work I tried collections.Iterable -> collections.abc.Iterable then it work if isinstance(inputs, collections.abc.Iterable) and not isinstance(inputs, str)

typing is imported at the start of the file,
from typing import XXXX,XXXx, Iterable
so you need to change it to Iterable

@DoManhQuang  Can you please elaborate what error do you get for typing.Iterable and what python version did you test on?

Yes, I imported typing and change collections.Iterable -> typing.Iterable and it not work. Now, try again then typing.Iterable is working. I can't reproduce the issue :(
Env: Colab, python 3.10.12

How to solve this problem on Kaggle, where we do not have access to the library to edit it?

@ferdzo Hi, you can try my way:
1/ git clone https://github.com/Deci-AI/super-gradients.git
2/ change collections.Iterable =>typing.Iterable or collections.abc.Iterable from https://github.com/Deci-AI/super-gradients/blob/7907c486d1a0bb479c68590ffd5bc8478f9f18b8/src/super_gradients/training/utils/utils.py#L595
3/ pip install -r ./super-gradients/requirements.dev.txt
4/ pip install ./super-gradients

It is fixed in master already: https://github.com/Deci-AI/super-gradients/pull/1178
You can expect the next version to be available soon.
Meanwhile you can install SG from master: pip install git+https://github.com/Deci-AI/super-gradients.git

@ferdzo Hi, you can try my way: 1/ git clone Deci-AI/super-gradients.git
2/ change collections.Iterable =>typing.Iterable or collections.abc.Iterable from
https://github.com/Deci-AI/super-gradients/blob/7907c486d1a0bb479c68590ffd5bc8478f9f18b8/src/super_gradients/training/utils/utils.py#L595
3/ pip install -r ./super-gradients/requirements.dev.txt
4/ pip install ./super-gradients

Well, I've fixed it on my computer and on Colab, because I can access the library and edit it, too much gymnastics to fix it on Kaggle, thanks anyway!

Fixed in 3.1.2


I have problems with celeba-hq 384*384

Hello ! I use multi_gpu to train 384*384 pictures on celeba-hq, but I still get an error when I train only one picture per gpu: RuntimeError: CUDA out of memory. I used NVIDIA Corporation GP102 [TITAN Xp].

I'm not sure what caused your problem. I trained HQ-384 with --batch_size=4 on four Nvidia GTX 1080 Ti. This means each GPU has one image tensor. My GPU usage is:
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 418.39       Driver Version: 418.39       CUDA Version: 10.1     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 108...  Off  | 00000000:02:00.0 Off |                  N/A |
| 29%   57C    P2    82W / 250W |  11153MiB / 11178MiB |    100%      Default |
+-------------------------------+----------------------+----------------------+
|   1  GeForce GTX 108...  Off  | 00000000:17:00.0 Off |                  N/A |
| 41%   74C    P2    79W / 250W |   7865MiB / 11178MiB |    100%      Default |
+-------------------------------+----------------------+----------------------+
|   2  GeForce GTX 108...  Off  | 00000000:65:00.0 Off |                  N/A |
| 35%   60C    P2    92W / 250W |   7865MiB / 11176MiB |    100%      Default |
+-------------------------------+----------------------+----------------------+
|   3  GeForce GTX 108...  Off  | 00000000:B3:00.0 Off |                  N/A |
| 29%   64C    P2    79W / 250W |   7865MiB / 11178MiB |    100%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      5437      C   python3                                    11143MiB |
|    1      5437      C   python3                                     7855MiB |
|    2      5437      C   python3                                     7855MiB |
|    3      5437      C   python3                                     7855MiB |
+-----------------------------------------------------------------------------+

Did you set --batch_size as the same size as the number of your GPUs?

Thanks, I can run it, but the network still doesn't work. Have you modified the parameters?

What do you mean it doesn't work? Are your network not converging? For example, it still had extremely high losses of G and D after an epoch training.

My dc_loss and gc_loss don't drop, and gf_loss is very large. The attributes of the generated image have not changed.

In my experiment, I trained it with a batch size of 8 on 4 GPUs. The attributes are not clear until the 8th epoch. It looks like you've trained 8,000 iterations, which is around 1 epoch if you also trained with a batch size of 4. Please be patient.
epoch = iteration * batch_size

To be honest, if you have only two GPUs. Multi-GPU training is much less efficient than single-GPU training. I recommend to use multi-GPU if you have at least four GPUs.
p.s. The following is my training history of 384_shortcut1_inject1_none_hq:

Thank you very much for your advice. I am too eager to see the results. I will train patiently.


painting fence problem

Is your feature request related to a problem? Please describe.
problem statement :
Given a fence with n posts and k colors, find out the number of ways of painting the fence such that at most 2 adjacent posts have the same color.
Describe the solution you'd like
This is a classic problem in dynamic programming
Do you want to work on it
I would like to implement this in c++
Please assign me this issue as part of GSSOC '21

Please assign it to me . @nakul-19 @raksha009 @plazzy99 @tarun26091999 @rish-singhal

Please assign this to me in Java.


✨ exit process when selected is None

This addresses https://github.com/zeenix/gimoji/issues/86
I'm not 100% certain that this exit code will work for all OSes (namely windows), but hopefully the checks will determine this.
Fixes #86.

This addresses #86

Thanks so much for this contribution! It's not biggie but in the future, please use the GH keywords to automatically resolve issues being addressed. I took the liberty of modifying the description for you here. :)

I'm not 100% certain that this exit code will work for all OSes (namely windows),

I think it will work but then again I've never actually tried gimoji on Windows. :)

but hopefully the checks will determine this.

Unfortunately we don't have any tests at all. :( I don't even know how to go about testing a fullscreen TUI like this. :shrug:

Thanks so much for this contribution! It's not biggie but in the future, please use the GH keywords to automatically resolve issues being addressed. I took the liberty of modifying the description for you here. :)

Apologies, thanks for that!


Screenshot has to be changed while starting a new instance

Tutorial: https://developers.sap.com/tutorials/cp-starter-ibpm-employeeonboarding-5-run.html
Step 4: Execute the workflow sample application
Screenshot has to be changed while starting a new instance
Expected:

Actual:

Best Regards,
Priyanka

Tutorial have been updated with new screenshot.


Reorganize the repository into a set of bare LSP modules and SourceKit modules

Reworks https://github.com/apple/sourcekit-lsp/pull/192 a bit to further reduce surface area.

This PR prepares the repository to allow splitting the LSP modules off. These LSP modules will provide the ability to write custom LSP servers and clients in Swift. The sourcekit-lsp repository will build on top of this new package to provide an LSP implementation that creates a language server for Swift and C-based-languages.
In particular this PR performs the following changes:

Move logging from SKSupport to new module LSPLogging
Sink some code from SKSupport to live alongside its only user, which is in JSONRPC message splitting.
Move CustomCodable to LanguageServerProtocol

This requires a workaround of directly initializing _range using the property wrapper because of a compiler bug that causes compilation to fail otherwise

Audit imports for those that are actually needed
Exhaustively declare all dependencies in Package.swift, not relying on transitive dependencies
This made it easier to find dependencies that should not exist and allowed easier moving of files if downstream package dependencies are broken. I think it’s good for review as well
Remove dependencies that are not needed

@swift-ci please test

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test linux


Drawing attention to -ADObjectDN parameter.

I noticed in AdSyncConfig.psm1, the comment-based help for the Set-ADSyncUnifiedGroupWritebackpermissions function doesn't include a .Parameter keyword for -ADObjectDN parameter, however, the parameter is there:
        # DistinguishedName of the target AD object to set permissions (optional)
        [string] $ADobjectDN = $null,

And the description also covers it:
 .DESCRIPTION
        The Set-ADSyncUnifiedGroupWritebackPermissions Function will give required permissions to the AD synchronization account, which include the following:
        1. Generic Read/Write, Delete, Delete Tree and Create\Delete Child for Group Object types and SubObjects

        These permissions are applied to all domains in the forest.
        Optionally you can provide a DistinguishedName in ADobjectDN parameter to set these permissions on that AD Object only (including inheritance to sub objects).
        In this case, ADobjectDN will be the Distinguished Name of the Container that you desire to link with the GroupWriteback feature.

...and also one of the examples covers it:
.EXAMPLE
       Set-ADSyncUnifiedGroupWritebackPermissions -ADConnectorAccountName 'ADConnector' -ADConnectorAccountDomain 'Contoso.com' -ADobjectDN 'OU=AzureAD,DC=Contoso,DC=com'

It is a good idea to use it, vs granting permissions all through the entire forest which won't be necessary (only the chosen OU needs the permissions set).

@JeremyTBradshaw : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

#sign-off


Fixed catalog search by collection id

In SentinelHubCatalog.search we also support searches by only a collection id, which in some cases of BYOC and batch collections might be more natural. This PR handles such a case correctly and adds a test.

Codecov Report

Merging #197 (74eb306) into develop (9ceda87) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #197      +/-   ##
===========================================
+ Coverage    89.58%   89.70%   +0.11%
===========================================
  Files           38       38
  Lines         3668     3670       +2
===========================================
+ Hits          3286     3292       +6
+ Misses         382      378       -4

Impacted Files
Coverage Δ

sentinelhub/sentinelhub_catalog.py
94.80% <100.00%> (+0.13%)
:arrow_up:

sentinelhub/os_utils.py
71.87% <0.00%> (-6.25%)
:arrow_down:

sentinelhub/download/handlers.py
81.48% <0.00%> (+11.11%)
:arrow_up:

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 9ceda87...74eb306. Read the comment docs.


Add fix_ligand flag to prepare_sys_for_amber scripts

We had some issues with ligands with wrong charge assignments and missing hydrogens, and Xavi suggested it would make sense to add a fix directly in the OpenDuck code.

ping @AlvaroSmorras, in case you missed this

Thanks!


Output descriptors to specify custom derivations and change

this is the actual way to specify the derivation of the key
{
"xpub": "xpub...",
"index": 0,
"derivationMode": "native_segwit",
"birthday": "2017/08/29"
}
Which is straight, simple and compatible with data a user can get from Ledger Live.
To make the application more future proof, It would be convenient to be able to specify somehow the Output Descriptor (https://github.com/bitcoin/bitcoin/blob/master/doc/descriptors.md) of the watch-only wallet we are going to import into bitcoin core.
It could cover:

Different derivations than bip44, bip49 or bip84
Internal addresses
In future also multisig configurations

Also importmulti RPC call in bitcoin core supports Output Descriptors and the future version of the wallet will be more and more "descriptor based".
I would think something like:
  {
      "descriptor": "wpkh([18734cbe/84'/1'/0']tpubDAZhhJ9SaAKCt2gngU8aA2babxVAM93oBgo2wGknMUDmtLrzvLnynFuAYo4eME3MqWpcTM3DpGL9J45nQw7CA7VroAY8e7v9Zo6r7mddUMS/0/*)#4c563hxu",
      "index": 0,
      "birthday": "2017/08/29"
    }

Thanks for your feedback @gabridome. It's a great suggestion indeed.
We actually already import output descriptors (see this). The descriptors, however, are constructed from the xpub, index, and derivationMode. To be fair, this is really brittle and much more error-prone than what you suggest.
I think specifying output descriptors in the config is not so user friendly, and easy to get wrong if you don't understand the schema. So for me, it needs to be complemented with displaying the receive and change descriptors directly on Ledger Live, in the account details section, so users can just copy-paste it in the config.

So for me, it needs to be complemented with displaying the receive and
change descriptors directly on Ledger Live, in the account details section,
so users can just copy-paste it in the config.

I agree and will try to suggest it.
Il giorno lun 15 giu 2020 alle ore 12:02 Anirudha Bose <
<EMAIL_ADDRESS>ha scritto:

Thanks for your feedback @gabridome https://github.com/gabridome. It's
a great suggestion indeed.
We actually already import output descriptors (see this
https://github.com/onyb/ledger-sats-stack/blob/634dc011dda2f7c0bc12cbfe478360315457b12e/bus/wallet.go#L91-L98).
The descriptors, however, are constructed from the xpub, index, and
derivationMode. To be fair, this is really brittle and much more
error-prone than what you suggest.
I think specifying output descriptors in the config is not so user
friendly, and easy to get wrong if you don't understand the schema. So for
me, it needs to be complemented with displaying the receive and change
descriptors directly on Ledger Live, in the account details section, so
users can just copy-paste it in the config.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onyb/ledger-sats-stack/issues/24#issuecomment-644032357,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADQQ3RAUDG4VMQFEYFS6B3RWXWSTANCNFSM4N53Q3UA
.

--
Please avoid unnecessary attachments. Write in the body of the message.
Thank you.
Gabriele Domenichini
pgp fingerprint: DC47 A9B4 CF41 6373 6EBB  E14E A48F BF05 1C43 F817

@gabridome In your example config, you used only the external chain, i.e. 0, in the output descriptor.
wpkh([18734cbe/84'/1'/0']tpubDAZhhJ9SaAKCt2gngU8aA2babxVAM93oBgo2wGknMUDmtLrzvLnynFuAYo4eME3MqWpcTM3DpGL9J45nQw7CA7VroAY8e7v9Zo6r7mddUMS/0/*)#4c563hxu

This also means that we'll need to add another output descriptor in the JSON config for the change path, i.e. 1, which seems repetitive to me. Do you think it is a problem to have two config entries per account, or is it better to be explicit even at the cost of some repetition?

This has been fixed in https://github.com/onyb/ledger-sats-stack/commit/f1677b44df05152eaacae099b8c3842ce9aa193f, and available in v0.4.0-alpha. The descriptor will be available on Ledger Live in a future release.
The configuration is now purely based on output descriptors. Here's an example of the new config format:
{
  "accounts": [
    {
      "descriptor": "wpkh([18734cbe/84'/1'/0']tpubDAZhhJ9...7mddUMS)#4c563hxu",
      "birthday": "2020/01/01"
    }
  ],
  "rpcurl": "localhost:8332",
  "rpcuser": "user",
  "rpcpass": "pass",
  "notls": true
}

Notice that the descriptor now describes the account rather than the address path, hence there needs to be only one entry per account, to answer my own question in the previous comment.

Notice that the descriptor now describes the account rather than the address path, hence there needs to be only one entry per account, to answer my own question in the previous comment.

Yeah. Sorry for the late reply.
I would have been very explicit with something like:
{
  "accounts": [
    {
      "external": "wpkh([18734cbe/84'/1'/0']tpubDAZhhJ9...7mddUMS/0/*)#xxxxxxxx", # obviously the checksum don't match
      "internal": "wpkh([18734cbe/84'/1'/0']tpubDAZhhJ9...7mddUMS/1/*)#xxxxxxxx",
      "birthday": "2020/01/01"
    }
  ],
  "rpcurl": "localhost:8332",
  "rpcuser": "user",
  "rpcpass": "pass",
  "notls": true
}

But I think that even if the solution you choose is not perfect (it doesn't define the whole path) at least is not "wrong".
We can see in the future what decisions will be taken at the "Ledger Live Level" to rethink about the issue.

Reopening this issue, because your suggestion is quite cool! 👍
It's always better to have the exact descriptors that'll be imported in the config file. The account-level descriptor is more convenient (less no. of characters), but is less transparent.

Fixed in v0.5.0-alpha. Thanks for your valuable feedback!


[Question] How to get params in deeply nested component?

Hello, thanks a lot for your hard work. This router works really good. The only question which I didn't find in docs is how to pass params in deep nested component? In react-router we had withRouter HOC for this. Right know I need to pass in by hands. Any suggestions?
<comp1>
  <comp2>
    <comp3> // here I can get query params

I would reach for React's Context API

Would be nice to see somewhere in docs an official suggestion.

I think the official suggestion would be implied where it describes route components as "just React components", so there's nothing you can't do that you could do with any React Component, and I don't think there's going to be any proprietary solution for it.
So... you could do render props from a parent, React.cloneElement with parent params, Context etc..

I've just find in docs the official solution. There is a component Location which returns location on any level.
<Location>
  {({ location })=> {
    // ...
  }}
</Location>

Here is a link in docs


New version: Google.Chrome.Canary version 128.0.6584.0

Created by 🥟 Dumplings in workflow run #6447.

Log
Updated: 128.0.6583.0 → 128.0.6584.0
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


A11y_AzureDataStudio_Notebooks - Run_Usability:When We run any Query in Notebook, Screen Reader is not announcing all query Results

GitHub Tags:
#A11yUsable; #A11yTCS; #A11ySev4; #DesktopApp; #BM-AzureDataStudio-Win32-Feb2022; #Azure Data Studio; #Win32; #Screen Reader; #NVDA; #Benchmark; #JAWS;
Environment and OS details:
Azure Data Studio Version: 1.42.0 (user setup)
Windows 11 Enterprise Version	22H2
Installed on	28-12-2022
OS build	22621.1413
Experience	Windows Feature Experience Pack 1000.22639.1000.0
NVDA Version 2023.1
JAWS version 2023
Pre-Requisite:

Install Azure data studio From here https://learn.microsoft.com/en-us/sql/azure-data-studio/download-azure-data-studio?view=sql-server-ver16&tabs=redhat-install%2Credhat-uninstall
Once installation done , Install MySQL Extension in Azure data studio
Tab Till  new connection and press enter
Enter a valid server name username and password and press connect

Repro Steps:

Launch Azure Data Studio.
Turn on NVDA
Open an existing notebook that has code cells in it.
Click Run Cells on Notebook toolbar.
Verify that Screen reader announcing all the results or not

Actual Result:
When We run any Query in Notebook, NVDA is not announcing all query Results
Its is not announcing "Commands completed successfully(300024 row(s) affected Displaying Top 5000 rows.
Total execution time: 00:00:16.918"
Notes:
1.Issue repro with JAWS also
2.For New query also this issue is repro
Expected Result:
Query  results should be announced by the screen reader
Attachment:

Primary Bug: https://github.com/microsoft/azuredatastudio-mysql/issues/194


Make ScriptRuntimeError importable from the pyjq module

It'd be nice if one could import ScriptRuntimeError from the pyjq module.
from pyjq import ScriptRuntimeError

currently one has to import use pyjq._pyjq.ScriptRuntimeError.

As an alternative pyjq could issue plain RuntimeError instead of creating a custom Exception.

I have done it.

I have done it.


resolves #24: fix async binary transfer, fix sync content length

Fixes async binary transfer, runs transfer tests both async and sync, and fixes a post content trimming mistake.

I added https tests and the ability to set rejectUnauthorized, with a test.... demonstrating a problem with sync errors.

This looks great but could you please create one pull request per change otherwise it's hard to review:

JSON.stringify(error)
rejectUnauthorized
Fix content-length with trailing \n
Fix async binary transfer

Thanks

Separate pull requests would be a lot of work and don't reflect the history, which was that I made binary transfer work in a project, wrote some tests, and found some fairly egregious bugs which I fixed.  Perhaps I could annotate the changes instead?

Separate pull requests would be a lot of work and don't reflect the history,

The problem is that there's no history, all changes are already squashed into a single commit.

Perhaps I could annotate the changes instead?

Yes that could help.
But it feels like you are fixing multiple things at once. So I would still like to have one commit/one pull request per fix/change.

All your changes has been integrated in other pull requests. Thanks for your fine work :ok_hand:
Could you please test against master to make sure that everything is working as expected?
Please note that I did a few changes to comply with the XMLHttpRequest.

Closing, thanks again for all these fixes! :clap:


build: drop lerna

closed via https://github.com/qiwi/masker/commit/ab8689a0a58860e38835234aae810e6ac38e8a37


DeltaTable class name collision

Description
We're working on building a Dask Delta Lake reader.  I was writing an integration test that wrote out a Delta Lake with delta and read it in with delta-rs.  The test was behaving unexpectedly because DeltaTable is defined in both repos.
I don't think these libraries will normally be used in conjunction, but think it's wise to avoid name collisions in any case.  Also think different name will make StackOverflow questions easier and debugging with Googling easier.
Thanks for making a great lib.

Thanks @MrPowers for the report. The concern of SEO is certainly valid. I also don't have a good solution in mind. You are right that they aren't intended to be used in conjunction. For the rare case that they need to be used together, we will simply have to rely on python package namespace or import one of the DeltaTable with alias.
Another way to look at this could be viewing DeltaTable as a generic search term like Dataframe and let users search DeltaTable together with project name. For example pyspark DeltaTable or native DeltaTable. There is also another pure python deltalake implementation that also uses DeltaTable as the class name, so even if we change to something else, people will still get confused.

@houqp - I'm building a dask-interop project that writes out Delta files with Spark and reads them in with delta-rs.  I just created different conda environments as a work-around.  Not ideal, but like you mentioned, not a common issue, so think it's fine as-is.
On a happy note, I was able to use the delta-rs API to easily read a Delta table into a Dask DataFrame, great work building this lib!

Interesting, @MrPowers do you have to create a separate conda env? Shouldn't the DeltaTable class namespaced by the python package?


Track progress of request

Hello everyone! I read the docs but couldn't find a way to track progress of current request – for example, to show progress bar. Is it possible in Moya?

Moya only supports progress notifications while uploading a file, not for regular requests (thought Alamofire does support this, so it shouldn't be too difficult to add this to Moya. I'm a little surprised that Moya doesn't already support this, to be honest, it's something we should consider adding. Could you tell us more about how you'd expect this to work?

It shouldn't be a ton of work (and should even be backwards compatible) to add support for Moya.ProgressBlock to the internal requestNormal method. Actually, now that I'm looking at it, sendUpload and sendRequest have a lot of duplicated code...

@colinta aye, could be worth factoring a bit out. If we can be backwards compatible, that's great, but we're about to have a major release with other breaking changes, so we could roll this feature into that release.

Thanks for answer @ashfurrow. I don't know how to implement this in Moya but I would suggest this interface for end user:
public func request(target: Target, progressClosure: ((progress: NSProgress) -> Void)? = nil, completion: Moya.Completion) -> Cancellable. Probably this interface won't break current code (not sure how it will work with trailing closure syntax though).

@alexdoloz yup, that makes sense! The trailing closure syntax is preserved that code.

@alexdoloz now that #543 is merged, this should work if you use the latest version of Moya, on the master branch. If you're using CocoaPods, you can change the Podfile to include pod 'Moya', git: 'https://github.com/Moya/Moya.git' and do a pod update and it should work. Let us know!

I might have ONE more PR...

On Jul 13, 2016, at 11:19 AM, Ash Furrow<EMAIL_ADDRESS>wrote:
@alexdoloz https://github.com/alexdoloz now that #543 https://github.com/Moya/Moya/pull/543 is merged, this should work if you use the latest version of Moya, on the master branch. If you're using CocoaPods, you can change the Podfile to include pod 'Moya', git: 'https://github.com/Moya/Moya.git' and do a pod update and it should work. Let us know!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Moya/Moya/issues/542#issuecomment-232424891, or mute the thread https://github.com/notifications/unsubscribe/AABrsnJZCNeLofJn60mOY489-OcM-nM6ks5qVR4hgaJpZM4JLOWe.

Ah, okay! No rush.

#544 whitespace changes only 😃

I think this was closed by #543, re-open if not.

Hello again, guys! Thank you for all your hard work implementing this feature. However, i tried to integrate it into application and noticed it doesn't work as I expect. For some reason, bytesExpected is always -1, therefore progress is always 1.0.
Progress: ProgressResponse(totalBytes: 21654, bytesExpected: -1, response: nil)
Do you have any ideas about it?

Can you open another issue about the -1 bytes expected? Link back here for context 👍
I think the bytes expected comes from Alamofire, and ultimately the server. It may be that -1 is what ALamofire provides when there is no expected number of bytes available, which might could from the server. Not sure, though.

Okay, I figure out what happened. Server doesn't return Content-Length header. I tested other resources and progress works fine (provided these servers return Content-Length). So it was my mistake, no need to create new issue :)

@alexdoloz I'm glad you got it figured out!

Glad to hear that you got to the bottom of it! Good to know about the Content-Length header.
When you feel good about it, please close this issue. 👍

On Jul 26, 2016, at 6:22 AM, alexdoloz<EMAIL_ADDRESS>wrote:
Okay, I figure out what happened. Server doesn't return Content-Length header. I tested other resources and progress works fine (provided these servers return Content-Length). So it was my mistake, no need to create new issue :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


build error with google::protobuf::internal

I get an error when building the cartographer_ros,the error log is bellow,can anyone help me?
==> Processing catkin package: 'cartographer_ros'
==> Building with env: '/home/coldrain/catkin_ws/carto_ws/install_isolated/env.sh'
build.ninja exists, skipping explicit cmake invocation...
==> ninja build.ninja in '/home/coldrain/catkin_ws/carto_ws/build_isolated/cartographer_ros'
ninja: no work to do.
==> ninja -j8 -l8 in '/home/coldrain/catkin_ws/carto_ws/build_isolated/cartographer_ros'
[8/45] Building CXX object CMakeFiles/cartographer_ros.dir/cartographer_ros/ros_map.cc.o
FAILED: /usr/bin/c++   -DROSCONSOLE_BACKEND_LOG4CXX -DROS_PACKAGE_NAME=\"cartographer_ros\" -isystem /usr/local/include -isystem /usr/local/include/pcl-1.8 -isystem /usr/include/eigen3 -isystem /home/coldrain/catkin_ws/carto_ws/install_isolated/include -isystem /opt/ros/kinetic/include -isystem /opt/ros/kinetic/share/xmlrpcpp/cmake/../../../include/xmlrpcpp -isystem /usr/include/pcl-1.7 -I. -I/home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros -I/usr/src/gmock/gtest/include -isystem /usr/include/cairo -isystem /usr/include/glib-2.0 -isystem /usr/lib/x86_64-linux-gnu/glib-2.0/include -isystem /usr/include/pixman-1 -isystem /usr/include/freetype2 -isystem /usr/include/libpng12 -O3 -DNDEBUG    -DEIGEN_USE_NEW_STDVECTOR -DEIGEN_YES_I_KNOW_SPARSE_MODULE_IS_NOT_STABLE_YET -pthread -std=c++11 -fPIC  -Wall -Wpedantic -Werror=format-security -Werror=missing-braces -Werror=reorder -Werror=return-type -Werror=switch -Werror=uninitialized -O3 -DNDEBUG -MMD -MT CMakeFiles/cartographer_ros.dir/cartographer_ros/ros_map.cc.o -MF CMakeFiles/cartographer_ros.dir/cartographer_ros/ros_map.cc.o.d -o CMakeFiles/cartographer_ros.dir/cartographer_ros/ros_map.cc.o -c /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/ros_map.cc
In file included from /home/coldrain/catkin_ws/carto_ws/install_isolated/include/cartographer/mapping/2d/proto/map_limits.pb.h:33:0,
                 from /home/coldrain/catkin_ws/carto_ws/install_isolated/include/cartographer/mapping/2d/map_limits.h:26,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/ros_map.h:25,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/ros_map.cc:17:
/usr/local/include/cartographer/transform/proto/transform.pb.h: In member function ‘void cartographer::transform::proto::Rigid2d::set_allocated_translation(cartographer::transform::proto::Vector2d*)’:
/usr/local/include/cartographer/transform/proto/transform.pb.h:1900:21: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       translation = ::google::protobuf::internal::GetOwnedMessage(
                     ^
/usr/local/include/cartographer/transform/proto/transform.pb.h: In member function ‘void cartographer::transform::proto::Rigid3f::set_allocated_rotation(cartographer::transform::proto::Quaternionf*)’:
/usr/local/include/cartographer/transform/proto/transform.pb.h:2230:18: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       rotation = ::google::protobuf::internal::GetOwnedMessage(
                  ^
In file included from /usr/local/include/cartographer/mapping/proto/trajectory_node_data.pb.h:32:0,
                 from /usr/local/include/cartographer/mapping/trajectory_node.h:25,
                 from /home/coldrain/catkin_ws/carto_ws/install_isolated/include/cartographer/mapping/2d/map_limits.h:28,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/ros_map.h:25,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/ros_map.cc:17:
/usr/local/include/cartographer/sensor/proto/sensor.pb.h: In member function ‘void cartographer::sensor::proto::ImuData::set_allocated_linear_acceleration(cartographer::transform::proto::Vector3d*)’:
/usr/local/include/cartographer/sensor/proto/sensor.pb.h:1004:29: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       linear_acceleration = ::google::protobuf::internal::GetOwnedMessage(
                             ^
/usr/local/include/cartographer/sensor/proto/sensor.pb.h: In member function ‘void cartographer::sensor::proto::ImuData::set_allocated_angular_velocity(cartographer::transform::proto::Vector3d*)’:
/usr/local/include/cartographer/sensor/proto/sensor.pb.h:1054:26: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       angular_velocity = ::google::protobuf::internal::GetOwnedMessage(
                          ^
/usr/local/include/cartographer/sensor/proto/sensor.pb.h: In member function ‘void cartographer::sensor::proto::OdometryData::set_allocated_pose(cartographer::transform::proto::Rigid3d*)’:
/usr/local/include/cartographer/sensor/proto/sensor.pb.h:1132:14: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       pose = ::google::protobuf::internal::GetOwnedMessage(
              ^
/usr/local/include/cartographer/mapping/proto/sparse_pose_graph_options.pb.h: In member function ‘void cartographer::mapping::proto::SparsePoseGraphOptions::set_allocated_optimization_problem_options(cartographer::mapping::sparse_pose_graph::proto::OptimizationProblemOptions*)’:
/usr/local/include/cartographer/mapping/proto/sparse_pose_graph_options.pb.h:432:38: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       optimization_problem_options = ::google::protobuf::internal::GetOwnedMessage(
                                      ^
In file included from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/node_options.h:25:0,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/map_builder_bridge.h:29,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/node.h:31,
                 from /home/coldrain/catkin_ws/carto_ws/src/cartographer_ros/cartographer_ros/cartographer_ros/node.cc:17:
/usr/local/include/cartographer/mapping/proto/map_builder_options.pb.h: In member function ‘void cartographer::mapping::proto::MapBuilderOptions::set_allocated_sparse_pose_graph_options(cartographer::mapping::proto::SparsePoseGraphOptions*)’:
/usr/local/include/cartographer/mapping/proto/map_builder_options.pb.h:329:35: error: ‘GetOwnedMessage’ is not a member of ‘google::protobuf::internal’
       sparse_pose_graph_options = ::google::protobuf::internal::GetOwnedMessage(
                                   ^
ninja: build stopped: subcommand failed.
<== Failed to process package 'cartographer_ros':
  Command '['/home/coldrain/catkin_ws/carto_ws/install_isolated/env.sh', 'ninja', '-j8', '-l8']' returned non-zero exit status 1

Reproduce this error by running:
==> cd /home/coldrain/catkin_ws/carto_ws/build_isolated/cartographer_ros && /home/coldrain/catkin_ws/carto_ws/install_isolated/env.sh ninja -j8 -l8

Command failed, exiting.

It seems that the errors are caused by google::protobuf::internal,but I have installed protobuf,what could be the reason of this

sparse_pose_graph_options

This was renamed to pose_graph_options several months ago. Please make sure you are working on the latest versions and not mixing versions of cartographer and cartographer_ros.

@gaschler Thank you for your answer!I delete the whole catkin_ws folder and create a new one,then I follow the cartographer_ros Building & Installation instruction again,then the problem is solved.But i am still confused about the problem.What I did in the first time was that I updated the program by deleting the old source code and downloaded the new code,then I skipped the previous steps and directly do the last step.The result as you know,I got an error.So why would this happen?Why I can't update the program by just deleting the code of old version and replace it by the new one?


Keeps Deprecated Fields in Step and StepTemplate When Switching Versions

Changes
Prior to this PR, we have information loss when converting the
step and steptemplate from v1beta1 to v1. This PR preserves
those information by serializing steps and stepTemplate, saving
them in object annoations when converting from v1beta1 to v1, then
deserializing when converting back.

/kind bug
fix #6575
closes #4546
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] ~Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps~
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

Closes https://github.com/tektoncd/pipeline/issues/4546

/test all

/test all

cc @dibyom  @lbernick  This PR is ready for review, the last piece of https://github.com/tektoncd/pipeline/issues/6575

Is it possible to only do serialization of the deprecated fields? context: #6561 fyi @vdemeester

Good point! Updated.

/assign

/lgtm

/hold cancel

/retest

/retest


Octopus Home Mini readings only updating every 30 minutes

Describe the bug
The current consumption is only updating every 30 minutes.
# custom_components/octopus_energy/api_client/__init__.py
live_consumption_query = '''query {{
	smartMeterTelemetry(
    deviceId: "{device_id}"
    grouping: HALF_HOURLY
		start: "{period_from}"
		end: "{period_to}"
	) {{
    readAt
    consumption
		consumptionDelta
    demand
	}}
}}'''

Should the "grouping" be a smaller time frame?
Reproduction steps
Fresh HA installation with HACS and integrated into this repo.
Expected behaviour
Meter Readings should update every minute.
Tariff Code
E-1R-AGILE-24-04-03-A
Integration Version
11.2.0
Home Assistant Version
2024.6.3
Fresh Install?
Fresh install
Home Assistant Logs
N/A
Confirmation

[X] I confirm that I cannot find my solution within the documentation
[X] I confirm that I cannot find my solution within the FAQ

P.S, this is my first issue contribution here, so apologies if the formatting is incorrect.

Hello. Ther grouping has always been half hour and that's just consolidated the data into 30 minutes chunks to try and reduce the volume of data that is returned (1 minute intervals by the end of the day is far too much data).
This is also a duplicate of https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/914, so closing in favour of that. Please watch that issue for updates.

Ah yes — thank you. Correct on duplicate, so will follow that thread. Thanks for taking the time to reply.


🐛 Tray Popups can overlap

branch
stable
Architecture
x86_64
Description
When I click a tray icon on the swaybar, the new popup should replace the previous one, not cover it.
Reproduction steps
click bluetooth tray icon twice, or any other with a popup

Screenshots
No response
Logs
No response

Added a little helper based on flock that ensures we run a command only once and hope I added it in all relevant places.

-> reading the title of this issue again - did you mean that none of the floating windows should overlap or none of the same windows? I only added exclusivity for each, not all of them. Best would be you give it a try and check if it fits your needs.

exclusivity across all of them would be good
try opening bluetooth and network popup and then closing them again, rather annoying experience

added the once script to more places and made them mutex by killing holders of the lock. it doesn't yet include clipboard and zeit, but not sure we should include them in this.