SANY crashes are presented to user as parse success

Version Info
VS Code TLA+ Extension Version: 1.5.1
JRE version: AdoptOpenJDK build 15.0.1+9
OS version: Windows 10
What happened?
I tried to use the TLA+ VS Code extension to parse specs that caused a NullPointerException in SANY (see https://github.com/tlaplus/tlaplus/issues/553) and the TLA+ VS Code extension presented as though no parse errors occurred in the module.
What did you expect to happen?
I expected to be notified in some way that there was a parse failure (in this case manifesting as SANY crashing with a NullPointerException)
Steps to reproduce

Create a file Test.tla filled with the malformed specs from https://github.com/tlaplus/tlaplus/issues/553
Parse the specs in the TLA+ VS Code extension

The fix is available in the v1.5.2-alpha.2 pre-release.

The fix is available in the v1.5.2-alpha.2 pre-release.

Fix published in v1.5.2

Fix published in v1.5.2


GHA, SSH, logstream SDK updates [In Progress]

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Blocked until there's SDK support for logstream / ssh auth tokens and the GitHub action create or update API is fixed (errors out on 201 response)


Servo for Power button presses?

Silly idea, but how difficult would it be to implement a servo or solenoid that actually presses the power button if the machine is fully off?
We just had a ton of power outages in my area that the thought crossed my mind.
Thanks!!

That was one of the solutions I considered, but it's a bit too unreliable.
The direction I'm currently pursuing is to have TinyPilot control a power strip that can cut power to the machine if it becomes unresponsive. You'll have to configure your BIOS to always power on LAN, though.
Expect to see this in the next 2-3 months.


purple-hangouts: 2018-03-28 -> 2018-12-02

Motivation for this change
Wanted to try it and thought I'd get various fixes and such :).
https://bitbucket.org/EionRobb/purple-hangouts/commits/all
Things done

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

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

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

(autosquash'd the fixup, ty, been using for a while and works for me! :))


[macOS] Patch 4

Reasons for making this change:
Split PR https://github.com/github/gitignore/pull/2204 into individual commits and use the correct line ending
Links to documentation supporting these rule changes:
Each commits description contains links to documentation and/or references
Important note: the commit https://github.com/github/gitignore/commit/0b29bbec71f5fde17f5365e2150af562c83383ec add a dir name that use four null chars. Github detect it as a binary file, that means starting from that commit the diff must be reviewed with a git client (not directly on Github)

I am not in favor of a .gitignore file that will be impossible to review on GitHub in the future.
So could you first make a PR without HFS+ Private Data, and let HFS+ Private Data be handled in a separate pull request?

This looks like it's gone stale and I agree with @Coeur that I'm not excited by a file that we can't review easily through a PR. Closing for now, but please feel free to re-open if you want to pick this up again.


Ori Will of the Wisps: low fps/GPU utilisation on NVIDIA

Running Ori: Will of the Wisps through DXVK/Proton yields very low (below ~30) fps and low (~20%) GPU utilization on RTX3080/Ryzen 5950X.
I've tried many, many things to find and debug this, because the game runs perfectly smooth at a locked 60fps on my Steam Deck, which is much less powerful. In the end I turned to enabling the Vulkan validation layer and debug logging, and found the game will spam the log with 100s of validation errors similar to this one, before hard-locking the machine (probably because of the amount of logging):
err:   UNASSIGNED-CoreValidation-Shader-InterfaceTypeMismatch:
err:   Validation Error: [ UNASSIGNED-CoreValidation-Shader-InterfaceTypeMismatch ] Object 0: VK_NULL_HANDLE, type = VK_OBJECT_TYPE_SHADER_MODULE; | MessageID = 0xb6cf33fe | Attribute type of `VK_FORMAT_R8G8B8A8_UINT` at location 4 does not match vertex shader input type of `ptr to Input vec4 of sint32`
err:   VUID-vkCmdDraw-None-02699:
err:   Validation Error: [ VUID-vkCmdDraw-None-02699 ] Object 0: handle = 0x8639c40000000ab7, type = VK_OBJECT_TYPE_DESCRIPTOR_SET; | MessageID = 0x1608dec0 | Descriptor set VkDescriptorSet 0x8639c40000000ab7[] encountered the following validation error at vkCmdDraw time: Descriptor in binding #0 index 0 requires UINT component type, but bound descriptor format is VK_FORMAT_R32G32B32A32_SFLOAT. The Vulkan spec states: Descriptors in each bound descriptor set, specified via vkCmdBindDescriptorSets, must be valid if they are statically used by the VkPipeline bound to the pipeline bind point used by this command (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-vkCmdDraw-None-02699)

I have attached a snipped version of the debug log for review.
The game otherwise looks fine while playing but I can imagine the validation errors are indicative of a vulkan issue that may affect performance?
Edit: some additional info:
To reproduce just start a new game. Framerate is still ok in the first few seconds of the intro sequence though you can already see stuttering/frame pacing issues when the 3 geese fly through the screen. The frame rate hovers between 50 and 60 fps until you get control, just walking left for a few seconds the frame rate progressively drops down to 30 where the spider thing is waving at you. Interestingly the frame graph in the HUD looks relatively clean, no big spikes.
When I enable the Vulkan validation layer + debug logging, the game starts up quite normally, HUD shows normal framerate during the shader compilation screen with the running Ori, but the moment the menu opens everything slows to a crawl immediately to the point it is almost impossible to start a game, eventually locking up the system.
I tried all kinds of configurations, all with the same result: Nvidia driver 525, start Steam BPM through steamcompmgr, from a KDE Plasma session, exclusive fullscreen, borderless, V-Sync ON/OFF, frame rate limit on/off, all with the same result.
This is on Arch x86-64 with kernel 6.1.1 by the way
Software information
Ori: Will of the Wisps
System information

GPU: Nvidia RTX 3080
Driver: 520.56.06
Wine version: Proton 7, Proton 6, Proton 5, GE-Proton 7, all same behaviour
DXVK version: v2.0

Log files

d3d9.log:
d3d11.log: oriwotw_d3d11.log
dxgi.log: oriwotw_dxgi.log

I tried to make an API trace, but running the game with the apitrace DLL's will crash Wine with an unhandled page fault before the game even starts. I tried to run the version installed directly through the Linux Steam client, as well as a copy installed through Windows Steam running under wine, but both crash. I can see the apitrace file being generated by the way, so at least that part is working ;-)

You can try to make one both with and without dxvk and see if it makes a difference. One made on Windows would also be fine if you have access to that.

@Blisto91 how do I make the API trace using DXVK? If I just put the DLL’s with the game exe and launch through Steam the trace is not generated, i guess I need to do the equivalent of overriding them using winecfg?

I've very frequently tested this game during DXVK 2.0 development on Nvidia and have never noticed performance being lower than expected (on a mobile 2060). It's also running perfectly fine on my AMD desktop.

Do the vulkan validation errors and the fact there are literally ~4000 emitted for just a few seconds of game runtime provide a clue? Surely they mean something is not quite right? Anything else I can do to debug further?
I know the game runs perfect on AMD, I’ve played it on steam deck. The performance I’m seeing on my desktop is terrible compared to that.

FWIW I'm running some older 5.19 kernel on my Nvidia machine right now. Building a 6.1 right now to try that, given that even on my main desktop I've had problems with 6.1 it might just be that.

Yes the kernel is the only thing I did not try to downgrade yet (proton/dxvk or driver version did not make a difference).
if you need more info or know something else I can try just let me know

Looks like a kernel regression indeed, with 6.1 I'm getting periods of like 10 FPS for a few seconds in the game and then a bit over 30. With 5.19 it's sitting at a locked 60.

RADV is less affected by this than Nvidia but the issue is there on both.

Ok, serves me right for using Arch with its bleeding edge kernels then, I guess 😆
Does this mean the issue is strictly on the kernel side and needs to be fixed there? And with the tsc=reliable boot option, does it run as before on a 5.x kernel or is it only 'kind of fixed' in the sense performance is better but still not great? If so I will just downgrade my kernel for now.
Thanks for debugging so far!

I'm GPU-bound now so performance is the same between the two kernels. I don't really have the time now to do extensive testing to compare the two, but with the boot parameter I'm not seeing any obvious red flags in perf top either, so it should be good.
As for who is at fault here, don't know, but it's certainly not something we can fix on the Proton/DXVK side.

Ok makes sense. I already observed that even with the 6.1 kernel with bad performance, both CPU and GPU utilisation were very low even despite the low framerates, so definitely something is fundamentally broken there. I guess I'll just add the boot option then.

I have since tried with kernel 6.1.1 and 6.1.5 with the tsc=reliable boot option, and with the Arch linux-lts (5.15) kernel, without any success.
My system is a Linux hypervisor running a QEMU Linux VM with GPU passthrough, which may or may not be related to this performance bug, but the fact of the matter is that a for example a game like GTA V runs perfectly in the same VM, and the Ori game itself also runs perfectly fine in a QEMU Windows 10 VM on the same machine. Which suggests there is something uniquely broken with the combination of running Ori WoTW through DXVK/Proton.
I've given up on this for now. I'm not fully convinced this issue really is 'not a bug' for DXVK though, considering I have no problems with other games. If there is anything I can do to further debug I'm open to that.


Custom Trace Profile / Collision Preset

I noticed the project has a Custom Channel "Ability" and custom Preset "TargetActorGroundLocation"
Are those critical to the project, and if so should they be documented?
Reason I ask, I'm trying to implement the Meteor GA in C++.  I know I have to take care of the "K2Node_LatentGameplayTaskCall automagic sorcery" myself, but for some reason the target just paints at 0,0,0 but doesn't animate or track when I move around.  I've been stepping through the code and thought maybe I'm missing some configuration regarding collisions.

Hello. I wouldn't call the custom channel and profile critical, but that AbilityTask does need a Trace Profile that collides (blocks) with the ground (World Static/Dynamic).
Custom channels and profiles are up to the developers. Most games will make their own channels for things like hitscan bullets and projectiles. Some games might reuse those for abilities or make their own separate ones for abilities. Some games may want their abilities to ignore other abilities in flight or they may want their fireballs to collide with each other. That's more of a general UE4 thing and less of a GAS thing so I will not include it in the docs as it's very game specific.

Sounds good.  Incidentally, I got it working in C++, I had missed the step to set the StartLocation of the SpawnedActor instance after calling WaitTargetData.
	// Get Player & Show HUD Message
	AXBZTGasSandboxCharacter* Player = CastChecked<AXBZTGasSandboxCharacter>(ActorInfo->AvatarActor.Get());
	Player->ShowAbilityConfirmCancelText(true);
	// Create WaitTargetData
	UAbilityTask_WaitTargetData* WaitTargetData = UAbilityTask_WaitTargetData::WaitTargetData(this, FName("Place WaitTargetData"), EGameplayTargetingConfirmation::UserConfirmed, InTargetClass);
	AGameplayAbilityTargetActor* SpawnedActor;

	//To reiterate, K2Node_LatentGameplayTaskCall only does automagic sorcery for Blueprint.
	// In C++, we have to manually call ReadyForActivation(), BeginSpawningActor(), and FinishSpawningActor().

	WaitTargetData->ReadyForActivation();
	WaitTargetData->BeginSpawningActor(this, InTargetClass, SpawnedActor);
	WaitTargetData->FinishSpawningActor(this, SpawnedActor);

	FGameplayAbilityTargetingLocationInfo TargetingLocationInfo = MakeTargetLocationInfoFromOwnerActor();
	// Set targeting location to the front and ahead of the Player
	TargetingLocationInfo.LiteralTransform.SetLocation(Player->GetActorForwardVector() * 200 + TargetingLocationInfo.LiteralTransform.GetLocation());
	//Assign TargetingLocationInfo to Targe Actor Instance - this is what I was missing
	SpawnedActor->StartLocation = TargetingLocationInfo;


[Bug]: No ability to propose a feature on GitHub

What happened?
The contributing guidelines mention "Proposing new features," but there are only templates for reporting a but or a security vulnerability.
Expected: I can go to Issues and propose a feature via a feature template.
Actual: If I want to propose a feature, I have to use a bug/security template.
Reproduction
Go to Issues. See that there isn't a feature (or blank) template.
Version
Stable
Relevant log output
No response
Code of Conduct

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

I would prefer to use GitHub discussion with category idea, since that would be right place for a discussion.

I'm not familiar with Discussions. I don't know anyone else that uses it. And I didn't see anything in Contributing. I'll try and check over there, tho. Thanks.

now added a issue template as well. https://github.com/oceanroleplay/discord.ts/issues/new/choose

Nice. Thanks!


QBMM + Preston

Verified on CPUs with and without parallel I/o

@anandrdbz please explain your changes to these files in the comments here. For example, I see many goldenfile changes and many others and I'm not sure I understand the provenance of all of them. Please make this easier for me to review by commenting on why and how each file was changed.

I do not like the addition of pb and mv to add of the output routines:
https://github.com/anandrdbz/MFC/blob/21c74df680cc98d8b5f10fb3e8c85dbfa456947a/src/simulation/m_start_up.fpp#L50
for example.
This makes the code more confusing to read. Most folks won't know what these variables are, and they are only used in a very small set of cases. Is there a way to deal with outputting these variables in parallel without adding these as arguments?

please (very briefly) explain your changes to these files in the comments here. For example, I see many goldenfile changes and many others and I'm not sure I understand the provenance of all of them. Please make this easier for me to review by commenting on why and how each file was changed.

Golden file changes are for all the qbmm tests (including previous polytropic ones). These are required due to a fundamental mistake in how we were dealing with moments, where the averaging wasn't being done properly in the R and Rdot directions.

please (very briefly) explain your changes to these files in the comments here. For example, I see many goldenfile changes and many others and I'm not sure I understand the provenance of all of them. Please make this easier for me to review by commenting on why and how each file was changed.

Golden file changes are for all the qbmm tests (including previous polytropic ones). These are required due to a fundamental mistake in how we were dealing with moments, where the averaging wasn't being done properly in the R and Rdot directions.

Got it, I agree.

@sbryngelson , I have also removed the redundant s_assign_patch_bubbles which was mentioned in one of the other issues

For reference https://github.com/MFlowCode/MFC/issues/177 task 2

This change appears to have mistakes: https://github.com/MFlowCode/MFC/pull/180/files#diff-c46f7cd6ce1da4d2b4e71a99cd0b6a96e70bb49fa0c73679edd804dcd7f81c40
In particular, @wilfonba removed coarsen_silo intentionally and you are putting it back in. Please fix.

@sbryngelson I have fixed all the files individually which had merge issues (it added back coarsen_silo inadvertently). They should all be resolved now

@anandrdbz this looks good. Can you do one (I promise) final thing, check the run times for the GPU code vs CPU code with the different bubble models and see how they differ before and after your PR?
I mostly just want to make sure that the QBMM without Preston and the method of classes cases are just as fast as they used to be.
You can run the cases on Summit or some similar machine and report the differences here in the PR comments.

@sbryngelson So there was a minor gpu allocation bug for large cases that is now fixed for qbmm + polytropic
Here are the results for slowdown/speedup on Summit for a 3D problem with 1M points :
CPU + Method of classes : No slowdown
CPU + QBMM w/ Poly : 1% slowdown
GPU + Method of classes : No slowdown
GPU + QBMM w/ Poly : 15% slowdown
MOC shows no slowdown as this part of the code was not touched
QBMM w/ Polytropic does show minor slowdown (esp on GPUs) due to introduction of terms for Re, Surf tension and pv (total of 26 terms in bubble dynamics as opposed to only 11 previously). All of this was missing in the previous implementation of qbmm so it's not a fair comparison. However, the slowdown is quite minor (15%) in the grand scheme of things

Also, I noticed the GPU test on phoenix failed, and I believe this is due to issues on phoenix. All the tests pass on Bridges2 and Summit, but I am unable to get a compute node on phoenix. In fact the case that failed was the very first one which has nothing to do with qbmm (or the change that I just made) .
@wilfonba was having similar issues with GPUs on phoenix today as well


Issue with auth callback

string indices must be integers
Reported on decksite by<PHONE_NUMBER>56466691
Request Method: GET
Path: /authenticate/callback/?state=NMPon7VGLhe8XYpGZJjrs8RtrTZjPE&code=n5WqmF9NJ0Dq8cxF4CLNW9WW5BpUoT
Cookies: {'__cfduid': 'd8378121517f27c367d348137d79a0cd01517076142', '_ga': 'GA1.2.987394123.1517076163', '_gid': 'GA1.2.337100979.1517773820', '_gat_gtag_UA_109131120_1': '1', 'session': 'eyJvYXV0aDJfc3RhdGUiOiJOTVBvbjdWR0xoZThYWXBHWkpqcnM4UnRyVFpqUEUiLCJ0YXJnZXQiOiJodHRwczovL3Blbm55ZHJlYWRmdWxtYWdpYy5jb20vcmV0aXJlLyJ9.DVpbYA.hKeR0Bdq0Mi8wZ8l7XA2HH2_ca4'}
Endpoint: authenticate_callback
View Args: {}
Person:<PHONE_NUMBER>56466691
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.119 Safari/537.36
Referrer: https://discordapp.com/oauth2/authorize?response_type=code&client_id=338056190779195392&redirect_uri=https%3A%2F%2Fpennydreadfulmagic.com%2Fauthenticate%2Fcallback%2F&scope=identify+guilds&state=NMPon7VGLhe8XYpGZJjrs8RtrTZjPE
Request Data: {}
TypeError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/main.py", line 401, in authenticate_callback
auth.setup_session(request.url)
File "./decksite/auth.py", line 34, in setup_session
if guild['id'] == configuration.get('guild_id'):

What do you mean by that? guild is a dict with strings inside and the value for guild_id in the configuration is also a string.

"string indices must be integers" --> this sentence is from the log? Hahaha, ok. I thought you wrote it. Anyway, I couldn't reproduce locally. I inserted debug prints and both guild and the configuration are dicts with the correct content, so ¯_(ツ)_/¯

string indices must be integers

Reported on decksite by<PHONE_NUMBER>60647424

Request Method: GET
Path: /authenticate/callback/?state=mXj80qP4ifqldbpsQDPLKArLCDRXeo&code=f7eG4ZOOAtxAzOzGgsb9h9IawnOoRA
Cookies: {'__cfduid': 'd0ba854efab1c57a0c3689858191f3c891516503670', '_ga': 'GA1.2.658395427.1516503676', '_gid': 'GA1.2.636774223.1518982686', '_gat_gtag_UA_109131120_1': '1', 'deck_id': '10020', 'session': 'eyJvYXV0aDJfc3RhdGUiOiJtWGo4MHFQNGlmcWxkYnBzUURQTEtBckxDRFJYZW8iLCJ0YXJnZXQiOiJodHRwczovL3Blbm55ZHJlYWRmdWxtYWdpYy5jb20vZGVja3MvMTAwMjAvIn0.DWumTA.a0vEsB0h-Hma679QKMLUogKxMqY'}
Endpoint: authenticate_callback
View Args: {}
Person:<PHONE_NUMBER>60647424
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.167 Safari/537.36
Referrer: https://discordapp.com/oauth2/authorize?response_type=code&client_id=338056190779195392&redirect_uri=https%3A%2F%2Fpennydreadfulmagic.com%2Fauthenticate%2Fcallback%2F&scope=identify+guilds&state=mXj80qP4ifqldbpsQDPLKArLCDRXeo
Request Data: {}

TypeError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/main.py", line 413, in authenticate_callback
auth.setup_session(request.url)
File "./decksite/auth.py", line 34, in setup_session
if guild['id'] == configuration.get('guild_id'):

string indices must be integers

Reported on decksite by<PHONE_NUMBER>60647424

Request Method: GET
Path: /authenticate/callback/?state=mXj80qP4ifqldbpsQDPLKArLCDRXeo&code=JKfPL47xAZBzYiH8qFpEaYnug0ONaH
Cookies: {'__cfduid': 'd0ba854efab1c57a0c3689858191f3c891516503670', '_ga': 'GA1.2.658395427.1516503676', '_gid': 'GA1.2.636774223.1518982686', '_gat_gtag_UA_109131120_1': '1', 'deck_id': '10020', 'session': 'eyJvYXV0aDJfc3RhdGUiOiJtWGo4MHFQNGlmcWxkYnBzUURQTEtBckxDRFJYZW8iLCJ0YXJnZXQiOiJodHRwczovL3Blbm55ZHJlYWRmdWxtYWdpYy5jb20vZGVja3MvMTAwMjAvIn0.DWumTA.a0vEsB0h-Hma679QKMLUogKxMqY'}
Endpoint: authenticate_callback
View Args: {}
Person:<PHONE_NUMBER>60647424
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.167 Safari/537.36
Referrer: https://discordapp.com/oauth2/authorize?response_type=code&client_id=338056190779195392&redirect_uri=https%3A%2F%2Fpennydreadfulmagic.com%2Fauthenticate%2Fcallback%2F&scope=identify+guilds&state=mXj80qP4ifqldbpsQDPLKArLCDRXeo
Request Data: {}

TypeError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/main.py", line 413, in authenticate_callback
auth.setup_session(request.url)
File "./decksite/auth.py", line 34, in setup_session
if guild['id'] == configuration.get('guild_id'):

string indices must be integers

Reported on decksite by<PHONE_NUMBER>6344576

Request Method: GET
Path: /authenticate/callback/?state=cIBSdJ6SswHDSRUdbz85XJo9yiRyQx&code=L2Cg47DIMiMs7E85oYRlWetgUPZCRT
Cookies: {'__cfduid': 'd2849dfa352ce3ded24ce83be279c5a881505189005', '_ga': 'GA1.2.1588039399.1509662043', '_gid': 'GA1.2.858974374.1519100658', '_gat_gtag_UA_109131120_1': '1', 'session': '.eJxFT1tPgzAU_i_nmRgupRQeyaYbWQgDFdQYUqGwBqQLLdlg2X-3M9E9nu981wvQ-psPEDS0l8wAXkMAPvaQjR1iOdhByPUwGCDopA52KRVVTFOqbZjVEc7kabPK0pf6ayFuEQl_5um8P9_5SnRMu1-AVhWT8u-GPNnm5D1OxVux9PnsPcay2E_Lc_u6O20irWfnIx-ZLKmCwHIt39M9bPLg2j5yLHz_37pjExHTNGBkjYYO_yHVUyLyltNYrHqSZCTrwrzDfrr2ujXa6RBZiaNe86Fns0HxZtZYO_G-lvBpwK9NqeYbBUJGRzbCVcN0bJluNUx9f_0BH9dmrw.DW3RwA.3dAXShf9c5nyueCY5t6USIUBTgA'}
Endpoint: authenticate_callback
View Args: {}
Person:<PHONE_NUMBER>6344576
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.167 Safari/537.36
Referrer: https://discordapp.com/oauth2/authorize?response_type=code&client_id=338056190779195392&redirect_uri=https%3A%2F%2Fpennydreadfulmagic.com%2Fauthenticate%2Fcallback%2F&scope=identify+guilds&state=cIBSdJ6SswHDSRUdbz85XJo9yiRyQx
Request Data: {}

TypeError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/main.py", line 413, in authenticate_callback
auth.setup_session(request.url)
File "./decksite/auth.py", line 34, in setup_session
if guild['id'] == configuration.get('guild_id'):

The error string indices must be integers implies that guild is a string, not a dict.

string indices must be integers

Reported on decksite by<PHONE_NUMBER>03646720
Request Method: GET
Path: /authenticate/callback/?state=ODuIcSp9ShV5WTzNsWWo0vYPRapBWy&code=2hZ27yNmEMAxRYZ3nIbZQsCl4AyMdY
Cookies: {'__cfduid': 'd5fad562e3b2fc07d544ecf4ab45307c51516748590', '_ga': 'GA1.2.395751054.1516748593', '_gid': 'GA1.2.1584967468.1519426732', 'deck_id': '10338', '_gat_gtag_UA_109131120_1': '1', 'session': '.eJxFj0FPg0AQhf_LnImZZVmg3ARD6aHaSHVjjCErLAVFQHahrU3_excT7XHevHnzvROI4qtuIShFo6QFdQEBEGQec1zfoUhdx_VsBAs6MerKzpQWWhrPw924ytN-kVbPjG9_7hXnHU4vm0fRh_x49evuU5r4E4g8l0r9zUDLZUXySfF1Gn7QA4_oVqAoojiJ4vXe3MtDXw9SZUJDQJiNxEHPozeEEnuB3nU_w7vo-IgWDLI0UvX_xE60r-P9eFx9d-ypqSbUGyaT26VW0fsMqfKuN21eTW_Z6rqctd1YN4WCNwt-YzJ9nC0QSjHIAc5GFsNOGqp2bJrzBRd2ZwI.DXPY5Q.hjucXV0bzCoTla4_AAb94tnJC2w'}
Endpoint: authenticate_callback
View Args: {}
Person:<PHONE_NUMBER>03646720
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.186 Safari/537.36
Referrer: https://discordapp.com/oauth2/authorize?response_type=code&client_id=338056190779195392&redirect_uri=https%3A%2F%2Fpennydreadfulmagic.com%2Fauthenticate%2Fcallback%2F&scope=identify+guilds&state=ODuIcSp9ShV5WTzNsWWo0vYPRapBWy
Request Data: {}
TypeError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/main.py", line 430, in authenticate_callback
auth.setup_session(request.url)
File "./decksite/auth.py", line 34, in setup_session
if guild['id'] == configuration.get('guild_id'):

After adding a debug print statement here is the result:
Feb 25 16:19:34 katelyngigante uwsgi[8797]: auth.py: guild: global (<class 'str'>)
So it seems that sometimes the guild json is just the string "global". Does that ring any bell?

Hypothesis: user is not a member of any guilds, and only uses Discord for
PMs/oauth.
On 26 Feb 2018 10:18, "jgabrielygalan"<EMAIL_ADDRESS>wrote:

After adding a debug print statement here is the result:
Feb 25 16:19:34 katelyngigante uwsgi[8797]: auth.py: guild: global (<class
'str'>)
So it seems that sometimes the guild json is just the string "global".
Does that ring any bell?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/issues/4227#issuecomment-368597063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL2zrYZSggT1aN0vK-K-exlENZBBcw1ks5tYvV1gaJpZM4R6HRm
.

@jgabrielygalan are we done here? Or are we just hacking past an error?

I think we can close this. What is missing is just understanding why Discord does this, but the code is fine.


Support arguments to scripts

Fixes #30

@oprypin let me know how I can help get this merged and released (it's summer, so things can go slower).


[Github] NEXT-00000 - Improve storefront render exception

1. Why is this change necessary?

If you are currently working with the storefront there is a custom error (Shopware-StorefrontException) which catches the TwigError and displays information about a view & parameters of the error.
However, the view name does not match the actual file containing the error, but rather the base template name of the page, which does not do much to identify the source of the error.
There is also no linked file & line in the Shopware-Exception. Because of this there is no file-preview and no way to jump to the error source, or even identify which line contains the error.
This PR fixes all these issues by using the correct Twig template name and showing the preview/linked file & line with the error source, which greatly improves the information output of the error message.

Before

After

2. What does this change do, exactly?

Changed the call parameters from StorefrontController to share the TwigError
Changed the StorefrontException to display the correct viewname (filename) and display the preview for file & line
Changed the StorefrontExceptionTest to match the changes of this PR

3. Describe each step to reproduce the issue or behaviour.

Create shopware development store
Insert error into a twig template
Try to access the page with the error in the view

4. Please link to the relevant issues (if any).
/
5. Checklist

[X] I have rebased my changes to remove merge conflicts
[X] I have written tests and verified that they fail without my change
[X] I have created a changelog file with all necessary information about my changes
[X] I have written or adjusted the documentation according to my changes
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at 813815a
This pull request improves the handling and logging of StorefrontExceptions that occur when rendering a storefront view fails due to a TwigError. It refactors the cannotRenderView function to take a TwigError as an argument and adds a unit test for it. It also updates the changelog and the StorefrontController accordingly.

🤖 Generated by Copilot at 813815a

Add a changelog entry for the improvement (link)
Replace the view name and message parameters of the cannotRenderView function with a TwigError parameter, and use its properties to populate the exception message and data (link, link)
Pass the caught TwigError instance to the cannotRenderView function in the renderView function of the StorefrontController class (link)
Create a mock TwigError instance with a given message, line, and source context, and use it to test the cannotRenderView function in the StorefrontExceptionTest class (link, link)
Add a use statement for the Twig\Error\Error class in the StorefrontException class (link) and the StorefrontExceptionTest class (link)

Imported from Github. Please see: https://github.com/shopware/shopware/pull/3426

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-32331.
[created from NEXT-32331, comment 485421]


Only check line numbers after the test is finished

Users should be able to freely change their test script without the interruption of the line numbers until the script is confirmed working.
Only then should the test framework check the script's line numbers with a suggested change (if it doesn't have them yet).
This change will disable line number check on the test framework's tests because most of those cases are designed to fail. But the number check wasn't design for those tests anyway.

@ono-max @ko1 can we merged this too? I think the small drawback of this change to bring back the "old" checking order is reasonable.
It's bothering me for a while as I started proposing such change since https://github.com/ruby/debug/pull/197

Yes we can. I agree with your opinion.

@ono-max thank you 😄


Error when using ShapeletModel

I am trying to fit ShapeletModel and passing an X array with data (dim=(72,78,6)) and Y array with labels (dim=(72,)) to the fit method.
I am getting an shape error:
Traceback (most recent call last):
File "/home/me/repos/core/ts/experiments/supervised_ts_clustering.py", line 50, in
instance.fit(x_train, y_train)
File "/home/me/repos/core/ts/sota/supervised/shapelet.py", line 35, in fit
self.instance.fit(TimeSeriesScalerMinMax().fit_transform(np.nan_to_num(data)), labels)
File "/home/me/anaconda2/lib/python2.7/site-packages/tslearn/shapelets.py", line 288, in fit
verbose=self.verbose_level)
File "/home/me/anaconda2/lib/python2.7/site-packages/keras/engine/training.py", line 1598, in fit
validation_steps=validation_steps)
File "/home/me/anaconda2/lib/python2.7/site-packages/keras/engine/training.py", line 1183, in _fit_loop
outs = f(ins_batch)
File "/home/me/anaconda2/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 2273, in call
**self.session_kwargs)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 895, in run
run_metadata_ptr)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1124, in _run
feed_dict_tensor, options, run_metadata)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1321, in _do_run
options, run_metadata)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1340, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Reshape cannot infer the missing input size for an empty tensor unless all specified input sizes are non-zero
[[Node: shapelets_0_2/Reshape_1 = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/cpu:0"](false_conv_0_2/convolution/Squeeze, shapelets_0_2/Reshape_1/shape)]]
Caused by op u'shapelets_0_2/Reshape_1', defined at:
File "/home/me/repos/core/ts/experiments/supervised_ts_clustering.py", line 50, in
instance.fit(x_train, y_train)
File "/home/me/repos/core/ts/sota/supervised/shapelet.py", line 35, in fit
self.instance.fit(TimeSeriesScalerMinMax().fit_transform(np.nan_to_num(data)), labels)
File "/home/me/anaconda2/lib/python2.7/site-packages/tslearn/shapelets.py", line 274, in fit
self._set_model_layers(ts_sz=sz, d=d, n_classes=n_classes)
File "/home/me/anaconda2/lib/python2.7/site-packages/tslearn/shapelets.py", line 375, in _set_model_layers
for di in range(d)]
File "/home/me/anaconda2/lib/python2.7/site-packages/keras/engine/topology.py", line 602, in call
output = self.call(inputs, **kwargs)
File "/home/me/anaconda2/lib/python2.7/site-packages/tslearn/shapelets.py", line 85, in call
xy = K.dot(x, K.transpose(self.kernel))
File "/home/me/anaconda2/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 991, in dot
xt = tf.reshape(x, [-1, x_shape[-1]])
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/ops/gen_array_ops.py", line 2619, in reshape
name=name)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
op_def=op_def)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2630, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/me/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1204, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InvalidArgumentError (see above for traceback): Reshape cannot infer the missing input size for an empty tensor unless all specified input sizes are non-zero
[[Node: shapelets_0_2/Reshape_1 = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/cpu:0"](false_conv_0_2/convolution/Squeeze, shapelets_0_2/Reshape_1/shape)]]
Process finished with exit code 1
Does anyone met the same error?

Hi,
Thanks for reporting this bug. I've never seen that before. Maybe this could be related to keras and/or tensorflow version. Could you tell me which version of these modules you have installed on your machine so that I can try to setup a similar virtualenv and check if I can reproduce the bug?

I have the following:
keras: 2.0.8
tensorflow: 1.3.0

I think we can remove this thread. I found the bug and this was on my side: I was passing to ts_sz not the number of segments but the dimensionality of the signal. Now, once i changed it, it doesn't throw me this error.

OK, great!


Error when using macros in shaders.

Godot version
4.0 alpha 13
System information
GNU/Linux Pop Os 20.04 GPU: Nvidia GeForce GTX 1070ti.
Issue description
The shader editor throws an error if I use a macros multiple times in one line of code.
No work

Work

Steps to reproduce
Define a macros and try to use it multiple times in one line of code.
Minimal reproduction project
No response

cc @Chaosus

Can confirm, one of the culprits might be this in expand_macros():
		// Remove expanded macro and recursively replace remaining.
		p_defines.remove_at(i);

The loop should try to expand macros until it no longer finds anything to expand and it naturally should not remove any defines during that loop. Looks like this regression happened when the regex was replaced with normal string logic. The whole expansion code feels unnecessarily complex too, I should have some free time to check this out, unless Chaosus wants to try.

I have the same problem. What should I do other than waiting someone to fix the bug?


[question] How do I create dynamic logfile path

Hi there,
I'm trying to use pino-roll to log to a file with this format: application/logs////xxx.1.log
but this is only work for application startup.
I want to dynamic create this format .
how to approach this problem

You'd need to send a PR to add that capability.


[docker] add docker_network tag to net metrics

What does this PR do?
Port the docker_network tag to docker network metrics:

change NetworkStat struct to ContainerNetStats, holding a list of InterfaceNetStats
add ContainerNetStats.SumInterfaces() for retro-compatibility
fix the network matching logic and update tests
update the docker check to use that

@conorbranagan managed to change to a []*InterfaceNetStats alias, but that means de-referencing the pointer to be able to iterate/append. Do you know if that has performance implications?


non web apis

I am wondering about using this for non web gui with the 3 golang gui projects / frameworks of like GIO, Ebiten and Fyne.
docs

I think this might be the Api for Darwin : https://github.com/gravitational/teleport/blob/master/rfd/0054-passwordless-macos.md
https://github.com/gravitational/teleport/blob/master/lib/auth/touchid/api_darwin.go and that folder seems to be the implementation.
github.com/go-webauthn/webauthn is also used by it, just like this repo.

I think this might be the one for Windows:
https://github.com/gravitational/teleport/blob/master/lib/auth/webauthnwin/zsyscall_windows.go

At first sight, it shouldn't cause any problems. The GUI part (with the js-client) isn't mandatory here. You may use any client implementation.
If we are talking about a non-HTTP API, to be honest, I haven't thought about it this way until now. But it sounds interesting, and it's a good reason to extend _examples.
Thank you for the references you provided! They look pretty useful. I'll check them out to understand how to use them in this project.


update FHIR Brief presentation and record

https://github.com/IHE/IT-Infrastructure/blob/master/Presentations/IHE-FHIR-Brief.pptx


Can't train to convergence

Hi, thanks for your implementation!
When i run the mnist_fwdgrad.py and choose the Conv model, the train loss doesn‘t decrease as in the paper, but decreases to 1.7 and then suddenly rises to more than 14. The training is divergent, unable to train to convergence. Have you ever encountered this problem?

Hi @wuyaozong99, sorry for the late response. Have you run the training with the default hyperparams? Have you tried also with the MLP network?

@belerico Thanks for your reply. I found that I used a fixed learning rate 2e-4 instead of the default setting, which decreased with iteration. After using the default settings, the divergence phenomenon is alleviated.

Now another problem is bothering me.
When using a model with normalization layers, such as BatchNormalization, I must chahge fmodel, params = fc.make_functional(model) to  fmodel, params, buffers = fc.make_functional_with_buffers(model)
Then i change the functional_xent, add a param buffers to pass the buffers. Now the functional_xent is like this
def functional_xent(params, model, x, t, num_classes,buffers) -> torch.Tensor:
    y = model(params,buffers, x)
    return _xent(y, t, num_classes)
And then,
f = partial(
             functional_xent,
             model=fmodel,
             buffers = buffers,
             x=images.to(DEVICE),
             t=labels.to(DEVICE),)
Then i run the mnist_fwdgrad.py, an error was reported when calculating the jvp:
RuntimeError: During a grad (vjp, jvp, grad, etc) transform, the function provided attempted to call in-place operation (aten::add_.Tensor) that would mutate a captured Tensor. This is not supported; please rewrite the function being transformed to explicitly accept the mutated Tensor(s) as inputs.
It's a little difficult to solve with my poor knowledge, do you have any method? I think that by solving this problem, we can extend forward gradient to more models which have the normalization layers.


Broken: Get repository stats

operating in /github/workspace/03-22-2307-A611/fluxcd/flagger
fetch.py for fluxcd/flagger
+ python //fetch.py fluxcd/flagger --snapshot-directory=newsnapshots --fork-ts-outpath=forks-raw.csv --stargazer-ts-outpath=stars-raw.csv
230322-23:07:59.183 INFO:MainThread: processed args: {
  "repo": "fluxcd/flagger",
  "snapshot_directory": "newsnapshots",
  "fork_ts_outpath": "forks-raw.csv",
  "stargazer_ts_outpath": "stars-raw.csv"
}
230322-23:07:59.183 INFO:MainThread: output directory already exists: newsnapshots
230322-23:07:59.439 INFO:MainThread: Working with repository `Repository(full_name="fluxcd/flagger")`
230322-23:07:59.575 INFO:MainThread: Request quota limit: RateLimit(core=Rate(reset=2023-03-23 00:06:53, remaining=997, limit=1000))
230322-23:07:59.575 INFO:MainThread: fetch top referrers
230322-23:07:59.697 ERROR:MainThread: this appears to be a permanent error, as in "access denied -- do not retry": 403 {"message": "Resource not accessible by integration", "documentation_url": "https://docs.github.com/rest/metrics/traffic#get-top-referral-sources"}
+ FETCH_ECODE=1
+ set +x
error: fetch.py returned with code 1 -- exit.

@kingdonb Do we need one more fix similar to #20?

Yes, I can take a look at it today 👍

https://github.com/jgehrcke/github-repo-stats/wiki/Tutorial#step-1-create-personal-access-token-and-store-it-as-repository-secret
I'm following through on this now, it looks as though we need a GITHUB_TOKEN with repo scope in order to use this.
Has it always been so? That seems excessive, from the description of repo scope in the GitHub Actions docs:

repo | Grants full access to public and private repositories including read and write access to code, commit statuses, repository invitations, collaborators, deployment statuses, and repository webhooks. Note: In addition to repository related resources, the repo scope also grants access to manage organization-owned resources including projects, invitations, team memberships and webhooks. This scope also grants the ability to manage projects owned by users.

We need all that just to read the stats from the repo?
Data that you don't persist today will be gone in two weeks from now.
High-level method description:
This GitHub Action runs once per day. Each run yields a snapshot of repository traffic statistics (influenced by the past 14 days). Snapshots are persisted via git.
Each run performs data analysis on all individual snapshots and generates a report from the aggregate — covering an arbitrarily long time frame.
Looking for a quick start? Follow the simple tutorial in the Wiki.

Can anyone see if we have really provisioned a secret here with repo scope, and maybe it has expired? The error 403 indicates that our access being denied is related to an authentication failure, IMHO.

I'm assuming that we did have one, probably it was created with the permission of the core maintainers, with an expiry date. I'm happy to design a process that can keep this token refreshed, but as it is repo scope maybe the decision is better that we just manage this by hand. I do not have access to repo settings or secrets on the fluxcd/stats repo, so I can't check on the secret or replace it.
I can copy all this to my own fork and test with a different repo, (just so we have someone that's fully aware of what this repo scope token does and having tested in a scope other than the production setting) but if it's just an expired token, then it will just need to be replaced, and someone else aware that we are doing this I guess.

It seems like we could automate this, updating the github secret with repo scope.
We could run a periodic workflow (hourly? every 4h/12h?) on the stats repo, where we'll store a Github App private key as a secret, that has permission to create GITHUB_TOKEN with repo scope for the fluxcd/flux2 and fluxcd/flagger repos. That job will store the new GITHUB_TOKEN – or maybe there is a way to handle this without managing secrets, and only using ambient / delegated credentials. I'll admit, I'm looking for an excuse to use the tf-controller here, (I will probably do it both ways, but only one will go into the permanent situation for fluxcd/stats.)
I will work out on my own personal repos how to do a cross-repo delegation of "write secrets" permission for GitHub Actions, so the fluxcd/flux2 . I am pretty sure we don't need Kubernetes or tf-controller in the workflow to accomplish this (it could just run the github-app-secret app and terraform in an hourly job, with no Kubernetes and just using terraform CLI.)
I was looking for an excuse to use tf-controller, but if there's no permanent Flux cluster I can run this workflow on, it might be better to keep it fully GHA. Then again, we could be spinning up a k8s cluster for tf-controller to run a job on, to completion, and make that another separate pattern that we use for demos and examples.
Any thoughts before I go ahead and implement such a process? (Does this sound like a good idea at all?)
@stefanprodan

Reference also:
https://github.com/fluxcd-community/github-app-secret
We have done some work on this problem already, in the fluxcd-community umbrella

Got 'em
BOT_GITHUB_TOKEN is the secret, (fixed in https://github.com/fluxcd/stats/commit/18067ded18f060abe3cb68b737d0184dcdc568f7)


Test not passing on MacBook Pro

macOS Monterey Version 12.3.1
MacBook Pro (Retina, 15-inch, Mid 2015)
Processor 2.8 GHz Quad-Core Intel Core i7
Julia v1.7.2
Test failed for Photodynamics.jl package running on the main branch:
     Testing Running tests...
Integration Methods... Done.
Impact Parameter... Done.
Lightcurve...   0.012285 seconds (83.17 k allocations: 12.065 MiB)
  0.024574 seconds (83.19 k allocations: 12.068 MiB, 47.56% gc time)
  0.004223 seconds (5.69 k allocations: 606.454 KiB, 78.28% compilation time)
  0.003783 seconds (5.71 k allocations: 608.688 KiB, 74.96% compilation time)
  0.001978 seconds (5.77 k allocations: 619.859 KiB)
  0.001894 seconds (5.80 k allocations: 622.703 KiB)
Lightcurve: Test Failed at /Users/ericagol/Software/Photodynamics.jl/test/test_compute_lightcurve.jl:53
  Expression: isapprox_maxabs(dlc_p.flux, lc_p.flux)
Stacktrace:
 [1] macro expansion
   @ /usr/local/Cellar/julia/1.7.2/share/julia/stdlib/v1.7/Test/src/Test.jl:445 [inlined]
 [2] macro expansion
   @ ~/Software/Photodynamics.jl/test/test_compute_lightcurve.jl:53 [inlined]
 [3] macro expansion
   @ /usr/local/Cellar/julia/1.7.2/share/julia/stdlib/v1.7/Test/src/Test.jl:1283 [inlined]
 [4] top-level scope
   @ ~/Software/Photodynamics.jl/test/test_compute_lightcurve.jl:4
Lightcurve: Test Failed at /Users/ericagol/Software/Photodynamics.jl/test/test_compute_lightcurve.jl:54
  Expression: isapprox_maxabs(dlc_c.flux, lc_c.flux)
Stacktrace:
 [1] macro expansion
   @ /usr/local/Cellar/julia/1.7.2/share/julia/stdlib/v1.7/Test/src/Test.jl:445 [inlined]
 [2] macro expansion
   @ ~/Software/Photodynamics.jl/test/test_compute_lightcurve.jl:54 [inlined]
 [3] macro expansion
   @ /usr/local/Cellar/julia/1.7.2/share/julia/stdlib/v1.7/Test/src/Test.jl:1283 [inlined]
 [4] top-level scope
   @ ~/Software/Photodynamics.jl/test/test_compute_lightcurve.jl:4
Test Summary:           | Pass  Fail  Total
Photodynamics.jl        |   65     2     67
  Integration Methods   |    4            4
  Impact Parameter      |   30           30
  Lightcurve            |   31     2     33
    Integrated Flux     |   14           14
    Non-Integrated Flux |   14           14
    Lightcurve          |    2     2      4
ERROR: LoadError: Some tests did not pass: 65 passed, 2 failed, 0 errored, 0 broken.
in expression starting at /Users/ericagol/Software/Photodynamics.jl/test/runtests.jl:6
ERROR: Package Photodynamics errored during testing

(Photodynamics) pkg> st
     Project Photodynamics v0.1.0
      Status `~/Software/Photodynamics.jl/Project.toml`
  [01eff828] Limbdark v0.1.0
  [69f646f0] NbodyGradient v0.1.3
  [189a3867] Reexport v1.2.2
  [f2b01f46] Roots v2.0.1
  [276daf66] SpecialFunctions v1.8.6
  [90137ffa] StaticArrays v1.4.7
  [37e2e46d] LinearAlgebra

This is perplexing since the tests pass for MacOS.  The difference is in the light curves with
and without derivatives computed. @langfzac thinks this is due to a difference in the
Limbdark.jl package, and may be a result of architecture-dependent differences in the
computation of special functions (perhaps logGamma?).

Hi @langfzac -
If there is some issue with Limbdark.jl on the MacBook Pro machine causing the tests to fail, one way we could track this down would be to save & compare the individual values of the flux evaluated at each instant of time (rather than compare time-integrated fluxes).  Since the time-integrated values have a 10^{-8} discrepancy, then we should see an even larger discrepancy for a subset of the individual values.
-Eric

@ericagol #22 adds the non-integrated exposure version to the tests, and outputs the maxabs of the differences.

I'm thinking of adding a debugging option in which the flux is computed with & without derivatives (in the gradient case), and flagging individual flux points which differ by more than, say, 1e-8.

@ericagol Can you paste the output of versioninfo() here? I'm wondering if there's a difference in LLVM versions or something

Yes:
julia> versioninfo()
Julia Version 1.7.2
Commit bf53498635 (2022-02-06 15:21 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin21.3.0)
  CPU: Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-13.0.1 (ORCJIT, haswell)

This came down to this issue with Limbdark.jl:
https://github.com/rodluger/Limbdark.jl/issues/94
When the gradient & non-gradient expressions were made to match, the
Photodynamics.jl no longer gets tripped up by the b=1-r example we
tried.   The Limbdark.jl tests pass, and the Photodynamics.jl tests pass on
the MacBook described above.

With this fix, updated Limbdark.jl to v0.1.1.  This should allow tests to pass on MacBook.


Yii2 Datepicker error display issue

i have issue while using yii2 datepicker widget
i am using
In View
 $modeljob,
        'attribute' => 'due_date',
        'options' =>['class'=>'form-control'],
       //'language' => 'ru',
      //'dateFormat' => 'yyyy-MM-dd',
     ]);?>
in Model
[['job_no','status_id','due_date'], 'required'],
while create new job validation error not displaying on page (red and green)

Could you provide a screenshot?

i try hard but not success...:)
when i hit submit button its return to create action and don't show validation error hence user confuse what is going wrong...

You are calling widget without ActiveForm. The following should work as expected:
$form->field($modeljob, 'due_date')->widget(DatePicker::className(), [
        'model' => $modeljob,
        'attribute' => 'due_date',
        'options' =>['class'=>'form-control'],
        //'language' => 'ru',
       //'dateFormat' => 'yyyy-MM-dd',
]);

ohhh my god!!!!!
sorry for my mistake....
Its amaging
thanks a lot @ SilverFire


feat: embed auto-generated card list to docs landing pages [HDH-110]

used Feature Flags Docs landing page as a sample
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

@wei-harness will review.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@ravilach I really don't understand what this PR is doing. Is there anything specific you want me to check? I see it has something to do with landing pages, but I don't understand any details of any of these file changes. I'm curious about the commented block in sidebars.js. What is that for?

@StrawberryPad was more of a prototype on how something could look on a landing page. @wei-harness can stash the FF changes and implement the card only changes, most likely will close this PR and raise another, keeping this branch for a later date.

@wei-harness this was already merged in, in a different PR. Closing.


Fix some hardcoded scala versions

The Scala version used in MillBuildBootstrapModule to compile the build.sc was hardcoded to 2.13.10. Instead, we want to always use the same version we used to build that Mill version.
As an side effect, Mill is now compatible with new Java 21, due to the use of a newer Scala version.

Fix #2769


Loss decreases fast but alignment cannot seem to coverage

After about 5k step, loss have coveraged to a low level. However, when I visualize the alignment results, it seems that the attention mechanism focus on the same encoder timestep and just not move on.

What are the characteristics of your training data? Also, can you attach the train.log file? It will contain the inputs used for generating the samples. Perhaps there may be an issue in how your input text is being converted to symbols?


Update mock-responses.md

Fix returning binary data

@garrytrinder
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@waldekmastykarz,
PRMerger will accept sign-off from the listed author only. However, if you are a Microsoft employee authorized to update this content, you can send a request to<EMAIL_ADDRESS>for this PR to be merged. Please include your role and an explanation of why the PR should be merged without approval from the listed author.

#sign-off


Too much logging

graphite-api's got logging now, which is great, but now it's always logging at the debug level, and there doesn't seem to be a setting right now to adjust that. It did remind me that that needs to be added to logrotate, but we should also have a setting to adjust the logging level there. Already it's getting kind of out of hand.

just need to add the following to /etc/graphite-api.yaml
logging:
  version: 1
  handlers:
    raw:
      level: INFO
      class: logging.StreamHandler
      formatter: raw
  loggers:
    root:
      handlers:
        - raw
      level: INFO
      propagate: false
    graphite_api:
      handlers:
        - raw
      level: INFO

Added log_level support to raintank-stack chef recipe. pending merge...
see: https://github.com/raintank/chef-raintank-stack/pull/4

Brought in on Friday.


Cannot run application after eject

Description
After running npm run eject de android project cannot resolve ReactApplication and ReactNativeHost and it does not compile.
Expected Behavior
I can compile the android project generated by npm run eject and execute it in the androi emulator
Observed Behavior
The project has some errors in the imports, like ReactApplication or ReactNativeHost. The MainActivity does not implement the method getUseDeveloperSupport() that is required as it extends ReactActivity
Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts: empty
npm ls react-native<EMAIL_ADDRESS>npm ls expo: empty
node -v: v8.11.3
npm -v: 5.6.0
yarn --version: 1.9.4
watchman version: The term 'watchman' is not recognized as the name of a cmdlet

Also specify:

Operating system: Microsoft Windows 10 Enterprise
Phone/emulator/simulator & version: Android emulator, API 27, Android 8.1 (Google APIs)

Reproducible Demo
$ npm install -g expo-cli
$ expo init my-app
$ cd my-app/
$ npm run eject

Then open Android Studio and try to compile the Android project generated after npm eject

Same issue, I tried to not use expo CLI and use :
react-native eject
(with defining app.json)
and I have the same issue

Same issue using both new syntax (expo cli) and old

I think it's because there is no index.js file after eject. Still, there is no update to .gitignore after eject too.

This issue still exists!


Chains does not work on GCP if Metadata Services are disabled

Expected Behavior
On a cluster deployed on GCP, the Tekton Chains controller should work properly and sign artifacts as per configuration.
Actual Behavior
On a cluster deployed on GCP, the Tekton Chains controller halts execution if the cluster is running on GCP and Metadata Services are not enabled.
Steps to Reproduce the Problem

Grab a hold of cluster on GCP
Do not enable Metadata Services
Create a TaskRun, make sure it finishes successfully

Notice how the controller does not produce any sort of log messages to indicate it's processing the finalized TaskRun.
Additional Info

Kubernetes version:

🐚 kubectl version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.0", GitCommit:"cb303e613a121a29364f75cc67d3d580833a7479", GitTreeState:"archive", BuildDate:"2021-07-22T00:00:00Z", GoVersion:"go1.16.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5+1f952b3", GitCommit:"3c28e7a79b58e78b4c1dc1ab7e5f6c6c2d3aedd3", GitTreeState:"clean", BuildDate:"2022-03-27T05:21:18Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.21) and server (1.23) exceeds the supported minor version skew of +/-1

