Issue on disabling dates, overlapping months, wrong date being selected

Using laravel 5.8 and having a min and max in pickadate options, it's selecting the wrong date.
I got this in javascript:
$('#date').pickadate({
    disable: [2],
    min: [2019,6,29],
    max: [2019,7,5]
});

It disabling every tuesday and you can max select a week from now. The dates are generated dynamicly (obviously). So if you would visit my page tomorrow it would say
$('#date').pickadate({
    disable: [2],
    min: [2019,6,30],
    max: [2019,7,6]
});

Going back to the first example, it results to this:

First thing first: the month selected is july wich is wrong (juli = dutch for july). Tomorrow, 29th is still june. Which then results in wrong date being selected, the date that get posted through is 2019/07/29, which should be 2019/06/29
<input type="hidden" name="date_submit" value="2019/07/29">

Anyone encountered this before? I'm using version 3.6.3 from npm. Is this an issue due the localization or the core?

Hi @MichaelBelgium,
your issue is about multiple single issues.
The month is 0-indexed so 6 is July.

See the following note in the docs:

NOTE: months in a JavaScript Date object are zero-indexed. Meaning, new Date(2016, 3, 20) is 20 April, 2016.

To stay consistent with this, whenever an integer is used in reference to a month, pickadate treats it as zero-indexed. Dates as strings are still parsed as expected.
https://amsul.ca/pickadate.js/date/
Can you also provide a small codepen which reproduces your issues?

Ooh okay, that solves it.

your issue is about multiple single issues.

My intention was to only post about the wrong date being selected. But it all seemed so confusing to me at first
Here's a pen

So is your initial issue and question solved with this?
If this is the case, please close the issue


Docker "tip"

