fix: Revert to 2.0.2

Description
Revert this commit ffcd5b69763dbc3c8481be97a8d1944d65b5c249

use git revert command is better

use git revert command is better
git revert b3d7939a5ed1d26d120f36482b301194dd8f65de

Done ✅


cairo: Configure without egl glesv2

@agherzan @floion
Signed-off-by: Theodor Gherzan<EMAIL_ADDRESS>

LGTM

Can one of the admins verify this patch?

test this please

LGTM


Adding a new flag.

Hello,
I would like to request a flag. This flag would be something like item-drop-on-death or something of the sort, it would be a flag that would allow items to drop when a player dies even if the gamerule keepInventory is on. This flag would be used for servers that want to have a peaceful world where their items don't drop, but that also want to have a pvp arena where players can fight each other for loot.

there's already something like this in B11's config, though I don't think it bypasses gamerule keepInventory.

Yeah I am trying to only set item drops on death for a specific area not the full map.


Problems with Tensorflow 2.4

Hi,
because there are problems with new CUDA 11.2 to run Tensorflow 2.3 I had to update to 2.4.
I want to run dqn_train.
Now I got error because of problems with tf.compat.v1.disable_eager_execution( ) in agent_keras_rl_dqn.py:
AttributeError: 'TensorBoard' object has no attribute '_should_trace'
I changed to tf.compat.v1.enable_eager_execution(), now I get error:
AttributeError: 'DQNAgent' object has no attribute 'distribute_strategy'
If I try to add "distribute_strategy" to DQNAgent(main=.....), it doesn't change anything.
What to do?
Here complete log:

2021-03-10 14:42:29.846718: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'cupti64_110.dll'; dlerror: cupti64_110.dll not found
2021-03-10 14:42:29.849472: W tensorflow/stream_executor/platform/default/dso_loader.cc:60] Could not load dynamic library 'cupti.dll'; dlerror: cupti.dll not found
2021-03-10 14:42:29.849573: E tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1415] function cupti_interface_->Subscribe( &subscriber_, (CUpti_CallbackFunc)ApiCallback, this)failed with error CUPTI could not be loaded or symbol could not be found.
2021-03-10 14:42:29.849915: I tensorflow/core/profiler/lib/profiler_session.cc:172] Profiler session tear down.
2021-03-10 14:42:29.850186: E tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1496] function cupti_interface_->Finalize()failed with error CUPTI could not be loaded or symbol could not be found.
Traceback (most recent call last):
File "main.py", line 255, in
command_line_parser()
File "main.py", line 78, in command_line_parser
runner.dqn_train_keras_rl(model_name)
File "main.py", line 204, in dqn_train_keras_rl
dqn.train(env_name=model_name)
File "C:\Users\neuron_poker\agents\agent_keras_rl_dqn.py", line 102, in train
start_step_policy=self.start_step_policy, callbacks=[tensorboard])
File "C:\Users\Shadow.conda\envs\neuron_poker\lib\site-packages\rl\core.py", line 103, in fit
callbacks.set_model(self)
File "C:\Users\Shadow.conda\envs\neuron_poker\lib\site-packages\tensorflow\python\keras\callbacks.py", line 286, in set_model
callback.set_model(model)
File "C:\Users\Shadow.conda\envs\neuron_poker\lib\site-packages\tensorflow\python\keras\callbacks.py", line 2110, in set_model
self._log_write_dir = self._get_log_write_dir()
File "C:\Users\Shadow.conda\envs\neuron_poker\lib\site-packages\tensorflow\python\keras\callbacks.py", line 2143, in _get_log_write_dir
self.model.distribute_strategy)
AttributeError: 'DQNAgent' object has no attribute 'distribute_strategy'

I had to revert to Tensorflow 2.3 to fix this error. See here: https://github.com/tensorflow/tensorflow/issues/44563


No signature of method: build_71y17k4eyt1asltzpoo9zhmdo.kotlin() is applicable for argument types: (build_71y17k4eyt1asltzpoo9zhmdo$_run_closure2) values: [build_71y17k4eyt1asltzpoo9zhmdo$_run_closure2@5fdb5b92]   Possible solutions: notify(), toString(), toString(), toString(), toString(), split(groovy.lang.Closure)

I found an above issue when I run my app in android.

I also had this problem because I was using an old Kotlin version (1.6.10).
When I move to version 1.8.22 it works like a charm.

sounds to me as well that it's related to an old kotlin version or kotlin gradle plugin or similar.. try updating all dependencies if it's still not working let me know.


Implement service metadata in GET:/metadata

The response of the endpoint should include information about the service associated with each metadatum if applicable.

Fixed in #388


[Python][CI] Support ORC in Windows wheels

Now we support building with ORC enabled on Windows (ARROW-17817), we could also add this to the Python wheel packages for Windows (vcpkg seems to have an orc port for Windows as well)
Reporter: Joris Van den Bossche / @jorisvandenbossche
Note: This issue was originally created as ARROW-18329. Please see the migration documentation for further details.

Resolved by https://github.com/apache/arrow/pull/40609


How leaflex sets the zindex layer order for geojason layers

Checklist

[X] I've looked at the documentation to make sure the behavior isn't documented and expected.
[X] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[X] I've searched through the current issues to make sure this hasn't been reported yet.
[X] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce
No response
Expected behavior
set geojson layer down or up layer zindex
Current behavior
setZIndex dont work
Minimal example reproducing the issue
No response
Environment
Leaflet version: 1.7.1, 1.8.0-beta.0
Browser (with version): Chrome 99.0.4844.82, Firefox 98.0.1
OS/Platform (with version): Windows 10 laptop

Please see https://stackoverflow.com/questions/39767499/how-to-set-the-zindex-layer-order-for-geojson-layers.

Duplicate of #8071
I would not say that we never implement this, so let us keep open #8071


Submit To Azure button not there when creating the job

I've followed the document and created the input and output etc, compiled the script, then got to the point where I want to submit the job. But after selecting the location nothing happens, there is no 'Submit to Azure' button.

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

ID: e8f1f47b-62e5-1652-d8d2-189b0d2f4128
Version Independent ID: 09ac7e5d-441c-7911-a105-16ae5d515524
Content: Quickstart - Create an Azure Stream Analytics job in Visual Studio Code
Content Source: articles/stream-analytics/quick-create-visual-studio-code.md
Service: stream-analytics
GitHub Login: @su-jie
Microsoft Alias: sujie

I've realised it's because I chose UK South. Although I don't think this is explained in the document, you don't seem to be able to use that location. West US, for instance, works.

Which version of asa-extension are you using? I tried with v1.0.1, it worked well with UK South.

Please feel free to reply~


ts merge not working (probably works, but i need help)

Hi, i am new with tsduck and i have to say i am amazed by this software! I am playing with tsduck for few hours so i could miss something but i have troubles with ts merging.
I am trying to merge few multicast SPTS or even MPTS to one big multicast MPTS. User guide is really straightforward with this and everithing should work automaticly but i didnt get right results.
After merging i only get original stram on output. Even compared by bitrate that merged stream is missing.
I've tried start with null or craft input but then i get stream without propper tables and i dont know how to right limit bitrate.
I am attaching analyze reports and command and debug output. Could somebody help me with right way how to do it? I've tested it with version 3.19 and 3.15 on debian buster (compiled) and newest ubuntu with same results. .
Thanks a lot for right kick :)
merge.txt
playboy.txt
command_and_debug.txt
esp.txt

Can I provide more informations for this issue?
Or could anybody show some example configuration for IP TS merging?


add width to card title

issue: #82

add width to card title string if there is a badge rendered to prevent title/badge overlap

/gcbrun

Temporary Url for review: https://apigw-dev--pr-85-ien8cezo.web.app


Carthage support.

This library already supports carthage without any changes.
The only requirement carthage has is the scheme should be public and Shoyu scheme is already public.

hey @bhargavg thanks!


swingthreshold,powerOnEffect,flickerEffect

in this new version, how to control the sensibility,
in the previous version lightsaber os you changed the parameters in this part of the code
/*** SOUNDFONT 2 ***/
case 3:
this->powerOnTime = 300;
this->powerOffTime = 700;
this->powerOnEffect = 0;
this->powerOffEffect = 0;
this->flickerEffect = 2;
this->swingthreshold = 200;
boot[0] = NR_CONFIGFOLDERFILES + NR_JUKEBOXSONGS + NR_SF1_FILES + 1;
boot[1] = boot[0];
powerOn[0] = boot[1] + 1;
powerOn[1] = powerOn[0];
powerOff[0] = powerOn[1] + 1;
powerOff[1] = powerOff[0];
swing[0] = powerOff[1] + 1;
swing[1] = swing[0] + 7;
clash[0] = swing[1] + 1;
clash[1] = clash[0] + 7;
lockup[0] = clash[1] + 1;
lockup[1] = lockup[0];
blaster[0] = lockup[1] + 1;
blaster[1] = blaster[0] + 3;
menu[0] = blaster[1] + 1;
menu[1] = menu[0];
hum[0] = menu[1] + 1;
hum[1] = hum[0];
break;

should work the same way, except it might be defined somewhere else. Look for swing sensitivity. Alternatively the new release also has a config mode menu where you can change the sensitivity.

Protonerd Thank you for answering so quickly
for example
in case 1 for example. mod pre establish,  powerOnEffect, powerOffEffect,swingthreshold
it is correct to configure it
or is modified in other parts of the code, to preset these effects of sensibility, flicker. etc..
#ifdef DIYINO_PRIME
case 0:
// soundFont directory 01 :
this->powerOnTime = 800;
this->powerOffTime = 800;
break;
case 1:
// soundFont directory 02 :
this->powerOnTime = 1050;
this->powerOffTime = 900;
this->powerOnEffect = 0;  ------
this->powerOffEffect = 0;--------
this->flickerEffect = 2;--------
this->swingthreshold = 200;-----
break;

