Rds backups

New Feature to add scheduled database backups that push .sql files over to an S3 bucket for each infrastructure.
Todo:

Finalise task definition and link it to an ECS Service so it can run
Determine if we want to host the container image available here: https://github.com/dxw/dalmatian-sql-backup on GitHub (and authenticate with a token) or on ECR. I think ECR would be best if we can add a workflow to the repo that pushes up to the ECR?
Determine if we need cloudwatch logs for this task? If so, they need connecting up to push the logs to the same place as the other ECS tasks

We can lose the changes to the s3 policies

Related PR https://github.com/dxw/dalmatian/pull/710

Hi - is this still something we want to go in?

Hi - is this still something we want to go in?

Hello, yes this is porting functionality over from v1 Dalmatian to v2 :)

Closing due to https://github.com/dxw/terraform-dxw-dalmatian-infrastructure/pull/136 happening before this was finished


Deployment of a performance\load testing engine build space in Azure

Describe the issue
Build space to be deployed for a JMeter build using a scalable load testing architecture for both external public facing performance tests and private\internal testing
Additional context

Solution to possibly include 'scale sets' to allow for autoscaling where needed and will remain off once initial testing done until it is needed again

Definition of done

Meeting to discuss how this could move forward
Deployment of Azure environment to test how stable SDN network is for performance
Confirmation and light handoff with Brian P
Brian to build the IaC based solution and do all testing

Meeting held on how this could initially be set up for the SDN load testing but further discussions to be held on furthering concept for a potential general service for at least OCIO internal (ticket# 1359).  Environment was created in Azure and hand off meeting held.  Brian to start build for this and run cost compares for various levels of testing which will be reviewed for a wider service discussion (#1359).  Closing ticket


vidmix: allow different pixel format

this looks fine.
Another option is to take the pixel-format from the first incoming video frame.


Can't get the picked date value on v-model

Great module but i can't get the value, added a v-model it's not work, how to get the value?
Thanks,

Hello, I faced this same problem as you, to solve it I added an emitter in the pickDate this.$emit('update:value', this.pickedValue) and heard this change in the component v-on:update:value="bindedValue = $event"

See also, you may add :value.sync in the component.
See documentation https://br.vuejs.org/v2/guide/components-custom-events.html#Modificador-sync

Thanks for your answer @wandersoncmendes unfortinately it's not work:
<datepicker id="date" :readonly="true" format="YYYY/MM/DD" name="date1" :value.sync="chosedate" :input-attr="{placeholder: 'Enter a date'}" :disabled-date="disabledDate"></datepicker>
when i call this.chosedate  to get the date value chosen, it's return null, even if i call document.getElementById("date").value) it's return null as well. any idea ?

Hello,
from what I realized the component does not support this .sync modifier.
I requested the inclusion of this support in the link
adding .sync modifier support
I am waiting for the return of the project owner.

Thanks for your response, Any other way to get the date picked. I want to get the date and send it to the server. if you find a way to do so, please let me know.
Thanks,

While the change is not approved, you can use an advanced technical feature (laughs) to get the value. Add an id attribute inside input-attr and get the value using document.getElementById
Exemplo:
<datepicker       :readonly="true"       :value="valor"       format="YYYY-MM-DD"       name="date2"       :input-attr="{ 'id': 'date' }"     ></datepicker>
Get a value:
document.getElementById('date').value


Stop using ament_target_dependencies.

We are slowly moving away from its use, so stop using it here.  While we are in here, notice some things that makes this easier:

pluginlib is absolutely a public dependency of this package. Because of that, we can just rely on the PUBLIC export of it, and we don't need to link it into every test.  But that also means we don't need some of the forward-declarations that were in loader_fwds.hpp, as we can just get those through the header file.
republish.hpp doesn't really need to exist at all.  That's because it is only a header file, but the implementation is in an executable.  Thus, no downstream could ever use it.  So just remove the file and put the declaration straight into the cpp file.

Linux
Linux-aarch64
Windows

@clalancette should we backport this to jazzy ?

https://github.com/Mergifyio backport jazzy


[BUG]: Snapshot tests failures

Commit 7a41ae8edc1f0f9a8e969d9f3cbf28a6daa3cfa8 introduced failures in 2 tests:

snapshot::tests::can_read_snapshot_after_writing
snapshot::tests::creates_all_dirs_while_writing_snapshots

There is a linking error that seems to have been introduced in the same commit: https://github.com/hyperledger/iroha/actions/runs/9284079730/job/25545677112#step:5:742

@nxsaken i can't see the issue in latest CodeQuality jobs for some reason


Fix /_admin/cluster/health if one agent gone

This is a rather trivial bug fix which addreses BTS-584.
It fixes a regression on /_admin/cluster/health if an agent is off line,
since the health endpoint retried for too long to reach the agent before
giving up.
This is a trivial fix which is covered by QA tests.

Good run: https://vpna.arangodb.biz:3456/view/PR/job/arangodb-matrix-pr/17090/

Last failures due to Jenkins, PR itself seems fine


Assertion `1 == ksp_dims[MAPS_DIM]' failed

bart pics -l1 -r0.001 file_ks file_sensemap image_out
bart: src/pics.c:230: main_pics: Assertion `1 == ksp_dims[MAPS_DIM]' failed.
Aborted (core dumped)

file_ks: size = (1, 8, 256, 320, 4), 8 is the number of coils, 4 is the slices
file_sensemap: size = (1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 8, 256, 320, 4)

Hello,
In BART, the locations of the dims are important. Please reshape both arrays to (256, 320, 4, 8, 1)

Please refer to the mailing list for help.


All positional operator $[] throws JsonParseException in 1.5.0

After upgrading Jongo 1.5.0, we started seeing JsonParseException in a couple of places where we use all positional operator $[]. Our usages are the same as the example in MongoDB doc.
BsonQueryFactory#addRequiredQuotesAndParameters seems like the culprit as it applies the following transformations, and therefore BasicDBObject.parse fails:
{ $set: { array.$[].id: 10 } } -> {"$set":{[  array.$]".id": 10 } }

My test case in BsonQueryFactoryTest:
@Test
public void allPositionalOperator() {
    assertThatCode(() -> factory.createQuery("{ $set: { array.$[].id: 10 } }")).doesNotThrowAnyException();
}

Failure:
java.lang.AssertionError:
Expecting code not to raise a throwable but caught
  "java.lang.IllegalArgumentException: Cannot parse query: { $set: { array.$[].id: 10 } }
	at org.jongo.query.BsonQueryFactory.createQuery(BsonQueryFactory.java:106)
	at org.jongo.query.BsonQueryFactoryTest.lambda$allPositionalOperator$0(BsonQueryFactoryTest.java:390)
...
Caused by: org.bson.json.JsonParseException: JSON reader was expecting a name but found '['.
	at org.bson.json.JsonReader.readBsonType(JsonReader.java:146)
	at com.mongodb.DBObjectCodec.readDocument(DBObjectCodec.java:397)
	at com.mongodb.DBObjectCodec.readValue(DBObjectCodec.java:322)
	at com.mongodb.DBObjectCodec.readDocument(DBObjectCodec.java:399)
	at com.mongodb.DBObjectCodec.decode(DBObjectCodec.java:167)
	at com.mongodb.DBObjectCodec.decode(DBObjectCodec.java:67)
	at com.mongodb.BasicDBObject.parse(BasicDBObject.java:83)
	at com.mongodb.BasicDBObject.parse(BasicDBObject.java:70)
	at org.jongo.query.BsonQueryFactory.createQuery(BsonQueryFactory.java:99)

Also, Jongo 1.4.1 returns the expected query:
{"$set": {"array.$[].id": 10}}

You can add quotes around your key, the following test runs as expected
@Test
public void allPositionalOperator() {
    assertThatCode(() -> factory.createQuery("{ $set: { 'array.$[].id': 10 } }")).doesNotThrowAnyException();
}

I’ve been trying to modify the query builder to accept the query without the quotes but it is quite tricky to do so without introducing a lot of complexity to the algorithm. Considering the doc you mentioned had those quotes I’m not sure we want to fix it.

Thanks @Dayde. It's still surprising this works in Jongo 1.4.1, though. 🙂

Not really to me as the query factory was rewritten from scratch while updating to mongo legacy driver 4.x


fixes for labs

Issue #, if available:
Description of changes:
Hi @ryvan here. I went through your lab and found a few issues

does not deploy to us-east-1 due to quirk in API/boto syntax
CLI parameter value changes
missing step in directions of putting DeepStream onto a fresh Jetson Nano / JP 4.4 install

I've corrected all of these items in the PR and fixed some general typos. However some feedback, at the end it feels incomplete- it tells you go to to another lab that looks for a public key cert not provided in the previous instructions. I'd recommend it just goes through the steps to get at least one example working so it feels like something is accomplished.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

+1 for the fix of the malformed CF stack parameters passing in the deploy script. I also came across this bug.


croc-hunter-jenkinsx to 0.0.141

Promote croc-hunter-jenkinsx to version 0.0.141

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Better support for testing with different storage drivers

This change updates the test helper macros to pass the right flags to tools to use the storage driver named in $STORAGE_DRIVER instead of vfs, if the variable is set when the tests are run.  It adds debug logging support to the imgtype test helper, and ensures that it actually shuts down storage, even when it exits with errors.

LGTM

@TomSweeneyRedHat PTAL

@TomSweeneyRedHat Can we merge?

@rh-atomic-bot r+ ccfde8a


No more js/config.js for developing from file://

More often than not, this results in an API getting checked into someone's repo. There are lots of options here.
My thought, a simple JS library that just wraped jquery's getJSON method if run from within a locker, or creates a hidden iframe loaded from the locker and uses postMessage to do ajax from within the iframe, avoiding the cross origin stuff altogether.

Looks good, +1 here, only thing I'd suggest is making it API() or something more generic, not using Singly() :)

Ya, I thought about that. It's a bit generic, but I think it will work.

+1, but in contrast to @quartzjer I like the idea of having it be named Singly(). Too many generic APIs out there already!

I've had problems in the past with using company and/or possibly trademarked names in code, always good to avoid and be generic if possible, esp if the symbol will be hard-wired for a long time :)

That's a good point -- maybe we can give it a non-Singly but still distinctive name. I suggest Trukkknutttz!

I like this, but checking for the 'file:' protocol might be too narrow (it excludes people using their own webservers, which is how I work). I think checking for the 'file:' protocol or a non-singly domain would make this work for both uses cases. :)

the problem with a non-singly domain, is that running in a local (development) locker on localhost or lvh.me will force you to enter an api key. What I'll do is move the baseUrl out a a global, so that you can use a config.js still to set it for local development if you want to run behind a local server and it won't override.

This is merge into master. Further improvements can be new issues.


S3 bucket handling broken in v1.3

I'm trying to upgrade my project to JAWS v1.3, and got some problem with S3 bucket management. jaws new project went fine, I added module, did jaws dash deploying lambda and endpoint, went fine. My lambda uses dynamo, so I added respective entries to resources-cf.json in my stage and tried jaws deploy resources and it fails, because bucket myapp.com already exists. Not in my account, though. Also, I was not asked about it during previous stages (during jaws new project I provided custom names for everything). So, I fixed the problem by editing default value of aaJawsBucket in resources-cf.json for my stage.
Therefore, I guess this can be improved somehow, this issue is to raise a flag about it.
Side question is - Default property of aaJawsBucket suggests that I can provide custom value to override default. How? Using env vars?

