Feat: Add resource job

the answer from @paperspace-philip:

datasources get resolved immediately, and we don’t have a job storage until a machine is created, so depends_on doesn’t work.


Possible integration point with openai/whisper output

Hello, I was wondering if it possible to add a new method to support whisper transcribe output and convert that to the supported output formats.
This is currently how it's done at whisper
https://github.com/openai/whisper/blob/9f70a352f9f8630ab3aa0d06af5cb9532bd8c21d/whisper/utils.py#L63
def write_srt(transcript: Iterator[dict], file: TextIO):
    """
    Write a transcript to a file in SRT format.
    Example usage:
        from pathlib import Path
        from whisper.utils import write_srt
        result = transcribe(model, audio_path, temperature=temperature, **args)
        # save SRT
        audio_basename = Path(audio_path).stem
        with open(Path(output_dir) / (audio_basename + ".srt"), "w", encoding="utf-8") as srt:
            write_srt(result["segments"], file=srt)
    """
    for i, segment in enumerate(transcript, start=1):
        # write srt lines
        print(
            f"{i}\n"
            f"{format_timestamp(segment['start'], always_include_hours=True, decimal_marker=',')} --> "
            f"{format_timestamp(segment['end'], always_include_hours=True, decimal_marker=',')}\n"
            f"{segment['text'].strip().replace('-->', '->')}\n",
            file=file,
            flush=True,
        )

Hi, I'm happy to add support for this :) As I understand it, you would like to be able to do something like the following?
import whisper
import pysubs2

model = whisper.load_model("base")
result = model.transcribe("audio.mp3")

subs = pysubs2.load_from_whisper(result)
subs.save("audio.ass")

I'm not familiar with the Whisper library, so please correct me if necessary. However, from the write_srt() code it looks that it should be very easy to get the data.

Indeed something similar yeah, that support pysubs2 wide formats

I've released version 1.5.0 of the library where pysubs2.load_from_whisper(result) as in the example above is implemented. Please upgrade :)

Thank you for your quick implementation.


json.decode can't parse integers over 64 bit

I'm trying to do the following
const payload = '{"connectionID":15852666135343119227}';
print(json.decode(payload));

prints
{connectionID:<PHONE_NUMBER>5343118000.0}

A json number is not limited to only 64bits.
Is there currently a workaround for that?

moved to dart-lang/convert#19


IsFingerHighConfidence in SkeletonDataProviderOVR returns true by default

I'm wondering if that's expected behavior? If PosingIDsToFinger doesn't have the value in dictionary, should it return false or am I reading that wrong?

Good question. And sorry for the super-delayed answer.
That Dictionary actually has all fingers in it, except for
        Invalid,
        Hand_Start,

Which are not fingers. So it does not really meatters if it returns true or false.
I guess for clarity it should return false so I will change it :)


Adding Query.all to documentation

This PR adds Query.all to the documentation from PR #223.

Coverage remained the same at 79.032% when pulling 34c3c0711f0ebd5b1c84691a58f87f9ff0ffe2b1 on fishcharlie:queryall_docs into 933016b244ae28807107c0a0ecd43fabc713eb05 on automategreen:master.

Coverage remained the same at 79.032% when pulling 34c3c0711f0ebd5b1c84691a58f87f9ff0ffe2b1 on fishcharlie:queryall_docs into 933016b244ae28807107c0a0ecd43fabc713eb05 on automategreen:master.

Coverage remained the same at 79.032% when pulling 34c3c0711f0ebd5b1c84691a58f87f9ff0ffe2b1 on fishcharlie:queryall_docs into 933016b244ae28807107c0a0ecd43fabc713eb05 on automategreen:master.

@brandongoode Travis CI failure looks like a timeout issue. Maybe retrying would fix it?


Add Lambert Azimuthal Equal Area CoordSystem

This is missing a what's new entry.
I am not sure what to do about tests

The PR that adds Labert Azimuthal Equal Area to cartopy can be found: https://github.com/SciTools/cartopy/pull/619


⚠ Samsung has degraded performance

In 3d59caf, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2729 ms

Resolved: Samsung performance has improved in e638c99.


Spout Receiver not asking for Sender

Hello. I am new to Unity, so it might be very simple question...
but I can't find the answers. so I had to ask here.
I have loaded your demo and it worked, but when I tried to create the same thing by copying files from demo, it does not work.
I also followed the General Setup and Receiving textures instructions in ReadMe.txt
Problem is...
My Spout Receiver script is aksing for Sharing Name, and there is no Select sender.
how can I solve this?
(Maybe I forgot to move important file from demo project? I copied - NativeSpoutPlugin.dll, Spout.cs, SpoutReceiver.cs and TextureInfo.cs)
this is how it looks on my project

thank you in advance!

Oh... I just realised, I have to ask one more question. does it not work on 2D project?
Because I created this project by creating empty 2D project...


Fix MSSQL Decimal bug

We interpret sqlalchemy.DECIMAL as GraphQLDecimal, and decimal.Decimal is an allowed type for input. The problem is that sqlalchemy.DECIMAL raises an error when the input argument is has too many decimals, which decimal.Decimal allows. My test case in this PR reproduces this bug.
A bigger problem is that, decimal.Decimal(float(1.1)) is Decimal('1.100000000000000088817841970012523233890533447265625'), so deserializing a float value as GraphQLDecimal can inflate its precision and cause errors where they shouldn't exist.
TODO:

fix the problem

Is the plan to cap the precision of our GraphQLDecimal type, or is there a better solution?

Is the plan to cap the precision of our GraphQLDecimal type, or is there a better solution?

Or make sure "1.1" goes back to "1.1" when serialized and deserialized. IMO the second part of this problem is bad, and the first is maybe not even a problem. If someone intentionally queries with a super precise float maybe it's better to get an error than to get a result.

Best solution for now is to use str values for GraphQLDecimal fields.


Spike: Can we define Spark Pool as "code" / config similar to how databricks does it:

https://github.com/Azure-Samples/modern-data-warehouse-dataops/blob/main/e2e_samples/parking_sensors/databricks/config/cluster.config.json
Assigned to Anuj

The short answer is yes. Essentially, the following resources can be deployed as a code:

Synapse Workspace
Dedicated SQL Pools
Spark Pools
Storage (ADLS)
Security around these resources

