feat: new mux quickstart

Summary
New Mux quickstart. Includes Dashboard, data source, and quickstart files. Creating as a draft until the docs are published.

Pre merge checklist

[x] Did you check you NRQL syntax? - Does it work?
[x] Did you include a Data source and Documentation reference?
[ ] Are all documentation links publicly accessible?
[ ] Did you check your descriptive content for voice and tone?
[x] Did you check your descriptive content for spelling and grammar errors?
[x] Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

[x] Does the PR contain a screenshot for each of your dashboards?
[x] Do your screenshots show data?
[x] Has the sanitization script been run on each dashboard?

Alerts

[ ] Did you check that your alerts actually work?

@aswanson-nr has this been merged yet? I'm thinking we should hold off for just a bit, I got some feedback from Mux and they want to make a few changes to the dashboard and text here.

Not stale! going to remove that label.
@jcountsNR this appears to be ready to go, but there's a test.json dashboard. If that's intended to be part of this quickstart, could adjust the name to be more descriptive?

Removing the stale logo. Waiting on JRCounts to get clearance before publishing.

@aswanson-nr / @seemantk - Updating this one. It should be ready to be reviewed in the next cycle!


alpacastore: cast _cash and _value as floats

to fix:
File "C:\Python36\lib\site-packages\backtrader\sizers\percents_sizer.py", line 49, in _getsizing
size = cash / data.close[0] * (self.params.percents / 100)
TypeError: unsupported operand type(s) for /: 'str' and 'float'

Ah, sorry for delay. We'll check it out soon.


default to deploying dspo's main tag

I noticed when deploying DSPO by pointing a kfdef at this repository, I get a month-old version of DSPO. When I do the same for dashboard, I get main. We might as well default to main too ... if someone is pointing at DSPO or building it from source, they probably want bleeding edge anyway.
How Has This Been Tested?
I didn't test it yet.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

