[ISSUE #2172] fix the issue ArrayIndexOutOfBoundsException

What is the purpose of the change
fix the issue#2172
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.02%) to 51.205% when pulling a500633366086a1b05a22ccb46dee364f862eb85 on fantow:fix_issue#2712_ into fac30c3596d5577cd80a52af9ad89a0839016137 on apache:develop.


Proposal: Rollout of v3 release

This proposal defines three alpha releases.  This will allow for testing three of the bigger new features in isolation: the removal of Tiller, initial support for Lua hooks, and using CRDs as the storage layer for release metadata.
alpha.1

Simplified client only architecture
Initial support for OCI repositories (helm chart push)
Schematized Values Files - Support validating chart values with JSONSchema
Remove global namespace -  Releases are stored within the same namespace as the Kubernetes objects
Vanity import path change

k8s.io/helm => helm.sh/helm

Consolidate requirements.yaml into Chart.yaml
Command renames:

inspect => show
fetch   => pull
delete  => uninstall

Name (or --generate-name) is now required on helm install
helm serve has been removed
Library chart support

alpha.2

Initial support for chart hooks in Lua
Helm tests as first class abstraction
Remove helm init and support XDG Base Directory
OCI support for helm install|template|upgrade|dep|pull commands

alpha.3

Support using CRDs as release storage
Three-way diff (apply)
Lua CLI plugins

Support using CRDs as release storage

I'm not sure this is a good idea for 2 reasons:

We won't have a controller looking for changes to these. The typical model of coupling a CRD with a controller won't be in use
To install a CRD into a cluster is an admin operation because it's cluster wide. Many people who work with clusters won't have that level of permission. Today we have people who can't install Tiller globally because they don't have access. A work around is to use tiller in a namespace. With CRDs the namespaced option won't be there

I would be curious as to what others thing. I'm trying to think of the end users who have to use helm for the first time in a locked down environment.

Thanks for your thoughts @mattfarina. I expressed some of the same concerns RE: requiring admin privileges to register a CRD. I don't think we ever resolved that conversation.
@adamreese when you have time, would you mind sharing your thoughts on that point? As I recall from the conversation, the decision was to stay with secrets for the storage layer because CRDs were still in beta and are expected to change over time (along with the admin privileges required to register CRDs). The only benefit was to allow fine-grained permission controls over who can helm install (create the CRD) vs who can helm upgrade (patch existing resources), and I don't think we've been hearing users ask for that functionality at this time.

As discussed on the public call, we will go with the plan outlined.  I will create the missing milestones for tracking.

closing as the milestones have been created for tracking these milestones. Please follow the beta.1 and the 3.0.0 milestones. Thanks!


Can we please just dock the running debugger, floaty thing?

VSCode Version:

OS Version: Windows 10 x64

Steps to Reproduce:

Launch your project

Reproduces without extensions: (shouldn't matter)
Can we please make it so that,

thing can be docked to just sit above this area:

it gets REALLY annoying having to move it back and forth, because it just loves to sit on top of the one tab that I am actively using, then I go to launch my attach to chrome task, and then it gets in the way of that, since I just moved it out of the way of my tabs!

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


More fixes

This is my next batched GH-issue-fixin' PR.
Regarding the new cop requested by @agrimm, which checks for paren-challenged method calls nested inside a paren-privileged method call, what do you think about this:
method_with_parens(block_taker { method_without_parens 1, 2 })

Is it good, or bad?

Is it good, or bad?

This looks fine to me, as it's the nest method call is not something that would result in a method argument for the outer method.

Thanks for all the helpful comments. PR updated.
I will probably push several more fixes to this branch over the coming days, but if and when you want to merge it, please go ahead; I can open another one.

Apart from the failing build, all changes look good.
Your ConditionalAssignment changes are going to conflict with @rrosenblum's open PRs regarding it. Whoever is ready first for a merge won't have to deal with those. :-)

Updated PR. Fixed build. Also added one new cop... see the last commit.

I like the new cop - quite useful.


Specific input data can trigger encode to return undefined.

If you try to encode certain Uint8Array's, encode will return undefined because the queue variable goes negative.
let array = new Uint8Array([114,14,42,44,172,152,61,121,54,115,14,238,1,93,77,172]);
let encoded: string  = encode(array);
console.log('Encoded '+encoded);

Will encode to 6o#DGft<cVD.N)9AaSundefinedundefined
Or with a hex string,
let hex_str = "\x8a\x04\x5a\x2a\x25\x5c\x23\x7d\x4a\x33\x64\x00\x65\x1a\x08";
let hex_array = new TextEncoder().encode(hex_str);
let encoded: string = encode(hex_array);
console.log('Encoded '+encoded);

