cdxgen: 10.0.5 -> 10.1.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AppThreat/cdxgen/releases.
meta.description for cdxgen is: Creates CycloneDX Software Bill-of-Materials (SBOM) for your projects from source and container images
meta.homepage for cdxgen is: https://github.com/AppThreat/cdxgen
Updates performed

NPM version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.0.6 with grep in /nix/store/na95x6izi1yzh022dk5hrzzsifjg1pzx-cdxgen-10.0.6
found 10.0.6 in filename of file in /nix/store/na95x6izi1yzh022dk5hrzzsifjg1pzx-cdxgen-10.0.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cdxgen

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/na95x6izi1yzh022dk5hrzzsifjg1pzx-cdxgen-10.0.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cdxgen https://github.com/r-ryantm/nixpkgs/archive/c1eb81cdf1c133afbe9b890594a9320961809542.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c1eb81cdf1c133afbe9b890594a9320961809542#cdxgen

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/na95x6izi1yzh022dk5hrzzsifjg1pzx-cdxgen-10.0.6
ls -la /nix/store/na95x6izi1yzh022dk5hrzzsifjg1pzx-cdxgen-10.0.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cdxgen

Maintainer pings
cc @dit7ya for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AppThreat/cdxgen/releases.
meta.description for cdxgen is: Creates CycloneDX Software Bill-of-Materials (SBOM) for your projects from source and container images
meta.homepage for cdxgen is: https://github.com/AppThreat/cdxgen
Updates performed

NPM version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.1.2 with grep in /nix/store/plx85pv27v6srbx557nvk2ilpjlvdcc0-cdxgen-10.1.2
found 10.1.2 in filename of file in /nix/store/plx85pv27v6srbx557nvk2ilpjlvdcc0-cdxgen-10.1.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cdxgen

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/plx85pv27v6srbx557nvk2ilpjlvdcc0-cdxgen-10.1.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cdxgen https://github.com/r-ryantm/nixpkgs/archive/e22cbf99004cf0e2f960801dddc065bb5dff1849.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e22cbf99004cf0e2f960801dddc065bb5dff1849#cdxgen

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/plx85pv27v6srbx557nvk2ilpjlvdcc0-cdxgen-10.1.2
ls -la /nix/store/plx85pv27v6srbx557nvk2ilpjlvdcc0-cdxgen-10.1.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cdxgen

Maintainer pings
cc @dit7ya for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AppThreat/cdxgen/releases.
meta.description for cdxgen is: Creates CycloneDX Software Bill-of-Materials (SBOM) for your projects from source and container images
meta.homepage for cdxgen is: https://github.com/AppThreat/cdxgen
Updates performed

NPM version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.1.3 with grep in /nix/store/jkb3kgxv0scc8qr29mr0zsgih5l2h2hk-cdxgen-10.1.3
found 10.1.3 in filename of file in /nix/store/jkb3kgxv0scc8qr29mr0zsgih5l2h2hk-cdxgen-10.1.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cdxgen

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jkb3kgxv0scc8qr29mr0zsgih5l2h2hk-cdxgen-10.1.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cdxgen https://github.com/r-ryantm/nixpkgs/archive/8d067592d2b23e793c77fc17deabb54bab69790b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8d067592d2b23e793c77fc17deabb54bab69790b#cdxgen

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jkb3kgxv0scc8qr29mr0zsgih5l2h2hk-cdxgen-10.1.3
ls -la /nix/store/jkb3kgxv0scc8qr29mr0zsgih5l2h2hk-cdxgen-10.1.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cdxgen

Maintainer pings
cc @dit7ya for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AppThreat/cdxgen/releases.
meta.description for cdxgen is: Creates CycloneDX Software Bill-of-Materials (SBOM) for your projects from source and container images
meta.homepage for cdxgen is: https://github.com/AppThreat/cdxgen
Updates performed

NPM version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.2.1 with grep in /nix/store/4m1amzfr95d3dibrilj6is4qgip02x0y-cdxgen-10.2.1
found 10.2.1 in filename of file in /nix/store/4m1amzfr95d3dibrilj6is4qgip02x0y-cdxgen-10.2.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cdxgen

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4m1amzfr95d3dibrilj6is4qgip02x0y-cdxgen-10.2.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cdxgen https://github.com/r-ryantm/nixpkgs/archive/848708793c5eff1d5bdb97c61a03d724ce6419e5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/848708793c5eff1d5bdb97c61a03d724ce6419e5#cdxgen

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4m1amzfr95d3dibrilj6is4qgip02x0y-cdxgen-10.2.1
ls -la /nix/store/4m1amzfr95d3dibrilj6is4qgip02x0y-cdxgen-10.2.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

cdxgen

Maintainer pings
cc @dit7ya for testing.


Unable to Import simple Text/CSV file

Issue for us - Unable to Import simple Text/CSV file
Steps taken:

Open SQL Flat File Import Wizard
Select Database and file to Import
Agree with Columns and Style of data
Click Import
Get long error message (which I cannot copy over to here).

Thanks, Neil Good.
System Info
CPUs	Intel(R) Xeon(R) CPU E5-2673 v3 @ 2.40GHz (8 x 2394)
GPU Status	2d_canvas: unavailable_software
flash_3d: disabled_software
flash_stage3d: disabled_software
flash_stage3d_baseline: disabled_software
gpu_compositing: disabled_software
multiple_raster_threads: enabled_on
oop_rasterization: disabled_off
protected_video_decode: disabled_off
rasterization: disabled_software
skia_renderer: disabled_off_ok
video_decode: disabled_software
viz_display_compositor: enabled_on
viz_hit_test_surface_layer: disabled_off_ok
webgl: unavailable_software
webgl2: unavailable_software
Load (avg)	undefined
Memory (System)	56.00GB (41.76GB free)
Process Argv
Screen Reader	no
VM	0%
Originally posted by @NeilGoodUKABC in https://github.com/microsoft/azuredatastudio/issues/11527#issuecomment-664418020

@Charles-Gagnon I was able to repro the bug using AAD on import extension. This is a shipblocker.

Is this a regression with the latest stable build? If it's a shipblocker why is it in the September release milestone?

Moving to September since this isn't a recent regression according to @aasimkhan30

Hi,
What's the latest on this - I have the same error - any workarounds?

@sj7272, the only workaround for now is to use SQL Auth with the server. I will look into adding AAD support for the next update

@aasimkhan30 I too would like the ability to sign in with AAD, added to the import text,csv wizard.

@aasimkhan30 I too would like the ability to sign in with AAD, added to the import text,csv wizard.

@batsauto, the lastest version of the import extension (1.5.0) in ADS insiders has support for AAD based connections. This version will be released to the stable in October.


Feature request: support for test service

Support for the official VIES test service at http://ec.europa.eu/taxation_customs/vies/checkVatTestService.wsdl would be very useful for unittesting code that utilizes this package.

There's already other great packages like https://github.com/DragonBe/vies that provide this. But I am investigating to maybe make use of these in the core in a future release.

There's already other great packages like https://github.com/DragonBe/vies that provide this. But I am investigating to maybe make use of these in the core in a future release.


chore(ci): build nvidia 1 hour after main

This just starts building nvidia sooner after main than we had previously configured as we chose to move akmods before main, not after.
This PR closes #72

Build failure expected due to PR tag.


Feature to filter deleted tickets from get input incrementally step

Hi,
I have a request. Within our company we have a kettle project where we use your plugin. Recently someone has done some testing against Zendesk and has deleted some tickets. Since then jobs are failing.
I think that I have managed to track it down to the zendesk incremental input step. Within ZendeskInputIncremental.java on line 95, the getTicketsIncrementally method is called. I believe this method corresponds to the following API call of Zendesk (https://developer.zendesk.com/rest_api/docs/core/incremental_export#start-time). Within that webpage it states that tickets that are deleted can be filtered using “status”: “deleted”.
Snippet from the website

Excluding deleted tickets
Starting on October 16, 2016, user-provided information in deleted tickets is scrubbed 30 days after deletion. Reporting on deleted tickets remains intact because the ticket record still exists. However, user-provided information (as opposed to system-provided information) is scrubbed with an X for text fields, a 0 for numeric fields, or nothing for fields that didn't have a value. You can exclude these tickets by filtering out any tickets with a "status": "deleted" value.

It seems that the zendesk incremental input step fetches all ticket ids, and does not filter out deleted tickets. Could you look into this maybe and add such a feature?

I'm hesitant to filter these within the step, but we could add an output field for status, where you can then use a "Filter Rows" step if you'd like to remove those specific tickets.

Merged #44, will work to release an update soon

Released in v1.3.1


Support play verb

Ready for review.

Left a few very small comments. Looks good though.

I've addressed the comments/answered questions.

Null Verb checks removed

:+1: once Travis is happy.


Smart Metering DeviceIdentification length

DeviceIdentification in Smart Metering domain is defined as follows:
<xsd:simpleType name="DeviceIdentification">
  <xsd:restriction base="tns:AlphaNumericType">
    <xsd:length value="17"/>
  </xsd:restriction>
</xsd:simpleType>

But, according to the DLMS COSEM specification DeviceIdentification length should be 16.
Here is the relevant part of the specification (DLMS UA 1000-1:2010, Ed. 10.0, 2010-08-26)

The logical device name is defined as an octet-string of up to 16 octets. The first three octets shall carry the manufacturer identifier. The manufacturer shall ensure that the logical device name, starting with the three octets identifying the manufacturer and followed by up to 13 octets, is unique.

As far as I have come to understand, the logical device name refers to a logical device, of which several can exist in a single physical DLMS device according to the specifications.
The DeviceIdentification refers to the E-meter equipment identifier as used by manufacturers to identify the physical devices.

Can you please provide OBIS code that holds E-meter equipment identifier?
I have Iskraemeco ME382-D1 smart meter and I am not able to find such ID which is 17 chars long.
In the attachment you can find the manual which describes my meter, Chapter 7 explains Identification numbers.
ME382-INSTALLATION-MANUAL.pdf

The OBIS code that holds the device ID for the identifier used as DeviceIdentification would be 0-0:<IP_ADDRESS>.
That would be Device ID 2 from DLMS, which you could find in the Blue Book with the "OBIS codes for abstract objects, general service entries".
This is defined as a 17 character E-meter equipment identifier in the Dutch DLMS companion standard DSMR.

This is exactly the same OBIS code that I have checked and on ME382 it's empty.
Check the attached screenshot.

Our apologies for the late response. We close this issue for now. If you still need any help on this issue, please let us know.


remeasure Lidl HG08131C

I've remeasured one of my lights (single light measurements) with the Shelly PM mini gen3. It still can't measure the standby power/minimum brightness without a dummy load, but the measurement looks much more consistent.
Measurements are in 0.05 W steps with this device and differ quite a bit from the Shelly Plug Gen1:
Hue:

Color Temp:

Checklist

[x] I have created a single PR per device. When you have multiple submissions please create separate PR's.
[x] For lights - I have only included the gzipped files (*.gz), not the raw CSV files.
[x] For lights - I have provided a CSV file per supported color mode. Look that up in Developer Tools -> States

Thanks @RubenKelevra
I can't get the add plot to PR job fixed now.
Has to do with python upgrades.
Should be fixed on master now, but I think the PR is still using old sources.
Will probably be fixed with next PR.

I see there was a problem with the last update profile github action.
Now the updated_at is set to today, that should cause the profile to be redownloaded after HA restart.

Thanks! Just wanted to confirm that :)


