Add custom ellipsis string #1195

Add feature requested in #1195.

add label-*-ellipsis-string to specify a custom string.
add deprecated_type_changed helper in config.hpp (display a warning if old key is used and recommends the new key which has a different type (here bool -> string)
update unit test and add one to check UTF-8 chars.

I also added a check that throw an exception if label-*-ellipsis = true and the string is empty. However I'm not sure it respects your way of doing things.

Codecov Report

Merging #1673 into master will decrease coverage by 0.08%.
The diff coverage is 26.31%.

@@            Coverage Diff             @@
##           master    #1673      +/-   ##
==========================================
- Coverage   23.34%   23.26%   -0.09%
==========================================
  Files          50       50
  Lines        2090     2106      +16
==========================================
+ Hits          488      490       +2
- Misses       1602     1616      +14

Flag
Coverage Δ

#unittests
23.26% <26.31%> (-0.09%)
:arrow_down:

Impacted Files
Coverage Δ

src/drawtypes/label.cpp
2.15% <0%> (-0.09%)
:arrow_down:

include/components/config.hpp
0% <0%> (ø)
:arrow_up:

src/components/builder.cpp
6.25% <100%> (+0.27%)
:arrow_up:

include/drawtypes/label.hpp
83.33% <100%> (+3.33%)
: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 db9a83a...cf8c24d. Read the comment docs.

Oh wow, you have been busy :smiley:
I have actually implemented this before (059c69e6256e6e2bb99f648dd6d89f320c117a26), but I never opened a PR because I was conflicted about how to handle failure modes. So I think you can check there to see if you are doing more or less the same. But it looks good at first glance.
I don't have a lot of time right now and I'm also going away for a few days, so you'll have to wait a bit for me to review this and all your other PRs as well.
Thank you very much for helping out, I'm looking forward to getting these features merged.

I have actually implemented this before (059c69e), but I never opened a PR because I was conflicted about how to handle failure modes.

Yeah I was also conflicted about how to handle failure modes. In my code I used exception like the old check you removed, you used logger::make().err(…).

I don't have a lot of time right now and I'm also going away for a few days, so you'll have to wait a bit for me to review this and all your other PRs as well.

Okay no problem. Are there feats/bugs that you want me to try to implement/fix (my exam week is over)? I am still new to polybar, but I'll do my best. ^^

There are a few bugs that you could work on:

#568, there has been some refactoring in #1213, maybe you could also incorporate that.
#428
#702, you would need to reproduce this first.
#916, Ignore the jgmenu part, the underlying issue is that click commands are not executed if they produce output. See also Known Issues
#1282. Here we need to make sure that we carry over the token properties of %percentage-cores% to %percentage-core[0-9]+%
#1180, this is also something you need to reproduce first.
#1081
#1630

Other things you could work on:

#1127
#1164

I think right now we should focus on bug fixes and refactoring to make the code more maintainable. Once you are somewhat familiar with the codebase, you could also try your luck on refactoring.
Thanks a ton for helping out!!

Fixed.
At the moment, the behavior is the following:

If label-ellipsis-string exists then label-ellipsis is ignored.
The new way to disable ellipsis is label-ellipsis-string = "" or equivalent.
The default value of the key label-ellipsis-string is ....

Examples:
label-maxlen = 5
label-ellipsis = false // ignored since `label-ellipsis-string` exists + depreciation warning
label-ellipsis-string = "…"

label-maxlen = 5
label-ellipsis-string = "" // same as `label-ellipsis = false`

label-maxlen = 5
label-ellipsis-string = "…" // ellipsis activated


give fake path to archive model to avoid annoying warning

Fixes https://github.com/fishtown-analytics/dbt/issues/460
@cmcarthur This problem will go away entirely in 0.8.4, right? I'm ok with not poking this hornet's nest -- lmk what you think

i don't think 0.8.4 has anything to do w/ this -- ship it for 0.8.3


How do you specify an error occurred in a lane?

New Issue Checklist

[X] Updated fastlane to the latest version
[X] I have read the Contribution Guidelines

Issue Description
This is a basic question, but I can't seem to find it on google.  When an external command exits with a non-zero response code, fastlane puts up an error that looks like:

How do you make that happen if an internal lane has an error (without calling out to an external script)?
In my build, I want to fail if a particular environment variable is not set.  So I want to do something like
    unless ENV.key?('DEPLOYAPP_TOKEN')
      puts("DEPLOYAPP_TOKEN not set.  Exiting")
      exit(1)
    end

But I don't want to do "exit 1".  I've tried return 1, as well, but that doesn't seem to work either.
Thanks.

Good question. I was using raise but it just stops execution and doesn't mark the lane as failed and doesn't even take be to error block.

You can use UI.user_error!("My error message") unless some command``
If some command fails the error message should be shown, could you give that a try? This will not print out the output of the command though.
Alternatively you can use
begin
  sh "command"
rescue => ex
  UI.error ex
  UI.user_error!("My error message")
end

Thanks @KrauseFx.  However, that doesn't exactly do the same as the failure of an external output or built in command.

Notice how the "fastlane summary" doesn't show an error, and it doesn't have the cute explosion emoji.
What I'm trying to do is output an error from within the Fastfile:
  lane ci:
    unless ENV.key?('DEPLOYAPP_TOKEN')
      UI.user_error!("DEPLOYAPP_TOKEN not set.  Exiting")
    end
  ...

Mh, I'd have to dig in deeper to see why the emoji isn't printed. However this doesn't seem like super high priority. I'll close the issue for now, as the basic functionality works now 👍


Environment variables support added

Closes #116
You can add to the settings.yml:
environement:
  Varname: something
  varnam033sjks_s: "something else"

Any idea when this PR is going to be merged?

:+1:

@jorgebastida any ETA on accepting this one?

hi @tdi  .. it looks like @jorgebastida has disappeared for a little while.. I've made a fork of this repo, and am interested in this.  I'm wondering if there's any way to specify a different variable value, based on other values that are known at the time of running gordon?  some way to set like "if stage is PROD then SERVERHOST=productionserver.host.com; else SERVERHOST=developserver.host.com".  Any idea how one would go about doing that?  If that's not already possible, is that something you'd find interesting?

Thank you very much @tdi  - Will try to be more responsive in the future :)

@tdi any option to add the ability to use parameters:
VARIABLE: ref://parameter?


Tweaks to two-fer instructions

I've been reviewing some solutions to the exercise and have noticed a lot of users simply writing a solution that checks if the name is Alice.
It appears they are not running the tests or not unskipping the other tests.

@CristianGiordano This used to be a side-exercise, so most people actually did not get mentored on this.
This change seems okay. You might want to also make it in the problem-specifications repo.

Ah yeh, I've just seen this from in the slack channel. Just getting started so am not up to speed with everything. Thanks for the information 👍


Front-trimmed search result preview should be ellipsized

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.84.2
OS Version: macOS 12.6.5

Steps to Reproduce:

make a file with a long line of code, e.g.// Test using the latest Insiders build to see if your issue has already been fixed: https://code.visualstudio.com/insiders/

search for something near the end of that line of code, e.g. "fixed"
observe that search results preview cuts the beginning of the line, e.g. showing only "your issue has already been fixed: https://code.visualstudio.com/insiders/"
make the search pane narrower so it starts ellipsizing the end of the preview, e.g. showing only "your issue has already been fixed: https://code…"
observe that if you hover over the search result preview, the hover title shows the search result shortened at the start but not at the end where it's ellipsized

This screenshot shows the current behaviour, implying the last search result is on a line starting with 12.

