Updated default mysql_upstream_version from 5.7 to 8.0 after eol of 5.7

MySQL 8.0 has been released (https://dev.mysql.com/downloads/mysql/) as a result the URL 'http://dev.mysql.com/get/mysql57-community-release-el7-9.noarch.rpm' for the yum repository package is no longer valid giving the following error if 'mysql_upstream_version' is left at the default '5.7'
TASK [ansible-galaxy/mjanser.mysql : ensure mysql upstream repository is installed] fatal: [host-example.com]: FAILED! => {"changed": false, "msg": "Failure downloading http://dev.mysql.com/get/mysql57-community-release-el7-9.noarch.rpm, HTTP Error 404: Not Found"}
In commit 2e558f5 I have fixed this by setting the default 'mysql_upstream_version' to '8.0'. Obvioulsy, an implication of this fix is that the default MySQL version is now 8.0. I hope this is acceptable.
In commit 7b7073c & a231c6d I updated upstream_mysql_yum.yml to make it is possible to continue using MySQL 5.7 if the user prefers. This required the yum repo URL composition to be split from the 'mysql_upstream_version' variable, because the repo does contain '57', the repo package can only be found at the URL: 'http://dev.mysql.com/get/mysql80-community-release-el7-9.noarch.rpm'. I used the MySQL documentation as model on how to enable/disable the 5.x repositories: https://dev.mysql.com/doc/mysql-yum-repo-quick-guide/en/

@mjanser Any chance of getting this merged and deployed to ansible-galaxy? Or do you maybe have an issue with my proposed changes?

Hi Martin, how should we proceed with this?

Hi @mjanser, I was reviewing my pull requests and this came up. How can I help to get this merged?


send transaction ,get error Error 3090003: provided keys, permissions, and delays do not satisfy declared authorizations

auto trx = transaction();
trx.actions.emplace_back(vector<permission_level>{{N(ocaskans), N(active)}}, test_action);    trx.send(0, receiver);
but get error!
Error 3090003: provided keys, permissions, and delays do not satisfy declared authorizations
Error Details:
transaction declares authority '{"actor":"ocaskans","permission":"active"}', but does not have signatures for it under a provided delay of 0 ms

change active to as follows:
trx.actions.emplace_back(vector<permission_level>{{N(ocaskans), N(ocaskans.code)}},transferfromcurrency);
trx.send( 0xffffffffffffffff, N(ocaskans));
--but-->
Error 3040000: transaction exception
Error Details:
action's authorizations include a non-existent permission: {permission}

@bytemaster  thanks very much, the problem has been solved and the specific solution is as follows:
cleos -u http://<IP_ADDRESS>:8889/ --wallet-url http://<IP_ADDRESS>:8890/ set account permission ocaskans active '{"threshold": 1,"keys": [{"key": "EOS8W7jnJH6n5vfBwnp3JW3PzvZpaWTSnWx4GMhPsezXfxR8ZwjJk","weight": 1}],"accounts": [{"permission":{"actor":"ocaskans","permission":"eosio.code"},"weight":1}]}' owner -p ocaskans
EOS8W7jnJH6n5vfBwnp3JW3PzvZpaWTSnWx4GMhPsezXfxR8ZwjJk is public key of you create,and then need import Corresponding private key to wallet

While creating account it asks for the public key : Error 3090003: provided keys, permissions, and delays do not satisfy declared authorizations
Ensure that you have the related private keys inside your wallet and your wallet is unlocked.
i given the correct key after generating the import key..

system: ~/eos/programs$ cleos create account eosio ping.ctr EOS8W7jnJH6n5vfBwnp3JW3PzvZpaWTSnWx4GMhPsezXfxR8ZwjJk EOS8W7jnJH6n5vfBwnp3JW3PzvZpaWTSnWx4GMhPsezXfxR8ZwjJk
Error 3090003: provided keys, permissions, and delays do not satisfy declared authorizations
Ensure that you have the related private keys inside your wallet and your wallet is unlocked.

hi friends,
I need the answer for the account creation again and again i found this error...
Error 3090003: provided keys, permissions, and delays do not satisfy declared authorizations while  creating an account

Hello, this problem is not shooting.


#839 enhancement : add Spark Task Component can switch Spark Version

What is the purpose of the pull request
This pull request adds spark task component can swithch spark version
Brief change log

Add spark version param in spark task component
According to the spark Version param specify SPARK_HOME1 or SPARK_HOME2
Add UT for spark task build args

Verify this pull request
This change added tests and can be verified as follows:

Added SparkTaskTest to verify the change.
Verified in CDH-5.16.2(Spark1.6 and Spark2.4)

Codecov Report

Merging #1494 into dev will decrease coverage by <.01%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             dev   #1494      +/-   ##
========================================
- Coverage   2.69%   2.69%   -0.01%
========================================
  Files        271     272       +1
  Lines      13593   13599       +6
  Branches    2159    2160       +1
========================================
  Hits         367     367
- Misses     13209   13215       +6
  Partials      17      17

Impacted Files
Coverage Δ

...nscheduler/server/worker/task/spark/SparkTask.java
0% <0%> (ø)
:arrow_up:

...he/dolphinscheduler/common/enums/SparkVersion.java
0% <0%> (ø)

...inscheduler/common/task/spark/SparkParameters.java
0% <0%> (ø)
: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 08e0562...9564af9. Read the comment docs.


Implement exemplar external plugin

Overview
Can be any like livemark-ckan or whatever

DONE in https://github.com/roll/livemark-ckan


chore: release v0.2.2

:robot: I have created a release beep boop
0.2.2 (2024-03-14)
What's Changed

chore(deps): bump toml from 0.8.10 to 0.8.11 by @dependabot in https://github.com/coffeebeats/gdpack/pull/162
chore(deps): bump anyhow from 1.0.80 to 1.0.81 by @dependabot in https://github.com/coffeebeats/gdpack/pull/164
chore(deps): bump reqwest from 0.11.25 to 0.11.26 by @dependabot in https://github.com/coffeebeats/gdpack/pull/165
chore(deps): bump thiserror from 1.0.57 to 1.0.58 by @dependabot in https://github.com/coffeebeats/gdpack/pull/166
fix: correctly set default version in install.sh by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/167
fix(ci): correctly update PATH in setup action by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/169
fix(ci): correctly export environment variable in setup action by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/170
fix(ci): correctly reference home directory in action; add missing gdpack command by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/171
fix(ci): correctly use environment variable in cache path by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/173
chore(deps): bump rust-ini from 0.20.0 to 0.21.0 by @dependabot in https://github.com/coffeebeats/gdpack/pull/172
fix(ci): use correct path in cache key by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/174
fix(ci): conditionally check for executable on path during setup by @coffeebeats in https://github.com/coffeebeats/gdpack/pull/175

Full Changelog: https://github.com/coffeebeats/gdpack/compare/v0.2.1...v0.2.2

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/coffeebeats/gdpack/releases/tag/v0.2.2 :sunflower:


TST: perf attrib regression test

test perf attrib with two years of data

LGTM, how did you create these?


Declare unofficial *BSD support

As title.

Depends on https://github.com/numba/numba/pull/6309 and https://github.com/numba/llvmlite/pull/627


Each realtime channel for chat should only be fetched once

See:

https://github.com/ably/ably-chat-js/issues/411
https://ably-real-time.slack.com/archives/C02NY1VT3LY/p1732108960506119

Well, this doesn't seem to be working. It seems that now that the channel modes (PRESENCE and PRESENCE_SUBSCRIBE) are no longer being dropped from the ATTACH protocol message, for some reason the ATTACHED protocol message no longer contains a channelSerial, breaking room message subscriptions:
Outgoing ATTACH protocol message before changes
DEBUG: 52C15A94-3307-448A-8BAE-5DF52E9E7C94:rx: (ARTRealtime.m:1285) RT:0x600001ff5d00 sending action 10 - Attach - <ARTProtocolMessage: 0x6000011f8240> {
 count: 0,
 id: (null),
 action: 10 (Attach),
 channel: basketball::$chat::$chatMessages,
 channelSerial: (null),
 connectionId: (null),
 connectionKey: (null),
 msgSerial: (null),
 timestamp: (null),
 flags: 0,
 flags.hasPresence: NO,
 flags.hasBacklog: NO,
 flags.resumed: NO,
 messages: (null)
 presence: (null)
 params: {
    agent = "chat-ios/0.1.0";
    occupancy = metrics;
}
}

Incoming ATTACHED protocol message before changes
DEBUG: 52C15A94-3307-448A-8BAE-5DF52E9E7C94:rx: (ARTRealtimeChannel.m:610) R:0x600001ff5d00 C:0x6000018f8270 (basketball::$chat::$chatMessages) <ARTProtocolMessage: 0x6000011f0990> {
 count: 0,
 id: (null),
 action: 11 (Attached),
 channel: basketball::$chat::$chatMessages,
 channelSerial: e7d0rZ8lgBikN401573530@1732124334504-0,
 connectionId: (null),
 connectionKey: (null),
 msgSerial: (null),
 timestamp: (null),
 flags: 983105,
 flags.hasPresence: YES,
 flags.hasBacklog: NO,
 flags.resumed: NO,
 messages: (null)
 presence: (null)
 params: {
    occupancy = metrics;
}
}

Outgoing ATTACH protocol message after changes
DEBUG: F50F4E3E-CF5D-429B-A8DA-EC5CDB1F835B:rx: (ARTRealtime.m:1285) RT:0x6000039eda00 sending action 10 - Attach - <ARTProtocolMessage: 0x6000037fec70> {
 count: 0,
 id: (null),
 action: 10 (Attach),
 channel: basketball::$chat::$chatMessages,
 channelSerial: (null),
 connectionId: (null),
 connectionKey: (null),
 msgSerial: (null),
 timestamp: (null),
 flags: 589824,
 flags.hasPresence: NO,
 flags.hasBacklog: NO,
 flags.resumed: NO,
 messages: (null)
 presence: (null)
 params: {
    agent = "chat-ios/0.1.0";
    occupancy = metrics;
}
}

Incoming ATTACHED protocol message after changes
DEBUG: F50F4E3E-CF5D-429B-A8DA-EC5CDB1F835B:rx: (ARTRealtimeChannel.m:610) R:0x6000039eda00 C:0x600003ef44e0 (basketball::$chat::$chatMessages) <ARTProtocolMessage: 0x6000037e4b40> {
 count: 0,
 id: (null),
 action: 11 (Attached),
 channel: basketball::$chat::$chatMessages,
 channelSerial: (null),
 connectionId: (null),
 connectionKey: (null),
 msgSerial: (null),
 timestamp: (null),
 flags: 589889,
 flags.hasPresence: YES,
 flags.hasBacklog: NO,
 flags.resumed: NO,
 messages: (null)
 presence: (null)
 params: {
    occupancy = metrics;
}
}

Internal Realtime issue: https://ably.atlassian.net/browse/REA-2204


🕵♂ Security Groups - Specific Ports Unrestricted

Checks security groups for rules that allow unrestricted access (<IP_ADDRESS>/0) to specific ports. Unrestricted access increases opportunities for malicious activity (hacking, denial-of-service attacks, loss of data). The ports with highest risk are flagged red, and those with less risk are flagged yellow. Ports flagged green are typically used by applications that require unrestricted access, such as HTTP and SMTP.
If you have intentionally configured your security groups in this manner, we recommend using additional security measures to secure your infrastructure (such as IP tables).
Note: This check only evaluates security groups that you create and their inbound rules for IPv4 addresses. Security groups created by AWS Directory Services are flagged as red or yellow, but they don’t pose a security risk and can be safely ignored or excluded. For more information, see the Trusted Advisor FAQ.
Alert Criteria
Green: Access to port 80, 25, 443, or 465 is unrestricted.
Red: Access to port 20, 21, 1433, 1434, 3306, 3389, 4333, 5432, or 5500 is unrestricted.
Yellow: Access to any other port is unrestricted.
Recommended Action
Restrict access to only those IP addresses that require it. To restrict access to a specific IP address, set the suffix to /32 (for example, <IP_ADDRESS>/32). Be sure to delete overly permissive rules after creating rules that are more restrictive.
Additional Resources
Amazon EC2 Security Groups
List of TCP and UDP port numbers (Wikipedia)
Classless Inter-Domain Routing (Wikipedia)

Examined TA, verified usage and excluded results relating to the following:

[ ] Dev
[ ] PreProd
[ ] Prod

acohen terraform namespace in Dev needs to be deleted to remove redundant items created in this repo.

Forwarded remaining groups over to the IGS PM Dotun for review and resolution.

Marked SG_Public group to be excluded in TA in prep and prod, doesn't appear in Dev. The SG Private group is also not mentioned.


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/network/application-gateway
Module Details
Module that deploys Azure Application Gateway resource
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
ilhaan
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @ilhaan,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
We just want to ask you to double check a few important things and take the next steps before you start the development.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - to name your module
TelemetryIdPrefix - to be used in your module's telemetry

For your module's repository:

Repository name and folder path are defined in RepoURL
Create the GitHub teams defined in the ModuleOwnersGHTeam and ModuleContributorsGHTeam columns and grant them permissions as described here.

In case of a Bicep module, please let the AVM core team know when you created these teams, and we'll help you with assigning the necessary permissions as described here.
In case of a Terraform module, follow the instructions here and assign the necessary permissions.

Grant the right level of permissions for  the AVM core team and PG teams on your GitHub repo as described here.
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment and request a review from the AVM core team once your module is ready to be published! Please include a link pointing to your PR if available. 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

ilhaan,
Due to a recent change in the naming convention for AVM Bicep modules, this module is getting an updated name.
The title and description of this GH issue has been updated to reflect the name change. Please make sure you adhere to the new naming convention when publishing your module!
Please see the new naming convention in:

RMNFR1 for resource modules
PMNFR1 for pattern modules

Please note that the Telemetry ID and name of GitHub Teams haven't changed.
If you feel uncertain about any details, please check the Bicep modules indexes.

Thanks @matebarabas. GitHub Teams created:

https://github.com/orgs/Azure/teams/avm-res-network-applicationgateway-module-owners-bicep
https://github.com/orgs/Azure/teams/avm-res-network-applicationgateway-module-contributors-bicep

I've just requested the -owners- team to be added to the -technical-reviewers- team. Please approve.

@matebarabas approved

@ilhaan, I just would like to check back on this issue and ask how far do you think you are? Can you please share your expected timeline on publishing this module? Is there anything you need help with?
Thank you!
CC: @jeanchg

@matebarabas I should be able to send out an initial PR by the end of next week.

That's great news, thanks for the update!

Draft PR: https://github.com/Azure/bicep-registry-modules/pull/835

Still working on this

Still working on this

Still working on this. Draft PR here: https://github.com/Azure/bicep-registry-modules/pull/835

PR ready for review: https://github.com/Azure/bicep-registry-modules/pull/835

[x] Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work @ilhaan! I'm closing this issue now.


How to remove the shadow of JFXTabPane?

I tried like this:
.jfx-tab-pane .tab-header-area {     -fx-effect: none; }
but it didn't work.

Hello, it worked for me with -fx-effect: null

@qtran Hello, thank you for your reply, but it didn't work for me either. I don't know why.

@qtran, what CSS selectors did you use?
@GreenLv, this should do the trick:
.jfx-tab-pane .depth-container { -fx-effect: none; }

@TurekBot Problem solved! Thank you very much!

@TurekBot with .jfx-tab-pane .tab-header-area { -fx-effect: null; }
but seems like it works only in an older version

Hello, I'm also trying to remove the shadow from the JFXTabPane. None of the above solutions are working for me. If it's any help, I'm using JFoenix 8.0.8 with TornadoFX 1.7.17.

It's hard to know how to help you. Try using SceneBuilder's CSS Analyzer
(View ->Show CSS Analyzer) to see which style classes you need to
affect.
On Wed, May 29, 2019, 12:29 PM l3njo<EMAIL_ADDRESS>wrote:

Hello, I'm also trying to remove the shadow from the JFXTabPane. None of
the above solutions are working for me. If it's any help, I'm using JFoenix
8.0.8 with TornadoFX 1.7.17.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jfoenixadmin/JFoenix/issues/636?email_source=notifications&email_token=AF7R6P6DF7WNFHCLDRXCAGDPX3DSDA5CNFSM4EUIBAF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWQHSVA#issuecomment-497056084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF7R6P427NGFPQYRIXPK4L3PX3DSDANCNFSM4EUIBAFQ
.


How can we give a image per secound for video creater?

Is that possible to select the image per second or 3 seconds per image

Yea thanks for the comment, I have completed the feature already thanks.


[GAD] Error when trying to use Generalized Aggressive Decoding for inference

I'm trying to use: https://github.com/microsoft/unilm/tree/master/decoding/GAD with pre-trained model in that repo for inference
Description: wmt14.en-de
Models:

at-verifier-base
nat-drafter-base (k=25)

My inference script
#!/bin/bash

data_dir=./unilm/decoding/GAD/data/wmt14.en-de # the dir that contains dict files
checkpoint_path=./wmt14-en-de-base-at-verifier.pt # the dir that contains AT verifier checkpoint
AR_checkpoint_path=./wmt14-en-de-base-nat-drafter-checkpoint.avg10.pt # the dir that contains NAT drafter checkpoint
input_path=./unilm/decoding/GAD/test.bpe # the dir that contains bpe test files
output_path=./unilm/decoding/GAD/output.txt # the dir for outputs

strategy='gad' # fairseq, AR, gad
batch=1
beam=5

beta=5
tau=3.0
block_size=25

src=en
tgt=de

python3 inference.py ${data_dir} --path ${checkpoint_path} \
      --user-dir block_plugins --task translation_lev_modified --remove-bpe --max-sentences 20 --source-lang ${src} \
      --target-lang ${tgt} --iter-decode-max-iter 0 --iter-decode-eos-penalty 0 --iter-decode-with-beam 1 \
      --gen-subset test --AR-path ${AR_checkpoint_path} --input-path ${input_path} --output-path ${output_path} \
      --block-size ${block_size} --beta ${beta} --tau ${tau} --batch ${batch} --beam ${beam} --strategy ${strategy}

Here's the content of ./unilm/decoding/GAD/test.bpe
Ro@@ asted barr@@ am@@ un@@ di fish
I'm facing this error when trying to run that script
GAD plugins loaded...
2022-06-24 03:44:45 | INFO | fairseq.tasks.translation |  [translation.py:313] [en] dictionary: 32768 types
2022-06-24 03:44:45 | INFO | fairseq.tasks.translation |  [translation.py:314] [de] dictionary: 32768 types
2022-06-24 03:44:45 | INFO | inference |  [inference.py:265] loading model(s) from /home/user/wmt14-en-de-base-at-verifier.pt
Traceback (most recent call last):
  File "inference.py", line 282, in <module>
    AR_models, _AR_model_args, _AR_model_task = load_model_ensemble_and_task(filenames=[cmd_args.AR_path],
  File "/home/user/unilm/decoding/GAD/fairseq/checkpoint_utils.py", line 350, in load_model_ensemble_and_task
    task = tasks.setup_task(cfg.task)
  File "/home/user/unilm/decoding/GAD/fairseq/tasks/__init__.py", line 44, in setup_task
    return task.setup_task(cfg, **kwargs)
  File "/home/user/unilm/decoding/GAD/fairseq/tasks/translation.py", line 299, in setup_task
    raise Exception(
Exception: Could not infer language pair, please provide it explicitly

Please help to answer the following questions

What should be the format of the file in input_path? Is my bpe input correct?
How to fix the error?

Thank you.

Hi @ndkmath1,
I noticed that you put the NAT drafter and the AT verifier in opposite positions. It should be placed as follows. Can you try again the correct way?
‘’‘
checkpoint_path=./checkpoints/wmt14-en-de-base-nat-drafter-checkpoint.avg10.pt # the dir that contains NAT drafter checkpoint
AR_checkpoint_path=./checkpoints/wmt14-en-de-base-at-verifier.pt # the dir that contains AT verifier checkpoint
‘’‘

You can pre-process the official WMT14 EN-DE test.de file with our bpemodel here.
Here are the first three lines of our test file:
Gut@@ ach : Incre@@ ased safety for pedestri@@ ans
They are not even 100 metres apart : On Tuesday , the new B 33 pedestrian lights in Dorf@@ park@@ platz in Gut@@ ach became operational - within view of the existing Town Hall traffic lights .
Two sets of lights so close to one another : inten@@ tional or just a sil@@ ly error ?

I noticed that you put the NAT drafter and the AT verifier in opposite positions. They should be placed as follows. Can you try again the correct way?
It works. Thank you!

I'm trying to train GAD from a different language pair. Could you please let know which script/code you are using to create your bpe model/vocab in this folder ? So I can apply the same way for the different language pair.

The bpe codes we used are from the released data by Google. However, you can find a pre-process script here, which works fine with wmt14.en-de raw data.

If you encounter any problems, feel free to contact me 😊

Thank you! I will contact you if there is any problems.

Thank you! I will contact you if there is any problems.

Hi @ndkmath1,
The training of the AT verifier is totally the same as the conventional Transformer in fairseq, just check the scripts below:
translation#training-a-new-model
#training-a-new-model-on-wmt16-en-de

Thank you. I will try the links.

Hi @hemingkx,
Now, I'm able to train both NAT drafter and AT verifier based on your suggestion. I'm facing the following error when trying to run inference.
~/xfspell$ sh src/03_inference.sh
GAD plugins loaded...
2022-07-01 10:50:07 | INFO | fairseq.tasks.translation |  [translation.py:313] [fr] dictionary: 80 types
2022-07-01 10:50:07 | INFO | fairseq.tasks.translation |  [translation.py:314] [en] dictionary: 80 types
2022-07-01 10:50:07 | INFO | inference |  [inference.py:265] loading model(s) from models/nat_drafter01/checkpoint_best.pt
Traceback (most recent call last):
  File "/home/user/unilm/decoding/GAD/inference.py", line 282, in <module>
    AR_models, _AR_model_args, _AR_model_task = load_model_ensemble_and_task(filenames=[cmd_args.AR_path],
  File "/home/user/unilm/decoding/GAD/fairseq/checkpoint_utils.py", line 339, in load_model_ensemble_and_task
    state = load_checkpoint_to_cpu(filename, arg_overrides)
  File "/home/user/unilm/decoding/GAD/fairseq/checkpoint_utils.py", line 271, in load_checkpoint_to_cpu
    overwrite_args_by_name(state["cfg"], arg_overrides)
  File "/home/user/unilm/decoding/GAD/fairseq/dataclass/utils.py", line 427, in overwrite_args_by_name
    with open_dict(cfg):
  File "/usr/lib/python3.8/contextlib.py", line 113, in __enter__
    return next(self.gen)
  File "/home/user/.local/lib/python3.8/site-packages/omegaconf/omegaconf.py", line 669, in open_dict
    prev_state = config._get_node_flag("struct")
AttributeError: 'dict' object has no attribute '_get_node_flag'

How to reproduce the error? Please use this repo.

Preprocess data: ~/xfspell$ sh src/00_preprocess_data.sh
Train AT verifier: ~/xfspell$ sh src/01_train_AT_verifier.sh
Train NAT drafter: ~/xfspell$ sh src/02_train_NAT_drafter.sh
Run inference: ~/xfspell$ sh src/03_inference.sh

Do you have any suggestion to fix that error?
Thank you.

Maybe this issue helps. Do you also have this problem when just running our code?

Hi @hemingkx,
I've installed fairseq as same as the instruction in readme.md
cd GAD
pip install --editable .

Step 3 and 4 (Train NAT drafter and run inference) is the same as your code. I just copy it into a separate shell script file.
Step 1 (Preprocess data). I use a bit different way from your suggestion
but I'm able to train normal transformer or edgelm based on that binarize data without any problems.
Step 2 (Train AT verifier). It is the same as the transformer models here.
I think the main difference is in Step 1 (Preprocessing data).

  File "/home/user/.local/lib/python3.8/site-packages/omegaconf/omegaconf.py", line 669, in open_dict
    prev_state = config._get_node_flag("struct")
AttributeError: 'dict' object has no attribute '_get_node_flag'

emmm it seems that the error comes from omegaconf. Could you try to update the omegaconf version in your environment and then try running the script?
BTW, please check this issue and this. Following their solutions might solve your problem😊.

Thank you. I will check your suggestions.

Thank you. I will check your suggestions.

  File "/home/user/.local/lib/python3.8/site-packages/omegaconf/omegaconf.py", line 669, in open_dict
    prev_state = config._get_node_flag("struct")
AttributeError: 'dict' object has no attribute '_get_node_flag'

@hemingkx @ndkmath1  I am facing similar issue, were you able to resolve the issue, any help regarding the issue?


"Only functions can be marked 'inline'" maybe wrongly detected

I was trying to define those two inline functions:
let inline isNull = function | null -> true | _ -> false
let inline isNotNull = not << isNull

which both give Only functions can be marked 'inline' (VS2015).
it seems the detection to validate inline rely on the fact that the function expressions are partially applied and don't have parameters.
Not sure if it's considered a bug, I believe those are functions and should be inlineable without ambiguity.

This is by design, is the message says you have to declare a function explicitly.  Changing this would be a feature request, please add to http://fslang.uservoice.com/.


JWTAuthenticationFailureResponse shouldn't be tagged with @internal

We need to translate the "Bad credentials" message which is displayed when a user tries to login with bad credentials.
To fulfill this requirement, we implemented a listener on the lexik_jwt_authentication.on_jwt_invalid event with the following code:
<?php

namespace MasterBundle\EventListener;

use Lexik\Bundle\JWTAuthenticationBundle\Response\JWTAuthenticationFailureResponse;
use Lexik\Bundle\JWTAuthenticationBundle\Event\JWTInvalidEvent;

class JwtListener
{
  // ...
    public function onJWTInvalid(JWTInvalidEvent $event)
    {
        $response = new JWTAuthenticationFailureResponse('Oturumunuz geçerli değil, lütfen yeniden giriş yapınız.');

        $event->setResponse($response);
    }
}

However, JWTAuthenticationFailureResponse class has the @internal phpDoc tag, which causes a warning on our IDE PhpStorm:
Class JWTAuthenticationFailureResponse is @internal
I don't know the point why this class is an "internal" one, however since we can listen the lexik_jwt_authentication.on_jwt_invalid event and listener has to use JWTAuthenticationFailureResponse I propose to remove the @internal tag from the JWTAuthenticationFailureResponse class.

👍  Can you submit a PR?

@chalasr Sure. Same is true for JWTAuthenticationSuccessResponse. Would you like me to remove its @internal tag as well?

👍  again :)

Here it is @chalasr #408

@chalasr When do you plan to release 2.4.3?

@lashae released, thanks for the reminder

You're welcome :-)


[Lib] Consider putting FragmentedByteBuffer/FragmentedReadStream into its own library

Until there is a clear use case nah.


Add license to gemspec

This change allows license compliance tools to confirm the license type.

Thank you!


add Sam address decode unit test

Issue Number

Overview

[ ] I have ...

Comments

Closing in favor of regression tests added as part of #1025 :+1:


Address some of Krista's PR comments

Also run black --config ../../../eng/black-pyproject.toml .

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-projects


test: print recorded screen frames when nextpect times out in E2E tests

Description of changes
When E2E tests timeout in nexpect, to see the error one has to play the entire asciinema by loading the test report page from the artifact section. Updating the timeout error message to include the few lines of screen recording to surface it in the exception

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #9664 (1e81e58) into master (4a87373) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9664   +/-   ##
=======================================
  Coverage   52.93%   52.93%
=======================================
  Files         829      829
  Lines       45865    45865
  Branches     9789     9789
=======================================
  Hits        24279    24279
  Misses      19581    19581
  Partials     2005     2005

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 4a87373...1e81e58. Read the comment docs.


 Thanks for pointing it out. I can take care of it.

Should be taken care of in v0.1.21.
Sorry it took a while - I had neglected my build process for a while and it revolted. Tests were failing for no reason, so I had to move to Jest along with a few other things


build: remove node_modules from SonarQube scan

Even ignored it is noisy, lets nuke 'em

Lets not do this, if we really need to, we can push the sonar as first action, before npm install


PlotIndicator for Custom Indicator written in Python

Expected Behavior
PlotIndicator method can be used by indicators defined in C# and Python.
Actual Behavior
The following exception is thrown: InvalidCastException : cannot convert object to target type.
Potential Solution
Create a C# PythonIndicator class with a public method Update, a public event Updated and protected method OnUpdated (see IndicatorBase.cs).
The Update method should wrap the Update method defined in the custom indicator class and call OnUpdated.
In PlotIndicator method with the PyObject overload, we need to try to convert into a PythonIndicator in order to use Update event.
Reproducing the Problem
Add
self.PlotIndicator('cSMA', self.custom)

in Initialize method of CustomIndicatorAlgorithm.py and run the algorithm.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Any news about this one?
I have the same issue when applying IndicatorExtensions.SMA to a custom SimpleReturn python indicator:
self.spy_simple_return = SimpleReturn('spy_sr')
self.spy_indicator = IndicatorExtensions.SMA(self.spy_simple_return, 10)

Then it get an exception as follows:
During the algorithm initialization, the following exception has occurred: InvalidCastException : cannot convert object to target type
  at Python.Runtime.PyObject.AsManagedObject (System.Type t) [0x00011] in <acb2876d541541d38dcc29b7394ceb4f>:0
  at QuantConnect.Indicators.IndicatorExtensions.SMA (Python.Runtime.PyObject left, System.Int32 period, System.Boolean waitForFirstToReady) [0x00008] in <300c943c785f405ea99e8312ac681a60>:0
  at (wrapper managed-to-native) System.Reflection.MonoMethod.InternalInvoke(System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00032] in <b0e1ad7573a24fd5a9f2af9595e677e7>:0
  at Initialize in main.py:line 37
 InvalidCastException : cannot convert object to target type
  at Python.Runtime.PyObject.AsManagedObject (System.Type t) [0x00011] in <acb2876d541541d38dcc29b7394ceb4f>:0
  at QuantConnect.Indicators.IndicatorExtensions.SMA (Python.Runtime.PyObject left, System.Int32 period, System.Boolean waitForFirstToReady) [0x00008] in <300c943c785f405ea99e8312ac681a60>:0
  at (wrapper managed-to-native) System.Reflection.MonoMethod.InternalInvoke(System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00032] in <b0e1ad7573a24fd5a9f2af9595e677e7>:0

Trying to inherit SimpleReturn indicator from Indicator class produces another exception:
TypeError : cannot instantiate abstract class


[DO NOT MERGE ] PP-2982 Postgres and Dropwizard start once

WHAT

Just created PR to compare build times

DO NOT MERGE

Closed as it was a Spike


Method-related recipes consider override dimension

A recipe like ChangeMethodName should also rename any matching override declaration and references to these override declarations. The same idea applies to other recipes changing the signature of a method declaration.
In a first step it probably makes sense to require the input to identify method declarations which are not overrides.

In a first step it probably makes sense to require the input to identify method declarations which are not overrides.

I'd like to understand this a bit better. Would not a match on an overridden method only influence that method and any descendent overrides of it? I'm not clear why it matters whether a method is an override or not. Perhaps an example would help?

I'd like to understand this a bit better. Would not a match on an overridden method only influence that method and any descendent overrides of it? I'm not clear why it matters whether a method is an override or not. Perhaps an example would help?

I was thinking it is possibly it will be more difficult to implement. Here an example:
class A {
  void m() {}
}
class B extends A {
  @Override void m() {}
}
class C extends B {
}

Let's assume I now want to rename B#m() to m1 while not renaming A#m():
class Test {
  void foo() {
    new A().m();
    new B().m();
    new C().m();
    A a = new C();
    a.m();
  }
}

The result should be
class Test {
  void foo() {
    new A().m();
    new B().m1();
    new C().m1();
    A a = new C();
    a.m();
  }
}

Thinking about it, it is probably not really more difficult to implement :-)
I assume it would be the user's responsibility to remove the @Override annotation on B#m (or B#m1) in a separate step or should the recipe (optionally) do that then?

If you have an example of ChangeMethodName or similar making an incorrect change I'd welcome a pull request adding a disabled test demonstrating the issue.


Add enterprise

Looks good to me
@amoskong can you please validate this PR works with the enterprise version?

There is only one problem, there is no streaming datapoints in http://<IP_ADDRESS>:3000/dashboard/db/scylla-enterprise-per-server-disk-i-o-2017-1

Other part is right.

@amoskong I think its the same as https://github.com/scylladb/scylla/issues/2258
If there is no streams, there are not metrics. It confusing, but not a regression.

Thanks @amoskong


This change ensures the mariadb-operator is built with FIPS compliance

Changed the build image to ubi9/go-toolkit:1.20
Changed the base image to ubi9/minimal
Added the default GO_BUILD_EXTRA_ARGS="-tags strictfipsruntime"
Added the GO_BUILD_EXTRA_ENV_ARGS build argument to allow custom build arguments at build time. It defaults to "CGO_ENABLED=1 GO111MODULE=on"
Those default parameters have been added to enable FIPS compliance
Fixed indentation
Added DOCKER_BUILD_ARGS variable in Makefile to pass custom parameters during podman build
Added export FAIL_FIPS_CHECK=true to prow_ci.env file.

/lgtm


Bug 1572129 - remove unnecessary SetState

cleaned up a bunch of logging to use *f formatted form
removed the SetState that causes update conflicts
cleaned up the dao logging to be more informative
changed formatting to be more narrow :)

[2018-05-04T02:57:29.055Z] [INFO] - Initiating Recovery Process
[2018-05-04T02:57:29.068Z] [DEBUG] - get service instance: dea8214c-4f46-11e8-b11d-0242ac110009
[2018-05-04T02:57:29.077Z] [DEBUG] - get spec: 1dd62d51c52cc2ac404d58abc0c8fa94
[2018-05-04T02:57:29.089Z] [INFO] - No podname. Attempting to restart job: dea8214c-4f46-11e8-b11d-0242ac110009
[2018-05-04T02:57:29.089Z] [DEBUG] - &{dea8214c-4f46-11e8-b11d-0242ac110009 0xc42028f4a0 0xc4205ba0a0 0xc42048a310 map[] }
[2018-05-04T02:57:29.089Z] [INFO] - Recovery complete
[2018-05-04T02:57:29.089Z] [NOTICE] - recover called


swift 4?

Tried using Notepad with swift 4 and Xcode 9, but get a few errors. Some have simple fixes, others not so much. Any plans for swift 4?

Hey, thanks for the issue! I would love for Notepad to support Swift 4. Unfortunately I haven't been able to work on Notepad for a little bit, but I'll try to get these issues fixed as soon as I download all of the latest Apple development tools.

I made the changes. I'm running them locally, and they seem to be working. I created a pull request.

@kobble-git Thanks, I'll check them out!


add feature coverage

Hi  @trimai3001 ,
the robotframework repository is special.
I'm not sure if coverage makes here sense. 99% of the codes here are not from us.
let us discuss this after my holiday.
Thank you,
Thomas


Double class getting binded

When a class is set in an dom element which is being initied with Vue and the same class is set in the component root dom element, the final result is a double class. You can see the results here, check with the console the dom and you should see "compClass compClass" instead of just one.
It's not a big deal but doesn't seem the expected behaviour.

I don't think this is a bug... You can add your own classes to the component this way which is quite useful

I'm not referencing the feature, which is totally fine. What is wrong is that your final component will end up with two identical class names, as can be seen in the fiddle.
You would never write your code as <div class"myClass myClass"> right? Vue shouldn't neither.

I see but you're the one writing myClass twice. The user is free to add any class multiple times.
It's like you are waiting for vue to fix your mistakes.

Despite the reasons behind the repeated class, Vue could be merging instead of appending the classes. The same way when using jQuery.addClass() you expect the library to add the class only if doesn't already exists. You don't expect it to fix your mistakes, but to help you keep your code clean.

It could be a feature yet the example you gave is clearly a developer mistake, do you have a real use case. The example you're providing doesn't relate to this because we are talking about adding a class to a component that already holds that class

In my application I'm using the same class to locate the dom and init Vue on it as to style it after. My initial dom would be

and to make it sure it stays that way once Vue has replaced the content I keep the same class again in the component template declaration. Yes, it's on purpose and I could change that only to end up duplicating classes names. You could still argument that I could remove it from the component, but then my component would be prompt to style issues if for any reason I decide to change the initial dom class.

Again, I don't think its a big deal, its just not what I would expect from a class merging feature.


Add support for PolylineVolume in CZML

Fixes #3062
Did my best to port forward https://github.com/CesiumGS/cesium/pull/7175. I'm somewhat fuzzy around the reference properties and time intervals and tried my best to follow other examples in the file. Please let me know if there are other test cases I should add or any other obvious issues.
CLA signing is in progress.

Thanks for the pull request @jrafidi!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Thanks @jrafidi!  We'll review this once we get your CLA

@hpinkos I believe the CLA has been submitted. Have you received it on your end?

Got it, thanks @jrafidi!  I'll have time to review this later this week.

@hpinkos Do you have time to take a look this week? Would be nice to have this in the next release.

@jrafidi sorry, I had a deadline for something else creep up on me.  I won't have time today, but hopefully before the end of the week?
@mramato @shunter would you have a minute to review this?

@jrafidi sorry, it looks like we won't be able to get this in for the June release, but I'll make sure to review it in time for the July release.  Thanks for your understanding

great work on this, thanks @jrafidi!
fyi @shunter

Finally taking a look at trying to add this to the CZML spec.  Note that in CZML "cartesian" exclusively refers to a three-dimensional value X, Y, Z, where "cartesian2" exclusively refers to a two-dimensional value X, Y.  So the "shape" of the volume should have been defined as a "cartesian2" list value.
I think for backwards compatibility we can keep the erroneous "cartesian" property support. There are other cases like this where we still support old incorrect syntax.


[feature]: arbitrary task_input and the possibility of developing shared resources as an abstraction

Both papers delve into the collaborative interaction possibilities.  I was curious about maintainer's opinion on the idea of establishing a more general Resource abstraction at the AIOS level, that enables the necessary concurrency control for accessing and manipulating shared resources on an AIOS interactive session.
It seems to me that the problem is twofold, depending on the resource at question

only utilized by agents

for this particular scenario a Resource object during runtime could act as a proxy for the resource in question, and hold a  threading.Lock or threading.Semaphore, attributes that are enough to cover current implementations of concurrency actions of tasks using the aforementioned resource.
in this scenario inputs for agents should admit arbitrary input that cannot be predicted at the agent level.

resource utilized both by agents and host OS-APP as well

in this case, there is no escape for using OS based lock mechanism

I am unaware of the existence of solutions to this type of problems, so forgive the ignorance. To me it seems like this is one of the most essential problems of the host OS - AIOS interaction. If not solved, it seems like the  agents are deemed to live in LLM space only :F

Cool! A discord subgroup will be open for you to discuss implementation plans and details.


Error: Interaction required

I tried your application on windows and it worked like a charm, I was wondering if this ADAL library can bypass MFA or not.
Our organization has MFA setup, it asks for MFA whenever a user logs in from a different location. When I tried this sample on my Linux box which is at a different location, I got the below error.  Please confirm if ADAL library can bypass MFA and give users text only interface to authenticate to the application.
Caused by: com.microsoft.aad.adal4j.AuthenticationException: {"error":"interaction_required","error_description":"AADSTS50076: Due to a configuration change made by your administrator, or because you moved to a new location, you must use multi-factor authentication to access '00000002-0000-0000-c000-000000000000'.\r\nTrace ID: 7903cd61-659e-4dc6-a8b0-3dac4c672700\r\nCorrelation ID: b541777f-e2cc-4b87-ab7b-8dd83654d445\r\nTimestamp: 2017-04-18 23:02:36Z"}

Hi @venkatmarepalli, ADAL4J can be used to sign in MFA enabled accounts, although this sample does not show how to do it. If you'd like to see a sample that handles MFA, take a look at this web app sample.

Thank you for your reply.  Can you tell me how MFA is handled here? Does it make users to enter the code on console or does ADALJ4 passes authentication without MFA ?

Hi,
I am trying to authenticate using username and password to an application i have registered on Azure portal.
But i get the below error.
The request body must contain the following parameter: 'client_secret or client_assertion.
How can i pass the client secret using the method aquiretoken with username and password.
There is no such method in the library.
Thanks,

This sample has been archived.
The newer one can be found at: https://github.com/Azure-Samples/ms-identity-java-desktop/tree/master/Username-Password-Flow


Feature/updated core data layer

JIRA URL
https://spothero.atlassian.net/browse/IOS-1929
Description

Pulled in Core Data files  from the SpotHero iOS app that were built when locally developing the feature set
Added necessary checks around testing extensions to ensure XCTest can be imported

Approved, thanks!
Plan is to add this as a pre-commit hook in the future. I tend to just run it every so often as needed right now.


Need to save instance id as fact

also, should statically tag workload as unprovisioned.

Fixed via
commit 479382c1a2c532569ada246edcabe9ae70399b13
and
commit f18685bf6502551f63ea050c9b879e23057bbb60


[cocoyaxi] Upstream changed repo name

Describe the pull request

What does your PR fix?
Fixes #25908

Continuation of #25916 by @neoblackcap.
Fix repo name and file hash to point to the new repo name.

Thank you!


Adds asciidoctor-mathematical to render math in pdfs

Hi,
I have added asciidoctor-mathematical to the gem packages that are installed in the container (full version only).
It is an alternative stem processor that can be used to render math in PDF.
Hopping the patch is and ok and is added soon to the docker file.
T.

@rochdev any chance the PR is reviewed?

I would rather keep the default image at parity with the official asciidoctor image since its purpose is to offer an alpine version of it.
It would be possible however to make a new tag alpine-asciidoctor:plus which would contain additional librairies like asciidoctor-mathematical.
If you are willing to make it I will gladly accept the contribution :)
Maybe it would even make sense eventually to have some sort of plus-mini and plus-micro tags, but I would like to at least keep existing tags at parity.

