Has something been changed?

Hi, I have been training faces on Realistic Vision model for about a week and results were always good, but today something is wrong, it seems that after loading trained model to stable diffusion it just generates photos based on my instance images and prompts wont work as it was before. I thought that I overtrained it, tried again with 25 images and lesser steps same results, even tried with 10 pics and still the same. I usually train train face with 30-70 photos with 5e6 unet training steps and 1e6 Text Encoder Learning Rate, yesterday I tried with 2e-6 unet and 1e-6 text and results were amazing. Today it just seems broken, even tried to train face over another model, still the same. Anyone had the same problem?
Sorry for my eng :)

And what is this? First time see it

And what is this? First time see it

That's just the model downloading

Try the latest colab, ghd one you're using might be broken

I'm having the same issue today, even with an empty prompt I'm getting only variations of my instance images 🤷🏼♂

Try the latest colab, ghd one you're using might be broken

Well, I always used the latest verison, even checket it today. Tried again and same awfull results. Idk...

Try the latest colab, ghd one you're using might be broken

Hello!
The problem is from one of your updates from nowadays.
I know 100% that the commit "e59c0fc - fix RAM problem for good, thanks to @Daviljoe193" was working very good. I don't really know what you did after that, but yea, now the model gives only the used images in training (distorted and weird).
I think that the solution is very simple. Just revert everything to that last working commit.

Oh, yes RAM is fine now! Been having fun with merges again 👍

I'm having the same issue today, even with an empty prompt I'm getting only variations of my instance images 🤷🏼♂

Yes. I'm using the lastest colab and today I have the same problem, the prompt does not work properly. Only generates rare variations of the instance images.

Also having this issue fwiw.

As I said, the easiest solution (for Ben) will be to revert everything he did in the last 2 days.

Same exact issue on my end

Since I got mentioned here (I guess), I'll weigh in on this issue.
People say that the last "good" commit was the one where my suggestion for fixing the memory leak was applied. Let's see what happened with those 14 commits from between then and now, commit by commit. Note, I'm NOT a developer here, despite my mention in a few commits, I'm just a clown that happens to look like a developer.
Commit 1

Pretty easy here. The learning rate for UNet training was lowered from 5e-6 for 1000 steps to 2e-6 for 1500 steps. Realistically, this should reduce any weird artifacting, and reduce the likelyhood of overfitting.
Commit 2-3

Here isn't too hard to explain either. A tarball containing the dependencies for Dreambooth/Automatic1111 was updated to use Python 3.10, instead of Python 3.9, and while the old tarball is still there on Huggingface, there shouldn't be anything here that could break anything.
Commit 4-5

So these commits are related, therefore I have to cover them together. First, part of this just surpresses a pointless "warning" that "warns" that there's a cool shiny thing that we could use, but don't need. Second, appears to be (I'm just a guy, not the maintainer, or a Python expert) prefetching a dependency for the webui, and maybe some mild code cleanup, Again, nothing here could break anything.
Commit 6

That same prefetch from commit 5, but applied to the dedicated SD notebook.
Commit 7

More guesswork needed here. It seems that all this change does is tell the webui to not attempt to fetch the stuff that commit 5-6 already have gotten.
Commit 8

Commit 7, but for Dreambooth.
Commit 9

A fix for an error I could've sworn I've seen on the issues page before, but can't seem to find at the moment. 100% couldn't affect training, though.
Commit 10-12

Self explanitory, adds everything needed for ControlNet to work, and it's only applied to the Automatic1111 notebook, not the Dreambooth notebook.
Commit 13

Self explanitory, just fixes a problem with resuming training on 768 v2.x models on Dreambooth. Likely not what's causing people trouble here.
Commit 14

Uh, it's a 4 character change for what url to git clone. Nothing to see here.
Not trying to invalidate what people have said here about having trouble with overfitting, since I too have had a ton of trouble getting anything that's not either overfitted or ugly (Though I still don't fully get Dreambooth's settings), but nothing of significance has changed during the last 14 commits.

So what do you suggest? How do we train over custom models now? If it's always overfitting. I never had such problem since I started to use it. The last time I trained without any problem was 18feb, then on the next day it's all started

@Daviljoe193 Thank you for taking the time to break these down. The only knowledge I have in regards to most of this stuff is what I have managed to pick up attempting to run SD over the last month or two. As such I can really only speak to what I've experienced in regards to yield. Which after spending much of the day trying to recreate a model that I created just 3 days prior, with all controllable inputs being identical, I am yielding vastly different results. As with many others I am also getting mostly instance images being generated which I've never had happen before.
This is not to say you are wrong about the recent changes being benign only that there does seem to be a measurable difference in what is being produced.

Something was definitely broken.

Resuming training for SD2.1-768px based models was throwing an error.
Training on SD2.1-512px did not crash, but the result was terrible. The upper 'loss' values varied around 0.5-0.6, which is normal. But the lower 'loss' values were about 1e-4, which is very strange for first training.
ps: I recently ran the training again with the same dataset and parameters. The 'loss' value varies between 1e-1 and 0.7. Looks like the problem is fixed.

So I tried it again today with 1e6 text encoder, 10 photos. It was a mess. Tried it now with the same photos and 4e7 and it's fine

There is no standard settings for all datasets, you have to find the right settings for your dataset.

I think it's something with unet, it became more sensitive. Now I used 23 photos with 650 steps only, even text is 1e6 and it's ok, Usually I would use 2300steps for that..

Yes it became more efficient, so you don't need 4000 steps to train on a single subject

Yes it became more efficient, so you don't need 4000 steps to train on a single subject

Oh, If only you said that earlier 😄 Where we can read about those updates?

I have finally made it to work as before.
Check out this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb
Took me some good hours to make it to work, because I'm no coder. Used my logical thinking to revert anything to a stable version. I don't know how to fork a certain commit, so I downloaded the older commit and uploaded on a new repository and modified everything accordingly.

Thanks mate!, going to test it today

I have finally made it to work as before. Check out this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb Took me some good hours to make it to work, because I'm no coder. Used my logical thinking to revert anything to a stable version. I don't know how to fork a certain commit, so I downloaded the older commit and uploaded on a new repository and modified everything accordingly.
Edit: No more distortions and weird images. Add the name of the .jpg files in the prompts to get more characteristics of your character.

THANKS!!! IT WORKS AMAZING! Finally, I was trying everything. But the new version always deform faces even with low steps.

Hi, I have been training faces on Realistic Vision model for about a week and results were always good, but today something is wrong, it seems that after loading trained model to stable diffusion it just generates photos based on my instance images and prompts wont work as it was before. I thought that I overtrained it, tried again with 25 images and lesser steps same results, even tried with 10 pics and still the same. I usually train train face with 30-70 photos with 5e6 unet training steps and 1e6 Text Encoder Learning Rate, yesterday I tried with 2e-6 unet and 1e-6 text and results were amazing. Today it just seems broken, even tried to train face over another model, still the same. Anyone had the same problem? Sorry for my eng :)

This has been happening to me as well. I trained my model despite working fine a few days ago. These are my results now.

Guys, what are your base models?
With SD1.5, it probably still works correctly. With SD2.1 (512 or 768), it gets terrible results.
Moreover, the larger the dataset size, the worse the resulting model. The picture I gave above is generated by a model trained on 180 photos.

Hi, I have been training faces on Realistic Vision model for about a week and results were always good, but today something is wrong, it seems that after loading trained model to stable diffusion it just generates photos based on my instance images and prompts wont work as it was before. I thought that I overtrained it, tried again with 25 images and lesser steps same results, even tried with 10 pics and still the same. I usually train train face with 30-70 photos with 5e6 unet training steps and 1e6 Text Encoder Learning Rate, yesterday I tried with 2e-6 unet and 1e-6 text and results were amazing. Today it just seems broken, even tried to train face over another model, still the same. Anyone had the same problem? Sorry for my eng :)

This has been happening to me as well. I trained my model despite working fine a few days ago. These are my results now. This is supposed to be a tiger eating fish in a jungle.

Just use less unet steps and text, it became more sensetive somehow

In the last example, I used only 30 steps per image. The result is still terrible.

Guys, what are your base models? With SD1.5, it probably still works correctly. With SD2.1 (512 or 768), it gets terrible results. Moreover, the larger the dataset size, the worse the resulting model. The picture I gave above is generated by a model trained on 180 photos.

I'm using 2.1 (768) and its getting trash results, however 3 days ago it was perfectly fine with the same database and training.

Hi, I have been training faces on Realistic Vision model for about a week and results were always good, but today something is wrong, it seems that after loading trained model to stable diffusion it just generates photos based on my instance images and prompts wont work as it was before. I thought that I overtrained it, tried again with 25 images and lesser steps same results, even tried with 10 pics and still the same. I usually train train face with 30-70 photos with 5e6 unet training steps and 1e6 Text Encoder Learning Rate, yesterday I tried with 2e-6 unet and 1e-6 text and results were amazing. Today it just seems broken, even tried to train face over another model, still the same. Anyone had the same problem? Sorry for my eng :)

This has been happening to me as well. I trained my model despite working fine a few days ago. These are my results now. This is supposed to be a tiger eating fish in a jungle.

Just use less unet steps and text, it became more sensetive somehow

I will just rollback a few days I don't want to figure out this new way to train models.

Test this colab from @Bullseye-StableDiffusion - https://colab.research.google.com/github/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb Just testing it right now

I just using the revision history

Guys, what are your base models? With SD1.5, it probably still works correctly. With SD2.1 (512 or 768), it gets terrible results. Moreover, the larger the dataset size, the worse the resulting model. The picture I gave above is generated by a model trained on 180 photos.

This wont help at all sense my dataset is 6gb

Yes it became more efficient, so you don't need 4000 steps to train on a single subject

Could you please specific what the settings should be now and was then?

will fix soon

will fix soon

Oh I thought you said this was an intentional feature. Sorry

@InternalMegaT it is supposed to make the training more efficient, but it appears it makes it instable in some cases, so I reversed back a few deps, it should be all good now.

@InternalMegaT it is supposed to make the training more efficient, but it appears it makes it instable in some cases, so I reversed back a few deps, it should be all good now.

Got it. May I ask' what was it supposed to do?

you can get results with 500 steps instead of 4000

I have trained faces with the Realistic Vision model.

could you please tell me how do you use this model? when i try to load it i get a conversion error. thx

So has there been a change or just new lower defaults?

I have finally made it to work as before. Check out this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb Took me some good hours to make it to work, because I'm no coder. Used my logical thinking to revert anything to a stable version. I don't know how to fork a certain commit, so I downloaded the older commit and uploaded on a new repository and modified everything accordingly.
Edit: No more distortions and weird images. Add the name of the .jpg files in the prompts to get more characteristics of your character.

THANKS!!! IT WORKS AMAZING! Finally, I was trying everything. But the new version always deform faces even with low steps.

Thanks for confirming! And yes, that problem from the newest version was kinda annoying.

So has there been a change or just new lower defaults?

It seems that only 2 settings are changed in the latest notebook:

UNet training steps = 1500 now and 500 before
Text Encoder training steps = 350 now and 100 before.
I don't really think that this actually solves the problem, but you can try.

Why are my results so bad still? I'm using the same settings as I did before. Its its not working well anymore. Its not my database thats the problem.

