Issue 175: cargo shuttle init (without bonus)

Motivation
This PR adds cargo shuttle init feature to #175. This is a partial solution to the feature as it does not include separate cargo shuttle init commands for each supported framework, e.g. axum and rocket.
I hope to continue on these separate commands on a separate PR and then start refactoring Shuttle.init method inside cargo-shuttle/src/main.rs as I think we could extract out some logic and add unit tests to the operations on Cargo.toml.
Changes

Add an Init(InitArgs) variant to Command in args.rs
Call cargo init --lib via the cargo crate
Automatically add the crate-type = ["cdylib"] to Cargo.toml
Add the shuttle-service dependency to Cargo.toml

How it works

Run cargo shuttle init initializes Cargo.toml and src/ in the current directory (similar to cargo init)
Run cargo shuttle init newproject initializes in ./newproject directory; if the directory newproject does not exist, it will be created
Run cargo shuttle init /full/path/to/newproject initializes Cargo.toml and src/ in the given path; if any directory does not exist, it will be recursively created

To-Dos

Update documentation?

@chesedo I opted to use cargo-edit on version "0.8.0" for fetching the latest shuttle-service version (could use cargo to do the same but it'd add quite a bit more code since we'd fetch all Summary instances of shuttle-service and then manually rank them to extract the latest one).
However, I cannot use cargo-edit = "0.9.0" due to a dependency conflict from cargo (not sure if we care?):
Caused by:
  failed to select a version for `libgit2-sys`.
      ... required by package `cargo v0.59.0`
      ... which satisfies dependency `cargo = "^0.59.0"` of package `cargo-shuttle v0.3.0 (/Users/nahua/projects/shuttle-hq/shuttle/cargo-shuttle)`
  versions that meet the requirements `^0.12.24` are: 0.12.26+1.3.0, 0.12.25+1.3.0, 0.12.24+1.3.0

  the package `libgit2-sys` links to the native library `git2`, but it conflicts with a previous package which links to `git2` as well:
  package `libgit2-sys v0.13.0+1.4.1`
      ... which satisfies dependency `libgit2-sys = "^0.13.0"` of package `git2 v0.14.0`
      ... which satisfies dependency `git2 = "^0.14"` of package `cargo-edit v0.9.0`
      ... which satisfies dependency `cargo-edit = "^0.9.0"` of package `cargo-shuttle v0.3.0 (/Users/nahua/projects/shuttle-hq/shuttle/cargo-shuttle)`
  Only one package in the dependency graph may specify the same links value. This helps ensure that only one copy of a native library is linked in the final binary. Try to adjust your dependencies so that only one package uses the links ='libgit2-sys' value. For more information, see https://doc.rust-lang.org/cargo/reference/resolver.html#links.

  failed to select a version for `libgit2-sys` which could resolve this conflict


Error while installing sqlite3 for electron under linux 7.10

Hello! Im trying to install sqlite3 for electron under Linux 7.10.
I search how to fix that without luck.
Thanks!
node-gyp: v3.7.0
node-pre-gyp: v0.10.0
node: v6.11.4
electron: v1.4.16

Try<EMAIL_ADDRESS>and watch #1006 if you are still having trouble.


ms_getgroups fails in desktop mode

we should return an empty table

:tada: This issue has been resolved in version 4.27.14 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add dryRun flag for backplane-cli

What type of PR is this?
feature
Which Jira/Github issue(s) does this PR fix?
Resolves https://issues.redhat.com/browse/OSD-18387
Special notes for your reviewer

Tested in my dev cluster with the dry-run flag and it works as expected to only show the JSON format of the test job without creating it.
The dry-run feature has been implemented in the backplane-api, so this PR is just to add dryRun flag to call the dryRun API.

Pre-checks (if applicable)

[x] Validated the changes in a cluster
[ ] Included documentation changes with PR

Codecov Report

Merging #210 (8204695) into main (949ee51) will increase coverage by 0.08%.
Report is 2 commits behind head on main.
The diff coverage is 83.33%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
+ Coverage   51.53%   51.62%   +0.08%
==========================================
  Files          51       51
  Lines        3415     3425      +10
==========================================
+ Hits         1760     1768       +8
- Misses       1364     1365       +1
- Partials      291      292       +1

Files Changed
Coverage Δ

cmd/ocm-backplane/testJob/createTestJob.go
70.34% <83.33%> (+0.71%)
:arrow_up:

I think we should better not print the job id if it is a dry run?
$ ocm backplane2 testjob create -d
...
...

TestId: openshift-job-1695719774

I think we should better not print the job id if it is a dry run?
$ ocm backplane2 testjob create -d
...
...

TestId: openshift-job-1695719774

if err != nil {
    utils.WriteJson(w, http.StatusInternalServerError, &utils.ErrorResponse{
        StatusCode: http.StatusInternalServerError,
        Message: fmt.Sprintf("Failed running dryrun for job on the cluster, error: %s", err),
    })
    return
}
utils.WriteJson(w, http.StatusOK, jobs.TestJobResult{
    TestId:  jobObjects.Pod.Name,
    Message: buf.String(),
    Status:  string(corev1.PodPending), // Always starts with Pending
})

@bmeng  - Thanks for your review. I think it's just defined in the backplane API, if we think the TestId should not be printed, we can update in API code then but it's a separated issue from this PR.

I think we still can deal with it from https://github.com/openshift/backplane-cli/pull/210/files#diff-85bea5392ad379d93bf38a12b6a10a1b754877f9fd898d19ebaaf89c1a0f860fR184
But it is not a big deal, let's leave it as is.

/label tide/merge-method-squash
/lgtm


az webapp log tail --filter option no longer available?

"To filter specific events, such as errors, use the --Filter parameter. For example:"
https://docs.microsoft.com/en-us/cli/azure/webapp/log?view=azure-cli-latest indicates the command should be --only-show-errors, there is no mention of the --filter option

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

ID: bfc316af-0d30-b449-a04f-3019b7a16bc1
Version Independent ID: 01c4c435-fda9-374a-70b3-f1d7f13c7651
Content: Enable diagnostics logging - Azure App Service
Content Source: articles/app-service/troubleshoot-diagnostic-logs.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@blueghostuk Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for feedback @blueghostuk. I've issued fix addressing this. The doc should be updated shortly.


Android Basics: change-app-icon

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-change-app-icon?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-change-app-icon#5
In which task and step of the codelab can this issue be found?
Step 6 ¨Change the app icon¨ - Move vector drawable files into -v26 directory
Describe the problem
it says that ¨you may notice that the foreground asset is located in the drawable-v24 folder, while the background asset is in the drawable folder.¨, but the background is the one inside drawable-24 folder, not the foreground.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio 4.2.1
Build #AI-202.76<IP_ADDRESS>51085, built on May 10, 2021
Runtime version: 11.0.8+10-b944.6916264 x86_64
VM: OpenJDK 64-Bit Server VM by N/A
macOS 10.16
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 8
Registry: external.system.auto.import.disabled=true
Additional information
Include screenshots if they would be useful in clarifying the problem.

@Maximgbb
The issue that you presented to us is extremely important, and I thank you for bringing it to our attention. The content is updated and will be published soon.


Login Flow: redirect back to article if user is already logged in

Given the user is logged in via the publisher domain. On mobile safari third-party cookies are disabled by default. No access to cookies mean that when the user views an article via the CDN, the publisher backend is not able to determine whether the user is logged in.
When the user clicks the login button, the login flow opens on the publisher domain. There the login flow is able to determine a logged in user and should redirect the user back to the article (after saving the readerid user mapping).
Our sample doesn't support this use case yet and shows the login dialog.
//cc @dvoytenko @ashwinlimaye @andreban

@sebastianbenz This is not 100% accurate, so I wanted to first confirm the technicals here. In the example you describe, if user is already logged in with publisher backend and the user is accessing the CDN document in the same browser context then regardless of the 3rd party iframe or not - the authorization request will see all the publisher's cookies (assuming they are set of the same or a parent domain).
The situation you describe would happen if the login dialog were to open in a different browser context. For instance, if we were to open the document in a WebView and open the login dialog in SFSafariViewController or CCT. Then - correct, your case applies and the expectation is that the user will be automatically logged in. Besides prototyping it this way, we can also emulate this case by putting authorization and login endpoints on different domains. If possible - then definitely, it would be a good example. Not sure if this would be easy enough to set up without over-bloating the sample code.
Another option I'm trying to pursue is to have iGSA folk to create SFSafariViewController-based login flow via an experiment - but I don't know the timing on that yet.
One thing you could do now is simply open your sample site on a separate domain for testing. This way you can emulate separate browser context.


Wrapper for SVI GP

With automatic batch selection?
Along the lines of SparseGP.

At the moment we do mini batches by just randomising and slicing up the data. Once a run through all the data has been completed, we randomise and repeat. How does the SparseGP stochastics work? Are you suggesting doing inference for each output separately (with shared model parameters) or something so we can deal with missing output values?

Do we have an easy wrapper like model.SparseGPRegression?
as in model.SVIGPRegression?

Any updates? @alansaul

I've whipped something together, but it is a very thin wrapper. I've also integrated Zhenwens Adadelta optimizer, so that now if you have set a batchsize, you can call m.optimize('adadelta') and it will optimize with adadelta with batches.

This is great thanks! The tests never check the batch update, though. Is there a mechanic (like method) which selects the new batch and and updates the model accordingly? You know the sparse gp minibatch code, in there the new batches get only selected on a call to do_stochastics on the stochastics object. I updated it to be used in GPy now. This is off for gradient checking and so on, so it works on one batch only until we want to optimize. Optimization goes as follows:

optimize gets overriden in SparseGPMinibatch and sets a flag _update_stochastics = True
in parameters_changed, we check for stochastics (as the same code handles missing data as well) and then we check if _update_stochastics is True. If so, we call do_stochastics on the stochastics object, which selects a new batch, which we then work on.
in optimize it is a try: finally: block. This allways sets _update_stochastics to False, no matter what happens during optimization.

You can use the great package coverage to test the branching (like if statements) of you implementation.
When installed you can run (from the root dir)
coverage run travis_tests.py
coverage html
which will create a folder htmlcov with the local coverage in html files. In there branching code is highlighted in yellow, if it is not fully covered. The goal of this is to write tests once, and it fully covers the code so that it will not create surprises later on. If code gets run in tests it is a good start, especially if it is difficult to tests correctness, at least the code runs during test time.
I have added the RProp optimizer to paramz (https://github.com/sods/paramz/pull/6), this can be added to stochastic optimizers, if I'm not mistaken?

#323

Just waiting for tests to pass and then merging in


Contributors

@all-contributors please add mrpatpat for code

@all-contributors please add ClaasBusemann for code

@all-contributors please add PaulaBre for code

@all-contributors please add tobim-dev for code

@all-contributors please add ravensinth for code

@all-contributors please add 2mawi2 for code

done

I think you added the wrong tobim. Should have been @tobim-dev, not me.

Yeah I noticed that and fixed it manually in the PR. Sorry for that :)


Stringify serializer with callback-api

Implementation of csv-stringify with callback-api but with no streams.

Duplicate PR https://github.com/metarhia/common/pull/320


0.1.15: Can no longer use ExecFork to run an executable from PATH

Since we updated from 1.13 to 1.15 we cannot use ExecFork to execute a command like 'npm' which should be found from the PATH. This seems to be related with a change where the 'executable' property is marked as InputFile - so validation tries to find the file, which obviously does not exist.
FAILURE: Build failed with an exception.

* What went wrong:
Some problems were found with the configuration of task ':ui:runCoverageServerCentral' (type 'ExecFork').
> File '/deployment/ui/npm' specified for property 'commandLine' does not exist.
> File '/deployment/ui/npm' specified for property 'executable' does not exist.

Our task is built up rather dynamically:
ext.sharedCoverageServeLogic = { task, List<String> target ->
    configure(task) {
        workingDir "$webappDir"
        group = BasePlugin.BUILD_GROUP
        if (!org.gradle.internal.os.OperatingSystem.current().isWindows()) {
            executable "npm"
        } else {
            executable "npm.cmd"
        }

        args = ["run", "--", "ng", "serve", "--no-watch", "--no-live-reload"] + target
    }
}

task runCoverageServerStandalone(type: com.github.psxpaul.task.ExecFork) {
    sharedCoverageServeLogic(it, ["-c", "cov-standalone"])
    waitForOutput = "Compiled successfully"
    timeout = 240
    stopAfter = triggerStopCoverageStandalone
}

At first we thought this has something to do with the upgrade to Gradle 7.0 which happened at the same time, but even after reverting to Gradle 6.8 it does not work. Only after downgrading both (0.1.13 and 6.8 respectively) it returns to working.

Any idea how to work around this issue? This prevents the update to gradle 7.0 for us...

Ping...? :) I'd really appreciate some advice.


Upgrade go-crypto-openssl to v0.2.4

Release notes: https://github.com/microsoft/go-crypto-openssl/releases/tag/v0.2.4

Are the test failures legitimate issues?

They seem to me to be, unfortunately. Can browse them at https://dev.azure.com/dnceng-public/public/_build/results?buildId=141404&view=ms.vss-test-web.build-test-results-tab. @qmuntal mentioned that there are failures in the sync, so I'm sure he'll be working on it. 😄

Are the test failures legitimate issues?

Not on my watch.
I've submitted a PR to fix them: https://github.com/microsoft/go-crypto-openssl/pull/46

Superseded by #855


feat: log network response and request with filter options in debug console

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/45275
PROPOSAL:
Tests

[X] Verify that no errors appear in the JS console

Run the app on environment other than Dev
Enable Client side logging
Close the App and re launch it
Click on View debug console in Settings-> Trouble Shoot
Click on the three dot menu
Filter the logs to Network
You should only see logs with Network keyword
Download the network only logs
Change the extension to .json and open in any editor
Verify that the output is readable and contains request and response  data for API related logs

You can also follow the video:

Video
https://github.com/user-attachments/assets/a18ea13b-480d-40c5-a879-d02e68ae4dfb

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/0d220070-c945-4d3d-8821-1be9d387ba21

Android: mWeb Chrome

https://github.com/user-attachments/assets/05608688-9d5e-4aa8-a9d7-779694fecbe7

iOS: Native

https://github.com/user-attachments/assets/21f3491a-34be-4eee-b76a-03f621fd1fb0

iOS: mWeb Safari

https://github.com/user-attachments/assets/dcd128b7-acce-446d-bea5-be7600767735

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/49ca7fbc-caa5-49a0-8b76-c22ed33c5e40

MacOS: Desktop

https://github.com/user-attachments/assets/1ebda14d-8c52-49d8-9301-6661041f86e8

Tagging this held issue here so we can unblock it later: https://github.com/Expensify/App/issues/45275

Linting and TS checks are failing because of missing type for extraData argument in Log.info function and clientLoggingCallback. This is fixed in expensify-common repo here

@hurali97 Lint and Type checks failing.

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@hurali97 Lint and Type checks failing.

@allroundexperts Reasons here

@hurali97 Conflicts.

Jest Unit Test is failing on CI because of some timeout issue. All tests passes on the local. Can we re-trigger the test?

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/puneetlath in version: 9.0.17-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/marcaaron in version: 9.0.17-2 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/marcaaron in version: 9.0.17-2 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/marcaaron in version: 9.0.17-2 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Direct rewrite rules

At the moment rewrite rules have to specified using a pattern language. For example:
addCommutes = pat (BinOp Add "a" "b") := pat (BinOp Add "b" "a")

This rewrite rule could also be specified with a function:
addCommutes = \case
  Fix (BinOp Add a b) -> Just (Fix (BinOp Add b a))
  _                   -> Nothing

I'll refer to this as a "direct" rewrite rule.
I imagine there are advantages to the pattern matching approach, but I was wondering if it would also be possible to specify rewrite rules using the "direct" approach too? The sort of situation I would like to handle is for example when your language functor has a case like:
data F a = Foo Int a | ...

and you want to match Foo n x only when n satisfies some property, and use n to rewrite the term in some non-trivial way.
And thanks for the great package!

Interesting idea! I think it would be lovely to be able to write "direct" rewrite rules as you've presented them.
I can't tell right away whether it's possible, but we might be able to pull it off.
In essence, equality saturation works by

E-matching the LHS against the e-graph (discovering the e-class in which the LHS is represented, and the e-classes corresponding to the variable bits of the pattern that matched)
For every LHS that matched, represent the RHS pattern (thereby discovering the e-class where the RHS is represented)
Merge the e-class of the LHS with the e-class of the RHS, making them equivalent in the e-graph.

The e-matching's is done through the Data.Equality.Matching.ematch function, where all the magic e-matching happens (currently implemented according to the "Relational E-matching" paper)

We convert the e-graph to a database (within the eq-sat function)
Match the pattern against this database
Resulting in a list of matches, one for each e-class of the e-graph that the pattern matched

Each match also has a Subst, mapping pattern variable ids to the e-classes representing them for that particular match

