CapsLock turns on when pressing combination Win+Ctrl with mapping of CapsLock->Ctrl

Microsoft PowerToys version
0.85.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
I have mapped left Ctrl to CapsLock which means that in place of CapsLock have left Ctrl and left Ctrl is left in place.
When using mappings connected to virtual desktops Win+Ctrl+ sometimes CapsLock (although it is mapped out to oblivion) turn on. But I effectively don't have Caps Lock so I have to manually turn off Keyboard Manager Toy, use CapsLock to turn it off and turn on Keyboard Manager.
✔ Expected Behavior
Don't turn on Caps Lock when it is mapped out of keyboard.
❌ Actual Behavior
No response
Other Software
No response

/bugreport

PowerToysReport_2024-10-08-15-26-04.zip

HAPPENED AGAIN, BUG REPORT ATTACHED

I checked logs. Something is wrong in shortcut remappings file. Can you share "default.JSON" file under "%localappdata%\Microsoft\PowerToys\Keyboard Manager" folder.

default.json

Hi @vvizzo
I can't reproduce this issue. I'm still trying to identify possible problems and find a solution by reviewing the code.
This might be related to keyboard language layout. What is your keyboard language layout? Are you making any changes while using KBM? And finally, if you encounter this problem again, let me know in which application you encountered it and which steps you followed and I will try to reproduce it.
Thank you! /needinfo

Kbd layout is Polish programmer layout.
This happens mostly when switching between virtual desktops and I hold longer Win+Ctrl combination.


Add a configuration key that actually can make the logger tracer act as such

I believe there is no easy (not eval or serialization please) way to pass an object through a configuration file.
So instead of trying that feat, Adding a "log_tracing" so we use logging if other methods are not available.
Also moved the format to be json which should be easier to parse.
@jfeltesse-mdsol

@jfeltesse-mdsol pushed requested changes.

LGTM. Don't have merge powers though.

with great power comes great responsibility.. do you want it? :)

@adriancole I guess I'm good since Jordi has The Power and we're sitting 2 desks away 😉
Thanks though!


ML Translation

uncompleted, but I want to push what I have so far

Code looks good. Can you add a screenshot or a demo of some sort?


Enabled use of MSVC 1900 patches for VS2017+ compilers

Microsoft Visual C++ 2017 or greater does not increment _MSC_VER to 2000 (2017) or 2100 (2019), they rather increment them to 1901 and 1920.
This patch allows those compilers (which should be compatible with vs2015 binaries as far as I know) to use the patches from VS2015 (MSVC 1900).

Thanks for the contribution :+1:


@type/node has updated. it's fail for my your project

Hi!.
I have a project Angular4. https://travis-ci.org/AngularVN/angular4-seed/jobs/237435131
It's not work in 7days.

https://stackoverflow.com/questions/44196321/type-node-error-angular-4-on-systemjs-with-systemjs-builder

It's work again


[python] Rename NdArray -> NDArray

Conforms with spec update https://github.com/single-cell-data/SOMA/pull/55

Codecov Report
Base: 64.06% // Head: 74.74% // Increases project coverage by +10.68% :tada:

Coverage data is based on head (c18d6ec) compared to base (fc7d239).
Patch coverage: 88.09% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #608       +/-   ##
===========================================
+ Coverage   64.06%   74.74%   +10.68%
===========================================
  Files          61       26       -35
  Lines        3225     1865     -1360
===========================================
- Hits         2066     1394      -672
+ Misses       1159      471      -688

Flag
Coverage Δ

python
74.74% <88.09%> (ø)

r
?

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

Impacted Files
Coverage Δ

apis/python/src/tiledbsoma/collection.py
87.70% <ø> (ø)

apis/python/src/tiledbsoma/dataframe.py
99.35% <ø> (ø)

apis/python/src/tiledbsoma/util.py
90.36% <ø> (ø)

apis/python/src/tiledbsoma/io.py
65.76% <73.33%> (ø)

apis/python/src/tiledbsoma/measurement.py
97.22% <91.66%> (ø)

apis/python/src/tiledbsoma/__init__.py
72.72% <100.00%> (ø)

apis/python/src/tiledbsoma/dense_nd_array.py
93.61% <100.00%> (ø)

apis/python/src/tiledbsoma/factory.py
87.23% <100.00%> (ø)

apis/python/src/tiledbsoma/sparse_nd_array.py
95.03% <100.00%> (ø)

inst/tiledb/include/tiledb/config.h

... and 34 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

FYI, this breaks (changes) the on-disk format, immediately rendering old data incompatible.  Any way we can make it backwards compat (ie. look for a soma metadata key of NdArray or NDArray?)

https://github.com/single-cell-data/TileDB-SOMA/pull/610


sync with master

v22.2.1 run tests against SQLite

[Current coverage][cc-pull] is 100%

