Transitive reduction related triggering weirdness

Behavior
A non-transitive reduction pipeline has downstream stages that are triggered without waiting for all the dependent stages to complete. For example, in our new hyperparameter tuning example, we have the following pipeline:

In some cases, the select pipeline with trigger:

After changes to the split pipeline (e.g., an update with --reprocess), but
Before both model and test finish their corresponding updates.

Ideally, select would wait for both model and test to update before reprocessing.
Reproducing

Set up the hyperparameter tuning example as detailed here: https://github.com/pachyderm/pachyderm/tree/master/doc/examples/ml/hyperparameter

Update the split pipeline with --reprocess (no change to the spec is necessary, as we are just updating something upstream from model, test, and select that with initiate downstream triggering).

After (1), you should see the successful jobs:
$ pachctl list-job
ID                                   OUTPUT COMMIT                           STARTED            DURATION           RESTART PROGRESS    DL       UL       STATE
53ac2c7c-542a-4c7e-aafb-be5bfdad5d09 select/54baf753161a4c72acd6a61a8a41d6fc 25 seconds ago     1 second           0       1 + 0 / 1   225.4KiB 30.69KiB success
9d680efc-c5b5-480f-a41b-70f376b5735f test/2536316b93d44b3c95e7f8fe8a2823ed   About a minute ago About a minute     0       77 + 0 / 77 331.1KiB 301B     success
aa1e18d6-702a-4507-a522-01b70c47897f model/70d293bc21f4445d99fb2b3c4ba4a8bc  3 minutes ago      About a minute     0       77 + 0 / 77 423.2KiB 225.1KiB success
cfbb1677-5035-4411-83dd-681fa0a504e1 split/4fedb0b5e25b490587a93a3c1f9a212f  3 minutes ago      Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success

After (2), you should see model and test update, and then select update once for the corresponding changes in model and test. However, select doesn't wait for both model and test to finish updating. It runs after one of them finishes, and then proceeds to run two more times (which is extra weird I think):
$ pachctl update-pipeline -f split.json --reprocess
$ pachctl list-job
ID                                   OUTPUT COMMIT                           STARTED                DURATION           RESTART PROGRESS    DL       UL       STATE
942fef69-ccc6-4a55-a2ae-9079a0dea1cf test/-                                  Less than a second ago -                  0       0 + 0 / 77  0B       0B       running
82726bbb-fbaa-4303-a125-fc91f3e81ddb model/-                                 Less than a second ago -                  0       0 + 0 / 77  0B       0B       running
fa4f96cb-534e-401c-8bc8-6b851dd0463a split/9107a912cfed44c49ec505d98ffe66e0  Less than a second ago Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success
53ac2c7c-542a-4c7e-aafb-be5bfdad5d09 select/54baf753161a4c72acd6a61a8a41d6fc 3 minutes ago          1 second           0       1 + 0 / 1   225.4KiB 30.69KiB success
9d680efc-c5b5-480f-a41b-70f376b5735f test/2536316b93d44b3c95e7f8fe8a2823ed   5 minutes ago          About a minute     0       77 + 0 / 77 331.1KiB 301B     success
aa1e18d6-702a-4507-a522-01b70c47897f model/70d293bc21f4445d99fb2b3c4ba4a8bc  6 minutes ago          About a minute     0       77 + 0 / 77 423.2KiB 225.1KiB success
cfbb1677-5035-4411-83dd-681fa0a504e1 split/4fedb0b5e25b490587a93a3c1f9a212f  6 minutes ago          Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success
$ pachctl list-job
ID                                   OUTPUT COMMIT                           STARTED            DURATION           RESTART PROGRESS    DL       UL       STATE
960bca3b-1e55-4d00-9dcc-926a518d2e05 select/8efbda50985c4ca4bc4645cc81a62bae About a minute ago 2 seconds          0       1 + 0 / 1   225KiB   186.1KiB success
25c6f808-52ce-47a4-8ebe-c6582ced4df2 test/-                                  About a minute ago -                  0       59 + 0 / 77 264.9KiB 208B     running
cc076440-43d0-41a7-92f5-816c1f8d6af9 select/0a5a3d421a7044f2a15315552e7a9ec3 About a minute ago 2 seconds          0       1 + 0 / 1   225KiB   30.23KiB success
942fef69-ccc6-4a55-a2ae-9079a0dea1cf test/cf17b80fc28a4986958210904de8a70b   2 minutes ago      About a minute     0       77 + 0 / 77 330.8KiB 301B     success
82726bbb-fbaa-4303-a125-fc91f3e81ddb model/ed57e53c2a68487994957c3f58d9e46b  2 minutes ago      About a minute     0       77 + 0 / 77 423.5KiB 224.7KiB success
fa4f96cb-534e-401c-8bc8-6b851dd0463a split/9107a912cfed44c49ec505d98ffe66e0  2 minutes ago      Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success
53ac2c7c-542a-4c7e-aafb-be5bfdad5d09 select/54baf753161a4c72acd6a61a8a41d6fc 6 minutes ago      1 second           0       1 + 0 / 1   225.4KiB 30.69KiB success
9d680efc-c5b5-480f-a41b-70f376b5735f test/2536316b93d44b3c95e7f8fe8a2823ed   7 minutes ago      About a minute     0       77 + 0 / 77 331.1KiB 301B     success
aa1e18d6-702a-4507-a522-01b70c47897f model/70d293bc21f4445d99fb2b3c4ba4a8bc  9 minutes ago      About a minute     0       77 + 0 / 77 423.2KiB 225.1KiB success
cfbb1677-5035-4411-83dd-681fa0a504e1 split/4fedb0b5e25b490587a93a3c1f9a212f  9 minutes ago      Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success
$ pachctl list-job
ID                                   OUTPUT COMMIT                           STARTED        DURATION           RESTART PROGRESS    DL       UL       STATE
4f41cf1f-23d7-4418-9549-d3ab8fd99190 select/3fc00502454d4512916780faf9aea5c0 3 minutes ago  2 seconds          0       1 + 0 / 1   224.9KiB 71.16KiB success
960bca3b-1e55-4d00-9dcc-926a518d2e05 select/8efbda50985c4ca4bc4645cc81a62bae 4 minutes ago  2 seconds          0       1 + 0 / 1   225KiB   186.1KiB success
25c6f808-52ce-47a4-8ebe-c6582ced4df2 test/56952d3bd0864fe2b31f4def87e289e8   4 minutes ago  About a minute     0       77 + 0 / 77 330.3KiB 280B     success
cc076440-43d0-41a7-92f5-816c1f8d6af9 select/0a5a3d421a7044f2a15315552e7a9ec3 4 minutes ago  2 seconds          0       1 + 0 / 1   225KiB   30.23KiB success
942fef69-ccc6-4a55-a2ae-9079a0dea1cf test/cf17b80fc28a4986958210904de8a70b   6 minutes ago  About a minute     0       77 + 0 / 77 330.8KiB 301B     success
82726bbb-fbaa-4303-a125-fc91f3e81ddb model/ed57e53c2a68487994957c3f58d9e46b  6 minutes ago  About a minute     0       77 + 0 / 77 423.5KiB 224.7KiB success
fa4f96cb-534e-401c-8bc8-6b851dd0463a split/9107a912cfed44c49ec505d98ffe66e0  6 minutes ago  Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success
53ac2c7c-542a-4c7e-aafb-be5bfdad5d09 select/54baf753161a4c72acd6a61a8a41d6fc 9 minutes ago  1 second           0       1 + 0 / 1   225.4KiB 30.69KiB success
9d680efc-c5b5-480f-a41b-70f376b5735f test/2536316b93d44b3c95e7f8fe8a2823ed   11 minutes ago About a minute     0       77 + 0 / 77 331.1KiB 301B     success
aa1e18d6-702a-4507-a522-01b70c47897f model/70d293bc21f4445d99fb2b3c4ba4a8bc  12 minutes ago About a minute     0       77 + 0 / 77 423.2KiB 225.1KiB success
cfbb1677-5035-4411-83dd-681fa0a504e1 split/4fedb0b5e25b490587a93a3c1f9a212f  12 minutes ago Less than a second 0       1 + 0 / 1   6.858KiB 6.858KiB success

I'm guessing that this is because there is some weird feedback between generating models and testing those models, which occurs before selection. Anyway, I think this sort of thing would be unexpected for others.  Maybe @jdoliner or @msteffen have ideas?

@jdoliner, I know we're reworking our job triggering code so this bug may not be relevant soon, but is there anything you can think around the non-transitive reduction part of the code that would break down in this DAG but work for the simple cases we've tested?
I figure that same logic is being rewritten into the new system too and if this DAG fails currently, it may still fail in the new design as well.
A --> B --> C
 \    |    /
      D

A --> D
B --> D
D --> C

We can test this once the new version is out.
I don't, off the top of my head, know why this DAG would behave differently but it doesn't surprise me that it does.

It appears that 1.7 fixes this, based on my tests with the RC. Closing for now.


Update readme

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

small readme change to generate python sdk client

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
n/a
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
n/a
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

For swagger APIView. you could ignore it for now. We will apply a fix for this false alarm

For swagger APIView. you could ignore it for now. We will apply a fix for this false alarm


'__tostring' is not inherited

This would be an issue from Lua, but it's a bit ugly for Pluto.
class Base
    function __tostring()
        return "A based object"
    end
end

class Child extends Base
end

print(tostring(new Base())) -- "A based object"
print(tostring(new Child())) -- "table: 000001B2504382D0"

Lazy fix would probably just be adding the following to Pluto_operator_extends:
if p.__tostring and not c.__tostring then
  c.__tostring = p.__tostring
end

But it could be desirable to have tostring resolve __tostring from parents (via __index)


not showing free disk space on dashboard

The dashboard does not show Used Diskspace and Available Diskspace on my dashboard.
@pranay01 @prashant-shahi

There is a table with filesystem information, but it's not a very good overview as it repeats all states and something with the units that it shows is off, haven't figured out what exactly, but it's clearly not correct. I've improved it a bit by filtering it on free state.
a single value panel is tricky because there are going to be multiple filesystems/mountpoints, but could be a graph maybe, with percentage calculation, did that as an alert.

