Integration of SuiteSparseGraphBLAS.jl

SuiteSparseGraphBLAS.jl provide the sparse representation of a graph and have better performance on algebraic operations, relative to SparseMatrixCSC in SparseArrays. It is suited for spectral-based graph convolutional layers like GCNConv or ChebConv. The integration will be made in GraphSignals.jl and GraphSignals.jl provides unified interfaces for graph convolutional layers to use.

That's a very nice project that we should definitely integrate (when it is AD and gpu ready)

I'll have to figure out what we're looking at on the GPU front. GraphBLAST is a GPU implementation, but I'll have to write a shim since I don't believe it has a plain C .h file. SuiteSparse:GraphBLAS (the library I wrap) will support GPUs in the future; the author is working with Nvidia to get that working.
Does GeometricFlux support CPU compute as well? I could work towards getting it ready from the CPU side in the meantime?

@Wimmerer Thank you for your contribution and engagedness. I can realize that it is not easy and it requires large amount of work. Currently, GeometricFlux supports cpu and cuda for model training. So, for compatibility, if SuiteSparseGraphBLAS.jl is taken as the solution for GeometricFlux for now, and CUDA support is not ready for now, then CUSPARSE can be a candidate. For CPU part, it runs as expected. You can just feel free on your own working roadmap.


fix: add better waiting logic for vault

also avoid logging the passwords/tokens when using vault
Signed-off-by: James Strachan<EMAIL_ADDRESS>

/approve


Vue syntax highlighting bugs when TS's optional chaining opreator is used within the template

Describe the bug

All syntax will become buggy if something like "something?.prop" is within double bracks inside the
<template>

Hello, {{ user?.name }}!

You have {{ user?.amount }} coins.

</template>

Expected behaviour

The syntax highligting is expected to recognise "object?.prop" as a valid object property and proceed to parse the next token instead of bugging everything following it.
Related discussion

Additional notes

From the troubleshooting docs:

If you experience an issue with the syntax-highlighting on GitHub, please report the issue to the upstream grammar repository, not here. Grammars are updated every time we build the Linguist gem so upstream bug fixes are automatically incorporated as they are fixed.

In the case of Vue, syntax highlighting is provided by vuejs/vue-syntax-highlight. I recommend filing an issue upstream, if it hasn't been reported already.
Closing this as it isn't an issue related to Linguist.

There has been no action in the vuejs/vue-syntax-highlight repository for 3+ years in the branch Linguist uses for Vue's syntax highlighting unfortunately.
See my issue from almost a year ago: https://github.com/vuejs/vue-syntax-highlight/issues/217 highlighting the issue (that's now grown to 2+ Stage 4 syntaxes (?. and ??)

I guess the only option is find another grammar (maybe a VSCode grammar?) that doesn't experience the issues and submit a PR to replace the grammar. You can find details on how to do this in the CONTRIBUTING.md file.

Are there any developments on this? I don't see why this issue was closed, since I have noticed this incorrect representation in our as well as many repositories from other organizations on GitHub that use Vue+TS. And as correctly expressed by @kaiarrowood here:

Vue 3 is pushing heavily towards typescript so instances of this are just going to get more and more common

If, as @wopian says, "there doesn't seem to be any interest from the maintainers to fix this either", would then @lildude's proposed solution be achievable instead?
Thanks

No developments from a Linguist perspective as this is closed as a problem with the upstream grammar. If the maintainers of the current grammar don't want to make changes then yes, the only option is to find a new grammar. We rely on the community to provide these kinds of updates as you're more likely in the know, so if you know of a Textmate/VSCode-compatible grammar that is suitably licensed, feel free to open a PR to change grammars.
No rush as you've just missed the latest release. The next release is scheduled to be in about three month's time.

The one in vuejs/language-tools (https://github.com/vuejs/language-tools/blob/master/extensions/vscode/syntaxes/vue.tmLanguage.json) looks good for me.


Haxe auto-indentation issue on case expression

This google code issue is still an issue.

https://gist.github.com/Simn/0dd44372e5b5cb699cac

Put the cursor at the end of line 7 and press enter. FD decreases indentation of the case expression.

I tried changing several settings, the only one that seems to affect it is disabling Smart Indent Type (which isn't really an option unless you get paid per keystroke).

FD <IP_ADDRESS>
This behavior can be very annoying when working with nested switches.

I have been writing my switches as
switch (value) {
	case 1: {
		//
	};
	case 2: {
		//
	};
}

for a while now, I suppose that in part due to this issue too.
It seems like you can only go certain length in terms of code complexity with switches before the fight with auto-indentation begins.

Scintilla auto indentation is a bit aggressive and difficult to control - we should tune it down, or disable it.

I don't know if it makes a difference, but I think this should be labeled bug, not enhancement.


Added Cloth Shopping App

This pull request establishes the basic structure and components for a cloth shopping app using React. The project is named "StyleEmporium," and the README includes instructions for setting up and running the application locally.

this is not a proper project
this is just displaying photos of clothes


Remove usages of avatarURL for policies

Currently held on https://github.com/Expensify/Web-Expensify/pull/34816
Details
This PR removes usages of avatarURL for policies in the front-end, since we're sending the actual avatarURL in avatar now from PHP in https://github.com/Expensify/Web-Expensify/pull/34740. This keeps policies more in-line with personalDetails, in that we access avatarURLs for both entities from avatar now.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/225573
Tests

Sign into any account, open a Workspace, and go to its Settings.

Click on the avatar picker and upload an avatar. Verify that no loading indicator appears, and that your avatar updates immediately after you click "Save" (the "Save" button in the cropping modal). Hard refresh (<Command><Shift><R>) the page, and verify that the new avatar is persisted.

Be ready to turn off your device wifi very quickly. Repeat step 3, but IMMEDIATELY turn off your device wifi before the avatar can finish uploading. It should still show the new avatar, but it should be greyed out like so:

Turn your device wifi back on, and verify the upload completes and the avatar isn't greyed out anymore.

Now click on the avatar picker again and select "Remove photo". Verify that the avatar is replaced with a default avatar. Reload the page and verify that this is persisted.

PR Review Checklist

Contributor (PR Author) Checklist

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

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

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

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

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that 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 was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[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] 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] Any functional components have the displayName property
[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 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 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] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The Contributor+ will copy/paste it into a new comment and complete it after the author checklist is completed

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

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

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

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

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that 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 was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[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] 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] Any functional components have the displayName property
[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 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 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] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

QA Steps

[ ] Verify that no errors appear in the JS console

Screenshots

Web

Mobile Web

Desktop

iOS

Android

I'll review when this is taken off draft ok? :D

@Beamanator No worries! I requested for review and then realized the tests were broken on https://github.com/Expensify/Web-Expensify/pull/34740 so put it back on draft
Code changes should be good to review now, but we're going to have to hold off on testing again until that PR is on production again 🙄. I personally haven't been able to attempt those testing steps myself yet for this very reason.

@Beamanator I'll lyk when the Web-E PR is on prod so we can test this one out :)
https://github.com/Expensify/Web-Expensify/pull/34740

@Beamanator I'll lyk when the Web-E PR is on prod so we can test this one out :)
Expensify/Web-Expensify#34740

@Beamanator eeez on prod, testing rn

Oooof it doesn't work, investigating what's going on

Dangit lol

Web-E PR merged 👍 still waiting on it getting deployed to prod to test this thoroughly

@Beamanator Looks like Web-Expensify PR is on prod! Just tested after updating my .env file to use the production API:
NEW_EXPENSIFY_URL=https://new.expensify.com/
SECURE_EXPENSIFY_URL=https://secure.expensify.com/
EXPENSIFY_URL=https://www.expensify.com/
EXPENSIFY_PARTNER_NAME=chat-expensify-com
EXPENSIFY_PARTNER_PASSWORD=e21965746fd75f82bb66
PUSHER_APP_KEY=268df511a204fbb60884
USE_WEB_PROXY=true
ENVIRONMENT=development

and it seems to still be broken? I'm so confused, since we're clearly copying avatarURL into avatar here, but the response still shows up in avatarURL. I also tested Web-Expensify PR policy update with a hard-coded avatarURL just to make sure this wouldn't happen.. ughhhhh
I'm investigating right now, looks like we might need to get another Web-E PR out, sorry :/

https://github.com/Expensify/Web-Expensify/pull/34816 is on staging, we'll be able to test soon!

Eyyy it's finally on prod!

Still working very well for me! Note: in step 3 of the tests, we can just start with the app offline, since we're using Pattern B
Also @jasperhuangg I thinkkk you still need to add QA tests :D

Took off hold since the Web-E PR is on Prod, and merged since tests pass 👍

: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 @Beamanator in version: 1.2.3-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@jasperhuangg This worked for me on staging. However, when we turn wifi back on the avatar momentarily reverts back to the old one. I think this happens because we call ReconnectApp first which still has the old avatar.
Uploading Screen Recording 2022-09-21 at 10.46.06 AM.mov…

🚀 Deployed to production by @luacmartins in version: 1.2.3-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@jasperhuangg This worked for me on staging. However, when we turn wifi back on the avatar momentarily reverts back to the old one. I think this happens because we call ReconnectApp first which still has the old avatar. We should avoid that flicker, what do you think?
Screen.Recording.2022-09-21.at.10.46.06.AM.mov

I agree, this flickering issue is actually an issue in a bunch of other areas of the App, I'm wondering if we should create a parent issue to deal with this.
Basically, when we reconnect the App, we should avoid updating Onyx values that have pendingActions. I do something similar here for workspaces that are created offline. Perhaps something needs to be built into the SaveResponseIntoOnyx middleware?

I agree, this flickering issue is actually an issue in a bunch of other areas of the App, I'm wondering if we should create a parent issue to deal with this.

Agreed. I'd bring this up in Slack and see where we go from there.

@luacmartins already on that haha message in my drafts rn

