Updates to match Flows service updates

Some of these include:

Renaming of role values and migration of filtering to filter_role
instead of filter_roles (plural). Impacts both SDK and CLI.

Update documentation to talk about different roles, and also
alternative methods of specifying values in expressions and the new
method of catching timeouts.

Where identities are entered for roles, allow the Globus Auth
username to be used in place of the full urn with uuid. Thus,
<EMAIL_ADDRESS>could be used to specify a user with a role rather
than using the UUID. Docs updated to guide people as such.

A few other cleanups here and there.

@jimPruyne I'm happy w/ this if you are!


Shader Hooks

Need reviewing as causing oddities with magic effects, specifically Quick Light SE

Narrowed down to:
ModelReferenceEffect1 and ModelReferenceEffect2
Fix undergoing testing.

Issue resolved.
PR 2 being released later on today.


doc: documented JSONRPC class as much as i could

should be less cryptic now

There are several issues you need to fix:

Only tab indentation should be used.
Individual arguments cannot have descriptions, so you need to describe each argument in the method description instead (use [code]argument_name[/code] to refer to arguments in the description).

is this still missing something?

Looks great overall, just found a few remaining issues to address.
Then the commits in this PR need to be squashed into one so that the PR is ready for merging. See PR workflow for instructions.

ok.

Thanks. I force pushed an update to rebase and solve merge conflits, and fix remaining style issues.

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 3.3.3.


[5.9] Navigator performance improvements

Explanation: Perf improvements to the Navigator when navigating links
Scope: Doc pages
Issue: rdar://105184773
Risk: Small
Testing: Manual testing on Doc pages
Reviewer: @marinaaisa
Original PR: #541

@swift-ci test


There are issues with the Dutch (NL) version of the tutorial

As i got stuck/confused a bit on the last exercise from part 3 of the tutorial in English, i thought i would check it out in my native language Dutch.  But in part one i aleady stumbled upon an error in the instruction of excercise 2 and although i completed exercise 3 by making the words in the instruction italic and bold it didn't complete the exercise.

This tutorial is certainly very helpfull to learn the basics of Markdown for a beginner like me, but i think that there are some tweaks needed in the Dutch version of it.

Hey, many thanks for the report! This should be fixed now.


fully connected spinglasses

just wondering if you have benchmarked this method for fully connected spin glasses?

No, I haven't. I do not think tropical tensor network is good at handling fully connected graphs.

thanks @GiggleLiu


Provide the option to calculate NPSP donation rollups based on Oppportunity Products (product family)

The structure of NPSP rollups basically requires that an organization represent a single 'type' of gift as a single Opportunity. This structure is overly simplistic for some organizations particularly those where multiple 'types' of revenue can be generated from a single transaction (event registration, membership, donations -- paid all at once).
The 'umbrella' record for all this revenue is best managed in the standard Salesforce model as the Opportunity record, with Opportunity Products representing the types of revenue included in that single commitment. Yet the NPSP rollups do not work in this scenario and must be recreated with custom code (or third party apps) which creates a significant burden for organizations wishing to follow this best practice. As well, the option to involve Opportunity Products in the data model opens up other possibilities with regards to third-party integrations (accounting, etc) and native features like Forecasting.
It seems to be the most meaningful change to expand the available data models would be to provide the OPTION for the NPSP roll-ups to be based on Opportunity Products (with a filter on Product Family to distinguish donations from memberships).

Salesforce has determined that this issue will not be explored further.


Refactor objects into submodules

Is your feature request related to a problem? Please describe.
Currently everything is available as a toplevel import which is more or less okay but I think we can effectively group the hypotrochoid and its subclasses in a hypotrochoid submodule and the epitrochoid classes in a separate submodule
This would help mentally separate the different objects and know "okay these are organized with these"

i will take it


project not found

Hey there :wave: and thanks for this amazing action.
Sadly, I am getting this error when running this action:
Run<EMAIL_ADDRESS>/usr/bin/docker run --name srggrsassignoneprojectgithubaction120_2c2dc5 --label c27d31 --workdir /github/workspace --rm -e GITHUB_TOKEN -e INPUT_PROJECT -e INPUT_COLUMN_NAME -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e RUNNER_OS -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/[org/repo/repo]":"/github/workspace" srggrs/assign-one-project-github-action:1.2.0  "https://github.com/[org/repo]/projects/1" ""
jq: error (at <stdin>:4): Cannot index string with string "html_url"
No project was found.
jq: error (at <stdin>:4): Cannot index string with string "name"
INITIAL_COLUMN_ID is not found.

I have no other actions in the repo defined. I c&p your example, so my action looks like this:
name: Auto Assign to Project(s)

on:
  issues:
    types: [opened, labeled]
  pull_request:
    types: [opened, labeled]
env:
  GITHUB_TOKEN: ${{ secrets.GithubToken }}

jobs:
  assign_one_project:
    runs-on: ubuntu-latest
    name: Assign to Project
    steps:
    - name: Assign NEW issues and NEW pull requests to [project] project (project 1)
      uses<EMAIL_ADDRESS>      with:
        project: 'https://github.com/[org/project]/projects/1'

Trying to use column_name without a value gives the same error and further an error is shown in the github ui when editing the action.
Note: I used [] for placeholders for real values, e.g. in the project url ;-)

I'm not sure what is going on in your case... but as you can see in this Issue, the action worked as expected, and assign your opened issue to the project Towards Usability as mentioned in the notifications above github-actions bot added this to To do in Towards Usability 2 days ago.
Have a look at my workflow file in here.
I suggest to check:

the right url of your project
the token, especially for repo owned by an organisation

Let me know how it goes! :smile:

@srggrs thanks for the reply. I double checked again and found that the GITHUB_TOKEN is a predefined action token, like mentioned here About GITHUB_TOKEN.
Now everything is working fine :+1:

@GRBurst I am having the same issue, I was wondering what you did to resolve it?

@gracetangg sorry, being off the radar lately. Do you still face the issue? The solution for me was to use the mentioned GITHUB_TOKEN instead of defining my own one.

@GRBurst Your good, I did that and it worked. Thanks!

@GRBurst @gracetangg @srggrs  I'm trying to do the same thing what you guys were saying.
name: Auto Assign to Project(s)
on:
issues:
types: [opened]
env:
MY_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
jobs:
assign_one_project:
runs-on: ubuntu-latest
name: Assign to One Project
steps:
- name: Assign NEW issues to Kanban
uses<EMAIL_ADDRESS>if: github.event.action == 'opened'
with:
project: 'https://github.com/users/saideep2000/projects/2'
column_name: 'Backlog'
I make sured that the project link is correct and even the cloumn name is correct and also using MY_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} for my predefined token to generate.
But still I'm getting the error, can you point me where I'm going wrong with this. I also saw the documentation but I don't way to actually see to solve this.
jq: error (at :4): Cannot index string with string "html_url"
No project was found.
jq: error (at :4): Cannot index string with string "name"
INITIAL_COLUMN_ID is not found.


aviv;content-management-systems-weeks24-26

aviv; content management systems

[x] 🥚 Understanding what a content management system is, and why people use it
[x] 🥚 Given an existing CMS installation fill in and publish a new piece of content
[x] 🥚 Perform a simple http call via postman to retrieve a piece of information.
[ ] 🐣 Extend and adapt an existing CMS configuration
[x] 🐣 Perform a http call with filtering via Postman
[ ] 🐣 Perform a http call with "include" via Postman
[x] 🐣 Perform a http POST call to create new data
[x] 🐣 Perform a http PUT call to update existing data
[ ] 🐣 Configuring endpoint permissions (Authentication vs Authorization)
[ ] 🐥 Design a data model from scratch based on user requirements

Week 24

we have pushed our progress to the group practice repo

Check-In
I Need Help With:
so far so good
What went well?
completed exercises with regular study group. looking into next week's data app
What went less well?
some http syntax is odd and i am unsure about handles: and how the data objects work
Lessons Learned
http requests

new syntax

Sunday Prep Work

keep studying/practicing

some http syntax is odd and i am unsure about handles: and how the data objects work

http can have its own module by itself. The this link covers the basics of HTTP. For this module you can focus on the requests and the request status code.

Week 25

we have pushed our progress to the group practice repo

Check-In
I Need Help With:
a bit confused about databases, but thats to be expected.
What went well?
completed exercises with regular study group. looking into authentication/authorization
What went less well?
some scheduling issues this week. not as much time as id like to read/review
Lessons Learned

database fundamentals

Sunday Prep Work

keep studying

Week 25

we have pushed our progress to the group api repo

Check-In
I Need Help With:
very confused about databases management on strapi and heroku.
What went well?
built api from scratch with help of group and coach Yoshi, working on app code currently
What went less well?
data gets randomly deleted from database after publishing. issues with npm, yarn and node
Lessons Learned

database fundamentals

Sunday Prep Work

keep working on code
write module retro


ValueError: too many values to unpack (expected 2)

CentOs 7
Python 3.6.6 :: Anaconda, Inc.
I was testing fvecSim using the mosquito data and found a bug:
python diploSHIC.py fvecSim diploid hard_0.msOut.gz test_hard_0.diploid.fvec --totalPhysLen 55000 --maskFileName Anopheles-gambiae-PEST_CHROMOSOMES_AgamP3.accessible.fa.gz --chrArmsForMasking 3R
Program output:
file name='hard_0.msOut.gz'vcfForMaskFileName='None': not masking any genotypes! reading masking data...reading Anopheles-gambiae-PEST_CHROMOSOMES_AgamP3.accessible.fa.gz checked genotypes at 0 sites Traceback (most recent call last):   File "/opt/software/SHIC/diploSHIC/makeFeatureVecsForSingleMsDiploid.py", line 64, in <module>     sampleToPopFileName=sampleToPopFileName) ValueError: too many values to unpack (expected 2) /opt/software/miniconda/4.4.10--GCC-4.9.4/bin/python /opt/software/SHIC/diploSHIC/makeFeatureVecsForSingleMsDiploid.py hard_0.msOut.gz 55000 11 Anopheles-gambiae-PEST_CHROMOSOMES_AgamP3.accessible.fa.gz None None None 0.75 3R 0.25 None test_hard_0.diploid.fvec
When removing the --maskFileName and --chrArmsForMasking parameters, it runs fine.
BTW, is the program designed to sample the provided mask file randomly (or sequencially?) to mimic true data?
Thanks,
Shujun

this looks like a bug that was recently introduced to the code. hold on and we will fix it.

Hi @andrewkern , @oushujun,
I just cloned the repo and setup. Still getting the same error when I invoke --maskFileName and --chrArmsForMasking. Is there a work around?
thanks,
@stsmall

this was code that @dschride had changed recently but i don't believe he has pushed his patch to github. @dschride did you push the new masking version?

I used the buggy version and not supplying any masking to the simulated
data - the genome I used is very good and only contains very limited Ns and
physical gaps, so I figure not making would not be too big a problem.
Shujun
On Wed, Nov 21, 2018, 7:16 PM Andrew Kern<EMAIL_ADDRESS>wrote:

this was code that @dschride https://github.com/dschride had changed
recently but i don't believe he has pushed his patch to github. @dschride
https://github.com/dschride did you push the new masking version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kern-lab/diploSHIC/issues/9#issuecomment-440859957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFt-NPIx_apXTLj0O9qM9J2hSJ_qLcnhks5uxezlgaJpZM4WZWnq
.

Shujun, please pull the latest version and try again. The cause of this issue is that diploid mode did not support masking sites without also masking genotypes, but I have added this functionality. Try running again and the same manner and let me know if this issue is resolved.

Thank you @dschride!
After this update the example data finishes without error. My data using diploid and a mask file also finished without error.

No problem!

I am having a similar problem
Ran the following
python diploSHIC.py fvecSim diploid sims/TEST.ms sims/TEST.fvec --totalPhysLen 110000
/anaconda3/bin/python makeFeatureVecsForSingleMsDiploid.py sims/TEST.ms 110000 11 None None None None 0.75 all 0.25 None sims/TEST.fvec file name='sims/TEST.ms'Traceback (most recent call last):   File "makeFeatureVecsForSingleMsDiploid.py", line 17, in <module>     trainingDataFileObj, sampleSize, numInstances = openMsOutFileForSequentialReading(trainingDataFileName)   File "/diploSHIC/msTools.py", line 150, in openMsOutFileForSequentialReading     program, numSamples, numSims = header.strip().split()[:3] ValueError: not enough values to unpack (expected 3, got 1)
I don't see anything wrong with my header do you?
Here is the .ms file in question.
TEST.ms.txt


groupby not working

Hi my friend, first of all thanks for your great work on this directive.
I put your sample exactly the same of your sample using groupby and does not work my friend. Please see the attached image:

Im doing something wrong?
Thanks!

what version of the component and of angular are you using?

Hi my friend, I'm using angular 1.5.8

Sorry, I close the issue by error

And what version of the component are you using?

According with my bower.json     "angularjs-dropdown-multiselect": "^1.11.1"

The examples are already for version 2.0.0-beta. I'm still looking into how I'll also provide examples for v1. Previously group-by was an attribute instead of a setting property.

Ok I understand. How can use the 2.0 version? Or.. how can use the attribute correctly on my actual version?

in bower.json "angularjs-dropdown-multiselect": "^2.0.0-beta.1"
in current version

Thanks for your support my friend.


Add lateral sub select join type "(+)"

SELECT
    T1.d,
    LV.c
FROM
    T1,
    LATERAL (SELECT T2.C FROM T2 WHERE T1.x = T2.x AND T2.k = 5)(+) LV;

The lateral view in query is merged to yield query. Like this
SELECT
    T1.d,
    T2.c
FROM
    T1,
    T2
WHERE
    T1.x = T2.x(+) AND T2.k(+) = 5;

But, JSqlParser did not support this syntax. So I added.

Coverage decreased (-0.1%) to 86.479% when pulling abc39a508cd0ac306bccfd69e263a3d3c09fcddf on evan-choi:oracle-lateral-join into abedce5395156387948ff4222fe66d8bf04d3c0e on JSQLParser:master.


Clarify requirements of Analysis Plugin

Description
Provide the ability to manage grouped collections of images associated to a KG resource (e.g. a dataset)
Acceptance criteria

[x] Dev team has reviewed mocks, identified points to clarify
[ ] Clarifications have been settled with Samuel and Jean-Denis
[x] Mocks have been updated
[x] Data Model has been defined
[ ] Implementation tickets has been created

References

Initial mocks

Notes

when defining the data model, consider the existing data models in prod (e.g. Analysis, CampaignAnalysis, AnalysisReport, AnalysisResult etc.)

Questions

Is this a public or BBP specific plugin?

BBP specific

What is the full scope of the plugin?

image management for Datasets

Is the real-estate of the plugin is sufficient for browsing/editing?

for the time-being, yes

Are the operations supported for a section or for an image in the section (such as select/delete).

all operations on both

restrictions on image size and format (eg: only jpeg and png under 5MB?).

only jpeg and png under 5MB

What are the data models for which the analysis plugin should show.

Dataset

Do we want to support the ability to download all the images or images per section?

no

Are there captions for images? (title/text).

yes

Does the caption belong to a section or individual images?

individual images

How does the plugin behave when the user does not have a permission for write or read action on the storage where images are stored

the edit view should not be available if the user doesn't have resources/write on the origin project
the images should be displayed with a default placeholder "you do not have access to view this image" for which the user doesn't have access
a "you do not have access to upload an image" should be presented if the user doesn't have access to any of the origin project storages

are we going to support thumbnails?

no

are we going support an image carousel for the user to browse through the images?

yes

are we going to support linking existing images? (for example from data cart)

no

are we going to support PDFs or other file types similar to that?

not at the moment

Tickets 3317, 3318, 3319, and 3320 are tickets associated with this plugin so far.


incorrect BatchCount is not caught by validation

ACH Version
v1.35.1 cli
What were you trying to do?
We were trying to validate an ACH file.
What did you expect to see?
The file input to the program contained an incorrect BatchCount, which should have been shown as an error.
What did you see?
The incorrect BatchCount was not mentioned.
How can we reproduce the problem?
Send a file through the program with an incorrect BatchCount.

We've historically treated the FileControl / BatchControl records as computable and haven't validated their contents strictly. Are you encountering a problem with the invalid counts? You can always call .Create() and get a properly tabulated file.

We are using this program to validate and cross-check files before submitting to the bank.  The bank rejected our file as invalid after being successfully validated by the program.  We expected the program to be correct.

I'm not comfortable changing the default behavior to begin validating BatchCount (and other BatchControl fields) so we'd need to add a ValidateOpt for that. Can you use the Go library (or HTTP server) to create ACH files? If so both have .Create() / /build methods available which tabulate BatchControl records.
You said you're using achcli to verify prior to submission. The tool you're using as the primary seems to have a bug with calculating BatchControl fields.

Per your previous comment, this library aims to implement the Nacha spec.  Does the Nacha spec allow invalid BatchCount and other BatchControl fields?
We do not use Go.
We have developed our own software for producing ACH files, and achcli has been a handy tool for validating and displaying them during the development process.  We have since discovered that achcli does not correctly validate all fields.


4k 60fps support?

I noticed that when I installed this extension 4k60fps option was removed on videos that have 4k60fps option.

Unfortunately, Youtube doesn't encode 4k60 videos in h.264, so the only available 4k60 streams are vp9. This is probably to save on bandwidth costs on their end.


[WIP] Add schema_extra metadata to Pydantic schemas

Purpose
add schema examples.
Changes
Add schema examples for:

[ ] schemas

[x] connection_configuration

[X] connection_config.py
[X] connection_secrets.py
[X] connection_secrets_bigquery.py
[X] connection_secrets_email.py
[X] connection_secrets_mariadb.py
[X] connection_secrets_mongodb.py
[X] connection_secrets_mssql.py
[X] connection_secrets_mysql.py
[X] connection_secrets_postgres.py
[X] connection_secrets_redshift.py
[X] connection_secrets_snowflake.py
[X] connections_secrets_https.py

[ ] email

[ ] email.py

Checklist

[ ] Update CHANGELOG.md file

[ ] Merge in main so the most recent CHANGELOG.md file is being appended to
[ ] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[ ] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #102

Hey @seanpreston and @pattisdr, thanks for the reviews. Alright, I am going to adjust the values accordingly and continue my series of documentation on all schemas.  One thing, would you mind adding hacktoberfest label for the repo?

One thing, would you mind adding hacktoberfest label for the repo?

Will do this today.
@wiseaidev did you see my email wrt the new repository? I sent it to the address listed publicly on your Github profile.

Hey @seanpreston, email received. I will move my contribs over there.

Appreciate it, thanks


Integrate PostgreSQL and MySQL reports

right now an independent RDF file is generated for each RDBMS, and the developers should only provide one file with all the results. The mappings used to generate the report use blank nodes, so naïve integration is not possible.

to test https://github.com/julianrojas87/ttl-merge

to test https://github.com/julianrojas87/ttl-merge

it works, we will use it!

it works, we will use it!


About human3.6m pre-processing?

About the pkl referenced into /lib/data/datareader_h36m.py, can I ask where can i find the repo for pre-processing human3.6m to obtain this pkl files?

The .pkl file is from LCN. You may check the following links for further information:

https://github.com/Walter0807/MotionBERT/issues/51
https://github.com/CHUNYUWANG/lcn-pose/tree/master#data
https://github.com/Embracing/GFPose/issues/2


Updating versions

#353 Update Micro-server to latest version of cyclops and cyclops-React is implemented in this PR


failed to format array with long string/identifier

❯ cargo +nightly fmt
error[internal]: left behind trailing whitespace
 --> /private/tmp/tmp/src/main.rs:6:6:1
  |
6 |
  | ^^^^^^^^^^
  |

fn main() {
    let aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa =
        1;
    let b = 2;
    let a = &[

                aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa,
        b,
    ];

    // or this
    // let a = &[

    //             "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
    //     "b",
    // ];
}

The max_width is a hard constraint for rustfmt. If it can't rewrite the identifier it'll fail to rewrite the let binding. Similarly, the long identifier in the list prevents it from being rewritten.
In this case there are two options to address the trailing whitespace error:

The developer can manually remove the trailing whitespace
increase the max_width to some sufficiently high value.


Kepler.gl visualizations for Dashboard

I think I would prefer to convert the async / await to a simple promise, removing the options for the eslint we added today (and of course, fix the pending tests), so we can get a PR without any hound issue at all


Tarantool Profiles vs GlobalProfiles

Hi!
Profiles doesn't exists on newer versions, it was replaced with GlobalProfiles.
Please check.

I'm running the latest BM (20170810-074109) and there is no GlobalProfiles space, only Profiles.

This is not BM related, but Tarantools/Registry and Replicator. Some how, you still have the old way working.
brandmeister -> 20170810-074109
brandmeister-registry-ng -> 20170811-160430
tarantool-mysql-replicator -> 20170528-113618


Gradle 7.6 (UnsupportedOperationException  -> PreResolvedResolvableArtifact.getModuleVersion)

Builds after 7.6-20220920234319+0000 release are failing with above error.
Expected Behavior

Build should pass
Current Behavior

Build is failing when starting with a clean cache directory
Context

Our jenkins pipelines clean caches before every build, hence we are unable to build with latest nightly releases
Steps to Reproduce

