Tests are failing on main

Seems conda-related: https://github.com/microsoft/vscode-python/runs/4900770674?check_suite_focus=true
Started failing with c63d38d148aec3a7b7a3890c7b3d9bfefbca36f5
1) PythonExecutableService
       Importing without a valid PYTHONPATH should fail:
     Error: No matching bindings found for serviceIdentifier: Symbol(IConfigurationService)
      at _validateActiveBindingCount (node_modules/inversify/lib/planning/planner.js:63:23)
      at _getActiveBindings (node_modules/inversify/lib/planning/planner.js:49:5)
      at _createSubRequests (node_modules/inversify/lib/planning/planner.js:86:26)
      at Object.plan (node_modules/inversify/lib/planning/planner.js:137:9)
      at /home/runner/work/vscode-python/vscode-python/path with spaces/node_modules/inversify/lib/container/container.js:319:37
      at Container._get (node_modules/inversify/lib/container/container.js:312:44)
      at Container.get (node_modules/inversify/lib/container/container.js:232:21)
      at ServiceContainer.get (out/client/ioc/container.js:24:89)
      at Object.getPythonSetting (out/client/pythonEnvironments/common/externalDependencies.js:112:47)
      at getCandidates (out/client/pythonEnvironments/common/environmentManagers/conda.js:111:60)
      at getCandidates.next (<anonymous>)
      at Function.locate (out/client/pythonEnvironments/common/environmentManagers/conda.js:177:26)
      at Function.getConda (out/client/pythonEnvironments/common/environmentManagers/conda.js:102:39)
      at CondaService.getCondaVersion (out/client/pythonEnvironments/common/environmentManagers/condaService.js:44:30)
      at PythonExecutionFactory.createCondaExecutionService (out/client/common/process/pythonExecutionFactory.js:96:31)
      at PythonExecutionFactory.create (out/client/common/process/pythonExecutionFactory.js:61:50)
      at Context.<anonymous> (out/test/common/process/pythonProc.simple.multiroot.test.js:50:35)

  2) PythonExecutableService
       Importing with a valid PYTHONPATH from .env file should succeed:
     Error: No matching bindings found for serviceIdentifier: Symbol(IConfigurationService)
      at _validateActiveBindingCount (node_modules/inversify/lib/planning/planner.js:63:23)
      at _getActiveBindings (node_modules/inversify/lib/planning/planner.js:49:5)
      at _createSubRequests (node_modules/inversify/lib/planning/planner.js:86:26)
      at Object.plan (node_modules/inversify/lib/planning/planner.js:137:9)
      at /home/runner/work/vscode-python/vscode-python/path with spaces/node_modules/inversify/lib/container/container.js:319:37
      at Container._get (node_modules/inversify/lib/container/container.js:312:44)
      at Container.get (node_modules/inversify/lib/container/container.js:232:21)
      at ServiceContainer.get (out/client/ioc/container.js:24:89)
      at Object.getPythonSetting (out/client/pythonEnvironments/common/externalDependencies.js:112:47)
      at getCandidates (out/client/pythonEnvironments/common/environmentManagers/conda.js:111:60)
      at getCandidates.next (<anonymous>)
      at Function.locate (out/client/pythonEnvironments/common/environmentManagers/conda.js:177:26)
      at Function.getConda (out/client/pythonEnvironments/common/environmentManagers/conda.js:102:39)
      at CondaService.getCondaVersion (out/client/pythonEnvironments/common/environmentManagers/condaService.js:44:30)
      at PythonExecutionFactory.createCondaExecutionService (out/client/common/process/pythonExecutionFactory.js:96:31)
      at PythonExecutionFactory.create (out/client/common/process/pythonExecutionFactory.js:61:50)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)
      at Context.<anonymous> (out/test/common/process/pythonProc.simple.multiroot.test.js:62:35)

  3) PythonExecutableService
       Known modules such as 'os' and 'sys' should be deemed 'installed':
     Error: No matching bindings found for serviceIdentifier: Symbol(IConfigurationService)
      at _validateActiveBindingCount (node_modules/inversify/lib/planning/planner.js:63:23)
      at _getActiveBindings (node_modules/inversify/lib/planning/planner.js:49:5)
      at _createSubRequests (node_modules/inversify/lib/planning/planner.js:86:26)
      at Object.plan (node_modules/inversify/lib/planning/planner.js:137:9)
      at /home/runner/work/vscode-python/vscode-python/path with spaces/node_modules/inversify/lib/container/container.js:319:37
      at Container._get (node_modules/inversify/lib/container/container.js:312:44)
      at Container.get (node_modules/inversify/lib/container/container.js:232:21)
      at ServiceContainer.get (out/client/ioc/container.js:24:89)
      at Object.getPythonSetting (out/client/pythonEnvironments/common/externalDependencies.js:112:47)
      at getCandidates (out/client/pythonEnvironments/common/environmentManagers/conda.js:111:60)
      at getCandidates.next (<anonymous>)
      at Function.locate (out/client/pythonEnvironments/common/environmentManagers/conda.js:177:26)
      at Function.getConda (out/client/pythonEnvironments/common/environmentManagers/conda.js:102:39)
      at CondaService.getCondaVersion (out/client/pythonEnvironments/common/environmentManagers/condaService.js:44:30)
      at PythonExecutionFactory.createCondaExecutionService (out/client/common/process/pythonExecutionFactory.js:96:31)
      at PythonExecutionFactory.create (out/client/common/process/pythonExecutionFactory.js:61:50)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)
      at Context.<anonymous> (out/test/common/process/pythonProc.simple.multiroot.test.js:70:35)

  4) PythonExecutableService
       Unknown modules such as 'xyzabc123' be deemed 'not installed':
     Error: No matching bindings found for serviceIdentifier: Symbol(IConfigurationService)
      at _validateActiveBindingCount (node_modules/inversify/lib/planning/planner.js:63:23)
      at _getActiveBindings (node_modules/inversify/lib/planning/planner.js:49:5)
      at _createSubRequests (node_modules/inversify/lib/planning/planner.js:86:26)
      at Object.plan (node_modules/inversify/lib/planning/planner.js:137:9)
      at /home/runner/work/vscode-python/vscode-python/path with spaces/node_modules/inversify/lib/container/container.js:319:37
      at Container._get (node_modules/inversify/lib/container/container.js:312:44)
      at Container.get (node_modules/inversify/lib/container/container.js:232:21)
      at ServiceContainer.get (out/client/ioc/container.js:24:89)
      at Object.getPythonSetting (out/client/pythonEnvironments/common/externalDependencies.js:112:47)
      at getCandidates (out/client/pythonEnvironments/common/environmentManagers/conda.js:111:60)
      at getCandidates.next (<anonymous>)
      at Function.locate (out/client/pythonEnvironments/common/environmentManagers/conda.js:177:26)
      at Function.getConda (out/client/pythonEnvironments/common/environmentManagers/conda.js:102:39)
      at CondaService.getCondaVersion (out/client/pythonEnvironments/common/environmentManagers/condaService.js:44:30)
      at PythonExecutionFactory.createCondaExecutionService (out/client/common/process/pythonExecutionFactory.js:96:31)
      at PythonExecutionFactory.create (out/client/common/process/pythonExecutionFactory.js:61:50)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)
      at Context.<anonymous> (out/test/common/process/pythonProc.simple.multiroot.test.js:77:35)

  5) PythonExecutableService
       Ensure correct path to executable is returned:
     Error: No matching bindings found for serviceIdentifier: Symbol(IConfigurationService)
      at _validateActiveBindingCount (node_modules/inversify/lib/planning/planner.js:63:23)
      at _getActiveBindings (node_modules/inversify/lib/planning/planner.js:49:5)
      at _createSubRequests (node_modules/inversify/lib/planning/planner.js:86:26)
      at Object.plan (node_modules/inversify/lib/planning/planner.js:137:9)
      at /home/runner/work/vscode-python/vscode-python/path with spaces/node_modules/inversify/lib/container/container.js:319:37
      at Container._get (node_modules/inversify/lib/container/container.js:312:44)
      at Container.get (node_modules/inversify/lib/container/container.js:232:21)
      at ServiceContainer.get (out/client/ioc/container.js:24:89)
      at Object.getPythonSetting (out/client/pythonEnvironments/common/externalDependencies.js:112:47)
      at getCandidates (out/client/pythonEnvironments/common/environmentManagers/conda.js:111:60)
      at getCandidates.next (<anonymous>)
      at Function.locate (out/client/pythonEnvironments/common/environmentManagers/conda.js:177:26)
      at Function.getConda (out/client/pythonEnvironments/common/environmentManagers/conda.js:102:39)
      at CondaService.getCondaVersion (out/client/pythonEnvironments/common/environmentManagers/condaService.js:44:30)
      at PythonExecutionFactory.createCondaExecutionService (out/client/common/process/pythonExecutionFactory.js:96:31)
      at PythonExecutionFactory.create (out/client/common/process/pythonExecutionFactory.js:61:50)
      at Context.<anonymous> (out/test/common/process/pythonProc.simple.multiroot.test.js:95:35)

#18338 addresses theses missing service identifiers, but then timeout issues are surfaced:
[11:15 a.m.] Kim-Adeline Miguel
    Same multiroot workspace tests failed sad Karthik Nadig Paula Camargo Velasquez https://github.com/microsoft/vscode-python/runs/4925250424?check_suite_focus=true

1) PythonExecutableService
 Importing without a valid PYTHONPATH should fail:
 Error: Timeout of 25000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\common\process\pythonProc.simple.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

2) PythonExecutableService
 Importing with a valid PYTHONPATH from.env file should succeed:
 Error: Timeout of 25000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\common\process\pythonProc.simple.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

3) PythonExecutableService
 Known modules such as'os'and'sys' should be deemed 'installed':
 Error: Timeout of 25000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\common\process\pythonProc.simple.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

4) PythonExecutableService
 Unknown modules such as'xyzabc123' be deemed 'not installed':
 Error: Timeout of 25000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\common\process\pythonProc.simple.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

5) PythonExecutableService
 Ensure correct path to executable is returned:
 Error: Timeout of 25000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\common\process\pythonProc.simple.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

6) Multiroot Linting
 Enabling Pylint in root and also in Workspace, should return errors:
 Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\linters\lint.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

7) Multiroot Linting
 Disabling Pylint in root and enabling in Workspace, should return errors:
 Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\linters\lint.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

8) Multiroot Linting
 Enabling Flake8 in root and also in Workspace, should return errors:
 Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\linters\lint.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

