Allow non-deterministic next states

At present, the Step function allows non deterministic outputs, but the next state must be determined based on current state, input, and output. (Being able to specify the output allows some amount of non-determinism but still limited)
There are some use cases where the next states could be different even if there are no differences in the output. For example: with paxos with 3 nodes, after a Put, it doesn't matter whether 3 nodes agreed or only 2 modes agreed. The system could be in either state even in a sequential system. (This could be tested differently, but for some usecase, being able to represent the non determinism is helpful. )
Is it possible to allow non-deterministic next states? I couldn't fully understand the algorithm, but from my understanding it allows non determinism on the states and it's currently limited by the API.
Proposal: instead of Step function returning a single next state, if the function could have a slice of possible next states, it would work.
Please let me know on the feasiblity.

Etcd implements non determinism in model as multiple alternative states that fork for equivalent states and collapse. See implementation in https://github.com/etcd-io/etcd/blob/main/tests/robustness/model/non_deterministic.go

There are some use cases where the next states could be different even if there are no differences in the output. For example: with paxos with 3 nodes, after a Put, it doesn't matter whether 3 nodes agreed or only 2 modes agreed. The system could be in either state even in a sequential system.

For the implementation state, yes; but why is this information part of the state of a sequential specification? At a higher level: what is the spec of the system you're trying to test?

Is it possible to allow non-deterministic next states? I couldn't fully understand the algorithm, but from my understanding it allows non determinism on the states and it's currently limited by the API.
Proposal: instead of Step function returning a single next state, if the function could have a slice of possible next states, it would work.

In theory, yes, we could support this, in that the implementation is basically just a (smart) brute force search, and so this would just increase the search space. An even easier way is to support writing nondeterministic specifications, and support converting them to deterministic specifications using a power set construction, which is what @serathius is doing in etcd.

Also, do you think this could be made into a reusable library?

Yes! I've implemented this in 9c11fdd2790b5c33641b38bca66a865c1e14fe43.

Just curious does this affect the visualizations or the linearization violations?

This transformation is sound: checking with respect to a nondeterministic specification (if we wanted to implement support for that directly in the checker) will return the same result as checking with respect to the deterministic version following that construction. It affects visualizations, in that we now need to visualize sets of states rather than single states; I've taken a simple approach and written the default functionality to show the set as { ... }; you can override this by calling NondeterministicModel.ToModel() and then mutating the .DescribeState attribute of the resulting Model. You can see an example here:

vis.html.zip

Thanks @anishathalye for the quick turnaround.

but why is this information part of the state of a sequential specification? At a higher level: what is the spec of the system you're trying to test?

I am the developer of FizzBee, formal verification platform. First, the developers would be able to model the system at a higher level and validate their design is correct. Once the design is validated, we can use that model/state space to test the implementation. If we can validate the design meets the implementation, and since we already verified the design is correct, then it will give a higher confidence that the implementation is correct.
For the Paxos example, if we are looking at it at the user level to check if the transactions appear sequential, we don't really need to know the details. But if we want to validate the recovery process or lower level details,  we will need to capture such information. With that, I believe we would need shorter histories to validate the smaller functionality, so this could be more effective.
In addition to this, many distributed systems do not provide linearizable consistency levels. But when we validate the implementation at a lower level, I think we could use the linearizability testing tools like porcupine to match the history with the model.

Neat, thanks for sharing!
I'm going to close this issue since we have nondeterministic model support now. If you have any other feature requests, please let me know.


Improve/harmonize segmentations panel with rtstruct and add new visibility toggles

Issue: #1801
This PR also fixes a bug related to #2164 where the labelmap was being updated during segment clicks.

@Punzo this one needs to be carefully tested since we have a couple of flows and this is a big refactoring of the segmentations panel.

Selecting overlapping segments should activate crosshair and correct labelmap.
New Bulk visibility toggle for segmentations.
New segmentation UI.
All previous behavior.

Thank you in advance.

@Punzo this one needs to be carefully tested since we have a couple of flows and this is a big refactoring of the segmentations panel.

Selecting overlapping segments should activate crosshair and correct labelmap.
New Bulk visibility toggle for segmentations.
New segmentation UI.
All previous behavior.

Thank you in advance.

Tested over IDC case <IP_ADDRESS>.4.1.327<IP_ADDRESS>715003805996641695765332389135385095 (overlapping segments) and <IP_ADDRESS>.4.1.145<IP_ADDRESS>.1706.4009.192455081946878476141947586891 (case with rstruct).
Everything looks good, I just a have a remark regarding the overlapping segments (probably this was an "issue" also before this PR):
when I disable all the visibilities of the segments, e.g.:

the "non-active" ones will be still visualized. Maybe (if all the segments visibilities are off) we could toggle automatically this checkbox too:

Tested over IDC case <IP_ADDRESS>.4.1.327<IP_ADDRESS>715003805996641695765332389135385095 (overlapping segments) and <IP_ADDRESS>.4.1.145<IP_ADDRESS>.1706.4009.192455081946878476141947586891 (case with rstruct).
Everything looks good, I just a have a remark regarding the overlapping segments (probably this was an "issue" also before this PR):
when I disable all the visibilities of the segments, e.g.:

the "non-active" ones will be still visualized. Maybe (if all the segments visibilities are off) we could toggle automatically this checkbox too:

Tested over IDC case <IP_ADDRESS>.4.1.327<IP_ADDRESS>715003805996641695765332389135385095 (overlapping segments) and <IP_ADDRESS>.4.1.145<IP_ADDRESS>.1706.4009.192455081946878476141947586891 (case with rstruct).
Everything looks good, I just a have a remark regarding the overlapping segments (probably this was an "issue" also before this PR):
when I disable all the visibilities of the segments, e.g.:

the "non-active" ones will be still visualized. Maybe (if all the segments visibilities are off) we could toggle automatically this checkbox too:

Alright, just pushed, now inactive ones will be set properly. Thanks Davide.

Tested over IDC case <IP_ADDRESS>.4.1.327<IP_ADDRESS>715003805996641695765332389135385095 (overlapping segments) and <IP_ADDRESS>.4.1.145<IP_ADDRESS>.1706.4009.192455081946878476141947586891 (case with rstruct).
Everything looks good, I just a have a remark regarding the overlapping segments (probably this was an "issue" also before this PR):
when I disable all the visibilities of the segments, e.g.:

the "non-active" ones will be still visualized. Maybe (if all the segments visibilities are off) we could toggle automatically this checkbox too:

Alright, just pushed, now inactive ones will be set properly. Thanks Davide.

Alright, just pushed, now inactive ones will be set properly. Thanks Davide.

Tested, now everything wroking, LGTM!

Alright, just pushed, now inactive ones will be set properly. Thanks Davide.

Tested, now everything wroking, LGTM!


AWS CLI: Fix algorithm to find the latest version

Hello, just found that I did a mistake in algorithm - it started always showing the wrong version, instead of latest one:
> Get-EvergreenApp -Name "AWSCLI"

Version Type URI
------- ---- ---
2.9.9   msi  https://awscli.amazonaws.com/AWSCLIV2-2.9.9.msi

After the changes it should be fine:
> Get-EvergreenApp -Name "AWSCLI"

Version Type URI
------- ---- ---
2.9.23  msi  https://awscli.amazonaws.com/AWSCLIV2-2.9.23.msi

Excellent. A couple of code items to address.
I need too add a contributing guide, but essentially I want to avoid aliases in the code.

Okay, not a problem at all. we all fighting for the clean code :)

Merging changes and bypassing require signed commit check


chore: bdd-gh-1586623130 to 0.0.1

chore: Promote bdd-gh-1586623130 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add DLC requirements to missions/slots

As suggested in https://github.com/MorpheusXAUT/slotlist-frontend/issues/131, some missons (or only specific slots within the mission) could require a certain ArmA3 DLC (e.g. Heli-DLC, Tanoa, etc.).
The mission list endpoint could also be extended to allow filtering for certain DLCs.
DLC requirements would be indicators (maybe using little icons, as suggested) for now, but could potentially be extended in the future (e.g. users selecting which DLCs they have for automatic slot/mission filtering).

Tasks

[x] Add DLC requirement flags to mission
[x] Add DLC requirement flags to mission slot
[x] ~Extend mission list endpoint to allow filter for DLCs~

Filtering is done on the frontend for now - can be added to the mission list endpoint later if required.


Default props

Not ready
This PR will add default props so we don't have to repeat common prop types.

This has gotten stale and offers little gain over explicitly listing common props in each component.  Closing.


Upgrade js-yaml to version 3.13.1 or later. For example:

Fixed with PR: https://github.com/GoogleCloudPlatform/stackdriver-sandbox/pull/25


adds pass_operation_arg_name to parameter_to_arg

Fixes https://github.com/zalando/connexion/issues/937
Changes proposed in this pull request:

adds pass_operation_arg_name to allow passing the current operation to the related handler (similar approach to pass_context_arg_name)

Coverage increased (+0.005%) to 96.884% when pulling 9cf10df27d6a2a2f48b99b0675b02e8d7f0a344e on brockhaywood:bh-provide-operation-to-handler into 1ed14fbde5be3d144760fec4982e90064f151e88 on zalando:master.

Found this PR through issue #937. Since the PR is open since 2020, is there a reason this isn't merged yet?

@rjkooijman Happy to have this change in connexion, however I would move away from the pass_operation_arg_name and provide the operation under a fixed name. I'd like to do the same for pass_context_arg_name moving to v3.0.
Feel free to submit this to the main branch.


Using Matomo self-hosted instance with trackEvent/trackGoal not working

🐛 The bug
Hey guys,
I am trying to use custom event/goal tracking with a self-hosted Matomo instance and I cannot get it to work properly. Page views are being tracked, but whenever I try to use the proxy._paq.push() method, nothing happens. I attached a small example project to this issue that shows the problem (I was unable to use StackBlitz because the provided starter always failed to fetch dependencies in my case).
If I don't use the proxy but instead choose to use the _paq object directly, the events/goals are being tracked, so I suspect the proxy-mechanism does not work in my case. This happens in dev and prod builds.example.zip
🛠 To reproduce
see attached project
🌈 Expected behavior
Custom goals and events being tracked.
ℹ Additional context
Nuxt 3.14.159
Scripts 0.9.5

did you figure out how to fix it? i seem to have the same issue. no events showing up in backend and as far as i can see also nothing in the network tab

update: using window._paq.push() seems to work
see: https://github.com/nuxt/scripts/issues/279#issuecomment-2379241208


Update events.json

https://github.com/boardroom-inc/protocol-Info/commit/84798251187c97fe0d7767deaec0a1d85dd367dc

thanks Boardroom! @foxdao is the github account that will be updating the events.json going forward


Stuck in a "Create index.html" loop

