Add sandbox failures to SARIF report (#1276)

Description
Added executions with UtSandboxFailure to the SARIF report to display them on the Problems tab.
Fixes #1276, #1168
Type of Change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Automated Testing
org.utbot.sarif.SarifReportTest
Manual Scenario

Enable the checkbox "Display detected errors..." in the project settings

Run test generation on
 public class Example {
     public String property(String key) {
         return System.getProperty(key);
     }
 }

Check the detected error:

Checklist:

[x] The change followed the style guidelines of the UTBot project
[x] Self-review of the code is passed
[x] The change contains enough commentaries, particularly in hard-to-understand areas
[x] New documentation is provided or existed one is altered
[x] No new warnings
[x] New tests have been added
[x] All tests pass locally with my changes

Please check test generation for InnerCalls.doubleCallFactorial()
There is an error test with StackOverflowError generated for it:

there is almost empty stacktrace for it

the wrong line is addressed in code:

There are 1024 lines in stack traces for Recursion:

for example Recursion.fib(), .factorial(), .sum(), ...
Is it Ok?


Sqlplus glogin.sql

Hi,
I've noticed that vorax4 does not run when there is a glogin.sql file.
However,
Is there a way to change vorax4 environment "set options". because I could not figure it out yet.
For example:
I would like to change "set echo [on|off]"
Finally,
Thank you for all your efforts on producing such a great tool.

Hello,
The variable you're looking for is g:vorax_sqlplus_options. You may have a look here to see how it's defined.

Thank you.
For others looking at the same thread.
Here is how to do it.
put the following in your vimrc file:
let g:vorax_sqlplus_options =
\	['set sqlprompt "&&_USER@&&_CONNECT_IDENTIFIER>"',
\	'set echo off',
\	'set time off',
\	'set exitcommit off',
\	'set null null',
\	'set numformat 9,999,999,990.99PR',
\	'set timing off',
\	'set verify off',
\	'set autoprint on']
Thank you.


Allow for injection of TcpStream

What
Allows for creating a Context with a custom tokio TcpStream.
Why
I want to be able to configure the socket. For example, you could set connect, read, and write timeouts. Really you could go hog wild and do whatever you want.

The last pre-commit failures are unrelated. Please rebase on #217 or wait until this PR has been merged.

I would like to see this as well, I'm having issues where a write doesn't timeout forever.

Ping.

Just to make sure, this functionality isn't just replicating https://docs.rs/tokio-modbus/latest/tokio_modbus/client/tcp/fn.attach.html ?

Just to make sure, this functionality isn't just replicating https://docs.rs/tokio-modbus/latest/tokio_modbus/client/tcp/fn.attach.html ?

Indeed. Looks like this PR is obsolete.


sorting objects by property

Hey, thanks for creating this essential functionality for the JS/TS ecosystem. I have a feature request that I'd be willing to look into implementing as a PR, but I figured I'd raise an issue first to see if that's something that'd be appreciated.
I'd like to be able to sort an array of objects 'naturally' by some property:
const items = [
 {
   name: 'Foo',
   created: '04/21/2021'
  },
  {
   name: 'Bar',
   created: '04/20/2021'
  }
].sort(natsort({property: 'created'}))

Or even use an accessor function, which would be useful for nested items: [...].sort(natsort({property: (item) => item.nestedItem.created}))
I'd love to hear some thoughts.

@nmokkenstorm
var sorter = natsort();
items.sort(function(a, b) {
  return sorter(a. created, b. created);
});


Build.yaml issue with VSCode and flutter

Hi
When I create a build.yaml I get the following error
Property targets is not allowed.yaml-schema: JSON Schema for hammerkit's build file
This is using flutter and vs code. Any pointers please?

please direct questions to the discord

Having the same issue. What's the discord channel?

any solution to this @kinski76 https://github.com/gql-dart/ferry/issues/250#issue-985104474

https://discord.com/invite/tXTtBfC


code hint not work in some case

case 1:
var instance:Something = new Something();
instance
.property1
/* code hint will not work after this comment */
.property2
so code hint will not work with property2 (Animate CC do work. So I think it seem like a bug)
case 2 (adding code hint support for "with" statement
with (instance)
{
// code hint will not work here
}
Please consider to fix it.
Thanks!

Case 1 is an old limitation - we should try solving it but it's not trivial.
Case 2: we don't plan to support with.

Case 3:
instance
.func('code hint will not work after ; (SemiColon)')
.hintMePlease
This work:
instance
.func('replace SemiColon to \x3B to fix it')
.gotTheHintThanks

@elsassph I've found a trick to force Case 1 work: use three dot:
instance
.property1
// one line comment
/* multiline comment */
...[code hint showed up]
That mean, I can use three dot to force display the code hint, then delete 2 dot later.
So I think FD could emulate (reproduce) this behavior to fix this issue?

Update:
Code hint will not work in this case:
function(param1:String = '[', param2: <= does not work
Additional, there're no code hint for built-in method like "hasOwnProperty"
Hope this will be fixed on future :)


Revert the PubKey to Address in aevm_ethereum_test_chain

I see build job failure though unrelated to this PR: I am taking the liberty of restarting job.
===> Fetching yamerl ({pkg,<<"yamerl">>,<<"0.7.0">>})
===> Failed to fetch and copy dep: yamerl-0.7.0


Remove modules in resolve webpack

[ ] I have read the contribution documentation for this project.
[ ] I agree to follow the code of conduct that this project follows, as appropriate.
[ ] The changes are appropriately documented (if applicable).
[ ] The changes have sufficient test coverage (if applicable).
[ ] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
Specify modules in resolve will cause application not resolve to correct version of dependencies

Can you please provide a minimal example repository of where this is a problem?

Can you please provide a minimal example repository of where this is a problem?

Do I have to create full project with electron-forge or just create simple project with webpack and how it create the problem?

A minimal Electron Forge project would be preferable.

@malept Created at https://github.com/kamontat/electron-forge-webpack-issue-2149

@malept When is will be merged? it kind of block my release right now.


postgres_filter: fix build error on m1

Commit Message: without build on m1 return with following error:
ERROR: /private/var/tmp/_bazel_zirain/c9b2d3f5dca31997d353206ef73d3b4e/external/envoy/contrib/postgres_proxy/filters/network/source/BUILD:17:17: Compiling contrib/postgres_proxy/filters/network/source/postgres_filter.cc failed: (Exit 1): wrapped_clang_pp failed: error executing command (from target @envoy//contrib/postgres_proxy/filters/network/source:filter) external/local_config_cc/wrapped_clang_pp '-D_FORTIFY_SOURCE=1' -fstack-protector -fcolor-diagnostics -Wall -Wthread-safety -Wself-assign -fno-omit-frame-pointer -O0 -DDEBUG '-std=c++11' ... (remaining 205 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
external/envoy/contrib/postgres_proxy/filters/network/source/postgres_filter.cc:217:12: error: implicit capture of 'this' with a capture default of '=' is deprecated [-Werror,-Wdeprecated-this-capture]
  217 |       if (!read_callbacks_->connection().startSecureTransport()) {
      |            ^
external/envoy/contrib/postgres_proxy/filters/network/source/postgres_filter.cc:214:55: note: add an explicit capture of 'this' to capture '*this' by reference
  214 |   read_callbacks_->connection().addBytesSentCallback([=](uint64_t bytes) -> bool {
      |                                                       ^
      |                                                        , this
1 error generated.

Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/wait

I am not sure whether this code-change is needed by the standard or if there's something odd about the compile sequence, e.g. '-std=c++11'. I think I'll kick this over to @phlax to decide for me :)


Add max task queue activities per second

Is your feature request related to a problem? Please describe.
This is in TypeScript and needs to be in Python.

Max activities per second is also missing in Python.


Confusion about phrasing on behaviors being accessed by modules

In Thinking in T3, heading "Loose Coupling of Components":

Modules cannot access other modules directly.
Modules cannot access behaviors directly.
Behaviors cannot access other behaviors directly.
Behaviors cannot access modules directly.
[fifth point not relevant to this issue]

These all make sense to me — except point 2, which seems flat-out wrong.
Later, in Understanding Behaviors, heading "Using Behaviors", this code example is shown:
Box.Application.addModule('test-module', function(context) {

    return {

        behaviors: [ 'behavior-name' ],

        init: function() {
            //code
        }

    };

});

That's a module accessing a behavior directly. In fact, it seems to me that the main raison d'être of behaviors is for modules to directly access them.
Or does this not count as "directly", because we're accessing the module through its string name, and the framework takes care of the lookup? But this doesn't hold up, because (as far as I could find out), this only happens with modules accessing behaviors, but not with any of the other three combinations.
Furthermore, the "Loose Coupling" section continues: "Only services can be accessed directly by modules and behaviors". This access looks a bit different, but is essentially also a string lookup:
var anotherService = application.getService('another-service');

So I'm confused. It seems to me that behaviors can be accessed directly by modules just as much as services can.
Am I missing something fundamental here? What is the purpose of point 2?

That's a module accessing a behavior directly. In fact, it seems to me that the main raison d'être of behaviors is for modules to directly access them.

This is a declarative reference, not direct access. Direct access would involve retrieving the behavior object and manipulating it directly. The module has no idea what methods exist on the behavior, or really, anything about it besides a name.

Furthermore, the "Loose Coupling" section continues: "Only services can be accessed directly by modules and behaviors". This access looks a bit different, but is essentially also a string lookup:

You'll note that in this example, the service object is returned so the module can use its methods. This is direct access.


New interesting article added.

Added this very interesting article which explain the mindset and the basics for flexbox.

Thank's :)


Keyboard doesn't work on Switch ( or i don't know how to setup it because of no instructions about it )

Like in the title, there is no instructions on how to use keyboard mode, and using Keyboard.press or Keyboard.write doesn't seem to work
Controller mode works fine
( using Arduino Pro Micro )

The instruction in located in this section, but you shouldn't have missed it in the first place. Could

Or i've interpreted it wrong and this doesn't work as Keyboard on NS?

Keyboard mode won't work for Nintendo Switch at all, since Nintendo Switch is using a non-standard HID protocol rather than the one Arduino library provided. May I ask why do you need the keyboard mode on Switch in your case?

I thought it would work because standard USB keyboards works, i need it to automate some writing in FUZE4 Nintendo Switch

In this case, you might have to dive into Arduino library to find out if the HID descriptor have any effects on being recognized by Switch. As far as I can tell, there is an "endpoint index" on providing descriptors of all types of devices on the same device. Switch doesn't seems with work with that and assumes the device never send the index. You may have to customize the HID class in Arduino library to fit your needs, in which case you would like to forge a descriptor with both joystick and keyboard input bits.

Thanks for help, i will try!


Add test_data to ICounterfactualData

Description
Add test_data to ICounterfactualData so that UI can make use of the test_data returned by SDK backend for counterfactual table. This is related to the PR:- https://github.com/microsoft/responsible-ai-toolbox/pull/1885.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1892 (b707d6b) into main (d02d2cb) will decrease coverage by 6.59%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1892      +/-   ##
==========================================
- Coverage   92.44%   85.85%   -6.60%
==========================================
  Files         103       29      -74
  Lines        4830      523    -4307
==========================================
- Hits         4465      449    -4016
+ Misses        365       74     -291

Flag
Coverage Δ

unittests
85.85% <ø> (-6.60%)
:arrow_down:

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

Impacted Files
Coverage Δ

responsibleai/responsibleai/_config/__init__.py

...ponsibleai/responsibleai/rai_insights/constants.py

...onsibleai/responsibleai/serialization_utilities.py

erroranalysis/erroranalysis/report/error_report.py

...i_core_flask/environments/public_vm_environment.py

...eai/responsibleai/databalanceanalysis/constants.py

...eai/responsibleai/managers/data_balance_manager.py

...ai/responsibleai/_tools/causal/causal_constants.py

...ai_core_flask/environments/azure_nb_environment.py

erroranalysis/erroranalysis/report/__init__.py

... and 64 more

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

What exactly is test_data here? Is that the test dataset? If so, are we duplicating it? I might be missing the larger point as this is distributed over several PRs, but passing around an entire dataset (additionally to what we're already passing around it seems?) worries me. A pointer to a centralized design document that explains the big picture while referencing these individual pieces of work may also help.

What exactly is test_data here? Is that the test dataset? If so, are we duplicating it? I might be missing the larger point as this is distributed over several PRs, but passing around an entire dataset (additionally to what we're already passing around it seems?) worries me. A pointer to a centralized design document that explains the big picture while referencing these individual pieces of work may also help.

For large data scenario, not all the test samples will be present in UI. For local counterfactuals we would need to respond with the test dataset row. The size of the test data is not a concern as there is only one point returned for local counterfactuals. For initial run of rai, we limit the number of test samples to 5k. Please bear in mind that the generated counterfactuals per point determine the size of the output and not the test data itself.
We discovered this late for local counterfactual work that @vinuthakaranth is driving. The design doc for large is already shared with the PM and dev teams to refer to.


Backport ES changes

Summary of the changes / Why this improves CrateDB
See commits
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Maybe worth backporting more RecoveryTests while backporting more changes in this area? Guess it's out-of-scope of this PR but worth to consider as a follow up.

Just saw #10487 so ignore my previous comment ;-).


Moving the metagov repo to an organization

Hi @metagov, we're thinking of moving the metagov repo to an organization, and were curious if you would consider donating your Github handle (github.com/metagov) to the project. We're nice people running a nonprofit research project, and we'd really appreciate it!
Let me know if you have any questions :)
Thanks,
Josh

Just pinging @metagov again in case you're still there! Happy to pay something for the Github handle as well.


Add the file parameter to MetadataReportBuilder

What is the purpose of the change
Supplement to this pr #7747

Codecov Report

Merging #8031 (159e2f5) into master (9f6b536) will decrease coverage by 0.66%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #8031      +/-   ##
============================================
- Coverage     60.85%   60.18%   -0.67%
+ Complexity      491      489       -2
============================================
  Files          1089     1090       +1
  Lines         43876    43923      +47
  Branches       6413     6394      -19
============================================
- Hits          26699    26437     -262
- Misses        14208    14521     +313
+ Partials       2969     2965       -4

Impacted Files
Coverage Δ

...nfig/bootstrap/builders/MetadataReportBuilder.java
97.56% <100.00%> (+0.19%)
:arrow_up:

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-73.34%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-57.15%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
31.42% <0.00%> (-54.29%)
:arrow_down:

...a/org/apache/dubbo/rpc/filter/AccessLogFilter.java
28.35% <0.00%> (-50.75%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

...va/org/apache/dubbo/rpc/support/AccessLogData.java
53.16% <0.00%> (-37.98%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-28.58%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

... and 79 more

Continue to review full report at Codecov.

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


Can't get the numbers to display when numbers are pressed.

Andrew,
I am doing a online class with Peter Dalmaris and he uses this Library for one of his sections. At the time of his class he used the MediumPrec statement but when I try that in my sketch the Arduino says 'MediumPrec' was not declared. I did some research on your latest version of the library and it appears that you no longer require that statement in the constructor. I tried running the sketch without it and the sketch will compile. I get all the text to appear on the LCD screen that is in the setup but nothing appears on the screen when I press the buttons.
Keypad_I2C.zip

Another user a while back made some changes to the library. I just merged them in, please redownload the library.

I found the issue, there was an IF statement I missed in SetKeyPadVoltage.
Either remove SetKeyPadVoltage from your sketch or download my fix.

Thank you so much for your quick response. That solved my problem.


Album search no longer working

Just shows nothing after searching

An update has been released!
Fixed in https://github.com/weespin/KhinsiderDownloader/commit/70119ace0745b6fbc64eeab023ab9ede6543f49b


Incorrect latest commit message in UI when viewing tag

Description
The latest commit is incorrectly rendered when viewing a tag, the first line (commit title) is absent. It's okay when viewing a branch.
Not OK (tag view) https://try.gitea.io/silverwind/remap-css/src/tag/5.0.4:

OK (branch view) https://try.gitea.io/silverwind/remap-css/src/branch/master:

Gitea Version
1.19
Can you reproduce the bug on the Gitea demo site?
Yes

Added a patch in #21693.


CORE-15964: Revert "Add full HSQLDB support to ledger. (#4422)"

This reverts commit fb9a468429be74dde95a3031b48b6492fac10cd5.

You realise that HSQLDB is a purely test database, right?

Indeed, but some files updated through commit fb9a468 are not 100% testing related only. This PR was just a test, that's why is marked as Draft.

True, but:
props += configuration.extraProperties

does nothing if configuration.extraProperties is an empty map.

You realise that HSQLDB is a purely test database, right?

Indeed, but some files updated through commit fb9a468 are not 100% testing related only. This PR was just a test, that's why is marked as Draft.

True, but:
props += configuration.extraProperties

does nothing if configuration.extraProperties is an empty map, which it is for Postgres.

True but, as I already said yesterday in offline communications: we didn't inspect the changes in details and, instead, tried a quick revert of the single commit that touched some files present in the stack trace we were seeing in our tests.
That said, commit fb9a468429be74dde95a3031b48b6492fac10cd5 doesn't have anything to do with the issue (as expected), so I'll close the PR.


camerarecord: add frame stats collection through dumpsys gfxinfo

Signed-off-by: Michele Di Giorgio<EMAIL_ADDRESS>

Pushed new version with requested changes...


Setup of OverviewPage and CreationPage API call strategy

As per now the OverviewPage and CreationPage APIs are only called once when the app starts.
The API requests will update Redux state, but the APIs should be triggered by user activity.
We will not aim for premature optimisation, but will try to avoid making an excess amount of API calls.
Thus this issue will initially aim to update API information in Redux upon reload of page, but not on every render.
A balance should be struck here, and this issue will explore different strategies for API calls.
### Tasks
- [x] design-pattern: CreationPage will have Notify-User-of-Success and reveal hidden button Go to Overview
- [x] design-pattern2: OverviewPage will do refresh of SystemUser list even after BrowserBackButton event
- [x] consider how many users will be using this app simultanously: probably few thus not high API load

Per 15.11.23: After discussion with Simen, the BFF will probably not be exposed to a high load, as we do not expect many users. Thus optimization of API calls should not be prioritized.
Also, the final design is not yet in, so we probably have to rewrite the API calls in future iterations.

User is now informed of successful SystemUser creation, and is requested to navigate back to OverviewPage.
If instead, the user uses the Browser Back Button, an useEffect() check has been implemented to correct the state and import the updated SystemUser list from backend.


Fix AudioStreamPlayer3D volume calculation for multiple viewports

Fixes: #94403.
Related: #76713.

Went with a different solution from #76713, cause the position of the listener is important to compute the volume, so just adding MAX wouldn't work.
Not quite sure if the solution I came up with is correct: always pick the listener with the biggest db_att and if there are multiple ones with the biggest db_att choose the one with the biggest multiplier among them. The biggest db_att is preferred to account for the emission angle, if the emission angle doesn't affect anything (listeners that are being compared are both out of the emission angle, they are both in the emission angle or there's no emission angle) or it affects only a little bit it basically chooses the closest listener. When comparing multiplier it just chooses the closest listener. There might be some edge cases when this doen't work (I think db_att might have less effect than multiplier depending on the sound being played, so multiplier should be preffered over db_att, I'm unsure how to fix something like this and if it's even possible to know for sure that this particular sound will cause issues).
Fixes the comment style in AudioStreamPlayer3D::_update_panning (as this PR only changes AudioStreamPlayer3D::_update_panning)

@kus04e4ek  I also want to ask about Audio Stream Player 3D. Sorry if this is off topic, but I asked in thematic communities and didn’t find an answer.
Is it possible in the future updates to provide the ability to output audio to other playback devices for different subviewports? so that the sound setting for the main window/viewport would be done via AudioServer.set_output_device(), but the sound for the subviewport with the audio_listener_enable_3d option so that it can be assigned to the same device as the main one, or to another one. Initially I was looking for an answer to this exact question, then I noticed that the sound was not playing, and wrote about it here. and thanks for this correction.
If this possibility already exists, please let me know.
Why this is needed: It would be possible to provide the ability to output audio to different output devices for different players in split-screen mode depending on the position of their cameras.

I asked in thematic communities and didn’t find an answer.

You can also ask in the contributors chat, it's not really a place to ask general questions, but if no one gives you an an answer on the forum, you can ask there.

Is it possible in the future updates to provide the ability to output audio to other playback devices for different subviewports?

It should be possible on most platforms, currently Android, iOS and Web have no way to set the different device though.

If this possibility already exists, please let me know.

Currently Godot doesn't provide functionality to use multiple audio devices at the same time.
As it's currently not supported you should open a proposal.

4.3 stable. The bug is still present


See failures before this: https://github.com/oprypin/crystal/runs/384687812#step:15:45


Build wheel

Please, upload whl packages for opentaxii on pypi. Wheels faster, more clear and secure. Sdists require setup.py execution step that can be quite painful.

We have recently made a decision to consider this project feature-complete. It means we still maintain it, however we focus only on bug fixes. As this issue is a feature request, unfortunately we won’t add it and therefore I’m closing this issue.

Welcome to the team, Eric :) This is not a feature request, I think. We had a few issues because of setup.py execution (#157). Also, all the dependencies are installed on CI on every pipeline run, and installation of wheels is much faster. Build it once, reduce GitLab CI runners costs forever. I'd consider doing it, it's actually easy:
python3 -m pip install wheel
python3 setup.py bdist_wheel
twine upload dist/*.whl

The reason I haven't done it is that I didn't have access to the pypi package.

Thank you for your clear explanation, they might still help in the future. Our current position to stop refactoring as well is that it could create new bugs, as any new code, we would have to fix afterwards. And it is not something we want now. Feel free to fork the project if you really need those changes.


Create a prototype User Interface

[x] Use figma to create prototypes
[x] Create at least two design options
[x] Add the Figma examples into the readMe file

This issue has been completed and linked to a pull request.


[No QA] Refactor Search component to use useMemo for data calculation

Details
Thanks to @Kicu we noticed a regression coming from react-compiler upgrade, on search page there was an infinite loop which caused the whole Search page to re-render indefinitely.
https://github.com/user-attachments/assets/8177ffc5-8cec-48af-a395-c85af04c0e08
Fixed Issues
$ https://github.com/Expensify/App/issues/49393
$ https://github.com/Expensify/App/issues/51505
PROPOSAL: N/A
Tests

[x] Open Search and open console, verify there are no Maximum update depth exceeded errors

Offline tests
N/A
QA Steps
N/A
PR Author Checklist

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

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

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

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

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

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/b3127fd5-d158-4deb-8b71-cb141eef7bdf

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


Retry failed uploads - #1556 Allow users to easily re-upload failed uploads...

Description (required)
Fixes #1556 Allow users to easily re-upload failed uploads... Currently, fail and cancel buttons moved to contrib list and they works as intended.
But we don not give any reason about fail. Because I couldn't find any table of Mediawiki API error codes. Maybe someone can help me.
It gives a possible solution for badtoken error code.
Meanwhile this can be merged.
What changes did you make and why?
Tests performed (required)
Tested API 26, betaDebug
Screenshots showing what changed (optional - for UI changes)

Codecov Report

Merging #2112 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2112      +/-   ##
=========================================
- Coverage     5.6%   5.59%   -0.01%
=========================================
  Files         225     225
  Lines       11379   11396      +17
  Branches     1065    1065
=========================================
  Hits          638     638
- Misses      10689   10706      +17
  Partials       52      52

Impacted Files
Coverage Δ

...java/fr/free/nrw/commons/upload/UploadService.java
0% <ø> (ø)
:arrow_up:

...ee/nrw/commons/media/MediaDetailPagerFragment.java
0% <ø> (ø)
:arrow_up:

.../commons/contributions/ContributionViewHolder.java
0% <0%> (ø)
:arrow_up:

...ommons/contributions/ContributionsListAdapter.java
0% <0%> (ø)
:arrow_up:

...w/commons/contributions/ContributionsFragment.java
0% <0%> (ø)
:arrow_up:

...rw/commons/mwapi/ApacheHttpClientMediaWikiApi.java
3.66% <0%> (-0.01%)
:arrow_down:

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 2175ba2...2569bb4. Read the comment docs.

We can switch from toast to any other thing to display this solutions. If you have any idea pleas share

Thanks for the PR, @neslihanturan . Unfortunately I have no failed uploads and thus cannot test this. The code looks good to me however. If @maskaravivek or @ashishkumar468 can take a look at the code and approve it as well, I think we can just merge... and let our alpha users test it. ;)

@misaochan you can test it like this: Upload something, disable network during connection. It will fail and you can test it. This is how I did during development:)

String is fixed, thanks @misaochan :)

You can manage to create failed uploads with technique I mentioned above. Anyway, I consistently created and retry button worked for me.

@neslihanturan Could you please rebase on master so that @maskaravivek 's Travis fixes are included? The Travis build is failing, probably because of that.

It was just old Travis report, currently it passed. You can go ahead and merge:)

@misaochan pinging:) So that I can mark this iğssue as solved on #2171

Yes you can mark the issue as solved. ;)


Full support for m3u_plus

Please add support for all m3u_plus tags like the user-agent and DRM (inputstream) related ones.

I join the request of MaZED-UP


Admin password as environmental variable

best practice and also we don't want a default password.
Read password from environment, and if empty then don't allow admin.

Better security


1.8r Cherrypick request-cherrypicks_30740: Fix for dropped metrics in evaluate function for Keras models.

This CL should be cherrypicked since it fixes the following:

Fixes metric name to metric value dictionary creation with the right 1:1 mapping.
The evaluate function dropped metrics since metric names were not set in compile or during evaluate.

@anj-s I needed to resolve a conflict in training_eager.py after merging #18747.  PTAL, thanks!


use system settings for cert file, not hardcoded path

Some systems are using another path for SSL keys like OPENSSLDIR: "/etc/pki/tls" (CentOS). Better way will be not to use hardcoded file paths, but system settings

as i found, it is not ideal way to avoid hardcoded values.


Switch to async/await in client codegen

Since async/await has been stabilized and surf reaching 1.0, it's time to support surf alongside reqwest (both using async/await) and eventually add a feature gate for surf. This is also the best time to rethink how ApiClient and Sendable should be used throughout the codegen.

async/await for reqwest is blocked on https://github.com/rust-lang/rust/issues/57017

Fixed in b9eb2dd0cd1081f0144359d4e67183ade578ab45


License

Can you provide a license for this project?
Edit: tagging @dramforever

Sorry for the delay. I was sick for the past week.
This project is abandoned because https://github.com/maralorn/nix-output-monitor works already. But if you want to use the code, feel free to use it under the MIT license. A LICENSE file has also been added.


Do you have any prototype PCB for sale?

I am interested in buying an early prototype.

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @bilgetutak, @yusufpeh, @ardabck, @Adora233, @leetesla, @Zhujunnan, @f4try

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @japv00, @HamiltonWang, @guaigufu276151, @jovana, @Ir1d, @jaNGOB, @78bash

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @obazna, @1491928274, @magicmarshmallow, @robertherrera, @irrenhaus, @ibiinc1, @smyja

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @easyguyme, @qiyueliuhuo, @Mosonchan, @oguzhan-ulutas, @rafaeljurkfitz, @arthurb123, @Akagi201

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @justuseapen, @nirasnet, @meet-cjli, @suprnurd, @bilgetutak, @sambly, @redpanda321

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @ethanator, @bernardobarreto, @sepidre, @rodrigooler, @kiwimartin, @omar-co, @stikhun


pCMA Control - Third Party Mgmt

Control Register

TP02
Third Party   Service Providers have TPRM Third Party Assessment (3PAS) completed
a) When   procuring Third Party services involving processing Business Information or   managing Information Systems:     - A TPRM Third Party Risk Assessment is completed.     - In case of a GxP relevant system or GxP supported service, QA eCPL is part   of the assessment.     - Assessments are approved by relevant service owner, and outcome is   documented.     - If Third Party services will be used despite an unsatisfactory assessment   outcome, any gaps or risks are managed in line with the Information Risk   Management process.

TP03
Third   Party Service Provider contracts include the necessary security and   compliance clauses, and are periodically reviewed
a)   When procuring Third Party services involving processing Business Information   or managing Information Systems, Third Party contracts define applicable   information security and Data Privacy requirements, and Third Party   responsibilities.      - Information security requirements are based on the ISRM ISC   approach.     - An ISRM pre-approved contract template is used (or contract is reviewed   by ISRM).     - For services involving GxP-relevant Information Systems, QA eCPL has   pre-approved or reviewed the contract.     - For services involving the processing of Personal Information (where   Novartis is the Controller), Data Privacy has pre-approved or reviewed the   contract.     - For services involving SOX-relevant Information Systems, the contract   requires the Third Party to provide independent assurance related to “IT   General Controls” they are responsible for (e.g., SOC 1 Type II or SOC 2 Type   II report), or specifies required “IT SOX controls” and a requirement for   annual testing of these controls.         - Third party responsibilities for information and data management are   contractually defined.     - The contract is signed prior to the Information System going into   production.     b) Contracts are periodically reviewed.     - Review period is annual (or risk-based and defined, review period can be   extended to 5 years for “long term contracts”).

Moved to MVG. Closing this ticket here.


Austin, TX no longer updating

Austin, and many other stations are no longer updating.

Yes, the UK has "dropped off the map" too e.g. Chilton, United Kingdom

Hi @techfixpros you can go on the official page kc2g.com,
search the station and verify yourself the TIME  column, Austin last update was 2024-04-17T19:05:05.
Unfortunately not all station are always active.
Thanx for your report
73 de IZ0IJD Emiliano

Thank you for the info. I did see they hadn't updated on the site in some time.


Op stop subcluster

Add planned subcluster shutdown by setting subcluster shutdown to true.

Also if a subcluster is down (shutdown true) and for some reason some pods get abruptly deleted and we lose quorum, normally the operator will start_db, but in this case, it cannot because one subcluster has shutdown true. We are kind of stuck and the only think we can do is raising a warning. We cannot override the user input, and assuming they shut down the instances, it would be pointless. This stop_subcluster feature can be very dangerous that is why we need to limit its use.


e10s support

Does this plugin supports e10s ? https://arewee10syet.com/ marks it as unknown

It does.
I'll add the multiprocess permission to the package.json file to flag it as compatible

@esphen any chance to get a release containing the flag ?

Ping @janodvarko

Done
New release 0.6.5 available (waiting for AMO review)
Honza

0.6.5 has passed AMO review

thanks. I'm now enjoying the e10s mode.


Enhanced usage of MPI_abort

This should fix https://github.com/gridap/GridapPETSc.jl/issues/71
cc @amartinhuertas

Codecov Report

Merging #70 (fd8e22f) into master (8a169ba) will decrease coverage by 0.07%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   91.57%   91.50%   -0.08%
==========================================
  Files           7        7
  Lines        2268     2271       +3
==========================================
+ Hits         2077     2078       +1
- Misses        191      193       +2

Impacted Files
Coverage Δ

src/MPIBackend.jl
92.47% <40.00%> (-0.97%)
:arrow_down:

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 8a169ba...fd8e22f. Read the comment docs.

@amartinhuertas can you please review this?

All good, except for the following I am not sure. In particular, I think we should update the following line
https://github.com/fverdugo/PartitionedArrays.jl/blob/master/test/mpi/mpiexec.jl#L13
and test for the code error returned by the program, right? (i..e, is it 0 or not?)

The fundamental change in this PR is to call MPI_Abort with a non-zero error code.
Calling it with a 0 error code (as before) was causing the problems related with https://github.com/gridap/GridapPETSc.jl/issues/71
The run command  was thinking that  mpiexec was returning smoothly, even when calling MPI_Abort (since we returned error code 0).
In contrast, if we call MPI_Abort with a non-zero code, run detects that the launched process failed and raises an exception. As a result the computation is stopped, line  https://github.com/fverdugo/PartitionedArrays.jl/blob/master/test/mpi/mpiexec.jl#L13 is not reached, and the CI environment receives a non-zero error code (caused by the exception lauched by run) which signals that the tests have failed (as we want).
I think that the PR is fine, just wanted to double check.
It makes sense now for you?

The fundamental change in this PR is to call MPI_Abort with a non-zero error code.

Yes, that I understood.

In contrast, if we call MPI_Abort with a non-zero code, run detects that the launched process failed and raises an exception. As a result the computation is stopped, line https://github.com/fverdugo/PartitionedArrays.jl/blob/master/test/mpi/mpiexec.jl#L13 is not reached, and the CI environment receives a non-zero error code (caused by the exception lauched by run) which signals that the tests have failed (as we want).

Ok. This is what I was missing from the discussion.  Still I find confusing test true. I would do test error_code==0, even if we know that error code is going to be always equal to zero (otherwise the execution is aborted).

I would do test error_code==0, even if we know that error code is going to be always equal to zero (otherwise the execution is aborted).

I see your point, but unfortunately, finding the error code would complicate the code.
I would keep it like this for simplicity. In fact, I don't find it so confusing. It is quite aparent that @test true will be reached only if run succeeds, right?

I would keep it like this for simplicity. In fact, I don't find it so confusing. It is quite aparent that @test true will be reached only if run succeeds, right?

Ok. Put at comment then. I dont think is quite aparent.

Ok. Put at comment then.
Done


Can you post the list of games supported right now?

Just to be clear: im not the dev if this page but rather a watcher who is very interested in the progress of this program.
If you mean Borderlands series yes (unsure of BL1 i'll test asap today) (hud would act like its at 4:3 ratio and wont stretch making it hard to read hud items. However multiple displays work flawlessly)
RE5 (though natively supported)
L4D2 (though doesn't seem to allow p2 to do anything hmm..)

"
PC Offline Co-Op Tools
This tool aims to support games to be offline multiplayer playable on PCs.
Supported Games

Borderlands 2
Borderlands: The Pre Sequel
Left 4 Dead*
Left 4 Dead 2*
Resident Evil 5
Portal 2¹

Plays from 2 to 4 players on the same PC. Needs at least 1 gamepad. 1 player can play using keyboard and mouse.
¹ Can be played with only 1 gamepad

"

portal 2 also has native support like re5

The updated list is in the code tab.


Syncrepl ldaps support

Description
This change allows the syncrepl configuration to be changed via attributes. In particular it allows
replication via ldaps.
Issues Resolved
There are no open issues for this.
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

The greater context of this is that the current set of syncrepl attributes are inherently inadequate for dealing with both a leader and a follower as the cookbook makes a ton of assumptions as to what values should be based on other values.
This is to say, while this does "fix" a problem it's really just moving around the brokenness and as discussed in Slack to address this properly the syncrepl support needs to be refactored accordingly along with a major version release.

@cheeseplus my pr was an attempt to allow, at least on the consumer side, the ability to change the syncrepl attributes. It would be trivial to do the same for the provider attributes. I don't agree that this is just 'moving around the brokenness', anyone with a basic knowledge of Chef and who can read the OpenLDAP documentation can configure it exactly as they like.

@cheeseplus I just pushed a change with a syncrepl provider config hash


Add 3:2 follow the UN common usage in SVG

For the fair treatment of Member States, the United Nations, irrespective of how nations stipulate the aspect ratio of its national flag, must, on the occasion of the United Nations or other international forums, use 2: 3 national flags in all its member states in order to reach an agreement on equality.

no, thank u.


Spelling: LePlan shouldn't it be LePlain?

Plan, ie project management plan
Plain, ie plain is a flat, sweeping landmass
Plane, ie airplane

Hi @gwald.
I think you're right.
I still need to make a good pass on the project objects nomenclature, before it's too late (when it's actually being used by other developers for their own projects). Plan is the French word actually.
I think LePlane is the right one:
https://en.wikipedia.org/wiki/Plane_(geometry)
Please tell me if you find some other names / terms that sound weird to you.
All the best.

Corrected and tested.
Thank you @gwald / Mike.


Refactor adaptive auth batch script

This PR contains

Addition of a function to delete a jar from a given directory.
Uses the function in all approriate places to eliminate duplicate code.
Fixes some indentation issues in the description.

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5128630783

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/5128630783
Status: success


Release Sonos Edge Driver

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 8 committers have signed the CLA.:white_check_mark: sbkimaqara:white_check_mark: sky-nie:white_check_mark: dljsjr:white_check_mark: ctowns:white_check_mark: greens:white_check_mark: maxelweb-vimar:white_check_mark: cjswedes:x: lelandblueYou have signed the CLA already but the status is still pending? Let us recheck it.


[feat] padding, testing, and benchmarking

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Reverts Backtrace: format to what it used to be (IDFGH-7580)

Was:  Backtrace: N:M N:M N:M ...
Now:  Backtrace:N:MN:M N:M ...
The problem with the new format is that it is hard to parse and
breaks the parser that is used by PlatformIO.  The old format
is much more reasonable.  I do not see how the pattern in IDFDUT.py
can work with the new format, due to the missing space after the :
Possible fix for #8722

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

Thanks for your contribution.

Hello @MitchBradley,
The backtrace parsing issue you mentioned is going to be fixed using the proposed changes.
I will keep you informed of the state of the PR.

Hello @MitchBradley,
The changes were merged internally and will be available on GitHub in the next commit on the master branch.
This could take from  a couple of days to a couple of weeks.
Thanks again for your contribution.


Add detach option in order to have mobile and VR friendly previews

I'm currently using JSBin to give webVR workshop. It allows to get instant feedback on ThreeJS project. The key advantage so far is that it allows to detach the output and thus load it on a mobile device then switch to VR mode.

So this relates to #36 ?

Please close if it is the same :)

Yes it's the same, only I wrote it earlier :p


Fix malformed summary of the quick sync case

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

🤣
/approve

Thank you. ww
/changelog

/changelog

/changelog


Add Ability to Set client_max_size for Webhook Server

Work has been undertaken in ACA-Py to explore supporting credential issuance for large creds 1 < 16Mb.
The current default for an aiohttp web.Application() is to support 1Mb max client request size. ACA-Py have added this additional config arg to change this on the agent https://github.com/hyperledger/aries-cloudagent-python/blob/68347f8e1f3b918464019bdc2d891926de023f5c/aries_cloudagent/config/argparse.py#L164
For our webhook server to receive and process events from an agent processing messages of this size, we would need to support somewhere in the order of 4x the credential size. This data gets represented multiple times in the webhook payload by the time the protocol gets into the credential_acked state.
For now we could just add an optional argument to specify the client_max_size when initialising the webhook?

Not relevant anymore


GodRay not using worldposition when updating

Hi! Really cool library!
I noticed something when I started animating my "sun" to revolve around the center of the scene.
The mesh moved but the effect stayed where it is.
Turns out it's because of my "shortcut" of rotating the sun object. I have a THREE.Group at the center (0, 0, 0) and the sun is a children of that at (10, 0, 0). And I'm just rotating the parent group to move the sun.
But the god-ray effect always reads that (10, 0, 0) (The relative position) instead of the world position.
Right now I solved this by switching my sun object to a pseudoSun (just another group), and copy the world-coordinates of that into my real sun's position when updating.
Is there a reason for the effect to use the position directly instead of WorldCoordinates?

Hi,
thanks for the issue report!

Is there a reason for the effect to use the position directly instead of WorldCoordinates?

No, there is no reason. The GodRaysEffect was originally inspired by the god rays example from three.js which didn't account for this either. Until now there hasn't been a use case that reveiled this issue.
A fix is on the way.

Should be fixed in<EMAIL_ADDRESS>Didn't test this thoroughly so let me know if you find a bug!

Thanks! I just noticed that the package got updated. And, yes it works as it should. Thanks for the quick fix!


Include alphaFold homology models through ProtVista viewer

The Protvista viewer in the Uniprot site shows the structures of the AlphaFold structural models.

Can we please...

[ ] Check with @xwatkins if it's as trivial as updating to the version 3.3.0 of Protista and whether is safe to do it as it's not tagged as a release. The relevant changes look associated with 3.3.0.

[ ] If confirmed, update to Protvista 3.3.0

@HelenaCornu keep an eye on this feature

Tagged this issue with 21.09 to remind the team to update the ProtVista viewer before the bundle is generated for the 21.09 release

Following up on @andrewhercules comment, we need to check the console for errors before accepting a final release. The current version throws some errors on trying to access various resources. I suspect the consequence is missing data on some of the protvista tracks.

If the errors are 404 errors to the proteins API you can safely ignore them. We do not know ahead of the call whether there are features of a specific type for an entry, and the API returns a 404 not found if there are none. We are working towards a better encapsulation of these errors.

@carcruz can you please update the protvista-uniprot NPM package package to version 2.7.5 and test to ensure that it works?
Targets to test:

NOD2 - ENSG00000167207
IL4 - ENSG00000113520
TSLP - ENSG00000145777

Ticket closed as work completed and merged into main


APPS-2549 Bulk describe optimization in scatter-collect to cover files in arrays, structs

Would depend on dxScala change
https://github.com/dnanexus/dxScala/pull/129/files

Looks good! In the ticket please:

Run a WF prior to changes, and provide a splunk stats showing file/describe from dxScala agent. Should be some.
Run after changes applied showing splunk stats for the same WF (new analysis) and decreased/eliminated dxScala-made file/describe.

done -- APPS-2549


can we derive a depth map rendering results with fast speed with jnerf?

can we derive a depth map rendering results with fast speed with jnerf?

same question. How to compute the depth map?


Update version to 1.2.50-1 on main

Update version to 1.2.50-1

We actually have a large decrease in TTI time due to us now mocking the API ⚡ Andrew will work on the tests to fix the noisy comments.


Reduce size of gltfio for Android.

The net effect of these 3 changes on arm64 reduces the uncompressed libgltfio so from 1.5 MB to 850 KB.  Over 55% of the newly optimized binary is its rodata section (i.e. materials).
As a bonus, this also reduces libfilament so by 8 KB because it fixes a typo in our CMake file wrt --gc-sections.
Later today, we would like to create a "gltfio-lite" which would only support lit opaque materials.  This would have a size of ~470 KB .

Are there variants we can strip out of the materials? matc lets you do that.


Issue: StructuredChatOutputParser regex may prevent agent's actions

System Info
Python == 3.11.3
Langchain == 0.0.330
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[X] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[X] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Python 3.11.3 (tags/v3.11.3:f3909b8, Apr  4 2023, 23:49:59) [MSC v.1934 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import langchain
>>> langchain.__version__
'0.0.330'
>>> from langchain.agents import load_tools, initialize_agent, AgentType
>>> from langchain.chains.conversation.memory import ConversationBufferMemory
>>> from langchain.chat_models import ChatOpenAI
>>> agent = initialize_agent(
...     agent=AgentType.STRUCTURED_CHAT_ZERO_SHOT_REACT_DESCRIPTION,
...     llm=ChatOpenAI(model='gpt-3.5-turbo', temperature=0),
...     memory=ConversationBufferMemory(),
...     verbose=True,
...     handle_parsing_errors=True,
...     tools=load_tools(tool_names=["requests_get"]),
... )
>>> agent.run('Search the latest version of python')

> Entering new AgentExecutor chain...
Thought: I can use the requests_get tool to search for the latest version of Python on the official Python website.

Action:
```{"action": "requests_get", "action_input": {"url": "https://www.python.org/downloads/"}}```

> Finished chain.
'Thought: I can use the requests_get tool to search for the latest version of Python on the official Python website.\n\nAction:\n```{"action": "requests_get", "action_input": {"url": "https://www.python.org/downloads/"}}```'

Expected behavior
I expect agent to execute the action, requests_get in the above case.
NOTE: In the above case, InvalidRequestError would be raised after executing requests_get because the number of tokens is very large.
Related: #12158

I have found that regex pattern in StructuredChatOutputParser does not match the following case:
{"action": "hogehoge", "action_inputs": {...}}
This may cause the prevention of agent's actions.
I think that StructuredChatOutputParser.pattern should be
pattern = re.compile(r"```(?:json(\r?\n))?(\{|\[)(.*?)```", re.DOTALL)

What do you think?


Update synthetic tests

what

Update synthetic tests
Add new Datadog monitors
Fix k8s monitors

why

Implement all features and attributes described in https://registry.terraform.io/providers/DataDog/datadog/latest/docs/resources/synthetics_test
New Datadog monitors to monitor RabbitMQ

related

Closes #42

/test all

/test all


Clear setattr if it was inherited && written by us

Fixes #661
Since the frozen behavior was deliberate, the changes should only affect new on_setattr code.

Looks like codecov is broken. :(

👯 nobody cares 👯


Modal position is bugged until resize (on Firefox!)

Greetings! First things first, great work on the plugin! Very useful.
So, I've been using it to display some PDF's on my company's website. On Google Chrome, it works flawlessly:

But, unfortunately, I can't say the same for Firefox - The position is way off, and the code is exactly the same!

I've searched around for a bit in the issues tab but got no good solution. I've tried using ConstructOnInit: true, or setting the width and height with onCreated and onOpen, to no success.
Here is the code that creates the Jbox:
    function createStampModal(uri) {
        var width = $(window).width() - $(window).width() * 0.1;
        var height = $(window).height() - $(window).height() * 0.1;
        var content = '<embed src="{0}" width="' + width + '" height="' + height + '"></embed>';
        content = content.replace('{0}', createDocumentUrl(uri));

        new jBox('Modal', {
            attach: '#buttonViewStamp',
            content: content
        });
    }

Thanks for the help! Hopefully we can find a fix together.

Hi @Momentanius,
what version are you using?
I can't reproduce the issue. Tested in Chrome, Firefox on Mac and Windows. See this fiddle: https://jsfiddle.net/StephanWagner/72L0rm5e/
Also, I just released version v0.5.5 which fixes a bug that positioned the jBox above the close button in the overlay. Could you try with the newest version? If the issue persists, can you send me a link or a fiddle where I can reproduce the issue?
Cheers,
Stephan


Completions broken in rust buffers

rust-analyzer not completing for some reason. Other functionality works.

Not actively writing rust at the moment, closing this.


atari breakout environment

I started messing around with this on the breakout branch.

closing for now because it's silly


Ensure Consistent Use of MeshKit Errors in Meshery-Linkerd.

Current Behavior

Currently, there are instances where errors are being used directly instead of MeshKit errors, which impacts the consistency and quality of error reporting across the project.
Desired Behavior

[ ] Replace all direct usages of errors with MeshKit errors.
[ ] Ensure that each MeshKit error includes:- Short Description, Long Description, Probable Cause & Suggested Remediation.

References
Meshery Documentation on Meshkit

Make sure to follow these guidelines during writing Meshkit errors
https://discuss.layer5.io/t/best-practices-for-writing-meshkit-errors/5536

Also here is an example PR to which you can refer:
#11340

Contributor Guides and Resources

📝 Meshery Adapters Spreadsheet
🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

hi i would like to work on this issue

hi i would like to work on this issue

Thanks for hopping in @moazfarrukh, here you go.

@singh1203 yes i have started working on it. I have built the project i have been testing each error  by trying to trigger to them to properly understand their behavior before refactoring them, which is taking me a while because there are more than 200 instances of errors in the whole project. i will try to make major progress over this weekend.

@singh1203 yes i have started working on it. I have built the project i have been testing each error by trying to trigger to them to properly understand their behavior before refactoring them, which is taking me a while because there are more than 200 instances of errors in the whole project. i will try to make major progress over this weekend.

That's excellent! However, I recommend creating a draft PR to represent the progress in the upcoming meshery dev meeting.


Update README URLs based on HTTP redirects

https rules!
Created with https://github.com/dkhamsing/frankenstein
HTTPS Corrected URLs

Was
Now

http://crystal-lang.org/api
https://crystal-lang.org/api

http://crystal-lang.org/docs
https://crystal-lang.org/docs

http://crystal-lang.org/docs/installation/index.html
https://crystal-lang.org/docs/installation/index.html

http://play.crystal-lang.org/
https://play.crystal-lang.org/

Other Corrected URLs

Was
Now

http://www.manas.com.ar
http://manas.com.ar/

Looks good, thanks!


[BUG] Error while picking image, Another permission request is still ongoing.

Describe the bug (描述)
Exception while picking an Asset. Stack trace
E/flutter (13533): [ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: PlatformException(PERMISSION_REQUESTING, Another permission request is still ongoing. Please request after the existing one is done., null, null)
E/flutter (13533): #0      StandardMethodCodec.decodeEnvelope
package:flutter/…/services/message_codecs.dart:597
E/flutter (13533): #1      MethodChannel._invokeMethod
package:flutter/…/services/platform_channel.dart:156
E/flutter (13533): <asynchronous suspension>
E/flutter (13533): #2      Plugin.getAllGalleryList
package:photo_manager/src/plugin.dart:26
E/flutter (13533): <asynchronous suspension>
E/flutter (13533): #3      DefaultAssetPickerProvider.getAssetPathList
package:wechat_assets_picker/…/provider/asset_picker_provider.dart:302
E/flutter (13533): <asynchronous suspension>
E/flutter (13533): #4      new DefaultAssetPickerProvider.<anonymous closure>
package:wechat_assets_picker/…/provider/asset_picker_provider.dart:260
E/flutter (13533): <asynchronous suspension>

How to reproduce (如何复现)

Run the example project.
Click Image picker (Only pick image from device)
Permission dialog pops up
Allow permission
Crash happens

请提供一个最小化的demo用于复现，而不要提供整个项目或者不完整的页面，否则我们有权拒绝你的请求。
Steps to reproduce the behavior (描述复现步骤):

Run the example project.
Click Image picker (Only pick image from device)
Permission dialog pops up
Allow permission
Crash happens

Expected behavior (期望行为)
A crash should not happen
Screenshots (If contains) (是否有截图可以提供)
Device information (设备信息)

Device: OnePlus 7 PRO
OS: Android 11
Package Version:
Flutter Version: Flutter (Channel master, 2.4.0-5.0.pre.85)

Additional context (附加信息)

See https://github.com/CaiJingLong/flutter_photo_manager/issues/563 .


View in-progress certification run

Once a run is started through the admin UI, we'll want to be able to leave it, and go back to check on progress.
Should include a measure of how many succeeded and failed. Also, if things fail, we should be able to drill into the details to find out why.

Might be worth using https://pusher.com/ to report back to the frontend regularly maybe?

What do we actually mean by 'success' ? Is it that the data was grabbed successfully and didn't fall over? Or do we mean that there weren't enough responses to give it a certificate level?

I've begun writing a report page but I'm realising this is mostly covered at the moment. The campaign show page displays how many are in progress and shows if responses are missing etc.

I've had another look at the campaign show page. Although it shows what's  worked / failed etc, it doesn't have the original value of how many datasets it added to the queue. So most of what's described here is currently covered but the page could be improved quite a bit with a queue count.

Todo: add a campaign dataset count.


Question about loss function when training?

I am training an image classfication model and ues two loss functions, just like nn.CrossEntropyLoss() and metrics loss (CircleLoss). How to initialize these two criterions and train?
criterionA = nn.CrossEntropyLoss()
criterionB = CircleLoss()
engine, train_dataloader, test_dataloader, _ = colossalai.initialize(model,
optimizer,
criterion,
train_dataloader,
test_dataloader)

Hello, we recommend using the Booster API, and the usage of 'initialize' will be deprecated. https://colossalai.org/docs/basics/booster_api， The definition of the 'criterion' function can be referenced in this example.https://github.com/hpcaitech/ColossalAI/blob/main/examples/images/vit/vit_train_demo.py

邮件已收到，辛苦

thanks


Adding separate hook to check if user is authenticated.

Hey there! Love this library, it's really neat, huge props! 🎉
This PR attempts to decouple the information from the useOidcUser() hook. I am aware that I can use the withOidcSecure() hook to secure a component to check if the user is authenticated or not and redirect to the login page. However, the latter is not useful when I'm dealing with multiple libraries for different providers (e.g., I use the msal-react library for Azure sign-in alongside this one for other on-prem identity providers like IdentityServer or CAS Apereo).
I'm decoupling because useOidcUser() causes unnecessary re-renders when I only want to know if the user is authenticated or not. This greatly  impacts the performance of my application currently, especially if it's a root component.
A picture tells a thousand words
Current hook
The following picture shows me rendering a new component on the demo project and showing the amount of times it renders when I'm calling:
    const isAuthenticated = useOidcUser().isLogged;     // -> the old one
    console.log("render")

    return (

{isAuthenticated ? "User is authenticated" : "User is not authenticated"}

    )

New hook
With the new hook I added - useUserIsAuthenticated() - it only renders once 😄 .
    const isAuthenticated = useUserIsAuthenticated();    // -> the new one
    console.log("render")

    return (

{isAuthenticated ? "User is authenticated" : "User is not authenticated"}

    )

This logic of decoupling hooks can further be extended to the rest of the components of useOidcUser(), even though I guess this one is more alarming, in my opinion. This is useful so I don't have to use libraries like react-hooks-in-callback to control re-renders.

hi @LuchoTurtle thank you very much for the PullRequest!
May be all isLogged should be renamed as isAuthenticated. It is a breaking change but I really prefer your namming 'isAuthenticated"
I do think, you missed the "useOidc" existing hook, that will not cause multiple rerendering (see code bellow). May be adding a new hook is not necessary. What do you think ?
export const useOidc =(configurationName=defaultConfigurationName) =>{
    const getOidc =  Oidc.get;

    const login = (callbackPath=undefined) => {
        return getOidc(configurationName).loginAsync(callbackPath);
    };
    const logout = () => {
        return getOidc(configurationName).logoutAsync();
    };

    let isLogged = false;
    const oidc = getOidc(configurationName);
    if(oidc){
        isLogged = getOidc(configurationName).tokens != null;
    }

    return {login, logout, isLogged };
}

Hey @guillaume-chervet!
Thanks for the fast reply, you're amazing pal!

May be adding a new hook is not necessary. What do you think ?
You're totally right. I quite forgot about that hook and that I could use the property. It runs smoothly, no problems there, thank you!

and may be useOidcUser hook sould return another kind of data for not confusing people. like
{user, userLoadingStatus} = useOidcUser()
and status be like :
const loadingStatus:{
"notAuthenticated",
"loading",
"loaded"
}

I agree, no reason on repeating the same information when one is clearly more performant than the other. With this in mind, I committed the changes you request and also optimized the useOidcUser() hook, saving one unnecessary render along the way 😄 .

May be all isLogged should be renamed as isAuthenticated. It is a breaking change but I really prefer your namming 'isAuthenticated"
I'm with you on this one! It's how MSAL uses it and I wager it's more intuitive for the folk around. Also added this breaking change to the commit so if you happen to merge it, you oughta 5.0 the library!

Thank you very much @LuchoTurtle , it will be breaking. It will generate a new 5.0.0 version. I will make a migration guide in another PullRequest.
I prefer make a breaking now before developers switch from v3 to v4.


wrong attribute data="tuple" in the XML produced by the "decompile" goal

Now, the decompile goal produces XML that contains data="tuple" attribute, which is not acceptable in XMIR. Instead, we should say star="".

@maxonfjvipon please, provide more details, if necessary

@volodya-lombrozo this one is urgent, since it blocks the benchmark

@volodya-lombrozo if you check XeEoListener you may see that we add star attribute if EO object is *
EO:
* 1 2

After parsing:
<o base="tuple" star="">
  <o base="int" data="bytes">...01</o>
  <o base="int" data="bytes">...02</o>
</o>

Then on optimize step such will be converted to
<o base="org.eolang.tuple">
  <o base="org.eolang.tuple">
    <o base=".empty">
      <o base="org.eolang.tuple">
    <o base="int" data="bytes">...</o>
  <o base="int" data="bytes">...02</o>
</o>

Which is equivalent to the next EO:

@yegor256 PhiMojo*

@yegor256  @maxonfjvipon Do you have an xml which I can just insert into plugin section of pom.xml? Or maybe an example of PhiMojo` usage?

@volodya-lombrozo you may check this test and these packs for the test.
Here we parse EO code, optimize, converts to phi and save

@volodya-lombrozo sorry, I didn't get you question correctly.
You meant something like this?
<plugin>
  <groupId>org.eolang</groupId>
  <artifactId>eo-maven-plugin</artifactId>
  <version>${project.version}</version>
  <executions>
    <execution>
      <id>to-phi</id>
      <goals>
        <goal>xmir-to-phi</goal>
      </goals>
      <configuration>
        <phiInputDir>${project.build.directory}/eo/2-optimize</phiInputDir> <!-- take XMIR from here -->
        <phiOutputDir>${project.build.directory}/eo/phi</phiOutputDir> <!-- Result .phi files here -->
      </configuration>
    </execution>
  </executions>
</plugin>

@maxonfjvipon Yes, I've already derived it myself

@yegor256 @maxonfjvipon I've tried to run the following goal on the one of my integration tests:
<plugin>
  <groupId>org.eolang</groupId>
  <artifactId>eo-maven-plugin</artifactId>
  <version>0.34.4</version>
  <executions>
    <execution>
      <id>convert-xmir-to-phi</id>
      <phase>process-classes</phase>
      <goals>
        <goal>xmir-to-phi</goal>
      </goals>
      <configuration>
        <phiInputDir>${project.build.directory}/generated-sources/xmir</phiInputDir>
        <phiOutputDir>${project.build.directory}/generated-sources/phi</phiOutputDir>
      </configuration>
    </execution>
  </executions>
</plugin>

And got exception:
[ERROR] net.sf.saxon.style.XSLMessage@45e2d71f: Attribute @line is absent at "ALOAD-24"

I don't have the information about lines yet (and it would be strange if I should). Most probably it will be much easier to disable this check from your side.

@volodya-lombrozo just put line="999", this should be enough. Completely disabling this check would be unreasonable, since XMIR is expected to always be the result of EO parsing (that's why the lines).

@maxonfjvipon I believe, both PrintMojo and PhiMojo will work well as a test, since they both use the same XMIR validation mechanism, through XSD. Am I wrong?

@yegor256 yes, PhiMojo checks validates XMIR through XSD

@yegor256 @maxonfjvipon Could you update jee-maven-plugin ( jeo, not opeo!) up to 0.2.17 version, please? It should help.

@volodya-lombrozo this error disappeared after the update of JEO to 0.2.17, thanks!


Restrict pannable area of VictoryChart

Bugs and Questions
Checklist

[ yes] This is a victory-native specific issue. (Issues that also appear in victory should be opened here)

[ yes] I have read through the FAQ and Guides before asking a question

[yes ] I am using the latest version of victory-native

[yes ] I have checked to make sure that my versions of react-native and react-native-svg are compatible with this version of victory-native. Read about version requirements

[yes ] I've searched open issues to make sure I'm not opening a duplicate issue

The Problem
I have various charts in react-native-swiper. Two pie charts and a bar chart for now. Everything worked fine until I added the bar chart with VictoryChart and VictoryZoomContainer, now the bar chart is working, but the pannable area of the of the bar chart extends all the way to the other pie charts (yes, I could pan on the other charts and the bar chart would get scrolled), so basically my other charts touch events aren't firing due to this. Is there anyway I could solve this? Perhaps by reducing the pannable area?
P.s. Thank you for all your support, appreciate it.

Okay so I added a custom svg component around my victory chart, and made the victory chart standalone as false, Now none of my touch interactions (panning) work.

Using standalone={false} will prevent victory from rendering any of the evented containers like VictoryZoomContainer that you rely on for things like panning. Instead it will render a plain group component.
For your original problem, you might try wrapping your whole chart in a View with overflow="hidden" and pointerEvents="box-none"

I can't thank you enough, it worked. Thank you!


Adds system client interface and tests for builder

@unleashed small change here - ignore the irrelevant branch name :)

Codecov Report

Merging #124 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #124   +/-   ##
=======================================
  Coverage   87.46%   87.46%
=======================================
  Files           9        9
  Lines         726      726
=======================================
  Hits          635      635
  Misses         67       67
  Partials       24       24

Impacted Files
Coverage Δ

pkg/threescale/threescale.go
84.84% <100%> (ø)
:arrow_up:

pkg/threescale/builder.go
85.71% <100%> (ø)
:arrow_up:

pkg/threescale/proxy_conf.go
92.42% <100%> (ø)
: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 7e3e0e1...fdb3949. Read the comment docs.


Make the timeline "zoomable"

Currently the UI doesn't do too well with handling multiple smaller cuts in a row. Especially on video files with long lengths, this can become a problem. To allow for precise graphical cutting, it would be nice to allow users to "zoom into" the timeline. Typically this would mean looking at only a portion of the timeline, in which the scrubber can then be moved with higher precision than normal.

Pinning this as it is quite far down the issue list, but also arguably the most requested feature we have. Work on this if you want to make a lot of people happy :)

from my experience I think that this is a must have feature for the foreseeable future - especially if you cut long videos :)

I totally agree.

from my experience I think that this is a must have feature for the foreseeable future - especially if you cut long videos :)

Yes. Zoom and maybe direct editing of time codes.
Editing of longer recordings is hard without.


DataType Page: Basic View Page

Basic Summary (name / description / parents/ children, app count, flow count),
People (related to the datatype ,not the apps)
Logical Flows (Boingy Graph + Table / not summary)
Bookmarks
Entity Stats / Indicators
Costs (?)
Raw Applications ( ? probably not ? )

Authoritative sources is a topic for another issue

See #390


Added specify token provider option to methods.

#964
Added specify token provider option to

GeneratePasswordResetTokenAsync
ResetPasswordAsync

GenerateEmailConfirmationTokenAsync
ConfirmEmailAsync

GenerateChangeEmailTokenAsync
ChangeEmailAsync

Hi @akanmuratcimen, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@akanmuratcimen, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

If you are managing the token provider instances instead of the user manager, you can just directly call into the provider with the appropriate purpose/user, you don't need to go through the user manager in this case.


upcoming: [M3-7902] - Add PG region limit help text

Description 📝
Simple UI change in the PlacementGroupsCreateDrawer to inform the user of the Placement Group limits based on selected region.
Changes  🔄
List any change relevant to the reviewer.

Conditional logic to render the Placement Group limits.

Target release date 🗓
04/15/2024
Preview 📷

Before
After

How to test 🧪
Prerequisites
(How to setup test environment)

Log in with a user account which has the placement-group customer tag.
Using the Cloud Manager developer tools, switch to the dev environment.

Verification steps
(How to verify changes)

Verify that the text copy is rendered upon selecting a region from the dropdown in the PlacementGroupsCreateDrawer component.
Close the drawer and verify the text is not rendered until a region is selected.

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

@carrillo-erik you still need to resolve conflicts however


I've made a change

/azp run

/azp run

Closing this now that I've confirmed that feature branches work as expected


Bug Make UDF's type always nullable

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.17 seconds
load time: 450 seconds
storage size:<PHONE_NUMBER>5 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221107042652_clickbench_pr_40798.html


some cleanup for bevy_ptr

change PtrMut::as_ptr(self) and OwnedPtr::as_ptr(self) to take &self, otherwise printing the pointer will prevent doing anything else afterwards
make all as_ptr methods safe. There's nothing unsafe about obtaining a pointer, these kinds of methods are safe in std as well str::as_ptr, Rc::as_ptr
rename offset/add to byte_offset/byte_add. The unprefixed methods in std add in increments of std::mem::size_of::<T>, not in bytes. There's a PR for rust to add these byte_ methods https://github.com/rust-lang/rust/pull/95643 and at the call site it makes it much more clear that you need to do .byte_add(i * layout_size) instead of .add(i)

Ping @cart for an A-Pointer tag.

bors r+


Massive tps lag caused by Apoli?

It seems Apoli is causing alot of tps issue on our server.
dont think a specific origin is causing because it still happens when everyone is set to human.
Minecraft 1.20.1
Forge 47.3.0
Origins 1.20.1-<IP_ADDRESS>
here is a spark report showing it https://spark.lucko.me/TLEsHF6k1j
We do have several addons and im trying to get the server owner to remove them to figure out if its a specific one causing it but hes hesitant....

This one is caused by a specific addon or datapack - Something is irresponsibly executing mutliple entity actions in an action_over_time power.
It's possible for this to happen even if everyone is set to human, as it could be a global power.

This one is caused by a specific addon or datapack - Something is irresponsibly executing mutliple entity actions in an action_over_time power.
It's possible for this to happen even if everyone is set to human, as it could be a global power.

I seem to be having the same issue, what is a global power exactly and where would I search for it in my datapack?

Ive removed all addons mods(we do not have any datapacks) and the issue still occurs.

where would I search for it in my datapack?

Datapacks are in the world folder, you likely dont have any unless you added them yourself

Ive removed all addons mods(we do not have any datapacks) and the issue still occurs.

Can you send the latest Spark Profile after doing this, then? The one you shared clearly shows 60% of the server thread being used by an action_over_time power executing multiple actions, with the majority of lag caused by a command condition due to target selectors.
Origins doesn’t have anything that does this built in.


Added FCM support for Android

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


Update 2015-04-15-mindshift-guide-to-digital-games-and-learning.md

Fixed some types in authors.

Closing because I merged this patch branch with another of my branch patches.


add support for Logs in AWS::Serverless::Function to control log group

Issue #, if available:
#257
Description of changes:
Adds support for LogGroup definitions for AWS::Serverless::Function
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Awesome @tylersouthwick will look at it in detail later.

Have this feature really released?
https://github.com/awslabs/serverless-application-model/releases

LGTM, but i'm not a SAM expert or a Expert Python developer (yet)
@Tanoemon it's not merged yet, if it's in develop it will be released in the next version if it's in master it's released and you can use it.

@Nr18 Thank you for your reply.
Is "CloudWatch Logs Trigger" feature in this release https://github.com/awslabs/serverless-application-model/releases/tag/1.5 is different from this issue?

Yes it's a different issue the one you refer to is execute a lambda based on a filter applied on a specified log group, this issue is about the output logs of the actual lambda function executions being stored for x amount of days instead of till the end of days.

@Nr18  uh ! Thank you for your time.

Any update on this feature? Would really love to have this

Thanks @tylersouthwick. I've approved this and will get someone else to provide second approval.

Would these configuration options be available in the Globals -> Function section as well? It would be nice to have the option to configure this once for all functions.

Is there any reason why this hasn't been merged in yet?  Would really appreciate this feature!

Looking through the attached Github issue: https://github.com/awslabs/serverless-application-model/issues/257#issuecomment-461896365 It seems like this was closed as something SAM cannot do without breaking customers.


Pass debug option down

LGTM but it would be nice if the called function can be refactored to use logging rather than regular print.
/lgtm

Why is the tester getting all those weird import errors? Flakes?

/retest

/lgtm


fix(ci): allow roxctl image check to be disabled (3.74)

Description
Backport from main branch.
Checklist

[x] Investigated and inspected CI test results

Testing Performed
CI is sufficient.

The 3.74 build is pretty broken but that is not related to my change, so merging.

The 3.74 build is pretty broken but that is not related to my change, so merging.

Yeah this one has lagged behind in the testing regard.  Next patch from here will be fun.


Webpack 3 support

Any plans to support Webpack 3?

Or 4

Haha part of me never wants to respond to this thread so it can just be a series of

Or 5

Or 6

I am not currently working on this project. But I will happily review a PR with passing tests! :smile: