Run with vmoptions: '-ea', error: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_201.jdk/Contents/Home/bin/java'' finished with non-zero exit value 1

I need 'assert' in my code, so i set '-ea' to vmoptions, but it can not run, and give me the error: Run with vmoptions: '-ea', error: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_201.jdk/Contents/Home/bin/java'' finished with non-zero exit value 1
I tried java project and maven project, both of them works well, so i guess i must be something wrong with gradle.
I have tried gradle-4.10,gradle-6.5 and gradle-6.6.1, all of them went wrong.
Expected Behavior
no error
Current Behavior

error: Run with vmoptions: '-ea', error: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_201.jdk/Contents/Home/bin/java'' finished with non-zero exit value 1
Context

Just new a gradle project in idea, and add a Class and a main function, just like
public class Test {

    public static void main(String[] args) {
        assert 2 > 1;
        System.out.println("2 > 1");
        assert 1 > 2;
        System.out.println("1 > 2");
    }

}

and add '-ea' to vm options, and the error will go out
Steps to Reproduce

public class Test {

    public static void main(String[] args) {
        assert 2 > 1;
        System.out.println("2 > 1");
        assert 1 > 2;
        System.out.println("1 > 2");
    }

}

Your Environment

very simple: jdk 8 or 11, gradle 4.10 - 6.6.1, nothing else

Can't reproduce.

please not use openjdk

Can't reproduce.

Please not use openjdk

Same with Oracle JDK:

Same with Oracle JDK:

😔
Thanks.............................


WIP - remove disclaimer

This PR addresses issue #61
Pending:

add laserdisc in the redis-doc repository

details: https://redis.io/clients#

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #64   +/-   ##
=======================================
  Coverage   22.84%   22.84%
=======================================
  Files          33       33
  Lines         950      950
  Branches       18       18
=======================================
  Hits          217      217
  Misses        733      733

Continue to review full report at Codecov.

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


can't download package

i've been constantly trying to download the file to run my javascript projects, but it constantly keeps telling me that i'm unable to download due some sort of connection to github
the message is the following:
npm ERR! git clone --template=/Users/alexgonzalez/.atom/.apm/_git-remotes/_templates --mirror git://github.com/monsterkodi/node-simple-plist.git /Users/alexgonzalez/.atom/.apm/_git-remotes/git-github-com-monsterkodi-node-simple-plist-git-6a71a541: Cloning into bare repository '/Users/alexgonzalez/.atom/.apm/_git-remotes/git-github-com-monsterkodi-node-simple-plist-git-6a71a541'...
npm ERR! git clone --template=/Users/alexgonzalez/.atom/.apm/_git-remotes/_templates --mirror git://github.com/monsterkodi/node-simple-plist.git /Users/alexgonzalez/.atom/.apm/_git-remotes/git-github-com-monsterkodi-node-simple-plist-git-6a71a541: fatal: unable to connect to github.com:
npm ERR! git clone --template=/Users/alexgonzalez/.atom/.apm/_git-remotes/_templates --mirror git://github.com/monsterkodi/node-simple-plist.git /Users/alexgonzalez/.atom/.apm/_git-remotes/git-github-com-monsterkodi-node-simple-plist-git-6a71a541: github.com[0: <IP_ADDRESS>]: errno=Operation timed out
npm ERR! Darwin 15.4.0
npm ERR! argv "/Users/alexgonzalez/Downloads/Atom.app/Contents/Resources/app/apm/bin/node" "/Users/alexgonzalez/Downloads/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/alexgonzalez/.atom/.apm/.apmrc" "--userconfig" "/Users/alexgonzalez/.atom/.apmrc" "install" "/private/var/folders/p2/cjg7x9y116517d_81w03hmfc0000gn/T/d-116724-25998-upx4ox/package.tgz" "--target=0.37.8" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! nam  v2.13.3
npm ERR! code 128

npm ERR! Command failed: Cloning into bare repository '/Users/alexgonzalez/.atom/.apm/_git-remotes/git-github-com-monsterkodi-node-simple-plist-git-6a71a541'...
npm ERR! fatal: unable to connect to github.com:
npm ERR! github.com[0: <IP_ADDRESS>]: errno=Operation timed out
npm ERR!
npm ERR!
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/p2/cjg7x9y116517d_81w03hmfc0000gn/T/apm-install-dir-116724-25998-3ssjj3/npm-debug.log

thanks for the help

I have the same problem.
I am running:

Windows 10

Atom 1.9.9

Electron 0.37.8

Chrome 49.0.2623.75

Node 5.10.0

npm 2.14.20

I was trying to install the 'Script' Package

I have the same problem.
I am running:
Windows 10 also
please give me the answer

Installing<EMAIL_ADDRESS>failed.Hide output…
Failed to install script because Git was not found.
The script package has module dependencies that cannot be installed without Git.
You need to install Git and add it to your path environment variable in order to install this package.
You can install Git by downloading, installing, and launching GitHub for Windows: https://windows.github.com
Run apm -v after installing Git to see what version has been detected.

@rgbkrk

npm ERR! error: cannot spawn sh: No such file or directory

Do we have some hardcoded "sh" call during installation?

Looks like duplicate of https://github.com/rgbkrk/atom-script/issues/1062 and should be related to https://github.com/rburns/ansi-to-html/issues/27


[rough draft] add info on configuring http servers as export storage

not linked anywhere yet, but we can link to it directly when asked by enterprise customers

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c9b297351919422589dbeeacc754bf5fdf54e21d/

a-ma-azing! Thank you, @dt. lmk if you want me to take this over to implement edits.
For the doc, though, what customer intent does this serve? Is it essentially "storing backups somewhere besides S3, GCE, or Azure?"

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions, all commit checks successful.

v1.1/custom-export-storage.md, line 7 at r1 (raw file):

---

BACKUP and RESTORE support writing to or reading from a variety of Cloud Storage providers like Amazon's S3. In addition to these, any HTTP service that supports GET, PUT and DELETE can also be used for storing exported data.

Small Formatting Nits

[`BACKUP`](backup.html) and [`RESTORE`](restore.html)

s/Cloud Storage/cloud storage/

`GET`, `PUT`, and `DELETE`

v1.1/custom-export-storage.md, line 15 at r1 (raw file):

## HTTP Export Storage API

HTTP Storage uses the GET, PUT and DELETE. After a PUT request to some path, a subsequent GET request should return the content sent in the PUT request body, at least until a DELETE request is received for that path.

GET, PUT, and DELETE verbs?

Comments from Reviewable

