Add flag of keepGraphOptimize

Add flag of keepGraphOptimize. If true, during inference, and initial warm up in the first inference will be needed, which cost ~3.6s latency. With keepGraphOptimize, the rest of the latencies are reduced. This applies to only resnet18 on gpu; the other models are not much affected.
https://github.com/deepjavalibrary/djl/pull/2153
https://github.com/deepjavalibrary/djl/issues/2151

Codecov Report
Base: 72.08% // Head: 71.45% // Decreases project coverage by -0.63% :warning:

Coverage data is based on head (0373713) compared to base (bb5073f).
Patch coverage: 72.03% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2156      +/-   ##
============================================
- Coverage     72.08%   71.45%   -0.64%
- Complexity     5126     6334    +1208
============================================
  Files           473      626     +153
  Lines         21970    27965    +5995
  Branches       2351     3015     +664
============================================
+ Hits          15838    19982    +4144
- Misses         4925     6515    +1590
- Partials       1207     1468     +261

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...modality/cv/translator/wrapper/FileTranslator.java
44.44% <ø> (ø)

...y/cv/translator/wrapper/InputStreamTranslator.java
44.44% <ø> (ø)

... and 560 more

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

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


Add dependence from coveralls task to codeCoverageReport

This missing dependence was exposed by enabling parallel builds, which allowed for the coveralls task to run before codeCoverageReport.

Hrm, for some reason coveralls didn't post back a comment on this PR, but the upload of the coverage report worked: https://coveralls.io/builds/45644152


Generate API docs

Proposed changes
Problem: We want to have our CRDs documented
Solution: Use gen-crd-api-reference-docs to automatically generate the
docs
Closes #1754
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto main
[ ] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

@nginxinc/nginx-docs I need your help to make the new page look good 😅

@Jcahilltorre @travisamartin Any thoughts on the formatting issues for this generated API documentation?
I think there's some examples without these formatting issues in NGINX Management Suite, which are OpenAPI based IIRC.

The issue with this structure is it's just in alphabetical order. I want to easily be able to delineate between the different policies and their sub-fields. Should be separated by policy.

Turns out there was a missing tag for the types, they're now showing correctly by policy


fix: release "Allow URL class object as an argument for fetch()" #1696

Purpose
Dummy change with fix: commit to trigger the release of https://github.com/node-fetch/node-fetch/pull/1696
Changes
Swapped RequestInfo | URL around to URL | RequestInfo 🤷♂
Additional information
CI fail is irrelevant, same as before

@jimmywarting would be great to release this, thank you :)

Perhaps @LinusU could take a look, please?

I'm sorry to ping, @jimmywarting but I have a weekly reminder to myself to check on the status of this PR, hope this small but important change can be released soon, thank you!


:pushpin: Update cacheops, filter, and Django consumption

Addresses #39
Initial import error was:
netbox-worker_1  |   File "/usr/local/lib/python3.7/site-packages/cacheops/query.py", line 17, in <module>
netbox-worker_1  |     from django.db.models.sql.datastructures import EmptyResultSet
netbox-worker_1  | ImportError: cannot import name 'EmptyResultSet' from 'django.db.models.sql.datastructures' (/usr/local/lib/python3.7/site-packages/django/db/models/sql/datastructures.py)

Original fix attempt was to merge the upstream release 2.8.9 (released yesterday) into our vapor-ware/develop. Resolved merge conflicts and still the error persisted. Instead, I decided to just start bumping dependencies.
After bumping the django-cacheops package to the latest version, the next import error was:
netbox-worker_1  |   File "/usr/local/lib/python3.7/site-packages/django_filters/utils.py", line 10, in <module>
netbox-worker_1  |     from django.db.models.fields import FieldDoesNotExist
netbox-worker_1  | ImportError: cannot import name 'FieldDoesNotExist' from 'django.db.models.fields' (/usr/local/lib/python3.7/site-packages/django/db/models/fields/__init__.py)

After bumping the django-filter package to the latest version, the next import error was:
netbox-worker_1  |   File "/opt/netbox/netbox/utilities/forms.py", line 11, in <module>
netbox-worker_1  |     from django.contrib.postgres.forms.jsonb import JSONField as _JSONField, InvalidJSONInput
netbox-worker_1  | ImportError: cannot import name 'InvalidJSONInput' from 'django.contrib.postgres.forms.jsonb' (/usr/local/lib/python3.7/site-packages/django/contrib/postgres/forms/jsonb.py)

This one was a bit different, as it involved the netbox import of django code. The django version is pinned pretty hard at Django>=3.0,<3.1 so instead of opening that can of worms, I found the django commit that "fixed" (🤣 ) the referenced import file. As you can see, there was an exception defined that simply passed when called. So I just put that exception in jsonb.py and removed the import.
Luckily it stopped there, and netbox is working for me locally. All initializers run successfully, and I recursively opened EVERY link in the localhost:8000/api route. All routes are responding with no error.
The one thing I'm uncertain about, is whether something broke with the virtual-circuit plugin. Navigating to http://localhost:8000/api/plugins/installed-plugins/ returns an empty list, as if the virtual-circuit plugin is not registering successfully.
But, from what I can tell, this unblocks development with netbox. I don't know why we're getting bit by dependency changes that happened 3 months ago.
How can we avoid this in the future? This came out of nowhere despite most of our dependencies being pinned.

django-cacheops 5.0.1 July 9
django-filter 2.3.0 on June 5

Please, lets not do this - lets just update to the latest version of netbox!

I can help with that this week if it's blocking (and document the process) but this out of band dep changes will just keep making syncs from upstream more difficult


expected tensor [5798 x 100] and src [600 x 100]

这个是什么错误 好奇怪的错误
While copying the parameter named encoder.embedding.weight, whose dimensions in the model are torch.Size([5798, 100]) and whose dimensions in the checkpoint are torch.Size([600, 100]), ...
inconsistent tensor size, expected tensor [5798 x 100] and src [600 x 100] to have the same number of elements, but got 579800 and 60000 elements respectively

@yanwii  我猜测 是 params.pkl原有的训练是600 但是这次想训练 5798 导致错误
但是 如何初始化一个空白的 params.pkl呢？


Reward indexing

Hello,
I'm interested in indexing the detail of validators rewards returned by the /eth/v1/beacon/rewards/attestations/:epoch endpoint.
Is it planned for chaind to do so ?
If so, I'd be willing to help.

This is not something that we are looking to add, but if you wish to do so I'd accept a PR.
Due to the size of the database already, it makes sense for the information to be added to existing tables.  The lowest-level table would be t_validator_epoch_summaries, which is rolled up to t_validator_day_summaries.  It may also be useful to add this information to t_epoch_summaries as well.
I'd be happy to provide guidance on how to add this functionality if necessary.

I'd be happy to provide guidance on how to add this functionality if necessary.

Sure, it'd be appreciated !

Can you tell what's your take on how to proceed ?
I've thought about querying the node, but it could be quite slow and necessary if all necessary information to compute detailed reward is already in the chaind db.
The way I'm trying to achieve this is by trying to compute them following the spec.


Added missing <immer/config.hpp> and  includes

The former needed for the new exception macros and the latter for the
usage of std::size_t.
Minor include ordering changes too.

Codecov Report

Merging #164 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #164   +/-   ##
=======================================
  Coverage   94.12%   94.12%
=======================================
  Files         103      103
  Lines       10011    10011
=======================================
  Hits         9423     9423
  Misses        588      588

Impacted Files
Coverage Δ

immer/array.hpp
100.00% <ø> (ø)

immer/array_transient.hpp
100.00% <ø> (ø)

immer/box.hpp
100.00% <ø> (ø)

immer/detail/arrays/no_capacity.hpp
89.85% <ø> (ø)

immer/detail/arrays/node.hpp
91.89% <ø> (ø)

immer/detail/arrays/with_capacity.hpp
88.72% <ø> (ø)

immer/detail/combine_standard_layout.hpp
100.00% <ø> (ø)

immer/detail/hamts/bits.hpp
100.00% <ø> (ø)

immer/detail/hamts/node.hpp
96.86% <ø> (ø)

immer/detail/rbts/node.hpp
95.80% <ø> (ø)

... and 9 more

Continue to review full report at Codecov.

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

LGTM, thanks! :+1:


master branch is now DC/OS 2.2.0-dev

https://jira.d2iq.com/browse/D2IQ-68316 - Set Master branch to 2.2.0-dev

@mesosphere-mergebot label  ready for review


Release 1.9.5: Coverage map is blank on resources

In the newest release on WWW, the coverage map has been pulled out of the coverage metadata tab, but it's blank on the page. Tested just now on www in Chrome on my Mac.

@horsburgh Seems to be working on other browsers (FireFox and MS Edge).

I was unable to reproduce this error. @pkdash, could you test on a Mac computer and report the error back to me?

Mac, Chrome, Little Bear resource:

@horsburgh - stale browser cache maybe?

Well... It's working now. I didn't specifically clear my cache, but let's chalk it up to that. Crisis averted!


Integrate Atom Linter

This gives a nice overview of all build errors.
Tasks left before this is mergable

[x] implement the primary integration
[x] specs
[ ] scroll the build output to the matched error: There doesn't seem to be any support in the linter for this. Issue created: https://github.com/steelbrain/linter/issues/1106. Will fix that in a separate PR )
[x] document the 'message' entry in error matcher
[x] document that the linter might be cool to have installed

Fixes #346

Just a side note, currently it'll only work if the users have linter already installed, but in case they don't (which is pretty rare), we have atom-package-deps


Fix scratch immortality

eq doesn't work on strings

Whoops! That must've been a buffer comparison at some point. Good catch, thanks!


Undo/redo does not scroll to selection

Description
This is one of the most frustrating things for me when using JupyterLab and it prevents me from working productively with larger files - see https://github.com/jupyterlab/jupyterlab/issues/15102
Reproduce
See  https://github.com/jupyterlab/jupyterlab/issues/15102
Expected behavior
Editor scrolls on undo/redo if the new selection is outside of the viewport.
Context
This was just fixed in y-codemirror.next: https://github.com/yjs/y-codemirror.next/pull/34 but jupyter_ydoc does not use that and has its own integration of undo manager.

It looks like we need something like this._undoManager.on('stack-item-popped', this._onStackItemPopped). For now I would propose to add it for text documents as in notebooks the behaviour is more complex.
Then, I wonder how is selection set currently if _onStackItemPopped is not implemented?

This is one of the most frustrating things for me when using JupyterLab and it prevents me from working productively with larger files

Well, I'm not alone: https://stackoverflow.com/questions/78818736/follow-changes-with-the-mouse-cursor-when-pressing-ctrl-z-z-undo-redo-in-jup