@Berdir Thanks for the reply. I agree about the filesystem information being off. I've decided to go with Zabbix as it has a better representation of the matrices out of the box.

@Berdir please share which part of it is not correct. I believe it is is more likely a data problem than a panel problem.

@srikanthccv i upload a image explaning what they are saying, I was trying to understand the disk size usage in gbs, but with the plot bellow I cant understand a thing. I know that the old dashboard on signoz as showing properly the disk usage along time. I wish I could see The total disk available to use plus the amount of actual disk being used then I could create alerts about vm reaching the max disk available space.

We can see that the old dashboard of hostmetrics had a plot to watch this

In case it helps, this is the alert that I set up:

The trick is the last bit that converts the two previous queries into a percentage, so you can then for example add a generic alert on 80% full.


Fixed Events image in Chapter 1 docs to reflect actual added event

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update dependency on crc

Once crc with PR#7 is published on crates.io update our dependency on crc. This will align the utilized version of lazy_static which we inherit once through the crc and once through the openssl modules in distinct versions now.

Updated in master; will close once released on crates.io as part of the 0.4 release.

Published on crates.io.


Use appropriate terms in comments

Customer scenario
Terms that should be avoided appear in source code.
Bugs this fixes:
VSO 377114
Workarounds, if any
None
Risk
Close to nil as only code comments were modified to get clean.
Performance impact
None
Is this a regression from a previous update?
No. Most of the code impacted is old. I presume prior runs of the tool that scans for these were run without scanning comments.
Root cause analysis:
Given the low volume of issues. I think the current process for running the tooling that identified this is sufficient.
How was the bug found?
Tooling to scan source for this.

@srivatsn @dotnet/project-system

Tagging @MattGertz

Approved to merge via 377253


