Fix/23768

Details

The values of the country/state pickers would remain cleared after reopening the picker:
https://github.com/Expensify/App/assets/138504087/e4a67198-a195-412a-b8f5-73f179a03998
Please note: the state picker is broken in Storybook due to an issue with HeaderWithBackButton that I have reported here. This issue exists on staging. As suggested by Jack Nam I still marked the Storybook checkbox as completed, since that issue is not caused by this PR.
Fixed Issues

$ https://github.com/Expensify/App/issues/23768
PROPOSAL: https://github.com/Expensify/App/issues/23768#issuecomment-1655803335
Tests

Testing the country selector

Go to settings > profile > personal details > home address
Click on country to open the country selector
Pick the United States
Repeat step 2 to reopen the country selector
Clear the country input field
Close the country selector by clicking at the top of the RHP ('< Country')
Repeat step 2 to reopen the country selector
Notice that the United States is selected by default

Testing the state selector

Repeat step 6 to close the country selector
Click on state to open the state selector
Select any state
Repeat step 10 to reopen the state selector
Clear the state input field
Close the state selector by clicking at the top of the RHP ('< State')
Repeat step 10 to reopen the state selector
Notice that the state you selected is selected by default

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

Offline tests

Same as regular steps
QA Steps

Same as regular steps

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

PR Author Checklist

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

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

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

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

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

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

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

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] 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)
[X] 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)
[X] All JSX used for rendering exists in the render method
[X] 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

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

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

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

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

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Looking through FORMS.md, I don't think there's anything that we need to update due to this PR

@joh42 Please adjust the title of the PR to reflect the change. Also please remove the first video showing the bug it may be confusing.

Please use the tests described below: (do same thing with state test)
It's better to avoid the "repeat step X" pattern as it's a little harder to read and reproduce

Go to settings > profile > personal details > home address
Click on country to open the country selector
Pick the United States
Reopen the country selector
Clear the input field and close the selector
Reopen the country selector
Verify that the United States is selected by default

👍 updated testing steps @s77rt

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] 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

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] 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)
[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 checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

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

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

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

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

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] 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)
[X] 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)
[X] All JSX used for rendering exists in the render method
[X] 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

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

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

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

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

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/16493223/896a34e0-88e4-48e2-8f53-20ef9f3036bf

Mobile Web - Chrome

https://github.com/Expensify/App/assets/16493223/8de8accb-f1c2-46e3-aa8e-d77466ad743c

Mobile Web - Safari

https://github.com/Expensify/App/assets/16493223/a2ff8ac7-eac1-4541-9045-94338dd7cc45

Desktop

https://github.com/Expensify/App/assets/16493223/eb468ec2-4b41-4dfa-9592-dab9e32a508e

iOS

https://github.com/Expensify/App/assets/16493223/bb04b1c5-cb48-4e1c-b75a-270bdb26f411

Android

https://github.com/Expensify/App/assets/16493223/23a5f20b-de5e-4239-815b-bce9be0cfbda

Please double check the state selector test steps

@joh42 Thank you!

@MariaHCD ⚡

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

🚀 Deployed to staging by https://github.com/MariaHCD in version: 1.3.50-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.50-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/MariaHCD in version: 1.3.51-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 1.3.51-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


The text "Type code here and press shift-enter to run" does not disappear when shuffling through previous commands

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

Open vscode and the interactive window
Run any command (i.e a=2)

Actual behavior

Press up-arrow to get the previuos command
Note that the command "a=2" is written across the text "Type code here and press shift-enter to run" making it difficult to read the command. The text disappears if you start deleting (backspace) the previous command

Expected behavior
Expected the text "Type code here and press shift-enter to run" disappears when shuffling through early commands. I

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running: Local
Extension version: 20YY.MM.#####-xxx
VS Code version: 1.45.0
Setting python.jediEnabled: No changes, so deppend on the default
Python and/or Anaconda version: Python 3.7.7 (64 bit)
OS: Windows | Mac | Linux (distro): Windows 10 home
Virtual environment: NA

Developer Tools Console Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Hi @Jakobhenningjensen,
I am not able to repro this at this time. Could you please provide a screenshot of the issue you are dealing with?

Hi @Jakobhenningjensen ,
After some more playing around I was actually able to repro this.
Thanks for your feedback, we will take a look!