Why are my results so bad still? I'm using the same settings as I did before. Its its not working well anymore. Its not my database thats the problem.

Try this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb.

Why are my results so bad still? I'm using the same settings as I did before. Its its not working well anymore. Its not my database thats the problem.

Try this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb. Reverted everything as it was like 2-3 days ago and it's working very good with the default settings.

Why are you keep saying that? I already tried that. Still the same thing.

Why are my results so bad still? I'm using the same settings as I did before. Its its not working well anymore. Its not my database thats the problem.

Try this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb. Reverted everything as it was like 2-3 days ago and it's working very good with the default settings.

Why are you keep saying that? I already tried that. Still the same thing.

Then I don't know. The problem may be from your training dataset, settings, prompts etc..

Why are my results so bad still? I'm using the same settings as I did before. Its its not working well anymore. Its not my database thats the problem.

Try this notebook: https://github.com/Bullseye-StableDiffusion/fixed/blob/main/fast_DreamBooth_fixed.ipynb. Reverted everything as it was like 2-3 days ago and it's working very good with the default settings.

Why are you keep saying that? I already tried that. Still the same thing.

Then I don't know. The problem may be from your training dataset, settings, prompts etc..

It worked fine before, I didn't change anything.

@Omenizer I changed the transformers version back to an older one

@Omenizer I changed the transformers version back to an older one

Just tried it, much better now!  🥳🥳🥳

@Omenizer I changed the transformers version back to an older one

Works great. Thank you for all the time and effort you put into this man.

will fix soon

It is working fine again. Thank you for the effort. Great work!

@Omenizer I changed the transformers version back to an older one

It is working fine again. Thank you for the effort. Great work!


Support for active ledger transports

This PR includes supporting two additional types of transport for Ledger and refactoring UI for Sign In With Ledger functionality.
Additionally handled scenarios:

when a user clicks send and there is no ledger connection yet - in this case, Global Alert is shown
when a user for example disconnected the device in a case where the connection was previously established - in this case, Global Alert is shown

Additionally rebuilt Sign In With Ledger components into a new structure.

@MaximusHaximus Please check this PR if you have a moment. I was able to handle a few additional situations and also refactored the Sign in with ledger components. With this implementation, we will be able to get rid of store usage in wallet.js I have an idea how exactly it will look like and I will handle it in a separate PR.
@corwinharrell
This is a fully functional version for new ledger transport and Sign in With Ledger UI. There are a few topics that I would like to discuss with you:

Connect Ledger button and screen.
If you check the Figma designs, we have a screen MacBook Pro 9 and a set of modals below. I'm not sure, but my idea is that this screen should be visible after the user clicks Connect Ledger button, because we need him to see additional information and we want him to confirm.

If so, then there are a few things:

the way establishing the connection is working, we don't need any modals, because it's happening instantly and the user doesn't have to confirm it on the device
the only situation that we would need modal is a case when the connection was unsuccessful
but there is one problem with MacBook Pro 9 screen - for example, the user is trying to send tokens, he puts the amount, chooses a token, enters the receiver id and then he's clicking Send button - now we are redirecting him to the MacBook Pro 9 screen - we can redirect him back to the Send screen after successful connection but the user would have to enter all data again.

The way I implemented it now is that there is no additional screen, if a user clicks Connect Ledger the connection is established. And if for example ledger device is not connected user will see Global Alert.
I think that a good solution would be to add a small additional window that will appear near the Connect Ledger button (the same type of window as we use for the account submenu) where we will be displaying all additional information - so it will be used for these situations:

the user clicks Connect Ledger and we want him to read additional information and confirm - in this case, a small window will appear with the information and the button to confirm
user is trying to perform an operation (Send, Sign in, etc) but the connection is not established yet - in this case, we can show him the same window with the information that there is no connection and a button to confirm a connection
connection is already established but a user disconnected the device fiscally - in this case, we will show the window with the information that the device was disconnected and he needs to connect it again.

Such a small window will not interfere with wallet functionality and will be used for several situations instead of global alert.

Importing accounts.
If you check MacBook Pro 4 screen - in this case, we are showing the list of accounts as the page and there is a set of modals below. The way I understand we want to show these modals while the user is accepting the accounts.

But the problem is that the accounts are accepted one by one, the user is accepting the first account, then we are showing the loader inside the modal, and after that, we instantly starting with next account, so in practice, these modals will be visible constantly, so the user will almost never see the list of accounts behind modals.
The way I implemented it now is that we don't use modals at all in Sign In With Ledger flow. Please check how it looks in this PR. When it comes to the screen with a list of account, all informations are visible there - the user sees when the account is being processed, that he needs to confirm on the device, was it a success or not, and which one will be next so it seems like modals are not needed.

There was an idea to add a possibility that the user will be able to choose which account he wants to import - now he can decline the account, but in a case where a user has few dozens of accounts and wants to import only one, he would have to decline all of the accounts manually. The idea was to add checkboxes for each account and options "select/unselect all" so it will be much easier. I can implement it with this PR, it would just need some small additional changes to the design.

There is also a window that appears if we didn't find any accounts - then the user is able to enter accountId manually. I prepared it using the same design but please check if it looks ok.

I suggest we split this PR out to two parts.  We need to index on getting the functional blockers that block chrome users from using ledger.  The UX approach can be revisited in a separate PR.  Let's make it work, then make it better.

@agileurbanite that's fine. Once @marcinbodnar and I have synced up and we've aligned on the things that need to be addressed, I'll leave it up to him to determine how best to divide the work in order to get a functional implementation out the door. He can then to follow up with any improvements to the UX if there are any that are considered supplemental and not critical.

@marcinbodnar checking out the latest preview link on Chrome:

using a ledger to create an account works but doesn't dismiss modal after account is created (still shows Processing...)
when adding ledger recovery method, if the user rejects when adding a key the modal doesn't dismiss
when recovering with ledger, the option is given to change the HD path: changing the path prevents recovery from working, even if you go back and try again... it only starts working if you clear localStorage

The only other thing I noticed is that we've dramatically increased the amount of logging we do to the console. Can we put this behind a debug flag or remove the logging please? Too much noise in the console makes it harder for users to submit relevant information when they encounter errors in the wallet, which in turn makes it more difficult for us to get a handle on errors that are written to the console.

@andy-haynes

using a ledger to create an account works but doesn't dismiss modal after account is created (still shows Processing...)

Done

when adding ledger recovery method, if the user rejects when adding a key the modal doesn't dismiss

Done

when recovering with ledger, the option is given to change the HD path: changing the path prevents recovery from working, even if you go back and try again... it only starts working if you clear localStorage

So I tested it and seems to work well, probably happens in specific scenario - could you please try to reproduce it?

The only other thing I noticed is that we've dramatically increased the amount of logging we do to the console. Can we put this behind a debug flag or remove the logging please? Too much noise in the console makes it harder for users to submit relevant information when they encounter errors in the wallet, which in turn makes it more difficult for us to get a handle on errors that are written to the console.

You're right. Done.

So I tested it and seems to work well, probably happens in specific scenario - could you please try to reproduce it?

I can still consistently reproduce it. While it's not a typical user flow, anything we break that requires clearing localStorage to fix is very bad.

So I tested it and seems to work well, probably happens in specific scenario - could you please try to reproduce it?

I can still consistently reproduce it. While it's not a typical user flow, anything we break that requires clearing localStorage to fix is very bad.

Great catch. Adding a comment to #2488 about it.


Fix typo

The site makes mention of 'ready-to-use'.

Codecov Report

Merging #410 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #410      +/-   ##
=========================================
+ Coverage   64.76%   64.8%   +0.04%
=========================================
  Files         116     116
  Lines       11697   11697
=========================================
+ Hits         7575    7580       +5
+ Misses       3925    3920       -5
  Partials      197     197

Impacted Files
Coverage Δ

devices/lepton/lepton.go
88.41% <0%> (+1.21%)
:arrow_up:

host/sysfs/thermal_sensor.go
83.84% <0%> (+2.3%)
:arrow_up:

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 2b0e8ad...b3494b6. Read the comment docs.

Thanks!


IoTCoreDefaultApp Language Setting does not change input locale

IoTCoreDefaultApp Language Setting does not change input locale. For example the second part of the OOB Expirience is WIFI Network Password. All the UI there is presented in German language to me but I have to type in the Password as if i had an english Keyboard.

@PatGet have you performed a reboot after changing the Language Settings; I had something similar with regards to date/time region; and it's because of the system/thread culture change isn't detected by the app.
A restart stored it for me

https://github.com/ms-iot/samples/issues/147

This locale change issue is a problem with the OS currently that we are working to fix.

We have added Keyboard layout setting in the IoTCoreDefaultApp. Skip OOBE and then goto settings to change the keyboard layout.


utils.ReadTextFile doesn't properly read certain files

// utils.ReadTextFile doesn't properly read certain files, e.g. UCS-2LE BOM (that's what notepad++ reports)
// TESTS:
// a) Test with a file written using utils.WriteTextFile: all OK
var t1 = [{"test1": "test1"},{"test2": "test2"}];
var p1 = fb.ProfilePath + "Test.json"
utils.WriteTextFile(p1, JSON.stringify(t1))
console.log("p1", "!!!" + utils.ReadTextFile(p1) + "!!!");
console.log(JSON.parse(utils.ReadTextFile(p1)));
// b) Test with a file written using the original text save method: problems:
// 1) If it's a json file, JSON.parse fails & crashes
// 2) If it's a text file, extra unwanted line at end
// 3) Effects any static json or text files written with the original text save method & not since updated
// 4) It seems that utils.ReadTextFile in SMP doesn't properly handle UCS-2LE BOM which the original text save method seems to create
// 5) All OK in JScript panel
var t2 = [{"test3": "test3"},{"test4": "test4"}];
var p2 = fb.ProfilePath + "Test_.json";
var fso = new ActiveXObject("Scripting.FileSystemObject")
// original text save method:
var save = function(l, n) {try {var ts = fso.OpenTextFile(n, 2, true, -1); ts.WriteLine(l); ts.Close(); return true;} catch (e) {return false;}}
save(JSON.stringify(t2), p2);
console.log("p2", "!!!" + utils.ReadTextFile(p2) + "!!!");
console.log(JSON.parse(utils.ReadTextFile(p2))); // CRASH in SMP (JSP OK)

Just FYI: UCS-2LE BOM is actually UTF-16LE BOM =)

Does it actually crash for you or is it just a SMP error like JSON.parse: unexpected non-whitespace character after JSON data at line 2?

Yes the example above gives a full crash + that message:
Error: Spider Monkey Panel v1.0.3 ({735AA13A-75AB-4867-9E30-B396829F70BD})
JSON.parse: unexpected non-whitespace character after JSON data at line 2 column 1 of the JSON data
File:
Line: 20, Column: 13
Stack trace:
@:20:13

Anyways, I've fixed it. You can try it out by using latest build.


Made the following changes in sanity script

OVS Offload port creation for SR-IOV in Sanity script
Deleted the ceph-osd-config.yaml.director

What is this PR for?  What is the Jira story/test case that this covers?

What is this PR for? What is the Jira story/test case that this covers?

