Reduce the memory pressure of JSON using Streaming extensions.

As the library manipulates JSON data, Its the best practices to use it as a stream rather than holding the entire memory for the same. More details can be found here.
https://www.stevejgordon.co.uk/sending-and-receiving-json-using-httpclient-with-system-net-http-json

Thanks Satish for this issue! :)
Will this improvement change the way of using meilisearch-dotnet? I mean is the getting started going to change after this improvement? 🙂

No @curquiza Its only internal change.


Somehow the library doesn’t work youtubeDL

I was looking for a bot for autoposting, but I got something wrong, here are the screens.

@sqLDak hey can you share a larger screen with specific error of ydl

@sqLDak hey can you share a larger screen with specific error of ydl

Yeah, but I’m busy after a while.


Cyclic reboot ESP32 Dev Module

Cyclic reboot when switching to engine.init(); . Arduino 2.3.4. Library 0.30.15.
ELF file SHA256: c69cac15c
Guru Meditation Error: Core  1 panic'ed (Unhandled debug exception).
 Exception happened during core dump!
Rebooting...
ets Jul 29 2019 12:21:46

Thanks for the report. Do you have some source code, which reproduces the issue ?

Example UsageExample.ino

Thanks for sharing. The init() does only perform few variable initialization. Does any sketch without FastAccelStepper work ?

Yes. Many projects have been written.

The problem here is

Thanks for pin pointing this down and I have overlooked that task creation is part of engine.init()
Anyway, this is another good example for one of nearly all Guru meditations, that the provided information is no help for debugging.
The xTaskCreate() is commonly used since years in FastAccelStepper and is expected to create a task successfully. That it fails in your setup is strange. Which esp32 do you use ?

ESP32-WROOM-32

Thanks for the info. This is just the base mode and should work. Please try platformio. This is more professional, less buggy and the IDE I use.

I reinstalled everything and now the following error.
E (21) rmt: rmt_new_tx_channel(216): mem_block_symbols must be even and at least 64
Error creation 258

Thanks for your patience and reinstalling has made progress. Now engine.init() does not fail. Pity, that it is not clear, what the root cause was.
The arduino library manager is buggy and 0.31.3 is not available. Which lib version of FastAccelStepper is now used in the reinstalled setup ?

0.30.15

please try to manually upgrade to 0.31.3.

Thank you. It works!!!


Update Program.cs

Summary
Describe your changes here.
Fixes #Issue_Number, dotnet/docs#Issue_Number or dotnet/dotnet-api-docs#Issue_Number (if available)

 Quality gate: Passed
 Everything looks good
Details


Fixed: read suppresses errors on overflow

This stops the “OSError: [Errno Input overflowed] -9981” from
interrupting program flow when doing listen_in_background().
I had issue #70 as well, this pull request fixes it.
See this Stack Exchange answer for a tiny bit more detail.

Specific PyAudio docs reference for more information on the fix.

Nice catch, much appreciated!


[feature] Simplify installation and launching at freedesktop compatible linux environment.

Modern linux distros have nice thing such as *.desktop files describing application to launch. Such files could be placed in a user local home directory to provide nice system integration.
For such integration to be made following conditions should be met:

A .desktop file should be created.
A link to executable should be put into $HOME/.local/bin
$HOME/.local/bin should be ensured to be in $PATH variable
Nice icon should be put into $HOME/.local/share/icons/hicolor

To do such a thing it would be nice to have a distro (*rmp, *deb, etc...) and installation scripts.
.desktop file example (derived from org.kde.application.dolphin.desktop) attached.
Archi.zip
Pass:1234

I'll close this because I'm not going to create another download option.


react-native inspector not Showing component file name and it's line number

Description
After updating to latest react native 0.71.7, not able to see component file name and with it's code line number.
React Native Version
0.71.7
Output of npx react-native info
System:
OS: macOS 13.3.1
CPU: (10) x64 Apple M1 Pro
Memory: 13.20 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.13.0 - ~/.nvm/versions/node/v18.13.0/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.19.3 - ~/.nvm/versions/node/v18.13.0/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.12.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.9862592
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
Languages:
Java: 11.0.18 - /Library/Java/JavaVirtualMachines/openlogic-openjdk-11.jdk/Contents/Home/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.7 => 0.71.7
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Step to reproduce :
go to app press D , will see a inspector option.
then inspect any element, observe only react native component are showing. we are not able to comment file name and code line number.
please help here.
Snack, code example, screenshot, or link to a repository

no file name and code line is there.i have scroll to it's edge also no file name with line was present.
please help here

after upgrade to 0.71.8. issue still persist.
System:
OS: macOS 13.3.1
CPU: (10) x64 Apple M1 Pro
Memory: 13.20 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.13.0 - ~/.nvm/versions/node/v18.13.0/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.19.3 - ~/.nvm/versions/node/v18.13.0/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.12.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.9862592
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
Languages:
Java: 11.0.18 - /Library/Java/JavaVirtualMachines/openlogic-openjdk-11.jdk/Contents/Home/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.8 => 0.71.8
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

Yes, facing the after react native upgrade to 0.71.7

I have this problem too since updating to 0.71

@mobster1940 can please confirm what was your previous react native version.

mark

It was 0.48 before

@mobster1940 can you please confirm what was your previous react native version.

It was 0.48 before

This is from a brand new React Native project. React Native 0.71.12 seems to have this feature missing.
If you can give some hints at what parts of the codebase are responsible for this feature, please reach out.

same issue here

I have this same problem in component inspector, and also the same info is not shown in component stack traces, example:
 ERROR  Error: Text strings must be rendered within a <Text> component.

This error is located at:
    in RCTView (created by View)
    in View (created by ListItem)
    in ListItem (created by Main)
    in RCTView (created by View)
    in View (created by Main)
    in RCTView (created by View)
    in View (created by Main)
    in RCTView (created by View)
    in View (created by Main)
    in RCTView (created by View)
    in View (created by Main)
    in Main (created by AppContent)
    in RCTView (created by View)
    in View (created by AppContent)
    in AppContent (created by App)
    in SesionProvider (created by App)
    in App
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in OrbitrixApp(RootComponent), js engine: hermes

Notice the missing file and line number, as opposed to the examples in https://morioh.com/a/50619ce875da/common-bugs-in-react-native
react: 18.2.0
react-native: 0.72.7
Should I downgrade to a previous react native version to have this feature? which is the last version where this feature was working? it really helps debugging a lot.

still missing?

I found a solution to this.
Before I export the component I need to set the displayName
The issue was due to wrapping my components with the observer from mobx.
So in every component I want to be able to inspect I basically add
MyComponent.displayName = "whatever name I want to see in the `inspector"
and then
export { MyComponent }

Same issue exist on "react-native": "0.73.8"
Any update?

Any update?

Any update ?


refactor: use utils.isString insteads of typeof

use utils.isString insteads of typeof

isString and similar helpers are primarily used in runtime code to reduce bundle size. Many of the cases here are in non-runtime packages (compiler or Node-only packages) so it doesn't matter.
isObject is also avoided where we already excluded null values first.


Problem in TChannel.cxx

I'm getting an EXC_BAD_ACCESS signal (either SEGSEGV or SEGBUS) on line 633 of TChannel.cxx using the latest version of leangrsi.
std::sort(chanVec.begin(),chanVec.end(),TChannel::Compare);
I don't see anything obviously wrong, so I suspect that memory is getting messed up elsewhere.  The error only appears on some machines.
I'm running: grsisort -aql --suppress_errors ../rawdata/offset_run03822_000.mid.bz2 (I've run bufferclean and offset fix on the midas file)

Can you use a version that isn't the latest version? We have been putting in a lot of changes the last couple of days and things might not be playing nicely.

On Jun 19, 2015, at 2:16 PM, gdemand<EMAIL_ADDRESS>wrote:
I'm getting an EXC_BAD_ACCESS signal (either SEGSEGV or SEGBUS) on line 633 of TChannel.cxx using the latest version of leangrsi.
std::sort(chanVec.begin(),chanVec.end(),TChannel::Compare);
I don't see anything obviously wrong, so I suspect that memory is getting messed up elsewhere. The error only appears on some machines.
I'm running: grsisort -aql --suppress_errors ../rawdata/offset_run03822_000.mid.bz2 (I've run bufferclean and offset fix on the midas file)
—
Reply to this email directly or view it on GitHub.

Also can you make sure there are data in your offset fixed file? Offset fix went bad for a bit during the grf2 bank change, I had an address check that was off by two units.

On Jun 19, 2015, at 2:16 PM, gdemand<EMAIL_ADDRESS>wrote:
I'm getting an EXC_BAD_ACCESS signal (either SEGSEGV or SEGBUS) on line 633 of TChannel.cxx using the latest version of leangrsi.
std::sort(chanVec.begin(),chanVec.end(),TChannel::Compare);
I don't see anything obviously wrong, so I suspect that memory is getting messed up elsewhere. The error only appears on some machines.
I'm running: grsisort -aql --suppress_errors ../rawdata/offset_run03822_000.mid.bz2 (I've run bufferclean and offset fix on the midas file)
—
Reply to this email directly or view it on GitHub.

I can work around the issue for myself, but thought that'd you'd like to know about it. I won't be able to fix this problem easily myself.
I'm still getting the same error with a raw .mid file.

Ok. I have a strong suspicion it comes from some changes we made this week with default channel numbers. LeanGrsi is going to be in main soon, so this should be fixed shortly.

On Jun 19, 2015, at 3:27 PM, gdemand<EMAIL_ADDRESS>wrote:
I can work around the issue for myself, but thought that'd you'd like to know about it. I won't be able to fix this problem easily myself.
I'm still getting the same error with a raw .mid file.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/366#issuecomment-113614358.

I haven't updated my version since June 8th if it helps.

Fixed as per pull request.


How do i use this exactly

how do i put the exe so it can obfuscate it?

whole project looks like an abandoned piece of junk.


Provide handler to remove border for frameless windows (Win32 only)

Provide handler to remove border for frameless windows (Win32 only)
Fixes https://github.com/brave/browser-laptop/issues/6258
Auditors: @bbondy, @bridiver

++ nice work!


Wrong initial drag element position

When the user clicks on an element to drag it,  the onLongPress function positions the draggable element on the top left of the screen.After the user triggers the onMove function, the position of the draggable element is getting fixed.
You can only notice this behavior if you click on element and you wait before you start dragging it.

