How do you make all cookies secure?

Is there an option we can use to force all the cookies used to be secure (cookie flag)? Thanks

@mflorenciapereira Thanks for opening this issue! Yes there is a way to specify that all cookies should be secure!
Here's a list of all the cookie settings we support:
https://segment.com/docs/connections/sources/catalog/libraries/website/javascript/persistence/#cookie-settings
To set it, you can do something like:
analytics.load('writeKey', {
  cookie: {
    secure: true
  }
});

Does this actually work?  After setting this option, the analytics object in the console contains visible properties for cookies that include secure: true however, the cookies collection displayed by the browser seem to indicate that they are not, in fact, marked secure?

@t2farrell
We discovered the secure setting wasn’t being applied to all cookies and made a fix in https://github.com/segmentio/analytics-next/pull/577
I don’t think this has been released to the CDN hosted version of the sdk but we should be able to release it this week.

Great, thanks!

I had tried and wasn't working but now with the new version we can try again. Thanks!


Support transition_to_archive lifecycle policy

Describe the Feature
AWS recently added a new storage class to EFS called Archive
AWS Terraform provider will soon have a release that adds support to it (see PR)
Expected Behavior
Similar to the existing transition_to_ia input, add a new transition_to_archive input
Use Case
The new Archive EFS storage class has potential savings of about 50% of the unused data
Describe Ideal Solution
Add a new transition_to_archive input that supports the relevant values
Alternatives Considered
No response
Additional Context
No response

Hello @osterman
Can you please add this variable transition_to_archive.
this is a blocker for us to keep using your EFS module.
Thank you.


[BUG] Somehow the Rebuilding field inside volume.meta is set to true causing the volume to stuck in attaching/detaching loop

Describe the bug
Somehow the Rebuilding field inside volume.meta is set to true causing the volume to stuck in attaching/detaching loop
To Reproduce
Not sure how to reproduce yet.
There is a situation in which the volume is trying to attach. Replica process is running. However, when the engine process trying to connect to the replica, it hits this error 2022-06-22T17:35:28.031550367Z [pvc-126d40e2-7bff-4679-a310-e444e84df267-e-f7e3192f] 2022/06/22 17:35:28 Invalid state rebuilding for getting revision counter. It is indicating that the engine process pvc-126d40e2-7bff-4679-a310-e444e84df267-e-f7e3192f  of the volume cannot start (and thus repeatedly crashed)  because it failed to get the GetRevisionCounter of the replica which is in rebuilding state. The replica is the only replica of the volume.
So the question here is how did the replica get into this non-recoverable state
See here for more details
Expected behavior
The volume should not be stuck in attaching/detaching loop
Environment

Longhorn version: Longhorn 1.2.4

Workaround:
Since the volume has only 1 replica in this case, we can:

Scale down the workload. Longhorn volume will be detached
Wait for Longhorn volume to be detached
SSH into the node that has the replica
cd into the replica folder (for example, /longhorn/replicas/pvc-126d40e2-7bff-4679-a310-e444e84df267-1a5dc941)
Change theRebuilding field from true to false in the file volume.meta
Scale up the workload to attach the volume

from @joshimoo @innobead
Feel we should have something in our validation webhook to avoid that deletion if that image is referenced by any volume anyway.

This is probably related to the active engine image being deleted. I think there is a reference counter but it only includes the volume.spec / engine.spec stuff not necessary the still active old replica processes.

I agree we should prevent deletion of an engine image that is still required in any form or fashion.
The manual deletion mentioned above is just a hunch on the potential cause of the invalid meta stuff.

Did an investigation of the engineimage.status.refcount.
The refcount is caculated in engineimage controller (here and here). It counts the volumes whose spec or status are using this engineimage.
To prevent from the mistaken deletion of the engineimage, we can count the number of replicas and engines using the image rather than the number of volumes.

To prevent from the mistaken deletion of the engineimage, we can count the number of replicas and engines using the image rather than the number of volumes.

Sounds good to me.

To prevent from the mistaken deletion of the engineimage, we can count the number of replicas and engines using the image rather than the number of volumes.

Sounds good to me.
However, we are not sure if this issue is caused by deleting the engine image. I think we can create a new ticket for the engine image reference count instead. How do you think @innobead

To prevent from the mistaken deletion of the engineimage, we can count the number of replicas and engines using the image rather than the number of volumes.

Sounds good to me. However, we are not sure if this issue is caused by deleting the engine image. I think we can create a new ticket for the engine image reference count instead. How do you think @innobead

Agree. I will open a new issue handling the refcount part.

Agreed. BTW, if somehow still unable to figure out the root cause of this issue, we can revisit this after the new ref count introduction then see if this issue still happens.

After investigating the logs of the three problematic volumes, the three volumes all only have one replica and the volume.meta's rebuilding field are set to true. Because the information in the log are insufficient, I'm still not sure how the volumes ran into the error or how to reproduce it.
But for now we can make the volume attachment process is more rubust during attaching

If a volume with multiple replica and some replicas' volume.meta have illegal values, we can consider deleting these problematic replicas and avoid the failure.
But if a volume with only one replica, we can also fix the illegal values and then attach the volume. But, although we fix the illegal values, the data in the volume might be lost. Not sure if the fix is reasonable or helpful.

If a volume with multiple replica and some replicas' volume.meta have illegal values, we can consider deleting these problematic replicas and avoid the failure.

I agree with this point

@derekbit @PhanLe1010 I’m not sure how to reproduce this issue and what’s the test steps to verify this issue. Could you give some hints? Thank you!

@derekbit @PhanLe1010 I’m not sure how to reproduce this issue and what’s the test steps to verify this issue. Could you give some hints? Thank you!

@yangchiu Update the steps https://github.com/longhorn/longhorn/issues/4212#issuecomment-1212736221

@yangchiu Update the steps https://github.com/longhorn/longhorn/issues/4212#issuecomment-1212736221

@derekbit isn't the step 4 as below?

Scale up the workload and the volume is attached: the problematic replica marked error then delete, all replicas back to running. The volume will be not stuck in the attaching/detaching loop

4. workload and the volume is attached: the problematic replica marked error then delete, a new replica rebuilt, all replicas back to running. The volume will be not stuck in the attaching/detaching loop

Yes. The fix will result in the case as you said.
What I mentioned is the case before the fix.

Verified passed on master-head (longhorn-engine 18fa3c5) by following the test steps.

Reopening this ticket because we have not identified the root cause. The PR here mitigates the consequence when volume has multiple replicas. But some users with single replica are still hitting this
cc @innobead

Yeah, the implementation is for resilience and the root cause is not identified.

From other user's report
There was one Volume with 2 replicas (A,B)
A failed first and B tried to rebuild A by syncing files
During the syncing B failed to send the file, so both A, B failed
time=\"2023-05-01T11:51:44Z\" level=error msg=\"Sync agent gRPC server failed to rebuild replica/sync files: replica tcp://<IP_ADDRESS>:10000 failed to send file volume-snap-c849fe9d-f938-403d-b0fc-e0cdd8d31467.img to <IP_ADDRESS>:10021: failed to send file volume-snap-c849fe9d-f938-403d-b0fc-e0cdd8d31467.img to <IP_ADDRESS>:10021: rpc error: code = Unavailable desc = transport is closing\""
ERROR: 2023/05/01 11:51:44 grpc: server failed to encode response:  rpc error: code = Internal desc = grpc: error while marshaling: proto: Marshal called with nil"

However, engine chose A who was just failed when rebuilding as salvage replica
But since replica A state was still in rebuilding: true
So the engine failed to star with the error
time=\"2023-05-01T11:52:09Z\" level=info msg=\"Starting with replicas [\\\"tcp://<IP_ADDRESS>:10090\\\"]\""
time=\"2023-05-01T11:52:09Z\" level=info msg=\"Connecting to remote: <IP_ADDRESS>:10090\""
time=\"2023-05-01T11:52:09Z\" level=info msg=\"Opening: <IP_ADDRESS>:10090\""
time=\"2023-05-01T11:52:09Z\" level=warning msg=\"backend tcp://<IP_ADDRESS>:10090 is in the invalid state rebuilding\""

And the reason why it got selected might because in it's replica spec
it had healthAt="XXX" and failedAt="" which seemed like had not been updated correctly

Let's close this to prevent confusion about the reason for reopening. Let's use https://github.com/longhorn/longhorn/issues/6626 instead.


Kucoin GetAmountsAsync() ignores USDT

It might be the Kucoin API that is bugged, but I'm unable to see my current USDT balance. GetAmountsAsync() and GetAmountsAvailableToTradeAsync() only return all amounts, except for USDT.

If you can post the API json response that will help get this fixed, feel free to change all your balance values to 0.01 before posting.


Epic: s3 synchronisation follow ups

This Epic is a collection of follow ups for the S3 integration that was implemented in #340
Here and below, "remote storage" is a synonym of "s3": we implement the logic for s3, but keep in mind, that other types of storages are involved and may require support later, so the solutions should be generic enough.
no specific order is required for these:

[ ] #1557
[ ] #987
[ ] #1558
[ ] #1559
[x] #1049
[x] serialize and write remote timeline index to s3 periodically to mitigate potential s3 list issues (done via https://github.com/neondatabase/neon/issues/1493, see https://github.com/neondatabase/neon/compare/kb/remove-remote-storage-archiving?expand=1 branch for the details)
[x] migrate to rusoto s3 client from rust-s3, to prepeare for a future aws-sdk-rust migration when it matures ("official" but immature crate, it has a similar api to rusoto, more beneficial for our streaming api in many ways). Rusoto was picked as the new library to use: the PR Also there's an alternative branch for aws-sdk-s3.
[ ] sort out issues with pageserver checkpoints diverging local state from the remote one: #971
[x] #755 (intersects with @LizardWizzard  's migration work)
[x] https://github.com/zenithdb/zenith/pull/1080 implement pageserver's s3 prefixes to accomodate multiple pageservers on an s3 bucket
[x] some preparation for IT tests is needed: s3 parameters ergonomics, refactor pageserver CLI params (currently working on that in #871 )
[ ] In pageserver recovery we may push whole timeline dir as a checkpoint not only the necessary data. See: https://github.com/zenithdb/zenith/blob/adb0b3dadaf5a4f099e13adeaf8291c2d4b94550/pageserver/src/remote_storage/storage_sync.rs#L725. Also this data may contain garbage as we do not have checksums right now, and we probably should avoid checking every file in timeline dir on startup. Only files which were created after disk consistent lsn. Or we should just remove possibly incorrect files and rebuild the delta from wal
[x] ?? whatever else @LizardWizzard  finds needed and useful during his tenant migration work, bugs, consistency bugs and performance issues presumably
[x] randomly allocated work would be needed on refactoring and general logging, metrics and tracing span placement around the s3 sync code to make it observable

...... Some time later:

[x] add a CLI tool to manage archives in the remote storage (to help with the debugging) https://github.com/zenithdb/zenith/pull/1203
[ ] introduce GC support in s3 (defragmentation tasks) #999
[x] #1493
[x] deal with branches properly: remove them from pageserver and sync logic entirely, using ZTimelineId's instead, adjust pageserver's http api

Some open questions from #1163:

How to match SLO on new compute start? Do we always keep timeline assigned to at least one pageserver or we can completely unload it. In that case cold start can take quite a long time especially for large timelines. IMO we should keep timeline on at least one pageserver
Can we/should we interleave download with loading? I e download some index parts, and begin bootstrapping it before all the data is available locally. It might be quite hard to organize given our mix of sync/async parts of repository/remote storage sync.

This epic is quite old, and since new tasks continue to fall into the scope, we (@stepashka and I) decided to close this epic and create a new one and move relevant tasks to it. Further enhancement of s3 integration in pageserver happens in #1556.


Enhancement remove schema hash from webui

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
schema hash is useless now, we should remove this property.  For example, remove it from webui first.

Solution
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I want to work on this issue can you please assign it to me?

@imSanko Yes, I already done.

No need to change the webui, the webui can dynamically display the data including the header from the fe. Schema hash is not hard code on the web ui page.
I strongly suggest close this issue


Add react-figma platform

Adds react-figma as a new platform.
(there's a Webpack resolve issue though, where the react-native-web index.js is resolved before .figma.js, as .figma.js is not in the config extensions: I've opened an issue at https://github.com/react-figma/webpack-config , so probably worth waiting for that before merging this, as a node_module injection is needed for now).
Example library using this PR: https://github.com/macintoshhelper/react-figma-styled-components
resolves #142

Looks good, can you update the readme as well?

Thanks! I've added the installation command to the README. Should I add yoga-layout-prebuilt also, as that's a dependency required by react-figma?


Doc build failing with ascii_binder v0.1.5

docbuild has been failing since ascii_binder v0.1.5 came out.
bundler: failed to load command: ascii_binder (/opt/rubies/ruby-2.2.5/bin/ascii_binder)
LoadError: cannot load such file -- /opt/rubies/ruby-2.2.5/lib/ruby/gems/2.2.0/specifications/bin/ascii_binder
  /opt/rubies/ruby-2.2.5/bin/ascii_binder:23:in `load'
  /opt/rubies/ruby-2.2.5/bin/ascii_binder:23:in `<top (required)>’

I saw that issue locally and I just can't understand it :(  Our manageiq_docs builds are failing for the same reason.  cc @nhr

check the change @jvlcek needed to do in his PR: https://github.com/ManageIQ/manageiq_docs/pull/139/files
changing bundle exec ascii_binder to bundle exec asciibinder in the Rakefile

See this AsciiBinder issue: https://github.com/redhataccess/ascii_binder/issues/64
Is it possible that under Ruby 2.3.1 that the gemspec line referenced there is not picking up the symlink? The odd part is that ascii_binder is the actual file and asciibinder is the symlink to same...

Oh, I take it back - asciibinder is the actual file. So I bet that we need to fix the gemspec.


Check truncated

I don't know if we want this, but it seems to not impact performance, so might be nice to have.


Enhance monitoring of violated synonyms

As an operator
I would like to see which synonym from synonym dictionary was violated
so that I can adjust the synonym dictionary on a data-driven approach

@fmueller you prefer to see it in logs, db or you have in mind some other communication channel?

We use metrics library for metrics already. Just have a look into ApiViolationsController. We need histogram per synonym like we have already for each rule.

@fmueller , so the goal is to enable rule back, make sure that it always returns no violations and instead update metrics, right?


Feedback

:wave:! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to master since the assignment started. Your teacher can see this too.

Notes for teachers

Use this PR to leave feedback. Here are some tips:

Click the Files changed tab to see all of the changes pushed to master since the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.
Click the Commits tab to see the commits pushed to master. Click a commit to see specific changes.
If you turned on autograding, then click the Checks tab to see the results.
This page is an overview. It shows commits, line comments, and general comments. You can leave a general comment below.

For more information about this pull request, read “Leaving assignment feedback in GitHub”.

Homework 1 Grading Rubric
Student: ratna
Initial Grade: 7/7
Point Breakdown

Task
Possible Points
Earned Points
Comments & Feedback

Read the CSV file into a pandas dataframe.
1
1
Nailed it!

Add columns to the dataframe containing: 1) The total (East + West) bicycle count, 2) The hour of the day, 3) The year
3
3
Great!

Create a dataframe with the subset of data from the year 2016
1
1
Superb!

Use pandas + matplotlib to plot the counts by hour. (i.e. hour of the day on the x-axis, total daily counts on the y-axis)
1
1
Nice! In the future, think carefully about your variable names -- "data2" is a tough one to figure out!

Use pandas to determine what is (on average) the busiest hour of the day
1
1
Cool!

General Feedback
n/a
n/a
Nice work getting to know Pandas!

Graded by: Tony Cannistra<EMAIL_ADDRESS>


Automatically provide global using

The new C# 10 feature lights up automatically via the provided item group with the Import element.
There's talk of a future opt-out (or opt-in?) MSBuild property to control these global usings, but the provided approach will work and integrate seamlessly with whatever the project/SDK configures.
For reference, see https://github.com/dotnet/aspnetcore/issues/32451.
Note we provide the "legacy" MSBuild xmlns attribute so as not to break downlevel IDEs or legacy project files that happen to reference this package.
Fixes #515

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

We've let this one hang out seeing where the .net world landed on global usings. Seems devs are handling the imports on their own and  packages aren't including them. Plus tooling is becoming more common to assist those not familiar with the syntax.
I'm gonna close this PR. Appreciate the thought, but just not a feature we want to push.


Clicking an AccordionTrigger within a <form> triggers the form submission

Bug report
Current Behavior
Clicking a trigger submits the form because button type is submit by default in browsers. https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button#attr-type
Expected behavior
Clicking a trigger should probably not submit the form and should just open/close the accordion panel.

Reproducible example
Repro Example
Suggested solution
Defaulting the type attribute to button on the AccordionTrigger would probably solve the issue. This might apply to other trigger primitives as well.

Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
Accordion
0.1.5

React
n/a

Browser

Assistive tech

Node
n/a

npm/yarn

Operating System

@jjenzz I just ran into this issue with the Tooltip component inside a form triggering a submit.
What's the special reason why type=button isn't used for the Tooltip?

i can't recall now exactly, do you @benoitgrelard? perhaps it had something to do with the fact that tooltips wrap actionable things, and radix likely wouldn't want to control what your action is. however, i'm not sure why consumers couldn't set type="submit" explicitly when that is the requirement.

@jjenzz @benoitgrelard this is not about setting type="submit", but preventing the tooltip trigger from becoming a form submit button.
The tooltip trigger button currently has no explicit type. When nested inside a <form> browsers will implicitly assume type="submit", which will trigger a form submit.

@slavanga yes, i understood that... i was proposing that maybe radix could set type="button" by default on the tooltip trigger and consumers could set it to "submit" explicitly when that is their requirement.
the same can be said in reverse tho—if you don't want your tooltip button to submit then add type="button" to the trigger. the bigger question tho is why all other comps are type="button" by default except for tooltip.

@jjenzz ok, thanks for clarifying.
overriding works, but it would be optimal if all triggers had the same button type if possible.

Some background: https://github.com/radix-ui/primitives/pull/1011
https://github.com/radix-ui/primitives/pull/1011#pullrequestreview-822604542

Trying to rationalise it, I feel like Tooltip.Trigger is the only trigger whose functionality isn't entirely relying on a click/pointerdown action. That makes me think that all the other triggers where we do type="button" have no business being replaced with other HTML elements as it wouldn't really be semantic anymore.

oh looks, like we explained further in the code:

does that help @slavanga ? if someone did <Tooltip.Trigger asChild>foo</Tooltip.Trigger> then the type attribute would be telling the browser that the MIME type for the linked resource is "button" which is invalid.

Yep, it's just that generally tooltip can be added to a lot more types, even input, so we wouldn't want input type="button" either.

@jjenzz @benoitgrelard thank you both for the explanations. that makes perfect sense.
maybe this info should be added to the tooltip docs.

Just spent longer than I'd like debugging an issue caused by the tooltip triggering a form. +1 for adding something in the docs about this.


BE-90: Add POST PUT DELETE to /api/artworks

Must consider both DB and Stripe.
DELETE most be a logical delete.

 BE-90: Expandir api/artworks para manejar post, put, delete tomando en cuenta stripe


Add parallel-letter-frequency exercise

cherry-picked from #29

Could do with more tests like https://github.com/exercism/xhaskell/blob/master/parallel-letter-frequency/parallel-letter-frequency_test.hs.
Might be easier to merge this, then do the extra tests as it's own PR.

Yeah, that's a good call.
I'd rather not change the code, since I just cherry-picked it from someone else. Formatting is one thing, additional code should probably be separate.

I'll merge and whoever wants to can add extra tests.


How can two devices share the same exported my library.json

Hello, I have a question, I have zotero installed on both devices, and the data has been synced, export a my library .JSON on one device, and create the corresponding MD notes, how do I do it on the other device What about syncing notes and annotations using my library exported on the first device?

I'm don't understand the question. Are you sure it's one for this plugin and not hans/obsidian-citation-plugin?


Non-generic adsr_live() and adsr_fixed(). Generic versions in prelude.rs.

As suggested by @harudagondi in #17, I created non-generic versions of adsr_live() and adsr_fixed() and placed them in hacker.rs and hacker32.rs. I moved the generic versions to prelude.rs.

I ran cargo fmt and it did not report anything.

Thanks @harudagondi. I'm running it now.

