disappearing text in tutorial world entry page

This has taken a while to pin down, and now I've pinned it down I don't have any understanding of what's going on. The comments below pertain to the debug branch of NNG4 which I made just to showcase this issue.
Uncommenting this line here in the file Game/Levels/Addition/L01zero_add.lean for some reason makes the text from this page magically stop appearing on the intro page to tutorial world http://localhost:3000/#/g/hhu-adam/NNG4/world/Tutorial/level/0 .
So for whatever reason, this import is bad. But I want add_zero and add_succ in addition world! What have I done wrong? Can anyone else reproduce? Go to the debug branch, compile, note that the intro to tutorial world has the text "Welcome to tutorial world! In this world we learn the basics of proving theorems. The boss level of this world is the theorem 2 + 2 = 4...." on the web page, then uncomment out the import, compile again, and the text is gone.

Somehow I have add_zero and add_succ in addition world anyway, without the import.
I think a summary of this issue is: I have no idea why adding the import as explained above causes problems. On the other hand I know how to work around it: it seems that I can just never import tutorial world at all, and just redefine stuff like one_eq_succ_zero a second time in another file and import that instead. So this is no longer blocking me but it's still pretty weird.

This is a known bug: You must not import a particular level of a different world, only entire worlds. (i.e. Game.Levels.Tutorial instead of Game.Levels.Tutorial.L05add_succ)
leanprover-community/lean4game/issues/60
PS: add_zero and add_succ are defined in Game.MyNat.Addition which is explicitely imported in the file linked, so that seems as expected.


Internationalization of Chinese and English

Resource extraction i18
2.Add Chinese resource files

Hi @Daishaojie111 I'v found a minor issue. It can't be persistent when I swtiched the language. It lost the my preference once I referesh the page.


"dotnet new mvc" now ignores "--auth Individual" option, so does visual studio too

After some latest updates, it is impossible to create new project with individual authentication.
In the project gererated, there is nothing related to user accounts (no such things as: AccountController,ManageController, logon/sing-in functionality).

@alexk8 It's by design.
This might help you .
https://blogs.msdn.microsoft.com/webdev/2018/03/02/aspnetcore-2-1-identity-ui/

You have to scaffold necessary parts if you want to customize identity logic.

ok, thanks, understood.
I think in that case, tools mentioned above should be updated to show a warning with that article...
thanks

sorry, default functionanity is still there, but was hidden under "cookie use policy" bar, so I couldn see it and  messed up, sorry. The issue probably may be closed


Fix remote repo name - issue #15

I changed the remote repo name from "upstream" to the default one "origin".

@himanshujaidka  Is it necessary to add GSSoV'22 label?

@himanshujaidka  Please modify the label 'Gssoc 2022' to 'GSSoC22' to be counted.


Feature Idea: Recieve automated offer preference configuration