Tekton Pipeline version:

🐚 tkn version
Client version: 0.21.0
Pipeline version: v0.28.3
Triggers version: v0.16.1

OpenShift version:

 🐚 oc version
Client Version: 4.8.0-202201210133.p0.g88e7eba.assembly.stream-88e7eba
Server Version: 4.10.0-0.nightly-2022-03-29-163038
Kubernetes Version: v1.23.5+1f952b3

The root cause of this issue appears to be an outdated version of the google/go-containerregistry library. See details.
Updating google/go-containerregistry should fix the issue. I have not been successful in doing this update.

hey @lcarva thanks for opening this bug. https://github.com/tektoncd/chains/pull/404/files includes an upgrade to go-containerregistry which should include the commit you pointed out

I can verify that using those changes the issue reported here goes away. Thanks!


Convert all vimeo videos to attribution non-derivative

I think I did them all; if not, I will fix them as I see them.


add SSL_get0_iana_groups() & SSL_client_hello_get_raw_extensions()

The two functions allow user can get groups and extensions without memory
allocations. So we could calculate the ssl fingerprint[1] in low cost[2].
[1] https://engineering.salesforce.com/tls-fingerprinting-with-ja3-and-ja3s-247362855967
[2] https://github.com/phuslu/nginx-ssl-fingerprint

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 61 days ago

