feat(GUI): warning makes drive-selector button red

We make the drive-selector button red when there is a warning attached
to the image-drive pair.
Change-Type: patch
Changelog-Entry: Make the drive-selector button red on warnings.

This feels like it might be confusing, as there's no indication as to why the button has changed color, since the warning is hidden behind the modal – wdyt @konmouz?

Maybe we should match the yellow warning colour instead? @konmouz

Also, is this something that's been in @konmouz' designs?

@jhermsmeier yeah, here it is on Zeplin – the difference is system drives aren't errors so they're orange. Also here's a screenshot with warning colors on the button instead, @konmouz let us know what you think.

@Shou have you made the button orange now?

So, have you changed the button to be orange already @Shou? If so, the commit message needs updating

I've pushed the commit with orange now and rebased


Extend logs

Coverage increased (+0.09%) to 34.527% when pulling b256f8e16da4d83eb044da0da85eeb68662fe341 on fcastilloes:extend-logs into 29733cb3c782e904fdc1ab85a252f9824d19bac0 on kusanagi:master.


Color changes aren't retained after game restart

Changes to, for example, GCD bar color, are reverted to default after restarting FFXIV.

Hello, thanks for the report!
I'm having some trouble replicating this issue, either with old config files or new instances. Could you provide some more information, which color are you selecting? Are you on version <IP_ADDRESS>?
Also, you might want to take a look at deleting/renaming the "GCDTracker.json" file at %appdata%\XIVLauncher\pluginConfigs. If it's an invalid config issue that should fix the issue.

Closing this due to not being able to replicate, If the issue persists feel free to reopen.


[Feature Request] IPInfo.io for core IP based operation

I work for IPinfo and I appreciate the project featuring IPinfo. I saw that in a ticket you mentioned that you favored no-key access to API. IPinfo does provide no-key access to the API service and a generous limit for that as well. Over your current setup, we have the following to offer:

HTTPS/SSL encryption
More accurate geolocation data backed by our probe network
1,000 requests per day without a token

Users can also sign up for a free account and get a limit of 50,000 requests per month. If you are looking to add more functionality, you can also check out our free IP to ASN database. Please let me know what you think. Thanks!

Hey @abdullahdevrel,
actually, I've already locally replaced RIPE IPmap with IPinfo.io precisely because of your generous no-key tier and above all because of your probe network, which I think is the only solution on the market that can compete (and very often, outperform) IPMap. Moreover, IPmap has recently failed me in several occasions (e.g. when destinations are not yet mapped on their probe network), so I took my decision already to replace it.
The IPinfo branch is not live yet, as I'm doing some more accuracy testing locally, but the switchover will happen in the next release of asn.
Thanks for your suggestion, and great job on IPinfo, which I honestly think is a truly awesome product!

Thank you very much @nitefood We really appreciate your kind words. Our free service, API and database is all dedicated to awesome projects like nitefood/asn. Let me know if you need to add more features, particualarly from our free IP databases: https://ipinfo.io/products/free-ip-database

the project is now using ipinfo.io as its default geolocation provider for non-bulk queries, with an option to add an API key to increase query limit. Thanks again for your input and product!


Something really odd is happening with the scrolling

Desktop: no issue
iOS: ???
Android (4.4): scrolling is smooth if the finger used to scroll started on the image, but is not smooth if you start scrolling without your finger anywhere else.