You accidentally added the .idea folder. You should undo your last commit

Yikes, sorry about that. I've removed .idea. Thanks for pointing it out!


How to use this inf file?

Hello,
Download the file.
In Windows Explorer, right-click the INF file name. A shortcut menu will appear.
Click Install.

I created PR #10 to add this info to the README

The "install" option doesn't show for me:
I've tried holding shift as well; still no dice.
I'm guessing since no one else has reported this, it's due to some change in my registry or something. Will post here if I find the solution.

I tried this alternative way to install the inf file: https://stackoverflow.com/a/41233871/2441655
However I get this error:
C:\Root\Apps\System\CMD>pnputil -i -a WindowsTerminalHere.inf
Microsoft PnP Utility

Processing inf :            WindowsTerminalHere.inf
Adding the driver package failed : The third-party INF does not contain digital signature information.

Total attempted:              1
Number successfully imported: 0

Okay, I found the shell command that works:
rundll32 syssetup,SetupInfObjectInstallAction DefaultInstall 128 .\WindowsTerminalHere.inf

Might be worth adding this to the readme for anyone else whose Windows Explorer does not show the "Install" option. (or just for people who prefer terminal-based installation)


File Not Found using Different Method

I've put i my token (I think, I'm getting 2 different tokens, from Network and Local Files. Both don't work, but local files shows nothing happening, Network at least shows soemthing going wrong). I have set up the json file with everything, and checked to make sure it's correct json using a verifier.
File Not Found using Different Method Exception ignored in thread started by: <bound method GatewayServer._response_loop of <discum.gateway.gateway.GatewayServer object at 0x000002775D4A3810>> Traceback (most recent call last):   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\gateway\gateway.py", line 299, in _response_loop     func(resp)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 748, in on_message     slashfus = threading.Timer(10.0,waifu_roll,args=[slashchannel,slashget,slashguild]) UnboundLocalError: local variable 'slashget' referenced before assignment Exception ignored in thread started by: <bound method GatewayServer._response_loop of <discum.gateway.gateway.GatewayServer object at 0x000002775D4A3810>> Traceback (most recent call last):   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\gateway\gateway.py", line 299, in _response_loop     func(resp)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 763, in on_message     msg = get_server_settings(gid,matched_channel)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 180, in get_server_settings     FsResults = bot.filterSearchResults(FsMsgs)   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\discum.py", line 313, in filterSearchResults     return imports.Messages(self.discord, self.s, self.log).filterSearchResults(searchResponse)   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\messages\messages.py", line 243, in filterSearchResults     jsonresponse = searchResponse.json()['messages'] KeyError: 'messages'

Also have updated Discum and Python to latest versions

just find a copy of your channel settings and place them in the channeldata folder

How do I do this? Just type $settings, and copy paste the message as a txt file? What do I name the file? The ID.txt?

{channelid}.txt be sure to include the bold "*"

I did that now, and it was able to read the file. Now im getting some other errors:
Mentions discum so not sure if that is the issue, but it shouldn't as I installed the latest version.
Reading from File for channel<PHONE_NUMBER>88007681 Exception ignored in thread started by: <bound method GatewayServer._response_loop of <discum.gateway.gateway.GatewayServer object at 0x000001D886BD35B0>> Traceback (most recent call last):   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\gateway\gateway.py", line 299, in _response_loop     func(resp)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 764, in on_message     c_settings = parse_settings_message(msg)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 226, in parse_settings_message     settings['prefix'] = val_parse(settings_p[0])[0] IndexError: list index out of range Exception ignored in thread started by: <bound method GatewayServer._response_loop of <discum.gateway.gateway.GatewayServer object at 0x000001D886BD35B0>> Traceback (most recent call last):   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\gateway\gateway.py", line 299, in _response_loop     func(resp)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 748, in on_message     slashfus = threading.Timer(10.0,waifu_roll,args=[slashchannel,slashget,slashguild]) UnboundLocalError: local variable 'slashget' referenced before assignment

also just checking, is this what is meant to be inside the txt file:
`· Prefix: $ ($prefix)
· Lang: en ($lang)
· Claim reset: every 180 min. ($setclaim)
· Exact minute of the reset: xx:48 ($setinterval)
· Reset shifted: by +0 min. ($shifthour)
· Rolls per hour: 8 ($setrolls)
· Time before the claim reaction expires: 45 sec. ($settimer)
· Spawn rarity multiplicator for already claimed characters: 1 ($setrare)
· Server game mode: 2 ($gamemode)
· $servlimroul = 27054 $wa, 26872 $ha, 18081 $wg, 15112 $hg
· This channel instance: 1 ($channelinstance)
· Slash commands: enabled ($toggleslash)
· Ranking: enabled ($toggleclaimrank/$togglelikerank)
· Ranks displayed during rolls: disabled ($togglerolls)
· Hentai series: disabled ($togglehentai)
· Disturbing imagery series: disabled ($toggledisturbing)
· Rolls sniping: 0 ($togglesnipe)
· Kakera sniping: 0 ($togglekakerasnipe)
· Limit of characters per harem: 8100 ($haremlimit)
· Reacts: for public wishes only ($togglereact)
· Custom reactions: yes ($claimreact list)
· Kakera trading: enabled ($togglekakeratrade)
· Kakera calculation: claims and likes ranks (and number of claimed characters) ($togglekakeraclaim/$togglekakeralike)
· Kakera value displayed during rolls: enabled ($togglekakerarolls)
· $kakeraloot wishprotect: enabled ($togglewishprotect)`

You forgot the bold *'s

Oh i didnt realise when you copy paste from discord it removes the bold *'s. I put all the *'s in, but Im still getting the same error as before
439215273288007681.txt

It looks like it is working now, hasn't found anyone to claim yet though so can't see if that works.
Also what needs to be commented out for this?
use_emoji - This setting only works if you change the Mudaebot.py code by uncommenting out the line (Custom emojis only) e.g. "<:emoji_name:795############214>"

I'm now getting another strange error. Bot was working fine, but now it doesn't work one one of my accounts. If I change the token to an alt, it works, but the original token for the original account isn't working. I tried copying the token again from discord and it doesn't fix it.
Exception ignored in thread started by: <bound method GatewayServer._response_loop of <discum.gateway.gateway.GatewayServer object at 0x0000021FAC7536E0>> Traceback (most recent call last):   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\gateway\gateway.py", line 299, in _response_loop     func(resp)   File "C:\Users\erago\Documents\!Games\MudaeAutoBot-main\MudaeAutoBot.py", line 755, in on_message     user = bot.gateway.session.user   File "C:\Users\erago\AppData\Roaming\Python\Python310\site-packages\discum\gateway\session.py", line 42, in user     return self.settings_ready['user'] KeyError: 'user'

Duplicate issue refer to the one that is still open and ongoing


Labels

I am suggesting you add labels to the X and Y axis, novices should be able to understand what the charts are measuring

I agree with @Shegofams anyone who visits the repo should be able to understand what the charts are for.


GEP: Client Certificate Validation for TLS terminating at the Gateway Listener

/kind gep
Fixes https://github.com/kubernetes-sigs/gateway-api/issues/91

will continue adding more content in the GEP once Ive received broad consensus on the goals & non goals

Thanks @arkodg! Will give @shaneutt and @youngnick a bit more time to review this one, but don't think there's anything controversial here.
/approve

@robscott @shaneutt @youngnick
Hey guys, since this is no more than a simple (I believe agreed to) problem statement that has been stagnant for a couple of months, can we either get this merged now or otherwise update it with an API proposal so we can start discussing the solution?
@argkodg do you have an API for this that you are waiting to propose?

Thanks for the ping on this one @frankbu! I don't think there's anything controversial in the GEP at this point. I've already approved, would like @shaneutt or @youngnick to sign off on this though.
/release-note-none

Yeah, this is pretty tightly scoped as written, so I'm supportive.
/lgtm


Added the AI-Powered Super-Resolution and Image Restoration model

Description
I have added the AI-Powered Super-Resolution and Image Restoration Model
This project aims to create an AI-Powered Image Restoration Tool that transforms low-quality historical photos by upscaling, denoising, and adding realistic color. Using deep learning, it revives old images with stunning clarity and detail, preserving memories in a vibrant, lifelike form. Perfect for heritage conservation and digital archiving!
Fixes #1614
Type of change

[x] Added a new machine learning frameworks, libraries or software.
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

@sanjay-kv
I have done the work. Kindly check it out and please give level 3

@sanjay-kv  Can you please give Level 3 . It will grateful

@sanjay-kv
Kindly give Level 3 . It will be an honor


Enable the use of secrets in the password configuration setting

Hi,
I've checked the documentation and I don't see a reference to using HA secrets file for retrieving the encryption password. Unless this is already possible, could you consider this feature request for future evolutions of the add-on?
thanks a lot for such an awesome add-on!!

Currently there is no support for this. But I'm curious, since this is stored inside the addon, what is the benefit of having it read from the secrets file ? It's not exposed in the configuration yaml file.

I must say I can't compare how secure the current approach is versus having the secret file in the secrets.yaml. I asked for this solely because have been a Google Backup user for a long time, and that add-on uses the secrets.yaml file.

I don't think either is really more secure. Both store the password in clear text. The addon stores it inside it's own container. the Secrets.yaml exists on the host itself also passwords in clear text.


[ClangImporter] Add a comment to 'recursivelySubstituteBaseType'

As Doug pointed out in #14682, this isn't the normal way to do a type substitution. However, all the usual methods depend on the conformance's generic signature being fully filled in. Add a comment because, well, I don't have a better answer.

@swift-ci Please smoke test


Bigger bundle size compared to solid-headless

I'm currently using solid-headless for a simple DropdownMenu. It is very basic, it doesn't need floating-ui dep as I can position the menu myself with CSS.
Tried comparing the bundle size of it and kobalte's equivalent, because I might want to switch to using kobalte in the future (better documented, maintained, etc)
But the DropdownMenu from kobalte appears to be ~8x times larger in bungle size. I was using https://bundlejs.com which supports tree-shaking.

solid-headless build

export { Menu, MenuItem, Popover, PopoverButton, PopoverPanel } from 'solid-headless'
// Bundle size is 22.92KB -> 5.32KB (gzip)

koblate build

export { DropdownMenu } from "@kobalte/core"
// Bundle size is 155.96KB -> 40.95KB (gzip)

