[Bug?] State is not updated when manually updating App

Bevy version
v0.14.0-rc.2
What you did
My library has a similar to the following test code, but it failed after I updated the bevy version from v0.13.2 to v0.14.0-rc2.
use bevy::prelude::*;

fn main() {
    let mut app = App::new();
    app.add_plugins(MinimalPlugins);
    app.init_resource::<StateChanged>();
    app.init_state::<GameState>();
    app.add_systems(Update, wait_state);
    app.update();
    assert!(!app.world().resource::<StateChanged>().0);
    app.insert_state(GameState::InGame);
    app.update();
    // panic!!
    assert!(app.world().resource::<StateChanged>().0);
}

#[derive(Resource, Default, Debug)]
struct StateChanged(bool);

#[derive(Eq, PartialEq, Debug, Default, Copy, Clone, Hash, States)]
enum GameState {
    #[default]
    Title,
    InGame,
}

fn wait_state(
    mut changed: ResMut<StateChanged>,
    state: Res<State<GameState>>,
) {
    if matches!(state.get(), GameState::InGame) {
        changed.0 = true;
    }
}

What went wrong
Console output:
thread 'main' panicked at src\main.rs:19:5:
assertion failed: app.world().resource::<StateChanged>().0

FYI @lee-orr @benfrankel @MiniaczQ

@alice-i-cecile - I believe the source of this is App::new() doesn't set up the main schedule, so StateTransition isn't added in.
Do you think a good solution here is documenting the need to set up that schedule when you aren't building on the main schedule, or ensuring it's run by default even if no meta-schedule has been added?

I agree with your diagnosis of the problem. IMO we should warn about this during .init_state / insert_state. Users won't read the docs for this sort of weirdness, but they will check for warnings.

I believe the source of this is App::new() doesn't set up the main schedule, so StateTransition isn't added in.

Not quite, StateTransition is no longer part of the default main schedule, it's added through StatesPlugin
Can the state installation methods check for plugin existence?

is_plugin_added is the method you want :)

Great, I'll have a fix in 10 min

For the author, you want to add StatesPlugin manually when using MinimalPlugins


Remove @-mentions from jira integration.

Because of a lot of annoying notifcations we decided to remove
all @-mentions in jira. We replaced it by just login (without @).

Merged, thanks @TomaszKolek!


AuthBoss as an OAuth2 provider

Hello,
Is authboss also an OAuth provider: i.e. can register server with client Id and secret.
Or its OAuth features are limited to use other providers like Google, Facebook etc...
Thanks!
Jean-Claude

Authboss is simply an OAuth2 client. It has no server capabilities in that regard. :)

@aarondl : Thank you for your response..


Update description for load testing extension

#sign-off

@Taojunshen can you please review and merge this?

@ninallam, Thank you for working to improve Azure CLI documentation, however, this GitHub file is autogenerated from the source code.  There are several Azure CLI extension repos, but looking in Azure/azure-cli-extensions is probably where you want to make this change.  A second option is to open an extension GitHub issue .


Different types of area graphs

Hi
I am planning to create area graphs using this library like Step-Area Charts, Spline-Area Charts, Streamgraphs, and Smoothed-Line Charts.
Unfortunately, I've been struggling to find a straightforward method to create these graphs, and I'm also facing challenges with labeling the chart and implementing click-related animations. If anyone has any suggestions or solutions to these issues, I would greatly appreciate your input. Thank you in advance!

Can you elaborate more? Do you have link to image of that chart?


feat: admin delete range of checkins

Brief description. What is this change?

Delete a Date Range of Shifts for Admin Fridge Check-ins

Implementation description. How did you make this change?

Added route called /admin/delete-check-ins
Added DeleteCheckIns.tsx - similar to create checkin page

Steps to test

Create checkins
Sign in as admin and go to /admin/delete-check-ins
Enter date range to delete checkins

startdate == enddate
no shifts in given range
check that deletion is inclusive of startdate and enddate

Checklist

[ ] My PR name is descriptive and in imperative tense
[ ] My commit messages follow conventional commits and are descriptive. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
[ ] I have run the appropriate linter(s)
[ ] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR
[ ] The appropriate tests if necessary have been written

If you look at my recent changes for create checkin PR, we can simplify some of the code for this PR as well by not using useForm cuz dont think its needed for a simple screen like this. We generally used useForm in screens where there were multiple steps (e.g. account sign up, donation scheduling). This way we wont have the useEffect function and also the handleDateRange one


Implement comparison operators ==, !=, <, >, <=, >=

Operators return 1 if true and 0 if false. Operators return undef if one of the operands is undefined. Returned type is byte.

Implemented by f56f7cc.

Hmm.
I had very similar code, but that was in a function called eval(), which then called upon
expr() and friends.  So, only if going through eval() we get the comparisons, such as
when you do
.if
and such.
I think your method is beter, since we keep a single expr() ..

One thing I do not like about my solution is that < and > now have a different meaning depending on context. a) byte select and b) as comparison operator. But I don´t like things like LT, GT, NE, EQ etc. neither. So I probably leave it as it is now.
I want to leave it as byte select because it is VERY common in the Commodore world.


Describe the tasks/roles/responsibilities of the board

Moving from https://github.com/uio-carpentry/governance/issues/14
During the 200325 board meeting it became clear that this description is needed.

I think we should have this done (at least: drafted) until the upcoming election in fall 2021!
I'll assign the current Board members to it.

I think the document 'Carpentry@UiO: board and library responsibilities' in the Google Drive (folder 'Board') covers this. I think we can close this issue?


optimize: optimize maven clean plugin to clear the distribution directory

Signed-off-by: slievrly<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
optimize: optimize maven clean plugin to clear the distribution directory
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #3415 (5b8dd8d) into develop (5fb5b38) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #3415      +/-   ##
=============================================
- Coverage      51.54%   51.44%   -0.10%
- Complexity      3340     3341       +1
=============================================
  Files            616      616
  Lines          20247    20221      -26
  Branches        2540     2493      -47
=============================================
- Hits           10436    10403      -33
+ Misses          8758     8745      -13
- Partials        1053     1073      +20

Impacted Files
Coverage Δ
Complexity Δ

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterRMRequestConvertor.java
90.47% <0.00%> (-4.77%)
3.00% <0.00%> (ø%)

...tobuf/convertor/BranchReportResponseConvertor.java
95.45% <0.00%> (-4.55%)
3.00% <0.00%> (ø%)

... and 27 more


chore(deps): update renovatebot/github-action action to 34.60.0

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
minor
v34.59.0 -> v34.60.0

Release Notes

renovatebot/github-action
v34.60.0
Compare Source
See the the changelog for changes in all releases.
34.60.0 (2022-12-16)
Bug Fixes

deps: update renovate/renovate docker tag to v34.60.0 (e9c2dae)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@balena-ci I self-certify!


fix(storybook): various react story fixes

Related Ticket(s)
#9119
#9117
#9116
Description
Fixing some of the React Stories:

Feature Card
Content group
Content Section

Changelog
Changed

changing the way parameters are being passed to the stories

@IgnacioBecerra Looks like the cta knob for content-group isn't working.

@IgnacioBecerra Looks like the cta knob for content-group isn't working.

you can look how i handled that here: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/074c64ccccb1123d2fc0b0d47c1c996df7b41633/packages/react/src/components/ContentBlock/stories/ContentBlock.stories.js#L64-L74

Had to make a small change to the ContentGroup due to Storybook erroring out when trying out the new solution, but now all knobs should work 👍


add shutdown to close

Description
Reference issue
Fixes #...

@craymond12 Do we not do a graceful close currently without this?

@craymond12 Do we not do a graceful close currently without this?

sorry, i was supposed to push on my fork to test things
Currently when we close the webrtc connection we do not send the shutdown message
This often cause the webrtc stream to hang until timeout on the producer side when the consumer leave
Chrome do send it

@craymond12 we can run the tests here! If you open the PR again I can run the tests and if they pass we can merge things.
Would it be possible to write a test? Otherwise someone might undo your fix accidentally.

@craymond12 we can run the tests here! If you open the PR again I can run the tests and if they pass we can merge things.
Would it be possible to write a test? Otherwise someone might undo your fix accidentally.

this was kind of hackish, would have to think more about the context sent to the shutdown

@craymond12 we can run the tests here! If you open the PR again I can run the tests and if they pass we can merge things.
Would it be possible to write a test? Otherwise someone might undo your fix accidentally.

the root of the issue is a lot of onvif camera support 2 to 4 streams, Having the webrtc hang in the camera for around 30 seconds can be an issue
I intended to explore this behavior in a fork to avoid creating some noise here
But once satisfied I will publish a PR here

@craymond12 we can run the tests here! If you open the PR again I can run the tests and if they pass we can merge things.
Would it be possible to write a test? Otherwise someone might undo your fix accidentally.

I managed to do some test with some real Axis device and it work
I am thinking of having a new peerConnection Close method to which we will be able to pass down the context for cancellation
I will be off for 1 week and in my return i will update the pr and look at the tests

@craymond12 we can run the tests here! If you open the PR again I can run the tests and if they pass we can merge things.
Would it be possible to write a test? Otherwise someone might undo your fix accidentally.

I managed to do some test with some real Axis device and it work
I am thinking of having a new peerConnection Close method to which we will be able to pass down the context for cancellation I will be off for 1 week and in my return i will update the pr and look at the tests

Given how new the method is, you can probably put the context into GracefulClose


Create continuous-integration.md