Source:  https://github.com/StephanBijzitter/Famous-Material-Examples
I also patched Scrollview.js with this PR (but it doesn't seem to change anything for this demo): https://github.com/talgleichger/famous/commit/6740e1da838d90dea6dfec0ff3e34e0673dd706a
I really don't understand what's going on and how this is happening, it makes no sense :'(

Ping @StephanBijzitter

Resolved on above issue


Doesn't seem work with coffeescript

I have following lines in my package file:
  "scripts": {
    "test": "ava --require coffee-script/register --verbose test"
  },
  "devDependencies": {
    "ava": "^0.11.0",
    "coffee-script": "^1.10.0"
  }

The test directory contains a main.coffee file with ava tests.
When I run npm test, however, it always says Couldn't find any files to test.
Any ideas?

@adius currently test files must be JavaScript files, ending in .js. Are you trying to use a test file written in Coffeescript?

You could compile it first to JavaScript and then run that file. Just like with TypeScript. Although it might be cleaner if AVA could handle .coffee files directly.

I haven't tried this, but couldn't you change the search pattern for files to include test/**/*.coffee either in AVA config in package.json or directly in your test command? Assuming you retain the --require coffee-script/register

couldn't you change the search pattern for files to include test/**/*.coffee

Unfortunately, no. We are discussing a similar issue for jsx here.

Closing in favor of #229.


Increase the usage of augmented assignment statements

:eyes: Some source code analysis tools can help to find opportunities for improving software components.
:thought_balloon: I propose to increase the usage of augmented assignment statements accordingly.
Would you like to integrate anything from a transformation result which can be generated by a command like the following?
(:point_right: Please check also for questionable change suggestions because of an evolving search pattern.)
[Markus_Elfring@fedora lokal]$ perl -p -i.orig -0777 -e 's/^(?<indentation>\s+)(?<target>\S+)\s*=\s*\k<target>[ \t]*(?<operator>[+\-%&|^@]|\*\*?|\/\/?|<<|>>)/$+{indentation}$+{target} $+{operator}=/gm' $(find ~/Projekte/python-tuf/lokal -name '*.py')

How do you think about to improve ten source code places? :thinking:

If there are places in the newer codebase (tuf/api/ and tuf/ngclient/) where augmented assignment is not used (and could be used), I'd be interested in a patch... but I think there aren't any.
Other locations we might not care as much -- most of it is code we'd like to deprecate sooner or later.

:thought_balloon: How do you think about to try a variant of the shown transformation command example out?
How many update candidates would be still relevant? :thinking:

My opinion is that there are currently no cases where changing the assignment would be worth a PR.

:thought_balloon: Can it be that adjustments according to the mentioned transformation pattern would trigger desirable effects on the software run time characteristics?

Theoretical perf improvement may be possible but it would be such a tiny one we'd be unable to measure it. Readability would be a better selling point... but as I said the code bases that matter seem to be good already.
Thanks for bringing this up, but I think I'll close it: there does not seem to be anything that needs fixing.


Datatype returned by dayofweek() is Timespan and not Int as document states

The data type returned by the function dayofweek() is Timespan but the documentation says it is an Int.
e.g.
print dayofweek(datetime(2021-03-04))
returned

print_0
4.00:00:00

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

ID: b9f82871-793a-1887-529a-0e7d9fd7df77
Version Independent ID: 5d6ae62b-1975-2c82-71b9-cffbb94333e8
Content: dayofweek() - Azure Data Explorer
Content Source: data-explorer/kusto/query/dayofweekfunction.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .


⚠ Bofrost.com has degraded performance

In 5eda337, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 730 ms

Resolved: Bofrost.com performance has improved in af833fb after 6 minutes.


Failed tests ("Bad plan"):  t/02.iterator.t  t/03-iterator-lots.t

Zef install of Math::Primesieve on an older MacOS failed tests; see output below.
(Note, I first used "--force-tests" to install the Math::Primesieve module,
then repeated the process using "-verbose --force-install" to get a printout of the test errors):
user@mbook:~$ zef install Math::Primesieve --verbose --force-install
The following candidates are already installed: Math::Primesieve
===> Searching for: Math::Primesieve
===> Found: Math::Primesieve:ver<0.4>:auth<github:CurtTilmes> [via Zef::Repository::LocalCache]
===> Testing: Math::Primesieve:ver<0.4>:auth<github:CurtTilmes>
[Math::Primesieve] t/01.basic.t .......... ok
[Math::Primesieve] t/02.iterator.t .......
[Math::Primesieve] Failed 3/4 subtests
[Math::Primesieve] t/03-iterator-lots.t ..
[Math::Primesieve] Failed 1/2 subtests
[Math::Primesieve] Test Summary Report
[Math::Primesieve] -------------------
[Math::Primesieve] t/02.iterator.t     (Wstat: 11 Tests: 1 Failed: 0)
[Math::Primesieve]   Non-zero wait status: 11
[Math::Primesieve]   Parse errors: Bad plan.  You planned 4 tests but ran 1.
[Math::Primesieve] t/03-iterator-lots.t (Wstat: 11 Tests: 1 Failed: 0)
[Math::Primesieve]   Non-zero wait status: 11
[Math::Primesieve]   Parse errors: Bad plan.  You planned 2 tests but ran 1.
[Math::Primesieve] Files=3, Tests=17,  3 wallclock secs ( 0.03 usr  0.01 sys +  4.13 cusr  0.23 csys =  4.40 CPU)
[Math::Primesieve] Result: FAIL
===> Testing [FAIL]: Math::Primesieve:ver<0.4>:auth<github:CurtTilmes>
Aborting due to test failure: Math::Primesieve:ver<0.4>:auth<github:CurtTilmes> (use --force-test to override)
user@mbook:~$
user@mbook:~$ zef --version
v0.8.5
user@mbook:~$ raku --version
This is Rakudo version 2020.06 built on MoarVM version 2020.06
implementing Raku 6.d.
user@mbook:~$

Nota bene:  I was able to run Math::Primesieve one-liners after installation and the first 100 primes agrees with the first 100 primes found at https://primes.utm.edu/lists/small/1000.txt .
HTH.

Some of the elements in the underlying library iterator structure were rearranged, so the module broke on later versions.  The normal functions worked ok, but iterator usage all crashed.
I updated the structure and bumped the 'api' to 2.  The tests now succeed for me.  I also tweaked the way the library gets loaded, hopefully in an improved way.
Just released version 0.6 with the changes.

It looks as though I can --force-install to update to raku module Math::Primesieve version 0.5, and I pass the previously failing iterator tests (zef tells me I'm still at Math::Primesieve version 0.4).
But I hit an error Failed to find dependencies: primesieve:ver<8>:from<native> as the module looking for native primesieve 8.0 as I only have version 7.5:
user@mbook:~$ zef --version
v0.8.5
user@mbook:~$ raku --version
This is Rakudo version 2020.06 built on MoarVM version 2020.06
implementing Raku 6.d.
user@mbook:~$ primesieve --version
primesieve 7.5, <https://github.com/kimwalisch/primesieve>
Copyright (C) 2010 - 2019 Kim Walisch
BSD 2-Clause License <https://opensource.org/licenses/BSD-2-Clause>
user@mbook:~$
user@mbook:~$
user@mbook:~$ zef install Math::Primesieve --verbose --force-install --force-test
The following candidates are already installed: Math::Primesieve
===> Searching for: Math::Primesieve
===> Updating cpan mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/cpan1.json
===> Updating p6c mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/p6c1.json
===> Updated p6c mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/p6c1.json
===> Updated cpan mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/cpan1.json
===> Found: Math::Primesieve:ver<0.5>:auth<github:CurtTilmes>:api<2> [via Zef::Repository::Ecosystems<cpan>]
===> Searching for missing dependencies: primesieve:ver<8>:from<native>
===> Failed to find dependencies: primesieve:ver<8>:from<native>
Failed to resolve some missing dependencies
user@mbook:~$
user@mbook:~$
user@mbook:~$ zef -v info Math::Primesieve
- Info for: Math::Primesieve
- Identity: Math::Primesieve:ver<0.5>:auth<github:CurtTilmes>:api<2>
- Recommended By: Zef::Repository::Ecosystems<cpan>
- Installed: Yes, as Math::Primesieve:ver<0.4>:auth<github:CurtTilmes>
Description:	 Raku bindings for primesieve
License:	 BSD-2-Clause
Source-url:	 http://www.cpan.org/authors/id/C/CT/CTILMES/Perl6/Math-Primesieve-0.6.tar.gz
Provides: 1 modules
--------------------------------------------
Module          |Path-Name
--------------------------------------------
Math::Primesieve|lib/Math/Primesieve.rakumod
--------------------------------------------
Support:
#   bugtracker:	https://github.com/CurtTilmes/raku-primesieve/issues
#   email<EMAIL_ADDRESS>#   source:	https://github.com/CurtTilmes/raku-primesieve.git
Depends: 5 items
------------------------
ID|Identity  |Installed?
------------------------
1 |NativeCall|✓
1 |NativeLibs|✓
1 |primesieve|
1 |Test      |✓
1 |Test::When|✓
------------------------
user@mbook:~$

Nota bene: I still am able to run raku Math::Primesieve one-liners after updating, so nothing broke from yesterday.
HTH.

But I hit an error Failed to find dependencies: primesieve:ver<8>:from as the module looking for native primesieve 8.0 as I only have version 7.5:

I most likely cannot help with your issue since I don't know Raku. But I confirm that the latest primesieve version is 7.5 (even on the master branch on GitHub). primesieve's ABI (binary API) has evolved slightly faster than primesieve's API so the ABI is currently 9.x for primesieve-7.x (i.e. libprimesieve.so.9.5.0 corresponds to primesieve-7.5). But usually programmers don't need to know the ABI version of a library...
primesieve-7.5 is backwards compatible up to primesieve-7.0 (Whenever I break the API I increase the major version of both the API and ABI).


Adding the namespace to the <module>.ready label.

Adding the namespace to the <module>.ready label.
This is handy when there are multiple modules on each node and make it
easier to determine where is each module.
Also, solving a race condition in a new pod was created while the old
pod of a moduleLoader is being terminated which resulted in the
<module>.ready label to be deleted.

Fixes https://github.com/kubernetes-sigs/kernel-module-management/issues/315

/lgtm

As a user, how can I use this label in a selector?

As a user, how can I use this label in a selector?

I think that we should discourage customers to use this label.  We need to update our ModuleStatus that will reflect all the data needed by the users. In case customers chooses to use this label, it is up to him to deal with the changes

@qbarrand I agree with @yevgeny-shnaidman, this label should not be part of the API IMO.
You can see how this labeled is used by the device plugin DS using nodeAffinity in this PR.

nodeAffinity was removed from the solution and I have also merged the adding of the namespace to the label to this PR.
@yevgeny-shnaidman @qbarrand PTAL.

/unhold
We are also keeping the old label for backward compatibility.

/retest

/test pull-kernel-module-management-lint

/assign @ybettan
(checking if prow bot is responding)

/lgtm

Looks like Prow's cluster has some issues - https://github.com/kubernetes/test-infra/issues/29003

/retest


Last report was a mistake - still, I am not getting pinged when I should be, do not know why

Issue created by kallisti#4556.

believe this particular issue is resolved (think it was a problem with punctuation not being stripped) but not enough information to replicate and test. You can assist future bughunting by including an example of a time something on your list did not result in a DM when it should have.


Cannot type in Youtube search

Youtube works, but when I try to search something in the search bar of youtube I am unable to type anything. I am able to right click and paste something but the search button also does not work for my self hosted php-proxy.
Does anybody else have a similar problem? could it be an IP related issue?

Try downloading my fork of php-proxy and hosting that (no need to change files for a quick test). I am able to search fine in mine so if it suddenly works, it was an issue with your installation. If it still doesn't work than there is a larger issue at play.

I checked your fork, I am not looking to provide HD videos yet since it would effect my content delivery capabilities and most of the user do not have a high speed internet as well.
Moreover I just found out that On a different browser I can type in youtube search through proxy.
Cannot type in Chrome, Tried Edge, it works fine. not sure if its a extension or what. Although unblockvideos.com youtube search works fine both of the browsers. Just not on my self hosted.

That’s strange. For the HD videos thing, that’s simply a line in the youtubeplugin.php file. Just remove the “” section that contains ‘mp4 720’.

From: kaliphx<EMAIL_ADDRESS>Sent: Wednesday, February 6, 2019 12:28 am
To: Athlon1600/php-proxy-app
Cc: Benji-Collins; Comment
Subject: Re: [Athlon1600/php-proxy-app] Cannot type in Youtube search (#147)
I checked your fork, I am not looking to provide HD videos yet since it would effect my content delivery capabilities and most of the user do not have a high speed internet as well.
Moreover I just found out that On a different browser I can type in youtube search through proxy.
Cannot type in Chrome, Tried Edge, it works fine. not sure if its a extension or what. Although unblockvideos.com youtube search works fine both of the browsers. Just not on my self hosted.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/Athlon1600/php-proxy-app/issues/147#issuecomment-460646900, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhYGy-7CrtJEgZSP3TRSp6wPPXi5gRfxks5vKY4YgaJpZM4aiYZs.

Alright will take a look again. Thanks


Days and Time Deck

A deck for time-boxing estimation, including half day days and 1/2 weeks, maybe sprints?

0.5 days
1 day
2 days
3 days
4 days
1 week
2 weeks

What about a split option?


flutter --version not response for a long long time

OS: win10 1803

and can't show flutter info on Android Studio Settings:

And, dart is ok

re-download flutter resolved!


Add the option to use default audio device.

Currently, when recording you can select the audio source from a drop down that lists all options, but if you, for example, plug a headset, it will not change automatically to the headset to record voice.
It would be nice to have a option like "Input - Default device" and "Output - Default device" to record desktop sounds.
One more suggestion is to the options to record audio from input and output devices.
Thank you very much for this awesome application.
It works really well and is impressively good.
Cheers!

Hello @klaytoncavalcante!
Which version are you using?
Because I've tried this at v9.0.0-beta4 version and works fine.

I was not in Beta. I was using the default version available in https://mathewsachin.github.io/Captura/download/ .
I will install the beta version then. Thank you very much and sorry for not checking the beta before talking about new features.
Have a nice day and thanks for this awesome tool!


Pinned ordering does not persist

Not sure how long it takes to reset, but whenever I've set the order of pins, it resets somehow

This will be commented out until 1.6.0. Removing from milestone


What is alternative for createFromObjects() here ?

shall i use batch ? how it works ?
Thx

There is convertTiledmap and convertTiledCollisionObjects, the former converts tiles into bodies, the later converts shapes you draw into bodies. The second one is more peformant and generally results in less shapes since you get to draw them yourself in Tiled.
Is that what you are asking? I'm very confused by your question.

i mean inserting objects and converting them to sprites in game.
http://phaser.io/docs/2.3.0/Phaser.Tilemap.html#createFromObjects

That functionality doesn't exist in this plugin.

sry for my endless questions,
is that possible to know which tile, character hit ?
(then i can hide that sprite which is a coin).
actually i want to put coins and enemies, how you do this ?

Sounds like you want to use objects, not tiles. Using an object layer, placing "tile objects" and giving them physics.

Sounds like this is answered, closing.


nixopsUnstable does not build on x86_64-darwin because of zfs-user-2.1.5

Steps To Reproduce
I made a repository that can reproduce the issue.
Build log
❯ nix build .#nixops_unstable
error: Package ‘zfs-user-2.1.5’ in /nix/store/7vr1ir078ybxzqg85ndc0b0cfdxizrcn-source/pkgs/os-specific/linux/zfs/default.nix:196 is not supported on ‘x86_64-darwin’, refusing to evaluate.

       a) To temporarily allow packages that are unsupported for this system, you can use an environment variable
          for a single invocation of the nix tools.

            $ export NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1

        Note: For `nix shell`, `nix build`, `nix develop` or any other Nix 2.4+
        (Flake) command, `--impure` must be passed in order to read this
        environment variable.

       b) For `nixos-rebuild` you can set
         { nixpkgs.config.allowUnsupportedSystem = true; }
       in configuration.nix to override this.

       c) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
         { allowUnsupportedSystem = true; }
       to ~/.config/nixpkgs/config.nix.
(use '--show-trace' to show detailed location information)