The SRIOV port creation command is different in case of OVS Offload enabled. This fixes the issue.  The Jira story is CES-10493 (https://jira.dellemc-community.org/browse/CES-10493) which is now in Testing on the Jira board.

Has this been tested?  Can you test this first with Offload disabled and make sure it works then enable it and test it again?

Has this been tested? Can you test this first with Offload disabled and make sure it works then enable it and test it again?

Yes I have tested this and the offload port is created only when SR-IOV offload is enabled in the sanity.ini file.


Removed boilerplate code

:tada: This PR is included in version 1.0.1 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Using http_proxy from the env within the plugin

I am trying to set-up a VSCode connection using SQLTools to connect to Athena.  This is connected to issue # 36 ( https://github.com/kovihq/sqltools-athena-driver/issues/36 )
This is my set-up

I have http_proxy defined and also tried setting up proxy in VSCode - but get the same error

When working with a colleague of mine when he also was trying to connect - we came to this conclusion
it is definately a proxy issue though, you wont be able to connect to the athena endpoint, without a hardcoded proxy in VScode, and that would explain all 3 scenarios: 1 - off network (no proxy needed) works, 2 - on-network terminal (proxy configured) works, 3 - on-network vscode, cannot conenct to the Athena endpoint
He also looked at the extension code and said 'ask for the plugin developer team  to use http_proxy settings from VSCode'

Sorry, pressed submit too soon.  Is is possible to add it to the extension, so it could be used when a proxy needs to be defined?
Thank you


iOS Hyperion app needs updating for Hyperion.ng

@ChaosCoder
We're getting close to feature complete with Hyperion.ng and the app will need some small changes:
Priority 0 is banned
Colour tuning model is all changed

No at all...

We have even no docu, now it's -not- integration time...
We came back to use when it's really time..

Yeah, would be nice to have a summary on all the changes that are required (like what is replaced by what, etc.). I will integrate the needed changes ASAP as I know what changed. What version number will hyperion.ng have, as in how can I detect I'm communicating with the new version?


[BUG] line numbers are broken in code blocks

Describe the Bug
When user has a long code block and scrolls it up and down, then line numbers behave incorrectly, suddenly changing by themselves.
Also there are no line numbers in Reading Mode.
Relevant Screenshots
Screen recording: https://github.com/subframe7536/obsidian-theme-maple/assets/3062744/8cc2c5cd-0db7-4522-a0ab-b3ac5aed3442
To Reproduce
Steps to reproduce the behavior, e.g.

Create a note with long code block
Enable code block line numbers
Scroll the note up and down

In which mode does the bug occur?

[ ] Reading/Preview Mode
[ ] Source Mode
[x] Live Preview

Which editor are you using?

[ ] Legacy Editor
[x] New Editor

Which version of Obsidian are you using?

[x] Public Release (Desktop)
[x] Public Release (Mobile)
[ ] Insider Release (Desktop)
[ ] Insider Release (Mobile)

Checklist

[x] I have updated to the latest version of the theme (Obsidian Settings → Appearance → Themes - Manage → Update).
[x] I made sure that the issue is related to this theme, by checking that the issue does not occur with other themes or the default theme.

Yeah, it is a known issue.
According to element tree, there is no way to acchieve these feature


Whats wrong?

I am looking for library for my project, I have tryed to setup this lib for 3 days, and there are no success. I installed chart lib instantly. I asked for help, because I cant even try to work with your lib.
3 issues in charts "issues asks" ,that mean people asking about it.
You are strange man.

I asked you in the other thread for your pod version and you said 1.2.0. This is not the pod version. It's the Swift version. I assume you're using a very old version of SwiftCharts that doesn't have the fix.
So far I'm seeing in the commit history, you should be fine using the Swift 1.2 branch:
pod 'SwiftCharts', :git => 'https://github.com/i-schuetz/SwiftCharts.git', :branch => 'swift1.2'

But I seriously recommend you upgrade your code base to Swift 2.3+ Nobody uses Swift 1.2 anymore and as mentioned in the readme, this is also not maintained here anymore. I can't even test since my Xcode will not compile Swift 1.2.

1.2.0 its current pod version, not swift, I am using latest xcode and swift 3.
Ok, in mondey I record the video of my steps of installing the library, to finds out wat is wrong.

Ok, sorry for misunderstanding. I assumed it was obvious that I was asking about the SwiftCharts (the pod) version and not the cocoapods version.
Can you upload the project where you are having these issues to your Github? I'll give it a look.

Ok, but my macbook in office, I will upload it to github in Monday, thanks for help:)

https://github.com/rolyanos/newChartTest
Hello, this is my project.

The problem is that ExampleDefaults is not a part of SwiftCharts, but of the example project. This is just a helper class to store the shared settings for the examples.
Either simply copy ExampleDefaults into your project, or pass your own fonts and settings.
Besides of this you seem to have copied the example from master, which isn't compatible with 0.5.1. You have to use the example code from 0.5.1.

Ok, I will try to do it, thanks a lot!


branch-line-manager

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: branch-line-manager
Document link: _index.md
My Issue/Suggestion
Could you add information about the button "Switch branch"? I wonder what is does.

Hi @axelenexis thanks for the question and feedback. We don't have a Switch branch button on the dialog box, so can you please clarify what you're looking for information on? Thanks.

It's in the screenshot in the mentioned page https://docs.mendix.com/refguide9/branch-line-manager-dialog/ .

Oh I see, sorry, I will take a look for how to document

Hi @axelenexis this is documented here: https://github.com/mendix/docs/pull/6262
Thanks for the feedback and helping to improve Mendix docs! Let us know if you have any other questions.
Cheers,
Adam


[C++][FS][Azure] Implement Move()

Describe the enhancement requested
It's not implemented yet.
This is a child of GH-18014.
Component(s)
C++

This is a potential complication with move. Azure blob storage does support efficient server side only moves.... but only if using SAS token authentication for the source. Alternatively everything will need to be downloaded and re-uploaded, but that should work with any authentication.

Maybe copy and then delete? That is at least how I have solved it previously with adlfs and Azure 😅

I don't think I was thinking very carefully when I posted that comment.
I think move is efficiently supported with hierarchical namespace. E.g. using https://github.com/Azure/azure-sdk-for-cpp/blob/4a32d7266cfac8bfc0eb87feb56011361a36f43c/sdk/storage/azure-storage-files-datalake/inc/azure/storage/files/datalake/datalake_file_system_client.hpp#L239
What I was thinking of was the copy blob by URI which has to use SAS token auth. But if we're only copying within one storage account we don't need to use that https://learn.microsoft.com/en-us/rest/api/storageservices/copy-blob?tabs=microsoft-entra-id#authorization. So I do think there is any problem 🙂

@kou if you haven't started to work on this one I'm happy to take it.

I haven't. Please do it!

felipecrv - Did you begin work on this one?

felipecrv - Did you begin work on this one?

Yes. I'm very close to sending a PR.


WIP : Find journey by headsign

In this PR I will add stuff related to vehicle journeys and stuff to call routes likes this : https://api.navitia.io/v1/coverage/fr-ne/vehicle_journeys/?headsign=843118

https://goreportcard.com/report/github.com/govitia/navitia check last items


Duplicate Detection Check

Hi, I tried Duplicate Detection Check Action on Account record and got following message:
An unexpected error occurred from ISV code. (ErrorType = ClientError) Unexpected exception from plug-in (Execute): UltimateWorkflowToolkit.CoreOperations.Common.CheckForDuplicates: System.NullReferenceException: Object reference not set to an instance of an object._

What to do next?
Thanks!
Zvjezdana

Hello Zvjezdana,
Can you open solution, navigate to "Solution Configuration Page" and click "Save" and retry workflow to run?
Thanks,
Andrew

It worked! Thank You Andrew on fast reply.


Allow redirect matching to include/exclude query string parameters when matching requests

Previous behaviour always added the query string to the request path before regex matching was performed. This behaviour has been revised so that by default the query string is not appended to the path - with the option to include the query string by using the new 'includeParams: {true|false}' configuration option.
In order to facilitate testing, the worker script has been converted to ES6 module format to facilitate unit testing with Miniflare and Jest. Worker unit tests have also been added.
@BigBlueHat Please review whether this is suitable for PR, or whether modification of the changes are required.

@alunwcom I'm getting SyntaxError: Cannot use import statement outside a module which may be why this is still a Draft?

@alunwcom I'm getting SyntaxError: Cannot use import statement outside a module which may be why this is still a Draft?

@BigBlueHat Sorry - I need to update the README here, as to run the tests you need to declare an environment variable for ES6 modules...
> $env:NODE_OPTIONS="--experimental-vm-modules npx jest"

or equivalent statement on *nix

If it's required to run the tests, you could just put it into npm run test. There should be a cross-platform friendly way to do the options stuff...I think you just use the all caps key/value thing without $env.

@BigBlueHat have addressed the issues raised so far ;) ... anything else?

@BigBlueHat I think you'll have to merge this - I don't appear to have access to do so....

@BigBlueHat I think you'll have to merge this - I don't appear to have access to do so....

You do now. 😁


Simplify element equivalence interface

This changelist simplifies the interface of the Element::isEquivalent method in MaterialX C++, aligning it with the behavior of Element::validate and proposed use cases for element equivalence in JavaScript.

Thanks for putting up this simplification. Looks good to me.


ci: move off of txdx

Evict the project from txdx and switch the build process to use vite or rollup

Scheduling for while I'm gone. This is the first step to automating the publishing of all OSS UI components.


Switch to GitHub Actions

So that we can have pre-compiled binaries for all 3 platforms

Done in #39

@dmerejkowsky, could you be so kind to update a binary for Windows?

I think all we have to do is make a 0.5.0 release - GitHub Actions is set up and should automatically generate binaries for Linux, macOS and Windows.
But I'm no longer part of the TankerHQ GitHub organisation so I can't do it just yet.

@dmerejkowsky
I think all we have to do is make a 0.5.0 release
But I'm no longer part of the TankerHQ

Dear @AlexandreBossard, @blastrock, @jeremad, @maximerety, and @theodelrieu, we need you assistance here, I guess.

Done - expect a new release soonish :)

Err… Sorry to bother you 10 days later, but is there any progress?
I am looking to update Windows binary 0.4.1 to something fresher, because of the controversial pattern highlighting as follows
Source: in.txt
$ ruplacer.exe "(?:<font.*ffffff.*>)([\s\S]*)(?:</font>)" $1 in.txt


Date format looks bad when single digit day

Originally created by Tom Smyth at 2015-08-30 13:00, migrated from redmine-#2993

Status changed from New to Closed

By Tom Smyth at 2015-08-30 18:03


Issues related to graph injection attacks on NIPA

Hello author, I attempted to run the NIPA injection attack in the deeprobust library, but while reviewing the source code, I found some issues in the "deeprobust/graph/rl/nipa_env. py" file (as shown in the following figure). The variable N represents the number of nodes before injecting malicious nodes, self.all_nodes represent all nodes in the original graph. May I ask if self.injected_nodes represents the injected malignant nodes? If so, it should not take the index from the nodes in the original graph, is that correct?

Hi,
The input adjacency matrix already contains the injected nodes. Please see the file bleow.
https://github.com/DSE-MSU/DeepRobust/blob/fb047ac918ca3525749b8d091898fc5670752e74/examples/graph/test_nipa.py#L60-L62

