WIP: Allow condition-specific baselines

Users can now set different baselines for different conditions by setting config.baseline to a dict with condition names as keys and (tmin, tmax) tuples as values.
Also improves documentation of config.conditions and moves this setting to the very top of the EPOCHING section.

So I'm hitting a limitation in MNE here:
I cannot concatenate Epochs that have differing baseline periods; and there's no equivalent of write_evokeds() for Epochs in MNE either. So I won't be able to store all Epochs in a single file if different baselines are used…

crap it's a hard one....

In theory we could do this in the Epochs object. The bmin / bmax are currently float but everything in FIF is really an array anyway, so no reason we can't make it "something that broadcasts to len(events)". It's backward compatible, and we could make epochs.baseline allow arrays. I actually don't think it would be too much work. I'm pretty sure there is one place where we cull the epochs.selection/epochs.data etc., this would just be one more thing to shrink in the non-singleton case.

basically we have 2 options:

change mne
change the pipeline

for mne I agree the best solution is to allow baseline to be array but
carefully think of the consequence
downstream. Do we expect to be able to average epochs with different
baselines etc.
the other route is to update the pipeline.
thinking about it we could make the conditions be dict and baseline a dict
with the same keys
conditions = dict(dots_onset=['dots'], coherence_level=['coherent',
'incoherent'])
baseline = dict(dots_onset=(-0.2, 0), coherence_level=(-0.7, -0.5))
and we would produce 2 epochs files one for dots_onset and one
for coherence_level
wdyt?

what's the use case for condition-specific baselines? Why do you want to do that instead of say taking the minimum of all the possible baseline periods?

@hoechenberger: Same question as @jasmainak – why does the pipeline require different baseline intervals?
I have done this for my own data by extracting the data into an array, doing the baseline correction manually, and re-creating an epochs array (I have variable cue-target intervals and wanted to use a pre-cue baseline, while looking at activity time-locked to the target).

for exactly this usecase of precue baseline that we have in the localizer
data. I would like in the
output of the pipeline the ERFs for motion onset as well as the onset
coherence. Are we
making things more complicated than necessary? the study pipeline should
smoothly handle this.
The manual step with EpochsArray should be avoided if possible

do you mean a different baseline per epoch or a different baseline per condition?
why specify this in the config? BIDS allows you to specify baseline with the "onset" and "duration" columns. If onset is negative, you know there is a baseline, I think?

actually never mind, I think I misinterpreted onset.

there is also a column called "response_time" though. Not sure if it addresses the use case

@agramfort, @hoechenberger, I agree with @jasmainak, we are talking about 2 different cases here:

different baseline per epoch: epochs with variable cue-target intervals (or event-response intervals), where we want to time-lock to the second event, while taking the baseline before the first (my case)

different baseline + different time-locking event per condition: epochs with fixed cue-target intervals, where we want to either time-lock to the first event, or the second event, and respectively take the baseline before the event we are looking at (localizer data)

For case 1, something like fieldtrip's redefinetrial, i.e. shift trigger time with variable intervals over trials, could work well, and the baseline would not have to be re-applied.
For case 2, I would rather create parallel, independent versions of the data. The only reason I could see for concatenating these different types of epochs is for the convenience of processing them in one step, but probably other operations applied would be different, too (like cropping).

ok so what do you suggest to handle smoothly the localizer data in study
template?
ie we want to have both the cue onset and the coherent vs incoherent ERFs

@agramfort, @hoechenberger I don't actually think that we need the two sets of epochs for the localizer data. In the original study, they epoched around the (in)coherence onset, but with a baseline of -.6– -.5s, i.e. before the stimulus onset.
The contrast will then be between the two conditions: coherent vs. incoherent.
Possibly a misunderstanding?

then let's keep it simple


Fix false positive in grpc_is_epollexclusive_available on ESXi

Hey, this the change @tbuyukliev proposed in the form of a pull request.

@yashykt

This is for issue #22182

Hi, please run
tools/distrib/clang_format_code.sh to format the code

There are still failures, I'm going to rerun the tests

There are some build failures that cannot be from this change, along with some that don't have an obvious connection to the change. Is there some way to deal with those? Would rebasing help?
Example:
https://source.cloud.google.com/results/invocations/fbb99cbe-26fb-4e27-bbe5-4a19fec3ee94/targets/github%2Fgrpc%2Finterop_docker_build/tests
package github.com/envoyproxy/go-control-plane/envoy/api/v2/core: cannot find package "github.com/envoyproxy/go-control-plane/envoy/api/v2/core" in any of:

Other pull requests seem to be getting all green. It is most likely an infrastructure issue. Rerunning once more

All checks have passed, is there anything else needed?


Hang when authorizing

Hello,
the preference pane hangs for me when trying to authenticate. I can enter my admin password, but after that, System Preferences freezes and eats 100% of my CPU.
OS: 10.8.3
MacBook Pro Retina 13"
invliD.

Please could you try the latest release, v1.2?
Download the latest release at: https://github.com/mckenfra/postgresql-mac-preferences/releases
Unless I hear otherwise from you, I'll assume this issue is fixed so I'll close it soon.

Please could you try the latest release, v1.4?
Download the latest release at: https://github.com/mckenfra/postgresql-mac-preferences/releases
Unless I hear otherwise from you, I'll assume this issue is fixed so I'll close it soon.

@mckenfra Unfortunately, I cannot test this anymore since my environment has changed.

OK thanks for the reply. I'll close the issue but feel free to reopen another issue if you see the problem again.


screenshot added! :sparkle: resolved issues and working fine

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy

@vineetjk please rename the screenshoot, put your github username.


[Request]: Show 'What's New?' screen after forced updates

Is your request related to a problem?
No
How is the feature request related to the problem?
No response
Describe your proposed solution
After forced updates, show update logs.
Ever since forced updates were implemented, the What's New screen will disappear before I even get to read a single bullet point as they are completed too quickly. How it is now defeats the whole point of having the What's New screen while updating (as you can't read the whole log in like 5 seconds lol).
If some people find the update logs annoying after each update, there could be a toggle which turns this feature on/off either in App Settings and/or in the update log page itself.
Describe alternatives you've considered
Open the Github release after each update (which requires more clicks)
Additional context
No response

What about a function to show changelog in settings instead?

I have implemented the button for it that will open the changelog file hosted in our ReleaseRepo. Collapse will open it either in-app or external browser depending on your setting.

Let us know if this implementation is enough or you want to see more advanced markdown implementation (like the actual changelog shown when an update is pushed). The only drawback with that implementation is that no video view.

Something like this ?

Implemented in 1.81.1 https://github.com/CollapseLauncher/Collapse/releases/tag/CL-v1.81.1-pre
If you still have problem, reopen this ticket or open another one.
Thanks for the support <3


Very long leading silence in synthesized speech in a2a-vc-vctk

Hi, I'm trying to use any-to-any voice conversion in this repo.
First, I trained successfully with vq_wav2vec upstream. Then tried with unseen speaker for both target and source. Both speakers are not in vctk, not in vcc2020.
Here's some result with hifigan.

It's waveform image opened in audacity. As you see there's long leading silence. Fortunately, voice at end is audible. It takes too long in taco2 because of leading silence.
Why does it happen and how can i resolve it?

Hi @seastar105 , can you show me the source waveform visualizations? Does the source waveform has a long leading silence? Also, what is the sampling rate of the source waveform? Is it in 24kHz?

@unilight
Yes, source wave has leading silence. I didn't notice that, it works properly with trimmed source wave. sorry for dumb question..
also it has been downsampled from lossy audio with 48kHz to 24kHz.


StartPowershellScript does not work with paths, that include spaces

It seems like the addin has problems with spaces inside the path:
Example:

StartPowershellScript("C:/Program Files/dev/Prepare.ps1");

// OR

var filepath = new FilePath("C:/Program Files/dev/Prepare.ps1");
StartPowershellScript(filepath.FullPath);

My bad. Of course we have to use StartPowershellFile….


JindoRuntime support hdfs backend and upgrade jindofs image version

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #641 (2579585) into master (0684903) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #641   +/-   ##
=======================================
  Coverage   17.61%   17.61%
=======================================
  Files          64       64
  Lines        2856     2856
=======================================
  Hits          503      503
  Misses       2303     2303
  Partials       50       50

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 0684903...2579585. Read the comment docs.


Is a mistake in the slide 9?

@inakleinbottle thanks for your open talk on Signature Methods for Time Series Data in PyData, London, 2022. I've bit confused in the slide 9 "I don't understand, show me an example". I've looked your explanation many times but I'm still not sure about the last line, which you call a "tensor product" of exp(I1) and exp(I2). Let me show my understanding how it was derived.

term 1 is the multiplication of the 1st elements in level 0, thats is 1 * 1 = 1
term 2 is the summation of the 1st elements in level 1, that is 1 + 3
term 3 is the summation of the 2nd elements in level 1, that is 2 + 4
term 4 is the summation of the 1st elements of level 2 and multiplication of two elements of level 1 (1st from 1st vector and 1st from 2nd vector), that is 0.5 + 1 * 3 + 4.5 = 0.5 + 3 + 4.5
term 5 is the summation of the 2nd elements of level 2 and multiplication of two elements of level 1 (1st from 1st vector and 2nd from 2nd vector), that is 1 + 1 * 4 + 6 = 1 + 4 + 6
term 6 is the summation of the 3rd elements of level 2 and multiplication of two elements of level 1 (2nd from 1st vector and 1st from 2nd vector), that is 1 + 2 * 3 + 6 = 1 + 6 + 6
term 7 is the summation of the 4th elements of level 2 and multiplication of two elements of level 1 (2nd from 1st vector and 2nd from 2nd vector), that is 2 + 2 * 4 + 8 = 2 + 8 + 8

Am I right? If so, there is a mistake in the slide.

I'm so glad you found my talk interesting. I shall try and explain here, but for reference there is some much better documentation that you can refer to. It is entirely possible that I made a mistake in my slides, so let's see.
If I write x_i for the ith element of x, so the free tensor is
(x_0, x_1, x_2, x_3, x_4, x_5, x_6, ...)

Then the 5th term in xy (the second element of the level 2 terms) is the sum
x_0y_5 + x_1y_2 +  x_5y_0

In the example on the slide, x_0 = 1, x_1 = 1, and x_5 = 1, and y_0=1, y_2 = 4,  y_5=6, so we can compute (xy)_5 as
(xy)_5 = 1*6 + 1*4 + 1*1 = 6 + 4 + 1 = 11

So in fact you are correct, the 0.5 should be 1. I apologise for the confusion - I did write these slides the morning of my talk.


StreamlitDuplicateElementKey: There are multiple elements with the same key='first_tree'.

Render more then one tree in a single page this error will occure.
Please extend it to pass a parameter to set the unique key.

this is already possible by using the "key" parameter.


a? i?

Seems to be useful attribute observable token, leaving v pure generic object/array observable, without mixins.
Input is also a bit redundant part here, really, like, an heuristic out of nothing.
Really, attribute/prop logic stands out from generic observables.
v([a(el, 'data'), a(el, 'loading')])(([data, loading]) => ...)
// instead of implicit
v(el, ['data', 'loading'])

And regarding input too - that's a bit separate logic, not attribute observable, also it has setter.
v([i(a), i(b)], ([a, b]) => a + b))

