[Bug]: Inovelli switch FW update only updates first target

Checklist

[ ]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
PKG executable
Z-Wave JS UI version
8.2.1
ZwaveJS version
10.3.0
Describe the bug
Everything works!
But if I update firmware for inovelli switches ether through OTA updates or advanced firmware updates nothing is updated.
i still get in report old version of firmware...
To Reproduce
same as above
Expected behavior
to update firmware
Additional context
No response

There's already an issue for Inovelli firmware issues, #5081.

Folks, kpine, I am not sure what is the error in firmware and which version...
I am trying to upgrade from 1.35 to 1.45, 1.57 to 1.61 (beta) with no success
Unrelated is that I can't even update Aeon usb stick to 1.2 version of firmware but this has nothing to do with truly excellent package that Lando created.
so i have
zwave-js-ui: 8.2.1
zwave-js: 10.3.0
home id:<PHONE_NUMBER>
home hex: 0xd6ee2767
Here is the log and screen shots of operation:

zwavejs_2022-10-19.log
and after upload of firmware:
same things reported:

Looks like the first part (target 1 of the update) gets applied successfully, but the 2nd part doesn't start (yet?). Unfortunately I cannot investigate this until I am back from paternal leave.
The 100% is a UI issue though. The update isn't done, only the first target. @robertsLando I extended the firmware update events to indicate the overall progress and which part is being updated etc.
You should probably use the new parameter (see docs) to display progress.

ok, thanks for the update, we will wait for change when it happens...
For now i will try alternate methods ...

The 100% is a UI issue though. The update isn't done, only the first target. @robertsLando I extended the firmware update events to indicate the overall progress and which part is being updated etc.

AFAIK I'm already using that, let me do a check.
Just to be clear @catchthemonster the problem is the second part is not done so even if I fix UI this wouldn't work

Yes, seems so. I need to figure out why, but the ui should not indicate that the update is complete.

Agree with both...
For what is worth, zwave2mqtt  framework did this correctly  for me ...
Maybe you can look at that project and compare the code base ...
Ether way thank you both and app is excellent aside from firmware ...
Regards,

For what is worth, zwave2mqtt framework did this correctly for me

What you mean? zwave-js-ui and zwavejs2mqtt are the same application

zwavejs2mqtt if I remember used open-zwave cpp libs. It might be the same up I don't know. Last year i was able to update firmware with it just fine

zwave2mqtt used open-zwave which never supported firmware updates.
zwavejs2mqtt and zwave-js-ui are the same and use node-zwave-js, which is (to my knowledge) the  only open source Z-Wave driver to support firmware updates.

Same issue here. I get a timeout error after it tries to start the second update.
`

`

@Tyfosho Seeing a driver log on loglevel debug of this would be helpful.
Please attach it here as a file using drag&drop

@AlCalzone - any instructions anywhere on how to grab that log? I can do it now if so.

Here you go: driver log
Make it loglevel debug and attach it here as a file (drag & drop into the text field).

I honestly do not remember exactly, but I know I was able to upgrade
firmware just fine with the old implementation of the  node.js  service.
There were no issues and targets worked.
There was a bug in inovelli dimmers that would kill the light if the 80%
threshold was crossed.
Now I see that whatever I build today has nothing to upload firmware ...
Sorry if I confused everyone...
On Thu, Oct 20, 2022 at 2:56 PM AlCalzone @.***> wrote:

zwave2mqtt used open-zwave which never supported firmware updates.
zwavejs2mqtt and zwave-js-ui are the same and use node-zwave-js, which
is (to my knowledge) the only open source Z-Wave driver to support firmware
updates.
—
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/5167#issuecomment-1286000674,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEC5FX72YNPOSXODYXB7NTWEGIWLANCNFSM6AAAAAARJDKMVY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Aleksandar Kacanski - Sasha
Python Rocks, natural or digital, regardless ...

https://github.com/zwave-js/zwave-js-ui/pull/2755

@AlCalzone driver log attached!
z-ui_2022-10-24.log

@catchthemonster I also recall being able to do firmware updates to my LZW31-SN (Red Dimmers) switches back when the add-on was known as zwavejs2mqtt. I don't have any concrete evidence of it though, so maybe I'm mistaken.
I also want to add the following error messages to help make this issue easier to find for those who are troubleshooting. I ran into both of these while mindlessly pushing buttons after the firmware update failed:

ERROR Z-WAVE-SERVER: Failed to start the update: A firmware upgrade is already in progress! (ZW1500)

The firmware update was transmitted completely, cannot abort anymore. (ZW1504) abortFirmwareUpdate undefined