https://expensify.slack.com/archives/C03TQ48KC/p1663797332998359 @luacmartins


Subscribe app/uninstalled webhook

Subscribe the app uninstalled webhook, when merchant uninstall app from shop, shopify will trigger app/uninstalled webhook.
Listen the app/uninstall webhook and expire the app association from shop.
https://shopify.dev/api/admin-rest/2022-04/resources/webhook#event-topics-app-uninstalled

We don't need to subscribe the app uninstall webhook, As Shopify sends the shop/redact web hook on app uninstall
https://shopify.dev/docs/apps/build/privacy-law-compliance#shop-redact
48 hours after a store owner uninstalls your app, Shopify sends a [payload](https://shopify.dev/docs/apps/build/privacy-law-compliance#shop-redact-payload) on the shop/redact topic. This webhook provides the store's shop_id and shop_domain so that you can erase data for that store from your database.


Format incorrect

When attempting to use CLI tools such as sha512sum to validate the signatures they are failing, reporting the checksums do not match.
See https://github.com/transcend-io/terragrunt-atlantis-config/issues/309

Hi,
I went through and updated everything. I did a manual test with some files in the local repos and the checksums matched. Let me know if the issue persists.

Thanks!

Looks like there's one more small change to make - I made a PR https://github.com/jmgilman/actions-generate-checksum/pull/4


Added round-robin subscription support to API POST /request

This fixes https://github.com/sensu/sensu/issues/1083

:+1:


listDatabases command

Hi, why I'm I getting this issue as I've specified the namespace already. So as per the docs the listDatabases permission is not required if you provide the namespace-set
2017-01-19 17:40:32,738 [ERROR] mongo_connector.util:86 - Call to <bound method MongoClient.database_names of MongoClient(host=['remotemongo-url:port'], document_class=dict, tz_aware=False, connect=True, replicaset=replicaset)> failed too many times in retry_until_ok
Traceback (most recent call last):
File "/home/ubuntu/.local/lib/python2.7/site-packages/mongo_connector/util.py", line 78, in retry_until_ok
return func(*args, **kwargs)
File "/home/ubuntu/.local/lib/python2.7/site-packages/pymongo/mongo_client.py", line 1149, in database_names
"listDatabases")["databases"]]
File "/home/ubuntu/.local/lib/python2.7/site-packages/pymongo/database.py", line 494, in command
codec_options, **kwargs)
File "/home/ubuntu/.local/lib/python2.7/site-packages/pymongo/database.py", line 406, in _command
parse_write_concern_error=parse_write_concern_error)
File "/home/ubuntu/.local/lib/python2.7/site-packages/pymongo/pool.py", line 419, in command
collation=collation)
File "/home/ubuntu/.local/lib/python2.7/site-packages/pymongo/network.py", line 116, in command
parse_write_concern_error=parse_write_concern_error)
File "/home/ubuntu/.local/lib/python2.7/site-packages/pymongo/helpers.py", line 210, in _check_command_response
raise OperationFailure(msg % errmsg, code, response)
OperationFailure: not authorized on admin to execute command { listDatabases: 1 }

Are you using mlabs or a shared database?

@ballerabdude i'm using mlabs mongodb database

I had the same issue! I have updated the mongo-connector code to use another set of credentials to do reads https://github.com/ballerabdude/mongo-connector I have not made a pull request yet since I am waiting for a question of mine to be answered https://github.com/mongodb-labs/mongo-connector/issues/626

Duplicates https://github.com/mongodb-labs/mongo-connector/issues/626.


Extension fails to capture a URL from managed profiles due to a race condition