Refactor testing to reduce duplication

When I pulled the apache client library out into a separate project, I had to split up the tests which caused duplication in the test code. It would be nice to come up with a "test kit" approach to for the HTTP method support tests so that the same tests can easily be run against different client implementations.

Fixed in development branch.


Using it on new data set.

I tried using the network on 4 new images collected from an Intel Realsense. The output of the model is no where close to accurate. Should the model be retrained again with new data set. Following is the result I got.

another:

Any helpful suggestions on what direction I should move to make it work will be really helpful.

I got better results when looking at an angle, like in the way data was collected at the scene.

How to pre-process the depth image obtained from Intel Realsense to match the data type used in the
example_images ? I get much confused.

Yes, it's ideal to collect data in a similar manner as the synthetic data is collected (mentioned in the paper). If I remember correctly, it's something like .5m - 1.2m above the table/planar surface, and at the edge of the table.
Are your images flipped in the y-direction? We don't expect that, and I think you will get better results if you fix that.

@BHWWS, to pre-process the depth image, just make sure the values are in meters, and that the y channel (of the XYZ point cloud image) is such that the values decrease as the pixel increases (y gets smaller as you travel down from pixel 0 to pixel H-1).
Let me know if that helps.

@chrisdxie I'd like to test my model on OCID and OSD datasets, but how to preprocess the datasets as examples you provide? I see in data_loader.py, the focal length is required. If you also generate npy in this way, can you please provide the focal length you used? Many thanks!


Add torrent sites

Can an adult torrent site be added? or add it myself (requires method)

We are using this API, make a PR to your changes there.


It would be useful if --postTargets would forward args to be used by run-commands

I have a need to forward arguments like issueKey for jira to postTargets, it would be nice if the contents of args would be forwarded like:
npx nx run project:version --args="myArg=myArgValue"
I can't figure out any other workaround for getting cli args in the postTargets, granted I suppose one could use environment variables instead.

This would be really helpful 🙏
Use case I was looking for is passing preid along I can set the tag for ngx-deploy-npm:deploy during prereleases. Opted to not use postTargets here instead


ThreadRuntimeError for CCSD calculations

Hi everyone,
I have encountered some issues when using the ccsd functionality in combination with larger molecules.
The error looks like this:
Traceback (most recent call last):
File "/sw/installed/pyscf/1.6.0-intel-2018a-Python-3.6.4/lib/python3.6/site-
packages/pyscf/ao2mo/outcore.py", line 322, in general
async_write(icomp, row0, row1, outbuf)
File "/sw/installed/pyscf/1.6.0-intel-2018a-Python-3.6.4/lib/python3.6/site-packages/pyscf/lib/misc.py", line 815, in async_fn
self.handler.join()
File "/sw/installed/pyscf/1.6.0-intel-2018a-Python-3.6.4/lib/python3.6/site-packages/pyscf/lib/misc.py", line 738, in join
raise ThreadRuntimeError('Error on thread %s' % self)
10 pyscf.lib.misc.ThreadRuntimeError: Error on thread <ThreadWithTraceBack(Thread-726, started<PHONE_NUMBER>8832)>
, whereas my run script looks like this:
28      sysname='ribose_after_run1'
29     atoms = read(sysname+'_final_geometry.xyz')
30     charge = 0
31     spin = 0
32     basis = 'aug-ccpVTZ'
33     max_cycle = 300
34     conv_tol = 1e-6
35     verbose = 4
36
37
38
39
41     [geo,nuclei,fod1,fod2,included] = xyz_to_nuclei_fod(atoms)
42
43     mol = gto.M(atom=ase2pyscf(nuclei),spin=spin,charge=charge,basis=basis)
44
45     mf = scf.UHF(mol)
46     mf.verbose = verbose
47     mf.max_cycle = max_cycle
48     mf.conv_tol = conv_tol
49
50
51     potentialenergy_hf = mf.kernel()
52
53     potentialenergy_hf *= Ha
88     cc = cc.UCCSD(mf)
89     cc.verbose = verbose
90     cc.conv_tol = conv_tol
91     cc.max_cycle = max_cycle
92     cc.direct = True
93
94     start = time.time()
95     cc.kernel()
96     end = time.time()
The error is thrown after the RHF calculation is done.
For some molecules I calculated, the error did not appear at all, whereas for others, it vanished if I set cc.direct = True. However, for some I have not found any solution, which is why I would be very grateful for any help and suggestions.
Have a nice day
Jakob

What's the problem size?
Without using ao direct CC, all integrals and intermediates, roughly ~N^4*4 bytes, have to be saved on disk in the temporary folder. By default, the temporary folder is /tmp on most linux OS. It may not be big enough to hold the MO integrals.

Thanks for the quick reply.
Ribose is a system with 80 electrons. However,  I thought that ao direct CC was already enabled by setting cc.direct = True in line 92 of my program.

Update: using the pyscf example script  pyscf/examples/cc/15-incore_mo_ccsd_t.py instead of my script I posted here has been promising. However, I would also like to include the calculation of ionization potentials via ccsd.ipccsd(). Is it possible to combine the ipccsd functionality with the example script?

cc.direct is the attribute to enable ao direct CC.
The error traceback does not show the reason of the error. I've improved the traceback and changes will be available in next release (pyscf 1.7).  My guess the problem here is out-of-disk error from h5py.
88-electron molecule seems not a small system for CCSD, assuming the virtual space is roughly 10 times of the occupied space. So that's a system of ~500 orbitals. The requirement of disk space is roughly 200 GB. It should be not a big issue for most modern computers. The calculation here may write the temporary data to the /tmp directory under OS partition, which is typically ~50 GB space in total. It's recommended to assign a different scratch directory for temporary files. It's controlled the environment variable TMPDIR or PYSCF_TMPDIR.

Example script pyscf/examples/cc/15-incore_mo_ccsd_t.py basically put all intermediate integral data in memory. ccsd.ipccsd() supports the same integral data structure. So you can add ccsd.ipccsd() in the code for IP. However, ipccsd (also eaccsd) will generate some intermediates at runtime. The intermediates will be written to disk. Currently, the code does not support to hold the intermediates data in memory.  TMPDIR or PYSCF_TMPDIR trick is still needed to change the temporary location.
A tip in case useful, setting TMPDIR=/dev/shm will lead to temporary data dumped into memory. /dev/shm is a folder virtualized by physical memory. You can use this trick if you need better IO performance and you are sure memory is enough for this system. Then you don't need the intermediates or the tricks of example 15-incore_mo_ccsd_t.py .

Thank you for your advice. I will test the TMPDIR=/dev/shm trick.


Unhandled error with jq: error (at :4): Cannot index string with string "body"

It seems there is an issue with an unhandled error:
INFO: Found no tfplan file. Using input argument.
INFO: Found 1 page(s) of comments at https://api.github.com/repos/<org>/<repo>/issues/236/comments?per_page=100.
INFO: Looking for an existing plan PR comment.
jq: error (at <stdin>:4): Cannot index string with string "body"
INFO: No existing plan PR comment found.

This is what I am using:
- name: "Generate PR comment from Terraform Plan"
  id: terraform-plan-pr
  uses: GetTerminus/terraform-pr-commenter@v3
  with:
    commenter_type: plan
    commenter_input: ${{ format('{0}{1}', steps.terraform-plan.outputs.stdout, steps.terraform-plan.outputs.stderr) }}
    commenter_exitcode: ${{ steps.terraform-plan.outputs.exitcode }}

From the releases I see that there has not been much activity in this repository but I hope this issue can be addressed

I have the same issue, with exactly the following error:
INFO: Found no tfplan file. Using input argument. INFO: Found 1 page(s) of comments at https://api.github.com/repos/<org>/<repo>/issues/18/comments?per_page=100. INFO: Looking for an existing plan PR comment. jq: error (at <stdin>:5): Cannot index string with string "body" INFO: No existing plan PR comment found. INFO: Found 1 page(s) of comments at https://api.github.com/repos/<org>/<repo>/issues/18/comments?per_page=100. INFO: Looking for an existing outputs PR comment. jq: error (at <stdin>:5): Cannot index string with string "body" INFO: No existing outputs PR comment found. INFO: Writing 1 plan comment(s) INFO: Adding plan comment to PR. INFO: Writing 0 outputs comment(s)

Getting the same here

Fairly sure it's something to do with this line (matches the line number and its calling jq): https://github.com/GetTerminus/terraform-pr-commenter/blob/master/utilities/comment_utility.sh#L5

I think I solved this - was permissions related potentially on my secrets.GITHUB_TOKEN token.

I think I solved this - was permissions related potentially on my secrets.GITHUB_TOKEN token.

What exactly did you change?

Ok you're right, I needed to make sure I have
permissions:
  pull-requests: 'write'

It seems to be granted by default, but I was overriding the permissions in the workflow for another reason, and this step was missing the ability to read or write PR comments


IGNITE-15266 Add isclass check to is_hinted