Will encode to Yelt,de)Uz)OYvMA0kundefinedundefined
The fix is to change  (numbits >= 13) to (numbits > 13) then queue will not go negative and return undefined values.
diff --git a/base91.ts b/base91.ts
index e94e6fd..2fc94e3 100644
--- a/base91.ts
+++ b/base91.ts
@@ -21,7 +21,7 @@ export function encode(uint8: Uint8Array): string {
   for (let i = 0, len = uint8.length; i < len; i++) {
     queue |= uint8[i] << numbits;
     numbits += 8;
-    if (numbits >= 13) {
+    if (numbits > 13) {
       value = queue & 8191;
       if (value > 88) {
         queue >>= 13;

Thank you, seems that even the less usual strings from the utf-8 sampler didn't contain anything that triggered this bug. I'll publish 1.1 with the fix. Also added one of your examples to the tests, though it's not a kind of library I'll probably be updating frequently enough for this to reoccur somehow :)

Glad to help. It was a total edge case that showed up maybe 50/10000 runs when encoding SHA sums.


Unable To Open Project - D:/Programowanie

On opening a folder with two fsharp projects in it ionide detects both of them, but fails to load with the error below:

It looks like a developer path is hard coded somewhere? I'm on MacOS so I don't even have a d drive, let alone that path.

I should add - the project builds just fine at the command line with dotnet build.

Huh, it doesn’t look like - https://github.com/ionide/ionide-vscode-fsharp/search?utf8=✓&q=programowanie&type=

Can you open F# solution explorer (F# icon in the activity bar), right click the failing project and then check project status?

I can't - not for lack of trying. I just get that same error again: "Unable to open 'Project NerdStrengthTests status': D:/Programowanie/ionide/src/Components/SolutionExplorer.fs."
I'm kinda shocked the unique directory path doesn't immediately make a contributor say "oh no that's my machine!"

Same thing happening to me

@Krzysztof-Cieslak Programowanie seems to a polish word meaning programming (but you know better than me ^^).
Perhaps, one of the dependencies of Ionide has been compiled on a Polish computer and have Programowanie in hard in the code or the path resolution functions.

Perhaps, one of the dependencies of Ionide has been compiled on a Polish computer and have Programowanie in hard in the code or the path resolution functions.

Yeah, sure. But it has nothing to do with the bug happening here - every unhandled exception will have some path displayed. And yeah probably our webpack config has some small issue (it should replace those paths with relative)... but it’s not connected to behavior in any way

Ah yes, I see what you mean.
And you are right #1008 will fix this issue. I didn't read correctly the thread and didn't understand that the error was coming when trying to show the project status...

Fixed by #1008

Can confirm, this problem went away. I'm now struggling with this issue: https://github.com/fsharp/FsAutoComplete/issues/326.  So progress of a sort.


how to debug in IntelliJ IDEA ?

how to debug in IntelliJ IDEA ? could some provide some info about this? thanks .

debug a restful API (via HTTP request).   could you provide some info about this? thanks

What do you want to know? You can start the application in IntelliJ and then issue a curl request to the HTTP you want to test.

Maintainers, you can probably close this issue..


Organization Name Change

PR Type

[ ] Breaking change
[ ] Feature
[x] Patch

Brief Description
This PR makes changes related to the change of the organization name from "NWChemEx-Project" to "NWChemEx" (also known as the NWChemEx Community).
Not In Scope
PR Checklist

[ ] Is documented
[ ] Is tested
[ ] Adheres to applicable organization standards

🚀 [bumpr] Bumped! New version:v1.0.16 Changes:v1.0.15...v1.0.16


一首歌曲播放过程中，直接播放另一首个会报一下异常信息。

Exception in thread Thread-41: Traceback (most recent call last):   File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\threading.py", line 926, in _bootstrap_inner     self.run()   File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\threading.py", line 870, in run     self._target(*self._args, **self._kwargs)   File ".\src\MusicCmd.py", line 464, in __play_pl_base     self.player.wait_for_playback()   File "C:\Users\andrew.he\AppData\Roaming\Python\Python37\site-packages\mpv.py", line 912, in wait_for_playback     self.wait_for_event('end_file')   File "C:\Users\andrew.he\AppData\Roaming\Python\Python37\site-packages\mpv.py", line 969, in wait_for_event     pass   File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\contextlib.py", line 119, in __exit__     next(self.gen)   File "C:\Users\andrew.he\AppData\Roaming\Python\Python37\site-packages\mpv.py", line 1000, in prepare_and_wait_for_event     self.check_core_alive()   File "C:\Users\andrew.he\AppData\Roaming\Python\Python37\site-packages\mpv.py", line 901, in check_core_alive     raise ShutdownError('libmpv core has been shutdown') mpv.ShutdownError: libmpv core has been shutdown

MusicCmd.py

In function __play_pl_base,
Change self.player.wait_for_playback() to self.player.wait_for_shutdown() .
Check here
    def __play_pl_base(self, url):
        self.player.play(url)
        # self.player.wait_for_playback()
        self.player.wait_for_shutdown()
        self.current_song = None
        self.event_pl.set()

Thanks.😁


Difference between ReactiveX/RxJS and Reactive-Extensions/RxJS

Hello,
I haven't found anything on what is the difference between the 2 projects (except their owners) and what are the (functional) differences between them.
Can anyone shed light on this?
Thanks!

As far as I know: ReactiveX/RxJS is a rewrite of Reactive-Extensions/RxJS and is intended to supersede it once it's ready. It's more performant and built with modern JavaScript (TypeScript to be more precise). The Reactive-Extensions/RxJS is built with JavaScript ES5 and doesn't even use CommonJS modules for instance.

@timotei the official line is that this is a rewrite with performance and modularity in mind. RxJS in the Reactive-Extensions group will continue for the foreseeable future while this is in development.  Once it has met and surpassed the existing RxJS in terms of functionality and performance, then we will do a cutover to the new architecture.  This means there is a lot work ahead of us in terms of ensuring our virtual time tests are correct here in addition to any other work items that we have here.  We'd love to have help with both obviously, RxJS in the existing repository in the short run in terms of performance and features, and here with building the future!

Yes... @timotei this is a (currently) pre-alpha "future" of RxJS. If you start using this now, I can roughly guarantee you'll pay some heavy pioneer's tax over the short term. The reason it's over here (ReactiveX) is to join RxJS with the rest of the ReactiveX community.

Thanks a lot for the info!

Thanks for the info. I was confused about this as it is not apparent from the home page what is happening with the two projects. It took a few different avenues for me to arrive here.


Build and push docker image to fabric8 org

@hectorj2f @aslakknutsen the build failed due to not finding the dockerfile.
Dockerfile: no such file or directory

@hectorj2f snapshot fabric8-ui-openshift-nginx image is available for testing.  docker pull fabric8/fabric8-openshift-nginx:SNAPSHOT-PR-10-5


Show suggestions onKeyPress

Hi guys,
I wanted to know if there is a way to show the suggestions when the user press key down.
Thanks in advance.

Hey @pola88
That sounds like a great idea and I can definitely see a use of this. At the moment though, there seems to be no such way. I'll see if I can get to this and a push this feature in the next release, but I can't promise when (extremely busy with school work). I'll keep this issue open so that I remember it.
Thanks!

Great!
Thanks @prakhar1989

Agree on this idea! It would be good if this did not sort tags alphabetically, instead in the order it's received from the API. That way, for example, it can be setup like a "recently used tags" item. Also would be good to be able to limit how many tags are shown.

@prakhar1989
It will be a great help if you could just let users show all suggestions onClick (and make it configurable of course).


"waiting for sign in" should give an option to reload/refresh page from UI

when user stuck in to "waiting for sign in " more than 10/20 seconds ,  there should be a way to refresh/reload  the page from UI.

so it seems like if login fails, command is rejected but it's not reflected in ui.
also if you close the window and open it again it goes back to google login page even when there is a pending login, which is not what i expected.
in addition to that we need decide what we need to decide what we need to do when user clicks log in twice, activate existing window, open a new one and abandon old one etc.
This seems like p1 to me

raising priority to P1

@salomegeo and I just discussed a solve for this

@salomegeo my understanding is that the agreed solution is to leave the login button there, and each time it is clicked to open a new tab for the user to login.

correct. So we can mark this as duplicate of #541. spinning hourglass is already removed.


Update stm32l562e-dk:nsh

Summary

Update TrustedFirmare-M instructions to latest version of STM32CubeL5
Increase idle thread stack size to not overflow during system init
Select ARCH_HAVE_TRUSTZONE for STM32L5
Set CONFIG_ARCH_TRUSTZONE_NONSECURE for stm32l562e-dk:nsh, since NuttX is running in the Non-secure world.

See https://github.com/apache/nuttx/issues/9316
Impact
Contained in board related files.  One exception: Marked STM32L5 to support TrustZone in respective Kconfig file.
Testing
Works fine for me and was verified in #9316

@michi-jung - Nice.
Can the target_cfg.c patch be applied as part of the build?

Within the NuttX build environment only the Non-Secure World Firmware (NuttX + App) is build.  This patch applies to the TrustedFirmware-M code within ST's STM32CubeL5 repository.  Their build process is quite complicated and requires manual intervention.  I don't think it can easily be integrated.
For my projects I am actually using buildroot to build both NuttX and TrustedFirmware-M (from upstream, not ST's variant) and the integrated and signed firmware image.


insider_trading function not in fmpsdk?

I don't see the insider_trading function in the python sdk. Is this a known issue? Are there any plans to add it?

I'm sure it is possible.  I either missed adding it or FMP added it recently.  Let me find time to do this.

I took a crack at adding this here https://github.com/ipl31/fmpsdk/tree/insider_trading but realized that it is on the V4 API endpoint. FMPSDK needs to support multiple version endpoints before it can be easily added. Will open a ticket for that.

I have added the API endpoints listed in the Insider Trading section of the FMP API documentation.  Granted, I didn't test them but I believe they will work.  Feel free to test and fix as you wish.  :-)


MLSD not implemented

I tried your example with my a common FTP server app on the phone, I kept getting 502 for MLSD command. I'm very newbie in all this but doing some research I found MLSD is new and commonly falls back to LIST, I've tested it and it does work. Will it be possible to have it as a fallback? I'm happy to fork and make a pull request if you think it makes sense.

@urielarg
MLSD is not «new», it's from 2007 rfc3659. Since LIST have no standard, parsing is «dangerous» :smile:
Just look at FileZilla's parsing code. If you want, you can try to make it proper working with fallback. But I think in this case you can just inherrit aioftp.Server class and reimplement list for parsing your FTP server response only. Cause implementing whole cases would be painful and borring.

From my cursory tests neither bftpd nor vsftpd seem to implement MSLD or MSLT. Please consider reopening this and implementing fallbacks for either command.

@heftig
As I said before, feel free to implement full LIST parsing, and we will make fall-back. But I think, that LIST is for humans, MLSx is for machines. I'm really disappointed why server-creators still did not release this functionality. It's not new, it's not too hard. So it is better to create issue for bftpd and vsftpd and ask them «why so?».


Reinstate zoom feature

The zoom feature was removed temporarily, issue #94 .
This feature needs to be reinstated so that:

when zooming in it is possible to scroll across the paper
when zoomed out, the paper size is adjusted when dragging elements across the paper
it is possible to pan across the whole diagram when zoomed in, see issue #72
anchors on the elements are scaled correctly when zoomed in, see issue #73
dead space can not be created by zooming in and adding elements, see issue #60

Note that zoom is in the paid-for Rappid version of jointjs, but obviously we stick with the open source (non licenced) version:
https://www.jointjs.com/rappid-comparison

Migrated to new issue in the OWASP area repo : https://github.com/OWASP/threat-dragon-core/issues/13


Adding Stiffness Ignores Gravity

Hello @mphe, I have noticed that when adding a stiffness amount to a rope where the gravity vector is not down, the stiffness simulation will cause the rope to tend towards down.
Now, my vector math is not brilliant, and I couldn't quite glean the meaning of parent_seg_dir as it seemed to involve the overall transform of the rope, but I figured that changing the basis transform to use gravity_direction instead of VECTOR_DOWN seems to address the issue.

Hi,
at the moment, when stiffness is active, the rope will try to snap back into downwards direction, rotated by the node's transform. This is intended, because the desired stiffness direction does not always correlate to the gravity direction, e.g. in a zero-gravity space. Also in general, when bending an object in real life, it will not necessarily snap back into gravity direction, e.g. consider a catapult or a bow and arrow.
That means if you want to change the stiffness direction, rotate the rope node to face the desired direction.
It is a bit unintuitive but the behavior is documented in the stiffness property and I don't want to break existing code by changing it.
Could you let me know a little more about your project and what you are trying to achieve?
I'm considering adding another property to make it more intuitive.

I guess I thought "downwards" was with respect to gravity, but it does make much more sense now that you've explained it.
My project will be using the ropes as cables to connect bits of fixed equipment together, so stiffness will come into it, but altered gravity will not; I was just experimenting with the rope properties when I discovered the behaviour.


feat: scaffold the cli

Current dependencies on/for this PR:

main

PR #1   👈

PR #2

PR #3

PR #4

This stack of pull requests is managed by Graphite.


Create conda requirement file.

We do not currently have instructions on how to run the notebooks or scripts included in the git hub repository. Currently Pangeo and Met Office Spice platforms are supported. There should be instructions for running on each, as well as a conda requirements file for getting running somehwere else. We should be able to run on the following platforms

Met Office Spice
Pangeo
Azure or AWS VM
Azure ML
AWS Sagemaker Studio

We should include the following libraries:

sklearn
pandas
matplotlib
bokeh
tensorflow
pytorch
XG Boost

There may be some conflicts, so we may need to create multiple libraries
The requirements files can then be created from the relevant libraries using the following commands to create a platform agnostic requirements file:
conda env export --from-history > environment.yaml/.txt

I have now created requirements files to create sklearn  and sklearn + xgboost environments in conda.


Release version 0.2.1

Hi @luckysori!
This PR was created in response to a manual trigger of the release workflow here: https://github.com/itchysats/10101/actions/runs/3545890190.
I've bumped the versions in the manifest files in this commit: 6653d833d7a455903a3a54eb22526cb9c7ec5ee1.
Merging this PR will create a GitHub release!

bors r+


Add DRLlibv2.py

This is an alternative to the current RLlib agent with hyperparameter optimization pipeline using Ray tune

good codes.
thanks for your effort.

I have several suggestions:
float | int can be revised as Union[float, int]
str | list[str] can be revised as Union[str, List[str]]
None | int can be revised as optional[int]
could you pls improve the codes?

Thank you for your suggestions @zhumingpassional
I have done the necessary changes

however, some are not changed, e.g., float | int.
pls double check it.

looks much better now.
with respect to num_gpus: Union[float, int], the type is int, and Union[None, int] can be revised as Optional[int].
with respect to function, we'd better point out the return type, e.g., train_tune_model.
could you pls revise it again? thanks for your effort.

For num_gpus, the value can be fractional, like for each trial allocate this percentage of GPU. It is a Ray feature to allocate GPU to multiple trials and number of gpus. And for checkpoint_num_to_keep, None means that keep all the checkpoints. So both None value and int can be passed

ok.
it's good


Missing input file error with backend_tflite

Problem
Missing file error continues when I build apk with 'with_mediatek=1' option.

Log says in terminal
...(Skip)...
INFO: Found 1 target...
ERROR: /home/mlperf/mobile_app_open/mobile_back_tflite/cpp/backend_tflite/BUILD:62:11: //mobile_back_tflite/cpp/backend_tflite:neuron_delegate: missing input file '//mobile_back_tflite/cpp/backend_tflite:neuron/libs/arm64-v8a/libtensorflowlite_neuron_jni.so'
Target //android/java/org/mlperf/inference:mlperf_app failed to build
ERROR: /home/mlperf/mobile_app_open/mobile_back_tflite/cpp/backend_tflite/BUILD:62:11 1 input file(s) do not exist
INFO: Elapsed time: 1899.267s, Critical Path: 109.37s
INFO: 795 processes: 2 internal, 791 linux-sandbox, 2 worker.
FAILED: Build did NOT complete successfully

Branch
The error comes up on 'merge-submission-v2.0-into-android-v2'.
However, it works fine on 'submission-v2.0' branch.

I would like to find out what I've been missing on  'merge-submission-v2.0-into-android-v2'.
Does it require vendor specific libraries pre-built manually before making apk?
Thanks.

*.so files are excluded from Git.
https://github.com/mlcommons/mobile_app_open/blob/af9103845dcbbed8e2e969e746f2a8cb558b32bc/.gitignore#L11
App should be built WITH_TFLITE=1 (default backend) without any issue. To build the app with other backends you need to contact the vendor to get the required files.
For MediaTek backend in this case, please contact @freedomtan.

Currently, the libtensorflowlite_neuron_jni.so from tensorflow-lite-neuron.aar  as stated in https://github.com/mlcommons/mobile_app_open/blob/master/mobile_back_tflite/cpp/backend_tflite/neuron/README.md.
I'll update MediaTek's Neuron Delegate repo by the end of June so that we can use the .so built from Neuron Delegate source code.

Can we close this?
It looks to me like this issue should have been automatically closed by #400 but wasn't for some reason.

Yes, let's close.


List Node Delay

Hello
I have got a problem with using List Node from your package.
The problem starts when I enable Timeout function

It is causing the following error:

Please help, as the Node output data in the format I want, just need to dump the stored information every 1 s.

In combine-list.js please add
        remove(key) {
            delete this.msgs[key];
            const msg = this.combine();

            this.send(msg);
        }

after line 39 and try again (restart NR)

Hello.
Thank you for your help. It is working now.
The Node Red Server is not crashing after your modifications.


Restore better error message for missing argument referenced with '#' in plural rule

Removed in a005b99561012510885cca61ba89864af83d9c3b

@eemeli Okay, I've switched to JSON.stringify and rebased to master. Please review, thanks!

Thank you!


Behavioural changes when using request body as the request content for a chained http request

Description
We have a C# function app calling a Java function app. The C# function app received a request and is supposed to just use the request body as the content for the HttpRequest towards the Java function. While this works in .NET 8 non isolated worker model functions, in dotnet-isolated the request body appears to be empty on the Java function side. This only occurs with some webservers. Reading the entire HttpRequest in the C# Function and passing it on as a StringContent "fixes" this behaviour, though now one would have to read every request body into a string, which
is not what we want when uploading large json files. It also does not seem only Stream related, as a FileStream which is read in the function can be passed as StreamContent to a backend service.I have attached a  minimum repro, I can provide the confidential request url if needed.
TestJasper.zip
Steps to reproduce
Try to send a HttpRequestData Body via a new HttpRequestMessage as Content fails on dotnet-isolated
###Educated guesses
HttpRequest.Body implementation of the Stream type changed from https://github.com/dotnet/aspnetcore/blob/main/src/Http/WebUtilities/src/FileBufferingReadStream.cs to
https://github.com/dotnet/aspnetcore/blob/main/src/Servers/Kestrel/Core/src/Internal/Http/HttpRequestStream.cs
when migration in-process-> dotnet isolated

I managed to isolate it further. transfer-encoding is set to chunked in the dotnet-isolated worker runtime. This causes issues when calling a java function host


Make company.Company.registered_address_country blank=True

Description of change
This makes the field company.Company.registered_address_country blank=True so that it becomes optional in the Django admin.
It was already null=True so no other logic should change.
Generated SQL
This shouldn't create any SQL but Django does this:
SET CONSTRAINTS "c_registered_address_country_id_4a27b881_fk_metadata_country_id" IMMEDIATE; ALTER TABLE "company_company" DROP CONSTRAINT "c_registered_address_country_id_4a27b881_fk_metadata_country_id"
ALTER TABLE "company_company" ADD CONSTRAINT "company_company_registered_address_c_4a27b881_fk_metadata_" FOREIGN KEY ("registered_address_country_id") REFERENCES "metadata_country" ("id") DEFERRABLE INITIALLY DEFERRED

It seems okay though. Another option would be to edit an older migration file to add blank=True and avoid an extra migration but seems messier.
Checklist

[x] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

This shouldn't create any SQL but Django does this:
SET CONSTRAINTS "c_registered_address_country_id_4a27b881_fk_metadata_country_id" IMMEDIATE; ALTER TABLE "company_company" DROP CONSTRAINT "c_registered_address_country_id_4a27b881_fk_metadata_country_id"
ALTER TABLE "company_company" ADD CONSTRAINT "company_company_registered_address_c_4a27b881_fk_metadata_" FOREIGN KEY ("registered_address_country_id") REFERENCES "metadata_country" ("id") DEFERRABLE INITIALLY DEFERRED

It seems okay though. Another option would be to edit an older migration file to add blank=True and avoid an extra migration but seems messier.

It would be good to check how long that takes. We have edited old migrations to avoid that before.

@reupen good call, I've just run the sql in staging and it completed in 0.4 seconds


ValueError: Invalid parameter linearforestregression for estimator Pipeline

Great work! I'm new to ML and stuck with this.
I'm trying to combine pipeline and GridSearch to search for best possible hyperparameters for a model.

I got the following error:

Kindly help : )

Hi,
it's linearforestregressor__n_estimators not linearforestregression__n_estimators
All the best

Oh, I made a stupid mistake. Thank you for your time.


additional-properties-add-test-cases-for-spread-of-record

fix https://github.com/Azure/cadl-ranch/issues/546
Cadl Ranch Contribution Checklist:

[ ] I have written a scenario spec
[ ] I have meaningful @scenario names. Someone can look at the list of scenarios and understand what I'm covering.
[ ] I have written a mock API
[ ] I have used @scenarioDocs for extra scenario description and to tell people how to pass my mock api check.

Python has no work for additional properties so it is better to get approval from JAVA/.NET

wonder do we have service using this spread of Record?


CircularReveal does not have any effect running on Android 5.1

It is weird that running on Android 5.1,ViewAnimationUtils.createCircularReveal does not have any animation starting.

well, it works on my phone furthermore on Genymotion emulator too, can you share any piece of code

device: mi2
system: miui6,android5.0.2
createCircularReveal does not have any animation starting.

Guys, can you share code that you are running?

you can see that it's a old version . you can replace the new code  https://github.com/yhsj0919/YHUtils/tree/master/YHUI

I confirm that it doens't work on Android 5.0 or above (tested on 5.0 and 5.1.1).
My code is the following:

                rootView.findViewById(R.id.consegnati_view).setVisibility(View.INVISIBLE);
                View myView = rootView.findViewById(R.id.choose_view);
                myView.setVisibility(View.VISIBLE);

                // get the center for the clipping circle
                int cx = myView.getRight();
                int cy = myView.getTop();

                // get the final radius for the clipping circle
                int finalRadius = Math.max(myView.getWidth(), myView.getHeight());

                SupportAnimator animator =
                        ViewAnimationUtils.createCircularReveal(myView, cx, cy, 0, finalRadius);
                animator.setInterpolator(new AccelerateDecelerateInterpolator());
                animator.setDuration(CIRCLE_DURATION);
                animator.start();

The purpose is to hide "consegnati_view" and reveal the "choose_view" from the top right corner. They're inside the same parent "io.codetail.widget.RevealFrameLayout" in my xml.
It works perfectly on Android versions below 5.0

marbat87, hm interesting remark, try to figure out today

@ozodrukh I think the code I wrote is correct,because it runs successfully below Android L.Maybe it is a problem of compatibility.Hope you can fix it.Thanks.:-)

Well actually I made a commit relative to this issue, and some sample, cab
you check out them, if everything is fine I will made release it
On May 30, 2015 2:55 PM, "Benson"<EMAIL_ADDRESS>wrote:

@ozodrukh https://github.com/ozodrukh I think the code I wrote is
correct,because it runs successfully below Android L.Maybe it is a problem
of compatibility.Hope you can fix it.Thanks.:-)
—
Reply to this email directly or view it on GitHub
https://github.com/ozodrukh/CircularReveal/issues/31#issuecomment-107017625
.

I'll like to add that it doesn't work for me too 5.0 and 5.1 but works pre-L.
I don't know how to get the fix from github thought.Could you help me

Can you clone repository and check out new version (it is not released in jitpack)

I can confirm that last changes working on 5.1.1, thanks for it.

Great :+1:
i'm closing this issue for now, if something happen please re-open it and provide some more information
will wait for 2 day and made release

When will the update be released?

Can confirm that version 1.1.1 works on Android 5.1.1. @aashrey99 it is now in jitpack

Thanks guys for help :+1:


Issue 110: Add epiweekly fits and scoring

This PR closes #110 .
NB this PR uses forecasttools functions, in particular daily_to_epiweekly. Very handy.
The goal is to compare the epiweekly forecasts of baseline models (which are fitted to epiweekly data) to the epiweekly grouped forecasts from pyrenew_hew.
The daily analysis is unaffected; that is baseline models are still run for daily reporting as well.
Tasks to contribute

[x] Add generate epiweekly truth data from daily truth data step
[x] Add run baseline models on epiweekly
[x] Add extra pyrenew-hew postprocess to chunk the daily posterior preds into epiweekly
[x] check how much revision the scoring function needs (might be zero)

Testing
At each step the test mode pipeline is run to check for failure or breaking changes.

Added a step to convert daily resolved data to epiweekly cf generate_epiweekly.R.

Modified timeseries_forecasts.R to run baseline forecasters for both for daily and epiweekly models.

Added scoring on epiweekly scale for pyrenew. The algorithm is that if epiweekly = TRUE then the pyrenew forecasts for counts and other counts are group summed by epiweek (which auto does by draw), pivoted out to create the correct weekly percentage (which is not the mean of the daily percentages), and then pivoted longer.
Dates for the week are supplied by more forecasttools helpers.
No other changes were need because the scoring is flexible enough to ingest this.

@SamuelBrand1 Since we are making even more model results and anticipate adding PyRenew-EW to the workflow soon, I would really like to push toward https://github.com/CDCgov/pyrenew-hew/issues/101. Could the fable-related parts of that task be accomplished in this PR (i.e. let's store all the results from these time series models in a fable directory)?

@SamuelBrand1 Since we are making even more model results and anticipate adding PyRenew-EW to the workflow soon, I would really like to push toward #101. Could the fable-related parts of that issue be accomplished in this PR (i.e. let's store all the results from these time series models in a fable directory)?

I'd rather avoid mission creep on PRs and complete then do another issue.

I don't see the point of adding new code that saves things in locations that we don't like, but I defer to you.


MSAL: Access token received from the first login(interactive flow) and silent refresh are same.

The access token received using interactive method (manual login) in Azure B2C and the token received using silent refresh (without user interaction) are same. This is leading to access problem in the APIs.
The method used for getting the token interactively -> b2cApp.acquireToken(parameters);
The method used for getting the token silently -> multipleAccountPublicClientApplication.acquireTokenSilentAsync(parameters);

@RamandeepSodhi Assuming the token is not expired, this is expected behavior. Can you describe the issue you are seeing?

Closing due to inactivity. The reported behavior sounds to be by-design.


Consistently say ValidatingWebhookConfiguration

@mamachanko please amend your commit to add a DCO sign-off Signed-off-by: Author Name<EMAIL_ADDRESS>Details

@scothis done!

Thanks


Null result messages are dropped by metadata analysis

Some metadata scanners may return messages even in the case of a null result.  DeceptiveAuthorDetector is an example of this behavior:
# guarddog/analyzer/metadata/deceptive_author.py:69

if len(emails) == 0:
    # No e-mail is set for this package, hence no risk
    return False, "No e-mail found for this package"

However, Analyzer.analyze_metadata() drops all such messages:
# guarddog/analyzer/analyzer.py:121

rule_matches, message = self.metadata_detectors[rule].detect(info, path, name, version)
results[rule] = None
if rule_matches:
    issues += 1
    results[rule] = message

We should keep all scanner messages regardless of findings.
Alternately, we could explicitly adopt a convention that scanners should only return messages with non-null results.  This would let us simplify the Analyzer.analyze_metadata() interface: a return value of None means no findings and a str means findings with the given message.

Alternately, we could explicitly adopt a convention that scanners should only return messages with non-null results
@ikretz this is the default behaviour.
But instead of using the sole string as you suggest, it uses an extra boolean flag ruleMatches for that purpose, if false then message is discarded.

in: guarddog/analyzer/metadata/detector.py, it reads:

# returns (ruleMatches, message)

I don't see it as a bug TBH

Agreed, as discussed, this is a case of a scanner returning a message that it knows will be dropped by the analyzer.  It is not a bug.  We might consider explicitly documenting this behavior of Analyzer somewhere.


Svg doesn't resize with viewbox. App crash if prop viewbox change to viewBox.

I try to make svg with diffenrent size. It's default size is width="30", height="28". When using viewbox="0 0 100 100" svg doesn't change it width and height to 50, 50. It is possible to resize svg like that?
render() {
    return (
        <Svg width="50" height="50" viewbox="0 0 100 100">
            <Path fill={ color } d="..." </Path>
        </Svg>
    );
}

And app crash if prop viewbox change to viewBox.

I'm having problems with the viewBox prop after upgrading to ^5.0.0, too.

related react-native-svg-uri/issues/33

I was experiencing this on Android while using
"react-native-svg": "^4.3.1"
After upgrading to
"react-native-svg": "5.1.7"
the Svg component resizes correctly in Android when the Svg height and width props changes.

Can you try with the latest version or my PR https://github.com/react-native-community/react-native-svg/pull/596
Viewbox is definitely working at least.


Styleguidist General Style

📝 Description

Box around examples
SideBar width and font bigger
border bottom on sections

🎥 Demo
Before

After

@yaacov

/lgtm


cherry-pick: fix out-of-bounds array deref caught by asan

Cherry-pick of #92
Details:

Longstanding oob error caused by incomplete buffer -> bvector conversion
ConvexHullXY closure point logic appended last point instead of first

I think xyz1.push_back (xyz1[0]) still has potential problems if the size and capacity are already equal...

Great point! I changed the push_back to use a local copy of the point, and also added a test to confirm that the compiler appends a valid point on push_back realloc just in case, as I'm sure I've seen (and used!) the anti-pattern elsewhere.

latest changes still pass the reported repro


"Missing required argument" error with version 4.12.0

Description
After upgrading to version 4.12.0, using this module fails with the error "Missing required argument".

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: 4.12.0

Terraform version:
Terraform v1.4.0
on linux_amd64

Provider version(s):

provider registry.terraform.io/hashicorp/aws v4.56.0
provider registry.terraform.io/hashicorp/external v2.2.3
provider registry.terraform.io/hashicorp/local v2.3.0
provider registry.terraform.io/hashicorp/null v3.2.1

Reproduction Code [Required]
module "sns_slack_lambda" {
  source = "terraform-aws-modules/lambda/aws"

  function_name = "sns-slack"
  description   = "Forwards SES notifications to Slack"
  handler       = "sns_slack.lambda_handler"
  runtime       = "python3.8"

  attach_policy_json = true
  policy_json        = data.aws_iam_policy_document.sns_slack_lambda.json

  source_path = "${path.module}/src/sns_slack.py"

  environment_variables = {
    WEBHOOK_URL_SECRET_ARN = var.slack_webhook_url_secret_arn
  }
}

Steps to reproduce the behavior:
terraform apply
Expected behavior
Module should be applied.
Actual behavior
Terraform fails with the error shown below.
Terminal Output Screenshot(s)
│ Error: Missing required argument
│
│   with module.sns_slack_lambda.aws_lambda_function.this[0],
│   on .terraform/modules/sns_slack_lambda/main.tf line 43, in resource "aws_lambda_function" "this":
│   43:   replacement_security_group_ids     = var.replacement_security_group_ids
│
│ "replacement_security_group_ids": all of
│ `replace_security_groups_on_destroy,replacement_security_group_ids` must be
│ specified

Additional context
This works fine with version 4.11.0.

same. For now I just hardcoded to the prior version.

This issue has been resolved in version 4.12.1 :tada:


feat(metrics): add actor execution time metric

What's changed and what's your intention?
Use total_poll_duration in tokio_metrics to get the cumulative execution time of actors.
Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)

LGTM, this will be very useful, thanks!


Can not parse latest teleports from downward messages

needs feedback from https://github.com/polkadot-js/api/issues/4083


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (ci/dependabot-updates@74212c6). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                   Coverage Diff                    @@
##             ci/dependabot-updates     #372   +/-   ##
========================================================
  Coverage                         ?   82.17%
========================================================
  Files                            ?       44
  Lines                            ?     1257
  Branches                         ?        0
========================================================
  Hits                             ?     1033
  Misses                           ?      224
  Partials                         ?        0

Flag
Coverage Δ

linux
82.17% <0.00%> (?)

linux-strategies
84.50% <0.00%> (?)

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

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


no parameter named 'multiprovider'

I am using puppet-openldap 6.1.0 on puppet 6.28 (client) and 6.20 (server) without problems.
when I upgrade to puppet-openldap 7.0.2 I get:
[...]
Info: Loading facts
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: no parameter named 'multiprovider' (file: /etc/puppetlabs/code/environments/ifi/modules/openldap/manifests/server/database.pp, line: 70) on Openldap_database[dc=ifi,dc=uzh,dc=ch] (file: /etc/puppetlabs/code/environments/ifi/modules/openldap/manifests/server/database.pp, line: 70) on node xaver.ifi.uzh.ch
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

I am using 'mirrormode' on my servers, but it does not help if I use 'multiprovider' instead (obviously). actually I checked the module code (puppet-openldap) and all seems fine to me, so I am a bit at a loss why I see this error. I also tried cleaning the puppet agent's cache/lib.
I anyone has an idea how to fix this, please let me know.

Puppet 6 is not supported since v7: https://github.com/voxpupuli/puppet-openldap/blob/master/CHANGELOG.md#v700-2023-07-10

Please check #399, I guess you have the same issue.

hey guys, thanks for your answers, it turns out you are both right.


transform() doesn't work

Try out calling .transform() after the fitting the dataset several times on the test set, the results will start converging to the same value for each item in the test batch.

Would you be able to elaborate on the problem? Why do you need to call transform for multiple times? How would the test data converge to each item in the test set?

So the problem was the following: I first have trained a pacmap transformation on my training data x
fit = pacmap.PaCMAP() transformed = fit.fit_transform(x, init='pca')
and then I wanted to project the new datasets x2, x3 and so on onto the learned embedding space with:
x2_transformed = fit.transform(x2, basis=x, init='pca')
however transform method changes the basis argument x during the call and therefore for the next calls like fit.transform(x3, basis=x, init='pca') the result becomes invalid, since x is not the same anymore. I have solved it by sending the copy of my basis x into the transform method like fit.transform(x3, basis=x.copy(), init='pca')

Thank you for reporting the undesired behavior! We will fix this problem soon.

This problem should have been fixed in the previous release of 0.7.0, so I'm closing this issue.


AIRFLOW-3573 - Remove DagStat table

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title.

https://issues.apache.org/jira/browse/AIRFLOW-3573

Description
The table DagStat can be removed after AIRFLOW-3561
Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation
No need, just removing unused code.
Code Quality

[x] Passes flake8

Codecov Report

Merging #4378 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4378      +/-   ##
==========================================
- Coverage   78.16%   78.14%   -0.03%
==========================================
  Files         204      204
  Lines       16530    16434      -96
==========================================
- Hits        12921    12842      -79
+ Misses       3609     3592      -17

Impacted Files
Coverage Δ

airflow/models/__init__.py
92.57% <ø> (-0.24%)
:arrow_down:

airflow/jobs.py
77.44% <ø> (+0.02%)
:arrow_up:

airflow/www_rbac/views.py
72.78% <ø> (+0.19%)
:arrow_up:

airflow/www/views.py
69.47% <100%> (+0.15%)
:arrow_up:

airflow/bin/cli.py
65.06% <100%> (+0.54%)
: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 0cc078a...41e7ac0. Read the comment docs.

Can you please resolve an error that was in the PR but because of broken Travis it wasn't detected:
======================================================================
3) FAIL: test_delete_dag_dag_still_in_dagbag (tests.api.common.experimental.test_delete_dag.TestDeleteDAGCatchError)
----------------------------------------------------------------------
   Traceback (most recent call last):
    tests/api/common/experimental/test_delete_dag.py line 61 in test_delete_dag_dag_still_in_dagbag
      delete_dag(self.dag_id)
   AssertionError: DagFileExists not raised

