Add Slack reference to the "Get more help" section in docs

In the API docs, there is a "Get more help" section at the end of every command. Currently it points users to Stack Overflow, IRC, etc. We should add Slack as a help/support option there.
/cc @chipotle @mglukhovsky

https://rethinkdb.com/api/ruby/
I think this issue has already been resolved for all the official drivers.

Yep look like it. Thanks for pointing this out @rchordiya .


Add Ghidra role

Installs Ghidra in kafl root folder
Helps to fix #75

@il-steffen Ci is passing ✔
Ready for review


Avatar Configure - options for bone retargeting

Planned but not sure how possible it is in practice, seems unity doesn't support this very well unfortunately.

Avatar configure tool has been replaced


[Mixer] unit tests fail for go test ./mixer/...

Mixer tests only work with bazel currently.  Mixer tests should not require bazel and should be compatible with dep ensure.  For example:
go get -u github.com/golang/dep/cmd/dep
cd $GOPATH/src/istio.io/istio; dep ensure
go test ./mixer/...

Bazel has significant memory needs and some circleci jobs fail due to OOM errors

seems failing on tools/codegen tests, possibly some expected filepath is different among bazel and go test?  @guptasu FYI
But, I think we will keep using bazel as for our canonical build tool. I'm not sure if "flaky-test" label can apply here.

This is related to https://github.com/istio/istio/issues/1578.
I have added  PR #1682 to run code coverage that includes running all tests except the one in the above issue.

bazel dependencies have been removed and go test ./mixer/... works now


How to run the examples?

Hi,
I am a newbie for the rust. Can you please help me with how to run the example and test it?
Currently I am getting following error with "rustc":
$ rustc examples/prepare_batch_execute.rs
error[E0463]: can't find crate for cdrs
--> examples/prepare_batch_execute.rs:2:1
|
2 | extern crate cdrs;
| ^^^^^^^^^^^^^^^^^^ can't find crate
error: aborting due to previous error
For more information about this error, try rustc --explain E0463.
Thanks,
Mahesh

I found way to add the external library using -L option, but still there is compilation issue:
$ cargo build
Compiling version_check v0.1.4
Compiling antidote v1.0.0
Compiling cfg-if v0.1.4
Compiling byteorder v0.5.3
Compiling libc v0.2.43
Compiling byteorder v1.2.4
Compiling log v0.4.3
Compiling uuid v0.6.5
Compiling scheduled-thread-pool v0.2.0
Compiling lz4-compress v0.1.0
Compiling lazy_static v1.1.0
Compiling rand v0.4.2
Compiling time v0.1.40
Compiling r2d2 v0.8.2
Compiling snap v0.2.4
Compiling cdrs v2.0.0-beta.1 (file:///home/jioadmin/Work/rust_testCode/cdrs)
Finished dev [unoptimized + debuginfo] target(s) in 41.33s
jioadmin@jioadmin-HP-ProBook-440-G3:~/Work/rust_testCode/cdrs$ rustc -L target/debug examples/prepare_batch_execute.rs
error[E0460]: found possibly newer version of crate byteorder which cdrs depends on
--> examples/prepare_batch_execute.rs:2:1
|
2 | extern crate cdrs;
| ^^^^^^^^^^^^^^^^^^
|
= note: perhaps that crate needs to be recompiled?
= note: the following crate versions were found:
crate byteorder: /home/jioadmin/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/lib/libbyteorder-9f1eca6b0d25955b.rlib
crate cdrs: /home/jioadmin/Work/rust_testCode/cdrs/target/debug/libcdrs.rlib
error: aborting due to previous error
For more information about this error, try rustc --explain E0460.

@maheshjena1
Try cargo run --example example_file_name. For instance, cargo run --example crud_operations to run https://github.com/AlexPikalov/cdrs/blob/master/examples/crud_operations.rs
Feel free to re-open this issues if you're still facing this issue.

thank you Alex, it indeed solved my issues.


Add docstring to custom files and functions

Is your feature request related to a problem?
Currently, there are plenty of function without docstring. Before these lines the pylint is set disabled like: # pylint: disable=missing-function-docstring.
As a product of this, the generated doc has not so much content. Here is a snapshot:

Describe the solution you'd like
It would be good the add some description about the role of the functions and its parameters, additionally a little description about the module.
Here is a doc that can help in the syntax of the docstrings: https://sphinx-rtd-tutorial.readthedocs.io/en/latest/docstrings.html#the-sphinx-docstring-format
https://www.sphinx-doc.org/en/master/usage/restructuredtext/domains.html#cross-referencing-python-objects
Describe alternatives you've considered
Additional context

Worth noting that none of that common module is intended to be user API so we should probably exlude them from the doc entirely or otherwise label it non-API.


Fix Intel DDC Swift optimization issue

Ok, we don't have to rush this. We could even have the 3.0.0 final unoptimized and have this in 3.1.0 only but I feel the release would be more complete with this resolved.
@the0neyouseek - if you have time to test this one, please add your review and let me know if it can be merged! Thank you!

Hey @waydabber !
I'm sorry I won't have time to test this till next week cause I'm still away on holiday

@the0neyouseek - ok, no problem! I merge it then and hopefully it works.
@JoniVR - maybe we could have a 3.0.0 final tagged temporarily as pre-release and ask for some additional feedback from those who are signed up to this issue and then change the designation to release if everything seems to work out?


32 bits version

Would it be possible to also have a 32 bits version of libdovi?

I don't want to provide 32 bit builds.
You can try building it yourself.

I'll close this, let me know if you need help building.

Yes, perhaps l you could describe what I’d need to do to build libdovi for 32 bits.

Am 18.08.2022 um 17:08 schrieb quietvoid @.***>:

I'll close this, let me know if you need help building.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

It's in the readme: https://github.com/quietvoid/dovi_tool/tree/main/dolby_vision
The rest is the usual Rust compilation setup, and building for the i686-pc-windows-msvc target.


idea  to lower docker image size

We could test those idea to lower the docker image size and reduce the download's time.
compress the kernel inside the docker image and decompress it on the user's computer.
compress all the dependencies and decompress and build the docker image on the user's computer.
The idea behind it, is that it's faster to decompress and build a docker image on the computer than download a full build version and the kernel.

When you update the docker image, it is pushed to the docker hub and everybody will automatically download the last version when MLfood.py runs. If you compress the kernel, each compilation run on a new container will have to decompress.
Sending one full working image will be more efficient than compress to send faster but decompress in each compilation will slow down a lot.

But what if we can test if we have an update, and rebuild the image in that case?
We won't lose so much time, because the image will be rebuilt once, and not for every compilation.
Yes, it can be bothersome when using dev image, but for global user, you will use prod image, ie an image who will not be change a lot of time.
Also, I think that it's faster to rebuild the image if needed : for low quality internet, it's usefull, because downloading at 50ko/s take more than 30min to do so, when you will probably spend less if you build it internally.
So I personnaly think that this idea could be tested.
For backward compatibility (and if it's successful), it can be offer as an option in the command line.
But netherless, we need to know if we can check if yes or no we really update the underlying image.
If yes, it can be an option. If not, we possibly can forgot that idea.

We talked about that, we tried to force a git pull at each compilation in the "dev" image only, but we stop that way of update because the pull became too long when the image was updated ( when it was developped, maybe now it can be a good solution )

Based on our discussions, here is the idea:

not decompressing the Linux kernel archive (do it at runtime, it should save ~600Mb)
not installing upfront all Debian packages (saving some spaces, we need to quantify it), but rather install deb at runtime

Overall, we will save some spaces and the Docker image will be much lighter. We will have a "small" overhead since we need to decompress and install deb packages
As a concrete task:

we need to rewrite TPDIM.py as well as Docker files to make it more understandable, usable, avoid some strange steps, and take our improvements into account
we need to adapt the Python scripts (eg for decompressing the archive) side note: the time needed to decompress/install should not count as part of compilation time
we need to test test test for ensuring that (1) we have the same environment (gcc version, libc version, etc.) (2) it does compile and send all information to the database (3) quantify a bit the "overhead"


Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

Git will help collaboration and ease version control.


PMID:16278445

decreased cytoplasmic translation initiation during cellular response to oxidative stress
decreased cytoplasmic translation initiation during cellular response to osmotic stress
abolished protein phosphorylation during cellular response to hydrogen peroxide

decreased cytoplasmic translational initiation during cellular response to oxidative stress FYPO:0006230
decreased cytoplasmic translational initiation during cellular response to osmotic stress FYPO:0006231
abolished protein phosphorylation during cellular response to hydrogen peroxide FYPO:0006232
edit file: 884374b24346e939e1e868297533dc49cc5fc21f
release: f08be34c1aaa2f3a32bc54e8fdcd703d89407fe4


NamedResourceSetRequest

Didn't see any docs around this:
TaskRequest.java {
     public NamedResourceSetRequest(String resName, String resValue, int numSets,
                                   int numSubResources) {
      this.resName = resName;
      this.resValue = resValue;
      this.numSets = numSets;
      this.numSubResources = numSubResources;
    }

....
}

Is it for the mesos agents --attributes ?

Amazing work! Thanks for hand holding me! I should have read the docs, I instead downloaded the source code and was playing around w/ the tests. hehe.


Rename 2017_11_04_000000_alter_post_nullable_fields_table.php to 2016…

…_11_04_000000_alter_post_nullable_fields_table.php
Timestamp was in the future.

It should be April 11th, 2017, since that's when it was added.  Please change the date stamp to 2017_04_11 instead of 2016_11_04.
This will cause issues with those that have already run the migrations, so we have to be careful with these changes.

Good point. I've changed the filename.


THRIFT-5693 - fix bug in serialization of enum default values

This change fixes a bug where the default values set in enum fields were unable to be serialized. Previously default enum values were being stored as enum objects upon initialization, where Thrift generally expects I32 values. With the changes in this pull request, serialization and deserialization are fixed and the python 3.4 style enums continue to work as expected.
JIRA ticket here: https://issues.apache.org/jira/browse/THRIFT-5693

[x] Did you create an Apache Jira ticket?  (Request account here, not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

we actually don't have python set up in github actions yet so this is not really tested (we only tested that it did not break other languages that's setup in github actions)
can you show the diff between old and new compiler generated python code?

Gotcha, here's a diff of the ThriftTest.thrift generated Python:
https://gist.github.com/andrewcunnin/802f4d8fa779c181e2272b5219b52a07

For the testing, is running the cross test and running make -k check in test/py sufficient? Or were you referring to a different set of tests that I should be running separately?

For the testing, is running the cross test and running make -k check in test/py sufficient? Or were you referring to a different set of tests that I should be running separately?

we have tests under both test/py and lib/py/test, and then we also have cross tests between different language libraries.


[QA] Emotes menu is overlapping the Wearables menu

Explorer Alpha build versions:
v0.25.3
Issue Description:
When the user presses the key B+E , the wearables and the emotes menu is overlapping each other.
STR:

Open v0.25.3
Press B + E consecutively (but not at the same time)
Validate that you are able to see the menus overlapping each other

Expected behaviour:
The emote menu should not appear on the wearables section
Current behaviour:
Emotes menu is overlapping the Wearables menu
Evidence:

https://github.com/user-attachments/assets/053021dc-87a3-4c6f-a8f2-eccd9a953edf
Additional Notes:
Windows 11
RAM 16GB

Give this a shot
https://bit.ly/4eaAnWN
Passcode: changeme
you must have a gcc compiler

It does not close the emotes wheel when the backpack menu is shown before the appearing animation of the wheel has finished. It's like it only configures the action of closing the wheel after the animation.


feat: new hook useAsync

What new hook does?
Checklist

[x] Have you read contribution guideline?
[x] Does the code have comments in hard-to-understand areas?
[x] Is there an existing issue for this PR?

#33

[x] Have the files been linted and formatted?
[x] Have the docs been updated?
[x] Have the tests been added to cover new hook?
[x] Have you run the tests locally to confirm they pass?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


tf.nn.log_poisson_loss outputs NaN

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):2.1.0
Python version:3.7.6
Bazel version (if compiling from source):N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version:N/A
GPU model and memory:N/A

Describe the current behavior
tf.nn.log_poisson_loss outputs NaN if log_input and targets contain large values
Describe the expected behavior
Expect a grace exception message if the input is invalid instead of Nan as output
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np
tf.nn.log_poisson_loss(log_input=np.array([1e+38], dtype=np.float32), targets=np.array([1e+38], dtype=np.float32))

Output:
<tf.Tensor: shape=(1,), dtype=float32, numpy=array([nan], dtype=float32)>

I am able to replicate the issue reported on tf 2.4 and nightly, please find the gist here

This is an expected result from floating point approximation. Checking for NaNs on every operation would result in unacceptable performance degradation and isn't consistent with other floating point libraries.
You can use tf.math.is_nan to check if a result is nan.

Ideally this should return infinity.  Currently
tf.nn.log_poisson_loss(log_input=np.array([100], dtype=np.float32), targets=np.array([100], dtype=np.float32))
tf.Tensor([inf], shape=(1,), dtype=float32)

But doesn't seem like a high priority and contributions welcome!

@kkimdev
What is the expected output that we are looking for here?

Hi @DNXie  @kkimdev !checked this issue is give "inf" output when compiled with float64  (used tensor instead of numpy array ) providing gist for reference


Guru Meditation Error: Core  0 panic'ed (Cache disabled but cached memory region accessed).

Hello Team,
I've tried to run the MQTT and Greengrass demos, and both worked well.
But when trying to write my own application using the demo's functions I've got errors.
My project structure is as the following:

amazon-freertos
freertos-conigs
src

main.c

CMakeLists.txt

In the main, I've tried 4 scenarios.
1- Called the MQTT'S DEMO function -RunCoreMqttMutualAuthDemo()- directly in the main.
2- Created a task to run the MQTT'S DEMO function.
3- Called the Greengrass DEMO function directly in the main -discoverGreengrassCore().
4- Created a task to run the Greengrass DEMO function.
The first case was the only one to succeed. The others usually fail after connecting to the wifi.
The logs for each attempt.
greengrass-direct-call-in-main.txt
greengrass-task.txt
greengrass-task2.txt
mqtt-fn-direct-call-in-main.txt
greengrass-mqtt-direct-call-in-main.txt
mqtt-task.txt
I always get the error "Guru Meditation Error: Core  0 panic'ed (Cache disabled but cached memory region accessed)." in all the cases.
- For the greengrass task attempt in the logs it fails here:
[Greengrass] [aws_greengrass_discovery.c:165] [INFO] [Greengrass_Discovery_Demo] GG

Re-enable cpu cache.
Guru Meditation Error: Core  0 panic'ed (Cache disabled but cached memory region accessed).

- Then when I tried to run it without a task it kept resetting until it ended as the following:
14 3779 [main] [aws_greengrass_discovery.c:165] [INFO] [Greengrass_Discovery_Demo] GG

15 3780 [main] Unexpected error<PHONE_NUMBER>) from dns_gethostbyname_addrtype() while resolving (xxxxxxxxxxxxxxx-ats.iot.eu-west-1.amazonaws.com)!
16 3781 [main] ERROR! Failed to resolve host address: ServerHost=xxxxxxxxxxxxxxx-ats.iot.eu-west-1.amazonaws.com
17 3782 [main] LwIP connect fail -1 118