Thank you very much for your answer! The input adjacency matrix does indeed include injection nodes, but at this point, the degree of the injected nodes is 0 because the injection nodes have not yet been connected to the nodes in the original graph (the "injecting_nodes" function in the "deeprobust/examples/graph/test_nipa. py" file seems to have only changed the shape of the initial adjacency matrix). However, the variable "N" in the "deeprobust/graph/rl/nipa_env. py" file represents the number of nodes with a degree greater than 0, which is the number of nodes in the original graph (the degree of all nodes in the graph I chose to attack is greater than 0), and the variable "self. all_nodes" should represent the index of all nodes in the original graph. I also debugged the code, which confirms my point.

In the above figure, I printed the shapes of the feature matrices before and after injecting nodes. Before injecting nodes, the number of nodes was 2708, and after injecting nodes, the number of nodes was 2735. Therefore, the index of the injected nodes should be 2708-2734. However, I checked the index stored in the variable "self injected nodes" in the "deeprobust/graph/rl/nipa_env. py" file, and it looks like this:

The value of the variable 'self. inputted_nodes' is actually stored between 2681 and 2707.

Hi,
The input adjacency matrix already contains the injected nodes. Please see the file below.
https://github.com/DSE-MSU/DeepRobust/blob/fb047ac918ca3525749b8d091898fc5670752e74/examples/graph/test_nipa.py#L60-L62

Hi,
The input adjacency matrix already contains the injected nodes. Please see the file below.
https://github.com/DSE-MSU/DeepRobust/blob/fb047ac918ca3525749b8d091898fc5670752e74/examples/graph/test_nipa.py#L60-L62

Hello, could you please reply to me?

Hi mmdhhhh,
Thank you for pointing out. I agree that there is a bug in this file. Can you help me fix this issue and submit a pull request to this repo? I appreciate your help.

Thank you for your answer！
I will submit a pull request soon.


Cannot read text from search input field

Hmm js errros are too complex cant fetch the inputted text.

New merge fixed it but with the cost of some good improvements


Update servicingPlan.json to 4.5.2

Coverage remained the same at 65.147% when pulling 9a4543578f3eea077e2d3a8ced254905effa3b1f on compulim:bump-plan-4.5.2 into e4f795ae3565a4c1d0bc654042dc15eb26fcbb3c on microsoft:master.


Some twitch streams not loading video.

http://www.twitch.tv/marioman847/popout
For some reason the video from this stream won't load at ALL in Livestreamer, even when I updated Media Player Classic - Home Cinema. Other streams load just fine, just not this one. I'm wondering if it has something to do with his encoding? His highlights load just fine, but not the actual stream.

Try one of the workarounds listed here. If that doesn't help, try another player.

This also happened to me when trying to watch an mlg livestream. It only occured when selecting anything else but best and worst as the quaity. But yeah, the workaround works.


error: Could not compile pear_codegen.

The whole 'docker-compose up' log can be found here:
https://ghostbin.com/paste/ge55e
On localhost:3000 I can see a React logo spinning, including the following message:
"Proxy error: Could not proxy request /api from localhost:3000 to http://api_server:3001/ (ECONNREFUSED)."
I have tried running 'docker-compose build --no-cache api_server', but it still resulted in the same errors above.
Tried on both a Debian and Ubuntu computer, same results.

This looks like the result of a newer version of rustc removing the quote feature.  Check again with the latest version.  Now with Rocket 0.4! Thanks to @huntwj


gamepad support (web only)

This is not integrated with unrust yet as it requires a refactoring of FirstPersonCamera which relies only on keyboard right now. You can check usage in the basic example.
I've tested it with an XBox gamepad on firefox and chrome. Support to more pads/browsers/os might need some tweaks.

Awesome, and looks good to me, but i do not have a gamepad right now, so i would test it next week, but i think i would merge it for now, so do you want to implement the native side first, or merge it now and add back the implementation later ?

You can merge it. For the native side, I think we have few choices. SDL2 seems almost mandatory. And if we're using SDL2 for gamepad, we could as well dump glutin and use SDL2 for everything native. This is a huge change that shouldn't occur  in 0.1.1 imo


build(deps): bump jackson to 2.14.2

There's been a couple new releases of both Jackson & Rewrite, removing the need for this fix (unless I'm mistaken!).

@timtebeek Aren't we on Jackson 2.15 already?

@timtebeek Aren't we on Jackson 2.15 already?

I think so indeed; hence why I closed this older PR. No need for action at this point I believe.


az help text

When there are mistakes in uses of the az cli, the output often comes out in a way that is not too helpful.  As a random example, see the below.  It looks like the output is made by printing the basic command+subs and then the list of arguments with some automatic line wrapping.  The thing is that the commands tend to be very nested and therefore very long, so there's little room for the argument list.  In particular, note that in my example the destination and source mandatory arguments are wrapped in a very confusing way.
It would be much better if the arguments were

specified at a fixed indentation from the left side (or maybe following some "arguments:" header
listed with only one argument per line
listed in a better & consistent order, eg, mandatory arguments first, optional ones next, and finally generic ones (-h / --output / etc)

(As a side note, this style of command-line is very fitting for someone who is coming from a powershell background, but the overall design is weird in a linux world: like the lack of non-flagged ("positional") arguments, and the flags that have multiple values as in the --metadata below.)
$ az storage blob upload-batch --destination "https://...windows.net/.../foo/" --source "..."
usage: az storage blob upload-batch [-h] [--output {json,tsv,table,jsonc}]
                                    [--verbose] [--debug] [--query JMESPATH]
                                    [--lease-id LEASE_ID]
                                    [--account-key ACCOUNT_KEY]
                                    [--content-language CONTENT_LANGUAGE]
                                    [--connection-string CONNECTION_STRING]
                                    [--account-name ACCOUNT_NAME]
                                    [--max-connections MAX_CONNECTIONS]
                                    [--if-modified-since IF_MODIFIED_SINCE]
                                    [--pattern PATTERN] --destination
                                    DESTINATION [--type BLOB_TYPE] --source
                                    SOURCE [--validate-content]
                                    [--content-encoding CONTENT_ENCODING]
                                    [--content-md5 CONTENT_MD5]
                                    [--if-match IF_MATCH]
                                    [--content-disposition CONTENT_DISPOSITION]
                                    [--metadata METADATA [METADATA ...]]
                                    [--dryrun] [--if-none-match IF_NONE_MATCH]
                                    [--maxsize-condition MAXSIZE_CONDITION]
                                    [--sas-token SAS_TOKEN]
                                    [--content-type CONTENT_TYPE]
                                    [--content-cache-control CONTENT_CACHE_CONTROL]
                                    [--timeout TIMEOUT]
                                    [--if-unmodified-since IF_UNMODIFIED_SINCE]
az storage blob upload-batch: error: incorrect usage: destination cannot be a blob url

cc/ @johanste

//CC @chewong, a bit different from #3994 though

add to S165.

As discussed in https://github.com/Azure/azure-cli/issues/10676#issuecomment-536253290, Azure CLI uses Python's argparse for the help text, so we have little control over the format if we don't override that. You may always use -h to view better-printed help text.
As for positional arguments, they are not supported by Azure CLI. All argument needs to be keyword arguments. The only exception is az find:
https://github.com/Azure/azure-cli/blob/5edbd822ee860d7ba5d5182811b68233e0846ba8/src/azure-cli/azure/cli/command_modules/find/_params.py#L9


feat: Support multiple namespaces on a XML Document

Fixes: #266

Hi @zregvart , here is a SchemaSet inspection input used for unit test.
https://github.com/atlasmap/atlasmap/pull/272/files#diff-a573f917c8175dba539006c9b5b74ba2
None of xs:import or xs:include is used. The inspector just retrieves all the xs:schema element from the file, and take the /atlas:SchemaSet/xs:schema as a root schema. The namespace prefixes are added if it's different from the target namespace of the root one.

@igarashitm awesome!


OpenDTU integration does not work

Hello,
I have a problem with the integration of my OpenDTU into the Victron system. It had worked before, but after a few settings it was gone again. I reinstalled everything according to the instructions, but still nothing. Finally, I reset my Cerbo GX completely to factory settings and set it up again, including the dbus-opendtu, but still nothing. I also completely reinstalled OpenDTU on the ESP32, but it just doesn't work anymore. OpenDTU itself works (can access it and get the data), but it just won't appear in the Victron system.
I have checked the IP in the config.ini several times and it matches. I then entered the command dbus-spy, but I don't see OpenDTU there either.
With the command svstat /service/dbus-opendtu I get values between 1 and 3, which indicates an error. The current.log then says the following:
line 1: 2023-03-19: command not found
line 2: syntax error near unexpected token `most'
line 2: 'Traceback (most recent call last)'.
I don't know what this means. I am a newcomer to Linux and have only ever followed the instructions. Can someone help me?
Die config und json Datei habe ich hier hochgeladen.
Regards

Can you please change this in config.ini
Logging=DEBUG
and I would then need the current.log
I hope this helps

I changed the config.ini
That hasn't fixed the problem yet. I have uploaded the new config as config copy.ini again and also the current.log file.

requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: http://<IP_ADDRESS>/api/livedata/status
it seems you have password protected your opendtu, but not put your login credentials and your config has no credentials
Username/Password leave empty if no authentication is required
Username =
Password =

Hello, indeed. I was not aware that the admin password had to be entered. I can access the DTU web interface without a password and only need it for some settings.
But thank you very much, it works now.

It should work when you have the setting like this

we will update documentation to have this available for anyone much easier next time

Perfect. I had understood the setting the other way around.


gnome-control-center crashes when clicking on "region and languages"

I installed crouton on my acer C720P.(upgraded with a 128 Gb SSD).
However, when I go to gnome-control-center and click on "region and languages", I got a crash with the following error message:
(gnome-control-center.real:3113): common-cc-panel-CRITICAL **: gdm_get_language_from_name: assertion '*name != '\0'' failed
Segmentation fault (core dumped)

Looks to me like a problem with gnome similar to this one ( https://bugs.launchpad.net/ubuntu/+source/gnome-control-center/+bug/1245140 ), not crouton.

Hmm, not sure it is the same.
My error message refers to gdm.

No, it's not the same - but similar, and probably nothing to do with crouton.

On C720p running crouton Trusty using GNOME Shell
gnome-control-center
(gnome-control-center.real:15699): common-cc-panel-CRITICAL **: gdm_get_language_from_name: assertion '*name != '\0'' failed
Segmentation fault (core dumped)
Any suggestions>

I have the same issue after I reinstall the trusty chroot today. Looks like it is because LANG is not set. It is fixed after I run "localectl set-locale LANG=en_US.utf8" and re-login.

Looks like another vote for #1680 then.


KCD Pickpocket

[x] Remember last pickpocketed NPCS
[x] do not generate items for tracked NPCS for 48h
- [ ] -X detection if pickpocketed in the last Y y

[ ] key responses
[ ] leave from pickmenu

[ ] fix pickpocketing a corpse


Cannot find module "npm-cli" when running nyc in scripts

Description
I encountered this error Cannot find module ...../node_modules/.bin/npm/bin/npm-cli.js when running nyc in my package.json script commnad. I run the command using yarn with yarn run test:cover.
The script command look looks like this:
"test:cover": "cross-env NODE_ENV=coverage nyc --reporter=lcov --reporter=text npm run test",
The full error output is this:
yarn run v1.1.0
$ cross-env NODE_ENV=coverage nyc --reporter=lcov --reporter=text npm run test
module.js:471
    throw err;
    ^

Error: Cannot find module 'C:\myproject\node_modules\.bin\node_modules\npm\bin\npm-cli.js'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Function.Module.runMain (module.js:604:10)
    at runMain (C:\Users\kwkong\.node-spawn-wrap-11076-03c06d2900cd\node:68:10)
    at Function.<anonymous> (C:\Users\kwkong\.node-spawn-wrap-11076-03c06d2900cd\node:171:5)
    at Object.<anonymous> (C:\Users\kwkong\Documents\Repos\cpp-react\node_modules\nyc\bin\wrap.js:23:4)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
----------|----------|----------|----------|----------|----------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
----------|----------|----------|----------|----------|----------------|
All files |  Unknown |  Unknown |  Unknown |  Unknown |                |
----------|----------|----------|----------|----------|----------------|
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Expected Behavior
Should run the command without error.
Observed Behavior
When I run this command ./node_modules/.bin/nyc --reporter=lcov --reporter=text npm run test directly in my CLI, it works fine. I suspect it is trying to access npm through my local node_modules when I tried to run it in my through script in my package.json.
More information
I am following this guide to setup nyc for mocha-webpack plugin: https://github.com/zinserjan/mocha-webpack/blob/master/docs/guides/code-coverage.md . Hope it helps
Forensic Information
Operating System: Window 10 Pro Version 1703
Environment Information:
https://gist.github.com/vKongv/f8862285cb2c2a3f99dfc16d2cd2b6a7

I encountered this issue to, and found this is caused by using npm run test after nyc'. using nyc mocha ....` directly is ok.

In the spawn-wrap lib, there's a call to get the npm.cmd location:
https://github.com/tapjs/spawn-wrap/blob/master/index.js#L173
When I open a Node repl, perform a which.sync('npm');, I get the location c:\\Program Files\\nodejs\\npm.CMD as a result, but when I console.log out what the npmPath is on that line when running npm test, it's spitting out {project_dir}\\node_modules\\.bin\\npm.CMD.
In the clean project I started, when I do the same console.log on that npmPath variable, it spits out:
c:\\Program Files\\nodejs\\npm.CMD
Again, probably a misconfiguration somewhere but haven't nailed down what and where.

Ok, I see what's going on now.
spawn-wrap uses the which npm lib to find the directory the npm command is located at looking at the path environment variable.  The path environment variable, for me, is getting the {cwd}, {cwd}/node_modules/npm-lifecycle/node-gyp-bin, and {cwd}/node_modules/.bin directories unshifted into the path environment variable.
When it looks for the npm CMD in Windows, it finds it in the {cwd}/node_modules/.bin directory and returns that directory.  spawn-wrap then appends /node_modules/npm/bin/npm-cli.js to that path.
Well, in that instance, that's an invalid path and is why we see the error:
Error: Cannot find module '{cwd}\node_modules\.bin\node_modules\npm\bin\npm-cli.js'
It also explains why, in the "clean" project, this isn't happening.  The npm CMD isn't installed in the node_modules/.bin directory.
So this feels like an issue with spawn-wrap assuming the /node_modules/npm/bin/npm-cli.js directory will be there.

Found the culprit on my end with the "misconfiguration".  This all started when I updated node from 9 to 10 and updated npm from 5 to 6.  I did this in the project's directory which I inadvertently installed npm as a dependency to the project.  This in turn added the npm CMD file to the node_modules/.bin directory.
Removing that dependency, deleting the node_modules, and doing a clean npm install fixed the issue.

Removing npm file from _/node_modules/.bin/ folder helped me.


bug(indent):  char doesnt render when an empty line is inserted on the first line of a scope

Did you check docs and existing issues?

[X] I have read all the snacks.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of snacks.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.10.2
Operating system/version
archlinux
Describe the bug
indent character doesnt render when inserting an empty newline below a scope.
Image before inserting an empty newline.

Image after inserting an empty newline.

Steps To Reproduce

use the repro code.
create a table with values.
insert an empty new line on the on the first line of the table.

Expected Behavior
indent character should render.
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "folke/snacks.nvim", opts = { indent = {}}},
  },
})

