WebXR Support

This PR removes VR APIs from engine, and adds XR support.
Currently it implements only starting/ending XR sessions (VR/AR). Input devices and reference space features will be added in another PR.
New APIs:
// pc.XrManager as 'xr' on pc.Application
app.xr // access to XrManager
// properties
app.xr.supported // true if WebXR is supported
app.xr.active // true if WebXR Session is active
app.xr.type // null or string (pc.XR_TYPE_*) of current active session
// methods
app.xr.isAvailable(type) // check if session of pc.XR_TYPE_* is available on current platform
app.xr.sessionStart(camera, type, [callback]) // attempt to start XR session for CameraComponent and of pc.XR_TYPE_*. With optional callback that can return Error if failed to start.
app.xr.sessionEnd([callback]) // attempt to end XR session. With optional callback with Error if failed to end.
// events
app.xr.on('available', function (type, available) { } ) // event fired when availability of specified type is changed. Where type is one of pc.XR_TYPE_* and available is a boolean.
app.xr.on('available:[type]', function (available) { } ) // event fired when availability of specified type is changed. Where [type] is one of pc.XR_TYPE_* and available is a boolean.
app.xr.on('session:start', function () { } ) // event fired when session is started
app.xr.on('session:end', function () { } ) // event fired when session is ended

// pc.CameraComponent
entity.camera.startXr(type, [callback]) // start session from specified camera component, where type is one of pc.XR_TYPE_*. With optional callback that can return Error if failed to start.
entity.camera.endXr([callback]) // attempt to end XR session. With optional callback with Error if failed to end.

Fixes #1748
I confirm I have signed the Contributor License Agreement.

Hm, noticed one issue with Box Projected CubeMaps, they are not projected correctly in VR.

Interesting. Screenshot?

Interesting. Screenshot?

Fixed normals issue.
I believe this is ready now.

This is great - we're so close to merging. Last outstanding thing for me:
If I merged this and deployed as v1.25.0, what will VR developers face? Especially those with PlayCanvas projects based on the old WebVR integration?

Do we need a migration guide for existing PlayCanvas VR apps?
What about VR controllers? What has to be done to re-enable those?

If I merged this and deployed as v1.25.0, what will VR developers face? Especially those with PlayCanvas projects based on the old WebVR integration?

Currently this PR only enough for HMD (including mobile). WebVR is getting deprecated in most browsers, new Chrome wont even have its support behind the flag.

Do we need a migration guide for existing PlayCanvas VR apps?

We will need migration guide once we have all the functionality in place.

What about VR controllers? What has to be done to re-enable those?

Previously, WebVR had controllers implemented through GamePad API. WebXR implements own interfaces to interact with physical devices (even mouse and touch) and is done through XRInputSource.
We can deprecate WebVR later, and deploy this, allowing developers to play with WebXR straight away. Later deprecate WebVR, as browsers slowly deprecate its APIs as well.

Updated: WebVR is brought back. So we will deprecate it later.
And slightly simplified API.

I definitely like how you've simplified some of the API new - nice move.

Tested on Oculus Quest, works as intended.


Fixed examples documentation

Description

Examples documentation has been invalid since the examples have been migrated to ion-examples and are now fixed

Reference issue
Fixes #216

awesome thank you!


Do not print entire block in log message

This log message prints an entire block which makes the logs unreadable. It should only print a block hash or block pointer.
There might be other log messages that do that, too.

Fixed in #3125


The Gradle build should use component metadata rules to adjust dependencies of external modules

At the moment the build uses the Client Module concept to override dependencies of external modules in dependencies.gradle. Once #3157 is implemented, that new mechanism should be used instead and the client modules should be deprecated.

Started to try out some new things: https://github.com/gradle/gradle/tree/jjohannes/dependency-management/gradle-build-with-metadata-rules

@jjohannes what's left to call this complete?

I think this is complete. The Gradle build now uses component metadata rules instead of client modules and @jar dependencies in dependencies.gradle.
https://github.com/gradle/gradle/commit/f4d7dde27b74f734e124d72bfaf5ca5ca995b799
We will continue to dogfood further improvements as part of separate issues.


build pdf error when rmd files contains no r code trunk

I replaced the .rmd files with my own files which contain no r trunks.While I try to build the book, gitbook(html) is ok, but pdf will report an error.

! LaTeX Error: Environment Shaded undefined.
Error: Failed to compile bookdown.tex. See bookdown.log for more info.
In addition: Warning message:
Your latexmk version seems to be too low. You may need to update the latexmk package or your LaTeX distribution.
Execution halted
Exited with status 1.

After some experiments I find that if any r trunk exists in any rmd files,the  pdf will come out as expected so I don't think this is an latex problem
I'm using rstudio 1.0.13. here's my sessioninfo

R version 3.3.2 (2016-10-31)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.1 LTS
locale:
[1] LC_CTYPE=en_AU.UTF-8       LC_NUMERIC=C               LC_TIME=en_AU.UTF-8
[4] LC_COLLATE=en_AU.UTF-8     LC_MONETARY=en_AU.UTF-8    LC_MESSAGES=en_AU.UTF-8
[7] LC_PAPER=en_AU.UTF-8       LC_NAME=C                  LC_ADDRESS=C
[10] LC_TELEPHONE=C             LC_MEASUREMENT=en_AU.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] backports_1.0.5    bookdown_0.3.9     magrittr_1.5       rprojroot_1.2      htmltools_0.3.5
[6] tools_3.3.2        yaml_2.1.14        Rcpp_0.12.9        stringi_1.1.2      rmarkdown_1.3.9002
[11] knitr_1.15.1       stringr_1.2.0      digest_0.6.12      evaluate_0.10

This stackoverflow question might be related but it's not answered yet.

Solved， same problem as #1 , should we consider to comment out this line for all documents?
Solution:

注销掉 preamble.tex 文件 \renewenvironment{Shaded}{\begin{kframe}}{\end{kframe}}

I cannot reproduce the issue. Perhaps you are using an old version of LaTeX, and maybe a different version of Pandoc, too.
> system('xetex --version')
XeTeX 3.14159265-2.6-0.99996 (TeX Live 2016)
kpathsea version 6.2.2
Copyright 2016 SIL International, Jonathan Kew and Khaled Hosny.
There is NO warranty.  Redistribution of this software is
covered by the terms of both the XeTeX copyright and
the Lesser GNU General Public License.
For more information about these matters, see the file
named COPYING and the XeTeX source.
Primary author of XeTeX: Jonathan Kew.
Compiled with ICU version 57.1; using 57.1
Compiled with zlib version 1.2.8; using 1.2.8
Compiled with FreeType2 version 2.6.3; using 2.6.3
Compiled with Graphite2 version 1.3.8; using 1.3.8
Compiled with HarfBuzz version 1.2.6; using 1.2.6
Compiled with libpng version 1.6.21; using 1.6.21
Compiled with poppler version 0.42.0
Using Mac OS X Core Text and Cocoa frameworks

> rmarkdown::pandoc_version()
[1] ‘1.17.2’

I'll make a little repo fork for demonstration

Oh wait, I see what you meant now. Sorry. I'll fix it in a minute.

Should be fixed via a3e392593b464ba31a7eceb0cd60f7e0bd112798. Thanks for the report!


Switch to setuptools entry_points

They work on Windows, unlike setuptools scripts.

Codecov Report

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

@@          Coverage Diff           @@
##           master    #153   +/-   ##
======================================
  Coverage    86.7%   86.7%
======================================
  Files          32      32
  Lines        1196    1196
======================================
  Hits         1037    1037
  Misses        159     159

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 c6ef9a9...5c75d90. Read the comment docs.


https://www.facebook.com/nickryanpedralba?mibextid=LQQJ4d

Type of issue
Other (describe below)
Description
My old facebook account was hacked i dont have the access please can you remove the gmail was link in my old facebook account thanks
[Enter feedback here]
Page URL
https://www.facebook.com/nickryanpedralba?mibextid=LQQJ4d
Content source URL
https://github.com/Azure/azure-docs-sdk-dotnet/blob/master/xml/ns-Microsoft.AspNetCore.Builder.xml
Document Version Independent Id
0004d424-fe13-43ee-87c5-e3e5133a4390
Article author
@azure-sdk
Metadata

ID: 8b9df84f-f19c-4e58-6899-75d86d5d465a

Spam


Codecov test coverage report

Codecov Report

Merging #2589 into master will increase coverage by 1.69%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2589      +/-   ##
============================================
+ Coverage     25.20%   26.90%   +1.69%
+ Complexity      394      392       -2
============================================
  Files           379      379
  Lines          7418     6723     -695
  Branches       1225     1224       -1
============================================
- Hits           1870     1809      -61
+ Misses         5417     4783     -634
  Partials        131      131

Impacted Files
Coverage Δ
Complexity Δ

...itlab/arturbosch/detekt/api/internal/Validation.kt
33.33% <0.00%> (-16.67%)
0.00% <0.00%> (ø%)

.../gitlab/arturbosch/detekt/cli/console/Colorizer.kt
61.53% <0.00%> (-9.90%)
0.00% <0.00%> (ø%)

...h/detekt/api/internal/DisabledAutoCorrectConfig.kt
55.55% <0.00%> (-8.09%)
4.00% <0.00%> (ø%)

...otlin/io/gitlab/arturbosch/detekt/core/TaskPool.kt
33.33% <0.00%> (-6.67%)
3.00% <0.00%> (ø%)

...in/io/gitlab/arturbosch/detekt/api/SingleAssign.kt
60.00% <0.00%> (-6.67%)
3.00% <0.00%> (ø%)

...in/kotlin/io/gitlab/arturbosch/detekt/api/Issue.kt
50.00% <0.00%> (-5.56%)
2.00% <0.00%> (ø%)

...n/io/gitlab/arturbosch/detekt/api/ConsoleReport.kt
75.00% <0.00%> (-5.00%)
4.00% <0.00%> (ø%)

...tlab/arturbosch/detekt/api/internal/EmptyConfig.kt
66.66% <0.00%> (-4.77%)
4.00% <0.00%> (ø%)