fixed doc-nits error and rebased to master

There is one last issue with the commit - it cannot have the CLA: trivial in the commit message. Please remove that. If you do not have CLA yet, we will need it as this contribution is definitely not trivial.

Removed CLA: trivial and Sent the signed CLA pdf from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Thanks.

ICLA processed.

This pull request is ready to merge

Merged to master branch. Thank you for your contribution.

A belated thank-you to @phuslu for the last round of updates and to @t8m for merging the change.
I was otherwise occupied for most of this week...


Remove solaredge_local duplicate code

Description:
Related issue (if applicable): fixes #25864
Example entry for configuration.yaml (if applicable):
Not changed but:
sensor:
  platform: solaredge_local
  ip_address: IP_ADDRESS

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Can be merged when build passes.

Please run black in the project root and fix the lint issues.
black --fast homeassistant tests

The updater component seems to have a flaky test.

Please rebase on latest dev branch to let the build pass.

Rebased :)


Migrate image APIs to ciao and cleanup

Migrate the image APIs from openstack based APIs with many unused fields and structures to the ciao http endpoint. Currently this still used the ciao-image boltdb based datastore and infrastructure but in a future PR I will merge that code into controller and migrate to the sqlite database.
See: #1451

Coverage increased (+0.1%) to 65.936% when pulling b2bc0abf6ad5c0e7c4e8cbfd0d7548b49d71ed8f on rbradford:image-service-cleanup into cd1430c0f4b7fd853e20d765e847f940675e3d36 on ciao-project:master.


