[Snyk] Fix for 2 vulnerabilities

Snyk has created this PR to fix 2 vulnerabilities in the maven dependencies of this project.
Snyk changed the following file(s):

kernel/kernel-impl/pom.xml
master/pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue
Score
Upgrade

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230364
  40
org.springframework.data:spring-data-jpa:  2.5.3 -> 3.2.11  org.springframework:spring-context:  5.3.18 -> 6.1.14  org.springframework:spring-context-support:  5.3.18 -> 6.1.14 Major version upgrade No Known Exploit

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230365
  40
org.springframework.data:spring-data-commons:  2.4.2 -> 3.2.11  org.springframework.data:spring-data-jpa:  2.5.3 -> 3.2.11  org.springframework:spring-beans:  5.3.18 -> 6.1.14  org.springframework:spring-context:  5.3.18 -> 6.1.14  org.springframework:spring-context-support:  5.3.18 -> 6.1.14  org.springframework:spring-core:  5.3.18 -> 6.1.14  org.springframework:spring-jdbc:  5.3.18 -> 6.1.14  org.springframework:spring-orm:  5.3.18 -> 6.1.14  org.springframework:spring-tx:  5.3.18 -> 6.1.14 Major version upgrade No Known Exploit

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: code/snyk check is complete. No issues have been found. (View Details)


Add json output

#32

Hello @vianneybacoup ,
In this PR I put all test results into "target" folder excepted "output.*" because it's the default value for the output_name.
I do that to keep root folder "clean", to easier clean test results and also to avoid to add all extensions into the .gitignore file.
But, as default value is "output", I don't really know if it's really useful.
What do you think about that?
Thank you for your feedback.

Hello @vianneybacoup , In this PR I put all test results into "target" folder excepted "output.*" because it's the default value for the output_name. I do that to keep root folder "clean", to easier clean test results and also to avoid to add all extensions into the .gitignore file. But, as default value is "output", I don't really know if it's really useful. What do you think about that?
Thank you for your feedback.

I see that we both had the same issues, I did way too much 'rm *.parquet *.csv' past months haha
I had in mind we could have like in Python a tearDown fucntion that cleanup the output of the test automatically, but that does not exist in rust, and this is not a mandatory feature, the folder is clearly enough.
Maybe rename it with target/test_generated or something like this just to avoid a possible confusion with the other targets?

I moved output tests into target/test_generated/ folder.
I kept "all options" tests into target folder because they are run only by github action and generation fails if folder not exists (maybe could be a good enhancement to automatically generate intermediate folders?).
I added ctor crate to add macro to create target/test_generated/ folder before tests and clean test outputs after tests.
About json output, I did a POC using arrow (https://github.com/bhagenbourger/Fakelake/tree/poc/use_arrow_for_all_format) because I found interesting to use the same "generator" for all formats. It works but with some limitations as using the same date format for all columns. So definitively, serde-json is more flexible and I keep this implementation.


chore: Update Readme

Updating to reflect current state

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


travis-cargo currently cannot upload / generate code coverage information for documentation tests

There should be a way in cargo for travis-cargo to be able to run the documentation tests with code-coverage generation, so that it can upload correct code coverage information to coveralls.
The travis-cargo issue is here: https://github.com/huonw/travis-cargo/issues/63

Thanks for the report! Mind reopening this on rust-lang/rust? I think this is specifically a bug for rustdoc --test perhaps?

Closing this issue in favor of https://github.com/rust-lang/rust/issues/36956.


Arduino IDE 1.8 can't work with relative paths, expand to full path

Arduino IDE 1.8 can't understand relative paths. If you change dir to your project and open test.ino in vim - expand('%') will return test.ino. expand('%:p') return full path to opened file.

Love it.
Thanks for the PR!


Move Governments listing page to the GOV.UK Design System

Description
This ports the governments listing page to the GOV.UK Design System. It appends legacy to the current view. Then adds the new views in the GOV.UK Design System and conditionally renders the Bootstrap or GOV.UK Design System based on whether the user has the "Preview design system" permission.
Screenshot

Trello
https://trello.com/c/quWvC2uo
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

I would squash the last two commits together (as the first one causes the issue)

Generally looks good.
The first commit seems to do a lot; maybe the No governments have been created context should be moved onto its separate commit.
Some commits where you're fixing things caused by the commits on the same PR should be fixed up so that it's done on the initial commit.

Sorry, @DilwoarH, just spotted that one of the design changes got lost. It should be Create new government, not Create a government

Great work Chris. I'm going to merge this and move it to the PA review column 👍


XmlGraph.java:39-44: Continue the work started with...

The puzzle 445-b9bbd966 from #445 has to be resolved:
https://github.com/yegor256/jpeek/blob/05d4100f0ab757ce44fd7f2087f8c35f5dfe3588/src/main/java/org/jpeek/graph/XmlGraph.java#L39-L44
The puzzle was created by Vytautas Žurauskas on 24-Apr-20.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@paulodamaso/z please, pay attention to this issue

The job #472 assigned to @marceloamadeu/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@marceloamadeu/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

The job #472 assigned to @victornoel/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat wait for #476 to be mergd

@0crat status

@0crat status (here)

@victornoel This is what I know about this job in C7JGJ00DP, as in §32:

The job #472 is in scope for 11days
The role is DEV
The job is assigned to @victornoel/z for 5days
There is a monetary reward attached
@victornoel/z will get 50% of their hourly rate for this job
The job has an impediment
The budget is 30 minutes/points
These users are banned and won't be assigned:

@0pdd/z: This user reported the ticket
@marceloamadeu/z: User was resigned from the ticket

Job footprint (restricted area)

@paulodamaso could you maybe re-run the master branch CI build to retrigger 0pdd and that it closes this issue as it should have?

@paulodamaso or @yegor256 said to do a push to master to re-trigger it

@paulodamaso ping

@victornoel Seems that there are trigger problems, I merged another PR yesterday and 0pdd did not updated this one

@paulodamaso so maybe can you tell 0crat out? If that can solve the problem...

@0crat out

@0crat out (here)

@paulodamaso GitHub issue is still open, won't close

@0crat status

@0crat status (here)

@victornoel This is what I know about this job in C7JGJ00DP, as in §32:

The job #472 is in scope for 18days
The role is DEV
The job is assigned to @victornoel/z for 11days
There is a monetary reward attached
@victornoel/z will get 50% of their hourly rate for this job
The job has an impediment
The budget is 30 minutes/points
These users are banned and won't be assigned:

@0pdd/z: This user reported the ticket
@marceloamadeu/z: User was resigned from the ticket

Job footprint (restricted area)

@paulodamaso can you retry?

@0crat out

@sereshqua/z please review this job completed by @victornoel/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@victornoel Finally, yay!

@0crat quality good

@paulodamaso @victornoel good job :)

@rultor release tag is 0.30.25

@rultor release tag is 0.30.25

@paulodamaso OK, I will release it now. Please check the progress here

@rultor release tag is 0.30.25

@paulodamaso Done! FYI, the full log is here (took me 19min)


Show full bitcoin address or collapsed link

showing full BTC address.  link is not really available in this context; only for section headers like the Project Tasks header/link.


Issue testing endpoint in lab 04 -

Module: Deploy a model to a managed online endpoint
Lab/Demo: 04
Task: Test the endpoint
Step:
Description of issue
When running the online-endpoint invoke command I receive the following error
cli.azure.cli.core.azclierror: Met error <class 'Exception'>:Invalid input format: expected an array of items.
Please check log by running the command with '--debug' for more details.
az_command_data_logger: Met error <class 'Exception'>:Invalid input format: expected an array of items.
Please check log by running the command with '--debug' for more details.
Repro steps:
Run -  az ml online-endpoint invoke --name diabetes-mlflow --request-file ./mslearn-aml-cli/Allfiles/Labs/04/mlflow-endpoint/sample-data.json

@Typod01 Could you be more specific? I don't think lab 4 includes a step that says to use az ml online-endpoint invoke? And the lab about online endpoints uses an endpoint with a different name?

Hi, here are some more details.  the link to the content is - https://microsoftlearning.github.io/mslearn-aml-cli/Instructions/Labs/05-deploy-managed-endpoint.html
but the command refers to lab04 files:
az ml online-endpoint invoke --name diabetes-mlflow --request-file ./mslearn-aml-cli/Allfiles/Labs/04/mlflow-endpoint/sample-data.json

Thanks. We'll review the labs and get back to you with a resolution.

We've updated the labs to fix some issues. I'm closing this for now. Please let us know if you're still experiencing some issues.


fix(deps): :arrow_up: update dependency aws-cdk to ^2.117.0

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

aws-cdk (source)
^2.116.1 -> ^2.117.0

Release Notes

aws/aws-cdk (aws-cdk)
v2.117.0
Compare Source
Features

update L1 CloudFormation resource definitions (#28489) (607dccb), closes /docs.aws.amazon.com/datasync/latest/userguide/working-with-locations.html#create-s3
api-gateway-v2: Add method to generate flexible execute-api ARN (#28400) (0932027), closes #23301
autoscaling: instance maintenance policy for AutoScalingGroup (#28092) (5eb2c26), closes #28042
codebuild: Lambda compute for codebuild projects (#27934) (a4a4c6f), closes #28418
core: option to disable cache for docker build (#27944) (84d81b6), closes #27916
ec2: trn1 instance type (#28477) (ed02d5c)
ecs: Implement method in ECS cluster to retrieve task ARN (#28381) (7635bbc), closes #26232
ecs-patterns: support disabling CPU-based scaling and custom target utilization (#28315) (3cb3e02), closes #20706 #20706
elasticloadbalancingv2: dualstack NetworkLoadBalancer (#27546) (e03a41f), closes /docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-elasticloadbalancingv2-loadbalancer.html#cfn-elasticloadbalancingv2 #27538
elasticloadbalancingv2: FIPS compliant SSL policy to ALB (#28479) (7bb72e1), closes #28455
globalaccelerator: add support for ip addresses and type (#28055) (943abe8), closes #28051 #28209
ses: synth-time naming validation for dedicatedIpPoolName (#28466) (be6ddb8), closes #28451

Bug Fixes

core: core constructs fail with Error: Cannot find module '../dist/core/<file>.generated' (#28467) (cd54c42), closes #28251 #28465
ecs-patterns: ApplicationMultipleTargetGroupsEc2Service ignores load balancer name (#28394) (9c0fecf), closes #23535

Alpha modules (2.117.0-alpha.0)
Bug Fixes

lambda-python-alpha: bundling hash logic includes auth tokens in PIP urls, causing an unnecessary rebuild (#27903) (00331a7), closes #27331
lambda-python-alpha: use function architecture (#18696) (#28449) (c724d27)

v2.116.1
Compare Source
Bug Fixes

core: core constructs fail with Error: Cannot find module '../dist/core/<file>.generated' (#28467) (e8be128), closes #28251 #28465

Alpha modules (2.116.1-alpha.0)
v2.116.0
Compare Source
Features

apigateway: validate integrationHttpMethod with non-MOCK integration types (#28316) (93cb6e4)
ec2: support creating key pairs (#28138) (6b3caff), closes #5252
ecr: Add emptyOnDelete CloudFormation property to Repository L2 construct (#28233) (a175da8), closes #24572 #28196
ecr: tag pattern list for lifecycle policy (#28432) (aa075cd)
ecs: log retention for FireLensLogDriver (#28354) (52a5579), closes /github.com/aws/aws-cdk/blob/db22b85c9b2a853aa2f830c182a340f0bcf95d1a/packages/aws-cdk-lib/aws-ecs/lib/base/task-definition.ts#L816 /github.com/aws/aws-cdk/blob/db22b85c9b2a853aa2f830c182a340f0bcf95d1a/packages/aws-cdk-lib/aws-ecs/lib/firelens-log-router.ts#L170 #28258
ecs: Support specifying revision of task definition (#27036) (de0d77b), closes #26983
events: add multiple event bus policies on a single event bus (#27340) (4dde502), closes #24671
rds: support aurora mysql 3.05.1 (#28370) (986db38)
ses: add scalingMode property to dedicated ip pool (#28392) (c687778), closes #27418
sns: delivery status logging configuration (#28433) (36bb696), closes #21971
stepfunctions: CustomState addCatch (#28422) (cf923bc), closes #25798
stepfunctions: support Map ItemProcessor (#27913) (ac41730), closes #27878
stepfunctions-tasks: EMR createCluster command support OnDemandSpecification (#27791) (73a5e74), closes #27761
implement code generated handler framework (#28251) (1a9c30e), closes #27303
update AWS Service Spec (#28407) (fcb701c)

Bug Fixes

core: messages from annotations.ts can show up as [object Object] (#28414) (3e6f10d)
custom-resources: cannot set logging for state machine generated in CompleteHandler (#27310) (bfeef00), closes #27283
ecs-patterns: taskDefinition ignored by queueProcessingFargateService (#28220) (5d6b8ee), closes #27360
eks: neuron plugin not added with addNodegroupCapacity() (#27909) (b87212b)
events-targets: imported sqs queue cannot be used as a rule dlq (#28165) (#28285) (588b106)
iam: importedRoleStackSafeDefaultPolicyName feature flag results in excessively long IAM policy names (#27548) (4f88db6), closes #27409 #24441
logs: adding a resource policy statement with AnyPrincipal fails (#27787) (0b2ba1c), closes #27783
s3-notifications: multiple notifications doesn't work (#28132) (37be7b9)
secretsmanager: cannot set hourly rotation (#28303) (09cb003), closes #28261
secretsmanager: hosted rotation with fromSecretNameV2() does not create correct iam policy (#28379) (4ab5bb2), closes #28308

Alpha modules (2.116.0-alpha.0)
Features

scheduler: flexible time windows (#28098) (6554e48)
scheduler-targets: add CodePipeline as target for scheduler (#27799) (8c44f32), closes #27449

Bug Fixes

integ-tests: apply correct IAM policy to waiterProvider (#28424) (c488035), closes 40aws-cdk/integ-tests-alpha/lib/assertions/sdk.ts#L136 40aws-cdk/integ-tests-alpha/lib/assertions/sdk.ts#L247 #27865
lambda-python-alpha: pipenv lock -r is no longer supported (#28317) (f85f486), closes #28015 /github.com/pypa/pipenv/blob/main/CHANGELOG.md#2022813-2022-08-13 #28015

v2.115.0
Compare Source
Features

eks: support Bottlerocket Nvidia AMIs (#28287) (6aa1b1b), closes #28241
rds: support RDS for MariaDB 10.11.6, 10.6.16, 10.5.23 and 10.4.32 (#28345) (0347a19)
rds: support RDS for SQL Server 16.00.4095.4.v1 (#28274) (7a22501)
stepfunctions-tasks: runtime role in EmrAddStep  (#27736) (314fbfa), closes #27691
stepfunctions-tasks: support for the Step Functions optimized integration for Bedrock InvokeModel API (#28276) (f3dafa4)
update AWS Service Spec (#28328) (15c7bb2)
ROADMAP: updates to public roadmap 🚀  (#28302) (fe30921)

Bug Fixes

rds: publiclyAccessible=false set on an instance is ignored when cluster is placed in a public subnet (#28038) (569593c), closes #28037 #28037
route53: delete old NS records on cross-account delegation renaming (#21249) (#27523) (448e2bd), closes #25285
stepfunctions-tasks: missing tags & perms for emr cluster creation (#28327) (14e5e50)
stepfunctions-tasks: state machine role is missing sagemaker:AddTags permission for SageMakerCreateTransformJob task (#27264) (8ce0283), closes #26012

Alpha modules (2.115.0-alpha.0)
⚠ BREAKING CHANGES TO EXPERIMENTAL FEATURES

scheduler: The typos in the Schedule and Group construct method names have been fixed, changing metricSentToDLQTrunacted to metricSentToDLQTruncated and metricAllSentToDLQTrunacted to metricAllSentToDLQTruncated.
redshift: Further updates of the Redshift table will fail for existing tables, if the table name is changed. Therefore, changing the table name for existing Redshift tables have been disabled.

Features

appconfig-alpha: add deploy method to configuration constructs (#28269) (c723ef9)
cloud9-alpha: support image ids for Amazon Linux 2023 and Ubuntu 22.04 (#28346) (93681e0), closes /docs.aws.amazon.com/ja_jp/AWSCloudFormation/latest/UserGuide/aws-resource-cloud9-environmentec2.html#cfn-cloud9-environmentec2
scheduler: start and end time for schedule construct (#28306) (0b4ab1d), closes /github.com/aws/aws-cdk/pull/26819#discussion_r1301532299

Bug Fixes

appconfig-alpha: extensions always create cdk diff (#28264) (2075559), closes #27676
redshift: tables were dropped on table name change (#24308) (7ac237b), closes #24246
scheduler: typo in metricSentToDLQ... methods (#28307) (8b91e10)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
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/@angular/core
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/@angular/animations
npm ERR<EMAIL_ADDRESS>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!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-12-30T21_33_23_435Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-12-30T21_33_23_435Z-debug-0.log


Malformed or complex PDFs cause an out-of-memory exception.

The program fails with an out-of-memory exception when processing some particularly complex pdfs. The memory requirement can be quite large, exceeding 10 GB for a single pdf of relatively small filesize, thus failing regardless of the amount of memory assigned to the jvm.
A temporary work-around is use a script to process just one pdf at a time (while catching memory exceptions), however a lot of time is lost to repeatedly firing up and closing the jvm.
A more desirable behaviour would be for pdffigures to abort the current pdf after a predefined memory usage is exceeded and proceed with others.

@tom-a-horrocks Once you have assembled the jar file (by running sbt assembly), you can restrict the memory usage by using the -Xmx option when running the jar.
java -Xmx2048m -jar pdffigures2/target/scala-2.11/pdffigures2-assembly-0.0.12-SNAPSHOT.jar
Let me know if this helps!


Fuel cell output not included in hybrid cashflow

Describe the bug
The fuel cell's power output is not registering in the cash flow for PPA sales
To Reproduce
Steps to reproduce the behavior:

Run attached SAM case
fuel_cell_not_in_cashflow.zip

Fuel cell is running in one situation but not the other

Cash flow is the same, but it should be different

Expected behavior
Fuel cell output should be included in cash flow

See https://github.com/NREL/ssc/issues/1056 and #1474


coordinate parsing fixes

Fixes #667 and #640

Rename KGS -> GTP in Python code.
SGF: parse tt as pass for board sizes <= 19 for both Python and C++ (I misread the spec and C++ previously only did this for board size == 19).
GTP: parse uppercase PASS as pass in Python code (C++ already does this correctly).

Note that oneoffs/prepare_bigquery.py and oneoffs/bigquery_moves_schema.json still have references to move_kgs but I don't know if those are safe to change.

@brilee re: the bigquery script changes.


[WIP,ENH] fMRI summary plot to take _confounds.tsv

Also updates the wrapper interface for MRIQC to keep working
Preparing for poldracklab/fmriprep#174 and poldracklab/fmriprep#877

This is how they look now:

That does look nice, and I think they give a good gist for people familiar with the metrics, but I think it would be good also to add a moderately detailed "How to read" section in the MRIQC/fMRIPrep docs.
That's not an issue for this PR, but just a heads up for the apps that use this.

Working already on that howto. Agreed on that this plot should be explained there (https://1387-54756129-gh.circle-artifacts.com/1/home/ubuntu/scratch/docs/interpreting.html)


drop special behavior to support Go pre-1.13

Codecov Report

Merging #446 (e1e5449) into main (08c6c71) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #446      +/-   ##
==========================================
- Coverage   53.25%   53.23%   -0.03%
==========================================
  Files          37       36       -1
  Lines        1842     1841       -1
==========================================
- Hits          981      980       -1
  Misses        710      710
  Partials      151      151

Impacted Files
Coverage Δ

pkg/build/gobuild.go
58.35% <100.00%> (ø)

Continue to review full report at Codecov.

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


docs: add travis and gitlab ci sections

adds travis and gitlab ci sections to the CI guide.
reformatted the Remote caching section to be more open-ended
was able to test travis, was not able to test gitlab (wouldn't let me run without handing over my credit card, but the file is considered valid by their validator).

Ah https://github.com/vercel/turborepo/pull/1131 totally changes this section - will refactor my contribution shortly

I broke each out into pages. Should be better for SEO

closed in favor of #1137


Update beginner workflow tutorial

With the changes that have been made to get_observations etc we need to update the beginner workflow tutorial to make sure it's up to date.

Done.


This is a fix for colons in the password of a HTTP Basic Authentication.

The existing code was taking the first component of the colon-separated array as the user name,
and the second as the password. In fact, all components after the first must be treated as belonging
to the password, and joined together with a colon separator.
Fix for https://github.com/IBM-Swift/Kitura-CredentialsHTTP/issues/54

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

Thanks for the PR! I think it might be worth reworking this code to use https://developer.apple.com/documentation/swift/string/2894564-split instead of components(). Then we can set maxSplits = 1 and won't have to recombine.

Fix looks good to me! Are you happy @Andrew-Lees11 ?


Pass proxy settings from GRADLE_OPTS to P2 director

I'm running from within a corporate environment  from behind a proxy server. I've set up all of the proxy options in my GRADLE_OPTS. When I run ./gradlew ide it pulls down all of the dependencies of the build just fine, but then barfs - I assume because whatever is being run can't get out to the internet because the proxy information isn't there...I'm running on MacOSX 10.11.4
$ ./gradlew ide
Downloading https://services.gradle.org/distributions/gradle-2.14-bin.zip
...........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
Unzipping /Users/deandre/.gradle/wrapper/dists/gradle-2.14-bin/76oc0mnc3ieqtsukq90mp0rxk/gradle-2.14-bin.zip to /Users/deandre/.gradle/wrapper/dists/gradle-2.14-bin/76oc0mnc3ieqtsukq90mp0rxk
Set executable permissions for: /Users/deandre/.gradle/wrapper/dists/gradle-2.14-bin/76oc0mnc3ieqtsukq90mp0rxk/gradle-2.14/bin/gradle
Download https://plugins.gradle.org/m2/com/diffplug/gradle/goomph/3.0.1/goomph-3.0.1.pom
Download https://plugins.gradle.org/m2/org/standardout/bnd-platform/1.4.0/bnd-platform-1.4.0.pom
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-core/1.2.0/durian-core-1.2.0.pom
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-collect/1.2.0/durian-collect-1.2.0.pom
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-io/1.2.0/durian-io-1.2.0.pom
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-swt/2.0.0/durian-swt-2.0.0.pom
Download https://plugins.gradle.org/m2/biz/aQute/bnd/biz.aQute.bndlib/3.2.0/biz.aQute.bndlib-3.2.0.pom
Download https://plugins.gradle.org/m2/org/eclipse/tycho/org.eclipse.osgi/3.10.101.v20150820-1432/org.eclipse.osgi-3.10.101.v20150820-1432.pom
Download https://plugins.gradle.org/m2/com/google/code/findbugs/annotations/3.0.0/annotations-3.0.0.pom
Download https://plugins.gradle.org/m2/biz/aQute/bnd/bndlib/2.2.0/bndlib-2.2.0.pom
Download https://plugins.gradle.org/m2/biz/aQute/bnd/parent/2.2.0/parent-2.2.0.pom
Download https://plugins.gradle.org/m2/de/undercouch/gradle-download-task/3.0.0/gradle-download-task-3.0.0.pom
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-concurrent/1.2.0/durian-concurrent-1.2.0.pom
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-rx/2.0.0/durian-rx-2.0.0.pom
Download https://plugins.gradle.org/m2/io/reactivex/rxjava/1.1.0/rxjava-1.1.0.pom
Download https://plugins.gradle.org/m2/org/osgi/org.osgi.core/4.2.0/org.osgi.core-4.2.0.pom
Download https://plugins.gradle.org/m2/org/apache/httpcomponents/httpclient/4.4.1/httpclient-4.4.1.pom
Download https://plugins.gradle.org/m2/org/apache/httpcomponents/httpcomponents-client/4.4.1/httpcomponents-client-4.4.1.pom
Download https://plugins.gradle.org/m2/org/apache/httpcomponents/httpcore/4.4.1/httpcore-4.4.1.pom
Download https://plugins.gradle.org/m2/org/apache/httpcomponents/httpcomponents-core/4.4.1/httpcomponents-core-4.4.1.pom
Download https://plugins.gradle.org/m2/com/diffplug/gradle/goomph/3.0.1/goomph-3.0.1.jar
Download https://plugins.gradle.org/m2/org/standardout/bnd-platform/1.4.0/bnd-platform-1.4.0.jar
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-core/1.2.0/durian-core-1.2.0.jar
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-collect/1.2.0/durian-collect-1.2.0.jar
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-io/1.2.0/durian-io-1.2.0.jar
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-swt/2.0.0/durian-swt-2.0.0.jar
Download https://plugins.gradle.org/m2/biz/aQute/bnd/biz.aQute.bndlib/3.2.0/biz.aQute.bndlib-3.2.0.jar
Download https://plugins.gradle.org/m2/org/eclipse/tycho/org.eclipse.osgi/3.10.101.v20150820-1432/org.eclipse.osgi-3.10.101.v20150820-1432.jar
Download https://plugins.gradle.org/m2/com/google/code/findbugs/annotations/3.0.0/annotations-3.0.0.jar
Download https://plugins.gradle.org/m2/biz/aQute/bnd/bndlib/2.2.0/bndlib-2.2.0.jar
Download https://plugins.gradle.org/m2/de/undercouch/gradle-download-task/3.0.0/gradle-download-task-3.0.0.jar
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-concurrent/1.2.0/durian-concurrent-1.2.0.jar
Download https://plugins.gradle.org/m2/com/diffplug/durian/durian-rx/2.0.0/durian-rx-2.0.0.jar
Download https://plugins.gradle.org/m2/io/reactivex/rxjava/1.1.0/rxjava-1.1.0.jar
Download https://plugins.gradle.org/m2/org/osgi/org.osgi.core/4.2.0/org.osgi.core-4.2.0.jar
Download https://plugins.gradle.org/m2/org/apache/httpcomponents/httpclient/4.4.1/httpclient-4.4.1.jar
Download https://plugins.gradle.org/m2/org/apache/httpcomponents/httpcore/4.4.1/httpcore-4.4.1.jar
p2AsMaven eclipse-deps is dirty.
Initalizing maven group eclipse-deps from p2
Only needs to be done once, future builds will be much faster
p2AsMaven eclipse-deps installing from p2
Generating JAR file 'gradle-api-2.14.jar'
Buildfile: /var/folders/_8/9zglx8rx4_77k28y2gm2y4t5917xyg/T/goomph-ant-build5905443405532642969.xml
[p2.mirror] SLF4J: Class path contains multiple SLF4J bindings.
[p2.mirror] SLF4J: Found binding in [jar:file:/Users/deandre/.gradle/caches/2.14/generated-gradle-jars/gradle-api-2.14.jar!/org/slf4j/impl/StaticLoggerBinder.class]
[p2.mirror] SLF4J: Found binding in [jar:file:/Users/deandre/.gradle/wrapper/dists/gradle-2.14-bin/76oc0mnc3ieqtsukq90mp0rxk/gradle-2.14/lib/gradle-logging-2.14.jar!/org/slf4j/impl/StaticLoggerBinder.class]
[p2.mirror] SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
[p2.mirror] SLF4J: Actual binding is of type [org.gradle.internal.logging.slf4j.OutputEventListenerBackedLoggerContext]
BUILD FAILED
/var/folders/_8/9zglx8rx4_77k28y2gm2y4t5917xyg/T/goomph-ant-build5905443405532642969.xml:2: org.eclipse.equinox.p2.core.ProvisionException: Unknown Host: http://download.eclipse.org/eclipse/updates/4.5/R-4.5.2-201602121500/artifacts.xml
at org.eclipse.equinox.internal.p2.repository.CacheManager.createCache(CacheManager.java:243)
at org.eclipse.equinox.internal.p2.artifact.repository.simple.SimpleArtifactRepositoryFactory.getLocalFile(SimpleArtifactRepositoryFactory.java:65)
at org.eclipse.equinox.internal.p2.artifact.repository.simple.SimpleArtifactRepositoryFactory.load(SimpleArtifactRepositoryFactory.java:88)
at org.eclipse.equinox.internal.p2.artifact.repository.simple.SimpleArtifactRepositoryFactory.load(SimpleArtifactRepositoryFactory.java:76)
at org.eclipse.equinox.internal.p2.artifact.repository.ArtifactRepositoryManager.factoryLoad(ArtifactRepositoryManager.java:73)
at org.eclipse.equinox.internal.p2.repository.helpers.AbstractRepositoryManager.loadRepository(AbstractRepositoryManager.java:768)
at org.eclipse.equinox.internal.p2.repository.helpers.AbstractRepositoryManager.loadRepository(AbstractRepositoryManager.java:668)
at org.eclipse.equinox.internal.p2.artifact.repository.ArtifactRepositoryManager.loadRepository(ArtifactRepositoryManager.java:104)
at org.eclipse.equinox.p2.internal.repository.tools.AbstractApplication.addRepository(AbstractApplication.java:147)
at org.eclipse.equinox.p2.internal.repository.tools.AbstractApplication.initializeRepos(AbstractApplication.java:116)
at org.eclipse.equinox.p2.internal.repository.tools.tasks.MirrorTask.execute(MirrorTask.java:59)
at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:293)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.apache.tools.ant.dispatch.DispatchUtils.execute(DispatchUtils.java:106)
at org.apache.tools.ant.Task.perform(Task.java:348)
at org.apache.tools.ant.Target.execute(Target.java:435)
at org.apache.tools.ant.helper.ProjectHelper2.parse(ProjectHelper2.java:180)
at org.eclipse.ant.internal.core.ant.InternalAntRunner.parseBuildFile(InternalAntRunner.java:387)
at org.eclipse.ant.internal.core.ant.InternalAntRunner.run(InternalAntRunner.java:675)
at org.eclipse.ant.internal.core.ant.InternalAntRunner.run(InternalAntRunner.java:568)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.eclipse.ant.core.AntRunner.run(AntRunner.java:511)
at org.eclipse.ant.core.AntRunner.start(AntRunner.java:608)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:380)
at com.diffplug.gradle.eclipserunner.EquinoxLauncher$Running.run(EquinoxLauncher.java:183)
at com.diffplug.gradle.eclipserunner.EquinoxLauncher$Running.access$100(EquinoxLauncher.java:165)
at com.diffplug.gradle.eclipserunner.EquinoxLauncher.run(EquinoxLauncher.java:156)
at com.diffplug.gradle.eclipserunner.JarFolderRunner.run(JarFolderRunner.java:36)
at com.diffplug.gradle.eclipserunner.JarFolderRunnerExternalJvm$RunOutside.run(JarFolderRunnerExternalJvm.java:98)
at com.diffplug.gradle.JavaExecable.main(JavaExecable.java:134)
Caused by: java.net.UnknownHostException: download.eclipse.org
at java.net.InetAddress.getAllByName0(InetAddress.java:1280)
at java.net.InetAddress.getAllByName(InetAddress.java:1192)
at java.net.InetAddress.getAllByName(InetAddress.java:1126)
at org.apache.http.impl.conn.SystemDefaultDnsResolver.resolve(SystemDefaultDnsResolver.java:44)
at org.apache.http.impl.conn.DefaultClientConnectionOperator.resolveHostname(DefaultClientConnectionOperator.java:259)
at org.apache.http.impl.conn.DefaultClientConnectionOperator.openConnection(DefaultClientConnectionOperator.java:159)
at org.apache.http.impl.conn.ManagedClientConnectionImpl.open(ManagedClientConnectionImpl.java:304)
at org.apache.http.impl.client.DefaultRequestDirector.tryConnect(DefaultRequestDirector.java:611)
at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:446)
at org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:863)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
at org.eclipse.ecf.provider.filetransfer.httpclient4.HttpClientFileSystemBrowser.runRequest(HttpClientFileSystemBrowser.java:259)
at org.eclipse.ecf.provider.filetransfer.browse.AbstractFileSystemBrowser$DirectoryJob.run(AbstractFileSystemBrowser.java:69)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)
Total time: 1 second
Installing p2 bootstrap 4.5.2... Success.
FAILURE: Build failed with an exception.

Where:
Build file '/Users/deandre/workspaces/gradleAndEclipseRCP/build.gradle' line: 49

What went wrong:
A problem occurred evaluating root project 'gradleAndEclipseRCP'.

A problem occurred configuring project ':target.p2'.
com.diffplug.gradle.SerializableMisc$ThrowableCopy: java.lang.reflect.InvocationTargetException: null

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 3 mins 15.314 secs

We can fix this!  Goomph works by downloading a minimal (~10MB) eclipse installation, and then using that installation to run headless applications that do the work.  We need to read your proxy settings from GRADLE_OPTS, and then pass them to these headless apps.
Here is the bug in Eclipse bugtracker for proxy support in headless apps, summarized below:
Make a file proxies.ini with content like this:
org.eclipse.core.net/proxyData/HTTP/host=someproxy.ericsson.se
org.eclipse.core.net/proxyData/HTTPS/host=someproxy.ericsson.se
org.eclipse.core.net/proxyData/HTTPS/hasAuth=false
org.eclipse.core.net/proxyData/HTTP/port=8080
org.eclipse.core.net/proxyData/HTTPS/port=8080
org.eclipse.core.net/org.eclipse.core.net.hasMigrated=true
org.eclipse.core.net/nonProxiedHosts=*.ericsson.com|<IP_ADDRESS>
org.eclipse.core.net/systemProxiesEnabled=false
org.eclipse.core.net/proxyData/HTTP/hasAuth=false

And pass it to command line with -plugincustomization proxies.ini.
The problem is that this mechanism doesn't work for proxies that need a password field. There is another mechanism (mentioned in the eclipse bug), but it's a little tricky.
Can you look up these things for me:

What GRADLE_OPTS parameters do you set
What would your proxies.ini file look like

Dummy values would be fine, of course, but I need a testcase for the parsing logic.  If you can supply 1 & 2 above, I can do the rest.  If you proxy ends up needing a password, we're going to have a second can of worms...

We need to specify proxy username & password......
On Friday, July 29, 2016, Ned Twigg<EMAIL_ADDRESS>wrote:

We can fix this! Goomph works by downloading a minimal (~10MB) eclipse
installation, and then using that installation to run headless applications
that do the work. We need to read your proxy settings from GRADLE_OPTS, and
then pass them to these headless apps.
Here https://bugs.eclipse.org/bugs/show_bug.cgi?id=382875 is the bug in
Eclipse bugtracker for proxy support in headless apps, summarized below:
Make a file proxies.ini with content like this:
org.eclipse.core.net/proxyData/HTTP/host=someproxy.ericsson.seorg.eclipse.core.net/proxyData/HTTPS/host=someproxy.ericsson.seorg.eclipse.core.net/proxyData/HTTPS/hasAuth=falseorg.eclipse.core.net/proxyData/HTTP/port=8080org.eclipse.core.net/proxyData/HTTPS/port=8080org.eclipse.core.net/org.eclipse.core.net.hasMigrated=trueorg.eclipse.core.net/nonProxiedHosts=*.ericsson.com|<IP_ADDRESS> http://org.eclipse.core.net/nonProxiedHosts=*.ericsson.com|<IP_ADDRESS>org.eclipse.core.net/systemProxiesEnabled=falseorg.eclipse.core.net/proxyData/HTTP/hasAuth=false
And pass it to command line with -plugincustomization proxies.ini.
The problem is that this mechanism doesn't work for proxies that need a
password field. There is another mechanism (mentioned in the eclipse bug),
but it's a little tricky.
Can you look up these things for me:

What GRADLE_OPTS parameters do you set
What would your proxies.ini file look like

Dummy values would be fine, of course, but I need a testcase for the
parsing logic. If you can supply 1 & 2 above, I can do the rest. If you
proxy ends up needing a password, we're going to have a second can of
worms...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/diffplug/gradle_and_eclipse_rcp/issues/5#issuecomment-236219917,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWLtyp7M6d-Rqgx1WFjibccKdJZ2o5Pks5qaiNFgaJpZM4JYIQQ
.

--
Sent from my iPad

Gotcha.  We've got one workaround we can do right now.  You'd have to open an eclipse GUI, put in your password, and then save it.  Then the headless apps would be able to read from it.  I've pinged the eclipse team to see if there's any update on a headless solution so you wouldn't have to open the GUI.  Regardless of whether we need the GUI workaround or not, 1&2 are the things I'll need to implement the rest.

Sounds good. Give me until sometime next week as I'm heading out on
vacation for a few days. I'll get something back over to you once I get
back.
On Friday, July 29, 2016, Ned Twigg<EMAIL_ADDRESS>wrote:

Gotcha. We've got one workaround we can do right now. You'd have to open
an eclipse GUI, put in your password, and then save it. Then the headless
apps would be able to read from it. I've pinged the eclipse team to see if
there's any update on a headless solution so you wouldn't have to open the
GUI. Regardless of whether we need the GUI workaround or not, 1&2 are the
things I'll need to implement the rest.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/diffplug/gradle_and_eclipse_rcp/issues/5#issuecomment-236222271,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWLtyxIIxp3gLuJzO3OY8aOH35WJ-WJks5qaiVtgaJpZM4JYIQQ
.

--
Sent from my iPad

Enjoy vacation.  Hopefully by the time you get back we'll have a way to avoid the GUI workaround.

Let's please continue discussion on actual Goomph #3 rather than here on its example project.


Update MapLibre GL JS Shaders after the TypeScript Migration

MapLibre GL Native is using the shaders from MapLibre GL JS before the TypeScript migration. I am worried about changing the shaders in GL JS if we cannot test the consequences in native.
@birkskyum created a branch to use the shaders after the TypeScript migration: https://github.com/maplibre/maplibre-gl-native/tree/generate-shaders-using-typescript
Since we forked GL JS, there have been only a hand full of updates to the shaders, see https://github.com/maplibre/maplibre-gl-js/commits/main/src/shaders for a complete list. Notable changes were:

https://github.com/maplibre/maplibre-gl-js/pull/209
https://github.com/maplibre/maplibre-gl-js/pull/416
https://github.com/maplibre/maplibre-gl-js/pull/928
https://github.com/maplibre/maplibre-gl-js/pull/1022

@louwers should this be closed?

@wipfli Yes we have copied them over and they are being refactored to take advantage of OpenGL ES 3.0.
I think WebGL 2 has some of the same functionality of OpenGL ES 3.0, but GL JS does not use WebGL 2 yet. In any case, keeping them in sync will need to happen manually going forward.

@louwers , it could very soon allow for some webgl2 though, so I have a hope that we might be able to copy some of these changes to the shaders back to GL JS.
https://github.com/maplibre/maplibre-gl-js/pull/2497
https://github.com/maplibre/maplibre-gl-js/pull/1891


Critical (and mindblowing) performance bug with empty-source layers

Describe the bug
Windows: 10
Ram: 16Gb
CPU: i7
Browser: Chrome 73
When we have layers using an empty source (no features!), performance and FPS drop in an unexpected and dramatic way.
We're measuring both ZOOM and PAN actions,
Surprisingly, if we just add 1 simple (painted!) feature to every layer, PAN performance works as expected hitting 60fps.
Using that very same 1 feature on every layer, ZOOM interaction improves but not with the scale that PAN does.
To Reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/7m0434on76
Pan (works fine)
Zoom works but it's kinda clunky (with very few features, 1/layer)
Now remove the 'olFeature' from the source array inside the loop and....boom!
Pan works incredibly slow with a huge drain on FPS (with 0 features painted/registered)
Zoom works even worse than before

Expected behavior
Regardless the number of layers, an empty source should not affect or impact performance since it's not really painting anything.
Indeed, it's always expected to get worse performance when more features are painted.
We reviewed the call tree and the method IntermediateCanvas.drawImage was consuming (scripting) our resources.
This is a (very) critical bug.

Extended information:
Using<EMAIL_ADDRESS>same project with 0 features/layer works fine but adding 1/layer hits the bug of unexpected performance drain again.
So, I understand there's some work being done in some direction but still a weird bug on the table.
Thanks.

A map with 100 layers is not a good idea. v6 shortcuts rendering when there are no features on a layer's source, but as soon as there are features, a canvas for the whole map viewport is created.
It would be interesting to hear about your use case and why you need 100 layers.

I have exagerated the issue in the example but we're experiencing this issue with 12 active layers, 26 lineString features distributed in 3 of those 12 layers and the rest, 9 layers, empty. As you can imagine, that is (or should be) a quite lightweight scenario for OpenLayers to handle.
Please also read that, in your 6. 0.0b3 version, things appear to be changing but not fixing the issue by any means.
We have also spent quite a few hours trying to determine where the issue was and isolating/sampling the issue for you.
Last but not least, we're releasing a quite big product soon and would love this issue to be addressed (asap) since we do love OpenLayers but we also take a lot of care about performance and helping you, guys.
Please, let us know if we can help you in any way.
IMHO, this is a very big issue.

@aalbericio The drawImage call you noticed with v5 is gone with v6, but still a canvas for each layer is created. As a rule of thumb, you should not have more than 3 active layers on a map.
Since you are pointing out how critical OpenLayers is for your big product, and also mention how urgent this matter is to you, you might want consider getting professional support, which could in turn help improve both your application and OpenLayers. If you are interested, feel free to reach out to me by e-mail.

The issue with empty vector layers taking a lot of render time is fixed in v6.

@ahocevar 3 active layers is definetely a very limited number for our app (and I can bet for many others).
Anyway, I can ask my company how we can work on this issue (and in the future) with you. We will be happy to collaborate in present and future features but that is something I have to scale up and may take some time to unlock.
Wether this collaboration takes shape or not, I still feel this is a big issue for the product and probably time to refactor some internals.
I will reach you by email next week.
Thanks for your time and effort on this great lib.

Glad to see this bug reopened.
Anyway, if it can help, I managed to workaround this by defining a "emptySource$" Observable and then:
emptySource$.subscribe((empty: boolean) => layer.setVisible(!empty));
This single subscription improves performance (a lot).


Various Updates from Recent Studies

This is a large PR incoperating several new things which I think would be best in the main branch:

Ensemble script updates
cosmics changes for the recent CORSIKA runs
updates to digi and reco for the new trigger configuration
added in the proto fclless scripts, further development will be in MDC2024

Hi @sophiemiddleton,
You have proposed changes to files in these packages:

JobConfig
CampaignConfig
Scripts

which require these tests: build.
@Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main.
:hourglass: The following tests have been triggered for c8e8c38d4b98f1e1a9c87d944740205eac8f160b: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at c8e8c38d4b98f1e1a9c87d944740205eac8f160b.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged c8e8c38d4b98f1e1a9c87d944740205eac8f160b at 88e8f941ca877b46269ed1e1597c438d4cc73876

build (prof)
:white_check_mark:
Log file. Build time: 11 min 00 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at c8e8c38d4b98f1e1a9c87d944740205eac8f160b after being merged into the base branch at 88e8f941ca877b46269ed1e1597c438d4cc73876.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

@sophiemiddleton Can you recommend an appropriate reviewer for this PR?


IaaS DNS Forwarders

Description
Template and Readme to deploy IaaS DNS Forwarders.
Issue reference
The issue this PR will close: #745
Checklist
Please make sure you've completed the relevant tasks for this PR out of the following list:

[X] All acceptance criteria in the backlog item are met
[X] The documentation is updated to cover any new or changed features
[X] Manual tests have passed
[X] Relevant issues are linked to this PR

Thank you for pushing the proposed change!
This looks good and I was able to successfully test this!


simplified the default blog layout value

Have simplified the default blog layout so that now it only has to be updated in one location rather than two.

@benfes Thanks for making this easier!


helm chart

make a  helm chart.

Hello @chenzhenyang, we are currently exploring/testing the creation of a helm chart in coordination with other CF component teams.  Please know that this is a near-term goal of the project, but we're unable to provide a delivery date at this time.


Simplify encode_to_slice by using chunks_exact_mut

Rather than using generate_iter to iterate over each 2-element chunks
of output buffer, use chunks_exact_mut.  This removes the need for the
custom generate_iter function and makes the code shorter.
Criterian shows improvement but to be honest I don’t trust those
results (partially because also decoding showed improvements even
though I didn’t touch it):
hex_encode              time:   [79.268 µs 81.856 µs 84.279 µs]
                        change: [-15.070% -12.678% -10.255%] (p = 0.00 < 0.05)
                        Performance has improved.

rustc_hex_encode        time:   [36.926 µs 36.949 µs 36.971 µs]
                        change: [-3.3521% -3.2536% -3.1583%] (p = 0.00 < 0.05)
                        Performance has improved.

faster_hex_encode       time:   [820.07 ns 821.16 ns 822.52 ns]
                        change: [-3.7706% -3.4712% -3.1944%] (p = 0.00 < 0.05)
                        Performance has improved.

faster_hex_encode_fallback
                        time:   [11.554 µs 11.557 µs 11.562 µs]
                        change: [-2.8788% -2.8220% -2.7676%] (p = 0.00 < 0.05)
                        Performance has improved.

hex_decode              time:   [100.90 µs 100.98 µs 101.08 µs]
                        change: [-3.9865% -3.5655% -3.2365%] (p = 0.00 < 0.05)
                        Performance has improved.

rustc_hex_decode        time:   [65.759 µs 65.788 µs 65.819 µs]
                        change: [-2.1181% -1.9775% -1.8495%] (p = 0.00 < 0.05)
                        Performance has improved.

faster_hex_decode       time:   [2.2675 µs 2.2684 µs 2.2693 µs]
                        change: [-2.9682% -2.8188% -2.6234%] (p = 0.00 < 0.05)
                        Performance has improved.

faster_hex_decode_unchecked
                        time:   [928.35 ns 929.18 ns 930.05 ns]
                        change: [-0.5734% -0.4510% -0.3314%] (p = 0.00 < 0.05)
                        Change within noise threshold.

faster_hex_decode_fallback
                        time:   [11.416 µs 11.426 µs 11.438 µs]
                        change: [-3.8020% -3.6669% -3.5267%] (p = 0.00 < 0.05)
                        Performance has improved.

@KokaKiwi, friendly ping.


[Dynamic DAG] Add UI for dynamic DAG.

We allow user to replace a processor.
UI fields:

oldProcessorId
new Name
new Parallism.
new Task Class Name.

From the unittest, I see a way to add new processors (including edges). So if you want to modify a dag, you should provide the whole dag body (including all processors and all edges), right?

Here are the related definitions for scalajs. We probably want a way to manipulate the graph on both client and server - so a scalajs based graph that has no dependencies. It would be interesting to see if org.grapht.Graph can be converted to JS via scalajs. I have not done this exercise.
  case class StreamingAppMasterDataDetail(
      appId: Int,
      appName: String = null,
      actorPath: String = null,
      clock: String = null,
      executors: Map[ExecutorId, String] = null,
      tasks: Map[TaskId, ExecutorId] = null,
      processors: Map[ProcessorId, ProcessorDescription] = null,
      processorLevels: Map[ProcessorId, Int] = null,
      dag: Dag = null
  ) extends AppMasterDataDetail

  case class ProcessorDescription(id: ProcessorId, taskClass: String, parallelism : Int, description: String = "")
  case class Dag(vertices: Seq[Int], edges: Seq[(Int, String, Int)])

StreamingDag API is below


Ensure homebrew updates its internal list of formulas before installing

This PR is a different approach for for #760

I ran the CI 3 times, the correct (latest) ktlint was used every time.

Good stuff, I tried this too, but it took more than 6 minutes to complete. It must have been a flake, since this run completed in 1m.


adding grumphp, fixing style, narrowing autoload namespace

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master     #1   +/-   ##
=======================================
  Coverage       100%   100%
  Complexity       32     32
=======================================
  Files             2      2
  Lines            63     63
=======================================
  Hits             63     63

Impacted Files
Coverage Δ
Complexity Δ

src/Zend/View/Helper/Partial.php
100% <100%> (ø)
20 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Add fingerprint to compiled css/js files

Case we have CDN front of our website, So main.css can not update automaticly.
Now, the css tag is:
<link rel="stylesheet" href="/css/main.css">

I think it should add fingerprint to static files.
Something like this:
<link rel="stylesheet" href="/css/main.build_hash.css">

Hi @tooooolong! How would one know whether a fingerprint should be used? Some sort of configuration option?

maybe something like this。
https://github.com/webpack/webpack/issues/86#issuecomment-179957661
this is a plugin for Webpack, it will add a hash to the static file name automatically.

This is a good idea 👍

is anyone taking this @yangshun ? I can work on thing in the coming weeks

Sure thing!

Hi, @yangshun can you check my code? https://github.com/facebook/Docusaurus/pull/694

Implemented in v2


SDC 8553 Allow Time EL for 'Resource URL' in HTTP processor

This commit is for the improvement feature requested on https://issues.streamsets.com/browse/SDC-8553.

Hi, @jarcec thanks for the update. I looked into the code and there isn't any tests for HttpClientCommon.java. Adding it should be alright, but was wondering, about should I mock resourceEval variable? because its initialization is done in another method.
Do you have any other unit test for the above scenario which I can take in as a reference?
Regards,
Nagaraj

I believe that tests for the HTTP Processor are written here @tan31989
https://github.com/streamsets/datacollector/blob/master/basic-lib/src/test/java/com/streamsets/pipeline/stage/processor/http/HttpProcessorIT.java

@jarcec These tests are for Integration Tests and @dimaspivak mentioned in the slack channel support that:

We actually are in the process of transitioning our integration and end-to-end functional tests out of the Java unit test harness and into a Python one that we built on top of our StreamSets SDK

Thus after I wrote it, I could not run that test. Running the above tests was not something I know off. You think I can test that individually?
Regards,
Nagaraj

At this point I will ask you to enhance the HttpProcessorIT, we will transition those tests to the engine that @dimaspivak  mentioned later @tan31989 .

@jarcec Yes sure, I have done that and can do that. Could you let me know how I can execute one that file? Any documentation which I can look up?

All test files ending with "IT" are executed through maven failsafe plugin @tan31989 . I believe that you can instruct the plugin to run only given test with -Dit.test="HttpProcessorIT"

@jarcec Have added a Integration test in there. Let me know if that suffices for the use case here.

Have you signed our contributor agreement @tan31989 ? I'm having troubles to locate your CLA on our side.
https://streamsets.com/contributing/

I can confirm that Media iQ have signed the CCLA, @jarcec

Thanks for the validation @metadaddy !
Running the new test on our side seems to fail with:
org.junit.ComparisonFailure: expected:<2018010[2]> but was:<2018010[1]>
	at org.junit.Assert.assertEquals(Assert.java:115)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at com.streamsets.pipeline.stage.processor.http.HttpProcessorIT.testHttpGetTimeEl(HttpProcessorIT.java:1039)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
	at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
	at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
	at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
	at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
	at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
	at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:367)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:274)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:238)
	at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:161)
	at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:290)
	at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:242)
	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:121)

Almost seem like some sort of a timezone issue perhaps @tan31989 ?

Daylight savings time???

@jarcec have fixed the test case which was missing a timezone. Now have verified this with other timezone, it works.

Thanks, @tan31989 , let me run the updated patch through our CI system and merge it if it succeeds. Do you have an account on our JIRA by any chance? I would like to assign the JIRA to you so that you get recognition for contributing the fix :)

Thanks @jarcec , I'm on Jira and have commented on that issue: https://issues.streamsets.com/browse/SDC-8553
Happy to help with this.

Thanks @tan31989 , I've assigned the JIRA to you then.

Merged, thank you very much for your contribution @tan31989 !


Create Sprite for Stun Status Indicator

Please see CONTRIBUTING.md first.

Have a PR up related to this: https://github.com/doficia/project-cordon-sprites/pull/83


createSvgSpritePlugin is not a function

Hi!
I'm trying to use this plugin in a Vue 3 TS + Vite 3 project, but it appears to be failing at the initial step. npm run dev fails with the above mentioned error. Here's a demo: https://stackblitz.com/edit/vitejs-vite-du7xik

Fixed on<EMAIL_ADDRESS>

Seems to work! Thanks a lot!


Does not work

It is also not working for me.. weird and sad :(

I tried to perform scale animation to the list of texts.
        hTextView = findViewById(R.id.textViewAnimate);         hTextView.setAnimationListener(this);         hTextView.animateText(animateWords[animateWordIndex]);
And in the listener, On Animation End Listener, I tried to move to next in the list of the words.
    @Override     public void onAnimationEnd(HTextView hTextView) {         animateWordIndex++;         if(animateWordIndex == animateWords.length)         {             animateWordIndex = 0;         }         hTextView.animateText(animateWords[animateWordIndex]);     }
But I always get the last word simply displayed in the TextView without any animation.


Busy is showing only one post repeatedly in feed and discussions

Expected behavior
When I go to busy.org, I should see the all the different posts from my feed and my blogs.
Actual behavior
When I go to busy.org, I only see one post repeated multiple times and scrolling to the end retrieves and shows multiple posts.
How to reproduce

In a new tab, open busy.org and see the posts on your feed.
Login to busy.org, and check discussions.

Browser: Chrome Version 69.0.3497.100 (Official Build) (64-bit)
Operating system: OSX 10.11.6

I found the reason of the bug and more bugs here: https://github.com/busyorg/busy/issues/2133


Configure context menu delay?

I did some brief googling and I don't think this is configurable as of right now, but what I'd like is for the which key context menu to have a brief delay before opening just like the default spacemacs configuration for example. I checked the documentation for vscode-which-key and didn't find any configuration settings in this regard. I also checked the vscode documentation and didn't find any configurability there either. Maybe it's not possible just yet, but it would be nice to not have this thing flashing every time I enter a key chord.

https://vspacecode.github.io/docs/whichkey/extra#display-menu-with-a-delay ?

Closing this as the delay settings seems to have addressed this. Thanks Marco for answering.


Config.py updated (ticket #72)

Added in three lines of code as provided by the professor in Slack to connect app to database in cloud

Codecov Report

Merging #88 (6ecedf3) into main (57b365c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #88   +/-   ##
=======================================
  Coverage   95.71%   95.71%
=======================================
  Files           5        5
  Lines         257      257
=======================================
  Hits          246      246
  Misses         11       11

Continue to review full report at Codecov.

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

Thank you!!!


Coin drop with ring of wealth equip causes loot tracker items to dissappear

The items appear again on next item drop.

@abex probably missing one of the repaints you took out? I can't check today

So you killed abby demons, then kurasks, ans when kurasks dropped coins that happened? And right when the next drop came along it fixed itself? Is this correct? I need to recreate the context

So you killed abby demons, then kurasks, and when kurasks dropped coins that happened? And right when the next drop came along it fixed itself? Is this correct? I need to recreate the context
Also, do you have any ignored items? You can check this in the loot tracker configa

There were other monsters in the list before. My ignores are "Bones,Ashes". I noticed that it happen without ring of wealth sometimes. The coins drop does not show up in the loot tracker. The only mob that this happened to me was kurasks.


[AutoPR sql/resource-manager] Improve documentation of sql database & elastic pool skus

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5756

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/2528


few sections above and below the "faq" section  are missing

Steps to reproduce:

check sections above faq section and below it
compare with figma

Actual result: few sections are missing
Expected result: please check if any of those should be there

Reported by: Vinod<EMAIL_ADDRESS>Source URL: https://wf-b.stg.prezzee.com/en-us
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size1366 x 768OSWindows 10 (10.0.0)Viewport Size1366 x 641Pixel Ratio@1x

Hidden for Phase 1

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


epinio-binary grown to 100Mi

RKE2-EC2-CI job started to fail because temporary PV for devel epinio-service binary (used by make patch-epinio-deployment) doesn't fit the binary anymore:
  pod/epinio-copier condition met
  Copying the binary on the PVC
  mv: write error: No space left on device
  command terminated with exit code 1

Its PVC was requesting 100Mi originally, now changed to 150Mi.
A bit strange is that it doesn't fail on other public clouds, most probably the used Longhorn storage provider is more strict or using a different block-size.
Testrun:
https://github.com/epinio/epinio/actions/runs/6971231046

Wow. For real? That's a lot.
Maybe we should see if we can shrink it a bit. Not for the server, but it's a bit annoying that the users just using the CLI needs to download a ~100Mb binary.

I'd say this is Longhorn specific as the binary can fit when other storage than Longhorn is used - my binary is also 90M only. We are requesting 100Mi PV on every cloud we are testing on.

See LH example about Volumes, point 1, bullet 2 - https://longhorn.io/docs/1.4.1/volumes-and-nodes/volume-size/#example

There is some meta info in the volume (12Gi)  hence the actual size is 260 Mi and is not exactly 0.


Add bad argument list to user error when possible

Addresses https://github.com/docopt/docopt.go/issues/37
This allows Parse callers to emit the list of unexpected arguments in some
common cases, e.g., extra arguments, out-of-order arguments.

Dead.


I don't know anything, OH WAIT YES I DO (.rem)

Muta_work
2:42 Yep
2:42 .rem my butts is the best butts
Hayt
2:42 I don't know anything about rem my butts.
2:42 OK, I'll remember my butts.

This is not possible currently because you import it externally. Does coffeescript support monkey patching?


Option to disable energy safe

You implemented this: https://github.com/tooll3/t3/commit/181ab4669bd62846a76fc0aca72e9efb5e6ef3a1
It would be usefull to keep the application running if i minimize.

Duplicate of #304


[Feature] Lan activity

Could you add LAN data in and out? This would be very useful.
Also Disk capacity info: amount used, amount available. etc.
Thanks

Network activity does not seem to be available using API.

got it confirmed, web interface just uses embedded rrd graphs

Reopened as it was implemented in upcoming OMV 6.0.18-1
https://github.com/openmediavault/openmediavault/issues/1247

So, i'm new to the language, but have found the relevant API code in the OMV repository:
https://github.com/openmediavault/openmediavault/blob/master/deb/openmediavault/usr/share/openmediavault/engined/rpc/network.inc
RPC name:

Network
The object is return as following:
$obj['type'] = $iface->getType();
$obj['description'] = $iface->getDescription();
$obj['stats'] = !$stats ? [] : $stats;
$obj['gateway'] = !$gateway ? "" : $gateway;
$obj['gateway6'] = !$gateway6 ? "" : $gateway6;
$obj['address'] = !$address ? "" : $address;
$obj['address6'] = !$address6 ? "" : $address6;
$obj['prefix'] = !$prefix ? -1 : $prefix;
$obj['prefix6'] = !$prefix6 ? -1 : $prefix6;
$obj['netmask'] = !$netmask ? "" : $netmask;
$obj['netmask6'] = !$netmask6 ? -1 : $netmask6;
$obj['ether'] = !$ether ? "" : $ether;
$obj['mtu'] = $iface->getMTU();
$obj['state'] = !$state ? gettext("UNKNOWN") : $state;
$obj['link'] = $iface->getLink();
$obj['speed'] = !$speed ? -1 : $speed;


Use active model to generate response in example on README (#269)

This PR is a simple doc change to the example in the README. The active model is used to generate the response in place of the reference model. This change accurately reflects the workflow highlighted in the diagram before the example.
Addresses issue #269
Fixes #269

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Create design results in duplicate designs sometimes

When i edit designs it sometimes happens he create a additional with the same name if you delate 1 both are gone,

The logs only show one create call for this design, so I assume this is a UI bug of some sort. Will investigate.


Support for Cloud Services

Add support for Cloud Services

Done with latest version. Dependency on Web API is removed.


TypeError after update.

Using Automatic1111 version: 1.6.1 and python: 3.10.6.
I opened the Extensions and checked for updates. Lora Queue Helper was the only update. I applied and restarted.
When I tried to use the script I get:
TypeError: Script.run() missing 9 required positional arguments: 'is_use_custom_path', 'custom_path', 'directories', 'selected_loras', 'checkbox_iterate', 'checkbox_iterate_batch', 'is_save_grid', 'is_auto_row_number', and 'row_number'
I used the Backup/Restore to roll back the change to commit be67ef53
And now get a slightly different error, but the same issue:
TypeError: Script.run() missing 7 required positional arguments: 'is_use_custom_path', 'custom_path', 'directories', 'selected_loras', 'checkbox_iterate', 'checkbox_iterate_batch', and 'is_save_grid'
I tried to manually remove the folder and update manually, but get the same issue regardless.

Thanks for reporting. I tried to check out A1111 to your specified commit, but it seems not  exist on the master branch of A1111.
The A1111 commit I am using is the current master, and it works fine on my local environment.
Based on the error log you attached, it shows that all the parameters are missing, which is nearly impossible under normal circumstances. But gradio did a poor job with error traceback, so I can't say it's impossible.
But I need more information to fix your issue.
Could you go to the A1111 directory and check git status? If it's normal, then go to {A1111 directory}/extensions/sd-webui-Lora-queue-helper and also check git status there. You can post them here if you can't decide if it's correct.
And maybe attach more of the error log, the whole log is better, as there could be some useful information at the very beginning of the error log.

And more version information is helpful, you can copy it from the bottom of your a1111 webui. Here is mine:
version: v1.6.1-3-ga254ce80  •  python: 3.11.5  •  torch: 2.0.1+cu118  •  xformers: N/A  •  gradio: 3.41.2  •  checkpoint: c3056f6175
The difference on python version should not cause this issue, I am thinking of gradio version issue.

version: 1.6.1  •  python: 3.10.6  •  torch: 2.0.0+cpu  •  xformers: N/A  •  gradio: 3.41.2  •  checkpoint: 0f1b80cfe8
To be clear I am running Stable Diffusion web UI with DirectML
Have an AMD card

stable-diffusion-webui-directml>git status
Refresh index: 100% (323/323), done.
On branch master
Your branch is up to date with 'origin/master'.

stable-diffusion-webui-directml\extensions\sd-webui-Lora-queue-helper>git status
On branch main
Your branch is up to date with 'origin/main'.

nothing to commit, working tree clean

Interestingly the script was working perfectly earlier today.
After the update I got the error and there seems to be no going back.
I have plenty of space, so I'm going to try a fresh install and will check back in with an update.

New install and it works fine.
I've saved detailed steps on how to do my setup so in 12 minutes it was back up and running.
The installed version that encountered the issue exists and can be run to see if we can correct the issue is desired.
Otherwise we can close this issue and chalk it up to localization issues.

So you set up a new stable-diffusion-webui-directml environment and it works now?
At least you can use SD without errors.
That might be an issue specific to directml version. It could be due to some random reason, like a bug in the required packages at a specific version causing that problem.
I don't have that environment to test, so I think I will leave this issue open for some time and see if there are other people  who encounter this problem. If not, we can just forget it. ;)


chore: simplify tsconfigs

Closes #209
This PR does two related things:

fixes the webpack typings issues which lead to excluding node_modules and using skipLibCheck flag
simplify the tsconfig.json setup with now having only one config which works for all our needed cases

I have tested the whole build pipeline and documentation generation and it works as expected.

FYI if you will try out the PR then don't forget to run npm ci! Otherwise you might get some compilation problems...


[Umbrella] Kyuubi Spark lineage extension supports Spark 3.4

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the proposal
Make Kyuubi Spark lineage extension supports Spark 3.4.
Task list

[x] #4925
[x] #5170

Are you willing to submit PR?

[x] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
[ ] No. I cannot submit a PR at this time.

@iodone is this completed?

@iodone is this completed?

In development.

Thanks for your great work! @iodone


Optimize index.html

While working on angular.io, I noticed that the site includes a number of both HTML and inline JS comments. That led me to note that the HTML and inline JS includes whitespace. In the grand scheme of things this is pretty minor, but it would be nice if the CLI could apply a few basic, safe optimizations for index.html:

Remove comments (probably configurable for HTML comments which can theoretically be load-bearing)
Remove non-significant whitespace (e.g. any whitespace outside of <body>)
Optimize inline JS (needs some due diligence to make sure this is safe) and CSS (much safer)

Duplicate of https://github.com/angular/angular-cli/issues/11360


COMPAT: Date format for object copy and copy part

Fixes #372

@ironman-machine work it

Hello @LaurenSpiegel
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "fix/copyDateFormat", "DEFAULT_BRANCH": "rel/6.2.1"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6151 for CI status.

end to end passed

@ironman-machine check it

Hello @LaurenSpiegel
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "fix/copyDateFormat", "DEFAULT_BRANCH": "rel/6.2.1"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6190 for CI status.

Thanks, all good now :)

@ironman-machine morning

Hello @LaurenSpiegel
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "fix/copyDateFormat", "DEFAULT_BRANCH": "rel/6.2.1"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6207 for CI status.

end to end passed: http://ci.ironmann.io/gh/scality/Integration/6207


Evnt 440/fix main dashboard

Preview:

Also please squash those two commits. :pray:

The zeroes are not handled when there are no events. See screenshot

Fixed:

LGTM!
Zeros are being handled and dashboard looks good


Never close an RTCDataChannel

This branch moves to managing DataChannels as a reusable pool.  Once a channel is open, it stays open, and may be reused for multiple connections over time.  The pool grows as needed to serve all simultaneously active connections.

OK, great. I beat this up for a while in the test lab and saw almost no strangeness.
The commit history is getting a little messy so I suggest we:

iterate on this branch today and Monday until we're happy
bundle up our changes into a single commit (or rebase) and publish a new NPM late Monday

ne important thing for now: I seem to be losing data at the end of requests. Just try this command with Simple SOCKS:
curl -x socks5h://localhost:9999 http://www.armoryonpark.org/css/screen.css

Can you investigate why this might be happening? The bright side is no peerconnection breakages :-)

Comments from the review on Reviewable.io

src/pool/pool.ts, line 215 [r4] (raw file):
Since we're light on tests, how about adding some warnings when an unexpected control message is received, e.g. CLOSE_ACK when we haven't attempted to close the channel? It looks like we are already storing most of the state we need.

src/pool/pool.ts, line 246 [r4] (raw file):
Suggestion: I think we could remove the need for this "wait" by requiring onceOpenedalready be fulfilled on channels passed to new PoolChannel instances. Ditto for close. Should make these methods significantly shorter.

Comments from the review on Reviewable.io

Thanks for pointing out armoryonpark!  I fixed that issue in d96d720.

src/pool/pool.ts, line 215 [r4] (raw file):
Done.

src/pool/pool.ts, line 246 [r4] (raw file):
Done.

Comments from the review on Reviewable.io

Great! Let's keep iterating on this branch and hopefully by Monday afternoon we'll be ready to merge and publish.

Comments from the review on Reviewable.io

Very cool! Not too complicated at all.
Remember to update the package: this will be a major version bump because of wire protocol changes. (if only we could do something even bigger than major bump...)
I notice that the SOCKS server unit tests are failing.
Would be nice to have some tests for this also.

Reviewed files:

Gruntfile.coffee @ r1
src/churn-pipe/freedom-module.json @ r1
src/pool/pool.ts @ r6
src/rtc-to-net/rtc-to-net.ts @ r2
src/samples/copypaste-socks-chromeapp/freedom-module.ts @ r2
src/simple-socks/freedom-module.ts @ r2

src/pool/pool.ts, line 2 [r7] (raw file):
Worth saying a little about the assumptions :)
e.g. that if openning from both sides is supported, but assumes the underlying pc_ implementation will deal with naming them distinctly. (i.e. doesn't work for negotiated channel ids).

src/pool/pool.ts, line 16 [r7] (raw file):
If this is the only thing exported, consider using a export = Pool; at the end of the file, and then it can be imported directly as a class without having extra name-space stuff.

src/pool/pool.ts, line 23 [r7] (raw file):
Could you use Object here instead of any... you want any when you want to enable arbitrary type conversions, you want Object when you don't care about the details of the type, and you want parameterization when you care about the type and want to pass that information on.
Anyway, I think pc can be private here.

src/pool/pool.ts, line 37 [r7] (raw file):
I'm not sure that LocalPool and RemotePool are doing much fore us, have you thought about just putting them into Pool?

src/pool/pool.ts, line 146 [r7] (raw file):
You should consider explicit state rather than two booleans which allows for some 'bad' impossible states. (and good practice to note the impossible states, and possible transitions)

src/pool/pool.ts, line 207 [r7] (raw file):
I think you probably want a try/catch block around the parse.

src/pool/pool.ts, line 213 [r7] (raw file):
Why throw an error but not log it? I think we should always log errors else a user can't report them.

src/pool/pool.ts, line 220 [r7] (raw file):
I find the translation of names from "DataFromPeer" to "DataForClient" a little confusing. It's not obvious to me what the client is. Given this is a one-line function, used only in two places above, I think you will be better inlining it above and avoiding introducing new terminology.

src/pool/pool.ts, line 224 [r7] (raw file):
an enum will be (very slightly) more efficient and type safe. :)

src/pool/pool.ts, line 242 [r7] (raw file):
It's probably impossible, so doesn't matter, but I think I would expect us to close if we got a close_ack... why return here?

src/pool/pool.ts, line 280 [r7] (raw file):
do you want a warning here?
Also: if you are already closing, you don't need to send close again, right?

src/pool/pool.ts, line 290 [r7] (raw file):
Can we make this more concise? maybe "PoolChannel(" + this.dc_.toString(); + ")"

src/samples/copypaste-socks-chromeapp/freedom-module.ts, line 21 [r7] (raw file):
These lines were here as a demo for how to use the setFilters. I think they are best left in.  Maybe with a comment to explain why they are here.

src/simple-socks/freedom-module.ts, line 23 [r7] (raw file):
These lines are in the sample apps as demos for how to set the filter.

src/socks-to-rtc/socks-to-rtc.ts, line 234 [r7] (raw file):
Lets be consistent with our log message naming here: either use session, or SOCKS client as below.

Comments from the review on Reviewable.io

src/pool/pool.ts, line 155 [r7] (raw file):
This sort of method makes me nervous, mostly because state generally makes me nervous. It's worth thinking about making this channel wrapper a "one-time" wrapper, i.e. once the wrapper has "closed", it cannot be used again.
However, we can make that a code change at some future point. Don't want to hold this up.

src/pool/pool.ts, line 213 [r7] (raw file):
+1, since in this case this is the "top-level" caller.

Comments from the review on Reviewable.io

I created an issue for this:
https://github.com/uProxy/uproxy/issues/1309

src/pool/pool.ts, line 2 [r7] (raw file):
Done.

src/pool/pool.ts, line 16 [r7] (raw file):
Done.

src/pool/pool.ts, line 23 [r7] (raw file):

Could you use Object here instead of any
Done.

Anyway, I think pc can be private here.
It's better than private: it's local.

src/pool/pool.ts, line 55 [r7] (raw file):
I think it's helpful for comprehensibility to keep the two state machines in separate classes, rather in than in various attributes and methods of a single class.  Actually, I was tempted to get rid of Pool.

src/pool/pool.ts, line 146 [r7] (raw file):
Done.

src/pool/pool.ts, line 184 [r7] (raw file):
I thought about this, but then who listens for the "open" message?  I couldn't figure out how to make it work.

src/pool/pool.ts, line 207 [r7] (raw file):
Done.

src/pool/pool.ts, line 213 [r7] (raw file):
Done.

src/pool/pool.ts, line 220 [r7] (raw file):
Done.

src/pool/pool.ts, line 255 [r7] (raw file):
Done, sort of.  I used the Typescript's enum-introspection to get as close as I could to a string-enum behavior.

src/pool/pool.ts, line 277 [r7] (raw file):
I wrote out a long explanation of why this was here ... and then decided that it would be better to just get rid of close_ack entirely instead.  Now, every virtual channel is closed with exactly one "close" message in each direction.

src/pool/pool.ts, line 280 [r7] (raw file):
close() is idempotent in the PeerConnection interface, so I don't think we need a warning.

src/pool/pool.ts, line 290 [r7] (raw file):
Done.

src/samples/copypaste-socks-chromeapp/freedom-module.ts, line 21 [r7] (raw file):
Done.

src/simple-socks/freedom-module.ts, line 23 [r7] (raw file):
Done.

src/socks-to-rtc/socks-to-rtc.ts, line 234 [r7] (raw file):
Done.

Comments from the review on Reviewable.io

src/pool/pool.ts, line 254 [r7] (raw file):
Now we're typesafe but sending strings. We could do what the SOCKS proxy does, which is send and receive integers over the wire and defer introspecting the enum to a logging statement.

Comments from the review on Reviewable.io

src/pool/pool.ts, line 254 [r7] (raw file):
It looks like we're OK sticking with strings for now.

Comments from the review on Reviewable.io

Reviewed files:

package.json @ r8
src/pool/pool.ts @ r6, r9
src/rtc-to-net/rtc-to-net.ts @ r8, r11
src/samples/copypaste-socks-chromeapp/freedom-module.ts @ r10
src/simple-socks/freedom-module.ts @ r10
src/socks-to-rtc/socks-to-rtc.spec.ts @ r8
src/socks-to-rtc/socks-to-rtc.ts @ r10, r13

src/pool/pool.ts, line 273 [r7] (raw file):
good!

src/pool/pool.ts, line 313 [r7] (raw file):
But you would probably like to know if you call it twice, no? That would be worrying, I think. :)

Comments from the review on Reviewable.io

LGTM

src/pool/pool.ts, line 313 [r7] (raw file):
(I'm happy either way)

Comments from the review on Reviewable.io

:+1:


Dev: Add docs for complex charts

Summary

What problem (or opportunity) are we solving? Why are we solving it? Is there supporting user conversations or research?

This page's content is not available in Carbon Platform yet:
https://carbondesignsystem.com/data-visualization/complex-charts
We can't render remote MDX because that file imports demo data:
https://raw.githubusercontent.com/carbon-design-system/carbon-website/main/src/pages/data-visualization/complex-charts/index.mdx
We can't build this page locally, because the platform does not support the ChartDemoGroup MDX component, per:
https://github.com/carbon-design-system/carbon-platform/issues/617
In auditing content parity for v1, @jeanservaas recommended that we ultimately need to move this content out of the collection's page, to usage docs per asset. See:
https://github.com/carbon-design-system/carbon-platform/issues/907
v1 will ship without the Complex charts content. This issue is to migrate that content into usage docs per indexed asset.
Job stories

What value will this deliver?

When displaying complex data sets, I want to reference the complex charts documentation, so I can use the correct assets.

Acceptance criteria

What requirements must be met to complete this request?

All https://carbondesignsystem.com/data-visualization/complex-charts content and demos are available in Carbon Charts library's asset detail pages.

@jeanservaas here's the issue if you want to take https://carbondesignsystem.com/data-visualization/complex-charts content and split it up by asset in Box or Figma.

@alisonjoseph confirming that this is the dev issue for the design issue stated as a blocker. For now, I'm moving this to the inbox because it's not ready for estimates until the design changes are specified in the design issue.


feat: odrder slices so that they are fetched according to their id order

fix ZEN-139

@cabreraalex this fetches slices in the correct order but somehow the VegaLite component does not re-render even though the data changes.


Use splinter in separate package

a bit cyclic here... not sure what to do about that.
This PR requires the splinter package from https://github.com/osrf/buoy_sim/pull/87 but buoy_sim: 87 also requires this PR
I suppose I'll move splinter here...

just stick to MPL-2.0.

That's what I was thinking... it doesn't look like ament_copyright includes those in --list-licenses

TL;DR The wrapper still didn't hide the bundled eigen, but I fixed it!
The bundled eigen was 3.2.8 and the latest on Ubuntu 20.04 is 3.3.7 and was causing a pesky and hard to find std::badalloc when the two were mixing in spite of my efforts to obfuscate the bundled eigen. I was able to get splinter to use the system eigen after a small fix to one line of their code using sed during the build.

so, CI is failing because of ament_copyright in splinter. To get it to work on my machine, the workspace needs to be sourced to pick up mpl2_ament_copyright after building and before testing... any ideas for CI?


Add ElegantEmojiPicker.podspec.

To make it possible for this lib being able to be used via CocoaPods.

Got below errors while i try to validate the podspec via:
pod spec lint ElegantEmojiPicker.podspec --allow-warnings --verbose

Could you suggest how to proceed with? @Finalet
I understand that the source code works well with Swift Package Manager, however, to make it support CocoaPods as well, seems we should apply some adjustment to it.

Seems version is required for podspec. Anyway, i have updated the podspec as you suggested.
And handover the remaining parts to you to make it finally work.

My understanding is we can specify one init version, 1.0.0, or 0.0.1, etc., to satisfy CocoaPods, then, leave the chance for user to use the lib via branch as you suggested, so that no need to add/maintain further versions/tags.
Formally publish the podspec with pod trunk push command. Because you are the owner of this lib, so, the honor to do that belongs to you.
(Pre-condition: setup an account on Cocoapods.org and register session using pod trunk register.)

Please feel free to let me know if you want me to take any further actions on that.

After multiple failed attempts to install cocoapods (some issues with the ruby manager on my mac), I wondered the rationality of what we are doing.
SPM can be used together with CocoaPods in the same project (like here), and it's a more modern, convenient package manager. So what are the benefits of maintaining CocoaPods separately?
Thank you for your help and effort! But I don't really understand why, @loongman.

Flexibility.
By supporting CocoaPods, we do not force users who already use CocoaPods in them project to switch to SPM + CocoaPods, even though SPM + CocoaPods might work well together. Actually, in my case, there is conflict while i try to use this lib by introducing SPM + CocoaPods into my project.

Increasing the possibilities for more users to use this lib.
As you mentioned on Readme, why is there no built in emoji picker in UIKit? This lib fills the gap well, it can help a good number of users and them projects for sure.

Personally, i have been searching for such a lib for quite a while by using pod search, it did not guide me to here due to this lib's lack of CocoaPods support.
Luckily and coincidentally, after a bundle of google search, finally, i got to here via https://iosexample.com/a-simple-and-powerful-emoji-picker-for-swift-built-with-uikit/
If we can add CocoaPods support to this lib, pod search can help much more users to reach to this lib. And the only thing we need to do is to add a podspec file and publish it the pod world. @Finalet


Remove Yandex and Yahoo

Emm, why?
Yandex is a very valid and respectable

Imho none of these offer enough differentiation compared to Google and Bing and I'd rather keep the list shorter. Do you use are planning to use Yandex or Yahoo? Happy to keep them if that's the case :)

No. Actually I use DDG which uses Yandex internally )))
I'm not sure about Yahoo but FYI the Yandex is widely used in Russia &
exUSSR space.
I understand your reasons so removing those is ok if it helps the app
overall.


build: package theming files into release output

This will be a breaking change for the next release.
I'm holding off on updating the theming guide until we're ready to release (since it updates on GitHub immediately).

I think after this PR, core/theming/prebuilt path need to be changed to prebuilt-themes in some of those files.


uuidPrefix should be able to change

in context.go file uuidPrefix should be able to change with the application.
var uuidPrefix = `uuid`

Resolved by https://github.com/pickme-go/traceable-context/pull/3#issue-345593432


User mentioning works with space

Hello,
What if we can use space in user mentioning?
What would be the policy?
Is there anyone who can help with that?

Facing same issue ...... is there any one who can help ??


VRC world does not work in proton

VRChat freezes completely when running through proton as soon as the speed is changed or something is loaded
Make the shader work when converted to vulkan smh

Look, I call myself a techno-masochist, but running this thing on Proton is too much even for me.
Try opening an issue on https://github.com/doitsujin/dxvk if you're serious ;)


Bug report: planner task get throws error when executed without --id and --title

Description
When executing planner task get an --id or --title option is required. However, it is possible to execute the command without them. This should not be possible, it should throw a validation error. But it doesn't. It throws a (to the user) unintelligible error.
Steps to reproduce
Execute m365 planner task get
or execute m365 planner task get --bucketId 'someid' --ownerGroupName 'Some Group'
Expected results
The command should throw some validation error like:
Error: Specify either id or title, but not both.
Actual results
m365 planner task get throws
Error: The specified ownerGroup undefined does not exist
Or if you execute:
m365 planner task get --bucketId 'someid' --ownerGroupName 'Some Group'
it throws:
Error: Cannot read properties of undefined (reading 'toLocaleLowerCase')
Diagnostics
No response
CLI for Microsoft 365 version
5.3
nodejs version
16.13.1
Operating system (environment)
Linux
Shell
zsh
cli doctor
No response
Additional Info
No response

Hi @martinlingstuyl
Can I work on this?

Of course, please do! Thanks for all the work @nanddeepn 🔥

Hi @martinlingstuyl, @waldekmastykarz, @garrytrinder
This command planner task get is currently using beta endpoint /beta/planner/tasks/{id} to enrich the results with the priority field.
The priority field is also available in v1.0.
Should we consider upgrading this endpoint from beta to v1.0?
Please suggest.

Hi @nanddeepn,
this would definitely make sense.
Please proceed with the suggested changes.
br,
Patrick


customize

my changes to the cursor-party live here in the customize branch
I use the PR to inspect the diff

Sorry, opened it accident.


Fail to build python2.7-pycairo

Issue description
The URL to file pycairo-1.10.0-waf_python_3_4-1.patch is broken and can not be downloaded during build:
trying http://www.linuxfromscratch.org/patches/blfs/svn/pycairo-1.10.0-waf_python_3_4-1.patch
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404 Not Found
error: cannot download pycairo-1.10.0-waf_python_3_4-1.patch from any mirror
...
cannot build derivation .../nix/store/f9ll92sns2d4y2njcvsfxb69hjcccimc-python2.7-pycairo-1.10.0.drv: 1 dependencies couldn't be built

Steps to reproduce

install latest nix on SL6
update unstable channel
run nix-env -i xpra

Technical details

System: (SL6)
Nix version: nix-env (Nix) 1.11.9

nix-build -A python2.pkgs.pycairo --check builds fine on both master and current nixpkgs-unstable.
It might have been possible the host was temporarily down.

That URL is down.  --check doesn't redownload.

@vcunat I just noticed another broken URL needs to be fixed in pkgs/development/python-modules/pycairo/default.nix:
from:
http://www.linuxfromscratch.org/patches/blfs/svn/pycairo-1.10.0-waf_unpack-1.patch;
to
http://www.linuxfromscratch.org/patches/blfs/8.0/pycairo-1.10.0-waf_unpack-1.patch;

@lzhang10: right, I could've noticed that.  Pushed as well, thanks.


fix(RDSCPUUtilization): critical now and fixed formatting of number

Increase visibility of critical CPU utilization to highlight as important clue
Fixed formatting of number (likely a typo)

#sre