To be clear, this seems to be regression from v1.2, where such situation had no place.

@Nopik great catch.  I, like you, do not want jaws creating or managing my s3 bucket that is storing my jaws files (env vars, lambda zips, CloudFormation jsons etc).  I just haven't had time to test the non default scenario in v1.3 yet (its still a WIP btw).
@ac360 This looks like an oversight in the 1.3 jaws bucket refactor.  In addition we should not default the value for aaJawsBucket to the domain name.
@Nopik I'm surprised your jaws new project completed successfully, since the resources-cf.json creates the bucket.  Did you happen to specify the -d flag on new project?
I will make a CLI option on the new project command that lets you specify an already existing s3 bucket to use as the jaws bucket.  If specified, it will not be created (or managed) by resources-cf.json.  Also if specified I will set the default value of aaJawsBucket to the option value.  I debated removing aaJawsBucket all together in this scenario, but I think for consistency we need to leave it in so other items in the CF can ref it in the future if needed.

I did just like the README.md says, plain old jaws new project and answered all the questions. No -d flag, so I have no idea why it didnt failed.

ok thanks @Nopik . this will get fixed

On it right now

Should be fixed in 6bedcc2a1b8068e4030a215bf6897e20fef4d597
Re-open if not


V2 of Writer Attribution – Pull From Github History

tl;dr (added by @bnb):
Automation of credit for content is vital if we do want to be inclusive and give credit where credit is due.

A couple of examples of this in the wild:
Example from Webpack:

Example from docs.microsoft.com:

Surprisingly, quite a few projects and platforms (I checked Babel, Vue, Angular, React, Parcel, Probot, GCP, AWS, DigitalOcean, Rust, Go, Kubernetes, Docker) don't do this. If you do know of others that do it, please include screenshots! 🙏

I suppose this was fixed in #215. Feel free to re open if not.


⚠ Bofrost.es has degraded performance

In 0656fea, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1587 ms

Resolved: Bofrost.es performance has improved in 94ecfbc.


New unit: Skylancer, Skylord rework

What's changed