The cutting at the start can make the preview confusing in circumstances where the first parts of the code line affect the meaning of the matched part (e.g. where it's in a string or in a comment rather than in code). It's confusing because there is no indication that the line is cut at the start.
This can be solved by adding an ellipsis … at the start of the cut preview, and by showing the whole line in the hover title.
This screenshot shows what it should look like:

PR coming in a few moments.

Verification steps:

Create a long multi-word line with a specific word at the end. (ie: eeeeeee eeeeeee eeeeee eeeeeee eeeeeee eeeeeeee eeeeeee eeeeeee eeeeeeeeeee test)
Search for that word (ie: test)
Verify that the match for it has a ... at the beginning.


Repo rename

Closes #1018

[x] fastmod -m 'kube-rs/kube-rs' 'kube-rs/kube'
[x]  branch rename in CI parts
[x] hit repo rename button, and rename default branch

All done. Required statuses moved with the default branch thankfully.


Font size discrepancy

Why is cancel smaller?

I believe that's how the designs had it. @morebrownies Should this be adjusted?

Looks off in practice...not sure if intentioned

I used a smaller font-size on the link to help with potential spacing issues.
Are there any scenarios when a link and button display like in the design below @rmisio?
If not, we can bump the font-size. Otherwise, we may be tight on space, especially with some of the translated languages.

Are there any scenarios when a link and button display like in the design below @rmisio?

Yes, I think the screenshot you showed is at least one example.

If not, we can bump the font-size. Otherwise, we may be tight on space, especially with some of the translated languages.

Yeah, I would agree. I think as it is now, we have no idea how long translations are, so there is a chance that some translations will cause that button to break to a new line... but if we keep the font size small, we minimize that risk at least a little.

closing as "working as designed".


chore(installer): bundle windows dependencies in installer

see this guide: https://wixtoolset.org/docs/v3/bundle/
and this example: https://wixtoolset.org/docs/v3/howtos/redistributables_and_install_checks/install_dotnet/
I believe that at lest the C++ runtime needs to be bundled with the windows installer.
Something similar may need to be done for Mac. Need to test on a non-dev machine as developer machines already have all the needed dependencies installed.

Linking Kevin's original ticket just in case https://github.com/Satellite-im/Uplink/issues/812

Is this still ongoing?
CC @sdwoodbury

@dariusc93  This one only is an issue on a brand new windows 10 machine. A few of us have looked into it but it will require some complicated WIX magic and personally I as just hoping the dioxus tools (or something like Tauri) would take care of this for us. We can for sure unassigned it from stu. This isn't high priority.


hyperlink is not working when export hyperlink in html to pdf

Thank you for submitting an issue to jsPDF. Please read carefully.
Are you using the latest version of jsPDF?
Have you tried using jspdf.debug.js?
Steps to reproduce
Ideally a link too. Try fork this http://jsbin.com/rilace/edit?html,js,output
What I saw
What I expected

And you are using which method?

#1567 Should be closed
reasons:

issuer doesnt explain the problem nor provides a jsfiddle/jsbin/plunkr... example
probably uses addHTML oder html2pdf routines which dont provide the functionality he expects
doesnt seem that the issuer is interested in a solution


Generation and load balance does not perfectly add up

Problem description
The combined generation and load plot columns do not add up to (numerically) 0 as they should. Instead, there is a deviation. This clearly is a postprocessing issue as pommesinvest demands the node balance to be levelled out at any time. (If this was not the case, the entire model would end up to be infeasible.)
TODO
Trace back and fix this bug.

Good news: It is indeed a postprocessing issue:
All data sets, production_results_raw, processed_results as well as aggregated_results show the same amount of energy and fulfill the node balance for "DE_bus_el" - as expected.

After spending quite some time, still I haven't detected the cause for the deviation. I'll keep trying.


Update dataobject-parser.js

Don't remove all falsy values.

Issue when parsing values that are falsy; they would not be included in the final object.

Can someone add a PR for some tests that check this?


Errors in some cases are unusable/PR was reverted

In https://github.com/microsoft/ClearScript/pull/204, which was approved and merged with master I fixed and added test cases for several areas where error reporting was quite non-useful. Shortly after in https://github.com/microsoft/ClearScript/commit/28d6aa212f7b2e95ec4bde43e543f246690253dc#diff-3e3487db6f5281f14631a689d9e902314cecb42d32976a21758dbbece8e7a801
those changes and the tests that prove they work were reverted or lost. I don't know if it was intentional or not.
However, the changes are actually very important for us as we would like good errors with line numbers to be reported in all cases for our customers. Can I ask for this to be revisited please?

Hi @phxnsharp,
As we indicated in the PR comment here, we ended up adjusting the error handling code slightly and moving the tests to a different file after merging. At the time there were massive code changes taking place, as ClearScript was going cross-platform, so it's certainly possible that we broke something.
However, we just ran your original tests from the PR, and all five worked with the following text changes:

Unknown documents show up as "[unknown]" rather than "Unknown" in the error messages.
In VBScriptEngine_ExceptionDetails2, the error message is "Object doesn't support this property or method: 'test'" rather than "Class doesn't support Automation: 'test'".

Please let us know if you're aware of any additional changes or regressions.
Thank you!

We have a suite of unit tests that test our integration with ClearScript. When we recently switched it from our custom compiled version from my fork to the latest release, we started getting a pile of failed tests due to error messages being much simplified. Based on that and seeing the mass deletions, I assumed it was totally broken. But I see that all 14 tests are really parameterized versions of a single test, and that other tests we put into place are still working. So it looks like just one case is broken.
The case is when you have provided a COM object to the script host and the script attempts to call a non-existent property. The expected error in our tests looks like:
    Object doesn't support this property or method: 'out1.description'
      at run (Script:3:18) -> out1.description = "new Value"

but all we are getting is:
   Invalid object or property access

There are a lot of permutations and I don't have my original PR up and compiling right now. The test I originally wrote named VBScriptEngine_ExceptionDetails_NoDebugger on line 545 of VBScriptEngineTest.cs in my PR might prove this case. Or, we might need to write a new test doing the same thing with ComVisibleTestObject (it might be specific to having a COM object).
Thanks for your help! Glad to know it is isolated to a specific case.

Hi @phxnsharp,

The test I originally wrote named VBScriptEngine_ExceptionDetails_NoDebugger on line 545 of VBScriptEngineTest.cs in my PR might prove this case.

We've run all five tests from your PR on the latest ClearScript (currently version 7.1.5), and they all succeeded with the text changes above. One of these tests, VBScriptEngine_ExceptionDetails2, uses a ComVisible object.
Please let us know if you can isolate an errant case.
Thanks!

I have someone trying to do that, but it looks like in the refactor the test projects were all removed from the "NoV8" solution. Is there still a way to run the tests without setting up V8?

I have someone trying to do that, but it looks like in the refactor the test projects were all removed from the "NoV8" solution.

The ClearScript.NoV8 solution never included test projects; that's mentioned in the build instructions.

Is there still a way to run the tests without setting up V8?

Sadly, it wouldn't be trivial. You could add one of the existing test projects to ClearScript.NoV8, but then you'd have to strip out any V8-dependent test code.
Another option might be to create a basic console app, bring in ClearScript via NuGet, and add your test code. We'll be happy to investigate any issue you find that way.
Cheers!

Hi @briell99,
Thanks for posting the test case! We've confirmed that it currently causes ClearScript to lose a portion of the error details. We'll  have a fix in the next ClearScript release.
Thanks again!

Fixed in Version 7.1.6.


Cert with p12 Password

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject

Hi, using fastlane to generate Certificate and Provisioning profile and then will be using phonegap build to actually build app. Its all worked well with cert and sigh but generated p12 is without password and phonegap build require password. I have checked issues and most of the references are for pem and commit https://github.com/fastlane-old/pem/commit/557747a8b91df6c8c300fda43aafad16fe957978
Question Description

Please help, what would be be possible option?

@wahmedswl you can use the p12_password option to specify your pem password for the p12 file. If you do not provide this option, it will use empty password as default value.

@RishabhTayal thanks, so i have to add pem in fastfile and cert will use that?
Thanks for your quick response.

@RishabhTayal i have added pem but p12 generated by pem doesn't match with the provisioning profile generated.
And when i also tried to upload the p12 of provisioning profile, i get password error on phonegap build.
I just want to use the cert and sigh part of fastlane, and then build will be carried over by phonegap build service.
Just want to generate valid p12 with password.
Thanks

I'm facing a similar issue, but will be uploading to MS AppCenter.
I'm creating a script which will be using commands cert and sigh for adhoc and production profiles.
AppCenter also requires a p12 with a valid password that is known to me.

@wahmedswl did you get this to work?

@RishabhTayal thanks, so i have to add pem in fastfile and cert will use that?

Could you share how?

@JesperBaltzersen nope, i have checked the api where that can be applied but still i am directly uploading right now and not using Phonegap Build due to this. I am also waiting for response.

Just to make sure I understood correctly:
Is the issue that you want cert to create a certificate in p12 format, set with a specific password?

@janpio I'm not sure if you're asking me or @wahmedswl but for me that is the case yes.
The work around I'm using now, is just to let cert do most of the work and then extract the p12 manually from keychain.

I was asking you both as you seemed to agree you have the same issue - but I wasn't sure if I had properly understood it.

Hi, can we please re-open this issue as its still not answered
Thanks

Please reply yourself when @fastlane-bot posts next time so it doesn't actually close the issue. Thanks.

@wahmedswl this needs to be kept open as still not answered.
Thanks

does anyone have workaround? i'm also having same issue here.

@bhavincb nope, still no luck with it. But, moved to fastlane match and it can generate certificates and provisioning profiles with password. Used that with phonegap build and its great 👍

@wahmedswl  can you please share the steps for that? it will really help me a lot.

@bhavincb fastlane match official guide is awesome and available at https://docs.fastlane.tools/actions/match/

@wahmedswl but p12 file generated by match gives error Too few bytes to read ASN.1 value. that's why i had asked for help.

@bhavincb its because fastlane match keep stuff encrypted. You need to follow https://docs.fastlane.tools/actions/match/#manual-decrypt to decrypt.
openssl aes-256-cbc -k "password" -in "certs/development/PFG6487A72.cer" -out "development/development.der" -a -d
openssl x509 -inform der -in development/development.der -out development/development.pem
openssl aes-256-cbc -k "password" -in "certs/development/PFG6487A72.p12" -out "development/development_key.pem" -a -d
openssl pkcs12 -export -out "development/development.p12" -inkey "development/development_key.pem" -in "development/development.pem" -password pass:password
openssl aes-256-cbc -k "password" -in "profiles/development/Development_com.app.id.mobileprovision" -out "development/Development_com.app.id.mobileprovision" -a -d

@wahmedswl  it's giving me error error reading input file. i have created certificates and provision using match action and issues follow command. password used is the same as github repo password.
openssl aes-256-cbc -k "password" -in "U2M49JU375.cer" -out "cert.der"  -a -d

@bhavincb seems like you are at the root of the repo, you need to give full path like openssl aes-256-cbc -k "password" -in "certs/development/PFG6487A72.cer" -out "development/development.der" -a -d

@wahmedswl  i'm using below command for match. and after navigating to .cert folder issuing decryption command.
match(type:"appstore",output_path:".cert/")
cd .cert
openssl aes-256-cbc -k "password" -in "U2M49JU375.cer" -out "cert.der" -a -d

@bhavincb you don't need to use match now, just clone the certificates repo and do this at there.

@wahmedswl  after using above command it download certificates and provision to .cert folder. and i can see it is already decrypted as .p12 file starts with -----BEGIN RSA PRIVATE KEY-----.
so i don't think there is any need to decrypt. but still i'm getting the error.

@bhavincb we go with the repo route and generated combined p12 and decrypted provisioning profile.
Do you get error on build.phonegap when unlocking certificate and it asks for password?

@wahmedswl  actually i'm using it with ionic build. in which i have to pass the path to p12 certificate ,provision and certificate password.

i was able to solve the issue by using following commands.
match(type:"appstore",output_path:".cert/")
cd .cert
openssl x509 -inform der -in development.cer -out development.pem
openssl pkcs12 -export -out cert.p12 -inkey development.p12 -in development.pem -password pass:password

and after that we will pass cert.p12 and provision profile to the build command.


Update FormRequest.php

Fix ValidationException

I know this is old, but you are not fixing anything here, first of all, you are changing all namespaces to yours namespace, that is totally wrong, and by saying "Fix ValidationException" you are not fixing it either because you are adding something that in the source code in Illuminate, it is exactly how it is in this repo (without your change). Look here and here.
So please, @albertcht decline this PR, it is all wrong.


Currency tests oddities.

for currency in currencies { for locale in locales { /* style.interpret */ } } // works
for locale in locales { for currency in currencies { /* style.interpret */ } } // crash

I found the problem: The currency lexicon key did not take into account the currency code, only the locale, because it was assumed that the format did not change in any other way than by using the the formatter’s  currency prefixed characters. This assumption was wrong, and on occasion led to the incorrect lexicon being used. Using a proper cache key fixes this.


Error on npm install

Hello, I tried your starter kit but it errors before finishing the installation.
- Creating a new Snowpack app in /code/node/starter-kits/snowpack

- Installing package dependencies. This might take a couple of minutes.
node:internal/errors:415
    ErrorCaptureStackTrace(err);
    ^

Error: Command failed with ENOENT: npm install react@17 react-dom@17
spawnSync npm install ENOENT
    at Object.spawnSync (node:internal/child_process:1084:20)
    at Object.spawnSync (node:child_process:668:24)
    at Function.module.exports.sync (/code/.npm/_npx/daf316dc1912ab5b/node_modules/execa/index.js:167:25)
    at installPackages (/code/.npm/_npx/daf316dc1912ab5b/node_modules/snowpack-start/src/index.js:362:11)
    at main (/code/.npm/_npx/daf316dc1912ab5b/node_modules/snowpack-start/src/index.js:579:3)
    at processTicksAndRejections (node:internal/process/task_queues:94:5) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawnSync npm install',
  path: 'npm install',
  spawnargs: [ 'react@17', 'react-dom@17' ],
  originalMessage: 'spawnSync npm install ENOENT',
  shortMessage: 'Command failed with ENOENT: npm install react@17 react-dom@17\n' +
    'spawnSync npm install ENOENT',
  command: 'npm install react@17 react-dom@17',
  exitCode: undefined,
  signal: undefined,
  signalDescription: undefined,
  stdout: '',
  stderr: '',
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}
npm ERR! code 1
npm ERR! path /code/node/starter-kits
npm ERR! command failed
npm ERR! command sh -c snowpack-start