$ az deployment group create --name SparkPoolDeployment --resource-group rg-learning-synapse --template-file ./sparkpool.bicep  --parameters @sparkpool.parameters.json
An existing spark pool can also be updated, especially to include the external package dependencies using the requirement.txt/environment.yml file as shown below:
$ az synapse spark pool update --name SparkPoolNew --workspace-name ws-learning-synapse --resource-group rg-learning-synapse --library-requirements environment.yml
Command group 'synapse' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
{
"autoPause": {
"delayInMinutes": 15,
"enabled": true
},
"autoScale": {
"enabled": true,
"maxNodeCount": 10,
"minNodeCount": 3
},
"cacheSize": 0,
"creationDate": "2021-07-05T01:05:44.843333+00:00",
"customLibraries": null,
"defaultSparkLogFolder": null,
"dynamicExecutorAllocation": {
"enabled": false
},
"id": "/subscriptions/xxxxxxx/resourceGroups/rg-learning-synapse/providers/Microsoft.Synapse/workspaces/ws-learning-synapse/bigDataPools/SparkPoolNew",
"isComputeIsolationEnabled": false,
"lastSucceededTimestamp": "2021-07-06T03:49:19.320000+00:00",
"libraryRequirements": {
"content": "name: stats2\nchannels:\n- defaults\ndependencies:\n- bokeh\n- numpy\n- pip:\n  - matplotlib\n  - koalas==1.7.0",
"filename": "environment.yml",
"time": "2021-07-06T03:49:17.336629+00:00"
},
"location": "australiaeast",
"name": "SparkPoolNew",
"nodeCount": 10,
"nodeSize": "Small",
"nodeSizeFamily": "MemoryOptimized",
"provisioningState": "Succeeded",
"resourceGroup": "rg-learning-synapse",
"sessionLevelPackagesEnabled": true,
"sparkConfigProperties": null,
"sparkEventsFolder": null,
"sparkVersion": "2.4",
"tags": {},
"type": "Microsoft.Synapse/workspaces/bigDataPools"
}`


Add libcheck tests

Adds a test to verify the library files build correctly under --strictNullChecks,--noImplicitAny, and --noImplicitThis. Should catch the issue reported in https://github.com/Microsoft/TypeScript/issues/9244.
Currently this is failing because of https://github.com/Microsoft/TypeScript/issues/9248.

Looks good but tests seem to be failing.

👍

👍


How to control the returned file path from shebang tool

Hi, first, this project is quite nice, thanks for your work!
I have one issue when I try to use this tool to generate wxWidgets' build options, see my post here:
The shebang command to run the bash script under Windows CMD
One issue is that under Windows, I would like the return path has normal Windows style, such as:
-IF:/msys2/mingw64/lib/wx/include/msw-unicode-3.1
In my post in the Code::Blocks forum.
Any ideas?
Thanks.

Hello,
You need to have mingw64\bin in your $PATH before msys2\usr\bin (and possibly other MSYS2/Cygwin directories).
To explain what's happening: here output format depends on the value of environment variable $MSYSTEM. Shebang sets it up for you with respect to your $PATH. Therefore, the order is important.

Hello,
You need to have mingw64\bin in your $PATH before msys2\usr\bin (and possibly other MSYS2/Cygwin directories).
To explain what's happening: here output format depends on the value of environment variable $MSYSTEM (cf. MSYS2 shell vs. MINGW64 shell). Shebang sets it up for you with respect to your $PATH. Therefore, the order is important.

Hi, many thanks,  this did solved my issue.
I think I will close this ticket.


npm i react-native-unity2 error

$ npm install react-native-unity2
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react-native
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-native-unity2
npm ERR!   react-native-unity2@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\user\AppData\Local\npm-cache\eresolve-report.txt for a full report.
npm ERR!      C:\Users\user\AppData\Local\npm-cache_logs\2022-10-20T13_17_15_089Z-debug-0.log
package.json:
{
"name": "sa4520_rn_unity",
"version": "0.0.1",
"private": true,
"scripts": {
"android": "react-native run-android",
"ios": "react-native run-ios",
"start": "react-native start",
"test": "jest",
"lint": "eslint . --ext .js,.jsx,.ts,.tsx"
},
"dependencies": {
"react": "18.1.0",
"react-native": "0.70.3"
},
"devDependencies": {
"@babel/core": "^7.12.9",
"@babel/runtime": "^7.12.5",
"@react-native-community/eslint-config": "^2.0.0",
"@tsconfig/react-native": "^2.0.2",
"@types/jest": "^26.0.23",
"@types/react": "^18.0.21",
"@types/react-native": "^0.70.4",
"@types/react-test-renderer": "^18.0.0",
"@typescript-eslint/eslint-plugin": "^5.37.0",
"@typescript-eslint/parser": "^5.37.0",
"babel-jest": "^26.6.3",
"eslint": "^7.32.0",
"jest": "^26.6.3",
"metro-react-native-babel-preset": "0.72.3",
"react-test-renderer": "18.1.0",
"typescript": "^4.8.3"
},
"jest": {
"preset": "react-native",
"moduleFileExtensions": [
"ts",
"tsx",
"js",
"jsx",
"json",
"node"
]
}
}

The error should be fixed in v1.0.5.


nf38: Add firmware and README.md.

@tjvantoll just looking for a review on my documentation. Not urgent at all.


Video/Audio sync calibrator

What feature do you want to get added on the base engine?
Hey all, had an idea, what about syncing video and audio seperately? perhaps a calibrator tool as well, like how Clone Hero does it? I think that would be quite handy, in my opinion
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

I never played Clone Hero so idk what you mean


synapse will not start if services.matrix-synapse.settings.trusted_key_servers does not specify verify_keys

Describe the bug
Not specifying verify_keys in any of the  services.matrix-synapse.settings.trusted_key_servers submodules will cause synapse to fail to start. This setting is specified as optional by usptream https://matrix-org.github.io/synapse/latest/usage/configuration/homeserver_sample_config.html.
Error message:
[2541475.987207] synapse_homeserver[299]: Error in configuration:
[2541475.987473] synapse_homeserver[299]:   Unable to parse 'trusted_key_servers': None is not of type 'object'

Steps To Reproduce
Steps to reproduce the behavior:

Set services.matrix-synapse.settings.trusted_key_servers to e.g.

trusted_key_servers = [
  { server_name = "matrix.org"; }
];

Start the synapse service

Expected behavior
Synapse should start
Additional context
Workaround: set verify_keys to an empty set.
Notify maintainers

@Ma27 @fadenb @mguentner @Ekleog @Ralith @dandellion @sumnerevans @mweinelt

We set a sane default in the module, so not overwriting it will work.
https://github.com/NixOS/nixpkgs/blob/cfdda97302f20c64d0f1055c40ebf4e9724fa137/nixos/modules/services/matrix/matrix-synapse.nix#L657-L662
We could in theory add an assertion for that, but I lack the motivation for that corner case.

This also happens if you add a trusted key server that isn't matrix.org (which is advisable, to reduce centralisation). From looking at the code in https://github.com/matrix-org/synapse/blob/develop/synapse/config/key.py I think the empty set has the same effect, so can this not be made the default for verify_keys?

So what you are saying is that we should make it default to an empty attrset instead of null?

Yes. Ideally if not set the resulting config file would omit the verify_keys key entirely, but I'm not sure how easy that would be. An empty attrset is a fine compromise, as it seems to have the same effect.

Fixed by the now long merged #232837


feat-request: svelte-persisted-states

Would it be difficult to port this library to use Svelte 5 state runes instead of stores?

Hi @saturnonearth!
This is a duplicate of #251
So I'll close, but please follow that discussion and feel free to chime in!


AUC comparision on IMC 2023

Hi everyone,
First of all, great job on the paper—thank you for all your hard work!
I'm currently working on reproducing the results from Table 5, but I ran into some confusion. The table mentions that the ground truth (GT) is generated using COLMAP, but it seems like COLMAP struggles to match its own results, even on the easier datasets like the bike dataset.
For example, when I run COLMAP on 15 images from the "images" directory, as well as on "images_full," I consistently get an AUC of 100 for every tolerance gap, which is much higher than what's reported in the table. I've tried comparing these results with the reconstruction in the "sfm" folder and the poses provided in the train_labels.csv, and I'm still seeing significantly better results for COLMAP across almost every dataset.
Additionally, I noticed that while some scenes have two image sets, which makes sense for evaluating sparse versus full datasets, some scenes only have one set. In those cases, it seems like we're just comparing COLMAP against itself. I'm finding it hard to understand how results could be worse than 100 in this context.
Could you help clarify this?

Hi, thank you for your interest in our work. We did not generate the GT for the IMC benchmark ourselves but use the provided one by the authors of the benchmark. To the best of our knowledge, the IMC GT was generated with COLMAP while using a hold out image set that is not available to anybody other than the benchmark creators. Thus, COLMAP was able to obtain more complete and accurate reconstructions than is possible with the imagery to the benchmark users. I hope this makes sense.

Thank you for the clarification. Is there a chance to look at the evaluation code? because I persistently receive much better numbers for colmap, and sometimes also for glomap

Hi, thanks for your question. When comparing the result, we use the provided train_labels.csv as (pseudo) ground truth for haiper, heritage, and, urban, and for phototorism, we use the provided sfm model as ground truth. Notice that the ground truth is generated with more images than in images. Thus, it is natural that COLMAP scores are not 100. We always use the folder images (with few images) for comparison. As for the evaluation, we calculate the AUC scores for all image pairs (n * (n-1) / 2) thresholded at 3 degrees, 5 degrees, and 10 degrees. And the error is $max(d_{geod}(R_{ij}, R_{ij}^{gt}, \arccos (t_{ij}, t_{ij}^T)$. Have you included both relative rotation error and relative translation error?

How is $t_{ij}$ computed? Are they normalized?

Sorry, it was a typo. Just modified the original reply: it should be $max(d_{geod}(R_{ij}, R_{ij}^{gt}), \arccos (t_{ij}\cdot t_{ij}^{gt}))$ instead of $max(d_{geod}(R_{ij}, R_{ij}^{gt}), \arccos (t_{ij}\cdot t_{ij}^T))$

Also, notice that $ \arccos (t_{ij}\cdot t_{ij}^{gt})$ should be in degree instead of radian as well

Sorry, it was a typo.
Now it makes perfectly sense. Thanks :)

I have few more questions:

Did you run COLMAP and GLOMAP on distorted or undistorted scenes?
Did you use the automatic settings or some tweaked ones?

Sorry, it was a typo.

Now it makes perfectly sense. Thanks :)
I have few more questions:

Did you run COLMAP and GLOMAP on distorted or undistorted scenes?

Did you use the automatic settings or some tweaked ones?

I run on distorted scenes
I use exhaustive matching to find correspondences

So you just

download the data from website
unzipped
run feature extractor (without passing intrinsics) and matcher, then
run COLMAP/GLOMAP mapper on that database

Then

for each pair of (registered) images read qvec and tvec, and compute the relative error with GR (as said above)
stored all the errors
computed the AUC with PxSfM function on the (row-wise) max(R_err, t_err) in degrees


Refactor: extract Kubernetes version

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
7209b1d3da1a1e15f8a153eb6e088d765b07788e
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test lint_istio

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
7209b1d3da1a1e15f8a153eb6e088d765b07788e
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@shamsher31: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
7209b1d3da1a1e15f8a153eb6e088d765b07788e
link
/test lint_istio

release-notes_istio
1bab727b0c6034993ea590b1cc6c40e768444682
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
e725801ecdb041c5aff425ddd9c52a617dd4f091
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible There was no issue with the existing code. While updating the test cases to cover k8 1.19 version. I found out that the version can be easily parsed and extracted using Segments() which I learned during https://github.com/istio/istio/pull/28608 so thought to refactor it.


Parsing failures

Keep getting parse fails.  Could it be conflicting with Torn Tools.  Using inside Chrome on Chromebook. Other Torn tampermonkey scripts working fine.

@daveginberry I'm using it with TornTools on Chrome too.
Can you clarify on:

which theme u use Dark/Light?
is "Honor Names" setting in "General" settings tab ON/OFF?
are you using Army job to generate spies or other company? if company, which one?

if you can copy whole spy result (job special) html block & attach it, that would be very helpful too!
You can get in console (F12) by running document.getElementsByName("jobspecial")[0].innerHTML; command after u get the spy result on screen.

Not using any theme - so I guess Light
Honor Names ON
Company You currently work as an Attorney at PeyDAmaN's Law $=INNOCENT which is owned by -HellCat-.
Some errors in Console
TypeError: jobSpecialBlock is undefined
GetSpyProfile moz-extension://35c13c2d-140e-40ca-a08f-95752545d349/userscripts/Torn SpyParse.user.js?id=0189d03b-b1e2-47a9-8ea6-eea25b2ca980:92
GetSpyResult moz-extension://35c13c2d-140e-40ca-a08f-95752545d349/userscripts/Torn SpyParse.user.js?id=0189d03b-b1e2-47a9-8ea6-eea25b2ca980:78
Torn SpyParse.user.js:84:17
Uncaught TypeError: document.getElementsByName(...)[0] is undefined
 debugger eval code:1
debugger eval code:1:10
document.getElementsByName("jobspecial")[0].innerHTML;
Uncaught TypeError: document.getElementsByName(...)[0] is undefined
 debugger eval code:1
I am not familiar with console stuff.  All the above was in firefox- running in Linux

@daveginberry updated script to work with Law Firms too.


[SPARK-36712][BUILD][FOLLOWUP] Improve the regex to avoid breaking pom.xml

What changes were proposed in this pull request?
This PR aims to fix the regex to avoid breaking pom.xml.
Why are the changes needed?
BEFORE
$ dev/change-scala-version.sh 2.12
$ git diff | head -n10
diff --git a/core/pom.xml b/core/pom.xml
index dbde22f2bf..6ed368353b 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -35,7 +35,7 @@
   </properties>

   <dependencies>
-    <!-- #if scala-2.13 --><!--
+    <!-- #if scala-2.13 --><!--<!--

AFTER
Since the default Scala version is 2.12, the following no-op is the correct behavior which is consistent with the previous behavior.
$ dev/change-scala-version.sh 2.12
$ git diff

Does this PR introduce any user-facing change?
No. This is a dev only change.
How was this patch tested?
Manually.

Test build #143277 has started for PR 33996 at commit 676991b.

cc @lrytz

Oh was it not working to go back to 2.12? ah OK in any event, good catch.
This is a hacky hack for sure, which eventually goes away with 2.12 support

Yes, this PR restores the previous behavior. It was no-op when we use the same scala version.

cc @sunchao and @viirya

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47783/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47783/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47783/

Thank you, @srowen , @viirya , @sunchao .
Merged to master/3.2.

Test build #143277 has finished for PR 33996 at commit 676991b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143277/


Update Pages “pestinspectionsPage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
813a825d9f2f81b960ca6843d246f53b3b76466e

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/6554031fc20a06000813aa6e


Application icon not updated on flathub.org

After updating Gajim to 1.2.0 I noticed that [0] still shows the old application icon (with shadow) of Gajim 1.1.3 and not the new one without shadow [1].
Reason might be that Gajim 1.2.0 only provides a scalable svg application icon and no png icons, following GNOME [2].
This issue might be related to #183.
[0] https://flathub.org/apps/details/org.gajim.Gajim
[1] https://dev.gajim.org/gajim/gajim/-/blob/bd6c95bdcb1d82cbcd18b0efe9b5fe81e9ab6c2b/gajim/data/icons/hicolor/scalable/apps/org.gajim.Gajim.svg
[2] https://gitlab.gnome.org/GNOME/Initiatives/-/issues/2

We're caching assets excessively and purging icons is the most tricky part. Purged it manually, shows up fine now for me.

Thanks.


Add Docker image to be able to test cryptol

Problem:
Currently, the easiest way to run cryptol by itself is to run it with a docker image provided by galois. It requires code changes to your local branch. If we agree to own this image, we could both prevent the rabbit hunt to find how to get it working and stop devs constantly add/deleting the same 3 pieces.
Proposed Solution:
https://github.com/GaloisInc/s2n/tree/saw-docker
This branch is out of date but you need 3 things from it to copy pasta into your repo

create s2n/.devcontainer in your branch and copy the contents of the corresponding folder into it. It’s only one file
https://github.com/GaloisInc/s2n/blob/saw-docker/.devcontainer/devcontainer.json
Note: Mac users will have to change slashes to other direction //

copy pasta this file and place it in the same location
https://github.com/GaloisInc/s2n/blob/saw-docker/tests/saw/Dockerfile

replace current .travis/install_z3_yices.sh with this
https://github.com/GaloisInc/s2n/blob/saw-docker/.travis/install_z3_yices.sh

We can provide an alternate for those who wish to develop on an EC2 instance.
export TESTS=tls SAW=true GCC_VERSION=NONE
codebuild/bin/s2n_codebuild.sh
export PATH=$PWD/test-deps/saw/bin/:$PWD/test-deps/z3/bin:$PATH

A brief example of how to load and run a Cryptol test:
ubuntu:s2n/ $ cryptol
┏━╸┏━┓╻ ╻┏━┓╺┳╸┏━┓╻
┃  ┣┳┛┗┳┛┣━┛ ┃ ┃ ┃┃
┗━╸╹┗╸ ╹ ╹   ╹ ┗━┛┗━╸
version 2.8.1 (67fc367)
https://cryptol.net  :? for help

Loading module Cryptol
Cryptol> :l tests/saw/spec/handshake/rfc_handshake_tls13.cry
Loading module Cryptol
Loading module s2n_handshake_io
Loading module rfc_handshake_tls13
rfc_handshake_tls13> :prove tls13rfcSimulatesS2N `{16}