Please fix it in the new PR.


onLCP may report even page was hidden previously

Some metrics like LCP & FID only report if the page wasn't hidden prior to the metrics.
FID: https://github.com/GoogleChrome/web-vitals/blob/main/src/onFID.ts#L59
LCP: https://github.com/GoogleChrome/web-vitals/blob/main/src/onLCP.ts#L63-L66
However, in case ActivationStart has a non-zero value like pre-render, LCP compare with the page hidden time after subtracting the activation time . Shouldn't we compare before substract?

@tunetheweb Hey Barry, can you help confirm since you're the prerender expert:)

Yes I think you are correct that this should only be reported if the lastEntry.startTime was before the firstHiddenTime.
However, prerendered pages cannot currently be activated as background tabs (the prerender is currently discarded and the page is loaded as a fresh navigation if someone does right-click->open in a new tab). So this only affects if the page was prerendered AND the LCP was NOT finished by the time the prerender was activated (so the prerender wasn't completed before activation) AND the page was then hidden before the LCP was showing AND the page was then reactivated AND the LCP was shown AND this happened in a time that allowed it to report. I'd imagine that's reasonable rare.
Take these examples:
Activation = 1 second
Hidden = 1.2 second
Unhidden 2.0 seconds
LCP = 2.5 second
In that case the check would be that 1.5 seconds < 1.2 seconds, which fails, so the event would not be reported.
However in this case, the current logic is wrong:
Activation = 1 second
Hidden = 1.2 second
Unhidden 1.4 seconds
LCP = 1.9 second
In that case it would be 0.9 seconds < 1.2 seconds, which succeeds, to the event would be reported (even though it shouldn't). But it does depend on hiding and then unhiding a page really quickly.
But it's still incorrect and an easy fix so will submit a PR now.

Happy with the change in #326 ?

That would be a super rare case I agree.
And the fix looks great, thanks!


more updates regarding to unit-test

[x] Error CS0103: The name 'Trace' does not exist in the current context (56, 13)
[x] Error CS0103: The name 'Trace' does not exist in the current context (57, 13)
[x] Error CS0103: The name 'Trace' does not exist in the current context (59, 13)
[x] Error CS0103: The name 'Trace' does not exist in the current context (61, 13)
[x] Error CS0103: The name 'Trace' does not exist in the current context (62, 13)

Looks good to me.


Images are broken

All the images seem to be broken, for example
Screenshot has gone missing from this page:
https://govuk-prototype-kit.herokuapp.com/docs/make-first-prototype/use-components

This is probably related to the issue #74 and the changes in #69.
I'll look at this now.


suppress Sierra Siri prompt

After a successful VM create and vagrant up, the user login is blocked by a Siri setup prompt.

This prevents the creation of some HOME/ folders, like ~/Desktop.
Workaround: restarting the VM, e.g. with vagrant halt && vagrant up, the second boot skips the prompt.

Thanks for raising this issue! I think this should address it, but let me know if it doesn't.

Worked perfectly, thank you!

🎉


Brew doesn't work

> brew cask install graphiql
Error: Cask 'graphiql' definition is invalid: Bad header line: parse failed

@tribou just ran brew update ; brew cask install graphiql to no avail. Same error as OP.

It was actually a brew cask error. Sorry that I haven't updated issue at the time but now cask is part of brew, so you have to:
brew uninstall --force brew-cask; brew update


Update to Bevy 0.10

DetectDespawn was removed. I figure it existed to make sure that removed components were not missed. This should no longer be an issue as RemovedComponents is now backed by an Events<T> which keeps the events around for an extra frame.
Todo

[ ] Port all examples
[ ] Fix sleeping bug in custom_system_setup3. Not present in boxes3.
[ ] Fix issue with PhysicsHooks. Only NoUserData currently compiles.
[ ] Await new Bevy release

Bevy 0.10 will now use glam 0.23. Should probably upgrade here too.
Related #331


Atoms opens the union of the given paths and the paths specified by the project

I normally start Atom using its icon which runs it without arguments under the home path.
I can then remove the home path as the root folder and add some other root folders that I am working on.
However, when I restart Atom, the home path will be re-added as a root folder which seems unexpected.
As far as I can tell from the code, projects/workspaces are uniquely identified under ~/.atom/storage by the paths that are given on the command line (the path that the command is executed from if there are no paths given). Is this correct?
Would only opening the project paths as root folders be suitable over also opening the given paths as root folders if and only if the project exists?
If the project path doesn't contain some of the given paths, that means the user has manually removed them, given the unique identifier of the project.
If there is interest in any changes, I would be excited to work on it @maxbrunsfeld  :)

I normally start Atom using its icon which runs it without arguments under the home path.

@wvanlint Atom shouldn't be opening your home directory when you run it from the app icon; that's a bug either in Atom or some package. It seems related to a bug that a few people have reported (#5621, #5617).
What OS are you on? Also, does it happen if you clear your ~/.atom/storage directory and open Atom with atom --safe?

I looked into it before, could it maybe be related to the following code at https://github.com/atom/atom/blob/master/src/browser/main.coffee#L136?
pathsToOpen = [executedFrom] if executedFrom and pathsToOpen.length is 0

I'm running Atom 0.182.0 on Ubuntu 14.04, it happens when clearing ~/.atom/storage and running atom --safe too.
I can check if it happens from the latest commit later too.
So am I correct in that your state is saved by the hash of the given paths in ~/.atom/storage?
If so, if you run atom some-path is it actually possible to remove some-path as root folder and add only for example some-path/a and some-path/b as root folders, or will some-path always be added back when you run atom some-path again?
If paths are used as project identifiers, I would think that the former would be the intended behavior.

I looked into it before, could it maybe be related to the following code at https://github.com/atom/atom/blob/master/src/browser/main.coffee#L136?

I think that executedFrom should be undefined unless you're running the atom executable from the command-line.

If paths are used as project identifiers, I would think that the former would be the intended behavior.

Currently, we don't yet have a concept of a 'project identifier' that is separate from a root folder. What we're working on currently is getting multi-folder support working consistently across all of Atom's built-in packages. But then, we may need to rethink our flows for working with multiple root folders. Maybe you can help us to think about the ideal flow.
I'd love it if you wanted to work on this area with me. I think first, we need to spend some time brainstorming about multi-folder workflows in Atom. I'm going to create an issue where we can discuss it with anyone who's interested.

@wvanlint see #5728. I'd like to consolidate our meta-level discussion about multi-folder UX there as much as possible.

It looks like I was indirectly calling the wrong executable through the Ubuntu Dash (the one that is also called from the command line).
Sorry but thanks for the help! I'll take a look at the other issue.


File.exists? is removed in Ruby 3.2

See https://www.ruby-lang.org/en/news/2022/12/25/ruby-3-2-0-released/

Fixes https://github.com/librato/librato-rails/issues/149

Great @espen!
Would love to get this merged, if this project is still active! =)

Hello @nextmat , @bdehamer , @chancefeick and @pat ,
Could anyone please take steps to merge this PR soon. Or could you please any alternative solution (gem) for the changes in this PR?
Thank you !
Asiful Islam Sakib
Software Engineer, WellDev

Hello @nextmat , @bdehamer , @chancefeick and @pat ,
Could anyone please take steps to merge this PR soon. Or could you please suggest any alternative solution (gem) for the changes in this PR?
Thank you !
Asiful Islam Sakib
Software Engineer, WellDev

Hello @nextmat , @bdehamer , @chancefeick and @pat ,
Could anyone please take steps to merge this PR soon. Or could you please suggest any alternative solution (gem) for the changes in this PR?
Thank you ! Asiful Islam Sakib Software Engineer, WellDev

You can monkey patch this pretty easily:
# Removed in Ruby 3.2 but needed for librato (even most recent version as of today).
# Using this monkeypatch seems to make sense (for now).
#
# https://github.com/librato/librato-rails/pull/150
class File
  def self.exists?(filename)
    self.exist?(filename)
  end
end

Not ideal, but will let you move on to some other things for now :)


