feat(source): Support RFC5425, 5426

Support RFC5425 (TLS) default port 5425 and RFC5426(UDP) 601

:tada: This PR is included in version 1.47.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.47.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Flaky test compile_invalid_pyc_invalidation_mode

I see this relatively frequently on macOS
Snapshot: compile_invalid_pyc_invalidation_mode
Source: crates/uv/tests/pip_sync.rs:2934
────────────────────────────────────────────────────────────────────────────────
Expression: snapshot
────────────────────────────────────────────────────────────────────────────────
-old snapshot
+new results
────────────┬───────────────────────────────────────────────────────────────────
    7     7 │ Installed 1 package in [TIME]
    8     8 │ error: Failed to bytecode-compile Python file in: [SITE-PACKAGES]
    9     9 │   Caused by: Python process stderr:
   10    10 │ Invalid value for PYC_INVALIDATION_MODE "bogus", valid are "TIMESTAMP", "CHECKED_HASH", "UNCHECKED_HASH":
   11       │-  Caused by: Bytecode compilation failed, expected "[SITE-PACKAGES]/[FIRST-FILE]", received: ""
         11 │+  Caused by: Failed to write to Python stdin
         12 │+  Caused by: Broken pipe (os error 32)

Yeah same.

(This has also failed on CI FWIW.)

@BurntSushi is reporting this on Linux now

I've been also seeing this on linux rarely, no idea how to prevent this from happening, i made a PR to just retry once on failure

We removed this test in #4863 — we should figure out how to test this some day.


Support for new Proto3 optional

Sorta announced: https://github.com/protocolbuffers/protobuf/issues/1606#issuecomment-618687169


How to feed continuous data from IMU to the model

Hi,
I am wondering how you feed continuous IMU data into the model ? did you use buffer ?

For future reference - in this implementation the classification was performed in a local server based on data received from an IMU via Bluetooth Low Energy (i.e. not on a device directly).


Cloudflare and "shield" records

Cloudflare lets you "shield" (cdn/dns) A and CNAME records. This enables their cdn functionality. Seeing how the feature is embedded into the way DNS records is handled on CloudFlare, I thought I should at least bring up whether or not this belongs in octodns.
(if nothing else, a closed issue might be useful if others are thinking of the same thing)

least bring up whether or not this belongs in octodns.

Short answer is that I don't know. Given that it's very specific to Cloudflare it's not really in the wheelhouse of the problem octoDNS was setting out to solve, but that said if someone is just managing Cloudflare alone with octoDNS it'd be nice to have.
I think that leaves it in a place where the answer is: maybe, someday, if/when there's a solid system for managing provider specific config. In practice that probably means it'd be a really low priority and won't land anytime soon.

/cc @jacobleonw

cheers @ross
might be time for me to learn some python 🐍

@ross As a user of Cloudflare and occasionally its CDN/proxy, I find it quite unfortunate that OctoDNS doesn't support this particular CloudFlare feature (because I think OctoDNS is such a uniquely useful tool).
Having said that, although I'm not familiar with the implementation, I can appreciate that it's most likely not a simple task and understand it is of low priority.
If someone were to work on this, what approach do you think they should take?
Should the CloudFlare provider implementation be modified and the remainder of OctoDNS's API remain unchanged, or are there plans to alter the OctoDNS API to allow for provider-specific (not just CloudFlare) configuration, which should be taken into consideration?

Having said that, although I'm not familiar with the implementation, I can appreciate that it's most likely not a simple task and understand it is of low priority.

I don't believe it'd be too difficult, but since I haven't directly had a use for it (professionally or personally) it is a low priority for me and hasn't made it up the TODO list. The other thing preventing it from getting a higher priority is that it's provider specific and in general octoDNS is about enabling multiple providers with a single configuration. That not to say that it isn't useful for managing a single provider, it just wasn't the problem it set out to solve.

If someone were to work on this, what approach do you think they should take?
Should the CloudFlare provider implementation be modified and the remainder of OctoDNS's API remain unchanged, or are there plans to alter the OctoDNS API to allow for provider-specific (not just CloudFlare) configuration, which should be taken into consideration?

There are a couple use of an octodns key under records for things like customizing healthchecks and specifying lenient validation for individual records. That data isn't currently stored in the record object through so it'd be "gone" by the time providers were able to take a look at it.
I think the first step would be holding on to the data in the record object so that providers could later look at it. The next bit would be updating the CF provider to look for the key and do it's thing, the config might look like:
foo:
  octodns:
    cloudflare:
      proxy: true
  type: A
  value: <IP_ADDRESS>

The provider can then manage that flag based on the "extra" info. It'd also have to make use of some of the more "advanced" bits of providers to flag records that otherwise match, but don't have the correct proxy setting, currently there are various bits of the Geo records that do this in Dyn and Route53. https://github.com/github/octodns/blob/61b30f2b41d5b209b302b0a3c1bdf8af44e8d3e0/octodns/provider/base.py#L37-L43. Beyond that it'd just be a matter of working out the kinks and getting it throughly tested (which is often the biggest part of the work.)

...Could you please elaborate on that?

There will be cases where you have a record in the config and in Cloudflare's system where the values match up, but the proxied flag doesn't. octoDNS's process won't consider that a "diff" and thus won't make updating that record part of the plan and give the provider a chance to fix it.
The Cloudflare provider will have to walk over all of the records in an overridden _extra_changes to look for things that don't get flagged as general changes, but still need to be added to the plan for the provider to update the proxy flag. If you search around for _extra_changes you'll see other examples of similar cases, healthchecks that need updating in geo enabled records etc.


The Japanese subtitle json file is left blank.

I tried to import a story, but the Japanese subtitle won't upload even though the English one will. I don't have Go installed (on Windows) if that's any indication.
Wistoria： Wand and Sword - Episode 01 [English Sub] [uAS2RXlPsAk].en.json
Wistoria： Wand and Sword - Episode 01 [English Sub] [uAS2RXlPsAk].ja.json
EDIT:
I was sleepy while writing this. I meant that the Japanese subtitle file (VTT) was fine, but the .json file created did not have anything (Just a []). The count imported number also did not go up. This only happens to the Japanese one and not the English, so is there anything I might not have installed on my computer that I need?

Hmm, are you sure the subtitle file had the correct name? In this case, it would be  Wistoria： Wand and Sword - Episode 01 [English Sub] [uAS2RXlPsAk].ja.vtt.
I've just made quite a lot of changes, so you may want to check out the latest release. (Note that the 'sources' directory is now under the root, not under 'static').

Ah, it works now. Thanks!


Question about Predictions: .detectText(.all) vs. .detectText(.plain)

Hello,
I've noticed the following:
When calling identify with .all _ = Amplify.Predictions.identify(type: .detectText(.all), image: content_url) the answer comes back as IdentifyDocumentTextResult.
When calling identify with .plain _ = Amplify.Predictions.identify(type: .detectText(.plain), image: content_url) the answer comes back either as IdentifyTextResult or IdentifyDocumentTextResult. Why is that?
But more importantly, within IdentifyDocumentTextResult the field words is always populated, but within IdentifyTextResult that is not always the case. IdentifyTextResult always seems to contain identifiedLines. This puzzles me, as words seem to be more basic than lines.
Anyways, currently I just run .all, but given that more testers use my app, this starts being expensive. All I care about is words being there. The post-processing I do myself, as I found the identifiedLines does not work well for my use-case. If I read the documentation of Textract correctly, .plain should be much cheaper.
Summarizing: Is there any way to always load the words field, such that I can only pay for what I actually use?

When .all is passed to identify(), AWSTextract is used to analyze the image and the result is returned as IdentifyDocumentTextResult.
When .plain is passed to identify(), both AWSRekognition and AWSTextract are used to analyze the image. AWSRekognition has a word detection limit of 50 for an image and if an image contains more than 50 words, results from both AWSRekognition and AWSTextract are compared and whichever service  returns more detections, the appropriate result is returned. (IdentifyDocumentTextResult for AWSTextract/IdentifyTextResult for AWSRekognition)
The fields for IdentifyDocumentTextResult and IdentifyTextResult are populated based on the types(word or line) of text returned by the  service used for the query (AWSTextract/AWSRekognition). .plain is used primarily to detect text and '.all' is primarily used to detect  document type relationships(key-value pairs, forms, table).

@thisisabhash thank you very much for your answer. That makes sense. So I guess, I'll have to stick with .all to reliably obtain the words instead of the lines.
Or can you think of another way to always get the words without paying for lots of stuff I don't use? The words is really all I need. I'm not interested in key-value pairs, forms or tables.

When .plain is passed to identify(), the result returned is taking into account both the Rekognition (DetectText API) and Textract (DetectDocumentText API) responses, when an image has more words than Rekognition limit.
Currently, there is no way to always get the words field populated as it depends on the responses from Rekognition and Textract.

@thisisabhash ,
Thank you very much for the clarification. This answers my question.


Fix to velocity calibration develop

Similiar merge to the one on master branch

Shouldn't thir PR be targetting master instead of develop?

There was one targetting master. This one is for develop, as requested by WK.


fix: users are able to accept expired invitations

IDE-499
fix: return error if user is tying to accept expired invite. Currently, user can accept expired
Testing:
API request:
curl --location 'http://<IP_ADDRESS>:9080/v1beta1/organizations/<org_id>/invitations/<invite_id>/accept' \ --header 'Content-Type: application/json' \ --header 'Accept: application/json' \ --header 'Cookie: sid= \ --data '{}'
Response code
400 Bad Request
Response body:
{"code":3,"message":"invitation expired","details":[]}

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