Thanks for the report. I think I know what the issue is, working on a fix now.

Just published, try the newest version 1.0.0-beta.3.

Thanks a bunch!


Qiskit Runtime primitive options difficult to find

URL to the relevant documentation
No response
Select all that apply

[ ] typo
[ ] code bug
[ ] out-of-date content
[ ] broken link
[X] other

Describe the fix.
At the Partner Forum I heard again that Qiskit Runtime primitive options are difficult to find in the docs. We attempted to fix this by adding a table in #1124, but perhaps that wasn't enough. This issue is meant to brainstorm on how we can improve on the experience.

one option is to link to the table in the API reference more: https://docs.quantum.ibm.com/api/qiskit-ibm-runtime/options#classes

Based on some other feedback, I suggest we rename the Advanced runtime options page to Overview. This page will have these sections:

Introduction

Explains the options tiered structure
Explains the client default (Unset) and where to find server defaults
Explains option precedence rules (related issue)

Options categories for v1/v2 sampler/estimator. Something similar to the current table, but maybe as clickable tabs and no code examples, to make it more readable.
Setting options for V2 primitives
Setting options for V1 primitives
Commonly used options
Turning off error suppression and mitigation (related issue).

I'd also suggest we rename Configure runtime compilation to Configure error suppression and focus on dynamical decoupling and twirling. Now that we don't transpile anymore in the primitives, "compilation" becomes less interesting compare to "error suppression".
One last comment - having both V1 and V2 information on the same page makes it really cluttered. Is it possible to have separate pages for V1 and V2 primitives? That's also make clean up easier when it's time to drop V1.
@javabster let me know what you think 🙂

One last comment - having both V1 and V2 information on the same page makes it really cluttered. Is it possible to have separate pages for V1 and V2 primitives? That's also make clean up easier when it's time to drop V1.

I'm not sure about this.  Of course it would be possible, but we don't split out the pages anywhere else; so would we want to then split them out everywhere?  Also, having two sets of information  can easily lead to them getting out of sync.  And might people be using the V1/V2 tabs to better understand the changes?  We of course have the migration guide, so this shouldn't be a problem, but it might be helpful.
That being said, I'm open to discussion.

Something similar to the current table, but maybe as clickable tabs and no code examples, to make it more readable.

Totally on board with making this better.  What would the tabs be?
I really like the code examples, as a non-coder.  What if we re-formatted it like this:

Category
Description

Resilience
Advanced options for configuring error mitigation methods such as measurement error mitigation, ZNE, and PEC. Estimator only.

estimator.options.resilience.measure_mitigation = True

Dynamical decoupling
Options for dynamical decoupling.

estimator.options.dynamical_decoupling.enable = True

Jessie's ideas sound good to me, but I agree with Becky I think we should keep v1 and v2 together.
Also worth considering as part of this discussion is @kevinsung is working on an "error mitigation techniques" page that will also cover some of the runtime options, but it's more focused on explaining the error mitigation concepts. we should just make sure that these pages flow nicely together and don't duplicate too much

Something similar to the current table, but maybe as clickable tabs and no code examples, to make it more readable.

Totally on board with making this better. What would the tabs be? V1 Estimator, V1 Sampler, etc? I think that would be nice / doable.

Yeah I was thinking one tab for each V1/V2 sampler/estimator combination.

I really like the code examples, as a non-coder. What if we re-formatted it like this:

Sure. The only reason I suggested the removal is because the code example makes 1 column really wide and the others really narrow. If we can solve this without removing code example that's even better.

Also worth considering as part of this discussion is @kevinsung is working on an "error mitigation techniques" page

That's really good to know! There was some feedback from Partner Forum that researchers tend to want to know more about the underlying concepts, so this would be quite helpful.

Another suggestion. I saw a table with all the options and their defaults in a challenge notebook. Today to find the default value of a ZNE extrapolator, for example, I'd need to go into the API reference, click on EstimatorOptions, then ResilienceOptions, then ZneOptions, and search for extrapolator. And this is after already knowing where to look. A table would be super handy.
Having said that, the down side of a table is of course we'd have to keep it up to date.


feat: when folding group, move cursor to folded group header

Current behaviour:
https://user-images.githubusercontent.com/85341530/141742264-3a2aee57-1b8a-47dc-8364-3f5c5308e396.mp4
New behaviour:
https://user-images.githubusercontent.com/85341530/141742330-45ef17b3-9919-4282-ba6c-f5ba9ad3507e.mp4
Would it make sense to automatically jump the first item when unfolding group?

Depends on #115

Development on the main branch is EOL.
Trouble has been rewritten and will be merged in main soon.
This issue/feature either no longer exists or has been implemented on dev.
For more info, see https://github.com/folke/trouble.nvim/tree/dev


⚠ Samsung has degraded performance

In 2e1f02c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2056 ms

Resolved: Samsung performance has improved in b2293a9.


Project settings refactor into subcomponents

Is your feature request related to a problem? Please describe.
The project settings component is only one component that has all the logic. As we add new features into this, it gets more and more complicated and should be split into few subcomponents. The main idea behind this is to separate the code, but also do some general refactoring within each of the components.
Describe the solution you'd like
The subcomponents should be be split based on the logic they have such as: data-schema (has all the information about the attributes and the records/project export/import), embeddings (has all the information about the operations with embeddings), labeling-task (has all the information about the labels and labeling-tasks) and project-metadata (has all the information about the project itself, enables editing some of its data and deleting the project).

Steps to implement:

Create four subcomponents: dataschema, embeddings, labeling-tasks, project-metadata (1h)
Move the logic from project-settings to dataschema (5h)
Move the logic from project-settings to embeddings (5h)
Move the logic from project-settings to labeling-tasks (5h)
Move the logic from project-settings to project-metadata(5h)
Discussions and meetings (2h)
Testing all functionalitites in project-settings(3h)
Review and review comments(3h - 4h)

ESTIMATION: ~30h


Allow DetailLevel to be set

Setting DetailLevel to ReturnAll is required to have the ExternalTransaction object returned in the response. And without this code addition I'm unable to set a DetailLevel string on the request object.

DetailLevel is a property of AbstractRequestType which GetOrdersRequestType extends.
As stated in the eBay documentation DetailLevel is a repeatable field which allows you to specify it more than once in the request. E.g.
<GetOrdersRequestType>
    <DetailLevel>ReturnAll</DetailLevel>
    <DetailLevel>ReturnHeaders</DetailLevel>
</GetOrdersRequestType>

The SDK handles this by treating the field as an array like property of the request object.
$request = new Types\GetOrdersRequestType();
// Notice that we use [] because DetailLevel is array like and we are pushing values into it.
$request->DetailLevel[] = 'ReturnAll';
$request->DetailLevel[] = 'ReturnHeaders';

Because the property is array like we can also do the following by assigning an array.
// Notice that we no longer need [] because we are assigning an array to the property.
$request->DetailLevel = ['ReturnAll', 'ReturnHeaders'];

Thank you for the clarification. If I missed this in the documentation I apologize.

Hi Josh,
It's me that should be apologizing. A few people have had the same issue regarding DetailLevel and it's down to the fact that I haven't created any documentation for it. :(


Pre-compiled Version

It's possible to upload a precompiled version? I don't want to compile it myself.

Thanks for the suggestion, but I have no plan to provide a precompiled version.

Yes i do not understand how Cmake works even with the documentation. Can you please provide the exact syntax


Removing reference to r.o

Description
As @jakehader mentioned in #830 , we have a strange "reactor.operator" object in a small number of places. Even though the primary reference is "operator.reactor".
This PR solves about half of those cases.  The second half will be harder to unwind.
I have also opened some PRs in downstream repos to help pre-empt the second half of ticket #830.