The issue seems to be caused by issubclass requiring its first parameter to be a class, which isn't the case for two-element tuples like the ones used by e.g. object arrays for hinting the object array type ((-1, [...])). Hence, an obvious patch would be to just double check that we are using it with a class.
Performance impact:
$ python -m timeit -r 25 -s "from inspect import isclass; a = int;" -- "isclass(a)"
2000000 loops, best of 25: 133 nsec per loop
$ python -m timeit -r 25 -s "from inspect import isclass; a = int;" -- "isclass(a) and issubclass(a, float)"
1000000 loops, best of 25: 236 nsec per loop
$ python -m timeit -r 25 -s "from inspect import isclass; a = [];" -- "isclass(a) and issubclass(a, float)"
2000000 loops, best of 25: 158 nsec per loop
$ python -m timeit -r 25 -s "a = int;" -- "try:" "  issubclass(a, float)" "except:" "  pass"
2000000 loops, best of 25: 114 nsec per loop
$ python -m timeit -r 25 -s "a = [];" -- "try:" "  issubclass(a, float)" "except:" "  pass"
1000000 loops, best of 25: 333 nsec per loop

Doesn't seem to impactful to me, we can do a million of those in a millisecond.

@bojidar-bg Great catch, Bojidar. Could you please add a test case for it? I've really appreciate your effort and create some tests for your case, please add tests to your patch. Also, we have already imported inspect here, please use inspect.isclass.
Or you can just use my patch. Apply it and commit to your branch. After it, I will merge you PR. Thank you!
Also I'd suggest not to use OBJECT_ARRAY at all, Ignite itself doesn't have a great support of it. Use Collections (simple list) instead.
Add_tests_to_patch.txt

Ah, good spot, didn't notice it was already imported! Thanks for the patch, just went ahead and used it ☺

@bojidar-bg Great, I'll merge a fix today. I cannot give your a promise to release 0.5.2 as ASAP, unfortunatelly. But let's wait for a while, may be others bugs appears, so we can ship a new bugfix release.


[agenda] Can't synchronize the agenda, "No events"

Affected hardware version
Bangle 2
Your firmware version
2v14
The bug
Describe the bug
When opening the agenda app, it's empty, showing a No events message. Can't figure how to sync my agenda.
Forcing the calendar sync does not work:

I go to Settings > Apps > Agenda > Force calendar sync
Click Yes, it shows a Request sent to the phone message
But apparently nothing happens, nor on the bangle or on android.

I've used both the standard F-Droid GadgetBridge and the  GooglePlay Bangle.js Gadgetbridge (as beta tester) did not help.
When using the "Bangle.js Gadgetbridge", I enable this settings: Allow Internet Access and Allow Intents.
Expected behavior

The agenda should open
If it's not initialized, it should tell me to read the README, there should be some instructions on how to initialize.

I volunteer on testing and documenting on the README.
Steps to reproduce

Download the agenda app from the app loader (version 0.02)
Start it. Agenda is empty
In settings, run Force calendar sync. Agenda is still empty.
Change the F-Droid version for the GooglePlay version, try again. Still, empty.
The README should have some instructions

Related

https://github.com/espruino/BangleApps/issues/144
https://github.com/espruino/BangleApps/pull/1884

Ping @gfwilliams

Thanks - I seem to recall that this may not be properly implemented on the Gadgetbridge side? Apparently there was a big refactor in progress and they seemed to have broken calendar sync...
@glemco is this correct?

Yes, that was my understanding at least. I enabled the functionality and it was working fine when I sent a PR about a month ago, after a couple of weeks I rebased and it stopped working. I don't have a pebble so I don't know if the functionality still works somewhere else but from what I could see in the code that wasn't the case.
When I find some time I'll check if they actually managed to get it working, at the moment I just hot-fixed it on my build but I wouldn't go ahead on implementing something that will be wiped out soon.
@DZPM if you really want to test the functionality I can send you the custom build for you to try.

@glemco thanks, but I can wait for the refactor to be ready

@DZPM the functionality has been merged in GB, I don't know how long will that take to be released, but if you can build yourself the apk from the master branch, you can try it.

Closing this ticket because it's fixed: with the new release, it's working properly. Thanks!

Which version of gadget bridge does this work? I have 0.74.0 and I still get an empty list of events

I don't remember exactly, something like 0.69 or so, you should be perfectly covered. Are you sure you enabled the calendar sync in the GB's device settings? (Gear icon on the device).
Have a look at Troubleshooting

Also might be worth checking Gadgetbridge has permissions to access the Calendar events - Android seems to have a habit of removing permissions if you haven't used the app for a while

Thanks both for chiming. I indeed checked all the things you mentioned, and gadgetbridge has access to the calendar, it shows no calendar blacklisted and my bangle has the calendar sync option activated. Still, both apps that I tried show no events, but I have events for today, and the entire week actually in several calendars. Is something else I can check to see if there is anything wrong?

How can I get into the log file? I don't see any option to export such log.

Turn on 'write debug log' in settings, then go to the Gadgetbridge Debug menu and tap Share Log


Illegal invocation

I attempted to use 4.0.0-next.1 of this package with Vite and a Vue 3 component. After I added the import, I received the following error:
index.js:278 Uncaught TypeError: Illegal invocation
    at index.js:278:42

I have another similar project where I used version 4.0.0-next.0 without issue. Any insight would be helpful.

I'm finding this error too in a React Webpack project. I created this test case that replicates the problem: https://stackblitz.com/edit/react-ts-grldcr?file=App.tsx


Please archive obsolete Catena-X repositories 'product-agents' and 'product-agents-edc'

Is your support request related to a problem? Please describe.
Someone could be tempted to develop things against these outdated repos (have been contributed to eclipse-tractusx/knowledge-agents and eclipse-tractusx/knowledge-agents-edc and forked into catenax-ng/tx-knowledge-agents and catenax-ng/tx-knowledge-agents-edc in the meantime).
Describe the solution you'd like
the repos should be archived (such that the artifacts could still be downloaded in beta environments that have not yet been migrated to the 23.09 release)
Later (after the 23.12 release), the repos can be deleted.
Additional context
https://github.com/eclipse-tractusx/knowledge-agents
https://github.com/eclipse-tractusx/knowledge-agents-edc
https://github.com/catenax-ng/product-agents
https://github.com/catenax-ng/product-agents-edc
https://github.com/catenax-ng/tx-knowledge-agents
https://github.com/catenax-ng/tx-knowledge-agents-edc

Hi @drcgjung ,
shall we update the "about" sections accordingly with a link to the replacement/equivalent? If so, can you provide a list to the successors / best matches?

that's a nice hint.
I already did that (at least verbose and pointing to the tx-forks, not as a hyperlink to eclipse-tractusx ... if that would be possible, I would appreciate that)
Would it be possible to update the eclipse-tractusx repos About sections too?
knowledge-agents -> 'Reference Implementations of CX-0084'
knowledge-agents-edc -> 'EDC Extensions for CX-0084'

yes, the tool of the EF also supports changing the description and hyperlinks are also no problem in there.
But I think we should avoid a simple notion of an internal name ("CX-0084") on a public repo - is this specification somewhere publicly available or maybe under another name?

And just to be sure:
catenax-ng/product-agents "refers" to catenax-ng/tx-knowledge-agents, which is a fork of eclipse-tractusx/knowledge-agents. And after this tickets all 3 (6 with the -edc ones) should be archived (and ultimately deleted).
So the whole code is lost after the 23.12 release? Is that the intention?

yes, the tool of the EF also supports changing the description and hyperlinks are also no problem in there.
But I think we should avoid a simple notion of an internal name ("CX-0084") on a public repo - is this specification somewhere publicly available or maybe under another name?

Fair enough, I think the standard will be published with the 23.09 Release under a hyperlink something like that:
https://catena-x.net/fileadmin/user_upload/Standard-Bibliothek/Archiv/Update_September_23_R_3.2/CX-0084-FederatedQueriesInDataSpaces-v1.0.0.pdf
Until the release wave is published, it could be found here
CX-0084-FederatedQueriesInDataSpaces-v1.0.0.pdf
You could refer to it more complete as 'CX-0084 Federated Queries In Data Spaces'

And just to be sure: catenax-ng/product-agents "refers" to catenax-ng/tx-knowledge-agents, which is a fork of eclipse-tractusx/knowledge-agents. And after this tickets all 3 (6 with the -edc ones) should be archived (and ultimately deleted). So the whole code is lost after the 23.12 release? Is that the intention?

God, no. The intention is just to deprecate/archive the catenax-ng/product-agents and catenax-ng/product-agents-edc repositories.
The eclipse-tractusx and their forks will stay forever ... at least will be maintained for 23.12

ok, thank you for the clarification

I would have one last plea ... I know that this was not part of the original description.
The associated maven packages

https://github.com/catenax-ng/product-agents/packages/1877564

https://github.com/catenax-ng/product-agents/packages/1877565

https://github.com/catenax-ng/product-agents/packages/1877566

https://github.com/catenax-ng/product-agents/packages/1877567

https://github.com/catenax-ng/product-agents-edc/packages/1868802

https://github.com/catenax-ng/product-agents-edc/packages/1868806

https://github.com/catenax-ng/product-agents-edc/packages/1868805

https://github.com/catenax-ng/product-agents-edc/packages/1868807

https://github.com/catenax-ng/product-agents-edc/packages/1868808

https://github.com/catenax-ng/product-agents-edc/packages/1868809

https://github.com/catenax-ng/product-agents-edc/packages/1868810

https://github.com/catenax-ng/product-agents-edc/packages/1865120

https://github.com/catenax-ng/product-agents-edc/packages/1868800

https://github.com/catenax-ng/product-agents-edc/packages/1868800

https://github.com/catenax-ng/product-agents-edc/packages/1868799

https://github.com/catenax-ng/product-agents-edc/packages/1868801

would need to be deleted too (else we cannot publish them under the tx- repos, that is a "bug" in github)
Thx much!

Hey @drcgjung! Deleted all the requested packages. Please confirm.

@almadigabor you are the best!!!!


Add a possibility to cancel a request. Resolves #16

Manually tested.
TODO:

[x] Discuss the API
[x] Add tests

Hello 👋 @dhuebner, do you plan to address the review remarks regarding the API? I see this PR has received some other valuable comments...  Thank you!