I think the issue is with libvirtd, but I can't prove it unfortunately.
Additional context
Notify maintainers
@roberth
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix-shell -p nix-info --run "nix-info -m"
these 2 paths will be fetched (0.34 MiB download, 2.06 MiB unpacked):
  /nix/store/nsikv84v16n9dgmfnmpr5fc572nhfs1y-bash-interactive-5.1-p12-dev
  /nix/store/z7r4pl0q7kvqjgd87zc60jj4b4djhn4h-bash-interactive-5.1-p12-doc
copying path '/nix/store/z7r4pl0q7kvqjgd87zc60jj4b4djhn4h-bash-interactive-5.1-p12-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/nsikv84v16n9dgmfnmpr5fc572nhfs1y-bash-interactive-5.1-p12-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-darwin"`
 - host os: `Darwin 21.6.0, macOS 10.16`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.9.1`
 - channels(root): `"nixpkgs"`
 - channels(ethan): `"darwin"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

You were correct about python3Packages.libvirt, see #194439. There's a second build problem though, involving missing setuptools dependencies. This may be the same problem that blocked #193552; not sure how to proceed.

@roberth Ah, that makes sense to me. I also ran into that second issue with setuptools when I tried to build nixops without the libvirt plugin. Unfortunate that we'll need to wait for a new release of poetry2nix!

Python packages need to add setuptools into nativeBuildInputs since 8895944c571bb37800e7c4641f2495dc21d0ed68. Poetry2nix does not seem to account for that yet.

They're already aware of the problem (in https://github.com/nix-community/poetry2nix/pull/736). I don't know how far along they are. Maybe they could use some help with identifying new overrides that nixops_unstable happens to need for its deps.

I think the fix for adding setuptools is pretty much done in poetry2nix (cc @K900), but there's a long tail of packages that will stop working because they depend on setuptools, and aren't yet packaged up in nixpkgs.

@ethnt If you want to dump a list somewhere on the poetry2nix issue tracker then that would be helpful. If you want to do a PR there that'd also be very welcome!

@K900 has repackaged it without poetry2nix, and after #292099 it builds on x86_64-darwin.


Remove object modified annotation and fix flaky tests

a part of #76 and as the next step in 0.1 after this discussion

/assign @adrianludwin

Approving myself since this is a simple removal of functionality.
/approve
/lgtm

Retroactive
/lgtm
/approve

Thanks Ryan, just didn't want to wait. We'll have another go at redesigning
this feature in v0.2.
On Fri, Oct 25, 2019 at 9:54 PM Ryan Bezdicek<EMAIL_ADDRESS>wrote:

Retroactive
/lgtm
/approve
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/multi-tenancy/pull/190?email_source=notifications&email_token=AE43PZAIXBYDJHIGT6DONDDQQOPM3A5CNFSM4JFI75T2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECJ5BRA#issuecomment-546558148,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZBYFHAHHGGZOSFJSBTQQOPM3ANCNFSM4JFI75TQ
.

No worries! Flaky tests are the bane of my existence.


pick a federation on setup

Amazing, I love it. I think the copy could improve, both here and in the setup. Been asked a few times what this means. I think we can explain it a bit better, something that focuses on what it provides the user, and something a little less scary than "use at your own risk" while still communicating that each is a new network. Hard to wrap it all up in a small package, could benefit from a "learn more" screen itself.

I think the copy could improve, both here and in the setup.

I'll take another pass at it. Really hard to explain without a wall of text but I'll try again. I can add a learn more explainer modal for the truly curious.

@TonyGiorgio updated with the text you provided

think this broke the playwright test

think this broke the playwright test

it was the confirm button's fault


Add docs for sb3 integration

Hey @simoninithomas, it would be cool if we could add documentation on how to download/push sb3 models at https://huggingface.co/docs/hub/models-libraries as done for other libraries

https://github.com/huggingface/hub-docs/pull/207

I'm closing it 🥳


Problem "Rising" models

Hi Junior,
I'm using the last version of RISE, but after the update,  an error is appearing just after attempt to "rise" any model -m = rise('model')-:
Error using rise/Contents
Attempt to execute SCRIPT Contents as a function:
<EMAIL_ADDRESS>Error in generic/set/set_all_options (line 443)
            this_options=dum_obj.(method_list{imeth});
Error in generic/set (line 136)
    set_all_options()
Error in dsge/set (line 133)
obj=set@generic(obj,varargin{:});
Error in dsge (line 368)
            obj=set(obj,varargin{:});
Error in rise (line 64)
            obj=obj@dsge(varargin{:});

Regards,
S.

I probably introduced that error. Let me track it down.

It wasn't from me actually, but it should be fixed now. I just made an update; could you check if that resolves your problem?

Hi SeHyoun,
Problem solved. Thanks
Regards,
S.

