Problems with Consent after app deployment

Hi there!
After the successful deployment of the app, I am not able to start the author app.
What I did so far

I used the manifest which have been generated during the PowerShell-Deployment
"Company Communicator (Authors)" has been allowed as a custom app using org-wide App Permission Policy
I granted all admin consents for the apps in the Azure-Portal after the deployment
App could be added to a MS-Teams Team.

What is not working?
I am now having the problem, that I cannot start the app. It looks like something is wrong with the sing-in.
I can add the "Company Communicator (Authors)" app to an existing MS-Teams Team. However, I cannot start it once it has been added as a tab since I cannot get over the Sign-In (see the following video)
https://user-images.githubusercontent.com/103029137/173878252-735d49b9-edbc-4dcc-b162-c3fc48eaaacf.mp4
Review App Permission in MS-Teams Admin-Center is not working (error message).
I also tried to review the Consent in the MS-Teams Admin Center under "Manage Apps", but even there I end-up with an error message.

After a successful login with a global admin user, I just get the error-message at the top (asking me to contact ms-support).
I could not find anything while googeling so I hope you guys can help me.
Please let me know should you have any questions or need any further information.
Looking forward to your feedback!
Best,
Mirco

Hi @enito-ch ,
There are multiple reason which might be causing the authentication error. Few areas which can be cross verified are,

Check if the secrets are correctly configured. Go to resource group, then to Key Vault and compare it with the secrets created for all the 3 app registrations.
Other possibility might be the app service URL getting blocked due to organization policy. Grab the app service URL and try accessing it from the browser. Share us the screen shot of the page with the console logs.

Also, verify the Set Up authentication steps of the graph app registration and if needed we can connect over call to troubleshoot it.
For the second issue, i.e. error which reviewing the policy. You can reach out to Microsoft Customer Support to check why you are not able to review or upload apps.

Hey @v-royavinash
The "Set up authentication steps" were not done yet. I expected that to be completed during the deployment.
I configured everything as described in the deployment guide under "Step 3 - Setup authentication steps" and now everything is working like a charm!
Thanks for your help! much appreciated!

Hey @v-royavinash
The "Set up authentication steps" were not done yet. I expected that to be completed during the deployment.
I configured everything as described in the deployment guide under "Step 3 - Setup authentication steps" and now everything is working like a charm!
Thanks for your help! much appreciated!


Cow eats everything

Description

Even mice and Hamlet!
Reproduction

Screenshots

Additional context

hes very hungry

big girl need her protein (hampter) and rubber (mice)


Keras for shower reconstruction

Hello, I'm from a team working on a shower reconstruction technique based on neural networks using Keras. We think it'd be nice to be able to use Keras models to create a shower reconstructor.
Currently we have a model that uses as input the calibrated data of an event from prod3b  using a custom Sequence (here) that allows the neural network to have a custom input and output. Maybe using a Sequence inside a new Reconstructor sub-class could help to have a common input and output in the predict method of that class for different reconstructors based on Keras.

Related: #1091
I would be reluctant to add one of the deep learning libraries as a dependency.
But as far as I know, all of them support exporting to onnx (at least tf and torch).
So what would you think about using onnx as serialization format?

Sorry for the late reply. I've been testing the conversion of our models used for shower reconstruction to the onnx format using keras2onnx and it seems to be working fine, but I had to make some adjustments to the code (nothing that changes the model architecture though).
ONNX as the serialization format seems like a good idea, as it could allow models from different frameworks, not only Keras, to be loaded and, compared to Tensorflow for example, doesn't seem too heavy to have as a dependency.
A while ago, we started to develop, as a contribution to ctapipe, a "Keras Reconstructor" for the usage of one or more models in a shower reconstruction. We'll be using this reconstructor as a base to an "ONNX Reconstructor" (or maybe "Deep Learning Reconstructor"?) and create a pull request when it's ready (in a couple of days).


ISSUE #111 Updated BacktoTop.tsx

🎯 Title and Issue Number

Closes #111

🗒 Checklist:

[y] I have mentioned the issue number in my Pull Request.
[y]  I have gone through rules of when PR will get merged.
y[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md
[] I have followed proper file structure.
[] I have gone through

Thank you for contributing!❤
Don't forget to show some support by ⭐ the repository!

Hello @Sumanth077s, thank you for your pull request! We will review it shortly. 😇❣
Your contributions help us grow and improve our project. Please make sure you have read and followed our CONTRIBUTING GUIDELINES.
Your efforts are appreciated, and we value your participation in our community. Don't forget to ⭐ our repository!
Happy coding!✨

Bro labels?


Delegate to set circleColor & textColor for selected date

Expected to have a delegate method to dynamically set custom color for the selected state as well like:
- (UIColor *)simpleCalendarViewController:(PDTSimpleCalendarViewController *)controller circleColorForSelectedDate:(NSDate *)selectedDate
and
- (UIColor *)simpleCalendarViewController:(PDTSimpleCalendarViewController *)controller textColorForSelectedDate:(NSDate *)selectedDate
Setting different colors here will only apply to the dates which is currently selected
This:
[[PDTSimpleCalendarViewCell appearance] setCircleSelectedColor:[UIColor orangeColor]];
[[PDTSimpleCalendarViewCell appearance] setTextSelectedColor:[UIColor purpleColor]];
alone won't help if you need different colors depending on the date.

I'd like to see this feature too, or at least an option to disable the use of custom colors on Today's date and Selected date.
I'm using this library to implement a color coded calendar that does not need selection or to emphasize today's date and there does not seem to be a way to remove those styles.
Thanks


[Fabric 1.19.2] Players get kicked when rerolling a mission

Describe the Bug
Whenever someone rerolls a mission, they get kicked from the server due to
Internal Exception: java.lang.NullPointerException: Cannot invoke "com.kryeit.missions.MissionType.id()" because the return value of "com.kryeit.missions.config.ConfigReader$Mission.missionType()" is null.
There is no further stack trace.
Create is version 0.5.1c, it wasn't selectable in the version dropdown
Reproduction Steps
Not sure if reproducible in a clean environment, haven't tested on a server without my modpack.

Start a server
Try to reroll a mission
You get kicked

Expected Result
The mission gets rerolled
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Probably linux, it's not self-hosted
Mod Version
0.2.6
Create Mod Version
0.5.1d
Minecraft Version
1.19.2
ModLoader and Version
Fabric 0.15.7
Other Mods
There's over 100, I can paste the list upon request if it's needed but I won't do it right now to not make this too long.
Additional Context
No response

no need to paste the modlist, it's definetly our issue. We will seek to fix this soon. Thanks!

Can you provide a client's log?

@MrRedRhino it's from the server log, but there is literally nothing else there besides the usual messages when a player disconnects. There is, however, a new issue, possibly related:
Sometimes when players attempt joining, they get immediately disconnected and the server log reads:
[11:39:37] [Server thread/INFO]: Syncing config to KjipGamer (310d13d0-b52e-4ad5-9aa4-e362d753e391) [11:39:37] [Server thread/ERROR]: Couldn't place player in world java.lang.NullPointerException: Cannot invoke "com.kryeit.missions.MissionType.id()" because the return value of "com.kryeit.missions.config.ConfigReader$Mission.missionType()" is null 	at com.kryeit.missions.DataStorage.createActiveMissionTag(DataStorage.java:117) ~[missions-fabric-0.2.6.jar:?] 	at com.kryeit.missions.DataStorage.reassignActiveMissions(DataStorage.java:106) ~[missions-fabric-0.2.6.jar:?] 	at com.kryeit.missions.MissionManager.reassignMissions(MissionManager.java:111) ~[missions-fabric-0.2.6.jar:?] 	at com.kryeit.missions.MissionManager.reassignMissionsIfNecessary(MissionManager.java:100) ~[missions-fabric-0.2.6.jar:?] 	at com.kryeit.Main.handlePlayerLogin(Main.java:85) ~[missions-fabric-0.2.6.jar:?] 	at com.kryeit.MainFabric.lambda$onInitialize$0(MainFabric.java:14) ~[missions-fabric-0.2.6.jar:?] 	at net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents.lambda$static$2(ServerPlayConnectionEvents.java:48) ~[fabric-networking-api-v1-1.2.12+def3f86d90-79c6c1b478adf57f.jar:?] 	at net.fabricmc.fabric.impl.networking.server.ServerPlayNetworkAddon.onClientReady(ServerPlayNetworkAddon.java:65) ~[fabric-networking-api-v1-1.2.12+def3f86d90-79c6c1b478adf57f.jar:?] 	at net.minecraft.class_3324.handler$bme000$fabric-networking-api-v1$handlePlayerConnection(class_3324.java:6013) ~[server-intermediary.jar:?] 	at net.minecraft.class_3324.method_14570(class_3324.java:200) ~[server-intermediary.jar:?] 	at net.minecraft.class_3248.method_33800(class_3248.java:151) ~[server-intermediary.jar:?] 	at net.minecraft.class_3248.method_14384(class_3248.java:138) ~[server-intermediary.jar:?] 	at net.minecraft.class_3248.redirect$bmf000$fabric-networking-api-v1$handlePlayerJoin(class_3248.java:563) ~[server-intermediary.jar:?] 	at net.minecraft.class_3248.method_18784(class_3248.java:73) ~[server-intermediary.jar:?] 	at net.minecraft.class_2535.method_10754(class_2535.java:248) ~[server-intermediary.jar:?] 	at net.minecraft.class_3242.method_14357(class_3242.java:183) ~[server-intermediary.jar:?] 	at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:886) ~[server-intermediary.jar:?] 	at net.minecraft.class_3176.method_3813(class_3176.java:289) ~[server-intermediary.jar:?] 	at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:819) ~[server-intermediary.jar:?] 	at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:665) ~[server-intermediary.jar:?] 	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:257) ~[server-intermediary.jar:?] 	at java.lang.Thread.run(Thread.java:840) [?:?] [11:39:37] [Server thread/INFO]: KjipGamer lost connection: Invalid player data
It seems like whenever this happens, after trying to rejoin a couple of times, it fixes itself, however usually not on the very next rejoin.

There is. The first 6 stacktrace elements contain our code. Can you send your server's missions.json config file?

this issue might be caused by you updating Create missions. Fan missions were deleted and you still have those in missions.json. Let me know if this is the issue

do you mind testing https://modrinth.com/mod/missions/version/o3w0znaS this version? its the latest 1.19.2 fabric version

We reworked how missions are randomly gotten, if this issue ever comes up again let us know!


Hook False

Hi @Retimeup ,
Thanks for testing IoctlHunter!
The tool works by using dynamic keyboard events to enable/disable features while conducting an analysis.
You can activate or deactivate the hooking mode by pressing [SPACE]. Please note that all key events can be listed by pressing [H].
Alternatively, the hooking mode can be enabled at the start by running IoctlHunter with the -hos (--hook-on-start) parameter.


Attach textcomplete to the active CKEditor only

Prevent memory leaks in single page application environment, when multiple CKEditor instances are initialized an destroyed one after another (As if posting several comments using CKEditor in a popup screen).

I think textcomplete will be attached to the initially initialized editor instance, but not attached to instances initialized later. Isn't it? (I'm not familiar with the API of CKEditor)