Yep, this is for linking to a customer when they say they want to use some machine or machines to store their backups, rather than s3/gcs/etc.  If you have a machine or a couple machines with disks you want to use, you need to run something on them to make those disks available to the rest of your cluster, and that's where we've been saying our suggestion will be to just start up an off-the-shelf HTTP server. This do is just a more concrete version of "just start up an off-the-shelf http server".
That said, there are also some S3 API-compatible servers they could run too, e.g. minio IIRC, so this this doc isn't comprehensive, just a couple example options.
(I PR'ed this from branch on origin to make it easy for any of y'all to edit, in case you prefer to do that directly, but I'm also happy to incorporate PR feedback myself too -- whatever you prefer)

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions.

v1.1/custom-export-storage.md, line 7 at r1 (raw file):
Previously, sploiselle (Sean Loiselle) wrote…
Small Formatting Nits

[`BACKUP`](backup.html) and [`RESTORE`](restore.html)

s/Cloud Storage/cloud storage/

`GET`, `PUT`, and `DELETE`

Done.

v1.1/custom-export-storage.md, line 15 at r1 (raw file):
Previously, sploiselle (Sean Loiselle) wrote…
GET, PUT, and DELETE verbs?

Done. (I went with methods, unless you really like verbs)

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2d32344b2dfd66a6a1a80b3db5368000ffa9e0b6/

Review status: 0 of 1 files reviewed at latest revision, 3 unresolved discussions, all commit checks successful.

v1.1/custom-export-storage.md, line 19 at r2 (raw file):

## Using Caddy as an Export Storage provider

[Download a caddy binary](https://caddyserver.com/download) that includes the `http.upload` plugin and run it with an [an `upload` directive](https://caddyserver.com/docs/http.upload), either in the command-line or via `Caddyfile`.

double "an"

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 3 unresolved discussions.

v1.1/custom-export-storage.md, line 19 at r2 (raw file):
Previously, mjibson (Matt Jibson) wrote…
double "an"

Done.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ec2a4a0d66959548ef0f083f53827211565ab90f/

Taking this over and bundling it into the import-sql-statement branch.


nullsafety and rename unite to fold

I published a prerelease. Thank

I published a prerelease. Thank

Hi 👋. I published a stable release with your code. In the new version, I added the extension FutureEither for a more compact code.


Test failure: DisplayTextFor_ReturnsSimpleDisplayText_IfSetAndValueNonNull

This test fails occasionally with the following error:
System.BadImageFormatException : Index not found. (Exception from HRESULT: 0x80131124)
   at System.RuntimeTypeHandle.ConstructName(RuntimeTypeHandle handle, TypeNameFormatFlags formatFlags, StringHandleOnStack retString)
   at System.RuntimeType.RuntimeTypeCache.ConstructName(String& name, TypeNameFormatFlags formatFlags)
   at Castle.DynamicProxy.Generators.BaseProxyGenerator.ObtainProxyType(CacheKey cacheKey, Func`3 factory)
   at Castle.DynamicProxy.Generators.ClassProxyGenerator.GenerateCode(Type[] interfaces, ProxyGenerationOptions options)
   at Castle.DynamicProxy.DefaultProxyBuilder.CreateClassProxyType(Type classToProxy, Type[] additionalInterfacesToProxy, ProxyGenerationOptions options)
   at Castle.DynamicProxy.ProxyGenerator.CreateClassProxyType(Type classToProxy, Type[] additionalInterfacesToProxy, ProxyGenerationOptions options)
   at Castle.DynamicProxy.ProxyGenerator.CreateClassProxy(Type classToProxy, Type[] additionalInterfacesToProxy, ProxyGenerationOptions options, Object[] constructorArguments, IInterceptor[] interceptors)
   at Moq.Proxy.CastleProxyFactory.CreateProxy(Type mockType, ICallInterceptor interceptor, Type[] interfaces, Object[] arguments)
   at Moq.Mock`1.<InitializeInstance>b__20_0()
   at Moq.PexProtector.Invoke(Action action)
   at Moq.Mock`1.InitializeInstance()
   at Moq.Mock`1.OnGetObject()
   at Moq.Mock.GetObject()
   at Moq.Mock.get_Object()
   at Moq.Mock`1.get_Object()
   at Moq.Mocks.CreateMocks[T]()+MoveNext()
   at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Boolean& found)
   at System.Linq.Enumerable.First[TSource](IEnumerable`1 source)
   at System.Linq.EnumerableExecutor`1.Execute()
   at System.Linq.EnumerableQuery`1.System.Linq.IQueryProvider.Execute[TElement](Expression expression)
   at System.Linq.Queryable.First[TSource](IQueryable`1 source)
   at Moq.Linq.MockQueryable`1.Execute[TResult](Expression expression)
   at System.Linq.Queryable.First[TSource](IQueryable`1 source)
   at Moq.Mock.Of[T]()
   at Microsoft.AspNetCore.Mvc.Rendering.DefaultTemplatesUtilities.GetHtmlHelper[TModel](ViewDataDictionary`1 viewData, IUrlHelper urlHelper, ICompositeViewEngine viewEngine, IModelMetadataProvider provider, IStringLocalizerFactory localizerFactory, Func`2 innerHelperWrapper, IHtmlGenerator htmlGenerator, String idAttributeDotReplacement) in /_/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Rendering/DefaultTemplatesUtilities.cs:line 296
   at Microsoft.AspNetCore.Mvc.Rendering.DefaultTemplatesUtilities.GetHtmlHelper[TModel](TModel model, IUrlHelper urlHelper, ICompositeViewEngine viewEngine, IModelMetadataProvider provider, IStringLocalizerFactory localizerFactory, Func`2 innerHelperWrapper) in /_/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Rendering/DefaultTemplatesUtilities.cs:line 213
   at Microsoft.AspNetCore.Mvc.Rendering.DefaultTemplatesUtilities.GetHtmlHelper[TModel](TModel model, IUrlHelper urlHelper, ICompositeViewEngine viewEngine, IModelMetadataProvider provider, IStringLocalizerFactory localizerFactory) in /_/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Rendering/DefaultTemplatesUtilities.cs:line 199
   at Microsoft.AspNetCore.Mvc.Rendering.DefaultTemplatesUtilities.GetHtmlHelper[TModel](TModel model, IModelMetadataProvider provider) in /_/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Rendering/DefaultTemplatesUtilities.cs:line 162
   at Microsoft.AspNetCore.Mvc.Rendering.HtmlHelperDisplayTextTest.DisplayTextFor_ReturnsSimpleDisplayText_IfSetAndValueNonNull() in /_/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Rendering/HtmlHelperDisplayTextTest.cs:line 163

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on 2.2.
CC @mkArtakMSFT
This issue was made automatically. If there is a problem contact ryanbrandenburg.

Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this actually a flaky test?

No, this is a regular test failure, fix the test/product (TODO: Link to commit/PR)
Yes, proceed below...

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

Is there any logging or extra information that we could add to make this more diagnosable when it happens again?

Yes?

[ ] Add the logging (TODO: Link to PR/commit)

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

@NTaylorMullen, can you please look into this? Thanks!

Issue with Moq - castleproject/Core#193. Using #8157 to track upgrading the dependency.


merge defaults + fix component

I had to make these changes to get it working both on pages where I assign an item (and in that case case still use defaults for values I don't define) and pages where only defaults are available. It's working nicely within Bonfire now :)

Current integrations for references:
https://github.com/bonfire-networks/bonfire_ui_common/blob/main/lib/seo.ex
https://github.com/bonfire-networks/bonfire_pages/blob/main/lib/web/page_seo.ex

You'll note I define a config/0 function rather than pass it in the use macro, because it relies on runtime config.

you beat me to it :) I have some more fixes coming in ff4240f1d62c2c883ac2de3373f9073fef05ebab

@mayel I see that, but I don't see where it's actually used. It looks like it's still using config/1

Well config/1 just calls config/0


Update slave versions and fix to CoP land repos

Updated the slaves to the latest cut of container-quickstarts. I would like to move the config more towards params_from_vars too as there is soo much repetition in the params dir

@springdo can you try changing your label from label: "jenkins-slave-ansible" to label: "jenkins-slave-ansible:v3.11 ? curious to know if that solves this problem.
It'd be nice to keep the v3.11 vs v3.9 vs v4 tag on there because then we wouldn't have confusion over what "latest" is.
Either way, I think the jenkins-slave-generic-template.yml should probably be defaulted to latest and then we should really specify the same v3.11 tag here because we are explicitly using the v3.11 jenkins-slave-base image. I'm going to put a PR on redhat-cop/containers-quickstarts to change that default. will update here

@springdo see https://github.com/redhat-cop/containers-quickstarts/pull/208


Performance optimisations for postgres databases

Addresses #19 and #18 (

[ ] Vocabulary table counts from postgres system tables


Clear canvas button is not working

Please assign me this issue under HACKTOBER-fest .

please go to https://github.com/GDSC-IIIT-Kalyani/Plot and fork the repo from there and rise issues


add origin to cf roles

Reason
In some cases, the same user exists in more than one identity provider. A common case could be a user existing in both the default SAP ID service and in an Identity Authentication service.
In those cases, cf set-org-role and cf set-space-role will not assign the user, as it doesn't know which user to choose from. The error it gives is:
Ambiguous user. User with username 'xxxxxxxx' exists in the following origins: sap.ids, xxxx, xxxx. Specify an origin to disambiguate.
FAILED

According to the cf set-org-role and cf set-space-role, you can use the option --origin <idp> to specify the idp of the user.
I added the same logic applied to the idp for the btp roles so that it only adds the option if they have set the property idp.
Thanks,
Pedro

@pirnz  Thanks for your contributions and closing the gap. PR is merged into dev. will be part of next release


Update instructions to add option to install via RubyGems

I ran into an issue with the Homebrew install because I was using a Ruby version manager...
❯ pod setup
dyld: lazy symbol binding failed: Symbol not found: _rb_str_new_static
  Referenced from: /Users/jared/.gem/ruby/2.4.1/gems/json-2.1.0/lib/json/ext/generator.bundle
  Expected in: flat namespace

dyld: Symbol not found: _rb_str_new_static
  Referenced from: /Users/jared/.gem/ruby/2.4.1/gems/json-2.1.0/lib/json/ext/generator.bundle
  Expected in: flat namespace

[1]    39800 abort      pod setup

Suggesting updating instructions to add installation via RubyGems. This is also the officially recommended way to install CocoaPods.

I signed it!

I signed it!

On Jun 24, 2017, at 9:22 PM, googlebot<EMAIL_ADDRESS>wrote:
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data https://cla.developers.google.com/clas and verify that your email is set on your git commits https://help.github.com/articles/setting-your-email-in-git/.
If your company signed a CLA, they designated a Point of Contact who decides which employees are authorized to participate. You may need to contact the Point of Contact for your company and ask to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the project maintainer to go/cla#troubleshoot.
In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flutter/flutter/pull/10960#issuecomment-310875720, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAVZtiblg-uza_Y6b4dUPrWDufdHi0rks5sHbZugaJpZM4OEfBM.

Can we detect which mode the user is in?

@Hixie that's a great question. I think you could make assumptions depending on the existence and location of the gem binary. It comes down to a preference call though so I'm not sure the best approach. The brew install approach does not work if you are not using a system or brew installed Ruby though.

I guess my problem is that as a user, I'd have no idea which option to use, so this actually in a way makes things more confusing. If there was a way we could pick one or the other, we could just give the "right" answer. If we weren't sure, but were pretty sure, we could add "(recommended based on an examination of your configuration)" or something next to the one we thought was the right solution.

@jfine Are you still working on this PR? (It's fine if you're not, I just want to make sure we don't forget about it.)

I'm closing this for now to get it off our review queue. I filed https://github.com/flutter/flutter/issues/11341 to make sure we don't lose track of this. Please don't hesitate to re-open this PR or provide a new one with a solution that addresses the review comments. Thanks!


fix download command in readme

FWIW I still get the following error when running the download command:
Access denied with the following error:

        Cannot retrieve the public link of the file. You may need to change
        the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:

         https://drive.google.com/uc?id=1dmjWGXlJnUxwosN6MVooCDQe970PkD-1

However manually downloading the file with the link at the bottom and placing the zip into the expected location seems to work.

Hi @hturki, if you use the latest version on main, the command actually changed to ns-download-data nerfstudio --capture-name=poster. We changed the CLI slightly in PR #968 to make it more conducive to adding more datasets.

The error you are running into is caused by too many people trying to download that file. The solution for that is for us to host the data somewhere other than google drive.


Update ComponentRefreshTrigger to not force update OmniSharp with non-compile items.

Prior to this when you would do a dotnet build at the command line while VSCode was open it would trigger document output changed events for the actual pre-compiled views (not just the declaration files). Problem with this is that the pre-compiled views shouldn't be considered as part of the compile output for the primary project, otherwise you get ambiguous reference issues.
Added tests to verify IsCompileItem functionality.

/cc @danroth27 this was causing the issue 😉

@NTaylorMullen :cool: Thanks for looking into this!

:shipit:


Update copy for pay fees

PR-Codex overview
The focus of this PR is to update the Pay configuration page with a new fee sharing model and a link for more information.
Detailed summary

Updated fee sharing information on Pay configuration page
Added a link for more information about fee sharing

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

/dashboard/connect/pay

(view full size)

Commit 2e5c19b66f970570476f5cfc9048b2c1c2f558c4 (https://thirdweb-hd009upkb.thirdweb-preview.com).


Accumulated Volume wrong type causing unmarshalling errors in the client

Addressing GH ticket
MinuteSnapshot is using the wrong type for the AccumulatedVolume field. A user reported an error with unmarshalling that I was able to reproduce.

This would be nice to have.

That said, the api from polygon.io has the field type set as a number (instead of an integer) which would imply there exist cases where the volume could be fractional. Is that possible?

That said, the api from polygon.io has the field type set as a number (instead of an integer) which would imply there exist cases where the volume could be fractional. Is that possible?

Yes, stock splits and acquisitions can cause fractional shares.

Oh good point, I didn't think of either acquisitions or reducing splits. I stand by my, good to have position.

@morningvera @jbonzo updated the test data to use a float; would you like me to merge? and if releasing can I include the version patch update in this pr or cut another pr?

@antdjohns yeah, you can merge it but let’s wait on releasing until tomorrow


libfprint-tod1-broadcom: init

Add package for broadcom finger print sensor firmware.
Description of changes

Things done
Tested on own machine.  Will require an additional change to nixos' fprintd module to link the firmware in the correct place.

Built on platform(s)

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

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

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

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

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2727

A quick heads up for anyone stumbling upon this PR when looking for the drivers: There is now a libfprint-2-tod1-broadcom driver available in the upcoming 24.11 release (from PR #326272). So I guess this PR can be closed.


24px - file-directory / file-directory-fill

#67
There's a ghost pixel we should fix in the 24px folder icons:
https://www.figma.com/file/2rjsQp42uKvzPXzLZIqOeA/Octicons-v2-Source?node-id=736%3A1393

@edokoa I'm not seeing a ghost pixel in these images. is it fixed?

I fixed it. It was in the top long borded before.


Better formatting of the deprecation warning about pipeline methods overrides and more accurate message

"reframe hooks" should be changed to "pipeline hooks" and we should consider coloring if possible.
/users/karakasv/Devel/reframe/cscs-checks/tools/profiling_and_debugging/notool.py:14: ReframeDeprecationWarning: 'JacobiNoToolHybrid.setup' attempts to override final method 'RegressionTest.setup'; consider using the reframe hooks instead
  class JacobiNoToolHybrid(rfm.RegressionTest):
/users/karakasv/Devel/reframe/cscs-checks/tools/profiling_and_debugging/ipm_mpi_omp.py:14: ReframeDeprecationWarning: 'Ipm.setup' attempts to override final method 'RegressionTest.setup'; consider using the reframe hooks instead
  class Ipm(rfm.RegressionTest):

@vkarak Do you want to colorize all deprecation warnings or this specific one?
Do you think user_deprecation_warning should be part of the PrettyPrinter (to get the colorize option) or should it get this option from somewhere else?

Do you think user_deprecation_warning should be part of the PrettyPrinter (to get the colorize option) or should it get this option from somewhere else?

If coloring the message would require substantial refactoring, I would avoid it for the moment. I don't know what are the implications of this. Just moving this function to  the PrettyPrinter will break an implicit assumption in framework's design that a core component (here a core component raising a deprecation warning) may not call a frontend component (currently the PrettyPrinter). This is to enforce a separation of concepts. So, as of now, what you propose here, would mean to move the PrettyPrinter to utility, which I don't have problem to do. I would suggest, though, to open a different issue for allowing coloring the deprecation warnings, since it might not be as trivial as it seems.


Slider with tabs (thumbnail)

This is a (multiple allowed):

[X] bug (idk?)

[X] enhancement (X)

[X] feature-discussion (idk?)

Swiper Version: 4.1.6

Platform/Target and Browser Versions: Chromium on Linux (newest)

Live Link or JSFiddle/Codepen or website with issue: Codepen (var swiperTabs)

What I did

[X] Building a slider with tabs (if possible with dynamic content) based on Thumbs Gallery With Two-way Control
[X] Tested if bootstrap has something to do with that (has not)

Expected Behavior

[ ] When setting centeredSlides to false still beeing able to navigate throuw the thumbs/tabs
[ ] (secondary) Looping both is buggy

Actual Behavior

[X] Tabs do not work, when centeredSlides: false -> Not able to navigate
[X] With centeredSlides: true it works fine.

When you disable centeredSlides then there is no room for sliding and there is no sliding for swiperTabs, so it doesn't work in this case

Is there an other case? :sweat_smile:


Removed redundant Value wraps in Substr function.

Do you know if these have been obsolete since Substr was introduced or if a subsequent change obsoleted them? I usually research that and include the info in the commit message (it helps ensure this isn't just missing test coverage), but it's not required.

@timgraham
It looks like it's obsoleted by e2d6e14662d780383e18066a3182155fb5b7747b.


formally close the file in get_binary_file()

in order to avoid IOError: [Errno 24] Too many open files: when many files are opened during a test with several Get Binary File

Isn't the file closed automatically by the with statement??


Why BlockBlobClient can support ClientSideEncryption

https://github.com/Azure/azure-sdk-for-net/blob/5178b1d1d8129ffe803988671e4244776fbf27c7/sdk/storage/Azure.Storage.Blobs/src/BlockBlobClient.cs#L373
There is scenario where we need to encrypt block block, we are using BlockBlobClient due to file size is large. Is there any suggestion for big file encryption supported.

@CallMeCJ, we highly recommending using one of our server-side encryption implementations instead, such as Encryption Scopes or Customer Managed Keys.  Our client-side encryption implementation is only present so customers of the legacy .NET SDK were able to upgrade.
If you absolutely must use client-side encryption, larger files are supported by the BlockBlobClient.


ci: add travis.yml

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9e80f22). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   68.75%
=========================================
  Files             ?        4
  Lines             ?      320
  Branches          ?        0
=========================================
  Hits              ?      220
  Misses            ?       88
  Partials          ?       12

Flag
Coverage Δ

#
68.75% <ø> (?)

#go111x
68.75% <ø> (?)

#gotip
68.75% <ø> (?)

#linux
68.75% <ø> (?)

#osx
68.75% <ø> (?)

Continue to review full report at Codecov.

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


[FEAT] Allow StrapiV4 CRUDFilters to do deep filter

Is your feature request related to a problem? Please describe.
On Strapi, you can use objects to make deep filters (https://docs.strapi.io/developer-docs/latest/developer-resources/database-apis-reference/rest/filtering-locale-publication.html#deep-filtering)
Example:
filters: {
    chef: {
      restaurants: {
        stars: {
          $eq: 5,
        },
      },
    },
},

GET /api/restaurants?filters[chef][restaurants][stars][$eq]=5
Refine, only allows arrays, with logical filters or conditional filters  (https://refine.dev/docs/guides-and-concepts/data-provider/handling-filters/):
Example:
const filter = [
    {
        field: "name",
        operator: "eq",
        value: "John",
    },

If I try to use a relation field as a filter, strapi will throw an error.
Describe the solution you'd like
I´d like to use deep filters so I can filter some relational tables. On my real case, I´m doing a todo list for each project. I also need to filter this todo list based on its procedure, on another component, and I need to mutate it too. Doing this filtering makes easier on the code logic.
Describe alternatives you've considered
I can cheat the URL constructor doing this:
const filter = [
    {
        field: "chef[restaurants][stars",
        operator: "eq",
        value: "John",
    },

It converts to this: /api/restaurants?filters[chef][restaurants][stars][$eq]=5
It works!

Hello! Thanks for the issue. To be honest I'm a bit skeptical about this structure and how we can support this. filters are handled inside @pankod/refine-core and the shape of filters are meant to be generic and provider agnostic.
I'm recalling a similar difference in @pankod/refine-supabase but it was in sort side. We've added an ability to supabase provider in order to transform field values with dot notation to the format supabase support.
If we're going to introduce a change in the core, we need to make it in a way that is aligned with other providers. Without having any change in the core, all I could think of right now is to export some helpers/functions to allow user to write filters in strapi format and let the helper transform this into refine format. This might be easier to write for users that are already familiar with strapi and remove the need of any change in the core.
I'm talking about something like:
import { strapiToRefineFilters } from "@pankod/refine-strapi-v4";

const filters = strapiToRefineFilters({
    chef: {
      restaurants: {
        stars: {
          $eq: 5,
        },
      },
    },
});

eventually will transform into this (currently supported way/workaround)
{
    field: "chef[restaurants][stars]",
    operator: "eq",
    value: 5,
}

My point is to keep the core clean from provider specific enhancements 🤔
@albcunha I'd love to hear your comments to my suggestion above and I'll be waiting for your solution ideas. 🙏

That would work, but as you can deepnest populate, you would expect to also be able to deep nest filter.
Maybe we could change dataProvider? refine/Strapi-V4 constructs dataprovider with strings (https://github.com/pankod/refine/blob/next/packages/strapi-v4/src/dataProvider.ts)
On Strapi, they throw the object on qs and retrieve the query filter string.
Maybe we can map refine.dev object and create a strapi.io qs object that strapi understands?
My  code probably wont handle edge cases, but maybe something like this:
const createStrapiQs = (filter) => {
    let finalJson: any = {};
    function traverse(o) {
      for (var i in o) {
        if (o[i] !== null && typeof o[i] === "object") {
          if ('field' in o[i] && 'operator' in o[i] && 'value' in o[i]) {
              o[i] = {
                [o[i].field]: {[`$${[o[i].operator]}`] : o[i].value}
              }
            }
          traverse(o[i]);
        }
      }
      return o
    }

    finalJson = traverse(filter, process);
    return finalJson;
  }

I did a codesandbox showing the concept:
https://codesandbox.io/s/clever-tdd-jv627l?file=/src/App.tsx
I couldn´t try on refine as I wasn´t able to rebuild strapiv4 package on node_modules.

I opened a similar issue on the hasura side for a similar feature (#2320).
I think we might want to introduce a new filter type in addition to LogicalFilter and ConditionalFilter: NestedFilter.
It could take any value:
{
operator: "nested"
value: {"category": {"type": {"_eq": "howto"}}},
}

hi @omeraplak  . Is this issue still open that we need to look into ?

hi @omeraplak . Is this issue still open that we can look into ?

I did it! Happy hacking! 🚀

Unassigned @Ruchika30 due to inactivity on this issue. Is there anyone wants to work on this issue? 🙏

Hello, can we do something like:
{
    field: "chef.restaurants.stars",
    operator: "eq",
    value: 5,
}

And its must works only in Sptrapi provider. This solution not implements "dirty" login in core. Changes only in provider of Strapi.
p.s. Dot separated like in Laravel.

I implemented what @rollsover proposed in the following PR #3191, tried it locally, and it's working.


[TS] Add typings

Hello, i would like to add typescript support (typings) to this package. If a PR is created, would it be accepted?

Absolutely!

Hi, thank you for your interest and your proposition.
Unfortunately I'll have to put a veto on that, I'm not a fan of typescript and the overhead it adds just for a small module like that. I like the idea of using typing for a more robust code, but I'd rather use flow than typescript.

In that case, you can make a PR to DefinitelyTyped (https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html) seeing that this package isn't originally written in typescript

ok!


Only allow pnpm

The other scripts in package.json, as well as the instructions in CONTRIBUTING.md, all use pnpm. This script produces an error whenever someone attempts to use npm or yarn.
More info

Good to know, thanks!


Update mvp category name and .net foundation role

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

rd/oren-novotny.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Hi Oren,
You change is live now. Please review it from: https://rd.microsoft.com/en-us/oren-novotny

Looks great, thanks!


Authorization header is ignored/overwritten in Azure Functions

When using static web apps, the "Authorization" header passed from the UI side gets ignored/overwritten with a (randomly generated?) bearer value.  I've reached out to Microsoft Azure help and they confirmed that this is "specific to static-web-apps" and not an issue in "regular" azure functions.
I created a public repo containing just the proof-of-concept code necessary to demonstrate the issue.  It's at https://github.com/pangaeatech/AzureFunctionTest and deployed to https://salmon-wave-0ffa80410.azurestaticapps.net/ using the github action in the repo.  No matter what the client specifies for the authorization header (including not specifying it) the function always receives an authorization token.  We have switched our production code to use X-Authorization instead, but have not found any documented reason why this was necessary.
I've deployed this same proof-of-concept code in multiple azure static web apps using different configurations and always see the same thing.
Note that the most important pieces of code in this repo are:

https://github.com/pangaeatech/AzureFunctionTest/blob/master/wwwroot/public/index.html
https://github.com/pangaeatech/AzureFunctionTest/blob/master/api/echo/index.ts

The deployment script is at https://github.com/pangaeatech/AzureFunctionTest/blob/master/.github/workflows/azure-static-web-apps-salmon-wave-0ffa80410.yml
Here is my original report to the Azure team - it was escalated from there to an Azure App Services Support Engineer:

https://docs.microsoft.com/en-us/answers/questions/52970/the-34authorization34-header-gets-truncated-at-372.html?childToView=99061#comment-99061

I think this is a repeat of this https://github.com/Azure/static-web-apps/issues/34

Yes, you are correct - this is a repeat of #34


Etcd Restore no longer works, relies on image that's been deleted

The restore operation is no longer working, the image it relies on seems to have been deleted.
https://github.com/coreos/etcd-operator/blob/a0523eca24f328c62dd52805f21d995b3efbef4d/pkg/util/k8sutil/k8sutil.go#L106
tutum/curl gives me a 404 on dockerhub.
https://hub.docker.com/r/tutum/curl/

We can replace tutum/curl with another Docker image providing curl command, e.g. appropriate/curl

Looks like it's no longer 404ing, so maybe it was an issue with dockerhub?


An account should not be able to claim same bounty multiple times

Currently there is nothing stopping a user from calling bounty_claim on the same bounty ID multiple times. This will likely happen by accident but confuses the logic in the rest of the contract, which assumes that an account only has one claim per bounty.
If a bounty has been paid out via an approval of a BountyDone proposal, it'll remove one key-value pair, but not additional ones.
It does work for the user to call bounty_giveup to remove the other, however since this is an strange case, it may not be discoverable via a frontend.
At the end of the day, it just make sense to make this limit for usability.
You can check this in the branch for #28 which takes care of another issue by using this script:
#!/bin/sh

# Change these to your account ids
./build.sh
export CONTRACT_ID=sputnikdao2.mike.testnet
export CONTRACT_PARENT=mike.testnet

# Redo account (if contract already exists)
near delete $CONTRACT_ID $CONTRACT_PARENT
near create-account $CONTRACT_ID --masterAccount $CONTRACT_PARENT

# Set up
near deploy $CONTRACT_ID --wasmFile ~/near/sputnik-dao-contract/sputnikdao2/res/sputnikdao2.wasm
export COUNCIL='["'$CONTRACT_ID'"]'
near call $CONTRACT_ID new '{"config": {"name": "genesis2", "purpose": "test", "metadata": ""}, "policy": '$COUNCIL'}' --accountId $CONTRACT_ID

# Add proposal for a Transfer kind that pays out 19 NEAR
near call $CONTRACT_ID add_proposal '{"proposal": {"description": "test bounty", "kind": {"AddBounty": {"bounty": {"description": "do the thing", "token": "", "amount": "19000000000000000000000000", "times": 3, "max_deadline": "1925376849430593581"}}}}}' --accountId $CONTRACT_PARENT --amount 1

# Show error when a user tries to vote along with log
near call $CONTRACT_ID act_proposal '{"id": 0, "action": "VoteApprove"}' --accountId $CONTRACT_ID

# Someone claims the same bounty twice
near call $CONTRACT_ID bounty_claim '{"id": 0, "deadline": "1925376849430593581"}' --accountId $CONTRACT_PARENT --amount 1
near call $CONTRACT_ID bounty_claim '{"id": 0, "deadline": "1925376849430593581"}' --accountId $CONTRACT_PARENT --amount 1

# Show bounty claims
near view $CONTRACT_ID get_bounty_claims '{"account_id": "'$CONTRACT_PARENT'"}'

# Add BountyDone proposal done
near call $CONTRACT_ID add_proposal '{"proposal": {"description": "test bounty done", "kind": {"BountyDone": {"bounty_id": 0, "receiver_id": "'$CONTRACT_PARENT'"}}}}' --accountId $CONTRACT_PARENT --amount 1

# Vote it in
near call $CONTRACT_ID act_proposal '{"id": 1, "action": "VoteApprove"}' --accountId $CONTRACT_ID

# See how many now. Expect it to be empty but it's not
near view $CONTRACT_ID get_bounty_claims '{"account_id": "'$CONTRACT_PARENT'"}'

If multiple claims are to be allowed on the same bounty by the same user, could you check this PR out @mikedotexe ?
https://github.com/mikedotexe/sputnik-dao-contract/pull/3

Is there more info about this PR, issue is still not closed?

Is there more info about this PR, issue is still not closed?

I believe this needs a merge of the external PRs into a PR for this main repo. I do not see code that reflects the change.


Added spanish

And added a couple of new keys

Good add and good catch on the un-localized strings. Thank you very much for translating!

Hey, thank you for making this module, it is amazing!


[FEATURE] SEMANTIC SEARCH

Hello,
I would like to know if this is feasible or if I am dreaming.
Is it possible with OpenAI embedding to insert a large number of documents (CIS BENCHMARK, Redmine Project, GLPI ticket, etc.) of different types (json, pdf, txt, etc.) into a vector database?
Subsequently, with OpenAIGPT, when I ask a question, will it conduct a semantic search or search by similarity or score to find the most relevant elements to assist in answering me?
If yes, could I get some advice?
Thank you in advance.

yep, have a look at templates to see what is possible:


Cannot open GeoJSON files anymore

See #238 for more examples that don't open!

You were right @martinRenou, this is most likely due to #165, and I was also wrong about that this issue persists in JupyterLAB as it doesn't.


Improve performance of subscription management in server

Describe the feature request
https://github.com/chkr1011/MQTTnet/blob/master/Source/MQTTnet/Server/MqttClientSubscriptionsManager.cs
MqttClientSubscriptionsManager does a linear search over all subscriptions to see if one matches the publish packet that needs to be distributed.
Which project is your feature request related to?

Server

Describe the solution you'd like
I created a prototype based on a dictionary approach that splits the topic and creates an index by level. I added special handling for wildcards. The protoype uses Span api to avoid allocations while doing the search so I have to adjust the algorithm a little but still. I think this should be a good improvement on the overall performance.
Benchmarks
the number indicates the number of subscriptions the algorithm has to go through:

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

LinearSearch 10
130.75 ns
0.855 ns
0.800 ns
0.0076
-
-
32 B

IndexSearch 10
93.06 ns
0.698 ns
0.619 ns
0.0153
-
-
64 B

LinearSearch 100
954.1 ns
12.05 ns
11.27 ns
0.0076
-
-
32 B

IndexSearch 100
138.3 ns
0.91 ns
0.85 ns
0.0229
-
-
96 B

LinearSearch 1000
7,381.3 ns
31.55 ns
24.64 ns
0.0076
-
-
32 B

IndexSearch 1000
137.7 ns
0.90 ns
0.84 ns
0.0229
-
-
96 B

regarding a pull request: should I replace the existing MqttClientSubscriptionsManager or should I make this part pluggable and create a separate implementation? also I would apprecheate any thoughts about edge cases I might have missed.

@chkr1011 Hi christian I ported the changes to the current dev branch but im not able to run the unittests because clients leak from one test to the next.
Is this also broken for you / do you have some advice?

Related to PR https://github.com/chkr1011/MQTTnet/pull/874.

@JanEggers I this still relevant? I changed the CI to use VS2019 now and it builds properly. But some tests are still failing. I assume due to race conditions...
I recommend to create a proper interface and "inject" a different implementation depending on the platform. I assume Span etc. will not work for 4.5 and UWP right?

the implementation is not platform dependent, I have to check if the tests are reliable now

OK if it works on all platforms then we don't need separate implementations I guess.
I also have a look at the UnitTests. The problem is that they are running twice because you added net461 and netcoreapp3.1 to the target platform. I am not sure if we should remove one of them or how to deal with that. But you can also have a look if you are able to fix some of the tests.

I assume this ticket is obsolete.


[feature] add support for cloudfront distributions and loadbalancers

What changed? Why?
DEVPROD-266
How has it been tested?

[x] go test
[ ] dev

Change management
type=routine
risk=low
impact=sev5

Review Error for brad-larson @ 2019-07-12 12:58:05 UTC
User failed mfa authentication, see go/mfa-help

Review Error for brad-larson @ 2019-07-26 16:58:55 UTC
User failed mfa authentication, see go/mfa-help


panic google block

I'm getting the following error when trying to run the script.
panic: google block

goroutine 6 [running]:
github.com/alpkeskin/mosint/cmd/controllers.Googling({0x7ffcf51ced54?, 0x1?})
        /root/mosint/cmd/controllers/googling.go:18 +0x1d8
github.com/alpkeskin/mosint/cmd.Do.func1()
        /root/mosint/cmd/handler.go:49 +0x32
github.com/gammazero/workerpool.worker(0x0?, 0x0?, 0x0?)
        /root/go/pkg/mod/github.com/gammazero/workerpool@v1.1.3/workerpool.go:237 +0x2a
created by github.com/gammazero/workerpool.(*WorkerPool).dispatch
        /root/go/pkg/mod/github.com/gammazero/workerpool@v1.1.3/workerpool.go:197 +0x2b8
exit status 2

Any help would be appreciated.

Bugs have been fixed in the new version (v3.0.0). Ensure the config file is configured correctly for Mosint to work properly.


python310Packages.neo: 0.10.1 -> 0.10.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.neo is: "Package for representing electrophysiology data in Python"
meta.homepage for python310Packages.neo is: "https://neuralensemble.org/neo/"
meta.changelog for python310Packages.neo is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.10.2 with grep in /nix/store/xrr7wy3wkb074bqcwcghqjim8klszqzm-python3.10-neo-0.10.2

found 0.10.2 in filename of file in /nix/store/xrr7wy3wkb074bqcwcghqjim8klszqzm-python3.10-neo-0.10.2

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.neo
python39Packages.neo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xrr7wy3wkb074bqcwcghqjim8klszqzm-python3.10-neo-0.10.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.neo https://github.com/r-ryantm/nixpkgs/archive/98dc9e6f4a0d68dbec0c9b2f298cdc587b616c49.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xrr7wy3wkb074bqcwcghqjim8klszqzm-python3.10-neo-0.10.2
ls -la /nix/store/xrr7wy3wkb074bqcwcghqjim8klszqzm-python3.10-neo-0.10.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.neo
python39Packages.neo

Maintainer pings
cc @bcdarwin for testing.

@ofborg eval

Result of nixpkgs-review pr 163435 run on x86_64-linux 1

2 packages built:

python310Packages.neo
python39Packages.neo


[FlatBuffers] Add BuildFlatBuffers.cmake to the distribution

Is your feature request related to a problem? Please describe.
Unable to compile .fbs files from CMake when using FlatBuffers via vcpkg, as necessary convenience functions are in BuildFlatBuffers.cmake, which isn't included to the port distribution.
Proposed solution
Add BuildFlatBuffers.cmake to port distribution
Describe alternatives you've considered
Additional context

BuildFlatBuffers.cmake has been parsed when configuring. I'd like to confirm your request, could you please share your usage scenarios and reproduction steps?

@r4nx ping~

BuildFlatBuffers.cmake is a private file of flatbutters, it has not been installed, and include(CMake/BuildFlatBuffers.cmake has been called in CMakeLists in the source code, so I want to confirm if it is necessary for you to use this here,

This is, as far as I know, the only method to compile the FlatBuffers schemas from CMake (i.e. without inventing your own function that will call flatc), thus I think we should have it in the distribution - it's directly impacting the library usage.
Also, a talk with  FlatBuffers maintainer:

If upstream thinks that BuildFlatBuffers.cmake should be used in downstream projects (i.e. is public API), then it would be upstream's task to add a proper install command (and integrate with the exported CMake config, and add documentation).

@r4nx any process? Have you submitted an issue upstream?

@dg0yt Maintainer of upstream project here, but I wasn't responsible for vcpkg integration. What needs to be done here?

@Adela0814 ping

Sorry for the delay. @dbaileychess  You need to add the install command to the CMakeLists.txt to install BuildFlatBuffers.cmake.

Fixed by: https://github.com/microsoft/vcpkg/pull/27298


Should non-physical tinting affect lower layers?

The spec currently mentions that if specular_color is used to affect the color of the specular reflection, he underlying layers (diffuse/SSS/transmission) should be tinted according to the complementary color to preserve energy.
This makes sense from a physical perspective. However, from what I can tell, these parameters are intended for artistic control rather than physically motivated. Therefore, I'd argue that affecting the lower layers is unintuitive and unexpected - for example, if a user wants to achieve a white object with a green specular highlight, they'd need to set the base_color to something like (0.96, 1.0, 0.96). Instead, I'd suggest to scale the lower layer's intensity according to the maximum value across components, in order to preserve energy without tinting it.

@lukasstockner  Just to note, the changes we propose in https://github.com/AcademySoftwareFoundation/OpenPBR/pull/157 explicitly decouple the underlying layers from the specular_color tint.
We now say:

This is still unphysical (we are artificially tinting a Fresnel factor so losing energy), but in a really clear, minimal way that is easy to account for theoretically.

.Addressed in https://github.com/AcademySoftwareFoundation/OpenPBR/pull/157, can be closed.


[Suggestion]: Better description of the new nugets

I'm not a professional developer, just have some apps for our SME that use ComponentFactory Krypton's for 3 years.
I was pretty excited to see some users take the lead and get the old ComponentFactory project and support it for the future.  But, from a NOT professional developer view, I'm getting pretty confused.
Shortening namespaces and dividing all components in individual nuget packages is a great idea.  But all the Lite and Canary packages that appear when you search Krypton in Nuget Manager is overwhelming.  I can't clearly see the differences between all the packages.
Also, there was a new github repository for each version, but all were update with the same changes.  The problem I see is:  If I have a suggestion, question, issue, where do I request it?  If you do that in your version repository, maybe users with other version and the same issue will have to navigate between each version repository to see all the Issues opened.
Thank you for your time.  Really appreciate all the work done to push forward Krypton Toolkit.
Thanks

Hi @NintendoPlayer
Sorry for the late response, as this repo doesn't get updated often. Please refer to this repo for current development.
In regard to your question, I understand your issue about the NuGet packages. When .NET 5 comes out in November, my aim is to archive all the older repos, and clean up the NuGet package situation. All active packages are named Krypton. with the version number of 5.500.xxxx.
Hope this helps!


Pre-check for jobs and release/push copr triggers

Allow to pre-check the jobs to fail early.
Use the right configured job when running copr builds.
Add copr-build for release and push. (related issue: #239)

I would like to wait with docs and test it properly on stage.

Refactor the handlers/events.

I will merge it to play with that on stage tomorrow. (So I have time to fix potential problems until Friday..;)
Feel free to continue with the review.


Question about AddAuxiliaryLoss?

In the code  AddAuxiliaryLoss, the loss is not stored or used in the forward function, does that mean the grad is constantly to be 1? should it be grad_output * loss?
Thanks a lot if you can straighten this out for me.
class AddAuxiliaryLoss(torch.autograd.Function):
    """
    The trick function of adding auxiliary (aux) loss,
    which includes the gradient of the aux loss during backpropagation.
    """
    @staticmethod
    def forward(ctx, x, loss):
        assert loss.numel() == 1
        ctx.dtype = loss.dtype
        ctx.required_aux_loss = loss.requires_grad
        return x

    @staticmethod
    def backward(ctx, grad_output):
        grad_loss = None
        if ctx.required_aux_loss:
            grad_loss = torch.ones(1, dtype=ctx.dtype, device=grad_output.device)
        return grad_output, grad_loss

@DeepSeekDDM @zwd003  Thanks a lot for helping.

In the code AddAuxiliaryLoss, the loss is not stored or used in the forward function, does that mean the grad is constantly to be 1? should it be grad_output * loss?
Thanks a lot if you can straighten this out for me.
class AddAuxiliaryLoss(torch.autograd.Function):
    """
    The trick function of adding auxiliary (aux) loss,
    which includes the gradient of the aux loss during backpropagation.
    """
    @staticmethod
    def forward(ctx, x, loss):
        assert loss.numel() == 1
        ctx.dtype = loss.dtype
        ctx.required_aux_loss = loss.requires_grad
        return x

    @staticmethod
    def backward(ctx, grad_output):
        grad_loss = None
        if ctx.required_aux_loss:
            grad_loss = torch.ones(1, dtype=ctx.dtype, device=grad_output.device)
        return grad_output, grad_loss

@DeepSeekDDM @zwd003 Thanks a lot for helping.

For the aux loss, the gradient with respect to itself is always 1. This is equivalent to adding the aux loss to the final loss.


Website auto-scroll does not work if last page is updated

If the document is scrolled up and you edit something on the last page, it won't auto-scroll down.

Hmmm, not always though. I can share my document with someone privately as a repro if needed.

Huh, actually I think it has to do with the number of characters you type? So you have to type 9 characters or backspace to get it to auto-scroll. Maybe that's intentional for some reason? I personally strongly dislike the inconsistency.

Not sure if this should be a separate issue, but if there is table of contents and I add a new heading, it auto-scrolls to the table of contents instead of staying where I'm adding the heading. Once I add some text it then scrolls back. I find this quite annoying, especially with larger documents. See attachment:

Otherwise in this simplistic example it auto-scrolls immediately even if I type a single character. But in my other document, I need to type 5 characters to auto-scroll 😅 The behaviour seems inconsistent indeed.

I have pushed some fixes here, primarily for #882. Can you still reproduce this bug?


Fix cp with wildcard and special characters

Previously if we try to use wildcard to copy files in path with special characters like '.', '+', '^', "$', '*', the cp will fail.
This test fix the escape function to escape special characters used in regrex pattern matching.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11304/
Test PASSed.

I see the review has passed. Any estimate on when it'll be released?

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.3 please

@NathanRSmith this will be available in Alluxio 2.4 and backported to versions 2.2 and 2.3.

alluxio-bot, cherry-pick this to branch-2.2 please


BUG - UX- Max width (and truncation) for descriptions in search results

These long texts are hard to read.

That's why in the designs the text block for descriptions is capped at 700px

We should also do something about these descriptions sometimes being really huge.
This is me setting the line-clamp to three lines and max-width to 700px:

Here's the code for guided-answer__tree__desc
color: var(--vscode-foreground);
height: auto;
font-size: 13px;
-webkit-line-clamp: 3;
-webkit-box-orient: vertical;
display: -webkit-box;
line-height: 18px;
max-height: 54px;
text-overflow: ellipsis;
overflow: hidden;
max-width: 700px;

max-height is sort of redundant because line clamp keeps it at 3 lines, but I don't know, maybe leave it just in case 😄

@ritarora @unseen1980

@carwelsh This has already been fixed in this PR. I just validated it.


[layers] When scale < 1f and not 1 / 2^n, performance degrades significantly

When scale = 1f and slightly above, performance is good, even with 7 layers. When scale is decreased a bit, even to 0.9f, performance degrades dramatically. There's something wrong.

Between levels, and with 7 layers in the demo app, the number of tiles to render can reach 1000, which is way more than an Android device can handle. When the scale is equal to 1 / 2^n or above 1, the number of tiles to render goes down to ~250.
Drawing tiles in a stacked way (for each layer) put more pressure on the rendering pipeline. Depending on the device, the pipeline cannot keep up with the incoming flow. On a Nexus 4a, having 3 layers is still within the acceptable limit.
The number of layers shouldn't exceed 3 or 4 on an average device.


Fetch mempool transactions for full nodes

Closes #4.
It is necessary to fetch all of the transactions inside the mempool and store them somewhere in a database in order to search for unconfirmed transactions by address later. That is because there is no native method for that in Bitcoin Core's RPC interface, and by extension Litecoin, Doge, etc.
This does not affect Ethereum because it has already been possible to fetch all of the pending transactions using Web3.

It appears that transactions are not actually being saved to the database because it doesn't issue a PEP249 commit() call.


Config for alternative substitute image.

I was wondering if we could have a config for an alternative substitution image. Something like:
The config could be something like:
{
  "alternativeImage": "`http://placehold.it/${width}x${height}`"
}

And the code.
Substitute.methods.image = function (width, height) {
  if(this.alternativeImage) {
    return `${this.alternativeImage}`;
  } else {
    return 'https://unsplash.it/'+width+'/'+height+(this.staticLipsum ? '' : '?random=' + Math.random()) ;
  }
};

Or something similar. Would love to know your thoughts. I'd be happy to make a pull request too.

I have made a pull request for this, if you're happy to merge: #28

Merged into maser: https://github.com/holidaypirates/nucleus/commit/a91d1c6c4bb040565ac86db0217151cb34ed1e73


OpenGL 3.1 fails to load texture array

Steps to reproduce:

compile the solution with BGFX_CONFIG_RENDERER_OPENGL=31
try to load fieldstone-rgba.zip, e.g. by replacing https://github.com/bkaradzic/bgfx/blob/master/examples/runtime/textures/fieldstone-rgba.dds and running example 6
observe following error

glCompressedTexSubImage2D( _target , _level , _xoffset , _yoffset , _width , _height , _format , _imageSize , _data ); GL error 0x500: GL_INVALID_ENUM

Now in my app I have another error
glCompressedTexSubImage3D( _target , _level , _xoffset , _yoffset , _zoffset , _width , _height , _depth , _format , _imageSize , _data ); GL error 0x502: GL_INVALID_OPERATION

although it can be unrelated. I will investigate and try to create minimal repro inside some existing bgfx example.

Ok, same repro, but set texture as srgb
m_textureColor = loadTexture("textures/fieldstone-rgba.dds", BGFX_TEXTURE_SRGB);

This is interesting, not sure why it's happening, as if array textures are not possible with mips+sRGB?! I'm still investigating.

Ok that was lame bug. Should be fixed now.


Change the title from AuthN to AuthZ

The title says "OAuth 2.0 authentication" and the first paragraph says "The OAuth 2.0 is the industry protocol for authorization"
It seems that someone will be confused.
It would be better to change the title to "authorization".
Thanks,

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

ID: 535cb7a2-f177-7ba0-bb06-25bc540d9055
Version Independent ID: 5e79865e-3d8d-3f7f-cae4-2670b408870f
Content: OAUTH 2.0 authentication with Azure Active Directory
Content Source: articles/active-directory/fundamentals/auth-oauth2.md
Service: active-directory
Sub-service: fundamentals
GitHub Login: @BarbaraSelden
Microsoft Alias: baselden

@akiyoshi OAuth is used in conjunction with OIDC for authentication. The document is focused on the flow of acquiring the token after successful authentication and not on the authorization. Hence it is titled OAuth 2.0 authentication and not OAuth 2.0 authorization.

@amanmcse Thank you for the response.
I understand what you mean, but if the document is focused on the OICD authentication flow, that is redundant because there is the OIDC authentication section.
Since this is an OAuth 2.0 section, it would be nice to have more explanation about authorization.
I would appreciate it if you could add it as an item for improvement in future documentation.
Thanks,


Import Suggestion Not Working As Expected

Issue Type: Bug

Behaviour
Expected vs. Actual Behaviour
When typing a Python class that is somewhere in my extraPaths, it does not suggest an automatic import as I would expect.
Import suggestions work fine for modules that are already imported. Ex: If I have already import from payments.models import PaymentMethod, it will suggest import payments.models.PaymentIntent if I type PaymentIntent. But a model from a completely different module, say users.models, will not be suggested if I start typing User.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.9.1
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Pylance

User Settings

defaultLS: {"defaultLSType":"Pylance"}

downloadLanguageServer: true

envFile: "<placeholder>"

venvPath: "<placeholder>"

venvFolders: "<placeholder>"

condaPath: "<placeholder>"

pipenvPath: "<placeholder>"

poetryPath: "<placeholder>"

languageServer: "Pylance"

linting
• enabled: true
• cwd: "<placeholder>"
• Flake8Args: "<placeholder>"
• flake8Enabled: true
• flake8Path: "<placeholder>"
• lintOnSave: true
• banditArgs: "<placeholder>"
• banditEnabled: false
• banditPath: "<placeholder>"
• mypyArgs: "<placeholder>"
• mypyEnabled: false
• mypyPath: "<placeholder>"
• pycodestyleArgs: "<placeholder>"
• pycodestyleEnabled: false
• pycodestylePath: "<placeholder>"
• prospectorArgs: "<placeholder>"
• prospectorEnabled: false
• prospectorPath: "<placeholder>"
• pydocstyleArgs: "<placeholder>"
• pydocstyleEnabled: false
• pydocstylePath: "<placeholder>"
• pylamaArgs: "<placeholder>"
• pylamaEnabled: false
• pylamaPath: "<placeholder>"
• pylintArgs: "<placeholder>"
• pylintPath: "<placeholder>"

sortImports
• args: "<placeholder>"
• path: "<placeholder>"

formatting
• autopep8Args: "<placeholder>"
• autopep8Path: "<placeholder>"
• provider: "black"
• blackArgs: "<placeholder>"
• blackPath: "<placeholder>"
• yapfArgs: "<placeholder>"
• yapfPath: "<placeholder>"

testing
• cwd: "<placeholder>"
• debugPort: 3000
• nosetestArgs: "<placeholder>"
• nosetestsEnabled: undefined
• nosetestPath: "<placeholder>"
• promptToConfigure: true
• pytestArgs: "<placeholder>"
• pytestEnabled: false
• pytestPath: "<placeholder>"
• unittestArgs: "<placeholder>"
• unittestEnabled: false
• autoTestDiscoverOnSaveEnabled: true

terminal
• activateEnvironment: false
• executeInFileDir: "<placeholder>"
• launchArgs: "<placeholder>"

experiments
• enabled: true
• optInto: []
• optOutFrom: []

insidersChannel: "off"

tensorBoard
• logDirectory: "<placeholder>"

Extension version: 2021.12.1
VS Code version: Code 1.63.0 (Universal) (7db1a2b88f7557e0a43fec75b6ba7e50b3e9f77e, 2021-12-07T05:15:48.091Z)
OS version: Darwin x64 21.1.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
32.00GB (0.22GB free)

Process Argv
--crash-reporter-id 70c8b375-d874-4c3a-b4db-84d518c23a12

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30384386
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedt:30391933
pythonvspyt187:30373474
vsaa593cf:30376535
vsena315cf:30406436
pythonvs932:30410667
vscop804:30404766
vscop940:30404999
vsrem710cf:30405999
vscexrecpromp3t1:30407762

I am also having issues with import suggestions. Not sure what changes have been made recently but it's not able to look at the installed packages and suggest those modules. However, once I open up that module then it's able to suggest that module for import. So clearly it's doing some sort of lazy loaded analysis. Really would like it if this can be fixed soon because it's a huge productivity loss.

Could you try turning on indexing and seeing if that fixes it?  You can turn on indexing by adding the following in the settings file
 "python.analysis.indexing":true,

Thanks for your response @judej - I tried adding that into my settings.json but in fact it made it worse :( It wouldn't even suggest imports for other classes in already-imported files

To add some more detail:
It seems like the indexing is not file-specific, but rather session-specific. For example, if I import from payments.models in one file, then I start to get auto-suggestions for classes from payments.models all over my workspace. But, if I reload the window, then its like all the indices are "cleared" (I get no auto-suggestions until I explicitly import that module/file again).

Is it always the case, that import suggestions wont work, until you open that file up at least once. Because when I try settings like these: python.analysis.extraPaths and python.autoComplete.extraPaths, they seem to have no impact.

@shanewazabbas can I see your settings? it sounds like "python.analysis.indexing" didn't have any impact.

Could you try turning on indexing and seeing if that fixes it? You can turn on indexing by adding the following in the settings file
 "python.analysis.indexing":true,

Enabling indexing worked for me! The indexing configuration line looks deprecated (light color), but auto imports started to work again.

It looks like a single user (not the user that reported the bug) has reported that enabling this (undocumented?) option has fixed it for them. This is still broken for me, and I can't find a way to fix this. Did something change on how pylance resolves imports recently? I'm not sure I see the reasoning on why this issue was closed?

This broke for me on multiple PCs during the past couple of months. It was working perfectly before, unsure what changed. Is this still a thing?

@thethiny can you be more specific about what's not behaving how you want it?
Some things have changed since this bug was logged.
To start with, auto imports are disabled by default. Auto imports is overly aggressive and orders things poorly, so we disabled it by default. You can turn it back on like so:
  "python.analysis.autoImportCompletions": true,

Then on top of that there are settings to control how many modules we will search through. This is called indexing and we do this when opening a workspace.
The settings are described here:
https://github.com/microsoft/pylance-release#settings-and-customization
The ones needed to be set are:
  "python.analysis.indexing": true,
  "python.analysis.packageIndexDepths": [
    {
      "name": "",
      "depth": 2,
      "includeAllSymbols": true
    },
  ],

That will index all 3rd party modules to 2 levels up the directory hierarchy. You could also set the depth to 1 if you want it to be faster and you don't need second level modules.

@rchiodo I took what you said into consideration and have opened a new issue

I'm glad we have a suggested solution for this now! This appears to be working for us!

Yes I agree that this should be part of the Documentation since I didn't see it in any What's New recently.


Improve tbios support

V2.1 L20A support for "体験版CD"

Address was the same as V2.1 L20.

distCandidates entry for "Townsシステムソフトウェア V2.1L40"

dist (372) was the same as V2.1 L50A, so no need to add entry.

All versions and levels above are from Ribbon screen on boot.

Thanks.  I'll take it in.
I was desperately looking for Towns OS V2.1 L40, but couldn't get hold of one.  Thank you for your help!


Use latest plugin version in deployment examples

Using a specific version of the plugin in deployment examples requires updating it for each release. This poses a risk of users
deploying an older version of the plugin than the specific one they checked out.
The proposal is to always use latest and then in every release, update yaml for the release and upload it to the release page.

Hi @almaslennikov, i think its fine to bump the .yaml to last release like we did before (i.e bump to v3.4.0)
So we dont change customer experience to much.
we should first agree on this approach in a community meeting,
there are a couple of issues around release proceess and image tag management::
https://github.com/k8snetworkplumbingwg/sriov-network-device-plugin/issues/367
https://github.com/k8snetworkplumbingwg/sriov-network-device-plugin/issues/374

@adrianchiris Let's bump it to the v3.4.0 now and discuss the transition to latest or stable tags in the future

@SchSeba @zshi-redhat could you please take a look on this PR once you have a time?

LGTM

Mellanox CI failed, shall we rerun? I cannot see the details of the CI job.

/retest


ngInject does not work on direct export

Hello,
ngInject does not work when exporting a class, it worked with ng-annotate.
/* @ngInject */
class A {
    constructor(test){
      this.test = test;
    }
}

This works.
/* @ngInject */
export class A {
    constructor(test){
      this.test = test;
    }
}

This doesn't.
/* @ngInject */
class A {
    constructor(test){
      this.test = test;
    }
}

export {A};

This does.

Interesting.
While I look into this, there are a few workarounds you can use:
export class A {
    /* @ngInject */
    constructor(test){
      this.test = test;
    }
}

and
export class A {
    constructor(test){
      'ngInject';
      this.test = test;
    }
}

will both work.
In general, the second syntax (prologue directives) is going to be more reliable, as other tools/transformations should leave these untouched (whereas comments are frequently moved around or mangled by other transformations).

fixed in 0.8.0


identity column

add possibility to export identity column

Hi @romanstr,
Many thanks for the feedback,
yes, I think it's a good idea, I will analyse it and get back to you on this.

Hi @romanstr ,
For this, I have added new Settings to Enable or Disable this behavior,
Kindly get the latest version to confirm.  See This

I will now close this issue.


vmsdk/python/tests: add logic for extension

when creating a PR, please create it on a branch not main branch. otherwise it will trigger the github action on the main.

when creating a PR, please create it on a branch not main branch. otherwise it will trigger the github action on the main.

OK, I'll discard this for now. I'll post another PR later when I made it ready.


Fuzzer jobs are blocked due to lack resource to execute

Description
CI is red due to fuzzer jobs are broken due to lack resources.
https://github.com/facebookincubator/velox/actions/runs/8995033151/job/24709371527
https://github.com/facebookincubator/velox/actions/runs/8996615537/job/24713399665
Error Reproduction
No response
Relevant logs
No response

CC: @kgpai @assignUser @majetideepak

It looks like there is only 1 16-core runner it seems like the  OSS team set concurrency for that runner to 1? @mbasmanova @kgpai could either of you check? (https://docs.github.com/en/actions/using-github-hosted-runners/about-larger-runners/about-larger-runners#autoscaling-larger-runners)

Thanks @assignUser , Let me reach out to our OSS Team

Looks like its because the runner names changed recently : https://www.internalfb.com/intern/opensource/ci/github-actions/runners
Just need to rename it to 16-core-ubuntu


[New utility]Sysinternals ZoomIt

Summary of the Pull Request
This PR integrates the Sysinternals tool ZoomIt into PowerToys.
Has the PowerToys Settings page with the options allowed from ZoomIt.
Also updates some C++ dependencies.
Still missing/WIP:

[x] spell check
[x] Make it pass CI
[x] OOBE page
[x] Entries in Dashboard and quick access
[x] Settings icons / images
[x] Telemetry
[x] Logging
[x] Tweak copyright issues
[x] Tweak Community.md
[x] Verify what else might we need to localize
[ ] Test it more

PR Checklist

[x] Closes: #149
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

/azp run

@jaimecbernardo
Did you see my review above?

@jaimecbernardo Did you see my review above?

@htcfreek , thanks for the comments 😄  I was waiting to really finalize the development before addressing any review. Thank you.

Should timer color be preserved? It is only correct after I change some other timer setting
DrawMode sometimes fails to start properly, it takes the cursor and "freezes" the desktop, but there's no actual cursor and nothing happens on clicking. I can exit that mode on Esc as expected. Type mode works as expected when this happens. Also, pressing a button to change color brings the cursor up and everything starts working.
My cursor gets stuck unvisible when: ctrl+4 to enter live zoom mode press 2, select area and either save or cancel cursor is gone and stays gone until PT is closed

@stefansjfw , thank you for the review. Not sure if I tested it exactly as you, but this seems to be original ZoomIt behavior too. Have you tested with original ZoomIt too? https://learn.microsoft.com/pt-pt/sysinternals/downloads/zoomit

/azp run


JECRC Submission

Submission for the Medical Vertical using Swarm Learning by JECRC College

That was a wrong pull request. Apologies


Update Scribd-owned websites

We at Scribd are going to remove the following link https://www.scribd.com/account_settings/preferences soon.
Thus, the PR does the following:

Updates the Scribd website information
Updates the Slideshare website information. The website is owned by Scribd
Adds the Everand website information. The website is owned by Scribd

Lovely, thanks for reaching out and updating the instructions!


Updated the tox and travis configs -- BOM-1554

Relevant JIRA issue here.

Once @UsamaSadiq approves, this is ready to merge and release.  Thanks.

I forgot to mention that this should be added to the changelog


V2 - (Map Bug) - Hard to Select 'Validate' & Selecting 'Species' takes you away from the 'Explore' menu

Found in Testflight V2 (Build 35).
Two issues:

It is hard to select 'Validate This Species!" ...please make text bigger or make the text easier to select

The 'selecting Species' option takes you AWAY from the 'Explore' / Map page. This is a bug. If you are selecting species while on the 'Explore' page you should return to the 'Explore' page - on the item popup you were trying to validate - once you select a species.

Demo of bug:
https://user-images.githubusercontent.com/62637055/105562490-301b1900-5ccf-11eb-86f0-fbb00c417b5c.mov

This still seems to be an occasional issue as of Build 2 (Version 36).
I tested on an unknown tree and once I selected the tree - I was sent to the badge menu...however, it didn't happen again after I tested on other trees.
Will keep eye on it.

This still seems to be an occasional issue as of Build 2 (Version 36).
I tested on an unknown tree and once I selected the tree - I was sent to the badge menu...however, it didn't happen again after I tested on other trees.
Will keep eye on it.


Fix test failures on windows, mostly paths

After merging the other code fixes.
Only unit tests, not functional tests...

--- FAIL: TestTemplate (0.00s)
    template_test.go:26: assertion failed: error is not nil: field mounts[0] must be absolute, got "/Users/dummy"
--- FAIL: TestTemplate9p (0.00s)
    template_test.go:54: assertion failed: error is not nil: field mounts[0] must be absolute, got "/Users/dummy"
FAIL
FAIL	github.com/lima-vm/lima/pkg/cidata	1.177s
--- FAIL: TestTXTRecords (0.07s)
    --- FAIL: TestTXTRecords/test_TXT_records (0.07s)
        dns_test.go:60: assertion failed: regex_check is false
FAIL
FAIL	github.com/lima-vm/lima/pkg/hostagent/dns	1.229s
--- FAIL: TestFillDefault (0.01s)
    defaults_test.go:212: assertion failed:
        --- &y
        +++ &expect
          &limayaml.LimaYAML{
          	... // 12 identical fields
          	Containerd: {System: &false, User: &true, Archives: {{Location: "https://github.com/containerd/nerdctl/releases/download/v0.21.0/"..., Arch: "x86_64", Digest: s"sha256:728f9b543374b1b1733f759608e156dbe578d7b140a081084a1f4bfb4"...}, {Location: "https://github.com/containerd/nerdctl/releases/download/v0.21.0/"..., Arch: "aarch64", Digest: s"sha256:1d0c822f7571042e71ef0b2f8d092f99b0034061726385ca90deaf0b3"...}}},
          	Probes:     {{Mode: "readiness", Description: "user probe 1/1", Script: "#!/bin/false"}},
          	PortForwards: []limayaml.PortForward{
          		{GuestIP: s"<IP_ADDRESS>", GuestPortRange: {1, 65535}, HostIP: s"<IP_ADDRESS>", HostPortRange: {1, 65535}, ...},
          		{GuestIP: s"<IP_ADDRESS>", GuestPort: 80, GuestPortRange: {80, 80}, HostIP: s"<IP_ADDRESS>", ...},
          		{GuestIP: s"<IP_ADDRESS>", GuestPort: 8080, GuestPortRange: {8080, 8080}, HostIP: s"<IP_ADDRESS>", ...},
          		{
          			... // 6 identical fields
          			HostPort:      0,
          			HostPortRange: {1, 65535},
          			HostSocket: strings.Join({
          				`C:\users\anders | C:\users\anders\.lima\instance | instance | S-`,
          				"1-5-21-0-0-0-1000 | ",
        - 				`UBUNTU\anders`,
        + 				"lima",
          			}, ""),
          			Proto:   "tcp",
          			Reverse: false,
          			Ignore:  false,
          		},
          	},
          	Message:  "",
          	Networks: {{Lima: "shared", MACAddress: "52:55:55:f0:96:23", Interface: "lima0"}},
          	... // 7 identical fields
          }

FAIL
FAIL	github.com/lima-vm/lima/pkg/limayaml	1.270s
--- FAIL: TestLogFile (0.00s)
    commands_test.go:47: assertion failed: C:\users\anders\.lima\_networks/name_daemon.stream.log (logFile string) != C:\users\anders\.lima\_networks\name_daemon.stream.log (string)
--- FAIL: TestUser (0.00s)
    commands_test.go:59: assertion failed: error is not nil: No security ID mapped.
FAIL
FAIL	github.com/lima-vm/lima/pkg/networks	0.880s

Note: you can run these on Linux, by installing wine-binfmt package:
anders@ubuntu:~/lima (windows)$ file _output/bin/limactl
_output/bin/limactl: PE32+ executable (console) x86-64 (stripped to external PDB), for MS Windows
anders@ubuntu:~/lima (windows)$ _output/bin/limactl --version
limactl version 0.11.1-5-g57c7f8f

It also installs wine64, as a dependency.  See https://www.winehq.org/

Thanks, can we enable Windows CI for unit tests?

I think so, might need some GitHub actions expert...
I used GOOS=windows and wine-binfmt (Ubuntu)

@afbjorklund LGTM, but could you address this https://github.com/lima-vm/lima/pull/915#discussion_r901962804

ping: https://github.com/lima-vm/lima/pull/915#discussion_r908278316

I moved the other test too, maybe I missed something ?

I moved the other test too, maybe I missed something ?

Can we move the whole commands_test.go to commands_darwin_test.go?


RGH is injecting code into websites other than Github.

Description
https://techcommunity.microsoft.com/t5/outlook/got-weird-internal-publishing-page-instead-of-login/m-p/3902338/highlight/false#M17590
This was my reply to someone having outlook login issues after doing a google search for a portion of the injected code appearing in my Userstyle CSS scripts as I installed them:
"This appears to be caused by the firefox addon Refined-Github. Turn off the permission "Access your data for all websites" on the permissions tab and it should fix it. For me it was inserting extra code to userstyle CSS scripts from userstyle.world and some already install userstyles in FireMonkey. I realized looking at you post what the "RGH" CSS handle was for Refined-GitHub."
How to replicate the issue + URL
Enable the "Access your data for all websites" permission.
Install a userscript from userscript.world into FireMonkey or apparently try to log into outlook.com and the follow code appears (note, this has been formatting via prettyprint):
:where(.user-mention:not(.commit-author)[data-hovercard-type="user"]):not(
.rgh-seen--26062470120
) {
animation: 1ms rgh-selector-observer;
}
:where(
.js-issue-row [data-hovercard-type="user"],
.notification-thread-subscription [data-hovercard-type="user"]
):not(.rgh-seen--26062499911) {
animation: 1ms rgh-selector-observer;
}
:where(
.js-hovercard-content
.Popover-message
div.d-flex.mt-3.overflow-hidden
> div.d-flex,
.js-hovercard-content
.Popover-message
div.d-flex.mt-3
> div.overflow-hidden.ml-3
):not(.rgh-seen-722523370) {
animation: 1ms rgh-selector-observer;
}
:where(
[itemprop="homeLocation"] svg.octicon-location,
[aria-label="User location"] svg.octicon-location
):not(.rgh-seen-668154795) {
animation: 1ms rgh-selector-observer;
}
:where(a[href*="?tab=repositories"]:is([href^="/"], [href^="https://userstyles.world/"]):not([href*="&type="], .issues-reset-query),a[href*="/repositories"]:is([href^="/orgs/"], [href^="https://userstyles.world/orgs/"]):not([href*="&type="], .issues-reset-query)):not(.rgh-seen--393030416)
{
animation: 1ms rgh-selector-observer;
}
:where(
.BorderGrid--spacious .f4.my-3,
.js-commits-list-item pre,
.js-commit-group pre,
.release-header,
.Box-row .mb-1 a,
#pull-requests a.Link--primary,
[id^="check_suite"] a.Link--primary,
.js-socket-channel[data-url*="/header_partial"] h3,
.js-wiki-sidebar-toggle-display a,
#wiki-wrapper .gh-header-title,
#user-repositories-list [itemprop="description"],
.js-hovercard-content > .Popover-message .Link--primary,
.search-title a,
.notification-thread-subscription [id^="subscription_link_"]
):not(.rgh-seen-27766609435) {
animation: 1ms rgh-selector-observer;
}
:where(
a:is(
[href*="/issues"],
[href*="/pulls"],
[href*="/projects"],
[href*="/labels/"]
):not([href*="sort%3A"], .issues-reset-query)
):not(.rgh-seen--1282261975) {
animation: 1ms rgh-selector-observer;
}
:where([data-hydro-view*="pull-request-hovercard-hover"] ~ .d-flex.mt-2):not(
.rgh-seen--1968497660
) {
animation: 1ms rgh-selector-observer;
}
:where(.comment-body a[href]:not(.rgh-linkified-code)):not(
.rgh-seen-26849672246
) {
animation: 1ms rgh-selector-observer;
}
Extension version
24.4.9
Browser(s) used
Firefox 127.0a1 (2024-04-20) (64-bit)

Duplicate of https://github.com/refined-github/refined-github/issues/7319

I guess I should have said RGH is ignoring the "Domain Selector" option as this behavior is observed with either setting. Also this is a rather recent change, I had that setting that way for a long while and never had an issue. I assumed it was for the preview popup cards that would appear during a mouseover of a link (which hasn't worked for me for a long while), so links outside Git leading into it could show those popups.
If it's required for Enterprise users, wouldn't a whitelist be more appropriate? Even Enterprise users still use Microsoft and Google. How are they limiting the scope of these injections? I don't see any option besides "Default" and "*" and I had it set to Default.
I don't mean to bug you and I don't require a fix as the feature I thought it was for doesn't work anyway, I am just trying to make sure you're not dismissing an issue that is pretty hard for the average user to trace and could be effecting many others without you ever knowing. You have an auto update mechanism to fix it rather easily in comparison to the thousands of hours being collective wasted.

Enable the "Access your data for all websites" permission.

I don't really understand what you expect to happen when you enable this option.  Don't waste ours

this is a rather recent change

It's not, our permissions haven't changed since 2017-ish

I assumed it was for the preview popup cards that would appear during a mouseover of a link

That's a native GitHub feature

wouldn't a whitelist be more appropriate?

That's what we already have, but the browser also shows users that toggle. You're not supposed to enable that if you don't need it, and you don't.

to fix it rather easily

There's no bug, there's nothing to fix. In order to have the whitelist as you mentioned, we must specify that optional permission for all websites.


Updated Bot Class

The bot now uses the standard bot class (discord.ext.commands.bot) used by discord bot developers. It also now has the message content intent to allow it to read messages. It also now contains a prefix for commands, currently $.

Thanks for your contribution !


The argument type 'JsObject' can't be assigned to the parameter type 'BuildContext'.

how can i use here navigator basically my requirement is to whenever a user click on list item it should be navigate to different page but i am using list tile inside the list tile i am unable to use navigator

Hi @rohan1234
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

hey @rohan1234
i too have same issue and when i looked into it actually it auto imported dart.js  which when i removed, the problem got solved. you can try it!!

I also faced same issue in one of my projects..

I did was:
removing 'import dart:js';
and use BuildContext context;
problem solved.


OOP Wrapper

Classes should be wrapped so that the C++ interface doesn't have to be accessed in a C-style way and so that users of the binding don't have to be aware of the name mangling the interface employs.

Graphite might be acceptable solution to this. It'd make the project move from being  a single source file, which might make the code more readable.

Graphite and Carbon are definitely too large for a project like this. I'll write an OOP implementation based around proxies and structs.


patchwork_parameters

hi,  enable_RNR and enable_RVPF is set false by default, Did you not use these two methods?

Hi, thank you for asking the question!
For both methods, I used both of them as stated in the paper. However, as you said, the both parameters are set false by default in the master branch now. This is because it needs additional codes for the general application ( e.g. point types(XYZ, XYZI, etc), sorting order of points ). The original codes, which I have used for experiments, have some dependencies on the data formats. Also, as I apply pybind, there's some changes in the codes compared to the original one and I'm now checking the potential issue in the new code, including R-VPF, now. Please wait for a while, I'm planning to update the generalized code which can execute Patchwork++ for cases as many as possible. Thank you.

Hi,
Is there any update or the planning finish data?

@whuzs, @Shaoqing26
At first, sorry for the late update of the code.
I have updated the code, which makes

Reflected Noise Removal(RNR) be available regardless of sorted order of points.

The previous code requires the sorted point cloud input in the descending order of ring information. However, the newely updated code can be operated with the unsorted point cloud.
The default value of following parameter enable_RNR has been changed to true.
As the new code doesn't require the ring information, the specific thresholds for RNR have been changed slightly from the previous one: pc_num_channel, noise_filter_channel_num and intensity_thr. The new parameters are RNR_ver_angle_thr and RNR_intensity_thr. But the essential role of RNR has not been changed.

Region-wise Vertical Plane Fitting(R-VPF) be corrected.

As I commented before, I have been struggled to correct the functionality of R-VPF due to the data formats and pybind.
The default value of following parameter enable_RVPF has been changed to true.
The updated code works properly now. If you find anything wrong, please let me know.

I will close the issue because the related codes are updated.
Thank you :)

@whuzs, @Shaoqing26
At first, sorry for the late update of the code.
I have updated the code, which makes

Reflected Noise Removal(RNR) be available regardless of sorted order of points.

The previous code requires the sorted point cloud input in the descending order of ring information. However, the newely updated code can be operated with the unsorted point cloud.
The default value of following parameter enable_RNR has been changed to true.
As the new code doesn't require the ring information, the specific thresholds for RNR have been changed slightly from the previous one: pc_num_channel, noise_filter_channel_num and intensity_thr. The new parameters are RNR_ver_angle_thr and RNR_intensity_thr. But the essential role of RNR isn't changed.

Region-wise Vertical Plane Fitting(R-VPF) be corrected.

As I commented before, I have been struggled to correct the functionality of R-VPF due to the data formats and pybind.
The default value of following parameter enable_RVPF has been changed to true.
The updated code works properly now. If you find anything wrong, please let me know.

I will close the issue because the related codes are updated.
Thank you :)

Thanks for your kind reply.
And I found that this work support ROS, Is this problems process in ROS Project?

Yes. I will also update the corresponding codes in the ROS repository as soon as possible.
Thanks for your interest :+1:.

Yes. I will also update the corresponding codes in the ROS repository as soon as possible. Thanks for your interest +1.

Thanks, Please info me if update it at your convenience. It's an exciting Job. Thanks.

The ROS repository is also updated. You can visit here to try Patchwork++ based on ROS. Thank you :)


get_angles always returns a fixed value

Hi,
I am using the MyCobot 280 Jetson Nano robot arm. I have initialized the interface in Python with the following commands:
from pymycobot.mycobot import MyCobot
mc = MyCobot('/dev/ttyTHS1',1000000)
When I run mc.get_angles(), the returned value is always: [180.08, 180.08, -180.08, 180.08, 180.08, 180.08]
Even after rotating various joints, mc.get_angles always returns: [180.08, 180.08, -180.08, 180.08, 180.08, 180.08]
Please let me know how this can be fixed.

Please use myStudio to re-burn the latest atom firmware to see if the problem is solved.


Map not showing up in android

Using the android app, the map at the top of the arrival predictions pages fails to show up. The space for it is still there but none of the tiles seem to load. This doesn't happen with the web app. Tried on two different android devices.

Thanks for reporting this! I think I know the issue. :)

Hi @Nate-Wessel,
I tried to reproduce this on the simulator and was not able to see the issue. Now, we have changed to open maps since then so that could be the difference. The only other thing I can think of is that maybe you ran into the old favorites issue? I wrote it up on Trello: https://trello.com/c/dCY58IZX
If you could check and see if this is still an issue for you, I would really appreciate it. :)
Thanks,
KT

KT,
It looks like it's working now. I'm pretty sure it was trying to use Google maps when I had the problem.
-Nate