Interesting... We will look into this. Thanks for the findings.


Emit OpenAPI 3.1 compliant specifications

We need to properly emit an standard compliant OpenAPI 3.1 API definition from TypeSpec.

This feature is something our team was looking into. We are hoping to be able to configure the version.

Pinging for updates. This is the only thing preventing our team from adopting TypeSpec.

I'm super excited about TypeSpec, but this and #2463 are blockers for us. They both seem important for the general adoption of TypeSpec. I'm a bit surprised that this is not supported already as OpenAPI 3.1 has been out there for a few years.

👍 glad to see it's added to this December milestone


[MAIN] Upgrade to v1beta1

Switched to v1beta1 in https://github.com/openshift-knative/serverless-operator/commit/5d6c2957660e4d1bba5647143b2e27bced4bd189. So there is no v1alpha1 any more.
Failures seen in #33

oc apply -f -
error: unable to recognize "STDIN": no matches for kind "KnativeServing" in version "operator.knative.dev/v1alpha1"

/assign @mgencur @nak3

/lgtm
/approve

/cherry-pick release-v1.8

/cherry-pick release-v1.7

/cherry-pick release-v1.6


Helm retry rollback fails when newly created resources are not cleared

Output of helm version: v3.10.3
Output of kubectl version: v1.25.3
Problem Description：
During Release rollback, if the resource waiting phase fails, Helm does not clear the newly created resources. As a result, subsequent rollback attempts fail.
1、During the first rollback, resources fail to be obtained because the network times out when waiting for the deployment to be ready. As a result, the rollback fails. Although CleanupOnFail is set to true, Helm does not clear the resources created during rollback.
2、During subsequent rollback retries, the resources created during the first rollback are not cleared. As a result, the subsequent rollback fails and the following error is reported:
warning: Rollback \"seq-gaussdb-demarcation-standby\" failed: no Service with the name \"seq-gaussdb-demarcation-standby\" found"

Cleanup on fail set, cleaning up 0 resources

an error occurred while cleaning up resources. original rollback error: no Service with the name \"seq-gaussdb-demarcation-standby\" found: unable to cleanup resources: object not found, skipping delete

Helm code:
https://github.com/helm/helm/blob/main/pkg/action/rollback.go#L234C4-L234C4
if r.Wait {
		if r.WaitForJobs {
			if err := r.cfg.KubeClient.WaitWithJobs(target, r.Timeout); err != nil {
				targetRelease.SetStatus(release.StatusFailed, fmt.Sprintf("Release %q failed: %s", targetRelease.Name, err.Error()))
				r.cfg.recordRelease(currentRelease)
				r.cfg.recordRelease(targetRelease)
				return targetRelease, errors.Wrapf(err, "release %s failed", targetRelease.Name)
			}
		} else {
			if err := r.cfg.KubeClient.Wait(target, r.Timeout); err != nil {
				targetRelease.SetStatus(release.StatusFailed, fmt.Sprintf("Release %q failed: %s", targetRelease.Name, err.Error()))
				r.cfg.recordRelease(currentRelease)
				r.cfg.recordRelease(targetRelease)
				return targetRelease, errors.Wrapf(err, "release %s failed", targetRelease.Name)
			}
		}
	}

Question：
When CleanupOnFail is set to true, can helm help clean up newly created resources when waiting for resources fails so that subsequent rollback retry succeeds?

I found that during the release upgrade, if the upgrade fails, the failRelease method provides a cleanup mechanism, but the rollback does not.


ConvertToSinglePlane memory overrwrite bug fix

ConvertToSinglePlane did not validate that the input images meet the height alignment requirements for the source format. This can result in a memory overwrite if untrusted content is passed from the DDS loader directly to this function.
This PR also includes additional hardening for the DDS loader, ComputePitch, and DDSTextureLoader modules to validate width & height requirements for planar video textures.

ConvertToSInglePlane bug


Unreleased Resource: Streams

https://github.com/Netflix/titus-control-plane/blob/f7a582d5c9a0b07b26fecb8c1acae8e04da6e8a5/titus-common/src/main/java/com/netflix/titus/common/util/PropertiesExt.java#L47
The program can potentially fail to release a system resource.

We are archiving this repo.


Add support for Ultimaker Format Package plugin

This plugin works just like #310 so we can add support for it with no extra work. Both plugins add the same new field thumbnail to the files API

@gdombiak this one doesn't yet support the new metadata field, but I was just finishing up that change and going to bed. I will be releasing an update tomorrow for UFP after some more testing. IF anyone wants to get ahead of the curve and do some testing for me, that would be great. Just use the URL below in plugin manager to install.
https://github.com/jneilliii/OctoPrint-UltimakerFormatPackage/archive/0.1.5.zip

Wow. I just created this ticket! :) I looked at the source code and saw the field so I assumed it was in production. My bad. No worries, I will release this code in a week or maybe more (depending on how many things I pack). Take your time to test the plugin! Thanks for all your plugins and hard work!
FYI, I already updated the wiki page with your new plugins.
Regards,
Gaston


[$500] Chat - The message got sent but it also stayed in the compose box

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open New Expensify app
Open any conversation
Quickly enter any text in the compose box
Tap the icon to send a message
Quickly repeat steps 3-4 several times

Expected Result:
The compose box should be scrubbed after each click on the send message icon
Actual Result:
The compose box doesn't have time to scrub after each click on the send icon and new text is added to the old text already sent
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: 1.3.59
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/3f44a226-4faa-4495-96e8-219185702ea1
https://github.com/Expensify/App/assets/78819774/1f004dc6-31e4-4452-a70d-fcde3d7443a0
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01bb269a50709f3aa2
Upwork Job ID:<PHONE_NUMBER>474398720
Last Price Increase: 2023-09-06

I feel like this is an age-old issue that we used to run into a LOT.
https://github.com/Expensify/App/assets/22508304/13ddd9cc-dc69-4a57-a4fd-d4f7e358ec66
Checking on in #expensify-open-source
https://expensify.slack.com/archives/C01GTK53T8Q/p1693872831763299

@NicMendonca  I'm off this week, can you please keep 👀 on this then I'll snag it back on Monday?  Thx

You can either wait for a brief moment (e.g., 1-2 seconds) after sending or clear the input field after clicking the button.
i don't have connects in upwork

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/en-gb/freelancers/~01d68a22e5d906bca6

Contributor details
Your Expensify account email: https://new.expensify.com/a/15606409
Upwork Profile Link: https://www.upwork.com/en-gb/freelancers/~01d68a22e5d906bca6

You can either implement a brief delay (e.g., 1-2 seconds) to prevent the client from clicking the button while it's sending or change the button's icon to a loading indicator to indicate that the action is in progress.

When the Send button is pressed, it should get disabled to prevent unintended, multiple presses. Then, the message should be sent, and the composer box cleared. After this the Send button gets enabled again, ready for new messages.
Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01634d5f25c3c5b849?viewMode=1

https://github.com/Expensify/App/blob/0e836df546939a9d905fee03b31879242de38a3b/src/pages/home/ReportScreen.js#L241

/**
 * @param {String} text
 */
const onSubmitComment = useCallback(
  (text) => {
    // Disable the function temporarily
    setSubmitDisabled(true);

    // Wait for 1 second (you can adjust the delay as needed)
    setTimeout(() => {
      // Re-enable the function
      setSubmitDisabled(false);

      // Now you can perform the comment submission
      Report.addComment(getReportID(route), text);
    }, 1000); // 1000 milliseconds = 1 second
  },
  [route]
);

// In your component, you can track whether submission is disabled
const [submitDisabled, setSubmitDisabled] = useState(false);

// You can conditionally disable the button based on submitDisabled
// For example:
{/* <button onClick={() => onSubmitComment('Your comment')} disabled={submitDisabled}>Submit Comment</button> */}

Contributor details
Your Expensify account email: https://new.expensify.com/a/15606409
Upwork Profile Link: https://www.upwork.com/en-gb/freelancers/~01d68a22e5d906bca6

Thanks @NicMendonca. @aimane-chnaif, can you review the @h0u554m 's proposal above? I'm hoping this is a new discrete issue and not related to a bug from a year ago 🤞

Is it new

@mallenexpensify

Certainly, here's a refactored version of your problem statement, root cause analysis, proposed solution, and contributor details:
Problem Statement:
We need to address an issue where, upon multiple rapid clicks, the message is not reset to its default empty state when sending a message.
Root Cause Analysis:
The problem arises due to multiple presses on the send message button in quick succession.
Proposed Solution:
To solve this issue, we can implement a brief delay of 1-2 seconds to prevent the client from clicking the button while the message is being sent.
Alternative Solution Considered:
Another alternative solution is to change the button's icon to a loading indicator to indicate that the action is in progress.
Solution Implementation:
You can find the code implementation for this solution at this GitHub link.
Here's the refactored code for the solution:
/**
 * @param {String} text
 */
const onSubmitComment = useCallback(
  (text) => {
    // Disable the function temporarily
    setSubmitDisabled(true);

    // Wait for 1 second (you can adjust the delay as needed)
    setTimeout(() => {
      // Re-enable the function
      setSubmitDisabled(false);

      // Now you can perform the comment submission
      Report.addComment(getReportID(route), text);
    }, 1000); // 1000 milliseconds = 1 second
  },
  [route]
);

// In your component, you can track whether submission is disabled
const [submitDisabled, setSubmitDisabled] = useState(false);

// You can conditionally disable the button based on submitDisabled
// For example:
{/* <button onClick={() => onSubmitComment('Your comment')} disabled={submitDisabled}>Submit Comment</button> */}

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/en-gb/freelancers/~01d68a22e5d906bca6

@aimane-chnaif

@aimane-chnaif can you explain more

Okay 1s i will fix it

Problem Statement:
We need to address an issue where, upon multiple rapid clicks, the message is not reset to its default empty state when sending a message.
Root Cause Analysis:
The root cause of the issue is multiple rapid clicks on the "send message" button. These rapid clicks can interfere with the expected behavior, preventing the message from being reset to its default empty state.
Proposed Solution:
disable the button when onSubmitComment is called, perform the comment submission asynchronously, handle any errors, and then re-enable the button using the try...catch...finally pattern.
Solution Implementation:
You can find the code implementation for this solution at this GitHub link.
Here's the refactored code for the solution:
`
const onSubmitComment = useCallback(
async (text) => {
// Disable the function temporarily
setSubmitDisabled(true);
try {
  // Now you can perform the comment submission
  await Report.addComment(getReportID(route), text);

  // Comment submitted successfully, you can add any success handling here if needed
} catch (error) {
  // Handle any errors here
  console.error("Comment submission error:", error);
} finally {
  // Re-enable the function
  setSubmitDisabled(false);
}

},
[route]
);
`
Contributor Details:

Expensify Account Email: Your Expensify account email
Upwork Profile Link: Your Upwork Profile Link

@aimane-chnaif  i think that better what is your opinion please

@aimane-chnaif can you review @h0u554m 's updated proposal and provide feedback?
https://github.com/Expensify/App/issues/26347#issuecomment-1717316009

Proposal
Updated

Problem Statement:
We need to address an issue where, upon multiple rapid clicks, the message is not reset to its default empty state when sending a message.
Root Cause:
The root cause of this problem stems from the inefficiency of the existing compose box clearing process. It fails to execute quickly enough to match the pace of rapid, successive input and sending of messages. As a result, the compose box clearance process lags behind, leading to the unintended overlap of old and new message text
Proposed Solution:
disable the button when onSubmitComment is called, perform the comment submission asynchronously, handle any errors, and then re-enable the button using the try...catch...finally pattern.
Solution Implementation:
You can find the code implementation for this solution at this GitHub link.
Here's the refactored code for the solution:

const onSubmitComment = useCallback(
  async (text) => {
    // Disable the function temporarily
    setSubmitDisabled(true);

    try {
      // Now you can perform the comment submission
      await Report.addComment(getReportID(route), text);

      // Comment submitted successfully, you can add any success handling here if needed
    } catch (error) {
      // Handle any errors here
      console.error("Comment submission error:", error);
    } finally {
      // Re-enable the function
      setSubmitDisabled(false);
    }
  },
  [route]
);

Contributor Details:

Expensify Account Email: Your Expensify account email
Upwork Profile Link: Your Upwork Profile Link

Proposal
Updated
Dag deeper into the code so hopefully my assessment is more accurate now.

Doubled to get more 👀 and proposals

@aimane-chnaif do you have a physical Android device you can test this on? I was unable to reproduce on v1.3.75-12 iOS native app

Bumped to $2000 and made external to see if we can get proposals THIS week. If not, I'll bring it back internal and try to rally an engineer to help

Proposal
Updated
Hopefully my solution is acceptable.

@aimane-chnaif Understood. Thanks for reviewing the proposal.
Just one thing, even though my solution doesn't fix the actual root cause, it does prevent the problem caused by that root cause from happening, and produces the expected result as requested in the body of the issue as:

Expected Result:
The compose box should be scrubbed after each click on the send message icon

Just a thought. It might be worth giving it a try before discarding it.
Cheers.

Proposal: Refactor ReportActionCompose and ComposeWithSuggestions for smooth message sending behaviour

Problem:
There is not one root cause of this issue but multiple. Following is the list which are the cause of the issue in composer, with first one having the highest participation in the root cause.

Report.addComment(getReportID(route), text) halts the main thread and due to which when user types in quickly and send a message and then continue typing, some parts of the message will still be in the input or the input won’t get cleared because of thread throttling.

Re-rendering ReportActionCompose and it’s children is not ideal here. isCommentEmpty is that state value which we can move around, more details in Solution.

draft related call saveReportComment is really expensive. This call is made when user starts to type and when hits the send button.

When click on send button, whole ReportActionCompose and it’s children gets re-rendered. Not ideal here as this is just an input based component and can probably live without a re-render.

With each key stroke ComposeWithSuggestions re-renders due to setValue and setSelection.

Solution:

Investigation in process on how to make addComment call more performant. ⌛

Move state for isCommentEmpty to a separate component, which will include SendButton and ComposeWithSuggestions

This is essential to avoid re-rendering ReportActionCompose directly and it’s children

Move draft related onyx calls when tapped on Send button to onClear