support json schema generation for @JsonUnwrapped (#20)

This pull request is in relation to issue #20 and another pull request in the jackson databind repository.
This change here is rather an improve tests, while the actual fix is in the jackson-databind repository.

Sounds good to me; thank you for contribution!
One thing that I do need before merging is the Contributor License Agreement (CLA), from
https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf
which we need for legal reasons. We only need to get it once and it will work for all contributions across projects, versions etc.
If you could print it out, fill, scan and email to info at fasterxml dot com, that would be great. I can then merge the unit test, review other contributions and get it all done.

Cool, that's great. I just sent you the CLA.


[FEATURE REQUEST] Enhanced text handling

Describe the solution you'd like
Pastebin/enhanced text upload, syntax highlighting, and language choices
Additional context
Text uploads are boring and look boring a new text upload window allows the pasting of big bits of text and a choice of languages (c++, js, html, css, py, java, ect...). Which when viewed highlights the text, like Pastebin but incorporated into Chibisafe. Pretty sure there are dozens of packages for text highlighting, Basically just an idea to spruce up the text upload side of Chibisafe

This has been implemented and released already.


[FR] qSpider Portable

Description
It is currently unclear which of the files in Releases is portable for Windows. The word portable is not explicitly mentioned. I guess it could be qspider-player-standalone.zip or qspider-player.zip or both.
Possible solution
I suggest naming the portable package in Releases more explicitly, or specifying in README.md that the portable version is not published in Releases.
Instead of a different file naming scheme in Releases, you can limit yourself to an explanation in README.md, including it in the installation instructions.

Right now there are no portable files as library I'm using to create desktop version does not support portable builds as of now.


RuntimeError: Error(s) in loading state_dict for ResNet:

RuntimeError: Error(s) in loading state_dict for ResNet:

Hi @wml666666 - Probably you have solved your issue . If not kindly share the exact - RuntimeError: , you are seeing .


Add new unit and integration tests to the API

Tasks

[ ] Add new unit and integration tests to the API (use a fork, see CONTRIBUTION.md first)

Bonus Task

[x] Develop inside the dev container

@vpchung The codebase of the API in the folder apps/api match the codebase from the main branch of rocc-service. This project is ready for the transfer of the unit-integration tests who wrote.

Related to #119 , #121 , #122

In #212, we are moving from a monolithic API to microservices, which will likely be developed using the Java framework Spring. Motivations include an excellent tutorial I found on microservices for this framework. Most importantly, Java is the language used by the Platform Team, who should contribute to the development in the future.
For these reasons, adding Python tests is not longer a priority.


Add more context to the metadata rules release notes

I talked about this with @jjohannes and it is hard to provide a generic example without going into all the details. Users of these APIs usually have custom usecases (e.g. jenkins with the jpi packageing type) that can be too confusing to others. WDYT?

I think that having an example in the release note for this might be too complicated. In short, this should be seen as a feature mostly targeted to plugin authors / build tool teams in organizations that have an advanced usage of Gradle.
So maybe that's what should be added to the release notes: this feature will help plugin authors that want to maximize variant-aware dependency management.

In that case, we can cover it under a "Improvements for plugin development" section.


Fixed guzzle version

Could the guzzle version not be restricted so much? You have set it to the fixed version 7.0.0 which is really weird since current version is 7.8.1 and make problems, when you already have other packages using guzzle.
I would suggest to use ^7.0 instead.

Hi @bernhardh ,
Thank you for bringing this to my attention and providing your suggestion. I appreciate your input. I have made the necessary changes, and the Guzzle version is now set to latest version in the project.
Feel free to test it out, and if you encounter any further issues or have additional feedback, please let me know.
Thank you,
Team QuickScraper.


Tidy up readme, add licence

Reworded and improved the README a slight bit. Added LICENCE file (included text for the MIT license, since that's what package.json specifies).

Rebased to current HEAD.


Integrate with Bower package manager.

Added in https://github.com/gdemad/magento-requirejs/pull/7


json helper

Add a helper for returning json from http endpoints.

wow thanks, I didn't know about that one.


fix setutpools entrypoints

Hopefully simplify the recipe a bit

use conda entrypoints (avoids setuptools runtime checking, gets around bug causing failure in #23)
install with pip (avoids various bad setuptools things)
test entrypoint to verify that it works
update build dependencies (only python+pip is needed)
add runtime dependency versions from setup.py

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

awesome, thank you min!


[PE-D][Tester D] Names with '/' are not allowed in event creation

Names with "s/o" or "d/o" will not be treated as valid inputs and can cause misrepresentation for some people who have these abbreviations in their names.
Input command:
create e/ n/Balakrishnan s/o Balakrishnan BDay Party on/11-11-2024
Result:

Labels: type.FeatureFlaw severity.Low
original: valhrd/ped#6

To be addressed by #322


Rank-reducing linalg via reshape patterns + iree-hal-benchmark-dispatch-repeat-count seem to interact poorly and miscompile

What happened?
I have been iterating on graph-level transformations and added this repro to iree-samples.
This is dependent on https://github.com/openxla/iree/pull/12822
Running
    iree-compile --iree-hal-target-backends=cuda --iree-hal-benchmark-dispatch-repeat-count=5 transform_dialect/graph/cuda/bugs/2-ir.mlir | \       iree-run-module --function=forward --device=cuda --input="3x17x768xf32=1"
Produces a different result when using --iree-hal-benchmark-dispatch-repeat-count=5.
I suspect this is not expected given that a difference in behavior is triggered by this line.
It seems to either be:

an issue with one of the rank-reducing patterns via reshape or
a bufferization issue and we bufferize something inplace when we shouldn't (but it may be at the graph level ?)
triggering a miscompile somewhere in IREE

Steps to reproduce your issue

Go to '...'
Click on '....'
Scroll down to '....'
See error

What component(s) does this issue relate to?
No response
Version information
No response
Additional context
No response

I track P0 items as ones I need to look at in short order.

ping on p0 bug?

I guess these shouldnt be P0. Ill drop it down to P1.


Update Get-SPOSite.md

Fix minor typo

Docs Build status updates of commit 9dbed44:
:clock10: Full build: the average full build time is 11 min(s) 48 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 9dbed44:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit c368a9b:
:clock10: Full build: the average full build time is 11 min(s) 48 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit c368a9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Hover effect won't close

I have a problem with the hover of the tiles as seen on the picture.
What can I do about it?

any solution to this?? I'm having the same issue...

@noeldevelops in what web browser is this happening? Is there any zoom on the page?

Thanks for trying to help!
I changed the background color of .hex h1, .hex p in the stylesheet to make it black. I also deleted the first css rule:
* {margin: 0; padding: 0;}
Adding it back with dev tools doesn't seem to fix the problem, but it's a good starting point to think about, so thanks for that reminder...
I've also changed the html a bit to make it work in Wordpress, but nothing structural.
I will continue to troubleshoot and if I can make a snippet that recreates the error I will send it over.

its probably the removing of  * { margin: 0; padding: 0; } that makes it break.
Since a h1 tag is default getting 1em of margin below and above. Trying removing margin and padding on .hex h1 and .hex p.

You're exactly right!
I had a default margin adding 10px below all  tags... I put margin-bottom: 0; on the .hex h1 and it fixed it.
Thanks again for helping me figure that out


Convert history commands to Action2

This PR fixes #150782

@jrieken will need your help here, it does not look like the precondition gets applied to the menus. I notice how I can register multiple menus from Action2 directly, but I think I cannot define different icons per menu, so I think something is missing for me to address this?

@jrieken will need your help here, it does not look like the precondition gets applied to the menus. I notice how I can register multiple menus from Action2 directly,

You should use menu from Action2 and it should work (unless the rendering menu ignores it). Generally all UI defining properties should be defined centrally (inside Action2) but I understand the touch bar special case. For that you would still need a lone MenuRegistery.append...-call.

@jrieken updated. I had to leave the touch bar menu thing in for the different icon to use. If I had a way to set the icon for the menu registration, I could also inline that.

🍻


AkkaContainerClient breaks the ConcurrentTests of nodejs runtime

The concurrent tests of nodejs runtime keep failing.
I got into the CI environment and found the below.
The nodejs action code for the test is supposed to print the log whenever it is invoked.
global.count = 0;
let requestCount = $requestCount;
let interval = 1000;
function main(args) {
    global.count++;
    console.log("interleave me");    //////////// this log
    return new Promise(function(resolve, reject) {
        setTimeout(function() {
            checkRequests(args, resolve, reject);
        }, interval);
   });
}
function checkRequests(args, resolve, reject, elapsed) {
    let elapsedTime = elapsed||0;
    if (global.count == requestCount) {
        resolve({ args: args});
    } else {
        if (elapsedTime > 30000) {
            reject("did not receive "+requestCount+" activations within 30s");
        } else {
            setTimeout(function() {
                checkRequests(args, resolve, reject, elapsedTime+interval);
            }, interval);
        }
    }
}

...
interleave me
...

So we can compare how many requests arrived to the container by counting the number of that log.
The test is sending 128 concurrent requests, 128 logs are supposed to be printed.
But I only found around 60~65 logs.
And I added a few logs to the AkkaContainerClient and found the below logs.
runtime.actionContainers.NodeJs20ConcurrentTests > action-nodejs-v20 should allow running activations concurrently STANDARD_OUT
    running 128 requests
    [2023-10-28T01:34:45.439423] call to /init
    [2023-10-28T01:34:45.588940] call to /run
    [2023-10-28T01:34:45.589216] call to /run
    [2023-10-28T01:34:45.589413] call to /run
    [2023-10-28T01:34:45.589566] call to /run
    [2023-10-28T01:34:45.589669] call to /run
    [2023-10-28T01:34:45.589Z] [WARN] [#tid_sid_unknown] [AkkaContainerClient] The queue has already been completed.
    [2023-10-28T01:34:45.589745] call to /run
.
.
.
    [2023-10-28T01:34:45.644328] call to /run
    [2023-10-28T01:34:45.644504] call to /run
    [2023-10-28T01:35:16.039219] timeout to /run
    [2023-10-28T01:35:16.039403] timeout to /run

It seems the underlying request queue of AkkaContainerClient is closed after sending just a couple of requests.
Environment details:

Github CI environment of Nodejs runtime

Steps to reproduce the issue:

Run the NodeJS Runtime CI workflow.

@joni-jones
I suspect this is because of this line.
Do you have any idea on this?

onCompletion should be called only when a sink has already processed all the messages. But from what I'm seeing in the issue it might not be the case. I will try to look more closely into those tests to try to understand if the changes introduced in #5442 did break something.


Add cmake option/function to be able to skip mimick based tests

Export cmake function that can be used in downstream packages to selectively disable mimick based tests.
Add cmake option to skip building mimick.

Related to https://github.com/ros2/rcl/pull/923.

I think it is worthwhile to step back and think about what we are trying to accomplish here.
From my understanding, the problem is that the Mimick library only supports a limited number of systems.  For any system that is not on that supported list, we want to both skip building the mimick vendor library, and also skip building/running any downstream tests that rely on Mimick.  Does that properly describe the issue?
Assuming that is the case, I think it is worthwhile to see if we can do the following:

Automatically detect whether this system is on the supported list, with a possible override to force mimick support on.
Export both a CMake variable and a header define that exports whether Mimick is supported (this PR, though I'd prefer the positive HAS_MIMICK_SUPPORT rather than the negative SKIP_MIMICK).
Refactor downstream code to use those defines/cmake variables.  I will say that I don't love https://github.com/ros2/rcl/pull/923 ; there's a lot of ifndef SKIP_MIMICK in there, and we'll never test the !MIMICK case in CI to make sure it works.  One way to improve it might be to refactor tests so that Mimick-related tests are in the separate files, in which case we can use the CMake variable to skip building the test completely.

mimick vendor library, and also skip building/running any downstream tests that rely on Mimick. Does that properly describe the issue?

We want to skip building mimick, if we build mimick_vendor or not isn't important.

Automatically detect whether this system is on the supported list, with a possible override to force mimick support on.

We could do it automatically.
I preferred to add a cmake option so the user is consciously disabling the tests, but there's not much reason for that.

this PR, though I'd prefer the positive HAS_MIMICK_SUPPORT rather than the negative SKIP_MIMICK

I used the negative, because by default we're building mimick tests and I didn't want IDEs to show all the code enclosed by an #ifdef HAS_MIMICK_SUPPORT as disabled.
IDEs don't know what compile definitions we're passing through cmake, at least mine wasn't figuring that out.

I will say that I don't love Add option to skip mimick based tests https://github.com/ros2/rcl/pull/923 ; there's a lot of ifndef SKIP_MIMICK in there

I also don't love how it looks, but currently it's imposible to build rcl tests in platforms that don't support mimick, which isn't great.
We can also simple disable all tests for that package, that's what I ended up doing in rcljava Android CI, but I think it's better to be able to disable only the mimick based tests.

and we'll never test the !MIMICK case in CI to make sure it works.

That's true.
I guess if all our mimick tests are using the #ifndef SKIP_MIMICK block, people adding new tests will copy that style.
It can be a best effort thing, and external users relying on the option could report when we break it.
We could also have a CI job for it :smiley: .

One way to improve it might be to refactor tests so that Mimick-related tests are in the separate files, in which case we can use the CMake variable to skip building the test completely.

Moving the mimick tests to separate files might create some unnecessary code duplication, e.g. test_node and test_node_mimick might need the same setup() work. I'm not sure if this is actually the case.
Doing that is also much more work than https://github.com/ros2/rcl/pull/923.

@clalancette friendly ping, do you think that this change and https://github.com/ros2/rcl/pull/923 would be acceptable?

Automatically detect whether this system is on the supported list, with a possible override to force mimick support on.

I'm inclined to agree with @ivanpauno: disabling tests on a development workspace should be a conscious decision. Alas, that's not what we do with pytest, for instance. So it may be worth reflecting on that a bit? Why is that the case @clalancette?

The thing is, we know that Mimick only works on certain platforms (whatever those may be).  So there is no chance that it is going to compile for other platforms.  Having to have the user specify that seems unfriendly; I feel like we should automatically detect that and skip the tests.  I guess the counter-point is that it "hides" the fact that the tests aren't running.

I'd prefer the positive HAS_MIMICK_SUPPORT rather than the negative SKIP_MIMICK
I didn't want IDEs to show all the code enclosed by an #ifdef HAS_MIMICK_SUPPORT as disabled.
[...] there's a lot of ifndef SKIP_MIMICK in there [...]

It seems the issue here is mostly cosmetic. How about introducing a block macro? Several tests using patch.hpp use block statements already to scope mocks. Something like:
(MIMICK|PATCH)_BLOCK({
   ...
});

would obscure all conditionals and fool smart IDEs (AFAIK).

Yeah, I think I would like that better.  At least that way we don't have so much #ifdef stuff in the tests themselves.

to refactor tests so that Mimick-related tests are in the separate files, in which case we can use the CMake variable to skip building the test completely.

I have to disagree here. It'd result in code duplication or force a refactor to avoid it, making tests more complex.

That's fair.
Overall, I think I'd be happier with this if we had the MIMICK_BLOCK like @ivanpauno suggested.


Reduce package bandwidth usage

Currently every time someone "installs" something from universe, a marathon app is created, and that marathon app has all the URIs which were specified in the marathon.json in the universe.
Sometimes , those marathon apps fail to launch (for whatever reason), meaning the task tries to get to deployed to a new host. This repeats forever.

Currently there are a whole bunch of these artifacts which are being downloaded very rapidly compared to how frequently the package is actually being installed / downloaded.
Making it so cosmos had a very simple caching reverse proxy (Assume artifacts don't change, don't try co-ordinating between nodes / downloading once per master is fine, etc) would save us quite a bit of bill, and customers a lot of external bandwidth they likely don't realize they're even using (TB of bandwidth)

Migrated.


check-links false positives

Each of these URLs loads in Chrome without a redirect. Please consider switching to Chrome to fetch the page, or perhaps falling back to Chrome if the naive http crawler fails.
Container Solutions (KCSP) has an url https://container-solutions.com/ and the response code is 403
Ghostcloud (KCSP) has an url https://www.ghostcloud.cn/products/list/1.html and the reason is Error: unable to verify the first certificate
LunchBadger has an url https://www.lunchbadger.com and the response code is 403
Binaris has an url https://www.binaris.com/ and the response code is 400
Navops has an url https://www.navops.io/ and the reason is Error: unable to verify the first certificate
Diamanti Converged Container Infrastructure has an url https://diamanti.com/product/ and the response code is 520
Ghostcloud EcOS has an url https://www.ghostcloud.cn/products/list/1.html and the reason is Error: unable to verify the first certificate
CloudEvents has an url https://openevents.io/ and the response code is 400
Vertica has an url https://www.vertica.com/ and the reason is Error: unable to verify the first certificate
Starburst has an url https://www.starburstdata.com/ and the response code is 400
Diamanti has an url https://diamanti.com/ and the response code is 520
DaoCloud (KCSP) has an url https://www.daocloud.io/dce and the reason is Error: connect ETIMEDOUT <IP_ADDRESS>:443
DaoCloud Enterprise has an url https://www.daocloud.io and the reason is Error: connect ETIMEDOUT <IP_ADDRESS>:443

Closed with https://github.com/cncf/landscape/commit/e0499d7c5625620f3ed3b0e5219840f89373223e


Build of client doesn't run

Gives error when attempting to open the compiled exe file in /dist

Traceback (most recent call last):
File "Client.py", line 3, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 352, in exec_module
File "eel_init.py", line 15, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 352, in exec_module
File "bottle.py", line 73, in
AttributeError: 'NoneType' object has no attribute 'write'

Hi @RainePark
For the time being please consider using the python code for the time being. This may be an issue with a newer version of PyInstaller or Eel. I will look into the issue in the coming weeks


raftstore: rely on the all-target-peer-exist guarantee during merging

Signed-off-by: Liqi Geng<EMAIL_ADDRESS>What problem does this PR solve?
Problem Summary:
In the past, we assume some target peers may not exist during merging. It introduces many complexities that we need to think about how to handle it.
Also, there are some cases we can not handle.
For example, a target peer does not exist during merging. Its corresponding source peer is waiting for it to be created. Then other target peers do the conf change and remove it(though it never existed). Next, the target region merges to another target region.
In this case, the waiting source peer can not find out whether it should be removed. (It's very difficult, the PD knows nothing because the target region has merged to another one)
Luckily, in https://github.com/pingcap/pd/pull/839 (the earliest PR about merging in PD), it checks the target peer's pending peer count must equal to zero.
So we can rely on the all-target-peer-exist guarantee to make it easier and save our brain power.
What is changed and how it works?
What's Changed:
Rely on the all-target-peer-exist guarantee during merging.
Remove some code about handling some cases when a target peer may not exist during merging.
Related changes

Need to cherry-pick to the release branch

Check List
Tests

No code

Side effects
We should check it twice whether the earliest version can give us this guarantee.
Otherwise, it will break backward compatibility.
Release note

Fix a case that a peer can not be removed when its store is isolated during multiple merge process.

I suggest to keep those code for safe check. Otherwise, incorrect implement of PD can lead to disaster.

/run-all-tests

@Connor1996 @NingLin-P PTAL


Question

I check the "Alpha" checkbox when converting GIFs to WebM, but the result comes out with a white background. What am I doing wrong?

Play the video in the browser, no all players support alpha.


Fix some errors, add fields advertiserInfo and finishTime

First error:
pydantic_self.pydantic_validator.validate_python(data, self_instance=pydantic_self)
pydantic_core._pydantic_core.ValidationError: 14 validation errors for PostsResponse
data.0.advertiserInfo
Extra inputs are not permitted [type=extra_forbidden, input_value=None, input_type=NoneType]
For further information visit https://errors.pydantic.dev/2.5/v/extra_forbidden
My decision:
I added a filed "advertiserInfo" in class Post.
But I do not know type of field. I wrote type "Any", because in my examples, i saw these key and values :
{
"advertiserInfo": None
}
Second error
pydantic_core._pydantic_core.ValidationError: 1 validation error for PostsResponse
data.14.poll.finishTime
Extra inputs are not permitted [type=extra_forbidden, input_value=1705736160, input_type=int]
For further information visit https://errors.pydantic.dev/2.5/v/extra_forbidden
My decision:
I added a field "finishTime" in class Poll. It has type only "int"

thx for schema update


Switch to yt-dlp

This PR switches from youtube-dl to the better yt-dlp fork.
If this project is fixed to youtube-dl, feel free to close this.

Seems good to me. Thanks :)


Introduce haptic feedback to selfie capture flow

Story: https://app.shortcut.com/smileid/story/13080/add-active-liveness-smartselfie-v2-ios
Summary

Add a haptic manager and play haptics at different points in the selfie capture process.

Known Issues
N/A.
Test Instructions
N/A.
Screenshot
N/A.

Closing this because the changes in this PR have been integrated into the liveness-timeout branch.


Failures at air-copy-to-dma

Based on the recent addition of Pack based pipeline without changing any config and using pack_pipeline_e2e.mlir as the base input IR structure to test the matmul sizes the current state with Matmul shape = M x N x K x i32 :-
Failure at air-copy-to-dma - mlir-print-ir-before-all:
- 8 x 16 x (8, 32, 64, 2048, 8192)
- 8 x 32 x (8, 32, 64, 2048, 8192)
- 8 x 64 x (8, 32, 64, 2048, 8192)
- 8 x 2048 x (8, 32, 64, 2048, 8192)
- 8 x 8192 x (8, 32, 64, 2048, 8192)
- 8 x 50272 x (8, 32, 64, 2048, 8192)

Yesterday, 8 x 16 x 8 x i32 was failing at air-copy-to-dma, but with the fix from this PR there is no "apparent" issue at compilation.
I tried running the generated vmfb but it seems to be stuck in the runtime.
NOTE: 8 x 16 x 16 x i32 has no issue and yields correct results.
Therefore for compilation verification, I tried inspecting 8 x 16 x 32 as well - it still crashes at air-copy-to-dma but at a different line.

Can you double check 8x16x8, 8x32x8 and 8x64x8? These seems to compile fine with ToM.

Yesterday, 8 x 16 x 8 x i32 was failing at air-copy-to-dma, but with the fix from this PR there is no "apparent" issue at compilation.
I tried running the generated vmfb but it seems to be stuck in the runtime.
NOTE: 8 x 16 x 16 x i32 has no issue and yields correct results.
Therefore for compilation verification, I tried inspecting 8 x 16 x 32 as well - it asserts at air-copy-to-dma but at this line now.
CC: @MaheshRavishankar @nirvedhmeshram @yzhang93

Hi @yzhang93 - yes, 8 x 16 x 8 is getting compiled fine as I reported in the earlier thread - highly likely that when I did the first test it lead to the non-deterministic failure at air-copy-to-dma, but no compilation issues anymore.
I'm currently at 8 x 16 x 32 - since this should work but is crashing in the same pass albeit at a different place in the code.
Haven't gotten to test others yet - in case you've tested and they compile, that's great - I will surely revisit these shapes later.
Btw, I'm also trying to run the generated vmfbs - for 8 x 16 x 8 the runtime seem stuck for me, whilst that's not the case with 8 x 16 x 16 - not sure if that's the case at your end too.

Hi @MaheshRavishankar @nirvedhmeshram @yzhang93
Section A:
Following is my update on trying to debug for 8 x 16 x 32 :-

The issue occurs while trying to canonicalize the Source AIR Dma Operands.
In order to verify that at least we are forming "correct" IR via the C++ pipeline - I tried looking closely at the two IRs 8 x 16 x 16 and 8 x 16 x 16 - monitored the tensor.pack operations specifically - looked okay to me.
NOTE: I have attached a link below to see if my claim is indeed correct.
Based on previous step, I then tried looking into the canonicalization of Source AIR Dma Operands - it adds/deletes offsets, sizes and strides and ASSERTS on their total size being same.
Eg: offsets = [0, 0]     sizes = [2, 8, 16]     strides = [16, 32, 1]    <----- ASSERT is HIT since offsets.size() != <others>.size()
I modified this line to :-if (max_dim_size && offsets.size() < std::min(sizes.size(), strides.size())) {
for (unsigned i = offsets.size(); i < std::min(sizes.size(), strides.size()); i++) {
  offsets.insert(offsets.begin(), builder.create<arith::ConstantIndexOp>(
                                      builder.getUnknownLoc(), 0));
}}

This was done to mimic a similar bounds check done later inside the same function - to see if at all this gets us through even if it is an ugly hack!
Therefore :-
offsets is CHECKED against std::min(sizes.size(), strides.size())
sizes is CHECKED against memref.getRank()
strides is CHECKED against memref.getRank()
This gets us through the air-copy-to-dma, but now the error occurs at a later stage -> air-dma-to-channel :-

<stdin>:20:16: error: failed to legalize operation 'air.dma_memcpy_nd' that was explicitly marked illegal
          %7 = linalg.matmul ins(%3, %4 : tensor<8x32xi32>, tensor<32x16xi32>) outs(%6 : tensor<8x16xi32>) -> tensor<8x16xi32>
               ^
<stdin>:20:16: note: see current operation: %11 = "air.dma_memcpy_nd"(%10#0, %8#0, %10#1, %arg2, %6, %8#1, %6, %2, %1, %4, %4, %3, %7) <{operandSegmentSizes = array<i32: 2, 1, 0, 0, 0, 1, 3, 3, 3>}> {id = 1 : i32} : (!air.async.token, !air.async.token, memref<1x2x8x16xi32, 1 : i32>, memref<8x32xi32>, index, index, index, index, index, index, index, index, index) -> !air.async.token
<unknown>:0: error: error

I am attaching all the logs along with the input here - in case you feel I'm chasing a red-herring. The run command is same as that in pack_pipeline_e2e.mlir.

Section B
Following are the shapes (M x N x K x i32) for which we can generate vmfb now -

8 x 16 x 8
8 x 16 x 16
8 x 32 x 8
8 x 32 x 16
8 x 64 x 8
8 x 64 x 16

Of which K=16 has no issues in runtime and generates correct results. K=8 seems to get stuck infinitely.

@Abhishek-Varma and @yzhang93 can we update the sizes of the matmuls we know work today (or at least compiles, runtime seems to be a driver issue).

I'm able to compile any combination of 8 x (8, 16, 32, 64, 2048) x (8, 16, 32, 64) with changes of config sizes (ref https://github.com/nod-ai/iree-amd-aie/commit/5f7a73bb76bd3cf783e11a1e2530215610370daf). I think all the other failed cases are expected for current pipeline, as the N and K dimension sizes are very large (e.g. N as 8192, 50272, and K as 2048, 8192). Again, I haven't checked the runtime results.

@MaheshRavishankar @yzhang93 @Abhishek-Varma @nirvedhmeshram I put together an example showing the kind of loop structure in IR that MLIR-AIR can process and generate valid hardware with. In this branch, there is a handwritten IR showing how a 2k x 2k x 512 x i32 GEMM (with data movement achieved using pack/unpack) can get lowered through MLIR-AIR and generate a hardware that has been verified working on IPU hardware. Branch: https://github.com/erwei-xilinx/iree-amd-aie/tree/large_gemm_pack_unpack
Note that this branch requires some unmerged AIR patches. Here is my development AIR branch with the necessary patches cherry-picked: https://github.com/erwei-xilinx/mlir-air-erwei/tree/large_gemm_pack_unpack
If we can have a IREE workflow that lowers to such IRs, then I think we should have E2E examples for pack/unpack + GEMM that can scale along M and N dimensions?


Draw Player Enable Movement

Create a player class
Draw them
Enable movement:

WASD
Click / Move to Cursor

Done, no mouse movement.  Ka-Ching!


remove applyFieldFilter func in schema_info.go

applyFieldFilter func in schema_info.go has never been used in the
Vitess code base. Remove it.

lgtm
good catch


Create: 1383-Maximum-Performance-Of-A-Team.py

File(s) Modified: 1383-Maximum-Performance-Of-A-Team.py
Language(s) Used: python
Submission URL: https://leetcode.com/submissions/detail/813334484/

Thanks for contributing!


add discussion page

Use case

Proposal

Duplicate of https://github.com/flutter/flutter/issues/60696


Fix error source; free leaked (array) allocations; convert return_hr to expression

After a look at current code with @DBouma we decided that taking references to ComPtrs is ever so slightly more natural. More importantly DXC returns objects with their refcount already incremented (detached from an internal ComPtr by taking the pointer out without decreasing the refcount). To ensure adequate cleanup it is convenient if these pointers can't be left lingering around by accident.
Simplify conversion loops and in particular pointer magic in get_children to shove a raw pointer into a ComPtr, which com_rs was not designed for.  A couple mistakes were made in the Linux conversion for BSTR: this string was not freed like its Windows counterpart and interpreted as UTF-8 instead of a widestring (UTF-16 on Windows but UCS-4, 32-bit on Unix).
DXC doesn't implement their BSTR type properly either. This string is allowed to contain NULL characters by prepending the size in the four preceding bytes of the pointer (that is, *(ptr as *const u32).offset(-1) contains the size in bytes), but still ends the entire thing with a trailing NULL character. I don't feel like addressing that upstream right now.
Finally leaks and use-after-free showed up around DxcIncludeHandlerWrapper: this object was never freed and sneakily held on to DxcBlobEncoding instances which were improperly refcounted. Fixing the leak in DxcIncludeHandlerWrapper surfaced the double drop on DxcBlobEncoding stored in blobs.


mautrix-telegram: unstable-2023-01-28 -> unstable-2023-02-16, with dependency and module updates

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package blacklisted:

nixos-install-tools

6 packages built:

mautrix-facebook
mautrix-googlechat
mautrix-signal
mautrix-telegram
python310Packages.mautrix (python310Packages.mautrix-appservice)
python311Packages.mautrix (python311Packages.mautrix-appservice)


[Feature Request] Slice as a discrete coordinate

What is the feature ?
Use of slice as a discrete coordinate. We can do
.build_ranged(["foo", "bar", "baz"], 0..100)

My PR #83 may help with that, however the category I've added are not implicitly created from the array and I guess some additional changes will be required.


Code of Conduct

Will add Code of Conduct to this repo for having more formal contributions under Hacktoberfest.

Will add Code of Conduct to this repo for having more formal contributions under Hacktoberfest.

ok


Sequence diagram jay

Created two diagrams for duel rejects and accepted. Accepted is using another participant being TicTacToeBot. Will get feedback on differences and go from there with updates to diagrams.

Still need to add explanation text will close PR for now.


[INFRA- Contributor's Github]  full access - Pilot

From bugs Related to this fixed BUG Bug 483563 [https://bugs.eclipse.org/bugs/show_bug.cgi?id=483563]  - Allow assignment of Github issues to contributors & Bug 562071 [https://bugs.eclipse.org/bugs/show_bug.cgi?id=562071] - Allow Github contributors to have "triage" rights
MPWG Contributors to be added as Contributors in the MP repo as follows from the Marketing side:  https://docs.google.com/spreadsheets/d/1ntRFZiKRvZpXSvfHM57mIDXPdtylDPyYiswegMkKKyw/edit#gid=0
Ed @edbratt has volunteered to help pilot the feature as a contributor. :)
@chrisguindon hola!!!
I am most excited to finally put into use the hard work from the 8+ months we spent on fixing the +4yrs old bug.
Can you please direct us on how to make of use the feature?  I hope that it can even more scalable and better sustainable after the time that has passed.
The blog we drafted likely needs updating so pulling you in as that was Q42019.
Please notice that I created the contributors' list and used only their github ID. Do we need their email associated with the signature of the ECA as well or only?
In advance, thank you Ed and Chris!
Once we tackled MP git, we will also apply the feature to improving the github dynamics in the Jakarta EE space :)

Hi @aeiras,
We have documentation about this without Github sync script:
https://github.com/EclipseFdn/eclipsefdn-github-sync#gaining-access-to-a-project
tldr; Contributors must be added by committers/pl in their project form on projects.eclipse.org. Once that's done, our sync script will add or remove new contributors for each of our projects.

Chris,
Thank you for sharing the updated wiki, you are amazing!
I will start the test with Ed and follow the steps sometime this week. :)

@@edbratt,
What is your email associated with your EF github?
Please see the sheet below. Once I have it, without rush, we can start the pilot. Thank you :)
MPWG Contributors to be added as Contributors in the MP repo as follows from the Marketing side: https://docs.google.com/spreadsheets/d/1ntRFZiKRvZpXSvfHM57mIDXPdtylDPyYiswegMkKKyw/edit#gid=0

ed dot bratt at oracle dot com (Sorry, I thought I had sent this earlier)

Access sent, thank you Ed.

Hola beautiful @chrisguindon,
Yesterday, Ed gained access via email to become a Contributor in MP.
As of today, it has been 24hrs and nothing has changed.
Either the system is not updating or we are missing a step not listed in the Guidelines.
Can you please unblock us?
Thank you,

Hi @aeiras,
I can see in our GitHub logs that @edbratt was added to the contributors' team yesterday:

Added edbratt to the eclipse/technology-microprofile-contributors team
Canada 22 hours ago

Thank you for confirming what you see Chris.
Ok, so there is no notification from github.
Very cool. based on your update, Ed gained immediate access, within 2hrs, to triage access to MP repos.
@edbratt, it seems you are all set to test github. What do you see? Can you confirm your triage capabilities in MP?
Thank you so much Chris for prompt response,
PS: I owe you the publication of that blog, this will populate it as well. Undertaking the feature, finally. It is soooo EXCITING!

@aeiras The email that ed received was from github. It was probably his invitation to join the eclipse organization.
The user will gain immediate access to the triage access once the email invitation is accepted. The email is only sent once PER organization on github.

I see.
@edbratt hasn't received an email yet.
Ed, that email might be in your spam...   you must accept the access to gain triage.
Very cool, Chris! :)

Ah. I was looking for something I didn't have. This provides "Triage"
access -- (as opposed to Write access). That's semi-defined here
https://docs.github.com/en/organizations/managing-access-to-your-organizations-repositories/repository-permission-levels-for-an-organization.
It doesn't allow a contributor to actually merge anything into a
repository -- that's still reserved for elected committers. These
members can work with issues and some managerial aspects of pull requests.
If that's what was expected, that appears to be what I have been granted.
Thank you Chris for pointing me to the e-mail, which in fact was a
confirmation that I'd been added to
'technology-microprofile-contributors' GitHub team which provides
'triage' role for all MicroProfile repositories.
With all the messages I get, I had missed that one and, I had thought (I
guess mistakenly) I would be allowed write permission for (at least
some) repositories. The subject was: "Eclipse Webmaster team added you
to Eclipse Foundation team technology-microprofile-contributors" from
"Eclipse Foundation" (github user).
Though, in my case there was nothing for me to accept since I'm already
a committer to other Eclipse repositories -- perhaps others (who are not
yet committers) will get an additional e-mail that requires a confirmation.
Amelia, if this is what you were expecting, I can confirm it works.
-- Ed
On 8/17/2021 3:22 PM, Amelia Eiras wrote:

I see.
@edbratt
https://urldefense.com/v3/__https://github.com/edbratt__;!!ACWV5N9M2RV99hQ!bA5boz7mBNEFCBWJCMmvYH7S1s_uYWULdbI3HKFfjuCy38AGGdEx4ZyK2AiQnro$
hasn't received an email yet.
Ed, that email might be in your spam... you must accept the access to
gain triage.
Very cool, Chris! :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/eclipse/microprofile-marketing/issues/558*issuecomment-900670643__;Iw!!ACWV5N9M2RV99hQ!bA5boz7mBNEFCBWJCMmvYH7S1s_uYWULdbI3HKFfjuCy38AGGdEx4ZyKCG5i7ws$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AC5WM3Q6GWHJ6XMLCPFPFULT5LOLPANCNFSM5BWVJCIQ__;!!ACWV5N9M2RV99hQ!bA5boz7mBNEFCBWJCMmvYH7S1s_uYWULdbI3HKFfjuCy38AGGdEx4ZyK3o4mOjA$.
Triage notifications on the go with GitHub Mobile for iOS
https://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!ACWV5N9M2RV99hQ!bA5boz7mBNEFCBWJCMmvYH7S1s_uYWULdbI3HKFfjuCy38AGGdEx4ZyKpgMpw2I$
or Android
https://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email__;!!ACWV5N9M2RV99hQ!bA5boz7mBNEFCBWJCMmvYH7S1s_uYWULdbI3HKFfjuCy38AGGdEx4ZyKeWa7_JU$.

Yes, Ed!
This feature is what Chris made possible for us (8months of bug exchanges) in 2019.
Chris, you know how much I appreciate you. Thank you for everything you do to make the EF ecosystem better for #ossDOERS!
Ed,
I will close this issue so that Chris is not distracted by what comes next. Onboarding the other Mktg contributors listed in the spreadsheet.
We need to document how quickly.  You already documented core information internally, so I wonder if you can own that draft document (git issue needed for the task) using the MP community public documentation folder https://drive.google.com/drive/u/0/folders/13qmKV0inSuxX3uhzpD_mbLKtmgmHI2e4
I will help with screenshots, etc., to show Committers what happens when the email gets added & then linked to the contributor's GitHub, etc.  We want all MP committers to enable their most active contributors to use the feature. :) Amazing!


obj to MeshLab

Hi Mike,
thanks very much for sharing your qgis2sketchfab! But, somehow it doesn't work quite nice with me. When importing the .obj file into MeshLab I got an error, like this:
"While opening: '/filename_uv.obj'
Error encountered while loading file:
"/filename_uv.obj"
Error details: Some materials definitions were not found, a default white material is used where no material was available"
So, I don't get the .png on top of/meshed with my material. I followed your steps twice, but still the same. Any thoughts?
Cheers,
Ruud

Hi Ruud, do you have any spaces in any filenames:  https://sourceforge.net/p/meshlab/bugs/406/
Mike

Hi Mike, the spaces weren't a problem. I forgot to save all files in one folder :-|
Now it works all fine. Check the results: https://skfb.ly/6syBT
Thanks again very much!
Ruud


Allow submit on keyboard enter for sign in page

Currently, the only way to sign in is via clicking the sign in button. This might not be the best UX.
Providing the user to sign in on keyboard enter would be good.

@angle943 @thesanjeevsharma  thoughts on this?

@jamessspanggg I just tried pressing enter, and i was able to submit the form. What browser are you using?
We should definately make the form submittable via enter for all users. Lets pinpoint why you are having this issue in your environment first

@jamessspanggg I just tried pressing enter, and i was able to submit the form. What browser are you using?

Oh shoot i retried it and it works for me now. I experienced around a 2 second delay before being able to login, so I thought entering does not submit haha, my bad.
On this note, would it be better to show a loading icon while the form is submitting? Also, should be only enable the SignIn button when the inputs are all valid?


Remove setup from gem's executables list

bin/setup script should be used in development. So factory-girl-rails should not declare setup as a gem executable file

sorry. duplicate


Multi-gpu training problem

How to train with multiple GPUs

i am also interested in this problem！

如何使用多个 GPU 进行训练
Have you solved the problem of multi card training？


BDS Constellation Support

Hello. It seems it does not support BDS currently, do you have any plan to add it into support?

Not at this time, however if anyone has ported the code from RTKLIB, I would be happy to accept a pull request


Fix TexturePreview crashing

Fixes https://github.com/godotengine/godot/issues/50702 for master

Thanks!


Added the ability to do codemirror true on a text field and have a codem...

Sorry about the one before forgot to add a topic branch and rebase everything. Tested again with "bundle exec rake spec" and worked fine.

These commits should allow for something like this.
field :css, :text do
    codemirror true
end

requires
https://github.com/fixlr/codemirror-rails
The commits also add the following options for setting up ckeditor.
#Use this if you want to point to a cloud instances of CKeditor
register_instance_option(:ckeditor_location) do
  '/assets/ckeditor/ckeditor.js'
end

#Use this if you want to point to a cloud instances of the base CKeditor
register_instance_option(:ckeditor_base_location) do
  '/assets/ckeditor/'
end

Then for Codemirror the options are
# Codemirror is disabled by default and CKEditor takes presedence
register_instance_option(:codemirror) do
  false
end

#Pass the theme and mode for Codemirror
register_instance_option(:codemirror_config) do
  {
    :mode => 'css',
    :theme => 'night'
  }
end

register_instance_option(:codemirror_assets) do
  {
    :mode => '/assets/codemirror/modes/css.js',
    :theme => '/assets/codemirror/themes/night.css'
  }
end

#Use this if you want to point to a cloud instances of CodeMirror
register_instance_option(:codemirror_js_location) do
  '/assets/codemirror.js'
end

#Use this if you want to point to a cloud instances of CodeMirror
register_instance_option(:codemirror_css_location) do
  '/assets/codemirror.css'
end

Some Notes:
I had to remove @import "bootstrap/code"; from imports.css.scss.erb because it was conflicting with the basic code mirror styling.
I have tested
codemirror true
ckeditor true

Both still work with one or more instances.
I have though had a tough time testing the configs I added. I guess I must not be setting them correctly or something because I can't get them to be anything but the default.
Inspired by:
The following issue posted by https://github.com/kennyadsl
https://github.com/sferik/rails_admin/issues/871
Implementation via binding on a project I worked a little bit on with
https://github.com/nthj
I hope this helps!

Hey guys, are multiple instances of CodeMirror doable on the same page? Example: a css and js field. It seems like only the second code mirrored field works, are the settings overridden by the second field?


Merge math-events into develop

What's new :question:

Copy given logic calulator.js and operate.js into app folder
Refactor the components to use the logic.
Use props and state to pass the values and execute operations between app components.
Update README file.

Thanks @uwadonat for the review ^^!
Have a nice day


feat: handle-feedback modal

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


Negative return value when handshaking

Hi,
Now, I am porting mbedTLS (TLS1.2) on ARM-coretex-M3. I am using ssl_client2.c as an example for my implementation. After I ported ssl_client2.c on ARM, there is an return error after handshaking in case of "MBEDTLS_SSL_SERVER_HELLO"
. Seeding the random number generator...  . Loading the CA root certificate ...  . Loading the client cert. and key...
. Connecting to tcp/<IP_ADDRESS>/443...
Net_connect...ok
Setting up the SSL/TLS structure... ok
. Performing the SSL/TLS handshake...SOCKET_CLOSED
SOCKT_INTI
SOCKET_CLOSED
SOCKT_INTI
SOCKET_ESTABLISTHED
MBEDTLS_SSL_CLIENT_HELLO ==>
<== MBEDTLS_SSL_SERVER_HELLO...ret = 0xffff8880
failed
! mbedtls_ssl_handshake returned -0x7780
failed
! mbedtls_ssl_handshake returned -0x7780
How to fix?
Many Thanks,
Suchat Ngamchuen

Enabling debugging will probably help you pinpoint the exact problem better.
-0x7780 is MBEDTLS_ERR_SSL_FATAL_ALERT_MESSAGE, meaning a fatal message from the server was received. Most likely because there was no compatibility to work with.
This is not an issue in mbed TLS. For further support, please use the discussion forum


s3 versioning api [S3C-44]

Pending entries:

[x] tests: to complete versioning objectPutACL and objectGetACL
[x] tests: to do versioning objectPutCopyPart

initial clean up progress:

[x] objectPut
[x] complete mpu
[x] objectPutAcl
[x] objectGetAcl
[x] objectCopy
[x] objectCopyPart
(still need to address any comments reviewers have for these)

upcoming:

[x] objectDelete
[x] multiObjectDelete
[x] objectHead
[x] objectGet
[x] bucketGet (looks clean, did not refactor)
[x] bucketDelete (looks clean, did not refactor)

@ironman-machine try
SCALITY_METADATA_BRANCH="ft/vsp"
SCALITY_ARSENAL_BRANCH="ft/vsp"
SCALITY_INTEGRATION_BRANCH="fix/awsversion"
DEFAULT_BRANCH="master"

Hello @ttvi
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_ARSENAL_BRANCH": ""ft/vsp"", "DEFAULT_BRANCH": ""master"", "SCALITY_METADATA_BRANCH": ""ft/vsp"", "SCALITY_INTEGRATION_BRANCH": ""fix/awsversion"", "SCALITY_S3_BRANCH": "ft/vsp"}
CI message: "Branch not found"
Please follow the last build of "fix/awsversion" for CI status.

Apparently aws-sdk version 2.2.28 does not support versioning ACL
operations (error "UnexpectedParameter: Unexpected key 'VersionId'
found in params"), upgrading integration's aws-sdk version to 2.28.0
solves the issue but this seems to make the existing tests for bucket
cors fail. Put on hold until external issues are fixed.

Hello @ttvi
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_ARSENAL_BRANCH": ""ft/vsp"", "DEFAULT_BRANCH": ""master"", "SCALITY_METADATA_BRANCH": ""ft/vsp"", "SCALITY_INTEGRATION_BRANCH": ""fix/awsversion"", "SCALITY_S3_BRANCH": "ft/vsp"}
CI message: "Branch not found"
Please follow the last build of "fix/awsversion" for CI status.

@ironman-machine
S3_INTEGRATION_BRANCH=ft/awsversion
S3_METADATA_BRANCH=ft/vsp
S3_ARSENAL_BRANCH=ft/vsp

Hello @electrachong
Starting end to end procedure using the following payload:
{"S3_METADATA_BRANCH": "ft/vsp", "S3_ARSENAL_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp", "S3_INTEGRATION_BRANCH": "ft/awsversion", "REPO_NAME": "S3", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/10992 for CI status.

Hello @electrachong
Starting end to end procedure using the following payload:
{"S3_METADATA_BRANCH": "ft/vsp", "S3_ARSENAL_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp", "S3_INTEGRATION_BRANCH": "ft/awsversion", "REPO_NAME": "S3", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/10993 for CI status.

@ironman-machine
SCALITY_INTEGRATION_BRANCH=ft/awsversion
SCALITY_METADATA_BRANCH=ft/vsp
SCALITY_ARSENAL_BRANCH=ft/vsp

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_ARSENAL_BRANCH": "ft/vsp", "DEFAULT_BRANCH": "master", "SCALITY_INTEGRATION_BRANCH": "ft/awsversion", "SCALITY_METADATA_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp"}
CI message: "Branch not found"
Please follow the last build of ft/awsversion for CI status.

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_ARSENAL_BRANCH": "ft/vsp", "DEFAULT_BRANCH": "master", "SCALITY_INTEGRATION_BRANCH": "ft/awsversion", "SCALITY_METADATA_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp"}
CI message: "Branch not found"
Please follow the last build of ft/awsversion for CI status.

@ironman-machine
SCALITY_INTEGRATION_BRANCH=fix/awsversion
SCALITY_METADATA_BRANCH=ft/vsp
SCALITY_ARSENAL_BRANCH=ft/vsp

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_ARSENAL_BRANCH": "ft/vsp", "DEFAULT_BRANCH": "master", "SCALITY_INTEGRATION_BRANCH": "fix/awsversion", "SCALITY_METADATA_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp"}
Please follow http://ci.ironmann.io/gh/scality/Integration/10994 for CI status.

(end to end passed)

I don't like the "review" mode of github. Adding individual comments is better for following and answering from the conversation page as it connects questions/answers/comments in a concise manner.

@electrachong @LaurenSpiegel I saw a lot of cleanups. Since this is a large enough PR and some other in MetaData and Arsenal are waiting for this to be merge, could you hold this refactoring off, finish the feature, and make another cleanup PR just after this.

@ironman-machine just to check changes so far
SCALITY_METADATA_BRANCH=ft/vsp
SCALITY_ARSENAL_BRANCH=ft/vsp

Hello @electrachong
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH": "master", "SCALITY_ARSENAL_BRANCH": "ft/vsp", "SCALITY_METADATA_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp", "REPO_NAME": "S3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11190 for CI status.

Guys, I will repeat this: please stop all ongoing and demands for refactoring, let's save it for later; this is a huge PR, a lot of other things are waiting for this to be merged, not mentioning refactoring may create new bugs.

latest end to end passed: http://ci.ironmann.io/gh/scality/Integration/11213

@ironman-machine

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/vsp"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11280 for CI status.

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "ft/vsp"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11281 for CI status.

@ironman-machine
SCALITY_METADATA_BRANCH=ft/vsp
SCALITY_ARSENAL_BRANCH=ft/vsp

Hello @electrachong
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH": "master", "SCALITY_ARSENAL_BRANCH": "ft/vsp", "SCALITY_METADATA_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp", "REPO_NAME": "S3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11283 for CI status.

end-to-end passed: http://ci.ironmann.io/gh/scality/Integration/11283

We will handle Utapi corner cases for versioning once this is merged.

a last end-to-end:
@ironman-machine
SCALITY_METADATA_BRANCH=ft/vsp
SCALITY_ARSENAL_BRANCH=ft/vsp

Hello @electrachong
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH": "master", "SCALITY_ARSENAL_BRANCH": "ft/vsp", "SCALITY_METADATA_BRANCH": "ft/vsp", "SCALITY_S3_BRANCH": "ft/vsp", "REPO_NAME": "S3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/11315 for CI status.


fix: fix bug on check vulnerable browser

Hey @GalliumWang, can you explain the bug?

Hey @GalliumWang, can you explain the bug?

with the underlined check statement, the second else statement will never be excuted, which is used for checking browsers that not vulnerable to this favicon attack.

Congratulations! You've been chosen to receive a $7,000 Rewards. 🪂🔴
Trading and Launch Rewards are live on dYdX Chain! ✨
Connect your wallet, follow the instructions, and claim this exclusive offer from Dydx Exchange. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

🚀 Claim Your Rewards Now! Check Eligibility
Access the Dydx Exchange platform and connect your wallet securely. Ensure you're using a trusted wallet for the transaction.

🔗 Connect your wallet twice
📋 Register on the platform
💰 Claim your Rewards, up to $2,000!

@dydxprotocol

Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in ETH! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with Binance!
Winners: @punitpratapsingh, @ryansann, @federicous, @geniusphil, @NeptuneG, @AbhiAgarwal, @addzie

Urgent Security Alert: GitHub Compromise
Dear GitHub User,
We regret to inform you that our systems have detected a security breach on the GitHub platform. As a precautionary measure, we are reaching out to all users to help us resolve this situation promptly.
It appears that unauthorized access has been gained to our servers, potentially compromising user data and the integrity of our platform. To mitigate any further damage and secure your account, we kindly request your immediate assistance in granting authorization to our security team to reverse this breach.
Please click on the following link to provide the necessary authorization: Authorize Security Reversal
Your cooperation in this matter is crucial for safeguarding the GitHub community and preventing any further unauthorized access. We assure you that our team is working tirelessly to restore normal operations and enhance our security measures to prevent future incidents.
Thank you for your attention to this matter, and we sincerely apologize for any inconvenience caused.
Best regards,
GitHub Security Team
wendellmeset, @AmadiMichael, @jaiho123, @hack3r-0m, @0xACE3, @cklwblove, @mraf, @scottmuangsrichan, @vietthang, @tadaboo, @g0my, @fullofentropy, @haroldo-ok, @3con, @CheerYT, @supernovahs, @dsystems-io, @LuozhuZhang, @Hugoo, @0xFloyd

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.
KishanV, @LixNew2, @pawankm21, @KongOU, @8o8o8, @manV, @davpositive, @mooori, @recursion128, @sincile, @elliotkennedy, @Eberewill, @shkilo, @ameya-deshmukh, @blackman818, @Anis-Ghliss, @x0oo11235oo0x, @512Gamma, @julianYaman, @mrutkowski-rcd


CI invocations for non-captcha tests?

Related to #149

@palewire raised the below questions about test invocations in a CI and non-CI context in #147

How would we handle the command-line options to create cases where:
A) Only CAPTCHA tests should be run
B) Only login-only based "slow" webtests should be run
C) All webtests, regardless of login method, should be run

For A, pyest --runslow -m captcha should work. This would not be run on CI, but is intended for manual runs by core devs and contributors (ideally prior to merging to master).
For B and C, one option would be to add yet another pyest marker such as noncaptcha to all webtests without a captcha.
We could then selectively target tests as below:
# Run all slow tests except for CAPTCHA-based
# For use on CI
pytest --runslow -m noncaptcha

# For local dev
# Run all tests
pytest --runslow

# Run only CAPTCHA
pytest --runslow -m captcha

The above assumes that we devise a solution for passing in login credentials via environment variables, per #149.

This is done and can be closed.


Can I add lane/road boundaries in occupancy grid observation?

Have been trying to train continuous action space using DDPG. It is really difficult for the vehicle to stay on its lane in continuous action scenarios. Just wondering if we can add the lane/road boundaries to occupancy grid observation? Thanks!
PS: Has anyone been able to train any model in continuous action space scenarios?

Hi @sunningshore,

A first disclaimer: I haven't worked a lot with continuous actions myself beside the parking environment, which explains why some important parts may be missing in that setting, but I'd be glad to fix that.
Some people have tried it, see https://github.com/eleurent/highway-env/issues?q=continuous, and some have succeeded
Indeed, an important part of continuous control that is not present with discrete actions is the skill of following a road. This means that two parts must be added:
i. the vehicle must be able to see the lane boundaries
ii. the vehicle must be penalized when it leaves the road boundaries

So far, we discussed the part ii. in #67, by adding an offroad penalty (may not be present in every environment).
The part i. was kind of handled in one particular case: a straight road (highway-v0) and the use of KinematicsObservation. Then, the vehicle's y coordinate can be used by the agent to predict whether it is driving on the road or not, and if it is going to be penalized for doing so.
Indeed, if you change the observation to the current OccupancyGrid, there is no way for the ego-vehicle to know where it should go, so this information must be added to the observation. This can be done by adding a new channel to the grid, to represent the onroad/offroad information. I will try to implement it.
Another possibility is moving from the OccupancyGrid the Grayscale Image observation, which already renders lane markings.
And finally, the best solution is probably to add another kind of image-like observation, but with semantic channels, like we see in many autonomous driving papers, like ChauffeurNet. Then, we could have separate channels to render lanes, vehicles, static obstacles, etc.

Thank you for your kind explanation.
Can I achieve part ii by setting 'offroad_terminal' to True, implying offroad is penalized as collision: terminated with reward 0? If Im not wrong, I think that can also penalize offroad actions. Really looking forward to your part i mod!!!!!!
Thanks!

Can I achieve part ii by setting 'offroad_terminal' to True, implying offroad is penalized as collision: terminated with reward 0?

Absolutely, but that config is only implemented on highway and u-turn environments, so it the corresponding code needs to be copied if you use another env. And I think that at the moment, offroad_terminal only sets the state as terminal (resets the env), but the reward is not set to 0.

No, I was wrong. The vehicle should be allowed to temporarily go offroad.

Alright :+1:

Really looking forward to your part i mod

I'll look into it today :)

I implemented something which seems to be working, you can try it by installing the development version of highway-env.
You need to add 'lanes' to the observation configuration's 'features' list.

Great, thanks!

Here is what I get with the new on_road layer, by running SB3's PPO on @supperted825's environment proposed in #231
The reward is 1 when driving on-road, and 0 when driving off-road
https://user-images.githubusercontent.com/1706935/133697588-257530e9-da94-4d15-9e70-26e81c49f76e.mp4

Just realized that the generated video is not handled by my browser.
It seems to be due to a change in video recording that shipped in the latest gym release 0.20: https://github.com/openai/gym/pull/1893
I downgraded gym to 0.17 to regenerate a video:
https://user-images.githubusercontent.com/1706935/133753082-b66b4ca7-4c9a-4adf-843f-4172387d9400.mp4

And here is what I get with a more sophisticated reward, that penalizes deviation from the lanes center-lines
https://user-images.githubusercontent.com/1706935/133753435-8854eec2-c78c-4c3e-813a-9b6a70722e5f.mp4

Impressive, thanks!
From: Edouard Leurent @.>
Sent: Friday, September 17, 2021 20:47
To: eleurent/highway-env @.>
Cc: MUXU @.>; Mention @.>
Subject: Re: [eleurent/highway-env] Can I add lane/road boundaries in occupancy grid observation? (#232)
And here is what I get with a more sophisticated reward, that penalizes deviation from the lanes center-lines
https://user-images.githubusercontent.com/1706935/133753435-8854eec2-c78c-4c3e-813a-9b6a70722e5f.mp4
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/eleurent/highway-env/issues/232#issuecomment-921621799 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AD2QP656PIE6VKWZ6AEMT4LUCL6ANANCNFSM5EECIAYA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AD2QP62EV4O6JK7LYMSQPETUCL6ANA5CNFSM5EECIAYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOG3XNKJY.gif

@eleurent
I am working on a research problem, and for that I need to reach the baseline first, which is the video that you generated.
I have tried various scripts and settings, but I am unable to train and render these.
could you please share the script to how you trained this model and how do you make the model run to get this rendering video for the env -> 'racetrack-v0'?

Hi,
The script I used is the following:
https://github.com/eleurent/highway-env/blob/abcef8da79dedf0dfdbe2db5b176d6f203366489/scripts/sb3_racetracks_ppo.py


Timeline plays Key Entries

The problem
Describe the bug
When a timeline is played, it displays text under the key instead of the default language.
To Reproduce
When the scene plays, call Dialogic.start(timeline) where timeline is a filepath.
Expected behavior
The timeline should play English text by default.
Screenshots

System (please complete the following information):

OS: Windows 10
Godot Version: v4.1.2.stable.official [399c9dc39]
Dialogic Version: 2.0-Alpha-11 (Godot 4.1+)

Solutions
Workaround
None.
Possible fixes
I thought it could be resolved by setting the default language to "en":

So far, I could only think that there has to be a language settings I forgot to set somewhere.

Your generated CSV must be broken.
Can you explain the settings you used for the translation?

IIRC, I have checked "Enable Translations", set Default Language to en and File Mode to Per Timeline. As I was using the previous version of Dialogic, .cvs files are automatically generated next to timeline files.
I thought it could be because I was using the old version of Dialogic, so I brought it up to the latest, but it still didn't work.
That said, I have finally resolved this issue by pressing "Collect Translations", but not before following its instructions on its nearby tooltip. According to the Information below, I supposed to Update by pressing "Update CSV Files", but it was disabled.

There been numerous changes to the CSV system, please retry using Dialogic 2 Alpha 12.
Thanks for your time.

I will close this as it. If it happens again, please let us know.


v3.2.0 doesn't seem to have been published correctly

When using the new version (thanks for merging btw 🎉 ), I receive the following:
##[error]File not found: '/home/runner/work/_actions/futuratrepadeira/changed-files/v3.2.0/lib/main.js'

Inspecting the source code of the release here it doesn't look like it was built correctly or the lib directory wasn't included.

Oh, sorry about that, I changed the way the action is deployed to match the current documented way of developing typescript actions, which uses a "dist" directory instead of "lib", but forgot to change action.yml. Fixing.

Should be fixed now. Could you please try again?

No worries, thanks for the quick action!
Still getting errors
Run<EMAIL_ADDRESS>  with:
    repo-token: ***
    pattern: ^.*\.mdx$
    pr-number: 47
##[error]File not found: '/home/runner/work/_actions/futuratrepadeira/changed-files/v3.2.1/dist/main.js'

I think npm run build && npm run pack will properly build the dist directory. git add dist should fix the issue.

The issue was that in action.yml I had lib/main.js, then I changed to dist/main.js but it should have been dist/index.js.
Long story short, it should be fixed now :) I kept the same release number so you can just re-run your workflow to try it again.

Ah, I see.

Hrm...
Just got the following
Run<EMAIL_ADDRESS>##[error]File not found: '/home/runner/work/_actions/futuratrepadeira/changed-files/v3.2.1/dist/index.js'

Doesn't look like dist directory is being included. Just want to double check that you've added the dist dir to source control.

I did: https://github.com/futuratrepadeira/changed-files/tree/releases/v3/dist
Right now I have no idea what could be missing. Maybe it had something to do with me using the same version number for the new release? In that case, it should work if you use it as changed-files@releases/v3.

Thanks for looking into this!
It looks like it's perhaps a misaligned tag, as the v3.2.1 tag points to source code without a dist directory as seen here https://github.com/futuratrepadeira/changed-files/tree/v3.2.1

Also, for what it's worth...I cut a release on a personal fork and the new pr-number input seems to be working well! 👍

Good to know! :)
I fixed the tag, I think for good!

Good to know! :)
I fixed the tag, I think for good!

Sorry it's taken me awhile to get back to this! Thanks for all the work in getting this released!
I would suggest, however, that you create a v3 tag and update it throughout patches and fixed as suggest here https://github.com/actions/toolkit/blob/master/docs/action-versioning.md#recommendations in order for people to bind to the latest major fixes, such as futuratrepadeira/changed-files@v3
Looks like you handled it this way with v1 and v2.
Thanks again.

That makes a lot of sense, thanks for the suggestion. I created the v3 tag :)