Ok, I can try.
What do I have to do exactly to add a tag to the docker hub?
If I understand, I have to add separate dockerfile (in a separate directory) and then?

maybe check the last version

:+1:


[5.3][ModuleInterface] Prefer the swiftmodule in the SDK over the prebuilt-module cache

Fix the search order for module interfaces to respect the documented behavior:

Local cache
Next to the .swiftinterface
Prebuilt cache

Fixing the order of the last two allows the XcodeDefault compiler to prioritize swiftmodule files with SPI information.

Explanation:
The compiler was picking swiftmodules from the prebuilt-module cache over those next to the swiftinterface files. The prebuilt-modules are public so then don't have SPI information. As such, even when SPI was available to a project the compiler couldn't see it.
Scope of Issue: Affects an internal project building with the default toolchain while using SPI.
Origination: Around the introduction of the prebuilt-modules cache, however this issue was made visible by the addition of Swift SPI.
Risk: Relatively low. The fix respects the intended behavior as it was originally documented. While it affects the module loading order of all projects most projects don't build with both the prebuilt-cache and swiftmodule files next to the swiftinterface files.
Reviewed By: Xi Ge
Resolves: rdar://problem/64097455
Cherry-pick Of: #32423

@swift-ci Please test

@swift-ci Please nominate