In our app we may initialize multiple instances of CKEditor in one page. Textcomplete leaks when a CKE instance is destroyed and after that another one is initialized because of the way textcomplete connects to CKE - with CKEditor.on('instanceready'). Textcomplete  attaches its old dropdown menu together with the newly created one to the new instance. And tries to replace twice. If the CKE is destroyed and another one is initialized, then we have three dropdowns an so on.
If you connect to CKE with CKEditor.once(...) the old dropdowns will never try to  reconnect again and so is the memory leak partly (the dropdowns are not garbage collected) prevented.

Is there any chance that this pull request might still be merged and made into a new version, even though the project is no longer being maintained?
I've been unable to figure out a workaround for this, since there seems to be no way to remove this specific listener from CKeditor.
Note: since the pull request doesn't affect https://github.com/yuku-t/jquery-textcomplete/blob/master/dist/jquery.textcomplete.js#L163, referencing the pull request through npm can't be used as a work-around.
I'd happily switch to the newer https://github.com/yuku-t/textcomplete but sadly it doesn't have CKeditor support yet (see https://github.com/yuku-t/textcomplete/issues/35).

@ThomasVandenbon OK, I'll merge this patch and release the next version soon

@ThomasVandenbon<EMAIL_ADDRESS>is out 😄

Thank you so much for your speedy intervention, this works brilliantly!

My colleague @beuseli has implemented a better solution, that works for both situations where only a single CKEditor is visible at a time and for situations where multiple are visible on a single page.
See #309
It would be great if you could merge this pull request and release a new version.
While this pull request (#294) fixed some issues, it also introduced a few new ones. Which is why we decided to make a pull request with a fiddle demonstrating the problems.


P0356R5 Simplified partial function application

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2018/p0356r5.html

実装済み


Hide deprecated or confusing options from command help

Follow up to #1181

@CollierCZ after some thought I've updated this to:

hide --quiet/-q - it's confusing to me what "do not show any messages" should do, I'd expect it to hide only stderr messages and leave normal output, but it hides everything, and it's more UNIXy to redirect to /dev/null, and it was yet another option that was disabling interaction
add --no-interaction (for consistency with other Symfony apps as suggested in #1054)
keep --yes/-y there, and shown, as it is a useful shortcut for --no-interaction
hide --no but keep it working as described
show a warning if the -n shortcut for --no is used, as it doesn't have a purpose in scripting and it can be confused with --no-interaction (it's the default shortcut for --no-interaction in other Symfony apps)
add --ansi and --no-ansi to be more like other Symfony apps (they do work as described), but keep them hidden
hide deprecated options from various commands (they are only there for backwards compatibility)


"Loturi Enescu" is listed in the drop-down menu

Description:
"Loturi Enescu" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Loturi Enescu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Loturi Enescu, Botoșani (Dorohoi)".

ATTENTION:
Request - ""Loturi Enescu, Botoșani (Dorohoi)"
In dropbox, the request is displayed correctly. - ""Loturi Enescu, Botoșani (Dorohoi)"
After you click on the text, what is displayed in the window this is different - "Loturi Enescu, Botoșani (DOROHOI)"


Add dbus model for fru's on splitter

Deletion of the Front panel board from the map is missing inside the deleteObject function

its there in libpldmresponder/platform.cpp right

https://github.com/ibm-openbmc/pldm/blob/1030/host-bmc/dbus/custom_dbus.cpp#L479 here we need to remove the entry from panel map


Lost funds

Hi, I was transferring funds #fantom from Trust Wallet to  Binance like I did the previous 2 times, but this time around, the transaction was stuck on pending and nothing was showing up on block chain explorer either so I reimported my wallet like the guide on the Trust Wallet forum. After reimporting, my pending transaction disappeared and the amount of funds I was transferring have vanished as well. They’ve been deducted from my Trust Wallet without the transaction actually going through. What should I do now to get lost crypto back

You can see details of your transfers on public explorers: https://explorer.solana.com/

Sounds like u don't even read my massage

Sounds like u don't even read my massage

Can you provide details of your address, transaction transfer or any information that can help debug the issue?

Hello ,
This my receiving fantom address
0xdaf3Fc19235D9F49EC79b7D0A2A60CBE6915C105
And if check fantom newwork
You will not found  6.840 FTM I sent. Because it’s get failed from trust wallet. And not getting back to my funds.
I still have screenshot from trust wallet
Sent from my iPhone

On 17 Sep 2021, at 22:54, Viktor Radchenko @.***> wrote:

Sounds like u don't even read my massage
Can you provide details of your address, transaction transfer or any information that can help debug the issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

https://ftmscan.com/address/0xdaf3Fc19235D9F49EC79b7D0A2A60CBE6915C105 - I do not see any transfer with 6.8 FTM, but there is many other transactions
If someone sent you 6.84 FTM and you never received, then you need to contact the sender.

It’s not some one send me
I send from my trust wallet account to Binance
6.840 fontm
Pls provide me your telegram account to send you screenshot of trust wallet when transaction get failed
Sent from my iPhone

On 18 Sep 2021, at 03:49, Viktor Radchenko @.***> wrote:

https://ftmscan.com/address/0xdaf3Fc19235D9F49EC79b7D0A2A60CBE6915C105 - I do not see any transfer with 6.8 FTM, but there is many other transactions
If someone sent you 6.84 FTM and you never received, then you need to contact the sender.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

So if you check here
https://ftmscan.com/address/0xdaf3Fc19235D9F49EC79b7D0A2A60CBE6915C105
You will not found the 6840 fantom I sent, so they still blocked in trust wallet server
,
Kindly let me know if u see the screenshot I sent
Sent from my iPhone

On 18 Sep 2021, at 03:57, Mo salah Salah @.***> wrote:
It’s not some one send me
I send from my trust wallet account to Binance
6.840 fontm
Pls provide me your telegram account to send you screenshot of trust wallet when transaction get failed
Sent from my iPhone

On 18 Sep 2021, at 03:49, Viktor Radchenko @.***> wrote:

https://ftmscan.com/address/0xdaf3Fc19235D9F49EC79b7D0A2A60CBE6915C105 - I do not see any transfer with 6.8 FTM, but there is many other transactions
If someone sent you 6.84 FTM and you never received, then you need to contact the sender.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

You need to contact the sender, there is nothing to do with Trust Wallet.

The sender said u should talk to trustwallet
Transaction get failed from the app. And the app doesn't give any information
Sent from my iPhone

On 18 Sep 2021, at 09:21, Viktor Radchenko @.***> wrote:

You need to contact the sender, there is nothing to do with Trust Wallet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The sender said u should talk to trustwallet Transaction get failed from the app. And the app doesn't give any information
…
Sent from my iPhone
On 18 Sep 2021, at 09:21, Viktor Radchenko @.***> wrote:  You need to contact the sender, there is nothing to do with Trust Wallet. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

As suggested before, it's not the app. Transaction never been completed on the blockchain level, here is the proof: https://ftmscan.com/address/0xdaf3Fc19235D9F49EC79b7D0A2A60CBE6915C105.


Integrating opacus with huggingface

Hello,
I was trying to see if the Opacus library can be used with the Trainer module of huggingface. I see a code snippet in the readme that says to do the callback dp_transformers.PrivacyEngineCallback in the Trainer module. However, it leads to an error that says the callback does not exist. I was unable to figure it out in the source code either.
It would be really helpful if I could be pointed to the part of the source code that has this callback or maybe guide me with any solution for the error.
Thanks!

Hi @SoumiDas, indeed this repo is all about integrating Opacus library with the Trainer module of HF. Have you seen these examples by any chance? https://github.com/microsoft/dp-transformers/tree/main/research/fine_tune_llm_w_qlora -- here we provide examples of DP training with the Trainer module of HF (https://github.com/microsoft/dp-transformers/blob/main/research/fine_tune_llm_w_qlora/fine-tune-dp.py). Let us know please if you have further questions.
You can see here https://github.com/microsoft/dp-transformers/blob/main/src/dp_transformers/dp_utils.py where we create OpacusDPTrainer from the HF Trainer and where the DPCallback is added.


Why is the current codebase not in sync with prettier?

As soon as I build the project, prettier changes a lot of the code formatting. Is there a reason why the current master isn't in sync with prettier?

I upgraded prettier in master. It should be fine now.


Support for Stan, Soss, and Gen

Currently, only Turing is supported; support for Stan, Soss, and Gen would be great.

In StatisticalRethinking v4, with StanSample loaded, for LOO_CV I will use:
function psis_loo(model::SampleModel; loglikelihood_name="log_lik")
    chains = read_samples(model) # Obtain KeyedArray chains
    psis_loo(chains; loglikelihood_name)
end

function psis_loo(chains::T; loglikelihood_name="log_lik") where {T <: KeyedArray}
    ll = Array(matrix(chains, loglikelihood_name)) # Extract log_lik matrix
    ll_p = to_paretosmooth(ll) # Permute dims for ParetoSmooth
    psis = psis_loo(ll_p) # Compute PsisLoo for model
end

which typically  is called as:
chns5_1s = read_samples(m5_1s)
psis_loo(chns5_1s) |> display

Hi Carlos (@ParadaCarleton),
Apologies for the delay in answering.
StanSample is a relative lightweight package on a lower level (it has no notion of comparing models). I would prefer to keep it that way.
ParetoSmooth based model comparison really fits into (the very much simplified) StatisticalRethinking.jl v4.0. The simplifications come from dropping usage of @reexports ... and all graphics
The code is included here and tested here in SR v4.0.3.
As I mentioned before, for the SR course clips in StatisticalRethinkingStan I will follow the book and present the results in a different format and re-introduce some columns and fields into the SR's version ModelComparison object (likely more like the old LooCompare object), also to improve presentation in Pluto notebooks (e.g. the Pareto_k value warnings currently show up in the console).
All graphics are now handled in StatisticalRethinkingPlots which does contain an option to plot the Pareto_k values. And in the future similarly in StatisticalRethinkingMakie.


iOS app build hangs with "Connection aborted"

Hello @antoinevg
Can you please check why since yesterday all the iOS app build stop with the message

[  ERROR] Something went wrong that we didn’t expect:
[  ERROR] (‘Connection aborted.’, error(54, ‘Connection reset by peer’))

Thanks

My apologies, the build machine crashed. It should be up and running again now.


feat: Add support for HONEYCOMB_API_KEY environment variable for provider

feat: Add support for HONEYCOMB_API_KEY environment variable for provider
@jharley Converted current tests to use new environment variable and updated documentation saying that both are supported, but new env var takes priority. If there's anything I should change or test please let me know.
Also I count not get the tests working locally with my API key, does it required an enterprise account for SLO tests?
Addresses #117

Ahh Sorry I missed that, I can update.

@jharley Updated to use correct env var name now, please let me know if there's anything else I should test/add/change. Also, thanks for the quick initial review!

@jharley Committed your suggested change, let me know if there's anything else I should change.


Ubuntu编译错误

Ubuntu16.04.4编译错误
g++: error: python/demo/familia.so: No such file or directory
Makefile:103: recipe for target 'python/demo/familia.so' failed
make: *** [python/demo/familia.so] Error 1

更详细的编译日志
Target build/libfamilia.a
ar crv build/libfamilia.a build/vose_alias.o build/inference_engine.o build/model.o build/vocab.o build/document.o build/sampler.o build/config.o build/util.o build/semantic_matching.o build/tokenizer.o build/demo/inference_demo.o build/demo/doc_distance_demo.o build/demo/query_doc_sim_demo.o build/demo/word_distance_demo.o build/demo/topic_word_demo.o build/demo/show_topic_demo.o
a - build/vose_alias.o
a - build/inference_engine.o
a - build/model.o
a - build/vocab.o
a - build/document.o
a - build/sampler.o
a - build/config.o
a - build/util.o
a - build/semantic_matching.o
a - build/tokenizer.o
a - build/demo/inference_demo.o
a - build/demo/doc_distance_demo.o
a - build/demo/query_doc_sim_demo.o
a - build/demo/word_distance_demo.o
a - build/demo/topic_word_demo.o
a - build/demo/show_topic_demo.o
g++ -I./include/ -I./include/familia -I./third_party/include -I/usr/include/python3.5 -pipe -W -Wall -fPIC -std=c++11 -fno-omit-frame-pointer -fpermissive -O3 -ffast-math -c python/cpp/familia_wrapper.cpp -o python/cpp/familia_wrapper.o
python/cpp/familia_wrapper.cpp:429:1: warning: missing initializer for member ‘PyModuleDef::m_slots’ [-Wmissing-field-initializers]
};
^
python/cpp/familia_wrapper.cpp:429:1: warning: missing initializer for member ‘PyModuleDef::m_traverse’ [-Wmissing-field-initializers]
python/cpp/familia_wrapper.cpp:429:1: warning: missing initializer for member ‘PyModuleDef::m_clear’ [-Wmissing-field-initializers]
python/cpp/familia_wrapper.cpp:429:1: warning: missing initializer for member ‘PyModuleDef::m_free’ [-Wmissing-field-initializers]
g++ -I./include/ -I./include/familia -I./third_party/include -I/usr/include/python3.5 -pipe -W -Wall -fPIC -std=c++11 -fno-omit-frame-pointer -fpermissive -O3 -ffast-math -shared python/cpp/familia_wrapper.o -L/home/karvn/Project/Familia-1.1.2/third_party/lib -L/usr/lib -L./build/ -lfamilia -lprotobuf -lglog -lgflags -l -o python/demo/familia.so
g++: error: python/demo/familia.so: No such file or directory
Makefile:103: recipe for target 'python/demo/familia.so' failed
make: *** [python/demo/familia.so] Error 1

解决了，python库没链接
将
$(CXX) $(INCPATH) $(CXXFLAGS) -shared python/cpp/familia_wrapper.o $(LDFLAGS_SO) -l$(PYTHON_VERSION) -o $@
修改为
$(CXX) $(INCPATH) $(CXXFLAGS) -shared python/cpp/familia_wrapper.o $(LDFLAGS_SO) -lpython3.5m -o $@


Calling io_uring_setup from multiple threads

Hi, I've probably found another problem triggered when my tests run in parallel.
Should it be ok to setup unique io_uring independently from multiple threads?
I've created simple test to reproduce it (at least on my Ryzen 7 3700X with Fedora 31 kernel 5.3.12).
Basically it fails during io_uring_setup call with ENOMEM.
I've tried to follow this guide to figure something out, but I'm not into kernel dev so this all seems too much woodoo for me ;-)
Anyway, here's the trace output if it helps somehow:
  1)               |  __x64_sys_io_uring_setup() {
  1)               |    io_uring_setup() {
  1)               |      capable() {
  1)               |        ns_capable_common() {
  1)  <...>-633600  =>  <...>-633604
  1)               |          security_capable() {
  1)  <...>-633604  =>  <...>-633600
  1)   0.230 us    |            cap_capable();
  1)   0.762 us    |          }
  1)   1.202 us    |        }
  1)   1.623 us    |      }
  1)   0.270 us    |      free_uid();
  1)   2.735 us    |    }
  1)   3.467 us    |  }