@dankeboy36
Hi! I can have a look at end of this week, I hope.

@dhuebner we should consider using AbortSignal as suggested in #16. It would be nice to do things the standard way if possible.

@spoenemann

we should consider using AbortSignal as suggested in https://github.com/TypeFox/vscode-messenger/issues/16

Using an own Implementation makes us more flexible in implementing new things and extending existing. I think this is also a reason why vs-code has not re-used AbortSignal but implemented something own.
Other suggestion were applied! The only open conversation is https://github.com/TypeFox/vscode-messenger/pull/17#discussion_r1487438404 where I didn't get your concerns. Maybe we can chat in Around about that?

packages/vscode-messenger-devtools/webview-ui/package-lock.json should be removed – there should be only one lock file in the project root.

This is because devtools package is not part of the main build. I already prepared a branch to add it to the main build. See: https://github.com/TypeFox/vscode-messenger/tree/dhuebner/add_DevTools_toBuild
I will open a PR after this one is merged.

Idea: is it possible to add an adapter class that creates a CancellationToken for a given AbortSignal? I think the concepts are very similar. Internally, it's a good idea to stay compatible with the way vscode-jsonrpc handles this.

Should it be a part of this PR, on can we do it later?

We can do it later, I was just curious of the new API proposed here is already conceptually compatible with AbortSignal, or we need to change something.

@dhuebner could you try this?
export function createCancellationToken(signal: AbortSignal): CancellationToken {
    return {
        get isCancellationRequested(): boolean {
            return signal.aborted;
        },

        onCancellationRequested: (callback: (reason: string) => void) => {
            const listener = () => callback(String(signal.reason));
            signal.addEventListener('abort', listener);
            return {
                dispose: () => signal.removeEventListener('abort', listener)
            }
        }
    };
}

@dhuebner could you try this?

Works, I also added an additional test.


⚠ Samsung has degraded performance

In 47c61ff, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2410 ms

Resolved: Samsung performance has improved in bc8c931.


New version(0.5.0) not working with maven plugin (delete pitest-maven as dependency )

The pitest-maven dependency has been added in version 0.5.0 (line 107 in the pom), it's a plugin and not a library.
<dependency>
      <groupId>org.pitest</groupId>
      <artifactId>pitest-maven</artifactId>
      <version>1.4.11</version>
</dependency>

I tryed to use the new version 0.6.2 with my maven plugin for CK, but it's not possible to compile anymore, because pitest-maven uses old maven libraries. ERROR : The API of the mojo scanner is not compatible with this plugin version.
I think it's very important to remove this dependency(too many sub-dependencies), but not the plugin, pitest is very cool to validate the tests.

That's indeed a problem!
I hope PR #75 will fix it!

Yes this PR will fix it, Thank you


Nestjs implementation

@tmkn @tobiaslins  @mxstbr
It is possible to implement this library into Nestjs? like others Passport strategy through @nestjs/passport or I need other way?
thanks in advance
Alberto

Bro, did you find how to do it?

@fred76 can you maybe provide your solution for all, who are using NestJS. This would be awesome!

Found it useful for this case: https://www.youtube.com/watch?v=69QmlAXNgHk


Fixes for issues #18 and #19

Do not apply this pull request until the corresponding p4c pull request has been merged as well.

i've applied manually the other patch for p4c.


1.0.0-alpha1 Nuxt with Koa

According to the Nuxt.js codes of 1.0.0-alpha1, initialization of nuxt has been changed into asynchronous.
So how can we use koa in server side codes, should I wrap all koa app.use inside  nuxt.then ?
const nuxt = new Nuxt(config)
nuxt.then((nuxt) => {
  nuxt.build()
  app.use(async (ctx, next) => {
    // ...
    await next()
    // ...
  })
  app.use(async (ctx, next) => {
    // ...
    await next()
    // ...
  })
})

The nxut/koa starter has been updated :+1:


refactor(oxc): add napi feature, change napi parser to use oxc crate

#6265  👈
main

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

Join @Boshen and the rest of your teammates on  Graphite


Error: While trying to resolve module react-native-device-info after installing v4

Bug
After installing v4, I get the following error. Anyone else get this?
┌──────────────────────────────────────────────────────────────────────────────┐
│                                                                              │
│  Running Metro Bundler on port 8081.                                         │
│                                                                              │
│  Keep Metro running while developing on any JS projects. Feel free to        │
│  close this tab and run your own Metro instance if you prefer.               │
│                                                                              │
│  https://github.com/facebook/react-native                                    │
│                                                                              │
└──────────────────────────────────────────────────────────────────────────────┘

Looking for JS files in
   /Users/sam/Projects/test-mobile-client

warning: the transform cache was reset.
Loading dependency graph, done.
error: bundling failed: Error: While trying to resolve module `react-native-device-info` from file `/Users/sam/Projects/test-mobile-client/js/env.js`, the package `/Users/sam/Projects/test-mobile-client/node_modules/react-native-device-info/package.json` was successfully found. However, this package itself specifies a `main` module field that could not be resolved (`/Users/sam/Projects/test-mobile-client/node_modules/react-native-device-info/src/index.js`. Indeed, none of these files exist:

  * /Users/sam/Projects/test-mobile-client/node_modules/react-native-device-info/src/index.js(.native|.ios.js|.native.js|.js|.ios.json|.native.json|.json|.ios.ts|.native.ts|.ts|.ios.tsx|.native.tsx|.tsx)
  * /Users/sam/Projects/test-mobile-client/node_modules/react-native-device-info/src/index.js/index(.native|.ios.js|.native.js|.js|.ios.json|.native.json|.json|.ios.ts|.native.ts|.ts|.ios.tsx|.native.tsx|.tsx)
    at ResolutionRequest.resolveDependency (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/node-haste/DependencyGraph/ResolutionRequest.js:65:15)
    at DependencyGraph.resolveDependency (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/node-haste/DependencyGraph.js:282:16)
    at Object.resolve (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/lib/transformHelpers.js:267:42)
    at dependencies.map.result (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/DeltaBundler/traverseDependencies.js:426:31)
    at Array.map (<anonymous>)
    at resolveDependencies (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/DeltaBundler/traverseDependencies.js:423:18)
    at /Users/sam/Projects/test-mobile-client/node_modules/metro/src/DeltaBundler/traverseDependencies.js:275:33
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/DeltaBundler/traverseDependencies.js:87:24)
    at _next (/Users/sam/Projects/test-mobile-client/node_modules/metro/src/DeltaBundler/traverseDependencies.js:107:9)
 BUNDLE  [ios, dev] ./index.js ▓░░░░░░░░░░░░░░░ 6.3% (31/243), failed.

Environment info

React native info output:
 // paste it here

Library version: x.x.x
Steps To Reproduce

Install v4
react-native start --reset-cache
...

Describe what you expected to happen:

Reproducible sample code

After upgrade react native version 0.61.2, I got this issue on metro bundler loading.
Could someone let me know how to fix this issue asap?
Thanks
Looking for JS files in
/Users/sam/Projects/test-mobile-client
warning: the transform cache was reset.
Loading dependency graph, done.
error: bundling failed: Error: While trying to resolve module react-native-device-info from file /Users/sam/Projects/test-mobile-client/js/env.js, the package /Users/sam/Projects/test-mobile-client/node_modules/react-native-device-info/package.json was successfully found. However, this package itself specifies a main module field that could not be resolved (/Users/sam/Projects/test-mobile-client/node_modules/react-native-device-info/src/index.js. Indeed, none of these files exist

You may need to do a deeper clean, everyone reporting this, please run a clean from the react-native-clean-project package (the only one I trust...)
This comes up often enough in other packages I maintain and use that at this point my only 100% reliable CI builds come when I call 'deep-clean' which is defined in my package.json like so
    "deep-clean": "./node_modules/.bin/react-native-clean-project --keep-node-modules --remove-iOS-build --keep-brew --keep-pods --remove-iOS-pods --remove-android-build && yarn clean",
    "clean": "\\rm -fr ./node_modules && \\rm -fr dist/* && \\rm -fr ios/build ios/Pods ios/KScoreApp.xcarchive && \\rm -fr android/build android/app/build",

Every single part of it that does a delete is important, and the parts that interact between react-native-clean-project and manual deletes are tuned carefully to (for instance) delete local pods but not delete your system's entire pod cache. So it should be fairly performant as far as full purges go.
I can't reproduce this on the example or in my app so my inclination would be to close this but of course there may be a real issue so please purge out all your caches and report back
Thanks!

(if you want to keep it simple and are on a fast network you can just npx react-native-clean-project clean-project-auto and be squeaky clean as well)

Hi, Mike
Thanks for your kindly answer.
I hope it will help me.
:)

Unrelated I just ran the example clean trying to reproduce things and I could not, it appears to be a project-specific issue

I have exactly the same problem. It looks for src\index.js but the package only has a TypeScript file there: src\index.ts.
So, the problem seems to be that react-native looks for .tsx files and not for .ts?

This still strikes me as some sort of caching. I continue to recommend my cleaning method outlined in detail above. If someone will confirm that they have BOTH run the clean as I mentioned above and are still seeing the problem, then we have a problem.
But if that's the case I'll need a lot more details - npx react-native info output at minimum plus the version of react-native-device-info in use and that you've run the clean
But we have a transpilation step from typescript to javascript prior to upload, so there is an index.js and it should look for it if it needs to https://github.com/react-native-community/react-native-device-info/blob/master/package.json#L7

Hi @mikehardy, I have the same problem as @afilp, the src directory in node_modules/react-native-device-info does not have index.js. I'll do a clean install to if I can reproduce

Ok, my bad, sorry everyone.
Please use npm install<EMAIL_ADDRESS>--save instead of npm install react-native-device-info --save.
The later installs version 4.0.0 which for some reason fails with above. So far 4.0.1-rc.1 is working great for me, thanks all.
I'll be closing this...

Ah ha! That's not really your bad, we had a release issue where we thought typescript (the other big change in v4) was ready to go but as you found it was not because of the issue you described. We unpublished that on npmjs.com so it should not have gotten you, but either referencing react-native-device-info@next or @4.0.1-rc.1 would work.
Just to put a nail in it, I haven't gotten any other bug reports so I'm going to make an official release of 4.0.1 today that will bring everyone past the broken 4.0.0, and I feel more confident doing so since you reported success with the new rc.1 - thank you!

