Provide a templatized Approx

Description
We are using a rather paranoid warning level including -Werror.
Consequently using Approx on float breaks compilation:
implicit conversion increases floating-point precision: 'float' to 'double' [-Werror,-Wdouble-promotion]

Could you provide a templatized Approx supporting float and double?
Extra information

Catch version: v2.0.0-develop.3 (git hash afc0b31)
Operating System: MacOs X 10.12.3
Compiler+version: Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>)

Wouldn't a float constructor be enough?

As long as the comparison operators are also taking floats and doubles.

On 29. Mar 2017, at 21:18, Martin Hořeňovský<EMAIL_ADDRESS>wrote:
Wouldn't a float constructor be enough?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/philsquared/Catch/issues/873#issuecomment-290196705, or mute the thread https://github.com/notifications/unsubscribe-auth/AXfJqEJ-QgZ4384pWKIInz57Uv8IMwmYks5rqq6LgaJpZM4MsxGT.

Also you can just cast the float to double.

That would yield bulky, ugly code:
CHECK(static_cast<double>(variable) == Approx(5.0))

@tomvierjahn If you are using C++11, all of the Approx's operators support any type that can be converted to double, including strong typedefs.

Would you mind also including it into the single-header version?

On 4. Apr 2017, at 15:19, Martin Hořeňovský<EMAIL_ADDRESS>wrote:
Closed #873 https://github.com/philsquared/Catch/issues/873 via 0354d50 https://github.com/philsquared/Catch/commit/0354d50278d725d52084601300eb955cee6756d8.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/philsquared/Catch/issues/873#event-1028374153, or mute the thread https://github.com/notifications/unsubscribe-auth/AXfJqOf5S40rxmQzR8JymzOReOMFrLabks5rskNkgaJpZM4MsxGT.

Next release, so hopefully at some point this week.
If you want it now, you can make it using scripts/generateSingleHeader.py.


Add Global Settings with Serialization

Solves #48

The settings does not hold on its state the actual types of each setting.
Whenever possible, it will return the value. Otherwise, raises an exception.


Large batch size question

In your paper, it says that you train the CAVP model with a batch size of 720 across 8 A100 GPUS, in other words 90 video samples per GPU. I am trying to reproduce your CAVP training pipeline and am also training on A100, but am struggling to scale the batch size beyond even 12 (per GPU) due to GPU memory limits (CUDA OOM errors). Would you be able to share your data loading code, or provide any tips for increasing batch size within an A100's memory limits in this sort of framework? Thank you very much!

Code provided.


Clarify how/if null property values are supported

Hi.  One of the things we're hoping for in the OpenAPI spec is support for null types.  This is especially important for SQL where "property":"" means something different from "property":null.  Using JSON schema you indicate something can be null by using a type array as in "type":['string','null'].  What would be the equivalent in the next version of OpenAPI?

The current plan is to have null support.  See https://github.com/OAI/OpenAPI-Specification/pull/741

Thanks Darrel!


How to connect websocket?

rake service:all[start]
rake service:daemons[start]
ws://ws.ranger.app.local not working
on monitor cant finds ranger,... http://monitor.app.local/dashboard/

TKS FOR YOUR HELP

fixed ?

fixed?


Recaptcha hide

How is it possible to show the Recaptcha logo only on a specific page(like auth) and hide or make it not work for others?
Thanks!

I implemented methods (hideGoogleReCaptcha, showGoogleReCaptcha).
Try it out! new README.md

I looked for a way to remove it from memory but could not find one. So I implemented a hide option like other libraries. If you want to hide completely and use it, please check the notes in the official documentation.
Thank you😀

Thanks :)


Set up CI.

We should get some continuous integration going here. I think Travis is the most popular but the Serilog guys are using AppVeyor which seems pretty nice for use with Windows. I think we want one that can integrate with Nuget publishing, but I'm sure both do.

Done.


Stabilize dask loading of Ocean Model data

This PR is a clone or mirror of #774 , which suffers from a broken CI.
The lazy evaluation from Dask is still lacking robustness and some users still experience errors in some setups, which this PR attempts to resolve.
Test are developed for NEMO, GlobCurrent and POP.

The code runs all tests fine, but the test_palaeo_dataset.py still fails.


Request to move a file from a blob to another blob in Azure Storage Explorer

We should able to cut a file and paste file to another blob.
Instead I see the clone or copy the files that mean it doesn't provide a file to move to a another blob container.

wrong place


Reveal in Explorer View and Reveal in Finder options both show up on Mac

Testing #150716
Even with extensions disabled, there's still reveal in explorer view on Mac:

Surely they do two different things, don't they?

True. I confused it with the "Reveal in File Explorer" from the TPI.


Npm update broken scully

🐞 Bug report
After updating to the latest npm 7.15.1 and to Angular 12.0.2, scully no longer builds.
Puppeteer cannot find or launch the browser. (by default chrome)
Try adding 'puppeteerLaunchOptions: {executablePath: CHROMIUM_PATH}'
to your scully.*.config.ts file.
Also, this might happen because the default timeout (60 seconds) is to short on this system
this can be fixed by adding the --serverTimeout=x cmd line option.
(where x = the new timeout in milliseconds)
When this happens in CI/CD you can find some additional information here:
https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md
Description
🔬 Minimal Reproduction
💻Your Environment

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 12.0.2
Node: 14.15.5
Package Manager: npm 7.15.1
OS: darwin x64

Angular: 12.0.0
... animations, common, compiler, compiler-cli, core, forms
... language-service, material, material-moment-adapter
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1200.2
@angular-devkit/build-angular   12.0.2
@angular-devkit/core            12.0.2
@angular-devkit/schematics      12.0.2
@angular/cdk                    12.0.2
@angular/cli                    12.0.2
@schematics/angular             12.0.2
rxjs                            6.6.7
typescript                      4.2.4

Scully Version:

{
  "name": "stryker",
  "version": "0.0.0",
  "scripts": {
    "prebuild": "node updateRoute.js",
    "ng": "ng",
    "start": "ng serve",
    "start:local": "npx netlify-cms-proxy-server",
    "build": "ng build",
    "build:dev": "ng build --configuration development",
    "test": "ng test",
    "scully": "npx scully --",
    "scully:watch": "npx scully -- --watch",
    "scully:serve": "npm run scully && npx scully serve --",
    "netlify": "npm run build && npm run scully"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "12.0.0",
    "@angular/cdk": "^12.0.2",
    "@angular/common": "12.0.0",
    "@angular/compiler": "12.0.0",
    "@angular/core": "12.0.0",
    "@angular/forms": "12.0.0",
    "@angular/material": "12.0.0",
    "@angular/material-moment-adapter": "12.0.0",
    "@angular/platform-browser": "12.0.0",
    "@angular/platform-browser-dynamic": "12.0.0",
    "@angular/router": "12.0.0",
    "@auth0/auth0-spa-js": "^1.15.0",
    "@fortawesome/angular-fontawesome": "^0.9.0",
    "@fortawesome/fontawesome-svg-core": "^1.2.35",
    "@fortawesome/free-brands-svg-icons": "^5.15.3",
    "@fortawesome/free-solid-svg-icons": "^5.15.3",
    "@fullcalendar/angular": "4.4.5-beta",
    "@fullcalendar/core": "4.4.2",
    "@fullcalendar/daygrid": "4.4.2",
    "@fullcalendar/interaction": "4.4.2",
    "@fullcalendar/list": "4.4.2",
    "@fullcalendar/moment": "4.4.2",
    "@fullcalendar/rrule": "4.4.2",
    "@fullcalendar/timegrid": "4.4.2",
    "@scullyio/init": "^1.1.4",
    "@scullyio/ng-lib": "^1.0.0",
    "@scullyio/scully": "^1.0.0",
    "apexcharts": "3.26.2",
    "crypto-js": "3.3.0",
    "highlight.js": "10.7.2",
    "lodash-es": "4.17.21",
    "moment": "2.29.1",
    "ng-apexcharts": "1.5.9",
    "ngx-markdown": "11.1.3",
    "ngx-quill": "14.0.0",
    "ngx-sharebuttons": "^8.0.5",
    "perfect-scrollbar": "1.5.1",
    "quill": "1.3.7",
    "rrule": "2.6.8",
    "rxjs": "6.6.7",
    "swiper": "^6.6.2",
    "tslib": "2.2.0",
    "web-animations-js": "2.3.2",
    "zone.js": "0.11.4"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "12.0.2",
    "@angular-eslint/builder": "12.0.0",
    "@angular-eslint/eslint-plugin": "12.0.0",
    "@angular-eslint/eslint-plugin-template": "12.0.0",
    "@angular-eslint/schematics": "12.1.0",
    "@angular-eslint/template-parser": "12.0.0",
    "@angular/cli": "12.0.2",
    "@angular/compiler-cli": "12.0.0",
    "@angular/language-service": "12.0.0",
    "@scullyio/scully-plugin-flash-prevention": "^1.1.1",
    "@tailwindcss/aspect-ratio": "0.2.0",
    "@tailwindcss/line-clamp": "0.2.0",
    "@tailwindcss/typography": "0.4.0",
    "@types/chroma-js": "2.1.3",
    "@types/crypto-js": "3.1.47",
<EMAIL_ADDRESS>"10.1.0",
    "@types/jasmine": "3.6.11",
    "@types/lodash": "4.14.169",
    "@types/lodash-es": "4.17.4",
    "@types/node": "12.20.13",
    "@typescript-eslint/eslint-plugin": "4.23.0",
    "@typescript-eslint/parser": "4.23.0",
    "autoprefixer": "10.2.5",
    "chroma-js": "2.1.2",
    "eslint": "7.26.0",
    "eslint-plugin-import": "2.23.0",
    "eslint-plugin-jsdoc": "34.2.2",
    "eslint-plugin-prefer-arrow": "1.2.3",
    "fs": "0.0.1-security",
    "jasmine-core": "3.7.1",
    "jasmine-spec-reporter": "5.0.2",
    "karma": "6.3.2",
    "karma-chrome-launcher": "3.1.0",
    "karma-coverage": "2.0.3",
    "karma-jasmine": "4.0.1",
    "karma-jasmine-html-reporter": "1.6.0",
    "lodash": "4.17.21",
    "postcss": "8.2.15",
    "protractor": "7.0.0",
    "scully-plugin-flash-prevention": "^0.0.41",
    "tailwindcss": "2.1.2",
    "typescript": "4.2.4"
  }
}

🔥 Exception or Error

=================================================================================================
Puppeteer cannot find or launch the browser. (by default chrome)
 Try adding 'puppeteerLaunchOptions: {executablePath: CHROMIUM_PATH}'
 to your scully.*.config.ts file.
Also, this might happen because the default timeout (60 seconds) is to short on this system
this can be fixed by adding the --serverTimeout=x cmd line option.
   (where x = the new timeout in milliseconds)