"TypeError: cannot pickle '_magnum.Color4' object" when running benchmark.py in v0.2.5

Hi there,
I'm trying to run the benchmarking script here in v0.2.5 and I'm getting the following error.
TypeError: cannot pickle '_magnum.Color4' object
Unfortunately, don't get more detail in the error message thought it seems to be because of multiprocessing and removing the multiprocessing here: https://github.com/facebookresearch/habitat-sim/blob/c8887c8cf421b6df2d77489d42f4e20488e184eb/examples/demo_runner.py#L391-L395
seems to make it work.
For completeness, here's my setup
habitat-sim v0.2.5 installed from source.
Ubuntu 18.04
Python 3.9 and Cuda version 11.7.

Thanks for raising this, @sanjayharesh,
I can reproduce this issue locally.
Looks like this particular code isn't run our CI, so possible it has become stale.

FYI, this should be implemented with https://github.com/mosra/magnum-bindings/commit/f561337caa9094c4a99411fac402342343f4ecb4, however I don't have an easy way to verify that it indeed works in the full context of Habitat.
Assuming #2243 is included in your working copy, it should be enough to just pull the submodule in src/deps/magnum-bindings to this commit to get the change and check.

I also ran into the same problem with v0.3.0, have you managed to solve it yet?


UDF mode process_frame_stack(stack)