The Symptom
You click on the extension, logseq loads, but nothing is added. If you watch carefully, the URL includes undefined.
Additional testing shows that the page, append, title, and content  fields of the URL are all undefined.
Root cause
The code is trying to use these values before their respective promises resolve (creating a race condition).
(incorrect code)
(async () => {
  const [tab] = await chrome.tabs.query({
    active: true,
    lastFocusedWindow: true,
  });

This does not automatically turn chrome.tabs.query synchronous. It returns a promise that still has to be resolved.
The fix
Package the top-level code into an async function then resolve the promise appropriately.
Example:
  async function getCurrentTab() {
    let [tab] = await chrome.tabs.query({
      active: true,
      lastFocusedWindow: true,
    });
    return tab;
  }

  getCurrentTab().then(tab => {
    // now it's defined
  });

The same problem affected reading the options with chrome.storage.sync.get -- they were undefined due to a race condition and also needed default values.
PR coming momentarily.

@hkgnp ping! Would you be willing to roll in this fix? There are reviews mentioning the two issues fixed here in the Chrome web store.


Unhandled objects can bug whole code flow

When using it to normalize something like an API request response, sometimes the incoming object won't attend to the "valid object"'s of the lib.
In this case, just returning the original object would make everything work just as expected, while the current behaviour breaks the application.
e.g.:


AIM doesn't run on Android 5.1.1/Marshmallow

When starting the application (AIM 0.5.0 installed 1/13/2016) on a Nexus 5 phone running Android 5.1.1/Marshmallow, I receive the error message "Unfortunately, Adapt It Mobile has stopped", after which the application exits. This happens consistently, including after phone was restarted and after multiple uninstall/reinstalls.
Installation seemed to work fine, produced no errors.

Thanks!
Most of these errors are related to the Crosswalk plugin, which has been a mixed bag as far as its usefulness goes. Is this error due to some weird interaction with the Android OS level? Some incompatibility with the processor (Crosswalk separates the debug .apk into 2 -- one for x86 and another for ARM)? Hopefully we can isolate what's going on.
Nexus 5 is listed as having a Snapdragon 800 processor, which is an ARM-based processor.

@KLBradford - grasping at straws here... would it be possible to reinstall from https://build.phonegap.com/apps/713362/share on your device and retest?
There was a recent fix to the PhoneGap build that allows us to now use the latest crosswalk plugin from npm (instead of the plugin from a couple versions ago that's hosted on the PG build site). The updated build is on our share site above. It may or may not affect the inability of your device to run.

@eb1 Hmm... tried to install, but now I am getting the message "App did not install.". I uninstalled the app, cleared cache, and rebooted, still get that message and app won't install. I will keep trying - any ideas?

Ugh. It looks like the PhoneGap build folk haven't really fixed their packaging issues. I tried a few permutations (older versions, etc.), but it doesn't install on my machine either, unless I remove crosswalk altogether. I'll shelve this issue for later investigation. Thanks for trying it out, though!

Okay, once more with feeling. :-) I've updated the code to remove crosswalk -- and the page transition animation it was meant to fix. Would it be possible to download from the same location (https://build.phonegap.com/apps/713362/share) and try it out again?
I've verified that it works on both my older phone and tablet. You might need to uninstall the current version in order to install this one.

My Moto G Android phone is running Android 5.1. I tried just upgrading the previous AIM. The installation of the apk file seemed to work OK, and data appeared to be preserved for the upgrade. However, when I selected the last opened chapter to adapt, all I got was "Loading ... Please wait" and that message never ended. So I uninstalled the app, and then installed AIM afresh. The installation wiped out all my data :(, but I was able then to import the first chapter of Galatians, and open the file for adaptation. Hence, AIM appears to work OK on my 5.1 smartphone.

Okay, thanks for testing! This code change is currently in the 0.5.5 milestone; I've added this fix to the issue list.
Part of the necessary change here was to disable the page transition that prompted us to add Crosswalk to the mix in the first place (see: https://github.com/adapt-it/adapt-it-mobile/issues/73). Disabling this page transition does result in page transitions that aren't that pretty, but there's no rendering problem on older Android devices. I'll write up a separate lower-priority issue to go poke at the page transitions again -- there's got to be some transition that will work on older mobile browsers that we can use.
We'll also need to keep an eye out for issues that having the default (older) browser could cause -- weird rendering, etc.


Dapp, https://multitoken.io/ isn't working in Dapp browser

It is used to work, I have bought some index before.

Their site is down.


Make sidebar status bar refresh more often

Revisit after shipping 5.0.

Consider it. We’ll see if it’s better or not.


Rename multiword keys to use dashes in export and serialization

This PR covers changes to serialnumber, endtime, starttime and possibly other keys that are less visible.
Data container serialization and export transparently rename fields in the output to contain - when the field is a multi-word field, i.e. foo_bar. The multiword keys have been therefore split to take advantage of this functionality, plus a few extra touches were needed.
Fixes https://github.com/teemtee/tmt/issues/2054

Blocked by https://github.com/teemtee/tmt/pull/2373

Should be unblocked now as #2373 has been merged.

Rebased

Does this pull request obsolete #2095?

Does this pull request obsolete #2095?

I believe so - please, leave it open, I'll check it tomorrow if there's anything interesting left there.

I believe so - please, leave it open, I'll check it tomorrow if there's anything interesting left there.

Ack.


Refactor getLine to return TextLines/Rope

As @michaelpj pointed out, for many use-cases it might be preferable if getLine stays within the world of Ropes, to allow for further processing (an example of this is in lsp and similar, currently incorrect usage of getLine-like functionality mentioned in this comment in hls: https://github.com/haskell/haskell-language-server/pull/4303#issuecomment-2156657031
The initial impl. ini this PR might not be ideal, because its complexity is linear in the length of line being wrapped. But let me open a PR to get initial feedback.

It feels a bit strange when lines gives [Text], but getLine returns TextLines / Rope. Does it save much on HLS side? One can apply fromText there, probably to the same effect.

That would be fine I guess. Bascially we need the mixed-encoding indexing that we get from Text.Rope.Mixed! It feels a little strange to be converting back to Text and then back to Rope, but maybe that's fine?

I guess the neatest alternative would be to provide some utilities for mixed-encoding indexing of Text itself, on the reasoning that it should be okay on the Texts we get from getLine, since they are likely to be short. WDYT @Bodigrim ?

After thinking about it, I'm largely fine with both options:

We keep functions as they were before this PR, TextLines -> Text, Rope -> Text, and HLS wraps results in fromText to get access to encoding-based indexing. The only downside I see is slight inefficiency: e. g., a hypothetical getLine :: Int -> TextLines -> TextLines knows that the result contains no newlines, while with fromText . (getLine :: Int -> TextLines -> Text) we would have scan Text once again. Not a big deal for the sort of strings HLS usually operates on.
We change function signatures as in this PR. I'm fine to merge it, if HLS finds it useful. The only requirement is to be efficient and avoid double conversion. E. g., getLine :: Int -> Rope -> Rope should not convert intermediate Rope to Text only to execute fromText later.

I'll get to it over weekend. Will probably go with option 2 (keep Rope/TextLines).
Regarding avoiding double conversion I'll just have to figure out how to drop the last \n from the Rope without converting it to Text (or alternatively throw away/change the nice looking property that Rope.getLine i r == Text.lines (Rope.toText r) !! i. Suggestions are welcome :smile:

@jhrcek @michaelpj I'm uncomfortable that HEAD is in unreleasable state at the moment. Let's either finish up this PR soon or agree to stick to the design merged in #6.

Coming back to this, I don't feel so bad about just getting Text. It's true that we would benefit from the mixed-indexing capabilities of Text.Rope.Mixed, but in this instance we don't want a rope per se because we only have one line. It's quite reasonable for text-rope to say: you've just got a line now, you're out of my domain. It would be handy to have some kind of Text.Mixed module that let us do efficient indexing into a single Text, but that's really a separate request.

Ok, unless there are strong objections, I would close this PR and keep the current Text-returning implementation.
I was struggling with avoiding the double conversion for a bit, but it was not easy to implement while keeping the current invariants (making sure there are no newlines at the end of lines returned by getLine).

I gave it a stab in 3a46aaf. Thanks for trailblazing!


namerd - "session expired"

Error message gets thrown over and over in the logs and gets raised as a HTTP 502 to the client.
E 0704 16:09:06.818 THREAD22: service failure Failure(session expired, flags=0x100000000)     with NoSources     at com.twitter.finagle.NoStacktrace(Unknown Source)
From @olix0r :
fwiw we it would _seem_ like a zk exception:
finagle-serversets/src/main/scala/com/twitter/finagle/serverset2/ZkSession.scala```
but we probably should try to prevent that from bubbling up into the request flow if we can

Hey @JonathanBennett, I'm having trouble reproducing this.  As I understand it, namerd's zk client sends a heartbeat to keep the zk session alive.  Do you have any idea what could cause the session to expire?  By default, the client requests a 10 second session timeout.
Is the error message you pasted from linkerd or namerd?  If it's from linkerd, is there anything interesting in the namerd logs?

Hey,
I don't have a reason why zk would have failed to respond to a session. Rather annoyingly due to our log rotation in dev I don't have the logs for this as it looks to have been failing for quite a while so I'll be on the lookout to try and reproduce it; only thing I can think of is maybe ZK was underload and therefore failed to respond but I would imagine that as unlikely.
In such a case how should namerd recover? Should I be restarting namerd or should it be more resilient than this in failing from a zk timeout? From my logs I do have (about 4 days of failures over a weekend) I just see all lookups/sets failing with an error similar to the following:
E 0701 01:47:17.364 THREAD25: binding name /http/1.1/GET/zeppelin.xxxxx.co
java.lang.Exception: session expired
	at com.twitter.finagle.serverset2.buoyant.ZkSession$$anonfun$watchedOperation$1$$anonfun$com$twitter$finagle$serverset2$buoyant$ZkSession$$anonfun$$loop$2$1$$anonfun$2.apply(ZkSession.scala:162)
	at com.twitter.finagle.serverset2.buoyant.ZkSession$$anonfun$watchedOperation$1$$anonfun$com$twitter$finagle$serverset2$buoyant$ZkSession$$anonfun$$loop$2$1$$anonfun$2.apply(ZkSession.scala:142)
	at com.twitter.util.Witness$$anon$17.notify(Event.scala:434)
	at com.twitter.util.Var$$anon$3$$anonfun$register$1.apply(Var.scala:99)
	at com.twitter.util.Var$$anon$3$$anonfun$register$1.apply(Var.scala:99)
	at com.twitter.util.Var$Observer.publish(Var.scala:151)
	at com.twitter.util.UpdatableVar$$anonfun$update$2.apply(Var.scala:474)
	at com.twitter.util.UpdatableVar$$anonfun$update$2.apply(Var.scala:469)
	at scala.collection.TraversableLike$WithFilter$$anonfun$foreach$1.apply(TraversableLike.scala:778)
	at scala.collection.immutable.RedBlackTree$._foreachKey(RedBlackTree.scala:109)
	at scala.collection.immutable.RedBlackTree$.foreachKey(RedBlackTree.scala:105)
	at scala.collection.immutable.TreeSet.foreach(TreeSet.scala:154)
	at scala.collection.TraversableLike$WithFilter.foreach(TraversableLike.scala:777)
	at com.twitter.util.UpdatableVar.update(Var.scala:469)
	at com.twitter.finagle.serverset2.client.EventDeliveryThread$.run(EventDeliveryThread.scala:18)```

namerd should definitely recover by retrying on a new session.  I have a change that I suspect will fix it, but without being able to reproduce the error it's hard to be sure.


{Core} Make the recommendation for SSLError more accurate

Description
Not all SSLErrors are caused by untrusted self-signed certificate. For example, when a firewall blocks https://management.azure.com/, an requests.exceptions.SSLError will be raised:
$ PYTHONPATH="/lib64/az/lib/python3.9/site-packages" python3.9 -c "import requests; print(requests.get('https://management.azure.com/').status_code)"
...
requests.exceptions.SSLError: HTTPSConnectionPool(host='management.azure.com', port=443): Max retries exceeded with url: / (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1129)')))

This PR refines the recommendation message to make it more accurate.
Testing Guide
TBD

Core


Move the thermal profile selection file to Step 3 instead of the loading screen

This should be doable, need to decouple a bit for the new multipoint feature that is embedded in the thermal profile in the draft version of r0.3.4

New feature was added at 4984b458b182c0126b05e79eff509b803f495641


Delete Yarn lockfile

Please describe the changes this PR makes and why it should be merged, as well as any and all proof of successful tests:
This PR aims to delete Yarn's lockfile from the codebase. As Yarn is not used with this project, it's fine to delete it. Yarn's lockfile can also cause useless merge conflicts during PRs, which must be resolved manually.
Semantic versioning classification:

[ ] This PR changes the framework's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[x] This PR only includes non-code changes, like changes to README, etc.

So you just.... deleted the entire file 👀does it still work?

So you just.... deleted the entire file eyesdoes it still work?

Yup, works perfectly fine (at least on my machine :wink: ).

ffs that removed the approvals

Sorry I've actually polluted this PR somehow


sage: use overrideScope instead of packageOverrides

Description of changes
This PR changes the python3 instantiation of sagemath from using packageOverrides to overrideScope, as the former one is not composable (compare #44426).
After this change, it is possible to modify dependencies of sagemath within overlays, i.e., so that it composes.
Relevant maintainer to ping is likely @collares (who did the majority of most recent changes).
Things done

Built on platform(s)

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

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

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

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

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

[X] Fits CONTRIBUTING.md.

Thanks! I can't say I fully understand the effects of this, but it looks like a good change from what I can tell.
Hopefully I'll find time in the near future to catch up with Sage modularization, and then these overrides will be just normal Python modules (accessible from the standard Python package set).


Feature/608 inline message component

One last personal comment: I don't feel like the open/close animation adds anything.

Is the colour of the hover button correct?

good one, button variant needs to be the same as inline message status. I need to change it :)


V2 - Company Not Found - Email is not sent to company owner of the new company when user identifies himself as CLA Manager designee and created account

Prerequisite

Create a CLA group with CCLA enabled (I tried the flow where CLA Group is at foundation level and there is only 1 subproject for the foundation)
Add github repository for the CLA group

Steps to Follow

Login to Github as corporate contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Corporate Contributor
8.Click on building icon
Click on Organization not listed? + Click Here
Enter new company details
Click Next
Click Proceed
Click Yes
Click Proceed
Create Account
Verify Email address
User is redirected to corporate console. Company owner role and CLA Manager Designee roles are assigned however email is not sent to company owner to complete the organization profile

Testing blocked due to #1774

As discussed during Contributor console review meeting, email should not be sent to user who created company to company profile and hence this bug is no more valid


Command not implemented in interactive mode closes program

In the interactive mode, ff the user tries to execute a command that is not implemented, he gets an error such as: Command not implemented:  , and the program closes.
I think that it would be more natural to show the error and continue the execution.

I definitely agree with that. We should probably think also of tests for these issues.

This should be solved in #56 . I agree, there are some parts of the code, like the interactive mode, that do not have tests.

This should be fixed, so I will close it. Thanks David!


NVM installing node but not npm

Operating system and version:
osx: Monterey 12.5
nvm: 0.39.3
nvm debug output:

nvm --version: v0.39.3
$TERM_PROGRAM: iTerm.app
$SHELL: /bin/zsh
$SHLVL: 1
whoami: 'tobymerlet'
${HOME}: /Users/tobymerlet
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${HOME}/.asdf/shims:${HOME}/.asdf/bin:${NVM_DIR}/versions/node/v18.13.0/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Applications/VMware Fusion.app/Contents/Public:/usr/local/MacGPG2/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'zsh 5.8.1 (x86_64-apple-darwin21.0)'
uname -a: 'Darwin 21.6.0 Darwin Kernel Version 21.6.0: Sat Jun 18 17:07:25 PDT 2022; root:xnu-8020.140.41~1/RELEASE_X86_64 x86_64'
checksum binary: 'sha256sum'
OS version: macOS 12.5 21G72
awk: /usr/bin/awk, awk version 20200816
curl: /usr/bin/curl, curl 7.79.1 (x86_64-apple-darwin21.0) libcurl/7.79.1 (SecureTransport) LibreSSL/3.3.6 zlib/1.2.11 nghttp2/1.45.1
wget: /usr/local/bin/wget, GNU Wget 1.21.3 built on darwin21.6.0.
sed: /usr/bin/sed
cut: /usr/bin/cut
basename: /usr/bin/basename
rm: /bin/rm
mkdir: /bin/mkdir
xargs: /usr/bin/xargs
git: /usr/local/bin/git, git version 2.37.2
ls: grep:: No such file or directory
grep: grep: aliased to grep --color=auto --exclude-dir={.bzr,CVS,.git,.hg,.svn,.idea,.tox} (grep --color=auto --exclude-dir={.bzr,CVS,.git,.hg,.svn,.idea,.tox}), grep (BSD grep, GNU compatible) 2.6.0-FreeBSD
nvm current: v18.13.0
which node: ${NVM_DIR}/versions/node/v18.13.0/bin/node
which iojs: iojs not found
which npm: ${HOME}/.asdf/shims/npm
npm config get prefix: ${HOME}/.asdf/shims/npm: line 5: /usr/local/opt/asdf/bin/asdf: No such file or directory
${HOME}/.asdf/shims/npm: line 5: exec: /usr/local/opt/asdf/bin/asdf: cannot execute: No such file or directory
npm root -g: ${HOME}/.asdf/shims/npm: line 5: /usr/local/opt/asdf/bin/asdf: No such file or directory
${HOME}/.asdf/shims/npm: line 5: exec: /usr/local/opt/asdf/bin/asdf: cannot execute: No such file or directory

nvm ls output:

       v14.21.2
->     v18.13.0
        v19.4.0
default -> 18.13.0 (-> v18.13.0)
iojs -> N/A (default)
unstable -> N/A (default)
node -> stable (-> v19.4.0) (default)
stable -> 19.4 (-> v19.4.0) (default)
lts/* -> lts/hydrogen (-> v18.13.0)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.21.2
lts/gallium -> v16.19.0 (-> N/A)
lts/hydrogen -> v18.13.0

How did you install nvm?

install script in readme
What steps did you perform?
added config to zshrc
What happened?
I can use NVM to change the node version, but NPM is not being installed
What did you expect to happen?
Is there anything in any of your profile files that modifies the PATH?

No

Of course after creating the issue I've managed to fix it 😅 I seemed to have old remaining asfd files lying around which were causing havoc...  having removed them, uninstalling and re-installing nvm, it all seems to work now... thanks

De hoi


Fix memory leak in to_json

Fixes #2865.

Coverage remained the same at 100.0% when pulling db980739306d5dfade50a19f3a4a05702b450053 on issue2865 into c9c5c016a20399151f251ab1e612497821e82b0b on develop.

(I started with a regression test to make sure the CI will detect such issues in the future. And indeed, Valgrind and ASAN noticed the leak.)


Not working with stable Vivaldi version 2.0.1309.42

I got the recommended flags (#enable-picture-in-picture, #enable-surfaces-for-videos) but it doesn't seem to be working. It does something like this in YouTube:

My best guess is that Vivaldi didn't implement Picture-in-Picture.
@mounirlamouri WDYT?

It does, but it's only in snapshot versions for the moment.

@petternilsen Ah that would explain it. I went on the snapshot versions and it's working perfectly! Thanks for the tip!


Valentin_Refactor

Just some more random cleanups and things that I noticed.

@thomaslehmann @thomas-lehmann I refactored the plugin so that invalid ACL values are caught early and and appropriate error message is issued, hope it helps!

Seems okay to me, merging.

thx


Feedback TA

Hi guys,
Can you please extend your names in the readme file for to the full names, as I use them to find your names through absalon for sending your feedback.
Jonas

Hi,
We have extended our names in the general readme file now and put our KU-names on it as well.


Justyns purge old data

Description: Auto purge data older the configured time on startup of hass.
Each time HASS starts, the recorder component will run _purge_old_data() before going into the event loop. _purge_old_data() will delete records from the events and states tables older than X days. After deleting those records, VACUUM is run on the database to free up disk space taken by the deleted records.
This also adds a purge_days config option to the history component. By default it is set to -1 which means no old records will be deleted.
Started by @justyns, this is just a clean up and some fixes to hopefully allow us to merge it soon.
Related issue (if applicable): #1681
Example entry for configuration.yaml (if applicable):
# Enables support for tracking state changes over time.
recorder:
  # Delete events and states older than 2 weeks
  purge_days: 14

Checklist:
If code communicates with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Code clean up of #1681 which can be closed once this gets merged.

Thanks for finishing this up, @infamy!


Create: 392-is-subsequence.py

File(s) Modified: 392-is-subsequence.py
Language(s) Used: python
Submission URL: https://leetcode.com/submissions/detail/812722396/

Can you please capitalize every word in the filename, otherwise lgtm

Will do!
On Sun, 2 Oct 2022 at 12:06 AM, neetcode-gh @.***>
wrote:

Can you please capitalize every word in the filename, otherwise lgtm
—
Reply to this email directly, view it on GitHub
https://github.com/neetcode-gh/leetcode/pull/1210#issuecomment-1264448105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSHEVQEZGZSSOB4CM2NJW3WBCABNANCNFSM6AAAAAAQ2QSGDU
.
You are receiving this because you authored the thread.Message ID:
@.***>


use return_tensors="pt" in the conversion script

Can you also take a look at tests/test_modeling_luke.py?


error 500 from turso endpoint when running on AWS Lambda

TRPCClientError: request to https://somedb.turso.io/v2/pipeline failed, reason: write EPIPE
    at Function.from (/Users/user/Documents/proj/linkedin/pup/node_modules/@trpc/client/dist/TRPCClientError.js:28:20)
    at /Users/user/Documents/proj/linkedin/pup/node_modules/@trpc/client/dist/links/internals/createHTTPBatchLink.js:61:76
    at processTicksAndRejections (node:internal/process/task_queues:95:5) {
  meta: {
    response: Response {
      [Symbol(realm)]: null,
      [Symbol(state)]: [Object],
      [Symbol(headers)]: [HeadersList]
    },
    responseJSON: [ [Object] ]
  },
  shape: {
    message: 'request to https://somedb.turso.io/v2/pipeline failed, reason: write EPIPE',
    code: -32603,
    data: {
      code: 'INTERNAL_SERVER_ERROR',
      httpStatus: 500,
      stack: 'FetchError: request to https://somedb.turso.io/v2/pipeline failed, reason: write EPIPE\n' +
        '    at ClientRequest.<anonymous> (/var/task/lambda.js:7:306176)\n' +
        '    at ClientRequest.emit (node:events:518:28)\n' +
        '    at TLSSocket.socketErrorListener (node:_http_client:495:9)\n' +
        '    at TLSSocket.emit (node:events:518:28)\n' +
        '    at emitErrorNT (node:internal/streams/destroy:169:8)\n' +
        '    at emitErrorCloseNT (node:internal/streams/destroy:128:3)\n' +
        '    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)',
      path: 'getSession'
    }
  },
  data: {
    code: 'INTERNAL_SERVER_ERROR',
    httpStatus: 500,
    stack: 'FetchError: request to https://somedb.turso.io/v2/pipeline failed, reason: write EPIPE\n' +
      '    at ClientRequest.<anonymous> (/var/task/lambda.js:7:306176)\n' +
      '    at ClientRequest.emit (node:events:518:28)\n' +
      '    at TLSSocket.socketErrorListener (node:_http_client:495:9)\n' +
      '    at TLSSocket.emit (node:events:518:28)\n' +
      '    at emitErrorNT (node:internal/streams/destroy:169:8)\n' +
      '    at emitErrorCloseNT (node:internal/streams/destroy:128:3)\n' +
      '    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)',
    path: 'getSession'
  },
  [cause]: undefined
}

  "dependencies": {
    "@libsql/client": "^0.5.6",
    "@trpc/server": "^11.0.0-next.320",
    "drizzle-orm": "^0.30.6",
    "drizzle-zod": "^0.5.1"
  }

happened once, the next request succeeded


3.0 main sync 24-07-09

No conflicts, should be good to merge if tests succeed.

This time even the DCO check worked…


GUI does not reflect pool monitor changes reposted with AS3

Do you already have an issue opened with F5 support?
No
ISSUE TYPE
Bug Report
AS3 BUILD/ VERSION
f5-appsvcs-3.11.0-3
BIGIP VERSION
<IP_ADDRESS>
SUMMARY
When reposting a declaration with a changed monitor type for a pool, the GUI still shows the old monitor type for the pool. TMSH shows the changed monitor type as well as the AS3 response after sending the POST.

STEPS TO REPRODUCE

1) Post a simple declaration, e.g. the example from the AS3 documentation:

{
     "class": "AS3",
     "action": "deploy",
     "persist": true,
     "declaration": {
         "class": "ADC",
         "schemaVersion": "3.0.0",
         "id": "urn:uuid:33045210-3ab8-4636-9b2a-c98d22ab915d",
         "label": "Sample 1",
         "remark": "Simple HTTP Service with Round-Robin Load Balancing",
         "Sample_01": {
             "class": "Tenant",
             "A1": {
                 "class": "Application",
                 "template": "http",
                 "serviceMain": {
                     "class": "Service_HTTP",
                     "virtualAddresses": [
                         "<IP_ADDRESS>"
                     ],
                     "pool": "web_pool"
                 },
                 "web_pool": {
                     "class": "Pool",
                     "monitors": [
                         "**http**"
                     ],
                     "members": [
                         {
                             "servicePort": 80,
                             "serverAddresses": [
                                 "<IP_ADDRESS>",
                                 "<IP_ADDRESS>"
                             ]
                         }
                     ]
                 }
             }
         }
     }
 }

Note: the monitor of the web_pool is of type "http"!
Result: The GUI shows the correct monitor as http at this step.

2) Post the same declaration again with a monitor of e.g. "tcp":

{
     "class": "AS3",
     "action": "deploy",
     "persist": true,
     "declaration": {
         "class": "ADC",
         "schemaVersion": "3.0.0",
         "id": "urn:uuid:33045210-3ab8-4636-9b2a-c98d22ab915d",
         "label": "Sample 1",
         "remark": "Simple HTTP Service with Round-Robin Load Balancing",
         "Sample_01": {
             "class": "Tenant",
             "A1": {
                 "class": "Application",
                 "template": "http",
                 "serviceMain": {
                     "class": "Service_HTTP",
                     "virtualAddresses": [
                         "<IP_ADDRESS>"
                     ],
                     "pool": "web_pool"
                 },
                 "web_pool": {
                     "class": "Pool",
                     "monitors": [
                         "**tcp**"
                     ],
                     "members": [
                         {
                             "servicePort": 80,
                             "serverAddresses": [
                                 "<IP_ADDRESS>",
                                 "<IP_ADDRESS>"
                             ]
                         }
                     ]
                 }
             }
         }
     }
 }

Note: the monitor of the web_pool is now set to "tcp"!
Result: The GUI does not show the tcp monitor, but the old http monitor.
The change is visible though via TMSH:

root@(bigipA)(cfg-sync Standalone)(Active)(/Common)(tmos)# list ltm pool /Sample_01/A1/web_pool
ltm pool /Sample_01/A1/web_pool {
    members {
        /Sample_01/<IP_ADDRESS>:http {
            address <IP_ADDRESS>
            session monitor-enabled
            state down
            metadata {
                source {
                    value declaration
                }
            }
        }
        /Sample_01/<IP_ADDRESS>:http {
            address <IP_ADDRESS>
            session monitor-enabled
            state down
            metadata {
                source {
                    value declaration
                }
            }
        }
    }
    min-active-members 1
    monitor min 1 of { **tcp** }
    partition Sample_01
}

EXPECTED RESULTS

GUI should reflect monitor changes introduced via AS3 Reposts.

ACTUAL RESULTS

GUI does not reflect monitor changes introduced by AS3 reposts.

Thank you for opening this issue. I was able to reproduce this in my lab as well. I find this same behavior to be true if the monitor is updated via TMSH as well (not showing in the GUI). Since the update appears in the running config of the BIG-IP this seems to be a bug with the UI and not AS3 itself. Please open a case through your account team for BIG-IP and not AS3.

Thanks for your hint, i have already opened a support case yesterday (C3018563), i will reference this issue there. thx!


Missing store_connection_channels after ExecutionContext refactor

Bug Summary
The function
fn store_connection_channels(
    &mut self,
    connection_id: ConnectionId,
    port_id: PortId,
    channel_id: ChannelId,
) -> Result<(), ContextError>;

got removed during the ExecutionContext refactor, but
fn connection_channels(
    &self,
    connection_id: &ConnectionId,
) -> Result<Vec<(PortId, ChannelId)>, ContextError>;

remains in ValidationContext.
Details
Since store_connection_channels got removed from ExecutionContext, I think there isn't a function any more that ties connections to (port, channel) pairs anymore, meaning connection_channels would always return an error.
Version

ibc-rs v0.29.0 onwards.

It can still be retrieved by querying for all channels using ChannelEndPath and matching the passed connection_id with the connection_hops field of each found ChannelEnd. (like here in basecoin-rs).
Though indeed, it is not an efficient way and in some implementations can end up with the error.

I think the main point though is that it is no longer used by the core handlers, and so serves no purpose. So IIUC, we should just remove ValidationContext::connection_channels().


playlist not working

playlist links not playing at all (currently only single video links work)

fixed playlists not working. -- now both playlist and stand alone videos can be played as long as they are entered correctly into the csv file


Fix: (android) content hint can only be set after constructing the publisher and capturer

Maxime told me to merge 😛


When installing the development environment for Energy CLI, the download timed out

A timeout occurred when Energy downloaded CEF.

I'm also stuck at $ Check version

Install command, please try again and again now


Improve CanonicalSimplify to handle Min, Max(#2248)

#2248
Also enable Mul caching for more cases
Tests
New unit tests pass.

@tqchen please take a look.

Thanks, @wweic this is merged


NoTestRun Not Accepted When Validating in a Sandbox Org

Summary
Setting the Test Level to NoTestRun is currently accepted when running a deployment (sf project deploy start) to a sandbox org.
NoTestRun is not accepted when attempting to run a validation (sf project deploy validate -x manifest/package.xml -l NoTestRun --verbose).
Additionally, when I omit the Test Level from the validation command above (sf project deploy validate -x manifest/package.xml --verbose), all local tests in the org run during the validation.
Omitting the Test Level during a Deployment of Non-Apex to a sandbox will set tests to Not Required after Salesforce scans the package.xml.
Steps To Reproduce

Declare a package.xml with non-apex components (custom field)
Run this command to start a validation with NoTestRun: sf project deploy validate -x manifest/package.xml -l NoTestRun --verbose
Run this command to start a validation without the Test Level: sf project deploy validate -x manifest/package.xml --verbose

Expected result
Validation to the sandbox org starts with No Tests Required when setting Test Level to NoTestRun or omitting the Test Level to default to Salesforce scanning the package.xml.
Actual result
Validation with NoTestRun does not start with this error statement.
Error (1): Expected --test-level=NoTestRun to be one of: RunAllTestsInOrg, RunLocalTests, RunSpecifiedTests
See more help with --help

System Information
PowerShell on Windows 10

{
  "cliVersion"<EMAIL_ADDRESS>  "architecture": "win32-x64",
  "nodeVersion": "node-v18.15.0",
  "osVersion": "Windows_NT 10.0.19045",
  "shell": "cmd.exe",
  "rootPath": "C:\\Program Files\\sf\\client",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 2.3.6 (core)",
    "@oclif/plugin-commands 2.2.23 (core)",
    "@oclif/plugin-help 5.2.17 (core)",
    "@oclif/plugin-not-found 2.3.37 (core)",
    "@oclif/plugin-plugins 3.2.7 (core)",
    "@oclif/plugin-search 0.0.22 (core)",
    "@oclif/plugin-update 3.1.32 (core)",
    "@oclif/plugin-version 1.3.8 (core)",
    "@oclif/plugin-warn-if-update-available 2.0.48 (core)",
    "@oclif/plugin-which 2.2.31 (core)",
    "@salesforce/cli 2.6.7 (core)",
    "apex 2.3.11 (core)",
    "auth 2.8.13 (core)",
    "data 2.5.7 (core)",
    "deploy-retrieve 1.17.5 (core)",
    "info 2.6.39 (core)",
    "limits 2.3.31 (core)",
    "login 1.2.28 (core)",
    "org 2.10.3 (core)",
    "schema 2.3.23 (core)",
    "settings 1.4.26 (core)",
    "sobject 0.2.6 (core)",
    "source 2.10.32 (core)",
    "telemetry 2.3.1 (core)",
    "templates 55.5.10 (core)",
    "trust 2.6.3 (core)",
    "user 2.3.29 (core)",
    "sfdx-git-delta 5.25.2 (user)"
  ]
}

Additional information
Setting Test Level to NoTestRun for a Validation of Non-Apex Package

Omit Test Level for a Validation of Non-Apex Package

Same behavior seen when updating to the latest CLI
{
  "cliVersion"<EMAIL_ADDRESS>  "architecture": "win32-x64",
  "nodeVersion": "node-v18.15.0",
  "osVersion": "Windows_NT 10.0.19045",
  "shell": "cmd.exe",
  "rootPath": "C:\\Users\\matthew.carvin\\AppData\\Local\\sf\\client\\2.7.11-5a5b1de",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 2.3.6 (core)",
    "@oclif/plugin-commands 2.2.24 (core)",
    "@oclif/plugin-help 5.2.18 (core)",
    "@oclif/plugin-not-found 2.4.0 (core)",
    "@oclif/plugin-plugins 3.3.2 (core)",
    "@oclif/plugin-search 0.0.22 (core)",
    "@oclif/plugin-update 3.2.0 (core)",
    "@oclif/plugin-version 1.3.9 (core)",
    "@oclif/plugin-warn-if-update-available 2.0.49 (core)",
    "@oclif/plugin-which 2.2.32 (core)",
    "@salesforce/cli 2.7.11 (core)",
    "apex 2.3.12 (core)",
    "auth 2.8.14 (core)",
    "data 2.5.7 (core)",
    "deploy-retrieve 1.17.6 (core)",
    "info 2.6.40 (core)",
    "limits 2.3.32 (core)",
    "login 1.2.28 (core)",
    "marketplace 0.1.1 (core)",
    "org 2.10.5 (core)",
    "schema 2.3.24 (core)",
    "settings 1.4.27 (core)",
    "sobject 0.2.6 (core)",
    "source 2.10.32 (core)",
    "telemetry 2.3.1 (core)",
    "templates 55.5.10 (core)",
    "trust 2.6.7 (core)",
    "user 2.3.31 (core)",
    "sfdx-git-delta 5.25.2 (user)"
  ]
}

@mcarvin8 we designed 2 distinct paths for validation in sf.
project deploy validate is meant to be run against a production org as a precursor to project deploy quick (and that's why it requires the tests in the validation).
project deploy start has a --dry-run option for any other sort of validation you might want to do (ex: deploy but not run any tests, deploy to a non-prod org with or without tests, etc.  It's possible to run this in a way that won't be valid for quick deploy, but it's also way more flexible.

Thanks @mshanemc for the update. The --dry-run option seems to be what I'm looking for in terms of validating separately ahead of a deployment. Tested that out with NoTestRun and that does the trick.


Implemented early dynamic memory allocation

Implemented a bootstrap memory allocator as suggested in this comment. The implementation is similar to LiteBSD's.
I was unsure to which namespace the allocator shall belong, so I ended up using vm.
Currently, allocating more than ~4000B will crash the kernel due to #25. However, the currently proposed fix for #25 solves the issue.

Note that bootstrap memory allocator doesn't belong to virtual memory subsystem, but rather kernel memory allocator. Thus the module should be named 'kmemand the function... well... maybekm_early_allocorkmalloc_early`.

While fixing address alignment, I've also added ALIGN_WORD_UP and ALIGN_WORD_DOWN macros to common.h, they may come in handy.


⚠ AsianCinema has degraded performance

In 7e4c70f, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 525
Response time: 20315 ms

Resolved: AsianCinema performance has improved in 994ae01.


Configuration parameters for Stackoverflow Component

First of all, thanks a lot for this nice components (particularly Stackoverflow Component). I think this component would be more useful and customizable by adding some extra configuration parameters e.g. width, height, padding. Thanks in advance.

Regarding to padding, there is an invisible element around the component and I cannot align it to the title as the other components in my README.md by using margin-left, etc. Is there any workaround to fix this problem?


Handle parallel request return status

Motivation
Dashboard cannot handle multiple Rest API response statuses.
See here.
Ideally, dashboard must popup all API response status or all failure response status.
Goal

Handle parallel request return status

Implementation

[ ] TBD

#13 #27 were fixed this issue.
Close.


src/view.php improvements

src/view.php could be given a sidebar with a library browser. It could display the whole library as a tree, possibly also adding an option to switch libraries. And, of course, an option to switch icl/dcl.
Then we could link to this page also from library names, e.g. 'Platform' in a search result could link to the page which does not open any source code but still has the side bar to browse Platform.

Definitly a good idea

Still to do:

Automatically open library until current module
Highlight current module (italics?)


Enhancing the DpmService API

Some thoughts on the DpmService widget.
As data is returned, we want to update as few widgets as possible. Ideally, we only want to update the Text() widget that’s displaying the actual data. In the param page, we have a Entry widget which displays an entire row of the param page. For devices, it displays the name, description, reading, setting, status, (timestamp?). So to redraw all that data for each point would be “expensive”. Plus, the row would be getting updates for more than one property, so we really don’t want to re-render multiple times per data point.
As of today, a device ParameterEntry widget calls dpm.monitorDevices([widget.drf]) which returns a stream of Readings. The stream is passed to a StreamBuilder which rebuilds the dependent widgets each time a new value arrives (slightly cleaned-up):
StreamBuilder(
  stream: _stream,
  builder: (context, snapshot) {
    final bool hasData = snapshot.connectionState == ConnectionState.active;

    return _buildParam(
      hasData ? snapshot.data!.value : null,
      units,
      key: Key("${widget.drf}")
    );
  }
})

I propose keeping this API in case someone is writing Dart code. A Dart service wouldn’t use the StreamBuilder but would do computations as the stream returns data. This seems a little too much boilerplate code for Flutter developers, though, so I’d like to add to the API. What if we add an AcquisitionWidget?
AcquisitionWidget(
  drf: "M:OUTTMP@p,60000",
  builder: (BuildingContext context, Reading reading) {
    return Text("${reading.data}");
  }
)

What’s the difference?

The first example didn’t show the call to dpm.monitorDevices() and saving the stream in the class’s fields.
The first example didn’t show the code that had to pass the _stream down through the functions calls.
The first example opened its own websocket.
The proposed API would make the call to .monitorDevices() and keep track of the stream (subscribing and unsubscribing, as necessary.)
The proposed API would merge all requests over one websocket and deliver the results to the correct AcquisitionWidget's builder function.
The proposed API searches for the DpmService widget, so you can simply wrap the widget that needs updating (the first example doesn't show that _buildParam() actually creates a Row widget containing Text widgets and a SizedBox widget because focusing on the one Text field requires passing the stream down further.)

Of course, I stink at naming things so, if this sounds like a good plan, please help me name the classes appropriately.

Let me make sure I follow...
So essentially, AcquisitionWidget will stand in for the reading/setting Text and manages the stream of data to that specific Text as displayed on the screen?
And then, the ParameterWidget could be composed of two AcquisitionWidgets - one for displaying the setting property and another for the reading property (or possibly a third for the control property)?

That's right, @cnlklink. A row has to start multiple requests for readings, settings, and status anyways. Why not let this widget do that and target the exact widget that needs to change when there's an update.


Add an --ignore OS errors

when i'm copying some rich directories, like system windows files, there're may be some problems like this one.
for example, standart cp (windows) ignore errors (and show them).

can be a feature like --ignore with logging path with problems?

I think I'd accept a PR for this, but I don't really like it.


Added support for Ampq v1.0

@AndreSteenbergen so is this going to publish as its own independent package?

I think it should, as amqp 0.9 (RabbitMq and the likes) and amqp 1.0 are incompatible. With RabbitMQ being quite popular, I don't think it is wise to remove 0.9 support.

BTW this pull request is just a first start to support AMQP 1.0. I hope good feedback can be given to the code, I hope this contributes something .

This pull request was made to support this: https://github.com/AkkaNetContrib/Alpakka/issues/37

Ah, got it.
CC @bobanco - any thoughts on this?

@AndreSteenbergen  left you a few remarks,. One more general question, aren't there any async methods for the queue, e.g. ReceiveAsync, AcceptAsync ... ?

Tested the latest commit using these brokers:

TestBroker from amqplite
ActiveMQ 5.15.4
ActiveMQ Artemis 2.6.2

All seem to work now.

@marcpiechura What do you think after the updates?

@AndreSteenbergen @marcpiechura @vasily-kirichenko any word on this? I'd like to push out the Akka.Streams.Kafka package but if this is ready for merge, we can do both at the same time.

Do the two tests cover all this PR functionality?

Sorry was away on vacation; I think all functionality is captured. With amqp 1 the server implements logic, instead of the client. I couldn't think of more tests besides receiving what was send.

@marcpiechura what is the status of this PR? Could we merge it?


Java api

This PR is open for quite some time. Is there something missing @pstadermann or can it be reviewed and merged?


builds list continuation token seems broken

it seems like the continuation token for the builds list request is broken in version 5.0 (tested with 5.1 preview 5 as well), the "x-ms-continuationtoken" returns a datetime which is different then other requests and sending that datetime back on a subsequent request returns the first page again
the documentatiom does not specify the use of "x-ms-continuationtoken" but i assumed it will be the same as other apis, am i missing something?

Can you clarify what you mean by the builds list request?  Which API are you using?  getBuilds?
Thanks!

4.1 has the same issue, if relevant

The format of the continuation token is opaque. It will be different for different types of requests. So this, in itself, is not a bug.  Of course whatever continuation token you're getting should work, so let's investigate that :)
Can you provide an example of the requests you're making, first the one that returns the continuation token, and then the one where you send it back?

Header from response of first request:
x-ms-continuationtoken: 2017-11-20T13:18:01.1275996Z

My subsequent request:
https://dev.azure.com/<organization>/<project>/_apis/build/builds?continuationToken=2017-11-20T13:18:01.1275996Z

Thanks for reporting!
We will be releasing a fix soon.
Until then, the workaround is to explicitly specify the statusFilter.
Eg (c#):

builds = await buildClient.GetBuildsAsync2(
        project: c_projectName,
        continuationToken: builds.ContinuationToken,
        statusFilter: BuildStatus.All,
        top: 3);

This should be fixed in azure devops.

it's still broken in Azure DevOps Server 2020 (API 6.0)


Initial render does not include stitches SSR styles

Describe the bug

I am unsure if I should be filing this here in stitches, but I have yet to encounter this issue with remix or nextjs SSR, so I am filing it here.

Context
Stitches is a CSS-in-JS solution that requires that users of the library call a method called getCssText() and inject it into the head of their project for SSR, such that all styles are present on the first render.
The Problem
I am currently injecting the styles into the svelte:head component like so (the funny additions are due to an open bug):
<svelte:head>
    {@html `<${''}style>${getCssText()}</${''}style>`}
</svelte:head>

However, the problem is that for some reason, svelte kit does ?? (I am assuming) on the first render such that the styles I have written are not present on the first SSR run of the page (check reproduction).
First render:

All renders afterwards (note the class declaration visible that sets the color to green):

This issue also extends to prerendering. If I set prerender to true and export that in the root layout, the missing stitches styles are never injected.
Reproduction

Clone the repository. (This is just a basic svelte kit skeleton that was generated)
Install the dependencies and start the development server.
Make a request to / with curl or xh or your HTTP utility of choice, and observe that the style tag that is injected does not have the color: green declaration.

Make another request after the previous one and observe that the style tag now includes the missing color: green declaration, as does every subsequent request after the initial render.

Logs
No response
System Info
System:
    OS: macOS 13.2
    CPU: (10) arm64 Apple M1 Max
    Memory: 2.16 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.9.1 - ~/.nvm/versions/node/v18.9.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.9.1/bin/yarn
    npm: 8.19.1 - ~/.nvm/versions/node/v18.9.1/bin/npm
  Browsers:
    Chrome Canary: 111.0.5525.1
    Firefox Developer Edition: 106.0
    Safari: 16.2
  npmPackages:
    @sveltejs/adapter-auto: 1.0.0 => 1.0.0
    @sveltejs/adapter-node: 1.1.0 => 1.1.0
    @sveltejs/kit: 1.0.7 => 1.0.7
    svelte: 3.55.0 => 3.55.0
    vite: 4.0.4 => 4.0.4

### Severity

annoyance

### Additional Information

_No response_

Candidly, I would avoid this technique. It involves runtime overhead, and because it uses


add Coda Hale's Metrics library

Hmm. I'm hesitant to add them as we already have @codehale's Dropwizard docs listed and these share the same design as Metrics. How about I add a Metrics mention in the same spot as the Dropwizard docs?

Works for me. I just thought they should definitely be mentioned here.

Agreed. Thanks for the contribution. Keep them coming.


[Analog clock using html css and js]

Description
responsive analog clock using html,css,js
Domain
Frontend Dev HTML/CSS/JS
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Different values for sum() in Query Builder and Eloquent

Laravel Version: 5.6.*
PHP Version: 7.2
Database Driver & Version: MySQL - 10.2.23-MariaDB-log

It seems that there is a difference between result of sum() aggregation in Query Builder and Eloquent.
User::where('role', '1')->sum('credit'); returns 194660
DB::table('users')->where('role', '1')->sum('credit'); returns 204660
When I run following query in phpmyadmin: SELECT SUM(credit) FROM users WHERE role = '0' it returns : 204660
I got version of MySQL by running SELECT VERSION() as version in phpmyadmin and I don't know that it is same as what you want or not...

It's hard to tell without code but it could be the SoftDeletes Trait.

@gerardnll Thank you, it was the reason.


Semiconductor blog

Description
This is a blog post for the semiconductor industry.
Type of change
Please delete options that aren't relevant.

[ ] This change requires a documentation update

How Has This Been Tested?
Please also list any relevant details for your test configuration
Checklist

[ ] Updated documentation if needed
[ ] Tests added/amended
[ ] bazel test //...  passes locally
[ ] PR is contained in a single commit, using git amend see some docs

This change is

I'm at a lost for what I have to do to make vale behave here. There are so many toys.


Whl upload: Handle special characters in whl file? Or improve error message

I've been using the new upload whl file approach quite a bit, it's very convenient!
I hit one edge-case: If the whl file name is not structured as expected, I run into the following error:
1733993758_collect_2_pip_install.sh: line 10: syntax error near unexpected token `('

This happened because my whl file was called fractal_faim_ipa-0.4.3-py3-none-any (2).whl. The "(2)" was introduced by macos because I had downloaded that whl file multiple times. I'm fine with this not being supported. In that case, we should just raise an informative error.
Alternatively, if we can support that, that's also nice of course :)

If the error becomes ERROR: pydantic-2.10.3-py2-none-any (2).whl is not a valid wheel filename., that's great. Because it's descriptive of the issue being with the wheel filename :)


how to use the partseg and semseg to predict new data?

Hello @pytholic ,
thank you for this pull of pointnet2 ,
I am trying t predict new data and then save them with segmentation, so is it possible if you can give any help or guidance on how to make the partseg or semseg to predict new data?
thank you in advance

Hi @madinwei
Sorry for the late reply.
I worked on it a long time ago so I forgot most of the details. Does this script give you some insight?


Should enter text in the correct field when used in loop

Expected behavior
Should enter text in the correct field when used in loop
Actual behavior
The text is being entered in the last text field
Steps to reproduce

clone https://github.com/sguptatw/loopIssueTaiko.git
cd loopIssueTaiko
npm install
gauge run

Versions
taiko - c4bd3810aa5790692479555507e40dc66a2a4fe3
Gauge version: 1.0.5

Plugins
-------
html-report (4.0.8)
js (2.3.5)
screenshot (0.0.1)

Work around
Use for loop instead of for each in code

@sguptatw The forEach does not handle async callbacks. So you have to use for loop instead of forEach for async tasks.

The node's forEach does not handle the  async callbacks. So any await inside the callback will not be considered, and the loop will go to the next element without completing the first task.


Remove NonDebuggable from Azure Blob Storage

Summary
Remove NonDebuggable from Azure Blob Storage and Storage Authorization
This PR removes the NonDebuggable from Helper procedure and other procedures where the data of the procedures is already known, since it's provided from upper codeunits.
I skipped the RequestHelper Codeunit for the moment, since I'm not sure if we can remove the NonDebuggable property there.
e.g. if we use the ReadSAS Authorization where the Secret is part of the request uri.
I added new Overloads to the codeunit 9062 "Storage Service Authorization" for CreateAccountSAS with SecretText.
Work Item(s)
Partially Fixes #875
Fixes AB#524277

@pri-kise, you seem to have compilation issues:
Error: AL0432 Method 'CreateAccountSAS' is marked for removal. Reason: Use CreateAccountSAS with SecretText data type for SigningKey.. Tag: 25.0.

@JesperSchulz I've updated the test to use SecretKey
Hopefull the CI pass now.


Update sbt-scalajs to 1.6.0

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

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

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #130 (c372382) into master (0f85f05) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #130   +/-   ##
=======================================
  Coverage   80.35%   80.35%
=======================================
  Files           6        6
  Lines         112      112
=======================================
  Hits           90       90
  Misses         22       22

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 0f85f05...c372382. Read the comment docs.


We still has names of people in the guide for internal process

Example: https://guide.rladies.org/organization/tech/accounts/

Yes it is linked to #18 #3 :cry:


make sure the matching fund transaction is made by the matching fund stripe customer

Coverage decreased (-0.0008%) to 76.098% when pulling 772b4ccf5d11c7abd326481c424640b0f5908166 on fixDoubleStripeReceipt into 727a4cb5efa9a69d991556993ea3b984e662b4e4 on master.


FEAT: move imports to top in notebooks

This is a temporary fix for https://github.com/ComPWA/compwa.github.io/pull/270 and related issues/PRs. It uses isort to move all imports to the top of a notebook and then Ruff will do the rest of the sorting.
In practice, by adding the --imports-on-top flag to the `check-dev-files hook, the following pre-commit hook will be added:
  - repo: https://github.com/nbQA-dev/nbQA
    rev: 1.9.0
    hooks:
      - id: nbqa-isort
        args: [--float-to-top]
        # exclude: (?x)^(
        #   regex for excluding notebooks
        # )$

Update: You'll have to add the following to your pyproject.toml in order to be compatible with Ruff:
[tool.isort]
profile = "black"

Update: You'll have to add the following to your pyproject.toml in order to be compatible with Ruff:
[tool.isort]
profile = "black"


[Gardening] SIL: Verifier messages use "lifetime".

Uses aren't life ending.

@swift-ci please test


Getting "manifest parsing has failed" when uploading to Teams

I have attached the Manifest file auto generated for the app template,
IceBreaker-manifest.zip
The App studio gave me this,

I don't understand what 'The file cannot be found' is referring too?

Hi, I ran into the same issue. I added en.json file to the zip file and replaced ourline.png with a new file with 32x32 in size. But no luck.

I had exactly the same issue.
I know this might not be ideal, but have you tried uploading to your tenant’s app catalog?

You can also upload it to your tenant's app catalog, so that it can be available for everyone in your tenant to install: https://docs.microsoft.com/en-us/microsoftteams/tenant-apps-catalog-teams
This method worked for me.

From: App-Dev-365 @.>
Sent: 08 September 2021 02:28
To: OfficeDev/microsoft-teams-apps-icebreaker @.>
Cc: csgdesk @.>; Author @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-icebreaker] Getting "manifest parsing has failed" when uploading to Teams (#191)
Hello,
I run into the same issue. I tried both automatic deployment (via powershell script) and manual deployment. Both methods failed. When I upload the custom app in Teams app, it threw the error "Manifest parsing has failed." I validated it in App Studio, which indicated an error message of "The file could not be found in the app package". But I don't know which file is missing. Below are the files in my package zip file. Any suggestion on what's going on?
Thanks!
[image]https://user-images.githubusercontent.com/26255679/132431029-4ed575b0-dfa6-4d87-9361-efc2145c0acd.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-apps-icebreaker/issues/191#issuecomment-914778337, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIWXSCT6PCF2PXIWLXGWWPDUA23Z3ANCNFSM5CKGWS7Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
Disclaimer
The Numerical Algorithms Group Ltd is a company registered in England and Wales with company number 1249803. The registered office is:
30 St. Giles, Oxford, OX1 3LE, United Kingdom.
This e-mail has been scanned for all viruses and malware, and may have been automatically archived by Mimecast Ltd, an innovator in Software as a Service (SaaS) for business.

I got the same error when uploading it to app catalog.

Had some similar issues today, trying to deploy IceBreaker to our tenant. Thanks to the hints here, I was able to resolve the issue:
Within the Manifest Folder, next to the Manifest.zip, there were a bunch of Language Jsons.
I just bundled the content of the manifest zip with all these language files to a new zip. This got uploaded succesfully.
Attached a screen of my new manifest zip

y.

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!

This is definitely still happening, thanks to those here who helped resolve it


Server Side Blazor g.cs files / component compiler optimization, minification

When compiling a Blazor solution, the components get compiled to files with extension g.cs.
Inside those files there're razor components converted to a rendertree, the problem I found is that the rendertree is polluted with lots and lots of lines like these:
__builder.AddMarkupContent(81, "\r\n                        ");
__builder.AddMarkupContent(18, "\r\n                        ");
__builder.AddContent(31, "                                ");

I've checked out some of my components,  and these lines add an extra of up to 50% more calls to the builder, which is a lot.
One example which I copied these lines from, had 172 calls to the builder from which 58 were only blank spaces and \r\n that don't change anything but I presume consume RAM and CPU for nothing useful in the final html output.
__builder.AddContent(155, "    ");
    __builder.CloseElement();
    __builder.AddMarkupContent(156, "\r\n    ");
    __builder.OpenElement(157, "div");
    __builder.AddAttribute(158, "class", "col-4");
    __builder.AddMarkupContent(159, "\r\n        ");
    __builder.OpenElement(160, "div");
    __builder.AddAttribute(161, "class", "card");
    __builder.AddMarkupContent(162, "\r\n            ");
    __builder.AddMarkupContent(163, "
\r\n
Log
\r\n
\r\n            ");
    __builder.OpenElement(164, "div");
    __builder.AddAttribute(165, "class", "card-body");
    __builder.AddMarkupContent(166, "\r\n                ");
    __builder.OpenComponent<Logger>(167);
    __builder.CloseComponent();
    __builder.AddMarkupContent(168, "\r\n            ");
    __builder.CloseElement();
    __builder.AddMarkupContent(169, "\r\n        ");
    __builder.CloseElement();
    __builder.AddMarkupContent(170, "\r\n    ");
    __builder.CloseElement();
    __builder.AddMarkupContent(171, "\r\n");
    __builder.CloseElement();

This is an example,  where probably more cpu & ram is consumed for generating white spaces and indentation than actual useful HTML for the browser, and this is an server side scenario could be a big waste in resources.
Regards!!

Thanks for contacting us, @chrdlx.
We intentionally avoid minification because in many situations customers need to preserve every space they add.


coverage debug sys can expose secrets

The coverage debug sys command shows lots of details of your system, including a subset of your environment variables.  Those can include secrets like PYPI_TOKEN.  These should be cloaked somehow.

FWIW, Django avoids showing settings with these words in them:
"API|TOKEN|KEY|SECRET|PASS|SIGNATURE"

https://github.com/django/django/blob/839d403e50eafa49a7e4a5a13ad818844ff9305b/django/views/debug.py#L113

This is fixed in 3fb8da56.

This is now released as part of coverage 7.2.6.


Mood Swing Animation[GSSOC'23]

Describe the project you want to add with tech stack

Hello Sir,
I want to add my new project of Mood Swing Animation into this repo.
Expected behavior

I have made a frog like face which is jumping from hills & swings it's mood with happy, sad & vice-versa.
Screenshots (optional)

Additional context (optional )

Please assign me this issue under GSSOC'23.

@MohitGupta121 @TusharKesarwani Please assign me this issue under GSSOC'23.

@MohitGupta121 @TusharKesarwani Please assign me this issue under GSSOC'23.


Duplicacy in cricketers data

About me:
Hey, I am Shivam Mishra a pre-final year open-source contributor.
Issue:
There is a duplicacy found in your cricketer's info list, for example, MS Dhoni's name with his info twice here, similarly, many more could be.
Solution:

If you assign me the task, I will check the full info list and remove all those duplicates which I will find here, because duplicate data make things worse than they shouldn't be.
Reference for proof:
I will attach the screenshot of that thing as a prove, so you can validate it before assigning me the task.

okay i have assigned this task to you can do the required changes

Crossing six weeks and 48 games, the celebration of white ball cricket has arrived as India has the thirteenth release of the ICC Cricket World Cup.
Britain comes into the competition as reigning champions after their undeniably exhilarating victory over New Zealand back in 2019.
Australia, South Africa and Pakistan are being upheld by savants to lift the prize on Nov. 19, however its the hosts - - right now positioned No. 1 on the planet - - that have been introduced as top choices to win the competition interestingly beginning around 2011.
Beneath, we'll frame the best live television real time features to use to watch each match of the 2023 Cricket World Cup, regardless of where you are on the planet.
Best apps to watch cricket world cup


Implement dynamic material example and dynamic geometry example

This pull request add the support for the following:
New examples

dynamic_geometry.py
dynamic_materials.py

Add support for interactive OpenGL examples:

Added the optix.sutil submodule to mimic the optix SDK sutil API (optix/SDK/sutil).
Added sutil.camera.Camera which implements all sutil/Camera.h features.
Added sutil.camera.Trackball which implements all sutil/Trackball.h features.
Added sutil.cuda_output_buffer.CudaOutputBuffer which implements sutil/CUDAOutputBuffer.h features (currently only supports CUDA_DEVICE mode, GL_INTEROP support may be added later once available in cupy).
Added sutil.gl_display.GLDisplay which implements all sutil/GLDisplay.h features.
Added sutil.gui which implements some GUI features of sutil/sutil.h (init_ui, display_text, display_stats).
Added sutil.vecmath which implements basic vector operations (dot product, cross product, normalize, length).
The python port closely follows the SDK by using the exact same graphical APIs:
1. OpenGL -> PyOpenGL
2. glfw ->  pyglfw
3. imgui -> pyimgui, pyimgui[glfw]

New features:
The two new examples required some additions to the existing wrapper:

Add a way to retrieve default build options to build custom user kernels: module.pyx:get_default_nvrtc_compile_flags().
OptixBuildInputInstanceArray.instances is now public to be able to update instances directly in python.
Instances that are modified in python in host memory can be updated in device memory with build.pyx:BuildInputInstanceArray.update_instance().
Allow to update traversable object reference and handle with build.pyx:BuildInputCustomPrimitiveArray.update_traversable().
Get a view on an instance transform matrix device memory with build.pyx:BuildInputInstanceArray.get_transform_view().
SbtRecord now store a list of ProgramGroup instead of a single ProgramGroup.
Program groups can now be updated with struct.pyx:SbtRecord.update_program_group().

Bug fixes

Fixed optix assertion in BuildInputCustomPrimitiveArray due to a CuPy stride bug when there was only one AABB passed as a numpy array (see https://github.com/cupy/cupy/issues/5897). This bug may affect other strides.
Fixed default visibility mask value from 2**(sizeof(unsigned int) * 8) - 1 to 2**max_visibility_mask_bits - 1.
Fixed Cython error on negative python int to unsigned int assignment (OPTIX_PRIMITIVE_TYPE_FLAGS_TRIANGLE = 1 << 31  is a negative C int affected to a python integer (enum.IntEnum.value). Optix expects unsigned int flags in its structs, which led to a Cython conversion error: cannot affect negative python int to unsigned int. This bug may affect other enums.
Other small fixes

Wow, really nice work!
I will review and merge this as soon as possible once I get back from my vacation.


fix: replace Instance Settings with Default Settings in i18n and docs

Which Problems Are Solved

The console shows the Instance Settings title and a description. Instance settings should be replaced with Default settings

How the Problems Are Solved

The DESCRIPTIONS.SETTINGS.INSTANCE.TITLE and DESCRIPTIONS.SETTINGS.INSTANCE.DESCRIPTION have been replaced with Default Settings and in the corresponding translation files.

Here's a screenshot:

Additional Changes

The docs have been updated to replace remaining texts mentioning Instance Settings with Default Settings
Two unused texts have been deleted from the translation files
VSCode with Prettier have applied some markdown changes

Additional Context

Closes #7632

Thanks @muhlemmer, swedish translations on the go then thank goodness I learned some Swedish at IKEA (I'll also fix the conflicts)


Compiler should detect and optimize loop invariants

=+  a=100
|-
?:  =(0 a)  a
=+  b=(add 1 2)
$(a (sub a b))

Stupid example, but it really shouldn't be evaluating b every single iteration, since it's static across all possible iterations.
@belisarius222 said that "if all you have is Nock 9's and no Nock 2's, maybe it's okay to optimize that. Probably easier to do in Hoon though."

Closing as I think this is too broad to be useful.  See:

This [..] falls into the scope of a much more sophisticated compiler.


Stuck on startup

This has happened to me a few times, basically open the app after not opening it for a while and it just stuck on the initial loading page (blue screen with logo) I have to force quit the app and restart and it works fine. Happened to me at 12:12pm PST on 1/28

Unfortunately if you are seeing a screen with only the logo and no other loading spinners, etc - that's just the splash screen for Android loading the app, and the app code is not even running yet. We'd need to look into what we could do to make loading better - maybe our bundle size is too large?


Error when trying to push file

I get the following error when I try to push a file:
Traceback (most recent call last):
  File "/usr/local/bin/pb", line 9, in <module>
    load_entry_point('pushbullet-cli==0.6.0', 'console_scripts', 'pb')()
  File "/usr/local/lib/python2.7/site-packages/click-5.1-py2.7.egg/click/core.py", line 700, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click-5.1-py2.7.egg/click/core.py", line 680, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click-5.1-py2.7.egg/click/core.py", line 1027, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/site-packages/click-5.1-py2.7.egg/click/core.py", line 873, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click-5.1-py2.7.egg/click/core.py", line 508, in invoke
    return callback(*args, **kwargs)
  File "build/bdist.macosx-10.11-x86_64/egg/pushbullet_cli/app.py", line 159, in push
  File "build/bdist.macosx-10.11-x86_64/egg/pushbullet_cli/app.py", line 65, in _push
  File "build/bdist.macosx-10.11-x86_64/egg/pushbullet/pushbullet.py", line 185, in upload_file
  File "build/bdist.macosx-10.11-x86_64/egg/pushbullet_cli/app.py", line 17, in wrapper
AttributeError: 'NoneType' object has no attribute 'decode'

I checked it with Python 2.7 in OS X 10.11 and it seems to work. Could you please give more details like the exact command that you ran or more information about the file (Is it large? Does its name contain non-English characters?)

Hmm, it was an executable (no extension). I also tried with a markdown file that failed as well. But a txt file worked.

Please check if 517e092c04bc70782ac1b415efe09b133bd6aa16 solves your issue

Worked! Thx! Btw, a short install instruction for us non-pythonists could be useful in the Readme. I had to look around a little before understanding how to install it. And it wont install with Mac's preinstalled Python, I had to brew install python.

I uploaded version 0.6.1 to PyPi which contains this fix, so you can stop using the develop branch

Great!


feat: add wrapper for unstable_getServerSession

Supersedes #289
cc @hnqg
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-08-15).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Adds a wrapper function for unstable_getServerSession, but removes the example of protecting a route as per @Fedeorlandau suggestion.

What'ya think @Fedeorlandau?
Do we need this?
Hope it was OK I continued on your work @hnqg.

What'ya think @Fedeorlandau?
Do we need this?
Hope it was OK I continued on your work @hnqg.

Yup, this is a nice feature to have. Makes using the server session much easier.


Fix for two bugs (including a fix for #229)

fix for Player being able to move during the Rival walking to its starter and during ledge hops

Fixes #229.