Unusual route values appearing when using Url.Action

From @MorayM on December 6, 2017 9:31
I'm trying to create a link to the following GET:
public class FooController : Controller
{
    [HttpGet]
    [Route("Foo/Apply/BarDetails/Part/{id}")]
    public IActionResult BarDetails(int id)
    {
    }
}

In my view I'm creating a URL with
Url.Action("BarDetails", "Foo", new {id = 1})

I was expecting this to generate Foo/Apply/BarDetails/Part/1 however I actually get Foo/Apply/BarDetails/Part/10?id=1 and I'm struggling to understand why. The page this link is on has no id parameter in its route (or indeed any parameters in its route) so there shouldn't be any ambient values. Removing the values parameter simply removes the query string part of the URL. The only route I have registered at startup is the default:
app.UseMvc(routes =>
{
    routes.MapRoute(
        name: "default",
        template: "{controller=Home}/{action=Index}/{id?}");
});

I suspect that the 10 is coming from a "1" + "0" operation happening somewhere but I haven't found it yet in the source! Does anyone know why this might be happening?
Copied from original issue: aspnet/Mvc#7112

From @MorayM on December 6, 2017 14:57
Hmm, looking at the eventual source I guess this should probably be moved to the Routing module?

From @MorayM on December 6, 2017 14:55
OK, I've figured it out. It's mostly a quirk in the the way I've written my FooController but as FooController originally came from an MVC5 project where the routing worked I thought I'd log it and leave it to the team to decide if the current MVC Core behaviour is a bug or a feature!
My FooController is responsible for a form that consists of ~250 views organised in sections and looks a bit like this:
public class FooController : Controller
{
	[HttpGet]
	[Route("Foo/Apply/Section1/Part/{id}")]
	public IActionResult Section1(int id)
	{
		var model = GetModelForSection1(id);//get the model for the requested section
		var viewPath = GetViewPathForSection1(id);//Calculate the path to the viewPath
		return View(viewPath, model);
	}

	[HttpPost]
	[Route("Foo/Apply/Section1/Part/1")]
	public IActionResult Section1(Part1Model model)
	{
		// Do whatever with the model
		return RedirectToAction("Section1", 2);
	}

	[HttpPost]
	[Route("Foo/Apply/Section1/Part/2")]
	public IActionResult Section1(Part1Model model)
	{
		// Do whatever with the model
		return RedirectToAction("Section1", 3);
	}

	// etc for 10 more views in Section 1...
	// etc for 6 more sections...
}

The POST actions are all named the same for each section so we didn't have to manually specify actions for the forms in each of the 250 different views!
The problem in generating routes with Url.Action comes when the Routing module tries to find a match for the controller name/action name combination. When given a controller name Foo and an action name Section1, LinkGenerationDecisionTree.GetMatches finds a list of matches like this:
"Foo/Apply/Section1/Part/{id}"
"Foo/Apply/Section1/Part/1"
"Foo/Apply/Section1/Part/2"
etc...

Which is fine until line 30 where GetMatches sorts the list of matches by precedence, which sends Foo/Apply/Section1/Part/{id} (with precedence 5.5553) to the bottom, and then by the route template text, and as the rest of the matches have the same precedence (5.555) this sends Foo/Apply/Section1/Part/10 to the top of the list. This is the first match for controller name Foo and action name Section1, hence why Foo/Apply/Section1/Part/10 always appears:
"Foo/Apply/Section1/Part/10"
"Foo/Apply/Section1/Part/11"
"Foo/Apply/Section1/Part/12"
"Foo/Apply/Section1/Part/1"
"Foo/Apply/Section1/Part/2"
etc...
"Foo/Apply/Section1/Part/{id}"

As I said, I'll leave it up to the team to decide whether this is the intended behaviour or not but I thought it was worth flagging up as the end result is different from the behaviour in MVC5. And also because I've had such a fun time this morning hunting it down I thought I'd share the joy :wink:
In the meantime I'll reorganise my controller to give my POST actions unique names...

From @davidliang2008 on December 19, 2017 19:40
I have the exact same issue! @Url.Action() and even the tag helper asp-controller and asp-action would have the route value appended at the end of the URL.
And that route value looks like the id of the current request URL.
For example, if I have an edit page /product/edit/10 but I want to display edit links for other products on that page, those links would look like
/product/edit/10?id=1
/product/edit/10?id=2,
....
/product/edit/10?id=10000

@pranavkm, can you please handle this? /cc FYI @rynowak, this might be an actual routing bug

This definitely intended, whether or not it's ideal is another question.
You can also configure the order in which routes are processed by setting an order on the route attribute. Setting a lower order will cause that route to be processed first for both matching and link generation


Is there a way to reduce the HAMI-Core verbosity level for workloads?

Please provide an in-depth description of the question you have:
I reviewed the HAMI-Core and confirmed that the verbosity level can be reduced by setting the LIBCUDA_LOG_LEVEL environment variable. However, configuring this for every GPU pod can be tedious.
Is there a way to set the verbosity level through HAMI’s Helm chart or scheduler configuration instead?
What do you think about this question?:
I believe the user should have easy access to configure this parameter, and it could be integrated with the already existing admission webhook. Additionally, I recommend setting the default HAMI-Core verbosity level to 0, ensuring consistent behavior with Nvidia’s device-plugin.
Environment:

HAMi version: 2.4.0
Kubernetes version: 1.26.5
Others: -

There is no good solution at the moment.
If HAMi can try to read global configuration information through webhook, set this parameter. Not sure if it is feasible, need to try

you can modify mutatingWebhookConfiguration in HAMi,  add env LIBCUDA_LOG_LEVEL=0 to GPU pods, by the way ,do you have a WeChat or Linkedin account?

@archlitchi thanks for the reply, I will try to implement setting LIBCUDA_LOG_LEVEL during admission.
Unfortunately I don't have WeChat but I have a linkedin account.

I have reviewed the code and believe it can be easily implemented, but I have a concern regarding ArgoCD and GitOps. Overriding the pod spec, whether it's to modify the environment variable for visible CUDA devices or any other environment variable, would likely trigger an out-of-sync state.
@archlitchi what do you think?

I tested various scenarios, and the out-of-sync state is triggered only when bare pod manifests are applied via ArgoCD with set environment variables that can be modified by the admission webhook. Given this, I think adding a note about it in the documentation and proceeding with the environment variable mutation approach would be the best solution.

I tested various scenarios, and the out-of-sync state is triggered only when bare pod manifests are applied via ArgoCD with set environment variables that can be modified by the admission webhook. Given this, I think adding a note about it in the documentation and proceeding with the environment variable mutation approach would be the best solution.

i haven't tried submitting tasks with ArgoCD, i think we can add a field in values.yaml, regarding the log-level, it can be set to 2(which is the default log level, errors, warns and msgs), 0(errors only), 3(errors, warns,msgs and infos), 4(debugs, msgs, infos, warns, errors). we only patch the 'LIBCUDA_LOG_LEVEL' env to container is not set to 2.


sslv3 problem in archlinux on raspberry

compile in archlinux on raspberrypi and in link step it shows:
tmp/objs/Mayaqua/Network.o: In fuction 'StartSSLEx':
Network.c(.text+0xa15c):undefinedreference to 'SSLv3_method'
how to change the source to prevent it?

why not using the available package?

There's a patch available here (AUR link)


mise plugins list should align with output of plugins section in mise doctor

Currently mise plugin list output just list of names of non-core plugins installed. And imo it's usefulness is pretty limited.
I propose to output same info as in mise doctor's plugins section.
mise plugins list

mise doctor

"core plugins" is a misnomer
Idk but to me that's exactly what they are. They act like a plugins and they are built-in (i.e. core).

at some point mise plugins may not involve tools at all
can you please elaborate?

Changing the output of mise plugins would be a breaking change
Agree, i didn't think about it. Would it make sense to adding extra command list-urls (or smth like that) that so it will output path/url of plugins (without "core plugins")

ok, i'll guess we can leave it as is for now


Better support for nested JavaScript classes

Following this Gitter question.
Here is an example of the original JavaScript:
var port = tessel.port.A;
var i2c = new port.I2C(slaveAddress)

Assume we have:
trait Port extends js.Object {
  ...
}

We can use js.Dynamic to create an instance of class defined within another JavaScript object:
val port: Port = ???
val i2c = js.Dynamic.newInstance(port.asInstanceOf[js.Dynamic].I2C)(DisplayI2CAddress)

But there is currently no way to simply express that there is a nested class. For example, the following doesn't compile:
@js.native
trait Port extends js.Object {
  @js.native
  class I2C(address: Int) extends js.Object
}

