OpAMP Agent heartbeats

Currently, if an agent is functioning successfully and without any changes to its health or component status, no messages will be sent after initial handshakes. For websockets, because no messages are sent from the agent to the server, after a certain period most of these connections will idle and be shut down causing a broken socket. I propose that we add support for an heartbeat interval as part of the specification. I have already implemented this capability in the opamp-bridge where a user is able to configure a heartbeat interval to periodically set the agent's health message. HTTP theoretically gets around this with its polling interval.
This heartbeat implementation is important for the bridge where many of the events in Kubernetes happen asynchronously. The bridge is not informed directly therefore must poll state to send this message. The collector's opamp extension, however, watches for some changes but is currently not busy. For the extension, the collector will idle after sometime and prevent the server from being able to send any more messages to the extension.
I think a heartbeat interval could be optional, however, it must be communicated as part of the initial AgentToServer message. This would allow the server to know when to mark the agent as unhealthy.
Open Questions

Should this functionality only exist for the socket transport?

I believe it would be valuable for this to exist regardless of the transport and that the HTTP poll interval could be deprecated in favor of this. This would allow the server to make decisions about the liveness of the agent regardless of the transport

What should the default interval be? 30s?
Should the heartbeat interval be negotiable from the server?

I'm happy to write the spec change for this issue, but would love everyone's thoughts here.

I think it is reasonable for the agent to actively report a healthy heartbeat, this helps opamp server keep the information up to date.

I think the main difference is that my proposal relates to heartbeats as a means of solving #28, allowing clients to automatically report health on an interval. A goal of this is definitely to keep the connection alive, but it also means that the client has a responsibility to reports its health on an interval. I can understand closing this issue in favor of that though...