So the pattern := pattern :| condition form lends itself well to this kind of workflow. However, I think the hegg implementation is a great playground to explore expressiveness of the interface to equality saturation and e-graphs.
(Small aside on conditional rewrite rule, which come from section 4.2 of the egg paper: I don't think they are sufficient to implement the kind of conditional matching you want, as they seem to be conditions based on analysis data of the variable parts of the pattern. In that sense, what you envision seems more appropriate and much more direct)
It is probably best to keep the e-matching main work function as is - receiving a pattern. Ideally, we should be able to go back and forth between those two implementations as they seem roughly equivalent...
We should probably start from the "direct" rewrite rule and try to reconstruct the pattern based one.
Then we can consider the conditions on top of the direct rewrite rule, which likely filter more matches at step 1.
I'll have more time to investigate this in October, but I'll be happy to participate in a discussion if you decide to pursue this line of thought.
Thanks!

I don't think they are sufficient to implement the kind of conditional matching you want, as they seem to be conditions based on analysis data of the variable parts of the pattern. In that sense, what you envision seems more appropriate and much more direct

Yes, this is what led me to the question: the direct approach allows much more control.

We should probably start from the "direct" rewrite rule and try to reconstruct the pattern based one.

I have only very vaguely looked at the code and papers you linked, but I'm not sure this is the right approach, I think the direct approach likely has significant disadvantages (it can't efficiently express non-linear patterns for example), so you wouldn't want to build on it, I was thinking of it more as an escape hatch.

Minor note: Currently languages are grouped by f () so Foo 1 () would be considered a distinct table from Foo 2 () and generic rules don't work. One workaround is to use Foo (Const 1) (), then you can use a variable in the pattern.
One approach I like is to separate matching and rewriting. Here with OverloadedRecordDot for some sugar:
forMatches ("eql" .@ pat (BinOp Add "a" "b") ) $ \r-> do
    r.eql .==  S (BinOp Add r.b r.a)

The  "eql" .@  is like at patterns in haskell so that you can name subpatterns.
Smart constructors or pattern synonyms could make the rewrites a bit nicer to look at.
With some pretty arbitrary rewrite monad with helper functions such as
data Subst f = S (f Subst) | C ClassId
(.==) :: (MonadEGraph lang ana cid) => cid -> Subst lang -> m ()
(.==) ident expr = addEquality ident =<< injectEgraph expr
injectEGraph :: (MonadEGraph lang ana cid) => Subst lang -> m cid
This is pretty powerful:

Conditionals are normal if/when
Pretty easy to extend to multi-root patterns
You can add multiple rewrites for a single rule
You can write recursive patterns, e.g x@(a+x) -> a == 0
You can mix in analysis/analysis propagation. For the current implementation not a problem, but when seminaive evaluation is ever added mixing analysis would require rerunning the rules when analysis changes, e.g. by tracking what is read in the MonadEGraph though.
You can implement the normal E-Graph interface in terms of this


Tests should be executed on "make check", not a custom "make test"

Fixed in a4adf96e28d0f50f293636804b4aa3d68c78db3a


rbd: add functions/constants related to rbd pool stat options

Added constants for rbd pool stat options along with wrappers for following functions:

rbd_pool_init()
rbd_pool_stats_create()
rbd_pool_stats_destroy()
rbd_pool_stats_option_add_uint64)
rbd_pool_stats_get()

Fixes: https://github.com/ceph/go-ceph/issues/278
Signed-off-by: Mudit Agarwal<EMAIL_ADDRESS>Checklist

[ ] Added tests for features and functional changes
[ ] Public functions and types are documented
[ ] Standard formatting is applied to Go code

I didn't dig into the tests much once I realized there was a bit of a mismatch between the Golang-way and the C api ceph uses here. Generally, we try to provide a thin wrapper around the C apis of Ceph, but try to make them feel Go idomatic as much as (reasonably) possible. Sometimes an api is awkward enough that I think we need to make a judgement call and deviate a little from what is happening at the C layer. As long as we don't place extra limits on the power of the API I think this is generally a good thing.
With generalities out of the way, the specific issue I see here is that the C api expects you to set up the "fetcher" with bunch of rbd_pool_stats_option_add_uint64 calls that take pointers to ints, and then call rbd_pool_stats_get which will fetch the stats and fill in pointer values. This is slightly similar to what I was also working on in PR #315 but a bit simpler and more homogeneous. Because the return types are the same I propose a simplified call, partially inspired by the how this function is wrapped by ceph's python code (in https://github.com/ceph/ceph/blob/octopus/src/pybind/rbd/rbd.pyx#L2556 ).
// GetPoolStats returns a map of requested PoolStatOptions to their values.
func GetPoolStats(ioctx *rados.IOContext, o []PoolStatOption) (map[PoolStatOption]uint64, error) {
    // .... implementation ....
}

Unlike the python code, the user specifies what options they want, which is more similar to the C api. If we wanted to, perhaps in the future, we could even do a convenience version of the function that gets all known PoolStatOptions, just like the python code.
Now, I'm not dead set on this suggested API but I do think we need to pause and consider the API design for this subset of librados. We do want to keep our layer thin, but not too thin. Let's discuss. :-)

So, AFAICT we have two options:

We don't expose AddOption() API and set all the options during GetPoolStats() only as done in the python code.

We expose AddOption() but ask user to only set the option not the pointer to the option value but internally define a map (of options and valeues) and based on that map passess the address of the respective pointer to the option value inside AddOption().

Something like:
createPooStatMap() {  // this can be created and destroyed during CreatePoolStats() and Destroy() if required.
  m[PoolStatOptionImages] = cImages
  m[PoolStatOptionImageProvisionedBytes] = cImageProvisionBytes
  m[PoolStatOptionImageMaxProvisionedBytes = cImageProvisionMaxBytes
}

AddOption(PoolStatOption option) {
      optionVal = m[option]
      C.rbd_pool_stats_option_add_uint64(poolStata.stats, C.in(option), &optionVal)
}

GetOption() {
// define another struct for GetOption, not the same map as in AddOption()
return &poolStats {
                cImages: cImages
                cImageProvisionBytes: cImageProvisionBytes
		cImageProvisionMaxBytes: cImageProvisionMaxBytes
	}

PS: Just a thought, haven't looked from implementation point of view.
In both the cases, GetPoolStats() will return the map of all the values as done in the python code.

I certainly lean toward option 1, as my suggested api might indicate. I feel that this api is the way it is due to some of C's quirks and since Go is much more oriented toward more & richer data types, and I generally assume people using the API want something simple to just get the stats they need, there isn't a great impetus to make the API in go-ceph as granular as the one exposed in C.

I have un exported AddOption() and added two APIs:

func (poolStats *PoolStats) GetPoolStats(ioctx *rados.IOContext, option PoolStatOption) (uint64, error)

Returns value for a single Pool Stat Option

func GetAllPoolStats(ioctx *rados.IOContext) (map[PoolStatOption]uint64, error)

Returns a map of all Pool Stat Options to their respective values.

Have pushed the changes to  https://github.com/agarwal-mudit/go-ceph/tree/pool but somehow that is not updating this PR automatically :(

@nixpanic @phlogistonjohn PTAL


Update telegram to 3.9.0

Please update my adapter ioBroker.telegram to version 3.9.0.
This pull request was created by https://www.iobroker.dev c0726ff.

RE-CHECK!

RE-CHECK!

RE-CHECK!


Add media query

This part of the portfolio  was done using these:

Media query
HTML/CSS linters
HTML and CSS files

Please I have one more review left. I hope this one goes well please 🙏
On Sat, 10 Jun 2023 at 16:20, Francisco Borja Lobato <
@.***> wrote:

@.**** requested changes on this pull request.
Hi @Wils-FOURR-speed https://github.com/Wils-FOURR-speed,
Required Changes ♻
[image: istockphoto-1079018918-612x612]
https://user-images.githubusercontent.com/58642949/244878855-3bcb3e1a-c84b-4ca8-910b-bd03cfc7332c.jpg
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
Highlights

Correct Github flow ✔
Descriptive PR ✔
Best practices of HTML & CSS ✔

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In index.html
https://github.com/Wils-FOURR-speed/My-portfolio/pull/5#discussion_r1225406515
:

As requested by the previous CR, please 🙏🏽 correct the margins to
be similar to the Figma design. We could use justify-content: center; and
align-items: center; in the class .grid-container, and in the class
.pic-holder-container specify the left and right margins, e.g. margin:
Xpx Ypx;

In index.html
https://github.com/Wils-FOURR-speed/My-portfolio/pull/5#discussion_r1225408480
:

Multi-Post Stories Gain + Glory

As requested by the last CR, please correct the distance between the
features buttons to be similar to the Figma design, we could do it by using
justify-content: center; in the .stories class.
[OPTIONAL] Kindly use the correct background color for the buttons
#EBF0EE to be similar to the Figma design.

In index.html
https://github.com/Wils-FOURR-speed/My-portfolio/pull/5#discussion_r1225411415
:

@@ -159,10 +172,13 @@ Multi-Post Stories Gain + Glory

About me

About me

As requested by the last CR please 🙏🏽 correct the width of the
paragraph to be similar to the Figma design, we could use width: 610px;
APPROX.

In index.html
https://github.com/Wils-FOURR-speed/My-portfolio/pull/5#discussion_r1225414643
:

@@ -224,21 +243,32 @@ Skills

After adjusting the paragraph you will see that the "Get my resume"
overlaps the Frameworks, kindly fix it by adding margin: Xpx Ypx; in
the .other-section class to be similar to the Figma design.
Ass required by the last CR, please correct the margins of the cards
by adding the margin attribute to the .other-section class to be
similar to the Figma design 👇🏽

Figma design Your design
[image: image]
https://user-images.githubusercontent.com/58642949/244878455-5d8ce11c-817e-4c5f-8d8d-0ad4036f3124.png [image:
image]
https://user-images.githubusercontent.com/58642949/244878434-5e8e85fe-37ba-4223-b189-1ba1f83d5b45.png
In index.html
https://github.com/Wils-FOURR-speed/My-portfolio/pull/5#discussion_r1225420396
:

           <form method="POST" action="https://formspree.io/f/xzbqlowk" class="footer-form">

                   <input type="text" name="first name" maxlength="30" placeholder="First name" class="first-name">

                   <input type="text" name="last name" maxlength="30" placeholder="Last name" class="last-name">

               <input type="text" name="full name" maxlength="30" placeholder="Full name" class="name" required>

               <input type="email" name="email" placeholder="Email address" class="email" required>

               <textarea name="comments" cols="30" rows="5" maxlength="500" placeholder="Hey I’d like to connect to..."></textarea>

               <input type="submit" value="Get in touch" class="get-button1">

<EMAIL_ADDRESS>

           </form>

As requested by the previous CR, please 🙏🏽 correct the items in
the contact form section:

The text on the left hand side is meant to be centered across the form
element.
The form input and textareas are supposed to be wider than that.
The background images aren't properly fixed, please use the background css
property position them properly using the css background-position property,
and other background properties, you can make your research.
The button Get in touch shouldn't be in the center.
Add spacing between First name and Last name input tags
1 Figma design Your design
[image: image]
https://user-images.githubusercontent.com/58642949/244878692-4403b5c0-774f-4401-83f7-526254bf7f6a.png [image:
image]
https://user-images.githubusercontent.com/58642949/244878774-92dcc0b8-d29b-4ed6-b999-f72fe3a0a3fc.png
—
Reply to this email directly, view it on GitHub
https://github.com/Wils-FOURR-speed/My-portfolio/pull/5#pullrequestreview-1473435616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5TIO5SISYV4WNEKT35MESTXKSGFFANCNFSM6AAAAAAZBSAMVM
.
You are receiving this because you were mentioned.Message ID:
@.***>