I seem to frequently encounter a situation where I am implementing a UDF which could quite easily be vectorized along a stack of whole frames. For example applying an FFT or some kind of filter always needs to have the whole frame at once, so i can't use process_tile but I could easily process a whole stack of frames in one operation.  A lot of image processing functions can be applied to the last two axes of stack (e.g. numpy.fft.fft2) for (in some cases) free speedups. Particularly on GPU for convolution (think about (nBatch, h, w, nFeatures) in deep learning).
I think this could essentially be forced even with the current API by playing with tileshape negotiation, i.e. implement process_tile(tile) but require tiles which are whole-frames tall/wide. It's a bit clunky, though. There might be space for a process_frame_stack method, given that any process_tile function should automatically work as a process_frame[stack](). It would be similar to process_partition but with some mechanism to restrict the size of a stack to less than a whole partition.
How this could be concretely provided to the user I am not yet sure. I was wondering if this has ever been discussed or thought about ?

I was wondering if this has ever been discussed or thought about ?

I have indeed thought about this before, the last time when working on the WDD UDF - there, compression is currently done frame-by-frame (so with two matrix-vector operations). It's clear that this can be improved upon by compressing multiple frames at once (which is then two matrix-matrix operations), but it was less clear to me how this would work for arbitrary tiles (it should be possible to "decompose" the compression such that is works on tiles, but it is of course much less complex to just work on full stacks of frames). /cc @bangunarya
Correct, this process_frame_stack method would be in the place between process_partition and process_frame, meaning we still have somewhat reasonable total size for the frame stack, and not just push full partitions through it.
I think we may want to start with a benchmark, to understand the performance better - one reason why the interface is the way it is currently is to kind-of match the I/O size to fit into the L3 cache, which works out for (some) individual frames, or deep-but-slim tiles, but probably no longer for stacks of full frames. Did you benchmark the fft2 case? It would be interesting to see the gains over process_frame for different detector sizes, and if there's a cliff for >L3 which we have to overcome with more gains by means of a larger stack.