Hi again,
I am re-opening this issue because a new problem appears when trying to "rise" any model-m = rise('model')-, which is the following:
Error using horzcat Dimensions of matrices being concatenated are not consistent. Error in parser.build_markov_regimes/update_markov_chains_info_add_regimes (line 191)             reg_info.regime_names(:),num2cell(tmp) Error in parser.build_markov_regimes (line 64)     [markov_chain_info,Journal_check]=update_markov_chains_info_add_regimes(... Error in parser.transition_probabilities (line 135) [~,markov_chain_info,myifelseif]=parser.build_markov_regimes(transition_matrix_symbolic,markov_chains); Error in parser.parse (line 521)     ]=parser.transition_probabilities(... Error in dsge (line 318)             dictionary=parser.parse(model_filename,cmfArgins{:}); Error in rise (line 64)             obj=obj@dsge(varargin{:});
Thanks in advance.
Regards,
S.

Please try again.
J.
On Thu, Jun 28, 2018 at 11:49 PM, SC187<EMAIL_ADDRESS>wrote:

Hi again,
I am re-opening this issue because a new problem appears when trying to
"rise" any model-m = rise('model')-, which is the following:
Error using horzcat Dimensions of matrices being concatenated are not
consistent. Error in parser.build_markov_regimes/
update_markov_chains_info_add_regimes (line 191) reg_info.regime_names(:),num2cell(tmp)
Error in parser.build_markov_regimes (line 64) [markov_chain_info,Journal_
check]=update_markov_chains_info_add_regimes(... Error in
parser.transition_probabilities (line 135) [~,markov_chain_info,
myifelseif]=parser.build_markov_regimes(transition_
matrix_symbolic,markov_chains); Error in parser.parse (line 521)
]=parser.transition_probabilities(... Error in dsge (line 318)
dictionary=parser.parse(model_filename,cmfArgins{:}); Error in rise (line
64) obj=obj@dsge(varargin{:});
Thanks in advance.
Regards,
S.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jmaih/RISE_toolbox/issues/81#issuecomment-401184111,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACagzxkbXyWDJfek8O3ERlXaZDHrh3IWks5uBU-CgaJpZM4UuMyS
.

It works now.
Thanks, Junior.
S.

Hi Junior,
I'm using the last version of RISE, but trying to run any example (m=rise('anymodel')), the following error shows up:
`Error using horzcat
Dimensions of matrices being concatenated are not consistent.
Error in parser.build_markov_regimes/update_markov_chains_info_add_regimes (line 191)
reg_info.regime_names(:),num2cell(tmp)
Error in parser.build_markov_regimes (line 64)
[markov_chain_info,Journal_check]=update_markov_chains_info_add_regimes(...
Error in parser.transition_probabilities (line 135)
[~,markov_chain_info,myifelseif]=parser.build_markov_regimes(transition_matrix_symbolic,markov_chains);
Error in parser.parse (line 521)
]=parser.transition_probabilities(...
Error in dsge (line 318)
dictionary=parser.parse(model_filename,cmfArgins{:});
Error in rise (line 64)
obj=obj@dsge(varargin{:});`
regards,
S.

Dear SC187,

I am not able to replicate the problem. Could you point to an example where
the problem occurs?
Thanks,
Junior


Ability to read HTML code from a file

Describe the feature you'd like
Ability to read HTML code from a file with macro like html_from_file! that parses from HTML (or similar) file. In case of invalid input, it fails at compile time.
Is your feature request related to a problem? Please describe.
The current html! really messes up IDEs. Although this can be solved with an IDE plugin which will also be needed for reading from another file but out of the box support (without plugins) would be comparatively better.
Describe alternatives you've considered
Use include!macro to include a file but that requires that you to move the html! invocation into the file which is a really messy solution.
Additional context
Moving HTML to another file would also allow components to have <style> alongside HTML which could potentially solve the component specific styles issue.
Questionnaire

[x] I'm interested in implementing this myself but don't know where to start
[ ] I would like to add this feature
[ ] I don't have time to add this right now, but maybe later

I would really like to see this idea made into Yew proper.
For example, I have Angular template partials that are ~100 LOC that get really messy to deal with without proper editor/IDE support. I'd rather take an existing Angular project and just import the html files into a Yew application than copy-pasting them into the html! macro. Plus, I personally find it easier to maintain by splitting the view partial into its own file than having it live side-by-side with the component implementation.

This proposal needs some clarification. If the idea is to have a macro that behaves exactly like html! (i.e. we pretend that the content of the HTML file is located at the call site) we end up with a few issues.
Apart from all the technical issues like macro hygiene, source span erasure, and breaking incremental compilation, I also find it very dubious to access the call site's local variables from another file.
Yew doesn't attempt to implement the HTML standard properly, it just happens to use an HTML-like syntax. The consequence of this is that tools probably won't handle the syntax properly anyway. Formatting almost certainly won't like the Rust-flavoured parts and linters will complain about various things. To give a few examples of what can go wrong: even normal text is written { "like this" } in Yew, self-closing tags (which Yew requires for tags without a closing tag) aren't allowed for some elements as per the HTML standard, and finally, component names start with an uppercase letter but tag names are supposed to be case-insensitive (for the most part) so it might automatically lowercase it for you.
The alternative is of course to just treat the file as vanilla HTML. I don't think this is very useful because now you can't have event listeners, components, or any dynamic content really. There are a few rare cases where the content really is static (like maybe a vector graphic as seen in the inner_html example) and these are already covered by include_str!.
Frameworks like Angular are designed with this kind of separation in mind, Yew obviously isn't. I feel like it would require entirely different html! macro semantics in order for this to work without turning into a giant hack.

@siku2 I can't speak as to the original poster's intent, but my desire would be for some macro that does the work of reading the desired html file and running the html! macro on those contents. Admittedly, I'm not the best with writing Rust macros, but I would envision that a macro that simply reads the file to an &str and then calls the html macro (it should just be feeding it to the html function in the yew-macro crate, right?) on the resultant is more than sufficient for what I would envision to be a first step. In terms of looking at it from a webpack/rollup/parcel standpoint, Yew would just be implementing a require() and/or import() function.
I'm using VSCode for development and tried out the html extensions I have loaded with Yew's syntax and it behaved as expected: no difficulties with adapting to Yew-flavored HTML as if it was just handlerbars/twig/jinja2/lodash templates/every other templating language under the sun. The only pain point was the lack of Rust autocomplete, but errors in any embedded Yew-flavored HTML should be caught at compile time, no?
Finally, referring to @jstarry 's comment on the previous pull request for this feature request: does Yew really need explicit code editor experience improvements when current solutions are more than sufficient. I don't think anyone really wants to build and maintain a Yew-flavored Rust plugin for all the text editors and IDEs that are available, nor do I think that there's enough distinction in Yew-flavored HTML to warrant adding a .html.yew file extension to the world for editors/IDEs to separate vanilla HTML from Yew-flavored HTML when the current offerings really do what's needed to make the developer experience nicer.

I think the problem here is mostly around formatting with the HTML macro (and possibly editor support, though I think that the problem there is with IntelliJ-Rust and the solution to that is to use rust-analyser which is open source, and also runs in an open source editor). I believe that it should be possible to construct a tool to format html macro invocations specifically – or perhaps look into what rustfmt does/plans to do.
Moving things into separate files works with Javascript because the Javascript ecosystem has been designed around preprocessors (e.g. module bundlers, tools to rewrite more recent syntax using older syntax, etc) and the there are therefore a number of libraries designed to work as modular parts for this sort of use case (e.g. babel which essentially offers you things like parsers, etc that you need to build these sort of systems) but I don’t think it’s a good fit for Rust.

On 7 Nov 2020, at 13:09, Joseph Quinn<EMAIL_ADDRESS>wrote:
@siku2 https://github.com/siku2 I can't speak as to the original poster's intent, but my desire would be for some macro that does the work of reading the desired html file and running the html! macro on those contents. Admittedly, I'm not the best with writing Rust macros, but I would envision that a macro that simply reads the file to an &str and then calls the html macro (it should just be feeding it to the html function in the yew-macro crate, right?) on the resultant is more than sufficient for what I would envision to be a first step. In terms of looking at it from a webpack/rollup/parcel standpoint, Yew would just be implementing a require() and/or import() function.
I'm using VSCode for development and tried out the html extensions I have loaded with Yew's syntax and it behaved as expected: no difficulties with adapting to Yew-flavored HTML as if it was just handlerbars/twig/jinja2/lodash templates/every other templating language under the sun. The only pain point was the lack of Rust autocomplete, but errors in any embedded Yew-flavored HTML should be caught at compile time, no?
Finally, referring to @jstarry https://github.com/jstarry 's comment on the previous pull request for this feature request: does Yew really need explicit code editor experience improvements when current solutions are more than sufficient. I don't think anyone really wants to build and maintain a Yew-flavored Rust plugin for all the text editors and IDEs that are available, nor do I think that there's enough distinction in Yew-flavored HTML to warrant adding a .html.yew file extension to the world for editors/IDEs to separate vanilla HTML from Yew-flavored HTML when the current offerings really do what's needed to make the developer experience nicer.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/yewstack/yew/issues/1441#issuecomment-723444387, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFSTPNMHCTZWJL67YRK36LSOVBJRANCNFSM4PF5KPDA.

I can't speak as to the original poster's intent, but my desire would be for some macro that does the work of reading the desired html file and running the html! macro on those contents. Admittedly, I'm not the best with writing Rust macros, but I would envision that a macro that simply reads the file to an &str and then calls the html macro (it should just be feeding it to the html function in the yew-macro crate, right?) on the resultant is more than sufficient for what I would envision to be a first step.

This was my intent when I created this issue but after using implementation from @teymour-aldridge's fork of it, I realized that Rust code in html files doesn't play nice, especially with linters. To give you an an example, IntelliJ shows me this for a simple event listener:

These are the errors/warnings pointed out by the inspector:

The only pain point was the lack of Rust autocomplete, but errors in any embedded Yew-flavored HTML should be caught at compile time, no?

They can be, but spans become an issue. The reported error doesn't tell you where the error is, or at least that's what I've observed while working with the aforementioned fork. It also broke incremental compilation which was a major pain point.
If these problems can be resolved, I believe that file-based markup would be a welcome improvement. With respect to editor support, the best solution would be have the the syntax of html! supported via a plugin but I'm not aware of any of way to add that functionality on top of intellij-rust (I've been exclusively using IntelliJ so I don't know about state of other editors)

@teymour-aldridge I disagree with your assessment of modularization of html from models/controllers/view-models. Preprocessing like you've said with javascript has become the fad of the modern web, but preprocessing has been a part of the web, with modularization, since ye old good days of Perl and the initial introduction of PHP. Rust already has other templating libraries, such as a port of handlebars and jinja2 (which I'm currently monkeying with to see if I can get to play well with Yew). I don't agree that the idea of building off template files is not a good fit for Rust. Rather, it's a matter of getting the tooling right to do so with macro expansion rather than having to load the template compilers into the wasm bundle.
@hamza1311 I'm using the Rust team's plugins for VSCode and Neovim. I doubt we'd be able to get them to add support for the html! macro as a part of the main Rust plugins, nor do I know of a way to get plugins for VSCode or Neovim to alter their behavior based on a single keyword.
And just to make sure I'm on the same boat, is breaking incremental compilation due to having to manually trigger recompilation when an html file is altered? Or is it all benefits of incremental compilation?

@teymour-aldridge https://github.com/teymour-aldridge I disagree with your assessment of modularization of html from models/controllers/view-models. Preprocessing like you've said with javascript has become the fad of the modern web, but preprocessing has been a part of the web, with modularization, since ye old good days of Perl and the initial introduction of PHP. Rust already has other templating libraries, such as a port of handlebars https://crates.io/crates/handlebars and jinja2 https://crates.io/crates/tera (which I'm currently monkeying with to see if I can get to play well with Yew). I don't agree that the idea of building off template files is not a good fit for Rust. Rather, it's a matter of getting the tooling right to do so with macro expansion rather than having to load the template compilers into the wasm bundle.

I feel like it’s better to minimise the use of macros and make use of the already very expressive nature Rust has –  I feel like macros are an escape hatch that we don’t need to use, because the language provides all the necessary features.

On 8 Nov 2020, at 11:20, Joseph Quinn<EMAIL_ADDRESS>wrote:
@teymour-aldridge https://github.com/teymour-aldridge I disagree with your assessment of modularization of html from models/controllers/view-models. Preprocessing like you've said with javascript has become the fad of the modern web, but preprocessing has been a part of the web, with modularization, since ye old good days of Perl and the initial introduction of PHP. Rust already has other templating libraries, such as a port of handlebars https://crates.io/crates/handlebars and jinja2 https://crates.io/crates/tera (which I'm currently monkeying with to see if I can get to play well with Yew). I don't agree that the idea of building off template files is not a good fit for Rust. Rather, it's a matter of getting the tooling right to do so with macro expansion rather than having to load the template compilers into the wasm bundle.
@hamza1311 https://github.com/hamza1311 I'm using the Rust team's plugins for VSCode and Neovim. I doubt we'd be able to get them to add support for the html! macro as a part of the main Rust plugins, nor do I know of a way to get plugins for VSCode or Neovim to alter their behavior based on a single keyword.
And just to make sure I'm on the same boat, is breaking incremental compilation due to having to manually trigger recompilation when an html file is altered? Or is it all benefits of incremental compilation?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/yewstack/yew/issues/1441#issuecomment-723563144, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFSTPJLL5GSJYWZSJOWPZTSOZ5GZANCNFSM4PF5KPDA.

@teymour-aldridge

I feel like it’s better to minimise the use of macros and make use of the already very expressive nature Rust has –  I feel like macros are an escape hatch that we don’t need to use, because the language provides all the necessary features.

Thinking solely in the context of generating the VDom for Yew here, but the only other option for doing so would be to ship a live template rendering engine alongside the application code which would be extremely costly for performance. On top of that, you're looking at even more performance overhead with the current 'Yew way' of breaking larger partials into smaller functions. Even if a templating system for Yew was made asynchronous, it'd be slower than preprocessing the html tokens to its Rust representation during compilation, especially on limited-resource devices.

At the end of the day, everything has to end up as Rust code and I think we could provide better abstractions over that code so that you could just construct a series of function calls which would be inlined and optimised away by the compiler – there’s nothing that specifically requires the use of a macro. I haven’t investigated this very much, but it’s something that (if I every get time) I’d like to look at.

On 8 Nov 2020, at 16:04, Joseph Quinn<EMAIL_ADDRESS>wrote:
@teymour-aldridge https://github.com/teymour-aldridge
I feel like it’s better to minimise the use of macros and make use of the already very expressive nature Rust has – I feel like macros are an escape hatch that we don’t need to use, because the language provides all the necessary features.
Thinking solely in the context of generating the VDom for Yew here, but the only other option for doing so would be to ship a live template rendering engine alongside the application code which would be extremely costly for performance. On top of that, you're looking at even more performance overhead with the current 'Yew way' of breaking larger partials into smaller functions. Even if a templating system for Yew was made asynchronous, it'd be slower than preprocessing the html tokens to its Rust representation during compilation, especially on limited-resource devices.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/yewstack/yew/issues/1441#issuecomment-723604812, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFSTPLTL2WKJ6WCKX3KB5DSO26RVANCNFSM4PF5KPDA.

At the end of the day, everything has to end up as Rust code and I think we could provide better abstractions over that code so that you could just construct a series of function calls which would be inlined and optimised away by the compiler – there’s nothing that specifically requires the use of a macro.

We're stating to get off topic, but inlining functions and macro expansion are two very different things. Inlining a function is only going to replace the function call at the site during compilation; it doesn't invoke the function call to create. As well, generally, inlining improperly will be a negative for performance, even with compiler optimizations.
As I said, using functions for generating Yew's VDom at runtime is going to be a major performance hit. Instead of doing a macro expansion at compile time such as:

You'll need to do all of that conversion from html tokens or an html string during runtime. That brings into question additions of new dependencies for handling the conversion, whether or not its safe to do the conversion of tokens in a live browser session (even sandboxed), and discussions about performance. And that list isn't even close to comprehensive.

I may not be communicating things very clearly here – I think that it should be possible to write some functions wrapping the virtual DOM which have no (or very limited) performance cost and allow one to write Rust code instead of some Rust-HTML hybrid.
Something like “hyperscript?” https://github.com/hyperhype/hyperscript
I may be wrong here :)