18 3782 [main] ERROR! SOCKETS_Connect call failed: ServerAddress=0, Port=64288, ReturnCode=-1

19 3783 [main] About to close socket.

20 3783 [main] Socket closed.

21 3784 [main] Stack high watermark for discovery helper task: 1388.

22 3784 [main] JSON request could not connect to end point

23 3785 [main] [aws_greengrass_discovery.c:177] [INFO] [Greengrass_Discovery_Demo] GGD_JSONRequestStart

24 3786 [main] [aws_greengrass_discovery_demo.c:615] [ERROR] [Greengrass_Discovery_Demo] Auto-connect: Failed to retrieve Greengrass address and certificate.

- When I use the greengrass without a task, in the logs it fails here:
[main] [aws_greengrass_discovery.c:165] [INFO] [Greengrass_Discovery_Demo] GG

E (9909) PKCS11: failed nvs get file size 4354 0

abort() was called at PC 0x40080fe0 on core 0

Here is the app_main() function.
the-main-fn.txt
Please let me know what could be causing all of this, and if there is a better way to follow to use these functions in my main.
System information

Hardware board: ESP32
Operating System: [ Windows ]

From a first glance, the main function looks okay. Can you share your code, which I can use to repro it?
Also, can you not directly edit the main function in https://github.com/aws/amazon-freertos/blob/main/vendors/espressif/boards/esp32/aws_demos/application_code/main.c#L127? Alternatively, you can use esp-idf and our standalone libraries to write your application.
Thanks.

Thank you for your response.
Well, we haven't edited the main function here directly:
https://github.com/aws/amazon-freertos/blob/main/vendors/espressif/boards/esp32/aws_demos/application_code/main.c#L127
We created another main.c file in an external src directory and used CMake to compile and flash. Following the instructions here to create our own application.  https://docs.aws.amazon.com/freertos/latest/userguide/getting_started_espressif.html#getting_started_espressif_cmake_project
Here is how my project's structure looks like.

amazon-freertos
freertos-conigs
src

main.c

CMakeLists.txt

We didn't edit or use the demo's main. We tried to imitate it and created another external one, trying to create our own app. If there is another way using esp-idf. Please guide me on how to do that.

If there is another way using esp-idf. Please guide me on how to do that.

You can consume esp-idf and our libraries in your application. Here is one layout to achieve that:
/
├─ esp-idf (submodule)/
├─ libs/
│  ├─ coreMQTT (submodule)/
│  ├─ Device-Shadow-for-AWS-IoT-embedded-sdk (submodule)/
│  ├─ Device-Defender-for-AWS-IoT-embedded-sdk (submodule)/
│  ├─ ota-for-aws-iot-embedded-sdk (submodule)/
│  ├─ ...
├─ app/
│  ├─ application_src1.c
│  ├─ application_src2.c

As we are using greengrass we are not able to use the AWS IoT Device SDK for Embedded C.
So we need to use AFR.
The issue is that this error Guru Meditation Error: Core 0 panic'ed (Cache disabled but cached memory region accessed appears when trying to run our application as described above.
To reproduce
Steps to reproduce the behavior:

Flash and monitor

Code to reproduce the bug
main.zip
Additional context
Amazon's demo working fine.
Thank you.

Hello @BasemTarek,
I see that you are using vTaskDelay in app_main. I am not sure that it is correct to do so since the scheduler is not running. This should not cause the issue that you are facing - however, it would be better if you remove the delay calls.

Removing all the calls for vTaskDelay in the main maybe have enhanced it just a little. But it still resets with the same Guru error. Here are the logs.
gg-without-delays-in-main2.txt
[main] gets printed only when we call vStartGreenGrassDiscoveryTask() directly in the main without creating a task.
Here are some of the backtraces. I don't know if it can help.
gg-backtraces.txt
Thank you.

[main] gets printed only when we call vStartGreenGrassDiscoveryTask() directly in the main without creating a task.

Thank you for sharing backtraces. I see that the crash is observed during TLS handshake. One possible reason can be stack overflow. Can you try increasing the size of the stack which calls this function?
Also, enable malloc failed hook, if you have not already - https://www.freertos.org/a00016.html
Thanks.

Also @SarahHamed, I take back my statement about 'one should not use delays in app_main'. Because the app_main function is called from a task after the scheduler is started as seen here.
Apologies.

@AniruddhaKanhere No worries. Thank you for your response.
I tried what @aggarg suggested and did the following:
1- Increased the stack size used in xTaskCreate to be multiplied by 16 instead of 12. In the file "aws_demo_config.h":
#define democonfigGREENGRASS_DISCOVERY_TASK_STACK_SIZE               ( configMINIMAL_STACK_SIZE * 16 )

xTaskCreate( cal,
"Greengrass",
democonfigGREENGRASS_DISCOVERY_TASK_STACK_SIZE,
NULL,
democonfigDEMO_PRIORITY,
NULL ) != pdPASS )

2- Added to the file "amazon-freertos\vendors\espressif\boards\esp32\aws_demos\config_files\FreeRTOSConfig.h"
#define configUSE_MALLOC_FAILED_HOOK      1
#define configCHECK_FOR_STACK_OVERFLOW     1

3- Added these to the main.c
extern void vApplicationMallocFailedHook( void );
extern void vApplicationStackOverflowHook( TaskHandle_t xTask,  char * pcTaskName );

As they were defined here:

https://github.com/aws/amazon-freertos/blob/fdcedfa1e973797da0c0d0d7211d8ecc9102d5ad/demos/demo_runner/iot_demo_freertos.c#L406
https://github.com/aws/amazon-freertos/blob/fdcedfa1e973797da0c0d0d7211d8ecc9102d5ad/demos/demo_runner/iot_demo_freertos.c#L430

But unfortunately, I couldn't find in the logs any of these errors:

"ERROR: stack overflow in task ".
"ERROR: Malloc failed to allocate memory"

4- Enabled INCLUDE_uxTaskGetStackHighWaterMark in aws_greengrass_discovery_demo.c
gg-logs.txt
Thank you.

Hello @SarahHamed @BasemTarek,
Could you share the entire project (maybe host it on GitHub) that gives you the error? It would help us understand the issue better.

Looking at the logs, it seems like a WatchDog reset. What are all the tasks in the system. Is it possible that the Idle task is starved?
Additionally, can you share the complete code in a fork or repo so that I can give it a try?
Thanks.

Here it is https://github.com/SarahHamed/amazon-freertos-proj
Thank you.

Hello @SarahHamed,
Could you check out this patch for your project - gme_cache_accessed.zip?
Also, there are some issues with the way you have used the demo in your main.c. Could you follow the template for executing demos from an external project as given here?

Hello @laukik-hase ,
Thank you for your help and the patch file. It has been useful.
I've applied it locally. Here are the updated files after applying it:
CMakeLists.txt
main.txt