[ip] Rely on SocketAddr enum Rather Than SocketAddrV4 struct

Description
Catnip uses the Ipv4Endpoint SocketAddrV4 structure to refer to remote and local endpoints. While this is fine when supporting IPv4 addresses only, once we add support to IPv6 it is make the stack a mess. Therefore, we should change the stack so as to refer to local and remote endpoints using a more generic type, like IpEndpointSocketAddr.

@anandbonde are we done with this?


[AutoPR] cognitiveservices/data-plane/CustomImageSearch

Created to accumulate context: cognitiveservices/data-plane/CustomImageSearch

Codecov Report

Merging #2555 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2555      +/-   ##
==========================================
+ Coverage   55.25%   55.39%   +0.13%
==========================================
  Files        6490     6490
  Lines      142522   143523    +1001
==========================================
+ Hits        78756    79501     +745
- Misses      63766    64022     +256

Impacted Files
Coverage Δ

...arch/entitysearch/models/entertainment_business.py
69.23% <0%> (-5.77%)
:arrow_down:

...eservices/search/entitysearch/models/intangible.py
69.23% <0%> (-5.77%)
:arrow_down:

...s/search/entitysearch/models/food_establishment.py
69.23% <0%> (-5.77%)
:arrow_down:

...ces/search/entitysearch/models/lodging_business.py
69.23% <0%> (-5.77%)
:arrow_down:

...ces/search/entitysearch/models/structured_value.py
69.23% <0%> (-5.77%)
:arrow_down:

...ices/search/entitysearch/models/civic_structure.py
69.23% <0%> (-5.77%)
:arrow_down:

.../eventhub/operations/consumer_groups_operations.py
71.2% <0%> (-2.72%)
:arrow_down:

...operations/disaster_recovery_configs_operations.py
13% <0%> (-2.58%)
:arrow_down:

...s/search/entitysearch/models/tourist_attraction.py
70% <0%> (-1.43%)
:arrow_down:

...rid/models/event_subscription_update_parameters.py
35.29% <0%> (-1.07%)
:arrow_down:

... and 84 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 c1208f6...a293852. Read the comment docs.

(message created by the CI based on PR content)

The samples for https://github.com/iscai-msft/cognitive-services-python-sdk-samples/blob/master/samples/search/custom_image_search_samples.py

@iscai-msft Please link your actual PR, not the file from your fork :)


Downloading status broken

Spotify Version: Spotify for Windows <IP_ADDRESS>2.gcc529952
Spicetify Version: 2.10.2
When downloading something, both the liked songs icon and the library button break

The Liked Songs button stays like this even after downloads are finished:

We can try enhancing it
But i don't have premium 🥲

@LarveyOfficial Lets see what we can do

