The underline connection was closed

I am authenticating through azure ad app and it used to work before.
but now I am getting this error
Error executing scan : The underline connection was closed. An unexpected error occurred on a send
I am trying execute it for classic workflow report.


"Nucșoara" is listed in the drop-down menu

Description:
When user writes "Nucșoara" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Nucșoara"

Expected results
The location appears in the drop-down menu as "Nucșoara, Argeș"


refactor(codegen): add undefinedLiteralTypeAnnotation prop in parsers

Summary:

part of codegen issue https://github.com/facebook/react-native/issues/34872

Add an undefinedLiteralTypeAnnotation: string property into the Parser object and implement it in the FlowParser (returning VoidLiteralTypeAnnotation) and in the TypeScriptPArser(returning TSUndefinedKeyword). Replace them in and parsers/typescript/components/events.js.

Changelog:

[Internal][Added]: Add undefinedLiteralTypeAnnotation property in parsers
Test Plan:

yarn test

/rebase

Hi @Shubham1429, 👋
Congratulations 🎉 on your "First Pull Request" to React Native! 🥳
Thank you, 🙏 for taking the time to contribute to this project and share your ideas.
Someone from the team will review your PR and get back to you as soon as possible with feedback if any. 😊 Post review, and if all tests are passing, your PR will be imported to internal FB phabricator and should be "merged" if it also passes all internal tests. 🥳

Contributing Guidelines | React Native Website | Framework discussions, proposals and RFCs

Hi @Shubham1429! Can I ask you to rebase and resolve conflicts, please? With more people working on Codegen is normal to have to spend some time in this kind of tasks. Thanks!


Package Version Performance Regression

Overview

Fixes performance issues with get_package_version by reverting #878.
Avoid reintroducing DeprecationWarnings by suppressing warnings when accessing package version attributes.

Accessing these first for speed is intended behavior whether or not the version attribute is marked as deprecated. The agent is robust enough to handle the removal of these attributes, which makes the DeprecationWarnings irrelevant.

Related Github Issues

Closes #927
Closes #918

Codecov Report

Merging #970 (a0877f3) into main (b2e9e74) will decrease coverage by 0.66%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main     #970      +/-   ##
==========================================
- Coverage   81.78%   81.13%   -0.66%
==========================================
  Files         190      190
  Lines       19597    19599       +2
  Branches     3405     3406       +1
==========================================
- Hits        16027    15901     -126
- Misses       2588     2694     +106
- Partials      982     1004      +22

Files
Coverage Δ

newrelic/common/package_version_utils.py
90.54% <83.33%> (-2.52%)
:arrow_down:

... and 17 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


[SILGen] Fix a bug where the wrong convention was being used for computing the type of a closure thunk

The ObjC selector family convention was being used instead of the C function type convention.
rdar://127090209

@swift-ci please test

@swift-ci please test

@swift-ci please test


Header for generated scripts, generated scripts are readonly

Header for generated scripts, generated scripts are readonly (#818).

5>C:\Users\Katelyn\Documents\Projects\ilwasm\third_party\JSIL\JSIL.Libraries\Targets\MSBuild.Gulp.targets(55,9): warning : [21:45:09] 'metascript' errored after 624 ms
5>C:\Users\Katelyn\Documents\Projects\ilwasm\third_party\JSIL\JSIL.Libraries\Targets\MSBuild.Gulp.targets(55,9): warning : [21:45:09] Error: EEXIST, mkdir 'C:\Users\Katelyn\Documents\Projects\ilwasm\third_party\JSIL\Libraries'
5>C:\Users\Katelyn\Documents\Projects\ilwasm\third_party\JSIL\JSIL.Libraries\Targets\MSBuild.Gulp.targets(55,9): warning : at Error (native)
5>C:\Users\Katelyn\Documents\Projects\ilwasm\third_party\JSIL\JSIL.Libraries\Targets\MSBuild.Gulp.targets(56,9): error : Error running gulp task 'build-Debug'. See Warnings for details.

Will check in few minutes.

Sorry for that, I'm only learning gulp and it worked sometimes :).

Not a problem :-)


refactor: replace derive_builder with bon

cargo build --release

package
commit
size
ratio

derive_builder
7876617
10095880 byte(s)
100%

bon
87988b2
10075400 byte(s)
99.8%

coverage

package
time[^1]

tarpaulin
10m44s

llvm-cov
2m5s

[^1]: record here in case that history of github actions get pruned


EfCoreValueConverter missing parameterless constructor

Describe the bug
In order for the value converter to work in the HaveConversion method a parameterless constructor is needed.
Currently trying to use the converter will throw
System.InvalidOperationException: "Cannot create an instance of value converter type 'EfCoreValueConverter'. Ensure that the type can be instantiated and has a parameterless constructor, or use the overload of 'HasConversion' that accepts a delegate."

In contrast the StronglyTypedIds lib does provide a parameterless constructor in the source generated converter which does solve this issue.
Steps to reproduce

Install latest version of EFCore (6.0.4) and Vogen (1.0.19)
In the DbContext override ConfigureConventions and configure a conversion with the source generated converter.
Create / Apply migrations will throw

Expected behaviour
The value converter should come with a default parameterless constructor.

I did look into making a PR but I really have no idea how to navigate this project 😅

Many thanks for reporting @PoolPirate . I'll take a look. I can also see you've submitted a PR, so many thanks for that too! I'll take a look now.


Add method for library call usage

Add from_points method for NAMatrix struct
Add small example to README.md

That all seems fine to me. I'm curious about what a benchmark would show though, but not so curious I'm going to actually do it.
I came across this unsafe version.
https://github.com/1crcbl/cykl-rs/blob/main/src/data.rs#L115
I suspect one could make a parallel version that avoids extra memory allocation and takes advantage of symmetry doing this.
I guess I should also mention that what I'm actually trying to do is TSP art and so far concorde & Lin-Kernighan has been my best option in terms of speed and looks. I just wish I could get this thing to build in windows.
https://github.com/equalis3r/concorde-rs
I'm generating TSPs from around 40,000 points and mostly just don't want the solution to visually cross over itself.

I'm curious about what a benchmark would show though, but not so curious I'm going to actually do it.

Yep, might be interesting, but I'm also not gonna do it.
Thanks again for your contribution.


fix: composer will crashed when add a new publish profile

Description
action function call error.
the setting update in file persistence lost projectId

Task Item
#minor
refs #3525

Screenshots

Coverage remained the same at 53.235% when pulling 430799bc6d2bd80d218e05dcd372499b6ab60104 on lei9444:actionbug into a2dcf998829dfc23fa66078986b67be93e8a48e5 on microsoft:master.


Setting cookies with auto-generated responseCookies are not passing in subsequent API /UI requests in newer Karate version 1.0.0 , 1.0.1

Submitting an issue regarding this - _https://stackoverflow.com/questions/67316742/setting-cookies-with-responseheaders-set-cookie-option-are-not-passing-in-s_ question that asked in StackOverflow as suggested.
API-cookie-tests.zip
I was successfully setting cookies with responseCookies in karate 0.9.6. But it stopped working when I upgraded karate version to 1.0.0/1.0.1.
I am facing error ReferenceError: "responseCookies" is not defined in karate 1.0.0 and 1.0.1 version.
Also, tried to set cookies with responseHeader Set-Cookie like,
ResponseHeaders["Set-Cookie"][1] = authcookie_key + "=" + authcookie_value in newer karate version.
Able to set responseHeader cookies but those are not passing in subsequent API request.

@meenalsanaye-coder before we look at this, can you try 1.1.0.RC1 and confirm if fixed

@meenalsanaye-coder I was starting to look at this but your example is no way near a way to replicate. please read the instructions again if needed: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue
as I said, this may be fixed. to re-open please follow the process properly

Hi @ptrthomas ,
I am re-opening this bug because we want to upgrade our project with newer version on high priority and we are stuck only due to responseCookies issue.
I have tried with 1.1.0.RC1 and 1.1.0.RC2 and can still repro this issue – 'ResponseCookies are not available in subsequent feature file' . This is working fine on 0.9.6.
I have updated the project with mockCookieURL.
•	You can just update karateversion in pom.xml with latest version to test
•	To run the project use ‘mvn clean test’
•	Expected Result – responseCookies should be printed from sample-api.feature file and tests should be passed
•	Actual Result in karate newer version (1.0.1, 1.1.0.RC1, 1.1.0.RC2) -
ReferenceError: "responseCookies" is not defined
api-cookies-tests.zip

@meenalsanaye-coder nope, sorry. your pom file has too many things in it like wiremock blah. try again or look elsewhere for help. please read the instructions and follow them EXACTLY as written, thanks


Place Musicautor

Make MusicAutor a contributor

@liaugminaite
If a user is not showing up from /content/authors, like Musicautor, the most likely problem is this:
In the YAML of Musicatuors page
Organizational groups that you belong to (for People widget)
user_groups:

Institutional partners

Must match with
in /content/contributorscontributors.md
content:
Choose which groups/teams of users to display.
Edit user_groups in each user's profile to add them to one or more of these groups.
user_groups:
- Institutional partners
- Creators


Troubleshoot compute-id

Depends-on: https://github.com/openstack-k8s-operators/edpm-ansible/pull/619

recheck
pre adoption VM failed to start:
 'stderr_lines': ['#=#=#                                                                         ',
                  '',
                  '######################################################################## '
                  '100.0%##O#-#                                                                        ',
                  '##=O#- '
                  '#                                                                      ',
                  '',
                  '                                                                           '
                  '1.2%',
                  '######################################################################## '
                  '100.0%',
                  'Warning: Identity file '
                  '/home/zuul/install_yamls/out/edpm/ansibleee-ssh-key-id_rsa '
                  'not accessible: No such file or directory.',
                  'Failed to discover available identity versions when '
                  'contacting http://<IP_ADDRESS>:5000. Attempting to parse '
                  'version from URL.',
                  'Could not find versioned identity endpoints when attempting '
                  'to authenticate. Please check that your auth_url is '
                  'correct. Unable to establish connection to '
                  'http://<IP_ADDRESS>:5000: '
                  "HTTPConnectionPool(host='<IP_ADDRESS>', port=5000): Max "
                  'retries exceeded with url: / (Caused by '
                  "NewConnectionError('<urllib3.connection.HTTPConnection "
                  'object at 0x7f93db2cf700>: Failed to establish a new '
                  "connection: [Errno 111] Connection refused'))",
                  'Warning: Identity file '
                  '/home/zuul/install_yamls/out/edpm/ansibleee-ssh-key-id_rsa '
                  'not accessible: No such file or directory.',
                  'Failed to discover available identity versions when '
                  'contacting http://<IP_ADDRESS>:5000. Attempting to parse '
                  'version from URL.',
                  'Could not find versioned identity endpoints when attempting '
                  'to authenticate. Please check that your auth_url is '
                  'correct. Unable to establish connection to '
                  'http://<IP_ADDRESS>:5000: '
                  "HTTPConnectionPool(host='<IP_ADDRESS>', port=5000): Max "
                  'retries exceeded with url: / (Caused by '
                  "NewConnectionError('<urllib3.connection.HTTPConnection "
                  'object at 0x7f0d957b2490>: Failed to establish a new '
                  "connection: [Errno 111] Connection refused'))"],

my bad
 'stderr_lines': ['+ declare -A computes',
                  '+ computes=(["standalone.localdomain"]="<IP_ADDRESS>")',
                  '+ for name in "${!computes[@]}"',
                  '++ oc exec -t openstackclient -- openstack hypervisor show '
                  'standalone.localdomain -f value -c id',
                  '+ uuid=0ba0860a-ab9b-48a8-b1fb-94bd5180705d',
                  "+ echo 'Writing 0ba0860a-ab9b-48a8-b1fb-94bd5180705d to "
                  "/var/lib/nova/compute_id on standalone.localdomain'",
                  '+ ssh -i '
                  '/home/zuul/src/github.com/openstack-k8s-operators/install_yamls//out/edpm/ansibleee-ssh-key-id_rsa '
<EMAIL_ADDRESS>'stat /var/lib/nova/compute_id ; echo "
                  '0ba0860a-ab9b-48a8-b1fb-94bd5180705d | sudo tee '
                  '/var/lib/nova/compute_id && sudo chown 42436:42436 '
                  "/var/lib/nova/compute_id)'",
                  'Warning: Identity file '
                  '/home/zuul/src/github.com/openstack-k8s-operators/install_yamls//out/edpm/ansibleee-ssh-key-id_rsa '
                  'not accessible: No such file or directory.',
                  "bash: -c: line 1: syntax error near unexpected token `)'",
                  'bash: -c: line 1: `stat /var/lib/nova/compute_id ; echo '
                  '0ba0860a-ab9b-48a8-b1fb-94bd5180705d | sudo tee '
                  '/var/lib/nova/compute_id && sudo chown 42436:42436 '
                  "/var/lib/nova/compute_id)'"],

recheck pre-launch VM failed, issue tracked in https://issues.redhat.com/browse/OSPCIX-227

recheck RETRY_LIMIT

recheck crc setup filed

recheck

recheck the adoption task timed out waiting for the first edpm deployment but it seem the deployment finished successfully when the must gather run. So I think we need to bump the wait time for the deployment during adoption

recheck

recheck the issue is not reproduced:
{'changed': True,
 'cmd': 'set -euxo pipefail\n'
        '\n'
        '\n'
        'oc wait --for condition=Ready --timeout=300s Nova/nova\n',
 'delta': '0:00:00.124146',
 'end': '2024-04-04 07:55:14.639534',
 'msg': 'non-zero return code',
 'rc': 1,
 'start': '2024-04-04 07:55:14.515388',
 'stderr': '+ oc wait --for condition=Ready --timeout=300s Nova/nova\n'
           'Error from server (NotFound): nova.nova.openstack.org "nova" not '
           'found',
 'stderr_lines': ['+ oc wait --for condition=Ready --timeout=300s Nova/nova',
                  'Error from server (NotFound): nova.nova.openstack.org '
                  '"nova" not found'],
 'stdout': '',
 'stdout_lines': []}

recheck issue is not reproduced
fatal: [localhost]: FAILED! => {"changed": true, "cmd": "set -euxo pipefail\n\n\noc wait --for condition=Ready --timeout=300s Nova/nova\n", "delta": "0:00:00.123971", "end": "2024-04-04 12:17:23.920450", "msg": "non-zero return code", "rc": 1, "start": "2024-04-04 12:17:23.796479", "stderr": "+ oc wait --for condition=Ready --timeout=300s Nova/nova\nError from server (NotFound): nova.nova.openstack.org \"nova\" not found", "stderr_lines": ["+ oc wait --for condition=Ready --timeout=300s Nova/nova", "Error from server (NotFound): nova.nova.openstack.org \"nova\" not found"], "stdout": "", "stdout_lines": []}

this not found error is strange. Nova/nova exists and ready according to https://logserver.rdoproject.org/70/370/773160e9ba239bb3547715a8e9a406dabf1a1203/github-check/data-plane-adoption-osp-17-to-extracted-crc-minimal-no-ceph/4ec60e5/controller/ci-framework-data/logs/openstack-k8s-operators-openstack-must-gather/namespaces/openstack/crs/nova.nova.openstack.org/nova.yaml

recheck all green

recheck

recheck all green (total 4 green job runs)

recheck all green (total 6)

and 1 in 8 runs reproduced the issue again:
/var/lib/nova/compute_id: operation not permitted
fatal: [standalone]: FAILED! => {"changed": false, "msg": "Failed containers: nova_compute"}

probably we need a wait somewhere

probably we need a wait somewhere

I thought about this.

the adoption side of the pre-generation of the file happens lot earlier than the container start as in between almost the whole 18 dataplane deployment runs
the edpm_nova role during the 18 dataplane deployment also sets permission on the file and that happens 6 seconds before the container failure. In successful runs I see 11 and 9 seconds between the two events. So maybe this can be a place to introduce delay.

Also this issue is not visible in greenfield deployment but there we have different podman version:

adoption 4.6.0.
greenfield 4.9.4
but looking at the changelog between these podman version does not show any relevant change.

recheck

recheck CI is back after a day off

recheck
minimal job timed out on the control plane deployment:
    message: OpenStackControlPlane Swift in progress

recheck

probably we need a wait somewhere

I thought about this.

the adoption side of the pre-generation of the file happens lot earlier than the container start as in between almost the whole 18 dataplane deployment runs
the edpm_nova role during the 18 dataplane deployment also sets permission on the file and that happens 6 seconds before the container failure. In successful runs I see 11 and 9 seconds between the two events. So maybe this can be a place to introduce delay.

Let's try with the edpm-ansible change that introduce 1 minute of pause after the permission change.

https://logserver.rdoproject.org/70/370/82d8d7f92733836cc9dfcaeeaf4678244fe5b21e/github-check/data-plane-adoption-osp-17-to-extracted-crc/0f9cd36/controller/ci-framework-data/logs/openstack-k8s-operators-openstack-must-gather/namespaces/openstack/pods/nova-compute-extraconfig-openstack-openstack-7425x/logs/openstackansibleee.log
so adding a 1 minute does not help so this is not a race around changing the permissions and starting a container.
Next I will try to not adding +i to the file to see if that helps

trying without the immutable flag on compute_id

recheck (2 pass)

recheck (4 passed)

recheck (6 pass)

recheck (7 pass)
we can ignore the retry limit as it did not reproduce the problem

recheck (8 pass)

recheck (10 pass)

After 12 passed runs I conclude that the older podman version used in 17 (4.6.0) is buggy if the mounted file has the immutable file attribute set. While the podman version used in greenfield 18 (4.9.4) does not show the issue. So I will propose a change that will not add the immutable flag to stabilize adoption.

Let's merge https://github.com/openstack-k8s-operators/edpm-ansible/pull/619 to stabilize adoption

https://github.com/openstack-k8s-operators/edpm-ansible/pull/619 has been merged as  fix


U2 olympics demo is not filtering based on text inputs

<select> inputs filter the list, but other controls do not.

Mark can you verify that this is fixed and close if it is. Thx.


Does Snap-and-Chat design split the minimum cost of attack for specific known attacks?