Describe your changes
Localize Continuous Integration in German
Related issue number or link (ex: resolves #issue-number)
resolves #2532
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Closing this PR in favor of https://github.com/cncf/glossary/pull/2605#issuecomment-1817465624.


Update various API docs

Description
This adds answers in the docs for various questions I had run into over the past few months, in the places where I looked for the answers.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.

Test exempt; documentation fixes.


focus issue w/ input texts on angularjs

         <input name="state" id="state" type="text" class="validate" ng-model="user.state">
         State / Province

this apparent field when pre-filled with scope data overlaps with the current label. trying to trigger the .focus() won't do anything. The only way you can trigger a change would be focusing the input with the keyboard itself.
Any work around with this?

#GMG

When prefilling data in angular you have to also change the class="active" for the label


[REFACTOR]: [Worldline] Remove Default Case Handling

:memo: Feature Description

We utilize match statements to make pivotal decisions, such as generating requests based on the payment method type and managing responses received from the connector.
These conditions generally go hand in hand with enum variants.
Default case is used because a match statement needs to be exhaustive i.e. every variant needs to be covered.
So, if all the explicit cases are handled then default is used to handle the rest.
Each connector have these match statements but many of them don’t provide reference to each variant in their default case, rather a _ is put to handle all the other cases.
This approach carries a risk because developers may inadvertently overlook the need for explicit handling of the new cases.

:hammer: Possible Implementation

Instead of relying on a default match case _, developers should handle each and every variant explicitly.
By doing so, if there are any changes in the future, they can readily assess the impact of their modifications simply by compiling the code.
You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/1955

:bookmark: Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/
:package: Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

:package: Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

:sparkles: Are you willing to submit a PR?

can you assign me this issue?

Hey @nik132-eng , sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

Hey @nik132-eng  ,
Let us know if you face any issues. Happy to help! : )

Hey @nik132-eng ,
Just checking in - are you working on it or shall we consider reassigning it to someone else?

Hello @nik132-eng  , we didn't hear back from you, so we are unassigning you from this issue. If you are interested, we will assign you with other issues, if the current one is taken.

Hey @swangi-kumari can you assign this to me I am willing  to submit PR

Hey @Hangsaai  , sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with.

I am not able to push the code shows that I am not a collaborator @swangi-kumari

Yep I have already tried using Brave aswell but same problem occuring

@Hangsaai Which repository are you trying to open a pull request from? Please ensure that you have write access to the repository you're opening the PR from.

I am directly pushing to a branch in hyperswitch itself and then creating a PR to main @SanchithHegde

@Hangsaai Could you please share the link to the repository/branch you're pushing to?

@Hangsaai Ideally you shouldn't have had access to push a branch to our repository. Could you push the branch to your fork instead and open a PR from there?

Yep I just figured it out I guess it's a vulnerability of github shouldn't have happened

Completed in 2674


Error Deployment

Repro steps:

Action: appService.Deploy
Error type: Error
Error Message: Deployment to "bienesraicessp" failed. See output channel for more details.
Version: 0.16.1
OS: win32
Product: Visual Studio Code
Product Version: 1.40.1
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:1:497269
o extension.bundle.js:1:495244
processTicksAndRejections task_queues.js:89:5

Hi @magrmobile
Please provide repro steps for me to assist you further.


Typo "Teams Room device"→"Teams Rooms device"

https://learn.microsoft.com/en-us/microsoftteams/teams-updates
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/blob/public/Teams/includes/teams-content-updates.md
#PingMSFTDocs

Hi @tiburd Hope all is well. Could you please help with this PR? Thanks!


Music support?

Hey, we have previously used voxels music addon for the main menu.
Is there any way to play music constantly or replicate this feature with custom main menu?

You can change the music playing in the main menu using a vanilla resource pack so Resource Loader is also able to do it.


Test Tool

1. Name of the Tool/Application
Tool Test
2. Short Description
No response
3. Application Area

[ ] Administration
[ ] Research/Teaching
[ ] Industry
[ ] Services

4. Use Type

[ ] simulation
[ ] monitoring
[ ] optimization
[ ] planning
[ ] control
[ ] advanced control
[ ] miscellaneous
[ ] visualization

4.1 Use Type Notes
No response
5. Lifecycle Phase

[ ] preliminary planning
[ ] planning
[ ] implementation
[ ] operation

6. User Interface

[ ] graphical user interface
[ ] web-based user interface
[ ] command-line user interface

6.1 User Interface Notes
No response
7. Target Group

[ ] Construction professionals
[ ] Engineers
[ ] IT professionals/Software developers
[ ] End users

8. Last Update
No response
9. Accessibility

[ ] unknown
[ ] chargeable
[ ] open-source
[ ] freeware

10. License
No response
10.1 License Notes
No response
11. Alternatives
No response
12. Specific use cases
No response
13. Developers/maintainers
No response
14. General type of tool

[ ] Digital Application
[ ] Digital Tool
[ ] Framework/Library
[ ] Platform
[ ] programming language
[ ] standard

15. Focus

[ ] Technical
[ ] Operational
[ ] Ecological
[ ] Legal

16. Published
unknown
17. Year of Publication
No response
18. Publication planned
unknown
19. Sources of information (Documentation, Literature, Git Repositories, etc.)
No response
20. Level of development
unknown
21. Programming Language
Python
22. Most important framework/libraries
No response
23. Database System
No response
24. Spatial scale of use cases

[ ] Component
[ ] System
[X] Building
[ ] District/Network
[ ] Region/Country

25. Norms/Standards
No response
26. Suggestion about this questionnaire
No response

Just a show of function.


Update mdoc, sbt-mdoc to 2.2.18

Updates

org.scalameta:mdoc
org.scalameta:sbt-mdoc

from 2.2.9 to 2.2.18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: library-update, semver-patch

Superseded by #239.


Referral history prototype

As a user I want to view a linear representation of a referral process so I can understand or audit it easily.

Please review and comment here: https://www.figma.com/proto/SRRHyBuXcHKbh0WgzodOj6/Proton-Material-2-UI-PRO?page-id=60017%3A0&node-id=60017%3A1&viewport=499%2C1217%2C0.7748788595199585&scaling=scale-down


chore: Bump dependencies

Bump direct and indirect dependencies
Also make this extension explicitly support VSCode 1.48+ (as the issue described in 798d906
seem fixed)

Welcome back!


Support for existing SLOs present in k8s env from record rules.

Tried the application, nice interface
have a question regarding importing/adding support for existing SLOs into the interface.
eg-> , we have a CI git ops setup for defining SLOs which generate prom record rules in k8s cluster (using sloth, is it possible to visualize those already present SLOs in pyrra interface.

Hm. I don't think so. I'm also unsure how we could make that work, actually.
Can you share the configuration you can in mind so it's a bit more concrete?
That would be helpful, thanks!

I was thinking in terms of while fetching the data from prom.
(assuming, operator must be generating record-rules for SLOs from the spec , interface gathering data via them), so is it possible to give support for fetching data generated from other record-rules (
( eg. say for sloth generated record-rules , slo:objective:ratio{sloth_service="${service}", sloth_slo="${slo}"} will give SLO objective for service's SLO )
assuming for pyraa, similar kind of expression will be building, was wondering can we add support for changing/adding these labels(sloth_service, sloth_slo) while querying the data .

I don't think this is something Pyrra will support this way around as described.
Pyrra will always go from raw metric to recording rules by generating the entire pipeline, rather than depending on some external source.
Currently, the recording rules for ratio SLOs would look like:
Input metric: prometheus_http_requests_total{job="prometheus",handler=~"/api.*",code=~"5.."}
Generated recording rules:
prometheus_http_requests:increase2w
prometheus_http_requests:burnrate3m
prometheus_http_requests:burnrate15m
prometheus_http_requests:burnrate30m
prometheus_http_requests:burnrate1h
prometheus_http_requests:burnrate3h
prometheus_http_requests:burnrate12h
prometheus_http_requests:burnrate2d

The naming of these recording rules is based on the Prometheus naming and aggregation best practices.
The downside being that it's hard to build a generic Grafana since we don't know that one specific metric ahead of time.
That being said, we could think of a compatibility layer, where, if enabled, would generate the additional recording rules that could map to the recording rules that Pyrra creates already.
It would mean doubling the amount of metrics Pyrra creates, if I'm not missing some way this could be done.

This has actually been implemented and is a feature called generic-rules. It'll ship with v0.5.0 in the next coming days, along with a Grafana dashboard.

oh, great to know, will check it out


Annotations on machine inventory from /etc/os-release are not updated

After upgrading nodes from latest stable release (v2.1.1) to staging OS (v2.1.2) the annotation  elemental.cattle.io/os.image is not updated to the new image reference.
Elemental-operator version: v1.6.3 (staging build as of today)

Please consider reworking or adding to this documentation: https://elemental.docs.rancher.com/upgrade-lifecycle#verifying-successful-upgrades

Please consider reworking or adding to this documentation: https://elemental.docs.rancher.com/upgrade-lifecycle#verifying-successful-upgrades

Makes sense, tracked in a separated issue:
https://github.com/rancher/elemental-docs/issues/360

The OS annotations are not updated because the elemental-registration.service is never triggered.
This is due to the elemental-registration.timer being disabled. Re-enabling the timer, will call the elemental-registry binary for registration at each boot.
Fixing this anyway will not be enough: the elemental-register binary will refuse to run a plain re-registration before 24 hours, so an OS update may be reflected int he annotations only after 24 hours from the last registration labels/annotations refresh.
So, to recap, two fixes are needed:

enable elemental-register.timer unit in the OS image
rework the re-registration behavior to allow refresh of annotations (and labels) at each boot


Specify function scope in jsonSerialise interface.

This makes the method stub autogeneration to add a function scope which is the expected behaviour.
I have signed the Contributor License Agreement.

https://youtrack.jetbrains.com/issue/WI-29970

I believe it's a wrong way to fix an initial request since {{public}] keyword is not required here. We have to update all our Interfaces to achieve desired behavior (if it's desired indeed: https://youtrack.jetbrains.com/issue/WI-29971).

Discussion in ticket.


[test] RISC-V Compliance for Earlgrey

Currently, we run RISC-V compliance tests on Ibex (though on the Ibex repo rather than in the vendored-in copy in OT). However, these tests do not run on the Earlgrey toplevel system; running these tests as part of CI would have caught https://github.com/lowRISC/opentitan/issues/350, for example.
To do this, we will need a toplevel system that can dump the results of the tests from memory; there's a few options for doing this, which boil down to either "dedicated peripheral" or "dump something through UART". I'm not sure how much of this work can be undertaken by software and how much of it is hardware work, though, so I'd appreciate some guidance there.

as PR #485 addressed this issue, can we close this issue @imphil?


[PDI-18776] Fix fo sub-job to be executed if Execute every input row …

…is checked

:white_check_mark: Build finished in 34m 5s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3721, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 32m 43s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3721, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 32m 53s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3721, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 31m 49s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3721, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 30m 31s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3721, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

IT tests added


Real Time Data

Hey,
What about this endpoint?
"https://api.boerse-frankfurt.de/v1/data/quote_box?isin={ISIN}&mic={XSC}".
I found nothing like this in your Script, but i want realtime data :D

Hi,
thanks for the hint. I had a look at these endpoints at the beginning, but since the data is usually delayed by 15 minutes they are not really "realtime" and I considered them not much advantageous ;) However I will have a look at it as soon as I get to and see what I can do because these endpoints need a different approach since they need a streaming client.

This single entpoint don't need streaming. Try this :D Am 10.05.2022 13:56 schrieb joqueka @.***>:
Hi,
thanks for the hint. I had a look at these endpoints at the beginning, but since the data is usually delayed by 15 minutes they are not really "realtime" and I considered them not much advantageous ;) However I will have a look at it as soon as I get to and see what I can do because these endpoints need a different approach since they need a streaming client.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Hi,
as mentioned this and some others are streaming endpoints. And they only send data during trading hours what limits my development possibilities because this is not possible in the evenings. However I'll see what I can do to make getting this live data as easy as possible. Using the regular internal functions does not work out. Please be patient.

Hi,
finally I added support for streaming of live data :)
Using one of the three new functions in bf4py.live_data will return an object which is able to stream data in a background process and can send the data to a function of your choice.
Have fun!


