Chainsaw

Description of changes
Closes : #291891

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Can we add list of chainsaw adopters somewhere to show user the potential of the chainsaw in Cloud Native .

Apologies @Sanskarzz, I made a mistake 😔 The suggestion to use finalAttrs applies to stdenv.mkDerivation but not to buildGoModule please revert those changes and squash the commits so that there is one commit for the added maintainer and one commit for the new package.

Can we please create new PR for this because i was using first time squash i have done some mistake too .

https://github.com/NixOS/nixpkgs/pull/292144

Closing in favor of #292144


Analysis continously restarts

STR:

Create map using a polygon dataset. I'm using barrios from the data library
Add a Filter points in polygons analysis
Choose "marias".becauseoffutbol (from the team org) as the target layer. I tried with other datasets, but something in this one is what causes the problem.

https://team.carto.com/u/javitonino/builder/ef5ccd36-53f1-11e6-8e4f-0ee66e2c9693/embed
cc @xavijam

This is also happening to some other people but I'm having a hard time replicating it...
@viddo @rochoa

The good thing about that marias dataset is that it reproduces 100% of the time (at least on my account), so it should help.

yay!
2016-07-27 16:12 GMT+02:00 Javier Torres<EMAIL_ADDRESS>
The good thing about that marias dataset is that it reproduces 100% of the
time (at least on my account), so it should help.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/9141#issuecomment-235597431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIENY7iBe6K2oTfZjI8-VzCsdaBPS_sks5qZ2dDgaJpZM4JWH5-
.

--
Sergio Alvarez Leiva

Taking a look...

@saleiva discovered that it is due to the table was not in cdb_tablemetadata.
Already working after a "touch" by @juanignaciosl.


problem on image size

Dear Authors,
Thanks for the amazing work.
I have some problem when the size of image h not equal to w,  when cal psnr on phase B the render image is [w,h], but the real img is [h,w], this problem also show on tensorboard 'rgb'
detail:
similarity.py
mse
value = (image_pred - image_gt) ** 2
Thanks for your help.

Are you using the most recently pushed code? The bug is fiexed in the commit "Fixed bug in dtu dataset (the order of image width and hight)" on Nov 11, 2021.

I used the latest code, but it does not converge when training phase B


Policy: Implement DenyAction

Overview/Summary
Implementation of DenyAction Delete into Azure Landing Zones for both Activity Logs and Diagnostic Settings as two seperate policy definitions and a policy initiative.
Address story AB#29619
This PR fixes/adds/changes/removes

Two policy definitions for DenyAction Delete Activity Logs and Diagnostic Settings
A policy initiative to apply these two policy definitions

Breaking Changes

N/A

Testing Evidence
To Be Delivered.
Testing URLs
Azure Public

Azure US Gov (Fairfax)

As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[X] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[] Performed testing and provided evidence.
[X] Ensured contribution guidance is followed.
[X] Updated relevant and associated documentation.
[X] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

Please redirect the PR to the policy-refresh-q1fy24 branch

Re-creating PR as merge conflicts aren't being resolved after resolving them.


Limit the published files

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add optional schema migrations; default to olm.bundle.object instead of olm.csv.metadata

… (#1384)

Allow disabling of migrations for bundles and sqlite DBs
intro new migrations package
schema migration optional; default to olm.bundle.object
review resolutions
use a type to protect migration interactions
review revisions

Upstream-repository: operator-registry
Upstream-commit: c80e8751e716c069017b55279a0f29d0254d426a

/jira cherrypick OCPBUGS-41540

/retest

/close Had to fix up a vendor. Done it in #874

/close


Week 4 exercise: Improve stream producer performance by using pipeline.

Drastically improves performance on the stream producer for the week 4 exercise by pipelining 1000 commands a time into Redis.

[ ] Check no HTML segments need updating
[ ] Check each stream's length and data
[ ] Make sure the consumer still works OK

Closes #35

Got performance up to 1 min 13 seconds to load all data using Docker container for Redis and my i9/64Gb RAM MacBook Pro.  Old code took 31 min and change on the same system ouch!


[Enhancement] Only fetch build logs if the build failed

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I added multiple integration tests (Spring) related to the features
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[ ] Client: I added multiple integration tests (Jest) related to the features
[x] Client: I documented the TypeScript code using JSDoc style.

Motivation and Context
Artemis (client) currently fetches build logs if it doesn't receive any feedback. This is a problem for exercises with hidden tests, as the build logs leak hidden test feedback.
Description
Client:

Only fetch build logs if we have a result and the submission is marked as failed. If no submission is available in the result object, attempt to fetch the results.
Consolidate to one service for fetching of build logs
Filter out static code analysis feedback before sending feedback to Orion
Provide separate services for each code editor instance in the exam

Server:

Fetch submission but don't fetch feedback (not used) for retrieval of build logs
Return forbidden if submission.buildFailed is false
Add integration tests

Steps for Testing

Log in to Artemis
Navigate to Course Administration
...

Screenshots

worked in the testing session :+1:

We tried out the issues with wrong build results from todays meeting for exam mode and it worked fine in the testing session :)


Unexpected stop because of check_convergence in stopping_criterion

When running benchmarks with multiple solvers and the params max_runs set, some solvers will end up producing a much lower number of iterates than max_runs because check_convergence makes them exit before their time. This might not be intended behavior for some users. Especially for DL people using mini-batch SGD like solvers where it often happens that the objective (test error) starts rising again before going down because of non-convexity (finding local minima then "escaping" making objective rise to find a better one afterwards) , variance or double-descent phenomenon
A couple of things that would help:

make it clearer that it stopped because of check_convergence in the logs
allow users to disable this check
implement more robust logic
explain more about what this check is doing in the docs (notably the role of patience and tolerance (EPS) and their default values)

Yes, I agree that this is a confusing part of benchopt, I would be happy to find a way to make it clearer/simpler.
The users can already change the way the checks are done by setting a class attribute stopping_criterion, as described in this part of the doc. We rework this not long ago, but I think this is still confusing as:

It is not clear what is the Solver.stopping_criterion, and how to overwrite it.
The default values are not described.
The default value to SufficientProgressCriterion(patience=3, eps=1e-10) is maybe not suited.

This choices also needs to take into account the selection of the proper sampling strategy, which is log scale for now but could be changed to linear by default. I would be happy to have your thoughts on this one.

I only found out about this part of the doc from your comment and actually had to "debug" this behavior by diving deep into the code. Maybe adding a simple example on how to override this stopping criterion using the least amount of code would help a lot new users. Indeed the interaction between this stopping criterion and the sampling rate should be emphasized.
Regarding the default frankly I have opinions but again they would be nice for DL people maybe not for the entire optimization community.


Code workflow

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Subit27 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Feat/popover-component

PR for early review before writing tests. The feature is related to #58

Codecov Report

Merging #75 (6d5f9d0) into main (6390c6b) will decrease coverage by 0.08%.
The diff coverage is 96.00%.

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
- Coverage   96.87%   96.79%   -0.09%
==========================================
  Files          20       22       +2
  Lines         288      312      +24
  Branches       71       76       +5
==========================================
+ Hits          279      302      +23
- Misses          9       10       +1

Impacted Files
Coverage Δ

src/components/Panel/PopoverPanel.tsx
94.73% <94.73%> (ø)

src/components/Panel/Panel.tsx
100.00% <100.00%> (ø)

src/components/Panel/index.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fixed missing BlankProject

Changes

Removed creation and deletion of BlankProject on Mac runners

Deleting the BlankProject on cleanup creates the following issue when calling the action multiple times in a row:
Couldn't set project path to:

Aborting batchmode due to failure:
Couldn't set project path to:

[Package Manager] Server::Kill -- Server was shutdown
Unclassified error occured while trying to activate license.
Exit code was: 1
Error: There was an error while trying to activate the Unity license.
Error: Build failed with exit code 1

There is no need to create the BlankProject as it already exists in the action.
Successful Workflow Run Link
This only affects self-hosted mac runners. Cannot share a workflow run here unfortunately.
Checklist

[x] Read the contribution guide and accept the
code of conduct
[x] Docs (If new inputs or outputs have been added or changes to behavior that should be documented. Please make a PR
in the documentation repo)
[x] Readme (updated or not needed)
[x] Tests (added, updated or not needed)

Any blocker to merge @GabLeRoux ?
I'd gladly look at the test runner action if I have some time.

The code that's being removed here doesn't fail by itself.

mkdir -p "$ACTIVATE_LICENSE_PATH" has the -p flag that makes sure it doesn't fail if the folder already exists.
rm -r "$ACTIVATE_LICENSE_PATH" will never fail because the folder is guaranteed to exist.

The change you're proposing doesn't actually fix any error as far as I can see. What am I missing?

cc: @GabLeRoux

I agree; removing those two lines doesn’t seem to address what’s mentioned in the description. If this issue persists, let’s open a discussion to confirm the right approach.
It’s possible we might be overlooking something, but I don’t currently see how these changes resolve what’s outlined. Let us know if we can help. :)


fix(iOS): title and title color handling added for refresh control

Summary:

Solve a part of this issue:  https://github.com/facebook/react-native/issues/46631
Changelog:

[IOS] [CHANGED] - Passed correct title and titleColor prop to updateTitle function
What's the Issue:
When updating the PullToRefreshViewProps in a React Native iOS app, changes to the title and titleColor were not being reflected properly in the RefreshControl. This happened because the function responsible for updating the title (_updateTitle) was not always receiving the correct or updated values for title and titleColor.
Updated _updateTitle function:
The _updateTitle method was modified to accept both title and titleColor as parameters. This ensures that the latest values are always used when updating the refresh control's attributedTitle.
If the title is empty, the attributedTitle is cleared by setting it to nil. Otherwise, both the title and titleColor (if present) are applied correctly.
Test Plan:

Is this a duplicate of https://github.com/facebook/react-native/pull/46628 ?


ADJST-707 review unused deductions

Can you add tests for all the changes in  remandRoutes.ts and taggedBailRoutes.ts.
Also there are no tests created for the unusedDeductionsRoutes.ts changes.


GROOVY-9915: use type(s) from static call site to resolve generics

https://issues.apache.org/jira/browse/GROOVY-9915

Merged. Thanks!

Merged. Thanks!


Debugging iOS 14 background message (when the app is closed)

Trying to build a chat app, which gets FCM messages and shows notifications even when the app is closed.
Clearly something in my top level handleBackgroundMessages function for FCM is wrong as the notifications are not showing (on a physical device- iOS emulators don't work with FCM).
In Android I've debugged a similar situation with adb logcat (and logs/print command).
Obviously adb isn't available for an iPhone, and I don't know how to see the logs after the app is closed (I've tried to look at the console at xcode but no Flutter messages are showing).
Because I can't run the app again after it's closed (see https://docs.flutter.dev/development/ios-14 the application can no longer be re-launched by tapping the application’s icon) I'm installing it on my device via "flutter install release".
How do I see the logs after install release, or otherwise how can I debug functionality (at least view flutter logs) when app is closed?

@yoniker
Since you seem to be using firebase_messaging plugin, it has a different repo for tracking: https://github.com/FirebaseExtended/flutterfire/issues.

How do I see the logs after install release, or otherwise how can I debug functionality (at least view flutter logs) when app is closed?

You can run the app in --verbose mode to see the logs. As far as debugging when app is closed, there should be a method in firebase_messaging to check whether the app is closed and what to do when it is the case.
Closing from here, as this isn't a Flutter issue.

Running the app in --verbose doesn't give me any logs after the app is closed, as explained in the original question.
"there should be a method in firebase_messaging to check whether the app is closed" - it is not the plugin's responsibility to enable me to see logs from Flutter. And of course it doesn't. It's Flutter's responsibility to enable the user to (at minimum) view logs.
Reopen the issue, as your answer seems to be invalid.


Add 'dev' branch to workflow triggers for push and pull requests

@CodiumAI-Agent /describe

Title
Add 'dev' branch to workflow triggers for push and pull requests

PR Type
configuration changes

Description

Updated the GitHub Actions workflow configuration to include the dev branch in the triggers for push and pull_request events.
Ensures that the build and test jobs are executed for changes in the dev branch.

Changes walkthrough 📝
Relevant filesConfiguration changes

build.ymlUpdate workflow triggers to include `dev` branch

.github/workflows/build.yml
Added the dev branch to the workflow triggers for both push and pull_request events.

+2/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information


Need to change googletest tag from master to main

According to googletest repo, tag has changed from master to main.
https://github.com/google/googletest/issues/3663
Can you change cmake configuration?
tests/gtest/CMakeLists.txt.in
aha-flow cannot build docker image.

@kongty Just fixed and merged to master

@rdaly525 Thanks!


Test PR

Test PR because PR test is failing again and again
Notes for Reviewers
This PR fixes nothing.
Signed commits

[✅] Yes, I signed my commits.

Codecov Report
Patch and project coverage have no change.

Comparison is base (2cb66e1) 5.73% compared to head (a63301a) 5.73%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #7319   +/-   ##
======================================
  Coverage    5.73%   5.73%
======================================
  Files         118     118
  Lines       16300   16300
======================================
  Hits          934     934
  Misses      15190   15190
  Partials      176     176

Flag
Coverage Δ

gointegrationtests
5.73% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/error.go
0.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fork paradox-material-theme?

As noted in https://github.com/apache/incubator-pekko-grpc/pull/36#issuecomment-1448137032, due to the usage of paradox-material-theme we are dealing with binary compatibility problems stemming from the fact that paradox-material-theme appears to be unmaintained for years and hence is built against an ancient version of circe.
I think the fastest solution to solve this problem for now is to fork the project, apply the changes in https://github.com/jonas/paradox-material-theme/pull/34 and just republish it another a different org.

Maybe we can work around the circe issue but in general forking is not super appealing

I already tried with no avail, we are dealing with a 6 year old version of Circe vs one that is quite recent

If we do it, we might just want to do it right here.

I was thinking of forking it into my own org (mdedetrich) just to unblock us, I don't see it taking more than a few hours (have done it before).
We can then look into a more appropriate solution later on

An alternative to fork might be to pull the styling files and paradox templates from paradox-material-theme into this repo. It would simplify the way we style our docs by reducing the layers from 3 (docs-> pekko-sbt-paradox -> paradox-material-theme)  to 2 (docs -> pekko-sbt-paradox).

For me, the real issue is #84. The requirement to 'release' UI changes and then go through all our repos and even multiple branches on repos and uptake the changes - that is a massive overhead for small UI changes.

@mdedetrich I think this issue can now be closed, given that you/we can maintain https://github.com/sbt/sbt-paradox-material-theme now and a new release is coming soon.


Supports allow_list

As Presidio supports allow_list, now this version of this repo supports it as well.
It also supports a permanent allow_list when initializing an instance.

Thanks, LGTM

Hi @roycboyc , can you please merge this?

Hi all, could you please incorporate this change?


[branch-2.7] Support setting KEY_BASED batch builder for Pulsar Sinks

Motivation

backports #11706 to branch-2.7

Modifications

include batchBuilder in ProducerSpec -> ProducerConfig.ProducerConfigBuilder conversion  since it was missing
support setting batch builder with "--batch-builder KEY_BASED" argument

Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

Thanks for your contribution. For this PR, do we need to update docs?

@Anonymitaet  No. This is a backport of #11706 which already includes the docs.


Decide on patch vs MagicMock

Is your feature request related to a problem? Please describe.
Currently, the tests are using two different ways of mocking (patch and MagicMock).
Describe the solution you'd like
There needs to be a decision on which mock to use. We should look into which suits the project better.

I think these cover different use cases. In both cases, a MagicMock object is passed to the code under test.
https://github.com/JonatanMartens/pyzeebe/blob/4202906527cd989ac80ba738dbb81f20ebf58ff0/tests/unit/grpc_internals/zeebe_workflow_adapter_test.py#L33
Here the code is designed to allow replacing with an alternative implementation (MagicMock), so it's easy to swap it out.
https://github.com/JonatanMartens/pyzeebe/blob/f83e992ede6fb11e80ebb2cb460bd295b2307947/tests/unit/worker/task_handler_test.py#L111
Here it's not easy (?) to replace the logger object, so patch is used (in this case, pytests caplog could be used instead though, but that's another discussion).
I do find the MagicMock example cleaner - it's closer to the dependency injection pattern. But it's not always going to be easy to use this pattern. I've almost always resorted to using patch in my projects, but that's because I haven't been very aware of DI pattern before.


Submarine 690. Enable pre-defined web interface in submarine workbench

What is this PR for?
Add pre-defined web interface.
The steps for users to create a pre-defined experiment are as following:

Click pre-defined button
Pre-defined data will be fetched from db
Render configurable params and spec on ng-modal
When the form is validated, the experiment can be submitted.

What type of PR is it?
[Feature]
Todos

[ ] Test predefined component (The e2e test failed now due to unknown reason)
[ ] Real-time fetch experiment status
[ ] The implementation logic of the experiment page should be improved (e.g. split components correctly, use resolver to avoid long-time waiting of data)

What is the Jira issue?
https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-690
How should this be tested?
https://github.com/ByronHsu/submarine/actions/runs/404158134
Screenshots (if appropriate)

@tangzhankun @pingsutw @xunliu
Please help me to review this PR. Thanks!

I will review as soon as possible~


Add additional data to node discovered event

Include the serial number, manufacturer and product from the
dmi catalog.

@keedya updated this with better error handling and a test.

@gavin-scott looks good. Will run our system testing and run it by the team.
It is a  +1 for me

Adding more reviewere:
@geoff-reid @anhou  @johren


[WIP] Fixes #25

Adds the ability to specify which platforms to hide a method for using the PythonHiddenAttribute.
I marked this as WIP because I would like some feedback on the naming and so forth so it's not confusing.

Naming things is hard  :smile:


python310Packages.mocket: 3.10.9 -> 3.11.0

Description of changes
Changelog: https://github.com/mindflayer/python-mocket/releases/tag/3.11.0
Fix build (https://hydra.nixos.org/build/209257363)

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages marked as broken and skipped:

python310Packages.pytest-ansible
python311Packages.pytest-ansible

25 packages failed to build:

ansible (python310Packages.ansible-core)
ansible-later (python310Packages.ansible-later)
ansible-lint (python310Packages.ansible-lint)
ansible_2_12
ansible_2_13
kargo
parsedmarc (python310Packages.parsedmarc)
python310Packages.ansible
python310Packages.ansible-compat
python310Packages.ansible-kernel
python310Packages.ansible-runner
python310Packages.geoip2
python310Packages.sopel
ttp (python310Packages.ttp)
python311Packages.ansible
python311Packages.ansible-compat
python311Packages.ansible-core
python311Packages.ansible-kernel
python311Packages.ansible-later
python311Packages.ansible-lint
python311Packages.ansible-runner
python311Packages.geoip2
python311Packages.parsedmarc
python311Packages.sopel
python311Packages.ttp

2 packages built:

python310Packages.mocket
python311Packages.mocket

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

2 packages marked as broken and skipped:

python310Packages.pytest-ansible
python311Packages.pytest-ansible

1 package failed to build:

python311Packages.sopel

26 packages built:

ansible (python310Packages.ansible-core)
ansible-later (python310Packages.ansible-later)
ansible-lint (python310Packages.ansible-lint)
ansible_2_12
ansible_2_13
kargo
parsedmarc (python310Packages.parsedmarc)
python310Packages.ansible
python310Packages.ansible-compat
python310Packages.ansible-kernel
python310Packages.ansible-runner
python310Packages.geoip2
python310Packages.mocket
python310Packages.sopel
ttp (python310Packages.ttp)
python311Packages.ansible
python311Packages.ansible-compat
python311Packages.ansible-core
python311Packages.ansible-kernel
python311Packages.ansible-later
python311Packages.ansible-lint
python311Packages.ansible-runner
python311Packages.geoip2
python311Packages.mocket
python311Packages.parsedmarc
python311Packages.ttp

Overlap with https://github.com/NixOS/nixpkgs/pull/216511

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

2 packages marked as broken and skipped:

python310Packages.pytest-ansible
python311Packages.pytest-ansible

1 package failed to build:

python311Packages.sopel

26 packages built:

ansible (python310Packages.ansible-core)
ansible-later (python310Packages.ansible-later)
ansible-lint (python310Packages.ansible-lint)
ansible_2_12
ansible_2_13
kargo
parsedmarc (python310Packages.parsedmarc)
python310Packages.ansible
python310Packages.ansible-compat
python310Packages.ansible-kernel
python310Packages.ansible-runner
python310Packages.geoip2
python310Packages.mocket
python310Packages.sopel
ttp (python310Packages.ttp)
python311Packages.ansible
python311Packages.ansible-compat
python311Packages.ansible-core
python311Packages.ansible-kernel
python311Packages.ansible-later
python311Packages.ansible-lint
python311Packages.ansible-runner
python311Packages.geoip2
python311Packages.mocket
python311Packages.parsedmarc
python311Packages.ttp


NextJS 13 Bug: Tracing Instrumentation breaks next/link elements with prefetching

Description
Enabling the Tracing Instrumentation in a Next13 project (App Router) with Faro breaks link that use prefetching (which is set to true for all links by default).
Steps to reproduce
I've included an example on CodeSandbox that you can fork to test it out on your own Faro Project :)

Fork this codesandbox project.

Create a Faro Project

Enter your project URL and project name into the corresponding environment variables in .env.local

Note that the <Link/> component on line 37 has it's prefetch set to true

Click on the Link

Note that the link has been clicked ('link clicked!' message in console) but no routing has occured (still on the same page)

Now set prefetch on the link on line 37 to false

Refresh the window

Click on the Link and notice that you are routed to the correct URL (now on test-route page)

Expected behavior
We expect that you are able to successfully route in both cases, with or without prefetching.
Actual behavior
Routing with next/link only works with TracingInstrumentation if prefetching is explicitly disabled for all <Link/> components
Environment

SDK version: 1.1.2
SDK instrumentations: WebInstrumentations, TracingInstrumentation
Device type: Desktop
Device name:  ASUS ROG Zephyrus G14
OS: Windows 11
Browser: Google Chrome 116

Demo
CodeSandbox Link
https://codesandbox.io/p/sandbox/crazy-matan-g5f27w?file=%2Fnext.config.js%3A1%2C1
Demo Video
https://github.com/grafana/faro-web-sdk/assets/59091763/aa8f3d1c-249d-496e-8e2b-7c738dc4852a
Context
Wasn't sure whether to report this on Faro repo, otel-js repo, or NextJS repo but if this is something that you all think belongs on there instead, I'll close this issue :)

same happening here, it would be nice to have a suggested configuration for nextjs projects too

I was thinking of adding a new issue since I'm not having the problem in a NextJS application, but this is too happening to me in a common React app with ReactRouter.
The thing is we're using the loaders functionality together with Await and Suspense components, which I think is causing the same problems as the prefetch prop in NextJS

For future reference: related to #219 and apparently could also impact Svelte (not verified, rumored)


Participant Details added!

This Pull Request resolves Issue #1
Description:
In this Pull Request we have done the following additions/edits for the code:

Are there any updates here or should we close the PR and label it as spam?

spam!


Add zwsp extension

Description

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Really small and basic extension. All it does is copy a zwsp to the clipboard.

Oh you’re totally right! I didn’t even know that was a thing. My bad.


Use GitHub Packages for storing containers to avoid HTTP 429 errors from Docker Hub in GitHub Actions

Quite a few times, GitHub Actions stops because Docker Hub triggers an HTTP 429 (too many request) error, like there: https://github.com/clemlesne/azure-pipelines-agent/actions/runs/7384635563/job/20087929649.
This could be solved by deploying the containers in GitHub packages, allowing lower latency, and no HTTP 429 errors. The packages could be deployed both in Docker Hub and GitHub packages.

ghcr.io also throws 429's

ghcr.io also throws 429's

Using GitHub Actions workflow on a public repository?

@amitie10g Not sure? We're seeing it from an Artifactory mirror

For using within GitHub Actions, the image must be pushed to GitHub Packages too, to avoid the issue described here.

As a workarround, I created some jobs and steps to pull the image from Docker Hub, then, push to GHCR, then, use the image from GHCR, then, tag as tonistiigi/binfmt, in order to get docker/setup-qemu-action pulling tonistiigi/binfmt without actually pulling from Docker Hub:
permissions:
  actions:  write
  contents: read
  packages: write

jobs:
  binfmt-to-ghcr:
    runs-on: ubuntu-latest
    steps:
    - name: Login to Docker Hub
      uses: docker/login-action@v3
      with:
        username: ${{ secrets.DOCKERHUB_USERNAME }}
        password: ${{ secrets.DOCKERHUB_TOKEN }}
    - name: Login to GHCR
      uses: docker/login-action@v3
      with:
        registry: ghcr.io
        username: ${{ github.repository_owner }}
        password: ${{ secrets.GITHUB_TOKEN }}
    - name: Get digests from tonistiigi/binfmt
      run: |
        I386_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "386") | .digest')
        AMD64_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "amd64") | .digest')
        ARMV6_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "arm" and .platform.variant == "v6") | .digest')
        ARMV7_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "arm" and .platform.variant == "v7") | .digest')
        ARM64_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "arm64") | .digest')
        PPC64LE_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "ppc64le") | .digest')
        RISCV64_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "riscv64") | .digest')
        S390X_DIGEST=$(docker manifest inspect tonistiigi/binfmt | jq -r '.manifests[] | select(.platform.architecture == "s390x") | .digest')

        echo "I386_DIGEST=$I386_DIGEST" >> $GITHUB_ENV
        echo "AMD64_DIGEST=$AMD64_DIGEST" >> $GITHUB_ENV
        echo "ARMV6_DIGEST=$ARMV6_DIGEST" >> $GITHUB_ENV
        echo "ARMV7_DIGEST=$ARMV7_DIGEST" >> $GITHUB_ENV
        echo "ARM64_DIGEST=$ARM64_DIGEST" >> $GITHUB_ENV
        echo "PPC64LE_DIGEST=$PPC64LE_DIGEST" >> $GITHUB_ENV
        echo "RISCV64_DIGEST=$RISCV64_DIGEST" >> $GITHUB_ENV
        echo "S390X_DIGEST=$S390X_DIGEST" >> $GITHUB_ENV

    - name: Push binfmt to ghcr.io
      uses: int128/docker-manifest-create-action@v2
      with:
        tags: |
          ghcr.io/${{ github.repository_owner }}/binfmt
        push: true
        sources: |
          tonistiigi/binfmt@${{ env.AMD64_DIGEST }}
          tonistiigi/binfmt@${{ env.ARM64_DIGEST }}
          tonistiigi/binfmt@${{ env.ARMV7_DIGEST }}
          tonistiigi/binfmt@${{ env.PPC64LE_DIGEST }}
          tonistiigi/binfmt@${{ env.RISCV64_DIGEST }}
          tonistiigi/binfmt@${{ env.S390X_DIGEST }}

  build:
    runs-on: ubuntu-latest
    steps:
    - name: Get digests from ghcr.io/<owner>/binfmt
      run: |
        AMD64_DIGEST=$(docker manifest inspect ghcr.io/${{ github.repository_owner }}/binfmt | jq -r '.manifests[] | select(.platform.architecture == "amd64") | .digest')
        ARM64_DIGEST=$(docker manifest inspect ghcr.io/${{ github.repository_owner }}/binfmt | jq -r '.manifests[] | select(.platform.architecture == "arm64") | .digest')
        echo "AMD64_DIGEST=$AMD64_DIGEST" >> $GITHUB_ENV
        echo "ARM64_DIGEST=$ARM64_DIGEST" >> $GITHUB_ENV

    - name: Pull binfmt from ghcr.io
      uses: int128/docker-manifest-create-action@v2
      with:
        tags: |
          tonistiigi/binfmt
        push: true
        sources: |
          ghcr.io/${{ github.repository_owner }}/binfmt@${{ env.AMD64_DIGEST }}
          ghcr.io/${{ github.repository_owner }}/binfmt@${{ env.ARM64_DIGEST }}

    - name: Set up QEMU
      uses: docker/setup-qemu-action@v3