On 8 Nov 2020, at 17:14, Joseph Quinn<EMAIL_ADDRESS>wrote:
At the end of the day, everything has to end up as Rust code and I think we could provide better abstractions over that code so that you could just construct a series of function calls which would be inlined and optimised away by the compiler – there’s nothing that specifically requires the use of a macro.
We're stating to get off topic, but inlining functions and macro expansion are two very different things. Inlining a function is only going to replace the function call at the site during compilation; it doesn't invoke the function call to create. As well, generally, inlining improperly will be a negative for performance, even with compiler optimizations.
As I said, using functions for generating Yew's VDom at runtime is going to be a major performance hit. Instead of doing a macro expansion at compile time such as:
https://camo.githubusercontent.com/8c3ea24b285a5d1d73ddc4e72b010c79e248b26bf11f462bf4d046f9366a8169/68747470733a2f2f692e696d6775722e636f6d2f375850675630792e706e67
You'll need to do all of that conversion from html tokens or an html string during runtime. That brings into question additions of new dependencies for handling the conversion, whether or not its safe to do the conversion of tokens in a live browser session (even sandboxed), and discussions about performance. And that list isn't even close to comprehensive.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/yewstack/yew/issues/1441#issuecomment-723638983, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFSTPJXXWHP4EK73OGIT4TSO3GXLANCNFSM4PF5KPDA.

Something like “hyperscript?” https://github.com/hyperhype/hyperscript

Looking at hyperscript, it's essentially just a functional-looking wrapper for document.createElement.
That's, again, a runtime hit, but on the javascript side. You're looking at something that will perform worse than static html, especially at the scale of a modern webapp.
@teymour-aldridge maybe we should split future discussion of this off into another feature request thread? We're starting to really venture off the topic of somehow implementing reading html into Yew's html! macro now.

Agreed :)
This sort of discussion is also not ideal for a Github issue thread, but definitely worth having.

On 8 Nov 2020, at 18:36, Joseph Quinn<EMAIL_ADDRESS>wrote:
Something like “hyperscript?” https://github.com/hyperhype/hyperscript https://github.com/hyperhype/hyperscript
Looking at hyperscript, it's essentially just a functional-looking wrapper for document.createElement.
That's, again, a runtime hit, but on the javascript side. You're looking at something that will perform worse than static html, especially at the scale of a modern webapp.
@teymour-aldridge https://github.com/teymour-aldridge maybe we should split future discussion of this off into another feature request thread? We're starting to really venture off the topic of somehow implementing reading html into Yew's html! macro now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/yewstack/yew/issues/1441#issuecomment-723648740, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFSTPITADR2JOQRI3XD6BLSO3QJLANCNFSM4PF5KPDA.

@teymour-aldridge It'd be better to open another thread so the project has a written, searchable record of what was discussed.

So I think the broad conclusion was that this is not practical to do.
@quinnjr I've written a small crate called Malvolio (https://github.com/lovelace-ed/lovelace – look in utils/malvolio) which might give you a better idea as to what I'm thinking of.
I agree that if we were to read from a file this would need to be done with a macro for performance reasons.
I guess we can close this?

So I think the broad conclusion was that this is not practical to do.
@quinnjr I've written a small crate called Malvolio (https://github.com/lovelace-ed/lovelace – look in utils/malvolio) which might give you a better idea as to what I'm thinking of.
I agree that if we were to read from a file this would need to be done with a macro for performance reasons.
I guess we can close this?

I guess we can close this?

Agreed

I guess we can close this?

Agreed


Typo in Eclipse Phase tooltip

https://github.com/Roll20/roll20-character-sheets/blob/8eeb3a691dd8ef54d1b4e93bd6e38e52d74c3ff0/Eclipse Phase V2/Eclipse Phase V2.html#L1571
It should read "Wound Threshold" instead of "Wound Tresold"

@Zakarik This is a minor issue, but perhaps the next time you update this sheet you might include a fix for this typo?


[Proposal] Rename Row class

If you use sqlite3 inside flutter App, you could have conflicts name between Flutter.Row and Sqlite3.Row.

Do you think it could be a good idea to rename you Row class into SqlRow or RowSql for example ?

In general my take is that users should use hide/show directives to resolve this:
import 'package:flutter/material.dart' hide Row;

Row is a good class name for a row, so I don't think renaming it would be worth it.


Bug: pydantic imports from aws_lambda_powertools.utilities.parser.pydantic no longer available

Expected Behaviour
As per the documentation FAQ I would expect to be able to import X from pydantic to include types such as EmailStr
Current Behaviour
Currently in 3.0.0 we are unable to import pydantic types via the parser.
Code snippet
from aws_lambda_powertools.utilities.parser.pydantic import EmailStr

Possible Solution
Update documentation to include the recommended way to import X from pydantic, or implement the feature that was available in 2.X.
Steps to Reproduce
Try and import a pydantic type from aws_lambda_powertools.utilities.parser.pydantic, which no longer exists.
Powertools for AWS Lambda (Python) version
latest
AWS Lambda function runtime
3.11
Packaging format used
PyPi
Debugging logs
No response

Hey @RyanFrench, thanks for opening this issue to report this behavior.
In Powertools v3 we have dropped support for Pydantic v1 and are only supporting Pydantic v2 which is why we are no longer exporting pydantic types directly and allowing customers to import them directly from Pydantic. First, I must say that I agree with your suggestion that we need to improve our documentation and upgrade guide to make it clear to customers, but I don't think we should revert this change for a few reasons:
1 - Pydantic V2 was a complete refactoring of the Pydantic project and they made severalchanges including, but not limited, to the way they export types, classes, methods, and others. If we continue to expose them directly from Powertools and some export in Pydantic has changed, our customers may get the impression that there is some bug in Powertools and that is not the experience we want to have for our customers.
2 - The way we were doing this was by exposing everything exposed from pydantic to our customers, but there is a long discussion in the Pydantic repository about how Pydantic v2 is slower than v1 when importing and loading things in this new version. We don't want our customers to inadvertently import everything from Pydantic and this could create some slowdown in their functions.
I will be sending a PR today to update our Upgrade Guide and documentation to clarify this change. I will ping you on this PR to get your feedback - if possible.
Thank you very much for bringing this up, I believe that improving our documentation will prevent other customers from having the same issue.

Thank you, I assumed as much but wasn't sure what the new recommended way was to use other pydantic classes within the parser.


⚠ gorzdrav.org has degraded performance

In eec20ef, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 2739 ms

Resolved: gorzdrav.org performance has improved in 45ff016.


host approval emails are only being sent for the open source collective

closing this since @xdamman deployed a fix today


SystemEvents.UserPreferenceChanged cannot be triggered

Describe the bug
I tried to attach an event handler to SystemEvents.UserPreferenceChanged. However, it seems that the event is not triggered for the WinUI 3 desktop app using the Windows App SDK.
Steps to reproduce the bug
You may just create a simple project with a main window and a main page on it. For the page_OnLoaded event, add:
using Microsoft.Win32;

# OnLoaded
SystemEvents.UserPreferenceChanged += SystemEventsOnUserPreferenceChanged;

private void SystemEventsOnUserPreferenceChanged(object sender, UserPreferenceChangedEventArgs e)
{
    throw new NotImplementedException();
}

# OnUnloaded
SystemEvents.UserPreferenceChanged -= SystemEventsOnUserPreferenceChanged;

Expected behavior
No response
Screenshots
No response
NuGet package version
Windows App SDK 1.2.1: 1.2.221116.1
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

A workaround is to handle WM_SETTINGCHANGE
(tested on Windows 10 21H1)

A workaround is to handle WM_SETTINGCHANGE (tested on Windows 10 21H1)

Could you show me your code?

With SetWindowSubclass, like in some samples I posted (MainWindow.xaml.cs for declarations)
on main window handle hWnd  :
   hWnd = WinRT.Interop.WindowNative.GetWindowHandle(this);
   SubClassDelegate = new SUBCLASSPROC(WindowSubClass);
   bool bReturn = SetWindowSubclass(hWnd, SubClassDelegate, 0, 0);

with (I tested by changing System colors and I get : "Settings = ImmersiveColorSet") :
    public const int WM_WININICHANGE = 0x001A;
    public const int WM_SETTINGCHANGE = WM_WININICHANGE;

    private int WindowSubClass(IntPtr hWnd, uint uMsg, IntPtr wParam, IntPtr lParam, IntPtr uIdSubclass, uint dwRefData)
    {
        switch (uMsg)
        {
            case WM_SETTINGCHANGE:
                {
                    // Console.Beep(6000, 10);
                    string sText = Marshal.PtrToStringUni(lParam);
                    System.Diagnostics.Debug.WriteLine(string.Format("Settings = {0}", sText));
                }
                break;
        }
        return DefSubclassProc(hWnd, uMsg, wParam, lParam);
    }