This is a mix of different pull requests (opened on https://github.com/zylon-ai/private-gpt/pulls and on https://github.com/qdm12/private-gpt/pulls) regarding Docker, and is the tip of the changes.
You can review each commit from this pull request to see what was changed.
Directions to build and use the Docker image from this branch:

Install git and docker (or podman etc.)

Build the image from this branch:
docker build -t privategpt --build-arg POETRY_EXTRAS="ui vector-stores-qdrant embeddings-huggingface" https://github.com/qdm12/private-gpt.git#docker-tip

You can add and remove extras to POETRY_EXTRAS, for example for Ollama, add  llms-ollama embeddings-ollama.

You can run it with for example the updated docker-compose.yml. Obviously you might need to adapt environment variables to match what you want to do/configure. I personally use vllm for the llm backend (PGPT_MODE=openailike) and PGPT_EMBEDDING_MODE=huggingface.

If you encounter any issue, please open an issue on this repo and I'll address it, and PR it to the base repository. Doing all of this since the zylon-ai repository maintainers look rather slow at reviewing/merging PRs. I'll also do my best to keep this fork updated with the base repository.

Hi! Running this, I get the error:
PermissionError: [Errno 13] Permission denied: '/home/worker/app/models/cache/models--mistralai--Mistral-7B-Instruct-v0.2'
Do you know why please ?

The error is still happening, but I might have find something else in the logs! Before this error, the Ollama docker yield : ollama-1  | Error: mkdir /root/.ollama/models/manifests: permission denied
It probably has something to do with this, no ?

The error is still happening, but I might have find something else in the logs! Before this error, the Ollama docker yield

I actually don't use the ollama container (I use vllm), so I don't know. That error is irrelevant to step 3 although similar. As a quick fix, you can try chmod -R 777 models to allow all users access to the models directory, that should fix the permission issue.

Unfortunately, I tried giving all permissions to the folders inside the main folder, but the error is still there.
I don't know if this: PermissionError: [Errno 13] Permission denied: '/home/worker/app/models/cache/models--mistralai--Mistral-7B-Instruct-v0.2' means that it doesn't have permissions to access the folder or the file and the docker exit on the error so I can change the permissions.

Also I fixed step 3, the directory names needed for privategpt are cache and local_data, not data +models:
mkdir -p local_data cache/models cache/tiktoken
chown -R 100:65534 local_data
chown -R 100:65534 cache

My bad!

@qdm12,
To make your steps work out of the box, can i suggest two changes,

change your default command to
docker build -t privategpt --build-arg POETRY_EXTRAS="ui vector-stores-qdrant embeddings-huggingface llms-ollama embeddings-ollama" https://github.com/qdm12/private-gpt.git#docker-tip

use $pwd for the local_data and cache folder?

@Sprooty 1. done, but 2. is not needed since docker-compose understands ./models as $(pwd)/models

Thanks for this! I had to update this in the docker-compose: PGPT_OLLAMA_API_BASE: http://host.docker.internal:11434
But now, with the UI mode and the LLM Chat, I get this error after submitting a sample:
private-gpt-private-gpt-1  | 00:03:11.660 [WARNING ] llama_index.core.chat_engine.types - Encountered exception writing response to history: [Errno -2] Name or service not known
Any ideas?

[Errno -2] Name or service not known

This is because host.docker.internal does not resolve from within the privategpt container. This is a bit out of scope, can you create an issue on the repository to discuss this further? I'll mark these 2 comments as off topic.

Branch rebased on top of https://github.com/zylon-ai/private-gpt/commit/9d0d614706581a8bfa57db45f62f84ab23d26f15


Heroku ?

hi i just used your lunik torrent and it was amazing,
i want to know that is this app is compatible with heroku or not.

Hi,
First this is not an application, it's a library.
I wrote an exemple on how to use it here
Download myapp.js, make your modification and run node myapp.js.
Be careful, torrents files are first downloaded on your computer and then uploaded to Mega.


fix: integration tests for fileio runtimes.

Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[X] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Related to TCOMP-390, I added a module to components-fileio that should exactly duplicate the behaviour we expect from the REST API when accessing Dataset runtime methods.
This revealed some missing dependencies that didn't show up in unit tests in the simplefileio-runtime module, which were fixed.
Does this PR introduce a breaking change?

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Also about the provided dependencies I think you can leave them as provided but then you need to add them in the test modules.

Pull request test result: failure :x:

Pull request test result: success :star:

Pull request test result: success :star:


Binary pixel stream is detected as text

This is originally from https://github.com/nexB/scancode-toolkit/issues/50
In this archive https://rubygems.org/downloads/chunky_png-1.2.8.gem the file:
chunky_png-1.2.8.gem-extract/data.tar.gz-extract/spec/resources/pixelstream.rgb
is detected as text even though this is clearly a binary data stream.
libmagic file detects it as octet/stream or data which is correct (for instance with this libmagic ctypes binding: https://github.com/nexB/scancode-toolkit/blob/master/src/typecode/magic2.py )
The issue comes from the fact that bytes above 127 are treated as text, and the test file first 1024 bytes are FF. The original Perl binary detection looked for bytes below 127 only and did some extra stuffs.

@audreyr There are a few possible ways to fix this:

change the heuristics to treat bytes below and above 127 differently. Use some new heuristic based on the proportion of high bytes to declare a file as binary.

keep the heuristics as-is and introduce chardet as a dependency. If a file is detected as binary in certain cases (excluding the null byte detection), apply a second level detection using chardet to see if an encoding can be guessed with a high enough confidence (say above 0.8). If so try to convert the buffer to unicode using the detected encoding. If this fails the file is binary, else the file is text.

mix approach 1 and 2, using 2. only when high bytes are found.

Note also this for testing for UTF8 https://github.com/etrepum/pyutf8

And fwiw, if we add chardet to things still detected as binary this way, we would get this:
./binaryornot/tests/isBinaryFile/encodings/bom_utf-16.txt: UTF-16BE with confidence 1.0
./binaryornot/tests/isBinaryFile/encodings/bom_utf-32le.txt: UTF-32LE with confidence 1.0
./binaryornot/tests/isBinaryFile/encodings/test-gb2.txt: GB2312 with confidence 0.99
./binaryornot/tests/isBinaryFile/encodings/test-kr.txt: EUC-KR with confidence 0.99
./binaryornot/tests/isBinaryFile/encodings/test-latin.txt: MacCyrillic with confidence 0.837785152976
./binaryornot/tests/isBinaryFile/encodings/big5.txt: Big5 with confidence 0.99
./binaryornot/tests/isBinaryFile/encodings/test-utf16be.txt: UTF-16BE with confidence 1.0
./binaryornot/tests/isBinaryFile/encodings/test-gb.txt: GB2312 with confidence 0.99
./binaryornot/tests/isBinaryFile/encodings/bom_utf-32.txt: UTF-32BE with confidence 1.0
./binaryornot/tests/isBinaryFile/encodings/bom_utf-8.txt: UTF-8 with confidence 1.0
./binaryornot/tests/isBinaryFile/encodings/big5_B.txt: Big5 with confidence 0.99
./binaryornot/tests/isBinaryFile/encodings/test-shishi.txt: utf-8 with confidence 0.99
./binaryornot/tests/isBinaryFile/encodings/utf_8.txt: utf-8 with confidence 0.7525
./binaryornot/tests/isBinaryFile/encodings/bom_utf-16le.txt: UTF-16LE with confidence 1.0
./binaryornot/tests/isBinaryFile/encodings/utf8cn.txt: utf-8 with confidence 0.99

Then we could try a unicode conversion on these to confirm the chardet encoding and decide text or binary for lesser travelled texts encodings.

This second commit builds on top of the previous to support add chardet encoding detection and now reports as text all UTF8/16/32 variants. Only pdf fails to be detected as binary (this is not a regression, your code failed at it before too)

@audreyr Please check these two commits and tell me what you like best... I will update and create a PR then

Hey @pombredanne, I was looking at your work and while I prefer the heuristics to keep dependencies down, I also see the virtue of using chardet. @audreyr will make the final call, of course. :smile:
Where I am concerned is Windows compatibility. Binaryornot is a dependency of Cookiecutter, which currently works on all current versions of Python (including PyPy) and works identically across Mac OS X, Linux, and Windows. We don't want to lose that capability, especially for Windows users (who tend to be newer to coding).
Therefore, what I'm going to do is add Windows continuous integration to this repo. I should have that done today, whereupon we can test solutions against a wider range of systems.

@pombredanne I like your mixed approach, amazing work here. I have no problem adding chardet as a dependency.
@pydanny Thanks for offering to work on adding Windows continuous integration. I don't think it would be a problem to keep Windows compatibility.

@pydanny FWIW, scancode is supported and tested on Linux, Windows and Mac...
I run tests on Travis for Linux 64: https://travis-ci.org/nexB/scancode-toolkit
For windows on appveyor:  https://ci.appveyor.com/project/nexB/scancode-toolkit
So whatever suggestions @audreyr accepts, it will have to be working there for me. Both binaryornot and chardet are already in use there and heavily tested on these 3 oses.

@audreyr wrote:

I like your mixed approach, amazing work here. I have no problem adding chardet as a dependency.
Let me transform this in a bona fide pull request.

@audreyr I assume that you have no issue with the inclusions of the excellent tests files of https://github.com/gjtorikian/isBinaryFile from @gjtorikian ? I included the license notice.

No issue, those are very nice tests. The more test files, the merrier. And isBinaryFile looks great, something I'll be studying for ideas :smile:

and @audreyr Thanks for the kudos. This means a lot as I hold your's  @pydanny 's code such as cookiecutter and django-packages/opencomparison in high respect!

@audreyr Do you really want to support Python 2.6?

I could easily be convinced to drop 2.6 if it's too much of a hassle.

Fixed by @pombredanne in #11.


Rkhan/navbar edit

Updated the menu and directed to the pages we have . There is navigation missing for 'how it works'.
Updated the hamburger menu as well

Great work

@leeroyp  Use the approve option instead of comment when reviewing changes. Comment would be for just commenting on something


Fix source distribution building actions

The actions don't all do what they say, e.g. build_sdist_wheels doesn't build wheels.

Issue is wrong, the actions seem to be fine.


Allow contentctl test to work on Windows by fixing a path problem.

When using contentctl test on Windows, a wrong path is generated. We need to force this path to be a posix path else we have an error that looks like this:
400 Client Error for http+docker://localnpipe/v1.46/containers/create?name=contentctl_0&platform=linux%2Famd64: Bad Request ("invalid mount config for type "bind": invalid mount path: 'C:/tmp/apps' mount path must be absolute"

This should not have an impact on Linux systems, but I have not tested myself.

Thank you for the PR @Res260 ! We do not have windows machines, so were not aware this was an issue!


manifest: save private key with 0600 permission

What problem does this PR solve?
Save keyinfo files with 0600 if it contains private key
What is changed and how it works?
Check if the KeyInfo contains private key block before saving it to disk
Check List
Tests

Manual test (add detailed scripts or steps below)

Code changes

Has persistent data change

Side effects

N/A

Related changes

N/A

Codecov Report

Merging #256 into master will decrease coverage by 16.91%.
The diff coverage is 31.47%.

@@             Coverage Diff             @@
##           master     #256       +/-   ##
===========================================
- Coverage   61.22%   44.30%   -16.92%
===========================================
  Files          40       37        -3
  Lines        2298     2961      +663
===========================================
- Hits         1407     1312       -95
- Misses        674     1414      +740
- Partials      217      235       +18

Flag
Coverage Δ

#coverage
?

Impacted Files
Coverage Δ

main.go
100.00% <ø> (ø)

pkg/repository/crypto/keys.go
0.00% <0.00%> (ø)

pkg/repository/v0manifest/component.go
26.66% <0.00%> (ø)

pkg/repository/v1manifest/local_manifests.go
0.00% <0.00%> (ø)

pkg/utils/ioutil.go
36.50% <0.00%> (-7.50%)
:arrow_down:

pkg/utils/sha.go
25.00% <0.00%> (ø)

pkg/version/version.go
80.00% <ø> (-10.00%)
:arrow_down:

pkg/repository/v1manifest/repo.go
6.23% <6.23%> (ø)

pkg/repository/v1manifest/types.go
13.04% <13.04%> (ø)

pkg/repository/mirror.go
20.33% <13.46%> (+4.67%)
:arrow_up:

... and 41 more

Continue to review full report at Codecov.

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


Aerogear 3466 delete datasource

JIRA: https://issues.jboss.org/browse/AEROGEAR-3466
Deletes the chosen data source when this option is selected from the dropdown
Provides a confirm window before deleting

ping @pb82 @josemigallas
updated pr with restructure


Add templates for issues

Issue templates are very helpful for a collaboration repo. When users identify a bug or want to add a new feature, you can provide templates so you can collect all the pertinent information you need to fix a bug or add a new feature.
We recommend creating a “Report Bug” and “Feature Request” issue template.
Some suggested prompts/questions you can add to a “Report Bug” template are:
Briefly describe the bug
What is the expected behavior?
Please provide step by step instructions on how to reproduce the bug
Some suggested prompts/questions you can add to a “Feature Request” issue template are:
Briefly describe your feature request
What problem is this feature trying to solve?
How do we know when the feature is complete?
Reference: https://github.com/Recode-Hive/Stackoverflow-Analysis/issues/new/choose
Click on Get Started button and copy the template to yours.

Okay I have added both report bug and feature request template.

Thanks, I will close this Issue. @Suchitra-Sahoo


Unable to read repo after key was added

I'm trying to set up access to a private repository set up as a dependency for a Flutter application.
I followed this: https://github.com/marketplace/actions/webfactory-ssh-agent#usage
and this: https://github.com/marketplace/actions/webfactory-ssh-agent#support-for-github-deploy-keys
The exact command I used to create the key was:
ssh-keygen -t ed25519 -a 100 -f id_ed25519_dk -C git@github.com:convo/HighlanderCPaaSAdapterInterface.git
Yet my current output is:
Run<EMAIL_ADDRESS>Adding GitHub.com keys to /home/runner/.ssh/known_hosts
Starting ssh-agent
SSH_AUTH_SOCK=/tmp/ssh-7bZnk0tNrFSD/agent.1834
SSH_AGENT_PID=1835
Adding private key(s) to agent
Identity added: (stdin) (git@github.com:convo/HighlanderCPaaSAdapterInterface.git)
Key(s) added:
256 SHA256:J3hmvJ33ojrGpMTZ6XkDo8o76HLDsoyBaowUSqX328c git@github.com:convo/HighlanderCPaaSAdapterInterface.git (ED25519)
Configuring deployment key(s)
Added deploy-key mapping: Use identity '/home/runner/.ssh/key-f1046220a395f9fc1c7756e3e0919722056ccaa2a46bac51ea8178d8ea5f8853' for GitHub repository convo/HighlanderCPaaSAdapterInterface
Comment for (public) key '' does not match GitHub URL pattern. Not treating it as a GitHub deploy key.

Followed by:
Running "flutter pub get" in flutter_tools...                       5.4s
Running "flutter pub get" in HighlanderSDK...                       4.9s
Running "flutter pub get" in example...
Git error. Command: `git clone --mirror git@github.com:convo/HighlanderCerberusAdapter.git /opt/hostedtoolcache/flutter/2.8.1-stable/x64/.pub-cache/git/cache/HighlanderCerberusAdapter-5f89de7adaf6d0c72ce34930f90772f706482734`
stdout:
stderr: Cloning into bare repository '/opt/hostedtoolcache/flutter/2.8.1-stable/x64/.pub-cache/git/cache/HighlanderCerberusAdapter-5f89de7adaf6d0c72ce34930f90772f706482734'...
ERROR: Repository not found.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
exit code: 128

I've tried creating keys the same way on my local machine to test them and they seem to work, at least using them to connect to my github account or pulling code locally.
Any thoughts on this?
Is the Comment for (public) key '' does not match GitHub URL pattern. Not treating it as a GitHub deploy key. message where the issue is?
Any help would be appreciated.
:)

Is the Comment for (public) key '' does not match GitHub URL pattern. Not treating it as a GitHub deploy key. message where the issue is?

I think that's a glitch in logging and not relevant here.
I noticed you added the key comment for convo/HighlanderCPaaSAdapterInterface but the error occurs when trying to clone convo/HighlanderCerberusAdapter.
Is that correct?

Hi @mpdude
Is there a way for this GH Action to work without using a deploy key?
I know there is a section on machine users in the README but the information on it is sparse

Hey @mpdude Thanks for your reply.
I re-did everything from zero and found a cross dependency hence why my key wasn't working. Fixed it and it worked.
Feel free to close this :D


Modify render[_anti]_criteria logic

Modify the implementation of LayoutBlock::should_draw so that it may
take into account multiple RenderCriteria and not only take into account
the last instance of RenderCriteria from each vector.
Additionally, apply the following logic in regards to render_criteria
and render_anti_criteria:
In order for a LayoutBlock to render, ALL of its render_criteria
should be satisfied, when NONE of its render_anti_criteria should.

Relevant discussion: #66.

Modify the implementation of LayoutBlock::should_draw so that it may
take into account multiple RenderCriteria and not only take into account
the last instance of RenderCriteria from each vector.

I'm fairly confident that we do consider all criteria in the vec.  Would you be able to provide a config that produces the bug?

Additionally, apply the following logic in regards to render_criteria
and render_anti_criteria:

In order for a LayoutBlock to render, ALL of its render_criteria
should be satisfied, when NONE of its render_anti_criteria should.
This too is the current behavior as far as I understand it.  Would you be able to provide a config that shows the difference between your implementation and the current one?

Hello,

Relevant discussion: #66.

Modify the implementation of LayoutBlock::should_draw so that it may
take into account multiple RenderCriteria and not only take into account
the last instance of RenderCriteria from each vector.

I'm fairly confident that we do consider all criteria in the vec. Would you be able to provide a config that produces the bug?

I have to admit you're absolutely right. I misread the code in this for loop, thinking that every iteration modifies the should_draw in a way that it may override the result of a previous iteration but the truth is, while it does, it does it in a manner that will not cause any problems. So I'm afraid I cannot provide you with a configuration file that produces a bug.

Additionally, apply the following logic in regards to render_criteria
and render_anti_criteria:
In order for a LayoutBlock to render, ALL of its render_criteria
should be satisfied, when NONE of its render_anti_criteria should.

This too is the current behavior as far as I understand it. Would you be able to provide a config that shows the difference between your implementation and the current one?

No worries, that code is ridiculously confusing for how simple it should be and I've been tripped up on it multiple times in the past week.


Godot not closing when quiting.

Godot version
4.2-beta6-mono
System information
Godot v4.2.beta6.mono - Manjaro Linux #1 SMP PREEMPT_DYNAMIC Thu Nov  9 02:34:53 UTC 2023 - X11 - Vulkan (Forward+) - dedicated AMD Radeon RX 7900 XTX (RADV GFX1100) () - AMD Ryzen 7 7800X3D 8-Core Processor (16 Threads)
Issue description
Hello,
When trying to quit godot, the godot editor goes grey and I got this error in the terminal:
EditorSettings: Save OK!
Unloading: Disposing tracked instances...
ERROR: Condition "!EditorSettings::get_singleton() || !EditorSettings::get_singleton()->has_setting(p_setting)" is true. Returning: Variant()
   at: _EDITOR_GET (editor/editor_settings.cpp:1140)

It's only happening in the beta6. No problem in the beta5. But I still have the same error showing. So I'm not sur if the error is relevant.
EditorSettings: Save OK!
Unloading: Disposing tracked instances...
ERROR: Condition "!EditorSettings::get_singleton() || !EditorSettings::get_singleton()->has_setting(p_setting)" is true. Returning: Variant()
   at: _EDITOR_GET (editor/editor_settings.cpp:1140)
Unloading: Finished disposing tracked instances.
XR: Clearing primary interface
XR: Removed interfaceNative mobile
XR: Removed interfaceOpenXR
PulseAudio: context terminated
Updated PSO cache (0.7 MiB)

Thanks
Steps to reproduce

Open Godot 4.2 beta6
Create a new project
Select Scene -> Quit or Ctrl + Q
Godot will stay open

Minimal reproduction project
Just create a new project from Godot 4.2 beta6

Thanks for the report.

Duplicate of https://github.com/godotengine/godot/issues/84728


Issues with teleporting

One of the issues is that if you first follow this path : 2.2->2.3->2.2->3.2 you seem to be spawned in the default spot instead of the entrance. If you just go from 2.2 to 3.2 everything seems to work fine


Pose landmarks type from mediapipe python package

Hello,
I'm using the the pose estimation in mediapipe python package and I would like to get other outputs as the confidence score and  the types of landmarks instead of the ids (like nose, left shoulder, right shoulder, ...). But I can't figure it out where these output are defined and how to change the code to get them.
Thanks a lot,
Hamedeh

After getting the pose landmark list, you can manually convert those 25 landmarks to joint points by index for now. See https://github.com/google/mediapipe/blob/c0124fb83c667917c7bf6a84f442a0136e8f2cf7/docs/images/mobile/pose_tracking_upper_body_landmarks.png for details.
The pose graph doesn't output the confidence score. To achieve this, the graph config needs to be modified.
https://github.com/google/mediapipe/blob/c0124fb83c667917c7bf6a84f442a0136e8f2cf7/mediapipe/graphs/pose_tracking/upper_body_pose_tracking_cpu.pbtxt#L6

Thanks a lot!
However the left and right landmarks that I get are swapped comparing to your image.
Also could you please help me find what corresponds exactly to the "InFrameLikelihood"  to add in the graph? I can't find it in the package. I checked also this file "pose_landmark_upper_body_cpu.pbtxt" but nothing there.
Thanks for your help.

Whoops. We need to flip the input image. The graph is for front camera only, and without it, the image looks reversed (opposite of looking into a mirror).
To fix this, go to your site-packages/mediapipe/examples/python/upper_body_pose_tracker.py and do the following changes:
--- a/mediapipe/examples/python/upper_body_pose_tracker.py
+++ b/mediapipe/examples/python/upper_body_pose_tracker.py
@@ -123,7 +123,7 @@ class UpperBodyPoseTracker:
       else:
         input_frame = cv2.imread(input_file)[:, :, ::-1]

-    pose_landmarks, annotated_image = self._run_graph(input_frame)
+    pose_landmarks, annotated_image = self._run_graph(cv2.flip(input_frame, 1))
     if output_file:
       cv2.imwrite(output_file, annotated_image[:, :, ::-1])
     return pose_landmarks, annotated_image
@@ -150,7 +150,7 @@ class UpperBodyPoseTracker:
       success, input_frame = cap.read()
       if not success:
         break
-      input_frame = cv2.cvtColor(input_frame, cv2.COLOR_BGR2RGB)
+      input_frame = cv2.cvtColor(cv2.flip(input_frame, 1), cv2.COLOR_BGR2RGB)
       input_frame.flags.writeable = False
       _, output_frame = self._run_graph(input_frame)
       cv2.imshow('MediaPipe upper body pose tracker',

Thanks a lot!
However the left and right landmarks that I get are swapped comparing to your image.
Also could you please help me find what corresponds exactly to the "InFrameLikelihood" to add in the graph? I can't find it in the package. I checked also this file "pose_landmark_upper_body_cpu.pbtxt" but nothing there.

Note that the landmarks are marked from the perspective of the person in the image rather than the viewer of the image. I think that's why you think the left and right landmarks are swapped.


Remove XX region for subregional nodes

In the data/csv/nodes.csv file, for regions that have subregional representation, there are still XX nodes. For example, Canada is displayed as below:

Node
Country
Population
Centroid_Lat
Centroid_Lon
Pop_Lat
Pop_Lon
Node_Verbose

CAN-XX
Canada
5429524
43.74
-79.37
43.74
-79.37

CAN-AB
Canada
1239220
51.05
-114.07
51.05
-114.07
Canada - Alberta

CAN-AR
Canada
403131
44.65
-63.59
44.65
-63.59
Canada - Atlantic

...

Other examples I believe are Canada, USA, Brazil, China, Australia, and Russia.
I think the XX nodes should be removed for all of these regions?

I think so to and the same for the aggregated regions that are used in the modelling for OG/FEO but are no part of the transmission database (e.g. Canada Central). I'll try to align the input files.

I've updated the node list accordingly in the latest commit. @amanmajid somewhere there's a '-' being added in the region names (e.g. 'USA-ER' and 'IRL-XX') which can lead to conflicts based on the naming convention in the transmission database and as used in OG ('USAER'). Is it possible to remove this? Will also need to be done in the nodes.csv if it can be removed. Also, is the 'node_code_name_mapping' csv being used somewhere as this still includes regions as used in FEO (e.g. Canada Central). If it's not used I suggest to remove it to prevent confusion.

@maartenbrinkerink @amanmajid I agree on merging the files! The ones I am seeing are:

data/csv/nodes.csv
data/csv/node_code_name_mapping.csv
database_mappings.csv (attached here from our slack messages)

The database_mappings.csv holds regions to be merged into one. This would make it easier to coordinate the spatial representation map. Not sure if the subregion names match up perfectly with the other files or not
database_mappings.xlsx

The nodes.csv has the exact same regions as the 'database_mappings' with the only difference that 'nodes.csv' has '-' as delimiter in the region names (which I suggest to change but I'm not sure where the '-' comes from in the script since it's not being read from the 'nodes.csv'). 'node_code_name_mapping' is redundant I think. Merging everything into one spatial mapping file that can be used for all maps makes sense to me (including the 'exclude' map and the 'spatial representation' map).

@maartenbrinkerink The '-' is being added here. I can remove this to keep the three-letter ISO codes only.
@trevorb1 Just to check, are you saying above that I can delete the three .csv files and replace them with the excel file you linked above? If so, that sounds like a great idea and I'm happy to refactor the code to do that.

@amanmajid @maartenbrinkerink My proposal for a single csv nodes file can be found here. Please note, it does not address the - issue. This file has been integrated into PR #6 already. Please feel free to change accordingly tho!

@amanmajid I have added shapefiles for each subregion in PR #6. Each country that has subregions has its own shapefile, as I thought this was more explicit. However, if you prefer all subregions in a single shapefile, that is an easy update. These shapfiles have a region, subregion, and geometry column. region is the 3-letter iso while subregion is 2-letter code.
After chatting this morning, it seems like you have a good idea on how to create the spatial representation map described in issue #4? If this is correct, maybe I can leave it to you to integrate everything together? If not, please just let me know and I can help! :)

@maartenbrinkerink when recreating the admin level 1 boundaries to regional mappings, a few questions came up. If you have thoughts on any of them, can you please let me know! All questions come from the comments in the mappers shown here.

For Australia there are three unaccounted for regions; AshmoreandCartierIslands, CoralSeaIslandsTerritory, and JervisBayTerritory. However, these seem to be very small geometry. I am assuming it is okay to drop them.

For India, there is a region called Ladakh that does not exist in the GADM data. However, there is an unassigned GADM region of Lakshadweep. Are these the same?

The Japan mapping seems incorrect? GADM lists about 40 admin level 1 regions, however, the current database mapping file only lists 5 regions? Right now, I have not mapped over Japan because of this.

In Russia, the database mapping file lists Chita, Chukchi Autonomous Okrug, Maga Buryatdan, and Moskovskaya, which do not appear in the GADM data. However, the GADM dataset list Chukot, Magadan, MoscowCity and Zabaykal'ye which do not appear in the database mappings file. Im pretty sure MoscowCity goes with Moskovskaya, and Maga Buryatdan goes with Magadan, but not sure about the others. I am happy to just match the GADM data to whatever the closest region is, but just thought I would check first if you knew.

Happy to hop on a call to discuss any of this it thats easier as well! :)

@maartenbrinkerink when recreating the admin level 1 boundaries to regional mappings, a few questions came up. If you have thoughts on any of them, can you please let me know! All questions come from the comments in the mappers shown here.

For Australia there are three unaccounted for regions; AshmoreandCartierIslands, CoralSeaIslandsTerritory, and JervisBayTerritory. However, these seem to be very small geometry. I am assuming it is okay to drop them.

For India, there is a region called Ladakh that does not exist in the GADM data. However, there is an unassigned GADM region of Lakshadweep. Are these the same?
The Japan mapping seems incorrect? GADM lists about 40 admin level 1 regions, however, the current database mapping file only lists 5 regions? Right now, I have not mapped over Japan because of this.
In Russia, the database mapping file lists Chita, Chukchi Autonomous Okrug, Maga Buryatdan, and Moskovskaya, which do not appear in the GADM data. However, the GADM dataset list Chukot, Magadan, MoscowCity and Zabaykal'ye which do not appear in the database mappings file. Im pretty sure MoscowCity goes with Moskovskaya, and Maga Buryatdan goes with Magadan, but not sure about the others. I am happy to just match the GADM data to whatever the closest region is, but just thought I would check first if you knew.

Happy to hop on a call to discuss any of this it thats easier as well! :)