Did you benchmark the fft2 case?

Here is a benchmark for np.fft, cp.fft and pyfftw.interfaces.scipy_fftpack :

with code like this:
for stack_size in stack_sizes:
    for sig_shape in sig_shapes.keys():
        warmup_data = xp.random.uniform(size=(stack_size, *sig_shape)).astype(xp.float32)
        fft.fft2(warmup_data)
        fft.fft2(warmup_data[0])
        stack_data = xp.random.uniform(size=(stack_size, *sig_shape)).astype(xp.float32)
        res_stack = %timeit -o fft.fft2(stack_data)
        frame_data = [f.copy() for f in stack_data]
        res_frame = %timeit -o [fft.fft2(f) for f in frame_data]

For useful frame sizes, seems like the CPU implementations don't gain much or are worse when processing a stack of frames, while the CuPy implementation gains a lot (not surprising!). This doesn't account for the potential additional bonus of reading a whole batch of frames in one I/O operation, and less run_udf overhead. I'd need to implement a special tileshape + process_tile to test the full system performance.

Here is a benchmark for np.fft, cp.fft and pyfftw.interfaces.scipy_fftpack :

:+1:

for stack_size in stack_sizes:
    for sig_shape in sig_shapes.keys():
        warmup_data = xp.random.uniform(size=(stack_size, *sig_shape)).astype(xp.float32)
        fft.fft2(warmup_data)
        fft.fft2(warmup_data[0])
        stack_data = xp.random.uniform(size=(stack_size, *sig_shape)).astype(xp.float32)
        res_stack = %timeit -o fft.fft2(stack_data)
        frame_data = [f.copy() for f in stack_data]
        res_frame = %timeit -o [fft.fft2(f) for f in frame_data]

I'm don't think this benchmark will give you correct results for the CuPy case - operations like cp.fft.fft2 are asynchronous. For accurate results, I think you need to use the benchmark function instead (or write a synchronizing wrapper, but that looks tricky).

This doesn't account for the potential additional bonus of reading a whole batch of frames in one I/O operation, ...

... which, on CPU, has to be weighted against falling off the L3 cache cliff.

... and less run_udf overhead. I'd need to implement a special tileshape + process_tile to test the full system performance.

I think a full perf benchmark would indeed be interesting, as there are many components at play.
A quick way to hack this is to include a process_frame no-op UDF in the list of UDFs, which should switch the base sig shape to the sig shape of the data set. Then you can write a process_tile UDF, which can influence the tiling depth using UDF.get_tiling_preferences. That should allow, at least, to influence the minimal depth - the data source is still allowed to give a larger tiling depth (and smaller at the edges, of course), so I'd advise to test with a generic data source like RawFileDataSet.

I'm don't think this benchmark will give you correct results for the CuPy case - operations like cp.fft.fft2 are asynchronous. For accurate results, I think you need to use the benchmark function instead (or write a synchronizing wrapper, but that looks tricky).

Thanks for the advice, am new to GPU perf benchmarking! Will adapt the code once I'm back from vacation and integrate it into a full UDF example while I'm at it 🤞

Here is the same graph but running fft2 through the UDF interface on frames / stacks of frames. Info:

inline context
1 partition of 512 frames in each dataset
uses standard RawFileDataSet for the process_frame case, and a hacked raw dataset to force a given tileshape using adjust_tileshape (verified tileshape in the UDF)
For the cupy case I decided to just use a call to array.get() to force synchronisation - this adds the overhead of transferring the result but is actually quite a realistic use within a UDF, whether working on a stack or individual frames !

As before the results are only interesting on smaller frames, and then only really with cupy and pyfftw. That said, the code is not optimised for each backend (pinned memory, byte-aligned arrays, pre-planned fft etc). It's clear that there would be no benefit to going beyond stack sizes of ~16 for (512, 512) frames, and then only in specific, data-size-dependent cases.
I think for this to be implementable there would have to be a clear way for a UDF to have multiple implementations (tile, frame, stack, partition) and choose which to use at runtime based on the sig_shape and proposed partition size. It's hard to see how this could be done and also maintain the current system of tileshape negotiation (which is influenced by the dataset and the UDF(s)).
Instead a pragmatic option could be a mode where (for performance and/or algorithmic reasons) a UDF can fully specify the tileshape it wants to receive at runtime, ignoring things like ds.max_io_size and ds.needs_decode, with the constraint that the UDF cannot be run in parallel with other UDFs. That said, for this kind of algorithm where a stack is necessary it might be easier to use process_partition and handle this internal to the UDF.

Here is the same graph but running fft2 through the UDF interface on frames / stacks of frames. [...]

Thanks for the updated graph! I didn't fully digest it yet, but one interesting outcome is the notable difference between np and pyfftw. I think part of it may be threading and plan caching: we disable plan caching for pyfftw for correctness because of https://github.com/LiberTEM/LiberTEM-blobfinder/issues/35 - so if we do per-frame processing, we probably need to re-create the plan for each call. Also, using the inline executor by default allows threads in numeric libraries, so pyfftw will probably have more data to munch per thread w/ stacks? In any case, interesting!

I think for this to be implementable there would have to be a clear way for a UDF to have multiple implementations (tile, frame, stack, partition) and choose which to use at runtime based on the sig_shape and proposed partition size. It's hard to see how this could be done and also maintain the current system of tileshape negotiation (which is influenced by the dataset and the UDF(s)).

Instead a pragmatic option could be a mode where (for performance and/or algorithmic reasons) a UDF can fully specify the tileshape it wants to receive at runtime, ignoring things like ds.max_io_size and ds.needs_decode, with the constraint that the UDF cannot be run in parallel with other UDFs. That said, for this kind of algorithm where a stack is necessary it might be easier to use process_partition and handle this internal to the UDF.

This seems to have some intersection with #1353 and related plans. One thought about multiple UDFs and compatibility/performance: right now, if we combine a process_partition and a process_frame UDF, I/O will read the whole partition, then run process_frame for each frame in the partition, and process_partition once on the whole thing.
An improvement on this would be to read with a by-frame scheme, use that to run the process_frame UDF, assemble the frames into a full partition, and then run the process_partition UDF afterwards. This could even be extended to tiled processing, reading a whole "block" of tiles (meaning read tiles until the full sig shape has been read) before running the per-frame UDFs. Then all UDFs can be fed approximately the shapes they are happy with, other than disagreements between two per-tile UDFs.


Update org.yaml

Adding @franciscojavierarceo as a contributor
Completed Steps

I've reviewed the guidelines
Here are three PRs (1, 2, and 3) but they are still very small as I'm just getting started. As noted in https://github.com/kubeflow/website/pull/3731, I am one of the maintainers of Feast and intend on contributing to the Training Operator and Kubeflow Pipelines