When this happens in CI/CD you can find some additional information here:
https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md
=================================================================================================

NPM 7 is incompatible with Angular right now.  There is not much we can change about that.

According to https://github.com/angular/angular-cli/issues/19957#issuecomment-786177315 it is compatible now?


"end-user" based routing not working with istio-1.0.0 for bookinfo application

Describe the bug
Configured user-based routing as described in the Istio docs (https://istio.io/docs/tasks/traffic-management/request-routing/). The star ratings do not appear for use jason.
Expected behavior
On the /productpage of the Bookinfo app, log in as user jason. Refresh the browser. What do you see? The star ratings appear next to each review.
Steps to reproduce the bug
Steps to reproduce the behavior.
Follow the instructions exactly as described in the Istio documentation
-- Setup Istio by following the instructions in the Installation guide.
-- Deploy the Bookinfo sample application.
-- Configure request routing
Version
What version of istio and Kubernetes are you using? Use istioctl version and kubectl version
ramki@ramki-virtual-machine:~/istio-1.0.0$ istioctl version
Version: 1.0.0
GitRevision: 3a136c90ec5e308f236e0d7ebb5c4c5e405217f4
User: root@71a9470ea93c
Hub: gcr.io/istio-release
GolangVersion: go1.10.1
BuildStatus: Clean
ramki@ramki-virtual-machine:~/istio-1.0.0$ kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.4", GitCommit:"5ca598b4ba5abb89bb773071ce452e33fb66339d", GitTreeState:"clean", BuildDate:"2018-06-06T08:13:03Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.6", GitCommit:"a21fdbd78dde8f5447f5f6c331f7eb6f80bd684e", GitTreeState:"clean", BuildDate:"2018-07-26T10:04:08Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Is Istio Auth enabled or not?
Did you install the stable istio.yaml, istio-auth.yaml.... or if using the Helm chart please provide full command line input.
Istio Auth is not enabled. Using stable istio.yaml as per the instructions.
Environment
Which environment, cloud vendor, OS, etc are you using?
-- Ubuntu 16.04 OS on VMware vSphere VM in a private cloud setup.
-- The entire k8S cluster is on a single VM.
-- K8S cluster uses flannel for networking.
-- Used Kubeadm for bringing up the kubernetes cluster.
Cluster state
If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue.
Thanks in advance for all the help.
istio-dump.tar.gz

I couldn't find the default Bookinfo DestinationRule in your istio dump.
Please make sure you apply it.

Many thanks for the pointer. The issue is fixed.


Correct exit code 126 and 127

The exit codes for 126 and 127 were reversed.  For the record, the exit
codes used are as follows:

125 if ‘chroot’ itself fails
126 if COMMAND is found but cannot be invoked
127 if COMMAND cannot be found

This resolves issue #367
Signed-off-by: baude<EMAIL_ADDRESS>

LGTM

@rh-atomic-bot r+


Requirements

It seems logical that either the Path Attribute, or InProcessServerDll child element would be required.
Similarly, it would be logical that either the Class or ClassReference element would be required as well.
I'm not sure that the schema specifies this, but it seems logical that you have to have a path and CLSID GUID defined at a minimum.

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

ID: 5b9899c3-e4cf-9f45-5a6c-d1f5d42911a3
Version Independent ID: 5b9899c3-e4cf-9f45-5a6c-d1f5d42911a3
Content: com4:Class (in InProcessServer) - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-com4-inprocessserver-class.md
Product: uwp
Technology: winrt-reference
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

@TimMangan I assume you are talking about InProcessServer here. Just to make sure I understand, are you suggesting that we call out that this element doesn't make any sense if you don't reference an actual dll? First, I think it's important that we keep the definition of "required" to what is enforced by the schema - or in some cases the OS code that processes the schema. It may not make sense to have an element with no contents, but I'm not sure that's something we need to call out in the docs. I think the right thing to do is to assume that developers using this extension are doing so with a purpose in mind. I'm going to close this one as "won't fix". Maybe I've misunderstood what you are suggesting. As always feel free to continue the discussion in the comments.


Go to summary link

On test.
Can you change the positioning of the 'summary link'.
I've added in two 's above it and removed the css class 'summary-link'
I've also renamed it 'Go to claim summary'.
This content will need tested as well.
It should look like this

This should also be done for any 'back' buttons

verified


Shared metadata index

This PR pulls the in memory index out into something that can be shared across shards. It keeps the information for field encoding local to each shard.

I took a cursory look and it lgtm.

Is all or most of the code going to move into package tsdb?

@dgnorton all of the query processing logic, database.go, shard.go, engine.go, functions.go, and tx.go will probably be getting moved there.


Istanbul default excludes not working

According to the istanbul docs:

The default exclude list is defined in the test-exclude module.

The defaults contain test{,-*}.{js,cjs,mjs,ts} and **/*{.,-}test.{js,cjs,mjs,ts}, so I don't know why such files are being coverage checked in my projects:

The workaround is to add a .nycrc.json file to your project like this:
{
  "exclude": ["**/*.test.*"]
}

The docs on github show master, unfortunately right now this is different from the currently released version.  The default excludes for the current version of nyc can be viewed at https://github.com/istanbuljs/istanbuljs/blob/test-exclude%405.2.3/packages/test-exclude/index.js#L176-L184 (test-exclude@5.2.3).
The additional default excludes will be provided as part of<EMAIL_ADDRESS>and<EMAIL_ADDRESS> No ETA on the full release of these new versions, progress can be followed at istanbuljs/nyc#1104.

As of tap 14.7.2, it'll also explicitly exclude any test files it runs.  Should fix this issue for you.


Remove possible warnings with redefinitions

If the library is used as part of a bigger project some definitions can
be provided via external build files.

So the issue you're trying to solve here is that you're already defining WIN32_LEAN_AND_MEAN on the command-line in your build system which is compiling cppcoro sources directly rather than using the output of the cake build system?
There are a couple of new files in the sockets2 branch that may require compilation without WIN32_LEAN_AND_MEAN defined. I'll try out this change with those branches but otherwise it seems like a reasonable change to me.

So the issue you're trying to solve here is that you're already defining WIN32_LEAN_AND_MEAN on the command-line in your build system which is compiling cppcoro sources directly rather than using the output of the cake build system?

Yes, exactly.


Autocomplete lists Qiskit modules twice

Information

Qiskit Terra version: master
Python version:
Operating system:

What is the current behavior?
This only occurs for Qiskit modules:

Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

I assume jupyter (well actually more likely ipython) is using jedi for tab completion. Which version of jedi do you have installed? Although there hasn't been a recent release of jedi in a while so it might not be that.
I'm thinking this might be an issue with jupyter. At least for me locally with jedi-vim and ipython I'm not able to reproduce this:

but I see the same behavior in jupyter as you are.

Yeah I have the latest Jedi. There is a definite level of compatibility with jedi. Ipython great, notebooks good, lab iffy.

This seems to be specifically when tab completing modules in import statements, such as from qiskit.<tab> - I can't reproduce with qiskit.compiler.<tab>, and similarly to Matthew, it only happens in Jupyter.

Actually, it's just a Jupyter Notebook issue as best as I can tell - I made a simple package with no content at all other than submodules called a and b, and was able to reproduce the duplication, but only in jupyter notebook and not in jupyter lab.  So this is an old notebook issue, and since Jupyter are generally moving towards deprecation of that in favour of JupyterLab, I'll close this.


left sidebar: CSS cleanup

Get better control of the order and nesting levels of left_sidebar.css. It’s a tangle, and makes working in the file unnecessarily tricky. Additionally, 4+ levels of nesting make for overly powerful selectors, necessitating undesirable uses of !important.

SHARP EDGES: It’s possible that in flattening the complexity of selectors, unwanted styles will be inherited from other parts of the CSS codebase. Need to keep a careful, pre-emptive watch on that in the browser inspector while working—and take plenty of screenshots to protect against regressions as this work proceeds.

Please assign this issue to me i will love to contribute

@zulipbot claim


Fix/unify people searches.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @showell!


Update ramsey/uuid's version constraints

Currently users that using ramsey/uuid version 4 can only install ramsey/uuid:4.0.0.
 I can't believe I missed that. Thanks.


[PIO-69] Update LICENSE.txt and NOTICE.txt for binary distribution

I added sbt-license-report plugin. It's possible to generate licences-notice.csv which contains only non-ASL libraries by following script.
sbt/sbt clean
sbt/sbt dumpLicenseReport

sbt/sbt storage/clean
sbt/sbt storage/dumpLicenseReport

find . -name "*-licenses.csv" -exec cat {} >> licences-concat.csv \;
cat licences-concat.csv | sort | uniq | grep -v Apache | grep -v ASL | grep -v "org.apache" | grep -v "commons-" | grep -v predictionio > licences-notice.csv