Describe the bug
I am trying to use LM Studio and meta-llama-3.1-8b-instruct.  When creating a site I am stuck in a loop where it constantly creates, then recreates, index.html.
It has occurred on each of my 3 attempts with different LLMs loaded via LM Studio.
Link to the Bolt URL that caused the error
http://localhost:5173/chat/my-website
Steps to reproduce

Start a new chat
Select "LM Studio" and your LLM
Enter a description
Hit Enter
Wait for your laptop to have an asthma attack while looping in the index.html loop

Expected behavior
After submitting a prompt it would progress through the steps required to preview a site based on my prompt.
Screen Recording / Screenshot

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

please try the latest version, this should be fixed


More details about delete command for users

ikar
/me
Aira
Hello, ikar!
Your address: 6efba8fb2ac5b6730729a972ec224426a287c3ad
Balance: 0.06 ETH approved / 0.46 ETH on contract / 6.574897803336373 ETH on account
ikar
/unregister
Aira
Do you want to delete account address?
Send me 'Do as I say!' to confirm.
ikar
Do as I say!
Aira
Account deleted
ikar
/me
Aira
Hello, ikar!
Your address: 6efba8fb2ac5b6730729a972ec224426a287c3ad
Balance: 0.06 ETH approved / 0.46 ETH on contract / 6.574897803336373 ETH on account
ikar
/me
Aira
You are not registered now!

"Account deleted"  - replace text to: "Account will be deleted on the next few block. Transaction <tx_id with etherescan link>"

@ensrationis please use code blocks and triple - as separator.


Theorem 7.13 references phi_i without ever defining i.

The most convenient way to report a typo should reference the file and line number where the typo occurs:

Find the file in the content tree: https://github.com/OpenLogicProject/OpenLogic/tree/master/content
Click on the line number; then on the menu indicated by three horizontal dots
Select "Copy permalink" and paste that link below.

Proposed correction:
Maybe you meant n or meant to define i.

Thanks!


Exporting png with scaling just extends the png

I am using the latest released version of GodSVG for windows as of posting this issue.
I am encountering issues with exporting scaled vector drawings to png : Scaling just extends the png, it does not scale the actual drawing.
I do not think this is intended because this,

outputs this,

This is alpha 2, not the current master, right?

Yes, I forgot to say

Can't reproduce this on the latest branch, it must have been fixed over the last two months. So this will be fixed in the next alpha soon.

Right, thank you

By the way, to try and export it using the current main branch build I would just have to clone the repo and do it with that?

If so, using this build produces the same error on my computer.

Very odd, I can't reproduce. Could you post the graphic?

Nevermind, I reproduce it.

Ok, i will try that, thank you


M3 511

Purpose
Allow users to create one or many Node Balancer config node(s) during the creation of a NodeBalancer.
Also addressed feedback from M3-510.

Looks 👍 other than some small stuff. Again, not totally sure if the extra validation on private IPs is for another story.

@alioso at this breakpoint the label and address field have no margin between them

@rjstires @asauber this is styled and responsive-ish
I changed the "Passive Checks" from a checkbox to a toggle as well to be consistent with this pattern. IMPORTANT the toggle component has been refactored since the creation of this PR so this instance will have to be modified (it now automatically integrates the FormControlLabel) once this is merged or rebased

Coverage decreased (-0.1%) to 26.835% when pulling e63e2d4a8d2aef3b2d20ea77e66f98e884ea4576 on rjstires:M3-511 into 44f75b40fe1be93dc28fee9ea942461066481736 on linode:develop.


build(deps): bump web3[-*] from 1.2.1 to 1.2.4

NOTE: this PR is against the 4.1.x 4.2.x branch, which was created from the v4.1.1 tag. If/when this PR is merged, and assuming no additional changes are necessary, the intention is to release v4.1.2 4.2.0 from the 4.1.x 4.2.x branch.

Refactor typings as necessary. In order for bignumber.js in the root node_modules to not conflict with @types/bignumber.js, specify "bignumber.js": "5.0.0" in devDependencies and "embark-ui/bignumber.js" in nohoist of the root package.json.
Bump simples from 0.8.8 to 0.9.0-alpha-10 in packages/embark-blockchain-process/package.json and refactor packages/embark-blockchain-process/src/proxy.js. Drop simples as a dependency in packages/embark/package.json since it's not needed there. The upgrade of simples allows embark 4.1.x's 4.2.x's proxy to work correctly with geth >=1.9.1.
Bump ganache-cli from 6.4.3 to 6.7.0 (latest) because 6.4.3 doesn't support eth_chainId and since web3 1.2.4 makes use of the eth_chainId RPC method (EIP 695).
BREAKING CHANGE: bump embark's minimum supported version of geth from >=1.8.14 to >=1.9.0 and its minimum supported version of parity from >=2.0.0 to >=2.2.1. This is necessary since web3 1.2.4 makes use of the eth_chainId RPC method (EIP 695) and those client versions are the earliest ones to implement it.
I've tested these changes by making a local release of v4.1.2 v4.2.0 with verdaccio and embark testing and embark runing status-teller-network and embark_demo with the local release. In the case of embark_demo the embark installation was global and in the case of teller it was local. Everything seemed to work as expected.

I have deployed this at: https://raw.dev.dap.ps/
It seems to work in Safari. Amazing work dude!

Added don't merge yet label:  I'm attempting an upgrade of ganache-cli to 6.7.0 on this branch. While embark's test runner in 4.1.x doesn't seem to make any usage of web3 v1.2.4 that involves the eth_chainId RPC method, it still makes sense to ship an embark version (e.g. pending 4.1.2) that includes a simulator that fully supports the web3 version that's included with embark.

PR description updated with respect to the intention to publish as embark v4.2.0.
Also indicated the BREAKING CHANGE: re: minimum supported versions of geth and parity (in PR description and commit message).
Also added a note (in PR description and commit message) re: the version bump of ganache-cli.


Add support for global labels

closes #507
blocked by https://github.com/elastic/apm-server/issues/2037

Codecov Report

Merging #543 into master will increase coverage by 0.33%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #543      +/-   ##
============================================
+ Coverage     63.51%   63.84%   +0.33%
  Complexity       68       68
============================================
  Files           178      178
  Lines          6687     6716      +29
  Branches        765      766       +1
============================================
+ Hits           4247     4288      +41
+ Misses         2180     2170      -10
+ Partials        260      258       -2

Impacted Files
Coverage Δ
Complexity Δ

.../apm/agent/report/serialize/DslJsonSerializer.java
82.4% <100%> (+0.55%)
0 <0> (ø)
:arrow_down:

...tic/apm/agent/configuration/CoreConfiguration.java
96.17% <100%> (+0.23%)
0 <0> (ø)
:arrow_down:

...pm/agent/report/IntakeV2ReportingEventHandler.java
76.23% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../main/java/co/elastic/apm/agent/impl/MetaData.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...a/co/elastic/apm/agent/report/ReporterFactory.java
79.16% <100%> (+0.9%)
0 <0> (ø)
:arrow_down:

...java/co/elastic/apm/agent/bci/ElasticApmAgent.java
75.47% <0%> (+1.88%)
0% <0%> (ø)
:arrow_down:

...c/apm/agent/bci/MatcherTimerLifecycleListener.java
93.75% <0%> (+18.75%)
0% <0%> (ø)
:arrow_down:

.../elastic/apm/agent/bci/bytebuddy/MatcherTimer.java
93.75% <0%> (+31.25%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7d6b96...b87eae5. Read the comment docs.


Can not preview successfully in slides forms.

After downloading the plugin and opening the file, the preview on the right couldn't successfully preview it as a PowerPoint. I tried to output in both HTML and PDF formats, and the output was not a horizontally paginated PowerPoint, but a vertical page of non paginated white paper as shown on the right side of the preview.

Marp for VS Code has been designed as an extension for VS Code built-in Markdown preview. Your screenshot looks like showing a different Markdown preview feature provided by an other VS Code extension. Try opening the default Markdown preview from the command palette (Markdown: Open Preview to the Side).
Similar

https://github.com/marp-team/marp-vscode/issues/82
#423

I tried to output in both HTML and PDF formats, and the output was not a horizontally paginated PowerPoint, but a vertical page of non paginated white paper as shown on the right side of the preview.

By Markdown spec, if there is --- immediately following a line of text, it is treated as part of a setext heading and not as a page break.
In other words,
good
---
gg
---

is exactly same meaning as:
## good
## gg

You should make a blank line between the text and a ruler to split slide pages, like that:
good

---

gg

---

Marp for VS Code has been designed as an extension for VS Code built-in Markdown preview. Your screenshot looks like showing a different Markdown preview feature provided by an other VS Code extension. Try opening the default Markdown preview from the command palette (Markdown: Open Preview to the Side).
Similar

Can't preview with Ctrl + K V & Ctrl + Shift + V if other markdown viewers are installed #82

Thanks a lot. This commod works.

Is it okay to assume that this issue has been solved?
If so, it is better to close the issue.


Feature/shell and term

close  #34
depends on https://github.com/kaz-yos/eval-in-repl/pull/44

[ ] saw that intern-soft only uses already created symbols to convert a string to a symbol. if the symbol is not already created, it returns nil. Is there a nice & straight forward way to convert a string to a symbol while only allowing certain values? (shell and term). ie to match eir-shell-type allowed values (see https://github.com/kaz-yos/eval-in-repl/pull/44/files#diff-8e40d8ad4ecdd61327ed3897d9aa27abeef5e67e2079b06159fefdb61a5b3fe6R127 )
the usage is having :shell with values shell, term or vterm in the src header to use the corresponding -mode

Open TODO/notes
I realize that the :shell keyword is not really appropriate. One would expect to maybe type there /bin/bash or /bin/fish or something like this. And actually, this could even be implemented in the future.

[ ] so, rename to something like :shell-type ??

rename to something like :shell-type ??
I also think :shell-type would be better. I tried to come up with other options, but couldn't find a reasonable naming except this one.

intern-soft
Sorry, where does this intern-soft appear? I couldn't find it in the code.
Either way, I think it's simpler to perform filtering first, then convert to symbols.

intern-soft appears here
(intern-soft (cdr (assq :shell (nth 2 (ober-src-block-info-light)))))
and intern-soft will return nil if you pass a string like "blahblahhhh", cause the blahblahhhh symbol was never created before

@actonDev Sorry for the delay. What do you think about this?:
(defcustom eir-shell-type-candidates '("shell" "term" "vterm")
  "The shell mode candidates to be used for newly spawned shells (ie shell or term)"
  :group 'eval-in-repl)

(defun intern-or-create-soft (symbol-name name-candidates default)
  (if (car (member symbol-name name-candidates))
      (intern symbol-name)
    default))

(intern-or-create-soft (cdr (assq :shell (nth 2 (ober-src-block-info-light))))
                       eir-shell-type-candidates eir-shell-type)

The last expression is intended to replace
(or (intern-soft (cdr (assq :shell (nth 2 (ober-src-block-info-light)))))
			     eir-shell-type)

this part in the PR.

Sorry from my turn for the delay.
I just pushed a change regarding the :shell-type
I liked your solution for "checked" string -> symbol, but I opted to not do this.
Proplery done indeed having the eir-shell-type-candidates in the eval-in-repl package. And I don't want to go again submitting a PR there. But I will have this approach in mind for whenever I need it again :)
I'm going to squash merge this in develop, and do the cask things there.
I'll open the PR then from develop for the last approval :)

I see. I think that's reasonable :)
Thanks for the update! I've approved the merge, but it seems TravisCI is taking long (where it shouldn't) for some reason. I'm looking into it now.


fix README.md, widgets' example has been removed on 25798a7

Context:
widgets' example has been removed on 25798a7
$ cargo run --example widgets --features miniraq --no-default-features
error: no example target named `widgets`

Contribution checklist:

[ ] Add documentation to all public structs, traits and functions.
[ ] Add unit tests if possible
[ ] Describe the major change(s) in the CHANGELOG.MD
[ ] Run cargo fmt to make the formatting consistent across the codebase
[ ] Run cargo clippy to check with the linter

Thank you


Check yaw rates instead of positions when yoyoing

This  PR depends on #89, #96.
Recommended review order is #89 > #96 > this PR. If we want I can cherry pick this branch as the changes are independent of the underlying code. The merge order should be this PR > #96 > #89.
There is a slight change in the center of rotation of the yoyo mission with #96. This leads to test expectations failing. Rather than use position to check whether the vehicle is yoyoing, I think we should use @tfoote's suggestion and use yaw rate which makes the test independent of the center of rotation and hence more robust while at the same time ensuring that the vehicle actually moves in a circle.

The tolerance for the yaw rate now is in the same order of magnitude than the rate itself (0.037 +- 0.02). So we should just make sure that we're not actually regressing and that's why we're not getting a circle anymore.

Thanks for spotting this :man_facepalming: . Yeah we don't need search tolerances I forgot to bump the tolerance back down. its now 0.037 +/- 0.002.

I can imagine fixes to lift/drag changing the forces on the rudder, which may change the yaw rate for example.

I do not believe yaw rate should change. The reason is that for this regime, the current LiftDrag plugin suffices to give an accurate model of the vehicle. Basically the yaw rate is determined by the lift generated by the rudder and the thruster thus resulting in circular motion. Our LiftDrag plugin is not however correct for situations where we are drifting, or where current plays a role or the depthVBS mission for instance where the angle of attack is very high.

I cherry-picked this PR onto main in https://github.com/osrf/lrauv/pull/133 so we can merge it faster.

#133 got merged. We should adjust the tolerance on #89 or #96.


Event on table 290

I need on table 290 on functions:
Update Lines
"VAT Calculation Type"::"Sales Tax":
BEGIN
"VAT Base" := CalcLineAmount;
IF "Use Tax" THEN
"VAT Amount" := 0
ELSE
BEGIN
"VAT Amount" :=
SalesTaxCalculate.CalculateTax(
TaxAreaCode,"Tax Group Code",TaxLiable,PostingDate,"VAT Base",Quantity,CurrencyFactor);
OnSalesTaxCalculateTax(Rec,Currency); // I need this event
END;
IF "VAT Base" = 0 THEN
"VAT %" := 0
ELSE
"VAT %" := ROUND(100 * "VAT Amount" / "VAT Base",0.00001);
"VAT Amount" :=
"VAT Difference" +
ROUND("VAT Amount",Currency."Amount Rounding Precision",Currency.VATRoundingDirection);
"Amount Including VAT" := "VAT Base" + "VAT Amount";
END;
[IntegrationEvent] OnSalesTaxCalculateTax(VAR VATAmountLine : Record "VAT Amount Line";PRecCurrency : Record Currency)

"VAT Calculation Type"::"Sales Tax":
BEGIN
"Amount Including VAT" := CalcLineAmount;
IF "Use Tax" THEN
"VAT Base" := "Amount Including VAT"
ELSE
BEGIN//LOC-ERS
"VAT Base" :=
ROUND(
SalesTaxCalculate.ReverseCalculateTax(
TaxAreaCode,"Tax Group Code",TaxLiable,PostingDate,"Amount Including VAT",Quantity,CurrencyFactor),
Currency."Amount Rounding Precision");
OnSalesTaxReverseCalculateTax(Rec,Currency); // I need this event
"VAT Amount" := "VAT Difference" + "Amount Including VAT" - "VAT Base";
END;//LOC-ERS
IF "VAT Base" = 0 THEN
"VAT %" := 0
ELSE
"VAT %" := ROUND(100 * "VAT Amount" / "VAT Base",0.00001);
END;
END
[IntegrationEvent] OnSalesTaxReverseCalculateTax(VAR VATAmountLine : Record "VAT Amount Line";PRecCurrency : Record Currency)

Should be posted in ALAppExtensions: https://github.com/Microsoft/ALAppExtensions/issues


[MRG] reset raw.annotations.orig_time to meas_date

This is the proposed behaviour when doing raw.set_annotations(annot):

When meas_date is set and orig_time is set : set the annotations, set orig_time to meas_date and reset onsets
When meas_date is set and orig_time is None : assume orig_time to be raw.first_samp
When meas_date is None and orig_time is set : Crash
When meas_date is None and orig_time is None : assume orig_time to be raw.first_samp and keep raw.annotations.orig_time as None
This needs to go in before #4936

The changes in 551b6a4, show that this is not backward compatible and that we might be breaking things. Shall we do a deprecation cycle? cc:@agramfort @larsoner

Apparently, the following tests are also broken. Either 'cos now we raise the RuntimeError, or 'cos the onsets need to refer to orig_time:

[ ] mne/channels/tests/test_montage.py:503 test_egi_dig_montage
[ ] mne/gui/tests/test_kit2fiff_gui.py:35 test_kit2fiff_model
[ ] mne/io/array/tests/test_array.py:38 test_array_raw
[ ] mne/io/edf/tests/test_edf.py:52 test_bdf_data
[ ] mne/io/edf/tests/test_edf.py:74 test_bdf_stim_channel
[ ] mne/io/edf/tests/test_edf.py:108 test_edf_reduced
[ ] mne/io/edf/tests/test_edf.py:115 test_edf_data
[ ] mne/io/kit/tests/test_kit.py:47 test_data
[ ] mne/io/nicolet/tests/test_nicolet.py:17 test_data
[ ] mne/io/tests/test_meas_info.py:411 test_anonymize
[ ] mne/simulation/tests/test_raw.py:274 test_simulate_round_trip
[ ] mne/tests/test_morph.py:103 test_xhemi_morph
[ ] mne/tests/test_report.py:140 test_report_raw_psd_and_date
[ ] mne/viz/tests/test_evoked.py:96 test_plot_evoked
[ ] mne/viz/tests/test_ica.py:56 test_plot_ica_components
[ ] mne/viz/tests/test_topo.py:149 test_plot_topo_single_ch
[ ] mne/viz/tests/test_topomap.py:50 test_plot_topomap_interactive
[ ] mne/viz/tests/test_topomap.py:127 test_plot_topomap

I don't see some of those failures in the complete run here, and I'm not sure how changing annotations could break them (e.g., xhemi):
https://travis-ci.org/mne-tools/mne-python/jobs/425453360

... also it would be amazing to have the diagram at the top represented in ASCII art in the Notes section of Annotations. Every time I have to think about this, my head hurts, and this visual would help.

the problem is that the meas_date is a singe int for some readers and not 2 int32 like for fif files.
our doc https://martinos.org/mne/stable/generated/mne.Info.html here says:
meas_date : list of int
The first element of this list is a POSIX timestamp (milliseconds since 1970-01-01 00:00:00) denoting the date and time at which the measurement was taken. The second element is the number of microseconds.

so it should always have 2 elements

Ok _test_raw_reader is confusing me. in 8a52e2f I think I managed to fix the io tests. But what puzzels me is this:
if I invert the condition like this
- expected_bad_boundary_onset = last_samp / raw.info['sfreq'] if concat_raw.info['meas_date'] is not None else (last_samp - first_samp) / raw.info['sfreq']
+ expected_bad_boundary_onset = last_samp / raw.info['sfreq'] if concat_raw.info['meas_date'] is None else (last_samp - first_samp) / raw.info['sfreq']

everything should fall appart, but it does not. Only two tests fail:
Results (65.42s):
     134 passed
       2 failed
         - mne/io/array/tests/test_array.py:38 test_array_raw
         - mne/io/fiff/tests/test_raw_fiff.py:624 test_io_complex
       7 skipped

This time just to make sure that things break I modify the test for something impossible like this:
    assert_array_almost_equal(concat_raw.annotations.onset[idx],
                              expected_bad_boundary_onset+42,
                              decimal=2)

and indeed, now more test break:
Results (54.16s):
     112 passed
      24 failed
         - mne/io/array/tests/test_array.py:38 test_array_raw
         - mne/io/artemis123/tests/test_artemis123.py:45 test_data
         - mne/io/brainvision/tests/test_brainvision.py:158 test_brainvision_data_highpass_filters
         - mne/io/brainvision/tests/test_brainvision.py:216 test_brainvision_data_lowpass_filters
         - mne/io/brainvision/tests/test_brainvision.py:271 test_brainvision_data_partially_disabled_hw_filters
         - mne/io/brainvision/tests/test_brainvision.py:294 test_brainvision_data_software_filters_latin1_global_units
         - mne/io/brainvision/tests/test_brainvision.py:305 test_brainvision_data
         - mne/io/bti/tests/test_bti.py:46 test_read_2500
         - mne/io/bti/tests/test_bti.py:61 test_crop_append
         - mne/io/cnt/tests/test_cnt.py:19 test_data
         - mne/io/ctf/tests/test_ctf.py:49 test_read_ctf
         - mne/io/edf/tests/test_edf.py:52 test_bdf_data
         - mne/io/edf/tests/test_edf.py:74 test_bdf_stim_channel
         - mne/io/edf/tests/test_edf.py:108 test_edf_reduced
         - mne/io/edf/tests/test_edf.py:115 test_edf_data
         - mne/io/eeglab/tests/test_eeglab.py:55 test_io_set_raw[fnames0]
         - mne/io/eeglab/tests/test_eeglab.py:55 test_io_set_raw[fnames1]
         - mne/io/egi/tests/test_egi.py:28 test_io_egi_mff
         - mne/io/egi/tests/test_egi.py:60 test_io_egi
         - mne/io/egi/tests/test_egi.py:110 test_io_egi_pns_mff
         - mne/io/eximia/tests/test_eximia.py:15 test_eximia_nxe
         - mne/io/fiff/tests/test_raw_fiff.py:624 test_io_complex
         - mne/io/kit/tests/test_kit.py:47 test_data
         - mne/io/nicolet/tests/test_nicolet.py:17 test_data
       7 skipped

At this point, intuition would say that all those examples meas_date is in a condition that it does not matter if its None or not. The invariant case. Maybe when meas_date is 0?. but actually is not 0.
i.e for test_crop_append:
~/code/mne-python orig_matrix* 56s
(mne) ❯ pytest mne/io -k test_crop_append
Test session starts (platform: linux, Python 3.6.6, pytest 3.6.3, pytest-sugar 0.9.1)
rootdir: /home/sik/code/mne-python, inifile: setup.cfg
plugins: smother-0.2, sugar-0.9.1, pudb-0.6, ipdb-0.1, faulthandler-1.5.0, cov-2.5.1

----------

(Pdb) l
 91
 92  	    expected_bad_boundary_onset = last_samp / raw.info['sfreq'] if concat_raw.info['meas_date'] is not None else (last_samp - first_samp) / raw.info['sfreq']
 93
 94  	    assert_array_almost_equal(concat_raw.annotations.onset[idx],
 95  	                              expected_bad_boundary_onset+42,
 96  ->	                              decimal=2)
 97
 98  	    if raw.info['meas_id'] is not None:
 99  	        for key in ['secs', 'usecs', 'version']:
100  	            assert_equal(raw.info['meas_id'][key], raw3.info['meas_id'][key])
101  	        assert_array_equal(raw.info['meas_id']['machid'],
(Pdb) !concat_raw.info['meas_date']
(1348401098, 0)
(Pdb) !concat_raw.annotations.orig_time
1348401098.0
(Pdb)

So, somehow I'm confusing myself. @larsoner, @agramfort any thoughts? I would expect more than 2 of those cases.

Ok, I think I'm on the last bit.
  /home/sik/code/mne-python/mne/channels/channels.py(882)add_channels()
-> new_info = _merge_info(infos, force_update_to_first=force_update_info)

when merging infos, meas_date gets set to 0 while orig_time keeps its value.

All these tests suffer from the same problem. they call _merge_info and they lose meas_date.
         - mne/simulation/tests/test_raw.py:79 test_simulate_raw_sphere
         - mne/simulation/tests/test_raw.py:226 test_simulate_raw_bem
         - mne/simulation/tests/test_raw.py:274 test_simulate_round_trip
         - mne/simulation/tests/test_raw.py:79 test_simulate_raw_sphere
         - mne/simulation/tests/test_raw.py:226 test_simulate_raw_bem
         - mne/simulation/tests/test_raw.py:274 test_simulate_round_trip
         - mne/simulation/tests/test_raw.py:79 test_simulate_raw_sphere
         - mne/simulation/tests/test_raw.py:226 test_simulate_raw_bem
         - mne/simulation/tests/test_raw.py:274 test_simulate_round_trip
         - mne/tests/test_epochs.py:1807 test_contains
         - mne/tests/test_epochs.py:1907 test_add_channels_epochs
         - mne/tests/test_epochs.py:1807 test_contains
         - mne/tests/test_epochs.py:1907 test_add_channels_epochs
         - mne/tests/test_epochs.py:1807 test_contains
         - mne/tests/test_epochs.py:1907 test_add_channels_epochs
         - mne/tests/test_epochs.py:1807 test_contains
         - mne/tests/test_epochs.py:1907 test_add_channels_epochs

But I guess that only mne/simulation/tests/test_raw.py:274test_simulate_round_trip
 breaks 'cos the rest might not have annotations.
This is the patch to replicate:
diff --git a/mne/io/meas_info.py b/mne/io/meas_info.py
index 527f06178..6975860a9 100644
--- a/mne/io/meas_info.py
+++ b/mne/io/meas_info.py
@@ -1654,6 +1654,11 @@ def _merge_info(infos, force_update_to_first=False, verbose=None):
     info : instance of Info
         The merged info object.
     """
+    check = False
+    if len(infos)>0 and infos[0]['meas_date'] is not None:
+        check = True
+        check_val = infos[0]['meas_date']
+
     for info in infos:
         info._check_consistency()
     if force_update_to_first is True:
@@ -1723,6 +1728,9 @@ def _merge_info(infos, force_update_to_first=False, verbose=None):
         info[k] = _merge_info_values(infos, k)

     info._check_consistency()
+    if check:
+        return_meas = info['meas_date']
+        assert return_meas == check_val
     return info

@massich yes there is a bug somewhere. There is no reason to loose the date although when concatenating raws maybe it does not make much sense to have it anymore.
@massich if was overlooked I think so can you just try to keep it?

This is ready for review. It fails only PEP8, and I was aware of it. So bring up your comments !

I think I asked already, but I'm not sure. Shall we deprecate the fact that before orig_time was not aligned with meas_date, and do a deprecation cycle and all that? or we consider this a bug, and we move on?

I think I've addressed everything. I'll be AFK so feel free to push any changes.

Fine with me @agramfort, @larsoner merge if happy

besides LGTM


3.0.0 checksum changed

Hi
Homebrew maintainer here. We shipped the 3.0.0  release a few days ago, but meanwhile the checksum of the release has changed. Can you confirm that this was a re-tag, and nothing was compromised on your side.
For reference: https://github.com/Homebrew/homebrew-core/pull/90670

Hey @iMichka! Yeah, I retagged 3.0.0 like a day after release. Sorry for the mess 😓

No issue. We prefer that wrong releases get their tag deleted, and a new 3.0.1 is published instead. That's the best solution for these cases.
The git manual also states that retagging is a bad thing do: https://git-scm.com/docs/git-tag#_on_re_tagging


Hidpi support

I've got a shiny new screen on my workplace, but now all examples fail :) Time to fix highres display support.
Some notes so far:

If the size of the render texture does not exactly match the size of the widget, you get a black screen.
In glfw, the widget height and weight are expressed in physical size. All good!
In Qt, the widget height/weight are in logical pixels, but seem to be rounded. I have yet to figure out how to get the physical size.
In Tk ... sigh there is no concept of logical pixels. Everything works but is tiny on highres displays.

Getting hidpi to work with Qt on Windows is quite a trip. We figured it out for one of our internal Qt apps.
First, you need to let Windows know that your process is DPI aware:
import ctypes
try:
    ctypes.windll.shcore.SetProcessDpiAwareness(True)
except:
    pass  # not windows

Then you can configure Qt:
QApplication.setAttribute(Qt.AA_EnableHighDpiScaling, True)
QCoreApplication.setAttribute(Qt.AA_UseHighDpiPixmaps, True)

And we already merged the method to get the devicePixelRatio for the screen the window is currently on:
class QtWgpuCanvas(BaseCanvas, QWidget):
    # ...
    def get_size_and_pixel_ratio(self):
        return self.width(), self.height(), self.window().devicePixelRatio()

@Korijn interesting that devicePixelRatio was already non-unit. Will look into it.
I applied the calls you mentioned earlier in Pyzo. I just learned that it should be ctypes.windll.shcore.SetProcessDpiAwareness(2), see https://github.com/pyzo/pyzo/pull/696 for details. You may want to update your code as well.

interesting that devicePixelRatio was already non-unit.

This was because I was running from Pyzo (which is also Qt), and the interpreter process inherits some stuff, apparently.

interesting that devicePixelRatio was already non-unit.

This was because I was running from Pyzo (which is also Qt), and the interpreter process inherits some stuff, apparently.

Yeah.

Ready for review. What happened to CI? Did it stop running because this was a draft?

That's right. You should be able to trigger a build manually for this PR in azure though!

Also tested on Linux.


[BEAM-2626] Fix split package in SDK harness

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

The Java SDK harness defined classes both in its own namespace org.apache.beam.fn.harness and the org.apache.beam.runners.core namespace, resulting in a split package across multiple jars.
Generally not a good practice since it breaks encapsulation and not actually useful as a hack since that method of breaking encapsulation isn't always compatible with classloader setups.

Coverage decreased (-0.007%) to 70.619% when pulling f1b4700f32c5ea39559145d6f5db3909439f6c80 on kennknowles:harness-split-package into 7e4719cd01f412c000bab61f99c687ab5c5491cc on apache:master.

R: @tgroh
also CC @vikkyrk in case there's something funky going on here


Reduce possibility for H2 client count mismatch

Details in issue #8200.  The code change pulls up the decrementing logic for the client stream count, so we can be more likely to be consistent with the H2 peers counting.
This closes #8200

[approve ci clang-analyzer]

[approve ci autest]

@masaori335 is going to look at it.

[approve ci]

[approve ci rocky]


[zeroc-ice] Revise features and build; Wrap FindIce.cmake

zeroc-ice

Build C++98 components also for windows, but move C++98 libs to manual-link.
(Note: some C++98 libs were already built in the past because they are used by tools.)
Align (libs) feature names with upstream's component names.
(Deprecated feature names should be remove with a future update.)
Add IcePatch2 feature.
Extend default features list to cover all components, similar to official binary distribution.
Use vcpkg_configure_make/vcpkg_install_make pattern for non-windows builds, reduce patching.
Improve cross-build support by using host slice2cpp.
Add wrapper for FindIce.cmake, fixes #33589.
(Tested in CI for COMPONENTS Ice Ice++11).

Tested with vcpkg test-features --triplet x64-linux-dynamic zeroc-ice (https://github.com/microsoft/vcpkg-tool/pull/802).
(zeroc-ice-)mcpp

Add "zeroc-ice-" prefix to the name because it is an unofficial fork.
Minor changes to the build.

cmake-user (CI test port)

Add facility to use CI test cmake config provided by actual ports.

Port mcpp didn't use the official upstream but a simplified source from zeroc-ice. This PR move the port implementation to zeroc-ice-mcpp. The mcpp name could be delisted.

Align (libs) feature names with upstream's component names.

TBH I would prefer a radical simplification of the feature interface, e.g.
core: C++98 API libs and slice2cpp
c++11: C++11 API libs. (Extra config in non-windows build.)
tools: all tools (shared only)
C++98 API libs are needed for almost all tools. That's why it makes sense to have them in core.
For the users' transition, the old now-deprecated features would depend on c++11 to pull in what was installed before.

Waiting for https://github.com/microsoft/vcpkg/pull/33967 to be merged first, unblocking arm windows CI.

Waiting for #34085 to fix baseline regression.

Align (libs) feature names with upstream's component names.

TBH I would prefer a radical simplification of the feature interface, e.g. core: C++98 API libs and slice2cpp c++11: C++11 API libs. (Extra config in non-windows build.) tools: all tools (shared only)
C++98 API libs are needed for almost all tools. That's why it makes sense to have them in core. For the users' transition, the old now-deprecated features would depend on c++11 to pull in what was installed before.

This is really a 'what do users of this thing' expect question. Tagging folks who have contributed to this port before.
@bold84 @jvbsl What do you think of @dg0yt 's proposal?

Everything @dg0yt proposed makes sense. 👍

@dg0yt First of all sorry for not answering earlier, the GitHub notification must have slipped by me, cause of the flood of notifications ^^
@bold84 thanks for tagging;)
Do I understand correctly that you mean that the c++98 libs are always built and installed, whereas the c++11 bindings and slice2cppare optional features(meaning if I select it, all libraries will be built using c++11?)
So that there would be no granular control of which specific libraries to install?
If so that would be completely fine by me. To be honest I changed the project I'm working on to use the c++11 bindings just because zeroc-ice vcpkg only provided those. I'm not even sure if the maintainers would be comfortable with such a big change^^
So it would be even better to have that option.
Thank you again @dg0yt for all your work it's creatly appreciated

Do I understand correctly that you mean that the c++98 libs are always built and installed, whereas the c++11 bindings and slice2cpp are optional features(meaning if I select it, all libraries will be built using c++11?)
So that there would be no granular control of which specific libraries to install?

slip2cpp should always be built (core) for native targets. (It is a tool which runs on the host.)
Upstream provide a complete SDK, so this is what I would expect from the default installation.
Why would users opt out of individual libraries?
For this port, the specific libraries have hardly any impact on external dependencies.
And AFAICS they also have no fine-grain impact on build times.
C++98 libs are always used for most tools ATM.

To be honest I changed the project I'm working on to use the c++11 bindings just because zeroc-ice vcpkg only provided those.

IMO this is a kind of change which the default installation of a vcpkg port shouldn't require.

I will go ahead with the proposed change to the feature interface, so that we can see the impact on what must be maintained in vcpkg.

I will go ahead with the proposed change to the feature interface, so that we can see the impact on what must be maintained in vcpkg.

Are you saying you want to make changes or that you intend to do those in a subsequent PR?

I want to make the changes first.

May I know what the current state of this draft is and whether there is something I could help with?

I hope I can move this to ready-to-review again soon.

Hey @dg0yt,
Don't want to be pushy and I of course know that all this is on a voluntary basis, but I hope you have just forgotten about this, and a small reminder would get it in your mind again :D
Apart from that I looked where the merge conflichts come from and rebased everything here:
https://github.com/jvbsl/vcpkg/tree/zeroc-ice
But it was just one merge conflict in scripts/ci.baseline.txt which was really easily resolved. So no need to use my branch as a basis. Just wanted to inform you that rebasing here is fortunately really easy(as I personally dislike fixing conflicts^^).
My message could perhaps sound entitled, but it is really not meant in that way. I would just really like to have this merged and if there is anything I could help you with(could also be on another topic or project) just do tell
But thank you again with all your help so far(also on other vcpkg topics)


Cleanup of ModelCriteria class

Are you able to finish this up today?
We need to start closing changes and start QA phase for release.


fix(ui): fix plurals

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[x] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Description
fix(ui): fix plurals in translation
fix(ui): prettier ignores translation files
fix(ui): add missing translation strings

How to get the last langeuage package, thanks.

如何获取最后一个语言包，谢谢。

Thanks, I have got the lastest en.json and translateed it into Chinese.


Ruff config file

Hi there 👋
Core maintainer of the repo decided for simplicity to not have a linter config, so every now and then something like this CLI command is applyed:
ruff check * --select E,W,F,S --extend-select C4,SIM,RET,PT,I001,ANN001,ANN201,ANN205,ANN206,ARG001 --ignore E501,E731,S108,S101,S113,S603,PT007,S310,E402,PT004,C408 --per-file-ignores "*/**/__init__.py":I001,"tests/*":ANN --line-length 120 --target-version py38 --fix; black -l120 -C --preview --target-version py38 chat eval finetune generate lit_gpt notebooks pretrain quantize scripts tests xla

If someone wants to apply changes inside, let's say VSCode, then this config should match the CLI command above:
# https://beta.ruff.rs/docs/rules/

target-version = "py38"

select = [
  "E", # pycodestyle Error
  "W", # pycodestyle Warning
  "F", # PyFlakes
  "S", # flake8-bandit
  "C4", # flake8-comprehensions
  "SIM", # flake8-simplify
  "RET", # flake8-return
  "PT", # flake8-pytest-style
  "I001", # isort: unsorted-imports
  "ANN", # flake8-annotations
  "ARG001", # Unused function argument
]

ignore = [
  "ANN101", # Missing type annotation for self in method
  "ANN102", # Missing type annotation for cls
  "ANN202", # Missing return type annotation for private function
  "ANN204", # Missing return type annotation for special method `__init__`
  "ANN401", # Dynamically typed expressions (typing.Any) are disallowed in `value`
  "C408", # Unnecessary {obj_type} call (rewrite as a literal)
  "E402", # Module level import not at top of file
  "E501", # Line too long ({width} > {limit} characters)
  "E731", # Do not assign a lambda expression, use a def
  "PT004", # Fixture {function} does not return anything, add leading underscore
  "PT007", # Wrong values type in @pytest.mark.parametrize expected {values} of {row}
  "S101", # assert detected
  "S108", # Probable insecure usage of temporary file or directory: "{}"
  "S113", # Probable use of requests call without timeout
  "S310", # Audit URL open for permitted schemes. Allowing use of file: or custom schemes is often unexpected.
  "S603", # subprocess call: check for execution of untrusted input
]

line-length = 120

[per-file-ignores]
"__init__.py" = [
  "I001", # isort: unsorted-imports
]
"conftest.py" = [
  "ANN001", # Missing type annotation for self in method
  "ANN201", # Missing return type annotation for public function {name}
  "D102", # Missing docstring in public method
  "S101", # assert detected
]
"test_*.py" = [
  "ANN001", # Missing type annotation for self in method
  "ANN201", # Missing return type annotation for public function {name}
  "D102", # Missing docstring in public method
  "S101", # assert detected
]

Just put it inside ruff.toml file and don't forget to add it to global .gitignore file.

[!IMPORTANT]
Sometimes Ruff/Black extensions for VSCode have a different view on how to apply code style changes in comparison to CLI command.
Remember that CLI command always supersedes VSCode extension!

If you can find a way to configure pre-commit/ruff to do this automatically on PRs without adding configuration files at the root, I would accept it.

You mean only .github folder is allowed?
Sounds suspiciously like a challenge 🤔.


Does the proxyAddress attribute in AAD always contain the UPN as a secondary email address?

Is there a way for an AAD user object to NOT have the on-premises UPN as a primary OR secondary SMTP address in the proxyAddresses attribute?
I created an outbound to AAD sync rule that sets the mail and proxyAddresses attribute, but when querying AAD, the UPN is always present as the secondary SMTP address.

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

ID: ab715116-6bb3-5328-e3d7-c19b00359cdf
Version Independent ID: c1c5d84e-ab3c-2cda-2b7f-d9f494c5481d
Content: How the proxyAddresses attribute is populated in Azure AD - Active Directory
Content Source: support/azure/active-directory/proxyaddresses-attribute-populate.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @genlin
Microsoft Alias: genli

@LijuV-MSFT For your question, I would suggest you contact Azure support for help.


Create auto-merge composite pipeline for spark-rapids-* repos

migrate https://github.com/NVIDIA/spark-rapids/tree/branch-24.10/.github/workflows/auto-merge
to the common repo

update all spark-rapids projects except JNI.
Going to apply a simple patch to avoid manual update for JNI first


[vcpkg-docs] Add docs for vcpkg_fixup_cmake_targets

From https://github.com/Microsoft/vcpkg/pull/1111#issuecomment-302231394

Docs for vcpkg_fixup_cmake_targets has been added to VCPKG, see vcpkg_fixup_cmake_targets


Swift

how to use in swift

import UIKit
import NDParallaxIntroView

class IntroViewController: UIViewController, NDIntroViewDelegate {

var introView : NDIntroView!;

override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view, typically from a nib.

    let pageContentArray: [AnyObject] =
        [[kNDIntroPageTitle: "Title 1",
        kNDIntroPageDescription: "Blah blah blah.",
        kNDIntroPageImageName: "image1"],

        [kNDIntroPageTitle: "Title 2",
        kNDIntroPageDescription: "More blah blah blah.",
        kNDIntroPageImageName: "image2"],

        [kNDIntroPageTitle: "Title 3",
        kNDIntroPageDescription: "blahblahblahblah.",
        kNDIntroPageImageName: "image3"],

        [kNDIntroPageTitle: "Title 4",
        kNDIntroPageDescription: "blah blah blah.",
        kNDIntroPageImageName: "image4"],

        [kNDIntroPageTitle: "What are you waiting for?",
        kNDIntroPageDescription: "Try this app now!"]]

    self.introView = NDIntroView(frame: self.view.frame, parallaxImage: UIImage(named: "parallaxBgImage"), andData: pageContentArray)
    self.introView.delegate = self
    self.view!.addSubview(self.introView)
}

internal func launchAppButtonPressed()
{
    //use a segue or something to go to next view controller
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}
}

I am extremely new to iOS dev (only started working with obj-C & Swift a little over a week ago) and I found these sites to be helpful (maybe it can help you too):
For swift: https://www.raywenderlich.com/115253/swift-2-tutorial-a-quick-start
For obj-c: https://www.raywenderlich.com/4872/objective-c-cheat-sheet-and-quick-reference

thanks :)

i got import NDParallaxIntroView no such module error

no problem i used pod version everything ok thanks again


DHCP: Populate nameservers D-bus property from resolv.conf

Currently when DHCP enabled, nameservers D-bus property does not have updated DHCP provided nameservers.
This commit populates nameservers D-bus property from resolv.conf
Tested By: verify nameservers D-bus property value with DHCP enable/disable
Change-Id: If1a2508927d5f4f0c9c5d8b3f533fdf384d68821

upstream commit: https://gerrit.openbmc.org/c/openbmc/phosphor-networkd/+/66476

@rfrandse @geissonator can you help check this CI failure for networkd Pls

jenkins run tests please


Implement "find" command

there is a find command already, unless this issue is talking about the bug in the searching by 'AND' keywords

I put this here for the rest of the find command functionality, like searching the archive/bin

ah, i see. yes that is an issue.

part of https://github.com/CS2103JUN2017-T2/main/issues/114


[GHA] test build

Description of the Change

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

/build clang


for firefox selenium tests.  Would the userpref permissions.default.image = 2 affect training?

...as well as gfx.downloadable_fonts.enabled = false
downloaded font files can make up a sizeable portion of a webpage size as well which is why I mention them too.
I wondered if disabling loading images would save bandwidth or make each site's load quicker during tests but not affect the actual test results.

Previously: #62.


Support for decorators

Is there a way to activate support for decorators. Using the latest version of the formatter, I have the following error:
     24 |    @MyAnnotation()
             ^ Unexpected '@'.

@Verdier what version did you use? I'm using 0.3.2. it can process decorators. see https://github.com/vvakame/typescript-formatter/commit/bec27f05b80488ea5a43d6923befc20bbea3c842

@vvakame that's true, I'm sorry. I've try with 0.3.1 yesterday... the new version was just published. Thank's for the project! And happy to use decorators with code formatting!

:+1:


Fixed multiline title issue

Summary of the Pull Request
Fixed issue with title spanning multiple lines in search result

PR Checklist

[x] Applies to #3729
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed

Manually validated that titles do not span multiple lines in result view.
Manually validated that tests are passed.

can we get a screenshot on this?


round corners on divider line in tablet layout

minor nitpick pull request! =)
on tablets, a snackbar has round corners. with this commit, a snackbar with a top line ('divider') will also have round top corners

great! :)


Log the signal that caused the dirty exit

LGTM but let's also pass a custom exception into this unregister_worker call to differentiate it?


How do I view logs?

Just want to have a little file that shows who connected from what IP and what time. That is all. Thank you!

Found out how to to it! Edit /etc/openvpn/server.conf and add the last line

Thanks for the issue, added here: https://github.com/Angristan/OpenVPN-install/commit/d8d0bbb5dad57efd0b962d46bdaa7d0ed6319c85


Bug with collateral and locking (KeyWallet)

It looks like UTxOs ordering affects which one gets chosen in selectCollateral for KeyWallet. This leads to a bug with SignMultiple: it fails with one ordering but succeeds with another.
See this example: https://github.com/Plutonomicon/cardano-transaction-lib/blob/bef018efa5eb82a086b690d6054e16035daec81d/test/Plutip.purs#L261
(introduced in #663)

I just discovered a slightly related bug: https://github.com/Plutonomicon/cardano-transaction-lib/issues/723
We only considered the case when collateral is provided by a single UTxO, but it's not actually a requirement of CIP-30.
We will need to re-consider this issue after I fix the current bug, because we may benefit from being able to use multiple utxos as collateral in KeyWallet


export.py: fix custom SDPA type conversion logic & re-enable for bfloat16

q.dtype is always torch.float at this point. Clearly meant to use the input dtype.

Stack from ghstack (oldest at bottom):

(to be filled)


Create component for text

Reason (Why?)
Currently About, Imprint and Privacy are including ContainerWithContent.scss, which indicates, that we need a component to display text in, which takes the text to be shown as input
Solution (What?)
Create a new component.
Maybe we can use slots to insert the markup-content.
Relation to other Issues
#286

I don't thing we need a component for this. This is more an issue about css refactoring. See #297.
We have to look about the css. Some css are located in components does not fit the scope of the components and has impact to other component. Also we could create some global css which could also include the ContainerWithContent.scss, so that we don't have to include the file in the specific components.
Furhtermore, we could look about some classed from vuetify which maybe can help remove some unessaray created css.


1.0.0-beta.11 is incorrectly not a valid version

The .11 is the problem. When you change it to +11 it works. But as I read the specs it should allow .11. Another lib does it correctly .package(url: "https://github.com/mrackwitz/Version.git", from: "0.8.0")

Good find!
I misread the important part about 'dot-separated` in the spec for prerelease identifiers.


fix(HLS): Fix seekRange for EVENT playlist not using EXT-X-PLAYLIST-TYPE

This change cherry-picked cleanly to v4.2.x, but did not build. If you think the fix is needed in v4.2.x, please create a new PR against that branch to backport it.


Export of query results

In the TF-Browser you can export your query results to Excel-friendly tab-separated files.
This should also be possible from the API that runs in a notebook.

There is one complication: in order to determine which features to export with the result nodes, you have to know the query. In the TF browser we know that.
In a Jupyter notebook, the results = A.search(query) performs a query, after which an A.table(results) or A.show(results) can be done. We want to add an A.export(results).
But in this context the set of features of the query is no longer known. One can also call these functions on an arbitrary iterable of node tuples, without any previous query.

The solution:
I will give the A.search(query) a side effect: it will do an A.displaySetup(extraFeatures=features) for the features in the query. From then on, all calls to A.table() and A.show() will "see" these extraFeatures settings, and display them in results. Likewise, the new A.export() can use that.

Usually, you call A.table(results) for the results of a previous query. So the extraFeatures will have the right features in them: those of the last query. If you have run a query in between, or if you chain searches like this:
results = A.search(query1) + A.search(query2)
A.table(results)

then the features of query2 will be in effect the moment you call A.table.
If that is not OK, you are on your own.

I will also make a function A.displaySettings() by which you can retrieve the current display settings.
The chained query can then be done as follows:
results1 = A.search(query1)
features1 = set(A.displaySettings('extraFeatures'))
results2 = A.search(query2)
features2 = set(A.displaySettings('extraFeatures'))
A.displaySetup(features1 | features2)

This is clumsy. In practice it might be handier to just inspect your queries and do a single display setup:
features = 'sp pdp gn'
results = A.search(query1) + A.search(query2)
A.displaySetup(features)
A.export(results)

A nice benefit it also that if you run pretty() after a query, it shows all features used in that query.
This reduces the need to call displaySetup() manually.


training epoch

Hi,
First of all, congratulation for your acception in ICCV. This work looks nice.
I have two questions,

In your paper, these models were trained for 150 epochs while this repo indicates 300epoch.

Which one is correct?

This repo notices the dependency of NVIDIA DALI which is not recommended by you. Without the DALI, I can train the model?

Thanks for your reminder and sorry for my previous typo, the training period should be 150 epochs for reproduction which is correct in the training command line.
DALI is optional for you to use which is not necessary, while PyTorch default dataloader will be more stable in my case.

@d-li14 thanks for your fast reply :)

@d-li14
I have another question.
If I use pytorch default dataloder, data augmentations are only randomcrop and flip?
In your paper, you use scale & aspect ratio augmentations.
The results in your repo are trained by pytorch default dataloader or NVIDIA DALI?

I use the PyTorch dataloader to get all the released results, data augmentations with respect to scale and aspect ratio are included in RandomResizedCrop.

@d-li14
Thank you so much!


user profile modal "manage user" tab should offer to reactivate deactivated users

Following #26031, it currently offers to deactivate the already-deactivated user, which is confusing.
We might also want to do something to show more clearly in the "manage user" panel that the user you're working on is deactivated; I'm not sure.

Can it be solved by creating a new function is_active in people.ts like is_active_user_for_popover and then disabling 'deactivate' button in 'manage user' panel after checking if user is active in settings_users.show_edit_user_info_modal?

@zulipbot claim


Slow update speed

The speed at which ADSB data is sent to FTS is slow. Running dump1090-fa and looking at the local web interface, it shows that multiple updates are received each second for every visible aircraft. Despite having sub 1 second position updates for all aircraft, aircraft position information is uploaded/updated on the server at a rate of one aircraft every 2-5 seconds using the http json method. Using tcp+beast results in an upload/update rate of one aircraft every 5-15 seconds and typically the application hangs after 4-6 aircraft are uploaded. Using tcp+raw results in a hang after the first aircraft is uploaded. The script doesn't generate any errors when it hangs.
The slow update speed does not appear to be a server resource limitation. While I have been testing on a Pi4, this same issue is (was?) occurring on the public server which I believe is more powerful than a Pi. In further support of this, it is possible to simultaneously run adsbcot and aprscot on the same pi without any negative impact on the rate of position updates, despite the server being subjected to twice the load. Conversely, stopping aprscot will not improve the refresh rate of adsbcot. Sending multiple cots and map objects using an EUD will also not have any negative impact on the rate of updates coming from adsbcot.
A significant problem will arise from this slow update rate in the form of a queue developing. It would seem that when a poll is performed, a snapshot in time is taken of all currently received aircraft. This list of aircraft is then sent slowly, one by one, to the server at a rate of one aircraft every 2-5 seconds. If another poll is performed before this list is finished being sent, the new snapshot is simply added to the end of the current list. As time goes on the list grows, and the positions of the aircraft begin to lag behind their real world locations. The result is the appearance of aircraft traveling in slow motion. Attempting to reduce the polling interval in order to update the aircraft positions more frequently will worsen the issue as it adds more snapshots to the queue. The workaround for this is to estimate the largest number of aircraft that might be simultaneously received, and plan for a large enough polling interval that the full queue will be cleared before the next poll. In high traffic areas this will necessitate very slow updates, and at times when the skies are relatively clear there will be large needless delays in position updates.

Has this been addressed at all? I also have an issue with the output to takserver updating very slowly. There are often new aircraft that are not uploaded to tak, and old aircraft that have not cycled off of tak.

Hi everyone,
Seems i have the same issue on my side. I use a RaspPi with ADSBxchange installed on it. I have interfaced my Pi with my ATAK Server by reading the aircraft.json file : FEED_URL = file:///run/readsb/aircraft.json
TCP+RAW and other methods are not working on my side.
The problem is after 2-3 minutes, I notice some delays between the aircraft seen on the dump1090 webpage and the aircrafts on ATAK. After a while, there is approx 45min to 1 hour delay between the two systems. With other words, on dump1090 i can see the real-time traffic but on ATAK i can see the traffic that past 1 hour before. For example, if I disconnect the antenna from the SDR dongle, I continue to see aircraft poping on ATAK during 1 hour.
Do you have an idea what is happening?
Thank for your help :)


Documentation does not coincide with actual website

Hello, I'm trying to follow the instructions but I don't see the options that the instructions show, I do not know if it's a permissions or a documentation issue.
Documentation:

Actual admin panel:

Here's the collapsed list:

What's going on? Am I missing something? Do I need to ask the admin of all admins to enable this for me?
Thanks,

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

ID: b3530b55-59c8-6eec-b257-b7a5f2ed046b
Version Independent ID: 022f7eff-f6c2-9a73-afa9-74e887b761e1
Content: Prepare your Microsoft 365 tenant - Teams
Content Source: msteams-platform/concepts/build-and-test/prepare-your-o365-tenant.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Yes, definitively a permissions issue.

It would be nice to "see" the teams app links but have a different message display, not a documentation issue but rather a product UX issue.
Thanks you!


What to do about no-shows?

This is a question that has been brought up to me a few times, and the answer from me was "I dunno, give them a loss?"
I figured it was worth asking, since not all tournaments are made perfect - what do you do if you have a no-show? Score a loss for them (ie, 1/0)? Remove them (although this has other implications, esp. if later in the tournament)?
Is it even worth scoping such a thing into the tournament base?

Scoring the match a loss for the no-show is the correct thing to do as far as propagation is concerned, and that's what tournament is good at. Removing a player has negative implications.
For user interface stuff; displaying a no-show, an abandoned player, or allowing a different scoring method, that's probably something for you to decide. I don't really want metadata like this to creep into tournament, and you can probably keep some map of match ids to metadata if you want a richer UI. That's my initial opinion of this at least.

That's totally fine. I wanted to double check before doing anything because if this was planned or scoped I didn't want to do something incompatible. Thanks for the comment!


Added TOPO NFTs

Added TOPO

I am closing this request as it has nothing to do with NFT at all. Technically it is just stake pool advertising.


Message 'Message' seems causing a conflict with io.protostuff.Message.

If I have something like the following, there would be an issue.
message Message {
required string messageId = 1;
}
Cheers
Seong

Hi.
Will gladly accept a pull request that fixes this issue.
It can be fixed by changing code generator to use FQN - io.protostuff.Message instead of Message + import. In general, it is safer to do not use imports in generated code at all.
P.S. This issue is already fixed this in new major protostuff-compiler version, but is not finished yet.

Thank you for the valuable information.
I am also glad that it's already been fixed.
Cheers
Seong


Git clone fails with "Connection reset by peer"

Component
agent, other
Describe the bug
I'm pretty sure that this is a bug in the plugin-git image or in my networking setup on my computer.
My git server is only available in ipv6 on my local network (as there is a bug on my router that prevent connexion to itself in ipv4).
Clonning from a empty alpine container works, clonning from the host works too.
Is this related to the other ipv6 issues ?
Sorry if this is just an error in my setup but I can't find any way to get this working.
System Info
Server build from source "https://github.com/woodpecker-ci/woodpecker"
version "next-328f1044" (I'm pretty sure that I build that on the v0.15.5 git tag)
The server is running on an alpine linux aarch64.

The agent is running on my main computer.
OS: Archlinux AMD64
Output of `woodpecker-agent -v`: `woodpecker-agent version dev` (I just took the package from the community repo)
I use podman as a drop-in replacement for docker

Additional context
+ git init -b main
  Initialized empty Git repository in /woodpecker/src/git.eragon.re/eragon/woodpecker-test/.git/
  + git remote add origin https://git.eragon.re/eragon/woodpecker-test.git
  + git fetch --no-tags origin +refs/heads/main:   fatal: unable to access 'https://git.eragon.re/eragon/woodpecker-test.git/': OpenSSL SSL_connect: Connection reset by peer in connection to git.eragon.re:443

Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A join our Discord Chat Server or the Matrix room.

we will publish plugin-git v2.0.0 soon ... I'm alos gona bump it in the v0.15.x release branch...
if it's not fixed by then ... we can revisit that issue

@Eragonfr can you check if it works now?

I can't get it to work...
Also I can't find how to test with the woodpecker-agent:next docker image, it seems that the server doesn't allow the agent to register, so I can't even test if enabling IPv6 networking in the agent is enough to fix it.
I'm almost certain that the problem could be solved by using IPv6 in the bridge networks made by the agent.

hmm if you are on next ... try export WOODPECKER_BACKEND_DOCKER_ENABLE_IPV6=true

I tried the server and the agent on next withWOODPECKER_BACKEND_DOCKER_ENABLE_IPV6 and I still got the same error.
Is it possible that the git binary in the container does not use IPv6 by default and fail because of the broken IPv4 ?
At this point I don't understand what's happening in the container.
Result of woodpecker-agent --version : woodpecker-agent version next-f1e9c841
Here is the log of the agent when run with WOODPECKER_BACKEND_DOCKER_ENABLE_IPV6=true :
6:28PM INF ../../data/projects/woodpecker/cmd/agent/agent.go:180 > Starting Woodpecker agent with version 'next-f1e9c841' and backend 'docker' running up to 1 pipelines in parallel
6:28PM DBG ../../data/projects/woodpecker/cmd/agent/agent.go:164 > loaded docker backend engine
6:28PM DBG ../../data/projects/woodpecker/cmd/agent/agent.go:171 > polling new steps
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:61 > request next execution
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:97 > received execution id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:111 > listen for cancel signal id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:75 > Executing 2 stages, in order of: ID=4 Pipeline=2 Repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:85 > stage ID=4 Pipeline=2 Repo=eragon/woodpecker-test Stage=wp_0_314432866923000364_clone_0 Steps=wp_0_314432866923000364_clone_0
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:85 > stage ID=4 Pipeline=2 Repo=eragon/woodpecker-test Stage=wp_0_314432866923000364_stage_0 Steps=wp_0_314432866923000364_step_0
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:157 > Prepare ID=4 Pipeline=2 Repo=eragon/woodpecker-test Step=wp_0_314432866923000364_clone_0
6:28PM DBG ../../data/projects/woodpecker/agent/tracer.go:51 > update step status exit_code=0 exited=false id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/agent/tracer.go:59 > update step status complete exit_code=0 exited=false id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:184 > Executing ID=4 Pipeline=2 Repo=eragon/woodpecker-test Step=wp_0_314432866923000364_clone_0
6:28PM DBG ../../data/projects/woodpecker/agent/logger.go:53 > log stream opened id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM TRC ../../data/projects/woodpecker/pipeline/rpc/line.go:69 > grpc write line: + git init -b main
 ID=4 name=git
6:28PM TRC ../../data/projects/woodpecker/pipeline/rpc/line.go:69 > grpc write line: Initialized empty Git repository in /woodpecker/src/git.eragon.re/eragon/woodpecker-test/.git/
 ID=4 name=git
6:28PM TRC ../../data/projects/woodpecker/pipeline/rpc/line.go:69 > grpc write line: + git remote add origin https://git.eragon.re/eragon/woodpecker-test.git
 ID=4 name=git
6:28PM TRC ../../data/projects/woodpecker/pipeline/rpc/line.go:69 > grpc write line: + git fetch --no-tags --depth=1 --filter=tree:0 origin +refs/heads/main:
 ID=4 name=git
6:28PM TRC ../../data/projects/woodpecker/pipeline/rpc/line.go:69 > grpc write line: fatal: unable to access 'https://git.eragon.re/eragon/woodpecker-test.git/': Could not resolve host: git.eragon.re
 ID=4 name=git
6:28PM TRC ../../data/projects/woodpecker/pipeline/rpc/line.go:69 > grpc write line: exit status 128
 ID=4 name=git
6:28PM ERR ../../data/projects/woodpecker/agent/logger.go:58 > copy limited logStream part error="io: read/write on closed pipe"
6:28PM DBG ../../data/projects/woodpecker/agent/logger.go:61 > log stream copied id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/agent/logger.go:77 > log stream uploading id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/agent/logger.go:81 > log stream upload complete id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/agent/logger.go:85 > log stream closed id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:190 > Complete ID=4 Pipeline=2 Repo=eragon/woodpecker-test Step=wp_0_314432866923000364_clone_0
6:28PM DBG ../../data/projects/woodpecker/agent/tracer.go:51 > update step status exit_code=128 exited=true id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/agent/tracer.go:59 > update step status complete exit_code=128 exited=true id=4 image=woodpeckerci/plugin-git:2 pipeline=2 repo=eragon/woodpecker-test stage=git
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:157 > Prepare ID=4 Pipeline=2 Repo=eragon/woodpecker-test Step=wp_0_314432866923000364_step_0
6:28PM DBG ../../data/projects/woodpecker/pipeline/pipeline.go:164 > Skipped due to OnFailure=false error="wp_0_314432866923000364_clone_0 : exit code 128" ID=4 Pipeline=2 Repo=eragon/woodpecker-test Step=wp_0_314432866923000364_step_0
6:28PM ERR ../../data/projects/woodpecker/pipeline/backend/docker/docker.go:238 > could not kill container 'wp_0_314432866923000364_clone_0' error="Error response from daemon: can only kill running containers. dcf0160133783e6afdf6fd75bf169ee861246143da714aaf699787e8c8a34c7a is in state exited: container state improper"
6:28PM ERR ../../data/projects/woodpecker/pipeline/backend/docker/docker.go:238 > could not kill container 'wp_0_314432866923000364_stage_0' error="Error response from daemon: no container with name or ID \"wp_0_314432866923000364_step_0\" found: no such container"
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:185 > pipeline complete error= canceled=false exit_code=128 id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:187 > uploading logs id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:189 > uploading logs complete id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:194 > updating pipeline status error= exit_code=128 id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM ERR ../../data/projects/woodpecker/pipeline/rpc/client_grpc.go:97 > grpc error: wait(): code: Unknown: rpc error: code = Unknown desc = Step finished with exitcode 128,  error="rpc error: code = Unknown desc = Step finished with exitcode 128, "
6:28PM WRN ../../data/projects/woodpecker/agent/runner.go:115 > cancel signal received error="rpc error: code = Unknown desc = Step finished with exitcode 128, " id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:127 > pipeline done id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:199 > updating pipeline status complete id=4 pipeline=2 repo=eragon/woodpecker-test
6:28PM DBG ../../data/projects/woodpecker/cmd/agent/agent.go:171 > polling new steps
6:28PM DBG ../../data/projects/woodpecker/agent/runner.go:61 > request next execution

hmm could be related to dns
often docker images use external dns servers ...
for example https://codeberg.org/woodpecker-plugins/plugin-docker-buildx/pulls/45 was added, to use the internal dns

I don't think that it is the issue here, except if for some reason the dns does not replicate the content of the OVH dns servers.

does it still exist?

Don't know, I moved to forgejo and act_runner, no longer using podman nor docker too.

It seems I bumped into this issue. I am also trying out podman instead of docker, and the git clone step fails on a aarch64/arm64 host:
+ git init -b main
Initialized empty Git repository in /woodpecker/src/git.vdx.hu/voidcontext/nix-config/.git/
+ git config --global --replace-all safe.directory /woodpecker/src/git.vdx.hu/voidcontext/nix-config
+ git remote add origin https://git.vdx.hu/voidcontext/nix-config.git
+ git fetch --no-tags --depth=1 --filter=tree:0 origin +a4b83a869d6366bf24921b8f4f3f5836f806f57e:fatal: unable to access 'https://git.vdx.hu/voidcontext/nix-config.git/': Could not resolve host: git.vdx.hu
exit status 128

Build: https://woodpecker.ci.vdx.hu/repos/109/pipeline/119/5

please open a new issue and link to old issues if you thin they are related - thanks :)
PS: do you run the woodpecker server on the same host as the arm agent? -> https://woodpecker-ci.org/docs/administration/forges/forgejo#forgejo-on-the-same-host-with-containers


[TODO] docs

https://github.com/unified-moderation-network/salamander/blob/79a5a6024c12fe639780149856f81d90862af579/src/bot.py#L1121-L1144
This needs documentation explaining the what and why here, copy local notes in as appropriate comments later.

Resolved by fd0e291


LL-1392  Sendmax functionality

In the current state of common, it only works with bitcoin family since the ethereum bridge doesn't implement a way of setting the useAllAmount extra (~i think this is the cause~ it was). Once https://github.com/LedgerHQ/ledger-live-common/pull/296 is merged in common and we bump it, this pr would work for eth too.

Type
Feature
Context
https://ledgerhq.atlassian.net/browse/LL-1392
Parts of the app affected / Test plan
Send flow

Everything except the wording check and the 0.99 comment updated, I'll put another pr for the comment to not mix things up, so if @dasilvarosa approves the wording, i'm done with this pr (🤞  @Arnaud97234 )

<EMAIL_ADDRESS>

@IAmMorrow will test it a bit. at worse, we'll re-review wording at topic/erc20 merge time


Timeouts and kicks are ignored by EiraIRC

If you are connected to a server and time out/get DCed, EiraIRC will not notice (until the next message is sent? Not sure on this), to the point of rejecting a /sirc connect <server> because it's "already connected to "
If you are in a channel and get kicked, EiraIRC will also not notice, and will also reject a /sirc join <channel>, saying "You have joined  in  already."

Hey


No license field when attempting to generate a PetStore client

Hello,
I am trying to generate a petstore client based on the following OpenAPI definition: https://petstore.swagger.io/v2/swagger.json in a fresh Fern workspace. I am using Fern version 0.11.2, and the following (automatically generated) generators configuration:
default-group: local
groups:
  local:
    generators:
      - name: fernapi/fern-typescript-node-sdk
        version: 0.6.4
        output:
          location: local-file-system
          path: ../../generated/typescript
      - name: fernapi/fern-openapi
        version: 0.0.28
        output:
          location: local-file-system
          path: ../../generated/openapi

I am running Windows 10 22H2 and am using PowerShell as my commandline.
I would expect the client to be generated, but instead receive the following output:
PS C:\Source\petstore> fern init --openapi https://petstore.swagger.io/v2/swagger.json
Created new API: ./fern\api
PS C:\Source\petstore> fern generate
[api]: pet.yml -> service -> endpoints -> findPetsByStatus
       Endpoint path /pet/findByStatus conflicts with other endpoints:
         - pet.yml -> getPetById /pet/{petId}
[api]: pet.yml -> service -> endpoints -> findPetsByTags
       Endpoint path /pet/findByTags conflicts with other endpoints:
         - pet.yml -> getPetById /pet/{petId}
[api]: pet.yml -> service -> endpoints -> getPetById
       Endpoint path /pet/{petId} conflicts with other endpoints:
         - pet.yml -> findPetsByStatus /pet/findByStatus
         - pet.yml -> findPetsByTags /pet/findByTags
[api]: user.yml -> service -> endpoints -> getUserByName
       Endpoint path /user/{username} conflicts with other endpoints:
         - user.yml -> loginUser /user/login
         - user.yml -> logoutUser /user/logout
[api]: user.yml -> service -> endpoints -> loginUser
       Endpoint path /user/login conflicts with other endpoints:
         - user.yml -> getUserByName /user/{username}
[api]: user.yml -> service -> endpoints -> logoutUser
       Endpoint path /user/logout conflicts with other endpoints:
         - user.yml -> getUserByName /user/{username}
[api]: fernapi/fern-openapi Downloaded to C:\Source\petstore\generated\openapi
[api]: fernapi/fern-typescript-node-sdk Command failed with exit code 2: yarn build
                                        warning package.json: No license field
                                        error Command failed with exit code 2.
                                        yarn run v1.22.18
                                        $ tsc
                                        src/api/resources/pet/client/Client.ts(404,17): error TS1117: An object literal cannot have multiple properties with the same name.
                                        Files:                         219
                                        Lines of Library:            27421
                                        Lines of Definitions:        37010
                                        Lines of TypeScript:          3722
                                        Lines of JavaScript:             0
                                        Lines of JSON:                   0
                                        Lines of Other:                  0
                                        Nodes of Library:           118506
                                        Nodes of Definitions:        90319
                                        Nodes of TypeScript:         15901
                                        Nodes of JavaScript:             0
                                        Nodes of JSON:                   0
                                        Nodes of Other:                  0
                                        Identifiers:                 79224
                                        Symbols:                     59610
                                        Types:                        8581
                                        Instantiations:              31575
                                        Memory used:               123715K
                                        Assignability cache size:     4012
                                        Identity cache size:           182
                                        Subtype cache size:            260
                                        Strict subtype cache size:      92
                                        I/O Read time:               0.03s
                                        Parse time:                  0.86s
                                        ResolveModule time:          0.06s
                                        ResolveTypeReference time:   0.01s
                                        Program time:                1.05s
                                        Bind time:                   0.50s
                                        Check time:                  1.04s
                                        transformTime time:          0.28s
                                        commentTime time:            0.09s
                                        I/O Write time:              0.04s
                                        printTime time:              0.69s
                                        Emit time:                   0.69s
                                        Total time:                  3.28s
                                        info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
┌─
│ x  fernapi/fern-typescript-node-sdk
│ ✓  fernapi/fern-openapi
└─

Note that the directory I started with is an empty directory - the full reproduction scenario is provided in the commandline output.
Edit: the same error occurs when running with generator version 0.7.1.

I am having this exact same issue. Did this get resolved?

@aevitas I am getting the same issue as you had here? Do you know how to fix it?


⚠ Avatar API Endpoint has degraded performance

In e213265, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1581 ms

Resolved: Avatar API Endpoint performance has improved in 7b6385a.


PEAR-2090 + 2103: Responsive Mutation Frequency App

Description
I couldn't quite change the content of the bar chart. Hence, they get a bit squished for smaller screens. We still get the clear tooltips though.
Checklist

[N/A] Added proper unit tests
[N/A] Left proper TODO messages for any remaining tasks
[N/A] Scanned for web accessibility with aXe, and mitigated or documented
flagged issues

Screenshots/Screen Recordings (if Appropriate)

I took a quick look and it looks good, but I defer to @cemile-ctds .
One thing I noticed that was happening to me but I didn't see in your screenshot. I'm not sure if it was intended -- after i add a mutation or gene for the Upload X filters, I see what seems to be where a scrollbar would show up?


Update README.adoc

@MaheepGupta Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

This change is not necessary.


What's joint model group name of uf850

Sorry i'm new to this.
I'm trying to convert moveit2 tutorial panda_arm to uf850, so basically it's this:
self.panda = MoveItPy(node_name="moveit_uf850", config_dict=moveit_config)
self.panda_arm = self.panda.get_planning_component("uf850") # The error occurs here
# self.panda.get_planning_component()
self.planning_scene_monitor: PlanningSceneMonitor = (
    self.panda.get_planning_scene_monitor()
)

The error goes with:
self.panda_arm = self.panda.get_planning_component("uf850")
RuntimeError: Could not find joint model group 'uf850'.

I saw that in xarm_moveit_config/srdf/_uf850_macro.srdf.xacro it says group name is {prefix}uf850 and the prefix is actually an empty string so the group name should be uf850? But it do not work, i've tried also xarm/controllers_initial_group_+xarm&uf850..., none of them works.
So how can i solve it?

Hi @ogios, it seems 'uf850' is correct for the group name. Could you please show us the specific procedures to reproduce the issue? Like what is the launch file and arguments given before executing this python script? It can imply what nodes and parameters are loaded and running in the background. And could you show us the full python code (can be a minimum version just to catch the error)?

@penglongxiang Thanks!
And as i go through the uf_ros_lib MoveItConfigBuilder, i found that it's because i should pass in the robot_type & dof as xarco arguments.
Now i can get this work.


make generic pipelines to DAG rendering work with Airflow 2.x

Describe the issue
Currently, even when using generic pipeline editor with python files and notebooks, the rendered DAG is not compatible with Airflow 2.x
 Broken DAG: [/opt/airflow/dags/repo/SvensWetterapprootpvc-0202162036.py] Traceback (most recent call last): File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed File "/opt/airflow/dags/repo/SvensWetterapprootpvc-0202162036.py", line 2, in <module> from airflow.contrib.kubernetes.volume_mount import VolumeMount ModuleNotFoundError: No module named 'airflow.contrib.kubernetes'
To Reproduce
Steps to reproduce the behavior:

Create a generic pipeline
Submit it to the Airflow runtime with Airflow 2.x

Screenshots or log output
If applicable, add screenshots or log output to help explain your problem.
Log Output

Paste the log output here.

Expected behavior
DAg is rendered correctly for use in Airflow 2.x
Starting with jinja template
https://github.com/elyra-ai/elyra/blob/main/elyra/templates/airflow/airflow_template.jinja2
and elyra/elyra/pipeline/airflow/processor_airflow.py
possible hints
https://airflow.apache.org/docs/apache-airflow-providers-cncf-kubernetes/stable/operators.html
https://airflow.apache.org/docs/apache-airflow/2.5.3/howto/upgrading-from-1-10/index.html
https://airflow.apache.org/docs/apache-airflow/2.5.3/core-concepts/dags.html
https://github.com/apache/airflow/pull/21653
Deployment information
Describe what you've deployed and how:

Elyra version: latest
Installation source: [e.g. PyPI, conda, from source, official container image, custom container image]
Deployment type: Open Data Hub pre-RedHat takeover
Operating system: [e.g. macos, linux]

My team was able to get generic pipeline DAGs to work with Airflow 2.5.3 on this commit (https://github.com/kflow-ai/elyra/commit/f9d132954e008d30145f18794aa543d97f121a5f). We don't have unit tests working locally, so this commit isn't ready for a PR. Perhaps we can work with an active maintainer to get this in proper shape for an upstream merge?
Off the top of my head, we need automated tests to work and potentially test with both earlier and later versions of Airflow. My understanding from this comment is we may actually want to take a different approach to keep Airflow 1 support at the same time.

Do we really need to test it with Airflow 1.x?
By the way, your code looks really good, I have exactly that in my local code base right now :-)

@ianonavy yup, we will certainly work with @lresende

@ianonavy "Off the top of my head, we need automated tests to work and potentially test with both earlier and later versions of Airflow." we will deprecate airflow 1.x support with elyra 4.0 and I am thinking of adding more unit tests aside from the existing ones, too.

@lresende I will add some new documentation to this PR changing some things in
https://elyra.readthedocs.io/en/latest/recipes/configure-airflow-as-a-runtime.html
so it is clear what is now Airflow 2 ready. Plus also, some configuration options in Airflow 2 have changed etc.


feat(tests): add euler-206 solution

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

This adds a solution for Euler 206.

Spelling big words is hard, ok :laughing:


vivaldi sha256 fix

Just like #9292 and #9308, the checksum has changed (again)
but the version number appears the same
I've updated the sha256 to the most recent value
Perhaps we should change the sha256 to :no_check?

We should, yes. Would you care to do it? Please don’t forget the comment.

Thanks. Merged as c87a5cc.


Update golang.org/x/net to v0.33.0

Why we need this PR

Fetch Golang fix to non-linear parsing.
Changes made

Update golang.org/x/net to v0.33.0 and module dependencies.
Which issue(s) this PR fixes

Test plan

/retest

/lgtm

/cherry-pick release-0.9


Upgrade to controller-runtime v0.17.2

Upgrades controller-runtime to v0.17.2.
/kind feature
/priority important-soon
/assign

/approve