I have 59 contributions so far (this is not much, I expect more)

@terrytangyuan  @andreyvelich  would you mind sponsoring my membership?
Below is the output from testing my PR as outlined in the docs

[patch-1]% pytest test_org_yaml.py
=============================================================================================== test session starts ================================================================================================
platform darwin -- Python 3.9.19, pytest-7.4.4, pluggy-1.5.0
benchmark: 3.4.1 (defaults: timer=time.perf_counter disable_gc=False min_rounds=5 min_time=0.000005 max_time=1.0 calibration_precision=10 warmup=False warmup_iterations=100000)
rootdir: /Users/farceo/dev/internal-acls/github-orgs
plugins: benchmark-3.4.1, cov-5.0.0, anyio-4.3.0, typeguard-4.2.1, env-1.1.3, ordering-0.6, mock-1.10.4, timeout-1.4.2, lazy-fixture-0.6.3, xdist-3.6.1
collected 1 item

test_org_yaml.py .                                                                                [100%]

================================================================================================ 1 passed in 0.13s =================================================================================================

And here's a screenshot of the 59 contribution count

@terrytangyuan updated!


Emulation bugs on 4.0.3 but not on 4.0.2

Describe the issue
My Windows 7 VM crashes on 4.0.3 but works just fine on 4.0.2
Configuration (required)

UTM Version: 4.0.3
OS Version: 15.6.1
Device Model: Xr
Is it jailbroken (name jailbreak used)? No
How did you install UTM? AltStore

debug.log
Debug log
For all issues, including crashes, you should attach a debug log. Open UTM, and open the settings for the VM you wish to launch. Near the bottom of the configuration options (or at the top of the QEMU page) is Debug Log. Turn it on and save the VM. After you experience the issue, quit the VM and re-launch UTM. Open the VM settings again and select Export Log... and attach it here.

Please also provide debug.log for 4.0.2


Error when the event is imported in the Events Acquisition tile.

Tutorial URL: https://developers.sap.com/tutorials/cp-starter-ibpm-employeeonboarding-5-run.html
Step 1: Set up sample events for workflow ----> Sub step 4.
After uploading EmployeeOnboardingEvents.json file getting the error "An error occurred while processing your request. Retry later or contact your service provider". It is also a Blocker.

Labels :
Landscape Labels: trail1, trail2 & trail3
SCPTest-eboard: Automate and Extend Employee Onboarding
Priority label :‘High-prio’
Best Regards,
Priyanka

This was the bug which is fixed now in the product. Please test again and confirm.
@MichaelCzcz - close the issue.


npm repo version out of sync with git tags

Installing the latest version (0.47.0) using npm, the code is missing some fixes that were made before to that release.
One example mismatch is in pubsub/src/subscription.js, line 735, looks like this when installed form npm :
if (self.activeRequest_ && self.activeRequest_.cancel) {
        self.activeRequest_.cancel();
      }

According to git, it was changed to
if (self.activeRequest_ && self.activeRequest_.abort) {
        self.activeRequest_.abort();
      }

The commit e92454e3a645c10f7ddd609f52cbd9ac4bbb36df added this change.

Good catch, the release process has gotten a bit more complex since we bundled, and it looks like we made a mistake when we released the bundled package. We didn't first release each bundled package (or at the very least, we skipped Pub/Sub). We are doing a release today, and will make sure this is included.
Sorry for the trouble, thank you for bringing this up.

Thanks for the quick answer!

@stephenplusplus I saw you tagged<EMAIL_ADDRESS>in https://github.com/GoogleCloudPlatform/google-cloud-node/commit/61598e937cef6ba564b49c0347cd194898836616
The package.json still lists<EMAIL_ADDRESS>as a dependency for pubsub 0.8.1, however I'm convinced that we need at least<EMAIL_ADDRESS>because the abort() fix which I contributed also required some changes in grpc-common which as far as I can tell were not part of 0.1.1 but should at least be part of grpc-common>=0.1.4.

I believe it's defined as a range, so new installs and re-installs will pick up the new dependency. Is it not working?

@stephenplusplus in situations where people use yarn with lockfiles or npm shrinkwraps they might not necessarily get the latest version in the range (AFAIK updating just pubsub does not necessarily update the transitive dependencies).
Even if it might less of a problem in practice I still think the dependencies in a package.json should correctly specify the minimum required version of a dependency, even when working with ranges. And for this particular version bump of pubsub the minimum version requirement has changed.

Agreed, thanks for the explanation. We should update and put out 0.8.2.

Thanks again @ctavan, I've put out the new release under<EMAIL_ADDRESS>


Feature/add strategies

Current coverage is 97.66% (diff: 97.84%)

Merging #17 into master will increase coverage by 0.38%

@@             master        #17   diff @@
==========================================
  Files             7          8     +1
  Lines           184        257    +73
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            179        251    +72
- Misses            5          6     +1
  Partials          0          0

Powered by Codecov. Last update df47a00...9b90ac3


Pullrequest l3gd20 fixes

This fixes l3gd20 test, and prevents spurious errors for l3gd20 on external SPI bus

Thanks, applied!


feat(performance): cache scope keys

Tag scopes must be resolved whenever a tag lookup occurs. While not slow in of itself, having to resolve an LSP client's root_dir every time a file tag is queried is excessive since this only changes an LSP server attaches (LspAttach) or detaches (LspDetach). The same goes for a directory, which seldom changes and has a well-known autocommand (DirChanged) that can be hooked into.
Cache resolved scopes

Scopes, when resolved from some function should be cached
The scope cache should be able to be explicitly invalidated (and possibly updated with varargs)
The config should expose a config.cache boolean to allow users to opt out of caching

Update README with relevant details

Update configuration section
Update tag scopes section

Some initial work for this has been done in the feat_performance_improvements branch. This is highly experimental


26016 - Feature - Child Landing Zone Management Groups Flexibility

Overview/Summary
AB#26268
Update MGDiagAll orchestration and introduction of a parameter to Management Group module that allows the deployer to true or false whether management groups beneath Platform are wanted in accordance with Trey Research architecture:
https://github.com/Azure/Enterprise-Scale/blob/main/docs/reference/treyresearch/README.md
Replace this with a brief description of what this Pull Request fixes, changes, etc.
This PR fixes/adds/changes/removes

Adds an additional parameter parPlatformMgAlzDefaultsEnable to the Management Group module. Values are either true or false whether child management groups are wanted under Platform management group, with true being default
Updated mgDiagsAll.bicep module to include Platform MG and any custom child MGs under there

Breaking Changes

N/A

Testing Evidence

As part of this Pull Request I have

[X] Read the Contribution Guide and ensured this PR is compliant with the guide
[X] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[X] Checked for duplicate Pull Requests
[X] Associated it with relevant GitHub Issues
[X] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.


fix/build: import missing Read trait

Building libsodium requires the Read trait to be imported to call the read_to_end function.
https://github.com/maidsafe/rust_sodium/blob/af37b943663150376676440eb9bc76c6ca9468ab/rust_sodium-sys/build.rs#L146
error[E0599]: no method named `read_to_end` found for type `zip::read::ZipFile<'_>` in the current scope
   --> C:\Users\***\.cargo\registry\src\github.com-1ecc6299db9ec823\rust_sodium-sys-0.10.1\build.rs:146:52
    |
146 |             assert_eq!(entry.size(), unwrap!(entry.read_to_end(&mut buffer)) as u64);
    |                                                    ^^^^^^^^^^^
    |
    = help: items from traits can only be used if the trait is in scope
help: the following trait is implemented but not in scope, perhaps add a `use` for it:
    |
17  | use std::io::Read;
    |

error: aborting due to previous error

VSCode deleted a trailing whitespace too. :)

@pierrechevalier83 @Viv-Rajkumar
I would consider this a critical bug since it prevents compiling on windows.


upgraded angular & material modules to latest

Description
Upgraded angular to 2.0.0 and angular-material2 to 2.0.0-alpha.9
https://github.com/angular/material2/blob/master/CHANGELOG.md#2.0.0-alpha.9
What's included?

Updated main package.json and platform/core/package.json for angular & angular2-material to use latest

Test Steps

[x] npm i
[x] ng serve

think we should skip this and jump to 2.0.0-alpha.9? i'm sure it's a bigger jump since they introduced themes, but probably might as well right?

yes, we should use this PR for that

What's included?

Updated to material alpha.9
Added theme.scss for theming.
Updated rollup build for platform to work with changes from material and webpack.
Added snackbar example.

Test Steps

[ ] npm i
[ ] ng serve
[ ] npm run build to test rollup build

Note: Dialogs seem to have a bug, so i couldnt create an example for them.. also, we need to update theme.scss and remove things from app.component.scss.

Theme Breaking Change Docs:
Now that angular-material has SCSS theme variables, this update will require Covalent users to:

Remove most of app.component.scss and just leave:

@import "../theme";
/* :host /deep/ lets shadowdom style child elements */
:host /deep/ {
    md-divider {
        border-top-color: rgba(black, 0.12);
    }
    md-nav-list {
        [md-list-item].active {
            md-icon[md-list-avatar] {
                background-color: md-color($accent);
                color: md-color($accent, default-contrast)
            }
            md-icon[md-list-icon] {
                color: md-color($accent);
            }
        }
    }
    a[md-icon-button] {
        line-height: 36px;
    }
    .md-tab-label {
        flex-grow: 1;
    }
}

add /src/theme.scss with (their color choices) in:

@import '~@angular/material/core/theming/all-theme';
// Plus imports for other components in your app.

// Include the base styles for Angular Material core. We include this here so that you only
// have to load a single css file for Angular Material in your app.
@include md-core();

// Define the palettes for your theme using the Material Design palettes available in palette.scss
// (imported above). For each palette, you can optionally specify a default, lighter, and darker
// hue.
$primary: md-palette($md-orange, 800);
$accent:  md-palette($md-light-blue, 600, A100, A400);

// The warn palette is optional (defaults to red).
$warn:    md-palette($md-red, 600);

// Create the theme object (a Sass map containing all of the palettes).
$theme: md-light-theme($primary, $accent, $warn);

// Include theme styles for core and each component used in your app.
// Alternatively, you can import and @include the theme mixins for each component
// that you are using.
@include angular-material-theme($theme);

my theme updates working, and docs for using custom and multiple themes updated as well

[ ] Added dialog examples
[ ] Moved global scss to styles.scss


how to add rtsp stream url

want to add rtsp url @theAIGuysCode please help

Have you been able to figure it out? @harshita2311-coder @theAIGuysCode

Have you been able to figure it out? @harshita2311-coder @theAIGuysCode

noooo

Have you considered using rabbitmq to read your stream?


Proposal for TaskGroup discardResults.