Format everything

clang-format for C/C++, black for Python

I suggest that, as a part of this refactor, we include a small section either on Development, or as a new Contribuiting section, on the code format we follow and maybe some examples on how to enforce it (i.e. integrate it with different text editors).
PS: unsure if it's worth it if we include a style phase in the CI pipeline.

I think the CI check and a little note in development.md to say that we use clang-format and black should do the trick. They're both sufficiently well known tools that people can work out how to integrate them into their own environments.
This can be done as part of the PR to reformat the repo.

Just checking-in here to remind ourselves to use the updated run_clang_format.sh with the newline trailing (I don't know in which circumstances we could forget about it, but anyway).


Add external block icon to pulldasher

In https://github.com/iFixit/pulldasher/issues/87 we decided to add an external_block label to pulls that require input or QA from someone outside of our department. I created the external_block label:

But now we need to implement the icon in pulldasher.
I was thinking we would use a red eye with a slash. Like so:

This mimics the QAing eye that we already have, which is green if someone else is QAing and blue if the viewer is QAing. I made it red with a slash to show that QA (by someone on the QA squad) is currently not possible.

is using a label where we landed? I think there was some confusion if the external_block "label" meant an actual GH label or if we "label" the pull like we do with dev_block.
cc @danielbeardsley @danielj41 @addison-grant

I think we landed on an actual GH label.

I thought that we wanted to use a GH label since the behavior is more in line with adding the QAing label (which just adds an icon to the pull), then commenting dev_block (which actually moves the pull into a separate column).

Hmm now that I think about it, having it be a text label like Dom mentioned is more in line to our other labels but meh, i dont have strong feelings either way

@BaseInfinity I think it's less in line with our other labels, because it behaves really differently.
Other text labels like dev_block or deploy_block move it into a different column. Which we didn't want to do here.
This behaves almost exactly the same as the QAing label. Which is a standard github label.

Ya I guess it's just weird that pulldasher uses a mix GH labels and text labels, though i like the idea that GH Labels seem to represent a state of the pull. i guess im cool with it as long as we document somewhere in the README since we plan on open sourcing this

since we plan on open sourcing this
Plan?

well, we plan on bloggin about it and officially announcing it to ze world

I think the github labels are fine for this.

https://medium.com/chris-opperwall/dockerizing-pulldasher-69af687b1a0c


How to use calendar-heatmap component with other kinds of data

Thank you for this great widget. I was wondering how one could use this heatmap with other kinds of data. For example I would like to use it display unemployment chronology similarly to bokeh plot here: https://bokeh.pydata.org/en/latest/docs/gallery/unemployment.html

Hi, glad you like it :) Nice example too, similar use case!
The heatmap can be used if the unemployment data has timestamps with values (e.g. date in iso and value representing unemployment rate on that date). This would lead to the default calendar heatmap as you can see in the demo.
The 'name' parameter can be used to show different unemployment rates based on country, or market sector, or anything else.
This calendar-heatmap was originally used in a project where data values are presented in hours/minutes/seconds - but that is just the presentation of it. If you need to present unemployment data in other units, feel free to fork and adjust as necessary.