The events of SystemEvents class cannot be raised because when the SystemEventInvokeInfo object captures the SynchronizationContext of the current thread, the SynchronizationContext.Send is called when RaiseEvent, and DispatcherQueueSynchronizationContext has not implemented the Send method.
Solution:
SystemEvents.InvokeOnEventsThread(() =>
{
    // No need to set, the default is null
    //SynchronizationContext.SetSynchronizationContext(null);

    SystemEvents.DisplaySettingsChanged += (s, a) =>
    {
        Debug.WriteLine("DisplaySettingsChanged");
    };
});


Exact ival-fabs ival-neg

This branch introduces two new function called ival-exact-fabs and ival-exact-neg. These functions work the same as their "inexact" copies - the main difference is - the result of the new functions is evaluated under the precision of input.
Some functions require exact versions for correctness, they are: ival-fmod, ival-remainder and etc.

@pavpanchekha test.rkt are adjusted for the new ival-mod feature

Looks good, merge whenever you want.


Stateful extension icon

idea was shared on twitter to change the visbug icon to the open winged logo when the tool is active 👍

would also be nice to have a dark mode supported icon
api for the task is here: https://developer.chrome.com/extensions/browserAction#method-setIcon

Could you add a version of the open-winged logo to the project? I poked around a bit but didn't see a version that could be used for this. Same for the dark mode icons.

Here's my Figma file of the logo (i think you can hack on this artboard, lemme know). There's open winged versions in there, i think i stopped hacking on this because the open winged broke out of the icon space. i dont remember! carry the torch yo!

hm, i think instead of wings open, close icon could be better. make it really obvious it's for closing the plugin, and would make it easy to spot. thoughts?


Add flake.nix to support building this project under nix.

Adds in a flake.nix file that allows users of the Nix package manager to build and install nwg-displays via the Flakes feature.

Hope you know what you're doing, as I know nothing about NixOS. Do you need a new release, when it's merged?

This shouldn't need a new release. This PR is only allowing nix users to take a dependency on the project. It is not changing nwg-displays at all so I don't think it's appropriate to do any sort of release/version bump. Versioning of the Nix side of things will happen via the git sha if someone takes a dependency on this anyways.
I actually had this as a separate repository for my own personal use. I figured it would be useful for a wider audience and it made the most sense to have it be in the project's repository. That way users don't have to find some random different repository.
Thanks for making this tool by the way. I was coding up my display configs by hand for sway and that was definitely tedious. Having a GUI to manage this is a big improvement.

Thank you!
Sure, if you use just the master branch, I'll merge and leave it as is for now.


Optional semicolon

Not all linters like line ending semicolons. Add a flag or something that omits the semicolon from the created version.js.

Because standardjs forces no semicolons, let the default template be minimal and without semicolon. Create a flag --with-colon or -c to add semicolon.

On the other hand, to avoid major release, use semicolon by default and create a flag --no-semicolon or -n.

Yet another approach: automatic. Omit semicolon if standardjs is a dependency. But, keep it simple and clear, thus --semicolon or -s is the best and final.

Implemented in ae30b2b9dd2727d5107c202f4adb44730862a35f


NEXT-7416 - Prevent double submit in checkout

Fixes shopwareBoostDay/platform#82

1. What does this change do, exactly?
2. Describe each step to reproduce the issue or behaviour.
3. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

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

Thanks for you contribution @claudiobianco I'll push this through our internal gitlab pipeline, because this is also a high priority issue for our next release.

We've merged it into shopware/platform: https://github.com/shopware/platform/commit/451b92be69732818b2c88955d5b3a6ab45db43c9


Return eos_token from decoding algos

Waiting for https://github.com/openvinotoolkit/openvino_tokenizers/pull/160

You can migrate to latest 2024.2 OpenVINO Tokenizers branch.


Disable regression test for #3070 on GCC <8.4

Disables part of the regression test for #3070 on GCC <8.4 (already disabled for MSVC).
Discussed here.
Once again, the ambiguous overload resolution isn't something we can fix (easily, if at all).

Coverage remained the same at 100.0% when pulling 7cd3120e7f147d18a79236cf41eae7c3ea9984eb on falbrechtskirchinger:disable-test-3070-gcc-84 into a94430615d8360272151f602b8c9eeb58509ecde on nlohmann:develop.

Thanks!

Currently building on Debian GNU/Linux 10 (buster) here, and hence stumbled over this. Will this be backported to the 3.10.x branch?

No, we will not backport changes. This will be fixed in the 3.11.0 release.


accuracy calculation

Is their any command to get accuracy of model that I have created using tensorflow 1.13 for custom object.

There isn't a accuracy metric like for image classification for object detection. You can check out the available metrics by taking a look at the evaluation_protocols.md file: https://github.com/tensorflow/models/blob/master/research/object_detection/g3doc/evaluation_protocols.md


By default reuse previously rendered element when the route is activated

Improve performance
Must be sure element properties are updated
Common example is when only the param of a route changes, e.g., from 'contacts/1' to 'contacts/2'

Not sure is good idea. This can lead to wrong DOM tree when route is not a leaf


IOU difference between single and multi-GPU evaluation when testing DeepLabv3 on COCO

🐛 Describe the bug
Hi, I'm starting to work on segmentation, and I'm running a quick test with DeepLabv3 with ResNet50 backbone on the COCO dataset. I have access to a server with 4 GPUs. I tried to reproduce the IOU of the pretrained model by running the following:
torchrun --nproc_per_node=4 train.py --dataset coco --model deeplabv3_resnet50 --weights DeepLabV3_ResNet50_Weights.COCO_WITH_VOC_LABELS_V1 --test-only --aux-loss

However, I only get IOU = 63.8, which is significantly smaller than the 66.4 I should get. The weird thing is that I can get exactly 66.4 by switching to single GPU evaluation, i.e. by setting --nproc_per_node=1. Any idea why this happens? Am I doing something wrong? Thank you in advance!
Versions
PyTorch version: 1.13.1
Is debug build: False
CUDA used to build PyTorch: 11.6
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.4 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: version 3.16.3
Libc version: glibc-2.31
Python version: 3.10.9 (main, Jan 11 2023, 15:21:40) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-5.4.0-135-generic-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: 11.6.124
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA RTX A6000
GPU 1: NVIDIA RTX A6000
GPU 2: NVIDIA RTX A6000
GPU 3: NVIDIA RTX A6000
Nvidia driver version: 510.108.03
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Byte Order:                      Little Endian
Address sizes:                   46 bits physical, 57 bits virtual
CPU(s):                          96
On-line CPU(s) list:             0-95
Thread(s) per core:              2
Core(s) per socket:              24
Socket(s):                       2
NUMA node(s):                    2
Vendor ID:                       GenuineIntel
CPU family:                      6
Model:                           106
Model name:                      Intel(R) Xeon(R) Gold 6336Y CPU @ 2.40GHz
Stepping:                        6
Frequency boost:                 enabled
CPU MHz:                         800.078
CPU max MHz:                     2401.0000
CPU min MHz:                     800.0000
BogoMIPS:                        4800.00
Virtualization:                  VT-x
L1d cache:                       2.3 MiB
L1i cache:                       1.5 MiB
L2 cache:                        60 MiB
L3 cache:                        72 MiB
NUMA node0 CPU(s):               0-23,48-71
NUMA node1 CPU(s):               24-47,72-95
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Mmio stale data:   Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Retbleed:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Enhanced IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 invpcid_single ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb intel_pt avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local wbnoinvd dtherm ida arat pln pts avx512vbmi umip pku ospke avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg tme avx512_vpopcntdq rdpid md_clear pconfig flush_l1d arch_capabilities
Versions of relevant libraries:
[pip3] numpy==1.24.2
[pip3] torch==1.13.1
[pip3] torch-pruning==1.0.0
[pip3] torchaudio==0.13.1
[pip3] torchvision==0.14.1
[conda] blas                      1.0                         mkl
[conda] ffmpeg                    4.3                  hf484d3e_0    pytorch
[conda] mkl                       2021.4.0           h06a4308_640
[conda] mkl-service               2.4.0           py310h7f8727e_0
[conda] mkl_fft                   1.3.1           py310hd6ae3a3_0
[conda] mkl_random                1.2.2           py310h00e6091_0
[conda] numpy                     1.24.2                   pypi_0    pypi
[conda] numpy-base                1.23.5          py310h8e6c178_0
[conda] pytorch                   1.13.1          py3.10_cuda11.6_cudnn8.3.2_0    pytorch
[conda] pytorch-cuda              11.6                 h867d48c_1    pytorch
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] torch-pruning             1.0.0                    pypi_0    pypi
[conda] torchaudio                0.13.1              py310_cu116    pytorch
[conda] torchvision               0.14.1              py310_cu116    pytorch

The weird thing is that I can get exactly 66.4 by switching to single GPU evaluation, i.e. by setting --nproc_per_node=1. Any idea why this happens?

Yes, unfortunately that's expected, and we recommend using only 1 GPU when precise evaluation is required. Using multiple GPUs with DDP results in bigger effective batch size, and some operation are actually really sensitive to batching - i.e. you'll get different results when batching 2 inputs vs when evaluating them separately. Those (sometimes not so) tiny differences add up, and that's how we can end up with such significant differences in results.


PHP 8 support

After creating #10, I figured the best way to check what issues exist for PHP 8 would be to run CI.
Closes: #10

Looks like we need to upgrade to phpunit 8.5 to get PHP 8.0 support. Also the PHP 7.3 and 7.4 builds appear to be using xdebug 3?

Thanks @ndench for this PR. Are you planning to come back and look at the issues? I may also be able to put sometime on it this week to find, fix and release

Closes #10 #11

@AlaaSarhan I'm happy to fix up issues, I just wanted to get your input first. Because it appears that we may have to drop support for PHP 5.6 and I thought there might be a bit of work in upgrading PHPUnit. Your PR looks pretty good though. I'll leave some comments on it.

@AlaaSarhan after pulling in the changes you made in #12 I just had increase the minimum PHP version to 7.1 to make everything play nicely. Hope this is ok.