In the main file. Why _initialize() was called before vDevModeKeyProvisioning(). Does the order matter?
If I should follow the template here to not get the guru error. I am not sure how to deal with a task function that never returns. The function _threadRoutineWrapper() returns and the task gets deleted.

My question. What if we don't want the task function to return and to loop forever, as we were trying to implement in the main using while(1){} . How can we do that without facing the guru error?
Thank you.

What if we don't want the task function to return and to loop forever, as we were trying to implement in the main using while(1){} . How can we do that without facing the guru error?

If your task keeps looping and never gives up CPU and it has higher priority than Idle task, then Idle task won't get a chance to run. As a result, Idle task won' t be able to kick WatchDog and you will get a WatchDog reset. Why do you want to loop endlessly? Your task should block when there is nothing useful to do instead of looping endlessly. Try adding a vTaskDelay after each iteration at least.

Hello @SarahHamed ,
Do you still need help on this topic or this issue can be closed?

Hello @pvyawaha & @aggarg,
We downloaded another version of amazon freertos from the console which should be more stable than cloning the repo.
We haven't tried the same main that we were using. Since we know now that this wasn't the best way. So we are now trying to use the template that you've suggested.
Thank you all for your help. I believe we can close it.

Thank you. Feel free to open another issue if you need any help.


Self-documenting Makefile

This PR applies a neat trick as described in https://www.freecodecamp.org/news/self-documenting-makefile/.
A new make target called help is introduced that lists all documented targets and their documentation comment.
The syntax is simple:
# Makefile

target: ## I'm the documentation for the target
    @echo I'm the target code

I've rebased onto master and squashed my commits.

Though I'm starting to think whether it would make more sense to move the kind and e2e stuff into the Makefile in the e2e dir.

I think this is a good idea, though I would like to complete the work on this PR for now. Would you be ok with me creating a new issue to reorganise the Makefile?

I've created the issue https://github.com/vshn/k8up/issues/198 to track the idea of extracting e2e related make targets to the relevant Makefile.


Add fab package

This adds the Fab package, useful for fancy text output in the terminal. Thanks for your time!

Cool. Some observations:

Don't use tabs in packages.json
Use the Nim terminal module, it supports all platforms (your code is Unix-only).

Yeah, I'm planning a refactor to implement the terminal module in my code. Or, I'll take a look at how its written and probably figure out Win support from that.


Return scalar multiplication in outer product if rank of one tenser i…

This intends to fix #196

@googlebot I signed it!

@googlebot I fixed it.

Closed this pull request as the commits were overwritten. Will send in a new pull request regarding this. Sorry about the confusion!


Add Nexus Worker interceptor

Add Nexus Worker interceptor based on the Nexus Java SDK interceptors proposed here https://github.com/nexus-rpc/sdk-java/pull/14

Note: CI will fail until a new Nexus Java SDK is released


Error while running plot_media_channel.... function

Hi, I am running "simple_end_to_end_demo" notebook on a GCP instance. All of the code runs fine except that I get the following error when running the code. Quick google search says numba needs to be turned off - I am not sure because all other plot functions are working fine. Any advice? Thanks.
plot.plot_media_channel_posteriors(media_mix_model=mmm)

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
TypeError: expected dtype object, got 'numpy.dtype[float64]'

The above exception was the direct cause of the following exception:

SystemError                               Traceback (most recent call last)
<ipython-input-55-4b8328576e48> in <module>
----> 1 plot.plot_media_channel_posteriors(media_mix_model=mmm)

/opt/conda/lib/python3.7/site-packages/lightweight_mmm/plot.py in plot_media_channel_posteriors(media_mix_model, channel_names, quantiles, fig_size)
    708           media_channel_posteriors[:, channel_i],
    709           quantiles=quantiles,
--> 710           ax=channel_axis)
    711       axis_label = f"media channel {channel_names[channel_i]}"
    712       channel_axis.set_xlabel(axis_label)

/opt/conda/lib/python3.7/site-packages/arviz/plots/kdeplot.py in plot_kde(values, values2, cumulative, rug, label, bw, adaptive, circular, quantiles, rotated, contour, fill_last, figsize, textsize, plot_kwargs, fill_kwargs, rug_kwargs, contour_kwargs, contourf_kwargs, pcolormesh_kwargs, is_circular, ax, legend, backend, backend_kwargs, show, return_glyph, **kwargs)
    248                 bw = "experimental"
    249
--> 250         grid, density = kde(values, circular, bw=bw, adaptive=adaptive, cumulative=cumulative)
    251         lower, upper = grid[0], grid[-1]
    252

/opt/conda/lib/python3.7/site-packages/arviz/stats/density_utils.py in kde(x, circular, **kwargs)
    529         kde_fun = _kde_linear
    530
--> 531     return kde_fun(x, **kwargs)
    532
    533

/opt/conda/lib/python3.7/site-packages/arviz/stats/density_utils.py in _kde_linear(x, bw, adaptive, extend, bound_correction, extend_fct, bw_fct, bw_return, custom_lims, cumulative, grid_len, **kwargs)
    625         x_min, x_max, x_std, extend_fct, grid_len, custom_lims, extend, bound_correction
    626     )
--> 627     grid_counts, _, grid_edges = histogram(x, grid_len, (grid_min, grid_max))
    628
    629     # Bandwidth estimation

/opt/conda/lib/python3.7/site-packages/arviz/utils.py in __call__(self, *args, **kwargs)
    181         """Call the jitted function or normal, depending on flag."""
    182         if Numba.numba_flag:
--> 183             return self.numba_fn(*args, **kwargs)
    184         else:
    185             return self.function(*args, **kwargs)

SystemError: CPUDispatcher(<function histogram at 0x7f66dc4f9a70>) returned a result with an error set

Hello @chsheth  !
Looks like a problem with the arviz library. What version of arviz are you currently using and what python version?

On the GCP instance, Python is 3.7.8 and arviz is 0.11.2.

Hello @chsheth !
I am unable to reproduce your error on a GCP instance with python 3.7 and the same arviz version.
Could you try pip install --upgrade git+https://github.com/google/lightweight_mmm.git ? Just in case some version got hanged or similar.
As per your other error, if its in colab could you please provide a reproducible colab for me to take a look?
Thanks!

Thank you @pabloduque0.
For GCP/arviz, I can email you the notebook, cannot add it here -- I do think it has to do with some versioning.
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-4-9a43dbab01ab> in <module>
      1 # Import the relevant modules of the library
----> 2 from lightweight_mmm import lightweight_mmm
      3 from lightweight_mmm import optimize_media
      4 from lightweight_mmm import plot
      5 from lightweight_mmm import preprocessing

/opt/conda/lib/python3.7/site-packages/lightweight_mmm/lightweight_mmm.py in <module>
     51 from lightweight_mmm import models
     52 from lightweight_mmm import preprocessing