9) Multiroot Linting
 Disabling Flake8 in root and enabling in Workspace, should return errors:
 Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()"is called; if returning a Promise, ensure it resolves.(d:\a\vscode-python\vscode-python\path with spaces\out\test\linters\lint.multiroot.test.js)
 at listOnTimeout (internal/timers.js:554:17)
 at processTimers (internal/timers.js:497:7)

  1) PythonExecutableService
       Importing with a valid PYTHONPATH from .env file should succeed:

      AssertionError: expected { Object (stdout, stderr) } to have property 'stdout' of 'Hello\r\n', but got 'Hello\n'
      + expected - actual

      -Hello
      +Hello

      at d:\a\vscode-python\vscode-python\path with spaces\node_modules\chai-as-promised\lib\chai-as-promised.js:302:22
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)
      at Context.<anonymous> (d:\a\vscode-python\vscode-python\path with spaces\out\test\common\process\pythonProc.simple.multiroot.test.js:65:9)

Failing CI due to an extra \r character: https://github.com/microsoft/vscode-python/runs/4986976848?check_suite_focus=true


Fix bugs in UG (excl diagrams)

Ensure that all optional fields are specified in the UG as per the expected behaviour
Reformat UG
Update command summary to the latest

fixes #512

Codecov Report

Merging #517 (b95618f) into master (a5f1eee) will increase coverage by 1.13%.
The diff coverage is 52.98%.

@@             Coverage Diff              @@
##             master     #517      +/-   ##
============================================
+ Coverage     59.88%   61.01%   +1.13%
- Complexity      541      615      +74
============================================
  Files           100      104       +4
  Lines          2064     2219     +155
  Branches        222      252      +30
============================================
+ Hits           1236     1354     +118
- Misses          736      756      +20
- Partials         92      109      +17

Impacted Files
Coverage Δ

...n/java/seedu/programmer/commons/core/Messages.java
0.00% <ø> (ø)

...main/java/seedu/programmer/logic/LogicManager.java
39.47% <0.00%> (-7.41%)
:arrow_down:

...grammer/logic/commands/DashboardCommandResult.java
100.00% <ø> (+100.00%)
:arrow_up:

...ogrammer/logic/commands/DownloadCommandResult.java
100.00% <ø> (ø)

...u/programmer/logic/commands/ShowCommandResult.java
36.36% <ø> (+3.03%)
:arrow_up:

...programmer/logic/commands/UploadCommandResult.java
100.00% <ø> (+100.00%)
:arrow_up:

...edu/programmer/logic/parser/EditCommandParser.java
61.11% <0.00%> (-3.60%)
:arrow_down:

src/main/java/seedu/programmer/model/Model.java
100.00% <ø> (ø)

...ain/java/seedu/programmer/model/student/Email.java
80.00% <ø> (ø)

...ogrammer/model/student/QueryStudentDescriptor.java
85.71% <ø> (ø)

... and 57 more

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 c7e1323...b95618f. Read the comment docs.


[PE-D] Doesn't make sense to be able to add contacts with same telegram handle

Labels: severity.Medium type.FunctionalityBug
original: snss231/ped#8

Need to update DG to warn users about duplicate detection limitations.

@Harish-coding This note already exists in the UG under add command, "all other fields do not have to be unique".

If you think this suffices as an explanation for this PE-D bug, then I think this issue can be closed 👍

I have reiterated it here as well :smiley:


Doc fix #18277

Clarified the documentation that if another CSS selector than an id is used, the class=collapsed is not added to the trigger element.

Bootstrap 3 is no longer being officially developed or supported.
All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!
<3,
@mdo and team


just a question

Can you please point out where i can find the 'json' format reference

This is the url to the JSON data for a user. Replace USERNAME with the username you want.
https://www.instagram.com/USERNAME/?__a=1

I am a little unsure about what you mean about "'json' format reference". If you are talking about the documentation or something similar on using python with JSON data then you might want to check out this link: https://realpython.com/python-json/
If I did not answer your question, please do let me know and I will try give a better answer :)

Thanks for your reply, and sorry I was unclear. What I wanted was is there any docs for the method, not json docs.

Hmm...I dont think so there are docs for the method. The way I found that you could get the JSON data by adding ?__a=1 to the end of the url is by finding it online. I think I just stumbled upon it someday and then thought of making a script which actually uses that data.


Limit implement Node

Current coverage is 55.93% (diff: 0.00%)

Merging #21 into master will decrease coverage by 0.09%

@@             master        #21   diff @@
==========================================
  Files            25         25
  Lines          1153       1155     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            646        646
- Misses          448        450     +2
  Partials         59         59

Powered by Codecov. Last update 39f6a24...706f64e


Start process with environment

Summary of the new feature/enhancement
As a user I want to Start-Process with new environment variables.
Proposed technical implementation details
Currently we do
Start-Process PowerShell -ArgumentList '-Command',"`$env:FOO='bar'; main.exe"

But this is cumbersome. It would be great if we can specify some environment as argument to Start-Process, like
Start-Process main.exe -Environment @{ FOO = 'bar' }

This feature request is related to #4671. See also this question on Stack Overflow.

@FranklinYu Thanks for your contribution!
It is dup #3316. Feel free to discuss there.


Provide a comment with instructions at the top of every OWNERS.yaml file

When someone new to the codebase wants to edit an OWNERS.yaml file, it's hard to find the format for declaring owners for individual files / directories, and for declaring a working group as an owner.
A single line comment that points to the OWNERS.yaml spec would be useful.
I'm happy to write up a PR if a link exists.
/cc @mrjoro

migrated to https://github.com/ampproject/amp-github-apps/issues/282


is there a watch feature similar to angularfire-1 ?

how to get notified when a firebase object/list (which i am listening to) gets modified ?
looking for something similar to $watch() in angularfire-1
Thanks 👍

Since objects/collections are Observables, you just use Observable.prototype.subscribe().
All issues have to follow the issue template provided. This is critical to keep things streamlined and manageable. Closing this. Feel free to resubmit according to the template; if you do, be sure to include a working, minimal repro of the problem and version info.

@katowulf thanks 👍
and what is the equivalent of $loaded feature ?

.first().subscribe().


Project's copyright not attributed to OpenZipkin developers

All file headings and nuspec files attributes the project to Bazinga technology

Fixed


changed resource urls.. app works now

Changed the request URLs according to the updated APIs of Metamug backend. App working now.

Merged with your code


Error: Could not find or load main class com.android.sdklib.tool.sdkmanager.SdkManagerCli

Hi i am trying to install flutter and getting an error when i type sdkmanager --list in the command prompt .I have added the environment variables correctly.I am attaching the screenshots that shows this problem and also the environment variables i am using and my directory structure.I am trying to install command line tools only and not android studio.How can i solve this error?

Hi @atheistrepublic
please double check your configuration with the docs
and provide your flutter doctor -v  (you may use ..\flutter\bin\flutter.bat)
Thank you

In ubuntu i solved it by using older version of sdk tools.It seems there was some bug on newer version of command line tools zip file.

HI @atheistrepublic
did the solution you mentioned in your last message solved the issue?
If it didn't could you please provide the information mentioned above
Thank you

I got this issue also. Then I found this solution. This is probably not a problem with the settings of Java or sdk tools.

Solution:
Don't use the Windows' embedded unzip tool. Switch to another, like 7z.
The directory will then be referenced correctly, and you can run sdkmanager.bat without errors.

Using 7zip instead worked for me.
After unzipping, I ran sdkmanager --android-licenses in terminal and replied y to confirm every update.


runtime: zero unused portion of temp buffer in stringtoslice{byte,rune}

Maybe we should zero the unused portion of the temp buffer instead of reducing the capacity of the returned slice.
See issue #14232

Zeroing the tmpBuf is only 4 instructions (on amd64), so it is probably worth it.  It allows things like the append in the following function to work without allocation:
func g(s, t string) string {
	b := ([]byte)(s)
	b = append(b, ([]byte)(t)...)
	return string(b)
}


Bad service name declaration in configuration file

@weierophinney I have not time to test the module in Apigility context but I think the last release is not stable because of wrong declaration of services.

These are fine.
The file now declares itself in the ZF\Hal namespace. PHP, when it encounters a ::class reference will resolve the given name based on the namespace and/or import statements, regardless of whether or not the class actually exists. If you want to give it a try:
<?php
namespace Foo\Bar;

echo Baz::class, "\n";

This will emit:
Foo\Bar\Baz

I know the behavior of namespaces but in our case the alias declared is not the good one to satisfy following usage of class constant in sub namespaces.
For instance, there is a ZF\Hal\JsonRenderer service according to ZF\Hal\View\HalJsonRenderer class and now we declare it like that:
namespace ZF\Hal;

return [
    'zf-hal' => [
        'service_manager' => [
            'factories' => [
                JsonRenderer::class => Factory\HalJsonRendererFactory::class,
            ],
        ],
    ],
];

The class ZF\Hal\JsonRenderer does not exist and even if we declare service with View\HalJsonRenderer::class, the service name would become ZF\Hal\View\HalJsonRenderer instead of ZF\Hal\JsonRenderer.
My reasoning is not good?

ping @weierophinney

@weierophinney Could you take a look to this possible issue please?

@neeckeloo What version of PHP are you running on?


How to Generate DKIM private/public keypair when hosting on portainer

Hi, I host anonaddy as stack on portainer, how do I Generate DKIM private/public keypair? I am changing domain so I am also wondering if its needed to regenerate key, and what is the key good for (I guess for DKIM record for adding custom domains)? Is it needed? I don't have PGP encryption turned on.
When I do exec and run the cmd, it doesn't work

When I tried to exec cmd as anonaddy user

It's not an artisan command but a dedicated script. Just call gen-dkim.

thanks


The field-dependencies JS implementation doesn't work

Internally the previous implementation (via PHP) is used instead of the new field-dependencies module.
This was working at some point, I have to see where/when it broke

https://github.com/aristath/kirki/commit/84c881ab7b01c8a541497810e52d389d621acf67 partially fixed the issue.
Now the JS implementation kicks in immediately so we don't have to wait for the ajax call in order to hide the control.
However, if we completely deprecate the PHP implementation, as soon as the ajax call is made, the control gets shown no matter what because it defaults to __return_true.
No time to investigate today so I'll leave it the way it is now.
Will continue another day

Moving to the 3.1 milestone.
The current implementation (a combo of PHP & JS) works.
It's not perfect, but it works fine and there are no issues with it.
This refactoring item can be taken care of after the release of 3.0


fix(Explore): Show the tooltip only when label does not fit the container in the Dataset panel

SUMMARY
This PR makes the tooltip appear only when the label does not fit the container in the Dataset panel. Also, it hides the tooltip when the drag event starts.
BEFORE
https://user-images.githubusercontent.com/60598000/129005898-a3b9a073-355d-41d2-9627-fc77f1083238.mp4
AFTER NO DRAG AND DROP
https://user-images.githubusercontent.com/60598000/129004634-224d4f17-8a2a-4c00-9376-0b70005b6f67.mp4
AFTER DRAG AND DROP ON
https://user-images.githubusercontent.com/60598000/129004722-337326ca-d255-4203-ae7c-f95502067803.mp4
TESTING INSTRUCTIONS