Checklist

[X] This PR has only one purpose or idea.
[X] Tests have been added/updated to verify that the new/changed code works.

[X] The code style follows good practices.
[X] The commit message(s) follow good practices.

[X] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[X] The documentation is still up-to-date in the doc folder.
[X] The dependencies are still up-to-date in setup.py.

Curious -- why is distortionVsTime getting removed?

Curious -- why is distortionVsTime getting removed?

Well, as far as I can tell, it references an Interface that hasn't existed in years, so it hasn't been USED in years.


Customize bootstrap with variables?

Is there a way to have a file with all the variables that gets loaded before bootstrap?

I am sorry, but I do not understand your question. What kind of variables are you referring to?

@simonihmig see:
http://getbootstrap.com/customize/
example with bootstrap-sass:
https://github.com/twbs/bootstrap-sass#sass-1

Ok, I see... this should be easy. You will have to install the css preprocessor addon you want to use, e.g. ember-cli-less, if you have not done so already.
Then you have to opt out of including the static bootstrap CSS, and instead import the bootstrap less/sass files and set your variables. See http://kaliber5.github.io/ember-bootstrap/getting-started/
There is direct support for less, as bootstrap ships with it. See the last chapter in the Getting Started Guide. To use Sass you would have to use Sass-port your already linked to.

Ah makes sense. Thanks @simonihmig


Added KryptosChain to Nova delegations

https://kusama.subscan.io/extrinsic/16932908-2

Thank you for your request!
@KryptosChain
You can check how it will look like:

App screens

If you like it we will merge

Yeah it looks great! Thanks very much @stepanLav !


Windows applications not terminating between feature files.

As a user of sentinel I want to be able to dictate if an application stays open or closed between feature files. Currently applications are not being closed correctly after a run, and there are no options for this.

Obsolete?


Goat counter issue on staging branch

Hello all,
So after a bit of investigation it seems like Google App Engine has some issues with external scripts being loaded on static sites that are client-side only. I got to this conclusion by deploying exactly the same code to Netlify to see if I could replicate the issue there but it worked perfectly fine for me. The url is below:
https://staging-netlify-test--elastic-noether-393172.netlify.app/
I then changed the above env to use SSR(server side rendering) on the front-end so we aren't relying too much on the client-side and we get the added benefits of everything being loaded into the app state at once. This would then stop the issue of the external script being loaded before the client-side was ready breaking the site on refresh.
To further back up my thoughts, I have attached a screengrab of the associated env with the network tab open displaying the Goatcounter script publishing a count event.

I have now created a PR to go into the main branch for staging on Google App Engine with the same config settings as above. I will need this to be approved to merge into main then we can further test.
https://github.com/ossf/scorecard-webapp/pull/74
Thanks,
Mark

Yep, that fixes the breakages but now the sidebar is gone (see image below). Also, the GitHub Stars icon is flaky. The number gets populated sometimes but sometimes it doesn't.

Yeah I saw that.
I am leaving in about 30 mins but I am going to look into this issue.

Another question is - we are now making these changes (like disabling ssr) for AppEngine deployment. If we move to Netlify at a later point, would these changes still work?

I am leaving in about 30 mins but I am going to look into this issue tomorrow

Thanks!

Yeah they will still work, and in that change which fixed the Goat counter issue, we actually enabled ssr
Ideally you need to move to Netlify sooner rather than later. It would be great if OSSF could get it setup now so we could easily port it over and make sure it is running on an optimal server for what you need.


Automatically change init-user-type when changing Date of Birth

A number of users have signed up with their child's date of birth, which is causing issues when they try do actions intended for adults, like start a dojo, or add child.
These users have generally updated their profile with their own details, but their user type stays as attendee-o13. We should change the user's user type according to the date of birth entered when updating their profile.

No.
DoB should be un-editable by non cdf to avoid bypass of restrictions.
I understand that it's problematic right now, but the solution is to fix the reg flow to avoid mistakes and to provide admin tools to change it more easily.
Letting an user change is DoB and his usertype i slike removing any restriction you have on any flow :)
@rosalanghammer

Issue is not about user being able to change DOB (this should be locked), but if CDF Admin update DoB then the usertype should update.
Agree that a better flow should negate a lot of these issues but this needs to be done to allow past users profiles to be updated.

The outcomes of this:

Date of Birth is locked for all users at sign up
CDF admins can change Date of Birth


Making agent's claim process more robust

The agent's claim mechanism is not designed for unreliable RPC endpoints. While we cannot account for any unreliability, we can make the process more robust.
This is currently only a collection of optimizations to make the claim process more robust.

increase validity period in frontend
increase min validity period in contracts
allow MEV claiming on behalf of others
introduce agent check that it only fills requests with X minutes validity time left
Allow new claims after request expiry (Used by the agent at own risk)

increase validity period in frontend
Currently standard usage comes from frontend. As an easy measurement we can increase the standard validity period to 30 min.
increase minValidityPeriod in contracts
We can strictly forbid values lower than minValidityPeriod and need to revise the current values.
Allow (MEV) claiming on behalf of others
Currently only who is in possession of the private key can claim for an associated address. If for whatever reason, the unit handling the private key is not able to claim then the request might expire. Similar as for challenges, it could be possible to claim on behalf of others.
In this case the game theory has to be understood and designed. The main question is, who is putting up the claim stake and takes the risk of being challenged and how or even should this entitiy be incentivized to do  so.
Additionally, an agent might want to enable or disable such a service (increasing failure risk, but getting higher rewards). The discussion seems similar to the Montiroing service in Raiden.
Open questions:

claim stake
enabling/disabling
whitelisting of specific addresses (Gelato use case)
Incentivization schemes for external parties
Time frames for external claiming (i.e. only if expiration date is in less than X minutes)

Agent only fills requests with at least X minutes before expiration
The agent does not check how much time is left before a request expires. This actually means that the agent assumes that there is always enough time to fulfill a request. Due to network problems or other technical reasons, it could be the case that the agent only sees the request right before expiration. In this case, he will still try to fill and claim.
We should introduce a check that there is enough time to fill and claim a request.
Open question:

Should this check be configurable ?

Allow new claims after request expiry
Currently we have the following model:
Claiming is only possible before the request expiration. After expiration it is only possible to withdraw the tokens and send them back to the user. This is a hard 0/1 requirement.
We initially introduced this condition to enforce the agent not to fill requests after expiration. Because he wouldn't be able to claim the tokens back, it wouldn't make sense to fill it in the first place. Even if the fill was correct it would effectively be giving tokens away for free.
Thinking about this I don't see any harm to allow claims after request expiration. The effective change here would mean that a user is still able to withdraw its tokens, with the same conditions as today (there must not be any active claims), but it is also possible for an agent to claim that he filled the request.
It still does not make sense for the agent to fill requests after expiration because the user has the option to withdraw its tokens, so most likely the agent software would not change, and the above measurements are still enhancements to the current code base.
For a rationale behaving agent it would simply be to risky filling a request which is expired or close to expiration, because he risks losing its tokens (same as in the current code base).
But if any unforeseen events happen and the agent was not able to claim a valid fill, he would still have at least the chance to rightfully claim its tokens. I don't really see a new attack vector opened to this change and don't see the point of not allowing new claims after request expiration.

Done


Duplicate Option

Description of the new feature / enhancement
Need a Duplicate option on right clicking a file.
Scenario when this would be used?
Whenever I need to new file but similar to something, that is already in hand. I have to copy and paste the file.
But clicking, duplicate in a single click would copy and paste the file.
  You can drag+ctrl and that will create a file or as you pointed out, copy & paste (both works via keyboard and right click)
I think this would just add yet another item to the right click menu for an operation that is pretty quick already. In theory, if /dup #139 is built, that would enable this as you could just point it against itself but wouldn't be a single click.
But for a single click, i don't see PowerToys team building this feature out as listed.


FIP-9-Domain Leasing Feature

Domain Leasing to mean a wallet domain owned by another entity that has an agreement with the wallet in question to inflate and share registration fees of it's users.
Motivation:
Domain owners view their domain as valuable assets and would prefer not to sell. Leasing feature would lead to longer participation of both the lessor and the lessee in the industry.
Implications:
Domain integration should be of two types.

Direct integration (would mean Domain is owned by the wallet)
Indirect Integration (would mean Domain is leased by the wallet in question).

Domain Leasing should have registeration fees split between 2 wallets:

FIO foundation wallet. ($2) constant.
FIO wallet to which domain is assigned.

Domain Leased means price will always be higher than $2.
At registration (in-wallet). Users pay to a smart contract address instructed to split payment in a way that:

$2 goes to FIO foundation wallet (konstant)
$balance (set by domain owners) goes to Fio wallet to which domain is assigned.

Bb

Hi

Hello baby

Technically this can be accomplished today using a custom instance of the Registration Site ran by the domain owner.
The challenge we wanted to address with "Registration Origination Bounty" is that the entity which brought the user to FIO (enabled them to register a FIO Address) only shares the initial registration fee. Each subsequent renewal is going to whatever wallet the user used to renew as it's going to the TPID which enabled the renewal transaction. "Registration Origination Bounty" envisioned that the TPID used on the original registration will receive 5% of all fees ever paid by that FIO Address, even if they switch wallets in the future. The wallet TPID will not be affected.
Here's the fee distribution before "Registration Origination Bounty":

85% BPs
10% TPID
5% Foundation
After "Registration Origination Bounty":
80% BPs
10% TPID
5% Wallet which facilitated the registration
5% Foundation

As to being able to mark-up addresses at Protocol level we contemplated that there might be a benefit in allowing domain owners to add % price increase over whatever fee was set by BPs in exchange for some benefits offered. For example may offer discounts on amazon.con for users who pay with @vip address.

I know it is possible through the registeration site. However, for most domain owners who have no technical knowledge this might create a drawback that's why we need this feature to be addressed at protocol level. Hence, the reason for this proposal.