I have attached a demo project to try with
grade-wrapper.properties file has both release versions mentioned
distributionUrl=https://services.gradle.org/distributions-snapshots/gradle-7.6-20220920234319+0000-all.zip
distributionUrl=https://services.gradle.org/distributions-snapshots/gradle-7.6-20220930142900+0000-all.zip
There is no issue with builds using<PHONE_NUMBER>4319 release but every release after this is having this issue
Start by deleting caches directory in ~/.gradle folder.
running ./gradlew clean build --stacktrace will fail the first time (With<PHONE_NUMBER>2900)
running ./gradlew clean build --stacktrace will pass next time (With<PHONE_NUMBER>2900)
demo.zip
stacktrace.txt
Your Environment

MacOS - 12.6
JDK - 17
Spring Boot - 2.7.4
Build scan URL:

Confirmed regression in 7.6, behaves exactly as described.

Thanks a lot for trying out nightlies and reporting discovered issues!
The issue was introduced in these commits and is still present on the latest nightly.
Following up to get this fixed.


Format-on-save triggers double save

I've set up a formatter for my Ruby code and I'm seeing the file being saved twice, once when I run :w and another time when the formatter modifies the file. VSCode doesn't behave this way and at least from my perspective this doesn't seem like "expected" behavior to me.

This triggers a double rebuild with my filesystem watcher; would be nice if this can be optimized to a single save.


New version: Grammarly.Grammarly version <IP_ADDRESS>0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110366&drop=dogfoodAlpha

/AzurePipelines run


grepWin: Add checkver for automated updates

Noticed that grepWin didn't get updated for quite a while, probably due to missing checkver.
Wanted to automate this using the Github repository with a small change - this should pick up the correct hashes at next update cycle.

Thanks for the good comment!
The newest release has this URL: https://github.com/stefankueng/grepWin/releases/download/2.0.8/grepWin-x64-2.0.8_portable.zip
The file inside is called grepWin-x64-2.0.8_portable.exe
So I assume that the correct autoupdate URL would need to be https://github.com/stefankueng/grepWin/releases/download/$version/grepWin-x64-$version_portable.zip#/grepWin-x64-$version_portable.exe, right?

No no
How can a ZIP file become an exe?

I see - the # then stands for renaming, I assumed it was for extraction.
I now simply used the MSI - let me know if this works for you 👍

@JamesCullum please update the versions offline and check once if installation takes place properly.
Check if all persist based configurations work as well and share screenshots for all steps
Scoop usually prefers portable files so we are more likely to update to portable ZIP files.

Tested and updated - grepWin persist nowadays via registry.
The MSI is just a ZIP with the portable files, but packed with more translations.

Tested and updated - grepWin persist nowadays via registry. The MSI is just a ZIP with the portable files, but packed with more translations.

No, I currently test it, and msi record grepwin.ini in AppData/Roaming/...
The portable version record the grepwin.ini in the current directory.

Good hint, didn't know that! Will take a look at the portable version then again!

Okay, I'm just renaming it via script now.
@Valinor I noticed that even the portable version in 2.0.8 doesn't automatically store the ini in the same folder. Its actually now only decided by the flag /portable flag, which I added to the shim. Maybe give it a try and let me know if it works 👍

@pratikpc Can you validate this push (approuve your "requested changes")? Seem that the next commit correct your point

/verify

@Valinor @JamesCullum could some screenshots be shared on this working with both 32 and 64 bit?
The manifest currently looks fine to me and passes verify too

32bits and 64bits works for me.


Try to detect out-of-memory on wasm and show an error screen

Right now the browser just gets stuck: We're hitting an exception that we're not handling and that's it.

Easy repro: Drag in a very large video file for instance.
Open question though whether we can: the error handling we already have on in our html/js stub relies on the panic handler and we're clearly not getting that far.
What the log looks like in chrome:
Uncaught RuntimeError: unreachable
    at re_viewer.wasm.__rg_oom (re_viewer.wasm-08170a66:0x1631efb)
    at re_viewer.wasm.__rust_alloc_error_handler (re_viewer.wasm-08170a66:0x1641e64)
    at re_viewer.wasm.alloc::alloc::handle_alloc_error::h246ea5f196567ce5 (re_viewer.wasm-08170a66:0x1648050)
    at re_viewer.wasm.alloc::raw_vec::handle_error::h6991f391977ec209 (re_viewer.wasm-08170a66:0x1632af8)
    at re_viewer.wasm.alloc::raw_vec::RawVec<T,A>::reserve::do_reserve_and_handle::h17a8c64ff011c017 (re_viewer.wasm-08170a66:0x14be01d)
    at re_viewer.wasm.re_mp4::reader::Mp4::read_bytes::hdeb2a7cc8a2776be (re_viewer.wasm-08170a66:0x461cbc)
    at re_viewer.wasm.re_video::mp4::load_mp4::h5202c42e796eb91d (re_viewer.wasm-08170a66:0x66b129)
    at re_viewer.wasm.re_video::VideoData::load_from_bytes::h9b3a333d055e4737 (re_viewer.wasm-08170a66:0xca0649)
    at re_viewer.wasm.re_types::archetypes::_archetypes::asset_video_ext::<impl re_types::archetypes::_archetypes::asset_video::AssetVideo>::read_frame_timestamps_ns::h5e9489acb72c5120 (re_viewer.wasm-08170a66:0xdd4cc1)
    at re_viewer.wasm.<re_data_loader::loader_archetype::ArchetypeLoader as re_data_loader::DataLoader>::load_from_file_contents::h36914c5005cc2181 (re_viewer.wasm-08170a66:0x67798)

(same in firefox, symbols just not as nicely resolved for some reason)

Maybe @jprochazk has a smart idea


Resevoir Sampling

Awesome project!
Is there any support with resevoir sampling with histograms? e.g. uniform, exponentially decaying, sliding window, sliding time window

I'm looking at the source code and couldn't find any reference to Resevoir. I think it should be implemented. I may suggest you to post on http://feedback.mathdotnet.com/forums/2060-math-net-numerics

Wikipedia: Reservoir sampling


App not loading components when added as Personal App in Teams

The app is not loading the components like Add Members, Digital Badge etc., when added as Personal App in Teams.

I have a similar issue

.

Hello @SriGutha Have you added any Champions to the underlying list?

@SriGutha thanks for being patient while we are trying to fix this.

Hello @SriGutha Have you added any Champions to the underlying list?

@karuanag - I have the same issue and I have added Champions.  It's only in the personal app context, not the Teams tab.

This feature is not there in the current version of CMP. Will try to incorporate on the coming versions with the same.

@SriGutha Recent update Champions Management Portal <IP_ADDRESS> fixed this issue. Please try again with the updated changes.

@eshwarmsft - Thank you !!


Start: Grab all necessary/preliminary information and prepare for implementation in Jupyter

Start:

Electrostatics
Electric Fields
To be continued..


Bug: Fix issue with wrapping certificate

This issue should close #560
Here's a demo on how it looks
https://user-images.githubusercontent.com/32577022/231542488-c0b5f1df-aff9-48a6-afcc-89bec229f40a.mp4

This is still broken for me on Mac

That's odd... on windows for mine everything seems to be fine, do you mind providing me with the width and height, just so I can be sure

Let's get some other people to test too.

I think it's not working on Mac.

Weird that it's no good on Mac.  If it works well on Windows, that's the most important (more users on Windows).  If we can fix on mac, that would be nice to have.

(Unrelated) Though I noticed this happening for smaller screen sizes such as Galaxy Fold (280 x 653, w x h):

@Eakam1007 I think that has something to do with the "Tabs" component