@gregsheremeta would you be willing to also set the imagePullPolicy for the container spec to Always, otherwise people deploying main` tag may end up deploying older versions because of cached images in the registry

If we hardcode it to Always, it will be Always in odh-manifests too, despite us not using main there.

That's a good point, let's leave it.
/lgtm
/approve


Request: making pbr.emissive compatible with "-vertexlighting" tagged models?

Just something that came up when I tried to add some decoration to one of many indoor areas. It turns out if a vertex lit model uses an emissive material, the albedo map will not be drawn at all, and the emissive material also ignores the "StaticallyLit" flag in the material file itself. (Not to mention emissive material ignores vertex colors, which are at times rather important for the model)
https://i.imgur.com/SuqxJGO.png
This is one case, of course. However, there are several models in the Cloud City map that will call for the same texture (ergo, material) for both interior and exterior areas, and they contain the emissive ceiling lights so it's going to provide a problem soon enough. The carbon freezing chamber is going to need emissive statically lit materials too. Would it be too much of a hassle to make these kinds of setups work in the future?

Just to comment on the photo: wow!

pbr materials ignoring vertex colours is intentional as they almost always carry simple baked lighting information and there is no good way to apply it over a pbr material. After v0.10 is done I plan to work on creating an easy way to bake high quality lighting for maps that will be usable with pbr materials. Does that sound like it would solve your problem?
One thing I could do fairly easily is create a normal_ext.emissive rendertype. normal_ext materials do respect vertex colours so that could be a solution as well. Do you think a normal_ext.specular.emissive rendertype could be a quick temporary solution to the problem?

I suppose so, yeah. The thing with the Cloud City map is that it has vertex colors almost literally everywhere, simulating light crossfading, interiors.. some models even use it for ambient occlusion. It would need a massive rework if that possibility wasn't there at all anymore. If any functionality that replicates the same effect can be reasonably used and implemented, I'm willing to give it a try.

In v1.0.0 normal_ext now supports emissive maps. That's as good as I can do for now I'm afraid, hopefully at some point in the future we'll get that hypothetical lightmap baking.


[Event Request] Codeunit 12 "Gen. Jnl.-Post Line" OnBeforeInitGLEntryVAT

Dear ALAppExtensions team,
On behalf of 4PS I would like to request integration event 'OnBeforeInitGLEntryVAT' to be changed in codeunit 12 "Gen. Jnl.-Post Line".
    [IntegrationEvent(false, false)]
    //local procedure OnBeforeInitGLEntryVAT(GenJnlLine: Record "Gen. Journal Line"; var GLEntry: Record "G/L Entry") //old
    local procedure OnBeforeInitGLEntryVAT(var GenJnlLine: Record "Gen. Journal Line"; var GLEntry: Record "G/L Entry") //new
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 51064.


water-well-15

@valleyofdawn
Would the existing water or drinking-water icon suffice for your needs?
If not, are there any other icon-concepts that might have a more unique silhouette?

This is an international symbol for wells (when in blue) used in British OS maps and Israeli government maps, among others.
Nowadays wells are more points of interest than sources of drinking water and I think warrant an icon of their own.
The suggested icons can be good if simplified to be legible at 11/15 pixels, which would be hard. In the middle east there is no roof over them.


Added exercise about excluding something from .gitignore

Thanks! As I go through the pull requests - one of the themes I notice is that you rock!


[AutoPR compute/resource-manager] Add readme.typescript.md for compute

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3924

(message created by the CI based on PR content)

TS PR


Add ethereum key generation

there seems to be some build issues. Can you fix this then we can merge this


better detection of forwarded / proxied HTTP protocol

while X-Forwarded-Proto is the defacto-standard, it is not actually a standard, and there has been many other alternative headers used that serve the same purpose, from the common (e.g. X-Real-Proto) to the obscure (e.g. cf-visitor) etc ...
forwarded-http handles all the commen headers as well as the less common ones, in addition to (and most importantly) defaulting to the RFC 7239 standard.

Coverage increased (+0.0%) to 99.61% when pulling 117118d12fd10f178569e67b319054c143fd0b04 on ahmadnassri:master into e9c9f95ade0f20a048861ac886d4767a839d5286 on strongloop:master.

Hi! Please make which headers are checked configurable; many people will not have blacklisted things like Forwarded and people will now be able to spoof things. In addition, this really should be contributed directly to our dependency: forwarded instead of us requiring two different forwarded header parsing modules.

many people will not have blacklisted things like Forwarded and people will now be able to spoof things

scratch that, as I see it's only being used for the proto and it really isn't that critical if that gets spoofed.

Hi! Please make which headers are checked configurable; many people will not have blacklisted things like Forwarded

fair enough, can be easily configured, however, what would the default behavior be?

backward compatibility: X-Forwarded-Proto only, however, I find that limiting, and as Express is a high level library, it should default to the HTTP standards.

light backward compatibility: X-Forwarded-Proto + overwrite with Forwarded when present.

and people will now be able to spoof things.

as per your comment, and everything is spoof-able with headers :)

In addition, this really should be contributed directly to our dependency: forwarded instead of us requiring two different forwarded header parsing modules.

I was studying proxy-addr to better understand what it does, as there might be some overlap with what forwarded-http does with IPs...
I reviewed forwarded and the reason I did NOT make changes there is because forwarded only deals with IPs, whereas forwarded-http looks for IPs, Protocol, Host, By as per RFC 7239 across many of the different implementations.
this means that proxy-addr could / should use forwarded-http in-place of forwarded .. unless you wish to keep a separation of modules for ip, proto, headers, etc ... (but lets discuss that further separately? I'd be more than happy to restructure and put under jshttp)

I reviewed forwarded and the reason I did NOT make changes there is because forwarded only deals with IPs, whereas forwarded-http looks for IPs, Protocol, Host, By as per RFC 7239 across many of the different implementations.

Correct, but there is no reason it cannot be expanded. The reason it's called forwarded is to match the HTTP Forwared header.

Correct, but there is no reason it cannot be expanded. The reason it's called forwarded is to match the HTTP Forwared header.

:+1: PR coming up :)

as per your comment, and everything is spoof-able with headers :)

absolutely not true; reading of the headers is disabled in express by default and enabling the feature means you are saying you definitely trust the X-Forwarded-* incoming headers (i.e. saying you have a proxy that strips them and/or updates them). if your proxy updates them, then you really need to specify a specific depth or a proper IP trust chain, which is what proxy-addr; this is standard prod-ops configuration.
ALSO I have seen your avatar around (and even spoke with Mashape a few times), and if you are interested in joining jshttp, let me know offline and we can discuss :)
Typically, to keep the codebase of Express from becoming a giant monolith, we divide functionality into the hierarchy express -> pillarjs -> jshttp, and so typically desire contributions to head to relevant sub-modules in these organizations, if one exists, and then can be incorporated up the chain ;)

because forwarded only deals with IPs

P.S. the forwarded module has nothing to do with IPs; the X-Forwarded-For header is just a comma-separated list of proxy servers: there is no IP requirement in the de-facto spec (just like in Forwarded) ;)

absolutely not true; reading of the headers is disabled in express
right, I meant on the HTTP/network layer spoofing occurs, we're still using trust proxy setting before processing those values here. we're on the same page.

you really need to specify a specific depth or a proper IP trust chain
absolutely, I had added a simple regex-based filtering of ips in http-forwarded for this very reason, but I prefer the way proxy-addr does it actually.

monolith, jhttp, etc ...

:+1: fully agreed, its good to get a conversation like this started, even if a PR gets closed / discarded, then move on to make the appropriate changes elsewhere as needed.
be happy to contribute my time to jshttp, and further any discussion offline, as it is already, I live in the HTTP world in Mashape daily (and in Node.js) what's the appropriate channel for further discussion? email, IRC?

I live in the HTTP world in Mashape daily (and in Node.js)

haha, that's what I figured ;)

what's the appropriate channel for further discussion? email, IRC?

Just email me and I would love to set something up to speak with you and really want to connect to grow these types of modules in Node.js :)

P.S. the forwarded module has nothing to do with IPs; the X-Forwarded-For header is just a comma-separated list of proxy servers: there is no IP requirement in the de-facto spec (just like in Forwarded) ;)

s/IPs/Addresses/g :)

(FYI for future readers) as per discussion above, a PR to update the functionality of forwarded is now where this conversation will continue


Improved test logging

This is mostly to help me diagnose a mysterious ToolLocationHelper test, but it also includes some infrastructure that should help many future tests (once they get opted into xunit logging).

:shipit:


[sqlite3] build failure

Not able to install the package it's showing
The following packages will be built and installed:
sqlite3[core]:x86-windows -> 3.35.5
No suitable Visual Studio instances were found

@arunabh42 ,Thanks for posting this issue,
have you installed Visual Studio?

Yes installed.
On Wed, 30 Jun 2021, 12:17 JonLiu1993, @.***> wrote:

@arunabh42 https://github.com/arunabh42 ,Thanks for posting this issue,
have you installed Visual Studio?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/18716#issuecomment-871141320,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVIDU7QUEGCYODXRW3MC4LTVK4YBANCNFSM47QI3CBA
.

@arunabh42 ,Can you provide your specific visual Studio version?

It's Visual Studio 2019
On Wed, 30 Jun, 2021, 1:39 pm JonLiu1993, @.***> wrote:

@arunabh42 https://github.com/arunabh42 ,Can you provide your specific
visual Studio version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/18716#issuecomment-871189049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVIDU3HTW2ZWMHOV4ZQKGLTVLGJ3ANCNFSM47QI3CBA
.

Please download the English language pack and try to install again

@arunabh42, does this still be a problem for you? have you try above suggestion?

Yes, it works.
On Thu, 15 Jul, 2021, 11:43 am Phoebe, @.***> wrote:

@arunabh42 https://github.com/arunabh42, does this still be a problem
for you? have you try above suggestion?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/18716#issuecomment-880425817,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVIDU6V3PGPDXLF6LX7MPLTXZ4ADANCNFSM47QI3CBA
.

It dsnt work! Windows 10 + Visual Studio 2022


Testing functions

In this update, my partner and I:

Wrote unit tests for the files operate.js and calculate.js using Jest.
Created unit tests for all React components: we Used Jest snapshots to test the components and React Testing Library to simulate user interaction.

Hi @BenM4k 😊,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request 🛑 💀
You have submitted a PR for the wrong milestone, kindly submit the correct PR for the right milestone
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
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.

Invalid Code Review Request does not count into the code reviews limit.


Create feature tests for testing UI and js

Will be awesome to test our UI and js code. We can use capybara for that.
What we need to test primarily:

Web app

New task page (create task, get issue)

Admin app

Moderation page (approve task or not)
User page (block, unblock users)

What we need to test:

Switcher on index page (http://www.ossboard.org)
Task status filter on task page (http://www.ossboard.org/tasks)
Export logic on new page (http://www.ossboard.org/tasks/new)
Preview switcher on new page (http://www.ossboard.org/tasks/new)

@davydovanton I can take this issue.
But I have some questions:

What driver we can use? Each driver has pros and cons. I.e. phantomjs is fast, but some times test is failed without any resons. Firefox/chrome is slow, but it's close to real using.
Can I use some abstraction layer for capybara? I'm talking about https://github.com/natritmeyer/site_prism. The main idea that you have an page object with declaritive description of testing page or pages and easy-breeze test cases. When structure of page is changed and test's are failed, you need to change the page objects not tests.

Also have an issue with configuration of capybara.
And this is reason https://github.com/ossboard-org/ossboard/blob/master/spec/support/capybara.rb#L5
I've changed it to ::Hanami::App.new as explainde here https://github.com/davydovanton/rspec-hanami/issues/1
but then i have another issue app.rb:21:in 'initialize': wrong number of arguments (given 0, expected 2) (ArgumentError)
That seems reasonable https://github.com/hanami/hanami/blob/master/lib/hanami/app.rb#L22
But how configure this properly?

I'm already working on this task using phantomjs and poltergeist. Thank you for attracting my attention to site_prism.

@inpego ok! If you have some questions about site_prism or how to use it you can ask me.
btw how you solved this issue with capybara.app? :)

I had no problems with the capybara, try checkout my branch and run tests.


Pin Binding refactoring

+ comments, fixes, tests, cleanup

Coverage increased (+0.05%) to 92.079% when pulling 6e7c940d9da6248193a81a76bc9df22c99499429 on gabrielburnworth:staging into 4372a2f4823bfe45108ce9c476818bd3076edd30 on FarmBot:staging.


Documentation revision

@yakutovicha I found it easier to just create a PR into your instead of creating hundreds of comments. Once you have merged this, I can approve your PR.


Theming - Bootstrap Variables

Map Bootstrap variables to custom theme variables defined by #1538.
Epic: https://github.com/deephaven/web-client-ui/issues/1537

Taken care of by #1603


Create all events page

Context:
User clicks on the events page, can see it's events, the discover section but wants to see it's events, so clicks the "see all" button
Page behavior:

tab “upcoming”.
all events created by the user or to be participated (and that have not started yet)

tab "past":
all events created by the user or that he/she has participated in but that have already happened

how every event should look like

note:
same event component as here https://github.com/Couchers-org/couchers/issues/4784 ignore the UI difference, it's just due the software I'm using to create this mockups


podman: tries to create directories in /var while being rootless

/kind bug
Description
When trying to run podman rootless with --log-level=debug this message occures.
DEBU[0000] Error creating parent directories for blob-info-cache-v1.boltdb, using a memory-only cache: mkdir /var/lib/containers/cache: permission denied

Expected
podman shouldn't try to write to /var/lib/containers/cache when it's in rootless mode.
Might be connected with https://github.com/containers/libpod/issues/2508

It seems like some paths might be set incorrectly - we should be using paths exclusively under /home with the default configuration.

Possibly related to #2508 - both of them seem to be failing to detect that we're rootless and adjust paths to appropriate defaults

PR here: https://github.com/containers/libpod/pull/2537

It's alive again on rootless.
DEBU[0000] Error creating parent directories for blob-info-cache-v1.boltdb, using a memory-only cache: mkdir /var/lib/containers: permission denied

@giuseppe Anything to do with your rootless merge?

I would bet this was the big rootless single userns patch that just merged.

this is related to a change in containers/image where we use os.Geteuid() inside of a user namespace

PR: https://github.com/containers/image/pull/612

also got bitten by this; will we get a new release soon?


[Chore] Code style refactor to be more consistent and easier to read

A bunch of little cleanups, recommended by the Vetur plugin in VS Code.

Review Error for almindor @ 2020-06-21 23:36:33 UTC
User failed mfa authentication, see go/mfa-help

This Pull Request was merged without enough reviews
Please go here to review and +1


Connection Handshake

RSR does not currently implement any form of a handshake. The handshake process should include, at minimum, the following elements:

Protocol version
Range of protocols the client can speak
Means of selecting a protocol in the intersection of supported ranges

Added basic handshake and called it from AcceptConnection and InitiateConnection. Right now, protocol version 1 is hard coded.
See ef7349f46b6457cba92e1eac26105c336b56d5eb


Resolve scss linting issues in new scss within framework

Includes new components, campaign admin and base styles like typography

Merging this, assuming Paul might create some follow-up PRs for the two bits of feedback I had. These whitespace fixes have a habit of messing up our workflow (lots of waste in branch rebasing and merge conflicts), so I don't want to keep it open for too long.


Debug failing windows build in CI

https://github.com/opendp/opendp/actions/runs/6555966894/job/17805224871

That particular problem was presumably fixed, and more recently we're gotten confirmation from windows user that the latest nightly works for them:

Using version 0.10.0a20240804001 works great.

#1885
#1758


Update IOU currency selection for offline first

Details
We should re-fetch data when we are coming back online. Part of offline first
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/208194
Tests

Go to a chat
Send/receive money
Click on the $ in the following page
Go offline
Go back online
Make sure you see https://www.expensify.com.dev/api?command=Mobile_GetConstants in the network tab

All other platforms:

Go to a chat
Send/receive money
Click on the $ in the following page
Go offline
Go back online
Make sure you the page does not refresh, and you can still select options

[ ] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (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 Staging and/or Production testing in the QA steps section
[ ] 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 included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] iOS / native
[x] Android / native
[ ] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[ ] 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)

[ ] 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 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 was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] 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
[x] I followed the guidelines as stated in the Review Guidelines
[ ] 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)
[ ] 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 */
[ ] Any functional components have the displayName property
[ ] 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

[ ] 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(themeColors.componentBG)

[ ] 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.

PR Reviewer Checklist

[ ] 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 verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] 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)
[ ] 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 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 was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] 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 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 */
[ ] Any functional components have the displayName property
[ ] 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 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(themeColors.componentBG)

[ ] 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.

QA Steps
Web:

Go to a chat
Send/receive money
Click on the $ in the following page
Open the network tab of the console
Go offline
Go back online
Make sure you see command=Mobile_GetConstants in the network tab

All other platforms:

Go to a chat
Send/receive money
Click on the $ in the following page
Go offline
Go back online
Make sure you the page does not refresh, and you can still select options

[ ] Verify that no errors appear in the JS console

Screenshots

Web
https://user-images.githubusercontent.com/42391420/166300369-ef741883-6a00-442f-ba0c-cb658053ab79.mp4
Mobile Web
https://user-images.githubusercontent.com/42391420/166313080-7f1bd661-a6dc-442e-8d73-ee22c489f460.mp4
Desktop
https://user-images.githubusercontent.com/42391420/166300099-c77df0a6-2081-48d6-9c79-aaee7f6411c5.mp4
iOS
Unable to test due to broken netInfo
Android
https://user-images.githubusercontent.com/42391420/166313056-f5d38a69-49cd-4227-b2f9-a008603c8fc9.mp4

: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.

🚀 Cherry-picked to staging by @sketchydroide in version: 1.1.57-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @chiragsalian in version: 1.1.57-17 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Prefix tuning experiment

Like the readme says, this is a proof of concept more than a fully-fledged experiment.

@AkshitaB, this is ready for review, at least after https://github.com/allenai/tango/pull/231 is merged.

🎉


Want to add CV template

[ ] Feature Request

CV template is not included in this front end templates
I will make nice CV template using HTML and CSS and add it.

\assign

@DhairyaWankhade finish it within 3 days.


Adding GCP Deployment Template

This pull request adds a new GCP Deployment Template to streamline the deployment process on Google Cloud Platform (GCP). The template includes the necessary configuration files and instructions for setting up the application infrastructure.
Changes included in this pull request:
Added the GCP Deployment Template directory with all relevant files and configurations.
Updated the README file to provide detailed instructions on using the GCP Deployment Template.
Ensured compatibility with the latest GCP services and best practices.
This contribution aims to enhance the user experience and reduce the setup time for deploying our application on GCP. By utilizing this template, developers and users can easily deploy and scale the application with minimal effort.

The changes have been resolved.

This is amazing, merging now. Ping me for Contributor roles in Discord if yall are in there!


Model different versions of grammatical number in word forms

Context: I am using JWKTL to work with declension tables for German nouns.
There's a feature I need (and can implement) but I'd like to first discuss, what would be the best way to model it.
Basically, I want to be able to produce the declension if a given German noun.
 So essentially this boils down to generating the full declension table or its columns.
For most cases (ca. 90%) this is pretty straightforward. Two grammatical numbers, four grammatical cases - 8 word forms. Sometimes few forms are missing, sometimes there are two versions for one number/case, but it's pretty trivial.
But in some cases it gets more complicated. Some words may have several genders and sometimes there are different singular and plural forms. The most extreme example is Eponym with two genders (m, n), two singular and two plural declinations and up to 3 variations per number/case giving a total of 28 word forms.
But apart from that extreme example, the case with several grammatical numbers is rare, around 4%.
To process such cases, I need to know which words belong to the same "number". Let us take Dschungel for example:
{{Deutsch Substantiv Übersicht
|Genus 1=m
|Genus 2=n
|Genus 3=f
|Nominativ Singular 1=Dschungel
|Nominativ Singular 2=Dschungel
|Nominativ Singular 3=Dschungel
|Nominativ Plural 1=Dschungel
|Nominativ Plural 2=Dschungeln
|Genitiv Singular 1=Dschungels
|Genitiv Singular 2=Dschungels
|Genitiv Singular 3=Dschungel
|Genitiv Plural 1=Dschungel
|Genitiv Plural 2=Dschungeln
|Dativ Singular 1=Dschungel
|Dativ Singular 2=Dschungel
|Dativ Singular 3=Dschungel
|Dativ Plural 1=Dschungeln
|Dativ Plural 2=Dschungeln
|Akkusativ Singular 1=Dschungel
|Akkusativ Singular 2=Dschungel
|Akkusativ Singular 3=Dschungel
|Akkusativ Plural 1=Dschungel
|Akkusativ Plural 2=Dschungeln
|Bild=Hopetoun falls.jpg|200px|1|ein Wasserfall im ''Dschungel''
}}

To create this declination table I have to know not just the basic grammatical number (SINGULAR or PLURAL). I have to know if it's Singular 1 or Singular 2 etc. Then I can group word forms into a column of the declension table.
However, at the moment JWKTL (quite logically) only models grammatical number SINGULAR or PLURAL. At the moment I can't know if it was Singular 1 or Singular 2 which is my problem.
I would like to add this information to IWiktionaryWordForm, but I am not sure which would be the preferred way to model this. My suggestion would be to simply add the string rawGrammaticalNumber property. There's already something similar in IWiktionaryEntry.getRawHeadwordLine(), so the concept should not be completely out of its way.
Still, I'd like to hear your opinion on this before I actually implement this.

The varying number of forms per table cell motivated the List-based model of IWiktionaryEntry.getWordForms() instead of using a data structure that mimics the inflection table. This turned out useful for modeling other languages as well, which distinguish other grammatical properties. Core of the IWiktionaryWordForm class is thus the word form (of course) plus all relevant grammatical properties of this particular form (e.g., number, case). Properties that are not applicable, are null.
Considering this basic idea, I suggest to add GrammaticalGender getGender() to IWiktionaryWordForm and resolve the numbers (e.g., Genus 1=m) during parsing, such that all masculine forms receive a gender marking, whereas the Genus lines should not create a word form.
Linguistically, this model is a bit debatable, but I would consider it a good practical solution.
This would simplify using the API, since having only the raw number leaves the interpretation burden to the downstream applications. Typically, the list of word forms should be ordered according to the original wiki syntax, such that alternative forms (e.g., two Genitive alternatives are accessible in the order as they appear in Wiktionary).
There's an additional pitfall here: The word form processor must process the singular forms and the plural forms differently, as the gender index numbers do not necessarily correspond to the plural form: In the Dschungel example, the singular forms are 1=MASC, 2=NEUT, 3=FEM, but the plural forms are 1=MASC/NEUT, 2=FEM. So apart from a grammatical-number-specific parsing process, we need to define how to set the plural genders for case 1. I see the following options: 1. use MASC, 2. introduce a MASC_NEUT type, 3. use null, 4. duplicate the form, one with MASC, one with NEUT. I currently lean towards 1, but let's discuss.
Below is a sample conversion for the 20 word forms for Dschungel (using option 1):

Dschungel NOM SING MASC
Dschungel NOM SING NEUT
Dschungel NOM SING FEM
Dschungel NOM PL MASC
Dschungeln NOM PL FEM
Dschungels GEN SING MASC
Dschungels GEN SING NEUT
Dschungel GEN SING FEM
Dschungel GEN PL MASC
Dschungeln GEN PL FEM
Dschungel DAT SING MASC
Dschungel DAT SING NEUT
Dschungel DAT SING FEM
Dschungeln DAT PL MASC
Dschungeln DAT PL FEM
Dschungel ACC SING MASC
Dschungel ACC SING NEUT
Dschungel ACC SING FEM
Dschungel ACC PL MASC
Dschungeln ACC PL FEM

What do you think?

I'm not quite sure what you mean here:

whereas the Genus lines should not create a word form.

What are "Genus lines"?

Considering this basic idea, I suggest to add GrammaticalGender getGender() to IWiktionaryWordForm and resolve the numbers (e.g., Genus 1=m) during parsing, such that all masculine forms receive a gender marking, whereas the Genus lines should not create a word form.
Linguistically, this model is a bit debatable, but I would consider it a good practical solution.

If I understand you correctly, you suggest to add gender property to the word form and resolve it during parsing? Then one "column" in the declination table will be defined by the number+gender, correct?
I am not quite sure if this will work in all cases. There may be cases where the same gender+number have different declinations. I don't have an example at hand, but I think this is possible. I'll have to experiment to find out. If it works, this is a definitely good way.

There's an additional pitfall here: The word form processor must process the singular forms and the plural forms differently, as the gender index numbers do not necessarily correspond to the plural form: In the Dschungel example, the singular forms are 1=MASC, 2=NEUT, 3=FEM, but the plural forms are 1=MASC/NEUT, 2=FEM.

I'm not quite sure if I understand you correctl. Do you mean that Plural 1 in the table below is for m/n and Plural 2 is for f?

And you know this how - because you're a native speaker, not from the data, correct?
If so then I would say this is an error in the data. We can't (easily) understand to which genus does a plural form belong.
I think I'd first check how many cases like this do we have. I think we can check this by checking if number of genera == number of plurals.
If there are not too many cases (hopefully), I'd consider them to be errors in the data and fix them in de.wiktionary.org.
If there are many then we'll have to consider cases like

3 plurals to 1 genus
3 plurals to 2 genera
2 plurals to 1 genus

An then deside what we take as genus for each case. But we're not there yet.

So here's a plan. I'll try the following:

Add getRawNumber() in a local branch - this will help me with my experiments.
Check if there may be different declinations for the same genus - and how often is this the case.
Check how often the number of genera in the declination table != number of singulars or plurals.
Output examples for the case when the number of genera in the declination table != number of singulars or plurals. Discuss what do we do exactly.

Fix it in Wiktionary?
Implement some heuristic?

Try to implement parsing of genus into the word form.

What do you think?

I checked Wiktionary a bit, and found that the "Genus 1=m"-like lines only apply to the singular forms (basically only to choose the correct article). Since the articles of plural forms are regular, there is no connection to the "genus index"/raw form number for plurals.

I'm not quite sure if I understand you correctl. Do you mean that Plural 1 in the table below is for m/n and Plural 2 is for f? [...] And you know this how - because you're a native speaker, not from the data, correct?

Yes. It is "der Dschungel" (MASC, SING), "die Dschungel" (FEM, SING), "das Dschungel" (NEUT, SING), "die Dschungel (MASC + NEUT, PL), and "die Dschungeln" (FEM, PL) for the nominative case. It's not really incorrect in Wiktionary, as the "genus" is only used for the articles (which is "die" for all cases). But of course this raises some problems when analyzing the data.
See https://de.wiktionary.org/wiki/Vorlage:Deutsch_Substantiv_Übersicht and the source code for the templates to see how inflection tables are rendered.
So my conclusions are so far:

Storing the raw form number is simple, but it is unintuitive for users, as (a) the gender assigned to a certain form number is not clear (In Wiktionary, this is done by writing Genus 1=m" for assigning form number 1 with MASC) -> so we would need to store a mapping of number to gender additionally.
Adding a gender attribute to the form class would work nicely for all singular forms. Also multiple alternatives (e.g., "des Staats" and "des Staates" could be stored as two different instances of WordForm, both with Gender=MASC and Case=GEN). I would prefer this solution over the raw form number.
Problematic are the plural forms (both for raw form number and gender attribute), since the raw form numbers do not clearly indicate which plural belongs to which gender. We could hence either leave gender=null (then we're on the save side, but lack the plural - gender link) or we try to resolve the gender heuristically (e.g., if there are multiple plurals, use the one which changes the NOM form according to typical inflection rules or check if there is a systematic order in Wiktionary such as m before f).

Ok, thank you for the clarifications.
I fully support adding gender to word forms. I'll file an issue and start working on it (at least for singular forms).
As for plural forms I don't think I will be able to implement this straight away.

One more questions: is it OK if I only implement this for German?

Yes.

Adding the gender property can be done in general - it will be null for other languages. A corresponding enum type already exists.
Gender would be filled for some German singular forms (if information is available).
For plural forms, we can leave gender = null, unless we find another good alternatives. It will still be possible to access the two different plural forms (per case) for Dschungel - it only remains unclear which gender they belong to.

For plural forms, we can leave gender = null, unless we find another good alternatives. It will still be possible to access the two different plural forms (per case) for Dschungel - it only remains unclear which gender they belong to.

I have a simple idea here: what if we introduce something like a "gender index"? To distinguish Plural 1 and Plural 2.
Frankly, I'm looking for a simple solution since I need to relate word forms to the respective "columns" in the declination table, but gender-detecting heuristics would be too hard for me to implement.

Please check #58.

@chmeyer Seems I've encountered a problem. Please see the following word:
https://de.wiktionary.org/wiki/Fels

It has two singulars, both masculine. With three alternatives in Genitiv Singular 1.
In this case it will not be possible to correctly group even singular word forms knowing only their respective gender. All singular forms will have MASCULINE.
So it seems the solution we were pursuing would not be universally sufficient. It still makes sense adding gender to word forms (and I'm pretty far with the implementation). I'm not sure how common or frequent this case is, though, did not do any statistics yet.
But it really seems that just adding gender won't quite solve my problem - which is grouping word forms per "declension" (i.e. detecting columns in the declension table):

There may be several declensions with the same gender (the problem I'm reporting now).
Detecting gender for plural forms is not solved anyway.

What I'm thinging about right now is maybe introducing an additional structure like IWiktionaryWordDeclension which would hold a list of IWiktionaryWordForms.  IWiktionaryWordDeclension would also have a gender and a number.
IWiktionaryEntry will get an optional list of IWiktionaryWordDeclensions in addition to already existing wordForms.
In this way we will not expose the interna (Genus i/Accusativ Singular i) but allow processing the declension table.
What do you think? I could sketch the API in yet another branch.

That's indeed a problem. From the wiki code, it will not be possible to reliably assign the plural form, as it applies only to one of the forms.
Regarding your IWiktionaryWordDeclension suggestion, I am a bit worried if this will nicely transfer to languages other than German, which have differently structured inflection paradigms. Also for verbs, it is not intuitive, according to which property they will be grouped (an then declension would be a bad term anyway). It might make the usage of the API more complicated, since one can access inflected forms both by the ungrouped word form instances and by the grouped IWiktionaryWordDeclension lists.
Given the entire discussion, we should maybe get back to your original proposal of just making the index accessible in the forms? This would be both easy and versatile, although not overly comfortable. I am thinking about a IWiktionaryWordForm.getInflectionGroup(): int method.
For the Fels example, we would thus generate 14 word forms with the following properties:

form=Fels, gender=MASC, num=SING, case=NOM, inflectionGroup=1
form=Fels, gender=MASC, num=SING, case=NOM, inflectionGroup=2
form=Fels, gender=null, num=PL, case=NOM, inflectionGroup=0
form=Fels, gender=MASC, num=SING, case=GEN, inflectionGroup=1
form=Felses, gender=MASC, num=SING, case=GEN, inflectionGroup=1
form=Felsens, gender=MASC, num=SING, case=GEN, inflectionGroup=1
form=Felsen, gender=MASC, num=SING, case=GEN, inflectionGroup=2
form=Felsen, gender=null, num=PL, case=GEN, inflectionGroup=0
form=Fels, gender=MASC, num=SING, case=DAT, inflectionGroup=1
form=Felsen, gender=MASC, num=SING, case=DAT, inflectionGroup=2
form=Felsen, gender=null, num=PL, case=DAT, inflectionGroup=0
form=Fels, gender=MASC, num=SING, case=ACC, inflectionGroup=1
form=Felsen, gender=MASC, num=SING, case=ACC, inflectionGroup=2
form=Felsen, gender=null, num=PL, case=ACC, inflectionGroup=0

What do you think?

Addition: I don't like "rawFormIndex" (or similar) too much, as this makes it difficult to handle for users. That's why I used "inflectionGroup" which seems a bit more logical to me.

I'm totally fine with inflectionGroup. This solves my problem. I'll finish the singular gender first and do it next. Thank yo.


Too many open files with MANY subworkflows.

With the following workflow
[nested]
parameter: idx=int
print(f'I am nested workflow {idx}')

[0]
parameter: rep=15
input: for_each=dict(rep=range(rep))
sos_run('nested', idx=_index)

If I run it with large rep, namely a large number of subworkflows,
sos run test --rep 300

I will get an error saying "too many open files".
The problem here is that SoS workers might have many slots, each with an open socket, so the total number of sockets/open files can be large.

Is this related to #1208 where we had several rounds of discussions and have already eliminated lots of them? Does the improve -j control help, or not because we are talking about cumulatively opened sockets?

I can reproduce the too many open files problem with
sos run test.sos --rep 300 -j4

but using the open file checker script in #1208 , I see
5 processes
79966 has 2 open files and 36 open connections
79993 has 1 open files and 13 open connections
79994 has 2 open files and 30 open connections
79995 has 2 open files and 30 open connections
79996 has 2 open files and 25 open connections
Overall 2 open files and 90 open connections

So there are not many open connections at all.

check_open_files.txt
test_sos.txt
test_sh.txt

zon a Linux system, I get the error when I use --rep 3000, and the process stops at 530 open files.

The patch makes sure all steps are processed before workflows so that workflows will not be submitted early and wait for their steps to be completed (while occupying a socket). With this change the maximum number of open connections is kept below 50 all the time.


Login failed since Easee-API update

Easee will remove one of their login endpoints during this maintenance. If you have any systems relying on 'https://api.easee.cloud/api/accounts/token' please move this to 'https://api.easee.cloud/api/accounts/login' before this maintenance.
https://easee.statuspage.io/incidents/zqqdv81f23sx

Duplicate of https://github.com/Newan/ioBroker.easee/issues/45


NPE in String constructor in JS when running in Cordova on emulator

This works in the browser, but fails with an NPE in the language module here:
function $_String(/*{Character*}*/value) {
if (value && value.getT$name && value.getT$name() == 'ceylon.language::String') {
//if it's already a String just return it
return value;
}
else if (typeof(value) === 'string') {
return new String(value);
}
var that = '';
if(value == undefined) return that; // MY FIX
var _iter = value.iterator(); // FAILS HERE OTHERWISE BECAUSE value IS undefined
var _c; while ((_c = _iter.next()) !== finished()) {
that += _c.string;
}
return that;
}

TBH I've no idea how this comes to be. It's very possible this is a bug in the caller who should not be calling this, but I've no stack trace, this appears to happen before I've had a chance to even print anything in my app.

I'll first try to make a test that runs on node without the browser. If that does not exhibit the issue, we'll have to devise a way to figure out who calls that constructor with undefined on Android.

As it is, it's too unclear for @chochos to fix. In the worst case, I'll share my Cordova app for him.

You mean make a test that fails on node? I think that constructor is tested in the language module tests.
Does console.log work for Cordova apps?

OK, it's due to this code in the language module:
if (navigator.languages||navigator.userLanguage||navigator.browserLanguage||navigator.language) {
properties["user.locale"]=$_String((navigator.languages || [navigator.userLanguage||navigator.browserLanguage||navigator.language])[0]);
}

And here's their values:
navigator.languages=""
navigator.userLanguage=undefined
navigator.browserLanguage=undefined
navigator.language="en-US"

Frankly that [0] in your JS code looks like it should only belong on navigator.languages (plural), no?

Pushed a workaround, I think Cordova's just broken because navigator.languages shouldn't be an empty string, it should be an array.


Use official GraphQL.js subscription API in example

This was using my old, janky "subscribe" adapter. There's been a real API since GraphQL v0.10.x, so let's use that instead.

Ping @edvinerikson – can we merge this? It's closer to what people should be doing these days.

So sorry for the delayed response! Yep let's merge it! Thank you!


No column name completion for SET in UPDATE.

If you type something wrong it marks it as an error, but it neither auto-completes nor shows a list when hitting cmd+space. It works in the WHERE clause, but not for SET.

Seems to work once the statement is complete with a semicolon. Not during initial typing though.


[DO NOT MERGE] Run all PostCommit and PreCommit Tests against Release Branch

You can run many tests automatically using release/src/main/scripts/mass_comment.py.

Re-running most of them.

Run Jpms Dataflow Java 11 PostCommit

Run XVR_Samza PostCommit

Run Go PostCommit

Run Release Gradle Build

Run Dataflow Streaming ValidatesRunner

Run Jpms Dataflow Java 11 PostCommit

Run XVR_Samza PostCommit

Run Jpms Dataflow Java 11 PostCommit

Run Java Spark v3 PortableValidatesRunner Streaming

Run Dataflow Streaming ValidatesRunner

Run XVR_Samza PostCommit

Run Jpms Dataflow Java 11 PostCommit

Run Jpms Dataflow Java 17 PostCommit

Run Python 3.7 PostCommit

Run Python 3.8 PostCommit

Run Python 3.9 PostCommit

I've found that trying to run all the dataflow based tests simultaneously simply causes spurious failures in various levels of API quota. apache-beam-testing simply can't do them all at the same time.
Thank you for pointing out the missing tests. They were not in the list I used to trigger them at the time.

Noting that Python 3.8 suite passed (https://ci-beam.apache.org/job/beam_PostCommit_Python38_PR/), barely making timeout cutoff:
01:34:24 BUILD SUCCESSFUL in 3h 59m 38s
01:34:24 220 actionable tasks: 157 executed, 57 from cache, 6 up-to-date

Run Python 3.7 PostCommit

Run Python 3.9 PostCommit

cxled 3.9 manually (forgot that we need not to run suites together), we can trigger once 3.7 finishes (fingers crossed!).

Run Python 3.9 PostCommit

Run Python 3.9 PostCommit

Run Java Examples_Flink


Add usage examples & detailed project description

Hello! This seems like a nice project however currently it is hard to understand what it is doing. A few code snippets for showing the usage of this tool/library would be nice 🐻

There's now a simple README and a pretty basic example, as well as a link to use in Linkerd. Let me know if there's anything that needs clarification!


This is achievable without TS Server changes, allowing it to work on TypeScript versions as old as 4.7, but perhaps not optimally so. As it is now, definitionAndBoundSpan is still called first, because findSourceDefinition does not return the text span used to draw the underline, and it also does not return fallback results to type declarations, which I think we would want if Go To Definition is fully replaced. findSourceDefinition is called subsequently, and the definitions it returns replace the ones returned by definitionAndBoundSpan. If findSourceDefinition returns no results, the response from definitionAndBoundSpan is used as a fallback.
definitionAndBoundSpan is typically very cheap, so this might be acceptable. But there are two ways we could improve performance:

Make a new TS Server command that returns source definitions, along with the text span, and falls back to type declarations when no other definitions are found. This would allow VS Code to trigger only one request, provided the TS version is new enough to have this capability.
A bigger change to VS Code, but the best thing for perceived performance would be to support separate requests for drawing the underline and showing the definitions. findSourceDefinition can be considerably more expensive than definitionAndBoundSpan, so it would be nice to trigger the latter when hovering with ctrl/cmd, and then trigger the former once the user clicks.

As discussed, we'll merge this as is since it seems good enough. In future TS release, we may update the source definition request to return a span


Add Register Devices endpoints

Hey! Me again with a small PR to add support for a new endpoint. I also updated the changelog and bumped the version so that it's easier for you to release a new version.
Let me know if that's fine 😄

Hey @BalestraPatrick, thanks for this new contribution!
May I ask you a small change: can you make the new version number 0.7.0?
I'd like to increment the "second" number as soon as there is a new feature.

@ppawlak Done!

@ppawlak Anything left to merge and make a new release with this change? 😄

@ppawlak Anything left to merge and make a new release with this change? 😄

Time 😉
I'll do it soon.

Version 0.7.0 is on PyPI


Testing and documentation for staking cli

We need tests and documentation for the oasis-node stake cli.
Estimated cost: ~1 sprint

Is this waiting on docs updates?

Yeah, docs are now also merged. Closing.


Permissions error while running npx create-react-app foo

Is this a bug report?
No, probably just a problem with my system

Hello, just installed xcode and homebrew and node on my Mac and am trying to create a react app now, but getting a permissions error on a file in my user directory, any help appreciated:

$ npx create-react-app foo
npx: installed 63 in 3.568s
Creating a new React app in /Users/mryerse001/Documents/GitHub/foo.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts...
npm ERR! path /Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall open
npm ERR! Error: EACCES: permission denied, open '/Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c'
npm ERR!  { Error: EACCES: permission denied, open '/Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c'
npm ERR!   cause:
npm ERR!    { Error: EACCES: permission denied, open '/Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c'
npm ERR!      errno: -13,
npm ERR!      code: 'EACCES',
npm ERR!      syscall: 'open',
npm ERR!      path: '/Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c' },
npm ERR!   isOperational: true,
npm ERR!   stack: 'Error: EACCES: permission denied, open '/Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'open',
npm ERR!   path: '/Users/mryerse001/.npm/_cacache/index-v5/86/df/7ce2a78c93b9a9f6c045f6060e8bb12a701c4af1f7b82cfd3f532a39e74c',
npm ERR!   parent: '@babel/types' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      /Users/mryerse001/.npm/_logs/2019-03-09T11_14_12_485Z-debug.log
Aborting installation.
npm install --save --save-exact --loglevel error react react-dom react-scripts has failed.
Deleting generated file... package.json
Deleting foo/ from /Users/mryerse001/Documents/GitHub
Done.

I am facing a similar issue.
Shalinidc$ npx create-react-app my-app
Unhandled rejection Error: EACCES: permission denied, open '/Users/Shalinidc/.npm/_cacache/tmp/d725d873'
npm ERR! cb() never called!
npm ERR! This is an error with npm itself. Please report this error at:
npm ERR!     https://npm.community
npm ERR!      /Users/Shalinidc/.npm/_logs/2019-03-12T06_12_48_215Z-debug.log
Install for create-react-app@latest failed with code 1
I even did a sudo chown
Shalinis-MacBook-Pro:projects Shalinidc$ sudo chown -R $Shalinidc:$Shalinidc '/Users/Shalinidc/.npm/'
Shalinis-MacBook-Pro:projects Shalinidc$ npx create-react-app my-app
Unhandled rejection Error: EACCES: permission denied, open '/Users/Shalinidc/.npm/_cacache/tmp/42ff481c'
npm ERR! cb() never called!
npm ERR! This is an error with npm itself. Please report this error at:
npm ERR!     https://npm.community
npm ERR!      /Users/Shalinidc/.npm/_logs/2019-03-12T06_16_26_810Z-debug.log
Install for create-react-app@latest failed with code 1
Any leads would be much appreciated.

These errors are the result of permission/ownership problems of ~/.npm. Somewhere along the line you probably ran npm or installed Node using sudo. Make sure all the files and the directory itself are owned by your user, not root. If you're still stuck maybe try asking on Stack Overflow as this problem isn't specific to Create React App.


PHPUnit argument passed after artisan test --parallel command do not work

Laravel Version: 8.27.0
PHP Version: 7.4
Database Driver & Version: Postgres 13

Description:
Command php artisan test --parallel --do-not-cache-result fails to run with error message

The "--do-not-cache-result" option does not exist.

but according to the docs at https://laravel.com/docs/8.x/testing#running-tests

"Any arguments that can be passed to the phpunit command may also be passed to the Artisan test command"

it should work just fine.
The command itself works fine without the --parallel parameter, but I want to run it in parallel so...
The --do-not-cache-result command is documented here:
https://phpunit.readthedocs.io/en/9.5/textui.html?highlight=--do-not-cache-result #command-line-options
Steps To Reproduce:
Run php artisan test --parallel --do-not-cache-result command

That's a current limitation of the underlying dependency Paratest. Just added a note on docs about that: https://github.com/laravel/docs/pull/6836.


capv + cpi: add user agents to vcenter api calls

Issue #, if available:
Description of changes:
While trying to debug vsphere connections in CI we are seeing lots of requests with the generic go-client user agent.  The csi-driver and gov properly set a useragent.  This patches capv and cpi to set a user agent on the vm25client as well.  As a note, capv is setting the useragent on some other session in their code, but that does not seem to come thru and it is after the login call.
Since we have patch directories for the specific versions of cpi, i went ahead and hardcoded the version in the source. I think longer term we could change that to be more dynamic.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/retest

/retest

/retest


Fix cc sign in

Description
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

We just need to close this. It is a DRAFT and the code climate is working. Will not interfere with what we're going to write


Simplify fetching logic

Remove retry-logic that was too complicated. We will no longer cache negative responses at all.
Increase expire time from 7 to 30 days.
Don't block on LiveSatisfaction initialization

Just to confirm: we have opted out from having an immediate retry mechanism. Instead we will poll for flags in background but also try to get fresh ones anytime when we can without having a "max tries"

For feature/flag fetching, there was never an immediate retry actually. It would only try on page loads. If there was multiple failed loads within 60s it would eventually give up and cache a "negative" result and would then use that for 60s. This was super confusing and the code was error prone. There's also no background refresh yet. I figured we'd go straight to #165 instead.


Validation of identity token dosen't validates its expiration date

Hi,
I want to validate the identity token at client level.
So I used to following link:
https://github.com/IdentityServer/IdentityServer4.Samples/blob/release/Clients/src/MvcManual/Controllers/HomeController.cs
I found out that it dosent validate the expiration date of the token.
I tried to add the TokenValidationParameters object some flags, but no one of them validated the expiration date of the token (such as ValidateLifetime).
Am I missing something?
Thanks,
Efi.

Am I missing something?

Perhaps the pre-configured 5-minute clock skew of the JwtSecurityTokenHandler.

Yes.
That's why.
I have another question and I hope you will help me.
I configured my client's TokenValidationParameters and set the ClockSkew to 0.
But still when I call the UserInfoEndpoint it still wait 5 minutes before it validates the expiration date.
This is how i configured it:
options.TokenValidationParameters = new TokenValidationParameters
{
    NameClaimType = "name",
    RoleClaimType = "role",
    ValidateIssuer = true,
    ValidateLifetime = true,
    ValidateIssuerSigningKey = true,
    ClockSkew = new TimeSpan(0, 0, 0)
};

Thank you,
Efi.

We don't do general Q&A on the issue tracker -- only bug reports or issues.
Please use one of the our free or commercial support options for Q&A
See here for more details.
Thanks!


LCFS - Make username not be case sensitive when signing up a new IDIR user

Describe the task
When signing up A. Ring for a new IDIR user account for the LCFS portal, he was not able to sign-in when his IDIR username was all lowercase letters. When his username was changed to all capital letters, he reported he was able to sign-in. Can we make the IDIR username entry field not case sensitive?
Purpose
The value this adds is less login errors for users or initial mapping discrepancies when setting new users up. IDIR usernames can be listed both in all lowercase letters as well as all capital letters when logging in to many systems that require IDIR login. Having a case sensitive restriction on this field when adding users may cause confusion both for the person adding the user and the person being added to the LCFS portal.
Acceptance Criteria

[ ] as an IDIR user, when logging in with any combination of case letters, the user should be able to log in

Notes

identify where the case check happens ( if any )
could possibly be on the authentication side, not the lcfs platform side.

@Grulin, thank you, Lindsy, for reporting the issue. Unfortunately, I couldn't reproduce it. I used both lowercase and capital letters when signing in and also tested different combinations of case letters in the "Edit user" to reproduce the issue but wasn't able to do so. As mentioned in the description, could this issue possibly be on the authentication side, rather than the LCFS platform side?

Hi @hamed-valiollahi. Thanks for looking in to this. I'm not sure if it was an authentication thing. When Al reported the login issue, the only thing I changed between him not being able to login and being able to login was his IDIR user name capitalization. I suppose we could try changing his username back to lower-case and see if he is unable to sign-in then? I'll connect with him.

@Grulin, thanks for your comment, Lindsy.
Yes, it's very weird. That's great! Please let me know if the issue still persists; I'll look into it again and will ask Alex to help me if I can't resolve it. Thank you!

Hi @hamed-valiollahi. I did another test with an IDIR user to see if the case sensitive user name issue was still happening. Unfortunately, it is. If I make the username all lower case and the user tries to login in with their user name all upper case letters, they are restricted from logging in. If I make the username all uppercase and the user tries to login with their username in all lower case letters, they are restricted from logging in. The username for logging in should not be case sensitive. Let me know if you have any questions. Thanks!

@Grulin Hi Lindsy, Thank you for checking again! I appreciate it. I couldn't reproduce the exact issue but found and fixed a related problem with the email address. I also made the username check case insensitive, which should fix the issue. If it doesn't, please let me know. Thank you!

Hi @hamed-valiollahi! This seems to have fixed it now. Thank you for the changes!

@Grulin Hi Lindsy, Great! I'm happy to hear that the issue is resolved. Thank you for testing!


Compiler error in connection with LovyanGFX.

Hi,
I found compiling errors in connection with LovyanGFX.
Do you have any idea/solution for this?
Many thanks in advance!
[{
"resource": "/c:/Users/tzs/Desktop/GitHub/WT32-SC01-PLUS/WT32-SC01-PLUS/.pio/libdeps/wt32-sc01-plus/LovyanGFX/src/lgfx/v1/platforms/esp32/Bus_SPI.cpp",
"owner": "cpp",
"severity": 8,
"message": "'gpio_matrix_out' was not declared in this scope; did you mean 'gpio_iomux_out'?",
"startLineNumber": 181,
"startColumn": 5,
"endLineNumber": 181,
"endColumn": 5
}]
[{
"resource": "/c:/Users/tzs/Desktop/GitHub/WT32-SC01-PLUS/WT32-SC01-PLUS/.pio/libdeps/wt32-sc01-plus/LovyanGFX/src/lgfx/v1/platforms/esp32/Light_PWM.cpp",
"owner": "cpp",
"severity": 8,
"message": "'ledcSetup' was not declared in this scope",
"startLineNumber": 40,
"startColumn": 5,
"endLineNumber": 40,
"endColumn": 5
},{
"resource": "/c:/Users/tzs/Desktop/GitHub/WT32-SC01-PLUS/WT32-SC01-PLUS/.pio/libdeps/wt32-sc01-plus/LovyanGFX/src/lgfx/v1/platforms/esp32/Light_PWM.cpp",
"owner": "cpp",
"severity": 8,
"message": "'ledcAttachPin' was not declared in this scope; did you mean 'ledcAttach'?",
"startLineNumber": 41,
"startColumn": 5,
"endLineNumber": 41,
"endColumn": 5
}]

Hello! 😄
Thank you for your feedback! 🙇
This is a strange error log.
Have you tried re-download the LovyanGFX library?

Hello!
Thanks for the info, full fresh install solved it!


Implement & expose godot_is_instance_id_valid() to GDnative

Godot version: Godot 3.1, custom build from git 6697fd9a05859914fc320b4b4321fd9fb6170f71 (Apr 17th)
OS/device including version: Gentoo Linux 4.19.27, KDE Plasma 5, GTX 1080 w/NVidia proprietary drivers 418.43
Issue description:
When referencing another scene node in GDnative, currently the only safe way to do that is by NodePath.
That requires 1. a string path lookup in the scene tree followed by 2. a cast_to to the actual node type in order to obtain the node. This often needs to be repeated each physics and/or visual frame.
This could be greatly improved if we could check for the validity of an instance_id in GDnative. It's already possible to obtain the instance_id, so we could cache the looked-up and already-cast node pointer together with its instance id and just check whether it's still valid (for the price of one has table lookup instead).
/// <summary>Controls the behavior of an actor within the scene</summary>
public: void set_kinematic_body_node(godot::KinematicBody *kinematic_body) {
    this->kinematic_body = kinematic_body;
    if(kinematic_body != nullptr) {
        this->kinematic_body_instance_id = kinematic_body->get_instance_id();
    }
}

public: godot::KinematicBody *get_kinematic_body_node() const {
    if(this->kinematic_body != nullptr) {
        if(!godot::is_instance_id_valid(this->kinematic_body_instance_id) {
            this->kinematic_body = nullptr;
        }
    }

    return this->kinematic_body;
}

While I'm at it, I'd also suggest killing the ObjectDB::instance_checks hash table which is currently used by ObjectDB::instance_validate() to check the validity of a godot Object * - it's redundant because there's already the instances hash table and risky because a deleted Object can well be reported a valid once another Object is allocated to the same address.
Switching internal checks to use the instance id instead would avoid those issues.

For a more in-depth explanation, also see this issue on the godot-cpp repository: https://github.com/GodotNativeTools/godot-cpp/issues/281#issuecomment-487317637
Overall, the method is easily implemented and would be a low-hanging fruit.


ARROW-15911: [C++] Find correct GTest on macOS with Xcode

12605

 Quality gate: Failed
7 Issues

 7 Code issues

Details

 Quality gate: Passed
1 Issues

 1 Code issues

Details


Documentation: Add Password Requirements to Common Failures

Describe the bug
When accidentally using a password that doesn't match complexity requirements, the component will fail inexplicably from the view of greengrass.log. This error can only be seen by checking the component-specific logs.
To Reproduce
Use a password that does not meet the complexity requirements and not see the true failure reason in greengrass.log.
Expected behavior
A common failure explanation in the documentation to cover this scenario.
Actual behavior
I had to dig through tons of logs to figure out what happened.
Environment
N/A as this is a documentation request.

Hi,
Thank you for using this component. Our README.md documentation specifies the password requirements as shown below:

Note that your password must be at least 16 characters long and must include uppercase and lowercase letters, numbers, and special characters

Please feel free to re-open the issue if you see that the password requirements in the doc are different from that of the component's.


Remember to add tensorflow in the requirement.txt

I tried tensorflow 1.14 and it works. You may add that in your requirement.txt, which can benefit anyone else who is interested in your code:)

@QuXinghuaNTU  What version of deepspeech did you use may I ask? I am having trouble getting it to work

@QuXinghuaNTU What version of deepspeech did you use may I ask? I am having trouble getting it to work

Think Rtaori has added the tensorflow version in requirements.txt. You can try again.


influx freezes and can't be written to

System info:
Influx 1.6.0, Raspberry Pi 3
Processor: BCM2835
Distribution: Raspbian GNU/Linux 9 (stretch)
Kernel version: Linux 4.14.52-v7+ armv7l
Firmware: #1123
Steps to reproduce:
I run Influx together with telegraf and home-assistant to log sensor data. At irregular intervals all three log messages that writing to Influx fails, e.g.
Jul 24 18:35:05 Zorin telegraf[3634]: 2018-07-24T16:35:05Z E! [outputs.influxdb]: when writing to [http://<IP_ADDRESS>:8086]: Post http://<IP_ADDRESS>:8086/write?db=telegraf: net/http: request canceled (Client.Timeout exceeded while awaiting headers)
Jul 24 18:35:05 Zorin telegraf[3634]: 2018-07-24T16:35:05Z E! Error writing to output [influxdb]: could not write any address
Jul 24 18:35:35 Zorin telegraf[3634]: 2018-07-24T16:35:35Z E! [outputs.influxdb]: when writing to [http://<IP_ADDRESS>:8086]: Post http://<IP_ADDRESS>:8086/write?db=telegraf: net/http: request canceled (Client.Timeout exceeded while awaiting headers)
Jul 24 18:35:35 Zorin telegraf[3634]: 2018-07-24T16:35:35Z E! Error writing to output [influxdb]: could not write any address
Jul 24 18:35:41 Zorin influxd[6013]: ts=2018-07-24T16:35:41.425230Z lvl=info msg="failed to store statistics" log_id=09VHdFFW000 service=monitor error=timeout
Jul 24 18:35:45 Zorin telegraf[3634]: 2018-07-24T16:35:45Z E! [outputs.influxdb]: when writing to [http://<IP_ADDRESS>:8086]: Post http://<IP_ADDRESS>:8086/write?db=telegraf: net/http: request canceled (Client.Timeout exceeded while awaiting headers)
Jul 24 18:35:45 Zorin telegraf[3634]: 2018-07-24T16:35:45Z E! Error writing to output [influxdb]: could not write any address
Jul 24 18:35:46 Zorin hass[3709]: 2018-07-24 18:35:46 ERROR (InfluxDB) [homeassistant.components.influxdb] Write error
Jul 24 18:35:46 Zorin hass[3709]: Traceback (most recent call last):
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/connectionpool.py", line 387, in _make_request
Jul 24 18:35:46 Zorin hass[3709]:     six.raise_from(e, None)
Jul 24 18:35:46 Zorin hass[3709]:   File "<string>", line 2, in raise_from
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/connectionpool.py", line 383, in _make_request
Jul 24 18:35:46 Zorin hass[3709]:     httplib_response = conn.getresponse()
Jul 24 18:35:46 Zorin hass[3709]:   File "/usr/lib/python3.5/http/client.py", line 1198, in getresponse
Jul 24 18:35:46 Zorin hass[3709]:     response.begin()
Jul 24 18:35:46 Zorin hass[3709]:   File "/usr/lib/python3.5/http/client.py", line 297, in begin
Jul 24 18:35:46 Zorin hass[3709]:     version, status, reason = self._read_status()
Jul 24 18:35:46 Zorin hass[3709]:   File "/usr/lib/python3.5/http/client.py", line 258, in _read_status
Jul 24 18:35:46 Zorin hass[3709]:     line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
Jul 24 18:35:46 Zorin hass[3709]:   File "/usr/lib/python3.5/socket.py", line 576, in readinto
Jul 24 18:35:46 Zorin hass[3709]:     return self._sock.recv_into(b)
Jul 24 18:35:46 Zorin hass[3709]: socket.timeout: timed out
Jul 24 18:35:46 Zorin hass[3709]: During handling of the above exception, another exception occurred:
Jul 24 18:35:46 Zorin hass[3709]: Traceback (most recent call last):
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/requests/adapters.py", line 445, in send
Jul 24 18:35:46 Zorin hass[3709]:     timeout=timeout
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/connectionpool.py", line 639, in urlopen
Jul 24 18:35:46 Zorin hass[3709]:     _stacktrace=sys.exc_info()[2])
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/util/retry.py", line 357, in increment
Jul 24 18:35:46 Zorin hass[3709]:     raise six.reraise(type(error), error, _stacktrace)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/packages/six.py", line 686, in reraise
Jul 24 18:35:46 Zorin hass[3709]:     raise value
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/connectionpool.py", line 601, in urlopen
Jul 24 18:35:46 Zorin hass[3709]:     chunked=chunked)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/connectionpool.py", line 389, in _make_request
Jul 24 18:35:46 Zorin hass[3709]:     self._raise_timeout(err=e, url=url, timeout_value=read_timeout)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/urllib3/connectionpool.py", line 309, in _raise_timeout
Jul 24 18:35:46 Zorin hass[3709]:     raise ReadTimeoutError(self, url, "Read timed out. (read timeout=%s)" % timeout_value)
Jul 24 18:35:46 Zorin hass[3709]: urllib3.exceptions.ReadTimeoutError: HTTPConnectionPool(host='localhost', port=8086): Read timed out. (read timeout=5)
Jul 24 18:35:46 Zorin hass[3709]: During handling of the above exception, another exception occurred:
Jul 24 18:35:46 Zorin hass[3709]: Traceback (most recent call last):
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/influxdb.py", line 316, in write_to_influxdb
Jul 24 18:35:46 Zorin hass[3709]:     self.influx.write_points(json)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/influxdb/client.py", line 468, in write_points
Jul 24 18:35:46 Zorin hass[3709]:     tags=tags, protocol=protocol)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/influxdb/client.py", line 532, in _write_points
Jul 24 18:35:46 Zorin hass[3709]:     protocol=protocol
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/influxdb/client.py", line 312, in write
Jul 24 18:35:46 Zorin hass[3709]:     headers=headers
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/influxdb/client.py", line 252, in request
Jul 24 18:35:46 Zorin hass[3709]:     timeout=self._timeout
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/requests/sessions.py", line 512, in request
Jul 24 18:35:46 Zorin hass[3709]:     resp = self.send(prep, **send_kwargs)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/requests/sessions.py", line 622, in send
Jul 24 18:35:46 Zorin hass[3709]:     r = adapter.send(request, **kwargs)
Jul 24 18:35:46 Zorin hass[3709]:   File "/srv/homeassistant/lib/python3.5/site-packages/requests/adapters.py", line 526, in send
Jul 24 18:35:46 Zorin hass[3709]:     raise ReadTimeout(e, request=request)
Jul 24 18:35:46 Zorin hass[3709]: requests.exceptions.ReadTimeout: HTTPConnectionPool(host='localhost', port=8086): Read timed out. (read timeout=5)
Jul 24 18:35:47 Zorin hass[3709]: 2018-07-24 18:35:47 ERROR (InfluxDB) [homeassistant.components.influxdb] Resumed, lost 2 events

The problem is not caused by home-assistant, telegraf or the Influx monitor service: When either are disabled the others still throw messages. So I presume Influx locks at random times.
For a test I moved the data directory to a USB-disk, but I still received the same messages.
Help wanted.
iostat.txt
profiles.tar.gz
vars.txt

@ElFishi we would need a stack dump if Influx has deadlocked. When influx freezes can you send a SIGQUIT signal to the influx process and please provide the stacktrace?

Thanks for picking this up.
I'd love to comply but I'm afraid I need a hand with this.
I presume influx freezes from the messages I see logged. However, this happens only intermittently. Most of the time there is no problem.
Can you devise a script that polls influx and  sends the SIGQUIT and records the stacktrace once influx locks? I don't have such skills.
cheers

I'm seeing the same behaviour on a Raspberry Pi 3, running only InfluxDb and Chronograph/Telegraph. I'm logging data to it via an external script running on a separate PC and it starts throwing errors. It happens after around 48 hours of uptime.
What's interesting is that the Pi is not locked up initially. I usually have an SSH connection open to it at all times, and I can start typing a new command. However, pressing enter on the commandline will hangup the Pi. I will attempt sending syslog messages via remote syslog and see what pops up.
My gut feeling is that influx triggers an Out-of-memory, though.

I found if the influxd.exe is just starting(the logging is showing on the screen)  , in this time , If you try to insert data , influxd.exe will  freeze.  My influx version is V1.5 .
I guess the influxd is analysing the data at start time .  I suggest the server no to provide service before data prepare complete.

closing due to age. please upgrade to the latest influxdb version and open a new issue if the problem persists.

Hi I have the latest influxDB and do see the exact same error:
influxd[1029]: ts=2019-11-11T02:35:50.026000Z lvl=info msg="failed to store statistics" log_id=0J1ywLal000 service=monitor error=timeout
After a while then influxdb hangs the whole system.
I run debian in a VMM on my NAS and my gut feeling is also that influxdb uses all memory and then the whole system hangs. root is then in read-only. If I then login root fs is not full but I have to restart the system. It happens always after a few days.
Searching the logs for hours, I believe it starts during backup (disk busy?) and then after 30-40 of such messages the system dies. The disk is locally and should deliver > 100mb/s
I am on debian 9.11 and influxdb  OSS 1.7.9


Openstack images - adding serverId extra

Images in Openstack may have a server dictionary with an ID, just like in the Rackspace API.  This patch adds a serverId extra to Openstack images.

Kami merged this with https://github.com/apache/libcloud/commit/edcb55e2771a804f0d6f93a7e4365c7fd6475877

Kami merged this with https://github.com/apache/libcloud/commit/edcb55e2771a804f0d6f93a7e4365c7fd6475877


Adds .travis.yml file

Makes Travis build Java instead of Ruby.
Helps with #21

In the Gradle projects we have:
script:
    - ./gradlew check --refresh-dependencies --no-daemon

How does this "empty" config understand that it must call ./mvnw package ?
Thanks

It's calling ./mvnw test by default. And it also calls install before to install dependencies.
https://docs.travis-ci.com/user/languages/java/#Default-script-Command
Do we want it to do anything differently?

On other projects we do something like this, but I think the default is fine for now.
script:
  # execute regular local build
  - mvn clean verify -B -V -q

Why did you merge this?
I didn't approve.
I really would like to see Checkstyle results in the Travis as well.
Thanks

Sorry @artembilan I thought we could get any changes merged to .travis.yml as we needed them.
./mvn test runs checkstyle, it will fail if there are any issues.

Yeah... Confirmed:
D:\SpringIO\spring-cloud\spring-cloud-gcp>mvnw test
[INFO] Scanning for projects...
...
[ERROR] src\main\java\org\springframework\cloud\gcp\pubsub\support\SubscriberFactory.java:[29,54] (whitespace) WhitespaceAfter: ',' is not followed by whitespace.

I was under impression that we couldn't manage to work Checkstyle in some other project only via just test goal. We have to run package there...
Cool! Thank your for the patience! And sorry for extra noise.


circom-lsp client: couldn't create connection to server.

Hi @vuvoth, when i installed the extension and opened a circom file, it complained with the following error message, do you know what it means?
[Error - 下午3:46:29] circom-lsp client: couldn't create connection to server.
Error: spawn Unknown system error -8
    at ChildProcess.spawn (node:internal/child_process:413:11)
    at Object.spawn (node:child_process:806:9)
    at /Users/johnson/.vscode/extensions/vuvoth.circom-plus-0.0.6/out/main.js:39:57554
    at Gl.createConnection (/Users/johnson/.vscode/extensions/vuvoth.circom-plus-0.0.6/out/main.js:39:12709)
    at Gl.start (/Users/johnson/.vscode/extensions/vuvoth.circom-plus-0.0.6/out/main.js:39:2940)

Thank you so much for feedback!
I'm not sure I know exactly what happened here. Look like the circom lsp binary not compatible with macOS(I build it on my linux machine). Sorry for inconvenient.
You can install the extension manually follow those commands:
git clone https://github.com/vuvoth/ccls
cd ccls
cargo xtask install --server
cargo xtask install --client

Please let me know if you have any problem.

@vuvoth Hey, thanks. I followed your commands and installed the extension, but the error still occurs.
Yes, I'm using macOS. This is the full output.
[Error - 下午8:20:25] circom-lsp client: couldn't create connection to server.
Launching server using command /Users/johnson/.vscode/extensions/vuvoth.circom-plus-0.0.6/bin/ccls failed. Error: spawn /Users/johnson/.vscode/extensions/vuvoth.circom-plus-0.0.6/bin/ccls ENOENT

I updated repo. Can you pull new code and install extenstion follow command:
git clone https://github.com/vuvoth/ccls
cd ccls
cargo xtask install --server
cargo xtask install --client

@vuvoth Nice! It works perfectly. Thanks for your help, this project is very helpful!

Thank you @chnejohnson <3 ! Some functions are not working perfectly right now. I'm try to fix this. If you have any problem please give me a feedback.

@vuvoth In fact, I have a small wish. It would be great if I could go to the definition by clicking the library include path :D

@vuvoth In fact, I have a small wish. It would be great if I could go to the definition by clicking the library include path :D

I create this feature :D Please update a new package, @chnejohnson!

@vuvoth hi, there's an error when I run cargo xtask install --server after pulling latest codebase

What is your rust version? Can you update to the lastest rust version?

I'm using rustc 1.74.0 (79e9716c9 2023-11-13) (Homebrew), I'm updating now, it'll take some time.

I will close this issues! If you have any idea please create another one!


Northern Ireland code "XI" is missing from getPrefixIntracomVAT() method

Northern Ireland code "XI" is missing from getPrefixIntracomVAT() method

duplicate of #5


DL: Split madlib_keras dev-check file

Prior to this PR, the dev-check test file for deep_learning module
was too big and there was no way of running a subset of the tests for
e.g. predict or evaluate.
This commit separates test/madlib_keras.sql_in test file into:

Setup files: for data creation used in the tests
Specific test files: for testing fit, predict, evaluate and transfer
learning

Additionally, madpack.py is updated to ignore any setup
files(*.setup.sql_in) in the test dir and not run it as a test in
dev-check/install-check.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/1004/

The changes LGTM. I had one concern, it's more of a trade-off though.
I think the main intent of this PR is to make the tests manageable, and this PR surely does that well. But the total time to run the same tests now increases significantly:
Time to run all the tests together in master:
TEST CASE RESULT|Module: deep_learning|madlib_keras.sql_in|PASS|Time: 35243 milliseconds

Time to run the same tests now split across 4 files in this PR branch:
TEST CASE RESULT|Module: deep_learning|madlib_keras_evaluate.sql_in|PASS|Time: 6623 milliseconds
TEST CASE RESULT|Module: deep_learning|madlib_keras_fit.sql_in|PASS|Time: 22986 milliseconds
TEST CASE RESULT|Module: deep_learning|madlib_keras_predict.sql_in|PASS|Time: 17447 milliseconds
TEST CASE RESULT|Module: deep_learning|madlib_keras_transfer_learning.sql_in|PASS|Time: 8327 milliseconds

It is ~55 seconds now against ~35 seconds on master. It might be ok, and we should probably do this only for modules that have large dev-check files (such as deep learning).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/1014/


C# extension not detecting

Issue Type: Bug
The c# extension is not getting recognised. I have tried reinstalling.
Extension version: 1.23.12
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8500 CPU @ 3.00GHz (6 x 3000)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.80GB (1.88GB free)

Process Argv
--crash-reporter-id 6649777c-de81-455a-bcfc-f53a41972a38

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
vspre833cf:30267465
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158cf:30286554
vscgsv2:30307504
vscorehov:30309549
bridgeflight:30302069
vscod805cf:30301675

Excuse me are u getting this error cause I am getting this error
[Running] scriptcs "f:\RayyanPrograms\c++\test.cs"
'scriptcs' is not recognized as an internal or external command,
operable program or batch file.
[Done] exited with code=1 in 0.116 seconds
i even opened an issue here https://github.com/OmniSharp/omnisharp-vscode/issues/4585

Excuse me are u getting this error cause I am getting this error
[Running] scriptcs "f:\RayyanPrograms\c++\test.cs"
'scriptcs' is not recognized as an internal or external command,
operable program or batch file.
[Done] exited with code=1 in 0.116 seconds
i even opened an issue here https://github.com/OmniSharp/omnisharp-vscode/issues/4585

Nope. Its runs alright until I add a specific command like time.deltatime
I moved to visual studio 2019
I would suggest you the same:)

I got the same message. The issue was not from the omnisharp-vscode extension but from the vscode-runner extension which uses the scriptcs executable to run C# code. I solved the issue by installing scriptcs
via chocolatey from an administration console (under Windows as the operating system):
cinst scriptcs


Is a list of acceptable language codes available anywhere?

I saw this issue on language requests, where people are requesting new languages.
But I google'd and I couldn't find a list of language codes that Argos currently accepts.
Is there a documentation anywhere that lists the languages that the latest Argos accepts, or that the various releases of Argos accept?

https://libretranslate.com/languages or https://www.argosopentech.com/argospm/index/

Great thank you :)


Ability to disable horizontal scroll to change month

1.3.12
For now, by default we have horizontal scroll physics to change month, but on some device it's  slowly. And I want to disable this scroll behaviour. Is there some features to make it?

make it smoother is also the solution, too 👍

How could I do that?

Thanks for the issue. I'll definitely come back with this one. Currently, it isn't supported.


ENOENT

I am trying to install the typings from one of my custom modules and keep receiving the following:

typings.json from the module that is giving me trouble looks like this:

The problem seems to be related to the request module, specifically, if I remove request from the package.json list of dependencies I am able to install the typings.
Does anyone have any ideas why the request module is giving me this error?
Thanks in advance!

@Beng89 Unfortunately this is an issue with using ambient dependencies. Ambient is considered a global module and not installed from sub-dependencies (that's how you get conflicts). As a result, request is never installed and it's trying to find the dependency from package.json. If you want dependencies to work, we'll need to retype that as an external module.

@blakeembrey Thank you for the info.
I imagine that others are going to have this issue when they upgrade from tsd to typings (since most of the tsd definitions are currently ambient). One possible solution would be to cause typings to emit a warning (rather than failing) when an ambient sub-dependency isn't found. That way users could install the definitions for the module (then manually install the ambient definitions).

That's possible, but I think if I install the typings for an NPM package I expect it all to work. I realise it's a bit of work today because these typings don't yet exist, but please create issues for the typings you need and we can make them together :smile:

Fair enough :)


support/b2cqa 1187 update swap tests

📝 Description
Resurrect Swap tests to work with new swap layout and providers with the following test cases (all mocked data and device actions):

Changelly Floating rate max swap with change of target account

For future:

Add test to go to DEX and verify the page loads
Drop a lot of the mocked responses in the frontend code and use Playwrights mocked API responses depending on the provider

❓ Context

Impacted projects: ``
Linked resource(s): ``

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo
https://user-images.githubusercontent.com/87659842/212331415-f24eef7c-dd22-4686-9e5c-0b1e7a6d38cb.mov
🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots


add makefile command to run test server more easily

storybook requires the test server to run and doing these steps manually multiple times is a pain
instead adding a make script to easily get the test server up & running

Will close - not a huge problem for now to use as is.


Standardize isPostOnly

a.k.a. MakerOnly or BookOrCancel or PendingOrCancelled
most exchanges support this, but vary in the parameters needed to specify this

@brigb123 and @yatendra, note starting with v0.9.0, IsPostOnly can be set though ExchangeOrderRequest. If it is left null, then it will revert to the exchange default, likely false for Coinbase.