These can be kept simple observables, without full v logic, just a channeled sync piece.

Ok, input is done, so a logic can be left to props/attributes observables.


chore(release): prepare for release

Description
Long awaited release of packages.
To workaround the problem we had with publishing that we had when share_plus dry run failed due to not being able to find the share_plus_platform_interface I have changed version of platform interface to be not 3.2.1 which we are going to publish, but 3.2.0. With this change we are able to run melos publish successfully and I don't expect breaking changes caused by this changes.
As soon as I publish this release I will create a new release for share_plus plugin only which should publish successfully as platform interface 3.2.1 will be already available on pub.dev.
Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I titled the PR using Conventional Commits.
[x] I did not modify the CHANGELOG.md nor the pubspec.yaml files.
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze) does not report any problems on my PR.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[x] No, this is not a breaking change.

Due to #1628 we also need to update our workflows file to run melos bootstrap, which should take care of dependency overrides.

All failing tests seems to be about version resolution, which I plan to address after this release, so merging the PR.


[DO NOT MERGE] Ignore drafts when fetching content items

We currently have a mismatch between the Audit Tool and the live GOV.UK
site when a draft of a content item exists.
If a draft exists, the Audit Tool will import the draft, and display
that, rather than the latest published edition (which is shown on
GOV.UK).
This change adds a method to our Publishing API client to fetch the
latest published edition, or to return nil, so that the content we
display in the Audit Tool will match that on GOV.UK.

 Audit page represents Whitehall not GOV.UK

Superseded by https://github.com/alphagov/content-performance-manager/pull/437


Invoice API total amount calculated by multiplying unit cost and units is faulty

Wallet Version:3.0.6
Core version: 5.0.5
Invoice module:1.0.5
OS: mac on M1
While paying invoices found that there is difference between the amount invoiced and amount payed out. When i checked found the amount calculated by API is faulty and the same is payed out.
http://api.nexus.io:8080/invoices/get/invoice?address=831XWLZSAvYXvvJ7CYWqzYEUiQKFteGcmqdYaHDxsXv2Gyo3Dwx
In the above invoice the unit amount=35.9 and unit=20.18 which should amount=724.462, but the API calculated it as 719
This has effected all the invoices.

This above needs some changes, we want to extract integers and avoid doubles as much as possible. If you ever do take a double you need to convert to integer to do the math otherwise you will get rounding errors


Ignore dangling messages from previous runs

This commit fixes cases where a dangling message would cause a subsequent run to fail.
This would happen when a timeout occurs, but the messages still get received by the calling process. Then on the next call, SSHEx would not differentiate between the messages of the previous run and the messages of the current run.
This commit fixes it by making the messages unambiguous, making sure that they belong to the conn and channel being used on the current call.
It also removes the compatibility with the {:error, reason} message format, as it is not a message the :ssh module sends, and it severely hinders the ability to differentiate between messages from the current and previous runs.

Hi @costaraphael , I'm glad someone is actually reading this code. So first let me show you my (moral) support.
Please don't take this long response wrong. It's meant to be fully constructive!
Next, let me share my thoughts about your proposal:

Any 'dangling' messages that may be left of previous runs must be processed too. I suggest you always leave some case open for those. My intention was just to process any incoming message equally regarding of the origin to avoid leaks, as a rule of thumb. I may have not taken into account all edge cases, because also my intended use cases were limited. But for those edge cases you find I suggest to add test cases to reproduce them first. Then get their fix.
I'm particularly fond of the long-line indentation for the pattern matching for incoming message. I mean lines 240-248 of the old code. It proved highly readable for newcomers to elixir many times. You propose to change completely the indentation for that part too. I wouldn't accept that change (on my repo).
As mentioned in point 1 I suggest to add test cases that show the issues you are fixing. Instead you are removing some test cases. Remember test code is also meant to explain and document the intentions of the library code. It's not merely to add coverage. I would suggest not to remove those test cases, just as to add test for further issues your PR is meant to fix.
For the error generated by the streaming helper function, line 144 of the old test code, this was meant for testability of that feature. I did find the case when such message was received, so I added the case to the test code. I suggest to ensure this message is not possibly received anymore (maybe stdlib changed in latest releases). You may want to keep it for the older erlang releases.

As a final note I would add that now that you opened it, I may end up merging this PR, given requested corrections and documenting the rest, but I wwill not release it.
I would be more than happy to give you the ownership of this repo so that you can go on maintaining it. As I explain in the readme, I don't have the time now.
That way, next release of sshex would be made by you. What do you think?


release 1.59.3 fails pip install on raspbian bullseye