I tried working on this locally on my Mac, and this seems to do it:
diff --git a/app/components/certificate/certificate-display.tsx b/app/components/certificate/certificate-display.tsx
index c80b76a..d499e86 100644
--- a/app/components/certificate/certificate-display.tsx
+++ b/app/components/certificate/certificate-display.tsx
@@ -110,8 +110,9 @@ export default function CertificateDisplay({
               <Text
                 fontFamily="mono"
                 maxWidth={{ base: '2xs', xs: 'sm', sm: 'md', md: 'full' }}
+                minWidth="max-content"
                 fontSize={{ base: '3xs', xs: '2xs', sm: 'xs', md: 'md' }}
-                css={{ whiteSpace: 'pre-line' }}
+                css={{ whiteSpace: 'pre-line', wordWrap: 'normal' }}
               >
                 {value}
               </Text>

Here I'm telling it to not wrap text, and also to never go narrower than the content.  Does this work?

I tried working on this locally on my Mac, and this seems to do it:
diff --git a/app/components/certificate/certificate-display.tsx b/app/components/certificate/certificate-display.tsx
index c80b76a..d499e86 100644
--- a/app/components/certificate/certificate-display.tsx
+++ b/app/components/certificate/certificate-display.tsx
@@ -110,8 +110,9 @@ export default function CertificateDisplay({
               <Text
                 fontFamily="mono"
                 maxWidth={{ base: '2xs', xs: 'sm', sm: 'md', md: 'full' }}
+                minWidth="max-content"
                 fontSize={{ base: '3xs', xs: '2xs', sm: 'xs', md: 'md' }}
-                css={{ whiteSpace: 'pre-line' }}
+                css={{ whiteSpace: 'pre-line', wordWrap: 'normal' }}
               >
                 {value}
               </Text>

Here I'm telling it to not wrap text, and also to never go narrower than the content. Does this work?

It didn't cause any issue when I tested it locally, should I push this code or have some who uses mac try out the changes for themselves?

@SerpentBytes do you mind checking the new commit to see if it works on Mac now?

It works!


jetty 12 / springboot 3.2

Proposal
As I'm creating a new small product I was evaluating springboot 3.2  and face an issue with wiremock as springboot3.2 will use jetty 12, and wiremock  jetty11.
Some jetty artifacts and packages are moving which will make this change challenging on wiremock side.
Workaround
As a workaround, we can replace the wiremock package by the standalone one ( org.wiremock:wiremock-standalone ) so this is not an absolute blocker.
References
https://eclipse.dev/jetty/documentation/jetty-12/programming-guide/index.html#pg-migration-11-to-12-maven-artifacts

Spring boot 3.2.0 is out now FYI.  I can confirm wiremock is not working.
Getting errors like this during tests:
java.lang.IncompatibleClassChangeError: class org.eclipse.jetty.http2.server.HttpChannelOverHTTP2 has interface org.eclipse.jetty.server.HttpChannel as super class
Presumably due to jetty version mismatch.

After updating to spring-boot 3.2.0
At initialisation getting  of
def wireMockServer = new WireMockServer(new WireMockConfiguration().dynamicPort())
Getting error
java.lang.IncompatibleClassChangeError: class org.eclipse.jetty.http2.server.HttpChannelOverHTTP2 has interface org.eclipse.jetty.server.HttpChannel as super class
	at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1012)
	at java.base/java.security.SecureClassLoader.defineClass(SecureClassLoader.java:150)
	at java.base/jdk.internal.loader.BuiltinClassLoader.defineClass(BuiltinClassLoader.java:862)
	at java.base/jdk.internal.loader.BuiltinClassLoader.findClassOnClassPathOrNull(BuiltinClassLoader.java:760)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClassOrNull(BuiltinClassLoader.java:681)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:639)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
	at org.eclipse.jetty.http2.server.AbstractHTTP2ServerConnectionFactory.<init>(AbstractHTTP2ServerConnectionFactory.java:82)
	at org.eclipse.jetty.http2.server.HTTP2ServerConnectionFactory.<init>(HTTP2ServerConnectionFactory.java:53)
	at org.eclipse.jetty.http2.server.HTTP2CServerConnectionFactory.<init>(HTTP2CServerConnectionFactory.java:53)
	at org.eclipse.jetty.http2.server.HTTP2CServerConnectionFactory.<init>(HTTP2CServerConnectionFactory.java:48)
	at com.github.tomakehurst.wiremock.jetty11.Jetty11HttpServer.createHttpConnector(Jetty11HttpServer.java:53)
	at com.github.tomakehurst.wiremock.jetty.JettyHttpServer.<init>(JettyHttpServer.java:89)
	at com.github.tomakehurst.wiremock.jetty11.Jetty11HttpServer.<init>(Jetty11HttpServer.java:44)
	at com.github.tomakehurst.wiremock.jetty.JettyHttpServerFactory.buildHttpServer(JettyHttpServerFactory.java:31)
	at com.github.tomakehurst.wiremock.WireMockServer.<init>(WireMockServer.java:81)
	at com.rewedigital.pushnotificationadapter.WireMockContextInitializer.initialize(WireMockContextInitializer.groovy:14)

Hello,
Same error after having trying to upgrade to Spring Boot 3.2.0:
java.lang.IncompatibleClassChangeError: class org.eclipse.jetty.http2.server.HttpChannelOverHTTP2 has interface org.eclipse.jetty.server.HttpChannel as super class
	at java.base/java.lang.ClassLoader.defineClass1(Native Method)
	at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1012)
	at java.base/java.security.SecureClassLoader.defineClass(SecureClassLoader.java:150)
	at java.base/jdk.internal.loader.BuiltinClassLoader.defineClass(BuiltinClassLoader.java:862)
	at java.base/jdk.internal.loader.BuiltinClassLoader.findClassOnClassPathOrNull(BuiltinClassLoader.java:760)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClassOrNull(BuiltinClassLoader.java:681)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:639)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
	at org.eclipse.jetty.http2.server.AbstractHTTP2ServerConnectionFactory.<init>(AbstractHTTP2ServerConnectionFactory.java:82)
	at org.eclipse.jetty.http2.server.HTTP2ServerConnectionFactory.<init>(HTTP2ServerConnectionFactory.java:53)
	at org.eclipse.jetty.http2.server.HTTP2CServerConnectionFactory.<init>(HTTP2CServerConnectionFactory.java:53)
	at org.eclipse.jetty.http2.server.HTTP2CServerConnectionFactory.<init>(HTTP2CServerConnectionFactory.java:48)
	at com.github.tomakehurst.wiremock.jetty11.Jetty11HttpServer.createHttpConnector(Jetty11HttpServer.java:53)
	at com.github.tomakehurst.wiremock.jetty.JettyHttpServer.<init>(JettyHttpServer.java:89)
	at com.github.tomakehurst.wiremock.jetty11.Jetty11HttpServer.<init>(Jetty11HttpServer.java:44)
	at com.github.tomakehurst.wiremock.jetty.JettyHttpServerFactory.buildHttpServer(JettyHttpServerFactory.java:31)
	at com.github.tomakehurst.wiremock.WireMockServer.<init>(WireMockServer.java:81)
	at com.github.tomakehurst.wiremock.junit5.WireMockExtension.startServerIfRequired(WireMockExtension.java:164)
	at com.github.tomakehurst.wiremock.junit5.WireMockExtension.beforeAll(WireMockExtension.java:230)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	Suppressed: java.lang.NullPointerException: Cannot invoke "com.github.tomakehurst.wiremock.WireMockServer.isRunning()" because "this.wireMockServer" is null
		at com.github.tomakehurst.wiremock.junit5.WireMockExtension.stopServerIfRunning(WireMockExtension.java:218)
		at com.github.tomakehurst.wiremock.junit5.WireMockExtension.afterAll(WireMockExtension.java:256)
		... 1 more

Spring Boot 3.2.0 references Jetty 12.0.3, and Wiremock 3.3.1 references Jetty 11.0.18.
In Jetty 11, org.eclipse.jetty.server.HttpChannel was an abstract class whereas in Jetty 12 it is now an interface.

Hi, I am getting the same issue with Spring Boot 3.2.0 and Wiremock 3.3.1.
As stated above using standalone is fine but would be appreciated if this issue is resolved moving forward.

Can confirm. In my case I was able to work around it by downgrading Jetty since my application uses Tomcat instead of Jetty:
    <properties>
        <jetty.version>11.0.18</jetty.version>
        <wiremock.version>3.3.1</wiremock.version>
    </properties>

    <dependencyManagement>
        <dependencies>
            <dependency>
                <groupId>org.eclipse.jetty.ee10</groupId>
                <artifactId>jetty-ee10-bom</artifactId>
                <version>12.0.3</version>
                <type>pom</type>
                <scope>import</scope>
            </dependency>
        </dependencies>
    </dependencyManagement>

Workaround
As a workaround, we can replace the wiremock package by the standalone one ( org.wiremock:wiremock-standalone ) so this is not an absolute blocker.

This is a great workaround, it also aligns well with the recommendation for Spring Boot users at https://masonm.github.io/wiremock/docs/getting-started/:

Currently it is recommended that you use the standalone JAR as a dependency with Spring Boot projects. This avoids a conflict over the Jetty version

One thing I noticed when migrating to wiremock-standalone was that its object mapper somehow started ignoring the @JsonProperty (some snake-case mapping) when doing ResponseDefinitionBuilder .okForJson(). I switched to the okForContentType with a string body but it was tricky to figure out. Other than that there were no issues.

@tomakehurst do you think it is viable to update to Jetty 12 in next major Wiremock release line (4.x I assume)? Happy to help there but curious what do you think since maintenance load definitely adds up.

I'd actually like to get it into 3.x but dynamically detecting the appropriate version (much like we did for Jetty 9.2/9.4 for quite a while) so that WireMock can co-habit with both.
I'm not going to get a chance to work on this immediately but happy to lend support if you'd like to try it this way.

For me, switching to -standalone was all I needed to do FWIW.

I'm not going to get a chance to work on this immediately but happy to lend support if you'd like to try it this way.

Sure, I will try to pick it up (happy to be assigned to the issue), thank you!

Jetty 11 is now EOL, so Jetty 12 support is needed.

Hi there!
Just out of curiosity, because I don't want to rush anyone who's doing good work. But could you tell me if there is an ETA for this change?
I ask because my team is anxiously waiting for this (well, I'm at least), since the wiremock-standalone route is not an optimal route for us.
I thank you for any info you can give me and thank you for the work you're doing.

@tomakehurst I think we could close this one, the documentation (with limitations) has been merged, thank you!

Jetty 12 supported from WireMock 3.5.2:
https://wiremock.org/docs/jetty-12/
Thanks @reta!


No build provided on npm installation

Hi guys, first of all, great job!
I'm trying to use the DatetimePicker with browserify. I found that you guys already published the package on npm. But when I do the install, I only get the src code.
Is this a bug on the distribution or is that the way you guys want to provide it?
Hopefully is a bug that you can fix, because I don't think build it on my project is something i can do in this case.
Thanks!

it seems you npm/bower users can't decide what you want included in the package: #899.
Mostly, it's complaints about what it does or doesn't do.
I don't use bower or npm so I don't know anything about it.

I agree that distribution is not something easy, and you can't please everyone.
But maybe, you can see what other libraries expose and do the same.
If you think about it, is not that crazy what i'm suggesting.
You are using Less as a preprocessor, i'm not (i'm using Sass actually).
In the way the Datepicker is being provided right now, i need to add a new dependency, and build your project on my build process.
That's great... only if i want to modify your src.
Don't worry if you can't fix this. It was only a question/suggestion.
Regards

Despite that I will likely get an influx of issues about how the packages don't contain everything, the next release will remove (again) the source files.
At some point, I'm just going to lock those files and like people deal with it...


use sed sepearator compatible with busybox sed