Yes, I think many domain owners have no technical knowledge, they indeed need those professional people to help them in this aspect. Most domain owners really want to help in promote the use of fio, and if they can get payment by domain leasing, both fio foundation and those who want to use the domain will sure can be benefited as parts of the fee collected will reward them also.  (Not just the domain owners).

Wouldn't that incentivize domain squatting? I think we would want the domain owners to provide some value to the ecosystem beyond just being first to register a domain, i.e. get users to register a FIO Address on their domain.

@pawmmm I don't think so. Every domain owner that doesn't work to keep it's domain would ultimately lose it at some point. It is the duty of every domain owner to act as an ambassador of FIO providing education to get more users registered on its domain thereby expanding the network. Even wallets who need it most feel adamant at integrating any protocol into their product. A wallet owner will have to hold series of meetings and continuous engagement on its own to get them interested (which would have cost FIO a fortune if they were to reach every company by themselves). I believe it's a combined work of personal efforts that would make the ecosystem grow faster. It's as simple as no work; no incentive.

I am also in support of integrating commission added by domain owners at the protocol level because it is very discouraging to have to think about spending other expenses (having a registeration site) in order to get some advantage on your investment. It also won't be smooth from user experience perspective. Domain owners still have to work to earn. I think you should accept this proposal.

I think that's being addressed with:
Premium FIO Addresses. This feature will allow domain owners to programmatically increase price of FIO Addresses on their domains in order to offer premium services to those who register such address. For example, if you pay with a FIO address on the :vip domain, you will get VIP perks.

Technically this can be accomplished today using a custom instance of the Registration Site ran by the domain owner.
The challenge we wanted to address with "Registration Origination Bounty" is that the entity which brought the user to FIO (enabled them to register a FIO Address) only shares the initial registration fee. Each subsequent renewal is going to whatever wallet the user used to renew as it's going to the TPID which enabled the renewal transaction. "Registration Origination Bounty" envisioned that the TPID used on the original registration will receive 5% of all fees ever paid by that FIO Address, even if they switch wallets in the future. The wallet TPID will not be affected.
Here's the fee distribution before "Registration Origination Bounty":

85% BPs
10% TPID
5% Foundation

After "Registration Origination Bounty":

80% BPs
10% TPID
5% Wallet which facilitated the registration
5% Foundation

As to being able to mark-up addresses at Protocol level we contemplated that there might be a benefit in allowing domain owners to add % price increase over whatever fee was set by BPs in exchange for some benefits offered. For example may offer discounts on amazon.con for users who pay with @Vip address.

You should not decrease the BP % to give the balance to another actor. I would reccomend that the TPID percentage be split since they are the "reseller". This is how it works in other industries.
The amount of the TPID split coud be  variable.  The TPID % would decrease as the facilitator % increases.

You should not decrease the BP % to give the balance to another actor.

Thanks for chiming in, Brad. One of the concerns I have is the percentage of fees going to the BPs. It could, over time, create a situation where they are entrenched. How do you feel about the overall tokenomics model in that regard? I'm open to discussing decreasing that percentage if it makes sense, but obviously that would have to be a change the BPs wholeheartedly support and that would only make sense for them if they thought it would lead to much larger adoption (i.e. more usage and more fees for them to collect over time).

@Brad-CurrencyHub I want to be believe that the proposed solution is better as it is for two reasons.

A wallet owner that owns it's domain and feels under-rewarded by the set reward system (10% TPID) could still go ahead and select option 2. (Indirect Integration) and adjust it's registeration fees as deemed fit (like adding extra $1) which goes directly to it's $FIO address to which domain is mapped.
The indirect Integration would in most cases be as a result of sharing agreement between lessor and lessee. Which has to be set precisely on what amount is agreed and not based on percentages as in TPID rewards

@lukestokes > "create a situation where they are entrenched."
Not sure what you mean by this.

@lukestokes > "create a situation where they are entrenched."
Not sure what you mean by this.

If too many tokens in the economic system go just to BPs and no other stake holders in the system then they can vote for themselves, change parameters, etc which benefit the BPs but potentially harm other stake holders. A healthy balance ensures BPs are well rewarded and other stake holders are also.

@lukestokes > "create a situation where they are entrenched."
Not sure what you mean by this.

If too many tokens in the economic system go just to BPs and no other stake holders in the system then they can vote for themselves, change parameters, etc which benefit the BPs but potentially harm other stake holders. A healthy balance ensures BPs are well rewarded and other stake holders are also.

Understood. Thanks.

See current initiatives here: https://fioprotocol.atlassian.net/wiki/spaces/WP/overview#Current-Initiatives


[BugFix] Make S3Filesystem thread-safe

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

Plz remember cherry-pick to branch-3.0, branch-2.5.

run starrocks_admit_test

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


llama7B issue

Hi, i am trying to run the SFT step, using 4 A100 80GB, report error:  starting 4 processes for FSDP training setting RLIMIT_NOFILE soft limit to 1048576 from 1048576 /opt/conda/lib/python3.8/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 2 leaked semaphore objects to clean up at shutdown warnings.warn('resource_tracker: There appear to be %d ' Bus error
Even if i use batch_size=1, the issue is still there.  And i wonder if you have the finished model training on anthropic_hh with llama7B?

I saw the same error even running the demo script using 8 A100 40GB
``
What I got is
building policy
starting 8 processes for FSDP training
setting RLIMIT_NOFILE soft limit to 1048576 from 1048576
/usr/lib/python3.8/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 2 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
Bus error (core dumped)

@Emerald01 have you figured the problem?

Not an exact replication of the FSDP version, but, I recently reimplemented DPO with QLoRA on LLaMA-7B model
The model is already available on huggingface: https://huggingface.co/abaheti95/dpo_qlora_hh
Here is the respective code for implementation: https://github.com/abaheti95/LoL-RL/blob/main/dpo_qlora_llama_hh.py

Looks good, thanks! But i wonder the performance gap between FSDP and QloRA?

Even I'm not sure. I would also love to know if there is a huge difference. Let me know if you notice anything.

I attempted using their trainer first but was noticing very slow training. I asked TRL about this and they had certain pointers on how to speed it up https://github.com/huggingface/trl/issues/729.
By that time, I already had my custom training loop ready, so I just went forward with it. I could finish 1 epoch of HH-RLHF in about 1.5 days using 2 48GB GPUs. In my experiments, I was able to simultaneously keep 2 models (reference and target) in two GPUs without any memory errors.
Personally, I'm not super familiar with Deepspeed and FSDP compression and multi-GPU. I tried using deepspeed earlier but had too much struggle with a custom training loop.

Awesome work, I will check on this to see how it works soon! BTW, besides the obvious speed issues, I also noticed there is discussions on how well DPO converges, it said that in some cases the chosen actions also got decreased or the margin between the chosen and the rejected is small. https://github.com/huggingface/trl/issues/800

In my DPO training attempt, I also saw that margin was increasing but both chosen and rejected reward decreased.

In my training attempt (full model fine tuning with llama on HH datasets), in SFT stage, the training loss is not going down. In DPO, all outputs are NAN.

In my training attempt (full model fine tuning with llama on HH datasets), in SFT stage, the training loss is not going down. In DPO, all outputs are NAN.

same question

Hi @JiuhaiChen , I also encountered this question. Did you solve it?

@chchenhui Hello, I have encountered the same problem, have you solved it?

@JiuhaiChen Have you solved the buds error problem?

I saw the same error even running the demo script of model Pythia28, using 8 A100 40GB at Google Cloud
python -u train.py model=pythia28 datasets=[hh] loss=sft exp_name=anthropic_dpo_pythia28 gradient_accumulation_steps=2 batch_size=64 eval_batch_size=32 trainer=FSDPTrainer sample_during_eval=false model.fsdp_policy_mp=bfloat16
What I got is
building policy
starting 8 processes for FSDP training
setting RLIMIT_NOFILE soft limit to 1048576 from 1048576
/usr/lib/python3.8/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 2 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
Bus error (core dumped)

The warning itself seems negligible as it is some hiccup from multiprocessing, but core dump is a serious problem. This codebase looks pretty clean and I like it a lot. Could you please check what is going on, so users can gain confidence on its robustness.

I have the same issue, so has the core dump problem not been solved?

★---> train stats after 160512 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  's0     eogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.3122', 'grad_norm': 'nan', 'counters/examples': 160512, 'counters/upog  dates': 5016}                                                                                tep _   -2 ★---> train stats after 160544 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  's0ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.3297', 'grad_norm': 'nan', 'counters/examples': 160544, 'counters/upog     2dates': 5017}                                                                                tep _-2 ★---> train stats after 160576 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  's0ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.2214', 'grad_norm': 'nan', 'counters/examples': 160576, 'counters/upog  dates': 5018}                                                                                tep _ ★---> train stats after 160608 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  's0ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.3119', 'grad_norm': 'nan', 'counters/examples': 160608, 'counters/upog  dates': 5019}                                                                                tep _ ★---> train stats after 160640 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  's0ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.3496', 'grad_norm': 'nan', 'counters/examples': 160640, 'counters/upog  dates': 5020}                                                                                tep ★---> train stats after 160672 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  'ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.3799', 'grad_norm': 'nan', 'counters/examples': 160672, 'counters/upog  3dates': 5021}                                                                                te 4 ★---> train stats after 160704 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'logps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.2856', 'grad_norm': 'nan', 'counters/examples': 160704, 'counters/up  8dates': 5022} ★---> train stats after 160736 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l66ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.3715', 'grad_norm': 'nan', 'counters/examples': 160736, 'counters/updates': 5023} ★---> train stats after 160768 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l    -ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.4876', 'grad_norm': 'nan', 'counters/examples': 160768, 'counters/up   -dates': 5024} ★---> train stats after 160800 examples: {'rewards_train/chosen': 'nan', 'rewards_train/rejected': 'nan', 'rewards_train/accuracies': '0', 'rewards_train/margins': 'nan', 'l  2ogps_train/rejected': 'nan', 'logps_train/chosen': 'nan', 'loss/train': 'nan', 'examples_per_second': '5.4887', 'grad_norm': 'nan', 'counters/examples': 160800, 'counters/updates': 5025}                                                                                24 Finished generating 1 epochs on train split writing checkpoint to .cache/root/anthropic_dpo_pythia69_2024-09-22_09-38-57_157738/LATEST/policy.pt... [rank0]:[2024-09-22 18:41:34,834] torch.distributed.fsdp._debug_utils: [WARNING] FSDP _optim_state_dict() profiling:  defaultdict(<class 'float'>, {'preprocessing': 0.012136668432503939, 'preprocessing_with_comm': 0.042172754649072886, 'state_converting': 13.85691294586286, <Type.ALL: 'all'>: 13.912817124743015}) writing checkpoint to .cache/root/anthropic_dpo_pythia69_2024-09-22_09-38-57_157738/LATEST/optimizer.pt... writing checkpoint to .cache/root/anthropic_dpo_pythia69_2024-09-22_09-38-57_157738/LATEST/scheduler.pt...
The output ： the losses are just NAN.
what is wrong with this ?