A crashed or misbehaving client may cause connection/goroutine leaks in the OpAMP server (open-telemetry/opamp-go#271). A heartbeat mechanism can help the server find out unresponsive peers and also defend against intermediaries (LB, proxy, network equipment) which may time out and terminate idle connections (#28).

According to the specification, AgentToServer.health may be omitted if nothing changed since last AgentToServer message. I think an empty status report is enough for the heartbeats.

I also wanted to give a big +1 to this. The regular status reports from the opamp-bridge have been really useful in

providing a regular update on status
acting as a heartbeat
making sure the opamp agent checks in (especially if it's connecting over HTTP) so we can send any ServerToAgent message it needs

I agree with @BinaryFissionGames that for the sake of the OpAMP protocol, this behavior should be transport agnostic (i.e. should work for both HTTP and Websockets given both are valid transport protocols for OpAMP).

@tpaschalis I have the PR up here if you can take a look!


import-module incorrectly reporting a loaded module was not found

missing break after module was loaded successfully so loop continues
but module isn't "found" and thus reports error even though loaded
Fix https://github.com/PowerShell/PowerShell/issues/5232

Shall we add the '[Feature]' tag to run all tests?

@daxian-dbw it wouldn't hurt to run [feature] tests, will update

@daxian-dbw looks like for mac Travis-CI installed beta.8, but the scripts tried to start pwsh.  Did we publish beta.9 to homebrew already?

@SteveL-MSFT The PR to update powershell.beta.9 in homebrew is here: https://github.com/caskroom/homebrew-cask/pull/40043
We submitted the PR early this morning, but it hasn't been merged yet.

@daxian-dbw thanks, I guess we'll just wait for that to merged then I'll restart the macOS job

@SteveL-MSFT restarted the Travis-CI macOS job. The homebrew PR is merged.

@JamesWTruher can you re-review?


Add option to verify LLVM IR generated by bpftrace

As proposed in #2297, this PR implements an option to verify generated LLVM IR via setting the BPFTRACE_VERIFY_LLVM_IR environemntal variable to 1. An environmental variable is used to easily allow this to be set in tests.
~It also enables this option for runtime tests, detecting invalid code that has to be fixed before merging this PR to make the tests pass.~
Checklist

[x] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

nice, lots of issues we still have to fix :D
I suggest we merge it but dont enable it yet, that way we can start fixing them

I removed the commit that enables the LLVM IR verification in CI, so that it can be merged without breaking the CI. This also means there is no documentation for the option, since it is not user facing and is not used by the tests yet.

Simplified CodegenLLVM::verify to make use of llvm::verifyModule instead of capturing the output of VerifierPass.

@fbs @danobi are you fine with merging this?


Graceful restarts

Using https://github.com/facebookgo/grace ?

Not to be implemented by default. It's too easy to implement a custom method using any alternative.


perf: speed up preprocessor define

Less code, faster and more reliable

:tada: This PR is included in version 1.3.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update stats.html

added Komodo and Zen block explorer links

bundled with: https://github.com/z-classic/z-nomp/pull/337, should we unbundle these PRs?

bundled with: #337, should we unbundle these PRs?

Hi,
I don't think so :)


issue #719 solved

Added the following as discussed in issue #719
Prerequisites
How to Report Bugs
How to Request Enhancements
Setting up the local environment
Forking repository
Clone repository
Setting up remote
Pushing your changes
Creating a pull request
Thank you

@joernu76 please review the documentation. Thank you

I do convert this to a draft state until it is continued

I do convert this to a draft state until it is continued

hey @ReimarBauer I will be resolving and doing all the changes. i just need some time. Hope you understand.

@ArchishmanSengupta How do you do?

@ArchishmanSengupta How do you do?

hey, I have done the necessary changes, now should I make a new pr or go ahead with resolve conflicts with the whole refactoring of the rst file?

will I close the PR as I have to update my local repo, so that I can contribute a new PR?

Just a note. This is a good start for giving the information. In a further step this needs a refactoring by moving all development related parts into the section Developmen.
This has to be done before we can release 5.0. I file for this a blocking issue.


Embedding

Can I know what embeddings you use to train the models?
Is it clinic data related embeddings?

Hi, the word vectors found here were used to train the models, taking the most frequent ~ 100,000 word vectors based on some subset of pubmed:
http://bio.nlplab.org/


CLI should warn if host can't find NuGet

See https://github.com/Azure/azure-webjobs-sdk-script/issues/799 for instance

Closed because we are packaging nuget with the CLI


Unify country imports order

Proposed change

Make the country imports look consistent.
Type of change

[ ] New country holidays support (thank you!)
[ ] Supported country holidays update (calendar discrepancy fix, localization)
[x] Existing code/tests/processes improvement (best practice, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (adds functionality to python-holidays in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts
[x] The code style looks good (make pre-commit)
[x] I've added tests to verify that the new code works and all tests pass locally (make test)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

Hmmm... Since we say that

The standard way to refer to a country is by using its ISO 3166-1 alpha-2 code,  the same used for domain names,

and don't even list or document the (nonstandard) long form name, stylistically doesn't it make more sense to import the documented 2-letter code first?
Also, as an aside,

We're importing a ton of alpha-3 codes, and nowhere in the documentation we say we support them. I don't really see a reason why we should support them (they are superfluous to the alpha-2 codes that are very familiar due to their use as TLDs); maybe it's a good time to start deprecating the existing alpha-3 codes (and stop adding them when new countries are added)? That would start the work on #998.
Do we even want a long-form nonstandard for countries, which also opens ourselves up to discussions on political issues such as whether the class for HK should be HongKong vs. HongKongSAR (the latter being closer to the official name), CN China vs. PeoplesRepublicOfChina and on and on?
I also found IRL that finding a class called "HongKong" ("HongKongSAR"?) or "Eswatini" or similar in the debugger is very confusing -- Eswatini what? What is the Eswatini object supposed to be? We should consider calling the base class in the format of HolidaysHK, HolidaysSZ, etc. to make it very clear when debugging what is the function of an object of this class.

Hmmm... Since we say that

The standard way to refer to a country is by using its ISO 3166-1 alpha-2 code,  the same used for domain names,

and don't even list or document the (nonstandard) long form name, stylistically doesn't it make more sense to import the documented 2-letter code first?
Also, as an aside,

We're importing a ton of alpha-3 codes, and nowhere in the documentation we say we support them. I don't really see a reason why we should support them (they are superfluous to the alpha-2 codes that are very familiar due to their use as TLDs); maybe it's a good time to start deprecating the existing alpha-3 codes (and stop adding them when new countries are added)? That would start the work on Maximize load performance  #998.
Do we even want a long-form nonstandard for countries, which also opens ourselves up to discussions on political issues such as whether the class for HK should be HongKong vs. HongKongSAR (the latter being closer to the official name), CN China vs. PeoplesRepublicOfChina and on and on?
I also found IRL that finding a class called "HongKong" ("HongKongSAR"?) or "Eswatini" or similar in the debugger is very confusing -- Eswatini what? What is the Eswatini object supposed to be? We should consider calling the base class in the format of HolidaysHK, HolidaysSZ, etc. to make it very clear when debugging what is the function of an object of this class.

Hi Mike,
these ideas are totally worth a discussion/implementation. I suggest creating a separate issue(s) (or discussion(s) -- up to you) as after this PR is submitted the content will be pretty much forsaken.

Coverage: 100.0%. Remained the same when pulling 79c46d3a91859ed138540747096962be1c2ffabb on arkid15r:reorder-country-imports into d34e0b3eee6720ca1be64066bec7159c5624e482 on dr-prodigy:beta.


[docs] Fix text from material-ui to @mui to reflect v5 name changes

[X] I have followed (at least) the PR section of the contributing guide.

@pupudu It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!

Thanks a lot Olivier. Happy to see the first PR got merged 🙏🏼


Plugin update not working

When I click Update, the button shows Updating then Updated and Joplin asks me to restart.

I restart and still have the v0.2.0 with the Update button.

This has nothing to do with the plugin.
Please create a topic for this in Support at https://discourse.joplinapp.org/


Sigma clipping with moving mean replacement to S4

A quick little PR to add sigma clipping to S4
The NIRSpec PRISM simulated data had several reads where many/all wavelengths were fully zeroed out which required sigma clipping to remove. For now I have added code to replace clipped data points with the median value from a moving box-car filter, but we can add other options in the future. There is also the ability to replace all outliers with a constant value.
Also now forcing printed log statements to flush to STDOUT as they would somtimes get clogged and not print until the end of the stage when the log was closed.

Alright, I've now ensured that masking of bad data in S4 works well and carries over well to S5. In the process I also discovered and resolved a big bug with S4-5 where they would not properly consider all of the different apertures run in S3 despite their claims that they were using those different apertures.


chore: Updated German translation

Usually I prefer Jadx in English but the last days a number of times my preferences were reset so I ended up with German as language and noticed that some translations were really hard to understand even for me as I know Jadx very good.

@jpstotz great! Thank you!

number of times my preferences were reset

Do any errors appear in the logs? Maybe I break something in recent commits 😨

@skylot I am not sure what caused the reset. I am also not sure if the whole preferences were reset or just the language. UI language is just the most obvious preference.
In the end it is nothing I was able to reproduce. Also I can't name a commit where this started or stopped as I use Jadx on multiple computers in different stable/unstable versions and frequently change preference options.


Memory Leak in Dialog Management: ReactRootView not removed from View Tree when Dialog is not showing

Description
There is a memory leak in the dialog management code of React Native's new architecture. When the hide() method is called and the dialog is not currently showing, the ReactRootView is not removed from the view tree. This results in the ReactRootView holding a reference to the Activity, causing a memory leak.
Steps to reproduce

Create a dialog with a ReactRootView.
Call hide() on the dialog when it is not showing.
Observe that the ReactRootView is not removed from the view tree.

React Native Version
0.74.2
Affected Platforms
Runtime - Android
Areas
TurboModule - The New Native Module System
Output of npx react-native info
info Fetching system and libraries information...
System:
  OS: macOS 14.3
  CPU: (8) arm64 Apple M1
  Memory: 45.19 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 21.7.3
    path: /opt/homebrew/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.5.0
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.11.3
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK:
    API Levels:
      - "25"
      - "26"
      - "27"
      - "28"
      - "29"
      - "30"
      - "31"
      - "32"
      - "33"
      - "33"
      - "34"
      - "34"
    Build Tools:
      - 26.0.2
      - 27.0.3
      - 28.0.2
      - 28.0.3
      - 29.0.2
      - 30.0.2
      - 30.0.3
      - 33.0.1
      - 34.0.0
    System Images:
      - android-34 | Google APIs ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.1
    path: /usr/bin/javac
  Ruby:
    version: 3.3.2
    path: /opt/homebrew/opt/ruby/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native: Not Found
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: Not found
  newArchEnabled: Not found
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

(node:66274) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)

Stacktrace or Logs
┬───
                                                                                │ GC Root: Global variable in native code
                                                                                │
                                                                                ├─ com.facebook.react.devsupport.LogBoxModule instance
                                                                                │    Leaking: UNKNOWN
                                                                                │    Retaining 40 B in 2 objects
                                                                                │    mReactApplicationContext instance of com.facebook.react.runtime.BridgelessReactContext, wrapping com.kk.
                                                                                │    MainApplication
                                                                                │    ↓ LogBoxModule.mSurfaceDelegate
                                                                                │                   ~~~~~~~~~~~~~~~~
                                                                                ├─ com.facebook.react.devsupport.LogBoxDialogSurfaceDelegate instance
                                                                                │    Leaking: UNKNOWN
                                                                                │    Retaining 20 B in 1 objects
                                                                                │    ↓ LogBoxDialogSurfaceDelegate.mReactRootView
                                                                                │                                  ~~~~~~~~~~~~~~
                                                                                ├─ com.facebook.react.runtime.ReactSurfaceView instance
                                                                                │    Leaking: YES (View.mContext references a destroyed activity)
                                                                                │    Retaining 1.2 kB in 18 objects
                                                                                │    View not part of a window view hierarchy
                                                                                │    View.mAttachInfo is null (view detached)
                                                                                │    View.mID = R.id.null
                                                                                │    View.mWindowAttachCount = 0
                                                                                │    mContext instance of com.kk.pages.RNActivity with mDestroyed = true
                                                                                │    ↓ View.mContext
                                                                                ╰→ com.kk.pages.RNActivity instance
                                                                                Leaking: YES (ObjectWatcher was watching this because com.kk.pages.RNActivity received Activity#onDestroy()
                                                                                callback and Activity#mDestroyed is true)
                                                                                Retaining 78.1 kB in 857 objects
                                                                                key = 45d74ebf-9e97-4545-90dd-1d24d3d975bc
                                                                                watchDurationMillis = 25443
                                                                                retainedDurationMillis = 20441
                                                                                mApplication instance of com.kk.MainApplication
                                                                                mBase instance of androidx.appcompat.view.ContextThemeWrapper

Reproducer
https://github.com/react-native-community/reproducer-react-native/tree/main
Screenshots and Videos
No response

There is a memory leak in the dialog management code of React Native's new architecture. When the hide() method is called and the dialog is not currently showing, the ReactRootView is not removed from the view tree. This results in the ReactRootView holding a reference to the Activity, causing a memory leak.

Can you provide a valid repro an a screenshot from the memory profiler of Android Studio?

Sure, here is the translation of your response:

First, in my issue, I provided the memory leak path captured by LeakCanary. When looking at the code, this issue becomes very clear: Regardless of the state of the Dialog, mReactRootView will always be indirectly created and held by LogModule, and when ReactRootView is created, it holds a reference to the Activity. Therefore, the hide method of LogBoxDialogSurfaceDelegate must ensure that it is removed from the Activity. However, the current implementation of the hide method does not do this. When the dialog is not created or shown, it directly returns, causing the view to not release the Activity reference, which leads to a memory leak.
To reproduce this issue is very simple. You just need to start by creating a Native Activity in the React Native Android template project, then navigate to a ReactActivity, and finally return to the main Activity from the ReactActivity. This will reliably reproduce the memory leak. Additionally, this reproduction path may also uncover other memory leaks under the new architecture. I suggest conducting similar tests as soon as possible and fixing the related issues.
My local fix code like this:
` @Override
public void hide() {
if (isShowing()) {
mDialog.dismiss();
}
if (mReactRootView != null && mReactRootView.getParent() != null) {
  ((ViewGroup) mReactRootView.getParent()).removeView(mReactRootView);
}
mDialog = null;

}`


 Please review.

Comparison is base (5bea360) 81.10% compared to head (51c60df) 81.17%.

Files
Patch %
Lines

crates/wasmi/src/module/compile/mod.rs
0.00%
7 Missing :warning:

crates/wasmi/tests/spec/context.rs
30.00%
7 Missing :warning:

crates/wasmi/tests/spec/run.rs
46.15%
7 Missing :warning:

...rates/wasmi/src/engine/regmach/translator/visit.rs
68.75%
5 Missing :warning:

crates/wasmi/src/module/mod.rs
78.26%
5 Missing :warning:

crates/cli/src/context.rs
0.00%
4 Missing :warning:

crates/wasmi/src/engine/regmach/translator/mod.rs
60.00%
4 Missing :warning:

crates/wasmi/src/engine/bytecode/utils.rs
0.00%
3 Missing :warning:

crates/wasmi/src/engine/regmach/tests/driver.rs
62.50%
3 Missing :warning:

.../engine/regmach/translator/stack/register_alloc.rs
50.00%
3 Missing :warning:

... and 13 more

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #798      +/-   ##
==========================================
+ Coverage   81.10%   81.17%   +0.07%
==========================================
  Files         274      274
  Lines       25323    25276      -47
==========================================
- Hits        20538    20519      -19
+ Misses       4785     4757      -28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ICE: ShiftLeft and ShiftRight are replaced by multiplications and divisions in optimization pass.

Aim
The above mentioned error comes up during nargo prove p, but not during nargo check. I am using the bitwise >> and << operators and that is when the error comes up.
Expected behavior
This should not be the case since I am implementing a working algorithm for hashing from another language, where it works.
Bug
The complete message is:
nargo prove p
The application panicked (crashed).
Message:  internal error: entered unreachable code: ICE: ShiftLeft and ShiftRight are replaced by multiplications and divisions in optimization pass.
Location: crates/noirc_evaluator/src/ssa/acir_gen/operations/binary.rs:244
To reproduce

Include << or >> in the code.
nargo check //no error
nargo prove p //error comes up as shown above

Installation method
Binary
Nargo version
nargo 0.3.2
@noir-lang/noir_wasm version
No response
@noir-lang/barretenberg version
No response
@noir-lang/aztec_backend version
No response
Additional context
No response
Submission Checklist

[ ] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

nargo 0.3.2

Hi @resurgencelabs , this version of nargo is quite old. Can you update to the latest version of nargo (preferably 0.5.0) and see if you still get this issue?
It would also be helpful to share an example program which exhibits this issue.

nargo 0.3.2

Hi @resurgencelabs , this version of nargo is quite old. Can you update to the latest version of nargo (preferably 0.5.0) and see if you still get this issue?
It would also be helpful to share an example program which exhibits this issue.

Thanks for the promptness, Tom. We updated to 0.5.0 but it still did not work. Here is the repository that gives this error.
If you were to fork it, and run just nargo check followed by nargo prove p, you should be able to replicate the issue.

Hey @resurgencelabs, the issue you're running into is that bitshifts by offsets which are known only at runtime are not implemented yet.
The reason for this is that decomposing the field element into bits and applying a shift to them before reconstructing a field from these bits is rather expensive to do inside a snark. This is a bit different from traditional programming languages as processors are well optimised for bitwise operations.
You may want to consider swapping out the hash function you're implementing for a snark friendly hash function such as Pedersen (which we have in the Noir standard library). I can't give any guarantees about exactly when runtime bitshifts will be implemented unfortunately.

Hey @resurgencelabs, the issue you're running into is that bitshifts by offsets which are known only at runtime are not implemented yet.
The reason for this is that decomposing the field element into bits and applying a shift to them before reconstructing a field from these bits is rather expensive to do inside a snark. This is a bit different from traditional programming languages as processors are well optimised for bitwise operations.
You may want to consider swapping out the hash function you're implementing for a snark friendly hash function such as Pedersen (which we have in the Noir standard library). I can't give any guarantees about exactly when runtime bitshifts will be implemented unfortunately.

No problem. We will take a look at that as well. But whenever this is implemented, just give us a heads up as well. Thanks :)

@resurgencelabs correction, I've been told you'll want to use the poseidon hash rather than pedersen.

This limitation is also in the new SSA. I will describe the solution which would give the best developer UX.
Shifts are converted to either multiplication by a power of two or division by a power of two. If the compiler had a method to exponentiate a constant by a runtime value, this would then be fixed.
This method would need to:

Decompose the exponent x into bits
Component the result by using the square-and-multiply algorithm

We would essentially want to generalize this method and copy it into the compiler: https://github.com/noir-lang/noir/blob/95b2417044c90a7f92eefddb86fe448902e37d70/noir_stdlib/src/field.nr#L25
@guipublic assigning this issue to you


difference between various propagation kinds?

What is the difference between various propagation kinds like forward_inference, forward, forward_training...etc ?
What is the best propagation kind for inference part?

Hi @tensor-go,
forward_training is a propagation kind one has to use during the training. In that case some operations compute extra things required to perform backward pass. For instance, max pooling would store not only the maximums themselves but also indices, where those maximums happen (in a workspace).
forward_inference is the best propagation kind for inference, since in that case some of the primitives can skip some operations (e.g. storing indices in a workspace in max pooling). With this prop_kind backward pass is usually impossible (for to lack of information).
forward is just an alias for forward_training.


processCellFromClipboard is being filtered from ag-grid-react component

Needs to be added to the list of FUNCTION_PROPERTIES in https://github.com/ceolter/ag-grid/blob/master/src/ts/components/componentUtil.ts#L58.

in now. thanks.


Feat: add background for selected background

Description ✏
Add background for selected model

Checklist ✅

[X] I have added screenshots (if UI changes are present).
[X] I have done a self-review of my code.
[X] I have manually tested my code (if applicable).

nice I think color needs to look slightly different from when you hover over though

nice I think color needs to look slightly different from when you hover over though

Changed, also modify the text color for better contrast

Changed, also modify the text color for better contrast
theme: pearai light/dark

@Fryingpannn how about, selected (lightGray55) vs not selected (lightGray33)


Mouse wheel event for OSX

As usual, please check if I did not set things on fire.

Your commit seems to be missing Mouse.h.


IoGT 97: Default styles

@saeedmb8 @projectmushroom  please review
When section does not have a theme, please use default colors:

👍


feat: add check() to check jsensible rules

close https://github.com/ocpddev/jsensible/issues/5

The way I do it is:

Get the error report for each ArchRule violation FailureReport as List<FailureReport>
Determine List<FailureReport> and throw an exception if it's not empty

This solution does not integrate with Junit, as an alternative.


ho fatto una buona/ottima traduzione in italiano dell intera estensione di python

ho tradotto in italiano (alcune righe sono in beta  , e hanno tra parentesi (w.i.p) sta a significare che, non sapevo come tradurle o la traduzione potrebbe non rispecchiare la funzionalita ) 475 righe in confronto alle 30 righe scrause che ci stanno nel file default in italiano .... questo è il file :
traduzione italiano estensione python .zip

ho chiuso il vecchio ticket e lasciate aperto questo.....

@alexr00 @vscode-triage-bot  I made a better Italian translation than the default one, the default one is 30 lines mine and 475 lines ..... my translation is in beta (some translations have in brackets (w.i.p) means that, either I didn't know how to translate them or the translation could be wrong .... I couldn't put every single function of the extension) so I wanted to publish the translation so that other people with more experience than me (who tested more functions of python) could correct the translations that I did wrong.
Translated with www.DeepL.com/Translator (free version)

....if i write it in english how do the italians understand that it is an italian translation????... then i don't understand why you have chisuo the section where one could give a hand to translate both v queues and extensions...... i'm talking about the link you put in the docs of vs code (https://aka.ms/vscodeloc) what's the use of this link if when i click it sends me somewhere else????

if I write the title/description in english how do the italians understand that it's an italian translation???... then I don't understand why you closed the section where one could give a hand to translate both vs code and maybe even the extensions...... I'm talking about the link you put in the docs of vs code (https://aka.ms/vscodeloc) what's this link for if when I click it it sends me somewhere else????
Translated with www.DeepL.com/Translator (free version)

so?

later I'll correct the mistakes in the general translation of VS Code...

obviously always for Italian

my translation is fine for VS Code version: Code 1.44.2.  what more information do you want?

ma scusa, traduci il bot di vscode e poi scrivi
"italian traslaction"

non avete capito....

ma che c'entra il bot? ma chi ha mai nominato il bot???!!! io boh non ho parole
....

Greve, comunque translascion me la segno

ma segnati quello che te pare professore

io mi segno greve


add getter methods to generated types

Adds getter methods to generated types. This serves two purposes:

Deeply nested fields can now be gotten with only a final nil check

 securityInfo := e.GetCorpActnNtfctn().GetCorporateActionGeneralInformation().GetUnderlyingSecurity().GetFinancialInstrumentIdentification()
if securityInfo != nil {}

becomes a safe operation (due to baked in nil checks)
2. Interfaces can be written for common parsing operations
type date interface {
      func GetDate() ISODate
}
func ParseDate(d date) (dates.Date, error) {
	return dates.ParseISO(string(d.GetDate()))
}

What magic is this.

this is a library we forked back during kairos. The original library takes xml specs and generates go types, Dylan added better enums/comments to it but the maintainer never accepted the pr so we just kept our fork IIRC.
Now I'm adding getters similar to those that the proto generated structs have to help with nil ptr handling :D


list() doesn't seem to work for request scopes

list() requires calls to Builder.isAddBeanFor but it seems the only place that's generated is the $DI class for singletons, not request scoped objects.

That could be done in DRequestScope by combining the results of  BeanScope beanScope; and ConcurrentHashMap<String, Object> supplied;.
Noting that ConcurrentHashMap<String, Object> supplied; is not an optimal structure to support list().  Do you want to play around with an implementation for that?

I can take a look I guess. At the moment I'm fiddling with my code to see if I can "swim with the current" so to speak and use singletons for everything instead of request scoping.

Deeming this fixed in 6.5-RC0


Sounds promising, asciinema would be awesome

https://asciinema.org/ linked in README.md would be awesome.

I will record a session in the coming days. Thanks a lot for the tip. :)

Great news!

Added link to asciinema demo recording

Good job! I think you can link a screenshot in the readme or even have it autorun (not sure how), so it is even more obvious :).

Thanks. I tried asciicast2gif to do what you suggested but is seems to fail for some reason. I will try again maybe it's something I'm missing


Fix code styling

Render should be at the bottom, callbacks and instance variables should be divided, life-cycle methods should be together. Interfaces should be at top. JSX tslint options should be enabled.

Look up style guide, perhaps anything not a lifecycle method use => notation

https://github.com/Microsoft/TypeScript/wiki/Coding-guidelines

https://github.com/airbnb/javascript

Closing for now, current style is well enforced by tslint


[QnA Maker] Active learning enabled flag added, More test case added,…

Active learning enabled flag added,
More test case added for QnA Maker action
Bug fixes on QnA Maker dialog
Feature link: https://github.com/microsoft/botbuilder-dotnet/issues/2595

REquested changes only to get more info on where the 0.95 comes from.


FIX #6855: Mangled JS in Requirements, escaping replacement values prior to passing to preg_replace().

PR to address issue #6855 where SS 3.x is currently mangling JavaScript code when forcing to bottom. This is due to using preg_replace() and interpreting $n (such as $8) as back references.
Copied ->escapeReplacement() from master and  attempted to place it in a spot least likely to require effort when merging up :crossed_fingers:

@patricknelson can you address the failing tests?

Ugh, talk about mangling :man_facepalming: sorry about that, here's an updated version. I was also in such a hurry that I forgot to put this in a separate branch. Anyway, I appreciate the quick response!


Setup uptime monitoring via Panopta

Problem description
We should know immediately once the site becomes unresponsive.
Replication Steps

Take the site down randomly
How soon until we notice it is down

Expected Behavior or Output

Get alerted immediately once the site is down

Before closing this, I think that it would be good to:

Use Panopta to simulate an outage to ensure that it notifies via the appropriate channels as expected
Open an issue to add a health check endpoint with django-watchman since a legitimate health check endpoint doesn't exist

I agree. I noticed that there wasn't a health check endpoint as well, but was comfortable using the root domain because other instances like PhillyHistory did the same. This is good follow up.

I simulated an outage, and it looks like the Slack integration worked as intended in #district-builder. Your iPhone would have also been notified.


Where's the documentation to subscribe to webhooks programmatically?

[Enter feedback here]
The Postman example is great but where's the documentation to achieve this using proper Oauth flow programmatically?
Adding resource to the token endpoint gives this:
AADSTS90100: The 'resource' request parameter is not supported.

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

ID: 5e4f4088-693d-ba29-ddbf-797f9499389e
Version Independent ID: aa0efa43-034a-8b34-593b-04c5109526ec
Content: Get started with SharePoint webhooks
Content Source: docs/apis/webhooks/get-started-webhooks.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

I am probably wrong about this, but are webhooks only meant for intranet sites? The subscription api does not seem to be calling the Graph API in the documentation.

but are webhooks only meant for intranet sites?

No. The webhooks works well for internet too.
You can see the (personal Microsoft account) in https://docs.microsoft.com/en-us/onedrive/developer/rest-api/api/subscription_post_subscriptions?view=odsp-graph-online

How to modify the controller method when using .Net Core Web Api? It doesn't work on .net core no calls to the method are performed.

@sshresthareach This article uses PostMan to show the webhook. There's another article int his same section that has a full reference implementation you can explore: https://docs.microsoft.com/en-us/sharepoint/dev/apis/webhooks/webhooks-reference-implementation


Character encoding issue

Hi,
While using Pebble I have encountered an issue where a word like "add'tl" is encoded to "add#39;tl" after template evaluation. How to fix this? Test code is below.
@Test
public void testTemplate() throws Exception {
PebbleTemplate template = pebbleEngine.getTemplate("{{msg1}}");
StringWriter writer = new StringWriter();
template.evaluate(writer, Collections.singletonMap("msg1", "Hello add'tl"));
System.out.println(writer);
}

Actual result : Hello add'tl
Expected result : Hello add'tl
Any help would be much appreciated.

It's not an encoding problem, it's the autoescaping filter. You should read the escaping section of the doc here : http://www.mitchellbosecke.com/pebble/documentation/guide/basic-usage
You can use {{msg1 | raw}} if you don't want to escape it (http://www.mitchellbosecke.com/pebble/documentation/filter/raw)

thanks a lot, I will try and post back


0.30.2

Handle Volvo's with dashes in their name (@molobrakos)
Fix some html5 push notification configuration options were discarded after first use (@T3m3z)
Fix Homematic device name with autodiscovery (@pvizeli)
Make 'pin' optional for zigbee device config (@flyte)
Fix when sending a notification to a service with target attached (i.e. notify.html5_unnamed_device_2) the target was not submitted to the platform as a list causing iteration over every character in the string. (@robbiet480)

Not releasing this yet as Robbie wanted to get https://github.com/home-assistant/home-assistant/pull/3769 in but it doesn't seem to be ready yet.

#3769 has been fixed.

May want to hold for #3821?

#3821 has been fixed by #3826. Let's merge #3826 into this.


EPMLABSBRN-128 add initial data and files

EPMLABSBRN-128
add initial data and files

it is just preporation to https://github.com/Brain-up/brn/issues/73
where hard code would be removed


Eliminate redundant device moves for callbacks

🚀 Feature
Make it possible to access batches on callbacks without moving them to the CPU
Motivation
Currently, on callback hooks like on_train_batch_end where a batch tensor is available, the tensor is on cpu by default. If I want to, for example, run that batch through another model, I have to move it back to CUDA with batch.to('cuda') even though it must have been on CUDA in order for the training step to have taken place. This creates a serious performance bottleneck.
Pitch
Either add an argument to on_train_batch_end and similar hooks for cuda or add new hooks for before the move takes place.
Additional context
https://github.com/PyTorchLightning/pytorch-lightning/discussions/6945

Dear @import-antigravity,
I don't think it would be good practice to implement a second model forward in on_train_batch as Lightning won't be able to optimise properly.
You should encapsulate your logic in training_step directly and use manual_optimization if needed.
Best,
T.C

Turns out this is a bug, see https://github.com/PyTorchLightning/pytorch-lightning/issues/7377


Chapter 06 - DuplicateKeyException Test for MongoDB

Not an issue, more to highlight for others if they run into the problem.
I couldn't get the duplicate() test for the product-service to pass for a while, until I looked at the source code for ProductServiceApplication in the code repo for Chapter06. Apparently, the @Indexed annotation isn't sufficient to actual create the unique index on productId, you need to use the IndexOperations ensureIndex functionality that's part of the Spring Data MongoDB library. So be sure to examine and copy the code from Chapter 06 here
Again, not an issue with the code, but it was something that wasn't called out specifically in the book, and this might be new to folks who are new to Spring Data MongoDB and index creation.
So far, enjoying the book, thanks!

Hello Jason!
Well spotted, that should have been mentioned in the book!
I have added an Errata section in the README.md file where I explain how index creation is handled for MongoDB.

Thanks all. This saved me lots of time to find out the issue. thank you very much!


[stdlib] Set, Dictionary: Make Cocoa indices resilient for now

This makes _CocoaSet.Index and _CocoaDictionary.Index fully resilient for now, allowing us to replace/fix its internals at any point in the future.

@swift-ci please smoke benchmark

@swift-ci please smoke test

cc @airspeedswift What do you think? This leaves the door open for a Cocoa indexing overhaul.

If you see any unexpected regressions, you should consider fixing the regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false alarms. Unexpected regressions which are marked with '(?)' are probably noise. If you see regressions which you cannot explain you can try to run the benchmarks again. If regressions still show up, please consult with the performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 8-Core Intel Xeon E5
  Processor Speed: 3 GHz
  Number of Processors: 1
  Total Number of Cores: 8
  L2 Cache (per Core): 256 KB
  L3 Cache: 25 MB
  Memory: 16 GB

Gosh; Dictionary.Keys and .Values uses IndexingIterator. That won't do at all.

I think #19576 (for swapAt) and #19602 (for Histogram et al) will cut down these performance regressions.

CStringLongAscii
3540
3289
-7.1%
1.08x

Code size: -Osize

TEST
OLD
NEW
DELTA
RATIO

Regression

Histogram.o
4480
5792
+29.3%
0.77x

DictionarySwap.o
28251
31830
+12.7%
0.89x

RGBHistogram.o
24529
26894
+9.6%
0.91x

DictOfArraysToArrayOfDicts.o
33558
34862
+3.9%
0.96x

SetTests.o
52945
54187
+2.3%
0.98x

Performance: -Onone

TEST
OLD
NEW
DELTA
RATIO

Improvement

RemoveWhereFilterString
1607
1314
-18.2%
1.22x

RandomShuffleLCG2
39131
34406
-12.1%
1.14x

RemoveWhereSwapInts
5303
4860
-8.4%
1.09x

RemoveWhereSwapStrings
5996
5586
-6.8%
1.07x

How to read the data
The tables contain differences in performance which are larger than 8% and differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false alarms. Unexpected regressions which are marked with '(?)' are probably noise. If you see regressions which you cannot explain you can try to run the benchmarks again. If regressions still show up, please consult with the performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

Hm, I'll have to make the Cocoa Index storage representation @_fixed_layout to reduce these regressions.
I'll do that by switching it to a CoW type, which I was planning to do anyway.

@swift-ci please benchmark

@swift-ci please smoke benchmark

@swift-ci please smoke benchmark

@swift-ci please smoke benchmark

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

Oops, I forgot to apply these changes to Dictionary; I'm afraid these next benchmarks will still disappoint

@swift-ci please smoke benchmark

@swift-ci please smoke benchmark

@swift-ci please smoke benchmark

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

Sigh
@swift-ci please smoke test linux platform

Okay, this should do the trick. (He said, with a naive look on his face.)

Build comment file:
Performance: -O

TEST
OLD
NEW
DELTA
RATIO

Improvement

CStringLongAscii
3552
3299
-7.1%
1.08x (?)

CharIteration_chinese_unicodeScalars_Backwards
225786
247267
+9.5%
0.91x

How to read the data
The tables contain differences in performance which are larger than 8% and differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false alarms. Unexpected regressions which are marked with '(?)' are probably noise. If you see regressions which you cannot explain you can try to run the benchmarks again. If regressions still show up, please consult with the performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

Oh hi! I'm just sitting over here, improving benchmarks by making things non-inlinable.
(This probably just means we lack indexing benchmarks for bridged sets/dictionaries.)


werf: 1.2.197 -> 1.2.198

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/werf/werf/releases.
meta.description for werf is: GitOps delivery tool
meta.homepage for werf is: https://werf.io
meta.changelog for werf is: https://github.com/werf/werf/releases/tag/v1.2.198
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.2.198 with grep in /nix/store/kcb0298jppk93afvjhbammv1y437wxcs-werf-1.2.198
found 1.2.198 in filename of file in /nix/store/kcb0298jppk93afvjhbammv1y437wxcs-werf-1.2.198

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
werf

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kcb0298jppk93afvjhbammv1y437wxcs-werf-1.2.198
ls -la /nix/store/kcb0298jppk93afvjhbammv1y437wxcs-werf-1.2.198/bin

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


Fix coverage collection

Proposed changes
This PR updates the tox config to run tests for dev packages only and adds no cover marker in the __init__.py modules
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

looking at the code cov report one observes that

several __init__.py files still are reported to be at 0% (instead of "No data available")
<...>_pb2.py files are being tracked, which are autogenerated from protobuf and should not be included

looking at the code cov report one observes that

several __init__.py files still are reported to be at 0% (instead of "No data available")
<...>_pb2.py files are being tracked, which are autogenerated from protobuf and should not be included

Why we want to add pragma: no cover to each individual __init__.py files instead of the specify the file pattern in .coveragerc?

I suggest you exclude the _pb2.py in the setup.cfg as we usually do.
Re __init__.py I prefer selective no cover as sometimes they do contain logic and a blanket ignore would be too intense.


Add initial attempt at an ssh plugin

First of all, full credit for the build work here goes to @Shanjian
This adds a basic SSH plugin that will work with username/password auth.
Calling connect gets you a connection object that then has methods for
either executing a command or disconnecting.
Coming soon: auth by key

Review status: 0 of 161 files reviewed at latest revision, 3 unresolved discussions, all commit checks successful.

client/plugin-src/cordova-plugin-ssh/src/android/Ssh.java, line 90 [r1] (raw file):
This throws an exception and it ends up getting switched to a rejected promise.

client/plugin-src/cordova-plugin-ssh/src/android/Ssh.java, line 108 [r1] (raw file):
Done.

client/plugin-src/cordova-plugin-ssh/ssh.js, line 6 [r1] (raw file):
Done.

Comments from the review on Reviewable.io


Minor fixes

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Numbers: #2978 #2890 #2968 #3086
What is the current behavior?
What is the new behavior?
Other information

Thanks for the PR @zxawry ! You know the drill, shoot us an email once this gets reviewed/merged.


Use openssl from Homebrew for TruffleRuby on macOS

Fixes https://github.com/oracle/truffleruby/issues/1818

The openssl built by ruby-build was not used by TruffleRuby,
and causes https://github.com/oracle/truffleruby/issues/1818
Improve error message when openssl from Homebrew is not available.
Change the definition code so it checks the operating system too.

cc @hsbt

I'll merge this now as it's kind of an urgent fix for TruffleRuby macOS ruby-build/rbenv users.

👍


Add documentation on resources application

This has been partially completed in PR #556.

The documentation appears to be at a decent level now that I'll close this issue.


unityhub: add unity 6 editor dependencies

added: [ harfbuzz ]
fixes: #346038

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

@huantianad this is my first contribution here, i hope i didn't mess anything up. its just a +3-0 change :')


The teacher information (exercise) is not being commited

I wrote down some notes at the meta/teacher information field (exercise), and published the guide,  but I don't see it in repo

In which item? Exercise or guide?
El vie., 10 de jun. de 2016 a la(s) 15:28, huenu<EMAIL_ADDRESS>escribió:

I wrote down some notes at the meta/teacher information field (exercise),
and published the guide, but I don't see it in repo
https://github.com/Orga-UNQ/mumuki-guia-q3-rutinas
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mumuki/mumuki-teacher-tools/issues/100, or mute the
thread
https://github.com/notifications/unsubscribe/AD6mDPI4tETjbySgBlXsn_UzbwSPrbBbks5qKay_gaJpZM4IzND_
.

@aguspina could you check both?
@huenu any ideas?

I think its a bibliotheca bug
El sáb., 11 de jun. de 2016 11:17, Franco Leonardo Bulgarelli <
<EMAIL_ADDRESS>escribió:

@aguspina https://github.com/aguspina could you check both?
@huenu https://github.com/huenu any ideas?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mumuki/mumuki-teacher-tools/issues/100#issuecomment-225364909,
or mute the thread
https://github.com/notifications/unsubscribe/AD6mDExeKUD-TVRG6KZkv-qTTybMIBrlks5qKsNsgaJpZM4IzND_
.

Lets assign high priority to it


Reduce the number of LLM parsing error with conversational chats

A lot of times, when the LLM responses a Final Answer with markdown in the answer text (e.g. asking the LLM to write some code), the parser raises an OutputParserException("Could not parse LLM output: ..."), when the answer from the LLM is totally acceptable.
This change should reduce the number of such occurances.

Description: Reduce the number of LLM parsing error with conversational chats,
Issue: the issue #1358 ,
Dependencies: None,
Tag maintainer: @hinthornw @baskaryan,

For example, this response from gpt-turbo-3.5 for "How to convert a list of dict to a JSON in Python" seems totally acceptable as the final answer, but will raise the OutputParserException, even with handle_parsing_errors=True. This PR fixes it.

I understand this is probably not the most elegant solution, so any suggestions are welcomed!

I'm currently facing this parsing issue problem and i tried appling this solution but i'm just getting the below output rather than the actual answer. I asked it to write a python function that adds two numbers together (which I was doing as a test to see if the bot worked and kept getting the parsing error).
{ "action": "Final Answer", "action_input": "Sure! Here's a Python function that adds two numbers together:\n\npython\ndef add_numbers(a, b):\n return a + b\n" }

Thanks, no longer relevant, I suggest using an agent with tool-calling


@estebank I would say discoverability, guessability, and -- maybe the biggest issue to me -- every one-more-thing you have to explain risks being the one thing that blows a newcomer's cognitive budget. Don't get me wrong, overall I've been pleasantly surprised at how quickly my colleagues have gone from zero to productive users of Rust. But there are still so many little surprises in the onboarding experience, that it's easy for a learner to just blow their stack and give up. (I think I'm just basically giving an exegesis for why I'm so excited about all the effort Rust is putting into ergonomics, at least the learning side of it.)

@estebank I would say discoverability, guessability, and -- maybe the biggest issue to me -- every one-more-thing you have to explain risks being the one thing that blows a newcomer's cognitive budget. Don't get me wrong, overall I've been pleasantly surprised at how quickly my colleagues have gone from zero to productive users of Rust. But there are still so many little surprises in the onboarding experience, that it's easy for a learner to just blow their stack and give up. (I think I'm just basically giving an exegesis for why I'm so excited about all the effort Rust is putting into ergonomics, at least the learning side of it.)
@SimonSapin My only point about JavaScript was that it shows us that it does in fact work in practice, at scale, to make de facto compatible changes. JavaScript of course does not tell us what we should do. From my perspective, doing it with language editions means we wait several years longer to solve a pain point simply because we have a mechanism that lets us do so, but that's obviously biased by my experiences I mentioned. Can you help me understand the rationale for why you wouldn't want to make the change now? Are you concerned about de facto risks that we haven't actually uncovered in our empirical investigations? Or is it an aesthetic about clarity of policy?
Originally posted by @dherman in https://github.com/rust-lang/rfcs/pull/2544#issuecomment-422193876

Oops... sorry... accidentally created.


ui: Add identity list

Please fix the shadow of close icon in identity list.

Hi, any updates here? @zhangdichn


This policy is for "Sign-in" risk. The instructions say to select "User risk".

[Enter feedback here]
This policy is for "Sign-in" risk. The instructions say to select "User risk".
"Conditions > User risk, set Configure to Yes. Under Select the sign-in risk level this policy will apply to
Select High and Medium."

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

ID: 199e51c2-d6f6-ed8f-39a2-4f395155cb50
Version Independent ID: 66ae328e-38b8-fe84-5ee9-976adbb99135
Content: Sign-in risk-based Conditional Access - Azure Active Directory
Content Source: articles/active-directory/conditional-access/howto-conditional-access-policy-risk.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@s0041464
Thanks for your feedback! We will investigate and update as appropriate.

Thanks. I just made a pull request to update this. https://github.com/MicrosoftDocs/azure-docs-pr/pull/132176


Convert readme to rst

refs #35

Don't know if you can see https://readthedocs.org/projects/max7219/builds/4467998/ (output as below), but the build failed with imgmath, so I just reverted back to pngmath and now the docs are generating ok - http://max7219.readthedocs.io/
Running Sphinx v1.3.5
making output directory...

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/max7219/envs/latest/local/lib/python2.7/site-packages/sphinx/cmdline.py", line 243, in main
    opts.warningiserror, opts.tags, opts.verbosity, opts.jobs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/max7219/envs/latest/local/lib/python2.7/site-packages/sphinx/application.py", line 144, in __init__
    self.setup_extension(extension)
  File "/home/docs/checkouts/readthedocs.org/user_builds/max7219/envs/latest/local/lib/python2.7/site-packages/sphinx/application.py", line 435, in setup_extension
    err)