busybox's sed errors with '[' as separator. This becomes an issue when
trying to run tensorflow/tools/ci_build/builds/configured on Alpine
Linux.

Can one of the admins verify this patch?

An easy way to confirm:
$ docker run -it --rm busybox /bin/sh -c "echo abc | sed -e 's[abc[xyz[g'"
sed: unmatched '['
$ docker run -it --rm busybox /bin/sh -c "echo abc | sed -e 's|abc|xyz|g'"
xyz

@tensorflow-jenkins Test this please


422 error when trying to sign up with email that already has an account

Should give an error like "try logging in, you dummy"

I would like to work on this issue


Use package:// paths for mesh filenames

The $(find ) syntax resulted in absolute paths in the robot_descriptions. These are not useful when looking at the description from a different pc.

Please see the discussion in #5. Yes, this is a use case not considered there, but your PR breaks the aspects discussed there.

Which aspect exactly is broken? This still uses the package and path from the yaml config file introduced in #9 , so it should still resolve to the correct meshes.

As far as I understood, the package:// syntax is not supported by gazebo, but I might be wrong here. Will test that later today or tomorrow.

@RobertWilbrandt Rebasing your changes ontop of #5 crashes startup with the gazebo launchfile. There are still some things to sort out for the gazebo and ignition simulation, please let me come back to this. I highly agree that the current situation is suboptimal as viewing descriptions from another PC is definitely something to support, but as argued, I don't want to merge it in like that, just yet.

The $(find ) syntax resulted in absolute paths in the robot_descriptions. These are not useful when looking at the description from a different pc.

What do you mean with "files from another PC"? How are files accessed in case of "pacakge://" syntax? They should be also resolved to absolute paths at some point. Isn't it?

The difference is that the $(find ) syntax resolves to absolute paths on the machine loading the description, while the package:// syntax is resolved locally on each machine. The problem occurs when you start the driver on one machine and try to look at the description using rviz from another machine (using robot_state_publisherss robot_description topic): It will not find the meshes and consequently not show anything. I believe this is a pretty common scenario.

I agree, but the problem are simulators. We could add additional ifs for now into xacro. It will not look nice, though.

We could add additional ifs for now into xacro. It will not look nice, though.

I had the same thought. But not supporting remote RViz instances is something I believe not being acceptable, so we'll probably have to go for that.

Edit: One quick note: On ROS1 we have a working gazebo sim using the package:// notation (same gazebo version). I assumed, this gets handled by the ROS abstraction layer of gazebo?

maybe is then ROS2 or something. Can you test gazebo sim one more time with package://?

maybe is then ROS2 or something. Can you test gazebo sim one more time with package://?

I'll dig a little bit. I want to know what is breaking and why.

Aha, see https://github.com/ros-simulation/gazebo_ros_pkgs/pull/1284
So, my suggestion would be to go the ugly way using the ifs until this is resolved...

Closing in favor of #15


Oracle DB upgrade scripts - 6.1

Create a remote branch on github
Brach name should have following pattern
issue_<number>_githubId  for example pull request related to 81 by Manoj will be named as issue_81_magrawa
Go to eclipse and commit or stash your master branch changes and sync your workspace
Do a Git fetch
Now switch over to remote branch you created in github
Once you commit your changes to new branch you can follow instructions here for creating a pull request
Create a New Pull Request
Learn more about pull requests

Hi Don,
I have created Oracle DB upgrade script.
I am now working on creating mysql  DB upgrade script.
Kindly review, so that I can incorporate the feedback comments to My SQL script as well.

Completed the testing. Merging with main branch.


Document observing, alive and dead player filters

<observing/> is missing as the inverse of <participating/> .
https://github.com/PGMDev/PGM/blob/dev/core/src/main/java/tc/oc/pgm/filters/matcher/player/ParticipatingFilter.java
<alive/> and <dead/> are missing as well.
https://github.com/PGMDev/PGM/blob/dev/core/src/main/java/tc/oc/pgm/filters/matcher/player/PlayerStateFilter.java

Thanks for sharing the sources in the code!


Flexible mapping provider to parser

This PR addresses issue #43
It creates a new custom field in Provider that can be used to define a custom Provider name in the circuit-maintenance-parser library, and when not provider, the Provider.slug would be used, same as today.

@glennmatthews I'm gonna merge this as it's already approved and 3985590 only updated the Readme and Changelog


:feet: In OVA provider, user can not edit the path

In OVA provider, user can not edit the path
Issue:
currently we allow to edit path in OVA providers details page
Fix:
a. remove option to edit path in OVA
b. add a comment in the path help popup explaining this is not editable

addressed by #850


[6.4.x] GUVNOR-2367: Strange date format in Asset Search form

See https://issues.jboss.org/browse/GUVNOR-2367
Corollary of https://github.com/droolsjbpm/kie-wb-common/pull/239
(cherry picked from commit 6abd3f1e5f17509fec91a13b1b0ed3a545bb246c)

6.4.x: dfdc51b2a1afc1eed5ff9d7168b82a22f82ac793


add dates to week days enhancement

Will try to work on this, you can assign me this :) thanks

done


Fish Rendering

Inside alacritty, fish does not render the last character.  Not sure if it is other terminals as well.
Track: https://github.com/fish-shell/fish-shell/issues/8164

Kitty bypasses this.  Badass.


Local Gallery Configuration with TFS 2015 Update 2 On Premise

I've got the need to add a new TFS local gallery connection endpoint, but am having trouble finding documentation on how to configure the build/release task extension by setting up this local gallery connection. As I'm on TFS Onpremise Update 2, I believe I'm required to use Basic Authentication. However, before I create any specific user I'm unclear if there is any issue in running in our intranet with encryption/ssl to protect the credentials.
Any guidance would be appreciated as I'm unable to find anything on this for the extension. The "more info" link in TFS Onpremise for this dialogue opens to the latest documentation which seems to focus on personal access tokens, which we can't use at this time.

My recommendation would be to upgrade to TFS 2017 and use the PAT features
therein.
Alternatively you can setup an NTLM proxy in your build agent that would
handle the upgrade from basic to NTLM.
If you enable basic auth you have a few options, we highly recommend
enabling ssl to protect your credentials. The disadvantage of turning in
basic auth is that most users will be prompted to enter their credentials
every time, as automatic windows auth will no longer work.
Alternatively you could setup a second application tier and enable basic
auth on that. Then point your build agent to use its machine name to
connect.
On Jan 27, 2017 21:38, "SheldonH"<EMAIL_ADDRESS>wrote:

I've got the need to add a new TFS local gallery connection endpoint, but
am having trouble finding documentation on how to configure the
build/release task extension by setting up this local gallery connection.
As I'm on TFS Onpremise Update 2, I believe I'm required to use Basic
Authentication. However, before I create any specific user I'm unclear if
there is any issue in running in our intranet with encryption/ssl to
protect the credentials.
Any guidance would be appreciated as I'm unable to find anything on this
for the extension. The "more info" link in TFS Onpremise for this dialogue opens
to the latest documentation
https://www.visualstudio.com/en-us/docs/setup-admin/team-services/use-personal-access-tokens-to-authenticate
which seems to focus on personal access tokens, which we can't use at this
time.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vsts-extension-build-release-tasks/issues/41,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD-uS9AFvPKJBpgc5jxYdnb-kx7Va_ACks5rWlXDgaJpZM4LwMK5
.


[OKD 4.12 installer][vSphere IPI]  IP expected to be in one of the machine networks: <IP_ADDRESS>/16

Describe the bug
new install of OKD 4.12 vSphere IPI fails with invalid API IP address
? Network VLAN_65
X Sorry, your reply was invalid: IP expected to be in one of the machine networks: <IP_ADDRESS>/16
? Virtual IP Address for API [? for help]
API VIP is <IP_ADDRESS>
The following bug which seems to be related is in the release notes for this version:
https://issues.redhat.com/browse/OCPBUGS-6015 fail to create install-config.yaml as apiVIP and ingress VIP are not in machine networks #6783
Version
OKD 4.12 Stable
[root@os-utils-d03  poc410]$ ./openshift-install version
./openshift-install 4.12.0-0.okd-2023-01-21-055900
built from commit e33d9bda73e58d6584f922d68260931c02992a41
release image quay.io/openshift/okd@sha256:8c5e4d3a76aba995c005fa7f732d68658cc67d6f11da853360871012160b2ebf
release architecture amd64
How reproducible
100%
Log bundle

This is also broken in the latest nightly:
DEBUG       Loading Platform...
ERROR failed to fetch Metadata: failed to load asset "Install Config": failed to create install config: invalid "install-config.yaml" file: [platform.vsphere.apiVIPs: Invalid value: "<IP_ADDRESS>": IP expected to be in one of the machine networks: <IP_ADDRESS>/16, platform.vsphere.ingressVIPs: Invalid value: "<IP_ADDRESS>": IP expected to be in one of the machine networks: <IP_ADDRESS>/16]

Workaround
Create a install-config.yaml manually and set
networking:
  machineNetwork:
  - cidr: "<IP_ADDRESS>/16"

Hi,
We are not working on FCOS builds of OKD any more. Please see these documents...
https://okd.io/blog/2024/06/01/okd-future-statement
https://okd.io/blog/2024/07/30/okd-pre-release-testing
Please test with the OKD SCOS nightlies and file a new issue as needed.
Many thanks,
Jaime


⚠ Spryr has degraded performance

In a1088f8, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 503
Response time: 25977 ms

Resolved: Spryr performance has improved in 9a6d7b1 after 8 minutes.


self-updating executable

The Server Control Panel executable should be self-updating.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29363548-self-updating-executable?utm_campaign=plugin&utm_content=tracker%2F164029&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F164029&utm_medium=issues&utm_source=github).

Hey everyone,
i want to share the latest development progress: the SelfUpdater finally landed. :tada: :rocket:
The Server Control Panel is now able to check, if you use the the latest available version.
If you are not up-to-date, it will automatically download the latest version and update itself.
Best, Jens

Yay! :fireworks: Thanx Jens!


mydomain.com:5001 to mydomain.com using custom nginx.conf

Issues:

dotnet works fine on port 5001 (mydomain.com:5001)
mydomain.com - 500 Internal Server

nginx.conf
under /opt/vestacp/home/admin/web/mydomain.com/private
location / {
  # force https-redirects
  if ($scheme = http) {
    return 301 https://$server_name$request_uri;
  }

  proxy_pass          http://<IP_ADDRESS>:5001;
  proxy_redirect      off;
  proxy_set_header    Host $host;
  proxy_set_header    X-Real-IP $remote_addr;
  proxy_set_header    X-Forwarded-For $proxy_add_x_forwarded_for
  proxy_set_header    X-Forwarded-Host $server_name;
}

location /error/ {
  alias   /home/admin/web/mydomain.com/document_errors/;
}

location @fallback {
  proxy_pass      http://<IP_ADDRESS>:5001;
}

location ~ /\.ht    {return 404;}
location ~ /\.svn/  {return 404;}
location ~ /\.git/  {return 404;}
location ~ /\.hg/   {return 404;}
location ~ /\.bzr/  {return 404;}

Vesta admin for mydomain.com

[x] Proxy Support NGINX

Proxy Template: Custom

Also, I tried your config above and get the following error: nginx: [emerg] invalid number of arguments in "proxy_set_header" directive in ... - you're missing a semicolon in X-Forwarded-For line.

Sorry typo just for this post, but on my config it should be closed with a semicolon.

I do the same with self hosted git and don't have a problem. Check error logs\site.com.error.log to see what your 500 error is all about.

Errors I found
*1 connect() failed (111: Connection refused) while connecting to upstream, client: (myIP), server: mydomain.com, request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:5001/", host: "mydomain.com"

I tried your config and still not accessible on mydomain.com

Error said that it cannot access http://<IP_ADDRESS>:5001 upstream.  Are both .net and vesta running in the same docker container?  Otherwise, you can't access with that ip through proxy pass.  Ssh into vestacp container and try curl http://<IP_ADDRESS>:5001
If they are two separate containers with the same docker-compose, then use the container name to access it on the same network.


VSCode zombie processes using lots of resources

Issue Type
Bug
Description
This is probably impossible to reproduce, and also likely a bug in node or electron. I wanted to file an issue anyway so this information isn't lost to time. Maybe I'll get electron debug symbols and figure out what happened.