The intent of this is to allow users to set more strict settings on what they want to receive from automated offers than what they would generally accept from users in general. This would provide a middle ground solution between the extremes of having users be forced to accept any offer that they would accept from regular offers or no offers at all
Minimum Feature Implementation Idea: A button for users to say "I only want to receive offers that involve wishlisted games from EnhancedBarter." even if they regularly allow offers for any unowned game.
Most likely Implementation Idea: Users to have the ability to have the flexibility to have a bot receive preferences similar to the Offer Preferences section on the main user page. eg: Even if users normally are configured to be ok to receive offers involving games that are 60%+ in rating from users, they can specify that they only want to receive offers involving games that are 90%+ in rating from EnhancedBarter.
Extreme Implementation Idea: Using the browse filtering [ https://barter.vg/browse/ ] as the basis for what games users are willing to receive from EnhancedBarter. eg: Beyond the most likely implementation users can state they only want to receive offers involving games that have only been bundled X or fewer times or games that have at least X reviews.

I don't see why offer preferences should be different for offers coming from Enhanced Barter or native barter. Both type of offers are crafted by regular traders. The only difference being that for native offers you select specific games and for enhanced offers you select a range.
That said, barter will also improve the offer preferences soon. Maybe you can ask him for the desired changes.


Charts: Rancher Backups: Default storage class no longer auto-selected when user leaves page

Setup

Rancher version: v2.9-head
Rancher UI Extensions:
Browser type & version:

Describe the bug
The default storage class is no longer auto-selected when the user leaves the page (the user clicks on another tab ('Edit YAML' or goes back to the previous step).
To Reproduce

login to rancher as admin
create a storage class and set as default
go to Apps > Charts > select Rancher backups
click Install
click Next
click Use an existing storage class
notice default storage class is auto-selected
click 'Edit YAML'
go back to 'Edit Options'
notice default storage class is not auto-selected

Result
Default storage class is auto-selected but not preserved.
Expected Result
Default storage class is auto-selected and preserved.
Screenshots
https://github.com/rancher/dashboard/assets/127343932/34f23d3d-c435-49c7-bb01-f4a80cdb6b64
Additional context

e2e test coverage lgtm. moving to Done cc @izaac


not supprt drawableLeft and drawablePadding

drawableLeft and drawablePadding will dismiss when input value.

Hey.
That is true, you cannot use the drawables that you set on a textview because the library uses exactly the same approach to draw the eye icon


Upgrade to Spring Boot 3

Hi, this lib will be upgraded to Spring Boot 3?

Hi, yes, sometime later when Spring Boot 3 will be more widespread

Nice.. thanks!

@suchorski New release supports Spring Boot 3.x

Very nice... Will take a look, thanks


Fixed the issue where memory allocated to the ID was not being freed

This is a very minor modification.
When OpenSSL is running in s_server mode, the memory allocated for the id is not properly freed.
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Sorry, It seems that the patch is causing issues in certain scenarios. :cry:

The OCSP_request_add0_id function passes ownership of the id to the req object. So when the req is freed so is the id. In order to prevent a double free we must assign NULL to id after the call to OCSP_request_add0_id. So the current code looks correct to me.

Thank you for your kind explanation!
Immediately after submitting this PR, I closed the PR by checking the case where the ownership transition takes place within the OCSP_request_add0_id as you mentioned.


support special missing values in mp_jsonout.sas

Links to https://github.com/sasjs/adapter/issues/607
Special missing values should be (optionally) converted to strings

:tada: This issue has been resolved in version 3.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Enhancement] OnFilesDropped Event

I want to input paths to uiEntry by drag and drop.
So, I made uiEntryOnFilesDropped to get the dropped files.
You can see my branch here.
I don't mean it should be merged because I'm novice at the native APIs and there might be better solutions like wxDropTarget.
But at least, I think there should be a way to input paths by drag and drop.

I started implementing something more general (text & files support) with uiWindow as a drop target a while back myself, here.
I guess there is a need to implement this with uiControl as a target? Got to look into how complicated that is to do for all platforms.
I would still like to rewrite the API as well, as GTK seemingly only supports async retrieval of the dropped data.
Looking at wxWidgets there might!? be a way around that though? I don't really understand how they are retrieving the data at all though, as seemingly none of the signals are hooked up.

I guess there is a need to implement this with uiControl as a target? Got to look into how complicated that is to do for all platforms.

I think so yes, should be able to drop target to an arbitrary HWND/NSControl/GtkWidget.
Doesn't mean it will make sense visually, but should work generically enough to implement for uiControl.

I don't have the bandwidth to get involved with this stuff at the moment, but I've used the Qt Drag & Drop API and found it really useful. I don't know what hoops it has to jump through on various platforms, but I'd love to see libui handle something along these lines eventually...
The QDrag object lets you create a drag operation and attach one or more mime-encoded payloads to it. The recipient of the drag just picks whatever mime type it prefers to handle (if any) and is served up the data. You can also provide a pixmap icon to represent the contents of the drag operation.
For example, I use it in my pixelart app, to allow the user to drag&drop palette colours within my app, but also between other apps (gimp etc).
When my palette widget detects a drag event, it starts a drag operation and adds in two distinct packets of data. One is a generic RGBA value of type "application/x-color" which is supported by lots of other drawing apps. But inside my app I also want to know which palette index the colour was dragged from, so I add an app-specific ("application/x-evilpixie-pen") value which includes that index. You can encode as many alternative payloads as you want. It also generates a pixmap swatch of the colour, which Qt shows being dragged.
https://github.com/bcampbell/evilpixie/blob/53f2354bcf143c338cd648e938a9ac38919c2048/src/qt/palettewidget.cpp#L187C33-L187C33
(I'd really like to drop Qt in my app and move over to libui, but there are still a few parts of the puzzle missing)

I think so yes, should be able to drop target to an arbitrary HWND/NSView/GtkWidget. Doesn't mean it will make sense visually, but should work generically enough to implement for uiControl.

Yeah, not sure if it can actually be implemented on all platforms for all types of uiControl but I believe it should be possible for most (win32 might be opinionated  here).
The implementation for darwin seems like it will need to touch all controls, as each control will need to implement the NSDraggingDestination protocol. Sadly this is one of the few instances where a delegate would be great, but I haven't found an API like that anywhere in the docs. So each uiControl will need to implement the protocol methods. Guess this could be done via C macros. Not pretty, but that should work.
The other option would be to extend all instances of NSView via the objc category mechanism. This seems to complicate things for bindings though, as it requires additional linker flags. I remember introducing categories for NSTextField and reverting that very quickly a while back.
Thanks @bcampbell for the inspiration as well, I'll check out the QT API as well.
I have so far only implemented two types text and/or files but adding pixbuf should be easy once the foundation is there. My design so far is with a type enum, as that is what both darwin and windows use. GTK does use mime types underneath but since the types we are implementing are very limited anyways, I didn't see much benefit in using mime types (strings) over simple enums.

After looking further into custom clipboard formats - possibly via mime types-  I've not gotten much further.
@bcampbell do you have any native and open source utilities that accept application/x-color? So I could peek at the code?
For your cross application use case:

darwin has NSPasteboardTypeColor
windows has CF_PALETTE
unix has good old mime types

QT does provide a conversion utility  for darwin's UTI to/from mime.
For custom formats:
To accept drops on darwin, the application must accept one or more NSPasteboardTypes. They are defined as typedef NSString *NSPasteboardType so maybe a mime type string could be registered here? Is this something other applications do though?
On windows custom clipboard formats need to be registered first via RegisterClipboardFormat. Again, nothing impossible, but I have no idea what real life applications register here in practice either.
GIMP (to my knowledge) is not native on windows or mac to check out the source code. And I'm not sure if wading through  <<QT>template>C++~insanity will be of any use for figuring out real life application behavior.

Just tried a few paint programs, and all of them support drag & drop with application/x-color.
(this is on Linux. I'm 85% sure I've dragged colours between evilpixie and gimp on windows too...)
gimp (C, Gtk) supports application/x-color:
https://gitlab.gnome.org/search?search="application%2Fx-color"&nav_source=navbar&project_id=1848&group_id=8&search_code=true&repository_ref=master
Interestingly gimp also seems to support a whole bunch of other drag types, many of them gimp-specific:
https://gitlab.gnome.org/GNOME/gimp/-/blob/master/app/widgets/gimpdnd.h
Mypaint (Python, Gtk) supports application/x-color and text/uri-list:
https://github.com/search?q=repo%3Amypaint%2Fmypaint x-color&type=code
Krita (C++, Qt) supports application/x-color plus a whole bunch of Krita-specific types:
https://github.com/search?q=repo%3AKDE%2Fkrita x-color&type=code
(Oddly, I had a lot less luck dragging colours to/from krita than any of the other apps, including my own. I'll have to investigate, but I did spot some hints that Krita might sometimes drag named colours, which might be a different mimetype?).
And for completeness, my own app, Evilpixie:
https://github.com/search?q=owner%3Abcampbell+QMimeData&type=code
(only in the palette editor. I discovered you can't currently drag colours from the ever-present palette widget in the bottom-right. Another bug to fix!)

Thanks for the unix pointers! I was more looking for native macOS and Windows apps, guess my message wasn't really that clear.
Regarding a proper API, my ideas for drop events so far:

New type uiDragDestination that contains all the callbacks and internal stuff
4 callbacks for uiDragDestination: OnEnter, OnMove, OnLeave, OnDrop`
Enum uiDragOperation: uiDragOperationNone, uiDragOperationCopy, uiDragOperationLink, uiDragOperationMove to specify what action to perform (and cursor to show) for all events
Enum uiDragType with uiDragTypeText, uiDragTypeURIs (lets see if I can figure out mime types for darwin and win)
uiDragDestinationSetAcceptType to set which content types our uiDragDestination accepts
uiControlSetDragDestination to connect the uiDragDestination  to a uiControl

So far so good. The exact details around parameters are something to still figure out but that can still be buffed out later.
The problem I'm facing and what I would like to get feedback on is how to handle the getting of data.
Both windows and macOS have ways to get the data via IDataObject*->GetData and [NSPasteboard* propertyListForType:]. Nice, synchronous calls that can be wrapped.
Gtk frustratingly is the outlier again and does data retrieval via gtk_drag_get_data which triggers a callback.
This is sadly highly annoying as it calls the same function from both OnEnter and OnDrop. In the first instance we might just want to take a peek at the data to figure out if we even want to accept it. Gtk actually has a way of determining in what state the GdkDragContext is in but it is still a major inconvenience to the user.
My question now is what do we expose as an API to the user here?

A callback OnDataReceived with a flag if we entered or dropped
Fake a synchronous call by looping uiMainStep(1) as is already done in some other parts of unix/window.c for example.

The first option IMO makes for an extremely ugly API but should be future proof.
The second option IMO makes for a nicer API but might get us in trouble with GTK4. In my eyes the GTK4 APIs are even more cumbersome and convoluted but maybe the same trick of running the event loop manually still works.
Another, entirely different option would be to make peeking the data impossible and only expose the data in OnDrop, which seemingly would be future proof regarding GTK4.
I tend to lean towards the second option as it is unclear if a GTK4 port will be possible without major API breakage anyways.

Made some good progress and I'm nearly done (sans mime-type atm). Still one road block however:

I think so yes, should be able to drop target to an arbitrary HWND/NSView/GtkWidget. Doesn't mean it will make sense visually, but should work generically enough to implement for uiControl.

How do we want to implement this? Currently there is no way to get a hold of GtkWidget, HWND, or NSView from the context of a uiControl.
To implement a generic  uiControlSetDragDestination() a complete refactor would be necessary to expose something akin to uiControlNative(). It is unclear to me if each uiControl actually has a corresponding native control. Especially on darwin this will be complicated since everything is done through subclassing and no generic control even exists.
An alternative would be to implement this in each individual control manually, similar to the way all other events are currently handled. uiButtonOnClicked() etc, add a uiButtonRegisterDragDestination() or something like that.
Ideas?


castxml: 0.4.6 -> 0.4.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/CastXML/CastXML/releases.
meta.description for castxml is: "C-family Abstract Syntax Tree XML Output"
meta.homepage for castxml is: "https://github.com/CastXML/CastXML"
meta.changelog for castxml is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.7 with grep in /nix/store/n7vc02yr6w4ajp7lsrqz19m76vw4zcsi-CastXML-0.4.7
found 0.4.7 in filename of file in /nix/store/n7vc02yr6w4ajp7lsrqz19m76vw4zcsi-CastXML-0.4.7

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
castxml
gnss-sdr
gnuradio
gnuradioPackages.gnuradio
python310Packages.pygccxml
python39Packages.pygccxml

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n7vc02yr6w4ajp7lsrqz19m76vw4zcsi-CastXML-0.4.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A castxml https://github.com/r-ryantm/nixpkgs/archive/6a8002d3ecf6f8f31cf534e469ca8d767841a6cb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n7vc02yr6w4ajp7lsrqz19m76vw4zcsi-CastXML-0.4.7
ls -la /nix/store/n7vc02yr6w4ajp7lsrqz19m76vw4zcsi-CastXML-0.4.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

castxml
gnss-sdr
gnuradio
gnuradioPackages.gnuradio
python310Packages.pygccxml
python39Packages.pygccxml

Maintainer pings
cc @AndersonTorres for testing.

Go to
https://github.com/NixOS/nixpkgs/pull/200950


Removing redundant double-parsing of resources

While taking a quick skim through the library to determine if it's something we'd like to use in a project (it is, great work btw) I ran into this strange looking part where resources appeared to be parsed twice.
Simply removing this doubling up causes tests to pass correctly so hopefully it's a cheap performance improvement.

:+1:

Thanks @jamesbrooks!


Docker : Error respinse from daemon: pull access denied on elf_on_elk, repository does not exist or may require 'docker login'

ubuntu@ip-172-31-0-164:~/Documents/elf_elk_docker-master$ make run
Sending build context to Docker daemon  23.04kB
Step 1/12 : FROM java:8
8: Pulling from library/java
5040bd298390: Pull complete
fce5728aad85: Pull complete
76610ec20bf5: Pull complete
60170fec2151: Pull complete
e98f73de8f0d: Pull complete
11f7af24ed9c: Pull complete
49e2d6393f32: Pull complete
bb9cdec9c7f3: Pull complete
Digest: sha256:c1ff613e8ba25833d2e1940da0940c3824f03f802c449f3d1815a66b7f8c0e9d
Status: Downloaded newer image for java:8
---> d23bdf5b1b1b
Step 2/12 : MAINTAINER<EMAIL_ADDRESS>---> Running in b65b75ab4e40
Removing intermediate container b65b75ab4e40
---> f28a2082c94e
Step 3/12 : RUN apt-get update
---> Running in 2dfdd44da951
Get:1 http://security.debian.org jessie/updates InRelease [44.9 kB]
Ign http://deb.debian.org jessie InRelease
Get:2 http://deb.debian.org jessie-updates InRelease [16.3 kB]
Ign http://deb.debian.org jessie-backports InRelease
Get:3 http://deb.debian.org jessie Release.gpg [1652 B]
Ign http://deb.debian.org jessie-backports Release.gpg
Get:4 http://deb.debian.org jessie Release [77.3 kB]
Ign http://deb.debian.org jessie-backports Release
Err http://deb.debian.org jessie-backports/main amd64 Packages
Err http://deb.debian.org jessie-backports/main amd64 Packages
Err http://deb.debian.org jessie-backports/main amd64 Packages
Err http://deb.debian.org jessie-backports/main amd64 Packages
Get:5 http://security.debian.org jessie/updates/main amd64 Packages [977 kB]
Err http://deb.debian.org jessie-backports/main amd64 Packages
404  Not Found
Get:6 http://deb.debian.org jessie-updates/main amd64 Packages [20 B]
Get:7 http://deb.debian.org jessie/main amd64 Packages [9098 kB]
Fetched 10.2 MB in 4s (2163 kB/s)
W: There is no public key available for the following key IDs:
AA8E81B4331F7F50
W: Failed to fetch http://deb.debian.org/debian/dists/jessie-backports/main/binary-amd64/Packages  404  Not Found
E: Some index files failed to download. They have been ignored, or old ones used instead.
The command '/bin/sh -c apt-get update' returned a non-zero code: 100
Unable to find image 'elf_on_elk:0.1' locally
docker: Error response from daemon: pull access denied for elf_on_elk, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.
Looks like an issue with MakeFile, this is throwing multiple issues permission issue and default value issue.
Could please help me here
Thanks In advance

Helo @abisek can you please your inputs here


Rename Gateway URL in devportal as API Endpoint URL

Describe your problem(s)
Needs to refactor the API overview in dev portal, Which includes the renaming of gateway URLs as API Endpoint(s)
Following is the design (need to get the XD separately), and there has to be some refactoring to the API Endpoint URL placement

@erangatl   Shall we update the design as per the offline chat and attach a screenshot ?

@erangatl   Shall we update the design as per the offline chat and attach a screenshot ?


add new (typed) random key constructor

This constructor unconditionally returns a typed key array, regardless of the value of jax.config.enable_custom_prng. We can switch to referring to it in randomness docs and tutorials as we complete the typed key upgrade (#9263).

Yeah, I left it undocumented on purpose. When we're ready, we can document it and update all documentation/tutorials to use key in place of PRNGKey.


🌱 Rearrange e2e upgrade tests

What this PR does / why we need it:
Move the Ironic and BMO upgrade tests to the e2e upgrade test and run them before the CAPM3 upgrade test.
What should we do about the ephemeral test? Should we still run BMO and Ironic upgrade there?

/retest

/test-v1b1-ubuntu-integration

/test-v1b1-upgrade-e2e

Ok, so I realized that the upgradeManagementCluster function is also setting up the management cluster and doing the pivot. This means we can obviously not do BMO and Ironic upgrades before that. I have instead included them inside the function for now. Not ideal but I don't think we should refactor upgradeManagementCluster as part of this PR.
I see two paths forward here:

(This is the current plan AFAIK.) Rework upgradeManagementCluster so that we can directly use the CAPI upgrade test spec. Ironic and BMO upgrades would run as a PreUpgrade step.
Refactor upgradeManagementCluster into smaller functions so that we can call them in order and place Ironic and BMO upgrades at proper places.

I suggest merging this PR as is (if it is working) and  implement 1) or 2) in separate PRs.

/test-v1b1-upgrade-e2e

/test-v1b1-upgrade-e2e

/test-v1b1-ubuntu-integration

/hold cancel
related PR is merged

/hold cancel


RHBPMS-4346: User with given role cannot see his human tasks in the t…

…ask perspective
Added the user roles to task queries conditions

@nmirasch overall it looks good, but I believe tests could be improved. Would be good if we could test that the actual role was used as a filter.

@cristianonicolai test changed

@nmirasch nice! thanks


Plugin does not handle 302 redirect correctly

AWS Cloud plugin does not seem to correctly handle a 302 Redirect when encountered on the S3 end-point
Not sure if this is a bug or that perhaps we should access the end-point via a proxy of some sort?
Below is the error from the ES log when attempting to create a new S3 repo...
The S3 repo service is actually provided by IBM CleverSafe and should be fully S3 compliant.
Accessing the service directly via IP address works fine where the redirect does not happen but when using via a generic DNS address which redirects then problem occurs...
[2016-10-25 12:05:16,324][WARN ][rest.suppressed          ] path: /_snapshot/s4-es-mon-prod, params: {repository=s4-es-mon-prod}
RepositoryException[[s4-es-mon-prod] failed to create repository]; nested: CreationException[Guice creation errors:

Error injecting constructor, com.amazonaws.services.s3.model.AmazonS3Exception: Found (Service: Amazon S3; Status Code: 302; Error Code: 302 Found; Request ID: null), S3 Extended Request ID: null
at org.elasticsearch.repositories.s3.S3Repository.(Unknown Source)
while locating org.elasticsearch.repositories.s3.S3Repository
while locating org.elasticsearch.repositories.Repository

Can't you define cloud.aws.s3.endpoint in such a case?
See https://github.com/elastic/elasticsearch-cloud-aws/tree/v2.7.1/#using-other-s3-endpoint

Yes, I am defining the S3 end-point but that is an internal DNS address which responds with a redirect.
To allow the owner of that end-point to point somewhere else in the event of system outages I guess only way is to implement some sort of proxy between the AWS Cloud plugin and the S3 end-point so handle the redirect?
If I use the IP of the S3 service directly then it works but they'd prefer me not to hard-code it in this way. The proxy is just my idea to work around this issue?
Ideally it would be good if the AWS Cloud plugin could respond to the 302 but it seems to just treat this as an error?

Ideally it would be good if the AWS Cloud plugin could respond to the 302 but it seems to just treat this as an error?

Yes. Actually we are using AWS SDK here which apparently does not support that.
Unless there is a magical knob I did not see :)
Thinking out loud: may be this is something which can be reported in AWS SDK project instead?

Yes, that seems to make sense! Is that something that you could raise there citing this ticket as a 'use case' calling for it?

I'm not familiar with the AWS SDK and only use the plugin so perhaps this additional feature request would be best explained by a developer?

BTW do you have in logs the full stack trace?

Is this what you're after?

Error injecting constructor, com.amazonaws.services.s3.model.AmazonS3Exception: Found (Service: Amazon S3; Status Code: 302; Error Code: 302 Found; Request ID: null), S3 Extended Request ID: null
at org.elasticsearch.repositories.s3.S3Repository.(Unknown Source)
while locating org.elasticsearch.repositories.s3.S3Repository
while locating org.elasticsearch.repositories.Repository

1 error
at org.elasticsearch.common.inject.internal.Errors.throwCreationExceptionIfErrorsExist(Errors.java:360)
at org.elasticsearch.common.inject.InjectorBuilder.injectDynamically(InjectorBuilder.java:178)
at org.elasticsearch.common.inject.InjectorBuilder.build(InjectorBuilder.java:110)
at org.elasticsearch.common.inject.InjectorImpl.createChildInjector(InjectorImpl.java:154)
at org.elasticsearch.common.inject.ModulesBuilder.createChildInjector(ModulesBuilder.java:55)
at org.elasticsearch.repositories.RepositoriesService.createRepositoryHolder(RepositoriesService.java:404)
... 11 more
Caused by: com.amazonaws.services.s3.model.AmazonS3Exception: Found (Service: Amazon S3; Status Code: 302; Error Code: 302 Found; Request ID: null), S3 Extended Request ID: null
at com.amazonaws.http.AmazonHttpClient.handleErrorResponse(AmazonHttpClient.java:1389)
at com.amazonaws.http.AmazonHttpClient.executeOneRequest(AmazonHttpClient.java:902)
at com.amazonaws.http.AmazonHttpClient.executeHelper(AmazonHttpClient.java:607)
at com.amazonaws.http.AmazonHttpClient.doExecute(AmazonHttpClient.java:376)
at com.amazonaws.http.AmazonHttpClient.executeWithTimer(AmazonHttpClient.java:338)
at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:287)
at com.amazonaws.services.s3.AmazonS3Client.invoke(AmazonS3Client.java:3654)
at com.amazonaws.services.s3.AmazonS3Client.headBucket(AmazonS3Client.java:1020)
at com.amazonaws.services.s3.AmazonS3Client.doesBucketExist(AmazonS3Client.java:978)
at org.elasticsearch.cloud.aws.blobstore.S3BlobStore.(S3BlobStore.java:84)
at org.elasticsearch.repositories.s3.S3Repository.(S3Repository.java:135)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
at org.elasticsearch.common.inject.DefaultConstructionProxyFactory$1.newInstance(DefaultConstructionProxyFactory.java:50)
at org.elasticsearch.common.inject.ConstructorInjector.construct(ConstructorInjector.java:86)
at org.elasticsearch.common.inject.ConstructorBindingImpl$Factory.get(ConstructorBindingImpl.java:104)
at org.elasticsearch.common.inject.FactoryProxy.get(FactoryProxy.java:54)
at org.elasticsearch.common.inject.ProviderToInternalFactoryAdapter$1.call(ProviderToInternalFactoryAdapter.java:47)
at org.elasticsearch.common.inject.InjectorImpl.callInContext(InjectorImpl.java:886)
at org.elasticsearch.common.inject.ProviderToInternalFactoryAdapter.get(ProviderToInternalFactoryAdapter.java:43)
at org.elasticsearch.common.inject.Scopes$1$1.get(Scopes.java:59)
at org.elasticsearch.common.inject.InternalFactoryToProviderAdapter.get(InternalFactoryToProviderAdapter.java:46)
at org.elasticsearch.common.inject.InjectorBuilder$1.call(InjectorBuilder.java:201)
at org.elasticsearch.common.inject.InjectorBuilder$1.call(InjectorBuilder.java:193)
at org.elasticsearch.common.inject.InjectorImpl.callInContext(InjectorImpl.java:879)
at org.elasticsearch.common.inject.InjectorBuilder.loadEagerSingletons(InjectorBuilder.java:193)
at org.elasticsearch.common.inject.InjectorBuilder.injectDynamically(InjectorBuilder.java:175)
... 15 more
^C

I opened https://github.com/aws/aws-sdk-java/issues/903. Will see what is the outcome of this.
For now, you need to define the right endpoint.
Please note that this repo is now deprecated and will only get bug fixes.
New features will only go to elasticsearch repo.

OK, great! thanks for following up on this...
I spoke to the internal provider of our S3 repo and he said that others are using the AWS SDK and they have worked around this 302 redirect problem so I will see if I can find any more info and pass it on if I do...

A guess is that we check manually this URL and if we get a redirect we change the configuration you defined with the new URL.
But IMO this is something that should be supported by the SDK.

So AWS team rejected the feature request for the reasons they exposed there: https://github.com/aws/aws-sdk-java/issues/903
I'm not sure if we want to implement such a thing on our side and I believe we don't want to make the code more complex.
@tlrx WDYT?

Fair enough, I've got the end-point hard-coded to the S3 service IP at the moment and working fine.
I guess the proxy approach is still an option if we choose to go down that route.
Thanks for your persistence in following this up :)

If I understand correctly, the IBM CleverSafe S3 implementation sends redirections to the client but the AWS SDK team indicates that it is not part of the service contract in AWS to send redirect responses?
If that's correct then I don't think we should implement such a thing on plugin's side.

Thanks Tanguy. We are on the same page.
Closing

Thanks (Tan)guys! Best to stick to AWS contract...


Smart text highlighting in log files

Issue Type: Bug
It appears to me VS Code tried to improve the highlighting even of raw text files like log files. For exaple, it seems to make timestamps green and numbers with more than two digits blue.
I like this feature, but it should either exclude hexadecimal numbers or learn to highlight them consistently altogehter. The current scheme makes logs that contain MAC addresses very difficult to read:
89:A1:23:45:AB:C0 has 89 in blue, 1:23:45 in green and the rest in white.
VS Code version: Code - Insiders 1.45.0-insider (d69a79b73808559a91206d73d7717ff5f798f23c, 2020-05-06T09:26:27.927Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (7.54GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Highlighting comes from the log grammar: https://github.com/emilast/vscode-logfile-highlighter/blob/master/syntaxes/log.tmLanguage

Moved to owning repo


hotfix/update default elasticsearch index for share search

changes
Right now, not adding a version query parameter defaults to an elasticsearch index that does not exist (share). This PR updates it to default to the index share_v2, which reflects the most recently updated schema.
side effects
None anticipated

The share index actually should exist - it should be an alias to the most current schema version, in this case 2

Oh! I ran it locally and everything broke until I gave it  share_v2, but perhaps another change should be made!

Nevermind!!


why Perform two constructors？？

why Perform two constructors？？

Can you be more detailed about your problem?

Constructor I print information when the result of the two print, I don't know why, I am using "dingo/API" : "1.0. * @ dev."

Can you share some code and results?

router

<?php
$api = app('Dingo\Api\Routing\Router');
$api->version('v1', function ($api) {
    $api->group(['namespace' => 'App\Api\Controllers'], function ($api) {
        $api -> get('/demo', 'HomeController@index');
    });
});

controller

<?php
namespace App\Api\Controllers;

use App\Http\Controllers\Controller;

class HomeController extends Controller
{

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        print("why two<br/>");
    }

    public function index()
    {
        echo "show result";
    }
}

result

why two
why two
show result

Are you still in?

see: https://github.com/dingo/api/issues/1001


how to add certificate for self signed repo

System information

node version: 8.16.2
npm or yarn version: 6.4.1
OS/version/architecture: Linux x64 mint 18.4
Applicable nodegit version: 0.26.2

I'm using a gog server with a self signed certificate. I'm trying to clone my repo but an error raised with this message : Error: the SSL certificate is invalid
It's quite easy and well documented to resolve for git by using git config http.sslCAInfo. But how to do that for nodegit ?
I don't succeed to find something in the documentation.
Thanks

I solved the issue by adding :
var opts = {
	fetchOpts: {
		callbacks: {
            certificateCheck: () => 0
        }
	}
};

in the clone command(url, path, opts).
The self-signed certificate may not be the root cause.


How to analyse a SMAC3 run with CAVE

Hi, I am not sure this is the place to ask this question as it is not really an issue, but I could not find any other contact details. Please let me know if there is a better forum.
I would like to use SMAC and analyse its results with CAVE. The smac optimisation I performed is very similar to the SVM example but done in a jupyter notebook.
Here is a very simple repository
I run SMAC from a notebook and then try to run CAVE from another notebook. (the notebooks are also in .py and .html format for easy inspection). Also, all the outputs from SMAC are in the repository.
The CAVE notebook has an error in the last output field.
I think it has to do with the fact that it is looking for a target algorithm implementation, but I am not sure why. My understanding is that once the optimisation process outputs all its history, then CAVE can be used to analyse it without needing to know what the ta is. Looks like I am wrong though.
What do I have to change to make this simple example work?
Update
After trying some more stuff, I identified that the same error occurs when I define a constant hyperparameter with integer value, run smac, which saves the configuration space in a file, and I try to recreate the runhistory (the snippet is the second part, so it assumes the run is already performed and saved):
from smac.scenario.scenario import Scenario
import os
run_dir = 'smac_output_cs4_runcount_80_ntrain_1024/run_1'
with open(os.path.join(run_dir,'scenario.txt')) as fin:
    scenario_kwargs = {}
    for line in fin.readlines():
        key, val = line.split('=')
        scenario_kwargs[key.strip()] = val.strip()
old_scenario = Scenario(scenario_kwargs)

from smac.runhistory.runhistory import RunHistory
rh1 = RunHistory(aggregate_func=None)
rh1.load_json(
    os.path.join(run_dir,'runhistory.json'),
    old_scenario.cs,
)

Error Message
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-10-a2804aeeaf0e> in <module>
      3 rh1.load_json(
      4     os.path.join(run_dir,'runhistory.json'),
----> 5     old_scenario.cs,
      6 )

~/anaconda3/envs/sandbox_env/lib/python3.6/site-packages/smac/runhistory/runhistory.py in load_json(self, fn, cs)
    402             int(id_): Configuration(
    403                 cs, values=values, origin=config_origins.get(id_, None)
--> 404             ) for id_, values in all_data["configs"].items()
    405         }
    406

~/anaconda3/envs/sandbox_env/lib/python3.6/site-packages/smac/runhistory/runhistory.py in <dictcomp>(.0)
    402             int(id_): Configuration(
    403                 cs, values=values, origin=config_origins.get(id_, None)
--> 404             ) for id_, values in all_data["configs"].items()
    405         }
    406

ConfigSpace/configuration_space.pyx in ConfigSpace.configuration_space.Configuration.__init__()

ValueError: Trying to set illegal value '42' (type '<class 'int'>') for hyperparameter 'random_state, Type: Categorical, Choices: {42}, Default: 42' (default-value has type '<class 'str'>').

The problem is the configspace - there are problems recovering integers or booleans in CategoricalHyperparameters as they are imported as strings. While this is annoying and we need better errors, it's complicated and probably not really worth fixing (at least not in CAVE), since using  .json instead of .pcs format works just fine. So for your example I created a configspace.json in your SMAC-run and redirected the scenario.txt-paramfile to that json. Now it works perfect.
Are you using the newest SMAC-version? Because then I'll file this as an issue there, since SMAC should at least save both configspace-formats...
Feel free to ask, if there is anything unclear :)

Thanks Joshua! The example is running indeed. It should get me started with CAVE. I am using SMAC 0.11 now and I do get configspace.json as an output, but when I created the repo, I think I may have used 0.10.


Fly rules aren't forcing

According to this site: https://www.pagat.com/kt5/pengyou.html

If anyone - even a partner of the leader - has a card or combination in the suit of the lead that can beat one of the cards or combinations in the lead, they must immediately show it.

But the rules don't force anyone to declare no-fly.

yeah, this is a known unimplemented feature, largely because i couldn't come up with a good UI for it.  when you play in person, this is sometimes a game of chicken, where you wait for someone else to contest the fly, even though you are also able to.  i guess i could block people from saying "pass" if they are able to contest, but that seems a little bit confusing still.

oh, in fact, i guess this was one of the first issues i filed.
Duplicate of #2


Cursor issue

Hey Sankar,
The cursor seems to appear by your method!! Attaching screen shot.

I have add a css attribute display: inline; to p tags. Seems resolved after that.


Fix calcuation of angle difference

In the function of angle difference calculation, NaN value is returned when the absolute value of inner product is larger than 1.
https://github.com/autowarefoundation/autoware.universe/blob/045009c4834f9c0ae9c2a0513e6f6c12ee88e86a/map/lanelet2_extension/lib/query.cpp#L38-L45
This is because arg of atd::acos shouldn't be outside the range [-1.0, 1.0] expalined here.
Numerical error happens in some cases so gurad needs to be added.

@kyoichi-sugahara Could you explain specifically in what situations the inner product will be larger than 1, please?

@kenji-miyake
Sorry for not describing the detail.
I had same a same problem with the link
When angle1 and angle2 are the same, vec1.dot(vec2) expected to be 1. but in some cases, the value which is larger than 1 is calculated because of numerical error.


Elements: changes to layout mechanism

Hi guys,
Learning from my mistakes :)
This issue is to discuss elements' layout update I suggested in https://github.com/playcanvas/engine/pull/816.
The idea is to have the following concepts of layout modifiers:

Anchors: left, bottom, right and top fractions in parent coordinate space which drive the bounding box of an element
Corners: left, bottom, right and top offsets of corresponding element edges from anchors
Pivot point: the origin of local transforms in current element coordinate space.

This "complication" comes in necessary to fluently support all possible layout variants, namely: dynamic width/height, fixed width/height, dynamic width/height with paddings, offsetting the element from anchor-driven position.
Pivot point is further separates transforms from element's positioning to allow for the things like rotating around an edge/corner, dynamic offsets etc.
The existing system of anchors and pivot offsets seems limited to me: for instance, it's quite difficult to make a button that takes 100% of the width and has 10px padding around it. Also, pivot point driving element offset makes animations (consider flipping animation) clash with position offsets – still doable, of course, but user code becomes more complicated.
The last but not least: this system is a "copy" of Unity's UI system that seems to be performing quite well – so why not re-use it? :)
@daredevildave please let me know if the above explanation makes sense to you and I will prepare a PR for this part of the original pull request.

I believe you can accomplish all this already using the left/right/top/bottom properties.
In your example something like this:
this.entity.element.anchor = [0,0,1,0]; // minX = 0, maxX = 1
this.entity.element.height = 100;
this.entity.element.left = 10;
this.entity.element.right = 10;

Technically you are suggesting the right thing, but the devil is in certain details, as usual.
I've been playing with master branch on that and found a few issues with the current implementation:

Anchor offsets aren't stored explicitly anywhere, and rely on properties like _width and _absLeft upon setting/retrieval. The most obvious implication is this code:

panel.addComponent('element', {
    anchor: new pc.Vec4(0, 0, 1, 1),
    pivot: new pc.Vec2(0.5, 0.5),
    left: 0,
    right: 0,
    top: 0,
    bottom: 0,
    type: "image",
    textureAsset: panelAsset
});

Which will not work (as there might be no parent yet or the width might not computed) – please mind I did fix element/system.js to pass these props on before testing it :)

For some reason pivot point also takes part in this computations. While I do appreciate it might be a "by design" thing, I still believe it's much more reasonable to separate layout box (driven solely by anchors and corners) from pivots and transforms: you don't want to mix those and surely you don't want changes to transforms (for animation purposes) affect the layout box.
Setting pivot point changes position, which means pivot does something other properties do. If you want to change the position, why don't you change anchors/left/top/bottom/right? And it's not about pivot being relative to the size of an element – the former properties do allow for that as well.
Minor, but still: you use a single property for anchors, but went for 4 properties for corners and that's why I introduced borders property. Just being consistent, you know :)

Hey @daredevildave,
Sorry to bother you during holidays – but maybe you have any comments on the above?

Sorry, I'm not back into the office until next week. So I'll be taking a proper look then.
Perhaps we are working from a different basic idea, but I don't really understand your suggested changes.
With respect to pivot and localPosition. You say: "how do I modify localPosition once pivot is set?" In the current system, the local position is the position of the pivot point with respect to the parent. To position an element you just call setLocalPosition(). If you move the pivot the then local position is unchanged, but the element moves (for example if I move pivot from left edge to right edge an image element would move left by one width).
Changing the values of "left", "right", "top" and "bottom" changes the width, height and local transform. Imagine the pivot point is on the left edge, and I change the value of "left". By definition the local transform must change because the left edge (where the pivot point is) has moved. How do you propose separating pivot from position?
There are 5 properties which define an element's size and position: x,y,z (local transform), width and height. The corners or left, right, top, bottom can all be expressed in terms of these properties.
And by the way, as I understand it. The existing system is very similar to Unity's UI already?

Hi,
No probs – holidays are holidays :)
First of all, in your example ("for example if I move pivot from left edge to right edge an image element would move left by one width") – I think that's the issue. You are moving pivot and thus change the position of the element. But why? You already have anchors and top/left/right/bottom affecting the position. Wouldn't it make more sense for the pivot to define the origin for transforms, and leave positioning (sans localPosition, of course) and sizing to layout mechanism?
Changing local transform with modifying left/top/right/bottom means you lose the flexibility to animate elements as local transform might be "dirty" making it difficult to animate the element to/from designed position. What I propose is to keep local transform operating "around" pivot point and make it be identity unless modified explicitly. This way you design the interface by placing elements and then animate to show, hide, rotate and do whatever you'd like. Resetting local transform to identity will put the element back and that's it.
Also, you aren't entirely correct re width and height driving positioning: width and height can be computed from corners, but not the other way around. The easiest example is an element without a parent: having undefined width and height it can still ask for specific corner values, and this shouldn't be a problem.
As for being very similar – as I said, the devil is in the details, and I just propose to eliminate these details :)

Hey @daredevildave, sorry to bother you - any thoughts on the above?

OK, I've been looking into this a bit more.
Using corners property and using that to calculate width/height could make sense. Though in your example (no parent) your code has invalid width and height so it's seems to just trade one issue for another.
I'm still not sure if I'm missing the point of your suggestion around pivot point, but I'm still pretty confused over what you gain.

The local position is (0,0,0) but because of the corners the pivot point is now at (64,64,0). This just seems wrong and means we would need another property to expose the pivot point. Why introduce another position?  I haven't seen a concrete example of a use case where the current system doesn't work.
Using the Unity UI as a preview. How would you set up the example in screenshot above in Unity. It doesn't seem like it is possible to me?

It looks like this (as I would expect).

Re invalid width / height – my point here was that the position of the element should be defined by corners and anchors, and should not depend on the presence of parent. What's wrong with creating a button, telling it to take full parent rect sans 10px marging, but adding it to the parent rect later?
Re pivot: technically the same applies here. If we lay elements out using anchors and corners, why would the positioning depend on the pivot? Consider two screenshots from Unity illustrating exactly this:

Pivot at bottom left corner: https://www.dropbox.com/s/od14ng22sn51p3u/Screenshot 2017-01-18 18.47.35.png?dl=0
Pivot at top right corner: https://www.dropbox.com/s/x0fyh2059agq18c/Screenshot 2017-01-18 18.47.45.png?dl=0

As you can see moving pivots around doesn't change the corner or anchor values, neither does it move the component (disregard float precision errors in the inspector).
The results of rotations, for instance, would, however differ for both cases:

Pivot at bottom left corner: https://www.dropbox.com/s/cczbodsv7oqnxx5/Screenshot 2017-01-18 18.49.39.png?dl=0
Pivot at top right corner: https://www.dropbox.com/s/i4vgzwmrf7jzdkm/Screenshot 2017-01-18 18.49.55.png?dl=0

And this is totally fine (I think): you lay your interface out using corners and anchors full stop; then you move pivot point as you wish and tune local transforms to animate things, tweak the position etc.
I should admit, however, the local position does get changed when pivot moves around – my mistake; but mind one wouldn't normally modify local position of UI elements – again, you have rect transform for that (and that's why in Unity you don't even see it unless switched to debug inspector mode).
Hope this makes sense :)

OK, well I think we're getting closer to the same thing now :-)
In your pictures from Unity you can see that Unity is exposing the l/r/t/b values when the anchors are split and yes those values remain constant when you change the pivot.
PlayCanvas has the same behaviour. Set the the same values as above:

OK, well I think we're getting closer to the same thing now :-)
In your pictures from Unity you can see that Unity is exposing the l/r/t/b values when the anchors are split and yes those values remain constant when you change the pivot.
PlayCanvas has the same behaviour. Set the the same values as above:
pivot: [0,0]
anchor: [0.35,0.35,0.65,0.65]

then set element.left = 30; element.right=30;element.top=30;element.bottom=30; and you will see the same behaviour. Changing the pivot point from 0,0 to 1,1 changes the local position but doesn't change the anchors or l/r/t/b.

I might've missed smth in my tests, but I clearly remember that changing pivot point did move the element (although I didn't change anchor). Is it supposed to be like so?

The element will move because setting the pivot in code (element.pivot = new pc.Vec2(1,1)) doesn't change the local position. If you counter the pivot change by changing the local position (which is what the Unity editor does by default) then the element won't move.
e.g.
// start with pivot at 0,0

// move pivot without moving element
entity.element.pivot = new pc.Vec2(1,1);
entity.translateLocal(entity.element.width, entity.element.height, 0);

So, what I'm going to do...
I'm currently looking into how to best store these values internally (e.g. possibly replace width/height with corners property). So that you can assign all values successfully before adding to the hierarchy.
But I believe the rest of the behaviour is correct (and consistent with how you expect it to behave). I'll try and put together a test page so that we can fiddle some numbers and check everything is behaving as expected.

Take a look at this: https://s3-eu-west-1.amazonaws.com/ui-layout.playcanvas.com/ui/positioning/index.html
It lets you fiddle with properties of a two element hierarchy.

Hey,
Just did play with the page. Generally looks sane, but it looks like I don't quite get what's the purpose of pivot point in your example. It does move element around why corners property is unchecked (not sure what it means logically though) and doesn't seem to change position when corners are set. Positioning behaves the same – no change when element is driven by corners.
Is the behaviour expected or it's some sort of a bug?

Just merged this change which adds support for margin property (this the collection of left/right/top/bottom)
This fixes also lets you define all properties up front before you add element to the hierarchy.
I'm pretty much considering this a fix to this issue, but let me know if you think there are any more use-cases that don't work.


[Question] Initialize SingleTenantAzureAuthorizationCodeBearer at non-top level

Related SO question: https://stackoverflow.com/questions/75262398/fastapi-read-configuration-before-specifying-dependencies.
Hi. Let me just say that I'm very happy to use this library because it made the Azure authentication for me a piece of cake!
I have a problem with initializing SingleTenantAzureAuthorizationCodeBearer with values from a config file.
After I've succesfully added authentication to my API following the guide on the docs page, I started to prepare a version that would be production ready. And that would mean: values read from a config file. That's because we use Kubernetes so we have an easy way to provide a ConfigMap for a container.
Let's say this is my API code (I can split this into multiple files, but in this case it doesn't matter):
from fastapi import FastAPI
from fastapi_azure_auth import SingleTenantAzureAuthorizationCodeBearer
from pydantic import BaseSettings
import json

class Settings(BaseSettings):
    client_id: str
    tenant_id: str

with open(os.getenv("CONFIG_FILE", "config.json")) as f:
    config_data = json.load(f)

settings = Settings(client_id=config_data["client_id"], tenant_id=config_data["tenant_id"])

azure_scheme = SingleTenantAzureAuthorizationCodeBearer(
    app_client_id=settings.APP_CLIENT_ID,
    tenant_id=settings.TENANT_ID,
    scopes={
        f'api://{settings.APP_CLIENT_ID}/user_impersonation': 'user_impersonation',
    }
)

api = FastAPI(
   title="foo"
)

def init_api() -> FastAPI:
   # I want to read configuration here
   api.swagger_ui.init_oauth = {"clientID": config.CLIENT_ID}
   return api

@api.on_event('startup')
async def load_config() -> None:
    """
    Load OpenID config on startup.
    """
    await azure_scheme.openid_config.load_config()

@api.get("/", dependencies=[Depends(azure_scheme)])
def test():
   return {"hello": "world"}

This obviously works because both azure_scheme and settings get evaluated before load_config and test route definition.
However, this makes testing ugly, because now there's a config file to be read. Even pytest fixtures cannot be evaluated before imports in tests, so I end up with errors - "no config.json" etc. Plus, even if I'm able to "hack" this, every time I do an import from api.py, this code gets run - it's just not good.
I provided an init_api function. This is a perfect example - in my code I do other stuff there, initialize logging, do some other stuff. That way, in tests, I don't have to call init_api, I just import from <app_name>.api import api and do test on it - therefore I dodge all the stuff that I'd otherwise have to mock.
My only hack was to do something like this:
# [...]
settings = None
azure_scheme = None

api = FastAPI(
   title="foo"
)

def init_api():
    with open(os.getenv("CONFIG_FILE", "config.json")) as f:
        config_data = json.load(f)
    global settings
    settings = Settings(client_id=config_data["client_id"], tenant_id=config_data["tenant_id"])

    global azure_scheme
    azure_scheme = SingleTenantAzureAuthorizationCodeBearer(
        app_client_id=settings.APP_CLIENT_ID,
        tenant_id=settings.TENANT_ID,
        scopes={
            f'api://{settings.APP_CLIENT_ID}/user_impersonation': 'user_impersonation',
        }
    )

    api.add_api_route("/", endpoint=test, dependencies=[Depends(azure_scheme)])

@api.on_event('startup')
async def load_config() -> None:
    """
    Load OpenID config on startup.
    """
    if azure_scheme:
        await azure_scheme.openid_config.load_config()

def test():
   return {"hello": "world"}

So I took out the @api decorator and used .add_api_route to init_api. It works, but it's not ideal. In some ways it creates more issues than it solves (typing, more complicated code, access to globals).
Sorry for this long monologue, my question is brief: How can I initialize SingleTenantAzureAuthorizationCodeBearer as a dependency without having to read the config globally? And maybe without so much hacking as I've shown above 😄

Hi! Thanks for the kind feedback 😊
I'm on my way out, replying from my phone, so I'll answer a bit short, and follow up at a later point.
What we do is load all these things as environment variables/.env file:
class CustomBaseSettings(BaseSettings):
    """Configure .env settings for all our setting-classes"""
    class Config:
        env_file = '.env'
        env_file_encoding = 'utf-8'
        case_sensitive = True

class Env(CustomBaseSettings):
    ENVIRONMENT: Literal['dev', 'lab', 'prod', 'qa', 'test'] = Field(..., env='ENVIRONMENT')

Then our pytest.ini looks like this:
[pytest]
addopts = --allow-hosts=localhost,<IP_ADDRESS>,::1
asyncio_mode = auto
env =
    ENVIRONMENT=test
 env_files =
     .env
 env_override_existing_values = 0

Don't exactly remember the purest plug-in, but it's either pytest-env or pytest-dotenv.

This is what I ended up doing. I mean I still have this hack with the config file on a separate branch I shown above, but it's kinda messy. When I went for the env variables, it gets easier. I don't provide them via pytest because I just have defaults for them and in tests I also use api.dependency_overrides to temporarily get rid of the SingleTenantAzureAuthorizationCodeBearer dependency.
I badly wanted to go for a config file, but seeing as clientId and tenantId are not exactly secrets, they may as well be passed by env variables, even hardcoded if someone doesn't care/wont be changing it on a regular basis. So, yeah, env variables are the way to go here, much easier.
Thanks for the feedback!

You're welcome. Thanks for following up!
Have a good weekend😊

This will be much cleaner once the dependency injection feature in FastAPI supports lifespan dependencies, https://github.com/tiangolo/fastapi/issues/617 . For now, I see no way around this than configuring azure_scheme with env-variables like mentioned above.

Agree.

In case anyone lands here, I opted to solve this using a global value and omitting the lifespan event entirely. This allowed me to use the dependency injection system without validating the app settings for each and every test. The important caveat is that, just like all other dependencies, the configuration is not validated on startup, but deterred until the first request. I get around this by writing some custom health-checks that k8s wait for (readiness hooks).
from typing import Annotated

from fastapi import Depends
from fastapi_azure_auth import SingleTenantAzureAuthorizationCodeBearer

from myapp.settings import Settings

azure_scheme = None  # When tests import this module, azure_scheme is not instantiated

def get_settings() -> Settings:
    """Settings are injected, not global"""
    return Settings()

SettingsDep = Annotated[Settings, Depends(get_settings)]

async def get_azure_scheme(settings: SettingsDep) -> SingleTenantAzureAuthorizationCodeBearer:
    global azure_scheme  # noqa: PLW0603 Use global until lifespan dependencies are available
    if not azure_scheme:
        azure_scheme = SingleTenantAzureAuthorizationCodeBearer(
            app_client_id=settings.azuread_app_client_id,
            tenant_id=settings.azuread_tenant_id,
            scopes={
                f"{settings.azuread_app_identifier_uri}/{settings.azuread_scope}": settings.azuread_scope
            },
        )
    return azure_scheme

AzureSchemeDep = Annotated[SingleTenantAzureAuthorizationCodeBearer, Depends(get_azure_scheme)]


a2dp source: how to confirm AAC is being used?

Hi,
I'm using a PiZeroW as a2dp source to a Bose Speaker.
I compiled bluealsa with AAC support (no debug option), but I'd like to verify/confirm this codec is indeed actually being used in place of SBC.
How can I confirm that?
I did not find an option (in bluealsa parameters or .asoundrc) to enforce it either.
Thanks!

See this issue: #283

Thanks for the feedback, sorry I missed it in my search.
Here is what I get from dbus command (removed MAC):
({objectpath '/org/bluealsa/hci0/dev_XX_XX_XX_XX_XX_XX/a2dp': {'Device': <objectpath '/org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX'>, 'Modes': <['source']>, 'Format': <uint16 32784>, 'Channels': <byte 0x02>, 'Sampling': <uint32 48000>, 'Codec': <uint16 0>, 'Delay': <uint16 150>, 'Volume': <uint16 32639>}},)
So it turns-out that it connects with SBC
I checked Speaker capability with hcidump avdtp as explained in related links.
HCI sniffer - Bluetooth packet analyzer ver 5.52
device: hci0 snap_len: 1500 filter: 0x400
< AVDTP(s): Discover cmd: transaction 0 nsp 0x00
> AVDTP(s): Discover rsp: transaction 0 nsp 0x00
    ACP SEID 1 - Audio Sink
    ACP SEID 5 - Audio Source
    ACP SEID 6 - Audio Source
< AVDTP(s): Set config cmd: transaction 1 nsp 0x00
    ACP SEID 1 - INT SEID 3
    Media Transport
    Media Codec - SBC
      48kHz
      JointStereo
      16 Blocks
      8 Subbands
      Loudness
      Bitpool Range 2-51
> AVDTP(s): Set config rsp: transaction 1 nsp 0x00
< AVDTP(s): Open cmd: transaction 2 nsp 0x00
    ACP SEID 1
> AVDTP(s): Open rsp: transaction 2 nsp 0x00

Which is somehow consistent, but I'm very surprised, as I thought this product (Bose Portable Home Speaker) was supposed to be AAC and AptX compatible, which do not show up here.

Maybe you have to enabled it somehow. The same thing is with Sony WH-XM headphones, you have to enable audio quality over connection stability by proprietary app. Otherwise you have SBC only.

Thanks. Not possible unfortunately.
More generally, if several advanced codecs are available (AAC, AptX,...) can bluealsa set a priority order of some sort to select one over another?

can bluealsa set a priority order of some sort to select one over another?

Nope, it's not possible I'm afraid. Currently, bluealsa tries to connect best codec at first. If none of AptX, LDAC, AAC can be connected, then SBC will be used. However, this behavior is true only for up to two BT devices connected to bluealsa. If you connect 3rd device, selected codec might be different.... I don't know how to force BlueZ to select codec in a specified order. In the future, it will be possible to change A2DP codec on runtime via BlueALSA D-Bus API :)

Thanks, very interesting.

bluealsa tries to connect best codec at first.

If all three AptX, LDAC, AAC are available, which is considered best by bluealsa then? (and 2nd best if first best missing?).

The order is here: https://github.com/Arkq/bluez-alsa/blob/master/src/bluez-a2dp.c#L412 but it is more or less hack over BlueZ. It seems that BlueZ uses codecs in the order of registration - firstly registered, firstly tried. I have not found any proper way of defining codec priority in BlueZ....

Got it, thanks.


Thread pool functionality

The Pi won't sync with the flight computer -- we need a "waiting room" of sorts to hold the flight computer requests and let the pi process them as they come or as fast as it can.

issue done -- merged into dev branch


LAB_AK_05_Implementing_Azure-based_recovery_services.md - Virtual Networks creation, Intermittent new UI presented

Module: 00
Lab/Demo: 05
Exercise 2
Task: 01
Steps: 02-08
Description of issue
This is likely something that cannot be (efficiently) resolved in Instructions until the Azure UI 'stabilizes' but I at least wanted to call it out as a potential issue:  When creating an Azure Virtual Network, there is both an old UI and a new UI.  The old UI matches the instructions here in Exercise 2, Task 1, Steps 2-8.
Unfortunately, since ~Oct 2022 last year there can appear intermittently a new UI that has slightly revised the order of tabs and the click-through a student must take to successfully create a Virtual Network.  The changes are relatively minor so hopefully the majority of Students are able to navigate the new process successfully.
I was just testing this morning and out of 10 launches, none had the new UI.  A few days ago, the same test had 2 launches with the new UI so unfortunately, we have no way to be sure what appears in any given launch.  I haven't been able to find any way to force the old (or the new) UI, so unless a whole subset of Instructions was written to deal with both potential UIs a student might get, the alternative is just to await the new UI becoming ubiquitous.
I posted about the issue here, and apparently the docs.microsoft.com people are aware of it so likely we just have to be patient until the new UI occurs consistently:  https://github.com/MicrosoftDocs/azure-docs/issues/105704

Good observation and summary of what has occurred. Agree that it's best to wait until the UI catches up. Thanks for pointing out this issue.


Rename disposed extension methods to disposedBy

First raised by @iasiu in a private company project. It's too similar to dispose.

@RobertOdrowaz, what do you think?


update checksum

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

The automatic pull request didn't have the right sha256 checksum and still installs prody 2.0.1 so this is a fix for that

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

From the logs: RuntimeError: SHA256 mismatch: '1253f21717cdca7637784080e7982c04f3a6402b6b6a2f49d56cfcaa1be7ed16' != 'ed23deb8f4a54b856829e3621b574ec38522a1f141dbb7b2ba446d1fd739fbb8'
What's the reason to update it actually?

I'm trying that now. I also thought about that. I'll let you know what happens

ok, that worked:
## Package Plan ##

  environment location: /home/jkrieger/anaconda3/envs/conda_prody

  added / updated specs:
    - prody=2.0.2

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    cryptography-36.0.0        |   py39h9ce1e76_0         1.3 MB
    prody-2.0.2                |   py39he80948d_0         8.2 MB  conda-forge
    ------------------------------------------------------------
                                           Total:         9.6 MB

The following NEW packages will be INSTALLED:

  biopython          conda-forge/linux-64::biopython-1.79-py39h3811e60_1
  brotlipy           conda-forge/linux-64::brotlipy-0.7.0-py39h3811e60_1003
  cffi               conda-forge/linux-64::cffi-1.15.0-py39h4bc2ebd_0
  charset-normalizer conda-forge/noarch::charset-normalizer-2.0.12-pyhd8ed1ab_0
  cryptography       pkgs/main/linux-64::cryptography-36.0.0-py39h9ce1e76_0
  idna               conda-forge/noarch::idna-3.3-pyhd8ed1ab_0
  prody              conda-forge/linux-64::prody-2.0.2-py39he80948d_0
  pycparser          conda-forge/noarch::pycparser-2.21-pyhd8ed1ab_0
  pyopenssl          conda-forge/noarch::pyopenssl-22.0.0-pyhd8ed1ab_0
  pysocks            conda-forge/linux-64::pysocks-1.7.1-py39hf3d152e_4
  requests           conda-forge/noarch::requests-2.27.1-pyhd8ed1ab_0
  scipy              conda-forge/linux-64::scipy-1.8.0-py39hee8e79c_1
  urllib3            conda-forge/noarch::urllib3-1.26.8-pyhd8ed1ab_1

Proceed ([y]/n)? y

Downloading and Extracting Packages
prody-2.0.2          | 8.2 MB    | ################################################################################## | 100%
cryptography-36.0.0  | 1.3 MB    | ################################################################################## | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done

Python 3.9.10 | packaged by conda-forge | (main, Feb  1 2022, 21:24:37)
Type 'copyright', 'credits' or 'license' for more information
IPython 8.0.1 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import prody

In [2]: prody.__version__
Out[2]: '2.0.2'

In [3]:

In [3]: from prody import *

In [4]: y = searchPfam("P19491", timeout=1000)
@> Retrieving Pfam search results: https://pfam.xfam.org/protein/P19491?output=xml
/home/jkrieger/anaconda3/envs/conda_prody/lib/python3.9/site-packages/urllib3/connectionpool.py:1043: InsecureRequestWarning: Unverified HTTPS request is being made to host 'pfam.xfam.org'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(
@> Pfam search completed in 0.71s.
@> Query 'P19491' matched 3 Pfam families.

Pull request closed

Thanks very much for your patience


New version: uazo.cromite version 128.0.6613.120

Created by 🥟 Dumplings in workflow run #4006.
Logs
Updated: 128.0.6613.116 → 128.0.6613.120
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.
Submitting WinGet manifests
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 894.)


feat(RHTAPREL-360): create add-fbc-contribution task

This PR creates the new task add-fbc-contribution to replace the  task create-internal-request

Is this meant to be a draft PR or actually ready for review? I ask because there are references to tasks from a fork

Ah yes. I forgot to change it back to the proper git ref. Now it is ready for review.


[Fix] Fix unit test of datasets

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Modification

Fix unit test of datasets by regenerating new annotation files.
Reduce cuda memory in UT.

BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

I guess sample_id in two places: https://github.com/open-mmlab/mmdetection3d/blob/dev-1.x/mmdet3d/evaluation/metrics/kitti_metric.py#L419 and https://github.com/open-mmlab/mmdetection3d/blob/dev-1.x/mmdet3d/evaluation/metrics/kitti_metric.py#L511 don't need to be replaced by sample_idx ?

The package of waymo-open-dataset-tf-2-1-0 does not support the env of python3.9.


Changed the color of sliders.

Fixes #(issue)
#75
Describe your changes in detail
Changed the color of sliders.
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

SWOC Participation

Name: Madhur Vatsal
Discord Username: Madhur Vatsal#3062
[x] I have included the issue number in my PR

LGTM. Thank You  for the PR. Merged. Also please update the PR description with your name and discord id.

Updated! Moreover,it was a great experience contributing in your project!


add Japanese (Japan) locale

@Leecason
Hi! I'm planning to use element-tiptap for a project and would like to use it in Japanese. As long as I see this PR and the Japanese translation looks great. Integrating this PR would significantly benefit users requiring Japanese localization, including myself. Could you please consider merging it? Thank you so much for your hard work on this project!


bump: Release @logdna/logger v1.0.0

Contains code improvements carried over from the soon-to-be-deprecated
nodejs package.
Semver: major

Holding off for a bit.  We might have to allow the ingester to handle more slashes in the userAgent string.

Let's move forward with the release and we can deal with the necessary ingester changes after. Our ingester today already has several known issues that we need to address anyways.


Initial feature-alternates branch commit

Description
The alternates feature allows one to have and maintain different versions of a configuration file in the same .dotfiles/ (sub)directory, where each version is specific to the operating system you are running. The bootstrap script will automatically link the correct configuration file, based on the operating system, host or user.
The mechanism is based on a certain file name convention, and is fully compatible with the standard .symlink extension.
Examples:
-have a .mailcap file for Darwin, and a .mailcap for Linux. Bootstrap will link the right one.
-a .kshrc (korn shell) does not get linked on Darwin. Bootstrap will ignore this file on Darwin, but link it when we are on NetBSD.
The inspiration for this feature comes from Tim Byrne's yadm.

I think this is interesting, but probably not the direction I want to head right now. My repo is set up more or less to be specific to one platform (as, well, a reflection of how I use it personally, really).
I think this is promising, though, and hopefully the pull can be interesting for others in a similar boat to come in and use for their own projects, so thanks for that. :sparkles:
I'm going to close it in the meantime, though- perhaps we'll revisit it in the future.


Added config flags to modify appium log levels

TODO update readme with config options

@wt-johncuthbert can you add this info to the readme.md? That way everyone knows you can actually modify the logs ;)


MOD: fix build error in golang tutorial code

In golang, if the import is not used, the code will not compile successfully.
So the "fmt" in the golang sample code must be removed, then it can be compiled successfully.

@sempr Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@sempr Thank you for signing the Contributor License Agreement!

Thank you!


Add zara website clone

Description
It will be a responsive clone of zara made with pure HTML , CSS & JS
Contribution made under GSSOC'22
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign


Firestore CI

Overview

Add Google Cloud Firestore emulator CI runner.
Add anonymous Firestore credential setup for emulators.
Change tox dependency to just Firestore instead of Firebase.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop-google-firestore-instrumentation@b8333fa). Click here to learn what that means.
The diff coverage is n/a.

@@                             Coverage Diff                             @@
##             develop-google-firestore-instrumentation     #877   +/-   ##
===========================================================================
  Coverage                                            ?   80.95%
===========================================================================
  Files                                               ?      186
  Lines                                               ?    18581
  Branches                                            ?     3229
===========================================================================
  Hits                                                ?    15042
  Misses                                              ?     2654
  Partials                                            ?      885

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix(color): Convert color space for useRGBA false

Fixes #3354
Context
The convertColorSpace function wasn't being called consistently or correclty for all situations.
According to the DICOM standard, as best I understand it:

The Photometric Interpretation (PMI) is used for uncompressed + RLE encodings
For colour images encoded in compressed syntaxes (JPEG, JPEG-LS, JPEG-LOSSLESS, JP2000, HTJ2K), the PMI and Planar Configuration are assumed to be whatever the decompressor provides - which is typically RGB, as the colour space is (usually) encoded in the image.

This does NOT get 100% of images correct because there can be mismatches between what is encoded and decoded, however, it gets the majority of them correct, and it is really obvious when they are wrong.
Changes & Results
Consistently use the RGB colour space as the output from compressed images, assuming the decoder takes care of conversion to RGB
Apply RGBA to RGB for RGBA decoders not applying colour space (that is, for JPEG)
Add colour space RGB with RGBA to RGB conversion included
Testing
Run the test dataset from
https://github.com/OHIF/Viewers/issues/3354#issuecomment-1669470931
Suggest using the dcm4che toolkit to convert some of these to various encodings
Then use static-dicomweb to store locally (including some unconverted instances)
**** Use a local/udpated version of mkdicomweb as a change was added to fix a bug in conversion
Make sure the images look right (see the original bug issue)
Checklist
PR

[] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[] My code has been well-documented (function documentation, inline comments,
etc.)

[] I have run the yarn build:update-api to update the API documentation, and have
committed the changes to this PR. (Read more here https://www.cornerstonejs.org/docs/contribute/update-api)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[] "OS:
[] "Node version:
[] "Browser:

The image below is against the original bug report:

interesting, thanks!


Resonsible navigation?

Hey!
I really like this theme. I want to use it for my portfolio too. I already started, now I noticed that the navigation is not really responsible. Are there any plans to change that?
Otherwise I'd unfortunately would need to switch to a different theme, as responsibility is a must for me.
Regards,
Peter

Hi Peter, could you please provide a screenshot of where the navigation is not responsive and the browser you're using?

Stale issue; closing.


altering gatk haplotypeCaller parameters

Does anyone know how to adjust the haplotypeCaller parameters when runnign bcbio? For example, this is a default for gatk 3.8:
standard_min_confidence_threshold_for_calling=10.0
How do I change it to 0?
Is this hard coded in a bcbio config file somewhere? Can we change it in the sample yaml file for example?
thanks - Robert

Robert;
You can change custom parameters for HaplotypeCaller with resources options in your sample YAML file:
https://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#sample-or-run-specific-resources
So having:
resources:
  gatk-haplotype:
    options: [--standard_min_confidence_threshold_for_calling=0.0]

Practically that's going to generate a ton of noise as you'll be emitting a lot of low quality calls.
Hope this helps.

yes thank please close ticket


Improve case Transfer Max

Describe the task
Improve case Transfer Max

[x] Still show incorrect Amount on the Confirmation screen in case change Destination
[x] Discuss more: Keep the Transfer Max when change token, change sender

Resolved on extension version v1.2.28


Textures directory is case sensitive on Linux

The source looks for the 'textures' directory but Linux is case-sensitive for file names. The repository contains a 'Textures' directory.

Thx for the info. Is fixed in 8e87ffd.


[Subtask] Ignore privileges in RangerAuthorization that are not supported

Describe the subtask
RangerAuthorization uses strict privilege checking and throws an exception if an unqualified privilege is passed in.
Parent issue
#4615

Fixed by #5190


Media transformer : Inquiry About Support for Transition Effects Between Media Items(Videos) in Media3

Hi Media3 Team,
I’m currently working with Media3 and have successfully implemented video effects on individual media items. However, I haven’t been able to find any support for transition effects(e.g., fade, slide, wipe effects) between consecutive media items (videos).
Could you please let me know if such transition effects are currently supported in Media3? If not, are there any plans to introduce this feature in future versions?
In documentation I see crossfade as a limitation but curious if any other transition effects are possible.
https://developer.android.com/media/media3/transformer/multi-asset#limitations
Screenshot from official media3 docs

For more clarity , below are the examples on what  I mean by transition effects between videos.
Fade In/Out: The video gradually appears (fade-in) or disappears (fade-out).
Crossfade: One video gradually fades out as the next video fades in, creating a smooth overlap between the two.
Wipe: One video is replaced by another as the second video slides in from a specific direction (like from left to right).
Dissolve: The first video gradually fades out while the next video fades in, overlapping in the middle.
Thank you for your time and support.
Best regards,
Avinash

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

@droid-girl  Could you please help us with this question.

Hi @avinash--reddy, transition effects you mentioned are on our roadmap.

Hi @droid-girl . Could you please help with this query.

Hi @droid-girl Thank you for the update! I’m glad to hear that transition effects are on the roadmap. Could you please share any details on the timeline or expected release when this feature might become available? It will help us in or project planning and evaluation of available options. Thanks !!

@avinash--reddy we do not have the exact timeline yet for this feature. I may have more information at the end of the month.

@droid-girl  Thanks for the reply. Until media3 transformer supports transitions,  any recommended approaches or third-party libraries that can be integrated with Media3 to achieve this?
I’m looking for a solution that can be integrated smoothly with the Media3 architecture, so any insights or recommendations would be greatly appreciated. Thanks.

There is currently no other way or a third part library that you can use to do that together with media3.


mongodb_user provider does not support creating the user in multiple databases

The mongodb_user provider does not support creating the user in multiple databases and including the resource more than gives a duplicate resources error.
I believe the mongodb_user provider should accept an array for the database parameter and create the user in each one. Something like
mongodb_user { testuser:
  name          => 'testuser',
  ensure        => present,
  password_hash => mongodb_password('testuser', 'p@ssw0rd'),
  database      => [ 'foo', 'bar' ]
  roles         => ['readWrite', 'dbAdmin'],
  tries         => 10,
  require       => Class['mongodb::server'],
}

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 5.5
Ruby: 2.4.3p205 (2017-12-14 revision 61247) [x86_64-linux]
Distribution: Centos 7
Module version: 2.1.2

I think I hit the same issue. I would like to create a monitoring user and this user needs access to admin db for server status and so on, and access to some other database to store some data about cluster health. So I thought I could simply do:
    mongodb_user { 'monitoring':
        database      => 'admin',
        roles         => ['readAnyDatabase', 'clusterMonitor', 'readWrite@nagios'],
        [...]
    }

but got:
Error: Failed to apply catalog: Parameter roles failed on Mongodb_user[monitoring]: Invalid value "readWrite@admin". Valid values match /^[\w-]+$/. at /etc/puppetlabs/code/environments/test_mongo/manifests/my-mongo.pp:60

I solved it by changing the  regex here
https://github.com/voxpupuli/puppet-mongodb/blob/v2.2.1/lib/puppet/type/mongodb_user.rb#L43
for ^([\w-]|@)+$ (to includes @) so it matches readWrite@nagios.
I am not an expert and it may break other things. Not sure if it is the right fix too and I have no idea how to test it properly, but seems to works for mongo 3.0+.


Data Analysis

In this paper I'm analyzing the Syrian using Data Science

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@KevorkSulahian Thank you for submitting a pull request (PR). I have closed the PR because, as noted in the other PRs I have closed, the changes you have made targets the wrong file. If you genuinely want to contribute by writing an R-related article, please refer to the comment @erictleung has kindly made in #20050, and perhaps on more general usage/topics, too.
I have also marked this PR invalid for Hacktoberfest as I believe that it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details

@KevorkSulahian I have removed the invalid label because I may not have thought through it well enough and there is a chance that it wasn't intentional. Many apologies for any confusion caused.


Stimulus.js comments do not reload the page

[ x] I have tried upgrading by running bundle update vite_ruby.
[ x] I have read the troubleshooting section before opening an issue.

Description 📖
Commenting a code that modifies the content of the DOM from a Stimulus controller will not refresh the page even though the server detects the change.
Refreshing the page manually will.
I tried the same thing from the application.js entrypoints and it worked and from the controllers/index.js and it worked.
I also tried the same thing from the hello_controller file but outside of the controller itself and it doesn't work eithe.
Reproduction 🐞
You can use this template to have the exact same repo I'm working with
https://github.com/wJoenn/rails-vite-templates

Vite Ruby Info
Run bin/rake vite:info and provide the output:
bin/vite present?: true
vite_ruby: 3.3.0
vite_rails: 3.0.14
rails: <IP_ADDRESS>
node: v16.15.1
npm: 8.11.0
yarn: 1.22.19
pnpm:
ruby: ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [x86_64-linux]

installed packages:
app@ /home/joenn/code/wJoenn/vite
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
└──<EMAIL_ADDRESS>

Logs 📜
If not providing a reproduction:

Output
Run DEBUG=vite-plugin-ruby:* bin/vite dev or DEBUG=vite-plugin-ruby:* bin/vite build and provide the output:
➜  vite git:(master) ✗ DEBUG=vite-plugin-ruby:* bin/vite dev
  vite-plugin-ruby:config {
  vite-plugin-ruby:config   base: '/vite-dev/',
  vite-plugin-ruby:config   build: {
  vite-plugin-ruby:config     emptyOutDir: true,
  vite-plugin-ruby:config     sourcemap: false,
  vite-plugin-ruby:config     assetsDir: 'assets',
  vite-plugin-ruby:config     manifest: true,
  vite-plugin-ruby:config     outDir: '../../public/vite-dev',
  vite-plugin-ruby:config     rollupOptions: { input: [Object], output: [Object] }
  vite-plugin-ruby:config   },
  vite-plugin-ruby:config   envDir: '/home/joenn/code/wJoenn/vite',
  vite-plugin-ruby:config   root: '/home/joenn/code/wJoenn/vite/app/javascript',
  vite-plugin-ruby:config   server: {
  vite-plugin-ruby:config     fs: { allow: [Array], strict: true },
  vite-plugin-ruby:config     host: 'localhost',
  vite-plugin-ruby:config     https: false,
  vite-plugin-ruby:config     port: 3036,
  vite-plugin-ruby:config     strictPort: true,
  vite-plugin-ruby:config     hmr: { clientPort: 3036 }
  vite-plugin-ruby:config   },
  vite-plugin-ruby:config   entrypoints: {
  vite-plugin-ruby:config     'entrypoints/application.js': '/home/joenn/code/wJoenn/vite/app/javascript/entrypoints/application.js',
  vite-plugin-ruby:config     'entrypoints/application.scss': '/home/joenn/code/wJoenn/vite/app/javascript/entrypoints/application.scss'
  vite-plugin-ruby:config   }
  vite-plugin-ruby:config } +0ms

  VITE v4.2.1  ready in 247 ms

  ➜  Local:   http://localhost:3036/vite-dev/
  ➜  press h to show help

Screenshots 📷
Provide console or browser screenshots of the problem.
https://user-images.githubusercontent.com/75388869/232280800-7ce9f4d8-7f84-4136-ac18-53d4a0e80cb4.mp4

Hi Louis!
Without this plugin, what you see is not HMR, but a full-page reload.
If you check the examples provided in this repo, you'll notice

I understand now. You are right I wasn't thinking about that commenting a DOM modification would not be restoring the DOM element to its initial value without reloading the whole page. This makes way more sense now, thank you for pointing that out to me.
Does reloading the controller with HMR actually refreshes the controller completely triggering the disconnect breakpoint in the process ? If so I didn't realize that it would. I had read about your point on idempotence but because I didn't know the disconnect method would trigger I wasn't sure what to make of that. I should have done more research.
Thank you very much for explaining that to me Massimo 🙏


Update wit-abi to keep up with wasmtime API changes

This is currently done, so closing it out! Any future changes can get new issues.


[Dubbo-1728] Fix problem: System property dubbo.service.delay invalid

What is the purpose of the change
Fixed issue #1728：System property dubbo.service.delay invalid
Brief changelog
ReferenceConfigBase.java
ServiceConfigBase.java
Verifying this change
As described in issue #1728：we can't set dubbo.service.delay to dubbo through System Property，and System-Enviroment、Property-File mechanism also can't work. furthermore, other similar-style config(eg: dubbo.service.filter、dubbo.service.retries) are also invalid.
The troublemaker which caused this problem is:
public String getPrefix() { return DUBBO + ".service." + interfaceName; }
Due to these above codes，when getProperty through CompositeConfiguration, the packed key is （for examle）: dubbo.service.org.yiocio.testService.org.yiocio.testService.delay，this key's format is wrong according to dubbo's config strategy. In my view, dubbo's property-style config have a namespace concept: the first segment is dubbo at all times, the second segment is tag for config category（eg: application、registry、service），and the third segment is the id of config object with specific category，of course，the third segment is optional, the last segment is config key. So the override of getPrefix for ServiceConfigBase will break the namespace rule of dubbo config.
ReferenceConfig's situation is similar with ServiceConfig.

trigger CI


Port one of LazyFoos SDL Tutorials to golang

There are many more tutorials over at http://lazyfoo.net/tutorials/SDL/index.php to port from C to golang. If you want to port a tutorial please comment here so we can avoid multiple people working on the same tutorial.

Can I work on 09?

@ivcoar sure :)


The type 'Map' is declared with 0 type parameters, but 2 type arguments were given.

Steps to Reproduce

I upgraded google maps to google_maps_flutter: ^0.3.0
tried using this Map<MarkerId, Marker> markers = <MarkerId, Marker>{};

error
The type 'Map' is declared with 0 type parameters, but 2 type arguments were given.dart(wrong_number_of_type_arguments)

and markers type as Map markers instead of Map<MarkerId, Marker> markers when I hover over the variable.
Logs

[√] Flutter (Channel beta, v1.2.1, on Microsoft Windows [Version 10.0.18342.8], locale en-US)
    • Flutter version 1.2.1 at C:\src\flutter
    • Framework revision 8661d8aecd (3 weeks ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\chadi\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:\Users\chadi\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
    • All Android licenses accepted.

[√] Android Studio (version 3.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 28.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[√] VS Code (version 1.31.1)
    • VS Code at C:\Users\chadi\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 2.24.0

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

I don't mean to troll or sound smart smart-alecky, but is the your class where you are declaring markers named Map? I has the same issue. Just renamed my class to something else.

@joserocha3 LOL😂, no worries and thanks, I would have taken longer to notice that never crossed my mind to check the class name.

lol, exactly the same facepalm as above. Had a google maps class as Map. Spent a while looking for the issue... cheers @joserocha3

lol, i had a same problem and spent almost 2hrs looking for issue, realize after reading this thread thanks bro @joserocha3

Boom, this thread saved my Hrs of debugging, Thanks for the solution bro @joserocha3 <B