tests: newlib: thread_safety: Add min. RAM constraint for stress tests

This commit adds a separate minimum RAM constraint for the thread-
safety stress tests because they have a significantly higher RAM
usage compared to the basic functional tests.
Signed-off-by: Stephanos Ioannidis<EMAIL_ADDRESS>Fixes #45593

The rpi_pico build failure is not related to this PR:
https://github.com/zephyrproject-rtos/zephyr/runs/6608681983?check_suite_focus=true#step:12:776
This issue is tracked in #46023 and #45932

@stephanosio I've merged the reset API renaming; do you need to rebase now?

I keep forgetting I can rebase with a couple of clicks in the UI these days. I took care of it.


GH-38071: [C++][CI] Fix Overlap column chunk ranges for pre-buffer

Rationale for this change

The C++ Parquet Arrow fuzz will generate bad Parquet file with bad row-range, this patch check the range before sending to Pre-Buffer.
What changes are included in this PR?
Just a checking

Are these changes tested?

No.
Are there any user-facing changes?
No.

Closes: #38071

@jorisvandenbossche @lidavidm I've add an Result<> as return value here

[ RUN      ] AzureFileSystem.UploadThenDownload
unknown file: Failure
C++ exception with description "Fail to get a new connection for: http://<IP_ADDRESS>:10000./ Couldn't connect to server" thrown in the test body.
[  FAILED  ] AzureFileSystem.UploadThenDownload (6053 ms)

Hmm...Seems this is not caused by me 🤔

I've rebase the master to rerun the test :-) Hope this time it passes...

Passed. :-)
I'll merge this.

Thanks a lot @mapleFU for fixing this!


Option value casing mismatch

[Enter feedback here]

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

ID: 1d1d9884-19f7-592b-f47c-8cf100639ff0
Version Independent ID: 555c3826-60d9-2589-54af-850efbfbb13a
Content: .NET Core CLI task - Azure Pipelines
Content Source: docs/pipelines/tasks/build/dotnet-core-cli.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @pulkitaggarwl
Microsoft Alias: puagarw

@aaronhallberg can you please have someone look at the issue.

Thanks for letting us know. This issue is fixed in our new task reference, which is generated using the task source code and includes all task inputs, including any aliases here:
https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/dotnet-core-cli-v2?view=azure-pipelines
Please check it out and let us know what you think. Thank you.


Cannot import and use transformer_engine after successful installation with No module named 'transformer_engine_extensions'

I have installed transformer_engine for use with Accelerate and Ray. I have the following requirements which work totally fine for all sorts of distributed training
torch==2.2.1
transformers==4.39.3
accelerate==0.29.2
deepspeed==0.14.0
datasets==2.18.0
sentencepiece==0.2.0
transformer_engine @ git+https://github.com/NVIDIA/TransformerEngine.git@stable

And for your information my Dockerfile looks like
FROM anyscale/ray:2.11.0-py39-cu121

COPY ./training-requirements.txt training-requirements.txt

RUN pip install -r training-requirements.txt --global-option=--debug

RUN python -c "import transformer_engine.pytorch as te"

But have also tried different versions of torch including latest.
I can import transformer_engine just fine when this image is deployed
>>> import transformer_engine
>>>

However, when I try to import the pytorch module I get
>>> import transformer_engine.pytorch as te
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ray/anaconda3/lib/python3.9/site-packages/transformer_engine/pytorch/__init__.py", line 6, in <module>
    from .module import LayerNormLinear
  File "/home/ray/anaconda3/lib/python3.9/site-packages/transformer_engine/pytorch/module/__init__.py", line 6, in <module>
    from .layernorm_linear import LayerNormLinear
  File "/home/ray/anaconda3/lib/python3.9/site-packages/transformer_engine/pytorch/module/layernorm_linear.py", line 13, in <module>
    from .. import cpp_extensions as tex
  File "/home/ray/anaconda3/lib/python3.9/site-packages/transformer_engine/pytorch/cpp_extensions/__init__.py", line 6, in <module>
    from transformer_engine_extensions import *
ModuleNotFoundError: No module named 'transformer_engine_extensions'

And when I try to import transformer_engine_extensions directly I get the same thing
>>> import transformer_engine_extensions
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'transformer_engine_extensions'

Wondering what is going on with the versions. Everything else works fine with torch it's just that transformer_engine seems to be installed incorrectly.
Here are some Nvidia specs as well
$ nvcc -V
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2023 NVIDIA Corporation
Built on Mon_Apr__3_17:16:06_PDT_2023
Cuda compilation tools, release 12.1, V12.1.105
Build cuda_12.1.r12.1/compiler.32688072_0

$ nvidia-smi
Sat May 18 16:47:56 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.104.12             Driver Version: 535.104.12   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|

Do you know if PyTorch is installed before you install the requirements.txt? TE supports DL frameworks other than PyTorch (e.g. JAX), so part of our build process involves checking what frameworks are installed. I suspect that TE doesn't find PyTorch, so it skips building the PyTorch extensions. To force it to build the PyTorch extensions, can you try setting NVTE_FRAMEWORK=pytorch in the environment?

@timmoon10
Thanks. If anyone wants to try importing transformer_engine_extensions after successfully building the PyTorch extensions. This is how you do it.

Python

import torch

import transformer_engine

import transformer_engine_extensions

I'm using cuda 12.5 and I'm using the nightly cuda 12.4 pytorch build on artix linux.

Not working for the current main branch of TE.

Solved. The transformer_engine_extensions has been renamed to transformer_engine_torch.


Trouble launching a WSL / Ubuntu driven task using the Windows Task Scheduler