azdev test gives usage error when given module and test

What is wrong with this???:
(env36) C:\Users\wilx\azure-cli>
azdev test --test test_storage_blob_batch_copy --modules storage --live

usage error: [--test TESTS [TESTS ...]] [--src-file FILENAME] | --modules MODULES [MODULES ...]

Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)

You don't need --module since it automatically figures out that's in the storage module. Technically, what your command is saying is: "Hey, run all the storage tests, and this other storage test again."  There may be cases where you would want to do that, to say run all the network tests and then some specific non-network tests, but I was trying to avoid exactly the situation you posted.

Makes sense. However, it's still a problem that the error message gives no real indication that this is the case. In addition, this is change in the behavior in how we used to use run_tests and should be documented better.

This has been fixed as --module has been removed.


Translate Section 2: Function Components

Please feel free to ask to work on this issue and I'll assign it to you 😄
https://github.com/typescript-cheatsheets/react#function-components

If you don't mind, I have assigned this one to myself. :)
I'd like to keep the flow.
Thanks.


[WIP/TESTING] Revert selectively

This reverts commit f9ffd76e1b6982666d2ac0556fde441357d25341.


fix(autoware_traffic_light_occlusion_predictor): fix functionConst

Description
This is a fix based on cppcheck functionConst warnings.
perception/autoware_traffic_light_occlusion_predictor/src/occlusion_predictor.hpp:74:8: style: inconclusive: Technically the member function 'autoware::traffic_light::CloudOcclusionPredictor::filterCloud' can be const. [functionConst]
  void filterCloud(
       ^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@miursh @tzhong518 kindly ping


⚠ Bofrost.de has degraded performance

In 453cd0b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1632 ms

Resolved: Bofrost.de performance has improved in 9325018 after 22 minutes.


SRL in SENNA

Just a quick raincheck. Is there a reason why semantic role labeling (SRL) was left out in the nltk.tag.senna.py? Is someone working on adding it?

@aboSamoor – would you please comment on this?

Srl could be added to the module, but I am not working on it.

Thanks @aboSamoor


Errors were encountered while processing:  crowdsec-firewall-bouncer-iptables

What happened?
doing apt update, it fails
What did you expect to happen?
for it not to fail
How can we reproduce it (as minimally and precisely as possible)?
sudo aptitude update && sudo aptitude safe-upgrade -y
Anything else we need to know?
❯ sudo apt install crowdsec-firewall-bouncer-iptables
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following NEW packages will be installed:
  crowdsec-firewall-bouncer-iptables
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 3271 kB of archives.
After this operation, 12.5 MB of additional disk space will be used.
Get:1 https://packagecloud.io/crowdsec/crowdsec/ubuntu jammy/main arm64 crowdsec-firewall-bouncer-iptables arm64 0.0.29 [3271 kB]
Fetched 3271 kB in 1s (4277 kB/s)
Selecting previously unselected package crowdsec-firewall-bouncer-iptables.
(Reading database ... 137434 files and directories currently installed.)
Preparing to unpack .../crowdsec-firewall-bouncer-iptables_0.0.29_arm64.deb ...
Unpacking crowdsec-firewall-bouncer-iptables (0.0.29) ...
Setting up crowdsec-firewall-bouncer-iptables (0.0.29) ...
cscli/crowdsec is present, generating API key
API Key successfully created
Created symlink /etc/systemd/system/multi-user.target.wants/crowdsec-firewall-bouncer.service → /etc/systemd/system/crowdsec-firewall-bouncer.service.
Job for crowdsec-firewall-bouncer.service failed because the control process exited with error code.
See "systemctl status crowdsec-firewall-bouncer.service" and "journalctl -xeu crowdsec-firewall-bouncer.service" for det
ails.
dpkg: error processing package crowdsec-firewall-bouncer-iptables (--configure):
 installed crowdsec-firewall-bouncer-iptables package post-installation script subprocess returned error exit status 1
Errors were encountered while processing:
 crowdsec-firewall-bouncer-iptables
needrestart is being skipped since dpkg has failed
E: Sub-process /usr/bin/dpkg returned an error code (1)

version
remediation component version:

version: v0.0.29-debian-pragmatic-arm64-9594360ca83272d5c15ed1c343a6d96868d83cbc
BuildDate: 2024-08-16_15:52:03
GoVersion: 1.22.2
Platform: linux

crowdsec version
crowdsec version:

❯ crowdsec --version
version: v1.6.2-debian-pragmatic-arm64-16bfab86
Codename: alphaga
BuildDate: 2024-05-31_09:15:42
GoVersion: 1.22.2
Platform: linux
libre2: C++
User-Agent: crowdsec/v1.6.2-debian-pragmatic-arm64-16bfab86-linux
Constraint_parser: >= 1.0, <= 3.0
Constraint_scenario: >= 1.0, <= 3.0
Constraint_api: v1
Constraint_acquis: >= 1.0, < 2.0

OS version

❯ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.4 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.4 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

❯ uname -a
Linux uptimekuma 6.5.0-1027-oracle #27~22.04.1-Ubuntu SMP Sat Jul 20 13:14:31 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

sudo journalctl -xeu crowdsec-firewall-bouncer.service
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ A start job for unit crowdsec-firewall-bouncer.service has finished with a failure.
░░
░░ The job identifier is 3751013 and the job result is failed.
Aug 17 06:13:37 uptimekuma systemd[1]: crowdsec-firewall-bouncer.service: Scheduled restart job, restart counter is at 1.
░░ Subject: Automatic restarting of a unit has been scheduled
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ Automatic restarting of the unit crowdsec-firewall-bouncer.service has been scheduled, as the result for
░░ the configured Restart= setting for the unit.
Aug 17 06:13:37 uptimekuma systemd[1]: Stopped The firewall bouncer for CrowdSec.
░░ Subject: A stop job for unit crowdsec-firewall-bouncer.service has finished
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ A stop job for unit crowdsec-firewall-bouncer.service has finished.
░░
░░ The job identifier is 3751101 and the job result is done.
Aug 17 06:13:37 uptimekuma systemd[1]: Starting The firewall bouncer for CrowdSec...
░░ Subject: A start job for unit crowdsec-firewall-bouncer.service has begun execution
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ A start job for unit crowdsec-firewall-bouncer.service has begun execution.
░░
░░ The job identifier is 3751101.

experiencing the same issue

Hello,
Can you provide the content of /var/log/crowdsec-firewall-bouncer.log ?

It could be that you declined the systemd service file update and the files have now moved?
Configuration file '/etc/systemd/system/crowdsec-firewall-bouncer.service'
==> Modified (by you or by a script) since installation.
==> Package distributor has shipped an updated version.
What would you like to do about it ?  Your options are:
Y or I  : install the package maintainer's version
N or O  : keep your currently-installed version
D     : show the differences between the versions
Z     : start a shell to examine the situation
The default action is to keep your current version.
*** crowdsec-firewall-bouncer.service (Y/I/N/O/D/Z) [default=N] ? D
-ExecStart=/usr/sbin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml
-ExecStartPre=/usr/sbin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -t
+ExecStart=/usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml
+ExecStartPre=/usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -t

After experiencing the error that modem7 encountered I uninstalled both crowdsec and the firewall bouncer hoping the reinstall would help however the crowdsec installation goes fine but the firewall bouncer's install doesn't go well
ubuntu@nimbus2000:~$ sudo apt install crowdsec-firewall-bouncer-iptables Reading package lists... Done Building dependency tree... Done Reading state information... Done The following additional packages will be installed:   ipset libipset13 The following NEW packages will be installed:   crowdsec-firewall-bouncer-iptables ipset libipset13 0 upgraded, 3 newly installed, 0 to remove and 0 not upgraded. Need to get 0 B/3367 kB of archives. After this operation, 12.9 MB of additional disk space will be used. Do you want to continue? [Y/n] y Selecting previously unselected package libipset13:arm64. (Reading database ... 101850 files and directories currently installed.) Preparing to unpack .../libipset13_7.15-1build1_arm64.deb ... Unpacking libipset13:arm64 (7.15-1build1) ... Selecting previously unselected package ipset. Preparing to unpack .../ipset_7.15-1build1_arm64.deb ... Unpacking ipset (7.15-1build1) ... Selecting previously unselected package crowdsec-firewall-bouncer-iptables. Preparing to unpack .../crowdsec-firewall-bouncer-iptables_0.0.29_arm64.deb ... Unpacking crowdsec-firewall-bouncer-iptables (0.0.29) ... Setting up libipset13:arm64 (7.15-1build1) ... Setting up ipset (7.15-1build1) ... Setting up crowdsec-firewall-bouncer-iptables (0.0.29) ... FATA[0000] unable to read config file: while reading yaml file: open /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml: no such file or directory  Failed to get unit file state for crowdsec-firewall-bouncer.service: No such file or directory Unit crowdsec-firewall-bouncer.service does not exist, proceeding anyway. Failed to enable unit: Unit file crowdsec-firewall-bouncer.service does not exist. dpkg: error processing package crowdsec-firewall-bouncer-iptables (--configure):  installed crowdsec-firewall-bouncer-iptables package post-installation script subprocess returned error exit status 1 Processing triggers for man-db (2.10.2-1) ... Processing triggers for libc-bin (2.35-0ubuntu3.8) ... Errors were encountered while processing:  crowdsec-firewall-bouncer-iptables needrestart is being skipped since dpkg has failed E: Sub-process /usr/bin/dpkg returned an error code (1)

Is there any way to completely remove both crowdsec and firewall bouncer so that the next install can be treated like a fresh install.

Is there any way to completely remove both crowdsec and firewall bouncer so that the next install can be treated like a fresh install.

With the apt package manager to "fully" uninstall, you use the purge command, using "uninstall" command tell apt that the configuration files will still exists on a apt install

thank you so much!! that fixed it!

It could be that you declined the systemd service file update and the files have now moved?
Configuration file '/etc/systemd/system/crowdsec-firewall-bouncer.service'
==> Modified (by you or by a script) since installation.
==> Package distributor has shipped an updated version.
What would you like to do about it ?  Your options are:
Y or I  : install the package maintainer's version
N or O  : keep your currently-installed version
D     : show the differences between the versions
Z     : start a shell to examine the situation
The default action is to keep your current version.
*** crowdsec-firewall-bouncer.service (Y/I/N/O/D/Z) [default=N] ? D
-ExecStart=/usr/sbin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml
-ExecStartPre=/usr/sbin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -t
+ExecStart=/usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml
+ExecStartPre=/usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -t

Could you also run systemctl cat crowdsec-firewall-bouncer.service
It seems there lots of binaries everywhere but also symlinked
07:51:26 root@laurencejones ~ → /usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -V
version: v0.0.29-debian-pragmatic-amd64-9594360ca83272d5c15ed1c343a6d96868d83cbc
BuildDate: 2024-08-16_15:51:54
GoVersion: 1.22.2
Platform: linux
07:51:29 root@laurencejones ~ → /usr/sbin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -V
version: v0.0.29-debian-pragmatic-amd64-9594360ca83272d5c15ed1c343a6d96868d83cbc
BuildDate: 2024-08-16_15:51:54
GoVersion: 1.22.2
Platform: linux

That's what I thought first, but I've done apt purge a few times since, but no go.
❯ sudo systemctl cat crowdsec-firewall-bouncer.service
# /etc/systemd/system/crowdsec-firewall-bouncer.service
[Unit]
Description=The firewall bouncer for CrowdSec
After=syslog.target network.target remote-fs.target nss-lookup.target crowdsec.service

[Service]
Type=notify
ExecStart=/usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml
ExecStartPre=/usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -t
ExecStartPost=/bin/sleep 0.1
Restart=always
RestartSec=10
LimitNOFILE=65536
# don't send a termination signal to the children processes,
# because the iptables backend needs to run ipset multiple times to properly shutdown
KillMode=mixed

[Install]
WantedBy=multi-user.target

crowdsec-firewall-bouncer.zip

It seems the configured LAPI is <IP_ADDRESS>:8080 is that correct?
if so it seems it not running or there a firewall connection in the way?
time="2024-08-19T09:32:52+01:00" level=error msg="auth-api: auth with api key failed return nil response, error: dial tcp <IP_ADDRESS>:8080: connect: connection refused"
time="2024-08-19T09:32:52+01:00" level=error msg="Get \"http://<IP_ADDRESS>:8080/v1/decisions/stream?startup=true\": dial tcp <IP_ADDRESS>:8080: connect: connection refused"

❯ sudo service crowdsec status
● crowdsec.service - Crowdsec agent
     Loaded: loaded (/lib/systemd/system/crowdsec.service; enabled; vendor preset: enabled)
     Active: active (running) since Mon 2024-08-19 09:41:22 BST; 24s ago
   Main PID: 3376846 (crowdsec)
      Tasks: 20 (limit: 28690)
     Memory: 123.1M
        CPU: 5.017s
     CGroup: /system.slice/crowdsec.service
             ├─3376846 /usr/bin/crowdsec -c /etc/crowdsec/config.yaml
             └─3376863 /usr/lib/crowdsec/plugins/notification-slack

Aug 19 09:41:18 uptimekuma systemd[1]: Starting Crowdsec agent...
Aug 19 09:41:22 uptimekuma systemd[1]: Started Crowdsec agent.
❯ sudo service crowdsec-firewall-bouncer status
● crowdsec-firewall-bouncer.service - The firewall bouncer for CrowdSec
     Loaded: loaded (/etc/systemd/system/crowdsec-firewall-bouncer.service; enabled; vendor preset: enabled)
     Active: active (running) since Mon 2024-08-19 09:41:33 BST; 17s ago
   Main PID: 3377111 (crowdsec-firewa)
      Tasks: 12 (limit: 28690)
     Memory: 21.4M
        CPU: 18.748s
     CGroup: /system.slice/crowdsec-firewall-bouncer.service
             ├─3377111 /usr/bin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml
             └─3398425 /usr/sbin/ipset -exist add crowdsec-blacklists <IP_ADDRESS> timeout 86388

Aug 19 09:41:29 uptimekuma systemd[1]: Starting The firewall bouncer for CrowdSec...
Aug 19 09:41:33 uptimekuma systemd[1]: Started The firewall bouncer for CrowdSec.


v1.3.0: Bad Request: Block not in forkChoice - Error producing SyncCommitteeContribution

Describe the bug
During sync committee duties, there are times when blocks are not in forkChoice in time to produce SyncCommitteeContributions. There doesn't seem to be any correlation with time and if it gets better over time. It is very sporadic when it happens.
Jan-26 18:31:43.150[]                error: Error on SyncCommitteeContribution slot=5661156, index=1 Bad Request: Block not in forkChoice - Error producing SyncCommitteeContribution
Error: Bad Request: Block not in forkChoice - Error producing SyncCommitteeContribution
    at HttpClient.requestWithBody (file:///usr/app/packages/api/src/utils/client/httpClient.ts:258:15)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

Expected behavior
We should hit SyncCommitteeContribution every time. Blocks not in forkChoice should not be happening.

could be the same root cause to https://github.com/ChainSafe/lodestar/issues/5098#issuecomment-1427504993
when there are multiple beacon node urls, SyncComittee block root is produced and published to one node while SyncCommitteeContribution is produced from another node, and the other node hasn't seen the block yet

For greater context, restarting the VC fixed this problem, but unsure of what actually caused it. Could it have been using or receiving old duties? I don't have the exact steps to reproduce this, but there was a manual restart on the BN/EL server which the VC was trying to consistently connect to it during that time. More of this in the private discord thread here: https://discord.com/channels/593655374469660673/1068138020127125505/1068245669468454912

to be clear, this is just a log on validator with SyncCommitteeContribution and not related to the miss of SyncCommitteeSignature. This should rarely happen (unlike attestation) because it's most likely that beacon block should come at 8s after the slot. And this happens randomly per slot, restarting the validator does not help.

It was a log that showed and was confirmed to be missing via beaconcha.in. So during this period of time, we missed all sync committee contributions until we restarted the VC.

Just looked at the incident on Jan 25/Jan 26:

It was caused by bad performance of beacon nodes
Also the rescue/fallback node was configured with subscribe_all_subnets=false at that time so we lacked some subnet peers on this node ( we only configured that to true after the incident): if submitPoolSyncCommitteeSignatures failed on main bn it could be failed in fallback/rescue node too since we lack subnet peers. Below metric was from the rescue/fallback node

The log in this issue is only related to SyncCommitteeContribution (not SyncCommitteeMessage), although it's not the root cause of missed SyncCommitteeMessage it could improve the issue: vc as a validator can aggregate SyncCommitteeMessages and publish to the network itself
I'm not sure how restaring vc fixed the issue but the rescue node was switched to subscribe_all_subnets=true around that time too

"Block not in forkChoice" error should be fixed with #5157


use IAO curation status tags

@brianjlowe this is something we can use in the fieldbook to see how sure we are about a given class semantics. You can maybe allow users to filter those classes out

See the the instances of curation status specification

not needed anymore


lib refactor

Long story short

0.3.0 release PR
Checks

[ ] I have run nox tests and all pipelines have passed.
[ ] I have written tests according to the code i added, modified or deleted.

Related issues

https://github.com/Bungie-net/api?tab=readme-ov-file#2024-07-10-update-for-the-final-shape


Fix duration in cli.create

Fix duration in cli client.create
Cli: Echo URL used for requests on stderr

no release notes as we already mention the duration fixes and there was no new release


Fix: addon-manager image not overrided.

Fix: https://issues.redhat.com/browse/ACM-6925

/assign @zhujian7

/assign @JakobGray

/unassign @JakobGray

/assign @dislbenn
Please give an /approve on this PR, thanks!

/lgtm


Implement startOfMonth & endOfMonth methods

I want to take it ..is there is any dependancy?

I want to take it ..is there is any dependancy?

Go ahead, no dep


Add in SSO support for Kale

Related Story: #37

I'm not quite sure how this should play into the user interface, but we can and probably should directly launch into the user's browser, instead of forcing them to cut and paste the url.
(.browse (java.awt.Desktop/getDesktop)
         (java.net.URI/create "https://login.ng.bluemix.net/UAALoginServerWAR/passcode"))

The challenge, as I see it, is telling the user what is going to happen (we're going to throw you into your browser now) and what they need to do next (grab that token that is displayed and get it entered into kale).
Also, I don't think we need to treat that token as a password type-in (that is: we can and should echo the type-in of the token).

In general, this looks great!
If opening the browser fails, I think we'd like to say a bit more about what the user needs to do. The current behavior looks like this (I ran in a Docker container to force the failure):
kale login -s
Endpoint (default: https://api.ng.bluemix.net)?
Using endpoint 'https://api.ng.bluemix.net'

To log in, you will need to provide a passcode from:
https://login.ng.bluemix.net/UAALoginServerWAR/passcode

If you already have a passcode, type it in now; otherwise
press ENTER to automatically open a browser to the URL:
Opening browser to URL...
Unable to open browser.
Passcode? xwnwST
Logging in...
Using org<EMAIL_ADDRESS>
I think the Unable to open browser. message needs additional text telling the user they need to open that URL themselves, maybe even pointing out that they can do that on a different machine.

Sure, done.


Fixing custom ratelimit sample

Purpose
fixes custom ratelimit sample

Codecov Report
Patch coverage has no change and project coverage change: +54.10% :tada:

Comparison is base (df6ffd2) 26.00% compared to head (9e7500c) 80.10%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1636       +/-   ##
===========================================
+ Coverage   26.00%   80.10%   +54.10%
===========================================
  Files         264        9      -255
  Lines       30656      548    -30108
  Branches     5088      246     -4842
===========================================
- Hits         7972      439     -7533
+ Misses      22412      109    -22303
+ Partials      272        0      -272

Flag
Coverage Δ

adapter
?

common-controller
?

config-deployer-service
?

idp-domain-service
80.10% <ø> (ø)

router
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 255 files with indirect coverage changes

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


add fuse-master image, and update eacruntime version

Ⅰ. Describe what this PR does
add eac-master image, which separate from eac-fuse image.
Ⅱ. Does this pull request fix one issue?

fixes #2603
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
no
Ⅳ. Describe how to verify it
no
Ⅴ. Special notes for reviews

/lgtm
/approve


Implement JsonSchema

Implement the JsonSchema part of SwaggerGen with the currently working generator.

After discussing with William on Fri. Sept 20 - 3pm:

I will be tying the response bodies to definitions inside of the OpenAPI spec using JsonSchemas
Will will take care of updating our annotations to support being passed in more complete response definitions

To consider:

Adding POJOs as possible definition standard on top of schemas
Ensuring request input properly maps over to definitions as well, and annotation supports it


clipgrab: use ffmpeg4

Motivation for this change
See migrate away from ffmpeg_3 #120705.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

1 package built:

clipgrab

Result of nixpkgs-review pr 122751 at b051d3c0 run on x86_64-linux 1

1 package built successfully:

clipgrab

3 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/applications/video/clipgrab/default.nix:45:3:
   |
45 |   meta = with lib; {
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/video/clipgrab/default.nix:39:3:
   |
39 |   installPhase = ''
   |   ^

warning: unnecessary-parallel-building
Meson, CMake and qmake already set enableParallelBuilding = true by default so it is not necessary.
Near pkgs/applications/video/clipgrab/default.nix:27:3:
   |
27 |   enableParallelBuilding = true;
   |   ^


Package is using deprecated npm modules

> npm install @nodesecure/cli -g
npm WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs
npm WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs

Just updated the dependencies of the CLI, I also need to update scanner dependencies (npm published a lot of major recently). Anyway should be fixed next week.

Fixed with v2.2.0


running on 32bpp machine segfaults

Hello,
I got it built (had to tweak Makefile to remove link to blahblah711 library). I ran this on a machine in 32bpp mode and it segfaulted. I know it's only supposed to run 24bpp but segfaults can be a security issue IIRC. Thanks.
don@oysters:/tmp/fbg/examples$ ./quickstart
fbg_init: Cannot open '/dev/fb0'!
Segmentation fault (core dumped)
don@oysters:/tmp/fbg/examples$ sudo ./quickstart
[sudo] password for don:
fbg_init: '/dev/fb0' (1366x768 (1366x768 virtual) 32 bpp (8/16, 8/8, 8/0) (4227072 smen_len) 5504 line_length)
fbg_init: '/dev/fb0' Unsupported format (only 24 bits framebuffer is supported)!
Segmentation fault

Indeed, this happen because there is no checks in the examples for when the library fail to initialize, i will see to check the library initialization on all examples.

Fixed in last commit.


Slight Grammatical Error

The first step of part 3 says:
"... your break your system."


[GLib] apache-arrow-glib 15.0.2 failed to build

Describe the bug, including details regarding any error messages, version, and platform.
👋 trying to build the latest release, but run into some build issue. The error log is as below:

error build log
[16/101] clang++ -Iarrow-glib/libarrow-glib.1500.dylib.p -I. -I../c_glib -I/opt/homebrew/Cellar/brotli/1.1.0/include -I/opt/homebrew/Cellar/lz4/1.9.4/include -I/opt/homebrew/Cellar/zstd/1.5.5/include -I/opt/homebrew/Cellar/utf8proc/2.9.0/include -I/opt/homebrew/Cellar/apache-arrow/15.0.2/include -I/opt/homebrew/Cellar/pcre2/10.43/include -I/opt/homebrew/Cellar/glib/2.80.0/include -I/opt/homebrew/Cellar/glib/2.80.0/include/glib-2.0 -I/opt/homebrew/Cellar/glib/2.80.0/lib/glib-2.0/include -I/opt/homebrew/opt/gettext/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX14.sdk/usr/include/ffi -fdiagnostics-color=always -Wall -Winvalid-pch -std=c++17 -O3 -Wmissing-declarations -Wno-incompatible-function-pointer-types -DUTF8PROC_EXPORTS -MD -MQ arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o -MF arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o.d -o arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o -c ../c_glib/arrow-glib/composite-array.cpp
FAILED: arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o
clang++ -Iarrow-glib/libarrow-glib.1500.dylib.p -I. -I../c_glib -I/opt/homebrew/Cellar/brotli/1.1.0/include -I/opt/homebrew/Cellar/lz4/1.9.4/include -I/opt/homebrew/Cellar/zstd/1.5.5/include -I/opt/homebrew/Cellar/utf8proc/2.9.0/include -I/opt/homebrew/Cellar/apache-arrow/15.0.2/include -I/opt/homebrew/Cellar/pcre2/10.43/include -I/opt/homebrew/Cellar/glib/2.80.0/include -I/opt/homebrew/Cellar/glib/2.80.0/include/glib-2.0 -I/opt/homebrew/Cellar/glib/2.80.0/lib/glib-2.0/include -I/opt/homebrew/opt/gettext/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX14.sdk/usr/include/ffi -fdiagnostics-color=always -Wall -Winvalid-pch -std=c++17 -O3 -Wmissing-declarations -Wno-incompatible-function-pointer-types -DUTF8PROC_EXPORTS -MD -MQ arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o -MF arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o.d -o arrow-glib/libarrow-glib.1500.dylib.p/composite-array.cpp.o -c ../c_glib/arrow-glib/composite-array.cpp
../c_glib/arrow-glib/composite-array.cpp:613:10: error: cannot initialize return object of type 'const gint64 *' (aka 'const long *') with an rvalue of type 'const typename LargeListArray::offset_type *' (aka 'const long long *')
  return garrow_base_list_array_get_value_offsets<arrow::LargeListArray>(
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

full build log, https://github.com/Homebrew/homebrew-core/actions/runs/8344623050/job/22837685244
relates to Homebrew/homebrew-core#166595
Component(s)
GLib

The error seems to be:
   FAILED: arrow-glib/libarrow-glib.1500.dylib.p/array-builder.cpp.o
  clang++ -Iarrow-glib/libarrow-glib.1500.dylib.p -I. -I../c_glib -I/opt/homebrew/Cellar/apache-arrow/15.0.2/include -I/opt/homebrew/Cellar/pcre2/10.43/include -I/opt/homebrew/Cellar/glib/2.80.0/include -I/opt/homebrew/Cellar/glib/2.80.0/include/glib-2.0 -I/opt/homebrew/Cellar/glib/2.80.0/lib/glib-2.0/include -I/opt/homebrew/opt/gettext/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX13.sdk/usr/include/ffi -fdiagnostics-color=always -D_GLIBCXX_ASSERTIONS=1 -D_LIBCPP_ENABLE_ASSERTIONS=1 -Wall -Winvalid-pch -std=c++17 -O3 -Wmissing-declarations -MD -MQ arrow-glib/libarrow-glib.1500.dylib.p/array-builder.cpp.o -MF arrow-glib/libarrow-glib.1500.dylib.p/array-builder.cpp.o.d -o arrow-glib/libarrow-glib.1500.dylib.p/array-builder.cpp.o -c ../c_glib/arrow-glib/array-builder.cpp
  ../c_glib/arrow-glib/array-builder.cpp:5171:41: error: cannot initialize a parameter of type 'const int64_t *' (aka 'const long long *') with an lvalue of type 'const gint64 *' (aka 'const long *')
      return arrow_builder->AppendIndices(values, values_length, valid_bytes);
                                          ^~~~~~
  /opt/homebrew/Cellar/apache-arrow/15.0.2/include/arrow/array/builder_dict.h:691:39: note: passing argument to parameter 'values' here
    Status AppendIndices(const int64_t* values, int64_t length,
                                        ^
  ../c_glib/arrow-glib/array-builder.cpp:5405:41: error: cannot initialize a parameter of type 'const int64_t *' (aka 'const long long *') with an lvalue of type 'const gint64 *' (aka 'const long *')
      return arrow_builder->AppendIndices(values, values_length, valid_bytes);
                                          ^~~~~~
  /opt/homebrew/Cellar/apache-arrow/15.0.2/include/arrow/array/builder_dict.h:691:39: note: passing argument to parameter 'values' here
    Status AppendIndices(const int64_t* values, int64_t length,
                                        ^
  2 errors generated.

cc @kou

Hmm. It seems that 64-bit integer type provided by GLib (gint64) and C++ (int64_t) became incompatible...
I'll add explicit casts...

Hmm. It seems that 64-bit integer type provided by GLib (gint64) and C++ (int64_t) became incompatible...

This seems to be due to a misconfiguration on our end. Should be fixed by https://github.com/Homebrew/homebrew-core/pull/166473.
Nevertheless,

I'll add explicit casts...

is probably still a good idea. See https://github.com/Homebrew/homebrew-core/pull/166595#issuecomment-2011017711

Thanks for the info!
I didn't know the gint64 note:

Note that on platforms with more than one 64-bit standard integer type, gint64 and int64_t are not necessarily implemented by the same 64-bit integer type. For example, on a platform where both long and long long are 64-bit, it might be the case that one of those types is used for gint64 and the other is used for int64_t.

Issue resolved by pull request 40736
https://github.com/apache/arrow/pull/40736


Can't start advertising on iOS

I have done a little function to "open a peripheral". On android it work perfecly but on iOS, the function startAdvertising give an error like this : "Please turn on Bluetooth in the Settings". I don't know why because I have both the bluetooth on and the info.plist updated.
My function look like this:
export const openPeripheral = () => {
  setName("My phone");
  addService(PhoneServiceUuid);
  addCharacteristicToService(PhoneServiceUuid, DataCharacteristicUuid, 1|16, 2|8|16, '');
  startAdvertising(1);
}

I think I found where the bug occurs. In the BluetoothClient.swift, it's the beginning of the startAdvertising function.
    func startAdvertising(_ time: Int, resolve: @escaping RCTPromiseResolveBlock, rejecter reject: @escaping RCTPromiseRejectBlock) -> Void{
        if(manager.state != .poweredOn){
            alertJS("Bluetooth off")
            let error = NSError(domain: "", code: 100, userInfo: nil)
            reject("Bluetooth power off", "Please turn on Bluetooth in the settings.", error)
            return ;
        }
        ...

manager.state != .poweredOn is always true and throws an error. By commenting this section, the function works. It's weird because the checkBluetooth always responds when the Bluetooth is on.


iam-role cloudtrail events aren't returning anything.

I've been trying to create a policy which would notify me if a role has a managed policy attached.
Using this policy i don't seem to be able to filter the event or action my notify.
even hardcoding a filter that would be a sure fire still doesn't seem to work.
the policy:
policies:
- name: iam-roles-with-escalated-permissions
  resource: iam-role
  mode:
    role: arn:aws:iam::<ACCOUNTNUMBER>:role/custodian
    type: cloudtrail
    events:
      - source: "iam.amazonaws.com"
        event: AttachRolePolicy
        ids: "requestParameters.roleName"
  filters:
    - type: event
      key: "detail.requestParameters.policyArn"
      op: regex
      value: (^arn:aws:iam::aws:policy\/)\w+

  actions:
    - type: notify
      template: default
      priority_header: 2
      subject: "test"
      violation_desc: "test"
      action_desc: "test"
      to:
          - arn:aws:sns:eu-west-1::mailer-sns
      transport:
        type: sqs
        queue: https://sqs.eu-west-1.amazonaws.com//mailer

log:
16:43:11
[INFO]   Found resource ids: [u'jenkins']

[INFO]   Resources []

END RequestId:

Is this planned on being updated soon? I feel like this falls more in the category of bug than enhancement since the current design does not allow full functionality.


fix: reintroduce connection stack in transformer v2

Description of changes
This commit reintroduces the dedicated connection stack in v2 of the GraphQL transformer. This allows v1 @connection users to migrate existing applications to v2.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #8757 (175b05c) into master (adb70b1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #8757   +/-   ##
=======================================
  Coverage   57.02%   57.02%
=======================================
  Files         757      757
  Lines       42949    42954    +5
  Branches     9566     9567    +1
=======================================
+ Hits        24491    24496    +5
  Misses      16544    16544
  Partials     1914     1914

Impacted Files
Coverage Δ

...fy-graphql-relational-transformer/src/resolvers.ts
97.19% <100.00%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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


Adjacent blocks are powered by redstone

Describe the Bug
When playing on the Forge (original) version of Create, placing two redstone-conductive blocks next to each other and powering one of them causes only that one block to be powered (shown in screenshots below). However, when attempting to do the same on the Fabric version, both blocks now become powered from powering only one of them, causing issues with certain contraptions in which components get powered by redstone when they shouldn't be.
Reproduction Steps

Place two redstone-conductive blocks next to each other (for example, a Gearshift and a Clutch).
Power only one of the blocks using any source (lever, button, etc.)
Observe as both blocks become powered when only one of them should be

Expected Result
Only the block that was originally powered by whatever means should be turned on rather than the power conducting to the attached block.
Screenshots and Videos
 Forge Version
 Fabric Version
Crash Report or Log
N/A
Operating System
Windows 10
Mod Version
0.4.1
Minecraft Version
1.18.2
Other Mods
Other than Create Fabric and the Fabric API, no other mods were running.
Additional Context
This bug is preventing me from making a cool elevator. I'll give y'all big mcthankies from mcspankies if it gets fixed

having this as well, this is an pretty big issue as it means condensed schematics wont work between Forge and Fabric versions


Final pull request for set game path notification

Whenever the game path is not set, a notification shows up that reminds you to do that. Hopefully there won't be conflicts anymore.

I don't have a chance to test this at the moment, but this seems like it will notify every time the launcher is reopened, correct? Even if they have changed it in a previous session? It also disregards whether the default path is actually already correct or not (since the default path that is set upon first install is the default install for most people)

I don't have a chance to test this at the moment, but this seems like it will notify every time the launcher is reopened, correct? Even if they have changed it in a previous session? It also disregards whether the default path is actually already correct or not (since the default path that is set upon first install is the default install for most people)
Nope, from my tests it will not notify every time you open the launcher and it also does not disregard the default path.
If there's no path, it notifies.

Works as described, but since the default path is set for every user, the only time this will notify is if the user manually removes their current path in settings. Would be nice if it checked default path existing, like Spike said above, and then also notifies in that case.
The average user with a different install will see the game is "set" even though the actual exe doesn't exist for them there, and still not know why the game isn't launching. They aren't seeing the notification since they have a game path set, just not a valid one.
Might also be a good idea to replace the span text with a tr so that other languages can have the right message there.

@NotThorny
Oh, I didn't understand what spike meant. This seems like a great idea.
Also I wouldn't even have that such a solution may exist. Thank you!

Sure thing, just updated it to work properly since I hadn't actually tested it when writing it the first time. So if the default path doesn't exist for the user, it will unset the game path and alert that one needs to be set.

@NotThorny
Do I just push the change or create a new request?

@NotThorny Do I just push the change or create a new request?

You can just commit the suggested changes in this branch/PR, no need to make a new one

Commited the changes, I think this should be working now

Prettier checks fail, mind running that and committing the changes pls?


feat(community): added code blocks in markdown into document's page content

Hi!
Big fan of Langchain here :)
Was working with the Confluence Loader and I noticed that code snippets weren't added thru the Document's pageContent.
Let me know what you think!

Looks reasonable, thank you!

Please avoid updates after approval as CI needs to run again

No worries, thanks again!


Luna-1471 Comms Example to print Dev Info

This is the alternate method I mentioned on Slack this morning

@siddirp   I'm not sure I want to add more stuff the Communications Example Project.  Most of our customers need brain dead simple projects so that the essential part of the project is easier to find.  What do you think?

@siddirp I'm not sure I want to add more stuff the Communications Example Project. Most of our customers need brain dead simple projects so that the essential part of the project is easier to find. What do you think?

@waltjohnson That's fine. I just threw this together as an example of a brain dead app to get health info aside from the CLTool reporting. :) 75% of it could be removed. Anyway, I'm fine with this going away.


fix desktop missing floating CTA

Desktop Floating CTA is missing on pages with useLottieArrow is toggled to off. This PR should fix it
Resolves: MWPW-NUMBER pending
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/entitled?martech=off
After: https://missing-desktop-fcta-fix--express--adobecom.hlx.page/express/entitled?martech=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4cf2c29) 31.91% compared to head (93822bc) 31.92%.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #521      +/-   ##
==========================================
+ Coverage   31.91%   31.92%   +0.01%
==========================================
  Files         126      126
  Lines       24109    24109
==========================================
+ Hits         7695     7698       +3
+ Misses      16414    16411       -3

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


Recoding for Medical Diagnosis

Please paste collapse code & renaming code chunks

   med_his_1 <- med_his_0 %>%       mutate(CDC_records = if_else(samodify %in% c("CHRONIC CARDIAC DISEASE (NOT HYPERTENSION)", "ISCHAEMIC HEART DISEASE", "CARDIOMYOPATHY", "CONGENITAL HEART DISEASE","CHRONIC CARDIAC DISEASE (NOT HYPERTENSION)_NEOPLASM (UNSPECIFIED)"),"yes",NA),                 CKD_records = if_else(samodify == "CHRONIC KIDNEY DISEASE","yes", NA),                 CPD_records = if_else((samodify  %in% c("CHRONIC PULMONARY DISEASE (NOT ASTHMA)", "CHRONIC PULMONARY DISEASE (NOT SPECIFIED)")),"yes", NA),                 DB1_records = if_else((samodify == "DIABETES MELLITUS - TYPE 1"),"yes", NA),                 DB2_records = if_else((samodify == "DIABETES MELLITUS - TYPE 2"),"yes", NA),                 DB_records =  if_else((samodify == "DIABETES MELLITUS - TYPE NOT SPECIFIED"),"yes", NA),                 asthma_records = if_else((samodify == "ASTHMA"),"yes", NA),                 hypertension_records = if_else((samodify == "HYPERTENSION"),"yes", NA),                 obesity_records = if_else((samodify == "OBESITY"),"yes", NA),              malnutrition_records = if_else((samodify == "MALNUTRITION"),"yes", NA),              dementia_records = if_else((samodify == "DEMENTIA"),"yes", NA),              precovid_records = if_else((samodify  %in% c( "PREVIOUS COVID-19 INFECTION", "PREVIOUS COVID-19 INFECTION - CONFIRMED BY TESTING", "PREVIOUS COVID-19 INFECTION - NOT CONFIRMED BY TESTING" )),"yes", NA),              liver_records = if_else((samodify  %in% c("MODERATE OR SEVERE LIVER DISEASE", "MILD LIVER DISEASE", "LIVER DISEASE (SEVERITY NOT SPECIFIED)")),"yes", NA),              AKI_records = if_else((samodify == "ACUTE KIDNEY INJURY"),"yes", NA))
 med_his_list <- list(CDC = c("CHRONIC CARDIAC DISEASE (NOT HYPERTENSION)", "ISCHAEMIC HEART DISEASE", "CARDIOMYOPATHY", "CONGENITAL HEART DISEASE","CHRONIC CARDIAC DISEASE (NOT HYPERTENSION)_NEOPLASM (UNSPECIFIED)"),                          CKD = c("CHRONIC KIDNEY DISEASE"),                           CPD =  c("CHRONIC PULMONARY DISEASE (NOT ASTHMA)", "CHRONIC PULMONARY DISEASE (NOT SPECIFIED)"),                           DB1 =  c("DIABETES MELLITUS - TYPE 1"),                           DB2 =  c("DIABETES MELLITUS - TYPE 2"),                           DB =  c("DIABETES MELLITUS - TYPE NOT SPECIFIED"),                           asthma =  c("ASTHMA"),                           hypertension = c("HYPERTENSION"),                           obesity = c("OBESITY"),                           malutrition =  c("MALNUTRITION"),                           dementia = c("DEMENTIA"),                          precovid = c("PREVIOUS COVID-19 INFECTION", "PREVIOUS COVID-19 INFECTION - CONFIRMED BY TESTING", "PREVIOUS COVID-19 INFECTION - NOT CONFIRMED BY TESTING"),                          liver =  c("MODERATE OR SEVERE LIVER DISEASE", "MILD LIVER DISEASE", "LIVER DISEASE (SEVERITY NOT SPECIFIED)"),                           aki_medhis = c("ACUTE KIDNEY INJURY"))

Example using upcoming features in v<IP_ADDRESS>19
med_his_list <- list(CDC = c("CHRONIC CARDIAC DISEASE (NOT HYPERTENSION)", "ISCHAEMIC HEART DISEASE", "CARDIOMYOPATHY", "CONGENITAL HEART DISEASE","CHRONIC CARDIAC DISEASE (NOT HYPERTENSION)_NEOPLASM (UNSPECIFIED)"),
                         CKD = c("CHRONIC KIDNEY DISEASE"),
                         CPD =  c("CHRONIC PULMONARY DISEASE (NOT ASTHMA)", "CHRONIC PULMONARY DISEASE (NOT SPECIFIED)"),
                         DB1 =  c("DIABETES MELLITUS - TYPE 1"),
                         DB2 =  c("DIABETES MELLITUS - TYPE 2"),
                         DB =  c("DIABETES MELLITUS - TYPE NOT SPECIFIED"),
                         asthma =  c("ASTHMA"),
                         hypertension = c("HYPERTENSION"),
                         obesity = c("OBESITY"),
                         malutrition =  c("MALNUTRITION"),
                         dementia = c("DEMENTIA"),
                         precovid = c("PREVIOUS COVID-19 INFECTION", "PREVIOUS COVID-19 INFECTION - CONFIRMED BY TESTING", "PREVIOUS COVID-19 INFECTION - NOT CONFIRMED BY TESTING"),
                         liver =  c("MODERATE OR SEVERE LIVER DISEASE", "MILD LIVER DISEASE", "LIVER DISEASE (SEVERITY NOT SPECIFIED)"),
                         aki_medhis = c("ACUTE KIDNEY INJURY"))

## verify
med_his_list |>
  verify_named_no_dup_values()

## sample crossmap
med_hist_xmap <- as_pairs_from_named(med_his_list, names_to = "analysis_group", values_to = "symptom") |>
  add_weights_unit(weights_into = "w") |>
  as_xmap_df(from = symptom, to = analysis_group, weights = w)


Quick and dirty fix for multifilter provider label

fixes #380

@bobular I addressed your suggestion about pluralization. I also added some comments and changed some variable names in the help function.
Regarding scrolling the provider names: I'm going to leave as-is for now. If anything, I think the section could be collapsible, but I'm not convinced it's better.


Can't customize the whole AudioSpec in open_audio_device

I'd like to be able to specify a few more fields in the SDL_AudioSpec struct passed to SDL_OpenAudioDevice (the samples field in particular to change the audio sample buffer size). However those are hardcoded to 0 in AudioSpecDesired::convert_to_ll.
I think it would make sense to add the missing fields in the AudioSpecDesired struct. It could also have a constructor that would set them to the current default values (if most people don't care about those, which seems to be the case) but still allow modifying them if one needs to.
Would you accept a pull request in that direction?

Actually, at a 2nd glance I realize that samples is the only missing field, the others are filled by SDL_OpenAudioDevice. So maybe a constructor is overkill, just adding the field to AudioSpecDesired would be enough. I'm writing a patch.


How to use the “/model" script？

I dont know how to follow the usage and documents, like below：
`import torch
from model import MattingRefine
device = torch.device('cuda')
precision = torch.float32
model = MattingRefine(backbone='mobilenetv2',
backbone_scale=0.25,
refine_mode='sampling',
refine_sample_pixels=80_000)
model.load_state_dict(torch.load('PATH_TO_CHECKPOINT.pth'))
model = model.eval().to(precision).to(device)
src = torch.rand(1, 3, 1080, 1920).to(precision).to(device)
bgr = torch.rand(1, 3, 1080, 1920).to(precision).to(device)
with torch.no_grad():
pha, fgr = model(src, bgr)[:2]`
where this code read my own image？Are the src = torch.rand(1, 3, 1080, 1920).to(precision).to(device) bgr = torch.rand(1, 3, 1080, 1920).to(precision).to(device)are only an example？I don't understand.
So I use this inference_images.py to infer my image. I sort my data and pretrained model like this：

And add args like this：
--model-type mattingrefine --model-backbone resnet50 --model-checkpoint /home/user/BackgroundMattingV2/A_matting/model/PyTorch/pytorch_resnet50.pth --images-src /home/user/BackgroundMattingV2/A_matting/input/1384624442.jpg --images-bgr /home/user/BackgroundMattingV2/A_matting/input/timg.jpeg --output-dir /home/user/BackgroundMattingV2/A_matting/output/out.jpg --output-types com -y
It didn't work and error reported like 0it [00:00, ?it/s] and nothing export out.
Could you help me please？
Thanks！
Best,
Joevaen.

Hi @Joevaen
So the code sample you showed is for if you wanted to write your own code using our model. You would find your own ways to load images in, so we just use random inputs as an example.
If (as it sounds) you are just trying to run inference on images, then I think your biggest issue is that all arguments should be directories (of inputs or outputs) - not individual images. Also, all extensions should be ".jpg" or ".png".
So you could move your "1384624442.jpg" and "timg.jpeg" into separate "input source images" and "input backgrounds" directories, and name the same thing (ending in ".jpg") so they match - and then supply those directories to the script.

I got that, thank you andrey! I will try.

Hi @Joevaen
So the code sample you showed is for if you wanted to write your own code using our model. You would find your own ways to load images in, so we just use random inputs as an example.
If (as it sounds) you are just trying to run inference on images, then I think your biggest issue is that all arguments should be directories (of inputs or outputs) - not individual images. Also, all extensions should be ".jpg" or ".png".
So you could move your "1384624442.jpg" and "timg.jpeg" into separate "input source images" and "input backgrounds" directories, and name them the same thing (ending in ".jpg") so they match - and then supply those directories to the script.

I followed your advice but error was encountered like below：
/home/qiao/anaconda3/bin/python /home/qiao/PythonProjects/BackgroundMattingV2/inference_images.py --model-type mattingrefine --model-backbone mobilenetv2 --model-checkpoint /home/qiao/PythonProjects/BackgroundMattingV2/A_matting/model_pre/PyTorch/pytorch_mobilenetv2.pth --images-src /home/qiao/PythonProjects/BackgroundMattingV2/A_matting/input/input_source_images --images-bgr /home/qiao/PythonProjects/BackgroundMattingV2/A_matting/input/input_backgrounds --output-dir /home/qiao/文档/A_matting/output --output-types com -y   0%|          | 0/1 [00:00<?, ?it/s] Traceback (most recent call last):   File "/home/qiao/PythonProjects/BackgroundMattingV2/inference_images.py", line 120, in <module>     for i, (src, bgr) in enumerate(tqdm(dataloader)):   File "/home/qiao/anaconda3/lib/python3.7/site-packages/tqdm/std.py", line 1165, in __iter__     for obj in iterable:   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 435, in __next__     data = self._next_data()   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 1085, in _next_data     return self._process_data(data)   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 1111, in _process_data     data.reraise()   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/_utils.py", line 428, in reraise     raise self.exc_type(msg) TypeError: Caught TypeError in DataLoader worker process 0. Original Traceback (most recent call last):   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 198, in _worker_loop     data = fetcher.fetch(index)   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch     data = [self.dataset[idx] for idx in possibly_batched_index]   File "/home/qiao/anaconda3/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>     data = [self.dataset[idx] for idx in possibly_batched_index]   File "/home/qiao/PythonProjects/BackgroundMattingV2/dataset/zip.py", line 19, in __getitem__     x = self.transforms(*x)   File "/home/qiao/PythonProjects/BackgroundMattingV2/dataset/augmentation.py", line 21, in __call__     x = transform(*x)   File "/home/qiao/PythonProjects/BackgroundMattingV2/dataset/augmentation.py", line 30, in __call__     return [self.transforms(xi) for xi in x]   File "/home/qiao/PythonProjects/BackgroundMattingV2/dataset/augmentation.py", line 30, in <listcomp>     return [self.transforms(xi) for xi in x] TypeError: ToTensor() takes no arguments
Why this happened？Should I fix the bgr and src with same  shape before input them？

@Joevaen In file inference_images.py, on line 95, replace A.PairApply(T.ToTensor) by A.PairApply(T.ToTensor())

@Joevaen In file inference_images.py, on line 95, replace A.PairApply(T.ToTensor) by A.PairApply(T.ToTensor())

@arthurmchr Thanks for helping us finding a bug.


Form Components throw error on Blazor with Preview 8 build

Describe the bug
Since updating to Preview 8, the form components Intellisense is showing that the <Input...> for , , and  as an error. However, the project is able to build and run without any issues. The exact error message is: "The type of component  cannot be inferred based on the values provided. Consider specifying the the type arguments directly using the following attributes: 'T'". It also makes the project display a huge list of errors.

To Reproduce
Steps to reproduce the behavior:

Create new preview 8 blazor server application using dotnet new blazorserver.
Add an InputSelect, InputNumber, or InputDate component
Errors will appear but project will be able to build and run as normal.

Expected behavior
Inputs will be mapped without any issue/errors.
Additional context

Thanks for contacting us, @dmarsh12.
Please make sure you're using the latest VS 2019 Preview release.


Fail of the AcquireCredentialsHandleA for TLS 1.3

Describe the bug
AcquireCredentialsHandleA return 0x80090331 for TLS1.3 protocol which leads to negotiation error AWS_IO_TLS_ERROR_NEGOTIATION_FAILURE. Same methods with same credentials  works fine for lower TLS versions
Expected Behavior
AcquireCredentialsHandleA should return SEC_E_OK for any TLS protocol versions
Current Behavior
AcquireCredentialsHandleA fails for latest master on given moment
https://github.com/awslabs/aws-c-io/blob/a3555c86bd10149a1ab0c1b2810756ec54cfbe3a/source/windows/secure_channel_tls_handler.c#L1738
Reproduction Steps
activate AWS_IO_TLSv1_3 version for tls handler
Possible Solution
According to MS learn
https://learn.microsoft.com/en-us/answers/questions/708734/tls-1-3-doesnt-work-on-windows-11-through-schannel

In order to use TLS 1.3 with schannel, you should use the SCH_CREDENTIALS structure instead of the SCHANNEL_CRED structure with AcquireCredentialsHandle().

SCH_CREDENTIALS - Win32 apps | Microsoft Learn
i.e current  sc_ctx->credentials which is pointer to SCHANNEL_CRED structure
SECURITY_STATUS status = AcquireCredentialsHandleA(
    NULL,
    UNISP_NAME,
    credential_use,
    NULL,
    &sc_ctx->credentials,//<-- currently it is SCHANNEL_CRED
   ...);

should be replaced on SCH_CREDENTIALS
Additional Information/Context

AWS CPP SDK version used
1.26.0, 1.30.0
Compiler and Version used
MSVS 2022 Professional
Operating System and version
Windows 11 x64

It look like you are trying to use the IoT sdk not this sdk. Can you open an issues here(aws-iot-device-sdk-cpp-v2) and provide a code sample that reproduces this?

Hello @jmklix
Thanks for reacting on my issue !
Looks like you are right my issue is about problem in AWS-C-IO which in it's turn is one of submodule into AWS CRT CPP
It's because this is submodule of current SDK

I made a little bit correct assumption about responsibilities about right team for fixing problem.
I'm going to address this issue to them, now.
Regards,
Yurii.

Hi @YuriL180821 ,
Thank you for reporting this issue.
As discussed, this one is for AWS Iot SDK for CPP and out common dependency AWS CRT / aws-c-io.
I will close this issue, please contact us again should you have any other issue.
Best regards,
Sergey


Update from master

Updated with changes and bugfixes from the master so that we can begin development on the debug flags again.

Hey, this is unrelated to this particular PR, but I thought you might appreciate it: http://codegolf.stackexchange.com/a/86157/31716
:smiley:

If you would like an explanation of my modulus I would be happy to provide one.

Lastly I used a version of my optimizer to shorten your solution to 93 bytes.
(({}<>))<>{({}[()])<>(({}[()])){{}(<({}[({})])>)}{}({}({}))<>}{}<>(([{}]{})){{}{}(<(())>)}{}
It is still your algorithm just my program figured out how to make it a bit shorter.

Hmm, that doesn't seem to be working right. http://brain-flak.tryitonline.net/#code=KCh7fTw-KSk8Pnsoe31bKCldKTw-KCh7fVsoKV0pKXt7fSg8KHt9Wyh7fSldKT4pfXt9KHt9KHt9KSk8Pn17fTw-KChbe31de30pKXt7fXt9KDwoKCkpPil9e30&input=MTAwCjQ
But hey, if you can outgolf me, feel free to make an answer on that question! I love a little friendly competition!

Yeah I had a bug in my developmental version.  I have updated it to a working albeit longer version.

What else is left to do with the debug flags?

I was going to add a flag to display both stacks and a flag for code injection.  I intend to start an issue for this.

Ah cool, yeah that would be useful


Event Listner Probem (error 406)

Hi I have problem with event listener, when run node-red i get payload (from event-listener). Several restart node-red, return error "0" - now event listener work good to the next restart.
{   payload: {     error: 406,     apikey: '4457b2fa-bf69-4e45-96d5-7d13f5dec9bd',     sequence: '1610213844000',     actionName: 'userOnline'   },   _msgid: '927883f2.041c4' }
Other ewelink blocks  works good.

https://github.com/ottoszika/node-red-contrib-ewelink/issues/110

https://github.com/ottoszika/node-red-contrib-ewelink/issues/110


Add github actions

To help with test coverage (#2) it is first important to automate this process for pull requests and also the main branch.
This commit adds a couple of jobs to run with github actions, the minimal CI requirements. A couple of fixes have been applied following the warnings returned by clippy. Most of them were trivial (like clone calls).
Because of the proc macro json_path_value, it was necessary to allow globally #![allow(clippy::vec_init_then_push)] (#!). This is not really convenient as it will certainly disable it for the rest of the library. Unfortunately this rule can't be added to the macro itself. It is triggered because of the use of a mutable vector, clippy doesn't see the expansion of the iterator over $v.
Formatting was also fixed but depending on the maintainer opinion it might be reverted for this commit in order to clean the diff and can be let for a subsequent PR.

CI successfully run there: https://github.com/theredfish/jsonpath-rust/actions/runs/4375476892
Not sure about this repo. First time contributor, PR from fork or repo settings might block the CI to run?

Hi @theredfish!
Thank you for this PR. That is awesome.

Not sure why the CI doesn't run though. Maybe some settings are necessary?

Let me check

Allow me to clarify the guess:
it expects a push to the branch main
but here the default branch is 'master'.
Can it be a problem?

I will try to remain the branch to main

Added the following pr

Oh sorry :) based the PR on my repo configs and I'm using main - definitely the issue there for this branch ! Glad to know it's now running!

Yes, thanks a million. And at long last, I have changed the name of the main branch :) (Always supposed to do it but did not have enough time :))


RemoveEdge incorrectly throws error in specific case

The RemoveEgde function throws "Key not found" error while trying to remove an edge a->b if there exist no edges with a as their source. This is happening due to the following code:
	item, err := txn.Get([]byte(from))
	if err != nil {
		return err
	}

where we are not checking if the error returned by badger is ErrKeyNotFound as is being done for the AddEdge function

Modify the RemoveEdge function to return (bool, error) instead of just a error where bool signifies whether the Remove actually removed an edge or whether it silently failed because the edge didn't exist.
Have the function return:

true, nil on successful removal of an edge that exists
false, nil if the edge doesn't exist
false, error if there is any error

Note that you will have to check for the edge not existing is 2 places: while getting the edgelist for the src node from badger, and then again in the edgelist before calling delete(dstNodes, to)
You are also required to add a unit test to cover this case
The above change is a breaking change to the RemoveEdge API. Make sure to edit all the calls to RemoveEdge in lib_test.go to reflect that
psst, remember to update the usage guide in the README too 👀

Hi @Smuzzy-waiii, could I be assigned this bug?

@JP-sDEV This repo is being prepped for a university club event called Hacknight where participants get to solve issues on FOSS repo's to get points. You can find out more about hacknight on the ACM PESU ECC instagram here. The event is from 4PM IST on Fri, 18th Oct  to 12PM IST on Sat, 19th Oct.
If this issue is still unresolved at the end of the event, I would love to assign it to you

No worries, thanks!

could i be assigned??

!assign @velukutty2194

The bot will deassign you in 45mins. If you need more time, show your progress and I can extend the time. Please feel free to text/call me at<PHONE_NUMBER>

@velukutty2194 Are you still working on this?

!extend

!deassign

@JP-sDEV Are you still interested in working on this? I can assign you

@Smuzzy-waiii  Yes, I am still interested! I will start working on it on Monday, if assigned.

@JP-sDEV Yes np go for it! 😃


PHOENIX-7138 Import netty-bom to make sure all netty 4.1 modules shar…

…e the same version

I've re-run the failing tests successfully locally.

Yes, Netty 3 and 4 can coexist, so that's fine (We could probably exclude netty3, but that's a different hill to climb)

merged manually


feat: better navigation; search by filename

As proposed by the Mux team, this upgrade brings several UX improvements to help editors working with larger video pools. It also uses the newer @mux/mux-player-react component.
Tagging @dylanjha to give the green light for review by the Sanity team.

@stipsan I've gone through the issues you've raised (THANK YOU!), but in the process of using your studio I realized uploads are being duplicated, sometimes triplicated 😨

I didn't put a finger on the upload logic as it wasn't needed and it requires more context than I have... I could've broken it unwittingly, but I think this started happening in the current version as well - see #298
Can you help me debug this? Perhaps asking someone involved in the original creation of the plugin and the Sanity /addons/mux endpoints? I'd think Espen or PK 🤔
UNLESS, of course, you actually did double uploads, which would be a big relief 🍉

@stipsan I've gone through the issues you've raised (THANK YOU!), but in the process of using your studio I realized uploads are being duplicated 😨

I didn't put a finger on the upload logic as it wasn't needed and it requires more context than I have... I could've broken it unwittingly, but I think this started happening in the current version as well - see https://github.com/sanity-io/sanity-plugin-mux-input/issues/298
I also notice the duplicated videos seem to have happened before I opened this PR

Can you help me debug this? Perhaps asking someone involved in the original creation of the plugin and the Sanity /addons/mux endpoints? I'd think Espen or PK 🤔

UNLESS, of course, you actually did double uploads, which would be a big relief 🍉

I haven't seen any unintentional double uploads in my own testing. I've only tested in production mode though, could it be a React Strict Mode "fire useEffect twice" kind of deal? We have #298 on our backlog to come back and investigate. And like you say, it's been reported before this PR so there's no need to hold-off on the PR any further 👍


bump grunt-lib-phantomjs

This brings support for modern PhantomJS releases as per https://github.com/gruntjs/grunt-lib-phantomjs/blob/master/CHANGELOG

@sandlerr Thanks!


Update shared ButtonsBar look #1819

Closes #1819

@vladyslav-iosdev would you help to rebase master for the PRs with merge commits? They are created likely because we did a git pull origin master on the feature branch it merged in the latest from the updated master branch. Just need to do:
git rebase master

and it should rewrite the history of the feature branch (there might be merge conflicts) and force push.
Generally, you don't want to do git pull origin master on a feature branch. You should:
1. Update the master branch locally (git pull origin master when master is checked out)
2. git rebase master (when the feature branch is checked out)
3. git push origin <feature> --force

If rebase fails and it seems stuck, git rebase --abort is the escape hatch.

Forced pushed after git rebase master

@vladyslav-iosdev I forced pushed after git rebase master, so you'll have to git pull origin --force to update (replace origin with your remote to your fork)

git push origin

i a little bit confuzed now, i did all you send now i got this
as i undarstand i need to do next
git push origin <feature> --force
what should i replace with <feature>?

there is now conflicts now.

git push origin

@hboon i a little bit confuzed now, i did all you send now i got this
as i undarstand i need to do next
git push origin <feature> --force
what should i replace with <feature>?

there is now conflicts now.

I noticed that you are on master branch in the screenshot. Generally, the only thing you want to do while the master branch is checked is is to do git pull <remote> master ( is the one pointing to the upstream AlphaWallet repo)

git push origin  --force
what should i replace with ?

git push <remote> <branch>

remote — one of the names in the left column in the output of git remote
branch — the (feature) branch name to push. So in this case "#1819"
(sorry, I should have typed "" earlier)

You shouldn't have to do https://github.com/AlphaWallet/alpha-wallet-ios/pull/1832#issuecomment-634089331 anymore because I have forced push at https://github.com/AlphaWallet/alpha-wallet-ios/pull/1832#issuecomment-634414034,
Sorry, I should have clarified that.
1. So, you should do this now:
(while checked out in branch #1819)
***** note that git reset --hard will lose any uncommitted changes *****
git reset --hard

2. And then this to get the latest #1819 (which I forced pushed):

I forced pushed after git rebase master, so you'll have to git pull origin --force to update (replace origin with your remote to your fork). If git creates a merge commit for you, it wasn't forced pulled correctly.

3. Then fix/update the commits
4a. Squash and force push
4b. Just push (and I'll ask you to squash and force push)

I noticed that you are on master branch in the screenshot. Generally, the only thing you want to do while the master branch is checked is is to do git pull <remote> master ( is the one pointing to the upstream AlphaWallet repo)

git push origin  --force
what should i replace with ?

git push <remote> <branch>

remote — one of the names in the left column in the output of git remote
branch — the (feature) branch name to push. So in this case "#1819"
(sorry, I should have typed "" earlier)

seems like i did this.

@hboon sorry my fail, closed it by mistake.

@vladyslav-iosdev can you rebase master? There might be conflicts when you do so

@vladyslav-iosdev can you rebase master? There might be conflicts when you do so

seems like done.

Sorry, Squash commits and force push?

Sorry, Squash commits and force push?

yes, i did Squash commits.

yes, i did Squash commits.

 So you can do a forced pull to get the latest

Notice that in the Commits tab (in this page), there is just 1 commit.

@vladyslav-iosdev remember to buzz me for review if it's done.

@vladyslav-iosdev ping me in case you are waiting for me to do something here.


Fixing compiler warnings with GCC 4.7.2

This is the correction of pull request 357. Since the commit message had to be corrected, I ve decided to close the previous pull-request and create a new one:

Considering remarks of @martinezjavier  and @williamcroberts
Correcting the commit message

Done.


As a user, I want to delete my account

User story
As a user, I want to delete my account, so I can remove all my data from the app

Acceptance criteria

[ ] Only the user himself could remove his account.
[ ] All data related to the user removed as well
[ ] unit test
[ ] documentation for the endpoint

Hello, I would like to be assigned this task. However, there is one aspect I have not grasped yet from the codebase-my local environment only displays the front-end. I would wish to understand how I can make a connection to the DB in my local machine, and test if my endpoints are working before making any commits. Kindly, I will appreciate your help.

Hi Nehemiah. Sure, you can take this one.
Thank you for letting me know about the issue you face during project installation on your windows machine. I'll double-check if there are any issues. If you have something to add, for example, a full docker container log, please do this in the corresponding issue https://github.com/ClimateMind/climatemind-backend/issues/447
Meanwhile, I hope you are able to plan architecture and your code changes for the user deletion function.
Feel free to ask me any questions. We've just created a chat for the project https://t.me/climatemind_chat you can find me there

Hi @Nehemiah60,
do you have any further questions?

Hi @Nehemiah60, do you have any further questions?

Yes, I am getting this error when trying to commit my changes....
remote: Permission to ClimateMind/climatemind-backend.git denied to Nehemiah60.

@Nehemiah60 yeah, probably you need to do the following

create a fork
then change the git origin to your fork
push a new branch to your fork
make a PR to the original repository from your fork

Hope this will help. If not, let me know. You can write to me in the telegram chat.
sorry about the misleading documentation, I'll change this next week.

@Nehemiah60 yeah, probably you need to do the following

create a fork
then change the git origin to your fork
push a new branch to your fork
make a PR to the original repository from your fork

Hope this will help. If not, let me know. You can write to me in the telegram chat.
sorry about the misleading documentation, I'll change this next week.

Hello Dan. I just wanted to have a little conversation. Let me know when available so that I can send a video link. Thanks.

@Nehemiah60 yeah, probably you need to do the following

create a fork
then change the git origin to your fork
push a new branch to your fork
make a PR to the original repository from your fork

Hope this will help. If not, let me know. You can write to me in the telegram chat.
sorry about the misleading documentation, I'll change this next week.

Also Dan my Telegram is also restricted. I am being told I send spam messages. So maybe I might not be able to send messages, as it shows te red notifications on my messages. I am waiting for you to join the meeting since I sent the link on Telegram. Thanks.

@Nehemiah60 I recommend you to start from this first https://github.com/ClimateMind/climatemind-backend/issues/396
to figure out how to use pytest


Installation bug - ModuleVersion listed as <IP_ADDRESS> when installing Master version

Do you want to request a feature or report a bug?
Reporting an installation bug
What is the current behavior?
After installing via the two Invoke-Expression installation methods from README.md, Get-Command indicates the version is <IP_ADDRESS>
Root cause: The version number listed in the module manifest ReportingServicesTools.psd1 contains the wrong version number.
# Version number of this module.
ModuleVersion = '<IP_ADDRESS>'
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

Uninstall the current version of ReportingServicesTools
Install via one of the two Invoke-Expression installation methods
Invoke-Expression (Invoke-WebRequest https://raw.githubusercontent.com/Microsoft/ReportingServicesTools/master/Install.ps1)
or
Invoke-Expression (Invoke-WebRequest https://aka.ms/rstools)
Note as part of the installation, the installed commands are returned and show version <IP_ADDRESS>

What is the expected behavior?
Get-Command -Module ReportingServicesTools should return the current version (<IP_ADDRESS>) for each of the alias and functions exported by this module.
Which versions of Powershell and which OS are affected by this issue? Did this work in previous versions of our scripts?
This is the first time I've installed ReportingServicesTools, however I got the same results via both pwsh v7.1.4 and powershell v5.1.19041.1023
Proposed solution
Update ReportingServicesTools.psd1 with the current version number.

I am now using powershell v5.1.19041.1023 since I realized that New-WebServiceProxy hasn't been ported to Powershell Core.


feat: add LF footer copytight

Working towards: https://github.com/external-secrets/external-secrets/issues/1395
This PR adds the LF footer as per website guidelines.

/approve


Store signature metadata on the original overload def, not on overload sigs

As a follow-up to #8303, set the method flags on the original method when signatures mark it as abstract, overridable, etc. instead of assuming that they're always on the first overload's signature. This simplifies many cases where we were needing to check if the method found was an overload, and then looking again for the first overload's signature so that we could determine if the symbol was abstract, etc.
Motivation
Simplifying the internal handling of symbols with overloaded signatures.
Test plan
This shouldn't affect any existing tests: it's only reorganizing internal datastructures.

At that point, I don't know whether it's still valuable to have the flags be on the namer overload version of the method, because also that method doesn't have type information, but I'd be willing to consider it at that point.

The point was to keep this metadata in one place so that we wouldn't miss conflicting definitions introduced on additional overload sigs. I agree that it's not great to be putting that information on a symbol that has no type information present, but I would argue that these annotations are about the method, and not a specific signature. For example, we currently don't raise errors for the following rbi, and we should:
# typed: __STDLIB_INTERNAL

class A
  extend T::Sig, T::Helpers
  abstract!

  sig {abstract.params(x: Integer).void}
  sig(:final) {params(x:String).void}
  def foo(x); end
end

Perhaps a better path forward here is to forbid any of these annotations on anything but the first signature seen? However, that doesn't resolve any of the issues related to needing to lookup the first overload to determine if a symbol with overloads present is abstract or final etc.

For example, we currently don't raise errors for the following rbi, and we should

Right, I was suggesting that we should fix that, which seems like the root problem. If we started with that change, then I'd be in favor of this change to copy those annotations down to the base method, so that we're never in a spot where GlobalState might have two sources of truth for something and have those sources of truth conflict with each other.

I'm going to close this and investigate checking that overloads agree. Once that's in place, I think it would make sense to revive some part of this and start keeping the canonical version of those flags on the original def.


Can't change fps without making a timeline

I'm considering an API addition that allows:
anim = amp.Animation([block], fps=fps)

where fps gets passed to the constructor of Timeline when no timeline is passed.
Another option is to allow Timeline to accept an integer for t so one would do:
anim = amp.Animation([block], Timeline(len(block), fps=fps))

cc @TomNicholas Thoughts?

Just seen this, I like the first option for an updated API.
I wonder if it perhaps makes more sense to think of the Timeline object as containing information about the slider, whereas fps would be a property of the Animation?


Add system-info callback

A new callback is available in garm which allows runners to send info about themselves while installing. This new callback allows instances to report their os_name, os_version and the GH agent_id.
See https://github.com/cloudbase/garm/pull/200
This change also makes sure that curl is installed in the image, otherwise the entrypoint will fail.

Thanks for the contribution @gabriel-samfira! Would it make sense to add this callback to the summerwind image as well and if so could you add it there too? :)

Thanks for the contribution @gabriel-samfira! Would it make sense to add this callback to the summerwind image as well and if so could you add it there too? :)

not sure. Will look at that image as well. We sould theoretically do it there as well before we delegate the rest of the entrypoint execution to the default one. Will have a look later this week (traveling ATM)

sorry for the delay on this. I added the system info code to the summerwind image as well. It now shows up as "Ubuntu 22.04".


"Google" <=> "Facebook" constant typo in analytics?

https://github.com/edx/edx-app-android/blob/c7d62c85c43b315e21d03480514c04ef771052f8/OpenEdXMobile/src/main/java/org/edx/mobile/module/analytics/Analytics.java#L415-L416
...I'm assuming that's a typo? (If not, it should probably have a comment clarifying that it's intentional)

Wow! that's definitely an error.

Its fix story is on the way.
https://openedx.atlassian.net/browse/LEARNER-3396

Issue has been fixed now so closing the issue.
PR: https://github.com/edx/edx-app-android/pull/1033


Javscript Sandbox: Safe Mode

Overview
Bruno supports execution of JavaScript code in Variables, Scripts, Tests, and Assertions.
Today, we use vm2 to execute javascript.
This approach however allows the code execute system commands, access the file system and this is a security risk.
Most collections do not need to execute system commands or access the file system.
We are introducing Safe Mode to execute JavaScript code in a secure sandbox environment using isolated-vm
The user can switch to Developer Mode if the collection needs access to the file system or system commands, provided the user is aware of the security risks and trusts the source of the collection.
Mockup

The PR is almost ready. We are doing some testing before getting to merged to main and moving forward with a release.

@helloanoop is there a way where we can make Safe Mode to be disabled via a config flag? We use scripts always by default in our dev environment (that import libraries), and want to make sure any new developers don't run into issues related to this.

Any way to make it default to safe mode without asking whenever I create a new collection?

Making this a nag prompt on EVERY SINGLE SCRIPT OPENED is insane.

Yes. We will consider this. An option to set either safe or developer mode as default make sense.


1.15 problems

can you do a rewrite for 1.15

I won't be rewritting this anytime soon. Feel free to submit prs fixing issues on them anytime tho


Issue #40862

Rationale
Issue 40862: Migrate wiki ad hoc <select> elements to SelectBuilders
Changes

Convert some ad hoc <select> elements to SelectBuilders
Override default cellTpl for Ext4.tree.Column

Build and BVTs are green. Now running the Wiki suite (disguised as BVT Postgres).

In addition to the ExtJS fix I've pushed a fix for the WikiLongTest failure.

Will this potentially double-encode in places like the schema browser, where I think/hope we've already done encoding?

Will this potentially double-encode in places like the schema browser, where I think/hope we've already done encoding?

Likely. It looks like we ask the server to do the encoding in that case.
LABKEY.ActionURL.buildURL('query', 'getSchemaQueryTree.api', null, {showHidden : this.showHidden, withHtmlEncoding:true})


Drop hiredis

Coverage remained the same at 100.0% when pulling 825be1ab8d7c8021fc297cb744cd1f60fbbdda4b on drop-hiredis into 5aa3ddd41ce04161e2d10cbfc140d8167bbfa329 on master.

Coverage remained the same at 100.0% when pulling 3f24808c29e13b659f4d1136d970c71858b71ecb on drop-hiredis into 5aa3ddd41ce04161e2d10cbfc140d8167bbfa329 on master.


tensorflow has no gfile

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with the
documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go
here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

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

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

@Adhi-T,
From tensorflow v2.x, tf.gfile is replaced by tf.io.gfile.
Could you try in latest stable version2.9 and also take a look at this doc link which delivers the information. Thank you!


«, » in generated swagger.json

Please take the time to search the repository, if your question has already been asked or answered.

What kind of issue is this?
Bug Report
The following java code:
https://gist.github.com/jannhendrik/7829ad026caae0e8beca2b2732573ed0
Results in this schema ref:
...
"schema":{"$ref":"#/definitions/Page«User»"}
...
Swagger Editor, AutoRest and some azure services cannot handle "«". Maybe it can be replaced by a supported character like _

Please use docket.forCodeGeneration(true)

Thank you very much！

Thank you!


Indepentent functional and integration tests

Current state is that the database is not emptied and refilled on every test case run. The workaround currently is that there is a different entity stored in the database for each test case. For small amounts of tests this is kinda ok, but it is not that well scalable and for example the ListAll returns always different values if more test data is added etc.

Make the test case initialization that the data will be always inserted by test case, so the data is more predictable.

Done with testcontainers and respawn