Great! FYI, I think the issue has been there for more than a year (I just haven't create a ticket). But yes, it is exactly like the picture

Repro steps:

Run interactive window
Type something and run it
Type something again and delete with backspace
Hit up error

Validated, the watermark is gone now in the input history.


data visualization

I'm trying to run the visualizer, however, it is failing because a few files are empty. The data dumped is
pid_2024_07_17_1403__layout__tr_0__ep_01__ts_104.png
pid_2024_07_17_1405__layout__tr_0__ep_01__ts_104.png
plan_df.csv
res_summary_df.csv
res_df.csv

the empty files are 'plan_df.csv', 'res_summary_df.csv' and 'res_df.csv' - maybe I haven't run it long enough.
Also, what are in the 2 pngs, obviously it shows a floorplan, but is that a computed floorplan or is that an initial condition?
Thanks!

Hi, you do not need the visualizer, it is my helper functions to visualize my results. If you want to use it, you need to make sure you have a trained agent.


chore(openapi): update OpenAPI specification file

This pull request was auto-generated
This PR adds the latest version of OpenAPI specification file.
Actions:
1. Review changes on OpenAPI file and validate if it s correct
2. Perform manual changes if needed or apply local patches for the OpenAPI files.
3. Merge this pull request only when the API has been released.

CC @craicoverflow

CC @craicoverflow

This has been released but is not yet in production.


Array update order error

Version
3.2.25
Reproduction link
github.com
Steps to reproduce
npm ci.
npm run dev.
open.chrome console.
click change image list.
What is expected?
/src/images/1.jpg
/src/images/2.jpg
/src/images/3.jpg
/src/images/4.png
/src/images/5.jpg
/src/images/6.png
/src/images/7.jpg
/src/images/8.jpg
What is actually happening?
/src/images/1.jpg
/src/images/2.jpg
/src/images/8.jpg
/src/images/7.jpg
/src/images/6.png
/src/images/5.jpg
/src/images/4.png
/src/images/3.jpg

This is expected behavior. The order in which child components are created during v-for loops is not guaranteed and depends on the kind of change that is being processed.

This seems to be the default behavior for Vue2, is there an alternative in Vue3？


Selector nesting in mixins

How do you handle descendants in mixins?
For example, this mixin creates media objects (image on one side, text on the other): https://github.com/Keats/snowflake/blob/master/components/_media-object.scss and needs to set some values on the nested items in addition to itself.
How would you do that without nesting the 2 selectors in the normal case and the --reverse case?
EDIT: I should mention I use libsass and there's no at-root that I could use I guess

I think this is a fairly specific case where nesting makes sense. You should not be concerned that much with this no nesting rule.

Yeah that's what I thought.
Thanks !


Changed the PR title to reflect this is a breaking change.

@johnnyaug  @itaiad200 I fixed the linting issue, not sure about the failing tests though, it seems to be from lakefs/pkg/gateway. If you could point me to what might be the issue I can add a fix.

It appears that Github Actions will only run it for the first time for external forks PR, when an approval was granted. Source.
Merging it as the same tests passed on the first try, thanks @mishraprafful !


[ZEPPELIN-3580] Adds rename note method to rest api

What is this PR for?
Adds rename note method to rest api
What type of PR is it?
Improvement
What is the Jira issue?
issue on Jira: https://issues.apache.org/jira/browse/ZEPPELIN-3580
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

LGTM

LGTM

@oxygen311 Could you please resolve conflicts? I recommend you not to change the unrelated code to avoid merge conflicts. After resolving, I'll merge it

@jongyoul Done


Reconsider migration guides placement

From the CNCF Tech Docs assessment of our docs (cncf/techdocs#89):

The Migration section isn't more important than Toolkit components, Toolkit dev guides, or the Flux CLI reference. Placing it above these in the information architecture doesn't make sense.

Recommendations

Move the Migration Guide to either a subsection of Get Started or a subsection of Installation

Consider renaming "Migration guide" to "Migrate from Flux v1" for clarity

@dholbach is this open to contributions? If yes, I'd like to raise a PR regarding it.

This issue is linked with #717

I am interested in this issue

Having migration under installation would confuse most Flux users, as this about migrating from a deprecated version (v1) and not about migrating between v2 releases. We'll probably have a migration section, for v2 APIs GA, alpha to stable controllers and so on.


Issue importing teams custom app

Hello,
We're having issues importing an app on Teams. If we export the zip and immediately re-import, it works (and validates). If we export the zip, unzip it on a Mac or Windows computer, then re-zip and re-import, it doesn't work. One guy has been able to at least do that by unzipping/zipping with WinZip.
We are trying to add "applicationPermissions":["User.Read", "profile", "openid", "email"] for a conversational AI bot.
Paul

@paulcloudmsg - Could you please share what error you are getting after re-import?

This is just after unzipping, zipping, and re-importing.

@paulcloudmsg - Could you please share from where you are uploading the app in Teams like via sideloading, from developer portal or Teams admin center?

@paulcloudmsg - Have you checked this?

@paulcloudmsg - Could you please confirm is your issue resolved or are you still facing this issue?

Yes, sorry. We were zipping the folder that contained the manifest and
images instead of selecting those three files and zipping. So the files
were buried in a folder and the dev portal was not expecting that.
Thanks.
On Thu, Aug 25, 2022 at 3:04 AM Nivedipa-MSFT @.***>
wrote:

@paulcloudmsg https://github.com/paulcloudmsg - Could you please
confirm is your issue resolved or are you still facing this issue?
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/msteams-docs/issues/6668#issuecomment-1226920049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2PSMACX5RCFS23QLCFCODLV24SHNANCNFSM56JW5Q6A
.
You are receiving this because you were mentioned.Message ID:
@.***>

@paulcloudmsg - We are closing this issue for now. Please feel free to reopen if required.


"null" data values erroneously passes schema check

What steps did you take:
With these files...
#@data/values-schema
---
gotta_have_this_key: true

#@data/values
---

#@ load("@ytt:data", "data")
---
data.values: #@ data.values

.. ran this ...
$ ytt -f .

What happened:
data.values: null

What did you expect:
I expected to see a schema error message, like...
ytt: Error: Overlaying data values (in following order: values.yml):
One or more data values were invalid
====================================

values.yml:
    |
  2 | ---
    |

    = found: null
    = expected: map (by schema.yml:2)

Anything else you would like to add:

Fixing this bug would be a breaking change: it's turns what is currently a "successful" result and error outcome.
This behavior is being used as part of a workaround for situations like #563.

For this reason, this fix ought to be fixed after implementing #418.

Environment:

ytt version (use ytt --version): v0.38.0

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Fixing this bug takes a behavior that was succeeding and now causes an error. This is technically a breaking change.
That said, relying on this behavior has the effect of creating a false positive. Breaks that occur from this fix will reveal potential issues that had not yet surfaced (or created enough damage to be noticed).


Work In progress: Misc config loading fixes

The main motivation behind this is simplicity and correctnes, but the new small config library is also faster:
BenchmarkDefaultConfigProvider/Viper-16         	  252418	      4546 ns/op	    2720 B/op	      30 allocs/op
BenchmarkDefaultConfigProvider/Custom-16        	  450756	      2651 ns/op	    1008 B/op	       6 allocs/op

Fixes #8633
Fixes #8618
Fixes #8630
Updates #8591
Closes #6680
Closes #5192

This is the default setup after this PR:
build:
  _merge: none
caches:
  _merge: none
frontmatter:
  _merge: none
imaging:
  _merge: none
languages:
  _merge: none
  en:
    _merge: none
    menus:
      _merge: shallow
    params:
      _merge: deep
markup:
  _merge: none
mediatypes:
  _merge: shallow
menus:
  _merge: shallow
minify:
  _merge: none
module:
  _merge: none
outputformats:
  _merge: shallow
permalinks:
  _merge: none
related:
  _merge: none
sitemap:
  _merge: none
taxonomies:
  _merge: none


Markdown links inside comments shouldn't be resolved

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
When using HTML comments inside markdown files, and these comments contain markdown links, our system still tries to resolve those links.
See also https://github.com/facebook/docusaurus/pull/6159

[WARNING] Docs markdown link couldn't be resolved: (./api/lifecycle-apis.md) in /Users/sebastienlorber/Desktop/projects/docusaurus/website/versioned_docs/version-2.0.0-beta.13/using-themes.md for version 2.0.0-beta.13

Steps to reproduce

# MyDoc

<!--

[bad markdown link](./does-not-exist.md)

-->

Expected behavior
no warning
Actual behavior
produces a warning
Your environment
latest
Reproducible demo
No response
Self-service

[ ] I'd be willing to fix this bug myself.

Slightly related to https://github.com/facebook/docusaurus/issues/6108

Duplicate of https://github.com/facebook/docusaurus/issues/5659 😄

And that's why I started working on #5670

😆 I was pretty sure this already came up


Async Daemon Tracker.WaitForShardState() should throw if shard name does not exist

Would have helped out a certain Marten contributor just now

Too much work for what it's worth right now.


Admission-webhook crashes after node restart

From time to time we see issues after node reboot - admission-webhook stays in CrashLoopBack, the logs complain that the mutatingwebhookconfiguration is already there.  Removing the existing configuration solves the problem:
WH=$(kubectl get pods -l app=admission-webhook-k8s -n nsm-system --template '{{range .items}}{{.metadata.name}}{{"\n"}}{{end}}')
kubectl delete mutatingwebhookconfiguration ${WH}

{
  "level":"fatal",
  "ts":1646820191.1554844,
  "caller":"build/main.go:328",
  "msg":"mutatingwebhookconfigurations.admissionregistration.k8s.io \"admission-webhook-k8s-7b9d7f98c8-5xgln\" already exists",
  "stacktrace":"main.main\n\t/build/main.go:328\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:225"
}

Probable reason:
It is related to thing that admission webhook is using self signed cert and when admission webhook restarts it generates a new cert that is not eual to cert from the registered k8s configuraiton

Should be fixed by https://github.com/networkservicemesh/cmd-admission-webhook-k8s/pull/98


Launch.json argument definitions ?

Where can I find what the argument definitions are when used within launch.json ?

Use inteli-sense inside that file to get suggestions or simply read our docs https://code.visualstudio.com/docs/editor/debugging#_launch-configurations


Add a Dockerfile for image building container

Contains:

packer
python 3.7
Openstack client
jq

Added Ansible, make commands and pushed a version to harbor


[FEAT] Add a 404 not found page [GSSoC'23]

Add a custom 404 not found page
Can you assign this to me, I want to work on this under GSSoC'23

What is need of 404 not found?

Nowadays, every modern website should have a custom 404 page to improve user experience and guide users when they encounter broken links or non-existent pages.

@jayk-gupta I want to contribute.Please assign me

Important Announcement: Closure of GitHub Repository Issues, Website Redesign, and Discord Channel Invitation
We apologize for the inconvenience caused, but we must inform you that all current issues in our repository will be closed. This decision aims to improve our development process and enhance organization.
Excitingly, we are announcing a complete redesign of our website, focused on improving user experience and incorporating new features aligned with our goals.
Please note that approximately half of the existing issues are redundant and will be closed accordingly.
We also invite you to join our Discord channel for project collaboration and communication. Visit GSSoC server and assign yourself Web Resource Hub project role using the self-roles feature.
Join here
Your feedback and support are invaluable to us. We look forward to hearing your thoughts on the upcoming changes.
Thank you for your understanding and continued support.
Sincerely,
Project Admin


Clean up Tuya Power Config

Proposed change

Currently TuyaPowerConfigurationCluster2AA and TuyaPowerConfigurationCluster3AA clusters inherit from TuyaPowerConfigurationCluster which creates a battery bus. The other TuyaPowerConfiguration clusters are built off of TuyaLocalCluster. This causes confusion when building a quirk via TuyaQuirkBuilder since it will fail if either of the two legacy power clusters are used.
This PR moves the power config clusters to tuya.power_config, adds 2AA and 3AA, and marks the two legacy 2AA and 3AA as Legacy.
A follow-on PR should convert the ts0601_trv quirk to use the newer style, remove the legacy clusters, and remove the import and re-export of the 2AAA and 4AA clusters from tuya/init.py.
Additional information

Checklist

[x] The changes are tested and work correctly
[x] pre-commit checks pass / the code has been formatted using Black
[ ] Tests have been added to verify that the new code works

Per above discussion, closing this.


doc: point to bblfsh documentation to be able to use correctly uast_xpath(blob, xpath) function

This was addressed in #543


Network Traffic Graph Option B

An alternative to https://github.com/bitcoin-core/gui-qml/pull/286 which only introduces one graph. Based on this design.

dark
light

nACK. Christoph's [comment] (https://github.com/bitcoin-core/gui-qml/pull/286#issuecomment-1469818032) sums it well.


[extractor/glomex] Add new extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
This patch adds support for Glomex videos, both embeddable and on the (demo) site video.glomex.com. Glomex operates a media exchange platform, which powers their video marketplace service, based on revenue sharing. This is used quite a lot in large parts of Europe, primarily by digital news outlets, which embed in their assets video clips provided by various content providers (for example TV channels), tied with advertising.

Add new IEs

GlomexBaseIE: Base IE class
GlomexIE: Extract videos from video.glomex.com (by deferring to glomex:embed)

The integrationId parameter is apparently common across video.glomex.com, but we have to hard-code it as it is very hard to extract otherwise: it is deeply hidden in a script asset (a webpack component, which is typically regularly updated)

GlomexEmbedIE: Extract Glomex videos by matching the player URL

Query the API to extract metadata, detect video formats and get the respective (JWT protected) stream/source URLs
The API query may return one or more videos: the latter case is treated as a playlist

As this is otherwise identically handled, a separate IE was not deemed necessary
However title and description fields are not set for playlist results

They do not exist in the parent object; obtaining them from the first entry is not indicative for the playlist content
As the playlist order is not always stable (this is true at least for related videos playlists), it makes writing test cases impossible

Let GenericIE detect embeds by matching all three integration methods:

HTML: glomex-player tag or data attributes
Javascript: naive parsing of inline scripts for string constants assigned to integration parameters
Iframe: src attribute GlomexEmbedIE._VALID_URL

Let GlomexIE and the former embed detection pass the origin URL to GlomexEmbedIE by smuggling it in the player URL, as this is an expected parameter in API requests
Add test cases for both single videos and two playlist flavors

This patch was ported from https://github.com/ytdl-org/youtube-dl/pull/30212 with the following changes/improvements:

Parse HLS subtitles

Is there anything else we can do to move this forward?

OK


Container in docker swarm throws panic: runtime error: invalid memory address or nil pointer dereference

This error is thrown when I run the postgres_exporter container as a service inside docker swarm
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x6fbe6a]

goroutine 18 [running]:
main.(*Exporter).scrape(0xc4201080c0, 0xc42010c1e0)
	/home/travis/gopath/src/github.com/wrouesnel/postgres_exporter/postgres_exporter.go:956 +0x25a
main.(*Exporter).Collect(0xc4201080c0, 0xc42010c1e0)
	/home/travis/gopath/src/github.com/wrouesnel/postgres_exporter/postgres_exporter.go:744 +0x39
main.(*Exporter).Describe(0xc4201080c0, 0xc42010c180)
	/home/travis/gopath/src/github.com/wrouesnel/postgres_exporter/postgres_exporter.go:737 +0xb0
github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus.(*registry).Register.func1(0x9459c0, 0xc4201080c0, 0xc42010c180)
	/home/travis/gopath/src/github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus/registry.go:218 +0x3b
created by github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus.(*registry).Register
	/home/travis/gopath/src/github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus/registry.go:220 +0xbc

Observations

It works well when I run the container as standalone container in same server
The DATA_SOURCE_NAME is set properly. I've verified this by getting inside container (I had to create docker image with alpine linux for debugging this)

/ # env
DATA_SOURCE_NAME="postgresql://postgres_exporter:maskedpassword@<IP_ADDRESS>:5432/postgres"

I've verified that psql client can connect to postgres using same connection string

apk --update add postgresql-client
/ # psql "postgresql://postgres_exporter:maskedpassword@<IP_ADDRESS>:5432/postgres"
psql (9.6.3, server 9.5.7)
SSL connection (protocol: TLSv1.2, cipher: ECDHE-RSA-AES256-GCM-SHA384, bits: 256, compression: off)
Type "help" for help.
postgres=#

Dockerfile used for creating image for debugging

FROM alpine:3.6

ARG POSTGRES_EXPORTER_VERSION=v0.2.1

RUN apk --no-cache add openssl

RUN wget -O /usr/local/bin/postgres_exporter https://github.com/wrouesnel/postgres_exporter/releases/download/$POSTGRES_EXPORTER_VERSION/postgres_exporter && \
    chmod +x /usr/local/bin/postgres_exporter

EXPOSE 9187

ENTRYPOINT ["/usr/local/bin/postgres_exporter"]

Should the condition be pErr == nil instead of pErr != nil in https://github.com/wrouesnel/postgres_exporter/blob/v0.2.1/postgres_exporter.go#L955?

Hm looks like a definite logic bug. I'll get that fixed.

Thanks @wrouesnel. It would be good to have a release with that fix. It will help debugging container failures

I'll get one released later tonight. Thanks for the report!

Okay I don't know how you've been tripping it, but bizarrely this code has actually been broken all along and the bug you identified stops it from crashing normally. The URL parse library almost always succeeds even with broken URLs, so we just never entered that section.
Interestingly I can't find a test case where the url.Parse() function actually fails running locally so I'm not sure why it's crashing for you.

I'm also not sure about the url parse error. I'm hoping that fixing the logic mentioned above would log the real error why DB connection is failing. Logging the url.Parse error would also be useful to understand why it fails

I was trying to fix that logic alone and build a binary which can be used in container for debugging my issue. I tried building the binary on mac osx, couldn't get it working
Running make gets stuck with these errors ->
➜  postgres_exporter git:(master) make
/Library/Developer/CommandLineTools/usr/bin/make -C tools
find: illegal option -- n
usage: find [-H | -L | -P] [-EXdsx] [-f path] path ... [expression]
       find [-H | -L | -P] [-EXdsx] -f path [path ...] [expression]
make[1]: Nothing to be done for `DEFAULT'.
gometalinter --disable-all --enable=gofmt --vendor
cat: /proc/cpuinfo: No such file or directory
cat: /proc/cpuinfo: No such file or directory
Using 0 processes
gometalinter -j        0 --deadline=30s --disable=gotype --disable=gocyclo

Unfortunately I don't develop of OS X so my makefiles tend not to be compatible across platforms. I have a PR up containing the fixes, I'll cut it to a release ASAP - #94.

Thanks for the release @wrouesnel. Getting the error logged now
monitor_postgres_exporter.1.bbsq2w5nqohe@swarmm-agentpublic-18950373000008    | time="2017-08-04T03:20:02Z" level=info msg="Error opening connection to database (could not parse DATA_SOURCE_NAME): missing \"=\" after \"\\\"postgresql://postgres_exporter:xxxxxxx@<IP_ADDRESS>:5432/postgres\\\"\" in connection info string\"" source="postgres_exporter.go:977"

The above error went away after adding ?sslmode=disable in connection string. I'm getting a new error now
monitor_postgres_exporter.1.g8atsb6uzvyw@swarmm-agentpublic-18950373000009    | time="2017-08-04T04:13:08Z" level=info msg="Error opening connection to database (could not parse DATA_SOURCE_NAME): pq: Could not detect default username. Please provide one explicitly." source="postgres_exporter.go:977"

It was issue with quotes around connection string in docker compose file. Found the answer here https://stackoverflow.com/a/37764191/69362. I had to change it to
environment:
    - DATA_SOURCE_NAME=postgresql://postgres_exporter:xxxxxxx@<IP_ADDRESS>:5432/postgres?sslmode=disable

earlier it was
environment:
    - DATA_SOURCE_NAME="postgresql://postgres_exporter:xxxxxxx@<IP_ADDRESS>:5432/postgres?sslmode=disable"

Thanks @wrouesnel for your timely help. It would be good to add this in README file for others trying to use this in compose file / docker swarm. Please let me know if you need help


pip in the container?

I know it's not essential but it would be useful to have pip in the fusion-neutronics-workflow container :-)

My bad, pip is in the container.... silly me. For some reason VS code cannot instal black! closing this


added tern to the OSPO Landscape

On @caniszczyk's suggestion, I added Tern to the OSPO Landscape.
cc: @nishakm and @rnjudge

Build failed because of:
Tern: Only svg logos are supported
Empty twitter for Apple (Adopter): https://twitter.com/apple
Item Tern has no image_data

do you a TERN logo in SVG format :)?
On Tue, May 18, 2021 at 10:08 AM CNCF AutoUpdater @.***>
wrote:

Build failed because of:
Tern: Only svg logos are supported
Empty twitter for Apple (Adopter): https://twitter.com/apple
Item Tern has no image_data
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/todogroup/ospolandscape/pull/52#issuecomment-843252197,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSIN4P2IXLGLL2JE4AJLTOJ7FRANCNFSM45CW4UEA
.

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org

btw @geekygirldawn is Tern an LF project or a VMWare thing?

btw @geekygirldawn is Tern an LF project or a VMWare thing?

Tern is part of the LF Automated Compliance Tooling (ACT) project


Update faq-defender-for-apis.yml

Added clarifications for mult-region deployments and APIM versions

@AlizaBernstein
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

#sign-off


Add router.removeRoute() - again

It's a problem similar to #129
I have used  @jdanyow solution but now I see that after the logout, where I call
setRoot('login');

router is not reset. It is still possible to paste the url of existing subpages and activate them (activate() is invoked, backend requests are sent, page title changes, only view doesn't change and shows login form).
What is your recommendation to handle this simple scenario with user relogin?
Should I reset the router manually and reregister routes available for user roles after successful login or use navigationStrategy for every route instead of setRoot methods as @bryanrsmith  suggests in #129 ?

I'd try resetting the router and re-registering the appropriate, role-based routes.


5/27 PM Publish

Docs Build status updates of commit 0b639a7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-6.0.0/index.yml
:white_check_mark:Succeeded
View (azps-6.0.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


gulp --production

Hi !
Postcss failed if i use gulp --production with  lost
@include max-width(tablet-portrait) {
	lost-masonry-wrap: no-flex 0px;
}

i've got this error
PostCSS - Error
   - Cannot read property 'join' of null

Maybe he can't parse css minified ?
Anyone as an idea ?
Thanks !

I fixed it with
  lost-masonry-wrap: no-flex 0.0000001px;

Maybe cssnano remove 0px ? and my lost postcss task failed after that...
Anyway thanks ;)


ENH: Faster dwtn and idwtn

Much faster implementations for dwtn and idwtn.  This was split out of #52 as it is independent of the waverecn or wavedecn functionaility, but obviosly makes those routines much faster as well.
The key modification here is refactoring the functions downcoef and upcoef to do the looping on the cython side rather than the prior implementation that used numpy's apply_along_axis to do the looping on the python side in dwtn and idwtn.
To quote myself from the prior pull request:
From testing on my machine on a 256x256 image, dwtn was taking approximately 70 ms prior to these changes and only about 1ms afterwards. This is comparable in performance to the Rice Wavelet Toolbox (https://github.com/ricedsp/rwt), but without the limitation to only 1D or 2D signals.

The following sequence gives an error:
$ python
Python 2.7.10 (default, Jul  1 2015, 10:54:53)
[GCC 4.9.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy, pywt
>>> pywt.dwtn(numpy.ones((256, 256), 'float32'), 'haar')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pywt/multidim.py", line 239, in dwtn
    axis=axis)
  File "_pywt.pyx", line 1183, in _pywt.downcoef (pywt/src/_pywt.c:20894)
  File "_pywt.pyx", line 1211, in _pywt._downcoef_lastaxis (pywt/src/_pywt.c:21963)
ValueError: Buffer dtype mismatch, expected 'float32_t' but got 'double'

I have opened a PR with a test case exposing this on your repository.

There seems to be a lot of shuffling data around to make the axis the transform is applied on contiguous. I would be happy to add a stride parameter to convolution.c on top of #59, so this is not necessary. That might simplify the implementation and reduce the number of copies necessary which should improve performance. If you are interested I will open a branch for this feature.

@kwohlfahrt : I will have to take a look in a bit, but I am open to the idea of the stride parameter in covolution.c.  Would there not be some performance penalty to strided array access vs. having the elements contiguous?  I guess it still may be faster than having to do the reshaping, etc. that is going on currently.  As long as the performance is reasonably similar it would be worth doing in any case if it simplifies the code overall.

Putting a reminder to myself here to try to fix the _upcoef_lastaxis routine for cases with level > 1.  If I get that working I can remove the old implementation of _upcoef that is stil hanging around.

Unfortunately I had to rebase off of master and then fast-forward to fix the merge conflicts so travis would run the tests again.  hopefully this was OK.

That's always OK, nothing unfortunate about it.

@grlee77 I've done a quick test implementation of a stride parameter. With a stride == 1 it is about 35-75% slower depending on the sizes of my input and filter on my Intel i7-4790K, and no change on my AMD Phemom II X4 B55.
It's a naive implementation (i.e. all input[idx] is translated to input[(idx) * stride]), but I have not been able to improve on it's performance by incrementing my loop counters directly by the stride or things like that. It would be possible to define this access with a macro and just #include the relevant function body with the correct define, like so:
#ifdef USE_STRIDE
#define INPUT(idx) input[idx]
#else
#define INPUT(idx) input[(idx) * stride]
#endif

So we can keep the performance for the 1D case and have the flexibility of the stride for n-dimensional transforms without much code duplication.

@kwohlfahrt   great! If you submit a pull request to my repository or point me towards your branch with the changes I can make a modified version of dwtn / idwtn that use this version.  I don't have an intuitive feel for whether there will be a net speed improvement for the 2D and 3D cases I most often use, but am happy to test it out.  Certainly it would avoid having to keep shuffling the axes orders so the one to be filtered is along the last axis as is being done here now.

Sorry, quick update to point out my previous testing results were incorrect. The AMD processor gives similar results to the Intel. The penalty appears largest for small input sizes.
I've currently only implemented it for downsampling_convolution_periodization, I'll expand it over the next few days to the remaining functions and open a PR. The work is based on my templating branch, which has quite aggressive changes to the C codebase, so it might delay merging. I'll make sure it applies cleanly to rgommers/master though.

I've done a bit more profiling, and it looks like a strided convolution is no good. It takes a roughly 50% performance penalty just for introducing the parameter, and compared to malloc'ing a temporary array is at best 10% faster for a filter length of 2, equal for a filter length of 4-6, and roughly 2x slower for a filter length of 16 once the stride gets above a few hundred elements (which is common for 3D or higher transforms).

@kwohlfahrt  Thanks for looking into the strided access performance.  It's unfortunate that it was a substantial performance penalty.
Please let me know if there is anything in particular that you think would improve the clarity of the implementation in this pull request.  I have a local branch where I converted _downcoef_lastaxis and _upcoef_lastaxis to cdef functions instead, but it ended up requiring some cumbersome additional dtype checks, etc., and I didn't think it was a net benefit.

I think I've worked something out, though it's incomplete at the moment. Have a look at my branch ndim and let me know what you think. The relevant functions are _pywt.dwtn and wt.c/TYPE_downcoef_axis. For changes, diff to my templating branch, that one is just cleanups to make the C easier to work with.
General outline is:

downcoef_axis applies downsampling_convolution along a given axis of an ndarray in C

This can probably be translated to Cython, but I don't get along very well with it so somebody else might want to have a go

dwt_axis is a wrapper around this
dwtn uses dwt_axis instead of numpy.apply_along_axis

Clocks in at about 6ms for a haar wavelet with a 256x256 image on my i5 laptop, but could probably have a lot done regarding nogil and such, but I don't know much about this.
Todo:

support axes with negative strides
anything else?

@kwohlfahrt
Great work!  I like the approach you have taken in this ndim branch .  I see that you are able to check the strides in downcoef_axis and avoid creating copies if it is unnecessary.
There was a serious bug with a wrong variable name in the new _pywt.dwtn function.  I submitted a PR to your ndim branch for that.
After the bug fix mentioned above, it is approximately 40% faster than the implementation in this PR for a 256x256 2D case and about 50% faster for a 256x256x256 3D cases I tested on my desktop!

Oh wow. Can't believe I missed that. Merged. And I imagine there will be quite a lot of boilerplate of this type - we'll probably want to apply the other transforms along axes too. All of the 1D transforms should have the same interface, so it might even be possible to use function pointers in C. Anyway I think I'll punt this issue till after we get around to implementing another transform, most likely swt.
There are a few things I'll need to check about strides. I know they can be negative, which I currently do not handle. Can they be non-integer multiples of itemsize? Can I have a float64 array with a stride of 12 for example?

I think handling negative strides would be necessary so that something like the following would work properly (should probably add a test case).  coeffs = dwtn(x[-1::-1, :], 'haar').  I am not sure about non-integer multiples of itemsize, but can't think of a scenario off-hand.

Yes, exactly. It needs to be done. I've managed to find a scenario unfortunately, so weird strides need handling too:
>>> import numpy
>>> dt = numpy.dtype([('', 'float64'), ('', 'byte')])
>>> a = numpy.zeros(5, dt)
>>> a
array([(0.0, 0), (0.0, 0), (0.0, 0), (0.0, 0), (0.0, 0)],
      dtype=[('f0', '<f8'), ('f1', 'i1')])
>>> a['f0']
array([ 0.,  0.,  0.,  0.,  0.])
>>> a['f0'].strides
(9,)

I'm not sure I follow you on the above example.  We do not need to handle those more exotic dtypes because we only process either 32 or 64-bit floats in these routines.

a['f0'] is an array of float64. There is just one byte of padding between each element of the array.

okay, that makes sense.

A faster implementation is now available in #104, so do not merge this one.  I will most likely close it soon.


bug: rm ut failed

What happened:

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

have fixed


Issues 17 18

Add config_rule_wait_for_compliance_results to simplify checking multiple resource_ids
Add config_rule_wait_for_absent_resources to allow checking for resources being removed from config.
Update timeout to 45 (15 minutes)

Fixes #17 and #18


fix: during bootstrapping, remember which security classes were not granted

fixes: #4630

@zwave-js-bot pack this


SMS Forwarding

When a SMS is being forward and you reply to the msg to start the conversation it says "Sorry, I did not understand your message.

If I try Betty: Call +Number I see the following error in the console

[30/May/2015:23:00:39 +0000] "POST /twiml/sms HTTP/1.1" 400 80 "-" "TwilioProxy/1.1"
Error: Error: Not Found
at app.use.res.status.send.error (/root/test/betty/lib/index.js:47:13)
at Layer.handle [as handle_request] (/root/test/betty/node_modules/express/lib/router/layer.js:82:5)
at trim_prefix (/root/test/betty/node_modules/express/lib/router/index.js:302:13)
at /root/test/betty/node_modules/express/lib/router/index.js:270:7
at Function.proto.process_params (/root/test/betty/node_modules/express/lib/router/index.js:321:12)
at next (/root/test/betty/node_modules/express/lib/router/index.js:261:10)
at jsonParser (/root/test/betty/node_modules/body-parser/lib/types/json.js:89:44)
at Layer.handle [as handle_request] (/root/test/betty/node_modules/express/lib/router/layer.js:82:5)
at trim_prefix (/root/test/betty/node_modules/express/lib/router/index.js:302:13)
at /root/test/betty/node_modules/express/lib/router/index.js:270:7


adding Nepal Disaster file to coordination section

Althio - please have a read through. I'm investigating how we could add a link from the front page of the site to this file. Feel free to made amendments - you're grammar is better than mine.

I am happy with that, see my minor proposals for amendments.
As we publish, I propose we ask feedback/review from other working groups (Activation / Communication).

Altho,
Thanks for that.  I agree.
If we can it would be good to get a link on the front page but I can't do
it from my phone and I'm on that until at least late this evening.
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 12:19, "althio"<EMAIL_ADDRESS>wrote:

I am happy with that, see my minor proposals for amendments.
As we publish, I propose we ask feedback/review from other working groups
(Activation / Communication).
—
Reply to this email directly or view it on GitHub
https://github.com/hotosm/learnosm/pull/367#issuecomment-101631010.

Will you modify by new commits directly in this pull request?

If we can it would be good to get a link on the front page

I propose (quick and dirty...) to modify _posts/en/0100-01-01-index.md from:
interspan: "More guides (currently in English)"
interspanlink: "https://github.com/hotosm/learnosm/wiki/English-Learning-Guides"

to:
interspan: "Nepal Disaster"
interspanlink: "http://learnosm.org/en/coordination/nepal-disaster/"

Althio,
Yes,  that would be brilliant.
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 14:32, "althio"<EMAIL_ADDRESS>wrote:

Will you modify by new commits directly in this pull request?
If we can it would be good to get a link on the front page
I propose (quick and dirty...) to modify _posts/en/0100-01-01-index.md
from:
interspan: "More guides (currently in English)"
interspanlink: "https://github.com/hotosm/learnosm/wiki/English-Learning-Guides"
to:
interspan: "Nepal Disaster"
interspanlink: "http://learnosm.org/en/coordination/nepal-disaster/"
—
Reply to this email directly or view it on GitHub
https://github.com/hotosm/learnosm/pull/367#issuecomment-101664535.

Althio,
Sorry need to clarify.  Can you do it,  if not it may take a day or so
before I get the chance.
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 14:38, "Nick Allen"<EMAIL_ADDRESS>wrote:

Althio,
Yes,  that would be brilliant.
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 14:32, "althio"<EMAIL_ADDRESS>wrote:

Will you modify by new commits directly in this pull request?
If we can it would be good to get a link on the front page
I propose (quick and dirty...) to modify _posts/en/0100-01-01-index.md
from:
interspan: "More guides (currently in English)"
interspanlink: "https://github.com/hotosm/learnosm/wiki/English-Learning-Guides"
to:
interspan: "Nepal Disaster"
interspanlink: "http://learnosm.org/en/coordination/nepal-disaster/"
—
Reply to this email directly or view it on GitHub
https://github.com/hotosm/learnosm/pull/367#issuecomment-101664535.

The interspanlink may need the mention of github.com
I'm not sure,  perhaps try it and modify if needed
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 14:39, "Nick Allen"<EMAIL_ADDRESS>wrote:

Althio,
Sorry need to clarify.  Can you do it,  if not it may take a day or so
before I get the chance.
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 14:38, "Nick Allen"<EMAIL_ADDRESS>wrote:

Althio,
Yes,  that would be brilliant.
Nick
(OSM=Tallguy)
dodgy didgits as using a phone with a spell chequer.
Humanitarian OpenStreetMap Team Member
On 13 May 2015 14:32, "althio"<EMAIL_ADDRESS>wrote:

Will you modify by new commits directly in this pull request?
If we can it would be good to get a link on the front page
I propose (quick and dirty...) to modify _posts/en/0100-01-01-index.md
from:
interspan: "More guides (currently in English)"
interspanlink: "https://github.com/hotosm/learnosm/wiki/English-Learning-Guides"
to:
interspan: "Nepal Disaster"
interspanlink: "http://learnosm.org/en/coordination/nepal-disaster/"
—
Reply to this email directly or view it on GitHub
https://github.com/hotosm/learnosm/pull/367#issuecomment-101664535.

Sorry need to clarify.  Can you do it,  if not it may take a day or so before I get the chance

Amendments to your text: Can I leave to you? Possibly with new commits on this current pull request.
Modification of the front page: I am a bit reluctant because I don't have access to a local repository for testing purpose right now.
Sorry for delaying this.

Just merging Japanese change - think this should work ok now as have tried it out locally. I'd missed one of the files the first time. This will also place the Nepal disaster page in the coordination section

@althio the 3 files that I changed to implement the Japanese change of folder were (on my computer, but you'll understand)
~/HOT/MyLearnOSM/all-translation.json
~/HOT/MyLearnOSM/_config.yml
~/HOT/MyLearnOSM/_layouts/default.html

:+1:


Refactor combined analysis functions

With this pull request the functions used for the combined analysis are refactored by following suggestions of flake8, isort and black. Any reviews are very much appreciated.

With this pull request the functions used for the combined analysis are refactored by following suggestions of flake8, isort and black. Any reviews are very much appreciated.

No problems for me. I think only thing is that in the diff I see that for functions.py it complains that no new line is present at the end of the file. Can you add it? Apart from that, fine for me!

Thanks Alessio, regarding the function.py I checked that there was no new line before refactoring the code but now there is, so it should be no problem.


format volume in ETH not shares, remove unneeded shares denomination …

…file
https://app.clubhouse.io/augur/story/15817/volume-and-open-interest-values-should-be-in-eth
verify that Volume is in ETH instead of Shares,

Create market preview card
Market Trading view
Markets List

Need the newest augur-node master to make sure the volume numbers are correct.

Coverage remained the same at 63.83% when pulling da5c882c9cd9d0c457ae5d5a61b621d688bfd9f0 on epheph/ch15817/volume-and-open-interest-values-should-be into 5d99970cbc9436aa6a472fa92d4640ef33ee972f on master.


Regions enriched for SNVs with >=3 haplotypes file download.

Thank you for providing a great tool. I am trying to download the 'Regions enriched for SNVs with >=3 haplotypes' from the 'region to filter out' section on GitHub. I would appreciate it if you could confirm this for me.

I also have a question about the demo version's results mentioned on the GitHub site, so I wanted to ask.
In the results of 'Extraction of read-level features,' why is there an ID 'test~11~40316580~C~T' that is not present in the phasing results? I appreciate your time and assistance.

Thank you for providing a great tool. I am trying to download the 'Regions enriched for SNVs with >=3 haplotypes' from the 'region to filter out' section on GitHub. I would appreciate it if you could confirm this for me.

Hi @dongjuleem ,
Sorry for the delayed response. You could use the files here: https://github.com/parklab/MosaicForecast/tree/master/resources, the "clustered regions" indicate SNVs with >=3 haplotypes.


Bug with PS600

Dear Julius,
I found a bug with my PS600,
in HA shows two devices like the temperature sensor and binary sensor with the same name
Below in attachment my screenshot :

``

Can you check if binary sensor gets activated if you remove back cover (not the front one which contains battery)?

@jvitkauskas OK, many thanks for your answer
But if you have will anytime, please hide this  entity in homeassistant UI


Relax dependency resolution.

For the resolution of dependencies (especially in the face of TypeScript actually checking) it makes sense to have ambiguous dependency versions set.

Need this to resolve a peer dependency conflict with graphql after updating to Apollo Client 3
Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>

Need this to resolve a peer dependency conflict with graphql after updating to Apollo Client 3
Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>

It wood be good if this could be merged soon. Otherwise it will be hard to use Absinthe with up-to-date JS projects.


Fix dormant snitches still being active & snitches not changing state over restarts

If a snitch changed from active to dormant or dormant to culled during a restart, it would not actually change state. This could result in a snitch remaining active indefinitely, but this would be rather rare.
Separately (and the original issue that prompted this fix), was that dormant snitches would still send notifications. This is because, on server startup, dormant snitches would be re-activated (if they were disabled previously) due to incorrect logic.

Fixes #37

Seems good :)


Fixed  Google Video Search

I didn't even know it had this feature.
I just found that we have tests, ran make test, and it failed.
Then I found out Google Video Search didn't really exist.
Rather than remove it I've limped it along so it works.

Thanks for this change, it allows us to resolve several issues on Google Code.


Activity Graph

Would be good to add in activity graph that updates user of its scale up and scale down behaviour

1 possible behaviour is to mimic hashi ui and display graph of cores scaling.
UI starts pinging for counts only after its being viewed, like traefik and hashi-ui


SimpleEnvelope._DEFAULT_PAYLOAD_TYPE should be public?

ngclient accesses SimpleEnvelope._DEFAULT_PAYLOAD_TYPE.
flake8-self linter complains about this, I think correctly. If we use the value from a different module, it should be public
CC @lukpueh for comment

Hi, I would like to work on it. I will create PR.
Thank you 👍


SpeechSynthesizerIntegrationTest Failed

Hi.
I will use alexa-client-sdk. Now I'm trying to run the SpeechSynthesizerIntegrationTest. But all of the tests  are failed.
Could you help me to fix this error?
Thanks

Logs from SpeechSynthesizerIntegrationTest:
[==========] Running 8 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 8 tests from SpeechSynthesizerTest
[ RUN      ] SpeechSynthesizerTest.handleOneSpeech
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x1272f50,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x1255c90,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x1255c90,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x1255c90,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.handleOneSpeech (11461 ms)
[ RUN      ] SpeechSynthesizerTest.handleMultipleConsecutiveSpeaks
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x1271600,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x1261520,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x1261520,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x1261520,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.handleMultipleConsecutiveSpeaks (11187 ms)
[ RUN      ] SpeechSynthesizerTest.bargeInOnOneSpeech
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x138e930,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x126dc90,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x126dc90,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x126dc90,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.bargeInOnOneSpeech (11221 ms)
[ RUN      ] SpeechSynthesizerTest.bargeInOnMultipleSpeaksAtTheBeginning
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x13b46b0,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x127fcd0,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x127fcd0,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x127fcd0,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.bargeInOnMultipleSpeaksAtTheBeginning (11193 ms)
[ RUN      ] SpeechSynthesizerTest.bargeInOnMultipleSpeaksInTheMiddle
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x13b2f20,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x13a2e30,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x13a2e30,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x13a2e30,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.bargeInOnMultipleSpeaksInTheMiddle (11230 ms)
[ RUN      ] SpeechSynthesizerTest.multiturnScenario
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x13cff20,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x13bf510,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x13bf510,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x13bf510,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.multiturnScenario (11275 ms)
[ RUN      ] SpeechSynthesizerTest.handleNoSpeakDirectives
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x14f9c10,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x1380e80,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x1380e80,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x1380e80,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.handleNoSpeakDirectives (11170 ms)
[ RUN      ] SpeechSynthesizerTest.handleNoDirectives
ConfigurationNode:initializeSuccess
HttpPost:doPostSucceeded:code=200
AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
AuthDelegate:onAuthStateChangeCalled:state=1,error=0
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
Could not setup downchannel, retry count: 1
Could not setup downchannel, retry count: 2
Could not setup downchannel, retry count: 3
/home/jkh/amazon/alexa-client-sdk-master/Integration/test/SpeechSynthesizerIntegrationTest.cpp:466: Failure
Value of: m_connectionStatusObserver->waitFor(ConnectionStatusObserverInterface::Status::CONNECTED)
Actual: false
Expected: true
Connecting timed out.
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechSynthesizer,name=Speak,handler=0x1496a20,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=SetMute,handler=0x141ab40,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=Speaker,name=AdjustVolume,handler=0x141ab40,policy=BLOCKING
DirectiveRouter:addDirectiveHandlers:action=added,namespace=SpeechRecognizer,name=ExpectSpeech,handler=0x141ab40,policy=BLOCKING
MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=DISCONNECTED
DirectiveSequencer:shutdown
[  FAILED  ] SpeechSynthesizerTest.handleNoDirectives (11191 ms)
[----------] 8 tests from SpeechSynthesizerTest (89930 ms total)
[----------] Global test environment tear-down
[==========] 8 tests from 1 test case ran. (89930 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 8 tests, listed below:
[  FAILED  ] SpeechSynthesizerTest.handleOneSpeech
[  FAILED  ] SpeechSynthesizerTest.handleMultipleConsecutiveSpeaks
[  FAILED  ] SpeechSynthesizerTest.bargeInOnOneSpeech
[  FAILED  ] SpeechSynthesizerTest.bargeInOnMultipleSpeaksAtTheBeginning
[  FAILED  ] SpeechSynthesizerTest.bargeInOnMultipleSpeaksInTheMiddle
[  FAILED  ] SpeechSynthesizerTest.multiturnScenario
[  FAILED  ] SpeechSynthesizerTest.handleNoSpeakDirectives
[  FAILED  ] SpeechSynthesizerTest.handleNoDirectives
8 FAILED TESTS

No updates on this for weeks.  If there are still questions, please update the ticket with more details and re-open.


Update Example 1

Example 1 referenced PnP

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

Docs Build status updates of commit 5d53df6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

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

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

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

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

File
Status
Preview URL
Details

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

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


[terra-form-select] Invalid attribute value for aria-activedescendant

Issue Description
Verified using axe on the terra core site, saw the same error just like the below stacktrace. Also, attached screenshots to better describe the issue.
expected no accessibility violations but got:
{
"description": "Ensures all ARIA attributes have valid values",
"help": "ARIA attributes must conform to valid values",
"helpUrl": "https://dequeuniversity.com/rules/axe/3.2/aria-valid-attr-value?application=axeAPI",
"id": "aria-valid-attr-value",
"impact": "critical",
"nodes": [
{
"all": [
{
"data": [
"aria-activedescendant="terra-select-option-null""
],
"id": "aria-valid-attr-value",
"impact": "critical",
"message": "Invalid ARIA attribute value: aria-activedescendant="terra-select-option-null"",
"relatedNodes": []
}
],
"any": [],
"failureSummary": "Fix all of the following:\n  Invalid ARIA attribute value: aria-activedescendant="terra-select-option-null"",
"html": "

health-benefits-management-engine.network-error",
"impact": "critical",
"none": [],
"target": [
"ul"
]
}
],
"tags": [
"cat.aria",
"wcag2a",
"wcag412"
]
}
Additional Context / Screenshots

What version of axe-core does your project use?

3.2.2

I  was unable to re-create this issue on our site using a newer version of axe. In fact, toolkit is being upgraded to use axe version 3.4.0 and above. Here is the issue link for you to keep track.This will be out soon once we fix the issues due to new rules in v3.4.0.
PS: If the wdios are failing in your project due to this rule you can disable it for now till toolkit is released.
cc @vb057454

Can we close this issue @yuderekyu ?

This appears to be an issue on both versions of axe-core.
Here is a branch to reproduce the violation running axe-core 3.2.0.
https://github.com/cerner/terra-core/tree/select-no-results-accessibility-control
Here is the branch upgraded to axe-core 3.4.0 that logs the same violation.
https://github.com/cerner/terra-core/tree/select-no-results-accessibility-axe-3.4.0
Briefly looking into this I think this issue can be resolved by checking for the active option before assigning the aria attribute here: https://github.com/cerner/terra-core/blob/master/packages/terra-form-select/src/multiple/Menu.jsx#L254

Thanks for the heads up @StephenEsser. I have a PR out to fix this.


move DisableWidgets from Specialize to DefaultUser

This prevents the "Some settings are managed by your administrator" banner in the Settings page for Taskbar customization, while still being able to disable the Widgets.
Additionally, this also lets users enable it later by simply flipping the toggle in the Settings instead of digging into the registry stuff.
In short, it flips the "Widgets" toggle in Taskbar settings off instead of locking it behind a policy.

Interestingly enough, I wrote the following because I was running into issues with UCPD, but ended up just doing policy level blocks because its too finicky and Microsoft can flip the table at any point:
`""
Function to display a message and prompt for continue
function Prompt-Continue {
Write-Output "Press enter to continue..."
$null = Read-Host
}
Define the name of the scheduled task to delete
$taskNameToDelete = "UCPD velocity"
$registryKeyToDelete = "HKLM:\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Schedule\TaskCache\Tree\Microsoft\Windows\AppxDeploymentClient\UCPD velocity"
$keyPath = "HKLM:\SYSTEM\CurrentControlSet\Services\UCPD"
$propertyName = "Start"
$propertyValue = 4
Attempt to Delete the scheduled task and report back
$taskToDelete = Get-ScheduledTask -TaskName $taskNameToDelete -ErrorAction SilentlyContinue
if ($taskToDelete -ne $null) {
Unregister-ScheduledTask -TaskName $taskNameToDelete -Confirm:$false
Write-Output "UCPD velocity Scheduled task entry has been deleted."
} else {
Write-Output "UCPD velocity Scheduled task entry was not found."
}
Prompt-Continue
Attempt to Delete the registry key
if (Test-Path $registryKeyToDelete) {
Remove-Item -Path $registryKeyToDelete -Recurse -Force
Write-Output "UCPD velocity task Registry key entry has been deleted."
} else {
Write-Output "UCPD velocity task Registry key entry not found."
}
Prompt-Continue
Write-Output "Now readding the UCPD velocity task in a broken and disabled state..."
Register the XML configuration as a new scheduled task
try {
$xmlConfig = @'

UserChoice\UserChoice
\Microsoft\Windows\AppxDeploymentClient\UCPD velocity
D:P(A;;FA;;;SY)(A;;FA;;;BA)(A;;GRGX;;;SU)

false

S-1-5-18
LeastPrivilege

IgnoreNew
true
true
true
false
true

true
false

true
false
false
false
false
true

P999D
P9999D
false

false
PT1S
9

%windir%\system32\UCPDMgr.exee

'@
Register-ScheduledTask -Xml $xmlConfig -TaskName "UCPD velocity" -TaskPath "\Microsoft\Windows\AppxDeploymentClient" -User "SYSTEM"
Write-Output "Scheduled task created successfully."

} catch {
Write-Output "Failed to create scheduled task: $_"
}
""
Prompt-Continue
Write-Output "Now adding or editing UCPD driver registry start status to disabled state..."
Attempt to edit or add the below registry key value
try {
if (Test-Path -Path $keyPath) {
# If the property exists, set its value
Set-ItemProperty -Path $keyPath -Name $propertyName -Value $propertyValue -Type DWORD
} else {
# If the property doesn't exist, create it
New-ItemProperty -Path $keyPath -Name $propertyName -Value $propertyValue -PropertyType DWORD -Force
}
Write-Output "Registry key entry modified or added successfully."
} catch {
Write-Output "Failed to modify registry key entry: $_"
}
""
Write-Output "Jobs done!"
Write-Output "Well, almost, restart your PC now to finalize!"`

So the bottom line is that there is no straightforward way to set the TaskbarDa registry value, neither under the HKCU key nor under the HKU\DefaultUser key, correct?
I had not heard of UCPD before, but it seems like this could become a problem in the future for any registry operations.


Slow rendering

Hello,
I put the webview in an activity. After App launch, if I open the activity for the first time and use WebView to load some page, the rendering performs very well. But if I close the activity and open it again, the WebView start rendering slow. Could you suggest any idea about it?

I have the same issue

hello.
I think this repo might help you.
if you're still looking for a better webview
https://github.com/shimohq/BisonView
Good Luck


[fix] Introduce intermediate virtual nodes for select concrete nodes

Fix #132
This PR introduces virtual nodes to create a layer of indirection between certain complex nodes and their child lists. This is necessary where nodes have varying amounts of certain types of children, and these children are adjacent in the "flat" representation, as otherwise there may be conflicts across syntactical boundaries.

Codecov Report

Merging #133 into master will increase coverage by 0.19%.
The diff coverage is 86.01%.

@@             Coverage Diff              @@
##             master     #133      +/-   ##
============================================
+ Coverage     80.11%   80.30%   +0.19%
- Complexity      443      466      +23
============================================
  Files            36       36
  Lines          1624     1721      +97
  Branches        243      261      +18
============================================
+ Hits           1301     1382      +81
- Misses          209      218       +9
- Partials        114      121       +7

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/se/kth/spork/base3dm/Pcs.java
64.00% <0.00%> (-9.92%)
9.00 <0.00> (-1.00)

...java/se/kth/spork/spoon/matching/SpoonMapping.java
85.88% <50.00%> (-0.87%)
38.00 <0.00> (+1.00)
:arrow_down:

.../java/se/kth/spork/spoon/wrappers/NodeFactory.java
77.19% <84.05%> (+7.96%)
15.00 <9.00> (+10.00)

...h/spork/spoon/pcsinterpreter/SporkTreeBuilder.java
81.39% <85.71%> (+0.88%)
35.00 <9.00> (+6.00)

...kth/spork/spoon/matching/ClassRepresentatives.java
92.42% <88.23%> (-2.13%)
11.00 <0.00> (+3.00)
:arrow_down:

src/main/java/se/kth/spork/base3dm/ChangeSet.java
94.11% <100.00%> (ø)
29.00 <1.00> (ø)

src/main/java/se/kth/spork/base3dm/ListNode.java
100.00% <100.00%> (+33.33%)
6.00 <1.00> (+2.00)

src/main/java/se/kth/spork/spoon/PcsBuilder.java
100.00% <100.00%> (ø)
9.00 <5.00> (+2.00)

...h/spork/spoon/pcsinterpreter/SpoonTreeBuilder.java
87.50% <100.00%> (+0.09%)
41.00 <0.00> (ø)

... and 2 more

Continue to review full report at Codecov.

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

The PR fails the file merge benchmark as, for some reason, the changes cause previously implicit type parameters to become explicit. This does not seem to be too much of a problem in practice, I don't think it affects anything other than textual accuracy.

I think the deteriorated score is related to virtual nodes that do not have children not getting the leaf child list as described in #116


adding ability to toggle rust syntax checking from command panel Fixes #121

As above, would love a code review @dten @andrii-zymohliad @Ostrovski

Looks good, but I've made small PRs to your fork
@Ostrovski I don't see these changes? Where are they? (i might be being stupid)


Support Centos 8

Which issue(s) this PR fixes:

Additional comments:

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004 and that you give permission to TOSIT,
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

New (maybe better) implementation @leopaul36 @gboutry


Rewrite and unit test PIFBuilder to be more modular

Supports all kinds of products and targets

@aciidb0mb3r This is the new PR for PIFBuilder. I added support for system targets, added more tests, and did the change you requests to create a PIFBuilderParameter struct.

@swift-ci smoke test self hosted

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test


Beautiful binary string problem: C# solution

Added a new solution for beautiful binary string problem using C# programming language.

Hi @RyanFehr
Although the readme mentions O(1) as ideal space complexity for this problem but your java solution is using O(n) space. IMHO, the O(1) space complexity is far more complex and requires lot of if condition checks. O(n) space complexity solution is very straight forward.
I've mentioned both the approaches in the submitted file for the sake of knowledge. The main method contains O(1) space complexity solution to reflect what is mentioned in readme.

Hi @RyanFehr
Is there any more detail that I can provide  from my end to get the pending pull requests approved. Please let me know.


[chore](thrift compatibility) Fix thrift id on master is not compatible with branch-2.1 #40384

cherry pick from #40384

run buildall


AsyncDriver doesn't prune graph based on overrides

Current behavior
Standard Driver prunes the DAG to only execute nodes that are not overridden using overrides. The AsyncDriver does not seem to do this, resulting in nodes getting run that shouldn't.
I believe it's just a missed arg where overrides should be passed in to self.graph.get_upstream_nodes(final_vars, inputs) here: https://github.com/DAGWorks-Inc/hamilton/blob/main/hamilton/async_driver.py#L290
That seems to be present in the standard driver: https://github.com/DAGWorks-Inc/hamilton/blob/main/hamilton/driver.py#L645
Stack Traces
(If applicable)
Screenshots
(If applicable)
Steps to replicate behavior

Build AsyncDriver
Provide overrides on driver.execute
Overridden nodes still execute

Library & System Information
Python 3.11, Hamilton 1.70
Expected behavior
Overridden nodes should not run on driver.execute
Additional context
Add any other context about the problem here.

Nice find! Yes, shoudl be easy. Ideally they'd share most of their code --we might be able to extract the common pieces into a helper fn (.prepare_execution(...) or something), but it's really broken up by the lifecycle hooks...

@elijahbenizzy can we close this once we push a release right?

This has been released in 1.71.0!


Adds authentication fix for UI.

Prior to this change, authentication did not allow for resources to be modified. They would simply just create new resources every time. This change allows resources to update by letting req.body.resourceId return non -1 responses. With this change, resources are being properly updated.

Ahh, good point.. you are right and good catch.. since I already pushed
that one through let's create a new issue for this.
On Tue, Sep 27, 2022 at 12:59 PM C. Marcus DiMarco @.***>
wrote:

@.**** commented on this pull request.
In server/controller/apis/resourceController.js
https://github.com/briangormanly/agora/pull/159#discussion_r981496386:

 resource.id = req.body.resourceId;

 // see if this is a modification of an existing resource

let existingResource = await resourceService.getResourceById( resource.id );

let existingResource = await resourceService.getResourceById( resource.id, false );

I found the line - makes sense when coming in from the front-end side, but
how would this play out if only using the API with Postman/ThunderClient?
Our tests for POSTing to the route create new resources when passed an
existing resource as JSON. Do you suspect that issue is unrelated to this?
—
Reply to this email directly, view it on GitHub
https://github.com/briangormanly/agora/pull/159#discussion_r981496386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATOOQOCQHJVCLA5HP37233WAMRV3ANCNFSM6AAAAAAQW6FYIE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
/Brian
Brian Gormanly
845.867.6919
@.***


.menu.nested vs dropdowns while on medium/large screens

How to reproduce this bug:
See the menu in the pen!
What should happen:
When you have a dropdown in your menu, it will add a margin to the sub-items ul and will not look ok for large screens.
What happened instead:
It should add the margin only for small devices.
Test case link:
https://codepen.io/wecodeart/pen/xddBZz

@BicanMarianValeriu would you like to create a PR for this ??

Not capable of properly fixing this. Someone with better knowledge about F core should do it. Sorry for late answer.

Fixed in #9670 ..... See this comment
Hopefully in 6.4 ..... sorry i dont think i will be able to create a patch for 6.3.2!


Not working in regions

In the regions i set at warzone people cant use their enchants
and some just dont work on players like rage and double damage

Can players fight each other and take damage?

yes

in spawn it does not work i know pvp is not enabled there

Did you fix this?

yes


Append the 'data' parameters to the url of GET request

Allow to make GET requests with params, when you use methot 'fetch' like:
myCollectionInstance.fetch({data: {some: data, other: param}})
make GET request with URL http://#{model_url}?some=data&other=param

Looks good to me, thanks!

Hi!
Is it possible to make a new release ? When installing via npm, this change is not in the code.

Oops, my bad, forgot to two that!
Just published the latest to NPM now as 0.4.2

Thanks, it works! :)

Thanks :)


Remove double padding from menu-text link

Solves https://github.com/zurb/foundation-sites/issues/8002.

Left a reply on #8002

Closing per my comment on #8002


Indy: Change trustees and stewards to a list

Change the network.yaml and Vault structure to a list of trustees and stewards.
Example:
      services:
        trustees:
        - trustee:
          name: authority-trustee

Closed by #82


⚠ Minecraft has degraded performance

In 5b2af43, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1337 ms

Resolved: Minecraft performance has improved in d221d6d.


Switching between tabs/editors

Hi VSC, when I use the "View: Open Next Editor" command is there a way to make it wrap back to the first one when I use it on the last editor? By default it goes to the first editor in the next group.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!

I worked it out. The command I need is "View: Open Next Editor in Group".


DFU Blaster

Hey, @kevinmcox
We're seeing an error when using this recipe with the Virus Total post processor.
Processor: io.github.hjuutilainen.VirusTotalAnalyzer/VirusTotalAnalyzer: Error: [Errno 2] No such file or directory: '/Users/sadmin/Library/AutoPkg/Cache/local.definition.DFUBlaster/downloads/PreBeta-DFU_Blaster_Build-2112_Version-2.2.dmg'

The Virus Total processor uploads %pathname% which has been moved: https://github.com/autopkg/kevinmcox-recipes/blob/master/Twocanoes Software/DFUBlaster.munki.recipe#L114#L124
Is this step needed? If you need the dmg in a specific location, would Copier work for you?
Thanks!

@paul-cossey let me take a look and see if I can remember what I was doing last year.

@paul-cossey Ah I remember now. The downloaded file is named PreBeta-DFU_Blaster_Build-2112_Version-2.2.dmg so I'm using FileMover to rename the DMG to something more succinct in DFU_Blaster.dmg.

Gotcha, so instead of FileMover you could add filename to URLDownloader and achieve the same result?
<dict>
    <key>Arguments</key>
    <dict>
        <key>filename</key>
        <string>%NAME%.dmg</string>
        <key>url</key>
        <string>https://bitbucket.org%match%</string>
    </dict>
    <key>Processor</key>
    <string>URLDownloader</string>
</dict>

Yep, great solution. Give me a few minutes to fix things up.

Awesome! Thank you! 😄