Swing sensitivity/threshold is now a config variable stored in EEPROM, a global variable in the config stucture::
uint16_t swingSensitivity;
It defaults to 1000, which works remarkably well, if you want to change it, there is a dedicated config mode item CS_SWINGSENSITIVITY that can be used to set the sensitivity level on-the-fly.

Did you try the swing sensitivity option? You can try the new commit, it contains some major bug fixes.

I'm not at home to prove it. tomorrow I can try it

Do you still need support here? Otherwise I would like to close.

Close


Automatic installation of updates

Description of the new feature / enhancement
The option to enable not only downloading updates automatically but also installing them
Scenario when this would be used?
It's important to me because I want to keep it up to date but I don't really see the value of getting notifications to manually click a button for that.
Supporting information
No response

this is hard due to the non-per user needs to be elevated to do the install and other features require elevation to be turned on for per-user installs

this is hard due to the non-per user needs to be elevated to do the install and other features require elevation to be turned on for per-user installs

Is it possible to ask for elevated permissions when this option gets chosen in the options menu?

/dup https://github.com/microsoft/PowerToys/issues/27651


Spark configuration options with spaces aren't parsed properly in default images

This is most notorious with spark.driver.extraJavaOptions and spark.executor.extraJavaOptions. The following configuration won't work properly:
--conf spark.driver.extraJavaOptions=-XX:+HeapDumpOnOutOfMemoryError -XX:+PrintGCDetails
Because we pass the driver's JVM options in a single environment variable, we don't handle the spaces in a single configuration value properly.

Hi @mccheah, would you like me to look into this?

ok I think we have a way to do this:

in the launcher, split the spark.*.extraJavaOptions variables using Utils.splitCommandString
add a series of environment variables to each pod, named EXTRA_JAVA_OPT_0, EXTRA_JAVA_OPT_1, EXTRA_JAVA_OPT_2, etc.
in the Dockerfile, read these variables out of the environment and apply to the Java command, possibly using bash arrays (requires bash 4+) to avoid an unbounded listing of variables

Is there a reason these sequences don't work when encased in quotes?
--conf spark.driver.extraJavaOptions="-XX:+HeapDumpOnOutOfMemoryError -XX:+PrintGCDetails"
(provided the quotes are re-propagated as needed)

I don't know why that isn't working, though this format is something I tried. I think it has something to do with the nuances of bash environment variable substitution.

It bugs me that this doesn't work, but OTOH I can't definitively suggest a fix.
You're saying that putting quotes around it like:
${JAVA_HOME}/bin/java "$SPARK_DRIVER_JAVA_OPTS" ...
wasn't working?

correct, just adding those quotes to the Dockerfile was insufficient to get the tests to pass (we spent most of Friday wrestling with this)

If quotes are placed around the entire string then the entire string is passed as a single argument to the JVM. That doesn't parse out and delimit the options properly.

Maybe something like:
echo $SPARK_DRIVER_JAVA_OPTS | readarray -d ' ' -t TEMP_OPTS
${JAVA_HOME}/bin/java ${TEMP_OPTS[@]} ...

That might at least remove the need for the separate env vars and the internal processing?

the -d option isn't supported by Bash as installed by our Docker images in alpina linux. I also think that this would fail because we want -Dspark.executor.extraJavaOptions=-Dopt1=opt1Value -Dopt2=opt2Value to be treated as a single JVM option.

I suppose we might work around the -d by doing something like
echo $SPARK_DRIVER_JAVA_OPTS | tr ' ' '\n' | readarray -t TEMP_OPTS

I'm a bit confused by the single-jvm-option consideration.  I thought the problem with "$SPARK_DRIVER_JAVA_OPS" was that it presented multiple options as a single option when it shouldn't.

This command: echo $SPARK_DRIVER_JAVA_OPTS | tr ' ' '\n' | readarray -t TEMP_OPTS - this suffers from the same issue for piping to readarray, in that the resulting array is always empty:
bash-4.3# export SPARK_DRIVER_JAVA_OPTS="-Dspark.app.name=spark -Dspark.executor.extraJavaOptions=-XX:+HeapDumpOnOutOfMemoryError -XX:PrintGCDetails"
bash-4.3# echo $SPARK_DRIVER_JAVA_OPTS | tr ' ' '\n' | readarray -t TEMP_OPTS
bash-4.3# echo ${TEMP_OPTS[@]}

(the blank line at the end is the output of echo which shows that the array is empty)

I've also encountered issues related to space characters in conf options when I set custom environments. So this issue should be solved thoughtfully.
During my test,

If one set value with space in spark.app.name, the error happens before launching any container. I havn't time to debug it.
If one set value with space("my value1") in spark.executorEnv.key1, the error happens with "Could not find or load main class value1".
The root cause is the wrong final java command line was constructed like below:

java ... -Dspark.executorEnv.key1=my value1 -D...

value1 was parsed by java as a main class name so failed to run.

@mccheah yeah the pipe thing, I guess instead use the temp file.  (still weird).
It seems like there are multiple layers of space problems here.  There's just the space between arguments/values, and then there's also spaces in values.


"sharewithmailuser" is "sharedwithmailuser"

The Get-Recipient page says "sharedwithmailuser" and when I actually run the command, it says "sharedwithmailuser". This page says "sharedwithmailuser", which is incorrect.
https://learn.microsoft.com/powershell/module/exchange/get-recipient#-recipienttypedetails

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