⚠ ClamAV has degraded performance

In d1aeb19, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 85 ms

Resolved: ClamAV performance has improved in 10bdc57.


Can't import JS script as JS --> functions lost

When choosing "Open" from the main bar:

neither

the current JS/Blocks mode
nor the mode the script was programmed in

appears to be respected. As a result, when re-loading javascript programs, such as the one attached, only the blocks representation gets shown. This removes JS code.
I haven't verified that the whole JS code is actually being saved into the hex, but the result is this:
Write JS using functions --> Download onto micro:bit --> re-upload into PXT --> Can't re-create original code.
The example attached is generated from the following code:
basic.showNumber(0)
show_other()

function show_other() {
    basic.showNumber(1)
}

After re-importing and switching back to JS I see this:
basic.showNumber(0)

microbit-JSBlocksReproducer.hex.zip
I think that when opening a script

If the script was 'compiled' from blocks, then it should be opened in blocks
If the script was 'compiled' from js (IE the user was in JS mode when 'download' was pressed) then it should be loaded as JS.

If the following actions result in the current mode of the editor changing, the user should be prompted with something non-modal, for example:

@microsoftsam started working on this yesterday.

Just a question on the fix, looking at #481 it doesn't appear to give any feedback about editor mode switching - shall I file that somewhere else as a feature request?

Yes, please split the issue.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Jonathan Austin<EMAIL_ADDRESS>Sent: Thursday, October 13, 2016 4:11:01 AM
To: Microsoft/pxt
Cc: Peli de Halleux; State change
Subject: Re: [Microsoft/pxt] Can't import JS script as JS --> functions lost (#470)
Just a question on the fix, looking at #481https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F481&data=01|01|jhalleux%40microsoft.com|571c633561f14a17ee5b08d3f359a0dd|72f988bf86f141af91ab2d7cd011db47|1&sdata=US%2BSJCUtRIcgQneaOqhbHj23Rp%2B2bnkSCLjdkYFtvPc%3D&reserved=0 it doesn't appear to give any feedback about editor mode switching - shall I file that somewhere else as a feature request?

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fissues%2F470%23issuecomment-253484160&data=01|01|jhalleux%40microsoft.com|571c633561f14a17ee5b08d3f359a0dd|72f988bf86f141af91ab2d7cd011db47|1&sdata=oJVF2bkEiefu0U47U7Etj8xny2yhTwQi1q0aO8rkOOI%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KZF9LN2uDU1I684573ikBXV3YFgTks5qzhHFgaJpZM4KUhN5&data=01|01|jhalleux%40microsoft.com|571c633561f14a17ee5b08d3f359a0dd|72f988bf86f141af91ab2d7cd011db47|1&sdata=vSioJ8nm5kkMDFcsDRHe3ZdVWJhrEkcus2dqHs1%2BWgU%3D&reserved=0.


Update MacOs actions image due to broken pipeline

Read this before submitting the PR

Always create a Draft PR first
Go through sections 1-5 below, fill them in and check all the boxes
Make sure that the branch is updated; if there's an "Update branch" button at the bottom of the PR, rebase or update branch.
When all boxes are checked, information is filled in, and the branch is updated: mark as Ready For Review and tag reviewers (top right)
Once there is a submitted review, implement the suggestions (if reasonable, otherwise discuss) and request an new review.

If there is a field which you are unsure about, enter the edit mode of this description or go to the PR template; There are invisible comments providing descriptions which may be of help.
1. Description / Summary
Fix broken changes in actions due to updtae in architecutre for underliying image
2. Jira task / GitHub issue
3. Type of change
What type of change(s) does the PR contain?
Check the relevant boxes below. For an explanation of the different sections, enter edit mode of this PR description template.

[ ] New feature

[ ] Breaking: Why / How? Add info here.
[ ] Non-breaking

[ ] Bug fix
[ ] Security Alert fix

[ ] Package update

[ ] Major version update

[ ] Documentation <!-- Should be checked if the PR adds or updates the CLI documentation -- anything in docs/ directory. -->
[x] Workflow
[ ] Tests only

4. Additional information

[x] Sprintlog
[ ] Blocking PRs

[ ] Merged

[ ] PR to master branch: If checked, read the release instructions

[ ] I have followed steps 1-8.

5. Actions / Scans
Check the boxes when the specified checks have passed.
For information on what the different checks do and how to fix it if they're failing, enter edit mode of this description or go to the PR template.

[x] Black

[x] Pylint

[x] Prettier

[x] Yamllint

[x] Tests

[x] TestPyPI

[x] CodeQL

[x] Trivy

[x] Snyk

For reference: Change motivated by the issue explained in https://github.com/actions/runner-images/issues/9741#issuecomment-2075569220
Where the architecture for the default MacOs image is arm instead of Intel.

Aren't we adding a build for macos-latest, or will this one work on MacOS 14 too

As I understand, it won't work for AMD based Mac's. I.e, the new M1 and M2, for example, as they don't use intel chips anymore

Yes, but we should definitely have binaries for the arm arch MacOS. Maybe we should go for x86 now since we are running out of time.
What is this "The base branch requires all commits to be signed" business about?

Yes, but we should definitely have binaries for the arm arch MacOS. Maybe we should go for x86 now since we are running out of time. What is this "The base branch requires all commits to be signed" business about?

Yes, I was thinking we need some task to rethink this actions bussiness as they are giving more trouble than they should.
My fault, I don't know how but my GPG got deleted after updating the laptop. Now it is rebased


Pagination: Multiple instances with asynchronous data

I've got the pagination library to work with my site.  So far it's working great, thanks for the library.
But I have a question regarding multiple instances and asynchronous data on the same page.  For the asynchronous data to work, you define the callback for on-page-change.  This callback only passes the page number.  When I have multiple instances on the page, how am I suppose to know which one pagination is being used?
I could do it where I define a unique function for each pagination, but that's ugly, especially with a very  dynamic page.
The best way I can think of doing this is if the on-page-change callback passed the newPageNumber as well as the paginationId.  Then I can identify which control is being used.  Would that change be possible?  Is there a better way of doing it?

Good point. For now I think you might be able to get around that by using a $watch on the value of each pagination's "current-page", and triggering a function when it changes.
Not tested and perhaps there are issues with that, but try it out and let me know.


Bug when drawing text on a gradient

I think I found a bug that occurs when drawing text on a gradient background, and that occurs on Windows but not on OS X.
"hello" >label
[ t >>bold? ] change-font
{ 10 10 } <border>
{ COLOR: white COLOR: grey85 } <gradient> >>interior gadget.

On Windows, the text appears on a black background, which is itself inside the gradient. On OS X, the text appears normally inside the gradient.
The problem doesn't occur when the gradient is replaced by a solid background.
On both systems I used recent 64-bit development versions of Factor.

Also, the text appears when it is selected.
Jon
On Mon, Jul 20, 2015 at 6:15 PM, nicolas-p<EMAIL_ADDRESS>wrote:

I think I found a bug that occurs when drawing text on a gradient
background, and that occurs on Windows but not on OS X.
"hello" >label
[ t >>bold? ] change-font
{ 10 10 }
{ COLOR: white COLOR: grey85 }  >>interior gadget.
On Windows, the text appears on a black background, which is itself inside
the gradient. On OS X, the text appears normally inside the gradient.
The problem doesn't occur when the gradient is replaced by a solid
background.
On both systems I used recent 64-bit development versions of Factor.
—
Reply to this email directly or view it on GitHub
https://github.com/slavapestov/factor/issues/1397.

This may be related:
https://github.com/slavapestov/factor/issues/152
I think the slides use gradients?
On Mon, Jul 20, 2015 at 9:28 AM, Jon Harper<EMAIL_ADDRESS>wrote:

Also, the text appears when it is selected.
Jon
On Mon, Jul 20, 2015 at 6:15 PM, nicolas-p<EMAIL_ADDRESS>wrote:

I think I found a bug that occurs when drawing text on a gradient
background, and that occurs on Windows but not on OS X.
"hello" >label
[ t >>bold? ] change-font
{ 10 10 }
{ COLOR: white COLOR: grey85 }  >>interior gadget.
On Windows, the text appears on a black background, which is itself
inside
the gradient. On OS X, the text appears normally inside the gradient.
The problem doesn't occur when the gradient is replaced by a solid
background.
On both systems I used recent 64-bit development versions of Factor.
—
Reply to this email directly or view it on GitHub
https://github.com/slavapestov/factor/issues/1397.

—
Reply to this email directly or view it on GitHub
https://github.com/slavapestov/factor/issues/1397#issuecomment-122940481
.

Good find! Lets fix this for next release.

When this is fixed, need to remove the workarounds in https://github.com/slavapestov/factor/blob/master/basis/ui/gadgets/labeled/labeled.factor#L16 and https://github.com/slavapestov/factor/blob/master/extra/slides/slides.factor#L57.

Let's close this since it is the same bug as #152.