And here is a test to reproduce it.
#include <stdio.h>
#include <pthread.h>
#include "liburing.h"

struct thread_info_t {
	pthread_t tid;
	int num;
};

static void *doTest(void *arg) {
	struct io_uring ring;
	struct io_uring_cqe *cqe;
	struct io_uring_sqe *sqe;
	struct thread_info_t *ti;
	int ret;

	ti = (struct thread_info_t *)arg;
	printf("%d: start\n", ti->num);

	ret = io_uring_queue_init(128, &ring, 0);
	if (ret) {
		printf("%d: ring setup failed: %d\n", ti->num, ret);
		return arg;
	}

	sqe = io_uring_get_sqe(&ring);
	if (!sqe) {
		printf("%d: get sqe failed\n", ti->num);
		return arg;
	}

	io_uring_prep_nop(sqe);

	ret = io_uring_submit(&ring);
	if (ret <= 0) {
		printf("%d: sqe submit failed: %d\n", ti->num, ret);
		return arg;
	}

	ret = io_uring_wait_cqe(&ring, &cqe);
	if (ret < 0) {
		printf("%d: wait completion %d\n", ti->num, ret);
		return arg;
	}

	io_uring_cqe_seen(&ring, cqe);
	printf("%d: done\n", ti->num);
	return NULL;
}

int main(int argc, char *argv[])
{
	struct thread_info_t threads[10];
	int ret;
	void *res;

	for (int i=0; i<10; i++) {
		threads[i].num = i;
		ret = pthread_create(&threads[i].tid, NULL, doTest, &threads[i]);
		if (ret) {
			fprintf(stderr, "Thread create failed\n");
			return 1;
		}
	}

	for (int i=0; i<10; i++) {
		ret = pthread_join(threads[i].tid, &res);
		if (ret) {
			fprintf(stderr, "Thread join failed\n");
			return 1;
		}
		if (res) {
			fprintf(stderr, "Test failed\n");
			return 1;
		}
	}

	return 0;
}

One of my outputs is:
0: start
1: start
2: start
3: start
4: start
4: ring setup failed: -12
5: start
5: ring setup failed: -12
6: start
0: done
6: ring setup failed: -12
3: done
7: start
7: ring setup failed: -12
1: done
8: start
8: ring setup failed: -12
9: start
9: ring setup failed: -12
2: done
Test failed

It's not a thread problem, I'm guessing you are running as non-root and you run into the rlimit memlock limit (ulimit -l).

Ah, thanks, I've totally overlooked that.
Test runs fine when run as root.


Merge ci from main

Issue: #2548
This pull request introduces significant improvements to the CI/CD workflow documentation and configuration. The key changes include a comprehensive guide to the CI/CD workflow, updates to the build and engine matrices, and enhancements to the workflow templates. These changes aim to streamline the testing process and ensure consistent execution across different environments and languages.
Documentation Enhancements:

Added a detailed CI/CD Workflow Guide to .github/DEVELOPER.md, covering workflow triggers, test coverage, language-specific workflows, shared components, and dynamic matrix generation.

Configuration Updates:

Updated build-matrix.json to include new fields for specifying languages and added support for Amazon Linux. [1] [2]
Modified engine-matrix.json to include new engine versions and types, ensuring specific versions are always tested.
Added supported-languages-versions.json to define supported language versions and always-run versions for various languages.

Workflow Template Improvements:

Enhanced the create-test-matrices/action.yml to dynamically generate matrices based on inputs, optimizing the CI/CD process.
Updated csharp.yml to utilize the new dynamic matrices and added support for workflow dispatch inputs. [1] [2]

Minor Adjustments:

Simplified the build-node-wrapper/action.yml and build-python-wrapper/action.yml by adjusting the npm install process and making the engine version input optional. [1] [2]
Improved the pull request template checklist formatting.

Checklist
Before submitting the PR make sure the following are checked:

[ ] This Pull Request is related to one issue.
[x] Commit message has a detailed description of what changed and why.
[ ] Tests are added or updated.
[ ] CHANGELOG.md and documentation files are updated.
[x] Destination branch is correct - main or release
[x] Commits will be squashed upon merging.

As it was discussed with the team, this PR is too big and contains changes from multiple PRs. It is definetely recommended to split it into multiple PRs and iteratively increase the test coverage.
Split PR
Split PR into multiple PRs. Some of them could be published and reviewed in parallel. Steps to do there.