---> 53 from lightweight_mmm import utils
     54
     55 Prior = Union[

/opt/conda/lib/python3.7/site-packages/lightweight_mmm/utils.py in <module>
     22 import jax.numpy as jnp
     23 import numpy as np
---> 24 import pandas as pd
     25 from scipy import interpolate
     26 from scipy import optimize

/opt/conda/lib/python3.7/site-packages/pandas/__init__.py in <module>
    140 from pandas.util._print_versions import show_versions
    141
--> 142 from pandas.io.api import (
    143     # excel
    144     ExcelFile,

/opt/conda/lib/python3.7/site-packages/pandas/io/api.py in <module>
      6
      7 from pandas.io.clipboards import read_clipboard
----> 8 from pandas.io.excel import ExcelFile, ExcelWriter, read_excel
      9 from pandas.io.feather_format import read_feather
     10 from pandas.io.gbq import read_gbq

/opt/conda/lib/python3.7/site-packages/pandas/io/excel/__init__.py in <module>
----> 1 from pandas.io.excel._base import ExcelFile, ExcelWriter, read_excel
      2 from pandas.io.excel._odswriter import _ODSWriter
      3 from pandas.io.excel._openpyxl import _OpenpyxlWriter
      4 from pandas.io.excel._util import register_writer
      5 from pandas.io.excel._xlsxwriter import _XlsxWriter

/opt/conda/lib/python3.7/site-packages/pandas/io/excel/_base.py in <module>
     30     get_writer,
     31 )
---> 32 from pandas.io.parsers import TextParser
     33
     34 _read_excel_doc = (

/opt/conda/lib/python3.7/site-packages/pandas/io/parsers/__init__.py in <module>
----> 1 from pandas.io.parsers.readers import (
      2     TextFileReader,
      3     TextParser,
      4     read_csv,
      5     read_fwf,

/opt/conda/lib/python3.7/site-packages/pandas/io/parsers/readers.py in <module>
     15 import pandas._libs.lib as lib
     16 from pandas._libs.parsers import STR_NA_VALUES
---> 17 from pandas._typing import (
     18     ArrayLike,
     19     DtypeArg,

ImportError: cannot import name 'DtypeArg' from 'pandas._typing' (/opt/conda/lib/python3.7/site-packages/pandas/_typing.py)

For '_png' from 'matplotlib' error, I found on stackoverflow that I would need to downgrade matplotlib version
!python -m pip uninstall matplotlib
!pip install matplotlib==3.1.3

I ran the above before any plot functions and there were no errors.

Actually I dont know about arviz error, if it got fixed -- the code didnt get to that cell and errored out much earlier. I am running the demo notebooks from here on my GCP instance, so there is nothing that I cannot share...
I was thinking of creating an environment, use requirements from here and then run -- maybe that might make the version issue (if there is one) go away.

If you can provide me the basic specifications of your instance I can try to re-create from my side.

Here are the details


Change API to return all of the originalFeature data as separate fields

rather than the current format (stringified-JSON-within-a-JSON)

This is implemented in 1.0 alpha. We can close when 1.0 ships and behavior is tested & documented.


go_router: bad state no element

Our app breaks using the latest version of go_router (5.2.2). When using 5.2.1 it works just fine.
I have boiled it down to:
https://github.com/flutter/packages/commit/250adea8b209b42cd6cf4a5515ac10a9da19ca08
Line 102-105
if (configuration.matches.last is ImperativeRouteMatch) {
      configuration =
          (configuration.matches.last as ImperativeRouteMatch).matches;
    }

In our specific case, in the first frames. configuration.matches is empty. Therefore it returns a bad state no element. I can locally resolve this by adding configuration.matches.isNotEmpty before the check. But I don't really know (yet) if that breaks other stuff or what the intended purporse is.
Is this something we are doing wrong when initializing go_router, a breaking API change or a oversight?
Greetz

See https://github.com/flutter/flutter/issues/116729 for an example (looks like every example would break) with code

I also ran into the same problem.  @chunhtai  need to solve this as soon as possible.
( I think this is critical bugs for developer to use go router.)
5.2.1 version also have pop problem...
Can I use this package for my project??...

closing this as duplicate.


Allure report generated via Jenkins failed to Load the Execution Video in Safari Browser

Describe the bug
Allure report generated via Jenkins failed to Load the Video [Execution Video].
Same code base running from local machine, loads the video in allure report.Same is not happening while running the test in Jenkins.
Execution Video attachment created in allure without loading the video.
Log
11:12:27  "Video" Reporter:
11:12:27 Using reporter config:[
11:12:27   "dot",
11:12:27   "spec",
11:12:27   "allure",
11:12:27   [
11:12:27     null,
11:12:27     {
11:12:27       "saveAllVideos": false,
11:12:27       "videoSlowdownMultiplier": 10,
11:12:27       "videoRenderTimeout": 5
11:12:27     }
11:12:27   ]
11:12:27 ]
11:12:27
11:12:27 Using config:{
11:12:27   "debugMode": false,
11:12:27   "logLevel": "info",
11:12:27   "videoRenderTimeout": 5,
11:12:27   "outputDir": "results",
11:12:27   "allureOutputDir": "allure-results",
11:12:27   "rawPath": "rawSeleniumVideoGrabs",
11:12:27   "usingAllure": true,
11:12:27   "saveAllVideos": false,
11:12:27   "videoSlowdownMultiplier": 10,
11:12:27   "excludedActions": [],
11:12:27   "jsonWireActions": [
11:12:27     "url",
11:12:27     "forward",
11:12:27     "back",
11:12:27     "refresh",
11:12:27     "execute",
11:12:27     "size",
11:12:27     "position",
11:12:27     "maximize",
11:12:27     "click",
11:12:27     "submit",
11:12:27     "value",
11:12:27     "keys",
11:12:27     "clear",
11:12:27     "selected",
11:12:27     "enabled",
11:12:27     "displayed",
11:12:27     "orientation",
11:12:27     "alert_text",
11:12:27     "accept_alert",
11:12:27     "dismiss_alert",
11:12:27     "moveto",
11:12:27     "buttondown",
11:12:27     "buttonup",
11:12:27     "doubleclick",
11:12:27     "down",
11:12:27     "up",
11:12:27     "move",
11:12:27     "scroll",
11:12:27     "doubleclick",
11:12:27     "longclick",
11:12:27     "flick",
11:12:27     "location"
11:12:27   ]
11:12:27 }
11:12:27
11:12:27
11:12:27 Max waiting time: 5s
11:12:27
11:12:27 --- Video /home/jenkins/workspace/PIT_WDIO_V5/results/add-to-cartpdp--Windows10_Chrome_v68-should-be-able-to-add-a-normal-sku-to-cart-lv--CHROME--07-18-2019--00-40-21-196.mp4 ---
11:12:27 Waiting for video to exist: 0s
11:12:27 Waiting for video to be ready: 0s
11:12:27 Waiting for video to be ready: 1s
11:12:27 Waiting for video to be ready: 2s
11:12:27 Waiting for video to be ready: 3s
11:12:27
11:12:27 Generated:[
11:12:27   "/home/jenkins/workspace/PIT_WDIO_V5/results/add-to-cartpdp--Windows10_Chrome_v68-should-be-able-to-add-a-normal-sku-to-cart-lv--CHROME--07-18-2019--00-40-21-196.mp4"
11:12:27 ]
11:12:27
11:12:27
11:12:27
11:12:27 Done!
11:12:27
11:12:27
11:12:27  "dot" Reporter:
11:12:27 F
11:12:27
11:12:27  "spec" Reporter:
To Reproduce
Try running test via Jenkins with allure report configurations
Expected behavior
clicking on 'Execution video' should loads the video  in allure report generated via Jenkins..
Environment (please complete the following information):

wdio-video-reporter version: [ 1.4.4,]
WebdriverIO version: [5.10.7]
Mode: [WDIO Testrunner- mocha]
If WDIO Testrunner, running sync/async: [ sync]
Node.js version: [10.15.4]
NPM version: [6.4.1]
Browser name and version: [All browsers in Saucelabs]
Platform name and version: [All browsers in Saucelabs]

Additional context
This problem we are seeing only when test run from Jenkins.
Local testing load the video as expected.
Might be related the storing the video Artifacts for allure report to refer

.

I will try to set this up and test it.
Until then, does it work if you run it on your local machine?

I have verified that safari does not want to play videos from the jenkins allure report.
It does however seem to be an allure issue, since the solution lies in modifying the generated html to to play nice with safaris autoplay policy.
See stackoverflow issue:
https://stackoverflow.com/questions/20347352/html5-video-tag-not-working-in-safari-iphone-and-ipad
No problems to view videos in chrome or firefox: https://imgur.com/a/sAbgSbx
Closing issue, since Allure need to do the fix.


Not able to set up Grid View

Hello:
I'm having problems to set a gridview.
I tried both options, xml and code (not at the same time)
< android.support.v7.widget.RecyclerView
xmlns:app="http://schemas.android.com/apk/res-auto"
android:id="@+id/recycler_view"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:scrollbars="vertical"
app:slm_section_sectionManager="grid"   //This is not set if i do it programatly
app:slm_grid_columnWidth="96dp"       //This is not set if i do it programatly
app:slm_grid_numColumns="3"/>          //This is not set if i do it programatly
public void onBindViewHolder(RedGridHolderView redGridHolderView, int position) {
//more code
GridSLM.LayoutParams params = GridSLM.LayoutParams.from(itemView.getLayoutParams());
params.setSlm(GridSLM.ID);  //This is not set if i do it in xml
params.setNumColumns(3);  //This is not set if i do it in xml
params.setColumnWidth(context.getResources().getDimensionPixelSize(R.dimen.column_width));  //This is not set if i do it in xml
//more code
}
I'm getting a "listview", i think it's not getting the parameter grid or the numbers of colums.
Could you help me?
Thanks in advance

I solved my problem. I was setting the wrong LayoutManager (Android manager  in stead of the one of the library) in the main activity.
Tanks.


batch updates unexpectedly stopped

Expected Behavior
batch updates should should only be stopped by graph.stopBatch() call once started by graph.startBatch() call

Current Behavior
if there are any mutations in event handlers, batch update gets stopped before actually calling graph.stopBatch()

Possible Solution

Steps To Reproduce
https://codesandbox.io/s/charming-swartz-zmpq7?file=/src/app.tsx

...
...
...

Error Message & Stack Trace
<!-- Provide a log message if relevant -->

Additional Context

Your Environment

x6: [e.g. 1.0.0]
OS: [e.g. macOS Sierra 10.12.3]
Browser: [e.g. chrome 78.0.3904.108]

maybe the batch:stop event is falsely emitted, the history undoStack is correct

I'm sorry I don't quite understand you. startBatch and stopBatch come in pairs, you can：
graph.on("batch:start", ({ name }) => console.log('start:batch', name));
graph.on("batch:stop", ({ name }) => console.log('stop:batch', name));

I'm sorry I don't quite understand you. startBatch and stopBatch come in pairs, you can：
graph.on("batch:start", ({ name }) => console.log('start:batch', name));
graph.on("batch:stop", ({ name }) => console.log('stop:batch', name));

So every mutation in event handlers will get a different batch session automatically , and if you start a batch session manually , these sessions' undo actions will be merged into a single batched action in history undoStack , but the session events will still be emitted nontheless ? This behaviour works fine , but i'm finding it quite confusing .

Batch actions can be nested. If you print the names, it would be more clear.


Foundry remapping paths are highlighted as not found

@juanfranblanco Since around 1 day I get the following issue (I guess it's related to the recent Foundry support PR):

My remappings file:

You can reproduce the behavior via my repo here.
To be clear, there is no compilation issue at all from Foundry, just the warning in the UI. Also, before (i.e. 2 days ago and before) everything was working smoothly.

I can see what is happening your foundry file has an empty array of remappings that is taking preference over the real remappings

Pinging @PaulRBerg, you got the same warnings?

Nope

I guess you're referring to this

So, foundry.toml remappings take preference over my remappings.txt? The default value is an empty array of remappings actually:

So I didn't do anything special here...

@pcaversaccio update the latest in a couple of minutes as I have put a check to ignore null remappings in the foundry config file.
@PaulRBerg @pcaversaccio I have assumed that foundry takes preference over remappings.txt thoughts?

It is out now the fix.. Thanks again @pcaversaccio

@pcaversaccio update to the latest in a couple of minutes as I have put a check to ignore null remappings in the foundry config file. Thanks for pointing it out that it was the default too!

Thanks a lot! We revert with a confirmation once tested.

@PaulRBerg @pcaversaccio I have assumed that foundry takes preference over remappings.txt thoughts? https://github.com/foundry-rs/foundry/blob/master/config/src/providers/remappings.rs#L40

I prefer working with remappings.txt files since otherwise the foundry.toml file gets too verbose.

It is out now the fix.. Thanks again @pcaversaccio

Thx a lot @juanfranblanco!

Confirmed - everything fixed.

I prefer working with remappings.txt files since otherwise the foundry.toml file gets too verbose.

Same. Also, separating the remappings in a bespoke remappings.txt makes it easier to compartmentalize the project settings - this is useful e.g. when documenting how to install a Foundry project, as I often include a reference to what the user should have in the remappings.txt files (whereas explaining what field one should set in the Foundry config is more cumbersome).

Great thanks, I see your point, maybe that is another option to having a common project file which has been my hope for years .. different settings, per file. In that scenario it makes more sense that the remappings file will take preference over the foundry if found, as it is a decision to make it generic.

@juanfranblanco still happening with me

Have you tried to change the setting from monorepo back and forth,?

I have added this
So sometime it doesn't bother  but sometime it shows error although I can access files by cmd + press

@pranjal1127
If you are defining your remapping with the @ symbol @openzeppelin, you should also be importing using that.
That is:
import<EMAIL_ADDRESS>
The "@" character is not treated specially.
Check whether you can compile - if not, it's not an issue with vscode-solidity :)


x/crypto/acme/autocert: httpTokenCacheKey allows limited directory traversal on windows

httpTokenCacheKey uses path.Base to extract the expected HTTP-01 token value to lookup in the DirCache implementation. On Window path.Base acts differently to filepath.Base since Windows uses a different path separator (\ vs. /), allowing a user to provide a relative path, i.e. .well-known/acme-challenge/..\..\asd becomes ..\..\asd. The extracted path is then suffixed with +http-01, joined with the cache directory, and opened.
Since the controlled path is suffixed with +http-01 before opening, the impact of this is significantly limited, since it only allows reading arbitrary files on the system if and only if they have this suffix.
Thanks to Juho Nurminen of Mattermost for reporting this.
This is CVE-2022-30636.

Fixed by https://go.dev/cl/408694.


Fix gomobile on android.

See: https://github.com/golang/mobile/commit/2f75be449fbada35ca42f481f9878ba7f02a7e7d
for the breaking change
@keybase/react-hackers @gabriel

👍


Simplify idle check used for idle process metric in Compactor

This PR simplifies the check for the Compactor that is used to determine when to change the idle process metric for the Compactor. The new code just returns true when no compaction is running and false when one is. It defers the time checking portion to the upstream logic in AbstractServer.
The old method was to check how much time has passed since the last compaction was completed. There are at least two issues with this:

There is logic that makes sure the amount of time since the last compaction happened is at least idleReportingPeriodNanos. This check also happens upstream when the Supplier<boolean> is read in AbstractServer so we are getting rid of that duplicate check in the new code.
There is a check that the System.nanoTime() value of the last completed compaction is not negative. This is not a safe check to make since System.nanoTime() may provide a negative value. (This bug was identified here)

In 04ed08b I added a test case that checks that the compactor idle metric correctly tracks what we expect it to be before, during and after a compaction. I am able to get the test to pass using the old logic in compactor but with the new logic, the idle value is not returning to 0 as I expect. I am not sure why that is happening at the moment so was hoping someone might be able to spot a flaw in the new logic that sets the idle metric value.

I added another test case to test that the scan server idle metric behaves as expected (very similar to compactor test case)

Since the scope of this PR has changed slightly since it has been created, here is an updated description of changes in this PR:

AbstractServer.updateIdleStatus() was changed from having Supplier<Boolean> as its input to just a boolean. This was done to simplify things since the method was just using it as a boolean anyways

the method was also renamed from idleProcessCheck() to updateIdleStatus()

The idle check in Compactor was simplified to just set the idle status to true or false during its operation. This removes duplicate logic that checks it has been idle for a period of time (already done in AbstractServer.idleProcessCheck())
Adds test cases to ensure that the idle metric for Compactor and ScanServer behave as expected. Tserver was omitted since it is not expected to be idle in 2.1.


[tests-only][full-ci]Add /Shares related favorite tests on ocis which are removed from core

Description
This PR adds the tests for favorite properties related to /Shares which is removed from this PR https://github.com/owncloud/core/pull/40274 as /Shares related implementation is not in core and is shifted to ocis.
Related issue
https://github.com/owncloud/ocis/issues/4154#issuecomment-1206026045

The CI failed at go-generate I'll restart


Ketrem preprod docs

Docs Build status updates of commit 57355d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-driver-docs-pr/dashboard/index.yml
:white_check_mark:Succeeded

windows-driver-docs-pr/dashboard/manage-preprod-submissions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hey @anaharris-ms  can I trouble you to merge this for me?

Done :D
From: Kevin-Tremblay @.>
Sent: Friday, August 5, 2022 9:29 AM
To: MicrosoftDocs/windows-driver-docs @.>
Cc: Anastasia Harris @.>; Mention @.>
Subject: Re: [MicrosoftDocs/windows-driver-docs] Ketrem preprod docs (PR #3225)
Hey @anaharris-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fanaharris-ms&data=05|01|anaharris%40microsoft.com|c8af149a9d3e4bf81c0808da76e68462|72f988bf86f141af91ab2d7cd011db47|1|0|637953029686043151|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=heMW5Zq6a5TPb2Tb3t3P6bsbAmbJdEIAo4OFPl2nn3g%3D&reserved=0 can I trouble you to merge this for me?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-driver-docs%2Fpull%2F3225%23issuecomment-1206460652&data=05|01|anaharris%40microsoft.com|c8af149a9d3e4bf81c0808da76e68462|72f988bf86f141af91ab2d7cd011db47|1|0|637953029686043151|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Ro4ZnWJjbzqIakpdHN8geIPC1WvOXxQMce7thQdmLNQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOV7TTZ6KN22H3T4JNW3JADVXUJLLANCNFSM55UCXQEQ&data=05|01|anaharris%40microsoft.com|c8af149a9d3e4bf81c0808da76e68462|72f988bf86f141af91ab2d7cd011db47|1|0|637953029686200364|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=lxcXhc4nVXS8ihdjLk0dyZlMZ3vY%2FnyEqmxSK3D8y6k%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

<3

Docs Build status updates of commit 57355d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-driver-docs-pr/dashboard/index.yml
:white_check_mark:Succeeded

windows-driver-docs-pr/dashboard/manage-preprod-submissions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Embed syntax with block references doesn't work

Embedding entire notes via ![[note]] works fine, but adding block references to the mix like ![[note^uid]] or even ![[^uid]] (which embeds a block inside the current note) will not work.
The concerning part is it will fail silently - no error notification. It eventually generates an export file but it takes absurdly long (presumably waiting for a timeout of some kind). The export just displays note > uid instead of the embedded note as desired.
Note to self: be careful not to regress the recursion limiting behaviour introduced in #5.
Is there an Obsidian API function to turn a ![[note^uid]] embed string into a source markdown string or similar? That would be a far more scalable way of doing this. (edit: yes, there's a method in the metadata cache API)

Note to self: make sure note embedding works for aliases as well.
The block level embedding behaviour seems quite complex, so it's definitely worth figuring out how to use the Obsidian API to get a markdown string from a link

Hi @OliverBalfour any progress with this? Killing me that this is not possible through any way apart from the built in pdf export.

In the Issue https://github.com/OliverBalfour/obsidian-pandoc/issues/20 it says that it will by solved together with this issue here. Is that true?

This issue persists as of Obsidian version: v1.5.3 and Pandoc Plugin v0.4.1.
Are there any updates?

Hi @OliverBalfour any progress with this? Killing me that this is not possible.


Unit test depends on network connection

I had a wrong idea of a unit test, and it turns out that unit test should run very fast and therefore cannot depend on external network. I have to divide up the functions I currently have and make them self-contained in terms of their functionality.

Networking moved to Integration test


"Enable Overclocking" does not take effect until initramfs regeneration

Enabling overclocking creates a file with amdgpu driver options in /etc/modprobe.d. In order for these options to be applied, they have to be loaded early in the boot process at the initramfs stage. At least on Arch, these settings are not included into initramfs until it is regenerated (either manually or via a kernel update). This should at least be mentioned in LACT (ideally handled automatically), also need to check what the behaviour is on other distros.
Workaround for arch-based systems (assuming the default initramfs generator is used): run mkinitcpio -P after clicking "enable overclocking".

I'm using endeavourOS with Kernel 6.5RC3 and even when enabled overclocking I don't have option to change anything on my 7900XTX.
amdgpu.ppfeaturemask=0xffffffff is added to boot parameter (Corectrl shows some settings, except for FAN control which does nothing)
aug 01 18:20:53 spajdre-linux systemd[1]: Started AMDGPU Control Daemon.
aug 01 18:20:53 spajdre-linux lact[745]: 2023-08-01T16:20:53.160942Z  INFO lact_daemon::socket: listening on "/var/run/lactd.sock"
aug 01 18:20:53 spajdre-linux lact[745]: 2023-08-01T16:20:53.182960Z  INFO lact_daemon::server::handler: initialized 1 GPUs
aug 01 18:20:53 spajdre-linux lact[745]: 2023-08-01T16:20:53.686081Z ERROR lact_daemon::server::gpu_controller: could not set fan speed: io error: Invalid argument (os error 22), disabling fan control
1002:744C-1849:5304-0000:2d:00.0 (Navi 31 [Radeon RX 7900 XT/7900 XTX])
GPU Vendor: Advanced Micro Devices, Inc. [AMD/ATI]
GPU Model: Navi 31 [Radeon RX 7900 XT/7900 XTX]
Driver in use: amdgpu
VBIOS version: 113-D70201-810009 [2022/11/29 01:30]
Link: LinkInfo { current_width: Some("16"), current_speed: Some("16.0 GT/s PCIe"), max_width: Some("16"), max_speed: Some("16.0 GT/s PCIe") }

This is likely related to overdrive support on 7000 series GPUs being new/incomplete, and not due to it not being enabled (if you've specified the overclocking feature mask as a boot option it's always applied)
Could you post the output of
ls /sys/class/drm/card*/device/
cat /sys/class/drm/card*/device/pp_od_clk_voltage

[spajdrex@spajdre-linux ~]$ cat /sys/class/drm/card*/device/pp_od_clk_voltage
OD_SCLK:
0: 500Mhz
1: 3005Mhz
OD_MCLK:
0: 97Mhz
1: 1250MHz
OD_VDDC_CURVE:
0: 0mv
1: 0mv
2: 0mv
3: 0mv
4: 0mv
5: 0mv
OD_RANGE:
SCLK:     500Mhz       5000Mhz
MCLK:      97Mhz       1500Mhz
VDDC_CURVE:    -450mv          0mv
Also adding what is shown by corectrl (fan control does not work at all) thanks for checking :-)

@spajdrEX sorry for taking a long time to get back to this, basic overclocking should now work.

Initramfs generation implemented for arch (mkinitcpio) and debian-based systems


Clearing cache can lead to a server overload

Hi.
Once, when I was clearing a cache in concrete5 5.7, and waited for 10 seconds to load the page after that, I realized, that in case of high load site, clearing cache can lead to a complete server stop, cause queue of requests will be generated faster than it will be processed by server.
Now sure though that selective cache clearing will be helpful. I've tried to clear just LESS cache ( by disabling caching, updating file and refreshing a page again ) and I've waited for long as well.
Probably some kind of smart cache clearing could be helpful, like, while LESS/Page/Block cache is generated, old cache is still serving new requests. Or I miss something?

I wonder how people running high traffic websites get around this issue? serving older cache seems like a good idea. Send the cached page and continue generating the page using pcntl_fork?


Update sqlite-api-guide.md

Signed-off-by: tpmccallum<EMAIL_ADDRESS>

It appears that the COMMIT in SQLite is automatic. I will close this now that the double execution has been diagnosed and resolved.


Bot should pause/slow down when its raining near his coordinates

Short Description
This bot is meant to be the most human-like possible, why not try to simulate a pause/slow down outside while its raining near the coords he is moving.
Possible solution
Using the random pauses type, linked it with a openweather provider, we could trigger a pause while the weather provider reports bad weather near the coordinates, it would be configurable like "pauseIfRaining" or "pauseIfBadWeather", there could be some kind of setpoints for define this kind of bad weather.
How it would help others
This won't actually avoid the bot being detected, unless Niantic does some kind of pattern analysis with weather cross data, but it would make the bot behaves more like a human on that situations.

I really can't see the purpose of this. They won't be factoring weather into bot detection as there'll still be players who go out in all sorts of weather.
All this would be doing is adding an unneeded level of complexity.

No reason why it can't be a feature request though, I'd be surprised if anyone wanted to implement it though. Let's see if anyone replies :)

No problem with this remaining to be a FR, but I imagine this will hang around for quite some time.

The bot should also walk slower during sales since there is obviously more people in the street during these days.
And we should be able to query any event website to check whether Justin Bieber has a concert around, cause streets are truly not practicable when it happen, so bot should stop walking.
;)

Hmmm, the sarcasm is strong in this one LOL

Sorry, this is too extreme for even the next 6 months worth of work. Plus the amount of work they would have to do to determine such things and consider them conclusive for bans is... unlikely to happen.


$getRecord() not working in angularfire 1.1.1

$getRecord() returns null even when passing a valid id.
I'm using:
angularfire 1.1.1
firebase 2.2.7
angular 1.4.0
When I downgrade angularfire to 1.0.0 the function works again, when downgrading to 1.1.0 the problem persisted.
I'm aware of ticket https://github.com/firebase/angularfire/issues/622, which also mentions $getRecord not working in 1.1.1, but when downgrading my angular version to 1.3.15 the problem persists.

Thanks for sending this in! Could you create a repro we could run locally to see the issue?

No problem, is a JSBin / Codepen? also ok?

Yes, anything that makes it easy for us to reproduce.

Closing this for now. Please reopen if you can provide a repro.


Add an option to set encryption to none during installation

Bug report
Title
Add an option for blank encryption option during install
Issue Description
When installing bagisto locally and you want to use mailtrap.io in you configuration I see that tere is no option for no encryption. There is only a choice between SSL and TLS and none of these are the correct configuration for mailtrap.
Preconditions
Please provide as detailed information about your environment as possible.
1. V.1.0.0
2. Stable Release via composer

Steps to reproduce
It is important to provide a set of clear steps to reproduce this bug.If relevant please include code samples.
1. Get bagisto package
2. Go thru installation till you get too Email configuration

Expected result
Tell us what should happen.
It should show an extra option "None" for no encryption.
This will result in me not having to change this manually afterwards in the dot env file.

Actual result
Tell us what happens instead.
After installing bagisto I have to go into my dot env file and remove the Mail_Encryption line or set it to blank

Hi @SamiXSous
Thanks for notifying this, the issue has been resolved.
You can reopen this issue if found again.


stateChange is always poweredOn

Hi! I would like to ask if there's a way to check bluetooth state is on or off?
I try switching the windows bluetooth on and off, but it's always stay in the state of "poweredOn".
Even when i try relaunching with bluetooth state off, it's still detect as "poweredOn".
noble.on('stateChange', function(state) {
console.log(state);
if (state === 'poweredOn') {
noble.startScanning();
} else {
noble.stopScanning();
}
});

This is not supported. If you want to implement it, that'd have to be implemented in BLEServer first. Then you'd need to adapt the bindings.js file accordingly.
I don't have any intent to implement it, but if you send a PR I am willing to spend the time to review it and eventually merge it.

I've made the changes needed to fix this issue to my fork of this repo.


Advanced preferences for Mouse Clicks regarding "Deletes History Temporary Containers"

Possibility to set "Middle Mouse" and "Ctrl/Cmd LeftClick" to always open "Deletes History Temporary Containers" instead of normal Temporary Containers. That way it's easy to decide between opening a "Deletes History Temporary Containers" and a normal one when clicking a link.

Just published version 0.33 with the feature


Async the group de-index during upload in content-index service

Recent performance problem we met for uploading & promotion is all
caused by the getAffectedBy operation, which is a time-consumed one for
a full scan on store data. After checking, I found that it is mainly
used for de-index or re-index content-index entries for the uploading
repo. I think this de-index will not be a block factor during the upload
or promotion process, which means we could make it async.
In this commit, I switched back to use a normal executor service to
async the de-index of the affected group. One potential problem for this
solution is the executor queue size, which could cause OOM if there are
bunch of uploading happened in a short time period. We need to monitor
if this can happen.

LGTM! +1

@ruhan1 I don't have a very good idea yet as each solution has its Pros and Cons, but maybe we can have a try with this scheduled solution. I'll try to do this in a separate pr.

Yep. This LGTM. Pls merge it and see what is going on. That batch thing can be done later if we need to.

Ok I'll merge this and have some load-test on stage. If something wrong there we can revert this. And we can discuss with @jdcasey whether to release this next week.


Demo improper clone of last object instead of empty form

@jpic I was able to recreate the issue shown in #1135 and the issue that I was having in the test project. I would say it is not an actual bug, but an improper cloning.
Specifically, if you clone the last element in the formset list and NOT the empty formset as demoed in select2_outside_admin.html you will run into this issue. I show both the wrong way and the right way in this PR. You don't really need to merge this, but maybe reject and point to the right way to do things...

I see, pretty funny sequence of event:

Did you find your way out with the empty formset ? I think it's better to clone from there, that's also how the admin deals with dynamic formsets as such it's going to be the preferred / most supported way from both django, admin and dal.

Yes! Empty formset is the way to go, but there are tutorials floating around that use the copying of the last element. I suggest in documentation highlighting that this will not work and you MUST use the empty formset to get it to work.

True that, we should have a micro external app reproducing dynamic formsets from outside the admin (but exactly the same way to be compatible) like we already have with django-addanother, @jonashaag please share your thought


Undefined in typedef causes type mismatches to be silently ignored

Externs:
/** @typedef {{  sausages: (?string)  }} */
var GOOD;

/** @typedef {{  sausages: (string|null|undefined) }} */
var BAD;

Code:
/** @param {GOOD} obj */
var testGood = function(obj) {};

/** @param {BAD} obj */
var testBad = function(obj) {};

testGood({bacon: "mmm"}) // correctly fails
testBad({bacon: "yuck"}) // should fail, doesn't

It appears changing from ?string to string|null|undefined causes the entire typedef to be disregarded silently.
https://closure-compiler-debugger.appspot.com/#input0%3D%252F**%2520%2540param%2520%257BGOOD%257D%2520obj%2520*%252F%250Avar%2520testGood%2520%253D%2520function(obj)%2520%257B%257D%253B%250A%250A%252F**%2520%2540param%2520%257BBAD%257D%2520obj%2520*%252F%250Avar%2520testBad%2520%253D%2520function(obj)%2520%257B%257D%253B%250A%250AtestGood(%257Bbacon%253A%2520%2522mmm%2522%257D)%2520%252F%252F%2520correctly%2520fails%2520%250AtestBad(%257Bbacon%253A%2520%2522yuck%2522%257D)%2520%252F%252F%2520should%2520fail%252C%2520doesn't%26input1%26conformanceConfig%26externs%3D%252F**%2520%2540typedef%2520%257B%257B%2520%2520sausages%253A%2520(%253Fstring)%2520%2520%257D%257D%2520*%252F%250Avar%2520GOOD%253B%250A%250A%252F**%2520%2540typedef%2520%257B%257B%2520%2520sausages%253A%2520(string%257Cnull%257Cundefined)%2520%257D%257D%2520*%252F%250Avar%2520BAD%253B%26refasterjs-template%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26MISSING_PROPERTIES%3D1
Background:
I'm trying to make it easy for a developer to pass a clean and simple JS object literal into a method, and have Closure Compiler verify the object literal matches all expected properties, e.g.:
someFunc({
  propA: 5,
  propB: ["a", "b"]
  // propC optional and missing
});

Hence I need to allow for undefined. I was surprised that ? didn't mean "null or undefined", and think the only way of doing this is using string|null|undefined in this example.
This issue is somewhat worrying because it is easy to inadvertently disable all typechecks. Presumably this is a bug. But if this is intended for some reason, it would benefit from some warning that could be turned off, rather than silently ignoring failures.

Update: still open, this time with NTI ticked:
https://closure-compiler-debugger.appspot.com/#input0%3D%252F**%2520%2540param%2520%257BGOOD%257D%2520obj%2520*%252F%250Avar%2520testGood%2520%253D%2520function(obj)%2520%257B%257D%253B%250A%250A%252F**%2520%2540param%2520%257BBAD%257D%2520obj%2520*%252F%250Avar%2520testBad%2520%253D%2520function(obj)%2520%257B%257D%253B%250A%250A%252F%252F%2520Comment%2520out%2520this%2520line%2520and%2520it%2520all%2520passes%2520without%2520error%253A%250AtestGood(%257Bbacon%253A%2520%2522mmm%2522%257D)%2520%252F%252F%2520correctly%2520fails%2520%250A%250AtestBad(%257Bbacon%253A%2520%2522yuck%2522%257D)%2520%252F%252F%2520should%2520fail%252C%2520doesn't%26input1%26conformanceConfig%26externs%3D%252F**%2520%2540typedef%2520%257B%257B%2520%2520sausages%253A%2520(%253Fstring)%2520%2520%257D%257D%2520*%252F%250Avar%2520GOOD%253B%250A%250A%252F**%2520%2540typedef%2520%257B%257B%2520%2520sausages%253A%2520(string%257Cnull%257Cundefined)%2520%257D%257D%2520*%252F%250Avar%2520BAD%253B%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26CHECK_TYPES_NEW_INFERENCE%3D1%26MISSING_PROPERTIES%3D1

see https://github.com/google/closure-compiler/issues/1811

Closing as this appears to be correct behaviour given my now deeper understanding of the type system.


Upload pacs dataset to cloud

@Car-la-F you can simply drag and drop the data from your local to vscode. Takes some time but worked fine for me!


when should i call close()

i try to close the connection but fail
it always show InterruptedException
i call close() at the end of my UsbReadCallback function , is it right?
on the other side , i can't send write() twice in one connection ,the second response is wrong with 00 00 00 00.....
that is the tip
W/System.err:     at java.lang.Object.wait(Native Method)         at java.lang.Object.wait(Object.java:442)         at java.lang.Object.wait(Object.java:568)         at com.felhr.usbserial.SerialBuffer$SynchronizedBuffer.get(SerialBuffer.java:117)         at com.felhr.usbserial.SerialBuffer.getWriteBuffer(SerialBuffer.java:72)         at com.felhr.usbserial.UsbSerialDevice$WriteThread.doRun(UsbSerialDevice.java:402)         at com.felhr.usbserial.AbstractWorkerThread.run(AbstractWorkerThread.java:21)

Have you solved it? I encountered the same problem as you.


Add test for distribution image

Add a simple test for distribution-image, that creates a custom container. The test container is pushed and pulled from the registry.

ticket: Create tests for the distribution-image

With podman you're running into the issue that we have turned on image verification via https://github.com/SUSE/BCI-tests/blob/e9c8fe1842a2109b98ee1d155ff8705e5a6d6c2e/.github/workflows/ci.yaml#L129, but only included the key for devel:BCI. Could you pick a different image or reconfigure podman to also accept the opensuse signkey?

This still needs https://github.com/dcermak/pytest_container/pull/120 to pass

This still needs dcermak/pytest_container#120 to pass

Oh, thanks for the update!

With podman you're running into the issue that we have turned on image verification via
https://github.com/SUSE/BCI-tests/blob/e9c8fe1842a2109b98ee1d155ff8705e5a6d6c2e/.github/workflows/ci.yaml#L129
, but only included the key for devel:BCI. Could you pick a different image or reconfigure podman to also accept the opensuse signkey?

Let's try out a devel image, and later on we can include opensuse signkey. Thanks for the hint

doesn't work with docker:
docker: Error response from daemon: invalid mode: U,noexec,Z.

doesn't work with docker:
docker: Error response from daemon: invalid mode: U,noexec,Z.

Oh dear, IIUC docker is missing U and noexec flags for mounts, e.g. https://github.com/moby/moby/issues/7054, and it has not been implemented there for several years.
Is there any recommendation from SUSE towards our customers how they should use the container using docker|podman?

Is there any recommendation from SUSE towards our customers how they should use the container using docker|podman?

podman is the recommended container runtime on ALP and SLES according to my knowledge.

In such a case, would it be acceptable to run the test with podman only for now and re-thing what to do with docker? This way we can at least have some regular testing and release of the container.

That's ok from my PoV. Add a skipIf like here to the test function: https://github.com/SUSE/BCI-tests/blob/3cb5a194cefb3e080f2a00ecb06b2fdc31495892/tests/test_init.py#L18

podman is the recommended container runtime on ALP and SLES according to my knowledge.
In such a case, would it be acceptable to run the test with podman only for now and re-thing what to do with docker? This way we can at least have some regular testing and release of the container.

No, we need to test with docker also, and there is no reason why that shouldn't work. In fact the main issue is the incorrect tagging in the test case, which breaks docker push. I'll post a fixup.

I've cleaned up the dedent stuff as well. now it looks good to me and should be ready for merging imho

:partying_face: @dcermak, @dirkmueller thank you guys, your support and help was very much appreciated, you rock! :partying_face:


How to build a multi-machine twitter heron cluster?

Hi,guys!
I have five machines.
How to install distributed cluster?
According to local installation mode,and then modify the configuration file?
3Q

@Jonathan-Wei
Unlike Storm, heron itself is a library.
To run topologies in a distributed environment, you need to setup:

scheduler for scheduling jobs
uploader for uploading and downloading your topology file, heron-core file
statemanager for managing the states during topology running

BTW, we currently have built-in support for aurora and slurm. Mesos and YARN are working in process and coming soon.

@Jonathan-Wei - you can read the sections on Deployment for understanding how to deploy it multi-node cluster.

Since Heron does not have a scheduler of its own - you need to have a scheduler installed. Currently, we support Aurora/Mesos - you need to install Aurora as in http://aurora.apache.org/

You need a zookeeper installed in some node.

Once you have both follow the configuration
https://github.com/twitter/heron/tree/master/heron/config/src/yaml/conf/aurora

@kramasamy
It does not support yarn?
https://github.com/twitter/heron/pull/419  not merge？

@Jonathan-Wei - we are working on testing it and cover a few corner cases. Once it is done - we will merge to master. Similarly native mesos scheduler is also in the works #803

I try it!
3Q guys!

@Jonathan-Wei - We have just merged the REEF scheduler on YARN to the master. It is still experimental and we are working on the documentation. You are welcome to try it out and let us know.

@kramasamy
Oh! It's great!
Thank you!


⚠ Processors has degraded performance

In 8f15307, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 107 ms

Resolved: Processors performance has improved in c7a0f91.


[bug] Fix SIGSERV on macOS Monterey, fixes #1790

Fixes SIGSERV segmentation violation error caused by go-ieproxy dependency. This is fixed in https://github.com/mattn/go-ieproxy/issues/35. Upgrading to a more recent version of the mentioned dependency fixes the issue.
Fixes https://github.com/Azure/azure-storage-azcopy/issues/1790
NOTE this replaces PR https://github.com/Azure/azure-storage-azcopy/pull/1979 since this one is against 'dev' instead of 'main' as requested.

@nakulkar-msft and @vibhansa-msft I believe this PR takes a dependency on a later version of go ie proxy which has a CGO dependency. We need to discuss this as a team before merging this PR.

@gapra-msft @vibhansa-msft @nakulkar-msft I think it's safe to consume this change since the cgo dep for darwin arm64 is already in place. We can just accept this and until we choose to migrate away from ie-proxy

Can this be part of the next release?

Can this be part of the next release?

should be in our next release

Can this be part of the next release?

unfortunately this PR actually results in an issue with some of our Windows builds, so we will have to investigate and reintroduce the PR in the future. We have reverted the PR in the meantime.

Is there a temp workaround for this issue? I got this issue on my M1 Mac and can't use azcopy at all. And I couldn't find v1.6.2 source code when I tried to build from source.

@kkkkkky 10.18 now has the fix!

@tasherif-msft does it? go.mod still mentions go-ieproxy v0.0.3 not a higher version which includes the fix.


Added solution for day 5 in go lang

@sagarchoudhary96 Updated.

@Jolly1234  these all PR needs to be updated. i added suggestion for it in Day4 PR. if you want i can close these PR and you can raise PR again.


[Fix-8296][API] Update process definition error

Purpose of the pull request
This pr will close #8296

Codecov Report

Merging #8408 (08ae4ea) into dev (ca086c0) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8408      +/-   ##
============================================
- Coverage     45.36%   45.34%   -0.02%
+ Complexity     4028     4026       -2
============================================
  Files           686      686
  Lines         26700    26702       +2
  Branches       2866     2867       +1
============================================
- Hits          12112    12108       -4
- Misses        13446    13450       +4
- Partials       1142     1144       +2

Impacted Files
Coverage Δ

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

...er/api/service/impl/TaskDefinitionServiceImpl.java
23.18% <0.00%> (-0.12%)
:arrow_down:

Continue to review full report at Codecov.

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

Is it necessary to add this sql in ddl, just for fix a duplicate data bug?

I think we could do this, but it better do not do it again.
I check table t_ds_task_definition_log, it seems we have sql syntax having count > 1 to make sure we only delete multiple records. So DML in t_ds_task_definition_log look good to me.
But I am not sure the unique key of table t_ds_process_task_relation_log, although it has having count > 1 to make sure the record is multiple. please double check table t_ds_process_task_relation_log unique key here @caishunfeng @JinyLeeChina @lenboo
If we make sure is unique key of two tables. I think it makes sense to me.
But as @caishunfeng said, it is a dangerous behavior due to we use delete syntax in DML because it will auto-run when user installs the software. Maybe we should add an optioned script about this next time. Or even use documents to alert users

But as @caishunfeng said, it is a dangerous behavior due to we use delete syntax in DML because it will auto-run when user installs the software. Maybe we should add an optioned script about this next time. Or even use documents to alert users

Yes, it's a dangerous operation. The user's database structure may be modified, which is beyond our control, so I tend to add this optional operation in the document or issue, and let the user evaluate whether to delete the data, not us.

I think we could do this, but it better do not do it again.
I check table t_ds_task_definition_log, it seems we have sql syntax having count > 1 to make sure we only delete multiple records. So DML in t_ds_task_definition_log look good to me.
But I am not sure the unique key of table t_ds_process_task_relation_log, although it has having count > 1 to make sure the record is multiple. please double check table t_ds_process_task_relation_log unique key here @caishunfeng @JinyLeeChina @lenboo
If we make sure is unique key of two tables. I think it makes sense to me.
But as @caishunfeng said, it is a dangerous behavior due to we use delete syntax in DML because it will auto-run when user installs the software. Maybe we should add an optioned script about this next time. Or even use documents to alert users

I think we could do this
agree +1

but it better do not do it again.
Because we will change our data structure according to unknown requirements in the future, we cannot determine what the future data structure will look like, including adding, modifying and deleting some fields.

But as @caishunfeng said, it is a dangerous behavior due to we use delete syntax in DML because it will auto-run when user installs the software. Maybe we should add an optioned script about this next time. Or even use documents to alert users
Yes, it's dangerous, but we have no choice. We should declare this in the documents, so that users can back up the meta data in advance.

Yes, it's dangerous, Because this is a data problem that only occurs when using version 2.0.1 or 2.0.2, pass 1.3.x will not appear after upgrading. It is more appropriate to put it in the FAQ. I will cancel this PR and add it to the FAQ


[IMPROVEMENT] - Handle multiple string patterns

Currently this library works for the text that takes account into space and does the job. But the text like following doesn't work as intended
text := "Dr. John Doe"
pattern := "<title:string>. <first_name:string> <last_name:string>"

Result:
Dr. instead of Dr
text := "Dr.John Doe"
pattern := "<title:string>.<first_name:string> <last_name:string>"

Result
Dr.John
Do
e
Instead of Dr, John, Doe

Thanks for reporting this issue. We would love your contribution as a patch for this. ✌🏻


Is there a plan to join the feature of obtaining the list of installed software in the system?

@jincandev yes! Version 6 of then package (which I am working on) will have this feature.

@sebhildebrandt I had to do this for my app and found that I got the best data from the following commands:
reg query HKLM\Software\Wow6432Node\Microsoft\Windows\CurrentVersion\Uninstall /s
reg query HKLM\Software\Microsoft\Windows\CurrentVersion\Uninstall /s
I was also using WMI but found that the reg queries had all of the information that WMI did and then some.
I could probably put together a PR for the current version 5 if you are interested.

@shoustech … for macOS and Windows I have a pretty good solution already, for Linux this is a little more tricky as there are different package managers, snap, … in place ;-) But yes, I could also add this code to version 5 … as soon as I have time for it, I will have a look on it.


Use of urllib3 exposed to vulnerability CVE2018-20060

Hey,
I see that you're keeping urllib3 for legacy reasons https://github.com/boto/botocore/issues/1613.
We use your boto3 lib and found out that its opening our code (and anyone else who is including botocore) to a high severity vulnerability CVE2018-20060
Please could you reconsider removing (or updating) this library, to remove the vulnerability?

It only opens your code to vulnerabilities if you are using it. If you are using the documented interface of botocore then you're fine. botocore does not and will not use it, so botocore / boto3 / cli usage does not expose you to those vulnerabilities.
We plan on removing that code in our next major version bump.

Do you know when that major version bump is planned? It does show up as a high-sev vulnerability in container image scans.

I don't consider myself an expert on vendored python modules, but I believe aws-cli may be unintentionally importing the vendored urllib3 from botocore via this line here:
https://github.com/aws/aws-cli/blob/develop/awscli/customizations/configure/__init__.py#L14
Probably the aws-cli issues is a better place for this but I thought I'd drop a note here, too.
I will file a report with the aws-cli issue tracker and hopefully someone can comment on whether there is an issue or not.

Also this issue seems to be essentially very similar to #1608.

@JordonPhillips,
According to @joguSD in aws/aws-cli#4082 --

The only usage of this vendored package is extending the exception classes in case customers were using the vendored exceptions in try/excepts.

Since the usage is so basic here, is there any reason not to update urllib3 to an unaffected version, verify that the exception still works, and include the change in a maintenance release? I understand that the package is unaffected by the actual vulnerability, but it does raise flags and, e.g. with aquasecurity/kube-hunter#112, that can cause minor headaches.

Right, this is currently our most common "high-severity" CVE reported across our containers. Having to make lots of exceptions & triple check that it really isn't used.

We don't vendor urllib3 directly anymore and allow up to the latest release. I'm going to resolve this since we shouldn't be exposing any of these issues in a current release.

Any update?

Hi @struginskij, could you clarify what update you were looking for? As stated in the last comment, Boto3/Botocore hasn’t provided or supported this version of urllib3 in years.

However there is still two files of urllib3 in botocore/botocore/vendored/requests/packages/urllib3/.
In init.py there is set version = '1.10.4' which is old, it causes issues with dependencies check that my company use.

This will be resolved with #2669.


op2 support #[string] with Option<&str> and Option<Cow<str>>

This is now supported: https://github.com/denoland/deno_core/blob/5bbf89874f2169b237051ca883aa601e649d10b9/ops/op2/signature.rs#L1768-L1771

proc-macro panicked: Failed to parse #[op2]:
 - Failed to map signature to V8
 - Unable to map argument OptionString(RefStr) to a slow argument

version deno_core = "0.312.0"
#[op2]
pub fn fnuc(
   #[string] s: Option<&str>,
) -> std::io::Result<()> {
   Ok(())
}

@bartlomieju

@littledivy please take a look


Using TypedController for nested input and useFormContext

Hello,
I'm trying to use a TypedController inside my nested input components combined with useFormContext.
I want to maintain the type-safety of name and defaultValue props, using the example from the README of this repository as a basis.
Here's my working reproduction of the issue: stackblitz/Amiryy/react-hook-form-strictly-typed-poc
So I've composed a NestedInput component:
import { DeepPath, DeepPathValue } from '@hookform/strictly-typed/dist/types';

// NOTE the usage of DeepPath for the types of name and defaultValue props.
// I need these props to match the types of TypedController's name & defaultValue props

interface NestedInputProps<T, Path extends DeepPath<T, Path>> {
  name: Path;
  defaultValue: DeepPathValue<T, Path>
}

function NestedInput<T, Path extends DeepPath<T, Path> = keyof T>(
  props: NestedInputProps<T, Path>
) {
  const { control, handleSubmit } = useFormContext<T>();
  const TypedController = useTypedController<T>({ control });

  return (
    <TypedController
      name={props.name}
      defaultValue={props.defaultValue}
      as={"input"}
    />
  )
}

To make TS accept my code, I had to use imported types such as DeepPath and DeepPathValue for my name and defaultValue props.
As you can see in the snippet above - the second argument of my generic NestedInputProps is Path, which for my understanding represents the path to a nested (or not) property in my form's object type. This type is then used for name prop to match TypedController's name prop's type.
Once I've tried to use my NestedInput inside a form I couldn't avoid having to pass the Path argument:
    {/*
      MY NESTED EXAMPLES: I am forced to pass a duplication of name prop to make both name
      and defaultValue props type-safe
    */}
        <NestedInput<FormValues, "flat">
          name={"flat"}
          defaultValue={"flat input"}
        />
        {/* ❌: not passing additional the argument - TS should complain here on defaultValue being a number but it doesn't */}
        <NestedInput<FormValues /*, "flat" */>
          name={"flat"}
          defaultValue={1}
        />

        <NestedInput<FormValues, ["letters", "a"]>
          name={["letters", "a"]}
          defaultValue={"A"}
        />
        {/* ❌: not passing additional the argument - keyof FormValues is expected by default */}
        <NestedInput<FormValues /*, ["letters", "a"] */>
          name={["letters", "a"]}
          defaultValue={"A"}
        />

        <NestedInput<FormValues, ["letters", "b"]>
          name={["letters", "b"]}
          defaultValue={2}
        />
        {/* ❌: Type 'string' is not assignable to type 'number'. */}
        <NestedInput<FormValues, ["letters", "b"]>
          name={["letters", "b"]}
          defaultValue={"B"}
        />
        {/* ❌: Type '"c"' is not assignable to type '"a" | "b"'. */}
        <NestedInput<FormValues, ["letters", "c"]>
          name={["letters", "c"]}
          defaultValue={1}
        />

It seems like when using <TypedController /> directly the types of name and defaultValue are inferred by their values, so when passing the name prop a property's path, makes defaultValue expect a corresponding type of the property's value.
(from the code example):
{/* ❌: Type '"notExists"' is not assignable to type 'DeepPath<FormValues, "notExists">'. */}
<TypedController as="input" name="notExists" defaultValue="" />

{/* ❌: Type 'number' is not assignable to type 'string | undefined'. */}
<TypedController
    as="input"
    name={['nested', 'object', 0, 'notExists']}
    defaultValue=""
/>

{/* ❌: Type 'true' is not assignable to type 'string | undefined'. */}
<TypedController as="input" name="flat" defaultValue={true} />

And so my bottom-line question is... how can I maintain this usability when passing down name and defaultValue props to a nested <TypedController />?
Thanks!

@Amiryy Hello, Thanks for your feedback 👍
TypedController supports Context.
const TypedController = useTypedController<FormValues>({});

You don't have to create a NestedInput.
import { useTypedController } from '@hookform/strictly-typed';
import { useForm, FormProvider } from 'react-hook-form';

interface FormValues {
  flat: string;
  count: number;
  letters: {
    a: string,
    b: number
  }
  nested: {
    object: { test: string };
    array: { test: boolean }[];
  };
};

function App() {
  const form = useForm<FormValues>();
  const TypedController = useTypedController<FormValues>({});

  const onSubmit = form.handleSubmit((data) => console.log(data));

  return (
    <FormProvider {...form}>
      <form onSubmit={onSubmit}>
        <TypedController
          name="flat"
          defaultValue=""
          render={(props) => <TextField {...props} />}
        />

        <TypedController
          as="textarea"
          name={['nested', 'object', 'test']}
          defaultValue=""
          rules={{ required: true }}
        />

        <TypedController
          name={['nested', 'array', 0, 'test']}
          defaultValue={false}
          render={(props) => <Checkbox {...props} />}
        />

        {/* ❌: Type '"notExists"' is not assignable to type 'DeepPath<FormValues, "notExists">'. */}
        <TypedController as="input" name="notExists" defaultValue="" />

        {/* ❌: Type 'number' is not assignable to type 'string | undefined'. */}
        <TypedController
          as="input"
          name={['nested', 'object', 0, 'notExists']}
          defaultValue=""
        />

        {/* ❌: Type 'true' is not assignable to type 'string | undefined'. */}
        <TypedController as="input" name="flat" defaultValue={true} />

        <input type="submit" />
      </form>
    </FormProvider>
  );
}

@kotarella1110  Thank you for the reply.
Frankly, you didn't really provide a solution for my use case.
My issue here was with maintaining the type-safety of TypedController when it is wrapped in a nested component inside a form.
My example was simple hence didn't practically require nesting the input, but I didn't try to avoid nesting my inputs. I tried to demonstrate to you what happens when I do.
My question is what if I want to nest my inputs? I am trying to integrate this TypedController in my form, and I don't like being forced to render everything directly under my <form> element, the use of excessive typing (see my explanation about the Path type argument) or having to copy-paste the same pattern:
<TypedController
    name={"fieldName"}
    defaultValue={"my default value"}
    render={({ onChange, onBlur, value }) => (
      <FormInputRaw
          onChange={onChange}
          onBlur={onBlur}
          value={value}
        />
    )}
  />

instead of just using a reusable wrapper that is rendering a TypedController around my FormInputRaw:
<FormInput name={"fieldName"} defaultValue={"my default value"} />

TypedController supports Context.

You didn't demonstrate it in any way. In what way does it support and how can it help me here? The code you shared is from the example, I couldn't find any special usage there.
Thanks.

@Amiryy I have two suggestions:

Currying for generics infer

import React from "react";
import { TextField } from "@material-ui/core";
import { useTypedController } from "@hookform/strictly-typed";
import { DeepPath, DeepPathValue } from "@hookform/strictly-typed/dist/types";

type FormInputProps<
  TFieldValues extends Record<string, any>,
  TFieldName extends DeepPath<TFieldValues, TFieldName>
> = {
  name: TFieldName;
  defaultValue: DeepPathValue<TFieldValues, TFieldName>;
};

export const createTypedTextField = <
  TFieldValues extends Record<string, any>
>() => {
  const TypedTextField = <
    UFieldValues extends TFieldValues,
    TFieldName extends DeepPath<UFieldValues, TFieldName>
  >({
    name,
    defaultValue
  }: FormInputProps<UFieldValues, TFieldName>) => {
    const TypedController = useTypedController<TFieldValues>({});
    return (
      <TypedController
        name={name}
        defaultValue={defaultValue}
        render={(props) => <TextField {...props} />}
      />
    );
  };

  return TypedTextField;
};

Pass control object for generics infer

import React from "react";
import { TextField } from "@material-ui/core";
import { useTypedController } from "@hookform/strictly-typed";
import {
  DeepPath,
  DeepPathValue,
  UnpackNestedValue,
  FieldValuesFromControl
} from "@hookform/strictly-typed/dist/types";
import { Control } from "react-hook-form";

type FormInputProps<
  TFieldValues extends Record<string, any>,
  TFieldName extends DeepPath<TFieldValues, TFieldName>,
  TControl extends Control
> = {
  name: TFieldName;
  defaultValue: DeepPathValue<TFieldValues, TFieldName>;
  control: TControl;
};

const TypedTextField = <
  TFieldValues extends UnpackNestedValue<FieldValuesFromControl<TControl>>,
  TFieldName extends DeepPath<TFieldValues, TFieldName>,
  TControl extends Control
>({
  name,
  defaultValue,
  control
}: FormInputProps<TFieldValues, TFieldName, TControl>) => {
  const TypedController = useTypedController({ control });
  return (
    <TypedController
      name={name}
      defaultValue={defaultValue}
      render={(props) => <TextField {...props} />}
    />
  );
};

export default TypedTextField;

CodeSandbox: https://codesandbox.io/s/react-hook-form-usetypedcontroller-14-lvv9r
Does this answer your questions?

@kotarella1110 Thank you for your answer, I highly appreciate the effort.
I had preferred solution No.1 over No.2
My goal was to simplify the typings as much as possible while maintaining the type-safety of the input in a reusable way.
No.2 requires passing control down to every input in order to make it work, which seems unnecessary as it is accessible from within the component, via context.
No.1 - Currying for generics infer worked really well for me. it does require an additional declaration but doing it once per file is enough to make all inputs within that file type-safe for my form data.
Eventually I came up with this structure:
interface NestedInputProps<
  FormType extends UnpackNestedValue<FieldValuesFromControl<Control>>,
  Path extends DeepPath<FormType, Path>,
> {
  name: Path;
  defaultValue: DeepPathValue<FormType, Path>
}

function createTypedInput<FormType extends Record<string, any>>() {
  return <Path extends DeepPath<FormType, Path>>(props: NestedInputProps<FormType, Path>) => {
    return <NestedInput {...props} />
  }
}

function NestedInput<
  FormType extends UnpackNestedValue<FieldValuesFromControl<Control>>,
  Path extends DeepPath<FormType, Path>,
>(
  props: NestedInputProps<FormType, Path>
) {
  const { control, handleSubmit } = useFormContext<FormType>();
  const TypedController = useTypedController<FormType>({ control });

  return (
    <TypedController
      name={props.name}
      defaultValue={props.defaultValue}
      as={'input'}
    />
  )
}

which can be used like this:
const TypedFormInput = createTypedInput<{ someField: string }>();
...
<TypedFormInput name={"someField"} />

I've updated my previous example with this working solution:
https://stackblitz.com/edit/react-hook-form-strictly-typed-poc?file=index.tsx
Thanks!

I posted an alternative solution for deeply nested structures in case it helps: