2.2.401

Releases.json gets used by automated tools (in particular Azure DevOps) to enable targeting SDKs and versions.
There are pressing security faults requiring an update to account for CVE-2019-1075.  Please add these versions so we can force build systems to pull the latest compilers and ensure we're clear of this security fault.

The checked-in JSON file is obsolete: https://github.com/dotnet/announcements/issues/87. It's going to be removed very soon: https://github.com/dotnet/core/pull/3168.
/cc @leecow

Thank you for the information!!!!!!!

From: Davis Goodin<EMAIL_ADDRESS>Sent: Monday, August 12, 2019 5:00 PM
To: dotnet/core<EMAIL_ADDRESS>Cc: MikeKenyon<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/core] 2.2.401 (#3173)
The checked-in JSON file is obsolete: dotnet/announcements#87https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fannouncements%2Fissues%2F87&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379170554&sdata=Q%2BpBj5PKcODv4I2N5k5y0YwhzM7W0u6%2FOuFTAOyUy90%3D&reserved=0. It's going to be removed very soon: #3168https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fcore%2Fpull%2F3168&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379180565&sdata=XQsu2EUIIMSeAWteStoPZy%2FpdS%2FuZqKvwS1bxD0gPTI%3D&reserved=0.
/cc @leecowhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fleecow&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379200586&sdata=IpAJxiJvKdB%2FuxwXZDuabWcVM5EObD3VT7yH48WQtV0%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fcore%2Fissues%2F3173%3Femail_source%3Dnotifications%26email_token%3DADBNIMVLS4EPXAZ7RLF4YVLQEHTRLA5CNFSM4ILERBR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4EB2BI%23issuecomment-520625413&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379210585&sdata=GOeb9Solvarie3RLE7g3lz3UtmVNsKSHuPE%2FxX0Fjn0%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADBNIMRRCREJMVBUC5LLJJTQEHTRLANCNFSM4ILERBRQ&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379220596&sdata=44wuPd2VuMWnP124FGGRbe%2F6IsYsysJ%2FFBFheVgBv5U%3D&reserved=0.

Hi,
Looking at this, it's looking like the schedule is have removed it 5 months ago.  Given that there's a medium-level security hole in Core that is being blocked in many automated builds because this isn't done and the old file isn't being kept up, is there a schedule on when one of these is going to happen?
Mike

From: Davis Goodin<EMAIL_ADDRESS>Sent: Monday, August 12, 2019 5:00 PM
To: dotnet/core<EMAIL_ADDRESS>Cc: MikeKenyon<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/core] 2.2.401 (#3173)
The checked-in JSON file is obsolete: dotnet/announcements#87https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fannouncements%2Fissues%2F87&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379170554&sdata=Q%2BpBj5PKcODv4I2N5k5y0YwhzM7W0u6%2FOuFTAOyUy90%3D&reserved=0. It's going to be removed very soon: #3168https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fcore%2Fpull%2F3168&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379180565&sdata=XQsu2EUIIMSeAWteStoPZy%2FpdS%2FuZqKvwS1bxD0gPTI%3D&reserved=0.
/cc @leecowhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fleecow&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379200586&sdata=IpAJxiJvKdB%2FuxwXZDuabWcVM5EObD3VT7yH48WQtV0%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fcore%2Fissues%2F3173%3Femail_source%3Dnotifications%26email_token%3DADBNIMVLS4EPXAZ7RLF4YVLQEHTRLA5CNFSM4ILERBR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4EB2BI%23issuecomment-520625413&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379210585&sdata=GOeb9Solvarie3RLE7g3lz3UtmVNsKSHuPE%2FxX0Fjn0%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADBNIMRRCREJMVBUC5LLJJTQEHTRLANCNFSM4ILERBRQ&data=02|01||2469e0e8da574c361ba808d71f78e3bb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637012476379220596&sdata=44wuPd2VuMWnP124FGGRbe%2F6IsYsysJ%2FFBFheVgBv5U%3D&reserved=0.

There's a team that has a dependency on the file and it working to resolve that azure-pipelines-tasks/11100. Once they have made updates (no eta from them), we can remove the file.

@leecow can you close this issue once the azure pipelines issue is resolved?

Got confirmation from the Pipelines team that they have updated.


clojure: <IP_ADDRESS>9 -> <IP_ADDRESS>2

Install again default deps.edn. deps.edn was embedded in clojure jar,
but that change was reverted, see
https://github.com/clojure/brew-install/commit/a34969513fb8f95e541fa4f4b65296c005d76320
Update derivation to produce only one output. Multiple outputs was
introduced by #35140, but I don't think is necessary anymore.
Code was automatically formatted with nixpkgs-fmt

Motivation for this change
Things done

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

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

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

Notify maintainers
cc @bendlas @yegortimoshenko

Merging this into a single output LGTM. I think the original change might have been spurious. nix-review shows lumo failing, but thats also the case for master ...
Thanks!

@bendlas thanks for reviewing and merging it!


Implement typing, linting, code format.

I'm interested in the implementation of the Python good practices on the project.
AC:

Possible to run easily the tools and validations in the local dev environment.
CI runs the tools automatically on push to validate commits and pull requests.

Typing added.


Griffin Upgrade Stone description updates.

Okay, I think im doing this right.
Anyway, I updated the descriptions for Griffin upgrade stones to match in game.

My bad, fixed.


Update to Pkl 0.27 and Spring Boot 3

Based on #5. Fixes #1.
Marked as draft because Pkl 0.27.0 has yet to be released.

update Pkl to 0.27.0 (not yet released)
update Spring Boot to 3.3.4
adapt failing test to Boot 3.x
remove obsolete idea.project configuration from build.gradle
samples: use Spring Boot's Gradle plugin (more realistic)
samples: update Kotlin to 2.0.21
samples: update Gradle to 8.10.2 (by copying the main build's wrapper files)
update changelog and docs (installation docs adapted from pkl-core)
bump version to 0.17.0
support developing against local Pkl build by following instructions
in settings.gradle.kts and samples/settings.gradle.kts

Until this PR is merged, there is no proof that pkl-spring works with Pkl 0.27 and Spring Boot 3.


support to laravel 5.4

I try to install this wrapper on laravel 5.4 thats not possible???

It should work just fine.
Are you getting a error?

i'm a begginer in laravel in your installation manual, the sintaxis is different in laravel 5.4 (Service Provider and facades) which is the correct to configure that in laravel 5.4

Hey,
This does work on 5.4
under 'providers', add:
    `Artisaninweb\SoapWrapper\ServiceProvider::class,`

under 'aliases', add:
  ` 'SoapWrapper' => Artisaninweb\SoapWrapper\Facade\SoapWrapper::class,`

@simplyrupin thanks for the help.
@mikE83 did you get it to work in L5.4?

I have error
[Symfony\Component\Debug\Exception\FatalThrowableError]
Class 'Artisaninweb\SoapWrapper\ServiceProvider' not found
help me

Try running : composer dump-autoload
Let me know how it goes.
Best,
Rupin

Thanks @simplyrupin but error continue in run $ php artisan serve
[Symfony\Component\Debug\Exception\FatalThrowableError]
Class 'Artisaninweb\SoapWrapper\ServiceProvider' not found

Did you run a composer install before artisan serve?
From the error, it seems that the package is not yet installed on your project

Run $ composer install but not install de package.
Run $ composer update and install de packege.
It's work for me. Thanks


Doesn't work with wepack

Description
when I use wepack to load materialize.js, it just show many error, likes:
Uncaught TypeError: $overlay.velocity is not a function
at openModal (materialize.js:962)
Uncaught ReferenceError: lStack is not defined
at openModal (materialize.js:929)
and so on.
i have no idea to fix many of them, anyone can help me ?
Repro Steps
main.js:
require(['jquery'], function($){
require('./asset/materialize')
$(document).ready(function(){
$('.modal').modal()
})
})
them, use webpack to package it and load in a html page.

Duplicate of #1422


Unicode handling meta-issue [JIRA: CLIENTS-791]

We need to review and have a policy for all handling of Unicode character data. This issue consolidates and supercedes:

#287
#156
#32

Key issues:

[ ] Uniform handling of object identifiers (bucket-type, bucket, key)
[ ] Uniform handling of values encoded to/parsed from JSON

Any plans on actually fixing this?

@cread - can you provide information (ideally code as well) of how this issue affects you?

While you're talking to product management, can you also ask them about this issue too: https://github.com/basho/riak/issues/789
Thanks again!


Weird protein split in flexible docking

Hello,
I want to perform flexible docking. However, when I run vina, the job doesn't proceed. No error is raised. I checked the rigid and flexible parts of the receptor, and it seems like something is wrong. Some residues are missing.
Here is the receptor 4lxd.pdb, and these are the outputs of prepare_flexreceptor.py:
4lxd_flex.pdbqt
4lxd_rigid.pdbqt
For the flexible part, I specified amino acids using -s ALA97_ASP100_PHE101_SER102_TYR105_ASP108_PHE109_MET112_VAL130_GLU133_LEU134_ASN140_TRP141_GLY142_ARG143_VAL145_ALA146_GLU149_PHE150_VAL153_PHE195_TYR199. They were chosen based on a distance of up to 5Å from the known ligand.
What could be wrong?

For people to be able to help you, you need to provide more detailed information:

exact vina command and all input files to reproduce the problem. You posted only part of what's needed.
"the job doesn't proceed" can mean a number of things. Does it print something?
"some residues are missing". What residues are missing?

Command lines:
prepare_flexreceptor.py -r 4lxd.pdbqt -s ALA97_ASP100_PHE101_SER102_TYR105_ASP108_PHE109_MET112_VAL130_GLU133_LEU134_ASN140_TRP141_GLY142_ARG143_VAL145_ALA146_GLU149_PHE150_VAL153_PHE195_TYR199
prepare_gpf.py -l H4Fst.pdbqt -r 4lxd_rigid.pdbqt
autogrid4 -p 4lxd_rigid.gpf -l 4lxd_rigid.glg
vina --flex 4lxd_flex.pdbqt --ligand H4Fst.pdbqt --maps 4lxd_rigid --scoring ad4 --exhaustiveness 6 --out output.pdbqt

The output:
#################################################################
# If you used AutoDock Vina in your work, please cite:          #
#                                                               #
# O. Trott, A. J. Olson,                                        #
# AutoDock Vina: improving the speed and accuracy of docking    #
# with a new scoring function, efficient optimization and       #
# multithreading, Journal of Computational Chemistry 31 (2010)  #
# 455-461                                                       #
#                                                               #
# DOI 10.1002/jcc.21334                                         #
#                                                               #
# Please see http://vina.scripps.edu for more information.      #
#################################################################

Scoring function : ad4
Flex receptor: 4lxd_flex.pdbqt
Ligand: H4Fst.pdbqt
Exhaustiveness: 6
CPU: 0
Verbosity: 1

Reading AD4.2 maps ... done.
Performing docking (random seed: 165862369) ...
0%   10   20   30   40   50   60   70   80   90   100%
|----|----|----|----|----|----|----|----|----|----|
*

It's just stuck at *. It doesn't change even though I waited an hour.
The files are in archive:
archive.zip
I use Chimera. Opening simulatenously 4lxd_flex.pdbqt with 4lxd_rigid.pdbqt you can see there are gaps between residues which are filled by AAs in case of original 4lxd.pdbqt file.

Then it's running. But with that many sidechains, it just takes extremely long.
Molecular viewers show gaps because the protein is split between different files, but all atoms should be there. Each atom should be in either _flex or _rigid, not both.

Hi @farmaceut
The inputs look correct. I was able to run the calculation, too; The job does proceed very slowly as @diogomart  pointed out. If you're not running the job on a cluster, I recommend starting with fewer flexible residues :>
About the missing residues: Gly, Pro and Ala cannot be made flexible, because their sidechains do not have rotatable torsions (for Ala, the nonpolar hydrogens are excluded from PDBQT)

@diogomart  @rwxayheee
Oh well. I though, it was me doing something wrong. At least I am aware my old PC is not enough :( Wonder if running on M3 would make it notably faster?

Possibly. But not as fast as having fewer sidechains.

Hi @farmaceut Forgive me for interjecting again, one more point to consider with your selection of flexible residues is the size of the box. If the box does not cover all the mobile atoms in the flexible residues (for example, Asp100, Glu133, Tyr199), you will get enormously large INTRA energies in the docking output (and those residues will appear to be strained, in the output


Allergies Changes

Includes the "how to run" statement in the tests.
Also renames one of the expected functions, since the standard I've been seeing in the community (Ericson, etc.) is to use snake-case for function names, not camel-case (used for variables).

:+1:


Sample 6a) ReactWebChat component - Avatar's not displaying

Is it an issue related to Adaptive Cards?
No
Is this an accessibility issue?
No
What version of Web Chat are you using?
Latest production
Which distribution are you using Web Chat from?
Bundle (webchat.js)
Which hosting environment does this issue primarily affect?
Web apps
Which browsers and platforms do the issue happened?
Browser: Edge (latest), Browser: Chrome (latest)
Which area does this issue affect?
Customization: Branding
What is the public URL for the website?
No response
Please describe the bug
The ReactWebChat component does not display the bot or user avatar, despite provided the appropriate styling.
Do you see any errors in console log?
Warning: Cannot update a component (`ActivityAcknowledgementComposer`) while rendering a different component (`InternalTranscriptScrollable`). To locate the bad setState() call inside `InternalTranscriptScrollable`, follow the stack trace as described in https://reactjs.org/link/setstate-in-render
    at InternalTranscriptScrollable (http://localhost:3000/static/js/bundle.js:61892:25)
    at div
    at http://localhost:3000/static/js/bundle.js:61488:36
    at Composer (http://localhost:3000/static/js/bundle.js:218008:29)
    at TranscriptFocusComposer (http://localhost:3000/static/js/bundle.js:84507:23)
    at BasicTranscript (http://localhost:3000/static/js/bundle.js:62108:26)
    at div
    at Surface (http://localhost:3000/static/js/bundle.js:74780:23)
    at BasicWebChat (http://localhost:3000/static/js/bundle.js:62375:24)
    at SendBoxComposer (http://localhost:3000/static/js/bundle.js:84989:23)
    at http://localhost:3000/static/js/bundle.js:83765:23
    at div
    at http://localhost:3000/static/js/bundle.js:62665:23
    at Composer (http://localhost:3000/static/js/bundle.js:215675:70)
    at ComposerCore (http://localhost:3000/static/js/bundle.js:62680:24)
    at ActivityTreeComposer (http://localhost:3000/static/js/bundle.js:82699:23)
    at ActivitySendStatusComposer (http://localhost:3000/static/js/bundle.js:52984:23)
    at ComposerCore (http://localhost:3000/static/js/bundle.js:44427:34)
    at ActivityAcknowledgementComposer (http://localhost:3000/static/js/bundle.js:51856:23)
    at ActivityKeyerComposer (http://localhost:3000/static/js/bundle.js:52294:23)
    at Provider (http://localhost:3000/static/js/bundle.js:213389:20)
    at ComposerWithStore (http://localhost:3000/static/js/bundle.js:44813:28)
    at PonyfillComposer (http://localhost:3000/static/js/bundle.js:53394:23)
    at RenderChildrenFunction (http://localhost:3000/static/js/bundle.js:50760:23)
    at ErrorBoundary (http://localhost:3000/static/js/bundle.js:50768:5)
    at Composer (http://localhost:3000/static/js/bundle.js:44863:39)
    at Composer (http://localhost:3000/static/js/bundle.js:62837:34)
    at ReactWebChat (http://localhost:3000/static/js/bundle.js:65876:24)
    at AdaptiveCardsComposer (http://localhost:3000/static/js/bundle.js:93314:38)
    at AddFullBundle (http://localhost:3000/static/js/bundle.js:93073:37)
    at FullReactWebChat
    at WebChat (http://localhost:3000/static/js/bundle.js:331:3)
    at div
    at div
    at MinimizableWebChat (http://localhost:3000/static/js/bundle.js:127:63)
    at div
    at App

How to reproduce the issue?

npm install and npm start the sample here - 06.recomposing-ui/a.minimizable-web-chat/
Ensure the sample is running locally (http://localhost:3000/)
In WebChat.js, replace from line 9-15 with this:

    const styleSet = useMemo(
        () =>
          createStyleSet({
            bubbleBackground: '#e8e8e8',
            bubbleTextColor: '#FFFFFF',
            bubbleBorderRadius: 5,
            bubbleFromUserBackground: '#0184D7',
            bubbleFromUserBorderRadius: 5,
            backgroundColor: 'Transparent',
            botAvatarImage:
              'https://docs.microsoft.com/en-us/azure/bot-service/v4sdk/media/logo_bot.svg?view=azure-bot-service-4.0',
            botAvatarInitials: 'Bot',
            userAvatarImage: 'https://github.com/compulim.png?size=64',
            userAvatarInitials: 'User'
          }),
        []
      );

In Chrome open http://localhost:3000/
Type a message in the chat and send it
Notice that the avatars are not appearing, but the other styles (bubble text and background colors) specified are appearing

What do you expect?
Both the bot and the user avatar to appear.
What actually happened?
The avatars for both the bot and the user are hidden.
Do you have any screenshots or recordings to repro the issue?

Adaptive Card JSON
No response
Additional context
Similar issue, but in an older version of the botframework-webchat

Overriding styleSet is not recommended, please use styleOptions instead: https://stackblitz.com/edit/stackblitz-starters-j4zqlx?description=React   TypeScript starter project&file=src%2FMinimizableWebChat.js&title=React Starter


Cursor is not moving back to previous box .

Describe the bug
I am using Kika keyboard for Xiaomi which is default keyboard for Redmi  6  Pro, and when I am pressing clear button on the keyboard the cursor is not moving back to the previous box.
I have tested a few other keyboards it seems fine but only for Kika keyboard your lib is not working.
To Reproduce
Steps to reproduce the behavior:
Use Kika keyboard for filling otp.
Screenshots
Check the video .
issue video.zip
Smartphone (please complete the following information):

Device: [Redmi 6 pro]
OS: [Android 9 (pie)]

Additional context
Add any other context about the problem here.

Thanks for the video. Looks like the back button click is not picked up by the RN. Will look into this. But most likely, it is an issue in RN.

any update? i got same issue

@vgvishal538 what React Native version is this?

I think the issue only happens to a newer version of Android. It works fine on Samsung Galaxy S3, but cannot move back on OnePlus 7 Pro. Works fine on iPhone also.

@vgvishal538 what React Native version is this?

"react-native": "^0.60.4",

Is there any update for this issue? i got same issue

From react-native document
There is a note: on Android only the inputs from soft keyboard are handled, not the hardware keyboard inputs.
That's why we have this issue. It just happens to the hardware keyboard inputs!

From react-native document
There is a note: on Android only the inputs from soft keyboard are handled, not the hardware keyboard inputs.
That's why we have this issue. It just happens to the hardware keyboard inputs!

Thanks! Its working :)

From react-native document
There is a note: on Android only the inputs from soft keyboard are handled, not the hardware keyboard inputs.
That's why we have this issue. It just happens to the hardware keyboard inputs!

this helped


Restore and fix post-process shadow blurring in 3D

When uncommented, this function resulted in the dithering pattern being less noticeable. However, it also introduced jaggies in distant shadows, which are especially jarring in motion. Its performance cost was fairly low, but it wasn't worth the tradeoff.
The filter was quite effective for blurry shadows that are up close, so maybe it could be resurrected in the future based on shadow texel density relative to the camera.
In the future, temporal antialiasing will be added to optionally hide the dithering pattern by changing it every frame (letting the accumulation buffer do its job of averaging it all). On high refresh rate monitors, the shadow filtering pattern can be jittered to let the monitor's response time perform this averaging (without relying on TAA).
Preview
To see the negative impact of the filter on distant shadows, look at the top of the first screenshot, and compare it with the Filtering uncommented screenshot.
Current shadow rendering

Filtering uncommented

I think we can actually fix this by using dFdxFine and dFdyFine and clamp the result so it doesn't exceed 1.0
I tested the below code and it removes the artifacts for me on Intel. I'm going to test quickly on AMD. It would be great if you can test on NVidia. Maybe we can fix this instead of removing it!
	interp_shadow -= dFdxFine(interp_shadow) * (float(fc2.x & 1) - 0.5);
	interp_shadow -= dFdyFine(interp_shadow) * (float(fc2.y & 1) - 0.5);
	interp_shadow = min(interp_shadow, 1.0);

You'll also need to change the call to blur_shadow in the directional light code to shadow=blur_shadow()

I've applied your suggestion (and pushed a new commit overriding the old one for that), but the blurring still seems to look bad. It almost looks like shadow maps are rendered at half resoution in the viewport:
1024×600

Before
After (this PR)

2560×1440

Before
After (this PR)

Testing project: test_shadow_blur_pass.zip

What is happening is the blurring applies only within screen aligned 2x2 quads. So in distant areas (where the penumbra is less than 2 pixels wide) it is creating aliasing as the shadow is much higher frequency than 2x2.

What is happening is the blurring applies only within screen aligned 2x2 quads. So in distant areas (where the penumbra is less than 2 pixels wide) it is creating aliasing as the shadow is much higher frequency than 2x2.

Could we detect the penumbra's length or use a distance-based metric to avoid this?

What is happening is the blurring applies only within screen aligned 2x2 quads. So in distant areas (where the penumbra is less than 2 pixels wide) it is creating aliasing as the shadow is much higher frequency than 2x2.

Could we detect the penumbra's length or use a distance-based metric to avoid this?

Yep! We could probably reuse ddx and ddy (based on vertex position) which are already calculated. I'm not sure what the performance implication would be though.

I've got a dumb question... is this post proc done in screen space or what?
I mean if it is, wouldn't it maybe be worth seeing if Gaussian Blur has good results?

I've got a dumb question... is this post proc done in screen space or what? I mean if it is, wouldn't it maybe be worth seeing if Gaussian Blur has good results?

It's not actually done as a post process. It's done at render time. And it only works within each 2x2 fragment quad, so guassian blur isn't an option

Note that https://github.com/godotengine/godot-proposals/discussions/7178 may be worth considering as an alternative, although it aims to resolve a more specific issue (aliasing due to shadow texel density being too high at certain spots of the image).

Well the Post-processing also reduces the graininess a little, which that wouldn't.
Though what was suggested in the proposal kinda hit a dead end, as the simple angle based blur increase causes it's own issues, and I have no ideas how exactly I'd go about creating my own Anisotropic Filtering shader to use for Shadows or how to Apply Hardware Filtering to it properly (I know Anisotropic Filtering works just fine without mipmaps, I tested, but unsure about how it'd respond to eh, 4 maps in 1 texture even if I figured out how to apply it...)


Regarding Android Issues

#8 is for Windows app.  I am getting this error. And please don't close this untill I am done with error 🙏

Read the Comment on it. Stop opening new issues.


[vip] supports specifying listen on specific network cards

Is your feature request related to a problem? Please describe.

https://github.com/openelb/openelb/discussions/397
Describe the solution you'd like

Describe alternatives you've considered

Additional context

Under OpenELB VIP Mode, i can select a NIC as a VIP binding.


fix deleted objects are not handled at all

Right now deleted objects are not removed from index. This fix allows to process deleted objects as well and remove them from index.

Not entirely certain but I think \Valantic\ElasticaBridgeBundle\Messenger\Handler\AbstractRefreshHandler::resolveElement also needs a fix in the try-catch as the handler might be executed (much) later and getById will fail and the part in lines 28 ff which is supposed to handle that, is skipped because of the exception that's thrown above it.
try {
    $element = $className::getById($message->id);
} catch (\Throwable) {
    throw new PimcoreElementNotFoundException($message->id, $message->className);
}

if (!$element instanceof AbstractElement) {
    // The element in question was deleted so we need a skeleton.
    // ...
}


How to add a spy to port ?

Hi, I have the following scenario.
I have 4 kernels that do processing of some sort. Each of them has one input and one output ( first and last are single output/ single input).
 m += a >> b >> c >> d;

I am designing a 'Spy' type of kernels that I want to attach to specific ports and monitor the content that passes though. They will not modify anything, just read the content. Image this will be used for visualization purposes, testing, logging or data dump.
One solution I found is to create an sky kernel and include it in the production chain.
 m += a >> spy >> b >> c >> d;

However, this has the disadvantage that I have to edit the production chain code.
I would much more like to write something like:
spy.attach( a, b);

Is there any way I can do this with raftlib ? Can you provide an example. Thank in advance.

Hi there,
how about inheriting all your kernels from a super-kernel where you can spy all you want.
This super-kernel will have the run() method, from which you will call a subRun() method where you will put the current logic in every kernel.
The data to be spied can be sent to subRun() as parameter so afterwards the super-kernel can do what it needs...
Hope it helps,
Cheers..

I've been contemplating something like this. A colleague of mine when I was a new grad student built this instrumentation system for yet another streaming framework called Auto-Pipe (which in many respects is where I got the idea of building something entirely new which is RaftLib). The TimeTrial system had an annotation language for the streams. Maybe we could build a kernel as @mr-j0nes suggested and it have basic hooks to do monitoring and instrumentation. Once I add multi-node support, that would be preferable to adding your own given the run-time could move your kernels out from under you and you'd have very odd readings on your instrumentation. Any thoughts on the hooks that'd be needed? Ones that immediately come to mind for me are arrival rates on streams, streaming averages of things, and some moment estimation on the streams. I think I still have kernels in the repo for most of that which can be used to tie-in things. I'd suspect we could provide some lower-level pull-outs from the ring-buffer class that are privileged to the "spy" class. I think given the recent security scares..and for ease of text searching, lets go with "monitor" or perhaps "instrument" as a class name.
If you're willing to put a proposal together, I'm willing to help code them. It can definitely be a collaborative process.


Recipe does not work in colab anymore.

Hi I was trying to run the icefall recipe in colab based on yesno recipe but its not working anymore. I believe this happened after colab updated to a newer python version most probably.
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 187, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/usr/lib/python3.10/runpy.py", line 110, in _get_module_details
    __import__(pkg_name)
  File "/usr/local/lib/python3.10/dist-packages/k2/__init__.py", line 23, in <module>
    from _k2 import DeterminizeWeightPushingType
ModuleNotFoundError: No module named '_k2'
Collecting environment information...
PyTorch version: 1.13.1+cu117
Is debug build: False
CUDA used to build PyTorch: 11.7
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: 10.0.0-4ubuntu1
CMake version: version 3.25.2
Libc version: glibc-2.31

Python version: 3.10.12 (main, Jun  7 2023, 12:45:35) [GCC 9.4.0] (64-bit runtime)
Python platform: Linux-5.15.109+-x86_64-with-glibc2.31
Is CUDA available: False
CUDA runtime version: 11.8.89
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: Could not collect
Nvidia driver version: Could not collect
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.9.0
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

Versions of relevant libraries:
[pip3] numpy==1.22.4
[pip3] torch==1.13.1+cu117
[pip3] torchaudio==0.13.1+cu117
[pip3] torchdata==0.6.1
[pip3] torchsummary==1.5.1
[pip3] torchtext==0.15.2
[pip3] torchvision==0.15.2+cu118
[conda] Could not collect

@nabil6391
Thanks for reporting.
It should work now.
Please try again.

Thanks, working now.


doc: Path not provided for "check_message" and "do_send_message".

In the documentation of the "Sending messages," path for the check_message and do_send_message function is not provided. So, I added the path of both for future contributors.

@100RABHpy this is probably an improvement, though I think we'd be better off teaching that you can find functions with e.g. git grep "def do_send_message" instead, but that's probably a larger problem to figure out where to teach that.  So I'll go ahead and merge this.
Thanks @100RABHpy!

@timabbott I myself used git grep for finding these functions. But I thought it would be a time saver for the future contributor. That's why I mentioned the issue.
We can mention about git grep in the "Using Git as you work" section of the git guide(or at least in the cheat sheet).

I can create a PR if you want this git command to be added to the git guide.

I'd check it's not already advertised, but if not, it's worth doing!
On Wed, Mar 10, 2021 at 12:58 PM Sourabh @.***> wrote:

I can create a PR if you want this git command to be added to the git
guide.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/17548#issuecomment-796093407, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU6NWWEW3QPXV4JPLCH4XLTC7MPBANCNFSM4Y6JRBXA
.

Yeah, git grep is mentioned in the detailed cheat sheet and also in the CSS-guideline section.


⚠ Bofrost.es has degraded performance

In d52ab18, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1599 ms

Resolved: Bofrost.es performance has improved in ff7e62e after 6 minutes.


Is it possible to use ollama in autocomplete mode?

To have a similar interaction to what Copilot does (autocompletes ahead of where you are).

Not planned for this plugin. If there is another plugin which does OpenAPI-driven autocomplete (ChatGPT), you can try configuring the API url/etc to point to your ollama server, as they have added OpenAPI-compatible endpoints in a recent update.


[Bug]: Why large value hides on yAxis - Line Chart?

Tremor Version
latest
Link to minimal reproduction
None
Steps to reproduce
I started using the Line Chart component to display the investment values, these values are in thousands and millions.
I have noticed that these values are hidden in my chart:

( See the chart y axis values)
Code

"use client"

import { Card, Title, LineChart as TremorLineChart } from "@tremor/react"

import { CardDescription, CardTitle } from "@/components/ui/card"

export const LineChart = ({ data }: { data: any }) => {
  const timeSeries =
    data &&
    data["Time-Series"].map((serie: any) => ({
      remainingCredit: serie["Remaining Credit"].toFixed(),
      discountCashFlow: serie["Accumulated Discounted period Cashflows"].toFixed(),
    }))

  const valueFormatter = (number: number) => `${new Intl.NumberFormat("de").format(number).toString()}`

  return (
    <div className='w-full'>
      <CardTitle className='text-lg'>Export/Import Growth Rates (1970 to 2021)</CardTitle>
      <CardDescription>ji</CardDescription>
      <TremorLineChart
        className='mt-6'
        data={timeSeries}
        index='year'
        categories={["remainingCredit", "discountCashFlow"]}
        colors={["rose", "lime"]}
        valueFormatter={valueFormatter}
        yAxisWidth={40}
        showAnimation
        showGridLines={false}
        animationDuration={2000}
        curveType='monotone'
      />
    </div>
  )
}

What is expected?
The values should have fit width while the chart should have responsive width.
What is actually happening?
The chart component froce the y axis to stretch the layout.
What browsers are you seeing the problem on?
Chrome
Any additional comments?
No response

Hi @MenaiAla, use the prop 'yAxisWidth' to make the label space bigger! :)

Thank you @christopherkindl for your help <3.


kernel config fragments: add RPMSG_CTRL & RPMSG_TTY

Since 5.18 RPMSG_CTRL is seperate from RPMSG_CHAR. Enabled it as a module.
Older kernels will ignore this.
You will get a warning.
Also enables RPMSG_TTY
Signed-off-by: Bill Mills<EMAIL_ADDRESS>

for this PR, can we introduce vendor-specific checks? as not all platforms formally support use of the RPMSG-TTY interface

Staged to master-next.

Code has been pushed to master.
It was decided we will come back and re-evaluate the default options in the future.


The way to use ip address based on mac

Hi @merdok, i tried to inject the module, which scan the local network and check the ip address of the tv at current moment by the mac address. This approach more flexible to check ip and the start connection. But i really don't understand where i should to past this code.
I used module 'local-devices'. and the following code:
function findDevice(mac, completion) { 	localDevices().then(devices => { devices          devices.forEach(function(item) {             if (item.mac == mac) {                 console.log('tv ip ' + item.ip)                 completion(item.ip)             }         })     }) }
Hope you will can to do this :)

I do not think that this is a good idea to include that. I mean if someone can find the ip address then he/she can also find the mac address very quickly. The scanning would just unnecessary complicate the plugin and could lead to potential bugs.

I think it is a good option to decide what the way do you prefer: provide your ip address and mac in the config, or just mac :)
Ok, maybe you can advice to me, where i should call this method, to receive the tv ip by the mac, and only after that start homebdridge configuration?

It is written in your router at the same place where you found the TV mac address.

Not really, i faced with the issue, when the TV starts to use different ip address in local network. Reserving ip address for the specific mac unfortunately not working for me :(
Therefore i start to investigate the possibility to bind ip and mac in runtime

If you cannot reserve an ip address for the the tv then this is clearly an issue without router/network.

Sure, but i offer to make the workaround to resolve this issue. Someone who will use your library will decide the way to bind a tv by mac or ip.
By the way, it will be very helpful if you could explain where i can use async method 'local-deivces' and only after that continue accessory configuration?

Sorry, i do not know what you mean by async method 'local-deivces'


Admin users can create invalid passports by failing to fill either a consumer site or a playlist

Bug Report
Expected behavior/code
When creating a passport, the admin user must link it to EITHER a consumer site OR a playlist.
Actual Behavior
It is today possible technically to create a passport linked neither to a consumer site nor to a playlist. Doing so generates 500 errors on launch requests trying to use this passport.
Steps to Reproduce

Create a new passport from the admin and link it neither to a consumer site, nor to a playlist
Make a launch request using this passport's credentials
And then the bug happens!

Environment

Marsha version: 1.0.0

Possible Solution
We should solve this problem at its root: add a custom form validation to the passport admin update view and check that either the consumer_site or the playlist fields are filled.

This was fixed in PR https://github.com/openfun/marsha/pull/188


dry_run doesn't work in (at least) SQLite3

I tried import-export on SQLite3 from the admin console, but it appears it always commits my changes before I "confirm" them as reported in #371 (Note that IMPORT_EXPORT_USE_TRANSACTIONS or use_transactions in Resource don't help).
I googled around the web and I found that the Django document states the following:

Savepoints in SQLite
While SQLite supports savepoints, a flaw in the design of the sqlite3 module makes them hardly usable.
When autocommit is enabled, savepoints don’t make sense. When it’s disabled, sqlite3 commits implicitly before savepoint statements. (In fact, it commits before any statement other than SELECT, INSERT, UPDATE, DELETE and REPLACE.) This bug has two consequences:

The low level APIs for savepoints are only usable inside a transaction ie. inside an atomic() block.
It’s impossible to use atomic() when autocommit is turned off.

I don't fully understand this, but suspect savepoint() in import_data_inner() doesn't work because it is called outside atomic() block (under the different connection instances?).

Closing due to lack of activity.


Fix runs query resolver to match schema

The query runs returns an error stating Expected Iterable, but did not find one for field Query.runs.  as the object returned from the resolver does not match the Query schema. So I fixed the getAllRuns api to use the fullRunReducer instead of the runFeedReducer
fixes #62

Will look into that later today


Updated standalone version

Hi and good day.
Is there a way to download an updated standalone version?
I see there is some bugs fixed from 3 months ago but the only link to download a standalone is from the official website and that version is from march 2020.
BTW: i do not nothing about github or how github works, sorry if i'm saying something very noob.

I simply merged some automatic pull request, stuffs related to dependencies stuffs.
But it doesn't change anything.
Sculptgl is not maintained anymore.
My main project is now Nomad Sculpt, there's an experimental web version


Remove trailing empty line in copyright comment (part 4)

Next part of https://github.com/helidon-io/helidon/pull/5324

/trigger


Delete logging-stackdriver.md

Ref: #20232
Per https://github.com/kubernetes/enhancements/tree/master/keps/sig-docs/1326-third-party-content-in-docs this content is not necessary and is vendor specific.

/hold
This needs a rebase...

/hold remove

/hold cancel

Thanks.
/lgtm
/approve


Reuse temporary variable in transformed code

This optimization greatly simplifies the transform, and results in smaller output code. For example, idx(a => a.b.c.d):
Before
var x, y, z;
(x = base) != null ?
  (y = x.b) != null ?
    (z = y.c) != null ?
      z.d :
    z :
  y :
x;

// minified (62 bytes)
var x,y,z;null!=(x=base)?null!=(y=x.b)?null!=(z=y.c)?z.d:z:y:x

After
var z;
(z = base) != null ?
  (z = z.b) != null ?
    (z = z.c) != null ?
      z.d :
    z :
  z :
z;

// minified (56 bytes)
var z;null!=(z=base)&&null!=(z=z.b)&&null!=(z=z.c)?z.d:z

If your minifier is less performant, the gains are higher.
FYI: While working on this, I noticed that the this is not correct for method calls. I'll try to fix that later on.

Looks good. We may want to eventually use Jest snapshots to more easily update the test expectations.

Don't tell @cpojer, but I wanted to switch away from jest because debugging using --inspect is so hard


fix: resolve 401 unauthorized issue with metadata endpoint

Without this fix, calls to the metadata endpoint to get eth0 network interface id fails.

Thanks for the contribution! Just as a note, the current AMIs are still compatible with IMDSv1, but IMDSv2 support is definitely needed. I'll merge once I've had a chance to test this out. Thanks again!


Personal lighting

Milwaukee Penlight 2105

Features

100 Lumens of TRUEVIEW™ High Definition Output
IP67 Rated: Waterproof and Dustproof
Protective Rubber Bite Zone
6000 Series Aircraft Aluminum Body
On/Off and Momentary-On Button
Removable Metal Clip
Impact and Chemical Resistant Lens
Powered by (2) AAA Batteries (Included)  4
3 Meter Beam Distance
Limited Lifetime Warranty (Excludes Batteries)


⚠ Bofrost.at has degraded performance

In 1b5a771, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1868 ms

Resolved: Bofrost.at performance has improved in e114093.


[CHORE] Upgrade version to v0.9.32

Goal
The goal of this PR is
Closes #705
Must go after #744
Notes:
PR for Substrate upgrade to 0.9.32
Release Notes .31
Release Notes .32
Changes from 0.9.30 via 0.9.31 to 0.9.32

Bump dependency version to 0.9.32

Weight V2 : introduction of max proof size for weights, for now I am using what cumulus uses and suggested on substrate stack exchange ...more.

Runtime builds now run a runtime integrity checks

Checklist

[ ] n/a Chain spec updated
[ ] n/a Custom RPC OR Runtime API added/changed? Updated js/api-augment.
[ ] n/a Design doc(s) updated
[ ] n/a Tests added
[ ] n/a Benchmarks added
[x] Weights updated

Branched out from #744, Ideally review after it merges


adding a highlight option

This PR adds a new --highlight (or -I) option that will use a regexp to highlight some parts of the logs.
This is similat to the --include option behaviour but will not filter logs, only add color.

Anyone ?


When the layout changes, updated all connected peers

To better support streaming, we need to send layout changes to all connected peers.

Done in v1.2.0


Use a newer API layer

Well, the official documentation (= JSON export of the schema) was not uptated since layer 23 (2014) :c
The custom JSON export of version 57 by zerobias/telegram-mtproto looks interesting.

{
		"id": "-2035355412",
		"method": "auth.sendCode",
		"params": [{
			"name": "flags",
			"type": "#"
		}, {
			"name": "allow_flashcall",
			"type": "flags.0?true"
		}, {
			"name": "phone_number",
			"type": "string"
		}, {
			"name": "current_number",
			"type": "flags.0?Bool"
		}, {
			"name": "api_id",
			"type": "int"
		}, {
			"name": "api_hash",
			"type": "string"
		}],
		"type": "auth.SentCode"
	}

I have absolutely no idea about those 'flags' type :c

From https://core.telegram.org/mtproto/TL-formal :

Identifier #: This type is used to specify a special type of nonnegative integers in the range from 0 to 2^31-1; its main purpose is the same as that of Type. There are two built-in constructors: O : # and S : # -> # (“null” and “next number”, respectively), which work as if # was defined using the schema

Predefined identifier Type: This type signifies the type of all types. It is usually used to specify the types of optional parameters in the constructors of polymorphic types. If strongly desired, it can be used in its own right, but this is very rarely needed in practice.

WHAT. THE. HELL.

021ef240e465f0efd80998faa43c57c5c9cf867a

See https://core.telegram.org/mtproto/TL-combinators
See https://core.telegram.org/mtproto/TL-formal

¯\(ツ)/¯

Looks like an answer : https://stackoverflow.com/questions/38903321/handling-flags-types-in-telegrams-tl-schema-language

This library is now able to Build/Parse schemas using those "flags".

Waiting for documentation...

Version 0.2.0-beta has been released with support for layer 57.


WPF example project could trow an exception

I just updated to VS2015 update 3 and suddenly the WPF examples project throws an exception that is not related with the library.

A work around is to disable UI Debugging Tools for XAML, ensure this option is not checked at:
Tools –> Options –> Debugging –> General –> Enable UI Debugging Tools for XAML
Then the examples will not throw anymore...
I already reported as much as I could to the VS team, hope they fix it soon.

I posted this on gitter but I'll put it here too.
I've been runing VS 2015 update 3 since it was released and did not have any issues with LiveCharts 0.7.11. I'm a similar but difference exception ever since updating to 0.7.12.
Here is the snippet that causes the exception at runtime. I removed the bindings:
<lvc:CartesianChart Margin="0,0,3,0" 					DisableAnimations="True" 					LegendLocation="None"> 	<lvc:CartesianChart.Series> 		<lvc:LineSeries Title="RA" 						MinWidth="0" 						Fill="#330000ff" 						LineSmoothness="0.35" 						PointGeometry="{x:Null}" 						PointGeometrySize="0" 						Stroke="#0000FF" 						StrokeThickness="2" 						Values="0,1,2" /> 		<lvc:LineSeries Title="DEC" 						MinWidth="0" 						Fill="#33FF0000" 						LineSmoothness="0.35" 						PointGeometry="{x:Null}" 						PointGeometrySize="0" 						Stroke="#FF0000" 						StrokeThickness="2" 						Values="2,1,0" /> 	</lvc:CartesianChart.Series> 	<lvc:CartesianChart.AxisX> 		<lvc:Axis MaxValue="50" 				  MinValue="0" 				  ShowLabels="False" /> 	</lvc:CartesianChart.AxisX> 	<lvc:CartesianChart.AxisY> 		<lvc:Axis MaxValue="1" 				  MinValue="-1" 				  Position="RightTop" 				  ShowLabels="True"> 			<lvc:Axis.Separator> 				<lvc:Separator Step="1" /> 			</lvc:Axis.Separator> 		</lvc:Axis> 	</lvc:CartesianChart.AxisY> </lvc:CartesianChart>
And here is the exception:

Well I am pretty sure I found an Issue in VS, for now I fond what caused this issue, and it was removed even the library would behave better with the previous code :cry:
If anyone at VS team sees this issue, it started at
d7540bc874953060a4e141fb1dd4383ca768251d
And what cased the issue was this
What happens there, is that I force the chart to update, as the code reaches this point, Otherwise (the alternative I took) the chart waits for the next updater tick to update.


Allow study description component to accept HTML

Need
The dbGaP response has a few types of HTML markdown elements that provide for the basic formatting of the description. Currently, the description section on AnVIL catalog displays the markdown instead of using it to format the description.
Approach
TBD - Update descritption.tsx to take HTML?
Let's discuss options.
DOD

The HTML items such as P and Links show properly in the detail pages.

@NoopDog TODO reminder - investigate potential uses for MDX.


Make a release?

Thanks for this great theme!
I want to use it for a doc site that has lots of Xarray notebooks, so I'm really glad to see #274 resolved. I'm currently building my site against the github master.
I notice you haven't had a release since early October. So this is a gentle 👈 to ask whether you might make a new release soon? Then I can pin my doc build on a stable version.

@rabernat releases are cheap, so I quickly made a v0.4.2 release with only that xarray patch (I want to first fix some things that are in flux on master before doing a release there)

@rabernat releases are cheap, so I quickly made a v0.4.2 release with only that xarray patch (I want to first fix some things that are in flux on master before doing a release there)

Thanks so much!

Thanks so much!


Systemd boot delete old specialisation

Description of changes
Same as https://github.com/NixOS/nixpkgs/pull/194928 but also reduce the try-except block scope.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Merge, because review happened in parent PR already.


fix cannot find nested ignore blocks

func case3(x int) { //+gocover:ignore:block
	var c, python, java = true, false, "yes!"
	if x > 0 { //+gocover:ignore:block
		fmt.Println(i, j, c, python, java)
	}
	fmt.Println(i, j, c, python, java, x)
}

For such nested case, it cannot ignore the if statement. This PR is to fix this issue.
As the cover profile block from the go testmay have start line and endline overlap on two different block, so should not return endline of a profile block to skip text match, and use map to indicates whether two lines are in the same profile block.

so the way to fix this issue is starting to using the "column" data of the block.
so the ignore hit be more accurate?

what does it mean "have start line and endline overlap."?

@yangzuo0621 please merge this after the issue reported fixed.


Windows (MinGW)

Supersedes PR https://github.com/tangrams/tangram-es/pull/1812 (thanks to @tehKaiN for the initial work on this):

Add platform layer for Windows platform.
Add initial AppVeyor configuration (https://ci.appveyor.com/project/karimnaaji/tangram-es -- currently building my own fork in these logs, but once merged, simply enabling AppVeyor for this repo should work), can be greatly improved by using a docker container having pre-installed dependencies, this initial work builds z-lib and curl from source which isn't the best.
Conan wasn't an alternative for the build as it doesn't have dependencies built for MinGW and was failing for unclear reasons when building from source. It could be of use if MSVC is ever supported.
curl multi API support hasn't been added yet, still TODO for that PR (added in https://github.com/tangrams/tangram-es/pull/1994).

After the installation of MinGW you should be adding the appropriate path to the MinGW libraries to your PATH. An issue like this is most likely the runtime .dlls not available in your PATH. Did you make that step?
Dependency walker is a great tool to manage and resolve those kind of issues.

Success! I got the exe to launch after updating my PATH to include my mingw64 bin folder and my "cmake install" bin folder. Might be worth noting that in the README for troubleshooting.
The next issue was that the file path to the default scene was incorrect at launch:
TANGRAM C:\Users\Matt\Documents\tangram-es\platforms\windows\src\main.cpp:25: Base URL: file://C:\Users\Matt\Documents\tangram-es\build\windows
TANGRAM C:\Users\Matt\Documents\tangram-es\core\src\map.cpp:244: Loading scene file (async): file://C:\Users\Matt\Documents\tangram-es\build\windows/scene.yaml
WARNING C:\Users\Matt\Documents\tangram-es\core\src\platform.cpp:29: Failed to read file at path: /scene.yaml

This was the output from launching either from Command Prompt or by clicking the exe in File Explorer. We might need to adjust the file path logic in WindowsPlatform - I don't think our Url class is able to reformat Windows-style paths.
However, once I corrected the file path manually the scene loaded and rendered! 🎉

@matteblair @karimnaaji  I've hacked around it in one of my commits, In main.cpp
    // Resolve the input path against the current directory.
    Url baseUrl("file:///");
    char pathBuffer[PATH_MAX] = {0};
    if (_getcwd(pathBuffer, PATH_MAX) != nullptr) {
        // URL handler apparently needs slash at end of it
        strcat(pathBuffer, "/");
        baseUrl = Url("file:///" + std::string(pathBuffer)).resolved(baseUrl);
    }

IIRC this fixed path problem for me. Of course proper fix would be better but perhaps it'd guide you to the root of the cause.

Any progress in this PR?

I've been sadly neglecting this PR for a long time. I had been attempting to create a "native" port that doesn't require MinGW, but it's clear that having a port of some kind is still really valuable! I'll attempt to fix the conflicts in this PR and test it again over the next week.


Adopt VS Code's 'asWebviewUri' API

Hi, I maintain VS Code's Webview API
Issue
Our telemetry suggests that your extension uses webviews and may be loading resources in these webviews using hardcoded vscode-resource: URIs. These URIs have some important limitations and don't work properly when VS Code is run in a browser. We are also making changes in the desktop version of VS Code that may cause these URIs to not work properly in future versions of VS Code.
While we are making our best effort to continue support existing webview extensions that use vscode-resource: URIs on desktop versions of VS Code, we will not able to fully support all uses cases.
Fix
To ensure that your extension continues to work in as many environments as possible—including on web and remotely—please:

Switch to use the Webview.asWebviewUri function for loading resources.

Switch to use the Webview.cspSource property in content security policies.

These APIs shipped around 2 years ago so they should be available in all modern versions of VS Code. You can find additional information about the issue here: https://github.com/microsoft/vscode/issues/97962
Let me know if you have any questions about this change

We should be clean of that hard-coded URI scheme since 3.0.0 (Sep 2020). Thanks for the heads up.


feat(cancun): EIP-5656: MCOPY - Memory copying instruction

Started working on this. Still needs work.
Will Close #523

left few comments

Awesome thanks for the comments. Will work some more on this today

So the thing i want to try and fix now is that the memory_copy_resize has some nasty unwraps because the peek returns a result. I thought to use the peek because its how i was reading the Back method in the go implementation. But im not sure we have a similar implementation. I could try to write a something with similar functionality. What do you think?

So the thing i want to try and fix now is that the memory_copy_resize has some nasty unwraps because the peek returns a result. I thought to use the peek because its how i was reading the Back method in the go implementation. But im not sure we have a similar implementation. I could try to write a something with similar functionality. What do you think?

In the geth implementation they validate the stack length in the interpreter here
https://github.com/ethereum/go-ethereum/blob/942ba4ddaa0e2591b77a42f40059babbaaa44154/core/vm/interpreter.go#L183C1-L187C4

I think we are close!

I think we are close!

Awesome, thanks for all the feedback it's been really helpful in learning all the details here. Let me know if there are any more problems you see here.

I think we are close!

Awesome, thanks for all the feedback! it's been really helpful in learning all the details here. Let me know if there are any more problems you see here.

Can you check why eth tests are failing?

Got the spec to pass, wrote a test for the copy too

For some reason the lint is failing with
  --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/proc-macro2-1.0.56/src/lib.rs:92:30
      |
92 |     feature(proc_macro_span, proc_macro_span_shrink)
      |                              ^^^^^

I looked into it and seems like this error usually occurs when using an incompatible version of the proc-macro2 crate with the current Rust toolchain. The lint passes locally by fails here. Interestingly it only started failing with the edition of my test

Wrote some tests and got CI passing!


Custom commands

My boot.img didn't become rooted after patching with Magisk (seemingly because it uses a rare SAR).
May I suggest an option that the app just dumps the boot.img to the accessible internal storage, so I can manually patch it.
Maybe making a commnad to run an external bash script would be best, as this allows custom dumping/writing partitions etc.
Many thanks,
B. Spitz

I would add to this, that just giving some shell/terminal would be also nice.
Would help us with https://github.com/polygraphene/DirtyPipe-Android/issues/3

https://github.com/vvb2060/Magica/blob/55afffa713acd8efe2413e28acbfea771e2b6d69/app/src/main/java/io/github/vvb2060/puellamagi/MainActivity.java#L85-L91
This is an open source project, you can do whatever you want, like start sshd. Why are you seeking my technical support?


Remove deprecated bind_hass decorator

This wasn't being used anyway and will throw a warning with 2024.3 release
See https://developers.home-assistant.io/blog/2024/02/27/deprecate-bind-hass-and-hass-components/

Thanks a lot for taking care of this!

No problem, I actually looked at your store code when I was learning how to do this myself so spotted the warning on my own repo so thought I'd say thanks for the inspirational code by fixing it.


ImportError: No module named dogpile.util.readwrite_lock

I think this should be
from dogpile.core.readwrite_lock import ReadWriteMutex
instead.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34935575-importerror-no-module-named-dogpile-util-readwrite_lock?utm_campaign=plugin&utm_content=tracker%2F321252&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F321252&utm_medium=issues&utm_source=github).

You need to update dogpile.cache to 0.6.0. It's in the requirements.

See 5d28f1497883cec52e1d48cf63f64610534fe682


Should dependency group names be normalized with underscores converted to hyphens?

Should dependency group names be normalized with underscores converted to hyphens?
Example:
uv init example
cd example
uv add --group some_group ruff
grep 'some.group' pyproject.toml

with output:
some-group = [

I'm looking into porting a project using Poetry and Nox, where Nox session names and Poetry group names use underscores.  These names are exposed in noxfile.py and on the nox command line.  It's looking like these names might need to switch to using hyphens in order to use uv dependency groups.  Is this expected?
I'm running uv version 0.4.27 on Ubuntu 24.04.

We might want to preserve the literal name you provide but those two names should be equivalent everywhere per the normalization rules. How is this affecting nox?

Sorry for the lack of clarity.  I don't know of a direct problem with nox caused by this normalization.  The question arose while starting to port a project that uses Nox and which has Poetry dependency groups.  I was looking for a way to avoid repeating dependencies from pyproject.toml in noxfile.py.
As an example, the old pyproject.toml defines a type_check group:
[tool.poetry.group.type_check.dependencies]
mypy = "*"
types-toml = "*"
types-setuptools = "*"

The old noxfile.py defines a Nox session with the same name:
@nox.session()
def type_check(s: nox.Session) -> None:
    s.install(".", "mypy", "types-toml", "types-setuptools")
    s.run("mypy", "src", "tests")

Users invoke this session via nox -s type_check to perform the defined action.
I was contemplating ways to avoid repeating the list of dependencies and had come up with a prototype based on a little code in noxfile.py that parses PEP 735 dependency groups in pyproject.toml, allowing something like this:
@nox.session()
def type_check(s: nox.Session) -> None:
    s.install(".", *dep_groups("type_check"))
    s.run("mypy", "src", "tests")

This failed because pyproject.toml doesn't contains the original group name type_check, but instead contains the normalized name type-check.
Thanks for the link to the normalization rules.  I hadn't realized they applied to "extras" and, presumably, to dependency group names as well; I think this answers my original question.
I was imagining a convention where the Nox session name (necessarily without hyphens) would be used directly to lookup the dependency group name, but it's easy enough to normalize the name first using the rules you linked.
That's all I needed to know from this issue; I'll leave it to you to close since it's got a "needs-decision" tag.

Actually the current implementation does not conform to what PEP735 states about dependency group names:

This PEP defines a new section (table) in pyproject.toml files named dependency-groups. The dependency-groups table contains an arbitrary number of user-defined keys, each of which has, as its value, a list of requirements (defined below). These keys must be valid non-normalized names, and must be normalized before comparisons.
Tools SHOULD prefer to present the original, non-normalized name to users by default. If duplicate names, after normalization, are encountered, tools SHOULD emit an error.

If the implementation would stick to the PEP735, the OP question

Should dependency group names be normalized with underscores converted to hyphens?

would be answered "No - dependency group names are preserved as given by user"
and this would be non-issue.
Discussion
Existing implementation by uv has advantages:

key names are predictable
conflict resulting of two groups using non-normalized names with the same normalized name has clear resolution
tools processing pyproject.toml can easily find exact key for any acceptable name and will not miss "My-Group" by trying to access it by "my-group" or "my.group" etc.

Anyway, the PEP states the requirements clearly and there is probably some reason for that. Could Stephen Rosen shed some light on it?

@drmikehenry no problem! Thanks for explaining your use case. Per the standards, that dep_groups function "MUST" normalize names when performing the lookup. So it's a technically a bug there that the group wasn't found.
@vlcinsky to be clear, I don't think we're in violation of the rules of PEP 735

These keys must be valid non-normalized names, and must be normalized before comparisons.

This just means that the keys present in a file must follow the schema described for non-normalized names. It does not specify that we cannot write the normalized name (which is also a valid non-normalized name).

Tools SHOULD prefer to present the original, non-normalized name to users by default.

This means that when presenting names we read from the file, we should present them in their non-normalized form. We probably don't do this, we generally normalize all of the names we read immediately (e.g., for extras and packages too). Regardless, this isn't a "MUST" and I don't think it's necessarily applicable here since we are normalizing a name during a write to the file which is a convenience we provide.
I believe this language matches other Python standards, which generally suggest showing the user the unnormalized name to avoid confusion about what you are referring to. For example, if uv says it "couldn't find package foo defined in group foo-bar" but the group is called "foo.bar" in the pyproject.toml, that could be confusing.
I could be convinced we should not normalize the name when creating a group during uv add, but I'd want to hear more reasons why first.
@sirosen is welcome to weigh in, though there is no obligation.

@zanieb Agreed regarding the requirement to normalize dependency group names for comparison purposes.  I've fixed my dep_groups() function to normalize before lookup (and I'm now also normalizing the dependency group names as they are read from pyproject.toml as well in case of editing by hand or by other tools that don't normalize).

@sirosen is welcome to weigh in, though there is no obligation.

I'm happy to help if there's some ambiguity WRT the spec, but I think you put it all perfectly, so there's not much for me to add! 😁
But I'm happy to listen in and these sorts of threads help educate me more about how the spec is playing out in the wild (and where any mistakes may have been made).


Product popover

https://jira.corp.adobe.com/browse/FYR-1873

Also, please change all quotes around strings from " to '.


Update common/utils and common/primitives to v10.277

Resolves #2172, resolves #2173, resolves #2174, resolves #2175, resolves #2176, resolves #2177, and touches #2166.

utils.mjs.d.ts is non-functional due to issues I tried to resolve upstream in Typescript. For the interested, I tried to fix it, but was not able to find a solution I liked for my PR, so I pushed a draft up here: https://github.com/microsoft/TypeScript-DOM-lib-generator/pull/1379

utils.mjs.d.ts is non-functional due to issues I tried to resolve upstream in Typescript. For the interested, I tried to fix it, but was not able to find a solution I liked for my PR, so I pushed a draft up here: microsoft/TypeScript-DOM-lib-generator#1379

I believe you mean url.mjs.d.ts...?

I believe you mean url.mjs.d.ts...?
Yup. Fried brain and running off for dinner.

Collection also needs to be updated.

Whoops. Somehow got missed when I was diffing.

Right, should be all good for now and updated the only things under common/utils and common/primitives to v10.278


When changing concurrency back to max for households, the individual profiles should be replace by aggregated ones

A group of 33 households should get an aggregated profile:

That's one of the problems I found when trying to implement #273


additional battery attributes

Please add these additional attributes to battery Entity's'
"state_class: measurement"
"unit_of_measurement: %"
Automate Blind "sensor.games_left_battery" requires these.
example device with the states - "sensor.front_door_battery"

Thanks.
Fixed in v0.9.6.
The ever changing HASS API does drive me a bit nuts... Apologies for the delay.

All good, There was no rush on it.. I've actually requested these same attributed from other plugins also :)


EdnError rework

Continuation of #120 with a rebase (should be a lot less noise). Currently a draft for #119

Oops, missed a couple tests that weren't run on my local setup. Fixed.
Please look/think about the new Error type and new usage. As it is right now, only Display and Debug are public. I'll probably make the error code pub too. I'll have to see how the walker impl ends up.
Obviously the TODOs are to do.

btw I started fuzzing with
https://github.com/Grinkers/edn-rs_fuzzer/tree/main
finding lots of things. Errors are also pretty annoying because I'll sometimes just end up with "ParseEdn" (not sure why on this one) or something at "index 2" but it's actually 500 characters deep.

Any other thoughts before doing the implementation?
I'd like to see #122 and #125 finished before progressing, as there's already conflicts.

While looking at clojure's read-string limitations for symbols I ran into
https://github.com/naomijub/edn-rs/blob/bd558559f2b36c7c1c2ba3d24b38c15aaf0fcab5/tests/deserialize.rs#L803
(clojure.edn/read-string "5011227E71367421e12")
java.lang.NumberFormatException: Invalid number: 5011227E71367421e12

I'll be expanding my fuzzer for things that are not valid edn. Here's a couple patterns that it has found that we consider valid EDN but is not.
"#'clojure.generated.ns1/var1"
"#object[clojure.lang.Namespace 0x6dcc40f5 \\\"clojure.generated.ns1\\\"]"

I don't think these issues should block #125, as that's fixing valid edn issues. I think I'll rebase after 125, implement a walker with the most minimal version of EdnError as drafted here (while making sure there's no regressions), then start fuzzing non-valid edn too.
For the new ErrorCode enum, I think I'll loosely try to match the names Clojure's reader gives. https://github.com/clojure/clojure/blob/c6756a8bab137128c8119add29a25b0a88509900/src/jvm/clojure/lang/EdnReader.java


Home manager won't build if no config is set in nixvim

Plugin affected: What plugin does this bug relate to (if applicable)
Nixpkgs channel: 22.11
Home-manager version: 22.11
Description
If I have the nixvim module added in my home-manager config and I do not use any options from nixvim, home-manager will fail to build with an error:
error: The option `programs.nixvim' is used but not defined.
(use '--show-trace' to show detailed location information)

This happened as I removed my config from my files, but didn't remove the module import yet.

This happens with the other ones too... I need to investigate what causes this


fix_all()

fix_all() only fixes the dofs associated to the current Optimizable, and not it's ancestors.
This is confusing when working with, e.g., ScaledCurrent.  Calling fix_all() on a ScaledCurrent does not fix the current at all.
After discussing with @florianwechsung, a proposed change would be to introduce a fix_full() or fix_local() to distinguish between these two use cases.

For other functions and attributes of an Optimizable object, full refers to "including both fixed and free dofs" rather than "including ancestors". See e.g. https://simsopt.readthedocs.io/en/latest/optimizable.html#function-reference. The proposed fix_full would break this pattern so maybe we should find a different name. Looking at the table of other functions, the pattern is to have local_ in front of functions that exclude ancestors and to have no prefix in front of functions that include ancestors. That means the names that would best preserve the pattern would be fix_all() when including ancestors and local_fix_all() when excluding ancestors.
@andrewgiuliani @florianwechsung @mbkumar what do you think?
Sorry I didn't catch this sooner.

Yes, fix_full is breaking the naming conventions employed. @landreman, I like your proposal. An alternative would be fix_global and fix_local.

Fixed in #222

Agree it is confusing. Will issue a fix.
On Fri, Apr 15, 2022 at 3:38 PM Andrew Giuliani @.***>
wrote:

Assigned #215 https://github.com/hiddenSymmetries/simsopt/issues/215 to
@mbkumar https://github.com/mbkumar.
—
Reply to this email directly, view it on GitHub
https://github.com/hiddenSymmetries/simsopt/issues/215#event-6441897949,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA62VEH76JCLF42VJ76DVFLVFHAU3ANCNFSM5TREJACQ
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Bharat Medasani
Engineer
Princeton Plasma Physics Lab (PPPL)


API secret keys in the public source code

Hello,
It seems that there is your API secret Keys in the source code (FranceConnect, Facebook and Google):
https://github.com/promethe42/cocorico/blob/1f422f4d801a30e75f9a46742a0205e7202ee90c/provisioning/inventory/group_vars/image.yml
Please revoke the secret keys ASAP before bad guys use it.
Paul

Yep, I know.
But AFAIK, Google and Facebook keys are useless for apps that are not on the right domain. And you can't change the authorized domains without my admin credentials. Isn't that right?
And it's not the case for FranceConnect. But I sent an e-mail to their tech support and they simply don't care...
The problem is we need at least 1 auth method to be set for the local dev config...
Or should we require every developer to provide their own credentials?

Yep this is correct, so an attacker would also need to find a cross-site scripting vulnerability on the website hosted on the right domain. It is still possible and in any case, the secret key must remain secret to the dev team (It is better if each dev has its own key).
Git(hub) can remove the keys automatically from source code during the commit process : https://help.github.com/articles/remove-sensitive-data.
On the long run, it might be interesting to have a look on Oracles for Ethereum (E.g. http://www.oraclize.it). In a perfect world, France Connect would provide its services directly on the blockchain. I sent them a message on twitter and they replied that the access "is not active".

Git(hub) can remove the keys automatically from source code during the commit process :

Sure. But then when you clone/provision the repo, you won't be able to test/dev on the platform because the API keys are missing... How can we solve that?

On the long run, it might be interesting to have a look on Oracles for Ethereum (E.g. http://www.oraclize.it).

http://dapps.oraclize.it/proof-of-identity/ looks cool. I'll have a look.

In a perfect world, France Connect would provide its services directly on the blockchain.

I want to boostrap a foundation to fund and manage Cocorico as an open and community driven project.
Having an alternative to FranceConnect or making it blockchain-ready could be a good project for this foundation.

Sure. But then when you clone/provision the repo, you won't be able to test/dev on the platform because the API keys are missing... How can we solve that?

Anyone who wants to run the server app should request its own credentials to api providers. I am not sure that it is possible with France Connect so sharing a single api key between the core dev is already a step forward (use a password manager).

Still, the idea behind this could be a good way to break the security/surveillance risk related to FranceConnect. I'll think about it.

I fully agree, IMO there is no better security than public blockchain for such political project.

Looks like you want to contribute! :) Maybe you could join the gitter so that we can talk!

I will join and try to support you but I don't have much free time.


[Backport 7.59.x] [incident-33304] Expect container tags on dogstatsd metrics

Backport 626f570cf380d1c1fc680ccb85934e5abcb24a9b from #32222.

What does this PR do?
Expect container tags on dogstatsd metrics in all scenarios.
Motivation
With the latest improvements of origin detection, the agent is now able to figure out the origin container in all cases.
Describe how you validated your changes
Run e2e tests.
Possible Drawbacks / Trade-offs
Additional Notes
incident-33304

No fixes need to be backported to 7.59.x.


Add CDT & MBRN ics20s to Stargaze

Here are the denom_traces for confirmation
CDT: https://rest.cosmos.directory/stargaze/ibc/apps/transfer/v1/denom_traces/B0263C28B6F44651F4596413B41FDB749EA010BD1220816DAC0ABF9947C1E806
MBRN: https://rest.cosmos.directory/stargaze/ibc/apps/transfer/v1/denom_traces/E94BB144B818CB8061F43E202BEA1E9273B87D6326C8C6F4E6AE71C62FD37854
This should fix the errors from PR #4718

thank you


Code Quality: Fallback to "C:" when var doesn't work

Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Closes #

Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

Opened Files ...
...

This didn't solve the issue


model with two lists

Hello.
I try to create a model with two lists and the ability to toggle between them
I've changed list-default example but got into an issue: the second list shows the number of its items but not showing the items themselves.
The first list is fine

The Second has amount of items and a counter below but items are hidden somehow

Could you please help me to fix it?
Here is the code:
package main

import (
        "fmt"
        "os"

        "github.com/charmbracelet/bubbles/key"
        "github.com/charmbracelet/bubbles/list"
        tea "github.com/charmbracelet/bubbletea"
        "github.com/charmbracelet/lipgloss"
)

type keymap struct {
        Toggle key.Binding
}

var keyMap = keymap{
        Toggle: key.NewBinding(
                key.WithKeys("T"),
                key.WithHelp("T", "toggle"),
        ),
}

var docStyle = lipgloss.NewStyle().Margin(1, 2)

type item struct {
        title, desc string
}

func (i item) Title() string       { return i.title }
func (i item) Description() string { return i.desc }
func (i item) FilterValue() string { return i.title }

type mode int

const (
        one mode = iota
        two
)

type model struct {
        mode    mode
        list    list.Model
        listTwo list.Model
}

func (m model) Init() tea.Cmd {
        return nil
}

func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
        switch msg := msg.(type) {
        case tea.WindowSizeMsg:
                h, v := docStyle.GetFrameSize()
                m.list.SetSize(msg.Width-h, msg.Height-v)
        case tea.KeyMsg:
                if msg.String() == "ctrl+c" {
                        return m, tea.Quit
                }

                switch m.mode {
                case one:
                        switch {
                        case key.Matches(msg, keyMap.Toggle):
                                m.mode = two
                        }
                case two:
                        switch {
                        case key.Matches(msg, keyMap.Toggle):
                                m.mode = one
                        }
                }
        }

        var cmd tea.Cmd
        var cmds []tea.Cmd

        m.list, cmd = m.list.Update(msg)
        cmds = append(cmds, cmd)
        m.listTwo, cmd = m.listTwo.Update(msg)
        cmds = append(cmds, cmd)
        return m, tea.Batch(cmds...)
}

func (m model) View() string {
        if m.mode == two {
                return docStyle.Render(m.listTwo.View())
        }
        return docStyle.Render(m.list.View())
}

func main() {
        items := []list.Item{
                item{title: "Raspberry Pi’s", desc: "I have ’em all over my house"},
                item{title: "Nutella", desc: "It's good on toast"},
                item{title: "Bitter melon", desc: "It cools you down"},
                item{title: "Nice socks", desc: "And by that I mean socks without holes"},
                item{title: "Eight hours of sleep", desc: "I had this once"},
                item{title: "Cats", desc: "Usually"},
                item{title: "Plantasia, the album", desc: "My plants love it too"},
                item{title: "Pour over coffee", desc: "It takes forever to make though"},
                item{title: "VR", desc: "Virtual reality...what is there to say?"},
                item{title: "Noguchi Lamps", desc: "Such pleasing organic forms"},
                item{title: "Linux", desc: "Pretty much the best OS"},
        }

        items2 := []list.Item{
                item{title: "Business school", desc: "Just kidding"},
                item{title: "Pottery", desc: "Wet clay is a great feeling"},
                item{title: "Shampoo", desc: "Nothing like clean hair"},
                item{title: "Table tennis", desc: "It’s surprisingly exhausting"},
                item{title: "Milk crates", desc: "Great for packing in your extra stuff"},
                item{title: "Afternoon tea", desc: "Especially the tea sandwich part"},
                item{title: "Stickers", desc: "The thicker the vinyl the better"},
                item{title: "20° Weather", desc: "Celsius, not Fahrenheit"},
                item{title: "Warm light", desc: "Like around 2700 Kelvin"},
                item{title: "The vernal equinox", desc: "The autumnal equinox is pretty good too"},
                item{title: "Gaffer’s tape", desc: "Basically sticky fabric"},
                item{title: "Terrycloth", desc: "In other words, towel fabric"},
        }

        m := model{
                list:    list.New(items, list.NewDefaultDelegate(), 0, 0),
                listTwo: list.New(items2, list.NewDefaultDelegate(), 0, 0),
        }
        m.list.Title = "My Fave Things"
        m.listTwo.Title = "My Fave Things Two"
        m.list.AdditionalShortHelpKeys = func() []key.Binding {
                return []key.Binding{
                        keyMap.Toggle,
                }
        }
        m.listTwo.AdditionalShortHelpKeys = func() []key.Binding {
                return []key.Binding{
                        keyMap.Toggle,
                }
        }

        p := tea.NewProgram(m, tea.WithAltScreen())

        if _, err := p.Run(); err != nil {
                fmt.Println("Error running program:", err)
                os.Exit(1)
        }
}

@bashbunni Hi. I've watched a couple of your vids and CLI Kanban Board playlist on youtube.
I just don't see any examples using two lists in one model so can't figure out what is my mistake here.
Could you pls help we with this?

Ah that's because you need to set the height and width on both lists to see the items. It looks like it's missing for listTwo. That should fix it!

@bashbunni Thank you.
It somehow wants me to set height/width only on second list. It is perfectly fine with 0, 0 on the first one. Anyway it works =)


Elm no longer supports ' in variable names

This feature was removed in Elm 0.18 in November 2016.
See documentation of change here: https://github.com/elm/compiler/blob/master/upgrade-docs/0.18.md#no-more-primes

Nevermind. I will find someone to do a more comprehensive fix!


Allow independent rendering from delta t

To improve performance allow rendering to be done every few time steps instead of every time step. This will improve performance and decrease delta t

(Alex) Show program working without collision
(Alex) Show the issue
(Matt) Pretend to have completed it
(Alex) Look at the PR and explain what a PR is
(Alex) Code review: .DS_STORE, Collisions Stack not queue
(Matt) Fix the requested changes and re request review
(Alex) Code review: API key
(Matt) Remove API key and re request review
(Alex) Accept Changes
(Alex) Talk about the CI
(Matt) Merge Changes


add logistic regression benchmark based off https://github.com/szilar…

added logistic regression workload
changed partition and sleep workload to output expected values (for testing)
why am i ocd about dependency ordering?
lr conf example..should be expanded.

Codecov Report

Merging #39 into feature/multiple-spark-submit will decrease coverage by 6.03%.
The diff coverage is 14%.

@@                        Coverage Diff                        @@
##           feature/multiple-spark-submit      #39      +/-   ##
=================================================================
- Coverage                          77.24%   71.21%   -6.04%
=================================================================
  Files                                 28       29       +1
  Lines                                479      528      +49
  Branches                              14       15       +1
=================================================================
+ Hits                                 370      376       +6
- Misses                               109      152      +43

Impacted Files
Coverage Δ

...bm/sparktc/sparkbench/workload/ConfigCreator.scala
50% <0%> (-5.56%)
:arrow_down:

...bench/workload/ml/LogisticRegressionBenchmML.scala
0% <0%> (ø)

.../workload/exercise/PartitionAndSleepWorkload.scala
100% <100%> (ø)
:arrow_up:

...a/com/ibm/sparktc/sparkbench/cli/ScallopArgs.scala
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM, merging to multiple-submit and then to develop.


Cleanup unused dependencies

It seems that most of these dev dependencies are actually not used in this project.

wanting is a strong word... :P
done!


Implement more natural human player input.

The comment on top should describe both the problem of the current approach, as well as the new proposed approach, so I won't repeat it here. Code is extensively commented as otherwise it looks needlessly over-engineered, but it isn't.
The magic numbers used (0.2 and 0.1) are what I remember using in past games, but could be tweaked. For example, my linux uses 0.66 and 0.025 (check yours using xset q | grep 'auto repeat delay') but I felt it was too delayed for a game.
Getting user input feel "natural" is actually nontrivial (I know from experience) and super-simple code almost never feels right. I know it wasn't a priority for this project, but it felt frustrating play-testing it and losing to SimpleAgent because of input issues, not skill. With this patch, I'm still losing regularly, but at least now it feels like I'm losing because of my poor skill :smile: I also play-tested it in multiplayer with my wife and it works nice.
Note that this PR comments-out the pause action since it crashes the game (as noted marked in a comment before my changes already) and we hit it several times by accident during play. I can undo that and leave it in if you prefer.

Left some comments in the PR. Thanks for this; I really appreciate the detailed comments.

Thanks for the comments! I addressed them all and rebased onto latest master, but I can't test it yet due this, so I'll hold off pushing the changes until then.

Addressed #48 --> ping to test this.

OK, rebased on master, solved conflicts, and addressed all your comments.

lgtm!


Update LICENSE

Happy 2015! :)

Coverage remained the same when pulling 8ef249c0854a4a071e58af70a883c384a1467e90 on illusionld:patch-1 into 838df74a5d1975dbcedfe45b9bf2108128d7cb88 on schmich:master.

Haha, that was fast, thanks! Happy new year!


Pool Parameters (Contracts)

Currently based on open PR #49 so must not be merged yet.

contracts: Emit missing PoolDeployed event
contracts: Add minimum stake amount and duration parameters

@wottpal made me aware of an edge-case in which the logic should be extended: a reset of the min. duration to 0.
If the min. duration was > 0, a supporter stakes, and then the min. duration is reset to 0, if that user stakes again, the lock timeout is not reset and the user cannot unstake, either the old or new stake, until the old timeout has passed. This is not consistent because, if the min. duration was reset to a very low duration, but not 0, then staking any amount would reset the lock timeout using this new low duration.
I see two solutions:

Extend stake with an additional check

if (minDuration == 0 && lockTimeout[supporter] > now) {
    timeout = 0; lockTimeouts[supporter] = 0;
}

Note that only setting the timeout to 0 in case the current lock is in the future (instead of just checking > 0) is just a gas-optimization.

Extend unstake to skip the timeout check if minDuration > 0.

Solution 1 seems cleaner on first sight, as it makes sure lock timeouts are always set at staking time, which is more consistent. Also, there's an even edgier edge-case if only Solution 2 was implemented: if someone topups their stake while minDuration is 0, then the minDuration is increased again, that supporter would again not be able to access their stake, if an old lock is still set.
On the other hand, if only Solution 1 was implemented, any supporter could get their timeout reset anyways when the minDuration is 0 by just staking any (or the min.) amount and then immediately unstake. So to avoid forcing users to do this detour, it also makes sense to also implement Solution 2.
In short, it makes sense to do both extensions of the current logic.

In short, it makes sense to do both extensions of the current logic.
Makes sense and I agree. Thanks for this detailed explanation.

Just realized that a simpler version of S. 1 is to always set the lockTimeouts[supporter] storage to 0 if minDuration == 0 and to now+minDuration otherwise, because resetting a storage slot from 0 to 0 just costs as much as loading it (SLOAD), see https://eips.ethereum.org/EIPS/eip-2200 for more info. Will do it like that because simpler logic is always better :)


feature request: implement addScopes method

you need to take care of the api for future:
https://react-native-google-signin.github.io/docs/original#addscopes
@riderx It's also a great project for inspiration

hi @riderx i mean add method addScopes:
// draft

const res = await GoogleSignin.addScopes({
  provider: 'google',
  scopes: ['https://www.googleapis.com/auth/user.gender.read'],
});

We use credential manager, the new library who use the webID it doesn't seems to have scope available in this setup
https://developer.android.com/identity/sign-in/credential-manager-siwg

@riderx it seems
https://developers.google.com/identity/sign-in/android/additional-scopes -> https://developers.google.com/identity/authorization/android

This is the deprecated SDK we don't use it.
For now, the new doc is a mess and misses many things, like login users who don't have account on the mobile won't work.
Here is the whole doc if you want to look for it:
https://developer.android.com/jetpack/androidx/releases/credentials
I wish I missed it

@riderx good, maybe reopen issue?

I believe that using addScopes MIGHT be possible using authorization

@reslear I found an example repo: here.
I was able to implement a small PoC with the addScopes. I believe that it can be implemented

Great catch !

I think scopes are working now.
"@capgo/capacitor-social-login": "^0.0.68-alpha.7",
https://github.com/Cap-go/capacitor-social-login/issues/46#issuecomment-2470064711


Vocabularies: add DCAT2 constants

A new version of DCAT (v2) became available, so it would be helpful to add new properties and classes to the org.eclipse.rdf4j.model.vocabulary.DCAT convenience class
https://www.w3.org/TR/2019/CR-vocab-dcat-2-20191003/
This would also require a similar class for W3C ODRL, PROV and TIME vocabularies.
(which are also useful outside a DCAT-context)

https://github.com/pwin/dcat-v2-xml-schema is a preliminary draft that might also be useful

Thanks, I started generating the classes for DCAT and other (ODRL, PROV...) ontologies


Throw new Error instead of string

enb assumes "throw 'My Error'" as success result.

Cool!

Thanks!


SIGPIPE を無視する

SIGPIPE だけじゃないか
node:events:368
      throw er; // Unhandled 'error' event
      ^

Error: write EPIPE
    at afterWriteDispatched (node:internal/stream_base_commons:164:15)
    at writeGeneric (node:internal/stream_base_commons:155:3)
    at Socket._writeGeneric (node:net:795:11)
    at Socket._write (node:net:807:8)
    at writeOrBuffer (node:internal/streams/writable:389:12)
    at _write (node:internal/streams/writable:330:10)
    at Socket.Writable.write (node:internal/streams/writable:334:10)
    at Stringifier.ondata (node:internal/streams/readable:754:22)
    at Stringifier.emit (node:events:390:28)
    at addChunk (node:internal/streams/readable:315:12)
Emitted 'error' event on Socket instance at:
    at Socket.onerror (node:internal/streams/readable:773:14)
    at Socket.emit (node:events:390:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -32,
  code: 'EPIPE',
  syscall: 'write'
}

fix in c8be0621849428b81ea2ee41bd0df2048e7054d0


crash on allocated size greater than the recommended max working set size

When trying to load a large context window ollama crashed due to llama.cpp throwing an exception:
size =   160.00 MB, (12018.69 / 10922.67), warning: current allocated size is greater than the recommended max working set size
ggml_metal_graph_compute: command buffer 0 failed with status 5
GGML_ASSERT: ggml-metal.m:1177: false
SIGABRT: abort
PC=0x1a0f58724 m=10 sigcode=0
signal arrived during cgo execution

To reproduce:

download a 16K model

FROM llongma-2-7b.ggmlv3.q4_0.bin
PARAMETER num_ctx 16000
TEMPLATE """
{{ .Prompt }}
"""

Specs:

This should be fixed in https://github.com/jmorganca/ollama/releases/tag/v0.0.18 but feel free to re-open otherwise!


Incorrect This code is unreachable warning

With #1948, we can call functions (that can be evaluated statically) when initializing constants. Unreachable code analysis is unaware of this and ends up throwing an incorrect warning.
script;

struct A {
   a : u64
}

fn foo(a : u64) -> A {
  A { a }
}

const B : A = foo(32);

fn main() -> u64 {
    B.a
}

produces:
 --> /home/vaivaswatha/c/working/sway2/src/main.sw:8:3
  |
6 |
7 | fn foo(a : u64) -> A {
8 |   A { a }
  |   ------- This code is unreachable.
9 | }
  |
____

Are the following issues related?
https://github.com/FuelLabs/sway/issues/705
https://github.com/FuelLabs/sway/issues/1914
https://github.com/FuelLabs/sway/issues/1258

They might be related, I can take a look once I get some cycles to look into this.


Scrollbar regression on Chrome and Edge

The styling added to scrollbars seems to have reverted with newer versions of Chrome and Edge.
https://itwin.github.io/iTwinUI/react/?story=table--basic
Chrome

Edge

Firefox

Disabling the scrollbar-color and scrollbar-width added by iTwin fixes the issue.

Looks like this is a known regression in Chrome 121. (Or maybe not a regression, but a change which causes us problems.)

Not a regression, this is just how scrollbars will look from Chrome 121 onwards.

Actually there is one part we should address: the table header should not display the track buttons, because there will never be a scrollbar there.

@HaveSpacesuit I cannot reproduce this issue on Windows. Where/how are you seeing scrollbar tracks in the table header? It looks fine for me:

Now fixed in 3.3.4 🎉 Thanks for reporting this.

Before
After

Can we put a fix on v2 as well?

Backported to 2.12.22 👍


test change

Description
Brief description of changes
Risks
Potential areas of concern that you would like looked at more closely
Validation
How you verified that these changes work
Issue
Link to corresponding issue
Operating System
If you're on Mac, please include whether you are on an Intel or M-series CPU

/gcbrun


Implement Undefined Variable Rule

Closes  #65
I wrote new module based on #64.

Would love to see this implemented!

@raa0121 thanks for the PR.

thank you for merging.