Visit Explore
Observe the labels with long text in the Dataset panel and make sure they are truncated with ellipsis
Hover the labels and check the tooltip
Expand the panel until the text isn't truncated and hover the labels
Make sure the tooltip isn't appearing
Drag an item
Make sure the tooltip isn't appearing

ADDITIONAL INFORMATION

[x] Has associated issue: #15996
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@rosemarie-chiu @jinghua-qa

/testenv up FEATURE_ENABLE_EXPLORE_DRAG_AND_DROP=true

Thanks for the improvement, this is awesome!

can we also add this enhancement to non-dnd? I think user can be benefit from it. There are quite a few features built upon dnd so far, can you guys help brainstorm how to apply those to all 3 modes, easily? @kgabryje @villebro

towards the end of the video, after i drop the super long metric in the zone, tooltip is not attached to the field, maybe because i resize the panel early on? let's address this issue in this pr 🙏:)
https://user-images.githubusercontent.com/67837651/129130547-352e010e-9393-41eb-bfd5-e82f664f87b4.mov


chore: :bookmark: release 1.2.0

:robot: I have created a release beep boop
1.2.0 (2024-11-03)
Features

:sparkles: Add background for all slides (0523373)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.2.0
:sunflower:


Floating IPs to Reserved IPs

This PR adds the new Reserved IP endpoints and adds notices to the Floating IP descriptions addressing the upcoming changes.

!deploy

Following two sections also require updates:

/v2/actions now support new HTTP header ("Accept: application/vnd.digitalocean.reserveip+json") to allow retrieving reserved_ips` actions
/v2/droplets/<droplet_id>/destroy_with_associated_resources* now support reserved_ips stanza in response body

!deploy


深度学习的实用层面 - ATuk's Blog

https://alberthg.github.io/2018/04/12/deeplearning-ai-c2w1/
Hello World

 Liquid Exception: Liquid syntax error (line 123): Variable '{{||w^{[l]}' was not properly terminated with regexp: /\}\}/ in /home/albert/Documents/Github Repository/alberthg.github.io/_posts/2018-04-12-deeplearning-ai-c2w1.md
             Error: Liquid syntax error (line 123): Variable '{{||w^{[l]}' was not properly terminated with regexp: /\}\}/
             Error: Run jekyll build --trace for more information.

Latex公式报错，无法生成网页。


Start backend

Changes
Added Frontend/Backend Directories
Created front end and back end directories and migrated the first phase of the project to the front end directory.
Setup Backend directory
Created a node project and installed dependencies for contacting Ms Sql database. As well as creating sample routes controllers and a sample Db schema to help create a sample database to test the connection.
Check out
Checkout branch: start-backend
Set Up

Run sampleSchema.sql in your preferer database management application (SSMS, Or a vs code extension)
Create a file called .env and structure it like the file .env.sample (.env will not be tracked)
In the backend directory run:

npm  i

Then run :

npm run dev

In your browser navigate to:

http://localhost:5000/index.html

Click the button to view the sample data, if you see some names great, you have created a DB and linked it to a interface.

Please check this out and make sure you can test the connection @thiagolima19 @manufflopes @mariannaAntunes

Just to clean up any questions about this Pr, I created a sample data back end, then @ArshdeepKaur96 improved on the file structure, Then I fixed a few things about the my class page that was bugging me.


Building Issue with Ubuntu 16.04 LTS

Change Dir: /home/bing/github/SFrame/release/CMakeFiles/CMakeTmp

  Run Build Command:"/usr/bin/make" "cmTC_a0b2c/fast"

  /usr/bin/make -f CMakeFiles/cmTC_a0b2c.dir/build.make
  CMakeFiles/cmTC_a0b2c.dir/build

  make[1]: Entering directory
  '/home/bing/github/SFrame/release/CMakeFiles/CMakeTmp'

  Building C object CMakeFiles/cmTC_a0b2c.dir/testCCompiler.c.o

  /home/bing/github/SFrame/deps/local/bin/cc -o
  CMakeFiles/cmTC_a0b2c.dir/testCCompiler.c.o -c
  /home/bing/github/SFrame/release/CMakeFiles/CMakeTmp/testCCompiler.c

  Linking C executable cmTC_a0b2c

  /home/bing/github/SFrame/deps/local/bin/cmake -E cmake_link_script
  CMakeFiles/cmTC_a0b2c.dir/link.txt --verbose=1

  /home/bing/github/SFrame/deps/local/bin/cc
  CMakeFiles/cmTC_a0b2c.dir/testCCompiler.c.o -o cmTC_a0b2c -rdynamic

  /home/bing/github/SFrame/deps/local/bin/../lib/gcc/x86_64-unknown-linux-gnu/4.9.2/../../../../x86_64-unknown-linux-gnu/bin/ld:
  /usr/lib/x86_64-linux-gnu/crti.o: unrecognized relocation (0x2a) in section
  `.init'

  /home/bing/github/SFrame/deps/local/bin/../lib/gcc/x86_64-unknown-linux-gnu/4.9.2/../../../../x86_64-unknown-linux-gnu/bin/ld:
  final link failed: Bad value

  collect2: error: ld returned 1 exit status

  CMakeFiles/cmTC_a0b2c.dir/build.make:97: recipe for target 'cmTC_a0b2c'
  failed

  make[1]: *** [cmTC_a0b2c] Error 1

  make[1]: Leaving directory
  '/home/bing/github/SFrame/release/CMakeFiles/CMakeTmp'

  Makefile:126: recipe for target 'cmTC_a0b2c/fast' failed

  make: *** [cmTC_a0b2c/fast] Error 2

Same here.

Hmmm.... any news? I'm getting this error too.  Thank you.

