[v6.0.2] Proposal: make form's submitFailed prop accessible in Field

I'm not sure if there's a better way to accomplish this -- if so, please let me know!
In order to reduce boilerplate, I've written a higher order component to wrap my widgets for use with redux-form:
const reduxFormify = Component => props => {
  const {input, label, meta: {touched, error}, ...custom} = props;
  return (

      <Component
        label={label}
        {...input}
        {...custom}
      />
      {touched && error && {error}}

  );
};

export default reduxFormify;

For my particular form, I only want to show errors if something went wrong (not synchronous) so I'd like to replace my display logic with {submitFailed && error && {error}}. Right now, I am passing submitFailed from my form to every instance of <Field>, and it would be handy if this was done under the hood. I imagine this is a common use case and would be a fairly minor change.
If this change is acceptable I can make a PR. I'm fairly new to open source (and programming in general) so I'm still unfamiliar with the whole process. Thanks!

meta.submitFailed added in v6.4.3!
Related: #2346, #567

This is also helpful for automatically focusing a field that has an error after submit failed, thank you guys!


✨ Video recording settings

Description
Current implementation of CamerAwesome does not provide any settings on Android and only a few on iOS on the video recording output.
This PR adds the following Android settings:

bitrate setting
quality selector (with fallback)
respects the selected aspect ratio

On iOS, it may be possible to record square videos using something like this:
https://stackoverflow.com/questions/36743842/record-square-video-using-avfoundation-and-add-watermark
It does not seem as straightforward as on Android though.
Not sure about other settings.
This should help #211 but doesn't include FPS.
Checklist
Before creating any Pull Request, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).

[x] 📕 I read the Contributing page.
[x] 🤝 I match the actual coding style.
[x] ✅ I ran flutter analyze without any issues.

Breaking Change

[x] 🛠 My feature contain breaking change.

If your feature break something, please detail it

Hi, Is this feature available on camerawesome: ^1.4.0 ?

No, it's for the 2.0 release (#315).

Ok, Thanks @apalala-dev . Can you please tell when the 2.0 will be released ?

Ok, Thanks @apalala-dev . Can you please tell when the 2.0 will be released ?

No ETA yet, there are still a few bugs that need fixes.
If should not be too far though, probably in 1 or 2 weeks.

Hi @apalala-dev , I have one requirement. I want to upload video chunks to AWS S3 while the video is recording. I tried reading the bytes from the video path (where the recording video is being written) and uploaded it to server as part by part using AWS multipart request. But I think the uploaded parts are corrupted or broken. Because after all parts upload is completed, the final stitched video is not playing. I guess the issue is in reading bytes from the video path while it is being written. Can you please tell me, is there any option in this plugin to get the valid video bytes as chunks while the recording is going on. Thanks.

Based on that discussion, I assume it's not possible (or at least not easily) right now.
CameraX doesn't offer enough control over the output so your use case is quite hard to implement. If you find a workaround, feel free to share it here or on a new issue (it might interest other people).
The iOS side being completely different, it may be possible there, I don't know.
If you are interested in this feature on Android, your best bet atm would probably to try to implement it with Camera2 on the native side. This means implementing the native part yourself.
You may try to create an issue to let know the CameraX team that you are interested in this feature. They may take a look at this feature faster.


Two different behaviors when quota is reached

Hi,
Thanks again for the wonderful package and the prompt help in the past.
I've been setting up several anchorbanks in a loop. There are two separate behaviors when my IP reaches the query quota:
1) "Quota reached! Please change IP and press any key to continue"
This seems to be the standard message which your code generates. When this comes up I can indeed just change the IP and continue from there.
2) The second option, I pasted the last few normal lines as well as the error message below:
(I was hoping there is something I can do to prevent this, as it seems like this way the progress on the anchorbank being built at the time is lost.)
Bad keyword '/m/013v0m', because The request failed: Google returned a response with code 400.
13%|█▎        | 74/568 [03:28<20:41,  2.51s/it]
Bad keyword '/m/02nm6c', because The request failed: Google returned a response with code 400.
14%|█▎        | 77/568 [03:36<19:43,  2.41s/it]
Bad keyword '/m/084cz', because The request failed: Google returned a response with code 400.
74%|███████▍  | 423/568 [3:41:14<1:15:50, 31.38s/it]
Bad keyword '/m/0d5nn', because The request failed: Google returned a response with code 429.
Traceback (most recent call last):
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\gtab\core.py", line 139, in _check_keyword
rez = self._query_google(keywords=keyword)
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\gtab\core.py", line 129, in _query_google
self.pytrends.build_payload(kw_list=keywords, **self.CONFIG['PYTRENDS'])
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\pytrends\request.py", line 169, in build_payload
self._tokens()
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\pytrends\request.py", line 179, in _tokens
trim_chars=4,
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\pytrends\request.py", line 148, in _get_data
response=response)
ResponseError: The request failed: Google returned a response with code 429.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\Hersh\Documents\RSV PROJECT\querying_code_updated.py", line 78, in
countries_search_data = get_search_data('/m/02f84_', iso_countries, "2021-05-05 2021-10-30")
File "C:\Users\Hersh\Documents\RSV PROJECT\querying_code_updated.py", line 60, in get_search_data
t.create_anchorbank() # takes a while to run since it queries Google Trends.
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\gtab\core.py", line 799, in create_anchorbank
google_results = self._get_google_results()
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\gtab\core.py", line 300, in _get_google_results
keywords = [k for k in tqdm(keywords, total=len(keywords)) if self._check_keyword(k)]
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\gtab\core.py", line 300, in
keywords = [k for k in tqdm(keywords, total=len(keywords)) if self._check_keyword(k)]
File "C:\Users\Hersh\Anaconda3\envs\VPN_SWITCHING\lib\site-packages\gtab\core.py", line 147, in _check_keyword
raise ConnectionError("Code 429: Query limit reached on this IP!")
ConnectionError: Code 429: Query limit reached on this IP!

Hi,
I've also encountered this issue with Code 429.


expose batch-size and other pipeline specs are ENV vars

Summary
Expose pipeline limits as env vars so that SDKs can optimize the internal buffer size etc using these.

Is this for source? Are we able to leverage the parameters passed in Read() ?

Can we get something at start time?


HAL 01: Zeta supply does not track assets correctly

### Tasks
- [ ] https://github.com/zeta-chain/node/issues/1352
- [ ] https://github.com/zeta-chain/node/issues/1353
- [ ] https://github.com/zeta-chain/node/issues/1354
- [ ] https://github.com/zeta-chain/node/issues/1355

@kingpinXD can we break down into several tasks?

Is this completed? @lumtis @kingpinXD ?

No, it's not
The supply checker initially implemented was disabled as we planned to test it on Athens to determine the frequency of these checks so that the notifications are helpful.
This would need to be prioritised / revisited to enable it backup again . Based on the current codebase , we would need to refactor the checking function

We decided to go with monitoring + using emergency proposals
The previous ZetaClient implementation was bloated since requiring new queries and unclear process to report accounting issues.
There is no reason for ZetaClient to be in charge of this if emergency proposals are executed by external parties
What we should do is refactor to use a lock/unlock model for ZETA instead of mint/burn for non-Ethereum chians but we should first decide if we want to transition ZETA to the contract v2 model

Closing in favor of https://github.com/zeta-chain/node/issues/881


Fix: Allows disabling root symlinks

Currently this configuration in the Caddyfile will always throw an error if resolve_root_symlink has a second argument
php_server {
    resolve_root_symlink false
}

This pull request fixes the behaviour, so it works in accordance to the docs and allows disabling checking for symlinks.
Not checking for symlinks actually makes a 'Hello World' in frankenphp about 5% faster as you can see in the following flamegraph (path/filepath.EvalSymlinks):

I just realized that in worker mode the value of resolveRootSymlinks is always true since worker mode seems to reference the global FrankenPHPModule and not the one created by the php_server directive.
Is this actually the desired behaviour? Maybe it would make sense to set the default to false here?
https://github.com/dunglas/frankenphp/blob/4a8555571cf2ac60c2d1038708f1b5c66cc244a0/caddy/caddy.go#L249-L252

Is this actually the desired behaviour? Maybe it would make sense to set the default to false here?

I think the most plausible reason for this behavior is that I forgot to add the needed config! This isn't intended, we should expose a config for that in the "worker" global block. Would you mind to add it?
I think that we should keep the default to true, because many tools rely on symlinks and because it would be a breaking change to change.
That being said, we should document the performance hit (maybe can we add some sort of cache to limit it by the way) and update our benchmarks to disable this feature!

After reading the code a second time, I think this is a leftover of some tests. Removing these two lines should be enough to fix the bug: https://github.com/dunglas/frankenphp/blob/main/caddy/caddy.go#L330-L331
(still, we must handle the error case too).

I removed the 2 lines and yeah it works now 👍
It also works in worker mode, I was initially just confused, because I added a php before file_server directive, which caused the second php_server directive to be ignored (since I guess it parses the first directive)
php before file_server
php_server {
    resolve_root_symlink false
}

I do have some other small ideas  that I might look into in the future based on these flamegraphs (created from the 'Lock Free Handles' branch):

