SpamAssassin reports DKIM invalid when email contains attachment

Hello,
I've been noticing that pretty much every email that contains an attachment, even if coming from @outlook.com or @gmail.com, for example, is marked with DKIM invalid.
The emails are 100% legit, because I reply to them and receive a reply. And, the emails without attachment are marked as DKIM valid.
I'm just wondering... could this be a problem with my setup? A bug with docker-mailserver? Or, a problem with SpamAssassin or OpenDKIM?
Thanks!
Nuno

How are you validating the DKIM?
I noticed something similar. When I send an email with attachement from a Google Account via Thunderbird, the Thunderbird plugin for DKIM validation shows an error. If I send the email directly via Google Webservice, then I don't get an error. Within the e-mail, both e-mails have dkim=pass.

I was mentioning about received emails, not sent by me. Most of emails I send are through AWS SES, so I hope they work fine.
I use the Mail::SpamAssassin::Plugin::DKIM extension of SpamAssassin, which adds DKIM_INVALID & DKIM_SIGNED to the X-Spam-Status header.
  header	__DKIM_EXISTS	exists:DKIM-Signature

  meta	   DKIM_INVALID	!__DKIM_EXISTS || !DKIM_VALID
  describe DKIM_INVALID	DKIM-Signature header does not exist or is not valid
  score    DKIM_INVALID 5.0

I was mentioning about received emails, not sent by me. Most of emails I send are through AWS SES, so I hope they work fine.
I use the Mail::SpamAssassin::Plugin::DKIM extension of SpamAssassin, which adds DKIM_INVALID & DKIM_SIGNED to the X-Spam-Status header.
  header	__DKIM_EXISTS	exists:DKIM-Signature

  meta	   DKIM_INVALID	!__DKIM_EXISTS || !DKIM_VALID
  describe DKIM_INVALID	DKIM-Signature header does not exist or is not valid
  score    DKIM_INVALID 5.0

And the emails do have DKIM header, but SpamAssassin sees them as invalid:
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com;
 s=selector1;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
...

I also mentioned the received emails that were not sent by the Docker mail server.

I was mentioning about received emails, not sent by me. Most of emails I send are through AWS SES, so I hope they work fine.
I use the Mail::SpamAssassin::Plugin::DKIM extension of SpamAssassin, which adds DKIM_INVALID & DKIM_SIGNED to the X-Spam-Status header.
  header	__DKIM_EXISTS	exists:DKIM-Signature

  meta	   DKIM_INVALID	!__DKIM_EXISTS || !DKIM_VALID
  describe DKIM_INVALID	DKIM-Signature header does not exist or is not valid
  score    DKIM_INVALID 5.0

And the emails do have DKIM header, but SpamAssassin sees them as invalid:
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com;
 s=selector1;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
...

DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=20161025;
        h=mime-version:references:in-reply-to:from:date:message-id:subject:to;
...

I also mentioned the received emails that were not sent by the Docker mail server.

Oh ok. I see.
It is not Thunderbird that generates the DKIM headers - it's Gmail/Outlook themselves.
So this does seem like a bug with SpamAssassin, as I doubt both Gmail and Outlook would be generating invalid DKIMs when emails have attachments.

It is not Thunderbird that generates the DKIM headers - it's Gmail/Outlook themselves.
That's obvious.