...in/io/gitlab/arturbosch/detekt/cli/OutputFacade.kt
77.77% <0.00%> (-4.05%)
5.00% <0.00%> (ø%)

...b/arturbosch/detekt/api/internal/DetektPomModel.kt
41.17% <0.00%> (-3.83%)
1.00% <0.00%> (ø%)

... and 259 more

Continue to review full report at Codecov.

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


feat: add test themes

TEST!

Automated Theme Preview
Hi, thanks for the theme contribution, please read our theme contribution guidelines.
We are currently only accepting color combinations from any VSCode theme or themes which have good color combination to minimize bloating the themes collection.

Also note that if this theme is exclusively for your personal use, then instead of adding it to our theme collection you can use card customization options

Test results

:x: moltack2
:x: codeSTACKr2
:x: rose_pine2

Result: :x: Some themes failed. Closing pull requests since some themes do not adhere to the theme guidelines. Please fix the following issues and we will re-open the pull request.
Details
Moltack2 theme preview

:warning: Theme name isn't in snake_case

CodeSTACKr2 theme preview

:warning: Theme name isn't in snake_case

Rose_pine2 theme preview

:warning: Theme name isn't in snake_case


Dump topics-and-qrels to reduce jar size

Old jar size: 204M
New jar size: 127M

Codecov Report
Patch coverage: 73.86% and project coverage change: +0.08 :tada:

Comparison is base (a7df7fc) 58.87% compared to head (c8ef226) 58.96%.

:exclamation: Current head c8ef226 differs from pull request most recent head e3870a7. Consider uploading reports for the commit e3870a7 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2097      +/-   ##
============================================
+ Coverage     58.87%   58.96%   +0.08%
- Complexity     1173     1192      +19
============================================
  Files           194      194
  Lines         11246    11319      +73
  Branches       1477     1486       +9
============================================
+ Hits           6621     6674      +53
- Misses         4145     4163      +18
- Partials        480      482       +2

Impacted Files
Coverage Δ

...arch/topicreader/BackgroundLinkingTopicReader.java
29.76% <ø> (ø)

...o/anserini/search/topicreader/CacmTopicReader.java
84.84% <ø> (ø)

...io/anserini/search/topicreader/CarTopicReader.java
100.00% <ø> (ø)

.../anserini/search/topicreader/CovidTopicReader.java
90.90% <ø> (ø)

...serini/search/topicreader/DprJsonlTopicReader.java
100.00% <ø> (ø)

.../anserini/search/topicreader/DprNqTopicReader.java
100.00% <ø> (ø)

...rini/search/topicreader/EpidemicQATopicReader.java
82.75% <ø> (ø)

...i/search/topicreader/JsonIntVectorTopicReader.java
0.00% <ø> (ø)

...rini/search/topicreader/JsonStringTopicReader.java
100.00% <ø> (ø)

...earch/topicreader/JsonStringVectorTopicReader.java
0.00% <ø> (ø)

... and 14 more

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

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

I'm getting an ERROR:
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.176 sec
Running io.anserini.collection.Cord19FullTextCollectionTest
2023-04-09 05:36:00,849 ERROR [TEST-Cord19FullTextCollectionTest.checkDocumentParser-seed#[E8A69DB855CD6CB1]] collection.Cord19FullTextCollection$Document (Cord19FullTextCollection.java:176) - Error parsing file at src/test/resources/sample_docs/cord19/sample1
null
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.178 sec
Running io.anserini.collection.EpidemicQACollectionTest

ERROR above is unrelated, patched with #2098


Include more packages in TSDocs/API docs build

The TSDocs are currently exempt from the broken link checker because there were so many broken links. The reason is that there are some packages that aren't included in the TSDocs currently, leading to broken links. Long-term, these packages should be refactored, but for now, this PR just adds all the needed packages to the docs build so that the links are fixed.
Note that #7469 and #7470 are also both required in order to fully enable link checking on the API docs.

@skylerjokiel I'd like your input on these package additions. The reason they're needed is that they contain stuff that ends up being exposed in the docs from the packages that we incorporate into fluid-framework.
So I think the trade-off is:

Do nothing for now (i.e. reject this PR) and live with broken/uncheckable links in the /apis section.
Take this PR and do follow-up work to reduce the exported API in fluid-framework.

What do you think?

I'm merging this in, but I will follow up with another PR that has a different, possibly better approach. I think there's enough value in merging this in as-is.


Not able to access the Word Document on my system using the UWP application

I am trying to edit a existing Word document using the UWP app (Universal Windows). But for some reason I am getting "File does not exist" error.
I have tried using the below code to access the word document:
using(WordprocessingDocument wordDoc = WordprocessingDocument.Open("C:\Users\Public\Desktop\Doc1.docx", true))
{
}
I am getting the below error:
System.IO.FileNotFoundException: 'Could not find document'
I have checked there are no typos in the file path. The same code(above specified) is working when I implement it on a normal Winforms code.

This is not an issue with a sample in this repo. You can open a support case with our Developer Support team:

Visit http://aka.ms/storesupport and sign-in with your Microsoft account
Under the “App Development” section choose "Windows 10 universal app development", then choose the appropriate problem type and category to get engaged through a formal support channel.

(My guess is that your app doesn't have access to that folder. Try using a FileOpenPicker.)


Shortcut autocomplete ( ⌃Space ) doesnt work

Description

Im trying to use Shortcut autocomplete ( ⌃Space ) on macOs for filter field
Expected result: Pop-up with Columns Names from the current Table is shown
Actual result : nothing happens (on windows Ctrl+Space do the same and it works)
DBeaver Version
23.0.4
Operating System
macOs monterey 12
Database and driver
Vertica
Steps to reproduce
No response
Additional context
No response

I don't think it is actually a bug, because that happens because MacOS understands ^Space as language change.
Workaround for this is to disable this shortcut in MacOS shortcut in preferences or change the content assist shorcut in DBeaver.


Removed zeromq (not a dep) and cppinterop (use run exports instead)

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


⚠ PrivateHD has degraded performance

In a48bc99, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2655 ms

Resolved: PrivateHD performance has improved in 640a251.


FastAPI/Uvicorn crashes every day around the same time, different APIs, different providers

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
app = FastAPI()

@app.get("/", response_class=HTMLResponse)
async def read_item(request: Request, pac_id: str = None, _oauth2_proxy: str = Cookie(default="")):
   pass

Description
Three different FastAPI's are crashing every day around the same time (different times per API), running Dockerized on Kubernetes with Uvicorn.
It is not DDOS or anything, as the APIs are using Basic Auth and are behind an nginx proxy which handles authentication.
It is not DB dependent, as one uses MongoDB, one PostgreSQL, one no DB.
Providers are Hetzner, AWS and Advin (Frankfurt, Nürnberg, Amsterdam)
The docker container has a huge CPU and Disk usage before Kubernetes "resets" the container, which causes a downtime of ~5 minutes every day.
I read in some tickets that people experience something similar with uvicorn - is that a problem?
I'm running the app in the docker image with: CMD ["uvicorn", "main:app", "--host", "<IP_ADDRESS>"] and am using python:3.10-slim
Any recommendations? I would say either a FastAPI problem or Uvicorn
Operating System
Linux
Operating System Details
Kubernetes 1.24.9
Docker Image 3.10-slim
Linux Host (Ubuntu Server 22.04)
FastAPI Version
0.89.0
Python Version
3.10.9
Additional Context

Do you have the access logs from 9:41 to the time it shut down?
Can you show me the signature of those endpoints?

yeah that is the thing, no logs from FastAPI except the typical "GET / 200" (nothing more than usual), and then the Kubernetes message "Container shut down", and the new one spinning up
The signature? Like a model how the architecture looks like?

Hello, you set a health check in your api and your pod config?

@jonatasoli yes, healthcheck and liveness probe - do you think this might cause it?
  livenessProbe:
            httpGet:
              path: /new
              port: 8000
            initialDelaySeconds: 5
            periodSeconds: 60
          startupProbe:
            httpGet:
              path: /new
              port: 8000
            initialDelaySeconds: 5
            periodSeconds: 10

@Kludex yes will post something later, need to anonymize first... It's pretty basic though, so the usual get post stuff

and yes i'll implement https://github.com/trallnag/prometheus-fastapi-instrumentator now and then I can send you the exact requests before it crashes tomorrow

Any news?

thanks for asking, no at least not on python log leveling... i think somehow fastapi starts using up a lot of ram, which triggers the container kill from kubernetes.... so strange! any ideas how I can get logs that are not shown by the fastapi application? there aren't even requests coming in except the usual healtchecks

@JustinGuese what about OOM Killer? requests & limits of resources configured in Kubernetes manifest?

You are running app directly using uvicorn command. According to https://www.uvicorn.org/deployment/, when deploying to production you should use gunicorn -k uvicorn.workers.UvicornWorker. I also looked up a few examples of docker combined with fastapi, most of them are using gunicorn with uvicorn as a worker class for production.

Alright, thanks @yallxe , will try and report back!

yes indeed @yallxe's solution of using gunicorn instead of uvicorn solves the problem! So it's a uvicorn bug, weird, but okay! Thanks!!

yes indeed @yallxe's solution of using gunicorn instead of uvicorn solves the problem! So it's a uvicorn bug, weird, but okay! Thanks!!

What probably happens now is that gunicorn will restart the workers, but the problem should still exists.

@Kludex actually I think not. I have tested it in 3 deployments and so far (3 days) no crashes/restarts
But sure maybe gunicorn restarts them so quickly that no traces are seen


[TEST] Fix integration test test_engine_live_upgrade_rollback_with_backing_image

What's the test to develop? Please describe.
I make a check and it's not related to backup refactoring.
Create a bug to track and fix this test case failure in the future.
Ref #2876
Describe the items of the test development (DoD, definition of done) you'd like.

[ ] Make the test PASSED

Did not see it at nightly CI build recently, close it.


acorn images -A lists images only from the first project in acorn projects.

Server version - acorn version v0.5.1-5-g28a7873+28a78730
Steps to reproduce the problem:
Have few projects created with images in each of the projects.
p1 - image1
p2 - image2
acorn images -a lists images only from the first project in acorn projects.
in the above case , only image1 will be listed.
% acorn images -a -A
REPOSITORY   TAG       IMAGE-ID
<none>       <none>    d5bae66abc18
<none>       <none>    2c7ba01b4cc9

 % acorn projects
NAME           DEFAULT   DESCRIPTION
mynewproject
acorn          *
mytest2
test

% acorn images -a -j mynewproject
REPOSITORY   TAG       IMAGE-ID
<none>       <none>    d5bae66abc18
<none>       <none>    2c7ba01b4cc9
 % acorn images -a -j acorn
REPOSITORY                              TAG       IMAGE-ID
<none>                                  <none>    d5bae66abc18
<none>                                  <none>    371cd2fcba06
<none>                                  <none>    17f2d0dfba2e
<none>                                  <none>    ac725dc8ea46
<none>                                  <none>    b32144f8faa1
index.docker.io/sangeetha/mytestacorn   v0.0.2    d02d3c8d9a22

Also , In the output for acorn images -A , we should be able to show which project the images belong to  like <project-name>/<image-id>

Based on out-of-band convo, this will probably get addressed holistically as part of the PR for project::resource. I'm going to move this issue back to "in progress" and it can be moved forward when that PR lands.

moving out to v0.8.0

We got rid of -A as a flag for acorn images. Its now just a command specific flag (i dont know all the commands that support it, but for acorn ps does)


Nested matching and handling

It is a common case that the client sends the HTTP traffic in a TLS session and the proxy forwards it upstream in PROXY protocol. I want to match PROXY protocol and then match TLS session, if the match fails then we should fallback to next route. To the best of my knowledge, there is currently no way to achieve such effect. I propose we implement a special handler called nested to handle such case.
Below is an example configuration. Assume that we have already implemented a matcher called proxyprotocol which matches all traffic in PROXY protocol and a handler called proxyprotocol which turns PROXY protocol traffic into normal TCP traffic. The below configuration instructs caddly-l4 to first matches proxy protocol, then unwraps PROXY protocol traffice and handles the unwrapped traffic with the specified routes, which in turn try to match it with tls and then terminate tls, if that fails, match it with the http matcher.
If the entire nested matcher failed to match the connection, we should fallback to the next route. My configuration below makes caddy-l4 accept both traffic in PROXY protocol and in normal TCP connection.
{
  "apps": {
    "layer4": {
      "servers": {
        "example": {
          "listen": ["<IP_ADDRESS>:5000"],
          "routes": [
            {
              "match": [
                {
                  "proxyprotocol": {}
                }
              ],
              "handle": [
                {
                  "handler": "proxyprotocol"
                },
                {
                  "handler": "nested",
                  "routes": [
                    {
                      "match": [
                        {
                          "tls": {}
                        }
                      ],
                      "handle": [
                        {
                          "handler": "tls"
                        },
                        {
                          "handler": "proxy",
                          "upstreams": [{ "dial": ["localhost:8080"] }]
                        }
                      ]
                    },
                    {
                      "match": [
                        {
                          "http": [{ "host": ["example.com"] }]
                        }
                      ],
                      "handle": [
                        {
                          "handler": "proxy",
                          "upstreams": [{ "dial": ["localhost:8080"] }]
                        }
                      ]
                    }
                  ]
                }
              ]
            },
            {
              "match": [
                {
                  "tls": {}
                }
              ],
              "handle": [
                {
                  "handler": "tls"
                },
                {
                  "handler": "proxy",
                  "upstreams": [{ "dial": ["localhost:8080"] }]
                }
              ]
            },
            {
              "match": [
                {
                  "http": [{ "host": ["example.com"] }]
                }
              ],
              "handle": [
                {
                  "handler": "proxy",
                  "upstreams": [{ "dial": ["localhost:8080"] }]
                }
              ]
            }
          ]
        }
      }
    }
  }
}

I think you're looking for subroute? Or is that not implemented for l4 yet? If not then that should probably be its name to be consistent with Caddy's HTTP app.

Yeah. That's what I want. I don't think it is implemented in l4.

@mholt I'm interested in working on this issue. This is my first issue in any module related to Caddy, am someone who's interested in knowing the implementation details of a proxy solution, hence the interest to contribute.
I did play around a bit generally with Caddy to understand the terminologies, architecture etc.. Also, spent some time reading this L4 app codebase. With that, I think, I understand the ask here, but have a few questions-

there seems to be two parts to the ask here; one is implementing a matcher that can match against PROXY protocol header and another is the ability to do subroutes with handlers.

does #27  cover the first ask of matching against incoming PROXY protocol header?

I understand that, both these features can be implemented independently, but do you have a preference on which one should be implemented first? I'm open to pick either of them.

Also, please feel free to correct my understanding where necessary.

there seems to be two parts to the ask here; one is implementing a matcher that can match against PROXY protocol header and another is the ability to do subroutes with handlers.

I think that is correct. They are very different things to implement, as you say. Should definitely be two separate PRs.

does Proxy Protocol Support #27 cover the first ask of matching against incoming PROXY protocol header?

I don't think so.

I understand that, both these features can be implemented independently, but do you have a preference on which one should be implemented first? I'm open to pick either of them.

The matcher is certainly easier. Probably would be a good place to start.
Thanks for your interest! Looking forward to some reviews. :)

Opened a PR (^^) for the matcher, could you review please @mholt ?
Also, if you could point me in the right direction for the below, that would be great.

not sure yet on how to build caddy locally that includes changes to a module. This instruction in README doesn't work me, I don't see the layer4 modules, if I build that way.

In the project folder, run xcaddy just like you would run caddy. For example: xcaddy list-modules --versions (you should see the layer4 modules).

Not familiar with how could we test this module which operates at layer4? Do we use certain tools to simulate traffic? Although this matcher is very simple, would like to test that it works, to be sure.

@MadhuvanthG

not sure yet on how to build caddy locally that includes changes to a module. This instruction in README doesn't work me, I don't see the layer4 modules, if I build that way.

Those instructions build with my (mainline) caddy-l4, not your fork. For developing plugins, follow the instructions on the xcaddy README: https://github.com/caddyserver/xcaddy#for-plugin-development

Not familiar with how could we test this module which operates at layer4? Do we use certain tools to simulate traffic? Although this matcher is very simple, would like to test that it works, to be sure.

I often use telnet or netcat (nc). But yeah you could also programmatically test it.

Thank you @mholt for the help, I was able to build and test the changes :) Have addressed other comments in the PR as well, please review when you get time.