Post and merge first PR, which contains CI changes only: .github/{json_matrices,workflows,DEVELOPER.md}. This will all CI backbone, but tests executed on every PR will remain the same. Scheduled job won't run on this branch. We can trigger it manually, but there is no reason until test fixed.
Merge test fixes for redis 6 (task #2524): #2523, #2535, #2536.
Manually run jobs on all redis/valkey versions.
Post a PR with the rest test fixes if needed.
(optional, but recommended) Add CI fixes to use cache for rust lib build and valkey/redis build (tasks #771/#1962 and #2544). This will speed up the full CI runs for at least an hour.
Reorganize test strategy and matrices. I'll describe this below.

Once step 4 completed we will have a full matrix CI green. Keep in mind that it still takes a lot of time and may suffer from unexpected CI or flakey test.
Known issues as of now (at least ones I'm aware about):

The repo has only 1 linux arm runner. GH-hosted linux arm runners are more expensive than others, so it the repo (or the org maybe) doesn't use them. So repo has only 1 runner of such type and it has very high occupancy.
Each job rebuilds the client from the scratch, and valkey/redis too, it takes ~1.5 min for each action. There are tasks to introduce/fix the cache use to speed up this.

Reorganize testing.
Running all tests on all platform on all language/framework versions on all server versions on ... etc ... isn't a good approach. Adding a small 5 sec test increases overall CI duration for few minutes. Adding a new server version will add few hours there.
Let me list few facts there and then I'll add proposals.

There is completely no reason to repeat all tests no all supported java versions for java client - we don't use java 11+ features (our min supported version) nor we don't have code like "if java 17 do this else do that". Same applicable for all other clients.
There is no reason to repeat all tests on all platform except platform specific ones.
There is no reason to repeat all tests on all server versions. All client mechanisms are version insensitive.
In 95% we're trying to test the server instead of testing the client.

Proposal:

Run all tests on one most common platform on most recent server version only. Linux x64 - Valkey 8.
Run sanity/smoke tests on all platforms on different (or random) server versions. Mininal benchmark  is a good enough as a sanity test there. We should also add there pubsub, script, cluster scan tests.
Run benchmark on different language/framework versions to cover that.

Following that proposal, full matrix tests won't run longer than an hour. We will be able to run full testing job on all release branches every night for all clients.
It is not too late to do this.

@Yury-Fridlyand
I get your intention, but we need to do fixes on release, and we need to unblock the team to make them able to go back to work.
Plus — team got a list of errors and pushed them into release, and we want to run the action on release. We don't have the luxury of rearranging the branch into separate PRs and taking it through a month of reviewing and fixing.
The first step, as agreed, is merging, with red CI, and from there start running fixes.
Next point, probably part of 1.3 — creating a plan to make CI work as needed.
Right now — make 1.2 possible by making the release branch green. To do so, we need to merge this into the release branch and run the actions on the release branch in loops.


[BEAM-8574] Revert "[BEAM-8427] Create a table and a table provider for MongoDB"

Reverts apache/beam#9806
Post Commit tests break.

Run JavaPortabilityApi PreCommit

R: @apilloud

Run sql postcommit


Some input events aren't sent

I'm trying to build a glium app right now, and am using glutin through that.
I'm trying to handle arrow key input, but am finding that if I have any two arrow keys held down, I don't get a new press event when I press a third arrow.
Similarly, if I'm typing keys from the main keyboard, I can type ASDFGH, but then any additional events don't show up.
This is on macOS 10.12.6, glutin 0.9.2 (the version in glium) and glutin 0.10.0.

On the other issue, @mitchmindtree correctly guessed that this is probably caused by the 6-key rollover on my Macbook's keyboard. I'm not sure why the arrow keys seem to be limited at 2, but that also appears to be a hardware limitation.


no_recursion with HasMap not working

Hi.
Next code fails with stack overflow
#[derive(Serialize, Deserialize, ToSchema)]
#[schema(no_recursion)]
pub struct Tree {
    left: Box<Tree>,
    right: Box<Tree>,
}

#[derive(Serialize, Deserialize, ToSchema)]
#[schema(no_recursion)]
pub enum TreeRecursion {
    Named { left: Box<TreeRecursion> },
    Unnamed(Box<TreeRecursion>),
    NoValue,
}

#[derive(Serialize, Deserialize, ToSchema)]
pub enum Recursion {
    #[schema(no_recursion)]
    Named {
        left: Box<Recursion>,
        right: Box<Recursion>,
    },
    #[schema(no_recursion)]
    Unnamed(Box<Recursion>),
    Map(HashMap<String, Recursion>),
    NoValue,
}

Without the Map(HashMap<String, Recursion>) line work.
And if you add it, the macro gives the following error
    #[schema(no_recursion)]
    Map(HashMap<String, Recursion>),

NoRecursion does not support `ToTokens`

Thanks

Oh, that is a bug indeed,
   #[schema(no_recursion)]
    Map(HashMap<String, Recursion>),

The above should be possible.


Defect in MORPHING function

The morph function does not work correctly. I tested my svg in many ways, but none works. With morph da grensock it works perfectly, but paying 99 dolars a year is a bit complicated for those who live in Brazil.
greensock Morph:
Codepen
I tried this way with the animejs:
Codepen

Anime's svg morphing is really basic and doesn't extrapolate missing points and things like that.
Both shapes needs to have the same number of points.
And I have no plan of improving this part in the next releases.

@juliangarnier Thanks for the reply Julian. I really enjoyed animejs, especially the examples part, it's really good to understand how it works, besides being very compact. A pity this is not a priority, but I understand that this part of morph should be pretty laborious. By chance do you know of any other library that is of the morphSVG level of greensock? I've looked at kutejs, but also only accept simple shapes.
I needed something free because I do not currently have a fixed job. I worked 4 years in a company that made only static websites, no animations and stuff like that. Now I am trying to perfect myself in this part and wanted to choose a library that has a good community and that does not risk being abandoned.


Wiki page

Hi, Great project. I see there isn't much on the Wiki page yet. Would be a good place to put in more details on what problems you encountered and how you solved them when you get around to that stage.
-Aurelie -Team2

The wiki is now updated. More updates to come


Fixed attributes processing in permission directive.

In case where you use
.authorize(new PermissionMap({
    only: permission.only,
    except: permission.except
})

permission.only and permission.except is undefined. This fix it and correctly retrieve values from only and except attributes.

@masterspambot I checked development branch. Looks like this problem was fixed. I try to use development version in my project and see if this bug reproduces.


Implement edit and remove for Clusters and Storage

I have 'remove' working.
Looking into 'edit' next.
@ibolton336 @eriknelson welcome any suggestions to improve

Part of the functionality for updating Cluster url and Secret value is working towards k8s.
Working on having the updates reflected in the redux Store, that is not yet working.
Also saw I broke add storage, so will need to fix.

At this point, edit and remove of Clusters is working.
Starting to look into 'Storage', first issue I see is that the storage secret was being sent to 'openshift-config' yet we declared it for 'mig' namespace.  I assume that the storage secret should be in 'mig' namespace so updated it to be there.
Have not addressed remove/edit of Storage yet, next on list to handle.

Have the below working now

Edit/Remove of Clusters
Edit/Remove of Storage

@ibolton336 @eriknelson @andyvesel
To get this ready to merge, I know of 2 areas to address, please let me know if there are others to improve:

Add a 'Confirm' component, perhaps something like https://react.semantic-ui.com/addons/confirm/#types-confirm, with PF4 components
Ensure after performing the update/remove actions for both Cluster and Storage we are sending 'success' events.

Any other comments to improve?  I am new to React/Redux, appreciate any help to learn and get better.

@jwmatthews in addition to ^ there's also an issue: after update/delete cluster, I always need to refresh the page manually to see that cluster was actually updated/deleted.

I think last thing left is for me to update the PR with some success messages when edit/remove has succeeded or messages when they've failed.

I think last thing left is for me to update the PR with some success messages when edit/remove has succeeded or messages when they've failed.

Yep that should be it.

As I'm testing I'm seeing "redux-logger.js?6f8a:1 Uncaught (in promise) TypeError: Cannot read property 'type' of undefined" after editing a cluster.
Need to debug further.

I think I got the redux pieces correct now.
I wasn't adjusting the actions/reducers so there was no change to local state.   I think last commit addresses this.

Rebased from master, now seeing below
ERROR in /git/fusor/mig-ui/src/app/common/components/ConfirmModal.tsx
./src/app/common/components/ConfirmModal.tsx
[tsl] ERROR in /git/fusor/mig-ui/src/app/common/components/ConfirmModal.tsx(20,7)
TS2322: Type '(isConfirmed: any) => void' is not assignable to type '() => void'.

ERROR in /git/fusor/mig-ui/src/app/common/components/ConfirmModal.tsx
./src/app/common/components/ConfirmModal.tsx
[tsl] ERROR in /git/fusor/mig-ui/src/app/common/components/ConfirmModal.tsx(20,7)
TS2322: Type '(isConfirmed: any) => void' is not assignable to type '() => void'.

Looks like PF may have made a breaking change to their API so the onClose prop expects a () => void instead of (isConfirmed: any) => void?

Getting close to OK to merge.
Last issue is that AddStorage is not handling the aws secret access key consistently.
Need to check what the key should be:  'aws-secret-access-key' or 'aws-secret-access-key-id'.
Will update PR to fix that, then think it will be ready.

Fixed all issues I know about.
Ready for review and merge if OK.

@jwmatthews commented on this pull request.
In src/client/resources/conversions.ts
https://github.com/fusor/mig-ui/pull/211#discussion_r291757193:

@@ -116,7 +159,7 @@ export function createStorageSecret(
apiVersion: 'v1',
data: {
'aws-access-key-id': encodedAccessKey,

 'aws-secret-access-key': encodedSecretKey,

 'aws-secret-access-key-id': encodedSecretKey,

Need to look more to determine which value this should be.
'aws-secret-access-key' or 'aws-secret-access-key-id'.

This should be 'aws-secret-access-keywithout the-id` suffix. Was a bug
I had originally introduced, realized it was a bad key during integration
testing last week and updated.

@.**** commented on this pull request. ------------------------------ In src/client/resources/conversions.ts <#211 (comment)>: > @@ -116,7 +159,7 @@ export function createStorageSecret( apiVersion: 'v1', data: { 'aws-access-key-id': encodedAccessKey, - 'aws-secret-access-key': encodedSecretKey, + 'aws-secret-access-key-id': encodedSecretKey, Need to look more to determine which value this should be. 'aws-secret-access-key' or 'aws-secret-access-key-id'.
This should be 'aws-secret-access-keywithout the-id` suffix. Was a bug I had originally introduced, realized it was a bad key during integration testing last week and updated.

ACK
That was an older comment, since then I looked at the mig-cluster types and confirmed 'aws-secret-access-key' and updated PR.
Think we are good now and on same page of using 'aws-secret-access-key'.


fix (dm-core): in useDocument, fix type of the document returned from the hook. Use the generic type T instead of any.

after https://github.com/equinor/dm-core-packages/pull/29 is merged, fix usage of useDocument everywehre


common: jittered backoff implementation

Signed-off-by: Rama<EMAIL_ADDRESS>Description:
Implements fully jittered exponential backoff algorithm and refactors router retry timer to use this.
Risk Level: Low
Testing: Added automated tests
Docs Changes: N/A
Release Notes: N/A

@htuch follow-up PR for https://github.com/envoyproxy/envoy/pull/3758 with jittered implementation. PTAL.

@htuch made the JitterBackOff default for management server connections also and removed the earlier ExponentialBackOff. Left couple of questions, PTAL.

@htuch made changes. PTAL.


[MySQL] Fails to start complaining about a non-empty database dir

Is this a request for help?:
No.
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

Version of Helm and Kubernetes:
Helm: 2.7.2
Kubernetes: 1.8.5 w/ RBAC
Which chart:
MySQL v5.7.14
What happened:
The pod failed repeatedly with the following error:
 Initializing database
2018-01-23T06:35:51.140578Z 0 [Warning] TIMESTAMP with implicit DEFAULT value is deprecated. Please use --explicit_defaults_for_timestamp server option (see documentation for more details).
2018-01-23T06:35:51.147754Z 0 [ERROR] --initialize specified but the data directory has files in it. Aborting.
2018-01-23T06:35:51.147838Z 0 [ERROR] Aborting

What you expected to happen:
Database initialization and pod to start with no problems.
How to reproduce it (as minimally and precisely as possible):

Configure kubernetes with a glusterfs storage cluster using heketi for dynamic provisioning.
Register a storage class for the glusterfs filesystem.
Configure your values.yaml to use the storage class.
Install the helm chart using the values.
Note the crashing pod with the above log.

Anything else we need to know:

Hi, @jtgans ,I have encountered the same problem,and the steps of recurring are the same.Do you have any suggestions or solutions?

Hi, @geebytes! You can use initContainer as here. I used arg
...
args:
  - "--ignore-db-dir=lost+found"
...


⚠ Bofrost.at has degraded performance

In 1438146, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2097 ms

Resolved: Bofrost.at performance has improved in 1a6104f.


Pull request for return-request-command

This PR add properties to command node. Now it output requestCommand and requestArguments with the value sent to the server

@Botched1 Does it look like you want? Did you have some corner case in your head about this feature?

Yes, this would work well I think! It would allow for creation of generic subflows to check device node value vs requested command.

For instance, today in my validation subflow I HAVE TO specify msg.command/msg.arguments to the comnmand node (instead of using the node configuration) so that I have a msg.command and msg.arguments to look at in the command node output. This change would allow me to go back using pre-configured nodes (and remove a lot of change nodes).

And after all your tests, do you still consider that output:

msg.requestCommand
msg.requestArguments
is needed to be different than input:
msg.command
msg.arguments

I don't have a strong opinion about it, but your deleted comment made me hesitate
My concern is only to avoid confuse people and I'm a too technical guy to know the answer :laughing:

msg.requestCommand
msg.requestArguments

To not break anyone's existing flows, the output from the command node should not use msg.command/msg.arguments.
I think you had the right idea by putting them in msg.requestCommnd/msg.requestArguments.


Obsolete 'reproduction of symbiont in host'?

Hello,
'GO:0061887 reproduction of symbiont in host' has only been used for one paper PMID:25158041 by CACAO. In this paper, the authors look at "intramacrophage growth phenotype in mutants of the respective genes that were deficient for intramacrophage growth". The mechanisms underlying the ability to reproduce are not characterized further.
I suggest we obsolete this term. 'reproduction of symbiont in host' is a property of the specific host-symbiont interaction, it's not a specific process; the symbiont just reproduces in that specific environment.
@dsiegele @mgiglio99 @ValWood @nsuvarnaiari
Does that seem ok with you ?
Thanks, Pascale

Hi Pascale - I see what you mean and I agree this is a problematic term. The only time I see needing something related to this would be for symbionts that carry out different life stages in different hosts (things like Plasmodium). But I think that we'd want a different term for those cases, not this one.
One question - I'm wondering if it makes sense to merge this into 'biological process involved in interaction with host' (new name)? rather than obsolete. Although if there's only one annotation, maybe it doesn't matter much.

I dont undertand how that term helps to annotate things like Plasmodium. Both the mosquito and the human phase of the life cycle would be annotated to the same term.
If I interpret the image blow correctly, I think those annotations would be more informative:

'GO:0000278 mitotic cell cycle' occurs in some Anopheles mosquito
GO:0051321 meiotic cell cycle occurs in some Human

To your second point - @cmungall suggests we avoid merging when the term meaning is not the same (or not close enough). In this specific case (and related ones) we end up with synonyms that are really confusing, unless we know the history of how the synonyms got there, which I dont think most users are aware of, so I would lean towards obsoletion.
Thanks, Pascale

I also agree that for the GO term, that actual process plus location in host would be preferable.
For those annotating plasmodium in-depth, there is also a life cycle ontology? so
occurs_during stage x as an extension (In the same way that we use phase terms as extensions in GO).
In the above case I would be inclined not to make a GO annotation beyond "interaction with host"    because we know nothing about the process. It is unlikely to be a cell-cycle process (especially as it refers to an effector  molecule), more likely some part of the invasion, migration etc. In fact, the final sentence of the abstract "This suggests that some Francisella pathogenicity island proteins were secreted while others remain within the bacterium during infection of host cells structural components of the secretion system and were necessary for secretion"
so basically these are  really likely to be required in some way for the secretion of other effectors.
I just don't think there is enough information here for anything more. At this stage it's no more than a reasonable speculation.
we would annotate the phenotype at PHI-base as
decreased pathogen growth within host + decreased virulence

I agree that obsoletion is the best thing to do.


Investigate how to test with s3

ref https://github.com/engula/engula/pull/99#discussion_r748133615
need some method to automatic test with AWS service like s3 in CI without leaking access&secret key, IMHO, maybe we can do it in two directions:

unit test: mock AWS SDK, there are exists an issue indicates SDK support TestConnection to mock/record request-response and we take some tries, but it seems we need more encapsulation works to make it easy for use.
integration-test: deploy a temp S3-like service(for example https://min.io/) in CI pipeline and make test case using temp S3-like service.

Welcome to give more advice or contribution code 😄

* unit test: mock AWS SDK, there are exists an [issue](https://github.com/awslabs/aws-sdk-rust/issues/199#issuecomment-904558631) indicates SDK support TestConnection to mock/record request-response and we take some [tries](https://github.com/engula/engula/pull/99/files#diff-f6197926e0b89ec759315b9bd0efe5800b0eb88a71ece6d2a60746dd07b98af7R60), but it seems we need more encapsulation works to make it easy for use.

Maybe have a look at rusoto_s3 ? I'm not sure, maybe a account is needed to test.

* integration-test: deploy a temp S3-like service(for example https://min.io/) in CI pipeline and make test case using temp S3-like service.

It's totally feasible

integration-test: deploy a temp S3-like service(for example min.io) in CI pipeline and make test case using temp S3-like service.

+1

@zojw I'm trying minio and find that it fails to respond to put_public_access_block request:
Error: expected element type <CreateBucketConfiguration> but have <PublicAccessBlockConfiguration> (xml.UnmarshalError)
       5: cmd/handler-utils.go:55:cmd.parseLocationConstraint()
       4: cmd/auth-handler.go:330:cmd.checkRequestAuthTypeCredential()
       3: cmd/auth-handler.go:282:cmd.checkRequestAuthType()
       2: cmd/bucket-handlers.go:695:cmd.objectAPIHandlers.PutBucketHandler()
       1: net/http/server.go:2046:http.HandlerFunc.ServeHTTP()

https://github.com/minio/minio/issues/13714

I pushed a branch running IT with MinIO here. You can take a look but I think given MinIO is not fully s3 compatible, we should try to find something better for our requirements, e.g., localstack.

I pushed a branch running IT with MinIO here. You can take a look but I think given MinIO is not fully s3 compatible, we should try to find something better for our requirements.

thanks, @tisonkun MinIO is probably the most popular S3 available replacement, it's a surprise that it doesn't support set PublicAccessBlock Policy - -, but it may indicate that the function isn't very useful
After some search, maybe we can remove set PublicAccessBlock logic and set ACL in CreateBucket to protect bucket data
@@ -68,25 +65,12 @@ impl Storage<S3Object, S3Bucket> for S3Storage {
             .build();
         self.client
             .create_bucket()
+            .acl(BucketCannedAcl::Private)
             .bucket(name.to_owned())
             .create_bucket_configuration(config)
             .send()
             .await?;

-        self.client
-            .put_public_access_block()
-            .bucket(name.to_owned())
-            .public_access_block_configuration(
-                PublicAccessBlockConfiguration::builder()
-                    .restrict_public_buckets(true)
-                    .block_public_policy(true)
-                    .ignore_public_acls(true)
-                    .block_public_acls(true)
-                    .build(),
-            )
-            .send()
-            .await?;
-

@zojw thanks! Let me try to create a pull request for this test.
But actually, as we go much deeper into s3 support, we have to introduce a sponsored s3 environment for testing - the most s3 "compatible" service is s3 itself.


Changed Sonarqube converage report path

Changed Sonarqube converage report path in order to fix the reporting path issue

Superceded by #1169


App Configuration Provider for Python

Currently azure-appconfiguration library return the secret uri for keyvault referenced secrects. App Config Provider library should be created for python so that these secrets should be auto-resolved
The issue was originally reported here

We'd like to bring the feature parity to Python but the work is not trivial as we don't have a python provider at this moment. It's in our backlog and may not be available anytime soon. BTW, regarding to the original issue, for java provider we do offer resolving key vault reference. https://docs.microsoft.com/en-us/azure/azure-app-configuration/use-key-vault-references-spring-boot

@shenmuxiaosen Tks for the update. I am already using azure-spring-cloud-appconfiguration-config for spring boot apps. I was looking to find a similar provider for python apps, but based on your comment it seems we will have to wait for it.

https://github.com/Azure/AppConfiguration/issues/657

Beta Released: https://pypi.org/project/azure-appconfiguration-provider/

When is this slated for a GA release? I want to use App Configurator with Python SDK but need to justify with my company - can only do that if it's GA

We don't have an exact date at the moment. We are currently finalizing a few things. We have a few changes that we want to make before the final release.

A new beta version has been released azure-appconfiguration-provider 1.0.0b2

The App Configuration Provider for Python is GA since March 9.

@Sidewinder53  @arunAtCSGI  Note: some breaking changes were made for the GA, check out the announcement for info and links. https://github.com/Azure/AppConfiguration-Announcements/issues/28


Room dependencies are repeated: Removed the same

Room dependencies are repeated: Removed the same

@googlebot I signed it!


GitHub templates

Context
A part of open sourcing code is to be nice with the community. Github helps with
https://github.com/openregister/managing-registers/community
Changes proposed in this pull request

Add an issue template.
Add a pull request template.

Besides that, I would like someone more familiar with the development cycle to put together a CONTRIBUTING.md document. Either as part of this PR or in a new one.
/cc @openregister/product-team

/cc @Maidment64

@gidsg @fjavierm Given that no one else gave an opinion on the matter I changed the issue template with @gidsg suggestion. If you are happy with it I'll merge and we can revisit it later on.

@arnau Happy to go with it. I guess that this is one of the things that we will see if needs more specification when we/people start to use it. Right now, is very open, it depends a lot is the user filling the template add enough information on every section.
I prefer this approach that one with a lot of specific sections, it gives you more flexibility but, it gives you more responsibility.


TileType error

Error with enum class TileType: not available for other classes
Class Map.hpp: error: 'TileType' has not been declared    55 |     void addTile(int id, TileType type);

With the restructuring and architecture added also in docs folder in previous commit, this issue was also fixed/worked around
std::cout << "Mission success, that's why you structure things" << std::endl;


Create service binding guide for main odo.dev site

/area documentation
/kind bug

What mistake did you find / what is missing in the documentation?
We have the blog: https://odo.dev/blog/binding-external-service-with-odo-v3/
But there's no "full" example (shows all the devfile.yaml),
We need to create some documentation that follows the similar format to the quickstart on https://odo.dev
What is the relevance of it?

/kind documentation

#6077 addresses this.
/close


Use proper HTTP Methods for a RESTful Service

Predictions method should be GET.

Fixed in https://github.com/davidgasquez/moser/commit/86f33d224b69535fa5748b7d81d7ae16686942f3


Fix: Build and copy on parent folder / Feat: Add return values for strict ts

When we build, js files in main folder are not updated.
And i have added return values in for functions to be stricter.

Thanks for the pull request. The only issue I have with it is that we were using var as to be backwards compatible with MSIE, but it's almost 2023 so I guess we can drop MSIE support without issue.

Never mind, it's still compiling with var. I was apparently wrong in my assumption it would break MSIE.

yes, the files generated by uglify-js still use var in this version. when you no longer want to take care of old browsers, just update the package. it's mainly to have a clean base :)


Missing Purchaser for items in 7.0 replays

Thank you for updating the parser for the new patch! I've only come across one problem so far and that CDOTA_Item_* entities don't contain the m_hPurchaser property anymore. I'm not sure if this is how it should be, but if it is, is there another way to get the purchaser of the item? I know m_hOwnerEntity gets the current holder of the item, but if the item changes hands, is dropped, or is on the courier, the player who initially purchased the item is lost. Thanks!

Just to add, I'm using the 2.1.1-SNAPSHOT and the replay I tested was<PHONE_NUMBER>. There's also a new property m_iPlayerOwnerID, so testing to see if this is different from m_hOwnerEntity or if they always point to the same hero.

I tested with m_iPlayerOwnerID and this seems to be now tied to the purchaser of the item (m_iPlayerOwnerID and m_hOwnerEntity are sometimes different, with m_iPlayerOwnerID never changing for the same item) and perhaps replaced m_hPurchaser.

I have not looked at changes in the data yet, so I can't help.


Reformat/typing

Multiples changes has been made into the code here.

Types have been moved out of the AST for a clearer code base and are now located inside the Types module.
The evaluator has been rewritten
The list type has been revamped (Note: the type isn't really usable as this, because te parser isn't able to create lists from variables)
Every type has now its overload of all the operators. For example, the type string has now an addition operator + that concat two strings
Several other bug fixes and modifications have been made

This pull request is linked to this issue: https://github.com/gabyfle/Lice/issues/5


Order bar chart

Reverts EverseDevelopment/Forge.StaticWebsite.Dashboard#7

Task linked: CU-865bbttz0 Dashboard order bars by amount


Feature Request: Keras method to extract the graph

Several parts of the Tensorflow documentation suggest to the reader to use the higher level APIs (Keras / Estimators) when there is no explicit need if the low level API.
Of the high level APIs, there are some upwards mobility between the levels. For example, a custom Keras model can be converted into an Estimator.
However, getting access to the lower level APIs from the higher level apis is not so simple.
It would be nice - at least for visualization purposes - to allow users to see the automatically generated graph produced by the higher level APIs.
Taking a Keras model, converting it to an estimator, defining the serving_input_receiver_fn, and then exporting the estimator via export_savedmodel to see the graph is a bit convoluted for "high level" apis.
A subclassed Keras model, once compiled, could have a method as_graph to convert to the computational graph.

I think  tf.keras.callbacks.TensorBoard  might solve your problem, right?

@SumNeuron can you please comment on @facaiy 's suggestion?

Nagging Assignee @cy89: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I'll close the issue, please be free to reopen it if new information becomes available.

hi @facaiy -- it doesn't seem possible to add keras model graph to tensorboard if you're using GradientTape... all the instructions involve use of callbacks and model.fit. If this is possible, how? If not, can we add an easy way to submit keras model graph to tensorboard for visualization?
tf.summary.FileWriter doesn't work in eager mode in 1.14... but tf.contrib is going away and tf.contrib.ResourceSummaryWriter cannot seemingly submit graph, and we can't get the graph session inside @tf.function to send to tensorboard.
Please advise how do you add tensorboard keras graph when not using model.fit?

@bionicles bion, can you file a new issue for it?


Collision filter with mouse for matter physics

I found two methods
body.setCollisionCategory(cat1);
body.setCollidesWith([cat1, cat2]);
Using these two methods I can easily filter collision between two bodies. But I didn't find any example or documentation about how to do collision filter with mouse. I want that some bodies can be dragged with mouse and some bodies can't. Can you give an example?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Check out this example:
http://labs.phaser.io/edit.html?src=src\physics\matterjs\drag with pointer.js

This example contains only one body which is draggable. Can you add another body which is not draggable?

The easiest way is to just set a collision group in the constraint options:
http://labs.phaser.io/view.html?src=src\physics\matterjs\drag filter with pointer.js
Not sure this needs new constraint methods as you can pass it in easily enough (and for a constraint it's unlikely to change beyond creation either)
If you're going to use matter then it's well worth spending some time learning about how it handles collision groups and filters internally, which you can find in the matter.js docs.

Ah, brilliant! I didn't realise that was possible. :)
@rimonmath:
this.matter.add.mouseSpring({ length: 1, stiffness: 0.6, collisionFilter: { group: canDrag } });

https://github.com/photonstorm/phaser3-examples/blob/73bc72df0a0aba50589e199400ba404cb47987c5/public/src/physics/matterjs/drag filter with pointer.js#L28-L46

No worries - @photonstorm made it for you! :)


Scaling tool should consider storage utilization

The scaling tool does not consider storage utilization per node when it resizes a cluster. It's already a best practice to look at storage utilization, and it will become a hard requirement on 2018-10-01, when we'll prevent users from exceeding the maximum utilization per node.
We should update the tool to look at the metric bigtable.googleapis.com/cluster/storage_utilization and factor that metric into the calculation of how many nodes to add or remove.
GoogleCloudPlatform/python-docs-samples#1695 tracks the same issue, but for Python.

Closing as a "Won't Fix".


Add support for using custom error message

I wanted to add support for printing a custom help message when the plugin finds duplicates.
In my project, we are using the plugin only for specific sets of the packages and we have a help page that describes what a developer needs to do to fix the problem with duplicated pages.

Hi @darrenscerri, could you take a look at this change?


Package "Zotero" fails to install on aarch64-darwin

Describe the bug
The Zotero package fails to install on aarch64-darwin
Steps To Reproduce
Steps to reproduce the behavior:

nix-shell -p zotero

nix-shell -p zotero
error: Package ‘zotero-<IP_ADDRESS>’ in /nix/store/m5y5zw94rgfj0i0fpg6m7nmfxk04837p-nixpkgs-21.11pre327990.4789953e5c1/nixpkgs/pkgs/applications/office/zotero/default.nix:146 is not supported on ‘aarch64-darwin’, refusing to evaluate.

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

            $ export NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1

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

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

Expected behavior
The package should install and function as expected
Notify maintainers
@i077
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 21.1.0, macOS 12.0`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.5pre20211007_844dd90`
 - channels(shauryasingh): `"darwin"`
 - channels(root): `"nixpkgs-21.11pre327990.4789953e5c1"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

https://forums.zotero.org/discussion/86103/apple-silicon-compatibility/p1
Nevermind, doesn't build on aarch64-darwin regardless, not something nix can/should solve

Currently the derivation for Zotero in nixpkgs is only for linux hosts. If you're interested, you could try modifying this derivation to support macOS hosts as well (x86_64 for now, but it should work on aarch64 via Rosetta 2).


DID Exchange fails when requester uses numAlgo 0

I added the following test case to didexchange-numalgo.test.ts:
  test('Connect using numalgo 0 for requester and default setting for responder', async () => {
    await didExchangeNumAlgoBaseTest({ requesterNumAlgoSetting: PeerDidNumAlgo.InceptionKeyWithoutDoc })
  })

Then the test fails with error:
CredoError: Did document creation failed: {"state":"failed","reason":"Missing key type"}

I expected did:peer:0 to be treated as a DID for an agent that does not have endpoints. This was supported in the connection protocol by using an endpoint "didcomm:transport/queue". This feature is beneficial for mobile agents that do not utilize mediators.
The agent could create a numAlgo 2 DID by assigning a dummy service with an endpoint "didcomm:transport/queue". However, for this use case, using did:peer:0 would be more convenient and concise.

Hmm but in that case we'd have to assume that the wallet wants use didcomm:transport/queue when using did:peer:0 which is very implicit behavior and would also require 'hacks' to make it work as the did resolver doesn't return a service for a did:peer:0. It's basically the same as did:key.
I think in this case you should use any of the other did:peer numalgos


Apply Proxy Default Option For Variants

Is your feature request related to a problem? Please describe.
Add a new property named ApplyProxy to the Variant type in util\types.go.
Describe the solution you'd like
This option is meant to optionally disable applying the system proxy to a variable or to stop it from being appended to the end of a command.

This property should either be the opposite of ApplyProxy or have a default value of true since we want to specify it only when set to false.


Predict(...) on large newData seems unordered

Hi,
Sorry for not providing a reproductible example. However the problems seems to arise when dataset are not "small".
I tried by my own Sparklyr on a bigger dataset. I picked up Airlines2007 and 2008. Merge with Weather on Date. Trained a model on 2007 (300k rows) and predicted on 2008 ((300k rows).
At the end i train a linear regression and a RandomForest.
my_ml_rf = ml_random_forest(x = train ,response = "ArrDelay", features = myFeatures)
p1 = predict(my_ml_rf, validation)
p2 = predict(my_ml_rf, validation)
qplot(p1, p2)

my_ml_linear = ml_linear_regression(x = train ,response = "ArrDelay", features = myFeatures)
p11 = predict(my_ml_linear, validation)
p22 = predict(my_ml_linear, validation)
qplot(p11, p22)

However and surprinsingly, here are the graphs

p11[120000:110000] == p22[120000:110000]

It clearly seems that "bloc" order change sometimes. I guess it is due to the inner functionnement of Spark. The first returned block are the first processed. However it is an annoying problem.
Best regards,
Jacques

This is the code we use for implementing predict():
predict.ml_model <- function(object, newdata, ...) {
  params <- object$model.parameters
  predicted <- sdf_predict(object, newdata, ...)
  column <- sdf_read_column(predicted, "prediction")
  if (is.character(params$labels) && is.numeric(column))
    column <- params$labels[column + 1]
  column
}

I wonder if the issue could be that the row order in the predicted data.frame, as returned by sdf_predict, is not as expected? What does the output from sdf_predict look like?

Not sure to have fully understood what you wanted.  Do you see the same weirdness?

I should have been more clever too ;)
p111 = sdf_predict(my_ml_linear, validation)
p222 = sdf_predict(my_ml_linear, validation)
local_p111 = as.data.frame(p111)
local_p222 = as.data.frame(p222)
head(local_p111)
head(local_p222)
# ArrDelay is the response variable
all.equal(local_p111$ArrDelay, local_p222$ArrDelay)
sqrt( mean( (local_p111$ArrDelay-local_p111$prediction)^2 ) ) # Calcul du RMSE
sqrt( mean( (local_p222$ArrDelay-local_p222$prediction)^2 ) ) # Calcul du RMSE

Same RMSE. Each observation is linked to its good prediction.
I guess you're right, the row order is not the same. If i want to export locally my prediction, i think i've to add an index column. Hence i'll be able to link it to the right observation.
However predict(...) is a bit confusing right now to me. And i don't think i'll the only one to miserably fail with it. Using sdf_predict(...) seems a bit more intuitive to use right now.
Thank you for your help.

Thanks for the update -- it looks likes we definitely need to make sure predict does the right thing as well; it should produce a column vector in the same row order as the input newdata data set.

+1

Looks like https://github.com/rstudio/sparklyr/commit/7eb70d0f55e30a68eebec1f69a152e2304542a77 fixed this issue since I can't repro the ordering issue with the newest bits, correct @kevinushey?
x <- flights_tbl %>% sdf_partition(training = 0.1, test = 0.9)
my_ml_linear <- ml_linear_regression(x = x$training, response = "dep_delay", features = c("dep_time", "sched_dep_time", "arr_delay"))
p1 <- predict(my_ml_linear)
p2 <- predict(my_ml_linear)

qplot(p1, p2)

Yes, I believe we should be handling this correctly now (although we might need to revisit this in the future for performance reasons)


multi-agent support

Thanks for this really cool work!
I wanted to ask if it supports multi-agent? As in, can I instantiate multiple robot models in the same environment and have them execute actions in parallel - navigate, interact with objects?
If yes, can the agents interact with the same object at a particular time step? For instance, the case where the agent needs to hand off an object to another agent.

Yes, we do support multi-robot use cases, and yes - it should be possible for multiple robots to handle the same object. The environment should accept either a dict or a concatenated list as the action for multiple robots, and produce a dict as the observation. Let us know if you run into any issues with this!

Awesome! Thanks! Will let you know.

Hi! It seems that each Task (eg. PointNavigationTask) is binding to only one robot. I wonder can we still achieve multiple agents training in such task?

@cgokmen hi! Is there any example or tutorial to run multi-robot?


Update the license list XSD

Build is failing because OSI site is still not accessible from Github - merging since there are no other issues.


Update Generic Pages “contact”

Automatically generated by Netlify CMS

👷 Deploy Preview for shimmering-marigold-4f5b4a processing.

Name
Link

🔨 Latest commit
26ccc9659883b2f551d8aa35e2c8ccb28145bf7d

🔍 Latest deploy log
https://app.netlify.com/sites/shimmering-marigold-4f5b4a/deploys/63381b8453cf3e00095aa949


schema caching needs xmlschema version

just so I don't forget to fix it: when testing #17 locally a few tests failed that weren't testing on CI, so I updated xmlschema to check... and they all failed.  I realized that it was ultimately due to the pickled schema cache being stale.  So that cache key should minimally have the xmlschema version in it, but maybe we don't cache at all.

We should grab the .xsd and include it with the generated python code, then just load that. That thing isn't changing so no sense in fetching it from a remote server every run. (This usually takes less than a second but it also requires a network connection)
The schema parse takes over 2 seconds on my machine which I agree isn't great, but the hassle of managing the cache may not be worth it as you're also realizing...

Remember there are only 2 truly hard problems in computer science: Naming things, cache invalidation, and off-by-one errors.

that's hilarious.

sounds good.  I'll add the xsd to source.  I think i'll leave the caching in there for now, but include the xmlschema version string and ome-types version in the key.  That should cover everything but development woes, and we can deal with that ourselves

Actually the schema parse only takes 1 second. I was accidentally timing it loading from the URL rather than a local file. I don't know if 1 second is even worth all the hassles of caching. (e.g. if it's a binary wheel install, that path doesn't even exist for writing a file to and you need to find somewhere else...)


s3images issue with upper case file extensions

I've been trying to use the s3 images field type and noticed it wasn't working properly, returning null from the graphql response for the image field.
Uploading to the bucket works fine however.
While debugging I noticed that the list of supported extensions is lowercase:
https://github.com/keystonejs-contrib/k6-contrib/blob/1c7861c6b2d8fa2d730679d3103888f3fdf851a0/packages/fields-s3images/src/lib/utils.ts#L7
The image I was testing with had an uppercase extension of .JPG. It appears that it then doesn't match this check:
https://github.com/keystonejs-contrib/k6-contrib/blob/1be465ebc16c0369759771f5b522103323ab4765/packages/fields-s3images/src/lib/image.ts#L51-L53

There's a similar issue with files with the .jpeg extension. They will upload fine, but will return null afterwards.


'targetChannel is not defined' when running !beat

Adding to spreadsheet works there's just an error. This also stops the sound from playing. Most likely related to add stream marker code.

Fixed in 9bfbc3d161972fb2b5380ddca725dddda1492e89


When LTFS Tape Mounted - lib/udev/scsi_id return "Device or resource busy "

Describe the bug
We are looking forward to understand the behavior of this sequence :

A tape is mounted successfully through LTFS binary
We execute the following command : /lib/udev/scsi_id --export --whitelisted /dev/sg15 --verbose

The result is "scsi_id: cannot open /dev/sg15: Device or resource busy"
To Reproduce
Steps to reproduce the behavior:
Execute the same sequence
Expected behavior
With Quantum - HPE - ... LTFS "driver", the behavior is to return the SCSI information related to this device :
Example :
ID_SCSI=1
ID_VENDOR=HP
ID_VENDOR_ENC=HP\x20\x20\x20\x20\x20\x20
ID_MODEL=Ultrium_8-SCSI
ID_MODEL_ENC=Ultrium\x208-SCSI\x20\x20
ID_REVISION=HB91
ID_TYPE=tape
ID_SERIAL=1HP_Ultrium_8-SCSI_C0F9000105
ID_SERIAL_SHORT=HP_Ultrium_8-SCSI_C0F9000105
ID_SCSI_SERIAL=C0F9000105
Linux:

OS: Centos 7.9

I think it is as designed because LTFS opens the device file exclusively to avoid unexpected operations from other processes. As a result, other applications get EBUSY against open().
https://github.com/LinearTapeFileSystem/ltfs/blob/0fae6d816c353cf82e2eecf7970649627ebcb21c/src/tape_drivers/linux/sg/sg_tape.c#L432-L463

Thanks Abe for the feedback and answer, this makes sens, i had something like this in mind. Thanks for the confirmation.
We will work in our side to manage this properly.


Different Character Sprites

https://github.com/MarkSort/batwaves/blob/master/godot/Player/Player.png

Done!


Pass the shell arguments directly to bash

This allows us to run the "run" script using "set -eo pipefail" by default.
The previous wrapper executed the arguments as a bash command string, which
allowed the user to run any command, but didn't give us a chance to special case
if the argument was a script passed from the GHA "run" command.
This is a breaking change.
Porting guide:

Instead of msys2 mybinary you now have to run msys2 -c 'mybinary'
To revert the bash error settings you can use set +e and set +o pipefail
in your run script

See #43

One possibility, I'm open to ideas...

However, I'm curious about the behaviour when the default shell is set. FTR, some months ago two different intermediate files were provided: msys2do.cmd and msys2.cmd. The former was the original one, which required to use it explicitly (run: msys2do whatever, and the latter was the one to be used with run: |. Then, I unified both of them.

We could also add a msys2-bash and keep the old command as is, basically replicating the old split and not breaking backwards compat.

Hence, it seems that the only context where -c is required is for executing msys2 -c "a command" inside a cmd or powershell. In all other cases, the arguments are passed as an intermediate file. It would be useful to make this explicit in the README.

When you set the shell the content is always passed through a file according to this https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#custom-shell so, at least for me, everything works as expected. Am I missing something?

We could also add a msys2-bash for setting shell and keep the old command as is, basically replicating the old split and not breaking backwards compat.

I'd rather enforce setting the shell and keeping msys -c for very specific cases (or even deprecating it). In the end msys -c is for executing MSYS2 from cmd/powershell, which is not the workflow we want to recommend.
Calling msys arguments from cmd/powershell was the only option when GitHub Actions was moved out of beta, but the service/ecosystem was not ready yet. At that time, it was not possible to set a custom shell (actions/runner#240) so we were forced to bash/cmd/powershell. Ideally, we would set it programmatically (see actions/toolkit#318), so the UX is equivalent to the default bash. Meanwhile, shell: msys2 {0} is the closest we can get.
Hence, I think now is a good moment to introduce this breaking change, before the action is more popular.

When you set the shell the content is always passed through a file according to this https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#custom-shell so, at least for me, everything works as expected. Am I missing something?

No, it is coherent with the current docs. But the environment has been changing and I was slightly outdated.


User logged-out even after correct login data

Hi,
I have setup the wlox and all is working fine, i can work on backend, i can even create new account on frontend, then i receive email with login data. My problem is that when i login my wlox return always
#this is part from the api/lib/Users.php
if (!($CFG->session_id > 0)){
			return array('message'=>'not-logged-in','attempts'=>$login_attempts);
}

The above is called by frontend here ->
# at frontend/lib/user.php
static function verifyLogin($query) {

# this return empty
if (isset($query['User']['verifyLogin']['results'][0]))
			$result = $query['User']['verifyLogin']['results'][0];

#this destroy session and back to loginpage
f (!empty($result['error']) || !empty($query['error']) || !isset($result)) {
			$session_id = session_id();
			if (!empty($session_id)) {
				session_destroy();
				$_SESSION = array();
			}
			return false;
		}

}

Even if i put good data and i am redirected to login page i dont know why, the session is created in the databse and also in /tmp. Any one can help?

@mbassan can u help me please?

also in frontend error log i can see
[06-Mar-2018 07:41:26 America/New_York] PHP Notice:  Invalid session data. in /home/user/exchange.domain.com/lib/stdlib.php on line 104
multiple times error spam whole error.log


add --input-file-buffer-size option

Set the buffer size for file type input to increase reading performance for large size documents.

https://github.com/taskrabbit/elasticsearch-dump/releases/tag/v0.18.0


Set the default implementation of getType method for ForeignKeyConstraint

Set the default implementation of getType method for ForeignKeyConstraint to Type.FORIENG_KEY

This change was pushed through from the upstream project, thanks!


Restructure of OpenAPI docs for .NET 9

This PR attempts to restructure the OpenAPI docs for .NET 9, while leaving the docs for .NET 8 and previous as is.
The intent is to split the current "Generating OpenAPI document" article, which currently covers both how to generate the document and how to customize it, into two separate articles, focusing on each aspect individually.
While this looks like a large change, it is mostly just moving some content from the existing article to a new one, and updating the links in the table of contents and within the articles themselves.

Internal previews

📄 File
🔗 Preview link

aspnetcore/fundamentals/openapi/aspnetcore-openapi.md
aspnetcore/fundamentals/openapi/aspnetcore-openapi

aspnetcore/fundamentals/openapi/customize-openapi.md
aspnetcore/fundamentals/openapi/customize-openapi

aspnetcore/fundamentals/openapi/include-metadata.md
aspnetcore/fundamentals/openapi/include-metadata

aspnetcore/toc.yml
aspnetcore/toc

I just reworked this to pull in Sofia's changes to describe build time generation of OpenAPI documents.

@mikekistler What are your thoughts on adding a section to the "Generate OpenAPI documents" page under "Configure OpenAPI document generation that covers what kind of customizations can be made on the document? I'm thinking about things like being able to change the document name by passing an argument the AddOpenApi call or changing the default route of the OpenAPI document in the MapOpenApi call.
Also, I feel like transformers should go at the top of the "Customize OpenAPI documents" page instead of at the bottom. Thoughts?

Regarding:

What are your thoughts on adding a section to the "Generate OpenAPI documents" page under "Configure OpenAPI document generation" that covers what kind of customizations can be made on the document?

I think that content is there but may be difficult to find because of the way I inserted the build-time generation content.
Here's the "table of contents" for the "Generate OpenAPI documents" article:
>grep '^#\+ [^[]' $f
# Generate OpenAPI documents
## Package installation
## Configure OpenAPI document generation
## Generate OpenAPI documents at build-time
### Customizing build-time document generation
#### Modifying the output directory of the generated Open API file
#### Modifying the output file name
#### Selecting the OpenAPI document to generate
### Customizing run-time behavior during build-time document generation
## Options to Customize OpenAPI document generation
### Customize the OpenAPI document name
### Customize the OpenAPI version of a generated document
### Customize the OpenAPI endpoint route
### Customize the OpenAPI endpoint
#### Limit OpenAPI document access to authorized users
#### Cache generated OpenAPI document

I think the content you are suggesting is in the "Options to Customize OpenAPI document generation". I inserted the section for generation at build time above that, but maybe it would be better to move that to the end? I am fine either way, so just let me know your preference.

Regarding:

I feel like transformers should go at the top of the "Customize OpenAPI documents" page instead of at the bottom.

I think the 80% use case will be that developers use the built-in mechanisms for adding metadata, and that transformers will be an advance use case that developers reach for only when a built-in mechanism isn't available. Based on this, I think the built-in mechanisms should be before the advanced use cases.
But another option is to separate these into two different articles, which might be better for folks to quickly find the content they want. Now that I've mentioned this I kinda like this idea. What do you think?

I inserted the section for generation at build time above that, but maybe it would be better to move that to the end?

Yep, that's exactly it. I was proposing moving "## Options to Customize OpenAPI document generation" between "## Configure OpenAPI document generation" and "## Generate OpenAPI documents at build-time".

But another option is to separate these into two different articles, which might be better for folks to quickly find the content they want. Now that I've mentioned this I kinda like this idea. What do you think?

This is my preference too. It also gives us a nicer place to land snippets of various transformers that people can use for common scenarios (e.g. changing enum handling).

@captainsafia I have pushed some updates that I hope address your comments. Please have another look.

@tdykstra This PR has one build warning:

[Warning] File aspnetcore/fundamentals/openapi/buildtime-openapi.md with URL /aspnet/core/fundamentals/openapi/buildtime-openapi was deleted without redirection. To avoid broken links, add a redirection.

The URL identified was only added a few days ago by Sofia's PR so I don't think a redirection is really needed. Can we suppress this somehow or do we just add the redirect to satisfy the tooling?

The URL identified was only added a few days ago by Sofia's PR so I don't think a redirection is really needed. Can we suppress this somehow or do we just add the redirect to satisfy the tooling?

Ignore the warning and merge the PR, it's a PR-level warning so it will go away after this PR is merged.

@tdykstra The merge was blocked until the warning was resolved so I just added the redirect to be able to move forward.
Do you want to give this a quick review or am I okay to merge this?


Add Steam ID to settings panel.

Please add the logged-in Steam ID to the settings panel, to confirm that you're logged into the right account.

The store.steampowered.com/dynamicstore endpoint does not show the username so it won't be possible. You can quickly check what account you are logged in to by opening the link to the store.


feat: remove support for Python 3.8

Langchain v0.3 removes support for py3.8.
Python 3.8 will be EOL at the end of next month (Oct 2024).
Removing support of it from future versions to allow for LangChain package bump.

@averikitsch Do you mind removing the Python 3.8 integration test from the required checks? Please and thanks 🙏 😄


CLOUD-44354 fixed azure rm image handling

@biharitomi or @keyki

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1790/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1791/


Is it possible to retrieve the version through plugin?

[x] I have read the guidelines for submitting issues

⚠ Is your feature request related to a problem? Please describe
More of a question
💡 Describe the solution you'd like
Is is possible to retrieve the version of a library?
e.g.  I have this versions.properties
version.firebase-crashlytics-gradle=2.9.2
##                      # available=2.9.3
##                      # available=2.9.4
##                      # available=2.9.5

And I want to find the version of firebase-crashlytics-gradle. Is this possible somehow?
🤚 Do you want to develop this feature yourself?

[ ] Yes
[x] No

Hello, pease, use discussions for questions, not issues. Yes, it's documented, you have versionFor.


Minor updates to tasks page

Protect the two "clear" buttons (both for all collections and for each single collection) with a confirmation dialog.
Implement the "delete single task" button, also with a confirmation dialog.
Rename the sections into "Add tasks", "Collect tasks from a package", "Add a single task", "Task List".
When task collection fails, there should be a way to see the logs (I think we had this in some previous version, didn't we? Right now it doesn't seem to work)

When task collection fails, there should be a way to see the logs (I think we had this in some previous version, didn't we? Right now it doesn't seem to work)

Apologies, I was using main but I just noticed that this feature is present in client-0.1.-refactor


Extra large build size

In the production version, the package weighs 451 KiB

what would you consider to be a small size?

I believe the main reason for the large bundle is moment/timezones dependencies. So moving them to the peerDependency section may help.

I am having a problem with the build on my computer. I have a lot of personal things I am dealing with at the moment,
won't rant, unless you are interested.
@pvozniuk Could you please take a look at this for me, please :)

Good morning,
This problem should be resolved in version 5.0.0-beta1
Please take a look and let me know by the end of the week.

Closing due to inactivity.


Explore modal

Implemented -

filters-grid block for explore modal, that is ingested as a fragment while creating the modal. It is authored here.
advanced-heading block to add capability of adding button with H2 heading, it is auto-blocked. Author need to only add a link after first H2 in a section and it will be decorated using advanced-heading block css.

Fix #37
Test URLs:

Before: https://main--infosys--aemsites.hlx.live/drafts/anuj/iki
After: https://explore--infosys--aemsites.hlx.live/drafts/anuj/iki

@anuj-adobe for me, Dialog was not opening. please check if it is working for you

@shrotia it was happening because content was not published.
I've published it and it is working fine.


Optimize db insertions

DB insertion can get exponentially slow (~90s for 20k entries) because the array that olds the objects is a Swift.Array and has to be copied every time we append new objects.
A cool trick for Swift value types is that they are always copied unless uniquely referenced.
That array is hold by a dictionary and to be mutated must be extracted from it and then re-assigned (2 references) making it not uniquely referenced.
To avoid this and use the uniquely referenced trick we can use a box that holds the array:
class ArrayBox<T> {
    var array: [T]
}

then the dictionary will be of type [String: ArrayBox<KStorable>]
And then we can append elements without reference more than once:
dictionary[someKey].array.appendContentOf(....)

[ ] Create a performance measure block to avoid regressions
[ ] Document in the code and link to the issue
[x] Reply with the performance gain (expected 1500% :parrot: )

:hope:

https://travis-ci.org/devlucky/Kakapo/jobs/121879811 5.8seconds


Fixed logout message

What issue does this PR address?
Logout message fixed
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[ ] The commit follows our guidelines
[ ] Unit Tests for the changes have been added (for bug fixes / features)

Other information:

thanks!

@leastprivilege my pleasure