Dragos Ciupureanu

Results 9 comments of Dragos Ciupureanu

Although functionally the PR is ready I want to update the readme too. Is this done automatically by the pipeline? Sorry, I couldn't find instructions on how to do it...

Apologies, didn't get a chance to revisit and address the comments. I expect to have an updated PR over the weekend.

I believe this is ready. Unfortunately, I cannot run the tests locally due to an issue with bats on MacOS

I see. In that case I guess we'll wait for them to update (or one of their dependencies) as they are still forcing pydantic

Hi, I'm reopening this as `transformers` have updated their code to use `pydantic2` https://github.com/huggingface/transformers/issues/27933 and it feels like `ludwig` could remove the version restriction.

Hi @Gowiem - I'll get this rebased and changed over the weekend. 👍

I renamed my fork's branch and this PR was closed automatically. A new one is open here https://github.com/cloudposse/terraform-aws-ecr/pull/106

I see the documentation has been updated with examples containing functionality that wasn't released yet (namely the AWS Sigv4 signer) - I'd have thought that everything that's on "main" is...

I don't know if it's visible on the opensearch.org website, but I came here looking for how to do sigv4 and stumbled across this page https://github.com/opensearch-project/opensearch-js/blob/main/USER_GUIDE.md#using-aws-v2-sdk that's referenced from the...