Notify Button Will Not Display

Describe the bug
Angular Getting Started In Products Alert Component Notify Button Will not Display.Pass Data To A Child Component Is Not Working
Link to the blitz that caused the error
https://stackblitz.com/edit/angular-15hdz1?file=src%2Fapp%2Fproduct-alerts%2Fproduct-alerts.component.html
Steps to reproduce
product-alert-component.html
product-list-component.html
Expected behavior
display notify button but it's not working
Parity with Local

[ ] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 99.0.4844.51
Major version = 99
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/99.0.4844.51 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 127048240,
  "usedJSHeapSize": 113384140,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 081c5c65a05b637353c8026163d5940db2997cac

Additional context
No response

Duplicate of https://github.com/stackblitz/webcontainer-core/issues/737


Error report during model training and explanation

Hi @fransilvion,
Thanks for the useful tool.
I've installed explainn 0.1.5 by pip install explainn==0.1.5. and followed the github instruction to train model on test set (you provided).
for model training, I ran into two errors when I call train.train_explainn() function.
Error1: patience parameter isn't available.
Error2: checkpoint can not be set as 0.
I gave it a go without using these two parameters, which works.
When I did the interpretation, I  tried to run the code below.
filter_size =19
pwms = interpretation.get_pwms_explainn(activations, data_inp, filter_size)
But I ran into the below error.
TypeError                                 Traceback (most recent call last)
 in
28 activations = interpretation.get_explainn_unit_activations(data_loader, model, device)
29 filter_size = 19
---> 30 pwms = interpretation.get_pwms_explainn(activations, data_inp, filter_size)
31 meme_file = "/g/data/zk16/xzhang/ExplaiNN/explainn_filters.meme"
32 interpretation.pwm_to_meme(pwms, meme_file)
TypeError: get_pwms_explainn() missing 1 required positional argument: 'filter_size'
I did assign filter_size argument, but it still popped up argument miss error.
Any suggestion for fix this?  Thanks in advance!
Best,
Xuan

Hi Xuan,
Please accept our apologies. I introduced some modifications to the code a while ago and just pushed the code recently during the reviewing process of the manuscript.
While I try to update the code to PyPI, you can install the updated code via python setup.py install.
I have also introduced scripts to train, test and interpret ExplaiNN models which are provided in the scripts folder. Instead of running the code through a Jupyter notebook, the scripts enable execution of ExplaiNN through the command line. For more information, you can check the tutorial that I provide under data/tutorial.
Sorry for the inconveniences.

Hi @oriolfornes,
Thanks for your reply. I've re-installed the updated version and followed the tutorial (slide.pdf) to learn the process of Explainn.
When I tested example 1 (you provided in the slide), I have stuck again before the model training( page 14 of the slide).
When I ran fasta2explainn.py, I got the below errors. There are small bugs in the code. Would you like to have a look?
Traceback (most recent call last):
  File "../../scripts/parsers/fasta2explainn.py", line 179, in <module>
    cli()
  File "/g/data/zk16/xzhang/miniconda3/envs/dnabert_gpu/lib/python3.6/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/g/data/zk16/xzhang/miniconda3/envs/dnabert_gpu/lib/python3.6/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/g/data/zk16/xzhang/miniconda3/envs/dnabert_gpu/lib/python3.6/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/g/data/zk16/xzhang/miniconda3/envs/dnabert_gpu/lib/python3.6/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "../../scripts/parsers/fasta2explainn.py", line 86, in cli
    args["output_dir"], args["prefix"], args["random_seed"], args["splits"])
  File "../../scripts/parsers/fasta2explainn.py", line 147, in _to_ExplaiNN
    df2 = pd.DataFrame(data, columns=list(range(len(data[0]))))
IndexError: list index out of range

Thanks in advance!

Hi, my apologies for the long wait.
A couple of things:

I have updated the installation. It is no longer through pip (I do not have the right permissions), but through git clone and then python setuo.py install; please see the README.md file.
I have updated all the parsers, trainers, and interpreters; have not tested them all, but they should work fine.

Let me know if you encounter any additional problems!
Thanks for your patience!


[ ] Configuration Infrastructure
[ ] Docs
[ X ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

Looks like the Endpoint and ValidatingWebhookConfiguration resources inherit the labels from parent resources, while they are not in the object cache pool, so the in prune loop, they will be deleted. But they will be finally recreated by parent.

I suppose one workaround would be to remove those from the prune whitelist. I think that is fine for Endpoints which we will always manage in the code but I'm less sure about the webhook config - there's a good chance that we should prune that in some situations. Thoughts @ayj?

We should prune the webhook config if the corresponding feature is disabled (validation and injection).
It would also be useful to prune as part of uninstall but I don't think that is in scope for this issue.
Something else to consider is the validation webhook endpoint temporarily becoming unavailable during an upgrade (e.g. because of pruning). It would be great if istioctl could help here. For example, it could ensure that istiod is ready before applying Istio CR. Alternatively istioctl could ensure that the webhook is reset to FAIL-OPEN before applying CRs. The latter method is partly implemented by https://github.com/istio/istio/pull/21164.

@ostromart You mean remove endpoint GVK from prune list: https://github.com/istio/istio/blob/master/operator/pkg/controller/istiocontrolplane/pruningdetails.go#L75
I'm afraid that would not be working, since we have some other endpoint resources create by operator directly, see: https://github.com/istio/istio/blob/master/manifests/base/templates/endpoints.yaml
If we remove its GVK, these created endpoints will also be leak in the cluster.

@ayj Do you think if it is safe to remove ValidatingWebhookConfiguration resources from prune list, https://github.com/istio/istio/blob/master/operator/pkg/controller/istiocontrolplane/pruningdetails.go#L113-L114
I think so, because the ValidatingWebhookConfiguration and MutatingWebhookConfiguration are not created by chart, they are dynamically created by pod.

Mutating and validating webhook configuration are part of the charts now. The former always was. The latter was added in 1.5.

As long as the Mutating and validating webhook configuration are created by the chart, the it will be added to object cache, and won't be removed during prune loop.
It makes sense. thanks @ayj

I removed Endpoints from prune whitelist in #21499.
There was a separate problem with the webhooks, they were incorrectly being labeled with a namespace in the charts which was causing them to get pruned. That was fixed in https://github.com/istio/istio/pull/21651.


Question around chattiness/performance/caching

Do the bullet-train clients do any caching, or have some mechanism to avoid huge amounts of chattiness? If there is caching, is there some expiry mechanism to ensure that the clients are up to date?
I ask because adding a REST call into each hop is a potentially expensive operation if you have a distributed system. I know some feature toggle systems (and things like consul) have mechanisms for mitigating this e.g. launch-darkly opens websockets and each client keeps a cached copy of the current togglestate taht is updated instantly

We have no caching on any part of our API or SDK clients. If you want to cache flags locally you are free to do that. It depends on the application architecture and your specific use cases. If you can provide a bit more detail we could maybe give you some pointers?

Closing this issue due to inactivity.


Add shadow button component

Removed the earlier changes and Added the shadow component as suggested by @cpdevelops .

Great job. Can you add a demo shadow button to the demo site in index.html and then it will be ready to merge.

Sorry forgot to add the index file earlier.

Great! I will merge now and I hope you keep contributing. All good with the demo.


[MB-3968] Get lib/pq working again under Pop 5.x

Description
This PR contains changes needed to get Pop 5.x and our the lib/pq driver working and passing all tests.
Reviewer Notes

Note that this PR is requesting a merge into rr-MB-2467-pop5 (the long-running branch for the Pop upgrade), not into master. Eventually we will merge the long-running branch into master after all tests are passing and we deploy/test on experimental.
I also upgraded Pop to the recently released 5.3.1 which has a preloading bug fix we'll need.
Previous PRs had changed some error codes to look for the pgx versions, but I'm changing them back to the pq versions here since we're going to stay with lib/pq as an intermediate step to getting on Pop 5.x.
Because of some changes under the hood for Pop, the Pop setup for the tests had to be modified to explicitly set the SSL mode.
Because Pop defaults to the jackc/pgx driver now, we have to explicitly pass the lib/pq driver (i.e., postgres) to the Pop setup for the tests.  The rest of the code uses a custompostgres driver that already wraps lib\pq.

Setup

make server_test -- ensure all tests are passing 100%
make server_run and make client_run, then try making a test move with the app and looking at it from the office side.

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Fails

:no_entry_sign:
Yarn Audit Issues Found:
high - Prototype Pollution
Package object-path
Patched in >=0.11.5
Dependency of react-scripts
Path react-scripts > resolve-url-loader > adjust-sourcemap-loader > object-path
More info https://npmjs.com/advisories/1573
1 high vulnerabilities and 0 critical vulnerabilities found

Messages

:book:
:link: MB-3968

  Generated by :no_entry_sign: dangerJS against dcc32e804b9a37869b3c20abf79bf303fc366b51

Changes look good, tests passed locally. I was able to create a move but I can't seem to see it in the office. Maybe we can walk through that together to get the approval on this?

I ran into this too but it appears to be caused by a recent change to restrict what you see in the TOO/TIO UI to what's associated with your transportation office.  I can walk you through what I discovered.

LGTM... I'm stuck at the same point John G was (seeing the move in the office app), but since it's working for him, 👍


xgb yaml container name should be consistent with xgb job default container name

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #
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.
5 unchanged lines in 2 files 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
76.97%

pkg/controller.v1/paddlepaddle/paddlepaddle_controller.go
3
53.35%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
2737

Relevant Lines:
6945

💛  - Coveralls

ping :) > @terrytangyuan @johnugeorge @gaocegege

Sorry. I missed it
/approve


3 to 6 05-02-2023

➡ Forward port
Port ign-gazebo3 to ign-gazbeo6
Branch comparison: https://github.com/gazebosim/gz-sim/compare/ign-gazebo6...ign-gazebo3
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

This needs a release of gz-gui.

@osrf-jenkins run tests

Any ideas regarding this windows build error: https://build.osrfoundation.org/job/ign_gazebo-pr-win/5706/consoleFull#13067844058ff58640-3599-4406-a210-216932f1748c?

CI is mostly green. The windows failure looks weird, especially since this PR is not touching python. The ABI checker is red, but Jenkins also says no failures were found. Homebrew builds, but has some test failures.
I could use someone else to check what, if anything, needs to be fixed here. Thanks.