I use a DKIM verifier plugin for Thunderbird (https://addons.thunderbird.net/de/thunderbird/addon/dkim-verifier). Probably SpamAssassin and this plugin are not able to check the DKIM correctly ...

Perhaps there is a correlation with ARC (https://en.wikipedia.org/wiki/Authenticated_Received_Chain)?

Thank you very much.
(sorry for the many dup messages above - I obviously completely messed up here, doing a number of different tasks simultaneously)
I submitted a report in SpamAssassin: https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7827
Unfortunately I'm not aware of ARC. Is there anything I can check in logs or so?

I do not know enough about the subject. It was just a wild guess with the ARC.

Bug marked as invalid with following response:

This cannot be considered a SpamAssassin bug unless you have some other
trustworthy DKIM signature verifier which validates a signed message that SA
deems invalid.

Mail systems routinely make harmless changes to email which break DKIM
signatures. This can include an encoding change that may be necessary to
transport a binary attachment over Internet email.

It is literally impossible to diagnose your problem without a specific
UNMODIFIED complete example.

This is the sort of probable non-bug problem which should be better
addressed on the SpamAssassin Users mailing list.

Redefining & rescoring DKIM_INVALID as you have done is deeply unwise. The
definition in the default ruleset only hits on signed messages whose signature
does not validate and it is scored at 0.1 because of the common problem of DKIM
signatures breaking in transit. Marking every unsigned email and every signed
email with a broken signature as spam (i.e. with a 5.0 score as your rule does)
is a choice to intentionally mislabel mail.


Make package.json main point to src/seamless-immutable.js

As it stands, if I require('seamless-immutable') I get the development version, and if I want to get the prod version, I have to change my code.  If main in package.json simply pointed to src/seamless-immutable.js, then I wouldn't have to change my code, and whether the prod or dev version gets used simply depends on process.env.NODE_ENV (which src/seamless-immutable.js is using anyway, so I don't understand why it's not the main).
As I'm sure you already know, this change would make it work just the same as React, and for those of us who are using Webpack with dev/prod configs for React, they would work right off the bat with seamless-immutable if you make this change.

+1

@rtfeldman are there any objections that prevent this from being merged into master?

Nope! Just hadn't had time to take a look at it. Now I feel bad because it's a one-line change that wouldn't affect our build process and sounds like it would improve yours.
Thanks!

Thanks @rtfeldman!


chore(ssa refactor): Use types from ssa ir directly in acir context

Description
An example of what #1529 could look like

Problem*

Resolves
Summary*

This PR sets out to
Example

Before:

After:

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This is still in the experimental phase and we need to be a bit careful because previously we were semantically saying that only integers can have a bit size, wheras now this allows field elements to have a fixed bit size too. The relevance being that we don't want to be able to do a comparison with a field element. This should be caught by the frontend though

Made a quick Proof of concept (not mergable) that shows the impact of removing tracking for blackbox functions

Got a link?


Select * from table not working

Hello,
Perhaps I'm doing something wrong.
I am attempt to use the column selector on a select * from table statement to populate the column list, but it responds with an error. "Error retrieving table schema".
If the command is not meant to function this way, please consider implementing this. If it is, Please advise on how to correct my error.
Thank you.

Hello,
There seems to be an issue with the tool’s ability to handle connections to the default database.  The workaround is to select a database from the connection settings to resolve this issue.
Thanks for reporting this issues!


fix: make-lint uses git safe directory

Closes #238
The issue only presents itself in Windows & WSL environments - the is not a git repository message is confusing, but if you exec into the container and run a git status you'll see a more verbose error:
fatal: detected dubious ownership in repository at '/workdir'
To add an exception for this directory, call:

        git config --global --add safe.directory /workdir

Some additional documentation about the error.
Fixed by using the --add safe.directory cmd

Also happens in macos :)

that's weird, last time I switched from my WSL environment to MacOS and didnt get any errors on there... FYI it's an M1 Macbook

My issue started from MacOs Sonoma (version 14.0)

Hi folks. This is happening because the lint container is running as "root" and the files that is has in /workdir do have the permissions set to your local user. That's why git is warning about the dubious ownership.
One way to work around this would be to run the container actually as the current user by passing the -u flag:
@docker run --rm --workdir /workdir --volume "$(CURRENT_DIR):/workdir" -u $(UID) quay.io/helmpack/chart-testing:v3.9.0 ct lint --target-branch master

But that brings another problem:
>>> helm repo add bitnami https://charts.bitnami.com/bitnami
Error: mkdir /.config: permission denied

That's because helm is trying to fetch the repos into some local dir that uses to be in the users home dir. But in this case the user doesn't have a home dir. So a workaround could be passing it HOME=/tmp.
So in that case we end up with:
@docker run --rm --workdir /workdir --volume "$(CURRENT_DIR):/workdir" -u $(UID) -e "HOME=/tmp" quay.io/helmpack/chart-testing:v3.9.0 ct lint --target-branch master

This works on my machine. Can you maybe try it @paolofacchinetti ? And if it works, could you update the PR with that? Thanks!

Oh. Update @paolofacchinetti . I accidentally already pushed my recommendation up to master: https://github.com/ethpandaops/ethereum-helm-charts/pull/249 . Sorry about that. But I would still appreciate any feedback. Thanks! 🙏

Don't worry @skylenet, appreciate the feedback. I've just tested the master branch on WSL and MacOS and it seems to be working fine!
Closing this PR now since it's fixed


Make zoom scale customizable

PR #487 changed the default zoom behavior (minScale and boundaryMargin). However, the new values seem to be for a specific use case and not desirable for standard PDFs. Version 2.6.0 had a much nicer default behavior.
This PR exposes these settings to allow for customization of these values:
PdfViewPinch(
  controller: pdfPinchController,
  minScale: 0.25,
  maxScale: 10.0,
);

@edbentinck thanks for your contribution!

published pdfx 2.8.0


RevokeRefreshTokenAsync doesn't clear out all refresh_tokens that have the same subject/client_id combination

Issue / Steps to reproduce the problem
I am able to mint 2 tokens via Extension Grants that have the same subject.
Revoking the refresh_token removes the refresh_token, but then doesn't remove all the others that have the same subject.  Looking at the code, it tries to remove all the reference tokens by subjectId/ClientId.
i.e.
/// <summary>
/// Revoke refresh token only if it belongs to client doing the request
/// </summary>
protected virtual async Task<bool> RevokeRefreshTokenAsync(TokenRevocationRequestValidationResult validationResult)
{
    var token = await RefreshTokenStore.GetRefreshTokenAsync(validationResult.Token);

    if (token != null)
    {
        if (token.ClientId == validationResult.Client.ClientId)
        {
            Logger.LogDebug("Refresh token revoked");
            await RefreshTokenStore.RemoveRefreshTokenAsync(validationResult.Token);
            await ReferenceTokenStore.RemoveReferenceTokensAsync(token.SubjectId, token.ClientId);
        }
        else
        {
            Logger.LogWarning("Client {clientId} tried to revoke a refresh token belonging to a different client: {clientId}", validationResult.Client.ClientId, token.ClientId);
        }

        return true;
    }

    return false;
}

My Unit Test is located Here..
[TestMethod]
public async Task Mint_multi_arbitrary_resource_owner_and_refresh_and_revoke()
{
    ...
}

I assume that the call to RemoveReferenceTokensAsync is meant to account for multiples.
I would think that the following;
 await RefreshTokenStore.RemoveRefreshTokenAsync(validationResult.Token);

needs to be replaced with a new remove;
await ReferenceTokenStore.RemoveRefreshTokensAsync(token.SubjectId, token.ClientId);

I have a workaround that shows how to deal with this until we get the new feature in.
workaround

Also, revoking all refresh_tokens with
await ReferenceTokenStore.RemoveRefreshTokensAsync(token.SubjectId, token.ClientId);

may not be enough.
I have a situation where I want all tokens removed based on only a subject.
await ReferenceTokenStore.RemoveRefreshTokensAsync(token.SubjectId);

My use case is as follows;

Client logins in and only gets back a token that is scoped to account_only information.  Pretty much all that I usually want from logging into google.
Client then using a custom extension_grant mints another token with custom scopes/claims with the login only access_token  being passed in as in the following grant_type;

POST http://localhost:21354/connect/token

Headers:
   Content-Type:application/x-www-form-urlencoded

Body:
   grant_type:arbitrary_oidc_resource_owner
   client_id:arbitrary-resource-owner-client
   client_secret:secret
   scope:offline_access nitro metal
   arbitrary_claims:{"some-guid":"1234abcd","In":"Flames"}
   access_token_lifetime:3600
   access_token:{{token from a previous login}}

I have many client_id(s) in play, all deriving their custom tokens from the original subject.  The subject is the only thing carried forward when clients mint custom tokens.
When a refresh_token gets revoked, all subjects from all clients have to go.

This is the opposite issue for https://github.com/IdentityServer/IdentityServer4/issues/1315
It changed the behavior of revoke all to only revoke 1 refresh token.
But it seems that no extra endpoint is added back for the revoke all functionality.

Yes, I think it would be useful to be able to delete all tokens for user
await ReferenceTokenStore.RemoveRefreshTokensAsync(token.SubjectId);
If, for example, refresh tokens are used in native applications and stored on user laptop or phone, and then he lost his device.

to clarify, currently there is no endpoint which will remove all refresh tokens for specified subjectId and clientId? I see that there is a method in IRefreshTokenStore but it's not used anywehere.
Currently Token revocation endpoint removes only one specified token. To handle that it'd require us to create custom endpoint, am i right?

Currently Token revocation endpoint removes only one specified token. To handle that it'd require us to create custom endpoint, am i right?

Yes.


New compile scheme

hi folks,
i develop, hack CB since a while, and few thing itch me, the compilation scheme is one of them
not be able to recompile deps and other CB apps, especially when hacking CB it self.
this pull request allow compilation like n2o does
https://github.com/synrc/active/
debian/ubuntu:
sudo apt-get install inotify-tools

for other system: mac or windows check this link
https://github.com/synrc/fs
i suggest you to have a look to their framework (clever people).
few thing to know is deps app are compiled with aleppo during the dev cycle
which is maybe bad.
benefit :

shorter dev cycle,
shorter hacking CB cycle,
no difference about rendering time between dev & prod,
no more complain to being slow
(in fact there is still a small difference)

cheers.

Good to hear that. But this pull request "The Travis CI build failed"

@dcy, actually, any PR will fail, as there is bunch of problems with dialyzer and even tests.

i think i have fixed most of the problem for this pull request

this time i hope it is my last commit on this PR,
i didn't want to make a such big PR but it endup like this,
issue fixed for 302, 227, 304, 322, 323: have to be confirmed,
plz test it on windows and mac.

beam going crazy :(

what do you mean?

false alarm :),
i use entop to spot the pid consuming a lot of reduction,
(o2o_adm@xyz)7> process_info(erlang:list_to_pid("<0.542.0>"),[current_function]).
[{current_function,{paginate,page_before,4}}]
then 90% it is a bulshit code i made in one of my controller.

I personally have a problem with adding inotify-tools as dependency. We already have some issues with CB on windows because of merl. Adding another requirement for system library on linux, may cause problems for users who don't have privilages to install software on their system.
I could propose to add configuration option, switching between old and new behavior. In such case, usage of inotify-tools will be optional.

hi, it is possible to make it optional an keep the both compile scheme.
but need more work, i see what i can do.

i had a quick look, it will be hard to maintain the both (personnaly i don't need it)
some extra work i can afford now (busy with my current project),
maybe later (or someone can take over this branch to make it work with the old scheme)
i made this pull to make my dev life a bit better, and it is now
it fix many issue i had when i develop a CB App.
maybe you can propose this version to people want to try and
get some feedback from the folk.
cheers
Chan.

@mihawk, problems? again?

no problem so far,
but i don't want to maintain this code,
i will be busy on other project.


Refactor tap to place and fix a few issues

FIrst I only wanted to do some cleanup but then found a few issues in the TapToPlace test scene:

Placing an object on another one will put both in IsBeingPlaced. Fixed this by setting TapToPlace as Modal rather than just a global listener.
When moving a child with PlaceParentOnTap the parent or siblings can still get hit by ray cast because only the child itself gets the IgnoreRayCast layer. Setting the parent recursively instead when it's used.
RoomMesh was missing in the test scene.

I refactored quite a bit of the code as it was very cryptic what it was doing like that nested tertiary operator with a ray cast. I've split the SetLayerRecursively method into 2 parts to make it clear that it does cache and moved it into GameObjectExtensions. There was another SetLayerRecursively in Utils that I've also moved in there and set obsolete.
void SetLayerRecursively(this GameObject root, int layer)
void SetLayerRecursively(this GameObject root, int layer, out Dictionary<GameObject, int> cache)
void ApplyLayerCacheRecursively(this GameObject root, Dictionary<GameObject, int> cache)

I've separated the traverse of the transform hierarchy and moved it into TransformExtensions, using a queue instead of recursive method calls.
IEnumerable<Transform> IterateHierarchy(this Transform root)
IEnumerable<Transform> IterateHierarchy(this Transform root, ICollection<Transform> ignore)
e.g.
foreach (var child in root.transform.IterateHierarchy())

The new extension methods are covered by several integration tests since we need some of those.
There are still a few methods that all use their own way of doing a recursive traverse, could probably clean those up too.
These 3 methods are still in TapToPlace but could be moved somewhere else, if so I wasn't sure where to put them.
static Vector3 GetPlacementPosition(Vector3 headPosition, Vector3 gazeDirection, float defaultGazeDistance)
static bool SpatialMappingRaycast(Vector3 origin, Vector3 direction, out RaycastHit spatialMapHit)
static Vector3 GetGazePlacementPosition(Vector3 headPosition, Vector3 gazeDirection, float defaultGazeDistance)

@Zod-,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Is there any reason there are 2 extensions for getting the full path of an object in a hierarchy?
There is
TransformExtensions.GetFullPath(this Transform transform, string delimiter = ".", string prefix = "/")
GameObjectExtensions.GetFullPath(this GameObject go)

Each with different results and implementations. We could just link the GameObject one to the more customizable Transform one if we want to keep it.

Is there any reason there are 2 extensions for getting the full path of an object in a hierarchy?

I'm okay with removing one of them.

I've written some more tests in a branch based on this one that doesn't include any changes to classes in the main toolkit.


question for Svelte book

@bestguy I opened an issue because I couldn't find another way to contact you. I'm writing the Manning book "Svelte and Sapper in Action". It contains a short section on integration with CSS libraries like Bootstrap. I'm planning to mention your sveltestrap library and include an example of using it.
Would you say it is not possible to use Bootstrap components in a Svelte app or just difficult. If I understand correctly, you reimplemented nearly all of the Bootstrap components as Svelte components which is great! Did you feel that was the only way to make them available in Svelte apps?

Hi @mvolkmann  Oh nice!   Thank you for reaching out.   Mentioning is fine by me and appreciated.

"Would you say it is not possible to use Bootstrap components in a Svelte app or just difficult"

I think that phrasing isn't correct; the purpose of Sveltestrap is to make using Bootstrap easier and offer a more Svelte-friendly interface as components.  More specifically this specifically means:

You need less markup/"div-soup" with the various combinations of bootstrap class names.  For example Carousels:

HTML:

    Previous

    Next

Sveltestrap:
const items = ['src1...', 'src2...', 'src3...'];
let activeIndex = 0;

<Carousel {items} bind:activeIndex>

    {#each items as item}
      <CarouselItem bind:activeIndex itemIndex={index}>

      </CarouselItem>
    {/each}

  <CarouselControl direction="prev" bind:activeIndex {items} />
  <CarouselControl direction="next" bind:activeIndex {items} />
</Carousel>

IMO Svelte components are a lot clearer vs parsing the lower level HTML.

Related to above, Bootstrap offers it's own JavaScript needed to use many of its interactive components, e.g.:

$('.carousel').carousel();

While fine, the drawbacks are Bootstrap currently depends on jQuery, and does not 'play well' with dynamic UI frameworks like React, Svelte, etc.  To integrate the DOM-based JavaScript API is a little awkward.
Also, the JavaScript is a complete library and hard to selectively include or use when you are only using a few components.
Sveltestrap takes a different approach and does not depend on Bootstrap's JS. It instead implements equivalent behavior with Vanilla JS and/or Svelte plugins.  This allows for more selective includes of JS, smaller bundles, and generally a more "Svelte-native" experience so that you don't need to separately initialize components.
Some of this is certainly subjective, but being a UI developer, I find the component-centric libraries like Sveltestrap much easier to develop and maintain.  I started Sveltestrap since I didn't see good alternatives at the time, and grew tired of the div-soup when using Bootstrap.
Does this help clarify?

@mvolkmann the TL;DR is using Bootstrap's class name HTML+JS based components directly are awkward compared to using reusable components designed for Svelte.  They use the same CSS under the hood so you get the benefits of Bootstrap without most of the drawbacks.

Thanks, makes sense! When the book comes out you’ll see an example of using your library in the “Advanced Svelte” chapter.


Expo release build requires .env.production and ignores .env

Describe the bug
My Expo application OAuth functionality started failing in release mode (eas build). After debugging for a while, I saw that a variable that is resolved from .env is no longer being captured. After adding a new file .env.production with the save value the functionality started working.  The .env file should be able to work for both development and production NODE_ENV values.
If requested I can whip up a bare application and
To Reproduce
Steps to reproduce the behavior:

Setup a project with a value in .env
eas build --local --profile preview // Preview has all the settings of release and has apk.
adb install <build.apk>
Importing the variable should fail.

Expected behavior
.env file contents should be picked up and if there is an env.production file it should override those values.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: Android and Iphone device tested

Additional context
Add any other context about the problem here.
babel.config.js:
module.exports = function(api) {
  api.cache(true);
  return {
    presets: ['babel-preset-expo'],
    plugins: [
      ["module:react-native-dotenv"],
      ["transform-remove-console", { "exclude": ["error", "warn"] }]
    ]
  };
};

eas.json
{
  "cli": {
    "version": ">= 2.8.0"
  },
  "build": {
    "development": {
      "channel": "development",
      "distribution": "internal",
      "developmentClient": true,
      "android": {
        "buildType": "apk"
      },
      "ios": {
        "simulator": true
      }
    },
    "preview": {
      "distribution": "internal",
      "developmentClient": false,
      "channel": "development",
      "android": {
        "buildType": "apk",
        "gradleCommand": ":app:assembleRelease"
      },
      "ios": {
        "simulator": true
      }
    },
    "production": {
      "distribution": "store",
      "channel": "stable",
      "android": {
        "buildType": "app-bundle"
      }
    }
  },
  "submit": {
    "production": {}
  }
}

api.cache(false) or upgrade to latest babel

How does the cache fix the issue? can you give me more info ?
    "@babel/core": "^7.20.2",
    "babel-plugin-transform-inline-environment-variables": "^0.4.4",
    ```

This library is a Babel plugin. Babel caches all non JavaScript files that are read including .env files by default
Earlier this year, I worked with the Babel team to add a feature to make exceptions. This feature was integrated, tested, and works well. Expo has not upgraded to the latest version despite my pleas.

I can reproduce this on a fresh expo project next week but incase you missed my problem, I suspect that release builds will never pickup variables in .env file. Are you confident that .env will be grabbed in production mode?  I already have the latest version of the plugin and this package.

See test repo https://github.com/goatandsheep/react-native-dotenv-expo-test

By comparing delta with this project, I have found that in react-native-dotenv can pick up my local .env file when I use  npx expo start but fails to do so when I run eas buid. Note that the test project is working fine (both eas build and expo start successfully pickup the file). So the issue is not about picking up .env.production but about picking up any env files when running with eas build.
As far as I know my .env is placed correctly:  I have src/app/ directory structure and .env is at the root of the project (i.e at the same level as src folder)
Can you think of a reason why this might be?   How does the library infer the root path? Would setting path='./' help?

I'd recommend putting your secrets on EAS through the online portal

Any other suggestions? I would prefer to use .env. Here is the log output of the failure
[RUN_GRADLEW] transform[stderr]: react-native-dotenv Error: ENOENT: no such file or directory, open '.env'
[RUN_GRADLEW] transform[stderr]:     at Object.openSync (node:fs:590:3)
[RUN_GRADLEW] transform[stderr]:     at readFileSync (node:fs:458:35)
[RUN_GRADLEW] transform[stderr]:     at parseDotenvFile (/private/var/folders/12/jzrnnf0s6yzdp4f7gh1by3600000gn/T/eas-build-local-nodejs/692b0e7a-8b8c-4676-9cc9-e9a1f9b8d18d/build/spark-mobile/node_modules/react-native-dotenv/index.js:9:15)
[RUN_GRADLEW] transform[stderr]:     at module.exports (/private/var/folders/12/jzrnnf0s6yzdp4f7gh1by3600000gn/T/eas-build-local-nodejs/692b0e7a-8b8c-4676-9cc9-e9a1f9b8d18d/build/spark-mobile/node_modules/react-native-dotenv/index.js:89:18)
[RUN_GRADLEW] transform[stderr]:     at sync (/private/var/folders/12/jzrnnf0s6yzdp4f7gh1by3600000gn/T/eas-build-local-nodejs/692b0e7a-8b8c-4676-9cc9-e9a1f9b8d18d/build/spark-mobile/node_modules/@babel/core/lib/gensync-utils/async.js:33:25)
[RUN_GRADLEW] transform[stderr]:     at sync (/private/var/folders/12/jzrnnf0s6yzdp4f7gh1by3600000gn/T/eas-build-local-nodejs/692b0e7a-8b8c-4676-9cc9-e9a1f9b8d18d/build/spark-mobile/node_modules/gensync/index.js:182:19)
[RUN_GRADLEW] transform[stderr]:     at /private/var/folders/12/jzrnnf0s6yzdp4f7gh1by3600000gn/T/eas-build-local-nodejs/692b0e7a-8b8c-4676-9cc9-e9a1f9b8d18d/build/spark-mobile/node_modules/gensync/index.js:210:24

I highly recommend against that because it is insecure but if you insist, then remove .env from your gitignore

Feel free to continue the discussion here https://github.com/goatandsheep/react-native-dotenv/discussions/388


Add support for specify proxy cookie path/domain

This change is

/lgtm

Coverage increased (+0.09%) to 46.236% when pulling 704a18cec9975d8ca7616df336bb4343afdabdf2 on gianrubio:custom-proxy-cookies into 96b1158a5b1a96be50add22a99c83640904b94c8 on kubernetes:master.

Coverage increased (+0.09%) to 46.236% when pulling 704a18cec9975d8ca7616df336bb4343afdabdf2 on gianrubio:custom-proxy-cookies into 96b1158a5b1a96be50add22a99c83640904b94c8 on kubernetes:master.

@gianrubio thanks!


Webpack rebuild error

OS?
Ubuntu 16.04
Versions
angular-cli: 1.0.0-beta.28.3
node: 7.4.0
os: linux x64
@angular/common: 2.4.5
@angular/compiler: 2.4.5
@angular/core: 2.4.5
@angular/forms: 2.4.5
@angular/http: 2.4.5
@angular/platform-browser: 2.4.5
@angular/platform-browser-dynamic: 2.4.5
@angular/router: 3.4.5
@angular/compiler-cli: 2.4.5
Repro steps.
Created the app with the angular-cli (ng new).  Ran ng serve to make sure everything was working.  All was good, webpack worked and reported:
Hash: 7a95e8c73d53b9f210c7
Time: 8041ms
chunk    {0} main.bundle.js, main.bundle.map (main) 17.2 kB {3} [initial] [rendered]
chunk    {1} styles.bundle.js, styles.bundle.map (styles) 205 kB {4} [initial] [rendered]
chunk    {2} scripts.bundle.js, scripts.bundle.map (scripts) 375 kB {4} [initial] [rendered]
chunk    {3} vendor.bundle.js, vendor.bundle.map (vendor) 2.83 MB [initial] [rendered]
chunk    {4} inline.bundle.js, inline.bundle.map (inline) 0 bytes [entry] [rendered]
webpack: Compiled successfully.

I was able to load the app in a browser.  Made a small change to the app.component.ts file (removed the trailing new line).  Saved the file, webpack attempted to re-bundle the file, but failed.
The log given by the failure.
Hash: 7a95e8c73d53b9f210c7
Time: 613ms
chunk    {0} main.bundle.js, main.bundle.map (main) 17.2 kB {3} [initial]
chunk    {1} styles.bundle.js, styles.bundle.map (styles) 205 kB {4} [initial]
chunk    {2} scripts.bundle.js, scripts.bundle.map (scripts) 375 kB {4} [initial]
chunk    {3} vendor.bundle.js, vendor.bundle.map (vendor) 2.83 MB [initial]
chunk    {4} inline.bundle.js, inline.bundle.map (inline) 0 bytes [entry]

ERROR in Cannot read property 'length' of undefined
webpack: Failed to compile.

same issue, works fine on first build, subsequent builds fail.
ERROR in ./src/app/campaigns/sequencer.ts
Module build failed: Error
    at WebpackCompilerHost.populateWebpackResolver (C:\msweb\studioweb\node_modules\@ngtools\webpack\src\compiler_host.js:218:51)
    at C:\msweb\studioweb\node_modules\@ngtools\webpack\src\plugin.js:352:33
    at process._tickCallback (internal/process/next_tick.js:103:7)
 @ ./src/app/campaigns/index.ts 26:0-40
 @ ./src async
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4208/ ./src/main.ts
webpack: Failed to compile.

Same issue ... :(
rebuilld is too fast 🥇

I agree, I started crying from joy of how fast it was... but started crying again when it crashed 👍 💯

Same here on ubuntu 16.10 and node 6.9.2.

I may be wrong, but I believe it may have to do with the fact that the typescript doesn't get compiled, or at least compiled correctly after the first initial build. See my other comment

I'm getting the same issue and have been debugging a little. Not found anything yet but here's an example project: https://github.com/intellix/angular-cli-b28/tree/master/src/app/lazy
I only have the problem when saving lazy SCSS or HTML. Saving a TS doesn't give Module build failed: error though

ok I reverted back to .26 so I can work for now but that was not enough.
I also had to:
npm install --save<EMAIL_ADDRESS>until .28 gets stable

@born2net I'll try it out

npm install --save<EMAIL_ADDRESS>
That's it! Works also with the new version.

so u r using @1.2.4  with .28 works?

pew, have to do a workshop tomorrow. saved my day/night

I have this problem.
Windows 10
Node 6.9.5
running ng serve
When I change an HTML file it throws in cmd:
ERROR in ./src/app/components/side-menu/side-menu.component.html Module build failed: Error     at WebpackCompilerHost.populateWebpackResolver (C:\sandbox\admin.marshallinstalls.co.nz\node_modules\@ngtools\webpack\src\compiler_host.js:218:51)     at C:\sandbox\admin.marshallinstalls.co.nz\node_modules\@ngtools\webpack\src\plugin.js:352:33     at process._tickCallback (internal/process/next_tick.js:103:7)  @ ./src/app/components/side-menu/side-menu.component.ts 21:22-59  @ ./src/app/app.module.ts  @ ./src/main.ts  @ multi main
However when I run ng serve --aot it compiles and runs just fine

beta 29 has the same error

The same thing with beta.30

I have updated to node 7.5 + cli beta.30 (from 6.9.5 & beta.28) and it seems the problem is no longer present.

npm install --save<EMAIL_ADDRESS>Worked for me too! yay

anybody facing this issue "@angular/cli": "^1.0.0-rc.0" in package.json?

I did some research based on my problem, maybe it helps someone else as well:
i introduced typings into the project, and since I did this, this problem happens with the live-reload of lazy loaded modules. No idea why this happens.

I'm using the oldschool way of typings as follows:
package.json:
{
  "scripts": {
    ...
    "install": "typings install"
  }
...
  dependencies: {
...
    "typings": "^2.1.0",
...
  }
}

typings.json:
{
  "globalDependencies": {
    "typings": "github:t4deon/FHIR-STU3-TS-Typings/typings.d.ts#e36b4cf4bb07734b97ea8d14893f7747fd990438"
  }
}

src/tsconfig.app.json:
...
{
  "files": [
    "../typings/index.d.ts"
  ]
}

this works for me
yarn add<EMAIL_ADDRESS>ng details:
@angular/cli: 1.0.0
node: 7.9.0
os: darwin x64
@angular/common: 4.0.2
@angular/compiler: 4.0.2
@angular/core: 4.0.2
@angular/forms: 4.0.2
@angular/http: 4.0.2
@angular/platform-browser: 4.0.2
@angular/platform-browser-dynamic: 4.0.2
@angular/router: 4.0.2
@angular/cli: 1.0.0
@angular/compiler-cli: 4.0.2
@ngtools/webpack: 1.3.0

hocam su adresde https://github.com/angular/angular-cli/issues/4326 da birkaç kişi sorun yaşamış
yarn add<EMAIL_ADDRESS>ile biri sorunu çözmüş  şimdiki güncel i
yarn add<EMAIL_ADDRESS>bir denermisiniz

@zinderud bende problem cözüldü bile aga


FAQ/Documentation Questions

The docs for Amphora need to be re-written, so what should be in there? I'd like to get a list of questions that need to be answered in the docs and then that will help determine structure.
Literally ask anything, it all helps. Answers won't be provided on this thread but will be answered in the docs that get written.

How can amphora be used outside of the Clay stack on its own? How could one write their own stack around amphora?

Piggybacking of Byron's question, maybe I can actually get the Clay stack documented. Includes plugins for Amphora, tooling, etc.

How can I write my own alternative version of amphora to plug into the Clay stack?

Can I use amphora with another NodeJS framework that's not Express?

Are there any performance benchmarks for Amphora?

How big is Amphora, in terms of file size?

What is amphora?
Why would I use it?
What does it do?
What doesn't it do?
What is it similar to?
Why is it better than choosing x?
Why redis? Do I have to use redis?

For each route, what is it for, what is the format of the request and format of the response? For example, creating a USER is a post to //users with a JSON body, but creating a uri is a PUT to /uris/ with a text/plain string body that is a ref to a page. SwaggerDocs maybe?
How does the compose process work, and what choices to we get to make, and how? For example, it took me a while to figure out what parts of bundling are done in a Clay implementation, which parts are Ampora, and how the two interact.
What is a "Clay implementation"?
What are all the settings in config.yml and what do they do?

Wrote some feedback in pink:
README.MD
Other notes:

Do we have an Amphora logo? We should add that to the README file so it's more welcoming.
I think it would make more sense to call the routes docs as "Api Routes"

From the Trello ticket

Data versioning
Environment variables
Renderers
Routes
Lifecycle (Bootstrap, Plugins, etc.)
Plugins
Publishing services
Database choice
Additional tooling (Amphora search, Kiln, etc.)
Best practices? (clustering, isomorphic services)
Roadmap?

Thanks for your responses, everyone! They've been added to our documentation push


ci: Add a test for upgrading from 7.0.

We build this image on bookworm, for diversity.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

lgtm; CI has already passed for the job in question.


Implement delete-commit

This PR implements DeleteCommit and exposes it in pachctl as delete-commit.  Currently it can only delete commits that are open (unfinished).

This LGTM, what's standing in the way of this working on finished commits as well?

It could work on finished commits; it's just that then we could end up with commits that have nonexistent provenance and we haven't really thought through the implications of that.


Uninformative error message

I'm not following from the README nor the error message how the git repo must be prepped before using this.
± |master ✓| → python setup.py git_version sdist
running git_version
Traceback (most recent call last):
  File "setup.py", line 19, in <module>
    'git_version': gitversion.GitVersion,
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/Users/martinmat/.virtualenvs/ver_test1/lib/python3.6/site-packages/setupext/gitversion.py", line 39, in run
    '{0}...HEAD'.format(starting_rev))
  File "/Users/martinmat/.virtualenvs/ver_test1/lib/python3.6/site-packages/setupext/gitversion.py", line 81, in _run_git
    stdout, stderr = _run_command('git', *args)
  File "/Users/martinmat/.virtualenvs/ver_test1/lib/python3.6/site-packages/setupext/gitversion.py", line 99, in _run_command
    ' '.join(exec_list))
subprocess.CalledProcessError: Command 'git rev-list --merges <IP_ADDRESS>.3...HEAD' returned non-zero exit status 128.

This occurs when there is no git tag corresponding to the package version. In your situation, there probably wasn't a tag for <IP_ADDRESS>.3.
@dave-shawley Wondering if it's sane to use the pre identifier when this happens.
https://www.python.org/dev/peps/pep-0440/#pre-release-spelling


Create code for a synoptic view of glider and science data during a deployment

It would be a nice thing to have to have something like this (https://marine.rutgers.edu/cool/data/gliders/deployments/deployment.php?deployment=bios_jack-20240812T1810) during a deployment to have a good long view of how the glider and science are sampling that looks better than the SFMC plots.
This may be done by scraping the data off of the SFMC then providing plots. This will probably be a team effort.

@flaimC is making great progress with this. His code is here

The code works locally on my machine now to email out useful plots generated from raw binary files. I am working on getting to code to run upon VM reboot and am waiting to hear back about GCP CLI issues.

The script now runs ~4hrs on the GCP VM and either sends new data plots to the pilot team or a "no new data" message to me.
All of the plots are saved on GCP and could be ported to a website of sorts for people to use. This will take some time due to the limitations of the VM using a hardened version of linux that does not easily support putting stuff on the internet.


Add network information to setupopts

Adding more/required information for network setup to the test config
file.  For now, I use 192.168 to avoid collision with the 10.8* subnets
that cni uses.
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

LGTM @mheon PTAL

/lgtm


Typo 'Futter' should be 'Flutter'

This error/warning was missing an 'l' so when i did a global find it came back with nothing in the manifest file.
The detected reason was:
  [/Users/david/dev/wwappSuper/withwineapp/wwapp/android/app/src/main/AndroidManifest.xml]() uses `android:name="io.flutter.app.FutterApplication"`

duplicate of #97950

Closing as dupe


[UNTHEMED]: sidebars are not themed or are themed improperly in dms and servers

Client Mod
Vencord, BetterDiscord
Please provide screenshots of the unthemed areas.

it's done this once before, and it fixed itself. even after reinstalling betterdiscord, switching back to vencord, and reinstalling the theme on both, it's stayed like this. looks awful. i wish there was another person who maintained an amoled-like theme because this is the second time this has happened within a few months
Before Submitting

[X] I checked for similar reports of this issue.
[X] I am on the latest stable Discord version.
[X] I am on the latest version of the theme.
[X] I have reinstalled the theme.

i wish there was another person who maintained an amoled-like theme because this is the second time this has happened within a few months

Discord recently just pushed out an update that broke themes again; based on the announcement posted in Vencord, this was literally in the last 5-10 minutes, but it was probably a bit before that. This will be fixed soon, thanks for your patience.

Discord recently just pushed out an update that broke themes again; based on the announcement posted in Vencord, this was literally in the last 5-10 minutes, but it was probably a bit before that. This will be fixed soon, thanks for your patience.

thank you for the quick reply, apologies for the passive-aggressiveness in my report. i'm not present in any of the modded discord communities so i'm not typically up to date with issues.

It appears that Discord has pushed a previous build they initially reverted (which was the one that caused issues the first time, then was reverted after a fix was pushed) so I was able to use everything from that reverted commit. I'm not going to merge this for the time being until I review it a bit more and confirm that Discord won't just randomly roll back the build again.

Confirm this as well. Experiencing the same behavior after restarting my Vesktop instance

Issue is back.

Discord pushed an update, which also seems to have broken a bunch of client mod functionality. Reopening this issue until theme issues are resolved.

Yeah was going to mention this, I hope this will be fixed soon! Like always, take your time

Closing this issue, it makes more sense to refer to a new one so when a pr or commit is made, it can properly reference it.
New issue is #84.


Command newrelic reporting junit does not work anymore

Description
This GitHub Action uses a deprecated environment variable NEW_RELIC_INSIGHTS_INSERT_KEY.
Steps to Reproduce
---
name: Release

on:
  - release

jobs:
  newrelic:
    runs-on: ubuntu-latest
    name: Basic Usage
    steps:
      - name: Checkout
        uses: actions/checkout@v2

      - name: Post JUnit test results to New Relic
        uses<EMAIL_ADDRESS>        with:
          accountId: ${{ secrets.NEW_RELIC_ACCOUNT_ID }}
          insertApiKey: ${{ secrets.NEW_RELIC_INSERT_API_KEY }}
          testOutputPath: test-output/integration.xml

Expected Behavior
The GitHub Action should run without any error messages.
Relevant Logs / Console output
level=fatal msg="a License key is required, set one in your default profile or use the NEW_RELIC_LICENSE_KEY environment variable"

Additional context
This GitHub Action has not been updated for almost 2 years, but the Dockerfile is still referencing to the latest version of the Newrelic CLI.
https://github.com/newrelic/junit-reporter-action/blob/main/Dockerfile#L1
The entrypoint.sh does not provide a real value for NEW_RELIC_API_KEY and NEW_RELIC_INSIGHTS_INSERT_KEY is not being used anymore. Instead NEW_RELIC_LICENSE_KEY should be used.
https://github.com/newrelic/junit-reporter-action/blob/main/entrypoint.sh#L3-L4

@kidk Can this version be re-opened? It doesn't look like version 0.2.0 is published to the marketplace unless I'm missing something?  Right now it still shows 0.1.1

@jsantinipax8 Fixed, the latest version is now 0.2.0

Thank You!


[Suggestion] Upload File from URL to AnonFiles (Remote Upload)

Suggestion
H!
I've my archived project from another File-Cloud and i want to import the assets to AnonFiles as Archive. It's really not efficient to download the assets first to my internal storage and upload it to AnonFiles. I'm hoping to see a feature that can give ability to fetch stream data from a direct-link and immediately upload it to AnonFiles without the files being saved first to disk.
Implementation
Can be a direct input or using a -U parameter
-anon up {url_1} {url_2}
-anon up -U {url_1} {url_2}

give some suggestion , if you can for this ?


AUI-3084

Hey @thiago-rocha,
Attached is an update for http://issues.liferay.com/browse/AUI-3084.
Please let me know if you have any questions. Thanks!

Just started reviewing :) :octocat: Sent from GH.

Just started reviewing :) :octocat: Sent from GH.

@jonmak08  The code lgtm, but it's missing a test for this use case that's being fixed. It will even help me understand what's being fixed, I'm not sure I see it at the moment. Can you add the test?

Hey @thiago-rocha,
@jonathanmccann will be adding the unit tests for this and we will send an updated pull request.
Please let me know if you have any questions. Thanks!

Pull request with test - https://github.com/jonmak08/alloy-ui/pull/350

New Pull Request: https://github.com/thiago-rocha/alloy-ui/pull/99


discriminator checkpoint

Hi, Could you share your discriminator checkpoint?
So that we can try fine-tune the HAG model on other language.
I tried user your HAG checkpoint model to Resynthesis chinese mandarin audio, the model can  do it well at most of the time when input MMS wav2vec feature of lower speech rate audio.
Based on the design of your model, the HAG model was not very related to language it trained on, if I'm not get wrong.
Before you release your model train code, I want to try train the TTV model on mandarin and english data from scratch,
and try fine-tune the HAG model on my data to get better pronunciation.
Most importantly, Thans for your sharing code and checkpoint!

Thanks for your interest!
As you said, our hierarchical speech synthesizer could synthesize any speech robustly because we utilize a continuous self-supervised speech representation, not phonetic-related information such as PPG.
I have attached checkpoints of generator and discriminator (v1.1) including optimizer.
link
You can load the model by
    _ = utils.load_checkpoint("ckpt_name", net_g, None)
    _ = utils.load_checkpoint("ckpt_name", net_d, None)

if you want to load the optimizer together, use this
    optim_g = torch.optim.AdamW(
        net_g.parameters(),
        hps.train.learning_rate,
        betas=hps.train.betas,
        eps=hps.train.eps)
    optim_d = torch.optim.AdamW(
        net_d.parameters(),
        hps.train.learning_rate,
        betas=hps.train.betas,
    _ = utils.load_checkpoint("ckpt_name", net_g, optim_g )
    _ = utils.load_checkpoint("ckpt_name", net_d, optim_d )

However, I have concerns about fine-tuning the model because we could not share the training code in this stage...
Although we describe the training details, training code must be elaborate which may hard to reproduce some components such as speech perturbation, w2v feature extraction, and Mel-preprocessing (https://github.com/sh-lee-prml/HierSpeechpp/blob/main/Mels_preprocess.py).
It is very important to remove speaker-related information by speech perturbation. We would appreciate your understanding. We will share this part after paper acceptance...!
In addition, we are still training the hierarchical speech synthesizer now.

The generator v1.1 is a checkpoint on 1,890k steps. The model is not converged yet! We will share the checkpoint after training more.

Sorry for the confusion.
We do not utilize the speech perturbation when training TTV and the weight of w2v reconstruction is 10!
 "c_mel": 10
this is the weight of w2v reconstruction. Sorry for the confusion.

https://github.com/sh-lee-prml/HierSpeechpp/blob/main/ttv_v1/config.json.
Here is our loss curve.

As you said, our w2v loss has a similar result because w2v features are not normalized. (Some value of w2v features have too big value above 500...!)
However, it is okay because we train our hierarchical speech synthesizer with perturbed w2v features so the synthesizer robustly generates the speech!!!
In addition, we try to modify the TTV architecture to further reduce this issue. (including hidden size and decoder hidden shapes)
Give permission for audio samples!
Thanks!

Sorry, I fixed the permission issue for audio samples.
I think model still is under-fitting, so there is mispronounce.
The TTV model, which I'm traning now, use perturbated w2v features as posterior. Because some voice style information, e.g. timber, may exist in w2v features.
So, I want to know for what reason you choose not perturbated w2v features...

We do not utilize the speech perturbation when training TTV

As you said, the w2v representation contains some voice style information.
There are some reason I do not use speech perturbation in this part,

We observed that the KL divergence loss between text and w2v is relatively high, resulting in optimization issue.
Because the hierarchical speech synthesizer is trained with perturbed speech, we thought that voice style is adapted with voice prompt in hierarchical speech synthesizer newly.
Although w2v representation contains some voice style, they have relatively little information as we observed in HierSpeech.

However, I agree that using speech perturbation in this part could improve the prosody style transfer performance by removing voice style if the model is robustly trained.
and I have a question about your TTV setting! Do you use a language embedding in text encoder? In my experience, adding language embedding in first block of text encoder improves the pronunciation!

Thanks for your response for my question again!
I don't use language embedding yet. I'm using the mixed phoneme set. Chinese phonemes include pinyin initial consonants and finals. English phonemes from IPA.

I think you don't need language embedding in your case.
If you have other questions, feel free to ask me!

hi, @sh-lee-prml, how about the ctc loss weight? Is it 45 the same with hierspeech?

@azraelkuan Yes It is same with hierspeech
 "c_pho": 45.0,

This is the ctc loss weight (https://github.com/sh-lee-prml/HierSpeechpp/blob/main/ttv_v1/config.json.)

@sh-lee-prml thanks! Sorry, forgot to check the config file！！！

@azraelkuan My pleasure! If you have other questions, feel free to ask me :)

@sh-lee-prml i have found that YAPPT  is not accurate for many samples in libritts, it will give all zero result...

In my case, about 10 samples were extracted with all zero results and we removed these files in training filelists.
Much more specifically, we use a default setting of YAPPT for fmin and fmax.
‘f0_min’ - minimum pitch searched (default: 60 Hz)
‘f0_max’ - maximum pitch searched (default: 400 Hz)

def get_yaapt_f0(audio, rate=16000, interp=False):
    frame_length = 20.0
    to_pad = int(frame_length / 1000 * rate) // 2

    f0s = []
    for y in audio.astype(np.float64):
        y_pad = np.pad(y.squeeze(), (to_pad, to_pad), "constant", constant_values=0)
        signal = basic.SignalObj(y_pad, rate)
        pitch = pYAAPT.yaapt(signal, **{'frame_length': frame_length, 'frame_space': 5.0, 'nccf_thresh1': 0.25,
                                        'tda_frame_length': 25.0, 'f0_max':800})
        if interp:
            f0s += [pitch.samp_interp[None, None, :]]
        else:
            f0s += [pitch.samp_values[None, None, :]]

    f0 = np.vstack(f0s)
    return f0

This is an example
            audio, sr = torchaudio.load(audio_path)

            p = (audio.shape[-1] // 320+ 1) * 320- audio.shape[-1]
            audio = torch.nn.functional.pad(audio, (0, p), mode='constant').data

            try:
                f0 = get_yaapt_f0(audio.numpy())
            except:
                f0 = np.zeros((1, 1, audio.shape[-1] // 80))

            f0 = f0.astype(np.float32)
            f0 = f0.squeeze(0)
            f0 = torch.FloatTensor(f0)
            torch.save(f0, f0_filename)

and before fed to model, we normalize it log-scale by
        f0 = torch.log(f0+1)

For expressive dataset, we increase the fmax to 800 for training dataset. (Additionally, we will utilize 1100 for singing dataset.)
Meanwhile, YAPPT is too slow to extract all samples when we increase the dataset so now we are testing other pitch extraction method such as praat.
Thanks!

@sh-lee-prml hi, i found that you missed a layer norm in wav2vec processor, will it have any influence for result?
https://github.com/huggingface/transformers/blob/371fb0b7dc1b533917e2f85b464a3ec9c74f28b9/src/transformers/models/wav2vec2/feature_extraction_wav2vec2.py#L227

@sh-lee-prml hi, i found that you missed a layer norm in wav2vec processor, will it have any influence for result? https://github.com/huggingface/transformers/blob/371fb0b7dc1b533917e2f85b464a3ec9c74f28b9/src/transformers/models/wav2vec2/feature_extraction_wav2vec2.py#L227

Thanks for your suggestion.
Actually, we directly use the hidden representation from the 7th layer of MMS (Wav2Vec 2.0) without layer-norm and we do not experience any problem yet.
However, this may improve the performance of some downstream tasks. I also observed that the hidden representation of some data has too large value so this could improve the robustness of our model.
Thanks!

hey,
thank you for sharing the training cure. I am training the model on Libritts 960 using exactly your preprocessing and training filse. However my training curve doesn t show the same losses you got. while you reached 2.74 w2v loss at 100k, my training is still at 6.


added singUp function

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sandeep Tharu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@TharuSandeep why close?

Wrong branch,
was supposed to give PR on our local master.
Sorry for the trouble.
On Thu, Aug 8, 2024 at 7:17 PM Yang Luo @.***> wrote:

@TharuSandeep https://github.com/TharuSandeep why close?
—
Reply to this email directly, view it on GitHub
https://github.com/casdoor/casdoor-ios-sdk/pull/4#issuecomment-2275844968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ4K6XM3N64XVNRXUIW2KI3ZQNXQDAVCNFSM6AAAAABMFUGS6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZVHA2DIOJWHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Feature request: Add INFO about cause of rejection in reject VCF

Liftover can fail for a variety of reasons...
Many of the liftover functions return -1 on any error, and the check is < 0
If you defined a range of negative constants, you could return specifically what went wrong, then look it up then add that to the INFO of rejected variants

If a variant is dropped it means that neither of the two anchors could be mapped using the chain files or if the two anchors mapped to different chains, or if the two anchors mapped to locations too far from each other. Compared to other liftover tools the reasons for dropping a variant are more limited and basically it is always related to the chain breaking or missing at the locus. Would this information really be useful?

When a liftover of their data fails, the bio people pick half dozen variants then we do a deep dive to try and work out what happened. This is to verify that it's not due to a bug in the code
I think it reassures them that things are working correctly, and they appreciate much more seeing an error "two anchors mapped to locations too far from each other" vs "liftover failed"
I can try and write a pull request but haven't written C in close to 20 years...

Maybe give me some minimal documentation for how you would like to see the INFO field engineered, given the three possible return codes, in a way that you deem usable by bio people, and I will write the code myself

For my particular use case, I'll be processing the VCF file then displaying it in a GUI for the BIO people
There are no enums in VCF types so I guess just make it a string and fill it in with constants, I'd expect the info to look a bit like:
##INFO=<ID=REJECTION_REASON,Number=A,Type=String,Description="Reason for rejection. Will be 1 of NO_MAP (neither of the two anchors could be mapped using the chain files), DIFFERENT_CHAINS (two anchors mapped to different chains) or DISTANCE_EXCEEDED (the two anchors mapped to locations too far from each other)">

But feel free to change to whatever you think best. Thanks a lot!

variants can also end up in the reject list for other reasons, eg the contig not being defined in the header - there should be a code for this (eg VCF record error) to distinguish it from ones to do with chains

Try to see if the development version here fixes both problems

Hi, there are no binaries in that release, I tried to follow the compiling instructions but it says to use "make" and there is no Makefile in there or in the master branch

Hi, thanks looks good.
Ran dev version with --write-reject and VCF was written with FILTER containing the reason


Feat DivModNSafeDivPlusOne hint (ec hint)

Summary
Resolves #527

Hey @TAdev0, when I run make integration I get these errors:

Do you know how I can solve those issues? I already have installed the cairo-compile command

Mmh don't know right now. Can you use Cairo compile and Cairo run without issue? Maybe Python version ?

Mmh don't know. Can you use Cairo compile and Cairo run without issue? Maybe Python version ? Did you follow the read me to install everything properly

I found what the issue was, I forgot to activate the python venv where I have installed the cairo 0 compiler dependencies

@TAdev0 Now I have this error:

Is related to the hint implementation in Go? or what does it means exactly 🤔

Can you successfully run all other integration tests ? If no , it's a setup issue

@TAdev0 Here is a better view to the results:

I think other integration test are running as expected

I changed the name of the new file to .small.cairo and now It returns this:

I tried locally using your hint impl and it worked.
Not exactly, because we also miss this hint :
    %{
        from starkware.cairo.common.cairo_secp.secp_utils import pack
        from starkware.python.math_utils import div_mod, safe_div

        a = pack(ids.a, PRIME)
        b = pack(ids.b, PRIME)
        value = res = div_mod(a, b, N)
    %}

but your hint works

@danielcdz  can you also work on this hint and add it to this PR?
https://github.com/NethermindEth/cairo-vm-go/issues/532
so that the integration test :
https://github.com/lambdaclass/cairo-vm/blob/db7fff18c9c1312024ebf4119fcdfeba23657b12/cairo_programs/div_mod_n.cairo#L29
will work entirely

@TAdev0 for sure!

What would be the name of this other hint? div_mod ?

@TAdev0 All the integration tests are passing now!

@danielcdz  can you resolve conflicts before final review?

You still need to add value variable in scope. Here it works because as we saw there is 2 hints in a row but the first one is only used to import N and actually adds value variable in scope, it's second one that is actually tested (the one you implemented). Yours should add value in scope as well (and overwrite the previous one in the test)
What I meant in dm is that you should create a copy of the variable you assign to res , and assign it to value

@danielcdz  you can simply use new(fp.elemen).Set to make a local copy instead of doing the computation 2 times


Forecasting API call

I'm currently running a residential solar system and using online forecasting predictions to set battery charge levels as to not over charge during off peak times.
My current provider has been pretty hit and miss (by a large percentage, today it's predicted less than 1kw at 1am this morning, but we ended up with 18.4kw)
You can review my public GitHub project for more details on what my application does. But I track it's accuracy over time
Detailed Description
Context
I wish to reduce my co2 footprint by reducing demand on my electrical provider by using more accurate predictions to set minimum charge on my battery to reach the point at which the solar panels can take over the runner or the house hold
Possible Implementation
A secure call to a exposed public API that gives the caller the total expected solar generation based on pv size, efficiency, orientation, geo location and pitch.

Sorry, I'm based in the west of Ireland (Europe)

Current forecasting accurately

Hi, really interesting project! Currently, we've been focused on regional and national forecasts for the UK's National Grid, but starting in July we are planning on also building out creating forecasts for individual PV systems. Would you be interested in maybe having a call about more of your requirements, and possibly being an alpha testor for our forecast when we get that running?

Sure sounds cool, give me a shout when your up and ready to go 😁
On Thu 16 Jun 2022, 11:43 Jacob Bieker, @.***> wrote:

Hi, really interesting project! Currently, we've been focused on regional
and national forecasts for the UK's National Grid, but starting in July we
are planning on also building out creating forecasts for individual PV
systems. Would you be interested in maybe having a call about more of your
requirements, and possibly being an alpha testor for our forecast when we
get that running?
—
Reply to this email directly, view it on GitHub
https://github.com/openclimatefix/graph_weather/issues/25#issuecomment-1157512172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6XXDU5CIFHIPJFYSFAHYDVPMALTANCNFSM5YVR6FKA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Hello from the CCCS! 🍁

We at the Assemblyline project perform our own file identification to ensure files are routed correctly to the corresponding file analysis modules. That is why the magika project is very interesting to us.
We have a set of files used for unit testing that we are confident* in their file type. We ran that set against the magika tool and found some discrepancies: see attached CSV.
All of the SHA256 hashes can be found on VirusTotal, and we would love to collaborate (join our Discord!) to improve magika to the point where we can integrate it into Assemblyline :)
AL_MAGIKA_COMP_revised.csv
Cheers,
🇨🇦

Much appreciated! We can add these to our golden dataset and track improvements to the model.
Can I assume these are MIT licensed like the rest of Assemblyline?
Btw - I love Assemblyline :)

For sure! If there's anything we can do to help improve the project, feel free to let us know!

Much appreciated! We can add these to our golden dataset and track improvements to the model. Can I assume these are MIT licensed like the rest of Assemblyline?
Btw - I love Assemblyline :)

Assemblyline itself is MIT-licensed but the hashes in that list should not be assumed to be MIT. They are not owned by Assemblyline and rather are just files found on VirusTotal :)


escape special chars when creating group filter.

The LDAP search filter used for group queries needs to be escaped so that group names with special characters will not break the search filter in queries. e.g. a query for a group with a Common Name with parens will return 'Bad search filter' and break authentication.

Thanks for catching that. Merged the PR.


Please update 1.19.4

As soon as I open the world for the network on version 1.19.4, the game closes.
message.txt

https://www.curseforge.com/minecraft/mc-mods/mcwifipnp/files?version=1.19.4


Generating constants that expose the wire-size of SBE variable length data fields

If I want to use SBE in combination with Aeron's BufferClaim API then I need to know the size of the message that I want to claim. If the message that I want to encode has no variable length data fields then this is easy, just claim the BLOCK_LENGTH. If I have variable length data then I need to claim BLOCK_LENGTH + the sum of the lengths of variable length + the sizes of the length fields for each variable length data.
The sizes of the length fields are stored your SBE's xml schema file, but there is no way from code to get to them without hard coding them in your codebase.  If the constants were simply generated as part of the encoders I think it would improve usability of SBE when combined with the BufferClaim API.
Again for me this is a nice-to-have rather than a high priority issue, however, if you agree with the concept but don't have time to implement I'm happy to contribute a patch.

The VarDataEncoding and GroupSizeEncoding classes are generated. So, it should be possible to know those aspects and write something that is not specific to the XML.

Thanks @tmontgomery, I wasn't aware of those. Looking at the VarDataEncodingEncoder/VarDataEncodingDecoder that I have I don't see any fields that correspond to the size of the length field. There's an ENCODED_LENGTH constant, but its -1 for me.

The decoder for var data has the length accessor generated but this is not in the encoder. We could add this.

Just to clarify I'm not talking about just having access to the encoded length of the field. I'm talking about the encoded size of the length field. For example in the following case its 2:
        <composite name="varDataEncoding">
            <type name="length" primitiveType="uint16"/>
            <type name="varData" primitiveType="uint8" length="0"/>
        </composite>

Yes that is what I assumedyou were asking. Look at the CarEncoder and CarDecoder in the samples. The Decoder has it but not the Encoder.

Perfect, thanks.


weekly image cve patching from Sec Pipelines

weekly image cve patching from Sec Pipelines

this should be merged IF no existing build will be used to release, otherwise merge https://github.com/Azure/AgentBaker/pull/2786


Select color programmatically

Is there a way I can select a color programmatically? just changing the $.val()  does not change the color in the preview.

You need to call updatePreview to update the color of square preview to the right. Since updatePreview is not exposed, I've created a version of pick-a-color with this possibility:
https://github.com/frangin2003/pick-a-color/commit/43963e438e9c3e8002f19f85bfa3a8c6df3776e9
See fiddle here to demo:
http://jsfiddle.net/frangin2003/yy8uxmjh/

This solution sets the color of the preview but does not actually set the values of the H/S/L.

I ended up needing to set them programmatically, so I made the changes here:https://github.com/lauren/pick-a-color/pull/81

See issue #94 for a quick solution


⚠ Bofrost.ch has degraded performance

In 4ad261d, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 1619 ms

Resolved: Bofrost.ch performance has improved in dff14a6 after 8 minutes.


Sample-proxies: api-key: Hardcoded api proxy

In file api-platform-samples/sample-proxies/apikey/invoke.sh there are some hard-coded things like
key=`curl -u $username:$password<EMAIL_ADDRESS>2>/dev/null \
     | grep consumerKey | awk -F '\"' '{ print $4 }'`

seems broken to me as well. I haven't been able to get this sample running.

Yeah, I noticed this too.  I forked the code so if I have time I'll create a pull request.  Probably just a copy / paste bug...  Thanks.


Split workflows and publish to PyPI

Fix #15
I must admit that I'm a bit confused about the current workflows in the project. Were they copied from the guide itself? Nevertheless, I have tried to keep current content, more or less, but added a workflow to publish to PyPI when a tag is pushed.
I've swapped out some of the Actions used.
Perhaps it would be beneficial to have a call and quickly sort out some of my confusion, as well as make sure I understand your goals.

@Midnighter one other question. could we use hatchling for our build back end here rather than setuptools? again i'm asking this as a question because of my lack of understanding around building a sphinx theme.

@Midnighter thank you for this pr! nothing pops out to me as something that won't work so let's approve and merge! In terms of link checking i'm ok wtih either dropping htmlproofer in favor of nox / sphinx linkchecker. Or doing that in another pr.
Finally i have a question for you and @Revathyvenugopal162 how si the theme actually being "built" as a package?
when i look here: https://github.com/pyOpenSci/pyos-sphinx-theme/blob/main/noxfile.py#L23 i see there is a build command but that's just building the html. if this is a package doesn't it need to have a wheel?
please note i still don't fully understand how a sphinx theme or book is a package so this is my attemping to better understand what the build step is here. many thanks!

We use the pyproject.toml file to configure and build the package as a wheel. You can build the package using the following commands:

To install the package, use pip install .
For development mode, use pip install -e .

It's true that the package does not explicitly define these details. If you would like, we can add commands for nox or hatch to the CI/CD pipeline to automate the build and publish process to PyPI. Also i would like to mention that, since we are  using pyproject.toml to define our dependencies, as shown here, we don't need a separate requirements.txt file. This file can manage all dependencies, streamlining the build process.


Feature/issue sg 3445 sync fn mutations

Changes:

Update bundle.sh to work with latest go-bindata
Removes save / restore buttons since they are known to exacerbate https://github.com/couchbase/sync_gateway/issues/3445 (see https://github.com/couchbase/sync_gateway/issues/3445#issuecomment-384757028)

Need to be done in unison with a sync gateway change to reference the new assets path.
Related to https://github.com/couchbase/sync_gateway/issues/3445

I reverted the change that disabled the save/restore buttons.  Ready for another review pass


Fix UI string ID in system_notice.jsx

Summary
Fix UI string ID in system_notice.jsx
I had the option of just fixing the string in en.json, but this would have been inconsistent with other strings. Hence fixed it in system_notice.jsx and updated the relevant test snapshots.
Wondering if there's anything we can do in future to capture errors affecting localization via unit tests?
Ticket Link
Reported by French community: https://pre-release.mattermost.com/core/pl/wmjsjq4dqffpbbq6ecisnbxpxa
Checklist

[X] Includes text changes and localization file (.../i18n/en.json) updates

There's definitely a way to test the localization strings for each components, maybe something for @cpanato or @saturninoabril to look into


No Icon (input field only) stack overflow when used with a initial value

I believe there is a bug when you use the datepicker with no icons and there is a value set for the input.
Check out the example.
https://jsfiddle.net/3cqL9t3x/61/

you don't have your picker setup correct, please see the docs


CDK-888: Improve error handling for CSV and JSON.

This adds the field name to help users narrow down what data or schema
doesn't match.

One nit, otherwise +1


[Installation] Update failed as of missing XAML framework

Yesterday I tried to update WT on my work computer using Add-AppXPackage.
The install failed because of missing package: Microsoft.UI.Xaml.2.8
How can I get this framework without MS Store?
OS: Windows 10 22H1
WT version: 1.17.*
Language: German

Solved. Found it in the PreinstallKit download. Maybe worth to mention in the readme.


Zend\Session\Container falls if other script start session

if other script has session_start() then Container drop identity.
test at https://gist.github.com/vnagara/5365296 .
Run container.php then start-session.php then container.php again which should output 'val'

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Modal close button needs accessible label

Currently, the button to close a modal is an "x" that is announced as "times" by screen readers. This issue could be fixed by adding aria-label="Close" to the <button> element.

:tada: This issue has been resolved in version @typeform/embed-v4.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Hello @yvonnetangsu
thank you for your suggestion. We have just released a new version with aria-label.
Enjoy 👍

Thank you so much for the quick fix @mathio !


[Bug]: Game launching as background process with SE installed

Operating System
Windows 10
BG3 Mod Manager Version
<IP_ADDRESS>.0
BG3 Game Version
Full Release Patch 5 Hotfix 15
Bug Summary
When I try to install the Script Extender, the game won't launch through the mod manager. It only launches as a background process. I'm certain I've installed the SE in the right place, and I'm not sure if this is an SE or mod manager issue. I thought I'd bring this to your attention. Thank you!
Links
No response

Same issue here. It's EXTREMELY difficult to get the game to run each time, as it sometimes works properly after I try 20-30 times after ending the task each time I try starting it up. Still no solution for this, as far as I've researched. I tried reinstalling and disabling achievements, as others have had this solved that way, but still no luck.


More flexible configuration of benchmark candidates

Currently, configuration of benchmark candidates is pretty limited: We have only a few combinations available, everything is geared towards single node benchmarks and also all that is hardcoded in the Rally code base. However, we rather want to make the configuration more flexible and also allow want to provision clusters where individual nodes have different roles (dedicated master nodes, data nodes etc.).
Given that #184 is around the corner to provision single nodes and we will implement multi-node provisioning in #71, it is about time that we think about how we can support flexible cluster topologies.
We will take a phased approach:

[x] First, we externalize car configuration (#196)
[x] next, we will enable benchmarking with plugins (see #60)
[x] finally, we add support for multi-node clusters (#71)

Candidate Management
Similarly to track management via git, we will also use a git repository to manage benchmark candidates. The idea is that users create their own repos but we will provide an official one so users can get started.
The structure in the repo rally-teams (team being an analogy for a cluster (consisting of multiple cars)) is as follows:
├── cars
│   └── defaults
│       └── config
│           ├── elasticsearch.yml
│           ├── jvm.options
│           └── log4j2.properties
├── plugins
│   └── x-pack
│       ├── config
│       │   └── elasticsearch.yml
│       └── install.cmd
└── teams
    ├── single
    │   └── node.yml
    └── two
        └── node.yml

The directories' purpose is:

cars: contains the configuration of a single node (one per sub-directory). The node in the example is called defaults and contains just the default (out-of-the-box) configuration of Elasticsearch.
plugins: contains the configuration of plugins (one per sub-directory). The install.cmd contains one or more shell commands to install and configure the plugin. The contents of config/elasticsearch.yml (if present) will be appended to the already existing elasticsearch.yml of the node.
teams: contains the configuration of a cluster (one per sub-directory). The node.yml file can override property placeholders either in track.json or the car configuration (this is needed to allow e.g. cluster specific settings such as replica count.).

The candidate repositories will be stored in ~/.rally/benchmarks/teams/$REPO_NAME with the default repo being our official one. ~/.rally/rally.ini will contain a reference to the available team repos (all that is nothing new and very similar to how tracks are managed today).
Command line usage
Here are few examples of the envisioned command line usage (examples assume we are on a machine with IP <IP_ADDRESS>).
# This command shows all implicit default values, i.e. this is the same as just invoking: esrally
esrally --team-repository=default --car=defaults --team=single --topology="node:<IP_ADDRESS>" --es-plugins=""

# remote deploy a node with car "defaults":
esrally --topology="node:<IP_ADDRESS>"

# but this could also get as complex as the following example.
# "master" is a node specification in the team folder (i.e. master.yml instead of the default "node.yml" and we choose a "4gheap" car for it (similarly for "data" and "24gheap")
esrally --team=complex --topology="master:4gheap:<IP_ADDRESS>,master:4gheap:<IP_ADDRESS>,data:24gheap:<IP_ADDRESS>,data:24gheap:<IP_ADDRESS>,data:24gheap:<IP_ADDRESS>"

# run with X-Pack
# TODO: can we make this work with source builds too?
esrally --es-plugins="x-pack"

For a later phase we could also think about a "mixin" concept. For example, if we want to run Elasticsearch with the following configuration:

4gb heap size (name in example below: 4gheap)
assertions enabled (name in example below: ea)
more verbose logging  (name in example below: verbose)

We could create "mixin" definitions for all components and then define a car with --car="4gheap,ea,verbose". This could reduce the number of necessary definitions significantly.

All sub-tasks are done, thus closing.


[FLINK-17210][sql-parser][hive] Implement database DDLs for Hive dialect

What is the purpose of the change
To implement database related DDLs for Hive dialect.
Brief change log

Add separate parser for Hive dialect.
Implement database DDL syntax and Calcite SqlNode.
Add FlinkSqlParserImplFactory to create parser according to SqlConformance.
Change PlannerContext to use FlinkSqlParserImplFactory when getting sql parser config
Change HiveCatalog to differentiate generic and hive databases.
Add/update test cases.

Verifying this change
Existing and added test cases.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

cc @JingsongLi @KurtYoung , please have a look, thanks.

CC: @danny0405

Had some discussion with @danny0405 offline and we decided to move the Hive parser to its own module.
I haven't removed the "useless" imports in Parser.tdd since they'll be needed shortly in follow-up PRs. So I'd rather avoid the back and forth.

@danny0405 @JingsongLi Thanks for your reviews. The azure build is successful and the travis build for my own repo also passed: https://travis-ci.org/github/lirui-apache/flink/builds/679603860
Please let me know if you have further comments.

@danny0405 I have updated the PR so that the unparse method reflects the original DDL.

Merging...


Your account is locked. Visit https://twitter.com/account/access to unlock it.

🚨 Account Lockout After Posting a Tweet: Is it My Code or an API Issue?
📝 Problem Description
I am encountering an account lockout immediately after trying to post a tweet using the Twikit library. Below is the code that I'm executing. I would appreciate any insights on whether this is due to my code, or if there is an issue with the Twikit API or Twitter’s restrictions.
❓ My Question

Is my usage of the API incorrect, leading to the lockout?
Is there a limitation on the API side that I might be triggering?

Any help on identifying the cause would be appreciated!
⚙ Code Example
import asyncio
from twikit.client.client import Client

async def login_with_cookies(client, username, email, password, cookies_file='cookies.json'):
    try:
        client.load_cookies(cookies_file)
    except FileNotFoundError:
        await client.login(auth_info_1=username, auth_info_2=email, password=password)
        client.save_cookies(cookies_file)

async def create_tweet(client, text):
    await client.create_tweet(text=text)

async def main():
    username = "your_username"
    email =<EMAIL_ADDRESS>    password = "your_password"
    cookies_file = f'cookies_{username}.json'

    client = Client(
        user_agent='Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36'
    )

    await login_with_cookies(client, username, email, password, cookies_file)
    await create_tweet(client, "Hey, it's my first tweet!")
    await client.logout()

if __name__ == "__main__":
    asyncio.run(main())

Perhaps this solution will help you figure out the problem click


Fix: Bypass Virus Checks in Google Drive Links (CNN-DM dataset)

This commit fixes the issue described in #3784. By adding an extra parameter to the end of Google Drive links, we are able to bypass the virus check and download the datasets.
So, if the original link looked like https://drive.google.com/uc?export=download&id=0BwmD_VLjROrfTHk4NFg2SndKcjQ
The new link now looks like https://drive.google.com/uc?export=download&id=0BwmD_VLjROrfTHk4NFg2SndKcjQ&confirm=t
Fixes #3784

Thank you, @albertvillanova!

Got it. Thanks for explaining this, @albertvillanova!

On the other hand, the tests are not passing because the dummy data should also be fixed. Once done, this PR will be able to be merged into master.

Will do this 👍

Hi ! I think we need to fix the issue for every dataset. This can be done simply by fixing how we handle Google Drive links, see my comment https://github.com/huggingface/datasets/pull/3775#issuecomment-1050970157

Hi @lhoestq! I think @albertvillanova has already fixed this in #3787

Cool ! I missed this one :) thanks

No problem!

Hi, @AngadSethi, I think that once:

#3787

was merged, issue:

#3784

was also fixed.
Therefore, I think this PR is no longer necessary. I'm closing it. Let me know if you agree.

Yes, absolutely @albertvillanova! I agree :)


Add TryBinaryOperator convenience class

I propose to add a BinaryOperator helper which simplifies working with BinaryOperators of Try values, e.g. when reducing a stream of Try values.
Note that the implementation of of() could also be written as
(a, b) -> a.flatMap(x -> b.map(y -> binaryOperator.apply(x, y)))

but I opted to keep the code more readable.
Given a class that has a method Instant getModifiedOn() throws IOException, a common usage example could look like this:
files.stream()
    .map(c -> Try.apply(c::getModifiedOn))
    .reduce(new Try.Success<>(Instant.MIN),
        TryBinaryOperator.maxBy(Comparator.naturalOrder()));

Hi Bernhard,
thank you for your contribution. I'll be more than happy to merge this PR but, if you don't mind, you should change it a bit:

I prefere the implementation of of to be:

(a, b) -> a.flatMap(x -> b.map(y -> binaryOperator.apply(x, y)))

This is the idiomatic way to use Try and is enough readable to me, more than having ifs and the like.

You should write a usage example under the com.lambdista.example package and a simple test in the TryTest class.

Thank you so much!

Ok, will do. But don't you think a separate Test class would be more appropriate? Whichever you prefer...

Yes, you're right. A separate Test class is more appropriate.
Thank you!

I've made the requested changes and added a test and example class. Please review.

Thank you. Merged!
I'll release it as version 0.2.0 in the Sonatype OSSRH as soon as possible.

Published as version 0.2.0.

Excellent, thanks!
(Sorry about the change to .gitignore)


crashed when lock to landscape mode

when using react-orientation, app crashes when I lock to landscape mode.
Terminating app due to uncaught exception 'UIApplicationInvalidInterfaceOrientation', reason: 'Supported orientations has no common orientation with the application, and [RCTModalHostViewController shouldAutorotate] is returning YES'
similar issue:
xgfe/react-native-datepicker#28

I've tested in my device, please approve this. :)

Sorry for delayed reply and thanks for issue and PR. Looks like this was an RN bug and in was fixed in 0.49.1. I will update this component in near future to 0.49.1 and will reopen this issue if anything goes wrong with orientation support.


Wrong image source in readme

Opss.. @LazuLitee Thanks for the attention. I just fixed that problem.
Let us know if you need anything else.  For now, I will close this issue.


Namespace object has no attribute lora_dir

Tried installing into SD Webui using the install from repo URL option, then when restarting it, the below was reported:
Exception in thread Thread-48 (load_previews):
Traceback (most recent call last):
  File "C:\Users\alpac\AppData\Local\Programs\Python\Python310\lib\threading.py", line 1009, in _bootstrap_inner
    self.run()
  File "C:\Users\alpac\AppData\Local\Programs\Python\Python310\lib\threading.py", line 946, in run
    self._target(*self._args, **self._kwargs)
  File "F:\StableDiffusion\stable-diffusion-webui\extensions\sd_civitai_extension\scripts\previews.py", line 15, in load_previews
    resources = civitai.load_resource_list()
  File "F:\StableDiffusion\stable-diffusion-webui\extensions\sd_civitai_extension\civitai\lib.py", line 189, in load_resource_list
    resources += get_resources_in_folder('LORA', shared.cmd_opts.lora_dir, ['pt', 'safetensors', 'ckpt'])
AttributeError: 'Namespace' object has no attribute 'lora_dir'

I am running in an anacaonda env

What version of Automatic1111 are you running?
At the bottom of your screen you should see something like this:


graphics/vncserver: Implement vsync

Summary
Support vsync on virtual device vnc server to avoid tearing.
Impact
VNC server only
Testing
Tested on sim with custom configuration.

hi @no1wudi please fix this warning promoted to error:
vnc/server/vnc_updater.c: In function 'vnc_updater':
Error: vnc/server/vnc_updater.c:359:7: error: unused variable 'val' [-Werror=unused-variable]
  359 |   int val;
      |       ^~~
cc1: all warnings being treated as errors

hi @no1wudi please fix this warning promoted to error:
vnc/server/vnc_updater.c: In function 'vnc_updater':
Error: vnc/server/vnc_updater.c:359:7: error: unused variable 'val' [-Werror=unused-variable]
  359 |   int val;
      |       ^~~
cc1: all warnings being treated as errors

Done


Fixed subscriber input variable handler

what

Fixed bug in subscribers loop

why

Fixed bug

references

This PR is based on this PR https://github.com/cloudposse/terraform-aws-config/pull/44

/terratest

/terratest

/terratest

/terratest


Added composer.json for installation from packagist

Hello,
I've been using you work for quite a while now and i really enjoy it. The only problem is that you don't have a composer package for easy installation in PHP project.
Please accept this pull request. It simply adds a composer.json file which would allow you to submit this package (also you need to create a packagist account).
Thank you in advance.
https://getcomposer.org/
https://packagist.org/

:+1:  ;)

Hello,
I just saw that you merged the branch with the composer.json file.
Thanks a lot for that. You should now be able to submit your package on Packagist (https://packagist.org/packages/submit).
This will allow anybody using Composer in PHP projects to easily install and use your theme.
Packagist also provide a GitHub service hook to ensures that your package will always be updated instantly when you push to GitHub (https://packagist.org/about#how-to-update-packages).
Also, you could add this installation method in your README.md and on startbootstrap.com (just like you did with bower).
Note: at the time i added the composer.json file, i thought you could use "iron-summit-media/startbootstrap-sb-admin-2" as a package name. But maybe you should update this to "blackrock-digital/startbootstrap-sb-admin-2".
Thank you in advance.

@foxou33 @challgren @Zeromax @davidtmiller
Looks like, this issue is abandoned.
I added this package from my own packagist account: iron-summit-media/startbootstrap-sb-admin-2
I cannot setup automatic update without repo access. If you want to republish package from another account let me know.

@AnatolyRugalev @davidtmiller can you please remove me from the maintainer list? sb-admin is not mine it is @davidtmiller and I am not allowed to add me as a maintainer, cause I am not... ;)


Intelligent re-processing on rule change

It should skip receipts that have already been processed when the matching rules haven't changed.  That would speed up incremental processing.
It should

store SHA256(image) as key with pointer to image and receipt
check to see if rules version has changed
if changed, reprocess.  If not, continue to next image.

OBE, starts from scratch every time with a temp db

OBE, starts from scratch every time with a temp db


APK Expansion fail to install

Godot version:
3.1

OS/device including version:
Android 5.0/Samsung Galaxy S5

Issue description:
APK Expansion enabled, devices report cannot locate resource.

Steps to reproduce:
Godot 3.1 Project on Galaxy S5.
Minimal reproduction project:

Are you using the official export templates?

Are you using the official export templates?

Custom android export with admob. the apk works fine if no apk expansion(and yes i'm testing live from the playstore). the problem only happens on Galaxy S5

did you enable READ_EXTERNAL_STORAGE permission?

did you enable READ_EXTERNAL_STORAGE permission?

I didnt add that permission, I'll try it and see if this resolves my issue.

@canvasbushi any update?

@canvasbushi any update?

i reduced the size of the engine to get an apk under 100mb, so i haven't tried it yet.

Closing since the issue seems to be resolved. @canvasbushi Feel free to reopen if it's not.


Extension registered schema providers random fail to get invoked

VSCode Version: 1.45.0-insiders (repros with non-insiders too)
OS Version: Windows

Steps to Reproduce:

Install the C/C++ extension (may repro with other extensions). Our extension uses

    class SchemaProvider {
        provideTextDocumentContent(uri) { // When the bug occurs this doesn't get called by VS Code
            return __awaiter(this, void 0, void 0, function* () {
		// ...
                return // file contents
            });
        }
    }
// This register always gets called by our extension
    vscode.workspace.registerTextDocumentContentProvider('cpptools-schema', new SchemaProvider());

Create a blank test.cpp file (to activate our C/C++ extension).
Use the C/C++: Edit Configuration (JSON) command to create a new c_cpp_properties.json.
Keep using reload window until the bug randomly repros (repro probability is high for me, it doesn't repro when debugging our extension). Other members of our team and some of our extension users have also hit this, but some people do not repro it.

Bug: See the screenshot. It fails to load the schema. It gets fixed after clicking the retry button in the status bar that appears.

Originally reported at https://github.com/microsoft/vscode-cpptools/issues/4841 .

Does this issue occur when all extensions are disabled?: Requires our C/C++ extension but we don't believe it's our bug.

We are using this to implement localizeable schemas, as suggested here: https://github.com/microsoft/vscode/issues/78737
In Sean's case, I suspect the issue may be that the file is already open when launching VS Code, and there is an attempt to access the uri before we call registerTextDocumentContentProvider().  If we registered this immediately in activate(), would that be sufficient to ensure it's always registered before used by VS Code?

I modified our code to register as soon as possible, but the issue was still easily reproduced.
Our package.json has
    "jsonValidation": [
      {
        "fileMatch": "c_cpp_properties.json",
        "url": "cpptools-schema:///c_cpp_properties.schema.json"
      }
    ],

so it seems like VS Code should be able to recognize that our extension needs to be activated before it tries to do json validation on that file.

What activation events is your extension using and can you guarantee that registerTextDocumentContentProvider is called during activate?

The relevant activation event is "workspaceContains:/.vscode/c_cpp_properties.json", which corresponds to the file that we're registering the schema for.
We can guarantee that the registerTextDocumentContentProvider is called during the activate -- our current shipping code doesn't call it right away (it does so asynchronously after some delay during activation), but I moved the call to the start of the activate hoping that would fix or at least help, but it didn't help.

Ok, understood. So, there is two problem here

VS Code doesn't wait for workspaceContains (and also not *) activation to finish. They just happen and are not part of a flow like "A command has been triggered, lets see if extensions to contribute something"
For text document content provider we actually don't send an event...

I will push a change for seconds (likely re-using the onFileSystem-event) and then your extension needs to adopt it

Actually - we already send out said activation event which leaves nothing to do here for us

Okay, thanks -- adding the onFleSystem:cpptools-schema activationEvent appears to have fixed the issue.


Some addresses not shown in the cardano explorer

For example:
https://explorer.cardano.org/en/address?address=addr1v8v3auqmw0eszza3ww29ea2pwftuqrqqyu26zvzjq9dt2ncydzvs5
as reference, cardanoscan does show it:
https://cardanoscan.io/address/addr1v8v3auqmw0eszza3ww29ea2pwftuqrqqyu26zvzjq9dt2ncydzvs5
this addresses has hundreds of thousands of transactions, seems like an exchange address, i.e.
https://explorer.cardano.org/en/transaction?id=3a312a368a55367dd02af0a09ce28ad18db22618b7080a07fb752eabf243af1b
https://cardanoscan.io/transaction/3a312a368a55367dd02af0a09ce28ad18db22618b7080a07fb752eabf243af1b
So when you search a transaction id it exists as input of the tx, but following the link

shows:

have seen this issue as well

Same here:
https://explorer.cardano.org/en/address.html?address=addr1q9d8rynu787z66qrjlulw3qvrwq0dm45k798xp53u4hjtxmcs9ftdl362vpna80k6zz4zmqztg3vu0p8vt2375xuz6zq0kl2fe
This address only has 54 txs though, so don't think it's related to the number of transactions. Same address on cardanoscan is here:
https://cardanoscan.io/address/addr1q9d8rynu787z66qrjlulw3qvrwq0dm45k798xp53u4hjtxmcs9ftdl362vpna80k6zz4zmqztg3vu0p8vt2375xuz6zq0kl2fe

This happens because Cardano's explorer doesn't seem to handle native tokens correctly. All the 'buggy' addresses, tx, and blocks contain native tokens and that makes the application throw errors when it tries to load their properties:

At the moment I encounter the same issue. My wallet is visible in cardano.io but not in the official explorer (explorer.cardano.org). I'm using a Ledger Nano S to access my wallet but it cannot find the wallet either probably as it makes use of explorer.cardano.org apparently.
This wallet does contains coins. I know that empty wallets do not show up sometimes. Can it be that the last transaction has been like two years ago it does not show?

I have the Yoroi wallet app to move the wallet content to a new wallet. That did the trick. I can see my ADA's in Ledger Live now.


fix pages-plugin-sentry types

Should fix Cannot use namespace 'Toucan' as a type..
node_modules/@cloudflare/pages-plugin-sentry/index.d.ts:6:36 - error TS2709: Cannot use namespace 'Toucan' as a type.

6 export type PluginData = { sentry: Toucan };
                                     ~~~~~~

Found 1 error in node_modules/@cloudflare/pages-plugin-sentry/index.d.ts:6

Thanks! I added a changeset to describe the change you've made.


Add compat details on MTK.jl bugfixes

MTK.jl v9.50 fixes the issues in ModelingToolkit.build_explicit_observed_function
closes #116

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.93%. Comparing base (3833301) to head (4fda91d).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage   98.93%   98.93%
=======================================
  Files          24       24
  Lines        3575     3575
=======================================
  Hits         3537     3537
  Misses         38       38

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.