Changes Skylord to use 4 interceptors, doing 218 air damage and 25 ground damage (ground attack is the same as regular interceptor's)
The old Skylord laser attack is moved to a new unit - Skylancer, that's now a Tempest variant, otherwise behaves like Skylord before
Skylancer has a new upgrade that doubles its attack speed but overheats while attacking (stops at original attack speed)

Motivation

Bring some consistency into the game as the upgrade paths for Carrier class are somehow inconsistent (Hull of Past Glories) but Graviton Catapult is impossible to fit the old Skylord
Unit shapes make their behavior more predicable
Skylord more matches its name (big air unit that's a great anti air)

Misc
See generator PR https://github.com/Ziktofel/Archipelago/pull/380

As discussed in the discord, I don't think we want another carrier unit, and I don't think it's important that Graviton Catapult applies to the whole class. Most people are against this change. I am neutral on renaming the Skylord to Skylancer; I like the old name largely because I'm used to it.
I'm against the model change for the Skylord as I designed the unit with the model in mind. The inspiration came from the sc1 opening art of carriers frying the surface of Chau Sara, so it feels weird to put the laser on a Tempest model. The precise laser model seemed to wrap around the contours of the Skylord model. And the larger form of the carrier-type hull gives the player impression of bulk, which matches with the unit's large health pool, which in turn felt appropriate to encourage the player to move the unit forward to get the most of the linear splash.
The increased attack rate to start is an interesting idea, but I'm not that big a fan personally. The slow attack rate of the Skylord would often lead to situations where the player would intentionally stagger the attack cycles of multiple skylords to make sure there was one in a small group that could react to a new anti-air attacker approaching. Having a variable attack rate makes this more difficult. A short-range blink, similar to a mothership, was requested and something I'd like to see on the skylord; the blink could reset the attack cycle if we want some way to increase the attack rate. Note the damage numbers and attack rate have already gone through a balancing / tuning phase following feedback, I'd expect another such phase would probably have to follow if we applied a general attack speed buff to the Skylord. Someone also pointed out that giving more power at the start of an engagement is a little odd on a unit that can just teleport out at any time.
If it's a high priority to add a new tempest-class unit or add more Protoss items, I can look into making a new Tempest variant or reworking the Warp Ray or splitting up / adding protoss upgrades to get more items on request; there are some tracker changes I was working on otherwise. I don't think it's that urgent, though, and I don't think it's worth adding repeat units or more generalist units like this when there isn't urgency.
There were many other comments in the discord, but suffice to say I don't think anyone really wants these changes in general. To me the largest point against is that I don't see any kind of artistic or design intent here -- it feels like moving things around for the sake of having another carrier, when we already have two carriers and that's already more than we need for a randomizer. The justification of having graviton catapult apply to more units feels unwarranted, as we generally want more items that apply to fewer units, with sentiment suggesting we should break protoss items up to apply to individual units rather than have the same items apply to move units. Additionally, a carrier variant, even one specialized into anti-air, that can still hit ground is enough of a generalist that it detracts from the randomizer experience. The most interesting situations are when the player is saddled with only niche/specialist units and must cobble them together into a strategy; this situation lapses as soon as the player collects 1~2 generalists. Protoss is already overloaded with generalists that can handle most situations, it does not need another one that is survivable, low-micro, and can hit both ground and air.


regctl tag rm seems to leave dangling/empty tag in docker hub

At  https://github.com/rgl/example-docker-buildx-go/blob/add-windows/.github/workflows/build.yml#L143 I'm using regctl tag rm to remove a tag, and indeed it removes it, but for some reason, they still appear in docker hub Tags tab, but without any image:

Do you have any idea why this is happening?

Unfortunately, Docker Hub doesn't support the delete API. They have been adding a Hub specific API under the paid tier that I'm considering supporting, but the implementation for that is rather kludgy at the moment (the API call goes to a different hostname, and the auth goes through a separate path).
As for why the empty tag is created, the tag delete goes through a couple steps. The first is to try the newer OCI tag delete API. This isn't implemented by many registries yet, so the fallback is to push a blank manifest with a unique digest and then delete that digest. This leaves any other tags on the registry that point to the same manifest, which you want when deleting a single tag. But it has the downside of leaving the blank manifest if the registry rejects the manifest delete API.

Thank you for the explanation!
By any chance, do you known whether the GitHub Container registry implements this?

I did a quick test and only managed to get a "405 Method Not Allowed" from ghcr.io. They do have a delete package role that can be added to a PAT, but I'm guessing that's for a GH specific API.


RON incorrectly adds extra level of Base64 when roundtripping Bytes

This code (playground):
#[derive(Debug, serde::Deserialize, serde::Serialize)]
#[serde(rename = "b")]
struct BytesVal {
    pub b: bytes::Bytes,
}

#[derive(Debug, serde::Deserialize, serde::Serialize)]
#[serde(untagged)]
enum Bad {
    Bytes(BytesVal),
}

fn main() {
    let v: Bad = ron::from_str(r#"(b: "dGVzdA==")"#).unwrap();
    dbg!(&v);
    let s = ron::to_string(&v).unwrap();
    dbg!(&s);
    let v: Bad = ron::from_str(&s).unwrap();
    dbg!(&v);
    println!("---");
    let v: Bad = serde_json::from_str(r#"{"b": "dGVzdA=="}"#).unwrap();
    dbg!(&v);
    let s = serde_json::to_string(&v).unwrap();
    dbg!(&s);
    let v: Bad = serde_json::from_str(&s).unwrap();
    dbg!(&v);
}

has the following output:
[src/main.rs:23] &v = Bytes(
    BytesVal {
        b: b"dGVzdA==",
    },
)
[src/main.rs:25] &s = "(b:\"ZEdWemRBPT0=\")"
[src/main.rs:27] &v = Bytes(
    BytesVal {
        b: b"ZEdWemRBPT0=",
    },
)
---
[src/main.rs:30] &v = Bytes(
    BytesVal {
        b: b"dGVzdA==",
    },
)
[src/main.rs:32] &s = "{\"b\":[100,71,86,122,100,65,61,61]}"
[src/main.rs:34] &v = Bytes(
    BytesVal {
        b: b"dGVzdA==",
    },
)

It fully survives the roundtrip with serde_json, but adds extra level of Base64 with RON.

This really is a tricky one! I think though that I now understand what is going on here. The problem is the #[serde(untagged)] which means that the ron string is deserialised without having any type information available. Since ron encodes bytes as base64 encoded strings, but without declaring that they are bytes specifically, the deserialiser just sees a string when no type information is available. When bytes::Bytes then deserialises, it sees a string and just gets its bytes. I.e. even though they were base64 bytes, at no point was ron asked to turn them back into bytes.
ron in general is a format that does not support deserialising without any type information (i.e. anything that touches deserialize_any). However, I agree that this is a very confusing and unfortunate case. The only way of fixing this (to work with untagged enums) that I can think of is to make a breaking change to ron's format in how bytes are represented. It could be as simple as prefixing any byte string with b, i.e. b"base64" during serialisation. During deserialisation, we could continue to accept a string without the b-prefix if type information is available. However, old ron code would no longer be able to read files produced by new code. Those would be tough decisions to make ...
@torkleyy What are your thoughts on this?

I think having special support built-in for bytes / base64 encoding can be an advantage, even if it means breaking some compatibility. I'm not sure if we can ease the transition somehow to not cause any breakage in practice.


HCAP-1341 Remove additional unneeded sorting

https://freshworks.atlassian.net/browse/HCAP-1341
I guess lodash and postgres treat spaces differently in their sorting algorithms

removed extra sort on FE for sitenames

Great call! thank you!

I think this will deal with some of the sorting behaviour, but postgres's sort is also case-sensitive so I'm thinking there may still be some issues. In the raw query you can do lower()/upper() on the order by field but not sure if massive will cooperate.

I think this will deal with some of the sorting behaviour, but postgres's sort is also case-sensitive so I'm thinking there may still be some issues. In the raw query you can do lower()/upper() on the order by field but not sure if massive will cooperate. !

It should at least get the sorting to be consistent between the two components which was my understanding of what this ticket was describing


pool-eu-001-anycast.mc.tritondigital.com is breaking Podcasts

This is used as a cname for Podcasts like
derspiegel.mc.tritondigital.com and you can't get any episodes on your device without whitelisting it.
Other subdomains like https://22523.mc.tritondigital.com and the main domain should be checked for further false positives.

Yes, seems it's a false positive, thanks. Though they have "Audio Advertising", "Audience Measurement", "Programmatic for Buyers" and this domain was in many lists, but it has been removed almost from everywhere.
Also, it seems they have tritondigitalmedia.com, it has been removed too.
Source list was updated, hosts files will be updated soon.


enable CodeQL checks

Contributes to: https://github.com/confidential-containers/confidential-containers/issues/46#issuecomment-2309559260
Please enable CodeQL analysis based on https://github.com/confidential-containers/operator/commit/1bde94e31996ec708eeb9c6c4bdb4f762881ec43

@lmilleri would you be able to take this based on https://github.com/confidential-containers/cloud-api-adaptor/pull/2158
I'd like to get https://github.com/confidential-containers/confidential-containers/issues/46#issuecomment-2309559260 to get the OpenSSF Badge passing...

unlike other repos, this is now enabled without adding any workflows. closing for now


SCCP-377

@noisekit when you can please 🙏🏼 i think my tests are fine, and it's having false positives on arbitrum? Thank you sir,

adding the badly needed tokens now, fartcoin.
honestly db, some of these coins have a lot of story behind them  😅
https://www.infinitebackrooms.com/dreams/conversation-1721540624-scenario-terminal-of-truths-txt


Add error check venture for assigned ventures

Plugin will loop assigning a venture if an error is on screen. Aimed to alleviate weird game issues where it bugs out and locks out the assign for no reason.

manually merged https://github.com/PunishXIV/AutoRetainer/commit/0504eec8113fb86e65605012efbc155233f80b6e


Initial Implementation and Operationalize Nucleus

Develop baseline pipeline for EN Tools and Services necessary for all nodes along with initial documentation outline for extension and adaptation.

📆 January status: deferred from B12.1 beta dev tasks as #1 was delayed due to unavailability of resources

@ramesh-maddegoda I created numerous requirements for user guides for Nucleus. Have a look and we can discuss in more detail. I am thinking we make this documentation on github.io? if so, you will want to use the python-template repo to add those Sphinx docs here

📆 January status: Detailed architecture complete. User guides in work. On schedule

📆 February status: Cloud deployment strategy trade study completed. Initial implementation and architecture nearly complete for MVP. On schedule


New Patient creation screen UI

Replace Sexual Orientation with "Gender Identity" dropdown.
FHIR extension > Patient.extension:genderIdentity
Dropdown:
Identifies as nonbinary gender
Male-to-female transsexual
Female-to-male transsexual
Identifies as non-conforming gender
Identifies as female gender
Identifies as male gender
Other
Unknown
Asked but declined
https://vsac.nlm.nih.gov/valueset/2.16.840.1.1137<IP_ADDRESS>1.32/expansion

Modified the UI accordingly, below is the screenshot

Add sexual orientation https://build.fhir.org/ig/HL7/cdmh/CodeSystem-pcornet-sexual-orientation.html
Alphabetize dropdown items
Change "Gender" to "Administrative Gender"
Move "Sex at Birth" "Administrative Gender" and "Gender Identity" closer together
Add tooltip to "Administrative Gender" (and other two). Tooltip for Admin Gender = "Also called legal gender or gender marker."

Modified Accordingly, below is the screenshot:


refresh Glue docs, provide dev hub app shortcut

This PR contains the following changes:

Refresh the Glue service docs according to our new guidelines.
Make sure every single sentence is in its own line in the markdown file.
Restructure the Developer Hub code and provide a shortcode which renders all developer hub samples which integrate with a specific service.

Preview Deployment
The result looks like this:

This could be very easily integrated for other services (with a single line of code).
I think this is a nice promotion of the developer hub apps directly on the docs page of the respective services.
This is where I would like to get input from @HarshCasper @tinyg210 @lukqw if this makes sense or if I should revert the changes.

Thanks for the input!
Based on your feedback (and with @lukqw's help with adjusting the styling), I now also extracted the tutorial partial and created a tutorials shortcode.
The result looks as follows:


.NET Events

This adds .NET native events for the following Gearman events:
JobCreate
JobComplete
JobData
JobFail
JobException
JobStatus
JobWarning
Unit tests written for create (event called), and complete (event called + verify data payload) + example code added to ExampleClient

Thanks @issacg
Sorry for not responding earlier. But as you might have figured out, we don't actively use or develop this library. The patch looks legit but it wouldn't feel right to merge without actually using it.
I'll close this PR, hope that people will find and use your fork!
Kind regards, Johan


How to train the model with private data?

Hi,
I have a collection of data for India and I would like to obtain a proprietary model with this data collection.
I looked around the available documentation but I was unable to find the process to do that myself..
Question:
Do we have a full documentation on how to prepare our own data and run individual pipeline to create a proprietary model by myself?

My country is
USA

This has been discussed in a few other issues, but we don't encourage training custom models (and cannot provide support for models trained on proprietary data sets as it's difficult-to-impossible to debug a parser that's not performing as well as expected without knowing how the training data was constructed).
In general we'd like to get more addresses for India out in the open. What we have is several orders of magnitude less than the population, and does not cover all regions of the country proportionally, so it would be great if there are sources, especially with landmarks, etc. that can be opened up and used in this project.

Thanks a lot for the answer and the patience.

@albarrentine I couldn't find an email address for you.
Are you available for consulting? i.e. to train libpostal on a specific area and specific (messy) addresses?


Decoding of negative numbers

I notice that the BInt data structure uses u64 whereas the BitTorrent Spec seems to require handling negative integers: https://wiki.theory.org/BitTorrentSpecification#Integers
The BInt data structure and bint_decode function should be changed to allow negative numbers

Spot on, my bad 😳
Thanks a lot for the pull requests, I've added some comments to it.

Merged, closing, thanks!


Two records with negative tested values.

@reustle
There are two records where the tested value is negative.
[
  {
    "confirmed": 36,
    "confirmedCumulative": 927,
    "deceased": 2,
    "deceasedCumulative": 32,
    "recovered": 0,
    "recoveredCumulative": 144,
    "critical": 0,
    "criticalCumulative": 46,
    "tested": -1130,
    "testedCumulative": 14525,
    "active": 34,
    "activeCumulative": 751,
    "cruiseConfirmedCumulative": 712,
    "cruiseDeceasedCumulative": 7,
    "cruiseRecoveredCumulative": 458,
    "cruiseTestedCumulative": 4234,
    "cruiseCriticalCumulative": 15,
    "date": "2020-03-18",
    "confirmedAvg3d": 34,
    "confirmedCumulativeAvg3d": 870,
    "confirmedAvg7d": 41,
    "confirmedCumulativeAvg7d": 795,
    "deaths": 2
  },
  {
    "confirmed": 97,
    "confirmedCumulative": 1311,
    "deceased": 3,
    "deceasedCumulative": 45,
    "recovered": 21,
    "recoveredCumulative": 272,
    "critical": 2,
    "criticalCumulative": 57,
    "tested": -909,
    "testedCumulative": 23521,
    "active": 73,
    "activeCumulative": 994,
    "cruiseConfirmedCumulative": 712,
    "cruiseDeceasedCumulative": 10,
    "cruiseRecoveredCumulative": 597,
    "cruiseTestedCumulative": 4234,
    "cruiseCriticalCumulative": 11,
    "date": "2020-03-25",
    "confirmedAvg3d": 69,
    "confirmedCumulativeAvg3d": 1204,
    "confirmedAvg7d": 54,
    "confirmedCumulativeAvg7d": 1094,
    "deaths": 3
  }
]

I believe those were days that the gov actually retracted test results. @liquidx knows better than I do.

Yes, we take the raw data from MHLW for tested numbers and there were days where they removed tested counts because of miscounting.


UnicodeDecodeError

Expected Behavior
Work
Current Behavior
Doesn't work.
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 4864: character maps to <undefined>
Failure Information
Windows 11
Steps to Reproduce
gpte [presumably, anything containing Unicode] -i
Failure Logs
Traceback (most recent call last):

  File "C:\Users\J\AppData\Local\pypoetry\Cache\virtualenvs\gpt-engineer-woB2nRrJ-py3.12\Scripts\\gpte", line 6, in <module>
    sys.exit(app())
             ^^^^^

  File "C:\Users\J\gpt-engineer\gpt_engineer\applications\cli\main.py", line 273, in main
    files_dict = fileselector.ask_for_files()
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "C:\Users\J\gpt-engineer\gpt_engineer\applications\cli\file_selector.py", line 108, in ask_for_files
    content_dict[str(file_path)] = content.read()
                                   ^^^^^^^^^^^^^^

  File "C:\Users\J\AppData\Local\Programs\Python\Python312\Lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 4864: character maps to <undefined>

Extra
Was fixed in https://github.com/gpt-engineer-org/gpt-engineer/pull/801 , but looks like it got lost in a move. gpt_engineer/applications/cli/file_selector.py around line 107 is what needs fixing again

Thanks - let me put up a PR for this

Can confirm changing line 107 from with open(Path(self.project_path) / file_path, "r") as content: to with open(Path(self.project_path) / file_path, "r", encoding="utf-8") as content: works

I actually think I addresd this in my other PR theo (since I had the same
issue) can we try merging that first?
Anton Osika
On Mon, Mar 04, 2024 at 11:02:13, Theo Mc @.***> wrote:

Thanks - let me put up a PR for this
Can confirm changing line 107 from with open(Path(self.project_path) /
file_path, "r") as content: to with open(Path(self.project_path) /
file_path, "r", encoding="utf-8") as content: works
—
Reply to this email directly, view it on GitHub
https://github.com/gpt-engineer-org/gpt-engineer/issues/1032#issuecomment-1976199815,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCCSUIKP6K6ODD6E6P7TG3YWRBCLAVCNFSM6AAAAABD3DP6L2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZWGE4TSOBRGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Let survey subscribed and then navigate

@anandwana001 Can you please clarify what you mean in the bug description?


Broken Links for Build Dependencies

Hello! I was following the tutorial and I wasn't able to download the project xerces-c and eigen. It leads to this page below. Could you update the link of the page below. Thank you!
https://github.com/BioGearsEngine/core/wiki/Build-Instructions

I just updated the linking to the correct eigen version. There are a few more difficulties with xerces-c that might take a little longer.

@StevenAWhite will you review the binary releases of xerces-c and see how we want to move forward. Might need to test to see if older versions of xerces will work.

Hi all, looks like you need to use the newest version of xerces, I've updated the link to the build instructions. You'll need to download and build the newest version from source. I looked over the build instructions and they seem pretty straightforward.

Hi @hienschi , I'm going to consider this issue closed now that the links have been updates. Unfortuantely xercers is only supporting source code downloads so people building biogears will need to first build xerces. You can get around this on a linux/mac by using brew or apt-get package managers.

Thanks for updating it quickly! If I run into anything, then I'll let you know.

Thanks for updating it quickly! If I run into anything. I'll let you know.

Hienschi

On Tue, Feb 9, 2021 at 11:08 AM Austin Baird, PhD<EMAIL_ADDRESS>wrote:

Closed #52 https://github.com/BioGearsEngine/core/issues/52.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BioGearsEngine/core/issues/52#event-4310910345, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYQITSZPHCWXAMBFHP2CYLS6GBNJANCNFSM4XIB24SA
.


Add support for reading and writing bits. Add support for reading multiple memory ares. Migrate to safe Buffer constructor methods

This replaces PR #17
Add support for reading bits
Add support for writing bits
Add support for reading multiple memory areas (it handles both bits and words in one command)
Migrate to safe Buffer constructor methods (removes the warning messages)
I have updated the README and exmaples/basic.js to show the usage. I can provide Wireshark logs if you require them.

Added support for copying and transferring memory area command tonight. I am not planning on adding any more features to this PR now.

@CharlieMarshall fantastic work! I'll have time to closely look over this PR this evening. If it's not too big of a deal, could you attach a filtered pcap file for each command you added in the comments? I don't have access to the PLCs anymore so it will be the only other way I can verify myself.
Thanks again!

I emailed you the pcap file. On making the changes I saw that _decodePacket doesn't seem to be used anywhere so I assume it is safe to delete

@CharlieMarshall I reviewed the pcap file and the changes. Looks great.
Thanks again for your help, it's much appreciated!


Investigate the use of boxed slice and boxed str to reduce size of token and node

The .into_boxed_str() calls are detrimental but if it reduces struct size, it might still be worth it.

#122 seems like the better approach.


Using unmaintained dirs crate

Cargo Audit shows that orbtk uses dir create which is unmaintained. It suggests to use dirs-next crate
warning: 1 warning found

Crate:  dirs
Title:  dirs is unmaintained, use dirs-next instead
Date:   2020-10-16
URL:    https://rustsec.org/advisories/RUSTSEC-2020-0053
Dependency tree:
dirs 3.0.1
└── orbtk_api 0.3.1-alpha4
    ├── orbtk_widgets 0.3.1-alpha4
    │   └── orbtk 0.3.1-alpha4
    └── orbtk 0.3.1-alpha4

warning: 1 warning found!

Maybe this check should be included in CI?

thank you for the hint. that's a good suggestion.

done


Adding Darkmode and Leaderboard to look appealing

#SSOC Adding dark mode and leaderboard on the basis of least move and timer in the rubbix cube ...for that we will make a count function so it will count each move and make leaderboard according to it and also make an login page for saving the credentials of the persons and also add timer too when solving the cube

Please assign me this issue under SSOC'24 @Dev-tanay


Wrong arguments in ./examples/timit/s0/run.sh stage 5 when runing recognize.py

Describe the bug
A clear and concise description of what the bug is.
"--result_file" should be "--result_dir" for current version, and the "--connect_symbol" is missing for this phoneme recognition task.
To Reproduce
Steps to reproduce the behavior:

Go to './examples/timit/s0'
Run 'run.sh'

Expected behavior
The output phonemes are not seperated without "--connect_symbol"

Screenshots

Desktop (please complete the following information):

OS: Linux

try latest code

try latest code

I just downloaded and tried on this repo, would you mind providing the link to the latest code?

add connect_symbol: ' ' in yaml, and try again
https://github.com/wenet-e2e/wenet/blob/main/examples/timit/s0/conf/train_conformer.yaml#L34

add connect_symbol: ' ' in yaml, and try again
https://github.com/wenet-e2e/wenet/blob/main/examples/timit/s0/conf/train_conformer.yaml#L34

It works! Thank you

@Pooookeman could u make a PR to fix this issue?

@Pooookeman could u make a PR to fix this issue?

Sure yes, but I'm afraid the way I'm modifying can be potentially harmful to the project by passing connect_symbol to tokenizer, especially other datasets with 'split_with_space'==FALSE.

elif tokenizer_type == "char":

    tokenizer = CharTokenizer(
        configs['tokenizer_conf']['symbol_table_path'],
        configs['tokenizer_conf']['non_lang_syms_path'],
        split_with_space=configs['tokenizer_conf'].get(
            'split_with_space', False),
        connect_symbol=configs['tokenizer_conf'].get(
            'split_with_space', ''))

If no further problems, I will submit a PR

if u set it to the default value (''), then it's ok
connect_symbol=configs['tokenizer_conf'].get(
            'connect_symbol', '')

if u set it to the default value (''), then it's ok
connect_symbol=configs['tokenizer_conf'].get(
            'connect_symbol', '')

Oh sry for the typo, will submit the PR soon


CI Pipeline for Test Containers

Hello,
How can i run test containers on ci pipeline? Can you add an example for that?
When i run all tests on pipeline, I'm getting an error as below;
System.Net.Internals.SocketExceptionFactory+ExtendedSocketException : Cannot assign requested address (/var/run/docker.sock)
or
System.Net.Internals.SocketExceptionFactory+ExtendedSocketException : Cannot assign requested address (/tcp://localhost:2375)
Also [.WithDockerEndpoint] extension method does not work.
Thanks.

Hello Furkan @bozd4g ,
Which CI pipeline are you using? In you pipeline you should be sure that you can access your docker environment like DIND (Docker in docker) or DND (Docker near docker)
In your first error, you are using docker socket -> /var/run/docker.sock , for this sample, you need to mount socket file.
In your second error, you are trying to connect over docker api. Try to make debug and check is docker available and accessible for that.
Please do not hesitate to contact with me for further questions.

Thank you so much @mustafaonuraydin 🙏


webpack resolver fails on CircleCI

Everything works great locally, but I kick it up to CircleCI & I get a few thousand errors saying:
Resolve error: Cannot convert undefined or null to object
example (under npm run lint)
https://circleci.com/gh/ParabolInc/action/2142
my settings:
  "settings": {
    "import/parser": "babel-eslint",
    "import/resolver": {
      "webpack": {
        "config": "./webpack/prod.babel.js" // points to a file that babelifies my webpack config
      }
    },
  },

@mattkrick I just hit this exact error in CircleCI... what did you do to fix it?

Oh dang, I don't remember. My project is all open source though so you can check out the commit log.
If you run my fork it'll show the error and that should point you in the right direction.

@mattkrick did you find a solution for this? It looks as though the rules related to imports were just switched off in this case see https://github.com/ParabolInc/action/pull/810/files

for me, the problem was in the application, but this module wasn't giving me the right error. My PR fixes that & provides the correct error so you can properly diagnose your app.


Wrong slide behavior

Consider for eg. if we are in step 3. Click the "Next step" icon. Now when you try to go to any previous step from the left panel you can note that the page slides from right which should have slided-in from the left side.

@lakshminathan I'm afraid we will not be fixing this here but we're working on a complete drop-in replacement using just Custom Elements spec, without Polymer or other frameworks/libs, in https://github.com/googlecodelabs/codelab-elements. See https://github.com/googlecodelabs/codelab-components/issues/49 for more details.
I cross-linked this issue from https://github.com/googlecodelabs/codelab-elements/issues/19.


Operator is not bonded to a staking provider

When I try to run my node, I receive this logs, can someone help?:
(nulink-venv) root@ip-172-30-20-191:~/nulink# docker logs -f 0cacad5cd407
Defaulting to Ursula configuration file: '/home/circleci/.local/share/nulink/ursula.json'
Authenticating Ursula
Learner seed_nodes 0
Loaded Ursula (horus)
✓ External IP matches configuration
Starting services [current version: 0.5.0]
✓ Node Discovery (Horus)
✓ operator address: 0xD019C4...
✓ Work Tracking
✓ Start Operator Bonded Tracker
✓ Rest Server https://<IP_ADDRESS>:9151
Working ~ Keep Ursula Online!
OperatorBondedTracker: ! Operator 0xD019C4a3AA3C28c373660AC8e382165E22d02401 is not bonded to a staking provider
OperatorBondedTracker: ! Operator 0xD019C4a3AA3C28c373660AC8e382165E22d02401 is not bonded to a staking provider
OperatorBondedTracker: ! Operator 0xD019C4a3AA3C28c373660AC8e382165E22d02401 is not bonded to a staking provider
OperatorBondedTracker: ! Operator 0xD019C4a3AA3C28c373660AC8e382165E22d02401 is not bonded to a staking provider

go and bond on the staking site


Arielle rails-activerecord-crud-diagnostic

Answered some diagnostic questions.

Making a curl request is not the same thing as adding it via active record .create!.  If you're a bit confused I'd recommend having a high level talk with on of your instructors.


Data loader for quantiles

@sarenehan

Add eta to the xarray training dataset in uwnet.train.get_dataset
subclass: XRTimeSeries. change setup_indices and set time_length=2. ConditionalXrTimeSteps(data: xr.Dataset, conditional_state=2).

sample = dataset[0]

sample should be
{
'QT': (2, nz, 1, 1)
'stochastic_state': (2, 1, 1, 1)
...
}

@sarenehan Since you are no longer training separate NNs for each bin, do you still need these data loader classes? If not, we could close this issue and delete the corresponding branch.


Path with Maximum Gold

In a gold mine grid of size m x n, each cell in this mine has an integer representing the amount of gold in that cell, 0 if it is empty.
Return the maximum amount of gold you can collect under the conditions:
Every time you are located in a cell you will collect all the gold in that cell.
From your position, you can walk one step to the left, right, up, or down.
You can't visit the same cell more than once.
Never visit a cell with 0 gold.
You can start and stop collecting gold from any position in the grid that has some gold.
Example-1
Input: grid = [[0,6,0],[5,8,7],[0,9,0]]
Output: 24
Explanation:
[[0,6,0],
[5,8,7],
[0,9,0]]
Path to get the maximum gold, 9 -> 8 -> 7.
Example 2:
Input: grid = [[1,0,7],[2,0,6],[3,4,5],[0,3,0],[9,0,20]]
Output: 28
Explanation:
[[1,0,7],
[2,0,6],
[3,4,5],
[0,3,0],
[9,0,20]]
Path to get the maximum gold, 1 -> 2 -> 3 -> 4 -> 5 -> 6 -> 7.

@tarun26091999  sir.
I am a GSSOC'21 participant. please assign this issue to me . I would like to code in  C++.

Please assign this to me in Python.


[1.21] Crashing issue with the modify_enchantment_level power type

Crash Report - https://mclo.gs/9kw16QZ
Log Report - https://mclo.gs/r43UUgU

When does this happen? Does this happen consistently?

When does this happen? Does this happen consistently?

It's like triggered on distance, somehow also gets triggered when mining.

It seems I have the same bug, I am attaching a report in case it is useful.
mclo.gs/vMweteU

Does this still happen in Origins 1.13.0-alpha.11?


How to configure UCP for separating app and management traffic

https://docs.docker.com/engine/swarm/networking/#use-a-separate-interface-for-control-and-data-traffic


Feature request for 2017 Lexus RX 450H Blind spot audio warning

This is a deficiency in the Lexus, not Openpilot.
Would it be possible to add an audio warning when there is something detected in the blind spot and the signal is on for that side?
My wife had a 2017 Sonata Hybrid that gave an audio warning when she had the signal on for the side where the blind spot detection was sensing a vehicle.
She has a 2017 Lexus RX 450H now and it does not give an audio alert when you have the turn signal on and the blind spot detection is sensing a car on that side.
I

I don't think it's worth the code complexity to add this feature. Feel free to create your own fork where you add this.


Compilation issues with xCode 8/Swift 3

Hi,
App seems to be using older version of Swift. Are there known issues with xCode 8/Swift 3? I see lot of compilation issues. If yes, what is the recommended way to compile and run?
It would be nice to update the readme file.
Thanks,
Karthik

This is not support to swift 3.0.
Now, this is converted to swift 2.3, and it is Xcode 8 supported.

It works now. Thanks.
Btw, this looks cool!

Will you support swift3 in the future? 👍


Open "confirm email" link

Hey, I added my a new email to my account and sent the confirmation mail, now I want to automatically confirm the email by opening the link from the email. The link is https://www.instagram.com/accounts/confirm_email/{code}, if you open it you have to log in manually, except when you are already logged in. Is it possible to just call this link using the _send_private_request() method when the session data is already in cache?

I do it like this:
cl.inject_sessionid_to_public()
cl.public.get(confirm_email_url)


fix: line ending commas

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

YAML doesn't have trailing commas, and the syntax generated an error when deploying the theme to the forum.

Thanks Mrugesh!
I've deployed to the forum again.


Split IsahcError::Timeout into IsahcError::DnsTimeout

Both DNS resolver timeouts and request timeouts are reported as IsahcError::Timeout and I can't differentiate between them.

I don't think we can make that distinction; libcurl just returns CURLE_OPERATION_TIMEDOUT without any specific error string for both cases. I think a new error code would have to be added to libcurl to make this possible.

Command line curl prints a different error when using --dns-servers with c-ares.


Any plans on supporting KMP, or have suggested alternatives?

When attempting to add his library to a KMP project, i get this error:
The dependency com.linecorp.abc:kmm-location:0.2.11 was published in the legacy mode. Support for such dependencies will be removed in the future. See https://kotl.in/0b5kn8 for details.
#Versions
kotlin.version=1.9.10
agp.version=8.0.2
compose.version=1.5.2

Would be absolutely helpful to republish the library

I simply used the library as a source material, and made it work with a couple files from my current project.
Here is a link to a project that uses it
Look for GPSLocationService.kr
Also includes how to handle permissions on android and iOS
I would make this into a library, but not sure exactly how that works for KMP.
But this should get you along nicely...
Please leave a github star!


type hint for console controller contents in Action

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues
#14882

Merged. Thanks!


There is a vulnerability introduced in your package

Hi @gaearon @Timer,I would like to report a vulnerability
Issue
1 vulnerabilities (medium severity) is introduced in react-dev-utils:
Vulnerability CVE-2021-23364 (medium severity) is detected in package browserslist(versions:>=4.0.0 <4.16.5):https://snyk.io/vuln/SNYK-JS-BROWSERSLIST-1090194
The above vulnerable package is referenced by react-dev-utils via:
<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>Solution
Since<EMAIL_ADDRESS>is transitively referenced by 3326 downstream projects (e.g., @react-workspaces/react-scripts 3.4.0-alpha-01 (latest version),build-plugin-rax-app 6.4.2 (latest version), tita-react-components 1.5.101  (latest version), react-dev-inspector 1.6.0 (latest version), @scandipwa/eslint-plugin-scandipwa-guidelines 2.0.22(latest version)),
<EMAIL_ADDRESS>is referenced by 2893 downstream projects (e.g., @storybook/core 6.3.0 (latest version), react-scripts 4.0.3 (latest version), @storybook/react 6.3.0 (latest version), @storybook/addon-docs 6.3.0 (latest version), @storybook/addon-essentials 6.3.0 (latest version)),
<EMAIL_ADDRESS>is referenced by 2283 downstream projects (@elyra/pipeline-editor-extension 2.3.0-rc.1 (latest version), @types/storybook__addon-info 5.2.3 (latest version), @sambego/storybook-state 2.0.1 (latest version), @storybook/react-native-server 5.3.23 (latest version), @types/storybook-react-router 1.0.1 (latest version)),
If react-dev-utils removes the vulnerable package from the above versions, then its fixed versions can help downstream users decrease their pain.
Could you help update packages in these versions?
Fixing suggestions
(1）In<EMAIL_ADDRESS>you can kindly try to perform the following upgrade (not crossing their major versions):
browserslist 4.10.0 ➔ 4.16.5;
(2）In<EMAIL_ADDRESS>you can kindly try to perform the following upgrade (not crossing their major versions):
browserslist 4.14.2 ➔ 4.16.5;
(3）In<EMAIL_ADDRESS>you can kindly try to perform the following upgrade (not crossing their major versions):
browserslist 4.7.0 ➔ 4.16.5;
Note:
<EMAIL_ADDRESS>has fixed the vulnerability CVE-2021-23364
Thank you for your attention to this issue!
Sincerely yours,
Paimon

A large number of downstream users transitively use<EMAIL_ADDRESS>and @9.1.* via unmaintained packages (the unmaintained packages cannot upgrade react-dev-utils from 10.2.* or 9.1.* to the latest version).So if you can kindly release fixed 10.2.* and 9.1.* versions, the downstream users can avoid introducing the vulnerablity.

There is no actual vulnerability here. I have already replied in https://github.com/facebook/create-react-app/issues/11177#issuecomment-875506806, please don't create new issues. They will be locked. The discussion on false positives from npm audit and how to avoid them happens in https://github.com/facebook/create-react-app/issues/11174.


Container Registry private images deploy

Events:
  Type     Reason     Age                From               Message
  ----     ------     ----               ----               -------
  Normal   Scheduled  67s                default-scheduler  Successfully assigned deployment-9/app-deployment-9-5857c56dc5-t8btj to gke-uffizzi-client-sandbox-482f3ba5-ez9l
  Warning  Failed     45s                kubelet            Failed to pull image "limoha/devops-for-programmers-projectlvl-1:latest": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/limoha/devops-for-programmers-projectlvl-1:latest": failed to resolve reference "docker.io/limoha/devops-for-programmers-projectlvl-1:latest": pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed
  Warning  Failed     45s                kubelet            Error: ErrImagePull
  Normal   BackOff    44s                kubelet            Back-off pulling image "limoha/devops-for-programmers-projectlvl-1:latest"
  Warning  Failed     44s                kubelet            Error: ImagePullBackOff
  Normal   Pulling    32s (x2 over 46s)  kubelet            Pulling image "limoha/devops-for-programmers-projectlvl-1:latest"

When the controller creates a new deployment the credential for a private registry hasn't been created yet. The pulling starts only after the second try. But the deployment in the rails app gets failed immediately after Error: ImagePullBackOff.
We need to re-run a deploy of containers when the app gets the ImagePullBackOff error and consider the retry count (because credentials can be invalid)

services:
  webhooks-test-app:
    image: uffizzitest/test-private-repo:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres

x-uffizzi-ingress:
  service: webhooks-test-app
  port: 80

x-uffizzi-continuous-preview:
  share_to_github: true
  delete_preview_after: 99h
  tag_pattern: uffizzi_request_*
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true
  delete_preview_when_pull_request_is_closed: true
  deploy_preview_when_pull_request_is_opened: true    # i.e., value should be added in the Uffizzi Dashboard

bash-5.1# bundle exec uffizzi preview create -o=json test-compose/uffizzi-compose-webhook-docker-private.yml
[✔] Creating containers...
[✔] Deploying preview...
  [✔] test-private-repo
{"id":"deployment-16","url":"https://deployment-16.default.app.qa-gke.uffizzi.com"}
bash-5.1#

services:
  webhooks-test-app:
    image: ghcr.io/nealarw/webhook-test-app:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres

x-uffizzi-ingress:
  service: webhooks-test-app
  port: 80

x-uffizzi-continuous-preview:
  share_to_github: true
  delete_preview_after: 99h
  tag_pattern: uffizzi_request_*
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true
  delete_preview_when_pull_request_is_closed: true
  deploy_preview_when_pull_request_is_opened: true    # i.e., value should be added in the Uffizzi Dashboard

bash-5.1# bundle exec uffizzi preview create -o=json test-compose/uffizzi-compose-webhook-ghcr.yml
[✔] Creating containers...
[✔] Deploying preview...
  [✔] webhook-test-app
{"id":"deployment-17","url":"https://deployment-17.default.app.qa-gke.uffizzi.com"}

services:
  webhooks-test-app:
    image: gcr.io/uffizzi-pro-qa-gke/webhooks-test-app:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres

x-uffizzi-ingress:
  service: webhooks-test-app
  port: 80

x-uffizzi-continuous-preview:
  share_to_github: true
  delete_preview_after: 99h
  tag_pattern: uffizzi_request_*
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true
  delete_preview_when_pull_request_is_closed: true
  deploy_preview_when_pull_request_is_opened: true

bash-5.1# bundle exec uffizzi preview create -o=json test-compose/uffizzi-compose-webhook-google.yml
[✔] Creating containers...
[✔] Deploying preview...
  [✔] webhooks-test-app
{"id":"deployment-18","url":"https://deployment-18.default.app.qa-gke.uffizzi.com"}
bash-5.1#

services:
  webhooks-test-app:
    image: uffizziqa.azurecr.io/webhooks-test-app:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres

x-uffizzi-ingress:
  service: webhooks-test-app
  port: 80

x-uffizzi-continuous-preview:
  share_to_github: true
  delete_preview_after: 99h
  tag_pattern: uffizzi_request_*
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true
  delete_preview_when_pull_request_is_closed: true
  deploy_preview_when_pull_request_is_opened: true

bash-5.1# bundle exec uffizzi preview create -o=json test-compose/uffizzi-compose-webhook-azure.yml
[✔] Creating containers...
[✔] Deploying preview...
  [✔] webhooks-test-app
{"id":"deployment-19","url":"https://deployment-19.default.app.qa-gke.uffizzi.com"}

Repository does not exist:
bash-5.1# bundle exec uffizzi preview create -o=json test-compose/uffizzi-compose-webhook-azure.yml
[✔] Creating containers...
[✖] Deploying preview...
  [✖] webhooks-test-app1
{"id":"deployment-4","url":"https://deployment-4.default.app.qa-gke.uffizzi.com"}
bash-5.1#

TESTED - OK


additional meta Content-Type is added to HTML5

Hello guys,
I have the following HTML file test.html.
<!DOCTYPE html>
<html>
  <head>
    <title>Test</title>
    <meta charset="UTF-8">
  </head>
  <body>
  </body>
</html>

I am opening the file with Nokogiri and then write it back to another file.
doc = Nokogiri::HTML::Document.parse(File.open('test.html').read)
File.open('output.html', 'w') do |f|
  f.puts doc.to_s
end

The output file will contain meta http-equiv="Content-Type" content="text/html; charset=UTF-8" although the meta charset for HTML5 is set.
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>Test</title>
<meta charset="UTF-8">
</head>
<body>
  </body>
</html>

Any idea how I could fix it not to add that meta tag which is for HTML4 and not for HTML5?
I am using nokogiri-1.6.0.
Any help is appreciated. Thanks!

Any news on this?

Hi, apologies for the extremely slow reply. Nokogiri does not support HTML5. You may want to check out the Nokogumbo project, which aims for HTML5 compatibility with the Gumbo parser.


What about the Selfie-Segmentation solution ?

Is there in your plans to implement it ? Can you provide hints for our selves to program it ?

Self-segmentation would require fetching segmented frames from mediapipe and uploading them to ue4 texture. Not planned in near future. But if I remember correctly MediaPipeUnityPlugin can do it.
In case you want to solve it,
attach to pipeline:
https://github.com/wongfei/mediapipe/blob/unreal/mediapipe/unreal/ump_pipeline.cc#L180
callback to ue4 via interface:
https://github.com/wongfei/mediapipe/blob/unreal/mediapipe/unreal/ump_api.h#L84
inside ue4 create dynamic texture and update it:
https://github.com/wongfei/ue4-gstreamer/blob/master/Plugins/GStreamer/Source/GStreamer/Private/GstTexture.cpp#L105
some work required, huh :)

Hello,
I vote for this feature !
Not sure i have enough skill.


docker.compose.build is failing when passing a str variable in services

This is what I'm passing
svc_list='stock'
docker.compose.build(services= svc_list , cache=False, progress=None, pull=True, quiet=False, ssh=None, stream_logs=True)
but it gives out error because instead of it doing
docker compose --file ./docker-compose.yml build --no-cache --pull stock
it does
docker compose --file ./docker-compose.yml build --no-cache --pull s t o c k
surprisingly  docker.compose.stop(services=svc_list) works fine.

It should be a list of strings, but I don't see why we can't accept just a string too. I'll see if it's possible to change this to convert automatically to a list when a string is passed. You should use a list as a fix in the meantime.


Added FB sticker detection

FB stickers can be distinguished from regular images sent by the user by looking at the message data for a sticker_id field. Right now, botkit doesn't support plumbing that through so it's impossible to distinguish a Like from someone sending an image. This fixes that issue.

Cool!
Should this maybe also throw a special event, like sticker_received?


Feature/1337 carlos apexdocs

Warning
Info
Issues
Fixes #1337

@davidhabib ~ We need to wait for the whole dev flow to pass and automatically merge those changes into my branch, but once that's done, this branch is ready.
It's mostly apexdocs. The only code changes are around reorganizing code, to follow the properties/constructor/methods order.

@davidhabib ~ this is ready to merge.


Missing proguard configuration in README.md

The library in the current state will not work when using proguard.
PhxMessage class member names are used to parse the response from WS.
The solution would be to add @SerializeName annotation, this way there is no need to add any additional proguard rules (apart from the ones for gson and okhhtp).
As for now you have to specify a rule like this
-keepclassmembernames class org.phoenixframework.PhxMessage { *; }

sory for duplicated issues :( GitHub is really unpredictable today


FE-63: Table of contents highlight issue

Overview
Issue: In most cases, when you click on the table of content, e.g. User click "Corda Package (CPKs)", the table of content will highlight "Cluster" section title. But user will probably expecting "Corda Package (CPKs)" to be highlighted.
Jira: https://r3-cev.atlassian.net/browse/FE-63

Current fix is to highlight clicked title and whatever is on the viewport of the user:

Generating a preview of these changes to facilitate tech writer review. Thanks Zheng

LGTM

@yarivus I haven't generated the preview yet.

@zhengmar3 thanks for raising PR. Can you please check PR as the build. Once that is sorted, I can generate a preview site to facilitate reviews. TIA

Preview site here: https://colin.preview.docs.r3.com/

Preview site here: https://colin.preview.docs.r3.com/

There seems to be an issue with this preview. There is a gap between the red banner and the navigation bar. @zhengmar3 can you please check that you didn't inadvertently introduce a bug?

Hi Conor,
I built it on Anna’s preview site: HEREhttps://anna.preview.docs.r3.com/
My preview site was used to build an earlier iteration before Peter pushed a fix.
--
Colin Hillis
Lead Technical Writer | R3.
Remote | Northern Ireland
M:  (+44)<PHONE_NUMBER>
<EMAIL_ADDRESS>| www.corda.nethttp://www.corda.net/
[signature_3478046804]
“The Power of 3”, only from R3: trust technology, connected networks, regulated markets expertise. Discover more at r3.comhttps://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fr3.com%2F&data=05|01|colin.hillis%40r3.com|6bb5ffce2b0e40f4caa408da8b5f3607|a4be1f2e2d10419587cd736aca9b672c|0|0|637975538291122586|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fHDSUHQijWtlpEbevCome%2BGqPgX6coKLQ2nEr31FT2o%3D&reserved=0.
From: Conor Pender @.>
Date: Monday, 27 February 2023 at 12:11
To: corda/corda-docs-portal @.>
Cc: Colin Hillis @.>, State change @.>
Subject: Re: [corda/corda-docs-portal] FE-63: Table of contents highlight issue (PR #1117)
[Screenshot 2023-02-27 at 12 09 57]https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F59569355%2F221560509-17a20591-0bc9-414f-80c6-ee9228552504.png&data=05|01|colin.hillis%40r3.com|d4d0e002d96c499f6a6508db18bbb6dc|a4be1f2e2d10419587cd736aca9b672c|0|0|638130966737433832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=RGHLGFFTLHI1epw7gjKLDyeKaDR2Z5zLnEObdLaRJCU%3D&reserved=0
—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcorda%2Fcorda-docs-portal%2Fpull%2F1117%23issuecomment-1446218998&data=05|01|colin.hillis%40r3.com|d4d0e002d96c499f6a6508db18bbb6dc|a4be1f2e2d10419587cd736aca9b672c|0|0|638130966737433832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oXzLDfcWG4wNDY8VrvLJ0d113r4PQQV4nV%2FqoefhrGo%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA25CVKXVM6EBRNBJABBWKI3WZSKV5ANCNFSM6AAAAAAVG3DL7Y&data=05|01|colin.hillis%40r3.com|d4d0e002d96c499f6a6508db18bbb6dc|a4be1f2e2d10419587cd736aca9b672c|0|0|638130966737433832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=T2hvv0sfKTtF59%2F7Jmn043s%2BEYpe0OPfTB%2FmOmw2CYE%3D&reserved=0.
You are receiving this because you modified the open/close state.Message ID: @.***>
R3 is committed to ensuring the security and protection of the personal data that we process, and to provide a compliant and consistent approach to data protection. Any personal data that is shared as part of any email correspondence with R3 shall be processed in line with the R3 Privacy Policyhttps://www.r3.com/full-privacy-policy/

Actually Conor…you are right. The banner looks OK on the homepage, but when you drill into doc pages, it changes to introduce a space, like you said. Good catch Conor!
--
Colin Hillis
Lead Technical Writer | R3.
Remote | Northern Ireland
M:  (+44)<PHONE_NUMBER>
<EMAIL_ADDRESS>| www.corda.nethttp://www.corda.net/
[signature_3478046804]
“The Power of 3”, only from R3: trust technology, connected networks, regulated markets expertise. Discover more at r3.comhttps://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fr3.com%2F&data=05|01|colin.hillis%40r3.com|6bb5ffce2b0e40f4caa408da8b5f3607|a4be1f2e2d10419587cd736aca9b672c|0|0|637975538291122586|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fHDSUHQijWtlpEbevCome%2BGqPgX6coKLQ2nEr31FT2o%3D&reserved=0.
From: Colin Hillis @.>
Date: Monday, 27 February 2023 at 12:25
To: corda/corda-docs-portal @.>, corda/corda-docs-portal @.>
Cc: State change @.>
Subject: Re: [corda/corda-docs-portal] FE-63: Table of contents highlight issue (PR #1117)
Hi Conor,
I built it on Anna’s preview site: HEREhttps://anna.preview.docs.r3.com/
My preview site was used to build an earlier iteration before Peter pushed a fix.
--
Colin Hillis
Lead Technical Writer | R3.
Remote | Northern Ireland
M:  (+44)<PHONE_NUMBER>
<EMAIL_ADDRESS>| www.corda.nethttp://www.corda.net/
[signature_3478046804]
“The Power of 3”, only from R3: trust technology, connected networks, regulated markets expertise. Discover more at r3.comhttps://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fr3.com%2F&data=05|01|colin.hillis%40r3.com|6bb5ffce2b0e40f4caa408da8b5f3607|a4be1f2e2d10419587cd736aca9b672c|0|0|637975538291122586|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fHDSUHQijWtlpEbevCome%2BGqPgX6coKLQ2nEr31FT2o%3D&reserved=0.
From: Conor Pender @.>
Date: Monday, 27 February 2023 at 12:11
To: corda/corda-docs-portal @.>
Cc: Colin Hillis @.>, State change @.>
Subject: Re: [corda/corda-docs-portal] FE-63: Table of contents highlight issue (PR #1117)
[Screenshot 2023-02-27 at 12 09 57]https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F59569355%2F221560509-17a20591-0bc9-414f-80c6-ee9228552504.png&data=05|01|colin.hillis%40r3.com|d4d0e002d96c499f6a6508db18bbb6dc|a4be1f2e2d10419587cd736aca9b672c|0|0|638130966737433832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=RGHLGFFTLHI1epw7gjKLDyeKaDR2Z5zLnEObdLaRJCU%3D&reserved=0
—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcorda%2Fcorda-docs-portal%2Fpull%2F1117%23issuecomment-1446218998&data=05|01|colin.hillis%40r3.com|d4d0e002d96c499f6a6508db18bbb6dc|a4be1f2e2d10419587cd736aca9b672c|0|0|638130966737433832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oXzLDfcWG4wNDY8VrvLJ0d113r4PQQV4nV%2FqoefhrGo%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA25CVKXVM6EBRNBJABBWKI3WZSKV5ANCNFSM6AAAAAAVG3DL7Y&data=05|01|colin.hillis%40r3.com|d4d0e002d96c499f6a6508db18bbb6dc|a4be1f2e2d10419587cd736aca9b672c|0|0|638130966737433832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=T2hvv0sfKTtF59%2F7Jmn043s%2BEYpe0OPfTB%2FmOmw2CYE%3D&reserved=0.
You are receiving this because you modified the open/close state.Message ID: @.***>
R3 is committed to ensuring the security and protection of the personal data that we process, and to provide a compliant and consistent approach to data protection. Any personal data that is shared as part of any email correspondence with R3 shall be processed in line with the R3 Privacy Policyhttps://www.r3.com/full-privacy-policy/

@colin-hillis  @conor-r3
Current live version of docs site works fine. And it got all my latest changes.
It's very unlikely caused by this PR.

Potentially Peter’s PR regarding the banner
--
Colin Hillis
Lead Technical Writer | R3.
Remote | Northern Ireland
M:  (+44)<PHONE_NUMBER>
<EMAIL_ADDRESS>| www.corda.nethttp://www.corda.net/
[signature_3478046804]
“The Power of 3”, only from R3: trust technology, connected networks, regulated markets expertise. Discover more at r3.comhttps://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fr3.com%2F&data=05|01|colin.hillis%40r3.com|6bb5ffce2b0e40f4caa408da8b5f3607|a4be1f2e2d10419587cd736aca9b672c|0|0|637975538291122586|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fHDSUHQijWtlpEbevCome%2BGqPgX6coKLQ2nEr31FT2o%3D&reserved=0.
From: Zheng Ma @.>
Date: Monday, 27 February 2023 at 12:32
To: corda/corda-docs-portal @.>
Cc: Colin Hillis @.>, Mention @.>
Subject: Re: [corda/corda-docs-portal] FE-63: Table of contents highlight issue (PR #1117)
@colin-hillishttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcolin-hillis&data=05|01|colin.hillis%40r3.com|670ba811ead94ca0126608db18bec022|a4be1f2e2d10419587cd736aca9b672c|0|0|638130979777365058|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=O8FdbwcfBJEHibBjMpuR0kkhMWk%2BBts9wJXYmvCIR3U%3D&reserved=0 @conor-r3https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fconor-r3&data=05|01|colin.hillis%40r3.com|670ba811ead94ca0126608db18bec022|a4be1f2e2d10419587cd736aca9b672c|0|0|638130979777365058|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=FqCFigHEzpo3ZeSLzxRIzip2OZvHwR9gTYsuasuLQ58%3D&reserved=0
Current live version of docs site works fine. And it got all my latest changes.
It's very unlikely caused by this PR.
[image]https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F80267895%2F221564393-e29e71e0-95c4-404b-bb4b-ad1716fa0e48.png&data=05|01|colin.hillis%40r3.com|670ba811ead94ca0126608db18bec022|a4be1f2e2d10419587cd736aca9b672c|0|0|638130979777365058|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=KZHAJAniJxvzB1EFG1JSWOCM9V7A4FMD342xk0Ol0gM%3D&reserved=0
—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcorda%2Fcorda-docs-portal%2Fpull%2F1117%23issuecomment-1446250024&data=05|01|colin.hillis%40r3.com|670ba811ead94ca0126608db18bec022|a4be1f2e2d10419587cd736aca9b672c|0|0|638130979777365058|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=javwJ8qy%2Bk%2FaX8%2BG55%2FD7%2FtA6FRKlIoc99IKC%2Fi1qk4%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA25CVKWNABUWDSLRMQWTLLTWZSNHNANCNFSM6AAAAAAVG3DL7Y&data=05|01|colin.hillis%40r3.com|670ba811ead94ca0126608db18bec022|a4be1f2e2d10419587cd736aca9b672c|0|0|638130979777521256|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ad8d%2BeApc4G5%2F4c6qqYquJ5Cg9TV9VKclpavHuoJ11E%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>
R3 is committed to ensuring the security and protection of the personal data that we process, and to provide a compliant and consistent approach to data protection. Any personal data that is shared as part of any email correspondence with R3 shall be processed in line with the R3 Privacy Policyhttps://www.r3.com/full-privacy-policy/


ActionMenu.Anchor should only accept button

Description
ActionMenu has preferred API of ActionMenu.Button which uses a Primer Button and wires it up correctly.
However, if you want to customise the anchor, we also provide a ActionMenu.Anchor that can be used to give a custom element.
While this API is required, it's possible to use it incorrectly.
Spotted in the wild: In this example, the developer is trying to add an "active indicator" on the IconButton by adding an additional element (with aria-label) and positioning it
<ActionMenu.Anchor>

    <IconButton aria-label="Filter files in tree" icon={FilterIcon} />
    {filterEnabled && (

    )}

</ActionMenu.Anchor>

The above JSX renders inaccessible html:

  <button
    data-component="IconButton"
    type="button"
    aria-labelledby=":rl0:"
    aria-describedby=":rl1:-loading-announcement"
  >
    <svg aria-hidden="true"></svg>
  </button>

    Filter files in tree

https://github.com/user-attachments/assets/988470d7-0f1a-43a8-aa7b-3a874dc3cb33
Video description:

Pressing tab on the close button seems to focus the filter button but does not show tooltip.
The screen reader reads out "Filter files in tree, Showing only changed files, menu pop-up, group"
You'd expect tabbing again would focus the text input, but it focuses the button instead. Now a tooltip is visible with text "Filter files in tree".
The screen reader now reads "Filter files in tree, button, Filter files in tree, Showing only changed files, menu pop-up, group" (still reading out the group)
Tabbing again finally focuses the text input

Proposed Solution
I have 2 suggestions:

Reduce: The most common use case of ActionMenu.Anchor is to use an IconButton, we should create a shortcut ActionMenu.IconButton as a companion to ActionMenu.Button.
A blessed shortcut would reduce the chances of implementing it incorrectly.
Validate: ActionMenu.Anchor should validate it's children, if it receives an incorrect element as the root, it should throw a warning and guide the developer to correct usage.
My guess is that only button is valid, but we need to validate that assumption. Non-interactive element is definitely a violation. For prior art, we have similar (if not more advanced) checks in Tooltip

Suggested prioritisation:
I have fixed the instance where this was spotted so I am not blocked.
But there are 258 instances of ActionMenu.Anchor that need to be audited for their children to decide if this is a widespread bug or a good to have

Steps to reproduce
Navigate to custom anchor story and replace the Anchor with:
<ActionMenu.Anchor>

    <IconButton aria-label="Filter files in tree" icon={FilterIcon} />
    {filterEnabled && (

    )}

</ActionMenu.Anchor>

Version
v37.5.0
Browser
Chrome

<ActionMenu.Anchor>

    {filterEnabled && (

    )}


Default cookie setter in OAuth is not setting samesite = None #352

WHY are these changes introduced?
Fixes #352
For context read the issue I created.
WHAT is this pull request doing?
Basically sets samesite=none for cookies created in OAuth2.
Type of change

[x] Patch: Bug (non-breaking change which fixes an issue)
[ ] Minor: New feature (non-breaking change which adds functionality)
[ ] Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] I have added a changelog entry, prefixed by the type of change noted above
[ ] I have added/updated tests for this change
[ ] I have updated the documentation for public APIs from the library (if applicable)

Wouldn't this make cookies 3rd party cookies? If so I'm not sure we want them as browser will start block them.


Update CUE schemas with frontend definition

This PR aims to update the cue schema with the current definition defined in the frontend.

I just added some "unit" test on the chart schemas in order to verify that the schemas validate a valid JSON file.
And here the errors I got:
testing charts gauge
options.thresholds.steps.1.color: incomplete value string:
    ../common/threshold.cue:5:9
options.unit.decimal_places: incomplete value number:
    ../common/unit.cue:11:18

I took the JSON example from the file nodeExporterDashboard.ts and it seems there are others things missing.
Here for instance, I have this JSON file:
{
  "kind": "GaugeChart",
  "display": {
    "name": "CPU Busy"
  },
  "options": {
    "query": {
      "kind": "PrometheusGraphQuery",
      "options": {
        "query": "(((count(count(node_cpu_seconds_total{job='node',instance='$instance'}) by (cpu))) - avg(sum by (mode)(rate(node_cpu_seconds_total{mode='idle',job='node',instance='$instance'}[$interval])))) * 100) / count(count(node_cpu_seconds_total{job='node',instance='$instance'}) by (cpu))"
      }
    },
    "calculation": "LastNumber",
    "unit": {
      "kind": "Percent"
    },
    "thresholds": {
      "steps": [
        {
          "value": 85
        },
        {
          "value": 95
        }
      ]
    }
  }
}

And CUE is yelling that thresholds are missing the field color.  So my question is, who is wrong ? The JSON file is incorrect and CUE is correct (so we have to provide the field color) or is it the opposite ? (so I have to change the CUE schema and potentially the typescript too)
Same question for the unit that expects decimal_places to be filled.
As a final note, I noticed in the DecimalUnitOptions we have this field suffix?: 'string';.
So from my understanding, suffix is an optional field that can have a unique value string. That doesn't look like it's a super useful field don't you think ? Or if it's just, we need to know if the user needs it or not, perhaps a boolean could simply do the job.
@sjcobb or @LukeTillman if you have some times to look at it, that would be cool. Thank you :)

@Nexucis for thresholds color, it looks like the type in thresholds.ts is wrong since it should be optional as shown here. Sorry about that! I can make a separate PR with that fix too if you want to go ahead and merge this one?

cool, thanks @sjcobb for your feedback.

The thought with suffix was it would support any of the units shown in this ECMA standard, I played around with it some in this dashboard for example.
On second thought, It actually conflicts with how kinds are defined though, so I'd be good with getting rid of it.

Perhaps on this particular point, is it ok if you do it ? I'm not sure to know what are the consequences to remove the field suffix from the datamodel. I supposed there will be some changes in the typescript.

alright, thanks a lot for your review and feedback @sjcobb @AntoineThebaud. Let's merge this one to unlock the situation.


Microsoft.SharePoint.Client.ServerException: Operation is not valid

Hi,
I used the original and webbexpert branch with the same result. Some files seem to trigger this output during processing, I don't know which file is causing the problem as it is not shown in the output. Results in a crash at the same point every time.
Original Balassy Code Error;
Unhandled Exception: Microsoft.SharePoint.Client.ServerException: Operation is not valid due to the current state of the object.    at Microsoft.SharePoint.Client.ClientRequest.ProcessResponseStream(Stream responseStream)    at Microsoft.SharePoint.Client.ClientRequest.ProcessResponse()    at Microsoft.SharePoint.Client.ClientRequest.ExecuteQueryToServer(ChunkStringBuilder sb)    at Microsoft.SharePoint.Client.ClientRequest.ExecuteQuery()    at Microsoft.SharePoint.Client.ClientRuntimeContext.ExecuteQuery()    at Microsoft.SharePoint.Client.ClientContext.ExecuteQuery()    at OneDriveVersionCleaner.Program.ProcessFile(ClientContext context, ListItem item) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master\OneDriveVersionCleaner\Program.cs:line 405    at OneDriveVersionCleaner.Program.ProcessItems(ClientContext context, ListItemCollection items) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master\OneDriveVersionCleaner\Program.cs:line 393    at OneDriveVersionCleaner.Program.ProcessFolder(ClientContext context, List list, String rootRelativeUrl) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master\OneDriveVersionCleaner\Program.cs:line 378    at OneDriveVersionCleaner.Program.Main(String[] args) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master\OneDriveVersionCleaner\Program.cs:line 338
Webbexpert Code Error;
Unhandled Exception: Microsoft.SharePoint.Client.ServerException: Operation is not valid due to the current state of the object.    at Microsoft.SharePoint.Client.ClientRequest.ProcessResponseStream(Stream responseStream)    at Microsoft.SharePoint.Client.ClientRequest.ProcessResponse()    at Microsoft.SharePoint.Client.ClientRequest.ExecuteQueryToServer(ChunkStringBuilder sb)    at Microsoft.SharePoint.Client.ClientRequest.ExecuteQuery()    at Microsoft.SharePoint.Client.ClientRuntimeContext.ExecuteQuery()    at Microsoft.SharePoint.Client.ClientContext.ExecuteQuery()    at OneDriveVersionCleaner.Program.ProcessFile(ClientContext context, ListItem item) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master [webexpert]\OneDriveVersionCleaner\Program.cs:line 153    at OneDriveVersionCleaner.Program.ProcessItems(ClientContext context, ListItemCollection items) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master [webexpert]\OneDriveVersionCleaner\Program.cs:line 141    at OneDriveVersionCleaner.Program.ProcessFolder(ClientContext context, List list, String rootRelativeUrl) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master [webexpert]\OneDriveVersionCleaner\Program.cs:line 126    at OneDriveVersionCleaner.Program.Main(String[] args) in C:\Users\Me\Desktop\OneDriveVersionCleaner-master [webexpert]\OneDriveVersionCleaner\Program.cs:line 55

I now noticed that webbexpert has posted a pull request fix for the issues.

I have now noticed that webbexpert has posted a Pull request for the recursive. Although their version still results in Operation is not valid in some situations.

Hello @Presjar,
I wrote this code as an experiment, and at the moment I don't have time to fully investigate this issue. Thanks for reporting it though, I let it here for others to see and comment.
I'm not sure, but my gut feeling is that this issue happens when multiple processes access the file at the same time. Maybe a template operation is not finished or a workflow is updating the properties of the file or a client is current viewing or synchronizing it.

Thanks for the reply. Understand you already have other things to be doing.
365 is so painful to use. Why there isn't an option in my personal drive to clear all old versions is beyond me. I'm not even able to turn off the Function.


Add suffix to Half library name

This was necessary for me to build on macOS.

Hmm... that's odd.  I wonder if the Ilm libraries build with different names on MacOS?  On Linux on my machine, the library files are named like so (building them from source):
libHalf.a      libIexMath-2_2.a     libIlmThread-2_2.a
libHalf.la     libIexMath-2_2.la    libIlmThread-2_2.la
libIex-2_2.a   libIlmImf-2_2.a      libImath-2_2.a
libIex-2_2.la  libIlmImfUtil-2_2.a  libImath-2_2.la

The libHalf library files don't have the -2_2 suffix.  So I think this PR would break things on Linux...?  Can you double-check what your library files are named?

Hey, I think this is an issue with version 2.3.  Just tried building on Linux, and I got:
openexr2.3/lib/libIex-2_3_s.a
openexr2.3/lib/libIlmImf-2_3_s.a
openexr2.3/lib/libIexMath-2_3_s.a
openexr2.3/lib/libHalf-2_3_s.a
openexr2.3/lib/libIlmImfUtil-2_3_s.a
openexr2.3/lib/libImath-2_3_s.a
openexr2.3/lib/libIlmThread-2_3_s.a

My guess: there should be an option in the build to remove the suffix which is now the default but wasn't in previous versions. Could be unintentional from the lib's maintainer.


Bug Report : Continuous posting

Continuously posting the same content every 5 minutes。Please check it。

shall be fixed in https://github.com/k-t-corp/nitter-xposter/commit/e0c6915c27399bca2d79b536b50306eb75395cab


Allow choice of WebDAV

This should likely be closed, as we had tons of problems with WebDAV, and the GradleRIO ssh setup is much better. Unless WebDAV has uses outside of FRC (I don't see many)


issues trying to install imagededup on mac

https://gist.github.com/yosun/67024b33998d9eaf59018fcf29908d4e

kindly change the setup.py requirements to remove the constraints on pillow, and it should work

We just released a new version. Please feel free to try it out.


Could you please share the 'run_openmm_simulation.py'

Could you please share the 'run_openmm_simulation.py' file on your GitHub repository?
I'm interested in taking a look at the code and understanding how it works. Thank you in advance for your help!

https://openmm.github.io/openmm-cookbook/latest/index.html


Plugin tries to convert SVG images.

Actual behaviour
If you have SVG images in the mediamanager the plugin tries to convert the image and adds webp thumbnails to the image object.
The template then puts those broken webp links in the  tag and the browser shows a broken image.
Expected behaviour
Do not handle SVG (vector) image types.
Environment
PHP 7.3, Shopware 5.6.6, Latest Plugin version
Steps to reproduce

upload SVG to media manager
use it somewhere on the site
check for brokem image.

Checklist

[ x] I have checked all issues and haven't found a similar report.
[ x] I am using the latest stable version of this plugin.

--
♥ ANY INCOMPLETE REPORT WILL BE CLOSED RIGHT AWAY ♥

Can you try the master branch?

https://github.com/FriendsOfShopware/FroshWebP/issues/62

@DanMan I searched the issues for "svg" no result for #62.
@shyim Thanks, master branch fixes my issue. Closing.


never type shouldn't be usable to variable and parameter declarations

It means unreachable code. So these declarations are useless and dangerous.
TypeScript Version:  2.3
Code
var a: never;
function f(p: never) {}

Expected behavior:
error
Actual behavior:
pass

But being able to define assertNever is kind of supposed to be a feature to ensure exhaustiveness checking. Check this out:
function handleMessage(msg: "subscribe" | "unsubscribe") {
    switch (msg) {
        case "subscribe":
        case "unsubscribe":
            console.log("Handled a " + msg);
            return;
        default:
            assertNever(msg);
    }
}

function assertNever(param: never) {
    throw new Error("Unhandled value: \n" + JSON.stringify(param));
}

Looks like a tslint rule rather than a compiler check.


Tweet layout

View hierarchy
I took a look at the view hierarchy of the app. I found the layout for the tweet layout too complex. So I refactored this.
@ligi @cbalster @stephanlindauer
Please run this on your devices (also tablets) to check if tweets still layout fine.
View hierarchy before

View hierarchy after

RTL support
I also added the needed attributes to prepare the view for right-to-left mode.

nice! I think this is safe to merge, so :+1:


chore: add 1.29.2 binary to cache

What type of PR is this?
/kind feature

What this PR does / why we need it:
Add 1.29.2 binaries into cache.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.39%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2292

Relevant Lines:
2887

💛  - Coveralls


Color function and percents seems confusing

Consider the following (outputs as comments):
new Color("color(srgb 100% 50% 30%)"); // rgb(100% 50% 30%)
new Color("color(lab 100% 50 30)"); // lab(1% 50 30)

I think I understand the intention:

Percents in color functions will always be translated to a range between 0 and 1.
Floats/integers are always taken at face value.

So with sRGB, this maps to sane values because they store their data in the range 0 - 1, but with Lab 100% maps to 1% as that space does not scale the color channels which makes sense as, for instance, a and b are theoretically unbound so they do not reasonably fit the mold that sRGB channels do.
So here are the questions:

Should percent syntax not be used with the color function if using color spaces Lab, LCH, HSL, HWB, etc?
Are percents in this case supposed to work for Lab, LCH, and others, but scaled differently than RGB color spaces?
Should colors like Lab scale their percent channels to be between 0 - 1 internally, but not the other channels?
Is this a one-way conversion? So 100% will translate to 1 but if you enter color(lab 1 0 0) you will not get 100%, but you should get 1%? And if 1 should translate to 100%, do you not scale the a channel of 50 to be 100 * 50 (meaning that the logic differs based on the channel)?

Basically, I can't tell if the translation of percents is supposed to be just a general handling of percent inputs in color functions, or if it is supposed to be color space aware. It is unclear looking at the level 4 color spec, so I'm not sure if the implementation here represents reality or not.

CIE Lightness is defined to use a 0 to 100 scale, where 100 is the same luminance as the reference white.
Industry practice is to not use this as a type of chromatic adapatation, but to first use a Bradford transform to transform to D50; and the reference white is D50 white.

with Lab 100% maps to 1%

This is why, when I finally caved on adding % to CIE Lightness  in CSS Color 4 (which is more CSS-like, but goes against entire industry practice) I made it mandatory. So there is no confusion with L=1 meaning 100 or 1, because it always takes a percentage. (And it is a percentage, I guess, of the luminance level of the reference diffuse white).
color.js though allows both 0-100 and 0-100% when specifying a CIE Lightness. If someone uses percent, they are most likely thinking of the CSS syntax, so I want 100% to map to  a Lightness of 100, not 1.
If I convert, say, papayawhip to Lab I get coords of [95.229, 2.539, 14.676]. color.toString() wil output lab(95.229% 2.539 14.676) (but Color.prototype.toString.call(color) will indeed output color(lab 95.229 2.539 14.676) rather than the CSS-compatible color(lab 95.229% 2.539 14.676)).

new Color("color(lab 100% 50 30)"); // lab(1% 50 30)

That is unintended, and a bug.

Great! This clarifies things quite a bit, so there should be a little awareness for percentage only channels.

A similar comment was made about CSS Color 4 which resulted in explicitly stating where percentages were allowed, required or forbidden
Related, I noticed this too without remembering this issue, so Percentages mishandled in color(lab L a b).
I also noticed a similar error in a CSS implementation, which was treating 50% as L=0.5 not L=100.
My preference would have been to not allow percent, which is merely a source of confusion and non-interoperability.

Thanks for the heads up. The spec being explicit about this kind of thing really takes out the guess work, so much appreciated!

CSS Color 4 now defines explicit reference ranges for each color space, so that interconversion between numbers and percents is clearly defined in all cases. For example for OKLab and OKLCH

Yep, I think I need to clean up and commit those new CSS changes. I think things are pretty clear now.


colab notebooks broken by upstream package changes

Right now this notebook and others from this page https://docs.openforcefield.org/en/latest/examples.html fail when running on colab with the error RuntimeError: No package can be installed for pin: cudatoolkit 12.2.*
This error is documented here: https://github.com/conda-incubator/condacolab/issues/60
Using this command:
!pip install -U https://github.com/conda-incubator/condacolab/archive/cuda-version-12.tar.gz
instead of
!pip install -q condacolab
to install condacolab works and is a good temporary solution until https://github.com/conda-incubator/condacolab/pull/63 gets merged in.
I made #30 if you are looking for a temp fix

Thanks Mike! @Yoshanuikabundi could you take a look at this and the linked PR at your earliest convenience?


Merge index exceptions with same handling

Tested with PHP 7.3 and 8.1

I'm not going to approve this as the index file needs to compile in PHP 5 to display a proper error to the user if he accidentally used an older version of PHP.

@mahagr Line 16 with PHP 7.0 so this file won't work in PHP 5 (?? syntax) and 65 is long time EOL and should run for last 3 years on anything production related. exception|exception was added in 7.1 and 7.2 is much more common of all the EOL PHP versions.

Hmm, true. We haven't had any complaints on it lately and that has been there for a few releases... So OK, it looks like people aren't trying to install Grav to PHP 5 servers anymore.


Create a man page for 'crystal' binary

man pages is a standard UNIX way to document how tools work. It would be great if crystal had a man page as well.

Sure! If anyone wants to do it, please send a pull request :-)

See PR #2147.

Looks like #2147 was abandoned, adoption highly appreciated.
git remote add upstream https://github.com/crystal-lang/crystal.git
git fetch upstream master
git fetch upstream refs/pull/2147/head
git checkout -b manpage FETCH_HEAD
git rebase -i upstream/master # Squash stuff together
# Fix issues mentioned in #2147
# Update for things that changed since

Hm... I guess I'll take this. :smiley:

Just a small question: what's the benefit of a man page in comparison to crystal --help?

People look for it to find help and you can be a lot more verbose since man calls a pager by default. It's just good practice to have one.

Check this:

git --help: quick usage reference;
man git: full reference with all sub-commands, constants, lots of details for each argument, and more.

#triage Re-tag as community:in-progress as there's an open PR that's waiting for someone to review, provide feedback.

The man page is in the repo, but it seems not to be bundled in the Ubuntu package at least as it is not accessible in /usr/share/man/man1.

Same on Debian:
➜ man crystal
No manual entry for crystal
See 'man 7 undocumented' for help when manual pages are not available.

Reopening this because otherwise there are super high chances that we won't go the last mile.

Same on Manjaro Linux:
>>> man crystal
No manual entry for crystal
>>> crystal -v
Crystal 0.23.0 (2017-07-13) LLVM 4.0.1

Closed in favor of https://github.com/crystal-lang/omnibus-crystal/issues/21