Thank you for your ideas!

No problem at all :) If you end up tinkering with it and need any help let me know


Add performance data to domain events

This PR comprises a number of changes to domain events to record information needed when processing to generate metrics for the performance hub. See notes from meeting of Dec 14th 2022.
NB: This PR causes breaking changes to the integrations which have been written to consume these events and update Delius. An accompanying PR is required to adjust those integrations.
Changes
application.submitted
Adds:
    "mappaCategories": "M2",
    "sentenceLengthInMonths": 57,
    "offenceDescription": "Wounding or inflicting grievous bodily harm (inflicting bodily injury with or without weapon) (S20) - 00801",
    "releaseType": "Release on Temporary Licence (ROTL)",
    "age": 43,
    "gender": "Male",
    "targetLocation": "LS2", // postcode district rather than placename

Reworks submittedBy key to include geographic information as well as staff info:
    "submittedBy": {
      "staffMember": {
        "staffCode": "N54A999",
        "staffIdentifier":<PHONE_NUMBER>,
        "forenames": "John",
        "surname": "Smith",
        "username": "JohnSmithNPS"
      },
      "probationArea": {
        "code": "N02",
        "name": "NPS North East"
      },
      "cru": {
        "code": "N02",
        "name": "NPS North East"
      },
      "team": {
        "code": "N54NGH",
        "name": "Gateshead 1"
      },
      "ldu": {
        "code": "N54PPU",
        "name": "Public Protection NE"
      },
      "region": {
        "code": "NE",
        "name": "North East"
      }
    }