Fix comet depends_on in docker-compose.yml

Proposed changes
This PR fixes comet depends_on in docker-compose.yml.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

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

Further comments
N/A

Codecov Report

Merging #90 (791ae72) into main (8e46d2f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   99.57%   99.57%
=======================================
  Files           2        2
  Lines        1166     1169    +3
=======================================
+ Hits         1161     1164    +3
  Misses          5        5

Impacted Files
Coverage Δ

lets-fiware.sh
99.53% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e46d2f...791ae72. Read the comment docs.


Publish EAO Profile Page

DOD: All approvals for launch of EAO Profile Page have been obtained and page launched.

EAO profile page is not in the current sprint.  Moving to product backlog.

Readme file needs to be updated before profile page goes live. This will not happen until after the sprint, so moving this story back to the backlog.


refactor: OPTIC-482: Archive feature flag fflag_feat_optic_2_ensure_draft_saved_short

Remove feature flag fflag_feat_optic_2_ensure_draft_saved_short setting the value to true:
As usual, I recoment review the changes without whitespace diff:
https://github.com/HumanSignal/label-studio/pull/5673/files?diff=split&w=1
https://app.launchdarkly.com/default/production/features/fflag_feat_optic_2_ensure_draft_saved_short/targeting
Original ticket: https://humansignal.atlassian.net/browse/OPTIC-2

PR fulfills these requirements

[ ] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[ ] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Backend (Database)
[ ] Backend (API)
[ ] Frontend

Describe the reason for change
(link to issue, supportive screenshots etc.)
What does this fix?
(if this is a bug fix)
What is the new behavior?
(if this is a breaking or feature change)
What is the current behavior?
(if this is a breaking or feature change)
What libraries were added/updated?
(list all with version changes)
Does this change affect performance?
(if so describe the impacts positive or negative)
Does this change affect security?
(if so describe the impacts positive or negative)
What alternative approaches were there?
(briefly list any if applicable)
What feature flags were used to cover this change?
(briefly list any if applicable)
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
(for bug fixes/features, be as precise as possible. ex. Authentication, Annotation History, Review Stream etc.)

@bmartel I can use your help.
I believe I may need to adjust label-studio-private as well. I can't find any chores here.

Thanks @bmartel for help with the repo I need to update.

/git merge

/git merge

@hlomzik, I don't believe maintaining two branches of code FF on/off will positively help to address the issues with drafts

@hlomzik will take care in other current ongoing efforts.


No sample for Single Page Application?

I want to migrate my MVC5 SPA to MVC6,but for Asp.net Identity 3 look like totally different from previous version.
Currently only have cookie-based sample,where I can get the token-based one so that I can make it clear.
Thanks!

Dupe of https://github.com/aspnet/Identity/issues/193


You have no release tags for anything after 8.6.2

All the releases after 8.6.2 are development releases and will never have release tags.
Perhaps I should change to a version numbering where the number indicates whether the release is official or developmental.  Perl uses even major numbers for official releases, odd for development
@dabrahams: Do you have a suggestion?  The Perl scheme has the advantage that I'm familiar with it, and the disadvantage that, despite my familiarity, I've already scewed up. :-)

Thanks for explaining; I have no particular suggestions.


(PUP-7372) Confine Hiera 3 interpolations to the global layer

This commit ensures that a data file that is reached using a Hiera 3
configuration and that in turn contains interpolation expressions that
use the 'hiera' or 'lookup' function, will have that lookup confined
to the global layer unless an environment data provider, version 5
is present. It also ensures that when an interpolation that originas
from Hiera 3 is calling out, that the special Hiera::Scope class
is unwrapped.
This was not the case earlier which resulted in that the special class
Hiera::Scope was passed on to the environment and module layers which
resulted in errors as they tried to access the Scope#environment
method which is not present in that class. Other side-effects of passing
that scope would be to make the special variables that it contains
available to data configured using Hiera 5.

interpolation needs to follow the same rules as the hiera_xxx calls wrt drilling down into other layers (an hiera.yaml in the env is taken as opt in to drill down)

good to merge when tests go green


Pleroma proxy setup

Are you in the right place?

If you are looking for support on how to get your upstream server forwarding, please consider asking the community on Reddit.
I have tried asking on Reddit: https://www.reddit.com/r/nginxproxymanager/comments/qlaam6/
However, I have received no answer as of yet. I replaced the

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
Hello there!
I have recently installed Pleroma on my Raspberry Pi and have now used a different Raspberry Pi to reverse proxy it using the Reverse Proxy Manager. However, the actual setup requires a much more complex and different setup than what the Nginx Reverse Proxy Manager can deliver via the user interface. I began using the reverse proxy manager because back then I didn't understand anything about hosting and nginx, nowadays, I understand quite a bit, at least enough to be able to ssh into the server and edit a config file myself. Is there any way to do that with the Nginx Reverse Proxy Manager?
This is my complex Nginx setup that I need:
    # default nginx site config for Pleroma
    #
    # Simple installation instructions:
    # 1. Install your TLS certificate, possibly using Let's Encrypt.
    # 2. Replace 'example.tld' with your instance's domain wherever it appears.
    # 3. Copy this file to /etc/nginx/sites-available/ and then add a symlink to it
    #    in /etc/nginx/sites-enabled/ and run 'nginx -s reload' or restart nginx.

    proxy_cache_path /tmp/pleroma-media-cache levels=1:2 keys_zone=pleroma_media_cache:10m max_size=10g
                     inactive=720m use_temp_path=off;

    # this is explicitly IPv4 since Pleroma.Web.Endpoint binds on IPv4 only
    # and `localhost.` resolves to [::0] on some systems: see issue #930
    upstream phoenix {
        server <IP_ADDRESS>:5000 max_fails=5 fail_timeout=60s;
    }

    server {
        server_name    social.uden.ai;

        listen         80;
        listen         [::]:80;

        # Uncomment this if you need to use the 'webroot' method with certbot. Make sure
        # that the directory exists and that it is accessible by the webserver. If you followed
        # the guide, you already ran 'mkdir -p /var/lib/letsencrypt' to create the folder.
        # You may need to load this file with the ssl server block commented out, run certbot
        # to get the certificate, and then uncomment it.
        #
        # location ~ /\.well-known/acme-challenge {
        #     root /var/lib/letsencrypt/;
        # }
        location / {
          return         301 https://$server_name$request_uri;
        }
    }

    # Enable SSL session caching for improved performance
    ssl_session_cache shared:ssl_session_cache:10m;

    server {
        server_name social.uden.ai;

        listen 443 ssl http2;
        listen [::]:443 ssl http2;
        ssl_session_timeout 1d;
        ssl_session_cache shared:MozSSL:10m;  # about 40000 sessions
        ssl_session_tickets off;

        ssl_trusted_certificate   /etc/letsencrypt/live/example.tld/chain.pem;
        ssl_certificate           /etc/letsencrypt/live/example.tld/fullchain.pem;
        ssl_certificate_key       /etc/letsencrypt/live/example.tld/privkey.pem;

        ssl_protocols TLSv1.2 TLSv1.3;
        ssl_ciphers "ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA384:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";
        ssl_prefer_server_ciphers off;
        # In case of an old server with an OpenSSL version of 1.0.2 or below,
        # leave only prime256v1 or comment out the following line.
        ssl_ecdh_curve X25519:prime256v1:secp384r1:secp521r1;
        ssl_stapling on;
        ssl_stapling_verify on;

        gzip_vary on;
        gzip_proxied any;
        gzip_comp_level 6;
        gzip_buffers 16 8k;
        gzip_http_version 1.1;
        gzip_types text/plain text/css application/json application/javascript text/xml application/xml application/xml+rss text/javascript application/activity+json application/atom+xml;

        # the nginx default is 1m, not enough for large media uploads
        client_max_body_size 16m;
        ignore_invalid_headers off;

        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection "upgrade";
        proxy_set_header Host $http_host;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;

        location / {
            proxy_pass http://phoenix;
        }

        location ~ ^/(media|proxy) {
            proxy_cache        pleroma_media_cache;
            slice              1m;
            proxy_cache_key    $host$uri$is_args$args$slice_range;
            proxy_set_header   Range $slice_range;
            proxy_cache_valid  200 206 301 304 1h;
            proxy_cache_lock   on;
            proxy_ignore_client_abort on;
            proxy_buffering    on;
            chunked_transfer_encoding on;
            proxy_pass         http://phoenix;
        }
    }

I like the fact that Nginx takes care of my SSL certificates. I like the fact that it "blocks common exploits" or "caches assets" however, I would really like for my service to work and this is the setup it will need for that.
This is how the file (in nginx/data/nginx/proxy\_host/28.conf) looks like:
    # ------------------------------------------------------------
    # social.uden.ai
    # ------------------------------------------------------------

    server {
      set $forward_scheme http;
      set $server         "<IP_ADDRESS>";
      set $port           5000;

      listen 80;
    listen [::]:80;

    listen 443 ssl http2;
    listen [::]:443;

      server_name social.uden.ai;

      # Let's Encrypt SSL
      include conf.d/include/letsencrypt-acme-challenge.conf;
      include conf.d/include/ssl-ciphers.conf;
      ssl_certificate /etc/letsencrypt/live/npm-73/fullchain.pem;
      ssl_certificate_key /etc/letsencrypt/live/npm-73/privkey.pem;

    # Asset Caching
      include conf.d/include/assets.conf;

      # Block Exploits
      include conf.d/include/block-exploits.conf;

      access_log /data/logs/proxy_host-28.log proxy;

    location /api/fedsocket/v1 {
            proxy_request_buffering off;
            proxy_pass http://<IP_ADDRESS>:5000/api/fedsocket/v1;
        }

        location / {
                proxy_pass http://<IP_ADDRESS>:5000;
        }

      # Custom
      include /data/nginx/custom/server_proxy[.]conf;
    }