Especially saveReportComment since it takes a lot of time from setting the state til the re-render
Also, we have to analyse why saveReportComment is taking a lot of time since it also hinders the toggling of Send button disable status. ( This is partially fixed by https://github.com/Expensify/App/pull/28469 )
If you'll comment out saveReportComment, with the above PR change the toggle becomes really fast.

When click on send button, whole ReportActionCompose and it’s children gets re-rendered.

In theory, there’s no need to re-render Compose because it’s not displaying any report messages, so ideally only the list should re-render

With each key stroke ComposeWithSuggestions re-renders due to setValue and setSelection. There are a couple of ways to reduce that.

Move setValue to the Composer directly and selection as well and let it handle everything and re-render itself. Which means removing setValue and setSelection from Parent and moving to Composer.
There are places where we need value and we can use ref for that.
Or we can just move the selection state to Composer, since it also has selection state internally, so we can combine them.
Since child components like Suggestions and SilentCommentUpdater uses value, we can add a debounce to setValue and refactor the above point.

In the attached videos, I have commented Report.addComment for both before and after. In after, I have applied only a couple of fixes from within the list.

Before
https://github.com/Expensify/App/assets/47336142/88f09ffe-3d37-4d7d-b6d7-129089df1b23

After
https://github.com/Expensify/App/assets/47336142/f809eb21-3e98-4317-a64a-ea985631959b

@kacper-mikolajczak can you please comment on this issue too.
Thanks everyone for the proposals, this was a tough issue and Callstack did a deep dive and identified some of the issues closer to root cause, if not the root cause itself (its a combination really).

Can I dig deeper in this one ?

@jeet-dhandha I believe Callstack engineers will start a PR soon so you can but this job is assigned to them

@mountiny I am present 👍

@kacper-mikolajczak to help set expectations, do you know when you might have a proposal ready to review? It sounds like this might be a complex fix

Hi @mallenexpensify, I am continuing the work Hur started with this proposal. I am currently double checking with my own measurments and finding the best suited implementation of solutions suggested in his proposal 👍
Do you think we need a brand new proposal, or can I based my work on Hur's one? Thanks!

Thanks for the update @kacper-mikolajczak !

Hi team!
Down below I present current state of the improvements. All changes are available in the PR https://github.com/Expensify/App/pull/29561.
Updates

memoized useDebounce callback to prevent re-rendering 🧹
personalDetails is now passed as a prop to Suggestion component through context instead of triggering withOnyx HOC for that on each key stroke. This enabled us to reduce rendering times when typing in the Composer from ~80ms to ~3ms on iOS simulator 🏎
removed withReportCommentDrafts HOC dependency from OptionRowLHNData component, that caused entire LHN list to be re-rendered when draft is being saved. With this improvement we avoid heavy calculation that is represented in the recording as a single spike in profiling trace. This needs to be further discussed if HOC is safe to be removed, so let's move this discussion into PR 👍

Results
Before
https://github.com/Expensify/App/assets/62747088/ee83ffe9-3ac6-4232-9821-95c722082320
After
https://github.com/Expensify/App/assets/62747088/807d2d18-9c99-4be0-b544-fc9a5678fdca
Let me know what you think about those improvements. It is still work in progress and I suspect there is more to be done in that matter 🔧  Have a nice weekend! 👋
CC @mallenexpensify

Hi @mallenexpensify,
with these changes we aim to solve parts of the issues brought up in original @hurali97's proposal.
In order not to lose the velocity, we decided that I will continue to work on implementation of aforementioned improvements and @hurali97 will focus on rest of the original proposal.
Existing PR will be merged separately and we will add more to it as we explore new findings 👍
Let me know what you think, thanks!

Thanks @kacper-mikolajczak , doesn't that mean there's be two PRs for this one issue? I don't foresee a problem, outside of some automation we have that might create some posts and such.  If needed, we could break this into two issues, but it doesn't seem like that's necessary

@mallenexpensify That's right, there is going to be two PRs, so we can work in parallel. We can split the issues if that makes more sense, let me know what you think 👍

PR  is in review

PR hit production forevvvver ago and I missed it cuz the Reviewing label wasn't removed and Overdue wasn't added.
https://github.com/Expensify/App/pull/29561#issuecomment-1805969919
First... sorry 😿
Second... I see both @situchan and @aimane-chnaif assigned here and they were both hired for the job.  Which is C+ and how much is due?  I accidentally paid each $2000.
Contributor+: @situchan paid $2000 via Upwork
Contributor+: @aimane-chnaif paid $2000 via Upwork.

I think its not done yet we still have this pr https://github.com/Expensify/App/pull/30168

Second PR got merged

@mountiny, both @situchan and @aimane-chnaif have been paid $2k, Is that correct?
What regression tests do we need for this??!?!

Not sure what are the best regression steps for this. I think its one of those we would raise over time if it comes back
Has there been reliable reproduction steps?

@situchan and @aimane-chnaif , from above, can you confirm this is correct?

Contributor+: @situchan paid $2000 via Upwork
Contributor+: @aimane-chnaif paid $2000 via Upwork.

I'm unaware of reliable reproduction steps.  @hurali97 @kacper-mikolajczak @aimane-chnaif @situchan , any of you know?

TestRail GH created - https://github.com/Expensify/Expensify/issues/371196

Not exactly sure about the amount.

I'm going to leave as-is then and close this. Comment/reopen if you disagree.


cargo-crev: 0.23.2 -> 0.23.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/crev-dev/cargo-crev/releases.
meta.description for cargo-crev is: "A cryptographically verifiable code review system for the cargo (Rust) package manager"
meta.homepage for cargo-crev is: "https://github.com/crev-dev/cargo-crev"
meta.changelog for cargo-crev is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.23.3 with grep in /nix/store/82s49vv473dsvsq864ngsh7scydws7yl-cargo-crev-0.23.3
found 0.23.3 in filename of file in /nix/store/82s49vv473dsvsq864ngsh7scydws7yl-cargo-crev-0.23.3

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cargo-crev

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/82s49vv473dsvsq864ngsh7scydws7yl-cargo-crev-0.23.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/82s49vv473dsvsq864ngsh7scydws7yl-cargo-crev-0.23.3
ls -la /nix/store/82s49vv473dsvsq864ngsh7scydws7yl-cargo-crev-0.23.3/bin

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

1 package built:

cargo-crev

Maintainer pings
cc @B4dM4n for testing.

Result of nixpkgs-review pr 186378 run on x86_64-darwin 1

1 package failed to build:

cargo-crev

On darwin intel:
  = note: Undefined symbols for architecture x86_64:
            "_CFURLSetResourcePropertyForKey", referenced from:
                cargo_util::paths::exclude_from_backups::h686f8e319fca6409 in libcargo_util-df7bf361a4847ee8.rlib(cargo_util-df7bf361a4847ee8.cargo_util.084eba0e-cgu.3.rcgu.o)
          ld: symbol(s) not found for architecture x86_64
          clang-11: error: linker command failed with exit code 1 (use -v to see invocation)


Add basic recycler implementation for 1.0

Now would be a great time to comment on names/APIs @dlasky / @anthonykoerber / @shuwen

@jcmoore this is old now right? can we close it?

yep


Fix default site name

No ticket for this one

Description of Changes Made
This is a small fix to change the default site name in Social Media Settings

MR Checklist

[x] Add a description of your pull request and instructions for the reviewer to verify your work.
[ ] If your pull request is for a specific ticket, link to it in the description. No ticket
[x] Stay on point and keep it small so the merge request can be easily reviewed.
[x] Tests and linting passes.

Unit tests

[ ] Added
[x] Not required

Documentation

[ ] Updated build docs
[ ] Updated editor guidelines (https://docs.google.com/document/d/1PAWccdQ4tfaZsrEWmpDhvP3GH5RRmBOARFVp4b-kje8/edit?usp=sharing)
[x] Not required

Browser testing

[ ] I have tested in the following browsers and environments (edit the list as required)

Latest version of Chrome on mac
Latest version of Firefox on mac
Latest version of Safari on mac
Safari on last two versions of iOS
Chrome on last two versions of Android

[x] Not required

Data protection

[ ] Not relevant
[x] This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Accessibility

[ ] Automated WCAG 2.1 tests pass
[ ] HTML validation passes
[ ] Manual WCAG 2.1 tests completed
[ ] I have tested in a screen reader
[ ] I have tested in high-contrast mode
[ ] Any animations removed for prefers-reduced-motion
[x] Not required

Sustainability

[ ] Images are optimised and lazy-loading used where appropriate
[ ] SVGs have been optimised
[ ] Perfomance and transfer of data considered
[ ] If JavaScript is needed alternatives have been considered
[x] Not required

Pattern library

[ ] The pattern library component for this template displays correctly, and does not break parent templates
[ ] The styleguide is updated if relevant
[x] Changes are not relevant the pattern library

Note
This is also done on the previous build at https://github.com/torchbox/wagtail-torchbox/pull/568


[Idea][project]Link a task to a map tag

I often want to associate a task with a location (e.g. build a wall at a choke point), and indeed map tags are a good alternative to an explicit todo list. - its what I used before I discovered this mod, actually!
This is obviously a large undertaking, but I think I can break it down to a few steps:

determine a way to link a task to a map tag under-the-hood (e.g. I imagine map tags have some kind of global id -  YARM works with them a bit, that might be a good place to research)
Add an option to automatically make a todo list task when a map tag is manually created.
(not sure if possible) Add a 'link map tag' button to the task edit/create popup to allow the user to link a tag after the task has been created
Automatically delete the map tag once the linked task is completed (it probably won't be possible to restore a map tag if the task is un-completed - I'm fine with that)

I'm planning to work on this myself, so code review and approval is probably the only support that's needed, and I'll make that as easy as I can :)

There is already #34 which kinda has the same goal.
I thought about adding a optional map UI component to the subtask detail screen that you can point to things you want.
But of course a map tag would be way cooler :)

cool. I'll copy this stuff to that issue and close this one. No need for a dupe :)

dupe of #34


Call stack: ensure thread status is displayed when label is long

When debugging a java process, and presumably in other situations, the displayed thread label may overflow the current size of the stack frame window.  It isn't that obvious what the state of the running debug session is in that case.

Expanding the call stack window way out shows the state.

It seems that a relatively minor CSS fix (if there is such a thing) like max-width:80%  on something would allow the thread state to always be displayed in the call-stack.
<edit> fixed to say "thread state"

Dup of https://github.com/Microsoft/vscode/issues/68276


Transation: virtual-threads.adoc.po

Hi!
I am working on virtual-threads.adoc.po guide.
@gastaldi are there any recommendation about translating "carrier thread", "pinning" and terms like that to Portuguese? Just use the English term?
I think it might be easier to learn when you read the original term, like "carrier thread" insteadd of "thread de transporte", "thread de suporte"...

Sure, using the English term is fine in this case


[bug] Provenance subject<EMAIL_ADDRESS>does not match the package<EMAIL_ADDRESS>

Describe the bug
I just set up the nodejs builder to publish my package with provenance; I had to use the main version because the npm version used in the published workflow is too old and fails, but when I do get it running, it appears to fail because the subject and the package do not match.
To Reproduce
Steps to reproduce the behavior:
npm notice Publishing to https://registry.npmjs.org/ with tag latest and public access
npm verb stack Error: Provenance subject<EMAIL_ADDRESS>does not match the package<EMAIL_ADDRESS>npm verb stack     at verifyProvenance (/tmp/tmp.FXPIVTdzY6/node_modules/npm/node_modules/libnpmpublish/lib/provenance.js:216:11)
npm verb stack     at async buildMetadata (/tmp/tmp.FXPIVTdzY6/node_modules/npm/node_modules/libnpmpublish/lib/publish.js:181:26)
npm verb stack     at async publish (/tmp/tmp.FXPIVTdzY6/node_modules/npm/node_modules/libnpmpublish/lib/publish.js:45:44)
npm verb stack     at async otplease (/tmp/tmp.FXPIVTdzY6/node_modules/npm/lib/utils/otplease.js:4:12)
npm verb stack     at async Publish.exec (/tmp/tmp.FXPIVTdzY6/node_modules/npm/lib/commands/publish.js:123:7)
npm verb stack     at async module.exports (/tmp/tmp.FXPIVTdzY6/node_modules/npm/lib/cli-entry.js:61:5)
npm verb cwd /home/runner/work/hereby/hereby
npm verb Linux 5.15.0-1041-azure
npm verb node v18.16.1
npm verb npm  v9.8.0
npm ERR! Provenance subject<EMAIL_ADDRESS>does not match the package<EMAIL_ADDRESS>npm verb exit 1
npm verb code 1

https://github.com/jakebailey/hereby/actions/runs/5483129736/jobs/9989176122#step:3:399
Expected behavior
Publish succeeds.

Indeed, the sigstore file does contain this:
{
  _type: 'https://in-toto.io/Statement/v0.1',
  subject: [ { name<EMAIL_ADDRESS>digest: [Object] } ],
  predicateType: 'https://slsa.dev/provenance/v0.2',
  predicate: {
    builder: {
      id<EMAIL_ADDRESS>    },
    buildType: 'https://github.com/slsa-framework/slsa-github-generator/delegator-generic@v0',
    invocation: {
      configSource: [Object],
      parameters: [Object],
      environment: [Object]
    },
    metadata: { buildInvocationId: '5483129736-1', completeness: [Object] },
    materials: [ [Object] ]
  }
}

Must be somthing here that is thinking that my package is scoped: https://github.com/slsa-framework/slsa-github-generator/blob/ffbc1e5a1af0e70584a8aad5a3529b627fa03b32/internal/builders/nodejs/generate-layout.sh#L26

Yeah, this code doesn't quite work:
raw_package_scope=$(echo "${PACKAGE_NAME:-}" | cut -d'/' -f1)
raw_package_name=$(echo "${PACKAGE_NAME:-}" | cut -d'/' -f2)

When there's no /, both -f1 and -f2 give the entire string, meaning when PACKAGE_NAME=hereby, it ends up thinking my package is hereby/hereby.

Right. This was a mistake on my part. We need to handle packages without a scope properly. Should have used the -s flag...


test_plot_barriers failing locally

Information

Qiskit Terra version: master
Python version:
Operating system: osx

What is the current behavior?

FAIL: test_plot_barriers (python.visualization.test_circuit_matplotlib_drawer.TestMatplotlibDrawer)
Test to see that plotting barriers works - if it is set to False, no

AssertionError: 'Error: Image files did not match.\n  RMS Value: 3.844171635643379\n  Expected:  \n    /var/folders/rn/vtkjhjc14412w6lx6d292rdw0000gn/T/tmpltoyc87x.png\n  Actual:    \n    /Users/paul/Desktop/Github_repos/qiskit-terra/test/python/visualization/references/matplotlib_barriers_ref.png\n  Difference:\n    /Users/paul/Desktop/Github_repos/qiskit-terra/test/python/visualization/references/matplotlib_barriers_ref-failed-diff.png\n  Tolerance: \n    0.0001' is not None

Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

Could you provide the image files you generate when running this test? It is passing for me using master and osx at the moment.

Looking at this, it looks that maybe it is calling my installed version instead of the dev directory where the tests are located. I say that because those missing elements are part of a visualization refactoring

Actually that is not the issue. Here is what is being drawn, and looks to be basically the same as the reference.

This could be a whitespace issue - with different matplotlib backends on OSX sometimes there is more/less whitespace around the circuit than there is in the reference image. This test is being modified with the removal of temp files in #2514, so it might be worth waiting until that is merged and seeing if that fixes this issue.

This now appears to be resolved.


Add conversion traits to primitive types

e.g. to be able to write 10.samples() or 20.seconds() or 2.channels(). This should be opt-in via feature.

not doing this, because it requires explicit type, e.g. 2_i32.channels().


Activate the selector check for the autoprint integration test as soon as possible

The Windows bot is usually slower than the Linux bot, and therefore text layer rendering is as well. However, the autoprint test awaited text layer rendering to complete before activating the selector check, which makes it timing-sensitive and causes it to never resolve because the page is already printed (and the printed page div removed) by then.
This commit should fix the issue by activating the selector check as soon as possible, namely as soon as the viewer appears, which should ensure we're always registering the selector check in time because we're doing it even before rendering is starting.
Fixes #16965.

I have been able to reproduce the Windows bot issues locally on Linux by introducing a delay (as small as 100-200 ms was already enough) before adding the .endOfContent div to the DOM. However, let's see if the bots also agree now...
/botio integrationtest

/botio-windows integrationtest

/botio-windows integrationtest


Fix the two-way model binding

At this moment the plugin only gives you headache when you realize the ng-model is not updated by both callendar and your controllers.
And there is already a fork fixing it!
Please merge it.
https://gist.github.com/mwadams/3a950170ec0636fa1036

+1

I have created a minimalist version, with plunker here:
http://plnkr.co/edit/mAmr2DAhmXBtVBkWc5Kv?p=preview

+1

Sorry could not wait any longer.
I added the changes to my fork: https://github.com/atais/angular-eonasdan-datepicker
If requested I can make a pull request and merge again sometime later.

@atais What do mean 'the ng-model is not updated by both callendar and your controllers'?


README

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

Update READ ME file #11


ENH add upper bound of mamba2 for conda-forge-feedstock-check-solvable

Checklist

[x] Used a static YAML file for the patch if possible (instructions).
[x] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[x] Ran pre-commit run -a and ensured all files pass the linting checks.
[ ] Ran python show_diff.py and posted the output as part of the PR.
[x] Modifications won't affect packages built in the future.

================================================================================
================================================================================
noarch
noarch::conda-forge-feedstock-check-solvable-0.3.3-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.3.6-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.3.5-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.3.1-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.2.0-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.3.4-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.3.0-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.3.2-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.1.0-pyhd8ed1ab_0.conda
-    "mamba >=0.23",
+    "mamba >=0.23,<2.0a0",
noarch::conda-forge-feedstock-check-solvable-0.5.0-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.5.2-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.4.0-pyhd8ed1ab_0.conda
noarch::conda-forge-feedstock-check-solvable-0.5.1-pyhd8ed1ab_0.conda
-    "mamba >=1.5.1",
+    "mamba >=1.5.1,<2.0.0a0",


Changing the suggestions box content

Hello, I am looking to compile c# , by using Roslyn.
So I need to be able to change the suggestions box content.
Can I do that with this package or not yet ?

Hey, I had exactly the same issue and have searched through Roslyn, VS code, OmniSharp-vscode and OmniSharp-Roslyn. In the end I was able to utilize the CompletionService and the SignatureHelpService of OmniSharp-Roslyn. I have put together a small sample (with additional use of some Javascript):

https://github.com/Apollo3zehn/MonacoBlazorSample
Maybe this can be a starting point to add native support by BlazorMonaco with the need to use Javascript.

@Apollo3zehn Thank you for your feedback and help. That's a very nice project.
I also found this package which is only a preview for .net conf 2020
https://github.com/SteveSandersonMS/MonacoRazor/tree/master/MonacoRazor
can be downloaded as nuget too.

Thanks for that link. I have just watched the .NET Conf video, it is very interesting. But it seems that currently this project offers a more advanced Blazor component.
Unfortunately the sample project from SteveSanderson does not run on my computer. In the browser it fails to load https://localhost:5001/_content/MonacoRazor/main.js. For now I will continue integrating this project with Omnisharp but of course it would be great if Microsoft decides to provide an officially supported Monaco editor for blazor.

Yes this project exposes more monaco features. Let's hope MSFT makes a Blazor component for Monaco.
If you'd like to run steve sanderson's monaco editor :

Create a new blazor project with .net 5.0 using vs2019
Install the nuget package ,make sure to tick the pre-release checkbox
copy from MonacoRazor.sample the index page.

Thank you, I will try that!
I have added support for hover so there is a way to get the type of infered variables:

That's great. I'm just using roslyn for completion, it's a bit of head busting process :) .
Looking forward to see your project as a nuget !

v3.1.0 of BlazorMonaco supports completion item providers. You can check that out.


[Bug]: ConfigAttributeError in 001_getting_started.ipynb

Describe the bug
Hi, Anomalib Teams,
Firstly, thanks a lot for providing this wonderful tools!
I'm planning to use it for my current research for long term development.
I'm prototyping using the 001_getting_started.ipynb in Google Colab

However, I encountered "ConfigAttributeError" when I run the following codes:
datamodule = get_datamodule(config)
Any idea to solve this?
Thanks a lot in advance.
Dataset
MVTec
Model
PADiM
Steps to reproduce the behavior
Steps to reproduce:

Run the 001_getting_started.ipynb
Run all codes until datamodule = get_datamodule(config)

OS information
OS information:

Python version: [e.g. 3.8.10]
Anomalib version: [e.g. 0.3.7]

I'm using the default setting in the Colab.
Expected behavior
ConfigAttributeError
Screenshots
No response
Pip/GitHub
pip
What version/branch did you use?
0.3.7 (the def
Configuration YAML
dataset:
  name: mvtec
  format: mvtec
  path: ./datasets/MVTec
  category: bottle
  task: segmentation
  train_batch_size: 32
  eval_batch_size: 32
  num_workers: 8
  image_size: 256 # dimensions to which images are resized (mandatory)
  center_crop: null # dimensions to which images are center-cropped after resizing (optional)
  normalization: imagenet # data distribution to which the images will be normalized: [none, imagenet]
  transform_config:
    train: null
    eval: null
  test_split_mode: from_dir # options: [from_dir, synthetic]
  test_split_ratio: 0.2 # fraction of train images held out testing (usage depends on test_split_mode)
  val_split_mode: same_as_test # options: [same_as_test, from_test, synthetic]
  val_split_ratio: 0.5 # fraction of train/test images held out for validation (usage depends on val_split_mode)
  tiling:
    apply: false
    tile_size: null
    stride: null
    remove_border_count: 0
    use_random_tiling: False
    random_tile_count: 16

model:
  name: padim
  backbone: resnet18
  pre_trained: true
  layers:
    - layer1
    - layer2
    - layer3
  normalization_method: min_max # options: [none, min_max, cdf]

metrics:
  image:
    - F1Score
    - AUROC
  pixel:
    - F1Score
    - AUROC
  threshold:
    method: adaptive #options: [adaptive, manual]
    manual_image: null
    manual_pixel: null

visualization:
  show_images: False # show images on the screen
  save_images: True # save images to the file system
  log_images: True # log images to the available loggers (if any)
  image_save_path: null # path to which images will be saved
  mode: full # options: ["full", "simple"]

project:
  seed: 42
  path: ./results

logging:
  logger: [] # options: [comet, tensorboard, wandb, csv] or combinations.
  log_graph: false # Logs the model graph to respective logger.

optimization:
  export_mode: null #options: onnx, openvino

# PL Trainer Args. Don't add extra parameter here.
trainer:
  accelerator: auto # <"cpu", "gpu", "tpu", "ipu", "hpu", "auto">
  accumulate_grad_batches: 1
  amp_backend: native
  auto_lr_find: false
  auto_scale_batch_size: false
  auto_select_gpus: false
  benchmark: false
  check_val_every_n_epoch: 1 # Don't validate before extracting features.
  default_root_dir: null
  detect_anomaly: false
  deterministic: false
  devices: 1
  enable_checkpointing: true
  enable_model_summary: true
  enable_progress_bar: true
  fast_dev_run: false
  gpus: null # Set automatically
  gradient_clip_val: 0
  ipus: null
  limit_predict_batches: 1.0
  limit_test_batches: 1.0
  limit_train_batches: 1.0
  limit_val_batches: 1.0
  log_every_n_steps: 50
  max_epochs: 1
  max_steps: -1
  max_time: null
  min_epochs: null
  min_steps: null
  move_metrics_to_cpu: false
  multiple_trainloader_mode: max_size_cycle
  num_nodes: 1
  num_processes: null
  num_sanity_val_steps: 0
  overfit_batches: 0.0
  plugins: null
  precision: 32
  profiler: null
  reload_dataloaders_every_n_epochs: 0
  replace_sampler_ddp: true
  sync_batchnorm: false
  tpu_cores: null
  track_grad_norm: -1
  val_check_interval: 1.0 # Don't validate before extracting features.

Logs
ConfigAttributeError                      Traceback (most recent call last)
[<ipython-input-15-9682ddf2d22a>](https://localhost:8080/#) in <module>
----> 1 datamodule = get_datamodule(config)
      2 datamodule.setup()  # Downloads the dataset if it's not in the specified `root` directory
      3 datamodule.prepare_data()  # Create train/val/test/prediction sets.
      4
      5 i, data = next(enumerate(datamodule.val_dataloader()))

8 frames
[/usr/local/lib/python3.8/dist-packages/omegaconf/dictconfig.py](https://localhost:8080/#) in _get_node(self, key, validate_access, validate_key, throw_on_missing_value, throw_on_missing_key)
    478         if value is None:
    479             if throw_on_missing_key:
--> 480                 raise ConfigKeyError(f"Missing key {key!s}")
    481         elif throw_on_missing_value and value._is_missing():
    482             raise MissingMandatoryValue("Missing mandatory value: $KEY")

ConfigAttributeError: Missing key val
    full_key: dataset.transform_config.val
    object_type=dict

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @yjwong1999, we recently merged #863 that addressed jupyter-related issues. Can you pull the latest changes and try again please?

Assuming that this has been resolved, I'm closing this issue. Feel free to re-open if you encounter any related issues.

Hi @samet-akcay
Thanks a lot for the solution!
I have tried with the latest changes, and it works.
Thanks a lot!


build workspace faild on 11/219 (apt-get update -yqq)

Info:

Docker version ($ docker --version): Docker version 18.06.3-ce, build d7080c1
Laradock commit ($ git rev-parse HEAD): 861ea6b16ea3c2e9706425f5f0a49cc7c8988d3d
System info (Mac, PC, Linux): Linux
System info disto/version: centos8.1095

Issue:

_____Dont know
Expected behavior:

_____want to build workspace
Reproduce:

Relevant Code:
// place a code sample here

Step 11/219 : RUN set -xe;     apt-get update -yqq &&     pecl channel-update pecl.php.net &&     groupadd -g ${PGID} laradock &&     useradd -u ${PUID} -g laradock -m laradock -G docker_env &&     usermod -p "*" laradock -s /bin/bash &&     apt-get install -yqq       apt-utils       libzip-dev zip unzip       php${LARADOCK_PHP_VERSION}-zip       nasm &&       php -m | grep -q 'zip'
---> Running in 583a1012a185

apt-get update -yqq
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/xenial/InRelease  Temporary failure resolving 'archive.ubuntu.com'
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/xenial-updates/InRelease  Temporary failure resolving 'archive.ubuntu.com'
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/xenial-backports/InRelease  Temporary failure resolving 'archive.ubuntu.com'
W: Failed to fetch http://security.ubuntu.com/ubuntu/dists/xenial-security/InRelease  Temporary failure resolving 'security.ubuntu.com'
W: Failed to fetch http://ppa.launchpad.net/ondrej/php/ubuntu/dists/xenial/InRelease  Temporary failure resolving 'ppa.launchpad.net'
W: Some index files failed to download. They have been ignored, or old ones used instead.
pecl channel-update pecl.php.net
Updating channel "pecl.php.net"
Channel "pecl.php.net" is not responding over http://, failed with message: Connection to pecl.php.net:80' failed: php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution Trying channel "pecl.php.net" over https:// instead Cannot retrieve channel.xml for channel "pecl.php.net" (Connection to ssl://pecl.php.net:443' failed: php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution)
ERROR: Service 'workspace' failed to build: The command '/bin/sh -c set -xe;     apt-get update -yqq &&     pecl channel-update pecl.php.net &&     groupadd -g ${PGID} laradock &&     useradd -u ${PUID} -g laradock -m laradock -G docker_env &&     usermod -p "*" laradock -s /bin/bash &&     apt-get install -yqq       apt-utils       libzip-dev zip unzip       php${LARADOCK_PHP_VERSION}-zip       nasm &&       php -m | grep -q 'zip'' returned a non-zero code: 1

Maybe cause by firewall.
Ref #2238, #2232

its ok？

@bestlong im not in china but I can not install


[Agent] Can skywalking support Hystrix

Please answer these questions before submitting your issue.
What version of sky-walking are you using?
3.2.4
What company or project?
just learn
What did you do?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

I will support this plugin in next version

expect the Hystrix plugin

@KaiwenWang Received your feedback. Hope @ascrutae will have time to do so. He is working on automatic tests now.

@ascrutae Pay attention for the multi threads in Hystrix. This component is really important in Spring eco system. But be advised, create span for Hystrix is meaningless, but tracing retry by creating new thread and propagate context, also log the exception it triggers. those are really matter.

Hystrix plugin has been merged. Closing this issue


EPersona Problem

Every time that I run this code -
"const SteamUser = require('steam-user');
const config = require('./config.json');
const client = new SteamUser();
const logOnOptions = {
accountName: config.username,
password: config.password
};
client.logOn(logOnOptions);
client.on('loggedOn', () => {
console.log('Logged into Steam');
client.setPersona(SteamUser.Steam.EPersonaState.Online);
client.gamesPlayed(440);
});"
I get this error -
"client.setPersona(SteamUser.Steam.ePersonaState.Online);
^
TypeError: Cannot read property 'ePersonaState' of undefined"
Is there a problem with that I have down?

client.setPersona(SteamUser.EPersonaState.Online);
Hope for it

client.setPersona(SteamUser.EPersonaState.Online);
Hope for it****

Seems to have worked for me!

i have the same problem

1 = online

1 = online

that doesnt help it still says TypeEror : ... EPersonaState

client.setPersona(SteamUser.EPersonaState.Online);
I think lol
On Sat, Apr 27, 2019, 12:33 AM spyM4trix<EMAIL_ADDRESS>wrote:

1 = online
that doesnt help it still says TypeEror : ... EPersonaState
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/andrewda/node-steam-guide/issues/94#issuecomment-487120076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK2BYINUGWT5QINARXG4SNTPSMVFBANCNFSM4GTWJDKQ
.

use this repo: https://github.com/DoctorMcKay/node-steam-user

This is correct
Client.setEPersona(SteamUser.Steam.EPersonaState.Online);

Same error, but if i try using the folder of this project installing dependencies, it's working

same error :/

sup, have the same problem: (new to this stuff)
(I have also tried SteamUser.Steam.EPersonaState.Online
const SteamUser = require('steam-user');
const SteamTotp = require('steam-totp');
const config = require('./config.json');
const client = new SteamUser();
const logOnOptions = {
accountName: config.username,
password: config.password,
twoFactorCode: SteamTotp.generateAuthCode(config.sharedSecret)
};
client.logOn(logOnOptions);
client.on('loggedOn', () => {
console.log('Success login');
client.setPersona(SteamUser.EPersonaState.Online);
});
C:\Users\acer\Desktop\bot\index.js:18
client.setPersona(SteamUser.Steam.EPersonaState.Online);
^
TypeError: Cannot read property 'EPersonaState' of undefined

@pringles4
Try removing the .Steam snippet of code from this

C:\Users\acer\Desktop\bot\index.js:18
client.setPersona(SteamUser**.Steam**.EPersonaState.Online);

to this

client.setPersona(SteamUserEPersonaState.Online);

I wanted to test this out, so i got in changed it.
If i type node index.js in nothing happens no "logged on" no  Errors I really don't knwo what to do

I got same problem, and finally I solved it by simply write it like this:

client.setPersona(1);


model id rename bug fix

@jasongrout  Because of a change in ipywidgets' WidgeModel (https://github.com/jupyter-widgets/ipywidgets/pull/1451) there was a bug in bqplot since we were relying on the id attribute of WidgetModel. This also fixes #517. Thanks @mpacer for reporting this.

phew good catch Sri

Whew, that was a lot! Thanks!

It looks like this fixed #517!


Parameter select autocompletion doesn't show a property linked to the given Label

Description:
When creating a parameter select, there is a problem when trying to find the property of a given Label.

The proposed properties are not those of the given label (it proposes every property since based on db.propertyKeys() which doesn't link the properties to the given Label
If your database contains a lot of properties containing the word you are looking for (in my case id) it's probable that your property will not be shown among the 5 proposed fields (I have to download the file and modify it manually, then import it back).

Tested version:

NeoDash version: 2.3.5
Neo4j Database version: 4.4.19 (Enterprise).

Steps to reproduce:

When selecting a Label, I want to see the autocomplete only on the properties of that label
When selecting a Label, I can see autocomplete on other properties containing the property I'm looking for, even not linked to the selected Label, and also, it is not shown since only proposing 5 properties.

Has this been completed? if not, there is a PR for this issue?


python.exe Application Error

Describe the bug
The az.cmd script won't run python.exe correctly. It responds with "The application was unable to start correctly (0xc000007b). Click OK to close the application."
To Reproduce
Install Azure CLI 2.0.41 or 2.0.42, attempt to run 'az login' from a cmd window in Windows.
Environment summary
Installed using the Windows MSI, running using cmd.exe on Windows 7 Pro, fully patched.

can you check out the eventlog and see any stack trace?

No stack trace is given. The attached is all that was in the event viewer:
error.txt

Is your Windows 7 32 bits or 64 bits?

64 bit.


The log function isn't being called in my custom transport

Please tell us about your environment:

winston version?

3.0.0-rc5

node -v outputs: 8.11.0
Operating System? Windows
Language? ES5

What is the problem?

I originally posted this to Stack Overflow last week, but no one helped me, so I'm also posting it here.
The log function in the custom transport isn't being called so the logs aren't being saved in my Electron app (the console.log('test') in the function is never output). I'm using electron-store to create the log files so that a proper JSON file is created. The custom transport worked before I started using winston.loggers, but I need to be able to save log entries from multiple files. It does generate the log entry because I also have it outputting to the console.
module.exports = {
    JsonDBTransport: class JsonDBTransport extends Transport {
        constructor(options) {
            super(options)

            if (options.level === "error") {
                this.logDBSystem = new Store({name: options.filename})

                if (this.logDBSystem.get('log') === undefined)
                    this.logDBSystem.set('log', [])
            }
            else {
                this.logDBUser = new Store({name: options.filename})

                if (this.logDBUser.get('log') === undefined)
                    this.logDBUser.set('log', [])
            }
        }

        log(info, callback) {
            let self = this

            setImmediate(function () {
                self.emit('logged', info)
            })

            let logDB

            console.log('test')

            if (info.level === "error")
                logDB = self.logDBSystem
            else
                logDB = self.logDBUser

            if (logDB !== undefined) {
                let log = logDB.get('log')
                log.push(info)
                logDB.set('log', log)
            }

            if (callback && typeof callback === "function")
                callback()
        }
    }
}

winston.loggers.add('logger', {
    format: combine(timestamp(), prettyPrint()),
    transports: [
        new module.exports.JsonDBTransport({ filename: userLogsPath + path.sep + logFile, level: 'info' }),
        new module.exports.JsonDBTransport({ filename: sysLogsPath + path.sep + logFile, level: 'error' })
    ],
    exitOnError: false
})

logger = winston.loggers.get('logger')
logger.info('test log')

What do you expect to happen instead?

Each log statement will be saved to a JSON file that depends on whether it's a general user action or an error.

The example you've provided seems to work for me. It writes the log message to the specified file through the configured store and it does put the test
message to the console. Make sure you've enabled the correct log levels in the
developer console of electron it might explain why the test message is not shown.

Actually, it suddenly started working again. I think maybe some Electron-related code was causing the problem. Thanks for looking into it.


New awesome repo

Can you add these repos to your list .. it's awesome 😉
https://github.com/microsoft/Xaml-Controls-Gallery
https://github.com/MaterialDesignInXAML/MaterialDesignInXamlToolkit
https://github.com/jordan-thoms/Retrofit.Net

Added the xaml controls gallery, the other two are either generic .net or WPF :)


📝 Recipe: Brunswick Stew

🔖 Recipe description
It's cold out so I'd like to try some stews.  Check this one out!
🎤 Context
Brunswick stew is a Southern dish that features a tomato base with beans, vegetables, and meat. Early Brunswick stews were often made with squirrel, rabbit, even opossum, basically whatever was available.  These days pork, chicken, and beef are common. The original thinking was to use local ingredients and those you have on hand, which remains the same today.
Serve this Brunswick stew with cornbread, pickles, and coleslaw or a salad on the side.
Some History on Wikipedia
✌ Possible Implementation
Quick Recipe I've made in the past (good starting point)
Yield: 20+ servings (lots)
Ingredients

1 stick unsalted butter
3 cups (2 large) diced sweet onions
2T minced garlic
1t cayenne pepper
1T ground black pepper
salt later (to taste)
1/4 cup Worcestershire sauce
1/2 cup vinegar BBQ sauce (Carolina Style)
1/2 cup sweet BBQ sauce (Texas Style)
1 pound pulled pork
1/2 pound smoked chicken (probably braised)
1/2 pound smoked turkey (same)
1 pound smoked chopped beef brisket
1 28oz can crushed tomatoes
1 15oz can corn
1 15oz can lima beans
1 qt (or more) chicken broth

Directions

Melt the butter in a large saucepan over medium heat
Add diced onions and garlic, saute until onions are translucent, about 12 minutes
Stir spices (cayenne pepper, black pepper, worcestershire) and simmer 5 minutes
Add BBQ Sauce
Stir in Pulled Pork, Chicken, Turkey, and Brisket
Add canned goods (vegetables)
Stir is chicken stock and simmer (about 2 hours)

Serve
Serve in a bowl with cornbread
👀 Have you tried this recipe before?  Had something similar?

Yes I've had this before.  Karen's cousin made it and it was pretty delicious.

Are you willing to submit PR?
Yes I am willing to submit a PR!

~3 cups (2 large) diced sweet onions~

Should we give Asafoetida a try?


Dracula for Kakoune

Dracula for Kakoune

A dark theme for Kakoune.

Sorry for the delay @alexherbo2. I just sent you an invite to join the Dracula team. Once you accept it, you can transfer the repository to this organization, and I'll create the official website page :)

It's live! https://draculatheme.com/kakoune


[BUG] "Delete page translation" delete the wrong translation

Description
On a page with two translations (for example: English and French) the CMS toolbar menu "Delete translation" does not always delete the selected translation (depending on query parameter language present in URL).

Steps to reproduce

Go to CMS toolbar "<Name of your site>", then "Pages..."
Click on the 👁 icon on a page with two translations (for example: English and French)
You are now on https://your-domain.tdl/a-page-with-two-translations/?edit&language=en
Go to "Language" menu on CMS toolbar, then "Delete translation", then choose "French..."

Expected behaviour
A modal appears, asking a confirmation to delete French translation.
Actual behaviour
A modal appears, asking a confirmation to delete English translation.
Additional information (CMS/Python/Django versions)

Django 3.2
django-cms 3.11.3

Do you want to help fix this issue?

[ ] Yes, I want to help fix this issue and I will join #workgroup-pr-review on Slack to confirm with the community that a PR is welcome.
[x] Sorry, I'm really short of time to work on this 😞 .

This issue still exists.

I can confirm the bug.
Here are my observations:
When the user clicks on the Delete translation > English button while the page url contains language=fr a request is sent to the following URL: http://<IP_ADDRESS>:8000/en/admin/cms/page/1/delete-translation/?language=fr&language=en&cms_path=/en/?edit (language parameter is added twice)
Then in the view here we have language parameter in the request as a list ( ["fr", "en"] and request.GET["language"] is "en" which is wrong.
I think we need to change the parameter name to something else then "language" for the "Desired language to delete the translation for" and keep the "Actual page editing language" as "language". I hope it is clear enough.
I can try to create a PR for this.


feat(rust):add ockam node logs subcommand

Current Behavior
No subcommand exists to get node logs
-Closes Issue #4129
Proposed Changes

-Added ockam node logs subcommand
Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

Let me know if there's anything I might have missed , also, I can change  the subcommand to add functionality to tail and read the logs. Currently, the subcommand prints out the node log path and returns it in Ok() as PathBuf.

Looks perfect @h20220145. I've only ask you to squash the two commits and do a rebase, then it will be ready to be merged!

Looks perfect @h20220145. I've only ask you to squash the two commits and do a rebase, then it will be ready to be merged!

Done


Folders do not exist

I have VS2022 installed and neither of the Interop folders mentioned on this page exist.

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

ID: 88fddd73-1fac-2149-71db-5f20c8c94934
Version Independent ID: 46e037ee-a5fd-7274-b072-8df92ce70eda
Content: Office primary interop assemblies - Visual Studio (Windows)
Content Source: docs/vsto/office-primary-interop-assemblies.md
Product: visual-studio-windows
Technology: office-development
GitHub Login: @John-Hart
Microsoft Alias: johnhart

@DreamU1952, did you select the Office Workload and Visual Studio Tools for Office when you installed VS 2022?  The workload is not selected by default.  If you did, the files should be installed to %ProgramFiles(x86)%\Microsoft Visual Studio\Shared\Visual Studio Tools for Office\PIA\ on a 64-bit machine.  If they're still not their please report this from inside Visual Studio using the Help > Send Feedback >  Report a problem menu

Thanks for responding. I did not select those components. But not sure why I would. The Workload called 'Office/Sharepoint Development says it is 'used to create Add-ins'. I do not want to create add-ins. I just want to open an Excel spreadsheet from C# and read/write some of the cells.
I see two Individual Components: 1) Office Developer Tools for VS and 2) Visual Studio Tools for Office which were not selected (probably because Office/Sharepoint Workload was not selected?). Do I need both of those? We do an offline install so I will need to download the right component for my offline layout, namely I assume 'Microsoft.VisualStudio.Component.TeamOffice'. Will that component include those two Individual components?
Frankly, I can already add a COM reference in VS2022 to 'Microsoft Excel 16.0 Object Library' so do I really need anything else? That works but I am assuming perhaps this page is describing an updated version I may want/should use??
Regards,
Jim Thoimas

From: John Hart @.>
Sent: Monday, April 11, 2022 9:12 AM
To: MicrosoftDocs/visualstudio-docs @.>
Cc: DreamU1952 @.>; Mention @.>
Subject: Re: [MicrosoftDocs/visualstudio-docs] Folders do not exist (Issue #7887)
@DreamU1952https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDreamU1952&data=04|01||5c24294146824cc073df08da1bd602c7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637852903244218606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=217lLuCN%2FCoj8ebCphJME4kJuwRDh9Co99UvE4t%2B%2B00%3D&reserved=0, did you select the Office Workload and Visual Studio Tools for Office when you installed VS 2022? The workload is not selected by default. If you did, the files should be installed to %ProgramFiles(x86)%\Microsoft Visual Studio\Shared\Visual Studio Tools for Office\PIA\ on a 64-bit machine. If they're still not their please report this from inside Visual Studio using the Help > Send Feedback > Report a problem menu
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fvisualstudio-docs%2Fissues%2F7887%23issuecomment-1095255944&data=04|01||5c24294146824cc073df08da1bd602c7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637852903244218606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=boxIdJZoEksOrLLFzCkCA8doW51CkL18RJkuI6s9L2A%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAP2Y7XFHN5B7YEAVGFVMMZTVERFNBANCNFSM5TC6GXNA&data=04|01||5c24294146824cc073df08da1bd602c7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637852903244218606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=cZ7iDs3Lgj9pCQl2hRVkkihjLvO4EhfK669wWEH5D18%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Can this compile to wasm?

Summary
I'm looking to import a ffmpeg wasm module and use it in combination with another go module moshpit, then compile that combination to webassembly.
Is that the right approach? Is this possible?
Is there a way to do this with go's native webassembly library?
My first thought was to create an i/o layer in javascript to send commands to/from the ffmpeg and moshpit, but this seems tedious to create and not easily extensible unless I create an exec implementation that works with this i/o layer.
I also saw this in the go source https://go-review.googlesource.com/c/go/+/290112/
but it seems it's limited and I can't import modules.
This module seems like it could be useful, but I doubt it would compile to wasm.
How should I approach this?

Hello,
Answering this kind of questions is out of the scope of this project, but let's give it a try because I'm curious!
So, ffmpeg.wasm is already a Wasm module, nothing to do on that side.
moshpit is a Go project. To get a Wasm module, we need to compile Go to Wasm. The Wasm backend of the Go compiler produces a Wasm with the WASI ABI but only tailored for the Web, i.e. to be consumed by JavaScript. It's an issue if you want to use it with wasmer-go because we support WASI but not the WASI-ish ABI produced by Go. However, Tinygo can also be used to compile Go to Wasm with a correct WASI ABI. I therefore encourage you to use Tinygo.

Both approaches sound good to me. To link multiple modules into a single one, I would recommend lld from LLVM. In the future, the Module Linking WebAssembly Proposal might be a thing, and then a third option will be availble to you.
After that, you can use wasmer-go to compile the Wasm modules, and then get instances from them.
How does it sound?

Hi, @AskAlice .
Looking at this question again at the end of 2021, the answer has actually changed a bit.
You can use tinygo and wasmer-go to solve this problem.
Regarding the solution, I submitted a simple demo, the code is very simple, so there is no need to describe too much:
https://github.com/wasmerio/wasmer-go/pull/315
If you want to know more details, you can read this technical blog, but it needs to be translated because I use Chinese for content writing. https://soulteary.com/2021/11/21/use-docker-and-golang-to-quickly-get-started-with-webassembly.html

thanks for pointing me in the right direction, i appreciate it

@soulteary I've followed your instructions, but it looks like something has changed since you wrote your article.
main.go
package main

func main() {}

//export multiply
func multiply(a int, b int) int {
  return a * b
}

tinygo build --no-debug -o simple.wasm -target=wasi -scheduler=none main.go

When I try to run it with wasmer-go I get:
Missing import: `wasi_snapshot_preview1`.`fd_write`

This is what I get with tinygo version:
tinygo version 0.26.0 linux/amd64 (using go version go1.18.1 and LLVM version 14.0.0)

I think it's because of the comment in this article: https://wazero.io/languages/tinygo/

Why does my wasm import WASI functions even when I don’t use it?
TinyGo has a wasm target (for browsers) and a wasi target for runtimes that support WASI. This document is written only about the wasi target.
Some users are surprised to see imports from WASI (wasi_snapshot_preview1), when their neither has a main function nor uses memory. At least implementing panic requires writing to the console, and fd_write is used for this.
A bare or standalone WebAssembly target doesn’t yet exist, but if interested, you can follow this issue.

Ah, that was the tip I needed, I can build it now with this:
tinygo build --no-debug -o simple.wasm -target=wasi -panic=trap -scheduler=none main.go

By doing -panic=trap it stops requiring painc() to do something clever. It just traps execution instead.


Release 2017-07-15

Coverage increased (+1.7%) to 81.272% when pulling d43ca86f0d5dc9d6cd9030630837e58bd94954e1 on develop into ec54f6918eec501c12f00b1161a46ec3d0ceacc1 on master.

Coverage increased (+1.7%) to 81.272% when pulling d43ca86f0d5dc9d6cd9030630837e58bd94954e1 on develop into ec54f6918eec501c12f00b1161a46ec3d0ceacc1 on master.


cannot search users on mobile

Android Lollipop Chrome v49.0.2623.105 cannot select the search box to find a specific user
also minor note, the buttons dont quite respond correctly to the small screen size

That is really strange. Will investigate


[Event Request] Function CopyTransLine in Codeunit 5704 "TransferOrder-Post Shipment"

Dear Team, we would like to have the following Publisher added to Function CopyTransLine in Codeunit 5704 "TransferOrder-Post Shipment":
local procedure CopyTransLine(var NewTransferLine: Record "Transfer Line"; TransferLine: Record "Transfer Line"; var NextLineNo: Integer; TransferHeader: Record "Transfer Header")
begin
    NewTransferLine.Init();
    NewTransferLine := TransferLine;
    [...]
    NewTransferLine.ResetPostedQty;
    NewTransferLine."Outstanding Quantity" := NewTransferLine.Quantity;
    NewTransferLine."Outstanding Qty. (Base)" := NewTransferLine."Quantity (Base)";
    OnBeforeNewTransferLineInsert(NewTransferLine, TransferLine);
    NewTransferLine.Insert();
    // START NEW EVENT
    OnAfterNewTransferLineInsert(NewTransferLine, TransferLine);
    // END NEW EVENT
end;

[IntegrationEvent(false, false)]
local procedure OnAfterNewTransferLineInsert(var NewTransferLine: Record "Transfer Line"; TransferLine: Record "Transfer Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Add Runtime Api support

Adds general runtime api support  via rpc call state_call:

add a RuntimeApi struct for a clean, separate runtime api interface. Otherwise, there would be overlapping function names.
add functions for commonly available runtime calls (full list can be viewed here: https://polkadot.js.org/docs/polkadot/runtime)
add system test to the testing examples where possible
add an example to show how to access the runtime api.

closes #520

I'm wondering if there is a way to check, if we are still up-to-date. Do you know if there is a file with the whole runtime api, from which we could generate tests? And if the generated test-file doesn't compile, we know something is wrong (?)
do you know what I mean?

We are actually doing that with our system tests in the testing crate. CI runs the newest substrate kitchensink node (https://github.com/scs/substrate-api-client/blob/master/.github/workflows/ci.yml#L176-L178) and runs all our function against the node (see https://github.com/scs/substrate-api-client/pull/703/files#diff-0cf9c44c6a61eeed79634c6ff344d8d096de895c5f48add5ce11a2ae343f2f71).  So long as we run our CI from time to time, we will know if we're up-to-date or not.
There's probably also a way to do it statically with the metadata - retrieve the metadata somehow from an up-to-date kitchensink node (not sure if Parity provide something like that or if we'd need to compile it ourselves) and use it to compare our functions. But I'm not sure what we'd gain from that compared to the already existing system tests.

The current tests will catch if an existing call changes its interface. But it will not notice, when there are new calls we should add. Generally it would be cleaner to have all of this automated. If the code for the runtime api was generated. Subxt is doing that (as far as I understand).
But I know, in this task we decided to write it manually. I just thought, there might be a way to automate the tests at least.
On the gains: automating these things is less error prone and reduces work for us in the long run.

Ah, I see. Yes, subxt generates the api code itself from the metadata. I suppose we could do the same for a completeness check. But that would be a separate issue, I believe.  Because if we decide to implement it, I'd not only do it for the runtime api but also for the rpc.

Ah, I see. Yes, subxt generates the api code itself from the metadata. I suppose we could do the same for a completeness check. But that would be a separate issue, I believe. Because if we decide to implement it, I'd not only do it for the runtime api but also for the rpc.

I agree. Also see my comment above. I think with these adjustments I propose there, we'd be well prepared for automating things at a later time.


ICS02 mock extensions + more tests in ICS02 and ICS03

Closes: #221

For contributor use:

[ ] Unit tests written
[ ] Added test to CI if applicable
[ ] Updated CHANGELOG_PENDING.md
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #222 into master will increase coverage by 25.6%.
The diff coverage is 56.8%.

@@            Coverage Diff            @@
##           master    #222      +/-   ##
=========================================
+ Coverage    13.6%   39.2%   +25.6%
=========================================
  Files          69      78       +9
  Lines        3752    5257    +1505
  Branches     1374    1794     +420
=========================================
+ Hits          513    2065    +1552
- Misses       2618    3044     +426
+ Partials      621     148     -473

Impacted Files
Coverage Δ

modules/src/events.rs
0.0% <ø> (ø)

modules/src/ics02_client/events.rs
0.0% <ø> (ø)

modules/src/ics02_client/raw.rs
0.0% <0.0%> (ø)

modules/src/ics03_connection/error.rs
11.5% <0.0%> (-21.8%)
:arrow_down:

modules/src/ics04_channel/error.rs
23.0% <0.0%> (-2.0%)
:arrow_down:

modules/src/ics04_channel/packet.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/client_def.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/error.rs
33.3% <0.0%> (+33.3%)
:arrow_up:

modules/src/ics07_tendermint/msgs/update_client.rs
0.0% <0.0%> (ø)

modules/src/mock_client/client_def.rs
0.0% <0.0%> (ø)

... and 124 more

Continue to review full report at Codecov.

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


Feature request: function to play song by index

Would simplify my use case significantly:

a stack of song arts is present, like on spotify
swiping left or right lets you skip to that song
a swipe runs a callback to get the index for the needed song
then, once you have that, you get to play the song, right?

But nope. I don’t see anything in the docs for this so far.

Can't you use Amplitude.getSongByIndex() and then Amplitude.playNow()?

To answer your question, @Bagera, yes! Sort of.
When you use playNow(), the track isn't played from the queue. It's like a temporary strap on.
You could use it. I chose a different way to do it, I think, which I can share if anyone is looking for it.
But it's hacky. That's the thing.

@mistermantas This is fixed in the dev branch here: https://github.com/521dimensions/amplitudejs/blob/dev/src/index.js

Use the Amplitude.playSongAtIndex()


Submitting forms in parent layout routes tries posting to wrong action.

What version of Remix are you using?
2.2.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
I have a layout route on
my-parent.$someParam.tsx
which adds a form and a action to handle the form.
inside that route there are three children:

my-parent._index.tsx
my-parent.$someParam.my-child.tsx
my-parent.$someParam.my-child._index.tsx

Now, when I am submitting the form on /my-parent/some-value/my-child, everything works as expected.
But if I am on /my-parent/some-value/my-child?index, and I try submit, I get the following error:

405 Error: Route "routes/my-parent._index.tsx" does not have an action, but you are trying to submit to it. To fix this, please add an action function to the route.

Expected Behavior
Forms should always submit to the action in their route, not some other route
Actual Behavior
For some reason, if i am on the ?index path of some child route, actions of parent layout routes also try to post on their respective index routes

In Remix, you can only post to a single route.
The URL /my-parent/:somParam/my-child matches both the layout and index routes. By default Remix will always post to the parent layout route. If you want to post to the index route, you need to include ?index. Typically you don't include that in the URL in the address bar, but as the <Form action> prop.
If you are including ?index, then Remix expects there to be an action in the index route. If you always want to submit to the layout route, then don't add the ?index param even when you're in the index route.
https://remix.run/docs/en/main/components/form#index

<Form method="post"> without an action prop submits to the contextual route.  If you need a <Form> in a child index route to submit to it's parent action you will need to add an action prop to the Form.

actions of parent layout routes also try to post on their respective index routes

I'm unable to reproduce this.  Here's a working example of submitting a parent form from a button in a child index route:
https://stackblitz.com/edit/remix-run-remix-wcmw5v
If I've misunderstood the issue, please feel free to re-open with a reproduction!

What happens when you rename this file:
parent.child._index.tsx

to this?
parent.child.tsx

Thanks for the reproduction!  I'll re-open and try to take a look at the repro repo soon


Origin/users/rahul/abuseipdb bugfix

Change(s):

Security Graph API requires targetProduct as 'Azure Sentinel' not 'Microsoft Sentinel'. Changed the value in playbook AbuseIPDB-BlacklistIpToThreatIntelligence

Reason for Change(s):

This bug was throwing an error during playbook execution.

Testing Completed:

Deployment and functional testing

Waiting from @v-amolpatil response. Thanks.


sol-fbp-to-dot: Fix handling of ports with the same name

When a node have input and output ports with the same
name, graph was wrong. It was using input ports
to connect as output.
So prefixing input ports ids with "IN" and output with "OUT"
should be enough.
I'm not touching labels.
@otaviobp ptal . After you changed oic generator
some samples were broken, as you may see at:
http://solettaproject.github.io/docs/nodetypes/#category_oic
Thanks
Signed-off-by: Bruno Dilly<EMAIL_ADDRESS>

+1.
Sorry, I couldn't note that this was incorrect after applying my patch. thanks for fixing.

np
merged
thanks


Support country/address fields for international transfers

International transfers to some countries need country and postal address fields (Switzerland at least it seems, others like Danske bank mark this field as optional).
These are currently not supported:
    <!-- from  -->
    <Dbtr>
      <Nm>XXXXX</Nm>
      <PstlAdr>
        <PstCd>75001</PstCd>
        <TwnNm>TwnNm</TwnNm>>
        <Ctry>CC</Ctry>
        <AdrLine>XXXXXX </AdrLine>
        <AdrLine>XXXXXX </AdrLine>
      </PstlAdr>
      <Id>
        <OrgId>
          <Othr>
          </Othr>
        </OrgId>
      </Id>
    </Dbtr>

It would be nice to support international DD from these countries.

I'd be very surprised if cross-country non-SEPA DD actually works, do banks really do that?
It'd be interested for CT, though, yes.

I'd be very surprised if cross-country non-SEPA DD actually works, do banks really do that?
It'd be interested for CT, though, yes.

These are actual SEPA DD transfers, as described here and here. For some reasons, some banks in those countries "just work" with "local"/EU rules, whereas others apply stricter checks. Beside Switzerland we have seen these with Finnish banks recently.
With GB, we just learned that due to Brexit as of 2021-12-31 SEPA transfers require the full address too.

Hello! Is there an option to put the address in these cases?
Thanks!

This finally is implemented now :)