(this is with adding solid-js and solid-primitives to external as I already use them)
Not exactly sure what causes that difference honestly. The @floating-ui/core dep is adding only ~3KB gzip, @kobalte/utils is also ~3KB gzip, the @internationalized thing is barely adding anything. So the biggest chunk must be in the main package.
Is there a way to improve tree-shaking? I get that kobalte may be stacked with more features, but not everyone will use them.

Hi, i'm not sure what causes this.
I tried on a fresh solidjs project and only import the dropdown menu, in the production build there is only the code needed for the menu but the bundle is about 115kb 😕 (same when using select, or others "floating" components).

Part of it might be the legal comments getting left in by default by vite.
Configuring esbuild to remove them reduces the bundle size of solid + DropdownMenu to ~34kb gzipped.
import analyze from "rollup-plugin-analyzer";
import { defineConfig } from "vite";
import solidPlugin from "vite-plugin-solid";

export default defineConfig({
  plugins: [solidPlugin(), analyze({ summaryOnly: true })],
  server: {
    port: 3000,
  },
  esbuild: {
    legalComments: "none",
  },
  build: {
    target: "esnext",
    // minify: false,
  },
});

Still quite a bit larger than solid-headless though, not sure how solid-headless gets away with much less code, would have to dig into their implementation 🤷♂.
Here's some of the output from rollup-plugin-analyzer in case it's helpful:

I'm working on a refactor of the way components are exported. Currently when you import DropdownMenu you get all parts even if not used.

Tree shaking is better in v0.3.0

@thetarnav can i close the issue ?

You decide.
I believe that the tree-shakeability and overall bundle size could be still improved.
Based on bundlejs.com the bundlesize actually increased, but not sure how reliable this is.
If you don't plan on reducing it now then there is no point in keeping the issue.

You decide. I believe that the tree-shakeability and overall bundle size could be still improved. Based on bundlejs.com the bundlesize actually increased, but not sure how reliable this is. If you don't plan on reducing it now then there is no point in keeping the issue.

The overall size of the lib is larger because the code related to calendar/datepicker that i'm working on was added.
I don't think tree shaking is working correctly on bundlejs.com, on your example components like Tabs or RadioGroup is in the output, which is not the case when i build a real solidjs project.


Splash page for origin UI

@CannonLock is working to finish up an initial landing page of the origin UI once the user has set the password.

https://github.com/PelicanPlatform/pelican/pull/105


Dutch Translation

added dutch translation... again

The previous PR had more words translated and several files modified. What happened?

Ah sorry @shprink I am not that experienced... Doing it again hahaha


Java2docfx phun

The default version of Java in the devops images is Java 11. Java2docfx needs to have Java 17 be the default version in order to process Spring library docs due to the latest spring boot requiring a higher version.
This has been tested on the following pipelines using SetDevVersion (publishes a nightly version to the dev feed and exercises this docs gen code path). The spring pipeline needs the updated version, the keyvault pipeline was a sanity check to ensure it continues to run with the updated version.

spring
keyvault

I'm not waiting for the last core pipeline test run to finish, it doesn't test these changes which have already been tested in two different pipelines.


GH-2: Update ADR for SSG frontend, selecting Astro as the solution

this close #2
Astro was chosen for its simplicity, excellent DX, and minimal client-side JavaScript by default. The decision also highlights its strengths in easy data fetching and tracker injection, with minor cons like learning Astro-specific syntax. This update refines phrasing and outlines the evaluation process for clarity.

this close #18
this close #21


Leet code site link contributed

Can you check the contribution?
Thank

Can you please check the contribution?
Thank you


feature/#164_2 - pantries are now available

For the moment, in order to copy products to a pantry:

go to a product list
click on the "copy" button (bottom bar)
go to the pantry
click on the "paste" button (bottom bar)

New files:

pantry.dart: a pantry (name, color, icon, list of barcodes with quantity and dates)
pantry_button.dart: a button for a pantry, with the corresponding color, icon, name and shape
pantry_dialog_helper.dart: a dialog helper for pantries
pantry_list_page.dart: a page where all the pantries are displayed as previews
pantry_page.dart: a page for one pantry where we can change all the data
pantry_preview.dart: a preview button for a pantry, with its N first products
product_list_view_helper.dart: a preview button for a list of products; used to be private in ProductListPreview

Impacted files:

dao_product_list.dart: new method getBarcodes
home_page.dart: added a section for pantries; minor side effect fix
product_list.dart: added generic method getTintedIcon
product_list_preview.dart: moved code to new class ProductListPreviewHelper
smooth_product_card_found.dart: square'd product thumbnail
user_preferences.dart: rough management of pantries get/put


Updates all Enums to UPPER_CASE across the code base

Resolves #605
Fixes all enums to have the uppercase standard and updates all calls + tests to uppercase as well.

Excellent! Thanks for this. As it looks like we can't check with ruff, we should put checking in StrictEnum and SubsetEnum. Please can you replace the definition of StrictEnum and SubsetEnum in ophyd_async/core/_utils.py with the following:
class StrictEnumMeta(EnumMeta):
    def __new__(metacls, *args, **kwargs):
        ret = super().__new__(metacls, *args, **kwargs)
        bad_names = [x.name for x in ret if not x.name.isupper()]  # type: ignore
        if bad_names:
            raise TypeError(f"Names {bad_names} should be uppercase")
        return ret

class StrictEnum(str, Enum, metaclass=StrictEnumMeta):
    """All members should exist in the Backend, and there will be no extras"""

class SubsetEnumMeta(StrictEnumMeta):
    def __call__(self, value, *args, **kwargs):  # type: ignore
        if isinstance(value, str) and not isinstance(value, self):
            return value
        return super().__call__(value, *args, **kwargs)

class SubsetEnum(StrictEnum, metaclass=SubsetEnumMeta):
    """All members should exist in the Backend, but there may be extras"""

Hopefully the tests will still pass after this and we can merge...


Issues using the keyboard prefab on PC

Overview
When using a uGUI text field and the virtual keyboard prefab on PC, both the virtual keyboard and OS keyboard shows up in the application.
HoloLens is unaffected.
Expected Behavior
A single keyboard
Actual Behavior
OS and keyboard prefab shows up
Steps to reproduce

Open keyboard test scene
Play in editor
Attempt to write in text field

Unity Editor Version
2017.2.0p1 MRTP4
Mixed Reality Toolkit Release Version
db59a1c7fa0c0c463466028c5f3e70088bbfda52

I am experiencing the same issue with my app.
Unity documentation say:
Windows Store Apps: On Universal Windows 10 Apps the touch screen keyboard is only supported in XAML based applications with touch input support.
My app is with Build Type D3D and not xaml so this is the first strange thing (did they intend .Net scripting backend?)
However since the keyboard is shown we can try to disable the touch input support.
To do this we can define a new BaseInput used by the BaseInputModule where touchSupported is overrided and setted to false.
BaseInputModule has a protected property m_InputOverride that we can use to tell unity to use the new BaseInput.
That said, as a temporary workaround, I made the attached script that can be added to the EventSystem game object of the scene or to another game object (in the latter case you must set the field BaseInputModule with the current input module).
In my case it seems to work (the native keyboard is not shown).
Can someone try this workaround so we can be more confident using it?.
Thank you
HideUnityTouchScreenKeyboard.zip

What changed in the last few days? The build on 2/06/18 did not bring up the OS keyboard now it does. Also in my project at least it opens the keyboard prefab, then opens the OS keyboard, then closes the OS keyboard and opens it again.

@265lutab, I don't know.
Can you try the workaround?
Thanks

The build I made today does not show the system keyboard. I'm not aware of any changes I've made that would effect it. I don't know why it is now working correctly.

In fact the builds I made that displayed the OS keyboard yesterday now don't. My guess is something in the mixed reality portal that was changed enabling the keyboards for a little while.

Is anyone else still having the problem?

@ivan2007 I did try bringing your script into the project anyways to see what would happen. I'm getting this error
"Assets/Scripts/HideUnityTouchScreenKeyboard/HideUnityTouchScreenKeyboard.cs(14,49): error CS1644: Feature `expression bodied members' cannot be used because it is not part of the C# 4.0 language specification" I am using IL2CPP for my project.

I have the Scripting Runtime Version set to 3.5 and the Api Compatibility Level set to 4.6. (I'm guessing the error is from the Scripting Runtime Version being 3.5 instead of 4.6.

You can change the body of the property
public  bool touchSupported { get { return false; } }

Thanks I did start having the keyboard error again so I used your script. It did mostly work for me. I'm not getting the system keyboard 90% of the time, but I did have it come up once. I haven't been able to repeat it so I'm not sure what caused it to be shown. Do you know if there is a call that can be used to close the system keyboard as a backup if it opens for some users?

@265lutab,
TouchScreenKeyboard has a field 'active' but it is not static.
The keyboard is opened by the InputField as you can see here (code from InputField):
private void ActivateInputFieldInternal()
{
if ((UnityEngine.Object) EventSystem.current == (UnityEngine.Object) null)
return;
if ((UnityEngine.Object) EventSystem.current.currentSelectedGameObject != (UnityEngine.Object) this.gameObject)
EventSystem.current.SetSelectedGameObject(this.gameObject);
if (TouchScreenKeyboard.isSupported)
{
if (this.input.touchSupported)
TouchScreenKeyboard.hideInput = this.shouldHideMobileInput;
this.m_Keyboard = this.inputType != InputField.InputType.Password ? TouchScreenKeyboard.Open(this.m_Text, this.keyboardType, this.inputType == InputField.InputType.AutoCorrect, this.multiLine) : TouchScreenKeyboard.Open(this.m_Text, this.keyboardType, false, this.multiLine, true);
this.MoveTextEnd(false);
}
else
{
this.input.imeCompositionMode = IMECompositionMode.On;
this.OnFocus();
}
this.m_AllowInput = true;
this.m_OriginalText = this.text;
this.m_WasCanceled = false;
this.SetCaretVisible();
this.UpdateLabel();
}
m_Keyboard is protected.
You can make a script to access this field with reflection.
During my test I noted that the script always work if you haven't set a breakpoint in the code (debug configuration)
So when you build with Master or Release configuration in my case the script works.

@ivan2007 I tested the reflection code and it's not working on Hololens. As soon as an KeyboardInputField is focused, the console starts spamming NullReferenceExceptions. I read somewhere, that the Hololens requires the InputSystem to be the StandaloneInputModule.

@Alexees
Thanks for your test
I don't know what you can do with HoloLens (I can't do any test on it).
I saw that exception also without the script during the test with HMD.
If you find any workaround please share here.