Adds callbacks from rust.

This is a WIP pull request implementing text edit callbacks in rust.
Current status:
I got imgui to call our basic callback, and I'm working on integrating a more rusty API for the end user.
I've started working on the end-user API, but I'm stuck on how we should allow the user to manipulate the ImString buffer.  I think it might be worth while to provide helper methods that do work the user could do themselves, but are pretty repetitive.
This should eventually resolve #430

Closing in favor of #516


Support folders for job inputs and outputs

To demonstrate, here are three example processors. The first is CreateSampleFolderProcessor that creates a sample folder as output
https://github.com/magland/dendro_analyses/blob/c2bf64d6921b1cc19bfb1afc85a4b8f4da0697f0/apps/dendro1/main.py#L102-L125
The second is TarProcessor that takes a folder as input and returns a .tar file as output
https://github.com/magland/dendro_analyses/blob/c2bf64d6921b1cc19bfb1afc85a4b8f4da0697f0/apps/dendro1/main.py#L128-L153
Finally, the third is UntarProcessor  that takes a .tar file as input and outputs the extracted folder
https://github.com/magland/dendro_analyses/blob/c2bf64d6921b1cc19bfb1afc85a4b8f4da0697f0/apps/dendro1/main.py#L156-L181
I tested this using the following script that queued up a simple pipeline:
https://github.com/magland/dendro_analyses/blob/main/projects/test/test_input_output_folders.py

Codecov Report
Attention: 316 lines in your changes are missing coverage. Please review.

Comparison is base (76ce93a) 70.67% compared to head (60f159b) 67.41%.

Files
Patch %
Lines

python/dendro/sdk/InputFolder.py
23.30%
79 Missing :warning:

python/dendro/sdk/OutputFolder.py
31.25%
44 Missing :warning:

...hon/dendro/api_helpers/routers/processor/router.py
39.62%
32 Missing :warning:

python/dendro/sdk/AppProcessor.py
50.84%
29 Missing :warning:

...thon/dendro/api_helpers/services/gui/create_job.py
35.71%
27 Missing :warning:

python/dendro/client/submit_job.py
7.14%
26 Missing :warning:

python/dendro/sdk/_run_job_child_process.py
12.00%
22 Missing :warning:

...dendro/api_helpers/services/_create_output_file.py
10.52%
17 Missing :warning:

python/dendro/sdk/Job.py
42.30%
15 Missing :warning:

python/dendro/sdk/App.py
12.50%
14 Missing :warning:

... and 4 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
- Coverage   70.67%   67.41%   -3.26%
==========================================
  Files          73       76       +3
  Lines        4487     4938     +451
==========================================
+ Hits         3171     3329     +158
- Misses       1316     1609     +293

Flag
Coverage Δ