[DDW-461] Fix flickering issue on Network Status screen syncing chart

This PR fixes the flickering issue on Network Status screen syncing chart.
Screenshots:

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (yarn run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[ ] There are no flow errors or warnings (yarn run flow:test)
[ ] There are no lint errors or warnings (yarn run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running yarn run manage:translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn run storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

@daniloprates code part looks great!
I have asked @DmitriiGaico to test this in the Testnet builds and to approve the PR if it works.

Perfect! Please go ahead @DominikGuzei 🚀


Normalization of Regularization?

In most frameworks we are using regularization which is automatically scaled to the number of features we have. There are rare cases where the training using non-normalized weights are causing issues, such as:

Not having direct access to the server which trains a model on significantly more data than locally
No direct way to query the number of observations to train
Unknown/unspecified number of observations for the future training data

It causes a deterioration of performance of LightGBM, especially when regularization is a required element for getting good performance.
Example, using AUC, where Reg=Regularization Weight=scaled regularization:

Number of observations
Weighted Reg
Unweighted Reg
No Regularization

1,000
0.765
0.765
0.714

10,000
0.812
0.797
0.728

100,000
0.843
0.788
0.739

1,000,000
0.854
0.759
0.747

10,000,000
0.861
0.745
0.749

In this case, a multiplication of 1,000 in the number of observations dropped the performance of the regularization close to as if there were no regularization (and by 10,000, identical).
xgboost recently implemented weighted regularization using the number of observations to have a consistent regularization: https://github.com/dmlc/xgboost/pull/3103

@Laurae2 it seems the xgboost's pr is for linear model, and LightGBM doesn't have linear models.
can you explain the idea with more details ? And where did you get the comparison table ?

And where did you get the comparison table ?

I'm using a private dataset which has very noisy features.

can you explain the idea with more details ?

Instead of doing regularization with absolute scaled values, regularization is set depending on the number of training observations to scale the gain/leaf value.
For instance, the gain and leaf calculation in GBDT should be the following by default:

value = [ ((sum(gradients) - L1)² / (sum(hessians) + L2)] (with sign for leaf value)

I wonder if it makes sense to have the following instead:

value = [ ((sum(gradients - L1))² / (sum(hessians + L2))] (with sign for leaf value)

As the sum of gradients/hessians are supposed to grow bigger with the number of observations, the impact of L1/L2 regularization is smaller. Issue is that moving L2 inside the sum means we are not directly assessing anymore the tree structure, it also breaks the maths behind boosting.
For linear regression this is fine but for GBDT it seems not directly possible. I tried some modifications to force the L1/L2 inside the sums and results were worse in general.

thanks @Laurae2 , got it.
I am aware about this problem, however, I think user should know the regularization of L1/L2 will be weaken when using large samples.
If we want it to be auto adaptive, we will have following new problems:

how do we scale the L1/L2 with number of samples ?
is that scale really needed ? or is regularization needed when #samples is large ?

@guolinke I'll be running a large experiment next week on Higgs to assess how regularization behaves when the number of observation changes (along with hyperparameter tuning). We might get practical hypotheses on a proper scaling from it.

@guolinke L1, L2, and hessian regularization does not seem to scale linear/log with the number of observation (although the optimal value increases with the number of observations), I think it's good for only for GLM-type models and not tree-based models.
No conclusion so far.


update skim to 1.4.17

Thank you for the contribution. It was merged directly as 1701a00402991f54ae7a25f07252011ad1305e74 to keep commit history cleaner. Your contribution is still credited to you.


Add  always-unzip and no-deps options to install command

Originally reported by: jaraco (Bitbucket: jaraco, GitHub: jaraco)

In pip 1749, it appears that setuptools could better support the use cases of pip by providing two new options, --always-unzip and --no-deps, to the install command.
What are the implications of adding such options? Would they simply fall through to easy_install? Could the same behavior be invoked using existing behavior, such as through setopt, e.g.:
python setup.py setopt -c easy_install -o always-unzip -s 1 install

While ugly, I believe that behavior would work today.

Bitbucket: https://bitbucket.org/pypa/setuptools/issue/206

I also have a usecase where I would like to be able to do
setup.py install --single-version-externally-managed --root --no-deps.
I am currently using --single-version-externally-managed and --root as well but it is not clear to me if these bring in dependencies or not (I would like them not to). The documentation doesn't say anything about it.
I think I have seen them bringing in dependencies (Cant be sure, I have been switching a lot between different versions of setuptools) but it seems they don't (https://github.com/pypa/setuptools/issues/570) ?

Pip non longer support --egg.


Improve type readability (fix type alias propagation, allow aliasing inferred types)

Suggestion

🔍 Search Terms
preserve type aliases,inferred types,type names,readability,type expansion
✅ Viability Checklist
My suggestion meets these guidelines:

[*] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[*] This wouldn't change the runtime behavior of existing JavaScript code
[*] This could be implemented without emitting different JS based on the types of the expressions
[*] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[*] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
One thing that makes working with Typescript's type system occasionally hopeless is when cumulative inferred types are displayed in type expansion and 'pollute' the information in quick info or errors.
So the proposal is two fold:

Fix type expansion so that the names of type aliases are preferred over their expanded inferred types wherever a type alias is available.
Add syntax that allows us to 'alias'/name inferred types and allow those alias names to carry through type expansion.

📃 Motivating Example
With any kind of sophisticated typing, we quickly end up with error messages and parameter info that takes multiple seconds to load, and shows a truncated mess of type syntax. Worse, the expansion of these inferred types accumulates and adversely affects any code that touches them.
Typescript fails us in two distinct areas with regard to type expansion, and the culprit seems to be inferred types...
1. Typescript doesn't always respect type aliases, even when they are explicitly provided
E.g. First, what works:
// Let's start with a contrived mapped type:

type Mapped<T> = {
  [P in keyof T]: never;
};

// And a generic function that takes an argument and returns the mapped type of it.

function map<T>(value: T): Mapped<T> {
  return {} as never;
}

// Create a Person via inference
const p = {age: 5, name: 'Bobby'};

// And quick info on 'inferred' displays: Mapped<{age: number, name: string}>
const inferred = map(p);

// Let's clean up the inferred type by defining a type alias.
type Person = {age: number; name: string};

// Now quick info on 'named' shows: Mapped<Person>
const named = map({age: 5, name: 'Bobby'} as Person);

So far, so good. The rule seems to be that if we explicitly alias a type, Typescript will use that name and terminate expansion.
But this rule gets broken when there is logic to the type alias.
// Use `io-ts` as an example and declare runtime types/validators like so:

import * as t from 'io-ts';

const Person = t.type({
  name: t.string,
  age: t.number,
});

// And infer the underlying data type from it...
type Person = t.TypeOf<typeof Person>;

// Now, let's put them through the same Mapped type and map function...

// We use the type alias 'Person' in the same manner to try to 'name' the type, but...
const named = map({age: 5, name: 'Bobby'} as Person);

// Quick info on 'named' is: Mapped<{name: string, age: number}> not Mapped<Person>

You could argue that this is handy, but the consequences of this deviation from the rule means that all other types referencing this one will repeat the full expansion.
2. There is no way to name an inferred type
There is also a scenario where fully expanded inferred types are unavoidable. More specifically, where there is no way to explicitly specify a type alias whose name should be displayed. E.g.
// Let's return to our Person type and try some composition:

const Family = t.type({
  mother: Person,
  father: Person,
  child: Person,
});

// The type of 'Family' is:  t.TypeC<{mother: t.TypeC<{name: t.StringC, age: t.NumberC}>, father: t.TypeC<{name: t.StringC, age: t.NumberC}>, child: t.TypeC<{name: t.StringC, age: t.NumberC}>}>

That's already pretty out of hand. All we would need to know, ideally, is something like (pseudocode) t.TypeC<{mother: Person, father: Person, child: Person}>.
But since the type of Person is inferred from the function's return type, we don't get a chance to create a type alias for it. In fact, that's the reason we're able to declare a type alias of the same name for the underlying data type.
So wherever we use the Person instance, we're now stuck with a fully expanded inferred type. If we were able to provide Typescript with an alias for the inferred type, we could escape expansion and clean up type expansions everywhere.
Proposed as type syntax
Such syntax could work like this:
// Combining the syntax for type assertion and type alias declaration:
const Person = t.type({
  name: t.string,
  age: t.number,
}) as type PersonType;

So that Family would then display: t.TypeC<{mother: PersonType, father: PersonType, child: PersonType}>
Since the above kind of factory that returns an inferred type is a common pattern, it could also be shorthanded like this:
const type PersonType = t.type({
  name: t.string,
  age: t.number,
});

...so that the name of the variable would double as the name of the type alias.
There could even be syntax that is used upstream by the function returning the inferred type...
function map<T>(value: T): Mapped<T> as type new {
   return {} as never;
}

// Here, the type of `MyMap` is `MyMap` (it gets aliased by default with the same name).
const MyMap = map({ age: 22, name: 'Bobby});

The MyMap alias expanded is Mapped<{ age: number, name: string }>
// We could also allow for when wrapping is needed...
function get<T>(value: T): Wrapper<T as type new> {
   return {} as never;
}

// Here Person instance is typed as Wrapped<Person>, where Person is { age: number, name: string }
const Person = get({ age: 22, name: 'Bobby' })

💻 Use Cases
I think I've covered quite specific use cases in my motivating example.
Leveraging type inference to construct sophisticated types is common enough now that we should pay particular attention to developer experience for type systems.
These two changes would especially allow both developers and library authors to have better control over the types their code emits to intellisense and allow them to craft an improved developer experience.
Specifically, we could use libraries like io-ts and zod without polluting our type intellisense.

This could be super helpful for some crazy generic code in some of the libraries. I imagine if we could use template literals as the "dynamic" type name this could be even more useful.

This would be great. I have a generic type that turns classes to plain interfaces (useful to work with the library class-transformer). a type that could be Plain<Chair> turns into a horribly long mapped type that, as it is so big, it ends up being unhelpful.

This would be incredibly helpful. Typescript is great to onboard newcomers to projects, but complex type expansions unfortunately end up scaring many junior programmers, even when the types themselves would be easily readable with a system like this.

The thing is that the Typescript compiler has been architected to describe types in full semantic accuracy, rather than communicate types to developers with full clarity. If we had a way to to provide names to inferred types, then we could communicate a developer's intent far more clearly.
In languages like C#, types come first for better or worse, so types become a defacto channel for developers to convey their intent. Developers spend proportionally more time thinking about type names and documenting them.
Such a shift would only benefit the Typescript ecosystem.

I often have the opposite problem and would prefer to read { [K in keyof Person]: never } or even { age: never, name:  never } over Mapped<Person> because those type transformers stack up and at some point you don't know anymore which type that mess computes to. Additionally, when you reach that point, utils such as Compute from ts-toolbelt no longer work and just add to the stack.
I agree that the problem identified in the OP is real but I wanted to point out that it's one end of a spectrum, and I would rather have a language server option like displayMode: 'preferAliases' | 'preferExpansion' I could switch on and off, or maybe that compiler option could come in addition to this feature in order to counterbalance.
I also want to add that this piece of syntax would be devoid of semantic value, which on top of being a little weird in a type system would add bloat to TS which is already extremely verbose when types get complex.
My final observation would be that TS is not a semantic tool: it's a programming language to hack types on top of unruly Javascript. I understand that one would prefer it to be differently, but that way of thinking caused me the greatest pain when I picked up the language. TS is not Haskell or whatnot, it looks procedural because it is.

Would it be possible to develop this as an extension to Intellisense(?) or at least VSCode so that it's possible to toggle between the two views? I don't know whether anything would be required from Typescript itself for that.

I don't think the syntax changes are needed, but the better alias inference would be great. Here's a minimal example (notably, this has no usage of libraries like io-ts):
type RawNumArray = number[];                                    // number[]: good!
type RawNumMatrix = Array<RawNumArray>;                         // RawNumArray[]: good!

type InferNumArray = [number[]] extends [infer T] ? T : never;  // number[]: good!
type InferNumMatrix = Array<InferNumArray>;                     // number[][]: Not good :(

This shows two aliases that resolve to the same type, but only one of them is compatible with the alias name preservation work done in #42149 and  #42284 . I believe the issue is specific to type aliases that infer a non-primitive type (hence my use of number[] instead of number in this example).
Outside the language level, it would be great if at the language server level types (aliases and otherwise) could be viewed as an expandable tree. That would hopefully be a happy medium for both the use cases suggeted by @martaver and @geoffreytools .


Update to v3.1.0

Hey, I just want to ask/request, when do plan to update this server project to v3.1.0 version which is released in the main/general MQTTnet repo?
There is nice amount of fixes  stated in MQTTnet v3.1.0 changelog.
Thanks

I have no plan yet but the library will be updated soon.


"TopicAuthorizationException: Topic authorization failed" for MSK

Describe the bug
I got the error as below when I config Kafka-ui via helm chart and connect to MSK. As image below, I successfully auth to MSK, but seem be to topics is not.

Set up

I used helm for install Kafka-UI. This is my helm value file:
replicaCount: 1
fullnameOverride: "kafka-ui"

existingConfigMap: ""
existingSecret: ""
envs:
  config:
    KAFKA_CLUSTERS_0_NAME: msk-g1-dev
    KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: "msk-broker-url:9096"
    KAFKA_CLUSTERS_0_PROPERTIES_SECURITY_PROTOCOL: SASL_SSL
    KAFKA_CLUSTERS_0_PROPERTIES_SASL_MECHANISM: SCRAM-SHA-512
    KAFKA_CLUSTERS_0_PROPERTIES_SASL_JAAS_CONFIG: 'org.apache.kafka.common.security.scram.ScramLoginModule required username="my_admin_user" password="my_pass_word";'

Steps to Reproduce
Steps to reproduce the behavior:
Expected behavior

Screenshots

Additional context

Hey, thanks for reaching out.
Th answer is simple -- your user doesn't have required permissions for some topic(s).
Try these MSK permissions:
      "kafka-cluster:Connect",
      "kafka-cluster:Describe*",
      "kafka-cluster:CreateTopic",
      "kafka-cluster:AlterGroup",
      "kafka-cluster:ReadData"

Thanks @Haarolean , I have to use IAM role for ACLs instead of SASL/SCRAM user

Glad it worked :)


When drag inertia > 0, the camera jumps a bit when the mouse stops moving and left click is released

The problem occurs when drag inertia is greater than 0.
Expected behavior: When the mouse stops moving and left click is released, the camera should stay in place.
What happens instead: When the mouse stops moving and left click is released, the camera jumps a bit.
Additional notes
While messing around with the script, I took a look into the Inspector and noticed that the _drag_movement variable doesn't set itself to 0 when the mouse stops moving.
A minimal reproduction project and a video showing the issue are provided below.
https://github.com/user-attachments/assets/d49f3d22-8362-40a7-ae8d-69d3bf07d082
camera_drag_is_broken.zip

Can you modify the script so that _drag_movement sets itself to 0 when the mouse stops moving?

It stays in place when there's no movement while releasing the left mouse button. You should be able to add some code at line 86 so that _drag_movement is set to 0 when there is very little movement, something like:
if _drag_movement.length() < 3:
    _drag_movement.x = 0
    _drag_movement.y = 0

Hey, it's been a while, but I've finally solved this problem for real. It didn't occur on my device (MacBook with trackpad), but a friend of mine had it too and it makes a lot of sense after all. I've just pushed an update to the script, which now only recognises throws that were quick and far enough. Asset Library version 1.2.2, pending approval!


Docker error

Hi, please take a look at the problem i got after executing cmd
docker compose build inside silly-dump repository.
The error:
=> [app stage-0  2/13] RUN apt-get update && apt-get install -y     curl     g++     git     libbz2-dev     libfreetype6-dev     libicu-dev     libjpeg-dev     libmcrypt-dev    66.6s  => [app stage-0  3/13] RUN echo "ServerName web-app.local" >> /etc/apache2/apache2.conf                                                                                0.2s  => [app stage-0  4/13] RUN sed -ri -e 's!/var/www/html!${APACHE_DOCUMENT_ROOT}!g' /etc/apache2/sites-available/*.conf                                                             0.3s  => [app stage-0  5/13] RUN sed -ri -e 's!/var/www/!${APACHE_DOCUMENT_ROOT}!g' /etc/apache2/apache2.conf /etc/apache2/conf-available/*.conf                                        0.2s  => [app stage-0  6/13] RUN a2enmod rewrite headers                                                                                0.2s  => [app stage-0  7/13] RUN mv "/usr/local/etc/php/php.ini-development" "/usr/local/etc/php/php.ini"                                                                               0.3s  => [app stage-0  8/13] RUN docker-php-ext-install     bcmath     bz2     calendar     iconv     intl     mbstring     opcache     pdo_mysql     zip                             131.1s  => [app stage-0  9/13] COPY --from=composer:latest /usr/bin/composer /usr/bin/composer                                                                                0.0s  => ERROR [app stage-0 10/13] RUN useradd -G www-data,root -u  -d /home/my_user my_user

Thank you, everything works right now.

Steps to reproduce:

Open app on localhost:8080
Login on user which hasn't been created.

No information what is valid password


Handle EOF panic when device is removed

As of this writing, when the serial device is removed after riftd starts, the main incoming packet processing loop panics, as its underlying read hits an EOF. Because this runtime panic is unhandled, this blows the whole process, without giving other goroutines time to clean up their resources.
Instead, we should handle the EOF and have it shutdown the exchange gracefully, such that we can avoid leaking any open sockets/fds.

E.g.
2017/04/14 18:58:43 18:58:43.573 ▶ ERRO 00f Error reading incoming packet: EOF
2017/04/14 18:58:43 18:58:43.573 ▶ INFO 010 Shutting down exchange
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x52a4cc]

goroutine 7 [running]:
panic(0x5eb1a0, 0xc420012110)
	/usr/lib/go/src/runtime/panic.go:500 +0x1a1
github.com/jerluc/gobee.(*FrameBuffer).WriteFrame(0xc420013730, 0x0, 0x0, 0x0, 0x0, 0x0)
	/home/jl/dev/src/github.com/jerluc/gobee/buffer.go:25 +0x3c
github.com/jerluc/gobee.(*Mailbox).Outbox.func1(0xc420074840, 0xc4200301f0)
	/home/jl/dev/src/github.com/jerluc/gobee/mailbox.go:36 +0x78
created by github.com/jerluc/gobee.(*Mailbox).Outbox
	/home/jl/dev/src/github.com/jerluc/gobee/mailbox.go:38 +0x67


Fix #144: Add sub_registry_with_labels method to Registry

Added sub_registry_with_labels method per discussion in https://github.com/prometheus/client_rust/issues/144
Updated the tests (which should be refactored or better documented I think, it's a bit hard to figure out the hierarchy or registries)

@mxinden any update/feedback on this?

Change looks good to me. Thanks. Though now that we have sub_registry_with_labelS we can simply sub_registry_with_label, right? What do you think?

Thanks for the suggestion! I updated the code.
Tests might need some extra work (I think individual tests per method would also be useful, alongside the existing one), but I will probably do that in a separate pull request.


Error details are missing on Firefox

On Firefox, when an assertion fails, there are no error details available, so a user can't tell what caused a test to fail.

Fixed with 47b89049f110dfb0eec98699a4bf87c9dc84bb33


Add checks for Python 3.10 (beta)

Coverage remained the same at 52.738% when pulling 93bf3226c0aefb919850059ac49a50331a6319d2 on pwt:add_py310_checks into e4b39cde97a72cb9b8896a92ebd6943c3720bb6e on SoCo:master.


New Theme: Linear

Good Morning,
I've create a new theme for Linear Desktop app.
Repo can be found here:
https://github.com/jamieshiers/Dracula-for-Linear

Hey @jamieshiers!
I'm glad to see that you create this theme! I've used Linear myself and loved the app.
Are you interested in transferring your repository to the official organization (https://github.com/dracula)? You'll continue maintaining this project, the only difference is that we can make it more visible and include it on the site (https://draculatheme.com).
I just sent you an invite to join the organization. If you accept, you'll be able to transfer the repo and we can continue with adding to the site.
Thanks for building it!

Good Morning @zenorocha!,
I've accepted your requested and transferred the repository.

It's live https://draculatheme.com/linear 🎉 🎉 🎉


[Question] How to measure the performance boost upon generating the responses with and without prefix?

With the latest addition of Prefix Cache support, it would be beneficial if someone could provide an example demonstrating the performance boost.

https://github.com/vllm-project/vllm/blob/main/benchmarks/benchmark_prefix_caching.py


Update/polish textbook algorithms tutorial from educators workshop

Thanks @balopat for the careful review! Fixed the typo and updated the install/imports.


[Bug]: error: Multiple commands produce 'RANDOME_PATH/.app'

What happened?
I created a new project with following dependency version.
mix.exs
{:live_view_native, github: "liveview-native/live_view_native", branch: "main", override: true},
{:live_view_native_swiftui, github: "liveview-native/liveview-client-swiftui", branch: "main"}
Also in xcode, I make sure that LiveViewNative dependency is pointing to main branch.
but still I am getting following error.
error: Multiple commands produce '/Users/rushikeshpandit/Library/Developer/Xcode/DerivedData/Resman-fnqbjsxsrgfenwgmqimixizklthc/Build/Products/Debug-iphonesimulator/.app'

note: Target 'Resman' (project 'Resman') has create directory command with output '/Users/rushikeshpandit/Library/Developer/Xcode/DerivedData/Resman-fnqbjsxsrgfenwgmqimixizklthc/Build/Products/Debug-iphonesimulator/.app'

note: Target 'Resman' (project 'Resman') has link command with output '/Users/rushikeshpandit/Library/Developer/Xcode/DerivedData/Resman-fnqbjsxsrgfenwgmqimixizklthc/Build/Products/Debug-iphonesimulator/.app'

Multiple commands produce '/Users/rushikeshpandit/Library/Developer/Xcode/DerivedData/Resman-fnqbjsxsrgfenwgmqimixizklthc/Build/Products/Debug-iphonesimulator/.app'

Library Version
main
Xcode Version
15.4 (15F31d)
Swift Version
5.0
On which device or simulator are you running into the problem?
iPhone
Target Device Operating System Version
iPhone 15 Pro Max
Relevant log output
No response

I have found temporary workaround for this issue as follow.
1. Select “PROJECT_NAME” from top left corner of xcode.
2. Select “PROJECT_NAME” under the target
3. Select “Build Settings” tab and Search for “Product Module Name” and set it’s value to “$(PRODUCT_NAME:c99extidentifier)”
4. Search for “Product Name” in the same window and set it’s value to “$(TARGET_NAME)”
5. Also make sure to “Swift Language Version” value to “Swift 5” by following the same process

I'll start looking into this prior to rc-2 but my immediate thought is that the new xcodegen vendored binary is the issue, as we previously were using an older release available through installation methods this vendored binary is from the repo's main branch and maybe their API changed

@rushikeshpandit I believe this is now fixed on main. Please let me know if you see it again and I'll re-open


Add support for .mjs file type recommended by Google

On Google's Web Fundamentals site, they recommend using the .mjs extension for modules:

A note on file extensions
You may have noticed we’re using the .mjs file extension for modules. On the Web, the file extension doesn’t really matter, as long as the file is served with the JavaScript MIME type text/javascript. The browser knows it’s a module because of the type attribute on the script element.
Still, we recommend using the .mjs extension for modules, for two reasons: ...

However if madge is pointed at .mjs files referencing other files, it can't resolve the dependencies.

I rather wait with adding support for this until the support for this is non-experimental in Node.JS. Closing this for now.


Chart width is jumping on series change from area to candles

Lightweight Charts Version: ^3.7.0
Actual behavior:
Width jumping on series change, see the next video:
https://user-images.githubusercontent.com/32956521/146533627-7bdf6313-e766-4b77-8b0f-b9407c8b3632.mov
Expected behavior:
No width jumping.
Steps/code to reproduce:
import React, { useCallback, useEffect, useReducer, useRef } from 'react'
import { useOnResize } from 'Core/utils/useOnResize'
import {
  ChartOptions,
  createChart,
  DeepPartial,
  IChartApi,
  ISeriesApi,
  SeriesType,
} from 'lightweight-charts'
import { useChartStyles } from './styles'
import { ChartCandlestickShape, ChartCommonShape, ChartType } from './types'

type Props = {
  className?: string
  data: Array<ChartCommonShape> | Array<ChartCandlestickShape>
  options?: DeepPartial<ChartOptions>
  type: ChartType
}

function ChartLocal(props: Props): JSX.Element {
  const { className = '', data, options, type } = props
  const [, forceUpdate] = useReducer((p: number) => p + 1, 0)
  const chartStyles = useChartStyles()
  const prevChartStyles = useRef(chartStyles)

  const chartRef = useRef<HTMLDivElement | null>(null)
  const chart = useRef<IChartApi>()
  const series = useRef<ISeriesApi<SeriesType>>()

  const handleResize = useCallback(() => {
    if (chart.current && chartRef.current) {
      chart.current?.resize(
        chartRef.current.clientWidth,
        chartRef.current.clientHeight,
        true,
      )
      chart.current?.timeScale().fitContent()
    }
  }, [])

  useOnResize(handleResize, 50)

  useEffect(() => {
    if (chartRef.current !== null) {
      if (chart.current === undefined) {
        chart.current = createChart(chartRef.current, {
          height: chartRef.current.clientHeight,
          width: chartRef.current.clientWidth,
          ...chartStyles.basic,
          ...options,
        })
      }

      if (series.current) {
        chart.current?.removeSeries(series.current)
      }

      if (type === 'Candlestick') {
        series.current = chart.current?.addCandlestickSeries(chartStyles.candle)
      } else {
        series.current = chart.current?.addAreaSeries(chartStyles.area)
      }

      if (data) {
        series.current?.setData(data)
      }

      chart.current?.timeScale().resetTimeScale()
      chart.current?.timeScale().fitContent()
      forceUpdate() // Update for tooltip
    }
  }, [chartRef.current, data, chartStyles])

  return (
    <div ref={chartRef} className={className}/>
  )
}

export default ChartLocal

@iksent can you provide complete example on https://codesandbox.io please?

Ouch. Do you already have an example with switching between candlesticks and area/line types?

No, I don't think so.

https://jsfiddle.net/iksent/rxpcd0eL/

I only ran into this problem with certain data.
Options and configs, in my opinion, do not affect this.

@iksent thanks! btw, if you add a new series first and then remove the old one it seems that it works fine. but this is quite strange for sure

I can confirm this issue is fixed, thanks a lot, @timocov!


add CSP

Here is a reference that can help: https://www.sciencedirect.com/science/article/pii/S0306261918316593


Focus suggestions, retooled

With the advent of the new focus system, I feel some new focus suggestions are in order! (Because let's be honest, spamming the suggestions folder is my reason for existing. I'm a shotgun, not a rifle.)
Most of these are obvious, but anyway here goes:
Effects:

Vector Manipulation
Redirects entity's direction of movement
Shrink / Enlarge
Changes the target entity's size
Temporal Shift (Fast / Slow)
Changes the tickrate for entities / tileentities affected
Conjure Block
Summons the targeted block
Conjure Item
Summons the targeted item
Conjure Entity
Summons the targeted entity
Apply Status Effect
Applies the specified potion effect to the target
(Could even include effects from other mods e.g. Potion Core / Additional Alchemy)
Glamour
Change target entity's appearance (perhaps through a 'glamour' status effect)
Illusion
Create 'Illusion' blocks. Can be disguised as other blocks. Goggles reveal illusions.
Dispel
Removes status effects from target entities

I've always liked the idea of Psi, but i've never played with it because thematically I found it a bit of a turn-off. I'm excited to see how deep the focus crafting mechanic in Thaumcraft 6 will be.

If you don't mind, I've got a few effect ideas of my own that I'd like to chip in here. Pardon me if my terminology is inconsistent (or incredibly melodramatic), I'm adapting to the 'foci-as-effects' part of the new system.
 With modification, it could potentially pull them towards the caster instead.
Sonic
Creates nigh-invisible blasts of intense sound that can damage and disorient enemies, vibrate and unsettle loose materials, and-- with some tuning-- be used as a remote trigger for the Arcane Ear.
Blades
Manifests various bladed, scything hunks of metal in different forms, from small cluster shots of needles/shrapnel to a heavy guillotine force that has a chance to behead one's enemies. Can potentially inflict a lingering bleeding effect.
Mental
Infests the brains of foes to make them run in fear, wildly attack nearby enemies, or become your personal bodyguard. Other effects might be possible.
Acid
Weaponizes the corroding effects of the mysterious 'liquid death' to melt enemies in their tracks with bursts of disintegrating energy. Modified, it can replicate the effects of Liquid Death and drop essentia crystals, or even have its chemical effects altered into a caustic substance that will, humorously, cause enemies to drop magical tallow out of their fat.
Healing
Draws from either the sheer aura, or the life essence of nearby plants, animals, and mobs to restore the caster's physical vitality.
Gravity
Harnesses the mysterious energies of the Shulker to influence the gravity of living things, from reversing or removing it and sending them into the air, to massively amplifying it and making any form of upward locomotion near impossible or even incredibly dangerous.
Spectrum
Colors things pretty colors. That's about it, actually.

@nnystyxx cool suggestions. The "wind" effect is kind of what I was going for with my suggestion for the "vector manipulation" effect. You could maybe change the "blades" effect so instead of metal it manifests scything winds that cut your opponent to ribbons instead or something. I like the idea of being able to get mob heads that way =3=

The idea is that the 'blades' effect creates temporary constructs kinda slapped together from vis that disintegrate moments later, like how the ice melts. So they're sort of blades, sort of really sharp bits of energy. One could describe it as wind too, though.

This issue was moved to Azanor/thaumcraft-suggestions#173


bundling: UnableToResolveError

I installed react-native-fbsdk (Facebook SDK) for the login in my app and I get this error which I am not able to understand.
error: bundling: UnableToResolveError: Unable to resolve module prop-types from C:\Users\suhas\Documents\FacebookTest\node_modules\react-native-fbsdk\js\FBLikeView.js: Module does not exist in the module map or in these directories:
C:\Users\suhas\Documents\FacebookTest\node_modules
,   C:\Users\suhas\node_modules
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:

Clear watchman watches: watchman watch-del-all.
Delete the node_modules folder: rm -rf node_modules && npm install.
Reset packager cache: rm -fr $TMPDIR/react-* or npm start -- --reset-cache.
at UnableToResolveError (C:/Users/suhas/Documents/FacebookTest/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:525:5)
at ResolutionRequest._resolveNodeDependency (C:/Users/suhas/Documents/FacebookTest/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:394:11)
at ResolutionRequest.resolveDependency (C:/Users/suhas/Documents/FacebookTest/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:141:29)
at dependencyNames.map.name (C:/Users/suhas/Documents/FacebookTest/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:145:59)
at Array.map (native)
at ResolutionRequest.resolveModuleDependencies (C:/Users/suhas/Documents/FacebookTest/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:145:42)
at Promise.resolve.then (C:/Users/suhas/Documents/FacebookTest/node_modules/react-native/packager/src/node-haste/DependencyGraph/ResolutionRequest.js:169:21)
at process._tickCallback (internal/process/next_tick.js:109:7)
Bundling index.android.js  93.0% (408/423), failed.

I am using Windows 10 and I am getting the same error even after running the following commands:
del node_modules && npm install
npm start -- --reset-cache

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


Admin Logs - Add Log Setting returns 401 for admins

Description of bug
Add Log Setting option in Admin Logs returns 401 for admins.
Steps to reproduce

Open MANAGE > Admin Logs > Log Settings
Click on "Add Log Setting"
Enable the Logging switch, set Log Type to "Application Shutting Down"
Click on "Save"

Current behavior
A "success" notification pops up saying "The log setting could not be added. Please try again later."
Expected behavior
Log setting should be added without issues, but in case an issue occurs, it should be displayed as an error, not as a successful operation.
Screenshots

Error information
Following error is shown in the browser's console:
Failed to load resource: the server responded with a status of 401 (Unauthorized)

and occurs when PB issues a POST request to /API/PersonaBar/AdminLogs/AddLogSetting
Additional context
It seems this behavior was introduced by this PR: https://github.com/dnnsoftware/Dnn.AdminExperience.Extensions/pull/231 three years ago.
Looking in the original ticket, it seems some QA test was expecting Admin users not to be able to add Admin Log settings and therefore this condition was added, although it doens't make any sense to me.
Anyone have a clue why admins should not be able to add admin log settings?? and whether this is a mistake or it is expected behavior?
Affected version

[X] 10.00.00 alpha build
[X] 09.08.01 release candidate
[X] 09.08.00 latest supported release
and all the way down to 9.2.0

Affected browser
All browsers.

That doesn't make any sense to me either.  Do you have access to the associated ticket, DNN-10163?

That doesn't make any sense to me either.  Do you have access to the associated ticket, DNN-10163?

Here's a screenshot of the ticket, but it doesn't help much:

Here's a screenshot of the ticket, but it doesn't help much:


METRON-527: Connection pool shut down seen when running threatintel_t…

…axii_load.sh script

The Taxii HttpClient closes connections in callTaxiiService as seen here:
https://github.com/TAXIIProject/java-taxii/blob/master/src/main/java/org/mitre/taxii/client/HttpClient.java#L335
Because TaxiiHandler attempted to reuse a closed connection, the error was seen.  The fix is just to ensure the connection is created fresh before each call.  Integration test is updated to just call the run() method twice without caring about results to ensure that the connection doesn't explode.
In addition, this was tested in quick-dev using a slightly modified version (to work with Vagrant) of https://community.hortonworks.com/articles/59698/pushing-stixtaxii-feeds-from-opentaxii-server-into.html, where the playbook was run with
ansible-playbook --private-key=./vagrant/quick-dev-platform/.vagrant/machines/node1/virtualbox/private_key -i <IP_ADDRESS>, playbooks/install-opentaxii.yml -e ansible_python_interpreter=python -e ansible_user=vagrant -vvvv
with the IP being quick-dev's IP and the install-opentaxii script modified to reflect this.
After the change, polling continues until ended and hbase dumps grow as new records are added.

+1 by inspection

+1 by inspection

Please hold off on merging this a bit.  Anand and I are looking into an issue that is almost certainly separate, but doing due diligence.

I'm satisfied with my PR after further investigation.


Exploratory testing feed back iteration

Core Auth API now takes list of IAuthorizationRequirements, or policy name.
Overload that takes AuthorizationPolicy instance moved to extension method.
Remove HttpContext from API and replace with ClaimsPrincipal instead
Add Operation requirement

Fixes https://github.com/aspnet/Security/issues/132
Fixes https://github.com/aspnet/Security/issues/116
Fixes https://github.com/aspnet/Security/issues/11
cc @lodejard @blowdart @yishaigalatzer @harshgMSFT

Added sync overloads, and switched default async implementations to just call the sync version by default.  Only the sync version of Handle is abstract

Fixes https://github.com/aspnet/Security/issues/117

Replaced by https://github.com/aspnet/Security/pull/151


Fix crash when closing colour component

I noticed that when the colour component is open and you click on one of these buttons than the onDismiss callback is invoked but refers to its parent which has been destroyed already it seems.

Thanks for this! Will take a look tomorrow.


BLD: conda build strings changed between versions

so using the old build string with --skip-existing, so it finds
our existing remote packages

Checking if this commit resolves the conda build failure or not.
Completely disabling conda is not easy, the whole travis build process uses conda 😿

Changes Unknown when pulling 223a30d83bb75adda4aab8b950be31fd2f92dbf5 on tibkiss:conda_build_fix into ** on zipline-live:master**.

@bartosh : As this fix resolves the conda problem I'd propose merging this PR.
In a long run we can get rid of conda completely, but that'll be a relatively big effort.


Ignore WebKitDomainError 101

Summary
fixes #960

what's happening on this? been 2 months and no movement


Unable to add OpenID Connect authentication scheme dynamically

Describe the bug
I am unable to add an OpenID Connect authentication scheme "dynamically" (outside of Startup.cs). Based on the discussion here, it sounds like it should be doable by following the DynamicSchemes sample, however that is not the case. I found this discussion in which another person is having the same issue.
To Reproduce
I made a fork of DynamicSchemes with the OpenID Connect code added to it to show what's going on.
Steps to reproduce the behavior:

Clone my DynamicSchemes fork
Replace the "XXX" strings with an Authority, ClientID, and Client Secret for an OIDC provider of your choosing
Run the sample
Click the button under "Add OpenID Connect scheme" to add the scheme dynamically
Click the "Sign in with OpenID Connect" button in the navbar to attempt to sign-in
At this point you will get the following error: "InvalidOperationException: Provide Authority, MetadataAddress, Configuration, or ConfigurationManager to OpenIdConnectOptions"
You can get around this by passing a ConfigurationManager to the OpenIdConnectOptions. Uncomment this line and fill in the "XXX" with the metadata address for your OIDC provider
Run the sample again, you will get the following error: "NullReferenceException: Object reference not set to an instance of an object."

Expected behavior
Signing in to the dynamically added OIDC scheme should work without errors.
Screenshots
N/A
Additional context
Add any other context about the problem here.
Include the output of dotnet --info
PS C:\git\AspNetCore\src\AuthSamples> dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100-preview-009812
 Commit:    e3abf6e935

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17134
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\3.0.100-preview-009812\

Host (useful for support):
  Version: 3.0.0-preview-27122-01
  Commit:  00c5c8bc40

.NET Core SDKs installed:
  2.0.0 [C:\Program Files\dotnet\sdk]
  2.0.3 [C:\Program Files\dotnet\sdk]
  2.1.4 [C:\Program Files\dotnet\sdk]
  2.1.100 [C:\Program Files\dotnet\sdk]
  2.1.102 [C:\Program Files\dotnet\sdk]
  2.1.104 [C:\Program Files\dotnet\sdk]
  2.1.201 [C:\Program Files\dotnet\sdk]
  2.1.202 [C:\Program Files\dotnet\sdk]
  2.1.401 [C:\Program Files\dotnet\sdk]
  2.1.402 [C:\Program Files\dotnet\sdk]
  2.2.100 [C:\Program Files\dotnet\sdk]
  2.2.101 [C:\Program Files\dotnet\sdk]
  3.0.100-preview-009812 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0-preview-18579-0056 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.3-servicing-26724-03 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0-preview-27122-01 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.0.0-alpha-27128-4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

That got me a bit further. I added the changes to my fork in this commit.
However, it's still broken. When the OIDC provider redirects back to my app at https://localhost:44383/signin-oidc, I get a 404. I did some Googling and found this thread which suggests putting app.UseAuthentication() in Startup.cs. After doing that, I get a crash and an "Access violation" when adding the OIDC authentication scheme.

If you were able to see the OIDC provider's page and to be redirected back to the app then it means everything is OK.
The issue you've got is because you want to Authenticate the user using 3rd party identity provider, but current app is not able to Authenticate due to the wrong project template. Seems like that project with 'No Authencitation' initially. You need to add an 'Individual user accounts' authentication to the project
https://stackoverflow.com/questions/41471782/visual-studio-2017access-the-change-authentication-screen-on-a-already-created
or better to create a new project with 'Individual user accounts' authentication and implement the same in AccountController. There will be ExternalLoginCallback action in AccountController where you can get user info from the provider.

@HaoK - can you add your thoughts to this?

You can also take a look at https://github.com/aspnet/AspNetCore/pull/6363 which demonstrates how to provide a different set of options/schemes per tenant but the main thing to be aware of is that we don't really support using the built in providers (OIDC/Oauth/Google/Facebook) being used in a multi-tenant way, we left enough hooks for it to be possible if you replace large pieces of the stack, but the auth layer (in what used to be the Security repo) was built in a way that is not rea.lly intended to be used with multitenant.  So the example PR demonstrates how you can configure options generally in a multi-tenant way, but it isn't going to address any of the specific issues that come up in actual providers like OIDC

Faced same issue today. As it turns out, in the controller where I dynamically added new scheme, IPostConfigureOptions was resolved to EnsureSignInScheme>. As a work around, created a new instance of OpenIdConnectPostConfigureOptions and called .PostConfigure(options) explicitly
    public AuthenticationController(
        IAuthenticationSchemeProvider schemeProvider,
        IOptionsMonitorCache<OpenIdConnectOptions> optionsCache,
        IDataProtectionProvider dataProtection)
    {
        _oidcSchemeProvider = schemeProvider;
        _oidcOptionsCache = optionsCache;
        _oidcPostConfigureOptions = new OpenIdConnectPostConfigureOptions(dataProtection);
    }

    public async Task<IActionResult> AddExternalIdentityProvider()
    {
        _oidcPostConfigureOptions.PostConfigure(scheme, oidcOptions);
        _oidcOptionsCache.TryAdd(scheme, oidcOptions);
    }

@HaoK, It'll be helpful to have this issue looked into. Appreciate this design. With earlier version, it was far more difficult to achieve this.


Find widget icon background

On some themes the background of the icon does not match the background of the input field:

@lszomoru how do you expect verification of these? I think I need to wait for a new version of live preview to be published or should I just build the sources

@lramos15, I will try to publish a new version of the extension today.

Extension has been updated in the marketplace.


What unit does get_raw_data_mean return?

We are using the hx711 to just read strain rather than weight

Hello,
you can use the method get_raw_data_mean(readings=30) to get just raw data (not converted and not subtracted by the offset) from HX711.
The argument reading sets how many readings should read and then return average. If you do not want to even use the filter or average then just set the reading=1 and this will return immediately single reading.

Yes that is correct the value you get is unitless. You may want to use the data in special situation. For example the outlier filter first filters out data and then the class converts the data to some units. Dou you have any specific problem or thing that you would like to solve?


Simplify Neuron classes

Current coverage is 100.00%

Merging #82 into master will not affect coverage as of 678b8bc

@@            master     #82   diff @@
======================================
  Files           28      28
  Stmts          941     941
  Branches         0       0
  Methods
======================================
  Hit            941     941
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 678b8bc

Powered by Codecov. Updated on successful CI builds.

Could somebody have a quick look at this?

This looks good for me. +2


Add LockSingleton for ECC lock

Why
To prevent ECC test fail, add the LockSingleton.  Use this singleton lock whenever the ECC is accessed.
How
Using Posix IPC, create a system wide semaphore to lock the ECC.
Also let hm-pyhelper provide convenient @ecc_lock decorator to keep the main logic simple and neat.
References
https://github.com/NebraLtd/hm-diag/issues/167

@posterzh the demos are nice but we should have unit tests for all new code, especially things this complex.
I took the liberty of adding two (sorry for breaking flake8). test_ecc_lock is working correctly. But test_lock_singleton_failure is hanging for an unknown reason when running pytest hm_pyhelper/tests/test_lock_singleton.py. Can you help me debug? Can you then finish implementing the test (you should just be able to uncomment the rest of the code).?

@marvinmarnold The reason for the hangout is because there are no resources available. I updated the LockSingleton constructor to set the count of resources explicitly if it's set in the parameter.
And I added 2 test cases for LockSingleton, test_lock_singleton_basic and test_lock_singleton_racing. The result of the test_lock_singleton_racing looks like below.

@marvinmarnold The available_resources of a InterprocessLock get reset on every restarting the system or docker container. It's tested in Ubuntu 20.04 desktop and diagnostics container in a Hotspot. I wrote about this in the Readme.md and comment section of interprocess_lock.py.

After restarting the diagnostics service.

@posterzh What happens if the process crashes after the lock is set? Wouldn't we then get an infinite lock? I see there is a timeout for acquiring the lock, but not for resetting the lock.


python3Packages.amcrest: 1.8.0 -> 1.8.1

Motivation for this change
Update to latest upstream release 1.8.1
Change log: https://github.com/tchellomello/python-amcrest/releases/tag/1.8.1

Related Home Assistant change: https://github.com/home-assistant/core/pull/56058
Target Home Assistant release: current (only dependency update, no code changes)

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.amcrest
python39Packages.amcrest

Result of nixpkgs-review pr 137272 at d2b6db71 run on aarch64-linux 1

2 packages built successfully:

python38Packages.amcrest
python39Packages.amcrest


Access Input component

Is there anyway to access or pass in your own input component to the DatePicker?  I'd like to add an Input adornment "calendar icon".

Nevermind. Found it through the custom example:
InputProps={{
    endAdornment: (
      <InputAdornment position="end">
        <IconButton> add_alarm </IconButton>
      </InputAdornment>
    ),
}}


Memory leak prevention / warnings

When I used socket.io-stream for a bigger project where I did things like pipe multiple streams and sometimes streams where not handled on the end I ran into lots of little memory leak issues. I really had to dive into how socket.io-stream works to figure out what was going wrong. So I'd like to come up with ways to make this easier for other people.
One of the "invisible" problem areas is that each socket.io-stream socket stores references to socket.io-stream streams it uses. When streams are not handled these streams keep piling up. This is especially problematic when you put streams in flowing mode and they already have a chunk of data in their buffer.

One first thing I would do is log (using debug) the amount of stored streams, this way people see this growing number. Preferably when the streams are added.
Another idea might be to give a warning when a certain amount is reached, like Node.js's max listeners warning.
Could it be that streams don't get removed when an error occurs? I wrote error handlers on streams that would manually end() and destroy() that stream.

I think streams would be cleaned up if you properly call stream.end() (or pipe()).
Do you mean you'd like to add a feature to clean streams even if they are not consumed?
If so, one way is to add timeout to each stream.

Could it be that streams don't get removed when an error occurs? I wrote error handlers on streams that would manually end() and destroy() that stream.

Streams should be removed upon error.
https://github.com/nkzawa/socket.io-stream/blob/master/lib/iostream.js#L264

Streams should be removed upon error.
https://github.com/nkzawa/socket.io-stream/blob/master/lib/iostream.js#L264
Very valid point. Maybe my issues where caused by other factors. Not sure what happens when the stream that it's piped to throws an error for example.

What do you think about adding the info to the debug logs and giving that warning?

Logging and warning also are great idea. And exactly we can use debug module to log them.

The following module might be useful to use combined with socket.io-stream, to make sure streams get closed.
https://www.npmjs.com/package/pump


[Bug?] Cannot execute the command "reload app without save"

There is no response when executing the command "reload app without save". I guess it would be hard to achieve but if it is possible to support this command. Thanks for reading.

Sorry I have tried to debug this issue and don't find any solution.


Improves installation and resources docs

Adds some small improvements for the documentation
Ref: #9

I've updated the PR.
@Dispader, good point, thanks!
I've created a Windows VM and installed Groovy using Windows Installer from official website and it doesn't handle the situation where the user doesn't have JDK installed.
I should have mentioned JDK part first, which I've added in the current version
And I've also rearranged sections as suggested.
Merge it if no objections


Conformal

Thanks again for a nice package, it would be great if we can weave conformal prediction into this function and they can be returned as columns.
from MFLES.Forecaster import fit_from_df

output = fit_from_df(df_signal,
                      forecast_horizon=210,
                      freq='W',
                      seasonal_period=[13, 26, 52, 104,210,420, 840],
                      id_column='ticker',
                      time_column='date',
                      value_column='total_revenue',
                      floor=0)

@firmai I totally agree! Should not be tough, I'll add it.


Gradle plugin: create task per source set

Expected Behavior
Plugin creates detekt task for each source set in a Gradle project.
Current Behavior
Plugin creates single detekt task per Gradle project.
Context
Required for #485 (which I will update and integrate with the Gradle plugin).
To enable type and symbol resolution when detekt runs as part of the Gradle plugin, detekt must be aware of the compilation classpath. If detekt can't access the complete classpath that's used for compilation it's not possible to resolve all types and symbols unless classpaths are setup manually on each detekt task which isn't practical for users and highly error prone.
Additionally the detekt task for a source set should by default scan the source files in the sourceset. The current default for detekt is to scan src/main/java and src/main/kotlin. This will fail for tasks which are created for other sourcesets (e.g. "test") since the source files for those would typically be a combination of src/main/java, src/test/java, src/main/kotlin and src/test/kotlin.
This also affects how reports will be configured, because they will necessarily have to be created per task. If the report filename is set using the extension (i.e. detekt.reports.xml.destination =) a report generated for one detekt task would be overwritten by the next detekt task in the same project because the destination file path would be the same. Instead, the report directory must be set on the extension and the destination path on the task.
It should be possible to minimise disruption by creating source set tasks like "detektMain" and "detektTest" which use the new behaviour for reporting DSL and keep the current "detekt" task as is but with a deprecation message.
Then suggest that prior to 1.0 release, update the "detekt" task so it depends on all configured detekt tasks (detektMain, detektTest etc) and remove the reporting config from the extension (with exception of reportsDir provided by CodeQualityExtension).

This seems similar to #540 which describes the same. Overall I agree with adding a task per sourceSet (while keeping one "global" detekt task).
For the reports issue we could either suffix the report name with the sourceSet name or merge instead of overwrite the reports if multiple exist. But that would probably be quite tricky.

Using a suffix or change the report directory for each task could easily be done. Does that imply the same config is used for all such tasks?
From reading #485 I did not quite get what is expected to be passed to the cli. Is it --input src/main/java;src/main/kotlin as well as the --classpath? Must the classpath contain all the compile time dependencies of the project?

I was looking into this earlier today and can upload a first snapshot of
the changes tomorrow.
On Tue, Oct 9, 2018, 22:13 marschwar<EMAIL_ADDRESS>wrote:

Using a suffix or change the report directory for each task could easily
be done. Does that imply the same config is used for all such tasks?
From reading #485 https://github.com/arturbosch/detekt/pull/485 I did
not quite get what is expected to be passed to the cli. Is it --input
src/main/java;src/main/kotlin as well as the --classpath? Must the
classpath contain all the compile time dependencies of the project?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/arturbosch/detekt/issues/1198#issuecomment-428334653,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF4k0VCUg0VMa0oW2FbGQdujN8sfPsjmks5ujQN6gaJpZM4XJjA7
.


Text extractor problems

Microsoft PowerToys version
0.74.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
TextExtractor
Steps to reproduce
Hello! I have some problems with text extractor. This is a main problem. Is very unexactly and is not as good as I expected, this tool is very innaccurate and is very unpowerful. The text extractor tool needs an serious upgrade beacuse is not very good at text extracting. Is veru innacurate and very unprecisely. Thanks! I'm waiting for an update!
✔ Expected Behavior
I want an upgrade at text extractor! Thanks
❌ Actual Behavior
The text extractor tool needs an serious upgrade beacuse is not very good at text extracting. Is veru innacurate and very unprecisely. Thanks! I'm waiting for an update!
Other Software
Microsoft powertoys 0.74.0 installed from microsoft store

/dup #20899


Add FreeBSD installation instructions

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


[SPARK-42723][SQL] Support parser data type json "timestamp_ltz" as TimestampType

What changes were proposed in this pull request?

Support parsing timestamp_ltz as TimestampType in schema JSON string.
It also add tests for both parsing JSON/DDL for "timestamp_ltz" and "timestamp_ntz"
Why are the changes needed?

timestamp_ltz becomes an alias for TimestampType since Spark 3.4
Does this PR introduce any user-facing change?

No, the new keyword is not released yet.
How was this patch tested?

New UT

@HyukjinKwon Thanks for the review.
Merging to master/3.4


Add task type information when importing

Summary

How to test

Checklist

[ ] I have added unit tests to cover my changes.
[ ] I have added integration tests to cover my changes.
[ ] I have added the description of my changes into CHANGELOG.
[ ] I have updated the documentation accordingly

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below).

# Copyright (C) 2024 Intel Corporation
#
# SPDX-License-Identifier: MIT

I have a question. I'm curious why datasets are designed to have a single task_type. For example, if a dataset has both label and bbox annotations, it can be used for both classification and detection tasks. (And even for anomaly cls/det. tasks if the labels are anomalous and normal). However, according to your implementation, it seems like the task_type becomes detection.

I have a question. I'm curious why datasets are designed to have a single task_type. For example, if a dataset has both label and bbox annotations, it can be used for both classification and detection tasks. (And even for anomaly cls/det. tasks if the labels are anomalous and normal). However, according to your implementation, it seems like the task_type becomes detection.

Hi @jihyeonyi, thank you for the question. We are able to identify the mapping between annotation types and tasks in task.py and there is some included or excluded relationships between them. So it is preferable to provide new transformations for changing/filtering dataset items per each desired task types. Let me add more explanations later.


[WIP] Enable unsafe.Pointer use with checkptr flag in CI test binaries

DO NOT MERGE, testing only.
Check unsafe.Pointer using the Go 1.14 -d=checkptr compiler option
in CI test builds.
For #10133

test-me-please

test-me-please

Coverage increased (+0.02%) to 45.528% when pulling 1bca5937ba230369bb07f7e8b0879b26892697b7 on pr/tklauser/test-checkptr-build into 60bd7f50d84db54d33e552ec0ccebc3f31628e03 on master.