application.assessed
Reworks assessedBy key to include CRU information as well as staff info:
"assessedBy": {
      "staffMember": {
        "staffCode": "N54A999",
        "staffIdentifier":<PHONE_NUMBER>,
        "forenames": "John",
        "surname": "Smith",
        "username": "JohnSmithNPS"
      },
      "probationArea": {
        "code": "N02",
        "name": "NPS North East"
      },
      "cru": {
        "code": "N02",
        "name": "NPS North East"
      }
    },

booking.made
Reworks bookedBy key to include CRU information as well as staff info:
    "bookedBy": {
      "staffMember": {
        "staffCode": "N54A999",
        "staffIdentifier":<PHONE_NUMBER>,
        "forenames": "John",
        "surname": "Smith",
        "username": "JohnSmithNPS"
      },
      "cru": {
        "code": "N02",
        "name": "NPS North East"
      }
    },

booking.not-made
Reworks attemptedBy key to include CRU information as well as staff info:
  "attemptedBy": {
      "staffMember": {
        "staffCode": "N54A999",
        "staffIdentifier":<PHONE_NUMBER>,
        "forenames": "John",
        "surname": "Smith",
        "username": "JohnSmithNPS"
      },
      "cru": {
        "code": "N02",
        "name": "NPS North East"
      }
    },