It looks like Mr. Calzone (congrats on the newborn - trying not to hail you) has a good understanding of the issue, but in case another logfile is needed:

2022-10-25T21:25:37.386Z CNTRLR » [Node 022] Sending firmware fragment 384 / 389
2022-10-25T21:25:37.461Z CNTRLR » [Node 022] Sending firmware fragment 385 / 389
2022-10-25T21:25:37.530Z CNTRLR » [Node 022] Sending firmware fragment 386 / 389
2022-10-25T21:25:37.598Z CNTRLR » [Node 022] Sending firmware fragment 387 / 389
2022-10-25T21:25:37.666Z CNTRLR » [Node 022] Sending firmware fragment 388 / 389
2022-10-25T21:25:37.734Z CNTRLR » [Node 022] Sending firmware fragment 389 / 389
2022-10-25T21:25:38.171Z CNTRLR « [Node 022] Firmware update (part 1 / 2) succeeded with status OK_RestartPending
2022-10-25T21:25:38.173Z CNTRLR [Node 022] Continuing with next part in 5 seconds...
2022-10-25T21:25:38.173Z CNTRLR [Node 022] Updating firmware (part 2 / 2)...
2022-10-25T21:25:38.173Z CNTRLR » [Node 022] Starting firmware update...
2022-10-25 14:25:44.571 INFO Z-WAVE: Node 22: value updated: 38-0-currentValue 0 => 0
2022-10-25 14:25:46.794 INFO Z-WAVE: Node 22: value updated: 50-0-value-66049 0 => 0
2022-10-25 14:26:38.239 ERROR Z-WAVE-SERVER: Received no matching command within the provided timeout! (ZW0201)
ZWaveError: Received no matching command within the provided timeout! (ZW0201)
at Timeout.<anonymous> (/opt/node_modules/zwave-js/src/lib/driver/Driver.ts:4379:6)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)

@crawford , agree! I know I am not senile yet, -)...
Yes congrats to Mr. Calzone, that is a wonderful thing...
Hopefully this will be fixed soon.
BTW, I am not super happy with Inovelly. They do have a cool dimmers, but buggy firmware, and for some reason when I loose power in house with generator, these switches have tendency to freak out and my light start acting up...
so firmware is critical to be up to date and easy to upload and install.
it would be also nice to include in selection beta drivers, so that people upload latest to test, instead of going number of clicks more to actually get a downloaded package from disk...
Regards,

@catchthemonster Beta drivers will be available, but AFAIK none of the UIs have this implemented yet.

Beta firmwares will be available, but AFAIK none of the UIs have this implemented yet.

What?

For what is worth that is correct. However, if zwave-js firmware driver is
capable via REST api calls to find beta drivers, then those drivers should
show as a choice in gui interface inline with stable drivers...
Just me...
On Fri, Oct 28, 2022 at 5:12 AM Daniel Lando @.***>
wrote:

Beta firmwares will be available, but AFAIK none of the UIs have this
implemented yet.
What?
—
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/5167#issuecomment-1294741596,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEC5FV6DS73MYEJPBKCEQDWFOKJRANCNFSM6AAAAAARJDKMVY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Aleksandar Kacanski - Sasha
Python Rocks, natural or digital, regardless ...

Opened a feat request: https://github.com/zwave-js/zwave-js-ui/issues/2768

It's already available

@robertsLando, Ok I didn't know as this issue wasn't closed nor is the update server showing any firmware updates for my devices as I know they're not on the latest.  I'll try to manually update my devices this weekend and see if that works.

No progress yet

Sorry closed it because I thought the issue was about the possibility to select a beta release from ota

Duplicate of #5081
(closing for housekeeping purposes, one issue about this is enough)


Question: Custom message loop and WinMain in WinUI 3?

I am interested in porting some Win32 XAML Islands code to WinUI 3 for a future release - but I have one major question that I can't seem to find an answer to anywhere: is there a way to host WinUI 3 in a custom wWinMain function and with a custom message loop?
I am asking this because currently:

I am using some Win32 features and third party components that require PreTranslateMessage to be inserted in the message loop
I need some custom initialization in that happens before anything from WinUI in wWinMain
I use APIs like WriteFileEx, which require to use MsgWaitForMultipleObjectsEx instead of the GetMessage loop.

Sure, I could just wait for WinUI 3's XAML islands and port all my existing code to that, but that seems like far away still, and I want to make use of WinUI's Window class to be able to remove most of my custom XAML island hosting code, since Window covers most of my use cases.
From what I understand, I can just disable the XAML generated main and use my own wWinMain for the first part, but the XAML generated main calls into Application.Start, which blocks until the app exits: this is undesired, and I can't find a non-blocking alternative.
Another side question: can we mix normal WinUI 3 and WinUI 3 XAML Islands in the same app?