The fileserver does work even when no files are being served (php_server is around 12% slower than php)
The zap.logger does work even when nothing is logged
frankenphp_register_bulk_variables becomes a bottleneck at some point (maybe registering php server variables just is that slow?)
runtime.futex sometimes takes up more time when there are more threads created (maybe that's just caused by contention for the request channels?)

Thanks for your research!
For the fileserver hit, I don't know how we can improve this, but maybe @francislavoie or @mholt will have an idea.
For Zap, wrapping all calls with the Check() method as we do on Mercure is known to improve performance: https://github.com/dunglas/mercure/blob/4e2e253d2c68b78026cccdb2adabfc6b125e14de/handler.go#L103-L105
frankenphp_register_bulk_variables is indeed a heavy operation (most of the work is actually done in this function). I already optimized this path, but there is likely room for improvement.
I've no specific idea for runtime.futex, but IIRC it is quite common and expected in Go (due to how the Go runtime works).

The filesystem hit is why we have it off by default in Caddy itself for php_fastcgi. I don't see how it can be avoided when you actually use symlinks.

@francislavoie thanks for the hint! Having the file server available is convenient and needed for the default setup of Laravel and Symfony, so I propose to keep it by default (especially because it's possible to disable it for maximum performance). However, we should update our benchmarks as well as the TechEmpower benchmark to use the php directive directly.
I'll try to add a "performance" docs entry to compile all these tips.

Thank you very much for this fix @AlliBalliBaba!


Fix chat_with_documents.py callbacks

Problems

The question is not displayed in on_retriever_start because the function is missing a positional argument (issue #24).
The last agent's answer displays the rephrased question before the answer. Happens only in the last message.

Screenshot showing both issues:

Solution

Add the missing serialized parameter in on_retriever_start (as described in the documentation).
Add a workaround for the rephrased question that appears in the answer:

Use on_llm_start callback to identify the run_id of the tokens from the "Human" question (LLM format for the human messages).
Save the run_id to ignore the tokens from the "Human".
Ignore the tokens from the run_id to ignore in on_llm_new_token, comparing with the run_id in the parameters.

Testing
Tested locally, not showing the error described in #24, and not seeing the rephrased question in the last LLM answer.
Screenshot showing the result (same input as the previous screenshot):

@sfc-gh-jcarroll can you please check this out? Thanks.


mrpt_path_planning: 0.1.1-1 in 'iron/distribution.yaml' [bloom]

Increasing version of package(s) in repository mrpt_path_planning to 0.1.1-1:

upstream repository: https://github.com/MRPT/mrpt_path_planning.git
release repository: https://github.com/ros2-gbp/mrpt_path_planning-release.git
distro file: iron/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.1.0-1

mrpt_path_planning
* Fix usage of obsolete mrpt methods
* update ros badges
* Contributors: Jose Luis Blanco-Claraco

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/preparing-for-iron-sync-2024-03-22/36722/2

Merging despite Iron hold as this PR aims to fix a regression.


[Question] Is there a way to extract keys from custom message declarations?

This is not a problem with this library. I see that it is using @formatjs/cli which is in turn using babel-plugin-react-intl to extract keys for each page, and babel-plugin-react-intl only extracts keys for defineMessages(), defineMessage(), and <FormattedMessage> imported from react-int and declared according to react-intl's API.
However, many developers, myself included, find those declarations overly verbose and use something like
// translate.js

export default const t = (id) => ( <FormattedMessage id={id} /> )

and then elsewhere in code
// app.js

import t from "./translate"
...
<span>{t("hello")}</span>

And so those declarations don't get extracted.
So the question is - is there a way to extract such declarations?

is there a way to extract such declarations?

For sure. This plugin actually gives the user control over how to extract the keys through the extractKeysFile option. Using @formatjs/cli is just a built-in default.
https://github.com/hjylewis/babel-plugin-inline-i18n-messages#extractkeys
Just define a function extractKeys function that works with your set up. In this case you'd want to parse the given file into an AST and look for a call expression of t from your translate module. Or use something simpler like regex.
Does that answer your question?

I see, thanks! I've never written a babel plugin before, but I'll look into it.

I would recommend astexplorer.net as a tool: https://astexplorer.net/#/gist/aa353a428a70e26aabc120dbbaf65126/988ede14a72803e8f4bef9b5652858dc29c5f0bb

That's a great tool, many thanks!


Python script to find duplicate files

Python script to find duplicate files
Dupsfinder -
What have you Changed
Added python script dupsfinder.py
Issue no. #198  -
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md with my script

Join Us on Discord:- https://discord.gg/JfbK3bS

@codePerfectPlus Waiting for review! :smile:

@codePerfectPlus Welcome :smile:


[Feature] iframe Widget

Summary
An iframe widget will allow app developers to embed other pages or components directly.
Developer Handoff File
https://www.figma.com/file/8aAbH6N9lYh4RnV37Rz4E8/Iframe-widget-design-Tue-4-May-2021?node-id=2%3A3024

@hetunandu can you suggest some of the properties of the iframe apart from URL?

Could I ask, would something like this be possible ?
https://stackoverflow.com/questions/11878947/auto-login-remote-site-inner-in-iframe

Could I ask, would something like this be possible ? (noting that this approach is insecure) ... but something like this.
https://stackoverflow.com/questions/11878947/auto-login-remote-site-inner-in-iframe

@ioloc you mean in the approach mentioned here ?
Here we are using form target attr to show the result in the iframe. Example
We could also provide an interface to communicate with the iframe using postMessage, could be useful if the dev has access to the page within the iframe

@ioloc @rishabhsaxena thanks for contributing here with these great points.
To summarise, we should have access to an

onURLChange action
onMessageReceived action
message property (object)
internal properties for the iframe
URL, hash, query params, message

Do we need to do something to allow downloads from inside an iframe?
Feature to scroll inside the page and lock a view of the page in the iframe.

The use of JS expression after an iFrame has loaded is limited with many restrictions. I think the intended behaviour to have additional functionality on the iFrame content will be difficult.
Otherwise feature like downloading a file is still going to be handled by the browser and work normally.

Iframes under lists are not scrolling under edit mode. Is that deliberate or by design?

@kamal0808 can you provide some visual / screen recording on this, Not able to understand what the problem is


MessageType prperty incorrectly set after transformatiin in pipeline

The message type property of messages is not being set correctly after transformation in pipeline components.  It is being overwritten by the original message type of the pre-transformation message

The code had a logic error (missing else) which was causing the issue.  This has been fixed.


Attempt travis fix and fix indentation

Coverage remained the same at 66.254% when pulling 489209730d317930f6e5473db2d06351efab099f on impliedfeline:travis-fix into 3a9bc1e2d01ba134c6f4a4f7efb4e9342677c14d on tdd-pingis:master.


EventTarget MaxListenersExceededWarning

Seeing the following in my logs.
date,stream,content
2024/09/18 09:14:59,stdout,(node:1) MaxListenersExceededWarning: Possible EventTarget memory leak detected. 11 abort listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit
Can we set the default a little higher? It defaults to 10 but my setup is a little larger.

I noticed the same. This seems to be because of a problem in Node's native fetch-api implementation undici: https://github.com/nodejs/undici/issues/1711
If you are using the Docker container, then you have an old version. I'll update Node to latest LTS version, let's see if the problem goes away with that. Otherwise, this could be a subtle bug in the cancellation/retry code.

Version 1.2.1 released!

Unfortunately I am still seeing the issue in my logs.

Thanks for reporting this. Back to the drawing board then!
This might be related to for example long polling BluOS players. However, before increasing the amount of maximum listeners I'd like to make sure that the current request cancellation implementation or the fetch api wrapper library do not leak listeners, because the error message would be exactly the same.

Absolutely, take your time. Makes sense to find the root cause and not mask real errors. =)

I released a new version v1.3.0 with a bugfix for cancellation using AbortSignal. Can you try that @zquestz? You can also try running with DEBUG_MAX_EVENT_LISTENERS=<number> env variable but I don't recommend changing the value if absolutely not necessary as the default value should work ok.

I have updated to the latest release and will get back to you. 😀

Unfortunately I am seeing the log warnings with the latest release. =\
Should I try updating that ENV var?

Should I try updating that ENV var?

Yes, that'd be great!

@zquestz I noted that there's something wrong with scrobbling in the latest release. I'll release a patch release once I've had chance to fix.

v1.3.1 fixes accidentally broken long polling. Sorry about that. Please provide any additional information that you can find @zquestz, this is a difficult issue to reproduce with my setup.

Still seeing the exact same error:
(node:1) MaxListenersExceededWarning: Possible EventTarget memory leak detected. 11 abort listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit

Odd thing is that I have DEBUG_MAX_EVENT_LISTENERS set to 30!

Still seeing the exact same error:
Odd thing is that I have DEBUG_MAX_EVENT_LISTENERS set to 30!

This is really weird. Practically it would mean that event listeners are leaking that would eventually cause the out of memory killer to stop the container so you would see crashing. I could try to run a profiler to see how the memory footprint develops over time.
If you are not seeing a memory leak, it means that Node's undici still produces this warning for some reason but it is harmless. I guess you have double checked already that you have the latest container?
One could also switch away from using fetch API, but as it is becoming the standard in Node it does not seem worth it.

You were totally right, updating the image on my Synology and restarting the container was not enough to get the new version. It is now updated and testing continues. =)

All appears to be working fantastic now. =)

You were totally right, updating the image on my Synology and restarting the container was not enough to get the new version. It is now updated and testing continues. =)

Great to hear that it works now! I have run into something similar when updating images locally. Cache invalidation, the hardest of problems. 😂 The new releases are tagged with both the version number and "latest".
Let's keep this issue still open for a little while if something still comes up related to this.

Everything seems to be working, so I'm closing this issue for now. Thank you for helping to make the project better! 🙏

My pleasure. Thanks for all the help!


Several improvements for handling whitespace including newlines

Codecov Report

Merging #199 into master will decrease coverage by 0.04%.
The diff coverage is 92.3%.

@@            Coverage Diff            @@
##           master    #199      +/-   ##
=========================================
- Coverage   93.55%   93.5%   -0.05%
=========================================
  Files           5       5
  Lines        2158    2157       -1
=========================================
- Hits         2019    2017       -2
- Misses        139     140       +1

Impacted Files
Coverage Δ

scour/scour.py
93.93% <92.3%> (-0.06%)
:arrow_down:

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 06ea23d...e1c2699. Read the comment docs.


Increment version for translation releases

Increment package version after release of Azure.AI.Translation.Document

/check-enforcer override


Add "nvidia GLCache" and "Microsoft Visual Studio Code"

This should work as documented here, but unfortunately it does not work on my machine:
https://download.nvidia.com/XFree86/Linux-x86_64/460.67/README/openglenvvariables.html

Thanks!


Add linter config

Technical changes

Add eslint and prettify configurations.

Non-technical changes

Update README.md

Good idea 👍
Maybe use standardjs instead to be consistent with current style?


Allow ETH transfer on the Sent Asset modal