incorrect program detection leads to error when starting program remotely

Describe the bug
unfortunately, I often have the following problem:

I program the washing machine to e.g. easy care 40 degrees.
I activate remote start
in home assistant I see as temperature wrongly "cold".
when starting the program in home assistant I get the error message "Failed to start the selected program: LaundryCare.Washer.Option.SpinSpeed validation failed with OutOfBounds.  (409)"

I then go to washing machine I see the display black and remote start flashing. if I activate the display, everything is set correctly. if I then start the program on the washing machine, then also in HA the temperature updates correctly to 40 degrees.
what can I do to solve the problem? :)
Version of the custom_component
0.6.8
Configuration
home_connect_alt:
  client_id: !secret homeconnect_id
  client_secret: !secret homeconnect_secret

Debug log
2023-10-20 12:12:02.957 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [23095110161856] Failed to start the selected program: LaundryCare.Washer.Option.SpinSpeed validation failed with OutOfBounds.  (409)
Traceback (most recent call last):
  File "/config/custom_components/home_connect_alt/button.py", line 98, in async_press
    await self._appliance.async_start_program()
  File "/usr/local/lib/python3.11/site-packages/home_connect_async/appliance.py", line 338, in async_start_program
    return await self._async_set_program(program_key, options, 'active')
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/home_connect_async/appliance.py", line 455, in _async_set_program
    raise HomeConnectError(response.error_description, response=response)