@Astromations fixed the issue by hiding those elements

But in near future we will try to bring them back as well suited progress bar which match with fluent systems and bring additional functionality

For now im closing this issue
But don't hesitate to open another issue when ever you want @LarveyOfficial


[Feature] Deploy Ray Serve replicas across multiple K8s nodes / regions / zones to achieve HA

Search before asking

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

Description
It's primarily writing a doc for the best practice.

Step 1: Create K8s nodes across different zones or regions.

In our doc, we can just create several K8s nodes in a Kind cluster

Step 2: Set up multiple worker groups in the RayService YAML.

Each worker group belongs to a different K8s node / zone / region.
K8s

topologySpreadConstraints
nodeSelector
nodeAffinity
PodAffinity / PodAntiAffinity

Step 3: Pass topology information to rayStartParams
Step 4: Configure Ray Serve applications to enable Ray Serve replicas to be scheduled across different Kubernetes nodes, zones, or regions.

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

can I take this one?

Configure Ray Serve applications to enable Ray Serve replicas to be scheduled across different Kubernetes nodes, zones, or regions.

It's worth noting that many providers do not provide cross-region Kubernetes clusters. For cross-region serving, you would often need to run it across multiple Kubernetes clusters


Add custom search function

The way the search is working now is a bit limited. We need to implemented the toString method, which is not always possible ( we might use records for example ).

We already have a mixin for this


x/time/rate: convert Limit to an Interface

What version of Go are you using (go version)?

$ go version
go version go1.15.5 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOOS="linux"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build868185860=/tmp/go-build -gno-record-gcc-switches"

What did you do?
examined using rate.Limiter to interact with an API
What did you expect to see?
A way to interact with an API where the api implements token bucket rate limiting, or The ability to create an implementation of the rate limit interface to do so.
What did you see instead?
rate.Limiter is not an interface.
Limiter should likely be an interface providing the programmer the options to choose between a token bucket or a leaky bucket implementations, if not sliding and fixed windows as well.
Looking at the API, Allow, Wait and Reserve (and their N counterparts) would all be part of the interface, while get/set limit and get/set burst would likely be specific to only some implementations, and so not part of the interface.

I just threw together a quick migration to an interface to play with here. I had to make the reservation returned by Reserve() an interface as well when thinking forward about implementation of other rate limit algorithms.
I think the name of the Reservation interface should be kept to Reservation, breaking the 'er' rule, but it just works better than anything else I could find.
The only other changes to the API would be the addition of NewTokenBucket(). NewLimiter() then just calls NewTokenBucket(), which should preserve all existing functionality.
I wouldn't have jumped the gun on the proposal process, but I need a sliding window implementation soon, and if i'm writing it anyway, I might as well do so in a manner than benefits everyone.

The x/time/rate package only has one implementation, so why does it need an interface? If you need an abstraction across multiple types of rate limiters, that can live in your code.
To quote the Code Review Comments page:

Go interfaces generally belong in the package that uses values of the interface type, not the package that implements those values. The implementing package should return concrete (usually pointer or struct) types: that way, new methods can be added to implementations without requiring extensive refactoring.