@quebn can you check the update? I made some slight changes to better deal with blank lines

@folke looks good now, thanks.


lock users and check user lock status

closes #142
This adds the ability for super users to lock user accounts and also adds a check of the user lock status in each API call and the portal credential requests.
This doesn't actually provide the ability to change user lock status, it just creates the endpoint, so there isn't a way to test that yet. We have to decide if we want to add that functionality to the portal or one of the tools, or both.
To test the experience of being locked out go to the UserAccount.scala file in the portal module (https://github.com/vinyldns/vinyldns/blob/d303c5ea593aa4a24499ea7aa7828a568e3d374b/modules/portal/app/models/UserAccount.scala#L34) and change false to true.

Codecov Report

Merging #166 into master will decrease coverage by 0.04%.
The diff coverage is 66.03%.

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
- Coverage    94.3%   94.26%   -0.05%
==========================================
  Files         106      106
  Lines        4955     4985      +30
  Branches      101       95       -6
==========================================
+ Hits         4673     4699      +26
- Misses        282      286       +4

Impacted Files
Coverage Δ

...les/portal/app/controllers/LdapAuthenticator.scala
97.1% <ø> (ø)
:arrow_up:

modules/portal/app/models/UserAccount.scala
100% <100%> (ø)
:arrow_up:

modules/portal/app/controllers/VinylDNS.scala
73.71% <64.7%> (+2.48%)
:arrow_up:

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 9480160...d303c5e. Read the comment docs.

We should probably do a custom action here, it would localize the code and simplify the update.
We can pair on this...
https://www.playframework.com/documentation/2.6.x/ScalaActionsComposition

@pauljamescleary I was reading about the custom action builders, but i got stuck on how to pass in the needed user information.

closing since this is out of date, new approach in #186.


Infinite Looping error with user's npm

My node was properly installed, but there was some error with ppm, even npm -v was not correctly working.
When I generated the stack report for a package.json, the process of generating effective pom went into an infinite loop of failure, instead of existing after the first try with a message.
After fixing my npm, the report worked as expected
VScode: 1.26.1
OS: Fedora
Packag.json: yarn
https://github.com/yarnpkg/yarn

Resolution is to not try to rerun npm install when stderr is present.

While for whole workspace there is an error in the developer log, for the single manifest there is no developer log and no action is taken. When the camelcase is changed, it worked.
We should take some action, to generate the appropriate error, and bring it to the foreground. Because it looks as if the extension stopped working, even though that is not the case.
cc @invincibleJai

@sara-02 yes it's true, current fix will avoid loop and it will show user with Stdout error.

This issue was moved to openshiftio/openshift.io#4248


Add support for v3.4 devices

Finally got a 3.4 device and got this working.  I modeled it after harryzz's code at https://github.com/harryzz/tuyapi/commit/ef4473b4679c29d07ae504490503b6fa87d3e733 .  The modifications were a lot more extensive than I initially anticipated due to the switch from CRC32 to HMAC checksums.
Closes #148
Closes #178

I just picked up the one linked in #178 as well as a couple others listed as "new model" from that same brand.  I have no idea what they actually are, I'll post links if they show up with v3.4 on them.

Ha!  I finally got around to opening the relays I got in last week, and... they're v3.4.  I got the 4-channel version: https://www.aliexpress.com/item/3256803462490229.html
There were 2 problems with this PR code: 1) the AES decrypt was blowing up because it was trying to .decode() binary data, and 2) session key generation cannot be done in _get_socket() because the payload is already encrypted by this point.  I already got 1) fixed but am still working on a clean way of fixing 2).

Oh yeah, a 3rd issue was the device rejects seqno 0, so that was updated to start at 1.
I ended up moving the session key generation into _generate_message() (which is called from generate_payload()) and made it also open the socket when the version is 3.4.  Not the prettiest, but I don't see any better option with the key needing to be negotiated before the payload is encrypted.  This also means I had to disable socket closing/reopening in _send_receive() as it has no way of re-encrypting the payload.  Moving the encryption into _send_receive() may be something to think about for a future update.

When I updated the version bytes detection in _decode_payload() I originally removed the device22 check as I didn't think it was needed, but as I have no device22 devices I don't know for sure and so I added it back in with a slight modification so it only triggers if it also has a non-mod-16 payload length.
With that, I think it's ready.  Closes #178 and #148
python3 v4-test.py
DEBUG:TinyTuya [1.6.6]

DEBUG:status() entry (dev_type is v3.4)
DEBUG:building command 16 payload=b'{"data":{"devId":"eb6a946d9829d5aeb7dkvv","uid":"eb6a946d9829d5aeb7dkvv","t":"1662858248"},"protocol":5,"t":"1662858248"}'
DEBUG:payload generated=b'000055aa000000010000000300000044a810ff13bbf370cf0aba4628c9aff7b0c21600a8f8a9d09645c51888825cdb02816bb316150339faebb2da28cce00c3f7b597a8121925cf5808bc9f0299776fe0000aa55'
DEBUG:sending payload
DEBUG:received data=b'000055aa0000527c000000040000006800000000f5dc71bba39b3c6bf71083216d2ffaf38af576260f9e5b74ea79e66decf317e2d32f4b5a946b6aa383a59f587e6cbcc7c21600a8f8a9d09645c51888825cdb02001f1ad33bcbb6e42c66bf57ec8ca64547638f0c36723bb8582da35d419cfad50000aa55'
DEBUG:received message=TuyaMessage(seqno=21116, cmd=4, retcode=(0,), payload=b'\xf5\xdcq\xbb\xa3\x9b<k\xf7\x10\x83!m/\xfa\xf3\x8a\xf5v&\x0f\x9e[t\xeay\xe6m\xec\xf3\x17\xe2\xd3/KZ\x94kj\xa3\x83\xa5\x9fX~l\xbc\xc7\xc2\x16\x00\xa8\xf8\xa9\xd0\x96E\xc5\x18\x88\x82\\\xdb\x02', crc=b'\x00\x1f\x1a\xd3;\xcb\xb6\xe4,f\xbfW\xec\x8c\xa6EGc\x8f\x0c6r;\xb8X-\xa3]A\x9c\xfa\xd5', crc_good=True)
DEBUG:decrypting=b'\xf5\xdcq\xbb\xa3\x9b<k\xf7\x10\x83!m/\xfa\xf3\x8a\xf5v&\x0f\x9e[t\xeay\xe6m\xec\xf3\x17\xe2\xd3/KZ\x94kj\xa3\x83\xa5\x9fX~l\xbc\xc7\xc2\x16\x00\xa8\xf8\xa9\xd0\x96E\xc5\x18\x88\x82\\\xdb\x02'
DEBUG:decrypted session key negotiation step 2 payload=b"16255b75574358c4\xa6\xa1 \xba\xb3}\xc82a}T\xb6\xd9a'\xe5%\xda\xe5_a\xc4}\xc8\xee2\r\\\xf3\x1bS6"
DEBUG:payload type = <class 'bytes'> len = 48
DEBUG:session local nonce: b'0123456789abcdef' remote nonce: b'16255b75574358c4'
DEBUG:payload generated=b'000055aa0000000200000005000000545431df7c1d6718d792036a21c55d3d40889876f8f36af120c28f90926d94064dc21600a8f8a9d09645c51888825cdb026d1e54518fe9217cc3bdcbeebb157e3d709dffa0ec4a2cec0657a39f5822e1150000aa55'
DEBUG:sending payload
DEBUG:Session nonce XOR'd: b'\x01\x07\x00\x06\x01W\x01\x02\r\x0eUQV\\\x06R'
DEBUG:Session key negotiate success! session key: b'\x0c\xee\x92k\xc6\x04AC\x82\xd5z\x1c\xb4\xb5\xf3\xf1'
DEBUG:payload generated=b'000055aa0000000300000010000000a43a17e99bfcdebaffe76971af41adfeb7dfb8c5676aa58b0775b2047bf4f3e9b08f93a66830c7e0f17ba96ac69934387e36ae2291663819fa360e2407284262841fd9c212f0d513a81b0bec1b451a84e524f82fde66e7d5576205a4ca7f5e0537fffe6b48b822c4dc1f37a45f4260386de0f4b0c09134a77c80bd8967a142c233482e746e6d661f56c7be7d5ce032bce2fc2900258eb280c4c31db495dabc8d7f0000aa55'
DEBUG:sending payload
DEBUG:received data=b'000055aa0000527d00000010000000a800000000665885f7b8d46f18fb655095aa3b2dcc68b4a5f2f3dc997ac7c0b7cf6d0cee04b8e4b801a3bd6b7ab33742e8e0a3ece5a1c2649c50f03eb6039b7696cdce63a95c787f64559c187b8fb41b5a47597c2fafacbe8175505dd54419306bfb4b49d1ff84c70549c4a17bcab45f701b21334ae97b4443c602adf792f5f81aadfbef575fba4e991775a462e69bc6f0627fd53da3411a37e9ffc40497259dcd7df783850000aa55'
DEBUG:received message=TuyaMessage(seqno=21117, cmd=16, retcode=(0,), payload=b'fX\x85\xf7\xb8\xd4o\x18\xfbeP\x95\xaa;-\xcch\xb4\xa5\xf2\xf3\xdc\x99z\xc7\xc0\xb7\xcfm\x0c\xee\x04\xb8\xe4\xb8\x01\xa3\xbdkz\xb37B\xe8\xe0\xa3\xec\xe5\xa1\xc2d\x9cP\xf0>\xb6\x03\x9bv\x96\xcd\xcec\xa9\\x\x7fdU\x9c\x18{\x8f\xb4\x1bZGY|/\xaf\xac\xbe\x81uP]\xd5D\x190k\xfbKI\xd1\xff\x84\xc7\x05I\xc4\xa1{\xca\xb4_p\x1b!3J\xe9{DC\xc6\x02\xad\xf7\x92\xf5\xf8\x1a\xad\xfb\xefW', crc=b'_\xbaN\x99\x17u\xa4b\xe6\x9b\xc6\xf0b\x7f\xd5=\xa3A\x1a7\xe9\xff\xc4\x04\x97%\x9d\xcd}\xf7\x83\x85', crc_good=True)
DEBUG:raw unpacked message = TuyaMessage(seqno=21117, cmd=16, retcode=(0,), payload=b'fX\x85\xf7\xb8\xd4o\x18\xfbeP\x95\xaa;-\xcch\xb4\xa5\xf2\xf3\xdc\x99z\xc7\xc0\xb7\xcfm\x0c\xee\x04\xb8\xe4\xb8\x01\xa3\xbdkz\xb37B\xe8\xe0\xa3\xec\xe5\xa1\xc2d\x9cP\xf0>\xb6\x03\x9bv\x96\xcd\xcec\xa9\\x\x7fdU\x9c\x18{\x8f\xb4\x1bZGY|/\xaf\xac\xbe\x81uP]\xd5D\x190k\xfbKI\xd1\xff\x84\xc7\x05I\xc4\xa1{\xca\xb4_p\x1b!3J\xe9{DC\xc6\x02\xad\xf7\x92\xf5\xf8\x1a\xad\xfb\xefW', crc=b'_\xbaN\x99\x17u\xa4b\xe6\x9b\xc6\xf0b\x7f\xd5=\xa3A\x1a7\xe9\xff\xc4\x04\x97%\x9d\xcd}\xf7\x83\x85', crc_good=True)
DEBUG:decode payload=b'fX\x85\xf7\xb8\xd4o\x18\xfbeP\x95\xaa;-\xcch\xb4\xa5\xf2\xf3\xdc\x99z\xc7\xc0\xb7\xcfm\x0c\xee\x04\xb8\xe4\xb8\x01\xa3\xbdkz\xb37B\xe8\xe0\xa3\xec\xe5\xa1\xc2d\x9cP\xf0>\xb6\x03\x9bv\x96\xcd\xcec\xa9\\x\x7fdU\x9c\x18{\x8f\xb4\x1bZGY|/\xaf\xac\xbe\x81uP]\xd5D\x190k\xfbKI\xd1\xff\x84\xc7\x05I\xc4\xa1{\xca\xb4_p\x1b!3J\xe9{DC\xc6\x02\xad\xf7\x92\xf5\xf8\x1a\xad\xfb\xefW'
DEBUG:decrypting=b'fX\x85\xf7\xb8\xd4o\x18\xfbeP\x95\xaa;-\xcch\xb4\xa5\xf2\xf3\xdc\x99z\xc7\xc0\xb7\xcfm\x0c\xee\x04\xb8\xe4\xb8\x01\xa3\xbdkz\xb37B\xe8\xe0\xa3\xec\xe5\xa1\xc2d\x9cP\xf0>\xb6\x03\x9bv\x96\xcd\xcec\xa9\\x\x7fdU\x9c\x18{\x8f\xb4\x1bZGY|/\xaf\xac\xbe\x81uP]\xd5D\x190k\xfbKI\xd1\xff\x84\xc7\x05I\xc4\xa1{\xca\xb4_p\x1b!3J\xe9{DC\xc6\x02\xad\xf7\x92\xf5\xf8\x1a\xad\xfb\xefW'
DEBUG:decrypted 3.x payload='{"dps":{"1":false,"2":false,"3":false,"4":false,"9":0,"10":0,"11":0,"12":0,"38":"off","42":"","43":"","44":"","47":"button"}}'
DEBUG:payload type = <class 'str'>
DEBUG:decoded results='{"dps":{"1":false,"2":false,"3":false,"4":false,"9":0,"10":0,"11":0,"12":0,"38":"off","42":"","43":"","44":"","47":"button"}}'
DEBUG:status() received data={'dps': {'1': False, '2': False, '3': False, '4': False, '9': 0, '10': 0, '11': 0, '12': 0, '38': 'off', '42': '', '43': '', '44': '', '47': 'button'}}
{'dps': {'1': False, '2': False, '3': False, '4': False, '9': 0, '10': 0, '11': 0, '12': 0, '38': 'off', '42': '', '43': '', '44': '', '47': 'button'}}

Well that thought was short-lived.  Seems it does not like setting DPS values.  Looking into it...

I ended up writing a decoder to decode packet captures from between the app and device in local mode.  To use:

Packet capture local traffic (offline mode) from between the app and the device.
Open it in Wireshark, find the stream, right-click a packet from it and select "Follow -> TCP Stream"
Change "Show and save data as" to "YAML" and then "Save as..." to somewhere
Install the YAML module for python.  pip could not find it for me, so I had to apt install python3-yaml to get it
Run this, after changing the lkey= to your key:

import sys
import yaml

import tinytuya

lkey = '0123456789abcdef'.encode('latin1')

if( len(sys.argv) is not 2 ):
    print( 'Usage:', sys.argv[0], 'somefile.yaml' )
else:
    with open( sys.argv[1] ) as f:
        data = yaml.load(f)#, Loader=yaml.FullLoader)

    skey = lkey

    for pkt in data:
        k = pkt.split('_')
        packet_id = int(k[1])
        peer = int(k[0][-1])

        if peer == 0:
            msg = tinytuya.unpack_message(data[pkt], hmac_key=skey, no_retcode=True)
        else:
            msg = tinytuya.unpack_message(data[pkt], hmac_key=skey)

        if len(msg.payload) == 0:
            #print('empty payload')
            continue

        if msg.cmd == 5:
            print('peer %d cmd %d seqno %d len %d - cmd 5 ignored' % (peer,msg.cmd,msg.seqno,len(msg.payload)))
            continue

        payload = msg.payload
        extra = len(payload) & 0x0F
        if extra != 0:
            print('payload len wrong!', len(payload), payload )
            payload = payload[:-extra]
            #payload += b'\0' * (16-extra)

        cipher = tinytuya.AESCipher(skey)
        payload = cipher.decrypt(payload, False, decode_text=False)
        print('peer %d cmd %d seqno %d raw len %d decoded len %d payload %r' % (peer,msg.cmd,msg.seqno,len(msg.payload),len(payload), payload) )

        if msg.cmd == 3:
            skey = lkey
            lnonce = payload
        elif msg.cmd == 4:
            rnonce = payload[:16]
            skey = bytes( [ a^b for (a,b) in zip(lnonce,rnonce) ] )
            cipher = tinytuya.AESCipher(lkey)
            skey = cipher.encrypt(skey, False, pad=False)
            print( 'local nonce: %r remote nonce: %r session key: %r' % (lnonce, rnonce, skey) )
        else:
            pass