When the HoloLens gets the rs4 update this will no longer be a problem.  Also reflection is probably not the best solution.
Simplest fix is to probably just check if the keyboard is opaque and return.

Hi Stephen,
I am sorry but I don't understand if your fix is also for HMD and where to check if the keyboard is opaque.
However I have seen that the keyboard already uses a custom InputField (SliderInputField).
If we make an override of LateUpdate like this:
       protected override void LateUpdate()         {             base.LateUpdate();             if (m_Keyboard != null )             {                 m_Keyboard.active = false;                 m_Keyboard = null;             }         }
the problem seems fixed (no second keyboard appear and no exception is thrown).
LateUpdate calls ActivateInputFieldInternal and opens the keyboard.
The script is no more necessary.
In my case I see that the keyboard is not null only one time
Tested with Unity 2017.3.1p1 and HMD.

Editing my response above.
I meant to check if the device is opaque via holographic settings API.

@ivan2007 I was wrong about your reflection fix, it could work on Hololens, the NullRef was caused by something else.

@Alexees,
to avoid reflection now I am using this (inside SliderInputField):
protected override void LateUpdate()         {             try             {                 base.LateUpdate();             }             catch { }             if (m_Keyboard != null)             {                 m_Keyboard.active = false;                 m_Keyboard = null;             }         }

@ivan2007 I tried that on my KeyboardInputFields and it does not work when built to MR, the Keyboard still shows up (Unity 2017.3.1p3)

@Alexees, did you try it with 'SliderInputField' of MRTK?

@ivan2007 That class is doing nothing on its own, not even opening the 3D Unity Keyboard, or am I missing something?

@Alexees, SliderInputField is the InputField used by MRTK keyboard prefab.
Unity initializes the system keyboard inside LateUpdate of InputField.
My code overrides LateUpdate, you must put it inside SliderInputField.
I use it with HMD and in my case it works.

@ivan2007 I'm really at a loss here and don't understand what the difference is. I've put your code into both, KeyboardInputField and SliderInputField, though I wasn't using the latter. I can confirm by logging that the keyboard is not null, deactivated and then set to null. Even though this code executes every time I gaze at the InputField and LateUpdate captures the keyboard it still shows up unfazed. Doesn't matter if I build a regular or debug build from Unity and then run either of those in VS in Release or Debug configuration. The behavior is always the same. It only happens with the real device though, it does not when using the simulator.
I really do not know what to do here.

@Alexees,
I am using Unity 2017.3.1p1, MRTK 2<IP_ADDRESS> and HMD.
Which versions are you using?

@Alexees,
I upgraded Unity to 2017.4.1f1 and now I have your same problem.
Waiting for a real solution I made a custom SliderInputField where I have completely removed the code that instantiate the TouchScreenKeyboard.
I used reflection to access private field, properties and methods so this is not a solution but is only being temporary workaround.
You can find the code in the attachment. Hope this helps.
SliderInputField.zip

@ivan2007 thanks for confirmation and the code you're using. I should have mentioned that I was on 2017.3.1p3. I kinda did the same thing, reverting to a text field and emulating the Behaviour of an InputField.

@StephenHodgson is this still an issue? can this be handled from the Toolkit side, or is that basically a bug or feature Unity needs to fix or expose to be disablable?

@ivan The fix which you have mentioned will work in Keyboardinputfield script as well ?

@swatvel I'm using MRTK 2<IP_ADDRESS>, Unity 2017.4.11f1 and this  SliderInputField.zip
In the Keyboard class I commented:

InputField.ActivateInputField() (PresentKeyboard() function)
InputField.keyboardType = (TouchScreenKeyboardType)(-1); (Awake function)
With these two lines there were NullReferenceExceptions building with VS

Bulk closing older HTK bug. Please reopen if you can repro this issue in MRTK v2.
Also we moved on to use OSK instead of the keyboard prefab in v2.
Thanks!
Yoyo


Add optional data parameter to HttpResponset.close()

This issue was originally filed by @kaendfinger

There should be an optional data parameter on HttpResponse.close that calls response.write before closing.
response.close("Hello World");
should be equal to
response.write("Hello World");
response.close();

Alternatives accepted. shelf package or cascades.


Timed out

Just trying to find a toast notification library since win10toast doesnt work on win 11 anymore, trying out the example code and it always prints out "ToastDismissalReason.TIMED_OUT".
Not sure whether this is the new win 11 update causing this issue or what but please let me know if you find a fix.
code:
from win11toast import toast

toast('Hello Python🐍')

You can suppress the stdout output of the toast with the code below.
from win11toast import toast

def empty_func(args):
    pass

toast('Hello Python🐍', on_dismissed=empty_func)

https://github.com/GitHub30/win11toast/discussions/4

大佬，你好，我也出现了这个问题，麻烦问一下现在有解决不，我WIN11的版本号是：21H2
代码如下：
from win11toast import toast

def empty_func(args):
    print('dismissed with args:', args)

if __name__ == '__main__':
    toast('Hello Python🐍', on_dismissed=empty_func)

输出如下：
dismissed with args: ToastDismissalReason.TIMED_OUT

我知道了，大佬，是因为设置了WIN11的专注助手，把专注助手关掉就可以通知了。

You can suppress timeout with notify function
from win11toast import notify

notify('Hello Python🐍')

I had the same issue, than I realized my OS was in 'Do not disturb' mode when i deactivated it worked, if might help someone

If its on Windows 11 see if notification do not disturb is turned on.


Project name edition

Is your feature request related to a problem? Please describe.
There is no way to rename a project.
Describe the solution you'd like
Permit project edition to rename it

I am taking this one, I want to learn a bit more about the update using Firebase, I'll do it this week if I find the time.


GradientExplainer has no expected values: how to interpret it.

I use GradientExplainer to explain a Recurrent Neural Net with a continuous target. I used KernelExplainer and switched to GradientExplainer as I expect it to be a better approximation of SHAP values.
However, it has no expected_values attr. The code is commented out.
In the documentation, it is usually written:

For models with a single output this returns a matrix of SHAP values (# samples x # features). Each row sums to the difference between the model output for that sample and the expected value of the model output (which is stored as expected_value attribute of the explainer). For models with vector outputs, this returns a list of such matrices, one for each output.

So I'm not sure how to interpret the values without the expectation. If a fix is required, can I somehow calculate it from the gradients?
Thanks, @slundberg ! Amazing package!

I had this issue as well, I ended up pulling the code that calculates the expected value for deep_tf.py. I put the following in gradient.py
        if (hasattr(self.data, '__call__')):
            self.expected_value = None
        else:
            if self.data[0].shape[0] > 5000:
                warnings.warn("You have provided over 5k background samples! For better performance consider using smaller random sample.")
            if not tf.executing_eagerly():
                self.expected_value = self.run(self.model_output, self.model_inputs, self.data).mean(0)
            else:
                if type(self.model)is tuple:
                    sel.fModel(cnn.inputs, cnn.get_layer(theNameYouWant).outputs)
                self.expected_value = tf.reduce_mean(self.model(self.data), 0)

It make my force plots have nearly the correct prediction, they're still off by about 0.01, but I can live with that.

Thanks, will give that a shot.

if (hasattr(self.data, '__call__')):
            self.expected_value = None
        else:
            if self.data[0].shape[0] > 5000:
                warnings.warn("You have provided over 5k background samples! For better performance consider using smaller random sample.")
            if not tf.executing_eagerly():
                self.expected_value = self.run(self.model_output, self.model_inputs, self.data).mean(0)
            else:
                if type(self.model)is tuple:
                    sel.fModel(cnn.inputs, cnn.get_layer(theNameYouWant).outputs)
                self.expected_value = tf.reduce_mean(self.model(self.data), 0)

Under which function you added the piece of code? GradientExplainer()'s or _TFGradientExplainer()'s init ?


promethues connection reset by peer

What did you do?
Configured promethues to scrape blackbox exporter behind F5 load balancer
  - job_name: 'prometheus-blackbox-exporter-lon-internal'
    scrape_interval: 20s
    scrape_timeout: 10s
    metrics_path: /probe
    scheme: https
    params:
      module: [http_2xx]
    static_configs:
      - targets:
        - https://gain.uat.clarksons.com
    relabel_configs:
      - source_labels: [__address__]
        target_label: __param_target
      - source_labels: [__param_target]
        target_label: instance
      - target_label: __address__
        replacement: promblackbox-lon.sea.live

What did you expect to see?
Promethues to succesfully scrape target
What did you see instead? Under which circumstances?
Error in target
Get "https://promblackbox-lon.sea.live:443/probe?module=http_2xx&target=https%3A%2F%2Fgain.uat.clarksons.com": read tcp <IP_ADDRESS>:43898-><IP_ADDRESS>:443: read: connection reset by peer
--

using wget on promethues pod I can succesffully reach target
wget "https://promblackbox-lon.sea.live/probe?module=http_2xx&target=https%3A%2F%2Fgain.uat.clarksons.com"

I've searched user forum but no answers from anyone so I'm assuming this is a bug.

Environment
Azure Kubernetes
I;m using the helm chart for promethues appVersion: 2.19.0

System information:

Linux 4.15.0-1089-azure x86_64

Prometheus version:

prometheus, version 2.19.2 (branch: HEAD, revision: c448ada63d83002e9c1d2c9f84e09f55a61f0ff7)
  build user:       root@dd72efe1549d
  build date:       20200626-09:02:20
  go version:       go1.14.4

Prometheus configuration file:

global:
  scrape_interval: 1m
  scrape_timeout: 10s
  evaluation_interval: 1m
  external_labels:
    cluster: pltfrmaks1-stg-ne
alerting:
  alertmanagers:
  - static_configs:
    - targets:
      - redalert-we.pltfrmaks-stg.sea.live
      - redalert-ne.pltfrmaks-stg.sea.live
    scheme: http
    timeout: 10s
    api_version: v1
rule_files:
- /etc/config/recording_rules.yml
- /etc/config/alerting_rules.yml
- /etc/config/rules
- /etc/config/alerts
scrape_configs:
- job_name: prometheus
  honor_timestamps: true
  scrape_interval: 1m
  scrape_timeout: 10s
  metrics_path: /metrics
  scheme: http
  static_configs:
  - targets:
    - localhost:9090
- job_name: prometheus-blackbox-exporter-lon-internal
  honor_timestamps: true
  params:
    module:
    - http_2xx
  scrape_interval: 20s
  scrape_timeout: 10s
  metrics_path: /probe
  scheme: https
  static_configs:
  - targets:
    - https://gain.uat.clarksons.com
  relabel_configs:
  - source_labels: [__address__]
    separator: ;
    regex: (.*)
    target_label: __param_target
    replacement: $1
    action: replace
  - source_labels: [__param_target]
    separator: ;
    regex: (.*)
    target_label: instance
    replacement: $1
    action: replace
  - separator: ;
    regex: (.*)
    target_label: __address__
    replacement: promblackbox-lon.sea.live
    action: replace

When you restart prometheus does it work again?

unfortunately not, I've killed the pod and after restart no difference.

And did it work with prometheus 2.18 ?

This is the first instance we have put in and haven't tried any previous versions but I'm happy to do so if you think it will work with older or newer version.

Yes issue might be with the HTTP/2 support introduced in 2.19 and 2.20. Downgrading could help, and a TCPDump coup help too. As this is connection reset and https there is not much risk of data leak but if you are afraid you can provide it to me privately.

Same issue :(
prometheus, version 2.18.2 (branch: HEAD, revision: a6600f564e3c483cc820bae6c7a551db701a22b3)
build user:       root@130a411dd4ff
build date:       20200609-09:05:58
go version:       go1.14.4

Then a tcpdump might help. Are you running the wget from the same machine/container?

yep, wget from same container
prometheus, version 2.18.2 (branch: HEAD, revision: a6600f564e3c483cc820bae6c7a551db701a22b3)
  build user:       root@130a411dd4ff
  build date:       20200609-09:05:58
  go version:       go1.14.4
/prometheus $ wget "https://promblackbox-lon.sea.live/probe?module=http_2xx&target=https%3A%2F%2Fgain.uat.clarksons.com"
Connecting to promblackbox-lon.sea.live (<IP_ADDRESS>:443)
wget: note: TLS certificate validation not implemented
saving to 'probe?module=http_2xx&target=https%3A%2F%2Fgain.uat.clarksons.com'
probe?module=http_2x 100% |*******************************************************************************************************************************|  2608  0:00:00 ETA
'probe?module=http_2xx&target=https%3A%2F%2Fgain.uat.clarksons.com' saved

Will attempt to get a TCPDump shortly.

Got a TCPDump, if it's OK can I share it privately as the dump contains all traffic from the cluster. How do I share privately with you?

Got a TCPDump, if it's OK can I share it privately as the dump contains all traffic from the cluster. How do I share privately with you?

<EMAIL_ADDRESS>

Can you tun
tcpdump -y any host <IP_ADDRESS> and port 443
to limit the scope

dump.zip
This is more resttricted so happy to share here, thank you.

The dump helps a lot. The TLS negotiation works as expected, then prometheus sends its payload and it "refused" by the target (F5).
You should look into the F5 logs to see what can cause the issue, but from what I see, everything is fine from a prometheus standpoint.

thank you for that, I will investigate with me networks team on this.
What is very strange is why wget works but promethues doesn't from the same container. Are you able to help explain why as it could help with troubleshooting the F5?

Prometheus uses the golang tls stack and adds some headers to the request. It also supports up to tls 1.3.

I will close this for now. Feel free to reopen based on your further findings if that ends up being an issue with our code.

I tested over http against the F5 and it works, I'm unsure where the problem lies as everything within our company goes through the F5 load balancer and nothing has this type of problem.
My guess right now is it's prometheus\go that's the cause here, we are looking at the logs to get more of an idea.


Remove TextDrawing arguments layout, global_phase, qregs, and cregs

Summary
In #7319, the parameters layout, global_phase, qregs, and cregs for qiskit.visualization.circuit.text.TextDrawing raise deprecation warnings. Time to remove the parameters and the warnings.
Details and comments
The deprecation cycle of these parameters started in 0.20.0 (released in 2022-03-31).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 84.655%

Files with Coverage Reduction
New Missed Lines
%

qiskit/pulse/library/waveform.py
3
91.49%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
64168

Relevant Lines:
75799

💛  - Coveralls

I suspect the circuit=None option may also want to be upgraded to a required argument as well now (I don't think None is a valid value for it).

Indeed, thanks! it should be ready now :)


UMI Support for hts_SuperDeduper!

Hello again,
I have added support for UMI-based PCR deduplication. It functions by just extracting the UMI from the read ID, converting it to bits, and appending it onto the sequence key used for PCR duplicate removal, essentially just extending the sequence. I also added an extra test function in hts_SuperDeduper_test that was useful during development.  Additionally, I have ran this new version of the tool on a variety of datasets and it appears to work as intended.  I originally wanted to provide some data for this PR, but ultimately I realized that testing fell more in line with bench marking then validating the algorithm. That being said, if you would like to see some of these results, I would be happy to add what I have found. Although, it is worth noting that the effectiveness of this method on single end reads, particularly TAGseq experiments with lower complexity, while comparable to umi_tools, is ultimately worse and for some reason very sensitive to whether or not you use hts_CutTrim before it. I am currently working on finding the best parameter settings for its application to TAGseq.
Anyways, let me know what you guys think, I am excited to finally have some eyes on this one.

In hindsight, umitools considers the edit distance when considering the umi in deduplication... let me know if you guys would prefer the implementation include this.

somewhere however, now incorrect QUAL characters are getting introduced. Brad have you seen this anywhere else on this branch?

I have not. Is this an issue present in the test dataset? Also, what is the
exact issue? Is it just characters some how being introduced or switched?
On Thu, Jun 6, 2024 at 9:36 AM Matt Settles @.***>
wrote:

somewhere however, now incorrect QUAL characters are getting introduced.
Brad have you seen this anywhere else on this branch?
—
Reply to this email directly, view it on GitHub
https://github.com/s4hts/HTStream/pull/261#issuecomment-2152953297, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMME3Q53LFNI6KNAVFRA7T3ZGCFW3AVCNFSM6AAAAABI5CZPJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSHE2TGMRZG4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

No in a large dataset, samtools fails with invalid QUAL character found.
Looking at the read in the bam file it show a bad character in the middle
of a read, only in overlapped SE reads though, Pairs work just fine.
I’m backing up to before you changes now to see if the error is repeated
there. And will report back
Matt
On June 6, 2024 at 9:40:37 AM, Bradley N. Jenner @.***)
wrote:
I have not. Is this an issue present in the test dataset? Also, what is the
exact issue? Is it just characters some how being introduced or switched?
On Thu, Jun 6, 2024 at 9:36 AM Matt Settles @.***>
wrote:

somewhere however, now incorrect QUAL characters are getting introduced.
Brad have you seen this anywhere else on this branch?
—
Reply to this email directly, view it on GitHub
https://github.com/s4hts/HTStream/pull/261#issuecomment-2152953297, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AMME3Q53LFNI6KNAVFRA7T3ZGCFW3AVCNFSM6AAAAABI5CZPJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSHE2TGMRZG4>

.
You are receiving this because you modified the open/close state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/s4hts/HTStream/pull/261#issuecomment-2152963162, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6RZXSVXMPMGLVNXX2VBTZGCGILAVCNFSM6AAAAABI5CZPJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSHE3DGMJWGI
.
You are receiving this because you commented.Message ID:
@.***>


Widget content loading failed (HTTP instead of HTTPS)

Hi,
We've been trying to setup a test integration for magento using our pre-production environments and have been running into this error:

We managed get around it by manually changing the form action in the HTML from http to https. It seems that the form will request the new widget to be loaded via HTTP and the options request fails as it receives a 301 redirect.
Cheers

Hi @AdamJHall
Thank you for reporting the issue!
For generation js routes we use FOSJsRoutingBundle that uses router.request_context... parameters. Did you redefine this parameters in your parameters.yml file, for example router.request_context.scheme?
If not we will create an internal issue to check this behavior. And I advice to try to set router.request_context.scheme: https for your case instead workaround with changing form action.

Hey @bemzaslava
That hasn't been altered at all as far as I am aware. Thanks for the workaround.
Regards,
Adam


JSONata was missing from the README

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Enable service endpoints on UCR

This PR enables Service Endpoints partials for the UCR runtime. Previously it wasn't possible to have a Service endpoints running on UCR.
Please test JSON to Form and Form to JSON. Launching and editing the app.
Testing
Choose UCR as a runtime

Give it a hard time testing permutations

Closes DCOS_OSS-1550
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run integration tests

One thing where I'm not exactly sure if it is as expected: when entering a port in the json, nothing happens in the form, but the port is actually send to marathon 🤔

@juliangieseke The host port isn't selectable. But does raise the question why we are sending the port to Marathon.

well, there is no way changing this at all on master, not sure if it introduces it, but before there was no form element to edit this fields. Not sure if we should reset the port when the form is edited or not 🤔

@juliangieseke I'm not sure what's gone wrong in your case. I created an app with UCR and dcos network. Then I edited the app and it didn't switch network to the host mode. If your bug regarding the host network is reproducible please create a jira.


List vs. IList

Hi,
you have a misleading info in your documentation or a bug at https://github.com/mbdavid/LiteDB/blob/master/LiteDB/Serializer/Mapper/BsonMapper.Serialize.cs
line 97:
else if (obj is IList || type.IsArray)
96             {
97                 var itemType = type.IsArray ? type.GetElementType() : type.GetGenericArguments()[0];
98
99                 return this.SerializeArray(itemType, obj as IEnumerable, depth);
100             }
Your documentation states that you support IList of T, which is not true.
You should not mix up IList of T and IList, because IList of T does not extend IList.
Furthermore, you should not expect the "non-array" type to have a generic argument (what about derived classes, or at least implementors like class MyList : "IList of MyClass"?).
The source code seems to be tailored for the use of List of T, which is too restrictive in many cases.
Anyway, thanks for LiteDB!
Pavel

Sorry, there was "IList of T", which I wrote normally using the sharp brackets (and it was taken away as a markup...)

Hi @PavelVonBahnhof, thanks for that. Yes, IList of T doesnt extend IList. Bet. I guess that's better use IEnumerable for that.
But, the problem it's the same: how detect which "collection item type" is?  Did you have any idea with how I can implement that?

Hi David,
there are basically two problems (at least I think so):

I don't know whether you are ready for polymorphism (ItemType might possibly be an abstract class and in reality there are concrete objects derived from this class that are to be stored in the database)
So, the ItemType of IList and IEnumerable... (non-generic versions) is System.Object (of course). But you hardly ever store System.Object instance in ArrayList, for example.
problems connected to generics (which in fact was what I personally missed in LiteDB)
Let's say you want to support IEnumerable only, nothing more - thus you have to iterate through all implemented interfaces to look for it... - something like that:

foreach (var i in type.GetInterfaces())
{
    if (type.IsGenericType && type.GetGenericTypeDefinition() == typeof(IEnumerable<>))
    {
        return type.GetGenericArguments()[0];
    }
}

But again - the ItemType might be the type of an abstract class and not the type of the real concrete class... (the same problem as for 1) ).