ID: 73d6ce18-0ec7-7509-4b9c-f8bb49d93e42
Version Independent ID: f87de628-1dd2-d169-ab30-57c117ae3d0b
Content: Get-User (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Get-User.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@teraco, fixed.
In the future, you too can make changes like this directly; no need to open an issue first. Instructions are here.
Thanks for the feedback.
Closing


Commits that have been forced overwritten remain on dashboard/activity

Gitea version (or commit ref): master
Git version: n/a
Operating system: n/a
Database (use [x]): n/a

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[X] Yes
[ ] No
[ ] Not relevant

Log gist: n/a

Description
Pushing a commit then overwriting that commit and force pushing results in both the overwritten and the new commit showing up in the dashboard/activity.
i.e.
git init .
echo "wrong commit" > file
git add file
git commit -m "wrong commit"
echo "right commit" > file
git add file
git commit --amend -m "right commit"
git push -f

Instead of just the expected right commit appearing in the dashboard/activity (since the old one has been force overwritten), both commits appear. Reproduced here: https://try.gitea.io/mqudsi/test2
Screenshot of incorrect result:

Is there anything I could do to fix this? If someone give me some guidance, I could work it out. Thanks.

I don't think there's a simple way of "fixing" this. I'm not even sure it's a bug. Since the dashboard should notify of new events, why should those events disappear if they actually happened?
If you're force pushing to e.g. cover some mistake, I understand that you may not want others to see such mistake. But IMHO there is plenty of other uses for force pushing that it would be incorrect to hide.
Also, you must take into account that other users might have enabled e-mail notifications. Those e-mails are out there and can't be claimed back. 😁

@guillep2k I think the problem is if you force-push to a branch, the hidden commit should not be displayed on dashboard. It should be a bug of calculation of the commits affected when force pushing.
He didn't mean deleted the old events.

But "not be displayed" and "delete the event" are equivalent from the UI point of view. I understand that if the commit was overwritten the link becomes invalid, but the entry should still show up in the dashboard. Perhaps something like:

After @guillep2k comment, it looks to me that he is right. Effectively, you pushed those commits, they must show up in the dashboard. My concern was that it was wrong, not to hide some mistake, but if you changed your history, you changed commits, if you changed commits, they should show up. Actually, this is a hint that something may have gone wrong.
I'm withdrawing my complaint... 😆
Thanks!

Is there a workaround to remove specific commits from the Dashboard? I accidentally pushed a file with passwords, but remains on the Dashboard after rewriting the git history... However, the Git log is correct.
Thanks!

Currently, I think the workaround is to delete the comment from database directly.

Thanks @lunny. I've deleted the entries manually from table actions and is no longer displayed at the Dashboard.
I ran the Garbage collect all repositories in the Site Administration, but the SHA1 link is still accessible. Is it possible to delete this as well?

The default garbage collect is 1 week ago unused commits You can change the default garbage collect time. See https://docs.gitea.io/en-us/config-cheat-sheet/#git-git GC_Args

@lunny Thanks for pointing to the documentation. That's very useful.
However, I don't understand why the manual "Garbage collect all repositories" did not work:

After running it, the commits are still available. I have to wait a few days and then I know if there is a difference with the Cron garbage collection.

indeed this issue also troubles me
at least I also believe that force push should no longer be displayed on the dashboard

You can understand that the content in the dashboard is the history or the log, which is stored in action.Content, and action.Content is not updated when the force push takes place.


IsOfType of ObjectType is not inferred

If no IsOfType function was declared we should have a default IsOfType function that tries to check if a resolver result os of its type.
Write test first.

This issue was fixed as part of the integration test bugfixes.


cleanup: *

Hey there, nice bot!
Did some cleanup stuff around the code after noticing some extraneous code that the lib already takes care of.
If you have any suggestions for the lib feel free to tell me!

hey, cool. I'll look at this shortly.
when I wrote it, I had trouble importing the RestManager class for a type error I couldn't determine. but the library was still in very active development as I was writing, so that probably got fixed now. and yeah, I probably should have used the parsed message helper.. @w@

everything looking good, thankyou!


luci-app-ddns: Fix check for musl library

Checking whether or not the second argument is actually used by 'nslookup' should not use a valid IP address. When compiled with the 'musl' library, this argument is ignored (no matter what it is). But when it is used, using '0' will result in Busybox nslookup to attempt to query the IP_ANYADDR '<IP_ADDRESS>'. This may typically take up to 15 seconds to timeout. It is better to provide an illegal argument, which will immediately return with an error.
Signed-off-by: Arjen M de Korte<EMAIL_ADDRESS>

And never mind. This change will only be able to detect whether or not Busybox nslookup is used, not if it is using the musl library (where the SERVER argument is ignored) or another one (where it may be honored or not).
I'm starting to doubt if it is possible to autodetect this, without having to resort to long timeouts for libraries that do support this.


Help Downloading videos you need to be logged in for on YT

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

WARNING
All invalid issues will be rejected!!

Before going further

If your problem is a bug with youtube-dl or a request for new site support please report it here

Make sure you are using the latest yt-dlg version (Click the Settings icon and then About to view the current version)

Make sure you are using the latest youtube-dl version (Click the Settings icon and then Update to update to the latest youtube-dl version)

Make sure you searched the bugtracker for similar issues including closed ones

Make sure to read the FAQs file

[ ] I think my problem is NOT with youtube-dl
[x ] I've verified and i assure that I'm running yt-dlg 1.X.Y
[x ] I assure that i am using the latest version of youtube-dl
[x ] Searched bugtracker
[x ] I've read the FAQs file

What is the purpose of your issue?

[ ] Bug report
[ ] Feature request (request for a new functionality)
[x ] Question
[ ] Other

Please remove any sections between (---) if they are not related to your issue

What is the expected behaviour ?
download YT video
What happens instead ?
error

I am having trouble downloading age-restricted videos on YT. How to do it?
Entering my credentials in the advanced tab does not work
also tried "--cookies /path" in  extra

Can share the URL to reproduce the bug ?


iPhone 6 Width is bigger than 320px

Hi,
on the new iPhones the width of the menu doesn't suit.

Use https://github.com/skywinder/ActionSheetPicker-3.0 instead.


feat(vault-keychain): add support for vault transit secret engine

Add support for following method to PluginKeychainVault class

transitSign : sign digest using private key stored in vault server
transitNewKey : create a new key
transitGetPub : return JWK formatted public key
transitRotateKey : rotate private key

For now it support EC (of size 256 , 384 , 521 ) only
Signed-off-by: Pritam Singh<EMAIL_ADDRESS>

Codecov Report

Merging #1243 (9936c46) into main (dcf4c46) will decrease coverage by 0.10%.
The diff coverage is 58.20%.

:exclamation: Current head 9936c46 differs from pull request most recent head bb31bb7. Consider uploading reports for the commit bb31bb7 to get more accurate results

@@            Coverage Diff             @@
##             main    #1243      +/-   ##
==========================================
- Coverage   71.95%   71.84%   -0.11%
==========================================
  Files         270      270
  Lines        9438     9501      +63
  Branches     1112     1127      +15
==========================================
+ Hits         6791     6826      +35
- Misses       2075     2089      +14
- Partials      572      586      +14

Impacted Files
Coverage Δ

...vault/src/main/typescript/plugin-keychain-vault.ts
74.21% <57.57%> (-12.39%)
:arrow_down:

...n-keychain-vault/src/main/typescript/public-api.ts
87.50% <100.00%> (+1.78%)
:arrow_up:

Continue to review full report at Codecov.

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

@petermetz I played with crypto package in my machine for both version v12 and v15 and it seems this package has version dependent methods . so instead I tried to  use jsrsasign package which is very rich cryptography package in js . And also I would be using jsrsasign package for generating CSR and encoding signature into a fabric understandable format in order to support vault to sign fabric messages .
But here is error message which got while running yarn run configure
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/main/typescript/plugin-keychain-vault.ts:32:19 - Unknown word (KJUR)
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/main/typescript/plugin-keychain-vault.ts:32:32 - Unknown word (jsrsasign)
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/main/typescript/plugin-keychain-vault.ts:395:49 - Unknown word (KJUR)
315/540 ./packages/cactus-plugin-keychain-vault/src/test/typescript/integration/plugin-keychain-vault.test.ts 35.71ms X
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/test/typescript/integration/plugin-keychain-vault.test.ts:34:19 - Unknown word (KJUR)
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/test/typescript/integration/plugin-keychain-vault.test.ts:34:32 - Unknown word (jsrsasign)
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/test/typescript/integration/plugin-keychain-vault.test.ts:409:26 - Unknown word (KJUR)
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/test/typescript/integration/plugin-keychain-vault.test.ts:424:26 - Unknown word (KJUR)
/home/pritam/go/src/github.com/hyperledger/cactus/packages/cactus-plugin-keychain-vault/src/test/typescript/integration/plugin-keychain-vault.test.ts:432:26 - Unknown word (KJUR)

these error are from CSpell but name KJUR is a namespace inside jsrsasign package . How to solve this ??

@petermetz I am closing this and going to implement vault transit client inside fabric connector where I need. I think Having this logic inside keychain won't be adding any value to cactus.

@Zzocker Fine by me, it is hard to know in advance where a piece of code would fit best so no harm done if you replace it elsewhere.
Also note: You don't need to close this and open a new one, you could just force push your local changes with the refactor later, the PR can stay in place (and maybe rename it to feat(connector-fabric): add support for vault transit secret engine )

@petermetz feat(connector-fabric): add support for vault transit secret engine incoming in two days :slightly_smiling_face:


Potentially missing data

If you take a look at the all items index "A" - we have fewer items showing on the new site.
https://v3.datadictionary.nhs.uk/retired/web_site_content/pages/domains/a__and__e_arrival_mode_de.asp?shownav=1
does not exist on new site.
We don't seem to have access to the "domains" sub-folder (totally new folder to me), so have never been processing it...
Question is - do we need to rectify this?

This is on the new site: https://datadictionary.nhs.uk/data_elements/retired/a_and_e_arrival_mode.html
Domains is a folder under the retired web site content pages folder structure. Have none of those folders been processed? I've checked a couple on V3+ and can't see them

V3 site all items has:
Item Name | Item Type |
-- | -- | --
A and E ARRIVAL MODE (retired) | Data Element |
A and E ARRIVAL MODE (retired) | Domain
New site doesn't have the domain item

I don't think we get these sub folders for processing - have checked the June 20 unmodified txaclass file zip - the folder isn't there

I don't have a more recent unmodifed set of txa files - i.e. pre pipeline processing
@likin1 when you get the release files for processing - do you have any content in a web_site_content/pages/domains folder?
Is the domains folder itself present?

Decision reached on collab call 22/10/2020 - NO decided not to bring these into mauro.


debug output is difficult to visually scan

I am trying to move from sw-toolbox to workbox and the logging output seems much less practical.
sw-toolbox logging is very clear and easy to visually scan for errors :
[sw-toolbox] Strategy: cache first [https://cdnjs.cloudflare.com/...]
[sw-toolbox] Strategy: network first [https://www.google.com/...]
...

Workbox is completely opaque.  I have a ton of assets and when I tweak the cache type of one and want to confirm it has taken effect, I am finding it impossible to know which line represents my asset without clicking into each one:
[Info] The router found a matching route.
[Info] The router found a matching route.
[Info] The router found a matching route.
...

Would it be possible to make the high level debug message contain the strategy/asset so people can easily scan (and drill into it for more info only if necessary)?

We are working on this in the next version: https://github.com/GoogleChrome/workbox/pull/902
Let me know if that would be better for you or if there are still concerns.

Ok, it looks like asset name was added...having asset name solves one critical need...being able to efficiently verify that a specific regex change has correctly taken effect.
But it looks like strategy is not yet there.  I think it is still vital having both asset and strategy printed without drilling in because I can't tell you how many times I have turned on sw-toolbox debug just to scan the log feeling like everything is fine but just wanting to make sure everything that should be "cacheFirst" is "cacheFirst" and said something like "oh $#!%... should/should not be cacheFirst".

The strategy has just been added in a PR that landed to today.
See them here: https://workbox-demos.firebaseapp.com/demo/workbox-runtime-caching/

Side note on debug...as a new user I found it a bit hard to know how to actually turn on debugging.  In sw-toolbox i just say "debug=true".  In workbox...I see there is a file called "prod" so I assume switching to the "dev" file is necessary?  And after that, is there logging levels and flags I should know about?
Logging didn't seem to go on right away but I run my server with "www.OneOfManyTestDomain.com" and feel like perhaps some logging was tied to "localhost" which made it more painful for me.  Anyway, just a quick request to consider:
having a doc section on how to enable debugging, making sure logging isn't tied to "localhost" so I can turn it on/off with commandline parameters on any domain (file switch is ok, I just want to be sure I can get debug logs on more than just localhost).
Cant wait for v3 to be live!

Once again v3 will have a fix for this - if you check out the source code for v3 you'll see debug: true. This will pull in a debug build for each module (it's true by default for localhost).
I have doc outline here for configuring workbox (Needs some outdating with changes landed today but nothing too drastic): https://docs.google.com/document/d/1Oh6vRCE-AKuIlcYaxxwa0uoAFO2XjX-NQN3Ee-7MDGI/edit?usp=sharing
Comments welcome on the docs. Gonna close this issue for now - please feel free to raise further issue if there is moore.


Missing UI Flows

Task Description

Vault Recreation:

The user should be able to recreate the vault from within the app through descriptors and vault config.
Users should have an option while adding SS/IKS if they are trying to create a new one or recover an old one. Through this, they can recreate the app for assisted keys.
For Every other signer, there is no change

Buy Bitcoin:

User should have an option for In which wallet they want to receive the funds.

2.https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/63cecf4f-972b-49db-9cd7-c935e6524609/

updated add wallet flow

https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/e8f62e6f-d5a4-43fc-8420-bc54e1236a56/

Verified this issue on keeper app v 1.1.11(295)


fix(kernel_crawler): fixed rpi debian kernel kernelrelease.

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area crawler
What this PR does / why we need it:
Avoid 1: at start of kernelrelease for rpi kernels.
Previoius output:
{
  "debian": [
    {
      "kernelversion": "1",
      "kernelrelease": "1:6.1.21-1+rpt1-rpi-arm64",
      "target": "debian",
      "headers": [
        "http://archive.raspberrypi.com/debian/pool/main/l/linux/linux-kbuild-6.1_6.1.63-1+rpt1_arm64.deb",
        "http://archive.raspberrypi.com/debian/pool/main/l/linux/linux-headers-6.1.0-rpi1-rpi-v8_6.1.21-1+rpt1_arm64.deb",
        "http://archive.raspberrypi.com/debian/pool/main/l/linux/linux-headers-6.1.0-rpi1-common-rpi_6.1.21-1+rpt1_all.deb"
      ]
    },

New output:
{
  "debian": [
    {
      "kernelversion": "1",
      "kernelrelease": "6.1.21-1+rpt1-rpi-arm64",
      "target": "debian",
      "headers": [
        "http://archive.raspberrypi.com/debian/pool/main/l/linux/linux-kbuild-6.1_6.1.63-1+rpt1_arm64.deb",
        "http://archive.raspberrypi.com/debian/pool/main/l/linux/linux-headers-6.1.0-rpi1-common-rpi_6.1.21-1+rpt1_all.deb",
        "http://archive.raspberrypi.com/debian/pool/main/l/linux/linux-headers-6.1.0-rpi1-rpi-v8_6.1.21-1+rpt1_arm64.deb"
      ]
    },

Didn't notice the issue at first glance friday afternoon :)
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/cc @leogr


security/acme-client: ProxmoxVE automation didn't start

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
os-acme-client 4.0 ProxmoxVE automation didn't start.
OPNsense\AcmeClient\LeAutomation\runAcme() stops at this line
To Reproduce
Steps to reproduce the behavior:

Setup automation with Run Command "Upload certificate to Proxmox VE".
Add newly created automation to the certificate that already issued.
"Run automations" for the certificate in certificate list.

Expected behavior
ACME deploy hook shoud be start, and store some logs in "ACME Log".
Relevant log files
Systemlog
2024-02-03T23:56:18 | opnsense | AcmeClient: running acme.sh command: /usr/local/sbin/acme.sh --deploy --syslog 7 --debug 3 --server 'https://acme.home.arpa/acme/acme/directory' --home '/var/etc/acme-client/home' --cert-home '/var/etc/acme-client/cert-home/65bda0274ad018.06230787' --certpath '/var/etc/acme-client/certs/65bda0274ad018.06230787/cert.pem' --keypath '/var/etc/acme-client/keys/65bda0274ad018.06230787/private.key' --capath '/var/etc/acme-client/certs/65bda0274ad018.06230787/chain.pem' --fullchainpath '/var/etc/acme-client/certs/65bda0274ad018.06230787/fullchain.pem' --domain 'holodeck.home.arpa' --deploy-hook proxmoxve
2024-02-03T23:56:18 | opnsense | AcmeClient: running automation (acme.sh): xxx
2024-02-03T23:56:18 | opnsense | AcmeClient: running automations for certificate: holodeck.home.arpa

ACME Log
nothing
Environment
OPNsense 24.1(amd64).
os-acme-client 4.0

Please switch Log Level to "debug 3" in Services->ACME Client->Settings and try again. The ACME Log really should not be empty. (Note that debug logging causes the log to look somewhat scrambled in the GUI, this is expected.)

OK, I think I found the bug. You should have an error in crash reporter:
[07-Feb-2024 10:37:11 Europe/Berlin] PHP Fatal error:  Uncaught TypeError: proc_open(): Argument #2 ($descriptor_spec) must be of type array, null given in /usr/local/opnsense/mvc/app/library/OPNsense/AcmeClient/LeAutomation/Base.php:133

You may apply this patch and try again:
opnsense-patch -c plugins 84644d4a

Can confirm patch is working!
Automations to deploy to Synology NAS now working again!

Thanks for testing, I'll close this issue. This bugfix will be available in the upcoming os-acme-client 4.1.


[WIP] Version space learning

Adds version space learning. Instead of evaluating and storing the entire hypothesis space, I have utilised the partial order property of hypotheses in the hypothesis space. Hence, I have implemented version spaces by storing only two sets of hypotheses: a most general boundary set (the G-set) and a most specific boundary set (the S-set).

@norvig Please have a look.


Nix 2.24 upgrade

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Mic92, just an idea (not a request / completely to your consideration),
Maybe target the current staging-next cycle to test before a broader rollout? (It isn't many rebuilds and could go directly to the main branch but due to the criticallity of nix for NixOS might be preferable to reduce the blast radius a bit)

Mic92, just an idea (not a request / completely to your consideration),
Maybe target the current staging-next cycle to test before a broader rollout? (It isn't many rebuilds and could go directly to the main branch but due to the criticallity of nix for NixOS might be preferable to reduce the blast radius a bit)

Sounds good. Also we can than fix instead of revert.

I found some issue with the nix-upgrade service. We need to patch switch-to-configuration to ignore /nix/store transient units.

I found some issue with the nix-upgrade test. We need to patch switch-to-configuration to ignore /nix/store transient units.

Also I think this is issue would also happen with the current nix version as systemd creates a nix-store.mount unit that it than tries to unmount.

So the nix-update tests revealed a regression in switch-to-configuration-ng, which needs to be fixed nevertheless as it will break any nix update: https://github.com/NixOS/nixpkgs/pull/343709

Result of nixpkgs-review pr 343655 run on x86_64-linux 1

1 package blacklisted:

nixos-install-tools

2 packages failed to build:

nix-plugin-pijul
nix-plugins

93 packages built:

appvm
attic-client
attic-server
bundix
cabal2nix
cached-nix-shell
common-updater-scripts
crate2nix
crystal2nix
disko
dub-to-nix
dydisnix
fusionInventory
libnixxml
lua51Packages.luarocks-nix
lua52Packages.luarocks-nix
lua53Packages.luarocks-nix
lua54Packages.luarocks-nix
luajitPackages.luarocks-nix
luarocks-packages-updater
nim_lk
niv (niv.bin ,niv.data)
nix
nix-binary-cache
nix-bundle
nix-direnv
nix-du
nix-index
nix-init
nix-pin
nix-prefetch
nix-prefetch-bzr
nix-prefetch-cvs
nix-prefetch-docker
nix-prefetch-git
nix-prefetch-hg
nix-prefetch-scripts
nix-prefetch-svn
nix-required-mounts
nix-required-mounts.dist
nix-template
nix-update
nix-update-source
nix-update-source.dist
nix-update.dist
nix-visualize
nix-visualize.dist
nix.debug
nix.dev
nix.doc
nix.man
nixStatic
nixStatic.dev
nixStatic.doc
nixStatic.man
nixci
nixos-anywhere
nixos-generators
nixos-install
nixos-rebuild
nixos-shell
nixpkgs-hammering
nixpkgs-manual
nixpkgs-review
nixpkgs-review.dist
nixtract
node2nix
npins
nuget-to-nix
nurl
nvfetcher
outline
prefetch-yarn-deps
python311Packages.nix-kernel
python311Packages.nix-kernel.dist
python312Packages.nix-kernel
python312Packages.nix-kernel.dist
sbomnix
sbomnix.dist
sonarr
swiftPackages.swiftpm2nix
terranix
typescript-language-server
update-nix-fetchgit
update-python-libraries
vimPluginsUpdater
vulnix
vulnix.dist
vulnix.doc
vulnix.man
wp4nix
yarn2nix
zon2nix

@vcunat gave me green light to merge this to staging-next.

829636f4173acb044ee430037762e2b30a03cb1e broke the test eval in staging-next as:
$ nix build --no-link -f. nix-plugin-pijul.tests
error:
       … while calling the 'derivationStrict' builtin
         at <nix/derivation-internal.nix>:34:12:
           33|
           34|   strict = derivationStrict drvAttrs;
             |            ^
           35|

       … while evaluating derivation 'localRepoCheck-nix-2.25.0pre20240920_ca3fc169'
         whose name attribute is located at pkgs/stdenv/generic/make-derivation.nix:336:7

       … while evaluating attribute 'buildCommand' of derivation 'localRepoCheck-nix-2.25.0pre20240920_ca3fc169'
         at pkgs/build-support/trivial-builders/default.nix:59:17:
           58|         enableParallelBuilding = true;
           59|         inherit buildCommand name;
             |                 ^
           60|         passAsFile = [ "buildCommand" ]

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error: function 'anonymous lambda' called with unexpected argument 'nix'
       at pkgs/by-name/ni/nix-plugin-pijul/package.nix:1:1:
            1| {
             | ^
            2|   lib,
       Did you mean lib?

Proposed a simple fix as:

https://github.com/NixOS/nixpkgs/pull/346689


samung device keyboard separator decimal not display

[] iOS
[x] Android
[] UWP
[] NET Standard 2.0

Version of OS(s) listed above with issue (if android - please include version of Xamarin.Android.Support.* you are using)
Android 7.1 Android 8.0
Version of ACR UserDialogs Library
7.0.4
when use input dialog NumberDecimal in samsung device not display separator decimal with italian language.
I resolve change txt paramter from textview to edittext in your PromptBuilder and add txt.SetRawInputType.
Please can you confirm my solution and update your project?

Raw input type is already set.  Please send a PR if you want me to look at something

as you can see not display any separator (comma or dot)

I get there is a bug.  What I’m trying to understand is what is being requested which is why I’ve requested a PR

io ti chiedo se è possibile risolvere il bug direttamente sulla libreria oppure se il problema è generato dal pacchetto nuget. aggiungendo il tuo codice sorgente al progetto sembra funzionare correttamente. scusami ma sono nuovo a non so cosa è un PR

I'm sorry - I still don't know what you're asking me to fix.  If you don't know how to PR, simply copy/paste it into the chat

Sorry, i hope to explain better.
i did two tests:

install your nuget package and i have problem with separator.
download you source code..recompile and add project reference to my code and seems to work.
My question...is possible nuget package problem??
Your work is beautiful and I would like to use your package not your source code

I don’t know.  I don’t have the problem and I haven’t tested it myself.  You have to tell me

I came across the same issue on an Samsung Galaxy Tab A (2016) with Android 8.1.0. It seems to be related to Samsung devices using the Samsung keyboard as soft keyboard. Same issue as described in #418.
To fix it you have to modify the SetInputType method in src/Acr.UserDialogs/Platforms/Android/Builders/PromptBuilder.cs. Setting the KeyListener before setting the InputType seems to fix the issue. Source: https://stackoverflow.com/questions/37721694/android-keyboard-with-number-and-decimal-for-lg-g4/40507863#40507863
PromptBuilder.cs.patch.txt
So put line 170
txt.KeyListener = DigitsKeyListener.GetInstance($"1234567890{CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator}{CultureInfo.CurrentCulture.NumberFormat.NegativeSign}");
before line 168
txt.InputType = InputTypes.ClassNumber | InputTypes.NumberFlagDecimal | InputTypes.NumberFlagSigned;

thanks for your suggest...i resolved long time ago with this fix but I didn't want to modify your source code.
great job your library

I have the same problem
With a Samsung S10, italian keyboard.
I show to the user a PromptAsync with InputType = InputType.DecimalNumber.
In Samsung keyboard I can see the '.' but I cant see the ','
If I tap the '.' the . is not added to the entry (correct) but I should see the ',' key (as in other phones)
Thanks


Can I collaborate?

I would like to know if it is possible to add more examples in this template. Or why not collaborate in solving or adding features to the Primereact library.

Absolutely.   We just recently added Vite and Remix Run examples but more examples are welcome!


Backticks are exported as double backticks when exporting resaults from query as SQL

Description
When trying to export the data that came out from a query result single backticks are converted to double.
Here is the data I got from the query:

And here is how it's generated after I click Export to SQL INSERT statements:

DBeaver Version
Version <IP_ADDRESS>405051837
Operating System
No response
Database and driver
MySQL 8
Steps to reproduce
Just select any column that has a backtick (`) and try to export it as SQL statement and the result will be double backticks (``)
Additional context
No response

https://github.com/dbeaver/dbeaver/pull/34167

Thank you for bug report.


Maximum active Virtual Machines issue

Describe the issue
In my UTM setup I have multiple macOS Guest VMs created for different purposes. Obviously with Apple Silicon Virtualization we can only have 2 VMs active simultaneously.  Occasionally, if I have had 2 VMs running, and I shut one of them down, then try to start a different one, I get presented the following error

I do not know if this an issue with UTM not cleaning up certain resources when a VM is stopped and closed, or if it's an OS framework level bug on Apple's side?
The only way I seem able to recover from this is to logout of the machine and log back in again. Quitting and re-opening UTM itself sometimes resolves the issue, but not every time.  This implies that there may be some process running as my logged in user that I could also possibly kill to rectify the problem but so far I've been unable to identify if that's the case.
Configuration

UTM Version: 3.2.4 (58)
OS Version: 12.5
Intel or Apple Silicon? Apple Silicon

Debug log
Debug log location instructions seem only relevant for QEMU based VMs, not Apple Virtualization framework - I cannot see an option anywhere in the settings for these VMs where I enable or locate the debug logs for them.

I'm seeing this same issue, even just with the one VM I have. After a few times of starting and stopping it, it get this error.

UTM Version: 4.0.9 (68)
OS Version: 13.0.1
Intel or Apple Silicon? Apple Silicon

I have the same issue.

Same thing happens to me.
UTM Version: 4.2.2
OS Version: 13.3
Intel or Apple Silicon? Apple Silicon

I have one MacOS VM and I shutdown UTM when on battery.  I end up having to reboot to get the VM started again.

Me and my colleges are also having this issue.
I feel that it happens more often lately (pretty much every time I stop one VM I need to at least logout to start different one).

@F-WRunTime I haven't found any solution as yet, I've been using UTM less frequently due to this issue and have instead been using the CLI VM app called tart as I don't seem to run in to this problem due to it.  I suspect maybe this is happening due to UTM pre-creating the VZVirtualMachine* type objects to represent the VM and not destroying them when a VM stops, which is then somehow not releasing a resource lock with the Virtualization framework
However, I don't have any proof of that and it's just a best-guess on my part.  I know that other GUI based Virtualization apps such as VirtualBuddy also exhibit this same behaviour, so it could be that it's a bug for Apple to fix (or at least, for Apple to improve their documentation and state which object instances need to be destroyed when the VM has stopped)


HADOOP-16536. Backport HADOOP-15273 to branch-2

HADOOP-16536
HADOOP-15273

Can you raise one PR for HADOOP-15273 after HADOOP-16158 (or the other way around)?  I'd rather not to squash these commits into one.

Sure, I'll split it to separate ones

Update this PR to backport HADOOP-15273 first.
Once it's merged, I'll submit another one for HADOOP-16158.


[HOLD PR #39757] Combine removeFromRoom and removeFromGroupChat. Implement offline behavior.

Coming from this comment https://github.com/Expensify/App/pull/39757/files#r1558787455
We had not initially planned to develop any specific offline behavior for Group Chats when adding or removing members as there are very few cases where it can even fail (if any), but noticed that some minimal offline behavior was implemented for rooms quite recently in this issue.
In light of that recent change - let's do the following to make the "rooms" behavior and "group chats" behavior more standardized....

Refactor removeFromRoom() and removeFromGroupChat() so removeFromGroupChat() will call removeFromRoom() instead of having it's own method.
Let's make sure that pendingChatMembers works for both methods.
And let's test the offline behavior so that they are consistent in both places.
Let's also ensure that the inviting people to Group chats will correctly display the pendingChatMembers on the Members page for the offline case.

Ended up implementing this in https://github.com/Expensify/App/pull/39757


generating bad qiime mapping files

In the past we allowed to have barcodesequence, barcode and BarcodeSequence in the mapping file and all of them were translated to BarcodeSequence when we created the qiime mapping files. Now we only accept barcode, which means that old studies that have barcodesequence do not have a valid BarcodeSequence column in the qiime mapping file.
The solution is to update all prep templates but I believe this should be done via a patch. Perhaps something like: if the preptemplate has barcodesequence or BarcodeSequence and values are different than XXQIITAXX we should rename to barcode and then recreate the file prep template and qiime mapping file.

After discussing we will create a local patch to fix the templates and mapping files in the main repository.

This was fixed in qiita.microbio.me


Exception in HighlightElements

I was checking out issue #87 and tried out the code there, but it fails with an exception raised:
File "/Users/runxel/dev/tapir-archicad-automation/python_package/src/tapir_py/core.py", line 270, in HighlightElements
    raise Exception("Script Error")
Exception: Script Error

The error message is not helpful, but I think it's not in the Python package (used the latest code cloned from here).

The input parameters of HighlightElements command was changed since issue #87 was created.
So the attached code in #87 does not work anymore, it results error.
'python_package/src/tapir_py/core.py' must be updated accordingly => its a bug in python package
Note, using the updated command we can use multiple colors for elements, so elements can be highlighted with different colors.
Please find working example scripts at https://github.com/ENZYME-APD/tapir-archicad-automation/blob/main/archicad-addon/Examples
An example script for HighlightElements command:
https://github.com/ENZYME-APD/tapir-archicad-automation/blob/main/archicad-addon/Examples/highlight_elements.py

Thank you for clarifying what I already suspected.
However the example script is not working for me either:
Command: HighlightElements
Parameters:
{
    "elements": [],
    "highlightedColors": [],
    "wireframe3D": true,
    "nonHighlightedColor": [
        0,
        0,
        255,
        128
    ]
}
Error:
{
    "code": 4010,
    "message": "Archicad does not have the registered Add-On command with the name : TapirCommand.HighlightElements"
}
Response:
null
Traceback (most recent call last):
  File "/Users/runxel/dev/tapir-archicad-automation/archicad-addon/Examples/highlight_elements.py", line 15, in <module>
    response = aclib.RunTapirCommand (commandName, commandParameters)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/runxel/dev/tapir-archicad-automation/archicad-addon/Examples/aclib/__init__.py", line 45, in RunTapirCommand
    if 'error' in result:
       ^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable

Uhhh, what now? 😅

The error message is clear, Archicad does not find TapirCommand.HighlightElements command.
Please make sure Tapir Add-On is loaded, you can check it in Add-On Manager (find it in Options menu).
The documentation says HighlightElements command is available from version 1.0.3.

Okay maybe I was not clear enough, sorry Tibor.
What I meant was: The error message does not make sense since Tapir is in fact installed. So now you understand my confusion.
Somewhat else is in play here, but I can't see what.

Only one thing comes to my mind: is it possible that multiple Archicad instances are running on your computer? Example scripts will find the one that is started first.


Upgrade AMQ broker CR

Current CR for AMQ Broker (https://github.com/karol-brejna-i/retailstoreofthefuture/blob/main/amq-broker/cr/broker.yaml) uses old API version (broker.amq.io/v2alpha2) and is fixed on an old Artemis version.
The newest version of the AMQ Operator (ATTOW: 7.10.x) brings ActiveMQ Artemis 2.21.0 with many performance and scalability improvements, and -- what is even more important in our case -- MQTT v5 support.
Update the descriptor to use the latest Artemis version.

https://github.com/tejones/retailstoreofthefuture/commit/1df7f603b8202c5650905b584c6c9de352ee8723 should address this

Fixed with #9


Create rule for minimum usage of Java17 and JakartaEE 9+

According to the migration guide:

Spring Framework 6.0 raises the minimum requirements to Java 17+ and Jakarta EE 9+.

This can be achieved:

By looking for JakartaEE/JavaEE related dependencies lower than JakartaEE 9
By peeking at the pom.xml/gradle.build if available and checking if there is any information about the Java version

In the rule description and message, try to be as verbose and clear as possible. Code examples are always welcome and useful. Feel free to use markdown syntax for these.

We can assume here that we are migrating from Spring Framework 5 to 6, therefore we can look for the java.version property to check the version of Java being used. The property maven.compiler.source can also be checked in case it exists.
Knowing the Jakarta version being used can be trickier. We need to take a look at potential JavaEE/JakartaEE versions being imported. If any javax version is found, a rule should be triggered.


Sending exceptions to Sentry

Hello,
First of all thank you very much for the wonderful library.
How do connect django-RQ to sentry. Nothing seems to work. I've configured sentry + logger under my django app. Everything works flawlessly when outside of RQ context, configured according to  sentry's documentation, but the moment I move to jobs executed in RQ nothing works.
I was naive and looked at the code of rq , hoping that just setting SENTRY_DSN in settings.py will work, I guess not.
Used https://docs.sentry.io/clients/python/integrations/django/ for configuration
What am I missing, what should I do ?

Created a pull request to fix the exception handling, see https://github.com/ui/django-rq/pull/253
but still loggin does not work.

It always worked for me when using sync transport like this:
    'dsn'<EMAIL_ADDRESS>    'auto_log_stacks': True,
}

Fixed in https://github.com/ui/django-rq/pull/253


[GHSA-ghg6-32f9-2jp7] XXE in PHPSpreadsheet encoding is returned

Updates

Affected products

Comments
Updates have been backported to v.1.29.1
See https://github.com/PHPOffice/PhpSpreadsheet/releases/tag/1.29.1

Hi there @oleibman! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


Delinquency buckets management for Loans

Description
As a user would like to be set Delinquency overdue ranges in order to classify disbursed loans for accounting
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@josehernandezfintecheandomx Please resolve the conflicts!

LGTM


Avoid osx-arm64 macos-14 runners in CI

macos-14 is osx-arm64 arch and ass far as I understand the current release of distopia doesn't support this

Thanks @IAlibay, no we don't support ARM at all sadly, with no easy fix other thandistopia2 (coming).

Thanks @IAlibay, no we don't support ARM at all sadly, with no easy fix other thandistopia2 (coming).


Pillar 3: Outreach and formation of Institutions

Forming a network of institutions EU-SSI? FuturSSI?
Offer education to existing RSEs as well as to scientists
Risks, Opportunities and Challenges for education and the institutions doing that education.

We could even think of a collaborative Master's program (across universities or online) similar to https://eumaster4hpc.uni.lu/
This would originate from such a consortium.

Use the existing structures like the masters as examples for outreach?

There are a couple of things we can think of:
multiple institutions:

Assuming we have institutions, which kinds of network can they form?
Assuming these networks exist, what other stakeholders will profit
Acreditation structures?

single institution:

What policy needs to be implemented at a university(Leibniz, MPG, ...) ?

What policies need to be present at a national level, such that these institutions will be created?

How do domains profit from having a connection to such an institution?

What are the benefits of these institutions for an individual?

What are the benefits for a university?

Are there policy changes in the funding structure necessary, such that Graduates can access their resources?

multiple institutions:

Assuming we have institutions, which kinds of network can they form?
Assuming these networks exist, what other stakeholders will profit
Accreditation structures?

single institution:

What policy/incentives needs to be implemented at a university(Leibniz, MPG, ...) ?

Pooling Resources in interest-driven networks
Networks enable interaction with industry and provide opportunities for students.

What policies/Incentive need to be present at a national level, such that these institutions will be created?

How do domains profit from having a connection to such an institution?

A teaching facility can jumpstart the transition to dgitalization. Imagine a humanities department who has no-one to start digitalization

What are the benefits of these institutions for an individual?

What are the benefits for a university?

proper students

Local RSEs could improve digital literacy in the domains
If there is a master, contribute to the master, e.g. as supervisors

Help Academics:

Graduate Programs

Maintainathons

Consultation programs

Benefits: less costs,  Reduce the average time of a PhD, Get more stuff done in the same time

Promotes Interdisciplinarity

Enables univeristies to distinguish themselves

Are there policy changes in the funding structure necessary, such that Graduates can access their resources?

Other important aspects:

institutions should aim for a mixture of profs and practitioners
maybe teacher is a good carreer aim for senior practitioners.


Simplify update docs task

Currently we have a lot of nested try..exception blocks.
We only need one, we are duplicating the same error handling.
Also, we are already do checks to avoid ending with undefined variables.
I refactored to run run_build after run_setup in the same try...exc block. The exception handling is the same, so it came be removed.

I'm guessing we added the two try..exc block because of https://github.com/rtfd/readthedocs.org/pull/5678 but now it's fixed.


Regarding odometry and teleop

Sorry i am quite new in ros and gazebo.
My questions are:
1.I know there is a odometry.cpp file but when i do "rostopic list" there is no "odometry" published, instead i check the cpp file, i saw there is few lines which brings to "pose/mobile" and "pose/world". Is it the same?
2.For the topic,  "pose/mobile" and "pose/world", i try to echo it but what i received was：
WARNING: no messages received and simulated time is active.
Is /clock being published?
Is it anything i did wrong?
3. There is an option to control the robot using another file namely "teleop_keyboard_omni3". I successfully controlled the robot using it, but there is an issue. The robot doesn't move normally. The be detail, when i pressed the control key, one of the (u, i, o, j, k, l, m, , ,.), there is a delay. If i press the key which allows the robot to rotate, not all the wheels rotate simultaneously. Is there anything i can do?
Thank you.

Hi there.

Sorry i am quite new in ros and gazebo.

No problem!

1.I know there is a odometry.cpp file but when i do "rostopic list" there is no "odometry" published, instead i check the cpp file, i saw there is few lines which brings to "pose/mobile" and "pose/world". Is it the same?

This is not a ROS/Gazebo thing. This is how I decided to implement this project. The idea is for the odometry node to read data from other sources, compute the robot's position and publish it to pose/mobile (the robot's pose relative to its coordinate frame) and pose/world (the robot's pose relative to the world's coordinate frame).
You need to decide which odometry strategy you'll use. You can read from the wheel velocities and apply kinematics to use dead reckoning, or you can read from an IMU sensor, or you can read from a camera pointing to a textured floor, or you can use GPS, etc.
I tried with dead reckoning at first by reading the wheel velocities that are provided by the sensor/wheel_velocity topic but it didn't work too well. Since this is not the focus of this project I decided to "cheat" and read from /gazebo/link_states, which outright tells the robot's pose. You can see in velocity controller.launch that I pass a parameter to odometry called pose_cheat that turns this on. Of course this only works in simulation.
When you decide which odometry strategy you want to use, you'll have to edit odometry.cpp to implement this strategy. I think that when I develop this project further I will remove the odometry node and let users implement their own in their own packages. All they will need to do will be to publish to the same pose/* topics, and they will be able to read from sensor/wheel_velocity just the same.

2.For the topic, "pose/mobile" and "pose/world", i try to echo it but what i received was：
WARNING: no messages received and simulated time is active.
Is /clock being published?
Is it anything i did wrong?

pose/mobile is not being published to when pose_cheat is enabled. This I still need to fix.
However, if you also can't echo pose/world, something else may be wrong. Check to see if you didn't also get the warning message Controller Spawner couldn't find the expected controller_manager ROS interface.. If you did, you can fix it by installing the packages listed by apt-cache search ros-<distro>-, where <distro> is the ROS distro you're using, such as indigo, jade, kinetic, lunar, melodic, etc.
If you didn't get this warning, paste here the terminal output after you launch the project.

There is an option to control the robot using another file namely "teleop_keyboard_omni3". I successfully controlled the robot using it, but there is an issue. The robot doesn't move normally. The be detail, when i pressed the control key, one of the (u, i, o, j, k, l, m, , ,.), there is a delay. If i press the key which allows the robot to rotate, not all the wheels rotate simultaneously. Is there anything i can do?

This you'll have to see with the author of teleop_keyboard_omni3, which is YugAjmera.

I hope I've been able to satisfy your doubts and I'll keep this issue open to see about that pose/world.
Thanks for taking an interest in my project!

Hi, Gui Ritter,
Sorry to get back to you after so long.
Hope you stay safe and healthy.
The answer provided helped me.
Thanks a lot.


Make mae-consumer-job container wait for Elasticsearch & Neo4j

I think Mae-consumer-job waiting for Neo4j is handled in #1628

Yes, thank you for that. Would you be interested in taking this and #1549 and send a PR? Should be fairly trivial.

Fixed by https://github.com/linkedin/datahub/pull/1687


Rust rewrite

This is like the mother of all PRs.

Ow yeah. Got to disable Now integration on PRs. Travis builds push a deployment for PRs anyway.


Is there support to automatically add Headers to .razor files?

Hello,
Is there a way to add headers to razor files (Blazor app)? I have the stylecop.json settings set as needed and it works great on .cs files, but it seems that rules are not being applied to .razor files type?

I don't believe this is currently supported.


layout.html v0.14.1 license

Hi,
Can you please specify which BSD applies on the file https://github.com/pallets/werkzeug/blob/0.14.1/examples/cupoftee/templates/layout.html ?
Is it the one from the LICENSE file or another ? Cause there is no version specified in the above file and from what I've noticed, there are other files, like https://github.com/pallets/werkzeug/blob/0.14.1/docs/werkzeugext.py which have an older bsd license.
Thank you

The same license as the rest of the project: BSD 3-Clause.


Error on messages webhook

When sending a mesage fails, the library fails to parse the content received on the wbehook, as now there is a new property "href" on the Error class which isn't yet defined:

I tired to open a new pull request but couldn't do it. In any case, the fix should be very easy.

Thank you @gbordoni  for reporting this bug. The fix is now available in version v0.3.1.


coredns-deployment.yaml is empty and installation fails

Hi, Great job to put this all together. I have an issue though.
Once the master is up & running, the K8s install start but one file is empty (it exists though :  /home/core/coredns-deployment.yaml).
This leads to an error : "error: no objects passed to create"
I run on Windows 10.
Thx

Fixed manually by running :
./plugins/dns/coredns/deploy.sh <IP_ADDRESS>/24 "cluster.local" ./plugins/dns/coredns/coredns.yaml.sed > ./temp/coredns-deployment.yaml
Issue probably came from first Vagrant run which failed.

K8s up and running after copying this file to /home/coreos on master.
Closing.


remove-from-mesh should skip system namespace

Maybe bug with https://github.com/istio/istio/pull/21107 because for the first time it says.
$ istioctl x remove-from-mesh deployement local-path-provisioner -n local-path-storage
deployment "local-path-provisioner.local-path-storage" updated successfully with Istio sidecar un-injected.

Should say no Istio sidecar injected or skip system namespace to remove sidecar injector.
If you run the above command for the second time it works. May be because of https://github.com/istio/istio/issues/20998#issuecomment-584367775
$ istioctl x remove-from-mesh deployement local-path-provisioner -n local-path-storage
deployment "local-path-provisioner.local-path-storage" has no Istio sidecar injected. Skipping.

Based on https://github.com/istio/istio/issues/25860#issuecomment-664089111
Probably It should also skip istio-system namespace to avoid human error eg: https://github.com/istio/istio/issues/25860
$ istioctl experimental remove-from-mesh deployment istio-egressgateway  -n istio-system
Error: 1 error occurred:
	* failed to update deployment "istio-egressgateway.istio-system" for service "istio-egressgateway.istio-system" due to Deployment.apps "istio-egressgateway" is invalid: spec.template.spec.containers: Required value

I tried to do something similar.  (Although John approved my PR, there was no one in UX with time to look at it so it went stale.)
Would it make sense to use the technique of https://github.com/istio/istio/pull/25887 rather than checking namespaces?  Or do both?

There a similar PR https://github.com/istio/istio/pull/27543 which I reviewed today and ask to refer your stale PR because that looks good to me.

Would it make sense to use the technique of #25887 rather than checking namespaces? Or do both?

I think we should keep both to avoid confusion like https://github.com/istio/istio/issues/25860 in future.


St Marys DEV

This is geolocated in the middle of nowhere and the Genuki entry is the same. (In actual fact is is a lane called St Marys Lane). There are several St Marys in DEV and the correct one depends on the location in DEV.
Please remove the search records and delete the Gazetteer entry. The search records culd belong to one of many places.
Geoff

Done


rpm.runc ignore ErrLibcryptoMissing

The change to not link against openssl was backported to 4.16-4.12
as part of OCPBUGS-36541 but seems to manifest with different error
there than in 4.17 due to differences in golang versions.

/retitle OCPBUGS-37846: rpm.runc ignore ErrLibcryptoMissing


HBASE-27025 Change Hbase book's description for 74.7.3.  U: .

 U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4530/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 31s
master passed

+0 :ok:
refguide
2m 49s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 44s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 13s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+0 :ok:
refguide
2m  6s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 42s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 12s
The patch does not generate ASF License warnings.

13m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4530/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4530

Optional Tests
dupname asflicense refguide spotless

uname
Linux 70acbe46080f 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / f526609331

refguide
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4530/1/yetus-general-check/output/branch-site/book.html

refguide
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4530/1/yetus-general-check/output/patch-site/book.html

Max. process+thread count
70 (vs. ulimit of 30000)

modules
C: . U: .

 U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4530/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

 Did you mean to commit them?

 Did you mean to commit them?

I clicked the patch, and do not see unrelated changes. What do you see? @bbeaudreault

Thanks @Apache9 and @bbeaudreault for review, pushed.


Adding the shorthand will be helpful to most

Shorthand dim to avoid variants is very useful and should be in this article. It makes the article more complete and leads to less verbose easier to read code.

Thanks for the suggestion!


Spring Boot 2.0.0

Minimal changes to get the starter working in Spring Boot 2.0.0.

Thanks for taking the time to do this.  I thought I had commented before, but I must have forgotten.  Since 2.0.0 is only a milestone right now, I'm waiting for the first release before pulling this.


Should pc.Entity.destroy() not recurse over non-entity graph nodes?

Hi,
We wandered into an interesting problem today. When we attach an object B to the skeleton of an object A, that object B is not destroyed when we destroy object A. This seems a bit weird to us, but could be by design. Can anyone confirm is this is indeed by design?
Because bones are not pc.Entity instances, the line of code that prevents recursion to continue through the skeleton is this. To make it work, we could simply remove this line and implement a recursing destroy method in pc.GraphNode.

The reason it happens, is that Skeleton is made of GraphNodes, but it is not part of Entity children, and belongs to AnimationComponent.
So, technically it behaves as it should behave. Although additional feature could be requested: iterate through Skeleton graph, and destroy any descendant entities in case of components entity destruction.

This is not an issue with the Render component, which unlike the Model component uses instances of Entity for its bones, and any attached entities get destroyed correctly.


RNGCryptoServiceProvider Dispose

Hi @NicolasDorier
Reading the RNGCryptoServiceProvider documentation on MSDN, it says:

This type implements the IDisposable interface. When you have finished using the type, you should dispose of it either directly or indirectly.

It is a common suggestion to use the Dispose() method to free memory with sensitive information immediately.
NBitcoin keeps only one long-life RNGCryptoServiceProvider instance. Do you think it worth changing that?
As a side question, can I add you to skype? I would like to have a less bureaucratic mean for this kind of questions/suggestions.

Well, after some research I found this: http://stackoverflow.com/questions/26371951/should-i-cache-rngcryptoserviceprovider-randomnumbergenerator-instances
The problem is that I am not sure if Mono implements this in exactly the same way.

yes "nicolas.dorier" is my skype.
For Mono, it does not. This is why I permit to submit your own "IRandom" implementation. In fact most of methods in NBitcoin will crash until you implement your own IRandom.
For the Android specific version, I use the SecureRandom class by default so it works right out of the box, but not the portable ones.
I don't think it is worth changing it, I'm not sure about the performance impact, nor security impact.

Sure, the IRandom interface is about that.
But just imagine this scenario:
Someone is using the RNGCryptoServiceProvider as his RNG provider. This person is not creating RNGCryptoServiceProvider instances in an explicit way but using the factory method instead. In code:
var _prng = new RNGCryptoServiceProvider(); // he doesn't do this!
but
var _prng =  RandomNumberGenerator.Create(); // he is doing it in this way
Note: by default RandomNumberGenerator.Create() creates an RNGCryptoServiceProvider instance so, both lines above are equivalent.
One day, this person wants to use his own RNG for all his applications (probably for performance, security or stupidity) and for that reason he decide to update his machine.config as follow:
<mscorlib>
		<cryptographySettings>
			<cryptoNameMapping>
				<cryptoClasses>
					<cryptoClass MySuperFastAndSecureRNG="MyCompany.Security.AwesomeRNGServiceProvider, MyCompany.Security, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=........."/>
				</cryptoClasses>
				<nameEntry name="System.Security.Cryptography.RandomNumberGenerator" class="MySuperFastAndSecureRNG" />
				<nameEntry name="RandomNumberGenerator" class="MySuperFastAndSecureRNG" />

Now, all his .NET applications will be using his own MyCompany.Security.AwesomeRNGServiceProvider class. But what if he wants to use this RNG with NBitcoin? Well, he will need to submit his alghoritm to you (probably he doesn't want to share it or it could be a propietary RNG, who knows).
I think that given .NET and Mono provide an standard mechanism for changing the RNG provider, it would be wise to use it. I have nothing against IRandom interface, it is okay. What I suggest is using the abstraction RandomNumberGenerator.Create() instead of the concrete implementation.

oh sorry, I had not understood what you meant. I did not know about RandomNumberGenerator class, I thought you wanted I use a factory pattern instead of IRandom.
It works with Mono are you sure ?
I'm a bit worried about a malware changing the machine.config though.

Yes, I am sure. Take a look at this README file: https://github.com/mono/mono/blob/master/mcs/class/Mono.Security.Win32/README
Malware can do whatever they want to do. They could modify/inject the CLR if are designed to do that.

interesting, sadly does not exist in portable library. :(
I am mixed about using this one over the current RNGCrypto by default. The point of using RNGCrypto explicitely by default is that a malware can't just modify machine.config for changing the prng. What do you think ?
I may just provide an implementation, but not use it by default ?

Servers with malware are seriouly compromised. Who knows what a malware can do. It can expose a remote shell, modify whatever it was designed to modify, replace dlls, inject itself in other processes, wipe processes memory, export wallets, install certificates.... who knows.
I watched a BlackHat Conf video where the hacker demostrated how to modify the .NET framework (nothing too funcy)  What I mean is that a virus could modify the IRandom.GetBytes in a way it returns predictable values.
It's okay to worry about these kind of things because we are dealing with bitcoins (money) but I think there is nothing to do when your server is running malware. You cannot develop a malware-proof/resistant software.
Anyway, I think this requires further discussions and probably other opinions. In fact, if you use RandomNumberGenerator.Create, you will be allowing custom RNGs and that means you should invoke the RNG Dispose method (because you don't know its implementation details).
On the other hand, what if microsoft changes (in a future) the default RNG returned by RandomNumberGenerator.Create? It could be a good thing if it is for security reasons. It could be bad if performance is not good.
Can we let this thread open?

yes let's let it open. What I fear is that it is simpler to just modify one line in machine.config than subverting the CLR. It is the point that I fear most.

If you fear about modification in machine.config, you shouldn't use things like this one:
using(var sha = System.Security.Cryptography.SHA256.Create())

Because this factory method creates a SHA256 instance that can be changed in the machine.config.
Remember that SHA256 is an abstract class, what means it was designed to be used as a base class for other/custom SHA256 implementations.
Here is the factory method.
public new static SHA256 Create(string hashName)
{
	return (SHA256)CryptoConfig.CreateFromName(hashName);
}

Please, take Reflector or ILSpy and see it by yourself. Go deeper and deeper and you will understand what I talk about.

very true, I will fix that. But having a bogus SHA256 is not as dangerous has having a bogus prng.

fixed 9db4a196b0b2051b95318bb4bb4d4ab4fd18038b (wrong issue number though :D)

do I close that ? Implementers of IRandom are responsible to Dispose their own internal generator. We don't need to dispo RNGCryptoServiceProvider though.
If one is using the factory method (RandomGenerator.Create), then he should dispose it. But we are not.

I close it.


Update the demo

Hi, the demo looks weak compared to this repo's potential... (speaking as a former user from v2.4.0)
As a first-point contact with this library you'd expect the demo to shine the most, perhaps even have a full-sized page to play around with, instead of a small screen in a code playground. Black as the default theme? What business is going to be thrilled by seeing that?
This is just personal opinion, do with it what you will + best of luck.

I think so too.


Selecting Value from Row Can't Find Commands

So im trying to find the correct commands todo the following with your grid and cant find the correct commands.
private async void dataview_ItemSelected(object sender, SelectionChangedEventArgs e)
{
	await Task.Delay(1000); // Temp

	DataRowView ID = (DataRowView)dataview.SelectedItem as DataRowView;
	if (dataview.SelectedItem != null)
	{

		string id = ID.Row["Id"].ToString();

		await Navigation.PushAsync(new Editjob(id));
	}

}

Figured it out , I called from the DataList instead of the Datagrid.
			var sel = dataview.SelectedItem as Dataview; 			var id = sel.Id.ToString();


[$1000] Default profile avatar is different in tooltip

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to any chat with default tooltip
hover over profile pic

Note: This happens in multiple places in the app
Expected Result:
Profile should not be different in tooltip
Actual Result:
Profile is different in tooltip
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: main (latest)
Reproducible in staging?: N
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686828292914429

https://expensify.slack.com/archives/C049HHMV9SM/p1686830645725319

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01cd1cc4ea3187cd31
Upwork Job ID:<PHONE_NUMBER>344900096
Last Price Increase: 2023-06-16

@s-alves10 please do, but please provide a proposal how you think this can be fixed first 🙏

.

Avatar needs name, source, and type props for the default avatar to work. I also reported that the workspace avatars are incorrect for the same reason. I can potentially also take this one

Proposal
Please re-state the problem that we are trying to solve in this issue.
Default profile avatar is different in tooltip
What is the root cause of that problem?
We are using login here in UserDetailsTooltip
https://github.com/Expensify/App/blob/d39643006874bf5ffec923291afa6b43884a4aec/src/components/UserDetailsTooltip/index.js#L18-L33
What changes do you think we should make in order to solve the problem?
We shall use accountID instead
What alternative solutions did you explore? (Optional)

@s-alves10 Can you please review my proposal. It requires changes on multiple files?

@alex-mechler is this resolved by https://github.com/Expensify/App/pull/20818 as well?

This is not @twisterdotcom

Is this still reproducible on staging? I can't reproduce

@s-alves10 this is reproducible in dev(main branch) not in staging

@s-alves10 @gadhiyamanan
I just took a fresh pull of master, the bug is there (v1.3.28-4)
https://github.com/Expensify/App/assets/27822551/3fd7109e-5aad-4730-adcb-fc8f4c002fb1

@twisterdotcom Sure I will make sure not to repeats this. I was supposed to type I solution but pulled off by somethings. But agree with you.

Noted 🖋 thank you @twisterdotcom

I can't really reproduce until it's on staging, and I still can't reproduce now:
https://github.com/Expensify/App/assets/9133401/4bab4095-0bb6-4fb5-bfe7-37bd19ff52b9

Yeah sadly the code that potentially broke this is still in main / only the dev environment. I'll test real quick

I think if it is, let's just throw External on it and assign @BhuvaneshPatil if the C+ assigned agrees.

I can reproduce in main 👍

(Note: There's no email shown in my environment b/c I'm testing some other changes)

@eVoloshchak - what do you think of the proposals here?

Bump @eVoloshchak to review these proposals.

@BhuvaneshPatil's proposal looks good to me, I think we should proceed with it
🎀👀🎀 C+ reviewed!
cc: @twisterdotcom

This was fixed already (https://github.com/Expensify/App/pull/21026#issuecomment-1597068835)

Latest codebase:
https://github.com/Expensify/App/blob/e9a7a54fb8b4f6fae634ec59464edf27d56d2b17/src/components/UserDetailsTooltip/index.js#L20-L23

Awesome, thanks @situchan!

Yeah, this was fixed by @Beamanator - https://github.com/Expensify/App/commit/2a67b3e76567c78208b72b48e1ea5cb27d5584d3

Okay, let's close then. Dupe of https://github.com/Expensify/App/pull/21026#issuecomment-1597068835


Command 'persist'

I am new to Alluxio. Recently, I study and exercise alluxio on my compute. And I got a problem which makes me puzzled.
Firstly, I entry ${alluxio_home}, execute command "touch hello", then "echo 'hello world' > hello", and do "copyFromLocal" to put file to alluxio;
Secondly, do command "bin/alluxio fs persist /hello";
Thirdly, restart alluxio.
After this steps, I could still get data in file(/hello) by "bin/alluxio fs cat /hello". But if skip the second step, I only get "Block xxx is not available in Alluxio". Although I can see the file by "bin/alluxio fs ls /".
How can I get the file back to alluxio. Please help me.

Please post your question here: https://groups.google.com/forum/?fromgroups#!forum/alluxio-users
If you can't access the above link, please post your question here: http://alluxio-users.85194.x6.nabble.com/


Translate concepts/overview/object-management-kubectl/imperative-command in Korean

Fixes #issue-12837-object-management-kubectl_imperative-command
/language ko

/assign

/approve

sorry for mistakes, seems no update
/approve cancel


feat: Queue "cached" commands if its execution timed out

If called "cached" command fails with OperationTimeoutError it should be queued like if the connection was absent at all.

:tada: This PR is included in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Stepper optimization

Optimazations for the steppers,

Stepper improvements:

3 Types are implemented, FULL4WIRE, HALF4WIRE, DRIVER
Outputs can be deactivated
backlash compensation

#fixes 220


Error in KeychainAccess for Xcode Version 8.0 beta 6

Xcode :Version 8.0 beta 6
Cocopods:/cocoapods-1.0.1
KeychainAccess version : KeychainAccess 2.3.6
Swift : 3.0

Is there any issue in converting the Existing code to swift-3. There are lot's of error in current version of KeychainAccess after compilation .
Could you help for above scenario .

@shrawan2015 Use swift-3.0 branch to use this lib in Swift 3 and Xcode beta.

Thanks . Still there is error in  KeychainAccess for swift3 branch and  Xcode 8 GM seed

@shrawan2015 You need writing install hook in your Podfile because CocoaPods doesn't respect project build settings.
platform :ios, '8.0'

target '...' do
  use_frameworks!

  pod 'Realm', git: 'https://github.com/realm/realm-cocoa.git', branch: 'master', submodules: true
  pod 'RealmSwift', git: 'https://github.com/realm/realm-cocoa.git', branch: 'master', submodules: true

  post_install do |installer|
    installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
        config.build_settings['SWIFT_VERSION'] = '2.3'
      end
    end
  end

end

But I've released v3.0.0, now you can use just pod 'KeychainAccess'.


fix: Allow linking to FAQ headers

Allow linking to FAQ headers for easier link sharing which also generates a button to copy the link:

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Relates to https://github.com/kedacore/keda-docs/pull/1272

Thanks!


add linter

Codecov Report

Merging #66 (65d51c6) into master (8c20751) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #66   +/-   ##
=======================================
  Coverage   89.86%   89.86%
=======================================
  Files          13       13
  Lines         602      602
=======================================
  Hits          541      541
  Misses         40       40
  Partials       21       21

Impacted Files
Coverage Δ

node_generate_strategy.go
89.83% <ø> (ø)

Continue to review full report at Codecov.

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