Description
Added the ability to sent native ETH using the Send Assets Modal.
This incorporates (https://github.com/decent-dao/fractal-interface/issues/754) with the addition of the input component BigNumberInput. Upon this PRs testing and completion, this component can be use throughout the rest app where other big numbers are used.
Notes
Disable USD switch and created issue to re-enable with correct functionality.
No toast / notification on transaction submit, created issue to add.
Issue / Notion doc (if applicable)
https://github.com/decent-dao/fractal-interface/issues/810
Testing
From the Treasury page. Insure DAO has ETH in the treasury and the current user is a delegate or owner.
Open the Send Assets modal
Test all possible combination of input on the token amount input component, i.e. leading zeros, non number characters, copy / paste, etc.
Create a proposal to send native ETH using the Send Assets modal, and verify the ETH is transfer once the proposal is executed.
Screenshots (if applicable)

@mobiusTripper-crypto the USD slider no longer works, it's permanently disabled now for some reason

Yes, I think it needs to switch between the token amount and the USD value. the way it works now it just confusing (and I think doesn't actually work). I am creating a follow up issue.


remove useless layout_constraintHorizontal_bias

I am closing this issue/PR, as it has been migrated to the new repo linked above in the comments. Thank you!


Some Recipes are messed up or still missing (Upgrades not craftable and smeltery recipes messed up)

Based on JEI "Reinforced Metal" and "Enhanced Galgadorian Metal" don't seem craftable in the current version of Steve's Carts. Or is there some "device" needed which is not displayed in JEI?

Just figured out what went wrong. A lot of items return "Refined Handle" when being smelted... including "Large Lump of Galgador" (supposed to yield "Enhanced Galgadorian Metal" and "Stabilized Metal" (supposed to yield "Reinforced Metal")

Upgrades don't have crafting recipes either yet.
Can provide MineTweaker/CraftTweaker script to fix those issues if wanted.

Yeah, looking at JEI it seems that everything in the mod smelts to a refined handle.

Humm that's strange, I will take a look.

For people who want a quick hotfix like me:
This CraftTweaker script fixes the furnace recipes for Steve's Carts so at least that part is (temporarily) fixed. Extension needs to be changed back to .zs for it to work though.
StevesCarts (Furnace Fix).zs.txt

This issue is fixed with the last version.


Updated old link to technet to docs.microsoft.com

Changed to https://docs.microsoft.com/en-us/system-center/index

@ramseygr  Can you change the proposed link to https://docs.microsoft.com/system-center.  We don't want to include the en-us so it auto resolves to the correct locale. Then, I'll approve. Thanks!


Add "Exit Review Mode" command

I didn't have the PR overview page and wanted to exit the review mode so I could get the list of PRs back and didn't see this there but it does exist in the over page. Maybe we should add the command to the command palette?

The language of "Exit Review Mode" has change to " Checkout  Branch", so I've added the command with the new language.


Automated cherry pick of #7912: fix: 支持自动根据云上项目修改本地项目名称

Cherry pick of #7912 on release/3.3.
#7912: fix: 支持自动根据云上项目修改本地项目名称

Codecov Report

Merging #7914 into release/3.3 will decrease coverage by 2.41%.
The diff coverage is 0.31%.

@@              Coverage Diff               @@
##           release/3.3   #7914      +/-   ##
==============================================
- Coverage         6.78%   4.37%   -2.42%
==============================================
  Files              676     281     -395
  Lines           118955   66268   -52687
==============================================
- Hits              8076    2901    -5175
+ Misses          110267   63171   -47096
+ Partials           612     196     -416

Flag
Coverage Δ

#aFlag
4.37% <0.31%> (-2.42%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cloudcommon/notifyclient/notify.go
2.58% <0.00%> (ø)

pkg/compute/models/buckets.go
1.14% <0.00%> (-0.03%)
:arrow_down:

pkg/compute/models/capabilities.go
0.00% <0.00%> (ø)

pkg/compute/models/cloudproviderregions.go
0.26% <0.00%> (ø)

pkg/compute/models/cloudproviders.go
0.87% <0.00%> (+0.01%)
:arrow_up:

pkg/compute/models/cloudregions.go
1.58% <0.00%> (-0.05%)
:arrow_down:

pkg/compute/models/cloudsync.go
0.00% <0.00%> (ø)

pkg/compute/models/dbinstance_backups.go
2.87% <0.00%> (-0.08%)
:arrow_down:

pkg/compute/models/dbinstance_skus.go
2.79% <0.00%> (+0.18%)
:arrow_up:

pkg/compute/models/dbinstancenetworks.go
0.80% <0.00%> (-0.09%)
:arrow_down:

... and 437 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 12aef33...ecade9a. Read the comment docs.


Zwave garagedoor

Description:
Support for Zwave garage door component.
Bugfixes for rollershutter component.
Change the way components are discovered to provide better filtering options for future components.
Pepper1 database has been used for reference.
Thanks to @emilhetty for providing info and testing garage door component and discovery.
Thanks to @wokar for testing and providing bugfix for the rollershutter component.
Related issue (if applicable): fixes #
Checklist:
If code communicates with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

@devdelay Didn't you have Zwave locks? If so can you test this code that they are still discovered correctly?

The code looks fine to me. I can't test it however. So if you have tested this with two people I'm fine to merge it when you think it's ready.

I have had @emilhetty and @wokar test for me. I just want someone with locks to confirm that all is well.

@emilhetty and @wokar has confirmed working setups. My setup works as expected too, so it is good to go. :dolphin:

Everything looks good to me with the locks and were discovered correctly 🐬. I wish I had a zwave garage door opener to test with. I returned one that used COMMAND_CLASS_BARRIER_OPERATOR which is now in dev branch of OpenZwave as of May 13.

Spoke too soon... getting errors when zwave detects my lights
16-06-24 00:04:33 openzwave: Error in manager callback
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/openzwave-0.3.1-py3.5.egg/openzwave/network.py", line 948, in zwcallback
    self._handle_value_added(args)
  File "/usr/local/lib/python3.5/dist-packages/openzwave-0.3.1-py3.5.egg/openzwave/network.py", line 1477, in _handle_value_added
    'value' : self.nodes[args['nodeId']].values[args['valueId']['id']]})
  File "/home/base/.local/lib/python3.5/site-packages/pydispatch/dispatcher.py", line 338, in send
    **named
  File "/home/base/.local/lib/python3.5/site-packages/pydispatch/robustapply.py", line 55, in robustApply
    return receiver(*arguments, **named)
  File "/home/base/home-assistant/homeassistant/components/zwave.py", line 299, in value_added
    if value.command_class not in command_class and \
TypeError: argument of type 'int' is not utterable

Reverted to https://github.com/home-assistant/home-assistant/pull/2271 and my lights came back

@devdelay Fixed the issue you reported. Please test latest :)

ok now 🐬

Then we are good to go. All components tested of zwave😆 @balloob 🐬
24. jun. 2016 07:00 skrev "devdelay"<EMAIL_ADDRESS>
ok now 🐬
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2361#issuecomment-228258462,
or mute the thread
https://github.com/notifications/unsubscribe/AP2koGW00tDZkf17O2MIQ9b60Yws9jpYks5qO2RZgaJpZM4I82qD
.

Second data point, we are :dolphin: on the lights.

Found some problems in the zwave garage door component so it was not reporting status when manually operated with a remote control.
@turbokongen commited my code to you, have a look, you might want to change something.

Looks alright, does it report correctly from both actions, pressing button in UI and manually?

I am pretty sure this: https://github.com/turbokongen/home-assistant/pull/7/files#diff-67a4c075e70d43434b8199c2ece38763R61
is not good. It means it will always report the opposite of what self._state is.

Yeap, did a hard test on it now to make sure that it report right status for both of them.
I thought you would say that, bu without "not" it indicates wrong status, closed when open and open when closed. Do you have any idea of how to invert it?

Does On report as closed and Off report as open?
But the action for on is open and off is closed?

Not sure, how could I see that it reports on or off?

Switch position ;)

When you press open, the garage door opens? But it eventually reports it as closed?

The switch position follows the open and closed.
Action for open is on and off for closed.
All the actions and status match each other, just my port that don't match before I inverted it.

Ok so without the not statement it show correct? :S

No, with it shows correct according to my port

Sorry, starting to get tired.
So what you submitted everything is as expected? :)

Please try my latest at the same branch. I have made some changes regarding another issue, it may be the same problem.

Yes, with the code I commited to you everything is as expected :)

Does the zwave switch component have impact on the garage door? Or could I use the garage door only?

Is this correct?:

It seems strange since it is garage door and not switch, or is it for the switch in UI?

You can use just the garage door.
But I committed new code just now, it should be self._value.value.data

Lost feedback on the latest and got this error:

Should actually be self._value.data
Now I'm overdue. Zzzz
24. jun. 2016 23:37 skrev "Emil Horpen Hetty"<EMAIL_ADDRESS>
Lost feedback on the latest and got this error:
[image: image]
https://cloud.githubusercontent.com/assets/7728206/16351243/db6470ac-3a6c-11e6-8301-b204e9eadcf7.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2361#issuecomment-228468031,
or mute the thread
https://github.com/notifications/unsubscribe/AP2koMgD8GDoEVANWNdZzsLGnxRdY976ks5qPE3xgaJpZM4I82qD
.

Removed one value so it is value.data and not value.value.data and it did work as before, but still my port is inverted HA.

Ok. Then the not is needed. Sorry for the hassle.
24. jun. 2016 23:48 skrev "Emil Horpen Hetty"<EMAIL_ADDRESS>
Removed one value so it is value.data and not value.value.data and it did
work as before, but still my port is inverted HA.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2361#issuecomment-228470274,
or mute the thread
https://github.com/notifications/unsubscribe/AP2koKS1iFkiVm0TjP2PHX5-0K0Cmiboks5qPFC8gaJpZM4I82qD
.

But when I look at this when I tested some days ago:
https://github.com/home-assistant/home-assistant/issues/2288#issuecomment-227572992
The status was correct when I was running as a switch component, and it worked as it should also.
Is it much difference between switch and garage_door int.py?

You are totally correct. The switch state is reported as is_on but the garage door component is reported as is_closed. So the not is absolutely right.

Great, then we leave the "not" in there?
Added it and tested now and it seems to work as previous code.
Will do some more testing tomorrow to make sure it works as it should on everything this time. My neighbors might be wondering why I'm running my port up and down in the middle of the night now :)

Did some testing today on my garage door and the latest code with "not" are working as expected for my door.
Got some Aeotec multisensor 6 today and found that the PIR is reported as Command class sensor binary with generic command class multilevel sensor and it did not appear since binary sensor only has binary sensor as generic class:
('binary_sensor',      [GENERIC_COMMAND_CLASS_BINARY_SENSOR],      [SPECIFIC_DEVICE_CLASS_WHATEVER],      [COMMAND_CLASS_SENSOR_BINARY],      TYPE_BOOL,      GENRE_USER),
GENERIC_COMMAND_CLASS_MULTILEVEL_SENSOR should be added for binary sensor.
Here is the xml if you would like to look at it.
Aeotec multisensor 6.docx

Submitted a fix :+1:  Thank you for noticing this.

Was gd00z-4 support added in this CL?

@dtraub no, command class barrier operator is not supported yet in current version of openzwave

Bummer, I was really hoping to get this working with HA.
Thanks for the response!