This feels very similar to #3408 @sylveon do you argree?

For the custom main part yes, but not the message loop

@ocalvo and @MikeHillberg FYI

Xaml islands is the design for hosting WinUI3 and running your own message pump. Just avoiding the call to Start won't help because in addition to running the pump it does some necessary initialization.
And I'm not sure that the Window class will work when running the thread as an island. Maybe we should make that a feature request, and at least call out in the spec/docs.

Why not make it possible to do the initialization as a separate API call so that a consumer can simply call that, new up Windows, and run their own message loop?
The XAML Island API is a lot more trouble than using Window (because you have to implement all the windowing/custom titlebar stuff that Window takes care of yourself).
Making Window supported in XAML islands would be a valid alternative yes.

@sylveon I added the label feature proposal to the issue. Does this sound good for you? Feel free to close the issue.

Yep, sounds good to me

Application.Run being blocking is a huge problem for me :(

Application.Run being blocking is a huge problem for me :(

Is there any update on this issue? I am facing the same issue

Application.Run being blocking is a huge problem for me :(

For me as well! I recently made a wide scoped feature request. I think your suggestion falls into this request as well. Can you relate to that?

Allowing custom message loops would probably be required to implement your feature proposal, but your proposal in specific isn't related to my use case (everything is contained within a single assembly/app, I just want to be able to use custom code for my message loop)

Also, I don't use C#

Your very technical specific suggestion is indeed one of the requirements to be able to implement mine. My suggestion may have a larger audience, which could give traction to yours. That's the reason I'm replying here.
Lastly, I don't think Microsoft would limit any solution for your problem to just C++.


Rip out Proposition world

The proposal is that we rip out function world, proposition world, and advanced proposition world, and then we have the problem that advanced addition world (and adv multiplication and inequality world) uses tactics which have not yet been taught, so they need to be taught within advanced addition world.
Let's focus on advanced addition world. What needs to be done here is that we need to see which tactics are being used in that world which were taught in the ripped-out worlds.

I take it you've done this now.


feat:Added Lark Mini Program IDP and upgraded lark idp

Added Lark Mini Program IDP and upgraded the interface and configuration of the original Lark IDP.
curl -X POST "https://localhost/api/login/oauth/access_token"
-H "content-type: application/x-www-form-urlencoded"
-d "tag=lark_miniprogram"
-d "client_id=your_casdoor_client_id"
-d "code=lark_miniprogram_login_code"
-d "username=your username"
-d "avatar=your idp avatar"

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@xyezir fix linter error:

fix golangci-lint error(gofumpt -w lark_miniprogram.go)

The code quality is below the bar. Too many diff. Closed for no.w


[resotocore][fix] Benchmark refinements

Code of Conduct
By submitting this pull request, I agree to follow the code of conduct.

Codecov Report
Patch and project coverage have no change.

Comparison is base (669db03) 89.62% compared to head (27ea850) 89.62%.

:exclamation: Current head 27ea850 differs from pull request most recent head 63e0da9. Consider uploading reports for the commit 63e0da9 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1496   +/-   ##
=======================================
  Coverage   89.62%   89.62%
=======================================
  Files          74       74
  Lines       11825    11825
=======================================
  Hits        10598    10598
  Misses       1227     1227

Flag
Coverage Δ

resotocore
89.62% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


pull request for e2e test at 12/10/2016 15:56:04

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat: Add enable deletion protection and drop invalid header fields

Description
This PR allows for configuring the enable_deletion_protection and drop_invalid_header_fields for the ALB. Its variables default to false, which is the default ALB module.
Motivation and Context
I needed to configure these options for compliance reasons.
Breaking Changes
No
How Has This Been Tested?
Tested on own infrastructure

@antonbabenko Are there any additional changes I need to make or does what is submitted look sufficient? Thx

@bryantbiggs Changes sound good to me. Please see updates

Thanks @bryantbiggs

Thank you guys for this PR!
v2.38.0 has been just released.


Question: Conditional Statements

Hello,
I have just found Sprache and have some questions.
Will it be able to parse conditional statements like this?
IF ($make == 'big') THEN
$size = '100'
ELSE
$size = '50'
END
How will it tokenize multiple lines?
many thanks,
Ferdinand

If you  don't make newlines a statement terminator then they are simply ignored.
Conditional statements are possible too. I'm on my phone at the moment but I can dig up an example later if you don't figure it out by yourself.

On 28 Oct 2016, at 19:13, insik<EMAIL_ADDRESS>wrote:
Hello,
I have just found Sprache and have some questions.
Will it be able to parse conditional statements like this?
IF ($make == 'big') THEN
$size = '100'
ELSE
$size = '50'
END
How will it tokenize multiple lines?
many thanks,
Ferdinand
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks


Fix circular dependency when embedding block roots in state by using signed_root

What is wrong?
Spec PR: https://github.com/ethereum/eth2.0-specs/issues/797
Spec issue: https://github.com/ethereum/eth2.0-specs/pull/816
How can it be fixed
Fill this in if you know how to fix it.

i'm tackling this as part of addressing the work in #397

fixed in #470.


"Satulung", "Arieșu de Pădure", "Fersig", "Finteușu Mic", "Hideaga", "Mogoșești" and "Pribilești" are listed in the drop-down menu

Description:
"Satulung", "Arieșu de Pădure", "Fersig", "Finteușu Mic", "Hideaga", "Mogoșești" and "Pribilești" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Satulung" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Satulung, MARAMUREȘ (Satulung)" and "Comuna Satulung, MARAMUREȘ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Arieșu de Pădure" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Arieșu de Pădure, MARAMUREȘ (Satulung)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Fersig" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Fersig, MARAMUREȘ (Satulung)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Finteușu Mic" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Finteușu Mic, MARAMUREȘ (Satulung)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Hideaga" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Hideaga, MARAMUREȘ (Satulung)".

Step 10
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 11
Type "Mogoșești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Mogoșești, MARAMUREȘ (Satulung)".

Step 12
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 13
Type "Pribilești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Pribilești, MARAMUREȘ (Satulung)".


FIX: prevent a transaction to appear twice in the mempool

Summary
A transaction might appear twice in the mempool, i.e., it is present more than once in txs, while txsMap only links a single of these occurrences.  The bug happens when the cache overflows.
Context
The bug is related to issues #3865 and #5281 in Tendermint. Both issues are closed with this fix. One may reproduce them easily in e2e, e.g.,
./build/generator -m v0.37.1:1 -d networks/generated/
./build/runner -f networks/generated/gen-0003.toml setup
find networks/generated/gen-0003/ -iname "config.toml" | xargs sed -i s,"cache_size = 10000","cache_size = 100",
find networks/generated/gen-0003/ -iname "config.toml" | xargs sed -i s,"prepare_proposal_delay = 200ms","prepare_proposal_delay = 3000ms",
./build/runner -f networks/generated/gen-0003.toml start
curl -s http://localhost:5701/broadcast_tx_sync?tx=\"apple=100\"; for i in $(seq 1 1001); do curl -s http://localhost:5701/broadcast_tx_sync?tx=\"orange=${i}\"; done; curl -s http://localhost:5701/broadcast_tx_sync?tx=\"apple=100\"

Remark
The present fix works because the mempool receives in order and sequentially all the asynchronous ABCI callbacks. Notice that in the past, this was not the case for the grpc client, as outlined here. Sidestepping such a constraint is feasible, yet presently open.

PR checklist

[x] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Could you please add a changelog entry?
You get inspiration from #383 on how to do it

Could you please add a changelog entry? You can get inspiration from #383 on how to do it

Done.

Could you please add a changelog entry? You can get inspiration from #383 on how to do it

I guess so. What should the backport tagert? (e.g., v0.34.x to v0.38x)

Could you please add a changelog entry? You can get inspiration from #383 on how to do it

I guess so. What should the backport tagert? (e.g., v0.34.x to v0.38x)

Yes. I am just wondering whether we can just use the bot to do so, as the organization of files has changed a lot from v0.37.x and v0.34.x.

There are some newly appeared lints, but they are not caused by any changed lines, so should be addressed in a different PR.

I rebased it and added the appropriate backport tags.

I would say to go ahead, squash and merge it.
I would only remove this FIX from the commit message, also from the PR name.


Textures appearing as yellow tinted

Godot version: 3.0 rc2 (with mono)
OS/device including version: Linux - Xubuntu 16.04 LTS 64-bit
I have a 3D model imported in OBJ format, and a texture in PNG format. I imported the model as a Mesh via MeshInstance. However, the texture looks very yellow! It looks like this in Godot:

Note how it's even yellow on the left side file list thing. But the PNG texture file looks like this:

This happens in pretty much a brand new project. I don't even think the 3D model is necessary to see the yellowed texture, you can try just importing it and looking on the left side file list. But regardless, here is the model and texture files: Model.zip

Looks like this is already fixed in the released version of Godot 3.0, but with one caveat. The fix only applies to newly imported files. I'll explain:
I have the problem you describe when I open a project that was created using an earlier custom-build of Godot 3 (from the beta days) but not when I open a project created using the released version of Godot 3.0. Note that I am opening both projects using the released version of Godot 3.0.
The difference is the import flags set for the .png file. The settings created by the beta-days version of Godot has "Normal Map" set to "Enable", which is what's causing the greenish yellow tint, whereas the settings created by the released version has it set to "Detect", which doesn't have the same issue. However, this is not the only difference. The Flags settings are different too (Mipmaps, Filter, SRBG, etc.), which cause other issues on their own.
You can manually fix the import settings of your old .png, but there's an easier way to ensure the default import settings of the final released version of Godot 3.0: Copy the .png file to a new name, let Godot automatically import it, delete the old .png, and rename the new .png to the original name.
I think this issue can be closed, as it seems to have been fixed. Anyone who lands on this page can use to procedure I've described in the last paragraph above to fix their old projects.

You can manually fix the import settings of your old .png, but there's an easier way to ensure the default import settings of the final released version of Godot 3.0: Copy the .png file to a new name, let Godot automatically import it, delete the old .png, and rename the new .png to the original name.

Even simpler is to delete the .import files for the PNGs you need to fix, and let Godot recreate them with default settings (and thus reimport them).


[Bug]: bazel run -- @pnpm//:pnpm does not work

What happened?
The documentation says that I should be able to invoke:
bazel run -- @pnpm//:pnpm --dir $PWD install --lockfile-only

However, on Windows, with rules_js 1.42.1 set up in MODULES.bazel with copy-and-paste from releases page, this command fails:
D:\udu\units\cb_0508_bazel>bazel run -- @pnpm//:pnpm --dir $PWD install --lockfile-only
WARNING: Target pattern parsing failed.
ERROR: Skipping '@pnpm//:pnpm': No repository visible as '@pnpm' from main repository
ERROR: No repository visible as '@pnpm' from main repository
INFO: Elapsed time: 0.744s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

Version
Development (host) and target OS/architectures:
Output of bazel --version:
7.2.0rc1
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
1.42.1
Language(s) and/or frameworks involved:
javascript, node18.20.0
How to reproduce
No response
Any other information?
No response

Excellent, this works well. Thank you!


Trunk

Coverage decreased (-0.5%) to 25.986% when pulling b31f1e91ddae9d6135cec66d61f24781447caaef on trunk into a8d8b97588621d6e406e0868d769ce6be2b8acf6 on master.


Recompiling scripts generates duplicate property modifiers.

If you decompile a .pex with Champollion and recompile the .psc with Caprica, properties generate duplicate modifiers:
Before:
bool Property OwnedByPlayer = False auto conditional
bool Property PlayerHasVisited auto conditional hidden
After:
bool Property OwnedByPlayer = False auto conditional conditional
bool Property PlayerHasVisited auto conditional hidden conditional hidden

This should be fixed in v0.1.1.


Adds a warning for using [PreferRecompute] on methods that may contain side effects

Also adds a way to suppress this warning by using [PreferRecompute(SideEffects.Allow)]
Not technically a breaking change since this will only generate a warning.
Resolves #3303

@saipraveenb25 there are CI failures here, will you take a look?


Prevent publishers from firing on startup

On every app start, the session sends events to the photoPublisher and the videoPublisher. I assume this is to restore the display of any previously shot photos or videos. However, in my app, this would trigger an upload to the server.
Is there a way to prevent these initial event sends - or at least detect if these photos and videos are published in response to a current user interaction (i.e. the user shoots a photo or records a video)?

PS: I also noted that the publishers (e.g. the photoPublisher) are called multiple times on startup, even though I create the session only once. What might be the reason for his?


feat: cra to nextjs and npm to yarn

Summary
Migrating from CRA to NextJS for performance. Also migrating from npm to yarn package manager for performance.
Checklist

[ ] This PR fixes an issue.
[x] This PR adds something new
[x] This PR is a breaking change
[ ] This PR is not a code change (e.g. documentation, README, ...)

Please do DM me on discord to remind me for changes :)

lgtm ig


template for nml tasks

First cut at creating task table remake file for creating nml files for running with glm. Will eventually need to adjust driver data file based on the NHD_ID / NLDAS cell lookup function, but still need to create that function so there is a placeholder driver file right now.

I was imagining setting up the nml files and paths to driver directories locally and then throwing that over to Yeti to run. We just need driver and nml files on yeti right? oh and maybe obs

sounds like a good plan

will Yeti be able to authenticate w/ gdrive to take advantage of the shared cache?
eek, i don't know! jake's plan sound fine, but maybe the answer here is also yes? might be worth some exploration.

@jread-usgs @aappling-usgs would it be alright to merge this? I have another PR coming up

Fine with me, yes


RFC: A new schedule way for distributed lookup join

I have an optimization to this, for distributed lookup join, we can add an exchange to the build side(outer side):
LookupJoin
   HashExchange(key = index)
       IndexTable

And the lookup join needs to buffer all data from index table. This is quite similar to dynamic filter approach, and there are several advantages:

We can avoid sending scan tasks to all inner table when build side is not large.
Compare to current approach, we also avoid sending duplicate task same vnode in primary table.

Per offline discussion, we agree that we can use this proposal for now.

I feel like this is the same idea as my proposed streaming delta join. We should push data instead of doing remote lookup. If we can make this possible, it will probably also be easier for us to re-enable streaming delta join (as now we have upstream hash shard information in optimizer).

... and the overall design looks great to me!


Including OCMock 3.x via cocoapod in XCode 6 (beta 6) breaks compilation

I have not tried including OCMock into my project manually, any expectation that would have a different effect?
If I include OCMock 3.x with my podfile, I cannot compile my project.  OCMock 2.x does work this way.
ld: framework not found XCTest
clang: error: linker command failed with exit code 1 (use -v to see invocation)

This should be fixed on OCMock 3.1.2 which no longer declares a dependency on XCTest.

Closed due to inactivity.


Add timeouts to config for Typhoeus to prevent NSX requests hanging for hours

New timeout is 10 minutes
Default Typhoeus connect timeout is 5 minutes, so we left that alone.

i think it would be nice if all the files containing the swagger codegen version changes. in a separate commit. as it would be easier to navigate and see whats the actuall changes are

Done!


kicad-testing: init at 7.0-2023-09-15

update update.sh to include it in versions.nix
sort the package inputs a bit
Description of changes
upstream has been encouraging the use of a testing package to test the changes on the stable branch
(that is, changes that have been backported to stable and will be in the next stable minor or patch release)
This adds such a package.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

resolved merge conflict

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2967

@pbsds have i resolved your nits?

Thanks for the ping, somehow marked this as read. See https://github.com/NixOS/nixpkgs/pull/255271#discussion_r1406476500

added the use of the create_at field from gitlab
updated the testing package
and tested the updated package

Result of nixpkgs-review pr 255271 run on x86_64-linux 1

2 packages built:

kicad-testing
kicad-testing-small

Tested the update script, which seems to be working. nix-update -u -f nixpkgs/ kicad-testing however fails with fatal: not a git repository, but nix-update -u kicad-testing  works when workdir is nixpkgs root. I consider that issue out of scope for this PR.
LGTM


Release 1.0: Navbar contextual components

Update to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Update conventional-changelog to gulp-conventional-changelog to get rid of deprecations
Update the ember-try config
Reinstate navbar contextual components, making them nested as well and treating nav-item as a non-nested contextual component

Thanks again! Highly appreciated! :)

Gladly! Any other part of this you'd like me to help with?


Garbled modals in landscape orientation on mobile

It looks good in portrait.

Getting closer. Could you please make the modal extend a little lower and add more of a white border.


error with 04_streaming/simulate/df05.py

I'm running this on qwiklabs.  This part here
$ python ./df05.py

Gives the following truncated stack trace:
File "apache_beam/runners/common.py", line 419, in
apache_beam.runners.common.SimpleInvoker.invoke_process
File "/home/google2864119_student/.local/lib/python2.7/site-packages/apache_beam/transforms/core.py", line 1161, in
wrapper = lambda x: [fn(x)]
File "./df05.py", line 98, in
| 'airports:tz' >> beam.Map(lambda fields: (fields[0], addtimezone(fields[21], fields[26])))
File "./df05.py", line 24, in addtimezone
import timezonefinder
File "/home/google2864119_student/.local/lib/python2.7/site-packages/timezonefinder/init.py", line 2, in
from .timezonefinder import TimezoneFinder
File "/home/google2864119_student/.local/lib/python2.7/site-packages/timezonefinder/timezonefinder.py", line 300
def closest_timezone_at(self, *, lat, lng, delta_degree=1, exact_computation=False,
return_distances=False,
^
SyntaxError: invalid syntax [while running 'airports:tz']
google2864119_student@cloudshell:~/data-science-on-gcp/04_streaming/simulate (qwiklabs-gcp-8a4cca54b8a375af)$

this has been fixed. if you do a new Qwiklab, you should see the fix.

Many thanks
On Tue, Mar 26, 2019 at 12:25 PM
A4q5PMrDW1d+vkfDYmdoLZOcKFKhZHgRZazKmBiqgKE=<EMAIL_ADDRESS>wrote:

this has been fixed. if you do a new Qwiklab, you should see the fix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/data-science-on-gcp/issues/56#issuecomment-476730816,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEN6HF88qQQXdXWtpAJVrkrntd-PPZlWks5vakoWgaJpZM4cJnz0
.

--
John F. Davis
6 Kandes Court
Durham, NC 27713
919-888-8358
Public Profile https://www.linkedin.com/in/netskink
独树一帜


yb file broken? how to start from point onwards

I am trying to use youtube-dl
youtube-dl -f 22 -o reydavid01.mp4 http://www.youtube.com/watch?v=fRwA0IagPik
The problem is that HD version (fmt 22) stops at 20% and 480p version stops at 67%. I cannot get past this point
Looks like the video file is broken. Even when I get the exact HTML link through DownloadHelper and use wget, it only gets to this particular % point.
BUT I can see the whole youtube video online fine. So I want to know if there is any way to start downloading form certain byte-point (e.g 328556227 onwards) or form a specific time point (e.g. 10 minutes 3 seconds). If this was possible I still might be able to download video in chunks.
Thank you.

Duplicate of #1184.


Is it possible to suppress http: TLS handshake errors?

What is your question?

Is it possible to suppress http: TLS handshake errors?
They're starting to spam and drown out real logs from bots and network scanners which are unaware of the domain and are trying to access port 80 and 443.
What have you already tried?

I've looked at the gopkg docs and past issues, I was unable to find out how to do this just yet.
Include any other information or discussion.

May 18 21:55:24 sudo[3730]: 2023/05/18 21:55:24 http: TLS handshake error from remote:port: no certificate available for 'public-ip-goes-here'

Bonus: What do you use this package for, and does it help you?

I've been using the package in a couple of places instead of a reverse proxy, because it's one less thing to configure when something needs to be on the Internet.
These logs are not required and spam application logs.

That error is emitted by the Go standard library.
In general, log consumers should simply ignore (or discard) logs that they aren't interested in, so "drowning out" problems are usually the result of a misconfiguration of a logging pipeline, or maybe even incompetent logging software that isn't capable of that.
That error is the result of CertMagic returning an error to tls.Config.GetCertificate(). If you're serving HTTP, it may be possible to customize the HTTP server's logger using http.Server. But since this log isn't emitted by CertMagic, we don't have control over it, sorry.

Interested in this as well; we're running into the same kind of logging issues. -- From what I could find, specifying the ErrorLog on the http.Server (if used) might help, but I haven't tried it yet in a production environment.
https://stackoverflow.com/questions/67047602/what-errors-are-outputted-to-http-servers-errorlog-struct-field

^ Yep, I think that should do it.
Note you will lose other (potentially helpful) log messages, this is a downside of turning off logging in this way. Recommended way to handle unwanted log messages is for your log ingestor to discard them.

In Caddy what we do is make an debug level logger named stdlib that we pass to http.Server's ErrorLog field: https://github.com/caddyserver/caddy/blob/38cb587e0f1b38db2c9fb422b4892e48753f00c0/modules/caddyhttp/app.go#L354 That way the logs are hidden by default, and only shown if users turn on debug level logging.

Thank you for the responses 🙏  - I'll take a look and report back what worked. This program doesn't use Zap yet, so will perhaps need to find a workaround for that.

You don't have to be using zap, just have a type that can fulfill the logger interface as per that server option.

@welteki could you provide an update when we have one please?
Thanks again folks :superhero:

certmagic.HTTPS() is a near-drop-in-replacement for http.ListenAndServe(), so you'll need to go down a layer of abstraction to customize the HTTPS server, i.e. create your own http.Server, pass in the listener; CertMagic has functions to give you listener and HTTP handler wrappers.


python3Packages.pyftdi: 0.53.1 -> 0.53.2

Motivation for this change
Update to latest upstream release  0.53.2
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129624 run on x86_64-linux 1

12 packages built:

expliot
home-assistant
python38Packages.adext
python38Packages.alarmdecoder
python38Packages.pyftdi
python38Packages.pyi2cflash
python38Packages.pyspiflash
python39Packages.adext
python39Packages.alarmdecoder
python39Packages.pyftdi
python39Packages.pyi2cflash
python39Packages.pyspiflash

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 129624 run on x86_64-linux 1

12 packages built:

expliot
home-assistant
python38Packages.adext
python38Packages.alarmdecoder
python38Packages.pyftdi
python38Packages.pyi2cflash
python38Packages.pyspiflash
python39Packages.adext
python39Packages.alarmdecoder
python39Packages.pyftdi
python39Packages.pyi2cflash
python39Packages.pyspiflash


Don't require dev-dependencies when not needed in certain cases

Specifically, when running cargo install and add a flag for this behaviour in cargo build.
This closes #4988.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks for the comments, I'll soon find some time to write tests and switch to -Z for the new flag.

Hey, I've added some tests and changed avoid-dev-deps to be a -Z unstable flag.
I wasn't quite sure what to rename require_optional_deps to, and it's a public struct so switching would carry some extra cost. I did add a comment to it, describing the new behaviour. I could change it if you want though, like require_extra_deps or require_unneeded_deps, let me know which you prefer.

:umbrella: The latest upstream changes (presumably #5063) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks for the update, tests look good to me! It's also ok to not worry much about naming, despite being public we bump the major version for Cargo regularly so it's not too critical to think too hard about APIs

I've resolved the merge conflict and dealt with all the remaining issues I think, please take a look and merge if good.

Ah looks like some tests may be failing?

Turns out the tests were also dependent on the fix for #5134, I've pushed a commit to work around it. Should be all good now.

@bors: r+

:pushpin: Commit d46db71 has been approved by alexcrichton

:hourglass: Testing commit d46db71b3ff17dfc0f4be6308c8b94613d65a572 with merge 879f48324d52c78322b6ad7513999b2a651c5023...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 879f48324d52c78322b6ad7513999b2a651c5023 to master...


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 11:18:29 on 2023.04.11
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 0, errors out: 0
Original problem ID: 9169
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=9169

Problem has been resolved in 9m 0s at 11:27:29 on 2023.04.11
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 9169


Dynamic route matching component error

Describe the bug
Before, my router file structrue is:
- dynamic
  - route.lazy.tsx
  - $id
    - route.lazy.tsx

But now, /dynamic/$id seems not working. It will match /dynamic route.
(Seems router navigation is correct, but match the lazy component of /dynamic. Maybe the problem is routerTree.gen file not generated correctly.)
I have to change my file structure to:
- dynamic
  - index
    - route.lazy.tsx
  - $id
    - route.lazy.tsx

Now I should match /dynamic route exactly.
Your Example Website or App
https://github.com/recallwei/dynamic-router-gen-recur
Steps to Reproduce the Bug or Issue
pnpm i
pnpm dev

Expected behavior
Expected to render the correct component.

Click dynamic-error/test button will render the component of dynamic-error route.
Screenshots or Videos
No response
Platform

OS: macOS 14.4
Browser: Chrome
Version: 122.0

Additional context
No response

This is expected.
In order to render a child match, a parent route must contain an <Outlet />.
Add it to src/routes/dynamic-error/route.lazy.tsx:
import { Outlet, createLazyFileRoute } from '@tanstack/react-router'

export const Route = createLazyFileRoute('/dynamic-error')({
  component: () =>
Hello /dynamic-error!<Outlet />

})


Hotfix/files extension

Fixing files extension & moving files to their corresponding folders

The Pull Request resolves Issue #2987

Description:
Several files did not have an extension; also there were files outside their corresponding folders. On the other hand, some files do not follow the guidelines for contribution because there are some files that do not start with a capital letter, but these changes cannot be detected by git so must be changed directly in the repository.

Also, I think would be great to add a configuration to work with a virtual environment for Python; I already added in the ignore file the default folder for the virtual environment (env)


Dependency causing conflicts don't appear in sbt dependencyTree

Here you can see the output of dependencyTree. The tree and then the "found version conflict(s) in library dependencies;".
Problem is the libs in conflict don't appear in the tree. example: org.typelevel:cats-free_2.12:1.1.0
sbt 1.5.5

@julienrf wrote over at #6618:

I think we can address your issue @adrian-salajan by showing a more complete dependency chain when sbt exhibits the conflicting dependencies. Currently, it only shows the conflicting modules, but sometimes these modules come from transitive dependencies, and it’s not easy to figure out where they come from. Instead, sbt could show the complete chain of dependencies that eventually pulled the conflicting modules.

Agree, this would be an excellent usability improvement.

Thanks for the report. Could you provide reproduction steps so we can reproduce your problem on our computers?