'numpy.flatiter' object has no attribute 'tolist'

Hi. Im running the programm winovf2vtk.py and it almost work, but at the last line it post this error: "AttributeError: 'numpy.flatiter' object has no attribute 'tolist'" instead of show the message "saving file (namefile.vtk)". The complete message is in the link: https://user-images.githubusercontent.com/64573135/80873136-1424a280-8c7c-11ea-8d38-4afb6faf32f1.png
Please helpe me with this errror,  I think that the problem is that in the program winovf2vtk.py, there is no obs variable defined, but I'm not sure.

I don't know, but I recommend you use the more recent discretisedfield packages for the conversion functionality: https://discretisedfield.readthedocs.io/en/latest/ipynb/12-tutorial-ovf2vtk.html


Problem [CommendBOT]

Description
My problem, is commend bot dont work. when i commend me... i open the game and i dont recive any commend, i turn off steam guard.. i wait 24 hours per commend.. and dont work
Reproduction Steps

before and after login (in game) : http://i.imgur.com/ibSfiXL.png
commend me : http://i.imgur.com/t8Guc08.png
log : http://i.imgur.com/3BBcCd9.png
"sucessfully" : http://i.imgur.com/c1KVvSX.png

System Information
Operating System: < Windows 10>
Titan version: <1.5.0 >

Are you able to provide log files (from the logs folder) and debug network packets (from the debug folder, please don't send these network packets publicly - send them privately to me over Discord or Steam)?

Hi, I changed the arguments passed to the CS:GO Game Coordinator in commit 2cc5f39. Could you try out with the latest build built from source?

Sent you in discord and sent invite of friends :D

any solutions?

This issue has been resolved via Discord. Closing.


PSN Offline

I entered the code to fix the problem that did not allow me to connect to the PSN but now it tells me that it is offline... what can I do?

Same problem here. All other platforms loged in. PSN saying that it's offline.

Closing as unrelated to this project. See https://github.com/gogcom/galaxy-integrations-python-api/issues/new/choose to find proper place to report Galaxy or plugin issues.


Handling creating multiple spaces within same component/page (React)

I am currently using the spaces SDK and I'm running into some issues with using the SpaceProvider for a specific feature for my teams application. Within in a page (or dialog in our case) we are interacting with a 3d viewing tool to view design files. For each file we would like to have a collaborative "space" using your product. Within the page we have an ability to switch between design files/documents that you can view using the 3d viewing tool. Ideally each design file/document has a space instead of one large one which could create confusion with loads of members potentially viewing several different documents. Using the SpaceProvider opens up for the use of useSpace() which is extremely handy which also seems to open up the useMembers() hook (these are very helpful hooks). However these seem to only be accessible when wrapping components with the SpaceProvider. I am encountering a few issues with the SpaceProvider.

SpaceProvider is manually creating a space as soon as I open up the dialog, which isn't the greatest considering it is a choice for a user to enter a space and work collaboratively or simply just individually use the viewer.

Secondly with the idea of each document having its own space every time the user chooses to switch within the viewer I'd expect the SpaceProvider which has its name field updating based on the document to update correctly (this is not the case, it is staying as the first space created).

user opens the dialog
const {space} = useSpace();
Space {any: Array(0), events: {…}, anyOnce: Array(0), eventsOnce: {…}, presenceUpdate: ƒ, …}

const {self} = useMembers()
null
^Showcasing the user is not in the space due to the fact that they have not decided to enter it, yet the space was created anyways

useEffect to update name
useEffect(() => {
        if (selectedDocument) {
            setName(`cad-spin-${selectedDocument.id}`);
        } else {
            setName("");
        }
}, [selectedDocument]);

Space provider to wrap the dialog/page
<SpacesProvider client={spaces}>
            <SpaceProvider name={name} key={name}>
                {children}
            </SpaceProvider>
</SpacesProvider>

What instead is required to happen is for the dialog to re-render (im using the key to do this), which in turn causes this flickering when switching between documents (not ideal). This is also now directly coupled with the space being created regardless of the users decision to actually interact with spaces, therefore causing a flicker when just individually using the viewer (not inside a space).
Our ideal and dream solution would be to enter the viewer, users chooses to use spaces, they create or get the space for that document, they then enter it and see all members within that space. User then chooses to switch documents, same sequence applies. Eliminate the re-rendering, allowing for a smooth and UX appropriate flow for the user (ideally allowing us to use you hooks you provide).
How exactly can I get around this? I've noticed it isn't necessary to use the SpaceProvider and gain access to your hooks, but those are very useful, leading to a potential recreation of those hooks to suit my specific use case.
A loom to showcase the problem in more detail.
https://www.loom.com/share/bf4dbf051b5447bdbf8e9a507e620174?sid=ae4f7ba5-b063-4105-8beb-da2eb122f873

Thanks for sharing this feedback @Keegs12 - very detailed context, much appreciated. We are discussing this internally and will get back to you soon.


Voxels... support of CubeMap mainVoxel function and inputs

Voxels still are the pain
Doesn't fully understand how this is made, but.
See Mandelbrot & Logistic Map.

Oh boi, rendering to a cubemap sounds like a daunting task. Not sure though, I will try to investigate. I'm also renaming the issue to be a bit more concise.


lottie crashes on android

https://snack.expo.io/@akshatraaj/book-santa-

thanks for removing lottie


added maze game

corresponds to issue-> #57
@urvashi-code1255

@Sloth-Panda Use PR template for description and always send the video/screenshots or you can setup this repository on your github pages and send the link for the same !!

checkout the images below,

@urvashi-code1255 why I am not able to find the code files for this after updating my origin with upstream ?

@Sloth-Panda did you commit your changes before updating your origin ?


Project Status?

Howdy :wave:!
I'm also interested in using Rust to author watch faces for Watchy, but I have been unable to find much material or prior art on the web.
What's the status of this project? Its got the looks of a crate that is intended to one day be published on crates.io, but maybe isn't stable yet... Your README does provide a lot of good context here :)
I should also say, I'm not trying to make work for you or make this a chore (I know the struggles that come with being an OSS maintainer), but I can't help but notice this seems like one of the only matureish implementations of anything that has to do with Rust and Watchy. Is that also your understanding or have you found something else worth looking into?
In any case, thank you for giving me something I can at least start playing with :heart:.

Hi! Months ago, I was working on a particular project with Watchy using Rust, which is what motivated me to start this crate.
The crate largely just collects the right dependencies for the various devices on the Watchy board (see the README) and either just re-exports them or wraps them in a nice object that passes in the correct pins and whatnot specific to the Watchy. I definitely intended to publish this on crates.io, but the only reason I haven't is because there are two dependencies (for the display and the accelerometer) that I needed to add features to, but those features had not yet been merged back into the official versions published on crates.io. This can be seen by looking at the dependencies in the Cargo.toml file. crates.io will not allow you to publish crates that have dependencies that are not also already published on crates.io, and in my case they are direct GitHub/GitLab links to my forked projects.
I have mostly abandoned the project I was working on. There was something else I wanted to do with it but then I ran into an issue with one of the dependencies working with the newer version of ESP and general HAL crates, and things would no longer compile.
My free time is pretty limited, but it would be fun for me to revisit this to see if this issue has been resolved. Things seem to have been changing fast in the embedded Rust world, so that is highly possible. I am generally happy to work with you to get you up and running and to resurrect this project, so I will take a look at it again when I get a chance in the next few days.
In the meantime, if you haven't yet already, I recommend you get your environment setup and get some basic Rust code compiling and running on your Watchy, which would not depend on this crate. The ESP on the board has great official Rust support, so here is a good place to start, and I will note that you want to go with the std version of everything. Obviously I have no idea what your experience level is, but I would also make sure that you're familiar with the embedded Rust ecosystem, which is very cleverly designed but can be complex.

Thanks you for the detailed response. Don't stress getting the project back up and running for me.

Obviously I have no idea what your experience level is, but I would also make sure that you're familiar with the embedded Rust ecosystem, which is very cleverly designed but can be complex.

This is a good suggestion. I'll go ahead and give the Embedded Rust book a read and maybe the Rust on ESP book after that.
A thin convenience wrapper for Watchy's specific pins, hardware devices, and intricacies actually sounds like a great way to get into Embedded Rust in general. If the display and accelerometer are working by the time I make I've freshened up my understanding of the ecosystem, I'll give this a whirl. And if not, I'll see if I could resolve that.
Really appreciate the status update! Don't feel any pressure from me to revisit this. But if you're called to it, know that you've got a user/contributor awaiting it ;)

Sounds good. I did have a look at this again the other night. Revisiting it made me remember that I was in the middle of some changes and new features with several different crates (including this one) when the problem I mentioned cropped up and stopped me dead in my tracks. That issue has now been fixed, at least partially, so I can continue.
It looks like the latest commit of watchy does not compile without some custom intervention, nor does the previous release tag v0.1.0 as it relied on some git repositories that have since changed. If you can find a previous commit that works you're welcome to go ahead and start using that. Otherwise give me some time (again, limited free time) to finish up what I was in the middle of now that the road block has been lifted. Afterwards, I'll add a new commit and summarize the issues and whatnot in the README.

It looks like the latest commit of watchy does not compile without some custom intervention, nor does the previous release tag v0.1.0 as it relied on some git repositories that have since changed.

Nice!

It looks like the latest commit of watchy does not compile without some custom intervention, nor does the previous release tag v0.1.0 as it relied on some git repositories that have since changed.

Hopefully just a few git bisects away from a workable path forward. I'll march forward with some general learning about the embedded rust ecosystem and then by the time I return here, if the issues are resolved I'll give it a whirl. If not, I can also hack on it a bit.