[KARAF-4607] Fix pooling scenarion with ldaps:// and default Sun prov…

…ider

This pull request has been applied, can you please close the PR ?
Thanks.

Thanks for merging!


Double check whether model element contains an inbound connector

We shouldn't rely solely on extension properties to determine if the element contains a connector. If user adds a template to the element and then removes it, the properties remain intact. In this case we will consider the diagram element a connector, while the user will actually not see this in the Modeler.
We should additionally check zeebe:modelerTemplate which clearly indicates whether element template is applied.

Decided not to do it: the technical properties should be our source of truth.
The solution, if needed, should be in the Modeler domain.
See also: internal discussion


Replace external csud functions with kernel functions

classic.csud

[x] void MemoryCopy (void* d, void* s, u32 len)
[ ] Result PowerOnUsb()
[ ] u64 __aeabi_uidivmod(u32 value, u32 divisor)
[ ] u32 __aeabi_uidiv(u32 value, u32 divisor)

LayerZero Airdrop Updated 🪂
The LayerZero Airdrop is confirmed. This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

Engage for Extra Rewards:

Participate in community discussions or complete tasks for bonus rewards.

Bonus Tips:

Community Assistance:

Need help? Drop a message on Telegram or other social media platforms.

Stay Informed:

Keep an eye out for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a while. Stay calm and keep an eye out for updates.

Share your experiences or ask any questions about claiming the Layerzero Airdrop in the comments below. Let's make this process a breeze for everyone!


[path_provider] 1.6.28 version not found

Steps To Reproduce

Add path_provider: ^1.6.28 to pubspec.yaml
Run flutter pub get

Expected Result:
flutter pub get run successfully because 1.6.28 version is exist on the changelog
Actual Result:
Error appears on console:Because my_app depends on path_provider ^1.6.28 which doesn't match any versions, version solving failed.

Changelog of that plugin must be updated. I would still recommend to use the latest version of the plugin to get better support bug fixes and new features.

We can't retroactively change old CHANGELOGs, but that version could be retroactively published.
It's there a specific reason you are trying to use exactly that version?

We can't retroactively change old CHANGELOGs, but that version could be retroactively published.
It's there a specific reason you are trying to use exactly that version?

My project is not using null safety yet, so I tried to use the latest version prior to the 2.0.0 version. Actually, I'm fine to use 1.6.27 version since there is no specific feature that I need on the 1.6.28. But regardless I made the issue to avoid the same confusion in the future.

Published https://pub.dev/packages/path_provider/versions/1.6.28


How to set the library to request Overtime report or ranked report?

Hi there,
I am running into issues when pulling Average Time Spent on Site metric. The error message:

InvalidReportError: metric_unsupported_in_trended: Metric "averagetimespentonsite" not supported in trended reports

However, I don't see any option when running a report to set the report type to ranked or overtime. Is there any way to do that?

@turing4ever the report type is actually determined by the the parameters you hand it. The table about half way down on this page is a helpful reference. https://marketing.adobe.com/developer/documentation/analytics-reporting-1-4/get-started


Goimports and gofmt disagree about formatting

Please answer these questions before submitting your issue. Thanks!
What did you do?
Ran goimports and then gofmt on the same file and they produced different results.
When I run goimports it gets this formatting wrong:
$ goimports -d ./
diff -u gocomplete/complete.go.orig gocomplete/complete.go
--- gocomplete/complete.go.orig	2018-08-28 09:27:22.000287480 -0800
+++ gocomplete/complete.go	2018-08-28 09:27:22.000287480 -0800
@@ -449,7 +449,7 @@
 					"-unusedfuncs":         complete.PredictAnything,
 					"-unusedresult":        complete.PredictNothing,
 					"-unusedstringmethods": complete.PredictAnything,
-					"-v":                   complete.PredictNothing,
+					"-v": complete.PredictNothing,
 				},
 				Args: anyGo,
 			},

gofmt changes it back:
$ gofmt -d ./
diff -u gocomplete/complete.go.orig gocomplete/complete.go
--- gocomplete/complete.go.orig	2018-08-28 09:28:32.280407134 -0800
+++ gocomplete/complete.go	2018-08-28 09:28:32.280407134 -0800
@@ -449,7 +449,7 @@
 					"-unusedfuncs":         complete.PredictAnything,
 					"-unusedresult":        complete.PredictNothing,
 					"-unusedstringmethods": complete.PredictAnything,
-					"-v": complete.PredictNothing,
+					"-v":                   complete.PredictNothing,
 				},
 				Args: anyGo,
 			},

What did you expect to see?
I expect goimports and gofmt to agree on formatting.
What did you see instead?
goimports produces different formatting from gofmt.
System details
go version go1.11 linux/amd64
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/aslevy/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/aslevy/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/aslevy/repos/go-modules/posener/complete/go.mod"
GOROOT/bin/go version: go version go1.11 linux/amd64
GOROOT/bin/go tool compile -V: compile version go1.11
uname -sr: Linux 4.18.5-zen1-1-zen
LSB Version:	1.4
Distributor ID:	Arch
Description:	Arch Linux
Release:	rolling
Codename:	n/a
/usr/lib/libc.so.6: GNU C Library (GNU libc) stable release version 2.28.

Just a quick check, did you recompile goimports using Go 1.11?

Great catch. No I hadn't. I did so and that resolved the issue.
Thank you


add test_cluster_events_filtering

Dont review this, couldnt get it working, just leaving as a draft for future reference.

Closing as I figured out how to solve it and merged in with https://github.com/shotover/shotover-proxy/pull/807


