[Event Request] Page 50 "Purchase Order" - procedure OnQueryClosePage()

    trigger OnQueryClosePage(CloseAction: Action): Boolean
    var
        InstructionMgt: Codeunit "Instruction Mgt.";
    begin
        if ShowReleaseNotification then
            if not InstructionMgt.ShowConfirmUnreleased then
                exit(false);

----------------------- ADD EVENT ----------------------
        IsHandled := false;
        OnQueryClosePageOnBeforeConfirmCloseUnposted(IsHandled);
        if not IsHandled then
----------------------------------------------------------

            if not DocumentIsPosted then
                exit(ConfirmCloseUnposted);
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


你好,SM2对于iOS开发真机报错

你好,这是在mac环境下编译后,导入项目,真机运行时报错:
ld: warning: ld: warning: ignoring file /Users/a123456/Desktop/SM2  SM4/libcrypto.a, file was built for archive which is not the architecture being linked (arm64): /Users/a123456/Desktop/SM2  SM4/libcrypto.aignoring file /Users/a123456/Desktop/SM2  SM4/libssl.a, file was built for archive which is not the architecture being linked (arm64): /Users/a123456/Desktop/SM2  SM4/libssl.a
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(CharacterKeyboard.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(NumberKeyboardThird.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(KeyboardTool.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(Keyboard.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(XYKeyboard.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(NumberKeyboardFirst.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(NumberKeyboardSecond.o)) was built for newer iOS version (9.0) than being linked (7.0)
ld: warning: object file (/Users/a123456/Desktop/SM2  SM4/zhlcs/Classes/SDK/XYKeyboard/libXYKeyboard.a(UIView+LVExtension.o)) was built for newer iOS version (9.0) than being linked (7.0)
Undefined symbols for architecture arm64:
"_EVP_sm3", referenced from:
_test_sm2_enc in sm2Plugin.o
"_EC_POINT_point2hex", referenced from:
_test_sm2_enc in sm2Plugin.o
"_EC_KEY_new", referenced from:
-[sm2Plugin sm2:] in sm2Plugin.o
"_EC_POINT_point2oct", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_KDF_get_x9_63", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_CRYPTO_malloc", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_BN_new", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_GROUP_get_order", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EVP_DigestInit_ex", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_POINT_new", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_BN_CTX_new", referenced from:
_SM2_do_encrypt in sm2_enc.o
-[sm2Plugin sm2:] in sm2Plugin.o
_test_sm2_enc in sm2Plugin.o
"_EVP_MD_CTX_create", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_KEY_get0_group", referenced from:
_SM2_do_encrypt in sm2_enc.o
-[sm2Plugin sm2:] in sm2Plugin.o
_test_sm2_enc in sm2Plugin.o
"_EC_GROUP_get_cofactor", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_KEY_get0_public_key", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_GROUP_get_degree", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_POINT_hex2point", referenced from:
-[sm2Plugin sm2:] in sm2Plugin.o
"_BN_rand_range", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_OpenSSLDie", referenced from:
_SM2_do_encrypt in sm2_enc.o
_test_sm2_enc in sm2Plugin.o
"_BN_free", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EVP_DigestUpdate", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EVP_DigestFinal_ex", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_POINT_mul", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_POINT_free", referenced from:
_SM2_CIPHERTEXT_VALUE_free in sm2_enc.o
"_BN_CTX_free", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_BN_num_bits", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_POINT_is_at_infinity", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EVP_MD_CTX_destroy", referenced from:
_SM2_do_encrypt in sm2_enc.o
"_EC_KEY_new_by_curve_name", referenced from:
-[sm2Plugin sm2:] in sm2Plugin.o
"_CRYPTO_free", referenced from:
_SM2_CIPHERTEXT_VALUE_free in sm2_enc.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

静态库不支持arm64

你好,我今天在自己尝试编译arm64位时,报了个这样的错:
e_skf.c:747:7: error: implicit declaration of function 'bind_helper' is invalid
in C99 [-Werror,-Wimplicit-function-declaration]
if (!bind_helper(ret)) {
^
1 error generated.
make[1]: *** [e_skf.o] Error 1
make: *** [build_engines] Error 1

iPhone用不上国密硬件engine，可以直接在e_skf.c所在目录的Makefile中将skf删除。

你好,
EC_KEY *ec_key = EC_KEY_new();
ec_key = EC_KEY_new_by_curve_name(NID_sm2p256v1);
const EC_GROUP *ec_group = EC_KEY_get0_group(ec_key);
BN_CTX *ctx1 = BN_CTX_new();
EC_POINT * point1 = EC_POINT_hex2point(ec_group, pubkey, NULL, ctx1);
ec_key -> pub_key = point1;
SM2_CIPHERTEXT_VALUE *cv = NULL;
cv = SM2_do_encrypt(EVP_sm3(), EVP_sm3(), (unsigned char *)message1, (size_t)strlen(message1), ec_key);
OPENSSL_assert(cv);

SM2加密的流程是这样的么?  用模拟器运行时加密成功了.后来 把e_skf.c删除了,以及在几个文件中添加个别头文件后,armv7 和arm64编译通过了,但是运行时会在
cv = SM2_do_encrypt(EVP_sm3(), EVP_sm3(), (unsigned char *)message1, (size_t)strlen(message1), ec_key);
这里崩掉.拜托指导一下下么?🙏

SM2_do_encrypt传参数错误，请看sm2.h中的定义

哦哦,已经改了,因为这个是我几个月前做的,没注意方法传参都已经改了.这是我改过后,
NSString * message = dic[@"plainText"];
NSString * eckey = dic[@"secretkey"];
const char * pubkey = [eckey UTF8String];
const char * message1 = [message UTF8String];
EC_KEY *ec_key = EC_KEY_new();
ec_key = EC_KEY_new_by_curve_name(NID_sm2p256v1);
const EC_GROUP *ec_group = EC_KEY_get0_group(ec_key);
BN_CTX *ctx1 = BN_CTX_new();
EC_POINT * point1 = EC_POINT_hex2point(ec_group, pubkey, NULL, ctx1);
ec_key -> pub_key = point1;

SM2_CIPHERTEXT_VALUE *cv = NULL;
SM2_ENC_PARAMS * params = NULL;
cv = SM2_do_encrypt(params,(unsigned char *)message1, (size_t)strlen(message1), ec_key);
OPENSSL_assert(cv);

那么这个paramas参数是什么意思啊?该传什么啊?因为没有注释,我也还不是太理解.传空进去会崩.

在SM2_do_encrypt方法中有用到
KDF_FUNC kdf = KDF_get_x9_63(params->kdf_md);
if (!kdf) {
SM2err(SM2_F_SM2_DO_ENCRYPT, SM2_R_GET_KDF_FAILED);
goto end;
}
所以这个params应该怎么取得传过去啊?我在GMSSL文件夹里面也没有找到生成或者初始化SM2_ENC_PARAMS的方法.

好了,解决了,用的5月份下载的GmSSL变异的静态库成功了.因为GmSSL有更新,方法传参,和方法内容有改动,我没发现,造成的崩溃.

@tongyu123 你好，我在当前的master版本下Mac平台编译爆出了一样的问题
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
请问您最终是怎么解决的呢？还有就是如何查看./Configure选项

您好ios怎么代码生成公私钥


Can't access mod's setings

I do not know if I am just plain dumb, but when checking settings/graphics, i see no options to change the mod settings

Try to go into the settings immediately after starting the game. They are not there initially or they disappear for one of the reasons described by lifestarse?

Try to go into the settings immediately after starting the game. They are not there initially or they disappear for one of the reasons described by lifestarse?

Sorry for not answering before, I did not see the message, after re-enabling the mod and restarting the game, the settings menu just appeared, but when resetting to default, it disappears, I think I just missed that detail, oops.

I know about this problem, but when I was developing the mod I did not figure out how to solve it

I think You can move Extended settings from graphics to main settings and
it's not disappear anymore.
On Sat, Apr 15, 2023, 00:30 Ferlern @.***> wrote:

I know about this problem, but when I was developing the mod I did not
figure out how to solve it
—
Reply to this email directly, view it on GitHub
https://github.com/Ferlern/extended-UI-v7/issues/13#issuecomment-1509299138,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN2F75HOSMO5X35UMEBSFQTXBG6ZDANCNFSM6AAAAAAWYLMD2A
.
You are receiving this because you commented.Message ID:
@.***>


Add school

Institution/School Name
Maghull High School
Instiution/School Website
maghullhigh.com
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[x] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[x] Community College or equivalent
[x] High School or equivalent
[ ] Elementary School or equivalent

Hi,
Looking at this institution's online presence, it would appear that this institution does not provide post-secondary education (i.e. College or University)
Referencing the README:

Swot is a community-driven or crowdsourced library for verifying that domain names and email addresses are tied to a legitimate university of college - more specifically, an academic institution providing higher education in tertiary, quaternary or any other kind of post-secondary education in any country in the world.

Is this the case or am I mistaken? If I am please reopen this pull request, and state why your institution should be included in swot.

@ReidWeb Yes, the school provides sixth form (16-18) further education, which is the equivalent to college.

We only consider for inclusion institutions offering degree, foundation-degree or HND (BTEC Level 5), not A-Level/GCE which would equate to BTEC Level 4.
As per ISCEC - level 4 and above would be included.
Sorry, but I have to stick within the constraints of this repository as defined by it's original creator, and we do not have any inclusions below ISCEC level 4, though this is not a strictly written rule. A2/GCE equates to level 3 in ISCEC


fix #27

add missing provider in lexicon responses

Pull Request Test Coverage Report for Build 199

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 85.803%

Totals

Change from base Build 197:
0.02%

Covered Lines:
733

Relevant Lines:
826

💛  - Coveralls


Find in document hilights last typed character even though the find field is empty

I have search existing issues to avoid creating duplicate but this is not reported it seems.

VSCode Version: 1.33.1
OS Version: macOS Mojave 10.14.4 (18E226)

Steps to Reproduce:

Open any document with text/code
Find (Cmd+F) for a word you know exists in document (e.g. bool)
Backspace until find field is blank
The last typed character is still hilighted in all locations

Does this issue occur when all extensions are disabled?: Yes

/duplicate of https://github.com/microsoft/vscode/issues/45211


WLED device not accessible when phone is connected to ESP AP and mobile network at the same time

When the phone is connected to an ESP AP and a mobile network at the same time, the app searches for devices on the mobile network, therefore the WLED devices are never discovered or accessible.
Accessing the WLED devices with a browser on the phone is possible.
When the mobile network is switched off, and the app restarted, access to WLED devices works.

Is your phone iOS or Android?

Hello, Loup-Garou911XD.
I made a web page and then used ESP AP mode to create an access point.I connect that access point with my android phone and can access my web page. But I can't access it with my iphone phone, do you know why? Whether or not this can be solved, I would be very grateful if you could suggest some thoughts, ideas.
@Loup-Garou911XD

Hello, Loup-Garou911XD.
I made a web page and then used ESP AP mode to create an access point.I connect that access point with my android phone and can access my web page. But I can't access it with my iphone phone, do you know why? Whether or not this can be solved, I would be very grateful if you could suggest some thoughts, ideas.
@Loup-Garou911XD

Try visiting <IP_ADDRESS> or whatever you have configured the IP to be in the browser
Also u should have made a seperate issue for this


allow.. receive.. not working with OpenStruct throws Wrong number of arguments

Recently we upgraded ruby from 2.7.3 to 3.0.1 but seems like allow /receive stub on OpenStruct is not working properly. Please check the below code snippet to easily replicate the issue
# frozen_string_literal: true

RSpec.describe 'Test OpenStruct' do
  describe 'OpenStruct failure' do
    let(:struct) { OpenStruct.new(non_working_method: nil) }

    before do
      allow(struct).to receive(:non_working_method)
    end

    it 'should have worked' do
      struct.non_working_method(:testing, 1, 2, 3)
      expect(struct).to receive(:non_working_method)
    end
  end
end

On running this I receive
  1) Test OpenStruct OpenStruct failure should have worked
     Failure/Error: struct.non_working_method(:testing, 1, 2, 3)

     ArgumentError:
       Wrong number of arguments. Expected 0, got 4.
     # ./spec/test_spec.rb:11:in `block (3 levels) in <top (required)>'
     # /Users/sahilbatla/.rvm/gems/ruby-3.0.1/gems/webmock-3.13.0/lib/webmock/rspec.rb:37:in `block (2 levels) in <top (required)>'

Whereas if I change OpenStruct to double it works fine.
What Ruby, Rails and RSpec versions are you using?
Ruby version: 3.0.1
Rails version: 6.1
RSpec version: 5.0.1
Observed behaviour
Same stub works fine if I use double instead of OpenStruct
Expected behaviour
Stubbing OpenStruct should work fine.
Can you provide an example app?
I have shared the code snippet with ruby, rails & rspec config but if required can push a repository for same

So I looked into this a bit, thinking it could be to do with keyword argument changes between Ruby 2.7 and 3.0, however your snippet is incorrect:
 struct.non_working_method(:testing, 1, 2, 3)
expect(struct).to receive(:non_working_method)

Fails (correctly) with:
 (#<OpenStruct non_working_method=nil>).non_working_method(*(any args))
           expected: 1 time with any arguments
           received: 0 times with any arguments

Because you have the expectation after the call, using have_received instead works correctly.
I'm closing for now as I think this is just a confusion with how the expectations work, I'm happy to keep helping, and if you can provide a snippet that fails due to a bug (e.g. if this was a copy pasta mistake when extracting) I'll reopen.


equivalent dotnet property for following for Producer(buffer.memory, max.block.ms, max.request.size) and Consumer(max.poll.records)

Description
I am trying to find equivalent dotnet property for following for Producer and Consumer while creating instance.
But I am not able to find one. Aren't those supported or is there any equivalent property available?
For Producer -- buffer.memory, max.block.ms, max.request.size
For Consumer -- max.poll.records
Please provide the following information:

1.5.3 Confluent.Kafka nuget version.

The consumer supplies one record at a time to the application, there is no equivalent property to max.poll.records.
On the producer, have a look at queue.buffering.max.kbytes and batch.size. There is no equivalent to max.block.ms (I would argue this is an anti-feature).
https://github.com/edenhill/librdkafka/blob/master/CONFIGURATION.md


Add end to end tests for React Navigation integration

Embellishes #1073 to cover off the end to end scenarios for React Navigation.

Pipelines steps:
I wouldn't want to get too hung up on this as it's likely to evolve soon enough as we add more build infrastructure and start to implement multi-stage quality gates.  But I find the steps hard to take in at the moment, which a few naming tweaks could mitigate.  I would suggest:

When referring to React Native, the whole technology, use just RN.
E.g.
Build RN 0.60 apk
Instead of
Build React-native 0.60 apk
When referring to a navigation technology, use the full name, all in lower case.
For e2e test steps, include "end-to-end tests" at the end.
E.g.
:android: react-navigation 0.60 Android 9 end-to-end tests
Instead of
:android: React-navigation 0.60 Android 9

Closed in favour of tests/react-navigation


Tracker sometimes POSTs when no events to post

This is evidenced in the bad rows bucket by payloads with "no events in array" failure.

Re-assigning to @jbeemster to attempt to reproduce. I'm keen we reproduce this locally before attempting any fixes (e.g. #176).


Run with docker and docker-compose

Hey there,
I've added docker-compose wrapper to run your tutorial via docker without impacting local python installation ;)
docker and docker-compose are necessary though

This is great, thanks for setting up Docker! This should make running the code much easier than a local setup.


Cross Project Support

Hi,
We have the dataset in a Project A and would like to run the Bigquery job from Project B. I already shared the Project A dataset to Project B's service account with Data Viewer access. I tested with bq command line and it works as expected. But when I try to run the job using this node module am getting error.
User does not have bigquery.jobs.create permission in project A.

Is there any workaround to specify the job execution Project ID?.

This is a feature I am currently working on.

Hi @steffnay , do you have any update on this issue?.

@satrox28 I opened https://github.com/googleapis/nodejs-common/pull/722 which will allow me to implement this if approved.

@steffnay thank you so much for taking time to work on it.

Hi @steffnay I can see the above project PR is merged. May I know, when can we expect the change to be implemented on this nodejs-bigquery module?.

Hi @steffnay Do you have any update on this request?.


GDPR Requirements

We have to ensure that we are within GDPR requirements for announcing user data that we track.

“‘personal data’ means any information relating to an identified or identifiable natural person (‘data subject’); an identifiable natural person is one who can be identified, directly or indirectly, in particular by reference to an identifier such as a name, an identification number, location data, an online identifier or to one or more factors specific to the physical, physiological, genetic, mental, economic, cultural or social identity of that natural person.”
Article 4(1)

We ask for email, which is a possible identifier. More importantly, searching shows we have to ask for permission before sending any email communications -- I'm still looking for a quote from the GDPR page to back this.
We also ask for country and institutions would could be used to geographically identify someone.

GDPR (in terms and conditions) document was revised to the best of our information.


Implement Server Metrics

Implement the Server Metrics Page as FAB
API: https://docs.hetzner.cloud/#resources-servers-get-2
Icon: fas fa-chart-area
In the Cloud Console Hetzner uses: https://gionkunz.github.io/chartist-js/ for the graphs. Check if it is possible to use the same library for the app.

Is this really needed? @janxb what do you think about it?

Metrics are always a nice thing.. ;) depends on implementation difficulty I think. The API endpoint seems like it‘s straightforward to use.

Done with 8e704e7


DO NOT MERGE! Fast Forward stable to b4676b942a35ab1c62363dadb325a86426ed6ae5.

This PR will be merged automatically once checks are successful.

Jenkins job manager/postsubmit passed


Dev 1.0.0

What is the purpose of the change

Update docs
Update init.sql and init_real.sql in appconn
Update exchangis_ddl.sql in db
Fix some bug

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes)
Anything that affects deployment: (yes)
The Core framework, i.e., JobManager, Server.: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

LGTM.


Validate Test Cases

SignedOff-by :   Oshi Gupta<EMAIL_ADDRESS>Related Issue(s)
Partially addresses #3121

Description
As a part of my LFX mentorship task, I am adding test cases and other manifests for the validate policies
Adding different validate policies for which I have added test cases :
Best-Practices Category

check-deprecated-apis
Note: the above is an updated policy for checking deprecation APIs and not the existing one in this policy repo.

Other Category

advanced_restrict_image_registries
allowed_label_changes
block-images-with-volumes
block_large_images
block_stale_images
check_nvidia_gpu
disallow_all_secrets
enforce_resources_as_ratio
ensure_readonly_hostpath
exclude_namespaces_dynamically
ensure-production-matches-staging
ingress_host_match_tls
limit-hostpath-vols
only_trustworthy_registries_set_root
require_emptydir_requests_limits
require_unique_external_dns
restrict_service_port_range
require_netpol
require_pdb
require_image_source
restrict_service_account
restrict_secrets_by_label

Trafeik Category

disallow_default_tlsoptions

Also, I have moved policies in the folder so that they will not be duplicated in rendering to the website and added the kyverno.io/kyverno-version: 1.6.2 and kyverno.io/kubernetes-version: "1.23" in the polices where it was missing, after checking on the respective versions.
Unique Ingress path policy for Other category has some issue with kyverno CLI , so its test cases are not added in this.
Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.

@oshi36 a couple of things if you please:

There are now conflicts that must be resolved.
Would you mark this as draft until you are finished with adding all necessary tests? This is so it is easier to review one time rather than tracking across scattered commits.

Thank you!

Sure @chipzoller, I will resolve the conflicts.
Also, I have finished adding all the test cases from my side. But as you are suggesting I will mark it as draft.

Looks like tests are failing.

Conflicts to resolve here.

Thank you @chipzoller for this.Looking forward to working more with your guidance.


Add keyboard shortcuts

We want to control kodadot with keyboard shortcuts for much pleasant and faster browsing

g n - go to next page on gallery/collections/collections detail
g p - go to the previous page on gallery/collections/collections detail
g a - go-to own artist profile
g c - go to your collected items
g s - go to your sold items
g m - go-to simple mint
g t - go to transfer
g i - go to series insights
g d - go to creators directory
g r - go to the random page at a particular view

on item detail

e p - expand price chart
e h - expand history chart
e d - expand the description

actions

a b - buy the item
a s - send the item
a c - consume
a l - list

probably everywhere

c c - copy current URL where is used to clipboard and show notification about that

Add a way to recall this manual for shortcuts like overlay, check gmail and smash shift /
Ref

#349
ref gmail https://support.google.com/mail/answer/6594?hl=en

I'll check how we can do this

https://www.npmjs.com/package/vue-shortkey
can we use a package for this?

That's Huuuge

@vikiival @prachi00 You can do keybinding without using that package, just use the appropriate keycodes from https://keycode.info/
An example for using W A S D as arrow keys is: https://stackoverflow.com/a/22766391

v-hotkey can be a good alternative. It has lesser issues, opened PR and just a year old
.

@MubarakSULAYMAN what v-hotkey does is already supported by vue natively, using @keyup

@chandradot99 would be nice to add tooltips where is possible to add to
Like on page previous, next and randomize could add tooltip with a hint in ( ) round brackets

sure @yangwao I will add tooltips as well.


Missing warp match functions

Hi,
As pointed out at https://github.com/ROCm-Developer-Tools/hipamd/issues/65 , match_any/match_all are not available in HIP.
These are available in CUDA (cf. https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#warp-match-functions ), and can be implemented on AMD GPUs on Vega+ architectures (such intrinsic corresponds to "WaveMatch" in HLSL shader model 6.5 https://microsoft.github.io/DirectX-Specs/d3d/HLSL_ShaderModel6_5.html#wavematch-function which is supported by Vega+).
Therefore it seems like they can and should be added.
match_any can for example be implemented as seen at https://github.com/llvm/llvm-project/issues/62477 :
static inline __device__ uint64_t  __match_any(int value) {
  bool active = true;
  uint64_t result = 0;

  while (active) {
    // determine what threads have the same value as the currently first active thread
    int first_active_value = __builtin_amdgcn_readfirstlane(value);
    int predicate = (value == first_active_value);
    uint64_t m = __ballot(predicate); // THIS LINE IS PROBLEMATIC

    // if the current thread has the same value, set its result mask to the current one
    if (predicate) {
      result |= m;
      active = false;
    }
  }

  return result;
}

There used to be compiler bugs making it hard to implement them as with the code above, but they have been fixed.
Feel free to use that code if you want to.
Best regards,
Epliz

@cjatin , you were kind enough to have a look at https://github.com/ROCm-Developer-Tools/clr/issues/2 ; this one is quite similar, so I would appreciate if you could have a look at this one as well :) .

https://github.com/ROCm/clr/commit/c5ab5680b400b5b21e9c8c3849ff67e29afcbfc9

Thanks a lot @cjatin , looking forward to trying it in the next release with it


for macos

For OS X, you need the kind key of "framework".
This patch and adding build.rs as below made compilation successful on Mac.
fn main() {
    let YOUR_CUDA_LIB_PATH = "";
    println!("cargo:rustc-link-search=native={}", YOUR_CUDA_LIB_PATH);
}

There were the following issues with your Pull Request

Commit: 639bf4ac84cbc111f367be660077bc1535d42a97
Commits must be in the following format: %{type}/%{scope}: %{description}

Guidelines are available at https://github.com/autumnai/leaf/blob/develop/CONTRIBUTING.md#git-commit-guidelines

This message was auto-generated by https://gitcop.com

I created https://github.com/autumnai/collenchyma/pull/52 to fix the commit format problem since we haven't heard from @y-ich in a while.

Closing in favor for #52 . Thanks @y-ich !


Ensemble Live in Web/Studio - top two tiles in kitchen sink app are not clickable

login to the studio
go here - https://studio.ensembleui.com/app/e24402cb-75e2-404c-866c-29e6c3dd7992/screen/nAvv0XLuq92N82qpKPCI
Pick any option like Form Widgets or Layout widgets
top two tiles are not clickable, rest work fine

Same issue when running kitchen sink separately in Ensemble Live on the web. I think this has been an issue for a while
cc @amin-nas @vusters

This is fixed now. It is happening only when the build is built with HTML renderer, its working fine with Canvaskit renderer
It's an issue caused by the Lottie widget for the web, the HTMLElementView widget is used for it which is not allowing other views to receive the mouse scroll event.
It can be fixed with a package (https://pub.dev/packages/pointer_interceptor). Already we have a style property called captureWebPointer boolean property for the widgets. Now I added it for the page styles too, here's the PR for it - https://github.com/EnsembleUI/ensemble/pull/1238

@vinothvino42 why are you marking it as released when the PR is still open? This is not released

@vinothvino42 your explanation is hard to understand.

Why did this just start happening when Lottie has been there for at least a year if not more for html renderer?
why is it just blocking the top two tiles?
there is no lottie on those pages?

We can't fix a new regression by adding another flutter package. You have to investigate the issue properly. We can't put band-aids like this.
cc @snehmehta

@vinothvino42 your explanation is hard to understand.

Why did this just start happening when Lottie has been there for at least a year if not more for html renderer?
why is it just blocking the top two tiles?
there is no lottie on those pages?

We can't fix a new regression by adding another flutter package. You have to investigate the issue properly. We can't put band-aids like this.
cc @snehmehta

@kmahmood74 First I need to tell you that I didn't implemented this package, we're already having it https://github.com/EnsembleUI/ensemble/blob/1978c082248a2a017756317a4eaa523ced386a92/pubspec.yaml#L98
and we've setter property for it in the widget controller https://github.com/EnsembleUI/ensemble/blob/1978c082248a2a017756317a4eaa523ced386a92/lib/widget/helpers/controllers.dart#L137
Answers to your questions

Not sure how it worked before, maybe because of canvaskit being used, not sure. The issue is with HTMLElementView which is used now for the Lottie widget if it's HTML Renderer and Web. See here - https://github.com/EnsembleUI/ensemble/blob/1978c082248a2a017756317a4eaa523ced386a92/lib/widget/lottie/web/lottiestate.dart#L71
I tried setting the background color to the lottie widget and tested it navigating to next page, the next page is not receiving the mouse scroll events only for this background lottie widget area. Its working if I scroll or click any other places (ex: in the content widget page or form widget page in ensemble kitchen sink app)
It's in the previous page that absorbs the click or scroll events which is happening in the next page (ex: the content widget page)

See this issues
https://github.com/flutter/flutter/issues/86021
Now I simply added the captureWebPointer style property for the custom widgets that we've in the Ensemble Kitchen Sink Live app widgets section, so now it'll receive the mouse scroll or click events to those widgets. I checked it in Kitchen Sink Ensemble Live and now it's working fine, that's why I marked this issue as Released.
I added the captureWebPointer support for the page styles too, that's what this PR is https://github.com/EnsembleUI/ensemble/pull/1238

@vinothvino42 we can't just ask people to add captureWebPointer: true on all their widgets. That's not right. We need to fix this properly. Also remove this property from all custom widgets in Kitchen Sink after you have a proper fix.
cc @vusters

@kmahmood74 @vusters New PR - #1240

@vinothvino42 get into the habit of closing the tickets once they are complete


Add metrics to bosh-dns

metrics are collected only if they are enabled
metrics are collected for internal and external dns requests
coreDNS metrics plugin is used to collect metrics for
all dns request done by bosh-dns.
there is configuration to enable metrics collection
they are exported locally only

It's also possible to have metrics for cached requests. The CoreDNS cache plugin collects metrics, but there is additional work needed to activate the metrics of of the cache plugin.

@xtreme-conor-nosal thanks for the review. I have updated the pr.

Validated on linux with
dig @<IP_ADDRESS> q-s4.group.network.deployment.bosh
curl <IP_ADDRESS>:53088/metrics

and windows with
Resolve-DnsName -Name q-s4.group.network.deployment.bosh -Server $VM_IP
Invoke-RestMethod -Uri <IP_ADDRESS>:53088/metrics

Thanks @beyhan


504 Error on laravel and bedrock sites after upgrading to macOS Ventura

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
https://gist.github.com/shayanabbas/5f7e4481027cce533824b845c76e1186
Current Behavior
This is the most weirdest issue I have ever had.
I am trying to run my bedrock projects, everything was working fine until I updated to macOS Ventura
After lot of debugging and trying everything it seems like when ever I try to run a laravel and bedrock project the browsers keep loading and show 504 error.
Expected Behavior
it should run out of the box, i am not sure what macOS Ventura might have changed?
Steps To Reproduce
I have tried creating fresh projects of Drupal, Wordpress, Bedrock, Laravel  from Quick Start guide
https://ddev.readthedocs.io/en/latest/users/quickstart/
Drupal and Wordpress (simple) are working and running
where as Bedrock and Laravel project runs successfully but when I visit in browser it keeps loading until it shows a 504 error page.
I tried changing with both nginx and apace webserver configuation in config.yaml
Server is working fine because when I put a index.php file with php info function, it does show that page. but as soon as I change it back to default laravel or bedrock it agains keeps showing 504 error.
I have updated version of ddev and dockers and docker is also has plenty of extra space assigned.
Anything else?
No response

Hi - Since Ventura has not been released, I haven't tried it. I haven't heard any announcement that Docker Desktop supports it either (I see you're using Docker Desktop)
It's interesting that you're successful with Drupal and Wordpress, thanks for the careful testing. And the gist shows that a trivial project also works.
Maybe you can see if ddev logs after you visit the Bedrock/Laravel projects has any clue. And of course, you may want to step through them to see what fails.
Since you're great at trying things out many ways, you may want to try Colima+mutagen as well.

I tried Colima+mutagen it seems to be working after setting up Colima.
thanks

I'd love to know what is wrong on the Docker Desktop side. I guess it will shortly be time to update a machine and try it out. Weird that it only affected those two project types, normally it's all-or-nothing.

that's true, I will try it out with latest updates of MacOS Ventura or ask one of my colleague to try out same scenario.

well i can confirm that this issue is not with Docker Desktop specifically atleast, with running it with Colima+Mutagen setup, I was having some code sync issues with mutagen so i decided to stop it, and run the project with just Colima and that issue again starts, it keep loading the page until it die with 504 error message.

I don't think you'll be happy with colima and not mutagen, because it uses sshfs for mounting, which is too slow; composer install fails, for example.
After it does with 504, please look at ddev logs.

@rfay I have run it and after the error i did ddev logs
[Wed Aug 17 16:41:02.386108 2022] [proxy_fcgi:error] [pid 2544] (70007)The timeout specified has expired: [client <IP_ADDRESS>:58022] AH01075: Error dispatching request to : (polling)
https://gist.github.com/shayanabbas/5825a3747894af631410e956cfc82446

Thanks, I see your ddev debug test was with Colima. Was the failure in https://github.com/drud/ddev/issues/4122#issuecomment-1218261837 also with Colima?
It looks like php-fpm is taking a request and never finishing, which could be because of database or an fpm problem.
If this is with colima it's super interesting that you're having the same problem as with Docker Desktop.
I'll try to upgrade a machine in coming days. It does seem like beta5 should be pretty close to ready.
I'm really surprised that there seem to be no ventura issues in the docker/for-mac queue at this point. Normally there would at least be some. https://github.com/docker/for-mac/issues?q=is%3Aissue+is%3Aopen+ventura+ - also seem to be none in colima queue, https://github.com/abiosoft/colima/issues?q=is%3Aissue+is%3Aopen+ventura
You're absolutely sure that this has to do with the OS version right? You can use the exact same built code/project on a machine that hasn't been upgraded without this problem?

Yes the failure in #4122 was with Colima,
I am not sure if its the OS issue or not but it seems like it, because after the upgrade I run the project and it was having this issue, after that i tried to updated docker and ddev but doesnt worked.
and yes same code/project is working on other devices. I tried with fresh projects for laravel and bedrock it had same issue as well.

Thanks. It sure sounds like a problem. Could you please push the simplest possible demonstration project to a GitHub test repo, with exact instructions on how to recreate the failure in the README?

I have created a repo with details in README https://github.com/shayanabbas/my-wp-bedrock-site

Thanks so much!

Oh, I saw your original ddev debug test was with an amd64 machine, do you have any experience with this on arm64/M1?

No unfortunately not with the M1 variation

I see from the README (thanks) you were testing on beta5. Trying to install now; it wants to give me beta3 first :(

Unfortunately, I'm unable to confirm your results. I installed your demo repo and had no trouble on Ventura beta 5. The only thing was the PHP8.1 deprecations all over the screen.
Version 13.0 Beta (22A5321d)
amd64, 16GB Dual-Core Intel Core i5
Mid-2017 MBP if I remember
My Docker Desktop settings are
Docker Desktop 4.11.1
CPUs: 2
Memory: 6GB
Swap: 1GB
I do occasionally see this with your WP demo site using ddev logs -f (which I recommend):
[18-Aug-2022 01:03:57] WARNING: [pool www] server reached pm.max_children setting (8), consider raising it
I assume that's a result of WP calling back to itself, and may be an argument for increasing pm.max_children, not sure. It also could be a factor in your Laravel situation.
I tried the same repo on a Mac M1 Pro with 5CPUs and didn't see that warning.
Could you please try this on a different machine, consider a Docker Desktop factory reset, and consider experimenting with your Docker Desktop settings?
My Ventura testing with

super weird, I tried with Docker desktop factory reset as well.
Version 13.0 Beta (22A5321d)
amd64, 16 GB 2.3 GHz Quad-Core Intel Core i7
Year 2020
My Docker Desktop settings:
Docker Desktop 4.11.1 (Docker Engine: v20.10.17)
CPUs: 4
Memory: 8GB
Swap: 2GB
after running ddev logs -f
Server started
+ echo 'Server started'
+ unset PHP_IDE_CONFIG
+ exec /usr/bin/supervisord -n -c /etc/supervisor/supervisord-apache-fpm.conf
2022-08-18 08:44:40,608 INFO Included extra file "/etc/supervisor/apache.conf" during parsing
2022-08-18 08:44:40,608 INFO Included extra file "/etc/supervisor/php-fpm.conf" during parsing
2022-08-18 08:44:40,608 INFO Included extra file "/etc/supervisor/conf.d/mailhog.conf" during parsing
2022-08-18 08:44:40,608 INFO Included extra file "/etc/supervisor/conf.d/supervisor.conf" during parsing
2022-08-18 08:44:40,667 INFO RPC interface 'supervisor' initialized
2022-08-18 08:44:40,668 CRIT Server 'unix_http_server' running without any HTTP authentication checking
2022-08-18 08:44:40,669 INFO supervisord started with pid 1
2022-08-18 08:44:41,680 INFO spawned: 'child_exit_monitor' with pid 2518
2022-08-18 08:44:41,709 INFO spawned: 'php-fpm' with pid 2519
2022-08-18 08:44:41,772 INFO spawned: 'apache2' with pid 2520
2022-08-18 08:44:41,785 INFO spawned: 'mailhog' with pid 2521
2022-08-18 08:44:42,889 INFO success: child_exit_monitor entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2022-08-18 08:44:42,890 INFO success: php-fpm entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2022-08-18 08:44:42,890 INFO success: apache2 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2022-08-18 08:44:42,890 INFO success: mailhog entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Thu Aug 18 08:44:42.924065 2022] [ssl:warn] [pid 2529] AH01909: <IP_ADDRESS>:443:0 server certificate does NOT include an ID which matches the server name
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
[Thu Aug 18 08:44:42.949577 2022] [ssl:warn] [pid 2529] AH01909: <IP_ADDRESS>:443:0 server certificate does NOT include an ID which matches the server name
[Thu Aug 18 08:44:43.013086 2022] [mpm_prefork:notice] [pid 2529] AH00163: Apache/2.4.54 (Debian) OpenSSL/1.1.1n configured -- resuming normal operations
[Thu Aug 18 08:44:43.013281 2022] [core:notice] [pid 2529] AH00094: Command line: '/usr/sbin/apache2 -D FOREGROUND'
[Thu Aug 18 08:44:43.966708 2022] [proxy:error] [pid 2543] (2)No such file or directory: AH02454: FCGI: attempt to connect to Unix domain socket /var/run/php-fpm.sock (*:80) failed
[Thu Aug 18 08:44:43.966871 2022] [proxy_fcgi:error] [pid 2543] [client <IP_ADDRESS>:35702] AH01079: failed to make connection to backend: httpd-UDS
[18-Aug-2022 08:44:45] NOTICE: fpm is running, pid 2519
[18-Aug-2022 08:44:45] NOTICE: ready to handle connections
[18-Aug-2022 08:44:45] NOTICE: systemd monitor interval set to 10000ms
[Thu Aug 18 08:50:12.697628 2022] [proxy_fcgi:error] [pid 2545] (70007)The timeout specified has expired: [client <IP_ADDRESS>:44524] AH01075: Error dispatching request to : (polling)

does this show the error?
[Thu Aug 18 08:44:43.966708 2022] [proxy:error] [pid 2543] (2)No such file or directory: AH02454: FCGI: attempt to connect to Unix domain socket /var/run/php-fpm.sock (*:80) failed
[Thu Aug 18 08:44:43.966871 2022] [proxy_fcgi:error] [pid 2543] [client <IP_ADDRESS>:35702] AH01079: failed to make connection to backend: httpd-UDS

Is there anything related to server setup apache/nginx configuration I can do here to resolve this?

Looks like php-fpm is ceasing to respond. You could increase the number of pm.max_children but I'll bet it will just happen again.
I hope you'll be able to try on another machine. You've already rebooted right?
Do you know WP well enough to know whether there's a reason for a single web hit to cause multiple http sessions?

okay so my issue is resolved now I need to test more, but after disabling NFS mount my project starts working
ddev config global --nfs-mount-enabled=false
so maybe you try it on Ventura with nfs mount enable, that might generate same issue on your system?

Thanks, they had a problem with NFS in a previous beta of a previous OS. But now since everybody uses mutagen, it shouldn't show up as much. Glad you caught it!

I'm able to recreate this with your same results. I have no idea why laravel or bedrock would be affected and Drupal9 not.
I used Console.app to watch for nfsd problems and it didn't show any, probably because nfsd just isn't working, it sticks/hangs.
Looks to me like others have had this class of problem, see https://www.reddit.com/r/MacOSBeta/comments/w070z4/ventura_file_sharing_issues/ (but that's all I found)

maybe this will take a little bit of time to acknowledge what's the exact issue.

I just upgraded to Ventura public beta 4 (same as developer beta 6) and I don't see any difference in the behavior. Even though ddev debug nfsmount shows things working fine, and even though ddev start on your test site and ddev describe show it working with nfs, it never responds to web browser, and curl -I https://my-wp-bedrock-site.ddev.site hangs.
I do see the interesting error in console.app
Can't open /var/db/mountdtab: No such file or directory (2)

Tried it with Public Beta 8, same behavior.
I did learn some interesting things though:

The behavior is the same with colima as with Docker Desktop
If I create an index.html in the docroot (web) of the bedrock site, it properly loads the index.html (with both Docker Desktop  and colima, using nfs)

So I think based on the difference between CMSs in this context, that something is actually going wrong with php-fpm and we might be able to put a breakpoint in the index.php and step through until it hangs, and that would probably give us a clue.

I stepped through the WP initialization in PhpStorm. What happens is that when it gets to requiring formatting.php it freezes. formatting.php is a 330KB 6106 line PHP file, and as soon as it gets accessed, everything freezes. I also did a ddev ssh and all actions that try to access the content of formatting.php freeze. For example, in the container, vi formatting.php or wc -l formatting.php or less formatting.php.
So it's an NFS bug in Ventura I would bet.
You don't have to do anything interesting to demonstrate it. Just ddev start (with NFS enabled) and ddev exec wc ./web/wp/wp-includes/formatting.php which will hang.

NFS is still broken on Ventura release, I'm also getting "nfsd send error 40" on a non-ddev Docker compose project.

Seems to work without Docker. I disabled the firewall just in case because now it says "File sharing (SMB)", and I removed host restrictions from my export.
Host:
/Users/karl -alldirs -mapall=501:20

Client:
mount -t nfs hostmac:/Users/karl testmount

Everything works in this case between Macs, but it still doesn't work in Docker.
Snippets from my non-ddev docker-compose.override.yaml (because I'm not sure how ddev's NFS support works under the hood):
services:
  php:
    volumes:
      - type: volume
        source: nfsmount-repo
        target: /var/www/app

        volume:
          nocopy: true

...

volumes:
  nfsmount-repo:
    driver: local
    driver_opts:
      type: nfs
      o: "addr=host.docker.internal,hard,nolock,rw"
      device: ":/System/Volumes/Data/Users/karl/the-path-here" # Also tried /Users/karl

In my most recent test I tried with Colima too, so it's likely not a docker-desktop-specific problem. And that makes me think it's a plain macOS problem. But it could possibly be a docker-ce problem with NFS on Ventura.

It is strange that it works cross-Mac with no Docker involved, so maybe it really is a docker-ce problem?
I have a helper script that does composer install inside this container so the right version of PHP is being used, and it also hangs on "Generating optimized autoload files".

I'm having a hard time reproducing this with a simple test case unfortunately... small docker-compose files that do some NFS mounts and install some composer packages all work fine. I'm pretty stumped.

Would you mind trying it out with the original WordPress case? In https://github.com/drud/ddev/issues/4122#issuecomment-1272707930 I found that it hung on particular files in bedrock, using the bedrock recipe from ddev quickstarts.

I confirm the same problem using bedrock install from https://ddev.readthedocs.io/en/latest/users/quickstart/#bedrock and nfs_mount_enabled: true.

#4122 (comment) was a great clue! Sorry I didn't read your debugging, it's definitely a file size issue. I've got a plain Docker reproduction.
If you can reproduce I'll open an issue in docker/for-mac.
Clone this repo and change the volume path to match your own in docker-compose.yml: https://github.com/karlshea/docker-nfs
Reproduction
Setup
In one shell launch the container: docker-compose up
Docker NFS
In another shell:
docker compose exec php php /var/www/app/small.php should print out "small"
docker compose exec php php /var/www/app/large.php should hang (or print out "large" if it works for you I guess).
Plain NFS
On another Mac, mount that same directory over NFS.
cat small.php works, as does cat large.php.

Well this is weird, I added small.sh and large.sh to that repo and they both work fine (docker compose exec php /bin/bash /var/www/app/large.sh), while PHP hangs. I wonder if it's an issue with NFS and the PHP images?

 In that repo running docker compose exec php php /var/www/app/test.php is a PHP script that just reads the large.sh file contents into a var and it works, even though it's the same size file as the large.php script which hangs.

Found this after encountering issue with docker NFS and wordpress and large plugins, disabling NFS fixed it, but is super slow but we are using 'ranger desktop'
so it starts to look like MacOs Ventura NFS is not working well with docker.

It does seem like most people have moved on to mutagen, ddev config global --mutagen-enabled, which is way faster than NFS. But I wish NFS wasn't broken in Ventura.

Added a docs warning about this in

https://github.com/drud/ddev/pull/4343

It does seem like most people have moved on to mutagen, ddev config global --mutagen-enabled, which is way faster than NFS. But I wish NFS wasn't broken in Ventura.

Mutagen is faster but has downsides in the amount of data which is used on the hard drive.
It would be good if both options are available.

Yes, on the NFS side; the problem reproduces when trying to iterate over a directory that has large number of files.
For example, listing somewhere between 800 and 900 files sometimes causes issues to reproduce; ls 3000 will also likely hang.
function create_files()
{
	test_file="${1:-1000}"
	mkdir -p $test_file
	for i in $(seq 1 $test_file); do touch $test_file/$i; done
}

create_files 800
create_files 900
create_files 1000
create_files 1500
create_files 2000
create_files 3000

The problem feels like any large packets cannot be sent from nfsd to the hosts on the bridged network, as cannot even cat or md5sum large files.

This is fixable in the short term with this ddev/docker-compose.nfs.yaml - you have to look at existing .ddev/.ddev-docker-compose-base.yaml to find the name (change from ddev-bedrock_nfsmount), the IP address for the o: line, and the device (change from /System/Volumes/Data/Users/rfay/workspace/bedrock)
volumes:
  nfsmount:
    name: "ddev-bedrock_nfsmount"
    driver: local
    driver_opts:
      type: nfs
      o: "addr=<IP_ADDRESS>,hard,nolock,rw,wsize=32768,rsize=32768"
      device: ':/System/Volumes/Data/Users/rfay/workspace/bedrock'

You also have to delete the NFS Docker volume so it can be recreated with those new options.

ddev stop deletes the volume.

@rfay I tried using the HEAD version of DDEV I have correctly setup the NFS
if i do ddev debug nfsmount i get following error, even nfs is setup correctly
NFS does not seem to be set up yet, see debugging instructions at https://ddev.readthedocs.io/en/stable/users/install/performance/#debugging-ddev-start-failures-with-nfs_mount_enabled-true
Details: error=failed to StartContainer: API error (500): error while mounting volume '/var/lib/docker/volumes/testnfsmount/_data': failed to mount local volume: mount :/System/Volumes/Data/Users/username/work/generogrowth:/var/lib/docker/volumes/testnfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock,wsize=32768,rsize=32768: stale NFS file handle
output=

if i run project I get this error
Container ddev-ssh-agent  Started
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Using nginx snippets: [/Users/username/work/bedrock/.ddev/nginx/uploads-redirect.conf]
Custom configuration takes effect when the container is created.
Should something go wrong and no effect be visible use 'ddev restart'.
Pushed mkcert rootca certs to ddev-global-cache/mkcert
Network ddev-bedrock_default  Created
Volume "ddev-bedrock_nfsmount"  Created
Container ddev-bedrock-dba  Started
Container ddev-bedrock-db  Started
Error response from daemon: error while mounting volume '/var/lib/docker/volumes/ddev-bedrock_nfsmount/_data': failed to mount local volume: mount :/System/Volumes/Data/Users/username/work/bedrock:/var/lib/docker/volumes/ddev-bedrock_nfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock,wsize=32768,rsize=32768: stale NFS file handle
Failed to start bedrock: ComposeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-bedrock docker-compose -f /Users/username/work/bedrock/.ddev/.ddev-docker-compose-full.yaml up --build -d', action='[up --build -d]', err='exit status 1', stdout='#1 [drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built internal] load build definition from Dockerfile
#1 transferring dockerfile: 591B 0.0s done
#1 DONE 0.0s

#2 [drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built internal] load .dockerignore
#2 transferring context: 2B done
#2 DONE 0.0s

#3 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built internal] load build definition from Dockerfile
#3 transferring dockerfile: 922B done
#3 DONE 0.0s

#4 [drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built internal] load metadata for docker.io/drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision
#4 DONE 0.0s

#5 [drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built 1/2] FROM docker.io/drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision
#5 DONE 0.0s

#6 [drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built 2/2] RUN (groupadd --gid 20 "username" || groupadd "username" || true) && (useradd  -l -m -s "/bin/bash" --gid "username" --comment '' --uid 501 "username" || useradd  -l -m -s "/bin/bash" --gid "username" --comment '' "username" || useradd  -l -m -s "/bin/bash" --gid "20" --comment '' "username" || useradd -l -m -s "/bin/bash" --comment '' username )
#6 CACHED

#7 [drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built] exporting to image
#7 exporting layers done
#7 exporting layers done
#7 writing image sha256:17ebb57d443ad4cc838f6b172c39c592ed0decf77a8af1a85a421b3860bdd501 done
#7 naming to docker.io/drud/ddev-dbserver-mariadb-10.2:20221026_uid_collision-bedrock-built done
#7 writing image sha256:b6ab343e2508683c8599ff9a80ea52d4fca3f328be048bec9ebe85a614a30b4f done
#7 naming to docker.io/drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built done
#7 DONE 0.1s

#8 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built internal] load .dockerignore
#8 transferring context: 2B done
#8 DONE 0.0s

#9 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built internal] load metadata for docker.io/drud/ddev-webserver:20221031_healthcheck_no_profile
#9 DONE 0.0s

#10 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built 1/4] FROM docker.io/drud/ddev-webserver:20221031_healthcheck_no_profile
#10 DONE 0.0s

#11 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built 2/4] RUN (groupadd --gid 20 "username" || groupadd "username" || true) && (useradd  -l -m -s "/bin/bash" --gid "username" --comment '' --uid 501 "username" || useradd  -l -m -s "/bin/bash" --gid "username" --comment '' "username" || useradd  -l -m -s "/bin/bash" --gid "20" --comment '' "username" || useradd -l -m -s "/bin/bash" --comment '' username )
#11 CACHED

#12 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built 3/4] RUN export XDEBUG_MODE=off; composer self-update --stable || composer self-update --stable || true; composer self-update --2 || composer self-update --2 || true
#12 CACHED

#13 [drud/ddev-webserver:20221031_healthcheck_no_profile-bedrock-built 4/4] RUN mkdir -p /home/username && chown username /home/username && chmod 600 /home/username/.pgpass
#13 CACHED
', stderr='Network ddev-bedrock_default  Creating
Network ddev-bedrock_default  Created
Volume "ddev-bedrock_nfsmount"  Creating
Volume "ddev-bedrock_nfsmount"  Created
Container ddev-bedrock-web  Creating
Container ddev-bedrock-dba  Creating
Container ddev-bedrock-db  Creating
Container ddev-bedrock-dba  Created
Container ddev-bedrock-db  Created
Container ddev-bedrock-web  Created
Container ddev-bedrock-dba  Starting
Container ddev-bedrock-db  Starting
Container ddev-bedrock-web  Starting
Container ddev-bedrock-dba  Started
Container ddev-bedrock-db  Started
Error response from daemon: error while mounting volume '/var/lib/docker/volumes/ddev-bedrock_nfsmount/_data': failed to mount local volume: mount :/System/Volumes/Data/Users/username/work/bedrock:/var/lib/docker/volumes/ddev-bedrock_nfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock,wsize=32768,rsize=32768: stale NFS file handle'
Instrumentation is opted in, but SegmentKey is not available. This usually means you have a locally-built ddev binary or one from a PR build. It's not an error. Please report it if you're using an official release build.

@shayanabbas - Please open a new issue, or come to Discord for support. In either case, please provide the results of ddev debug test in a gist on gist.github.com (give a link) and also give the results of going through the macOS NFS Debugging section in https://ddev.readthedocs.io/en/stable/users/install/performance/#nfs
The most likely situation is that you just don't have working NFS (run the script again, check your /etc/exports, etc)


Long Press Also Executes Press

My understanding is that the library would distinguish between a click and a hold, not perform both actions when the button is held, which is what is happening.

Sorry, I was using setTapHandler instead of setClickHandler,


Fixed NPE in ExpressionHandler

Fixes #252: ScopeManager::getCurrentRecord() is declared as @Nullable, so one should not assume that it is safe to call getThis() on it.

@oxisto @vfsrfs This looks like a better quickfix to me, mostly because it is quicker. We can look at how to improve on https://github.com/Fraunhofer-AISEC/cpg/pull/274 for the final integration of templates


Fix fairseq api changes towards SUPERB-ST task

The ST is developed on f2146bd, but fairseq's api has changed which results in a mismatch to the dataset class (e.g., no tokenize_text function in the dataset class)

Many thanks @ftshijt and @bearhsiang !


Fix SortedDict deprecation

Port master change to v4

Can you rebase your change please?

Did it merged here: https://github.com/aljosa/django-tinymce/commit/7bda704b7217ced416a9ab48217801f94242234d


TC-41

created unit tests for getAllShifts, findShift, login, updateExtras, and clockOut middleware
tidied up middleware

Merging in TC-41.


Write contribution guidelines

A lot of projects have a CONTRIBUTING.md file and now that we have some PRs (yay! :smiley: ) we should have some guidelines.

I'd love to get some feedback on the contribution guidelines. Any takers? =)


I want to know there is a threeD table in Mainm?

I want to make a video about three dimension  DP problem. So I'm finding the three dimension table. So is the feature in Mainm? please.

duplicate of https://github.com/ManimCommunity/manim/discussions/3758

Duplicate


add video resources from youtube for git github and open source

pls assign

hi sir please assign this task to me.

please assign me


CSV - Unsupported beautification result in Osx (no tested in windows)

Description
The results of beautification are not what I expect.
Input Before Beautification
This is what the code looked like before:
file name: "a.csv"
Year,Make,Model
1997,Ford,E350
2000,Mercury,Cougar

Expected Output
The beautified code should have looked like this:
A beautiful .csv file.
Actual Output
The beautified code actually looked like this:

Steps to Reproduce

Add the csv file
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: debug.md
Another information
Unsupported beautification result 'Year,Make,Model,1997,Ford,E350,2000,Mercury,Cougar'.
Unsupported beautification result 'Year,Make,Model,1997,Ford,E350,2000,Mercury,Cougar'.
Hide Stack Trace
Error: Unsupported beautification result 'Year,Make,Model,1997,Ford,E350,2000,Mercury,Cougar'.
at beautifyCompleted (/Users/User/.atom/packages/atom-beautify/src/beautify.coffee:140:17)
at tryCatcher (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/util.js:16:23)
at Promise._settlePromiseFromHandler (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/promise.js:512:31)
at Promise._settlePromise (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/promise.js:569:18)
at Promise._settlePromise0 (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/promise.js:614:10)
at Promise._settlePromises (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/promise.js:693:18)
at Async._drainQueue (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/async.js:133:16)
at Async._drainQueues (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/async.js:143:10)
at Async.drainQueues (/Users/User/.atom/packages/atom-beautify/node_modules/bluebird/js/release/async.js:17:14)
at
Checklist
I have:

[ x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[ x] Reloaded (or restarted) Atom to ensure it is not a caching issue
[ x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[ x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[x ] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

Same issue here. Running Atom over Ubuntu.


New animation component doesn't work for clone object

When we clone an object with anim state component, basically this.entity.anim.setTrigger('Run'); function doesn't work for cloned object.
I tested this behavior with script clone and also editor clone, both doesn't work. Function only sets original object's anim state.
Expected behavior : this.entity.anim variable should set current object's animation evaluator.
In this examples, both characters has animation state component and test.js script.
Test.js script basically triggers "Run" animation when you press "Space". You'll see that in example, only first and original character triggers animation properly.
Project editor link : https://playcanvas.com/editor/scene/1052339
Project demo link : https://launch.playcanvas.com/1052339?debug=true

Hi @devcem, thanks for highlighting this issue! It looks like the same stateGraph parameters are being used by both entities. I'll create a PR to resolve this issue shortly.


Change MRLA route53

Code has been planned and will work with production OK.

Change was in another request (https://github.com/ministryofjustice/modernisation-platform/pull/3833) which has been merged. Closing this PR


fix(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da to v1.7.5

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da
module
patch
v1.7.4 -> v1.7.5

Release Notes

terraform-ibm-modules/terraform-ibm-watsonx-saas-da (github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da)
v1.7.5
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cos/ibm to v8.14.1 (#206) (c83a017)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

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

GitHub release
v2.5.19

Your semantic-release bot :package::rocket:


[W5][M11-3] Dannica Ong

hi @dannong
interesting add of person’s gender but needs further debugging


Configure start up application

Hey, great project - Enjoying it so far, thank you! :D
I would like to make my HTPC a Gaming + Media center, and as such want to launch Kodi first, with the option to launch Steam from within Kodi.

Why Kodi first?
Kodi supports HDMI-CEC and I plan on using this to control Kodi using my TV remote using this CEC Adapter.
If I don't plan on playing games, I don't want to have to use my game controller

If this is possible, how can I configure the start up behavior?

Create a .conf file under ~/.config/environment.d and set the STEAMCMD environment variable to whatever you want to launch.
I don't expect this will work very well though.

Alternatively you can run steam and ask it to launch a specific gameID. take a look at cli arguments for the steam client.


Add support for minlength/maxlength/regularexpression attributes on array and dictionary properties

Support minlength/maxlength/regularexpression attributes on dictionary and array properties
Add JsonSchemaPatternPropertiesAttribute on dictionary properties
Improve nullability checks
Add DefaultDictionaryValueReferenceTypeNullHandling setting (default: NotNull)

Closes #991
Closes #992

Do the PR builds publish a nuget package anywhere? I can validate this if they do

Pushed release
https://rsuter.visualstudio.com/NJsonSchema/_releaseProgress?releaseId=49&_a=release-pipeline-progress
to
https://www.myget.org/feed/Packages/njsonschema


[Feature Request] Improve Cloud Console with Statistics and Reporting

Describe the solution you'd like
Currently the Maestro Cloud Console for uploads is pretty basic. All we as users have is the ability to do some simple filtering.
We would love the ability to be able to see some statistics at-a-glance, such as pass/fail % rate by month/day/app etc, or even something as small as the time it's taken to run all of the tests. Maybe with some more granular reporting abilities, like seeing which tests are the most prone to failure. This would really help us to identify if our tests are flaky and need more work, as well as demonstrating to our stakeholders the kind of value our tests are giving to the company.
If this is something that would take a long while to implement, then just allowing us to export bulk data as xml/csv from the Console would be great too, as we could import this into our own dashboards and monitoring.

Hey, thanks for sharing your ideas and thoughts with us.
We do not have any plans to build this right now, unfortunately.


[Feature] Adding light/dark theme

Description
Addin the dark theme and light theme option will enhance the user experience
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Congratulations, @sharmi4590! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Hey @sharmi4590 Nice issue but IDK we can go ahead with this as of now :/


dev: compute some statistics about validation and execute steps per transactions

context: Ilia from Starkware is pushing to change core constants in the next Starknet mainnet version, 0.13.2. The update is scheduled for July and the decision will take place mid June. We want this update to allow Kakarot to be deployed and operate well on Starknet mainnet and testnet.
such constants are: step limit per transaction, divided between validate and execute steps; block size, block lifespan, and calldata limit per transaction.
goal

compute max steps in edge cases validate scenarios (biggest calldata in an L1 transaction, big contract deployment, etc)

compute max steps in edge cases scenarios for execute step

bonus: compute median steps, mean steps
results:
ability to give with good confidence the constants that will be unchangeable for the entirety of 0.13.2 and ability for Kakarot to operate on Starknet mainnet
we want to avoid cases where Kakarot is deployed on Starknet mainnet but unpredictable certain operations are impossible (same example as when we discovered addLiquidity of Uniswap was 7m steps > 4m steps current per tx limit)
note: need scoping cc @ClementWalter

Validate
Validation is bounded by the amount of calldata sent with the transaction. When running validation, we need to

Unpack the 31-bytes packed calldata into individual bytes
RLP-decode the transaction
Keccak-hash the tx to re-compute the message hash
verify the signature of the message hash

Steps 1,2,3 are dependant on the amount of data sent with the transaction. Considering that a contract has a 24kB size limit, and that deploying a UniswapV2Router contract of size 18kB takes 800k steps, we could extrapolate that in order to deploy the biggest contract possible we would ~1.1M steps would be enough for the validation step.
On the other hand, if we consider that the Ethereum block gas limit is 30M and that each non-zero byte of calldata consumes 16gas, one could send up to 1.8MB of calldata and the theoritical requirement would be 75M steps for a transaction that big - but this is not a common use case that we aim to support.

Validate
Validation is bounded by the amount of calldata sent with the transaction. When running validation, we need to

Unpack the 31-bytes packed calldata into individual bytes
RLP-decode the transaction
Keccak-hash the tx to re-compute the message hash
verify the signature of the message hash

Steps 1,2,3 are dependant on the amount of data sent with the transaction. Considering that a contract has a 24kB size limit, and that deploying a UniswapV2Router contract of size 18kB takes 800k steps, we could extrapolate that in order to deploy the biggest contract possible we would ~1.1M steps would be enough for the validation step.
On the other hand, if we consider that the Ethereum max calldata size is 1MB, the theoritical requirement would be ~50M steps for a transaction that big - but this is not a common use case that we aim to support.

Last step, as discussed, is to check the "sweet spot" for supported max KB in a single transaction. The tension is between DoS risk on the __validate__ step on the Cairo side vs. widespread support for EVM transactions.
Can we check the median, or P99 (is that the correct word?) for L2 transactions?

Analysis of ~1500 optimism transactions. 99.99% of have less than 50kb of calldata, which would pass validation of 2M.
The most calldata-intensive seem to be related to seaport

Closing since becoming off topic with #1246


multiple UAC elevation prompts during updates

ℹ Computer information

PowerToys version: 0.27.1 (while updating to 0.29)
PowerToy Utility: installer
Running PowerToys as Admin: yes
Windows build number: [run "winver"] 19042.685

📝 Provide detailed reproduction steps (if any)

allow PowerToys to update itself

✔ Expected result
Since PowerToys is already running as Admin, there should be no UAT prompts... if it's not, the installer should trigger a single UAT prompt
❌ Actual result
the uninstall step triggers a UAT elevation, then the install triggers another, then (if it was running as Admin before the update) launching triggers a 3rd prompt!
📷 Screenshots

this work should be fixed with the large work item #10126 and tentatively resolved in .49 release


Modify TokenRegistry contract

Simplify registration to one tx instead of two and prevent registering the same token twice.

Couple of things worth considering with these changes:

I believe it was a past intent to in some way limit the registration of tokens to either deploy time or some sort of access control (see tokenRegistry in https://github.com/thehubbleproject/hubble-contracts/issues/584). For example, anyone might be able to request a token registration but only an owner/admin could finalize the registration.
With these changes (and even before), anyone can register a token to a Hubble network. This could be nice as it allows anyone to onboard at token at their own purview/cost, but also means more scammy/less reputable tokens can also be registered which could have impacts on the reputation of the network. If having token registration fully open is a concern, I would at the very least recommend adding @openzepplin/contracts's Ownable to the TokenRegistry and adding the onlyOwner modifier to the registerToken.

I have no strong opinion on either your changes or the access control schemes mentioned above as they all have their tradeoffs. Since you are the most active devs I will leave it at your discretion. Current changes look good :+1: assuming the above has been considered.

Hey @jacque006, thanks for your answer.

Current changes look good 👍 assuming the above has been considered.
// yes I believe we need to protect tokenregistry and spokeregistry
// this PR does only simplifies the already open token registration
// the other change is making sure that the same token is not registered twice which ma

Hey @jacque006, thanks for your answer 👍

Current changes look good 👍 assuming the above has been considered.

I agree that we should to protect TokenRegistry (and SpokeRegistry similarly), so that possibly-malicious tokens are not added arbitrarily. Optimally I think it should be the responsibility of a DAO to accept/reject token registrations.
We're not there yet though, so this PR is just meant to simplify the token registration to a single tx - it is open to anyone now and will still be. The second, more important change in this PR is preventing repeated registration of a token contract - currently finaliseRegistration can be called multiple times for the same token.
I suggest merging this PR now and tackling the issues that you mention in the future.


Cmake failed in Windows 10

Hi. I am trying to compile bareflank on windows 10. I followed the instructions in the readme(Install vs 2019 and wdk etc, but not cygwin). When I execute cmake ..\hypervisor, I got the following error:
`CMake Error at C:/Program Files/CMake/share/cmake-3.16/Modules/ExternalProject.cmake:2962 (get_property):
get_property could not find TARGET libcxxabi_x86_64-vmm-elf.  Perhaps it
has not yet been created.
Call Stack (most recent call first):
C:/Program Files/CMake/share/cmake-3.16/Modules/ExternalProject.cmake:3239 (_ep_add_configure_command)
scripts/cmake/macros.cmake:408 (ExternalProject_Add)
scripts/cmake/depends/libcxx.cmake:55 (add_dependency)
scripts/cmake/macros.cmake:188 (include)
CMakeLists.txt:111 (include_dependency)
-- Extension: C:/Users/Ne0/Downloads/hypervisor-master/hypervisor-master/examples/cpuidcount
-- Extension: C:/Users/Ne0/Downloads/hypervisor-master/hypervisor-master/examples/hook
-- Extension: C:/Users/Ne0/Downloads/hypervisor-master/hypervisor-master/examples/rdtsc
-- Extension: C:/Users/Ne0/Downloads/hypervisor-master/hypervisor-master/examples/vpid
CMake Error at scripts/cmake/macros.cmake:975 (message):
ENABLE_BUILD_VMM is not supported on Windows
Call Stack (most recent call first):
scripts/cmake/validate.cmake:93 (invalid_config)
CMakeLists.txt:198 (include)`
I modified scripts/cmake/config/default.cmake to make it support vs 2019, but it still failed. Is it possible to compile it on Windows without cygwin? Any help is appreciated.

No. You must have cygwin as we cross compile which is not supported by MSVC


Adapted parallel::{count|count_if} for Ranges TS (see #1668)

Extends the count and count_if algorithms for Ranges.
Proposed Changes

Add projection function to count, count_if
Add range version of the above functions (#1668)
Added corresponding tests

@cogle could you please add the new files to the build system for the documentation as well (here: https://github.com/STEllAR-GROUP/hpx/blob/master/docs/CMakeLists.txt#L115) ?

What is a possible reason for failing this test case?
$ docker run -v $PWD:/hpx -w /hpx ${IMAGE_NAME} ./build/bin/inspect --all --output=./build/hpx_inspect_report.html /hpx

It appears I failed the following test
docker run -v $PWD:/hpx -w /hpx ${IMAGE_NAME} ./build/bin/inspect --all --output=./build/hpx_inspect_report.html
Just wondering what may be the source of this failure may be.

inspect is a small tool that checks for things like formatting issues or missing includes. In your case it's complaining about trailing whitespace. You can see the report here. There are links at the bottom of the report to the offending lines.
In general this report should be available under artifacts on circleci, but you have to be signed in to see it.

What are some possible reasons for failing the pycicle daint-clang-6.0.0 Test that I could look into?

@cogle Actually at the moment even we can't tell what went wrong in that one because the results are not viewable. Most likely the failed test is completely unrelated to your changes but once things are back to normal clicking on details should get you an idea of what's failed.

Ok thanks! I was wondering if I was like with circleci missing a way to view the results.


Fixes #241 and other

here's a fix for #241 included. @gwainer can you check that it also covers your issue?
i hope i didn't miss anything, the solution was not as straight-forward as finding the problem. there's been a blind spot concerning the affects of other rules so far.
why did i end up renaming the cerberus module? well,

i wanted to run the tests on Pyston and thus needed an invokable script. (the result is surprisingly only one fail that may be a bug in Pyston's exception system.) unfortunately it doesn't work with pytest yet.
i noticed at this point that the imports all don't follow PEP8 which favor absolute imports and fixed it.
suddenly it looked javaish (from cerberus.cerberus import x) and was ambigous (from cerberus.import x), hence the renaming

there's no rush to merge as i will have a look at the incorrect error representation from the initial post.

Just tested it @funkyfuture, and it solves the problem I mentioned in #241. Nice work! Thanks.

it's ready to merge

Excellent.


KAFKA-7670: Admin client testUnreachableBootstrapServer() flaky test

This test easily fails locally around once every 20-30 runs.
java.util.concurrent.ExecutionException: org.apache.kafka.common.errors.TimeoutException: Timed out waiting for a node assignment.

at org.apache.kafka.common.internals.KafkaFutureImpl.wrapAndThrow(KafkaFutureImpl.java:45)
at org.apache.kafka.common.internals.KafkaFutureImpl.access$000(KafkaFutureImpl.java:32)
at org.apache.kafka.common.internals.KafkaFutureImpl$SingleWaiter.await(KafkaFutureImpl.java:89)
at org.apache.kafka.common.internals.KafkaFutureImpl.get(KafkaFutureImpl.java:262)
at org.apache.kafka.clients.admin.KafkaAdminClientTest.testUnreachableBootstrapServer(KafkaAdminClientTest.java:277)
at sun.reflect.GeneratedMethodAccessor1.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:298)
at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:292)

After spending considerate time debugging, I found out that the weakly-consistent iterator of futureResponses does not iterate through the two responses at all in the failing runs.
No elements are ever removed from futureResponses except in the MockClient#reset() method which I verified is never called. I think the cause is the weakly-consistent iterator of Java which states:

they are guaranteed to traverse elements as they existed upon construction exactly once, and may (but are not guaranteed to) reflect any modifications subsequent to construction.

I'm still unsure how to solve this in the cleanest way. I'm opening this PR as a chance to discuss it with other people

You mean that we add a method to the collection while it's being iterated and clear the whole collection even though we never processed some of the elements?

@stanislavkozlovski There is a timing window in the way metadata response is prepared because send of the metadata request can occur on the admin client network thread before the response is added on the main test thread. But metadata requests should be retried and we would expect this to work on retry. So perhaps the issue in the failing case is that retry is not occurring, so may be worth checking if it is retrying and if not, why it is not retrying.

MockClient is not thread-safe. So we can't safely call MockClient#setUnreachable after the admin client is started and it starts sending metadata requests. I think in the failing scenario, setUnreachable is disconnecting after a metadata request is sent by the admin client, but we are not retrying because the disconnect response is lost (due to concurrent disconnect/poll on a class that is not thread-safe). I think we need to either rewrite MockClient#setUnreachable to make it thread-safe or rewrite the test so that it creates the admin client only setUnreachable is invoked on MockClient. Rewriting the test is probably the simpler option.

@ijuma No, we don't ever clear the collection. I only found out that while iterating through it in some runs we never find any elements inside.
@rajinisivaram I didn't know that, thanks. I can't reproduce this failure when I add more code for debugging purposes - it just doesn't fail. It's unclear to me why that is. I wonder if it is worth it to change the code such that we instantiate the AdminClient on demand
Update: @rajinisivaram just responded as I was writing this. I'll go with the approach of instantiating AdminClient

This looks good to me. Would it be less error prone to make the AdminClientUnitTestEnv take a MockClient in the constructor (pre-prepared), instead of relying on the user to create the env, then prepare the MockClient, then initiate?

@lbradstreet I guess you meant take in a AdminClient? That would result in way more boilerplate I'm afraid.
You're right that this is a bit more error prone. My thinking is that it should be caught in the first test run and as such maybe not have as big as an impact. I don't have strong feelings about this though

@stanislavkozlovski, you're probably right that it would be significantly more boilerplate. Do all prepare statements have to take place prior to the initiate call? Perhaps those could go through the admin client env, to ensure proper ordering. I don't feel too strongly about it though - what you've done seems fine overall.

@rajinisivaram can you please review these changes?

@stanislavkozlovski I found a simpler explanation for the transient failures. Basically MockClient.poll() is not safe for concurrent sends. Previously we did this:
        List<ClientResponse> copy = new ArrayList<>(responses);
        ClientResponse response;
        while ((response = this.responses.poll()) != null) {
            response.onComplete();
        }
        return copy;

I changed it to this:
        List<ClientResponse> copy = new ArrayList<>();
        ClientResponse response;
        while ((response = this.responses.poll()) != null) {
            response.onComplete();
            copy.add(response);
        }
        return copy;

After this change, I can no longer reproduce the failure. Of course it's debatable whether we should allow concurrent sends, but these tests currently depend on it.

@hachikuji agreed this seems like a cleaner solution. I also tried and failed to reproduce after using your suggested change. Thanks a lot!


Introduce BugPattern for removing duplicate Mockito.verifyNoInteractions() calls

Problem
When verifying that mocks didn't have interactions in tests sometimes many calls to Mockito.verifyNoInteractions() are used
verifyNoInteractions(mock1);
verifyNoInteractions(mock2);
which unnecessarily pollutes the code, because this method actually accepts varargs.
This can be simplified to just:
verifyNoInteractions(mock1, mock2);
Description of the proposed new feature

[x] Support a stylistic preference.
[x] Avoid a common gotcha, or potential problem.
[ ] Improve performance.

We should rewrite multiple calls to verifyNoInteractions to one call.
I would like to rewrite the following code:
verifyNoInteractions(mock1);
verifyNoInteractions(mock2);
...
verifyNoInteractions(mockN);

to:
verifyNoInteractions(mock1, mock2, ..., mockN);

Considerations
Tricky parts in my opinion are:

That there might be a variable number of mocks verified in this way.
That multiple individual calls might not be consequential.
That the scope we need to check this must be just inside one test method.

Participation

[x] I am willing to submit a pull request to implement this improvement.(I'm willing to try to submit a pr, this will be my first encounter with error prone 🙂)

Nice idea @Kamil-Gabaydullin! I think important to consider is where we put the verifyNoInteractions that combines all the mocks.

I'm willing to try to submit a pr, this will be my first encounter with error prone :slightly_smiling_face:.

Sounds good, let us know if you have any questions. We can definitely have a chat about it.
I started thinking about this check and can write out my train of thoughts :smile:. Let me know if it is clear or you have different ideas, we're definitely open to that as well 🙂. A rough idea on a possible implementation direction:

Create an Error Prone BugChecker that matches on MethodTrees.
For a given MethodTree determine whether it is a test method using the MoreJUnitMatchers#TEST_METHOD.
If so, traverse the MethodTree using a TreeScanner (a relatively unrelated example of such a scanner is in the LexicographicalAnnotationAttributeListing). The scanner should find all MethodInvocationTrees that are calling verifyNoInteractions.
Next up, collect the mocks that are passed as parameter.
Insert a statement that combines all the mocks in one invocation (or insert them in the last occurrence of verifyNoInteractions.

We can also discuss implementation details IRL if you'd prefer that 😄. It can sound a bit daunting at first.

Should we also add verifyNoMoreInteractions?


Chat: make the broadcast cooldown actually work

There's a repetition cooldown in canTalk though.

What exactly does this actually fix?

It lets the check at l715 work, in the current code this.broadcastMessage isn't defined by that point.

The cooldown here exists in every room, stopping you from broadcasting the same message as the last message broadcast within 20 seconds.

Okay but can you not move the location of the canTalk check? I'm afraid that's going to break other things.

I don't see what that'd break unless you had some very strange chatfilters, but sure, updated

Anyone can make whatever chatfilter they want, so those "very strange chatfilters" may very well actually exist.
(I'm pretty sure they do actually exist, too.)

Chatfilters are supposed to be supported as message logging: If a chatfilter doesn't return false, the message is guaranteed to be output to the room.

Fair enough

Looks like a lot of users are annoyed by this affected !roll etc.

We should maybe add an exception or something.

Agreed, I tried at #4594


ref: Drop symfony/polyfill-uuid in favour of a standalone implementation

As we only use one single method of symfony/polyfill-uuid, I'm in favor of removing this as a dependency and implementing the method directly in the SDK.
The code is based on symfony/polyfill-uuid, hence the @copyright annotation.

What's the benefit of this? Every code we have in this package is code that we have to maintain, and UUID implementations are one of those things that are likely to have bugs because of weak algorithms

I think that dropping some dependencies makes sense, maybe this is not the most obvious one but it's also yet another dependency for some very simple piece of code that generates random IDs. The SDK docs also play nice in on that: https://develop.sentry.dev/sdk/philosophy/#dependencies-cost. The chance is slim but this polyfil might clash with user defined global functions and constants so having our own implementation based on a maintained one makes sense here.
Looking at the actual ID format (from: https://develop.sentry.dev/sdk/envelopes/#prerequisites):

UUIDs are declared as either 32 character hexadecimal strings without dashes ("12c2d058d58442709aa2eca08bf20986"), or 36 character strings with dashes ("12c2d058-d584-4270-9aa2-eca08bf20986"). It is recommended to omit dashes and use UUID v4 in all cases.

We might take the opportunity to generate the UUID without the dashes in the first place instead of stripping them later removing a little string stripping around the code.

I think that dropping some dependencies makes sense, maybe this is not the most obvious one but it's also yet another dependency for some very simple piece of code that generates random IDs.

Not all code that is a few lines long is also simple. In particular, generating UUIDs is something that if done incorrectly leads to bugs because there is not enough randomness or entrophy, and that's the reason why a lot of libs exist to help people generating them. Moreover, we are importing the code of this package but we are not testing it properly besides ensuring that it returns a string in the correct format, which honestly means nothing in this case, and this is one more thing that once we do we have to maintain. I think that we have greater problems to solve or features to add than changing things that honestly bring 0 real impact to the end-user, so it would be wiser to not reinvent the wheel every time, even more since we're not in like in the JS SDK where every byte count. :relaxed:

@ste93cry The main benefit here is getting rid of one dependency. In this specific case, we're using a single method from this library. The code in question wasn't updated in the last two years, so I consider this as a good sign that it works well and is stable.
As we're still using the exact same code as we did before, I do not worry about bugs introduced by this.
Given the complex nature of having to support multiple PHP versions at once, fewer dependencies are a good thing.
I also like the idea of @stayallive in terms of already providing the correctly formatted uuid.
Nothing major, but a small improvement at least.

Given the complex nature of having to support multiple PHP versions at once, fewer dependencies are a good thing

Sure, but there are libraries and libraries, and certainly a polyfill providing a few functions is not the one that will change this complexity.

The code in question wasn't updated in the last two years, so I consider this as a good sign that it works well and is stable

Sure, but to reach this stability it took quite a few bug fixes and changes, and if a new bug gets discovered in the future we won't even know about it because we now have this code as part of our package. If you told me that it's time to ditch Httplug I would happily answer "yes!", but trying to drop a polyfill just for the sake of "having one less dependency" is something I can't get really see the benefit at all. This dependency is a polyfill for the ext-uuid extension which provides an optimized way (at work we are using it for example) to generate UUIDs, so if people have it installed they can take advantage of native UUID generation. Last, this package is so common that it's probably you will have it installed anyway on your system because some other dep is requiring it, especially if those deps are Symfony or Laravel


STRA

Describe the issue
A clear and concise description of what you want to happen.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
Add any other context, attachments or screenshots
Definition of done
Identify what will need to happen/be delivered for this to be completely done

This seems like an epic - it should be broken into smaller subtasks,


getting 502 on submitting a query

I have been trying to use the instance at stalker.zense.co.in and all of my queries redirect to a error 502 from cloudflare.

The project is live here


QC metrics are not of numeric type.

The QC metrics should be all of numeric type. If not detected, there should be a None value instead of a sting 'not detected'.
web       | qc1_peptide_charges                                                 object
web       | N_qc1_missing_values                                                object
web       | reporter_intensity_corrected_qc1_ave                                object
web       | reporter_intensity_corrected_qc1_sd                                 object
web       | reporter_intensity_corrected_qc1_cv                                 object
web       | calibrated_retention_time_qc1                                       object
web       | retention_length_qc1                                                object
web       | N_of_scans_qc1                                                      object
web       | qc2_peptide_charges                                                 object
web       | N_qc2_missing_values                                                object
web       | reporter_intensity_corrected_qc2_ave                                object
web       | reporter_intensity_corrected_qc2_sd                                 object
web       | reporter_intensity_corrected_qc2_cv                                 object
web       | calibrated_retention_time_qc2                                       object
web       | retention_length_qc2                                                object
web       | N_of_scans_qc2                                                      object
web       | N_of_Protein_qc_pepts                                               object
web       | N_Protein_qc_missing_values                                         object
web       | reporter_intensity_corrected_Protein_qc_ave                         object
web       | reporter_intensity_corrected_Protein_qc_sd                          object
web       | reporter_intensity_corrected_Protein_qc_cv                          object

if I change to None value for not detected peptides, this assertion should throw an error!?
https://github.com/LewisResearchGroup/lrg-omics/blob/bf05475d9042b91c99ecbf887f515e0c737bcf44/tests/proteomics/test__quality_control_maxquant.py#L256

I changed the assertion because the previous one was not working (https://github.com/LewisResearchGroup/lrg-omics/pull/73/commits/3172d9bee7f6fe0883f72b04e590f12176224831)... I still keep "not detected" in case the QC peptides are not detected since changing the values in the dictionary to None will cause this assertion to throw an error... we can decide later on how to define these cases where the intensity or other values for the QC peptides are NaN...


Update camera awesome dependency to fix conflicting classes

Description
Conflicting class from the flutter SDK and Camera Awesome package i.e. CarouselController.
Fixes #(issue number)
Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please describe):

Checklist
Please review the following list before submitting your pull request:

[ ] I have read the contributing guidelines
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented on my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)
If applicable, add screenshots to help explain your changes.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Additional Information
Linked Issues
If this pull request is related to one or more issues, please list them here:

Closes # (issue)
Related to # (issue)

Thank you for your contribution! Please review the checklist and ensure that all items are completed before submitting your pull request.

Can you please merge this PR, we really need it?

Hi @HenrySarukh, can you please update the following details?

Checklist
How Has This Been Tested?
Test Configuration

Hi @GhagSagar23!
Truly, I have not tested it, so I cannot provide you with more information, but I know that many people really need these changes, that would be very nice if you do smth with it, thanks in advance!

No problem @HenrySarukh, I'll do the testing part and release a new version.

Hi @HenrySarukh, I am not able to checkout in your branch. I'll do the same changes and publish it.

Hey @GhagSagar23!
Okay, no problem!
For your knowledge, the camerawesome package is using carousel_slider ^4.2.1 version, which is incompatible with newer Flutter versions, it is why that change is crucial for many people.

Hi @HenrySarukh, the code was merged and deployed in this PR. Also, it is published on pub.dev with version 0.0.1+5.

@GhagSagar23 Thank you very much!


Added sample for DLP : De-identify table data using conditional logic and replace with infoTypes.

Implemented sample for De-identify table data using conditional logic and replace with infoTypes.
Added test cases for the same
Reference: https://cloud.google.com/dlp/docs/examples-deid-tables.md#transform_findings_only_when_specific_conditions_are_met_on_another_field

Combined this PR into https://github.com/GoogleCloudPlatform/php-docs-samples/pull/1837


Update scalafmt

Updates scalafmt and reverts removal of return function type in streamProbesWithOffsetFactory.
Resolves: https://github.com/apache/incubator-pekko-connectors-kafka/issues/14

merged because upcoming package name changes are going to cause conflicts