Fix Ninja Build Issue

This PR fixes the Ninja Build issues introduced since Ninja 1.12.0, which failed the build CI for 123.0.6312.122 (#161) by simply removing the flag (used by Chromium) that is no longer supported.
I didn't find anything on Vanilla Chromium end on how this issue will be solved by them as of now.

This will come with the update to 124.0.6367.60 then, as the PR for updating in the core repo is now out of draft state.


SCRUM-1709 Don't clear form on clicking 'Save & Add Another'

Redundant due to code incorporated in SCRUM-1903


Add new notification on web when cannot create new job

webui was stucked when cannot create job (job request return error)
webui was stuck

log from server

I think server should return what error and webui should noti that error message

Sounds great, want to make a pull request?

Yep 😊


Update quickstart to remove manual docker login and docker pull steps.

Remove below two steps after accepting the terms and condition to use the coherence images

In a terminal window, log in to the Oracle Container Registry:
docker login container-registry.com

Pull the coherence image with the command:

docker pull container-registry.oracle.com/middleware/coherence:<IP_ADDRESS>.2
We do not want user to explicitly do docker login and docker pull as login to docker registry and pulling the image steps are done by kubernetes using the docker-registry secrets that are created by the user as describe in https://oracle.github.io/coherence-operator/docs/quickstart.html#set-up-secrets-to-access-the-oracle-container-registry when coherence helm charts gets installed.


FR: independent loading of sources

I work with Perl at work and the LS it's slow as hell.
So when I want to trigger for example a snippet, nvim-cmp waits for all the sources to be ready to show the suggestions. That makes it impossible to use any snippet. It will be cool if cmp could return the local/smaller sources first.
example:

And BTW thanks for all the hard work 💪🏽

It seems to be regression to me. I will improve it.

Now it works perfectly, thank you!


The NET Framework Setup subkey in the registry path does not begin with a period.

Please explain whetehr there is any logical reason for "The NET Framework Setup subkey in the registry path does not begin with a period." thank you.

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

ID: 2115da38-e95e-46fc-1e2d-da88ef92c6cf
Version Independent ID: a3790e8a-0654-851b-c991-ab7382025406
Content: Determine which .NET Framework versions are installed
Content Source: docs/framework/migration-guide/how-to-determine-which-versions-are-installed.md
Product: dotnet-framework
Technology: dotnet-appcompat
GitHub Login: @gewarren
Microsoft Alias: gewarren

Hello,
Are you asking about the note or about the specific name? The reason the note points this out is that that people are used to typing .NET Framework with a . at the start, so the note points out this discrepancy.
I'm going to close this. If this didn't answer your question, please follow up. Thanks!


e2e tests in CI fail with: snapshotter controller creation failed

Describe the bug
On occasion e2e testing fails with
snapshotter controller creation failed

The deployment is then aborted, before the actual e2e testing is started.

From #1918 :

https://jenkins-ceph-csi.apps.ocp.ci.centos.org/blue/organizations/jenkins/mini-e2e_k8s-1.20/detail/mini-e2e_k8s-1.20/641/pipeline/
https://jenkins-ceph-csi.apps.ocp.ci.centos.org/blue/organizations/jenkins/upgrade-tests-cephfs/detail/upgrade-tests-cephfs/1302/pipeline/

@nixpanic I havent seen this issue in recent runs especially at release time. Do we need to keep this open ?


Workflow suggestion

Hi,
I'd like to suggest a new way to enter the lenght of a step by pressing a step and then rotating the knob so you extend the lenght of a step by the steps you need, I mean
you need for example to add a gate that is 4 step long, so you press the first step where you want it to begin and then rotate the knob until the 4 steps are done, I think this way is faster and comfortable than pressing the 4 steps or shift+the steps you need and making the way it is now
just saying :)

Nice!


compiling error on arch linux

Compiling servo-fontconfig v0.2.0 (https://github.com/jwilm/rust-fontconfig#419135e5)
Running rustc /home/elio/.cargo/git/checkouts/rust-fontconfig-74c44fcd18a59a5d/419135e5e1106ec0973dd4923bd9c70d8e438cc8/src/lib.rs --crate-name fontconfig --crate-type lib -C opt-level=3 -g -C metadata=7de0c55f3beac4a1 -C extra-filename=-7de0c55f3beac4a1 --out-dir /home/elio/Downloads/alacritty-master/target/release/deps --emit=dep-info,link -L dependency=/home/elio/Downloads/alacritty-master/target/release/deps --extern fontconfig_sys=/home/elio/Downloads/alacritty-master/target/release/deps/libfontconfig_sys-5f3b046d6b3fe525.rlib --extern libc=/home/elio/Downloads/alacritty-master/target/release/deps/liblibc-6ec63c5a0e74a074.rlib --cap-lints allow -L native=/usr/lib -L native=/usr/lib -L native=/usr/lib -L native=/usr/lib
Build failed, waiting for other jobs to finish...
error: Could not compile parking_lot_core.
Caused by:
process didn't exit successfully: rustc /home/elio/.cargo/registry/src/github.com-1ecc6299db9ec823/parking_lot_core-0.2.0/src/lib.rs --crate-name parking_lot_core --crate-type lib -C opt-level=3 -g --cfg feature="nightly" -C metadata=b776954c2db52c62 -C extra-filename=-b776954c2db52c62 --out-dir /home/elio/Downloads/alacritty-master/target/release/deps --emit=dep-info,link -L dependency=/home/elio/Downloads/alacritty-master/target/release/deps --extern smallvec=/home/elio/Downloads/alacritty-master/target/release/deps/libsmallvec-cf4677c7857fc4a9.rlib --extern libc=/home/elio/Downloads/alacritty-master/target/release/deps/liblibc-6ec63c5a0e74a074.rlib --extern rand=/home/elio/Downloads/alacritty-master/target/release/deps/librand-8ea7d489d4a383a0.rlib --cap-lints allow (exit code: 101)

Did you install the dependencies and set the specific rustc compiler version with rustup as described in the README?

https://aur.archlinux.org/packages/alacritty-git/
You have to install rustup from the repos to fulfill the nightly Rust version for now.

It sounds like you may be missing the fontconfig-dev dependency or equivalent on your system. The are instructions in the README for doing so on Arch.

yeah I completely missed that single one, once I am done with work I plan on getting it running :) Will report back.

https://aur.archlinux.org/packages/alacritty-git/
You have to install rustup (in the repos) to fulfill the nightly Rust version for now.

I just compiled alacritty using the mentioned AUR package and can confirm this. After installing rustup, everything builded smoothly.

I agree. Just ran it and using it for work as we speak. So far love it.


Add pyjwt to pyproject

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
I tried to set up from scratch today, and hit an error with jwt missing.

Give a summary of what the PR does, explaining any non-trivial design decisions
Adds pyjwt to the pyproject.toml

Other references

@tobitege I think I used the standard precommit...
No idea why the tests are failing

The pyproject.toml file needs "build" added.

@rbren I've added build to the pyproject.toml file, could you pull this and do a poetry lock --no-update, please?

Done!


Update provider to versioned provider interface

Summary
This commit updates the honeywell provider to use the latest versioned
abstract interface from terra (which was introduced in 0.16.0). Terra
will be deprecating the legacy interface in the near future, so to get
in front of that we should update the provider to use the current
interface version. At the same time this fixes some compatibility issues
that arise when running with the latest version of terra (which is
necessary because the minimum version is bumped to ensure the new
provider interface exists).
Details and comments