As a workaround, I implemented the following:
object Port {
  implicit class PortOps(val port: Port) extends AnyVal {
    def I2C(address: Int) =
      js.Dynamic.newInstance(port.asInstanceOf[js.Dynamic].I2C)(address)
  }

But it's a lot of boilerplate compared to just using a @js.native annotation.

Fixed in 9e9883e4eb2f7e76e246d3fcf51e57bbeef45e42.


Protectecd APK File Could NOT Upload to Play Store!

Hi and thank you for this amazing tool it's very good and helpful to protect our content from reverse engineering or modding the two main feature Refactor and Protect are so great to protect APK file against any editor or modder but it have a small bug when you want to upload the protected file to play store you get some errors that app not compiled properly or it doesn't have meta data com.android.vending.derived.apk.id my apk file is already included that meta data and it's compiled without any errors and it's signed with same signature certificate I can install on my devices but I don't know what is the problem play store console doesn't let me upload it to release new protected version.
Need some help please
Here is the screenshot:

Most probably the parser on google is failed because of our protection on manifest. The solution is to protect files without manifest. Maybe I should add -skip-manifest  ?

Most probably the parser on google is failed because of our protection on manifest. The solution is to protect files without manifest. Maybe I should add -skip-manifest ?

Yes you right the protecting/refactor is somehow changes the manifest that should not touch it becuase it can't be uploading to play store. Could you plesae fix that :)

You can skip AndroidManifest protection by using flag -skip-manifest
java -jar APKEditor.jar p -skip-manifest -i your_file.apk
Check the latest update

Okay bro thank you let me check it now wait please..

You can skip AndroidManifest protection by using flag -skip-manifest
java -jar APKEditor.jar p -skip-manifest -i your_file.apk
Check the latest update

I tested it's same warning like before this happens only when you Protect apk file it's okay with Refactor
I used the latest jar file on release page

My guess is the parser at playstore failed when it  tries to resolve references ( like app name, icon ...) from manifest, this references linked to resource.arsc  which is protected

My guess is the parser at playstore failed when it tries to resolve references ( like app name, icon ...) from manifest, this references linked to resource.arsc which is protected

So what things do I need to do to fix that problem in my app content?

I don't know may be use custom resources just like facebook which loads at runtime ??? BTW I am surprised play store still accepts .apk format

I don't know may be use custom resources just like facebook which loads at runtime ??? BTW I am surprised play store still accepts .apk format

No I doesn't used any facebook resources and my app is no need any permission to work only used this <uses-permission android:name="android.permission.VIBRATE"/> one to vibrate when user tap on keys!
And yeah play store is still support .APK format but it always advise you to use an .AAB app bundles to save storage and faster installation time on devices.
Anyways thanks for your work I like it so much the Refactor is maybe good enough for now.


Handle multiple NuGet package sources

Read list of sources from %APPDATA%\NuGet\NuGet.config (need to find location for Linux computers)

This file is in these locations on Linux depending on the installation:
~/.config/NuGet/NuGet.Config
~/.nuget/NuGet/NuGet.Config
~/.config/NuGet/config/NuGet.config
~/.nuget/config/NuGet.config

Thanks! I ran Pinax on some of my current client's code and noticed I wasn't able to pick up things in their private NuGet source

I primarily do .NET stuff in the CLI on Linux so if you need those resources I can provide them.   There are some other possibilities depending on installation (WSL, WSL2, etc. but.. those 4 should cover it. but these locations are also managed sources to NuGet stuff:
~/.local/share/NuGet/Cache
~/.nuget/packages

I checked in changes that "should" read the NuGet.config and read multiple sources. I don't have that setup locally, so will test Monday with my client.
@gbowne1 If you have an environment with multiple NuGet sources, can you please test these changes (especially if you have this in a Linux environment). It's using "Environment.SpecialFolder.ApplicationData", and I hope that handles the different folder locations for Windows/Linux/Mac.
If this is good, I will probably release this as version 1.1 and do some much-needed refactoring before starting work on 1.2

Sory I missed this, Scott.  Yeah I am goin to do some testing with this in Linux as well since that's my main environment. There's been some minor issues, but I still need to see if it's me, Pinax or something else I'm doing, otherwise it's been fine.
I do have several customers with multiple NuGet sources and I still need to do more testing.  Same though, this is working pretty well for what I have needed.  I have some clients/customers that just don't update much.  I've been looking for som

I think this needs some more work. I still have the outstanding issue to handle security. I mostly checked this change in since it didn't seem to be hurting anything - even if it doesn't fix everything. I need to re-think the purpose of the program, now that I've been using it a bit.


Missing CryptoPosition

Hey @tstromberg great stuff!
I saw you mentioned here how you added Crypto Position based on my PR here.
I wasn't able to find the it in the codebase in the main branch.
Keep up the great work!
Nico

Oops! I will check out what happened there. I have not yet tested the
crypto-specific parts of this library, but plan to soon.
On Fri, Oct 1, 2021, 01:38 Nicolas Vayias @.***> wrote:

Hey @tstromberg https://github.com/tstromberg great stuff!
I saw you mentioned here https://github.com/tstromberg/roho/pull/7 how
you added Crypto Position based on my PR here
https://github.com/andrewstuart/go-robinhood/pull/21/.
I wasn't able to find the it in the codebase in the main branch.
Keep up the great work!
Nico
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tstromberg/roho/issues/11, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYYMCFBXX4JMKBVDTUWKLUEVXOPANCNFSM5FEE5OQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[DOC] _nodes/stats coordinator search stats documentation

Description
Adding documentation for commit https://github.com/opensearch-project/OpenSearch/pull/8386
https://opensearch.org/docs/latest/api-reference/nodes-apis/nodes-stats/
Issues Resolved
Resolves https://github.com/opensearch-project/documentation-website/issues/5446
Checklist

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

Thanks! @vagimeli @natebower


[🐞] ESLint dev configuration errors

Which component is affected?
Qwik Rollup / Vite plugin
Describe the bug
I am running dev mode with a tsconfig.json that has project references defined.
I am getting a Parsing error: ESLint was configured to run on `<tsconfigRootDir>/src/root.tsx` using `parserOptions.project`: <tsconfigRootDir>/tsconfig.json warning (which is slowing down the initial dev build since it's printing it for every file referenced).
I expect there not to be an error.
Reproduction
https://stackblitz.com/edit/qwik-starter-ro4pwa?file=tsconfig.app.json,tsconfig.json
Example warning:
[vite] warning: Parsing error: ESLint was configured to run on `<tsconfigRootDir>/src/root.tsx` using `parserOptions.project`: <tsconfigRootDir>/tsconfig.json
However, that TSConfig does not include this file. Either:
- Change ESLint's list of included files to not include this file
- Change that TSConfig to include this file
- Create a new TSConfig that includes this file and include it in your parserOptions.project
See the typescript-eslint docs for more info: https://typescript-eslint.io/linting/troubleshooting#i-get-errors-telling-me-eslint-was-configured-to-run--however-that-tsconfig-does-not--none-of-those-tsconfigs-include-this-file
  Plugin: vite-plugin-qwik
  File: /home/projects/qwik-starter-wmq6la/src/root.tsx:undefined:undefined

Steps to reproduce

npm install
npm run dev
Terminal should be logging the warning.

System Info
System:
    OS: macOS 13.5.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 1.29 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.13.0 - ~/.nvm/versions/node/v18.13.0/bin/node
    npm: 8.19.3 - ~/.nvm/versions/node/v18.13.0/bin/npm
  Browsers:
    Chrome: 116.0.5845.96
    Firefox Developer Edition: 116.0
    Safari: 16.6
  npmPackages:
    @builder.io/qwik: ^1.2.7 => 1.2.7
    @builder.io/qwik-city: ^1.2.7 => 1.2.7
    undici: ^5.23.0 => 5.23.0
    vite: 4.4.9 => 4.4.9

Additional Information
I think this is related to: https://github.com/BuilderIO/qwik/pull/4873
Digging into the code a little bit, I think the issue comes from the eslint options defined here:
https://github.com/BuilderIO/qwik/blob/main/packages/qwik/src/optimizer/src/plugins/eslint-plugin.ts#L30
I can fix this by exposing the tsconfigFileNames option here:
https://github.com/BuilderIO/qwik/blob/main/packages/qwik/src/optimizer/src/plugins/plugin.ts#L83
and defining something like { tsconfigFileNames: ['tsconfig.json', 'tsconfig.app.json'] } in the vite plugin config.
(I'm happy to put up a PR that exposes this option.)

PR would be great! Go for it!


actions events don't register for ie9 virtual machine (windows 7)

When reporting a bug, make sure you provide all required info below. Reports without this information may be closed.
ie9 vm (windows 7) does not register 'actions.sendKeys'.  Test indicates the suite passed, but input field is empty, and no onchange events are triggered. The behavior work on firefox and chrome running on the host machine, I haven't tried it with other ie/vm combinations yet.
Edit: running both server-standalone (v. 3.8.0) and iedriver (v.3.8.0) on the vm.
Output of gemini --version:
5.1.2
...

Contents of `.gemini.js` file:
{
	"rootUrl": URL,
	"gridUrl": "http://<IP_ADDRESS>:4444/wd/hub",
	"compositeImage": true,
	"browsers": {
		"ie9": {
			"gridUrl":VM_GRID + ':4444/wd/hub',
			"rootUrl":URL_for_VM,
			"desiredCapabilities":{
				"browserName": "internet explorer",
				"version": 9,
				"platform": "WINDOWS",
			}
		},
		// "chrome": {
		// 	"desiredCapabilities": {
		// 		"browserName": "chrome"
		// 	}
		// },

               ...
}

Test source code:
gemini.suite('suit 1', (suite) => {
  suite.setUrl('/')
      .before((actions, find) => {
        actions.executeJS(function (window) {
          var style = window.document.createElement('style');
          style.innerHTML = 'html { overflow: hidden; }';
          window.document.head.appendChild(style);
        })
      })
      .setCaptureElements('[data-test="main"]')
      .capture(`stepone: fill first-name`, (actions, find) => {
          var inputField = find(`[data-test="first-name"]`);
          actions.click(inputField)
                      .sendKeys(inputField, 'first-name');
  })
})

Command used to run the test:
gemini update

EDIT: works on ie10 and ie11

maybe it because of such selector? can you try to match on class instead of attributes?

hmmm tried it with id and class, it came back with the same results. Here are the things I have tried so far:

Using different combinations of iedriver and server-standalone versions (from 2.39 to 3.9.1)
Setting the selenium-server-standalone role to 'node' and hosting a hub on the host machine


Followed all instructions and post CMAKE, make -j does nothing, no error

Hi,
finally compiled binary dbscan and can run it.
The sample file given in the tutorial is 40 lines long. When I run with it and eps 0.1, minpts 10 dbscan runs and outputs all points as outliers. Is this OK?
thanks!

With larger EPS, smaller Minpts one can get clustering.

Hi @sleeplessTLV
For some data sets a suitable set of parameter needs to be chosen to cluster properly.
Feel free to share the data set and parameter where you got errors and I can take a look.

Dear Dr. Wang,
in the end we managed to set relevant eps and minpts values for this
dataset and got good clustering.
Sorry for bothering you. We will now proceed to try on more heavy datasets.
Thank you very much for the prompt response!
BR
Tsvi
On Sat, Jan 7, 2023 at 10:50 PM Yiqiu Wang @.***> wrote:

Hi @sleeplessTLV https://github.com/sleeplessTLV
For some data sets a suitable set of parameter needs to be chosen to
cluster properly.
Feel free to share the data set and parameter where you got errors and I
can take a look.
—
Reply to this email directly, view it on GitHub
https://github.com/wangyiqiu/dbscan-python/issues/14#issuecomment-1374607705,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMEPFILIK3MGD44OGI2TNTWRHJG5ANCNFSM6AAAAAATUCGHWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Dear Dr. Wang,
in the github code, it is not clear to us if/how the various modes of
BCP/Delaunay/USEC can be used or just one? And same question for grid vs
box?
thanks
Tsvi
On Sat, Jan 7, 2023 at 10:50 PM Yiqiu Wang @.***> wrote:

Hi @sleeplessTLV https://github.com/sleeplessTLV
For some data sets a suitable set of parameter needs to be chosen to
cluster properly.
Feel free to share the data set and parameter where you got errors and I
can take a look.
—
Reply to this email directly, view it on GitHub
https://github.com/wangyiqiu/dbscan-python/issues/14#issuecomment-1374607705,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMEPFILIK3MGD44OGI2TNTWRHJG5ANCNFSM6AAAAAATUCGHWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @sleeplessTLV
Glad that you got the parameter working. No problem - I will be happy to answer your questions.
This dbscan-python repository was re-written later, and is slightly different from our code for the paper.
The code for the paper is here: https://github.com/wangyiqiu/dbscan
The reproducibility instructions are here: https://github.com/wangyiqiu/dbscan-reproducibility
More specifically, to switch among different methods, please refer to the instructions here: https://github.com/wangyiqiu/dbscan
I will have to dig a little bit to find the code that switches among BCP/Delaunay/USEC, grid/box. Hopefully the links above are helpful to you.

Thanks a lot! Very helpful.
On Mon, Jan 9, 2023 at 7:25 PM Yiqiu Wang @.***> wrote:

Hi @sleeplessTLV https://github.com/sleeplessTLV
Glad that you got the parameter working. No problem - I will be happy to
answer your questions.
This dbscan-python repository was re-written later, and is slightly
different from our code for the paper.
The code for the paper is here: https://github.com/wangyiqiu/dbscan
The reproducibility instructions are here:
https://github.com/wangyiqiu/dbscan-reproducibility
More specifically, to switch among different methods, please refer to the
instructions here: https://github.com/wangyiqiu/dbscan
I will have to dig a little bit to find the code that switches among
BCP/Delaunay/USEC, grid/box. Hopefully the links above are helpful to you.
—
Reply to this email directly, view it on GitHub
https://github.com/wangyiqiu/dbscan-python/issues/14#issuecomment-1375987976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMEPFPX4OHY7BK6SSABQGLWRRCXFANCNFSM6AAAAAATUCGHWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Implement itemized description list

Implement the itemized description list first introduced in Asciidoctor EPUB3. This list combines the semantics of an itemized list and a description list.
See https://github.com/asciidoctor/asciidoctor/issues/821.

I'm going to move this to core since it's worthy of discussing for the AsciiDoc spec.


Detect when scroll is not needed

I want to hide the scrollbar when there's not enough content for the scroll to be needed but I don't see a way to detect when this is happening.

Nevermind, I had a weird margin/border issue causing my scrollbar to stay at 99% height, not a problem with the plugin


For Interest - Do Not Merge

Changes since last running the polymer 3 converter script in Aug (includes script changes, and icons repo changes)

Closing, Polymer 3 conversion is complete


Implement backward compatibility to old HS grid

Consider making it optional via a boolean.
https://github.com/TheWebTech/CrankShaft/blob/master/CrankShaft/code/cs/cs-grid.css

added backward compatibility in, but did not add boolean to toggle this functionality. Should add.


Include parameter info in the 'change' event

This allows 'change' event listeners to know what has changed in the view.

@googlebot I signed it!


Terminal Screen is all covered with  black cant even see the text

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: newest by 16 july 2023  1.80.1
OS Version:  windows 10

The terminal is all black cant see the commands im typing and i have tried uninstalling and app again and again but nothing is working

Steps to Reproduce:

any solution for that  ??

/duplicate https://github.com/microsoft/vscode/issues/187198

i got the isuue and the solution is you have to degrade the version of vs code as its a bug in vs code


python3Packages.autobahn: 20.7.1 -> 20.12.3

Motivation for this change
Update to latest upstream release 20.12.3.
This fixes CVE-2020-35678 -> Upstream change.
Change log: https://autobahn.readthedocs.io/en/latest/changelog.html
Things done

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

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

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

Result of nixpkgs-review pr 108390 1

1 package failed to build:

grab-site

28 packages built:

buildbot (python38Packages.buildbot)
buildbot-full (python38Packages.buildbot-full)
buildbot-ui (python38Packages.buildbot-ui)
gnome-keysign
magic-wormhole (python38Packages.magic-wormhole)
python37Packages.autobahn
python37Packages.buildbot
python37Packages.buildbot-full
python37Packages.buildbot-ui
python37Packages.channels
python37Packages.channels-redis
python37Packages.daphne
python37Packages.magic-wormhole
python37Packages.magic-wormhole-mailbox-server
python37Packages.python-binance
python38Packages.autobahn
python38Packages.channels
python38Packages.channels-redis
python38Packages.daphne
python38Packages.magic-wormhole-mailbox-server
python38Packages.python-binance
python39Packages.autobahn
python39Packages.channels
python39Packages.channels-redis
python39Packages.daphne
python39Packages.magic-wormhole
python39Packages.magic-wormhole-mailbox-server
python39Packages.python-binance

Result of nixpkgs-review pr 108390 1

1 package failed to build:

grab-site

28 packages built:

buildbot (python38Packages.buildbot)
buildbot-full (python38Packages.buildbot-full)
buildbot-ui (python38Packages.buildbot-ui)
gnome-keysign
magic-wormhole (python38Packages.magic-wormhole)
python37Packages.autobahn
python37Packages.buildbot
python37Packages.buildbot-full
python37Packages.buildbot-ui
python37Packages.channels
python37Packages.channels-redis
python37Packages.daphne
python37Packages.magic-wormhole
python37Packages.magic-wormhole-mailbox-server
python37Packages.python-binance
python38Packages.autobahn
python38Packages.channels
python38Packages.channels-redis
python38Packages.daphne
python38Packages.magic-wormhole-mailbox-server
python38Packages.python-binance
python39Packages.autobahn
python39Packages.channels
python39Packages.channels-redis
python39Packages.daphne
python39Packages.magic-wormhole
python39Packages.magic-wormhole-mailbox-server
python39Packages.python-binance

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108390 run on x86_64-linux 1

1 package failed to build and are new build failure:

grab-site: log was empty

28 packages built:

buildbot (python38Packages.buildbot)
buildbot-full (python38Packages.buildbot-full)
buildbot-ui (python38Packages.buildbot-ui)
gnome-keysign
magic-wormhole (python38Packages.magic-wormhole)
python37Packages.autobahn
python37Packages.buildbot
python37Packages.buildbot-full
python37Packages.buildbot-ui
python37Packages.channels
python37Packages.channels-redis
python37Packages.daphne
python37Packages.magic-wormhole
python37Packages.magic-wormhole-mailbox-server
python37Packages.python-binance
python38Packages.autobahn
python38Packages.channels
python38Packages.channels-redis
python38Packages.daphne
python38Packages.magic-wormhole-mailbox-server
python38Packages.python-binance
python39Packages.autobahn
python39Packages.channels
python39Packages.channels-redis
python39Packages.daphne
python39Packages.magic-wormhole
python39Packages.magic-wormhole-mailbox-server
python39Packages.python-binance

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108390 run on x86_64-linux 1

1 package failed to build and are new build failure:

grab-site: log was empty

28 packages built:

buildbot (python38Packages.buildbot)
buildbot-full (python38Packages.buildbot-full)
buildbot-ui (python38Packages.buildbot-ui)
gnome-keysign
magic-wormhole (python38Packages.magic-wormhole)
python37Packages.autobahn
python37Packages.buildbot
python37Packages.buildbot-full
python37Packages.buildbot-ui
python37Packages.channels
python37Packages.channels-redis
python37Packages.daphne
python37Packages.magic-wormhole
python37Packages.magic-wormhole-mailbox-server
python37Packages.python-binance
python38Packages.autobahn
python38Packages.channels
python38Packages.channels-redis
python38Packages.daphne
python38Packages.magic-wormhole-mailbox-server
python38Packages.python-binance
python39Packages.autobahn
python39Packages.channels
python39Packages.channels-redis
python39Packages.daphne
python39Packages.magic-wormhole
python39Packages.magic-wormhole-mailbox-server
python39Packages.python-binance

The blocker seems to be lmdb:
Executing setuptoolsBuildPhase
py-lmdb: Using bundled liblmdb with py-lmdb patches; override with LMDB_FORCE_SYSTEM=1 or LMDB_PURE=1.
sh: /usr/bin/patch: not found
Traceback (most recent call last):
  File "nix_run_setup", line 8, in <module>
    exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\\r\\n', '\\n'), __file__, 'exec'))
  File "setup.py", line 119, in <module>
    raise Exception('Applying patch failed')
Exception: Applying patch failed

The blocker seems to be lmdb:
Executing setuptoolsBuildPhase
py-lmdb: Using bundled liblmdb with py-lmdb patches; override with LMDB_FORCE_SYSTEM=1 or LMDB_PURE=1.
sh: /usr/bin/patch: not found
Traceback (most recent call last):
  File "nix_run_setup", line 8, in <module>
    exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\\r\\n', '\\n'), __file__, 'exec'))
  File "setup.py", line 119, in <module>
    raise Exception('Applying patch failed')
Exception: Applying patch failed

The blocker seems to be lmdb:
Executing setuptoolsBuildPhase
py-lmdb: Using bundled liblmdb with py-lmdb patches; override with LMDB_FORCE_SYSTEM=1 or LMDB_PURE=1.
sh: /usr/bin/patch: not found
Traceback (most recent call last):
  File "nix_run_setup", line 8, in <module>
    exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\\r\\n', '\\n'), __file__, 'exec'))
  File "setup.py", line 119, in <module>
    raise Exception('Applying patch failed')
Exception: Applying patch failed

I already have a fix for that locally.

The blocker seems to be lmdb:
Executing setuptoolsBuildPhase
py-lmdb: Using bundled liblmdb with py-lmdb patches; override with LMDB_FORCE_SYSTEM=1 or LMDB_PURE=1.
sh: /usr/bin/patch: not found
Traceback (most recent call last):
  File "nix_run_setup", line 8, in <module>
    exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\\r\\n', '\\n'), __file__, 'exec'))
  File "setup.py", line 119, in <module>
    raise Exception('Applying patch failed')
Exception: Applying patch failed

I already have a fix for that locally.

I already have a fix for that locally.

Thanks

I already have a fix for that locally.

Thanks


Registration logging issues

Move logging of RHN & SubscriptionManager workflow specifics to LinuxAdmin where the logging can easily be changed based on the tool used.
Depends on: https://github.com/ManageIQ/linux_admin/pull/114
https://bugzilla.redhat.com/show_bug.cgi?id=1069247

@Fryguy @gmcculloug Please review

@miq-bot add_label bug

@gmcculloug Spec updated based on your comment


grpc failing on npm / yarn install

Help us reproduce - tell us about your environment

Using Mac 10.14.4, node v12.1.0, latest master version

Steps to reproduce

yarn / npm install

Expected result

Install all modules successfully
Actual result

CXX(target) Release/obj.target/grpc_node/ext/byte_buffer.o
In file included from ../ext/byte_buffer.cc:21:
In file included from ../../nan/nan.h:223:
In file included from ../../nan/nan_new.h:189:
../../nan/nan_implementation_12_inl.h:356:37: error: too few arguments to function call, expected 2, have 1
return v8::StringObject::New(value).Asv8::StringObject();
~~~~~~~~~~~~~~~~~~~~~      ^
/Users/valentinfilip/.node-gyp/12.1.0/include/node/v8.h:5380:3: note: 'New' declared here
static Local New(Isolate* isolate, Local value);
^
In file included from ../ext/byte_buffer.cc:21:
In file included from ../../nan/nan.h:223:
In file included from ../../nan/nan_new.h:189:
../../nan/nan_implementation_12_inl.h:356:58: error: expected '(' for function-style cast or type construction
return v8::StringObject::New(value).Asv8::StringObject();
~~~~~~~~~~~~~~~~^
../../nan/nan_implementation_12_inl.h:356:60: error: expected expression
return v8::StringObject::New(value).Asv8::StringObject();
^
3 errors generated.
make: *** [Release/obj.target/grpc_node/ext/byte_buffer.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit ([...]/react-native-starter-kit/node_modules/node-gyp/lib/build.js:262:23)
gyp ERR! stack     at ChildProcess.emit (events.js:196:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:256:12)
gyp ERR! System Darwin 18.5.0
gyp ERR! command "/usr/local/Cellar/node/12.1.0/bin/node" "[...]/react-native-starter-kit/node_modules/node-gyp/bin/node-gyp.js" "build" "--fallback-to-build" "--library=static_library" "--module=[...]/react-native-starter-kit/node_modules/grpc/src/node/extension_binary/node-v72-darwin-x64-unknown/grpc_node.node" "--module_name=grpc_node" "--module_path=[...]/react-native-starter-kit/node_modules/grpc/src/node/extension_binary/node-v72-darwin-x64-unknown" "--napi_version=4" "--node_abi_napi=napi" "--napi_build_version=0" "--node_napi_label=node-v72"
gyp ERR! cwd [...]/react-native-starter-kit/node_modules/grpc
gyp ERR! node -v v12.1.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute '/usr/local/Cellar/node/12.1.0/bin/node [...]/react-native-starter-kit/node_modules/node-gyp/bin/node-gyp.js build --fallback-to-build --library=static_library --module=[...]/react-native-starter-kit/node_modules/grpc/src/node/extension_binary/node-v72-darwin-x64-unknown/grpc_node.node --module_name=grpc_node --module_path=[...]/react-native-starter-kit/node_modules/grpc/src/node/extension_binary/node-v72-darwin-x64-unknown --napi_version=4 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v72' (1)
node-pre-gyp ERR! stack     at ChildProcess. ([...]/react-native-starter-kit/node_modules/grpc/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:196:13)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:1000:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:267:5)
node-pre-gyp ERR! System Darwin 18.5.0
node-pre-gyp ERR! command "/usr/local/Cellar/node/12.1.0/bin/node" "[...]/react-native-starter-kit/node_modules/grpc/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build" "--library=static_library"
node-pre-gyp ERR! cwd [...]/react-native-starter-kit/node_modules/grpc
node-pre-gyp ERR! node -v v12.1.0
node-pre-gyp ERR! node-pre-gyp -v v0.12.0
node-pre-gyp ERR! not ok

Hi @vasiabikeru
Looks like an environment issue with node. Can you please try:

https://github.com/nodejs/node-gyp/issues/809#issuecomment-155019383
And/or install a fresh version of React Native - does that work?

Thank you for your reply @mcnamee. I tried both steps and sadly nothing worked.
The error I get seems to be related to fsevents.
`react-native-starter-kit valentinfilip$ npm install
npm WARN deprecated<EMAIL_ADDRESS>Way too old
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Please, upgrade to core-js@3 or at least to actual version of core-js@2.
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Please, upgrade to core-js@3 or at least to actual version of core-js@2.
npm WARN deprecated<EMAIL_ADDRESS>CircularJSON is in maintenance only, flatted is its successor.
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to kleur@3 or migrate to 'ansi-colors' if you prefer the old syntax. Visit https://github.com/lukeed/kleur/releases/tag/v3.0.0\ for migration path(s).
npm WARN deprecated<EMAIL_ADDRESS>I wrote this module a very long time ago; you should use something else.
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated<EMAIL_ADDRESS>use String.prototype.padStart()

<EMAIL_ADDRESS>install /Users/valentinfilip/Dropbox/Work/workspace/WeRecycle/react-native-starter-kit/node_modules/chokidar/node_modules/fsevents
node install

node-pre-gyp WARN Using request for node-pre-gyp https download
[fsevents] Success: "/Users/valentinfilip/Dropbox/Work/workspace/WeRecycle/react-native-starter-kit/node_modules/chokidar/node_modules/fsevents/lib/binding/Release/node-v72-darwin-x64/fse.node" is installed via remote

<EMAIL_ADDRESS>install /Users/valentinfilip/Dropbox/Work/workspace/WeRecycle/react-native-starter-kit/node_modules/react-scripts/node_modules/fsevents
node install

node-pre-gyp WARN Tried to download(404): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.2.4/fse-v1.2.4-node-v72-darwin-x64.tar.gz
node-pre-gyp WARN Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v72 ABI, unknown) (falling back to source compile with node-gyp)
SOLINK_MODULE(target) Release/.node
CXX(target) Release/obj.target/fse/fsevents.o
../fsevents.cc:50:32: error: no template named 'Handle' in namespace 'v8'
static void Initialize(v8::Handlev8::Object exports);`
I installed fsevents individually and the current version is :
`react-native-starter-kit valentinfilip$ npm add fsevents
npm notice created a lockfile as package-lock.json. You should commit this file.

<EMAIL_ADDRESS>added 1 package from 4 contributors and audited 1 package in 0.703s`

Gotcha. Given you've tried installing a fresh version of React Native and React Native isn't work, I'd recommend opening an issue on React Native or fsevents.

@vasiabikeru do you have a solution? I have the same problem


Working Example with Zepto?

Is there a working example using Zepto instead of jQuery? I've been unsuccessful in getting this to work.

Could you give a bit more infos about what exactly isn't working?

Same here - when I use zepto I get this error when I click to open the modal:
Uncaught TypeError: b.when is not a function
    at new x (lity.min.js:5)
    at HTMLAnchorElement.y (lity.min.js:5)
    at HTMLDocument.l (zepto.min.js:2)
    at HTMLDocument.a.proxy (zepto.min.js:2)

Here's my setup that doesn't work:
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/lity/2.3.1/lity.min.css" />

But this works:
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/lity/2.3.1/lity.min.css" />

My markup is...

I'm fine using jQuery, just posting in case it helps troubleshoot.

Zepto needs additional modules, check the README.


Refactor command line flag parsing to include required env vars

This expands the scope of the command line flag related functions to
also handle environment variables which are required, but unrelated to
command line flags.
Signed-off-by: Scott Garman<EMAIL_ADDRESS>

Codecov Report

Merging #2 (4db0fa4) into tink-worker-rewrite (1720666) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@                   Coverage Diff                   @@
##           tink-worker-rewrite       #2      +/-   ##
=======================================================
+ Coverage                39.75%   39.88%   +0.12%
=======================================================
  Files                       42       42
  Lines                     2998     3004       +6
=======================================================
+ Hits                      1192     1198       +6
  Misses                    1708     1708
  Partials                    98       98

Impacted Files
Coverage Δ

cmd/tink-worker/cmd/cmd.go
100.00% <100.00%> (ø)

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 1720666...4db0fa4. Read the comment docs.

Closing this in favor of PR4


ABI default implementations

It would be useful to create a default implementation in the ABI and then override it in the impl if needed.

Not doing by design.


Request: Relax Python version requirement, if possible

Many distros still package versions below 3.10 as the main Python package.
Can the required minimum Python version for this project be lowered?

I'm planning to do so anyway - I'll look into this. I've no intention of supporting version of python that are EOL, but this should only need a few tweaks to get it working in 3.7+


Unable to build from 4.00.00 releases

Environment
4.00.00alpha/dev
Download both releases from 4.00.00
Linux 913374a2f5ee 4.13.16-300.fc27.x86_64 #1 SMP Mon Nov 27 18:19:43 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux - It's a CentOS 7.4 Docker container
Current Behavior:
Running tesseract -oem 0 indicates warnings about leaks:
[root@913374a2f5ee tesseract-4.00.00alpha]# tesseract -oem 0
Tesseract Open Source OCR Engine v4.00.00alpha with Leptonica
Error in fopenReadStream: file not found
Error in findFileFormat: image file not found
Error during processing.
ObjectCache(0x7f9538b735a0)::~ObjectCache(): WARNING! LEAK! object 0x2fb87b0 still has count 1 (id /usr/local/share/tessdata/eng.traineddatapunc-dawg)
ObjectCache(0x7f9538b735a0)::~ObjectCache(): WARNING! LEAK! object 0x2fb85c0 still has count 1 (id /usr/local/share/tessdata/eng.traineddataword-dawg)
ObjectCache(0x7f9538b735a0)::~ObjectCache(): WARNING! LEAK! object 0x2fb8670 still has count 1 (id /usr/local/share/tessdata/eng.traineddatanumber-dawg)
ObjectCache(0x7f9538b735a0)::~ObjectCache(): WARNING! LEAK! object 0x2fb8720 still has count 1 (id /usr/local/share/tessdata/eng.traineddatabigram-dawg)
ObjectCache(0x7f9538b735a0)::~ObjectCache(): WARNING! LEAK! object 0x2fbb780 still has count 1 (id /usr/local/share/tessdata/eng.traineddatafreq-dawg)

Expected Behavior:
After building from source from the master branch I get the expected behaviour:
[root@913374a2f5ee tesseract]# tesseract -oem 0
Tesseract Open Source OCR Engine v4.00.00alpha with Leptonica
Error in fopenReadStream: file not found
Error in findFileFormat: image file not found
Error during processing.

Unable to build from 4.00.00 releases #1224

In both cases it is clear that the you were able to build the software successfully.
https://en.wikipedia.org/wiki/Software_release_life_cycle#Alpha

https://help.github.com/articles/creating-releases/

If the release is unstable, select This is a pre-release to notify users that it's not ready for production.

@zdenop,
Is it possible to add this label?
Also, why those xx.xx.xxdev are in the releases?

@amitdo: I do not see them in releases:
https://github.com/tesseract-ocr/tesseract/releases
@L30Bola: your report is wrong: As already pointed by amitdo: you claim you are not able to build tesseract and you also run tesseract with wrong way (without required parameters).

@zdenop, the releases 4.00.00alpha and 4.00.00dev are listed between 3.05.00 Release and 3.04.01 release.

Ahh... Than this has to be because of tags.

Yes.


Down- and upvoting of online suggestions

Could there be a way to report completely bogus or inapplicable translation suggestions to whatever online TM they are coming from?
I don't mean suggestions that are just "wrong" for a particular context, but suggestions that are e.g. in the wrong language, or contain typos. I recall seeing a suggestion given in German when everything else was in Spanish. Also the first suggestion in the screenshot below is not very useful for anyone despite its 99% match.

Poedit uses 3rd party service that has no such capability in its API — there's very little I can do right now.
Mid- to long-term, I plan to proxy Poedit's suggestions through a server under my control to apply some gettext specific fixups (I'm sure you noticed not everything is done perfectly) and it would certainly make sense to do as you suggest (thanks!).
In fact, Poedit should arguably "upvote" suggestions that are used automatically and then only have a button for reporting/downvoting the Really Bad ones as in here. Leaving the issue open for this, but I can't promise any timeframe just yet.

Starting with 2.0.7, the suggestions that I have control over (i.e. user-contributed ones from Poedit) are automatically upvoted when manually used.


[zh] sync blog: 2023-01-05-retroactive-default-storage-class

Add zh text to this blog:
content/zh-cn/blog/_posts/2023-01-05-retroactive-default-storage-class.md

Preview: https://deploy-preview-38916--kubernetes-io-main-staging.netlify.app/zh-cn/blog/2023/01/05/retroactive-default-storage-class/

/lgtm


Get dilution of precision information from the GPS chip

Hi,
Would it be possible to retrieve dilution of precision information from the GPS chip and exploit them as separate values?
I am not able to figure out what's under the hood of the 'accuracy' value.
Many of the GPS tracking application are displaying such informations + the number of visible satellites, the satellites contributing to the fix (i.e. the determination of the actual position) etc. These would be good to store somewhere. Specific applications may need them.
Thanks.

Short answer is no, we cannot get satellite information. But if you're just looking for dilution of precision, browsers provide that value in the accuracy field. I'll explain why.
For most platforms, we rely on the browser implementation for the geolocation APIs. If the browser implementation is lacking on a particular platform, then Apache Cordova will implement it following the W3C spec. Android for example doesn't actually have a geolocation implementation since the webview provides an implementation; the plugin just simply handles permissions. The browser/W3C spec doesn't define APIs for obtaining satellite information, therefore it's not something we can implement in this plugin. Third-party geolocation plugins that uses the native APIs may have this implemented, or at the very least has the flexibility of providing APIs to obtain satellite information.
The browser however does give you an accuracy field, which looks a lot like the dilution of precision that you're looking for. It's read is a 95% confidence that the point is within the accuracy radius of the actual location in meters. In other words, if the accuracy reading is 5, the actual point is within 5 meters distance of the given point. If the accuracy reading is 50, then the actual point is within 50 meters distance of the given point. The greater the accuracy value, the less accurate the GPS point is. And in my experience, DOP Meaning Chart matches the healthy check of accuracy values. Most mobile devices will only be as accurate as 3-5m at their best. For most apps, the accuracy field is enough to determine if a received GPS point is safe to use. Knowing the actual satellite positioning/fixing will only be useful for displaying that data to the user if they care about that information. It's also worth noting that most devices uses A-GPS and doesn't solely rely on satellites for obtaining GPS points.
In my apps, I use a mixture of accuracy values and some other information to determine accuracy of a received GPS point. For example, I know users of the GPS are in a vehicle driving on a road, so if the distance/time of the next point compared to the previous point doesn't make sense (like they suddenly went from 50km/h to 300km/h) obviously the incoming point is garbage and I ignore it. 9 times out of 10 though a garbage point is accompanied by a high accuracy reading.
I'll be closing this issue because the requested API is something I don't foresee Apache Cordova implementing since it's not defined in the W3C spec.


[Question] Any chance to have a place for discussion/chat ?

Any chance to have a place for discussion/chat ? For example, Slack, GitChat.
I think it can help more people to choose SimplCommerce if there is a place to ask questions.

Sorry I just found there is already a place for discussion: Gitter. I'm closing this issue.


Queue status field

Hi! Cold you add status field to the Queue object?
We want to see current queue status from jqm client api:
 List<Queue> getQueues();
Is queue paused or no.

Hi,
the Queue object is purely a descriptor, meant to be used with enqueue. I would prefer not to add any overhead to it, not to change its usage. Would a isQueuePaused(int/Queue) verb do instead?
MA

Yes,  isQueuePaused(int/Queue) is good for us, thanks

New API is QueueStatus getQueueStatus(Queue)


Is still actively maintained?

@bolandrm
This gem is the recommendation to replace sass-rails, but the latest released version was in 2016.
Thanks!

Yes, most of the work takes place here:  https://github.com/sass/sassc-ruby
That being said, I will make an effort to merge PRs in this repo soon.


KAFKA-14809 Fix logging conditional on WorkerSourceTask

This shouldn't be a controversial change: the if condition here is checking the wrong variable (this.committableOffsets.isEmpty() always evaluates to true because of L#216). I assume this bug went undetected because it's only used to log information around committable offsets.
Should I file a JIRA?
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

LGTM, thanks @hgeraldino!
It would be nice to have a Jira for this so that others who notice the logging issue on older versions can know 1) that it is a bug and 2) which versions it is fixed on. Would you mind filing a ticket and linking it to this PR in the title? We can merge after that.

Done

Thanks @hgeraldino, I've merged and backported as far back as applicable.


Class Map Generator removed

You mention the Class Map Generator (bin/classmap_generator.php) on the docs but removed it when zend-loader got separated from zf2. Did you miss the file ou should I remove from docs?

+1 I used the classmap_generator.php of zendframework/zendframework package (https://github.com/zendframework/zendframework/blob/release-2.5.3/bin/classmap_generator.php) but actually i'm including each package as separate package, so I do not have those files anymore.
How can I recover a classmap generator?

Meanwhile I published this: https://github.com/pensiero/zend-generators

A classmap generator shouldn't be needed at all if you just rely on composer's...

Actually in my Module.php the getAutoloaderConfig() method use it:
public function getAutoloaderConfig()
    {
        if (getenv('ENV') == 'development') {
            return [
                'Zend\Loader\StandardAutoloader' => [
                    'namespaces' => [
                        __NAMESPACE__ => __DIR__ . '/../src/' . __NAMESPACE__,
                    ],
                ],
            ];
        } else {
            return [
                'Zend\Loader\ClassMapAutoloader' => [
                    __DIR__ . '/../config/autogenerated/autoload_classmap.config.php',
                ],
            ];
        }
    }

should I replace it with something else?

PS: autoload_classmap.config.php is the file generated by the classmap_autoloader.php script

@pensiero just move all that stuff in your composer.json "autoload" section, sit back and relax :-)

Am I good this way?
  "autoload": {
    "psr-4": {
      "Uala\\": "module/Uala/src/",
    }
  },

Correct. No more need for pesky manual classmaps.

I wanted to provide a PR, updating the documentation. However, ClassMapAutoloader documentation suggests the usage of classmap_generator.php.
Is there a valid use case for ClassMapAutoloader, without the generator? How would a classmap be generated, if the generator is gone? Is it compatible with a classmap generated by Composer? But then why would one not use Composer?
It seems that ClassMapAutoloader can be removed as well. @Ocramius / @froschdesign can you confirm that?

This repository has been closed and moved to laminas/laminas-loader; a new issue has been opened at https://github.com/laminas/laminas-loader/issues/1.


connect EADDRINUSE

When run tests on<EMAIL_ADDRESS>part of them failed with error:
connect EADDRINUSE
Screenshot
What could be the problem? If you need any logs, ready to provide them.
Output of gemini --version:
4.11.3

@SevInf @sipayRT @SwinX @j0tunn Are there any suggestions?

@ghost, according to your screenshot you use port 4444 (<IP_ADDRESS>:4444)
selenium uses the same port (http://localhost:4444/wd/hub)
and in .gemini.yml you can find the line:
gridUrl by default it is http://localhost:4444/wd/hub


Add option to give feedback.

Describe the feature you'd like

Add option to give feedback to an event.
Would you like to work on the issue?

[x] Yes
[ ] No
Other:

Duplicate

Duplicate of? Link please


Inconsistent default syntax in changing Terminal Default Shell

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.56.2
OS Version: Microsoft Windows [Version 10.0.19043.985]

Steps to Reproduce:
I want to use cmd in the terminal instead of PowerShell. So, I proceeded as guided in the official docs by using Select Default Profile. There are 3 issues that occur.

When I restart the VSC, it doesn't capture CMD as the default shell but PowerShell instead.
Given that and an environment interpreter is used. When I click the new terminal button, a new terminal is opened but with a PS command to activating the virtual environment.
When I run a line of code from editor by using 'ctrl + enter'. It also doesn't get the correct syntax for cmd.

I believe there is a setting to resolve this but I can't find that on the official docs page. Even so, the setting should be changed accordingly with the default profile selected.

One more thing, there is a new option to display terminals by show tabs. It looks nice. However, after I switch to that, there is no way to switch back from the UI. I need to navigate to the settings.json file to change the value to 'false'.

One more thing, there is a new option to display terminals by show tabs. It looks nice. However, after I switch to that, there is >no way to switch back from the UI. I need to navigate to the settings.json file to change the value to 'false'.

Right clicking in the empty space of the tabs panel offers a hide tabs option
Please provide a gif of you running Select Default Profile and your settings

#111396 #10759 #123368 #124911 #124910

Sorry for my late response. Following is the gif of the default shell and show tabs.

This looks like an issue with whatever extension the play button comes from, probably code runner or python extensions

@Tyriar Can you elaborate? I am not clear how to fix this? The python extension is from Microsolf.

@datvuong please file an issue in the repo of that extension.

I just installed the insider version. Look like it is fixed. So, I assume it is not the extension issue.


GCP refactoring: logging, storage cleanup, worker credentials

This PR addresses couple of outstanding things, which popped up during the initial GCP PR and later during review of Azure support for Cloud API.
Main changes:

Moved all logging out of the GCP internal library to the callers (worker and the CLI upload tool), similar as it is done for Azure.
Made all GCP methods a single purpose and not to do any extra tasks (e.g. cleanup). Rather than cleaning up under the hood, an error is returned and the caller can use provided cleanup methods to delete any unwanted resources. This is true for importing image. If the image upload to Storage fails, there is no need to clean up the object since it has been not created.
Modified the worker and CLI upload tool code to do all necessary clean up in case of errors
Modified worker GCP-related code to try to share the imported image only if the list of accounts is not empty.
Minor simplification of Google Compute Node API calls in the GCP library based on project's code examples.
Added building of osbuild-upload-gcp to Makefile's build target. This was previously missed.
Refactored handling of GCP credentials file on the worker. Instead of passing the path to the file to job, the file is read on the start up and its content is passed to the job. This mimics what is done for Azure and helps to detect issues with the credentials file early. Note that there is no easy way to actually test the validity of the content of the file and credentials other than making some actual API calls. I decided to not to do any "random" calls for now.

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
no new tests have been added
[ ] adequate documentation informing people about the change
changes should be transparent to the end user, thus no documentation has been added

CI "almost" passed - only "F33 aarch64 Base" test was aborted. This change set didn't touch any Fedora-related. I'm happy to reschedule to make it completely green...

Awesome work!
As a caller of the StorageObjectUpload method, I would expect that it either uploads a file correctly or it fails and doesn't leave any traces in the storage. This can be implemented in a future PR though.

Thanks for your opinion Ondrej. I reworked the code a bit more yesterday and it should be already behaving this way. I just didn't update the PR text. Now the MD5 checksum is provided when uploading the file and the server side rejects the object on Close() if there is a mismatch, rather than checking it on our side after the file has been uploaded.

// Uploaded data is rejected if its MD5 hash does not match the set value.
Ah, I missed this comment, thanks for the clarification!

The only failed test in CI is "F32 New OSTree" which passed for the same code (commit) in previous run number 4.
I would call this as overall pass and merge. Any objections @ondrejbudai @teg ?

Merged, thanks!


Fix build failures after master merge

Changes

Fix checkstyle errors from #243
Fix failure after bad merge in #244

@navina
Please take a look.

Thanks @shanthoosh for fixing the checkstyle nits in my previous PR!


Fix: Running Test with a comment fails test on Understand The immediately invoked function expression iife

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #39346

[x] Regex now checks if the function is an IIFE function
[x] Passes all 4 solutions

Thanks @nhcarrigan...
I just realized that I made the changes in the master branch. I forgot to branch, but I'll make sure to do it next time so I don't lose any progress.


[hotfix][core] fix run in standalone mode classNotFound.

Purpose of this pull request

Which issue you fix
fix run in standalone mode classNotFound.

Checklist:

[ ] I have executed the 'mvn spotless:apply' command to format my code.
[ ] I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have checked my code and corrected any misspellings.
[ ] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

DNM.


✨ RecDotGov Tours + Tickets + Timed Entry

Description
0.9.0 Release Branch
Has This Been Tested?
Checklist:

[x] I've read the contributing guidelines of this project
[x] I've installed and used .pre_commit on all my code
[x] I have documented my code, particularly in hard-to-understand areas
[x] I have made any necessary corresponding changes to the documentation

:tada: This PR is included in version 0.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Request: Mark message as read once copied to clipboard

I wanted to request a feature if possible!
It would be awesome if once the code is copied to the clipboard it could mark the message as read, so it clears from Notification Center and such.
Perhaps the logic is, once copied, wait 1min, then mark as read? Perhaps it is a setting people could enable/disable?

Yeah looked into this and we were a bit afraid it would cause the iMessage database to corrupt. I'm gonna look into how Bluebubbles and Airmessage handle it to see if I can find a clean way.

Just adding a comment because I would also love if it market these messages as read.

Kindly requesting this feature... Mark as read would be awesome


Footer: update support copy

@picturepan2 can you please update this copy for Chinese too?

Can we push this through?


Refactor code

Connection with issue(s)

Close #110
Solution description
Splits up the code into better testable and more maintainable code.
Screenshots or Videos

To Do

[ ] Read contributing guide
[ ] Check the original issue to confirm it is fully satisfied
[ ] Add solution description to help guide reviewers
[ ] Add unit test to verify new or fixed behaviour
[ ] If apply, add documentation to code properties and package readme

@deandreamatias I've finished all the validators. Next up are the tests. Feel free to review the API surface as it is now so i can adjust if needed.

@deandreamatias I've finished all the tests. Some are still failing and I'll look into those.
Can you review and let me know if anything needs to change, or where to add more tests?

@deandreamatias this is now ready to go!

@deandreamatias can you make the release?

@martijn00 need more documentation about this changes.

Add breaking changes and how migrate on README
Add link to these section on changelog
Update the table of contents on readme
Update documentation on readme about how add new validator

If you can do this, I will appreciate


[bug] Pull accessed denied for pyro docker images

Issue Description
Can't pull docker images of pyro releases.
Environment
bilow@koro-sensei:~/pyro/docker$ docker -v
Docker version 18.03.1-ce, build 9ee9f40
bilow@koro-sensei:~/pyro/docker$ uname -a
Linux koro-sensei 4.13.0-43-generic #48~16.04.1-Ubuntu SMP Thu May 17 12:56:46 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Code Snippet
bilow@koro-sensei:~/pyro/docker$ make notebook-gpu img=pyro-gpu-dev-3.6
## Create shared volume on the host for sharing files with the container.
##
mkdir -p /home/bilow/pyro_docker
## Start a juptyer notebook on the Pyro GPU docker container.
## Args:
##   img: use image name given by `img`.
##
docker run --runtime=nvidia --init -it -p 8888:8888 --user pyromancer \
-v /home/bilow/pyro_docker:/home/pyromancer/workspace/shared \
pyro-gpu-dev-3.6
Unable to find image 'pyro-gpu-dev-3.6:latest' locally
docker: Error response from daemon: pull access denied for pyro-gpu-dev-3.6, repository does not exist or may require 'docker login'.
See 'docker run --help'.
Makefile:121: recipe for target 'notebook-gpu' failed
make: *** [notebook-gpu] Error 125

Also tested the CPU branches from a separate Mac. Didn't work either.

I am closing this issue. Please feel free to ask on the forum if you continue to face issues with installation.


Is there a convention for comments?

I am wondering if this project favors sentence case or all lower case for comments?
i.e.
// This is my comment
or
// this is my comment
Or if it even matters.

I'm not entirely consistent in my own code, though I think proper capitalization is better. There's no rule for this in ESLint, but there's a proposal to add it. When that gets added, I'll post an issue to consider it.

ESLint now has a capitalized-comments rule.
I propose enable it, as follows:
"capitalized-comments": [
        "error",
        "always",
        {
            "ignoreInlineComments": true,
            "ignoreConsecutiveComments": true
        }
    ]

Woo!
I hate having to think about this :+1:

@feross not disagreeing but why ignoreInlineComments?

@timoxley Maybe you're right. I'll test the ecosystem impact of ignoreInlineComments enabled vs. disabled and report back here before merging this.

You typically see this for those who use the type systems analysis tools which pick up on inline comments for type declaration in standard JS code.  IIRC

@dcousens If we want to, we can explicitly allow certain uncapitalized words to start a comment without triggering this error. The rule supports exceptions. But a blanket exemption on inline comments is certainly simpler and won't need constant updating.

The inline eslint pragmas need to all be lowercase:
/* Eslint-env mocha */   // Does not work!
/* eslint-env mocha */   // Works!

I wonder how many other transpilers/helpers would break because of forced uppercase.
Flow? JSDoc?
I tending to thumbs down on this just because the value may not be worth the additional breakage (or the additional maintenance of a whitelist).

I don't think we should - I don't feel this particular rule would be
useful, and might, in fact, become frustrating. With stuff like code
comments, function names and what not there's simply too many edge cases
On Wed, Jan 11, 2017 at 5:55 PM Dan Flettre<EMAIL_ADDRESS>wrote:

The inline eslint pragmas need to all be lowercase:
/* Eslint-env mocha /   // Does not work!/ eslint-env mocha */   // Works!
I wonder how many other transpilers/helpers would break because of forced
uppercase.
Flow? JSDoc?
I tending to thumbs down on this just because the value may not be worth
the additional breakage (or the additional maintenance of a whitelist).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/feross/standard/issues/682#issuecomment-271924716,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlerpT-XhPG3RABtA1JNit7X5DsGOXks5rRQlxgaJpZM4KvDVD
.

@Flet All eslint pragmas are special-cased by the rule already.
Note that the following words are always ignored by this rule: ["jscs", "jshint", "eslint", "istanbul", "global", "globals", "exported"]

JSDoc is not affected because those declarations all start with a symbol (@) and that's allowed.
@yoshuawuyts Personally, I think this is a pretty well-thought-out rule: http://eslint.org/docs/rules/capitalized-comments I'd like to try shipping it in the standard v9 beta if there isn't strong objection so we can see how it works in practice.

Note that the following words are always ignored by this rule: ["jscs", "jshint", "eslint", "istanbul", "global", "globals", "exported"]

Yeah I'm a little concerned about accidentally breaking non-eslint/less popular/custom pragmas. Would be interested to see ecosystem impact.

Wouldn't commented out code be affected as well? If so, this seems pretty overreaching.

@jprichardson That's a good point that I hadn't considered. This rule basically prevents commented code from being committed.
For example, this would error:
function foo (x) {
  // if (x === 1) return 10
  return x

In one way, it would be nice to prevent commented code from ending up in a codebase. But sometimes this is not avoidable, so I think I agree with @jprichardson now: this is a bit overreaching.

Yeah, I agree in most cases, leaving commented code in place is a bad idea, especially since tools like Git are so pervasive. However, there are legitimate reasons to have commented code, like config files. Here is a real example from my codebase:
export const EXODUS_SERVER = EXODUS_PRODUCTION_SERVER
// export const EXODUS_SERVER = EXODUS_STAGING_SERVER

EXODUS_PRODUCTION_SERVER and EXODUS_STAGING_SERVER are defined near the beginning of the file.
So now if I just implemented a new feature on the server, I can quickly uncomment the aforementioned block to this:
// export const EXODUS_SERVER = EXODUS_PRODUCTION_SERVER
export const EXODUS_SERVER = EXODUS_STAGING_SERVER

and then the whole app is using EXODUS_SERVER still, but its value is different. Then when I'm ready to go back to a production build, I can change. Now I realize that it may be better to configure the aforementioned based upon NODE_ENV, but that'd feel like unnecessary ceremony required by a linter.
I'm sure there's other cases that are equally as valid where this rule my impede.

@jprichardson sounds like a better use-case of process.env

@jprichardson sounds like a better use-case of process.env

You just have missed this part 😛

Now I realize that it may be better to configure the aforementioned based upon NODE_ENV, but that'd feel like unnecessary ceremony required by a linter.

@jprichardson I did,  so I edited :laughing:

This rule has a big ecosystem impact:
1..422
# tests 422
# pass  259
# fail  163

To reduce the impact, I investigated allowing commented out code by using the ignorePattern option. We can ignore comments that start with a JS reserved word (do, if, in, for, let, etc.). Comments that start with non-ascii characters are automatically allowed ([, (, etc.)
So, if we only catch comments with real words that need to be capitalized, we get:
1..422
# tests 422
# pass  268
# fail  154

This is automatically fixable, so it's a quick fix. But jury's out on whether we actually want to do this.

For example, here are some diffs after running standard --fix:

No please
On Thu, Feb 9, 2017, 02:36 Feross Aboukhadijeh<EMAIL_ADDRESS>wrote:

For example, here are some diffs after running standard --fix:
[image: screen shot 2017-02-08 at 5 35 16 pm]
https://cloud.githubusercontent.com/assets/121766/22765421/0ecc8e72-ee25-11e6-87d3-65b96fd1ab6d.png
[image: screen shot 2017-02-08 at 5 35 28 pm]
https://cloud.githubusercontent.com/assets/121766/22765422/0ecf4612-ee25-11e6-8100-de3b35a87740.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/feross/standard/issues/682#issuecomment-278520124,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlemDgdfp2g8B_ljW5oOn2jiiPmWBmks5ram2UgaJpZM4KvDVD
.

Leaving open for more feedback, but yeah, I tend to agree with you @yoshuawuyts

This has good intentions but my feeling is that the benefits are minimal and likelihood of it being a nuisance is high. Also don't like the risk of hard-to-detect logic corruption caused by uppercasing commented out source which is later uncommented.

@dcousens thoughts?

Along similar lines, maybe one could consider enforcing a particular formatting for multi-line jsdoc /** styled comments i.e. ensure jsdoc style comments conform to the/a particular jsdoc spec.

Agreed with @timoxley,  its junk to think about,  but,  its harmless by definition and IMHO,  not worth the community impact at all.
I could not morally justify this refactor to someone paying the bills, haha.

Alright, it's decided. 💀

This has been a great thread :)


Normalize cmake

@ghisvail @SylvainCorlay
Similar to xtensor

This needs a release for xtensor.

You could support both, in case backward compat matters

If we release semver-breaking version number this should be fine?


Source non installé

Steps to reproduce
Quand j'essaie d'installer l'une des source, le message d'erreur suivant capture ci-dessous se montre.
Expected behavior
Source bien installé
Actual behavior
Mihon a rencontré une erreur inattendu. Nous vous suggérons de nous partager les rapports de plantage dans notre salon d'assistance sur Discord
Crash logs

Mihon version
0.16.5
Android version
Android 11 - MUIU Global 12.5. stable
Device
Redmi note 11 4G
Other details

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Do not download the above, it is a scam

This seems like the usual Xiaomi extension installation issue, here's a couple things to try:

Go to your Phone Settings > Developer Options, turn off system optimization/MIUI Optimization

If you don't see this setting, scroll down your Developer Options, and tap multiple times on "Reset to default values" until it appears

Make sure your Mihon app has permission to "Install Unknown Apps"
Try downloading said extension now

If that doesn't work:

Go to Mihon Settings > Advanced, and change the Installer to something else (other than Shizuku)
Try downloading said extension now

If you have any other questions, please check with whoever maintains your extensions or ask about this crash in the Mihon Discord support channel.

Closing as chinaphone issue
Follow the instructions in my previous message, otherwise, join the Mihon Discord support channel for more troubleshooting if the above doesn't help.


feat support for clear parameters #6521

About: #6521
Support --clear command line to clear console and debugger.

[x] Docs
[x] Tests

I opened up my may machine and ran this script:
create a playgroud project.
add --clear command line to build/dev/watch, like  "cross-env NODE_ENV=development node ./node_modules/@remix-run/dev/dist/cli watch --clear",.
Then run npm run dev to compile remix-run project.After that, the console/debugger in the code will be cleared.
build and watch is the same as dev, threr are all support clear.

hi @machour ,I  think this feature should exist within the remix, because most of the time we can't ensure the code of other third-party node_modules packages. Do you have any suggestions?

@pcattori @machour I do need this feature. If you have time, maybe under the approval of this function, I think there should also be a change function in the community.

#6719 changed remix dev to not clear the screen. You can still call clear && remix dev in your scripts if you want the initial clear.


Suggestion: Merge ambient function and class declarations with the same name.

Summary
To better support subclassing built-ins, and still support built-ins and pre-ES6 "classes" that can also be called as functions, we should be able to merge the call signatures of ambient function declarations with the static side of a class declaration.
Current state
Currently, if you want to declare an ambient "class" that has both call and construct signatures, you need to use an interface-interface-var pattern, as follows:
// instance-side interface type
interface Array<T> { /*...*/ }

// static-side interface type
interface ArrayConstructor {
  <T>(length: number): T[];
  new <T>(length: number): T[];
  /* ... */
}

// value declaration for constructor
declare var Array: ArrayConstructor;

This is a problem, however as "classes" defined using this pattern cannot be used in the extends clause of a class declaration in TypeScript.
Proposal
Merge ambient function and class declarations with the same name. Ambient function declarations with the same name as an ambient class declaration in the same lexical scope would contribute their call signatures to the static side of the class declaration. For example:
// lib.d.ts
declare function Array<T>(length: number): T[];
declare class Array<T> {
  constructor(length: number);
}

// app.ts
var ar1 = Array(10); // T[]
var ar2 = new Array(10); // T[]

Out of scope

This proposal does not cover the case where classes in TypeScript today cannot be re-opened. This case is covered in #2957.
This proposal does not cover the case where authors may have already extended the interface of a built in. This case is covered in #2961.

Looks like the ES7/ES Later proposal for callable constructors would cover the part of definition it, we will only need to allow redefinition classes.
@rbuckton do you agree?

closing in favor of callable constructors.


Added support for JQ

This adds support for the JQ language and resolves #1894.
@iilei Could you please verify that this PR highlights JQ code correctly and consistently?
I'm not familiar with JQ so someone who is looking over it would be really appreciated.
Example

Thank you for contributing the PR so swiftly 👍
I can confirm it does a great job in highlighting also a more complex example of jq code.
Just one question or a wish, if possible – could the pipe | operator be treated similar as the vscode highlighter does? Maybe keep the operatorclass and add a pipeline-operator class too?

The | operator combines two filters by feeding the output(s) of the one on the left into the input of the one on the right. It’s pretty much the same as the Unix shell’s pipe, if you’re used to that.

jq manual
credits for the example code

@iilei | and |= will now also have the pipe alias which should allow more granular highlighting.
I also changed the style of . depending on the context. I.e. .foo and .[0] will still be punctuation but a single . will be of class dot.
I hope this works for you.

very cool


New methods

I want to use this library but I need some methods. I'll try to implement them this weekend if I have spare time. Naming can be improved.

fn len_of_union(&self, element: Element) -> usize: Length of the union that this element belongs to.
fn unions_of_size_one(&self) -> Vec<Element>: Elements that are not in a union with anything else. This is not crucial as they can be kept in a Set and removed when a union is made.

Huh, interesting. Thanks for opening the issue. From what I can tell, len_of_union can be implemented in terms of the the existing API:
fn len_of_union<Element: ElementType>(uf: &UnionFind<Element>, a: Element) -> usize {
    let mut result = 0;
    for b in 0 .. uf.len() {
        if uf.equiv(a, Element::from_usize(b).unwrap()) {
            result += 1;
        }
    }
   result
}

That should be O(n log* n). I don't think you can do better by putting it inside the abstraction boundary.
As for unions_of_size_one, you can do it in O(n log* n) as well, in terms of the current API:
fn unions_of_size_one<Element: ElementType>(uf: &UnionFind<Element>) -> Vec<Element> {
    let mut remaining: HashSet<usize> = (0 .. uf.len()).collect();

    for i in 0 .. uf.len() {
        let element = Element::from_usize(i).unwrap();
        let root = uf.find(element);
        if element != root {
            remaining.remove(&i);
        }
    }

    remaining.into_iter().flat_map(Element::from_usize).collect()
}

Given that these operations seem to be not too hard to express in terms of the existing API, and that moving them into the API would not offer any performance benefits, I'm not sure it's worth adding them to the library. That said, I'm open to further persuasion.

Thanks for taking the time for doing the actual implementations. I agree that these helpers aren't interesting enough to add to the library. A higher level abstraction with a builtin HashSet could be a better approach, without cluttering the well known Union-Find.


reset pong interval on every input

Description
update last pong on new input to maintain the connection and prevent unnecessarily restarting the connection
Changes
List of changes this PR includes
Related Issues

#923

Checklist

[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstring

Great 🔥

I explained to @Eslam-Nawara why this is wrong and need to be redone correctly


MAINT: CODEOWNERS syntax fix and changes

Reference issue
None
What does this implement/fix?

As per https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners#example-of-a-codeowners-file, our syntax isn't quite right. I noticed that sometimes reviews were not requested when nested files were modified, but top-level files remained the same. 99% sure this isn't deliberate but maybe I'm wrong.
Also made some changes for things that are outdated and made some additions.

Additional information
I imagine there are other changes that would make sense as well, maybe this is a good opportunity to reconsider the entries. Or we can just go with these changes.

I wonder whether it is worth pinging people to ask if anyone doesn't want the notifs anymore, but perhaps better to just keep as is.


Once you limit to a limited set of colors, try to build picture with bigger than 1x1 pieces

And also generate a parts list.
Added here for discussion :D I realize this could be a lot of work. But the payoff! Oh, the payoff!

I guess you'll want a packing algorithm to work out which pieces to use in the spaces.

Yah, thats a big part of it. I've already forked to see how hard that would be. Simplifying assumptions: square/rectangular lego pieces only. A limited palette (on the order of several dozen colors only).
The other slightly nontrivial aspect is determining the shapes of similar color that are to be filled with lego pieces of a single color (this is analogous to doing a "paint fill" in a paint app, I think... so should not be too hard).

Nice idea.
I would not use a packing algorithm. Most humans do not use one, as well ;-). Create a set of 4, 8, 6 ( more rare) pieces and use them a bit randomly.

Yah, I was thinking of using something more random. Since the actual space to be packed is not that large (30x30), I have a random algorithm in mind that should be fairly simple to implement... when i get a chance ... :)

That would definitely be a great idea and as @gwachob says (by the way any update on your algorithm ?) it should not be that hard with a random algorithm but it comes tricky when you have a look at the existing pieces, some like 2x1 are existing in almost any colour but some like this cross seem to be more limited in term of colours. Don't know if there is a solution to deal with existing colours/shapes for >1x1 bricks (but having a look at the price it would cost me to actually buy the bricks to build the picture I want, finding an easy way to find which 8x8 or 2x8 I should buy would be a relief, a 2x8 is defintely cheaper than 16 bricks of 1x1).

Could be good if for vertical sections it could try and default to offset patterns if possible.
[    ][    ][    ][    ][    ]
[ ][    ][    ][    ][    ][ ]
[    ][    ][    ][    ][    ]
[ ][    ][    ][    ][    ][ ]

Thought of a small algorithm.

go through shrunken image's pixels
for each pixel, check if surrounding pixels are the same color, favor squares over rectangles, take a random choice if you can have a horizontal or vertical rectangles but not square.
do stuff

@JuanPotato that seems a good start. Following my comment I have been searching for algorithms aiming at solving this issue and there seems to be a lot of options. What seems to be the most advanced software (regarding algorithms available) is PicToBrick and the source code is available. It contains a ElementSizeOptimisation.java file which seems to do this kind of thing. I have just had a quick look but it might be worth having a look (the guy doing the software seem to have spent a lot of time of theoretical aspect, I am pretty sure the solution they have used is one of the best in term of complexity/efficiency)

Hey Everyone, I created www.BrickIt.co using just jQuery and HTML5's Canvas element to do this and actually got a great algorithm for parting things out I came up with when I was 20 (3 years ago), it's wicked fast (like 1024 pegs x 1024 pegs image in less than 10 seconds) and as optimal as any complex optimization problem attempt. Everyone wayy over complicates it. I think the parting out results speaks for themselves.
It's simple. Given a set of brick dimensions (2x4, 2x3, 1x6, 2x2, 1x4, 1x3, 1x2, 1x1). Start out with the highest priority piece (2x4) in the top left corner and work your way from left to right, top to bottom, checking:
Does this piece fit here (doesn't exceed the boundary)
Has any piece already been placed here before (If I were to place this piece here, would it overlap
Are all colors the same for the pegs that that piece would take up if it were to be placed there
If so, place it, if not, move to the left 1 (or start a new row) etc.
Go through the entire image trying to place the top priority piece, then repeat starting in the top left corner for the second highest priority piece, etc.
Once you realize you can place a piece, draw it with the borders, otherwise it'd be really hard to figure out where are the borders are supposed to go once you are done.
Theoretically, you could save maybe 1% of pieces or costs if you started your optimizations in other places other than the top left corner, but you would also spend millions of times longer processing every possible starting location and sprawl pattern for a 1% improvement gain.
How I have this implemented is ridiculously easy as well, just have two matrices, one to indicate the color index of a particular peg, one to keep track if that peg has a piece placed on it or not, and then I guess you have to have the matrix representing your actual image as well.
I'll try and add my BrickIt project to github soon, none of the source code is obfuscated though so feel free to look through it (literally just vanilla javascript, jQuery and HTML5 canvas) - I was going to try and turn it into a business or sell it but it's been years now.

@shaun3141 that sounds great! Can't wait to see it. :smile:

@JuanPotato I was willing to give a try to @shaun3141 suggestion but trying to build 1x2.png and so on from your original 1x1.png file, I notice that just copying/pasting the file and trying to remove the central vertical line does not work as the background gray is not the same on the left and on the right of the image ( see the result and the tiny difference in the middle of the image). Any suggestion based on the process you used on the best way to produce the 1x2, 1x3, 1x6 and others ?
Thanks

@symac I took the photos of the art works that were made into lego like this one and took one block from it. So you may want to try to find good pictures of blocks and just edit them so that the color and size is correct.

Thanks @JuanPotato I rather have been trying to build something from the 256px version of the 1x1 you had provided in another issue. My fork of the project now contains a --optimize parameter that can be used with cli.py to limit the number of bricks used. That seems to work pretty well according to my tests but the low quality of the bricks I have created doesn't give a good result. That is a good start but I hope someone will be able to create better png for the "big" bricks.


Google Moving from UA to GA4

Note that google is moving from using Universal Analytics to Google Analytics 4 starting July 1st 2022, making this plugin unusable.
I'd love to see an update for GA4.
Thanks!

It's technically relevant until 1/7 2023 where Universal Analytics sunsets.
But yes, a GA4 option/version would be appreciated 😊


TST: allow assert_almost_equal to handle pandas instances

Closes #11584.
Because many existing tests expects assert_almost_equal to compare pandas instances, it looks better to call appropriate test functions rather than raise.
I've also fixed some failure cases caused by the change (comparing Series and list, etc).

just a small question


ci: OOM while validating PTV's GTFS schedule dataset

Bug report
Describe the bug
As of march 24th 2021 it has been noticed that PTV's dataset produces an OOM error. This happens after this particular dataset was updated on march 23rd 2021.

How we reproduce the bug
Check execution log of end_to_end_100 worflow: https://github.com/MobilityData/gtfs-validator/runs/2185660518?check_suite_focus=true
Expected behaviour
GTFS dataset should be validated with no error.
Observed behaviour
gtfs-validator crashed on CI with OOM error.
Screenshots:

Environment versions

validator version:
Java version:
OS versions: Github actions

Closing as fixed by https://github.com/MobilityData/gtfs-validator/pull/895.


fix(chat): fix collapsing nested folders on import (Issue #1958)

Description:

fix nested folders collapsing while importing nested chat or prompt

Issues:

Issue #1958

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


Extremely inconvenient formatting

Expected Behavior
After upgrade do not have prev formatting sample
Actual Behavior
Extremely inconvenient formatting of a response json for a dark schema
picture
And this is indeed an error - since it is impossible to read the answer
Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
Version: Insomnia 8.0.0
Build date: 9/27/2023
OS: Darwin arm64 21.6.0
Electron: 25.8.1
Node: 18.15.0
Node ABI: 116
V8: <IP_ADDRESS>-electron.0
Architecture: arm64
Insomnia Version
8.0.0
What operating system are you using?
macOS
Operating System Version
Monteray
Installation method
upgrade from prev
Last Known Working Insomnia version
No response

Select simple dark -- success

select other theme

There are many other themes you can choose from.

Fixed in 8.2 released a few hours ago.