Jervis Bay is a peninsula of AUSSW so can be added (but dropped is also fine if easier) and the other two are non-covered islands so you can drop them.
Lakshadweep are islands and can be removed. Ladakh is a contested area part of the larger Kashmir area which I think is part of the dataset ('Jammu and Kashmir' in the original dataset)? If Kashmir is in GADM then Ladakh can be dropped.
You're right, they were missing in the original mapping file since the Cartopy package I used (for which I created the mapping initially) did not support admin 1 for Japan. I've added the mapping below so you can integrate it where needed.
Chukchi and Chukot correspond to the same node (RUSFE), Maga Buryatdan/Magadan are the same and Moskovskaya/MoscowCity as well. Chita is a former oblast that merged with some other smaller regions into 'Zabaykal'ye` so you can swap Chita with that one.

Aichi | JPNCE
Akita | JPNTO
Aomori | JPNTO
Chiba | JPNTO
Ehime | JPNSH
Fukui | JPNCE
Fukuoka | JPNKY
Fukushima | JPNTO
Gifu | JPNCE
Gunma | JPNTO
Hiroshima | JPNCE
Hokkaido | JPNHO
Hyogo | JPNCE
Ibaraki | JPNTO
Ishikawa | JPNCE
Iwate | JPNTO
Kagawa | JPNSH
Kagoshima | JPNKY
Kanagawa | JPNTO
Kochi | JPNSH
Kumamoto | JPNKY
Kyoto | JPNCE
Mie | JPNCE
Miyagi | JPNTO
Miyazaki | JPNKY
Nagano | JPNCE
Nagasaki | JPNKY
Nara | JPNCE
Niigata | JPNTO
Oita | JPNKY
Okayama | JPNCE
Okinawa | JPNOK
Osaka | JPNCE
Saga | JPNKY
Saitama | JPNTO
Shiga | JPNCE
Shimane | JPNCE
Shizuoka | JPNCE
Tochigi | JPNTO
Tokushima | JPNSH
Tokyo | JPNTO
Tottori | JPNCE
Toyama | JPNCE
Wakayama | JPNCE
Yamagata | JPNTO
Yamaguchi | JPNCE
Yamanashi | JPNCE

Great, thanks @maartenbrinkerink!

I included JervisBayTerritory and dropped AshmoreandCartierIslands and CoralSeaIslandsTerritory
I dropped Lakshadweep and ignored missing Ladakh, as JammuandKashmir is already included
The Japan mappings matched perfectly, thanks!
Updated MoscowCity and Magadan, Chukot, and Zabaykal'ye following your notes

I've slightly lost track of what the actual issue is here.
It seems @trevorb1 has fixed the nodes with subregional representation above (thanks!).
Also, I've now changed the code so that -XX does not appear in any of the interactive plots. However, there are still nodes with -XX in their name in nodes.csv, which I don't think is much of an issue. In fact, we might need to keep it there as that's the notation used in OSeMOSYS-Global, so should probably keep that notation to keep mapping consistent.
Given the above, can we go ahead and close this issue?

@amanmajid ahhh. My fault on changing topics within the same issue ticket. The original issue has been resolved, and spatial map conversation can take place in ticket #4.


Enable timeout ensureSynchronized everywhere

Provide the mocha test timeout LoaderContainerTracker to limit how long to wait for ensureSynchronized.  Currently it is set to half of test time.  The purpose is to provide a better error message and stack if ensureSynchronized got stuck.
Also, change the behavior for timeoutPromise, when duration is <=0 or Infinity.  Instead of waiting for the default amount, it will now mean it won't timeout at all.   This is so that we can avoid timeout when debugging.
Fix AB#2175

Timeouts usually do not work :) Is the reason to add them - aid in debuggability where it's hard to figure out for developers why tests hang? Anything we can do to improve it without using timeouts? Like maybe mocha has some facility to run code when test fails and for code like that to provide extra info for easier debugging?

@vladsud yes, this is here to aid in debuggability so that they know where it hangs.   mocha test has timeout, but when it does, it only kill the test.
It doesn't give you any idea where it hangs, especially around promises.
@anthony-murphy  added  the timeout concept for promises used in test.   I don't know if he has other alternatives.

@vladsud @anthony-murphy Please also look at #12303, which instead of estimating using a half the timeout. #12303 will trap test run and calculate how long until the test would timeout and set the duration for promise wait just before that.

Closing in favor of  #12303


Added missing '('

@akasnik : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Create mdm-remotewipe-dowipepersistprovisioneddatamethod.md

@aaronlower : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevewhims : Thanks for your contribution! The author(s) have been notified to review your proposed change.


SAMZA-1289: Default id generator if not configured

Right now in standalone deployment we require the user to provide an id generator. Since most of the time the users can simply use the UUID generator for id generation, we should default it using that.

@xinyuiscool In this version, we are maintaining backward compatibility with "processor.id" config. So, are you going to default to UUIDGenerator if processor.id is not configured?

right, if processor.id is configured, we will not use the generator, otherwise it's using UUID generator. There is a check for this logic in every place which generates the processorId.


adds alias handle method to commands

Coverage decreased (-0.2%) to 21.208% when pulling 8e875527e02825a7fa7ad3ade9575888c483be7d on ahmetkapikiran:patch-1 into 0a0784c6fb4d383bceebd7ee4f8a2dcac15df5cf on rcrowe:master.

Duplicate of #321 that I made last week.


Can`t reproduce groovy ExampleFileUploadSpec test on java

Hi!
I found the single test on groovy for multipart/form-data uploading https://github.com/DiUS/pact-jvm/blob/master/pact-jvm-consumer-junit/src/test/groovy/au/com/dius/pact/consumer/junit/ExampleFileUploadSpec.groovy
I need to write it on java. I just rewrote groovy code to java:
	public  static final String HOST = "<IP_ADDRESS>";
	private static final String PROVIDER = "provider";
	private static final String CONSUMER = "consumer";

	/**
	 * Pact Rule to represent provider.
	 */
	@Rule
	public PactProviderRuleMk2 mockProvider = new PactProviderRuleMk2(PROVIDER, HOST, PORT, this);

	@Pact(provider = PROVIDER, consumer= CONSUMER)
	RequestResponsePact createPact(PactDslWithProvider builder) throws IOException {
		return builder
				.uponReceiving("a multipart file POST")
				.path("/upload")
				.method("POST")
				.withFileUpload("file", "data.csv", "text/csv", "1,2,3,4\n5,6,7,8".getBytes())
				.willRespondWith()
				.status(201)
				.body(new PactDslJsonBody())
				.toPact();
	}

	@Test
	@PactVerification
	public void runUploadTest() throws IOException {
		String url = String.format("Http://%s:%d", HOST, PORT);
		CloseableHttpClient httpclient = HttpClients.createDefault();
		HttpEntity httpEntity = MultipartEntityBuilder.create()
				.setMode(HttpMultipartMode.BROWSER_COMPATIBLE)
				.addBinaryBody("file", "1,2,3,4\n5,6,7,8".getBytes(), ContentType.create("text/csv"), "data.csv")
				.build();
		HttpUriRequest httpUriRequest = RequestBuilder
				.post(url + "/upload")
				.setEntity(httpEntity)
				.build();
		CloseableHttpResponse response = httpclient.execute(httpUriRequest);
		System.out.println(response.getStatusLine());
		}`

But it fails where is execute() run. I get next:

`org.apache.http.conn.HttpHostConnectException: Connect to <IP_ADDRESS>:8088 [/<IP_ADDRESS>] failed: Connection refused: connect`

What wrong in my code?

The HTTP client library is getting a connection refused, so you need to check the port the mock server is running on. I can't see anything wrong, apart from not seeing what you're setting PORT constant to. The Groovy example does not provide a port, so a random one is used and then mockProvider.url (`mockProvider.getUrl() in Java) is used to get the port.
Rerunning it with DEBUG level logging will give you more info.

I found what was wrong. Unlike groovy in java, the method for pact creation must be public.


Feature request: LFS support

It would be great if lfs is served via gittorrent. Torrent is famous for sharing large binaries, so it seems natural to expand the territory of gittorrent.

Могу хранилище выделитъ

@nakosung Sorry; which LFS are you talking about?

@cjb I didn't know there were many lfs's out there. https://github.com/github/git-lfs


UnitTest for KafkaSource

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-715

Description

[x] Here are some details about my PR, including screenshots (if applicable):
We have an abstract class KafkaSource which contains a function called getWorkunits that be used in many use cases. But we have no unit test for this function. We should implement a simple subclass of KafkaSource and have a unit test to test the logic inside the function to make sure it returns the desired WorkUnits.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@ibuenros @htran1 @autumnust

@htran1 can you merge?

+1

Codecov Report

Merging #2582 (c94e99b) into master (f4a9966) will decrease coverage by 1.37%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2582      +/-   ##
============================================
- Coverage     47.14%   45.77%   -1.37%
+ Complexity    10897     2156    -8741
============================================
  Files          2148      413    -1735
  Lines         85045    17961   -67084
  Branches       9439     2185    -7254
============================================
- Hits          40093     8222   -31871
+ Misses        41309     8868   -32441
+ Partials       3643      871    -2772

see 1738 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix: update to eye v3.23.2

:tada: This PR is included in version 4.23.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v4.23.0

Your semantic-release bot :package::rocket:


Add "Signify Netherlands B.V." as a manufacturer for ZHA Philips lights

The 1.7x.xx firmware that is being rolled out for Bluetooth capable lights (so Philips lights with the ZHA profile instead of the ZLL profile) changed the manufacturer name from "Philips" to "Signify Netherlands B.V.".
Tested and working using a "Lightstrip Plus" "LCL001".
Signature (with quirk applied due to these changes):
{
  "node_descriptor": "NodeDescriptor(byte1=1, byte2=64, mac_capability_flags=142, manufacturer_code=4107, maximum_buffer_size=82, maximum_incoming_transfer_size=128, server_mask=11264, maximum_outgoing_transfer_size=128, descriptor_capability_field=0)",
  "endpoints": {
    "11": {
      "profile_id": 260,
      "device_type": "0x010d",
      "in_clusters": [
        "0x0000",
        "0x0003",
        "0x0004",
        "0x0005",
        "0x0006",
        "0x0008",
        "0x0300",
        "0x1000",
        "0xfc01"
      ],
      "out_clusters": [
        "0x0019"
      ]
    },
    "242": {
      "profile_id": 41440,
      "device_type": "0x0061",
      "in_clusters": [],
      "out_clusters": [
        "0x0021"
      ]
    }
  },
  "manufacturer": "Signify Netherlands B.V.",
  "model": "LCL001",
  "class": "zhaquirks.philips.zhaextendedcolorlight.ZHAExtendedColorLight2"
}

Codecov Report

Merging #728 (8262045) into dev (b12ebc3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #728   +/-   ##
=======================================
  Coverage   82.79%   82.80%
=======================================
  Files         172      172
  Lines        3872     3873    +1
=======================================
+ Hits         3206     3207    +1
  Misses        666      666

Impacted Files
Coverage Δ

zhaquirks/philips/zhaextendedcolorlight.py
100.00% <ø> (ø)

zhaquirks/philips/__init__.py
54.63% <100.00%> (+0.47%)
:arrow_up:

zhaquirks/philips/zhadimmablelight.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #728 (8262045) into dev (b12ebc3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #728   +/-   ##
=======================================
  Coverage   82.79%   82.80%
=======================================
  Files         172      172
  Lines        3872     3873    +1
=======================================
+ Hits         3206     3207    +1
  Misses        666      666

Impacted Files
Coverage Δ

zhaquirks/philips/zhaextendedcolorlight.py
100.00% <ø> (ø)

zhaquirks/philips/__init__.py
54.63% <100.00%> (+0.47%)
:arrow_up:

zhaquirks/philips/zhadimmablelight.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 493510007

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 82.804%

Totals

Change from base Build 490651328:
0.004%

Covered Lines:
3207

Relevant Lines:
3873

💛  - Coveralls

Pull Request Test Coverage Report for Build 493510007

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 82.804%

Totals

Change from base Build 490651328:
0.004%

Covered Lines:
3207

Relevant Lines:
3873

💛  - Coveralls

I'm also wondering, does ZHA re-read the manufacturer name at any time?
(Who would assume that the name changes...)
Since both Philips and Signify are now matching, it shouldn't cause any issues if it does or does not re-read it.
(I'd also keep both in, as many don't update their Philips Hue lights and some BT-capable lights have not even received the update.)

I'm also wondering, does ZHA re-read the manufacturer name at any time?
(Who would assume that the name changes...)
Since both Philips and Signify are now matching, it shouldn't cause any issues if it does or does not re-read it.
(I'd also keep both in, as many don't update their Philips Hue lights and some BT-capable lights have not even received the update.)


devices from coolkit are unavailable in new HA 2024

Hi,
after installation new HA 2024, sonoff devices are unavailable from coolkit CK-BL602-SWP1-01(182)
all other devices original sonoff are working well.
thx
config_entry-sonoff-94546c61063f4b9b0ec43955ad4474c4.json