I've been working on this, and have made a new commit that fixes the issues. I added some notes to the README about the reasons why I cannot yet release on crates.io. In any case, you should be able to setup a new ESP-IDF project, and add watchy as a dependency, directed at this repository. watchy pulls in the other driver crates and re-exports them, so you shouldn't really need any other direct dependencies (which is the entire point of a board support crate!).
The only other thing I'll add is that I did implement axes remapping for the accelerometer driver that is not yet released (I have a pull request in for that). In the unlikely even that you need this feature immediately you can patch in my fork here. Otherwise, watchy will re-export the latest version on crates.io.
Let me know whether you get things up and running with watchy or if you run into any issues, which I'd be happy to advise on if needed. I'll try to monitor the issues preventing a crates.io release so we can get it released there once those are resolved. Once you're satisfied, we can close this issue.

Wow, this is awesome news. Really appreciate you dusting this off, getting it to compile, and adding some notes to the README. I'll go ahead and say "I'm satisfied" :smile_cat:.
I'll open any additional hurdles or issues as their own separate issues if I encounter any.


Config-flow kon niet geladen worden: Unknown error

When using HACS to install the component, I get this error. It takes +10 minutes when loading and configuring the component and then I get this error message.
In the logfile I see this:
2023-08-31 15:04:32.318 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration nhc2 which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant

The error in the logfile is a generic message from Home Assistant.
Can you please enable debug for the integration, see the first step in https://github.com/joleys/niko-home-control-II#i-need-to-see-the-device-list
Can you check if you are using the most recent version of Home Assistant.
Were you able to configure a controller? If so: did you use the auto detected controller or did you enter the host/ip manually?

Hi Tijs. I use the latest (and totally fresh install) of homeassistant on a RP4.
I enabled the debug and now the install has been busy for 50 minutes: "Een ogenblik geduld terwijl nhc2 wordt ingesteld".
I was not able to configure a controller. I don't see any configuration steps. Just blocked on this loop.

I is not easy for me to debug.
Could you try the following:

Open custom_components/nhc2/config_flow.py
Comment / remove lines 74 → 83, this are the lines below:

       disc = CoCoDiscoverProfiles()

        self._all_cocos = await disc.get_all_profiles()
        for coco in self._all_cocos:
            if coco[2] is not None:
                coco[2].insert(0, {
                    'Uuid': 'hobby',
                    'Name': 'hobby',
                    'Type': 'hobby'
                })

Restart Home Assistant
Try to configure the integration again.

Impressive. In an instance I see the option to set my IP of the controller. So it works! Thank you for the help! Much appreciated.

@tomazzive glad it works. This is some code related to discovering profiles and controllers on your network.
@joleys If I'm not mistaken this was added by you? Any idea to improve the error handling? Or should we just remove this part and let people fill in the details themself?

Thanks for the quick help! In the meanwhile I managed to add 8 controllers for a total of 520 entities. I'm using it in one of our hotels. When a customer checks out at the reception desk, all the lights of that room will be switched off automatically. Awesome.

@tijsverkoyen This was already in the original code.  I can try to find some time to dig into this part of the code.


Implement update-schema rpc

Schema author should be able to update a schema. When a schema is updated a new revision with new key will be added.

Revisions have been implement in create-schema rpc.


Improve readme doc

I spent at least 3 hours trying to understand why when trying to export from LOCAL to an AWS shared ES instance I was receiving:
Error Emitted => connect ETIMEDOUT

Basically in the end I found that if you try to export using default elasticsearch-dump infinite sockets, Amazon will block your ip immediately for a specific period of time (30 mins for my case).
The importing was starting like:
got 10 objects from source elasticsearch

but no response received so it ended with timeout.
In the end it worked if I specify:
--maxSockets=1

I think we should improve readme file because there is no clear reason for others if it happens the same.

I think that's more of a specific setting based your firewall or application configuration more than a default AWS setting.  It seems that you are only allowing, perhaps, 1 request per IP?

Yeah I think so...Basically searchly.com created my shared instance on request so I don't have too much info (like the aws admin) that's why I had so much struggle...


chore(localize): updated translation from Crowdin

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


Files and folders are not hidden until I recharge the plugin manually

Affected Platforms
iOS, Windows
Expected Behaviour
Selected file are hidden
Actual Behaviour
File are not hidden until I recharge manually the plugin or re-hide the files/folder.
Have you?

[X] I have checked that there isn't already a bug on GitHub for this issue

Urgency
Mildly Urgent

I'm not sure I understand what you mean by "recharge the plugin", is this the toggle switch in the "Community Plugins" tab of the Obsidian settings?

Hello !
I use the 1.1.0 version, and you're right : i need to manually unload / load the plugin through obsidian community plugin.

Okay, thanks for the info; I'll take a look and see what I can do about it.

Hey, I'm experiencing the same issue. A few more details that might help narrow down the issue:

The issue occurs the next day from what I can see (no need to restart Obsidian)
Toggling the hide option is enough to fix the issue for the day (no need to unload/load the plugin)

The files I'm hiding are on OneDrive

Thanks for making this plugin. It keeps my workplace clean and enjoyable.
Best,

Yeah, I continue to have this problem... I prefer to use CSS, it takes one line in my snippet css to hide. At last it keeps to work.

Yeah, the way the plugin works is odd, I have been attempting to make it more reliable and am aware of the issues that still happen (at least those that have been mentioned in GH issues), but life's been busy and whatnot so the times I get to work on it aren't super frequent.
(Though I have not forgotten / abandoned this plugin, so it will be updated to fix these issues, I just can't give a solid timeline for said fixes)

Just adding that I see this issue also (a file in the root folder is not hidden until the plugin's file visibility status is manually toggled on every restart). Perhaps you could update the readme so it's clear to users that the automated features of the plugin are unavailable? That is the main reason to use it.

@callumhackett what version of the plugin are you using? If it's 1.1.0, try updating to 1.1.1 and if the issue still happens let me know.

@callumhackett what version of the plugin are you using? If it's 1.1.0, try updating to 1.1.1 and if the issue still happens let me know.

This is version 1.1.1. I did some extra testing and it appears that the issue is persistent on Android but not on Windows. The plugin works manually on Android by toggling the file visibility status but files will appear unhidden on app restart. On Windows, the behaviour is as expected.


chore(ssa): Fix spelling regression

Related issue(s)
Description
Since adding the spellchecker, some spelling regressions were added. This PR fixes those.
Summary of changes

Dependency additions / changes

Test additions / changes

Checklist

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[ ] I have linked this PR to the issue(s) that it resolves.
[ ] I have reviewed the changes on GitHub, line by line.
[ ] I have ensured all changes are covered in the description.
[ ] This PR requires documentation updates when merged.

Additional context

closing due to #784


make LICENSE and README?

just to give people permissions and context, respectively.

I linked a PR.


Some Rive animations have a huge peak memory usage, leading to hardware freeze or crash

Mirror issue for https://buganizer.corp.google.com/issues/150220596.
This is with respect to some very complex Rive animations. We've done some initial communicating with the Rive team before --- see "https://github.com/2d-inc/Flare-Flutter/issues/99 and  https://github.com/2d-inc/Flare-Flutter/issues/249.
That said, until a general diagnosis is found, conversation should occur on the upstream issue.
cc @zanderso @dnfield

Customer provided reproduction appears to be working fine at head. I've updated the internal bug requesting either an updated reproduction or more information about what is wrong. I'll leave this assigned for now, but lowering the priority until customer can provide more detalis.

Upstream bug now has a reproducible case for Android. Upgrading priority.

Per Ray, Dan is conducting one final investigation but there isn't expected to be too much work on this front. Setting an appropriate milestone.

Dan has one item remaining in his investigation, pending some work done by the upstream team. Leaving open, but no need to discuss at Tuesday's triage.

@ravenblackx actually did something similar to what I was going to try (rewrite the animation as something just in Dart).
Locally, it looks like the RSS usage is slightly lower, but still has a sizable jump, particularly when the animation is played.  For example, I observe about a 20MB spike after loading the animation initially, and a 30-40mb spike after playing it.
AFAICT, this is all or mainly GPU driver related. We really don't have control over this, and it's not something the Flutter team can further investigate - closing as WAI.


General documentation about the yaml file

I interleave comments explaining the contents of sections in a general way befroe documenting each of the fields, example:
###
# This is a config file.
#
# The idea is to use it to configure stuff.

###
# this is a field
my-field:
   ###
   # This is a key
    key: 1

###
# The next fields describe things related to XXX.
# They should be used for YYY.

###
# this is another field
my-other-field:
   ###
   # This is another key
    another-key: 2

All the comments that are not next to a field are not rendered by autoyaml. Is there a way to render these as well?
It is better for documenting the file to put those explanations in place, and not to repeat them in the documentation of the package.

It's not possible to render such fields. Internally YAML documents are converted into definition lists based on comments. Definition lists in Sphinx require some key apart from content.


image inside pdf

I have a pdf which is an image inside the pdf. It doesn't log me an error. just write as null

Can I get an example PDF?

I have the same issue. I have attached a sample image pdf.
sample.pdf

Yes. Support for this would be soooo nice. Anyway, amazing job @dbashford! Looking forward for updates :D

Did you try this?

textract image.png --tesseract.lang=deu

Can you make default pdf method tesseract?


Routing to a query returns to the default route

Demo: http://jsfiddle.net/qc73v9va/49/
If you click Profile, then click the Click me for query link, you'll be redirected to the homepage.
My use case is that I want to be on any route /x/y and then easily attach a query param to it / update it.
Maybe I'm doing this wrong, but I expect route to do that. I can obviously grab the location, but just wondering if this is expected behavior.

Definitely worth adding yup. Technically this is a dupe of #129 - if we supported relative paths in route() and links, route('?foo') would be relative to the current route instead of being treated as /?foo.

There's a PR open for the behavior if you're up for reviewing! #138


MINOR: Fix directory name inconsistency in the Kafka Streams tutorial

In the Kafka Streams tutorial, "streams.examples" is specified
for mvn archetype:generate's artifactId parameter.
So the project directory is created as the same name, but it's
changed to "streams-quickstart" in the following explanations.

@sekikn -- Thanks for the PR -- could you rebase it so we can merge it?

Closing this stale PR in favor of https://github.com/apache/kafka/pull/14541