(In any case, it can't be changed now in a backward-compatible way.)

sounds like it's backwards incompatible and out of scope.


Setup a workflow for Docker release

The docker release is done by manually.
The workflow should be triggered when VERSION is updated.

https://github.com/sunglim/systemtrading/pull/57


Files are not made

Describe the bug
When the program is first run, the cozy.txt and 100_cozy.txt files are not made. This then causes an error of the files not being able to open.
To Reproduce
Steps to reproduce the behavior:

Run Program
View Error

Expected behavior
The two files should be created on startup so the user does not have to do it manually.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Linux(Ubuntu)

Additional context
N/A

#19


ZIP file displayed nicely

Icons for directory, basic MIME types

http://www.filamentgroup.com/examples/menus/flyout.php
http://www.filamentgroup.com/lab/jquery_ipod_style_and_flyout_menus/
http://wiki.jqueryui.com/w/page/12137997/Menu
http://jdsharp.us/jQuery/plugins/jdMenu/
http://jquery.bassistance.de/treeview/demo/async.html
http://www.adventuresindevelopment.com/2009/05/19/how-to-create-a-jquery-file-browser-with-jquery-file-tree/
http://www.filamentgroup.com/lab/jquery_ui_selectmenu_an_aria_accessible_plugin_for_styling_a_html_select/
http://stackoverflow.com/questions/2010391/jquery-select-plugin-that-will-allow-for-folder-file-structured-tiered-options

This enhancement is done: https://github.com/scipy/SciPyCentral/pull/188


Prepare 0.11.3

Opening as draft since we may want to include a few more PRs

v0.11.3 (2024-01-02)
Features

Derive Clone for FileTransport and AsyncFileTransport ([#924])
Derive Debug for SmtpTransport ([#925])

Misc

Upgrade rustls to v0.22 ([#921])
Drop once_cell dependency in favor of OnceLock from std ([#928])

I didn't have time to look into the rest of the PRs so I'll release this so that we don't leave these 4 changes waiting anymore.


Using BP_NODE_PROJECT_PATH should keep only the file specified in the path, not the root directory

Describe the Enhancement

Currently when I use the BP_NODE_PROJECT_PATH environment variable, all the files in my monorepo are copied into the final image. What I want is for the command to recognise the yarn.lock or package-lock in the root workspace and use that to build the project in the specified path, but the source code files in the final image should only be the files in the specified path
Possible Solution

No idea because I don't know how this buildpack is implemented
Motivation

I use paketo in my monorepo but I want the final image to be rid of unnecessary files.

In your monorepo, are the sub-projects separate, or are there shared files? I ask because it is possible to run a build with a subset of the repository structure, eg. pack build my-app --path ./some-sub-directory.

Yes, I do that already. I was suggesting this because the docs mentioned using BP_NODE_PROJECT_PATH for monorepo cases.

I think that in the case where you'd use the BP_NODE_PROJECT_PATH environment variable, there is an expectation that the application shares some files with other parts of the monorepo and therefore we cannot safely remove those files from the image. We could update the docs to mention the --path flag on pack build as the preferred solution when your application sub-directories are independent. Does that help?

I think mentioning that when application sub-directories are independent, --path is likely the better option. But for my use case, it's not what I want to have. I'd prefer a way to build a monorepo but only copy the files for the workspace/project I want to build.
I have a Typescript monorepo where workspaces/projects share a root tsconfig.json file. Let's say I have this file structure:

The tsconfig.json for the product project is:
{
  "extends": "../../tsconfig.json",
  "include": ["src"],
  "exclude": ["scripts", "node_modules"],
  "compilerOptions": {
    "outDir": "dist"
  }
}

If I use that, then there's an error when building the project because it can't find the root tsconfig.json. So, I would like to be able to specify that env variable and build projects in monorepos. Currently, if I use BP_NODE_PROJECT_PATH I can't get it to build successfully, maybe I need to tweak something. The build command runs successfully if I compile normally, but using the buildpacks I get weird compile errors. If I build local and just run pack build it copies all the files including the locally built /dist folder and runs fines (but copying all the files isn't my desired goal)
Also, is there a buildpack that can detect TypeScript and automatically use the build script in the project, rather than me specifying BP_NODE_RUN_SCRIPTS=build? I think the buildpack should detect the build script just like it does with the start script.

@ryanmoran any thoughts on this?

There are a couple of requests here. Let me enumerate each and give some guidance on what can be done.

the built image should exclude some extra files that belong to other apps
For this, I'd suggest to try out the source-removal buildpack. We have discussed adding this to a few of the existing language families, but don't have enough data yet to make the best choices for users. Ideally, you could try using this buildpack and giving us some feedback about the settings you used to achieve your desired outcome.

the buildpack should automatically detect and run the build script
This feature was recently added in https://github.com/paketo-buildpacks/node-run-script/pull/154 and released in https://github.com/paketo-buildpacks/node-run-script/releases/tag/v0.5.0 and https://github.com/paketo-buildpacks/nodejs/releases/tag/v0.25.0. Please try those releases out and let us know if it satisfies your expectations.

If I've missed anything else in your request, please let me know.

@ryanmoran yes I just noticed yesterday that the buildpacks automatically runs the build script. That's what I want, and I'm glad it works.
I didn't find that buildpack in the docs. Could you give me some pointers as to how I can use this with the pack cli and combine it with the nodejs buildpack

Sorry, I should have included a link to the source-removal buildpack: https://github.com/paketo-buildpacks/source-removal. The README includes some more details, but to summarize, you could do something like the following:
pack build my-app \
  --buildpack paketo-buildpacks/nodejs \
  --buildpack paketo-buildpacks/source-removal \
  --env BP_INCLUDE_FILES=dist/*

This will remove any files from the /workspace directory that are not in the dist subfolder.


Include how to install in readme

How to install tool from NuGet.org
Example of global package folder.
Video

It's mostly already done other than Video, last one is not necessary.


fix: always convert json payloads to ascii

The POST, PUT and PATCH methods do not convert all payloads to ASCII as ensure_ascii=False is set on the calls to json.dumps().
Currently, if you to send a POST or PATCH request that contains UTF-8 characters to the ConnectWise API, you receive an error.
e.g.
>>> data = [{"op": "replace", "path": "firstName", "value": "Liám"}]
>>> payload = json.dumps(
...   data, ensure_ascii=False # current connectpyse behaviour
... )
>>> response = requests.request("PATCH", url, headers=headers, data=payload)
>>> response.raise_for_status()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".venv/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://redacted/v4_6_release/apis/3.0/company/contacts/5651
>>> response.json()
{'code': 'ConnectWiseApi', 'message': 'The JSON input is not in the expected array format for a JSON Patch document. A JSON Patch document should be a JSON array of patch operations. Please revise your JSON structure and try again.'}

If you set ensure_ascii=True, which is the default of json.dumps(), the request is accepted:
>>> payload = json.dumps(
...   data, ensure_ascii=True # default json behaviour
... )
>>>
>>> response = requests.request("PATCH", url, headers=headers, data=payload)
>>> response.raise_for_status()
>>> response.status_code
200

This PR replaces all usage of ensure_ascii=False with ensure_ascii=True.
I'm happy to flesh this out a bit more if you want this setting to be controlled by the user.
Note that I am testing against an on-prem instance of ConnectWise (v2024.6.100619).

Hi @0xliam, thanks for the PR.

For my own edification, what does the output look like in ConnectWise when you submit these UTF-8 characters?  Does it display them correctly?
I'm leery of making this change globally.  What do you think about defining it as a kwarg instead so the user can optionally affect this when creating a controller?  Something like this:

from connectpyse.sales import opportunity_api
o = opportunity_api.OpportunityAPI(url=URL, auth=AUTH, ensure_ascii=True)

Thoughts?

Hi @markciecior, thanks for reviewing! ConnectWise does display these characters correctly when submitted as ASCII, however I agree it might be risky making the change globally - I've done a minimal amount of testing and it doesn't seem to have caused any problems, but I am happy make this a kwarg with the default as the existing behaviour.
I'll rework my PR and submit it later today! :)

Hi @markciecior, I've reworked this PR as suggested - users can now instantiate the API with the ensure_ascii kwarg, or via the configuration file - and it will default to False, which is the existing behaviour.

Hi @markciecior,
Just wondering if you've had a moment to review this PR? :)


Export request/response types from generated client

Is there a way to add export to request and response types to the client generated via Integration?

It seems to that they are already exported, @bobgubko
https://github.com/RobinTail/express-zod-api/blob/3ae174ac000db2a9e628f9fae1f55c8084078b83/example/example.client.ts#L107-L123
Or maybe I don't understand what you mean.
Please clarify.

So if I need to access GetV1UserRetrieveResponse, I can access it as Response["get /v1/user/retrieve"]?

exactly, @bobgubko

@bobgubko , that's how Provider type works.
You can make your own helper similarly.
https://github.com/RobinTail/express-zod-api/blob/3ae174ac000db2a9e628f9fae1f55c8084078b83/example/example.client.ts#L141-L145

Thanks for clarifying!


Extend projection recipe for template reach-sets

This PR extends the logic in _project_reachset(R::TemplateReachSet, vars) such that the "fast" and exact AbstractPolyhedron projection method from LazySets is used if it is applicable, otherwise we overapproximate the lazy projection.

Moreover, I put back the Platoon model to the Examples section of the docs.


Is it possible to use OpenGL backend on Windows?

First, let me say, it's a pretty cool library, and great bindings. But I have some questions.
Is there a way to pick OpenGL backend when compiling on/for Windows? I'd rather not bother with multiple backends, I'm developing on Linux, and only interested in Windows/Linux desktop application.
I also prefer writing glsl shaders by hand rather than using extra tools to convert between different formats. Converting shader code to byte array is easy, but writing hlsl shaders is a pain, and I couldn't make shdc work so far (and I prefer not using anything except cargo to be honest).
I've edited the texturecube example a little here to show what I want (kept D3d11 backend for now to compile on Windows). Made a little utility to translate shader code to byte arrays. Included an input loop. So far this library is exactly what I want, except for the hlsl shaders on Windows.

Looking at the build.rs file it looks like the backend is configurable from the outside via environment variables (not sure if this is the "best practice" to configure Rust projects though:
https://github.com/floooh/sokol-rust/blob/c445b2bb930be6885093242eb563b069460157f5/build.rs#L33
...e.g. setting an environment variable `SOKOL_BACKEND=GL" before building should do the job (the actual compiler definition is then selected here: https://github.com/floooh/sokol-rust/blob/c445b2bb930be6885093242eb563b069460157f5/build.rs#L72-L118

Regarding sokol-shdc, in the Zig build file I have added a build step like this, but this is just a helper to recompile the shader files, it's not as fancy as the cmake integration I use in the sokol-samples (where the shader compiler is integrated fully into the regular build process).
https://github.com/floooh/sokol-zig/blob/24134d507a9b6d61ea1c438c0ec2ffd19530a93c/build.zig#L212-L254

Setting the global variable (in global Cargo config file on Windows) worked like a charm. Thank you!
I know I'm missing out on many features, but I'd rather stick to something I know for now.

Sorry, forgot, closing the issue now


search is not working with arabic pdf in azure

Please help to create a site like https://jfkfiles2.azurewebsites.net/  for arabic pdf

Shared link is not the one associated with this repository, so closing this issue. For what its worth, this demo should in theory work with Arabic PDFs as well if you make a few small changes (not tested so not for sure this will work, might need some other changes as well):

Change this line to use "generateNormalizedImagePerPage" to ensure that your PDFs will be converted to images and thus can be OCR'd
Change this line to use "OcrSkillLanguage.Ar" so that the OCR will be Arabic.


CI: Add workflow to validate config files

Use the config file JSON schema.
Closes #7.

LGTM :+1:

I think you forgot to click approve @jamesturner246 :stuck_out_tongue:


Low number suppression added

Added low-number suppression in demo pipeline

Thanks for this @MattStammers .  This applies low number suppression on the entire data frame and not on the output tables. The latter is tricky to achieve.
For instance, Penal is one of the categories in Discharge Destination. Let's say there are 6 patients discharged to this destination. This would not get suppressed with the current method using a threshold of 5.
However, when cross tabulating Discharge Destination against ACSC vs non-ACSC, we may find that of these 6 patients,  1 patient had ACSC while 5 did not. This will still be visible in the output. If we are applying SDC, it should be at this stage.
A simplistic solution would be to just mask the cells with the low numbers. But as @quindavies  pointed out, it is trivial to recalculate cell counts from the row totals. An alternative would be to mask the entire row for the Penal category - but it will still be possible to recalculate this from the column totals!
For certain features such as age bands, it may be possible to assimilate low number bins into neighbouring bins.  Others will need to be dealt with on a feature by feature basis.
I am not confident we will be able to achieve low number suppression the way it probably should be done and alternative strategies may end up being tokenistic. My preference would be to send the tables as is to Sheffield and for the lead team to combine the tables and apply SDC on the final outputs before publishing but we can discuss on Thursday.

You are right @vvcb In the end we have stuck with the above locally and suppressed it to a higher number (in our case 20) to guarantee no n<5 columns. However, as you say we will see what happens with the national decision-making. If no suppression is to be applied we can easily regenerate the dataset


Demos => Error: Trying to replace an element that wasn't mounted!

In some demos I get errors.
This seems to happen in the latest versions of patternplate.
I can work around it with awkward HTML structures (I haven't found out how and why).
Steps to Reproduce

Create a parent Component that allows children
Create a child Component
ensure that the parent only renders 2 elements
force 3 elements to be rendered
the error should occur

e.g. might fail
export default function() {
  return (
    <MyComponent>
      <Assignment/>
      <Assignment/>
      <Assignment/>
      <Assignment/>
    </MyComponent>
  )
}

while might work
export default function() {
  return (
    <Foo>
      <Bar/>
      <Baz/>
      <MyComponent>
        <Assignment/>
        <Assignment/>
        <Assignment/>
        <Assignment/>
      </MyComponent>
    </Foo>
  )
}

Demos that were working perfectly before (< 2.0.0-60 ?) are now affected

Versions
"@patternplate/render-styled-components": "2.0.0-63",
"@patternplate/cli": "2.0.0-63",

Log
patternplate.web.vendors.js:33555 Uncaught Error: Trying to replace an element that wasn't mounted!
    at TextNodeBrowserTag.replaceElement (VM11873 patternplate.web.vendors.js:33555)
    at TextNodeBrowserTag.inject (VM11873 patternplate.web.vendors.js:33500)
    at StyleSheet.inject (VM11873 patternplate.web.vendors.js:33710)
    at ComponentStyle.generateAndInjectStyles (VM11873 patternplate.web.vendors.js:34698)
    at StyledComponent.generateAndInjectStyles (VM11873 patternplate.web.vendors.js:34354)
    at StyledComponent.<anonymous> (VM11873 patternplate.web.vendors.js:34410)
    at getStateFromUpdate (VM11873 patternplate.web.vendors.js:23231)
    at processUpdateQueue (VM11873 patternplate.web.vendors.js:23313)
    at updateClassInstance (VM11873 patternplate.web.vendors.js:23792)
    at updateClassComponent (VM11873 patternplate.web.vendors.js:25057)

Workaround: Use render-react/render
Before:
module.exports = {
  render: '@patternplate/render-styled-components/render',
  mount: '@patternplate/render-styled-components/mount',
}

After
module.exports = {
  render: '@patternplate/render-react/render',
  mount: '@patternplate/render-styled-components/mount',
}

This is an issue with styled-component versions, most likely I'll deprecate render-styled-components soon because of this.


LLT changes

What type of PR is this?

unit test code for server.go API

What this PR does / why we need it:
This PR has unit test for "/home/root1/gopath/src/github.com/soda-cdm/kahu/providers/nfs_provider/server/server.go" file
Which issue(s) this PR fixes:
added for Coverage
Test Report Added?:
added
Test Report:

Special notes for your reviewer:

/stale
closing the PR.
Feel free to open new PR.


STM32 DAC Driver - Add support to DAC3 for STM32G4 devices

Summary
Add support to DAC3 for STM32G4 devices. The nucleo-g431kb:comp configuration was changed to use DAC3CH2 signal on.
COMP2 inverting input. The voltage on inverting input can be changed using dac tool.
Impact
Testing
Tested on nucleo-g431kb board.

Hi @daniel-p-carvalho please update your config:
./tools/refresh.sh --silent photon/wlan nucleo-g431kb/comp

You don't need to close, just modify it, do a git rebase interactive and push forced ;-)

I decided to close it because I made a mess when I did a git rebase. I
found it easier to create a new branch and import the changes.
Thanks for the help
Daniel Pereira de Carvalho
Em qui., 29 de jul. de 2021 às 18:02, Alan Carvalho de Assis <
@.***> escreveu:

You don't need to close, just modify it, do a git rebase interactive and
push forced ;-)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-nuttx/pull/4254#issuecomment-889455232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOEMMGYA26VKFA2SBY27C5LT2G6VNANCNFSM5BHDVOMQ
.


Add AirBase

Insert URLs to the project here: https://github.com/JohnPaton/airbase

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All new projects listed will be posted on our Mastodon channel.

@avaldebe @JohnPaton

Thanks!


Add macro verbatim blocks

What
This adds the ability to specify verbatim blocks in macros, so macro code isn't immediately execute.
Why
Take a look at how the macro def_clone is implemented:
  macro def_clone
    # Returns a copy of `self` with all instance variables cloned.
    def clone
      clone =<EMAIL_ADDRESS>      clone.initialize_copy(self)
      GC.add_finalizer(clone) if clone.responds_to?(:finalize)
      clone
    end

    protected def initialize_copy(other)
      \{% for ivar in @type.instance_vars %}
        @\{{ivar.id}} =<EMAIL_ADDRESS>      \{% end %}
    end
  end

Note all the \{% and \{{ in there. It's needed because we don't want to execute @type right away: we want to defer it until the method is executed, so it works like a macro method. The \ is ugly, but it works.
With this PR, we can write it like this:
  macro def_clone
    {% verbatim do %}
      # Returns a copy of `self` with all instance variables cloned.
      def clone
        clone =<EMAIL_ADDRESS>        clone.initialize_copy(self)
        GC.add_finalizer(clone) if clone.responds_to?(:finalize)
        clone
      end

      protected def initialize_copy(other)
        {% for ivar in @type.instance_vars %}
          @{{ivar.id}} =<EMAIL_ADDRESS>        {% end %}
      end
    {% end %}
  end

verbatim basically won't treat the block inside it as macro code, but instead as just a string that's being pasted. Then when the clone and initialize_copy methods are invoked, the pasted code that's actually macro is executed.
So this is just a better way to write and do what we can already do.
I also need this for a complete implementation of https://github.com/crystal-lang/crystal/pull/6082 . The JSON::Serialize module must inject an initialize in the included type so that types referenced in attributes (like converter) work well. For example:
class Foo
  class MyConverter
  end

  @[JSON::Field(converter: MyConverter)]
  property x : Int32
end

If MyConverter is pasted in the included JSON::Serialize module, it will give an error because MyConverter can't be reached from that location (that's the reason why I had to remove all the private types in the specs in that PR). If initalize is injected with an included macro, MyConverter can be accessed.
But since that injected initialize method already uses @type in macros, I'll have to escape with \ all of that. But it's so much simpler to just surround everything with a verbatim block.

And can you add a spec?

Oh... the formatter will definitely not work with how it's implemented. Damn...

Yeah, this will have to wait. It's too much work to make the formatter work, I'll probably need to introduce a new AST node. I'll close this PR until then.

Just another reason for me to consider the formatter harmful :/


Close/Hide method to dispose of the dialog

Are there any methods to remove the dialog from the UI? I didn't see anything in the .js files to do this. Thanks.
Oh and a huge :+1:  for this, I've been trying to achieve this for a couple days with no luck.

Hey @bradmartin,
I've pushed new version to npm - let me know will this work for you!
Thanks

@enchev - thanks for adding that. It just seemed necessary to be able to manipulate the show/hide with a method exposed.


MGet action crash when some keys are not found

When I'm trying to retrieve mutliple values via mget action and some keys in skytable are not present, result will fail on this error:
failed execution: failed to read from conn: an error occured when reading entry#1/1: failed to read typed array entry #9/11: strconv.ParseInt: parsing "\x00": invalid syntax
Is that an expected behaviour?

Just in case, are you running Skytable 0.8 or 0.7?

Please see if the fix works for you, thanks.

I'm using Skytable 0.7.6. Right now I'm getting a different error:
panic: interface conversion: interface {} is nil, not []uint8

Yes it's working now. Elements needs to be checked on nil type and all is fine. Thanks!


Update dwm version

Hey, currently caviar is using outdated version of dwm which throws error while running:
import SCANCODE_WIN from "./scancode_win.json" assert { type: "json" };

  at https://deno.land/x/dwm@0.3.3/src/platform/glfw/window.ts:64:1

which is now fixed in recent update of dwm

forgot to update this repo after update dwm, will get on this shortly

Cool, thanks

published a new release, thank you for reminding me! props to you


Remove "Dataflow" from apache_beam init.py file

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @aaltay

Coverage remained the same at 70.222% when pulling 6d0451a99455453c736c69c4255a1e20d491d1f4 on charlesccychen:remove-dataflow into 6e8d8d65708e3d5b0525a7b7f97597d4af71d51e on apache:master.

LGTM


Add platform dependent download link

that reminds me we should really do flatpak builds, and advertise those as well.

@sofar in the case you were not aware, there are already unofficial flatpak builds at the forum: https://forum.minetest.net/viewtopic.php?f=42&t=15336
Github link: https://github.com/est31/minetest-flatpak
What's missing is three things:

singing keys (prefferably c55 has a copy so that in the case sth happens we won't lose them)
server to host the builds as a flatpak repo. Just having bare flatpak files is not the way flatpak ought to be used. You are supposed to have a repo.
(maybe) some automation to have a nightly repo as well

I hoped flathub would have became a thing, but seems nothing is happening here, so we'll have to roll our own. :/

updated

added all Linux platforms in the linux section, although I have no idea if they work (yet) :P

Updated.
Worrying about old versions existed before this PR. I think this should be solved in a separate PR by adding apt-repo instructions.

I think we should go the same path big browsers are going.

Detect OS family, Linux, Windows, macOS
Offer most common package options and architectures for that family. For example. If Linux, offer [Linux 64 DEB], [Linux 64 RPM], [Linux 64 TGZ], [Linux 32 DEB], [Linux 32 RPM], [Linux 32 TGZ], [Source]. If Windows offer, [Windows 64 Installer], [Windows 32 Installer]. If macOS [macOS 64 DMG].

Should be
sudo pacman -S Minetest

For arch, as its generally up to date.

For Elementary OS it could automatically open in the App Store.


Implement xyz chemistry file generator.

Based on the old code.

Can you provide more information? Where should this enhancement live, can we use existing functionality from PySCF/Psi4, where do we want to use it?

@m-i-w Hi, we already have a code for that in one of our proof of concepts. Just needs to be adapted for our incoming protein folding feature.

Closed via #655


Default request_strategy

Hi @pyromaniac,
Going through the Chewy config in my app, I noticed that I was not using any Async index updates perviously when I thought I was. This is due to Chewy always wrapping requests in the atomic strategy. I believe that I want to use an async strategy like resque or active_job in controllers as well.
Is there any reason why using async strategies in a controller is a bad idea?
If not, I would recommend either documenting Chewy.request_strategy in the README or possibly not pushing the atomic strategy if the current Chewy.strategy already inherits from Atomic?
relevent code here: https://github.com/toptal/chewy/blob/8cb6248ae11c5309b73cbdaff2bbb6b8862031e8/lib/chewy/railtie.rb#L16
Thanks for your feeback

Put another way, the README suggests to me that setting Chewy.strategy in an initializer will apply to the controller request cycle, when it does not.

Using async strategy is not a bad idea, but atomic is default just because it is available always. If you will do a PR with readme fixed - it would be awesome. Thanks.


feat(coil-extension): add accept-payment via declarativeNetRequest

Note this a different approach to https://github.com/coilhq/web-monetization-projects/pull/1910, as suggested in the comments by @sublimator https://github.com/coilhq/web-monetization-projects/pull/1910#discussion_r633034791
This PR proposes the addition of the Accept-Payment header which follows the Content Negotiation pattern used by Accept, Accept-Language, etc.
If this PR is merged, the following header will be added to web requests from the client:
Accept-Payment: webmon/*;q=0.8

HTTP servers can then choose specialized content for the client, or issue a content negotiation sequence: a 402 status code followed by user-facing instructions on how they might pay for the site.
More on the Accept-Payment proposal.
This is opportunistic and doesn't replace the client-side web monetization standard.
This is a signal, not a guarantee. Well behaved clients can get the benefit of individual content without having to wait for detection / complete page load.
Verify / QA
The see this in action you can build the extension and then inspect the outgoing http headers. There should be a new header:
Accept-Payment: webmon/*;q=0.8

To test how content negotiation might work you can run the express server at Accept-Payment proposal and visit the pages referenced for web monetization.

@mankins
Hey, so of course Firefox doesn't support the declarativeNetRequest api/permission :/
It's a pity cause it's a decent API with performance advantages (one assumes at least)

@sublimator I could add back https://github.com/coilhq/web-monetization-projects/pull/1910 into this PR?
I haven't looked into how the different builds happen, but you could branch if the browser has support? Or does this make the proposal DOA?

Yes, we could make that work! :)
I can look into hacking the build for you.
I just forgot before its not impl on FF. A pity Mozilla is so against it.

@mankins
I guess you saw this:
https://blog.mozilla.org/addons/2021/05/27/manifest-v3-update/
Mozilla are implement declarativeNetRequest :)


build: fix CLIVersion ldflag ref

seeing some build failure on the homebrew side, https://github.com/Homebrew/homebrew-core/pull/181762, and realized the package re-org per https://github.com/cilium/cilium-cli/pull/2762

Did the build failure on the Homebrew side catch that debugging symbols had not been stripped?

we stripped the debugging symbols for the release build


non pixel perfect resizing

Some kind of not perfect stretching, some lines have 2 pixels height instead of 1

It might just be that you can't scale up the game to some resolutions without 1 pixel offset, but if that's the case, tha offset should be at the top/bottom left/right side of the screen so it isn't noticeable


Llama(n_ctx = 0) doesn't work

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[x] I carefully followed the README.md.
[x] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[x] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
As stated in the docs, creating Llama with n_ctx=0 should default to the model's trained context length and work.
Current Behavior
Instead, llama_cpp crashes after loading the model.
Environment and Context
Linux 6.2 kernel, Python 3.11, latest llama_cpp instaled with CUBLAS support.
Failure Information / Steps to Reproduce
import llama_cpp
model = llama_cpp.Llama(model_path="../models/llama-2-7b-chat.Q4_K_M.gguf", n_ctx=0)
print(model.n_ctx())
print(model("The quick brown fox jumps ", stop=["."])["choices"][0]["text"])

Extra Info
I think this is related with the following line inside Llama._init_(), effectively setting self.n_batch=0, since n_ctx is 0:
        self.n_batch = min(n_ctx, n_batch)  # ???

Removing this line and setting self.n_batch = n_batch, avoids the crash. The above code prints the model's trained n_ctx. However later raises an exception during inference and doesn't complete it:
ValueError: could not broadcast input array from shape (39,) into shape (0,)

When you pass --n_ctx 0 to llama.cpp main it refuses to run, so llama-cpp-python should probably do the same. I think changing the assert from a strict None check might be appropriate:
    def n_ctx(self) -> int:
        assert self.ctx is not None
        return llama_cpp.llama_n_ctx(self.ctx)

https://github.com/abetlen/llama-cpp-python/blob/8e44a32075de4aba2fc9877d4a2a34a0e7314c0d/llama_cpp/llama.py#L432