Cloud auth doesn't work for you (don't know why). And this devices don't discovered in your LAN (don't know wny).

In ewelink app works everything, but in HA works just basics, duals.

same here, devices stopped working in latest HA version.

https://github.com/AlexxIT/SonoffLAN?tab=readme-ov-file#issues

I forgot to write that I downgrade HA to 2023.12 version as well and sonoff to 3.5.3.
Anyway I assume that it was my mistake with mixed versions of core, os, supervisor and sonoff lan, because I had problems with one esp mini and weather forecast as well
Thx
Regards O.

Things just went offline for me the moment I updated HA core to 2024.3.0.
Cloud WS Connection error: Cannot connect to host us-pconnect5.coolkit.cc:443 ssl:default [None]
Cloud WS Connection error: Cannot connect to host us-pconnect8.coolkit.cc:443 ssl:default [Connect call failed ('<IP_ADDRESS>', 443)]
Cloud WS Connection error: Cannot connect to host us-dispa.coolkit.cc:443 ssl:default [Connect call failed ('<IP_ADDRESS>', 443)]

Cloud ERROR: {'error': 504, 'reason': 'Request Timeout', 'deviceid': '100153c92e', 'apikey': '1XXX', 'sequence': '1XX'}
Cloud ERROR: {'error': 529, 'reason': 'Internal Server Error', 'deviceid': '100153c92e', 'apikey': '1XXX, 'sequence': '1XX'}

sonoff-513c5dffce155fb2397773d228ec7529-Z906-f1d4d73feadf6253d591c547e4d4cde5.json
config_entry-sonoff-513c5dffce155fb2397773d228ec7529.json

Things just went offline for me the moment I updated HA core to 2024.3.0.
Cloud WS Connection error: Cannot connect to host us-pconnect5.coolkit.cc:443 ssl:default [None]
Cloud WS Connection error: Cannot connect to host us-pconnect8.coolkit.cc:443 ssl:default [Connect call failed ('<IP_ADDRESS>', 443)]
Cloud WS Connection error: Cannot connect to host us-dispa.coolkit.cc:443 ssl:default [Connect call failed ('<IP_ADDRESS>', 443)]

Cloud ERROR: {'error': 504, 'reason': 'Request Timeout', 'deviceid': '100153c92e', 'apikey': '1XXX', 'sequence': '1XX'}
Cloud ERROR: {'error': 529, 'reason': 'Internal Server Error', 'deviceid': '100153c92e', 'apikey': '1XXX, 'sequence': '1XX'}

sonoff-513c5dffce155fb2397773d228ec7529-Z906-f1d4d73feadf6253d591c547e4d4cde5.json
config_entry-sonoff-513c5dffce155fb2397773d228ec7529.json

Exactly the same for me. Nothing works since 2024.3. I need to update to 2024.4 since support for my Alarm panel has been added in 2024.1

Any 500+ errors not related to integration. It is cloud or devices issues.

I've been running into the same issue here of no pow updates in auto or cloud mode since updating to HA 2024.3. I had been using the integration problem free, but now only can get cloud pow updates when manually reloading the integration (only get a single update), or when I open the ewelink app (triggers a single update).
Can see these entries in the HA logs.
WARNING (MainThread) [custom_components.sonoff.core.ewelink.cloud] Cloud WS Connection error: Cannot connect to host us-dispa.coolkit.cc:443 ssl:default [Try again]
Ewelink app will allow me to view instant readout of power, but cannot get things to update in HA aside from the above mentioned manual ways to get a single update. Wondering if ewelink has throttled back the API access?

@mattryyc It's just your local Internet connection issues

@mattryyc It's just your local Internet connection issues

If that’s true, why would reloading the Sonoff integration allow it to retrieve an updated power value successfully?

You should check debug logs. The integration has all the tools for answers.


[HA-Integration]: Synology NAS

Description
Purchased a Synology NAS in 2021 and been very happy with it. Would recommend
This will take a few steps

[x] #14
[ ] NAS / Linux / PiKVM is moved to the new house
[ ] Setup NAS on local network
[ ] Integrate into Home Assistant

Integration URL
https://www.home-assistant.io/integrations/synology_dsm/
Automation Idea

[ ] Keep track of status
[ ] Keep track of drive capacity

New Hardware?
No
Add Amazon / store URL for referencing if new hardware is required
NAS 1621+

Added with integration worked like a charm


Program to check for plugin updates

Thiss pull request is related to the issue #35.

I'm merging this to see how the automatic workflow works. Feedback is still welcome.


Fix merge resolution

Probably my fault, but you went a bit far in the other direction in response to my comment. Relocating the component is great for mobile, we just needed to correct the text and the wrapper/container. So this pr

reverts 8ba943d and ca438e8
changes the Version container in ConfigServer from ExternalLink to Text
Removes the $ characters left over from the template string
Removes the href prop I retained by mistake

OBE


adminserver environment and setup scripts - BLOCKER

It looks like it tries to use a directory to mount the secret to for the weblogic credentials based on /var/run/secrets-$DomainUid, instead of the secretsMountPath from create-domain-job-inputs.yaml.
Also, the domain creation script in the configmap seems to leave the domain name as base_doamin, even after writing the domain. when I exec to the adminserver pod and check the env it lists the DOMAIN_PATH correctly after sourcing the /shared/domain/widomain/bin/setDomainEnv.sh file, but the DOMAIN_NAME is always base_doamin. When I connect and check the name and read the domain via wlst.sh it seems correct. Thoughts???

I'm not finding any code in the weblogic operator git repository that sets the pod's DOMAIN_NAME environment variable to base_domain.  I'm guessing this is coming from the weblogic docker image itself - i.e. store/oracle/weblogic:<IP_ADDRESS>

On secretsMountPath, there's a comment in https://github.com/oracle/weblogic-kubernetes-operator/blob/master/site/creating-domain.md that says that it should not be modified:
secretsMountPath | Path for mounting secrets.  This parameter should not be modified. |
On the other hand, when I tested it, I noticed that it MUST be set to /var/run/secrets-
I don't see much benefit in letting customers control this in-pod pathname.  Instead, I think we should probably just pick a fixed pathname, just like we do for the operator's config map and secrets.

If you decide to force the secrets mount path based on convention (/var/run/secrets-${domainUID}), then I would suggest taking the secretsMountPath out of the yaml...since when ever you need it you could compute it.
In my version of the yaml file it just reads:
Path for mounting secrets
secretsMountPath: /var/run/secrets-domain1 - so if you don't use domain1 for your domainID you're kind of out of luck.

DOMAIN_NAME is set to base_domain in the create domain process from the wls.jar domain template I think. It usually gets set to the real name when you read and write the domain from the file system. For some reason the environment variable doesn’t seem to get set though. Maybe someone else can verify what I am seeing by running env in a domain and rerun env after sourcing the setDomainEnv.sh file.

I'm working on a fix for the DOMAIN_NAME issue.
Basically, the wls image supports several environment variables (e.g. DOMAIN_NAME) that the container can override.  I need to fix the weblogic operator to set these environment variables on
the pod it creates for the wls server.  The variables are: DOMAIN_NAME, DOMAIN_HOME, ADMIN_NAME (the name of the admin server), and ADMIN_PORT.

I'm going to work on a fix for the secretsMountPath problem too.
I'm going to choose a fixed file or computable name in the pod for the secrets for the domain
credential, and I'm going to remove support for secretsMountPath from create-domain-job-inputs.yaml.

Sounds like a good plan. Thanks.

Both issues are fixed:

the DOMAIN_NAME, DOMAIN_HOME, ADMIN_NAME and ADMIN_PORT env vars are properly set in the weblogic server pods.

the ADMIN_USERNAME and ADMIN_PASSWWORD env vars are set to empty strings in the weblogic server pods.

secretsVolumeMount has been removed from create-operator-inputs.yaml.  The secrets are mounted in the domain job and weblogic server pods under the fixed pathname '/weblogic-operator/secrets'.  NOTE: this change is not backwards compatible - you'll need to recreate your domains.


Non-functional configuration with Rellax

Set wrapper to parent element node
Set relativeToWrapper to false
Nothing moves

Your wrapper should have scrollbar and you wrapper scrollTop should change when scrolling, is it?

No, my wrapper has no scrollbar and it scroll as a part of the page.

Then you should set wrapper: null or relativeToWrapper: true to make it work.


feat: Add eks:eks-cluster-name tag and remove karpenter.sh/managed-by annotation

Fixes #N/A
Description
Dropping karpenter.sh/managed-by as it is an artifact of a migration path and not something needed in the future. Adding eks:eks-cluster-name to support pod identity sessions.
How was this change tested?
make test
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

7 of 7 (100.0%)  changed or added relevant lines in 4 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 79.132%

Files with Coverage Reduction
New Missed Lines
%

pkg/fake/utils.go
2
94.56%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.03%

Covered Lines:
5927

Relevant Lines:
7490

💛  - Coveralls


Add support for displaying PR labels

Fetches each PR as a GitHub issue (required to get the labels array) and displays any labels found on the wall in their GitHub chosen colour.
Example screenshot:

Closed, I meant to PR this on ONSDigital fork first before considering for upstream merge.

This looks really good, are you planning on reopining this PR at some point?


fix: XamlRoot may be null when the SKXamlCanvas is unloaded

Description of Change
Avoids an NRE when using a null XamlRoot on WinUI when the control is unloading.
API Changes
None.
Behavioral Changes
None.

Required skia PR
None.

@mattleibow could it be possible to backport this one to 2.88? Thanks!

jeromelaban is a null check in on the root.Invalidate call needed for the same reason? Couldn't hurt.

That's fair. I ignored that particular site because the XamlRoot.Changed event is called only when XamlRoot is set, but indeed, it does not hurt.


implement a check for duplicate keys

this will cause parsing to fail when encountering something like:
- hosts: localhost
  tasks:
    - debug:
        msg: task 1

  hosts: localhost
  tasks:
    - debug:
        msg: task 2

By way of additional explanation, we were recently bitten by exactly this problem in https://review.openstack.org/#/c/349564/, and it turns out that at the moment it is largely an un-catchable error: neither ansible itself nor ansible-lint provides a way of flagging duplicate keys as a problem.
There are clear situations in which you want duplicate keys, for example, ansible/ansible#13603, but there are other situations where duplicate keys are a clear sign of an error.

Having also been bitten by this (slipped in due to a typo; it should have been two separate plays), i'm in favor of merging!

I am cleaning up my stale pull requests. It looks as if this one wasn't of interest.


Event Stuff

What is your question?
Where is the event data in the source?

found it

PlayState for the actual code, ChartingState for chart editor descriptions

which line

You can just look up one of their names and you should find them


Subliminal plugin (v1.0.1) is not working

Latest version of flexget, and subliminal 1.0.1 (in a python env) is not working, i get this:
2016-01-20 05:19 CRITICAL task          SubtitlesSearch Plugin subliminal cannot be used because dependency subliminal is missing.
2016-01-20 05:19 WARNING  task          SubtitlesSearch Aborting task (plugin: subliminal)
Basically flexget is not recognizing subliminal executable for some reason, despite i can successfully query both versions:
root@cubox-i:~# subliminal --version
subliminal, version 1.0.1
root@cubox-i:~# flexget --version
1.2.431
You are on the latest release.

It is because Flexget uses Guessit2, but Subliminal is still using Guessit1. If you don't use the guessit parser, you can reinstall Subliminal. This will result in the guessit parser failing to load, but it's not a problem if you don't use it.

I´m not sure i get your solution. I also tried with latest subliminal (1.1.1) and i get the exact same error.

Rollback to guessit 1.0.2

Successfully installed guessit-1.0.2
root@cubox-i:~# flexget --version
Traceback (most recent call last):
File "/usr/local/bin/flexget", line 5, in
from pkg_resources import load_entry_point
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 3130, in
@_call_aside
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 3116, in _call_aside
f(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 3143, in _initialize_master_working_set
working_set = WorkingSet._build_master()
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 644, in _build_master
return cls._build_from_requirements(requires)
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 657, in _build_from_requirements
dists = ws.resolve(reqs, Environment())
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 830, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'guessit>=2.0rc5' distribution was not found and is required by FlexGet

Strange. Worked for me.

Can you provide your current setup regarding this ? Flexget, Subliminal and Guessit versions... plus if you are using virtualenvs or not ?

osmc@osmc:~$ flexget -V
Latest release: 1.2.433
osmc@osmc:~$ flexget -V
1.2.431
Latest release: 1.2.433
osmc@osmc:~$ guessit --version
+-------------------------------------------------------+
+                   GuessIt 1.0.2                       +
+-------------------------------------------------------+
|      Please report any bug or feature request at      |
|       https://github.com/wackou/guessit/issues.       |
+-------------------------------------------------------+
osmc@osmc:~$ subliminal --version
subliminal, version 1.1.1
osmc@osmc:~$ uname -a
Linux osmc 4.3.3-3-osmc #1 SMP PREEMPT Sat Jan 2 04:01:24 UTC 2016 armv7l GNU/Linux

I am having the same problem, although for me it's even worse as I can't even get  subliminal --version to work and get the following error:
  File "/usr/local/bin/subliminal", line 7, in <module>
    from subliminal.cli import subliminal
  File "/usr/local/lib/python2.7/dist-packages/subliminal/__init__.py", line 10, in <module>
    from .api import (ProviderPool, check_video, provider_manager, download_best_subtitles, download_subtitles,
  File "/usr/local/lib/python2.7/dist-packages/subliminal/api.py", line 14, in <module>
    from .subtitle import compute_score, get_subtitle_path
  File "/usr/local/lib/python2.7/dist-packages/subliminal/subtitle.py", line 7, in <module>
    from guessit.matchtree import MatchTree
ImportError: No module named matchtree ```

I have tried reinstalling subliminal without any succes.

EDIT: in some weird way, rolling back to flexget 1.2.429 solved it for me. Subliminal now works again and gives me version 1.0.1.

That's not weird at all. We moved to guessit 2.0 in version 1.2.431 and subliminal requires guessit 1.0.2.

This works now, right? That all guessit stuff is over?

No. New Subliminal version has not yet been released.

 I guess we'll keep this open then.

Try to update sublime to new RC version. Worked for me
pip install subliminal==2.0-rc1

I have updated the subliminal plugins to require >=2.0-rc1. Update and all problems should go away.

Working like charm, thanks !


Project data flow

Implemented the basic flow of data from ProjectStore to the UI. It's not hooked up to any kind of storage yet, but it's a start.

For some reason Travis was running on this even though I thought I'd turned it off. It's not even set up... OH WELL


Reload balance after transaction has sent #4394

Closes #4394

Cheking

Need me to do anything?

nope


Adding docker support

Type of Pull Request

[ ] Bug fix
[x] Enhancement

Related Issue #s or links (if any):
Description of Changes
Added support for Docker. You can find the details on this issue: https://github.com/developertools-tech/developertools.tech/issues/42

Indeed! Reverting the changes to have a single click development setup using docker-compose up

@dlford the changes are integrated. let me know if there's anything else.


Fix opening settings app with clean state on iOS

Seems like XCUIApplication launch method is not working as it is described in the documentation:

Launches the application synchronously. On return the application ready to handle events. If the application is already running, the existing instance will be terminated to ensure a clean state for the launched instance.

What is even more strange, adding self.preferences.activate() before self.preferences.launch() fixes the issue. The flow looks kind of glitchy (settings app is opened twice in a row), but the whole flow completes successfully.

Please add in PR description fix #2273 and update changelog


Bug: Some templates are not running

What happened?
Just reporting issues running templates for admiral family packages :

admiralonco :
ad_adrs_basic.R :

Error in filter_joined(dataset, by_vars = subject_keys, order = order,  :
  argument "dataset_add" is missing, with no default
Calls: %>% ... filter_joined -> assert_data_frame -> is.data.frame
In addition: Warning message:
The `filter` argument of `filter_joined()` is deprecated as of admiral 1.0.0.
ℹ Please use the `filter_join` argument instead.
ℹ The deprecated feature was likely used in the admiralonco package.
  Please report the issue at <https://github.com/pharmaverse/admiralonco/issues>.
Execution halted

ad_adtr.R :
Error in derive_vars_joined(., dataset_add = adtr_sum, by_vars = exprs(USUBJID),  :
  argument "join_type" is missing, with no default
Calls: %>% ... derive_vars_joined -> get_joined_data -> assert_character_scalar
Execution halted

admiralvaccine :
ad_adce.R :

Error in derive_vars_joined(adce02, dataset_add = adperiods, by_vars = exprs(STUDYID,  :
 argument "join_type" is missing, with no default
Calls: %>% ... der

ad_adface.R
Error in derive_vars_joined(adface, dataset_add = period_ref, by_vars = exprs(STUDYID,  :
 argument "join_type" is missing, with no default
Calls: %>% ... derive_vars_joined -> get_joined_data -> assert_character_scalar
Execution halted

ad_adis.R

Error in derive_vars_joined(adis_crit, dataset_add = period_ref, by_vars = exprs(STUDYID,  :
  argument "join_type" is missing, with no default
Calls: derive_var

ad_adsl.R

Error in library(admiral.test) :
  there is no package called ‘admiral.test’
Execution halted

(for some of them it seems just related to the use of deprecated functions and use of admiral.test that has been decommissioned )
Session Information
No response
Reproducible Example
No response

@manciniedoardo : just tagging you here, if you want to test the run of the templates you can run data-raw/create_adams_data.R on this branch https://github.com/pharmaverse/pharmaverseadam/tree/add_script
Note that you can also add github_token.R script under data-raw folder with the following code :
github_pat <- "<YOUR GITHUB PAT TOKEN" (let me know if you need more instructions to create github access token)

thanks @dgrassellyb! I've tried to run the file - it fails at this step?


Should hooks always point to actual executables

Hello!
As per @zvonkok's comment in https://github.com/container-orchestrated-devices/container-device-interface/pull/2 we should consider specifying that hooks must always point to actual executables and not symlinks.

runc , for example, will not execute a runtime hook if it is a symlink. Should we mention this fact somewhere that hooks should be "actual" executables and not symlinks?

This might be something we could check in the go library or an alternative would be that we follow the symlink before merging it in the OCI spec.
Thoughts?
/cc @zvonkok

I wonder if there is any reason for it? or just something that can be "fixed" in runc

Hello!
This topics was discussed in yesterday's meeting here are some of the notes:
TLDR:

This is the case in runc to prevent security issues.
We thought it made sense to start with a limit on pointing to executable and then expand if we see this as a blocker in the future.

More notes:

For StartContainer it seems like you might need to follow symlinks.
Should “runtime” hooks be in a root owned directory (e.g: /etc/cdi/bin)?

Concern about it being a “trashcan”
Vendors might want to install binaries in /usr/local/bin

Would be nice to get more input / think about this more


HDR AV1 metadata is parsed wrong

Hello,
mediainfo is parsing HDR side_data wrong. It seems to have been tested on x265 data mainly, which uses /50000 values:
[SIDE_DATA]
side_data_type=Mastering display metadata
red_x=34000/50000
red_y=16000/50000
green_x=13250/50000
green_y=34500/50000
blue_x=7500/50000
blue_y=3000/50000
white_point_x=15635/50000
white_point_y=16450/50000
min_luminance=1/10000
max_luminance=10000000/10000
[/SIDE_DATA]

However, for example SVT-AV1 uses /65536 (and others), which lead to wrong values. Test file attached.
ffprobe (exact same raw data as the example above):
[SIDE_DATA]
side_data_type=Mastering display metadata
red_x=44564/65536
red_y=20972/65536
green_x=17367/65536
green_y=45220/65536
blue_x=9830/65536
blue_y=3932/65536
white_point_x=20493/65536
white_point_y=21561/65536
min_luminance=2/16384
max_luminance=256000/256
[/SIDE_DATA]

But the info displayed by mediainfo is wrong:
Mastering display color primaries        : R: x=0.891280 y=0.419440, G: x=0.347340 y=0.904400, B: x=0.196600 y=0.078640, White point: x=0.409860 y=0.431220
Mastering display luminance              : min: 0.0002 cd/m2, max: 25.6000 cd/m2

Mastering display luminance should be Mastering display luminance              : min: 0.0001 cd/m2, max: 1000 cd/m2  for example
https://user-images.githubusercontent.com/7951225/162791238-a185f277-8415-4f64-8279-796858b4019b.mp4.

However, for example SVT-AV1 uses /65536 (and others), which lead to wrong values.

There is a standard about values, with exact numbers, but someone decided to use another denominator... Fun! It leads to non precise values e.g. 20493/65536=0.312698 vs 0.312670 elsewhere, and I hope for the person who chose that the future will not require the value 1.0 (impossible to store in the stream with this denominator).
Thank you for the precise report, we won't o as FFmpeg with the explicit denominator but we'll use the right one for showing the right floating point value.

@mxsrm I have that now:
Mastering display color primaries        : Display P3
Mastering display luminance              : min: 0.0001 cd/m2, max: 1000 cd/m2

But still:
Maximum Content Light Level              : 2594 cd/m2
Maximum Frame-Average Light Level        : 1698 cd/m2

Are there the expected values, else what are they?

@mxsrm how do you get the mastering metadata with ffprobe?
While checking the only AV1 file I have, and it is 50000 based, I don't remind where I got it and I am curious about what FFmpeg says. I bet it will show 65536 based values but I would like to be sure (in case there is somewhere a check on "bad" behavior in "old" encoders?). This is not for not implementing the 65536 based analysis as the spec is clear about it ("0.16 fixed-point").

Have you tried the file I attached above? I think the denominator is dependent on the encoder. Maybe AOM produces 50000-based, I'm not sure.
ffprobe -i test -select_streams v:0 -show_frames -read_intervals %+#1
Maximum content light and frame-average light level are correct, because there is no denominator involved:
[SIDE_DATA]
side_data_type=Content light level metadata
max_content=2594
max_average=1698
[/SIDE_DATA]


Feature request: specify transition kind

Totally no rush in this, but it would be nice to be able to specify your own transition in the motion.div, easeInOut for example.
Additionally, dragging currently uses the transition, but opening and closing by setOpen does not.

@costyn sounds good, will add it!


Add a messageBubble type, add option to set open status on loading

Can we add a messageBubble type? like as google canlendar.
This is a chatbot example with calendar appointment.
https://truchat.io/rtgc/rtgc-4003/?utm_source=truchat-rtgc-gallery&gclid=CjwKCAjw3-bzBRBhEiwAgnnLCgZShmnNF9roJ29JbJcK02iNCulHU9jGXW2OcR5cEPAPEA7hz9g-vhoC5SUQAvD_BwE
You can complete all questions til last and check schedule calendar.
And I'd like to add an option for open-status on loading time.
Thanks.

Hi there,

About the first steps in your reference, you can do the same just by using the text & button message type.

About the calendar, I think that's not a common component and shouldn't be in this package. Because that seems like a custom type and used for your project only.

I will add the isOpen prop soon.

Updated


feat: Reorganize sell screen

This PR resolves ONYX-869
Description
Reorganize Sell screen.

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Reorganize Sell screen - ole

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (Reorganize Sell screen - ole)

  Generated by :no_entry_sign: dangerJS against dacdeeb2ea47a62e3079891a4af70e91d61ff6fe


ocdav service still using machine auth

The ocdav service (living in reva) is still using machine auth in some cases. We want to use service accounts instead
Acceptance criteria:

[ ] pass service account config to ocdav
[ ] find occurences of machine auth (search for MachineAuthAPIKey) and replace them with service accounts
[ ] remove MachineAuthAPIKey config var in ocdav (reva)

Two occurrences of this in revas ocdav service:
https://github.com/cs3org/reva/blob/edge/internal/http/services/owncloud/ocdav/ocdav.go#L343
https://github.com/cs3org/reva/blob/edge/internal/http/services/owncloud/ocdav/ocdav.go#L370


Character cause error when compiling projects

Hi.
I've been using this extension and compile the project to get errors. I could see that the problem is in the "demo/index.html" file, because there is a zero in the air within a style class, on line 170.
.locale_example {         text-align: left;         padding: 8px; 0       }
Can you fix this? Please.
Regards

@darwinserrano Mind submitting a PR for fix that?


[Codegen][GPU] Support all kinds in combiningKindToAllReduce()

Changes this function so it supports all members of the CombiningKind enum; the min/max discrepancies that originally motivated their exclusion no longer exist. Since it covers all enum members, it is changed to no longer return an optional. Note that this changes the generated MLIR (gpu.subgroup_reduce is generated), but that should ultimately lower to the same pattern of shuffles as before.
Function is made static as it is not used anywhere else and has no prototype in a header.

Side comment:: vector::CombiningKind currently uses I32BitEnumAttrCaseBit which cause enum to be generated like this:
enum class CombiningKind : uint32_t {
  ADD = 1,
  MUL = 2,
  MINUI = 4,
  MINSI = 8,
  MINNUMF = 16,
  MAXUI = 32,
  MAXSI = 64,
  MAXNUMF = 128,
  AND = 256,
  OR = 512,
  XOR = 1024,
  MAXIMUMF = 4096,
  MINIMUMF = 2048,
};

Which is most likely not what original author intended

Which is most likely not what original author intended

Good catch, @Hardcode84. File an issue / submit a patch upstream? This could be a good starter task.

Yeah, I'm not sure why they used a bitfield-style enum either.

Good catch, @Hardcode84. File an issue / submit a patch upstream? This could be a good starter task.

Will create an issue later today.

https://github.com/llvm/llvm-project/issues/107448

I can make a PR later if we don't want to keep it as starter issue.


ProgressBar calls the _draw() function an infinite number of times

Tested versions

Reproducible in 4.3.stable - Not reproducible in v4.2.1.stable

System information
Godot v4.3.stable - Windows 10.0.19045 - GLES3 (Compatibility) - NVIDIA GeForce RTX 3050 (NVIDIA; <IP_ADDRESS>70) - AMD Ryzen 5 3600 6-Core Processor (12 Threads)
Issue description
When I add fill_mode = 1 to _draw() function for Progress Bar and run scene with that progress bar in it game freezes before loading properly. In Profiler Debugger you can see that the function _draw() is called houndreds of thousands of times. After some time game unfreezes.
Steps to reproduce

Create a scene
Create Progress Bar
Attach Script to Progress Bar
In that script paste

func _draw() -> void:
    fill_mode = 1

Run scene
Game freezes

Minimal reproduction project (MRP)
progress_bar_glitch.zip

Setting fill_mode queues a redraw, so if you do it inside _draw(), it will just queue itself infinitely. This applies to all similar properties/nodes.

So the problem here has nothing to do with either ProgressBar or fill_mode.
Here's what I could reproduce:
Any Control node that calls queue_redraw in _draw, which exists when the game starts up, will result in an infinite loop (of redrawing) on startup (causing an infinite Godot logo screen).
Example code:
extends Control

func _draw():
	print("Drawing")
	queue_redraw()

Put this on a node on startup to reproduce.
I have a feeling this may already be known, but if not, this issue should probably be renamed/changed.

Got a fix for this up:

https://github.com/godotengine/godot/pull/97328


Docker, unable to open database file: out of memory

YADMB Release 0.15.0 on Ubuntu 22.04.1 LTS
Cloned the repo git clone https://github.com/TheTipo01/YADMB.git
Created a config.yml based on sample and placed into data folder
Ran docker-compose up -d to run in background.
Check logs and:
2023-06-16 22:05:05 [LIT0] main.go:49:0() youtubeapikey: required validation failed
2023-06-16 22:05:05 [LIT0] main.go:147:main() No token provided. Please modify config.yml

Seemed Youtube API key is required, not just recommended.
Anyways got a API key, removed the container, and tried to remake it. Check logs and:
2023-06-16 22:10:40 [LIT0] utilities.go:15:ExecQuery() Error executing query, unable to open database file: out of memory (14)
2023-06-16 22:10:40 [LIT0] utilities.go:15:ExecQuery() Error executing query, unable to open database file: out of memory (14)
2023-06-16 22:10:40 [LIT0] utilities.go:15:ExecQuery() Error executing query, unable to open database file: out of memory (14)
2023-06-16 22:10:40 [LIT0] utilities.go:15:ExecQuery() Error executing query, unable to open database file: out of memory (14)
2023-06-16 22:10:40 [LIT0] utilities.go:15:ExecQuery() Error executing query, unable to open database file: out of memory (14)
2023-06-16 22:10:40 [LIT0] main.go:95:0() Error loading blacklist: unable to open database file: out of memory (14)
2023-06-16 22:10:40 [LIT0] main.go:101:0() Error loading DJ settings: unable to open database file: out of memory (14)

Hello there!

Seemed Youtube API key is required, not just recommended.

The latest commit fixes this problem.
As for the database issue, i'll investigate further tomorrow, and hopefully find the problem

The only way i've been able to replicate your issue is by passing a datasourcename like in this error from the sqlite library we're using.
Another user in that issue says that they got that error when the user didn't have write permission on that file/directory.
What does your datasourcename string look like in the config file?


SIL: Don't try to lower various illegal tuple types

I added some assertions to prevent us from serializing tuple types with parameter flags except when part of a function type. This turned up some cases where SIL was constructing illegal tuple types.
Fix these in a sort of hacky way so that I can land the other changes until we rework SIL type lowering to use the new function type representation.

@swift-ci Please test

@swift-ci Please test source compatibility

@jckarter so as you were saying, all of this will be much simpler once we stop imploding the entire parameter list in re-abstraction thunks, and also if we stripped labels off in tuple type lowering. But for now, I'm trying to centralize the invariants here and add some tests for the edge cases that came up.


chore: Separate building Node from starting it

Another step towards customising behaviour for different applications.
Everything has the same init order to avoid unintended changes
during refactor.
As a side effect, we can stop the node now by simply dropping the instance of
RunningNode. This can become useful in tests.

Closing this PR in favour of #1062


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 960 ms

Resolved: Bofrost.de performance has improved in ba37aed after 18 minutes.


Update and do release in pypi

We need a stable version of the repository for other projects (ya2ro)
@dakixr please do a pull request to this repository and we will do a release in pypi. We need installation and usage instructions

Hii, I need a stable version of SCC on PyPI to upload the next release of ya2ro 0.1.0

Makes sense.. we'll try to produce it soon


refactor(coral): Add helper to use specific feature flag for mocks in tests

Linked issue
Resolves: #1143
What kind of change does this PR introduce?

[ ] Bug fix
[ ] New feature
[x] Refactor
[x] Docs update
[ ] CI update

What is the current behavior?
With the current recommendation how to mock a feature flag in tests, we set all available feature flags to true.
What is the new behavior?
We want to provide and recommend a way to mock specific feature flags. That way it's possible to test exceptions based on specific feature flags in components that use more than one. In the past, this was for example the case for MainNavigation.
This PR

adds a test helper that provides a function to mock specific feature flag
updates documentation about feature flags

Requirements (all must be checked before review)

[x] The pull request title follows our guidelines
[x] Tests for the changes have been added (if relevant)
[x] The latest changes from the main branch have been pulled
[x] pnpm lint has been run successfully

Note: This is a small improvement in case we have to test a feature flag. There's currently no flag active that is used in tests (I removed some unneeded mocks). I added a flag and test for it locally to check out if everything works like expected and updated the docs accordingly.


Part 8 of RFC2906 - Keep InheritableFields in a LazyCell inside `…

Tracking issue: #8415
RFC: rust-lang/rfcs#2906
Part 1: #10497
Part 2: #10517
Part 3: #10538
Part 4: #10548
Part 5: #10563
Part 6: #10564
Part 7: #10565
This PR focuses on optimizations surrounding InheritabeFields and searching for a WorkspaceRootConfig:

Keep InheritableFields in a LazyCell inside to_real_manifest so we only search for a workspace root once per to_real_manifest
Changed calls for getting a workspace root to use inherit_cell.try_borrow_with()

Testing Framework
Test Results:
nest=1, runs=7, members=75, step=25

25 Members:

Optimized: 0.145s
Not Optimized: 0.181s
Normal: 0.141s
Percent change from Normal: 2.837%

50 Members

Optimized: 0.152s
Not Optimized: 0.267s
Normal: 0.141s
Percent change from Normal: 7.801%

nest=2, runs=7, members=75, step=25

25 Members

Optimized: 0.147s,
Not Optimized: 0.437s
Normal: 0.14s
Percent change from Normal: 5.0%

50 Members

Optimized: 0.159s
Not Optimized: 1.023s
Normal: 0.141s
Percent change from Normal: 12.766%

Using a LazyCell for InheritableFields brought down runtimes to more acceptable levels. It is worth noting that this is a very harsh test case and not one that represents real-world scenarios. That being said there are still some optimizations that could be done if we need to. The biggest one is making sure we only parse a Cargo.toml once.
Remaining implementation work for the RFC

cargo-add support, see epage's comment
More optimizations, as needed

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

r? @epage

@bors r+

:pushpin: Commit 125c274b0abddddf24fd90d61001f3287faa9b76 has been approved by epage

:hourglass: Testing commit 125c274b0abddddf24fd90d61001f3287faa9b76 with merge b75281bee39818bd538f81dd8f82ec2fec059a39...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing b75281bee39818bd538f81dd8f82ec2fec059a39 to master...


Fix links in when expressions migrations announcements

Changes

Today, those announcements link to https://github.com/tektoncd/pipeline/issues/1836 which is an unrelated issue.
In this change, we add a link to the issue tracking that work - https://github.com/tektoncd/pipeline/issues/4461.
/kind cleanup
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
[n/a] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

NONE

/test pull-tekton-pipeline-alpha-integration-tests
/lgtm

/test pull-tekton-pipeline-alpha-integration-tests
sidecar test

/approve
sidecar teeeeeesssst fist shake

/test pull-tekton-pipeline-integration-tests
known flake reported in https://github.com/tektoncd/pipeline/issues/4427


Function javadoc about aggregate not being supported, outdated?

The main Function javadoc states:

Aggregate functions are not yet supported, but coming soon.

see:
https://github.com/xerial/sqlite-jdbc/blob/master/src/main/java/org/sqlite/Function.java#L53
However there is a sub-class Function.Aggregate that seems to be doing just that. Is the javadoc outdated?

I noticed this a few months back too; I think it's just outdated.  You could submit a PR to fix it if you wanted.


Update DiskIo telemetry device to persist the counters

Ensure that DiskIo telemetry does not rely on Rally being a parent
process of Elasticsearch and persists the disk counters at the beginning
of a benchmark and can read it again afterwards.
Relates to #697

Re-opening https://github.com/elastic/rally/pull/721 here as the repository state got messed up.


update input binding code block - ReadMe

This aligns it with other examples in the file so seems reasonable to update


classic game.py

this is for Ogiroid

I doubt this will even work with the Discord bot.

Red flag number 1 You are not importing disnake
Red flag number two you are importing pygame...
Red flag number 3 this is a GUI based game... 😂

Great Game but this is a discord bot lol

Yeah, uhm... not gonna work with Ogiroid, at all...

Great Game but this is a discord bot lol

HeLlO eVeRyOnE wElComE bACk TO mY yOOtOoBe ChAnnEL gUYS, tOdAy i WIlL RuN GuI bAsEd CmD oN DiSCoRD wiThouT imporTing ThE lIbRARy! LeTs GO!!! [boy that too too long to type]


A Tribute page containing mostly html

A tribute page with inline stylings and internal stylings. Best for everyone to start working on web development.

Can you share the UI and tech stack?

https://wool-flax-journey.glitch.me
you can refer this link. This is just a basic tribute to help beginners.
I can align the paragraphs to make it look good


Examples are for old apiVersion

The ingress examples are for the extensions/v1beta1 version that should not be used for new deployments.
edit:
Examples throughout the documentation are for old Kubernetes resource versions, also in the How-To section, e.g https://docs.microsoft.com/en-us/azure/application-gateway/ingress-controller-letsencrypt-certificate-application-gateway

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

ID: be6f5f5f-6567-b685-004b-e7b537f0934f
Version Independent ID: f8623293-7ede-ec51-d25d-3f0389edbd08
Content: Application Gateway Ingress Controller annotations
Content Source: articles/application-gateway/ingress-controller-annotations.md
Service: application-gateway
GitHub Login: @Caya
Microsoft Alias: caya

@obesser , thank you for your feedback. We have assigned this to the content owner/author for review.
@mscatyao , for your review.

Hi - I will update the article as I am able. I am closing this issue as duplicate because we have others saying the same thing. Thanks for bringing this up. #please-close


JavaDoc grammar issue: 'Emits an game event'

The sentence 'Emits an game event' should read 'Emits a game event'.
https://github.com/FabricMC/yarn/blob/5a3bdbd522adfa5ad26bd4220ce5ca7cfefff269/mappings/net/minecraft/world/WorldAccess.mapping#L7

That sentence does not add useful information to be honest. It's repeating the method name.

I think there's still some value in providing javadoc for such methods; the ability to show that a method is noteworthy is one of the benefits.


Improvements to consumable options

Added extra support for consumable options

Now allows for a spiritbond potion to be used as Medicine
Now allows for the usage of engineering manuals
Now allows for the usage of squadron manuals

Is there a way to enable multiple different squadron manuals at once?


--health-cmd ?

I (wherever possible) monitor my containers with the excellent willfarrell/autoheal docker container.  What would be a suitable --health-cmd for traefik-cop to test it's state of health?

@TheFlyingBadger the program operates fairly simply -- it continuously watches the docker API for any changes and when one is detected, tries to write those changes to redis. If there is any unexpected (unhandled) error in processing the config change or writing it into redis, the program will panic and exit with an error. In that situation, the container should automatically restart.
Otherwise, if the container is running, it should be working. I haven't seen a situation so far where it stopped working for some unknown reason and needed to be manually restarted. Hope that helps.


Levitation spell does not respect havens

Probably should hook this check into all event-based spells. Makes it so it isn't checked multiple times for spells with the same event. Slight performance boost, but I worry about the readability.

Probably should hook this check into all event-based spells. Makes it so it isn't checked multiple times for spells with the same event. Slight performance boost, but I worry about the readability.

No, some are ambiguous - should the attack block fail when the player is in a haven, or the block? It may change depending on the spell.


NStepBi_xxx 's behaviour is inconsistent with the document

I am using Chainer version: 2.0.0
Documents of NStepBi_xxx (like this) says that the variables like hx (argument of the function) and hy (value returned by the function) have the shape such that (S, B, N) where S is number of layers and is equal to n_layers, B is mini-batch size, and N is dimention of hidden units.
However, the actual shape returned by the function is inconsistent with the document.
For example:
x = chainer.Variable(np.random.randn(1,5).astype('f'))
l1 = L.NStepBiLSTM(n_layers=2, in_size=5, out_size=5, dropout=0.0)
h, c, hx = l1(None, None, [x])
print(h.shape) # (S=4,B=1,N=5), thus S != n_layers

The actual shape is twice the value of n_layers (as it is bi-directional).
Thus I guess that the document should be fixed.
This inconsistency might be due to the difference between the L.NStepBiLSTM and F.n_step_bilstm. Still, it should be noted on the document of L.NStepBiLSTM.

L.NStepBiLSTM have bug, I encounter in running output size of L.NStepBiLSTM   is 2 x I pass out_size

@sharpstill
Do you mean that output size of L.NStepBiLSTM is  2 x out_size?
(e.g.,)
x = chainer.Variable(np.random.randn(1,5).astype('f'))
l1 = L.NStepBiLSTM(n_layers=2, in_size=5, out_size=5, dropout=0.0)
h, c, hx = l1(None, None, [x])
print([_.shape for _ in hx]) # [(1, 10)]   # (1, 2 x 5)

I think the output size should be concatenated by the hidden vector.


Cannot read properties of undefined (reading 'includes) on build with depedencies

Hi,
I'm trying to make a package, that contains a class.

Then I try to import it in another project using "dependencies" array in configuration file

The problem is when Aventus try to build it throw an "Cannot read properties of undefined (reading: 'includes')". Maybe it's my package that is not properly implemented. If it's my package can you add to documentation how to build a package.
Thanks

You shouldn't create a package file manually. A package file is the compiled version of one build. You can find more informations about how to import package here and how to export package via config file here under outputPackage


Signature help formatting issues after new implementation

Too much escaping (Fixed in pull request #608):

Chopped up signature in LSP from pyls:
(cannot reproduce this after updating from pyls 0.19.0 to 0.26.1)

Strange signature from isinstance from pyls:
(signature label is Return whether an object is an instance of a class or of a subclass thereof., looks like https://github.com/palantir/python-language-server/issues/309 may be relevant)

also consider using variable.function instead of entity.name.function:

variable.function is for function(-like) calls.
entity.name.function is for function definitions/declarations.

That choice is intentional as I want the signature to look like the definition, not the invocation.

Cool, then that's the viewpoint we take.

Verified that the is instance is fixable from pyls, all known LSP issues are fixed for the moment.


Remove happiness of error message

In the even that black encounters an error, the output gleefully tells the user it is "All done!", followed by the emoji ligature of sadness.
This PR corrects that by ensuring that there are no mixed feelings on error, that indeed two collision emoji and a broken heart emoji is a very not good state to be in, and now returns "Oh no!"
Sample output:
Before:
error: cannot format app.py: Cannot parse: 13:33: @app.route("/", methods=["GET"]):
All done! 💥 💔 💥
1 file failed to reformat.

After:
error: cannot format app.py: Cannot parse: 13:33: @app.route("/", methods=["GET"]):
Oh no! 💥 💔 💥
1 file failed to reformat.

Love the branch name

Haha thanks for making my day :) What if there are multiple input files, some of them just fine, and some with errors?

Hi @zol. I haven't changed any of the functionality; there's one error_code returned by one report object, so the functionality is the same: if there's any errors at all, sadness is eternal.
Oh no! 💥 💔 💥
1 file left unchanged, 2 files failed to reformat.

Fair enough. Thanks 💯


refactor(shared-data): update TC schema v6 commands

Overview
This PR updates JSON schema v6 TC commands to match our updated interface. All frontend + backend changes to accommodate the schema change should be made in this PR.
Will close https://github.com/Opentrons/opentrons/issues/10137
Changelog

Update TC schema v6 commands

Review requests
Risk assessment
Med

thank you! are y'all planning on doing that before the beta is cut, or should i revert this merge commit? cc: @b-cooper

Recapping from slack convo: We won't need to revert the commit since it doesn't break any existing behavior.

blockMaxVolumeUl isn't used by any clients nor PE
setTargetBlockTemperature exists in PE but isn't consumed by any entity
waitForTargetTemperature exists in schema and not in PE but is not used by client/ PD. (And hence is not immediately needed in PE)


Scannet Depth inpainting

Hi, as there is no news from you regarding a previous issue (https://github.com/huaaaliu/RGBX_Semantic_Segmentation/issues/17), i will open a new one.
Looking at the HHA generated images, the repo to generate them use inpainted depth with the raw depth as optional input, but the scannet only provide the raw depth images. Did you first inpaint the depth to fill the empty value or did you directly use the raw depth ?
If it is the case, how did you inpaint them ?
Thanks for your answer.

Hi,
thank you for your question and interst in our work. You could try simply filling the empty areas with maximum values, which can already help deliver good segmentation results, as shown in our work. Using inpainting methods would further be beneficial.
Best,