The breaking changes to the integrations repo which this PR introduces have been fixed up in https://github.com/ministryofjustice/hmpps-probation-integration-services/pull/680 so this ready to merge as an agreed design.


Automate Releases to PyPI: grep command for latest version not working correctly

The problem
Retrieving the latest_previous_version in the release.yaml from the 'GitHub Actions for Python Packages: How to Automate Releases to PyPi' code with the grep command will retrieve the first version released and not the latest.
https://github.com/ArjanCodes/examples/blob/0b8c8ab74908be5ed9239fcdaed875548d3f595c/2024/publish_pypi/release.yaml#L51
Minimal example
I'm testing this with my own package f3dasm:
response=$(curl -s https://pypi.org/pypi/f3dasm/json || echo "{}")
latest_previous_version=$(echo $response | grep -oP '"releases":\{"\K[^"]+' | sort -rV | head -n 1)
echo $latest_previous_version

Will yield 0.0.1, which is the first version released and not the latest (1.5.5 at the time of writing). See here
Possible solution
I changed the grep line to the following in my own code:
latest_previous_version=$(echo $response | awk '/"releases":/ {flag=1} flag' | grep -oP '"\d+\.\d+(\.\d+)*"' | tr -d '"' | sort -rV | head -n 1)

I'm no expert and this; this searches for every "x.x.x" string (where x = digit) after "releases": has been found

Thanks for the issue! Would you like to contribute to the repo? If so, you will be mentioned as an contributor :)


Data Compression Bug - GZipDecompress

The GZip decompression do not work the same in BC 22 as it's been working in BC 21. In previous version it would decompress file that was compressed using deflate (FlateDecode) algorithm and now it fails.
Minimal repro steps:

Decompression breaks in codeunit 421 “Data Compression Impl.”, line 41.

The error that is shown:

There is a theory, it might've happened because .NET is now running on 6.0.
Here  a file that was decompressed without a problem in BC 21.6, but fails in BC 22 can be found.

@UrbMigle Thanks for reporting this.
Have you created a support case for this issue, too?
The github team has limited capacity and if a customer is impacted by this issue you should created a support case to get this prioritised.
It's always easier if you provide your minimal example as text instead of images, then this can be checked more easily.

Support case is still in progress, waiting for support advocate to contact me.
Ok, I'll have in mind in the future to provide examples as text :)

@UrbMigle : Do you have any updates? We are using GZipDecompress to deflate PDF ObjectStreams - this functionality is broken with BC22...

@donmikele75  Unfortunately not. Support case is not getting any attention as well :(
We were using this functionality for the same exact purpose :)

We'll take a look. We've got an open support case for this.

Any news?

We finally found some time to look into this case, and indeed this turns out to be a side-effect of the upgrade to .NET core. The data you are trying to pass is not GZIP, it is ZLib (0x68 0xDE header). Hence, if you use the IsGZIP check in Data Compression you will see that it (correctly) returns false. In the .NET framework, it must have been more relaxed around what it supported, but now it is strict.
That is of course unfortunate, but the code now works as it was designed to work. To decode this, you will need to use ZLibStream instead. You would need new methods for ZLib (and maybe also for raw DeflateStream).
We would of course accept a PR adding ZLib capabilities in the Compression module.


How to run action without closing current fzf-lua window?

          > > @ibhagwan How to use `fzf_lua.fzf_exec` inside `setup.buffers.actions`? Or this is not recommended?

No problem, you can call whatever you want from the actions, it will override the existing fzf-lua interface, just make sure to wrap the function inside a table to prevent the “flash” as it tells fzf-lua to reuse the interface rather than close and reopen.

I am trying to bind ['ctrl-m'] = { function() end } in files actions, but when i presss 'ctrl-m', the fzf-window shows process exit 0. (window is not closed).
i am trying to integrate flash.nvim.
Originally posted by @towry in https://github.com/ibhagwan/fzf-lua/issues/499#issuecomment-1879977737

more context:
I use {} because i'd like fzf window do not close and use flash.nvim to quickly jump to a selection.
So i don't have to use  or enter more keys to narrow down the filter to select.

My currently almost work attempt is:

create after/ftplugin/fzf.lua and bind buffer local to mode 't' with key <c-a>.
in the keymap callback run:

  require('flash').jump({
    pattern = '^',
    label = { after = { 0, 0 } },
    search = {
      mode = 'search',
      exclude = {
        function(win)
          return false
        end,
      },
    },
    action = function(match)
      vim.print(match)
      -- local picker = require('telescope.actions.state').get_current_picker(prompt_bufnr)
      -- picker:set_selection(match.pos[1] - 1)
    end,
  })

The result is:

So, i need to:

find the list window id, exclude other windows in exclude filter fn.
in action, move cursor line to the match.

I’m not sure if I understand correctly what you’re trying to do, are you trying to have an action navigate within the fzf-lua selection?
If so, wrapping the action with {} won’t help as this just prevents from the float windows being closed but the fzf process is already dead (and its selections too), if you want the action to be executed while the fzf process is still running you need to use { fn = function() … end, exec_silent = true } (this gets converted to fzf’s execute-silent action), similar to how the “copy commit hash” function is defined:
https://github.com/ibhagwan/fzf-lua/blob/5890b84b07f842da292ee9067b7363ac2af02d92/lua/fzf-lua/defaults.lua#L276

Sorry for the my bad english. The original question is addressed.
The current question is, how to navigate(, ) selection item in action.

In the above picture, if i press j, I want to move cursor to the line highlighted with char j, in the assumption that I already know the line number and col number of line j.

You can’t use nvim_win_set_cursor with buffer=0 (current buffer) as I’m not sure which buffer context the action is called under, iterate neovim windows, find which one is a terminal buffer and use that buffer number with nvim_win_set_cursor.

use that buffer number with nvim_win_set_cursor.

But nvim_win_set_cursor use window id, not buffer id, I am a little confused.

use that buffer number with nvim_win_set_cursor.

But nvim_win_set_cursor use window id, not buffer id, I am a little confused.

You’re correct, find the correct win id then :)

I already have the correct win id.

You can use require'fzf-lua'.utils.fzf_winobj().fzf_winid to get FzfWin’s class win id.

I already have the correct win id.

I already have the correct win id.

Not sure what this quote means, did my last comment solve your issue?

No, because I already have correct win id, and then you suggest me

You can use require'fzf-lua'.utils.fzf_winobj().fzf_winid to get FzfWin’s class win id.

But in my understanding it seems not possible because fzf main window is terminal window that runs fzf inside it, the nvim_win_set_cursor api can not change the fzf selection item.
And I don't want to occupy too much of your time so closing this issue for now until i have more clues.


BTC to USD converter

Student, group: Gasilov Dmitry, 90203
URL: https://github.com/KoTTiG/BTC-Converter
Git approach: Git-Flow