When run on a capture I made it said:
python3 wireshark-yaml-dec.py br10-20220910-67-only.yaml
peer 0 cmd 3 seqno 12101 raw len 32 decoded len 16 payload b'$\x1e\xf1\x10\xc8`+\x93\x12\xf0J?\xfeJ^\x98'
peer 1 cmd 4 seqno 22593 raw len 64 decoded len 48 payload b'afd5d71d1711a42eX\x93Z=|\xa0\x93\x80\x84z\x10B\xf7\x1d%\xfbK(sn\xff@Lx\xa9;\xa5_\x8b\x87\\V'
local nonce: b'$\x1e\xf1\x10\xc8`+\x93\x12\xf0J?\xfeJ^\x98' remote nonce: b'afd5d71d1711a42e' session key: b'\x1b\x94\x06\xdd\xa6\x17\x15J\x01\xc2\xf8\x14\xae\xdc\xe2\x83'
peer 0 cmd 5 seqno 12102 len 48 - cmd 5 ignored
peer 1 cmd 16 seqno 22594 raw len 128 decoded len 124 payload b'{"dps":{"1":true,"2":false,"3":false,"4":false,"9":0,"10":0,"11":0,"12":0,"38":"off","42":"","43":"","44":"","47":"button"}}'
peer 0 cmd 13 seqno 12105 raw len 80 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x07\x00\r\xfc\\{"data":{"dps":{"2":true}},"protocol":5,"t":1662871092}'
peer 1 cmd 8 seqno 22596 raw len 80 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbo\x00\x00\x00\x01{"protocol":4,"t":1662871093,"data":{"dps":{"2":true}}}'
peer 0 cmd 13 seqno 12106 raw len 80 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x08\x00\r\xfc\\{"data":{"dps":{"3":true}},"protocol":5,"t":1662871093}'
peer 1 cmd 8 seqno 22598 raw len 80 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbp\x00\x00\x00\x01{"protocol":4,"t":1662871094,"data":{"dps":{"3":true}}}'
peer 0 cmd 13 seqno 12107 raw len 80 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\t\x00\r\xfc\\{"data":{"dps":{"4":true}},"protocol":5,"t":1662871094}'
peer 1 cmd 8 seqno 22600 raw len 80 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbq\x00\x00\x00\x01{"protocol":4,"t":1662871095,"data":{"dps":{"4":true}}}'
peer 0 cmd 13 seqno 12108 raw len 80 decoded len 71 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\n\x00\r\xfc\\{"data":{"dps":{"1":false}},"protocol":5,"t":1662871096}'
peer 1 cmd 8 seqno 22602 raw len 80 decoded len 71 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbr\x00\x00\x00\x01{"protocol":4,"t":1662871097,"data":{"dps":{"1":false}}}'
peer 0 cmd 13 seqno 12110 raw len 96 decoded len 91 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x0b\x00\r\xfc\\{"data":{"dps":{"2":false,"3":false,"4":false}},"protocol":5,"t":1662871101}'
payload len wrong! 84 b'#\xe6\xa2h 5q\x94\xbc\xef\xea\x0e\xf9\x8e\x90\xdc\xf6\t:\xea\xf4\xc2\xfe\xa0\x1d\xfb\xe5\x05\\\x10+k]dE\x1e\xaei\xeew\x88\x14\xca\xacK\xd5\x84|\x93@h\xa7kSv\xeag\x0b\x0cm\x8f\xf3X[\xab\xff\xaehf\xda/\xac\xa8Y\xd20\xfabC\x8c\xbb\xc5l`'
peer 1 cmd 8 seqno 22605 raw len 84 decoded len 71 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbs\x00\x00\x00\x01{"protocol":4,"t":1662871102,"data":{"dps":{"4":false}}}'
peer 0 cmd 13 seqno 12111 raw len 112 decoded len 97 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x0c\x00\r\xfc\\{"data":{"dps":{"1":true,"2":true,"3":true,"4":true}},"protocol":5,"t":1662871104}'
payload len wrong! 84 b'$i\xf5\xcb\x9f\x92\x81\x86_\x9d*\xd3t\xde9\xc7\xf6\t:\xea\xf4\xc2\xfe\xa0\x1d\xfb\xe5\x05\\\x10+k\x83\xd4\x1d1,\xd6|8\xe1K\xb4\xc2\xa7d\x8bC\xe3\x06A\x87R\x91Q\x89\x8f\xc2\x1b\xc4\xe1\x9d\xeb\x0f6\x1c\tU0\xeb\xfd\xc00\xae\xf3\xd5\xacl\xd7k\xe6\r6x'
peer 1 cmd 8 seqno 22609 raw len 84 decoded len 70 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbv\x00\x00\x00\x01{"protocol":4,"t":1662871105,"data":{"dps":{"4":true}}}'
peer 0 cmd 13 seqno 12113 raw len 80 decoded len 73 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\r\x00\r\xfc\\{"data":{"dps":{"47":"flip"}},"protocol":5,"t":1662871118}'
peer 1 cmd 8 seqno 22615 raw len 80 decoded len 73 payload b'3.4\x00\x00\x00\x00\x00\x00\xfbz\x00\x00\x00\x01{"protocol":4,"t":1662871119,"data":{"dps":{"47":"flip"}}}'
peer 0 cmd 13 seqno 12115 raw len 80 decoded len 75 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x0e\x00\r\xfc\\{"data":{"dps":{"47":"button"}},"protocol":5,"t":1662871121}'
peer 1 cmd 8 seqno 22618 raw len 80 decoded len 75 payload b'3.4\x00\x00\x00\x00\x00\x00\xfb{\x00\x00\x00\x01{"protocol":4,"t":1662871122,"data":{"dps":{"47":"button"}}}'
peer 0 cmd 13 seqno 12116 raw len 80 decoded len 71 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x0f\x00\r\xfc\\{"data":{"dps":{"38":"on"}},"protocol":5,"t":1662871127}'
peer 1 cmd 8 seqno 22620 raw len 80 decoded len 71 payload b'3.4\x00\x00\x00\x00\x00\x00\xfb|\x00\x00\x00\x01{"protocol":4,"t":1662871128,"data":{"dps":{"38":"on"}}}'
peer 0 cmd 13 seqno 12117 raw len 80 decoded len 75 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x10\x00\r\xfc\\{"data":{"dps":{"38":"memory"}},"protocol":5,"t":1662871130}'
peer 1 cmd 8 seqno 22622 raw len 80 decoded len 75 payload b'3.4\x00\x00\x00\x00\x00\x00\xfb}\x00\x00\x00\x01{"protocol":4,"t":1662871131,"data":{"dps":{"38":"memory"}}}'
peer 0 cmd 13 seqno 12119 raw len 80 decoded len 72 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x11\x00\r\xfc\\{"data":{"dps":{"38":"off"}},"protocol":5,"t":1662871132}'
peer 1 cmd 8 seqno 22625 raw len 80 decoded len 72 payload b'3.4\x00\x00\x00\x00\x00\x00\xfb~\x00\x00\x00\x01{"protocol":4,"t":1662871134,"data":{"dps":{"38":"off"}}}'
peer 0 cmd 13 seqno 12121 raw len 80 decoded len 73 payload b'3.4\x00\x00\x00\x00\x00\x00\x00\x12\x00\r\xfc\\{"data":{"dps":{"44":"AQAB"}},"protocol":5,"t":1662871145}'
peer 1 cmd 8 seqno 22628 raw len 80 decoded len 73 payload b'3.4\x00\x00\x00\x00\x00\x00\xfb\x7f\x00\x00\x00\x01{"protocol":4,"t":1662871146,"data":{"dps":{"44":"AQAB"}}}'
peer 1 cmd 8 seqno 22629 raw len 80 decoded len 71 payload b'3.4\x00\x00\x00\x00\x00\x00\xfb\x80\x00\x00\x00\x01{"protocol":4,"t":1662871146,"data":{"dps":{"1":false}}}'

This was a friggin can-o-worms.  I think I got everything now though.  Unlike 3.3 devices, 3.4 devices encrypt the version header in addition to the payload, and if you send them a bad command they drop the connection and you need to close and re-open it (and renegotiate the session key) before you can send anything else.

I see the problem... this works:
d = tinytuya.OutletDevice('id', '<IP_ADDRESS>', 'key')
d.set_version(3.4)
d.set_socketPersistent(True)
print(d.status())

But if you don't set socker persistence it fails:
d = tinytuya.OutletDevice('id', '<IP_ADDRESS>', 'key')
d.set_version(3.4)
print(d.status())

Traceback (most recent call last):
  File "/Users/jason/Code/tinytuya/sandbox/test.py", line 17, in <module>
    print(d.status())
  File "/Users/jason/Code/tinytuya/sandbox/tinytuya/core.py", line 1225, in status
    payload = self.generate_payload(query_type)
  File "/Users/jason/Code/tinytuya/sandbox/tinytuya/core.py", line 1214, in generate_payload
    return self._generate_message(command_override, payload)
  File "/Users/jason/Code/tinytuya/sandbox/tinytuya/core.py", line 919, in _generate_message
    self.socket.settimeout(self.connection_timeout)
AttributeError: 'NoneType' object has no attribute 'settimeout'

Correct me if I'm wrong, but we really can't have a 3.4 device that is non persistent.  It negotiates a session based key so there is no point in a non-persistent version.  I need to work through implications or what I'm missing, but this is a quick fix in core.py:

I confirmed that this change also fixes scanner.py functions for 3.4 devices too.  I'm pushing change up for now and staging it and release notes as version 1.7.0 (not pushing to PyPI yet).

I don't see why it couldn't be non-persistent, it's just instead of "connect -> send request -> disconnect" it will "connect -> negotiate key -> send request ->disconnect".  In fact if someone has a program that waits more than ~30 seconds between requests then it must do this as otherwise the socket will be closed due to inactivity and the next _send_receive() will fail because it has no way of re-opening the socket or re-encrypting the payload.
The more I think about it the less I like how generate_payload() / _generate_message() do the encrypting.  I think I'm going to fix persistence for the current code and then rework it so the encryption is done in _send_receive().  This may cause issues if people are calling _send_receive() with something other than what generate_payload() returns, but I think ultimately it will be the better solution.


python312Packages.pydaikin: 2.13.4 -> 2.13.6

Diff: https://github.com/fredrike/pydaikin/compare/refs/tags/v2.13.4...v2.13.6
Changelog: https://github.com/fredrike/pydaikin/releases/tag/v2.13.6
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

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

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

5 packages built:

home-assistant-component-tests.daikin
python311Packages.pydaikin
python311Packages.pydaikin.dist
python312Packages.pydaikin
python312Packages.pydaikin.dist


Support for H410M S2H

Hi Guys, i wanted to ask if is possible to have ambient led working on H410M S2H for now i'm running Arch Linux with a custom kernel CK. My motherboard have this chip: [  554.735528] gpio_it87: Unknown Chip found, Chip 8688 Revision 1 and i think isn't supporting by the gpio_it87 driver. I hope you can help me :+1:
Best Regards,
NextWorks

closing as I am not longer actively maintaining this project


fix: support int64 conversion between the pf message and JSON object

Similar to #1015, support int64 conversion from protobuf message to JSON object.
protobufjs
Message.toObject(message: Message [, options: ConversionOptions]): Object
converts a message instance to an arbitrary plain JavaScript object for interoperability with other libraries or storage. The resulting plain JavaScript object might still satisfy the requirements of a valid message depending on the actual conversion options specified, but most of the time it does not.
var object = AwesomeMessage.toObject(message, {
  enums: String,  // enums as string names
  longs: String,  // longs as strings (requires long.js)
  bytes: String,  // bytes as base64 encoded strings
  defaults: true, // includes default values
  arrays: true,   // populates empty arrays (repeated fields) even if defaults=false
  objects: true,  // populates empty objects (map fields) even if defaults=false
  oneofs: true    // includes virtual oneof fields set to the present field's name
});

@summer-ji-eng I added one commit that adds a call to .fromObject() for the request (making possible to pass enums and big integers as strings into the request) and renamed a few variables to increase readability of the code.


fix: error thrown when setting value in redis cache

Description:
ERROR (cache-service/7325): Error occurred while setting the cache to Redis. Fallback to internal cache. Error is: Error: ERR value is not an integer or out of range

This error is appearing multiple times in the logs of multiple workflows in the CI, right after doing the getAccount query from the mirror node.
After doing some investigation, it looks like when we transform the passed TTL to seconds, it was possible that we get a floating point number (ETH_GET_TRANSACTION_COUNT_CACHE_TTL: 500) => which resolves to resolvedTtl = 0.5
const resolvedTtl = (ttl ?? this.options.ttl) / 1000; // convert to seconds

And the setEx method of Redis expects an integer, hence why this error is thrown.
This PR:

modifies the RedisCache#set method to use set (with PX option which allows us to directly pass milliseconds as argument for TTL) instead of using setEx to avoid this issue.

Related issue(s):
Fixes #2838
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Nice catch with the tests!
LGTM


how to progress drag property  disabled?

Hi,
Is there any way to disable the drag property of the progress?

If you do not include the onProgressChange property it will automatically not show the progress drag indicator

Yes it works Thank you


Move the core upgrade recipe to a separate JAR

The maven-jar-plugin can only include/exclude resources, but not move them to a different directory in the output folder; the assembly plugin can.
@ia3andy Keep in mind that a similar assembly will have to be provided for every directory under src/main/resources/quarkus-updates if it releases in a separate JAR file.