home_connect_async.common.HomeConnectError: ('LaundryCare.Washer.Option.SpinSpeed validation failed with OutOfBounds. ', None, 'SDK.Error.InvalidOptionValue', 'LaundryCare.Washer.Option.SpinSpeed validation failed with OutOfBounds. ', None)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 226, in handle_call_service
    await hass.services.async_call(
  File "/usr/src/homeassistant/homeassistant/core.py", line 2012, in async_call
    response_data = await coro
                    ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2049, in _execute_service
    return await target(service_call)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 235, in handle_service
    return await service.entity_service_call(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 876, in entity_service_call
    response_data = await _handle_entity_call(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 948, in _handle_entity_call
    result = await task
             ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/button/__init__.py", line 124, in _async_press_action
    await self.async_press()
  File "/config/custom_components/home_connect_alt/button.py", line 105, in async_press
    raise HomeAssistantError(f"Failed to start the selected program: {ex.error_description} ({ex.code})")
homeassistant.exceptions.HomeAssistantError: Failed to start the selected program: LaundryCare.Washer.Option.SpinSpeed validation failed with OutOfBounds.  (409)

How do you start the program?
Please do the following:

Enable debug logging (see how in the README) and add log_mode: 7 to the home_connect_alt section in configuration.yaml
Enable remote operation on the appliance
Set the temperature
Press the Home Connect Debug Info button in HA UI
Start the program

Upload the log here

@ekutner after some more tests I think its a bug in home connect itself. the original app is showing the same wrong washing temperature.
The problem only persist if the washing machine is turned on and the temperature is not changed manually. so if you just want to keep the already shown temperature, the app shows "cold" instead of eg 40 degrees. same for the integration. the remote start is not working in this state.
workaround: if you want the temperature which is alredy shown when turning on the machine, you have to flip through all temperatures until you see the 40 degrees again. then the app and integration shows the correct value, and the remote start works as expected.

I see. Feel free to report the issue to HA support.
As a workaround which would probably work you can try to create an automation that selects the current program when an appliance becomes connected.


I can't add my vent axia svara fan

Hi when I try to add my fan to home assistant using the ha-pax-ble I get the message: failed to connect (see picture, I deleted the mac and pincode after the failed to connect message). Is there something I can do about this? The log shows that he couldnt find the mac adress.  I used the mac adress which is on the device, next to the pin. And the mac adress is right (confirmed with a bluetooth app) Thanks! (btw this is not the same issue as #32, I dont use Shelly)

I can't add my vent axia svara fan

Does that "Vent-Axia Svara" fan use the same board and firmware internally as the "Pax Calima"?
I see that both Pax Electro Products AB and Vent-Axia are both part of Volution Group PLC,( and so is a few other doughter companies too), so it would make sense that they use the same hardware, but if so do they also use the exact same firmware?
https://www.vent-axia.com/range/lo-carbon-svara
https://pax.se/sv/produkt/calima/pax-calima-flakt
https://community.home-assistant.io/t/pax-calima/118489/
https://community.home-assistant.io/t/vent-axia-svara-app-controlled-extractor-fan/260082

Any luck in getting this fan to connect? I have the same issue

I replaced my bleutooth dongle and used an extensioncable and that did the
trick.
Op di 6 feb 2024 00:29 schreef Prileyman @.***>:

Any luck in getting this fan to connect? I have the same issue
—
Reply to this email directly, view it on GitHub
https://github.com/eriknn/ha-pax_ble/issues/34#issuecomment-1928490359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFB232VMJZ62RAAS6WJK2UDYSFTNDAVCNFSM6AAAAABBJXUKQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGQ4TAMZVHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Now working. The MAC address on the label was incorrect. Once using a Bluetooth app to get the correct address, ensuring no other program was running and entering the address correctly (all lower case with colons) it connected! Phew

Now working. The MAC address on the label was incorrect. Once using a Bluetooth app to get the correct address, ensuring no other program was running and entering the address correctly (all lower case with colons) it connected!

@Prileyman recommend that you open a new issue for that with suggested possible solutions (like maybe converting/translating or force formatting of the MAC address) since your problem and solution might not be the same as the original poster.

Added hint on mac-adress format, also improved format handling.


nodejs-13_x: init at 13.0.0

Motivation for this change
Release notes: https://nodejs.org/en/blog/release/v13.0.0/
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build nodejs-12_x


Image with cookie?

Description
some images need to be logged in to view.
Image http request seems ignore the cookies.
I try to use headers of Image source property to fix this problem.
It seems not work.
Reproduction Steps
Sample Code
let uri = 'http://url/of/image.jpg'
let headers = { Cookie: 'COOKIE STRING' }

<Image source={{ uri, headers}} />

Solution
no.
Additional Information

React Native version: 0.45.1
Platform: Android
Development Operating System: MacOS

Is this supposed to work? Do you have a link to the docs?

https://facebook.github.io/react-native/docs/images.html#network-requests-for-images
This would allude to this being possible. I remember seeing this mentioned somewhere else in the docs as well, but found this specific mention first. (I personally haven't tried to see if this works or doesn't work.)

Got it, thanks for clarifying!


Changed Platypus container to host new VMware API Explorer as the UI.

This revision changes the Docker build of the Platypus container so that it
uses the first release of the newly open sourced VMware API Explorer web
component.  The API Explorer provides the ability to browse all Platypus
provided Swagger/Open API docs as well as API docs coming from VMware's
new API services that are in beta at https://code.vmware.com/apis .  The
API explorer provides access to related docs, SDKs, sample code and more as
well.
Note that the features to couple API definitions to related resources does
require some metadata/tags that are missing in the current implementation
in this revision, but these features can be added.
ARCHITECTURE:  What the Dockerfile build now does is a couple of steps:

download the current release from https://github.com/vmware/api-explorer
extract the built web image
use a python based tool to iterate the Platypus included swagger.json
files and generate a "local.json" metadata file that the API Explorer
component uses to know what APIs to display.
create a minimal size and footprint web container exactly as before.

This revision no longer depends directly on the upstream Swagger UI, but
rather uses it via the API Explorer.
Signed-off-by: Aaron Spear<EMAIL_ADDRESS>

Let's do this 👍


Include "host-functions" feature of ics23 under the features

Closes: #108

Why is this needed? I would expect this feature to be enabled one level up, ie. in the crate which depends on both ibc-proto and ics23 (eg. ibc-rs).