I just came across with this issue and I noticed I was missing ts in my metro.config.js
module.exports = {
  resolver: {
    sourceExts: ['jsx', 'js', 'ts'],
  },
};

That's an interesting point, but it should not be necessary? The react-native template does not come with that: https://github.com/facebook/react-native/blob/master/template/metro.config.js
yet the example (which I auto-refresh via script so it always gets just the template) works?  https://github.com/react-native-community/react-native-device-info/blob/master/refresh-example.sh (and indeed it is the raw template file, no .ts in there https://github.com/react-native-community/react-native-device-info/blob/master/example/metro.config.js) 🤔

Well, when I started the project, like 3month ago, I had to make some adjustments to the metro config file. I guess I just forgot to add the ts extension back then.

@ricbermo that's the opposite of what I mean - there should be no need for the '.ts' extension. It is not in the template, it is not in our example app, yet the example app works.

@mikehardy maybe RN loads TS files by default and once I adjusted my metro config to load only JS and JSX files, v4.0.1 stopped working for me.

@ricbermo good thinking! I checked and I think you're right - as long as you don't override it to a specific list (which excludes .ts), the default will include it apparently https://github.com/facebook/metro/blob/master/packages/metro-config/src/defaults/defaults.js#L52


Missing shared library when upgrading to AGP 7.1+

Upgrading to AGP 7.1+ causes the openxr_loader shared library to go missing from the generated Android library AAR binary.
Repro steps:
Expected behavior:

Checkout the project using the build instructions and run the command to generate the shared libraries:

./gradlew generateSharedLibs

This should generate two shared libraries in the android/build/outputs/sharedLibs/release/arm64-v8a directory: libopenxr_loader.so and libgodot_openxr.so.

Buggy behavior:

Delete the android/build directory: rm -rf android/build
Navigate to config.gradle and update the gradlePluginVersion to 7.1.2
Navigate to gradle/wrapper/gradle-wrapper.properties and change the version of gradle to 7.2: distributionUrl=https\://services.gradle.org/distributions/gradle-7.2-all.zip
Run the command to generate the shared libraries again: ./gradlew generateSharedLibs
Notice only 1 shared library is generated this time in the android/build/outputs/sharedLibs/release/arm64-v8a directory: libgodot_openxr.so

