Custom logger

In order to use CocoaLumberJack in addition with Network Logging, I added a closure to manage the print operation itself.
Its default behaviour is the exact same as today, just a printcall.

We've also added the possibility to filtered what should be logged with a white or black list.


Arduino_1.ino

Please format your PR according to the contribution guide.

I have changed the name according to the contribution guide. Please merge my pull request.

You changed the PR name but not the file name...


ERROR: ExpansionError during parsing /home/jarvis/poky/meta-ros/recipes-extended/wxwidgets/wxwidgets_2.9.5.bb

Hi all,
I am trying to build ros to my yocto image poky and i have followed ROS's official page documentation but when i run "bitbake roslaunch" or any of other ros package name i am getting the following error
since we found two links which helpus build ros and opencv bettter we followed them today and ended up with erros for ros, both are cross-compilation method
the respective errors i have posted a thread in the respective github and meta-ros thread
Do help me overcome this error

I am sorry but I don't know what error you are reporting and how to reproduce this. The issue report seems to be incomplete.

what else are you expecting with respect to the issue so that i can provide the required complete information

Is this a duplicate of https://github.com/bmwcarit/meta-ros/issues/578? Are you using the current master of meta-ros? What is your build configuration?

Basically i am following the Installation Guide: http://wiki.ros.org/hydro/Installation/OpenEmbedded to cross-compile meta-ros so that i can add the required packages to my my Build image
Yes that is the error i was referring to almost same type, but in #578 i dint get the rectification method
And yes, i am using meta-ros in master branch.
I'll send my build configuration in a while

This is my build configuration:
Build Configuration:
BB_VERSION           = "1.36.0"
BUILD_SYS            = "x86_64-linux"
NATIVELSBSTRING      = "universal"
TARGET_SYS           = "i586-poky-linux"
MACHINE              = "qemux86"
DISTRO               = "poky"
DISTRO_VERSION       = "2.4.2"
TUNE_FEATURES        = "m32 i586"
TARGET_FPU           = ""
meta
meta-poky
meta-yocto-bsp       = "rocko:3d66766a49682d8e236390ebedae6ae59b97116c"
meta-oe
meta-python
meta-networking      = "master:4598769a96980b756b3489ac71ed46a075cfabd7"

Hmm, do you actually need the wxwidgets recipe? If not, you can simply delete that file and the depending recipes in meta-ros and it should work. That is certainly a simple work around.
To actually repair the wxwidgets recipe, I would to see the specific error message, "ExpansionError" does not tell enough information; but I assume you can fix it by understanding what we did for #578 and the according commit #579.

Sure thanks,
So finally what method would you suggest for installing ROS related
packages on to yocto?
Because when I directly included the meta-ros layer and then when I added
the recipes to core-extra-image-instsll and build the image ,the image was
successfully built but the ros packages like : rospy, roslib were not
importing in the python shell of poky.
And that is when I found the ros official website guidance,
So what would be the exact methodology that you'd suggest me to go ahead.
That is what is the exact procedure to follow to have Ros related packages
be included in poky.
These tips and suggestions would be of great value to my work.
Thanks
On Mon, 26 Mar 2018, 19:26 Lukas Bulwahn<EMAIL_ADDRESS>wrote:

Hmm, do you actually need the wxwidgets recipe? If not, you can simply
delete that file and the depending recipes in meta-ros and it should work.
That is certainly a simple work around.
To actually repair the wxwidgets recipe, I would to see the specific error
message, "ExpansionError" does not tell enough information; but I assume
you can fix it by understanding what we did for #578
https://github.com/bmwcarit/meta-ros/issues/578 and the according
commit #579 https://github.com/bmwcarit/meta-ros/pull/579.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bmwcarit/meta-ros/issues/584#issuecomment-376174603,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWbYr1ZozZf3q4wrGJHK65Mmkk1LNrvcks5tiPOFgaJpZM4S6zEl
.

I am sorry but I don't understand your question. I am the maintainer of meta-ros and the author of this documentation on the ROS wiki. I try to keep meta-ros running on the current master branch of openembedded-core and meta-openembedded, but some minor issues can always appear on "older" release versions of yocto due to backwards-incompatible changes that are simply unavoidable.

Hi Lukas,
Sorry for the confusions in my questions, can i have your personal email-id
so i can give you a clear picture of mu current stance on meta-ros
implementation...?
Thank You
Rajath C S,
9964182112,
Bangalore - 560060
On Mon, Mar 26, 2018 at 7:59 PM, Lukas Bulwahn<EMAIL_ADDRESS>wrote:

I am sorry but I don't understand your question. I am the maintainer of
meta-ros and the author of this documentation on the ROS wiki. I try to
keep meta-ros running on the current master branch of openembedded-core and
meta-openembedded, but some minor issues can always appear on "older"
release versions of yocto due to backwards-incompatible changes that are
simply unavoidable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bmwcarit/meta-ros/issues/584#issuecomment-376185757,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWbYr-jqU-WUrdvZUGZ-NLAfSZRDglOEks5tiPtSgaJpZM4S6zEl
.

Rajath C S,
9964182112,
Bangalore - 560060
On Mon, Mar 26, 2018 at 7:59 PM, Lukas Bulwahn<EMAIL_ADDRESS>wrote:

I am sorry but I don't understand your question. I am the maintainer of
meta-ros and the author of this documentation on the ROS wiki. I try to
keep meta-ros running on the current master branch of openembedded-core and
meta-openembedded, but some minor issues can always appear on "older"
release versions of yocto due to backwards-incompatible changes that are
simply unavoidable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bmwcarit/meta-ros/issues/584#issuecomment-376185757,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWbYr-jqU-WUrdvZUGZ-NLAfSZRDglOEks5tiPtSgaJpZM4S6zEl
.

As this is an open-source community project, I usually provide some basic support through github and discourse.ros.org. However, I assume that as a maintainer I can only provide some hints and the user needs to actually figure out the problem and resolve the issue in his/her build configuration.
I prefer public communication for the sake of better support in the community, and I only discuss very limited personal topics in private email threads. For this current issue, I don't see the need for   a private discussion. If you still feel that a private email thread would be required, you can find my email on my github profile or in the recent git commit messages.

Question was resolved in non-public email.

i'd like to know does the process being followed in this link lead us to
installation of ROS-kinetic / indigo or others, hope i am clear..? that is
want to know if the process of including and compiling meta-ros layer for
yocto equal to install ros on ubuntu i mean in literal sense

http://wiki.ros.org/hydro/Installation/OpenEmbedded

Is the wxwidgets_2.9.5.bb recipe important because i am getting the
below error when 'bitbake roslaunch' is used