Thanks for fleshing it out and PRing @Lukasa !
Prior discussions: https://forums.swift.org/t/pitch-task-pools/61703

This is in good shape. I'll manage the review


Suggestion about perl var names

I suggest you to stick to latin letters for variable names.
This way it would be easier for everyone wanting to help with the code.
Variables like $заглавѥто, $дѣсна, $ѿговоръ, etc, could not be easily written in most IDE/Editors. Even if I have Cyrillic keymap installed.
here is a command to find all of these variables (execute it in the root folder of the project):
grep -RIHnE "\\\$[^a-zA-Z0-9\|\_\=\/\@\"\{\ \(\^\#\!\?\,\.\)]+"

I understand. Yes, I use my own keymap file in vim for Old Bulgarian.
I realize not everyone will want to use that keymap file and not everyone uses Vim.
May be I will keep only some very private ones or such that I would prefer explicitly to stand out. Something as a warning.
Thanks for the grep.

BTW, interesting why I do not get notification emails for issues. May be I have some wrong setting in my profile. Will have to check that too.

Have you unchecked the "watch" button, on the left side of "star" and "fork" buttons?

No, I fixed it. It was an account-wide setting in https://github.com/settings/emails.
I had selected radio-button "Only receive account related emails, and those I subscribe to. "


Add endpoint to abort background tasks

As said in #471, I now have a task which migrates gigabytes to the wrong datastore. It would be great if there was a way to stop it.

It'll give up due to age eventually, but stopping tasks isn't really desirable at this stage.


Add Foundy Support

Problem
As a developer I'd like to be able to use Foundry with the Hedera JSON-RPC Relay.
Solution
We'd want to test out all available scenarios with the tool, document and add example in the tools folder.
If we encounter any missing functionality, list it down below, so it can be addressed.
### Tasks

Alternatives
No response

There are many use cases for Foundry - running scripts written in Solidity, using the cli tools to deploy contracts or call existing ones, but the main one is testing contracts with Solidity tests. Unfortunatelly Foundry can be used with the Hedera network for all use cases, except for testing, because it runs the tests against it's own internal evm. It can be configured to call the relay during testing, but it only uses it to read the current state of the network and never actually calls eth_sendRawTransaction. This means that if users want to test their contracts that use the Hedera precompiles, they wouldn't be able to do that with Foundry. On the other hand if they want to test regular Solidity contracts, that do not call any precompiles - they can just use the default configuration, there is no need to connect to the Hedera network to test those contracts, they will work the same way on any evm.
However Foundry can be used with the relay for all of it's other use cases. It's possible to run Solidity scripts, deploy and invoke contracts, etc.
We can add a new foundry-example directory with an example configuration and script, and explain in a readme that it's not feasible to use forge test with the relay.

Upon further testing running scripts against the relat is also unfeasible.

We added some extensive examples here https://github.com/hashgraph/hedera-json-rpc-relay/tree/main/tools/foundry-example
@Ivo-Yankov Can we add some limitation quotes to that README and close this ticket?


[REQUEST] Script to add more usb-ethernet devices

Is your feature request related to a problem? Please describe.
There is only a small amount of compatible usb-ethernet devices and they're getting harder to find and very expensive.
Describe the solution you'd like
A script to run in the green menu that adds newer usb-ethernet devices into the device list on the MIB.

On firmware 1367 at least there are more easily findable usb ethernet adapters available: https://github.com/jilleb/mib2-toolbox/issues/119

On firmware 1367 at least there are more easily findable usb ethernet adapters available: https://github.com/jilleb/mib2-toolbox/issues/119


Retrieve bound transaction in $beforeUpdate

Hello,
I'm trying trigger a separate insert (based off the model) whenever an update occurs on a model. Obviously, I need this to be atomic, so I'm looking for a way to bind a transaction to the update and the insert.
Ideally, I'd create the transaction in the $beforeUpdate function and bind it to the current object as well as the additional insert object. However, that seems quite difficult to accomplish with the way bindTransaction works. So, alternatively, I'd like to bind a transaction to the update outside of the model class, and be able to retrieve that transaction object from within the $beforeUpdate function so I can create the insert statement with a bound transaction.
Of course, I realize that I could create the insert outside of the model (in the service in the context of the transaction) but I want to abstract this insert away from the service developer. Any information you can provide to help me accomplish an additional atomic action within the model is appreciated.
Thanks,
Jeff

You can do this:
SomeModel.prototype.$beforeUpdate = function () {
  var PossiblyBoundSomeModel = this.constructor;
  // The `knex` object and the `transaction` object are the same thing. Only difference
  // is that the transaction object holds on to the database connection between queries
  // instead of taking a new one from the pool. This works even if `PossiblyBoundSomeModel`
  // is not bound to a transaction.
  var tranasction = PossiblyBoundSomeModel.knex();
  return SomeOtherModel.bindTransaction(tranasction).query().insert({foo: 'bar'});
};

Does this solve your problem? This doesn't create the transaction in the $beforeUpdate but instead binds the new query to the (possibly) existing one.

Maybe I should add a $transaction() method to model's prototype so that you we could write this:
SomeModel.prototype.$beforeUpdate = function () {
return SomeOtherModel
.bindTransaction(this.$transaction())
.query()
.insert({foo: 'bar'});
};
which is a lot cleaner. What do you think?

I like that syntax, a lot. Does it require that a transaction is bound to SomeModel in the first place? It would be best if this.$transaction would bind itself to this if this does not currently have a transaction. That way the service which invoked SomeModel.update() could be oblivious to the transaction or pass one in if it is part of a greater transaction.
By the way, I still plan on testing your other suggestion, but I'm in the middle of refactoring my code to clean up how we write transactions. Thank you for the responses.

Both of my suggestions would work even if there is no transaction bound to SomeModel, but they don't start a new transaction. In case there is no transaction bound to SomeModel the bindTransaction call would do nothing.
The problem with creating a transaction in the $beforeUpdate (or anywhere else for that matter) by calling the this.$transaction() is the ownership of the transaction Who is responsible for committing or rolling back the transaction? If a transaction is started by this.$transaction(), the original update operation should probably take ownership of the transaction. But what if a transaction already existed? You cannot add a transaction.commit() for example to $afterUpdate method because the update call may be a part of a larger transaction.
Also binding a model to a transaction in-place is not possible, since binding is done by creating a new Model subclass. What if you have 100 requests running in parallel that all create queries using a model class.  Model binding, the way done in objection, needs to be done by creating a new instance (subclass) of a model and binding that instance so that other parallel query chains using the same model are not affected.
I've been thinking about adding a transacting method to the query builder so that one could use transactions just like they are used in knex. I've been avoiding that because I have written some horrible bugs using that style of transaction handling, but the way it is currently done in objection is just as error-prone. Would you prefer the knex-style transaction over objection's way?