2 of 5 (40.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 9.507%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/invitation/service.go
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>6:
0.002%

Covered Lines:
10360

Relevant Lines:
108974

💛  - Coveralls


The user can create the Kafka destination connector with an invalid value "-1"

Based on the title, it looks like this is about the Kafka connector, not this Kafka Connect wrapper? Also, we should specify for what field is -1 an invalid value.

This was opened by mistake, the actual issue is https://github.com/ConduitIO/conduit-connector-kafka/issues/42.


Json error

Hello,
How can ı solved this problem
I have a problem but where I don't know json
Thank you for your supports,
Error;
FURKANs-Mac:~ furkankaya$ homebridge
[2/1/2019, 8:52:26 PM] There was a problem reading your config.json file.
[2/1/2019, 8:52:26 PM] Please try pasting your config.json file here to validate it: http://jsonlint.com
[2/1/2019, 8:52:26 PM]
/usr/local/lib/node_modules/homebridge/lib/server.js:214
throw err;
^
SyntaxError: Unexpected token { in JSON at position 141
at JSON.parse ()
at Server._loadConfig (/usr/local/lib/node_modules/homebridge/lib/server.js:208:19)
at new Server (/usr/local/lib/node_modules/homebridge/lib/server.js:56:38)
at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:30:16)
at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (internal/modules/cjs/loader.js:689:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
at Module.load (internal/modules/cjs/loader.js:599:32)
at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
at Function.Module._load (internal/modules/cjs/loader.js:530:3)
My config.json file;
"accessories": [
{
"accessory": "webostv",
"name": "My webOS tv",
"ip": "tvipaddress",
"mac": "tvmacaddress",
"keyFile": "/home/pi/.homebridge/lgtvKeyFile",
"pollingInterval": 10,
"tvService": true,
"inputs": [
{
"appId": "com.webos.app.livetv",
"name": "Live TV"
},
{
"appId": "com.webos.app.hdmi1",
"name": "Chromecast"
},
{
"appId": "youtube.leanback.v4",
"name": "YouTube"
},
{
"appId": "com.webos.app.photovideo",
"name": "Photo Video"
}
],
"volumeControl": false,
"channelControl": false,
"mediaControl": false
}
]
}

Just use the link specified to validate your json... You can also use some online json beautifier to make it more readable.

You missing a { at the begining.


Nuxt.js support

Hi,
Is there a way to get this working with Nuxt.js? I tried to run it as plugin with SSR false, but it can't find the window.

I was trying to make it possible but there are some issues with inject / provide I guess. I'll try to fix this and make it Nuxt.js supported. If u know what should I change to make it possible, just give me an advice or feel free to make a PR.
Thanks!

Ok, I'll take a look at it. Maybe I can come up with something.

Have you tried wrap it with  tag?


Update juniper to 0.16

https://github.com/graphql-rust/juniper/blob/juniper-v0.16.1/juniper/CHANGELOG.md#0161--2024-04-04
It's probably a rather large and involved update, but we want to update eventually anyway and it gives us at least a newer GraphiQL version, which is neat.

done in https://github.com/elan-ev/tobira/pull/1298


[BUG] import setuptools raises a DeprecationWarning on Windows

setuptools version
setuptools==65.0.2
Python version
Python 3.9
OS
Windows
Additional environment information
No response
Description
On Windows, import setuptools raises a warning added in https://github.com/pypa/distutils/commit/c802880.
This issue does not occur in setuptools 65.0.1.
Expected behavior
No warnings.
How to Reproduce
python -Werror -c "import setuptools"

Output
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "C:\Users\maehashi\.venv-py39\lib\site-packages\setuptools\__init__.py", line 247, in <module>
    monkey.patch_all()
  File "C:\Users\maehashi\.venv-py39\lib\site-packages\setuptools\monkey.py", line 99, in patch_all
    patch_for_msvc_specialized_compiler()
  File "C:\Users\maehashi\.venv-py39\lib\site-packages\setuptools\monkey.py", line 136, in patch_for_msvc_specialized_compiler
    msvc = import_module('setuptools.msvc')
  File "C:\Program Files\Python39\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "C:\Users\maehashi\.venv-py39\lib\site-packages\setuptools\msvc.py", line 61, in <module>
    from distutils.msvc9compiler import Reg
  File "C:\Users\maehashi\.venv-py39\lib\site-packages\setuptools\_distutils\msvc9compiler.py", line 34, in <module>
    warnings.warn(
DeprecationWarning: msvc9compiler is deprecated and slated to be removed in the future. Please discontinue use or file an issue with pypa/distutils describing your use case.

Is it possible to have the setuptools tests fail if setuptools itself produces warnings? And to run the integration tests on Windows too?
It would be nice if the CI tests could help avoid the excitement that we're experiencing in future.

Setuptools does fail on warnings, but intentionally suppressed this DeprecationWarning because it gets triggered when those modules get tested. I did not realize that Setuptools actually imported this module. That's slightly surprising given that I removed the module and the tests continued to pass. But now that I look at it, it's not all that surprising with all the except ImportError wrapped around it.
The choice to deprecate these modules came a little rushed when backing out the removal of those modules.
I definitely don't want these deprecation warnings to emit when using Setuptools.

Please try again with v65.1.0 (releasing now).

Thanks, I will give it a try!

Would it be helpful for setuptools to have a sort of buildbot system where a few projects volunteer to build and test against setuptools master, say, daily in their own repos, and to gather the build statuses somewhere? It might help with testing things a bit more. And it would be okay if the list of projects was a bit fluid and changed over time.
I could volunteer the main project I work on (qutip) and HPy for the list.
Perhaps there is a wider discussion to be had about how we ensure things work nicely for C extensions while still allowing setuptools to evolve. I'm happy to be involved in that discussion (or even to start it) if someone points me to the right place.
Apologies for cluttering this issue report a bit.

@jaraco Worked for me. Thanks for the prompt action!


newest tuning eps

Codecov Report
Patch and project coverage have no change.

Comparison is base (d54325f) 0.09% compared to head (778680f) 0.09%.

: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     #18   +/-   ##
======================================
  Coverage    0.09%   0.09%
======================================
  Files          11      11
  Lines        1024    1022    -2
======================================
  Hits            1       1
+ Misses       1023    1021    -2

Impacted Files
Coverage Δ

src/ensemble/tuning.jl
0.00% <0.00%> (ø)

src/tuning.jl
0.00% <0.00%> (ø)

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.


Update fi.coffee

Started Finnish translation

Hello, @pettevi! Thank you for contributing! Could you sign the CLA so that I can merge your pull request?
Note: Could you also remove the # from the strings you translated, so that they take effect?

Thanks, @pettevi!


[Feature] Optional Tanh transfromed distributions in algorithms

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (02fc1b6) 89.68% compared to head (b39e2a5) 84.48%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   89.68%   84.48%   -5.21%
==========================================
  Files          64       64
  Lines        2656     2668      +12
==========================================
- Hits         2382     2254     -128
- Misses        274      414     +140

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


server send ping time duration!

need this options.

server did not send pings :(
but it response to ping, you can put it at client side


Vibrance can't be less than 50

When the vibrance is changed to a value less than 50, no changes occur :(

It is intended, I should document it!
https://github.com/OtaK/nvapi-rs/blob/feature%2Fdvc/hi/src/display.rs#L44
As to why, NvAPI_SetDVCLevel that allows to change vibrance for through the NVAPI (NVIDIA API) is absolutely undocumented; but from what I've seen everywhere, it allows only 51 different values (see: https://github.com/OtaK/nvapi-rs/blob/feature%2Fdvc/hi/src/display.rs#L2) that translate roughly from 50 to 100%.
So I could try to set thing less than 50, but there are no guarantees it'd work and would probably crash the program to be honest.


Make pure lua

Thank you for this! I am holding back merging it for the moment because I have rewritten a lot of the code in this repository, and this version is not the most up to date. I will make sure the latest version is pure lua though


Camera anchor moves the camera off center

This is how anchoring the camera with the CameraAnchorDetector looks

Not entirely sure on what is needed, the only time the camera was off centered in LevelTest was here:

Should this be prevented?

In my image above, the anchor is the purple square, it's clearly off center. You need to visualize the anchors to see that. The anchor should be in the center of the window.

Forgot about this bug. This might be a godot bug with the camera. You know the camera has those offsets to follow the player after the player moves away from center a certain distance. This bug makes me think that those offsets don't reset when focusing on another point, I'll try some tests. The fix should be easy enough if this is it, we just have to set the offsets to 0 when the camera moves away from the player.

@NathanLovato where can I grab the example you had in the screenshot above? You got it in some branch I can explore?


Copy LineUp build files to auto update the app

Closes #366
Prerequisites:

[x] Branch is up-to-date with the branch to be merged with, i.e. develop
[x] Build is successful
[ ] Code is cleaned up and formatted
[ ] Tested with Firefox ESR, latest Firefox, latest Chrome, Edge 18

Summary
TODO needs to be tested in develop or master branch

TODO needs to be tested in develop or master branch

What do you suggest here? Is there a way to test this without merging this PR? Otherwise we can merge it and fix it later.

I would just merge it


Svalbard and Jan Mayen

Any reason for not including "Svalbard and Jan Mayen" (country code SJ)?
I understand that it is basically part of Norway, but ISO-3166 does recognize it separately.  Is there any reason to not have this dataset use the ISO-3166 info as it's official basis?

Hey @dlong500, great question! It can be a little slippery. The reason I created this repo way back when was because no such thing existed at the time. The goal was a practical list of country-region data that I could use in forms. I didn't realize it was such a political issue: the problem with the ISO standard is it's a list of internationally recognized countries, which may not cover countries that we expect to see.
But to answer you question, no - no reason for not including Svalbard + San Mayen. If you can provide a link to the data set I'm happy to get that imported.
Thx!

I didn't actually do an in-depth comparison of the data with the ISO standard, but simply saw it contained 248 country codes versus 249 that are in the ISO dataset.  Are you aware of other differences beyond the missing "Svalbard and Jan Mayen"?  I don't have any political issue with anything--just curious about the data.

Nice, thanks for the info. Possibly just outdated data? I think originally it was based on that dataset so I'm not sure why they'd be missing.

"Svalbard and Jan Mayen" is now considered a country. with country code SJ
I'm not sure what regions they have. But can we at least have that as a country?
and have a region with the same name.


Fix a spelling error in memoization.py

Hi @timarmstrong !

Hi @danielskatz! I saw @yadudoc today and he said I should make a contribution.

First of many I hope :smile:

I'm merging this in.


Fix versions for dtype checks. Don't check v0.4

My bad: For https://github.com/ome/ome-ngff-validator/pull/16#issuecomment-1297636546 the validator checks dtypes for v0.4 NGFF and asserts failure if dtypes mismatch (so that the PR could be tested).
However, v0.4 NGFF with mismatching dtypes are not actually invalid.
I forgot to revert that change before merging. So this PR fixes that and ONLY asserts that mismatching dtypes are invalid for versions after v0.4, to correspond to https://github.com/ome/ngff/pull/154
So, checking the mismatching dytpe sample from the PR above (v0.4) should no-longer give a warning:
https://deploy-preview-18--ome-ngff-validator.netlify.app/?source=https://minio-dev.openmicroscopy.org/idr/v0.4/idr0077/9836832_z.zarr

Thanks, @will-moore. MSTM.
I assume it's considerably more work, but do you think it would be possible to get an informational message that this would cause an error/warning in the next version?

@joshmoore https://deploy-preview-18--ome-ngff-validator.netlify.app/?source=https://minio-dev.openmicroscopy.org/idr/v0.4/idr0077/9836832_z.zarr
It will look like this with Error for versions 0.5+:

But for versions 0.1 - 0.4 you'll just see it as a warning:

Wow, nice!


LILYGO® TTGO T7 Mini32 V1.5

I recently came across your video and was inspired to immediately install your library. I'm delighted to report that it worked seamlessly with my TTGO T7 Mini32.
I am writing to kindly request the inclusion of the LILYGO® TTGO T7 Mini32 V1.5 board in your library. Despite my efforts to figure out how to add it myself, it seems that this might not currently be feasible. If possible, I would greatly appreciate any guidance or instructions you could provide on this matter.
For your reference, here are the relevant links:
Homepage: https://www.lilygo.cc/products/t7-mini32-v1-5
GitHub: https://github.com/LilyGO/TTGO-T7-Demo
Thank you for considering my request. I believe that including this board could benefit many users of your library.
Best regards,
Andreas

The board is now available.


fix: use bosh-release images from Github Container Registry

This PR uses the images on ghcr.io built with the latest stemcell.
The following images still reside on Dockerhub (when deploying with Diego; there will be additional images used by Eirini):
docker.io/cfcontainerization/cf-operator
docker.io/cfcontainerization/coredns
docker.io/cfcontainerization/pxc
docker.io/cfcontainerization/quarks-job
docker.io/cfcontainerization/quarks-secret

Ref #1557

There are 2 images in a different org that probably should be moved to cloudfoundry-incubator:
ghcr.io/cfcontainerizationbot/kubecf-apps-dns
ghcr.io/cfcontainerizationbot/kubecf-kubectl

I found the current postgres version does not exist for autoscaler. There are newer releases, so I tried this patch:
--- a/chart/config/releases.yaml
+++ b/chart/config/releases.yaml
@@ -53,7 +53,7 @@ releases:
     condition: false
   postgres:
     condition: features.autoscaler.enabled
-    version: "39"
+    version: "42"
   sync-integration-tests:
     # XXX SITS only makes sense when using Diego; add error check somewhere?
     condition: testing.sync_integration_tests.enabled

But then asdatabase-0 pod never got ready. This blocks init containers in other autoscaler pods, so I don't know what the status of the autoscaler images is.

The autoscaler issue was a hard-coded version in a path in the readiness probe; pushed a commit to make that work.  Also, bump the SITS version down since v0.0.3 doesn't seem to be on GHCR (will need to address that in a bit).

Full KubeCF installations from this PR should now work. Ready for review.
Comitted the change for eirinix images, created from the updated eirinix Concourse pipeline. Some images where missing from ghcr.io, as they predate the pipeline change on publishing, so I copied those tagged images manually to  ghcr.io/cloudfoundry-incubator.
Tested by deploying diego, eirini, ha & autoscaler successfully. Only missing image was docker.io/cfcontainerization/pxc:0.9.11
Which I manually copied to ghcr.io/cloudfoundry-incubator/pxc:0.9.11 meanwhile we wait for resolution of https://github.com/SUSE/cloudfoundry/pull/201.

Instead of opening a new PR for consuming the pxc image, I have edited the title and body of the PR and expanded the PR scope to all images, not only bosh-release images.
My intent was to have only 1 PR with all the new images, so it's easy to pass through CI.


Need better names for two types of method extraction

Method extraction (along with the calling of extracted methods) is a core use case of this proposal. For example, method extraction is essential for security from prototype pollution and other global mutation.
However, a non-goal of this proposal is what I currently call “tacit method extraction”: a (confusing) term I made up to refer to when a method is implicitly bound to its original owner, creating a bound function.
// “Explicit” method extraction results in a `this`-dependent ordinary function.
// This is handled by this proposal.
const { slice } = arr;
// The `this` value is supplied in the call.
arr::slice(...args);

// “Tacit” method extraction results in a `this`-independent bound function.
const slice = arr&.slice;
// The `this` value is not supplied in the call, because the function is bound.
slice(...args);

(Note that “tacit” method extraction is only useful on instance objects; it would not be useful on prototype objects, because calling a method on a prototype object is not very useful. In contrast, “explicit” method extraction is useful on prototype objects; the extracted prototype method could then be called on.)
// “Explicit” method extraction can extract from a prototype object…
const { slice } = Array.prototype;
// …and then a different `this` value may be supplied in the call.
arr::slice(...args);

I think these two concepts could use better names than “explicit” versus “tacit”. I don’t really have any other ideas, though.
(There’s even a third type of method extraction called “call-binding”, in which the this binding gets “uncurried” into a new (bound) function, using fn.call.bind, so that this is once again supplied at the call. @ljharb’s call-this call-bind library does this; Node’s primordials also use this pattern. I don’t even want to think about how they should fit in our nomenclature.)

Another way of saying it is, what I call "method extraction" is "call-bind a receiver-sensitive function, and cache it for later-run code".
A syntactic .call operator would obviate the need for call-binding, but not the rest.

I'm confused why these need names. How you get to Array.prototype.slice isn't really relevant - it's the same function either way. The important point is that you can get it (which this proposal does not help or hinder), and then syntactically call it with an arbitrary array as the receiver.

Yeah, I’m just looking for a precise way to refer to the non-goal of not repeating ourselves when we do x::(x.m).
As you know, the old proposal allowed this with its unary form, ::x.m. That’s a non-goal of this proposal.
But I don’t want to call that non-goal simply “method extraction”. I think this proposal does functionally address “method extraction” in that we can extract method functions and then use them (const { foo } = obj0; obj1::foo(arg);. It’s just that, in one case (when the extractor and the receiver are the same: const { foo } = obj0; obj0::foo(arg)), you have to repeat yourself.
My concern about this confusion comes from https://github.com/js-choi/proposal-bind-this/issues/12#issue-1019470707 by theScottyJam:

Both pretty-format and graceful-fs are basically doing a form a method extraction, except in both cases they're choosing to supply the "this" value at the call location instead of the extract location.
This proposal stated that it doesn't wish to focus on method extraction, but ironically, I think this use case might be the most important one among the ones that have been analyzed.

My concern is that, if we say, “Method extraction is a non-goal of this proposal,” then some other people will also think erroneously that that non-goal includes const { foo } = obj0; obj1::foo(arg);. These people may think that “method extraction” includes use cases not necessarily involving call-bind.


UserManager::hasRole and "role_hierarchy"

We may define role hierarchy in security.yml so user with role ROLE_ADMIN will have virtual role ROLE_TICKET_ADMIN
But in UserManager::hasRole you just check if role string is in array of UserInterface::getRoles()

Use 'security.authorization_checker' service and isGranted method to check it

Send you PR #94


Enforce redirect uri match

Explain your changes
Fixes #47  by adding an optional enforce redirect uri match boolean to Kinde client options.
This variable adds an extra check into the Kinde Client init method that will ensure the window.location matches the redirect URI.
This will ensure it does not hijack any other OAuth flows.
Checklist

[x] I have read the “Pull requests” section in the contributing guidelines.
[x] I agree to the terms within the code of conduct.

🛟 If you need help, consider asking for advice over in the Kinde community.

Thanks @DaveOrDead, was erring on the side of caution for that one, but agree it's better to just enforce it 💪. That change has been done.


Prefix test targets with SAP

Similar to https://github.com/apple/swift-argument-parser/commit/d8fb6bdd63fa541e0ea52a99556093e1c3a60586 and #18.
Prior to Swift 5.2, test target names need to be unique throughout the entire package graph. This change adds "SAP" to each test target to avoid ambiguities.
Context: I'm trying to use swift-argument-parser in a package repo which also depends on Carthage, and they have a nested dependency with a UnitTests target. Since the argument parser is under active development and I assume I'm not the only one in this situation, I figured I would propose the change here.
Checklist

[n/a] I've added at least one test that validates that my change is working, if appropriate
[n/a] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[n/a] I've updated the documentation if necessary

Thanks for getting this workaround ready, @elliottwilliams! Looking at all the names, we'd rather go with an ArgumentParser prefix for everything. Would you mind adopting that prefix, including renaming SAPTestHelpers?

@natecook1000 sure thing, updated!

Thanks, @elliottwilliams! Merged locally.


Implement Tracks and Rooms filter on schedule page

The schedule page does not have filter options. Please implement Tracks and Rooms filter on schedule page and make all items clickable as on other pages.

i would like to work on this


Repo.CommitsCount() slows down page renders for big repos

The "Repo.CommitsCount" API really slows things down.
I have a mirror of Linus' linux.git tree, and viewing that in the gogs web interface is a miserable experience. It takes several seconds for it to render the page. I dug through and found that gogs was executing this command:
# time git rev-list --count bcc981e9ed84c678533299d7eff17d2c81e4d5de
589485

real   0m6.707s
user   0m6.550s
sys    0m0.150s

One way to improve this is to cache the constant data. You could permanently cache the rev-list --count values for tags, as that value would not change (as long as it's keyed by the tag hash rather than the tag name):
# time git describe --tags --abbrev=0 bcc981e9ed84c678533299d7eff17d2c81e4d5de
v4.6-rc5

real	0m0.185s
user	0m0.170s
sys	0m0.010s
# time git show-ref v4.6-rc5
8ef3ad9a813abdf0817acb0b2be30be70bf25a9b refs/tags/v4.6-rc5

real	0m0.009s
user	0m0.000s
sys	0m0.000s
# time git rev-list --count 8ef3ad9a813abdf0817acb0b2be30be70bf25a9b
589482

real	0m6.800s
user	0m6.690s
sys	0m0.100s

So for the commit in question, the most recent tag in its ancestry is v4.6-rc5, and we could permanently cache the rev-list --count for the corresponding tag hash.
Once the cache is populated, calculating the rev-list --count for an arbitary hash would go something like this:
# time git describe --tags --abbrev=0 bcc981e9ed84c678533299d7eff17d2c81e4d5de
v4.6-rc5

real	0m0.187s
user	0m0.160s
sys	0m0.020s
# time git rev-list --count v4.6-rc5..bcc981e9ed84c678533299d7eff17d2c81e4d5de
3

real	0m0.009s
user	0m0.000s
sys	0m0.000s

If you add that '3' to the tag's cached rev-list --count value, then you get the final CommitsCount(). You could cache that value as well, but give it a more sensible expiration.

This is even worse when viewing the "releases" page of a large repo with quite some releases as it gets called for every tag.

@DeX77 Nice catch, I missed that. All the more reason to cache the rev-list --count at each tag...
Repro is sitting up here if anyone wants to see this painful experience in action: https://git.uplinklabs.net/gogs/steven/linux

Note that there is a rather new and unknown git feature called bitmap indexes (it think it is not activated by default) which already does exactly this: caching the number of commits reachable from a given commit and that could be used as a cache for rev-list --count. This article is an amazing introduction to this feature: http://githubengineering.com/counting-objects/
Note that once it is activated, rev-list --count automatically takes advantage of it. So fixing this issue could probably be as simple as activating bitmap indexes on the gogs side.

Handy. I'm adding this to /etc/gitconfig on my git host and repacking all the repos to see if that helps:
[repack]
    writeBitmaps = true

Hmm, that didn't seem to help. I verified the bitmap was created:
# find . | grep pack-
./objects/pack/pack-02667941ad4b27d69d787dcdafe2e2cc8bff78fc.pack
./objects/pack/pack-02667941ad4b27d69d787dcdafe2e2cc8bff78fc.idx
./objects/pack/pack-02667941ad4b27d69d787dcdafe2e2cc8bff78fc.bitmap

But I don't think git rev-list --count is taking advantage of the new bitmap indexes:
# time git rev-list --count 8ef3ad9a813abdf0817acb0b2be30be70bf25a9b
589482

real    0m6.875s
user    0m6.810s
sys     0m0.050s

I looked at the rev-list.c source and apparently there's an extra flag that makes it behave. Wonder if there's a config option too...
# time git rev-list --use-bitmap-index --count 8ef3ad9a813abdf0817acb0b2be30be70bf25a9b
589482

real    0m0.426s
user    0m0.360s
sys     0m0.060s

Great! So should this become the default configuration for Gogs?

I don't think it's the full solution. The repository below is fully repacked and has the bitmap indexes. I've also patched gogs to add the --use-bitmap-index flag to rev-list --count invocations.
https://git.uplinklabs.net/gogs/steven/linux
This particular page loads significantly faster. Well... relatively. It's still very slow compared to e.g. GitHub. The "Releases" page is still completely inoperable though -- Chrome gives up waiting for it after a while. So it still seems like some caching mechanism is needed, even if that doesn't particularly help initial page load times.

The commit count will make file history a problem too, e.g. via Repo.FileCommitsCount():
$ time git rev-list --use-bitmap-index --count HEAD -- MAINTAINERS
6145

real	0m3.321s
user	0m3.220s
sys	0m0.080s

I think the simplest solution is to stop caring about the number of commits unless a way to cheaply calculate them can be found. In a lot of places, the rev-list count is only a mildly interesting statistic, without any practical use.

In a lot of places, the rev-list count is only a mildly interesting statistic, without any practical use.

Indeed, this info is mostly not very usefull anyway. Don't know if gogs can do that but what about doing that async and displaing it "when its done" ?

what about doing that async and displaing it "when its done" ?

I think that just shifts the pain from the end user to the server admin. The work still happens, it's just not apparent to the end user. Personally I'd rather not have my server spend the CPU cycles on it.

Note: show Commits and Releases only in repositories home page as GitHub does.

Merge this thread to https://github.com/gogits/gogs/issues/3518

@Unknwon I don't see how the aesthetic design issue you referenced is relevant to a crippling performance issue.

Hmm, you're right.


"import public" not handled correctly when generating import statements

The way we currently generate import statements (for the .proto file -> module map) doesn't take into account messages used transitively through import public.
For example,
// File: a.proto
message A {}

// File: imports_a_publicly.proto
import public "a.proto";

message ImportsAPublicly {
  A a = 1;
}

// File: uses_a_transitively.proto
import "imports_a_publicly.proto";

message UsesATransitively {
  A a = 1;
}

Imagine we produce a separate module for each of the generated Swift files and use the following mapping when generating uses_a_transitively.proto:
mapping {
  module_name: "ImportsAPublicly"
  proto_file_path: "imports_a_publicly.proto"
}
mapping {
  module_name: "A"
  proto_file_path: "a.proto"
}

Because ProtoFileToModuleMappings only looks at the import statements literally present in the .proto file, and because Swift module imports aren't transitive, uses_a_transitively.pb.swift does not get the import A line that it needs to successfully compile.

There are two ways to solve this:

At the dependency level, just do imports for all public deps (recursively)
Hack it, and never use the dependencies, and instead keep track of what modules we used for type references and just import those.

The first could end up importing things the file doesn't need (because it doesn't actually use one of those public imports of its dependency), but it is likely the cleaner thing to do.


Customize Django Admin for Title

IMO, it would be better to removing the Django branding on the admin backend and also from the site title, replacing them with ReadTheDocs branding. If the community see it as a good idea, I can actually make a PR on that :laughing:

Hi @nkwate-kazim! Thanks for your suggestion!
I think it doesn't worth the effort since it's an internal page that just a couple of people see it. I mean, I prefer to use your contributing time in something more valuable that you can pick from the issues list: a good place to start is "Good First Issue" tag.

Closing due to inactivity. Feel free to reopen if you want to take this on.


Can't select all output characters at once

Describe the bug
Can't select all output characters at once, users must select all characters using mouse line by line.
无发一键快速选中所有输出的字符，必须要通过鼠标一行一行的操作，尤其是当输出信息比较多时，不是很方便呢。
To Reproduce
NULL
Expected behavior
Users could select all output characters using A hot key, like CTRL+A.
Screenshots
NULL.
Desktop (please complete the following information):

OS: [Windows]
OS Version [Win10]
NxShell Version [1.6.3]

Additional context
Aoligei...
加油，奥里给！

Will implement in next release

fixed in v1.6.4


Error: Timed out after 30000 ms while trying to connect to Chrome!

(node:20081) UnhandledPromiseRejectionWarning: Error: Timed out after 30000 ms while trying to connect to Chrome! The only Chrome revision guaranteed to work is r549031
at Timeout.onTimeout (/home/ubuntu/affiliate/node_modules/puppeteer/lib/Launcher.js:266:14)
at ontimeout (timers.js:482:11)
at tryOnTimeout (timers.js:317:5)
at Timer.listOnTimeout (timers.js:277:5)

The error was on ubuntu,
The message of alex88 helped me to fix the issue before giving more details.


switched from natbib to biblatex + added Makefile

biblatex has more options, such as online.
Makefile introduces "make keep", followed by "make diff" to track any chances.
Great for any minor changes your supervisor asks for.

Thanks, they're great suggestions, but please only submit one change per commit/pull request to keep the repo clean and tidy, ie one separate commit for the Makefile, one separate commit for the change from natbib to biblatex


[ENH] Pivot longer and other fixes

PR Description
Please describe the changes proposed in the pull request:

Clearer error messages for pivot_longer h/t @tdhock
Refactored encode_categorical to use functools.dispatch
select_columns uses variable arguments (*args), allowing for a cleaner selection, without the need for lists for multiple selections.

This PR resolves #836.

PR Checklist

Please ensure that you have done the following:

[x] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

[x] If you're not on the contributors list, add yourself to AUTHORS.rst.

[x] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl

Codecov Report

Merging #837 (bbc5fac) into dev (cdfab2e) will increase coverage by 0.01%.
The diff coverage is 97.50%.

@@            Coverage Diff             @@
##              dev     #837      +/-   ##
==========================================
+ Coverage   95.34%   95.35%   +0.01%
==========================================
  Files          19       19
  Lines        1975     2003      +28
==========================================
+ Hits         1883     1910      +27
- Misses         92       93       +1

It looks like we should merge. Thanks @samukweku!


Clarify security; state that [] is undefined behaviour

Port of #4007 to v3.0.4-dev

Oh I didn't see that there is a 3.1.1 that we're doing before this 3.0.4?
This is why we don't usually post the same fix to multiple branches at once- it's too confusing when there are a lot of notifications.  We usually do the oldest branch, and then after that is accepted and merged we post any forward-port PRs.

That's why this is a draft.


Feature request: Adding an L1 norm calculation

For some use cases an additional compute_norm1 function would be useful. For consistency, however, a conclusion to #796 should be reached first.

Closed by #900


MonkeyMaster plugin not working on AndroidStudio

Even though, I have devices running in my emulator,clicking on refresh in not returning device id
AndroidStudio version : 3.6.3

same issue with 4.0

Sorry for reply so late, it has been a long time since the last commit.
I would refactor it in some days.

After careful consideration, I think it might be a good time to announce that this plugin project is DEPRECATED.
Nowadays, instead of a IDE plugin, a simple script file could do the same thing and might  be even better than this.


Embedded createdAt field is not updated

If I have a model like this:
export class Comment {
  @Field()
  content: string;

  @Field()
  name: string;

  @Field({ timestamp: "createdAt" })
  createdAt?: Date;
}

@TigrisCollection("post")
export class Post {
  @PrimaryKey(TigrisDataTypes.UUID, { order: 1, autoGenerate: true })
  id?: string;

  @Field({ timestamp: "createdAt" })
  createdAt?: Date;

  @Field({ timestamp: "updatedAt" })
  updatedAt?: Date;

  @Field()
  title: string;

  @Field()
  content?: string;

  @Field({ default: false })
  published?: boolean;

  @Field(TigrisDataTypes.ARRAY, { elements: Comment })
  comments: Array<Comment>;
}

And then I run this:
 await setup();
  const db = await tigrisClient.getDatabase();
  const postCollection = await db.getCollection<Post>(Post);

  const post = await postCollection.insertOne({
    title: "A book review of Designing Data-Intensive Applications",
    content: `
        Design Data-intensive Applications by Martin Kleppmann is a must read
        for anyone that loves learning about databases and build distributed systems.
        It covers a large range of topics on the complexity of building distributed systems.
        It is one of my favourite technical books.
    `,
    comments: [],
  });

  post.comments.push({
    content:
      "Thank you for the excellent book review. I've added it to my reading list",
    name: "Henry",
  });

  post.comments.push({
    content:
      "I strongly disagree, everyone should be learning distributed systems from reading academic papers only!",
    name: "Angry Max",
  });

  const updated = await postCollection.insertOrReplaceOne(post);

  console.log(updated.comments);

In the bottom line when I console log the comments the createdAt field is undefined. We don't seem to be processing the embedded array and adding the default fields.

A duplicate of https://github.com/tigrisdata/tigris-client-ts/issues/245


runtimetest/hostname: small fix for null hostname

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

LGTM

LGTM


Create a stats GET API endpoint to fetch stats data

Resolve: https://github.com/xamfy/x-server-stats/issues/5

@xamfy Please review and let me know if there is any change required.

Overall, LGTM. Have left few suggestions that we can incorporate for better code readability.

Thank you for the review. Fixed all the review comments can you please check once.

Could you please suggest other beginner friendly issue?

#14 is more of a reseach(for more stats data) and implementation task. Since we've already implemented stats get API now, we can start adding more data.

#14 is more of a research(for more stats data) and implementation task. Since we've already implemented stats get API now, we can start adding more data.

I wil check


Problem with "Any" Condition in Variable Product Translation Sync

Hi,
First of all, great plugin!
We are facing a issue that when we duplicate a variable product (clicking cross) to other language, the "Any" option values disappear in the front-end of that language. It turns blank (Choose a option, and no options available). To be more exact:
[1st Condition] (Example Structure):  "Structure Type 1"
[2st Condition] (Example Finishing):  "Any Finishing"  - because theres alot of them.
Like i said, it is working on main language but no syncing/appearing properly at translating. Only shows those with 2nd Condition specified.
Can you please help us?
Our website is projectsource.pt.
Thank you,
Diogo Melo - Coder & Designer, Portugal

Can you reproduce this issue on default Wordpress theme (eg Storefront)?
Can you reproduce this issue when all other plugins are disabled except WooCommerce, Polylang and Hyyan WooCommerce Polylang Integration?
What product versions and settings are you using when this issue occurs?

PHP:
WordPress:
WooCommerce:
Polylang: [state if using Polylang PRO]
Hyyan WooCommerce Polylang Integration:
Browser:

Steps to Reproduce

What I Expected
What Happened Instead
WordPress Environment

```
Copy and paste the system status report from **WooCommerce > System Status** in WordPress admin here.
```

You fixed it on ealier updates, thank you! :D  Closed


fix(api): add local.settings.json so add just works

#414 - force push the local.settings.json so that project's azd deploy works without changes.

@glaucia86 Maybe I overwrote your PR? I didn't see this in my API-test branch final push so I'm adding it back.
Its required for local deploy and local run.
@manekinekko @anfibiacreativa Please review and approve


got Problem with relative paths from android gradle plugin 3.5.0

Hello,
I've got an issue using the gradle Plugin. With the last update I'm not able to open relative paths in my project. The issue appeared by upgrading com.android.tools.build:gradle from 3.4.2 to 3.5.0.
thenReturnFile is not able to open asset files and crashes when using mockserver in connected tests with the following error:
E/RESTMock: <- Response FILE READ ERROR
   java.io.FileNotFoundException

We could have brought it to work by using the following code:
context.getAssets()
    .open(**FILEPATH**)
    .use { Log.d("test", it.bufferedReader().readLine()) }

thanks for reporting this, I'll take a look

I've tested this on 3.5.0 gradle plugin today and it works as intended:

thenReturnFile("api/response/GET_200_success.json")
file location: ${rootProject}/app/src/androidTest/assets/api/response/GET_200_success.json

Hmm ok thank you very much for the fast answer. I really appreciate it. This is awkward.


Removing openssl dependency

I wanted to open this issue as a central place for discussing the removal of the openssl dependency. As far as I understand, this is desired, see https://github.com/algesten/str0m/issues/107#issuecomment-1484644296 for example.
It seems that webrtc-rs has a pure-Rust DTLS implementation at https://github.com/webrtc-rs/webrtc/tree/master/dtls. I guess it would need to be forked to be easily embedded in str0m? Has this been considered?

Why is openssl support a problem?
It's performant and vetted and some organizations are not allowed to use other crypto libraries.
Even Webrtc-rs has added dependency on it now: https://github.com/webrtc-rs/webrtc/pull/477
I can understand why one would want a feature to build without it, but removing support seems counterproductive....

I don't see a reason to remove it, it will be there with a feature flag, but i'd like the default to be a rust native crypto.
I'm a bit hesitant about the webrtc-rs DTLS since, afaik, it's only used by webrtc-rs. I think it's bespoke and it's unclear how many crypto experts have read the code.
This is why my preferred path is for someone to engage with the rustls project and do DTLS there.

That makes sense, thank you for clarifying. I'll reword the title and description! :)

I'm a bit hesitant about the webrtc-rs DTLS since, afaik, it's only used by webrtc-rs. I think it's bespoke and it's unclear how many crypto experts have read the code.

I agree with this concern, but it is worth noting that the webrtc-rs implementation is a port of pion/dtls so not entirely bespoke.

Some associated discussion in #413

I did some further investigation. openssl does not support the latest DTLS 1.3 standard yet (https://github.com/openssl/openssl/issues/13900). libWebRTC uses boringssl which is a fork of openssl. I think that means most WebRTC traffic uses DTLS 1.2 (though firefox might support 1.3)
That means a pure DTLS 1.3 implementation wouldn't be of much use to str0m – any attempt to build this with rustls or otherwise must do 1.2 compatibility.

@thomaseizinger maybe good to summarize what you found regarding full OpenSSL during your ice-agent work here.

@thomaseizinger maybe good to summarize what you found regarding full non-openssl feature during your ice-agent work here.

I am not sure I understand. The only reason I made openssl optional is because we only use the ICE agent which doesn't use any crypto besides a sha1-hmac. The only "findings" I have is that aes_128_cm_sha1_80 seems to be so outdated that neither rustls nor ring support it.

RIght that was what I was looking for.
So potentially a rust only implementation could skip the older aes_128_cm_sha1_80 and only implement GCM.

Logic needs to be tweaked a little for this, currently we assume the supported SRTP profiles are common across all crypto implementations

Logic needs to be tweaked a little for this, currently we assume the supported SRTP profiles are common across all crypto implementations

Yeah that is why I argued for not creating too many abstractions regarding the crypto implementations for now. The real requirements will show once another one is integrated :)

And now that Substrate pulled litep2p that pulled str0m I'm also looking forward to get rid of OpenSSL.
OpenSSL is a huge dependency and is a pain to deal with in cross-platform way, so I was avoiding it at https://github.com/subspace/subspace successfully until today.
Looking forward to ability to disable OpenSSL in str0m, even if second option is not yet audited or as performant as OpenSSL.

Sadly Rustls has taken an awkward direction for us here. They are now primarily relying on aws-lc for crypto instead of ring (which is an optional dep). My vision for str0m would be to have a rust native solution and aws-lc is not that. I have made one attempt at writing DTLS myself, but didn't make it very far and of course, rolling our own is only slightly better than using webrtc-rs variant.

They are now primarily relying on aws-lc for crypto instead of ring (which is an optional dep).

My understanding is that aws-lc is just a crypto provider, so it should be perfectly possible to use Rustls with Ring without relying on aws-lc at all.

My understanding is that aws-lc is just a crypto provider, so it should be perfectly possible to use Rustls with Ring without relying on aws-lc at all.

Yeah, I know. My comment is more about being uncertain whether the rustls project is going in a direction is compatible with str0m. The ambitions might be mismatched.

@xnorpx why is openssl a problem? It's native code that isn't rust and thus on a variety of systems it requires more work to make it compile than rustls or similar which I have never had a failed build from. Can you guess why I'm here? It's to implore you to have a pure rust option.
Error building OpenSSL:
*failed* |       Command: cd "/target/release/build/openssl-sys-bf1da25cfeb50714/out/openssl-build/build/src" && MAKEFLAGS="-j --jobserver-fds=16,17 --jobserver-auth=16,17" "make" "build_libs"

@gilescope Rustls and aws-lc is in some respects neither here nor there because that project doesn't have a DTLS implementation either. The requirements are:

DTLS 1.2 (not 1.3!)
Sync
Pure Rust

Any solution in that direction is interesting to explore.

@xnorpx why is openssl a problem? It's native code that isn't rust and thus on a variety of systems it requires more work to make it compile than rustls or similar which I have never had a failed build from. Can you guess why I'm here? It's to implore you to have a pure rust option.
Error building OpenSSL:
*failed* |       Command: cd "/target/release/build/openssl-sys-bf1da25cfeb50714/out/openssl-build/build/src" && MAKEFLAGS="-j --jobserver-fds=16,17 --jobserver-auth=16,17" "make" "build_libs"

I can't see rustls not having a pure rust option in the future. That would be a very retrograde step and would do damage to the wider ecosystem as many tools would only work if you happened to have various native bits properly configured.

From a build perspective OpenSSL is a pain no disagreement there. I fully support having a feature once someone is motivated enough to implement their own DTLS implementation and get it integrated.
This is probably what is closest to @algesten requirement so far but looks WIP atm.
https://github.com/webrtc-rs/rtc/tree/master/rtc-dtls
Maybe @gilescope Polkadot has people/resources to spare to help finalize that work and integrate it?
Some relevant links:

https://github.com/rustls/rustls/issues/40
https://www.reddit.com/r/rust/comments/5tdcj9/rust_and_dtls_need_advice/

Might be worth pointing out in this thread for Windows str0m builds you can now use "wincrypto" featureflag and you don't have to deal with installing/compile OpenSSL/Perl on Windows.


using querySelectors for the filters

Hi,
I'm trying to something that's not currently covered in the filter framework. For some of the element preprocessing, we use class name to denote different type of nodes, so filtering by tag wouldn't work.

Please see the section on filters: https://github.com/domchristie/to-markdown#filter-stringarrayfunction
You can use a function for filtering. e.g.
{
  filter: function(node) {
    return /my-class/.test(node.className)
  },
  replacement: …
}

Thanks I will have a look at that.

Cool. Solve my problem.


Proxy clean-up

What should be done?

Make the polling time of a request for available workers configurable from the configuration file.
Update Pingora to 0.4
Use uuid instead of u128 for request ID.

How should it be done?

We should add a new field to the ProxyConfig struct to keep track of the polling time for available workers in the configuration file, and then keep track of the value with a new field in LoadBalancer, that way we can access the value in each request to perform the sleep. Default poll time should be 20 ms.
Update Pingora and Pingora-related crates to 0.4 version in the Cargo.toml of the miden-tx bin. In the changelog of the version no breaking change seems to appear so it should work out of the box.
Replace the data types. Should work out of the box.

When is this task done?

We can consider it done when a user can set manually any polling time in the configuration file.
All pingora crates are using 0.4.
u128 is replaced with uuid for request ID.

Additional context
No response

Closed by #997.


The occasional mistake of 504

I start a server in docker container, sometimes works fine. But sometimes result 504.
HeadlessChrome version 67.0.3396.79
node server.js of master branch code
error log print as follows:

2018-06-29T02:32:40.577Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-06-29T02:32:59.652Z getting http://www.gamersky.com/
2018-06-29T02:33:16.590Z capture screenshot timed out for http://www.gamersky.com/
2018-06-29T02:33:16.630Z got 504 in 16978ms for http://www.gamersky.com/
2018-06-29T02:34:03.873Z getting http://www.gamersky.com/
2018-06-29T02:34:19.509Z capture screenshot timed out for http://www.gamersky.com/
2018-06-29T02:34:19.511Z got 504 in 15638ms for http://www.gamersky.com/
2018-06-29T02:35:06.833Z getting http://www.gamersky.com/
2018-06-29T02:35:23.078Z capture screenshot timed out for http://www.gamersky.com/
2018-06-29T02:35:23.079Z got 504 in 16246ms for http://www.gamersky.com/

The strange thing is that, it works fine on my computer when not use docker container.
I try to modify the PAGE_LOAD_TIMEOUT,but no effect!
Is there some optimizations to do?thanks!

What kind of CPU/RAM does your server have available? Sometimes this can happen if you run out of RAM. It looks like your requests are pretty spread out so it doesn't look to be a load issue.
Are you seeing Chrome crash before you see those 504s? Does it fix itself after a certain number of requests?

cpu info: Intel(R) Xeon(R) CPU E5-26xx v4
cores: 2
I looked cpu loading too high. Maybe the bottleneck is cpu.

2018-07-03T10:04:38.417Z got 504 in 3ms for http://www.watertu.com/
2018-07-03T10:04:38.423Z getting http://www.gusuwang.com/
2018-07-03T10:04:38.426Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-03T10:04:38.426Z got 504 in 3ms for http://www.gusuwang.com/
2018-07-03T10:04:38.431Z getting http://www.gusuwang.com/
2018-07-03T10:04:38.435Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-03T10:04:38.436Z got 504 in 5ms for http://www.gusuwang.com/
2018-07-03T10:04:38.442Z getting http://www.gusuwang.com/
2018-07-03T10:04:38.446Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-03T10:04:38.446Z got 504 in 4ms for http://www.gusuwang.com/
2018-07-03T10:04:38.452Z getting http://www.lanin.net/
2018-07-03T10:04:38.455Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-03T10:04:38.455Z got 504 in 3ms for http://www.lanin.net/
2018-07-03T10:04:38.462Z getting http://www.lanin.net/
2018-07-03T10:04:38.465Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-03T10:04:38.465Z got 504 in 3ms for http://www.lanin.net/
2018-07-03T10:04:38.468Z Chrome connection closed... restarting Chrome
2018-07-03T10:04:38.469Z Starting Chrome
2018-07-03T10:04:38.480Z getting http://www.lanin.net/
2018-07-03T10:04:38.982Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-03T10:05:08.511Z getting http://www.ih5.cn/
2018-07-03T10:05:09.133Z page timed out http://www.lanin.net/
2018-07-03T10:05:09.243Z got 504 in 30763ms for http://www.lanin.net/
2018-07-03T10:05:38.543Z getting http://www.ems.com.cn/
2018-07-03T10:05:38.575Z page timed out http://www.ih5.cn/
2018-07-03T10:05:42.950Z got 200 in 4407ms for http://www.ems.com.cn/
2018-07-03T10:05:43.112Z getting http://
2018-07-03T10:05:43.112Z invalid URL: http://
2018-07-03T10:05:43.112Z got 400 in 0ms for http://
2018-07-03T10:05:43.118Z getting http://
2018-07-03T10:05:43.118Z invalid URL: http://
2018-07-03T10:05:43.118Z got 400 in 0ms for http://
2018-07-03T10:05:43.123Z getting http://
2018-07-03T10:05:43.123Z invalid URL: http://
2018-07-03T10:05:43.124Z got 400 in 1ms for http://
2018-07-03T10:05:43.134Z getting http://www.kuwo.cn/

Hmm, it looks like Chrome crashed or something and the Prerender server wasn't aware that it crashed so it kept trying to send requests.
After restarting the Chrome server though, you can see that it connects to Chrome. It looks like the request to http://www.ems.com.cn/ worked fine since that took 4407ms to complete...so is it working correctly after it restarts itself? Did you take a look at the high CPU usage? Are you running Chrome with any custom flags?

There is no custom flags to use for the server. Maybe, the reason is that the Chrome server and prerender is running in docker, I can see the highly CPU loading and Chrome always restarting after few requests.
So, is there any advice ?

How high of CPU usage are you seeing? And how many minutes are you seeing in between the Chrome restarts? By default, we have Chrome try to restart itself every 10 minutes if there are no requests in flight.
I'm not sure if this would be due to docker or not, but we haven't run into this issue yet and we are not using docker.

According to my grep log file, I can see the chrome's frequent restarts.
2018-07-06T17:50:46.093Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:51:28.147Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:52:16.246Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:53:25.411Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:53:58.256Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:54:19.559Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:54:40.468Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:54:58.698Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:55:20.281Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:55:46.257Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:56:25.544Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:57:12.047Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:57:30.705Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:58:34.133Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:58:41.467Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:58:57.251Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:59:12.375Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:59:29.916Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T17:59:50.252Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T18:00:56.297Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T18:02:05.559Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T18:02:22.588Z Started Chrome: HeadlessChrome/67.0.3396.79
2018-07-06T18:02:40.224Z Started Chrome: HeadlessChrome/67.0.3396.79

2018-07-11T09:57:31.583Z getting http://www.bailingjie.cn/
2018-07-11T09:57:31.586Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-11T09:57:31.587Z got 504 in 4ms for http://www.bailingjie.cn/
2018-07-11T09:57:32.099Z getting http://www.leadong.com/
2018-07-11T09:57:32.102Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-11T09:57:32.102Z got 504 in 3ms for http://www.leadong.com/
2018-07-11T09:57:32.617Z getting http://www.100yy.com/
2018-07-11T09:57:32.619Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-11T09:57:32.619Z got 504 in 2ms for http://www.100yy.com/
2018-07-11T09:57:33.135Z getting http://www.yunos.com/
2018-07-11T09:57:33.142Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-11T09:57:33.144Z got 504 in 8ms for http://www.yunos.com/
2018-07-11T09:57:33.657Z getting http://www.hongdezk.com/
2018-07-11T09:57:33.660Z { Error: connect ECONNREFUSED <IP_ADDRESS>:9222
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1158:14)
errno: 'ECONNREFUSED',
code: 'ECONNREFUSED',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 9222 }
2018-07-11T09:57:33.661Z got 504 in 4ms for http://www.hongdezk.com/
2018-07-11T09:57:33.790Z capture screenshot timed out for http://www.gfedu.com/
2018-07-11T09:57:33.791Z got 504 in 30057ms for http://www.gfedu.com/
2018-07-11T09:57:33.791Z error closing Chrome tab Error: not opened
at WebSocket.send (/code/node_modules/ws/lib/WebSocket.js:355:18)
at Chrome.enqueueCommand (/code/node_modules/chrome-remote-interface/lib/chrome.js:124:16)
at /code/node_modules/chrome-remote-interface/lib/chrome.js:86:28
at new Promise ()
at Chrome.send (/code/node_modules/chrome-remote-interface/lib/chrome.js:85:16)
at Object.handler [as closeTarget] (/code/node_modules/chrome-remote-interface/lib/api.js:32:23)
at Promise (/code/lib/browsers/chrome.js:148:22)
at new Promise ()
at Object.chrome.closeTab (/code/lib/browsers/chrome.js:146:9)
at Object.server.finish (/code/lib/server.js:277:16)
2018-07-11T09:57:34.177Z getting http://www.5u5u5u5u.com/

I gave up to use docker, and resolved this problem.

Maby, I find the better way to this problem. Docker run with --shm-size 1G and that crashes not exists.
Detail: https://bugs.chromium.org/p/chromium/issues/detail?id=571394

Oh yeah? So everything looks good after giving it access to more RAM?

yeah

@thoop @lican09 Problem exists for chrome headless running on docker and can be solved be either setting --shm-size or starting chrome with flag --disable-dev-shm-usage. Second option is useful for AWS. More information can be found on https://developers.google.com/web/tools/puppeteer/troubleshooting

Thanks for the info!

@thoop @lican09 Problem exists for chrome headless running on docker and can be solved be either setting --shm-size or starting chrome with flag --disable-dev-shm-usage. Second option is useful for AWS. More information can be found on https://developers.google.com/web/tools/puppeteer/troubleshooting

Thanks! It saved my day

none of the solutions above works for me. :(


Optional lists with mandatory items (license, security_and_privacy)

There are cases where a list must contain at least one item but at the same time the list is optional in the parent object, so the limitation does not make much sense:

https://github.com/RDA-DMP-Common/RDA-DMP-Common-Standard/blob/master/examples/JSON/JSON-schema/1.0/maDMP-schema-1.0.json#L499 (actually in README is 0..n - so maybe just a "bug" in the schema)
https://github.com/RDA-DMP-Common/RDA-DMP-Common-Standard/blob/master/examples/JSON/JSON-schema/1.0/maDMP-schema-1.0.json#L678 (same, again 0..n in [https://github.com/RDA-DMP-Common/RDA-DMP-Common-Standard#properties-in-dataset])

@SotosTsepe can you please check this and update the JSON schema if necessary

@MarekSuchanek Thanks for your feedback! Putting the property minItems set to 1 ensures that an empty array is not given, because it doesn't make sense. This property doesn't affect the cardinality of license for example; one can still omit license and the JSON will pass validation, but were he to specify it, the array would have to have at least one object contained.

OK, so if there is no license, it should be omitted ("we don't care about licensing at all"). Implementation-wise it is more complicated when compared to allowing empty array (which means "there is no license"). One must have extra checks to leave it out when list of licenses is empty during importing and have default for non-present list when exporting. But if it has some strong meaning in the schema, why not...
Nevertheless, it would be good to explain it briefly in the description as I think this may cause issues to others as well.

We eventually decided not to put any constraints on the specific serialization. In this case JSON specification. This means that both empty arrays, as well as simply omitting a field is permitted. There is a new version of JSON schema (1.1) that permits that.

OK, great... I will revisit the template for DSW and update it according to the new version.
(Will be there also a git tag for reference?)

Yes, it will be 1.1- later today.


Server tests

Add TokenVerifier tests and fix the server delay by removing sleep from TokenVerifier.

Note that this pull request also addresses issue #4 (server-side delay)


Enable Istio injection for namespaces, for blue/green namespaces including jmeter-service pod

The internal service routing between blue/green services is not done correctly.
Reproduction of the bug:
Prerequisites:
onboarded carts service as described in the 0.4.0 docs.
Bug:
When accessing the carts.sockshop-production endpoint from another service inside the cluster the routing is done via round-robin, i.e., sometimes the blue and sometimes the green version of the service is called.
Expected behaviour: only the service that is defined in the VirtualService is called, e.g., 100% to blue.
Please note:
This does not apply to services that are exposed via an http endpoint, they are working fine. I.e., if accessing carts.sockshop-production.xx.xxx.xx.xx.xip.io everything is working as expected.

To make this work, we need Istio sidecar injection in namespaces which are configured to use blue/green deployments. A newer version of Istio with the possibility of sidecar injection has been introduced with PR #719 .
Additionally, we found that for internal routing via VirtualServices requires named ports to be used in the Service definitions for the services to be accessed via VirtualServices. In the VirtualService definitions, we also need to register the cluster-internal URLs (e.g. carts and carts.sockshop-production.svc.cluster.local). This will be fixed in the new version of the Helm service that's currently being developed (Issue #611 )

Just to add to this discussion. A flexible option would be if keptn can provide both internal and external URLs as part of the cloud native event. In this case the service itself, e.g.: JMeter, Neotys, ... can decide on whether they want to test against the internal or external endpoint

The istio-injection can for now be enabled by executing kubectl label namespace sockshop-staging istio-injection=enabled. Therefore this can be solved with a Doc-entry instead of implementing this in the helm service for now.


More info on https://github.com/square/okhttp/issues/1531

I've continued to have the issue described on #1531 however today I realized I can turn off IPv6 on my router and as soon as I did that (after a router reboot) the page started loading in 1 second instead of 60 seconds. So I don't know if this is something that needs to be fixed or the expected behavior but I figured I should report it.

It's difficult to say what OkHttp should do here. From its perspective the IPv6 address should work. Happy Eyeballs might be our best bet. Other ideas?

Figured as much. Personally I think my ATT router has issues with IPv6 because if enabled Chrome can take a long time to load Google searches, but when disabled it will load them right away. And it isn't just on one machine, happens on both my Mac and my Linux machine. So maybe/hopefully this is just an isolated issue affecting only me :)

Cool. Hopefully this gets better with happy eyeballs.


There, input your own GPS coordinates..... how?

Thanks for the code, it works perfectly, but the documentation is not clear.
After wifi connection how to send new gps coordinates?
Is it possible to do it from the smartphone in those two minutes?
This step is not explained.

Hi, thanks for catching this! I've fixed it in the docs. Apologies for the confusion. After the wifi connection, go to the website <IP_ADDRESS>, there, you can input the GPS coordinates.

Thanks!


[bug] Height properties not changing 2 - flutter web

Using #300 as a reference, the .h and .sh properties still aren't changing on the latest version (5.0.1+2), that should have fixed this bug.
But everything is ok when i set flutter_screenutil: 5.0.0 on pubspec.yaml.
flutter version 2.8.0 stable

As said in #295 something was broken in this transition.

Thank you, I will find time to upgrade the flutter version and troubleshoot this issue,

Set splitScreenMode: false

why it's not the default, @lizhuoyuan ?

Set splitScreenMode: false


windows does not support export COLUMNS=512

need an alternate way to specify windows terminal width control

checking if stty [column width] would work instead


@conda-forge-admin, please re-render

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as requested, but nothing actually changed.


Add policy compliance check binary.

Adds a check_policy_compliance binary with the following usage:
check_policy_compliance: This tool takes an IR representation of a system and returns whether it is policy compliant.

  Flags from src/backends/policy_engine/souffle/check_policy_compliance.cc:
    --ir (the IR file); default: "";
    --sql_policy_rules (file containing the SQL policy rules); default: "";

This PR also adds a simple sh_test with the following usage:
Usage:
  check_policy_compliance_test.sh \
     pass|fail <ir_file> <sql_policy_rules_file>

The test script can be improved further. e.g., instead of adding 'passorfail` expectations to the script, we can have it encoded in the given IR file itself (e.g., https://github.com/google-research/raksha/blob/953dee1139eb1d71e949d12010941cd3e63919a6/src/analysis/souffle/tests/arcs_manifest_tests_todo/fail_no_inputs.arcs#L3). However, we are starting simple to get this tool in place and unblock other work.

@asejfia FYI.

@markww PTAL.


Typo FILE_OBECT -> FILE_OBJECT

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


Docs no longer showing API

RTD builds somehow broke and now the API page does not display any more. Wiping and will rebuild in ~ 2 days. H/T @aphearin

Keep me posted.

Needed an __import__(Corrfunc), addition of root directory to sys.path. That seemed to solve it. I get warnings from local sphinx about duplicate entries - but the docs build fine locally and on RTD.

Phew!


OOM

torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 508.00 MiB (GPU 0; 11.70 GiB total capacity; 11.32 GiB already allocated; 37.88 MiB free; 11.33 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
`+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 545.23.08              Driver Version: 545.23.08    CUDA Version: 12.3     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA GeForce RTX 4070 Ti     On  | 00000000:2B:00.0  On |                  N/A |
|  0%   42C    P8              13W / 285W |     51MiB / 12282MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|  No running processes found                                                           |
+---------------------------------------------------------------------------------------+
`
How much VRAM is needed to run?
(Never mind. Saw it needs around 20gb) - please delete

Never mind. Saw it needs around 20gb


chore: use roboto font for docs

"Roboto" only for docs
Preview: https://dvc-org-font-roboto-onl-glzpuy.herokuapp.com/

I like it much more than Jost; what do you think?

To me the new one looks a bit weird :) Also, not consistency (docs and outside) - which is not good. Any other fonts to try? @0x2b3bfa0 are there doc websites that you like?

TL;DR, documentation should preferably use a clear, run-of-the-mill font.

As a person with low vision, I hate the trend of these thin fonts. dvc.org is very difficult to read for me. Also, these thin fonts only look good on higher resolution displays. My 2cs.

Closing for now. Not a priority. Also we need to find better fonts I think if we want to change it.


Slider dimensions

Whenever I add a picture to the slider it expands all over the page, no matter what size the picture has or what dimensions I set in the admin panel.

Please use the community forum to get help. Also, you should make sure you have the correct dimension set.


Clean up nix CI and add GHC9 build

Merged the two separate build and unit test workflows into one, as it seems pointless to run both separately. Also added a build of the kore-exec binary via GHC9 to make sure we keep all our deps in sync across both versions.

@F-WRunTime In order to merge this, we need to do the following:

remove merge rights from everyone (except admin)
merge this PR
update all PRs to be up to date with master
give merge rights back to everyone

remove status checks from required

@ana-pantilie I can also Merge without waiting for requirements to be merged? Based on what you're asking for I believe that meets the need here as well no? I can go ahead and just merge it and go through PRs to update the branches.
Give me the go ahead and will begin to execute. Please ping me on slack if you do not get a response from me within a day. Thank you.

Talked with Sam and will proceed.


Improve distro coverage on molecule tests

We should check if it's possible to have an openwrt docker container image to use in molecule testing.

Currently, this fails on missing bash in the image. :sigh:


tools: remove unused buildozer in check_format.py

Commit Message: remove unused buildozer in check_format.py
Additional Description:
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

The CI failure is caused by the network problem:
ERROR: An error occurred during the fetch of repository 'com_github_unicode_org_icu':
   Traceback (most recent call last):
	File "/build/bazel_root/base/external/bazel_tools/tools/build_defs/repo/http.bzl", line 132, column 45, in _http_archive_impl
		download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://github.com/unicode-org/icu/archive/release-72-1.tar.gz] to /build/bazel_root/base/external/com_github_unicode_org_icu/temp18178779363271786173/release-72-1.tar.gz: java.net.SocketTimeoutException: Read timed out

https://dev.azure.com/cncf/envoy/_build/results?buildId=144633&view=logs&jobId=8c169225-0ae8-53bd-947f-07cb81846cb5&j=8c169225-0ae8-53bd-947f-07cb81846cb5&t=4d0399a5-fd0d-5446-d022-32daa00183cc

i believe this is required by the envoy_build_fixer.py script

Got it. Thanks!

The CI failure is caused by the network problem:

its caused by a failure of the "cache-warming" run - i have kicked it


Add Support for Thomas Jefferson University

Thomas Jefferson University
1020 Walnut St, Philadelphia, PA 19107
(215) 955-6000

@Omnitragedy Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


files [dumper]: ES dump/load

Files fields require special dumping/loading regarding how they are indexed in ES:

[ ] Instead of a dictionary-like structure, files.entries should be indexed as a list of objects, so that it is possible to query for e.g. `filename:"article.pdf"
[ ] When loading from ES, bucket and files information has to be parsed to make them accessible via record.files and record.bucket

I've linked the issue into the product board, and I'm closing this issue here, as I don't see we will fix this anytime soon.


Ignore test as this is failing in CI machines. Need to revert once in…

…fra is fixed
Fix Issue # .

[ ] I ran Unit Tests locally.

For significant contributions please make sure you have completed the following items:

[ ] Design discussion issue #

[ ] Changes in public surface reviewed

[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.

[ ] The PR will trigger build, unit tests, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you.
If you want to to re-run the build/tests, the easiest way is to simply Close and Re-Open this same PR. (Just click 'close pull request' followed by 'open pull request' buttons at the bottom of the PR)

Please follow [these] (https://github.com/Microsoft/ApplicationInsights-dotnet/blob/develop/.github/CONTRIBUTING.md) instructions to build and test locally.

Issue isolated to a single bad agent. Mergeing now. WIll come back and renable once the rogue agent is fixed :)


Missing types refs when using jsx accolade

Bug Report
This is a sample code for have a clean issue.
import React from 'react';
class A {
	Renderer = RendererA;
}
class B {
	Renderer = RendererB;
}

const RendererA = () => {
	return <div> A </div>;
};

const RendererB = () => {
	return <div> B </div>;
};

const TestRendererTypes = () => {
	const AB: A | B = null; // just for the demo, AB can be A or B !

	AB.Renderer; //🟩 okey cool,ts find all refs !
	const LostRef = <AB.Renderer />; //🟥 not cool ,ts seem remove refs !
	return LostRef;
};

playground

🙁 Actual behavior
Ts decode only one reference to a function jsx when using inside  tag </>
It should show both Renderer ref here !

🙂 Expected behavior
We should get all refs like that's . (working if use only a variable ref without a jsx tag <></> )

The example is basic, but inside my project i losing all refs, and this is a issu if i need fast track where a rendering is pointing !
And also will give incorrect props Intelisence !
Example here is missing all my others plugins !

But if i add a direct ref like that's , i can get all refs, but is polluting my code

It seem that's TS have some difficulty to understand JSX  !
Is there a hacky and (clean way) to fix this ? or tell me if you guys have temp solutions  ?
I hope i exposed correctly my issue !
thanks @+

TSC: Version 4.2.3
Version : 1.55.0 (user setup)
Commit : c185983a683d14c396952dd432459097bc7f757f
Date : 2021-03-30T16:01:55.261Z
Electron : 11.3.0
Chrome : 87.0.4280.141
Node.js : 12.18.3
V8 : <IP_ADDRESS>-electron.0
OS : Windows_NT x64 10.0.19042

Doesn’t repro for me. Can you share a project/repo?

Doesn’t repro for me. Can you share a project/repo?

hum wait ! , how did you get this work !! do you use a secret options or a kind of blackMagic ?
It also not work in a ts playground , and i tried all versions and many advance settings !

Are you able to tell me if you have special settings or versions ?
link to playground

Doesn’t repro for me. Can you share a project/repo?

okey sry i look the img too fast, yes you get the same issue,, this is what a i mean.
I should get this .

Oh, you’re talking about definitions, not references. That repros.

Oh, you’re talking about definitions, not references. That repros.

ho yes, i'm sorry for the confusion, i'm not always comfortable with english terminology.
thanks for the information !

No worries, TypeScript terminology is confusing even for most native English speakers 😄

i remake the issu, wihout JSX tag, because this issue also happen without react.
playground
class A {
	Renderer = RendererA;
}
class B {
	Renderer = RendererB;
}

const RendererA = (prop:{instance?:A}) => {};
const RendererB = (prop:{instance?:B}) => {};

declare const Test:A|B;
// should accept instance A|B And Point to RendererA & RendererB, why only RendererA ?

Test.Renderer({instance:Test});

i hope i can have more explanation !


How to compress png without losing transparency?

Apologies for the newbie question, but what options exist for reducing png filesize? I noticed other formats have quality settings, and the png does not -- presumably because it is lossless.
So I figured I could convert pngs to jpegs employing the lossy compression and then convert them back to pngs. But in this scenario I've lost the transparency and I'm not sure A) how to convert a color in the jpeg back into transparency in the png, and B) whether this whole approach is even how to go about reducing png filesize.
Thank you

Hello, you are correct that PNG is lossless. JPEG does not support an alpha channel, hence the loss of transparency you're experiencing.
Further reduction in PNG filesize can be obtained by using a quantising (and therefore lossy) tool such as pngquant.

Thanks I was able to get compress the image using pngquant.
I was also able to use sharp to convert from png to webp and back to png (which is super fast and kept the transparency).
I'll have to play with it all more.


<EMAIL_ADDRESS>+<EMAIL_ADDRESS>works but 3.1.1 does NOT

I spent a lot of time trying to understand why the examples from this repo were working and not my specs files...
I found out that the examples of this repo work even with after updating cypress via npm i<EMAIL_ADDRESS>-D.
The issue seems to be related to<EMAIL_ADDRESS>In order to make my tests work, I had to:

npm uninstall cypress-image-snapshot
npm i cypress-image-snapshot -D

I hope this can help others.

I'm using 4.0.2 Cypress with cypress-image-snapshot 3.1.1 and its working
thanks for the heads up

I spent a lot of time trying to understand why the examples from this repo were working and not my specs files...
I found out that the examples of this repo work even with after updating cypress via npm i<EMAIL_ADDRESS>-D.
The issue seems to be related to<EMAIL_ADDRESS>In order to make my tests work, I had to:

npm uninstall cypress-image-snapshot
npm i cypress-image-snapshot -D

I hope this can help others.

After updating cypress, I got error like
args.find is not a function
below link helped me to fix this error.
6391


Add capability to copy claims from existing JWT in header

Add the ability to specify location (header field) of an existing JWT to copy a configurable list of claims from.
Use case would be to proxy certain claims (jti, aud, exp, etc) from the consumer directly to the upstream system as-is.

Hi @rsbrisci and @jeremyjpj0916, is this something you guys would consider including as part of this plugin?


Add scan-websites

Closed with the spiritual successor. https://github.com/cds-snc/automatic-website-scanning/issues/19


SW-2490 bump version to pick up checkbox changes

new version coming

@hannah-cai can you check the vercel link here?


Nix flake cannot be entered

Description
As of 126964483d188c2362393d2a6eea8c65dfc43097, entering the Nix flake with nix develop fails with:
Warning: The package list for 'hackage.haskell.org' is 19816 days old.
Run 'cabal update' to get the latest list of available packages.
Resolving dependencies...
make-install-plan: dieVerbatim: user error (Error: make-install-plan: Could not resolve dependencies:
[__0] trying: haskell-language-server-<IP_ADDRESS> (user goal)
[__1] trying: haskell-language-server:+hlint
[__2] trying: hls-hlint-plugin-<IP_ADDRESS> (dependency of haskell-language-server
+hlint)
[__3] next goal: hlint (dependency of hls-hlint-plugin)
[__3] rejecting: hlint-3.8 (conflict: hls-hlint-plugin => hlint>=3.5 && <3.7)
[__3] trying: hlint-3.6.1
[__4] next goal: ghc-lib-parser (dependency of hls-hlint-plugin)
[__4] rejecting: ghc-lib-parser-<IP_ADDRESS>31121, ghc-lib-parser-<IP_ADDRESS>31009
(conflict: hlint +/-ghc-lib => ghc-lib-parser>=9.6 && <9.7)
[__4] trying: ghc-lib-parser-<IP_ADDRESS>40109
[__5] next goal: base (dependency of haskell-language-server)
[__5] rejecting: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (conflict: ghc-lib-parser =>
base>=4.16.1 && <4.19)
[__5] skipping: base-<IP_ADDRESS> (has the same characteristics that caused the
previous version to fail: excluded by constraint '>=4.16.1 && <4.19' from
'ghc-lib-parser')
[__5] rejecting: base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS> (constraint from non-upgradeable
package requires installed instance)
[__5] fail (backjumping, conflict set: base, ghc-lib-parser,
haskell-language-server)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: hlint, ghc-lib-parser,
hls-hlint-plugin, haskell-language-server, base, haskell-language-server:hlint
Try running with --minimize-conflict-set to improve the error message.
)

Environment
My /etc/nix/nix.conf is as follows:
extra-substituters = https://cache.nixos.org https://cache.iog.io
extra-trusted-public-keys = hydra.iohk.io:f/Ea+s+dFdN+3Y/G+FDgSq+a5NEWhJGzdjvKNGv0/EQ= cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
experimental-features = nix-command flakes auto-allocate-uids configurable-impure-env
allow-import-from-derivation = true
build-users-group = nixbld
trusted-users = root smelc

I'm using Ubuntu 22.04.
Context
I'm not savvy with Nix, so unable to investigate this myself; but happy to try fixes on the behalf of others :slightly_smiling_face:
Workaround
This is non-blocking, as I succeeded building with cabal alone (outside the Nix flake), using cabal <IP_ADDRESS> and GHC 8.10.7; but it is surprising.

This is resolved due to recent work to upgrade Nix configuration in: https://github.com/IntersectMBO/cardano-addresses/pull/231


Option to not need animations

Would be nice to have a plain show/hide mode without angular/animations dependency.

That's not possible sorry. Angular animations are required for all the projects using animations.


Provide an option to write fims to a central library location

Provide an option in create_fim_lib process to write fims in a provided fim_library directory using the following structure.
|-- 2820002
|   |-- 2820002.db
|   |-- z_0_0
|   |   |-- f_10804.tif
|   |   |-- f_11683.tif
|   |   |-- f_12562.tif
|   |   |-- f_13440.tif

Duplicate of #169


Alert on variant detection

I noticed that a specimen with a variant image (second image was open fragment envelope) was renamed BARCODE_A. There was no alert issued to the console. I assume the first taken image of a specimen is the "original", and timestamps could be used to assume the later images should be the variants with suffix(es) appended, but an alert would let the technicians double check these.

Would you want an alert displayed on the command line or in the log file or both?

Both is better I think. For the log it could just be an addition to "status_details".