That's it.
Thank you for your response.
If you need any other help/assistance, do not hesitate to ask.

Pavel

Od: Mauricio David<EMAIL_ADDRESS>Komu: "mbdavid/LiteDB"<EMAIL_ADDRESS>Datum: 13.11.2015 16:41
Předmět: Re: [LiteDB] List vs. IList (#77)

Hi @PavelVonBahnhof https://github.com/PavelVonBahnhof, thanks for that. Yes, IList of T doesnt extend IList. Bet. I guess that's better use IEnumerable for that.
But, the problem it's the same: how detect which "collection item type" is? Did you have any idea with how I can implement that?
—
Reply to this email directly or view it on GitHub https://github.com/mbdavid/LiteDB/issues/77#issuecomment-156467268.

I need get item type of a List just to compare with each instance of object. If are diferent, I add in BsonDocument an field called _type with full name of concrete object type.
https://github.com/mbdavid/LiteDB/blob/master/LiteDB/Serializer/Mapper/BsonMapper.Serialize.cs#L151)
I will take a look how MongoDB/Json.NET implements that.
Thanks for your help

Sorry, there was a typo in my answer ("type" -> "i") :(
I've corrected it...
Pavel

Well, after looking at BsonMapper.Deserialize.cs, I think you'd better use IList instead of IEnumerable in the code above, as I think it would be hard to change the contents of:
private object DeserializeList(Type type, BsonArray value)
to fit the usage of IEnumerable


Promotion on Facebook

Create a Facebook account that would allow to link Twitter updates on the Facebook NodeOS page.

Does this just mean fb would mirror what's posted on Twitter?

Yes. Maybe we could publish something there, but mostly just only what's on
Twitter. Just a way to get diffusion using Likes.
El 6/8/2016 2:11, "Tyler James Leonhardt"<EMAIL_ADDRESS>escribió:

Does this just mean fb would mirror what's posted on Twitter?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/282#issuecomment-237993228, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgfvp9mJ7QG_p6nxV6XOlJvxh002Jitks5qc9E_gaJpZM4JeBJR
.

How about:
https://ifttt.com/recipes/114287-when-i-tweet-it-then-feeds-to-my-facebook-fan-page

I tried to use native Twitter integration but give me error because my
Facebook wall is already integrated. I checked iftt and didn't asked me to
provide the page, so I think it would do the same... I'll check later on
Facebook what happened.
El 7/8/2016 3:24, "Tyler James Leonhardt"<EMAIL_ADDRESS>escribió:

How about:
https://ifttt.com/recipes/114287-when-i-tweet-it-then-
feeds-to-my-facebook-fan-page
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/282#issuecomment-238057639, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgfvoBYCThUrvnEW51tPV334Z1N0Vwaks5qdTO4gaJpZM4JeBJR
.


Add more SeekOps

org.fusesource.lmdbjni.SeekOp includes only two SeekOps, KEY and RANGE.
Lmdb supports a ton more in JNI.java. In particular, MDB_FIRST would be helpful so we can seek to the first record. But there's no reason not to add the rest of them.
I would do this myself and do a pull request, but I still can't do a build in Eclipse.

Sounds like a good improvement.
I tried to open the project in Eclipse and the project doesn't look very happy so I fixed these just now.

Removed lmdbjni-all since it's not released
Removed unused Java 8 import.
Removed Unsafe Java 8 specific operations.

The only thing I don't understand is that hawtjni plugin 'build' goal is not recognized, not sure what this is about? Anything more that messes up the Eclipse build?

Eclipse provde quickfixes that makes the plugin errors go away but it tries to add e2 specific stuff to the poms, which is a bad idea. Maybe try configure default mappings in eclipse without using poms?
https://www.eclipse.org/m2e/documentation/m2e-execution-not-covered.html

I added the m2e specific stuff to the poms and they worked. It's ugly,
but not fatal.
On 1/23/2015 1:11 PM, Kristoffer Sjögren wrote:

Eclipse provde quickfixes that makes the plugin errors go away but it
tries to add e2 specific stuff to the poms, which is a bad idea. Maybe
try configure default mappings in eclipse without using poms?
https://www.eclipse.org/m2e/documentation/m2e-execution-not-covered.html
—
Reply to this email directly or view it on GitHub
https://github.com/deephacks/lmdbjni/issues/17#issuecomment-71247456.

--
Chris Cleveland
Dieselpoint, Inc.
+1<PHONE_NUMBER> x116
+1<PHONE_NUMBER> mobile
http://dieselpoint.com
<EMAIL_ADDRESS>This email and any attachments contain information from Dieselpoint,
Inc. and should be considered confidential. If this email is
received in error, please delete it and notify the sender.

It seems that both .seek() and .get() hit the same mdb_cursor_get() function. Perhaps using GetOp is basically equivalent?

Also, on the poms, there was a bunch of version errors. Some places the version was 0.3.1 and other places 0.1.3. I made them all 0.3.1 and everything compiled.

It's important to stay close to the LMDB API and there is a little bit of mix up right now. MDB_cursor_op is not consistently exposed in Cursor since there is seek and get, but I think all options are there, no?
Do you any suggestions on how to improve it?

My best suggestion is to duplicate the LevelDB API. Look at org.iq80.leveldb.DBIterator. It supports .seekToFirst(), etc.

Yes, that's indeed more user friendly, but not quite complete in terms of functionality.
I was actually thinking of modernizing the whole API (as a separate artifact), but haven't found time to do it yet. Also not sure if this would useful?

If you've got the time to do it, by all means...
I think there are some advantages to having a BerkeleyDB / LevelDB / RocksDB compatibility layer. Makes it much easier for people to adopt.
I also think that having some kind of zero-copy API would be helpful. If Lmdb or some other Java-based db can surface some MappedByteBuffers then the performance will be much better. It's hard to do because the ByteBuffer interface is so awful, but I've tried to do it by creating my own wrappers. I've had limited success. It looks like the Lmdbjni code tries to do the same thing with its DirectBuffer.

I agree.
The zero copy should be working quite well. Have you tried it? According to my tests it's almost 4 times faster than buffer copy for cursor scans, and a lot faster than the JNI versions of rocksdb and leveldb.
https://github.com/deephacks/lmdbjni/blob/master/lmdbjni/src/test/java/org/fusesource/lmdbjni/PerfTest2.java

Here is a summary of a JMH test I just did. As you can see, the zero copy is A LOT faster than anything else.
Benchmark                    Mode  Cnt         Score         Error  Units
Iteration.leveldb           thrpt   10   7624941.049 ±  995999.362  ops/s
Iteration.lmdb_buffer_copy  thrpt   10   3066605.928 ±  610793.399  ops/s
Iteration.lmdb_zero_copy    thrpt   10  15029604.092 ± 1309367.614  ops/s
Iteration.rocksdb           thrpt   10   1505814.770 ±  420279.355  ops/s

Here is the full report if you're interested: http://pastebin.com/gPFVcakL

BTW you may want to have a look at https://github.com/deephacks/graphene which is built on lmdbjni and provide a higher level API similar to BerkleyDB.
It is Java 8 lambda friendly, uses the javac compiler for generating entity and builders classes with fast custom serialization and has a simple query language. It's still very experimental but works for basic stuff.
Let me know what you think.

Looks interesting. I'll poke around.
Here's one that I wrote: https://github.com/dieselpoint/norm

Let me know what you think of these additions.

I have work a bit on the BufferCursor API in order to make it safer and easier to use. You no longer need to juggle DirectBuffers which is nice and it's waaay faster than the regular buffer copy API.


Don't allow registration with OAuth2

Hi,
First of all thanks for your awesome work!
I have use case where I wan't to allow sign in with OAuth2 for existing application users, but I don't want to allow new registrations (the registration process happens manually by an administrator).
However, I'm not sure how I can achieve that. I've looked at the code of the OAuth2 module and I noticed that there is no event fired before attempting to save the user into the OAuth2ServerStorer:
https://github.com/volatiletech/authboss/blob/e74112f6170d67dafda5c9a9033bda38cd9df4e5/oauth2/oauth2.go#L235-L249
The only solution I see is to implement my own OAuth2 module that fires an event prior to saving the user. In the event handler I'll check whether the user exists, and if it does not, I'll stop the execution of the OAuth2 module (effectively skip saving user into the store).
Other option I see is to have a custom OAuth2 module that handles specific error returned from the OAuth2ServerStorer.SaveOAuth2 and again skip persisting the user.
I like the first approach better, but still I wanted to hear whether this is something that anyone else has considered and if it will eventually fit into authboss itself.
Best,
Ivan

Hi Ivan, I haven't considered this one before. The event seems a bit strange as it's a sort of bail out event which is more rare.
It's considered a fine practice to fork these modules, they don't change much at this time so I wouldn't hesitate.


MoreApp Forms - Improve "Create task" action

Hello @beestree,
Kindly check the below swagger validation errors, the errors are still exists, please update your swagger files duly.

@vmanoharas All errors that were indicated by the paconn validate command are solved. I don't see a new failing check, did you run it again?


SSL Error

errorRiskQuant.txt
I have tried to install RiskQuant by follwing the instructions in the ReadMe. I get an SSL error every time and so do other engineers. This is a direct clone and pip install . nothing more.

I'm not getting the same error. The error message looks like a certificate issue at pypi.org--maybe it was intermittent, can you try again now?

This is the error I'm gettin:
ERROR: Command errored out with exit status 1:
command: /usr/local/opt/python/bin/python3.7 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/pr/gypqzb0x4cd3569k14s8xw2c0000gn/T/pip-req-build-81f_z_dj/setup.py'"'"'; file='"'"'/private/var/folders/pr/gypqzb0x4cd3569k14s8xw2c0000gn/T/pip-req-build-81f_z_dj/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /private/var/folders/pr/gypqzb0x4cd3569k14s8xw2c0000gn/T/pip-req-build-81f_z_dj/pip-egg-info
cwd: /private/var/folders/pr/gypqzb0x4cd3569k14s8xw2c0000gn/T/pip-req-build-81f_z_dj/
Complete output (37 lines):
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(136, '[X509] no certificate or crl found (_ssl.c:4109)'))': /simple/setupmeta/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(136, '[X509] no certificate or crl found (_ssl.c:4109)'))': /simple/setupmeta/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(136, '[X509] no certificate or crl found (_ssl.c:4109)'))': /simple/setupmeta/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(136, '[X509] no certificate or crl found (_ssl.c:4109)'))': /simple/setupmeta/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError(136, '[X509] no certificate or crl found (_ssl.c:4109)'))': /simple/setupmeta/
ERROR: Could not find a version that satisfies the requirement setupmeta (from versions: none)
ERROR: No matching distribution found for setupmeta
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/setuptools/installer.py", line 128, in fetch_build_egg
subprocess.check_call(cmd)
File "/usr/local/Cellar/python/3.7.6_1/Frameworks/Python.framework/Versions/3.7/lib/python3.7/subprocess.py", line 363, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/usr/local/opt/python/bin/python3.7', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/var/folders/pr/gypqzb0x4cd3569k14s8xw2c0000gn/T/tmpuh4g6wv5', '--quiet', 'setupmeta']' returned non-zero exit status 1.