This is what I added myself through the interface.
    location /api/fedsocket/v1 {
            proxy_request_buffering off;
            proxy_pass http://<IP_ADDRESS>:5000/api/fedsocket/v1;
        }

        location / {
                proxy_pass http://<IP_ADDRESS>:5000;
        }

It made some of the important features work, but sadly not the most important.
Obviously I could replace the SSL section with the one provided by the Nginx reverse proxy manager, right?

Please help me! You are my only hope. Any comment, question or help is appreciated!
Nginx Proxy Manager Version
v2.8.1
To Reproduce
Steps to reproduce the behavior:

SSH into your host machine
From wherever you installed nginx, go to /nginx/data/nginx/proxy_host
Go to the newest config file, or create a new one
Paste the configuration that I've specified above
See error

Expected behavior
It should work and reverse proxy my Pleroma instance
Operating System
I am on arm64 on my Raspberry Pi 4b 4GB.
Additional context
I just need the exact configuration specified here: https://docs-develop.pleroma.social/backend/installation/otp_en/#edit-the-nginx-config to work on my system. However, Pleroma isn't hosted on the same Raspberry Pi as is the Nginx Reverse Proxy Manager, so it needs some editions (localhost needs to be changed to an IP address, etc.)

What "most important feature" haven't you got working? SSL should work out of the box when you have created the proxy host and associated a certificate with it.
You can add these headers via the advanced config:
    ssl_session_timeout 1d;
    ssl_session_cache shared:MozSSL:10m;  # about 40000 sessions
    ssl_session_tickets off;

    ssl_protocols TLSv1.2 TLSv1.3;
    ssl_ciphers "ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA384:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";
    ssl_prefer_server_ciphers off;

    ssl_stapling on;
    ssl_stapling_verify on;

    gzip_vary on;
    gzip_proxied any;
    gzip_comp_level 6;
    gzip_buffers 16 8k;
    gzip_http_version 1.1;
    gzip_types text/plain text/css application/json application/javascript text/xml application/xml application/xml+rss text/javascript application/activity+json application/atom+xml;

    # the nginx default is 1m, not enough for large media uploads
    client_max_body_size 16m;
    ignore_invalid_headers off;

This snippet
proxy_cache_path /tmp/pleroma-media-cache levels=1:2 keys_zone=pleroma_media_cache:10m max_size=10g
                 inactive=720m use_temp_path=off;

would have to be written in a file on your host and mounted into the docker container at /data/nginx/custom/http_top.conf.
Then you can also add a custom location ~ ^/(media|proxy) with target <IP_ADDRESS>:5000 and advanced config
        proxy_cache        pleroma_media_cache;
        slice              1m;
        proxy_cache_key    $host$uri$is_args$args$slice_range;
        proxy_set_header   Range $slice_range;
        proxy_cache_valid  200 206 301 304 1h;
        proxy_cache_lock   on;
        proxy_ignore_client_abort on;
        proxy_buffering    on;
        chunked_transfer_encoding on;

Thank you very much for your help and your easy to follow answer!
So, I've SSH'd into the docker container using
sudo docker exec -it --user root containerID sh
I've created the file and directory ("custom" didn't exist)

This is how the file "http_top.conf" looks like:

I have successfully added
location /api/fedsocket/v1 {
        proxy_request_buffering off;
        proxy_pass http://<IP_ADDRESS>:5000/api/fedsocket/v1;
    }

gzip_vary on;
gzip_proxied any;
gzip_comp_level 6;
gzip_buffers 16 8k;
gzip_http_version 1.1;
gzip_types text/plain text/css application/json application/javascript text/xml application/xml application/xml+rss text/javascript application/activity+json application/atom+xml;

# the nginx default is 1m, not enough for large media uploads
client_max_body_size 16m;
ignore_invalid_headers off;

proxy_cache        pleroma_media_cache;
slice              1m;
proxy_cache_key    $host$uri$is_args$args$slice_range;
proxy_set_header   Range $slice_range;
proxy_cache_valid  200 206 301 304 1h;
proxy_cache_lock   on;
proxy_ignore_client_abort on;
proxy_buffering    on;
chunked_transfer_encoding on;

So, everything you suggested worked! I think. I even restarted nginx using systemctl.
Whenever I entered any of this code:
ssl_session_timeout 1d;
ssl_session_cache shared:MozSSL:10m;  # about 40000 sessions
ssl_session_tickets off;

ssl_protocols TLSv1.2 TLSv1.3;
ssl_ciphers "ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA384:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";
ssl_prefer_server_ciphers off;

ssl_stapling on;
ssl_stapling_verify on;

Into my advanced config, it seemed to go offline, so I think the nginx reverse proxy manager already takes care of SSL.
The only thing that I have not done is add a custom location called "~ ^/(media|proxy)". Should I just write it into the same http_top.conf file with <IP_ADDRESS>:5000 ?

You have created the file inside the container, which is fine for now, but be warned that it will probably no longer be there when you update. What I meant is mounting it into the container via your docker-compose. That way the file stays on your host and gets mounted into every version of the container there will be.
Not sure why the ssl config crashes your host, it does not do it for me. When you hover over the Offline status, you can see the error message. NPM takes care of generating the ssl certificate, but these configs are additional options for the tls connection between the host and the proxy. Some of them are already set and I just guessed which ones you should add, but it turns out only these three ssl settings are not yet set:
ssl_session_tickets off;
ssl_stapling on;
ssl_stapling_verify on;

However they are most likely not required for the service to work correctly, so you could also just not add them.
I also assumed you added the custom location via the custom locations tab, but you have added it via the advanced config? You can add them in the custom locations tab:

But this is most likely also optional as this only changes the caching behavior of media files. If you do not add this custom location, you also do not need the http_top.conf file.

Okay, thank you very much! I did exactly as you described and all works, including the media proxy.

I have also added the http_top.conf file to my nginx reverse proxy manager docker-compose.yml file so that when I rebuild it this file and change will not be lost!

(although I am quite uncertain whether this works or makes a difference as the entire "data" directory is mounted)
So, after everything worked I added the three not-already-set variables:
ssl_session_tickets off;
ssl_stapling on;
ssl_stapling_verify on;

And it is still online.
You can visit https://social.uden.ai/ and it seems to all work. Now I just need to verify whether the federation feature all works or not. It should as essentially everything has been added except for:
upstream phoenix {
        server <IP_ADDRESS>:5000 max_fails=5 fail_timeout=60s;
    }

as well as:
        location / {
            proxy_pass http://phoenix;
        }

The upstream phoenix (or any other name) cannot be simply set in the advanced settings for obvious reasons (or can I add it to the http_top.conf?) and if I add the second string of code it wouldn't work without the first one (obviously) and if I were to replace "phoenix" with my local IP, it would simply stop enforcing https, so, yeah.
However, main things should be fixed. All of the nginx config suggested by the Pleroma-OTP install here:
https://docs-develop.pleroma.social/backend/installation/otp_en/#edit-the-nginx-config
Are added. So, I will test and see whether I am online to the federation.
Thank you so much for your help!!

Okay, wonderful. I have just been followed by someone from another Pleroma instance! The federation is online. I am very grateful for your immense help @chaptergy Thank you very much! Nginx is not my strongest suite, but thanks to you that did not stop me from putting a Pleroma instance online. Very, very nice.

Great to hear everything works! You don't actually need the upstream phoenix part. If you have set the host configuration on the main config page to use <IP_ADDRESS> and port 500 the location / is automatically generated for you. No need for the upstream section. I'm not sure why you'd think it would stop enforcing https, if you enable HSTS you should be absolutely fine.

Thank you all for this discussion it was very helpful. 1 additional note on something that took me a while to figure out. I had to change the configuration on Pleroma so it would allow an IP address other than <IP_ADDRESS> to connect to it.
https://docs.pleroma.social/backend/configuration/cheatsheet/#pleromawebendpoint
I had to change the ip and then restart Pleroma.

Thank you all for this discussion it was very helpful. 1 additional note on something that took me a while to figure out. I had to change the configuration on Pleroma so it would allow an IP address other than <IP_ADDRESS> to connect to it.
https://docs.pleroma.social/backend/configuration/cheatsheet/#pleromawebendpoint
I had to change the ip and then restart Pleroma.

Wait, could you post a copy of your configuration?

Well... there are secrets in the config file, so I'm not going to share it all. But this is part of my prod.secret.exs
This is the part in config :pleroma, Pleroma.Web.Endpoint, ... I think I don't really know how to read these .exs config files.
http: [ip: {0, 0, 0, 0}, port: 4000],

I think this setting means it will allow any IP address can connect to my Pleroma server, in theory I could restrict this to just my Nginx Proxy Manger.


refactor: remove obsolete clippy allows in non_finalized_state

Motivation
I was reading the codebase and found these two #[allow()]s obsolete
PR Author Checklist
Check before marking the PR as ready for review:

[x] Will the PR name make sense to users?
[x] Does the PR have a priority label?
[x] Have you added or updated tests?
[x] Is the documentation up to date?

For significant changes:

[x] Is there a summary in the CHANGELOG?
[x] Can these changes be split into multiple PRs?

If a checkbox isn't relevant to the PR, mark it as done.
Solution
Just remove the obsolete allows
Testing
cargo +nightly clippy -p zebra-state

Reviewer Checklist
Check before approving the PR:

[ ] Does the PR scope match the ticket?
[ ] Are there enough tests to make sure it works? Do the tests cover the PR motivation?
[ ] Are all the PR blockers dealt with?
PR blockers can be dealt with in new tickets or PRs.

And check the PR Author checklist is complete.

@teor2345 I'd appreciate some issue to get started. Do you have anything in mind?

@teor2345 I'd appreciate some issue to get started. Do you have anything in mind?

You could try something from this list of tickets:
https://github.com/ZcashFoundation/zebra/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+label%3A"good+first+issue"
Or you could ask the developers in the #zebra channel in the Zcash R&D discord:
https://discord.com/channels/809218587167293450/809251029673312267
Here's a link that will let you join the Discord for the next week:
https://discord.gg/GvhwHwXR