@devdelay @dtraub They have rudimentary support for Barrier Operator in the dev branch now. See: https://github.com/OpenZWave/open-zwave/issues/490#issuecomment-218787953
I'm able to patch OZW master with that change and control the Linear GDZ00-4 directly with it. Would it make sense to add support to HA for an unreleased/possibly-not-final OZW implementation? Or should we wait for a proper release? I am new to python/HA but happy to take this on if some folks can point me in the right direction. I just need a couple days to order a second Gen5 stick I can use for development.

@freerobby Doesn't it appear in the HASS gui? I know it won't work but it should appear as a garage door opener, or at least show that it didn't in the HASS log. What commands are  sent while controlling it? I guess commands in the Barrier operator class are used. Have you got a zwcfg_xxxxx.xml  with the command class present?

@turbokongen Let me look into this tonight against the latest dev branch. I had been working on this in parallel with your change to allow secure devices and add garage door support, and I was working off of the 0.23 release, so it showed up in HASS GUI as a bunch of sensors, but not as a garage door opener for me. I will try removing and re-adding it to my zwave network on 0.24.1, hopefully tonight, and report back with findings and logs.

A head start for you:
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/zwave.py#L171

@turbokongen I'm now having trouble, on 0.24.1, getting it to show up in the HASS GUI. I was able to see the sensors when I added it on 0.23 with a manual patch to add things in secure mode. Now when I add them to the network, they are added in non-secure mode and do not show up in HASS. Logs below if you have any thoughts.
https://gist.github.com/freerobby/a4d769b15f795db5878c2e52ef813a77


Recomendation for AV exclusions also apply for EDR?

Next to default AV solutions companies also add Endpoint Detection & Response (EDR) solutions like Crowdstrike and Cynet to their environment. Would the recommended AV exclusions also apply to EDR solutions?

Short answer, yes!


Modify Profile Test to Fix Intermittent Failures

Changed the profile test to use waitFor() instead of an await
on findByText().
Occasionally, certain conditions caused findByText() to throw an error,
likely due to the long load times for the imported controls in the
profile viewer. The errors caused the tests to fail even though the
code should pass.
This change should resolve these errors as waitFor() retries getByText()
until it does not throw an error, more resiliently handling the
conditions that caused findByText() to throw errors.

Might need some help testing this one - this change seems to work but I may just be getting really lucky (unlucky?) with the tests passing all the time. Before the fix, the test giving a false negative was pretty rare, so if anyone sees that behavior occur again even with this change please drop a comment in this PR.

I do not see any behavior occur during my testing. I think this should be good to go.

Turned up some other weird issues while testing; seems that the explicit timeout of 10000 ms doesn't actually do anything. A failing test will now return this output:

This is because jest sets a timeout for every test that takes precedence over the timeout field of waitFor()
As the error message says this can be remedied by using jest.setTimeout() to make that timeout higher than 10000ms, which gives this error instead:

I believe the second error message is preferable on a failed test because it's a bit more descriptive of the actual error.
For now I'm going to reduce the explicit timeouts to 5000ms and call jest.setTimout(6000)because that produces the more descriptive error and 5000ms seems to be enough time anyway. But if there's a better way of doing this, or a more preferable timeout, please let me know.
This may also be something to look into if it affects the other tests as well.

I noticed that you didn't use waitFor on the other tests in the profile test file. Why not use waitFor for findByText and findAllByText as well?
When testing the code for this branch, I did not see any profile test failures, but still am curious as to the above question.

I noticed that you didn't use waitFor on the other tests in the profile test file. Why not use waitFor for findByText and findAllByText as well?
When testing the code for this branch, I did not see any profile test failures, but still am curious as to the above question.

Good point. I only changed what was needed to fix the tests, but it would be better if the whole test was consistent. I'll make that change now.


Reporting on NOPKG and MobileConfig Profiles

Feature Request - would love to be able to pull a report of NOPKG install status and MobileConfig Profile install status.  Thanks!

This will need a rewrite of the munkireport module, so the installs are in a separate table.

There is a profile module available that reports on profiles.

I've looked at the profiles report, but I don't seem to get accurate data out of it.  Not sure why or what the common denominator is there.

If the profile module/report is not accurate, please file a separate issue.

Should be addressed in the managed installs module


[Typechecker] Covariant subscript check must also take into consideration mismatched types

When we override a settable subscript with a different argument type, the compiler emits an error complaining that the types are not covariant:
class A {
  subscript (foo: Int) -> Int {
    get { fatalError() }
    set {}
  }
}
class B: A {
  override subscript (foo: String) -> Int {
    get { fatalError() }
    set {}
  }
}

The compiler currently emits an error saying cannot override mutable subscript of type '(Int) -> Int' with covariant type '(String) -> Int', but the argument types (Int and String) are unrelated.
We should make sure that we don't have mismatched types, so we can print the right diagnostics.
Resolves SR-10323

@swift-ci please smoke test


[Event Request] codeunit 90 "Purch.-Post" - OnPostItemJnlLineOnAfterSetCheckApplToItemEntry()

Describe the request
Wrap with handled pattern until the end of with ItemJnlLine do begin
        IsHandled := false;
        OnPostItemJnlLineOnAfterSetCheckApplToItemEntry(PurchLine, IsHandled); //CustPub
        if not IsHandled then begin
            PostWhseJnlLine := ShouldPostWhseJnlLine(PurchLine, ItemJnlLine, TempWhseJnlLine);

[IntegrationEvent(false, false)]
local procedure OnPostItemJnlLineOnAfterSetCheckApplToItemEntry(PurchLine: Record "Purchase Line"; var IsHandled: Boolean) //CustPub
begin
end;

Additional context
Publisher request marked with //CustPub tag.Internal work item: AB#494260

Hello @StefanSosic
Where is the END of this "if not IsHandled then begin"  ... block?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14917.


feature: Faster CI Builds

This change decreases both the size of the builder image and the time it
takes to create the builder image. The current yarn install step installs both
development and production dependencies at the same time to leverage
the shared yarn cache on that layer. Now that the yarn cache now lives in an
external docker build cache we can share it between images.
The decrease in size will make the five test tasks that run within the
builder image boot faster.
The decrease in time will make PR builds (our most common build
flow) faster.
There is a minimal increase in production image build time.
The following statistics are from my local machine and as such any
comparisons should by the percentage of total time only:
Current     Time     Size
builder     5m 54s   1.47GB
electron       56s   1.26GB
production     13s    388MB

New         Time    Size
builder     4m 43s  1.29GB
electron       58s  1.26GB
production     29s   388MB

Now that the yarn cache now lives in an
external docker build cache we can share it between images.

Nice pattern!

Now that the yarn cache now lives in an
external docker build cache we can share it between images.

Nice pattern!


Rework the definition process for value IDs

Some thoughts:

There should be a central property of each CC that provides named access to all defined value IDs (or functions that return a value ID) as well as metadata
If possible, this property should be used in the decorators (think about this)
It should be possible to define if this value is always accessible (e.g. writeonly values #532) and/or private
This should replace or improve registerValue.

What about using msg-pack-lite to decode the valueId json and encode it back to valueId? We could use it's encoding as unique Id. Also we could provide  a public getId(ValueID valueId) : String method to calcolate a the valueId unique id

Based on our slack conversation:

Is  there a file where I can find all properties/ propertiesKeys of all command classes?

We could create a file with all properties/properties keys that actually are hardcoded in protected [SET_VALUE]:
Like https://github.com/zwave-js/node-zwave-js/blob/master/packages/zwave-js/src/lib/commandclass/MultilevelSwitchCC.ts#L249

As discussed in Discord, this could be used to auto generate CC documentation. Once you get there, please include whether the CC has stateless values


fix(rules): Fixed an issue where contains would not work as expected …

…when comparing 2 lists

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


Hopper Filters do not work for items with durability

Setting a basic inventory hopper to filter items with durability (such as tools and weapons) will only filter items with that specific durability.

I think it's a problem with NBT tags, I wish there was a filter for "Don't specify item damage"


Adding Dreary Theme

This is way too messy at that point, considering just making another repository entirely instead of squashing

This is way too messy at that point, considering just making another repository entirely instead of squashing

I would suggest so. You can do your own repository with this theme and I'll add a link in the README to that.

@morpheusthewhite Wouldn't it be easier to just fork the spicetify-themes repository, and just drag the Dreary folder in? Then i can just try and keep my commits organized.

If you push only on the Dready folder then it's not really useful to keep other themes since everyone will just use that repository for your theme.

@morpheusthewhite So should ai close my other pull request and create a repository just for dreary? And what should I do if I plan on making more themes?

@morpheusthewhite So should ai close my other pull request and create a repository just for dreary?

I would say so.

And what should I do if I plan on making more themes?

Open a PR here.


False positive when setting a tablename value in a sqlalchemy model

 Environment data

Language Server version: 2021.11.2
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda): 3.10
python.analysis.indexing: undefined
python.analysis.typeCheckingMode: basic

 Expected behaviour
class Test(Base):
    __tablename__ = 'test' # << This code has no any problems

 Actual behaviour
class Test(Base):
    __tablename__ = 'test' # << Expression of type "Literal['test']" cannot be assigned to declared type "declared_attr" "Literal['test']" is incompatible with "declared_attr"

 Logs