ExtensionError: Could not import extension sphinx.ext.imgmath (exception: No module named imgmath)

Extension error:
Could not import extension sphinx.ext.imgmath (exception: No module named imgmath)

that's a really old version of Sphinx @rm-hull. If you use the latest 1.4.x you'll see the warning. But I guess this is a better solution, being able to support both.

or maybe even better, simply remove the reference, we're not using imgmath or pngmath afaik.

Presumably readthedocs selects the sphinx version when it builds, but you're right: i'll just remove the reference :)


Settings should return resolved values

I would like to start a PR to change how settings are read, inspired by how allauth does that (https://github.com/pennersr/django-allauth/blob/master/allauth/account/app_settings.py):
Add a new file shop/app_settings.py. This would allow us to import settings without having to rename the module:
from django.conf import settings
# from shop import settings as shop_settings
from shop import app_settings

In this File, create a class AppSettings that has properties for each setting. This property returns already the resolved value, e.g. SHOP_CUSTOMER_SERIALIZER currently returns a string, every module reading that property needs to import that module to use that serializer.
https://github.com/pennersr/django-allauth/blob/master/allauth/account/app_settings.py#L200
A nice way to keep the settings compliant is to replace the module by an instance of the AppSettings:
https://github.com/pennersr/django-allauth/blob/master/allauth/account/app_settings.py#L328
What is also an advantage next to getting resolved values:

We avoid the trap of circular dependencies, as the evaluation of a setting is only done when the setting is accessed, not when the shop settings are imported the first time.
We allow that a project can test against different settings by using override_settings (https://docs.djangoproject.com/en/1.10/topics/testing/tools/#django.test.override_settings)

For backward compatibility the shop/settings.py would also return that instance of AppSettings from shop/app_settings.

I never encountered this problem, but I get your point. For SHOP_CUSTOMER_SERIALIZER ist probably is useless, since that directive should go away anyway, but for all other directives in shop.settings.py we could follow that approach.
So we can schedule this for 0.9.5.

This now is part of version 0.10, so I assume we can close it?


sqlalchemy does not support the mysql data type: geometry

Describe the bug
When I am performing the integration operation with the datahub command, an error is reported：
lib/python3.8/site-packages/sqlalchemy/dialects/mysql/reflection.py:192: SAWarning: Did not recognize type 'geometry' of column 'GeoShape'
To Reproduce
Steps to reproduce the behavior:

datahub ingest -c

Screenshots

Desktop (please complete the following information):

DataHub Version [0.8.17]
MySQL Version [4.7]

@iduanyingjie the fix was just merged, can you check it now?

Closing this issue as fixed via #3726.
pip acryl-datahub>=<IP_ADDRESS> should have the fix.
Please re-open if this is still an issue.


Correction for raw affiliation Department of Physics and Astronomy, The Johns Hopkins University, Baltimore, MD 21218, USA; Laboratory for X-ray Astrophysics, NASA/Goddard Spaceflight Center, Mail Code 662, Greenbelt, MD 20771, USA

Correction needed for raw affiliation Department of Physics and Astronomy, The Johns Hopkins University, Baltimore, MD 21218, USA; Laboratory for X-ray Astrophysics, NASA/Goddard Spaceflight Center, Mail Code 662, Greenbelt, MD 20771, USA
raw_affiliation_name: Department of Physics and Astronomy, The Johns Hopkins University, Baltimore, MD 21218, USA; Laboratory for X-ray Astrophysics, NASA/Goddard Spaceflight Center, Mail Code 662, Greenbelt, MD 20771, USA
new_rors: 00za53h95;0171mag52
previous_rors: 00za53h95
works_examples: W2030094391;W2082592477;W3100701982;W3215847683
contact: 5155a3bf3cfa519804f824d2c66ecccd:afaf9257 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Install for Maintenance Solution seems broke on the URL

Install-DbaMaintenanceSolution seems broke on trying to download from GitHub. Also when the download fails the extraction is trying to still run so fails with red because of the file not existing.

Command being run:
Install-DbaMaintenanceSolution -SqlInstance server -Database master -LogToTable -Solution IndexOptimize

Module version: 0.9.321

I've found a workaround for this.
I made a change to the allcommands.ps1 in module version (0.9.326)
Comment out the Start-BitsTransfer line for the Install-DbaMaintenanceSolution and replace with:
$wc = New-Object System.Net.WebClient
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
$wc.DownloadFile($url,$zipfile)

Without specifying TLS1.2 the transfer encounters an SCHANNEL error.  Believe the possible cause this has to do with github URL's and Amazon redirects, though could be wrong, didn't explore the full reasons.
I was unable to get Start-BitsTransfer to work even with specifying the security in the manner I did so tried using the .NET WebClient which worked!
NetMon or Ethereal(WireShark) can be helpful if someone wanted to look into this further.

already fixed via https://github.com/sqlcollaborative/dbatools/commit/d203b4e8543f8aa7c41fdc14593e3bfa08965d46


Add fix for field guide images in Webkit

This is a (hopefully temporary) fix for the issue we were having where images wouldn't display in the field guide dialog in Webkit (Safari). We now replace the images in the field guide dialog with the 512 square versions only on Webkit so we can set a default width/height for those.

The photos all load but they look very squished on my iPhone SE.

@audreylemeur This only happens on iOS 10, which we don't support.


Internal HEAD request not working with Pre-signed URLs

It looks like only URLs without query params are supported. The gem seems to toss away the query part which may result in 403 errors for Amazon S3 resources where the auth token is part of the query param. On top of that, the gem just throws a "the file does not exist" error which totally obscures the 403 code.

As a workaround, I'm doing this:
def ffmpeg_video_from_remote(remote_file_url)
  open(File.join('public', file.file.filename), 'wb') do |file|
    file << open(remote_file_url).read
    FFMPEG::Movie.new(Rails.root.join(file.path).to_s)
  end
end

Edit: I was wrong. It looks like the query params do get used but what is happening is my pre-signed URL to Amazon S3 is only meant to be used with GET but the library does a HEAD request.

Ok


[FR] - Indicate graceful shutdown with a log message

Internal/External
External
Area
Cardano Node
Describe the feature you'd like
Currently, there is no indication in the log of graceful shutdown. i.e. you cannot tell whether the process was killed and will take > 15min to restart (because of block data re-validation) or gracefully shut down with no additional restart time
Describe alternatives you've considered
None. IMHO, it'd be a simple/useful addition
Additional context / screenshots
More context about this is given in #2608 and #2627

Just checked and actually there is a log:
Shutting down..

when killed with SIGINT. The log is absent when killed with SIGTERM.

Yes, this is out of date


Fix for #118: StringType defaults to TEXT which is varchar(256)

Current coverage is 95.07%

Merging #119 into master will not affect coverage as of ed280b6

@@            master    #119   diff @@
======================================
  Files           12      12
  Stmts          508     508
  Branches       122     123     +1
  Methods          0       0
======================================
  Hit            483     483
  Partial          0       0
  Missed          25      25

Review entire Coverage Diff as of ed280b6

Powered by Codecov. Updated on successful CI builds.

Thanks for submitting this PR. I left a bunch of nitpicky comments because I'm kind of a perfectionist; if you don't have time to address them, I can take care of them myself on merge tomorrow.

Ha! Neh I'll do it.

I made the example slightly less abstract too.

Merged. Thanks!


[Fix] Improved UI in error screen

[Fix] Improved UI in error screen

Se arregla la UI de pantalla de error para que se parezca un poco más a los diseños actuales.

Antes

Después

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@b3b83b0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #862   +/-   ##
==========================================
  Coverage           ?   31.63%
==========================================
  Files              ?      326
  Lines              ?    17392
  Branches           ?     1556
==========================================
  Hits               ?     5502
  Misses             ?    11231
  Partials           ?      659

Continue to review full report at Codecov.

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


[Travis] Test lowest version dependencies

To avoid BC breaks its best to test the normal dependencies and the lowest dependencies

Thanks, I was already thinking about that.
I prefer not to use a JSONish syntax, so could you please update both PRs to follow a syntax like the following:
matrix:
  include:
    - php: 5.5
      env: 'COMPOSER_FLAGS="--prefer-stable --prefer-lowest"'

I guess that could work just like the same.
Thanks again

@sagikazarmark Code has been updated


Cut release v0.26.2

What's Changed

Fix NetworkMachineIndicator and machines dynamically showing in CommandBar (#4311)
Bugfix: arc paths were stored as straight line paths (#4310)
Make application aware it saved the buffer and not something else (#4314)
Add a radius length indicator to the circle sketch tool (#4304)
Update machine-api spec (#4305)
Chore: Don't let draft lines receive mouseEnter/Leave events, or create invalid overlays (#4306)
Tags should refer to full paths, not just base paths. (#4299)

Full Changelog: https://github.com/KittyCAD/modeling-app/compare/v0.26.1...v0.26.2

Windows:
Manual Tasks

[ ] Mash keyboard in code editor
[ ] https://github.com/KittyCAD/modeling-app/issues/3373
[ ] coredump will open github issue with stuff populated (from home and project page) there are different ways to trigger it as well (! button at top, bug at bottom, mod + shift + .)
[ ] running app.{app,exe} path/to/file opens the kcl file (all oses)
[ ] running app.{app,exe} path/to/dir opens the directory and any kcl file in it or creates a main.kcl if empty (all oses)
[ ] If someone at the office, test you can hit make from the app and it works
[ ] Test updating versions works
[ ] until we have a performance test, sanity check that time to open file directly from command line is not shit
[ ] https://github.com/KittyCAD/modeling-app/issues/3455
[ ] https://github.com/KittyCAD/modeling-app/issues/3469
[ ] https://github.com/KittyCAD/modeling-app/issues/3465
[ ] double click to open files from finder opens new window each time (mac/windows)
[ ] export from the app works and opens a file picker ux (needs to be manual since we override for playwright)
[ ] Ensure that a loading toast appears when testing the auto-updater, saying the download has begun
[ ] https://github.com/KittyCAD/modeling-app/issues/4038

Mac:
Manual Tasks

[ ] Mash keyboard in code editor
[ ] https://github.com/KittyCAD/modeling-app/issues/3373
[ ] coredump will open github issue with stuff populated (from home and project page) there are different ways to trigger it as well (! button at top, bug at bottom, mod + shift + .)
[ ] running app.{app,exe} path/to/file opens the kcl file (all oses)
[ ] running app.{app,exe} path/to/dir opens the directory and any kcl file in it or creates a main.kcl if empty (all oses)
[ ] If someone at the office, test you can hit make from the app and it works
[ ] Test updating versions works
[ ] until we have a performance test, sanity check that time to open file directly from command line is not shit
[ ] https://github.com/KittyCAD/modeling-app/issues/3455
[ ] https://github.com/KittyCAD/modeling-app/issues/3469
[ ] https://github.com/KittyCAD/modeling-app/issues/3465
[ ] double click to open files from finder opens new window each time (mac/windows)
[ ] export from the app works and opens a file picker ux (needs to be manual since we override for playwright)
[ ] Ensure that a loading toast appears when testing the auto-updater, saying the download has begun
[ ] https://github.com/KittyCAD/modeling-app/issues/4038


MF-552 - Use event sourcing to keep Bootstrap service in sync with Things service

What does this do?
This PR adds the use of event sourcing from Things service to keep corresponding Bootstrap service entities in sync.
Which issue(s) does this PR fix/relate to?
This PR resolves #552.
List any changes that modify/break current functionality
There are 2 major API changes:
- List of Configs does not return Channels
- Update Config is split to Config fields update and Config connections update
Have you included tests for your changes?
Yes.
Did you document any new/modified functionality?
Yes, docs are up to date.

Codecov Report

Merging #603 into master will decrease coverage by 0.89%.
The diff coverage is 77.43%.

@@            Coverage Diff            @@
##           master     #603     +/-   ##
=========================================
- Coverage   87.13%   86.24%   -0.9%
=========================================
  Files          62       62
  Lines        3437     3621    +184
=========================================
+ Hits         2995     3123    +128
- Misses        304      348     +44
- Partials      138      150     +12

Impacted Files
Coverage Δ

bootstrap/state.go
100% <ø> (ø)
:arrow_up:

bootstrap/api/transport.go
97.89% <100%> (+0.15%)
:arrow_up:

bootstrap/api/endpoint.go
100% <100%> (ø)
:arrow_up:

bootstrap/postgres/init.go
92.85% <100%> (+3.11%)
:arrow_up:

bootstrap/postgres/configs.go
67.41% <64.48%> (-8%)
:arrow_down:

bootstrap/api/requests.go
95.55% <80%> (-4.45%)
:arrow_down:

bootstrap/service.go
84.42% <90.47%> (+2.65%)
:arrow_up:

Continue to review full report at Codecov.

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


Open source license

I hope this message finds you well! I recently stumbled upon your amazing open source repository,  and I must say, I'm truly impressed with your work. I wanted to reach out and kindly request something important: adding a license to your repository. Your project has great potential, and by including a license, you can provide clarity and legal protection for users and contributors while fostering collaboration.
Would you consider adding a commonly used open source license, such as the MIT License or Apache License 2.0?
Thank you!

Thanks for the suggestion! I didn't attach a license as this repo is not officially a Samsung repo, but it is very likely a MIT license can be added.


Implement pet list

To make revision easier, I'll try to put screenshots of the app after each relevant commit.
This approach will be an experiment.

I can't decide what should appear in the right pane after launching the app.

Should it show the details of first pet entry?
Should it be blank?

I'm more willing to go with the "blank" option, but it won't be exactly a blank page.
It would be an adw::StatusPage, although it also brings up more design choices:

Should it say "No Pet Selected"?
Should it say "Select a Pet"?

I'm having trouble using a custom icon in the adw::StatusPage, though:

I can't figure out why.

Showing pet details
I like the adw::StatusPage approach, but I had a problem with the icon, so I just left it out.
Not up to my taste, but I'll leave this as is for now until I figure it out how to make it work, which will probably be in another PR.

Adding pets
Notice how they're listed in the order they're added.
Also notice that the you can click on the button and start typing the pet name right away, since the field gains focus automatically.

Screencast from 2024-04-12 22-29-37.webm
Now they're in alphabetical order.
I think it makes more sense.
It's this way in a "Contacts" app.

I realize now that the sidebar should be an adw::StatusPage for when there are no pets yet.

Not sure I like the description mentioning + the way it is, but it is basically the example in the HIG, but adapted.

I was thinking of implementing persistence in this PR, but I think it'll just get bigger.
Also, since these pet models aren't complete (they just have a name), I think it'll just be harder to adapt them later while preserving the user's current ones.

Here's a recap of all the things until now:

List pets in the sidebar
Show pet details in the right pane
Add new pets (in alphabetical order)

Here's how it currently looks like:
Demo


python3Packages.pywizlight: 0.4.4 -> 0.4.5

Motivation for this change
Update to latest upstream release 0.4.5
Change log: https://github.com/sbidy/pywizlight/releases/tag/v0.4.5
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

3 packages built:

python37Packages.pywizlight
python38Packages.pywizlight
python39Packages.pywizlight

Result of nixpkgs-review pr 112867 at 4d9d1840 run on aarch64-linux 1

3 packages built:

python37Packages.pywizlight
python38Packages.pywizlight
python39Packages.pywizlight


AC resetting to "Cool" instead of restoring previous state

When AC is running on Auto for example and you turn it off and then on it will not start in the state it was in when it was turned off but it resets to Cool mode.

We don’t change mode during power on/off. Hose to reproduce, on my side is working as expected, may be someone change the mode with  RC or MELCLoud app.

The remote is not being used. MELCloud as well. This used to work in previous versions.
To reproduce, you set AC to AUTO, turn it off, then turn on again and it will set it self to COOL instead of starting in a state that it was left in when turned off.


[FLINK-22818][runtime] IgnoreInFlightDataITCase is blocking the map s…

…ubtasks until the first checkpoint barrier reaches the sink

What is the purpose of the change
Fix the test - IgnoreInFlightDataITCase
Brief change log

IgnoreInFlightDataITCase is blocking the map subtasks until the first checkpoint barrier reaches the sink

Verifying this change
The fix test itself
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@rkhachatryan I pushed the separated commit for easing review but then I can merge it with the previous one.

@rkhachatryan I pushed one more version but infinity loop until success doesn't look the best idea. I will think more about it but anyway you can take a look at the last changes.

@flinkbot run azure

CI failure unrelated, merging.


[FEA]Spark 3.1 FileSourceScanExec adds parameter optionalNumCoalescedBuckets

see the change in Spark at https://github.com/apache/spark/commit/7b8683820bf408a82fcdb8716d963990a6955cee.
We need to handle this in the plugin.

this is fixed by https://github.com/NVIDIA/spark-rapids/pull/547


Simplify raster tile layer / key / database name handling

Current complexity:

DOMAIN_TO_DB_MAP is a JSON string in an environment variable (mapping domain to database name)
URL requests for a raster layer start with a domain, then other keys to find the right TIFF
terracotta metadata tables have columns for "type" (which is currently passed from the URL domain), then other keys to find the right TIFF

Suggest:

[x] remove the DOMAIN_TO_DB_MAP JSON environment variable
[x] use the same string for domain and db name
[x] OR use the raster_tile_sources table to transform URL domain to db name
[ ] remove "type" variable from all etl pipeline metadata JSONs
[ ] rename "type" variable to "hazard" for isimip and aqueduct (where it is needed to find the right TIFF)
[ ] refactor backend handling of URL path parts so domain is separate from tiff keys (see use of _parse_keys)
[ ] adjust frontend requests for Aqueduct and ISIMIP

currently, domain is used to identify db and then also as type key https://global.infrastructureresilience.org/api/tiles/fluvial/100/baseline/present/WATCH/3/4/3.png?colormap=blues&stretch_range=[0,5]

/api/tiles/{domain}/{return_period}/...

adjusted, domain will only be used to identify db, then for Aqueduct and ISIMIP we will need a hazard key to distinguish coastal/fluvial or extreme heat/drought https://global.infrastructureresilience.org/api/tiles/aqueduct/fluvial/100/baseline/present/WATCH/3/4/3.png?colormap=blues&stretch_range=[0,5]

/api/tiles/{domain}/{hazard}/{return_period}/...

Some improvement as of #182 - dropped DOMAIN_TO_DB_MAP and now use raster_tile_sources as source of truth for the backend to find source_db name for a given domain.


React-group-project: Reserved rocket

In this PR
I worked on:

Filtering through booked rockets and display the booked rockets on My Profile page

STATUS: APPROVED ✔
Hello @kkolade
Great job, I like how it's turning out so far, I think we can merge it now.
Highlights 🎯
The Pull Request follows GitFlow ✅
Your branch is properly named ✅
Your commits are descriptive ✅
Required Changes ♻

None

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
You can also consider:
N/A
Cheers and Happy coding!👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question, so I can receive the notification.


[Bug]: Ratings widget flashes previous rating when rating is reduced

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Reducing stars on rating widget causes previous value to flash before setting the selected rating

Steps To Reproduce

Click on a rating on a rating widget and reduce the rating - observe the widget flash previous value before settling on the lower rating

Environment
Release
Version
Cloud

Might be related to the change in private widget.
@ramsaptami can you confirm if this happens outside the list widget?

It happens when it's outside the list widget, i.e., on the canvas

@wmdev0808 kindly triage this issue


Enhancement/issue 16 make pipe static like

I thought we had already merged this changes, on last PR. Changes are still necessary on master to resolve issue #16

Can one of the admins verify this patch?

ok to test


net: TestReadTimeoutMustNotReturn failures with `use of closed network connection

2020-01-24T19:37:58-67539f6/darwin-amd64-race
--- FAIL: TestReadTimeoutMustNotReturn (0.33s)
    timeout_test.go:389: set tcp <IP_ADDRESS>:49974: use of closed network connection
    timeout_test.go:392: set tcp <IP_ADDRESS>:49974: use of closed network connection
    timeout_test.go:395: set tcp <IP_ADDRESS>:49974: use of closed network connection
FAIL
FAIL	net	16.643s

See previously #14986.
CC @ianlancetaylor @mikioh @bradfitz

greplogs -l -e 'FAIL: TestReadTimeoutMustNotReturn .*(?:\n .*)* use of closed network connection'
2022-06-15T21:16:14-0cd0c12/freebsd-arm64-dmgk
2021-11-04T21:52:51-1e0c3b2/openbsd-386-68
2021-09-23T19:41:41-0626ac0/openbsd-amd64-68
2020-01-24T19:37:58-67539f6/darwin-amd64-race

(CC @neild)


Fix subclasses taking way longer than they should to be constructed

Currently, there's about a 1.46 second difference between vanilla subclassing and our subclassing, over 1 million calls to class().
This is due to the loop we use to copy metamethods from the superclass into the subclass. We currently have a bit of a messy fix for this, but it could still use some more work.
local function lookUpMetamethods(subclass, superclass)
  for _, metamethod in ipairs(INHERITED_METAMETHODS) do
    subclass[metamethod] = function(...)
      local method = superclass[metamethod]
      if method then
        return method(...)
      end
    end
  end
end

Before we implement this, it needs to be broken down into smaller functions. We might even want to try a different implementation. This was initially from an older version of Middleclass, and its current source code is vastly different. We might be able to find some inspiration for how to tackle metamethod inheritance in it.

Number 1 way to make it faster: remove ipairs loops and use simple for loops instead.


[10.x] Add mapToInstance() to Collection

A new Collection class mapToInstance() helps you to deal with class strings within a collection. It maps them into concrete classes using service container.
$jobs = [
        Job\One::class,
        Job\Two::class,
        Job\Three::class,
];

// Before
collect($elements)->map(fn(string $class) => App::make($class))->each->run();

// Now
collect($elements)->mapToInstance()->each->run();

A collection already has support to filter by class names with whereInstanceOf(), so this appears to be a consequent extension.

To round it up my questions are:

Exception handling: Is a dedicated Exception helpful to throw instead the Service Container Exception when thrown?
Should I add a @throws \Illuminate\Contracts\Container\BindingResolutionException?

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Multiple Remapped Keyboard Saves

Description of the new feature / enhancement
I'm hoping it can be made possible to have more than one saved keyboard mapping.
I have more than one situation where the keyboard could be mapped differently. None of the remappings are used on a very regular basis, but do need to be pulled up at least once per month. And because they are different situations, they required different mappings. So being able to create the mappings and save the configurations for future use (rather than having to change the current, single remapping to fit a different situation each time) would be greatly welcomed.
I submitted a different, but somewhat similar request about two years ago to be able to save a keyboard mapping that could be turned on and off for when I need to call it into action. That change was made quite quickly...and it's working very well. It's a little thing, but it's one of those things that just works every time I need it to. It saves me a lot of frustration. THANK YOU SO MUCH FOR ENABLING THIS FEATURE!!!!!
MIKE
Scenario when this would be used?
I would use the various keyboard mappings when the various applications and situations for which I'm entering data, is called upon.....which is on at least a monthly basis.
Supporting information
No response

/dup #1881


Feature request: Allow saving multiple solutions per problem

Sometimes we solve the same problem with multiple approaches: brute-force, then recursion, then add memoization, then DP. Ideally, we would like to preserve all those solutions in the repository.
Suggestion:
Allow a custom postfix for the file when saving it using Sync w/ LeetHub?
Alternatively, if the button was available in Submissions table, it could take the postfix from Notes field in the table?

I have faced this issue what I would noramlly do is just have my other approaches in comments. It's not the best way but works for me.
at the extension level I was thinking of adding a custom folder input which would be create a subfolder in the question dir itself
would like to hear what are your thoughts on it @arunbhardwaj


More proposals

Checking in more in-progress proposals in the hopes of sparking discussion and/or guiding future implementation work.

Your proposal text doesn't compile without errors ;)
On error handling
The point is brought up about different types of errors - but the rest of the discussion seems to be about the user handling scenario. The other scenarios are perhaps worth discussing.
For example something akin to rusts panic. Both as a language construct, a way of reporting kinds of failures. Being able to handle in some limited way. What happens to defer? What happens with multiple failures. What data can be carried around. What guarantees does the runtime supply (like closing handles etc).
There is also the kind of markup we see within Slang code. assert or marking something as unexpected. All these could lead to panics and in debug builds producing some useful information/breaking into debugger etc.

Typically there is neither a reason nor a desire to do anything about nearly-unrecoverable errors; the program has well and truly crashed.

It seems to me the panic mechanism is reasonable and useful idea over just crashing.
Exceptions
Whilst exceptions introduce a variety of issues, there are a few things going for them

They have some kind of debugging environment support

None of what is discussed precludes this, but it's actually a pretty important feature
We have support in Slang codebase by changing SLANG_ON_... macros

Exceptions are close to zero cost in most modern implementations if an exception isn't thrown. If an exception is thrown it's cost is considerably more than a return value

We may want to support actual exceptions from an ABI point of view to support interop.
The point about ABI issues and exceptions is sound, but like some other recent issues, you can change the behavior 'at the boundary' such that inside the boundary does something different (faster etc) than what is available at the boundary. It is common in C++ code to do this at COM boundary for example.
Return codes/things
One nice thing about exceptions is that a payload can be varied, and it trafficed to a catch site as is.
A problem(?) with these kinds of return mechanisms, is that because different functions have different return types typically, there are mechanisms that translate the return type as the stack is unwound. This requires more programmer effort (to do the conversions), information can be lost, and requires more code (to traffic between types) as the stack unwinds.
Secondly it implies returning multiple things - so returning a struct for example. In Rust there is the nullptr optimization such that null can mean false on Optional. But that relies on pointers not being able to be null. If more than a pointer is needed, then the return type becomes bigger than a register, and this typically leads to a function returning a struct. This leads to much less efficient code as returning this way typically means through memory.
A work around might be returning an HRESULT like thing, and making the rest of the return value an out parameter. If the HRESULT indicates failure it can imply features of the out structure (such as it's invalid).
Go
Go has a by convention error return "system" through tuples. Having not developed much go, not sure what the pros and cons are around that choice. It is a far simpler language feature than what is being described here.
Other approaches

Mapping all 'error cases' to int type, or say a pointer to static memory
Using "thread local" memory as a side channel

On the last point, note that thread local memory doesn't actually have to be through thread local accesses (which are comparatively slow), if a context is threaded through an added function parameter. So as an example (with bad syntax to illustrate)..
float smallAndCheap(float f)
{
   // I can still *access* the context to write an error/panic/go to a handler etc
   // as can read the context from thread local memory (slow - but this is exceptional)
   // It can't 'throw' as in cause myFunc to unwind though
}

[can_throw] int myFunc(...)
{
   auto smallAndCheap(...);
}

The function with [can_throw] by it's signature is saying this from ABI point of view, threads a 'context'.
How does this context work across an ABI boundary - because thread localness is something that has ABI issues? Perhaps a context has to always be passed.
A thread local context can be handy in lots of ways...
How does this work with multithreading? The context belongs to the thread. If an op happens that can change threads, post that point any state tied to the context would have to be reloaded.

As a quick extra thought it is probably worth at least considering how 'error' codes could map down to something like a HRESULT.
Having a single type, with a well defined structure of course has some significant limitations (and no or very limited 'payload'). But it does mean that trafficing between error types across boundaries isn't required. If the code is 64 bits (rather than COMs 32), that provides quite a lot of extra bits to make error codes work across large code bases, in a modular fashion. One of the issues with 32 bit HRESULT is the 'facility' has relatively few bits. You don't want facilities to clash across modules.
Another problem, that could be somewhat addressed with language support would be mapping from 'HRESULT's back to user understandable strings.
HRESULT like type also encourages the defining and using of commonly used error code values.


ROS driver for the wrist camera

I have a robotiq wrist camera installed on UR5, and I wish to access the images through a ROS topic. I have noticed that on ROS wiki it says "Wrist Camera is currently not supported". I wonder when it will be supported, or whether there is a plan to support it.
Besides, is there another approach to access the camera images on the PC?
Thank you very much!

I have the same question. Do you have any methods to use the camera on PC now?
Thank you very much!

Hi, @Mingrui-Yu can I ask which topic did you use to get the image? Thank you.


Parsijoo is not being tested by Travis CI

I'm submitting a ...

[x] bug report
[ ] feature request

Current behavior:

Parsijoo is not being tested by Travis CI https://github.com/fossasia/query-server/blob/master/app/test_server.py#L40-L41
Expected behavior:

Parsijoo is being tested by Travis CI
Steps to reproduce:
Run the tests

Claiming this!