unittests
67.41% <37.79%> (-3.26%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Error: not enough arguments in call to raven.NewStacktraceFrame

Hi,
the package from https://github.com/getsentry/raven-go/ has a new release which should be a minor change but seems a major one, because has added a new argument to a function:
v 0.1.0: func NewStacktraceFrame(pc uintptr,        file string, line, context int, appPackagePrefixes []string) *StacktraceFrame
v 0.1.1: func NewStacktraceFrame(pc uintptr, fName, file string, line, context int, appPackagePrefixes []string) *StacktraceFrame
This change causes the following error on the logrus_sentry package:
vendor/github.com/evalphobia/logrus_sentry/sentry.go:316:36: not enough arguments in call to raven.NewStacktraceFrame
        have (uintptr, string, int, int, []string)
        want (uintptr, string, string, int, int, []string)```

You could use gopkg to control which third-party version use, or update your package.

Regards,
Dani

Hello,
the problem seems resolved.
The package https://github.com/getsentry/raven-go/ has released a new version v0.1.2 that fixes the previous problem and now the logrus_sentry package works again.
Regards,
Dani


Country data is missing after install

Have you ever seen this?  I npm install --save country-data then var countries = require('country-data').countries;, and countries.all looks like {}.

I've not seen that. If you can reproduce please let me know the various versions of node, this module etc.
The following works for me:
$ npm install --save country-data
<EMAIL_ADDRESS>node_modules/country-data
├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
$ echo "var countries = require('country-data').countries; console.log( countries.all );" | node | head
[ { alpha2: 'AC',
    alpha3: '',
    countryCallingCodes: [ '+247' ],
    currencies: [ 'USD' ],
    ioc: 'SHP',
    languages: [ 'eng' ],
    name: 'Ascension Island',
    status: 'reserved' },
  { alpha2: 'AD',
    alpha3: 'AND',
$

Closing, but please re-open if needed.


Title fix for Small & Minority Businesses

Fixing the title on Small & Minority Businesses
Changes

Fixed title on small and minority businesses

Testing

Visit http://localhost:8000/doing-business-with-us/small-businesses/

Review

@sebworks
@anselmbradford
@jimmynotjim

Preview

Preview this PR without the whitespace changes
Notes

I confirmed with @sarahsimpson09 that only the title on the top should be changed. Navigation items should stay the same.
DFJR-81 on the JIRA board

What's going on with the mixed capitalization in the breadcrumb?

Shouldn't the sidebar menu item and the main title be the same? cc @schaferjh

I confirmed with @sarahsimpson09 that only the title on the top should be changed. Navigation items should stay the same.

I confirmed with @sarahsimpson09 that only the title on the top should be changed. Navigation items should stay the same.

This looks like a mistake to me, but okay.

:+1: after addressing @Scotchester's breadcrumb question

:+1:

Breadcrumb thing is being addressed in #1030


please add a permissive license

Noticed a PR open on this. If it is not your intention for this to be non-free work, can you please add a license?

Thanks!


Remove compute_strategy from solvers

Why

cost of maintaining strategies is negligible in comparison to actually computing them (we need to do this anyway)
the strategies are only returned when requested
complicates adding the threshold parameter for the threshold solver slightly

What to keep in mind

some old code might use this option (testing, some show-case, notebooks)

We might add **kwargs to make this painless. Or just correct all the other code.

Solved by d78e63c743348d538e98ff954aee72cf750ae74f


Classification visualization: test_tsne function cannot output TSNE graph

How to visualize classification tasks? For example, visualizing TSNE or confusion matrices. I found that there is a test_tsne function in  runner_finetune.py, but it doesn't seem to be called?

After calling the test_tsne function, an error occurred:
File "/home/g512/liubin/PointMamba/tools/runner_finetune.py", line 625, in test_tsne
test_only_tsne(base_model, test_dataloader, args, config, logger=logger)
File "/home/g512/liubin/PointMamba/tools/runner_finetune.py", line 582, in test_only_tsne
_, concat_f = base_model(points)
ValueError: too many values to unpack (expected 2)
I haven't modified test_tsne anywhere, does this mean that a certain parameter needs to be set?

Hello, I know concat_f is a feature.
Now, there are 39 concat_f, each shape is [40]; and there are 39 label, of which 38 shapes are [64],  one is [36].
The shape of the test_feature formed by concatenating is [1560] (40 * 39), while the corresponding test_label is [2468] (38 * 64+36=2468).
I think the problem lies here. I don't know why the shape of the label and concat_f are different, and only the shape of the last label is [36], while the others are [64]. I think the shape of concat_f is correct because modelnet40 has 40 categories and the output shape is [40], which is reasonable. The question is why the label are [64] and [36], and how should they be modified?
I hope you can give me some advice, thank you very much!!
This is the code for printing shapes and its output：

Change the code from line 547 to line 554 to the following：
        test_label = torch.cat(test_label, dim=0)
        test_feature = torch.cat(test_feature, dim=0)
        test_feature = test_feature.cpu().numpy()
        label = test_label.cpu().numpy()


someone has got into my system help please

[Enter feedback here]

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

ID: 1bc4cf6d-384c-09e4-f3c1-e8152e3b1302
Version Independent ID: 1bc4cf6d-384c-09e4-f3c1-e8152e3b1302
Content: JavaScript Profiler tool - Microsoft Edge Development
Content Source: microsoft-edge/devtools-guide-chromium/javascript-profiler/javascript-profiler-tool.md
Product: microsoft-edge
Technology: devtools
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Closing as incomplete.


Exampel for bad bicycle routing in Bad Hindelang (Germany)

Hi,
I'm very new here an I hope I'm right!
I'm a very heavy user of graphhopper as a (bicycle) biker and mostly very satisfied about this router.
But now I have planned a route for next weekend and found this:
https://graphhopper.com/maps/?point=47.505802%2C10.367285&point=47.506128%2C10.368208&locale=de-DE&vehicle=bike&weighting=fastest&elevation=true&layer=Omniscale
All streets are residential.
Regards
Bernhard Kuisle (Alias OSM bk)

Thanks (again :)) and the reason could be some problems with elevation see #384 and #517 and the discussion here where a user of GH tries to improve elevation. But this is important to get fixed!

This is the same thing as analysed in #330 and the reason for https://discuss.graphhopper.com/t/elevationweighting-implementation-and-bike2/213. Even if you have 100% right elevation data there always will be findings like this.
@bkuisle: You can try the MTB bike or the racebike profile: These profiles do not weight any elevation data. If you know this segment, please let us know how steep it is in real. According to our data and mathematics we use it is a 10 meters ascend on 82 meters. For such an ascend our calculated speed is about 6km/h. The longer segement has 12 meters ascend. But ss the ascend is much lower, the calculated speed is faster.

@bkuisle do you have any real elevation or speed data for us here?

Hi, thanks for asking.
Yes, I rode it with my bike in Nov 2015 equipped with my ciclomaster bicyle computer to measure the elavations (not absolut, but relatively exact)  and the distances. I startet at the crossing Karl-Haffner-Str. - Marktstraße via Badstraße (as proposed by Graphhopper) to crossing Zillenbachstraße-Schützenstraße and back the direct way.

Attached are the datas in an screenshot from an excel table.

Thanks! So the 10 meters are okay but we still do something wrong here. E.g. when looking at the current picked path there is also a steep increase here Not sure if the 1m less is the reason this is preferred over this expected.

Is this related? Wrong routing only with vehicle=bike. All others, including mtb and racingbike, work correctly. The yellow road is just 1 unsplit way in the concerned area.

https://graphhopper.com/maps/?point=49.066275%2C8.804088&point=49.066155%2C8.804491&locale=de&vehicle=bike&weighting=fastest&elevation=true&layer=Omniscale
I accidentally stumbled on this when testing routing through destination highways (Staffelweg has motorcar/motorcycle=destination) with various OSM routers.

Yes, this seems to be an elevation issue too. We should probably reduce the impact of the elevation a bit to see if this fixes the issue.


Kwxm/lazy machine doc

This is a document about the implementation and performance of a lazy PLC evaluator.  I'd be glad if someone could check that it builds properly (type make) in case I've forgotten anything.  Apart from that, the best thing to do is probably to read the pdf.

I don't know about nix.  Is there any particular reason to do that?  Typing make isn't too onerous.

Well, it will ensure it continues to build, and it'll be on hydra etc. Generally it's our "serious" build tooling, so if we want this to be an artifact we care about I think we should include it. If it's more of a throwaway thing then less so.
Nix will indeed download its own latex distribution. This is admittedly annoying if you're not then going to use that yourself to work on it. But it'll be the same one for the spec and for this doc.

^ If you want to do that, then please go ahead.

Also, it occurs to me that in the name of reproducibility (and protecting me against me own tendency to lose things) I should put my test cases and the scripts that I used to run them and plot the results in the repository somewhere.  Would a subdirectory of this one be sensible?

Er, "in the same directory" meaning "somewhere under the same directory".

Yes, a subdirectory of the one with the tex files in it.

I'll add some nix scaffolding in a follow-up PR. Looks good to me now, I'll merge in a bit barring further comments.


codegen: change static func names to avoid conflicts

Static functions were generated as public functions using the method name found in the windows metadata. Since all static functions of a namespace are added to the same Go package, this could cause collisions when two classes had the same static method (FromBluetoothAddress for example can be found in both Bluetoothdevice and BluetoothLEDevice).
This commit changes the naming strategy to include the class name as a prefix of the static function names. And the existing methods will be kept for a while marked as deprecated to maintain backwards compatibility.
fixes #85

Considering that the change only involves renaming a few functions and the simpler code, I agree


feat(api): impl some web3 interfaces

Which docs this PR relation:

sha3
accounts
clientVersion

Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d262f4f). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage        ?   39.93%
=======================================
  Files           ?       75
  Lines           ?     4953
  Branches        ?        0
=======================================
  Hits            ?     1978
  Misses          ?     2975
  Partials        ?        0

LGTM


Synchonize network defines to workaround libvirt bridge name race condition

This PR means to supersede/replace https://github.com/dmacvicar/terraform-provider-libvirt/pull/740
libvirt assigns the same bridge to networks defined in parallel (See https://gitlab.com/libvirt/libvirt/-/issues/78).
This code synchronizes network definition.
Note that there is still a problem in the code that generates wrong DHCP ranges. I could only prove it comes from our generated XML, but I could not figure how the race happens. We will need to address that as a separate issue.
I believe we need to remove most of the network code, and be a thin layer on top of libvirt. It may require us to align our definitions, eg. no dhcp option in the network block, and a explicit range option.
Fixes https://github.com/dmacvicar/terraform-provider-libvirt/issues/703

I thnk for the DHCP issue we can have a follow-up issue.


Transition to short-lived service account credentials

Forseti uses the signBlob API to get the delegated credentials for calling G Suite. Transition from the IAM API to the new short-lived credential API.

The short-lived credential cannot be used with impersonation.


Support 4 finger tap

Please Support 4 finger tap
Thanks in advance.

Hey @moranno!
If I understand you correctly, do you mean to support for binding 4 finger taps to specific actions?

Hey @moranno!
If I understand you correctly, do you mean to support for binding 4 finger taps to specific actions?

Yes, that's right!

Thank you for your suggestion. I will investigate this feature.
BTW, have you seen this feature in other apps?

Thank you for your suggestion. I will investigate this feature.
BTW, have you seen this feature in other apps?

check this app: https://multitouch.app/

Or https://github.com/artginzburg/MiddleClick-Ventura
Or in BTT
Or https://www.middleclick.shop/


Update Wootric library

Hi guys, we want to update the integration so it uses the latest version of our library.
The integrations loads a segmentioSnippet here
https://github.com/segment-integrations/analytics.js-integration-wootric/blob/master/lib/index.js#L22
How can I update that segmentioSnippet so it loads the latest wootric version?

Nevermind! I found it. 😅


arrow use largestring and date32/date64

This PR proposes using LargeString in favor of String type. The String datatype offsets the bytes in the &str buffer with an i32 meaning that the columns can only hold 2^32 == ~2GB of string data. This limit is very easily met.
The LargeString uses an i64 as offset type. A limit I don't see reached any time soon. :)
This also adds Date32 and Date64 for the naive dates/datetimes.
What would be needed next to make this work? I assume some python integration. And from the Rust side?

The code looks good to me! I think a natural next step is letting Arrow use a specialized type system as defined here https://github.com/sfu-db/connector-x/blob/main/connectorx-python/src/pandas/types.rs#L20 as well as providing conversion rules for Postgres / SQLite like here https://github.com/sfu-db/connector-x/blob/main/connectorx-python/src/pandas/transports/postgres.rs for pandas. Arrow uses the dummy type system because I was mainly using that for PoC at the beginning of this project.
The type system and type conversion part are quite macro-heavy. Just let me know if you need some help with this.
BTW are you going to add more code into this PR or create a new one?

The type system and type conversion part are quite macro-heavy. Just let me know if you need some help with this.

Ok, I will take a look. Probably somewhere in next week.

BTW are you going to add more code into this PR or create a new one?

Nope, this one is free to merge. :)


[SPIRV] Handle sub-byte loads with EmulateNarrowTypes

Fixes #14972 by preventing offset calculation from temporarily exceeding the maximum i32 value (for 32 bit indexing) even if the byte offset does not exceed it. Additionally this bring handling of sub-byte loads more in line with other backends.

This needs a little more investigation based on the PkgCI results. Moving to a draft until I've verified performance locally.
***WARNING*** CPU scaling is enabled, the benchmark real time measurements may be noisy and will incur extra overhead.
experimental/regression_suite/tests/pregenerated/test_llama2.py Downloading https://storage.googleapis.com/shark_tank/llama_regression/09152023/llama2_7b_int4_stripped.mlir -> /_work/iree/iree/artifacts/llama2_7b_f16qi4/llama2_7b_int4_stripped.mlir---------------------------------------------------------------------------------------------------------
Benchmark                                               Time             CPU   Iterations UserCounters...
---------------------------------------------------------------------------------------------------------
BM_first_vicuna_forward/process_time/real_time       30.3 ms         6.37 ms           23 items_per_second=32.9727/s
2023-09-25T15:54:53+00:00
Running /_work/iree/iree/venv/lib/python3.11/site-packages/iree/_runtime_libs/iree-benchmark-module
Run on (32 X 5759 MHz CPU s)
CPU Caches:
  L1 Data 32 KiB (x16)
  L1 Instruction 32 KiB (x16)
  L2 Unified 1024 KiB (x16)
  L3 Unified 98304 KiB (x2)
Load Average: 0.37, 0.09, 0.03
***WARNING*** CPU scaling is enabled, the benchmark real time measurements may be noisy and will incur extra overhead.
----------------------------------------------------------------------------------------------------------
Benchmark                                                Time             CPU   Iterations UserCounters...
----------------------------------------------------------------------------------------------------------
BM_second_vicuna_forward/process_time/real_time       21.9 ms         1.09 ms           32 items_per_second=45.5694/s
: Retrieved 2525199 bytes

Testing locally it seems like the PkgCI results are noise (we've seen noise like the above before too, so there is a precedent).
2023-09-25T14:57:16-04:00
Running /home/quinn/iree-build/tools/iree-benchmark-module
Run on (32 X 5879.88 MHz CPU s)
CPU Caches:
  L1 Data 32 KiB (x16)
  L1 Instruction 32 KiB (x16)
  L2 Unified 1024 KiB (x16)
  L3 Unified 32768 KiB (x2)
Load Average: 0.43, 0.49, 0.32
***WARNING*** CPU scaling is enabled, the benchmark real time measurements may be noisy and will incur extra overhead.
***WARNING*** Library was built as DEBUG. Timings may be affected.
----------------------------------------------------------------------------------------------------------
Benchmark                                                Time             CPU   Iterations UserCounters...
----------------------------------------------------------------------------------------------------------
BM_second_vicuna_forward/process_time/real_time       8.99 ms         1.06 ms           76 items_per_second=111.2/s

If this turns out not to be the case we can revert later.


Feat: Fastify adapter

This can be merged into here, then from here into master.
https://github.com/jlalmes/trpc-openapi/pull/254
Should I have done it directly into master?

Here are the commands I ran to fix it:
gh pr checkout 177 && git fetch origin master && git merge origin/master
rm package-lock.json
manually merged dependencies in package.json
npm i --package-lock-only
final testing:
npm i && npm run test

Would love to use this! Anything blocking merge and release?

Hi guys, is there anything that needs doing before this can be merged? 🙂

Would be great to use this soon, anything blocking the merge still?

Nothing new ?

Still waiting for my pr to be accepted😴
In the meantime, you could just take this file from the pr and put it in your project to get the adapter working.

when pr merge ?

@jlalmes what's the ETA on the release for this? I could definitely use this in my Fastify project.

Available in<EMAIL_ADDRESS>


Disneyland Hong Kong

The Disneyland Hong Kong website uses a lot of similar functionality to the main WDW websites.
The resort has the ID "hkdl", and the URL https://www.hongkongdisneyland.com/authentication/get-client-token/ can be used to generate an access token similar to the one used by the mobile app. However, this token will have different access, as it's got a completely different scope (it generates one for a desktop client, as opposed to the mobile client).
It's a good start, we could probably fish out schedules, but I don't know how to go about finding the wait times. Anyone have any thoughts? Half expecting an official app to appear, that will probably shed some light on the specifics.

Stumbled on required data. I can now get wait times if I pass "hkdl" instead of "dlp" to API calls and use the park ID "desHongKongDisneyland".
Working on schedule data to get opening/closing times of rides... should have it all working in a few days I reckons.
This has unveiled that the code for the main Disney request logic needs a serious overhaul since they've started adding loads of other parks in. I may start a large refactor soon.

Latest commit to the refactor branch now supports Disneyland Hong Kong ride wait times and park schedules.
I can't even find an official app that shows HK wait times, so that's kind of cool. A first perhaps?
https://github.com/cubehouse/wdwJS/commit/ea556fd29fdd64191d0256b72c7d53787a98450b

Disneyland Hong Kong support is now in the refactor branch for version 3.0.
Closing as I am now tracking support in pull request #7


Cleaned up run-setup.sh, added separate hook generation, cleaned up o…

This is a part of the comprehensive rework and redeployment of the SSP gitops configuration.
Signed-off-by: Knicholas Kennedy<EMAIL_ADDRESS>

@Knickkennedy , the PR only contains files that are to be deleted, including the script and pvc template yamls.  Are additional changes coming for this PR?  If so, can we get this into draft mode if its not ready to be merged.

@hollisc For some reason the file changes didn't push, just pushed them. Thanks!

@Knickkennedy, is there a reason why instances/sterling-secure-proxy-engine/values.yaml is blank?
Leaving some other comments in the files within the PR as well.

The files instances/sterling-secure-proxy-setup/ssp-cm-pvc.yaml_template and instances/sterling-secure-proxy-setup/ssp-engine-pvc.yaml_template in the PR is empty and the PVC yamls are hardcoded to the IBM dynamic storageclass.  Can we revert this back so the script will generate the PVC yaml from the template.

Hi @hollisc I'm not sure why on the PR it's showing 'empty file' but if you go to the actual code the files are there! Let's check again.

@hollisc This should also be fixed now

is this still needed @Knickkennedy @hollisc what is the status?

Hey @Knickkennedy, can you confirm that this has been tested.  Also, looks like the commit has not been signed off but we can help with this once we get confirmation on the validation.  Thanks.

I haven't been able to implement the changes that Budi suggested, unfortunately. It technically works as-is right now, but is not up to standard yet.


Could NOT find Matlab

Hi, I have Mex but in Windows, when I build, I got this error:
Could NOT find Matlab (missing: Matlab_MEX_LIBRARY MX_LIBRARY) (found version "9.5")

Hi, Windows is not currently supported. Will let you know when the support is implemented.


feat(docs): asm functions

Rewrote the method ID collisions section to remove all logical jumps and make it much more streamlined :)
Also adjusted the structure a little towards the upcoming PR revamping this page. I'll push the draft of it right after we deal with asm functions here.
P.S.: I actually call argument to return position mappings "arrangements" and not "shuffle" as in grammar.ohm, because the latter kinda implies randomness, while those are actually deterministic. Hence, "asm arrangments".
Issue

Closes #1011.
Closes https://github.com/tact-lang/tact-docs/issues/300 (old issue in tact-docs)

Also, resolved two teeny tiny issues from tact-docs — virtually 3-5 lines of fixes for each, no need in a separate CHANGELOG entry:

Closes https://github.com/tact-lang/tact-docs/issues/374
Closes https://github.com/tact-lang/tact-docs/issues/368

Checklist

[x] I have updated CHANGELOG.md
[x] I have run the linter, formatter and spellchecker
[x] I did not do unrelated and/or undiscussed refactorings

I agree that this is confusing, and I think that the cause is that we are mixing two mental models: a stack (low-level) and tuples (high-level).
We should pick only one and stick with it the entire explanation. It seems that describing everything in terms of tuples is more intuitive, but then we should not mention the stack (or mention how tuples are pushed and popped from a stack in a separate subsection, and only in that section mention the stack).
So, for example, I would start the examples saying something about the TVM instructions, something like this:
"
Even though TVM instructions work with a stack, TVM instructions can be seen, intuitively, as maps from tuples to tuples. To see how TVM instructions map tuples to tuples in the TVM stack, see [link: here]. Thinking in terms of tuples makes the explanation of asm functions much clearer, but for those who want to see an explanation using the stack directly, see [link: here].
"
Then, explaining the meaning of a declaration like:
asm(len self -> 1 0) fun testFun(self: Slice, len: Int): Result { TVM_INSTRUCTION }

struct Result {
   res1: Int;
   res2: Bool;
}

amounts to saying simply:
"
testFun passes the argument tuple (len, self) to instruction TVM_INSTRUCTION. Suppose (r0, r1) is the tuple result of TVM_INSTRUCTION, then testFun reorders the result according to the map -> 1 0, i.e., the 1-th index element (r1) has now index 0, and the 0-th index element (r0) has now index 1, producing the tuple (r1, r0). Finally, testFun assigns the tuple (r1, r0) into the Result struct one field at a time, producing Result {res1: r1, res2: r0}.
"

@jeshecdom interesting note. The cases with multiple instructions should be covered too. And tuples on TON are denoted with square brackets [], so it's better to use those. Also, it's best for readability to remove parentheses as much as possible, so things like "and the 0-th index element (r0)" will be "and the 0-th index element r0" — no need to add indirection and visual pauses with parens :)
@anton-trunov wdyt about https://github.com/tact-lang/tact/pull/1061#issuecomment-2489235137?

TVM instructions can be seen, intuitively, as maps from tuples to tuples.

this is incorrect in a very specific technical sense: a tuple is a TVM data structure that occupy precisely one TVM stack position but can contain multiple other TVM primitives, including tuples
the term you probably intended to use is tensor

testFun passes the argument tuple (len, self)

I find it confusing

@novusnota just adapt the corresponding calling convention description from tvm.pdf

@novusnota you also need to check how structures that are returned from a function are actually encoded

To fully finish this section, #910 needs to be resolved too

TVM instructions can be seen, intuitively, as maps from tuples to tuples.

this is incorrect in a very specific technical sense: a tuple is a TVM data structure that occupy precisely one TVM stack position but can contain multiple other TVM primitives, including tuples
the term you probably intended to use is tensor

I meant mathematical tuple, but now I see that this would introduce much more confusion because of the technical terms in TVM. So, the explanation should stick with the stack and use the technical terms in TVM.

@jeshecdom interesting note. The cases with multiple instructions should be covered too. And tuples on TON are denoted with square brackets [], so it's better to use those. Also, it's best for readability to remove parentheses as much as possible, so things like "and the 0-th index element (r0)" will be "and the 0-th index element r0" — no need to add indirection and visual pauses with parens :)
@anton-trunov wdyt about #1061 (comment)?

Yeah, we should stick with the stack explanation and use the technical terms in TVM, because I see everyone is confused now :). But the way, a question:
In a function like this:
asm fun testFun(a: Int, b: Int): Result {
 INS_1
 INS_2
 .....
 INS_n   // Let us suppose that after INS_n finishes,
         // there are 5 results in the stack
}

Does Tact know that after executing those instructions, there will be exactly 5 results in the stack?
What happens if struct Result has more than 5 fields? Will Tact pop more than 5 elements from the stack until it fills the struct, having as consequence the popping of elements that are not part of the intended result?

Does Tact know that after executing those instructions, there will be exactly 5 results in the stack?

not in the current implementation

Will Tact pop more than 5 elements from the stack until it fills the struct

nope (the consequence of the previous answer)
This should be documented, of course, but an even more important question is "are returned structs actually represented as tensors (multiple TVM values)?"

and, of course, the symmetrical question for input function parameters

in any case, each such point should be accompanied by a concrete example of an asm-function

@jeshecdom when we have the grammar and AST for our embedded assembly language then we will be able to typecheck asm-functions and warn the user their stack discipline makes sense

just adapt the corresponding calling convention description from tvm.pdf

Sure, the -> 0 1 is much better explained in terms of s0..s255 stack registers.

We could do the following with specific examples (previously, we should have explained how function arguments are pushed into the stack):

First, explain how the result tensor is popped from the stack. Suppose after popping, you get some tensor (r0, r1, ..., rn).
Explain how the notation -> n m l is just a re-arrangement of the result tensor.
Explain how the re-arranged tensor is mapped into the result type of the function (this includes explaining how the tensor is mapped into a struct or a primitive type, or whatever).

Eh, Structs are represented as tensors (...), but that's the doing of FunC. If we were to target TVM directly, we would've dealt with stack entries ourselves, all without tensors.
So I'm a bit hesitant on explaining things in tensors, and instead let's just properly describe about stack and stack registers.

Eh, Structs are represented as tensors (...), but that's the doing of Tact+FunC. If we were to target TVM directly, we would've dealt with stack entries ourselves, all without tensors.
So I'm a bit hesitant on explaining things in tensors, and instead let's just properly describe stack and stack registers.

Sounds good.

So I'm a bit hesitant on explaining things in tensors, and instead let's just properly describe stack and stack registers.

sounds good to me too, since tvm.pdf does not even mention tensors, looks like it's a term coined by the FunC community

Added description of the current Tact-flavored assembly from https://github.com/tact-lang/tact/pull/1064
Described everything from the stack point of view, including its "registers"
Refined the overall top-to-bottom reading flow

All done here, @anton-trunov @jeshecdom.


ci: The workspace publish job seems to be failed on 2024.08.16

Describe the bug
The workspace publish job seems to be failed on 2024/08/16 with the following error:
error: Following packages don't exist, follow the links and create them:
 - https://jsr.io/new?scope=std&package=cache&from=cli
Error: Process completed with exit code 1.

It seems to be caused by @std/cache not being created in JSR :eyes:
Environment

OS: ubuntu-latest
deno version: canary
std version: release-2024.08.16

Thanks for catching this @uki00a !
Fixed in https://github.com/denoland/std/actions/runs/10440240479/job/28909900942


Added Docker images for Debezium 0.3 release and tutorial

THIS IS NOT READY FOR MERGING
This is still using the staging repository in Maven Central for the yet-to-be-released 0.3 artifacts.

@rhauch does this have the same connection issues with kafka <IP_ADDRESS> ?

@hchiorean, I haven't tested the images with <IP_ADDRESS> yet. Will be doing that today.

@rhauch: I tried building the latest images locally, but the following URL https://oss.sonatype.org/content/repositories/iodebezium-1007 doesn't exist (and is referenced atm from connect/0.3)

@hchiorean, yes, I know. It doesn't build for me but this morning I'm going to try another release. Local builds using local plugin archives works after I corrected the port default. I'll update this PR when the official archives are in Maven Central.


KMS-encrypted S3 Transfers with S3 Events enabled not possible with IAM user

Use case: Using DTH to transfer any changes made to the source S3 bucket into the corresponding destination bucket in cn-north-1. The source S3 bucket is in the same account as the DTH, however, it is KMS encrypted. That is, the vanilla S3 Transfer Task for Source Buckets in the current region will not work for this scenario. One possible solution to this is described in Issue https://github.com/awslabs/data-transfer-hub/issues/73
The approach is to create an IAM User in the source bucket account and pass the credentials for it to the DTH S3 Transfer Task. For doing so, one must select "Is bucket in this account?" with "No" in the Source Settings. This will disable the option to listen to S3 events from the source bucket as "Enable S3 Event?" will not be available when selecting that the source bucket is not in the current account.
Expected behavior
We should make sure that either:

One may add credentials from ASM even if the source S3 bucket is in the current account or
The "Enable S3 Event" is still an available option for source buckets "outside" of the current account

[x] Version: v2.4.0
[x] Region (source): eu-central-1
[x] Region (destination): cn-north-1

Hi Kaplana
Thanks for the feedback and contribute the PR meanwhile. I have discussed with team and right now for the KMS-encrypted S3 we suggest to use https://github.com/awslabs/data-transfer-hub/blob/main/docs/S3-SSE-KMS-Policy.md approach, and we are going to enhance UI to provide better user experience.
For the suggestion that use AK/SK to access all the S3 bucket including in/ out same account, this might impact too many customers, especially that for those who didn't initiate KMS in source S3, might leading them addition effort to create AK/ SK compare with existing feature.
Let me know if you have other thoughts.
Best Regard/ Eva
DTH Product Manager

Hi @evalzy , thanks for your reply. I see that but there must be a way in which one could do it via UI instead of manual modification within the Cloudformation stack after creating said S3 transfer task, right? Did I understand this correctly that there are UI features in plan which will incorporate this? Or even select from available KMS keys of that account (just like the ASM secrets) would be nice!
I think if this is what is coming up, that's totally fine!
Best regards
Ayhan

We're looking forward to it. Thank you!

Hi @kaplanan,
After reviewing the documentation and the available workarounds mentioned in https://github.com/aws-solutions/data-transfer-hub/issues/73, I believe it would be more appropriate to classify this as an enhancement rather than a bug. The reason for this change is to ensure that our labels are accurate, which will help us prioritize backlog items more effectively.
Thank you
Bassem


Add dynamic metric configuration support

https://github.com/open-telemetry/opentelemetry-specification/pull/751

Closing as the issue is no longer applicable.


Change project name and namespace

To bring in to line with other projects, propose to move to a IFY namespace.
I have long been considering a name-change for the project, and I think this is the opportunity to rename to "Shimr".
So, all existing Shimterface.* namespaces will become IFY.Shimr.*.
The "Standard" can be dropped from the package, as this legacy reference is no longer useful.

Idea will be to push a final 1.x change noting the name change and then 2.0.0 start using the new terms.

Useful info: https://docs.microsoft.com/en-us/nuget/api/registration-base-url-resource#package-deprecation


replace IOBuf with independent buffer implementation

this is along the lines removing folly dependency.
What are the options here?
cc: @tmontgomery

closable?

At this point, very little value from removing this dependency. So, probably closable.


AutoPr-Graph.RBAC-yugangw-msft-REST Spec PrNumber 5873

@azuresdkci Retest this please

/azp run azure-sdk-for-net - mgmt

@shahabhijeet do we disable tests and merge this?


Add RK4 fixed step solver and parallelized JAX fixed step solvers

Summary
The goal of this PR is to:

Add jax and numpy-based RK4 fixed step solver
Add a new "fixed step LMDE solver template" for JAX that uses parallel operations, i.e. all time-steps are integrated simultaneously in parallel using vmap.

This is specific to LMDEs as it assumes that structure for the parallelization to work.
This method works by simulating the full propagator over each time-step in parallel, then multiplying them all together (also in parallel, using jax.lax.associative_scan). A consequence of this is that, if the user wants to simulate only a single state, this approach will actually be more expensive than serial computations (in terms of linear algebraic operations), and may not be worth the parallelization. However, if a user wants to simulate the full propagator itself, then no additional computational cost is created.

Using the above, add jax parallel versions of both RK4 and expm solvers.

Details and comments
Currently implemented the jax parallel template, the jax parallel expm solver, and have added tests for the parallel expm solver derived directly from the already existing tests for expm solvers through inheritance. The template code currently assumes that y0 is square, so it needs to be expanded, but the tests are passing so this is a good start!
To do:

~Generalize the parallel template to accept non-square y0.~

~Test!~

~Add RK4 solvers~

~Test! (Can probably abstract out content of expm tests.)~

Add proper interface access to new solvers through solve_lmde and solve_ode.

Add some higher level tests for these.
Update documentation.

Check if update necessary in module docs.

Before proceeding with this PR we need to do some speed testing of the parallelized JAX solvers on both CPU and GPU.
I've just tried some test cases with CPU on my laptop, and what I'm seeing is:

jax_expm is consistently faster than jax_expm_parallel. Not by too much, e.g. 10s v.s. 13s for a large system with a ton of time-steps, but it does seem to be consistent (even when varying system size and number of time steps). Watching the CPU usage, it seems like jax_expm keeps usage consistently high, whereas jax_expm_parallel has periods during which the CPU usage drops.
jax_rk4_parallel is insanely slower than jax_rk4.

I've even tried writing some smaller toy functions to strip out all of the other stuff going on in dynamics, and the above behaviour is the same.
@brosand  is going to run the same tests on a GPU, to see if this is an XLA CPU v.s. GPU utilization efficiency issue, or if the usage of vmap and associative_scan (as opposed to serially doing everything with scan) is really worth it or not.

Following up with the results of the GPU testing:
For jax_expm we do observe a speedup with jax_expm_parallel, when run on a GPU, specifically a v100. With dimension 70, we see jitted times of 2 s for the serial function and 130-150 ms for the parallel solver.
For the rk4 solver, on the same gpus with d=40, we see a 10 second run time for the jitted serial solver, and then 700ms run time for the jitted parallel solver.
It seems like both the parallel and serial solvers are highly accelerated by the gpus. However the parallel solver becomes even faster than the serial solver once it has access to gpus.

Based on the CPU v.s. GPU speed issues with these solvers, added a sentence to the description of each parallel solver that we recommend only using it with GPU.

One more thing to add to this PR, as suggested by @brosand :
While the addition to the doc string that the parallel solvers are only intended for GPU is good, it doesn't seem like quite enough to discourage their use on CPU. The jax.default_backend() command can be used to determine if JAX is going to cpu or gpu to execute, so we can add a line in the parallel solver template that raises a warning if jax.default_backend() == 'cpu'.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: DanPuzzuoli:x: Ben RosandBen Rosand seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Suggest clarifying comments and argument names

Your library is most interesting and I like the clear bignum implementation. However, it could be good to more clearly mark which functions work in Ed25519, which do in Curve25519, which parameters and points and which scalars, or whether they are in bignum or buffer format. There are a very few comments and sometimes the arguments are named in a way where this is not at all apparent, at least unless one is already familiar with this field.

Thanks for the feedback.
Are you using TypeScript, or js with vscode type hints? TypeScript shows all types which makes it very simple to reason about.

Just browsing on Github so far. Heard of this library today.

@covert-encryption Are you browsing index.ts, or index.js?

Ah, I didn't realise that was compiled code. Yes, index.ts solves the type problem at least. Perhaps at some points still could use clarification on which curve was used but I did not look too closely yet.

Well, this library is called noble-ed25519, which means it's always ed25519 (edwards curve), not curve25519 (montgomery). Why did you think it's curve25519?

There is at least a conversion function for it, and I saw variables u and v being mentioned in a few places. Also the field element code (arithmetic mod 2^255-19) should be compatible with both if I am not mistaken. But if this is indeed intended to only support Ed25519, I will consider this issue closed.


fix: exception should come in correct way to user on configuration page and corrected the message

Original issue raised:
Error while setting advanced image processing on ['jpeg', 'jpg', 'png', 'tiff', 'bmp'] · Issue #1250 · Azure-Samples/chat-with-your-data-solution-accelerator (github.com)
Describe the bug
While the documentation (https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/blob/main/docs/advanced_image_processing.md ) states that Advanced image processing can be done for ['jpeg', 'jpg', 'png', 'tiff', 'bmp'] , however configuring in the admin app the advanced image processing i do get the following errors:
Traceback (most recent call last): File "/tmp/8dcb7fb38d2a08c/pages/04_Configuration.py", line 404, in ConfigHelper.save_config_as_active(current_config) File "/tmp/8dcb7fb38d2a08c/batch/utilities/helpers/config/config_helper.py", line 204, in save_config_as_active ConfigHelper.validate_config(config) File "/tmp/8dcb7fb38d2a08c/batch/utilities/helpers/config/config_helper.py", line 224, in validate_config raise Exception( Exception: Advanced image processing has been enabled for document type pdf, but only ['jpeg', 'jpg', 'png', 'tiff', 'bmp'] file types are supported.
Expected behavior
please see link : (https://github.com/Azure-Samples/chat-with-your-data-solutionaccelerator/blob/main/docs/advanced_image_processing.md )
Image
Steps to reproduce
just attempt to configure advanced image processing in the Admin App as showing on the attachment to this issue
Purpose

...

Does this introduce a breaking change?

[ ] Yes
[X] No

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code
Open CWYD admin url
Click on 'Configuration' tab
Go to 'Document processing configuration' section
Check the checkboxes in 'Use advanced image processing' column
Click on 'Save Configuration' button.

What to Check
Verify that the following are valid

...

Other Information

Exception message issue got fixed, but We found one issue while checking checkbox quickly getting page refreshed and for that issue, we need more time for refractor this meanwhile we can merge this issue fix.


Murmur3token size panic

This should actually be part of token aware routing, but will make it as an individual issue for its high priority.

It looks like the input to Murmer3Token::generate needs to have a length
 that is a multiple of 8 but we pass it something that is not a multiple
 of 8. Not sure if we need to guarantee that our input is multiple of 8 or if Murmer3Token::generate needs to handle any value.


Add metadata editor to page views

I want to merge this change because it adds metadata editor to page views.
PR intended to be tested with API branch: add/pages-metadata-1414
Pull Request Checklist

[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Translated strings are extracted.
[x] Number of API calls is optimized.
[x] The changes are tested.
[x] Data-test are added for new elements.
[x] Type definitions are up to date.
[x] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-1415 Add metadata editor to page views


Gitea only shows first line of commit message

Gitea only shows the first line of my commit message.
Take for example this repository (https://git.klingt.net/alinz/klingt.net/commits/branch/master) there are a lot of commits with multiline messages but for all of them only the title is shown.
This is most probably a configuration issue but I couldn't find a solution so I decided to open this issue.
PS: I am using gitea 1.3.2 and my configuration template can be found here.

This has been implemented in #2980 and will be release 1.4.0


Right-click dropdown doesn't work

I am not able to trigger the dropdown from a right-click, it just triggers the toggle as if I had left-clicked.
M1, 13.5.2

Hi, I run an M1 mac too and can't reproduce this on 14.1.2. Feel free to get in touch if you have any further questions.

Hi, I run an M1 mac too and can't reproduce this on 14.1.2. Feel free to get in touch if you have any further questions.

Works now that I've updated to 14.1.2, thanks!


update locations seed data

Description
This updates the locations seed data to include coordinates
What work was done?
A script was written to get coordinates for all new addresses, new addresses were found manually via Google Maps, the script was run against these new addresses and the resulting locations are now stored in addresses_data.js. The locations seed file was refactored to grab the new locations and no longer uses faker to retrieve address information.
Why was this work done?
The locations in the locations table did not have coordinates.
Type of change

Bug Fix

Change Status

Completed

Checklist

[ x ] Endpoint tested with Postman, or Unit test.
[ x ] My code follows the style guidelines of this project
[ x ] I have performed a self-review of my own code
[ x ] My changes generate no new warnings
[ x ] There are no merge conflicts

https://www.loom.com/share/6e76efeb15624e4cb7ba48bfedaa2fd1
Coauthored by: Murray Warnock<EMAIL_ADDRESS>Coauthored by: Michael Quint<EMAIL_ADDRESS>Coauthored by: Salomon Maldonado<EMAIL_ADDRESS>

Tested on my local env. Nothing broken, has a little bit of slow down on seeding but that's to be expected with the amount of data being seeded, and will prolly show on the production build as well. Works great. Well done!

nice!


[substrate-ros/robonomics] Asynchronous robot liability content provider

Problem definition
Current liability engine load liability's content in blocking manner. By design liabilities in robonomics network is asynchronous. Current implementation of IPFS file loader is synchronous and it should be adopted to new asynchronous liability player.
Proposal
Make a way to load liability contents (ROSBAG file in IPFS) in async function. Player builder should be runned after successfull file loading.
This issue has an aim to introduce service that could be used for asynchronous liability content downloading.

Is this still open?

@janus Yes, I'm waiting PR from @strdn.

@janus PR merged, this issue finished.

Hello @Web3Foundation this task was completed by @strdn.


Bump JRuby to v<IP_ADDRESS> (fixes #1035)

Kind of change

[x] Bug fix
[ ] New non-breaking feature
[ ] New breaking feature
[ ] Documentation update
[x] Build improvement

Description
What is the goal of this pull request?
Fix issue #1035 as commented https://github.com/asciidoctor/asciidoctor-maven-plugin/issues/553#issuecomment-1793240139
How does it achieve that?
Yes, validated in maven runs and also building a custom distribution of AsciidoctorJ.
Are there any alternative ways to implement this?
n/a
Are there any implications of this pull request? Anything a user must know?
The only thing is that there's still another warning for Java > 13. But we can treat it in a separated PR.
OpenJDK 64-Bit Client VM warning: Options -Xverify:none and -noverify were deprecated in JDK 13 and will likely be removed in a future release

Issue
If this PR fixes an open issue, please add a line of the form:
Fixes #1035
Release notes
Please add a corresponding entry to the file CHANGELOG.adoc

@robertpanzer do you see any issue with this? I can merge myself if agree.

Sorry for not being responsive, I am drowning a bit atm :D
Thanks for the chore! 🙏

Sorry for not being responsive, I am drowning a bit atm :D

No problem, I was tempted to merge myself seeing all was working fine. But I still like to hear from you.


A detailed document

fluent and anna is  cool ，Is there a detailed document?
It’s hard to learn without documentation.

Hi @yuinm -- thanks for your note. What sort of detailed documents are you looking for? You can find some getting started docs in the repo. If there are other things you'd like more detail on, please let us know, and we'll try to add docs for those things.

Thank you for your reply.
I want to know what the principle is.
And want to know how to compare with tbb.

You can find a more detailed comparison with TBB in this paper. We are working on another paper with the architecture of the system and will post it when we can.


[6.0][Concurrency] Retain the actor around the CAS in enqueue() when necessary.

Cherry-pick https://github.com/swiftlang/swift/pull/77031 to release/6.0.
It's possible that the job we enqueue holds the last strong reference to the actor. If that job runs on another thread after we enqueue it, then it's possible for this to be destroyed while we're still in this function. We need to use this after the enqueue when the priorities don't match. When it looks like that will happen, retain this before the enqueue to ensure it stays alive until we're done with it.
Introduce a defensive retain helper class that makes it easy to do a single retain under certain conditions even in a loop, and does RAII to balance it with a release when the scope exits.
rdar://135400933

@swift-ci please test


Update BotManTest.php

Test for Unicode support

Please add this test to your patch-1 branch, so it updates this PR: https://github.com/mpociot/botman/pull/313


Riff Metrics: Turn Chart only uses participant's last utterance in the meeting

Description

The Turn Chart metric isn't correctly accumulating the total time a participant spent
talking in the meeting, it is only using the participant's last utterance.
Steps to Reproduce
Have a meeting preferably w/ several people having some discussion amonst the
participants, such that everyone speaks at least twice.
End the meeting and view the Riff Metrics on the Dashboard page.
It's a little hard to know how long each participant actually spent talking, but you
should see that the time list is way too short for each participant.
Expected behavior
The time allocated to each participant should be the total time of all of their utterances
not just the last one.
Possible Solution
The cause is actually known. There is a bug in the utility function groupByPropertyValue
such that instead of adding element's to the value's array, replaces the array w/ a new
array with the last element. This has already been fixed in the riff-rtc repository in commit rifflearning/riff-rtc@f308119289f3dfe514ad3378d35ecda71544c881.
Context
Site the bug occurred on:
localhost (development machine)
Additional Context
The bug was manifested by the cleanup of the dashboard in commit d6394028b351ffeb28a25db1b4aa71e4376051c4 which used the groupByPropertyValue function for processing utterances. This commit is in version 1.0.0-dev.27

This still needs to be tested in Mattermost.

Verified on staging.


For OpenJDK projects, automatically detect JTReg from OpenJDK configu…

…ration; show a warning dialog when not found before opening Project Properties.

I see no reason not to merge this. Anyone?


Skipped. Expected quality not found!

Hi there! I'm getting an error:
[CRITICAL-238] Skipped. Expected quality not found!

On different courses. I've tried to add the -q flag, but the same error.
Can you help me out?

@raymestalez you are not using this script you must be sing udemy-dl from python repository,  to fix that install aria2c but again you are using the wrong script...

You're right, cloned script from this repo, installed dependencies, and everything works! Awesome! Thank you very much for this excellent project!


Attack sequence needs to be broken down in multiple parts

Hi, let me start by expressing my gratitude for your great work! Thank you!
The issue I would like to point out is that because the "Accuracy" step and the "Damage" step of an attack are currently done simultaneously from a single menu. Because of this, we cannot use charms such as "Hail-Shattering Practice" and "Reckless Fury Discard" which interacts with the attackers attack roll but before applying the defender's defense value.
It is also worth noting that these charms cannot be fully automated since the character using them can choose not to exploit the full amount of 1's and 2's from the attackers’ roll in order to conserve motes creating the need for the player or DM to input information after the attacker's roll is known but before applying the defender's defense value.
Both these examples would also modifie by which margin the attack would hit the target, modifying the amount of damage dices rolled if not making the attack miss entirely.
(Sorry if my post is hard to understand, english is not my native language. Please let me know if any part of my post is unclear, I will try to better explain myself.)

This will probably not be done anytime soon.  Most charms have to be activated at the start of the attack sequence and adding in steps would require a major rewrite of the attack code.

I see a major rewrite of the dice-roller code, any chance this would translate in split attack phases, as the defense is very variable, charms affect it and it can be stunted once per turn.  Also some attacks are unparryable or undodgeable and the automatic choice to default to whichever is highest causes problems in some combats,  Thanks for your work.


Add Continuous Integration via Github Action

Add continuous integration via github actions

I totally forgot to mention, you should update the build status badge on README.md too :)
https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/adding-a-workflow-status-badge


Separate room between the shop level

Before the day starts at the shop, and after the day ends, I think the player should spend time in a different level - perhaps their home, where the "meta game" happens. Here the player can choose to advance to the next day (i.e. going to bed), see their bank account balance, or upgrade to a bigger shop.

They should also be able to order new products into the shop for the next day to sell.

This would be the main game loop – order products, try to sell them to make profit, pay the shop's rent, order and sell more products... Eventually the player would make enough money to upgrade to a bigger shop, hire staff, etc.


Io/convert to ts stats

Describe your changes
Converting more files to TS
Issue ticket number and link
N/A
Motivation and Context
Converting the entire project to TS
How Has This Been Tested?
Locally
Screenshots (if appropriate):
N/A

closing because we have completed this!


autorandr: add rotate option

Add rotate option for autorandr

Thanks for the pull request! I just added a suggestion about the option's type.

I updated my pull request regarding your proposal :)
I'm new to Ǹix, So thx for the enum hint :+1:

Great! I've rebased the change to master in 6aa44d62ad6526177a8c1f1babe1646c06738614. Thanks for the contribution 🙂


Typing any keyboard button in the documentation site will remove code highlighting

Qwik Version
0.9.0
Operating System (or Browser)
All browsers
Node Version (if applicable)
No response
Which component is affected?
Qwik Runtime
Expected Behaviour
Code highlighting should not be affected by keyboard typing.
Actual Behaviour
Highlighting is broken once any keyboard button is pressed.
Additional Information
After typing on the keyboard, the network tab shows that q-58078c4c.js is being downloaded.

But when blocking this chunk, the code highlighter is not affected anymore.

Duplicate https://github.com/BuilderIO/qwik/issues/1421


"Create application" wizards - with templates

In the wizard, you would give a unique namespace name, choose from options like nginx or databases, point to a docker file somewhere.
Monokle would create a folder with all the resouces that is ready to be applied to the cluster.

@tommywhitehead as discussed - need your input on how to display the template repository


How to get game's lastupdate in "userGameList"?

if the function:
can get the game's time of lastupdate from 'userGameList'
can get image of trophy which user not earned from 'userGameTrophyList'
then, this api will be pretty good!!!

Hi viskem,
Thank you for your feedback.
Regarding to your advises, I'm working on getting the image of unearned trophy. It cannot directly retrieved from current psn json result, but could be achieved by collecting the trophy information from someone who's earned. So my approach will be collect the trophy information as much as possible and cache them in database or files.
The result of userGameList is ordered by recent played, but to get the last update timestamp of user's game list, it is not available so far. If you got any idea about it, could you shared that?

i see you get data from "https://my.playstation.com/playstation/psn/public/trophies/?onlineId=ViskemEvan&_=1438837460933" & "https://my.playstation.com/playstation/psn/profile/compareGames/trophies/data?gameId=NPWR06221_00&userIds=ViskemEvan" ?
can you get data from "https://{{region}}-tpy.np.community.playstation.net/trophy/v1/trophyTitles?fields=%40default&npLanguage={{lang}}&iconSize={{iconsize}}&platform=PS3%2CPSVITA%2CPS4&offset={{offset}}&limit={{limit}}&comparedUser={{id}}'" ?
trophyData:'https://{{region}}-tpy.np.community.playstation.net/trophy/v1/trophyTitles?fields=%40default&npLanguage={{lang}}&iconSize={{iconsize}}&platform=PS3%2CPSVITA%2CPS4&offset={{offset}}&limit={{limit}}&comparedUser={{id}}'	// NOTE: All server are in the US, the only change are market restrictions
trophyDataList:'https://{{region}}-tpy.np.community.playstation.net/trophy/v1/trophyTitles/{{npCommunicationId}}/trophyGroups/{{groupId}}/trophies?fields=%40default,trophyRare,trophyEarnedRate&npLanguage={{lang}}'
trophyGroupList:'https://{{region}}-tpy.np.community.playstation.net/trophy/v1/trophyTitles/{{npCommunicationId}}/trophyGroups/?npLanguage={{lang}}'

(from gumer-psn/psn.js)

Hi viskem,
I checked the URLs you mentioned, when I call these URLs, they results a http 401 status. It seems the URLs use another authorization route than the one I used.
After checked gumer-psn/psn.js, I found that it follows the route after android app does, but the authorization part does not work anymore. I have no packet analysis tool which could work with android so far.

I updated new distribution which will cache the trophy data into hsqldb. It can be used to collect the trophy infos in daemon. I think you may wanna try it out.


[TextField] Fix first character composition issue

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".

Tiny change. It seems to me that, when the component is conrolled,
'value' needs to be checked only in componentWillReceiveProps,
and event.target.value in handleInputChange has nothing to do with
hasValue state.
At least, this PR can fix #3394 issue which, still remaining despite #5540,
breaks 'first' character composition of CJK or other input methods
when used with, for example, redux-form.
Closes #3394.

@keifuji The idea behind that change looks good to me. Would you mind adding a test in case it's introducting a regression and we need to to further work?

@oliviertassinari I added some tests to check hasValue state regarding event and value prop.
(I'm not familiar with enzyme, so it might be odd.)
Anyway, I found the problem has the same origin as inconsistency between state.hasValue and props.value. In fact, the question is not re-rendering but pre-rendering before it receives next props. Until then, the state rest true even though props.value is still invalid.
These are logs from TextField wrapped by reduxForm:

handleInputChange -----------------------------------
states and props: {this.props.value: "", hasValue: false}
reduxForm.onChange  {firstName: "a"}
** render TextField ** 									// unnecessary rendering triggered by setState
componentWillReceiveProps ---------------------------
states and props: {this.props.value: "", hasValue: true} 		// <==== Inconsistency!
componentWillReceiveProps {nextProps: Object, value: "a"}
** render TextField **									// reasonable rendering
(callback) setState in componentWillReceiveProps: {this.props.value: "f", isValid: true, hasValue: true}
(callback) setState in handleInputChange: {this.props.value: "f", isValid: true, hasValue: true}

So I included additional tests for controlled component, which will be able to prevent the issue from reappearing, and necessarily fails in the current version.

@keifuji Thanks!


Fix for flight task to allow attitude estimator only.

FlightTasks requires a local position publication. I'm doing some development with an attitude only estimator and don't see why we need position publication to fly in manual.
This PR simply always returns true and doesn't reset the set points if the position publication is not being sent. It was continually calling and resetting my yaw setpoint which was an issue. Also if I don't return true it will not allow control after arming due to the failed return status.
I think it is important that we sync on this and that it doesn't cause any edge cases by always returning true.

@Stifael so #10683 replaces this one right?

@jgoppert preferably yes, because i still want to ensure that if there is a valid position estimate, the time delta is checked as well

but what we really should do is to move out the Attitude-ctl-mode into the attitude controller as discussed here
https://github.com/PX4/Firmware/pull/10670#issuecomment-428196178

Ah, sorry for the duplicate development. I didn't think to check.


poetry lock takes forever

I'm attempting to update the dependencies using poetry lock. However, it still doesn't write the lock file after more than 20 thousand seconds. I stopped it because it was using an increasing amount of swap memory.
Then I tried poetry cache clear --all pypi before running poetry lock again, but it still doesn't update.
Is there anything we can do?
Hoping @alecandido and @scarrazza can provide some suggestions. Thank you!

Thank you @alecandido , let me investigate and give it a try! Appreciate your time and advice! ;)


Error baseline correction

Basic information

My operating system and version: Windows 10 Enterprise LTSC, version 1809
My mzmine version: 3.9.0

What happened
I have an error message when I try to do a baseline correction, when I try to do  "Asymmetric baseline corrector", it says "Could not start Rserve (R>install.packages(c('Rserve')) ). Please check if R and Rserve are installed and, in case the path to the R installation directory could not be detected automatically, if the 'R executable path' is properly set in the project's preferences." I already downloaded the packages on R, and check the R executable path.
mzmine error.docx
mzmine_0_0.log
https://www.dropbox.com/scl/fo/h3xoc14i615z9ka2w9c67/AINZUmYO5GXU7JiZYRtj0ck?rlkey=2gsoni9fg8yqgovvmbpt34cme&st=g5ter2ii&dl=0
Thanks for your help,
Ludivine

Please describe what happened and how it differs from what you expected to happen.
Please include all parameter values for the mzmine modules you used.
Please upload screenshots of any errors and your log/mzmine.log file.
Please upload a standalone mzmine project file (or a batch + raw files) that demonstrates the problem you are reporting. You can use Google Drive, Dropbox, or a similar repository to upload the project.
Issues opened without an example project are unlikely to be processed

Hi,
sorry for the inconvenience. Unfortunately noone of the current developers contributed was maintaining the R modules, so we have disabled them in the latest releases.