While creating an automated backup solution for myself I ran across a couple of interesting issues when calling out to WSL tasks using the Windows Task Scheduler and I haven't been able to solve them all.
The main problem is that WSL tasks will not run unless you specifically set "Run only when user is logged in", however this constraint is unreasonable because let's say you want a task to run at 3am, but you have a lock screen come on after 30 minutes of inactivity. Now you're considered logged out, so it won't run.
To combat that, MS has a task option "Run whether user is logged on or not", and then it saves your password along with the task. However, when this is enabled, WSL driven tasks fail with exit code 2 which is "the system cannot find the file".
My task's action is: C:\Users\Nick\AppData\Local\Microsoft\WindowsApps\ubuntu.exe
My task's arguments are: run ~/src/scripts/backup
The path to ubuntu.exe exists and the task works correctly when I set the task to run when logged in but the same task fails when it is set to run without the user being logged in. It even fails if I'm logged in and manually run the task with the "Run whether user is logged on or not" option enabled but works as soon as I switch to "Run only when user is logged in" and re-run the task.
Using bash.exe and setting -c ... doesn't work because then environment variables such as $HOME get resolved to the Windows version instead of WSL inside of the script.
Any ideas on how to get around this? Thanks.
I'm using Windows 10 Pro (1709), build 16299.125.

@nickjj - $HOME will always be your user's home directory.  However the current working directory will be a Windows (DrvFs) path.  I suspect your script is using relative paths.
You can use wsl.exe ~ or bash.exe ~ to launch a command in your user's home directory.

My script uses $HOME as the start of the path for some of my WSL paths (such as dotfiles and folders) and everything else is absolute paths.

I think I'm confused in what you're reporting.  Are you saying ubuntu.exe doesn't work but bash.exe or wsl.exe works but the $HOME variable is wrong?

The main issue is that the Windows task scheduler seems to have issues trying to run a WSL driven task when the user is not logged in.
Let's ignore the ubuntu.exe vs bash.exe differences for now.


Support Model Bound Attributes to add select/expand limits, etc