Python Language Server Log
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
Background analysis message: analyze
[BG(1)] analyzing: /workspace/myproject/core/test/models.py ...
[BG(1)]   parsing: /workspace/myproject/core/test/models.py (5ms)
[BG(1)]   binding: /workspace/myproject/core/test/models.py (0ms)
[BG(1)]   checking: /workspace/myproject/core/test/models.py (74ms)
[BG(1)] analyzing: /workspace/myproject/core/test/models.py (80ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: getSemanticTokens delta
[BG(1)] getSemanticTokens delta previousResultId:1638430169428 at /workspace/myproject/core/test/models.py (9ms)
Background analysis message: resumeAnalysis
[BG(1)] indexing: /workspace/myproject/core/test/models.py [found 8] (0ms)
Indexing Done: /workspace/myproject/core/test/models.py
Background analysis message: getDiagnosticsForRange
[FG] parsing: /workspace/myproject/core/test/models.py (1ms)
[FG] binding: /workspace/myproject/core/test/models.py (0ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange

Could you please provide a complete, stand-alone sample? Where does the symbol Base come from? I don't see such a symbol exported from the sqlalchemy module.

Thanks for the waiting, I've a custom Base
from sqlalchemy.ext.declarative import declarative_base, declared_attr

class Base(object):
    @declared_attr
    def __tablename__(cls):
        return cls.__name__.lower()

Base = declarative_base(cls=Base)

Having the same issue. @deterok have you found a workaround?

By default, type checking is disabled in pylance. I presume that you have enabled type checking.
This isn't a false positive, at least from the perspective of the type checker. The base class declares the class variable __tablename__ as having the type declared_attr, but you're attempting to assign a str to it within the subclass. Pylance is correct in pointing out the type violation.
This is analogous to:
class Parent:
    foo: int

class Child(Parent):
    foo = "not an int" # Type violation error

I don't know enough about sqlalchemy to offer a suggested workaround. You might want to ask the maintainers of sqlalchemy how this is intended to be used.

Closing inactive issue


feat:Optimize retry logic v1.0

What type of PR is this?
optimize
What this PR does / why we need it (English/Chinese):
en:
zh: 优化 client 重试策略

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wyfwangyafeng_i seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Project description

Update the project's description on GitHub

Project description has been updated following our Wiki and might be updated according its changes.


Use IsDefined to check for IgnoreAttribute

See #558

Ah, interesting! I never would have guessed people were using inherited attributes. Thanks for the fix!

It is not a very common use case for normal use, indeed.
It is mostly commonly used with custom property behaviour and extension frameworks. (Ex: https://bitbucket.org/twincoders/sqlite-net-extensions )
To implement their extension code for only certain attributes, the libraries inherit from the ignore attribute to make sure the default sqlite-net code isnt executed for those properties.


Lag spikes in editor

Expected Behavior
It should run smooth
Actual Behavior
slight lag spike every few seconds
Steps to Reproduce
place 1 block down in the editor and move the code area around and look closely and compare it to how it runs on turbowarp.org
Operating System and Browser
Windows 11, Microsoft Edge

seems fine to me?

do you have any browser extensions or other apps open while using the editor

yeah i agree with charkpole ive never gotten any lag from the editor. what are your specs? like cpu, gpu, etc

The only time youd get a lag spike is by switching sprites/tabs. And maybe if your running a lot of code. Some more information/video showing this would be helpful

the editor makes sense to run the slowest since alot of what it does (or atleast is supposed to do) when running is checking debug information
also because blockly is piec-
it spiking is weirc tho... do you happen to be running on a device that has gardio (or something akin to it) installed?

these are my extensions

PuzzlingGGG these are my specs
Processor	Intel(R) Core(TM) i5-3470 CPU @ 3.20GHz   3.20 GHz
Installed RAM	8.00 GB (7.88 GB usable)
System type	64-bit operating system, x64-based processor

nvm yeh extensions was the problem I think

im probably gonna start suspecting web extensions more when it comes to lag, i already have a VR emulator chrome extension for testing things and it started lagging the editor constantly until i found it was the chrome extension


Add RuTube LiST support

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This pull request adds support for RuTube LiST videos that can only be downloaded when logged in. This closes #24018.
The original research was done by @Gregory-Gerard, but I ported his code to Python, incorporated it into rutube extractor and tried to make it readable and reliable enough.
Both specifying --username/--password and using a cookie jar works.

Is this still valid?

It's been almost 2 years since I posted this and I've never tested it with newer versions of youtube-dl. And frankly, I don't really use this video hosting anymore.
So I think it's okay to just close this if nobody else is interested.


helm: added clusterIP to grafana service

service.clusterIP already existed in values.yaml but was no used in the service definition

Sorry for the inconvenience, but the prometheus-operator and kube-prometheus helm charts are currently being migrated to the upstream stable helm charts. Because of this we are currently not accepting any pull requests. We apologize for the inconvenience, but this is necessary for the migration to actually happen. We announced this here: https://github.com/coreos/prometheus-operator/tree/master/helm#warning
You can view the progress and help here and here.
Thank you for your understanding! :slightly_smiling_face:


[vm_service] IsolateReload event not sent after reloadSources call

Tested with:
Dart SDK version: 2.17.6 (stable) (Tue Jul 12 12:54:37 2022 +0200) on "windows_x64"
Dart SDK version: 2.19.0-5.0.dev (dev) (Sat Jul 16 03:26:30 2022 -0700) on "windows_x64"

With:
vm_service: 9.0.0

In an attempt to answer the following Stackoverflow question:
https://stackoverflow.com/questions/73046539/how-can-one-respond-to-a-hot-reload
I got into trying to use the vm_service package to receive events about isolates being reloaded (hot reloaded?). But it does not seem like IsolateReload (or any Isolate event) are being sent when doing reloadSources.
I also checked the web traffic to Dart Observatory and Dart DevTools (both also subscribes to the EventStreams.kIsolate but they seems to also not getting any event for reloading isolates.
But the code (and therefore the Isolate) are being reloaded since I can verify changes to the someMethod() are being loaded every 5 second.
Program for reproduce the issue:
import 'dart:async';
import 'dart:developer';
import 'dart:isolate';

import 'package:vm_service/vm_service.dart' as vm;
import 'package:vm_service/vm_service_io.dart' as vmi;

void main() async {
  final serverWebSocketUriString =
      (await Service.getInfo()).serverWebSocketUri.toString();
  final vmService = await vmi.vmServiceConnectUri(serverWebSocketUriString);

  // Documented to subscribe to:
  //   IsolateStart, IsolateRunnable, IsolateExit,
  //   IsolateUpdate, IsolateReload, ServiceExtensionAdded
  vmService.onIsolateEvent.listen((event) {
    print('onIsolateEvent: ${event}'); // <-- never called
  });

  final status = await vmService.streamListen(vm.EventStreams.kIsolate);
  print(status.json); // {type: Success}

  final isolateId = Service.getIsolateID(Isolate.current)!;
  startHotReloadTimer(vmService, isolateId);
}

Timer startHotReloadTimer(vm.VmService vmService, String isolateId) =>
    Timer.periodic(const Duration(seconds: 5), (_) async {
      final response = await vmService.reloadSources(isolateId, force: true);
      // Same issue when doing:
      // final response = await vmService.reloadSources(isolateId);

      print('response: $response'); // response: [ReloadReport success: true]
      someMethod();
    });

void someMethod() {
  print('String I manipulate manually to verify the code have been reloaded');
}

I have tried executing the program with the following different ways without any observed difference:
dart --enable-vm-service program.dart
dart run --enable-vm-service program.dart
dart --observe program.dart
dart run --observe program.dart

//cc @bkonyi

I can confirm  this. Workaround: https://pub.dev/packages/hotreloader
Hot reload is one of the best features of dart, so it's weird to see it so neglected. I'm writing a script that parses dart code into a syntax tree and then generated some stuff from it.
It takes few seconds to run the script and parse the tree.


[Feature Request] Use a POI to start or end a route

It would be great to provide the user a way to use a POI as a start or end point of a route.
New options in the the POI popup:

"Add as new start point"
"Add as new end point"

(I would also like to see the option to add a POI as an intermediate waypoint. But this might be a bit more complicated to decide, between which existing waypoint it needs to be placed.)

This is an interesting proposition, but I feel like it would add a lot of complexity to the POI popup while not bringing so much value since you can simply manually click at the same location.


fix: proposer_index rustdoc

proposer_index is not "The execution block number of the parent block".

failures unrelated


BEiT v2 Semantic Segmentation  BEiT: __init__() got an unexpected keyword argument 'rel_pos_bias_interpolation_type'

Describe the bug
Model I am using (UniLM, MiniLM, LayoutLM ...): beit2
The problem arises when using:
the official example scripts: (give details below)
./tools/dist_train.sh \
configs/beit/upernet/upernet_beit_base_12_512_slide_160k_21ktoade20k.py 4 \
--work-dir outputs --seed 123 \
--options model.pretrained=https://conversationhub.blob.core.windows.net/beit-share-public/beitv2/beitv2_base_patch16_224_pt1k_ft21k.pth

When I run the training script, it show:
Traceback (most recent call last):
  File "./tools/train.py", line 167, in <module>
    main()
  File "./tools/train.py", line 134, in main
    model = build_segmentor(
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmseg/models/builder.py", line 66, in build_segmentor
    return build(cfg, SEGMENTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmseg/models/builder.py", line 33, in build
    return build_from_cfg(cfg, registry, default_args)
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmcv/utils/registry.py", line 182, in build_from_cfg
    raise type(e)(f'{obj_cls.__name__}: {e}')
TypeError: EncoderDecoder: BEiT: __init__() got an unexpected keyword argument 'rel_pos_bias_interpolation_type'
Traceback (most recent call last):
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmcv/utils/registry.py", line 179, in build_from_cfg
    return obj_cls(**args)
TypeError: __init__() got an unexpected keyword argument 'rel_pos_bias_interpolation_type'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmcv/utils/registry.py", line 179, in build_from_cfg
    return obj_cls(**args)
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmseg/models/segmentors/encoder_decoder.py", line 30, in __init__
    self.backbone = builder.build_backbone(backbone)
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmseg/models/builder.py", line 38, in build_backbone
    return build(cfg, BACKBONES)
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmseg/models/builder.py", line 33, in build
    return build_from_cfg(cfg, registry, default_args)
  File "/root/opt/miniconda3/envs/torch_lts_1.8.2/lib/python3.8/site-packages/mmcv/utils/registry.py", line 182, in build_from_cfg
    raise type(e)(f'{obj_cls.__name__}: {e}')
TypeError: BEiT: __init__() got an unexpected keyword argument 'rel_pos_bias_interpolation_type'

To Reproduce
Steps to reproduce the behavior:

Run the above script

Expected behavior
The training starts

Platform: Ubuntu 16.04.7 LTS
Python version: 3.8.13
PyTorch version (GPU?): 1.8.2  cuda 10.2
mmcv-full 1.3.0
mmsegmentation 1.11.0
apex 0.1

Thank for your attention~
You could remove rel_pos_bias_interpolation_type=0 in the config file configs/beit/upernet/upernet_beit_base_12_512_slide_160k_21ktoade20k.py.

Thank for your attention~
You could remove rel_pos_bias_interpolation_type=0 in the config file configs/beit/upernet/upernet_beit_base_12_512_slide_160k_21ktoade20k.py.

Thank for your relpy~
I removed that and started training. But I still have some trouble to reproduce the results on ADE20k.
The result I got was:
+--------+------+-------+-------+
| Scope  | mIoU | mAcc  | aAcc  |
+--------+------+-------+-------+
| global | 52.7 | 63.97 | 84.39 |
+--------+------+-------+-------+

The mIoU is 4.84 lower than your results (57.54). I only change two settings:

Set the samples_per_gpu to 4 to ensure same efficient batch size on 4 GPU.
Remove --seed 0  --deterministic for fast traning.

20220913_141418.log
20220913_141418.log.json.txt

Hello,
We report 57.54% mIoU by using the larger size, so you can check it again. I will check the hyper-parameters I used. And I will release the training log after I prepare.
Thanks~

I trained it with 8 V100 and --seed 0 --deterministic, but could not reproduce your results either.
20220921_202702.log

I used to train with BEiTv2-base, thereby the 57.54 mIoU can not be reproduced. I train with BEiTv2-large now and get Iter(val) [144000]	mIoU: 0.5761, mAcc: 0.6929, aAcc: 0.8608.
Thanks for your reply~


Release v24.11_REL1.0.9   indentation error

line 352 : indentation error:
https://github.com/ARM-software/arm-systemready/blob/main/common/log_parser/apply_waivers.py
Log:
[   55.338703] init.sh[375]:   Waiver File        : /mnt/acs_tests/config/acs_waiver.json
[   55.697604] init.sh[381]:   File "/usr/bin/log_parser/apply_waivers.py", line 353
[   55.697811] init.sh[381]:     output_json_data = None

Please propose a patch with the fix.

Hi @ricardosalveti and @quic-pansing,
We will be upstreaming the fix for the issue by eod.
Thanks,
Chetan

The changes for the fix is merged in arm-systemready repo: https://github.com/ARM-software/arm-systemready/pull/235


Train model for specific translation like Spanish to Russian...

hi,
I have successfully replicate the translation model for en->fn and en->de, working great. But now want to create costume model like es->ru. I came across one thing that model are dedicate to specific language pair like  En-De (lightconv_wmt_en_de_big) or En-Fr (lightconv_wmt_en_fr_big) or vice-versa.
So if i want es-ru translation model so what should I follow cause I cant able to find any docs, after data processing.
And lightconv have

lightconv
lightconv_iwslt_de_en
lightconv_wmt_en_de
lightconv_wmt_en_de_big
lightconv_wmt_en_fr_big
lightconv_wmt_zh_en_big
lightconv_lm
lightconv_lm_gbw

So dedicated model are optimize for some language pair.
so what model should i use for translation for es->ru

i don't know that any of us have experimented with this language pair.  You'll need to try for yourself and see what works best.

hi @lematt1991,
let's take an example
is there any difference between lightconv_iwslt_de_en, lightconv and lightconv_wmt_en_de in their architecture ??

Is there a standard configuration for language model for transformer_lm_wiki103,the official config can not replicate the result 18.7,but I can use the model to inference,and get the score.And my parameters num is 247.2M, but the offical model is 246.9M.


Create a process for changing the theme in a way that might be breaking

Currently we change things about the default theme rendering of RTD without notifying users. Especially since we try and make it so that users don't really have to interact with RTD, having changes break their stuff is a particularly bad thing to happen.
Process
We aren't going to be able to avoid making changes to our theme and builder code. That said, we should be able to communicate these changes in a good way.
Django and other projects have these in releases with "Backwards Incompatible Changes" lists -- which doesn't make a lot of sense because we don't do formal releases.
I think the proper response is to have a "dev" mailing list where we publish possible breaking changes of the theme or website.
First Steps
We should write up a first draft of documentation around the process that we will follow. After that we need to document where people should go to get notified. We could use our existing "readthedocs" google group.

This is a theme issue, and should be moved there.


Build(deps-dev): Bump css-loader from 6.7.1 to 6.7.3 (#4647)

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

Merged in 64a04bb36dcff3d40e697a5b5c2574b95a13fce7


Add Grapheme Breaking and Normalization

This adds the Unicode data needed and the stdlib implementations of grapheme breaking and normalization for use in _StringProcessing.

@swift-ci please test Linux


Query not executing correctly

First, I wanted to give thanks to you for creating this amazing database!
I'm not sure if this issue has been raised or not, but today the database stopped working correctly for me out of nowhere. It started when I would try and run a query, but nothing I ran gave results anymore. I thought maybe I needed to update the db, but now I get different variations of this:
(<nfldb.types.Player object at 0x02D5B990>, 4878L) no matter what I run. Not sure if I'm missing
(<nfldb.types.Player object at 0x02D5BBD0>, 4793L) something, but I was confused as to what was
(<nfldb.types.Player object at 0x02D5BB70>, 4770L) happening here. Thanks!
(<nfldb.types.Player object at 0x02D5B9F0>, 4671L)
(<nfldb.types.Player object at 0x02D5B6F0>, 4591L)
(<nfldb.types.Player object at 0x02D5B810>, 4447L)
(<nfldb.types.Player object at 0x02D5B750>, 4428L)
(<nfldb.types.Player object at 0x02D5BB10>, 4262L)
(<nfldb.types.Player object at 0x02D5B510>, 4210L)
(<nfldb.types.Player object at 0x02D5B630>, 4166L)

Please provide a program that reproduces the problem. Please also include the actual output of running the program and the expected output.


Fix: remove quote from cron value

Fixes: https://github.com/cockroachdb/helm-charts/issues/192

I remember removing these quotes in one of my PRs. Might be reverted back later in subsequent PRs
https://github.com/cockroachdb/helm-charts/pull/122/files#diff-ff2766895bfdb2dcaa571bcf0647d73f5f82c7fc065aec5548e5c12500d63c5f


How is this different than what's already in Xcode?

How is this different than Xcode's built-in location simulation tool? Can this be run on the iOS device and simulate GPS independent of Xcode?

@pruppert It's not, we just documented it out and added a few new location spots for testing. It's mostly just a documentation piece for new developers.


[Feature] Allow more complexe gestures

Hi, would like to have more customizable / complexe gestures for states.
Problem :
Users are used to the navigation of softwares like photoshop or paint.net and want the same feel (Ctrl + Scroll = zoom, Left Click + Space = Pan, Scroll = Pan vertically, Scroll + Shift = Pan horizontally).
Solution :
Allow more complexe gestures on states :

Mix mouse and keyboard events (for  Left Click + Space)
Separate handling of vertical and horizontal panning

Here is a fork feature/extended-gestures where I did one possible implementation of that, there is still some things that could be checked :

Handling panning with keyboard
Regroup the pan vertical / horizontal gestures like the SelectionGestures

Unrelated in the fork : I also added custom cursors for panning which is unrelated but would be cool

Hi, the following code allows zooming with Ctrl + Scroll and panning with Space + Left Click in the playground app:

Click to expand code block
 public partial class NodifyEditorView : UserControl
 {
     public NodifyEditor EditorInstance => Editor;

     private EditorGestures DefaultEditorGestures { get; set; }

     public NodifyEditorView()
     {
         InitializeComponent();

         DefaultEditorGestures = new EditorGestures();
         DefaultEditorGestures.Editor.ZoomModifierKey = ModifierKeys.Control;

         EditorGestures.Mappings.Apply(DefaultEditorGestures);
     }

     private void Minimap_Zoom(object sender, ZoomEventArgs e)
     {
         EditorInstance.ZoomAtPosition(e.Zoom, e.Location);
     }

     protected override void OnKeyDown(KeyEventArgs e)
     {
         if (e.Key == Key.Space)
         {
             EditorGestures.Mappings.Apply(PanningGestureMappings.Instance);

             Cursor = Cursors.Hand;
         }
     }

     protected override void OnKeyUp(KeyEventArgs e)
     {
         if (e.Key == Key.Space)
         {
             EditorGestures.Mappings.Apply(DefaultEditorGestures);

             Cursor = Cursors.Arrow;
         }
     }
 }

 internal class PanningGestureMappings : EditorGestures
 {
     public static readonly PanningGestureMappings Instance = new PanningGestureMappings();

     public PanningGestureMappings()
     {
         Editor.Pan.Value = new AnyGesture(new MouseGesture(MouseAction.LeftClick));
         Editor.Selection.Apply(SelectionGestures.None);
         Editor.Cutting.Value = MultiGesture.None;
         ItemContainer.Selection.Apply(SelectionGestures.None);
         ItemContainer.Drag.Value = MultiGesture.None;
         Connector.Connect.Value = MultiGesture.None;
     }
 }

Things I need to think about:

allow clicking through ItemContainer and Connector while in panning mode
pan with scroll wheel

Hi, what also would be cool to add touch gestures. So for example zooming with two fingers, pan vertical / horizontal with one finger, select a node with one tap,... I tried to implement that but had troubles with selection and pan and had not looked into that any further

I'm currently working on moving the logic of the editor states inside the editor itself and just call those methods with the appropriate parameters from those states. That would make it much easier to control the editor behavior from different input sources.

I found a better solution and created a PR (#167). Now you can just write the following and it will work just fine:
EditorGestures.Mappings.Editor.Pan.Value = new MouseGesture(MouseAction.LeftClick, Key.Space);

Hi, what also would be cool to add touch gestures. So for example zooming with two fingers, pan vertical / horizontal with one finger, select a node with one tap,... I tried to implement that but had troubles with selection and pan and had not looked into that any further

@jpk6789 With PR #170, it should be easier. Subclass the editor and override OnTouch* methods, then forward the events to the InputProcessor. And implement a custom IInputHandler that can handle these events.
I will eventually add these methods in the base editor. I would also like to add support for touchscreen myself but I don't have a display to test with.

This is really really nice. Thank you for all you effort. I hope, I have some time during the Christmas holidays to look into your code. I will try to implement the touch stuff in a similar way in a PR.
For the touch screen: I have actually bought one of those 15'' mobile USB-C displays with multi-touch last year to have a second screen for my laptop and also to program such touch stuff in DMXControl 3 as well. So I can test that.


Arnej/merge serializers

@baldersheim please review and merge to 7
@bratseth @vekterli please also take a look

@arnej27959 There is a related test failure.


Update maven plugin to 1.2.0

Update maven plugin to 1.2.0

You need to update the archetype as well


Check warning while compiling java

When javac compiles, it returns no output on successful compilation. Use this fact to check for warnings and issues, and fail build if this happens.
Eg: https://github.com/iiitv/algos/issues/274

Working Fine on my machine.

@aviaryan please assign me this one.


README.md contains a broken link

https://github.com/sezna/nps/blob/6eb54ac33b43ac9e2ea55a2cbf1ccb32463a9de8/README.md#L537
The URL on line 537 of the project README is broken. While the resource currently exists on the medium.freecodecamp.org domain, the documentation links to the medium.freecodecamp.com domain. freeCodeCamp may have routed the URL to the correct destination in the past but it currently appears not to.

Thanks! I'll go ahead and fix this.


ClientConfig should support a ProjectID

Recently OpenAI has added the support for Projects, within an Org; and Assistants are now project-scoped.
When creating a new Client, ClientConfig should support a ProjectID field, similar to what the Python API does:
from openai import OpenAI

client = OpenAI(
  organization='org-xG5HLfr7CRdMSiSBlycUpcIw',
  project='$PROJECT_ID',
)

I suspect there may be a workaround currently by adding a custom header:
  -H "OpenAI-Project: $PROJECT_ID"

but this is not documented.
Is your feature request related to a problem? Please describe.
Assistants (and by induction, Threads and Messages) are Project-scoped in the OpenAI API, so not having this limits us to using only the DefaultProject which causes issues of isolation, etc.
Describe the solution you'd like
Add a ProjectID to the ClientConfig struct
Describe alternatives you've considered
Custom OpenAI-Project header
Additional context
I'd be happy to contribute the code to implement this: just say the word!
:heart: go-openai!!!

Actually, while researching this issue, I just discovered that now OpenAI API Keys can be created "project scoped" so this is unnecessary, so long as the user selects a project-specific API key.
This largely solves the issue here, but I still think that, for the sake of parity (and because there may be use cases where a "super-user" API key may be used to manage multiple projects) it would be good to add support for a Project ID.


Getting CWL for oxog workflow with no tag set cannot find CWL file

Workflow: ICGC-TCGA-PanCancer/OxoG-Dockstore-Tools/oxog-preprocessor
I believe this is because if no tag/branch is set we look at master, but in the case of this workflow the master branch is not valid. We should find a better way to deal with get CWL and WDL when no branch/tag has been given.

Grab default branch instead (if exists), else try master.

Related to #585

This is probably fixed, I cannot find the workflow on dockstore prod.

@SolomonShorser-OICR


The Leaderboard submission for motion forecasting dataset.

@benjaminrwilson , The leaderboard submission shows a status of "submitted," but the execution hasn't started, and no results are being generated. Could you please look into this issue?

I'm also facing the same issue. This may require some fixes. @wqi

Should be back up now, thanks for reporting the issue!

The issue remains unchanged @wqi


Twitter compress images when greater than 5MB

Twitter disallows images over 5Mb
From Zach Phillips on Twitter

Audit: Won't do


fetchcargo is non-deterministic

Basic info
To make sure that we are on the same page:

Kernel: (run uname -a) 4.5.0
System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...) 16.09.git.cd86f80 (Flounder)
Nix version: (run nix-env --version) 1.11.2
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion) 16.09.git.cd86f80

Describe your issue here
During testing of https://github.com/NixOS/nixpkgs/pull/14088 I managed to get at least two different hashes for exa. After some discussion with @jgillich it seems that this derivation can produce non-deterministic output.
Expected result
Hashes of fetchcargo outputs are always the same.
Actual result
They are not.
Steps to reproduce
Build exa several times.

How would one work around that? It basically means the system cannot be rebuild :cry:

cc @wizeman

@despairblue You can work around it by updating SHA checksum in the affected package locally.

@abbradar :confused: Is there any documentation you could point me too. I know how I would to it for packages that I install as a user:

checking out nixpkgs repo
find the last commit that build for unstable on hydra
check that out
change the exa's default.nix file
5 run nix-env -f ~/git/nixpkgs -i exa

But I think it's also possible to override it in a nix expression, like the one in /etc/nixos/configuration.nix, since I installed it as a systems package. The thing is that all ressources I find either explain how to do it for .nixpkgs/config.nix or are that much out of date that they link to non existent documentation

I changed my configuration.nix to include that:
  environment.systemPackages = with pkgs; [
    # ...
    (pkgs.lib.overrideDerivation pkgs.exa (attrs: {
      depsSha256 = "1klg78ynqvj5gbs4cqsah4vmdfv8iv2va4fsv41l3g0ssja6z5xm";
    }))
    # ...
  ];

but to no avail :confused:

Sadly we don't have a mechanism to override depsSha256 (it needs to be supported). However, you can rebuild your whole NixOS from a modified repository:
$ nixos-rebuild switch -I nixpkgs=/absolute/path/to/patched/nixpkgs

Another way to do it is to call patched nixpkgs from your configuration.nix:
(import /path/to/nixpkgs {}).exa

Thanks for your help! I really appreciate it.
I tried your last suggestions, but that leads to:
building path(s) ‘/nix/store/2w7vws133m7x7ib31js9ric1gpq5wys7-exa-2016-04-20’
unpacking sources
unpacking source archive /nix/store/884pf1gxkcxiydpkfcjfrwvy3jsxax1l-exa-110a1c716bfc4a7f74f74b3c4f0a881c773fcd06-src
source root is exa-110a1c716bfc4a7f74f74b3c4f0a881c773fcd06-src
Using cargo deps from /nix/store/7h2gz0ghd9xsqjymhp22q893ync0zzaa-exa-2016-04-20-fetch
Using indexHash '-ba82b75dd6681d6f'
Using rust registry from /nix/store/wcl2cfdl2v1hfgmgklqwdh71gpb62dwf-rustRegistry-2016-08-10-fe018be
warning: custom registry support via the `registry.index` configuration is being removed, this functionality will not work in the future
    Updating git repository `https://github.com/rust-datetime/zoneinfo-compiled.git`
warning: spurious network error (2 tries remaining): [12/-1] curl error: Couldn't resolve host 'github.com'

warning: spurious network error (1 tries remaining): [12/-1] curl error: Couldn't resolve host 'github.com'

error: failed to load source for a dependency on `zoneinfo_compiled`

Caused by:
  Unable to update https://github.com/rust-datetime/zoneinfo-compiled.git#f56921ea

Caused by:
  failed to fetch into /tmp/nix-build-exa-2016-04-20.drv-0/deps/git/db/zoneinfo-compiled-4887a505c85fb388

To learn more, run the command again with --verbose.
builder for ‘/nix/store/zs8481q35d8xl5gjqvw4v6zv61qgx2rg-exa-2016-04-20.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/l6k5mknhb3hn7g5k43q5iwv471ndbapn-system-path.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/4l9cc5l452h5p2mqz56ck96sq4jn86az-nixos-system-firefly-16.09pre88945.5120af0.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/4l9cc5l452h5p2mqz56ck96sq4jn86az-nixos-system-firefly-16.09pre88945.5120af0.drv’ failed

It looks like some network hick up, but I tried it multiple times and I opened the github repository from the same machine and it loads fine.

Hm, unfortunately I have no idea what's going on -- you've fixed the hash, so it seems like another problem in fetchcargo.

Still thanks for the help. I learned a lot!

exa depends on a git repository in Cargo.toml which doesn't work with fetchcargo

maybe fetchcargo could take an argument to pin git-based dependencies on a certain commit?

I can confirm that this is still an issue when building exa:

error: failed to load source for a dependency on `zoneinfo_compiled`

Caused by:
  Unable to update https://github.com/rust-datetime/zoneinfo-compiled.git#f56921ea

through AUR/exa-git.
cc @ogham

Are there any updates on this issue, please?

Should be fixed.


Adding the flag for cram to run tests from the target directory

with the option cram_run_test_from_target to True (default is False)
cram tests will be executed using the built artifact to be found in
the target directory
this way, filtered scripts
(http://pybuilder.github.io/documentation/plugins.html#Filteringfiles)
which contain needed information for the tests, can be used.

I (the author of the cram plugin) collaborated on this. We chose to keep running cram from src by default since this makes it backwards compatible. We feel however that running cram from target would be the better alternative but were unsure, which task to depend on. We tried making target the default but this seemed to cause problems.

Also, the appveyor stuff isn't our doing.

Yes you can ignore the appveyor for now. I have to move the hook to the windows compat branch but haven't gotten around to doinc it just yet.

We now set cram_run_from_target to true. Can you remind me, what is the task that put's things in the target directory? Is it package if so, maybe we should depend on that task in the cram plugin?

Though it seems like verify already does that, so I don't think this change will cause any breakage in existing tools.

Also, once this is merged, I will update the documentation for the cram plugin in docs.

:+1:

Allright after some internal discussion with @mriehl I'll merge this now and release a new version.

done


Remove CoffeeScript cql-execution engine (moved)

Removing all source code related to the cql-execution engine since it
now exists in its own repository.  The README.md and OVERVIEW.md
have been retained to inform people where the new code it and since
these pages may be linked from external sources (like the spec).
See: https://github.com/cqframework/cql-execution/

@edeyoung -- are you fine with the cql-execution code being removed from the clinical_quality_language repo?


Feature/rest api (advanced and cleaned)

Based on the work of @kohlsalem, but slightly improved with the following topics:

added getStatus endpoint
added getMetadata endpoint, which includes the list of available plugins
switched the set.... endpoints to PATCH instead of GET
added (separate) getData endpoint, which just returns the current displayed data
added a bit of errorhandling
moved method implementation to messages.h, to make it same as the existing /message endpoint
removed arbitrary "OK" message from Status 200 return codes

May be im not firm enought to gh, but wouldnt it hae been smarter to let you join on PR #98? I guess now i should cose that, right?
After brief reviev: Since the handlers have nothing at all to do with message handling, i would not want to see them there nor would have ever serched them there. Having them distributed all over the place is even not so nice and comes with a include dependency hell (ok, a lot of).
I think it is way better to keep them with the webserver, as it was done with the web socket implementations.
I would be totally ok if you move the message handlers as well out of message.cpp...

May be im not firm enought to gh, but wouldnt it hae been smarter to let you join on PR #98? I guess now i should cose that, right?

you mean PR #99 i guess, #98 is the issue. but i know what you mean. Well yeah - propably you are right, but i did not know how i can contribute to your branch - so i did the best i could think of, created a fork, a branch, merged your branch to mine and built on top of it.
Only afterwards i saw that you already also did implement the getStatus, i guess we really just missed each other by short time.

After brief reviev: Since the handlers have nothing at all to do with message handling, i would not want to see them there nor would have ever serched them there. Having them distributed all over the place is even not so nice and comes with a include dependency hell (ok, a lot of).
I think it is way better to keep them with the webserver, as it was done with the web socket implementations.

I half agree and half disagree. You are right, the rest of the message file does have nothing to do with handling web requests and all those handlers belong somewhere else.
But definitely not in the asyncwebserver, because the purpose of that class is to provide the server and distribute the requests - NOT to handle them in detail.
I'll take care of that, give me a bit.

I would be totally ok if you move the message handlers as well out of message.cpp...

I'll look into that as well.

Dependency hell was if i put the handlers where they beloned to (Screen, Plugin Manager, ...)

Thank you both! I only have one thing. There are often comments that offer no added value. I would suggest identifying and deleting them.
I mean something like this:
// Send a response to the client
request->send(200, "text/plain", "Message received");

// Call the add function with the extracted parameters
Messages.remove(id);

// Extract the 'value' parameter from the request
int value = request->arg("value").toInt();

The code itself explains what happens here.

Fair point. I never wrote them, they are a gift of "Check and comment the following code" request to chatgpt.
@gelse, it least i take no offence from getting them removed, if anyone woulf have cared :-D

Thanks for the work here! I'm using this currently with home assistant, the matrix now shows the artist + track that my internet radio is playing, which beside the clock animation is pretty cool.
in configuration.yaml
  shell_command:
     show_playing: curl http://obegransad.lan/message?text={{ state_attr('media_player.heritage_2', 'media_artist') | urlencode }}%20-%20{{ state_attr('media_player.heritage_2', 'media_title' ) | urlencode }}&delay=100&repeat=2&id=1
     dim_matrix: curl -XPATCH http://obegransad.lan/setbrightness\?value\=30
     wake_matrix: curl -XPATCH http://obegransad.lan/setbrightness\?value\=100

@ph1p any idea if it can be merged?

Sure! I'll test it tomorrow and merge it afterwards :)

Many thanks to al of you! :) The PR has been merged. I've updated the code a bit so that each endpoint is now prefixed with /api. I also removed get... because if it's a GET request, it's not necessary.


Postgres Native Enum with * schema

Describe the bug
While trying to use the schema: * option to allow multiple postgres schemas to be created with the same classes, I have noticed that the update command does not generate the proper Enum types. There are all generated as *.${nativeEnumName} instead of ${options.schema}.${nativeEnumName}.
I have tracked the issue down to this code block, where meta.schema is * and thus that is what is used in generating the sql. I am unfamiliar with the project and how best this could be fixed.
output
create type "*"."axis_move_axis_type_enum" as enum ();

desired output
create type "test"."axis_move_axis_type_enum" as enum ();

https://github.com/mikro-orm/mikro-orm/blob/5180b6aacec442689169cf1edbc6f7826e32dd83/packages/knex/src/schema/DatabaseSchema.ts#L88-L101
additional context
the create sql function also has issues with the * schema. It generates this sql, where the enum has the schema in it twice, but the create table references the enum differently.
create type "test"."test.axis_move_axis_type_enum" as enum ();
create table "test"."axis_move" ("id" serial primary key, "axis_type" axis_move_axis_type_enum not null);

Reproduction
//Type def
import { Entity, Enum, PrimaryKey, Property } from "@mikro-orm/core";

export enum AxisType {
...
}

@Entity({tableName: "axis_move", schema: "*"})
export class AxisMove {
  @Enum({
    type: 'AxisType',
    name: "axis_type",
    nativeEnumName: "axis_move_axis_type_enum",
  })
  axisType: "linear" | "rotary";
...
}

// schema generator
import { MikroORM } from '@mikro-orm/core';
import config from './config.js';

const orm = await MikroORM.init(config);
const schemaGenerator = orm.getSchemaGenerator();

console.log(await schemaGenerator.getUpdateSchemaSQL({
  schema: 'test',
}));
orm.close();

What driver are you using?
@mikro-orm/postgresql
MikroORM version
6.2.1
Node.js version
v20.11.1
Operating system
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Slack.
[X] The provided reproduction is a minimal reproducible example of the bug.

I was looking into this yesterday, and the challenging part is diffing combined with support for enums in a specific schema (for entity in wildcard schema). Will need to give it a bit more time, worst case I'll implement at least the support for wildcard enums in general, as that should be rather easy (and I got that mostly already implemented locally).

not perfect, but it should cover your use case


Haveged seems to fail with opts

Having a bit of an issue getting haveged to start with opts.
Adding the following to /etc/conf.d/haveged
-F -w1024 -v 1
The server seems to stall on boot, and I have to drop back to rescue, and revert the changes, before it will boot again.

Have you tried without the -F option? It should only be used with systemd afaik.

Tried with these options (haveged -F -w1024 -v 1) on a VC1, no problem at all
-F is for foreground, but openrc seems to handle this gracefully
System infos:
Welcome to Alpine Linux 3.3
Kernel 4.5.7-std-3 on an x86_64

Can you check on the console what happens and maybe give more info on your versions ?

I've been neglecting this somewhat as I moved to a CentOS image for my testing, however, I've solved most of my other problems and will now try Alpine again. Hopefully get around to it this weekend.


Station can't be opened because Apple cannot check it for malicious software

Application version: 3.1.0
OS: MacOS 14.6.1

Steps to Reproduce:

Unzip Station app from Station.zip
Copy Station to Applications directory
Launch Station from Applications directory
See error

Screenshots

I understand that you don't have the ability to distribute DMG files at the moment, but I will not be using the app until this is resolved, and I am sure there are others that feel similarly. This is unfortunate because Station seems really cool!

Same issue. An older mac os allowed us to force open, but now that is no longer an option.
So for now, Station cannot be used by any new Mac users. Thanks for any suggestions

You can override the security check on MacOS.
https://support.apple.com/en-us/102445


Remove ThreadPooling self reference in Pooled object to prevent memory leak

There is an apparent leak in the ThreadPooling class. We pass a reference to the ThreadPooling class into the ThreadLocal's value, which creates a strong reference to the class, which in turn creates a strong reference to the threadLocal object and therefore prevents the class from being garbage collected.
We solve it by removing the reference to the ThreadPooling class.

@jkni - after reviewing the class more, I agree with your conclusion. Thanks for taking a look. This is ready for re-review.


[DO NOT MERGE] Remove reference to port 19200

This is dependant on this PR being merged and deployed: https://github.com/alphagov/govuk-puppet/pull/6452
https://trello.com/c/KMIQEH5p/281-cleanup-tasks

@suzannehamilton done


不能下载数据库

fate_db_200331.7z
返回404

@HappinessMan
new address:
https://github.com/babyname/fate/releases/download/v3.5.1/fate_db_200331.7z


fix(art-quiz): add auth redirects to artQuizRoutes

The type of this PR is: fix

This PR solves GRO-1507
Description
This PR adds a redirect for logged out users who hit any of the /art-quiz routes. Now, instead of seeing a blank screen or a 500, a logged out user will be redirected to /login and redirected back to the art quiz once they've signed in.

question [non-blocking]:
Wondering if it's better to redirect to login or send them to the artsy / homepage? No opinion either way, but do you think we should stick to convention here?
format via Conventional Comments

You would send them to login with a redirectTo to the quiz page. If you're hitting this route then by default you want to login.

You would send them to login with a redirectTo to the quiz page. If you're hitting this route then by default you want to login.

does this hold true for users who log out while viewing their results page though? just making sure all bases are covered

for users who log out while viewing their results page though

I'd imagine this is an extremely rare event, but sure, why not? Presumably you would want to pick up where you left off. No one is going to log out, then immediately log back in in order to navigate to the homepage.

I'm gonna hit merge on this. We can prolly check out how the auth redirect feels for navigating from the results page during our next round of QA??


Open Inventory Action

So I was having a look at the inventory actions and noticed one for everything except when the player goes to open their inventory, I was wondering how I can detect this action, deny it and run a command? So essentially running a command when the inventory is opened without actually opening it.

Hello,
seems like there is no possibility to detect when a player opens their main inventory.
You can only detect when they open other inventories, like chests, furnaces, ...
BloodEko

You only can detect when items are clicked on an inventory

There is in fact a on players open <inventory>: event... this does not, however, apply to the main inventory due to minecraft limitations, as BloodEko said.

So could I say do determined cancelled or whatever it is when the player tries to open their inventory but cancel it correct?

... As I just said, and BloodEko said as well, the event does not apply to the player's own main inventory.


Showing Recent Issues Ad Hoc code signing is not allowed with SDK 'iOS 11.2'

I didn't know how to solve this problem when I was packing. The project is successfully packaged before using this component

Thanks @ailehui for raising the issue. I have encountered the same issue while packaging the app for production.
I am able to resolve this issue using below steps:

Removed Bundle Identifier from FileBrowser project.
Assigned Team & Signing Certificate in FileBrowser project.

Please let me know in case you still face the same issue.

Thanks
</ Pranav >

Hi @ailehui, I hope the issue was resolved for you. If yes can you please close the raised issue.
Thanks
</ Pranav >


[Enhancement] Consider adding support to YT-DLP, a fork of YT-DL that loads cookies directly from the browser

https://github.com/yt-dlp/yt-dlp
This is a fork of YT-DL that has the command "--cookies-from-browser", which already extracts and reads the cookies. That would make things a lot easier.
Plus, with the current version, even though play-with-mpv recognizes YT-DLP and plays videos from it normally, it doesn't load your already downloaded cookies (error message appears). Please, take a look in making your script also compatible with this!
Thank you for your reading my two suggestions,
Thomas

Ah, compatible with multiple downloaders gradually leads to mpv-handler becoming more and more complex.
I'm considering refactoring it, thanks for the suggestion.

See https://github.com/mpv-player/mpv/commit/cc4ada655aae06218b900bb434e3521566394cde
Upstream mpv are using yt-dlp by default when you have the yt-dlp binary, and mpv-handler should be compatible with yt-dlp (because youtube-dl and yt-dlp options are mostly compatible)
Then, you just need add yt-dlp binary directory to the environment variable PATH or put yt-dlp binary at mpv binary directory (for help the ytdl-hook find the binary)
The yt-dlp option --cookies-from-browser, I will support it asap.

So, this issue should be solved, right? 😉


[rt][ocl] Select Generic Scheduler if Driver is < 550

Description
TornadoVM can assign big blocks of  threads depending on the driver. This can increase performance in some platforms (e.g., NVIDIA GH200) using the NVIDIA Driver 545.X.
Problem description
n/ a.
Backend/s tested
Mark the backends affected by this PR.

[X] OpenCL
[ ] PTX
[ ] SPIRV

OS tested
Mark the OS where this PR is tested.

[X] Linux
[ ] OSx
[ ] Windows

Did you check on FPGAs?
If it is applicable, check your changes on FPGAs.

[ ] Yes
[X] No

How to test the new patch?
make
make tests
tornado --threadInfo  -m tornado.examples/uk.ac.manchester.tornado.examples.compute.MatrixMultiplication2D

I clean up the gitignore file