I don't have any way to test this, so I assume I missed something. But I don't have any method to run against the honeywell API (and don't know anyone who does) so I'm not sure how to fix that.

@mtreinish, we can give you access to our QA environment if it helps and will test this update shortly.

@mtreinish, we can give you access to our QA environment if it helps and will test this update shortly.

Yes, that would help a lot and let me locally test these changes to verify everything still works with this PR applied.
We also could follow up and have a post merge CI job that tests the provider against the QA env too, because most of the tests in the unittests are skipped conditional on having credentials to hit the api.

Yes, that would help a lot and let me locally test these changes to verify everything still works with this PR applied.

@mtreinish, please reach out via email to<EMAIL_ADDRESS>to obtain login credentials at your convenience.

While you are getting access to our testing infrastructure.  The following were found to be the minimum update to pass our regression tests.  I have not had, as yet, luck migrating past pyjwt 1.7.1.  If it is a requirement that we update to 2.x, then additional updates will be required.
`diff --git a/qiskit/providers/honeywell/credentials/credentials.py b/qiskit/providers/honeywell/credentials/credentials.py
index 97137d4..7a7d62b 100644
--- a/qiskit/providers/honeywell/credentials/credentials.py
+++ b/qiskit/providers/honeywell/credentials/credentials.py
@@ -277,7 +277,7 @@ class Credentials:
id_token = self._get_token('id_token')
     # check id_token is not expired yet

   expiration_date = jwt.decode(id_token, verify=False)['exp']

   expiration_date = jwt.decode(id_token, verify=False, algorithms=["RS256"])['exp']
   if expiration_date < (datetime.datetime.now(datetime.timezone.utc).timestamp()):
       print("Your id token is expired. Refreshing...")

diff --git a/qiskit/providers/honeywell/honeywelljob.py b/qiskit/providers/honeywell/honeywelljob.py
index 147bfdb..6806d71 100644
--- a/qiskit/providers/honeywell/honeywelljob.py
+++ b/qiskit/providers/honeywell/honeywelljob.py
@@ -160,6 +160,7 @@ class HoneywellJob(JobV1):
self._job_ids = []
self._experiment_results = []

   self._qobj_payload = {}
   if circuits:
       if isinstance(circuits, qobj_mod.QasmQobj):
           self._qobj_payload = circuits.to_dict()

@@ -171,7 +172,6 @@ class HoneywellJob(JobV1):
self._experiments = circuits
self._job_config = job_config
else:

       self._qobj_payload = {}
       self._status = JobStatus.INITIALIZING
       self._job_ids.append(job_id)

diff --git a/requirements.txt b/requirements.txt
index c712420..4844323 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -2,5 +2,5 @@ nest-asyncio>=1.2.0
qiskit-terra>=0.16.0
requests>=2.19
websockets>=7
-pyjwt>=1.7.1
+pyjwt>=1.7.1,<2.0
keyring>=10.6.0
diff --git a/setup.py b/setup.py
index 618a45d..1fa3fcc 100644
--- a/setup.py
+++ b/setup.py
@@ -35,7 +35,7 @@ requirements = [
'qiskit-terra>=0.16.0',
'requests>=2.19',
'websockets>=7',

'pyjwt>=1.7.1',

'pyjwt>=1.7.1,<2',
'keyring>=10.6.0',
]

diff --git a/tox.ini b/tox.ini
index 5e0c6eb..6d4a87f 100644
--- a/tox.ini
+++ b/tox.ini
@@ -23,7 +23,7 @@ deps =
pycodestyle
pylint
setuptools>=40.1.0

pyjwt>=1.7.1

pyjwt>=1.7.1,<2
keyring>=10.6.0
commands =
pycodestyle qiskit/providers/honeywell test/
`

Yeah, I think capping pyjwt is fine, especially against a major version boundary like that. If/when we find a pattern for pyjwt >= 2.0.0 we can do that in a followup. @dlucchetti do you want to open a separate PR with those fixes so we can get the main branch working again and then I'll rebase this on top of that?


fix(unhead): handle undefined style attributes

🔗 Linked issue
#336
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Resolves #336
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks!


Make all params and models lower-case

Closes #81 and closes #70. All of the params and models in the BEP016 doc are lower-case so this might fit better.
Changes proposed in this pull request

Rename models and params in config files to be lower-case.
For any models or params that are defined within the workflows, make them lower-case and remove any non-alphanumeric characters automatically.
Remove non-alphanumeric characters from bundle entities.

Codecov Report
Attention: Patch coverage is 12.50000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 33.67%. Comparing base (f20d5ac) to head (6e5d9ea).

Files
Patch %
Lines

qsirecon/utils/misc.py
25.00%
3 Missing :warning:

qsirecon/workflows/recon/dsi_studio.py
0.00%
2 Missing :warning:

qsirecon/workflows/recon/mrtrix.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
- Coverage   33.69%   33.67%   -0.03%
==========================================
  Files          56       56
  Lines        6914     6922       +8
  Branches      907      907
==========================================
+ Hits         2330     2331       +1
- Misses       4481     4488       +7
  Partials      103      103

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


scrape more than 1000 users

I noticed github only gives back 1000 users in a single search, so this splits the search up so it finds more (2,346 users).

@cuddyz this should pick you up now


drive microcontrollers

The microcontrollers used to regulated the brushed motors will need configuration and documentation of how they are interfaced.
Goals:

Determine and document what pins of microcontroller need to be use.
Implement connection between gamepad input and microcontroller interface.

Moved on from brush motors


code exchange - /templates revamp

All new changes are confined in the /pages/templates folder

[x] filter by tags
[x] filter by keyword (using search bar)
[x] individual template pages now load their own readme.md content
[x] add SEO for those individual pages
[x] tags page (for listing all templates under that tag)
[x] no useEffect - why does it even exist
[x] tags and keyword queries should reflect in the url params
[x] (optional) add CONTRIBUTION.md for this?

PR-Codex overview
The focus of this PR is to enhance the template filtering and tagging system.
Detailed summary

Added new tag pages for better organization
Updated filter menu with new filter options
Improved contribution guidelines in CONTRIBUTE.md
Implemented dynamic filtering in TemplateWrapper
Added server-side filtering logic in Templates page
Created TemplateTagPage for displaying templates by tag
Updated tag data structure in tags.ts

The following files were skipped due to too many changes: components/templates/Wrapper.tsx, pages/templates.tsx, pages/templates/[templateId].tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

/templates/[templateId]
/templates

(view full size)
(view full size)

/templates/tag/[tagId]

(view full size)

1 deleted route:

/template/[id]

Commit fae0ca0e8c32a3c8bbeae1b35a80cf3fee74300e (https://thirdweb-72241ajjb.thirdweb-preview.com).


Drivers not detected due to permission error in android 11?

Hi,
First of all thanks for this library reference to understand the Android UART. Actually facing an issue with this is whenever I run the app then it wasn't able to read the drivers.
Exception:- java.io.FileNotFoundException: /proc/tty/drivers: open failed: EACCES (Permission denied)
Caused by: android.system.ErrnoException: open failed: EACCES (Permission denied)
I am using this in the rooted device. Can you please help with some guidance for this? Thanks @xmaihh

Right, cannot open /dev/tty: Permission denied. As mentioned above, #FAQ This library does not provide ROOT permissions, please open the serial port '666' permissions yourself. I think it's because you have broken permissions for that device file.
I suggest you should just double-check if you are have permissions.
See which UARTs are detected in /proc/tty/driver/serial. Lines with uart:unknown meaning: Nothing detected (probably not present).
# cat /proc/tty/driver/serial
serinfo:1.0 driver revision:
0: uart:16550A port:000003F8 irq:4 tx:0 rx:0
1: uart:16550A port:000002F8 irq:3 tx:111780 rx:1321 RTS|DTR|DSR
2: uart:unknown port:000003E8 irq:4
3: uart:unknown port:000002E8 irq:3

If something is connected and driving the CTS, DSR or CD lines (those are input lines), you can even be sure that something is there... same goes for rx-byte-count.
There is also a command setserial which uses /proc/tty/driver/serial get its data.
# setserial -g /dev/ttyS[0123]
/dev/ttyS0, UART: 16550A, Port: 0x03f8, IRQ: 4
/dev/ttyS1, UART: 16550A, Port: 0x02f8, IRQ: 3
/dev/ttyS2, UART: unknown, Port: 0x03e8, IRQ: 4
/dev/ttyS3, UART: unknown, Port: 0x02e8, IRQ: 3

You can confirm that this device file is accessible like this.
crw-rw-rw- system   system     4,  64 1970-01-01 08:00 ttyS0
crw-rw-rw- system   system     4,  65 1970-01-01 08:00 ttyS1

Hi @xmaihh
The cat /proc/tty/driver/serial does show two UARTs, so they are already available there as well as the permission is also changed to "rwx" for any user type but still it doesn't work.
Also, the test is done in Android 11 as per mentioned in the title of the issue but generated the permission denied error in android 11 and working in Android version 7 (SDK 25).
The same reference code is tested in a device having Android 7 and the device is not rooted. The device is logging the detected drivers as below:-

Found new driver g_serial on /dev/ttyGS
Found new driver mtk-uart on /dev/ttyMT

The detection of drivers is done in the older version of Android (7.0 – Nougat) but not detecting the drivers in Android 11 which we need to do. Detection of the driver is possible only till Android SDK version 25 (Android version 7 - Nougat). After Android SDK version 26 (Android version 8.0 - Oreo), it’s not working and generating the permission denied error. I also tried the commands by making the app a system app, but I still, have the same permission issue in Android 11.
Are there any known issues with Android 11?

Thank you for sharing @NikhilUsadadiya, I didn't find the proper solution yet, I didn't test in Android 11 environment now.
On an Android 11 device your app only has access to its own files and to general mediafies in public directories. You have options to read the file. Request MANAGE_EXTERNAL_STORAGE in manifest and let the user confirm.
Regards,
X.

Yes, @xmaihh, I have added the request permission in the manifest file also and tried to change the permission and detection of drivers in android 11, but fails to do so. Also, the Request MANAGE_EXTERNAL_STORAGE in the manifest will only allow the user to access the public directories right. It will not allow access or change the value in the root directories right?

@NikhilUsadadiya I'm facing the same issue. Did you solve it?

look this
https://blog.csdn.net/jspping/article/details/131852570?spm=1001.2014.3001.5501

look this
https://blog.csdn.net/jspping/article/details/131852570?spm=1001.2014.3001.5501

你好，我也遇到这个问题，也看到了这篇帖子，但是具体怎么操作怎么配置完全不懂，可能的话麻烦指导一下，感谢！

请问下，这个有啥简单的解决方法不？

https://github.com/Deemonser/AndroidSerialPort


Update sbt-scalajs-crossproject to 1.0.0

Updates org.portable-scala:sbt-scalajs-crossproject from 0.6.1 to 1.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.portable-scala", artifactId = "sbt-scalajs-crossproject" } ]

labels: sbt-plugin-update, semver-major

Oh, we must have inherited this from @ChristopherDavenport's template.  I guess we should publish for that.


move vtt_talk_sources to sharedconditions.py

This pull request addresses the problems described at issue #2076

I think you need to adjust the test, too :D I'm on it already: https://github.com/googlefonts/fontbakery/pull/2077

thanks!


${{ expression }} syntax is not supported at task level

The below template is valid but generates 'unexpected property ... the first property must be a task' errors in vscode.
parameters:
- name: customSteps
  type: stepList
  default: []

steps:
- ${{ each step in parameters.customSteps }}:
  - ${{ step }}

- ${{ if and(in(variables['Agent.JobStatus'], 'Succeeded', 'SucceededWithIssues'), notIn(variables['Build.Reason'], 'IndividualCI', 'BatchedCI')) }}: # Don't run for CI builds
  - bash: |
      echo "test"

Duplicate of #187?

Duplicate of #187?

Agree it is a dupe of #187, thank you!

Agree it is a dupe of #187, thank you!

Duplicate of #187

Duplicate of #187


How to NOT render the title global attribute.

Hello, 👋
First off, I really love the lib and use it often for its simplicity. 🙂
From MDN, I understand the title attribute is basically a tooltip for representing advisory information related to the element it belongs to. In our case, the time element. (Thanks for making this semantic.)
But here is my issue; I wanted to provide my own tooltip, but trying to override the rendered title doesn't seem to work.
I tried passing null and an empty string '' to the title prop, but nothing seemed to work.
						<Time
							relative
							format="D MMMM, YYYY"
							timestamp={created_at}
							class="text-sm text-muted-foreground"
							title={null}
						/>

The result, in my case, is repetitive information and I would like to avoid this:

Do you know of a way to:

disable the tooltip, but retain the element title in the DOM (for accessibility's sake?)

forego on adding the title altogether

I appreciate the help.

Hey – thank you for the feedback and taking the time to open this issue. I'm working on a PR that will allow the title attribute to be overridden.

Released in v0.9.0.
You can now specify a custom title or omit it altogether by setting title={undefined}.
<Time relative title="Custom title" />

Set the value to undefined to omit the title altogether.
<Time relative title={undefined} />

Hello, @metonym
Thank you very much for the release. It works splendidly.
I appreciate the help.
Be safe. 🙂


[CWOD-DAV20] Translate FR

Changes / Comments
Traduction de la fiche en FR pour CWOD DAV20.
Modification du fichier JSON.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hello @hacyrak,
Your Pull Request includes changes to translation files. Unfortunately, we no longer accept translations submitted outside of Crowdin. To learn more about Crowdin and how to submit translations through Crowdin, please see our article on How to Become a Translation Volunteer. Thanks for all you do!


⚠ AtmoVantage Homepage has degraded performance

In 197a141, AtmoVantage Homepage (https://atmovantage.com) experienced degraded performance:

HTTP code: 200
Response time: 6448 ms

Resolved: AtmoVantage Homepage performance has improved in feffedc.


Install NetworkPolicies after Namespaces

Helm should install NetworkPolicy resources after Namespace resources.
Currently the InstallOrder at https://github.com/kubernetes/helm/blob/master/pkg/tiller/kind_sorter.go#L29 does not list the NetworkPolicy resource. If a chart creates a Namespace resource and a NetworkPolicy resource in this Namespace, the deployment of the chart might fail because helm tries to create the NetworkPolicy before the Namespace has been created.

Since they are omitted from the known list of objects they should be installed last according to TestKindSorter. Would you mind adding a unit test to verify that behaviour or by supplying a chart to test this?


Encourage squads to adopt a no-accumulation-of-slips policy

In order to maintain a monthly release cadence with minimal squad-to-squad disruption and make life easier on Zowe vendors and extenders that need to run their operations on a predictable cadenee, I would propose we strongly encourage all squads to adopt a no-accumulation-of-slips policy.
What this would mean is that, while it is ok to release a monthly release late, it is not ok to change the target date of the next release.  Thus a release date slip will always reduce the time until the next release date by an equal number of days as the size of the slip.

Problem solved!


Cannot create domain and then add objects to it.

Hi,
I've been trying to automate CMA deployment using Ansible. To do that, I use two role. One for creating the CMA using ansible host without the "ansible_checkpoint_domain" and then another role that use it with the name of the CMA I created:
ckps:
  hosts:
    mdm:
      ansible_host: a.a.a.a
      ansible_httpapi_use_ssl: True
      ansible_httpapi_validate_certs: False
      ansible_user: user
      ansible_password: password
      ansible_httpapi_port: 443
      ansible_network_os: check_point.mgmt.checkpoint
      domain: test_domain
      cma_name: test_cma
      mds: mds_name
      cma_ip: "x.x.x.x"
    cma:
      ansible_host: a.a.a.a
      ansible_httpapi_use_ssl: True
      ansible_httpapi_validate_certs: False
      ansible_user: user
      ansible_password: password
      ansible_httpapi_port: 443
      ansible_network_os: check_point.mgmt.checkpoint
      ansible_checkpoint_domain: test_domain

with main role:
- name: Create CMA
  hosts: mdm
  connection: httpapi
  roles:
    - add_cma
- name: Insert default objects
  hosts: cma
  connection: httpapi
  roles:
    - create_cma_objects

Creating the CMA works like a charm using:
- name: Create CMA
  check_point.mgmt.cp_mgmt_add_domain:
    name: "{{ domain }}"
    servers:
      - ip_address: "{{ cma_ip }}"
        multi_domain_server: "{{ mds }}"
        name: "{{ cma_name }}"

- name: Add trusted clients for API
  check_point.mgmt.cp_mgmt_trusted_client:
    name: AnyHost
    state: present
    domains_assignment: "{{ domain }}"
    auto_publish_session: yes

but then,  in my role where i create object in the CMA (with the host with the specified domain):
- name: add-host
  cp_mgmt_host:
    ip_address: "{{ item.get('ipv4-address') }}"
    name: "{{ item.name }}"
    comments: "{{ item.comments }}"
    state: present
  loop: "{{ hosts.list }}"

it crashes, and says:
failed: [cma] (item={'name': 'name', 'ipv4-address': 'y.y.y.y', 'comments': 'some commment'}) => {"ansible_loop_var": "item", "changed": false, "item": {"comments": "some comment", "ipv4-address": "y.y.y.y", "name": "name"}, "module_stderr": "Traceback (most recent call last):\n  File \"/home/sami/.ansible/tmp/ansible-local-29740hyz_1fy5/ansible-tmp-1642418637.9183214-30174-87115745096485/AnsiballZ_cp_mgmt_host.py\", line 100, in <module>\n    _ansiballz_main()\n  File \"/home/sami/.ansible/tmp/ansible-local-29740hyz_1fy5/ansible-tmp-1642418637.9183214-30174-87115745096485/AnsiballZ_cp_mgmt_host.py\", line 92, in _ansiballz_main\n    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)\n  File \"/home/sami/.ansible/tmp/ansible-local-29740hyz_1fy5/ansible-tmp-1642418637.9183214-30174-87115745096485/AnsiballZ_cp_mgmt_host.py\", line 41, in invoke_module\n    run_name='__main__', alter_sys=True)\n  File \"/usr/lib/python3.7/runpy.py\", line 205, in run_module\n    return _run_module_code(code, init_globals, run_name, mod_spec)\n  File \"/usr/lib/python3.7/runpy.py\", line 96, in _run_module_code\n    mod_name, mod_spec, pkg_name, script_name)\n  File \"/usr/lib/python3.7/runpy.py\", line 85, in _run_code\n    exec(code, run_globals)\n  File \"/tmp/ansible_cp_mgmt_host_payload_zm5xyskm/ansible_cp_mgmt_host_payload.zip/ansible_collections/check_point/mgmt/plugins/modules/cp_mgmt_host.py\", line 333, in <module>\n  File \"/tmp/ansible_cp_mgmt_host_payload_zm5xyskm/ansible_cp_mgmt_host_payload.zip/ansible_collections/check_point/mgmt/plugins/modules/cp_mgmt_host.py\", line 328, in main\n  File \"/tmp/ansible_cp_mgmt_host_payload_zm5xyskm/ansible_cp_mgmt_host_payload.zip/ansible_collections/check_point/mgmt/plugins/module_utils/checkpoint.py\", line 278, in api_call\n  File \"/tmp/ansible_cp_mgmt_host_payload_zm5xyskm/ansible_cp_mgmt_host_payload.zip/ansible_collections/check_point/mgmt/plugins/module_utils/checkpoint.py\", line 71, in send_request\n  File \"/tmp/ansible_cp_mgmt_host_payload_zm5xyskm/ansible_cp_mgmt_host_payload.zip/ansible/module_utils/connection.py\", line 200, in __rpc__\nansible.module_utils.connection.ConnectionError: Invalid JSON response: <ns1:XMLFault xmlns:ns1=\"http://cxf.apache.org/bindings/xformat\"><ns1:faultstring xmlns:ns1=\"http://cxf.apache.org/bindings/xformat\">com.checkpoint.management.web_api_is.exceptions.WebApiGeneralException</ns1:faultstring></ns1:XMLFault>\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

as If the targeted domain wasn't specified in the connection.
I'm using this Ansible version, and just runned ansible-galaxy to install the checkpoint module
ansible --version
ansible [core 2.11.7]
  config file = None
  configured module search path = ['/home/sami/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/sami/.local/lib/python3.7/site-packages/ansible
  ansible collection location = /home/sami/.ansible/collections:/usr/share/ansible/collections
  executable location = /home/sami/.local/bin/ansible
  python version = 3.7.3 (default, Jan 22 2021, 20:04:44) [GCC 8.3.0]
  jinja version = 3.0.2
  libyaml = True

Meanwhile, when I execute the role independantly by using two ansible-playbook commands, it does works properly. It seems it doesn't login again and credentials failed ?
How can I execute roles on MDS and multiple domains without invoking ansible-playbook multiple time ?
Thanks in advance for your answer, I can add info if needed.
Regards,

Hi @chkp-shirango
seems to work fine with that solutions.
Notice: I tried to use retries/until with delay but once a task failed, it continues to fails with the same error.
Thank you for your solution.


Can I integrate bulk omics data with GLUE?

Hello and thank you for very interesting software!
The GLUE framework seems to be designed for single-cell omics data in mind. Is it possible to integrate bulk ATAC and RNA sequencing data with GLUE? I'm not very familiar with machine learning, but I understand that the training requires lot of data and you reference in the paper that below 2000 cells the alignment error starts to increase. Is it a lost cause to try to use GLUE with this type of data? I have multiple cell lines, treatments, and replicates from each condition, but not nearly enough samples to resemble anything like a single-cell experiment.
Apologies for my lazines, I haven't tried pre-processing the data in the form that is used as input by GLUE. I haven't used python much and wanted to first get your general opinion.
Best,
-Konsta

Hi Konsta! Thanks for your interest in GLUE. You are right, GLUE does not work well with small sample sizes (i.e., <2,000) because it is an over-parameterized neural network model and definitely needs a reasonable number of samples to train. Nevertheless, there are certain things you can try to make it more small-sample-friendly:

Decrease the number of features (e.g., genes, peaks) used as input. Bulk data probably don't have that much subtle variation to distinguish when it comes to samples, so maybe a small set of well-selected features would be sufficient.
Decrease the input preprocessing dimensionality. The rationale is the same. Bulk data typically don't need 50 PCs to capture the majority of data variation. Something like 5 PCs would be more appropriate?
Scale down the model architecture by reducing the number of hidden layers (to 1), reducing the number of hidden layer dimension (e.g., to 32).

These might increase the possibility of getting a reasonably trained model. Let me know if there were further issues, or if it actually works (I'm also interested 😀)

Thank you for a prompt response Zhi-Jie. I will try your suggestions and report whether it works! :)

Hi Konsta! As you have found out, the model argument in the fit_SCGLUE function only accepts a model type, rather than an already constructed model object.
To tinker the model structure, you can specify the model construction arguments using the init_kws argument, which will be passed on to construction of model objects inside the fit_SCGLUE function.
For the above example, you may use something like this:
my_mod = scglue.models.fit_SCGLUE(
    adatas={"rna": rna, "atac": atac},
    guidance,
    init_kws=dict(
        latent_dim=5,
        h_depth=1,
        h_dim=32,
        dropout=0.2,
        shared_batches=False,
        random_seed=0,
    )
)

Oh, So those parameters should be passed to init_kws, not to model. Got it! Thank you


haskell-packages-0.3 does not build with aeson-0.11

src/Distribution/HaskellSuite/Packages.hs:331:10:
    Duplicate instance declarations:
      instance ToJSON Version
        -- Defined at src/Distribution/HaskellSuite/Packages.hs:331:10
      instance ToJSON Version
        -- Defined in ‘aeson-<IP_ADDRESS>:Data.Aeson.Types.Instances’

src/Distribution/HaskellSuite/Packages.hs:333:10:
    Duplicate instance declarations:
      instance FromJSON Version
        -- Defined at src/Distribution/HaskellSuite/Packages.hs:333:10
      instance FromJSON Version
        -- Defined in ‘aeson-<IP_ADDRESS>:Data.Aeson.Types.Instances’

Fixed in haskell-packages-0.4.

Do you want to own the package in Stackage?

Yes, I can do that.
On Wednesday, 13 July 2016, Jens Petersen<EMAIL_ADDRESS>wrote:

Do you want to own the package in Stackage?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/haskell-suite/haskell-packages/issues/23#issuecomment-232235319,
or mute the thread
https://github.com/notifications/unsubscribe/AAM8snb4ZvbaBjhE30gi8CFIun4jp8mnks5qVElNgaJpZM4H3n06
.


disabled the flag or the text input

Can I disabled the flag or the dropdown flag ?

When you add
  countryPickerProps={{
                                disableNativeModal: true
                            }}

The styled is broken.

Workaround that I use:
Prevent modal to show.
<PhoneInput
  countryPickerProps={{ modalProps: { visible: false } }}
/>

disableNativeModal: true  worked for me. styles we can adjust accordingly.


fix: hide placeholder when there are no targets

Hi! First of all, thanks so much for for this library, it's really awesome, tremendous work!
I hope this proposal makes sense. It's aimed to improve the UX of the drag and drop in a specific scenario.
Problem
When using the custom placeholder, there is a case in which you still see it rendered but you can't actually drop the item being dragged. That happens when you try to drag any item at the very top or bottom of the tree. If your cursor goes outside the bounds of the container, the placeholder is still there, hinting that the item will be dropped right there, but in reality if you drop it nothing happens... it's misleading because the placeholder is there, but the item cannot be dropped.
https://github.com/minop1205/react-dnd-treeview/assets/1586378/ffab0175-fca3-4988-aecf-33d67c6f7030
Solution
What I'm proposing is just to hide the placeholder when there are no targets. I don't know if this is the correct solution technically speaking but I believe it's the correct one in terms of UX. Happy to change the implementation as you see fit. Any feedback also very welcome. Thanks!
https://github.com/minop1205/react-dnd-treeview/assets/1586378/d409892c-e888-4d25-b3d1-feea666f4b1c

Thank you @nuragic! I've reported this issue a few weeks ago.
https://github.com/minop1205/react-dnd-treeview/issues/220
Seems working well :)

@nuragic I think this is a simple and clear way to address the issue.
Thank you for your cooperation!
And sorry for the late reaction.

#100

Thank you!


Build(deps-dev): Bump @testing-library/dom from 8.20.0 to 9.0.1 (#4705)

Pull request opened by the merge tool on behalf of #4705

Merged in c07011c9b31198aa2519e1f3add386c849c13df0


Error when using with three 0.149.0

The error seems to be coming from SplineLoader, but since that is not an open source library I decided to raise the issue here.
Error:
Uncaught Error: Could not load https://prod.spline.design/IcnOpHOp4rFkJJie/scene.splinecode: Cannot read properties of undefined (reading 'x'))
    at index-7dbf758c.esm.js:1586:36
    at Object.onLoad (SplineLoader.js:1338:75098)

Codesandbox:
https://codesandbox.io/p/sandbox/untitled-copy-forked-4qc4wv?selection=[{"endColumn"%3A1%2C"endLineNumber"%3A14%2C"startColumn"%3A1%2C"startLineNumber"%3A14}]&file=%2Findex.html

I am having this issue too. A temporary fix is to downgrade three to 0.138.3.

@iambarnaby I'm using three 0.142.0 which works in both chrome and firefox for me

Fixed in @splinetool/loader version 0.9.226


Blueprints "ignore" field removes tab options but not the tab itself

Inheriting the "default" blueprint, but wanting to ignore a tab in it. This is a modular template blueprint.
Using ignore in the following code removes the options (content) in the tab, but does not remove the tab itself:
title: Showcase
@extends: default

form:
  fields:
    tabs:
      fields:
        advanced:
          fields:
            columns:
              fields:
                column1:
                  fields:
                    name:
                      default: modular/showcase
                      @data-options: '\Grav\Common\Page\Pages::modularTypes'
            overrides:
              fields:
                header.template:
                  default: modular/showcase
                  @data-options: '\Grav\Common\Page\Pages::modularTypes'

        styles:
          type: tab
          title: Row Style
          fields:
            header.styles:
              type: textarea
              label: CSS Classes

        foundation:
          type: ignore

The Gitter conversation is here:
https://gitter.im/getgrav/grav?at=56a0614928b4586d1c8de357
The Grav Docs are here:
http://learn.getgrav.org/advanced/blueprints#special-form-fields-available-only-in-admin
This is using the Bones Site Skeleton on the "/modular/showcase.yaml" blueprint for testing:
https://github.com/smartgravity/grav-skeleton-bones-site/releases

Fixed in https://github.com/getgrav/grav-plugin-admin/commit/29f7aa6428d0750709aa001acf32cbf14ef4ef88


dev: add clang-format

Resolved by https://github.com/alibaba/inclavare-containers/pull/1113/commits/ccf169990ec4cce93efddae3ccc14bf7242c7b88


Added posibility to scan barcodes on input

Right now it is not possible to scan barcode on input, text area or other editable field. This change should allow some users to use it with those fields.

@kybarg If I'm reading this code correctly, this looks like something I could use as well. Is there any chance of getting this merged?

@warnwar will look into it this week


⚠ Samsung has degraded performance

In 0bc634a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2005 ms

Resolved: Samsung performance has improved in 90f5ab0.


Flexiblespacebar inkwell ontap misbehaving duplicate #60891

In FlexibleSpaceBar -> background attribute I've container -> column -> [stack, colum -> [ container, row]]
In row widget i have three Inkwell but only the first onTap is working others is not.
I duplicate the row with Inkwell with three simple text widget this time only first two inwell is working
I'm  not able to find what is the exact problem which overriding the ontap on not invoking.
FLUTTER VERSION : 2.5.2
DART VERSION : 2.14.3

@danagbemava-nc
[✓] Flutter (Channel stable, 2.5.2, on macOS 11.5 20G71 darwin-arm, locale
en-IN)
• Flutter version 2.5.2 at /Users/syedhussainmehdi/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 3595343e20 (11 days ago), 2021-09-30 12:58:18 -0700
• Engine revision 6ac856380f
• Dart version 2.14.3
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at /Users/syedhussainmehdi/Library/Android/sdk
• Platform android-31, build-tools 30.0.3
• Java binary at: /Applications/Android
Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.5, Build version 12E262
• CocoaPods version 1.10.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
[✓] IntelliJ IDEA Community Edition (version 2021.2)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 61.2.4
• Dart plugin version 212.4746.57
[✓] VS Code (version 1.61.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.27.0
[✓] Connected device (2 available)
• iPhone 12 Pro Max (mobile) • 9901ED86-CAAA-4EC4-931A-6BC408AB381E • ios
• com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
• Chrome (web)               • chrome                               •
web-javascript • Google Chrome 94.0.4606.81

@shmehdi01, please provide a complete minimal reproducible code sample as well.
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


[Editor] Update caret browsing mode when the pref is changed in Firefox (bug 1886586)

I'd appreciate if this functionally was a "better" fit for the existing preference/option handling, to avoid inconsistent state within the viewer and to prevent this becoming "hacky" if more preferences need similar treatment.
The fastest way for me to demonstrate what I'm imagining here was to throw together a patch: master...Snuffleupagus:pdf.js:updatePrefs

Ok let's do that.
You just have to change the listener name.


dbeaver: 7.2.0 -> 7.2.1

Motivation for this change
New upstream release https://dbeaver.io/2020/09/20/dbeaver-7-2-1/
Things done

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

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

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

ported to release-20.09 in 38e59a3385ad10d7970448948c68daa9208a8852


DGS Code Generation Maven Issue

I'm trying to build out a simple example with the DGS Code Generation Feature. I started by creating a brand new project with the following depeendencies

Spring Web
Spring for GraphQL
DGS Code Generation

From there I created a simple schema and placed it in /src/main/resources/graphql-client
type Post {
    id: ID!
    title: String
    body: String
    published: Boolean
    date: String
}

type Query {
    posts: [Post]
    post(id: ID!): Post
}

You can find this simple project here -> https://github.com/danvega/dgs
From the command line I ran the following command  ./mvnw graphqlcodegen:generate and it failed with the following error:
Execution default-cli of goal io.github.deweyjose:graphqlcodegen-maven-plugin:1.50:generate failed: Please specify a packageName ->

If you look in pom.xml you will see a packageName defined so I'm not sure why this is happening.
<configuration>
	<schemaPaths>
		<param>src/main/resources/graphql-client</param>
	</schemaPaths>
	<packageName>dev.danvega.dgs.codegen</packageName>
	<addGeneratedAnnotation>true</addGeneratedAnnotation>
</configuration>

This works using Gradle without any issues.

Sorry for the late reply.
I'm not sure we're meant to call the plugin directly as it's supposed to generated sources as part of the regular build. ./mvnw compile works for me and generates sources accordingly.
I haven't found any reference of the ./mvnw graphqlcodegen:generate task in the plugin documentation, nor whether this way would take our plugin configuration into consideration. Can you share more about that?
I guess if you'd like to call a new plugin execution on the CLI, you should also provide the relevant arguments:
$ ./mvnw clean graphqlcodegen:generate -DschemaPaths="src/main/resources/graphql-client" -DpackageName="com.example.dgsmaven.codegen" -DaddGeneratedAnnotation=true

The command above works for me.

I couldn't figure out how to use DGS so I was just guessing. I also ran ./mvnw compile and that worked for me, thank you!


Transfer the repository to the project or institute repository

I saw that this repository is quoted in the He-san's draft and I think that it would be an good opportunity to transfer this repository to either @Subaru-PFS (PFS project repository) or @Subaru-SciOp (Subaru's science operation repository). Which is more appropriate? @yukimoritani @takagi-naoj
I'm wondering if the transfer remove all issues as it seems that the Issues is disabled in the PFS repository. In this case, I should make them as Jira tickets before doing so.

I have talked with @yukimoritani about this, and we suggest:

Use Subaru-PFS to manage stuff developed by PFS project and obsproc (especially during the commissioning)
Use Sudaru-Sciop to manage stuff developed by Subaru science operation (like website or other tools that might be made by the SA in the future)

So I think it will be good to transfer it to Subaru-PFS.

sounds reasonable to me. Then, I have a few questions:

Can the repository name be kept as pfs_target_uploader or need to be changed to conform the naming convention of the project (e.g., spt_target_uploader)?
Do you know if Issues will be kept? I think Subaru-PFS disables Issues by default.

(I'm so sorry to be super late to this...) According to  our repository name convention,

pfs_ Repositories for project general
spt_ Survey Planning and Targetting (SPT) sofotware, includes survey coordination and operation, exposure time calculator, and so on.

So, spt_ sounds matched to me, as uploader cares observation plan.
Yes, to encourage discussion on JIRA, the repositories doesn't have Issue by default.. when I create a new one, I manually disable it. But because you already have had many discussions.. maybe we should think an exception.

Transfer is done. New repository is Subaru-PFS/spt_target_uploader. You can modify the origin of your local one by git remote set-url origin https://github.com/Subaru-PFS/spt_target_uploader.git.


Added the possibility of defining data attributes on actions

Example
...
"actions": [
    {
        "ident": "view_en",
        "label": {
            "en": "View",
            "fr": "Voir"
        },
        "target": "_blank",
        "dataAttributes": [
            {
                "key": "lang",
                "value": "en"
            }
        ],
        "condition": "{{ isEn }}",
        "url": "{{# withBaseUrl }}{{ url.en }}{{/ withBaseUrl }}"
    },
    {
        "ident": "view_fr",
        "label": {
            "en": "View",
            "fr": "Voir"
        },
        "cssClasses": "hidden",
        "target": "_blank",
        "dataAttributes": [
            {
                "key": "lang",
                "value": "fr"
            }
        ],
        "condition": "{{ isFr }}",
        "url": "{{# withBaseUrl }}{{ url.fr }}{{/ withBaseUrl }}"
    },
...

I would add:
if ($menuItem['data_attributes']) {
	$menuItem['data_attributes'] = $menuItem['data_attributes'];
}

Cause snake-case is part of our lives.

We should also consider adding a function or method to stringify an associative array to HTML attributes. Like in mcaskill/charcoal-support for the TableWidget.

I agree about snake casing, but ActionContainerTrait already makes use of camel case in a few places, so I followed suit.
buttonType actionType cssClasses.

True. I wonder if it shouldn't have just been all camelCase everywhere.


"ReferenceError: admin is not defined" on generateThumbnail

I deployed the generateThumbnail function, and uploaded the image file to Firebase Storage, but the following error was output.
ReferenceError: admin is not defined
    at exports.generateThumbnail.functions.storage.object.onFinalize (/srv/index.js:57:18)
    at cloudFunctionNewSignature (/srv/node_modules/firebase-functions/lib/cloud-functions.js:105:23)
    at /worker/worker.js:731:24
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:228:7

Looks like this was fixed by #486


⚠ Bofrost.be has degraded performance

In 525f6ed, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1156 ms

Resolved: Bofrost.be performance has improved in 071e763 after 6 minutes.


Unexpected behaviour for object property assignments.

Code like:
class Test[A](name: String, left: A, right: A) {
    var @name = name
    var @left = left
    var @right = right

    define output {
        print("----")
        print(self)
        print(@name)
        print(@left)
        print(@right)
    }
}

var a0 = Test("a0", 0, 0)
var a1 = Test("a1", 1, 1)
var b = Test("b", a0, a0)
var c = Test("c", a1, a1)

print("==== BEFORE ====")
a0.output()
a1.output()
b.output()
c.output()

b.@left = c
c.@right = b

print ("\n==== AFTER ====")
a0.output()
a1.output()
b.output()
c.output()

results in output like (with lily 4973a917612c85f24be2f852af2f75f0f2722cad):
==== BEFORE ====
----
<Test at 0x7f94a1405b90>
a0
0
0
----
<Test at 0x7f94a1405c20>
a1
1
1
----
<Test at 0x7f94a1405cb0>
b
<Test at 0x7f94a1405b90>
<Test at 0x7f94a1405b90>
----
<Test at 0x7f94a1405d40>
c
<Test at 0x7f94a1405c20>
<Test at 0x7f94a1405c20>

==== AFTER ====
----
<Test at 0x7f94a1405b90>
a0
0
0
----
<Test at 0x7f94a1405c20>
a1
1
1
----
<Test at 0x7f94a1405d40>
c
<Test at 0x7f94a1405c20>
<Test at 0x7f94a1405c20>
----
<Test at 0x7f94a1405d40>
c
<Test at 0x7f94a1405c20>
<Test at 0x7f94a1405c20>

In the "BEFORE" output, there are four distinct objects.
Then some properties of b and c are assigned to. b and c themselves aren't assigned to, though.
In the "AFTER" output, b appears to be referring to c, although b itself wasn't assigned to.
Not sure if this is a bug, or just a misunderstanding on my part...
I would have expected b and c to continue to refer to the original objects, since it was only their properties that were assigned to.
Should those assignments even be allowed? If I'm understanding this correctly, then:

a0 and a1 would have type Test[Integer]
b and c would have type Test[Test[Integer]]
b.@left, b.@right, c.@left, c.@right would have type Test[Integer] since they were initialized with a0 or a1

It seems like it shouldn't be possible to assign Test[Test[Integer]] values to properties with type Test[Integer]?

b.@left = c
c.@right = b

Huh, I thought I had prevented that. If you do b.left and c.right, then a syntax error is properly printed.


Merge terra branch into master

Description

After a few people look at the terra branch let’s merge it in
Your Environment

QISKit version:
Python version:

This looks good to me. There are probably a few places to polish, but they can be done in the bigger tutorials refactor.


Configure proportional routing fee consistently

Fixes https://github.com/get10101/10101/issues/927.
Fixes https://github.com/get10101/meta/issues/178.

@holzeis gave me the thumbs up privately.
bors r+


Add meta support to __setitem__

With changes in this PR it's now possible to assign values to Dask arrays backed by CuPy and other libraries supporting __array_function__.

Thanks Peter! 😄

Thanks @jakirkham for review and merge!


please update for Rails 6

Done. In fact we don't depend on active_support/time anymore, so that we don't need it. So I updated the dependencies, thanks!


[linkis] error reported when executing hive SQL: Cannot initialize Cluster. Please check your configuration for mapreduce.framework.name and the correspond server addresses.

Search before asking

[X] I searched the issues and found no similar issues.

Linkis Component
linkis-commons
What happened + What you expected to happen
[linkis] error reported when executing hive SQL: Cannot initialize Cluster. Please check your configuration for mapreduce.framework.name and the correspond server addresses.
Relevent platform
.
Reproduction script
.
Anything else
This question comes from the QA documentation of the Linkis community
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

A: You can refer to this solution, https://stackoverflow.com/questions/19043970/cannot-initialize-cluster-please-check-your-configuration-for-mapreduce-framewo
尝试解决


[BUG] TF dataloader stop iteration restart

This bug reports an issue with the tensorflow iteration logic. Specifically with the termination logic. Currently when we stop iteration we actually restart the iterator and give the first batch from the iterator again.

adding pytest that shows the issue here
@pytest.mark.parametrize("batch_size", [10, 9, 8])
@pytest.mark.parametrize("skip_last", [True, False])
@pytest.mark.parametrize("num_rows", [100])
def test_tf_catname_ordering(tmpdir, batch_size, skip_last, num_rows):
    df = cudf.DataFrame(
        {
            "cat1": [1] * num_rows,
            "cat2": [2] * num_rows,
            "cat3": [3] * num_rows,
            "label": [0] * num_rows,
            "cont3": [3.0] * num_rows,
            "cont2": [2.0] * num_rows,
            "cont1": [1.0] * num_rows,
        }
    )
    path = os.path.join(tmpdir, "dataset.parquet")
    df.to_parquet(path)
    cat_names = ["cat3", "cat2", "cat1"]
    cont_names = ["cont3", "cont2", "cont1"]
    label_name = ["label"]

    data_itr = tf_dataloader.KerasSequenceLoader(
        [path],
        cat_names=cat_names,
        cont_names=cont_names,
        batch_size=batch_size,
        label_names=label_name,
        shuffle=False,
        skip_last=skip_last
    )

    all_rows=0
    for X, y in data_itr:
        all_rows += len(X["cat1"])
        assert list(X["cat1"].numpy()) == [1] * batch_size
        assert list(X["cat2"].numpy()) == [2] * batch_size
        assert list(X["cat3"].numpy()) == [3] * batch_size
        assert list(X["cont1"].numpy()) == [1.0] * batch_size
        assert list(X["cont2"].numpy()) == [2.0] * batch_size
        assert list(X["cont3"].numpy()) == [3.0] * batch_size

    if skip_last and num_rows % batch_size > 0:
        assert num_rows > all_rows
    else:
        assert num_rows == all_rows


collapsePanel() speed

Is it possible to change the speed with which the panel collapses?

Not currently.


Extend EventTarget to support event bubbling

This solves: https://github.com/ungap/event-target/issues/8
As discussed there, EventTarget is extended to add bubbling and parent node knowledge only in the DOM context.

Couple of changes needed, and I'd love to have stopImmediatePropagation() tested too, otherwise, awesome job!

Couple of changes needed, and I'd love to have stopImmediatePropagation() tested too, otherwise, awesome job!

Thanks!
I can't really test the full use case without updating "ungap/event-target" (I could do a PR for that later if that's working for you), but I'll add more tests to cover stopImmediatePropagation here.

Couple of changes needed, and I'd love to have stopImmediatePropagation() tested too, otherwise, awesome job!

Thanks!
I can't really test the full use case without updating "ungap/event-target" (I could do a PR for that later if that's working for you), but I'll add more tests to cover stopImmediatePropagation here.

Oh ... I see. Sorry, busy days in here ... I'll try to update/change that later on but if you are, somehow, in a rush, please go ahead with the PR (ungap is a bit weird to work with though, no modern JS and stuff) ... as you prefer.

Oh ... I see. Sorry, busy days in here ... I'll try to update/change that later on but if you are, somehow, in a rush, please go ahead with the PR (ungap is a bit weird to work with though, no modern JS and stuff) ... as you prefer.

No worries, I didn't mean to rush you :)
I can't say I am in a particular hurry, but I can understand you would be busy and I don't mind at all doing a PR there (especially since I already pulled the code and played with it... so it's mostly ready).
To be honest the only thing I was confused about were the unit tests (in ungap) and for my local tests I used your assert there (but I'll refactor that to follow existing code).

So ... what are we missing here? is that still ungap for the event target? Thanks!

So ... what are we missing here? is that still ungap for the event target? Thanks!

yes, I can make a PR for that next, it should be way simpler.

@mikemadest this looks good to me, but I don't understand why there are unrelated changes in this MR ... by any chance you can remove those and pull rebase from main instead?
git pull --rebase origin main

I'd rather keep changes per PR/MR, thanks. Once you do that, I'll squash merge and publish.

@mikemadest this looks good to me, but I don't understand why there are unrelated changes in this MR ... by any chance you can remove those and pull rebase from main instead?
git pull --rebase origin main

I'd rather keep changes per PR/MR, thanks. Once you do that, I'll squash merge and publish.

Sorry I'm not sure what happened there, will cleanup

@mikemadest this looks good to me, but I don't understand why there are unrelated changes in this MR ... by any chance you can remove those and pull rebase from main instead?
git pull --rebase origin main

I'd rather keep changes per PR/MR, thanks. Once you do that, I'll squash merge and publish.

ok so I was just behind main because of other merge, this looks all good now

v0.11.1 is up and running 🍻
Thanks a lot for your help ❤

v0.11.1 is up and running 🍻
Thanks a lot for your help ❤

thank you for bearing with me :)


Numbericupdown data binding error

Describe the bug
When the keyboard is used to change the value, the Numbericupdown control produces a data binding error.
To Reproduce

My xaml use NumericUpDown control like below:

<Controls:NumericUpDown Minimum="1" Maximum="8" TextAlignment="Left" Value="{Binding Path=SignalWave, Mode=TwoWay}"  UpDownButtonsWidth="32" >
Note: The Maximum is 8.

And my the binding source property defind like below:

public int SignalWave
{
get
{
return mSetting.ETConfig.SignalWave + 1;
}
set
{
if (mSetting.ETConfig.SignalWave + 1 != value)
{
mSetting.ETConfig.SignalWave = value - 1;
Console.WriteLine("Current signal wave {0}", mSetting.ETConfig.SignalWave);
OnPropertyChanged("SignalWave");
}
}
}

Change the value by keyboard, i.e. you can input 9
Error: The 'SignalWave' is changed to 9, but 9 is an invalid value
The control won't change to a valid value until it lost focus,even i press ENTER key.

@jackylinwh Why do you inc and dec in code behind? The NUD do this for you.


Add Support for Faroro Pet Feeder

I've been using and tweaking this config for over a year. A mostly normal pet feeder with a a couple quirks-

Manual Feed has mode set to box, slider resulted in accidental dispensing on tap
Manual Feed has persist set to false, otherwise brief disconnections would result in food dispensing

All changes have been made, thank you for your help!


Tracker Functionality Addition - Distance From Player/Tracker Update

New Functionality for Vehicle Tracking

Tracker windows now show distance from the player
Added "Ping" button on each Tracker window to update the information. This updates the current vehicle/engine health, and the distance from the player.

Minor changes

Made vehicle/engine health display as a flat integer and not a float.

Note: Distance tracking is intended for use on player vehicles. Anything that despawns will cause the tracker to misbehave. Additional fixes are forthcoming.

Closing PR after discovery of minor issues. Resolving then opening new PR.


Feature/multi access manager

Allows for multiple access managers to be used.
Time permitting this week (and figuring it out) will be creating a fake Session handler and modifying the builds to use TestTemplates for each possible method. But if not done with that by Thursday will need to rely on manual testing to get this out to LRL for them to continue on working on their usage.

Still a work in progress, but feedback would be good before going too much further. Next step is mimicing the Tomcat Session store so I we "test" CWMSAAA CaC Auth users. Hooks in place, need to sort out details. but this would let us loop over various users at different times.

Traefik and authelia are new to me.  Nothing in here seems crazy yet.

Yeah, those are for the docker-compose to mimic a more complete environment for manual testing. They won't affect the automated tests except perhaps to inform some direct... might need authelia in a testcontainer though, OpenID Connect is simple but the flow is difficult to mock simply.

Okay, this gets the basic idea across. We'll need to expand more of the tests to actually use all the possible credential types but I'd like to get a release of new features out and this does properly test the multiple access manager mechanism.

Traefik/authelia will be used in future work for actually Keycloak/Login.gov integration. Original scope was just OpenID but I realize these changes are required first.

I'd like to just merge this in as-is. I've started on the actual keycloak integration but that should just be a separate PR.
This gives us a working multi auth system and a new structure to the tests. Future tests with auth should be parameterized like those in AccessManagerTestIT.java


Use brackets for array indexes on path formatting

Format array indexes with brackets in the path:
Actual: data.0.name
Expected: data.[0].name (or data[0].name)
See https://github.com/OliverJAsh/io-ts-reporters/issues/3#issuecomment-305706904 for a potential implementation (might be outdated)

I prefer data[0].name since it mirrors how you would express it in JS/TypeScript


Fixed issue with adding categories

Fixed issue with adding categories in Control/Action/Edit this Action/Tag Settings/Categories
Error message: undefined method 'id' for nil:NilClass [miq_policy/tree_select]

Reviewed, tested, looks good.

@ZitaNemeckova possible to add a spec that fails before and succeeds with your fix?

Spec added as requested and code refactored


Remove dependency on SHA to validate generated html

The previous version relied on the SHA file checksum value to validate that the test notebook was properly converted to HTML.  However, a change to a transitive dependency changed the HTML output - breaking the test (and CI).
This change simply removes that dependency and confirms the generated HTML begins and ends with the appropriate HTML tags and ensures a small portion of code from the notebook file is properly converted to HTML.
Fixes #67
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@lresende or @akchinSTC - could one of you please review (and merge) this PR so all sequent PRs don't produce CI failures that make their review more difficult?


v2.15

I think this commit changed the behaviour so that the displayTrack logic always chats the track being played. Is there a way to turn that off ?

I think this commit changed the behaviour so that the displayTrack logic always chats the track being played. Is there a way to turn that off ?


Build a Snowflake Native App to Analyze Chairlift Sensor Data

A few optional nits, overall looks good to me!

Thank you! Updated.


Preserving css variables

Maybe I'm declaring this wrong, my custom variables are being processed via cssnext, but I want them preserved as is.
Part of my gulp task:
cssnext({
  // I tried this option first, thinking it would ignore all custom properties. It doesn't.
  features: ['customProperties: false']

  // Then I tried this option based on this thread, https://github.com/MoOx/postcss-cssnext/issues/336
// Has no effect:
  customProperties: ['preserve: true']
})

In other words…
:root {
  --mainColor: #333;
}

body {
  color: var(--mainColor);
}

…should return exactly that, NOT:
body {
  color: #333
}

God bless you @djmtype.


[webpack-import-meta-loader] webpack_public_path is not being taken into account

When using the webpack-import-meta-loader plugin to rewrite import.meta to a correct fixed url, the webpack_public_path variable is not being taken into account. This variable (dynamically) defines the root path of the project.

I'll try to whip up a PR for this. :-)

What's the public path intended for? I tried it out before, but it didn't seem to change the build output.

What's the public path intended for? I tried it out before, but it didn't seem to change the build output.

The public path is used to define the http path of the root of your build output.
In some cases the (static) build output (js,css,images, ..) is stored on a different path than the bootstrap html. Some use cases:

using a cdn for the assets# bootstrap html
https://www.example.com/path/to/page
# asset url
https://cdn.example.com/my-project/2.0.0/bootstrap.js

the same bootstrap html is used under different paths# bootstrap html
https://www.example.com/path/to/page
https://www.example.com/path/to/sub/page
# asset url
https://www.example.com/bootstrap.js

The default way to solve this is to use the base tag to change the baseURI. Unfortunately this will also change the behavior of relative anchor tags. When defining the Public Path, you define the root (eg. https://cdn.example.com/my-project/2.0.0/) of the project. This allows WebPack to resolve dynamic modules correctly. The definition can be done at build time or at runtime.
import.meta however is often used to get the root of some component, but doesn't take into account these different paths.

Happy to accept a PR, but in the interest of honesty we're also looking to deprecate the import meta loader in favor of the babel import meta plugin.
We already use this babel plugin for production builds, but use the import meta loader during development because it's faster than running babel just for the import meta rewrite. Since we're going to start promoting https://open-wc.org/developing/es-dev-server.html for development, we might as well drop the meta loader as well especially since we run into quite some limitations with it.

We will probably switch the babel import meta plugin as well, but we'd like to finish our first production build as well first, before changing again. :-)
When using babel to transform import.meta and keep the same functionality, we'll need to investigate how to support the dynamic examples of above.

that's pretty interesting  stuff, but I'm not sure if that should be within the scope of any tool that handles import.meta.
The goal of the webpack and babel plugin is to preserve the import.meta.url value from before bundling, and to mimick the same behavior as if you would use import.meta.url natively in the browser. If you use import.meta.url in a native module it would also not know about the public path, right?

The goal of the webpack and babel plugin is to preserve the import.meta.url value from before bundling, and to mimick the same behavior as if you would use import.meta.url natively in the browser. If you use import.meta.url in a native module it would also not know about the public path, right?

Indeed the plugins are there to  mimick the same behavior to be able to use import.meta.url as if it was natively. The issue is that the value of import.meta.url cannot completely be known at build time. Let me take  https://cdn.example.com/assets/project/src/file.js as an example.

The part that can be known at build time is the path within the project ./src/file.js. This file could have been generated by bundling multiple modules together. Let's call it the "local path".
The part before it (https://cdn.example.com/assets/project/) is only known at runtime. Let's call this the "prefix".

Both the prefix and the local path are needed to have the full url. So, a native module would intrinsically know about the prefix. And the way webpack allows to set/use this dynamic prefix, is to define __webpack_public_path__. So, when using webpack, I feel this could be a way to make sure the value is correct.

My point is that if you would use the native browser module system and have import.meta.url in there, would it take into account the webpack public path?

The polymer cli will create the import.meta.url url by combining at runtime:

a runtime part: the meta.url
This serves as base url and is the absolute url of the (combined) amd script file.
meta is an amd dependency inherently provided by the esm-amd-loader.
a static / build part, the path to the original module
This path is relative to the build (combined) script file. It's computed at build time by polymer-build and babel.


Installation failed: Unable to load cache

Hi, I am having issue installing the sick_scan_xd driver Windows version without ROS. I have Visual Studio 2022 v17.8.4, would that be an issue?
Please find log attached.
Thank you.
Log20241113.txt

Thanks for your feedback. The cmake generator string for Visual Studio should be "Visual Studio 17 2022" (instead of "Visual Studio 17.8.4 2022"). To build with Visual Studio 2022, run the following commands:
set _os=x64
set _cmake_string=Visual Studio 17 2022
set _msvc=Visual Studio 2022
set _cmake_build_dir=build
if not exist %_cmake_build_dir% mkdir %_cmake_build_dir%
pushd %_cmake_build_dir%
cmake -DROS_VERSION=0 -G "%_cmake_string%" ..

Open sick_scan_xd.sln in Visual Studio 2022 and build.
Depending on your system, you may need to set the Visual Studio environment explicitly by calling "VsDevCmd.bat":
call "%ProgramFiles(x86)%\Microsoft Visual Studio\2022\Community\Common7\Tools\VsDevCmd.bat" -arch=amd64 -host_arch=amd64
Change the path to Visual Studio if it is installed in a different folder.

Hi ROSTest,
Sorry I tried the method above but encounter the error
CMake Warning:
Ignoring extra path from command line:
".."
CMake Error: The source directory "C:/Users/teebo" does not appear to contain CMakeLists.txt.
Specify --help for usage,
Log20241115.txt
or press the help button on the CMake GUI
Please see log attached.
Also I tried looking VSDevCmd.bat, but cannot find the directory VisualStudio2022
(See photo)

Thank you.
Best Regard,
Alicia Tee

Thanks for following up. You have to clone the sick_scan_xd repository first into a folder (sick_scan_xd by default) and then execute these commands in that folder (i.e. in folder sick_scan_xd). Here is the complete command sequence to clone and build sick_scan_xd with Visual Studio 2022:
mkdir ws_sick_scan_xd\src
cd ws_sick_scan_xd\src
git clone https://github.com/SICKAG/sick_scan_xd.git
cd sick_scan_xd
set _os=x64
set _cmake_string=Visual Studio 17 2022
set _msvc=Visual Studio 2022
set _cmake_build_dir=build
mkdir %_cmake_build_dir%
pushd %_cmake_build_dir%
cmake -DROS_VERSION=0 -G "%_cmake_string%" ..
REM Open sick_scan_xd.sln in Visual Studio 2022 or run cmake --build .
REM cmake --build . --clean-first --config Debug
REM cmake --build . --clean-first --config Release
popd

The workspace folder ws_sick_scan_xd is just an example, feel free to rename it.
Open file ws_sick_scan_xd\sick_scan_xd\build\sick_scan_xd.sln in Visual Studio 2022 and build.
Depending on your system, you may need to set the Visual Studio environment explicitly by calling "VsDevCmd.bat":
call "%ProgramFiles(x86)%\Microsoft Visual Studio\2022\Community\Common7\Tools\VsDevCmd.bat" -arch=amd64 -host_arch=amd64
Change the path to Visual Studio if it is installed in a different folder.


Home, Find Projects and About section links are not working in the Footer section

Describe the bug/feature
In the footer section Login/Signup, Contact Us, Trending Projects links are working fine but the above mentioned 3 links are not working fine as there are not redirected to the correct path.
I would like to work on this issue.
Could you please assign this issue to me under LGMSOC so that I can fix this bug.
Additional context

Hey @ROHAN842 bro, only the Find Projects page is not created yet so it is right. And the only thing that needs to be changed is About. Only change the about route everything is fine. You are given 2 days to do it.

@rockingrohit9639 OK


Color palettes should be defined in terms of Color, not specific color spaces

What problem does this solve or what need does it fill?
In the course of #12163, I've been working to use the polymorphic Color enum as the primary way for end users to supply colors.
Because the palettes are stored in Srgba, this results in a ton of .into() or Color::from calls.
What solution would you like?
Just define these palettes in terms of Color.
Additional context
Migration tracking issue can be found at #12056.
I don't particularly care whether this or #12163 is done first.

I was really hoping to minimize the use of Color.

Yeah, I'm not thrilled by it either. But if all of our APIs where we might use a color palette take a Color, it seems silly to store them in any other format. As to whether we should use that particular form of API, I suppose that's the controversial question in #12163.

Well, in my original plan the Gizmo APIs (for example) would take an Into<LinearRgba>, not an Into<Color>.
The problem with using Color everywhere is that it negates some of the advantages of making bevy_color in the first place: you can no longer do color operations like mix, darken etc., you have runtime checks and so on.

So, we would use some type of color as the library default? For example, in gizmos, the default would be LinearRgba?

Also, UI themes are certainly going to be defined in Srgba. For example, I have code that looks like this:
if hovering {
    BUTTON_BG.lighten(0.1)
} else if pressed {
    BUTTON_BG.lighten(0.2)
} else {
    BUTTON_BG
}

I can't do this if BUTTON_BG is Color (at least, not cheaply). So now we have two different kinds of palette - one for UIs, one for non-UIs.
I can't speak for everyone else, but for me having a lot of .into() calls isn't necessarily a bad thing, if it tells me "Hey, there's a conversion here you should be aware of."

Well, in my original plan the Gizmo APIs (for example) would take an Into<LinearRgba>, not an Into<Color>.
The problem with using Color everywhere is that it negates some of the advantages of making bevy_color in the first place: you can no longer do color operations like mix, darken etc., you have runtime checks and so on.

I'd love to be able to do this, but it doesn't work for bundle types. We're constructing the fields directly :(

Also, I want to point out that the decision of whether to use Color or one of the other types also affects when the conversion is done. Using Color means that the conversion is deferred until the color is dereferenced - in other words, lazily. Using Srgba or LinearRgba means that the conversion is done eagerly.
This can affect performance, although the specific impact will depend on how often the color field is evaluated. If the conversion is eager, it happens when the user calls the API. If the conversion is lazy, it happens during extraction or whatever internal system Bevy uses for processing the color. The question is, which of those operations happens more often? The answer of course is that it depends on which part of Bevy we're talking about.

One final (hopefully) comment: I think it's important to accept that there is no perfect solution here. There's an inherent tradeoff between these choices, and we have to pick one or the other and accept the consequences. Using Color as the basis for APIs does make the code more succinct, at least for the examples. But are we optimizing for examples, or for real-world use cases?
Real games aren't going to use the standard palettes much, if at all. The main user will be debugging displays and gizmos, but even these may opt to define their own color constants.

Well, I said I was going to shut up about this, but I have one more thought: There's a reason why these colors are defined in sRGB and not in other color spaces, because this is what sRGB is for. The CSS colors, for example, are defined in terms of sRGB values, as are the tailwind colors. If the user needs them in another space, we've made it easy enough to convert.


Problem opening files

I am now getting a few users saying they are getting a file not found error when my application reads a XML file (using this library).
I have had a look at this method:
static FILE* callfopen( const char* filepath, const char* mode )
{
    TIXMLASSERT( filepath );
    TIXMLASSERT( mode );
#if defined(_MSC_VER) && (_MSC_VER >= 1400 ) && (!defined WINCE)
    FILE* fp = 0;
    errno_t err = fopen_s( &fp, filepath, mode );
    if ( err ) {
        return 0;
    }
#else
    FILE* fp = fopen( filepath, mode );
#endif
    return fp;
}

I have noticed that if there is a error with the fopen_s call that we lose the actual error number.
I have changed it like this:
XMLError XMLDocument::LoadFile( const char* filename )
{
    Clear();
int iErrorCode = 0;
    FILE* fp = callfopen( filename, "rb", &iErrorCode );

    if ( !fp ) {
	    if (iErrorCode == ENOENT)
		    SetError( XML_ERROR_FILE_NOT_FOUND, filename, 0 );
		else
		{
    		char *sErrorNo = new char(15);
			XMLUtil::ToStr(iErrorCode, sErrorNo, 15);
			SetError(XML_ERROR_FILE_COULD_NOT_BE_OPENED, filename, sErrorNo);
			delete sErrorNo;
		}

        return _errorID;
    }
    LoadFile( fp );
    fclose( fp );
    return _errorID;
}

I don't know if it is the best way. But at least I can get at error number now.

Perhaps there is a better way to do it?

So the problem is that when a file exists but cannot be opened the library returns "not found" which is misleading, isn't it?

Exactly. I can't say exactly what the problem is on their computers. The file is in the right place, the path is correct. Yet for about 4 people around the world so far it will not read. And the error says file not found.
I have removed my code as I did not like what I had done. I would like something better designed that provides me why the file did not open (as it is present).

For example, if the user does not have access permissions, it should return that as the error.

That's going to be tough to tackle in a generic way - handling error pass through from the OS open routines. Would renaming the error have helped? (FILE_NOT_FOUND_OR_COULDNT_OPEN?)

Well, I came across the web link:
https://msdn.microsoft.com/en-us/library/t3ayayh1.aspx
Not sure if you have seen that before. Your code eventually calls fopen_s:
https://msdn.microsoft.com/query/dev14.query?appId=Dev14IDEF1&l=EN-US&k=k("stdio%2Ffopen_s");k(fopen_s);k(DevLang-C%2B%2B);k(TargetOS-Windows)&rd=true
That webpage directs you to the former for the errors.
So I can't see why it is not possible to add a bespoke error number and even text string that is just used by the open/save routines and public methods to access those values.
I really would like us to know why it failed and it seems that we can access this error information using the system. Yes?

@ajtruckle Good, but the code needs to work on multiple platforms which means careful analysis is needed.

I leave that in your capable hands. Beyond me ... I'll break it!

A good, simple, cross platform way to return more detailed error codes is going to be tricky and a lot of fiddle, hard to test code. I think a better approach for @ajtruckle to handle errors might be this:
// Call with the OS variant you use
FILE* fp  = fopen( "myfile.xml", "rb" );
// ...and handle the errors as needed.

doc.LoadFile(fp);
fclose(fp);

I can look, thanks. I think I must use fopen_s.
And for saving?
Sent from myMail app for Android Saturday, 13 February 2016, 05:10pm +00:00 from Lee Thomason <<EMAIL_ADDRESS>:

A good, simple, cross platform way to return more detailed error codes is going to be tricky and a lot of fiddle, hard to test code. I think a better approach for  @ajtruckle to handle errors might be this:
// Call with the OS variant you use
FILE* fp  = fopen( "myfile.xml", "rb" );
// ...and handle the errors as needed.

doc.LoadFile(fp);
fclose(fp);

—
Reply to this email directly or  view it on GitHub .

FILE* fp = fopen("myfile.xml", "w");
// errors, etc.

doc.SaveFile(fp);
fclose(fp);

It's better to use 'b' on open so that TinyXML2 can do newline processing / normalization. But it's probably better to use the OS end-of-lines on save, so omit the 'b' on writing.

But what if you will do read and write with the same FILE buffer?
Sent from myMail app for Android Saturday, 13 February 2016, 05:32pm +00:00 from Lee Thomason <<EMAIL_ADDRESS>:

FILE* fp = fopen("myfile.xml", "w");
// errors, etc.

doc.SaveFile(fp);
fclose(fp);

It's better to use 'b' on open so that TinyXML2 can do newline processing / normalization. But it's probably better to use the OS end-of-lines on save, so omit the 'b' on writing.
—
Reply to this email directly or  view it on GitHub .

I got the saving to work and test accordingly:
	FILE *fStream; // We save it this way so that we can use UNICODE filenames
	errno_t e = _tfopen_s(&fStream, strStudentXML, _T("wt"));
	if (e == 0)
	{
		tinyxml2::XMLError eXML = myDoc.SaveFile(fStream);
		if (eXML == tinyxml2::XMLError::XML_SUCCESS)
		{
			fclose(fStream);
			AfxMessageBox(IDS_STR_STUDENT_INFO_EXPORTED, MB_OK | MB_ICONINFORMATION);
		}
		else
		{
			fclose(fStream);
			CString strError, strErrorCode(CA2CT(myDoc.GetErrorStr1(), CP_UTF8));
			// TODO: Use STRINGTABLE
			strError.Format(_T("An error was encountered when trying to save the file:\n\nFile: %s\nError: %s\nError code: %d"),
				strStudentXML, strErrorCode, eXML);
			AfxMessageBox(strError, MB_OK | MB_ICONINFORMATION);
		}
	}
	else
	{
		CString strError, strErrorCode;
		_tcserror_s(strErrorCode.GetBufferSetLength(_MAX_PATH), _MAX_PATH, errno);
		strErrorCode.ReleaseBuffer();

		// TODO: Use STRINGTABLE
		strError.Format(_T("An error was encountered when trying to save the file:\n\nFile: %s\nError: %s\nError code: %d"),
			strStudentXML, strErrorCode, errno);

		AfxMessageBox(strError, MB_OK | MB_ICONINFORMATION);
	}

Will check the reading now.

I have just done the reading code too. So I leave it with you for tweaking the error descripting for the load/save when one passes a file name.
Thanks for the suggestion.


Notifications not working due to snapshot or clip not being available

Describe the bug
Not getting notifications after updating Frigate to 0.14. However, I can see the events in Frigate (Proxmox) and with Home Assistant.
Expected behavior
Getting notifications with clip and snapshot as before updating Frigate to 0.14.
 Press Ctrl-C to quit.
2024/09/26 23:30:17 +0200 INF Connected to MQTT.
2024/09/26 23:30:17 +0200 INF Subscribed to MQTT topic: frigate/events
2024/09/26 23:32:40 +0200 INF New event received event_id=1727386359.746662-7hdh9y
2024/09/26 23:32:40 +0200 INF Event Detected camera=garaje event_id=1727386359.746662-7hdh9y label=person zones=
2024/09/26 23:32:40 +0200 INF Event dropped - No snapshot or clip available event_id=1727386359.746662-7hdh9y
2024/09/26 23:32:40 +0200 INF Event update received event_id=1727386359.746662-7hdh9y
2024/09/26 23:32:40 +0200 INF Event Detected camera=garaje event_id=1727386359.746662-7hdh9y label=person zones=
2024/09/26 23:32:40 +0200 INF Event dropped - Already notified on this zone camera=garaje event_id=1727386359.746662-7hdh9y label=person zones=
2024/09/26 23:32:46 +0200 INF New event received event_id=1727386365.908437-0zftc0
2024/09/26 23:32:46 +0200 INF Event Detected camera=garaje event_id=1727386365.908437-0zftc0 label=person zones=
2024/09/26 23:32:46 +0200 INF Event dropped - No snapshot or clip available event_id=1727386365.908437-0zftc0
2024/09/26 23:32:46 +0200 INF Event update received event_id=1727386365.908437-0zftc0
2024/09/26 23:32:46 +0200 INF Event Detected camera=garaje event_id=1727386365.908437-0zftc0 label=person zones=
2024/09/26 23:32:46 +0200 INF Event dropped - Already notified on this zone camera=garaje event_id=1727386365.908437-0zftc0 label=person zones=

Additional context
Only one camera, motion mask, not zones.
I check the clips file and the snapshot does exist (see screenshots).
Screenshots

Possibly my problem is the same I also do have frigate 0.14.

Hi,
Thank you for the quick reply and fix. I remotely updated to v0.3.5. However I need to be there to activate the motion, I'll came back here soon.

Working! I'll just post the logs so you can see actual behaviour.
 2024/09/27 17:05:18 +0200 INF Frigate Notify - v0.3.5-dev 2024/09/27 17:05:18 +0200 INF Starting... 2024/09/27 17:05:18 +0200 INF Config loaded. 2024/09/27 17:05:18 +0200 INF Successfully connected to http://<IP_ADDRESS>:5000 2024/09/27 17:05:18 +0200 INF Config file validated! 2024/09/27 17:05:18 +0200 INF App running. Press Ctrl-C to quit. 2024/09/27 17:05:18 +0200 INF Connected to MQTT. 2024/09/27 17:05:18 +0200 INF Subscribed to MQTT topic: frigate/events 2024/09/27 19:11:41 +0200 INF New event received event_id=1727457100.352984-g0hw1c 2024/09/27 19:11:41 +0200 INF Event Detected camera=garaje event_id=1727457100.352984-g0hw1c label=person zones= 2024/09/27 19:11:41 +0200 INF Event dropped - No snapshot or clip available event_id=1727457100.352984-g0hw1c 2024/09/27 19:11:41 +0200 INF Event update received event_id=1727457100.352984-g0hw1c 2024/09/27 19:11:41 +0200 INF Event Detected camera=garaje event_id=1727457100.352984-g0hw1c label=person zones= 2024/09/27 19:11:41 +0200 INF Alert sent event_id=1727457100.352984-g0hw1c provider=Ntfy 2024/09/27 19:11:47 +0200 INF Event update received event_id=1727457100.352984-g0hw1c 2024/09/27 19:11:47 +0200 INF Event Detected camera=garaje event_id=1727457100.352984-g0hw1c label=person zones= 2024/09/27 19:11:47 +0200 INF Event dropped - Already notified on this zone camera=garaje event_id=1727457100.352984-g0hw1c label=person zones= 2024/09/27 19:12:12 +0200 INF New event received event_id=1727457132.007978-ylv80h 2024/09/27 19:12:12 +0200 INF Event Detected camera=garaje event_id=1727457132.007978-ylv80h label=person zones= 2024/09/27 19:12:12 +0200 INF Event dropped - No snapshot or clip available event_id=1727457132.007978-ylv80h 2024/09/27 19:12:13 +0200 INF Event update received event_id=1727457132.007978-ylv80h 2024/09/27 19:12:13 +0200 INF Event Detected camera=garaje event_id=1727457132.007978-ylv80h label=person zones= 2024/09/27 19:12:13 +0200 INF Alert sent event_id=1727457132.007978-ylv80h provider=Ntfy 2024/09/27 19:12:19 +0200 INF Event update received event_id=1727457132.007978-ylv80h 2024/09/27 19:12:19 +0200 INF Event Detected camera=garaje event_id=1727457132.007978-ylv80h label=person zones= 2024/09/27 19:12:19 +0200 INF Event dropped - Already notified on this zone camera=garaje event_id=1727457132.007978-ylv80h label=person zones= 2024/09/27 19:13:12 +0200 INF New event received event_id=1727457190.178922-t4nh2z 2024/09/27 19:13:12 +0200 INF Event Detected camera=garaje event_id=1727457190.178922-t4nh2z label=person zones= 2024/09/27 19:13:12 +0200 INF Event dropped - No snapshot or clip available event_id=1727457190.178922-t4nh2z 2024/09/27 19:13:13 +0200 INF Event update received event_id=1727457190.178922-t4nh2z 2024/09/27 19:13:13 +0200 INF Event Detected camera=garaje event_id=1727457190.178922-t4nh2z label=person zones= 2024/09/27 19:13:13 +0200 INF Alert sent event_id=1727457190.178922-t4nh2z provider=Ntfy 2024/09/27 19:13:20 +0200 INF New event received event_id=1727457198.657951-7sy7rs 2024/09/27 19:13:20 +0200 INF Event Detected camera=garaje event_id=1727457198.657951-7sy7rs label=person zones= 2024/09/27 19:13:20 +0200 INF Event dropped - No snapshot or clip available event_id=1727457198.657951-7sy7rs 2024/09/27 19:13:20 +0200 INF Event update received event_id=1727457198.657951-7sy7rs 2024/09/27 19:13:20 +0200 INF Event Detected camera=garaje event_id=1727457198.657951-7sy7rs label=person zones= 2024/09/27 19:13:20 +0200 INF Alert sent event_id=1727457198.657951-7sy7rs provider=Ntfy 2024/09/27 19:13:24 +0200 INF New event received event_id=1727457202.52572-jpb32b 2024/09/27 19:13:24 +0200 INF Event Detected camera=garaje event_id=1727457202.52572-jpb32b label=person zones= 2024/09/27 19:13:24 +0200 INF Event dropped - No snapshot or clip available event_id=1727457202.52572-jpb32b 2024/09/27 19:13:25 +0200 INF Event update received event_id=1727457202.52572-jpb32b 2024/09/27 19:13:25 +0200 INF Event Detected camera=garaje event_id=1727457202.52572-jpb32b label=person zones= 2024/09/27 19:13:25 +0200 INF Alert sent event_id=1727457202.52572-jpb32b provider=Ntfy 2024/09/27 19:13:27 +0200 INF Event update received event_id=1727457198.657951-7sy7rs 2024/09/27 19:13:27 +0200 INF Event Detected camera=garaje event_id=1727457198.657951-7sy7rs label=person zones= 2024/09/27 19:13:27 +0200 INF Event dropped - Already notified on this zone camera=garaje event_id=1727457198.657951-7sy7rs label=person zones= 2024/09/27 19:13:53 +0200 INF New event received event_id=1727457232.161868-38jqm8 2024/09/27 19:13:53 +0200 INF Event Detected camera=garaje event_id=1727457232.161868-38jqm8 label=person zones= 2024/09/27 19:13:53 +0200 INF Event dropped - No snapshot or clip available event_id=1727457232.161868-38jqm8 2024/09/27 19:13:54 +0200 INF Event update received event_id=1727457232.161868-38jqm8 2024/09/27 19:13:54 +0200 INF Event Detected camera=garaje event_id=1727457232.161868-38jqm8 label=person zones= 2024/09/27 19:13:54 +0200 INF Alert sent event_id=1727457232.161868-38jqm8 provider=Ntfy 2024/09/27 19:14:29 +0200 INF New event received event_id=1727457268.826359-3fubdi 2024/09/27 19:14:29 +0200 INF Event Detected camera=garaje event_id=1727457268.826359-3fubdi label=person zones= 2024/09/27 19:14:29 +0200 INF Event dropped - No snapshot or clip available event_id=1727457268.826359-3fubdi 2024/09/27 19:14:29 +0200 INF Event update received event_id=1727457268.826359-3fubdi 2024/09/27 19:14:29 +0200 INF Event Detected camera=garaje event_id=1727457268.826359-3fubdi label=person zones= 2024/09/27 19:14:29 +0200 INF Alert sent event_id=1727457268.826359-3fubdi provider=Ntfy 2024/09/27 19:14:40 +0200 INF New event received event_id=1727457279.986025-tndzis 2024/09/27 19:14:40 +0200 INF Event Detected camera=garaje event_id=1727457279.986025-tndzis label=person zones= 2024/09/27 19:14:40 +0200 INF Event dropped - No snapshot or clip available event_id=1727457279.986025-tndzis 2024/09/27 19:14:41 +0200 INF Event update received event_id=1727457279.986025-tndzis 2024/09/27 19:14:41 +0200 INF Event Detected camera=garaje event_id=1727457279.986025-tndzis label=person zones= 2024/09/27 19:14:41 +0200 INF Alert sent event_id=1727457279.986025-tndzis provider=Ntfy

just to confirm, I had this bug too and the dev fixed it, good job!

Thank you! I really appreciate the feedback 😊

Closing, as this is now released in v0.3.5