Thanks, that explanation makes sense. Definitely a little confusing as to where the commit should occur (though perhaps $afterUpdate could check a status flag to see if this model initiated the transaction... and you still have to find a way to pass the transaction back to the service... it's getting pretty ugly at this point.)
I'm not too familiar with knex's transactions, but looking at the documentation, I don't see much difference between the way transacting works and the way bindTransaction works. Since you're curious, I'll add a few other of my thoughts on transactions at this point:

I initially preferred the callback approach, but I was able to create a scenario that resulted in a non-atomic comi

Thanks, that explanation makes sense. Definitely a little confusing as to where the commit should occur (though perhaps $afterUpdate could check a status flag to see if this model initiated the transaction... and you still have to find a way to pass the transaction back to the service... it's getting pretty ugly at this point.)
I'm not too familiar with knex's transactions, but looking at the documentation, I don't see much difference between the way transacting works and the way bindTransaction works. Since you're curious, I'll add a few other of my thoughts on transactions in objection at this point:

I initially preferred the callback approach, but I was able to create a scenario that resulted in a non-atomic commit. The scenario I remember was an insert followed by an update where the update timed out because of a table lock. In this scenario or a server crash I was able to see the insert commit without the update.
I switched to the transaction object approach, but I found it hard to externalize the inner functions and I hated that I had to keep writing back to local variables. We eventually settled on an approach where each function is externalized and depends on the transaction object being saved to the request object. The functions also share data through request.scope since the computed value is often needed outside of the immediately following function. See below.

request.scope = {};
objection.transaction
	.start(Entity, Company)
	.then(function(transaction) {
		DEBUG && console.log('DEBUG: Initiated transaction.');
		request.transaction = transaction;
	})

	.then(promiseCreateEntity.bind(this, request))
	.then(function(entity) {
		DEBUG && console.log('DEBUG: Created entity: ' + JSON.stringify(entity));
		request.scope.entity = entity;
	})

	.then(promiseReadUsedCodes.bind(this, request))
	.then(function(usedCodes) {
		DEBUG && console.log('DEBUG: Retrieved used codes: ' + JSON.stringify(usedCodes));
		request.scope.usedCodes = usedCodes;
	})

	.then(promiseCreateCompany.bind(this, request))
	.then(function(company) {
		DEBUG && console.log('DEBUG: Created company: ' + JSON.stringify(company));
		request.scope.company = company;
	})

	.then(promiseCreateUserCompany.bind(this, request))
	.then(function(userCompany) {
		DEBUG && console.log('DEBUG: Related user to company: ' + JSON.stringify(userCompany));
		request.scope.userCompany = userCompany;
	})

	// Commit the transaction if everything was successful
	.then(function() {
		var scope = request.scope;
		var validator = revalidator.validate(scope, { properties: {
			company: { type: 'object', required: true }
		}});
		if ( ! validator.valid ) { throw new objection.ValidationError(validator); }

		response.send(201, scope.company);
		DEBUG && console.log('DEBUG: Committing transaction.');
		return request.transaction.commit();
	})

	// If any of the above queries throw an error, rollback the transaction
	.catch(function(error) {
		ERROR && console.log('ERROR: Company creation failed: ' + error);
		response.send(error.status || 500, error);
		if (request.transaction) {
			DEBUG && console.log('DEBUG: Rolling back transaction.');
			return request.transaction.rollback();
		}
		return;
	})
;

// Example promise function which expects the request to contain an authenticated user and a company object in request.scope
function promiseCreateUserCompany(request) {

	var validator = revalidator.validate(request, { properties: {
		user: {
			id: { type: 'integer' }
		},
		scope: {
			company: { type: 'object', required: true }
		}
	}});
	if ( ! validator.valid ) { throw new objection.ValidationError(validator); }

	return UserCompany
		.bindTransaction(request.transaction)
		.query()
		.insert({
			userEntityId: request.user.id,
			companyEntityId: scope.company.entityId,
			order: 0,
			role: 'OWNER'
		})
	;
}

I usually use a very similar approach where I use the request as a scope. There really isn't an easier way to pass forward a large amount of variables using promise chains. This all gets a lot prettier with async/await or generators.
Have you used the bluebird spread method? For smaller chains it is quite handy. You can do stuff like this:
Person
  .query()
  .where('firstName', 'Sylvester')
  .then(function (sylvester) {
    return [
      sylvester,
      sylvester.$relatedQuery('children').insert({firstName: 'Sage'})
    ];
  })
  .spread(function (sylvester, sage) {
    return [
      sylvester,
      sage,
      sage.$relatedQuery('pets').insert({name: 'Fluffy'});
    ];
  })
  .spread(function (sylvester, sage, fluffy) {

  });

Now back to the topic. I will add the $transaction method for model instances and a transaction method for model classes. They can be used to fetch the current model's transaction like discussed above.

Fixed in 470187cc8fc8a5c90ff5489046b3b3f87ab5272b


Missing Persistent Volume and Storage Class Configuration in MySQL-WordPress Tutorial

The current MySQL-WordPress tutorial does not include the necessary steps for creating Persistent Volumes (PVs) and defining the storage class. This can lead to Persistent Volume Claims (PVCs) remaining in a Pending state due to the absence of available PVs with the required storage class.
Create Persistent Volumes (PVs) with the Standard Storage Class:
apiVersion: v1
kind: PersistentVolume
metadata:
  name: mysql-pv
spec:
  capacity:
    storage: 20Gi
  accessModes:
    - ReadWriteOnce
  persistentVolumeReclaimPolicy: Retain
  storageClassName: standard
  hostPath:
    path: /mnt/data/mysql
---
apiVersion: v1
kind: PersistentVolume
metadata:
  name: wordpress-pv
spec:
  capacity:
    storage: 20Gi
  accessModes:
    - ReadWriteOnce
  persistentVolumeReclaimPolicy: Retain
  storageClassName: standard
  hostPath:
    path: /mnt/data/wordpress

Define Persistent Volume Claims (PVCs) with Matching Storage Class:
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: mysql-pv-claim
  labels:
    app: wordpress
spec:
  accessModes:
    - ReadWriteOnce
  **storageClassName: standard**
  resources:
    requests:
      storage: 20Gi
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: wp-pv-claim
  labels:
    app: wordpress
spec:
  accessModes:
    - ReadWriteOnce
  **storageClassName: standard**
  resources:
    requests:
      storage: 20Gi

/language en
/sig docs

Page related to the issue : https://kubernetes.io/docs/tutorials/stateful-application/mysql-wordpress-persistent-volume/
/sig storage

I'm confident that the tutorial needs an update; maybe we should split it into two tutorials:

setting up a storage driver, defining a StorageClass and making an initial PersistentVolume
working with PVCs and deploying a sample workload

How does that sound? This could be quite a bit of work.

+1
Thanks
Deva
On Tue, 18 Jun 2024, 20:23 Tim Bannister, @.***> wrote:

I'm confident that the tutorial needs an update; maybe we should split it
into two tutorials:

setting up a storage driver, defining a StorageClass and making an
initial PersistentVolume
working with PVCs and deploying a sample workload

How does that sound? This could be quite a bit of work.
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/46814#issuecomment-2175971374,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPYD5LVL4DX4NC2MG4LGRDZIARCRAVCNFSM6AAAAABJJN3CNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZVHE3TCMZXGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Stop allowing people to set any port (osc or vmc) to the tracker port

I see people time and time again messing around with the ports and setting for example the osc input to 6969. I think the gui shouldnt allow you to set any port to the tracker port, since doing so will prevent trackers from connecting.

I use other ports to route things for VTubing and Mocap purposes routinely. If anything a warning might be in order, but disabling the ability to set ports is a no-go for me.

no, id still want the port to be settable, just not to the tracker port, since setting any port to be the tracker port makes the server stop accepting any trackers at all. A warning might be fine though. If there really is a need to route something through port 6969, you could still change the tracker port in the config, but i guess just adding a warning might be fine too.


Feature: Show helpful tips/docs

Show hopeful tips:

Hotkeys
Query capabilities

Done in https://github.com/Caspeco/lumberjack/commit/aa50f8046caa7f6d74be7dc9af63c826323d5ee1


defaultProfile setting doesn't work

After update to version 0.11.1121.0 setting defaultProfile doesn't work and always open PowerShell terminal by default.
Environment
Windows 10 Home Build 19592
Steps to reproduce
Edit settings.json to use Ubuntu terminal

Actual behavior
After change defaultProfile to Ubuntu terminal or even cmd by default uses PowerShell.

same here

The Settings format has been changed, and there is no longer a specific "globals" section.

@mdtauk thanks a lot! I moved settings from globals section to root and it works as expected.

Oh, I read that in CHANGELOG... (´・ω・｀)
https://github.com/microsoft/terminal/releases/tag/v0.11.1121.0

BREAKING CHANGE We've deleted a lot of legacy settings handling (#5190)
Please see our blog post and our most recent status update for more information.
If you had settings hanging out in the globals dictionary, you'll want to move them out.
Failure to do so will cause us to ignore your default profile, launch mode, etc., etc.

Thanks for the report! 0.11 introduced some breaking settings changes which is probably causing this issue for you. This is tracked by another issue on our repo - please refer to #5458 for more discussion.
/dup #5458


Add option to share URL to an app (open Share dialog)

It would be great to not only be able to open an URL or save it to the clipboard, but also to be able to open Android's Share dialog, in order to e.g. share the clicked URL directly to someone via a chat app.

i closed your other issues ( #2 #3 ) and i reply for all in this issue.
i'm already planning good ui and everything in single dialog for ux and i will implement soon.
if you enjoying this app and wanna see more improvements you can support me to faster implementations.
i already mentioned my liberapay link in fdroid and my github profile. i won't say no to some drinks. :)

Sounds great, thanks!
Here's an example of what FairEmail's URL dialog looks like:


added setter for subscribers

this will be important in replay, so that we can specifically set who is going to listen to the events to be replayed :older_man:

I don't really see the added benefit of this at the moment.
You can already add listeners, why do we need another method to add them in batch?

The reasons is simple. Once an event bus is configured, all listeners are in place and that is fine. All has been done through configuration and say in a symfony context the container is all compiled and ready to go.
However it is the case in a dynamic environment where you want to disable certain listeners dynamically without having to configure a new symfony environment or hack a compiler pass to rebuild the container just for this change.
This setter then allows you to grab the service and in situ provide the listeners that should exclusively be plugged into the bus such as in a specific replay or partial reconstruction of the read model. The setter is a commodity and will not harm at all the encapsulation of the class.

Not yet.
The point is, that I don't really see the problem. This solution feels a bit like a hack, where you want to reuse an event bus instance, because you don't want to configure/build a new event bus with only the listeners you need. Or am I missing something?

your assessment is correct. However, when you do a replay of events, you publish the stream say sequentially. You would do it with the default kernel, in which you have yeah perhaps the bus that you use commonly and the other bus that you just mentioned that is only for this listener. They will run in parallel unless you shut down the common one. You must shut down but then there is no switch in particular helpful for this, this is the reason you want to ensure either you wipe out the common bus of subscribers and add the listener to the latter bus, or you just wipe the listeners and add this particular listener on that single bus. In either case there is no simple way to publish the stream without affecting the other listeners because you are replaying.

closing as lack of response. I have overridden this already but was trying to contribute. Is cool, just that I don't like PRs lingering if they are rejected.


Mknod variants

Enable creation of a safe character device: /dev/null .
Also add mknodat$null and mknodat$loop .

This is useful for #2380.

What does mean Error: seed=0? https://github.com/google/syzkaller/pull/2439/checks?check_run_id=1881854710#step:4:164

Commit<PHONE_NUMBER>5bdff6f79c9a2a5ce1c9ae7ad05286 (alone) causes VM crashes very often:
panic: runtime error: index out of range [3] with length 3 [recovered]
	panic: runtime error: index out of range [3] with length 3

My mistake…

vm-1: crash: panic: runtime error: index out of range [3] with length 3

Was it crashing somewhere in syzkaller code? Even if you wrote wrong descriptions, it should crash with panic. Can there be something to fix in syzkaller code as well? If so, please post full panic message with stack.

What does mean Error: seed=0? https://github.com/google/syzkaller/pull/2439/checks?check_run_id=1881854710#step:4:164

This link now points to something else for me... To a "2021/02/11 21:51:30 read 0 programs" line in a passed execution. Is it still an issue?

vm-1: crash: panic: runtime error: index out of range [3] with length 3

Was it crashing somewhere in syzkaller code? Even if you wrote wrong descriptions, it should crash with panic. Can there be something to fix in syzkaller code as well? If so, please post full panic message with stack.

It should probably not panic, but it is a mistake in the syscall description. It's simple to reproduce, just remove dirfd fd_dir,  from the mknodat variant.

What does mean Error: seed=0? https://github.com/google/syzkaller/pull/2439/checks?check_run_id=1881854710#step:4:164

This link now points to something else for me... To a "2021/02/11 21:51:30 read 0 programs" line in a passed execution. Is it still an issue?

I fixed the variant description. It is OK now.

FYI, I updated my previous comments with more information.

panic: runtime error: index out of range [3] with length 3
/syzkaller/gopath/src/github.com/google/syzkaller/sys/linux/init.go:150 +0x3c

Ah, I see, it's in sys/linux/init.go. That code is kind of in it's own. It's a bit of a hack and it just needs to be in sync with the descriptions. The panic message  could be better, but ultimately there is not other suitable handling than panicing.

I also noted that there is also /dev/vcs* with the same major and minor numbers as the loop devices. The only difference is the group owner.

I also noted that there is also /dev/vcs* with the same major and minor numbers as the loop devices. The only difference is the group owner.

Are you sure? How kernel would then understand what device to create with mknod?
That's also not what I see on my machine:
$ ls -l /dev/ | egrep "(vcs|null)"
crw-rw-rw-   1 root    root      1,     3 Jan 17 09:18 null
crw-rw----   1 root    tty       7,     0 Jan 17 09:18 vcs
crw-rw----   1 root    tty       7,     1 Jan 17 09:18 vcs1
crw-rw----   1 root    tty       7,     2 Jan 17 09:18 vcs2
crw-rw----   1 root    tty       7,     3 Jan 17 09:18 vcs3
crw-rw----   1 root    tty       7,     4 Jan 17 09:18 vcs4
crw-rw----   1 root    tty       7,     5 Jan 17 09:18 vcs5
crw-rw----   1 root    tty       7,     6 Jan 17 09:18 vcs6

Are you sure? How kernel would then understand what device to create with mknod?

It was for loop and vcs devices, but they are block or character devices, so it doesn't make sense.


Update readme: add a link to a list of cmp sources

Closes #837

Very good contribution!!! Thank you~!