`+


wiki changes: how to fork wiki and submit a PR against it

I cannot seem to figure out how you can fork the wiki and submit a PR with my changes. I can clone the wiki and can manually create a remote on GH but I don't see how GH lets me submit a PR against a non-code branch (the wiki).
Idea? Process?

I also hit this and gave up because it wasn't worth the effort for the change.

removed the footer.


allow ondblclick for works.ejs

Hey @Haojen, here's a quick fix for something I came across -
Problem:
Double clicking the nav bar on a works.ejs layout page doesn't scroll to the top.  You can see it happen on the demo.
Reason:
A property error posts when using the works.ejs layout (blog.js:79 Uncaught TypeError: Cannot read property 'offsetHeight' of null).  works.ejs does not use the css class intro-header which is used as a selector by blog.js on line 79.  This prevents the double click scroll event from attaching, because it is attached in the next code block.
Fix:
Wrapping the entire post navigation function body in the isPages('data-ispost') conditional prevents the querySelector() call until needed.

@slmoore  thank u very much  👍

@Haojen glad I could help, nice theme!


Sass configuration: adding output group

It may be useful to create more than one output group.
Here's an example:

module.exports = {
  sass: {
    observe: '**/*.{scss,sass}',

    entries: [
      // group A
      //  -> desired output: assets/css/FILENAME.css
      'assets/{scss,sass}/*.{scss,sass}',

      // group B
      //  -> desired output: assets/css/components/FILENAME.css
      'views/blocks/**/*.{scss,sass}',
      'views/includes/**/*.{scss,sass}',

      // group C
      //  -> desired output: assets/css/pages/FILENAME.css
      'views/pages/**/*.{scss,sass}',
    ],

    output: 'assets/css'
  }
}

It's ok with forge 2.1.0 : https://github.com/wide/forge/releases/tag/2.1.0
Example :
targets: ['sass_global', 'sass_pages', 'sass_components' ],
sass_global: {
    compiler: 'sass',
    flatten: true,
    observe: 'assets/scss/*.scss',
    entries: [
      'assets/scss/*.scss',
    ],
    output: 'assets/css'
  },
  sass_components: {
    compiler: 'sass',
    flatten: true,
    observe: 'views/{blocks,includes}/**/*.scss',
    entries: [
      'views/blocks/**/*.scss',
      'views/includes/**/*.scss',
    ],
    output: 'assets/css/components'
  },
  sass_pages: {
    compiler: 'sass',
    flatten: true,
    observe: 'views/pages/**/*.scss',
    entries: [
      'views/pages/**/*.scss',
    ],
    output: 'assets/css/pages'
  },


Postgresql yum add

This should add the postgresql repo for yum based distros.

I finally figured out what was wrong....
Try this ;)
IFS=$'\n'; wget_args="-nH -np -r -A "*.rpm""; set -f; wget ${wget_args} http://download.postgresql.org/pub/repos/yum/10/fedora/fedora-25-x86_64/; set +f; unset IFS
It turns out that when we set $IFS for the for loop to read each line in the repo_filter.conf file so that it will read to the end of the line instead of to the first whitespace encountered, that messes up wget parsing it's own arguments.  So I will rewrite the original for loop to read the contents of repo_filter.conf into an array so that $IFS can be reset before getting to wget.

You won :P  Merge away.

The args aggregation is starting to get a little convoluted with the addition all the new conditional arguments. I'll attempt to clean it up later. Also, just FYSA, here is a more streamline way to do your array population:
 root@dt:/home/codz# > test.txt
root@dt:/home/codz# for i in {0..10}; do echo "$i $i" >> test.txt; done
root@dt:/home/codz# cat test.txt
0 0
1 1
2 2
3 3
4 4
5 5
6 6
7 7
8 8
9 9
10 10
root@dt:/home/codz# while IFS='\n' read -r line; do array+=( "${line}" ); done < test.txt
root@dt:/home/codz# echo ${array[1]}
1 1

$IFS resets automagically upon read exit.


Remove keys from repository

Currently we check in our test keys to the repository. But they should be created at startup of the test bed.

This could be implemented with the pull request #129, creation of certificates at startup of the testbed.


chore(Algebra/Order/GroupWithZero/Unbundled): remove useless lemmas, use ZeroLEOneClass

There are still some useless lemmas that were simply ported from Algebra.Order.Monoid.Lemmas, such as just chain an existing lemma with an assumption and lemmas whose assumptions imply 1 ≤ 0. This PR removes them.
Also, some lemmas have both assumptions like 1 < a 0 < a. This PR uses ZeroLEOneClass to remove redundant assumptions.

Ported from https://github.com/leanprover-community/mathlib/pull/16525 and https://github.com/leanprover-community/mathlib/pull/18158

Could you please deprecate them instead?


deletion not covered properly

[Enter feedback here]
IDK why, but I'm being told we're at our 20 IP limit, when I click on Public IPs, it listed 8...I deleted 2, and it still says we're at the limit of 20.  so there must be a step after the deletion instructions given here to make them truly free.

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

ID: 19d6d70c-eed7-8fdc-e04b-995cabf0a902
Version Independent ID: a33391e9-3023-8212-6246-d64071efc2b5
Content: Create, change, or delete an Azure public IP address - Azure Virtual Network
Content Source: articles/virtual-network/virtual-network-public-ip-address.md
Service: virtual-network
Sub-service: ip-services
GitHub Login: @asudbring
Microsoft Alias: allensu

Thanks for brining this to our attention. We will investigate it further and update you shortly.

@stevesmoot , there is no particular step post deletion of Public IP on Azure portal that would do a cleanup. The only requirement is that the public IP object shouldn't be associated to any IP configuration or virtual machine network interface.
But sometimes, it may happen that the IP resource is still being referenced by another resource in the backend due to any internal error and this would cause the Public IP limit being hit despite the less number of IPs available in portal.
I would suggest you to follow the below steps to validate if there are any stale Public IPs in the backend:

Open a web browser and navigate to https://resources.azure.com

Log-in to the appropriate Azure account

In the blade on the left side of the browser, expand subscriptions, then expand the desired subscription name

Under your subscription, expand providers

Under providers, expand Microsoft.Network

Under Microsoft.Network, click on PublicIPAddresses
And check how many IP addresses you can see there. If they match with your Azure portal or you find any stale Public IP addresses which were already deleted from the portal. In case you find any stale Public IP addresses, please note their resource ID to find the resource group they are in and then follow the below steps:

In the blade on the left side of the browser under your subscription, expand resource groups

Under resource groups, expand the proper Resource Group Name where you found the stale Public IP

Under the Resource Group, expand providers

Under providers, expand Microsoft.Network

Under Microsoft.Network, expand PublicIPAddresses

Near the top of the content window of the browser, select the Read/Write push button

Select the Actions(POST, DELETE) tab and click on Delete.

Repeat the same for any other stale Public IP that you may have found.

Once these PublicIPAddresses is deleted, you can check in your Azure portal if you are able to create a new one.
If it does not succeed and you receive an error or you do not find any stale Public IPs in resources.azure.com, it may require deeper investigation. Hence, if you have a support plan, I request you to file a support ticket, else please do let us know, we will try and help you get a one-time free technical support. In this case, could you send an email to<EMAIL_ADDRESS>referencing this thread as well as your subscription ID. Please mention "ATTN gishar" in the subject field.


Can not update product thumbnail

If we create a product without thumbnail, then we unable to update it. The new media item is created, but it doesn't update the media id to product table.

after upgrade EF to RTM, this bug has gone


get_started: add description of all etna modules

 Issue by iKintosh
Wednesday Feb 02, 2022 at 17:21 GMT
Originally opened as https://github.com/tinkoff-ai/etna/issues/495

🚀 Feature Request
This way we can describe the concept of how etna package is organised.
Motivation
It may be not clear for the users where to look for etna modules.
Proposal
Look at the https://github.com/shazow/unstdlib.py#organization--philosophy and adapt it to etna library.
Only we write it in get_started notebook.
Test cases
No response
Alternatives
Create documentation page.
Additional context
No response
Checklist

[ ] I discussed this issue with ETNA Team

I think it isn't relevant with a new documentation.


Decouple the openwebui and the ollama. in inference-cpp-xpu dockerfile

Description

Following the suggestion on https://github.com/intel-analytics/ipex-llm/issues/12364, remove the openwebui start script in ipex-llm-inference-cpp-xpu docker image.
Change the openwebui document of inference-cpp.
Have tested.

Need to update https://github.com/intel-analytics/ipex-llm/blob/main/README.md#docker and https://github.com/intel-analytics/ipex-llm/blob/main/README.zh-CN.md#docker too

Have updated document in https://github.com/intel-analytics/ipex-llm/blob/main/README.md#docker, which refer to docs/mddocs/DockerGuides/docker_cpp_xpu_quickstart.md

Have updated document in main/README.md#docker, which refer to docs/mddocs/DockerGuides/docker_cpp_xpu_quickstart.md

Have updated document in main/README.md#docker, which refer to docs/mddocs/DockerGuides/docker_cpp_xpu_quickstart.md

I removed the OpenWebUI part of the docker image and only kept ollama; and the linked document mentions how to use OpenWebUI as the front end to connect to ollama. Now I also remove 'OpenWebUI' in the readme.md and readme.zh-CN.md.


Unexpected warning for "$('.class')", for multiple elements

Hi,
in Protractor $() returns one element while $$() returns multiple elements. I would expect $() to work like $$().first(), however it warns me about more than one element found for locator. I think this is a little bit unexpected, because $() doesn't just expects IDs. It's normal to pass generic selectors like $('.class').

This warning gives you heads up that there are more elements matching your selector, so your test may be unpredictable - when you wrote $$().first() you do it deliberately, so no warning there.
So, in my opinion warning is expected and desired, unless there is a function that throws error when more then one matching element is found.

Yes, when you use $, it means, give me the element that matches a selector, not give me the first element that matches. That could mean $().first(), $().last(), or $().get(n) when there are multiple matches.

Despite the warning all $() does is $$().first() or not? I think by not using an ID, but a selector like p or .common-class my intention to get one element out of multiple elements would be clear enough. If the user should really be prevented to query one element out of multiple elements $() should only accepts IDs (imo).


[Feature Request] Ability to change some colors

Sometimes when I render Image, it goes wrong color, but I completely understand that I can change it to another and make image better. This will be much easier to change color in program, that changing it by editor

So you want an interface to change which 16 colors it's chosen?

Yes. Something like to make
pink -> red or pink -> lightBlue


Metamist Comments Backend

Implementation of comments for metamist.
See RFC for details: https://docs.google.com/document/d/1S_LFxLHfVMGPL7mJ3O6ozPjSDWMkWtlk5QX0wIcOAq4/edit?usp=sharing

Do we need to DELETE FROM these tables in db/python/tables/project.py's delete_project_data()?

Do we need to DELETE FROM these tables in db/python/tables/project.py's delete_project_data()?

Ooh that's a good point, I think we might need to rethink those project deletes a bit – I suspect it is currently broken as we missed that in the first class files PR too, so the analysis_outputs table FK would be breaking deletes.
That probably also means that our test coverage for that route isn't good. I'll add a issue to the tracker to come back around and see if we can improve tests and make it a bit harder to miss adding tables to the project deletes.


fix(workflow): paths-filter error on push event

Resolved https://github.com/opensearch-project/opensearch-dashboards-functional-test/issues/993
With this change, the workflow triggered by pull_request event will use GitHub REST API to fetch a list of modified files which is the same as it is now.
However, for workflow triggered by push event, the changes are detected against the most recent commit on the same branch before the push
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This can be replaced with https://github.com/opensearch-project/opensearch-build/blob/main/.github/workflows/manifests.yml#L6-L9 I think.

Nice! Thanks for the suggestion, will try that github flow builtin solution


Support unsigned versions of integers.

Unsigned versions can be automatically mapped to the signed versions that database uses.

Lossy - definitely not. A runtime bound check should be performed

Since it just bit me, for varints columns I think it should Just Work

I think that this idea may backfire on us later. Lack of unsigned types in Scylla / C* is of course an issue for some application developers because it increases complexity (as the developer now needs to validate data and deal with errors).
This complexity is however inherent to developing with Scylla / C* (as long as they don't have unsigned types) and trying to hide this complexity may cause a perfectly working application to suddenly start throwing errors after long time.
New deserialization framework will probably allow to downcast error to specific implementation and check what exactly went wrong - but doing this will be much more difficult that creating 2 structures (first with signed types, to interact with DB, second with unsigned types, to use in rest of the application) and handling conversion errors.
for those reasons I think it's better to make users aware of this complexity and deal with it explicitly - it will result in more robust, less error-prone applications.

Instead of implementing SerializeCql and FromCqlVal for u* types, this crate could provide wrapper types for handling different serialization patterns.
There are 4 different approaches:

as: Just convert the unsigned to signed and back. (Could be a wrapper like SignedU*)
MappedU*: e.g. MappedU32(u32) would map the range of an u32 to the range of an i32. (So, u32::MIN..u32::MAX -> i32::MIN..i32::MAX)
BlobU*: e.g. BlobU32(u32) would convert the u32 to [u8; 4] and back. (Also supports u128)
BigInt: Store the signed number as a BigInt. (Could be a wrapper like VarU*)

I'm using MappedU* and BlobU* in my project.


Add weblinks for node.js and ember-cli in README

Added weblinks for node.js and ember-cli in README

Merged. I missed those, thank you!


Update polymer to 0.4.0 js version

https://github.com/Polymer/polymer/releases/tag/0.4.0

This issue has been moved to dart-lang/polymer-dart#287.