I've used crane to clone both tonistiigi/binfmt and moby/buildkit into my GHCR account within all tags and platforms available, as a workarround for my  own builds and anyone's usage.
jobs:
  build:
  steps:
  - name: Set up QEMU
      uses: docker/setup-qemu-action@v3
      with:
        image: ghcr.io/amitie10g/binfmt:latest
    - name: Set up Docker Buildx
      uses: docker/setup-buildx-action@v3
      with:
        driver-opts: |
          image=ghcr.io/amitie10g/buildkit:master


siege: 3.0.8 -> 4.0.1

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Bump siege version after build was fixed (#15038).

Thank you


How do you grab other paraments via xpath other then identifiers?

I am trying to assert on values in certain boxes using the FlaUI library with Robot Framework. Can someone help me with the xpath identifier that I should use.
My xpath to find the specific element is:
'/Window[@Name='RigSense Main Client']/Window[@Name='Setup Charts']/Pane[1]/Pane[1]/Pane[3]/Edit[@AutomationId='txtPensPerChart']'
The screenshot shows the FlaUInspect for that element, I want assert that the "Value" parameter is '1'. So how do I either save that value parameter as a variable and then i can 'should be equal' to it or is there a better way?

Yes, I also want to know how to create xpath expressions. Where can I see the tutorial about xpath


Upon executing r.init(), Chrome opens, but nothing happens afterward; it just keeps hanging on r.init().

I have been using RPA for automation for over a month now, and it has been running smoothly. However, issues started arising from yesterday. Upon executing r.init(), it launches Chrome, but Chrome automatically closes, showing an error with r.init(). I reinstalled Chrome, but it didn't resolve the issue. Subsequently, I reinstalled the Python environment and RPA. Now, what happens is that Chrome opens after r.init() is executed, but there's no response afterward. Looking at the status in Jupyter, r.init() is constantly busy with no prompts whatsoever.

The operating system I am using is Windows 10

Hi @bwfyyy, it sounds like for some reason, the package cannot connect to the websocket connection to the Chrome browser. Is this a work laptop? It might be company IT policy or firewall or anti-virus blocking connection to Chrome's websocket. Can you try the following?

run in debug mode to see what happens

r.debug(True)
r.init()

There should be other command prompt windows when init() is run, one of them is for Chrome browser. Is there any message showing up there?


Add arquillian cube version.

Add arquillian cube version.

This is very old and I do not think it is needed anymore.


Runs for a day(?) and then hangs

So I've got some test code running on a Raspberry Pi2:

import minishift
import time
import sys
vid, pid = 0x04d8, 0xf517
width = 72
ms = minishift.Minishift(minishift.MCP2210Interface(vid, pid), width)
canvas = minishift.Canvas(width)
while True:
timestring = time.strftime(str(sys.argv[1]))
print '\r'+timestring,
sys.stdout.flush()
canvas.write_text(0, timestring)
ms.update(canvas)
for y in range (0,8):
for x in range (0,width):
canvas[x,y]=0
time.sleep(1)

And when I run it (either foreground or background):
python time.py "%a %T" &
It runs fine for maybe a day and then stops updating the display.  If I run it again in the foreground, with the background process 'stuck', I get this in the foreground:
Traceback (most recent call last):
File "time.py", line 14, in
ms.update(canvas)
File "/usr/local/lib/python2.7/dist-packages/minishift/draw.py", line 192, in update
self.interface.send(canvas.to_bytes())
File "/usr/local/lib/python2.7/dist-packages/minishift/draw.py", line 25, in send
self.device.transfer(data)
File "/usr/local/lib/python2.7/dist-packages/mcp2210/device.py", line 210, in transfer
Tue 06:11:23    self.transfer_settings = settings
File "/usr/local/lib/python2.7/dist-packages/mcp2210/device.py", line 63, in setter
self.sendCommand(set_command(value))
File "/usr/local/lib/python2.7/dist-packages/mcp2210/device.py", line 137, in sendCommand
response = command.RESPONSE.from_buffer_copy(response_data)
ValueError: Buffer size too small (0 instead of at least 4 bytes)
And then the background process starts running again!
Any idea what's up?  This ran for weeks with a 5-unit display using %X for the format string...
Thanks!

Correction, it's the background process that gives the above error message, and then the 'new' process starts running.
But it still hangs after a while, which it didn't do with just five segments, is there a way to do some kind of debugging?


[QUERY] Spring Cloud Azure 5.14.0 availability on Maven Central

Yesterday I've received a notification on Github about Spring Cloud Azure release 5.14.0.
I see that git tags were created last Friday.
However, appropriate artifacts are still inaccessible on Maven Central.
How long it usually takes from official release to General Availability?
Or there was some issues with actually releasing binaries to Maven?

Hi @OleksandrShkurat, thank you for reaching out.
There were some issues publishing to maven, we are trying to find and fix and sorry for the inconvenience.

Hi @OleksandrShkurat , now you can find them on maven


Menuhandlers should return promises.

The process in the menu handlers sometimes does some ascync stuff, but does not use promises. This results in weird erros. E.g.
menuHandler.createMultiModel = (path) => {
    $(init.mainView).load("multimodel/multimodel.html", (event: JQueryEventObject) => {
        let project: IProject = require("electron").remote.getGlobal("intoCpsApp").getActiveProject();
        if (project != null) {
            let name = Path.basename(path, ".sysml.json");
            let content = fs.readFileSync(path, "UTF-8");
            let mmPath = project.createMultiModel("mm-" + name + " (" + Math.floor(Math.random() * 100) + ")", content);
            IntoCpsApp.getInstance().emit(IntoCpsAppEvents.PROJECT_CHANGED);
            menuHandler.openMultiModel(mmPath + "");
        }
    });
};

menuHandler.openMultiModel(mmPath + ""); This results in an async operation, and therefore it has to be last in the method, because we do not wait for it. Therefore it should return a promise, so waiting is possibly.

@CThuleHansen , I don't think we ended up doing anything with this. I want to archive for after INTO-CPS. What do you think?

Yes archive.

Done


Drop the charge_taxes_on_shipping flag from site settings.

In the new approach we don't use the global charge_taxes_on_shipping flag, instead we check if there is a tax class assigned to the shipping method and use this tax class's rate.

For Avalara we keep getting the shipping code from the plugin config; but we change the way to determine whether the tax should be added, based on the above assumption.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-6794 Add logic to use flat tax rates in Saleor calculations

Task linked: SALEOR-6391 Simple taxes


[Bug] [SavePoint] When the task is restored from savepoint, the pending splits are lost

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I stopped a synchronization task by triggering savepoint. I then try to recover the task from savepoint, and I expect the task to continue running where it stopped until all the data is synchronized.
However, I found that after the task was restored, no data was written and the task ended quickly. I checked the problem and found that due to the execution order of the tasks, the pending split would be extracted by map.remove() and assigned to the corresponding SourceReader. However, the current SourceReader is not yet registered with the environment, resulting in the loss of these pending split.
SeaTunnel Version
2.3.3
SeaTunnel Config
env {
    job.mode=BATCH
    job.name=DEMO
}
source {
    Jdbc {
        url="jdbc:mysql://xxxx/transfer_source"
        driver="com.mysql.cj.jdbc.Driver"
        user="root"
        password="xxxx"
        query="select * from order_info"
        partition_column=id
        partition_num=20
        parallelism=2
    }
}
transform {
}
sink {
    Jdbc {
        url="jdbc:mysql://xxxx/transfer_sink?rewriteBatchedStatements=true"
        driver="com.mysql.cj.jdbc.Driver"
        user="root"
        password="xxxx"
        database="transfer_sink"
        table="order_info_sink"
        batch_size=1000
	enable_upsert=true
     	generate_sink_sql=true
	primary_keys = [id]
        query = ""
    }
}

Running Command
./bin/seatunnel-local.sh -c config/savepoint.config

./bin/seatunnel-local.sh -s {jobid}

./bin/seatunnel-local.sh -c config/savepoint.config -r {jobid}

Error Exception
no exception

Zeta or Flink or Spark Version
Zeta
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Task execution order related code:

reader is registerd with in cycle.open(), but back splits are assigned with in reportTaskStatus(WAITING_RESTORE) as follows:

As you can see, when the back splits were assigned, the reader in the current environment was not judged, leading to this problem


[Question] How to generate images without optimization

Awesome paper! It is an excellent result.
I watched the youtube video that you uploaded yesterday.
In the demo, it generates images in real-time without optimization.
When I experimented with your colab, I had to optimize it for each text.
How do you generate images like the youtube demo?
If possible, it would be great if you could publish the code of the youtube demo.

Hi @tommy19970714 ,
Thanks for your interest in our paper!
We are working on organizing all the code right now. Hopefully it will be released in the next few days!

I checked your notebook that you uploaded.
Thank you very much for your sharing!


ceil in BST added

Have you read the Contributing Guidelines on Pull Requests?
(Yes)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #356

please review sir @ankitaggarwal23 @siddharth25pandey @atarax665 @iamrajiv

@iamrajiv sir all changes done

@siddharth25pandey all changes done sir


Not all outputs have names

When inspecting the generated onnx models I saw that the decoder and init_decoder have a lot of unnamed outputs.
You can see the output names of an onnx InferenceSession like this: list(map(lambda x: x.name, session.get_outputs()))
We can see that the first self attention is called past_key_values while the other outputs have generated names.

I was just wondering if this can be a problem, if the dynamic axes are not set for the other outputs.
Also, could it be possible that the output_past_key_values dynamic axes for sequence should be 2 (as in dyn_pkv inputs) instead of 1?
Cheers!

the onnx model does not produce the same output shape as the PyTorch model, the PyTorch model gives past_key_values as a nested tuple of tensors of shape (6x4). but the onnx model outputs everything as a single list. ( logits + 24 pkv tensors ) (the eg: shape here is of t5-small)

I was just wondering if this can be a problem if the dynamic axes are not set for the other outputs.

I've tested it and it does not seem to cause any problem as it is generated by model but you can set the dynamic axis for output past_key_values as we did here  for input past_key_values.

Thanks for the answer. Yeah, I also could not find any information in the onnx documentation on whether it is important to set output dynamic axes or not.


Investigate Kibana for log storage

It is now set up. The "Show in Kibana" link on a pod works correctly.


Google Analytics?

Not very familiar with the involved languages, but glancing through the code I noticed Google Analytics are referenced a few times, but not mentioned anywhere (ie, README). I was wondering why they're being used?

I was using it to determine how much the site was being used and which functions were in use. (spoiler: very few people were using the site)
I haven't looked at the metrics in a long while, and I suspect it's broken since Google had an API change recently.

Feel free to message me on Discord if you have any other questions.


[Feature] postgres__get_columns_in_relation should support materialized views

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt-postgres functionality, rather than a Big Idea better suited to a discussion

Describe the feature
The macro postgres__get_columns_in_relation (link) only looks at the information_schema.columns to get the column names for the particular relation. This doesn't work for materialized views because information_schema.columns doesn't include information about materialized views. I think that adaptor macro could be enhanced to account for this and potentially look at pg_attribute (docs) instead. Based on this Stack Overflow it looks like its possible. I'm happy to research some more if that's helpful.
Describe alternatives you've considered
I considered reporting this as a bug but I think its more of a feature request. I imagine the information schema choice was intentional and maybe there is something I'm not considering about pg_attribute that you all have already considered that makes it unfeasible. In that case I'd propose documenting this nuance for postgres.
Who will this benefit?
Anyone using the dbt_utils.star macro with postgres (like I was)
Are you interested in contributing this feature?
I'm happy to contribute if this request is accepted
Anything else?
No response

Does this solution help you @adase11?
https://github.com/dbt-labs/dbt-codegen/issues/155
https://github.com/dbt-labs/dbt-core/issues/9419

@drewblinn it looks like yes. The implementation of the PR I didn't fully check, but I can look to see if it will cover what I'm looking for (it looks like it intends to). The way I went about it for my custom macro was that I queried the information schema for non-materialized views and pg_attribute only for MVs.

Double checked, the PR looks right to me. Although it looks like that was aimed at dbt-core and the adaptors have been decoupled from dbt-core and so those changes would be better served in this module (dbt-postgres).

I also overrode it in our project, but I'll get a PR together to roll these changes out to dbt-postgres the the entire community can reap the benefits! @adase11


[Question]: How to force the LLM to use a tool through the query engine?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I want to get the query engine to return pydantic output, and it internally uses a tool call for that. However, the tool_choice it uses is always auto which sometimes causes it to not use any tool and I get a ValueError: Expected at least one tool call, but got 0 tool calls.
What I want is to force him to always use a tool, instead of being auto so I don't get that error anymore, and it always tries to return the pydantic output I want.
Does anyone know how to do this?
I am using AzureOpenAI with a gpt3.5 model that support functions.

Leaving an update here as I was able to find a solution:
When defining the LLM, in my case AzureOpenAI, I added the tool choice on additional_kwargs:
            deployment_name=AZURE_OPENAI_DEPLOYMENT_NAME,
            api_key=AZURE_OPENAI_API_KEY,
            additional_kwargs={
                "tool_choice": {'type': 'function', 'function': {'name': 'ResponseModel'}},
            },
        ),```
 Where the name is the name of the output_cls you're giving to the query engine.


Formatting error when JSX element contains invalid JS identifier

If tests/cases/fourslash/formattingJsxElements.ts is amended as follows (adding data-id="test" to the <div>):
/// <reference path='fourslash.ts' />

//@Filename: file.tsx
////function () {
////    return (
////        <div className="commentBox" data-id="test">
////Hello, World!/*autoformat*/
/////*indent*/
////        </div>
////    )
////}
////

format.document();
goTo.marker("autoformat");
verify.currentLineContentIs('            Hello, World!');
goTo.marker("indent");
verify.indentationIs(12);

and then run, we see a failure:
$ jake runtests tests=tests/cases/fourslash/formattingJsxElements.ts
rm -rf tests/baselines/local/
rm -rf internal/baselines/rwc/local
Running test(s): tests/cases/fourslash/formattingJsxElements.ts
mocha -R mocha-fivemat-progress-reporter -g tests/cases/fourslash/formattingJsxElements.ts --colors  -t 20000 built/local/run.js

    fourslash tests

Test failure:
fourslash test formattingJsxElements.ts runs correctly
  [․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․]

  0 passing (196ms)
  1 failing

  1) fourslash tests tests/cases/fourslash/formattingJsxElements.ts fourslash test formattingJsxElements.ts runs correctly:
     Error: Debug Failure. False expression:
      at Object.assert (eval at <anonymous> (built/local/run.js:54637:13), <anonymous>:1558:23)

This backs up what we're seeing in actual .tsx files that are being formatted with typescript-formatter, i.e. that non-valid JS identifier attributes are causing the formatter to fall over.
This feels like something the formatter should be able to handle.
Thoughts?

@vladima - thanks for the quick fix. Works as expected.


EPMRPP-62674: Edit global integration

https://jiraeu.epam.com/browse/EPMRPP-62674
DESCRIPTION:

Added hideModalAction (actionCreators component) that duplicates the Report Portal's component and should be removed after the changes in the core ui.
Implemented integration editing and deleting.

looks ok


Add name attribue to mock hass object

Some of my automatons make use of the Hass.name attribute, but this doesn't exist in the test framework as the Hass.__init__() function mocked without any behavior. This PR adds a new mock that will set the name attribute.

Created a spec'd mock for __init__ so we can get access to the self instance object to set name
automation_fixture will set the automation name to the name of the automation class
Add tests for new behavior
Strip trailing white space in edited file. My editor defaults to this, I'm happy to revert this if you like.

I'm not super familiar with mocking in Python, so the spec'd solution I came to  was more pragmatic than anything else. I'm very open to discussing other ways of accomplishing this, any maybe learning something new :).

Thanks for the PR. Before merging I'd like to get it locally and spend some time to be sure I understand what's happening. However, this week is tight, so this PR might hang a little while. I won't forget about it, for sure ;)

No problem about the time, I know how it is :). I've got another larger branch I'm working on now. I'm going to open an issue for that so we can have some design discussion before I bring it to completion.

Oh and one more last thing. Could you fix the conflicts in init_framework.py ?
It's a simple fix: I merged your first change with the extra functions, you just need to update the revamped mocking mechanism with the extra functions you added in the other PR ;)

I merged in upstream master and resolved the conflicts. This should be good to go.
One note, the work I'm doing for issue #18 will overhaul some of this further, but I think it's still worth while to get this incremental improvement in.

Thanks a lot! I'm merging 🙂
And I finally some have time to take a look at The big one 😁 Will do so now 😉


Step 1 URL should be updated

It seems that the current Step 1 URL should be appropriately updated with the Power Platform administration panel.
Thanks!

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

ID: adb92ca9-41dc-d1d0-90df-09e7159e47d7
Version Independent ID: 6c2a02de-a8db-cafa-6d81-861a027c07f2
Content: Delete environment - Power Platform
Content Source: power-platform/admin/delete-environment.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@jorgemorenog
Thanks for bringing this to our attention. This will be corrected today.


Release main scripts

When will you release main scripts for training, inference, and evaluation ?

Thank you for pinging me about it, I was postponing it for too long. I will do it by the end of the week (tomorrow), there isn't much work to do.


Normalizer should re-commit lambda application

Workaround a mysterious normalizer bug discovered in #403

Steps to debug this problem:

Add breakpoint in Literalte.java:126

Add breakpoint in TermView.java:35

Set advanced breakpoint like this

debug test.aya (file contents follows)

open data Nat | zero | suc Nat
open data List (A : Type) : Type
  | nil
  | infixr :< A (List A)
def overlap infixl + (a b : Nat) : Nat
  | zero, a => a
  | a, zero => a
  | suc a, b => suc (a + b)
  | a, suc b => suc (a + b)
def overlap infixr ++ {A : Type} (xs ys : List A) : List A
  | nil, ys => ys
  | xs, nil => xs
  | a :< xs, ys => a :< (xs ++ ys)
def join {A : Type} (l : List (List A)) : List A
  | nil => nil
  | xs :< xss => xs ++ join xss
def map {A B : Type} (f : A -> B) (l : List A) : List B
  | f, nil => nil
  | f, a :< l => f a :< map f l
def pure {A : Type} (x : A) : List A => x :< nil
def infix >>= {A B : Type} (l : List A) (f : A -> List B) : List B => join (map f l)
def t1 => pure zero >>= (\ a => pure (suc zero) >>= (\ b => pure (a + b)))
--| t1{mode=nf}

Sorry for the inconvenience that I cannot find a simpler repro code 😢

@imkiva 冰有空的话看看这个

See feizhu branch

superseded by #417


SIGSEGV - Corretto-<IP_ADDRESS>.1 - async-profiler 1.8.3

Hi, we had a crash of one of our JVMs with async-profiler in continuous mode:
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f705579b173, pid=8748, tid=9285
#
# JRE version: OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (11.0.11+9) (build 11.0.11+9-LTS)
# Java VM: OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (11.0.11+9-LTS, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x27f173]  forte_fill_call_trace_given_top(JavaThread*, ASGCT_CallTrace*, int, frame) [clone .isra.20]+0x15f
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# If you would like to submit a bug report, please visit:
#   https://github.com/corretto/corretto-11/issues/
...
---------------  T H R E A D  ---------------

Current thread (0x00007f6fbda6ff20):  JavaThread "http-nio-8080-exec-2" daemon [_thread_in_Java, id=9285, stack(0x00007f6fc81f2000,0x00007f6fc82f3000)]

Stack: [0x00007f6fc81f2000,0x00007f6fc82f3000],  sp=0x00007f6fc82eab40,  free space=994k
Native frames: (J=compiled Java code, A=aot compiled Java code, j=interpreted, Vv=VM code, C=native code)
V  [libjvm.so+0x27f173]  forte_fill_call_trace_given_top(JavaThread*, ASGCT_CallTrace*, int, frame) [clone .isra.20]+0x15f
V  [libjvm.so+0x7866ee]  AsyncGetCallTrace+0x18e
C  [libasyncProfiler.so+0x22561]  Profiler::getJavaTraceAsync(void*, ASGCT_CallFrame*, int)+0x561
C  [libasyncProfiler.so+0x22858]  Profiler::recordSample(void*, unsigned long long, int, _jmethodID*, ThreadState)+0x1f8
C  [libpthread.so.0+0xf630]
V  [libjvm.so+0x8c4db9]  JavaCalls::call_helper(JavaValue*, methodHandle const&, JavaCallArguments*, Thread*)+0x3b9
V  [libjvm.so+0x8c2fad]  JavaCalls::call_virtual(JavaValue*, Handle, Klass*, Symbol*, Symbol*, Handle, Thread*)+0x1fd
V  [libjvm.so+0xe8efc6]  SystemDictionary::load_instance_class(Symbol*, Handle, Thread*)+0x246
V  [libjvm.so+0xe8d7ba]  SystemDictionary::resolve_instance_class_or_null(Symbol*, Handle, Handle, Thread*)+0x93a
V  [libjvm.so+0xe8dbe2]  SystemDictionary::resolve_or_fail(Symbol*, Handle, Handle, bool, Thread*)+0x22
V  [libjvm.so+0x977760]  find_class_from_class_loader(JNIEnv_*, Symbol*, unsigned char, Handle, Handle, unsigned char, Thread*)+0x30
V  [libjvm.so+0x978e87]  JVM_FindClassFromCaller+0x147
C  [libjava.so+0xe6c8]  Java_java_lang_Class_forName0+0x178
J 1613  java.lang.Class.forName0(Ljava/lang/String;ZLjava/lang/ClassLoader;Ljava/lang/Class;)Ljava/lang/Class<EMAIL_ADDRESS>(0 bytes) @ 0x00007f701f8f2f47 [0x00007f701f8f2e40+0x0000000000000107]
J 32888 c2 javax.el.ImportHandler.resolveClassFor(Ljava/lang/String;)Ljava/lang/Class; (52 bytes) @ 0x00007f70217f60fc [0x00007f70217f5b80+0x000000000000057c]

Async-profiler 1.8.3.
Does it a JVM bug?

Hi,
The bug occured in the JVM when async-profiler attempted to recover otherwise unwalkable stack.
I expect the problem to disappear with safemode=4 option. However, this may result in a large number of [unknown_Java] samples. To get to the root cause and to create a proper workaround I'd need more information: this can be the full hs_err.log and/or the problem reproducer.

hs_err.log
I had to obfuscate some parts. I marked them with <obfuscated>.
It will be really hard to reproduce. We use async-profiler 1.8.2+ in continuous mode from ~september 2020 (after you have fixed #350) in almost 30 of JVMs at production. That crash happened one time.

Thank you for the crash report. The problem happened in a transient state when calling a Java method from VM. Thread has already switched to in_Java state, but no Java code has been executed yet. I'll try to simulate this situation and stress test it.

Thank you.

I was not able to reproduce this exact crash, however, I completely redid the stack recovery in async-profiler 2.8.
It should be more reliable now.
Let me close this issue as outdated. Please open a new one, if you see a problem in 2.8.


The OBS plugin is not working. gives an error message.

Please help
I click configure plugin but an error appears
18:42:30 [Macro Deck] [Warning] >> PluginCredentials: Unable to decrypt credentials for OBS-WebSocket Plugin. Perhaps the machine GUID changed?
18:42:30 [Macro Deck] [Warning] >> PluginCredentials: Unable to decrypt credentials for OBS-WebSocket Plugin. Perhaps the machine GUID changed?
18:42:30 [Macro Deck] [Error] >> MacroDeck: ApplicationThreadException: The given key 'host' was not present in the dictionary.
at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
at SuchByte.OBSWebSocketPlugin.GUI.PluginConfig..ctor() in D:\Temp\SuchByte.OBS-WebSocketPlugin\GUI\PluginConfig.cs:line 35
at SuchByte.OBSWebSocketPlugin.Main.OpenConfigurator() in D:\Temp\SuchByte.OBS-WebSocketPlugin\Main.cs:line 321
at SuchByte.MacroDeck.GUI.CustomControls.ExtensionsView.ExtensionItemView.BtnConfigure_Click(Object sender, EventArgs e) in D:\Macro-Deck\Macro Deck 2\GUI\CustomControls\ExtensionsView\ExtensionItemView.cs:line 109
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ButtonBase.WndProc(Message& m)
at System.Windows.Forms.Button.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

I have the same issue. I noticed it is seemingly using a D:/ file location so I removed my disk drive in hopes of fixing the issue. Nothing changed and the log remained the same
11:01:52 PM [Macro Deck] [Warning] >> PluginCredentials: Unable to decrypt credentials for OBS-WebSocket Plugin. Perhaps the machine GUID changed? 11:01:52 PM [Macro Deck] [Warning] >> PluginCredentials: Unable to decrypt credentials for OBS-WebSocket Plugin. Perhaps the machine GUID changed? 11:01:52 PM [Macro Deck] [Error] >> MacroDeck: ApplicationThreadException: The given key 'host' was not present in the dictionary.    at System.Collections.Generic.Dictionary2.get_Item(TKey key)
at SuchByte.OBSWebSocketPlugin.GUI.PluginConfig..ctor() in D:\Temp\SuchByte.OBS-WebSocketPlugin\GUI\PluginConfig.cs:line 35
at SuchByte.OBSWebSocketPlugin.Main.OpenConfigurator() in D:\Temp\SuchByte.OBS-WebSocketPlugin\Main.cs:line 321
at SuchByte.MacroDeck.GUI.CustomControls.ExtensionsView.ExtensionItemView.BtnConfigure_Click(Object sender, EventArgs e) in E:\Repos\Macro-Deck-WinForms\Macro-Deck\GUI\CustomControls\ExtensionsView\ExtensionItemView.cs:line 110
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ButtonBase.WndProc(Message& m)
at System.Windows.Forms.Button.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)`