@rsynek the jar will all recipes will still be generated right?

@rsynek the jar will all recipes will still be generated right?

Yes, there will be two 2 JARs (3 if we count the sources JAR): one containing the 'META-INF/rewrite/3alpha.yaml' and another with the recipe in its original place.


Running from Docker doesn't work

When pulling directly from the docker instructions, get a exception from Npgsql.

Core_AppSetting" does not exist

It works now


Support yaml

environment vars are getting a bit tricky to support - especially for advanced usage like #49
Perhaps we can do something like:
version: '2'

services:
    crossdock:
        image: yarpc/crossdock
        volumes:
            - .:/usr/src/crossdock

Where we look for a crossdock.yaml in /usr/src/crossdock (path up for debate), and if its not there, use env vars.

Suggest that environment variables always work, and they take the highest precedance.
This will allow #44 to work.

Also, if someone wants to override the default yaml config location, they should be able to do so with an environment variable: YAML=/var/cd.yaml.


Add support for Pandas Dataframes

In Hummingbird we currently support ndarrays as input. We should add support for Pandas Dataframes too. This is particularly critical for pipelines since each operator in a pipelines can work on a set of columns on the dataframe. This will require extracting schema information from the dataframe.

each operator in a pipelines can work on a set of columns on the dataframe

@interesaaat  Will the columns be referenced by the index or by the column name?
I remember there are some operators like ColumnTranformer which refer columns by index.
I think these cases should be easy to support by simply converting the dataframe to numpy (df.to_numpy())

Yes this is exactly what we have right now. It would be good to extend it to support dataframes more in general.


Corrected hide & show implementation in Cobalt plugin.

However Cobalt Browser itself does not support hide & show.
Should I return ERROR_UNAVAILABLE instead for JSON-RPC visibility set and get requests?

https://jenkins.code.rdkcentral.com/job/github-triggered/1832/ started


Reformat with Rust 1.83

Introduces some new rules, apply them

In response to a cherrypick label: new pull request created: #1375


Ornl aisd ex

This PR is currently for review only as we need to decide the destination repo and branch (currently set to allaffa/main).
This PR adds the capability of reading in tar.gz files for preprocessing in addition to the existing mollist.txt. Now, we can read in both GDB-9-Ex as well as ORNL_AISD_Ex datasets.
New parser arguments --input_data and --tmpfs have been added that will accept a path to the input data (directory of tar files or mollist.txt) and tmpfs for extracting tar files. Tar files are extracted into tmpfs and the resulting molecule directories are then parsed to generate graph objects.

Also adding @jychoi-hpc to review PR.

Max, as this branch was cloned from the ddstore_equivariant branch from your fork, there are several commits from that branch that are visible in this PR. Please ignore those for now as we decide on where this PR should be merged.

I made some minor changes and it works.
Here are my changes:
         # Iterate through molecule directories or tar files and create graph objects
         for item in iterate_tqdm(dirlist, verbosity_level=2, desc="Load"):
+            if item == "README.txt":
+                continue
             fullpath = os.path.join(dirpath, item)

             # Verify that files in the directory are either tar.gz files or molecule directories
             if (not fullpath.endswith(".tar.gz")) and (not os.path.isdir(fullpath)):
-                raise ("Can't understand if {} is a molecule directory or tar.gz file".format(fullpath))
+                raise RuntimeError("Can't understand if {} is a molecule directory or tar.gz file".format(fullpath))

             # If tar.gz files, extract tar file in a tmp dir
             if fullpath.endswith(".tar.gz"):
                 extract_path = extract_tar_file(fullpath, tmpfs)
                 mdirs = [os.path.abspath(d) for d in os.scandir(extract_path) if d.is_dir()]
                 for mdir in mdirs:
-                    data_object = dftb_to_graph(mdir, dftb_node_types, var_config)
-                    self.dataset.append(data_object)
+                    try:
+                        data_object = dftb_to_graph(mdir, dftb_node_types, var_config)
+                        self.dataset.append(data_object)
+                    except:
+                        pass

Just for your information, I put a try-except block to handle the following error:
Traceback (most recent call last):
  File "/gpfs/alpine/lrn026/world-shared/jyc/HydraGNN-max/examples/dftb_uv_spectrum/train_discrete_uv_spectrum.py", line 241, in <module>
    total = DFTBDataset(
  File "/gpfs/alpine/lrn026/world-shared/jyc/HydraGNN-max/examples/dftb_uv_spectrum/train_discrete_uv_spectrum.py", line 132, in __init__
    data_object = dftb_to_graph(mdir, dftb_node_types, var_config)
  File "/gpfs/alpine/lrn026/world-shared/jyc/HydraGNN-max/examples/dftb_uv_spectrum/train_discrete_uv_spectrum.py", line 69, in dftb_to_graph
    ytarget = np.loadtxt(
  File "/gpfs/alpine/world-shared/csc143/jyc/summit/sw/anaconda3/2022.10/lib/python3.9/site-packages/numpy/lib/npyio.py", line 1067, in loadtxt
    fh = np.lib._datasource.open(fname, 'rt', encoding=encoding)
  File "/gpfs/alpine/world-shared/csc143/jyc/summit/sw/anaconda3/2022.10/lib/python3.9/site-packages/numpy/lib/_datasource.py", line 193, in open
    return ds.open(path, mode, encoding=encoding, newline=newline)
  File "/gpfs/alpine/world-shared/csc143/jyc/summit/sw/anaconda3/2022.10/lib/python3.9/site-packages/numpy/lib/_datasource.py", line 533, in open
    raise IOError("%s not found." % path)
OSError: /tmp/jyc/ornl_aisd_ex_unprocessed/mol_9054620/EXC.DAT not found.

Closing this PR to create a new one


0.1.0 Candidate

Describe your changes

Finish hub mesh network.
Add an example and corresponding e2e test.
Add required subnet_address_prefix field in firewall object so the module's caller won't need to define a subnet for the firewall that about to be created.

Issue number
#000
Checklist before requesting a review

[ ] The pr title can be used to describe what this pr did in CHANGELOG.md file
[ ] I have executed pre-commit on my machine
[ ] I have passed pr-check on my machine

Thanks for your cooperation!

@lonegunmanb LGTM!


Crash on macOS with atom >= 1.39.0

Issue
The atom window crashes when interracting with the project-viewer bar.
Can be reproduced steadily on every atom version >= 1.39.0
(it does not seem to bug on atom 1.38.2)
Environment

OS: maxOS 10.12.6
Atom: >= 1.39.0
No crash without this plugin.

Steps

Click to toggle one folder (open/close, does not matter)
Click again to select a project or toggle a folders ==> crash

and if possible provide a screenshot. :+1:

A video to illustrate :

The atom crash log associated with this crash
atom-crash-log.txt

hi @7ute, yeah, I have not maintained this project as I am focused on the new successor.
But this is breaking Atom, I will try to make a solution for this as well. Thanks for letting me know of it.

this has been resolved, will run some tests and deploy soon today

@7ute please validate

Yup, that fixed it! 👍
Thanks a lot!


Migrate to travis-ci.com

As announced in 2018 travis-ci.com is to be used for both open source and commercial projects. In December 2020, travis-ci.org will be going away, so we need to move before then.
Migration instructions are here: https://docs.travis-ci.com/user/migrate/open-source-repository-migration
The travis-ci.org builds will become read-only after the migration, so they can't be restarted, etc, but activity seems pretty low right now, so I don't know if we need to do any upfront coordination of the migration. Opinions?

Thanks for taking care of this! I can't think of anything that would be conflicting with it.

Migration complete for the main OpenRefine repo and it doesn't look like any of the other repos have Travis CI builds enabled.


Updated tsconfig-paths to fix https://github.com/advisories/GHSA-9c47-m6qq-7p4h

Is there someone that can review this MR?

This is a breaking change, and a duplicate of #2447, and a duplicate of #2646, and a duplicate of #2657. Additionally, json5 v1.0.2 fixes the issue, so there's no need to update. Additionally, this is not a vulnerability for the way eslint-plugin-import uses json5.
Like most JS CVEs, nothing needs to be done except "wait, and then update".


[PoC] ipstate: show basic link and address state

Initial commit for usage of pyroute2.IPRoute to get ip link and ip address details and print them as yaml

Hello contributor, thanks for submitting a PR for this project!
I am the bot who triggers "standard-CI" builds for this project.
As a security measure, I will not run automated tests on PRs that are not from white-listed contributors.
In order to allow automated tests to run, please ask one of the project maintainers to review the code and then do one of the following:

Type ci test please on this PR to trigger automated tests for it.
Type ci add to whitelist on this PR to trigger automated tests for it and also add you to the contributor white-list so that your future PRs will be tested automatically. ( keep in mind this list might be overwritten if the job XML is refreshed, for permanent whitelisting, please follow #3 option )
If you are planning to contribute to more than one project, maybe it's better to ask them to add you to the project organization, so you'll be able to run tests for all the organization's projects.

@erav Is there big picture of adding support of ip route?

@erav Is there big picture of adding support of ip route?

@EdDev is considering it as a POC

Is there big picture of adding support of ip route?

Interest has been raised to check the state on the kernel directly.
The show seems to be simple enough to implement and as a PoC it shows well the potential of the pluggable provider need.
@erav will probably continue this on a separate repo.
What is needed from Nmstate side is a pluggable provider integration, something we planned but did not implemented good enough.

We need to prepare the pluggable API for providers. Then we can use this PoC to show how it works.
Closing this PR for now.


Allow optionally using Homebrew's curl

If HOMEBREW_FORCE_BREWED_CURL is set then allow it to be used instead of the system curl.
Closes #3480.

:confetti_ball:

Shouldn't this be documented in man brew?  This PR didn't address that.


File input

The user currently has to type a lot of information in and most of it can pawned off in the coding side by simply giving the code access to see the sprite-sheet.
Describe the solution you'd like
The user can use an image to generate the atlas

No longer using this to reduce the amount of time it takes


Generate CHANGELOG and package-lock.json updates again during the release CI

The openedx/cla status check was recently made required to ensure all pushes to master have a signed contributor agreement.
However, Paragon uses semantic-release to release to NPM and generate a CHANGELOG.
The issue that was blocking NPM releases was our usage of @semantic-release/git which attempts to commit changes to CHANGELOG file and package.json to master, but fails due to openedx/cla status check.
As an interim solution to unblock releases, we removed the @semantic-release/git plugin. This made releases work again (since that is done by @semantic-release/github to create GH releases/tags).
The consequence of that interim solution is that we're no longer generating CHANGELOG changes and/or committing updates to the package.json file after a PR is merged to master.
AC

[ ] Ensure CHANGELOG entries are generated again after a PR is successfully released to NPM.
[ ] Ensure package.json file is updated with new SemVer version after a PR is successfully released to NPM.
[ ] Ensure file changes (CHANGELOG, package.json) is committed to master after a PR is successfully released to NPM.

Implementation notes

Likely add back @semantic-release/changelog again to generate the CHANGELOG entries based on the commit(s) and update the file.
Look into alternative ways to commit updates to CHANGELOG and package.json to master.

Consider something along the lines of how the dependent-usage-analyzer opens/auto-merges a PR (example).

Closing this a duplicate of https://github.com/openedx/paragon/issues/2575