Merging [#589][cc-pull] into [v22.2.2][cc-base-branch] will not change coverage

@@           v22.2.2   #589   diff @@
=====================================
  Files           28     28
  Lines          439    439
  Methods        111    111
  Messages         0      0
  Branches         0      0
=====================================
  Hits           439    439
  Misses           0      0
  Partials         0      0

Powered by Codecov. Last update [25739ba...aad9da4][cc-compare]
[cc-base-branch]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/branch/v22.2.2?src=pr
[cc-compare]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/compare/25739ba27d2ccd5a81611f3d3cfaedcde49e1109...aad9da4f5576974741a4b2902744cbeb0436ec20?src=pr
[cc-pull]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/pull/589?src=pr


[Bug][GitHub] Missing Pull Request data when changing the Time Range in Sync Policy

Search before asking

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

What happened
Some users set up a new project whose connection is GitHub with onboarding guide,  because https://github.com/apache/incubator-devlake/issues/7703, graphql API is not enabled.This produces two records in pull_requests table.
When users update the time range in project's sync policy config, and click COLLECT DATA. The new pipeline is triggered  and finished successfully.
But the data  in pull_requests table is incorrected: it still has only two records, but the new time range has more pull requests definitely.
What do you expect to happen
When Time Range option in Sync Policy Config changes, all pull requests should be fetched according to the new time range.
How to reproduce
Ses "what happened"
Anything else
No response
Version
2e56bdd
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I think  it's related to GitHub's PR collector, but due to the small rate limit in GitHub restful API. We should use graphql as much as possible.


feat: use ha distance unit

I ran into a chat yesterday on talking about the UK using both celsius and miles. This was in the home assistant community thread.    I know I was originally the one thinking we should use the HA setting.     I wonder if that was the reason to only override distance?

Hey, you can default to HA settings but do not remove existing functionality

closing, new solution in https://github.com/fuatakgun/kia_uvo/pull/149 seems an odd use case but what do I know I'm stuck using imperial units ...


NFS v4 Locking Leads to EBADF on Initial Startup

Hi,
I am trying to get etcd (v2.3.6) to run with its data dir on an NFS v4 mount. The directory is empty.
This however fails with the error: etcdserver: open wal error: bad file descriptor.
This applies at least also to etcd v2.3.1.
An strace seems to indicate, that etcd is trying to get an exclusive lock on a file, which has been opened in read-only mode, which is apparently not possible on NFSv4, since it is emulated with fcntl, which requires write rights.
But for me it looks strange, as I read the etcd code, the only place where it tries to get a lock is when
it opens a file in write-only mode, while in the part where it opens it for reading, no lock seems to explicitly acquired.
The strace is as follows:
openat(AT_FDCWD, "/var/lib/kubelet/pods/aee1fed5-2895-11e6-ac84-d2f798530d4e/volumes/kubernetes.io~nfs/pv-020/member/wal", O_RDONLY|O_CLOEXEC) = 8
getdents64(8, /* 3 entries */, 4096)    = 112
getdents64(8, /* 0 entries */, 4096)    = 0
close(8)                                = 0
openat(AT_FDCWD, "/var/lib/kubelet/pods/aee1fed5-2895-11e6-ac84-d2f798530d4e/volumes/kubernetes.io~nfs/pv-020/member/wal/0000000000000000-0000000000000000.wal", O_RDONLY|O_CLOEXEC) = 8
openat(AT_FDCWD, "/var/lib/kubelet/pods/aee1fed5-2895-11e6-ac84-d2f798530d4e/volumes/kubernetes.io~nfs/pv-020/member/wal/0000000000000000-0000000000000000.wal", O_RDONLY|O_CLOEXEC) = 9
flock(9, LOCK_EX|LOCK_NB)               = -1 EBADF (Bad file descriptor)
write(2, "2016-06-02 08:21:40.424460 C | e"..., 792016-06-02 08:21:40.424460 C | etcdserver: open wal error: bad file descriptor
) = 79

Thanks for your attention,
Fabian
P.S: It seems also odd to me, that the file is opened twice.

@fwiesel is this on a linux kernel <3.15? etcd needs OFD locks for NFS
a lock call should look like:
fcntl(7, F_OFD_SETLK, {l_type=F_WRLCK, l_whence=SEEK_SET, l_start=0, l_len=0}) = 0

it's on coreos stable

@SchwarzM @fwiesel OK, I see the problem. The OFD support for NFS locking didn't make it to 2.3.x, but it's in 3.0-beta.

Ah, I see, we will give it as spin. Thanks for the quick response.


Crates that are fetched but not added to the index appear to be lost

This is a slightly different issue than #628 - we record the failure internally, it won't be lost, we just don't display it anywhere but in the logs. So it seems from the outside as if the build was never there, but we can still rerun it with UPDATE queue SET attempts = 0;
Originally posted by @jyn514 in https://github.com/rust-lang/docs.rs/issues/628#issuecomment-660687177

The crate I originally noticed this for was zbus_macros 1.0.0, which failed to fetch due to a dependency on a yanked crate.

Looks like #797

Yes, this looks like the same issue. I was looking for it but couldn't find it.


PATENTS and LICENSE and competition with core Google products?

hi there
don't want to come across as a troll here but I'd like a plain English language interpretation of the PATENTS and LICENSE documents included in Dart Lang SDK
https://github.com/dart-lang/sdk/blob/master/PATENTS
https://github.com/dart-lang/sdk/blob/master/LICENSE
hypothetically, could I use Dart Lang SDK and Angular Dart to make a competing product to Adwords / Adsense ?
do I then risk losing my license by "inducing infringement" ?
I know quite a few companies chose not to use React JS because of the PATENTS clause and although FB has different language, there seems to be a similar issue being addressed
thanks

Hi, open source attorney for Dart here.

You may use our stuff.
Give us credit by replicating our license files in any distributions.
Don't sue us over patents related to this, or we'll terminate any patent licenses that may or may not attach to this product.
Feel free to make competing products. But don't claim you are Google, or are endorsed by Google.

hi Max
thanks for hosing me down so quickly, dont want to be a FUD merchant
2 follow ups;

can I use Dart in an AGPL licensed product ?  (I see EdX is AGPL for instance )  is the current Dart license and patents addition compatible?

also, why are different Dart projects including the PATENTS file and some not? I am not encouraging you do add them by the way, I'd actually find it less emotionally laborious and a simpler story to sell to a business decision maker if there where no PATENTS files at all

Sure.
I'll investigate and try to make this more consistent. Where present, Dart PATENTS files give you an extra explicit patent license from Google over the project, so nothing to be scared of.

Thanks. But what patents are relevant?
There is no patent for Dart Lang ?
On Mar 24, 2017 9:22 AM, "Max Sills"<EMAIL_ADDRESS>wrote:

Sure.
I'll investigate and try to make this more consistent. Where
present, Dart PATENTS files give you an extra explicit patent license from
Google over the project, so nothing to be scared of.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/29103#issuecomment-289070924,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbsplq8TAkA38_kj9ui5vCjux5LIrn5ks5ro-3dgaJpZM4MhEkp
.

I think license language doesn't treat each individual patent.
And, existing patents at any point of time are not the matter, because patents will granted and expire at some point in the future.

@sillsm - is there more to do on this issue? Can it be closed?

Sorry to be blunt. But I dont trust Google Inc to honor the spirit, if not
the law, of this arrangement.
Just have to look at search or censorship or Android as psuedo open source.
Very sad. Close the issue. I for one will never advocate a Google sponsored
open source project
On Sat, Sep 1, 2018, 6:01 AM Cat-sushi<EMAIL_ADDRESS>wrote:

I think BSD+PATENTS License of Dart is totally different old license of
Facebook.
So, I believe there is no problem here, regardless whether dart has
patents or not.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/29103#issuecomment-417858160,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbsplg9XGUxCi813UPO2NPCUrgjeqh_ks5uWoUSgaJpZM4MhEkp
.

@niccolox What do you think is the problem?
Apache License 2, which also has patent clauses, is widely recognized as a open source license.


[Basic] Add a CacheableSequence.

This adapts an arbitrary sequence to one which can be consumed multiple
times.

I'm open to suggestions on a better name.

MemoizedSequence could be an alternative but I am fine with either.

@swift-ci please smoke test


Host project with type providers

Hi, first of all thank you for this library, it's really great!
I'm trying to mock some project with scripting capabilities using this package, but I've run into an issue.
My host application manages a connection to a PostgreSQL database, for which I choose to use FSharp.Data.Npgsql type provider.
Sadly after adding this dependency to my project, the compiling of the scripts no longer works.
I get these errors:
["startup (1,1)-(1,1) parse error The type provider '{project_path}\bin\Debug\net5.0\FSharp.Data.Npgsql.dll' reported an error: Assembly attribute 'TypeProviderAssemblyAttribute' refers to a designer assembly 'FSharp.Data.Npgsql.DesignTime' which cannot be loaded from path '{project_path}\bin\Debug\net5.0\FSharp.Data.Npgsql.DesignTime.dll'. The exception reported was: System.IO.FileNotFoundException - Could not load file or assembly '{project_path}\bin\Debug\net5.0\FSharp.Data.Npgsql.DesignTime.dll'. The system cannot find the file specified.";    "startup (1,1)-(1,1) parse warning Referenced assembly '{project_path}\bin\Debug\net5.0\FSharp.Data.Npgsql.dll' has assembly level attribute 'Microsoft.FSharp.Core.CompilerServices.TypeProviderAssemblyAttribute' but no public type provider classes were found"]
I tried to understand the issue and from what I saw it's trying to load the design time component of the type provider described by the TypeProviderAssemblyAttribute assembly attribute specified inside FSharp.Data.Npgsql.
Do you have any idea on how should I approach this?
Thank you in advance!

@artipo does it work fine in dotnet fsi ? Type providers do lot of magic under the hood so I am not sure how difficult to solve the problem is. Could you add a repro so I can take a look?

Sure thing!
I thought it could be a FSharp.Data.Npgsql's problem, so I've replicated the issue using the JsonProvider from FSharp.Data.
#5.zip
I wasn't too specific in my previous post, but the script does not use the TP directly.
I don't know if it matters, but the FSharp.Data.Npgsql sln has two project, with the runtime one referencing the design one.
Thank you for your time.

You can try disabling including host entry assembly in fsc-host compiler options like:
    let getScriptProperties () =
      File "script.fsx"
      |> CompilerHost.getMember { Options.Default with Compiler = { Options.Default.Compiler with IncludeHostEntryAssembly = false }}
        (Member<string>.Path "Script.message")

Then it won't try loading the offending design time dll.

I might consider making false the default as the syntax completion doesn't work very well in the script if using any assembly referenced by the host project.

I saw that option, sadly I cannot set it to false, since my script is designed to enable custom behavior.
I have a service defined in the host app:
type IMyService =
    abstract member GetMyObj : Guid -> Task<MyObjDto option>
    abstract member GetMyObjs : unit -> Task<MyObjDto seq>
    abstract member CreateMyObj : CreateMyObjDto -> Task<Result<MyObjDto, Error>>

That get used inside the script by some custom code:
let worker: Worker =
        fun (service : IMyService) logger ->
            async {
                logger.Verbose("Script worker initiated")

                [...custom code...]
            }

In my scenario it's not the end of the world, I can just fall back to Npgsql directly.
But not being able to use Fsc-host in conjunction with FSharp.Data seems a bit more concerning.
I've "solved" the syntax completion issue creating a paket references load script and by loading it during interactive only
#if INTERACTIVE
#load "./references.fsx"
#endif

In reality this does not works, since fsc tries to execute #load ... anyway.
So I load the script as string and then regex out that part
let scriptText = File.ReadAllText "script.fsx"

let scriptText =
    Regex.Replace(
        scriptText,
        @"#if\s+INTERACTIVE.*#endif",
        "",
        RegexOptions.Singleline
        ||| RegexOptions.IgnoreCase
    )

let export =
    scriptText
    |> Inline
    |> CompilerHost.getScriptProperty
        options
        (Property<IMyService -> ILogger -> unit Async>.Path "Script.export")
    |> Async.RunSynchronously

Anyway, if there is anything I can do to help in the TP issue I will gladly try to.

Ok, I see. I've got a better understanding now. I'll need some time to play around.

In reality this does not works, since fsc tries to execute #load ... anyway.

I guess this is a bug. I'll look into it too.

I think we might be hitting https://github.com/fsprojects/FSharp.Data/issues/647 here.

Yeah, that make sense. So a workaround is to copy the .dll manually, I will give it a try.
I think the Issue can be closed, since it's obviously not yours library fault.
Either way, thank you very much!


Add Nhost auth example to the playground

Adds an example with Nhost Auth to the playground

This is looking good! I've approved the PR, but let's wait with merging until the implementation has landed in a RW release.

@Tobbe the Nhost PR has been merged into main. How would you like to handle the timing for merging this?

@thedavidprice I'd like to wait until a proper release of RW has been made, so we can upgrade the version of RW used in the playground to something that supports NHost before merging

@Tobbe did you want me to look into getting this integrated with the latest updates?

@Tobbe did you want me to look into getting this integrated with the latest updates?

Yes please, that'd be awesome

@Tobbe this might be a noob question on my part but how do I push the changes to this PR? I've got them all set locally but it's blocking me from pushing

@Tobbe this might be a noob question on my part but how do I push the changes to this PR? I've got them all set locally but it's blocking me from pushing

You probably can't :(  Only project maintainers can usually do that. Easiest is to just do a new, separate, PR. Mention it here when it's up, so we get cross-links. Thanks!


try something

About the PR

maybe fix it??
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

no cl no fun

who writes lerp that can throw an error what

who writes lerp that can throw an error what

my pet theory is that lerp is overflowing float and causing a negative value to get set for the time which could potentially cause problems for when the mission is supposed to end.

would rather know for sure what it is so someone please give more info.

ok this isn't it we all go to hell ig


Release phan 0.7.0

It's been quite some time since a phan release, and there are lots of good changes. In particular phan 0.6.0  can't finish analysis on our codebase, where dev-master does. Would be nice to prep a new release version so CI infrastructure can point to something specific rather than a random git hash or dev-master.

You're right. I just cut 0.7: https://github.com/etsy/phan/releases/tag/0.7
Sorry about the delay. Enjoy!

Thank you!


NoExportPreprocessor runs cells that are exported

The docs for NoExportPreprocessor say "... executes cells that are not exported ..." but exported cells are run during testing.

nbdev_test_nbs --fname 70_test_no_export_preprocessor.ipynb
testing: /home/peter/github/pete88b/decision_tree/70_test_no_export_preprocessor.ipynb
...
Exception: Expected [] in `test_log` but found ['nbdev_export', 'nbdev_export_and_show', 'nbdev_export_internal']

Should NoExportPreprocessor be updated to not execute exported cells? or do we just need to update the docs?

I didn't update the documentation, it does run the exported cells: at first I was just not executing them and importing the default_exp module, but the problem is that cell exported can be exported in other modules so I didn't manage to have everything.
Thus it's just easier to execute everything except the cell marked with some test flags.
Will update the doc.


build: Enable documentation export to generated Swift/Obj-C code

Overview

Enable documentation to be exported to the generated Swift/Obj-C code.
Fixes #
Checklist
My PR contains...

[x] No code changes (changes to documentation, CI, metadata, etc.)
[ ] Bug fixes (non-breaking change which fixes an issue)
[ ] Improvements (misc. changes to existing features)
[ ] Features (non-breaking change which adds functionality)

My changes...

[ ] are breaking changes
[x] are not breaking changes
[ ] If yes to above: I have updated the documentation accordingly

Documentation

[x] My changes do not require a change to the project documentation
[ ] My changes require a change to the project documentation
[ ] If yes to above: I have updated the documentation accordingly

Tests

[x] My changes can not or do not need to be tested
[ ] My changes can and should be tested by unit and/or integration tests
[ ] If yes to above: I have added tests to cover my changes
[ ] If yes to above: I have taken care to cover edge cases in my tests

Code Coverage

Total Project Coverage
91.66%
:white_check_mark:

:tada: This PR is included in version 1.2.11 :tada:
The release is available on:

v1.2.11
GitHub release

Your semantic-release bot :package::rocket:


Added repo and licence to package.json

Current coverage is 87.44% (diff: 100%)

No coverage report found for develop at fd203f1.

Powered by Codecov. Last update fd203f1...30b75be

+1


Credentials and impersonalize

Hi.
I have problems assigning the Suppressed credentials.
`
            WindowsIdentity identity = (WindowsIdentity)HttpContext.User.Identity;
            WindowsIdentity.RunImpersonated(identity.AccessToken, () =>
            {

                Uri uri = new Uri(@"\\SVWSIOPRDAPP01\colibri$\Agente\AgenteColibri.config");
                FileWebRequest fw = (FileWebRequest)FileWebRequest.Create(uri);
                fw.Credentials = System.Net.CredentialCache.DefaultNetworkCredentials;
                FileWebResponse fr = (FileWebResponse)fw.GetResponse();
                Stream reader = fr.GetResponseStream();
            });`

they can guide me

Thanks for contacting us. Please provide more details about your scenario, what you're trying to do and what's you're having trouble with.

A netcore WebApi. with integrated authentication. What other information do you need?

@shirhatti is this something you can provide guidance on?

What trouble are you actually seeing? Are you getting errors?
Why are you attempting to use a webrequest to write to a UNC share?

It is a system with windows authentication and I want to modify a file that is in a share that only certain users have access to.
That's why I need to log in with the credentials of the logged in user.
but these credentials go blank.

For example, when I execute this command outside the impersonation it creates a file where the owner is usrdelegadesa.
FileStream fileStream = new<EMAIL_ADDRESS>FileMode.Create);
but when I run it inside the impersonalization I don't have access.
                 WindowsIdentity identity = (WindowsIdentity)HttpContext.User.Identity;                 WindowsIdentity.RunImpersonated(identity.AccessToken, () =>                 {                     FileStream fileStream = new<EMAIL_ADDRESS>FileMode.Create);                 });
and what I want is to take the credentials of the user loguedo to execute this command


It doesn't work at all

Am I doing something wrong or is it just not working? My steps are presented in pictures

The default regex matches // TODO: something

ok, I didn't noticed that I have to write ':' after 'TODO' Thank you so much!


Error: Illegal argument: line must be non-negative

I have been seeing this error in the logs of vscode for sometime. Eventually I tracked it down to the client side of the lsp. They explained that negative line numbers are not allowed in the LSP and told me it must be coming from the server.
I did a search and it seems like we assign -1 to lots of places. I stumbled across this method here. Which seemed like it could be a potential place to fix this.
I really struggled in debugging this. I first tried printing the values out if they were less than 0, but since the LSP communicates by stdout/stdin that would just crash the client =/. I then tried attaching a debugger by commenting out the line in the client extension but that didn't see to do anything after it connected (break points didn't work etc.). Any debugging tips would be greatly appreciated.
Eventually I just set any line or column numbers less than 0 to 0 and that made my errors go away. I havent submitted a PR because I wasnt sure this was the best place to fix this issue. I also wasn't sure if and other IDE's were relying on this behavior. Let me know if I should submit a PR with my quick and dirty fix or if you have better idea where this could be handled.

I first tried printing the values out if they were less than 0, but since the LSP communicates by stdout/stdin that would just crash the client =/

System.err will work.

"groovy.trace.server": "verbose" also helped.
I'm using a fix for this in my fork. https://github.com/DontShaveTheYak/groovy-language-server/commit/10f68522083e07f8bc6a978e1894392ee68188c6
If you want this here, then let me know and I will open the PR. I'm just not confident this a real fix, seems more like a bandaid.


Delete a word based on space separator

VSCode Version: 1.16
OS Version: Ubuntu 16

Steps to Reproduce:

place the cursor middle of a word. I can delete left half of the word or right half of the word but can not delete the complete word at once. like if I would like to delete a line i can do that from anywhere the cursor is on the line. but for word i would like to delete a word from anywhere the cursor is on the word.

Reproduces without extensions: Yes/No

@khaled-hossain-code The only workaround now is to select the word ctrl+d and delete it.

Is it solvable with macros?
settings.json
"macros": {
    "deleteWholeWord": [
        "editor.action.addSelectionToNextFindMatch",
        "deleteRight"
    ]
}

keybindings.json
{
    "key": "ctrl+shift+delete",
    "command": "macros.deleteWholeWord",
    "when": "editorTextFocus && !editorReadonly"
}

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Add HEADERS_NO_MOC, HEADERS_GLOB_NO_MOC, HEADERS_GLOB_RECURSE_NO_MOC options for excluding some files from processing by moc

I've dune some timings.
I've just run current master's version and the one from this PR one by one.
It seems that we have achieved some speed up:

Facelift without extra moc processing
Current Github version

1:27
1:48

1:03
1:05

1:20
1:47

1:30
1:08

1:10
1:56

0:49
1:07

1:03
1:23

1:17
1:33

0:52
1:01

we achieve less then 1 minute  time(last four rows) after today's last push (with HEADERS_CAN_MOC_WILDCARD, HEADER_NO_MOC_WILDCARD and deleting one more HEADERS cycle before moc processing  )

My suggestion was, and still is:
you can let the code generator generate a new file containing the following:
set(GENERATED_HEADERS_MOC
File1.h
File2.h
....
)
set(GENERATED_HEADERS_NO_MOC
File3.h
File4.h
....
)
set(GENERATED_SOURCES
File1.cpp
File2.cpp
....
)
In order to identify which files should be MOCed, the code generator can search for "Q_OBJECT" in the files.
Then, the implementation of the "facelift_add_interface" function can use the "include" command (https://cmake.org/cmake/help/v3.0/command/include.html) to get the definitions of those variables and call the facelift_add_library with those file lists.


[Enhancement][data-prepper] Support configurable image.registry

Is your feature request related to a problem? Please describe.
Many organizations do not pull images from docker hub, but always thorugh a private registry (cache). Now, we have to modify image.repository to something like my.registry/opensearchproject/data-prepper.
Describe the solution you'd like
Follow the pattern from the opensearch char and offer a dockerRegistryvalue:
global:
  # Set if you want to change the default docker registry, e.g. a private one.
  dockerRegistry: ""

Describe alternatives you've considered
None
Additional context
None

[Triage]
Thanks @janhoy I agree with we should support custom dockerRegistry, can you please contribute to the fix?
@peterzhuamazon @TheAlgo

@prudhvigodithi Did you have a chance to look at my PR?


SITL Extension: Make drones reset position when sim stopped

Currently the drones don't reset to their origin position.

Fixed in #68


Surface any EDQUOT|EIO|ENOSPC|EROFS error when updating settings

This is to make it easier to troubleshoot when unable to update the settings file, as opposed to "wrong code".
This will only give this information to already authenticated users, so should be safe.
This needs to be checked on both open() and write().
Issue #13 is about actually letting logins (optionally) happen under these conditions.

Also ac18a012cecc46c7603f5ce5597411461a3854d8 was a commit implementing this.


Is this still being maintained?

Hi!
Is this project dead? Should I just fork and fix the sqlalchemy>=2.0 issues myself?
Thank you for listening to my Ted Talk.

Was just looking and asking myself the same. If you want to tackle together, LMK!

Not sure if you've already fixed the problem, but I'd forked this repo a while ago and made it SQLAlchemy 2.x compliant: https://github.com/hrshdhgd/pansql

I'd suggest looking at DuckDB. It allows you to run SQL directly on Pandas (and Polars) DataFrames and is blazingly fast, as it's optimised for analytical workloads and can parallelise across multiple threads.

I'd suggest looking at DuckDB. It allows you to run SQL directly on Pandas (and Polars) DataFrames and is blazingly fast, as it's optimised for analytical workloads and can parallelise across multiple threads.

Yes, that's what our team is doing !


Use PyCode_GetCode

Discussed here

Thanks for the investigation Dennis. Have you tried seeing if it improves anything with the coverage.py benchmarks ?

Have you tried seeing if it improves anything with the coverage.py benchmarks ?

I haven't -- I was having trouble getting those to run on Windows.

I am on PR to add coverage benchmark to pyperformance to aid benchmarking.

This results in a 20% speedup on main branch vs this branch:

Benchmark
3.12
PyCode_GetCode

coverage
204 ms
170 ms: 1.20x faster

Benchmark: https://github.com/python/pyperformance/pull/213

This looks like a slight improvement.  Not the 20% that @kumaraditya303 found, but worthwhile:

pyver
proj
6.4.1
#1394
#1394 vs 6.4.1

python3.11
bug1339.py
0.828 s
0.807 s
97%

python3.11
bm_sudoku.py
55.887 s
52.904 s
94%

python3.11
bm_spectral_norm.py
68.005 s
61.789 s
90%


【iconvsr bug】 Expected to have finished reduction in the prior iteration before starting a new one

pytorch1.7.0
RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used i
n producing loss. You can enable unused parameter detection by (1) passing the keyword argument find_unused_parameters=True to torch.nn.parallel.DistributedDataParallel;
(2) making sure all forward function outputs participate in calculating loss. If you already have done the above two steps, then the distributed data parallel module wasn
't able to locate the output tensors in the return value of your module's forward function. Please include the loss function and the structure of the return value of forw ard of your module when reporting this issue (e.g. list, dict, iterable).

This bug should have been fixed in #290. Are you using the latest version of MMEditing?

Closing due to inactivity. Please reopen if there are any further problems.


Fixes 24969. Fix eds endpoint selection for subsets with no or empty …

…subset labels
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
This PR fixes the behavior reported in #24969.
Currently, when a DestinationRule has a subset with no label selector it is treated as an array of label.Instance with one element, nil.
When computing the endpoints for EDS in DiscoveryServer.loadAssignmentsForClusterIsolated() that match this subset there is a special handling for label selectors with no elements which results in all endpoints accepted:
https://github.com/istio/istio/blob/2fa151ab385999535517c489165164a8deb650b1/pkg/config/labels/collection.go#L23-L37
But when c contains a single element nil, then the function always returns false when the endpoint also has no label, resulting in no endpoints assigned to that cluster.
(I didn't test it, but I would assume that if an endpoint has at least one label that the code should panic in line 32 on a nil dereference)
The fix handles no or an empty label selector on the subset as nil instead of of []labels.Instance{nil}
Note: I already opened #24970 which targeted the release-1.6 branch.
This PR is for master which seems to show the same problem.

Hi @robertpanzer. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

Thanks for merging!
It would be great if this could make it into 1.6 too.

I added the cherrypick label to push it to 1.6. But nothing happened. Let me try again

/cherrypick/release-1.6

@howardjohn @rshriram cherrypick seems to be not working for me. Can u please cherrypick this for 1.6?

/cherrypick release-1.6

@howardjohn: new pull request created: #25027

In response to this:

/cherrypick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


SharePoint Framework Web Part not working properly in Viva Connections

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
SPFx version 1.11.0
Describe the bug / error
Using Viva Connections we have added one of our SharePoint Online sites to the Teams App Bar. If we navigate to a SharePoint Modern Page using de App Bar Menu, our SharePoint Framework Web Parts on that page do not work properly.
Among things we've noticed thus far:

onActiveItemChanged event of DetailList Component not firing
SharePoint Theme is not being applied

If we use the App Bar to navigate to a different page and from there use SharePoints own menu's to navigate to the page containing the Web Parts, everything works as expected. It somehow feels similar to the partial page loading issue involving the SharePoint left vertical menu.
Steps to reproduce
Steps to reproduce:

using Viva Connections, create an App in de Teams App Bar pointing to a Modern Team or Hub Site
have a Modern Page with a SharePoint Web Part displaying a DetailList and a Button
make sure this Modern Page is accessible in e menu (e.g. horizontal navigation)
open the SharePoint site using the App in de Teams App Bar
click on the App for a second time to open the App Menu
open the menu item pointing to the Modern Page containing the Web Part
verify that the DetailList onActiveItemChanged event will not fire

We've have been able to reproduce this on all our SharePoint Online tenants using a very basic SPFx Web Part created with SPFx v1.11.0.
Expected behavior
Web Part should function the same when using SharePoint in Teams using Viva Connections.

Hi @tripleseven73 - just to completely confirm things here.  Is the problem that extensions that you have installed on a list aren't working on the list webpart when it is hosted in the Viva Connections app?  Or is it that you have a custom webpart using fabric react details list and it isn't firing properly?
Can you share out a simple project that shows the issue?

Hi @patmill ,
It is the latter: a custom Web Part using a fabric react details list. I have attached the SPFx solution containing the ShowList Web Part I'm using to reproduce the issue.
Important note: In the original bug report I mentioned the use of SPFx v1.11.0. I recently upgraded all our SPFx solutions to v1.12.1.  This includes the ShowList Web Part I have attached. The issue is still present after upgrade.
csgDetailListInVivaConnections.zip

Sorry for the long delay - it has been fixed and should be rolled out WW


Unable to use Nested Accordions

Describe the bug
While I am not sure if this is an issue, or maybe it's just a limited with the current Flowbit Accordion. Tried creating nested Accordions and ensuring that all of the ids used are unique and match the correct group of elements, but once you try to click the child  Accordion to open it up, it closes the parent.
To Reproduce
Steps to reproduce the behavior:

Go to example: https://typescript-prfna8.stackblitz.io/
See HTML: https://stackblitz.com/edit/typescript-prfna8?file=index.ts

Expected behavior
Should be able to have nested Accordion, and child can open and close as expected without closing parent.
Screenshots

Desktop (please complete the following information):

OS: MacOS
Browser: chrome
Version : 104.0.5112.101

Additional context
Bootstrap example: https://codepen.io/WinterSilence/pen/abpopXa

Hey @dimitriharding,
I'll check this out. It may be a bug, a limitation of the Accordion API or you may have missed an id or attribute.
I'll get back with this.

@zoltanszogyenyi Any update on this bug? Is there a workaround for using nested accordions with Flowbite?

same here, using nested accordions triggered the parent accordion event

I can reproduce the issue. The reason seems to be that all [data-accordion-target] descendants are considered items instead of just the "direct children".
For nesting, it would be required to associate all [data-accordion-target]s with their closest [data-accordion] parent. The code would looks something like this:
const targetCandidates = [
    ...$accordionEl.querySelectorAll('[data-accordion-target]'),
] as HTMLElement[]

// Group items by closest accordion parent to make nested accordions work.
const items = targetCandidates
    .filter(
        ($triggerEl) =>
            $triggerEl.closest('[data-accordion]') === $accordionEl,
    )
    .map(
        ($triggerEl): AccordionItem => ({
            id: $triggerEl.getAttribute('data-accordion-target'),
            triggerEl: $triggerEl,
            targetEl: document.querySelector(
                $triggerEl.getAttribute('data-accordion-target'),
            ),
            iconEl: $triggerEl.querySelector(
                '[data-accordion-icon]',
            ),
            active:
                $triggerEl.getAttribute('aria-expanded') === 'true'
                    ? true
                    : false,
        }),
    )

This should the fix parent-child relation, however it doesn't work for me yet. 😕

I had the time to update Flowbite and verify that nested accordions work.
However, in addition to init functions attached to window (new feature) in the same release (1.6.6, it would be very helpful to be able to access initialized components (not only accordions). The following options come to my mind:

like a static member of e.g. Accordion: window.Accordion.instances
via a function-based API, e.g. window.Accordion.getInstances()

Since there are already quite a few Flowbite variables attached to window, a namespace would reduce the window-namespace pollution: window.Flowbite.Accordion etc.
@zoltanszogyenyi Should I open a new issue for this feature request?

We have nested accordions:
https://flowbite.com/docs/components/accordion/#nesting-accordions


Exception during build: AttributeError: 'tuple' object has no attribute 'rebuild'

Hi,
When attempting to build the website using sphinx I get the following error:
Exception occurred:
  File "<..>\.venv\Lib\site-packages\sphinx\config.py", line 433, in __iter__
    yield ConfigValue(name, getattr(self, name), opt.rebuild)
                                                 ^^^^^^^^^^^
AttributeError: 'tuple' object has no attribute 'rebuild'

Any idea where this could be coming from? I am using Pydantic 2.7 and the following extensions:
extensions = [
    "sphinx-jsonschema",
    "sphinx_jinja",
    "sphinx.ext.napoleon",
    "sphinx.ext.autodoc",
    "sphinx.ext.autosummary",
    "sphinx.ext.viewcode",
    "sphinx.ext.intersphinx",
    "sphinx.ext.githubpages",
    "sphinx.ext.linkcode",
    "myst_parser",
    "sphinxcontrib.autodoc_pydantic",
]

Also attaching the log in case its helpful
sphinx-err-497mbx2s.log
Thanks!

Hey @bruno-f-cruz, ran into this issue as well (#274).  Downgrade Sphinx to 7.2.6 and it will fix it

Literally just tried this and its working, thanks :)
Closing this as a duplicate of #274


Use posix file advisory locks on supported platforms

the current storage implementation uses a home brew approach to locking a file for a specific process.
this is a feature built into posix compatible filesystems.
certmagic filestorage should capability detect if posix locks are available, and if they are, use them to protect access to files.
advisory locks have two major weaknesses, that users of sqlite, mdbx, lmdb, are probably well familiar with.

they do not work well on network filesystems (depends on implementation, very flaky regardless)
you cannot use them to lock within a process - you still need mutex in process code to protect access to it.

#1 already exists as an issue in the current implementation
#2 can easily be dealt with by go's mutex.
as a result, I think that it would be good to incorporate this feature into certmagic storage.
that said, I don't use the filesystem module in production, so I don't really care that much. maybe someone who does would be happy to do this work :)
some resources:
https://gavv.net/articles/file-locks/
https://github.com/juju/fslock

This could be a good idea. As mentioned in Slack, I don't know much about them, so I'm not sure I'd be the right person to implement or review this patch.


OWASP ZAP scans

Web Application Vulnerability Scanners are automated tools that scan web applications, normally from the outside, to look for security vulnerabilities such as Cross-site scripting, SQL Injection, Command Injection, Path Traversal and insecure server configuration.
So, here this can be achieved by installing ZAPCLI

Hey! Thanks for the contribution. Can you add a description what this would be for pls?

How is this related directly to Epiphany as our container is just used to deploy infrastructure, not directly manage/scan applications.

I would recommend to use separate (dedicated) image for scanning.

I would recommend to use separate (dedicated) image for scanning.

Yeah, it can be based  on our image using:
FROM epiphanyplatform/epicli:x.x.x ....

@Harshini097t for me this is something that does not need to be part of our repository as this seems really project specific.  @to-bar would you agree with that?

@Harshini097t for me this is something that does not need to be part of our repository as this seems really project specific. @to-bar would you agree with that?

I agree, one can just run pip3 install zapcli in their container if ZAP is needed or keep this Dockerfile in their fork or another repo. I'm not aware of business case in our organization that would require ZAP.

Closing because of lack of feedback. Please recreate the pull if this is still relevant.


dotnet: migrate nuget lockfiles to JSON

This changes the buildDotnetModule lockfiles to be JSON instead of Nix.
Changes were made in a backwards-compatible fashion, however a warning was added to guide users to migrate to the new format.
Existing in-tree packages were migrated to the new format using a helper script and manual changes for places that didn't have the file directly specified.
cc: @NixOS/dotnet @MattSturgeon @UlyssesZh @TomaSajt @SuperSandro2000 @Atemu
Closes #362182, closes #325053.
Fixes #358025.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[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

[x] 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/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362278

x86_64-linux

:white_check_mark: 2 packages built:

nixpkgs-manual
nuget-to-json

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362278

x86_64-linux

:white_check_mark: 3 packages built:

nixpkgs-manual
nuget-to-json
nuget-to-nix

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362278

x86_64-linux

:fast_forward: 1 package blacklisted:

nixos-install-tools

:white_check_mark: 3 packages built:

nixpkgs-manual
nuget-to-json
nuget-to-nix

@corngood / @MattSturgeon any other requests for this? I'd like to have it merged before I have to re-do the treewide commit again 😅

It might be better to do split this PR into adding support and migrating treewide.

It might be better to do split this PR into adding support and migrating treewide.

Are you suggesting that so that we can backport this through the tag or just to make it easier in general?

I'm not quite sure what you mean by that but this would make working on supporting JSON easier to as you wouldn't have to rebease the treewide all the time. You implemented this in a backwards-compatible fashion which should allow doing that.
After this has been thoroughly reviewed and landed, the actual tree-wide could be merged after about 5 min when CI has passed because it's trivial once we have determined that the migration script is good.

Fair enough, I'll remove the treewide commit for now.

I think it's good enough if each of us manually migrates and spot-checks a few packages that we actually use; we weren't going to check each and every package of the treewide anyways.

(or, even better, have identical outputs to before this PR).

Well, that's the thing that's being proved by the nixpkgs-review result, that their inputs haven't changed so their outputs don't either.

Removed the lib.warn on nix-based lockfiles since the SDK still uses them and I don't plan to migrate it away from it right now as it seems rather complicated.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362278

x86_64-linux

:fast_forward: 1 package blacklisted:

nixos-install-tools

:white_check_mark: 3 packages built:

nixpkgs-manual
nuget-to-json
nuget-to-nix

I think this PR should be good to merge now since all feedback has been addressed.

@GGG-KILLER do you think this is all suitable to backport to 24.11?

do you think this is all suitable to backport to 24.11?

Other than the treewide commit and the changelog one, it should be

Other than the treewide commit and the changelog one, it should be

Should we put the release notes in 24.11 if we're going to backport it there?
I think it's important to backport this to allow easier backporting of package updates.

Should we put the release notes in 24.11 if we're going to backport it there?

I don't think it's necessary since it has already released and we only actually put it into practice in 25.02.
Unless if we also backport the documentation changes, then I think we should.

I think it's important to backport this to allow easier backporting of package updates.

Yeah, I was planning to do so already.

I was thinking of:

WARNING: nuget-to-nix has deprecated in favor of nuget-to-json.

Which is pretty self explanatory, and unlikely to alarm anyone.

I think we'd better backport the documentation changes and the release-notes if the functionality is there. There's a new warning here that people will see if they follow the old docs.

I guess it depends on if we consider it deprecated in 24.11 too?
Since that version is fairly fresh, I'm fine with that. But it feels unusual to deprecate things on an already released stable.
My preference would be to backport the JSON compatibility, but not the warnings. Non-issue if the warning's already removed.
Documentation could be backported, but the deprecation notice staying in the 25.0s release notes seems reasonable to me.
:thinking:

I was thinking of:

WARNING: nuget-to-nix has deprecated in favor of nuget-to-json.

Which is pretty self explanatory, and unlikely to alarm anyone.

Oh, fair enough, I totally forgot about that warning, my bad 😅

I guess it depends on if we consider it deprecated in 24.11 too?
Since that version is fairly fresh, I'm fine with that. But it feels unusual to deprecate things on an already released stable.

Yes, that would indeed be something to consider. Deprecating things on a release that is already out is a big no-no afaik.

My preference would be to backport the JSON compatibility, but not the warnings. Non-issue if the warning's already removed.

Yeah, my worry is whether leaving the 24.11 and master branches different might not cause any issues with conflicts or something similar.
I don't know exactly how things flow between them, if it's only master -> release-24.11 or if there's some merges from release-24.11 to master sometimes.

Documentation could be backported, but the deprecation notice staying in the 25.0s release notes seems reasonable to me.

If the documentation is backported, I'd say we should move the release notes back as well, as they mention the support for JSON files.

Documentation could be backported, but the deprecation notice staying in the 25.0s release notes seems reasonable to me.

If the documentation is backported, I'd say we should move the release notes back as well, as they mention the support for JSON files.

I disagree, because the change happened after 24.11 was released, so it doesn't make sense to me to be in its release notes.
However if you did go down that route, you'd end up with two lots of release notes:

24.11: added support for JSON lockfiles
25.05: deprecated support for nix lockfiles; this will be removed in...

Since that version is fairly fresh, I'm fine with that. But it feels unusual to deprecate things on an already released stable.

I don't think it does any harm to announce deprecation as early and as widely as possible.  It's not breaking or removing anything.  It's just a plan.
I think we should aim to have evaluation warnings for deprecation in 25.05, and then remove the functionality no earlier than 25.11.

I think we should aim to have evaluation warnings for deprecation in 25.05

The only blocker for this really is combine-deps.nix, if that's changed to support JSON instead of nix files, we could already put it into practice.

and then remove the functionality no earlier than 25.11.

Yeah, that's the original plan.

I disagree, because the change happened after 24.11 was released, so it doesn't make sense to me to be in its release notes.

I thought about that, but it looks like release notes have been updated recently for older releases.

The only blocker for this really is combine-deps.nix, if that's changed to support JSON instead of nix files, we could already put it into practice.

Yeah, I'll try to do that at least for the next SDK updates.

The only blocker for this really is combine-deps.nix, if that's changed to support JSON instead of nix files, we could already put it into practice.

Yeah, I'll try to do that at least for the next SDK updates.

Thanks, sorry I didn't do it myself but I really don't feel confident to even attempt it. I feel like I'd just end up breaking everything instead 😅

Thanks for all the work on this.  I'd like to merge it soon, but I just wanted to give anyone a last chance to raise an objection.

The backport will probably have to be done manually.
We could also opt to not backport the tree-wide (as the Nix files still work) but that'd necessitate every update backport for dotnet packages to be backported manually.
Given that we have nearly perfect QA for this (eval is the same), I don't see a great risk.


add(tools): Tailwind Typography Playground

Name
Link

Tailwind Typography Playground
https://tailwind-typography-playground.vercel.app/

A little tool I wrote for myself to learn Tailwind CSS and quickly experiment to see how different Google font combinations would look when used with the Tailwind Typography plugin.

[x] My item is in the right category
[x] My item is logically grouped below similar items
[x] My item's name and description respects the conventions of the list
[x] My item is awesome
[x] I have read and followed the contribution guidelines

Thanks!

@innocenzi NP. I hope people find it useful!


fix(deps): update terraform terraform-ibm-modules/key-protect/ibm to v2.8.3

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/key-protect/ibm (source)
module
patch
2.8.2 -> 2.8.3

Release Notes

terraform-ibm-modules/terraform-ibm-key-protect (terraform-ibm-modules/key-protect/ibm)
v2.8.3
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cbr/ibm to v1.23.5 (#606) (9ad5d31)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


Add property schema for subnet tags objects

What this PR does / why we need it
Towards https://github.com/giantswarm/roadmap/issues/2127
The goal of this PR is to add missing schema for objects related to AWS resource tags.
My understanding is that the goal of these objects is to allow entry of user-defined key-value-pairs. This can be achieved using additionalProperties or patternProperties.
I'm deciding for additionalProperties here for now, which leaves greatest possible freedom on the format of the key. The patternProperties method instead would allow to impose certain regex restrictions on the key format, like e. g. disallow whitespace.
The validation pattern for values encodes what's written in https://docs.aws.amazon.com/general/latest/gr/aws_tagging.html#tag-conventions
Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests
/test create
/test upgrade

/test create

/test upgrade


0.5.3 warns about undefined lint rules

dartlang version: 0.5.3
Atom version: 1.3.2
System: Slackware 14.1 x64
Supported lint rules are: https://dart-lang.github.io/linter/lints/
The dartlang plugin warns/complains about two of those rules:

always_declare_return_types
always_specify_types

Actually I'm not sure it is the dartlang plugin that is emitting the warnings, and neither am I sure that the above posted lint rules are globally accepted as valid. These are just wild assumptions made by me.

Do you have an .analysis_options files for the project? I believe that file is used to enabled and disable specific lints.

It is on that file that the warnings are shown.
analyzer:
strong-mode: true
linter:
rules:
- always_declare_return_types
- always_specify_types
- camel_case_types
- constant_identifier_names
- empty_constructor_bodies
- library_names
- non_constant_identifier_names

Which version of Dart are you using?
I believe it is only fully supporting in dev channel (1.14-dev.xx).
And not in the stable channel (1.13.2)

I'm on stable, so that might be it.

Cool. Adding @pq to confirm, but that sounds like the issue.

Adding @pq to confirm, but that sounds like the issue.

:+1: spot on.


Streamline android build configuration

When integrating aFreeRDP in F-Droid, @akallabeth and I stumbled over several (minor) issues:

OpenH264 is disabled by default, therefore it is not tested by FreeRDP continuous integration. (Requires another bugfix with OpehH264 versions newer than 1.8.0).
32-bit builds are configured with unsupported target 14 but need tmerfd_create to compile (NDK_TARGET=21 required, as as already done for 64-bit builds)
scripts/android-build-freerdp.sh does not forward NDK / SDK correctly from command line arguments (--ndk, --sdk) to environment variables needed by OpenSSL (ANDROID_SDK / ANDROID_NDK / ANDROID_NDK_HOME)
Building for all architectures from command line in could be put into one single line. Currently two invocations of android-build-freerdp.sh are needed for 32-bit and 64-bit architectures, and it needs to be executed from within the scripts directory.

I am no expert in gradle, but would it be even possible to let gradle invoke the native build step?
See https://gitlab.com/fdroid/fdroiddata/-/merge_requests/6700, https://gitlab.com/fdroid/fdroiddata/-/merge_requests/7703 and
https://gitlab.com/fdroid/fdroiddata/-/issues/2021

One further action was needed to finally compile the Android app (discussed in https://gitlab.com/fdroid/fdroiddata/-/merge_requests/7703)
OpenH264 is currently at v1.8.0 in the android build scripts, and it cannot be upgraded because of an issue. That version does not build with recent Android NDK, but r15c latest. The other components (OpenSSL and FreeRDP) need a newer NDK.
As suggested from F-Droid team I added a patch to their build script which builds the components using their respective NDK. It actually runs and does seems to produce binary compatible libraries.
The patch works by configuring NDK r15c separately for OpenH264 through $ANDROID_NDK_OPENH264 using this patch:
diff --git a/scripts/android-build-freerdp.sh b/scripts/android-build-freerdp.sh
index b27fa6c9b..7c504b11e 100755
--- a/scripts/android-build-freerdp.sh
+++ b/scripts/android-build-freerdp.sh
@@ -111,7 +111,7 @@ do
         then
             common_run bash $SCRIPT_PATH/android-build-openh264.sh \
                 --src $BUILD_SRC/openh264 --dst $BUILD_DST \
-                --ndk $ANDROID_NDK \
+                --ndk $ANDROID_NDK_OPENH264 \
                 --arch $ARCH \
                 --target $NDK_TARGET \
                 --tag $OPENH264_TAG


Customization Broken with FF38

All items on add-on bar are forced to the right.
There used to be a large space to the left side when this happened that you could drag off.  It is no longer there, it is now an area you cannot get to or do anything with.

Hi,
Sorry for the very very late response.  Is this still an issue for you with the latest version?


Support for 2020.2 CE?

Just tried to add this to my PyCharm 2020.2 CE and found that it isn't yet supported.  I'm hoping that's just a packaging issue and can be quickly updated.

@dorth, hello, thank you for this issue!
Plugin updated and now wait for Jetbrains approval.


xen-start fixes for domain reboot

During restart of domain (initiated by reboot from inside of DomU) it changes the ID, so we need to take care of it in xen-start script.
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

@adarsh-zededa can you take a look.

Honestly, I think at this point this whole thing is "workaround'y" enough so that we should take a step back and perhaps consider if there's a better way. One idea is to hook directly from our go layer into Xen's toolstack, the other is using their newfangled xenfs.
So what do you guys think? An opportunity for a more fundamental refactoring?

What I also find here, we use xl list "${1:- }" which is actually equals to xl list   in case of empty output from domID.
I will take a look onto libxl bindings. So far, the developers are optimistically writing that they are experimental.

Seems, that it try to fix the same things


Fix: Remove .gitignore from package build

Before submitting the PR:

[x] Does your PR reference an issue? If not, please chat to the team on Discord or GitHub before submission.
[x] Did you update and run tests before submission using npm run test?
[x] Does your branch follow our naming convention? If not, please amend the branch name using branch -m new-branch-name
[x] Did you update documentation related to your new feature or changes?

What does your PR address?
Delete's .gitignore from the final package build.

gitignore in /package is completely useless, totally fine to nuke it

@niktek I'll let you take this one. If the PR looks good feel free to merge!


Fixed typo in README: Should probably mean MiniSafe2 IP instead of HomeSafe2 IP

Based on the description and screenshot, this is probably just a small typo.
Thanks for your plugin!


Do not show announcement banner if contents are empty

Hello folks!
Over at https://github.com/napari/docs we would like to use a temporary announcement banner on our website. We are pointing to a remote announcement-banner.html file. But in the future, instead of turning temporary announcements on or off on the conf.py file, we would like to not show the announcement space at the top of the page if the contents of the announcement-banner.html file are empty.
I tried poking around and found that it could work if we do the following:
https://github.com/melissawm/pydata-sphinx-theme/compare/master...empty-announcement
Is this an acceptable feature request? If so, I can submit a pull request with the suggestion above. Thanks!
cc @jni

Yeah, so the rationale is: updating the conf.py for already deployed sites (such as project.org/stable) is tricky. It's easier if the conf points to (say) project.org/dev/_static/announcement.html, that gets deployed in the conf, and then the banner can be updated by changing the text in that file (simple PR to main) or removed by removing the text altogether (again by a simple PR to main).
Does that make sense to folks?
Thanks @melissawm for getting the ball rolling here!

+1 on this. I am also looking for this feature when switching to pydata-sphinx-theme for scikit-learn.

Is this an acceptable feature request? If so, I can submit a pull request with the suggestion above.

LGTM. In the PR, can you add a bit to the docs about this feature? Probably in this section: https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/announcements.html#insert-remote-html-with-javascript

Will do, thanks!

Incredible! ❤ 🚀


[staging] Some labels in Dataset section in right panel for Single Run page are not per design

Steps to Reproduce

Open a Run single page
Click on "More Info" link or button
Open Dataset section
Pay attention to "Last Modified", "Authors" and "Affiliation Name"

Actual Results
It reads "Last Modified", "Authors" and "Affiliation Name"
Expected Results
It should be "Last Modified Date", "Author(s)" and "Affiliation Names"
Design

Evidence

Note: Affiliation Names row should be removed from panel entirely

Marking this as medium severity due to @Janeece comment above, thank you!

@lauramarcos i think this might be fixed. will you could check please?

Last Modified Date is still not reflected in panel and "Affiliation Names" is still displayed and should be removed. This bug remain as opened until this changed are reflected. thank you!


Cannot upgrade to new nixos channel

I upgraded my nix channel and switch to my new configuration. Now everything that requires graphics gives an Illegal instruction core dumped. The source is in the asahi mesa driver:
> gdb glxgears
...
Reading symbols from glxgears...
(No debugging symbols found in glxgears)
(gdb) run
Starting program: /nix/store/aw4nbyd3z2k3rqygrpn4lxxqf6qy2jma-mesa-demos-9.0.0/bin/glxgears
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/nix/store/jz866c0q32vx9pag7br7dxrqqxkd0m1x-glibc-2.38-27/lib/libthread_db.so.1".

Program received signal SIGILL, Illegal instruction.
0x0000fffff66b08e0 in driCreateNewScreen2 () from /run/opengl-driver/lib/dri/asahi_dri.so
(gdb) bt
#0  0x0000fffff66b08e0 in driCreateNewScreen2 () from /run/opengl-driver/lib/dri/asahi_dri.so
#1  0x0000fffff7766a3c in dri3_create_screen () from /run/opengl-driver/lib/libGLX_mesa.so.0
#2  0x0000fffff7757060 in AllocAndFetchScreenConfigs () from /run/opengl-driver/lib/libGLX_mesa.so.0
#3  0x0000fffff7758118 in __glXInitialize () from /run/opengl-driver/lib/libGLX_mesa.so.0
#4  0x0000fffff7753760 in glXChooseVisual () from /run/opengl-driver/lib/libGLX_mesa.so.0
#5  0x0000000000402cf0 in make_window.constprop ()
#6  0x0000000000401df4 in main ()

The encoding of udf #0 is 0x00000000. Did something go wrong with the file system? Can you verify the relevant store paths have the right hash? Try nix-store --verify --check-contents.

Doesn't seem to be an issue. After some thinking it came back with:
> nix-store --verify --check-contents
reading the Nix store...
checking path existence...
checking link hashes...
checking store hashes.
>

Which I expect means everything looks good.

Yeah, that means the store is okay. Doesn’t exclude the possibility something went wrong building. What Nixpkgs hash are you on?

nixos-unstable channel at commit 97b17f32362e

Perhaps we need to explicitly set withLibunwind to false?
https://github.com/NixOS/nixpkgs/commit/73f6621a3713fc01c09d2e237bf1cc877be45cfe

It's already set to disabled in the override.

I'm also having mesa issues, specifically when in overlay mode:
https://github.com/tpwrules/nixos-apple-silicon/issues/152
Given the bizarre disassembly above, I wonder if this is more evidence for my llvmPackages theory?

I'm using replace mode.

According to Janne Grunau in Matrix chat, Fedora Asahi now builds mesa+asahi with libunwind, whereas ALARM built mesa+asahi without it.
This is the build specification file for mesa in Fedora Asahi, which I admit I find confusing:
https://copr-dist-git.fedorainfracloud.org/cgit/@asahi/mesa/mesa.git/tree/mesa.spec?h=f39&id=197e90d54335ef8fe2c47aa9ead5e5d2a8a3b6d4

The disassembly looks fine using that...

I still think something machine-specific went wrong when building and the file ended up corrupt. I set my system to use the latest main commit of this repo (6e324ab) and the nixpkgs you mention (97b17f32362e). I cannot replicate this issue, nor the faulty disassembly, on my system.
I examined store path /nix/store/yj2pxkiwyf64xi50y6zy7zpmfa705wd6-mesa-24.0.0-drivers/ which came from derivation /nix/store/6qamlsrgfqzgl5k134bvvr6xic4lmzhw-mesa-24.0.0.drv. Unfortunately it seems the mesa build is not fully deterministic so it's a little hard to figure out exactly what went wrong.
You can fix this by remounting /nix/store read-write, nuking the store path (and the main output /nix/store/9qx6h9jhxczji2f94rdn4bjlzyw7mjb6-mesa-24.0.0/) (might also be smart to back them up for examination later), then nix-build --repairing the derivation. Make sure you have the derivation beforehand. Once that's done, reboot and make sure it all works, then run nix-store --verify --check-contents for good measure.

Can confirm that nuking the nix store paths and just rebuilding the issue went away. I have been using nix for ages and never seen anything like this.

I had the same issue just now! weston was crashing with illegal instructions.
I removed the mesa folders from /nix/store/ path (made a backup before just in case)
and then ran sudo nix-store --repair --verify --check-contents which succesfully re-build the mesa driver from source.
i could go back to a graphical system right after.
I noticed a lot of people are having issues related to GPU in the repo. I wonder how many could affected by this thing? Maybe it could be helpful for this to be in the maintenance/repair instructions?

It is not the same mesa, its the newer 24.0.1
Attached below are the misbehaving files:
broken-mesa.zip


[FLINK-32343][jdbc-driver] Fix exception in jdbc driver for jdbc tools

What is the purpose of the change
This PR aims to fix exception in jdbc driver for jdbc tools.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

@libenchao Please have a look at this PR, thanks


Question? Running in API only mode

Hi!
So was looking forward to trying this out, but I get the following error when running with a Rails app that is in api_only mode and this happens when trying to install the migration.
rails console1984:install:migrations
        16: from app/config/environment.rb:5:in `<main>'
	15: from /.rvm/gems/ruby-2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:384:in `initialize!'
	14: from /.rvm/gems/ruby-2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:60:in `run_initializers'
	13: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:205:in `tsort_each'
	12: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:226:in `tsort_each'
	11: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:347:in `each_strongly_connected_component'
	10: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:347:in `call'
	 9: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:347:in `each'
	 8: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:349:in `block in each_strongly_connected_component'
	 7: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:431:in `each_strongly_connected_component_from'
	 6: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:350:in `block (2 levels) in each_strongly_connected_component'
	 5: from /.rvm/rubies/ruby-2.7.2/lib/ruby/2.7.0/tsort.rb:228:in `block in tsort_each'
	 4: from /.rvm/gems/ruby-2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:61:in `block in run_initializers'
	 3: from /.rvm/gems/ruby-2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in `run'
	 2: from /.rvm/gems/ruby-2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in `instance_exec'
	 1: from /.rvm/gems/ruby-2.7.2/gems/audits1984-0.1.1/lib/audits1984/engine.rb:22:in `block in <class:Engine>'
/.rvm/gems/ruby-2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/railtie/configuration.rb:97:in `method_missing': undefined method `assets' for #<Rails::Application::Configuration:0x00007fec796d75a0> (NoMethodError)

Obviously I'm still running Rails6 - but I am pretty sure this error is unrelated to that and this is specific to being run on an api only application. I'm kinda missing where exactly the assets are being looked up from, but searching the repo I find this in the tests files which that line throws the same error if I just type into rails console.
I know this might not be a priority to fix (or even on the roadmap) but hopefully there might be a tweak we can make to ignore anything to do with assets (and leave the assets to audits1984?).
Either way - thanks for this and looking forward to a response!
Thanks!
-scott

Hey @superscott, is it possible that you have added audits1984 to your Gemfile? console1984 doesn't reference audits1984 (the dependency is in the other direction: audits1984 -> console1984). Your app seems to be trying to initialize audits1984 so my guess is that you are including it in your Gemfile. Try removing it from the Gemfile.
Another problem I am afraid you'll find is that console1984 requires Active Record Encryption, which is an upcoming Rails 7 feature, no available in Rails 6.

hi @jorgemanrubia you're right - it was audits1984 and removing it gave me the undefined method 'encrypts' error.
sorry about that! thanks!


is there a way to increase the print size (printBytes())

Can I see how you use this method (printBytes()) The byte printing of my input image has no effect


CPU/Memory scalers not working in 2.0.0-rc

I'm trying to test out the cpu/memory scalers, but its keda-operator does not seem to recognize it.
Im getting this in the logs:
2020-10-20T06:32:27.721Z	ERROR	controller	Reconciler error	{"reconcilerGroup": "keda.sh", "reconcilerKind": "ScaledObject", "controller": "scaledobject", "name": "scaled-obj-name", "namespace": "default", "error": "error getting scaler for trigger #0: no scaler found for type: cpu"}
I guess it might be that this https://github.com/kedacore/keda/pull/1215 is not released yet, even though it is in the docs?
Expected Behavior
CPU/Memory scalers working
Actual Behavior
CPU/Memory scalers not working
Steps to Reproduce the Problem

Add ScaledObject with cpu trigger

Specifications

KEDA Version: v2.0.0-rc
Platform & Version: Please elaborate
Kubernetes Version: 1.16
Scaler(s): cpu/memory

Yeah, it is not a part of RC, we are gonna release RC2 soon (in a matter of days), which will contain this scaler.

Yeah, it is not a part of RC, we are gonna release RC2 soon (in a matter of days), which will contain this scaler.

Thanks for the update,
I was testing and got the same error. When can you release it?

@kenotsolutions once this is fixed https://github.com/kedacore/keda/issues/1257

FYI RC2 is out

@zroubalik Fantastic and quick, thanks. I had this same issue and just installed RC2 into my cluster in hopes of scaling off of CPU.
I know it is just an RC, but if you/y'all are not already aware, it seems as though creating ScaledObjects does not create an HPA resources in RC2. That issue was not present in the 1.5.0 release, but I'm assuming the issue will be fixed in the real 2.0.0 release. Please correct me if I'm wrong, I'm happy to create a more detailed issue if needed.


Revert "Streamlined return values from grab* (#2054)"

This reverts commit 4a769f144c9b02708f3e2c6f30590aaafeb0756c.
Previous PR broke backwards compatibility.
We will change how grab* methods work but once we will be releasing 3.0
Motivation/Description of the PR
Applicable helpers:

[ x ] Webdriver

[ ] Puppeteer

[ ] Nightmare

[ ] REST

[ ] Appium

[ ] Protractor

[ ] TestCafe

Description of this PR, which problem it solves

A link to the corresponding issue (if applicable).

Type of change

[ ] Breaking changes
[ ] New functionality
[ ] Bug fix
[ ] Documentation changes/updates
[ ] Hot fix
[ ] Markdown files fix - not related to source code

Checklist:

[ ] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[ ] Local tests are passed (Run npm test)

I think, we should no revert everything, just make PR with previous signature

grab* will be fixed in #2098


406 Not Acceptable [Content type: video/mp4]

Hello,
I've got a little problem. I can't get my service to stop converting everything to application/json
I tried:
services.AddMvc(config => {config.RespectBrowserAcceptHeader = true;});

i was even trying with [Produces("video/mp4")] but it says " doesnt have output formatter "
Still nothing result:
application/json
or
406 Not Acceptable
How to make it work in .net core 1.0 ?

We're going to need more information about what you're trying to do, including some code from your controller/actions.
If you returns a non-IActionResult from an action method, it gets run through the output formatters to try and write it to the response. By default we have the JSON formatter configured.

Hmm, so is there any output formatters implemented by asp.net core to handle "video/mp4"
This is my controller
        [HttpGet]
        [Produces("video/mp4")]
        public HttpResponseMessage Get()
        {
            string videoPath = @"D:\sample.mp4";
            if (System.IO.File.Exists(videoPath))
            {
                HttpResponseMessage response = new HttpResponseMessage(HttpStatusCode.OK);
                response.Content = new StreamContent(new FileStream(videoPath, FileMode.Open, FileAccess.Read));
                response.Content.Headers.ContentType = new MediaTypeHeaderValue("video/mp4");
                response.Content.Headers.ContentDisposition = new ContentDispositionHeaderValue("attachment")
                {
                    FileName = "sample.mp4"
                };
                return response;
            }
                return new HttpResponseMessage(HttpStatusCode.InternalServerError);
        }

Hmm, so is there any output formatters implemented by asp.net core to handle "video/mp4"

No.
Here's the equivalent in MVC Core 1.0:
[HttpGet]
public IActionResult Get()
{
    string videoPath = @"D:\sample.mp4";
    if (System.IO.File.Exists(videoPath))
    {
        return PhysicalFile(videoPath, "video/mp4", "sample.mp4");
    }
    return StatusCode(StatusCodes.Status500InternalServerError);
}

Done :dagger:


Add microsaccades to Brain-Score core

In this PR we add microsaccades to Brain-Score core. Integration of model_tools PR #75 to Brain-Score 2.0.
Conceptual Problem: We need variance in model responses to static stimuli.
Conceptual Solution: Microsaccades are a prevalent phenomenon in vision and visual experiments. They are involuntary movements of the eye at very small scales. Microsaccades are not (and often can not due to their small extent) be controlled for in experiments, and are thus implicitly baked into the variance in primate responses. As such, microsaccades are a phenomenon that is either explicitly or implicitly believed to be unimportant for behavioral/neural outcomes, and thus a good candidate for getting variance in responses to static stimuli. (e.g. Rolfs 2009, Vis. Res.; Haddad & Steinmann 1973, Vis. Res.)
Technical problem: The current Brain-Score interface does not support microsaccades.
Technical solution: the ActivationsExtractorHelper will be changed. The from_paths method now takes a path, loads the image from the path, does pixel position jittering to the image, and saves the jittered image into a temporary new path. The new path is passed to the downstream functions to get the model activations.
To accomplish this, the from_paths method will now receive a dictionary of tags. This dictionary of tags comes from a call to the look_at method, where number_of_trials is replaced with a new dictionary. This new dictionary then traverses the following path:

run_activations
A call to activations_model
In practice, this calls a TensorFlowWrapper/PytorchWrapper/etc.’s self._extractor class’s __call__() method in its __call__() method. (I.e., PytorchWrapper.__call__() => self._extractor.__call__() )
This extractor’s call method is ActivationsExtractorHelper.__call__()
Thus we pass the new dictionary to ActivationsExtractorHelper.__call__() where it will be parsed.

TODO list:

[x] Implement microsaccades
[x] Implement unit tests for microsaccades

Other Brain-Score benchmark PRs depend on this PR (https://github.com/brain-score/vision/pull/365).
Huge thanks to Hannes Mehrer (@hanme) who implemented microsaccades, which I've only adapted for the generalized case here.

It seems that the Jenkins build is skipping tests on this PR, which is not the intended case: http://braintree.mit.edu:8080/job/unittest_brainscore/1516/parsed_console/ shows the log:
fatal: ambiguous argument 'origin/vision_microsaccades': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
...
MODIFIES_PLUGIN: False
PLUGIN_ONLY: True
Changed files:
Only plugins modified. Will skip tests.

Unsure as to why this happens, and whether it's something I can fix as a part of the PR? The intended case certainly is for tests to run on this PR. Any help would be much appreciated!

Hi @benlonnqvist - thanks for flagging this; it looks like you've caught an issue with the way we're handling forked PRs in Jenkins. This should raise an assertion error in the Travis PR build, but that isn't able to run while there are unresolved conflicts with master. I should be able to push a fix later today and will let you know!

Thank you @kvfairchild! Much appreciated.

Hi @benlonnqvist - can you try again now?

Hi @kvfairchild, tests seem to be working now, thanks again!

Hi! I think there are some remaining tests that fail on Jenkins that I don't think have anything to do with the PR itself - although I could be wrong. Perhaps it might be possible to have a quick look as to what might be going wrong with the few remaining tests and whether they have anything to do with this PR specifically? Thanks!

Hi, it seems that Jenkins is now crashing on building the PR altogether (see here). I don't think this is related to the PR - is there something I can do to resolve the issue potentially?

re-trigger jenkins

All tests on Jenkins (and Travis) are green.


Error: The requested module does not provide an export named 'default'

When setting up a simple test project, I get following error:

Steps to repoduce:

npx degit solidjs/templates/ts my-app
npm install --save solid-map-gl
Paste any of the examples of the README to App.tsx
npm start

It looks like a bundler issue, however I'm not very familiar with that. Might be related to this:
https://github.com/mapbox/mapbox-gl-js/issues/8676

Hi Jean-Claude,
After a bit of research I found a solution.
I can replicate your issue when I'm using npm or yarn.
For bundling solid-map-gl I'm using pnpm and rollup which seem to have no problem with the Mapbox issue.
As a workaround until Mapbox fixes the issue please use pnpm.
- npx degit solidjs/templates/ts my-app
- pnpm i
- pnpm add solid-map-gl
- pnpm dev

Hi Kay,
Thanks for looking into this. Unfortunately, for me, pnpm didn't make a difference.  I created a fresh test project as you described and i I still get the error. My initial project where I wanted to use solid-map-gl uses pnpm.
I could find following workaround:

Add directly mapbox-gl as project dependency (eg. pnpm add mabox-gl)
Add following lines to vite.config.ts:

  optimizeDeps: {
    include: ['mapbox-gl']
  }

I got the hint for using optimizeDeps from here: https://github.com/vitejs/vite/issues/813.

Thanks for finding this workaround.
I've included it in the documentation.


Ingress controller in zeebe full chart doesnt work

Deploying the zeebe ful chart $ helm install test zeebe/zeebe-full-helm will depeloy a ingress controller, which should forward traffic to operate (?).
But accessing the external ip doesn't work and shows a bad gateway.

Reported by an user https://camunda-cloud.slack.com/archives/C6WGNHV2A/p1643705936901069
Workaround port forward traffic to operate service directly.

We removed the controller from the ccsm helm chart, other charts are deprecated.


Question: How do you setup the web project without the classic Initialize Firebase script

Hey @nicolasgarnier,
How do you guys make the firebase init vars configurable only by using firebase use add --
and not hard coding the vars as the classic docs say as below:
<script src="https://www.gstatic.com/firebasejs/4.6.2/firebase.js"></script>
<script>
  // Initialize Firebase
  var config = {
    apiKey: "<cdscdscd>",
    authDomain: "<cdscdscd>",
    databaseURL: "<cdscdscd>",
    projectId: "<cdscdscd>",
    storageBucket: "<cdscdscd>",
    messagingSenderId: "<cdscdscd>"
  };
  firebase.initializeApp(config);
</script>

I analysed this repo code but I wanted a clearer info around this, since is a nice setup for a multi env project that needs different db, storage and others.
Can I query the firebase project to change some logo image too, ex: if is a dev env show my dev logo at home screen.
Thanks.

Instead of the "static" / inline firebase init you can use the init script:
<script src="/__/firebase/init.js"></script>

The init.js script is only available in a Firebase Hosting environment though so you have to test locally using firebase serve and deploy using firebase deploy and use Firebase Hosting as your hosting provider.
If you plan on using your own hosting servers you can generate a JSON file containing the Firebase config using  firebase setup:web --json > ./frontend/firebase-config.json


Fix KMS regression in MCG deployment

Flag 'sc.Spec.Encryption.Enable' is deprecated and now we use
'sc.Spec.Encryption.KeyManagementService.Enable'.
Signed-off-by: Arun Kumar Mohan<EMAIL_ADDRESS>

Working on the test failures...

Tests fixed... Ready for some initial reviews... Thanks

/cherry-pick release-4.11

/cc @umangachapagain @jarrpa


Support for ASCII format

Will there be a support for xyzrgb file format? 1.x versions work fine, but 2.02 fails to convert the file.
You software is awsome!

Unfortunately not in the near future since there are quite a bunch of higher priority tasks to work on. Also, LAS/LAZ are specifically chosen because they also contain the bounding box, which is essential for the converter.


[Improvement] Add a generic lock mechanism in Graviton

What would you like to be improved?
Currently, we lack the lock mechanism in Graviton, which will potentially make some operations inconsistent, for example:

The alterTable operation will load the table first and update to the underlying storage.
The various operations that needs to both manipulate the external metadata source and underlying storage.

Currently, The code will potentially be in race condition, we should have a way to fix this issue.
How should we improve?
We may need to have a lock mechanism to solve this problem. Also, be aware that the lock mechanism should consider that we may have multiple Graviton services, so a process-level lock may not be enough.
Besides, currently we rely on a transaction mechanism to achieve this, it might be an issue for some storage systems like fdb (it has a short transaction timeout requirement). For some operations like Hive operations, it might take a long time to get the results.

This is dup of #407


[openCypher tck case] date/datetime/localdatetime/duration

neo4j test:
neo4j@neo4j> return date("1970-6-8") as d;
+------------+
| d          |
+------------+
| 1970-06-08 |
+------------+

1 row available after 27 ms, consumed after another 3 ms
neo4j@neo4j> return date({year: 1816, week: 1}) as d;
+------------+
| d          |
+------------+
| 1816-01-01 |
+------------+

1 row available after 2 ms, consumed after another 1 ms
neo4j@neo4j> return date({year: 1984, month: 10, day: 11}) as d;
+------------+
| d          |
+------------+
| 1984-10-11 |
+------------+

1 row available after 44 ms, consumed after another 2 ms
neo4j@neo4j> return datetime({year: 1984, month: 10, day: 11}) as d;
+-------------------+
| d                 |
+-------------------+
| 1984-10-11T00:00Z |
+-------------------+

1 row available after 33 ms, consumed after another 2 ms
neo4j@neo4j> return localdatetime({year: 1984, month: 10, day: 11}) as d;
+------------------+
| d                |
+------------------+
| 1984-10-11T00:00 |
+------------------+

neo4j@neo4j> return datetime('2015-07-21T21:40:32.142+02:00[Europe/Stockholm]');
+-------------------------------------------------------------+
| datetime('2015-07-21T21:40:32.142+02:00[Europe/Stockholm]') |
+-------------------------------------------------------------+
| 2015-07-21T21:40:32.142+02:00[Europe/Stockholm]             |
+-------------------------------------------------------------+

neo4j@neo4j> return duration('P2.5W');
+-------------------+
| duration('P2.5W') |
+-------------------+
| P17DT12H          |
+-------------------+

nebula test:
(czp@nebula) [nba]> yield date({year: 1816, week: 1}) as d
+----------+
| d        |
+----------+
| BAD_DATA |
+----------+
Got 1 rows (time spent 23669/32619 us)

Fri, 22 Jan 2021 15:23:18 CST

(czp@nebula) [nba]> yield date("1970-6-8") as d
+----------+
| d        |
+----------+
| BAD_DATA |
+----------+
Got 1 rows (time spent 26484/38375 us)

(czp@nebula) [nba]> yield datetime({year: 1816, week: 1}) as d
+----------+
| d        |
+----------+
| BAD_DATA |
+----------+
Got 1 rows (time spent 23521/33883 us)

Fri, 22 Jan 2021 15:28:37 CST

(czp@nebula) [nba]> yield localdatetime({year: 1816, week: 1}) as d
[ERROR (-12)]: SemanticError: `localdatetime({year:1816,week:1})' is not a valid expression : Function `localdatetime' not defined

(czp@nebula) [nba]> yield datetime('2015-07-21T21:40:32.142+02:00[Europe/Stockholm]')
+-----------------------------------------------------------+
| datetime(2015-07-21T21:40:32.142+02:00[Europe/Stockholm]) |
+-----------------------------------------------------------+
| 2015-07-21T21:40:32.000                                   |
+-----------------------------------------------------------+

(czp@nebula) [nba]> yield  duration('P2.5W');
[ERROR (-12)]: SemanticError: `duration(P2.5W)' is not a valid expression : Function `duration' not defined

This pr https://github.com/vesoft-inc/nebula-graph/pull/984 resolve a little.


Decide how to store 4 associated Images, and 4 associated numbers with each Image

Each game image will have 4 associated image (The actual original image, and 1 image for each heat map). Additionally, each game image will have 4 associated numbers (X, Y, Z, Label).
Figure out how the best way to store this image and manage it


Update requirements

Check and update requirements.txt, requirements_production.txt, bower.json

@normanjaeckel Can you check the python requirements for 2.0b2 release please?

Checked  requirements.txt and requirements_production.txt. There are no newer releases. So we do not have to change anything. #1889 is just coding style.

Now also package.json is updated. Only bower.json is still open.


Enable run button when python interpreter is ready

A render function that returns a TemplateResult is
pretty different in nature and intent from a render
function that takes care not to destroy and recreate
dom nodes - as the code surrounding the CodeMirror editor
must. My naming solution is to add "in place" to the
render function and file names.
Also fix script PATH ordering.

Don't know why build is red... something needs to be cherry picked in from later branches


Clean up the Stage Test Folder to remove the CP4D Dependency

Based on Slack
https://ibm-cloudplatform.slack.com/archives/C02KB99P2JC/p1652387897855369

test folder doesnt have any CPD module dependency from beginning. I have added the files for reference and commented the content.


Support sending out without specifying

Add support for sending out without specifying a port if there is one port defined aside from the error port
https://github.com/noflo/noflo/issues/450

@aretecode the specs are good coverage but they are not checking actual output the component sends. If you add code listening for the output and checking what comes out before calling done that would be way better.

@aretecode @bergie this one is good to merge. I wonder why AppVeyor keeps failing on this saying that it's "non-mergeable", but maybe it will be just fine when merged.

@bergie ☝


Follow GHC HEAD's HsTypes.Promoted -> BasicTypes.PromotionFlag change

It got introduced in ghc/ghc@ae2c9b40f5b6bf272251d1f4107c60003f541b62 and without the patch from this PR, we cannot build haddock.

Thanks!
We should probably also port over to the various backends the pretty-printing logic that SPJ added, but let's start by unbreaking things...


Cycles of symlinks cause infinite loop/recursion

When used in combination with guard/listen, cycles of symlinks cause nontermination. I don't quite know whether the reason lies here or in listen.
See also here.

I think I have run into this with no guard/listen involved. The following code triggers the issue (tested on Ubuntu Trusty, with both rb-inotify 0.9.2 and 0.9.7, on Linux kernel 3.13.0 and ruby 2.0).
#!/usr/bin/ruby2.0
# -*- coding: utf-8 -*-

require 'rb-inotify'

notifier = INotify::Notifier.new

events = [ :dont_follow, :recursive, ]
path   = '/proc/self'

notifier.watch(path, *events) do |event|
  puts "Got event at: #{ event.absolute_name }"
end

The process gets into an apparent infinite loop, going through paths such as /proc/self/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/8/task/28712/fd/9/28713/root/sys/devices/system/memory/memory53/power
I do not think :recursive should behave this way, and especially :dont_follow should prevent that. A link to "/" is enough, it does not have to go through "/proc", but if "/proc" is mounted I think that will happen anyway.
I suppose the RECURSIVE_BLACKLIST in code is meant to guard against this somehow, but it is not effective.

Can you confirm this is fixed?

Running the example provided by @juhaerk seems to work correctly.


FT: Add bucketDeleteCors to API

Depends on scality/S3#497 (putBucketCors)

@ironman-machine try it

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "ft/deleteBucketCORS", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/8394 for CI status.

@ironman-machine again, what's up? >.<

Hello @electrachong
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "ft/deleteBucketCORS", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/8471 for CI status.

end-to-end passed


Square (Polygon) that follows the zoom

Hey guys!, I need a square in the map that follows the zoom.
Currently I am using a function that you pass the center and a distance in km and it returns the vertices for the square. Is working perfect, but I need to resize the polygon as the user zooms in and out.
Is there any relation that I can use for creating the function for converting the zoom into the distance to use for getting the square points?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11110800-square-polygon-that-follows-the-zoom?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github).

You can use polygon.setPoints()

Yes but I need to know which points to set, could be difficult to implement google.maps.geometry.spherical library?

Never mind, is not what we need, thanks anyway!