I had two instances of Code open with small-medium rust projects opened. I noticed my computer becoming sluggish, and checked Task Manager. 3 Code processes were using significant CPU and allocating tons of memory. After closing both code windows, the three processes remained and continued allocating. They'd each get up to ~2GB before reducing to ~750MB and back to 2GB in the span of about 2 seconds. They didn't seem to have any child processes running. I took a minidump of each while they were running. Shortly after, they each levelled off at ~2.2GB but still used some CPU. I then killed each process; killing one did not cause the others to quit cleanly.
If some intrepid debugger wishes to explore this rabbit hole, you'll find the crash dumps here (warning: each decompresses to >2GB):
https://1drv.ms/f/s!AoK0UShcpqOEhrhvARmT7PbBQa6ZhA
VS Code Info
VS Code version: Code 1.20.1 (f88bbf9137d24d36d968ea6b2911786bfe103002, 2018-02-13T15:34:36.336Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4690K CPU @ 3.50GHz (4 x 3492)

Memory (System)
7.95GB (4.89GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (17)

Extension
Author (truncated)
Version

autofoldyeah
bob
1.0.6

better-toml
bun
0.3.2

cpptools
ms-
0.15.0

csharp
ms-
1.14.0

PowerShell
ms-
1.6.0

debugger-for-chrome
msj
4.1.0

vscode-witcherscript
nic
1.0.5

vscode-witcherscript-addon
nic
1.0.3

rust
rus
0.3.2

code-settings-sync
Sha
2.8.7

shader
sle
1.1.2

vscode-hexdump
sle
1.6.0

enumerator
swi
0.0.6

language-stylus
sys
1.9.1

debug
web
0.21.2

change-case
wma
1.0.0

markdown-pdf
yza
0.1.7

Reproduces only with extensions

I'm not aware of us having any tools to dig into a memory dump. @bpasero do you have some tools up your belt?
@mattico Here we have put together some steps for collecting more information. When this happens to you, can you please run some of these steps  --- https://github.com/Microsoft/vscode/wiki/Performance-Issues#visual-studio-code-is-consuming-a-lot-of-cpu

@alexandrudima so far I have not tried to read into native memory dumps

@alexandrudima If this happens to me again, I'll try to do that.

Ah, if I'd checked the wiki I could've used LeakTrack. So many good debug tools hidden here and there on Windows. I setup symbol servers for MS and Electron, and did a bit more digging into the dumps.
In all three cases, the worker threads looked like this:

Knowing nothing about node, I'd expect these look normal.
The stack traces also don't look very helpful, as they look like every other node stack trace I've seen:
https://gist.github.com/mattico/cba4e232b40ae79893f1324215d4aee0
The first dump's main thread was inside code.exe, so I'll look into that.

I could've sworn I saw somewhere on the wiki that you could download a debug build of vscode, but I can't find it anymore. Making a local build to try to get a stacktrace.

@alexandrudima @joaomoreno this is spdylog, and it probably is related to https://github.com/Microsoft/vscode/issues/42752

@bpasero I see a spdlog thread but I don't see spdlog showing up on the stack. #42752 does cause CPU, overload, but not memory.

Alright a local VSCode build doesn't actually include code.exe. I assume that code.exe is just repackaged electron.exe, but it's different enough that the symbols don't match. Beyond that, I'm sure the 'smoking gun' is in those memory dumps somewhere but I have neither the time nor the expertise to go looking.
Feel free to close this issue. Its already done its job to record this information in case it is useful. I'll be back if this happens again and I have more info.


Add a litmusbook for busybox-liveness

What this PR does / why we need it:

Adds a litmusbook for busybox liveness.
Litmus takes LIVENESS_TIMEOUT, POD_NAME, LIVENESS_RERY_COUNT as environment variable.
It performs CRUD operation on busybox using dd-command and for each successful iteration of CRUD-operation it sets liveness status as running.

Special notes for your reviewer:

The liveness client for busybox is part of PR in openebs/test-tools(link: https://github.com/openebs/test-tools/pull/82)

Following is the log of the liveness-container:
10240+0 records in
10240+0 records out
41943040 bytes (40.0MB) copied, 0.431196 seconds, 92.8MB/s
liveness-running
Unable to use a TTY - input is not a terminal or the right kind of file
10240+0 records in
10240+0 records out
41943040 bytes (40.0MB) copied, 0.395782 seconds, 101.1MB/s
liveness-running
Unable to use a TTY - input is not a terminal or the right kind of file
10240+0 records in
10240+0 records out
41943040 bytes (40.0MB) copied, 0.394063 seconds, 101.5MB/s
liveness-running
Unable to use a TTY - input is not a terminal or the right kind of file
10240+0 records in
10240+0 records out
41943040 bytes (40.0MB) copied, 0.545244 seconds, 73.4MB/s
liveness-running

@shashank855 Can we include the liveness script in the deployment itself under livenessProbe. cc: @dargasudarshan

Example: https://github.com/openebs/elves/blob/master/e2e/percona-liveness/percona.yaml


Arbitrary clip paths

It’d be nice to support arbitrary clip paths. I can implement one by wrapping a mark like so:
function clip(mark, renderClip) {
  return {
    initialize(facets, channels) {
      return mark.initialize(facets, channels);
    },
    filter(index, channels, values) {
      return mark.filter(index, channels, values);
    },
    render(facet, scales, values, dimensions, context) {
      const fragment = document.createDocumentFragment();
      const svg = fragment.appendChild(mark.render(facet, scales, values, dimensions, context));
      const clipPath = fragment.appendChild(renderClip(facet, scales, values, dimensions, context));
      svg.setAttribute("clip-path", "url(#clip)");
      clipPath.setAttribute("id", "clip");
      return fragment;
    }
  };
}

That should probably extend Mark, though? And it should generate a unique identifier rather than using “clip”.
Then I could have a function that creates a clipPath element like so:
function renderClip(facet, scales) {
  const clipPath = htl.svg`<clipPath>`;
  for (const {a, b} of abDisplayabilityCoordinates.filter(d => d.displayable)) {
    clipPath.appendChild(htl.svg`<rect
      x=${scales.x(a - 0.0025)}
      y=${scales.y(b + 0.0025)}
      width=${scales.x(a + 0.0025) - scales.x(a - 0.0025) + 1}
      height=${scales.y(b - 0.0025) - scales.y(b + 0.0025) + 1}
    />`);
  }
  return clipPath;
}

Ref. https://observablehq.com/@mjbo/oklab-named-colors-wheel

Maybe duplicate of #181.

I have tried to add clip-path to Plot.image() but it seems not handled by applyIndirectStyles() (that is what my browsing of the code brings me to...)
I would like to clip an image to a geo path:
    Plot.image(italy, {
      x: (d) => d.properties.lon,
      y: (d) => d.properties.lat,

      width: (d) => d.properties.width,
      height: (d) => d.properties.height,
      preserveAspectRatio: "none",
      src: (d) => d.properties.flag,
      clipPath: (d) => `url(#iso-${d.id})`,
      title: (d) => d.id
    })

I have some tinkering going on here: https://observablehq.com/d/a8fe9e54cf07cb7a
Any thoughts?

Here's a snippet that uses the new render transform:
  marks: [
    Plot.geo(perimetro_mexico, {
      render: (i, s, v, d, context, next) =>
        svg`<clipPath id="x">${next(i, s, v, d, context).children[0]}` // create the clipPath "x"
    }),
    Plot.raster(data, {
      x: "longitude",
      y: "latitude",
      fill: "banda_interes",
      interpolate: Plot.interpolatorRandomWalk(),
      render: (i, s, v, d, context, next) =>
        svg`<g clip-path="url(#x)">${next(i, s, v, d, context)}` // reference "x" as clip-path
    }),

see https://observablehq.com/d/d5d3052622043025

That’s really nice @Fil. I bet we could package that up into something reusable. Perhaps a clip transform where you supply a geometry channel, and it uses the geo mark under the hood?

We should consider using CSS clip-path instead of SVG, since it is now widely supported and much more convenient since you don’t need a globally unique identifier.

clip-path + path is still very poorly supported: https://observablehq.com/d/bf434fc5675c8f13

Chrome doesn't seem to support view-box + polygon(coords)—which works under Safari.

The Chrome bug has been fixed in 119 https://chromestatus.com/feature/5068167415595008 https://developer.chrome.com/blog/new-in-chrome-119

@Fil just helped me with an issue applying clip paths to a faceted Ridgeline plot:
https://observablehq.com/@chrispahm/ridgeline-plot-with-average-values
Here we had to remove the transform property from the clipPath elements manually in order to get the position right. Fil came up with the following snippet to achieve that:
return svg`<clipPath id=${encodeURI(i.fy)} style="transform: none">${
              next(i, s, v, d, c).children[0]
            }`;

Maybe we could offer a Plot.geoClip render transform that you can pass GeoJSON and does this?

Plot.plot({
  projection: "albers",
  color: {scheme: "YlGnBu"},
  marks: [
    Plot.density(walmarts, {
      x: "longitude",
      y: "latitude",
      bandwidth: 10,
      fill: "density",
      render(index, scales, values, dimensions, context, next) {
        const {document} = context;
        const g = next(index, scales, values, dimensions, context);
        const clipPath = document.createElementNS("http://www.w3.org/2000/svg", "clipPath");
        const path = document.createElementNS("http://www.w3.org/2000/svg", "path");
        const geoPath = d3.geoPath(context.projection);
        path.setAttribute("d", geoPath(nation));
        clipPath.setAttribute("id", "clip");
        clipPath.appendChild(path);
        context.ownerSVGElement.appendChild(clipPath);
        g.setAttribute("clip-path", "url(#clip)");
        return g;
      }
    }),
    Plot.geo(statemesh, {strokeOpacity: 0.3}),
    Plot.geo(nation),
    Plot.dot(walmarts, {x: "longitude", y: "latitude", r: 1, fill: "currentColor"})
  ]
})

"the clip option is set to a GeoJSON object" would be fantastic

As @jwolondon remarks in https://observablehq.com/@fil/multiscale-density-spatial-interpolator#comment-451a923b320875f1, the technique with view-box doesn't work in Safari when the page is scaled 😠
However, that's not what we're doing in #2243, which works fine 😅


Feature Request: Add support for Youtube Music

Thanks for the great work :)

Seems like a duplicate of https://github.com/music-assistant/hass-music-assistant/issues/118

Would love to see this.! ❤

Can you please add your votes to the existing issue #118 and close this duplicate request ?
That way I have the votes aggregated ;-)

Duplicatw


Dependent plot is not working on DeepExplainer

I am trying to draw a relationship plot. I model using <<shap.DeepExplainer>> .   Here only the summary plot is working. I need to draw another plot especially the relationship plot between two features.  coherence matrix is added benefit for this if possible.

Yeah I was trying to do a beeswarm plot for the deepexplainer and it does not work for some reason.

Yeah I was trying to do a beeswarm plot for the deepexplainer and it does not work for some reason either.

So what will be soln for that?

Found an example of beeswarm for deepexplainer. Follow this link (https://colab.research.google.com/github/kweinmeister/notebooks/blob/master/tensorflow-shap-college-debt.ipynb#scrollTo=WRNTlt1qkEAf)


:zap: feat(bundler): Increase peformance by only returning the ast abi and userdoc

Description
Make solc only return abi natspec and ast
Solc compilation is needlessly expensive because we are compiling bytecode and everything else as well. Bytecode is currently niche use case so remove to improve peformance
Testing
Existing tests
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #438   👈

This comment was auto-generated by Graphite.


'BedrockChat' object has no attribute 'bind_tools'

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
import boto3
import json
from langchain_community.chat_models import BedrockChat
from langchain.agents import tool
from langchain_core.prompts import ChatPromptTemplate, MessagesPlaceholder

boto3_bedrock = boto3.client('bedrock-runtime')
modelId = "anthropic.claude-3-sonnet-20240229-v1:0"

llm = BedrockChat(
    model_id=modelId,
    client = boto3_bedrock
)

@tool
def get_word_length(word: str) -> int:
    """Returns the length of a word."""
    return len(word)

#get_word_length.invoke("abc")

tools = [get_word_length]

prompt = ChatPromptTemplate.from_messages(
    [
        (
            "system",
            "You are very powerful assistant, but don't know current events",
        ),
        ("user", "{input}"),
        MessagesPlaceholder(variable_name="agent_scratchpad"),
    ]
)

llm_with_tools = llm.bind_tools(tools)

Error Message and Stack Trace (if applicable)
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[7], line 1
----> 1 llm_with_tools = llm.bind_tools(tools)

AttributeError: 'BedrockChat' object has no attribute 'bind_tools'

Description
I am trying to follow this guide with Bedrock and it's throwing this error.
System Info
langchain==0.1.12
langchain-community==0.0.28
langchain-core==0.1.32
langchain-text-splitters==0.0.1

any updates on when/if this is going to be implemented?

Any update? Clearly Bedrock API is currently supporting this but LangChain has not updated yet.
Tool Calling

I am also facing the same issue, is it updated?

I just updated langchain and tried again. It appears tool call are now working with 0.2.6


when does tokens field get cleared

I have this question too.  Every request generates a new token.  After some clicking around:
[18] pry(main)> u.tokens.count
=> 50
Is this correct?
Shouldn't the latest token be the only one that is needed?
Is it safe to delete everything else?

+1

+1

It turns out this was caused by the CSRF token not being present.  I did this in my ApiController:
  # protect_from_forgery with: :exception

And things are now working!

@fowkswe did you disable the forgery protection? I don't think it's acceptable for my use case.
If you mean the opposite, I have the protection enabled and still have my tokens growing.

@henadzit yes, I removed forgery protection which, from my understanding, is only something to be concerned about in a situation where a cookie is present.
For me, the token authentication is in leiu of cookie / session based auth, so not having csrf protection is fine.   I think this is the case for most API based traffic.
Someone please correct me if my thinking here is wrong.

@fowkswe I gave it a bit of thought, you seem to be right. Even though I have a web client and ng-token-auth stores auth data in cookies, there is no way another site may get access to the cookie. It may send a request but it won't able to read the cookie which is needed in order to set auth headers.
Anyway, I tried to remove the forgery protection but it didn't solve the problem for me :(

Having the same issue here. The tokens array keep growing and growing.

@mustela, try adding
''''
protect_from_forgery with: :null_session
''''
to your controller

I am using Rails 5 API and I get

undefined method `protect_from_forgery' for ApplicationController:Class (NoMethodError)

4 months later and this is not answered? So is it safe to delete the tokens? Because if it is safe, should I do like a cron job to delete all tokens at the end of the day?

+1

Hi there @chirag7jain    ,
In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!
If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.
Hope all is well.

I am ok with it


Starred workflows/tools can run off the box or get arranged weirdly

Bug Report
Expected behaviour
Workflow urls do not run off, boxes are arranged in a more normal left to right, top to bottom manner
The same applies for tools
Actual behaviour

Environment (Browser or OS and Dockstore version)
DockstoreUI - 1.2.1
DockstoreApi - 1.2.0

@JWKaiqi  review and close if no longer an issue

No longer an issue in UI2


Remove all hardcoded EOL K8s version

Description
cnf testsuite must be verified vs up-to-date K8S versions.
By removing the harcoded K8S versions, the gates will leverage the latest stable K8S version as proposed by Kind.
Issues:
Refs: #1948
How has this been tested:
A verification job must be executed to see the testcases which would have to be updated.
This PR may ask for others PR if obsolete test cases ask for updates.
Types of changes:

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

@agentpoyo @HashNuke could you please have a look the failures as for instance
https://github.com/cnti-testcatalog/testsuite/actions/runs/8567585939/job/23479668576?pr=1949
They all failed because of Error: Cannot perform an interactive login from a non TTY device which makes me think that DOCKERHUB_USERNAMES and DOCKERHUB_PASSWORDS are empty.
This error can be easily reproduce by calling docker login -u -p from a non tty device

I just pushed a build to GitHub Actions. I have a wild guess what might be causing this issue. I'll share some notes incase the build fails.

There are some failing tests. Here's the build - https://github.com/cnti-testcatalog/testsuite/actions/runs/8611760709
I've tried re-running the failed tests again. They should complete soon.
@collivier I checked the changes in this PR. Looks like this is related to an older issue.
Here is some info I dug up from my notes:

New kind clusters for k8s 1.25+ are based on the Distroless project.
These are minimal. They do not contain tar, sh and few other things.

For GitHub Actions to work we'll have to configure to the kindest/node image that is the latest version not based on distroless. This is the reason that the kind node image was hardcoded in the GitHub Actions.
I posted some notes in an older issue:
https://github.com/cnti-testcatalog/testsuite/issues/1631#issuecomment-1240470157
From what I can remember, some parts of the testsuite rely on being able to ssh into the node and run certain commands (cluster-tools most likely). The distroless images don't help with this. Fixing the root cause would mean looking into this specifically.

@HashNuke I frankly don't get your points, sorry.
Many issues can easily fix by removing the useless -ti in kubectl calls.
$ git diff
diff --git a/spec/airgap_task_spec.cr b/spec/airgap_task_spec.cr
index 3f6c724e..c14f3cc5 100644
--- a/spec/airgap_task_spec.cr
+++ b/spec/airgap_task_spec.cr
@@ -44,9 +44,9 @@ describe "AirGap" do
     # Get the generated name of the cri-tools per node
     pods.map do |pod|
       pod_name = pod.dig?("metadata", "name")
-      sh = KubectlClient.exec("-ti #{pod_name} -- cat /usr/local/bin/crictl > /dev/null")
+      sh = KubectlClient.exec("#{pod_name} -- cat /usr/local/bin/crictl > /dev/null")
       sh[:status].success?
-      sh = KubectlClient.exec("-ti #{pod_name} -- cat /usr/local/bin/ctr > /dev/null")
+      sh = KubectlClient.exec("#{pod_name} -- cat /usr/local/bin/ctr > /dev/null")
       sh[:status].success?
     end
     (/All prerequisites found./ =~ response_s).should_not be_nil

You're free to install any software in the kind images
$ sudo docker exec -ti kind-control-plane /bin/bash
root@kind-control-plane:/# apt-get update
Get:1 http://deb.debian.org/debian bookworm InRelease [151 kB]
Get:2 http://deb.debian.org/debian bookworm-updates InRelease [55.4 kB]
Get:3 http://deb.debian.org/debian-security bookworm-security InRelease [48.0 kB]
Get:4 http://deb.debian.org/debian bookworm/main amd64 Packages [8786 kB]
Get:5 http://deb.debian.org/debian bookworm-updates/main amd64 Packages [12.7 kB]
Get:6 http://deb.debian.org/debian-security bookworm-security/main amd64 Packages [151 kB]
Fetched 9204 kB in 7s (1318 kB/s)
Reading package lists... Done
root@kind-control-plane:/# apt-get install tar
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
tar is already the newest version (1.34+dfsg-1.2+deb12u1).
tar set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 9 not upgraded.
root@kind-control-plane:/#

I'm diving into the litmus-chaos based tests in parallel which looks more difficult

I believe this PR is now outdated by https://github.com/cnti-testcatalog/testsuite/pull/1998 which was merged already. @wavell @denverwilliams can you confirm?

@agentpoyo @wavell @denverwilliams I locally rebased this PR on top of main. It remains 3 changes removing the hardcoded obsolete K8S versions. Do you want me to pin v1.29.2 as you do in the other PR or do I just push the rebase here ?

I'm thinking @denverwilliams is saying that it should stay pinned to avoid intermittent failure. Maybe we should update every month or so?

I'm thinking @denverwilliams is saying that it should stay pinned to avoid intermittent failure. Maybe we should update every month or so?

@denverwilliams @wavell It's indirectly pinned via the kind version. I can doubly hardcode the version if really needed. please let me know

@collivier Pinning via the Kind Version at the top level and removing the image pinning is fine.

Note:
The test spec test for pod_memory_hog is failing https://github.com/cnti-testcatalog/testsuite/actions/runs/9020615104/job/24793752233 after this was merged. It seems to be a problem with the spec for that test and not the test itself. It also does not appear to be related to the changes in this PR.
cc: @denverwilliams


Row selection not matching activation status

The main callback in the validation app is designed to automatically select entries for which is_active is True, whenever data is loaded. However this doesn't seem to be working properly. I'm not sure what's going wrong.

@tsmbland , I cannot reproduce the error. If I manually validate a couple of entries in the Admin, marking one as active and the other one as inactive:

the checkbox appears correct checked/unchecked based on that activation status when going to the validation screen:

Maybe it was fixed automatically in the meantime?

Obviously, they only appear when requesting validated data. When requesting not validated data, only active entries appear since they can only be set as not active if they are also validated. See https://github.com/ImperialCollegeLondon/paricia/blob/cefd92384a26767c68204e02e3ad0b1456b84664/measurement/models.py#L254

Closing, for now as it does not seem to be an issue anymore.


Restrict access to a specified domain with flask_util.

I am using oauth2client.contrib.flask_util for an application whose access I want to restrict to accounts at a specified Google Apps domain. To do this, I've edited the callback view to check the 'hd' field of the credential's id_token just before saving the credentials to storage. The particular domain is given as a kwarg to init_app, which means that it gets passed to client.OAuth2WebServerFlow, which already recognizes it and adds a helpful message to the log-in page indicating that an email address at that domain is expected. (It appears, however, that that argument is not enough to restrict access, hence the changes to the callback view, although if there is already a built-in way to restrict I'd be happy to use that instead.)
If this is something that'd be generally useful, I can make a pull request.

@rfinck interesting. Does the authorize_callback not cover this use case without needing to override the callback view?

I had initially used an authorize_callback for this purpose, but didn't like that I was storing the credentials for a user who was not actually authorized. Once they're stored, you'd need to delete them in the authorize_callback otherwise oauth2.required will be able to fetch them and allow access. I believe it's cleaner and safer to  prevent the credentials from being stored rather than storing then deleting them for unauthorized users. Perhaps the storage.put and the authorize_callback could swap their order and that'd solve this? I haven't checked if that'd cause other problems.
I also tried a wrapper around oauth2.required to check the domain, but had a problem when using that with redirect loops that I thought this approach fixed, but right now I'm seeing a similar problem again so it may be unrelated. It might actually be an issue with a session token being updated but not saved -- but I digress, although it may be another issue in a few minutes.

Interesting. The reason I ask is because I'd rather figure out how to make flask_util flexible enough to handle this case without hardcoding it. Do you think if we made it where if authorize_callback returns False it aborts?

I think it could work if authorize_callback returns False and it is called before the credentials are stored. Most existing usages of authorize_callback probably return None, however they could possibly return False which would then make this a breaking change. But if the authorize_callback redirects to, say, an application's logged out page, would that be sufficient without the callback view needing to evaluate its return value?
I see your point about flexibility, and the above approach may be better, but if it's not I wouldn't consider using a kwarg that is already recognized by OAuth2WebServerFlow to be 'harcoding' something in. The entire change I made to the callback view was
domain = flow_kwargs.get('hd', False)
if domain and credentials.id_token.get('hd') != domain:
    return ('Unauthorized: you must sign in with an account belonging '
            'to the {0} domain.'.format(domain), httplib.FORBIDDEN)

I've looked around and seen a few other questions where people want to restrict to a domain -- not necessarily with flask, but I do think it's a common enough requirement that some optional behavior based on domain isn't obscure.

Any resolution on this?  I'm trying to do the same in Django and am coming up short without overriding the callback.

I've been using the above patch successfully. If you end up with a different Django solution I'd be interested to see it.

@rfinck I agree that the ability to restrict to a domain would be useful, especially for people working under G Suite accounts that don't want to go with a complicated SAML SSO solution. Did you fork this library to integrate your patch or just override the callback_view() method? I'm trying to determine the most simple way to restrict to a company domain without being hacky.

I had forked it thinking that'd be a temporary fix.
@jonparrott, what are your current thoughts on this (or another) solution?

My current thoughts are that it's highly unlikely I'll be adding any new
features to this library.
On Thu, Mar 30, 2017, 8:27 AM rfinck<EMAIL_ADDRESS>wrote:

I had forked it thinking that'd be a temporary fix.
@jonparrott https://github.com/jonparrott, what are your current
thoughts on this (or another) solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/oauth2client/issues/677#issuecomment-290447093,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc6KOPy1T4Evr-SFPDEezoBeijtUsks5rq8n3gaJpZM4K3ZV9
.

Thank you for creating this issue, however, this project is deprecatedand we will only be addressing critical security issues. You can read moreabout this deprecation here.
If you need support or help using this library, we recommend that you ask yourquestion on StackOverflow.
If you still think this issue is relevant and should be addressed, pleasecomment and let us know!


Suggestion: Add Rageshaper to Tailwind of the Hardy (and possibly one of the Shifter archetypes to Good Hunting?)

Since the Shifter came out well after Midnight Isles, it feels like it's kinda sidelined as an option for the tailwinds, and I figure these two are the two that could fit it best.
Between Rageshaper having a direct tie to Barbarians and Bloodragers (fitting the standard for the archetypes available for the tailwind), and Shifters in general having a 'hunting' flavor to them, it feels empty not having them be options for these tailwinds.

Seems like a good idea. I'll put it in my bucket when I look at this mod again.


add default_background option to AnnotationResponse type

Description
This PR adds the default_background variant to the color property in RichText's Annotation type.
Related Issue

#536

 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.


RS bridge exportItemToPeripheral Error

Descripe
If you look at the peripheral device command, you can see that there is inventory in "left".
When trying to extract an item from RS to inventory of left through exportItemToPeripherial, an error occurs saying that there is no inventory in left.
Steps to reproduce
I installed a computer on the right side of minecolony warehouse and proceeded.

rs = peripheral.find("rsBridge")
rs.exportItemToPeripheral({name="minecraft:stone", count=1}, "left")

Multiplayer?
LAN
Version
1.18.2-0.7.16b (Latest 1.18)
Minecraft, Forge and maybe other related mods versions
Forge 40.1.69 Minecraft 1.18.2
Screenshots or Videos
No response
Crashlog/log
No response

Following
You wrap the RS Bridge so you need to connect the warehouse also with an modem
You could place it at the modem or connect the bridge and the warehouse with modems & cables


Popup screen relocate

Describe the solution you'd like
Is it possible to move the popup window
eg
--Centre
--bottomleft
--topleft
--topright
--bottomright

Hi Eugene,
thanks for your request. We're already working on this feature for pop-up UIs but we don't have any ETA yet.
I hope you don't mind if I rename this issue to better describe the FR. You can follow the updates of the future related deployment from here.
Thanks

Don’t mind at all , thank you

This FR is currently in QA Process


[UBS] POST Client controler make order again

Environment: mac OS Big Sur 11,2 , browser Firefox 91.0b9
Reproducible: always
Preconditions
user should be logged
user should have more than one done order
Steps to reproduce

Go to ' greencity-ubs.azurewebsites.net/ubs/client/{id}/make-order-again'
Select POST method
submit the action

Actual result
{
"timestamp": "2021-08-12T20:25:43.197+0000",
"status": 500,
"error": "Internal Server Error",
"message": "No enum constant greencity.entity.enums.OrderStatus.PAID; nested exception is java.lang.IllegalArgumentException: No enum constant greencity.entity.enums.OrderStatus.PAID",
"path": "/ubs/client/601/make-order-again"
}
Expected result
{
"bagOrderDtoList": [
{
"bagAmount": 0,
"bagId": 0,
"capacity": 0,
"name": "string",
"price": 0
}
],
"orderAmount": 0,
"orderId": 0
}
User story and test case links
E.g.: "User story #2196
Labels to be added
"Bug", Priority ("pri: high"), Severity ("severity:critical"), Type ("Functional"), "API" (for back-end bugs).

We don't have orders in our db with status PAID, and status PAID cannot be applied to order, so i don't have an idea, how you can get this error message; P.S. Controller works fine.


Fix out-of-bounds and reversed TextBox queries in computing caret metrics

Fixes #50563.
Fixes #98516.
Fixes #118403.
Fixes #122477.
It turns out that Paragraph.getBoxesForRange will not accept a negative index for the start of the range; if you pass one, it will return an empty list.  We'd been doing that in TextPainter._getRectFromUpstream; this was causing #50563, #98516, and #118403.  (Arguably this makes them duplicates of each other, but they looked quite distinct until being debugged: they involve complex emoji, zalgo text, and Indic script respectively.)
Fix that by clamping the start to zero, because what we really intended here in that case is the whole text up to the given end offset.
Then it turns out that we were sometimes reaching this line with a negative end index too (when the selection is empty on reaching RenderEditable._updateSelectionExtentsVisibility.)  That was giving an empty list of boxes, but if we clamp the start but not the end, we'd get back boxes for the whole text instead.  Avoid that by short-circuiting that case at the top of the function, like we do when the offset is out of bounds in the other direction.
Finally, when we get back more than one box from getBoxesForRange, we were looking at the wrong end of the list: if we're looking upstream, we want the end of the last preceding box, not the first one.  Similarly when looking downstream we want the start of the first following box, not the last one.  This was causing #122477, which I discovered while studying this code to diagnose #50563. Fix that too.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

There are also several issues in the tracker that sound potentially related to the issues this fixes but involve RTL text:

#78660
#110416
#118566
#120049
#120604

or WidgetSpans:

#107432

I haven't yet investigated whether this fixes any of those issues.

Resolved conflicts after #120834.

Revised. Thanks for the review!
I took the liberty of adding a couple of TODO comments referring to you (@LongCatIsLooong). They come about because I ran into this existing TODO comment:
    // TODO(LongCatIsLooong): skip painting the caret when the selection is
    // (-1, -1).
    if (selection == null || !selection.isCollapsed) {
      return;

From the Git history, I see that's from https://github.com/flutter/flutter/issues/79495 , for which a previous fix was reverted because fixing it was a breaking change.

cc: @exaby73 for later reference https://github.com/flutter/flutter/issues/120049#issuecomment-1476910624

Thanks for the review! Pushed a version with those tweaks to the tests.

[√] Flutter (Channel main, 3.9.0-20.0.pre.91, on Microsoft Windows [Version 10.0.19044.2728], locale he-IL)
• Flutter version 3.9.0-20.0.pre.91 on channel main at C:\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 3a1a47c243 (22 minutes ago), 2023-04-03 05:51:18 -0400
• Engine revision 0ed4b5ecbc
• Dart version 3.0.0 (build 3.0.0-401.0.dev)
• DevTools version 2.22.2
`
TextEditingController tec = TextEditingController(text: "בדיקה בדיקה בדיקהבדיקה בדיקה בדיקה
שדגשבדיקה בדיקה בדיקהבדיקה בדיקה בדיקה
שדגש");
TextField(
controller: tec,
keyboardType: TextInputType.multiline,
maxLines: null,
textDirection: TextDirection.rtl,
textAlign: TextAlign.right,
)
`
Issue persists with simple textfield and hebrew text within when using multilines.
Keyboard arrows are reversed and cursor jumps to the one before last latter when double-clicking a word (web)

Issues persist when using multiline text
TextField(
keyboardType: TextInputType.multiline,
maxLines: null,
textDirection: TextDirection.rtl,
textAlign: TextAlign.right,
)

@RomanIvn Please file a new issue for the problem you're seeing. That will be a more effective place to address it than in a merged PR's thread. Do include all the information in the issue template, including details of what behavior you expected and what behavior you see instead.
Since you mention TextDirection.rtl, take a look also at these still-open issues and see if one of them describes what you're seeing:

There are also several issues in the tracker that sound potentially related to the issues this fixes but involve RTL text:

Arrow keys in RTL move the wrong way #78660
Incorrect cursor position in RTL TextField when moving between letters #110416
Text field cursor position in RTL goes to the penultimate character #118566
RTL textField Cursor movement has a problem with the last position and Emojies and text selection #120049
TextField cursor stands in a wrong position while using persian words #120604

or WidgetSpans:

WidgetSpan wrong caret position #107432

I haven't yet investigated whether this fixes any of those issues.


Revamp event mechanics

Closes #1889
Introduced changes

Revamp event mechanics

Checklist

[x] Linked relevant issue
[x] Updated relevant documentation
[ ] Added relevant tests
[x] Performed self-review of the code
[x] Added changes to CHANGELOG.md

We decided on different approach so I wouldn't break it twice. If you disagree with event emission writing stuff to vm memory implicitly feel free to discuss it on Slack

^ Yep, we'd like to start this when we start the refactor and use different approach (more compilcated than this, requires keeping segment info in the runtime and writing to that segment dynamically on event). Closing for now, sine we don't want to start this refactor yet.