@warpdriv - as mentioned in the top level readme, this repository is deprecated. All of this functionality has moved to (Turi Create)[https://github.com/apple/turicreate]. That repository builds fine on Ubuntu 16.04.


README Changes

We're starting a new iteration of 100 Days of Code and we need to change the readme for this.
Join in the discord for more information on it: dsc.gg/invide

can I contribute to this issue?

Yes, everyone can contribute. But there are important changes that needs to be discussed. Please make sure it's not just a way to get away with Hacktoberfest swags. We at Invide, do not condone this behavior.

Okay @Bhavya0304 please do suggest the changes and what all you'd like to work upon. Also meanwhile join our discord server: dsc.gg/invide


Add consolidated balance in Budget (inlcuding projects and events)

As a Collective admin, I want to know how much balance I have across main profile, projects and events.
Today, I only see the balance of the parent.

This should also be reflected in the "total raised" stat in the budget section

+1 to this and also I would like to add the project/events budget to the project/event cards in the projects/events list. For example there is currently $580k in the UI project on the web performance fund, but looking at the collective it only appears to have $~250 in it :D

Seems resolved by https://github.com/opencollective/opencollective-frontend/pull/7981?

@Betree yes!


Declare packages as installed on host

Is your feature request related to a problem? Please describe.
When building programms for a complex infrastrukture like a cluster it would be nice do be able to define certain packages as already installed on the machine so that one dosen't has to build tings like openmpi and figure out the right configuration for the environment.
Proposed solution
As this configuration sould pobably not go into the project directory one could add a config file for vcpkg to ~/.config/vcpkg to define this packages. For the defined packages it is asumed that they are installed to system paths (or the path are in the aplicable environment variables) and can be fond by find_package.
Describe alternatives you've considered

Modifing the vcpkg.json manifest for the project on the cluster, but this dosn't work dependencies of packages that are not installed.
Adding the config file to the project root, but that would then not apply to other projects build on the system and would explictly be removed from source control

Additional context
Building certain common packages on clusters (like boost, hdf5, openmpi) takes a long time and normaly a recent enough version is already configured and installed on the system.

@L0ric0, I'm not sure if I understand the case correctly, currently we have .\installed\vcpkg\status file to record the installed package, and 'vcpkg list' would display the installed package as well.

It is not to get the installed packages, but to tell vcpkg that a certain package is installed and doesn't need to be build.
So lets say i have openmpi installed from a distro package and a Project that depands on boost and i know that the already installed openmpi meets the requirements of the project, but have another host on which the project also has to run where openmpi is not installed. so i define a vcpkg.json file that lists openmpi in its dependencies. At the moment both on the host that has openmpi already installed and the host where it is not openmpi is build. What i propose is a file where the vcpkg user can define which packages do not need to be build as the host system already provides them. So the host with openmpi already installed would not have to build it again and the user wouldn't have to worry about its configuration for this particular environment.

I see, could you check if the binary caching meet the requirements?

Hi @L0ric0 Have you tried the above suggestion form @PhoebeHui? Does it work for you?

I strongly doubt the need to add this feature.
The pre-compiled binary downloaded manually by the user has a different version/different configuration from the vcpkg in most cases, so other errors may occur.


Issue with Metadata cache setting?

When using the command "COBOL: process files in workspace for metadata" I now get:

Metadata cache is off, no action taken

As this is a multi-folder workspace (primarily because of the suggestion to do so here) the workspace setting should be used and that has:
		"coboleditor.cache_metadata": "storagepath",
		"coboleditor.parse_copybooks_for_references": true,
		"coboleditor.pre_parse_line_limit": 250,

Looks like fallout from merging the coboleditor.cache_metadata and coboleditor.cache_directory_strategy settings.

resolved

Thanks for 8fa9f320c10b498c9ff675425bcb00d302abe795 (only not referenced because of missing space)


[BUG]: UI is not showing

I have done all things successfully but the app UI not showing, I'm trying all browsers still UI does not show
App run smoothly not a single error show but I don't know why UI not showing

can you share your console logs?

please check @misshula

can you share your console logs?

Hey @misshula  I have shared error please check


Foundry VTT v10 support?

Hi.
Excellent module. Just wondering if you're intending to update for Foundry v10?
I've had a look at the code and couldn't quite get my head around how the presets would work in the new DataModel (and it's use in Settings)
Edit: Premature 'save' :D - I meant to say: I was going to do a PR but as mentioned, got a little lost with how the new datamodel would effect presets and had to give up :D

Hey @joevaughan , I have been testing this module on v10 for some time (last couple of builds in fact). I only have a passing familiarity with the changes to the DataModel that you mentioned but so far the presets seem to still work. Are you experiencing a problem with them?

I've just been reading round on this and according to the API documentation, the method I'm using should still work: https://foundryvtt.com/api/classes/client.ClientSettings.html.
I wonder if the issue here is related to the word "undefined". Have you used the hourglass module before on your current installation? I'm just wondering if the validation is now checking whether the requested value from the "game.settings.get" request is null/undefined and throwing an error as a result.
Does the module break/stop working once this error is thrown or does it continue?
It's entirely possible that I'm storing this preset data in an inappropriate place and that I should move to work with the new data model moving forward, but since it still worked in my own testing I didn't dig further.

I think it may be because I've never run hourglass on this particular FVTT world before. So there probably aren't any presets at the point the gui is calling this (they'll be populated later) and the new Data validation routines are throwing (as we say in the UK) a wobbly.

Ok that makes sense now. I imagine anyone who is installing the mod for the first time will have the same issue, I'll spin up a new world on my installation later on this evening and do some testing. Hopefully I can just check if this is a first time load and initialise the setting value to some default (an empty array). Thanks for reporting this!

Ok so I was able, following your instructions, to quite quickly replicate the bug. I have wrapped that game.settings.get method call in a try catch for now and log to the console the fact that no existing presets were present for the new game world in event of this error popping up.
As I suspected, saving a preset has the effect of initialising the settings value which negates the "undefinied" error.
I will need to do some proper research on this presets storage as I think I should probably be storing that data slightly differently to match the ideal workflows.
I have released a hotfix (v1.5.1) which contains the fix for this bug. You should be good to go. Thanks again so much for reporting this bug. I imagine it was preventing a least a few other folk from using the module on new game worlds.

Awesome. Many thanks @Octarines :D


use invariant to assert state shape in api middleware

saw you merged this already but was just gonna suggest using invariant to assert state shape

LGTM


Add Wix Corvid

I would like to create a content hash to use with my external API on my wix.com website which uses Corvid as the coding language. This code is something I have been unable to figure out after weeks of relentless research and forum posting.

I think this PR is messed up a bit - the "Files Changes" matches what's already on master.


Demo weirdly broken with adblocker

Chrome stable 71.0.3578.98, uBlock origin 1.17.4, MacOS.

load the demo page https://react-idle.modus.app/
"render on idle" should be off
click some months (>5)
quickly tab some months (>5)
quickly tick "render on idle"

After the last step, the screen starts blinking and months are continuously selected and deselected.
It's as if the page "buffered" really many tabs and is now replaying each change.
The weirdness doesn't stop on its own.
Leaving the browser tab (e.g. focusing on another tab or window), allows process to conclude.

v1.1.0 published to npm. Please reopen the issue if the problem persists. Thanks, @dimaqq


FR: acceptable errors amount while importing out of CSV

It would be nice to have a flag like --errors-allowed while doing a command like:
cat file.csv | clickhouse-client --database=test --query="INSERT INTO test FORMAT CSV";
Is this possible? Thanks in advance.

It is possible to add this feature.
Possible implementation:
Add syncAfterError method for input formats (IRowInputStream).
For example, for CSV, it will skip to next line.
And then add wrapper method, that will call read and syncAfterError.

@alexey-milovidov Nice to hear, mate. Is there any ETA on its implementation, or may be an approximate date when this feature would be included in the release? Or I'm asking for too much?)

No ETA, it is not included in our plans right now.
BTW, if you are C++ developer or you have your collegues, who know C++, I may write more verbose task description.

https://github.com/Civil has intention to implement this task. Here is detailed description:

Add virtual method
void syncAfterError()
to IRowInputStream.
By default, it throws not implemented exception.

Implement these methods for TabSeparated, CSV, TSKV, JSONEachRow. They will ignore bytes in ReadBuffer until next newline or eof. Use findFirstSymbols method, see examples of its usage.

Add setting input_format_allow_errors_num and input_format_allow_errors_ratio. See Settings.h.
Both are zero by default. Pass these settings to BlockInputStreamFromRowInputStream, see FormatFactory.

Use these settings in BlockInputStreamFromRowInputStream. Settings are OR-ed.
Wrap reading next row into try/catch block. Catch only exceptions that could be thrown from functions in ReadHelpers.h and from various DataTypes. If both settings are zero, then rethrow. Otherwise, increment counters and use syncAfterError(). Check that amount of errors is lower than at least one of thresholds.

Be careful to save initial exception message if syncAfterError() is not implemented or if errors amount is greater than threshold.

This has been implemented back in 2017.

This has been implemented back in 2017, see input_format_allow_errors_num and input_format_allow_errors_ratio settings.


How can I deserialize the response.all_messages_json() in order to use for the next query message_historys?

Consider a Redis key available with the initial value of []. I'm storing the response.all_messages_json() every time I make a query. Only the first time everything works and the message history will be put into the database. Next time when I try to retrieve the history and make a new query based on that history, the system breaks. Here is an example of a message history retrieved from Redist:
[
    {"content": "you are a helpful assistant.", "timestamp": "2024-12-10T18:58:25.794340Z", "role": "system"},
    {"content": "hello", "timestamp": "2024-12-10T18:58:25.794340Z", "role": "user"},
]

Traceback:
AttributeError: 'dict' object has no attribute 'role'

I assume I need to deserialize the retrieved history to some MessageHistory object thing, but I can't find it.

@Inxpy,
Perhaps you're looking for messages.Message? Specifically:
Message = Union[SystemPrompt, UserPrompt, ToolReturn, RetryPrompt, ModelTextResponse, ModelStructuredResponse]
"""Any message send to or returned by a model."""

Thank you for your response, @sydney-runkle!
I've already looked into it, but it's just a type and I guess I can't go like:
response = await agent.run(
    request.message,
    message_history=[Message(**i) for i in chat_history],
)

Use
https://github.com/pydantic/pydantic-ai/blob/08808eb78616913ec8d305205a75774cb34404ac/pydantic_ai_slim/pydantic_ai/messages.py#L203
MessagesTypeAdapter.validate_python(messages)


Related components

Given that components are often released in sets, we should allow users to discover sets of components that have definitions and add these to the components list.
This one is a little abstract as we don't have a concrete way of defining "related". One relationship is Source. Another is "other package format". The scenario that inspired this issue was a set of packages that all go together (e.g., foo-sdk, foo-cli, ...). Unfortunately without some additional context that may be hard to automate.

@jeffmcaffer I believe this issue is superseded by the suggestions work we're doing. Can you confirm and we can close this?

This is different. Suggestions is suggesting how the user can curate a given definition. This is aimed at helping the user find related components to curate. So say you are looking at Foo 1.0 and find the license is messing. It should be really easy for you to also then add all other versions of Foo (or 1.* or just ones also missing licenses or, ...) to your curation list. At that point you can use the Suggestions facility to correct the license (whatever) and do one or more PRs with the curations.

This needs to be discussed. Probably with the mongo store in place, now we can provide this kind of relationship and find all the related components of a specific one.
We should understand how this would impact both the UI and the service side of things.

see https://github.com/clearlydefined/service/issues/402 for the service side of this task.
I think a nice place to prompt the user to add related components to their definition list is right after a contribution is sent.
Another place may be inside the definition list when there is only 1 component added?
Any other ideas?

@dabutvin maybe could be like a "suggestion" for components?
Currently, if a field has a suggestion, an icon appears near to the field and the user is able to see the details and eventually pick a selection.
Maybe the same thing could happen for a definition, with a custom icon that identifies that the component has some relations, and after clicking it we could show a list of all the relationships, so the user could select which ones add to the list.
Thoughts?


feat: update JSON schemas for global config and project/app config, fixes #6097

The Issue

#6097

How This PR Solves The Issue

add corepack_enable, missing hooks pre-delete-snapshot, post-delete-snapshot to config.yaml
add instrumentation_user, last_started_version and mkcert_caroot to global_config.yaml
remove host_dba_port from config.yaml
remove hooks from global_config.yaml
add enums for omit_containers and database in config.yaml
update documentation for bind_all_interfaces, instrumentation_user, pre-delete-snapshot, post-delete-snapshot
add more punctuation to JSON schemas.

Manual Testing Instructions
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Oh, you just did this and then in comes

https://github.com/ddev/ddev/issues/6100

:)


Add a way to pass binary data

Hi!
I'd like to pass binary data between PHP and the Rust crate. As far as I understand both on PHP side this is just a string while on rust side they'd be slices of u8 (or vectors?).
Is something like that possible or planned?
Thanks again for this great library!

Will take a look into it this weekend, seems simple enough.

from_utf8_lossy will replace invalid chars with replacement character so it may break the underlying data if it's not UTF8 https://doc.rust-lang.org/std/string/struct.String.html#method.from_utf8_lossy
I need to check the actual example that you have. Thanks for help!

Could you have a look at #32 and tell me what you think? Seems to work for most types that pack works with. Here's a quick example:
/// Unpacks a binary u8 array
pub extern fn skel_unpack(ex: &mut ExecutionData, ret: &mut Zval) {
    let mut packed = Arg::new("packed", DataType::String);
    parse_args!(ex, packed);
    let value = unsafe { packed.zval().unwrap().binary::<u8>(); };
    dbg!(value);
}

/// Packs a binary u8 array
pub extern fn skel_pack(ex: &mut ExecutionData, ret: &mut Zval) {
    ret.set_binary(vec![12u8, 13, 14, 15, 20, 25]);
}

$x = [15, 30, 45];
skel_unpack(pack('C*', ...$x));
// [15, 30, 45] from Rust

$x = unpack('C*', skel_pack());
var_dump($x); // array(6) [12, 13, 14, 15, 20, 25]

David, sorry for the super late reply but I did write a small proof-of-concept code using set_binary and it works very much fine: https://gitlab.com/willbrowning/anonaddy-sequoia/-/merge_requests/7/diffs I did file_put_contents the binary and it saved all bytes just fine. So :+1: from me!
Thanks for your time and hard work on this lib! I really appreciate it.

No worries, thanks for confirming it works. I'll merge it into master now.


Add Pypi feed testing

Adds some tests to the pypi feed using httptest as a mock endpoint.

+1 for more testing!

Closing as this will be added to https://github.com/ossf/package-feeds/pull/76


Emulator: Services pane should show help text whether or not services exist

Version
Mac - 4.0.0-preview-one-off
Windows: - 4.0.41221
Describe the bug
Help text below the list of services should appear when services exist (not just when there are no services.)
To Reproduce
Steps to reproduce the behavior:

Open a bot file.
Open the Services pane.
If your bot file has services, it will look like this:

Expected behavior
It should show this help text (which currently only appears when there are no services):

There are extra spaces before the links and there is a period missing after "learn more about using services".
Additional context
Design is on Invision: https://microsoft-fuse-design.invisionapp.com/d/main#/console/15012340/317169752/preview

[bug]

Fixed with #984


Ctrl+Tab in Firefox[-nightly] overlapping on ultrawide monitor (3440x1440@144)

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit f8857e6072bd85b95393499688872aaf7f088b5b  (input: find surface pos correctly when mouse drag is active).
Date: Sat May 18 22:20:01 2024
Tag: v0.40.0-113-gf8857e60, commits: 4719

flags: (if any)

System Information:
System name: Linux
Node name: Archee
Release: 6.9.1-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Fri, 17 May 2024 16:56:38 +0000

GPU information:
00:02.0 VGA compatible controller [0300]: Intel Corporation Raptor Lake-S GT1 [UHD Graphics 770] [8086:a780] (rev 04) (prog-if 00 [VGA controller])
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation TU106 [GeForce RTX 2060 SUPER] [10de:1f06] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description
When doing CTRL+Tab in Firefox[-nightly] on ultrawide monitor (3440x1440@144), the tab previews overlap, making it unusable. When moving to smaller monitors, it work normally.
See screenshot below

How to reproduce
Simply launch firefox on ultrawide monitor, configure it to Ctrl+Tab in recent order and observe the overlap. When moving firefox to smaller monitors it works okay. Also, sometimes (very rarely) it even works on the ultrawide.
 I've never seen that preview. Ctrl+tab for me just cycles

Lol. Why is a bug in an experimental version of Firefox being reported as a Hyprland bug? Can you test this on another WM/DE and let us know if this only happens on Hyprland?
I don't have an ultrawide monitor or the nightly version of firefox.

what the hell is that? I've never seen that preview. Ctrl+tab for me just cycles

It's when set that Ctrl+Tab cycles in recently used order.

Lol. Why is a bug in an experimental version of Firefox being reported as a Hyprland bug? Can you test this on another WM/DE and let us know if this only happens on Hyprland?
I don't have an ultrawide monitor or the nightly version of firefox.

It happens even in normal firefox and for many versions now. Testing out in different WM/DE will take me a bit atm.

Can you point me to how to enable this? I pulled latest release from arch repository this morning and I control tab just brings me to the next tab.

Can you point me to how to enable this? I pulled latest release from arch repository this morning and when I control tab just brings me to the next tab.

looks fine on both of my monitors (neither are ultrawide)

Same, I got this to work, its got a little jank in terms of window stretching occasionally. Maybe I'll play with this more later but I've never used this sort of feature. I just use vimiums keybinds for tab navigation.
Lmk if you can reliably replicate breakage though, and provide config so we can make sure we have the same window rules.

Same, I got this to work, its got a little jank in terms of window stretching occasionally. Maybe I'll play with this more later but I've never used this sort of feature. I just use vimiums keybinds for tab navigation.
Lmk if you can reliably replicate breakage though, and provide config so we can make sure we have the same window rules.

Well, it works on my smaller resolution monitors as well. Also, when I switch to pseudo tile it works again.
My window rules config
```
# ###################################
#           Browser                 #
# ###################################
windowrulev2 = opacity 1 1,class:^(firefox-nightly)$
windowrulev2 = workspace 1, monitor 1,class:^(firefox-nightly)$
windowrule = idleinhibit fullscreen,class:^(firefox-nightly)$
windowrulev2 = opacity 1 1,class:^(firefox)$
windowrulev2 = workspace 1, monitor 1,class:^(firefox)$
windowrule = idleinhibit fullscreen,class:^(firefox)$
exec-once = LIBVA_DRIVER_NAME=iHD firefox-nightly
windowrulev2 = opacity 0.90 0.90,class:^(Brave-browser)$
windowrulev2 = workspace 2, monitor 1,class:^(Brave-browser)$
windowrule = idleinhibit fullscreen,class:^(Brave-browser)$

Music Player

windowrulev2 = opacity 0.90 0.90,class:^(rhythmbox)$
windowrulev2 = workspace 5, monitor 1,class:^(rhythmbox)$

Email client

windowrulev2 = opacity 0.80 0.80,class:^(BlueMail)$, title:^(BlueMail)$
windowrulev2 = workspace 3 silent, class:^(BlueMail)$, title:^(BlueMail)$
exec-once = bluemail -in-process-gpu
windowrulev2 = opacity 0.80 0.80,class:^(thunderbird)$, title:^(Mozilla Thunderbird)$
windowrulev2 = workspace 3 silent, class:^(thunderbird)$, title:^(Mozilla Thunderbird)$
exec-once = thunderbird

DOCs

windowrulev2 = opacity 0.80 0.80,class:^(libreoffice-startcenter)$, title:^(LibreOffice 7.6)$
windowrulev2 = workspace 4 silent, class:^(libreoffice-startcenter)$, title:^(LibreOffice 7.6)$

Note Tacking

exec-once = notion-app
NOTE: note sure about the class/title, not using notion-app anymore
windowrulev2 = workspace 3 silent, class:^(notion-app)$, title:^(notion-app)$

Steam

windowrulev2 = opacity 0.80 0.80,class:^(Steam)$
windowrulev2 = opacity 0.80 0.80,class:^(steam)$
windowrulev2 = opacity 0.80 0.80,class:^(steamwebhelper)$

Spotify

windowrulev2 = opacity 0.80 0.80,class:^(Spotify)$

vs code

windowrulev2 = opacity 1 1,class:^(Code)$
windowrulev2 = opacity 1 1,class:^(code-url-handler)$

Terminal

windowrulev2 = opacity 0.80 0.80,class:^(kitty)$
windowrulev2 = float,class:^(cool-retro-term.cool-retro-term)$
windowrulev2 = opacity 0.80 0.80,class:^(cool-retro-term.cool-retro-term)$

Theming apps

windowrulev2 = opacity 0.80 0.80,class:^(nwg-look)$
windowrulev2 = float,class:^(nwg-look)$
windowrulev2 = opacity 0.80 0.80,class:^(Lxappearance)$
windowrulev2 = float,class:^(Lxappearance)
windowrulev2 = opacity 0.80 0.80,class:^(kvantummanager)$
windowrulev2 = float,class:^(kvantummanager)

GTKs

windowrulev2 = opacity 0.90 0.90,class:^(com.github.rafostar.Clapper)$ #Clapper-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(com.github.tchx84.Flatseal)$ #Flatseal-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(hu.kramo.Cartridges)$ #Cartridges-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(gnome-boxes)$ #Boxes-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(app.drey.Warp)$ #Warp-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(yad)$ #Protontricks-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(Signal)$ #Signal-Gtk
windowrulev2 = opacity 0.80 0.80,class:^(io.gitlab.theevilskeleton.Upscaler)$ #Upscaler-Gtk
windowrulev2 = float,class:^(Signal)$ #Signal-Gtk
windowrulev2 = float,class:^(app.drey.Warp)$ #Warp-Gtk
windowrulev2 = float,class:^(io.gitlab.theevilskeleton.Upscaler)$ #Upscaler-Gtk
windowrulev2 = float,class:^(yad)$ #Protontricks-Gtk
windowrulev2 = float,class:^(com.github.rafostar.Clapper)$ #Clapper-Gtk
windowrulev2 = float,class:^(eog)$ #Imageviewer-Gtk
windowrulev2 = float,class:^(gwenview)$ #Imageviewer

QTs

windowrulev2 = opacity 0.80 0.80,class:^(com.obsproject.Studio)$ #Obs-Qt
windowrulev2 = opacity 0.80 0.80,class:^(net.davidotek.pupgui2)$ #ProtonUp-Qt
windowrulev2 = float,class:^(net.davidotek.pupgui2)$ #ProtonUp-Qt
windowrulev2 = opacity 0.80 0.80,class:^(qt5ct)$
windowrulev2 = float,class:^(qt5ct)$

ELECTRONs

windowrulev2 = opacity 0.80 0.80,class:^(discord)$ #Discord-Electron
windowrulev2 = opacity 0.80 0.80,class:^(WebCord)$ #WebCord-Electron

File explorer

windowrulev2 = opacity 0.80 0.80,class:^(org.kde.dolphin)$
windowrulev2 = float,class:^(org.kde.dolphin)
windowrulev2 = opacity 0.80 0.80,class:^(nemo)$
windowrulev2 = float,class:^(nemo)

KDEs

windowrulev2 = opacity 0.80 0.80,class:^(org.kde.ark)$
windowrulev2 = float,class:^(org.kde.ark)$
windowrulev2 = float,class:^(org.kde.polkit-kde-authentication-agent-1)$
windowrulev2 = opacity 0.80 0.70,class:^(org.kde.polkit-kde-authentication-agent-1)$

pavucontrol

windowrulev2 = opacity 0.80 0.70,class:^(pavucontrol)$
windowrulev2 = float,class:^(pavucontrol)$

wlogout

windowrulev2 = opacity 0.80 0.70,class:^(wlogout)$
windowrulev2 = float,class:^(wlogout)$
windowrule = fullscreen, wlogout
windowrule = float, title:wlogout
windowrule = fullscreen, title:wlogout

blueman

windowrulev2 = opacity 0.80 0.70,class:^(blueman-manager)$
windowrulev2 = float,class:^(blueman-manager)$

Network Manager

windowrulev2 = opacity 0.80 0.70,class:^(nm-applet)$
windowrulev2 = float,class:^(nm-applet)$
windowrulev2 = opacity 0.80 0.70,class:^(nm-connection-editor)$
windowrulev2 = float,class:^(nm-connection-editor)$

Launcher

windowrulev2 = opacity 0.80 0.70,class:^(wofi)$
windowrulev2 = float,class:^(wofi)$
windowrulev2 = opacity 0.80 0.70,class:^(tofi)$
windowrulev2 = float,class:^(tofi)$
windowrulev2 = opacity 0.80 0.70,class:^(rofi)$
windowrulev2 = float,class:^(rofi)$
windowrulev2 = opacity 0.80 0.70,class:^(viewnior)$
windowrulev2 = float,class:^(viewnior)$
windowrulev2 = opacity 0.80 0.70,class:^(feh)$
windowrulev2 = float,class:^(feh)$

Others

windowrulev2 = opacity 0.80 0.70,class:^(file_progress)$
windowrulev2 = float,class:^(file_progress)$
windowrulev2 = opacity 0.80 0.70,class:^(confirm)$
windowrulev2 = float,class:^(confirm)$
windowrulev2 = opacity 0.80 0.70,class:^(dialog)$
windowrulev2 = float,class:^(dialog)$
windowrulev2 = opacity 0.80 0.70,class:^(download)$
windowrulev2 = float,class:^(download)$
windowrulev2 = opacity 0.80 0.70,class:^(notification)$
windowrulev2 = float,class:^(notification)$
windowrulev2 = opacity 0.80 0.70,class:^(error)$
windowrulev2 = float,class:^(error)$
windowrulev2 = opacity 0.80 0.70,class:^(splash)$
windowrulev2 = float,class:^(splash)$
windowrulev2 = opacity 0.80 0.70,class:^(confirmreset)$
windowrulev2 = float,class:^(confirmreset)$
windowrulev2 = opacity 0.80 0.70,class:^(Open File)$
windowrulev2 = float,class:^(Open File)$
windowrulev2 = opacity 0.80 0.70,class:^(branchdialog)$
windowrulev2 = float,class:^(branchdialog)$
windowrulev2 = opacity 0.80 0.70,class:^(file-roller)$
windowrulev2 = float,class:^(file-roller)$
windowrulev2 = opacity 0.80 0.70,class:^(Media viewer)$
windowrulev2 = float,class:^(Media viewer)$
windowrulev2 = opacity 0.80 0.70,class:^(Volume Control)$
windowrulev2 = float,class:^(Volume Control)$
windowrulev2 = opacity 0.80 0.70,class:^(Picture-in-Picture)$
windowrulev2 = float,class:^(Picture-in-Picture)$
</details>

This issue doesn't have anything to do with Hyprland, because I have the same issue on KDE Plasma 6 (wayland), with a non-nightly version of Firefox, but it's one of the few places I've seen it reported.
This bug has been previously reported on Bugzilla and closed, but it seems the fix doesn't work. I opened a new issue here: https://bugzilla.mozilla.org/show_bug.cgi?id=1904240

closing then


Spec tests break on the "certbot -h" command

When running spec tests, there is an error because the CertBot command is not specified in the correct way:
error during compilation: Validation of Exec[initialize certbot] failed: 'certbot -h' is not qualified and no path was specified. Please qualify the command or specify a path. at ../modules/certbot/manifests/init.pp:98

Issue seems to be in this part:
if $install_method == 'package' {
    $command      = $package_command
    $command_init = $package_command
  } elsif $install_method == 'vcs' {
    $command      = "${venv_path}/bin/certbot"
    $command_init = "${path}/certbot-auto"
  }

And the params.pp set it to:
  } elsif $::osfamily == 'RedHat' and versioncmp($::operatingsystemmajrelease, '7') >= 0 {
    $install_method = 'package'
    $package_name = 'certbot'
    $package_command = 'certbot'

We have to excplitly set the $package_command now, which works, but does not feel right.

Hey @fvanboven! The $::path fact must not be set correctly in the spec tests. I'll have a look at this. Thanks for reporting!


PV should not accept /confirmation_blocks

PV should not accept POST /confirmation_blocks

@buckyroberts already done in a branch, will make a PR soon.


When I am trying to do Pretraining for custom dataset I am getting this error ,In this i am just using two images and two captions

Start training
Traceback (most recent call last):
File "pretrain.py", line 173, in
main(args, config)
File "pretrain.py", line 131, in main
train_stats = train(model, data_loader, optimizer, epoch, device, config)
File "pretrain.py", line 48, in train
for i, (image, caption) in enumerate(metric_logger.log_every(data_loader, print_freq, header)):
File "/content/BLIP/utils.py", line 179, in log_every
header, total_time_str, total_time / len(iterable)))
ZeroDivisionError: float division by zero
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 1426) of binary: /usr/bin/python3
Traceback (most recent call last):
File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.7/dist-packages/torch/distributed/run.py", line 765, in
main()
File "/usr/local/lib/python3.7/dist-packages/torch/distributed/elastic/multiprocessing/errors/init.py", line 345, in wrapper
return f(*args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/torch/distributed/run.py", line 761, in main
run(args)
File "/usr/local/lib/python3.7/dist-packages/torch/distributed/run.py", line 755, in run
)(*cmd_args)
File "/usr/local/lib/python3.7/dist-packages/torch/distributed/launcher/api.py", line 131, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/usr/local/lib/python3.7/dist-packages/torch/distributed/launcher/api.py", line 247, in launch_agent
failures=result.failures,
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
pretrain.py FAILED
Failures:
<NO_OTHER_FAILURES>
Root Cause (first observed failure):
[0]:
time      : 2022-10-14_06:20:20
host      : 6d68fc040fa4
rank      : 0 (local_rank: 0)
exitcode  : 1 (pid: 1426)
error_file: <N/A>
traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html

For training on customized dataset, it is recommended to use our new LAVIS library: https://github.com/salesforce/LAVIS


Coverage system functions 20.14 test contains illegal verilog

initial begin
        i = $coverage_control(`SV_COV_CHECK, `SV_COV_TOGGLE, `SV_COV_HIER, $root);
                                                                              ^------ $root cannot be used without a hierarchical path, no commercial tool accepts this.

The LRM says it's allowed. See section <IP_ADDRESS>, they give that exact line as an example.

All the commercial tools fail with a syntax error and the grammar does not support a naked $root.
https://insights.sigasi.com/tech/systemverilog.ebnf/#hierarchical_identifier
How do you reconciled that?

Yeah, it's weird for sure, but this is certainly not the first legit feature in the LRM that commercial tools don't support, and it's also not the first feature to be described concretely in prose but not be represented correctly in the grammar. The LRM is simply not very rigorous about consistency.
I don't feel like the coverage control example is a typo because of the description: "checks all instances to verify they have coverage" which you couldn't accomplish without having syntax like this, but I also don't really care too much as it's likely nobody in the world has ever tried to do this (since most tools don't support it anyway).

OK, so how did you alter your grammar to support the naked $root? Did you change the hierarchical_identifier? Or something more targeted?

I allow the standalone $root during parsing and then check at analysis time whether it should be allowed (which only the coverage control system tasks do). It wasn't too difficult with my setup: https://github.com/MikePopoloski/slang/commit/81c0103ad1d9d92e525c6a5499ba8c5b843e29c3

Got it.


Add minimal viewer (size related)

Because size matters

The idea:
A few built viewers, one with full UI capabilities, including all of the needes dependencies, and one with the smallest filesize possible, with no unneeded dependencies.
Also - minify all builts automatically.

I ran a few tests. The entire unneeded dependencies (images, handlebars, htmls) are ca. 100kb not-minified. The majority of the viewer.js file i actually Babylon and its modules (loaders, materials, and Babylon itself).
I added a minified version, which is 2.3MB, and optimized everything I could in order to get the smallest size possible without making any breaking changes.
Using the "minimal" configuration and the minified viewer will have a very short loading time.

So we can close this issue?

Why, yes please! 😄


ModuleNotFoundError: fused_layer_norm_cuda

Following the QuickStart instruction, I load the model through PyTorch Hub.
model, alphabet = torch.hub.load("facebookresearch/esm", "esm1b_t33_650M_UR50S")

Exception experienced:
ModuleNotFoundError: No module named 'fused_layer_norm_cuda'

Double checked torch.__version__ as 1.7.0. Can provide more info if needed.

Sounds like an issue with your nvidia apex install; if it's not installed we'll fall back on default layernorm.
So you can check if you can either (a) uninstall apex or (b) any of these solutions work for you:
https://github.com/NVIDIA/apex/issues/161
https://github.com/NVIDIA/apex/issues/954


Use of @EnableScheduling in hystrix-amqp can cause problems with other TaskScheduler beans

I am working on a project that makes use of Spring's WebSocket support via @EnableWebSocketMessageBroker. Next, I decide to enable circuit breaker support by adding @EnableCircuitBreaker. When I try to run the app, it throws an exception:
[ERROR] Failed to execute goal org.springframework.boot:spring-boot-maven-plugin:1.2.5.RELEASE:run (default-cli) on project demo: An exception occured while running. null: InvocationTargetException: More than one TaskScheduler exists within the context. Remove all but one of the beans; or implement the SchedulingConfigurer interface and call ScheduledTaskRegistrar#setScheduler explicitly within the configureTasks() callback. No qualifying bean of type [org.springframework.scheduling.TaskScheduler] is defined: expected single matching bean but found 2: messageBrokerSockJsTaskScheduler,taskScheduler
The problem is that hystrix-amqp internally sets @EnableScheduling as seen here. Enabling scheduling in this way creates a taskScheduler bean, which is expected by  ScheduledAnnotationBeanPostProcessor. However, only ONE TaskScheduler is expected. This means that this situation can happen whenever there are any task scheduler beans declared, and in this case, because a messageBrokerSockJsTaskScheduler bean is also created for websocket support, we now have two beans, and the post processor throws an exception.
If I had set the @EnableScheduling annotation in my app, I could take responsibility at this point and configure the default scheduler, like the following:
@Configuration
public class TaskSchedulingConfiguration implements SchedulingConfigurer {

	@Autowired
	private TaskScheduler taskScheduler;

	@Override
	public void configureTasks(ScheduledTaskRegistrar taskRegistrar) {
		taskRegistrar.setScheduler(this.taskScheduler);
	}
}

This is in fact, how I worked around this issue in my app. The problem is that I didn't set the @EnableScheduling annotation in my app, and it's presumptuous of me to decide which TaskScheduler to set as default, since I didn't configure either one of them. While this may be a potential solution for this issue within spring-cloud, a better solution may be to not use @EnableScheduling, and thus avoid the post processor magic.
I created a demo project to illustrate this issue.
Cc: @scottfrederick @mstine @rstoyanchev

Since Spring 4.2 ScheduledAnnotationBeanPostProcessor fixes this by choosing a bean named taskScheduler if there is more than one. IMO it's nothing really directly to do with Spring Cloud anyway (since you would have the same issue essentially using Spring Integration with Spring Web Sockets).


feature: add motoko error codes for runtime errors and provide short explanation articles for them.

Messages such as “Canister trapped explicitly: could not perform self call” are not helpful enough and need a code to help web search.
Maybe we can use or adapt the infrastructure we already provide for other, static errors.

This seems like a great feature to expand upon in the VS Code extension, since we could then provide Markdown descriptions, links, and code examples for common errors and warnings.
We could also ship this documentation with the motoko npm package so web developers can easily display this error information in online environments (e.g. icblast, Blocks, Motoko Playground, Embed Motoko, etc).

@crusso It seems like these error codes should be readily available here to pull from, no?
https://github.com/dfinity/ic/blob/7919a756d3b2925a9bc5068f1cf50256c57bfbf9/rs/types/error_types/src/lib.rs#L115.
Can you expose these via Error.code() (or a different error API)?
For example, right now if I expect to receive a 512, or a CanisterInvalidController Error, Error.code() only returns #canister_error; (which could potentially be many errors).
This forces me to parse the associated text from the error with something like
``
Text.startsWith(errorMessage, #text("Only the controllers of the canister"));
In order to identify this error - but this is not nearly as resilient of a way to handle errors as using the error code.

Ideally, the `Prim.ErrorCode` variant can be expanded to include all of the different error types currently listed in the [ic repo](https://github.com/dfinity/ic/blob/7919a756d3b2925a9bc5068f1cf50256c57bfbf9/rs/types/error_types/src/lib.rs#L115)

https://github.com/dfinity/motoko/blob/c8c0917052eda95669dabd612834d66cba5a19ec/src/prelude/prim.mo#L239


feat: blue tick mark near the header is aligning according to designation length. Its not fixed

While we type our designation, if its is small the blue tick is perfect in its place
But when we have a bigger designation, the position got aligned according to the length of the designation we type
We have to make the designation in a fixed position, It will make the post more real

I have added a reference image for more clarification

Thanks for the valuable report. We have identified and fixed it.
e030a5ba7af9707196e24c67e89245488f52e007

Thank you. I have checked it works nice


callbacks doesn't work on windows

I'm writting a tiny game on c++ with glew library and use next functions:
glfwSetKeyCallback(window, keyboard);
glfwSetWindowCloseCallback(window, close);
glfwSetCursorPosCallback(window, mouse);

but the program doesn't reach the contents of functions.
I tried this code on linux Arch x86-64 and it works here. but don't on windows 7 x64 (same machine). I don't know is this a bug or I'm doing something wrong.
more code:
glfwInit();

glfwWindowHint(GLFW_CONTEXT_VERSION_MAJOR, 2);
glfwWindowHint(GLFW_CONTEXT_VERSION_MINOR, 0);

glfwWindowHint(GLFW_RESIZABLE, GLFW_FALSE);
/* for tilling window manager i3 on arch linux */
glfwWindowHint(GLFW_FLOATING, GLFW_TRUE);

GLFWwindow *window = glfwCreateWindow(640, 640, "Test", NULL, NULL);
glfwMakeContextCurrent(window);

glewInit();

/* glew and opengl stuff */

/* callbacks here */

/* current width and height of window */
static struct {
	int width = 640;
	int height = 640;
} context;

/* then i create another thread for catch events */
/* thread content */
for (;;) {
	/* sleep for 1/60 of second */
	std::this_thread::sleep_for(std::chrono::milliseconds(1000 / 60));

	/* my own flags system */
	if (flags.exit) break;

	int width, height;

	glfwGetFramebufferSize(window, &width, &height);
	glViewport(0, 0, width, height);

	if (context.width != width || context.height != height) {
		context.width = width;
		context.height = height;

		/* mutex stuff */
		updater.allow();
	}

	glfwPollEvents();
}
/* end thread content */

/* yes, i know about
	while (!glfwWindowShouldClose(window))
but it is too performance expensive for me */

/* and main loop */
for (;;) {
	/* mutex stuff */
	updater.wait();

	if (flags.exit) break;

	glClear(GL_COLOR_BUFFER_BIT);

	render();

	glfwSwapBuffers(window);
}

P.S. Sorry for my english.

glfwPollEvents must be called from main thread.

I just rewrote my code and combine main loop with content of thread function. now it works. Thank you. but i still don't understand why i can put glfwPollEvents in another thread on linux system but can't on windows. I found it weird a bit. imho.

It's the way Windows and Linux work. Nothing to do with GLFW.

Should i close the issue now?

Go for it.


Remove the UTF-8 fallback, when TextDecoder is missing, from the Content-Disposition parser

Given that TextDecoder is now supported by all modern browsers/environments, please see https://developer.mozilla.org/en-US/docs/Web/API/TextDecoder#browser_compatibility, there's no longer any good reason to keep a UTF-8 fallback in the Content-Disposition parser.

/botio unittest

Thank you for simplifying this!


Support object types in when conditions

This PR implements #74699
This change allows the contextValue property on a TreeItem to have any type, rather than just string. The value is added to the Context under the viewItem key as before. The implementation of Context.getValue has been updated to treat a.b.c as a lookup of the key a followed by accesses to the b property on a and the c property on the result of a.b.
Backward compatibility concerns
Are there existing context keys that contain ., other than for config.*, which is handled in a special IContext implementation?
Testing
I have not yet implemented any tests for the new feature. It appears that there are existing contextKey tests, but only for the expression parser and evaluator. Should I just add more tests for Context? Or should I test this some other way?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: dave-bartolomeo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I've added a test for the Context.getValue function's support for object properties.
Is there any additional work I need to do before this can be merged?

@dave-bartolomeo I would not recommend to introduce this support for all context values.
I am expecting that the tree item gives context object and the tree implementation inside VS Code creates individual context values from it.
It means that you have to still provide each key and value in your when expression while contributing the menu item.
For example, tree item can provide following context object
{
  "key1": true,
  "key2": "value1",
  "key3": false
}

And the menu item contributing to the view item is as follows
"menus": {
	[
		"view/item/context": [
				{
					"command": "nodeDependencies.editEntry",
					"when": "view == nodeDependencies && key1 && key2 == value1 && !key3",
					"group": "inline"
				}
		]
	]
}

Let me know if we are on same page

@sandy081 Thanks for taking a look. I can certainly make it work the way you suggest. However, I'm not sure I see any real semantic difference between your suggestion and mine. For example, with my implementation, I could do:
{
  key1: true,
  key2: "value",
  key3: { nested: false }
}
"when": "viewItem.key1 && viewItem.key2 == value && !viewItem.key3.nested"

With your solution, I could do the same thing by replacing . with _:
{
  viewItem_key1: true,
  viewItem_key2: "value",
  viewItem_key3_nested: false
}
"when": "viewItem_key1 && viewItem_key2 == value && !viewItem_key3_nested"

Both let me express the same rich context, but the second solution seems less natural, and inconsistent with the existing handling of config.key.nestedKey. Your suggestion also has the potential for name collisions between the properties of the context object and other keys already present from other sources (e.g. view and config). That problem could just be solved by making the new keys named viewItem.key1, viewItem.key2, etc., though.
In my particular scenario, there is actually one extension (vscode-test-explorer) that is creating the actual TreeItems based on data provided by multiple other extensions. I would be nice if one of those providers could give the extension a context object, and have the original extension expose that context object as a field of the actual viewItem context object, in addition to whatever other fields the extension itself puts directly on viewItem. I could make the extension flatten the context object with _s, but it seemed more straightforward a consistent to just handle property access notation within the "when" condition.
That said, if you don't find my above argument convincing, I'm happy to just implement it as you suggest. That design still lets me do what I need to do, even if it requires a little more code on the extension side.

I suppose I could also have the tree view contribute the individual keys after flattening the context object using . as the separator, That would make the code seem natural from the extension's point of view; it would just shift the implementation from the Context class to the tree view.

@dave-bartolomeo I did not mean about the syntax. Saying that, I liked the syntax proposal from you.
My point is to not to implement this as generic support in ContextKeyService since it is specific to Tree, let Tree View implement this by setting the context values from the context object received from the tree item.
Ref - https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/browser/parts/views/customView.ts#L858

Got it. I'll change the implementation to be in the TreeView.

any update? :D

I am wondering too. Would be a great to have api addition.

This feature would greatly simplify the implementation of my extension, which uses a TreeView for quilt patch data. The view displays hierarchies of folders (i.e. projects), patches, and files. Each type has attributes that determine which context menu items should be displayed (initialized, snapshot available, applied, top patch, patched, ...). Currently, I encode type and attribute values in a string and decode that string in the when clause using regular expressions. Because of the limitations of when clause operators (see #131819), I have to encode all boolean attributes (e.g. initialized or not_initialized, ...). All this would be much cleaner if for example I coud write something like contextValue.type == folder && contextValue.initialized.


Update self-managed-enterprise-edition.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

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

Issue(s)

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

[ ] Tested Locally
[ ] Optional Screen Shoot.

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


Fix pytorch trainer

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] I have read the CONTRIBUTING.md document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] I have verified the code in my local setup.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Fix issue #90

@hamzamaiot I have pushed the changes. Let me know if any other changes required.


Solution Code missing test suite

URL of codelab
In which task and step of the codelab can this issue be found?
Unit 3: Pathway 4: Advanced navigation app examples
Describe the problem
In step 7 there is a link to the solution code, but the JUnit test classes are missing or the link is wrong

Yeah, the link is there, but you have to change to the test_solution branch to check the file.
Is this what you are looking for?

https://github.com/google-developer-training/android-basics-kotlin-cupcake-app/blob/test_solution/app/src/test/java/com/example/cupcake/ViewModelTests.kt


Implement non-deterministc mode

Part of #653
This allows us to properly implement getrandom(),
which unlocks the default HashMap type (e.g. HashMap<K, V>)
with RandomState)
This commit adds a new '-Zmiri-seed=' option. When present,
this option takes a 64-bit hex value, which is used as the seed
to an internal PRNG. This PRNG is used to implement the 'getrandom()'
syscall.
When '-Zmiri-seed' is not passed, 'getrandom()' will be disabled.

This is pretty cool!
To make the test suite pass with your changes to the HashMap tests, you'd have to also implement this for macOS and Windows.
Windows says
error[E0080]: constant evaluation error: can't call foreign function: SystemFunction036
 --> C:/Users/appveyor/.rustup/toolchains/master/lib/rustlib/src/rust/src/libstd/sys/windows/rand.rs:8:9
  |
8 | /         c::RtlGenRandom(&mut v as *mut _ as *mut u8,
9 | |                         mem::size_of_val(&v) as c::ULONG)
  | |_________________________________________________________^ can't call foreign function: SystemFunction036
  |
  = note: inside call to `std::sys::windows::rand::hashmap_random_keys` at C:/Users/appveyor/.rustup/toolchains/master/lib/rustlib/src/rust/src/libstd/collections/hash/map.rs:3181:23

and macOS says
error[E0080]: constant evaluation error: can't call foreign function: open
   --> /Users/travis/.rustup/toolchains/master/lib/rustlib/src/rust/src/libstd/sys/unix/fs.rs:467:13
    |
467 |             open64(path.as_ptr(), flags, opts.mode as c_int)
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't call foreign function: open
    |
    = note: inside call to closure at /Users/travis/.rustup/toolchains/master/lib/rustlib/src/rust/src/libstd/sys/unix/mod.rs:143:19

Alternatively, you could make the affected tests linux-only. Then we should open an two issues to track the missing support for Windows and macOS.

I've implemented random number generation for Windows.
For now, I've left the test disabled for OS X. Implementing random number generation on OS X will be trickier, since we'll need to handle attempted reads from /dev/urandom (also, I don't have a Mac to test on).

@oli-obk @RalfJung I've made the changes you requested

@RalfJung: I've addressed your comments

just some nits then this looks ready to go to me

@oli-obk Fixed

@RalfJung Fixed

Awesome, thanks for bearing with our nitpicking :)


Why use spinlocks by default？

Hello.
When I upgrade rocketmq version 3.2.6 to 4.31,I found some configuration defaults changed
useReentrantLockWhenPutMessage=false
sendMessageThreadPoolNums=1
sendThreadPoolQueueCapacity=10000
but when I was doing a send message performance test,I found the TPS is very low and cpu is high.so I changed the configuration:
useReentrantLockWhenPutMessage=true
sendMessageThreadPoolNums=32
sendThreadPoolQueueCapacity=100000
I wonder why use this default configuration?
** The machine configuration **
8cpu and 16g memory and 300g disk

agree that enable 'useReentrantLockWhenPutMessage' and change sendMessageThreadPoolNums to "16 + 4 * core" will highly improve the throughput. And we found many "CODE: 2 DESC: [TIMEOUT_CLEAN_QUEUE]" exceptions using default configuration.

@Cicizz use spinlock can reduce context switch as you mentioned and can use fewer threads to obtain a relatively high throughput. but the issue only used to bug report or feature request,  so if you have similar questions in the later, please send to our user mailing list. (Notice: subscribe it before send)


Remove sysadmin from default sudo group

This is not only broke OS without sysadmin defined but looks like a security hole to predefine group with super power.
I understand there is an argument whether this should be on major version fix.  No matter it is on major or minor version, this will break people who assume sysadmin group exists on their system.   If people define authorizaton['sudo']['group'] = ['sysadmin'] in their node , role or environment, same privilege will appy and won't break them.
If this really need to go to major version change, at least keep this PR.  If we close this, the next major version will not catch this.   my  .02

I'm making this change in https://github.com/chef-cookbooks/sudo/pull/109 since it needs a readme update as well.


[BUG] File access is not ensured before attempting to use them in IIIF/OpenSeadragon

What steps does it take to reproduce the issue?

When does this issue occur?

This was encountered when setting a "file"-level embargo using the dgi's fork of the embargoes module; however, any other mechanism to constrain access to particular file entities would similarly cause it to happen.

Which page does it occur on?

Viewing images/paged content which make use of IIIF manifests where the current user does not have access to to the file entities referenced from the media selected for use in the displays.

What happens?

Broken viewers are shown, with 403 errors emitted to browser consoles (and presumably to Drupal's watchdog logs).

To whom does it occur (anonymous visitor, editor, administrator)?

Those without access to the referenced files.

What did you expect to happen?

Viewers would not be visible or not attempt to render elements for entities to which the user does not have access.
Which version of Islandora are you using?
2.4.0
Any related open or closed issues to this bug report?
Unknown; though, here's a number of PRs addressing things:

https://github.com/Islandora/islandora/pull/884

addressing the inclusion of access-controlled file entities in IIIF manifests generated by islandora_iiif

https://github.com/Islandora/openseadragon/pull/42

addressing the inclusion of access-controlled file entities in the one-off media formatter field (and some caching metadata on the block)

https://github.com/Islandora/openseadragon/pull/41

an optimization to better deal with caching, associated with https://github.com/Islandora/openseadragon/pull/42

Screenshots:

Anything else, @adam-vessey?

Nah, that was it... except... I guess, releases of the two things, islandora and openseadragon? Drafted... sane?:

https://github.com/Islandora/islandora/releases/tag/untagged-1e1e9e3fd185b3012538
https://github.com/Islandora/openseadragon/releases/tag/untagged-7d244167655e8480fbe0


Non-open source EULA in source-build-reference-packages

This code in microsoft.dotnet.common.itemtemplates claims it's under an non-open source license
https://github.com/dotnet/source-build-reference-packages/blob/bcdc58a2ae9c2c06d769acfb8375d27adf5b2332/src/textOnlyPackages/src/microsoft.dotnet.common.itemtemplates/1.0.2-beta3/microsoft.dotnet.common.itemtemplates.nuspec#L9
Can this be fixed? This blocks us from using this package in RHEL.

cc @leecow @dleeapho

I think we can can close this now. With https://github.com/dotnet/source-build-reference-packages/pull/215 merged, I no longer get this EULA flagged by our tooling.


ui: redesign implementation: use ionicons

This ticket outlines how to use ionicons with the project. The UI design uses ionicons for its icons. We'll need to pull in the react library.
Steps

See the ionicons usage section for how to import it into our app. You'll need to include the script tag in the index.html.
Use the icons in the UI. You'll need to look at the figma and try to see which icons were used. Then find the names of those icons.

✅ 1. See the ionicons usage section for how to import it into our app. You'll need to include the script tag in the index.html.
I have a question about the No.2 - What do you mean by "Use the icons in the UI."? Are you suggesting I style the imported icons before closing the issue or would you like me to do the styling in a separate issue? Just want to clarify and make sure. Thanks.

For now, lets just add these two icons to the UI:

let me know if that makes sense

I have already imported all the icons shown in the Figma. Here's what I'll do, I'll make a PR to showcase the import and then do another PR where they are all styled and formatted according to the Figma.


fix(slides): expose interface to provide custom animations

Short description of what this resolves:
This PR exposes the interface to allow users to tap into certain callbacks in Swiper. The result is that users can provide their own custom animations, allowing them to use fade, cube, flip, coverflow, etc, without us having to introduce the code into the framework code base.
Changes proposed in this pull request:

Ensure that users can tap into swiper callbacks but not override the callbacks Ionic uses to make ion-slides work
Added documentation on how to add custom animations
Add demos for all 4 additional animations
Fixed usage for slides

Fixes: #17424
This PR would allow #16616 to be closed as well.

Hey @liamdebeasi
I made a video! Check this bug: https://send.firefox.com/download/4eb61d0b2a643f83/#KHRL1rCut418GnvbZgheCA

Can you post the code you are using for that? Thanks!

Sorry =/
Actually this is with the current slides of v4.2. Can you guarantee that this does not happen in your PR ?!
I'll test your PR

Simple code:
<ion-slides pager="false" [options]="slideOptions" #tagSlides>
        <ion-slide *ngFor="let tag of tags">
          <ion-card class="tag-image" mode="ios" (click)="goPage(tag)">

          </ion-card>

{{tag.title}}

        </ion-slide>
      </ion-slides>

slideOptions = {
    loop: false,
    initialSlide: 0,
    slidesPerView: 1,
    centeredSlides: false,
    spaceBetween: 10,
    effect: 'coverflow',
    coverflowEffect: {
      rotate: 0,
      stretch: 0,
      depth: 250,
      modifier: 1.93,
      slideShadows: false
    }
  };

Hi there,
If you have found a bug in ion-slides in the latest stable, please open a new bug report here: https://github.com/ionic-team/ionic/issues/new?assignees=&labels=&template=bug_report.md&title=bug%3A+
This PR is to expose interfaces for custom animations and does not fix any other bugs.
Thanks!

I'm a bit confused by the usage:

Do I have to pass effect now? What's the point of passing slide if the default is slide and flip if I still have to define the effect?
Where can I find more animation examples? I know there are links to the framework tests but there are some problems with this

what if we get rid of those tests?
what if we want something that isn't listed?

I think it would be better to just show the usage for them instead of having it in a test and then link to where it came from (probably swiper docs?)

Yea for some of them there is corresponding CSS so you'll need to pass in the effect name. There's no point in passing in "slide" if it's the default.

Good question - still trying to figure out a good place for these. We provided the remaining animations that are bundled by default in Swiper, but beyond that that's on the user to come up with. For the actual code, I got it from the Swiper repo so unless we host the code somewhere, we're just going to be pointing users to some github repo.

Took a look again and seems like this was due to us always explicitly saying to use the slide effect internally. I updated it so you don't need to provide any effect value and swiper will just default to slide if the animation handlers are not overriden.

Friendly reminder, don't forget about this comment: https://github.com/ionic-team/ionic/pull/16616#issuecomment-480042124

I'm a bit late to this party, but I have seen the changes on the docs.
Has anyone looked at my suggestion to just add a swiperlib attribute so that all swiper plugins can be exposed to "advanced" users looking for them?  I think the current stream of changes to ion-slides is akin to running on a treadmill: every new feature someone wants from the Slider library won't be readily available to someone using ion-slides until another PR is pushed.
Me personally, I need the "navigation" arrows, and I have a branch ready that proposes the required changes to ion-slides to make them work. I've seen others looking for "parallax" effects, virtual slides, etc. and of course this PR addressed those looking for the flip/coverflow/fade/cube effects.
I'm not sure if it's a great developer experience to have devs copy/paste chunks of swiper code into their slides options to make them work.
If we allow devs to import their own Swiper library (custom built with the features they want) and have ion-slides consume that library per my suggestion here, then all Swiper options would be available to them with all the added Ionic benefits that come with ion-slides.
Thoughts?


Firmware repository

Hi,
awesome research. Are you planning to publish the collected firmwares that this was ran on or at least the URI's (maybe script) that can be used to scrape them? E.g: AnalysisSet, LatestSet,CamSet?
Alternatively, any tip on how to collect such a repo of fw's or a pointer to similar projects would be appreciated.
Best regards,
Attila

Hi Attila, thanks for your interest.
We already provide the dataset. Please check Readme.
To collect the dataset, we ran scraper, which is the same as Firmadyne's. Most vendors (at least the ones in our experiment) provide their firmware images through their websites, so one can download them manually or run scripts (as in our case) to crawl them.
Dongkwan


Infinite loop in assignment 9 tune method

Hello Prof. Zhe,
I'm running into an infinite loop in the tune() method.
while self.life > 0 or self.iter < self.max_generation:
    self.select()
    if self.compete(self.pf, self.pf_best):
        self.life = self.max_life
    else:
        self.life -= 1

Here, self.compete always returns True for me with the default values passed to my_GA, so self.life is never decremented. The following change in the while loop condition fixes it for me:
while self.life > 0 and self.iter < self.max_generation:

Changing or to and ensures that the loop breaks if self.life is less than 1 or if self.iter crosses the self.max_generation limit.
Please let me know if this is correct.
Thank you

That probably means your compete function is incorrect.

But yes, it should be and instead of or

Thank you Prof. Zhe. I'll recheck my compete method.


ActivityIndicator does not render in some cases

Left: Andoid 6.0.1
Right: Android 4.4.2
Both: Tabris.js developer app 1.9.0 running on stock TouchWiz
var page = new tabris.Page({
    topLevel: true
}).open();

new tabris.ActivityIndicator({
  centerX: 0,
  centerY: 0
}).appendTo(page);

Confirmed with another user running on Android >= 5.0 (not TouchWiz) that the indicator does appear so like #844 this may be specific to TouchWiz.

We couldn't reproduce the problem on a Galaxy S6. Could you provide some more details on the model?

@mpost It's a Galaxy S5

@mpost I tested this with a Nexus 5 running 6.0.1 that also does not display the ActivityIndicator.  I set the background property to red and on the 4.4.2 device I saw a red square with the activity indicator in the center of the screen.  On the Nexus 5 and the S5 both display nothing--even with the background set to red.

@cookieguru Thanks for bringing the issue up again. Is it possible to replicate your described behavior on an emulator with a certain configuration?

@mpost Testing both the Tabris.js and Tabris.js 2 developer apps from the Play Store yielded identical results using the fetch example.

On  the S5, if I set the animator duration scale to 1x, the ActivityIndicator appears.  When it is off, it does not.
On the Nexus 5, setting the animator duration scale to either 1x or off has no effect and the ActivityIndicator does not appear at all.
I haven't found an emulator config that can reproduce the error, though I had only tried the 1.x client and with the duration set to 1x.  I can't get any emulator config to start right now.

I have Tried the following snippet in the Tabris.js 2 app on a Nexus 5:
new tabris.ActivityIndicator({centerX: 0,centerY: 0})
  .appendTo(tabris.ui.contentView);

Which yields the following result:

Closing this issue for now. Please reopen of the problem can still be reproduced with the snippet above.

After a little more testing, I have concluded that the Animator duration scale is to blame.  This one can be marked as WontFix