The result in the latest develop branch is:
BSD,BSD (LICENSE.txt),jline # jline # 0.9.94,
BSD,BSD (http://software.clapper.org/grizzled-slf4j/license.html),org.clapper # grizzled-slf4j_2.10 # 1.0.2,
BSD,BSD (http://www.jcraft.com/jsch/LICENSE.txt),com.jcraft # jsch # 0.1.42,
BSD,BSD-like (http://www.scala-lang.org/downloads/license.html          ),org.scala-lang # scala-library # 2.10.6,
BSD,BSD-like (http://www.scala-lang.org/downloads/license.html    ),org.scala-lang # scala-compiler # 2.10.6,
BSD,BSD-like (http://www.scala-lang.org/downloads/license.html    ),org.scala-lang # scala-reflect # 2.10.6,
BSD,BSD-like (http://www.scala-lang.org/downloads/license.html    ),org.scala-lang # scalap # 2.10.6,
BSD,New BSD License (http://www.opensource.org/licenses/bsd-license.php),com.esotericsoftware # kryo # 3.0.3,
BSD,New BSD License (http://www.opensource.org/licenses/bsd-license.php),com.esotericsoftware # minlog # 1.3.0,
BSD,New BSD License (http://www.opensource.org/licenses/bsd-license.php),com.esotericsoftware # reflectasm # 1.10.1,
BSD,New BSD License (http://www.opensource.org/licenses/bsd-license.php),com.esotericsoftware.kryo # kryo # 2.21,
BSD,New BSD License (http://www.opensource.org/licenses/bsd-license.php),com.esotericsoftware.minlog # minlog # 1.2,
BSD,New BSD License (http://www.opensource.org/licenses/bsd-license.php),com.esotericsoftware.reflectasm # reflectasm # 1.07,
BSD,New BSD license (http://www.opensource.org/licenses/bsd-license.php),com.google.protobuf # protobuf-java # 2.5.0,
BSD,New BSD license (http://www.opensource.org/licenses/bsd-license.php),com.google.protobuf # protobuf-java # 2.6.1,
BSD,The BSD License (http://www.opensource.org/licenses/bsd-license.php),xmlenc # xmlenc # 0.52,
CDDL,COMMON DEVELOPMENT AND DISTRIBUTION LICENSE (CDDL) Version 1.0 (http://www.sun.com/cddl/cddl.html),javax.xml.stream # stax-api # 1.0-2,
CDDL,Common Development and Distribution License (CDDL) v1.0 (https://glassfish.dev.java.net/public/CDDLv1.0.html),javax.activation # activation # 1.1,
Category,License,Dependency,Notes
Common Public License,Common Public License Version 1.0 (http://www.opensource.org/licenses/cpl1.0.txt),junit # junit # 4.11,
GPL,GPL2 w/ CPE (http://glassfish.java.net/public/CDDL+GPL_1_1.html),com.sun.jersey # jersey-core # 1.8,
GPL,GPL2 w/ CPE (http://glassfish.java.net/public/CDDL+GPL_1_1.html),com.sun.jersey # jersey-json # 1.8,
GPL,GPL2 w/ CPE (http://glassfish.java.net/public/CDDL+GPL_1_1.html),com.sun.jersey # jersey-server # 1.8,
GPL,GPL2 w/ CPE (https://glassfish.dev.java.net/public/CDDL+GPL_1_1.html),javax.xml.bind # jaxb-api # 2.2.2,
GPL,GPL2 w/ CPE (https://glassfish.java.net/public/CDDL+GPL_1_1.html),com.sun.xml.bind # jaxb-impl # 2.2.3-1,
GPL,GPL2 w/ CPE (https://glassfish.java.net/public/CDDL+GPL_1_1.html),org.jvnet.mimepull # mimepull # 1.9.5,
MIT,MIT (https://github.com/softprops/semverfi/blob/0.1.3/LICENSE),me.lessis # semverfi_2.10 # 0.1.3,
MIT,MIT License (http://www.opensource.org/licenses/mit-license.php),com.github.scopt # scopt_2.10 # 3.5.0,
MIT,MIT License (http://www.opensource.org/licenses/mit-license.php),org.slf4j # slf4j-api # 1.7.14,
MIT,MIT License (http://www.opensource.org/licenses/mit-license.php),org.slf4j # slf4j-log4j12 # 1.7.18,
MIT,MIT License (http://www.opensource.org/licenses/mit-license.php),org.slf4j # slf4j-log4j12 # 1.7.5,
Mozilla,Mozilla Public License Version 1.1 (http://www.mozilla.org/MPL/MPL-1.1.txt),org.jamon # jamon-runtime # 2.3.1,
Public Domain,Public Domain (http://creativecommons.org/licenses/publicdomain),com.github.stephenc.high-scale-lib # high-scale-lib # 1.1.1,
Public Domain,Public Domain,org.tukaani # xz # 1.0,
unrecognized,none specified (none specified),asm # asm # 3.1,
unrecognized,none specified (none specified),com.fasterxml.jackson.core # jackson-annotations # 2.6.0,
unrecognized,none specified (none specified),com.fasterxml.jackson.core # jackson-core # 2.6.6,
unrecognized,none specified (none specified),com.fasterxml.jackson.core # jackson-databind # 2.6.6,
unrecognized,none specified (none specified),com.fasterxml.jackson.dataformat # jackson-dataformat-cbor # 2.6.6,
unrecognized,none specified (none specified),com.google.guava # guava # 11.0.2,
unrecognized,none specified (none specified),com.sun.jersey # jersey-core # 1.9,
unrecognized,none specified (none specified),com.sun.jersey # jersey-json # 1.9,
unrecognized,none specified (none specified),com.sun.jersey # jersey-server # 1.9,
unrecognized,none specified (none specified),com.thoughtworks.paranamer # paranamer # 2.3,
unrecognized,none specified (none specified),com.thoughtworks.paranamer # paranamer # 2.6,
unrecognized,none specified (none specified),javax.servlet # servlet-api # 2.5,
unrecognized,none specified (none specified),javax.servlet.jsp # jsp-api # 2.1,
unrecognized,none specified (none specified),org.codehaus.jettison # jettison # 1.1,
unrecognized,none specified (none specified),org.hamcrest # hamcrest-core # 1.3,
unrecognized,none specified (none specified),org.mortbay.jetty # jetty # 6.1.26,
unrecognized,none specified (none specified),org.mortbay.jetty # jetty-util # 6.1.26,
unrecognized,none specified (none specified),org.objenesis # objenesis # 2.1,
unrecognized,none specified (none specified),org.ow2.asm # asm # 5.0.3,
unrecognized,none specified (none specified),org.slf4j # slf4j-api # 1.7.18,
unrecognized,none specified (none specified),org.slf4j # slf4j-api # 1.7.2,
unrecognized,none specified (none specified),org.slf4j # slf4j-api # 1.7.5,
unrecognized,none specified (none specified),org.slf4j # slf4j-api # 1.7.7,
unrecognized,none specified (none specified),tomcat # jasper-compiler # 5.5.23,
unrecognized,none specified (none specified),tomcat # jasper-runtime # 5.5.23,
unrecognized,none specified (none specified),xml-apis # xml-apis # 1.3.04,

I checked the license of all libraries in this report and all of them can be included in the binary distribution. Also I added their license text to LICENSE.txt and updated make-distribution.sh to include LICENSE.txt and NOTICE.txt in the binary distribution.

I added license information for the binary distribution to LICENSE.txt and NOTICE.txt, but rather it might be nice to create separate files for the binary distribution.

I checked the license of all non-ASL libraries and all them can be included as binary form in the distribution. I added their license text to LICENSE.txt and updated make-distribution.sh to include LICENSE.txt and NOTICE.txt in the binary distribution.
I think all tasks about 3rd party license is done. Let me know if I have to do something else.

@takezoe Excellent work. Thanks! 🎉
I think the only concern left is to make sure whenever we add a new dependency that does not have clear indication of license compatibility, it will be caught automatically. We probably need to write a script and put in an exclusion file for those that we have manually verified, and put this check as part of our automated tests.

@dszeto

We probably need to write a script and put in an exclusion file for those that we have manually verified, and put this check as part of our automated tests.

OK, I will include them in this pull request.

@dszeto I added tests/check_libraries.sh to check library dependencies and it's also available on the Travis build. It would be failed when there are any changes in library dependencies. Then we can check the report and update LICENSE.txt.


"Frecăței" is listed in the drop-down menu

Description:
"Frecăței" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Frecăței" in the search bar.

Expected results
The location is listed in the drop-down menu as "Frecăței, Brăila (Frecăței)".


The Size in FrameHeader is the number of remaining bytes in that Frame

The comment in the definition is incorrect:
// The size is sizeof(FrameHeader) + size of the content of the frame
should be just:
// The size is size of the content of the frame

Thanks for reporting this. I've updated the template file.


get_queryset() is not called

I'm trying to add a .get_queryset() class method to one of my types, but it is never called when querying. I just put a print on top of the method and it never happens. Fields on the type behave normally so I know the type is in use.
I can't get even the simple example from the docs to work:
@strawberry_django.type(models.Fruit)
class Berry:

    @classmethod
    def get_queryset(cls, queryset, info, **kwargs):
        return queryset.filter(name__contains="berry")

What could be going on here?
I'm on version 0.28.2.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Hi @alimony ,
Can you provide a MRE or at least give more information about how you are using using that type? Because there are some differences between using it directly on a root type (e.g. Query) or nested inside another type.

It looks something like this:
@strawberry_django.type(
    Customer,
    filters=CustomerFilter,
    fields=[
        "name",
        ...
    ],
)
class CustomerType:
    name: auto
    org: OrgType

    @classmethod
    def get_queryset(
        cls, queryset: QuerySet[Customer], info: Info, **kwargs
    ) -> QuerySet[Customer]:
        return queryset.filter(...)

(Edited for NDA reasons.)
I can make a proper test case next week if needed.

@alimony I mean, some examples of where that CustomerType is used.
For example, is is used in a root query like this?
@strawberry.type
class Query:
    customers: list[CustomerType] = strawberry_django.field()

Or in a nested django type:
@strawberry_type.type(SomeModel)
class SomeModelType:
    customers: list[CustomerType] = strawberry_django.field()

    # or like this
    @strawberry_django.field
    def customers(self) -> list[CustomerType]:
        ...

It's at the root, if I understand your question correctly. At the top level I have something like this:
@strawberry.type
class Query:
    customers: CustomerType | None = strawberry_django.field(extensions=[auth_required])

...

schema = strawberry.Schema(
    query=Query,
    mutation=Mutation,
    extensions=schema_extensions,
)

and then this:
@strawberry_django.type(
    Customer,
    filters=CustomerFilter,
    fields=[
        "name",
        ...
    ],
)
class CustomerType:
    name: auto
    org: OrgType
    ...

    @classmethod
    def get_queryset(cls, queryset: QuerySet[Customer], info: Info, **kwargs) -> QuerySet[Customer]:
        print('calling custom get_queryset')  # this never gets called
        return queryset.filter(...)

Hey @alimony ,
I'm trying to replicate the issue in here but it is working correctly on my tests.
Take a look at this test for example: https://github.com/strawberry-graphql/strawberry-django/commit/7b5ee33c9f1c9692611ac7c07cb1d8156f74e13c#diff-723b95803cbfa95b72fb2bc64e3000c86c7cf4920592b4dec6e15cc2ae08e821R167 . I defined it just like your example and get_queryset is being called (the test passes ok)
The only major difference I can see is that you have extensions=[auth_required] defined. Is it doing something to the field that could cause its behaviour to change?

@bellini666 How can I reopen this issue? I'm running into this again, but this time using a custom field class. For example, something like:
class CustomField(StrawberryDjangoField):
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        print("init")

    def get_queryset(self, *args, **kwargs):
        queryset = super().get_queryset(*args, **kwargs)
        print("get_queryset")
        return queryset

@strawberry_django.type(
    MyClass,
    field_cls=CustomField,
    ...
)
class MyClassType:
    ...

My output shows that .__init__() is called, but .get_queryset() is never called, when using this type in any query.

Hi @alimony ,
Indeed, the filter might now have its get_queryset called, depending on how it is solving itself. Please take a look at get_result to check how it works.
Either way, since this issue is related to a type's get_queryset, would you mind opening another issue to register yours? It would be nice if you could also add a reproduction example together with it


broken links in COC

hey,
just to inform the link https://adainitiative.org/2014/02/howto-design-a-code-of-conduct-for-your-community/ no longer exists or has moved somewhere else
Update seems like hoodie link is also not available anymore: http://hood.ie/code-of-conduct/

Thanks!
We probably should do a bit of research on newer texts.
@contributors  Meanwhile we could link to

Ada initiative: https://web.archive.org/web/20150701085442/https://adainitiative.org/2014/02/howto-design-a-code-of-conduct-for-your-community/
Hoodie: https://web.archive.org/web/20200827104710/http://hood.ie/code-of-conduct/


[TODO] Wind effects for cloth simulation

The wind effect used in [Bergou+06] looks reasonable:

We modeled wind by a constant homogeneous velocity field, with force proportional to the projection of the wind velocity onto the area-weighted surface normal.

https://dl.acm.org/doi/10.5555/1281957.1281987

This paper also looks nice.
Modelling Effects of Wind Fields in Cloth Animations
http://wscg.zcu.cz/wscg2004/Papers_2004_Full/C73.pdf
A similar discussion can be found in the following paper as well.
Pteromys: Interactive Design and Optimization of Free-formed Free-flight Model Airplanes
http://www.nobuyuki-umetani.com/publication/2014_sigg_pteromys/2014_siggraph_GliderDesign.html

According to the above aerodynamics papers, it seems reasonable to model air forces to be proportional to the surface triangle area and proportional to the squire of the relative velocity.

This one-page abstract provides a simple wind model and it looks reasonable to me.
Simulating wind effects on cloth and hair in Disney's Frozen [Wilson+14]
https://dl.acm.org/doi/10.1145/2614106.2614120


?module should not 404

Hi there - I'm trying to load @tensorflow/tfjs as a module from unpkg. Adding the ?module flag correctly transpiles all of the import requests for all dependencies except for the package seedrandom. Visiting https://unpkg.com/seedrandom@2.4.4?module gets back a 404, which causes the whole import to fail. I feel like in this case ?module should just fall through to the un-transpiled version, correct?
I opened https://github.com/tensorflow/tfjs/pull/4087 to bump the version, but I have to imagine this impacts other places too.

As far as I know, unpkg does not do any transpiling besides altering the URLs of modules imported by an ES Module script. In this case, the seedrandom package does not provide an ES Module build (see https://github.com/davidbau/seedrandom/issues/72) so there is no way to fix this without a new version of that package.

I guess I'm wondering why adding ?module should 404.
Shouldn't the default behavior be to serve the index package?

That wouldn’t work. ?module will either output an ES Module file that can be used by native ES Module implementations, or 404. If it were to serve a CommonJS/AMD/etc module, it would result in clients getting an empty exports object, which is not useful.

Good point - I guess the only fix is to make all dependencies esm-aware, which kinda sucks.
I guess as a fallback this could rely on the global Math.seedrandom instead if the module fails, but it's probably not worth the hassle. Seems like the platform level needs to mature a bit more.


grpcui: 0.2.0 -> 0.2.1

Motivation for this change
Upstream update
Things done

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

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

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

Notify maintainers
cc @pradyuman

@GrahamcOfBorg build grpcui

ofborg is reporting a different hash for the go modules.

That's an instance of https://github.com/NixOS/nixpkgs/pull/68135#issuecomment-532889690


chore: update SCC profile_id to fe96bd4d-9b37-40f2-b39f-a62760e326a3

/run pipeline

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


Dockerode not opening up two ports

When I attempt to have a second port in the create options, it correctly creates the first, but not the second port.
{
"createOptions": {
"Env": [
"DATA_DIR=/tmp/localstack/data",
"DOCKER_HOST=unix:///var/run/docker.sock",
"SERVICES=apigateway"
],
"Image": "localstack/localstack:0.12.10",
"name": "commandeer-localstack-default-local",
"ExposedPorts": {
"443/tcp:": {}
},
"HostConfig": {
"PortBindings": {
"4566/tcp": [
{
"HostPort": "4566"
}
],
"443/tcp": [
{
"HostPort": "443"
}
]
},
"AutoRemove": true,
"Binds": [
"/var/run/docker.sock:/var/run/docker.sock"
]
}
}
}
This should expose 4566 and 443, but when I inspect the newly created container, it has this info.
"NetworkSettings": {
"Bridge": "",
"SandboxID": "f55fe8aad382e3fb418c419bdee6ad52c2540c160b93d0f7164a5cd8088ea00a",
"HairpinMode": false,
"LinkLocalIPv6Address": "",
"LinkLocalIPv6PrefixLen": 0,
"Ports": {
"443/0": null,
"4566/tcp": [
{
"HostIp": "<IP_ADDRESS>",
"HostPort": "4566"
}
],
"4571/tcp": null,
"8080/tcp": null
},
You can see that 4566 is setup correctly, but 443/0 : null for the 443 port.  I have tried this without the ExposedPorts section, changed the order of the HostOptions, etc. But it still never works.  Any ideas?

.

Notice the typo - : inside "443/tcp:". Should be:
"ExposedPorts": {
  "443/tcp": {}
}


Feature Request: sort items on character sheet alphabetical

There should be a possibility to reorder the items on the character sheet alphabetically, maybe with a warning dialog to not unintentionally by misclicking destroy carefully curated ordering by draging and dropping, but there should be an option.

Implemented in 3.0. Still missing on all other Sheets, but hopefully this will be added as the new versions come.


Emptying the skip list of .ansible-lint.

Crypto_policy role is free from ansible-lint errors.

Can you also fix the 208 error in tests/tests_include_vars_from_parent.yml ?

Can you also fix the 208 error in tests/tests_include_vars_from_parent.yml ?

Sorry, I forgot about it... :( Adding it.

@Jakuje do we need to do something else to enable rhel9 support? - from rhel-x/ansible-2.9: https://fedorapeople.org/groups/linuxsystemroles/logs/linux-system-roles-crypto_policies-pull-linux-system-roles_crypto_policies-18-52a355c-rhel-x-20210326-222714/artifacts/ansible.log
TASK [Check the current policy is DEFAULT and common variables] ****************
task path: /tmp/tmp5te0ydyh/tests/tests_default.yml:13
fatal: [/cache/rhel-x.qcow2]: FAILED! => {
    "assertion": "'NO-CAMELLIA' in crypto_policies_available_modules",
    "changed": false,
    "evaluated_to": false,
    "msg": "Assertion failed"
}

@Jakuje do we need to do something else to enable rhel9 support? - from rhel-x/ansible-2.9: https://fedorapeople.org/groups/linuxsystemroles/logs/linux-system-roles-crypto_policies-pull-linux-system-roles_crypto_policies-18-52a355c-rhel-x-20210326-222714/artifacts/ansible.log
TASK [Check the current policy is DEFAULT and common variables] ****************
task path: /tmp/tmp5te0ydyh/tests/tests_default.yml:13
fatal: [/cache/rhel-x.qcow2]: FAILED! => {
    "assertion": "'NO-CAMELLIA' in crypto_policies_available_modules",
    "changed": false,
    "evaluated_to": false,
    "msg": "Assertion failed"
}

There are changes in the RHEL9 policies. We will need a different example here for RHEL9. Lets bring it in the new PR.


Filedrop: Failed to resolve relay addresses. Failed to succefully (sic) dial any bootstrap node

The File Drop app has yet to work for me -- joining a space either never loads, or the space shows as empty despite the seeded files showing on the other device.
Test on Firefox and Chrome on Windows, as well as Firefox and Chrome on Android. Enterprise network.
aquire id at 0 index-JSApnn92.js:150:95174
Create client index-JSApnn92.js:180:4839
Ignoring inability to install OPFS sqlite3_vfs: Cannot install OPFS: Missing SharedArrayBuffer and/or Atomics. The server must emit the COOP/COEP response headers to enable those. See https://sqlite.org/wasm/doc/trunk/persistence.md#coop-coep sqlite3.worker.min-B-hRTo3R.js:4:15191
Create done index-JSApnn92.js:180:5287
C04FV3U1WA3NGbprbEnrBTa9YHhgSfTm49PiAd7T6/Q= index-JSApnn92.js:180:5314
Bootstrap start... index-JSApnn92.js:180:5748
Firefox can’t establish a connection to the server at wss://3df267fccbe9bb6b38c5ff5dde57f5401078b579.peerchecker.com:4003/p2p/12D3KooWN6xTJ8ZkRX4HZX4fa8NR9sDzMgoAxizwHMsDYSVpE7xy. index-JSApnn92.js:150:78536
Firefox can’t establish a connection to the server at wss://23a1839722ad07345186f20bd33110da13dc8d2e.peerchecker.com:4003/p2p/12D3KooWCHABWRzqR9iZQA8WzdNzSnELZVs7nwHmuJKWUYLQmoCM. index-JSApnn92.js:150:78536
Firefox can’t establish a connection to the server at wss://cc75656fce6cc671ffaf32bb795279a44ca0ec10.peerchecker.com:4003/p2p/12D3KooWSYTYuNbyfkTEXmoTbegpwWGUSrPPhmD9y7JqVfZv3AKs. index-JSApnn92.js:150:78536
Object { module: "client" }
 Failed to dial bootstrap address(s): "/dns4/23a1839722ad07345186f20bd33110da13dc8d2e.peerchecker.com/tcp/4003/wss/p2p/12D3KooWCHABWRzqR9iZQA8WzdNzSnELZVs7nwHmuJKWUYLQmoCM". Reason: CodeError: Could not connect to /dns4/23a1839722ad07345186f20bd33110da13dc8d2e.peerchecker.com/tcp/4003/wss/p2p/12D3KooWCHABWRzqR9iZQA8WzdNzSnELZVs7nwHmuJKWUYLQmoCM index-JSApnn92.js:113:254896
Object { module: "client" }
 Failed to dial bootstrap address(s): "/dns4/3df267fccbe9bb6b38c5ff5dde57f5401078b579.peerchecker.com/tcp/4003/wss/p2p/12D3KooWN6xTJ8ZkRX4HZX4fa8NR9sDzMgoAxizwHMsDYSVpE7xy". Reason: CodeError: Could not connect to /dns4/3df267fccbe9bb6b38c5ff5dde57f5401078b579.peerchecker.com/tcp/4003/wss/p2p/12D3KooWN6xTJ8ZkRX4HZX4fa8NR9sDzMgoAxizwHMsDYSVpE7xy index-JSApnn92.js:113:254896
Object { module: "client" }
 Failed to dial bootstrap address(s): "/dns4/cc75656fce6cc671ffaf32bb795279a44ca0ec10.peerchecker.com/tcp/4003/wss/p2p/12D3KooWSYTYuNbyfkTEXmoTbegpwWGUSrPPhmD9y7JqVfZv3AKs". Reason: CodeError: Could not connect to /dns4/cc75656fce6cc671ffaf32bb795279a44ca0ec10.peerchecker.com/tcp/4003/wss/p2p/12D3KooWSYTYuNbyfkTEXmoTbegpwWGUSrPPhmD9y7JqVfZv3AKs index-JSApnn92.js:113:254896
Failed to resolve relay addresses. Failed to succefully dial any bootstrap node index-JSApnn92.js:180:5645
Bootstrap done index-JSApnn92.js:180:5806
Source map error: Error: URL constructor:  is not a valid URL.
Resource URL: wasm:https://files.dao.xyz/assets/index-JSApnn92.js%20line%2040%20%3E%20WebAssembly.instantiate
Source Map URL: null

Could "peerchecker.com" be gone for good?

Most likely relatede to this issue
https://github.com/dao-xyz/peerbit/issues/284
I fixed it now by restarting the servers, but I am trying to resolve this issue permantly


Install as default parameter option

Great work jenv team!
As docker gaining momentum on our workflow, is there already a way similar to non-interactively install candidate as default? Such as jenv install --default=no java 1.7 jdk-1.7 instead of creating auto=true on ${JENV}/conf/settings file. Although we can do it using symlink in two step or more. This feature would be Dockerfile friendly.
Thank you.

:+1:

Sorry guys.  it's hard to add "--default=no" in script, please use "JENV_AUTO=true; jenv install maven 3.3.9" for silent mode.

Thanx for the explanation. Previously I just discovered on config file.


Add conduit methods to search legalpad documents and signatures

With this diff, you can search for documents or signatures in conduit. Additionally, you can request all the signatures as an attachment on a document.
Some details:

Adds phid to LegalpadDocumentSignature so that it can be exposed through the standard Conduit search interface
Fixes typo in LegalpadDocument search for contributorPHIDs

This is a bit too large to be an extension, and really should be upstreamed.

Screenshot of my dev instance with both attachment types selected:


Google crawling for files.php (asynchronous loading through JS)

Let google load all of the files page in javascript? Can it do that?
Also add metadata in other places

http://www.vadweb.us/view.php?name=image_33.jpg


when?

When will your warehouse be updated？ thanks

The associated paper is under review. We will publish our codes and datasets as soon as our paper is accepted.

Please when will the relevant code be released? So that other people can follow up on it.

Could you open source code？tks


admin menu give item

admin menu give item does not work, not seeing anything in the log about why

Probably has to do with dynamic items. Ty, will investigate

this was an issue on my side fixed and disregard


Establish code style and linting rules for the d1lod package

The current d1lod codebase was written styled with Black which I am pretty happy with and setup.py is already configured to install it. We might also pull in one of the common linters (pep8) if that looks useful. The README should reflect whatever we come up with.

I've set up Black and Flake8 and updated the readme to reflect that. I also tweaked Flake8's config to not warn about long lines since Black emits lines >80 chars sometimes.


conten are encoded in "content" tag

I used to forward stackoverflow newsletter to kill-the-newsletter and subscribe with tiny tiny rss.
Before January 29, 2019, I got the newsletter in tt-rss normally, but the next letter received in February 12, 2019 break. I find the content is encoded in "content" tag, cause tt-rss to display it as blank.
Is this intended or is this a bug?

I find it is because tt-rss don't support html content type

You can check your feed at https://validator.w3.org/feed/
If the validation passes, the problem is your reader (for example, tt-rss); otherwise the problem is in Kill the Newsletter! and we can investigate.


runtime/internal/sys

go version go1.17.1 linux/amd64
go install github.com/iangcarroll/cookiemonster/cmd/cookiemonster@latest
runtime/internal/sys
/usr/local/go/src/runtime/internal/sys/stubs.go:9:7: PtrSize redeclared in this block
/usr/local/go/src/runtime/internal/sys/arch.go:24:38: previous declaration
/usr/local/go/src/runtime/internal/sys/stubs.go:10:24: undefined: Uintreg
/usr/local/go/src/runtime/internal/sys/stubs.go:16:7: StackGuardMultiplier redeclared in this block
/usr/local/go/src/runtime/internal/sys/arch.go:27:74: previous declaration

Sorry, it seems like you have a corrupted install of Go. These don't seem to relate to this repository. You should be able to reinstall it pretty easily.


Setting playbackRate property of GrainPlayer has no effect

Example: https://jsfiddle.net/t9kjs72d/
<button id="one">one</button>
<button id="two">two</button>
<script src="https://cdnjs.cloudflare.com/ajax/libs/tone/14.6.8/Tone.js"></script>

player = new Tone.GrainPlayer('https://cdn.jsdelivr.net/gh/Tonejs/Tone.js/examples/audio/505/snare.ogg')
player.connect(Tone.Destination)
document.getElementById('one').addEventListener('click', () => {
  player.playbackRate = 1
  player.start()
})
document.getElementById('two').addEventListener('click', () => {
  player.playbackRate = 0.1
  player.start()
})

Expected: The sound is played at modified speed.
Actual: The sound is played at original speed.

Passing new Tone.GrainPlayer(url, {playbackRate: 10}) into the constructor does not affect the played sound either.

good catch! thanks for the issue. I think it should be fixed now. I wasn't testing at such extreme playbackRate values so i didn't discover that i was computing the grainSize wrong within scheduling callback. Should be fixed now.
I would suggest using a longer sample to validate the fix, otherwise the size of the grain might be nearly the size of the entire snare sample that you're using. Also adjusting the grainSize parameter to something much smaller (like 0.01) for values like playbackRate=0.1


Don't use rootPath in built-in extension

Discovered as part of #88391
Looks like some of out built-in extensions are still using rootPath:

[ ] Php @roblourens https://github.com/microsoft/vscode/blob/ca22de4b17af4ec84394537546998bba195e84cd/extensions/php-language-features/src/features/validationProvider.ts#L250
[ ] Emmet @octref  https://github.com/microsoft/vscode/blob/99d5ff9c0f5a66a9e1cd7d49f9af6892e13f96a4/extensions/emmet/src/util.ts#L35
[ ] json @aeschli https://github.com/microsoft/vscode/blob/ce31ace9d9e4a67851c0fb4ed5633ebb494f8c6f/extensions/json-language-features/client/src/jsonMain.ts#L415

lol, how did I miss those?

The builtin PHP validation is very simple and I've wanted to delete it, and nobody has ever complained that it doesn't work correctly in multiroot, so I'm not bothering to actually fix it for multiple folders.

Looks like #87493 is this for Emmet

Emmet seems a bit more complicated — I'll track it in #87493 then.


Asset proxy failing to load static files from production

The asset proxy fails to get static files from the production instance even with proper config.yml configuration, resulting as 404 errors when accessing the local site. Steps to reproduce:

Clone Seravocom (private) repo and switch to legacy branch
Run vagrant up in repo and don't pull production db (yet)
Run vagrant ssh and run wp-pull-production-db in the virtual machine, check that all presented site urls are valid
Check local site in browser, static files give 404 error codes
Attempt to run wp-use-asset-proxy in vagrant, does not fix the issue

There has also been at least one customer report on this issue. This could originate from some overlapping Nginx configurations, but we should look more thoroughly into this.

The problem was caused due to /wordpress/wp-content/ not being a symlink to /wp-content/.
Fixed it now for wp-palvelu.fi.


Check SBT caching performance

This test proved that the caching policy is indeed reducing build time for subsequent builds. The first build took around 3minutes 45 seconds and the subsequent one took 1minute.


Add Armadillo Run

Hi @cristaloleg,
First, I'm so sorry I missed this PR from a few weeks ago! I didn't mean to ignore it for so long, just didn't notice it. My apologies.
As for the game itself, unfortunately, right now this list only lists games on Steam. It's mostly just a proxy for "awesomeness", so we don't get a bunch of games without any community reviews.
If Armadillo Run ever gets added to Steam, I'd be glad to add it under the New Games section.
Thank you, and sorry about that!

Hey, not a problem at all. I missed that this list include only Steam games, so closing PR looks fair. Thanks for the list.


Do not show an error message in cases where  session is already deallocated

Do not show an error message in cases where RLMProgressNotificationToken session is already deallocated (#7831, since v2.3.0).

I wonder if the token should actually store a strong pointer to the session rather than a weak one? All of our other tokens keep the thing they're tracking notifications for alive, and it's sort of weird that this one doesn't.

@tgoyne updated the PR to have a reference so the SyncSession


Split Table C.2 into subclauses or numbered items in a list

The table does not fit on a single page.
See render at https://github.com/SMPTE/html-pub/pull/183

Specifically single page as PDF.
@palemieux should we note that in the documentation that especially long tables should be reconsidered? I know of a few documents that have longer'ish tables... do we really need to change those due to a PDF bug?


Add toObject hook for documents.

Includes tests and upgrades feathers version to 1.3.0.  One test is being skipped because I can't figure out how to test for the error thrown inside the hook when it's not used/initialized as a function.
Fixes #29 and #25, once published to npm.

This is only failing in node 0.10 on the $populate test.  I'm going to go ahead and merge this since we'll be switching over to the new tests, today, anyway.


Fix typechecking issues

The main remaining typechecking issue is that in core.WorldModel the attribute space is intialized to Space(). However, in the concrete subclass of WorldModel in the entities module, the attribute is restricted to a subclass of Space with more methods, which gives type errors when using those methods. This can probably be fixed using typing.Protocol.
Apart from that, there are errors related to the imports from mesa.

The solution to the above is probably to make classes in core generic, as explained here.
The issues related to capabilities should be resolved using protocols. To some extent, the expectations a capability has on the agent's interface are already described in docstrings, but would be much clearer by specifying the protocol.


Guide - Install for Windows - WIP

Issue: to be tagged as documentation
I am looking myself into creating a full guide for windows users that might feel left out a little on how to get the lounge set up from start to finish.
This guide will fully support:

[ ] Windows 10, Windows 11 and Server 64 Bit OS installations.
[ ] Batch script to install the core automatically.
[ ] A simple batch script to start the service once fully installed.
[ ] A manual way on how to install it, section by section.

Has anyone actually wanted to get the lounge up and running on windows OS
.. or failed or would like to know more information about it?

With the help of a few people in #thelounge (great thanks to Activ8)  I've got TheLounge working on Windows 11 Pro 21H2.

Start with https://thelounge.chat/docs/install-and-upgrade#from-npm-releases
Create user/pass
You should now have TheLounge working. You can stop here since its working but I personally didnt like the fact that i had to have terminal open while its running and the fact that I had to manually launch it. To avoid that follow ahead.
Close the terminal.
Create a new text file, just add thelounge start in the text file and save it.
Rename that file to file to anything you want, for convenience I'll name it lounge. Important part is change the .txt to .bat (i.e, lounge.txt -> lounge.bat). Windows will prompt you that changing the extension can make it unusable but ignore it and rename it.
To test your bat script, double click lounge.bat (or whatever you named it). It should launch terminal and start theLounge server.
Close terminal.
Create a new text file and paste this into it:

Set WshShell = CreateObject("WScript.Shell")
WshShell.Run chr(34) & "path\to\your .bat file" & Chr(34), 0
Set WshShell = Nothing

Replace "path\to\your .bat file" with the path of your .bat file
So for example:
I saved the lounge.bat  file in Documents. Right-click it in Windows 11 and click Copy As Path (For Windows 10, the Copy As Path option won't show up when simply right-clicking. Do SHIFT+Right Click and this time you'll see a bunch of extra options including Copy As Path. Path has been copied to your clipboard (something like "C:\Users\username\Documents\lounge.bat" and now just replace the placeholder path.
It should look like this after editing:
Set WshShell = CreateObject("WScript.Shell")
WshShell.Run chr(34) & "C:\Users\username\Documents\lounge.bat" & Chr(34), 0
Set WshShell = Nothing

Save the file.
Rename that file to file to anything you want, for convenience I'll name it launch_lounge. Important part is change the .txt to .vbs (i.e, launch_lounge.txt -> launch_lounge.vbs). Windows will prompt you that changing the extension can make it unusable but ignore it and rename it.
Now double click launch_lounge.vbs (again whatever name you gave it as long as the extension is .vbs). Nothing should show up but you should be able to connect Lounge IRC from your browser now. You can check if its running by going to Task Manager and look for Node.js JavaScript Runtime.
The First problem has been solved, that was, having the terminal open in the foreground. It's still running but now in background which is what i prefer.
You can use the .vbs file you just created as a launcher but i would prefer it launches at startup so ill use Task Scheduler.
Search "Task Scheduler" in Windows
Open it
On the Right Pane, go to Create Task and click it.
A new window will open. Give the task whatever name you want. Ill name it  theLoungeStartUp
Go to the next tab, Triggers
Click the New button
New Window opens. Go to the top dropdown. I would want it to start at Windows StartUp, so ill change it from On a schedule to At startup but you can adjust it according to your needs
Save, move to next tab Actions
Click the New button
New Window Opens. Leave the dropdown at default, move to second field that is Program/Script, click the browse button and locate your .vbs file (NOT the .bat initially created)
Click Ok.
You should see the new task you created in the Task Scheduler Library. Click it, go to the right pane, click run.

All setup done. Now everytime PC starts, TheLounge Server will launch in background :D
Side Note:
Other methods I tried but failed were installing it as a service through default Powershell and NSSM.

Thanks for this, very precise info.  This is a step by step guide but don't want to commit this to the main website.  I feel it's a little long and a lot of steps but could easily condense it.
Basically this guide is only to hide the command prompt window using schedule tasks.
Will look into gathering more information before I put up a windows guide.

I have decided to keep this on my own GitHub page and close this as an issue.  There's little interest in it here but this is the link for future reference.
https://github.com/aab12345/thelounge-windows
/closed.


fix: menu background color and update theme

Changes

Popup menu didn't show the dark mode background color correctively before. This PR hides Mui's default background image to show background color properly
Update theme based on design (minor color changes)

Design

Before

After

@lyjeileen shouldn't rename be the selected option? could you check the selected/active colors in the theme ?


feat: support arbitrum sepolia testnet [APE-1493]

What I did

fixes: #
How I did it
How to verify it
Checklist

[x] Passes all linting checks (pre-commit and CI jobs)
[x] New test cases have been added and are passing
[x] Documentation has been updated
[x] PR title follows Conventional Commit standard (will be automatically included in the changelog)

Test failures are because it's not released yet in ape-optimism

rebase on https://github.com/ApeWorX/ape-infura/pull/67 when it lands


register/handlers: remove context manager

Signed-off-by: Stephen J Day<EMAIL_ADDRESS>cc @vikstrous

It's worth noting that the previous use of context had a memory leak because it was never deallocating request objects because they were all in a global map and a release() call was missing in one of the places where context() was called.
We'll test out this commit on distribution 2.5 and report back.

Also, note that while this starts to address #1764, it does not remove the handlers.Context object. More must be done to address #1764.

It might be safer and easier for backporting to solve the memory leak this way: https://github.com/docker/distribution/pull/2240 and the context manager can be removed independently of this particular issue

Confirmed that this PR works and fixes the memory leak.


[ROCM 5.7] fix default installation path on Windows

The PR is for the 5.7 branch because we need it for the ROCm 5.7 release on Windows.
It is unusual on Windows to install to C:\Program Files manually. It requires Administrator privilege, which is not standard or not easily granted. Only applications managed by the Windows installer are allowed to install in that directory, and cmake install is not one of them.
Installing tests in a custom directory given to CMake is also impossible when HEADER_INSTALL_PREFIX is set to C:/Program Files. Moreover, the directory C:\Program Files is for 64-bit applications only. In the case of 32-bit, the installer should use the C:\Program Files (x86) directory instead.
The separate PRs will add that change to the current development branch.

The changes from the latest develop branch work for us.


Example of a complexity calculation in lecture 9

Hej Jeppe
I am a bit confused about your example 2.3 in lecture 9. You calculate the total number of operations to
3n^2+6n+4.
But as i understand the reasoning behind number of operations shouldn't the total number of operations be
6n^2+5n+4
as 1) loop over n*n combinations with 3 assignments and 3 multiplications gives 6n^2 operations 2) loop over n combinations with 2 assignments, 2 multiplications and 1 addition gives 5n?
In advance, many thanks for the help!
Best
Jens

You are correct. I can apparently not count.


Missing Commit Message Conventions

Bug Report 🐛

When submitting my first pull request, I noticed that I've been prompted several times to adhere to the "commit message conventions". The documentation of the convention is supposed to be on a documentation page, but it seems to be missing.

Possible Solution

I'd like to (re)add it, could you please let me know what would you like in it?
Steps to Reproduce

Context (Environment)

Upon checking out some other Accord repositories, I noticed they all tend to lead to this page in the techdocs repository as well, so it's something that impacts the whole organisation.

Desktop

OS: [e.g. macOS]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 0.22.15]

Detailed Description

Possible Implementation

Hi @anablaz
Can you please confirm if you are looking for this?

Hello,
@sanketshevkar, the missing information that I'm talking about is also referenced on the page you linked - in the Pull Request Submission Guidelines section of the page, there are two hyperlinks titled "commit message conventions". Upon clicking either of them, the link takes you to the site I listed when opening this issue.
This issue is about the fact that there is no commit message conventions information on the link. I'm assuming it used to be documented there, since it's referenced both in the pull request submission guidelines as well as on the developer checklist when you make a pull request  (the check for "commits messages follow AP format").

Thanks for the detailed explanation @anablaz.
Yes you are right we need commit convention rules linked. This what we follow https://www.conventionalcommits.org/en/v1.0.0/
@DianaLease I remember having details about conventional commits, but they are not present there now.


JAVA BigInteger

💥 Proposal
(A clear and concise description of what the proposal is. We have to add and multiply huge numbers! These numbers are so big that you can't contain them in any ordinary data types like a long integer. We will use the power of Java's BigInteger class.
fFirst integer(a) and second integer(b) are non-negative integers and can have maximum  200 digits.)
Have you read the Contributing Guidelines on Pull Requests?
Yes.

/assign


fix: add fix for common path function when formatting files at direct…

…ory root
PR to prevent the following error:
Error: ENOTDIR: not a directory, chdir '...' -> 'i18n/index.ts/'
Glob used: "i18n/*.{ts,tsx,js,jsx,json,yaml}""
Example Structure:
├── i18n
│   ├── en
│   │   └── common.json
│   ├── es
│   │   └── common.json
│   ├── index.ts

We should probably add a few tests as well. Feels tricky to change this without a few cases being checked for.

I'm flexible. Jest or Vitest?

I think vitest is great for this type of lib


More symbols in "Text Extractor"

Description of the new feature / enhancement
It would be really helpful if there was more symbols like mathematical inputs and symbols in the text extractor feature. Also being able to give bars over any letter will also be really handy
Scenario when this would be used?
I can use it to write symbols when writing in a textbox of a pdf reader or a website like "Stack Exchange" etc.
Supporting information
No response

this would be dependent for the OCR engine itself.  /dup #20899


[Snyk] Security upgrade org.apache.kafka:kafka_2.12 from 2.5.0 to 3.4.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

orion-server/pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity
Reachability

525/1000  Why? Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-2421244
org.apache.kafka:kafka_2.12:  2.5.0 -> 3.4.0
Yes
No Known Exploit
No Path Found

520/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038424
org.apache.kafka:kafka_2.12:  2.5.0 -> 3.4.0
Yes
Proof of Concept
No Path Found

520/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038426
org.apache.kafka:kafka_2.12:  2.5.0 -> 3.4.0
Yes
Proof of Concept
No Path Found

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)
🦉 Denial of Service (DoS)
🦉 Denial of Service (DoS)

Kafka version in Orion is used as a client not as server. Jackson version can be upgraded as long as it is compatible with dropwizard.


SSL Certificate Generation

Would it be possible to implement SSL certificate generation?

I certainly should be, since Bouncy Castle can do it as well.
However, I'm not sure if I can do it. @izaera used to make all implementations and ports from BC, so I don't have real experience doing so. But I might want to give it a try for sure.
For now, I first want to get this working in browsers again (some issues with reflectable keep it from working, but I'm working on it).
May I ask why you want Dart to generate the certificate instead of just using OpenSSL for it?

@stevenroose Our code is a very portable IoT Application Platform server. We have at least one user for every single platform/architecture that the Dart VM supports. This includes Android ARM and Android x86. Windows and Android do not ship with OpenSSL natively, and even if we were to ship it with our code, it is less reliable to work on all platforms.

The only "problem" I see is the usual: AFAIK you need strong random number generators and we don't have a reliable source of entropy right now in Dart. @stevenroose maybe you can get true randomness from the JS crypto API in the browser. For the server case you can resort to /dev/random in unix. In the case of Windows you would need a binary Dart extension to access CryptoAPI.
Unless Google has already resolved the randomness problem (which I doubt).

Another possibility for the client would be calling the server using HTTPS to get true randomness for it. It is a riskier scenario, but may make sense if app is not too critical.

@izaera I was about to say that as well, but then I noticed that private key generation currently is also done with just a SecureRandom instance.
But fwiw, I think implementing a dynamic entropy source should be possible. The question then is whether the library should call this under the hood, or whether it should ask for entropy. (Like with a ParametersWithEntropy or something).
Currently with the dynamic registry, it would theoretically be possible to create io-dependent or html-dependent entropy sources and by letting the user just import this class in his project, the dynamic registry can find it when new EntropySource() is called. This would even work for OS-specific ones.

That was my idea when I wanted to split cipher in two: one common for server and client with the algorithms, another with the entropy generation (the only server/client dependent part).
I think the most important point here is not to hide the entropy “issue” from the user as it is key for the security of all crypto to know the source and quality of entropy being used. Having two libraries, where the entropy one is interchangeable by another -hopefully- better implementation is a very good thing IMO.

Well, yes indeed. I also thought about just asking for a load of entropy
instead of asking for an entropy source.
On Mon, Feb 1, 2016 at 4:40 PM, Ivan Zaera<EMAIL_ADDRESS>wrote:

That was my idea when I wanted to split cipher in two: one common for
server and client with the algorithms, another with the entropy generation
(the only server/client dependent part).
I think the most important point here is not to hide the entropy “issue”
from the user as it is key for the security of all crypto to know the
source and quality of entropy being used. Having two libraries, where the
entropy one is interchangeable by another -hopefully- better implementation
is a very good thing IMO.
—
Reply to this email directly or view it on GitHub
https://github.com/PointyCastle/pointycastle/issues/94#issuecomment-178029866
.

But how do you know how much entropy you need in advance? Entropy is expensive and async in nature. If you don’t ask for enough you have a problem, if you ask for too much either, then you are slowing things unnecessarily. That’s why I defined an EntropySource instead of passing an array of bytes.

El 1 feb 2016, a las 17:58, Steven Roose<EMAIL_ADDRESS>escribió:
Well, yes indeed. I also thought about just asking for a load of entropy
instead of asking for an entropy source.
On Mon, Feb 1, 2016 at 4:40 PM, Ivan Zaera<EMAIL_ADDRESS>wrote:

That was my idea when I wanted to split cipher in two: one common for
server and client with the algorithms, another with the entropy generation
(the only server/client dependent part).
I think the most important point here is not to hide the entropy “issue”
from the user as it is key for the security of all crypto to know the
source and quality of entropy being used. Having two libraries, where the
entropy one is interchangeable by another -hopefully- better implementation
is a very good thing IMO.
—
Reply to this email directly or view it on GitHub
https://github.com/PointyCastle/pointycastle/issues/94#issuecomment-178029866
.

—
Reply to this email directly or view it on GitHub https://github.com/PointyCastle/pointycastle/issues/94#issuecomment-178067240.

@stevenroose @izaera Dart v1.14 adds a Random.secure constructor for the Random class to get a secure random number generator. However, it probably isn't a good idea to use it yet, as they will up the SDK requirement. (We have platforms that will only run Dart v1.12.2 and below currently, so this would be a problem for us)

Where do you plan to get entropy from, then?

Would it be possible to implement SSL certificate generation?

Hello @kendfinger  ,Did you try  implement SSL certificate generation with this library?


Http data plane only allows query parameters with unique keys

Bug Report
Describe the Bug
It's a bug in how the Http data plane proxies the query parameters of incoming requests.
Expected Behavior
Http data planes can be configured in such a manner that they forward query parameters provided by the consumer. In that case a call GET https://provider-data.plane?param=value will be proxied as GET https://provider-back.end?param=value. This also works if multiple distinct keys are provided for separate query parameters. If query parameters are of type array, transmitting the entries of said array as separate query parameters with equivalent keys is a common practice, yielding requests like GET https://provider-back.end?param=value1&param=value2&param=value3.
Observed Behavior
The http data plane currently doesn't proxy these parameters properly. A request GET https://provider-data.plane?param=value1&param=value2&param=value3 will be proxied as GET https://provider-data.plane?param=value1.
Steps to Reproduce

(Provider) setup arbitrary asset with "type":"HttpData" and "proxyQueryParams":"true"
(Provider) publish via contract definition
(Consumer) find asset in catalog
(Consumer) negotiate, initiate transfer process, receive EDR
(Consumer) call GET https://provider-data.plane?param=value1&param=value2&param=value3
(Provider) Observe resulting request in backend

Context Information
Observed with tractusx-edc 0.6.0 (thus eclipse-edc 0.5.1). This was first raised in the EDC matrix channel by @sachinargade123.

Hello @arnoweiss, could You tell me if it  is planned to solve the issues in 24.05 release?

Hello @stefan-ettl, is it planned to solve this issue for the 24.05 release?

@arnoweiss and @ciprianherciu not exactly sure this really is a bug, as RFC 3986, section 3.4 Query is kinda vague about it. I would thus classify it as feature request.
What is your use case for having multiple identical query params?
@ciprianherciu this is the Eclipse Dataspace Components project, any downstream releases such as Tractus-X 24.05 are not relevant here, nor can you expect committers here to be aware of it. Also mentioning Tractus-X Matrix chatrooms is not really helpful.

This issue becomes relevant when a client is attempting to access a Digital Twin Registry (DTR) via the http data plane.
sequenceDiagram
    participant C as Consumer
    participant CCP as Consumer Control Plane
    participant PCP as Provider Control Plane
    participant PDP as Provider Data Plane
    participant DTR as Digital Twin Registry
    CCP->PCP: catalog, negotiate, trigger transfer
    PCP -->>C: EDR
    C->>PDP: /lookup/shells?assetIds=foo&assetIds=bar
    PDP ->>DTR: /lookup/shells?assetIds=foo
    DTR -->> PDP: response (only for assetIds=foo)
    PDP -->> C: response (only for assetIds=foo)

A patch to the Asset-Administration-Shell-Spec has clarified that the query parameter of the operation GetAllAssetAdministrationShellIdsByAssetLink  is an array of encoded objects. The reference implementation eclipse-tractusx/sldt-digital-twin-registry has implemented it as described in the initial post. Thus, whenever a client executes a request to a DTR with mutliple assetIds parameters, the http data plane removes all but one. I don't want to comment on the quality of the AAS spec here - either way, clients and servers rely on this mechanism until there's a more reliable and disambiguous mechanism.

Why can't this be corrected in the SDTR? Given the ambiguity associated with this in the referenced RFC, the fact that other data planes may not support it, and the fact that there are better ways to represent this as a RESTFul HTTP request, the proper solution is probably to address this in AAS and SDTR.

As @arnoweiss mentioned, the /lookup/shells endpoint of the DTR is specified in this way in the AAS standard linked in his comment above. So, a solution in the DTR is not a simple implementation detail but would require a change in the standards.
I have also checked the behavior of the Swagger Tooling for an OpenAPI spec, and it translates a query parameter of type array[string] into exactly this sequence of a multi usage of the same key in the query param string. So, although the RFC might be ambiguous, this seems to be expected standard behavior of a rest GET endpoint.
And finally, in general, in the role of a proxy, it is non-expected behavior, that the semantics of the request are changed by removing parts of the query that needs to be forwarded to the proxied system.
As this issue basically prevents a proper usage of the standardized DTR endpoint, we at Cofinity-X consider this as an urgent issue to be solved for the upcoming release.

I completely agree to @arnoweiss, @ciprianherciu, and @lgblaumeiser.
The DTR is, as aligned in the whole Catena-X activities, compliant to the specifications of the Asset Administration Shell of the IDTA and ALL use cases, which rely on Digital Twins are blocked by this bug.
How can we support here @jimmarino, @paullatzelsperger to get the bug fixed within Release 24.05.
Please let us know.

This is not a bug but an enhancement. We will need to triage and prioritize it according to standard procedures. We will discuss this during our next committer's meeting.

@lgblaumeiser I understand, and the technical functionality is not the issue, since the fix would be trivial to do. It's only a couple of lines.
I do reject the notion that it is a bug, simply because the aforementioned RFC does not classify it as one.
All issues in EDC are subject to the committers' triage process and decision, and will be prioritized accordingly. It is understood that you think it is urgent and it should be in the next EDC version, and we will consider it. I do want to point out though, that EDC version releases are not coupled to releases of downstream projects, such as Tractus-X.
@thomas-henn so there really is nothing to support here, as all that is required is adherence to proper procedure.

I did read it: I assume there is another process for bugs. So I do not understand why you are not accepting that this is a bug and follow the faster bug-process.

@BirgitBoss the only people who get to decide the class of issue (bug vs feature request) and its associated priority and thus: release schedule, are the committers of EDC, as per Eclipse governance rules.
In the same spirit I think this discussion is not a waste of time, and indeed provides great value, because there clearly is a gap in the understanding of open-source procedures.

I did read it: I assume there is another process for bugs. So I do not understand why you are not accepting that this is a bug and follow the faster bug-process.

Because classifying it is a group decision of the committers, and we have procedures for that. Whether you agree to them or the outcome is immaterial.

Thanks, @paullatzelsperger @jimmarino for taking this up. It was my only intention, that it is considered with the appropriate process and priority. I think, everything is said and lets proceed according to the process.
Btw, there is imho no value in discussing the classification of the issue. It is a requirement that is currently not met by the EDC and the urgency has been made clear.

Update from today's committer call: since the fix is trivial, we will include this in the Milestone 14 release (EDC Version 0.6.0), which is scheduled to be released on March 27.
Further, the requested behaviour will only be implemented in the public API of the DataPlane Signaling feature pack, it won't be backported to the legacy data plane API.
For the people involved in this thread that won't matter, because Tractus-X will only have the new data plane API.
Note: Whether Tractus-X 24.05 (= Tractus-X EDC 0.7.x) will use EDC 0.6.0 or a subsequent bugfix release is yet to be determined.


wip: issue-310 migrate to junit5

closes #310

:grey_exclamation: Build Aborted

The PR is not allowed to run in the CI yet

Expand to view the summary

Build stats

Build Cause: [Pull request #1433 opened]

Reason: The PR is not allowed to run in the CI yet

Start Time: 2020-10-07T17:30:56.510+0000

Duration: 3 min 48 sec

Commit: 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917

Steps errors
Expand to view the steps failures

Name: Error signal

Description: githubPrCheckApproved: The PR is not allowed to run in the CI yet. (Only users with write permission

Duration: 0 min 0 sec

Start Time: 2020-10-07T17:33:40.006+0000

log

Log output
Expand to view the last 100 lines of log output

[2020-10-07T17:33:14.069Z]  > git config remote.origin.url git@github.com:elastic/apm-agent-java.git # timeout=10
[2020-10-07T17:33:14.079Z]  > git rev-parse --verify HEAD # timeout=10
[2020-10-07T17:33:14.091Z] No valid HEAD. Skipping the resetting
[2020-10-07T17:33:14.092Z]  > git clean -fdx # timeout=10
[2020-10-07T17:33:14.110Z] Fetching upstream changes from git@github.com:elastic/apm-agent-java.git
[2020-10-07T17:33:14.111Z] using GIT_SSH to set credentials GitHub user @elasticmachine SSH key
[2020-10-07T17:33:14.118Z]  > git fetch --no-tags --progress --prune -- git@github.com:elastic/apm-agent-java.git +refs/pull/1433/head:refs/remotes/origin/PR-1433 +refs/heads/master:refs/remotes/origin/master # timeout=15
[2020-10-07T17:33:14.898Z] Merging remotes/origin/master commit 2259495d153766fe36aa2833a0455394b2fadbb8 into PR head commit 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917
[2020-10-07T17:33:15.170Z] Merge succeeded, producing 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917
[2020-10-07T17:33:15.170Z] Checking out Revision 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917 (PR-1433)
[2020-10-07T17:33:14.906Z]  > git config core.sparsecheckout # timeout=10
[2020-10-07T17:33:14.910Z]  > git checkout -f 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917 # timeout=15
[2020-10-07T17:33:15.125Z]  > git remote # timeout=10
[2020-10-07T17:33:15.132Z]  > git config --get remote.origin.url # timeout=10
[2020-10-07T17:33:15.139Z] using GIT_SSH to set credentials GitHub user @elasticmachine SSH key
[2020-10-07T17:33:15.147Z]  > git merge 2259495d153766fe36aa2833a0455394b2fadbb8 # timeout=10
[2020-10-07T17:33:15.161Z]  > git rev-parse HEAD^{commit} # timeout=10
[2020-10-07T17:33:15.173Z]  > git config core.sparsecheckout # timeout=10
[2020-10-07T17:33:15.177Z]  > git checkout -f 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917 # timeout=15
[2020-10-07T17:33:18.926Z] Commit message: "issue-310 migrate to junit5"
[2020-10-07T17:33:18.926Z] First time build. Skipping changelog.
[2020-10-07T17:33:18.926Z] Cleaning workspace
[2020-10-07T17:33:19.100Z]  > git --version # timeout=10
[2020-10-07T17:33:19.107Z]  > git --version # 'git version 2.17.1'
[2020-10-07T17:33:19.124Z] fatal: bad object e53095c434317b592806658835c646c2e93e6122
[2020-10-07T17:33:19.930Z] Masking supported pattern matches of $JOB_GCS_BUCKET or $NOTIFY_TO
[2020-10-07T17:33:20.020Z] Timeout set to expire in 1 hr 0 min
[2020-10-07T17:33:18.929Z]  > git rev-parse --verify HEAD # timeout=10
[2020-10-07T17:33:18.933Z] Resetting working tree
[2020-10-07T17:33:18.933Z]  > git reset --hard # timeout=10
[2020-10-07T17:33:19.046Z]  > git clean -fdx # timeout=10
[2020-10-07T17:33:20.057Z] The timestamps step is unnecessary when timestamps are enabled for all Pipeline builds.
[2020-10-07T17:33:20.454Z] [INFO] Number of builds to be searched 10
[2020-10-07T17:33:20.734Z] [INFO] 'shallow' is forced to be disabled when running on PullRequests
[2020-10-07T17:33:20.769Z] Running in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1433/src/github.com/elastic/apm-agent-java
[2020-10-07T17:33:20.824Z] [INFO] gitCheckout: Checkout SCM PR-1433 with some customisation.
[2020-10-07T17:33:20.866Z] [INFO] Override default checkout
[2020-10-07T17:33:20.950Z] Sleeping for 10 sec
[2020-10-07T17:33:31.109Z] using credential f6c7695a-671e-4f4f-a331-acdce44ff9ba
[2020-10-07T17:33:31.200Z] Cloning the remote Git repository
[2020-10-07T17:33:31.217Z] Cloning repository git@github.com:elastic/apm-agent-java.git
[2020-10-07T17:33:31.244Z]  > git init /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1433/src/github.com/elastic/apm-agent-java # timeout=10
[2020-10-07T17:33:31.258Z] Using reference repository: /var/lib/jenkins/.git-references/apm-agent-java.git
[2020-10-07T17:33:31.259Z] Fetching upstream changes from git@github.com:elastic/apm-agent-java.git
[2020-10-07T17:33:31.259Z]  > git --version # timeout=10
[2020-10-07T17:33:31.271Z]  > git --version # 'git version 2.17.1'
[2020-10-07T17:33:31.271Z] using GIT_SSH to set credentials GitHub user @elasticmachine SSH key
[2020-10-07T17:33:31.276Z]  > git fetch --tags --progress -- git@github.com:elastic/apm-agent-java.git +refs/heads/*:refs/remotes/origin/* # timeout=10
[2020-10-07T17:33:32.828Z]  > git config remote.origin.url git@github.com:elastic/apm-agent-java.git # timeout=10
[2020-10-07T17:33:32.835Z]  > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # timeout=10
[2020-10-07T17:33:32.843Z]  > git config remote.origin.url git@github.com:elastic/apm-agent-java.git # timeout=10
[2020-10-07T17:33:32.857Z] Fetching upstream changes from git@github.com:elastic/apm-agent-java.git
[2020-10-07T17:33:32.857Z] using GIT_SSH to set credentials GitHub user @elasticmachine SSH key
[2020-10-07T17:33:32.861Z]  > git fetch --tags --progress -- git@github.com:elastic/apm-agent-java.git +refs/pull/1433/head:refs/remotes/origin/PR-1433 +refs/heads/master:refs/remotes/origin/master # timeout=10
[2020-10-07T17:33:33.671Z] Checking out Revision 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917 (origin/PR-1433)
[2020-10-07T17:33:33.846Z] Commit message: "issue-310 migrate to junit5"
[2020-10-07T17:33:33.864Z]  > git --version # timeout=10
[2020-10-07T17:33:33.871Z]  > git --version # 'git version 2.17.1'
[2020-10-07T17:33:33.875Z] fatal: bad object e53095c434317b592806658835c646c2e93e6122
[2020-10-07T17:33:33.665Z]  > git rev-parse origin/PR-1433^{commit} # timeout=10
[2020-10-07T17:33:33.674Z]  > git config core.sparsecheckout # timeout=10
[2020-10-07T17:33:33.677Z]  > git checkout -f 8d9a862cd045e1ace3b5cd6d71ad7067c41e9917 # timeout=10
[2020-10-07T17:33:35.044Z] Masking supported pattern matches of $GIT_USERNAME or $GIT_PASSWORD
[2020-10-07T17:33:35.660Z] + git fetch<EMAIL_ADDRESS>+refs/pull/*/head:refs/remotes/origin/pr/*
[2020-10-07T17:33:36.668Z] Archiving artifacts
[2020-10-07T17:33:37.392Z] + git rev-parse HEAD
[2020-10-07T17:33:37.774Z] + git rev-parse HEAD
[2020-10-07T17:33:38.094Z] + git rev-parse origin/pr/1433
[2020-10-07T17:33:38.159Z] [INFO] githubEnv: Found Git Build Cause: pr
[2020-10-07T17:33:38.495Z] Masking supported pattern matches of $GITHUB_TOKEN
[2020-10-07T17:33:39.529Z] [WARN] githubApiCall: The REST API call https://api.github.com/repos/elastic/apm-agent-java/pulls/1433/reviews return 0 elements
[2020-10-07T17:33:39.610Z] [INFO] githubPrCheckApproved: Title: wip: issue-310 migrate to junit5 - User: kananindzya - Author Association: CONTRIBUTOR
[2020-10-07T17:33:40.026Z] ERROR: githubPrCheckApproved: The PR is not allowed to run in the CI yet
[2020-10-07T17:33:40.026Z] ERROR: githubPrCheckApproved: The PR is not allowed to run in the CI yet. (Only users with write permissions can do so.)
[2020-10-07T17:33:40.123Z] [INFO] Let's stop build #1. The PR is not allowed to run in the CI yet
[2020-10-07T17:33:40.152Z] Sleeping for 5 sec
[2020-10-07T17:33:41.445Z] Stage "Build" skipped due to earlier failure(s)
[2020-10-07T17:33:41.668Z] Stage "Tests" skipped due to earlier failure(s)
[2020-10-07T17:33:41.886Z] Stage "Unit Tests" skipped due to earlier failure(s)
[2020-10-07T17:33:41.901Z] Stage "Smoke Tests 01" skipped due to earlier failure(s)
[2020-10-07T17:33:41.915Z] Stage "Smoke Tests 02" skipped due to earlier failure(s)
[2020-10-07T17:33:41.930Z] Stage "Benchmarks" skipped due to earlier failure(s)
[2020-10-07T17:33:41.944Z] Stage "Javadoc" skipped due to earlier failure(s)
[2020-10-07T17:33:42.141Z] Failed in branch Unit Tests
[2020-10-07T17:33:42.151Z] Failed in branch Smoke Tests 01
[2020-10-07T17:33:42.161Z] Failed in branch Smoke Tests 02
[2020-10-07T17:33:42.170Z] Failed in branch Benchmarks
[2020-10-07T17:33:42.180Z] Failed in branch Javadoc
[2020-10-07T17:33:42.471Z] Stage "Integration Tests" skipped due to earlier failure(s)
[2020-10-07T17:33:42.574Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-10-07T17:33:42.621Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-10-07T17:33:43.009Z] Running on Jenkins in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1433
[2020-10-07T17:33:43.342Z] [INFO] getVaultSecret: Getting secrets
[2020-10-07T17:33:43.531Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2020-10-07T17:33:44.545Z] + chmod 755 generate-build-data.sh
[2020-10-07T17:33:44.545Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1433/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1433/runs/1 ABORTED 167763
[2020-10-07T17:33:45.096Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1433/runs/1/steps/?limit=10000 -o steps-info.json
[2020-10-07T17:33:45.647Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1433/runs/1/tests/?status=FAILED -o tests-errors.json
[2020-10-07T17:33:45.647Z] Retry 1/3 exited 22, retrying in 1 seconds...
[2020-10-07T17:33:46.559Z] Retry 2/3 exited 22, retrying in 2 seconds...


freeze during downloading comments

Hey, first of all nice work! I tried your code to download some comments and it works pretty good in the most cases, but as I tried your code on YouTube-Sites that contains thousands of comments such as PewDiePie-videos, the downloading process seems like to freeze.
I used the following videoID:
comments = commentExtract("jvO1GDJXOgo") print(comments)

Thanks! Sorry for the late reply(very late actually). The freeze that happened during downloading comments is status error 429. It happens when we send too many requests. Even though I use the YouTube-API the error still happens. One alternative is to use different APIs at one time. If you find other best alternative tell me also :)


Populate light sdf::ElementPtr from Light DOM

🎉 New feature
needed by https://github.com/ignitionrobotics/ign-gazebo/pull/1209
Summary
similar to #753
Adds a function in Light DOM class to populate an sdf::ElementPtr
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Codecov Report

Merging #755 (5d45a23) into sdf12 (ad00668) will increase coverage by 0.03%.
The diff coverage is 95.12%.

@@            Coverage Diff             @@
##            sdf12     #755      +/-   ##
==========================================
+ Coverage   88.57%   88.61%   +0.03%
==========================================
  Files          76       76
  Lines       11740    11781      +41
==========================================
+ Hits        10399    10440      +41
  Misses       1341     1341

Impacted Files
Coverage Δ

src/Light.cc
92.15% <95.12%> (+0.74%)
:arrow_up:

src/Link.cc
96.66% <0.00%> (+1.11%)
:arrow_up:

Continue to review full report at Codecov.

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


[SMALLFIX] Use an ephemeral worker port for test.

Otherwise this fails/does not test correctly if you have a process running on port 29998.

Merged build finished. Test FAILed.

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

Jenkins, test this please.

Merged build finished. Test PASSed.

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


GUI: Date Time

Dev story - https://github.com/openbmc/phosphor-webui/issues/46

@gkeishin : Please assign this story to @sivassrr

[ ] Test Plan Creation.

Thanks @sivassrr closing