Please validate if v1.4.0 fixes this issue. Thanks!

The log shows a slight change in that the lines that the error occur on are different.
OBS-WebSocket-Plugin error.txt

Never mind. I did a clean install, deleting the files that referenced this plugin. After a restart the plugin works without issue.


Remove site but continue inversion

Ability to remove a site without data from the inversion and continue the inversion without that site being added to acrg repo. This could be added here too.

This is added in #29, but we should look at the ACRG version too.

This change has been added now.


Position of Comment symbol "#"

I have one request and one error when I type gcc in normal mode, to comment out a line of code.
Firstly, I get the following error: E685: Internal error: get_op_type().
Secondly, I would like to know about the positioning of the comment symbol. Currently, when I type gcc it adds the comment symbol at the begging of the line, instead of just near the text I'm commenting. Same thing happens for blocks, and it doesn't keep indentation.
What I'm currently getting is this:
class MyClass
  def my_method
    (0..1).each do |item|
#       item += 1
#       item.to_s
    end
  end
end

What I expected it to do (and what I know tcomment does is:
class MyClass
  def my_method
    (0..1).each do |item|
       # item += 1
       # item.to_s
    end
  end
end

Any way I can achieve this now?

Thanks for logging the issue, @tscholiadis !
Looks like the first issue is tracked here: https://github.com/onivim/oni2/issues/2409
The second issue is a bug - we should be respecting that leading whitespace by default like tcomment does, too.


Error running timer: (void-function encode-hex-string)

That function is defined in hex-util, but you for some reason expect it to be defined in hmac-md5.

Fixed in 83e44a0c62157260. Thanks!


lang(org): update readme to show recent changes

Updates org module's readme to show the changes made upon merging #6033

Thanks for your help!


pathfinding: probability for bimodal distribution

This draft PR implements a probability variant based on the formalism introduced by @renepickhardt as a follow-up from bimodal liquidity distribution research see #5988. No in-depth review yet, looking for high-level feedback on configuration, see below.
In this PR I introduce an Estimator interface for probability estimation. The two systems (apriori and bimodal) have different configuration parameters but also share some (time decay parameter, weight of other channel result inclusion). The question is how the specific configuration parameters should be exposed to the user (right now I use a shared config). We could introduce a similar structure as with the chain backend, to distinguish different probability sources and to add specific documentation. This way we could configure both approaches and allow for A/B testing in the future.
Todo:

[x] configuration
[x] algo needs documentation
[x] mission control configuration via RPC
[ ] benchmark comparison of the current and bimodal approaches
[x] QueryRoutes and QueryProbability need to be supplemented with channel capacities
[x] update sample-lnd.conf
[ ] performance checks
[x] release notes
[x] lncli setmccfg/getmccfg

A lot of changes to the current probability estimator. It's not easy to see what the actual effect on payment success will be. Left a few comments, but for me it is hard to be certain that the change set is an improvement.

Thank you a lot for the feedback! Just to clarify, the approach would be to provide an alternative opt-in implementation. Right, without a benchmark we won't know if there's an improvement, however, there are some limits in the math that make me confident there will be improvements (for example, if the algo learns that there's a success for a  100ksat payment, it will also favor that pair for 200k sat). In the next I'll focus on establishing a benchmark method.

An alternative approach would be to make a much smaller incremental step and use most of the existing algorithm with the addition of penalizing amounts close to the channel capacity harder.

I was thinking about that and wanted to do that after this PR, but I agree, it would be a quicker way to benefit users if we split this PR and introduce such a penalty first. I would take the approach of the first four commits and build on top of that. Passing the capacity as a parameter is due to performance reasons (haven't compared to looking up capacity from the graph). In the long run, I think we may need to restructure mission control to provide the capacity and other graph-derived data efficiently (maintained by ChannelRouter).

An alternative approach would be to make a much smaller incremental step and use most of the existing algorithm with the addition of penalizing amounts close to the channel capacity harder.
I would take the approach of the first four commits and build on top of that.

Yeah FWIW doing something like that isn't mutually exclusive with this PR (can even be done concurrently with just those first few commits). In fact, the work in this PR makes adding modified estimators much easier.

Added a suggestion for how to handle configuration. I added subgroups for the different estimators, which is a breaking change for the apriori parameters.
Shares initial commits with #6857, so if questions come up, would appreciate review comments in that PR concerning these commits.

Rebased on https://github.com/lightningnetwork/lnd/pull/6857. Probability estimators are now swappable at run time to allow for custom A/B testing. I probably need to remove EstimatorConfig and just incorporate config fields directly into the probability estimator implementations, as I don't see a way around the type switches for configuration.
There are some performance issues, where a pathfinding attempt would take ~0.5s compared to ~0.1s (apriori) on my machine, which needs investigation. Probably calling math.Exp adds to computation time and perhaps can be replaced with a more efficient approximation.

There are some performance issues, where a pathfinding attempt would take ~0.5s compared to ~0.1s (apriori) on my machine, which needs investigation. Probably calling math.Exp adds to computation time and perhaps can be replaced with a more efficient approximation.

One approach would be to make a benchmark, then expose profile capture during the benchmark. This way, we'd be able to see exactly where things are slower and if we need any memoization or w/e. You could also just try to do path finding attempts locally on the CLI, then capture a profile right before you execute the command to see where things are slowing down.

Can be rebased now that the dependent PR has been merged!

@bitromortac, remember to re-request review from reviewers when ready

@positiveblue: review reminder
@bitromortac, remember to re-request review from reviewers when ready

@positiveblue: review reminder
@bitromortac, remember to re-request review from reviewers when ready

@bitromortac, remember to re-request review from reviewers when ready

@bitromortac, remember to re-request review from reviewers when ready

As discussed offline, it may be worth adding clear steps to the PR description explaining how users can do a side-by-side comparison of both estimators with a "mission control state restore" in between.

As discussed offline, it may be worth adding clear steps to the PR description explaining how users can do a real life side-by-side comparison of both estimators with a "mission control state restore" in between.

Sounds like a good candidate for a blog post or lnd mailing list post. Either of those would be easier to consume/distribute compared to a PR description.

I've looked up XImportMissionControl and traced towards importSnapshot:
// importSnapshot takes an existing snapshot and merges it with our current
// state if the result provided are fresher than our current results. It returns
// the number of pairs that were used.

Not sure if this is usable for a side-by-side comparison, because it merges mission control state rather than replacing it.
I think it is worth spending some extra time on figuring this out and documenting it somewhere to really unlock the value of this PR for users. Otherwise it might be yet another set of pathfinding config flags for which users can't really decide what to do with.

Not sure if this is usable for a side-by-side comparison, because it merges mission control state rather than replacing it.

Possible to first do a reset, then a merge.

I think it is worth spending some extra time on figuring this out and documenting it somewhere to really unlock the value of this PR for users.

I think the value is unlocked right after the merge. We can update the PR description later in a non-blocking manner. As mentioned a PR description isn't as searchable/indexable as a mailing list post or blog post.

Pushed some minor changes mostly related to API backward compatibility. I updated the PR description to give some more context and included a hint on how one could measure differences between the two estimators (exactly, with a reset of mc in between).


Update modelTimer.lua

@codeAC29 Here's another change I had to do.  I was getting this error otherwise: cuda runtime error (77) : an illegal memory access was encountered at. Could you please explain why this happened? The end result was Forward average time on THIS GPU : 45.37 ms on a NVIDIA Quadro M1000M in a laptop.

It should not have ideally given an error. That line converts modules like nn.SpatialConvolution into cudnn.SpatialConvolution. This gives us little bit better performance while using cuda. If all your modules are using nn then you don't need to even use this line. The code runs fine for me on my GPU. I think it might be an issue with your specific GPU and cudnn. Lets see if someone else also encounters the same problem.

It should be profiled using cudnn, as it provides a more optimised implementations of many modules than nn. Do you have nn.SpatialMaxUnpooling in the network somewhere? Conversion to cudnn breaks the links with MaxPooling modules, because they're converted to their cudnn versions. This line should go like this:
cudnn.convert(model, cudnn, function(m) return torch.type(m):find('MaxPooling') end)

Hi @apaszke , your solution worked! I have the updated the PR with this change. There is nn.SpatialMaxUnpooling in the network. I also checked the effect of that line on the network and saw it doesn't convert the nn.SpatialMaxUnpooling and nn.SpatialMaxPooling to cudnn as you suggested. Do you know why it might be that conversion to their cudnn version is breaking these layers? is it documented anywhere?
The performance increase was quite dramatic - with the cudnn my run went from ~45 ms to ~33 ms on a Quadro m1000m.

Told you cudnn is much faster ;) It breaks because Unpooling modules are still connected to nn.SpatialMaxPooling (these are the only references to these objects now - their fields won't get updated when you call :forward), which were replaced in the network by cudnn layers. There's no better way to do this, since cudnn doesn't save the pooling indices, so you can't unpool using them.

Aah that makes sense 😃


Step in and stop on entry debugging issues

OS: macOS Catalina 10.15.2
VSCode version:  1.41.1
Extension version: 1.4.5
Python version: "Could not initialize Python interpreter - only native expressions will be available."
There is Python3.7.3 in Path and I also run Python 3.6.8 in conda env.
LLDB version: lldb-1<IP_ADDRESS>
Apple Swift version 5.1 (swiftlang-1100.0.270.13 clang-1<IP_ADDRESS>)

There are some issues debugging.

StopOnEntry=true causes the debugger to go to Assembly code (see image)
If StopOnEntry=false and I add breakpoints in the code, the debugger seems to go through the code 'Stepping Over' but the "Step Into" will not go to STL code. Eventually debugger goes to Assembly code.

Here are my .json files:

launch.json

{

"version": "0.2.0",
"configurations": [
    {
        "name": "Launch",
        "type": "lldb",
        "request": "launch",
        "program": "${workspaceFolder}/a.out",
        "stopOnEntry": false,
        "args": []
    }
]

}

Build using tasks.json :

{

"version": "2.0.0",
"tasks": [
    {
        "label": "Build with Clang",
        "type": "shell",
        "command": "clang++",
        "args": [
            "-std=c++17",
            "-stdlib=libc++",
            "eg.cpp",
            "-o",
            "a.out",
            "--debug"
        ],
        "group": {
            "kind": "build",
            "isDefault": true
        }
    }
]

}

c_cpp_properties.json

{

"configurations": [
    {
        "name": "macOS",
        "includePath": [
            "${workspaceFolder}/**"
        ],
        "defines": [],
        "macFrameworkPath": [
            "/System/Library/Frameworks",
            "/Library/Frameworks"
        ],
        "compilerPath": "/usr/bin/clang",
        "cStandard": "c11",
        "cppStandard": "c++17",
        "intelliSenseMode": "${default}"
    }
],
"version": 4

}
Here is the code I am debugging as an example:
`
#include
#include
#include
using namespace std;
int main()
{
vector<string> msg{"Hello", "C++", "World", "from", "VS Code!"};

for (const string &word : msg)
{
    cout << word << " ";
}
cout << endl;

}`
a.out and a.out.dSYM are in the workspace folder as seen in the screenshot.

Hi, I am on macOS Catalina (with lldb-1<IP_ADDRESS>) and I have the same problem. My tasks.json and c_cpp_properties.json files are the same as above except for the file name, and even after adding "initCommands": ["settings set target.process.thread.step-avoid-regexp ''"] to launch.json, my debugger still goes into the Assembly code.
My launch.json:
`{
"version": "0.2.0",
"configurations": [
    {
        "name": "Launch",
        "type": "lldb",
        "request": "launch",
        "program": "${workspaceFolder}/test",
        "stopOnEntry": false,
        "initCommands": ["settings set target.process.thread.step-avoid-regexp ''"],
        "args": []
    }
]
}`

I also checked this page but couldn't find how to disable the disassembly view. Am I missing a step? Thank you!

Eventually debugger goes to Assembly code
How to solve this problem？

The following sets up and deletes a breakpoint with every debug session to help those who are having trouble with getting stuck in assembly code before main has been called. This is probably the behavior that most people want instead of stopOnEntry.
"initCommands": ["breakpoint set -n main -N entry"]
"exitCommands": ["breakpoint delete entry"]
Replace main with the entry point to your code if necessary.

You can also achieve this with function breakpoints.
If your entry point's symbol is namespaced or mangled, regex breakpoints might also be useful.

I personally use  a magic command that I found long time ago searching the net. It works with C and RUST both in local and remote debug. I just add within 'lldb' launch configuration
"initCommands": ["target stop-hook add -n _start -o continue"],

My configuration
    "configurations": [
        {
            // reference https://stackoverflow.com/questions/68888706/remote-debug-of-rust-program-in-visual-studio-code
            "name": "Remote target 'serial-enedis'",
            "type": "lldb",
            "request": "custom",
            "preLaunchTask": "rust:remote-debug-setup-serial",
            "targetCreateCommands": ["target create ${workspaceFolder}/target/aarch64-unknown-linux-gnu/debug/serial"],
            "processCreateCommands": ["gdb-remote raspberry.local:9999"],
            "initCommands": ["target stop-hook add -n _start -o continue"],
        },
        {
            "name": "Native Debug 'serial-enedis'",
            "type": "lldb",
            "request": "launch",
            "preLaunchTask": "rust:native-compile-serial",
            "program": "${workspaceFolder}/target/debug/serial-enedis",
            "args": ["/dev/ttyUSB0"],
            "cwd": "${workspaceFolder}",
            "initCommands": ["target stop-hook add -n _start -o continue"],
        }
    ]
}


Error in github authentication sync settings

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.77.1
OS Version:  macOS Ventura 13.3.1

Steps to Reproduce:

I try to log in to vscode with github to synchronize my settings, and the page shows that the github login is successful. Vscode does not show successful and synchronization. I try to log in and it displays the following information in turn.
2.

I try many times and it's still the above result.

I tried to use AppCleaner to uninstall vscode and reinstall it, but the above result is still the same. I suspect that it is an incompatibility problem after upgrading the system, because this problem did not occur before I updated the system. In addition, I also found that a brother appeared similar to me. question, he posted on stack overflow https://stackoverflow.com/questions/75977433/vscode-1-77-1-problems-with-installing-extensions

Can you follow these steps so that we can get more information on what's going on?

Trigger the sign in
Wait for the issue to happen...
Open the Output pane (cmd+shift+u)
drop in this issue he contents of the following:

GitHub Authentication Output channel

I don't know what this says;

but I think it's the notification that says "do you want to try a different way to sign in?" is that it? Have you tried following through with that?

yes i tried,
At the beginning, the verification login will jump to the web page to verify the identity of github. The web page shows that the verification is successful, and return to vscode. He asked me to try another method again. I click Yes, and then it jumps to the web page again to show that the login is successful, which is the successful login to the github page above. Picture, return to vscode and it will display: Open settings synchronization error, Cancelled! I don't see any output information in the console. I've tried many times and it's the same result! And the problem appeared after I updated the system.

I don't see any output information in the console.

Just to be clear, I don't want the Console, I want the Output:

Note: "GitHub Authentication" in the drop down on the right.

Ok, come back later and give you an answer


Prepare eventgrid for the polymorphic resource changes

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.EventGrid

/azp run net - resourcemanager - mgmt - ci


Blackbox tests for more complex scenarios

Following https://github.com/astriaorg/astria/pull/946 Conductor should gain more blackbox tests for scenarios like duplicate blobs, missing blobs, and others.

This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue
be closed in 7 days.

This issue was closed because it was stale


HttpClient not able to retrieve plain text

[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
HttpClient does not retrieve API that returns plain text.
HttpClient cannot set responseType: 'text' as Get options as described in the documentaion.
Expected behavior
It should be able to accept plain text from API response.
Minimal reproduction of the problem with instructions
... ...
constructor(http: HttpClient){}

GetData(){
  // will not accept responseType: 'text' -> would not compile
  return http.get<U>(url, {responseType: 'text')
     .catch((err:HttpErrorResponse) => {
          // will get error here, although the http response code is 200
     });
}

Use: webpack
Dev environment run on webpack dev server
running Angular 4.3.1

Sorry wrong repo


[SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

What changes were proposed in this pull request?
This fixes a URL in R doc.
Why are the changes needed?
Currently, this invalid link causes a warning during CRAN incoming feasibility. We had better fix this before submitting.
BEFORE
* checking CRAN incoming feasibility ... NOTE
Maintainer: ‘Shivaram Venkataraman<EMAIL_ADDRESS>
Found the following (possibly) invalid URLs:
  URL: https://wiki.apache.org/hadoop/HCFS (moved to https://cwiki.apache.org/confluence/display/hadoop/HCFS)
    From: man/spark.addFile.Rd
    Status: 404
    Message: Not Found

AFTER
* checking CRAN incoming feasibility ... Note_to_CRAN_maintainers
Maintainer: ‘Shivaram Venkataraman<EMAIL_ADDRESS>
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Check the warning message during R testing.
$ R/install-dev.sh
$ R/run-tests.sh

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14342/
Test PASSed.

Test build #109274 has started for PR 25483 at commit 22bf974.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14343/
Test PASSed.

Test build #109274 has finished for PR 25483 at commit 22bf974.

This patch fails R style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109274/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14344/
Test PASSed.

Test build #109275 has started for PR 25483 at commit ec66cdc.

Test build #109275 has finished for PR 25483 at commit ec66cdc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109275/
Test PASSed.

Could you review this please, @srowen , @felixcheung , @HyukjinKwon ?

Thank you for review and approval, @srowen .
Merged to master/2.4/2.3.

LGTM, thanks for fixing


bind can be improved for

I have to do this in order to get the initial value loaded into the textarea:
<textarea @bind(myvar)>@myvar</textarea>

It would be nicer and more consistent with <input> if this would work the same way:
<textarea @bind(myvar)></textarea>

It should be fixed I guess once PR #406 is merged as they're revamping the bind system to be more flexible and powerful.

I think that even after #406 we'll still need to make a tweak in BrowserRenderer.ts in the tryApplyValueProperty function so that it understands that textarea requires special handling for value (equivalent to how it already does for input and select).
This should be a fairly easy tweak (and doesn't need to wait for #406) if anyone's interested in submitting a PR :)

@SteveSandersonMS  Sounds easy enough xD can I give it a shot?

Resolved in #439


Additional build system bug fixed and improvements

Some additional changes to fix or improve building, particularly in the Docker containers.  This gets us closer to testing automatically via Docker from Jenkins.

Also, I meant to add that if my PR technique is sloppy, please let me know.

Is it not possible to get LLVM 3.7 on these systems? I'm okay moving to 3.9 - I've tested it up to 4.0 and it all works.  But if we're doing this, we should be consistent on travis and in documentation too.

On 2017-08-01 09:19, Prashanth Menon wrote:

Is it not possible to get LLVM 3.7 on these systems? I'm okay moving to
3.9 - I've tested it up to 4.0 and it all works. But if we're doing
this, we should be consistent on travis and in documentation too.

In most cases it is still possible.  My only motivation was that it
seemed like support for that particular version is getting spotty in
upstream repositories, e.g., script/installation/packages.sh:66, and
will probably get spottier over time.
That being said, I don't want to make a mess in your project and if this
is just causing distraction, I'll cut it out :)  If this direction is OK
though, I'd be happy to review travis and the documentation for relevant
changes as well though.
--
-Chad

Closing this PR for the time being.  I think it was a bit hasty on my part.  I'm going to try a few different things and then will submit a different PR.


Build an AppImage on Travis CI

Great work @sudo-give-me-coffee, let's create the virtual counterpart of  https://www.ikea.com/ms/en_US/customer-service/our-services/assembly.html :-)

This pull request, when merged, builds an AppImage on Travis CI and uploads it to GitHub Releases.
PLEASE NOTE: For this to work, you need to set up GITHUB_TOKEN in Travis CI for this to work; please see https://github.com/probonopd/uploadtool.
Here is an AppImage for testing:
https://github.com/probonopd/flatpak2appdir/releases/tag/continuous

Now it should be a matter of removing the need for the FIXMEs in .travis.yml and getting it to work for other applications as well.

I will work on it now, thank you so much


opening files with non-ANSI filenames on Windows

Thank you for maintaining PEGTL.
My understanding is that currently file_input and read_input don't work with Unicode filenames on Windows? Because I see that internal::file_reader uses fopen(_s) not _wfopen(_s).
If that's true, this is a feature request to add support for Unicode filenames.

Are you saying that fopen on Windows only accepts ASCII, but not Unicode encoded as UTF-8? What does wfopen get as argument, a string or a wide string? What type and encoding?

fopen takes filename in Windows (ANSI) code page, which is ASCII + 128 extra characters.
_wfopen takes wchar_t * which is either UCS2 or UTF-16 (I don't know).
I don't have a lot of experience with it, but I came across one example of a library handling this issue: zlib has an extra function gzopen_w which does the same as gzopen but takes wchar_t* as an argument and exists only on Windows.

FYI:
https://msdn.microsoft.com/en-us/library/yeby3zcb.aspx

FILE *fopen(
const char *filename,
const char *mode
);
FILE *_wfopen(
const wchar_t *filename,
const wchar_t *mode
);

Using _wfopen() instead of fopen() is easy, the problem is that we also store the filename as source, i.e. name-of-the-source-of-the-parsed-data.
This can currently be a const char *, or a std::string, so that we can construct the std::stringin thepositionin theparse_error` exception class.
The question is, (how) can we use _wfopen() without supporting wchar_t or std::wstring as source?
Can we convert the wide characters provided to _fopen() to a regular string - or do we have to bite the bullet and enable wide strings in parse_error etc...?

@skyrich62 Thanks for the link; at the moment we are just looking at the encoding of the file name, not the content, but we'll keep the possibility of an on-the-fly conversion in mind.

As a work-around for this: Would it be sufficient to add a ctor taking the FILE* directly and accept an additional "normal" std::string as the source? It makes usage slightly more verbose, but I think it fits better with the rest of the design.
@wojdyr Would that be OK with you or does it cause additional problems? Note that our class takes ownership of the FILE*, e.g. it closes it during destruction.

Yes, it would work fine for me.

That makes things easier, read_input now has a constructor that takes a FILE* as first argument.

Thanks!