I can confirm that this issue is still reproducible with AGP 7.2.1 and Gradle 7.4.2:
diff --git a/config.gradle b/config.gradle
index 17ecf55..d890471 100644
--- a/config.gradle
+++ b/config.gradle
@@ -1,6 +1,6 @@
 ext {
     versions = [
-            gradlePluginVersion: '7.0.3',
+            gradlePluginVersion: '7.2.1',
             compileSdk         : 30,
             minSdk             : 19, // Also update 'android/src/main/AndroidManifest.xml#minSdkVersion' value
             targetSdk          : 30, // Also update 'android/src/main/AndroidManifest.xml#targetSdkVersion' value
diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties
index a0f7639..92f06b5 100644
--- a/gradle/wrapper/gradle-wrapper.properties
+++ b/gradle/wrapper/gradle-wrapper.properties
@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-7.2-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.4.2-all.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists


Update readme.md with newest cypress github action version

Update cypress github action version to the latest major version.
I didnt change the actual .yml files, assuming you will want to test it out on your own before merging PR from a random person on the interwebs :)
I can tell that we have been using cypress-split with cypress github action v6 for couple of weeks and it works good so far.

Super, thanks for submitting this fix


[BUG] when launching phal : ntpd[1065]: must be run as root, not uid 1000

OpenVoiceOS specifications (please complete the following information):

Device: Mark2-Devkit
Image release: 20230117

Describe the bug
When launching the systemd phal component we have this:
mycroft@OpenVoiceOS-e383a2:~ $ tail -f /var/log/mycroft/phal.error.log
Connection Refused. Is Messagebus Service running?
Exception closing websocket at ws://<IP_ADDRESS>:8181/core: [Errno 111] Connection refused
Message Bus Client will reconnect in 10.0 seconds.
 * Running on http://<IP_ADDRESS>:36536/ (Press CTRL+C to quit)
/bin/sh: line 1: service: command not found
 4 Mar 19:42:04 ntpd[937]: must be run as root, not uid 1000
/bin/sh: line 1: service: command not found
/bin/sh: line 1: service: command not found
 4 Mar 19:42:08 ntpd[1065]: must be run as root, not uid 1000
/bin/sh: line 1: service: command not found

Thanks, I have also seen this. I will look into it. Switching it over to use timesyncd of systemd. That one will allow for better user control.

For this to resolve we need this PR merged first;
https://github.com/OpenVoiceOS/ovos-utils/pull/200#issuecomment-1913171531


Switch to psutil for system reads

Consider switching to psutil to standardize system reads:
https://pypi.org/project/psutil/

Closing as not a priority.


Adding back hbase client

Snyk is flagging a vulnerability with the hbase client's use of org.jruby.jcodings:jcodings:1.0.8. Looks like it's a false positive though -- the vulnerability was reported against JRuby 1.4.0 and below, which used jcodings 1.0.1. The fix (in JRuby 1.4.1) was to upgrade to jcodings 1.0.3. Snyk is flagging versions of jcodings earlier than 1.4.1 though, which is to say all of them (as the latest release is 1.0.26).


Reorganize requirements, make albumentations optional

The albumentations requires an old version of the imgaug library, which is incompatible with a lot of other software stacks (namely the one I'm using) which uses the latest imgaug library.
It looks like albumentations is not tightly coupled with the system and mmdet can work just fine if it isn't installed. Therefore I'd like to propose to make it an optional requirement instead of a forced requirement.
This PR does that and also reorganizes the requirements into a format that should be easier to use. Namely, requirements are broken down by what they are required for (e.g. runtime, tests, build, and options) and they all live in a requirements folder now. The base requirements.txt file uses the -r syntax to refer to each of those files, so pip install -r requirements.txt will install all requirements.
I added an enhanced version of the parse_requirements function being used in the setup.py script, which handles the -r syntax. I also used extras_require to register each type of requirements separately. So by default when you pip install mmdet you will only get the runtime requirements, but if you pip install mmdet[all] you will get everything. Similarly you can pip install mmdet[optional] to only grab runtime and optional requirements.

Awesome! The installation guide may also be updated accordingly.

I modified INSTALL.md and made imagecorruptions optional.

I rebased on the latest master.


slidemap save() and save_umap() encountered zip file problems in Databricks

Description

Hi James, I mentioned this in Databricks compatibility request but just to better track this issue. I encountered zip file issue again when running slide_map.save_umap('path') even after I defined the environment variable. I believe SlideMap.save() has the same issue.
To Reproduce
Steps to reproduce the behavior:

slidemap=features.map_activations()
slidemap.save_umap('path')
slidemap.ave('path')

Expected behavior

Environment:

Slideflow Version (e.g., 1.0): 2.0.3
OS (e.g., Ubuntu): Ubuntu in Databricks
How you installed Slideflow (pip, source): pip
Python version: 3.10
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Additional context

Perfect! Thank you James.

Unfortunately, this isn't something that can be easily patched. The culprit issue is the np.savez() functions in SlideMap.save_umap() and SlideMap.save_encoder. In both cases, np.savez() is used to save the UMAP range and clipping information (values used for normalizing the UMAP-transformed data into the range 0-1). This information is saved to an npz file, storing the range information under the key range and the clipping information under the key clip.
The range and clipping information is necessary when load a saved UMAP object from file, otherwise the UMAP object will no longer be able to transform data in the same way.
The npz file format uses zipping under the hood, which is why it's incompatible with databricks.
For the index files, I was able to switch from npz files to npy files, which don't use zipping. Unfortunately, npy files can only store a single array - and in this case we need to store two: the range information and clip information. So you won't be able to save fully save the UMAP encoder on a file system that doesn't have zip support.
You can, however, still save an image of the UMAP and the coordinates for all shown tiles. Instead of using SlideMap.save(), use SlideMap.save_plot() to save only an image of the UMAP. Use SlideMap.save_coordinates() to save the coordinates for tiles shown on the plot.
I hope that's helpful, and sorry we can't make that functionality fully compatible with the databricks system.

Hi James, it's okay. I was able to output UMAP coordinates and plots. So it's not a big deal. Thanks for looking into it.


CLOUDSTACK-10283: Sudo to setup agent keystore, fail on host add failure

This would make keystore utility scripts being executed as sudoer
in case the process uid/owner is not root but still a sudoer user.
Also fails addHost while securing a KVM host and if keystore fails to be
setup for any reason.
@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1715

Trillian test result (tid-2258)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26556 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2454-t2258-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_host_maintenance.py
Smoke tests completed. 67 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

Tests and reviews are ok. I will merge this one then.

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✖centos7 ✔debian. JID-1731

@rafaelweingartner can you kindly squash merge in future, otherwise, we'll have commit+merge commits on the branches that make the git graph look weird and difficult to track changes.

I am sorry @rhtyd, but I find using merge commits when merging PRs a good idea. We do not have a protocol for merges yet (use or not use the merge commit). There was a thread a while ago discussing this topic in [1]. Some people preferred maintaining the merge commits when merging a PR (myself included). However, the thread died after a while, I believe the author (Daan) was busy with something else to finalize the discussion and create something formal.
[1] http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201801.mbox/<CAGQtxva9xnipUXvc56wA4NT5fuwLXzR3cpD%3DFf1mO%3DxZYoKQ7w%40mail.gmail.com>

@rafaelweingartner feel free to rekick the discussion and start a vote. I've shared my preference with you which is based on two things - (a) allows git-users like myself who prefers to keep track of changes using command-line, (b) for maintainers who would want to backport a feature/fix which is easier if every fix/pr is a single commit than bunch of them.


Vehicle

Your mod is not allowing the vehicle to appear and be used that is our purchased from the kiosk

Details
No technical details other than the vehicle shows after disabling your mod.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
No
Reproduction Steps
Haven't made a new game to see if this can be recreated

You have an outdated version of the SFCP via the Creations platform. Fully uninstall the mod, then reinstall it.


Maturin generated CI not pushing binaries to PyPI

Hey, I have a project that uses PyO3, and I used the command maturin generate-ci to create the GH actions workflow.
It had worked in the past, but for some reason the last step of the workflow fails silently, there is no error message so I'm not sure what to do.
This is the workflow:
https://github.com/shner-elmo/flashtext2/actions/runs/9792664428/workflow

Have you used a tag?
See: https://github.com/shner-elmo/flashtext2/actions/runs/9792664428/workflow#L104

You have modified the generated workflow: https://github.com/shner-elmo/flashtext2/commit/0f4e1db7176874af40bf9395ed6e753f8a1fca4d


Request: logback setup with module-info.java

Love the site, very helpful, nearly always first page results in google.
I followed the logback tutorial and have a module-info.java with requires logback.core.  When I build in maven I get
[WARNING] ***********************************************************************************************************************************************
[WARNING] * Required filename-based automodules detected: [logback-core-1.2.11.jar]. Please don't publish this project to a public artifact repository! *

Please update the tutorial to mention what should be done and I will follow this ticket for updates.
Thank you!

ah, intellij did it wrong and auto-filled in with 'logback.core' instead of 'ch.qos.logback.core'

Hi @i-make-robots,
Glad you like the site! And good you already found an issue :)

If you could write something about how to diagnose serviceloader problems, that would be great.  for me they fail silently and I never know which part is wrong - the services descriptor? the module info?  the version implemented?  ugh.


Rediscovery services and mdns

I use this package to manage some hardware , my source is her https://github.com/ErwanMAS/godfrey4home/blob/master/src/homekit-switch.go .
After registration , and a long time .
When i restart , the bridge ,  my iphone can not reconnect to the hk bridge .
I ran some tcpdump on 5353 , and i saw that the phone always made a query  on _hap._tcp.local. PTR .
The mdns hc bridge provide by /brutella/hc/ , does not reply , with a answer .
If i comment this line , i have what i think the good behavior, the bridge reply with a response to himself .
https://github.com/brutella/dnssd/blob/61d1bc6e53ace5b3d9ecb46ffd0b3158c00ae9d0/responder.go#L437
I think the object req.msg.Answer , must not store answers from his own local services .

This mechanism is described in RFC 6762 7.1 as Known-Answer Suppression.

A Multicast DNS responder MUST NOT answer a Multicast DNS query if
the answer it would give is already included in the Answer Section
with an RR TTL at least half the correct value.

Do you have any reference that the responder always has to respond with a message including its own local service, even though the answer is already in the Answer Section?

i dont have more insight, or test case , you can close , if you want .

My last version run only with my pull request ( #12 )

#12 is now merged


Roll deps to pick up fix in archive

Pick up fix for vulnerability https://github.com/advisories/GHSA-9v85-q87q-g4vg
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

A few things:

rolling these deps looks fine
this PR doesn't update package:archive, it's pinned because of https://github.com/flutter/flutter/issues/115660
I believe I have worked around this in the tool with https://github.com/flutter/flutter/pull/96565. Meanwhile, I am actively working on https://github.com/flutter/flutter/issues/133371, which is eliminating the use of package:archive altogether.

Thanks; closing this as we're tracking the flutter_tools pin in https://github.com/flutter/flutter/issues/134037


Will markless support table like GFM?

GFM support table like this:
| field  | value  |
|--------|--------|
| field1 | value1 |
| field2 | value2 |

will be:

field
value

field1
value1

field2
value2

No. Tables in ASCII are a horrid mess, both to read, parse, and specify.


PR Page UI Inconcistency

(really not sure how to name this)
When you open the page for a pull request, where it asks you how to merge the pull request you have a dropdown list:

The font family doesn't look right, and the dropdown itself doesn't seem to follow anything else in the UI.

Extension version: 0.60.0
VSCode Version: 1.76.1
OS: Windows 11

Steps to Reproduce:

Open the page for a pull request that has no merge conflicts and is ready to merge
The dropdown where you can select method of merging is in a different font.

Closing as as-designed since this is just the native dropdown.


Original value with change event

Is it possible to pass an original value with change events and keep a default event object?
With this way, @change="onChangeEvent(id)", I can't get default 'value'.

I don't think it is possible.

So if I wanna pass index and status value, the code is going to be like this?
<toggle-button v-model="status" @change="onChangeEvent(index, status)"/>

Pass it where? I do not understand what you are trying to do tbh, if you want to use an event from the @change you can get by using $event:  @change="myfunctions($event)". I do not know what indexis and why you need to passstatusin onChangeEvent (because you already have it your model, you can just usethis.statuswithinonChangeEvent`).
Check out these docs, they might be useful:
https://vuejs.org/v2/guide/events.html#Methods-in-Inline-Handlers
https://vuejs.org/v2/guide/components.html#Using-v-model-on-Components

Sorry I didn't make clear.
I need to use both an index of v-for and toggle status but I didn't know how to pass them.
The answer I wanted is @change="myfunctions(index, $event)".
Thanks for the docs refferences.  I need to study the basic of Vue.js more.
Sorry for bothering you and thanks for your kinds!

Great, happy to help 👍 Thanks for using plugin!


Clicking on 'right-arrow' or 'Next' jumps a few pages instead of 1

Seems there's a bug when clicking on 'right-arrow', for me it's jumping a few pages.
Also, I need to use the :limit option but also change the '»' arrows to 'Next' which seems impossible currently, I might dive into the plugin code to enable this, and would also like to add optional 'First' and 'Last' pager buttons...
Please let me know if what seems to be a bug is, or I'm doing something wrong in my settings. thanks!

Also noticed the 'Next' button disappears and the 'Prev' appears after a few clicks, before it reaches the end of the list..

Seems there's a bug when clicking on 'right-arrow', for me it's jumping a few pages

Can you please show me your settings so I can reproduce it?

Change the '»' arrows to 'Next' which seems impossible currently

I'm currently working on that; the fix will be released soon (related to issue #27)

Add optional 'First' and 'Last' pager buttons

Can you please elaborate more on this? The limited paginateLinks already shows them, but they're not optional, obviously.

Ah, now I see what you mean by jumping a few pages. This is actually how the plugin works, but I think you're totally right. The arrows should jump one page at a time, and the ellipses should reveal the next chunk of links. I'll include this fix in my next release.
Thank you so much for using the plugin and for filling this issue :).

Can you please elaborate more on this? The limited paginateLinks already shows them, but they're not optional, obviously.

I was talking about having designated buttons 'first' & 'last' at the edges of the pager.. but if the first and last numbers are always shown I guess that should be enough..
Regarding the other issues, thanks for looking into it and fixing.. I'll be waiting on an update :)

In the latest release, I've added a new feature called step links which should solve both of your problems.

right on man I'll check it out.. thanks for your work !


Fix Markdown for CustomResourceDefinition task

Fix Markdown for https://k8s.io/docs/tasks/access-kubernetes-api/custom-resources/custom-resource-definitions/#advanced-topics
Fixes #20937

This is a bit rough-and-ready; once issue #20780 is dealt with, we could go back and tidy this page once again.

Page preview:
https://deploy-preview-20976--kubernetes-io-master-staging.netlify.app/docs/tasks/access-kubernetes-api/custom-resources/custom-resource-definitions/

@sftim Thanks for this. Agreed that it's worth revisiting post-#20780.
/lgtm
/approve


test: test

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Broken breadcrumb from BREAD management

Version information

Laravel: v5.6.28
Voyager: Installed 2018-07-31 (HOW TO FIND THE VOYAGER VERSION???)
PHP: 7.2.7
Database:  MySQL 5.7.22-0ubuntu18.04.1

Description
Broken link in the breadcrumbs nav. It throws a Symphony MethodNotAllowedHttpException.
Steps To Reproduce
Install Voyager with dummy data. Use the GUI to navigate to
/admin/bread/categories/edit
But don't edit any fields or save any changes. Just go to that page.
Now click on the breadcrumb at the top of the pages that says "Categories". See the exception error.
Expected behavior
No error.
Screenshots
Additional context
I'm loving Voyager, thanks! But if you require people to know their Laravel and Voyager version numbers to file bug reports, please tell us how to find the Voyager version number. I couldn't find that info with a quick google or a grep -R.
Also, maybe include a note in the bug intro comments that the Laravel and PHP versions can be obtained with "php -v" and "php artisan --version", and MySQL with "mysql -v".

Its written in the footer ;)

Oddly, I cannot see the footer ("Made with love by The Control Group...") unless I manually scroll down any /admin/ page with my mouse. Even if I view it full screen at 1920x1200px, the footer is hidden below the bottom of my screen.
It's like they hid the version number on purpose! :)

Just hiding unnecessary things to have room for stuff that really matters ;)


nuclei: 3.2.5 -> 3.2.6

Diff: https://github.com/projectdiscovery/nuclei/compare/v3.2.5...v3.2.6
Changelog: https://github.com/projectdiscovery/nuclei/releases/tag/v3.2.6
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 309161 run on aarch64-linux 1

1 package built:

nuclei


Metal support?

Hi, awesome work on this project!
I'm building some Swift apps using llama.cpp, and I'd love to try getting clip.cpp running on my app too.
I'm curious if you're going to support running clip.cpp on Metal like llama.cpp?

Hi @kchro3, thanks for reaching out! It needs Conv2D implementation on Metal. I'm currently implementing LLaVA with llama.cpp and clip.cpp combined (initial release in a few days). The CLIP part will run on CPU in the first place, but then I'll implement Conv2D on Metal to support end-to-end Metal inference for multimodal models.

wow, that's awesome. i'm looking forward to the release!

where can i follow your progress btw? will it be in this repo or are you going to make a new one

WIP PR for the initial release of large multimodal model inference is in another repo. Metal implementation of Conv2D will require another pr to the main GGML repo after this PR is merged and released.

awesome work, @monatis ! cheering you on

Hi, awesome work on this project!
I'm building some Swift apps using llama.cpp, and I'd love to try getting clip.cpp running on my app too.
I'm curious if you're going to support running clip.cpp on Metal like llama.cpp?
welcome to my project, support all openai clip and ofa-sys chinese clip model,at the same time  support metal gpu
and f16,q8_0 quantization
https://github.com/yysu-888/clip.cpp


How to change name(John Duo) and employeer designation

Hello I add one image in header also i want to change the header name and designation where is its support?

Just add your own header layout to the menu view.
<nl.psdcompany.duonavigationdrawer.views.DuoMenuView
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:tag="@string/tag_menu"
    app:footer="@layout/view_footer"
    app:header="{your layout}"
    ... />

ok now i understand but how to call this layout and past my own text in it dynamically...

This is the view you want to adjust:
https://github.com/PSD-Company/duo-navigation-drawer/blob/master/duo-navigation-drawer/src/main/res/layout/duo_view_header.xml

No I did not
On Wed, Dec 27, 2017 at 2:57 PM, Alexander Pot<EMAIL_ADDRESS>wrote:

Closed #14
https://github.com/PSD-Company/duo-navigation-drawer/issues/14.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PSD-Company/duo-navigation-drawer/issues/14#event-1402332496,
or mute the thread
https://github.com/notifications/unsubscribe-auth/API8C1OrRaxH8q1EVJA7y2BVpmbB-dDZks5tEhSMgaJpZM4Qsv8y
.


⚠ Bofrost.fr has degraded performance

In 9445daf, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1292 ms

Resolved: Bofrost.fr performance has improved in 31c01dc after 13 minutes.


Blur Filter not Working in Chrome Version 62

Hi I'm running Chrome : Version 62.0.3202.75 (Official Build) (64-bit)
I've recently been working on a liquid simulation using PixiJs and LiquidFun.
At work I'm using a MacBook also running Chrome and it's all running well, now I just opened the Codepen from home and it seems that none of the filters are working. When I open the same project in Firefox, everything works as expected, same goes for Chrome on my Android Device. So it's just broken on my PC, and there only in Chrome (Even Edge runs it correctly 😱 )
Can anybody else confirm this?
Edit: I should have probably mentioned that I'm running on a regular Windows Machine at home (Win 10 - 64Bit - Nvidia Graphics Card)

Running Chromium 62.0.3202.75 on Linux I have no issues with the blur filter, it works as expected.

Running Chrome Version 60.0.3112.90 (Official Build) (64-bit) on Ubuntu
Example not working:
http://pixijs.io/examples/index.html?v=v4.0.2#/filters/blur-filter.js
However it's fine on FF

That example is oscilating even on my GTX 620. Why do you use 4.0.2?

@ivanpopelyshev I changed to see if the problem  persists on different versions.
v4.6.0 not working as well.
All the little dudes are crystal clear, no blur at all!

I'm using V4.6.0 (using this link "https://pixijs.download/dev/pixi.js")

I looked at console on chrome ( where there is problem rendering filters) and pixi's hello function says it's "canvas" while on firefox it says "webgl".
When I force it to webgl it works fine on chrome, and when I force it to Canvas, it doesn't work on FF either.
why autodetectrenderer detects canvas for chrome?

Because chrome says so. Try https://threejs.org/examples/ in chrome

on Linux , maybe you need some arguments for booting chrome :
--enable-webgl --enable-3d-apis --ignore-gpu-blacklist
otherwise you can't use WebGL.


the procedures of compiling a C project using Cmake/Cmake tools and Vscode

Are there any tutorials/procedures explaining compiling a C project (including multiple C file and header file) using CMake and VScode. This tutorial posted only covers compiling a single file. Thanks.

You can also explore these documents from our GitHub repository:
https://github.com/microsoft/vscode-cmake-tools/blob/develop/docs/configure.md
https://github.com/microsoft/vscode-cmake-tools/blob/develop/docs/build.md
Let us know if there is anything that you can't find and we will work on improving our general documentation.


Move AllowClientRenegotiation tests

Move them from test_renegotiation to renegotiation in ssl_new

Not passing :(  Any clue appreciated.

Rebased.  All tests pass.  Added a commit to remove the "-immediate_renegotiation" flag and support, since it is not being used because the tests is now done in the SSL 17-renegotiate.cnf tests as @mattcaswell requested.

richsalz requested review from mattcaswell and kaduk 11 hours ago

FWIW, I don't expect to be able to put time into review this week

LGTM.
Ping for second review.

@kaduk, I didn't really expect you to do a full review as we both work for Akamai.  It was just to see that your drive-by comment was exactly on-target.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master. Thank you for the contribution.


Make update notifications less intrusive

It seems like VS Code has some kind of update each time I run it, which is great. However, the big notifications break my workflow. I would love it if the notifications could get out of my way.

For me the notification has no use. I always click the button 'Later' and the update gets installed silently when I close/reopen at some time. It is a text editor, not a banking app that needs the latest security patch.

The notification is rather big, so (for me) hard to ignore. Even if I could resist it I am forced to do it because it covers the tabs in the editor.

If there are release notes, then I want to read those when it suits me. I don't want to interrupt something I'm working on to check the release notes and then figure out what I was supposed to be doing.

What I would like is there to be some kind of small unobtrusive notification (like the flag in regular Visual Studio) that would tell me that there is an update. It could also link to the release notes (after an update) so that I can leave it flagged until I have time to check out the new features. Or just have an option to turn off all notifications related to updates, and always install them silently.

Thanks, we're exploring this here: https://github.com/Microsoft/vscode/issues/22388


No "escape hatch" for inserting raw HTML into a node

I'm interested in rendering Markdown files using this library, but reading the code it seems impossible, even if I use a Widget or a Hook.

Have you tried innerHTML?

Oh, I'm so embarrassed. I forgot that virtual-dom worked by setting properties on the DOM node itself. Thanks for your help!

No worries :)


User Sessions shared across multiple users behind Proxy or VPN

Describe the bug
When multiple users access the Chainlit application from the same IP (e.g., when behind a proxy or VPN), the user_session is shared across those users, leading to data from one session being accessible by another. This results in a mixing of session data, causing unexpected behavior, such as retrieving or storing incorrect session-specific information.
To Reproduce
Steps to reproduce the behavior:

Deploy a Chainlit application and expose a specifc port to the internet.
Have two or more users access the application through a proxy or VPN (e.g., an office network).
Each user sets and retrieves session-specific data using cl.user_session.set() and cl.user_session.get().
Observe that session data from one user is accessible by another.

Expected behavior
Each user should have a unique session, even when accessing the application from the same IP address, ensuring that session-specific data remains private to each user.
Screenshots
If applicable, add screenshots to help explain the problem.
Desktop (please complete the following information):

OS:
Browser:
Version:

Smartphone (please complete the following information):

Device:
OS:
Browser:
Version:

Additional context
The issue seems to stem from how user sessions are identified and stored. In the current setup, it appears that sessions are identified solely by the session ID, which may not be properly isolated for different users behind the same IP. As a result, users sharing the same IP (e.g., through a corporate proxy or VPN) can end up sharing the same session, causing a mix-up of session data.

#1442 is probably related.

Should be resolved with #1575. Would love feedback as to whether indeed the issue is resolved!


Prisma CLI brings up "Upgrading from Prisma 1 to Prisma 2?" message

Bug description
Running prisma db pullon returns the following message:
❯ npx prisma db pull
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "mydb", schema "public" at "localhost:5435"

Introspecting based on datasource defined in prisma/schema.prisma …

✔ Introspected 1 model and wrote it into prisma/schema.prisma in 969ms

┌────────────────────────────────────────────────────────────────────────┐
│                                                                        │
│  Upgrading from Prisma 1 to Prisma 2?                                  │
│                                                                        │
│  The database you introspected could belong to a Prisma 1 project.     │
│                                                                        │
│  Please run the following command to upgrade to Prisma 2.0:            │
│  npx prisma-upgrade [path-to-prisma-yml] [path-to-schema-prisma]       │
│                                                                        │
│  Note: `prisma.yml` and `schema.prisma` paths are optional.            │
│                                                                        │
│  Learn more about the upgrade process in the docs:                     │
│  https://pris.ly/d/upgrading-to-prisma2                                │
│                                                                        │
│  Once you upgraded your database schema to Prisma 2.0,                 │
│  continue with the instructions below.                                 │
│                                                                        │
└────────────────────────────────────────────────────────────────────────┘

Run prisma generate to generate Prisma Client.

How to reproduce
Run migrate dev and then db pull
Expected behavior
Introspect the db and not bring up the "Upgrading from Prisma 1 to Prisma 2" message.
Prisma information

generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["fullTextSearch"]
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

model Book {
  id      Int      @id @default(autoincrement())
  title   String
  content String
  url     String
  authors String[]
}

Environment & setup

OS: Mac OS
Database: PostgreSQL v14
Node.js version: 16.13.0

Prisma Version
Environment variables loaded from .env
prisma                  : 3.6.0
@prisma/client          : 3.6.0
Current platform        : darwin
Query Engine (Node-API) : libquery-engine dc520b92b1ebb2d28dc3161f9f82e875bd35d727 (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli dc520b92b1ebb2d28dc3161f9f82e875bd35d727 (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine    : introspection-core dc520b92b1ebb2d28dc3161f9f82e875bd35d727 (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary           : prisma-fmt dc520b92b1ebb2d28dc3161f9f82e875bd35d727 (at node_modules/@prisma/engines/prisma-fmt-darwin)
Default Engines Hash    : dc520b92b1ebb2d28dc3161f9f82e875bd35d727
Studio                  : 0.440.0

The minimal schema you shared has all the characteristics we can rely on suggesting that potentially the Introspection is on a Prisma 1 schema. The message is intentionally formulated as a question, as we are aware that we can not be sure about that. Just removing the message alltogether is not really what we want.
Do you have a suggestion on how to improve the message to make that clearer?

A first possible cutoff date can only be after Prisma 1 Cloud is not a thing any more.


Allow non-members to create issues without access to source code.

I think this would be a handy feature. This would allow public feedback without giving everyone access to the source code.

Merged with #208


Add Daniel Belenky to reviewers

This PR also includes auto-generated code that updates the year on the copyright notes.
Signed-off-by: Daniel Belenky<EMAIL_ADDRESS>Release note:

NONE

/lgtm
/approve

/lgtm
/approve

/override pull-kubevirt-e2e-kind-k8s-sriov-1.14.2

Merging, since it obviously does not break anything and it fixes make generate which tries to insert the new year.

/cherry-pick release-0.24

/cherrypick release-0.24


vendor: update mongo-driver to 1.5.1 to fix CVE-2021-20329

Signed-off-by: André Martins<EMAIL_ADDRESS>

test-me-please

test-1.16-netnext is duplicate of #16399


SimplicialComplex function Error

hi! I find typo:
is it right ? complex_modex to complex_model
class SimplicialComplex(Module):
    def __init__(self, n_simplex):
        super(SimplicialComplex, self).__init__()
        self.n_simplex = n_simplex

    def forward(self, complex_modex):   -> def forward(self, complex_model):

        ## first need to pick a simplex to sample from ##
        vols = []
        n_verts = []
        for ii in range(self.n_simplex):
            vols.append(complex_volume(complex_model, ii))
            n_verts.append(len(complex_model.simplexes[ii]))

        norm = sum(vols)
        vol_cumsum = np.cumsum([vv/norm for vv in vols])
        simp_ind = np.min(np.where(np.random.rand(1) < vol_cumsum)[0])

        ## sample weights for simplex
        exps = [-(torch.rand(1)).log().item() for _ in range(n_verts[simp_ind])]
        total = sum(exps)
        exps = [exp/total for exp in exps]

        ## now assign vertex weights out
        vert_weights = [0] * complex_model.n_vert
        for ii, vert in enumerate(complex_model.simplexes[simp_ind]):
            vert_weights[vert] = exps[ii]

        return vert_weights

Yes, i think so.

Should be fixed now


TMR Expiration Date Default Value - 120 days? 60 days?

The docs says that the default value is 120 days. But, the blog says 60 days, and the default value on my tenant is 60 days, too. Which is correct?
Docs:
All newly created Teams meeting recordings (TMRs) will have a default expiration of 120 days.
Blog:
New recordings will automatically expire 60 days
https://techcommunity.microsoft.com/t5/microsoft-teams-blog/how-to-manage-microsoft-teams-meeting-recording-auto-expiration/ba-p/3053035

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ee6d4c1c-6f16-b1a4-c4a9-be4e6d1f95b7
Version Independent ID: 50ad835d-ed7f-18c6-34ba-322c1574f9c2
Content: Meeting policies and meeting expiration in Microsoft Teams - Microsoft Teams
Content Source: Teams/meeting-expiration.md
Service: msteams
GitHub Login: @mkbond007
Microsoft Alias: mabond

Hello @shtakeda
Please see Message Center Id MC274188

For any tenant that does not have a custom policy in place already, we are updating the default expiration days from 60 to 120 days.

@scanum


Update Node.js minimum required version.

According to package.json, this package seems to support Node.js v8.
https://github.com/paulmillr/readdirp/blob/d9c72137579b439fe0f8bb8d668f6b6846c75c45/package.json#L21
However, Node.js v8 has already been end-of-life on Dec 31, 2019.
https://github.com/nodejs/Release
If it is Node.js v10.13, fs.promises can be used. It is available to use fs.Dirent too, which is faster than fs.stat to check whether an entry is a file or a directory.
Another idea is that the minimum required version will be Node.js v12 for #105 .
Node.js v12 has already been LTS.
If minimum required Node.js version is updated, it will be a breaking change.
Are you planning to update Node.js minimum required version?

Have you checked the actual readdirp code? We are using Dirent already. And we are using bigint inodes for win32, which are only available on node 10+.
We fallback to old methods for v8. Readdirp 3 was released one year ago, and so far we haven’t even switched half of users to it. The reason being: the packages support even older nodes, like v6. It’s a very lengthy task, and bumping min req to node10 won’t help much here.

Thank you for your reply. I did not know this has already used Dirent. I should have read code more carefully.
I understood what you wrote. So I will close this issue.
Thank you for your great works.

Cheers.


2 complete support for ieee 8023dj

The COM results from this code for the "KR" test channels from Adam Healey's Jan. 2024 presentation to the IEEE P802.3dj task force don't match his exactly, but I think this code is correct/stable enough to merge back to main, continuing improvement from there.

Note: This merge breaks the GUI! Jupyter notebook usage only, for now. :(
(This was necessary, in order to move to Python 3.13.)


(RuntimeError) invalid session id when running on alpine

Hello,
I'am attempting to install chromedriver for hound e2e tests on top of https://github.com/bitwalker/alpine-elixir-phoenix by adding:
RUN apk --no-cache --update add chromium chromium-chromedriver

However, I'am getting this error when trying to run this in CI.
$ chromedriver &
$ mix test
Starting ChromeDriver 72.0.3626.121 on port 9515
Only local connections are allowed.
Please protect ports used by ChromeDriver and related test frameworks to prevent access by malicious code.
Compiling 16 files (.ex)
Generated you_must_goto app
..

  1) test home page has correct title (HoundTest)
     test/you_must_goto_web/e2e/hound_test.exs:8
     ** (RuntimeError) invalid session id
       (Driver info: chromedriver=72.0.3626.121,platform=Linux 4.19.23-coreos-r1 x86_64)
     code: navigate_to("/")
     stacktrace:
       (hound) lib/hound/request_utils.ex:52: Hound.RequestUtils.handle_response/3
       test/you_must_goto_web/e2e/hound_test.exs:9: (test)

I'am not entirely sure what this means or what the issue could be.
Thanks! Any assistance would be greatly appreciated!

Hi @edwinthinks,
I experienced the same error locally in earlier today and installing google-chrome-stable helped in my case. Thought to share it in case it might help someone else. (I use Vagrant with Ubuntu 18).

Having a similar issue, macOS Mojave.
@edwinthinks have you solved this?
@mimimalizam. How did you install google-chrome-stable? Any difference in the workflow? If so, could you please post them here?
Thank you!

@pdgonzalez872 hey there; I didn't end up getting any solution :( I haven't had much time with using hound lately and left it as is. Hope you find a solution :)!

You may want to load hound as a local dependency and put some IO.puts on the requests and responses. I was getting the same error message as you, and when hound made a call to http://localhost:9515/session the result was:
{"sessionId":"d6187f3aee07899a2f7773ffcd967ff6","status":33,"value":{"message":"session not created: This version of ChromeDriver only supports Chrome version 77\n  (Driver info: chromedriver=77.0.3865.10 (bc3579f611bbc73331171afe020ec7a45e6ccc55-refs/branch-heads/3865@{#93}),platform=Linu\
x 3.13.0-24-generic x86_64)"}}

I didn't investigate further, but you can see that there's a sessionId that hound probably tries to use again, but according to the embedded message it's not valid.
Would love to see this error surfaced cos I just spent 3 hours tracking it down. 💯
It makes me think that libraries that wrap http apis should have a simple debug toggle so you can see all requests and responses, cos something in those usually seems to be the problem. Maybe they do and I just missed it.
Love that hound exists, it's crucial! 🥇

Turns out that you need the version of Chrome to be the same as the version of Chromedriver. That solved our problem.

Can we close this issue @edwinthinks?

@pdgonzalez872 sure although I haven't been actively working with hound. If its reported working I can close this and only reopen if there is still an issue. Thanks @pdgonzalez872 !

I have the same problem collecting performance metrics from a company-internal website.
I do not have the problem with the first session, that works perfectly, but if I try to asynchronously start another session I get the following error and both sessions crash (Browser windows open and the first session already navigated to the website):
13:13:25.005 [error] Task #PID<0.274.0> started from #PID<0.269.0> terminating
** (RuntimeError) invalid session id
    (hound) lib/hound/request_utils.ex:52: Hound.RequestUtils.handle_response/3
    (hound_playground) lib/hound_playground.ex:56: HoundPlayground.collect_performance_metrics/2
    (elixir) lib/task/supervised.ex:90: Task.Supervised.invoke_mfa/2
    (elixir) lib/task/supervised.ex:35: Task.Supervised.reply/5
    (stdlib) proc_lib.erl:249: :proc_lib.init_p_do_apply/3
Function: #Function<0.22341430/0 in HoundPlayground.async_pisa_test/2>

Both chromedriver and chromium have the same version:
sc1se@VM-65220044-Linux:~$ chromium-browser --version
  Chromium 77.0.3865.90 Built on Ubuntu , running on Ubuntu 18.04
sc1se@VM-65220044-Linux:~$ chromedriver --version
  ChromeDriver 77.0.3865.90 (58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830})

I initialize each session with the following:
@chrome_prefs %{"protocol_handler" => %{"excluded_schemes" => %{"psadims" => false}}}

def start_new_session() do
    args =
    if System.get_env("browser") == "headless" do
      ["user-data-dir=/home/sc1se/.chromedriver", "headless"]
    else
      ["user-data-dir=/home/sc1se/.chromedriver"]
    end

    Hound.start_session(
      metadata: %{pid: self(), safe: false},
      driver: %{
        chromeOptions: %{
          "args" => args,
          "prefs" => @chrome_prefs
        }
      },
      browser: "chrome_driver"
    )
  end

The error also occurred when I used google-chrome (same major-, minor- and patch-version, only the build was a different one) instead of chromium.
Therefore this error is not only due to version differences in chromedriver and chrome. Any ideas why I get an invalid id error?

Turns out that you need the version of Chrome to be the same as the version of Chromedriver. That solved our problem.

Confirmed, same here.   Windows 10.


Any tutorial how to use danger-swift with github action?

I'm trying to convert my travis configuration to Github Action, most of the steps are converted, only danger swift has some problem that it doesn't work.
This is the configuration I have https://github.com/superarcswift/SwiftCommunity/blob/bugfix/github_action/.github/workflows/main.yml#L37
And this is the error from Github Action

https://github.com/superarcswift/SwiftCommunity/pull/120/checks?check_run_id=316993774
As mentioned, the same line of code works perfectly on travis.
Anyone has more insights how to run it on Github Action?

maybe danger isn't installed in npm?

yep, true, it looks better after and add installation script https://github.com/superarcswift/SwiftCommunity/blob/bugfix/github_action/.github/workflows/main.yml#L37
Thank you for pointing it out.

No worries (BTW you can probably delete bundle install from those steps)
I'm gonna re-open this as Danger Swift should show an error message if danger can't be found