Try
pip install --upgrade setuptools
That may fix your issue. In any case, the problem doesn't seem to be directly related to riskquant (which is installing from the local directory) but instead is happening when pip tries to fetch one of the dependencies.
The problem has something to do with your version of pip, or setuptools, or possibly something on your network intercepting SSL connections.

If you are still having this issue, we also just merged a PR with instructions on how to run riskquant using Docker.   This might be an option that resolves your issue:
https://github.com/Netflix-Skunkworks/riskquant#using-riskquant-via-docker


make format and make clippy

Signed-off-by: qupeng<EMAIL_ADDRESS>What have you changed?
make format && make clippy.
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?

No code

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
Does this PR affect tidb-ansible?
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Any examples? (optional)

/run-all-tests

/run-all-tests


Change project file structure/naming

Hey, nice work! I like the app, especially the work on themes :)
So, I made a few changes. If you're interested, I'll submit pull requests like this and we can talk about the changes. Just remember, its always your decision, and if you're unsure about anything, ask! Happy to help.
Remember to checkout the pull request and try it out yourself. As the reviewer, its welcome/expected for you to get involved however you want. You should criticize freely. One important job is to make sure its working as I will definitely make mistakes sometimes, and that's another good reason to ask about things. You're still learning, but always trust your instincts.
About the changes: when I started learning, I remember seeing similar file-structure in other people's repositories. It depends on the type of project, but there really aren't any rules you have to follow. Mostly, the goal is to make life easier. Let me know if you're unsure about any of the changes, or anything about the pull request process, and I'll be back to answer.

Rename icons/images/json (prefixes, case, spaces, etc)
Rename primary calculator script to 'main.py'
Add folder 'data' for 'themes.json' (was 'color_attributes.json')
Add folder 'icon' for icons (png/ico)
Add folder 'img' for all other images
Update readme: links
Update readme: 'Usage' (note: avoid linking to .exe files (7-zip)!)

thanks for contributing to my project :)

thanks for contributing to my project :)

@warmachine028 you're very welcome.

@warmachine028 you're very welcome.


Optimizer initialization with DDP

❓ Questions and Help
What is your question?
I would have expected optimizers to always be initialized after parameters have been moved to their destination device.
However, some ddp backends such as
ddp_backend,             ddp_spawn_backend,            ddp2_backend
initialize the optimizer with the CPU parameters before moving the model to the GPU while others such as gpu_backend pass the GPU parameters.
I'm currently trying to understand two things:
(1) Where does the linking from the CPU to GPU parameters happen?
(2) Is it actually necessary to initialize the optimizer before moving to the specific device or could it be done afterwards? (Most tutorials initialize the optimizer after placing the parameters on the corresponding device, e.g.  this one)
The reason I'm asking is that I did some parameter/gradient bending to be views into other tensors. This does not work with the current implementation as the optimizer keeps the reference to the CPU parameters with these tweaks but works fine when adapting the pytorch_lightning code by moving the optimizer creation after the model has been moved to the correct device.

Hi, could you share what you changed to defer the initialization?

For ddp_backend, I've moved lines 174-177 ( which is mainly optimizers, lr_schedulers, optimizer_frequencies = self.trainer.init_optimizers(model)) to line 199, i.e., after model.cuda(self.trainer.root_gpu). Note that so far I have only verified that the parameters receive gradients again, so I don't know if this breaks something else.

Looks good, I don't see a problem with this. Could you send a PR, then we will see if the tests pass? Then we should also check the other backends to make it consistent.

Done :)


NOISSUE -update message module

Sending message as selected thing to multiple channels at once
Signed-off-by: Ivan Milošević<EMAIL_ADDRESS>What does this do?
Update message module to send message as selected thing to multiple channels at once

Codecov Report

Merging #599 into ui will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               ui     #599   +/-   ##
=======================================
  Coverage   86.86%   86.86%
=======================================
  Files          62       62
  Lines        3381     3381
=======================================
  Hits         2937     2937
  Misses        305      305
  Partials      139      139

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 8c52fc6...1e7af53. Read the comment docs.


Startup error

Microsoft PowerToys version
0.49.1
Running as admin

[x] Yes

Area(s) with issue?
General
Steps to reproduce
It just show up error with windows 11 startup
2021-12-18.txt
!
✔ Expected Behavior
No response
❌ Actual Behavior
Powertoys didn't run at windows startup and instead asked me to report this error!
Other Software
No response

I would say try 0.51.1 but be aware this is one of those weird issues we are having issues tracking down that pops up. Does 0.51.1 help resolve it?


Dual US + Bulgarian (BDS) layout for XP15 and the likes

ISO layout
Dual US + Bulgarian layout based on the Bulgarian National Standard aka BDS. Tried to be as close as possible to the regular keyboards sold locally.
Both Latin and Cyrillic letters are of the same size, because they are both "main" layouts.
Cyrillic letters using Arial font, due to the lack of support in Radjhani font.
For convenience, I have also created file links for the laptops with the same keyboard.

Please let me know if something is misaligned or any character is too close to the key edges. I had to fit 4 or 5 characters on some keys, so I had to balance between readability and the key space.

Thanks for your contribution! :partying_face:


Configuration 'proxy.config.json' could not be found in project with cli version 6.0.0-beta.6

When serving content using ng serve -pc proxy.config.json
I get a error message like the following:
Configuration 'proxy.config.json' could not be found in project.
Error: Configuration 'proxy.config.json' could not be found in project.
at Architect.getTarget (d:\Projects\components\lob\node_modules@angular-devkit\architect\src\architect.js:114:23)
at MergeMapSubscriber.architect.loadWorkspaceFromJson.pipe.operators_1.concatMap [as project] (d:\Projects\components\lob\node_modules@angular\cli\models\architect-command.js:56:45)
at MergeMapSubscriber._tryNext (d:\Projects\components\lob\node_modules\rxjs\operators\mergeMap.js:122:27)
at MergeMapSubscriber._next (d:\Projects\components\lob\node_modules\rxjs\operators\mergeMap.js:112:18)
at MergeMapSubscriber.Subscriber.next (d:\Projects\components\lob\node_modules\rxjs\Subscriber.js:90:18)
at MergeMapSubscriber.notifyNext (d:\Projects\components\lob\node_modules\rxjs\operators\mergeMap.js:145:30)
at InnerSubscriber._next (d:\Projects\components\lob\node_modules\rxjs\InnerSubscriber.js:23:21)
at InnerSubscriber.Subscriber.next (d:\Projects\components\lob\node_modules\rxjs\Subscriber.js:90:18)
at Object.subscribeToResult (d:\Projects\components\lob\node_modules\rxjs\util\subscribeToResult.js:17:25)
at MergeMapSubscriber._innerSub (d:\Projects\components\lob\node_modules\rxjs\operators\mergeMap.js:132:38)
but with version 6.0.0-beta.4 it all works fine.
Versions
6.0.0-beta.6

I have also seen this issue after upgrading. If I move the proxy.conf.json file into src it works. I'd rather keep it in the root directory though if possible.

issue seems to be that you can no longer use -pc, use --proxy-config

Yes.  CLI v6 now follows more standardized option parsing.  -pc is the equivalent of -p -c.  A note in the changelog should probably be added to mention this change.

Follow the below steps, it helped me
i)create proxy.conf.json file under your project root instead of src folder
ii) add this exactly in package.json under scripts   "start": "ng serve --proxy-config proxy.conf.json"
iii)do "npm start" instead of "ng serve"

Oof. Thank you @rtaft Makes sense cause the standardization, but didn't even think about it since I've used --pc for every project.
@clydin A note for the 'update.angular.io' would be useful.

Changing ng serve --proxy-config proxy.conf.js to ng serve --proxy-config=proxy.conf.js, solved my issue. NOTE: i use .js not .json file

Changing ng serve --proxy-config proxy.conf.js to ng serve --proxy-config=proxy.conf.js, solved my issue. NOTE: i use .js not .json file

ng serve --proxy-config=proxy.conf.json  it worked! Thanks

step 2
rename your folder " proxy.config.json " to " proxy.config.json "
---------------------------------------- then -------------------
step 3
change the path in " package.json " the line with " ng serve nproxy.conf.json --open " with " ng serve --proxy-config proxy.conf.json -o "
then you can run " yarn dev "

@sushobitha thanks for sharing!