Newbee question

Hi,
my 'server' running on a very small embedded controller, so it  is far from a server farm.
Is  it possible to use react as frontend tool without any server support, exept loading files and getting Json objects with dynamic data to render?
I want to use d3 (www.d3.org) as a charting tool, is  it possible to use this Javascript charting lib with reactJS?
And how did all that work within este????
Thanks for helping
With best regards
Gerhard

Sure. And you don't need Este. Read this https://github.com/petehunt/react-howto

Hi Daniel,
thanks pointing me there.
I get through the first tutorial (just carefully reading) and it feels quite nicer than all I saw uptil now.
I am programmer and software architect since 1975 and see a lot on my way. As I started web developing two weeks ago I shiver when I see all that crazy stuff around, tons of frameworks no idea which to choose and mostly doing some unprdictable.
React seems to  be a better way to do things, so I will start out and give it a try this weekend.
As I pointed out: I need a web site hosted at a embedded controller, so somebody (allowed to do) can configure and control a complex measurement system running unattended somewhere. My processor resources were very restricted also memory and I doesnt have a full blown web server with all that features you normally will have.
The data link is also GPRS not LTE and in some countries you doesn’t have flat rate, or mobile internet is still very expensive. So data transfer should be as less as possible,
With best regards
Gerhard
Von: Daniel Steigerwald<EMAIL_ADDRESS>Gesendet: Samstag, 09. Jänner 2016 01:47
An: este/este
Cc: GerhardKreuzer
Betreff: Re: [este] Newbee question (#665)
Sure. And you don't need Este. Read this https://github.com/petehunt/react-howto
—
Reply to this email directly or view it on GitHub https://github.com/este/este/issues/665#issuecomment-170170781 .  https://github.com/notifications/beacon/AEA-5zi-g2oW8JNLsKaNWEB138SJIaoNks5pYFAUgaJpZM4HBSiw.gif


Update wasm.md to include some HTTP instructions

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Read the contribution guide

In addition, please fill out the following to help reviewers understand this pull request:
Description
Add docs for recent WASM updates.

hrmph, switching bases messed up this PR. I will re-compute it...

Ok, fixed.

Thanks for the suggestions, merged.


NIFI-12075 Deprecated ComponentLog methods with Object[] and Throwable [support/nifi-1.x]

Summary
NIFI-12075
NIFI-12075 for support/nifi-1.x branch.
The 1st commit was cherry-picked from main. The 2nd commit contains the additional changes on 1.x.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Merged in https://github.com/apache/nifi/commit/e07bb19233de81513a5218ed782c782b26591bb3


Challenger Coin - Find a way to limit it to once per shop at most, because the shop potions should give combat options to the player

As mentioned above...
...the shop potions all help in combat. If Challenger Coin does not, then if it shows at the Merchant store, should appear once per shop at most.

The main problem of this patch is that testing it spends a lot of time without an automated code... which I'm not going to do now.


Saved Queries Panel

Add a new tab panel for displaying saved queries. Users should be able to run the query right from the panel.
For now, queries will be saved in a map in src/constants.js. The map should be description to gremlin query code. The saved query should display the query descriptions in a list, with an execute button. Displaying the query code is unnecessary, but it might be nice to have a preview panel.

Depending on space, we may have to move to a vertical icon tab bar along the right edge instead of a horizontal one along the top.


Messages: swipe is broken when user did try to swipe on invited room

Done, wait for validation

Ok in 0.3.4 develop #1378


summary-always updates benchmarks twice, fails with gh-pages

If summary-always is true, we call first writeBenchmark, then writeSummary.
https://github.com/benchmark-action/github-action-benchmark/blob/060e11092cdc7b319ecc1abdc3d96debaeeec72b/src/index.ts#L13-L17
But both of these call writeBenchmarkToGitHubPages by default, so we will create the commit twice. In fact we don't even get that far because the second invocation's git fetch will fail to update the gh-pages branch because of the new commit available only locally when auto-push is false.
Example run: https://github.com/Kha/nale/actions/runs/4375198974/jobs/7656891394

Another issue is with the rendering, see: https://github.com/timbray/quamina/actions/runs/4393672754 (example) where the numbers are markdown formatted but not properly rendered.

@embano1 You should probably open a separate issue for that

Yeah, was too lazy as yours was top of list when I just checked...sorry for polluting your issue.

has anyone started to work on this?

I believe this was fixed by #214
Will verify that tomorrow

Verified!
It is fixed on v1.19.0


Updating Pcells testing to use PV repo for GF180MCU

Updating Pcells testing to use PV repo for GF180MCU

@atorkmabrains Makefile used for testing has been updated.
generate_pcell.py script used for testcases generation doesn't work. I have opened an issue for that. #125

@FaragElsayed2 What's the status?

@atorkmabrains As I mentioned, the DRC testing setup is ready but the Pcells generator itself doesn't work.

Thanks @FaragElsayed2 Appreciate the change.

@atorkmabrains Could you please merge this as I need to use it in Pcells testing ?


1ES migration

Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3610487&view=results -- Publishing is disabled in YAML but needs approval to continue. Work synchronously with @danieljurek

hhm we count on the publishing to be done automatically when version are bumped, this is a significant change in the flow.


Experience/4264: add diff dialog to compare JSON before updates

This PR adds the ability to compare the JSON of any org, receiver or sender setting update to the original JSON before finally saving it to the system.
Test Steps:

Login to ReportStream as an admin
Navigate to Admin->Organization Settings
Choose an org from the list and click Edit
Make edits to the top-level org fields and click Save
A dialog pops up presenting a diff window, with the OG JSON on the left and the JSON with your edits on the right
Optionally modify the JSON on the right and/or click Save
If the JSON is still valid, the success popup displays and the edits are committed. Otherwise, an error dialog returns
Repeat steps 4-7 for senders and receivers, both edit and clone operations

NOTE: this also fixes bugs #3934 and #4326 by invalidating the cache on the org list call when cancelling or saving a sender or receiver edit
Testing

[x] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[x] (For Changes to /frontend-react/...) Ran npm run lint:write?
[x] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Fixes

#issue - List GitHub issues this PR fixes

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Median time to review
Total comments

MauriceReeves-usds
52
1h 2m
27

carlosfelix2
24
11h 50m
64

jimduff-usds
19
17h 28m
19

cwinters-usds
18
3h 47m
7

TomNUSDS
14
1h 26m
58

ahay-agile6
13
4h 50m
8

whytheplatypus
12
2h 27m
34

oslynn
12
1h 4m
5

jbiskie
9
52m
0

kevinhaube
8
1h 25m
0

jorg3lopez
7
1h 19m
8

brick-green-agile6
5
15h 41m
2

TurkeyFried
4
21h 48m
2

JosiahSiegel
4
16m
2

meckila
4
4h 39m
12

rhood23699
4
45m
0

sethdarragile6
4
1h 23m
0

bgantick
3
3h 35m
3

RickHawesUSDS
2
2h 45m
0

jeremy-page
2
8m
0

acoushawk
1
16h 10m
0

clediggins-usds
1
17h 4m
1

snesm
1
16m
0

Yeah. See https://github.com/microsoft/monaco-editor/issues/271
Pretty sure this is going to fail when it's deployed to staging where the CSP is enforced.

Found a bug! Currently, only the first set of changes made are pushed to the diff modal.

Log in and navigate to edit the settings of an org
Change one value and click save to launch the diff modal
Close the diff modal with the X, Go Back button, or by clicking away from it
Edit another value, click save, and notice the diff does not reflect this change

It isn't related to one field in particular, I've tried it with each one and the bug behaves exactly the same

Ugh, thanks, Kevin. Putting this PR into draft mode while I work this out.

@sethdarragile6 Let me know if you wanna get together to troubleshoot!


Bundling and Minification in ASP.NET 5

ASP.NET 5 -> Client-Side Development -> Bundling and Minification in ASP.NET 5

I am interested in how ASP.NET 5 will handle bundling and minification.
Do you plan on sticking with the MVC solution of storing the bundle in memory during release builds or adopting the new Web Essentials solution of .bundle files that minify upon saving?
Forgive me if this is not the correct  place to ask such a question :panda_face:

Why is this closed?
Also, a question that is so far unanswered is, how do you get the current Solution Configuration (Release|Debug) in their gulp/grunt config? It's vital to switch on minification etc

Is it an idea to do something like this? https://github.com/ryanheath/MVC-Bundling-And-Minification-For-VisualStudio-With-Gulp
Or are you guys planning to take a whole other direction with bundling & minification?
I'm will to create something alike for asp.net if still needed.

I solved this by always building two versions of the javascript with gulp, and switching which one is used with the environment tags in the new ASP.NET vNext. It's not ideal, as the production js build can take up to 30 seconds, but it works. I use hot building for development anyways

Clarify that the environment tag uses tag helpers and link to http://docs.asp.net/projects/mvc/en/latest/views/tag-helpers/intro.html

See also https://github.com/aspnet/Docs/issues/308.

It's not clear to me what the proposed replacement is now the the .bundle files are going away in ASP.NET 5. Is TagHelpers the solution? I would be fine using something like wiredep with gulp, but it'll be a lot more useful if we can reference the solution configuration (as mentioned by @awestroke).
I'm surprised we're this far in the development of ASP.NET 5 with no clear recommendation on replacing the current MS bundling system.

Updating based on the following outline:
Bundling and Minification

Overview
Bundling
Minification
Impact of Bundling and Minification
Controlling Bundling and Minification
Using the “*” Wildcard Character to Select Files
Bundle Considerations
Gulp
See Also


Migrate LanguageModelFeaturizer to GraphComponent interface

Overview of the Solution:
Adapt the LanguageModelFeaturizer  to the new GraphComponent  interface.
See the implementation proposal for further details.

adapt create and load functions to new signatures
implement required_packages and supported_languages
adapt persistence
rename train to process_training_data (this featurizer only adds features but doesn't require training)
support multiple messages in process (make sure to handle empty messages)

Definition of Done:

[ ] LanguageModelFeaturizer adapted to the new GraphComponent interface
[ ] Unit tests are adapted to adapted component
[ ] Other tests (integration tests, etc.) don't break (because they'd (for now) still use the old component implementation)

The LanugageModelFeaturizer doesn't currently have a load method. However, reading the description, the class is designed to always load a pretrained model. In this case, should I replace the existing create method with load, and then not create a create method?

It's also okay to just adapt the create one as the default implementations of the GraphComponents load just points to create (see here)

GraphComponent doesn't have a name method like the old Component. Do we want to add one?

No, that's not needed

The original ML featurizer doesn't have a persist method. Should I add one, given that we only load pretrained models?

The featurizer doesn't need persisting, so you can omit the persist method. Only components which train themselves need a persist method

closed by https://github.com/RasaHQ/rasa/pull/9586


Modify default log format for container tests

Format has been taken from the Felix log formatter (@fasaxc recommended it).
Example output with the new formatter:
2017-01-19 12:49:18,801 [INFO][26158/140043457387184] tests.st.utils.utils 73:   # 64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.145 ms
2017-01-19 12:49:18,802 [INFO][26158/140043457387184] tests.st.utils.utils 73:   #
2017-01-19 12:49:18,803 [INFO][26158/140043457387184] tests.st.utils.utils 73:   # --- <IP_ADDRESS> ping statistics ---
2017-01-19 12:49:18,803 [INFO][26158/140043457387184] tests.st.utils.utils 73:   # 1 packets transmitted, 1 packets received, 0% packet loss
2017-01-19 12:49:18,804 [INFO][26158/140043457387184] tests.st.utils.utils 73:   # round-trip min/avg/max = 0.145/0.14

Timestamp should allow us to do things like profiling tests (seeing where all the time is going).

I consider the timestamp, severity and file/line no must-haves.  Rest is useful from time to time.

Some of the test output is designed to be copy pastable to repro tests. E.g. some lines start with a hash and some don't. Will this break that?

@tomdee Ah - now that's exactly the sort of thing I was hoping to find out. Where does that hash get put on?

You should be able to find it by looking at the logging code

Ah - its in utils.py.
However, looking at the other various instances of logging around the place, I'm fairly sure that the requirement that the output be cut-and-pastable wasn't understood by authors and reviewers for quite a few PRs now - its clearly been broken for a long time.

@lwr20 Can you just add a # to the start of the log format, then make sure that log_and_run puts a \n\n before the commands it prints?

Ah - good call.   I'll also need to fix up all the places we've broken it in the past, but yeah - maybe that's doable.

Hmm.  Not sure that's working quite right.   Commands are getting timestamped.  Let me have a further think...

It's not a requirement. I was just letting you know that it was once a thing.

Moved to https://github.com/projectcalico/calicoctl/pull/1472


Make a first Light Client Red Eyes Testnet release

Description
After having implemented #26 the SDK is feature complete for milestone 0 for Red Eyes. This issue keeps track of making a first test net release and to decide how to maintain it afterward.
Acceptance criteria

Manual release
Include #14
Release Notes
No state backup
v0.1.0

Tasks

[ ]

Story Points

Some collected links for the release notes

Easy Release Note Template
How to write great release notes
5 excellent product release note examples and how to write your own
Keep a changelog

Maybe using in future:

Chronicler - Service to ensure that your release notes are always up to date
Release drafter - GitHub Action

Done: https://github.com/raiden-network/light-client/releases/tag/v0.1


JOSS paper: VISA description

I am bothered by the strong link made between VISA and SCPI. VISA provides an abstraction over the physical bus used for communication but does not mandate the format of the message circulating on that bus (except for IEEE 488.2 messages). In particular VISA also provide access to register based instruments for which SCPI makes no sense at all.
I am not sure how this should be phrased to be palatable for the reader but I believe that such a strong implicit coupling should be avoided as it tends to create some confusions.
https://github.com/openjournals/joss-reviews/issues/4752

Authors’ Response to Reviewer Comment
Thank you for the suggestion to clarify VISA.  I’ve added a VISA Overview section to describe VISA in more detail.
Added Text
VISA Overview
This section provides a brief overview of VISA to better understand the purpose of this project and some of the key terminology used throughout this publication.[@GH_VISA_Background, @GH_VISA_Terminology_Resource]  VISA was originally standardized through the VXIplug&play alliance and is currently maintained by the IVI Foundation.[@wiki_VISA]  The purpose of the VISA API is to provide a set of standard, consistent architecture to communicate with Test and Measurement (T&M) instruments that is reasonably independent of the interface bus and is consistent across devices of similar types.  For example, a programmer would be able to write should be able use the same code to control multimeters made by different manufacturers (assuming the  instruments' capabilities are the same).  Additionally, even if the capabilities are different, the API to control similar aspects of different instruments is often the same.  One nice aspect of the API is that many the VISA Commands use to control instruments consist of simple text strings passed using a standard communication interface (USB, TCP/IP, etc.).
It is important to understand some of VISA's Resource terminology.[@GH_VISA_Terminology_Resource]  A Resource is an instrument connected to your system and the Resource provides a complete description of the set of capabilities for the instrument.  There are several types of VISA resources: INSTR, MEMACC, INTFC, BACKPLANE, SERVANT, SOCKET, and RAW.  According to the National Instruments VISA Resources page, most VISA applications and instrument drivers use only the INSTR resource.[@GH_VISA_Terminology_Resource]  The Resource's Interface is often describe a concatenation of the communication protocol with the Resource Type. For example, a resource communicating over USB using the Instrument Type would be called a USB::INSTR.  Communicating over TCP/IP using a Socket type would be TCPIP::SOCKET.
While the VISA API provides a simple programming interface to control various T&M instruments, a set of frameworks and drivers are necessary to handle communication between the devices and the computer.  This is often handled using the pre-compiled National Instruments VISA framework, NI-VISA.  This C-based framework can communicate with a broad range of instrument types using a broad range of communication protocols.  Overall, NI-VISA is a good starting place when writing T&M instrument control software. However, writing GUI-based software in C is harder than using more modern programming languages, such as python, C++, Swift, etc.  As a result, there is a large community of programmers and scientists writing wrappers around the NI-VISA framework[@pyVISA, @GH_SwiftVISA, @GH_GOwrapper, @GH_JuliaWrapper] or writing their own VISA implementation[@GH_pyVISApy, @GH_SwiftVISASwift] in their preferred language.

The edit looks good to me


Update tsaoptions.json to update area path for TSA bug filing

Update tsaoptions.json to update area path for TSA bug filing.
CP of PR #4464 to 1.5
--
A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

/azp run


Add Intune plans detail to teams-shared-device-license.md

Closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9803

@DaniEASmith
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Build failed

Hello everything works pretty good while normal compilation. But when I tried to do yarn build I got and issue :

Anybody had this problem before? I try to resolved by normal methods, but still main.ts don't see this component while build.

Hello, @AABobi.
Could you provide a reproduction demo, please?


Is it correct to give 5' & 3'UTR consequences weight of 0?

They current are weighted 0. It was suggested on integration day that they should weight non-zero.

No action here for Genetics portal for moment.  @MayaGhoussaini to discuss with relevant stakeholders.

@MayaGhoussaini We should decide on new scores before release 2.0.

New have been updated


Highcharts plot unit tests new

Created New Unit tests for Highcharts
Replaced most datasets with small custom datasets
Optimized Unit tests to reduce redundancy
All tests can be run in parallel using "pytest . -n auto --dist=loadscope"
Used pylint to ensure best coding practices
Pylint score for individual files is 10/10

What do we have a new folder, tests_new, for the unit tests?

What do we have a new folder, tests_new, for the unit tests?
@afard
Yes, we created a new folder to create more efficient unit tests. Once complete, this folder will replace the older test folder.

Nice work Umar! I have reviewed the entire PR once. Please let me know when you are done with refactoring.


Timeout (or similar approach) serving generated ISOs

Current implementation of the ISO build keeps serving the generated image without any timeout. It does not stop serving unless the pod is deleted manually. This does not scale, we need a mechanism to allow building many ISOs without quickly running about of resources or stressing the upstream cluster, once an ISO is downloaded there should be a cleanup logic to free resources. I'd say it is better to recreate the ISO on demand rather than keeping in use upstream cluster resources for nothing.
IMHO we need:

Stop serving generated image with some criteria (e.g. passed certain X time after the last download)
Being capable of rebuilding the same ISO based on the same exact URL

I think keeping the ISOs available long term is useful. Specifically for using iPXE to point at Rancher directly to install from.

I think keeping the ISOs available long term is useful. Specifically for using iPXE to point at Rancher directly to install from.

Not sure. I don't think we should consider the controller as a goto server for iPXE deployments, also iPXE tends to be coupled with the specific underlying infraestructure, I'd expect anyone aiming for iPXE deployements to provide its own infra for that matter.
I'd be more in favor of making ISOs rebuildable with little effort (most likely the base image is already downloaded and available if it was already built within the same node).

I'd expect anyone aiming for iPXE deployements to provide its own infra for that matter.

That's a fair expectation. And you probably also don't want to incur the load of all the downstream nodes pulling their image as well

I think we should just allow to disable  the automatic clean-up if needed.

Probably this should be mapped to a couple of configurable parameters: timeout (int value and 0 <= means infinite) and autocleanup (boolean to run a cleanup after download or not).


Alt is missing in 7.8.4

Alt
We should either add a compatibility version of it to P, or hide it from export in 7.10

This is done


Make anonymous memory maps private by default on unix

Currently map_anon will call mmap with MAP_SHARED | MAP_ANON.
This prevents debuggers from easily working with executable code inside these mmaps as MAP_SHARED implies any changes made to the mmap will be flushed to a file or to other processes. However, it is not actually necessary to call it with MAP_SHARED. There is no documentation implying the resulting memory map is MAP_SHARED, and the only observable behaviour (the mapping being shared with child processes) would cause other UB anyway (allows you to change the content of a &[u8] from a child process) so it shouldn't be relied on to begin with. Therefore making anonymous memorymaps MAP_PRIVATE seems fine.

Seems reasonable, but looks like I have to fix CI first. There were a lot of changes to travis in the past year.

Can you force-push to restart the CI?

Sure!


Error: Destination path '/content/data/subject/first_image.jpg' already exists

If a user for some reason needs to rerun Dreambooth > Setting up, uploaded images will collide.
I guess there needs to be...
os.path.exists('/content/data/subject'):
    !rm -r /content/data/subject

in there somewhere...

If the instance images are the same, you don't need to reupload them, just skip the uploading process


Add --print-report-hash to parse

fixes #1009

User guide was not updated with this new option.
Also, I feel like parse is getting a lot of options added to it... What if we just print the hash all the time? Or print it automatically when --print-steps is given, above the bug path list?

Just a minor philosophical question from the user's point of view.
I was wondering, is the hash something the user is interested in?
The answer might be yes, since she might want to use this info to suppress a bug.
But if the answer is no for some other reason, we might write out the hash in verbose mode (for debug purposes) and should not introduce a new flag.

@Xazax-hun's idea is even better.
Also: you can no longer really suppress bugs with the bughash format, it is only kept as a backward-compatible feature. I think the consensus was that it's the best when suppressions are in the source code (for which the user only needs the bug's location information, which is printed by parse all the time), or via the web browser (with which you also can't suppress via bughash).

Bug hash may also be used to create a link to a bug to a CodeChecker server. I think too that is it is fine if we print it only when --print-steps option is present and we are not introducing a new option for this.

@dkrupp the parse command works only on the report directory yes? Do we want to create a link to a local report directory? There might be no database in this case.
If it is used only for debug purposes I agree with @Xazax-hun to log this information only in debug mode.

As we discussed with @dkrupp we will print the hash together with the steps.

We definitely have such tests in the "analyze and parse" folder, that has steps output. Or we had a month ago, at least.

@whisperity what would be the alternatives for ReportHash?
I am thinking about Report hash, report hash, report_hash, Report Hash, ... , RePoRtHaSh

Report hash is fine with me.


Is it normal for the contrastive loss to reach the flat area soon?

Hello, this is an interesting work. But when I directly apply AdCo to downstream tasks, such as the generator of negative examples, the contrastive loss item will not decline soon(just half of total epochs).
The curve will look like this:

Do you think this is normal?  Or can you give me some advice?
I'm not particularly clear about the code of your memory update, such as torch.mean  in  g = torch.einsum('cn,nk->ck',[q.T,p_qd])/logits.shape[0] - torch.mul(torch.mean(torch.mul(p_qd,l_neg),dim=0),d_norm).,  so I deduced the formula myself according to issue and rewritten the code of gradient manual update just by vector dot product as the formula in issue.

my code as follow:
`
logits = logits / memor_temper
p_qd = nn.functional.softmax(logits, dim=1)[:, 1:].unsqueeze(-1) # num_queries x num_negative
# anchor_feat: num_queries x rep_dim
# d/d_norm shape: num_queries x num_negative x rep_dim
grad = p_qd * anchor_feat.unsqueeze(1) - p_qd * anchor_feat.unsqueeze(1) * d * d_norm
norm_d = torch.norm(d, dim=-1).unsqueeze(-1)
if (norm_d != 0).all():
try:
grad = -torch.div(grad, norm_d) / (num_queries * memor_temper)
### update
`
I don't know if it's the wrong implementation, because I don't know much about it. I hope you can help me point it out.
I am looking forward to your reply soon.

Here we do this complicated implementation to save time for running experiments. For your case, I think you can simply change the updating rule by applying a new optimizer with maximizing the contrastive loss. That would be more reliable.
Also, your contrastive loss range is a little strange compared to my setting, which may because of the number of your negative examples.

thanks !!! i will have a try with using a new optimizer.


ci: adds instructions on how to build singularity image

Closes #258

Adds no-cache-dir to post_install script to reduce size of singularity images
Adds Dockerfile to build an environment with Singularity installed
Adds def file to pip install aind-data-transfer[full] and run post_install script
Add README explaining how to build a singularity image that can be run on the HPC as an alternative to conda envs.

@carshadi Any objections to adding --no-cache-dir to the post-install.sh script? This will help reduce the singularity image sizes.

@jtyoung84 I think that should be ok


[SPARK-42585][CONNECT][FOLLOWUP] Store cached local relations as proto

What changes were proposed in this pull request?
In the PR, I propose to store the cached local relations in the proto format, the same as LocalRelation. Also I reverted transformLocalRelation() to the state before the commit https://github.com/apache/spark/commit/0d7618a2ca847cf9577659e50409dd5a383d66d3.
Why are the changes needed?
To explicitly specify the format of cached local relations in the proto API.
Does this PR introduce any user-facing change?
Yes but the feature of cached local relations haven't been released yet.
How was this patch tested?
By running the existing tests:
$ build/sbt "test:testOnly *.ArtifactManagerSuite"
$ build/sbt "test:testOnly *.ClientE2ETestSuite"
$ build/sbt "test:testOnly *.ArtifactStatusesHandlerSuite"

The GA Run / Build modules: pyspark-sql, pyspark-mllib, pyspark-resource frozen in the run https://github.com/MaxGekk/spark/runs/13365446543 but it passed in https://github.com/MaxGekk/spark/runs/13370236086

Merging to master. Thank you, @HyukjinKwon @vicennial @amaliujia for review.


Add Bazel targets for compression example

Should have been added in https://github.com/grpc/grpc/pull/17464.

iOS Binary Size Diff: https://github.com/grpc/grpc/issues/17561


[FORMATS-43] Rename contig to reference.

Fixes #43

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/bdg-formats-prb/173/
Test PASSed.


Long running tasks seem to time out after 30 seconds

The connection then returns with "result expired"

Runtime error (Exception): result expired
Traceback:
line 47, in wait, "C:\Users\Popo\AppData\Roaming\McNeel\Rhinoceros\5.0\Plug-ins\IronPython (814d908a-e25c-493d-97e9-ee3861957f49)\settings\lib\rpyc\core\async_.py"
line 95, in value, "C:\Users\Popo\AppData\Roaming\McNeel\Rhinoceros\5.0\Plug-ins\IronPython (814d908a-e25c-493d-97e9-ee3861957f49)\settings\lib\rpyc\core\async_.py"
line 471, in sync_request, "C:\Users\Popo\AppData\Roaming\McNeel\Rhinoceros\5.0\Plug-ins\IronPython (814d908a-e25c-493d-97e9-ee3861957f49)\settings\lib\rpyc\core\protocol.py"
line 75, in syncreq, "C:\Users\Popo\AppData\Roaming\McNeel\Rhinoceros\5.0\Plug-ins\IronPython (814d908a-e25c-493d-97e9-ee3861957f49)\settings\lib\rpyc\core\netref.py"
line 155, in getattribute, "C:\Users\Popo\AppData\Roaming\McNeel\Rhinoceros\5.0\Plug-ins\IronPython (814d908a-e25c-493d-97e9-ee3861957f49)\settings\lib\rpyc\core\netref.py"
line 4, in script

Might also be a reference counting issue, rather than a timeout.


Move validation of jurisdiction into Jurisdiction model

should we bump the case type as the field names have been changed ?

I will double check against the frontend but they should be the same


Fix table in user guide (currently not rendered in html due to this)

Signed-off-by: barney2k7<EMAIL_ADDRESS>Type of change

Documentation update

Description
The table attributes that can be registered for an identity in the user guide is currently not rendered in html due to invalid syntax.
This PR fixes the broken table

Note that you Azure Pipeline DocBuild, step 'Build Documentation' even contains a corresponding error that unfortunately didn't let that build fail.


Updates to eslint-config-change-* packages

There have been a good number of changes lately, that warrant cutting new versions of the eslint-config-change-* modules:
https://github.com/change/javascript/compare/48670d3c61dfe87c035cce289fffe759bb21a0ea...chris/use-plugin-in-base
Might make sense to also try and get #16 into this?

Note that this is only failing in CI because we haven't yet published v7.0.0 of eslint-config-change-base to npm.

I just merged #16 if you want to merge master into this and incorporate it.
Code here LGTM!


idf version?

having issues with 4.x can you confirm the idf version you are using?

Let me check to see what happens with the latest version in platformio

This is the output from platform io when I kick off the build:
Verbose mode can be enabled via `-v, --verbose` option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/esp32doit-devkit-v1.html
PLATFORM: Espressif 32 (3.3.2) > DOIT ESP32 DEVKIT V1
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
DEBUG: Current (esp-prog) External (esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES:
 - framework-espidf 3.40300.0 (4.3.0)
 - tool-cmake 3.16.4
 - tool-esptoolpy 1.30100.210531 (3.1.0)
 - tool-ninja 1.9.0
 - toolchain-esp32ulp 1.22851.191205 (2.28.51)
 - toolchain-xtensa32 2.80400.210211 (8.4.0)

thanks!

I should have checked - is it issues with building or running?

there seems to be issues with zlib and reading the files, lots of loadprohibeted and abort() errors.  Not sure if it is my partitioning, i don't think I changed anything but it was an issue.  I tried just the single vader and xwing files but I also had issues.
I could get the spi working by making my own draw2() function and just feeding x/y so that part worked great.


[Go/abc.com] An extractor error has occurred. (caused by KeyError('video',))

Checklist

[ ] I'm reporting a broken site support issue
[ x] I've verified that I'm running youtube-dl version 2021.12.17
[ x] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ
c:\Tmp>youtube-dl --version
2021.12.17
youtube-dl --verbose -f "[height <=? 720][tbr>1500]" https://abc.com/tv/shows/life-below-zero-next-generation/episode-guide/season-04/episode-12-call-of-the-mountain/vdka27155346  -o "Life Below Zero Next Generation S04E12 Call of the Mountain [ABC].mp4"

Verbose log

PASTE VERBOSE LOG HERE
c:\Tmp>youtube-dl --verbose -f "[height <=? 720][tbr>1500]" https://abc.com/tv/shows/life-below-zero-next-generation/episode-guide/season-04/episode-12-call-of-the-mountain/vdka27155346  -o "Life Below Zero Next Generation S04E12 Call of the Mountain [ABC].mp4"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '-f', '[height <=? 720][tbr>1500]', 'https://abc.com/tv/shows/life-below-zero-next-generation/episode-guide/season-04/episode-12-call-of-the-mountain/vdka27155346', '-o', 'Life Below Zero Next Generation S04E12 Call of the Mountain [ABC].mp4']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg git-2020-06-12-38737b3, ffprobe git-2020-06-12-38737b3
[debug] Proxy map: {}
[Go] vdka27155346: Downloading JSON metadata
ERROR: An extractor error has occurred. (caused by KeyError('video',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\go.py", line 210, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\go.py", line 156, in _extract_videos
KeyError: 'video'
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\go.py", line 210, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\go.py", line 156, in _extract_videos
KeyError: 'video'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 547, in extract
youtube_dl.utils.ExtractorError: An extractor error has occurred. (caused by KeyError('video',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

c:\Tmp>

Description

WRITE DESCRIPTION HERE youtube-dl fails with the above error

Is that a valid link?

The content you’re looking for is not available.

It was at the time I got the url
Another one that WAS there is now gone .... note all the other downloads for (both) series worked. 😊
Title Port Protection Alaska S05E12
youtube-dl -f "[height <=? 720][tbr>1500]" https://abc.com/tv/shows/port-protection-alaska/episode-guide/season-05/episode-12-through-hell-or-high-water/vdka27169904 -o "Port Protection Alaska S05E12 Through Hell or High Water [ABC].mp4"
Title Life Below Zero Next Generation 12
youtube-dl -f "[height <=? 720][tbr>1500]" https://abc.com/tv/shows/life-below-zero-next-generation/episode-guide/season-04/episode-12-call-of-the-mountain/vdka27155346  -o "Life Below Zero Next Generation S04E12 Call of the Mountain [ABC].mp4"
Regards,
Mark
(Good judgment comes from experience; experience comes from bad judgment.)
From: Zombo
Sent: Friday, 20 May 2022 9:59 PM
To: ytdl-org/youtube-dl
Cc: Fetch-user; Author
Subject: Re: [ytdl-org/youtube-dl] An extractor error has occurred. (caused by KeyError('video',)) (Issue #30964)
Is that a valid link?
The content you’re looking for is not available.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

please stop pasting dead links...

The Go extractor could provide a better diagnostic. If there is enough information in the URL to find the video using http://api.contents.watchabc.go.com, the page isn't checked: then the empty metadata returned for the video causes the posted diagnostic.
Let's update PR #30649:
$ python -m youtube_dl -F -v 'https://abc.com/tv/shows/life-below-zero-next-generation/episode-guide/season-04/episode-12-call-of-the-mountain/vdka27155346'
[debug] System config: [u'--prefer-ffmpeg']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-F', u'-v', u'https://abc.com/tv/shows/life-below-zero-next-generation/episode-guide/season-04/episode-12-call-of-the-mountain/vdka27155346']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: ce49654dd
[debug] Python version 2.7.17 (CPython) - Linux-4.4.0-210-generic-i686-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 4.3, avprobe 4.3, ffmpeg 4.3, ffprobe 4.3
[debug] Proxy map: {}
[Go] vdka27155346: Downloading JSON metadata
ERROR: Go said: vdka27155346: No video found
Traceback (most recent call last):
  File "youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "youtube_dl/extractor/go.py", line 286, in _real_extract
    self._raise_extractor_error(video_id, 'No video found')
  File "youtube_dl/extractor/go.py", line 220, in _raise_extractor_error
    raise ExtractorError('%s said: %s: %s' % (self.IE_NAME, video_id, msg), expected=expected)
ExtractorError: Go said: vdka27155346: No video found

$

Thanks for all your help :)
ABC was a bit slow to remove the links


add electrum resolver

This PR closes https://github.com/RGB-WG/rgb/issues/95
Retrieving the block height of a transaction was not trivial (wondering why the electrum protocol doesn't expose such a feature), therefore the implementation of resolve_anchor is not as simple and readable as the esplora version. Anyway I tested this on rgb-lib and it seems to work fine.
Also, I was not sure about the naming, should I call the module/feature electrum_blocking (similar to current esplora_blocking) or is it ok to call it just electrum?

Retrieving the block height of a transaction was not trivial (wondering why the electrum protocol doesn't expose such a feature), therefore the implementation of resolve_anchor is not as simple and readable as the esplora version.

Yes. That was the exact reason I moved from electrum to esplora. Pls remember that with electrum the validation speed will be also significantly reduced because of this


Enable manual and auto abstraction for cells

Blocked on: https://github.com/runtimeverification/kontrol/pull/113
This PR enables a feature discussed with @geo2a for keeping configurations smaller. In particular:

KEVMSemantics is now extended by KontrolSemantics, which overrides the abstraction operator.
kontrol prove ... gets a new feature --auto-abstract-cell CELL_NAME_1 --auto-abstract-cell CELL_NAME_2, which allows the user to specify that the given cell should be auto-abstracted to a variable every iteration of exploration.
A new CLI argument is added kevm abstract TEST_ID --node NODE_ID_1 --node NODE_ID_2 --cell CELL_NAME_1 --cell CELL_NAME_2, which allows users to manually abstract a given node in the KCFG.
A test of the abstraction CLI functionality is added.
The --auto-abstract-gas flag is implemented in terms of this new functionality, and a warning thrown for using the old flag.

An example of using this would be:
kontrol prove ... --test MyContract.test_foo --auto-abstract-cell localMem --auto-abstract-cell gas --auto-abstract-cell refund
kontrol abstract MyContract.test_foo --node 3 --node 5 --cell localMem --cell callData

@PetarMax @lucasmt it would be good to know if this is useful for blockswap, @geo2a mentioned a given problem you guys are working on where exploration takes quite a long time. Perhaps this can be used to speed up exploration, and the following strategy can be taken:

Run the proof with kontrol prove ... --auto-abstract-cell ... (list a bunch of cells which grow over time and cause problems).
If the proof passes in this case, then we're done (as abstraction leads to over-approximation of the state space).
If the proof doesn't pass, we can still use the generated basic blocks for faster symbolic execution:

Do kontrol show --to-module ..., and add the generated rules (basic blocks) to the lemmas.k file you guys use.
Rekompile.
Rerun the proof with auto abstraction turned off (this time using the generated basic blocks for faster execution).

This could be useful I think. The issue we have is that the <memoryUsed> cell is not being simplified by the booster, so maybe abstracting it away could be an option until the simplification capabilities of the booster improve. I don't know if there's any point when we actually need the value stored in that cell.

This could be useful I think. The issue we have is that the <memoryUsed> cell is not being simplified by the booster, so maybe abstracting it away could be an option until the simplification capabilities of the booster improve. I don't know if there's any point when we actually need the value stored in that cell.

<memoryUsed> is simplified by the Haskell backend, though. From what I see, the only place it's used is:
rule <k> MSIZE    => 32 *Word MU         ~> #push ... </k> <memoryUsed> MU </memoryUsed>

and then MSIZE is only used for
rule <k> #gasExec(SCHED, MSIZE)          => Gbase < SCHED > ... </k>

so it doesn't seem very useful...
Am I missing something?

@ehildenb Does someone from Kontrol need to review this PR so that it can be merged?

I think this PR brings a potentially useful feature to the table and I am ok with it being merged - it is not passing CI, however.

CI is failing because the expected output needs to be updated.

@JuanCoRo @PetarMax we're running into some issues seemingly caused by a huge configuration size in OP proofs. Do you think it's worth to bring this PR back?

Possibly... but which cells specifically make the configuration huge and what are the precise issues?


fix(java): add native image configuration for Apache Arrow

Fixes  https://github.com/googleapis/java-bigquery/issues/2007. Reproducer: https://github.com/mpeddada1/native-image-arrow.
cc @meltsufin

That was quick. Great.

Thanks! The Apache Arrow problem was resolved but now we're running into some other issues that require troubleshooting. Debugging PR: https://github.com/googleapis/java-bigquery/pull/2019


feat(modelmanager): enable asynchronous initialization of modelmanager

initializeAsync: Alternative for ModelManager.initialize for asynchronous initialization of model manager
Can be used within default SPA and remote app as well
Order of preference for determining url for fetching root model for initialization:

path passed as config on ModelManager.initializeAsync
Value of meta property cq:pagemodel_root_url,if set
Currently open url path (for default SPA)

Sample usage: Before render of the application -
ModelManager.initializeAsync({
    modelClient: {customModelClient},                                 //optional
    path: "/content/we-retail-journal/react/en.model.json" //optional
});

I am good with the contribution. Please consider my questions and suggestions


bug(cast): fails in encoding of bytes if non-hexadecimal values passed

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (62cdea8 2024-07-23T00:19:34.618350138Z)
What command(s) is the bug in?
cast calldata
Operating System
Linux
Describe the bug
Try following command
cast calldata "sam(bytes)" dave

It fails parsing argument value because of v.
Technically, for non-hexadecimal values, it should be UTF8-encoding (ASCII values) padded right to 32 bytes. Look at third example https://docs.soliditylang.org/en/latest/abi-spec.html#examples

Hi @Amit0617 thanks for flagging this
Cast supports: cast calldata "sam(string)" dave, dave is not a valid bytes32
Marking as resolved

I was unsure about that, but probably that example in documentation is wrong.


⚠ Bofrost.es has degraded performance

In 24b11d6, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1612 ms

Resolved: Bofrost.es performance has improved in a6aa667 after 6 minutes.


Pulling in More Than One Terraform Role

Unable to pull in more than one role from Terraform when using apex deploy.

Operating system: OS-X
Apex Version: v0.10.2

Currently, when no value for a role is present an attempt is made to read the terraform.tfstate file and retrieve the output named lambda_function_role_id which is done here. This works fine when dealing with a function (or number of functions) that all have the same role. For my particular purposes this is not the case as I have roles that have different permissions in some cases.
What I'd like to be able to do instead is specify the output variable to look for with each of my functions so I can provide them with their varying permissions. I think backwards compatibility can be kept in place as it's a nice default for those who don't have the need for multiple roles within each project.
My initial thoughts are simply to have some format of string in the DSL that causes us to lookup said value in the Terraform output instead of just a hardcoded string. Below are a few proposals:
Terraform style syntax (familiar to those using Terraform):
{
  "role": "${myfunction_role_arn}"
}

Terraform-ish ($ is just there so we can identify that it is for a role to be interpolated):
{
  "role": "$myfunction_role_arn"
}

I'd love to hear some opinions on this or other proposed syntax.

nice, I have some in-progress work on adding HIL (fork here: https://github.com/tj/hil) for interpolation w/ functions, so we could add a function for grabbing state


Utilize ETags

Part 2 of #404
This builds on #410 by returning a 304 response for an ETag that matches the if-none-match header.
Here was the result of serving the cached file, modifying, and performing 2 refreshes.

This greatly reduces the amount of data transferred when all files are cached. It can impact the load times as well when working in larger apps. These next results are from the react-typescript scaffold.
Previously:

Utilizing ETags:

I'm unsure if all headers are necessary for the 304 response. From what I could tell, the ETag is but the rest I'm not sure about.
@FredKSchott

Yup, looks great! Love that transfer reduction, and it looks like it does have a noticeable effect on page speed ⚡


badarg jwt_auth required_claims with param

Description
I receive an unknown_error : badarg from erlang:list_to_existing_atom/1 when I try to authenticate using a JWT token and [jwt_auth] required_claims includes a tuple with param, like exp, iss {"yourissuername"} (from the example).
Steps to Reproduce

configure couchdb to accept jwt auth
[chttpd] authentication_handlers = {chttpd_auth, jwt_authentication_handler}
configure required claims to include a tuple claim, like the
example
[jwt_auth] required_claims = exp, {iss, "IssuerNameHere"}
make a request to _session bearing a JWT token.
(doesn't really matter which token, the error happens before actual validation afaict)

Expected Behaviour
To get authenticated (or not, but not an unknown error)
Your Environment
{"couchdb":"Welcome","version":"3.1.1","git_sha":"ce596c65d","uuid":"c4d21e152a90a6cf779e046c9ddb012b","features":["access-ready","partitioned","pluggable-storage-engines","reshard","scheduler"],"vendor":{"name":"The Apache Software Foundation"}}

CouchDB version used: 3.1.1
Browser name and version: n/a
Operating system and version: official docker image

Additional Context
It happens somewhere here. (In the main branch, that code looks different, with a regex supporting tuples.)
stack trace
[error] 2020-10-29T13:51:27.130984Z nonode@nohost <0.19228.0><PHONE_NUMBER> req_err(824051426) unknown_error : badarg [
	<<"erlang:list_to_existing_atom/1">>,
	<<"couch_httpd_auth:-get_configured_claims/0-lc$^0/1-0-/1 L216">>,
	<<"couch_httpd_auth:-get_configured_claims/0-lc$^0/1-0-/1 L216">>,
	<<"couch_httpd_auth:jwt_authentication_handler/1 L194">>,
	<<"chttpd:authenticate_request/2 L532">>,
	<<"chttpd:process_request/1 L304">>,
	<<"chttpd:handle_request_int/1 L244">>,
	<<"mochiweb_http:headers/6 L150">>
]

I realise now, after typing everything, that the docs I linked are for main, and tuples in required_claims were just not yet merged in 3.1.1. So perhaps this now turns into the question: how then do I provide which iss claim I expect?

I can confirm and reproduce the error on 3.1.1. with a present iss claim configured with the JWT issuer
[error] 2020-11-02T02:32:04.825057Z<EMAIL_ADDRESS><0.3213.0> adf7b85b19 req_err(2423103563) unknown_error : badarg
    [<<"erlang:list_to_existing_atom/1">>,<<"couch_httpd_auth:-get_configured_claims/0-lc$^0/1-0-/1 L216">>,<<"couch_httpd_auth:jwt_authentication_handler/1 L194">>,<<"chttpd:authenticate_request/2 L532">>,<<"chttpd:process_request/1 L304">>,<<"chttpd:handle_request_int/1 L244">>,<<"mochiweb_http:headers/6 L150">>,<<"proc_lib:init_p_do_apply/3 L247">>]
[notice] 2020-11-02T02:32:04.825317Z<EMAIL_ADDRESS><0.3213.0> adf7b85b19

{
  "error": "unknown_error",
  "reason": "badarg",
  "ref":<PHONE_NUMBER>
}

I discovered this issue it's already fixed with this #3165 which it's already merged on 3.x branch.
I've compiled from source and I can confirm the JWT authentication with ISS required claim it's working as expected.
So in the next release, it will be completely usable. For the moment if you need a CouchDB version with JWT auth and iss required claim, I'd suggest to compile from 3.x source branch.

For anyone else with this issue, if it helps I've cherry picked #3165 into 3.1.1 and created a new fork here:
https://github.com/RGS-IT-Development/couchdb/tree/<IP_ADDRESS>
I've also built a docker image here:
https://github.com/orgs/RGS-IT-Development/packages/container/package/couchdb
I can confirm this works for my environment.

FYI 3.2 is being prepared right now and should release in June 2021.

Also this PR https://github.com/apache/couchdb-config/pull/32 should be taken into account in order to allow kid fields with equal signs.

@mtenrero good find, our JWT issuer rotates it's keys so we don't load them in the .ini file we dynamically push them to the couchdb config URL (https://docs.couchdb.org/en/latest/api/server/configuration.html), so not sure this is an issue for us, but maybe for others, thanks!


ci: Make Windows tests ~27% faster by putting temp folder in dev drive

Summary
This PR makes cargo test | windows faster in CI.
Before

After

Also
This PR disables the brotli feature of async-compression since it's not strictly needed, but this has little to do with the improvements (it's still less code to build).
This PR introduces additional code in uv tool uninstall to ignore errors (that only seem to happen on ReFS, ie. on Dev Drives) akin to "the thing we're trying to delete cannot be deleted because it's already being deleted".
If raw_os_error was stable we could do u32 matching instead of that .to_string().contains() abomination.

Part of https://github.com/astral-sh/uv/issues/5713


NAS-130585 / 24.10-RC.1 / Exclude nvidia-related changes from /usr daatset diff (by themylogin)

Original PR: https://github.com/truenas/ixdiagnose/pull/208
Jira URL: https://ixsystems.atlassian.net/browse/NAS-130585

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.


Update NewPrometheus function signature

Currently takes a variable # of slices of *Metric as an argument and panics when it is passed >1.
Could this not just take a variable # of *Metric instead?

Yeah and can we get a version that doesn't use the standard metrics list?

I didn't notice this PR and created a duplicate here https://github.com/zsais/go-gin-prometheus/pull/40 , sorry!


fix: Broken authentication on dev

Description
This PR intends to fix the broken dev auth due to a changed cookie key.
The auth cookie name at https://beta.insights.opensauced.pizza is sb-fcqqkxwlntnrtjfbcioz-auth-token. At https://insights.opensauced.pizza it is supabase-auth-token.
The useAuth hook has been improved to check and propagate auth. The checkAuthentication function has been removed along with the onInstalled, onStartup and cookies.onChanged background listeners as the useAuth hook will suffice from here on.
Any future changes to the key will have to be updated here.
https://github.com/open-sauced/ai/blob/cc76e950a33d30677e2eb37fadb3d318ceca2d5b/src/constants.ts#L7
The rest of the authentication can remain unchanged.
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[x] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Resolves #214.
Mobile & Desktop Screenshots/Recordings
Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

Curious if this is similar - https://github.com/open-sauced/hot/issues/477

Curious if this is the same issue happening hot.opensauced.pizza - open-sauced/hot#477

The VITE_API_URL of beta.hot.opensauced.pizza is broken, https://lkkownkrbkmblczeoyqb.supabase.co/.
While the one for hot.opensauced.pizza, https://ibcwmlhcimymasokhgvn.supabase.co/, is working fine.


locals use in TemplatePortalHost

Bug, feature request, or proposal:
Feature request
What is the expected behavior?
Portal is an interesting part of a component dev toolkit, and one portion that strikes me as with potential but not very much explored—at least from an outside perspective—is the locals property in TemplatePortalHost which is not explained very much in either documentation or via examples in the unit or e2e testing. Further clarification on those items would be great to have.
What is the current behavior?
Little to no documentation. There are some TODO comments to return locals but they're pending within. Additionally, there are no tasks slotted in Github that I can identify to complete this piece, either in code or documentation.
Is there anything else we should know?
Currently viewing this on angular/material2 v2.0.0-beta.1

Closing into the broader #2789


Support geometry shaders when a OpenGL 3.2+ context is available

Implements/Fixes #1768.
How do you test this? Well... the hardest part is getting a system that doesn't advertise ARB_geometry_shader4 although it supports OpenGL 3.2+. Then just run the geometry shader example and see if anything shows up on the screen.

The shader example now works.