What version of gRPC and what language are you using?
1.59.3
python 3.9.2
What operating system (Linux, Windows,...) and version?
Linux - raspberry pi zro w - raspberry pi os - bullseye
What runtime / compiler are you using (e.g. python version or version of gcc)
python 3.9.2
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
i was in the process of installing mavsdk python module, this module depends on your grpc library
i executed the following command:
sudo pip3 install --upgrade --no-cache-dir mavsdk
5 days back i performed the same install and all worked well - at that time the grpc version was probably the previous one , 1.59.2, as i noticed that .3 was released just 2 days ago.
What did you expect to see?
i expected to see a clean installation
What did you see instead?
instead i saw pip3 failure - details below
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [10 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/tmp/pip-install-5jvqy26k/grpcio_90b923fe38084649aca68c47802549c2/setup.py", line 95, in
import commands
File "/tmp/pip-install-5jvqy26k/grpcio_90b923fe38084649aca68c47802549c2/src/python/grpcio/commands.py", line 31, in
import support
File "/tmp/pip-install-5jvqy26k/grpcio_90b923fe38084649aca68c47802549c2/src/python/grpcio/support.py", line 111, in
errors.CompileError: diagnose_compile_error,
AttributeError: module 'setuptools.errors' has no attribute 'CompileError'
[end of output]
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?
here is the full error:
sudo pip3 install --upgrade --no-cache-dir mavsdk
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting mavsdk
Downloading mavsdk-1.4.9-py3-none-linux_armv6l.whl.metadata (6.6 kB)
Collecting aiogrpc>=1.8 (from mavsdk)
Downloading https://www.piwheels.org/simple/aiogrpc/aiogrpc-1.8-py3-none-any.whl (10 kB)
Collecting grpcio>=1.50.0 (from mavsdk)
Downloading grpcio-1.59.3.tar.gz (24.9 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 24.9/24.9 MB 1.4 MB/s eta 0:00:00
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [10 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/tmp/pip-install-5jvqy26k/grpcio_90b923fe38084649aca68c47802549c2/setup.py", line 95, in
import commands
File "/tmp/pip-install-5jvqy26k/grpcio_90b923fe38084649aca68c47802549c2/src/python/grpcio/commands.py", line 31, in
import support
File "/tmp/pip-install-5jvqy26k/grpcio_90b923fe38084649aca68c47802549c2/src/python/grpcio/support.py", line 111, in
errors.CompileError: diagnose_compile_error,
AttributeError: module 'setuptools.errors' has no attribute 'CompileError'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Hi, I see this error: AttributeError: module 'setuptools.errors' has no attribute 'CompileError', this means you don't have a higher enough version of setuptools, can you try update it first and try install again?
Most likely install will still fail, but we'll have a better idea why.

looks like setuptool version 52.0.0 makes the issue go away - thanks


Update chart releaser to use environment variable

Description of changes:
Allow changing which CR to use via an environment variable.
cd into the proper directory for each command.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #166 (f59f16a) into main (f67d927) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #166   +/-   ##
=======================================
  Coverage   52.88%   52.88%
=======================================
  Files           7        7
  Lines         711      711
=======================================
  Hits          376      376
  Misses        326      326
  Partials        9        9

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Script results are not escaped for HTML in the web interface

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[x ] 3.3

When the UI is used to run a foxx service script which returns a string which contains html/xml tags, the results are displayed as JSON inside a <pre> tag, but they are not HTML-escaped.
Steps to reproduce:
(1) Create this script in a foxx service:
module.exports = `foo <b>bar</b>`;

(2) Go to the service settings, choose the script from the dropdown and run it.
Produced results:
The UI displays "foo bar"
Expected results:
The UI should display "foo <b>bar</b>"

Sorry for the late reply. Applied bugfix PR's for all supported versions.

Closing this as per above comment.


SQLite Transactions Being Left Open - Caused by "not an error" SQLiteException during rollback?

I'm working on a Xamarin.Forms multi-platform application that is currently using version 1.7.335 of this package, and I'm intermittently running into an issue where it seems like a SQLite transaction is getting left open until I either close the app and relaunch it, or until the transaction is explicitly rolled back with the Rollback() method. In the code, BeginTransaction() is not being called to explicitly begin transactions, and Commit() is not being called to explicitly save the data. Instead, whenever some database operations need to be handled in a transaction, the code calls SQLiteAsyncConnection.RunInTransactionAsync(). There are also places in the code where SQLiteAsyncConnection.InsertAllAsync() or SQLiteAsyncConnection.UpdateAllAsync() are called without specifying a value for the optional runInTransaction parameter, which would be starting a transaction as well.
The database is being opened with the following SOF flags: SOF.ReadWrite | SOF.Create | SOF.FullMutex | SOF.SharedCache
The following pragma statements are run when connecting to the database:
PRAGMA journal_mode = MEMORY;
PRAGMA synchronous = OFF;
PRAGMA cache_size = -4000;
PRAGMA temp_store = MEMORY;
There's one exception I saw being reported when I noticed this happening most recently that caught my eye, and it made me wonder if it might be relevant to the behavior that I'm noticing.
SQLite.SQLiteException: not an error
at
SQLiteCommand.ExecuteNonQuery ()
SQLiteConnection.Execute (System.String query, System.Object[] args)
SQLiteConnection.RollbackTo (System.String savepoint, System.Boolean noThrow)
SQLiteConnection.Rollback ()
SQLiteConnection.RunInTransaction (System.Action action)
SQLiteConnectionExtensions.InsertOrReplaceAll (SQLite.SQLiteConnection target, System.Collections.IEnumerable objects)
Unfortunately the "not an error" message doesn't let me know what went wrong, but is the fact that this happened when calling ExecuteNonQuery as part of a Rollback potentially to blame for this? Is there a way that I can potentially handle a case where a Rollback doesn't work successfully & leaves a transaction open (if something like that is possible)? Would it be safe to try a commit before attempting to rollback again, to ensure that data inserted while the transaction is active is not lost, or could that corrupt the database? Would upgrading to version 1.8.116 of the package to take advantage of the "Replace synchronous locks with async" changeset potentially help with this type of issue? At this point, I'm just looking for any potential leads that can help me figure out how to handle this very strange issue. In addition to the above exception, I'll also see exceptions like these being reported while this is happening.
SQLite.SQLiteException: cannot start a transaction within a transaction (this will happen when RunInTransactionAsync is called)
at
SQLiteCommand.ExecuteNonQuery ()
SQLiteConnection.Execute (System.String query, System.Object[] args)
SQLiteConnection.BeginTransaction ()
SQLiteAsyncConnection+<>c__DisplayClass80_0.b__0 (SQLite.SQLiteConnectionWithLock conn)
SQLiteAsyncConnection+<>c__DisplayClass34_0`1[T].b__0 ()
System.ArgumentException: savePoint is not valid, and should be the result of a call to SaveTransactionPoint. Parameter name: savePoint (this will happen when InsertAllAsync is called)
at
SQLiteConnection.DoSavePointExecute (System.String savepoint, System.String cmd)
SQLiteConnection.Release (System.String savepoint)
SQLiteConnection.RunInTransaction (System.Action action)
SQLiteConnection.InsertAll (System.Collections.IEnumerable objects, System.Boolean runInTransaction)
SQLiteAsyncConnection+<>c__DisplayClass77_0.b__0 (SQLite.SQLiteConnectionWithLock conn)
SQLiteAsyncConnection+<>c__DisplayClass33_0`1[T].b__0 ()
Thanks in advance for any help or additional information!

I've been trying to sort through as much as I can related to these messages again to determine a proper way of working around them, and I guess there's one thing that I'm just not understanding and am having a hard time figuring out how it could happen within the SQLite code.
I can see in SQLiteCommand.ExecuteNonQuery that the only Result Code that won't result in an Exception being thrown is the SQLite3.Result.Done code. That generally makes sense, as I would expect that receiving anything other than "Done" as a result of calling SQLite3.Step would indicate that something didn't run fully or correctly. But I just can't find information that would suggest why calling Step would simply return the SQLite3.Result.OK code (which is not actually an error code), which would then come back to my code as a SQLiteException with the "not an error" message. If Step returns this code, does that mean that SQLite did not actually attempt to run the command statement at all, and that I should treat this as a legitimate error? Or if I get this code, does it mean that the command was run successfully, and the code returned is just different from what's expected (and is throwing an exception when it shouldn't be doing that)? I think if I can understand why I'm getting an OK code instead of Done here, that'll be a big help in determining how I need to handle this message in my own codebase. Its just extra perplexing because this isn't something that happens very consistently, it just randomly happens from time to time (so its not something I can get a reliable reproduction test case or even a set of reproduction steps going for).
Is it possible that ExecuteNonQuery is running into a similar problem that was fixed in the SetKey method as part of this merge request? #1026


Add headerType, defaults to hyperdrive

Adds support for the headerType option to track the type in the hypercore header.
Fixes #294

@mafintosh is there any issue with merging this PR? I'm guessing @RangerMauve is using this for his multi-hyperdrive project (right?)

@pfrazee follow the discussion here https://github.com/hypercore-protocol/hypertrie/pull/43

Ah thanks

@mafintosh @andrewosh This should be ready to merge now that mountable-hypertrie and hypertrie have merged it.
Might need to bump the mountable-hypertrie version before release. 😁

LGTM from me. Will let @andrewosh get a chance to review as well

Yep LGTM

Published 10.18.0

Sweet, thank you!


Is there any way of playing an animation on the stage with a event?

What is your question?
Are there any ways in that i could play a animation, pause or play an animation on a stage?

Source or lua??????

.lua

objectPlayAnimation('tag' 'animation' false)


Update of UMLs following HLGF'20 discussion

See corresponding meeting minutes

PR #377 updates invocation and execution diagrams based on HLFG2020 meeting notes.


Email notifications for public and private cloud product PO/TL as to list of contacts changing

Describe the issue
product PO/TL don't get notifications regarding contact list changing (private cloud PO/TL get notification about contact list changing in a case of quota change, but not only contact list changing)
Rational/Why
PO?TL should be notified if contact list was changed
Definition of done/Acceptance Criteria

public cloud product TP/TL get notification in case list of product contacts was changed
private cloud product TP/TL get notification in case list of product contacts was changed

When I change one user to another I see the requested changes in the Current section and vice versa.
In the screenshot is the email that was received after changing platofrm.services account to artem.kravchenko account:
Private Zenhub Image


Remove unrealistic support for mutibrowser

At moment we pass through env Var the browser Name.  The Problem is that Firefox and others doesn work. In current conf weekend cannot use a non headless browser. So we Pass only Phantoms . Remove unneded if and vars.

https://github.com/SUSE/spacewalk-testsuite-base/pull/203

Fixed


Develop data set landing page high-fidelity designs (draft 1)

💡 Description

Tightly coupled with #129

to begin work on this

https://www.figma.com/proto/9y6Davo80c1cuKvPFfIcAu/Mockups?page-id=29%3A7092&node-id=29%3A7092&viewport=-11721%2C-19696%2C0.79&scaling=min-zoom&starting-point-node-id=1104%3A49509

To be presented to WMWG on 9/6: https://drive.google.com/drive/u/0/folders/1w2uACVm4Yz6waA8keNCsZvvH9V8oKKx3


Optional attributes in Dry::Data::Struct

Would it be possible to have optional attributes when using Dry::Data::Struct? When I try to instantiate an "incomplete" object, I get the strict hash SchemaKey error (well, the StructError since that's caught by the Struct class).
I might not be using this gem right, but some of my models are not "complete" sometimes and it doesn't matter (stuff is lazily loaded later when required.)
A more precise use-case: a MongoDB::Collection model has the attributes :ns (string) and :count (integer). However, sometimes we don't get the count right away due to the fact that it's an extra call to the database.
Would it be desirable to set nil on the attribute if that hash key is not present on instantiation? It would only work with the optional types, and so would be aligned with the basic premise of this gem.

An optional type means that the value can be nil, not the key in the hash can be skipped. I'm not sure how to solve this. I need to think about it a little bit and get back to you. Thanks for bringing this up.
BTW there's still some work to be done related to optional types, default-value types and hash schemas.

Ah yes, sorry, I didn't mean that I expected the behaviour.
My idea of how this could work would be: If a type is optional, when instantiating, if !attributes.has_key?(schema_key), just assume nil as the value instead of throwing.
I imagine that might be a new schema constructor. Maybe it could be configured in the Dry::Data::Struct inheriting Class.

For now I have something like this defined in my app:
module Dry
  module Data
    class Struct
      class Tolerant < Dry::Data::Struct
        def self.attributes(new_schema)
          prev_schema = schema || {}
          @schema = prev_schema.merge(new_schema)
          @constructor = Data['coercible.hash'].schema(schema)
          attr_reader(*(new_schema.keys - prev_schema.keys))
          self
        end
      end
    end
  end
end

with this line being of most interest:
@constructor = Data['coercible.hash'].schema(schema)

I'm thinking adding a Dry::Data::Struct.schema_type(type) method where type might be :strict or :tolerant and then using that type when creating the constructor might be a nice. Or maybe it could follow the way the rest of the library seems to do it like:
class User < Dry::Data::Struct.tolerant

Thanks for feedback. I'll get back to you on Monday, I'll be offline during the weekend.

@solnic There are cases where we want to omit part of the schema. On top of my mind:

Unit tests (see example below)
Instantiate the entity with data coming from the database table, but selecting only a few keys. Instead  of SELECT * we do SELECT name.

Example:
class User < Dry::Data::Struct
  attribute :name, Types::Maybe::Coercible::String
  attribute :x,    Types::Maybe::Coercible::String
  attribute :y,    Types::Maybe::Coercible::String
  attribute :z,    Types::Maybe::Coercible::String

  def foo
    # do something only with name
  end
end

# Unit test
it "does something great" do
  user = User.new({name: 'L'}) # It raises an error
  user.foo
end

If we have 10 attributes for that entity, it's tedious to have to specify all the keys just to instantiate it and unit test.
What do you think?

I'm OK with adding a different type of a struct that will use a hash schema which can fill in missing attributes and assign their default values. This is actually related to dry-validation too, where we can benefit from using types with default values.

I'm gonna experiment with a hash schema that will look if there's at least one default-value type, and if it exists, it's gonna use a special constructor that will set default values while constructing hash from the input. This will make structs with defaults working.
There's another thing I'm wondering about, which is using required and optional rather than just attribute to clearly specify what the expectation is. In case of optional we would fill in missing keys assuming they have types with default values. This would be much cleaner wrt intentions.

I would like to add a :+1: for this, but also indicate that I'm willing to help where help is necessary to get it up and running - my use case is in conjunction with https://github.com/apotonick/roar to parse incoming XML API calls to an Object.  Roar uses single-property assignment rather than hash assignment to make it happen, so I'm using Virtus.model for the time being :)

I'll add support for default values in hash schemas before next release. Hopefully next week.

OK I added support for default values in hash constructors and ability to set a different constructor type in structs, so ie this is possible now:
class User < Dry::Types::Struct
  constructor_type(:schema)

  attribute :name, Types::Strict::String
  attribute :admin, Types::Strict::Bool.default(true)
end

user = User.new(name: 'Jane')
user.admin # => true

However, I think this feature should not be used typically. Structs and Values are meant to be strict, you are constructing them from input that is supposed to be in valid state and have all the keys and values in place. For dealing with untrusted input it's better to use dry-validation's schemas.
Furthermore, if you need this just for testing, we can provide struct builders that can generate test data automatically.
I decided to add this feature just because I feel like it can help with adoption, personally I won't be using it.

@solnic Thanks for adding this feature. IMO It helps to improve flexibility.

Thanks for adding this ability! Agree with solnic's comment above, it is useful for adoption...


Unhandled exception during windows login

Microsoft PowerToys version
0.51.1
Running as admin

[X] Yes

Area(s) with issue?
General, Awake
Steps to reproduce
Only seen once on initial login after reboot
✔ Expected Behavior
Don't try to launch debugger for normal installations
❌ Actual Behavior
Failed to launch Just-In-Time debugger
Showed dialog with the following content:
[Window Title]
Visual Studio Just-In-Time Debugger

[Main Instruction]
An unhandled win32 exception occurred in [15044] PowerToys.Awake.exe.

The Just-In-Time debugger was launched without necessary security permissions. To debug this process, the Just-In-Time debugger must be run as an Administrator. Would you like to debug this process?

[V] View process details  [Yes, debug PowerToys.Awake.exe] [No, cancel debugging]

[Expanded Information]
Process Name: C:\Program Files\PowerToys\modules\Awake\PowerToys.Awake.exe
User Name: company\initials [Elevated]

Other Software
No response

It seems like you are using an outdated version of PowerToys. Please update to the latest version of PowerToys and see if the issue persists. 😄

/needinfo

@jhv2017kam can you confirm if the issue still persists after you update PowerToys?

Could not reproduce the exact behavior, instead PowerToys repeatedly asks for elevation rights during initial login after reboot which I am not able to give, so I need to press the no button 8 times

Are you still seeing the persistent elevation rights ask? /needinfo

Are you still seeing the persistent elevation rights ask? /needinfo

No it works as expected now


Documentation missing mandatory feilds

Hi
Thanks for spending the time to develop this wonderful library, I'm confused about which of the options are required and can use defaults.

sorry for the long delay. I haven't spent much time on Github in December.
You are right. It is not clear from the docs which options are required. It should have been auto-generated though.
Have a look at the typings (https://github.com/nknapp/node-child-service/blob/master/types/index.d.ts#L10) or the jsdoc-comments and it will become clear (https://github.com/nknapp/node-child-service/blob/master/src/index.js#L17)
I guess the jsdoc-comments are wrong. It should be Regex= instead of Regex?

sorry for the long delay. I haven't spent much time on Github in December.
You are right. It is not clear from the docs which options are required. It should have been auto-generated though.
Have a look at the typings (https://github.com/nknapp/node-child-service/blob/master/types/index.d.ts#L10) or the jsdoc-comments and it will become clear (https://github.com/nknapp/node-child-service/blob/master/src/index.js#L17)
I guess the jsdoc-comments are wrong. It should be Regex= instead of Regex?

I fixed this in 2.4.1, hope it's not too late for you...

I fixed this in 2.4.1, hope it's not too late for you...

No it’s not too late, would be great, I’m using it with an electron app, I have number of binaries I need to execute, I think your library is an excellent fit 😄

No it’s not too late, would be great, I’m using it with an electron app, I have number of binaries I need to execute, I think your library is an excellent fit 😄


Removed clone statement

The repo is cloned in the puppet script now.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

All is well.


created docs_ci_check

Description
This is a bit duplicate, but I think this would be a good check for the validate branch as well since we are pushing and working a lot with it currently
for reference please check this PR merged into develop

closing this PR because I think it is already within the develop branch
https://github.com/C-Accel-CRIPT/Python-SDK/tree/develop/.github/workflows


Documentation for node operators

mainnet disaster runbook

[ ] How to disable f3 participation if something goes wrong

Closing this ticket in favour of: #638 which includes subtasks as Operators Guides, Blog post and disaster runbooks


Use the Async Clipboard API when available

This also introduces a breaking change, i.e. returns a Promise instead of a boolean since the new Clipboard API is async.
More info: https://developers.google.com/web/updates/2018/03/clipboardapi

Excited for this new API, seems awesome 👏
Left a few small comments ☺

Published as 2.0.0


Do not default chain node (fixes #950)

Fixes #950
When starting lnd, we no longer assume the user is using btcd over bitcoind.
When you now run lnd without bitcoin or litecoin node specified, you get the following error...
lnd --rpclisten=localhost:10001 --listen=localhost:10011 --restlisten=localhost:8001
Bitcoin Node: loadConfig: only btcd, bitcoind, and neutrino mode supported for bitcoin at this time

Alternatively, we can create a separate error message (similar to the one above) that gives a little more detail that Node wasn't provided at all (vs not one of the ones supported)

LGTM 🐲

Closing due to inactivity.


Fixed a hasMany field's bug when added a new form.

Hi, I always use this library.
Problem

When I added a new form in hasMany field, The validation error is not displayed.
This happen on submit only the first time after added form.
Also, after I added some forms, submit once with a validation error and then submit again, it's reduced to only one form.

Cause of this problem
When I added a new form, the new form's "name" attribute's value is $relation[new_1][$column].
But, after on submit with validation error, the new form's "name" is $relation[new___LA_KEY__][$column].
If I add some forms and submit with validation error, all forms "name" will be $relation[new___LA_KEY__][$column].

The error is not displayed because the "name" attribute of the form has changed between when it was posted and when it was displayed.
Due to duplicate form "name", some forms are reduced to one.

Solution
I have changed the following classes.
Encore\Admin\Form\Field\HasMany

I changed HasMany::buildNestedForm method because NestedForm::key property should be have changed to the correct value before NestedForm::formatField method was called.
Before
protected function buildNestedForm($column, \Closure $builder, $model = null)
{
    $form = new Form\NestedForm($column, $model);

    if ($this->form instanceof WidgetForm) {
        $form->setWidgetForm($this->form);
    } else {
        $form->setForm($this->form);
    }

    call_user_func($builder, $form);

protected function buildRelatedForms()
{
    omit......

    if ($values = old($this->column)) {
        foreach ($values as $key => $data) {
            if ($data[NestedForm::REMOVE_FLAG_NAME] == 1) {
                continue;
            }

            $model = $relation->getRelated()->replicate()->forceFill($data);

            $forms[$key] = $this->buildNestedForm($this->column, $this->builder, $model)
                ->fill($data);
        }
    } else {

After
//Add the argument $elementName.
protected function buildNestedForm($column, \Closure $builder, $elementName = null, $model = null)
{
    $form = new Form\NestedForm($column, $model);

    if ($this->form instanceof WidgetForm) {
        $form->setWidgetForm($this->form);
    } else {
        $form->setForm($this->form);
    }

    //Add
    if (!is_null($elementName)) {
        $form->setKey($elementName);
    }

    call_user_func($builder, $form);

protected function buildRelatedForms()
{
    omit......

    if ($values = old($this->column)) {
        foreach ($values as $key => $data) {
            if ($data[NestedForm::REMOVE_FLAG_NAME] == 1) {
                continue;
            }

            $model = $relation->getRelated()->replicate()->forceFill($data);

            //Add $key to argument $elementName.
            $forms[$key] = $this->buildNestedForm($this->column, $this->builder, $key, $model)
                ->fill($data);
        }
    } else {

The Encore\Admin\Form\Field\Table class extends from the hasMany class, so I changed it too.
Thank you.

Review please :)


New Integrations Pages / Removal of Openai / Adding of Autogen OAI Requests

This is a self updating post, please check back for changes.

Added a page for each item on the "Integrations" page
Removed items from the "Integrations"
Changed "open a pr" to "open a issue"
Removed Openai from how-tos
Added Autogen OAI requests to how-tos

❌ Deploy Preview for localai failed.
Name	Link
🔨 Latest commit	14bdb7c
🔍 Latest deploy log	https://app.netlify.com/sites/localai/deploys/6528cdc056037200094b3ac8

Can you retry please?

@mudler ready for review and merge


default domain for original image url

Hi.
There is a default scheme setting: HTTP_LOADER_DEFAULT_SCHEME: "https"
http://localhost:8000/unsafe/https://example.com/name.jpg
->
http://localhost:8000/unsafe/example.com/name.jpg

Is there anything similar for domain name?  HTTP_LOADER_DEFAULT_DOMAIN: "example.com"
http://localhost:8000/unsafe/example.com/name.jpg
->
http://localhost:8000/unsafe/name.jpg

I am not sure about default domain as it would be too ambiguous. e.g. Docker/Kubernetes service hostname
http://minio/name.jpg

does not have a valid domain name but can be resolved properly. Populating a default domain
http://example.com/minio/name.jpg

would become invalid in this case

I am not sure about default domain as it would be too ambiguous. e.g. Docker/Kubernetes service hostname
http://minio/name.jpg

does not have a valid domain name but can be resolved properly. Populating a default domain
http://example.com/minio/name.jpg

would become invalid in this case

I would like to hide the source of the image.
Something like this
h.DefaultDomain = if h.DefaultDomain not empty then h.DefaultDomain + "/"
image = h.DefaultScheme + "://" + h.DefaultDomain + image
I think if a person sets up a domain, then he will expect that an automatic domain assignment will occur.
Maybe I don't see the whole picture. Maybe there is another option to hide the source?

Even if default domain is expected, there is also a chance the path has a valid domain e.g.
https://example.com/aaa.org/name.jpg

This will also conflict with the automatic domain assignment

I suggest a base URL option, and it should cast the base URL no matter base valid domain or not:
https://example.com/foo.org/bar/name.jpg <- source image

with base URL set to https://example.com/foo.org:
http://localhost:8000/unsafe/https://example.com/foo.org/bar/name.jpg
->
http://localhost:8000/unsafe/bar/name.jpg

I suggest a base URL option, and it should cast the base URL no matter base valid domain or not:
https://example.com/foo.org/bar/name.jpg <- source image

with base URL set to https://example.com/foo.org:
http://localhost:8000/unsafe/https://example.com/foo.org/bar/name.jpg
->
http://localhost:8000/unsafe/bar/name.jpg

Excellent =)

You may test the docker build at master branch:
https://github.com/cshum/imagor/pkgs/container/imagor/71301029?tag=master
ghcr.io/cshum/imagor@sha256:d9dfe9a06a0813cf655e10529a9e7ee8096e0c0a51c52b27d09e55d39c883bc9

available with fill(transparent) or fill(none)

HTTP_LOADER_BASE_URL=https://raw.githubusercontent.com/cshum/imagor/master

Thank you, good man. it works!


Bug: quote_sheetname Function Does Not Correctly Quote Sheet Names with Hyphens and Other Characters

Current behavior
When opening the workbook in Excel, the print titles are not set correctly for sheets with hyphens (e.g., Sheet-A, Sheet-B-C). The image attached demonstrates that the print titles are not showing up in the Page Setup dialog.
Analysis:
The function quote_sheetname currently quotes worksheet names only if they contain spaces, exclamation points, or single quotes. However, worksheet names with hyphens and other characters also need to be quoted to work correctly in Excel.
Current quote_sheetname function:
pub(crate) fn quote_sheetname(sheetname: &str) -> String {
    let mut sheetname = sheetname.to_string();

    // Ignore strings that are already quoted.
    if !sheetname.starts_with('\'') {
        // double quote and other single quotes.
        sheetname = sheetname.replace('\'', "''");

        // Single quote the worksheet name if it contains any of the characters
        // that Excel quotes when using the name in a formula.
        if sheetname.contains(' ') || sheetname.contains('!') || sheetname.contains('\'') {
            sheetname = format!("'{sheetname}'");
        }
    }

    sheetname
}

Expected behavior
Quoting Sheet Names:
The quote_sheetname function should quote sheet names that contain hyphens (-) and other special characters in addition to spaces, exclamation points, and single quotes.
Sheet names like "Sheet-A" and "Sheet-B-C" should be quoted correctly when needed.
Setting Print Titles:
The XML output should properly reflect the quoted sheet names in the  elements.
When opening the workbook in Excel, the print titles should be correctly recognized and applied, allowing the user to see and use them in the Page Layout -> Print Titles dialog.
Handling Special Characters:
The function should handle and quote a variety of special characters as mentioned in this StackOverflow discussion, including hyphens and brackets.
Sample code to reproduce
use rust_xlsxwriter::{Workbook, Worksheet, XlsxError};
use std::fs::File;
use zip::read::ZipArchive;
use std::io::Read;

fn main() -> Result<(), Box<dyn std::error::Error>> {
    // Create a new workbook
    let mut workbook = Workbook::new();

    // Add worksheets
    for _ in 0..3 {
        workbook.add_worksheet();
    }

    // Set worksheet names
    let sheet_names = ["Sheet-A", "Sheet-B-C", "Sheet C"];
    for (index, sheet_name) in sheet_names.iter().enumerate() {
        let worksheet = workbook.worksheets_mut().get_mut(index).ok_or("Worksheet not found")?;
        worksheet.set_name(sheet_name.to_string())?;
    }

    // Set print titles for specific sheets
    let print_titles = [
        ("Sheet-A", "$1:$2"),
        ("Sheet-B-C", "$1:$2"),
        ("Sheet C", "$1:$2"),
    ];

    for (sheet_name, titles) in &print_titles {
        if let Some(worksheet) = get_worksheet_by_name(&mut workbook, sheet_name) {
            let titles = titles.split(':').collect::<Vec<&str>>();
            if titles.len() == 2 {
                let first_row = titles[0].trim_start_matches('$').parse::<u32>().unwrap_or(0) - 1;
                let last_row = titles[1].trim_start_matches('$').parse::<u32>().unwrap_or(0) - 1;
                worksheet.set_repeat_rows(first_row, last_row);
            }
        }
    }

    // Save the workbook
    let file_path = "workbook.xlsx";
    workbook.save(file_path)?;

    // Print XML content for debugging
    print_xml_content(file_path)?;

    Ok(())
}

// Helper function to find a worksheet by name
fn get_worksheet_by_name<'a>(workbook: &'a mut Workbook, name: &str) -> Option<&'a mut Worksheet> {
    for worksheet in workbook.worksheets_mut().iter_mut() {
        if worksheet.name() == name {
            return Some(worksheet);
        }
    }
    None
}

// Helper function to read and print XML content from the workbook file
fn print_xml_content(file_path: &str) -> Result<(), Box<dyn std::error::Error>> {
    let zip_file = File::open(file_path)?;
    let mut zip = ZipArchive::new(zip_file)?;
    for i in 0..zip.len() {
        let mut file = zip.by_index(i)?;
        if file.name().ends_with(".xml") {
            println!("File: {}", file.name());
            let mut contents = String::new();
            file.read_to_string(&mut contents)?;
            println!("{}", contents);
        }
    }
    Ok(())
}

### Environment

```text
-Rust version: 1.78.0
-rust-xlsxwriter version: 0.68.0
-Operating System: Windows 11 Pro

Any other information
We have identified an issue with the quote_sheetname function in the rust-xlsxwriter library where sheet names containing hyphens and certain other characters are not being correctly quoted. This results in Excel not recognizing the print titles specified in the workbook.
Steps to Reproduce:
Create a new workbook using the rust-xlsxwriter library.
Add worksheets with names containing hyphens and spaces.
Set print titles for these sheets.
Save the workbook and inspect the generated XML.
Open the workbook in Excel and check if the print titles are applied.

workbook.xlsx

Thanks for the detailed report. That is a bug. I'll look into it.

I've pushed a fix for this issue to main.
I also want to add support for adding user-quoted worksheet names via Worksheet::set_name() for cases where the current automatic handling isn't sufficient such as worksheet names that contain A1 or RC style reference. So I will leave this open until I fix that as well.

I've pushed a more comprehensive fix to main that also takes into account A1 and RC cell references in the name and also emojis. Basically all the cases in the StackOverflow link you posted.

Fixed on main and in v0.69.0. Thanks for the report.


Referencing properties of a data structure in the document.

Hi,
I opened a question on StackOverflow a while back but it didn't get much traction so I assume this feature does not exist. However, it would be so cool if it did!
So to summarize:
I want to reference properties inside of a declared data structure in order to reuse them later in various parts of the document where the actual content of the property might not be as memorable and it would be nice if it had a name.
The first time I thought about it is HTTP return codes. If you want to standardize return codes throughout the document then you need to declare properties and refer to them but people make mistakes and a 201 is different from a 200 so a property named success inside a Http data structure would be very helpful.
Actual more detailed explanation with example:
https://stackoverflow.com/questions/47849041/referencing-a-property-inside-a-defined-data-structure-in-api-blueprint-mson

This is not possible and there's no plan for this. You can create 3 different data structures for each status code and then use them individually. And then combine them together to get the major HTTPStatusCode data structure.


Problem with Partial loader for prefered language 'ru'

I faced such a strange problem: when i execute this command $translate.use('ru') - nothing happens and no locales loaded with partial Loader. But when i use 'en' (for english) or 'es' (for spanish) everything goes right and all used locales are successfully loaded. Very weird problem.

Hard to guess without an example Plunkr. Could you setup one?

Closed due inactivity.


python311Packages.losant-rest: 1.19.3 -> 1.19.4

Diff: https://github.com/Losant/losant-rest-python/compare/v1.19.3...v1.19.4
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

4 packages built:

python311Packages.losant-rest
python311Packages.losant-rest.dist
python312Packages.losant-rest
python312Packages.losant-rest.dist


Remove jyasveer from admin-list

Offboarding. Internal ref: https://gitlab.cee.redhat.com/dtsd/housekeeping/issues/2319

Thanks Roman


DogeWallet misnamed in Wallets

On Mar 22, the name of "DogeWallet", the Dogeparty wallet released in Oct 2021, was changed to "XDP Wallet". This has never been the wallet's actual name. I understand there may potentially be some confusion as the wallet was initially titled "Doge Wallet" on dogecoin.com, when the actual name of the wallet is "DogeWallet" (without the space). This is the only Dogecoin wallet client with active development that has this name and its GitHub shows up as the first result in a Google search for the term.
To allay any potential confusion with "Dogecoin Core" (if this was the reason the name was changed in the first place), perhaps additional clarification could be made by titling the wallet "DogeWallet (Dogeparty)". But leaving the name as "XDP Wallet" doesn't make sense as its never been called that and it is confusing for people looking for the DogeWallet.
Here is the pull request for the name change, which was committed on Mar 22:
https://github.com/dogecoinfoundation/dogecoin.com/pull/270
Thank you for your attention to this matter.

I propose changing the name to 'dogewallet' as requested but adding an interim gate/catch that downloaders must read, explaining what 'dogewallet' actually is, and that it is not endorsed by or the 'official' application of dogecoin. (Not even Dogecoin Core does this.)  - as there is community history in people feeling that the name 'dogewallet' inferred primacy / officiality and de-facto endorsement.
If the issue truly is about the "name being changed" this alleviates that concern; there is nothing saying that the dogecoin.com site cannot disavow endorsement and indicate the software roots/origin.

I propose changing the name to 'dogewallet' as requested but adding an interim gate/catch that downloaders must read, explaining what 'dogewallet' actually is, and that it is not endorsed by or the 'official' application of dogecoin. (Not even Dogecoin Core does this.) - as there is community history in people feeling that the name 'dogewallet' inferred primacy / officiality and de-facto endorsement.

I'm not entirely certain what you mean by "gate/catch" but if you feel adding something beyond "DogeWallet (Dogeparty wallet)" is necessary then that is OK with me.

If the issue truly is about the "name being changed" this alleviates that concern; there is nothing saying that the dogecoin.com site cannot disavow endorsement and indicate the software's roots/origin.

The issue stems from the title of the DogeWallet being changed to an incorrect name. It's akin to operating a directory of fast food places and changing the listing for "Burger King" to "Home of the Whopper".
If you feel additional descriptors are required to ensure "primacy/officiality" or endorsement is not inferred, that is fine. From my personal point of view adding "(Dogeparty wallet)" should suffice in terms of resolving potential confusion.

Hello, we're coming up on 2.5 months since this issue was raised. It looked like we were in agreement on a solution in April. Can you please change the name of the wallet to "DogeWallet (Dogeparty wallet)"? As a reminder, it has never been referred to as "XDP Wallet". This is an incorrect name. Thank you for your attention to this matter.

@nutildah Kindly close this issue please. The changes have been merged and the issue is corrected on the website.

This issue was already resolved.


Syncapi fatal error if visibilities already present

Shopware Version: v6.1.3 Stable Version
Description:
If i try to sync a shopware product with the sync api, i get the 500 error if visibilities are already present in the database. So the second upsert of the product always fail because shopware is trying to insert the item instead of ignore it.

Expected behaviour:
Pass visibilities with product upsert. If visibilities are present, ignore insert.

Actual behaviour:
First sync is successfully if no product present in shopware. Second sync fails, because visibilities already created. This should not be the case on upsert.

Also an issue if using repository directly in <IP_ADDRESS>, push

@StevendeVries no its not expected behavior, its an UPSERT request to the sync api, which means update OR create the visibility (since we cant know the visibilityId if its already existing)

@digitalkaoz Not saying I agree with the approach, but if you do an upsert without an identifier it is considered as an insert. The unique constraint will result in a failure. First you will have to search for the visibilities and add the appropriate identifiers, it will then resort to an update.

Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know.
Thank you again for your valuable feedback.


Feature/rocket page styling

Styled the rocket page

Hi @Ayokunnumi1
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
✅ Descriptive README
✅ Descriptive PR summary
✅ Clean code
✅ Good first attempt
✅ Good use of separation of concerns
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.


Does kubevirt support live migration of local disks?

Hello,
The hostDisk only supports raw format, not qcow2 format, and does not support live migration.
The emptyDisk supports qcow2 format and live migration, but does not support specifying the qcow2 file path, and the disk is deleted when the virtual machine is shut down or deleted.
My requirement is: does kubevirt have a disk type that can combine the characteristics of hostDisk and emptyDisk? That is

qcow2 format;
Can specify the qcow2 file path on the host;
Supports live migration;
The disk is retained and not deleted when the virtual machine is shut down or deleted.

close


Typos in Enable Forced Tunneling

Under the heading 'Enable Forced Tunneling":
"Some security experts consider this to be a potential risk and therefore recommend that split tunneling is disabled and all connections"

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

ID: a1856224-8709-d6db-c533-50e5e05f2091
Version Independent ID: f4f18c8f-c849-6da6-e0ab-a52c6eeec07a
Content: Azure Network Security Best Practices
Content Source: articles/security/azure-security-network-security-best-practices.md
Service: security
GitHub Login: @TomShinder
Microsoft Alias: TomSh

Hit enter before I was done, leaving a somewhat ironic comment :)
Anyway, the typo is "disabled and all connections", which doesn't make grammatical sense.

@Alexei-B Thanks for your feedback! We will investigate and update as appropriate.

@Alexei-B We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[SCB-478]revert netty's version to 4.1.17 and vertx's version to 3.5.0

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage remained the same at 87.492% when pulling 9ffc114dd7de177ccaacb1843c48e7c6bfcefc76 on xuyiyun0929:master into 6f66e2622b0c056d26682672b881c15920f06fba on apache:master.

https://github.com/vert-x3/vertx-web/issues/906
3.5.1 bug already fixed with 3.6.0-SNAPSHOT
when 3.6.0 released we will upgrade to 3.6.0


Lua and plugin files download for unsupported devices

@dkerr64 There are 2 endpoints on Midea servers which provide information about the specified device, so in case someone wants support for a device which is not supported by the other plugin it would be possible (in theory) to use these files to reverse engineer the packets, etc... for the device and create the code for it. I've already implemented part of the code in feat/e2-support which will handle the downloading of the files.
The question is: how the flow should look like? My suggestion is to add a button in the config UI when there is an unsupported button. The button will download both of these files, and there should be a popup somewhere to create an issue and attach these files. I just don't know how user-friendly it is to download 2 random files from 2 random URLs without knowing what they really are. I mean they should mean no harm, but I would kind of understand if a user wouldn't want to download these without at least seeing the URL or something, but I might overthink this a bit.
What do you think?

Not sure I have a strong opinion.  But certainly anything that makes it easier for a user to capture required information and submit it in an issue to ease adding support for an unknown device sounds like a good thing.

This feature will be available with the next release.


Script behaviour can't ne abstract

Abstract class for Trackbinding (warning)
"Can't add script behaviour DataListener. The script class can't be abstract!"
Doesn't cause any issues so far.

solved in latest release


ecto.gen.repo generates configuration in config/config.exs even if there is an import_config

We should rather try to insert right below the use Mix.Config call.

See https://github.com/phoenixframework/phoenix/issues/1182.


Do not require primitive constructor values by default

This matches jackson behavior. There was previously an error trying to set primitives to null, this patch assigns them the default value, instead.
Also introduces support for @JsonProperty(required=true) to allow requiring primitive properties as well.
Fixes #169

could you improve the sonar coverage and fix the smell?

@graemerocher main uncovered thing is the not-primitive branch of DefaultSerdeRegistry getDefaultValue. That isn't actually reachable by existing serialization code though


Block proposal voting simulation

Depends on #2
Block Proposal Voting
This should replace the use of the proposal acceptance variable
We want to simulate the block proposal acceptance where a voting among every validator is performed. Until now, this acceptance was modeled with a Bernoulli variable, but now we want to delegate the voting part to the other validators and to make things a little bit salty, every validator will know its owner and owners can be Byzantine or not. Finally, every node has a certain probability of suffering a timeout.
If the owner is Byzantine then every node he owns will vote in favor of its own proposals and decline honest ones.
If the owner is not Byzantine then every node he owns will vote in favor of every proposal, but if the leader's owner is byzantine, it has a certain probability of detecting this and it has to make a decision between leaving the network and declining the vote.
Cases to simulate:

20 byzantine nodes and 10 honest nodes.
10 byzantine nodes and 20 honest nodes.
30 honest nodes.

Congratulations @ilitteri you just made an awesome and complex simulation :tada: :tada: !!!
Just some comments to make the code more reproducible:

If you use leader_election_simulation/package_manager.jl the way you are doing, its not possible to manage the packages version, and for some people the code wont compile.
To be able to manage versions, you have to create the following files:
Project.toml and Manifest.toml (the last one autogenerated).
Here are some links you can follow:

https://pkgdocs.julialang.org/v1/toml-files/
https://towardsdatascience.com/how-to-setup-project-environments-in-julia-ec8ae73afe9c
After doing so, explain how to run the environment in the README.txt file.

Please add the folders were you are writing the csv files. You can simple push those or create them in the code. Something like:

    mkpath("./data/stake")
    mkpath("./data/balance")
    mkpath("./data/proposals")
    CSV.write("./data/stake/$(scenario).csv", DataFrame(evolution_of_validators_stake_in_rounds["stake"]), compress=true)
    CSV.write("./data/balance/$(scenario).csv", DataFrame(evolution_of_validators_stake_in_rounds["balance"]), compress=true)
    CSV.write("./data/proposals/$(scenario).csv", DataFrame(evolution_of_validators_stake_in_rounds["proposals"]), compress=true)

I leave it up to you  :P
That's all on the setup side!
Finally, I was thinking that you could upload some graphics so that the simulation results can be better appreciated.
And that's it! Again I wanted to take this opportunity to congratulate you, you are doing an incredible job. Keep it up!

First of all, thank you very much for the feedback! I really appreciate it.

1. If you use `leader_election_simulation/package_manager.jl` the way you are doing, its not possible to manage the packages version, and for some people the code wont compile.
   To be able to manage versions, you have to create the following files:
   `Project.toml` and `Manifest.toml` (the last one autogenerated).
   Here are some links you can follow:

https://pkgdocs.julialang.org/v1/toml-files/ https://towardsdatascience.com/how-to-setup-project-environments-in-julia-ec8ae73afe9c
After doing so, explain how to run the environment in the README.txt file.

I'll definitely start working on this!

2. Please add the folders were you are writing the csv files. You can simple push those or create them in the code. Something like:

    mkpath("./data/stake")
    mkpath("./data/balance")
    mkpath("./data/proposals")
    CSV.write("./data/stake/$(scenario).csv", DataFrame(evolution_of_validators_stake_in_rounds["stake"]), compress=true)
    CSV.write("./data/balance/$(scenario).csv", DataFrame(evolution_of_validators_stake_in_rounds["balance"]), compress=true)
    CSV.write("./data/proposals/$(scenario).csv", DataFrame(evolution_of_validators_stake_in_rounds["proposals"]), compress=true)

About this, is it more preferable to do it in the code or to add the creation to the makefile directive like this:
define create_data_dir
	if [ -d "./data" ]; then \
        rm -r data; \
		mkdir data; \
		mkdir data/stake; \
		mkdir data/balance; \
		mkdir data/proposals; \
	else \
		mkdir data; \
		mkdir data/stake; \
		mkdir data/balance; \
		mkdir data/proposals; \
    fi
endef

run:
	$(call create_data_dir)
	julia leader_election_simulation.jl


rethink mutuallyExclusive argument in intesect

Doesn´t seem particularly intuitive. Probaly we should either move/ remove it, or at least better explain it.

Another issue in this function as is now, is that if mutuallyExclusive is TRUE, the argument  keepOriginalCohorts can be missleading since even though individual cohorts will be returned they might not be as the original (intersecctions will have been excluded)

As discussed:

Add "minus" cohorts as new cohorts rather than overwriting the old
rename "mutuallyExlusive" to "returnNonOverlappingCohorts"

returnNonOverlappingCohorts defaulted to TRUE or FALSE? @edward-burn


Fixed the executable formation

When the program is compiled, the output is stored in build/filename.d file, from where it is executed.
And the executable is stored in build/dist/

Hope you will merge the commits
Hopefully, Pranav

Sir, I have opened another pull request, with more features added to it. Please check it.
#35
Thank you


task/internal/syslog: blacklist log from ceph-create-keys

the log format of ceph-create-keys is "$level:$name:$message"
Fixes: http://tracker.ceph.com/issues/20171
Signed-off-by: Kefu Chai<EMAIL_ADDRESS>

See http://tracker.ceph.com/issues/20171#note-12 for analysis of the jewel integration test failures and why this change fixes them.

@zmc ping?

I tested this, and it works fine.

@smithfarm thanks for testing.

test this please

test this please

test this please


Adding TCP keepalives support for the broker's connection

Addresses issue #407
Our Google Compute Engine applications have a couple of under utilized topics/partitions that only receive messages once an hour or so.  Google Compute Engine's networking automatically kills any connection thats been idle for more than 10mins.   This causes the Sarama clients on GCE to return this error read tcp <IP_ADDRESS>:9093: i/o timeout.
From Google Compute Engine's Docs:

https://cloud.google.com/compute/docs/troubleshooting#communicatewithinternet

Note that a system-wide "hidden" firewall rule is set to disconnect idle TCP connections after 10
minutes. If your instance initiates or accepts long-lived connections with an external host, you can
adjust TCP keep-alive settings to prevent these timeouts from dropping connections. ...

Keepalive Refs:

http://felixge.de/2014/08/26/tcp-keepalive-with-golang.html
http://tldp.org/HOWTO/TCP-Keepalive-HOWTO/overview.html

PS thanks for all the hard work!  You guys rock!

A few nits, but generally LGTM. You're welcome to add a line to the changelog if you wish.

LGTM apart from the nitpicks.

@wvanbergen or @eapache - Does the new config setting new a validator ?   https://github.com/Shopify/sarama/blob/master/config.go#L187

Assuming that a setting <0 is incoherent then yes, a validator is important - good catch!


oc logs group-sync-operator-controller-manager-5fccd67fd6-fnm5q  manager I0703    1 request.go:601] Waited for 1.027797416s due to client-side throttling, not priority and fairness, request: GET:https://<>:443/apis/jaegertracing.io/v1?timeout=32s 1.6883884619252145e+09  ERROR   setup   unable to start manager {"error": "configmaps lock is removed, migrate to configmapsleases"} main.main         /home/runner/work/group-sync-operator/group-sync-operator/main.go:78 runtime.main         /opt/hostedtoolcache/go/1.18.9/x64/src/runtime/proc.go:250

How do i resolve this the 2 operators are stuck.

Issue solved by uninstalling the Group Sync Operator and reinstalling it.
You can take a backup of your configs before you uninstall.


make canvas boxing aware

on itch ( or fullscreen) , canvas STYLE should fill up itch STYLE.WIDTH, but STYLE.HEIGHT should be set according to canvas.WIDTH /canvas.HEIGHT aspect ratio (which is game's parameter and could be runtime set )

now games are resizing to fit all parent available space a.k.a. "windowed fullscreen"


Custom Select not showing after clicking outside and click back on it  .

Issue Description
Current Behavior
The Custom Select component is experiencing an issue where it works fine on the first click, but when I click outside of the select box and try to show the select list again, it quickly shows and closes.
https://github.com/tamagui/tamagui/assets/73201789/6563d7e0-3b0e-421f-8e01-e7183697dc6e
Expected Behavior
1 - Click on the select box it shows select list .
2 - Click outside hide the select list .
3 - Click again on the select box to show the list.
Steps to Reproduce
1 - Retrieve the code snippet for the Select component from the Tamagui documentation.
2 - Display the Select component on a Google Chrome or in Edge browser .
3 - Click on the select box to show the select list.
4 - Click outside the select box on any place on the screen.
5 - Click again on the select box to show the list.
Tamagui Version
1.22.10
Additional Information
I have tested this issue on Google Chrome and Microsoft Edge browsers , but it might be present in other browsers as well.

related: https://github.com/tamagui/tamagui/issues/1058

@la55u
The #1058 appears only on Firefox , but this issue happens in multiple browsers and it's not the same behavior .

I updated floating-ui to latest and just tested in FF and can't replicate now, hbu?

The issue I am facing is not limited to a particular browser. Despite updating Tamagui to version 1.30.16 yesterday, I still experience the same behavior with the Select component.


Collect functions used for documentation building in docs_utils.py

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coverage: 91.126% (-0.4%) from 91.495%
when pulling 643c3c5558e796ecc14f70ae687ee57dca2b89c0 on docs_utils
into 6effa1e8f9449166796a4df2cdcad544b14a3d3d on master.


Style of a table inside a table detail row changed

Overview of the problem
Buefy version: 0.5.2
Vuejs version: 2.4.2
OS/Browser: MacOS 10.12.5 / Chrome 60
Description
Since Buefy 0.5.2 the styling of a table inside a table detail row is different compared to previous versions.
In 0.5.2. each row or cell contains a box-shadow, striped rows are missing and hovering over a table row does not change the color of the row. The inset shadow of the border of the detail row itself also changed.
Steps to reproduce
0.5.2

0.5.1

It seems that we have any issue with the overrides. I will try to correct it.

I'm implementing some new changes into it. Let me check what happened with the new version of Bulma :)
Thanks for the review!

Fixed in  #403


[v4] Move pollen defaults from config argument to import?

From #55.
Potentially move Pollen's defaults from an argument provided to a config function, to a seperate import provided in pollen-css/defaults or something, and have pollen.config.js export a plain object instead.
Current implementation
module.exports = (pollen) => ({
  output: './styles/pollen.css',
  modules: {
    radius: { ...pollen.radius, xs: '2px', sm: '4px ' },
  }
});

Proposal
const pollen = require('pollen-css/defaults');

module.exports = {
  output: './styles/pollen.css',
  modules: {
    radius: { ...pollen.radius, xs: '2px', sm: '4px ' },
  }
};

This would be a breaking change and therefore needs to to resolved before shipping a final v4. I'm currently leading towards the former (current impl) as more of a standard pattern, especially since untouched keys (or keys with a value of true) inherit pollen defaults, which I think the current pattern communicates better. But open to arguments on the contrary.

Actually after just documenting this I think the existing approach is the right one to go with. Muddying the package exports with defaults that are only used in this config file seems redundant. And since Pollen's config can take a plain object or function regardless, this can be added later if needed for different use cases (programmatic usage? Dunno).
Closing this for now


[kustomize_role] Integration into deploy-edpm

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

recheck

Please consider updating this to call the updated cifmw_ceph_client role [1] which can produce the values.yaml to replace archiecture's examples/va/hci/values.yaml for the post-ceph stage
[1] https://github.com/openstack-k8s-operators/ci-framework/pull/1130

Please consider updating this to call the updated cifmw_ceph_client role [1] which can produce the values.yaml to replace archiecture's examples/va/hci/values.yaml for the post-ceph stage
[1] #1130

Take it for granted @fultonj


New Gearing

Increases speed for the new gearing on the intake. Also adds some flags to disable subsystems in the robot map.

Whoops turns out I changed the wrong value for the speed.


Fix linux icons (electron-builder)

closes https://github.com/paritytech/fether/issues/190

Working for .deb not for AppImage or the binary

the icons seem to be correctly packaged in the appimage; I think the icon should show up fine in the .desktop file that would be created when installing with appimaged
same for the .deb package, the .desktop file and icons in the package look valid to me.

Awesome, I can confirm, AppImage from the latest draft and the binary is showing the icon too :tada:


Access denied for Youtube

The README's installation and usage example doesn't work for Youtube URLs.
$ sudo curl -L https://yt-dl.org/downloads/latest/youtube-dl -o /usr/local/bin/youtube-dl
$ sudo chmod a+rx /usr/local/bin/youtube-dl
$ youtube-dl https://www.youtube.com/watch?v=B1A9fjoqTn8
/usr/local/bin/youtube-dl: line 1: syntax error near unexpected token `newline'
/usr/local/bin/youtube-dl: line 1: `
Access denied
'

Access denied

yt-dl.org has been taken down by A German court.
See: issue #32495.


Insert Audio

This included adding "checkbox" options (and resizing of labels so that they display correctly)
Complete with an example page
This is to be a one of many plugins for inserting media.

I've released your plugin in v2.2.0
Thanks for your contribution!


Cannot save untitled notebook

Testing #101295

Create Untitled notebook.
Enter some content.
Try so save:

Failed to save 'Untitled-1': Unable to resolve filesystem provider with relative file path 'untitled:Untitled-1'
onDidChangeNotification @ notificationsAlerts.ts:40
(anonymous) @ notificationsAlerts.ts:26
fire @ event.ts:587
addNotification @ notifications.ts:207
error @ notificationService.ts:59
j @ fileCommands.ts:412
processTicksAndRejections @ internal/process/task_queues.js:85
async function (async)
j @ fileCommands.ts:410
G @ fileCommands.ts:372
handler @ fileCommands.ts:422
invokeFunction @ instantiationService.ts:61
_tryExecuteCommand @ commandService.ts:84
(anonymous) @ commandService.ts:73
Promise.then (async)
executeCommand @ commandService.ts:73
_doDispatch @ abstractKeybindingService.ts:200
_dispatch @ abstractKeybindingService.ts:159
(anonymous) @ keybindingService.ts:251

I can still reproduce this following these steps:

new untitled file
reopen with "Notebook"
this works and I can edit
try to save

 ERR Unable to resolve text model content for resource vscode-notebook-cell:/Users/bpasero/Desktop/Untitled-1#0000000: Error: Unable to resolve text model content for resource vscode-notebook-cell:/Users/bpasero/Desktop/Untitled-1#0000000
    at S.resolveTextModelContent (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:6045:7)
    at processTicksAndRejections (internal/process/task_queues.js:85:5)
    at async S.doCreateReferencedObject (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:6043:841)
    at async b.createModelReference (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:6045:676)
    at async resolveTextModelRef (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:3252:937)
    at async c.resolveTextModel (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:3288:866)

Btw maybe we could add a "New Untitled Notebook" command.

I'm not seeing any error in Insiders and master by following steps:

CMD+N to create an untitled file
Reopen With Github Issue Notebook
Edit
CMD+S to save, save as test.github-issues
The editor is then replaced with new github notebook editor

Closing and removing verification-found to let others give it a test. Maybe @bpasero can reproduce this issue in other ways, if so we can reopen and then fix in next iteration.

@rebornix my steps involve not setting any file extension and you get this error: Unable to open 'Untitled-1': Cannot read property 'kernel' of undefined.
Your steps work fine. I can open a new issue if you want?

@bpasero thanks, the error should be avoided as the file extension is necessary for saving a github notebook. I created a new issue https://github.com/microsoft/vscode/issues/101827 to track it.


Add sprunge.us and paste.rs providers

I added a couple of providers more:

sprunge.us
paste.rs

I prefer paste.rs (take a look if you don't know it)
I changed the docs too but I didn't included the compiled lua files because they change all the variable names, maybe it's better if you do it as yourself :)

Thanks! :airplane:

Thank you for your work 😄


TabControl does not respect initial value of SelectedValue

Paste this XAML into any window and execute it:
<StackPanel>
	<ListBox SelectedValue="{Binding $self.Tag}" Tag="World" SelectedValueBinding="{Binding Content}">
		<ListBoxItem Content="Hello"/>
		<ListBoxItem Content="World"/>
	</ListBox>
	<TabControl SelectedValue="{Binding $self.Tag}" Tag="World" SelectedValueBinding="{Binding Header}">
		<TabItem Header="Hello"/>
		<TabItem Header="World"/>
	</TabControl>
</StackPanel>

You will see that the ListBox has selected "World" as expected, but the TabControl has selected "Hello" instead.

Inspecting the logical tree shows that TabControl.Tag has been incorrectly changed to "Hello".
Expected behavior
The TabControl should start a SelectedValue of "World" and the second tab item should be selected.
Environment

OS: Windows
Avalonia-Version: 11.0.9

Maybe you need to set DataContext also to $self? Inspect with DevTools maybe the current DataContext of TabControl.

There is no DataContext at all in this sample.


Timeouts on a single connection

@jvf @harunzengin this PR can be a collaborative attempt at figuring out what the flying horse crab hell is going on with #356.
At the time of opening it, the PR only adds a test to reproduce the timeouts (which does reproduce them ~90% of the time in my experience) and some additional logging.
Most of the time, Xandra does not receive the frame that times out. This is weird. I’m running this with a locally-running Dockerized Cassandra in case it helps.
Btw, I’m opening this because I won't have a ton of time to dedicate to this as I’m pretty busy at work, but I figured we can dig in together, especially after @jvf's fantastic reproducing steps and tests in #356 🙃

We will have a look.

I may have found something:
CASSANDRA_NATIVE_PROTOCOL=v3 mix test --only test:"test concurrent requests on a single connection"

and
CASSANDRA_NATIVE_PROTOCOL=v4 mix test --only test:"test concurrent requests on a single connection"

do not produce a failure, only
CASSANDRA_NATIVE_PROTOCOL=v5 mix test --only test:"test concurrent requests on a single connection"

does. Tested with up to max_requests = 100 (in test/xandra_test.exs:349). So this may be a problem with native protocol v5!

Wooooah fantastic find!!!!

I started looking at the v5 implementation, but nothing jumped out at me. Since the workaround (forcing protocol_version: :v4) is sufficient for us, I did not get the approval to investigate this further.

I've created https://issues.apache.org/jira/browse/CASSANDRA-19753 to see if this might be a C* issue.

With the help of Sam in the JIRA issue, we figured out that the issue was that I screwed up decoding multiple frames in a single envelope in native protocol v5 🤦 My bad. I pushed fixes into this PR.


pruned_transducer_stateless7_streaming export for ncnn

I trained a model with pruned_transducer_stateless7_streaming. I'm experimenting with Portuguese and was able to produce an lstm model a while ago with modest success, exporting it to ncnn format and using it in my code, so used the same data and dataloader. I used train.py and the training completed, decoding made sense, I exported to ncnn. In my code I then get 'layer pnnx.Expression not exists or registered'.
Then I noticed a comment about train2.py and zipformer2,py only existing for exporting to sherpa-ncnn (the comment at the bottom of the readme says 'emformer2.py' rather than zipformer2.py, but...). So I tried those. And I get an assert at line 1884 of zipformer2.py,
  File "/working/portuguese_zipformer/zipformer2.py", line 1884, in forward
    assert self.x_size == x.size(0), (self.x_size, x.size(0))
AssertionError: (16, 380)

I'm now lost. I'm not a Python expert but there's an implication that if you run zipformer2.py 'standalone' it runs some tests, but that asserts too. Anyone can try this...
Any advice on training and exporting models suitable for sherpa-ncnn?

Have you followed our documentation at
https://k2-fsa.github.io/icefall/model-export/export-ncnn.html
It is a detailed step-by-step guide.

A confusion over my build of pnnx seems to have led me on a wild goose chase yesterday afternoon. Reset according to the instructions and it all works. So sorry for wasting your time and thanks for being so helpful.
FWIW I had to change line 27 of ncnn/tools/pnnx/CMakeFiles.txt because it seems Torch now requires at least C++17.


new project can't run normally

Hi,
when I build a new project and run it, the http://localhost:3010/ is a white page and don't has any information,

just a warning:
WARNING in ./app/app.js
Critical dependencies:
7:17-24 require function is used in a way, in which dependencies cannot be statically extracted
@ ./app/app.js 7:17-24
Thanks for your help.

React update to 0.13.3 make this issue, back to 0.13.2, it will be OK.


Multiple actions, is it possible to check if an action (identifiable by taskname) is already running ?

Is it possible to pass a taskName in BackgroundJob.isRunning() to check if that particular action is already running ?

@mx-kshitij, I am sorry, but as it is stated in the documentation. It is only possible to run one task at a time.


Retrieve Safe App info for signed messages

Depends on https://github.com/safe-global/safe-client-gateway/pull/1006

Retrieves Safe App info (name and logoUri) for Message
Adds safe_app_info_by_id to InfoProvider – this function tries to retrieve a Safe App with the given id. If the Safe App is not found, an ApiError with 404 is returned
Sets safeAppId as nullable – the Safe Transaction Service supports messages without an associated safeAppId

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 26 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 31.395%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/messages/routes.rs
0
9
0.0%

src/providers/info.rs
0
17
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
12987

Relevant Lines:
14832

💛  - Coveralls


array.shuffle() does not work properly

https://godot.readthedocs.io/uk/latest/classes/class_array.html#class-array-method-shuffle
if you use that, you restarted your game, from the editor, you will have the same values over and over again.

That is expected behaviour. If you want to get random values each time you run the game you should use the function randomize().
A note could be added to the docs pointing users to randomize<EMAIL_ADDRESS>

Thanks for pointing me to randomize()
Should I close the issue now or after the docs gets updated?

You should close it after the docs get updated.

Should there just be a whole page in the docs describing how RNG works and how seeds/randomize work? I've seen similar issues of people reporting random things being exactly the same every time they run their game, and it's because they haven't randomized or set a seed.

By the way why not randomize the seed by default? what is the benefit of not doing that? doesn't that become predictable and thus as if one never used array.shuffle() at all? I mean what is the point of having a shuffle that can't 'shuffle'. If Something isn't random, it isn't shuffled, so why shuffle doesn't call randomize by default?

@CowThing That is a good idea, it would be a welcome addition. You should make an issue in the godot-docs repo and if you have time maybe put together a PR. It would be a good tutorial to put in the math section.

By the way why not randomize the seed by default?

@elieobeid7 Oftentimes what you want when using randomness is to have a different result each time the function is called rather than a different result each time the game is run. This is especially the case for games that use procedural generation. You want to reliably generate the same data each run.

I mean what is the point of having a shuffle that can't 'shuffle'. If Something isn't random, it isn't shuffled, so why shuffle doesn't call randomize by default?

It is still randomized, its just the same random sequence each run. If you run array.shuffle() multiple times in the same run it will return a different order each time.
Computers actually cannot produce 'random' numbers. All random number generation is actually psuedo random number generation. It takes a seed as input and produces a seemingly random set of results as output. This is how nearly all RNG works on computers except for high-tech cryptography algorithms and such.

@clayjohn But there is the RandomNumberGenerator class now, so if you need reproducible noise for a given task, it's best to setup a local generator with a seed you control. The way I see it, the core rand functions (and Array.shuffle by extension) shouldn't be expected to be reproducible because the same generator can be accessed from everywhere.
Unless I'm having a giant brain fart, It would be preferable imo to randomize the core random functions on startup and have the user use their own RandomNumberGenerator when reproducible noise is desired.
Also, Array.shuffle and other functions that depend on core random functions could probably use an optional RandomNumberGenerator argument to be used instead of the core functions!

The way I see it, the core rand functions (and Array.shuffle by extension) shouldn't be expected to be reproducible because the same generator can be accessed from everywhere (and used by stuff like particles).

I don't see why that would be a problem. But I also have no preference for the default behaviour. I'm just stating the current behaviour. 🤷♂

Unless I'm having a giant brain fart, It would be preferable imo to randomize the core random functions on startup and have the user use their own RandomNumberGenerator when reproducible noise is desired.

Thats a good idea! It just isn't the way things work now. Good idea for a PR!

I don't see why that would be a problem.

Should have phrased this better! I meant that any unforseen call to rand will mess your pattern (user input, threading, etc). It's only a problem if you need the pattern to be the same in all runs (think of procedurally generating a world), in which case I think you have a choice but to use RandomNumberGenerator.
I don't have a preference either, but the argument against randomizing on startup is good for simulation softwares, not so much for a game engine I think.

Fixed by #28832.