Question
In Ebb-and-Flow there are two subprotocols providing availability and finality respectively which are then coupled.
When starting a "known attack case analysis" (such as in #39) and also from conversations at Zcon4, we are concerned that some attacks have a lower min cost of attack estimate than their equivalents in pure PoW or PoS, because the cost to attack is the minimum of either component.
a. Is this concern true in general for Ebb-and-Flow?
b. If not, is it specific to TFL?
One confusion or complication is that if the two subprotocols in Ebb-and-Flow both rely on the same pool of stake for security cost estimates, then the two subprotocols are not mere black boxes, but have an important underlying coupling. (My understanding of modern Ethereum Ghasper is that it is similar to Ebb-and-Flow with shared staking pool for security.)

From §I.D (p.3):

Theorem (Informal). Consider a network environment where:

Communication is asynchronous until a global stabilization time GST after which communication becomes synchronous, and
honest nodes sleep and wake up until a global awake time GAT after which all nodes are awake. Adversary nodes
are always awake.
Then
(P1 - Finality): The finalized ledger LOGfin is guaranteed to be safe at all times, and live after max{GST, GAT},
provided that fewer than 33% of all the nodes are adversarial.

Provisional answer to this ticket:
Yes, Ebb-and-Flow splits the cost of attack if the two subprotocols use independent "security resources".
For the specific case of a PoW $\Pi_{lc}$ combined with $\Pi_{bft}$ there's an informal assertion shown in #50 because safety or dynamic availability assumptions are of the form:

[a security property] holds if [something about PoS nodes/resources] AND [something abot PoW nodes/resources].

Therefore, violating either branch of the "AND" clause violates the security guarantee.
So finally: iff the cost of compromising either side of the branch is independent, then the security property can be invalidated by cheaper of the two.
If this analysis is correct, how should we proceed? I can think of a few options:

Determine if the minimum of the two thresholds is still adequate for security for a global permissionless cryptocurrency, at least for a safe estimate for the period of time during which TFL will be active (because we intend to transition away from it to a better protocol in a subsequent phase. In this case, we proceed with current TFL design.
If not, explore potential ways to correlate the "security resource costs" in a variant of TFL so that compromising either branch of the "AND" clause requires compromising the other. The intuition is that then the "security budget" is not "split between resources".
Switch away from PoW+PoS Snap-and-Chat to a Snap-and-Chat with a shared security resource such as shared PoS stake (similar to Ethereum's current Gasper).
Switch away from Snap-and-Chat altogether.

Assigning @daira to review the above assessment. If ze concurs, we can close this ticket as resolved and open a new ticket for "how to proceed".

@nathan-at-least wrote:

For the specific case of a PoW combined with
there's an informal assertion shown in https://github.com/Electric-Coin-Company/tfl-book/issues/50 because safety or dynamic availability assumptions are of the form:

[a security property] holds if [something about PoS nodes/resources] AND [something about PoW nodes/resources].

Therefore, violating either branch of the "AND" clause violates the security guarantee.
So finally: iff the cost of compromising either side of the branch is independent, then the security property can be invalidated by cheaper of the two.

Safety
Safety (rollback-freeness) of $\mathsf{LOG}{\mathrm{fin}}$ depends only on the safety and assumptions of the BFT protocol, not $\Pi{\mathrm{lc}}$. You might be confused by Figure 7, but the text to the right of the figure confirms this:

We see from Figure 7 that the safety of $\Pi_{\mathrm{bft}} (box 1) implies the safety of $\mathsf{LOG}_{\mathrm{fin}}$ (box 5).

Note that the input to the BFT protocol is only "confirmed" snapshots, i.e. chains truncated by the last $\sigma$ blocks where $\sigma$ is the $\Pi_{\mathrm{lc}}$ confirmation depth. A failure of $\Pi_{\mathrm{bft}}$ can cause a rollback in $\mathsf{LOG}{\mathrm{fin}}$ and $\mathsf{LOG}{\mathrm{da}}$ past the finalization point, because snapshots may be agreed on in a different order. However, a node that is online through such a rollback can detect it and just refuse to roll back. (A node that is newly synced can't tell which chain to choose in this situation without reliable out-of-band information.) So, even if the BFT protocol fails it is no worse than our current 100-block rollback limit. Recovery from such a condition would also require reliable out-of-band information.
(The paper actually has a bug that affects ability to spend from some finalized outputs, which doesn't show up in the security analysis. It's fixable; I'm writing it up now.)
On any node $i$, $\mathsf{LOG}{\mathrm{fin},i}$ is always a prefix of $\mathsf{LOG}{\mathrm{da},i}$ by construction. Safety of $\mathsf{LOG}{\mathrm{da}}$ does depend on both $\Pi{\mathrm{lc}}$ and $\Pi_{\mathrm{bft}}$, but you would not expect otherwise, by the nature of an ebb-and-flow protocol. In particular the prefix property requires that if $\mathsf{LOG}{\mathrm{fin},i}$ changes then so does $\mathsf{LOG}{\mathrm{da},i}$. This is unavoidable, and it would potentially invalidate transactions after the finalization point that depended on outputs from the old $\mathsf{LOG}{\mathrm{fin},i}$ or that are double-spends of outputs in the new $\mathsf{LOG}{\mathrm{fin},i}$. However, again, a node that is online through such a rollback can refuse it, as above.
If $\mathsf{LOG}{\mathrm{fin}}$ does not roll back, then the remaining safety dependency of $\mathsf{LOG}{\mathrm{da}}$ is only on $\Pi_{\mathrm{lc}}$, and that dependency is "tight". That is, if $\mathsf{LOG}{\mathrm{fin}}$ does not roll back and there is no $k$-block rollback of $\Pi{\mathrm{lc}}$, then there will be no $k$-block rollback of $\mathsf{LOG}_{\mathrm{da}}$, for any $k$.
In terms of "split" resources (e.g. issuance) between $\Pi_{\mathrm{lc}}$ and $\Pi_{\mathrm{bft}}$ reducing the security of each of them, that's a potential issue but I don't think we have enough information to answer it in the abstract. It really depends on what the split is, and on the specific protocols involved.
Liveness
Liveness of $\mathsf{LOG}{\mathrm{fin}}$ and $\mathsf{LOG}{\mathrm{da}}$ is complicated. As the paper says, in both cases it is dependent on liveness of both $\Pi_{\mathrm{lc}}$ and $\Pi_{\mathrm{bft}}$, but snap-and-chat as described makes some potentially unrealistic assumptions in its network communication model that could be a problem in practice, and we can do better. I'll address this in the note I'm writing.
As I've argued elsewhere, having the $\mathsf{LOG}_{\mathrm{da}}$ chain become unavailable in a long finalization stall is not a bug.

I wrote:

Note that the input to the BFT protocol is only "confirmed" snapshots, i.e. chains truncated by the last blocks where is the confirmation depth.

More precisely, in the snap-and-chat paper we have this (middle-left of page 3):

This motivates the last ingredient of our construction: in the $\Pi_{\mathrm{bft}}$ sub-protocol, each honest node boycotts the finalization of snapshots that are not confirmed in $\Pi_{\mathrm{lc}}$ in its view.

That is, an honest node does not vote for snapshots that are unconfirmed in its view.
Recall that a confirmed snapshot of $\Pi_{\mathrm{lc}}$ is defined as a snapshot that is at least $\sigma$ blocks deep in a valid chain of $\Pi_{\mathrm{lc}}$. (Importantly, the chain must be valid to $\sigma$ blocks ahead of the snapshot, not just have sufficient work.)
The reason for the "honest" and "in its view" caveats is that the verifier of a signature by node $i$ on a proposal cannot directly check that the proposal actually was confirmed in the view of node $i$, because the verifier cannot know what chains node $i$ has seen. Nevertheless, there is an efficient way to strengthen this condition. If a proposal includes $\sigma$ block headers after the one that is being proposed, then anyone who sees the proposal can directly check whether it has sufficient proof-of-work "on top" of it, and indirectly check (if they have the blocks in the associated longer chain) that it is confirmed by the above definition. Note that validator signatures are on the proposal including these headers, and so the verifier of a validator signature can now be sure that the validator had seen sufficient information that it should have been able to check that the snapshot is confirmed.
Note that honest validators already need to have seen the chain $\sigma$ blocks ahead of a proposal and be full nodes in $\Pi_{\mathrm{lc}}$, in order to check whether it is confirmed. They should not block on obtaining the full chain for an arbitrary proposal (because that could allow a denial-of-service on validators); they should just not vote for it if they don't have enough information to check its validity. If they do and the chain exists but is not valid, then it is provable that they did not act honestly, and their stake can be slashed.
More generally, snap-and-chat as described in the paper keeps the interaction between subprotocols to the minimum that is necessary to prove their security claims. This is quite reasonable for a paper, because it makes snap-and-chat applicable to a wider range of subprotocols, and avoids distracting reviewers with things they may object to or misunderstand. While it's intuitively appealing not to entangle the two protocols "too much", I think that the trade-offs for an actual implementation favour having participants in each of $\Pi_{\mathrm{bft}}$ and $\Pi_{\mathrm{lc}}$ verify as much as they can of the other protocol, because this actually provides substantively stronger security. In particular, it constrains even dishonest nodes to only casting a vote that will be seen as valid for a chain that is confirmed at the time of the vote. This means that we know that 2/3rds of validators correctly voted for a confirmed chain in a successful proposal, not just 1/3rd.

I propose we change the scope of this ticket to Crosslink only, and we postpone it until the security proofs for safety and liveness of Crosslink are complete.

I need to re-review the comments on this ticket in the context of Crosslink.


How to pass a boto session to dd.read_parquet

S3fs accepts a botocore Session object, however it doesn't seem to be exposed in dd.read_parquet.
Passing the session via storage_options throws an AttributeError.
location = "s3://..."
# get a boto session with my institutional access SDK
boto_session = <institutional access thing>.get_boto_session(location)

df = dd.read_parquet(
    location,
    engine="fastparquet",
    columns=["id", "feature_x"],
    storage_options={"session": boto_session}
)
df.persist()

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
~/.conda/envs/dask-ol/lib/python3.7/site-packages/distributed/protocol/pickle.py in dumps(x)
     37     try:
---> 38         result = pickle.dumps(x, protocol=pickle.HIGHEST_PROTOCOL)
     39         if len(result) < 1000:

AttributeError: Can't pickle local object 'lazy_call.<locals>._handler'

I recalled that the parquet IO code was recently re-written. Should the session be handled a different way?

cc @mdwgrogan

Your usage storage_options={"session": boto_session}, looks correct. That would be passed through to s3fs.S3FileSystem on the worker. It just seems that boto3 sessions aren't pickleable
In [32]: session = boto3.session.Session()

In [33]: pickle.dumps(session)
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-33-ec304ee4ccf9> in <module>
----> 1 pickle.dumps(session)

AttributeError: Can't pickle local object 'lazy_call.<locals>._handler'

I can raise an issue with boto if you want. In the meantime: why do you need to pass through a session in the first place? I wonder if we can avoid it entirely.

We could just pass the creds like so...
df = dd.read_parquet(
    loc,
    engine="fastparquet",
    columns=["acct_id", "snap_dt"],
    storage_options={
        "key": creds.access_key,
        "secret": creds.secret_key,
        "token": creds.token,
    },
)

But we would rather not have the need to expose them to the user directly.  The magic institutional access thingy just gives us a boto_session from user-level context baked into the underlying infrastructure.

Gotcha. And using environment variables is not as nice of an option?
I wonder if we can register a custom serializer within distributed to work around this. The object itself is fine with cloudpickle
In [5]: import cloudpickle

In [6]: cloudpickle.dumps(session)
Out[6]: b'\x80\x04\x95\x95f\x00\x00\x00\x00\x00\x00\x8c\rboto3.session\x94\x8c\x07Session\x94\x93\x94)\x81\x94}\x94(\x8c\x08_session\x94\x8c\x10botocore.session\x94h\x01\x93\x94)\x81\x94}\x94(\x8c\x11...

However, once we actually do anything with it like
In [11]: s3 = session.client("s3")

In [12]: cloudpickle.dumps(session)

We get a long traceback like
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-12-70612137f216> in <module>

ending with
/usr/local/Cellar/python/3.7.5/Frameworks/Python.framework/Versions/3.7/lib/python3.7/pickle.py in save(self, obj, save_persistent_id)
    522             reduce = getattr(obj, "__reduce_ex__", None)
    523             if reduce is not None:
--> 524                 rv = reduce(self.proto)
    525             else:
    526                 reduce = getattr(obj, "__reduce__", None)

TypeError: can't pickle SSLContext objects

Will explore a bit.

I was wondering about a custom serializer as well. Thanks Tom!

My guess is that boto maintainers won't want to pass sessions around.  Semantically they're tied to, well, a particular session.
I think that institutionally credentials should probably be handled outside of Dask collections / task graphs, and instead maybe we should think about how to move them around with dask.distributed, dask-gateway, or other institutional infrastructure.

Played around with this a bit, and it's exactly straightforward to write a generic one. I don't know what all state needs to be extracted... Boto doesn't seem interested in supporting this either: https://github.com/boto/botocore/issues/636.
I see two options:

Register your own custom serializer with dask for these session objects. Send through just the bits you need to re-connect on the other side (assuming you can do the special institutional access thing on the worker).  One thing to check: will distributed use custom serializers for values in a dictionary? If not that may be a non-starter.
Override the lookup for s3 to point to a custom subclass of s3fs.S3FileSystem. Then your subclass' __init__ could take care of the institutional access thing.

Excellent, thanks for looking into this @TomAugspurger ! This gives us some options to consider until we have a deployment level solution.
@mrocklin Would you want to discuss this further in dask-gateway or distributed? I can open an issue there and we can close this one.

I'll defer that to @jcrist

The magic institutional access thingy just gives us a boto_session from user-level context baked into the underlying infrastructure.

How is the original boto session being created?
FWIW I agree with Matt that we probably shouldn't mess with serializing boto sessions, and should instead figure out how to make recreating those sessions easier for the end user. The parameters for instantiating them have to come from somewhere.

maybe we should think about how to move them around with dask.distributed, dask-gateway, or other institutional infrastructure.

Having admin or user defined environment variables is already supported in dask-gateway, but depending on backend they may not use the most secure method of defining them (e.g. in kubernetes we should allow setting environment variables from secrets). If you want to open an issue in the dask-gateway repo we can discuss further there.

It sounds like we'd prefer to handle this outside of dask and distributed. Does that seem reasonable? Anything else to do her @mmccarty?

Yes, that makes sense. Thanks everyone.


add the new certificate names

Adding new exported names for new certificates - one for data plane and one for control plane

/assign @ReToCode , @nak3, @evankanderson


[image_picker] [iOS] build fails on XCode 14, iOS 16, Stored properties cannot be marked potentially unavailable with '@available'

When I try to run my app that uses image picker 0.8.6 with Xcode 14 on iOS 16 I get this error:
Stored properties cannot be marked potentially unavailable with '@available'
[path]ios/Pods/DKImagePickerController/Sources/DKImagePickerController/DKImagePickerController.swift:67:5

Here is the code in that line:
/// false to prevent dismissal of the picker when the presentation controller will dismiss in response to user action.
    @available(iOS 13.0, *)
    @objc lazy public var shouldDismissViaUserAction = false

Steps to Reproduce

Add image_pikcer ^0.8.6 as dependency
Run flutter pub get
Execute flutter run

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.8, on macOS 13.0 22A380 darwin-arm, locale en-EG)
    • Flutter version 3.3.8 on channel stable at /Users/shadysamir/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 52b3dc25f6 (4 days ago), 2022-11-09 12:09:26 +0800
    • Engine revision 857bd6b74c
    • Dart version 2.18.4
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/shadysamir/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.73.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.52.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 13 (API 33) (emulator)
    • iPhone 14 Pro Max (mobile)  • 91EC2807-0D63-4522-96FB-96CF17053F25 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-1
      (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 13.0 22A380 darwin-arm
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 107.0.5304.110

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

pod update fixed the issue.

pod update fixed the issue.
what version of DKImagePickerController after you updated?


Unable to build the crate on archlinux

Whenever I try to build this crate on archlinux I get these errors:
error: could not find native static library `miniz`, perhaps an -L flag is missing?

error: Could not compile `miniz-sys`.
Build failed, waiting for other jobs to finish...
error: could not find native static library `openssl_shim`, perhaps an -L flag is missing?

But it builds good on debian 8. How to build it on archlinux?

Could you gist some more logs? It's pretty difficult to diagnose from just what you pasted :(

Yes, give me a hour approx.

This is what I have regarding miniz-sys while running cargo build --verbose:
rustc --crate-name miniz_sys /home/fx/.cargo/registry/src/github.com-1ecc6299db9ec823/miniz-sys-0.1.9/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=371886580b0e42eb -C extra-filename=-371886580b0e42eb --out-dir /home/fx/workspace/stonedguy/target/debug/deps -L dependency=/home/fx/workspace/stonedguy/target/debug/deps --extern libc=/home/fx/workspace/stonedguy/target/debug/deps/liblibc-026076ef8cc90032.rlib --cap-lints allow -L native=/home/fx/workspace/discord-pickupbot/target/debug/build/miniz-sys-18005000ddedadf4/out -l static=miniz

error: could not find native static library `miniz`, perhaps an -L flag is missing?

error: Could not compile `miniz-sys`.

Caused by:
  process didn't exit successfully: `rustc --crate-name miniz_sys /home/fx/.cargo/registry/src/github.com-1ecc6299db9ec823/miniz-sys-0.1.9/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=371886580b0e42eb -C extra-filename=-371886580b0e42eb --out-dir /home/fx/workspace/stonedguy/target/debug/deps -L dependency=/home/fx/workspace/stonedguy/target/debug/deps --extern libc=/home/fx/workspace/stonedguy/target/debug/deps/liblibc-026076ef8cc90032.rlib --cap-lints allow -L native=/home/fx/workspace/discord-pickupbot/target/debug/build/miniz-sys-18005000ddedadf4/out -l static=miniz` (exit code: 101)
Build failed, waiting for other jobs to finish...

Hm is this just a normal cargo build? Or is there some other build system in play here? The library in question should be inside /home/fx/workspace/discord-pickupbot/target/debug/build/miniz-sys-18005000ddedadf4/out, mind poking around in there and seeing if anything is located there?

Please don't close this issue. I have similar one with rust-openssl and it prevents me from checking is everything okay (is this isue fixed or not) because rust-openssl errors are shown now instead of miniz-sys.

Everything was okay actually. I must have performed cargo clean before updating because I have renamed target directory to another one and it tried to find the library in unexistent place.


feat: Terraform docs default markers

Put an x into the box if that apply:

[ ] This PR introduces breaking change.
[ ] This PR fixes a bug.
[x] This PR adds new functionality.
[ ] This PR enhances existing functionality.

Description of your changes
Allow users to switch to the standard insertion markers used by terraform-docs, to enable CI jobs to check that docs have been updated.
How can we test changes
Based on this suggestion for testing hooks: https://github.com/antonbabenko/pre-commit-terraform/pull/25#issuecomment-431649551
Add the following to the pre-commit config:
- repo: local
  hooks:
  - id: terraform_docs
    name: terraform_docs
    entry: ./hooks/terraform_docs.sh
    args:
    - --hook-config=--path-to-file=README.md
    - --hook-config=--use-standard-markers=false
    - --hook-config=--create-file-if-not-exist=true
    - --hook-config=--add-to-existing-file=true
    language: system
    files: (\.tf)$

Add an `output.tf:
output "my_output" {
  value       = aws_api_gateway_stage.blah.invoke_url
  description = "Description of the test output"
}

Using the existing README.md as a target:

Test default behaviour has not changed:

pre-commit run -a should add the terraform docs between the existing markers
Delete the README. Then pre-commit run -a should create the file as normal
Delete the contents of the README. Then pre-commit run -a should add the terraform docs as normal

Test the behaviour with "standard" markers

update the pre-commit config to set --hook-config=--use_standard_markers=true
Delete the README. Then pre-commit run -a should create the file as normal, but with different markers
update using terraform-docs
there should be no changes made by terraform docs
repeat this process in reverse (terraform-docs, then pre-commit)

For completeness, repeat the standard markers test for insertion between existing markers, and appending to an existing file with no markers (or check by code inspection).

While code in hooks/terraform_docs.sh looks decent per se, the PR looks to be a mess of superfluous files local to contributor 🤔

Yep, it's still WIP.  Better now, but needs a little tidy up before review. I will hopefully finish this tomorrow

@MaxymVlasov this is ready for review now. Thank you

@yermulnik @MaxymVlasov thanks for the review. Please take another look

Leaving this to @MaxymVlasov to also review and merge if he's fine with the change.

This PR is included in version 1.86.0 :tada:

Thanks for a great contribution experience everyone!

I just found this:  https://github.com/terraform-docs/terraform-docs/#pre-commit-hook
Can you offer any comparison between the two @antonbabenko ?

Wow, didn't know that they have hooks.
As far as I see, https://github.com/terraform-docs/terraform-docs/#pre-commit-hook was added 2021-05-19 in 0.12.0, when terraform_docs hook exist from 2018-05-16 (1.7.0)
First of all, example in README
repos:
  - repo: https://github.com/terraform-docs/terraform-docs
    rev: "v0.17.0"
    hooks:
      - id: terraform-docs-go
        args: ["markdown", "table", "--output-file", "README.md", "./mymodule/path"]

just not works, if, ofc, you want to add
      - id: terraform-docs-go
        args: ["markdown", "table", "--output-file", "README.md", "./mymodule/path"]
      - id: terraform-docs-go
        args: ["markdown", "table", "--output-file", "README.md", "./mymodule/path2"]
     ....
      - id: terraform-docs-go
        args: ["markdown", "table", "--output-file", "README.md", "./mymodule/pathN"]

And I don't get how to make work
repos:
  - repo: https://github.com/terraform-docs/terraform-docs
    rev: "v0.17.0"
    hooks:
      - id: terraform-docs-go

with default .terraform-docs.yml (with formatter: "markdown") -

So far, I can say that terraform-docs/terraform-docs just not works in real-case scenarios


:feet: Enhance the DetailsItem component to enable both hyperlink and edit

Reference: https://github.com/kubev2v/forklift-console-plugin/issues/1106
Up till now the DetailsItem component supported 2 modes:

A non editable content (w/o hyperlink).
An editable content while both content and pencil icon are an inline button for triggering the onEdit callback.

This PR enhances the DetailsItem component to support a new third mode which is a combination of those existed modes: treating the content text as a hyperlink and treating the pencil icon as an inline button for triggering the onEdit callback.
Screencast for showing old and new modes:
Screencast from 2024-04-18 22-11-13.webm
We can consider adding a 'Edit' label next to the pencil icon.
only pencil icon

pencil icon with a label

Nice,
The idea is to be able to support external links in the "value" part of the item, makes sense to me.
my concern is that we will have two identical items that will behave differntly without visual indication, one will have "value"+"pencil" where the "value" will work as an "edit button", and in the other the "value" will work as "external link". Can you think of another way to have "value" as a "external link" while keeping only one behavior, maybe switching to only support the pencilOnlyButton mode   ?

Ref: https://github.com/kubev2v/forklift-console-plugin/issues/1124

Another suggestion for enhancement:
Why not adding the console's commonly used annotations field to the providers details? This will enable the user to easily add annotations  like for the external provider console link. E.g.

my concern is that we will have two identical items that will behave differntly without visual indication, one will have "value"+"pencil" where the "value" will work as an "edit button", and in the other the "value" will work as "external link". Can you think of another way to have "value" as a "external link" while keeping only one behavior, maybe switching to only support the pencilOnlyButton mode ?

The reason I used this "combined" solution is since other details pages used the same style. For example, check the Secret details page (Labels and Annotations are both editable fields, each label, if added, can be clicked as well):

Anyway, I agree that it's confusing and also suggested in main comment above to consider adding an 'Edit' label next to the pencil icon, as done for the Console's Labels field. Maybe it's even better to implement that for all editable fields in the provider details page. Something like this:
Screencast from 2024-04-21 21-58-45.webm
WDYT?

Another suggestion for enhancement:
Why not adding the console's commonly used Annotations field to the providers details? This will enable the user to easily add annotations like for the external provider console link

That is an excellent suggestion and in line with the product plan, we can add an enhancement feature and prioritize according to our capacity.

consider adding an 'Edit' label next to the pencil icon, as done for the Console's Labels field. Maybe it's even better to implement that for all editable fields in the provider details page

Adding Edit next to each Pencil icon, is another enhancement we can consider, my concern with adding Edit always is that is does not align with the current console design, usually in the console they add Pencil when it's a one-item view and the Pencil is placed next to the item the user will edit, and when the view is large, like we have in Labels and Annotations  they put the item in a box, and put Edit + Icon on the top right corner of this box.

consider adding an 'Edit' label next to the pencil icon, as done for the Console's Labels field. Maybe it's even better to implement that for all editable fields in the provider details page

Adding Edit next to each Pencil icon, is another enhancement we can consider, my concern with adding Edit always is that is does not align with the current console design, usually in the console they add Pencil when it's a one-item view and the Pencil is placed next to the item the user will edit, and when the view is large, like we have in Labels and Annotations  they put the item in a box, and put Edit + Icon on the top right corner of this box.

Yes I agree. In general, I think it's confusing for the user to use 2 kind of edit button styles within the same page.
I'll add only the pencil icon style and we can enhance it later.

Updated the main comment with a screencast.


I lost my master ball from jade monthly, about 25 ultraballs, 15 or so quick balls, 16 dusk balls, a bunch of pokeballs, 4 wishing pieces, my exp all, eff5 unb 3 pick, eff 4 shovel, 3 pcs, a ha lure, and a few other items I can't remember while typing this. Any chance I can get those items back ? Kinda missing the exp all the most

Apologies for the wait, a moderator+ will be refunding them soon. Please comment back if you can with your username and server you play on.

Issue was resolved by faty (I think) I got items refunded a while back ty tho


The great migration

@LTLA , trying to understand what the actual error is in the tests. This is for seqlengths
⋊> ~/D/p/a/dolomite-ranges on migration ⨯ python                                                                              (artidb) 15:05:46
Python 3.9.18 (main, Sep 11 2023, 13:41:44)
[GCC 11.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> path = "/tmp/tmpi8x676v7/seqinfo"
>>> import h5py
>>> f = h5py.File(path + "/info.h5", "r")
>>> f["sequence_information"]["length"]
<HDF5 dataset "length": shape (3,), type "<i4">
>>>


[BUG] Crash on Startup

Describe the bug
App crashes on startup.

Tried fresh installs, redownloaded, with league client open and closed. Always crashes with the same log.

Something with the async controller I believe, but I don't know enough C# yet to understand it fully.
To Reproduce

Launch LeageBroadcast.exe

Expected behavior
App starts up.
Crash Log
--- LeagueBroadcast Crash Report ---
Gotta fix this somehow... Up Up Down Down Left Right Left Right B A Enter... Nah, still broken

Time: 2021.07.24-11:29
Description: The type initializer for 'LeagueBroadcast.Common.Controllers.IngameController' threw an exception.

Stacktrace:
   at LeagueBroadcast.Common.Controllers.IngameController..ctor() in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Controllers\IngameController.cs:line 42
   at LeagueBroadcast.Common.Controllers.BroadcastController.Init() in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Controllers\BroadcastController.cs:line 116
   at LeagueBroadcast.Common.Controllers.BroadcastController.<EarlyInit>b__35_0(Object s, EventArgs e) in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Controllers\BroadcastController.cs:line 72
   at LeagueBroadcast.Common.Data.Provider.DataDragon.Init() in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Data\Provider\DataDragon.cs:line 155
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__139_1(Object state)
   at System.Threading.QueueUserWorkItemCallback.<>c.<.cctor>b__6_0(QueueUserWorkItemCallback quwi)
   at System.Threading.ExecutionContext.RunForThreadPoolUnsafe[TState](ExecutionContext executionContext, Action`1 callback, TState& state)
   at System.Threading.QueueUserWorkItemCallback.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

-- System Details --
League Broadcast Version: <IP_ADDRESS>02
OS: Win 10 2009

Additional context

It was working for me when Patch 11.15 just came out. Stopped working this morning.

Can you go to configs -> component and set LogLevel to 3, then post the normal log when this happens? It looks like it cannot find your League Install for some reason, though it should throw an error, not crash. Mind also sharing where your league is installed?

I see thats weird, my League install hasn't moved at all and it was working before.
It's installed at C:\Riot Games\League of Legends. I see that it's looking at an F: drive, but I don't even have one.
I set the log level to 3, here's the bottom half after it loads in the other config files:
[15-02-31] Config Component loaded
[15-02-31] Loading config watchers
[15-02-31] Watching PickBan.json for changes
[15-02-31] Watching Component.json for changes
[15-02-31] Watching Ingame.json for changes
[15-02-31] Log Level set to Verbose
[15-02-31] League Broadcast Version 1.4.55
[15-02-31] [Update] Checking for Updates
[15-02-31] [Update] Latest release version v1.4.55 vs. current version 1.4.55
[15-02-31] [Update] Local version up to date
[15-02-31] Using current Version
[15-02-31] Early Init Complete in 962.7647ms
[15-02-31] DataDragon Provider Init
[15-02-31] Getting latest versions from dataDragon
[15-02-31] Retrieving latest patch info
[15-02-31] Downloading https://ddragon.leagueoflegends.com/realms/euw.json
[15-02-31] https://ddragon.leagueoflegends.com/realms/euw.json downloaded
[15-02-31] Champion: 11.15.1, Item: 11.15.1, CDN: https://ddragon.leagueoflegends.com/cdn
[15-02-31] Downloading https://ddragon.leagueoflegends.com/cdn/11.15.1/data/en_US/champion.json
[15-02-32] https://ddragon.leagueoflegends.com/cdn/11.15.1/data/en_US/champion.json downloaded
[15-02-32] Loaded 156 champions
[15-02-32] Downloading https://ddragon.leagueoflegends.com/cdn/11.15.1/data/en_US/summoner.json
[15-02-32] https://ddragon.leagueoflegends.com/cdn/11.15.1/data/en_US/summoner.json downloaded
[15-02-32] Loaded 15 summoner spells
[15-02-32] Downloading https://ddragon.leagueoflegends.com/cdn/11.15.1/data/en_US/item.json
[15-02-32] https://ddragon.leagueoflegends.com/cdn/11.15.1/data/en_US/item.json downloaded
[15-02-32] Detected 222 items
[15-02-32] Loaded 117 full items
[15-02-32] Checking Local Cache
[15-02-32] Cache C:\Users\Brandon\Code\LeagueBroadcast-1.4.55/Cache/11.15.1 exists already
[15-02-32] DDragon loaded
[15-02-32] Loading PickBan Controller
[15-02-32] Loading Ingame Controller
[15-02-32] Checking League install for LiveEventsAPI
[15-02-32] Found League install location
[15-02-32] LiveEvents API found in Game config
[15-02-32] [WARNING] Could not parse game config. Replay API may not be enabled!
[15-02-32] Verifying LiveEvents list
[15-02-32] [WARNING] App Crash detected. Writing Crash report to CrashLogs\Crash-2021-07-24-15-02.log

Crash report says the exact same thing.

Ok. So it looks like its having trouble reading your game config. This shouldnt cause a crash, but I obviously overlooked something.
Can you try the following steps:

Close both LB and the client
Rename C:\Riot Games\League of Legends\Config\game.cfg to something like gamebackup.cfg or something of the sort.
Start the client
Start LB

This should reset your cfg and will probably reset some ingame settings, which is why you wanna keep the backup.
If this works, mind sharing your game.cfg? It would interest me what changed which causes this issue

Have to do something now, so I will update tonight on if that fixes it. But if you think its the game.cfg
I only made two manual modifications which were for broadcast:
I added: EnableReplayApi=1 under [General]
And I added [Spectator] eSportsNeutralTimers=1.
Here's the full file:
[General]
CursorOverride=1
EnableReplayApi=1
CfgVersion=11.15.388.2387
SystemMouseSpeed=0
WindowMode=2
Height=1080
Width=1920
WaitForVerticalSync=0
ShowGodray=1
MinimizeCameraMotion=0
BindSysKeys=0
HideEyeCandy=0
EnableLightFx=0
EnableCustomAnnouncer=1
AlwaysShowExtendedTooltip=0

[Spectator]
eSportsNeutralTimers=1

[Performance]
ShadowQuality=4
FrameCapType=2
EnvironmentQuality=4
EffectsQuality=4
CharacterQuality=4
EnableGrassSwaying=1
EnableFXAA=1
CharacterInking=1

[MapSkinOptions]
MapSkinOptionDisableWorlds=0
MapSkinOptionDisableURF=0
MapSkinOptionDisableStarGuardian=0
MapSkinOptionDisableSnowdown=0
MapSkinOptionDisableProject=0
MapSkinOptionDisablePopstar=0
MapSkinOptionDisablePoolParty=0
MapSkinOptionDisableOdyssey=0
MapSkinOptionDisableMSI=0
MapSkinOptionDisableLunarRevel=0
MapSkinOptionDisableArcade=0

[Replay]
EnableDirectedCamera=1

[Mobile]
LastTickerTime=
AppRegion=
SelectedQueue=0
OfferedTutorial=0
iOSMetalRendererEnabled=0

[Highlights]
VideoQuality=0
VideoFrameRate=60
ScaleVideo=720
AudioQuality=1

[Voice]
InputDevice=Default System Device
InputVolume=0.5000
ActivationSensitivity=0.6500
InputMode=0

[ItemShop]
BootsPanelPinned=1
ConsumablesPanelPinned=1
NativeOffsetY=-0.0167
NativeOffsetX=0.1120
CurrentTab=0

[HUD]
MinimapScale=0.8100
SmartCastOnKeyRelease=1
ShowFPSAndLatency=1

[ColorPalette]
ColorPalette=0

[Volume]
MasterVolume=0.1400

[LiveEvents]
Enable=1
Port=34243

By the way, thanks for a quick response, this is an absolute game-changer for my broadcasts and I will be looking into this project more to see where I can help out 😄

Just reset my game.cfg, the issue still persists.

The next step would be to delete LiveEvents.ini from the same folder and see if that helps, though I doubt it.
What is the file type and line ending type for game.cfg and LiveEvents.ini? Check to make sure they are UTF-8 and have Windows (CR LF) line endings, Notepad++ or similar should tell you. Other formats should in theory work, though I havent tested that yet.
You can also try again using the newest build. It should have more logging so I can better tell whats going wrong

The newest build works so far thanks! I will update if the issue persists again after a day. The last one randomly popped up after a day.

@floh22 Hello, League Broadcast is also crashing for me after opening League Client. I have a different error though:
--- LeagueBroadcast Crash Report ---
Everything went according to plan. Seriously, this is the best case scenario.

Time: 2021.07.31-09:01
Description: Object reference not set to an instance of an object.

Stacktrace:
   at LeagueBroadcast.Common.Controllers.AppStateController.GetLocalGameVersion(String rawPatch) in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Controllers\AppStateController.cs:line 240
   at LeagueBroadcast.Common.Controllers.AppStateController.ConnectToClient() in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Controllers\AppStateController.cs:line 138
   at LeagueBroadcast.Common.Controllers.AppStateController.InitConnection() in F:\Repos\LeagueOfLegends\LeagueBroadcast\LeagueBroadcast\Common\Controllers\AppStateController.cs:line 107
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__139_1(Object state)
   at System.Threading.QueueUserWorkItemCallback.<>c.<.cctor>b__6_0(QueueUserWorkItemCallback quwi)
   at System.Threading.ExecutionContext.RunForThreadPoolUnsafe[TState](ExecutionContext executionContext, Action`1 callback, TState& state)
   at System.Threading.QueueUserWorkItemCallback.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

-- System Details --
League Broadcast Version: <IP_ADDRESS>07
OS: Win 10 2009

This seems to be an unrelated issue. Please open a separate issue.
Though im guessing your issue should probably be resolved with a PC restart. If its not the case, open a separate issue

Closed due to inactivity and issue being seemingly resolved


Ensemble placeholder

[ ] Update ensemble drilldowns to new designs
[ ] Remove tera-simulation-chart component, it is outdated and only used in the (old) ensembles

This will be rewritten


Automatically inject the identityId while using azure-workload provider

Proposal
I am using the same setup as @chrismilson posted on https://github.com/kedacore/keda/issues/5451. However, I want to override the identityId (Azure UMI clientId) without hardcoding it.
I couldn't find a way to do something like
apiVersion: keda.sh/v1alpha1
kind: TriggerAuthentication
metadata:
  name: pipeline-trigger-auth-workload
spec:
  podIdentity:
    provider: azure-workload
    identityIdFrom:
      configMapKeyRef:
        key: clientId
        name: "auto-generated-cm-identity-settings"
# OR
    identityIdFromEnv: CONTAINER/AZURE_CLIENT_ID

Use-Case
I'm using TriggerAuthentication and azure-workload. However, currently I have to either manually/hardcode the identityId, share permissions with the main keda UMI or use a PAT.
I couldn't find a way to automatically inject the UMI's clientId. In my case, the UMI is generated automatically using the Azure Service Operator, so I don't know the clientId in advanced. I inject the clientId to the pod using an automatically generated configMap:
env:
- name: AZURE_CLIENT_ID
  valueFrom:
    configMapKeyRef:
      key: clientId
      name: "auto-generated-cm-identity-settings"

Is this a feature you are interested in implementing yourself?
No
Anything else?
Depending on how complex the implementation is, I might be able to do it myself. First, I'd like to hear your opinion regarding this issue and maybe you would have a better/already existing solution for me.
Thank you!

This sounds reasonable, maybe using a Secret instead of ConfigMap?
@JorTurFer WDYT?

I think that this can be useful tbh. I'm not 100% sure about how to implement this, if reading a secret/config directly or an env var of the pod 🤔
The "standard" way to configure this is via service account annotation and for example for AWS auth, we support reading the service account to get the pod annotation and recovering the rolearn (clientID). Are you annotating somehow the service account 	@Shaked or you directly push all the envs into the pod? I mean, AZURE_CLIENT_ID is only one of the required envs

@JorTurFer the azure service operator creates the config maps which are later mounted to the relevant pods respectively and used via the AZURE_CLIENT_ID env var

I guess that we can support it, Definitively all the UX improvements related with this topic is always nice. Maybe we can include something like valueFrom

Keeping this feature request alive


Asynchronous Subscription Begins to Fail After Some Time

Hello! I have a long-running script that is continuously pulling messages from a pubsublite topic asynchronously and executing them as they come in. This works fine for a while, but eventually runs into an AioRpcError due to receiving a FlowControlRequest that that tried to remove messages. Once that happens, the subscription continues to fail indefinitely. See below.
Environment details

OS type and version: Linux
Python version: 3.9.7
pip version: 21.2.4
google-cloud-pubsublite version: 1.2.0

Code example
import asyncio
from google.cloud.pubsublite.cloudpubsub import AsyncSubscriberClient
from google.cloud.pubsublite.types import (
    CloudRegion,
    CloudZone,
    FlowControlSettings,
    SubscriptionPath,
)
from google.cloud.pubsub_v1.types import PubsubMessage

per_partition_flow_control_settings = FlowControlSettings(
    messages_outstanding=20,
    # 10 MiB. Must be greater than the allowed size of the largest message (1 MiB).
    bytes_outstanding=10 * 1024 * 1024,
)

async def process_message():
    ...

async def main():
  async with AsyncSubscriberClient() as subscriber_client:
      streaming_pull_future = await subscriber_client.subscribe(
          subscription_path,
          per_partition_flow_control_settings=per_partition_flow_control_settings,
      )
      async for message in streaming_pull_future:
          process_message_coro = process_message(message)
          asyncio.create_task(process_message_coro)

if __name__ == "__main__":
    asyncio.run(main())

Stack trace
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/google/api_core/grpc_helpers_async.py", line 105, in _wrapped_aiter
    async for response in self._call:  # pragma: no branch
  File "/usr/local/lib/python3.9/site-packages/grpc/aio/_call.py", line 326, in _fetch_stream_responses
  await self._raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/grpc/aio/_call.py", line 236, in _raise_for_status
  raise _create_rpc_error(await self.initial_metadata(), await
  grpc.aio._call.AioRpcError: <AioRpcError of RPC that terminated with:
    status = StatusCode.INVALID_ARGUMENT
    details = "Received FlowControlRequest that tried to remove messages"
    debug_error_string = "{"created":"@1643639089.487575412","description":"Error received from peer ipv4:<IP_ADDRESS>:443","file":"src/core/lib/surface/call.cc","file_line":1074,"grpc_message":"Received FlowControlRequest that tried to remove messages","grpc_status":3}"

Rest of Stack Trace

Hello,
I'm sorry you're having this issue. My guess is that you are hitting an edge case where you are somehow making the flow control count go negative. What I would suggest you do is disable byte-based flow control: you already have a very small message count limit.
This would involve the following change:
per_partition_flow_control_settings = copy.copy(DISABLED_FLOW_CONTROL)
per_partition_flow_control_settings.messages_outstanding = 20
I will also fix the client to not send an invalid request when this happens.
-Daniel


Enabling timeout based usage of docker client API during startup

Using timout based context while using docker client apis on plugin side to
popluate the existing volume data

upgrading the docker client api version to 1.25

Removing environment variable to enable the volume discovery

Testing:

Restarting docker
Create volume test1
Attach it to three containers and keep containers running in -d mode
Following is the refcount from the logs

2017-03-27 16:24:19.548291542 -0700 PDT [INFO] Mounting volume name=test1
2017-03-27 16:24:19.548342906 -0700 PDT [INFO] Already mounted, skipping mount. refcount=3 name=test1

restart docker using
service docker restart
Timeout based context prevents deadlock
2017-03-27 16:30:24.331275264 -0700 PDT [INFO] Plugin options - port=1019
2017-03-27 16:30:24.331309301 -0700 PDT [INFO] Getting volume data from unix:///var/run/docker.sock
2017-03-27 16:30:26.332302249 -0700 PDT [INFO] Can't connect to unix:///var/run/docker.sock due to (An error occurred trying to connect: context deadline exceeded), skipping discovery
2017-03-27 16:30:26.332612952 -0700 PDT [INFO] Docker VMDK plugin started version="vSphere Volume Driver v0.4" port=1019 mock_esx=false

docker engine initializes properly and volume created can be seen

Restarting docker-volume-vsphere managed plugin

Create volume test1
Attach it to three containers and keep containers running in -d mode
Following is the refcount from the logs
2017-03-27 16:34:25.864897963 -0700 PDT [INFO] Mounting volume name=test1
2017-03-27 16:34:25.864943268 -0700 PDT [INFO] Already mounted, skipping mount. refcount=3 name=test1

kill plugin using docker-runc
plugin is restarted by docker
discovery complete
2017-03-27 17:03:36.418383937 -0700 PDT [INFO] Starting plugin driver=vsphere log_level=info config="/etc/docker-volume-vsphere.conf"
2017-03-27 17:03:36.418642322 -0700 PDT [INFO] Plugin options - port=1019
2017-03-27 17:03:36.418855025 -0700 PDT [INFO] Getting volume data from unix:///var/run/docker.sock
2017-03-27 17:03:36.434455603 -0700 PDT [INFO] Discovered 1 volumes in use.
2017-03-27 17:03:36.434493635 -0700 PDT [INFO] Volume name=test1 count=3 mounted=true device='/dev/disk/by-path/pci-0000:03:00.0-scsi-0:0:0:0'
2017-03-27 17:03:36.434502803 -0700 PDT [INFO] Docker VMDK plugin started version="vSphere Volume Driver v0.4" port=1019 mock_esx=false
2017-03-27 17:03:36.434768084 -0700 PDT [INFO] Going into ServeUnix - Listening on Unix socket address="/run/docker/plugins/vsphere.sock"

Fixes #1050

Per issue 31903 on docker the plugin restore will not be able to use the Docker API (yes?). In which case this change will not work as the docker API isn't available?
Believe a simpler approach is to use a refcount-restore task to run the "post init" for the plugin (essentially restoring any ref counts for volumes) and skip doing any ref count resotre during plugin startup itself. The plugin initialization starts the task and completes initialization and let Docker continue on its way.
The refcount-retore task attempts to connect and use Docker API within some number of attempts with an "exponential backoff" and ultimately restores or gives up on the refcounts.
The plugin can then service volume requests based on what Docker is asking while the refcount restore goes on. As long as refcount-restore task is running, the plugin will skip doing unmounts. The refcount-restore task checks all mounted volumes and those that aren't having any refcounts will be unmounted by it. While those volumes in use have their refcounts restored.

BTW, you also need to enable Tests

@govint thanks for pointing this.
Essentially, this PR will not let plugin enter into a deadlock where plugin hangs on docker api and docker waits to talk to plugin.
If docker doesn't respond to api calls, we timeout and mark ref counts to be zero.
I get your point of docker being up but could be busy trying to talk to plugin with exponential backoff (docker might do this due to some volume command it receives in the middle when plugin is initializing).And we timeout on our docker client api call and give up on refcounts.
Agree that a safer way to work with refcounts would be to calculate them post init of the plugin
But I think that change can be done separately.
CC @pdhamdhere @msterin

@pdhamdhere addressed your comments.

Addressed the changes requested in latest commit.


Add order parameter

This PR adds order parameter to browse endpoints.
Closes #5602.
Supplied input string (e.g. title desc) is converted to the same format as produced by orderDefaultOptions():
{
    title: 'DESC'
}

Example
PostAPI.browse({ order: 'title desc, updated_at asc' });

Upper-cased DESC and ASC are also allowed. Spaces after commas are not required, this input is also valid: title desc, updated_at asc.
Note: If order parameter contains attributes, that do not belong to the current model, they will be ignored: bunny desc, title asc (bunny is skipped).
Todo

[ ] test fields with dots in them, e.g. posts.title desc
[ ] tests for other models (not sure this is needed)
[ ] decide, if custom order option should completely overwrite or merge with default order options (orderDefaultOptions())

looks good :+1:, @ErisDS does ordering interfere with filtering?

Rebased latest changes, now ready to merge.

This doesn't interfere with filtering - it just needs to play nicely with it :) Using the model name prefix as mentioned here would help with that I think.
This is all looking great so far, it might be worth adding a test to show that you can order posts by author.name?

Sure, will update ;)

So, while implementing support for ordering by rows from outside tables (when using join), I discovered an issue with the way Bookshelf fetches records (or how Ghost uses Bookshelf). It does not use join, but uses extra select query to fetch additional rows from other tables and then merges them into the final result.
For example, I want to fetch posts and include author information in them:
PostAPI.browse({include: 'author'});

I'd expect a select query with a join statement. But here's what happens instead:

It finds posts, then looks for author_id and finds that there are 3 unique author_ids. After that, it executes one more select with those ids. When that extra query is finished, author results get merged into posts result (into author column).
Summary: Because of this issue, it is not possible to order by "joined" values, e.g. author.name.

@ErisDS If we somehow make it use actual join statements, do you think any possible damage could be done?

We are going to merge this PR as it is, without support for ordering by joined columns. Work on that problem will continue in a separate issue/PR (will be linked here).

Hey @vdemedes I have been trying to test this using the {{#get}} helper.
It is working nicely so far for posts but if I fetch users or tags then the ordering does not seem to be working.
Example
{{#get "tags" limit="8" page="1" order="slug asc" as |tags|}}
    {{#foreach tags}}
        <span>{{slug}}</span>
    {{/foreach}}
{{/get}}

Would you expect that work/have I missed something?

Hey @cobbspur, that's my oversight, thanks for pointing this out!

@cobbspur I added ordering to Tag and User models ;) See #6019. I had to re-submit, because I removed my original fork.


Extend punctuation marks

Pre-separate text by punctuation, tab and line break characters prior to tokenization.


End-Of-Song Dialogue

Hi again, I've got a custom stage and a beginning dialogue for a song set up currently. I also want my song to have an ending dialogue, but I've poked around the wiki and haven't found anything. How do I do this (on the download build)?
And also I apologize if this actually on the wiki and I'm just blind.

Same as the example dialogue script, but instead of it blocking onStartCountdown(), it should block onEndSong().

Thanks


Issue using latest 12.0.0 gem

Hi,
I tried to update the gem to the latest version but face a weird issue:
client
=> #<Google::Ads::GoogleAds::GoogleAdsClient:0x00007fa1a6c9c940 ...

client.service.google_ads
Traceback (most recent call last):
        1: from (irb):9
LoadError (cannot load such file -- /Users/xfx/github/release-sanity-check/lib/google/ads/google_ads/v8/services/google_ads_service)

I checked examples (https://developers.google.com/google-ads/api/docs/samples/get-keywords) and it seems to still be the valid syntax.
Worked fine with all previous versions, gem update is the only change.
Thanks in advance!

Hi Benjamin, I'll take a look at the issue. Would you let me know the step to reproduce this? e.g. what code example did you run

Hi @xflesym I just built a client, then tried client.service.google_ads. Not much really

I can reproduce the same issue on my side as well. Will investigate and fix ASAP.

I've pushed a new gem version 12.0.1. Could you gem update again and see if the issue is fixed?

so yeah this one is solved but there is something else:
# using 12.0.1
client.resource.target_spend
=> <Google::Ads::GoogleAds::V8::Resources::AccessibleBiddingStrategy::TargetSpend: >

# using 11.0.0
client.resource.target_spend
<Google::Ads::GoogleAds::V7::Common::TargetSpend: >

which then leads to crashes as I cannot assign a Google::Ads::GoogleAds::V8::Resources::AccessibleBiddingStrategy::TargetSpend to a bid strategy

This is because now we have two TargetSpend types under AccessibleBiddingStrategy and Common, which leads to a naming conflict using the client.resource.type syntax.
The workaround is to specify the full path to the type you need, e.g.
campaign.target_cpa = Google::Ads::GoogleAds::V8::Common::TargetCpa.new
campaign.target_cpa.target_cpa_micros = 5_000_000

We have issue #316 filed to address the name conflict issue. Please use the above workaround in the meanwhile.

ok thanks!


organization_description: Add placeholder for organization description.

Fixes #26947

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@afeefuddin Please complete the self-review checklist and post a comment when ready.

Please add a screenshot for an admin view.

@alya Done!!

@sahil839 would you be up for reviewing this one? I have not tested.

I am not sure whether having a input placeholder is a good idea. We use placeholder to indicate that you can write such text in the input, but in this we actually show the description as "The coolest place in ..." in the signup page.

Hey @sahil839 , As mentioned in the issue it was asked to add a placeholder in organization description. Would love to hear a better approach to do this, if exists .

Oh, right, I think what I had in mind in the issue was actually having the text in that field.

Hey @alya , it seems few tests are failing because I have set a default organization and those test checks default organization as empty. If this is okay we will have to rewrite tests . Can you please tell me what can I do now?

I have updated the issue; if tests are failing when you implement it, you'll need to update the tests,
'

Oh, right, what I had in mind in the issue was actually having the text in that field.

Hey @alya, is this issue still open, I would love to work on it.
Can you please specify what text do you need in that field ?

@amit-077  This is a PR, not an issue.


Ready event not triggered at the right time

First off: what a nice tool this is.
I do have a little problem with events though. It looks like the ready event not triggers like it says on the site:  "When audio is loaded, decoded and the waveform drawn." It triggers after I start play playback. Maybe I'm doing something wrong here, could be. I'm using it inside of AngularJS

Can you reproduce it with the AngularJS example?

@pkeuter the example Thijs is referring to is this: https://github.com/katspaugh/wavesurfer.js/blob/master/example/angular/app.js
Make sure your AngularJS directive isn't turned off with ng-if until you press the play button.

I can reproduce it with the example. http://plnkr.co/edit/LRKeAglgUxwHBLzW1veN?p=preview
As you can see, the text only dissapears when you press play.

I'm using MediaElement mode (without autoplay) and loading an hour-long MP3 which takes a little while. The event fires for me as soon as the player instantiates (before I press play). Again this is not quite how the event is described because it takes 10-15s before the waveform is drawn.
I was hoping to be able to implement a spinner or "Generating waveform..." message during that time which I could remove when the "ready" event fired.

@liamdv4 thanks, looks like a bug.

I think it would be ideal to leave the ready event as it is (since the player is, indeed, ready to play at that point), but add a new event for when the waveform has been drawn. Thanks.

Yes, makes sense.

+1 for new event. I had this problem with "ready" as well, but have since used it to know when the player is ready as suggested.

@pkeuter check if this #565 works for you?

Solved in ##856.


Quickly execute code action by index

When the Code Action window is open, pressing keys 1-9 will immediately execute the action corresponding to that number.
This retains the existing behaviour of moving the cursor to a line then pressing <cr>.
There's probably a way to auto-generate the functions or something, but there's only 9 of them, so I figured why not hard code them ¯\_(ツ)_/¯

Hey @jesstelford , I've been meaning to add these feature. but somehow got side tracked and never did.
😆 indeed why hard code them.
I was going to do something like this before
  for i = 1, 9 do
    vim.cmd(("noremap <silent><buffer> %s :lua require('lspsaga.codeaction.window').execute(%s)<cr>"):format(i, i))
  end


How to change PUID and PGID ?

Hi,
how do i  pass the environment variables to change the UID and GID from 12345 to 1000 ?
There are two environment varialbes for the base image which should allow to change the UID and GID in /etc/passwd and /etc/group... However, i am passing both variable for user insync and group insync but it does not change the UID or GID in the image.
Would be great if i can get some help. Other then this issue the sync seems to work fine.
Here is my docker-compose configuration :
insync-app:
image: tiredofit/insync
container_name: insync-app
networks:
- vlan99
environment:
- ZABBIX_HOSTNAME=insync-app
- INSYNC1_USERNAME=xxxx
- INSYNC1_AUTH_CODE=xxxx
- INSYNC_DOWNLOAD=ms-office
- DEBUG_PERMISSIONS=TRUE
- USER_INSYNC=1000
- GROUP_INSYNC=1000
- TIMEZONE=${TZ}
volumes:
  - ${DOCKERDIR}/insync:/data
restart: always

thx!

Try ENABLE_PERMISSIONS=TRUE - looks like for some reason that's disappeared out of the README. I will add it back in.

I am added "ENABLE_PERMISSIONS=TRUE" and it works fine for the user. However the group insync still has ID 12345.
Those are the corresponsing entries in the container :
/etc/passwd -> insync:x:1000:12345:Insync,,,:/home/insync:/bin/bash
/etc/group -> insync:x:12345:"${varenvuser2add[$i]}"
btw, is there also a way to set default permissions to e.g. 770 ?
thank you!

I'll have to look into the code here as to why the group isn't working - there's definitely a problem there it sending that environment variable over. I won't be able to look into this for a couple of days, but I can implement a custom script functionality that will let you execute something before insync starts mounted from an external volume. I can do that one sooner than the permissions debugging.

I just tagged tiredofit/insync:2.3.1 which allows you to mount a volume into /assets/custom-scripts/ that will execute any .sh scripts in that folder.
For example you could have two scripts:
01-rename_permissions.sh and do a dirty permission change on the /etc/passwd and group files while I figure out why that function has stopped working on debian.
02-set_folder_permissions.sh chmod 770 and any other file operations you wanted to make on your /data folder.
Hope this helps in the short term.

No worries, take you time i am not in rush. For the moment it's total fine, for the long run it would be nice to tighten the access to my google drive data.
Thank you!

ok just see your last email... i will try the work around. Thank you so much for your effort!

No problem! Just make sure you set the scripts to be executable.

Same issue here - updated recently, and copied in some files from my PC share, and insync created the GDrive folders, but didn't copy the files.
Restarted it, the files copied, but then I couldn't copy them back as they were owned 12345:12345
Added the ENABLE_PERMISSIONS=TRUE and the UID change is working, just the GID stuck per the above.
I'm still on insync 1, as although I'd like to remove the security warning in Google, I'm not moving to a sub model with insync :)

Same issue here - updated recently, and copied in some files from my PC share, and insync created the GDrive folders, but didn't copy the files.
Restarted it, the files copied, but then I couldn't copy them back as they were owned 12345:12345
Added the ENABLE_PERMISSIONS=TRUE and the UID change is working, just the GID stuck per the above.
I'm still on insync 1, as although I'd like to remove the security warning in Google, I'm not moving to a sub model with insync :)

Can you try pulling tiredofit/insync:groupfix and see if this solves your group issues?

Can you try pulling tiredofit/insync:groupfix and see if this solves your group issues?

That appears to be working - I've got my intended UID and GID showing now after a pull/restart :)

That appears to be working - I've got my intended UID and GID showing now after a pull/restart :)

OK perfect. The problem resides in the upstream debian base image i wrote. What I'm going to do is fix it inside of that image and let it trickle across to all other dependent Debian images. Usually takes 2-3 days for Docker Hub to perform a build of all images. After that you can head back to :latest :)

OK perfect. The problem resides in the upstream debian base image i wrote. What I'm going to do is fix it inside of that image and let it trickle across to all other dependent Debian images. Usually takes 2-3 days for Docker Hub to perform a build of all images. After that you can head back to :latest :)


Can quicktype keep the origin json name for generate codes

{   "m_bTestBLEStatus" : true }
Can I generate c++ codes as
struct Welcome {   bool m_bTestBLEStatus; };
not, "mBTestBLEStatus", "m_b_test_ble_status", "MBTestBleStatus", "mBTestBleStatus", "M_B_TEST_BLE_STATUS", "MBTestBLEStatus", "mBTestBLEStatus".

You could add an option to do this. Closing, as it is a feature request.


Feature Request | C sharp support?

how complicated will it be to add support for C#? or any other language for this matter (like GO)?
if I wanted to add those myself as a PR, what steps would I need to take?

The main work to support a new language is parsing the error message to get filenames / line numbers so that the source can be read and transmitted. As of right now, this is happening with regular expressions to match filenames.
https://github.com/plasma-umass/cwhy/blob/3cea72ea3b698d061fbe25fadd27761ea33588c0/src/cwhy/cwhy.py#L206-L207
https://github.com/plasma-umass/cwhy/blob/3cea72ea3b698d061fbe25fadd27761ea33588c0/src/cwhy/cwhy.py#L210-L211
https://github.com/plasma-umass/cwhy/blob/3cea72ea3b698d061fbe25fadd27761ea33588c0/src/cwhy/cwhy.py#L214-L215
One of the existing regexes may already be matching filenames in C# diagnostic messages.
If not, the best way to add C# or another language as of right now:

Have a couple sample minimal C# files generating error messages (they can live in tests).
If filenames are not matched already, add a new regex in explain_context.
Verify the regex works, but most importantly that it doesn't mess with existing things by matching additional parts of a C++ clang diagnostic for example. This is typically signalled by "file not found" warnings.

If we keep adding support for more languages maybe we should 1. have a per-language mode, attempt to auto-detect the language, and make it a CLI option 2. have more robust per-language filename matching (or get a machine parseable version if one exists?).
I'm not very familiar with C# personally, so I don't really have any sample code with common or less common error messages at hand.

@BenjaminShinar Thanks for the question/suggestions! cwhy now supports C# and Go. Can you please give it a try? (python3 -m pip install -U git+https://github.com/plasma-umass/cwhy) Assuming everything works, we will push a new release out incorporating these changes.

We have pushed this out as a release. Give it a try!


Improper image rendering in the alaska form

The logo renders wrong on pare 1 of appk_sec3.pdf

Adopted a new jpeg reader that handles YCCK bitmaps correctly. in 8e23f3c825a0d10b7bd3525d89d1ae9bec8c9ebf


Build Admin Automation Process -- For review

Description
Proposed automation for Build "admin" centric tasks including:

creating new repos
add new users to the rubrikinc organization
granting users permissions to specific repositories and teams

@rubrikinc/octo, @rggoss, and @dterei
Can you review this PR and let us know if you have any feedback. This is our proposal for utilizing Slack to automate the various admin related requests related to the rubrikinc GitHub organization.


Test mode

One thing that I plan to add to the sanic-test module is the ability for Sanic app to be aware of whether or not it is being tested. This adds a variable to the Sanic app that is set by the internal test clients.

What's up with the failing tests, is it related to this one?

Ummm.... this is super weird. At one point I had added a check here to check against test_mode and not throw that warning. Which would make the test that failed fail. We would have needed to change it. But... I decided against that since if someone wants to suppress that warning it would be simple enough, and it might be good for them to see it in their test suite and ask themselves why that is.
But, none of that happened. I left the test the way it is.
I am going to resubmit the test suite.

Ignore the ridiculous number of commits. I was going too fast.

Ready for review and merge.


Expose application REST resources

It may be useful to expose available application REST endpoints. bootique-swagger helps with API exposure to the client, but sometimes all we may need is just listing the resources in the app log, or building a simple web console listing high-level app capabilities. Or tying resources to metrics (how many requests per resources, response time statistics, etc.)
We still need to think on a better focus and use cases for this feature. But the algorithm for collecting a list of resources is pretty simple as mentioned in this forum post.

In fact we already have a ResourceModelDebugger, so if you set `log.loggers.io.bootique.jersey.level" to "debug", it will output some info. We just need to work on the output to be more human readable.


Implemented OrderedConsumerVerticle

Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>Part of #589

Proposed Changes

Implemented OrderedConsumerVerticle
Refactored dev.knative.eventing.kafka.broker.dispatcher.consumer.impl tests to share test code

/retest

/lgtm
/approve

/retest

Whoops, there are conflicts.

Control plane / API stuff in another PR?

Sure, I want to do the wire up in another PR.

Can you open an issue to track control plane and API changes?

@pierDipi i'll add to the original issue


added pdf2video converter

Description
Convert a PDF to Video
Fixes #1036
Type of change
New feature (non-breaking change which adds functionality)
Checklist:

[x] - My code follows the style guidelines(Clean Code) of this project I have performed a self-review of my own code
[x] -  I have created a helpful and easy to understand README.md
[x] -  My documentation follows Template for README.md

Category:

[x] PDF

Title: Pdf2Video Converter
Folder: \pdf2Video_Converter
Requirements: one pdf file and one video file
Script: script.py
Arguments: name_of_pdf_file_with_ext and name_of_mp4_video
Contributor: Sukriti Sood
Reference PR 1064

@kaustubhgupta Please guide me about wrong PR template , I am unable to interpret it.

@kaustubhgupta Please guide me about wrong PR template , I am unable to interpret it.

Basically, you need to fill in the project metadata without removing any template material (except the text marked in <>). See this example:

Your template should look like this.

@kaustubhgupta Please guide me about wrong PR template , I am unable to interpret it.

Basically, you need to fill in the project metadata without removing any template material (except the text marked in <>). See this example:

Your template should look like this.

the text above this should remain the same as you have done currently, just don't mess with the project meta data heading and other things ahead


Nothing is the Modal box is selectable

Go to https://wellcomecollection.org/search/images
Click on an image or the "All filters" button
Try to select anything; it doesn't work

We should try to fix this, but couldn't figure out quickly what was going on, so creating a ticket.

Ohh interesting, you use Firefox right, and PC?
It doesn't work in Safari and Chrome (for me anyway), but does on FF - all Mac. Does it for you?

I use Chrome on a PC.
Also checked and I can select images and filter results with Edge on a PC and with Firefox on a PC, too.

Alright! Then it'd be good if someone else on a mac could test and see if it's just me 👀

I can select filters, checkboxes and colours work (edited)

View expended image works as well when I click on an image (using Chrome)

All the above fine on Safari too

Source: Slack
Looks like it might just be you @rcantin-w ?

Works for me, Safari, Chrome and Firefox on Mac.  It is a little slow. There is a very noticeable pause between the click and the tick.

Thanks all! Will close as it's not replicable.


Fixed non-working Docker command in README.md

As the comment at https://hub.docker.com/r/nilqed/fricas_jupyter/
notes, the Docker command that's written produces an inaccessible
page.  Adding --ip=* to Jupyter's options fixes this.

Merged ;)
Thank you.
On 16 December 2016 at 15:33, Chris Hodapp<EMAIL_ADDRESS>wrote:

As the comment at https://hub.docker.com/r/nilqed/fricas_jupyter/
notes, the Docker command that's written produces an inaccessible
page. Adding --ip=* to Jupyter's options fixes this.
You can view, comment on, or merge this pull request online at:
https://github.com/nilqed/fricas_jupyter/pull/1
Commit Summary

Fixed non-working Docker command in README.md

File Changes

M README.md
https://github.com/nilqed/fricas_jupyter/pull/1/files#diff-0 (2)

Patch Links:

https://github.com/nilqed/fricas_jupyter/pull/1.patch
https://github.com/nilqed/fricas_jupyter/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nilqed/fricas_jupyter/pull/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcVYhy843Ui_Vmqpyt6R4jCG22kuW-Nks5rIqFNgaJpZM4LPPgZ
.


How to use "USE KEYS" in inner select with QueryBuilder in Java?

Hi there,
Not sure if it is an issue, but i cannot find a way around this:
I have a Task document which has a list of UserIds:
Task
[
  {
    "description": "Description for task Task-21",
    "name": "Task-21",
    "userIds": [
      "1946da8f-bcd9-4a26-a877-397bc9056b3b",
      "84864453-00c6-4746-9945-dd2e0d12b11d",
      "ec83e97d-9532-4413-a92c-f399f3fc5eb2",
      "6de7058f-bf9c-4716-952a-631ff1f47b58"
    ],
    "type": "Task"
  }
]

User
[
  {
    "name": "User1",
    "type": "User"
  }
]

I use the following query in Couchbase Query Editor:
select task.*, (select users.* from bucket users USE KEYS task.userIds) as users
from bucket task
where task.type = "Task"

Which will give me the Task with the array of User objects instead of the IDs:
[
  {
    "description": "Description for task Task-21",
    "name": "Task-21",
    "Users": [
      {
        "name": "User1",
        "type": "User"
      },
      {
        "name": "User2",
        "type": "User"
      },
      {
        "name": "User3",
        "type": "User"
      },
      {
        "name": "User4",
        "type": "User"
      },
    ],
    "type": "Task"
  }
]

How would one convert this into the QueryBuilder.select statement in Java (Android)
Hope someone knows the answer,
Thanks
Denis

Can you please ask this question in our forums? GitHub is for filing product bugs.
Thanks


Registration - clarify username/email situation

This came out of looking at #38 more carefully. Not sure why I didn't spot it before, apologies for that.
Ideally, I think we should just have email; fewer things are better (although in Django this is slightly more work). Usernames are nice vanity things, but TYM is a private service, not a social one, and it's just something else to forget.

Totally agree.

Merged to new-design


MINOR: reduce verbosity of cache flushes

This log message tends to be extremely verbose when state stores are being restored

LGTM

LGTM! Merged to trunk and 0.10.2. Thanks @xvrl


Your sponsor information is copied when using the template

Example: https://github.com/matthewmcgarvey/asdf-test
Maybe this is intentional, but if not is there a way to avoid that being in the resulting repo?

Thanks, Matthew! I hadn't noticed this in my own use, and I'm 99% sure that GitHub doesn't have a way to omit specific files from being copied.
We could remove it in the setup script, but I'm also fine with removing it from the repo altogether.


excel will be reopened, how can i close & save ,after write - pending details

code：
[test.xlsx]Sheet1!A1 = [['中文', 'Test2', 9], [4, 5, 6]]
[test.xlsx]Sheet1!A2 = [['中文', 'Test', 9], [4, 5, 6]]
wait 3

Hi @cenwenchu can you share more on what do you mean by excel will be reopened?

Also, are you on windows or macOS?
I tried running your code and am able to write to excel which saves the value and closes the application after the run.

@ruthtxh thanks,i run program on macOS, and code only as below:
[test.xlsx]Sheet1!A1 = [['中文', 'Test2', 9], [4, 5, 6]]
[test.xlsx]Sheet1!A2 = [['中文', 'Test', 9], [4, 5, 6]]
wait 3
run first line , will open chrome and open excel,
run second line, will info me ,it will reopen excel (but first line excel not save), so will alert "excel not save, reopen will lose content".
so i don't know how to open file  and save file , "[test.xlsx]Sheet1!A2 = [['中文', 'Test', 9], [4, 5, 6]]" as auto open or reopen excel file, and  no save and append open file command line.

@cenwenchu what version of TagUI are you running on?
I tested your code on TagUI version 6.113 and it is able to programmatically open, save and close the file with the data written to it. Hence, I am unable to replicate your issue.
If you're not running on the latest version of TagUI, you can do tagui update in your terminal to update. If error still persists, you may look into the following:

Check your folder location of the excel file - which may deny read/write permission
Check excel settings which may be causing the above behaviour

Closing but please ping back if this is still relevant or an issue.