The windows failure also happens on the stable CI, although inconsistently (https://github.com/gazebo-tooling/release-tools/issues/724#issuecomment-1564561059). The ABI checker issue could be related to https://github.com/gazebo-tooling/release-tools/issues/939, but it's also reporting an error due to the duplicated BaseView.hh
In file included from /tmp/of3phyEMhD/dump2.h:417:
/usr/local/source_branch/include/ignition/gazebo6/ignition/gazebo/detail/BaseView.hh:46:8: error: redefinition of â€˜struct ignition::gazebo::v6::detail::ComponentTypeHasherâ€™
   46 | struct ComponentTypeHasher
      |        ^~~~~~~~~~~~~~~~~~~
In file included from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/detail/View.hh:32,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/EntityComponentManager.hh:42,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/System.hh:23,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/comms/MsgManager.hh:30,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/comms/Broker.hh:25,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/gazebo.hh:23,
                 from /tmp/of3phyEMhD/dump2.h:5:
/usr/local/source_branch/include/ignition/gazebo6/gz/sim/detail/BaseView.hh:46:8: note: previous definition of â€˜struct ignition::gazebo::v6::detail::ComponentTypeHasherâ€™
   46 | struct ComponentTypeHasher
      |        ^~~~~~~~~~~~~~~~~~~

The macOS issue is probably due to an outdated ign-math bottle (https://github.com/osrf/homebrew-simulation/issues/2269).

The windows failure also happens on the stable CI, although inconsistently (gazebo-tooling/release-tools#724 (comment)). The ABI checker issue could be related to gazebo-tooling/release-tools#939, but it's also reporting an error due to the duplicated BaseView.hh
In file included from /tmp/of3phyEMhD/dump2.h:417:
/usr/local/source_branch/include/ignition/gazebo6/ignition/gazebo/detail/BaseView.hh:46:8: error: redefinition of â€˜struct ignition::gazebo::v6::detail::ComponentTypeHasherâ€™
   46 | struct ComponentTypeHasher
      |        ^~~~~~~~~~~~~~~~~~~
In file included from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/detail/View.hh:32,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/EntityComponentManager.hh:42,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/System.hh:23,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/comms/MsgManager.hh:30,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/sim/comms/Broker.hh:25,
                 from /usr/local/source_branch/include/ignition/gazebo6/gz/gazebo.hh:23,
                 from /tmp/of3phyEMhD/dump2.h:5:
/usr/local/source_branch/include/ignition/gazebo6/gz/sim/detail/BaseView.hh:46:8: note: previous definition of â€˜struct ignition::gazebo::v6::detail::ComponentTypeHasherâ€™
   46 | struct ComponentTypeHasher
      |        ^~~~~~~~~~~~~~~~~~~

The macOS issue is probably due to an outdated ign-math bottle (osrf/homebrew-simulation#2269).

Right, I forgot to commit the BaseView update. I'll do soon.

@azeey, can I get one last LGTM from you when you're good with this? Or, just merge whenever you're comfortable.

I'm trying to figure out why the abichecker is failing. It might be a false-positive, but I'm not sure.

Well, it's done. Now I can start the next forward port. Yah!


Update mvc dependency in sample to remove vulnerability warning

👍

Thanks! 👍
It's weird that one of the builds out of the two is failing (especially because those should be identical I think), maybe one of our tests is not isolated enough, so it fails randomly? I'll try to check.

I could reproduce the failure locally, and indeed it's random, sometimes it fails, sometimes it passes.
I'll look into what this test is doing, and will try to fix it.

Ok, I think what the problem is that the Dictionary we store the commands in is static. And Xunit by default is running the tests in parallel, so sometimes by chance tests step on each other's feet.
I added this commit to the branch to prevent the tests from failing.
I know that this is just hiding the symptom rather than a proper solution, but I don't want to spend more time on it now. (Probably that dictionary should be changed to be non-static, but I don't know what effects that would have.)
cc @JorritSalverda

thank you, @markvincze !


SASS-740 - Donation of shares, securities, land and property to overseas charities

Description
Creating page for donation of shares, securities, land or property to overseas charities
Add a link to the relevant story in Jira
[SASS-740]
Checklist PR Reviewer
Before Reviewing

[ ]  Have you pulled the branch down?
[ ]  Have you assigned yourself to the PR?
[ ]  Have you moved the task to “in review” on JIRA?
[ ]  Have you checked to ensure all dependencies are up to date?

Whilst Reviewing

[ ]  Have you run the tests?
[ ]  Have you run the journey tests?
[ ]  Have you looked at the JIRA story to make sure all Acceptance Criteria has been met?

After Reviewing

[ ]  Have you checked for merge conflicts?
[ ]  Have you checked to make sure there are no builds in the pipeline before you merge?
[ ]  Have you moved the task to “in pipeline” on Jira?

[x]  Have you addressed warnings where appropriate?

After PRs been Reviewed

[ ]  Have you checked the PR Builder passes?
[ ]  Have you checked for merge conflicts?
[ ]  Have you checked code coverage isn’t lower than previously?

@dnyuar You need welsh Jsoup tests adding for this buddy

I'll add in the welsh Jsoup tests now


some configuration options not working

I tried configuring the card and it seems like only some of the configuration options are working.  I wanted to use 24 hour time, and show the azimuth and elevation.  However, only the title and language options have any affect on the card when I refresh/clear cache.
My config is:
      - type: 'custom:sun-card'
        title: Sunrise & Sunset
        language: en
        fields:
          - azimuth: true
          - elevation: true
        use12hourClock: false

The title shows up and changes, the language changes if I use another language.  The times are still in 12 hour format and azimuth and elevation do not show.  I even tried the following config and there was no change to the display:
      - type: 'custom:sun-card'
        use12hourClock: false

There are no errors in the logs.

I have the same problem. Can't force 24 hour clock. My config is below but I force 24 hours clock in user profile in the latest release 2021.6.0
unit_system: metric
time_zone: Europe/London
use12hourClock: false does not work for me.

My bad, Github is showing the  develop branch instead the main one, I need to fix that, in the current version you need to use timeFormat: '24h

Works now, thanks!

No problem! next version will change it but it will include a visual editor so it will be easier to set up

thanks, that fixed the 24h format.  However, my elevation and azimuth is still not showing up.
      - type: 'custom:sun-card'
        fields:
          - azimuth: true
          - elevation: true
        timeFormat: '24h'

never mind.  I see those options are different now on the readme.  Probably like the timeFormat, these options are slightly different from the dev branch.


Unable to Recognize Huawei Modem Firmware (Wrong recognition)

Hello @devttys0
I'm trying to identify a Huawei modem firmware binary using binwalk but it doesn't recognize it as jiffs, squash etc
$ binwalk -v Huawei.bin

Scan Time:     2018-01-22 21:34:10
Target File:   /media/data/tmp/Huawei.bin
MD5 Checksum:  e1ddf1d896631b07331b5188d5b31ca2
Signatures:    344

DECIMAL       HEXADECIMAL     DESCRIPTION
--------------------------------------------------------------------------------
4992299       0x4C2D2B        MySQL ISAM index file Version 1

It's definitely not MySQL ISAM index file :)
So, how can I go further? Is there an option to force binwalk to extract it as jiffs file system or squash?
Here is the file:
Huawei.bin.zip
binwalk opcodes result:
$ binwalk -A Huawei.bin

DECIMAL       HEXADECIMAL     DESCRIPTION
--------------------------------------------------------------------------------
12713598      0xC1FE7E        ARM instructions, function prologue
16463970      0xFB3862        ARM instructions, function prologue

Entropy:
$ binwalk -E Huawei.bin

DECIMAL       HEXADECIMAL     ENTROPY
--------------------------------------------------------------------------------
0             0x0             Rising entropy edge (0.996797)

@alivelimeli
How/Where did you find that FW?
If you somehow extracted it from an update blob, it is likely encrypted. But Huawei is know for shitty encryption, so it may be easy to find how it was done. Also for what hardware is it meant?
(E.g. if it is for a Qulacomm based phone, then we have other options...)


optimize:EXTRACT function dispaly

It should return an integer type.

EXTRACT(MONTH FROM T2.CREATE_TIME) AS MONTH

What is your database type

pgsql

Okay, I'll test it now

There seems to be no problem with my testing here. Can you provide your table structure

what 's your version?

3.1.19

wait,I'll update it

ok

no,still not return  integer type.

Can you provide table structure?

here is it.
CREATE TABLE t_wms_outstock_doc (
receive_doc character varying(50)  null,
finish_time timestamp without time zone  null,
doc_no character varying(50)  null,
doc_type character varying(50)  null,
dispatch_sn character varying(50)  null,
doc_status character varying(255)  null,
outstock_date timestamp without time zone  null,
light_flag character varying(10) DEFAULT 'N'::character varying null,
data_auth_out character varying(33)  null,
id character varying(40)  not null,
dept_id character varying(40)  null,
create_user character varying(40)  null,
create_time timestamp without time zone  null,
edit_user character varying(40)  null,
edit_time timestamp without time zone  null,
data_auth character varying(40)  null,
plan_outstock_date timestamp without time zone  null,
close_emp character varying(33)  null,
close_reason character varying(500)  null,
is_urgent character varying(1)  null,
is_upload_erp character varying(1)  null,
memo character varying(500)  null,
close_time timestamp without time zone  null,
area_sn character varying(20)  null,
doc_source character varying(1)  null,
is_release character varying(1)  null,
pick_doc_type character varying(255)  null,
pick_doc_attr character varying(255)  null,
supplier_id character varying(255)  null,
cust_id character varying(255)  null,
data_version numeric DEFAULT '0'::numeric null,
cost_center character varying(40)  null,
expense_type character varying(100)  null,
ascription_type character varying(100)  null,
interior_orders character varying(40)  null,
sap_doc_no character varying(50)  null,
data_sources character varying(10)  null,
connect_no character varying(255)  null,
process_order_id character varying(40)  null,
upload_erp_status character varying(2)  null,
del_flag character varying(5)  null,
cost_item_text character varying(4)  null,
project_base_id character varying(40)  null,
use_dept character varying(50)  null,
sap_create_user character varying(50)  null,
sap_create_time timestamp without time zone  null,
duty_user character varying(40)  null,
duty_dept character varying(40)  null,
other_outstock_item character varying(50)  null,
internal_transfer_order character varying(255)  null,
wbs_id character varying(40)  null,
CONSTRAINT t_wms_outstock_doc_pkey PRIMARY KEY (id)
);

Okay, I'll test it later

thankyou.

It seems like there's still no problem

emm,so strange…

What is the version of your database driver

same as you. But when I export the data, it is an integer


Typo in .verb.md

Should have been included in PR #3. Sorry.

No worries! I would have copied it over, I appreciate you catching the typo.


python310Packages.jupyter-console: 6.6.1 -> 6.6.3; rename from jupyter_console

Description of changes
Changelog: https://github.com/jupyter/jupyter_console/releases/tag/v6.6.3

Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

50 packages built:

ihaskell
python310Packages.ansible-kernel
python310Packages.ansible-kernel.dist
python310Packages.augmax
python310Packages.augmax.dist
python310Packages.einops
python310Packages.einops.dist
python310Packages.encodec
python310Packages.encodec.dist
python310Packages.ilua
python310Packages.ilua.dist
python310Packages.jupyter
python310Packages.jupyter-console
python310Packages.jupyter-console.dist
python310Packages.jupyter.dist
python310Packages.nbdev
python310Packages.nbdev.dist
python310Packages.phik
python310Packages.phik.dist
python310Packages.pydeck
python310Packages.pydeck.dist
streamlit (python310Packages.streamlit)
streamlit.dist (python310Packages.streamlit.dist)
python310Packages.vega
python310Packages.vega.dist
python310Packages.xarray-einstats
python310Packages.xarray-einstats.dist
python311Packages.ansible-kernel
python311Packages.ansible-kernel.dist
python311Packages.augmax
python311Packages.augmax.dist
python311Packages.einops
python311Packages.einops.dist
python311Packages.encodec
python311Packages.encodec.dist
python311Packages.ilua
python311Packages.ilua.dist
python311Packages.jupyter
python311Packages.jupyter-console
python311Packages.jupyter-console.dist
python311Packages.jupyter.dist
python311Packages.nbdev
python311Packages.nbdev.dist
python311Packages.pydeck
python311Packages.pydeck.dist
python311Packages.streamlit
python311Packages.streamlit.dist
python311Packages.vega
python311Packages.vega.dist
quarto


Can't install plyvel

What should I do?

is window

do i need to download leveldb?

I used linux. Everything is solved


Add port quirc

Add port quirc for #2899
It's my first time to add port;
I add a project to test this port: https://github.com/ReyzalX/test-quirc-vcpkg
I have test it in windows 10 + VS2015 and hope this will work well in other environment.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ReyzalX sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the PR!
I made some cleanups while merging, which enables shared builds for quirc and fixes static builds for fastcdr


disblae logging

Question
Executing study.run() creates lots of log information for me, how can I disable this logging?
Further Information
this happens in jupyter web version
Screenshots

https://github.com/Microsoft/LightGBM/issues/1157 I noticed that it is lgbm classifier logs. I still have problems with its logs and don't know if it is possible to set verbose to -1 with autoprognosis.


Commit tagged as v0.7.0 actually has version 0.6.0 in Cargo.toml

At time of writing, this https://github.com/aantn/smag/commit/d1d7b62d842099ba708870ec024cca14cbb3db9b is tagged as v0.7.0. However, the corresponding Cargo.toml has version = "0.6.0" in it.

@aantn, is this really fixed? As far as I can tell, tag v0.7.0 is still d1d7b62d842099ba708870ec024cca14cbb3db9b, which still has version = "0.6.0" in its Cargo.toml.


[RFC] respond_to? for library functions

Many libraries have functions that may or may not exist depending on version, configure options, or platform.  Optional functions allow additional features/methods to be used depending on the specifics of the library installed.  Querying for the availability of a function can be done using dlopen and dlsym.
Suggested syntax:
lib LibC
  fun? foo
end

With
def foo
  if LibC.foo
    LibC.foo
  end

  LibC.try &.foo
end

Or
def foo
  if LibC.respond_to?(:foo)
    LibC.foo
  end
end

Please note the above can be evaluated at compile time and no runtime conditionals are necessary.

This would probably not be feasible before integration with https://github.com/manastech/crystal_lib is done.

I'm not so sure about that.  The logic is pretty simple and can be implemented as compiler directives.
Pseudo code to run during compilation.
def fun?(...)
  if library_has_function ...
    fun(*args)
  else
    fun(name, nil)
  end
end

def library_has_function path, name
  dlopen(...)
  dlsym(handle, name) != NULL
end

@technorama what about usage of static libraries, which cannot be dlopen or dlsym?

Hi, what is going on with this addition?

Hmm, I tried something like this, however I'm always getting false no matter what name I pass into it:
lib LibFoo
  fun version = version : Int32
end

macro exists?(name)
  {{ LibFoo.class.methods.any? { |m| m.name == name.id } }}
end

p exists?("version") # Always false, IDK why

@Heaven31415 As I said, it only works when you use it inside a method, not at the top-level...

Sorry, I just didn't get it first time, now I see, thanks for help.


Initial port to OSS

This is a #minor# change to get the code into OSS, there will be more updates

:+1:

Approved


JWT claim validation incorrectly throws an error for non-integer NumericDate fields

Describe the bug
JWTClaims.validate() throws an InvalidClaimError if either iat, nbf or exp (or any NumericDate field) do not result in integer values once decoded. This is contrary to the JWT specification, which states that noninteger values are allowed (see "Expected behavior").
Error Stacks
Traceback (most recent call last):
  File "test_jwt.py", line 14, in <module>
    claims.validate()
  File .pyenv\lib\site-packages\authlib\jose\rfc7519\claims.py", line 100, in validate
    self.validate_iat(now, leeway)
  File ".pyenv\lib\site-packages\authlib\jose\rfc7519\claims.py", line 198, in validate_iat
    raise InvalidClaimError('iat')
authlib.jose.errors.InvalidClaimError: invalid_claim: Invalid claim "iat"

To Reproduce
from datetime import datetime
from authlib.jose import jwt

with open('jwt-key') as key_file:
	private_key = key_file.read()

with open('jwt-key.pub') as key_file:
	public_key = key_file.read()

token_data = {'iat': datetime.utcnow().timestamp()} # timestamp() returns a float
token = jwt.encode({'alg': 'RS256'}, token_data, private_key).decode('ascii')

claims = jwt.decode(token, public_key)
claims.validate()

Expected behavior
The iat value used above (a float) should be considered valid, as per the spec.
From RFC 7519, section 2 (emphasis mine):

NumericDate
[...]
This is equivalent to the IEEE Std 1003.1,
2013 Edition [POSIX.1] definition "Seconds Since the Epoch", [...],
other than that non-integer values can be represented.

That last part is important since the "Seconds Since the Epoch" definition from the POSIX spec (section 4.16) gives a specific C expression to use where all terms are said to be integer, meaning the resulting value can only be an integer amount of seconds, and RFC 7519 explicitly add the exception that non-integer JSON numbers should be allowed.
Environment:

OS: Windows 10
Python Version: 3.8.0
Authlib Version: 0.14.3

Thanks for your report. I've added float for numeric date validation.


use redis instead of files

Golang Client
https://redis.io/clients#go

Choose Redigo
https://github.com/garyburd/redigo


[SRVKP-5563] Fixed issue with VERBOSE=False for skopeo-copy

https://issues.redhat.com/browse/SRVKP-5563
The issue still persisted due to an extra "" remaining in the command when calling skopeo-copy, fixed it now.

/retest


Explicit queue setting for

registration.tasks.vumi_fire_metric is missing from settings.py

@imsickofmaps Please review.

:+1:


issue regarding colab_selenium

CalledProcessError: Command '['sudo', 'apt', 'upgrade']' returned non-zero exit status 100.
The above exception was the direct cause of the following exception:
GoogleColabSeleniumError                  Traceback (most recent call last)
/usr/local/lib/python3.10/dist-packages/google_colab_selenium/colab_selenium_manager.py in update_upgrade_apt(cls)
49
50         except Exception as e:
---> 51             raise GoogleColabSeleniumError('Failed to update and upgrade APT') from e
52
53         else:
GoogleColabSeleniumError: Failed to update and upgrade APT
    If you believe this isn't due to a configuration error on your end, please
    report the issue with any details you can provide about the steps leading
    up to this error.

Can you send a Notebook recreating the error? The package seems to be working as expected on the example Notebook (in the README.md)


Voyages ms cannot connect to SSL enabled Kafka

Since the Appsody repackaging (#50) I have discovered that the voyages microservice is not connecting to Event Streams successfully.  It fails with an 'SSL error' (without any other useful information).  I'm debugging this locally but don't have a solution yet.  I suspect this applies to any SSL-enabled Kafka (I don't have another one to hand to check).
In the meantime, deployments against SSL enabled Kafkas should use version 0.1.25.

Submitted a fix for this in the nodejs-express Appsody stack: https://github.com/appsody/stacks/pull/838
Turns out, the slim image used to build the application was missing the ca-certificates package, which meant no public certificate verification would work.  The managed Event Streams on IBM Cloud provides a properly signed certificate, which is why we do not need to provide the certificate to the client, however this only works if the CA roots are installed in the image.

This is now fixed in appsody/nodejs-express:0.4.13, so if we re-run the CI, the service should work properly with Event Streams again.

Multitasking at the moment, but was able to kick off another build via https://github.com/ibm-cloud-architecture/refarch-kc-ms/commit/8ed8440988592ef51c772cdbece4f4bd411014ba. You should be able to see voyages:0.1.31 based on the updated Appsody stack now.


Disabling testSelectionManagerEnd test in DatePicker

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Binary change:

DatePicker had been failing again in a PR with unrelated changes. It had already been disabled in #1495 , but only in the Development scheme. We should disable it in all schemes and address this with a follow-up fix once #1502 is merged.
Verification
pod lib lint passes
Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

@sophialee0416 @joannaquu : i thought we were running test only in debug build?

@sophialee0416 @joannaquu : i thought we were running test only in debug build?

You're right it's in debug build, but the error message here says that the scheme is FluentUI-iOS. I can revert the change I made to the Release scheme if you'd like, I only disabled it here just in case


Dump Tools: Use an offscreen canvas

See https://forum.babylonjs.com/t/createscreenshot-in-worker/40198/1.
This change will allow to use the screenshot tools in a worker thread.
Note that OffScreenCanvas is supported by Safari only since 16.4 (https://caniuse.com/?search=offscreencanvas)...

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@RaananW The build fails with the error:
src/Misc/tools.ts(699,44): error TS2339: Property 'convertToBlob' does not exist on type 'OffscreenCanvas'.

How can it be fixed (the method does exist on OffscreenCanvas and I don't have any error locally)?

Checking

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13803/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13803/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13803/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13803/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13803/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13803/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13803/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13803/merge#BCU1XR#0


use of s1s2 in HyperspectralSamples function of HyperFunctions module

Dear YonghaoXu, can you please clarify the use of s1s2 =1 and s1s2=2 in HyperspectralSamples function of HyperFunctions module.

Hi, s1s2 is the index to indicate the strategy utilized for band grouping.
Strategy 1 ( s1s2 = 1 case ) focuses on the local features and makes the signals inside a group continuous without any intervals. Each group concentrates on a narrow spectral range.
By contrast, strategy 2 ( s1s2 = 2 case ) focuses on the global features and each group in this case will cover most of the spectral range compared with strategy 1.

Can you clarify the expression X_reshape[:,j,:] = X[:,j:j+(nb_features-1)*timestep+1:timestep]. When I am trying to run j:j+(nb_features-1)*timestep+1:timestep in a jupyter notebook ,it is showing syntax error. I want to test for every value of j in python. Can you please suggest a method for testing the loop.


Upgrade dependencies

Coverage decreased (-32.8%) to 5.452% when pulling 80caa28bedbaee21b1b6b2266d94473768b87f7d on u9520107:upgrade-dependencies into 5e28b025f3cd1f879ce3716a57c12d45ecd3ea57 on ringcentral:feature/proxy.

Coverage decreased (-0.02%) to 38.28% when pulling 42ea9a8ce2ab8e3ec449491dbf73a1b15b9dbfa6 on u9520107:upgrade-dependencies into 5e28b025f3cd1f879ce3716a57c12d45ecd3ea57 on ringcentral:feature/proxy.


starting packed app: package.json not found, platform dependent

I use pkg 4.4.0 ... 4.4.4 on Windows to create a binary from my node application. Works fine. I can start the resulting binary successfully on the same machine and also on some others.
But there are also machines where the same binary says, package.json (I guess: that one of my application) was not found in /snapshot/appname, I should recompile and include it. I'm sure it was included. I also tried to copy my package.json to the /snapshot-directory before running the binary, no success.
Now, when I recompile my app on the machine where the binary failed to run, it will then run there without errors, but again not on other machines: package json not found. Why???
I use a very concrete target specification to avoid platform specific problems, node10-win32-x64, but this seems to be not concrete enough. Probably the machine where pkg runs has still an impact on the resulting binary.
What can I do to resolve this? And is there a command line switch for getting the /snapshot directory not deleted?

The problem was on my side. I didn't recognize the meaning of the __dirname symbol. I just couldn't imagine that I have a symbol in my code which comes from a package I don't use. I calculated the location of my package.json file myself but not as good as __dirname.


Bug: Color of text in textfields in API Explorer is the same as the background

Description:
The color of text in textfields in the API explorer is the same as the background color
Are you intending to fix this bug?
No
Component(s) Affected:

UI

Steps to Reproduce:

Activate Dark Mode
Go to API Explorer
Expand any Endpoint
Click "Try it out"
Enter anything in a textfield
Text is not visible due to color

Expected Behavior:
Input text should be visible.
Actual Behavior:
Input text is not visible
Screenshots/Code Snippets/Sample Files:
See "Test" written in the "Prefer" textfield in light mode, but not in dark mode.

Environment Information:
BloodHound: v5.15.0
Browser (if UI related): Firefox 129.0.2
Node.js (if UI related: 18.19.0
Contributor Checklist:

[x] I have searched the issue tracker to ensure this bug hasn't been reported before or is not already being addressed.
[x] I have provided clear steps to reproduce the issue.
[x] I have included relevant environment information details.
[x] I have attached necessary supporting documents.

Corrected as of v6.0


Added project 2 scalability weather API test plan and reports

Added project 2 scalability weather API test plan and reports

Good Work ! Merging PR


Add an external cache load balancer and allow https traffic to it.

We should also add a CNAME for www-origin like we have done here


_389-ds-base: 2.0.7 -> 2.1.1

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

1 package failed to build:

_389-ds-base

Closed by #195096.


build: fix github build workflow (macos build)

I approved this pr but check this one thing.
check indentation of line below. Sorry for not commenting inline
https://github.com/noir-protocol/fc/compare/414d0655641e30c7c53eca0a79d866e91f95272e...2c84988e239b2d044124f55e3513d3600e5a176c#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR235

@xedonman Thank you for your detailed review.


Replace python script with tracegen

Which problem is this PR solving?

Resolves #4503

Description of the changes

Replaces the python example with tracegen in order to reduce maintenance needed on the script.

How was this change tested?

Ran locally E2E to confirm traces and metrics are appearing in Jaeger UI.

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
~- [] I have added unit tests for the new functionality~
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

you can also remove the respective section in .github/dependabot.yml

you can also remove the respective section in .github/dependabot.yml

Nice, thanks. Removed in: b850f02


Generate source map for CSS preprocessing

Source map is needed so the developers could find the exact styles in a specific js file.

Has there been any work on this? It would be great to see the style line numbers in chrome inspector when making my styles. I feel like this is one of the few things stopping me from using this framework rather than just scss.

Hello @SkiftCreative,
the generation of source maps is easily achievable if we have AST representation of the code. However, Absurd is not transpiling and it is not dealing with AST transformations. It uses a custom JavaScript logic for generating the CSS which makes the source maps implementation difficult. This issue is opened for years without any actions on it but I'll try addressing this feature in the next months.

That would be awesome. I don't know enough about that part to help. I think having that could really help with adoption as well. Since it will make dev easier.
thanks.


feat: add pybind11 and Cython instructions

Adding tabs for pybind11 and Cython.
I have not previewed this yet.

all three methods of doing this

Great, wasn't sure what to cover.

Could you add the docs URL to GitHub's URL spot?

Good idea, done!


Bug 1152841 - make pdf.js viewer app handle open requests from file viewers

Fixes #5982.

/botio-linux preview


DOC-9880 Update Observability docs for Cloud 2.0 - Resource Usage

Fixes DOC-9880
(1) In cloud-deployments.json, replaced link with resource-usage-basic.html.
(2) Added filter-tabs/resource-usage.md include file.
(3) Added resource-usage-basic.md, (a) copied from resource-usage.md, (b) removed references to standard, (c) removed standard sections.
(4) In resource-usage.md, (a) removed references to basic, (c) removed basic sections.

Hi @Amruta-Ranade, I requested a review from you because we discussed this change weeks ago. I initially had updated the Resource Usage page with internal filters, but that caused problems (described in DOC-9880). This PR creates separate pages for Basic and Standard.


Publish the models as github package

The E2E tests are currently using a models version manually built and pushed in the E2E repo, but if we could publish the model publically we will not have to do it anymore.
I'm proposing GitHub packages because is probably going to be the easiest way, but also maven is fine if we can publish it there

FYI @secondsun @wtrocki

Amazing suggestion! Are you blocked/expecting quick resolution for this?

No no blockers, just want forget it, let's do critical things before :)

like this ? https://github.com/secondsun/app-services-operator/packages/735636
It isn't too hard so if you get blocked we can roll up some hand deployment scripts with a manual version for a fast deployment and then script the proper maven tooling to automatically version the resources.

Nice - thank you so much GitHub!
Can we reset version to operator version as well.

@wtrocki Yes. We haven't been publishing the files so the versions are basically undefined.

Done

@wtrocki can I find it on mvnrepository?

Depending on the package - example for kafka-management
https://github.com/redhat-developer/app-services-sdk-java/blob/main/packages/kafka-management-sdk/README.md


Add option to validate component on blur

Description of changes:

in met-formio add validate on option in validation tab
clean up Form Submit to separate single page from multi page
update package json

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1526 (ad78076) into main (71413a9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1526   +/-   ##
=======================================
  Coverage   72.00%   72.00%
=======================================
  Files         336      336
  Lines        9907     9907
  Branches      720      720
=======================================
  Hits         7134     7134
  Misses       2655     2655
  Partials      118      118

Flag
Coverage Δ

metweb
65.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Remove rest of unused telemetry

Followup to finalize #2858.

Codecov Report

Merging #2863 (816671d) into main (1ac3fd4) will decrease coverage by 1.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2863      +/-   ##
==========================================
- Coverage   28.08%   26.60%   -1.48%
==========================================
  Files         302      125     -177
  Lines       35653    13408   -22245
==========================================
- Hits        10012     3567    -6445
+ Misses      25641     9841   -15800

Impacted Files
Coverage Δ

src/ApiService/ApiService/OneFuzzTypes/Enums.cs
32.39% <ø> (ø)

src/ApiService/ApiService/OneFuzzTypes/Model.cs
69.62% <ø> (+0.04%)
:arrow_up:

src/ApiService/ApiService/onefuzzlib/Config.cs
0.00% <ø> (ø)

src/ApiService/ApiService/onefuzzlib/Defs.cs
0.00% <ø> (ø)

src/agent/coverage/src/allowlist.rs

src/agent/coverage/src/binary.rs

src/agent/onefuzz-task/src/local/coverage.rs

...c/agent/onefuzz-task/src/tasks/coverage/generic.rs

src/agent/onefuzz-telemetry/src/lib.rs

src/agent/coverage/src/record/linux/debugger.rs

... and 171 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Release Zulip Server 9.2.

Might also be worth mentioning that zerver/migrations/0576_backfill_imageattachment.py might be a big one if they have a lot of attachment history?

LGTM other than that.

Closing as 9.2 has been released.


PASTE: handle images

Input (https://www.bbc.co.uk/news/business-50723392):
Select the image and the para just below it
Wag! App Walk Report CardImage copyrightWAG INC.
Technology giant SoftBank has reportedly agreed to sell its stake in dog-walking app Wag.
Error:
[Error] TypeError: firstChild.hasBlockChildren is not a function. (In 'firstChild.hasBlockChildren()', 'firstChild.hasBlockChildren' is undefined)
rethrowCaughtError (react-dom.development.js:6284)
runEventsInBatch (react-dom.development.js:748)
runExtractedPluginEventsInBatch (react-dom.development.js:880)
handleTopLevel (react-dom.development.js:5807)
batchedEventUpdates$1 (react-dom.development.js:24368)
batchedEventUpdates (react-dom.development.js:1419)
dispatchEventForPluginEventSystem (react-dom.development.js:5898)
attemptToDispatchEvent (react-dom.development.js:6014)
dispatchEvent (react-dom.development.js:5918)
dispatchEvent
unstable_runWithPriority (scheduler.development.js:818)
discreteUpdates$1 (react-dom.development.js:24384)
discreteUpdates (react-dom.development.js:1442)
dispatchDiscreteEvent (react-dom.development.js:5885)
dispatchDiscreteEvent

Maybe a duplicate of #209

This seems to be resolved.


Issues with elastic apm rum configuration with webpack

Please find the details wrt.. issue
package.json
"react": "^18.2.0",
"react-dom": "^18.2.0",
"react-scripts": "5.0.1",
"webpack": "^5.75.0"
"webpack-cli": "^5.0.1",
"webpack-dev-server": "^4.11.1"
"@elastic/apm-rum": "^5.12.0",
"@elastic/apm-rum-react": "^1.4.2",
bootstrap.js
import React from "react";
import ReactDOM from "react-dom/client";
import "./index.css";
import App from "./App";
import apm from "../rum.js";
import config from "../Config.js";
apm.setInitialPageLoadName(config.getApmServiceName());
const root = ReactDOM.createRoot(document.getElementById("root"));
root.render(
<React.StrictMode>

</React.StrictMode>
);
rum.js
import { init as initApm } from "@elastic/apm-rum";
import Config from "./Config.js";
const apmServerUrl = Config.getApmServerUrl();
const apmServiceName = Config.getApmServiceName();
const apmAppEnvironment = Config.getApmAppEnvironment();
var apm = initApm({
serviceName: apmServiceName,
serverUrl: apmServerUrl,
verifyServerCert: false,
active: true,
instrument: true,
environment: apmAppEnvironment,
});
export default apm;
with the above configurations, while starting the application we are getting below error in the console and stopping the page load.
Error:
Uncaught (in promise) TypeError: (0 , elastic_apm_rum_core__WEBPACK_IMPORTED_MODULE_2_.bootstrap) is not a function
at getApmBase (index.js:21:81)
at eval (index.js:32:15)
at<EMAIL_ADDRESS>(vendors-node_modules_elastic_apm-rum_dist_es_index_js.js:513:1)
at webpack_require (main.js:339:33)
at main.js:551:198
at webpack_require.m. (main.js:1152:34)
at webpack_require (main.js:339:33)
at fn (main.js:628:21)
at eval (rum.js:2:74)
at ./src/rum.js (node_modules_babel-loader_lib_index_js_ruleSet_1_rules_3_use_src_app_bootstrap_js.js:28:1)
Please help me on this issue.

Hi, without a reproducible repo, we can't really help. You may ask it on stackoverflow.


Have to commit the node_module folder

Unfortunately, there is an issue with Handlebars that stops it from being used with ncc.
Until this is addressed the node-modules folder needs to be committed to the repo containing the production dependencies for use at runtime.
There error is
Processing template
Error: Invalid left-hand side in assignment
##[debug]Node Action run completed with exit code 1
##[debug]Finishing: Run /./

This has been logged with Handlebars as #375

A test was done using #82 but the problem still exists

Re tested with "@vercel/ncc": "^0.29.0" still has the same issue


Switching deprecated babel-preset-latest to babel-preset-env

replacing deprecated babel-preset-latest with babel-preset-env
to avoid the following npm installation warning:
npm WARN deprecated<EMAIL_ADDRESS>preset-latest accomplishes the same task as babel-preset-env.
Please install it with 'npm install babel-preset-env --save-dev'.
'{ "presets": ["latest"] }' to '{ "presets": ["env"] }'.
For more info, please check the docs: http://babeljs.io/docs/plugins/preset-env

Ref. #1667

Coverage remained the same at 99.242% when pulling 33b99d9978b073c02142beb064e440b597681f5a on dennybiasiolli:babel_latest_to_env into ea1e1c451c12bd4b36057d884a1087d45a27310a on react-boilerplate:dev.

Coverage remained the same at 99.242% when pulling 3e5778d1312b91df910134788937a57d571002e6 on dennybiasiolli:babel_latest_to_env into 106985f436dcea84d3ac75675d1dfdc368e708bc on react-boilerplate:dev.

Coverage remained the same at 99.242% when pulling 5dcb610ae9bc38bafff735f6980d0a676f960bea on dennybiasiolli:babel_latest_to_env into cfe0c7ce16e376bb32df1ac6b27ac715a6cf6c1f on react-boilerplate:dev.

Coverage remained the same at 99.231% when pulling c1f8904cb78ce021e142c6972f97824cadef2faa on dennybiasiolli:babel_latest_to_env into a2173a26335cb63e9cb13707afafd12ced7cdfa5 on react-boilerplate:dev.

Coverage remained the same at 99.231% when pulling 48df4e08c12a756100126693f7ac6043cce1e199 on dennybiasiolli:babel_latest_to_env into 5ec8dd423a56be9a9ac1ef943e8779b4b8e9646d on react-boilerplate:dev.

Coverage remained the same at 99.231% when pulling 48df4e08c12a756100126693f7ac6043cce1e199 on dennybiasiolli:babel_latest_to_env into 5ec8dd423a56be9a9ac1ef943e8779b4b8e9646d on react-boilerplate:dev.

Coverage remained the same at 99.237% when pulling d7cb0f34a0f56294a1dc0d2ecfdf187116c03140 on dennybiasiolli:babel_latest_to_env into 24de895d587870ab0e204e9d64f6da0f0b837349 on react-boilerplate:dev.

@KarandikarMihir does I have to rebase this PR?

@dennybiasiolli No we'll squash and merge.

Coverage remained the same at 99.237% when pulling 15180ebbfe6327a894395e182250f43ca269b206 on dennybiasiolli:babel_latest_to_env into dbde338bf4b74a4bc77dbd58ecbc25d36cba7a5d on react-boilerplate:dev.

@Dattaya it was a pleasure! 😉


rev to 0.1.0

rev to 0.1.0
update various markdown badges (now that we have a published version)

@eliasyishak - if we want to enable this - 'merge and tag to publish' - there's a bit of config to do on the pub.dev page.


Need rule which catches assignments to powershell builtin variables

assignments to variables like $args and $_, etc should be caught.
see https://github.com/PowerShell/PSScriptAnalyzer/issues/265

For the code, the FindAll() call looks for AssignmentStatementAst and not ParameterAst, it may be a similar control but in a different context.
According to this:
TOPIC
    about_Automatic_Variables

SHORT DESCRIPTION
    Describes variables that store state information for Windows PowerShell.
    These variables are created and maintained by Windows PowerShell.

Except for $OFS which sets a processing, I do not see for the moment, other variables of this list for which one authorized an assignment.
$Matches and $PSBoundParameters can be modified but by method calls.
Maybe $Matches could be excluded to allow this type of code :
#Version 1
function Test($p) {
 $p -match "e"
 return $matches
}
Test 'abc'
'test' -match "e"
Test 'abc'

#Version 2
function Test($p) {
 $matches=$null
 $p -match "e"
 return $matches
}
Test 'abc'
'test' -match "e"
Test 'abc'

And these Plaster, PSSA
Maybe $PSBoundParameters could be excluded to allow this type of code
and :
# # ==============================================================
# @ID       $Id: GetChildItemExtension.ps1 1291 2012-11-09 19:59:07Z ww $
# @created  2011-07-01
# @project  http://cleancode.sourceforge.net/
# ==============================================================

...
		# 2011.08.04 msorens bug fix: no parameters threw null exception
		if ($PSBoundParameters.Count -eq 0) { $PSBoundParameters = @{} }

		$wrappedCmd = $ExecutionContext.InvokeCommand.GetCommand( `
			'Get-ChildItem', [System.Management.Automation.CommandTypes]::Cmdlet)
		$filters = @("$wrappedCmd @PSBoundParameters")
...

Can be considered the following cases ?
#With Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.dll, 3 february 18, 13:18:28
#https://ci.appveyor.com/project/PowerShell/psscriptanalyzer/build/1.0.2099/job/seht4ms91da3mbss
$sb2={
$T=@('Foo','Foo')
$Error,$?=$T
}
Invoke-ScriptAnalyzer -ScriptDefinition $sb2.tostring()

$sb22={
$T=@('Foo','Foo')
$Ok,$?=$T
}
Invoke-ScriptAnalyzer -ScriptDefinition $sb22.tostring()

$sb3={
 $Error,$?='Foo','Foo'
}
Invoke-ScriptAnalyzer -ScriptDefinition $sb3.tostring()

$sb33={
 $ok,$?='Foo','Foo'
}
Invoke-ScriptAnalyzer -ScriptDefinition $sb33.tostring()

$Texte=@'
 foreach ($Error in 0..5)
 {
   "Error=$Error"
 }
'@
Invoke-ScriptAnalyzer -ScriptDefinition $Texte

$Texte=@'
  $local:Error=10
'@
Invoke-ScriptAnalyzer -ScriptDefinition $Texte

$Texte=@'
 Invoke-Command -ComputerName '.' -ScriptBlock {$pwd=$using:pid}
'@
Invoke-ScriptAnalyzer -ScriptDefinition $Texte

$Texte=@'
 $Sb={$pwd=$using:pid}
 Invoke-Command -ComputerName '.' -ScriptBlock $sb
'@
Invoke-ScriptAnalyzer -ScriptDefinition $Texte

Another extreme case :
function Test{ param([ref]$v) $v.value;$v.value=''}
Test ([ref]$PID) #Exception read only
Test ([ref]$pwd)
Test ([ref]$profile)

Although the title indicates 'assignment', does this rule apply to all modifications of builtin variables?


fhirServiceToCdm.json variable never used

The blobUrl variable is never used.

The final step is showing succeeded in Synapse, but the stderr log is showing:
'Unhandled exception: System.UriFormatException: Invalid URI: The URI is empty.
at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
at Microsoft.Health.Fhir.Transformation.BatchExecutor.Program.<>c__DisplayClass0_0.<b__2>d.MoveNext() in C:\Users{user}\source\repos\FHIR-Analytics-Pipelines\Microsoft.Health.Fhir.Transformation.BatchExecutor\Program.cs:line 95
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Invocation.CommandHandler.GetResultCodeAsync(Object value, InvocationContext context)
at System.CommandLine.Invocation.ModelBindingCommandHandler.InvokeAsync(InvocationContext context)
at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<b__21_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass16_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass25_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass23_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<b__22_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<b__20_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<b__11_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<b__10_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass14_0.<b__0>d.MoveNext()'
I don't understand why it reference my local computer path when this is being executed remotely on the VM node. It seems to relate the urlBlob not being set, but I'm not 100% sure.

I see now. @item().url. The CheckExportTaskStatus step passes the list of discovered Resources and then the ForEach loops them. That makes sense.


enhance stm32_print_devices_by_family

Thanks. I already started to do some stuff. I'll merge this one in first and then apply some minor improvements.


Add CRC field to #kafka_message{}

It would be great to have that field in #kafka_message{}, is it possible?

it is possible.
but, do you have a use case example ?
i.e. why would upper level application care about a message checksum ?

It's a very specific case and I believe it's related only (maybe) to our application.
I want to use CRC as a fingerprint of the message for additional type of authorization.
It's hard to explain more detailed without the whole description of our system.
But I hope that field addition should not impact the brod and may be useful for someone else.

interesting use case, i'd love to know how it works if you are willing to share.
Add one more field to kafka_message record should not impact anything.
(except for the users who's doing hot swapping upgrades,
i.e. some queued messages might get badrecord error after new beam is loaded)
I'll look into this


add description for service account token

PR https://github.com/kubernetes/website/pull/9182 has added docs for the service account projection feature. This PR gives a brief introduction of this new feature in serviceaccount docs.
ref:https://github.com/kubernetes/kubernetes/pull/63819

@mistyhacks PTAL

/lgtm
/approve


Flaky installation of Solc on CI

We use svm-rs by @roynalnaruto, which is used for managing the global solc binary's version, by installing each version under ~/.svm/4version/solc-$version. For some reason, when running on CI the installation process seems to be flaky, e.g. below:

We should fix that. Maybe there's a bug downstream in svm-rs (e.g. there should be some sort of global mutex that says "hey a solc version is being installed right now, wait for it to unlock before you try running solc"
I tried doing that manually with the lock inside solc/lib.rs but it was not enough apparently.

@roynalnaruto may have some capacity this week to tackle this


xnu not building on aarch64-darwin

Describe the bug
The xnu package, which is a dependency of openconnect via network_cmds, doesn't build on aarch64-darwin (M1 macs).
Steps To Reproduce
Steps to reproduce the behavior:

Clone current master
nix-build -A xnu
See failure

Expected behavior
Build succeeds
Additional context
This might be related to #150880.
Failing build: https://hydra.nixos.org/build/172392835
clang-11: error: cannot use 'cpp-output' output with multiple -arch options
mig: fatal: "<no name yet>", line -1: no SubSystem declaration

Notify maintainers
@NixOS/darwin-maintainers
Metadata
$ nix-shell -p nix-info --run "nix-info -m"

- system: `"aarch64-darwin"`
 - host os: `Darwin 21.4.0, macOS 12.3`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.6.1`
 - channels(kubukoz): `"nixpkgs"`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Did it build before the openssl drop?
I don't think any of the macOS 11 source releases build (except by coincidence) since the stdenv was based on the SDK because it's a lot of effort to get the source releases to build.

tbf it probably didn't build, sorry 😅 now I realised I was using the x86 versions of everything. These don't work now either, though - I'll open a new issue.

See #170454 to fix network_cmds.

See #179921 for a fix.

Fixed by #179921.


Give the Manage Repo Auto Imports popup a dedicated route

In general - consider the Manage Repo Auto Imports area - it's currently a full page "popup" which can't be directly linked to. We could:

Just connect the url to the popup (so that /repos/auto-imports maps to the open popup)
Make it a dedicated page in our UI (with the rest of the nav context)

Closing this out as it's no longer relevant


Is it possible to add 'remove link' feature?

So far we can add links, but not remove it. Seems like a bug.

ok, that makes sense. thanks


Illegal character ((CTRL-CHAR, code 31))

Hi,
I am getting this error when calling a Jax-rs Webservice using the Jackson libraries.
the problem is same to that one https://github.com/square/okhttp/issues/635
Does someone have an idea of how to resolve the problem ?
Many thanks

@stephano88 - Having same problem, just want to check if you were able to find a solution for this (I understand this post is very old)..

咋解决的

This is caused by transparent GZIP compression.
To remove compression, add a header request.header("Accept-Encoding", "identity")


HackaComm Edit description and banner

CUSEC logo should be added to the hackacomm banner

The meta description makes it look like registration is open until January 10, change to January 5.

Closed with PR #83


id "com.github.hierynomus.license" version "0.15.0"

0.14.0 已经无法找到了

It seems like that 0.14.0 is still alive, but it is not the latest version.
Could you please give us a PR?
Thanks.


Snapshot tests 0.8.x [in progress]

Partial fix to  #1099
Additional description
In general, the movement of the library is to allow story-creation to be test creation.
This updates Storyshots to do PNG and markup snapshots for:
export Blank from '../tests/initial-blank-stories';
export Accordion from '../components/accordion/__docs__/storybook-stories';
export Alert from '../components/alert/__docs__/storybook-stories';
export Avatar from '../components/avatar/__docs__/storybook-stories';
export BrandBand from '../components/brand-band/__docs__/storybook-stories';
export Breadcrumb from '../components/breadcrumb/__docs__/storybook-stories';
export Button from '../components/button/__docs__/storybook-stories';
export ButtonGroup from '../components/button-group/__docs__/storybook-stories';
export ButtonStateful from '../components/button-stateful/__docs__/storybook-stories';
export Card from '../components/card/__docs__/storybook-stories';
export Checkbox from '../components/checkbox/__docs__/storybook-stories';
export Combobox from '../components/combobox/__docs__/storybook-stories';
export Filter from '../components/filter/__docs__/storybook-stories';
export DataTable from '../components/data-table/__docs__/storybook-stories';
export GlobalHeader from '../components/global-header/__docs__/storybook-stories';
export Input from '../components/input/__docs__/storybook-stories';
export Icon from '../components/icon/__docs__/storybook-stories';
export Illustration from '../components/illustration/__docs__/storybook-stories';
export MediaObject from '../components/media-object/__docs__/storybook-stories';
export PageHeader from '../components/page-header/__docs__/storybook-stories';
export ProgressIndicator from '../components/progress-indicator/__docs__/storybook-stories';
export Tree from '../components/tree/__docs__/storybook-stories';

CONTRIBUTOR checklist (do not remove)
Please complete for every pull request

[ ] First-time contributors should sign the Contributor License Agreement.  If you haven't before, wait a few minutes and a bot will comment on this pull request with instructions.
[ ] npm run lint:fix has been run and linting passes.
[ ] Mocha, Jest (Storyshots), and components/component-docs.json CI tests pass (npm test).
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

REVIEWER checklist (do not remove)

[ ] TravisCI tests pass. This includes linting, Mocha, Jest, Storyshots, and components/component-docs.json tests.
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

Required only if there are markup / UX changes

[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

Updating tests


Language Injection Plugin for IntelliJ

EdgeQL looks very nice but without syntax highlighting and autocompletion it would be harder to use.
An IntelliJ plugin could be created to detect strings with EdgeQL in them and automatically set the language for the string's contents. This is called language injection.
You can create custom language injection plugins which will allow us to provide syntax highlighting and more importantly autocompletion.

This is a bit related to https://github.com/edgedb/edgedb-editor-plugin/issues/20
But I think you're asking for multiple things here:

IntelliJ syntax highliting
IntelliJ language injection: it probably makes sense to start with already supported editors and maybe by the most popular: VS Code.
Autocompletion: this is pending on LSP and it's post 1.0

Any update here? Autocompletion is a nice to have, but simple syntax highlighting is really crucial

My opinion here. I love EdgeDB idea and everything, but it has cons that make it almost non-usable:

No syntax highlight in IntelliJ - that's a huge CON because it disables ability to do an efficient refactoring or usages lookup. Imagine searching for all usages of "user" table in a big project. I would better write raw SQL with its cons, but I would have highlighting and refactoring features in IDE.
No query builder - compared to ORMs, EdgeDB brings back a giant problem of manual concatenating strings with dynamic WHERE and ORDER BY conditions. And another problem it adds is that these concatted conditions will miss syntax highlight since IDE will not recognise EdgeDB syntax in these little strings - so we loose refactoring and usage lookup features again.

I understand that it's a young project and it will become a lot better later, but these problems makes it non-usable in real production projects for now.
p.s. vscode or any other editor is not a solution. It's just an editor that doesn't have refactoring features almost at all.

This project is not that young any more, been very disappointed to be waiting so long for JVM / IDE support.
I get that the team wants to focus on their "cloud offering" but this feels ridiculous :(
Not like I could adopt their cloud offering before they have basic editor and language support anyway.

True. Cloud offering is great feature, I would like to use it, but only after I can at least use the db in the project 😄

JVM support is here, and IntelliJ is introducing LSP support in their next version.  https://blog.jetbrains.com/idea/2023/06/intellij-idea-2023-2-eap-7
Is 2024 the year of EdgeDB?

https://youtrack.jetbrains.com/issue/DBE-17080/EdgeDB-support
Official response for native integration

No plans, demand low

Averaged user opinion from this topic:

No support - no use

So unless team makes first step, it will be endless loop of "no support because low demand because no support because low demand because ...".
Meanwhile the only response from team in this topic is 3 years ago, waiting for v1 (v5 is already out as of today).


Stacking Popup of same type

Issue: Unable to Stack Popups of the Same Type
Problem
Currently, it is not possible to stack popups of the same type. For instance, displaying multiple TopNotification popups simultaneously is not feasible because they all share the same identifier.
Code Example
public extension Popup {
    var id: String { .init(describing: Self.self) }
    var body: V { createContent() }

    func configurePopup(popup: Config) -> Config { popup }
}

Issue Details
Due to the same identifier (TopPopup_Notification), these messages can only replace each other rather than stack.
Proposed Solution
The id of the Popup should be a property of the caller, making it an optional description of self. This change would allow for multiple notifications of the same type to be displayed concurrently.
Benefits of Custom Popup IDs

Stacking Multiple Popups: Enables multiple notifications of the same type to be displayed simultaneously.
Targeted Removal: Allows a specific popup to be removed from the stack.
Existence Check: Facilitates checking for the existence of a specific popup on the stack before taking action.
Maintaining Current Behavior: Keeps the current behavior intact while allowing for multiple instances of the same type.

Visual Example

This has a while loop on this message on the demo, but only one message is shown.
Expected Behavior

Messages of same Notification Type are reused and stacked.
Conclusion
Modifying the id property to be customizable will enhance the flexibility and functionality of the popup system, allowing for a more dynamic and responsive user experience.

Hey @dentvii,
Thanks for this post. It does indeed make sense. I'll try to squeeze this feature into the current sprint, but no promises, as I need to finally finish the new library I put on hold about two months ago (so in the worst case, this will be implemented at the end of August) 😅
Thanks for the ticket and have a nice day,
Tomasz

Just adding to this as i'm also hitting this issue for a feature!
Making the ID property public/ with public initialisers should resolve for us?

I think i'm having the same issue, the loading popup here does not get replaced
    @MainActor
    private func startLoading() {
        MiddlePopup(state: .loading)
            .showAndStack()
    }

    @MainActor
    private func completeLoading(_ state: PopupState) {
        print("loading completed: \(state)")
        MiddlePopup(state: state)
            .showAndReplace()
    }

Hey, this feature is in our backlog and will be implemented next month.

Cool thanks! And thanks for the great library, it’s much nicer to use than the pure state driven SwiftUI popup view libraries.

@jamesyorke, @dentvii, @praveenperera,
Just letting you know that the feature is already implemented in the branch patch-3.0.0. It can be used as follows:

I would definitely advise to not use the branch on prod, but you can test the changes if you have some time.
I plan to publish a pre-release version on 16 October and launch it 7-10 days later.

Hey @jamesyorke, @dentvii, @praveenperera
Just letting you know that I've finished the update. There are some changes to the API and the GitHub documentation (with migration guidelines) is not yet ready, but the library has extensive documentation with some code examples that I hope will help you understand the new features.

Let me know if you need any help and have a nice day,
Tomasz


(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/: dial tcp: lookup registry-1.docker.io on [::1]:53: read udp [::1]:40044->[::1]:53: read: connection refused"}

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Are you behind a firewall or VPN?
See our wiki for possible solutions:
https://github.com/docker/kitematic/wiki/Common-Proxy-Issues-&-Fixes
Verify that you can do docker run hello-world from the CLI first


Improve LICENSE text file to be correctly recognised

This pull request renames the file license.txt to the de-facto standard name LICENSE. In addition, it replaces the URL by a heading that points to the used BSD 3-Clause License.
Note that this does not change the license at all, just its representation to a more standard-conform way. Currently, the license can't be correctly recognised by GitHub. For instance, in the list of repositories, the license symbol is missing for this repository:

After accepting this pull request, GitHub also shows information about what is allowed when accessing viewing the LICENSE file, as in my fork:

Thanks a lot Falco :)


WIP: Add strict types for all php files

Resolve #352
This PR adds declare(strict_types=1) to all php files and an explanation what it means.
Changes
Rule to enfore strict types
An additional phpcs rule enforces a proper declaration of strict types for each file. The rule SlevomatCodingStandard.TypeHints.DeclareStrictTypes from slevomat/coding-standards takes care of this.
Rule to enforce explanation of strict type
... tbd ...
Using composer for lint check
To properly include the custom rules, composer is uses to run the check in Github actions.
Specifying php version
To properly specify the version used to CI action, the key php-version needs to be used, instead of version. See shivammathur/setup-php

If this is WIP can we make it a draft PR? Looks good so far, just needs the explanatory message

Yes, please make it a draft. I know there are some things open and I'll ping you when I think it is ready.

@neenjaw I feel like the PR is ready to be checked. All changes should be explained in my comment above, I hope 😉

Cool, I'll check it out later today

Yes, good idea to go for the 80 chars. I am gonna change it and then you can merge.

@neenjaw I tried to justify the text a bit more but it is hard to not "destroy" the sentence with unneeded line breaks. Do you think it is ok so?

Thanks @SleeplessByte, I think one of those formats would be great.  @codedge, which formatting do you think works best for this use?

I change the text to match the second suggestion. Looks better now?

Looks great, let's merge it


Error in estimator's preprocessing TensorflowV2

The preprocessing in tensorflow does not work as expected when,
mean = [125.307, 122.95, 113.865]
std  = [62.9932, 62.0887, 66.7048]
classifier = art.estimators.classification.TensorFlowV2Classifier(..., clip_values=(0, 255), preprocessing=(mean, std))

A possible workaround, that is working, is to replace the lines 65-66 in preprocessing/standardisation_mean_std/standardisation_mean_std_tensorflow.py
x_norm = x  - mean
x_norm = x_norm / std

with
x_norm = x - tf.broadcast_to(mean, tf.shape(x))
x_norm = x_norm / tf.broadcast_to(std, tf.shape(x_norm))

Hi @shashankkotyan Thank you very much for raising this issue. Currently all estimators expect either a tuple of two int, float or np.ndarray for mean and std. Can you try with:
mean = np.ndarray([125.307, 122.95, 113.865])
std  = np.ndarray([62.9932, 62.0887, 66.7048])


Pass custom arguments to build commands

Problem
I would like to pass --bins to the cargo build command run as part of cargo chef cook. This option is not supported though. There already are a lot of command line flags that are simply copied and passed to the underlying command.
Proposed Change
Allow custom arguments to be added for example through --additional-args or simply through cargo chef cook --clippy -- --clippy-arg-1 --etc. Then cargo-chef does not need to expose every cargo flag directly. Similarly, cargo-chef could allow custom commands to be used (instead of the predefined set check, build, clippy, and zigbuild)
Potential issues
Users could set flags that would prevent the clippy commands from working.

We made an explicit choice of not doing this: certain flags influence the way in which cargo-chef might behave.
When a flag is desired, we investigate it and add it.


No longer able to use sass

After the commits over the past two days, I'm no longer able to use sass like so:  <style lang="sass" scoped>
ERROR in ./~/css-loader?sourceMap&-autoprefixer?sourceMap!./~/vue-loader/lib/style-rewriter.js?id=data-v-c87e08b0&scoped=true!./~/sass-loader?sourceMap!./~/vue-loader/lib/selector.js?type=styles&index=0!./client/views/Home.vue
Module build failed:
<template>
^
      Invalid CSS after "": expected 1 selector or at-rule, was "li:hover"
      in /Users/mack/Projects/gamewatch/client/views/Home.vue (line 1, column 1)
 @ ./~/vue-style-loader!./~/css-loader?sourceMap&-autoprefixer?sourceMap!./~/vue-loader/lib/style-rewriter.js?id=data-v-c87e08b0&scoped=true!./~/sass-loader?sourceMap!./~/vue-loader/lib/selector.js?type=styles&index=0!./client/views/Home.vue 4:14-317 13:2-17:4 14:20-323
 @ ./client/views/Home.vue
 @ ./client/router/index.js
 @ ./client/app.js
 @ ./client/index.js
 @ multi client

@egoist Hrm, I'm not sure what's going on. Still getting the same error after your fix.

what's your actual sass code?

<style lang="sass" scoped>
  li:hover
    .title
      color: #6BFF4F
      opacity: 1

  .box
    &--subscribed,
    &:hover
      .box__details
        opacity: 1
      img
        opacity: 0.2
    &--subscribed + .title
      opacity: 0.2

  .box__details
    opacity: 0

  .title
    overflow: hidden
    display: -webkit-box
    -webkit-line-clamp: 2
    -webkit-box-orient: vertical

  label
    cursor: pointer

  label
    content: ''
    background-color: #fff
    border-radius: 3rem
    padding: 2px
    position: relative
    width: 44px
    height: 1.5rem

    &::after
      transition: transform 0.2s ease-in-out
      content: ''
      background-color: #050506;
      border-radius: 100%
      position: absolute
      height: calc(1.5rem - 4px)
      width: calc(1.5rem - 4px)

  input[type="checkbox"]:checked + label
    content: ''
    background-image: linear-gradient(-90deg, #6BFF4F 0%, #26FF44 98%)
    box-shadow: 0 2px 36px 0 rgba(0,255,21,0.23)
    color: black

    &::after
      transform: translate3d(100%, 0, 0)

  select
    appearance: none
    background: transparent
    color: #6BFF4F
    border: none
</style>

Fixed again 😅 https://github.com/egoist/vuepack/commit/17dfbed361a46641001542b2a2355d67f976cc38

Works like a charm! Thanks much @egoist 💙


Dynamically loading 2 (or more) grids isn't working

My application requires grids to be added and loaded dynamically.
Adding a single grid works fine, but loading two (or more) grids results in the second one working properly but the first one failing.
My method is as follows:

get a 'serialisation' array of grids and widgets (from indexeddb) to load grids and widgets
dynamically add each of the gridster divs and uls inside a container row. Attribute unique ids to each gridster div as the namespace
dynamically add the li widgets inside each grid ul. Each widget contains serialize params including the parent grid.

All works fine this far (a bit like recreating the 'grid-from-serialize.html' demo but with 2 grids and multiple widgets per grid.

Instantiate each grid where gridster[i] will be gridster[0] and second time round gridster[1]  and where thisNamespace will be the unique gridster id in the parent div.

$(function () {
        gridster[i] = $(thisNamespace+" ul").gridster({
        namespace: thisNamespace,

The second one works fine. In the first grid, the widgets are piled up on row 1 col 1 and won't move and the grid is not of the proportions defined. It's as though it hasn't been instantiated.
Here's the full code for instantiating the grids in a for loop:
    $(function () {
        gridster[i] = $(thisNamespace+" ul").gridster({
        namespace: thisNamespace,
        widget_base_dimensions: [200, 140],
        widget_margins: [10, 10],
        rows: 1,
        max_rows: 1,
        extra_cols: 5,
        min_cols: 20,
        max_cols: 25,
         serialize_params: function ($w, wgd) {
        return {
            'data-plotlineid': $w.attr('data-plotlineid'),
            'data-heading': $w.attr('data-heading'),
            'data-plotlinegrid': $w.attr('data-plotlinegrid'),
            col: wgd.col,
            row: wgd.row,
            size_x: wgd.size_x,
            size_y: wgd.size_y
            };
        },
        scroll_container: $('#story-panel-body-middle'),
        shift_widgets_up: false,
        move_widgets_down_only: true,
        s_down: false,
        collision: {
        wait_for_mouseup: true
            },
      draggable: {
                handle: 'header'
            }
      }).data('gridster');

Can anyone advise on what I might be doing wrong?
P.S. I've tried adding the widgets to each grid in a for loop after instantiating but hit the same wall.
Many thanks.

Looping in a promise or setting a timeout doesn't resolve the problem.


Add Check template sample for testing code generation and config/requ…

We have one sample for report that is used for testing of the code gen work. This PR adds another sample for Check kind templates.
It also defines a CacheabilityInfo info message.

This change is

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed


All info boxes show up

Fresh install.  But it seems like every single info box says "ready" - the normal time tracker plugin shows nothing is ready besides some compost.

We also show "Ready" when nothing is planed in the spot (since you have to go there and do something). Is that the case?

Yes, that is the case. Interesting.

👍 That's intended then. You can disable the ones you can't do yet in the settings :)


Downloading<EMAIL_ADDRESS>

When trying to execute the boilerplate  boilerplates\materialize-iron-router
It starts dowloading meteor-tool and never ends...

Well after a while,,, it ended...


Peer attrition fix

This PR fixes issues observed with peer attrition.

Some extension to this fix might still be needed, as my node was still able to exhaust its peers and not seek further ones at all; the following is a perf report at the time when the node stopped doing anything:
Samples: 40  of event 'cycles:u', Event count (approx.): 245819
  Children      Self  Command          Shared Object       Symbol
+  100.00%     0.00%  tokio-runtime-w  libc-2.33.so        [.] __GI___clone (inlined)
+  100.00%     0.00%  tokio-runtime-w  libpthread-2.33.so  [.] start_thread
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] std::sys::unix::thread::Thread::new::thread_start
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once (inlined)
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] core::ops::function::FnOnce::call_once{{vtable.shim}}
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] std::sys_common::backtrace::__rust_begin_short_backtrace
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] tokio::runtime::blocking::pool::Inner::run
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] tokio::runtime::task::harness::Harness<T,S>::poll
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] tokio::loom::std::unsafe_cell::UnsafeCell<T>::with_mut
+  100.00%     0.10%  tokio-runtime-w  snarkos             [.] tokio::runtime::thread_pool::worker::Context::run
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] tokio::runtime::thread_pool::worker::run
+  100.00%     0.00%  tokio-runtime-w  snarkos             [.] tokio::macros::scoped_tls::ScopedKey<T>::set
+   73.41%     0.00%  tokio-runtime-w  snarkos             [.] tokio::runtime::thread_pool::worker::Context::park_timeout
+   73.41%     0.00%  tokio-runtime-w  snarkos             [.] <tokio::runtime::park::Parker as tokio::park::Park>::park
+   55.44%     0.05%  tokio-runtime-w  snarkos             [.] tokio::time::driver::Driver<P>::park_internal
+   35.87%     0.00%  tokio-runtime-w  libc-2.33.so        [.] __clock_gettime_2 (inlined)
+   28.03%    28.03%  tokio-runtime-w  libc-2.33.so        [.]<EMAIL_ADDRESS>+   19.77%    19.77%  tokio-runtime-w  snarkos             [.] tokio::time::driver::wheel::Wheel::poll
+   19.77%     0.00%  tokio-runtime-w  snarkos             [.] tokio::time::driver::<impl tokio::time::driver::handle::Handle>::process_at_time
+   18.66%    18.66%  tokio-runtime-w  snarkos             [.] tokio::runtime::queue::Steal<T>::steal_into
+   15.05%     0.00%  tokio-runtime-w  [unknown]           [.] 0xffffffff90a0007c
+   15.05%     0.00%  tokio-runtime-w  [unknown]           [.] 0xffffffff908a7281
+   12.77%     5.56%  tokio-runtime-w  snarkos             [.] parking_lot::condvar::Condvar::wait_until_internal
+    7.84%     7.84%  tokio-runtime-w  [unknown]           [k] 0xffffffff8fe70989
+    7.84%     0.00%  tokio-runtime-w  snarkos             [.] tokio::runtime::thread_pool::worker::Context::run_task
+    7.84%     0.00%  tokio-runtime-w  snarkos             [.] parking_lot::condvar::Condvar::notify_one_slow
+    7.84%     0.00%  tokio-runtime-w  [vdso]              [.] __vdso_clock_gettime
+    7.84%     0.00%  tokio-runtime-w  [unknown]           [.] 0xffffffff8ff36555
+    7.84%     0.00%  tokio-runtime-w  [unknown]           [.] 0xffffffff8ff35f45
+    7.84%     0.00%  tokio-runtime-w  [unknown]           [.] 0xffffffff8ff2d1fc
+    7.58%     0.06%  tokio-runtime-w  snarkos             [.] tokio::io::driver::Driver::turn
+    7.52%     0.00%  tokio-runtime-w  libc-2.33.so        [.] epoll_wait
+    7.52%     7.52%  tokio-runtime-w  [unknown]           [k] 0xffffffff90a00d70
+    7.22%     7.22%  tokio-runtime-w  [unknown]           [k] 0xffffffff8ff43720
+    7.22%     0.00%  tokio-runtime-w  libc-2.33.so        [.] syscall
+    5.19%     5.19%  tokio-runtime-w  libc-2.33.so        [.] __sched_yield
+    5.19%     0.00%  tokio-runtime-w  libc-2.33.so        [.] __GI___sched_yield (inlined)
     0.00%     0.00%  tokio-runtime-w  libc-2.33.so        [.] __libc_disable_asynccancel

Surprisingly, after around 30 minutes the nodes just decided to unblock itself and resumed regular work :eyes:; sadly, I missed the exact moment and don't know what happened at the very beginning, but I presume it was some disconnect that triggered it (it was at ~3 connections last time I checked before it fixed itself).


I built davinci in remote server, and it's always failed connect when I browse it

when I browse davinci with remote server's IP(<IP_ADDRESS>), such as http://<IP_ADDRESS>:8080, it will show "curl: (7) Failed connect to <IP_ADDRESS>:8080; 拒绝连接";
I changed the server.address=<IP_ADDRESS>, and there was a error when I start the server

我也是这样，我想远程连接，可是连接不上。到时候把davinci部署好了总不能让别人都来机房看吧....请问你解决这个问题了吗？

最近没有在弄这个

set server.address=<IP_ADDRESS>


noremap not working?

Hey,
I just installed the plugin and tried using my usual vim remaps for COLEMAK and it seems they dont have any effect. Noremap seems to be in the codemirror file, but im not sure whether its usable. Vim is enabled, so is the plugin, it loads without errors in the developer console, .obsidian.vimrc is in the vault root and the config works in vim without issue.
Here are the contents of .obsidian.vimrc:
" COLEMAK
noremap i l
noremap l i
noremap n j
noremap j n
noremap e k
noremap k e
noremap I L
noremap L I
noremap N J
noremap J N
noremap E K
noremap K E

The goal is for hjkl to be remaped to hnei, e to k, i to l and n to j in all modes but insert. Can you help me out?

This command is not part of the default CodeMirror Ex commands, but in a very nice timing @nadavspi posted a PR to add it this weekend :)
https://github.com/esm7/obsidian-vimrc-support/pull/13
(merged it a few mins ago)

This command is not part of the default CodeMirror Ex commands, but in a very nice timing @nadavspi posted a PR to add it this weekend :)
https://github.com/esm7/obsidian-vimrc-support/pull/13
(merged it a few mins ago)


Open the specified settings item after click open button

I had used this extension with Company network, and found some issue about downloading packages, I made some small improvements for this extension:

Open settings with configuraion ID, according to https://github.com/microsoft/vscode/issues/84324.
Remove empty command to avoid error when click the status bar during download package.

@microsoft-github-policy-service agree


Setup script shouldn't change working directory

Currently, running the setup script with feature zsh enabled causes а new shell session to be started in the scripts folder. This could be confusing; it should instead be started in the same directory as the setup script was run from.

Blocked by #193.

Resolved in dd091cb4a5f276071b55c5386aeee68f95db9cc9.


Path negotiation commands

This command will look for existing clients, connections, and channels that link any pair of chains on the relayer and create a Path or Paths in the config to mirror that. Optionally, if some primitives don't exist, then names for them are generated and those primitives are created. This may be a number of different commands as the number of options here will grow quickly.

Blocked on: https://github.com/cosmos/cosmos-sdk/issues/5829
Blocked on: https://github.com/cosmos/cosmos-sdk/issues/5617

So I managed to pull this off manually as follows:

rly query clients <src-chain>, find the entry with the chain_id of the <dst-chain>, and grab that client id. now we have the <src-client-id>
rly query client-connections <src-chain> <src-client-id> to get the connection id. now we have the <src-conn-id>.
rly query connection <src-chain> <src-conn-id> to get the destination chains client id and connection id. now we have <dest-client-id> and <dest-conn-id>
rly query channels <src-chain>, find the one that has the <src-conn-id> in the connection_hops, and from there get the destination channel id.
rly query channels <dst-chain>, find the one that has the <dst-conn-id> in the connection_hops, and from there get the source channel id

Now we have the client/conn/channel ids for both src and dst and can make the path manually. Whew!
Obviously it would be amazing if one command could just get us what we wanted, but in the meantime this "works". One thing I found weird though is I can't find channel ids directly on the chain, only the counterparty channel id. Is that blocked on something upstream, or can we fix it here?
Given we can get all this, we could consider implementing it as a command that just does multiple queries as necessary to get the data we want to compose the path, and then possibly don't need to block on the upstream changes? Not sure how difficult those upstream changes will be.

Obviously it would be amazing if one command could just get us what we wanted, but in the meantime this "works". One thing I found weird though is I can't find channel ids directly on the chain, only the counterparty channel id. Is that blocked on something upstream, or can we fix it here?

See the linked issues.

Given we can get all this, we could consider implementing it as a command that just does multiple queries as necessary to get the data we want to compose the path, and then possibly don't need to block on the upstream changes? Not sure how difficult those upstream changes will be.

Should be pretty easy. Should have this working mid-week next week.

And this has taken a bit longer than expected but I have a good deal of this functionality in the relayer. Once #104 lands rly paths gen will try to reuse any existing primitives automagically. Previously, I've already added an experimental rly paths find command that will find all paths between all configured chains. It works pretty well on the 4 node testnet I've been working with locally.
This functionality should be developed further. I will leave this issue open for that work.

Closing in favor of new issue about path command.