Currently using OData/WebApi, I use the ODataConventionModelBuilder to build the EdmModel, but this also gives me access to the entity types before building the EdmModel, which allows me to add restrictions (we keep our model constrained as much as possible, especially with regards to expanding through properties.
For example:
var builder = new ODataConventionModelBuilder(serviceProvider);

builder
    .EntitySet<Department>("Departments")
    .EntityType
    .Expand(1, x => x.Organization)
    .Expand(3, x => x.CategoryMembership)
    .IgnoreExcept(c => new
    {
        c.Id,
        c.Code,
        c.Name,
        c.OrganizationId,
        c.Organization,
        c.Roles,
        c.CategoryMembership
    });

builder
  .EntitySet<Employee>("Employees")
  .EntityType
  .Expand(2, x => x.Department)
  .Expand(1, x => x.Position)
  .IgnoreExcept(c => new
  {
      c.Id,
      c.FirstName,
      c.MiddleName,
      c.LastName,
      c.DepartmentId,
      c.Department,
      c.PositionId,
      c.Position
  });

// ...

return builder.GetEdmModel();

After some reading, I think this could be best accomplished in ODataToEntity by supporting the Model Bound Attributes.  For example (pulled from referenced page):
[Expand("Orders", "Friend", "CountableOrders", MaxDepth = 10)]
[Expand("AutoExpandOrder", ExpandType = SelectExpandType.Automatic, MaxDepth = 8)]
[Page(MaxTop = 5, PageSize = 1)]
public class Customer
{
    public int Id { get; set; }
    public string Name { get; set; }
    [Expand(ExpandType = SelectExpandType.Disabled)]
    public Order Order { get; set; }
    public Order AutoExpandOrder { get; set; }
    public Address Address { get; set; }
    [Expand("Customers", MaxDepth = 2)]
    [Count(Disabled = true)]
    [Page(MaxTop = 2, PageSize = 1)]
    public List<Order> Orders { get; set; }
    public List<Order> CountableOrders { get; set; }
    public List<Order> NoExpandOrders { get; set; }
    public List<Address> Addresses { get; set; }
    [Expand(MaxDepth = 2)]
    public Customer Friend { get; set; }
}

which would match this Fluent API approach (just here for reference)
var builder = new ODataConventionModelBuilder();
builder.EntitySet<Customer>("Customers")
    .EntityType.Expand(10, "Orders", "Friend", "CountableOrders")
    .Expand(8, SelectExpandType.Automatic, "AutoExpandOrder")
    .Page(5, 2);
builder.EntityType<Customer>()
    .HasMany(p => p.Orders)
    .Expand(2, "Customers")
    .Page(2, 1)
    .Count(QueryOptionSetting.Disabled);
builder.EntityType<Customer>()
    .HasMany(p => p.CountableOrders)
    .Count();
builder.EntityType<Customer>()
    .HasOptional(p => p.Order)
    .Expand(SelectExpandType.Disabled);

add wiki


How to install

Hello, i can't quite understand how to compile this project
i tried make command with the makefile.am as input but it tried to use some file in the lol folder which is empty

It uses git submodules. You will need to retrieve the submodules using your Git client. If you are using the command line:
git submodule update --init --recursive

thanks you for your answer, managed to download the whole lol directory using the command but using
make -f Makefile.am
in the lolremez directory still gives me
Makefile.am:2: /lol/build/autotools/common.am: No such file or directory make: *** No rule to make target '/lol/build/autotools/common.am'.  Stop.
i'm sorry i'm a complete newcommer to command line compilation

You need to first run ./bootstrap and ./configure before make.
I have now added build instructions to the README.md!


fix: post with crashTypeId

Allows us to post other crash types with Pulse such as Crashpad, Electron, and macOS.

@daveplunkett please review when you're done skiing!

:tada: This PR is included in version 8.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


router: 1.52.0 -> 1.52.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/apollographql/router/releases.
meta.description for router is: Configurable, high-performance routing runtime for Apollo Federation
meta.homepage for router is: https://www.apollographql.com/docs/router/
Updates performed

Rust version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.52.1 with grep in /nix/store/8km03acnr02ik8vnrmin6h06wnn1gdv5-router-1.52.1
found 1.52.1 in filename of file in /nix/store/8km03acnr02ik8vnrmin6h06wnn1gdv5-router-1.52.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

router

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8km03acnr02ik8vnrmin6h06wnn1gdv5-router-1.52.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A router https://github.com/r-ryantm/nixpkgs/archive/69017878bf3739e04116d7c05979dc2523557616.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/69017878bf3739e04116d7c05979dc2523557616#router

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8km03acnr02ik8vnrmin6h06wnn1gdv5-router-1.52.1
ls -la /nix/store/8km03acnr02ik8vnrmin6h06wnn1gdv5-router-1.52.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

router

Maintainer pings
cc @bbigras for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

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

1 package built:

router


CB-8505-AssociatePeriscopeRepositoryWithTransaction

All existing Periscope Repository interfaces should be annotated with Transactional attribute similar to CB Repository interfaces, since without that Query Results will not be pre-flushed and stale data will be returned in certain scenarios.
See detailed description in the commit message.

Does this end up making every DB operation a transaction?
Is there an alternate way to prevent caching?

By default all DB Operations including read are transactional at spring jpa level. But when we extend CrudRepository to write custom query api, the Transaction annotation is not inherited and has to be explicitly defined since annotation are not inherited for interfaces. If we use Spring "@Transactional" there is some optimization where in "readOnly=true" can be specified for read only queries, but this differs from CB Repository Classes which uses Javax "@Transactional"  instead of Spring "@Transacational" and I didnt want to change this mechanism only for periscope repositories.

Swagger compatibility checks failure ignored because of CDPCP-2673 based on comments in eng_cb_dev_internal slack channel.


Correct outbound ports documentation for Virtual Machines

The existing documentation states that the source ports must be set to 443. This should be  the destination ports.

@Porges : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Porges : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Confirmed with @alfpark that this is the correct configuration.

#sign-off


Feedback form on success behavior

Makes form replacement the default success behavior for feedback forms. Currently some feedback forms are replaced by the success notification (on owning-a-home/help-us-improve & Spanish Ask answer pages), but the default behavior (on '/owning-a-home', Ask English answer pages) is to leave the form on the page and add the notification to it. This makes the behavior consistent across the feedback form modules.
Additions

Add method to Notification.js to get the notification element.

Changes

In feedback.html

Add the block background to the feedback form itself instead of the containing div
Move the notification into the feedback form so warning messages appear inside the background

In feedback-form.js

Always replace feedback form with success notification

In FormSubmit.js

If replaceForm option selected, swap form out for its notification module (ensures that notification doesn't appear inside block__bg)

Testing

Check out branch & run gulp
Navigate to Ask answer page and verify that feedback form is replaced on success

Screenshots
Feedback form error:

Feedback form success:

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

This looks good!  I think this would be well suited for a browser test.  I am happy to pair on it with you if you'd like!

@richaagarwal Sorry, I missed your comment earlier! I would love to pair on adding a browser test. Can we merge this and work on the test next week, or should they go in together?

I'd prefer we merge it once the test is in. Technically we shouldn't be merging new code in without test coverage, but I think coveralls only counts python code.

On Jul 18, 2017, at 11:54 AM, sebworks<EMAIL_ADDRESS>wrote:
@sebworks commented on this pull request.
In cfgov/unprocessed/js/molecules/Notification.js:

@@ -133,6 +133,10 @@ function Notification( element ) { // eslint-disable-line max-statements, inline
return this;
}

function getElement() {
I'm saying why not just expose it with a property like:

this.element = _dom
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Is there a plan to create a test and merge this, or is it ready once it's updated?


Overhead Intake

Code overhead Intake motor
Overhead Intake Wiki

@Chimkenwaffle if this feature was cut lets close this ticket

not being used.


Welcome back! Do you have updates  for the espruino repository?

Welcome back @jeffmer!!
Do you have updates (2v10) for the repository, it is a little older https://github.com/jeffmer/Espruino

Thanks for reminding,  I will check it.


default-bilder ved error

beskrivelse for relasjonslister

A picture tells a thousand words
Before this PR
After this PR

Coverage decreased (-0.08%) to 41.601% when pulling 56a1771efa62b71f3d251fcffb6db0e970e18931 on feature/default-bilder into dd124204211f9f0cdef561ac2a0ebd61553dc95a on master.


Trim node titles in graph

Trim node titles in graph via JS
fix for #413
Sadly, the title attribute to show something on hover does not work in
SVG. So we might need another solution for that.

Not too worried about no hover on the graph, as they can get the full name from the details panel.

Can confirm this works; LGTM


Audio Classes do not allow data for a single time/frequency

General
but they should

I was trying to reproduce the issue with:
signal = pf.Signal([1], 44100) signal2 = pf.TimeData([1],[1]) singal3 = pf.FrequencyData([1],[1])
but that works as it's supposed to. What is the exact issue here?

Maybe it refers to the fact that an initialization of a Signal in frequency domain with only one bin leads to an error?
pf.Signal([1], 44100, domain='freq', n_samples=1)
without n_samples=1 causes
ZeroDivisionError: float division by zero


Migrate CAPI Turtles extension to be Vue3 compliant

Migrate CAPI Turtles extension to be Vue3 compliant
Also, re-enable the check on test-plugins-build when there's a valid vue3 version in main that generates successful builds

https://github.com/rancher/capi-ui-extension/releases/tag/capi-0.8.0


Added a checker for max selections

Added a checker to the Multiselect for max selections
Changes

Adds a max-selections class when user reaches five selections
Removes the class once the user removes the fifth selection

Testing

gulp build and navigate to /blog and make five selections in the Multiselect

Review

@schaferjh
@KimberlyMunoz
@sebworks
@anselmbradford

Screenshots

Todos

Add tests (slowly working on this in other PRs)

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

:+1:

This works great :+1:
In the future we might style the max reached text differently to have it stand out against the other text better.

In the future we might style the max reached text differently to have it stand out against the other text better.

I thought about that too, also for when there's no result, because unless you're paying attention, it's not immediately obvious. It'll prob be a v2 task, since we're running out of time, but I'll let @schaferjh decide.

:+1:

:+1:


Icon request: Zipmex

URL of the site or app (required): https://zipmex.com/
Logo resources for the icon (optional):

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/Zipmex.svg
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/.Variations/Zipmex bg.var.svg

❓ How to add icon(s) to Aegis?
⭐ Support this project by giving a star! Just adding like to this comment is also appreciated.


Respect DEFAULT_VALIDATION_STATUS set on Parser

see https://github.com/sloria/webargs/issues/180#issuecomment-394869645

LGTM.
Just a typo in the CHANGELOG: Suppor -> Support.


Dynamic field not checking for parts List length

mongoengine\base.py
line 983:
if self._dynamic and parts[0] in self._dynamic_fields:
changed to:
        if self._dynamic and len(parts) and parts[0] in self._dynamic_fields:

Looks like the assumption is made that parts will always have an element.
I have found that is not always the case.
Checking for length first has seemed to patch this.

@Amorano thanks for the report - this is now fixed in dev


Created docs for the injector

Table of Contents
Link to GitHub issues it solves
closes #8
Description
Created docs for the injector
Contributing has more information and tips for a great pull request.

@zeobot set lc to pass


Some imprecission in clipping

The error here is about .002 degree and there is some re-projection involved. I will explore further. (Using QGIS to explore).

it appears to be shifted (rather than cut short). The offset exists on the other side as well.

I need a little more context here, not sure what I am looking at. If the raster is being reprojected as well, that could account for the shifts.

I agree, I am not sure what's going on here. I will look at your tests and see whether the problem occurs as well or whether it happens somewhere on the way. Currently, I passing a EPSG 4326 geometry in the clip method on a raster in MODIS sinusoidal (which seems to be dangerous anyways). Reprojecting happens implicitely.

Falk Schuetzenmeister | The Nature Conservancy | m: 1-510-604-8010 | skype: falk.schuetzenmeister

From: Brian Galey<EMAIL_ADDRESS>Sent: Tuesday, February 09, 2016 11:38 AM
To: bkg/greenwich
Cc: Falk Schuetzenmeister
Subject: Re: [greenwich] Some imprecission in clipping (#9)
I need a little more context here, not sure what I am looking at. If the raster is being reprojected as well, that could account for the shifts.
—
Reply to this email directly or view it on GitHubhttps://github.com/bkg/greenwich/issues/9#issuecomment-182024802.


Build(deps-dev): Bump @types/node from 17.0.31 to 17.0.32 (#3465)

Pull request opened by the merge tool on behalf of #3465

Merged in 9064e521d055140a78db4a8c528173b9155aac79


Add keplerian element in IOD

Is it possible to add keplerian element result for Initial Orbit Determination (IOD) module?
Now, there is only the velocity solution of Lambert problem.
I think since it is orbit determination, keplerian element seems better.

Hi @AngelMamba , thanks for your interest in poliastro!
Perhaps you can try https://docs.poliastro.space/en/latest/autoapi/poliastro/maneuver/index.html#poliastro.maneuver.Maneuver.lambert, which returns a Maneuver object and therefore can be applied to a existing Orbit using https://docs.poliastro.space/en/latest/autoapi/poliastro/twobody/orbit/index.html#poliastro.twobody.orbit.Orbit.apply_maneuver.
Please let us know if this fulfills your needs.

Hi @AngelMamba , thanks for your interest in poliastro!
Perhaps you can try https://docs.poliastro.space/en/latest/autoapi/poliastro/maneuver/index.html#poliastro.maneuver.Maneuver.lambert, which returns a Maneuver object and therefore can be applied to a existing Orbit using https://docs.poliastro.space/en/latest/autoapi/poliastro/twobody/orbit/index.html#poliastro.twobody.orbit.Orbit.apply_maneuver.
Please let us know if this fulfills your needs.

Hi @AngelMamba , thanks for your interest in poliastro!
Perhaps you can try https://docs.poliastro.space/en/latest/autoapi/poliastro/maneuver/index.html#poliastro.maneuver.Maneuver.lambert, which returns a Maneuver object and therefore can be applied to a existing Orbit using https://docs.poliastro.space/en/latest/autoapi/poliastro/twobody/orbit/index.html#poliastro.twobody.orbit.Orbit.apply_maneuver.
Please let us know if this fulfills your needs.

Hi @astrojuanlu Thank you for the reply. It helps a lot.
But I just need to calculate keplerian elements from postion and velocity vector, not a orbit maneuver case. So I think Orbit object from twobody module works.
Anyway, I kind of figure this out. thank you.

Hi @AngelMamba , thanks for your interest in poliastro!
Perhaps you can try https://docs.poliastro.space/en/latest/autoapi/poliastro/maneuver/index.html#poliastro.maneuver.Maneuver.lambert, which returns a Maneuver object and therefore can be applied to a existing Orbit using https://docs.poliastro.space/en/latest/autoapi/poliastro/twobody/orbit/index.html#poliastro.twobody.orbit.Orbit.apply_maneuver.
Please let us know if this fulfills your needs.

Hi @astrojuanlu Thank you for the reply. It helps a lot.
But I just need to calculate keplerian elements from postion and velocity vector, not a orbit maneuver case. So I think Orbit object from twobody module works.
Anyway, I kind of figure this out. thank you.

Excellent! Don't hesitate to ask if you have any further questions by opening a new issue or joining http://chat.poliastro.space

Excellent! Don't hesitate to ask if you have any further questions by opening a new issue or joining http://chat.poliastro.space


Find a way for "os family's" default cloud-init

cloud-init may differ from one OS to the other
The driver should be able to recognize  accurate cloud-init to inject.
Possible solutions :

use some kind of os_family:  family
auto-detect (HOW ?)

May not need if #12

Closing untill real use case is known


Clean up the files, directories, branches, Trello cards, repos and other resources.

Clean up the files, directories, branches, Trello cards, repos and other resources.
Purpose
The purpose for this GitHub Issue is to

organize all my resouces in only one or two (or three) locations so they're easy to find
and to diminish the number of unnecessary GitHub repos, Trello cards, Google Keep cards, Google Drive files and directories, etc.

so I have less to search through to find what I need when I need it.
Tasks

In this repo, clean up the

files,
directories, and
branches.

Specifically,

update the ./README.md file,
address all the relevant TODOs in each file in this repo,
and systematically go through each branch to address the Procedure below.

Then comb through the following resources to extract relevant code, links, notes, etc.

Trello cards
my other repos
related directories and files in the LearningDirectory Repo
other miscellaneous resources such as Google Keep.

To do this, systematically go through each of the above to address the Procedure below.

Then delete any of that which is no longer of any use.

Procedure
From the actionable items create GitHub Issues.
From the notes and links

if it is relevant to this repo, find a GitHub feature to document the info.
if it is something I may reference at a later date for another project, make note of it in

https://github.com/JamieBort/Digital_Knowledge_Base/blob/main/pages/Personal Dashboard.md,
or https://github.com/JamieBort/Digital_Knowledge_Base/,
or https://github.com/JamieBort/LearningDirectory/.

When that branch, directory, file, Google drive doc, Google Keep card, etc. has been fully explored and all of its resources extracted/copied, delete it.
When that Trello card has been fully explored and all of its resources extracted/copied, repurpose it.
Todo

[x] #17
[x] #23
[x] #19
[x] #21
[x] #22
[x] #24
[x] Before closing out this #14 Issue, review the No Status column of this https://github.com/users/JamieBort/projects/2/views/1
Pull Issues relevant to this milestone into the ToDo column. Then address them accordingly.

Completed this Issue.


Add check for problematic SPARQL Update statement

FCREPO-1409
https://jira.duraspace.org/browse/FCREPO-1409

Resolved with: https://github.com/fcrepo4/fcrepo4/commit/74b625372e394b08a5d4af64f98e8b4621419442


Adding support for multiple axis ( up to 2 ), axis on right, and log scale

This is added because of pressing needs at NSLS2... while we find a better long term supported plotting solution

@kasemir ( sorry I could not add you as a reviewer )

Bummer that you had to go into jquery.flot.logaxis.js since that would be clobbered if we update to a newer flot version.

Are you  running this on tomcat 9 or older? For tomcat 10, we'd have to update some package imports because the API moved, #12

Ya.. I was annoyed at the label formatter being hard coded.
but honestly the changes to the logaxis.js can be lived without
As I mentioned... after working on these changes I am convinced we need a more sustainable solution which should start with a new library

Are you running this on tomcat 9 or older? For tomcat 10, we'd have to update some package imports because the API moved, #12

currently I am running on tomcat 9.. but I will update to 10 soon... thanks for the reference

I'm on older tomcat releases, and the changes are incompatible. Might mean we need to keep a "tomcat10" branch for a little while....


Silent install for appliance.yml

Is your feature request related to a problem? Please describe.
Silent install for appliance.yml - No longer prompting for Cobbler, AWX and MariaDB password
Describe the solution you'd like
No longer prompting for Cobbler, AWX and MariaDB password
Describe alternatives you've considered
Prompting the password for Cobbler, AWX and MariaDB
Additional context
Add any other context or screenshots about the feature request here.

We will accommodate this change in separate request.

We will accommodate this change in separate request.


Reactivate user account if soft deleted

CUR-537 - User should be able to re-register the account with updated data in case of soft deleteed

Duplicated.


Coverage flapping.

There's some time-sensitive test somewhere (I suspect in the pipeline/queue package) that's taking a different code-branch some of the time and causing coverage to flap about.
This manifests as failed codecov reports for PRs that have 100% diff coverage, like #187.

Ok, I've found at least one of these lines. I doubt it's the only one; I'll start a list.

https://codecov.io/gh/dogmatiq/infix/src/master/eventstream/networkstream/server.go#L156

https://codecov.io/gh/dogmatiq/infix/compare/a07b167cb048cd0169bd0937634668fa67a44f5c...f047b2af3322430ae1844bbc40fe4f63f28d1f00/changes#L39

Ok, something very strange is going on.  The builds finish and Codecov reports a coverage failure because some coverage has dropped.  But then they change the status to passing?

Many of the tests have been refactored, and this issue does not appear to be happening any longer.

I can't win with this one! This line is flapping now:
https://codecov.io/gh/dogmatiq/infix/compare/1780c5321d24e3a3040ffb87277ceab3dfd9a251...489a2037c120b8ba695985dca69c7a588c2b3bbd/changes


Unbreak kafka

Description of changes
Fixes passthru of jre so the module works by default and selects a supported jre by default.
Fixes tests to be compatible with newer zookeeper-less CLI.
Drops old unused test compat

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/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg test kafka.kafka_2_8 kafka.kafka_3_0 kafka.kafka_3_1 kafka.kafka_3_2 kafka.kafka_3_3

@SuperSandro2000 #185169 was merged a bit too early, this fixes the remaining issues.

https://github.com/ofborg test kafka.kafka_2_8 kafka.kafka_3_0 kafka.kafka_3_1 kafka.kafka_3_2 kafka.kafka_3_3


can not connect the testnode

Hello,
When I run the verge -testnode .The debug.log in the /root/.VERGE/testnet2 told that "trying connection <IP_ADDRESS>:7777  connect() failed after select(): No route to host ." could tell me how can I deal with it ? thank you very much !

Pls try checking if you got a server route to reach the ip: <IP_ADDRESS>
On Thu, 10 May 2018 19:44 sergiojiu<EMAIL_ADDRESS>wrote:

Hello,
When I run the verge -testnode .The debug.log in the /root/.VERGE/testnet2
told that "trying connection <IP_ADDRESS>:7777 connect() failed after
select(): No route to host ." could tell me how can I deal with it ? thank
you very much !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/issues/744, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJWN2i29qt5ZS8HcpoQkILv9zPa-kPW-ks5txAwRgaJpZM4T5okJ
.

we will be posting new tor and clearnet testnet after the new update is up


Spanish translation

Spanish translation

Thank you @Sepho, just added to 4.0.8 branch

Fixed only the collation identifier to es-ES. Is it okay? (https://github.com/malcommac/SwiftDate/commit/067cc18c70fab099d9373696913ba2bea90bfc57)


Explicitly listing groups/arrays inside group metadata?

I'm curious if explicitly listing groups/arrays inside group metadata has been discussed before.
The downside is that this is a redundant and potentially duplicate information (but in some sense so is all group metadata, see "implicit groups").
One advantage would be that this would eliminates the need to list the contents of a store and check for the existence of metadata objects, which can sometimes be rather expensive. It's kind of a half-way step to the consolidated metadata of Zarr v2.
It's also potentially useful for making group creation/modification atomic without race conditions like https://github.com/zarr-developers/zarr-python/issues/1435, because the canonical list of a group's contents is a single metadata file rather than a collection of sub-directories, which usually cannot be modied in an atomic fashion.

We have been discussing an extension that would provided links to parents and children so that from any point within the hierarchy, you could navigate up or down the tree without having to list the store. This is conceptually similar to how STAC works.
This approach would be more scalable than the current consolidated metadata approach with the obvious tradeoff that walking the tree one node at a time will often be more expensive than loading a single mapping of all the metadata.

Adding @kbgg who's working on this.


github actions draft

Codecov Report

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   92.22%
=======================================
  Files           ?       13
  Lines           ?      373
  Branches        ?        0
=======================================
  Hits            ?      344
  Misses          ?       29
  Partials        ?        0


Access denied when copying from s3

After about few minutes I get "Access denied" errors for copying dataset. In beginning, all files can be coppied.
What is the transaction limit per second for this dataset? Maybe I am downloading it too fast?
Or is there some other reason?

I tested this also on ec2 instance and I confirm that more than a half of data is inaccessible in bucket

@evaldsurtans we have changed the structure of the data, all are now .tar.gz files and have made corrections to permissions. Take a look now. Thanks for reporting.


It would be nice to have if we could show supply per net level in a chart

During the internal presentation of the net calculation it was mentioned that a supply per net level (lv, mv, hv) chart was a nice to have.
@antw Do you have an estimation how much effort this will take?
As example the demand per net level:

@antw Do you have an estimation how much effort this will take?

I imagine modellers could do this without needing much effort from me.

The chart type used for showing supply needs a small change by me to make it reusable.
Queries for many of the electricity suppliers already exist. These could probably be sorted into LV, MV, and HV like we do with demand, but otherwise most of the work is already done.

I imagine modellers could do this without needing much effort from me.

Great.


Support shebang line

My node scripts with an initial shebang line #!/usr/bin/env node are getting converted to ///usr/bin/env node. These lines should be output unchanged in the formatted code.

Ah, haha, yes definitely.

When writting the test, my first issue is that flow parser wiped out the shebang.
As run_specs.js forces the use of flow parser, the result between the command line and the jest tests are not consistant. Shouldn't we test the presence of @flow to select the parser?


Feature/frax

Add FRAX metadata
Add FRAX bridge addresses
TODO: add FRAX bonder addresses

Closing since this is now deprecated.


disable auto command event trigger

When used with highlightedyank, this plugin triggers TextYankPost and highlightedyank highlights everything inside tag.
Creator of highlightedyank suggested this fix, and I tested that on local machine (using neovim).
Solution works perfectly and seems that it doesn't break any functionality and stops unnecessarily highlight.
Original issue here (from highlightedyank): https://github.com/machakann/vim-highlightedyank/issues/47#issuecomment-782584079

That makes sense, thanks :+1:.


fix(trading): volume in market header should display value in settlement asset

Related issues 🔗
Closes #6964

Looks good


Extract VCloseButton and add descriptive labels

Fixes

Fixes #618 by @sarayourfriend
Fixes #609 by @sarayourfriend
Description

This PR extracts the close button into a new component, VCloseButton.
Some labels are updated to be more descriptive:

Translation Banner: "Close the translation contribution help request banner"
Migration Notice: "Close the announcement"
Content settings menu on mobile: "Close the content settings menu"
Internal header pages menu black modal: "Close the pages menu"

To make the buttons work well with the focus- custom Tailwind styles, I made it possible to remove the border from the filled VIconButton.
Testing Instructions

Check the popovers (external sources, license explanation, report form), modals (mobile content settings, pages menu). The buttons should look and behave the same, but the name that you hear when using Screen reader should be more descriptive.
Checklist

[x] My pull request has a descriptive title (not a vague title like
Update index.md).
[x] My pull request targets the default branch of the repository (main) or
a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible
errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Also the fact that none of the existing stories changed is good (if it means that the button remains visually unchanged) or not-so-good (if is means that the button appears in none of them).

The buttons did not appear in stories, but they did appear in other snapshots (External sources, global audio player, filter and header modals), and I had to tweak some things to make them pass. One such change was adding focus-visible: modifier to the after: element in the focus utilities. Without this change, even though the button on the global audio player did not have a border set,  it had a ring around it in resting state.
Great idea about stories! I've added them, but did not add tests. Do you think it's worth bloating the snapshots directory and making the tests longer, if we already have the snapshots in the components or pages VR tests, @dhruvkb ?


Deploy FHIR Services

Include definitions and commands to set up a FHIR instance, including a FHIR server and mysql database
PR should include:

docker compose file to easily set up services
Cloudformation templates for AWS-based deployments

in mysql database, create a db with name cohort


[MP Website] Add Program Plan Presentation

Hi,
We need to add program plan presentations in all available languages to the website.
Currently we have a block with MP release presentation
https://microprofile.io/resources/#presentation
I think this block can be adjusted to include both presentation types.
The text on the page can be changed in the following way:
--
MicroProfile Presentation
These presentations cover the when, where, what, how and why of the MicroProfile initiative. Following the links you will find all the required information within the files that can be reviewed, downloaded and shared!

MicroProfile Overview Presentation
MicroProfile Program Plan Presentation

Get The Presentations
--
MicroProfile Overview Presentation should be updated (as far as I understand there is appropriate task for that) and uploaded as Google Slides to this folder.
I've already uploaded MicroProfile Program Plan Presentations in PDF format to the required folder. Anyway, if there are going to be any edits we can re-upload them later.
Let me know what you think on the offered steps and text changes.

Then the MicroProfile Overview Presentation can be linked to the folder
https://drive.google.com/drive/folders/1U599utXHUt1CrmnyJSwdiOtxbnjacEGx
Here later we can upload other release presentations if any appear this year.

@rstjames
as discussed on the MP marketing call yesterday, here are some edits that can be already done on the site to include the links to presentations:

https://microprofile.io/

in the second block under the specs add the button MicroProfile 4.0 Presentation
in The Mission block add button Program Plan with link

https://microprofile.io/workinggroup/
in Program Plan & Budget tab change the link to https://drive.google.com/drive/folders/1hj-o5s2-LXBpuw0h_xKkJVQ-iz265K6C

https://microprofile.io/resources/#presentation
in MicroProfile Presentation block change the text and delete the button (there will be two links instead) in the following way:

These presentations cover the when, where, what, how and why of the MicroProfile initiative. Following the links you will find all the required information within the files that can be reviewed, downloaded and shared!

MicroProfile Latest Release Presentation
MicroProfile Program Plan Presentation

Also, I think this MicroProfile Presentation block should be moved up instead of Eclipse MicroProfile White Paper 2019 (as it is already outdated). And below (where currently presentation is placed) there can be added a new block with the link to C2A cards that people can use for social media posts (I left appropriate comment in the related ticket).

All updated.

@rstjames looks great but it seems one edit is missed
the link on Working Group page here should be https://drive.google.com/drive/folders/1hj-o5s2-LXBpuw0h_xKkJVQ-iz265K6C

@TetianaFTV  fixed

As for the program plan link, I think it links to 2021 files. What will happen when we enter 2022. I think it is better off to link to a folder which contains the up to date program plan and then move the past program plans to archived folder or something similar. @rstjames @TetianaFTV thoughts?

+1 Emily. Anything that is scalable and requires no human attention should be always preferred.

Also agree with this @Emily-Jiang . This should be the standard for any linked files on the site. That was we do not need to update all links each year.

No objections to that. I renamed the folder with Program Plan presentations (deleted the year) and added extra folder Archived to move outdated presentations there when we have new one next year. The link remains the same, so there is no need to change it at the site.
In order to use the same scheme with unchangeable link for release presentations, I've create a separate folder and placed there a copy of 4.0 release. So now we need to paste this link everywhere where we have a release presentation mention
https://drive.google.com/drive/folders/1zqIzegskeYUCualDek3Wq4Sle9qW-_eY
In future, we'll be just moving these presentations to Archived folder also and uploading there the latest one without need to change anything on the site. Is it ok for everyone?

+1! :)

Thank you @aeiras @TetianaFTV @rstjames ! Great team work!

updated the links on the home page and resources page. I couldn't find where else it needed to be replaced. If anyone finds a link I missed please let me know.

seems all fine. thanks, Ryan!


Ov2640 module not working!!

Hi i am using this module for testing https://www.aliexpress.com/item/OV2640-camera-module-Module-2-million-pixel-electronic-integrated-with-jpeg-compression-new-big-promotion/32915971973.html . It doesnt have XCLK pin on header as it says it already has an integrated 12mhz crystal. I have already succeeded with esp32-cam board (small camera one) with this firmware. But this ov2640 camera module not working. Its showing "Camera probe failed with error 0x20001". I think this code refers to connection issue. I have changed the wiring and pin configuration but still no change. My pin configuration is:
vsync-25
href-33
reset-23
d1-32
d3-12
d5-13
d7-21
scl-26
sda-27
d0-35
d2-34
d4-5
d6-18
plck-19
pwdn-gnd
Thanks. Waiting for replies.

Hello,
Have you found a solution for this ? I am facing the same error with the same module as you

Hello,
I have the same challenge. Did you find a solution?

unfortunately, not yet

Hi,
I'm having the same problem. Has anyone found a solution yet?

I'm having the same issue (ESP32-CAM with Ov2640), I know the module is working fine since I uploaded and used a demo Arduino sketch without any issues.
With ESP32_CAMERA_QR:
I (537) sccb: pin_sda 26 pin_scl 27
I (567) sccb: SCCB_Probe start
E (1827) camera: Camera probe failed with error 0x20001
E (1827) camera_qr_demo: Camera init failed with error 0x20001
Was anyone ever able to get it working with an Ov2640 camera module ?

Hey guys,
So, I figured out a solution that worked well for my ESP WROVER.
Here  is the configuration I used :)