ERROR: ExpansionError during parsing
/home/jarvis/poky/meta-ros/recipes-extended/wxwidgets/wxwidgets_2.9.5.bb
| ETA:  0:02:07
Traceback (most recent call last):
bb.data_smart.ExpansionError: Failure expanding variable CXXFLAGS[:=],
expression was ${@oe_filter_out('-fvisibility-inlines-hidden', ' -O2 -pipe
-g -feliminate-unused-debug-types
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0=/usr/src/debug/wxwidgets/2.9.5-r0
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot-native=
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot=
-fvisibility-inlines-hidden', d)} which triggered exception NameError: name
'oe_filter_out' is not defined
Thanks

Hi Lukas,
So, I tried the meta-ros inclusion again on my system and even this time i faced the similar error of wxwidgets_2.9.5.bb expansion error and as you suggested me in earlier comments i removed the respective recipe and performed the build again for which it threw the following error
"ERROR: No recipes available for:
/home/jarvis/poky/meta-ros/recipes-devtools/python/python-nose_1.2.1.bbappend
/home/jarvis/poky/meta-ros/recipes-devtools/python/python-numpy_1.7.0.bbappend
"
i will try including those python related recipes and build again, i'd also like to attach the complete details of previous error as asked (since i had'nt sent it previously)
`ERROR: ExpansionError during parsing /home/jarvis/poky/meta-ros/recipes-extended/wxwidgets/wxwidgets_2.9.5.bb                          | ETA:  0:02:07
Traceback (most recent call last):
bb.data_smart.ExpansionError: Failure expanding variable CXXFLAGS[:=], expression was ${@oe_filter_out('-fvisibility-inlines-hidden', ' -O2 -pipe -g -feliminate-unused-debug-types -fdebug-prefix-map=/home/jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0=/usr/src/debug/wxwidgets/2.9.5-r0 -fdebug-prefix-map=/home/jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot-native= -fdebug-prefix-map=/home/jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot=  -fvisibility-inlines-hidden', d)} which triggered exception NameError: name 'oe_filter_out' is not defined
Summary: There was 1 ERROR message shown, returning a non-zero exit code.
`
I'd like to know some solving pointers for this specific error
Thanks,
Rajath

i understand that this isnt specific to ros queries but i do have posted
the same at the ROS mailing-list and to ROS maintainers and the issue is to
be yet resolved perfectly.
Thanks
Rajath C S,
9964182112,
Bangalore - 560060
On Fri, Apr 6, 2018 at 12:34 PM, Rajath C S<EMAIL_ADDRESS>wrote:

i'd like to know does the process being followed in this link lead us
to installation of ROS-kinetic / indigo or others, hope i am clear..? that
is want to know if the process of including and compiling meta-ros layer
for yocto equal to install ros on ubuntu i mean in literal sense

http://wiki.ros.org/hydro/Installation/OpenEmbedded

Is the wxwidgets_2.9.5.bb recipe important because i am getting the
below error when 'bitbake roslaunch' is used

ERROR: ExpansionError during parsing /home/jarvis/poky/meta-ros/
recipes-extended/wxwidgets/wxwidgets_2.9.5.bb                          |
ETA:  0:02:07
Traceback (most recent call last):
bb.data_smart.ExpansionError: Failure expanding variable CXXFLAGS[:=],
expression was ${@oe_filter_out('-fvisibility-inlines-hidden', ' -O2
-pipe -g -feliminate-unused-debug-types -fdebug-prefix-map=/home/
jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-
linux/wxwidgets/2.9.5-r0=/usr/src/debug/wxwidgets/2.9.5-r0
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/
build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot-native=
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/
build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot=
-fvisibility-inlines-hidden', d)} which triggered exception NameError: name
'oe_filter_out' is not defined
Thanks

`ERROR: ExpansionError during parsing /home/jarvis/poky/meta-ros/
recipes-extended/wxwidgets/wxwidgets_2.9.5.bb | ETA: 0:02:07
Traceback (most recent call last):
bb.data_smart.ExpansionError: Failure expanding variable CXXFLAGS[:=],
expression was ${@oe_filter_out('-fvisibility-inlines-hidden', ' -O2
-pipe -g -feliminate-unused-debug-types -fdebug-prefix-map=/home/
jarvis/poky/openembedded-core/build/tmp-glibc/work/i586-oe-
linux/wxwidgets/2.9.5-r0=/usr/src/debug/wxwidgets/2.9.5-r0
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/
build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot-native=
-fdebug-prefix-map=/home/jarvis/poky/openembedded-core/
build/tmp-glibc/work/i586-oe-linux/wxwidgets/2.9.5-r0/recipe-sysroot=
-fvisibility-inlines-hidden', d)} which triggered exception NameError: name
'oe_filter_out' is not defined
Summary: There was 1 ERROR message shown, returning a non-zero exit code.
`
I'd like to know some solving pointers for this specific error

Can you spot the exception's message "NameError: name 'oe_filter_out' is
not defined"? This is the first pointer. I presume the prefix "oe_" stems
from "OpenEmbedded", so there's a chance that this is a function meant to
be defined in OpenEmbedded-Core. Try to find where this function is
documented. Usually Google Search helps with it. It might happen that the
function got deprecated and was removed from the core. In such case there
must be a replacement available.
Or it's your project that was configured improperly and the function was
not imported in the recipe for some reason. Then check what's required to
make the function imported.


For repo scoped tokens make sure we use the base repo

External contributions were using head, the fork, as the repo name when it should have been using the base repo.
Change-type: patch

@klutchell pull_request.base is the branch being targeted by the pull request, right (i.e. master)?

@klutchell pull_request.base is the branch being targeted by the pull request, right (i.e. master)?

correct, or in this case the repo name or the master branch being targeted


resolve issue #216 for 1.5.x branch return static noquirks instances

Here is the fix for issue #216

Not sure why this broke. Is it because travis can't work with the 1.5x branch? Same patch works on the master branch. Also works fine locally. Is there something I can do to help?


A11y_Azure Data Studio_Toolbar_Zoom In_Zoom: At resize the tooltip present at some of the nodes are getting truncated.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:
#A11yMAS; #A11yTCS; #A11ySev2; #SQL Azure Data Studio; #SH-SQL Azure Data Studio-Win32-OCT2022; #DesktopApp; #Zoom; #WCAG1.4.4; #FTP; #Regressed:11-01-22; #Closed
Environment Details:
Application Name: Azure Data Studio
Windows Version: Windows 11
Repro Steps:

Launch the "Azure data studio" application.
Connect with the provided "sqltools2019-3" SQL server.
TAB to "New Query" button and hit ENTER key.
Enter the SQL query and TAB to "Estimated Plan" button and hit ENTER key.
"Query Plan (Preview) screen appear.
TAB to the toolbar present at right side of the screen.
TAB to "Zoom In" button and hit ENTER key.
Zoom the screen to 200% and tab to any node having long tooltip observe the issue.

Actual:
At resize the tooltip present at some of the nodes are getting truncated.
This issue is also observed for the Zoom Out/ Zoom to fit operations also.
Expected Result:
At resize also the tooltip present at any of the nodes should not truncate and should be properly visible.
Attachment:

@Rama8345, this issue is fixed in the latest insiders. Can you please re test it?

@aasimkhan30
Verified the bug in the latest insiders build, but we are getting below error.

Verified the issue in the latest insiders build. The issue is fixed. So closing the bug. Please find attachment for reference.


Do not send preedit when closing Maliit Keyboard

It's wrong to send preedit on reset/close, it's the framework's or input
context's responsibility to handle preedit correctly when focus is lost.

Closing as this hasn't been touched in almost 9 years, and plugins is unmaintained in favor of github.com/maliit/keyboard.


fix food chain exercise

#399

PR is already pending for this work.  I should pay more attention.

😆 😆
It happens from time to time. Don't let that to stop you 💪


tensorflow 1.9.0:  libforestprotos.so: cannot open shared object file

Issue description
On loading tensorflow.contrib, the library libforestprotos.so can not be found.
Steps to reproduce
% nix-shell --pure -p 'python36.withPackages (p: with p; [ tensorflowWithCuda ])' --run 'python -c "import tensorflow.contrib"'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
...
  File "/nix/store/mwcxhmw4dib7g33ly2hwyxlgqwwha8l8-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/python/ops/tensor_forest_ops.py", line 28, in <module>
    resource_loader.get_path_to_datafile('_tensor_forest_ops.so'))
  File "/nix/store/mwcxhmw4dib7g33ly2hwyxlgqwwha8l8-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/util/loader.py", line 56, in load_op_library
    ret = load_library.load_op_library(path)
  File "/nix/store/mwcxhmw4dib7g33ly2hwyxlgqwwha8l8-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/python/framework/load_library.py", line 56, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: libforestprotos.so: cannot open shared object file: No such file or directory

The file exists in the module directory.
% find /nix/store/mwcxhmw4dib7g33ly2hwyxlgqwwha8l8-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/ -iname libforestprotos.so
/nix/store/mwcxhmw4dib7g33ly2hwyxlgqwwha8l8-python3-3.6.6-env/lib/python3.6/site-packages/tensorflow/contrib/tensor_forest/libforestprotos.so

Technical details
% nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.67, NixOS, 18.09pre150708.083220867c7 (Jellyfish)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.0.4`
 - channels(bara): `""`
 - channels(root): `"nixos-19.03pre152634.218ce4de508, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@jyp seen your name on #44599 and #45709 do you have an idea where to look?

@jyp thank you for the help.
I have added contrib/tensor_forest to the rpath and it is working https://github.com/NixOS/nixpkgs/compare/master...baracoder:tf-fix-contrib
Would you add it to your pull request for tensorflow 1.10.1 or should it be a different pull request?
I have checked for any other missing libraries with
find /nix/store/pns825s1pbqkihavqi6cmclhb6mv64xn-python3.6-tensorflow-1.9.0/ -iname '*.so*' -exec bash -c 'echo {} ; ldd {}' \;|grep -C 6 'not found'

There is libnvinfer.so.4 missing, it is nowhere on my system, but I don't know what the effect of it is.

Thanks for debugging this!
The tensorflow 1.10.1 PR has been queued for a very long time now. You may
have a better chance with a separate PR.
On Fri, Sep 21, 2018 at 11:49 AM baracoder<EMAIL_ADDRESS>wrote:

@jyp https://github.com/jyp thank you for the help.
I have added contrib/tensor_forest to the rpath and it is working
master...baracoder:tf-fix-contrib
https://github.com/NixOS/nixpkgs/compare/master...baracoder:tf-fix-contrib
Would you add it to your pull request for tensorflow 1.10.1 or should it
be a different pull request?
I have checked for any other missing libraries with
find /nix/store/pns825s1pbqkihavqi6cmclhb6mv64xn-python3.6-tensorflow-1.9.0/ -iname '.so' -exec bash -c 'echo {} ; ldd {}' ;|grep -C 6 'not found'
There is libnvinfer.so.4 missing, it is nowhere on my system, but I don't
know what the effect of it is.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/46841#issuecomment-423478277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABsYxLsQ8v17wIVqUygYnxY_hdtCPBRks5udLY9gaJpZM4WuOpz
.


Add a test for Mistral-NeMo.

See issue #1285.
What does this PR do?
Tries to add a test case for #1285.
Note that at this time there's an nvFuser bug that prevents this from running.
As discussed interactively, I used from_config to skip downloading any weights. I couldn't get around downloading "tiny_shakespeare", but at 1.2mb this doesn't seem like an odious requirement. Still, if anyone has ideas around that please comment here.

Latest version moves the code where it belongs, in test_networks.py, as Masaki pointed out.
I also rebuilt my container and that made it clear that there's a couple difficulties with this:

I needed to turn trust_remote_code on to get the tiny shakespeare dataset to work
One needs to have an active login to huggingface's hub to grab configurations
This requires some dependencies (transformers, datasets) that we wouldn't otherwise depend on.

As such I made this skipped by default. I still think it makes sense to store as a thunder test, as that's a logical source of truth for the whole team to work on. But I invite discussion.

@t-vi this is ready for review.
The CI failure was a node timing out running the tests, even though all this does is add a single, skipped test; seems like there's more going on there. Do you want me to add empty commits until this happens to pass or can you override that?

thank you @tfogal
I don’t think we need tiny shakespeare to be downloaded, we can get away with something simpler or  random (just like with other examples), this way we don’t even have to get the tokenizer
do we actually need credentials for the configs or is it the tokenizer checkpoint that requires it?

do we actually need credentials for the configs or is it the tokenizer checkpoint that requires it?

There should be a way to avoid using any credentials since the tokenizer is unnecessary and we don't load any weights for the config.
Usually, we check Thunder's ability to run a network with a sample random input and then invoke backward. A mock training loop is unnecessary here and can be avoided. Here's a patch that would verify that Thunder successfully runs the model without any dataset, optimizers, etc:
diff --git a/thunder/tests/test_networks.py b/thunder/tests/test_networks.py
index 57a9a759..9bf0cb57 100644
--- a/thunder/tests/test_networks.py
+++ b/thunder/tests/test_networks.py
@@ -362,7 +362,7 @@ def test_quantization():

 @thunder.tests.framework.requiresCUDA
-@pytest.mark.skip(reason="Dependencies, trust issues")
+# @pytest.mark.skip(reason="Dependencies, trust issues")
 def test_thunderfx_mistral_nemo_small():
     """
     Runs a small version of Mistral-NeMo
@@ -370,17 +370,9 @@ def test_thunderfx_mistral_nemo_small():
     This is largely based on code from Alexandros Koumparoulis.
     """
     import transformers
-    import datasets

     model_id = "mistralai/Mistral-Nemo-Base-2407"

-    tokenizer = transformers.AutoTokenizer.from_pretrained(
-        model_id,
-        torch_dtype=torch.bfloat16,
-        ignore_mismatched_sizes=True,
-        trust_remote_code=False,
-    )
-
     # Setup a "small" version of NeMo-Mistral that does not require downloading
     # weights. This is not a configuration that is worth benchmarking.
     # This was created by using
@@ -389,7 +381,7 @@ def test_thunderfx_mistral_nemo_small():
     #   transformers.AutoConfig.from_pretrained(model_id)
     # until they lined up.
     config = transformers.models.mistral.configuration_mistral.MistralConfig(
-        num_hidden_layers=2,
+        num_hidden_layers=1,
         torch_dtype=torch.bfloat16,
         max_position_embeddings=1024,
         architectures=["MistralForCausalLM"],
@@ -404,53 +396,18 @@ def test_thunderfx_mistral_nemo_small():
     model = transformers.AutoModelForCausalLM.from_config(config)
     device = torch.device("cuda")
     model.to(device)
-    mdl = torch.compile(model, backend=thunder.dynamo.ThunderCompiler())
+    backend = thunder.dynamo.ThunderCompiler()
+    mdl = torch.compile(model, backend=backend)
     del model

-    # Add a padding token to the tokenizer
-    if tokenizer.pad_token is None:
-        tokenizer.add_special_tokens({"pad_token": "[PAD]"})
-        mdl.resize_token_embeddings(len(tokenizer))
-
-    dataset = datasets.load_dataset("tiny_shakespeare", split="train", trust_remote_code=True)
-
-    def tokenize_function(examples):
-        return tokenizer(examples["text"], padding="max_length", truncation=True, max_length=2)
+    batch_size = 1
+    input_ids = torch.randint(0, config.vocab_size, (batch_size, config.max_position_embeddings), device=device)
+    attention_mask = torch.ones_like(input_ids)

-    tokenized_dataset = dataset.map(tokenize_function, batched=True, remove_columns=["text"])
-
-    # Convert the dataset to PyTorch format and specify columns to return as tensors
-    tokenized_dataset.set_format(type="torch", columns=["input_ids", "attention_mask"])
-
-    dataloader = torch.utils.data.DataLoader(tokenized_dataset, batch_size=1, shuffle=True)
-
-    # Define optimizer and learning rate scheduler
-    optimizer = torch.optim.AdamW(mdl.parameters(), lr=5e-5)
-    num_epochs = 3
-    lr_scheduler = transformers.get_scheduler(
-        "linear",
-        optimizer=optimizer,
-        num_warmup_steps=0,
-        num_training_steps=num_epochs * len(dataloader),
-    )
+    output = mdl(input_ids=input_ids, attention_mask=attention_mask, labels=input_ids)
+    logits = output.logits
+    grad_logits = torch.randn_like(logits)
+    logits.backward(grad_logits)

-    mdl.train()
-    for epoch in range(num_epochs):
-        total_loss = 0
-        for batch in dataloader:
-            # Move input tensors to device
-            input_ids = batch["input_ids"].to(device)
-            attention_mask = batch["attention_mask"].to(device)
-
-            # Forward pass
-            outputs = mdl(input_ids=input_ids, attention_mask=attention_mask, labels=input_ids)
-            loss = outputs.loss
-            total_loss += loss.item()
-
-            # Backward pass
-            optimizer.zero_grad()
-            loss.backward()
-            optimizer.step()
-
-            # Update learning rate
-            lr_scheduler.step()
+    # Check that Thunder has actually compiled the model
+    assert backend.subgraph_infos, "No subgraphs found"

There should be a way to avoid using any credentials

Thanks Luca, Ivan. I've applied Ivan's patch + some minor other changes, and indeed it appears to not download anything new (into ~/.cache/huggingface, at least), now.
Unfortunately in the interim something to seems to have tickled things so that #1240 is now a blocking issue, so I am leaving the skip designation for now :-(

#1240 has been fixed just now, so we can probably remove the skip.

hrm my merge of main seemed to have not gone well. meetings now but i will fix after...

Hi, sorry for the weirdness. I couldn't figure it out why the github diff was wild even though git diff main..tfogal/nemo-test-case was sane, so I just ended up rebasing it.
It's pretty tiny (thanks to Ivan's patch), so hopefully it's not too painful to review from scratch again.

CI failure is real: Found two different const extents in the same set: { bS39{1}; bS43{1}; bS66{1}; bS70{1 ex 8}; bS50{1}; bS54{1 ex 32} } nvFuser issue.
However, it works fine with nvFuser e33316d9480508b49db788a7472f4df52e53af92. Do we need an nvFuser upgrade for this to work?


Toggling comment of a first line of JSX in a block does not work

If I comment out twice in tsx, it will behave strangely.

require("kommentary.config").configure_language("typescriptreact", {
    hook_function = function()
       pre_comment_hook = require('ts_context_commentstring.internal').update_commentstring()
    end,
    prefer_single_line_comments = true,
})

The issue is that if you comment the first line (only the first line) in any JSX block, then there will be a syntax error. For example, this is not valid JS/JSX:
const App = () => (
  {/*
 */}

Hello world

)

Now, treesitter thinks that the first line of JSX with the comment is an object.
But I don't think that it's super high priority for now. I probably won't be dealing with it any time soon.
Initial issue from here: https://github.com/JoosepAlviste/nvim-ts-context-commentstring/issues/10#issuecomment-853151154

I came across this bug today too, is this in essence, an issue that needs to be fixed in TreeSitter?

Hey @sparkbuzz!
I don't think that this is a problem with treesitter because it makes sense that it can't parse the commented code, it's not actually valid syntax. So, I think that something should still be added to this plugin to handle the weird treesitter syntax tree, but it probably requires being able to write more complex configuration for commentstrings.
But there might be another good solution, I haven't given this issue too much thought.

I'll close this in favor of #29 since that one has more information.


DBZ-406 Support for look-ahead buffer with unlimited tx size

Contains both https://issues.jboss.org/browse/DBZ-405 and https://issues.jboss.org/browse/DBZ-406

Done

@jpechane I've pushed one commit with a few adjustments. It looks good to me overall, apart from the testing issue just discussed on Gitter. If you push a fix for that, too, then it's good to go IMO.
One additional optional change is this: should we log a warning when encountering a ROLLBACK in non-buffering mode, hinting people to use the buffering mode? That seems like a friendly help for people running into this issue. Your call whether to add it or not :)
Thanks!

Rebased and applied. Thanks a lot for driving home this one, @jpechane! Just one question - and I guess you saw that coming - can you file a doc PR, too? Cheers!

Doc PR - https://github.com/debezium/debezium.github.io/pull/96


NIFI-8398: Bump ParCEFone version

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Fixes bug NIFI-8398._
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[X] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@exceptionfactory FYI

@joewitt

@exceptionfactory FYI

Thanks @trixpan, I just submitted #4989 to address the package name change for MacAddress and confirmed a successful build on MacOS with Apache Maven 3.8.1.

saw your PR. closing this one


A translation about initialize from qiskit in OpenQASM cannot translate back into qiskit

Environment

Qiskit Terra version: 0.36.1
Python version: 3.7.4
Operating system: Windows

What is happening?
After got the OpenQASM code about initialize operator, it will fail to use from_qasm_file/from_qasm_str to translate it back into qiskit code.
Part of error message is shown below:

File "D:\Program Files\pycharm_project\qiskit-benchmarks\venv\lib\site-packages\ply\yacc.py", line 1120, in parseopt_notrack
p.callable(pslice)
File "D:\Program Files\pycharm_project\qiskit-benchmarks\venv\lib\site-packages\qiskit\qasm\qasmparser.py", line 400, in p_id_e
raise QasmError("Expected an ID, received '" + str(program[1].value) + "'")
qiskit.qasm.exceptions.QasmError: "Expected an ID, received 'reset'"

How can we reproduce the issue?
from qiskit import QuantumCircuit
circuit = QuantumCircuit(1)
circuit.initialize('0', circuit.qubits)
qasm_str = circuit.qasm()
print(qasm_str)
same_circuit = QuantumCircuit(1)
same_circuit = same_circuit.from_qasm_str(qasm_str)
print(same_circuit)
What should happen?
Run successfully.
Any suggestions?
No response

Sorry for the slow reply!  This is very related to #7351 and the reasons in https://github.com/Qiskit/qiskit-terra/issues/7750#issuecomment-1062405591.  The gist is that initialize is a non-unitary instruction (it includes a reset), and so we can't actually write it as a valid OpenQASM 2 gate.  It's a bug that the exporter here puts a reset instruction within a gate - it should probably just fail at that point.
There won't be a way to round-trip a circuit with initialize in it from QASM 2 even if we do some tricks in the export, because the language just can't represent that instruction without it being inlined into the circuit, sorry.  We could do some tricks with the (unitary) StatePreparation class, but extending the OpenQASM 2 support is a low priority for us right now, sorry.


Typo fix in Definition 7.1 in asterix.tex

Describe the bug
In the definition of a partial solution, the second condition is stated as
The function $f$ is a bijection from $E_1 \backslash E_0$ to $E_2 \backslash E_0$.

but it becomes apparent from later arguments that the intended second condition is
The function $f$ is a bijection from $E_1 \backslash E_0$ to $E_2 \backslash E_1$.

Additional context
I was directed to file a bug report before posting my PR to fix it.

claim

propose #471


Set communities_hostname on main menu

Depends on #213

Closes #201

Could you maybe open a new PR with just the that fix? ^^
(i.e. with no dependency on #213 which is why this was closed in the first place I assume)

I thought this was fixed by a different PR already? I might be wrong though.

Not that I'm aware ^^


Resolve promise with generated manifests

What

now resolves promise with all generated manifests (grouped by cluster)

Testing

verified using functional tests

What was this for?

This is to allow for an endpoint that will return the generated manifests for a given deploy request.


Check to make sure default npm exists at path before trying to use it

Bug
If the typings installer is run under a copy of node that does not include npm—but on a machine that does have npm installed—it will incorrectly try to use the npm that does not exist next to its running version of node
Fix
Make sure that we check that npm we select exists before trying to use it as the default. Otherwise, fall back to using plain old npm

Fixes #30909

@sheetalkamat I checked my mail box and chat histories but couldn't find anything about ATA from last May.  I'm not sure I was involved.
We changed node versions and install paths a few VS versions ago, so it's possible this is no longer an issue.

@amcasey Would old VS Versions still be broken though? Can we perhaps pass in a command line flag to enable this new behavior?

@mjbvz Good catch - yes, it would have to be behind a flag (assuming it works now).


Override container exit with environment variable

With the previous hm-gwmfr container we had the ability to stop the container from exiting using an environment variable:
https://github.com/NebraLtd/hm-gwmfr/blob/1abdf511c71a59a8c24b411f32a7b2117e17b5da/eccProg.py#L72-L78
We need to add this to hm-diag so we can fix the ECC key remotely on devices with errors
Also look at PREVENT_START_GATEWAYCONFIG and see if that pattern will work here.

This is partially solved by #270

We have this now with PREVENT_START_DIAG set to 1
https://github.com/NebraLtd/hm-diag/blob/c8ef9542e845236ca925231362c1f6dec7f09e1a/start_diagnostics.sh#L15
Just needs documenting in #504


fix(backend): apply not getting ran

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#366  👈
#365
#364
#363
#362
#361
#360
#359
#358
#357
#356
#355
#354
#353
#352
#351
#350
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Implement unit conversion for a wider range of units

Adds a pint unit registry, that can be used to convert between a wide range of units.
Eventually, should be able to handle a large set of units, including all valid resqml uoms.

Codecov Report

Merging #102 (e69182d) into master (dfbfc6e) will increase coverage by 0.08%.
The diff coverage is 100.00%.

:exclamation: Current head e69182d differs from pull request most recent head 0889dc5. Consider uploading reports for the commit 0889dc5 to get more accurate results

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   38.75%   38.84%   +0.08%
==========================================
  Files          49       49
  Lines       21726    21757      +31
==========================================
+ Hits         8420     8451      +31
  Misses      13306    13306

Impacted Files
Coverage Δ

resqpy/olio/weights_and_measures.py
87.25% <100.00%> (+2.28%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for the discussion Andy! I'll close this PR for now without merging. To recap our current plan:

Implementing RESQML units of measure explicitly from a built JSON database looks more promising than using pint
We could repurpose some of this code in other repositories that would benefit from broader, general-purpose unit conversion
We can cherry-pick the tests from this branch and use them with our future alternate implementation


Android native focus event firing too fast

Description
Hello everyone !
I'm facing a simple issue I guess.
I'm developing an Android TV app.
Let's imagine an horizontal ScrollView, with 10 TouchableNativeFeedback inside, Then there is few element out of the screen, normal.
When i go right, onFocus fired, and set the ScrollView offset to 200 * index of the element.
The animation goes well, but if i go left, on an element outside of the screen, then the focus is fired too fast to bring the element in the visible screen, and then the animation not fire properly.
I probably missing something, about the native event managing.
Thanks ! :)
Version
70.5
Output of npx react-native info
System:
OS: Windows 10 10.0.17763
CPU: (16) x64 AMD Ryzen 7 5800H with Radeon Graphics
Memory: 4.06 GB / 15.40 GB
Binaries:
Node: 16.13.1 - C:\Program Files\nodejs\node.EXE
Yarn: Not Found
npm: 8.1.2 - C:\Program Files\nodejs\npm.CMD
Watchman: Not Found
SDKs:
Android SDK:
API Levels: 31
Build Tools: 30.0.3, 31.0.0, 33.0.1
System Images: android-29 | Google Play Intel x86 Atom, android-31 | Android TV ARM 64 v8a, android-31 | Android TV Intel x86 Atom, android-31 | ARM 64 v8a, android-31 | Intel x86 Atom_64, android-31 | Google TV ARM 64 v8a, android-31 | Google TV Intel x86 Atom, android-31 | Google APIs ARM 64 v8a, android-31 | Google APIs Intel x86 Atom_64, android-31 | Google Play Intel x86 Atom_64
Android NDK: Not Found
Windows SDK: Not Found
IDEs:
Android Studio: AI-213.7172.25.2113.9123335
Visual Studio: Not Found
Languages:
Java: 1.8.0_352 - C:\Program Files\Eclipse Adoptium\jdk-8.0.352.8-hotspot\bin\javac.EXE
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.5 => 0.70.5
react-native-windows: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Create a new project for Android TV
Display a ScrollView fullscreen with enough Touchables elements inside to go out of the screen on the right and assign a style like 200px.
Set an onFocus on Touchables elements, that update the x position of the scrollview ( 200 * index ). Then the scrollview moove to keep the focus element not mooving.
Navigate with remote or keyboard
Snack, code example, screenshot, or link to a repository
No response

You seem to be using version 0.70.5 of the core React Native repo, and not the React Native provided by this repo, which is currently at 0.69.6-0.


Applying markdownlint fixes

The markdownlint from VSCODE found seven warnings:

MD004 - Unordered list style
MD009 - Trailing spaces
MD012 - Multiple consecutive blank lines
MD022 - Headings should be surrounded by blank lines
MD031 - Fenced code blocks should be surrounded by blank lines
MD032 - Lists should be surrounded by blank lines
MD034 - Bare URL used

Good to fix issues. But it would be even better when everyone can run the test, and we don't rely on specific IDE features. Otherwise the next one editing the files will (re-)introduce old or new issues.
In other words: can we add this check to the Makefile and CI?


[Suggestion] Allow renaming of devices

Allow renaming of devices for users with multiple devices. When you have many devices, the device names can be confusing and not user friendly for some users in the house hold. Removing the device name can show multiple devices with the same name (Eg: Speakers). A custom label or name to replace the device name in the quick select menu would be helpful for these situations.

You can already rename Audio devices from the Control Panel audio devices menu.
Right click on the device you want to rename, then go to properties. There you can set a different name and icon for the device.

Added in latest version


mac build / get_parent is returning boolean

when using the latest mac build (july 9th) get_parent is returning a boolean type.
this also happened in the previous build (july 8th) and those were the only ones that I've tried.
my project is just a blank scene with 2 node2D objects. the child one is calling get_parent() on _ready(). really easy to reproduce. I think this is exclusive to the mac build since other linux users where not able to reproduce it.

Where did you get the build from ?
Have you tried compiling from source ?

Hi. I've got the build from fixnum. Will try to compile from software and post the results here.

From what I read, the fixnum OS X build is broken :

Notes
I currently can't build for iOS. OS X builds are generated with osxcross, using Xcode 7.3.1.
Prior to June 10, 2016 I was unable to produce builds for OS X as well. tonymtbird helped out by making sporadic builds of the tools and templates on his Mac to be provided here. Note that these builds might not correspond to the exact commit of the other platforms.

Try compiling from source.

Please attach a zip of the project you use to reproduce this bug. I don't see how it could be a platform-related bug, a compiler wouldn't change a return type automatically from Node to bool: https://github.com/godotengine/godot/blob/master/scene/main/node.cpp#L1575

From what I read, the fixnum OS X build is broken :

Notes
I currently can't build for iOS. OS X builds are generated with osxcross, using Xcode 7.3.1.
Prior to June 10, 2016 I was unable to produce builds for OS X as well. tonymtbird helped out by making sporadic builds of the tools and templates on his Mac to be provided here. Note that these builds might not correspond to the exact commit of the other platforms.

This actually says that iOS builds are broken, not OS X ;) The part about pre-June 2016 is likely outdated by now.

Ok. But well, I don't see other reason how a Node* could be converted into a bool but from a problem in the code version. As the fixnum builds are "unofficial", I assumed the problem should probably come from the build. But I might be wrong... ^^
Also, this cannot be reproduced on ubuntu.

Just built it from source and the issue remains.
There are some errors on the console though:

Here's a barebones project that reproduces the error:
godot_get_parent_issue.zip
I'm using macOS Sierra in a macbook pro retina mid 2012.

I've just tested your project with latest master and I can confirm this too a OS X Yosemite.

Some paranormal activity on OS X :D

Runs as expected (returns position) using latest build from Git master, Ubuntu Linux 17.04 64-bit.

I made a build yesterday from master and got the same problem.

Ditto here.... doesn't make any sense..

Does this still happen? Could anyone confirm?

Just re-tested with latest master and its fixed :D

\o/
Closing.


Add an option to save files atomically

(prior art: https://github.com/microsoft/vscode/issues/56361)
All our writes in VSCode today operate on the target file:

truncate the file to 0 bytes
write the contents into the file

If a crash happens after truncation, the file will remain empty. This should typically not result in dataloss because we have a backup to recover from. However, some external tools may act up when they get an event for the file being empty and in certain bad situations we still might end up with an empty file (e.g. when auto save is on where we do not backup).
An alternate strategy to ensure a file is consistent is to:

write the contents of the editor into a temporary file
move the file over the original one in a single atomic file operation (my understanding is that moves are atomic unless you move across disks)

While this reduces the chance of corrupt files, the downside to this approach is that some external file watchers might be confused about a file constantly being replaced and not just written to.

This would also be helpful for users of gulp-sass as it can sporadically fail to build when running in watch mode and the recommended solution is to enable atomic saves.
https://github.com/dlmanning/gulp-sass/wiki/Common-Issues-and-Their-Fixes#partials-sporadically-not-found

Since this would be the fix to data loss when disk is full, reopening.

Re: https://github.com/microsoft/vscode/issues/116600

Where is the "backup to recover from"?  (I'm trying to recover from a vscode glitch which deleted the last 1/3 of a 37 MB text file.)

The alternate strategy seems like it might reduce disk fragmentation, for files which gradually grow in length. (Though that doesn't matter much for SSDs.)

This is becoming more interesting in the context of https://github.com/microsoft/vscode/issues/134180: when faced with remote file systems, saving large files is always at risk of partial corruption and slow write operations. Forcing the user to wait for this operation to finish can lead to bad user experience. A better approach would be to implement conditional atomic write support for large files.

I ran out of space on my filesystem and now a source file which I had not yet committed to source control is empty and I have to write the code again.
If files were saved atomically, this would not have happened.

To mitigate the worry of file watchers acting up, instead of renaming the file to the destination, we could also pipe the contents into the file from the temporary file to be as close to the write we do now as possible.

I don't think that's atomic either. In my opinion file watchers should be fixed to support files being saved in this way since that seems to be the standard in many editors. Providing a setting to save files non-atomically for users who don't care about data loss would also be an option.

Running out of space, crashes and power loss are not the only issues here.
Some scripting languages (for example bash) don't load/cache the whole file content before running it.
If I have a long running script and edit it (make improvements / add features) while it's running in the background then it will mix the code from the old script with the new one which can have potentially fatal consequences. Proper code editors that have atomic save (for example vim) don't have this issue since the new save is a different inode (file instance) completely.


Adding external sensors OTA

I tried to add a external sensor (NDIR CO2 sensor) over the air and it failed... the I tried all kinds of figures, sending just the config for new sensor, sending everything.
The configuration is arriving partially in the configuration and the new sensor didn't work. after flashing the howl stuff tu the SPIFFS everything worked nice.

In fact the problem is slightly more global than that. It has to do with the REST API and the fact that we only get the difference between reported and desired.
Example:
shadow state:
{"val1"=1, "val2"=2, "val3"=3}
json from user:
{"val1"=2, "val2"=2, "val3"=1}
json in desired channel
{"val1"=2, "val3"=1}
json in memory after update
{"val1"=2, "val3"=1}
the REST API acts as intended but the coolboard does not keep unchanged values. "that's all"...

To review / define


PLATFORM-6124 admin recovery

Related issue(s)

Checklist

[ ] I have read the contributing guidelines.
[ ] I am following the
contributing code guidelines.
[ ] I have read the security policy.
[ ] I confirm that this pull request does not address a security
vulnerability. If this pull request addresses a security. vulnerability, I
confirm that I got green light (please contact
<EMAIL_ADDRESS>from the maintainers to push
the changes.
[ ] I have added tests that prove my fix is effective or that my feature
works.
[ ] I have added or changed the documentation.

Further Comments

Tests are failing because Travis started the service on a different port


Age restriction bypass fails

The age restriction bypass fails again. Youtube must have changed something. Reproducable with:
go test -timeout 30s -run ^TestDownload_SensitiveContent$

please provide example URL

You find it in the test TestDownload_SensitiveContent(). It's the video ID MS91knuzoOA.

That one only works with authentication:
POST /youtubei/v1/player HTTP/1.1
Host: www.youtube.com
Authorization: Bearer ya29.A0ARrdaM-z6i2TpC5CQ9mw78GU4rDGe956hnSnMv2xhh4plzUoT...

{
 "context": {
  "client": {
   "clientName": "ANDROID",
   "clientVersion": "17.09.33"
  }
 },
 "videoId": "MS91knuzoOA"
}

Found another method that works with most videos, without authentication:
POST /youtubei/v1/player HTTP/1.1
Host: www.youtube.com

{
 "context": {
  "client": {
   "clientName": "TVHTML5_SIMPLY_EMBEDDED_PLAYER",
   "clientVersion": "2.0"
  }
 },
 "videoId": "Tq92D6wQ1mg"
}

However the video youve linked has the highest restriction, so it requires login regardless.

This problem continues... while downloading an age-restricted video in Termox, I get this message:
can't bypass age restriction: embedding of this video has been disabled !
Is it possible to solve this problem by adding a login page?

personally I got tired of the hacks. I try a normal ANDROID request, then ANDROID_EMBEDDED_PLAYER. if those both fail, then I just use OAuth:
POST /youtubei/v1/player HTTP/1.1
Host: www.youtube.com
Authorization: Bearer ya29.a0AfB_byBXjabItN3g_zxyh-9dGNx-tQefkn8hxpbLO_1aYbKlB...
User-Agent: com.google.android.youtube/18.43.39

{
   "contentCheckOk": true,
   "context": {
      "client": {
         "clientName": "ANDROID",
         "clientVersion": "18.43.39"
      }
   },
   "racyCheckOk": true,
   "videoId": "MS91knuzoOA"
}

@3052 How do you get the OAuth token? Thank you.

https://github.com/3052/platform/tree/v1.4.7/youtube

post issue on my repo or contact me privately


Add scottmcm

I said a while back I'd help out with some libs-impl stuff, and I think core is where I best fit.

cc @rust-lang/libs


Numbered List Bug

Hi @daphliu
Thanks for sharing your implementation. I have found it very helpful on a project I am working on.
I discovered a bug though. When you create an unordered list and immediately create a numbered list underneath it, it breaks the numbered list rendering.
This is an example of the HTML:

unordered list 1

unordered list 2 with bold

unordered list 3 with italic

ordered list 1

ordered list 2 with bold

ordered list 3 with italic

This is the screenshot of the broken numbered list:

Any thoughts on how to resolve this?

Thanks for sharing @daphliu , I have the same issue with two unordered lists at the same level. The indentation of dot marks of the second list looks like the example above.
Do you think it's posible to avoid this issue? Thanks in advance.


zh-cn:Polish certain words that is confused to Chinese player

I checked some translation and here is why I decided to change it
Line 43,44,45:“启停，键入” are not what native Chinese player will say.
Line  77,78：“消灭” has lots of meaning,but the main meaning is Eliminate.And it's same to 克隆.It means clone.
Line 137:In Chinese social games,we usually say 好友 to describe friends who is playing in this game.
Line 165,186,200.201:It's really hard to convey host and session.But by reviewing my experience on hosting the game,I found it they get used to saying 房主 as a host.Here is a picture to show how they start session to play the game.

Line 181:To telling the truth,I have no idea what“套接字” means.The original words steam networking sockets is probably means connection between steam server.So I think 网络通讯 is easier to understand.
Line 209:I think mobile in this locale is means a platform.
Line 277:If I do want to translate out of,we say x到x,but in here,This locale wants to convey each session's numbers of people.So I decide to use / as out of
Line 474:Appraently this is not what the proper translation,it's likely a machine translation.
Line 535:“销毁” words doesn't sync to the Line 77,78
Line 1944:As a resonite user,I know this is a script tool.But for the new player,they don't understand what ProtoFlux means.So I think adding a script is easier to undetstand.
Here is all of why I change it.Please tell me if there's any confused question above.

If is possible,can I enter the translator list?

Why does this process takes too much time on checking?Please don't hesitate to tell me what's wrong.

Is someone can check this pull request?Please let me know if there's any problems with it so that I can adjust it.

Thank you very much for the contribution! Merging this in!


Android build fails.

Hello,
Android build fails as follows.

I've installed the necessary plugins from Android SDK Manager:

My SDK tools are:

My plugin list is
cordova-plugin-admobpro 2.13.0 "AdMob Plugin Pro"
cordova-plugin-android-support-v4 21.0.1 "Android Support v4"
cordova-plugin-extension 1.3.0 "Cordova Plugin Extension"
cordova-plugin-inappbrowser 1.3.1-dev "InAppBrowser"
cordova-plugin-whitelist 1.2.1 "Whitelist"

install also the one: local maven repository for support libraries


Commit inside context manager

Motivation and Context
Commits inside a context manager is not handled in the master
If it fixes an open issue, please link to the issue here:
#81
Description
When commit is being triggered

It calls context manager teardown function which flushes the data to disk
then executes the commit operation
then call context manager setup function

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Documentation update
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Is this PR ready for review, or a work in progress?

[ ] Ready for review
[x] Work in progress

How Has This Been Tested?
Put an x in the boxes that apply:

[x] Current tests cover modifications made
[x] New tests have been added to the test suite
[ ] Modifications were made to existing tests to support these changes
[ ] Tests may be needed, but they are not included when the PR was proposed
[ ] I don't know. Help!

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have signed (or will sign when prompted) the tensorwork CLA.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hey @hhsecond this needs to be rebased ontop of master before we evaluate the PR

Ok @hhsecond some interesting stuff ended up happening here. Long story short your additions worked very well. you were right in thinking that they wansn't a code-path which would lead to us needing to check the TxnRegister.
However...
During my own tests I notices that the following behavior would occur when using context mangers.
>>> co = repo.checkout(write=True)
    # same behavior occurred for ``write=False``

>>> with co.metadata as m:
...     m['hello'] = 'world'
...     cmt1 = co.commit('here is the second commit')
...     with co.datasets['dummy'] as d:
...         d['1'] = np.arange(10)
...         cmt2 = co.commit('again commit')

>>> co.close()
OK

>>> repr(co)
Unable to operate on past checkout objects which have been closed.  No operation occurred. Please use a new checkout.
---------------------------------------------------------------------------
PermissionError                           Traceback (most recent call last)
~/projects/tensorwerk/hangar/hangar-py/src/hangar/checkout.py in _repr_pretty_(self, p, cycle)
    268         '''pretty repr for printing in jupyter notebooks
    269         '''
--> 270         self.__acquire_writer_lock()
    271         res = f'\n Hangar {self.__class__.__name__}\
    272                 \n     Writer       : True\

~/projects/tensorwerk/hangar/hangar-py/src/hangar/checkout.py in __acquire_writer_lock(self)
    440                   f'closed. No operation occurred. Please use a new checkout.'
    441             logger.error(err, exc_info=0)
--> 442             raise PermissionError(err) from None
    443
    444         try:
PermissionError: Unable to operate on past checkout objects which have been closed. No operation occurred. Please use a new checkout.
-----------------------------------------------------------------------------
# Note: That error is what we want to happen

# This, however, is not!
>>> repr(d)
Hangar DatasetDataWriter
    Dataset Name             : dummy
    Schema Hash              : 051a8e928804
    Variable Shape           : False
    (max) Shape              : (50,)
    Datatype                 : <class 'numpy.int64'>
    Named Samples            : True
    Access Mode              : a
    Number of Samples        : 27
    Partial Remote Data Refs : False

# we even had full access to the data files on disk as they were being
# kept alive by the references.
>>> d['1']
np.array([0, 1, 2, 3, 4, 5, 6, 7, 8, 9])

>>> repr(m)
Hangar Metadata
    Writeable: False
    Number of Keys: 1

>>> m['hello']
'world'

Long story short, there was an issue because context managers need to return a true instance of self in their __enter__() methods. By utilizing the fact that we are returning a handle to the weakref.Proxy through an ObjectProxy, if we do a much more thorough cleanup of the objects down the stack, everything will actually work out.
That's what i've included in the rest of this PR. Take a look and I'm good to merge when you are!


Daily Task accounts

I'd like to be able to set reminders for specific accounts. Not all my accounts can collect herb boxes daily but they all do get the messages, same for staves for example.
Is it possible to set the reminders per OSRS account instead how it is set now?

Currently no. The way the config system works does not support multiple accounts at the moment.
Sorry

Unfortunate @Hydrox6 ! It would be cool to have something like an account picker when you open RL, and an easy switcher when switching accounts. (Similar to Netflix and quick switching like the google website ecosystem)

That would likely be the case once multiple accounts are handled. That, or a config would be loaded depending on the username of the account you log in with, which could make a switching system become part of the Username sync plugin.

That would make even more sense.
However it does seem to do this already for farming patches. I noticed it earlier. If you open up a new client and type in only your username with the RL farming tab open it will load the current statuses without even logging in completely. @Hydrox6

The way the farming tracker works is by storing patch data inside your settings.properties file. It stores this along with your username, hence why it knows your patch data when you type in your username. This isn't really feasible for all the settings, as there are ~560 settings (or ~620 if you include farming), and duplicating the data with entries for different usernames would cause the file to balloon in size pretty quickly, causing potentially slower load times.

So it currently stores everything locally? Would an alternative be that everything is stored server side along with your RL login?
Only then the problem is moving it to the server instead of locally..

Yeah, farming plugin is entirely local. Have a poke around in your settings file, searching for farmingTracker. Lots of encoded gibberish.
The server does store your config, if you log in with the Account plugin that is. The dream would be multiple files, maybe names after each user, that contain the settings for that specific RuneScape account. These files could be easily stored on the server like the current config is now, and since they're separate, loading each would take as long as the current settings take to load.

Yeah that sounds like a solution. Would have to find a solution that not everyone can just type in a username and dig up their information. Are you aware of any plans of this in the near future? I'd like to see farming times stores on the server side as well. (struggling when switching between laptop and pc).

Well, there wouldn't be information leaks if you had to log in to the correct Google account to actually sync the settings, like you have to do now.
Farming timers should be stored on the cloud with your settings, you just have to log in to RuneLite with your Google account so that the settings sync (if you haven't logged in, it's a little green door at the top right)

I might have to check, I should be logged in on both. Thanks for the help.
Does this ticket stay open?

Until one of the admins closes it, or you do.


Changelog update (TestCafe 2.0)

Closed in favor https://github.com/DevExpress/testcafe/pull/7262.


Fix handling regex symbols in tests path on Windows

Summary
Fixes #2381
Test plan

Codecov Report

Merging #5941 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #5941   +/-   ##
=======================================
  Coverage   64.39%   64.39%
=======================================
  Files         217      217
  Lines        8273     8273
  Branches        4        4
=======================================
  Hits         5327     5327
  Misses       2945     2945
  Partials        1        1

Impacted Files
Coverage Δ

packages/jest-regex-util/src/index.js
60% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@hron thanks for making Jest better on Windows! :)


Fdmnes reader

@newville I have added a simple reader for FDMNES ASCII files. Should not break anything else. By the way, I have moved the choice of the reader in the Wxlib using the common guess_filereader.

@maurov looks great!


Updated bump-version to update README.md as well

I noticed in #161 that the bump-version script does not update the README.md file.  This PR addresses that.

Good catch, thank you!


Improve E2E tests for the gang-scheduling

What this PR does / why we need it:
In E2E, after we created Jobs with unschedulable configurations, we verify the number of conditions is 1 when using scheduler plugins for gang scheduling.
But, the test seems flaky, as reported in #1779.
So, I modified the test to verify whether unschedulable Jobs have a Created condition and don't have a Running condition instead of the current way.
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #1779
Checklist:

[ ] Docs included if any changes are user facing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 39.482%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob_controller.go
2
77.16%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2742

Relevant Lines:
6945

💛  - Coveralls

/assign @johnugeorge @nagar-ajay

Thanks for fixing the issue. The changes looks good to me.

Thanks @tenzen-y for explaining
/lgtm
/approve

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 39.41%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob_controller.go
2
77.16%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
2737

Relevant Lines:
6945

💛  - Coveralls


Blog: Empty Tags on Display

We feature tags in the sidebar that have no posts attached for some reason. Bit strange

When clicking partner under this post it does load, when clicking it in the sidebar it's empty. Seems somethings up.

fixed in d2c6853159a16e5384b529f5a205c745551ef99e


feat: support ranges

Codecov Report
Patch coverage: 93.10% and project coverage change: -42.96 :warning:

Comparison is base (cead656) 98.74% compared to head (1090e59) 55.78%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #23       +/-   ##
===========================================
- Coverage   98.74%   55.78%   -42.96%
===========================================
  Files          40       15       -25
  Lines        4128      536     -3592
===========================================
- Hits         4076      299     -3777
- Misses         52      237      +185

Impacted Files
Coverage Δ

include/emio/detail/format/formatter.hpp
54.90% <ø> (-41.84%)
:arrow_down:

include/emio/detail/format/parser.hpp
60.00% <ø> (-34.88%)
:arrow_down:

include/emio/formatter.hpp
100.00% <ø> (ø)

test/unit_test/test_ranges.cpp
92.85% <92.85%> (ø)

include/emio/detail/format/args.hpp
100.00% <100.00%> (ø)

... and 36 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Enhancement] Upgrade pickers to Material M3 standards

Before submitting...

[X] I have searched for duplicate or closed issues.
[X] I have read the CONTRIBUTING document and my issue is following the guidelines.
[X] I have read the template completely before filling it in.

Context
Pickers could use an improvement as they currently do not comply with the Material M3 standards
This would require SCSS styling refinement on the components
Currently the pickers can only be rendered as modals, would be nice if additionally we can implement docked rendering options as requested per #140
Current Behavior
No response
Expected behavior
No response
Possible Solutions or Causes
No response
Your Environment

Version used: 2.1.1

Ok please wait with this change, i am refactoring it atm.
For the modal part
I think we should remove the integrated Modal feom the pickers completely and let the user decide where the picker sits. Wdyt?

Agree with letting the user choose how the pickers should be rendered, having options "Modal - hovering in a popup" and "Docked - inline as a kind of dropdown below the field" available where only this latest options should present a new feature implementation. My idea with be with a "Docked" integration that we'd have the functionality the datepicker is visible when even the user is updating the main input field and represents the changes in real-time visually on the expanded picker


Vi mode: minor issue/request

When using / or ? to search history pressing tab instead of enter draws an unprintable character on the screen.
At the very least the tab key should just be ignored but I would like to request that tab initiates the search just like pressing enter.

I'll look into it.

Just pushed this change, tab now acts like enter.

Awesome! I just tested it out and it works just like my fingers want it to... :wink:


Fix Mavros Mission tests for VTOL

Describe problem solved by this pull request
Recently Mavros Mission Tests have been failing for VTOLs in Firmware.
Describe your solution
Update the container from ROS melodic to ROS noetic to run the tests as the same environment as the rest of the tests

Failure:
[ERROR] [1597924035.046962627]: [registerPublisher] Failed to contact master at [7f9311d946d5:44367].  Retrying...
Traceback (most recent call last):
  File "/opt/ros/noetic/lib/gazebo_ros/spawn_model", line 20, in <module>
    import rospy
  File "/opt/ros/noetic/lib/python3/dist-packages/rospy/__init__.py", line 49, in <module>
    from .client import spin, myargv, init_node, \
  File "/opt/ros/noetic/lib/python3/dist-packages/rospy/client.py", line 52, in <module>
    import roslib
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 906, in _find_spec
KeyboardInterrupt

@TSC21 Updating to noetic might not be a good idea, since we don't have catkin on focal working yet

any progress on this end @Jaeyoung-Lim ?


[Feature Request] Separate event listeners from components

What is the need and use case of this feature?
In version 2.0 it is recommended to use React Native Hooks which are tied to a UI Component to listen to events -e.g.: usePlaybackState, useTrackPlayerEvents, useProgress -. See docs here: https://react-native-track-player.js.org/docs/v2-migration
This is not ideal, as it forces the event listener code to be tied to a component. This component, that should only be responsible of drawing and react to screen events, is taking care of something that might not be related to that component.  For example, remote events which have nothing to do with a react component -they are generated outside the app- are dispatched to a hook in the app component. This breaks the single responsibility principle and makes the component more difficult to test and maintain.
Even more, the current documentation advises to use addEventListener here: https://react-native-track-player.js.org/docs/basics/playback-service , and then discourages to do it here: https://react-native-track-player.js.org/docs/v2-migration
Describe the ideal solution
The previous solution with eventListeners was a good one. The library can even provide the hooks to listen for events in a component as it does now, or let the user listen them in an eventListener, but not forcing the user to listen for events in a component.
Hence, in my opinion, using eventListeners solution is the best approach as it will:

Let the user decouple the events listeners from UI Components.
Let the user use hooks, if needed. But not being forced to.

How I can Help
I can work on it and create a PR, but I think everything you need is already there. As hooks are just a wrapper over the real addEventListener. So I assume that it is just a matter of updating the documentation explaining that hooks are just that, a wrapper over addEventListener and that the user may use addEventListener if needed.

I haven't looked at the docs in this capacity but I can speak to how thing should be implemented.
Inside any react UI components, hooks should be used. Apart from that the playback service should employ listeners (this is where all headless events and logic should live).
If this doesn't track in the docs I'd happily review a PR rectifying it.

@pabloromeu I'm going to close this as it seems that this is more of a misunderstanding. I do think it could be clarified in the docs  a bit and if you want to submit a PR I'll happily review it.


Invalid API count is displayed on the Store in apim cluster

Description:
Invalid API count is displayed on the Store in apim cluster pattern-6. There are only 2 APIs but it displays as 17

Affected Product Version:
APIM2.1.0 WUM
OS, DB, other environment details and versions:
Ubuntu, JDK1.8, MySQL
Steps to reproduce:
Repeated below steps.
Create and publish an API >> Subscribe to the API >> Invoke >> Delete
Followed same steps using same API name for each time.

This issue can be reproduced in a non-clustered environment as well. Here are more details.
Affected Product Versions:

APIM-2.x series

Steps to reproduce

Start a APIM setup with default configuration.
Create more than 12 (since default pagination size is 12) different APIs (different names).
Visit https://localhost:9443/store/?page=1 Store page and identify an API that is visible in the 1st page.
Goto Publisher and create a new version of that API. The publish it.
Refresh the https://localhost:9443/store/?page=1 Store page and you will notice only 11 APIs are visible.

Root cause of this problem can be summarize as follows,

The Jaggery layer requests 0-12 page (12 APIs) from the backend.
The backend APIM layer query for 0-12 page from the registry and registry returns 12 artifacts.
Then APIM layer (APIConsumerImpl class) filter out multiple versions of the same API if <DisplayMultipleVersions> configuration is set to false (which is the default value). This reduce the APIs count to <12.
Then the Jaggery layer renders the UI assuming there are 12 APIs, which is inaccurate.

Possible workarounds:

Setting <DisplayMultipleVersions> to true in the api-manager.xml file so that multiples versions are shown in the UI.

Cannot reproduced in master


fix: error jump link for setup_rbac.md

Signed-off-by: kerthcet<EMAIL_ADDRESS>
What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm
/approve
/ok-to-test


Targeted fix to enable the arm64 static library build

The current build doesn't support building Windows arm64 debug and retail libraries. This change fixes the solutions to enable the Windows arm64 builds that depend upon sqlite. This is a targeted fix and doesn't try to resolve any issues with other non-sqlite configurations.
In addition to the targeted fix, this change removes the SampleCppUTC.vcxproj from the solution since it doesn't appear to be included in the repository anymore and can give build errors.

In addition to the targeted fix, this change removes the SampleCppUTC.vcxproj from the solution
since it doesn't appear to be included in the repository anymore and can give build errors.

It's included if you build with additional private modules. You can specify the list of dependencies you want to build explicitly, as it's done in this script, for example:
https://github.com/microsoft/cpp_client_telemetry/blob/9aa9a8dd9cdda146a43b13874de6de3927d1cfd2/build-win10-min.bat#L32
where /target: option specifies exactly what project you need to build, while /p:BuildProjectReferences=true option specifies that all dependencies (references) of the project are also built, thus you build only the projects that are needed by MIP SDK.


feat: add regtest support to BTC address config files

Try out this version of the Hiro Wallet - download extension builds.

Clone of @kenrogers' PR https://github.com/hirosystems/wallet/pull/3513
Some changes made here:

Removed use of 2 for regtest coin type in derivation path. Afaict, all testnets use 1
Removed regtest keychain. Because of above, we can reuse testnet keychain
Created new, more specific bitcoin mode containing regtest/signet
Removed chain id changes, this is a stacks-only concept which has no regtest

Can we get a screen recording posted here to review the UX? 🙏

@markmhx there are no UI changes, we default to regtest for devnet only in this PR


correct encode tag value method

[x] correct encode tag value method

Codecov Report
Attention: 106 lines in your changes are missing coverage. Please review.

Comparison is base (a7e483f) 47.31% compared to head (8fc95c1) 47.12%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

banyand/measure/query.go
3.44%
56 Missing :warning:

pkg/query/logical/measure/topn_analyzer.go
0.00%
18 Missing :warning:

banyand/measure/topn.go
0.00%
16 Missing :warning:

banyand/measure/write.go
0.00%
13 Missing :warning:

pkg/query/logical/measure/topn_plan_localscan.go
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #385      +/-   ##
==========================================
- Coverage   47.31%   47.12%   -0.19%
==========================================
  Files         172      172
  Lines       21250    21350     +100
==========================================
+ Hits        10054    10062       +8
- Misses      10356    10447      +91
- Partials      840      841       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[NFR] Added support for Zephir Language

Hi ;)
First thanks contributors for this library :smiley:
https://github.com/phalcon/zephir
This will be similar for PHP
but without <?php open tags, $ for variables
Can anyone help with it?

@Sannis Any news? It's rly important. For upcomming conference
Thanks

@Sannis ping
@isagalaev Can you help?

Can't speak for @Sannis what he meant by assigning this to himself but generally we don't take requests on implementing languages, instead we accept implementations from contributors (http://highlightjs.readthedocs.org/en/latest/language-requests.html). In any case, we're all volunteers here and don't adhere to any time schedule.

PHP detection depends on PHP-specific things like $-started variables. I've stat working on it but currently I have many test broken :-(

Парни вопрос жизни или смерти
Даже 1С есть, прошу молю :smiley_cat:

Fixed in #793.


upgrade the hover speed of play button

upgrade the aniamtion pulse and the transition effect for increase the hover speed at play button
#944
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #(issue no.)
Description
[upgrade the animtion pulse and the transition effect for increase the hover speed at play button]

Type of PR

[ ] Bug fix
[* ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
before

after upgrade
https://github.com/anuragverma108/SwapReads/assets/145912232/b535b81a-96b8-4481-a2f9-cebb95113945
Checklist:

I have made this change from my own.
I have taken help from some online resources.
My code follows the style guidelines of this project.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation.
My changes generate no new warnings.
I have tested the changes thoroughly before submitting this pull request.
I have provided relevant issue numbers and screenshots after making the changes.

@shuhaki please mention the issue number


niUI/Line.cpp: EditBox: Fixed a problem that Chinese character didn't cut correctly ;

Formal Notes

This PR fixes:

RN: niUI/Line.cpp: EditBox: Fixed a problem that Chinese character didn't cut correctly ;

How Has This Been Tested?

Command line to run all the tests that I run locally:
ham Run_Test_niScript_tu_linter

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that cause existing functionality to change)
[ ] Documentation (code docs, comments, or changes to the doc systems)
[ ] Testing/Build (test coverage or the test/build subsystems themselves)
[ ] Packaging (adds examples or modifies a release package)

I think this is the test you want to start from:
ham Run_Test_niUI_EditBox

That's in niLang/sources/niUI/tsrc_ni/EditBox.gml

if want to reproduce the issue, put some random Chinese text in the EditBox

then type Enter in the middle

you will end up like this, the first line missing half of the characters


[CLI] mem_limit breaking cli

Rancher versions:
rancher/server or rancher/rancher: v1.6.14
rancher/agent or rancher/cli: v0.6.9
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Suse Linux 12 - 4.4.114-94.11 (4309ff9) x86_64 x86_64 x86_64 GNU/Linux
Steps to Reproduce:
create stack with the following contents:
version: '2'
services:
  web:
    mem_limit:<PHONE_NUMBER>00
    image: nginx

Results:

cli errors out, following error logged:

ERRO[0002] Failed Creating web : json: cannot unmarshal number 2.68435456E11 into Go value of type int64
ERRO[0002] Failed to start: web : json: cannot unmarshal number 2.68435456E11 into Go value of type int64
FATA[0002] json: cannot unmarshal number 2.68435456E11 into Go value of type int64

@aemneina Can we collect the cli output by adding two things?

add export RANCHER_CLIENT_DEBUG
add --debug flag. Example: rancher --debug up

Confirmed with the end user that was having this issue that v0.6.11-rc2 has fixed the issue.


how can I change highlight-color?

I want to change highlight-color for a custom one, don't use colors by default, I'd like to know, if this is possible.
Thank you

The highlight property just takes whatever string you pass it and appends highlight- to it.
So you could do.
<ion-md-input highlight="myColor"></ion-md-input

Then create the following css
 .item-md-label .highlight-myColor {
    background: #ffffff
}

Or what ever color you wanted.


prompt input box missing?

works on localhost but there is no prompt input box with lama model. do you need sd checkpoint?

prompt input box only appears when starting the server using sd1.5 model. LaMa model does not support text-driven image editing


Tkinter filedialog.askdirectory() freezing when importing pywinauto

Hi,
I've logged this in the Python issue tracker already, but I've been told to follow up with you guys and see if it's something that can be fixed on your end.
https://bugs.python.org/issue34029
Essentially, when pywinauto is imported, the tkinter.filedialog.askdirectory() function freezes the application and doesn't open the askdirectory dialog.
From the bug:
Debugging a bit, the hang happens in Lib/tkinter/commondialog.py, line 43:
s = w.tk.call(self.command, *w._options(self.options))
The value of self.command is 'tk_chooseDirectory', and w._options(self.options) just gives an empty tuple, so the actual call is:
s = w.tk.call('tk_chooseDirectory', ())
This means that the hang is within w.tk.call().

Hi @rsteel1 thanks for reporting this! I need some time to take a deeper look for this. Can't promise to do this in July, maybe in August.

Hi @rsteel1, finally we got the chance to look into this issue. I was able to reproduce the reported problem on my Win8.1 64-bit machine with 64-bit Python 3.6 using the code below
import pywinauto
import tkinter.filedialog
tkinter.filedialog.askdirectory()

Since this involves windows file dialogs I suspected that the origin of the problem is incompatible COM threading model. By default, pywinauto sets up the Multithreading COM model (MTA) on init if no other mode was defined prior to import of pywinauto or specified explicitly through sys.coinit_flags. My guess is askdirectory call uses "appartment threaded model" that is incompatible with MTA mode that was defined during pywinauto import. So to solve the problem we should specify the appropriate threading model explicitly as in the example below:
import sys
sys.coinit_flags = 2  # COINIT_APARTMENTTHREADED

import pywinauto
import tkinter.filedialog
tkinter.filedialog.askdirectory()

@rsteel1 , @vasily-v-ryabov , should we close this issue? We may need to add these COM initialization gotchas to our docs, though. For example, create "Troubleshoting" chapter.

I think yes. We can close this after adding notes to the docs.

What could be the reason that askopenfilenames() work and not askdirectory() and this was working all fine till I decided to use pyglet or something. But this isnt the first time I'm using all this together and it has worked.

What could be the reason that askopenfilenames() work and not askdirectory() and this was working all fine till I decided to use pyglet or something. But this isnt the first time I'm using all this together and it has worked.

This seems to be a pyglet issue. I am using Pyglet 1.5.15 and on python 3.7.

Hi @rsteel1, finally we got the chance to look into this issue. I was able to reproduce the reported problem on my Win8.1 64-bit machine with 64-bit Python 3.6 using the code below
import pywinauto
import tkinter.filedialog
tkinter.filedialog.askdirectory()

Since this involves windows file dialogs I suspected that the origin of the problem is incompatible COM threading model. By default, pywinauto sets up the Multithreading COM model (MTA) on init if no other mode was defined prior to import of pywinauto or specified explicitly through sys.coinit_flags. My guess is askdirectory call uses "appartment threaded model" that is incompatible with MTA mode that was defined during pywinauto import. So to solve the problem we should specify the appropriate threading model explicitly as in the example below:
import sys
sys.coinit_flags = 2  # COINIT_APARTMENTTHREADED

import pywinauto
import tkinter.filedialog
tkinter.filedialog.askdirectory()

After using pyinstaller to convert my project to exe, I had the issue of freezing window when clicking open directory. Turns out I imported pywinauto in another file, and this solution does solve my problem. I added "import sys" and "sys.coinit_flags = 2". The new exe created by pyinstaller behaves normally now when I click the open directory. 👍

I moved on from this long ago but thank you for looking into it, I enjoyed receiving the email! Good spot, and I hope anyone else who comes across this error finds this thread. Thank you :)
-------- Original Message --------
On 24 Nov 2022, 17:35, xuejianma wrote:

Hi @.***(https://github.com/rsteel1), finally we got the chance to look into this issue. I was able to reproduce the reported problem on my Win8.1 64-bit machine with 64-bit Python 3.6 using the code below
import
pywinauto
import
tkinter
.
filedialog
tkinter
.
filedialog
.
askdirectory
()
Since this involves windows file dialogs I suspected that the origin of the problem is incompatible COM threading model. By default, pywinauto sets up the Multithreading COM model (MTA) on init if no other mode was defined prior to import of pywinauto or specified explicitly through sys.coinit_flags. My guess is askdirectory call uses "appartment threaded model" that is incompatible with MTA mode that was defined during pywinauto import. So to solve the problem we should specify the appropriate threading model explicitly as in the example below:
import
sys
sys
.
coinit_flags
=
2
COINIT_APARTMENTTHREADED
import
pywinauto
import
tkinter
.
filedialog
tkinter
.
filedialog
.
askdirectory
()

After using pyinstaller to convert my project to exe, I had the issue of freezing window when clicking open directory. Turns out I imported pywinauto in another file, and this solution does solve my problem. I added "import sys" and "sys.coinit_flags = 2". The new exe created by pyinstaller behaves normally now when I click the open directory. 👍
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

sys.coinit_flags = 2
Including this solved this problem for me in customtkinter


Chrome extension

I've organized things into 2 separate directories webclient/ and chrome_extension/. There is also a common/ folder that is symlinked into both aforementioned directories. Git seems to handle this flawlessly. Everything appears to work.
I've also updated gulpfile to insert the git-sha and buildtime in addition to package version.

https://bugs.chromium.org/p/chromium/issues/detail?id=27185
Symlinked scripts don't get loaded as contest scripts.


Pre-generate HTTPS certs

I was thinking about this since it affects SOSP evaluations. Right now app/driver install time is not insignificant, and the majority of a time a container takes to start up is used for generating HTTPS certificates. It would be cool to pre-generate a pool of certs that the CM can just grab and pass on to containers as they start, and top the pool up during idle time. Would make app/driver startup significantly faster.

This may change (or at least be affected by) a move to docker swarm as it handles the cert generation differently.

Fixed in 0c92150be1449c0987b876830cded977b4493fbb

Certs and secrets for the platform components are now pre-generated reducing startup time.


[Addition]: Hummingbird selfwitnessing cluster

Hotspot b58 Addresses
112An9gj5A9eKZsszQgAkaDCSjxVfSog65x2zdHeXYNoFENvgkaE
112eAPNW2NPCAp8yMAVEV8Ge5Xhh2f77mtafr1tLBTtGrckoyWNM
112gk3tgtxsKV2rFxQm76Xen6y7NduwvNSaz6kNP66H8q9793jZh
112hAVXf8627BN4CQRaZqnRupoKuFiYe1858v6pLBq55PGrNMHXF
112HRX6b9obTigc8GkATaEDYuLkqPa2EEQfCoacjoa3iPuwWNCZN
112LtppbE1L4aiNUW7CmqGLRH6khyH3DFiY7gRourkvtDrijrx6Q
112NNRjqtuuThng6nDgYYQMKK4Bh1Bcb33K2MUsX8xZRkn1LebZX
112t5ryiyVDaLuxchnevGm153sTewuJ5gpEV17dzYJ4rUuuxryWx
112X79Fzd9UYyoiKVrViKP9CqAvW5gnTKCsnd3kk6Cka7ewFPzeA
117YEaWkzFofJGT2AQqfB3AQWVQG5uzLAipoz1wX1wQmPsftChi
11eC4xU31LKHmuJYQNqWumdg7g3smu7qPVvtZ2beJumUNohPt62
11iRA36iGsymXEtzK1kXRP9X9bjrWNobqRSJFfkEpNTPx9WmuPH
11ivpMxABtNzTk6Wkqci7tXGVYzmyGsNng7dFiA442eN827Lmpc
11L4K7h5L2xrubacrtdbMu4gf51wg7vqN3QM8hhFuu17xpceB6h
11nmm8jE6xpc8xJKzBuGTbEom1w3nW6bxQTY5YwkxMhAteRrNgL
11rnhd7362xKxaxAShog1vrh5sABzSgR5TCSEwFqSSVoaojtu5K
11SRpHmQNpbC2egBxrCb7przHuJFDEcpseNQq5Jb2VYfdLLnTNq
11U7coQq8WWz8SMUZExqjXM46zhCzJsAzxRf84CBK1v987MxcPS
11WJamkdfVgUfPqBifM4EQr5rtefcriR33Z5zMtu8HSkxpiAH5C
11xvtWMSBzo2a9eTyE5dMUEd1s1KtdXRqRJuwSFsfhhCgjTATYC
Discord Handle
No response
Reason(s)

All but 1 account for all Hummingbird gateway's didn't show any data transfers.
So data transfer is possible! only 109 out of 3853 showed packet transfer data, all these 109 are in same account.
the gateway's in that account are part of a selfwitnessing cluster so they probably all heared the same
signal.

Most of all Hummingbird gateway's witness only Hummingbird gateway's.

We iterated the witnesses of the witnesses to find the clusters.
We used the node-red server we setup for this to get the results out of api.

All HNT stolen with the total amount of 21067 HNT as by todays date/time.

We conclude that out of the 3852 gateway's we inspected only 109 gateway's in 1 account showed accidental
data transfer. so data transfer for gateway's Hummingbird is possible.

We conclude that out of the 3852 gateway's we inspected only a couple of accounts showed communication with other makers. so communication with other makers for Hummingbird gateway's is possible.

Rssi/ distance / SNR calculations are to perfect, we have found some invalids -59 dbm but we suspect they are from cables touching eachother.

We haven't targetted any maker, it was the result of a count WHERE actors1.actor_role = 'witness'
AND actors2.actor_role = 'challengee' are in same account.

Resolved in https://github.com/helium/denylist/pull/4048


add wechat support

add wechat support

share to wechat

+1 to this

:+1:

+1

+1

why this pull request is rejected to merge ? I have created a updated version here: https://github.com/pluswave/SocialSharing-PhoneGap-Plugin  for latest master and fix some issues and add title support, should I create a new pull request ?

@pluswave I don't think this PR is rejected, the maintainers just haven't gotten around to reviewing it.
:+1: too btw

My main issue with this is that this plugin doesn't embed any native SDK's. Wechat would be the first and the plugin jumps from 0,9KB to 7,3MB. If I'd add all native SDK's you can imagine what would happen.
Perhaps a sep wechat plugin would make more sense. Curious to learn what you guys think, although you're a bit biased I guess ;)
But I'll give it some thought since there's high demand. Can you submit a PR as this one can no longer be merged (or an update to this one is fine as well of course).

We really want to this plugin can support sharing to wechat, even with plugin or whatever method. At least an image and url can be shared. Thanks a lot for the hard working for this plugin.

Great! Is it merged to the main branch already?


[BUG] Check github actions are updated

Describe the bug
Looks like GitHub actions are not being updated automatically..

They are being updated, but was already using the latest so no issue here.


fix typo in integer type (causes compile error)

Thank you for reporting. I am a bit shocked that I didn't catch that. I tested extensively with the Arduino IDE and also the travis build running the demos didn't complain. May I ask what build environment you are using? E.g. which version of Arduino IDE?

I use platform.io
I'm not done, need to update the function arguments as well, I'm on it.


Notification Settings panel not working

Please make sure that you have checked the boxes:

[x] Review the Quickstart guide
[x] Search for both open and closed issues regarding the problem you are experiencing
[x] For permissions issues (Access Denied and credential related errors), please refer to the requisite docs before submitting an issue:
AWS, GCP, OpenStack, GitHub

Description of issue:
Notification Settings panel is not allowing me to enable notifications on my account.
I select the Notify box on my account, set the Change Emails dropdown to With Issues, and click Save.

After clicking Save, I get the following error:

I was able to replicate.  Need to dig in more to see why.

The UI is not sending a list but rather a string of the first account ID in the list. The Security Monkey API is expecting a list of IDs.

I was spending some time looking into this, and there is some oddities going on in Dart.
Since we are actively trying to get rid of Dart (it's awful), we're going to put this on the backburner or fix it via the replacement.

Is there a workaround in the meantime to enable notifications? Sorry if this is a dumb question. I'm new to Security Monkey.

Hello, I there any update regarding this? Maybe a workaround?

Unfortunately, the bug lies somewhere in an underlying Dart package that is extremely hard to debug.
The best work around is for there to be a monkey command that does this. It's not ideal, but it would unblock you. You would have to implement this, however.

I don't mind implementing it if you could give me a little more direction. Not much of a developer.

I don't think that this is a UI bug. I tried to do the same with curl and POST and got the same result.
Here is my curl without cookies and tokens:
curl -X POST -d "{'accounts': [1,2,3], 'daily_audit_email': true, 'change_report_setting': 'ISSUES'}" -H "Accept: application/json" -H "cookie: session=.....; XSRF-COOKIE=....." -H "X-CSRFToken: ....." -k --insecure 'https://localhost/api/1/settings'
And here is the result:
"message": {"accounts": "Must provide accounts"}}
I found a workaround in case you have only one account. You just need to change a type from list to str in the file /views/user_settings.py, line 151
self.reqparse.add_argument('accounts', required=True, type=str, help='Must provide accounts', location='json')
But this is definitely a dirty hack. And it will not work in case you want to receive notifications for more than one account.

@arnitolog
That's a great find but, sadly I have like 9 accounts I need notifications for and I am unable to make a  Monkey command to enable the  notifications.

@arnitolog : I am going to take another look.
When I was debugging this the last time, what I saw was that Dart was sending over an incorrect JSON (as seen in the Chrome networking console).
I will double-check with the curl you posted above.

It looks like there are some issues with flask_restful (among other things).
I'm going to replace it with Marshmallow.

yeah, I'm also thoughts that it is something in the flask_restful. But didn't have enough evidence to mention that. Anyway, thanks Mike for your help.

Just Submitted #1042 with a possible fix.
That should fix the API at the very least.

thanks. I will test it a little bit later

Good news: Code wise, it now appears to be working (I don't know what issues were with Dart before... but now it's working properly) .
Once I get TravisCI to cooperate, I will merge it in.

This should now be fixed in #1042 . Please make sure that you move celeryconfig.py to security_monkey/celeryconfig.py.

Confirmed that it's now fixed.