Interactive GitHub OAuth token request fails

Using #83:
I was attempting to download a package from a custom GitHub VCS in a fresh Docker container without any composer credentials set up. The GitHub API requests were rejected by GitHub's API, with PIE emitting an informational message that credentials were required. However it crashed while actually requesting the credentials:
root@26098011df0d:/pwd# php bin/pie download -v redacted/redacted:dev-pie
You are running PHP 8.3.13
Target PHP installation: 8.3.13 nts, on Linux/OSX/etc x86_64 (from /usr/local/bin/php)
Reading composer.json of https://github.com/redacted/redacted.git (v0.2.0)GitHub API limit (60 calls/hr) is exhausted, could not fetch https://api.github.com/repos/redacted/redacted/contents/composer.json?ref=redacted. Create a GitHub OAuth token to go over the API rate limit. You can also wait until 2024-11-14 13:45:56 for the rate limit to reset.
When working with _public_ GitHub repositories only, head here to retrieve a token:
https://github.com/settings/tokens/new?scopes=&description=Composer+on+26098011df0d+2024-11-14+1246
This token will have read-only permission for public information only.
When you need to access _private_ GitHub repositories as well, go to:
https://github.com/settings/tokens/new?scopes=repo&description=Composer+on+26098011df0d+2024-11-14+1246
Note that such tokens have broad read/write permissions on your behalf, even if not needed by Composer.
Tokens will be stored in plain text in "/root/.composer/auth.json" for future use by Composer.
For additional information, check https://getcomposer.org/doc/articles/authentication-for-private-packages.md#github-oauth
Reading composer.json of https://github.com/redacted/redacted.git (v0.1.0)GitHub API limit (60 calls/hr) is exhausted, could not fetch https://api.github.com/repos/redacted/redacted/contents/composer.json?ref=redacted. Create a GitHub OAuth token to go over the API rate limit. You can also wait until 2024-11-14 13:45:56 for the rate limit to reset.
When working with _public_ GitHub repositories only, head here to retrieve a token:
https://github.com/settings/tokens/new?scopes=&description=Composer+on+26098011df0d+2024-11-14+1246
This token will have read-only permission for public information only.
When you need to access _private_ GitHub repositories as well, go to:
https://github.com/settings/tokens/new?scopes=repo&description=Composer+on+26098011df0d+2024-11-14+1246
Note that such tokens have broad read/write permissions on your behalf, even if not needed by Composer.
Tokens will be stored in plain text in "/root/.composer/auth.json" for future use by Composer.
For additional information, check https://getcomposer.org/doc/articles/authentication-for-private-packages.md#github-oauth
GitHub API limit (60 calls/hr) is exhausted, could not fetch https://api.github.com/repos/redacted/redacted/git/refs/heads?per_page=100. Create a GitHub OAuth token to go over the API rate limit. You can also wait until 2024-11-14 13:45:56 for the rate limit to reset.
When working with _public_ GitHub repositories only, head here to retrieve a token:
https://github.com/settings/tokens/new?scopes=&description=Composer+on+26098011df0d+2024-11-14+1246
This token will have read-only permission for public information only.
When you need to access _private_ GitHub repositories as well, go to:
https://github.com/settings/tokens/new?scopes=repo&description=Composer+on+26098011df0d+2024-11-14+1246
Note that such tokens have broad read/write permissions on your behalf, even if not needed by Composer.
Tokens will be stored in plain text in "/root/.composer/auth.json" for future use by Composer.
For additional information, check https://getcomposer.org/doc/articles/authentication-for-private-packages.md#github-oauth
In HelperSet.php line 67:

  [Symfony\Component\Console\Exception\InvalidArgumentException]
  The helper "question" is not defined.

Exception trace:
  at /pwd/vendor/symfony/console/Helper/HelperSet.php:67
 Symfony\Component\Console\Helper\HelperSet->get() at /pwd/vendor/composer/composer/src/Composer/IO/ConsoleIO.php:292
 Composer\IO\ConsoleIO->askAndHideAnswer() at /pwd/vendor/composer/composer/src/Composer/Util/GitHub.php:112
 Composer\Util\GitHub->authorizeOAuthInteractively() at /pwd/vendor/composer/composer/src/Composer/Util/AuthHelper.php:130
 Composer\Util\AuthHelper->promptAuthIfNeeded() at /pwd/vendor/composer/composer/src/Composer/Util/Http/CurlDownloader.php:548
 Composer\Util\Http\CurlDownloader->isAuthenticatedRetryNeeded() at /pwd/vendor/composer/composer/src/Composer/Util/Http/CurlDownloader.php:414
 Composer\Util\Http\CurlDownloader->tick() at /pwd/vendor/composer/composer/src/Composer/Util/HttpDownloader.php:392
 Composer\Util\HttpDownloader->countActiveJobs() at /pwd/vendor/composer/composer/src/Composer/Util/HttpDownloader.php:363
 Composer\Util\HttpDownloader->wait() at /pwd/vendor/composer/composer/src/Composer/Util/HttpDownloader.php:114
 Composer\Util\HttpDownloader->get() at /pwd/vendor/composer/composer/src/Composer/Repository/Vcs/VcsDriver.php:170
 Composer\Repository\Vcs\VcsDriver->getContents() at /pwd/vendor/composer/composer/src/Composer/Repository/Vcs/GitHubDriver.php:448
 Composer\Repository\Vcs\GitHubDriver->getContents() at /pwd/vendor/composer/composer/src/Composer/Repository/Vcs/GitHubDriver.php:377
 Composer\Repository\Vcs\GitHubDriver->getBranches() at /pwd/vendor/composer/composer/src/Composer/Repository/VcsRepository.php:321
 Composer\Repository\VcsRepository->initialize() at /pwd/vendor/composer/composer/src/Composer/Repository/ArrayRepository.php:308
 Composer\Repository\ArrayRepository->getPackages() at /pwd/vendor/composer/composer/src/Composer/Repository/ArrayRepository.php:62
 Composer\Repository\ArrayRepository->loadPackages() at /pwd/vendor/composer/composer/src/Composer/Repository/RepositorySet.php:199
 Composer\Repository\RepositorySet->findPackages() at /pwd/vendor/composer/composer/src/Composer/Package/Version/VersionSelector.php:87
 Composer\Package\Version\VersionSelector->findBestCandidate() at /pwd/src/DependencyResolver/ResolveDependencyWithComposer.php:29
 Php\Pie\DependencyResolver\ResolveDependencyWithComposer->__invoke() at /pwd/src/Command/DownloadCommand.php:61
 Php\Pie\Command\DownloadCommand->execute() at /pwd/vendor/symfony/console/Command/Command.php:326
 Symfony\Component\Console\Command\Command->run() at /pwd/vendor/symfony/console/Application.php:1078
 Symfony\Component\Console\Application->doRunCommand() at /pwd/vendor/symfony/console/Application.php:324
 Symfony\Component\Console\Application->doRun() at /pwd/vendor/symfony/console/Application.php:175
 Symfony\Component\Console\Application->run() at /pwd/bin/pie:64
download [-j|--make-parallel-jobs MAKE-PARALLEL-JOBS] [--with-php-config WITH-PHP-CONFIG] [--with-php-path WITH-PHP-PATH] [--] <requested-package-and-version>

Also note that the formatting in the error messages is a bit off. A newline is missing before “GitHub API limit (60 calls/hr) is exhausted” and the message is also printed twice, because apparently it did not fail after the first failed API call.

I believe this should just be a case of defining the helper set; it was left empty originally, but it would make sense to populate this properly.

{
...,
"repositories": [
        {
          "type": "vcs",
          "url": "https://github.com/shahmal1yev/atprotocli"
        }
      ]
...
}

I tried several different methods, but I couldn't reproduce the issue for some reason. Do you have any idea what I might be doing wrong?

i have opened a PR for it https://github.com/php/pie/pull/163.


StdoutLock is unsound

Run the following program:
use async_std::io::prelude::WriteExt;
use async_std::{io as async_io, task};
use std::io::{self, Write};
use std::thread;

#[async_std::main]
async fn main() -> io::Result<()> {
    let mut async_locked = async_io::stdout().lock().await;
    let fut = task::spawn_blocking(|| {
        let stdout = io::stdout();
        let mut locked = stdout.lock();
        writeln!(locked, "sync stdout in thread {:?}", thread::current())?;
        locked.flush()?;
        Ok(())
    });
    async_std::writeln!(
        async_locked,
        "async stdout in thread {:?}",
        thread::current(),
    )
    .await?;
    async_locked.flush().await?;
    fut.await
}

Cargo.toml dependencies
[dependencies]
async-std = { version = "1.6.0", features = [ "attributes", "unstable" ] }

This program violates the invariant of StdoutLock (there must never be two instances of StdoutLock owned by different threads), leading to UB and panics like the following.
sync stdout in thread Thread { id: ThreadId(10), name: None }
thread 'main' panicked at 'already borrowed: BorrowMutError', /rustc/8d69840ab92ea7f4d323420088dd8c9775f180cd/src/libcore/cell.rs:878:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Considering lock is an unstable API, I would recommend removing the API, and suggest usage of spawn_blocking instead. I don't think it's possible for async StdoutLock to ever be sound with threaded executors

Hi, thanks for reporting. You're right the current implementation isn't correct. We had a PR to remove it a while ago, but had to revert it because it included additional changes that caused regressions.
The right approach here now indeed seems to remove the unstable stdio types.


MSC1974: Crypto Puzzle Challenge

Crypto Puzzle Challenge
Proof-of-Work rate-limiting, via cryptographic puzzle challenge, to fight spam.
Proposal
New Errors:

M_PUZZLE_NEEDED

extra field seed (String)
extra field bits (Integer)
extra field algorithm (String bcrypt)

M_PUZZLE_INVALID

New Headers:

X-Matrix-Puzzle

seed:bits:algorithm:n - where n is an integer
When this header is hashed with algorithm is must produce at least bits many leading zeros

In response to a request, the receiving party may error with a M_PUZZLE_NEEDED challenge, which requires a crypto-puzzle be solved and request re-submitted with proof-of-work attached as X-Matrix-Puzzle.
The puzzle is solved by incrementing n and re-hashing the header using algorithm, then checking for leading zeros.
Inspired by Hashcash
Tradeoffs
The original Hashcash is not a request-response challenge but instead only opt-in by the sender. This allows for pre-computation and does not allow dynamic bits to be requested (necessary as computers get faster) or a change in algorithm (again, necessary because of dedicated hardware (ASICs)). Though it could be made viable by re-specifying these in the Matrix spec.
bcrypt is used here but there are other options e.g. Argon2d, scrypt. bcrypt was chosen because it's popular (and thus most software ecosystems should have a package for it) and because it is hard to parallelize on GPU/ASIC hardware.
Potential issues
Giving clients free-reign over when to use the error and how much work they require could lead to some unintended consequences.
E.g. If the major server implementations use too high values as their default then it might force out low-end hardware altogether (and make running your own server potentially costly (in CPU time), decreasing decentralization).
Conclusion
This proposal provides a powerful spam-fighting mechanism which relies on compute resource consumption.
Signed-off-by: Zolmeister<EMAIL_ADDRESS>

Is this intended for client-server interactions, server-server interactions, or both?

Both

A major consideration you don't mention here is power consumption, both from the perspective of an end user ("I have to use whatsapp instead of riot because riot kills my battery") and from an ethical perspective: ("whoops, matrix now uses more energy than a small country").
Sticking to the first point, is it really possible to pick a difficulty such that a user on a low-end smartphone or a tiny microprocessor on a battery-powered IoT device sees no impact, while an attacker with desktop GPUs (or worse) is severely impeded?
Overall, I can't really see the need for this: can you explain how your proposal solves problems that traditional "come back in ___ seconds" rate limiting (based on IP or user account) doesn't?

@JJJollyjim Excellent points, battery life added to issues
That's also a strong argument for specifying a correct mode of operation for when to serve the error.
Perhaps the biggest difference from "come back in ___ seconds" is the ability to rate-limit initial connections. For example the first connection from an unknown IP address / user. This makes this method significantly stronger against distributed attacks using multiple IP addresses.
I suspect that IoT devices would use a federator that did not challenge them with a PoW. Otherwise this may actually be a positive given the prevalence of IoT botnets.
I posit that even a small proof-of-work would deter spammers in favor of a cheaper target.

A major consideration you don't mention here is power consumption, both from the perspective of an end user ("I have to use whatsapp instead of riot because riot kills my battery") and from an ethical perspective: ("whoops, matrix now uses more energy than a small country").

I think that could be solved by servers only asking for a proof of work when connecting for the first time or when sending the first few messages. Normal users don't join a server all that often, so it shouldn't cause much of an overhead.
However, lots of spambots these days will join a lot of servers and immediately send a lot of spam messages. With a proof of work approach, that would make sending spam more costly.


Can dowhy identify a target estimand under PAG or CPDAG?

Currently, dowhy only support DAGs. It will be really good to add support for CPDAGs. I've added it as a project item. If you would be interested in contributing, let me know.

Also linking #429 discussion page where we are discussing this functionality


[FIX] https://github.com/nats-io/node-nats/issues/89

Stacktraces are not meaningful because Error objects are created with the library. Library now simply adds the error messages and constructs the error where thrown.

Coverage remained the same at 95.21% when pulling 86b9e8c81ae1c0971748861f5a805be56d2927d9 on error-on-throw into adaaaa5819f3cba67d48c2e2658af523fcef7ab9 on master.

Coverage remained the same at 95.21% when pulling 86b9e8c81ae1c0971748861f5a805be56d2927d9 on error-on-throw into adaaaa5819f3cba67d48c2e2658af523fcef7ab9 on master.

@ColinSullivan1 @kozlovic just looking for another pair of eyeballs on these simple changes.

LGTM

LGTM


Expose panning strength parameters

Exposes tightness/panning strength as a project setting and as properties on the 2d and 3d audio player nodes.
Bugsquad edit: Follow-up to https://github.com/godotengine/godot/pull/58823 (can be merged independently).

Testing project for 3D: test_audio_panning_strength.zip
I feel setting Panning Strength to 0.5 by default would be an optimal default, so maybe the default project settings' values should be halved.

Did we ever get to a resolution on how to properly cache a project setting for fast access every frame? Invalidating the cache gets very tricky.

I just updated the settings to require restart, which is a bummer but not the end of the world. If we want to pursue the caching route someone else might have to tackle it because ProjectSettings is confusing me. :)
@Calinou you seemed to have some opinions on this PR. Did I resolve everything?

I tested both 2D and 3D functionality rebased on latest master and it works as expected.
Updated testing project: test_audio_panning_strength_updated.zip

What's the status on this PR? Is it blocked on me or does Reduz need to look at it? It would be an enormous win for users IMO because the current stereo panning isn't great. Just want to make sure it doesn't get lost.

What's the status on this PR? Is it blocked on me or does Reduz need to look at it? It would be an enormous win for users IMO because the current stereo panning isn't great. Just want to make sure it doesn't get lost.

I had this comment pending a resolution: https://github.com/godotengine/godot/pull/58841/files/0be393fcf225eb8197a38b4f6afd1ceed1a787b6#diff-425ab30a63dbbf819f27d912b74dd7cfc929681b8cf68037216b96a607ebb801
And I added a couple more.
Otherwise a quick check from @reduz would have been nice but since it's not happening, I think we can merge.

I force pushed an update to resolve my review comments.

I force pushed an update to resolve my review comments.

Thank you! I'm on vacation right now and in general I've been very focused on another project so I didn't get to this.

Thanks!


Luminex ManageGuideSetPanel update to add null check for grid store before trying to load data into it

Rationale
Some intermittent TeamCity test failiures with the LuminexGuideSetTesting test cases show a client side error: TypeError: this.allRunsGrid.getStore() is null. This PR adds a check for the grid store to the if statement that handles loading the data into that store.
Changes

Luminex ManageGuideSetPanel update to add null check for grid store before trying to load data into it

The LuminexGuideSetDisablingTest has passed 6 times on this FB on TC:
https://teamcity.labkey.org/project.html?projectId=LabKey_Trunk&branch_LabKey_Trunk=luminexGuideSetTestFailures


Provide tool to inspect misclassified test images

retrain.py can report misclassified test images. It would be nice to have a convenient way to inspect them to get insights into how the model is working.

I might use such a tool, but I won't develop one as part of this repo.


verified dolly-v2-3b

Verified dolly-v2-3b with greedy_causal_lm.cpp and beam_search_causal_lm.cpp
Added it to supported models list

FYI @Vishwa44, right after i merged i figured out that styling should be corrected. Done it here https://github.com/openvinotoolkit/openvino.genai/pull/296


Issue: Homepage Slider Not Functioning Properly

Description:
The slider on the homepage is completely non-functional due to multiple issues in the JavaScript code. Incorrect element selectors, broken event listeners, invalid data types, and logical errors prevent the slider from running as expected.
Steps to Reproduce:
Open the homepage.
Try clicking the "Next" or "Previous" buttons of the slider.
Observe that the slider does not change or function as intended.
Expected Behavior:
The slider should smoothly transition between items when the "Next" or "Previous" buttons are clicked. It should loop through the items, moving back to the first slide after reaching the last, and vice versa.

i have fixed the issue.

I have fixed the issue!


context deadline exceeded

We are using chromedp for our service, which is deployed in a container to periodically scrape the content of a website. We added "--no-sandbox --disable-dev-shm-usage" to /opt/google/chrome/google-chrome, but we found that it would report a "context deadline exceeded" error after running for a while.

Google Chrome 81.0.4044.92
ChromeDriver 81.0.4044.138 (8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999})

@styst please note the version of Google Chrome you're using is extremely old, and will not be usable with chromedp. Please upgrade Chrome and try again.


docker image in scicatproject/backend:stable does not seem to work for SciCat-live

Issue Name
Trouble with docker image  scicatproject/backend:stable
Summary
docker-compose fails during backend boot
Steps to Reproduce
have a look at https://github.com/SciCatProject/scicatlive/pull/10
Current Behaviour
when using the backend image scicatproject/backend:stable  (30/05/2022, DIGEST 33eb7d45ba02) I get the following error message when launching docker-compose -u
catamel_1        | node:internal/modules/cjs/loader:936
catamel_1        |   throw err;
catamel_1        |   ^
catamel_1        |
catamel_1        | Error: Cannot find module './config.local'
catamel_1        | Require stack:
catamel_1        | - /usr/src/app/server/server.js
catamel_1        |     at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
catamel_1        |     at Function.Module._load (node:internal/modules/cjs/loader:778:27)
catamel_1        |     at Module.require (node:internal/modules/cjs/loader:1005:19)
catamel_1        |     at require (node:internal/modules/cjs/helpers:102:18)
catamel_1        |     at Object.<anonymous> (/usr/src/app/server/server.js:7:19)
catamel_1        |     at Module._compile (node:internal/modules/cjs/loader:1105:14)
catamel_1        |     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
catamel_1        |     at Module.load (node:internal/modules/cjs/loader:981:32)
catamel_1        |     at Function.Module._load (node:internal/modules/cjs/loader:822:12)
catamel_1        |     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
catamel_1        |     at node:internal/main/run_main_module:17:47 {
catamel_1        |   code: 'MODULE_NOT_FOUND',
catamel_1        |   requireStack: [ '/usr/src/app/server/server.js' ]
catamel_1        | }

using the old image  dacat/catamel:latest things still seem to work fine.

The mountpaths need to be adjusted, the current version of the scicat image has all the code in /home/node/app.
So change the paths in the docker-compose.yml file accordingly.


Unclear parameter 'one'

What is this parameter supposed to be doing?
https://github.com/thenomemac/dataScienceLeaderboardPlatform/blob/master/leaderBoardApp.py#L74
Maybe rename it to something like get_first_row_only?

About half of the code in the leaderBoardApp.py file is artifacts from a flask tutorial I used as a starting point. I'm planning on refactoring the code in the next day or two and  re commiting with only the relevant functions/ constants.

Still need to fix that, I'll probably just remove that argument as it isn't used in app. Either way I finished v2 of the app. Lots of new features: added: bootstrap JS, jquery, mathjax, pygments, markdown tables support. As well as functionality for users to download the dataset, display images in the markdown docs that are hosted in a common location, users can now select which model they'd like to use for scoring, and the app has a "deadline" where the contest ends.


Update how-to-use-pipelines-prompt-flow.md

update with correct notebook URLs.
For issue #112204

Trying a close/reopen to complete the build.

@balapv
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

The related issue has been closed. Could you please review the update and apply to publish?

#sign-off


AAObject(AAPlotLinesElement) - cannot set visibility

I want to optionally make AAPlotLinesElement visible/invisible. I added a
visible
property to the AAPlotLinesElement class, but it does not work for some reason. AAPlotLinesElement is alway visible.
In AAPlotLinesElement.h:
AAPropStatementAndFuncStatement(assign, AAPlotLinesElement, BOOL      , visible);
In AAPlotLinesElement.m:
AAPropSetFuncImplementation(AAPlotLinesElement, bool      , visible);
Request: can you help with that and add a working 'visible' property?
Thanks! :o)

Nevermind - I just saw that plotLines does not support visibility... :o(


No ABI version for Electron 12 release

Hello. We faced one problem after upgrade ours Electron app from version 11 to 12:
$ electron-rebuild --parallel --force --types prod,dev,optional --module-dir app
× Rebuild Failed
An unhandled error occurred inside electron-rebuild
Could not detect abi for version 12.0.0 and runtime electron.  Updating "node-abi" might help solve this issue if it is a new release of electron
Error: Could not detect abi for version 12.0.0 and runtime electron.  Updating "node-abi" might help solve this issue if it is a new release of electron
at Object.getAbi (D:\Users\max\Documents\work\post\PartPriem\offline_lk\node_modules\node-abi\index.js:30:9)
at Rebuilder.get ABI [as ABI] (D:\Users\max\Documents\work\post\PartPriem\offline_lk\node_modules\electron-rebuild\lib\src\rebuild.js:111:39)
at ModuleRebuilder.prebuildNativeModuleExists (D:\Users\max\Documents\work\post\PartPriem\offline_lk\node_modules\electron-rebuild\lib\src\module-rebuilder.js:129:146)
at Rebuilder.rebuildModuleAt (D:\Users\max\Documents\work\post\PartPriem\offline_lk\node_modules\electron-rebuild\lib\src\rebuild.js:170:35)
at async Promise.all (index 126)
at async Rebuilder.rebuild (D:\Users\max\Documents\work\post\PartPriem\offline_lk\node_modules\electron-rebuild\lib\src\rebuild.js:148:13)
at async D:\Users\max\Documents\work\post\PartPriem\offline_lk\node_modules\electron-rebuild\lib\src\cli.js:146:9
error Command failed with exit code<PHONE_NUMBER>.
This error can be fixed when manually change abi_registry.json and replace target from "12.0.0-nightly.20201013" to "12.0.0"
We use Node-abi version 2.19.3. Upgrading to versions 2.20.0 and 2.21.0 has no effect

Any news on this issue?

I am dealing with this in my own project right now. However, I think the problem is with electron-rebuild using an older version of node-abi. The problem goes away when I use yarn's selective dependency resolution feature to force electron-rebuild to use the latest node-abi:
  "resolutions": {
    "node-abi": "^2.26.0"
  }

I am dealing with this in my own project right now. However, I think the problem is with electron-rebuild using an older version of node-abi. The problem goes away when I use yarn's selective dependency resolution feature to force electron-rebuild to use the latest node-abi:
package.json
{
  "name": "my-electron-app",
   ...
  "dependencies": {
    ...
    "node-abi": "^2.26.0",
  },
  "devDependencies": {
    ...
    "electron": "12.0.5"
  },
  "resolutions": {
    "node-abi": "^2.26.0"
  }
}

Wery helpful, thanks!

I am dealing with this in my own project right now. However, I think the problem is with electron-rebuild using an older version of node-abi. The problem goes away when I use yarn's selective dependency resolution feature to force electron-rebuild to use the latest node-abi:
package.json
{
  "name": "my-electron-app",
   ...
  "dependencies": {
    ...
    "node-abi": "^2.26.0",
  },
  "devDependencies": {
    ...
    "electron": "12.0.5"
  },
  "resolutions": {
    "node-abi": "^2.26.0"
  }
}

Thanks a bunch!
Note for npm users. Instead of resolutions, use overrides.

You don't need to use resolutions to fix this, you just need to update the node-abi dependency using npm/yarn the standard ways.
E.g. npm has npm update node-abi --depth=100 and for yarn you can use npx uuaw node-abi


Added sorting to tables

Allowed multi sorting on values within columns in the table.

Looks good. We'll have to make sure nothing is broken because the order of the table changes.


Bump upper bound for GHC 8.6

I haven't actually tried 8.6, but I'll take your word for it. I'll try to get a new version up on hackage later today. sorry for the lag...


Detecting Pokémon but not on map

In log it detected Pokémon but in the actually map none are shown.  Example : parsing found 5 Pokémons, 5 gym, etc. also stats show Pokémons but map is none.

same..

How long do you let the map run before you check it?  If you wait to long with the search function enabled then you might overshoot the pokemon thus resulting in newly spawned pokemon around you to not be seen. Play with the slider that says "Scanned Locations" to see if it might be user error.

Same log as me.  Still no solution ?


unnamed field fetch_object error

Dear Developers,
PHP Driver version or file name
sqlsrv  5.3.0   stable
SQL Server version
Microsoft SQL Server 2017 (RTM-CU15-GDR) (KB4505225) - 14.0.3192.2 (X64)
Client operating system
debian 9.9 stretch
PHP version
7.0.33-0+deb9u3
Microsoft ODBC Driver version
msodbcsql17 <IP_ADDRESS>-1
Table schema
field1 int (not relevant)
Problem description
fields without name or alias doesn's fetch object
Expected behavior and actual behavior
actually when i fetch "select field1, count(1) from tbl group by field1" fetch_object doesn't fetch object, i think it would be better to fetch the object with "field1" property, and if possible "Expr1" property to the "count(1)" column like SSMS does or "_ColN" property to the Nth column
Repro code or steps to reproduce
$link = sqlsrv_connect(...);
$result = sqlsrv_query($link, "select field1, count(1) from tbl group by field1");
$object = sqlsrv_fetch_object($result);
print_r($object, true);
Why i ask this

i don't know the result is really empty or just fetch_object doesn't fetch object.
i use Stored Procedures (SP-s), what wasn't i wrote, the SP-s sometimes updating, and sometimes happening the new SP result fields unnamed. in php when i see the property is missing, i can fix my php code in a few minutes, ie. i change "count" property to "_Col2", or i can start searching for the bug in the SP.

Thanks!
Sorry for my bad english!

hi @szollosi, sqlsrv_fetch_object returns false if something goes wrong. If there is no data you should get null instead.
For example, your script can be modified as follows:
$link = sqlsrv_connect(...);
$result = sqlsrv_query($link, "select field1, count(1) from tbl group by field1");
$object = sqlsrv_fetch_object($result);
if (is_null($object)) {
    echo "OBJECT is NULL" . PHP_EOL;
} elseif ($object === false) {
    echo "sqlsrv_fetch_object FAILED" . PHP_EOL;
} else {
    print_r($object, true);
}

The documentation clearly says that If a field with no name is returned, sqlsrv_fetch_object will discard the field value and issue a warning. To check the error message(s), you can do the following:
$errors = sqlsrv_errors(SQLSRV_ERR_ALL);
var_dump($errors);

In your case, the warning is An empty field name was skipped by sqlsrv_fetch_object.
By default, warnings are returned as errors unless you have set sqlsrv_configure('WarningsReturnAsErrors', 0);
Hope this helps.

hi,
this helps, i can do a workaround now :)
Thanks!


Rewrite async callback identification

ATM middleware will be executed async if the final argument of the middleware function is called next or callback, but I’d like to change this behaviour for a few reasons: some time in the future I’d like to

make grappling-hook middleware registration to be fully compatible with mongoose hook callback registration
make grappling-hook isomorphic, and the current implementation will raise some serious trouble, since it depends on argument names. When the middleware functions are minified next will be renamed to c or something

So, what I propose is the following: instead of checking the name, I’d check the number of parameters the callback function accepts, if this is 1 more than the number of parameters passed to callHook :arrow_right: bingo, it’s async
Extra benefit: it would make grappling-hook backwards-compatible, i.e. we could push grappling-hook into keystone 0.3.x

@creynders given the things this will unblock, I'm happy to go with the change :)


ServerMapping

I cannot make a pull request

You need to make the pull request yourself we don't help with creating we only validate. You can use tools such as youtube for help with making a pull request.


Add new local Gateway sharing same deployment as ingress Gateway.

Related to https://github.com/knative-sandbox/net-istio/issues/212.
The idea is to create a new Gateway named knative-local-gateway (more consistent with knative-ingress-gateway than cluster-local-gateway). This Gateway binds to the same Envoy Pods as the Istio ingress Gateway, but on a different port (8081 because 8080 is already used by Istio). There is a corresponding Kubernetes Service mapping named knative-local-gateway living in knative-serving (cleaner than cluster-local-gateway living in istio-system) mapping port 80 to port 8081.
knative-local-gateway is added to the default Istio Gateways, this way the VirtualServices of local Knative Services bind to it (i.e. ready to serve traffic), but because the KIngress status uses the first local Gateway in its status, the Services of type ExternalName still point to cluster-local-gateway (and it is not accessible to external requests since 8081 is not exposed through the LoadBalancer Service).
Effectively, this is a no-op, but in the next release we can remove reference to cluster-local-gateway in default config-istio and there will be no traffic interruption and the requests will flow to knative-local-gateway.
After that, everything related to cluster-local-gateway can be safely removed, including the Deployment, which is the end goal here.
/hold
/assign @tcnghia
/assign @ZhiminXiang
Let me know what you think.

/assign @nak3

But we may still want to double check that users cannot access cluster-local ingress from outside of the cluster just in case.

Yes, I tested that. Since the cluster local Gateway is on a port that is not exposed internally, it can't be accessed.
Either you hit <PUBLIC_IP>:8081 and you can't connect, or you hit <PUBLIC_IP>:80 and it goes to the public Gateway and you get a 404 for a cluster local service.

/lgtm

/approve

@JRBANCEL is this ready for /unhold?

Also I assigned you https://github.com/knative/serving/issues/6269 since I thought it would be related to this work.

/hold cancel


A functional Cozy

Thank you @nledez
@QuentinPerez can you give a look ?

LGTM maybe we will use update-motd instead of welcome message into the bashrc file

Yep, I merge this functional cozy, then we can propose improvements :)

The image is now building: https://travis-ci.org/scaleway/qa/builds/94886653


Build error: undefined: push.AddFromGatherer

Hello. Thanks for the library. But I've got an error:
vendor/github.com/tikv/client-go/metrics/push.go:44:10: undefined: push.AddFromGatherer

push.AddFromGatherer was removed from prometheus client in release 1.0.0.

Hi @borodyadka , the client is now using go mod to manage dependency and the Prometheus client is pinned to v0.9.2. Could you try to migrate your vendored prometheus to 0.9.2 too?
Of course, it is totally reasonable to update to the new stable version. I will create an issue first, and if you are interested, you can help us to upgrade :smile:

Hi @disksing , thanks for the answer. I'm not sure I can help you with it, because I don't have enough time for fixing this issue. I'm switch to prometheus 0.9.2 for now.


loadinternal: cannot find runtime/cgo

What version of Go are you using (go version)?

$ go version
go version go1.14.3 windows/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
D:\checktool>go env
set GO111MODULE=on
set GOARCH=386
set GOBIN=
set GOCACHE=C:\Users\john\AppData\Local\go-build
set GOENV=C:\Users\john\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\john\go
set GOPRIVATE=
set GOPROXY=https://goproxy.cn,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set GO386=sse2
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=0
set GOMOD=D:\checktool\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m32 -fmessage-length=0 -fdebug-prefix-map=C:\Users\john\AppData\
Local\Temp\go-build468324490=/tmp/go-build -gno-record-gcc-switches

What did you do?

What did you expect to see?
I want to build go to 32bit dll, but is error.
# e.coding.net/jiftle/inwarehousetool/checktool
loadinternal: cannot find runtime/cgo

set GOARCH=amd64 is ok.
What did you see instead?

Try setting CGO_ENABLED=1. It seems you are cross-compiling from amd64 to 386 (GOHOSTARCH=amd64, GOARCH=386). Cgo is by default disabled with cross-compilation. You need to enable it explicitly.

Try setting CGO_ENABLED=1. It seems you are cross-compiling from amd64 to 386 (GOHOSTARCH=amd64, GOARCH=386). Cgo is by default disabled with cross-compilation. You need to enable it explicitly.

My Question is ok , thank you.  --by china developer

set CGO_ENABLED=1 didn't solve this issue for me ? does this error means cgo not installed ?
C:\Users\user\Desktop\dll>go build -buildmode=c-archive
# _/C_/Users/user/Desktop/dll
loadinternal: cannot find runtime/cgo

@Co0nan This issue is closed.  I suggest that you ask your question on a forum.  See https://golang.org/wiki/Questions.


Release tools

npm version minor should:

bump minor version in package.json
copy the version over to appinfo.json
create new version commit
tag said commit

Also deploy target is now installing properly named .ipk from a local directory.

This is cool!

I'll need to rework webpack service build to copy over "main" package.json (or create a dummy file) during build, since nested package.json files make eslint angry...


When generating a video from a prompt, Error: 401 Missing Authentication header or invalid API key

After updating the Env file and providing a valid OpenAI Key, and setting up OpenRouter to use Nous:Hermes3 with the Base_Url in the env file, I still get this error, it seems the openai module can't access my API key in the .env.
I thought  /api/GenerateScript/Route.ts was missing the import "OPENAI_API_KEY", but when adding it to the file, it still displays this error.
Here is the snippet :
⨯ Error: 401 Missing Authentication header or invalid API key
at APIError.generate (webpack-internal:///(rsc)/./node_modules/openai/error.mjs:63:20)
at OpenAI.makeStatusError (webpack-internal:///(rsc)/./node_modules/openai/core.mjs:305:65)
at OpenAI.makeRequest (webpack-internal:///(rsc)/./node_modules/openai/core.mjs:348:30)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async POST (webpack-internal:///(rsc)/./src/app/api/generateScript/route.ts:17:22)
at async eval (webpack-internal:///(rsc)/./node_modules/next/dist/esm/server/future/route-modules/app-route/module.js:228:37)
at async AppRouteRouteModule.execute (webpack-internal:///(rsc)/./node_modules/next/dist/esm/server/future/route-modules/app-route/module.js:157:26)
at async AppRouteRouteModule.handle (webpack-internal:///(rsc)/./node_modules/next/dist/esm/server/future/route-modules/app-route/module.js:290:30)
at async EdgeRouteModuleWrapper.handler (webpack-internal:///(rsc)/./node_modules/next/dist/esm/server/web/edge-route-module-wrapper.js:92:21)
at async adapter (webpack-internal:///(rsc)/./node_modules/next/dist/esm/server/web/adapter.js:179:16)
✓ Compiled /_error in 275ms (783 modules)
POST /api/generateScript 500 in 810ms

you dont need to provide "openai" key, u just 3 api keys
NEXT_PUBLIC_OPENROUTER_API_TOKEN - for script
NEXT_PUBLIC_PEXELS_API_KEY - for videos
NEXT_PUBLIC_RAPIDAPI_KEY - for audio
i used NEXT_PUBLIC_ before the variable names so you need to check that.


Fix cmake build for arch. Fixes #9

This fixes the following issue:
https://github.com/nicehash/nheqminer/issues/9

so it should work now for arch? I keep getting the error CL_ext.h no such file or directory

@s-matthew-english there were many changes. Arch was not tested what project are you building?

I'm trying to build Linux (Ubuntu 14.04 / 16.04) Build CPU_XENONCAT, getting stuck at make -j $(nproc)

CPU_XENONCAT works please double check you are running the correct cmake file. CL_ext.h errors shouldn't be coming up in this build.

ok cool, it worked now.
but what directory do I run this command from?
nheqminer -l equihash.eu.nicehash.com:3357 -u YOUR_BTC_ADDRESS_HERE.worker1 -t 6
should I put my zcash address there or my bitcoin address?


URL cannot be opened on echo

hi there, setting up the skill worked, it's opening silk with the right url - but then it's stuck telling me the website cannot be reached.
any thoughts on what im doing wrong?
I do not have the encrypt/duck setup done yet -> my HA cannot be reached from the outside as of now. Could that have an influence on this?

i noticed that using "homeassistant/" works while "homeassistant.local/" does not - maybe a clue?

I had a similar problem when using homeassistant.local as URL. This is not a problem with the skill, but rather a problem with the way the Echo Show resolves this URL. I have no issues when I use the local IP address of my Home Assistant instance (e.g. 192.168.0.XXX:8123)

Same problem here. The skills works ok, but Silk won't resolve the URL. When I change the URL to the ip address of my HA, I have to enter user & password manually. Any idea how to solve that?

As I already said: There is nothing I can do if the Silk browser doesn't resolve a URL, because that problem isn't related to the skill, but rather to the device itself or your local network. However, from your comment it seems like the skill is working fine. Of course you will need to enter the username and password to sign in, because this skill literally just opens Home Assistant in the Silk browser. Like in any other browser, you will need to sign in to access Home Assistant. If you don't want to do this every time you open the skill (which is probably what you want), you need to tick the "Keep me signed in" checkbox (like in any other browser).

I had a similar problem when using homeassistant.local as URL. This is not a problem with the skill, but rather a problem with the way the Echo Show resolves this URL. I have no issues when I use the local IP address of my Home Assistant instance (e.g. 192.168.0.XXX:8123). I thought that is likely an issue with my network, but since this seems to be a general problem, I will probably mention this in the instructions.

Just had the same issue and I fixed changing the code to my Home Assistant IP.


Syncing latest changes from main for ocs-client-operator

PR containing the latest commits from main branch

/lgtm
/hold
@rewantsoni unhold when ocs-op vrc pr is synced to 4.18

/unhold

/lgtm


Visible cell numbers

I use the plugin to display the jupyter notebooks with Mode A Option 2. When the notebook is displayed on my website the cell numbers are shown on the left like this:

How do i get rid of the cell numbers?

You can use css, e.g.:
.input_prompt {
  visibility: hidden
}

Yes, this is possible with just a little bit of CSS on your theme.
You can also customize with the IPYNB_EXPORT_TEMPLATE settings: https://github.com/danielfrg/pelican-ipynb#settings


feat(label): support align for min/max labels

Brief Information
This pull request is in the type of:

[ ] bug fixing
[x] new feature
[ ] others

What does this PR do?

In some cases, especially when the canvas size is limited, we wish to confine the axis labels to be within an area rather than may be outside of the canvas when the label text is long. In this PR, I proposed a way to set align/verticalAlign for the first and last labels so that we can make sure they are within the boundary of the grid area.
Fixed issues
#19215
Details
Before: What was the problem?
The first and last label may be too long to be displayed when the text is long and the canvas size is limited.

After: How does it behave after the fixing?
When setting alignMinLabel: 'left', alignMaxLabel: 'right' we can make sure the text is within the grid's width and the inner labels may hide when overlap.

Document Info
One of the following should be checked.

[ ] This PR doesn't relate to document changes
[x] The document should be updated later
[ ] The document changes have been made in apache/echarts-doc#xxx

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

@Ovilia hllo mam myself harsh ,mam i am new to apache so i want to join its slack but i don't have email auhtorised by apache with workspace @apache.org so how can i join the slack

@Ovilia hllo mam myself harsh ,mam i am new to apache so i want to join its slack but i don't have email auhtorised by apache with workspace @apache.org so how can i join the slack

Hi, we don't use Slack. Please subscribe our mailing list.

@Ovilia hllo mam myself harsh ,mam i am new to apache so i want to join its slack but i don't have email auhtorised by apache with workspace @apache.org so how can i join the slack

Hi, we don't use Slack. Please subscribe our mailing list.

okk sure mam

Will it be more flexible if we can provide something like minLabelStyle / maxLabelStyle?

Will it be more flexible if we can provide something like minLabelStyle / maxLabelStyle?

Thanks for reminding. I think it should be enough with alignment for the first and last label, because the first / last label should not be more important than the others (think the case with dataZoom, where the first label is not fixed). The reason why we need alignMinLabel and alignMaxLabel is to avoid overriding the canvas, instead of wanting to make them distinctive. Also, if we provide minLabelStyle / maxLabelStyle with exact attributes like axisLabel, it may bring more bugs without a thorough test.
So, I would suggest provide alignMinLabel and alignMaxLabel unless developers expressed requirements beyout that. Hope this makes sense to you. Thanks.


ci(.github/workflows): add discover-lean-pr-testing.yml

This workflow finds branches of the form lean-pr-testing-NNNN
corresponding to core PRs that have been merged into the latest nightly.
These branches should be merged into the nightly-testing branch of
mathlib.
The workflow only reports branches that have a non-trivial diff.

Looks like I forgot about this, sorry! Let's try it out!
bors r+


foundry.toml

I download it, but couldn't use it.
It prompt "Could not find foundry.toml" error.
My OS is Microsoft windows 11.
Please help.

Hi @joesmith2019!
The debugger only works for Foundry projects and expects a foundry.toml file in your project root folder. For more detailed instructions, you may look at our Getting Started Guide.


adding @dgershman

Description
Adding @dgershman (from @radiusmethod) as a CODEOWNERS.
Related Issue
N/A
Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

welcome as a maintainer of zarf. :)


added corsFilter to the behaviors of RestController

I was presenting problems with cors and I noticed you do not include corsFilter in your rest controller.

Thank you for your contribution. Merged!


Fixed typo in media-preloading.md

Hello,
I've just fixed a typo (timline) in media-preloading.md
Best regards

Thanks for catching this and fixing it @pablobernabeu, much appreciated!

Thanks for catching this and fixing it @pablobernabeu, much appreciated!


Allow users to config init image url

Removed "Always" imagePullPolicy from init container spec
Bump broker crd version v2alpha4

Updated the branch but one more thing: need to port commit (allow user to update address-settings) from v2alpha3 to v2alpha4.


Edit token permissions endpoint

An endpoint to edit token's permissions is needed. Would probably require a new permission or use create_tokens_perm or its alternatives if it gets removed.

Onm the othr hand, tokens should be created with needed permissions, so editing token's permissions will not be implemented


fix js release

The latest release job pushed @saucelabs/visual 0.7.1 to npm but failed to push the changeset updates to git. This PR does a manual fix for that.


ViewPresenter with getView() != null and scope.isDestroyed() == true

In a ViewPresenter, we post a runnable on the UI thread just after onLoad(). We have a check when this runnable executes to make sure we are still on the current screen.
if (getView() == null){
    return;
}

However, we are tracking crashes that occur because the user switches screens quickly and this runnable executes after the scope has been destroyed. This means getView() is not null while the scope bas been destroyed. This does not seem like it should be possible, but we did solved it by modifying our check at the beginning of the Runnable:
if (getView() == null || scope.isDestroyed()){
    return;
}

Since this was filed ScreenConductor has been replaced by the SimplePathContainer in the Flow sample. You'll note that uses Animator instead of buggy old Animation. Animator's callbacks are reliable.
If you can possibly move from Animation to Animator it's well worth the time spent. If that's not an option, we worked around this problem by writing the name of a dying scope in a parent scope, and having that parent kill named scopes in its onload method — usually redundantly, which is safe, but it also catches dangling scopes left by this case.


yaml-dsl: enable flow mode deserialization by default

Some code may be removed with camel 3.11 (https://issues.apache.org/jira/browse/CAMEL-16693)


I found the bug in newest list!

I am user to use the movie guide app. Before reporting bugs, it's always helpful to use Movie Guides as a movie lover. But I fount the bug.  I would like to tell you the bug I found in order for movie guide app to go in a good direction. Even if a movie is loaded from the newest list, the same movie is loaded continuously. My wish is that when I load a movie, I want a new movie from before.
Thank you for reading the long article and I look forward to your reply ^_^

@esoxjem
Please answer for me once ㅠㅠ

Hi @tjalsrb97, thanks. I will take a look.

Already reported in #96


Pdf Text and logo hided or showed incorrectly in light grey after desabling textlayer color transparent setting.

Attach (recommended) or Link to PDF file here:
MissingLogoAndText.pdf
Configuration:

Web browser and its version: IE 11.125.16299.0 / firefox 58.0.2 64
Operating system and its version:Win 10
PDF.js version: 1.8.188 + 1.10.88
Is a browser extension: either directly with pdf.js from firefox or embedded in a web app.

Steps to reproduce the problem:

Upload the pdf with pdf version 1.7
Display with pdf.js

What is the expected behavior? (add screenshot)
Show the right logo and the text

What went wrong? (add screenshot)
Logo was transformed in some kind of ascii place holder and only few letters from the whole text are displayed

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Addendum: here are the custom fonts used for this document:

The document is readable without troubles in Acrobat XI 11.0.23 and PDF-XChange 6.0
Thx a lot in advance for any helpful hints.

Attach (recommended) or Link to PDF file here:
MissingLogoAndText.pdf

It appears that the provided PDF file contains nothing more than an image of the problem.
In order for this issue to be actionable, you must provide the actual PDF file where the issue was encountered.

PDF.js version: 1.8.188 + 1.10.88

Is this issue reproducible in the latest version of PDF.js, which can always be found at https://mozilla.github.io/pdf.js/web/viewer.html (use the "Open file" button on toolbar to open your file)?

Hi there,
the document is a high confidential encrypted customer document and I cannot provide it anywhere unfortunately, sorry. The issue isr reproducible, even with the latest release, that I already took for testing bevor opening this issue.
There are no troubles at all neither with Acrobat Reader X nor  PDF-XChange nor with Chrome; the rendering is correct. Only with pdf.js, almost the whole ascii content is kept hidden, cause of an activated transparency property, that when desabled in the inspection popup shows the content in light grey. Only few somehow random letters are displayed in bold black and the logo itself is changed through an ascii place holder.
"Virtual printing" with PDF24 ends also with the following error

due to a ghoscript ps transformation problem 8as an additional hint).

the document is a high confidential encrypted customer document and I cannot provide it anywhere unfortunately, sorry.

In that case we'll unfortunately have to close this issue as incomplete, since it's simply impossible to help without access to a test-case. My suggestion would be to, if possible, try and generate a PDF document with "dummy" content (which can be freely shared) using the same PDF creation software.

[...] cause of an activated transparency property, that when desabled in the inspection popup shows the content in light grey.

Note that this is the text-layer, which is used only to allow text-selection and searching, but has nothing to do with the actual rendering of the PDF file itself (which is done using HTML <canvas> elements).

Well this is a very unusual, uncommon and strange way nowadays to "solve" problems and fix real existing bugs.

For the lgiht grey text its ok, but how and why and under what kind of circumstances a graphic component is replaced through ascii place holder?

Why and how are some letters randomly picked up and showed in bold black.

Have you at least an idea about these effects and at least which part of the code is relevant for both cases so I can at least keep on investigating on my own.
Thx in advance.

@jameliddin PDF is a very complex format. I work in a different PDF project (Apache PDFBox) and most of the time we can't fix problems without having the PDF. And when we got it, most of the time the cause of the problem wasn't the first guess. It's like when your pet is sick, you'll have to bring it to the vet despite 1) you thinking that the symptoms are obvious 2) the pet hating to go to the vet.
I suggest you open the file in the tools mentioned in https://github.com/mozilla/pdf.js/wiki/Debugging-PDF.js or Apache PDFBox PDFDebugger or itext RUPS to look for any weird things or any log messages.

Great thx. I know its much easier to have the original material but as Ive mentionedit  bevor its a highly confidential encrypted bank document and am not allowed to make it public.
Anyways I'll give it a try with your suggestion and we'll see.
Greetings.

same problem. did you fix this? @jameliddin


TShock update

Recent TShock update break the plugin again.
People from mobile devices can't join on <IP_ADDRESS> version. They get an error "Lost connection. You are not using the same version as this server".

Make sure all of your server files are updated (most importantly OTAPI.dll) to <IP_ADDRESS>. Make sure crossplay is also updated to 1.10.2

Thank you! My problem was that the plugin was outdated.


Task 1

Hi,
I am new to uipath. Can you please tell me what I have to do? You are
telling me to change the activity name.
On Thu, Aug 3, 2023, 7:26 AM swathinelakurthi @.***>
wrote:

Activities are not named properly.
[image: Screenshot 2023-08-03 195011]
https://user-images.githubusercontent.com/76664104/258142575-85d747f8-02f5-40fe-aff9-4b37a309159e.png
—
Reply to this email directly, view it on GitHub
https://github.com/vajrang-b/RPA-Developer-in-30-Days/pull/774#issuecomment-1664087200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BA62GO2D3BEK37U3ZQW5A5LXTOYKHANCNFSM6AAAAAA3CD2JZI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I changed Activity name

@swathinelakurthi Please check


fix: improve exception messages when duplicating aliases

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

Comparison is base (3ed207a) 72.77% compared to head (2b7f2a2) 72.86%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #809      +/-   ##
==========================================
+ Coverage   72.77%   72.86%   +0.08%
==========================================
  Files         263      263
  Lines        3387     3398      +11
  Branches      591      593       +2
==========================================
+ Hits         2465     2476      +11
  Misses        769      769
  Partials      153      153

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


Adds Service Account model object, DAOs, and controller.

Moving static configuration of service accounts from Fiat to Front50.
@cfieber @duftler @jtk54 PTAL
@ajordens FYI

LGTM, deferring to others to merge.


please release also on PyPI

The Python Package Index is the repository of software for the Python
programming language. When you make a new release we expect to find it there.

Please don't forget to include it when you make a new one.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 27 Oct 2013 at 5:19

This is not fixed.


#3474 #3467

#3467 RTVS locks package dlls, interferes with package (re)installation

track installation requests better. Throttle index update if multiple commands are processed in quick succession

#3474 ESC key in Interactive Window cancels session

track readline() and similar. If session is waiting for the user input, do not handle ESC as cancel/interrupt. This also alleviates issues that interrupting R while readline() is active never returns to the prompt and REPL has to be reset.

This also alleviates issues that interrupting R while readline() is active never returns to the prompt and REPL has to be reset.

Can you file a separate bug for this? Interrupt R should work when waiting on ReadConsole.

Sure. It interrupts, but interactiveWindow.ReadStandardInput is not cancelleable. However, it does return on, say, Enter.

        public async Task<string> ReadUserInput(string prompt, int maximumLength, CancellationToken ct) {
            _coreShell.DispatchOnUIThread(() => _interactiveWindow.Write(prompt));
            var tcs = new TaskCompletionSource<string>();

            Task.Run(() => {
                using (var reader = _interactiveWindow.ReadStandardInput()) {
                    tcs.TrySetResult(reader?.ReadToEnd() ?? "\n");
                }
            }).DoNotWait();

            tcs.RegisterForCancellation(ct).UnregisterOnCompletion(tcs.Task);

            try {
                return await tcs.Task;
            } catch (OperationCanceledException) {
                _coreShell.DispatchOnUIThread(() => _interactiveWindow.Operations.TrySubmitStandardInput());
                throw;
            }
        }

👍


Pre-Processing Cache Fails when using a Factory pattern

Describe the bug
If you utilize a pre-processing function which is created using a factory pattern, the function hash changes on each run (even if the function is identical) and therefore the data will be reproduced each time.
Steps to reproduce the bug
    def preprocess_function_factory(augmentation=None):
        def preprocess_function(examples):
            # Tokenize the texts
            if augmentation:
                conversions1 = [
                    augmentation(example)
                    for example in examples[sentence1_key]
                ]
                if sentence2_key is None:
                    args = (conversions1,)
                else:
                    conversions2 = [
                        augmentation(example)
                        for example in examples[sentence2_key]
                    ]
                    args = (conversions1, conversions2)
            else:
                args = (
                    (examples[sentence1_key],)
                    if sentence2_key is None
                    else (examples[sentence1_key], examples[sentence2_key])
                )
            result = tokenizer(
                *args, padding=padding, max_length=max_seq_length, truncation=True
            )

            # Map labels to IDs (not necessary for GLUE tasks)
            if label_to_id is not None and "label" in examples:
                result["label"] = [
                    (label_to_id[l] if l != -1 else -1) for l in examples["label"]
                ]
            return result

        return preprocess_function

capitalize = lambda x: x.capitalize()
preprocess_function = preprocess_function_factory(augmentation=capitalize)
print(hash(preprocess_function)) # This will change on each run
raw_datasets = raw_datasets.map(
        preprocess_function,
        batched=True,
        load_from_cache_file=True,
        desc="Running transformation and tokenizer on dataset",
)

Expected results
Running the code twice will cause the cache to be re-used.
Actual results
Running the code twice causes the whole dataset to be re-processed

Ok - this is still an issue but I believe the root cause is different than I originally thought. I'm now able to get caching to work consistently with the above example as long as I fix the python hash seed export PYTHONHASHSEED=1234

Hi!
Yes, our hasher should work with decorators. For instance, this dummy example:
def f(arg):
    def f1(ex):
        return {"a": ex["col1"] + arg}
    return f1

gives the same hash across different Python sessions.
Could you please make the example self-contained? This way, we can reproduce the bug. Additionally, you can try to find the problematic object yourself by testing their hash with datasets.fingerprint.Hasher.hash(obj)
This could be related to https://github.com/huggingface/datasets/issues/3638.

#3638 was indeed my issue. Thanks!


Restore old functionaliy for some Quill toolbar classes

Description
Today I have again spent time to make my project looks as it must. I do not do anything critical just restore the funcitonality of some toolbar classes as they were. You may consider this PR as a starting point for all other buttons which I didn't change. The only thing is required is change call of QuillToolbarIconButton in build methods of embed buttons, search button and maybe others. You may see what I did. I also restored old QuillToolbarSelectAlignmentButton and save it as QuillToolbarSelectTextAlignmentButton.
I need this modifications to make my project compiled and continue working. You may merge this PR or use it for understanding how to make package backward compatible. It will take less effors.
Related Issues

Fix #1628

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[x] No, this is not a breaking change.

If we solve the conflicts and merge this, does that fix your issue?

If we solve the conflicts and merge this, does that fix your issue?

I restored only buttons which I used but there are other ones which are not fixed.
I think the package should works properly for Material 2 and Material 3. This is the only thing you should care. Agree?
So if button in web looks like

before modifications it should looks like so after modifications but not like

I tested example app so you may read https://github.com/singerdmx/flutter-quill/issues/1628#issuecomment-1864783307 for results.

Closing this as it can be solved without reverting, it's looks like the issue of using the new Material 3 recommended widgets instead of the previous ones that work for Material 2 and 3, updating to 9.x.x require using Material 3 and the new widgets listed in Flutter 3.16
See the following for more details and suggestions:

https://github.com/singerdmx/flutter-quill/issues/1628#issuecomment-2133522902

https://github.com/singerdmx/flutter-quill/issues/1628#issuecomment-2135463200

If you have any questions or suggestions, don't hesitate to create a new discussion or leave a comment here.


Ensure Prototype Kit works with Node 14 (Fermium)

What

Test that the Prototype Kit runs correctly under Node v14
Fix any issues identified when running under Node v14
Add Node v14 to the matrix for Travis

Why
Node v14 will become the active LTS release in October 2020, at which point we should recommend that users start installing it. (v12 will remain supported under LTS until 30 April 2022)
Until then, some users may only be able to install v14 as the current/latest version – for example, if a department provides Node through a managed software centre – so we should make sure it works with it.

I think the kit crashes with v14. At least it does for me - though I'm using an older version of the kit.
This is the error I get:
(node:37097) Warning: Accessing non-existent property 'padLevels' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
/Users/edwardhorsford/Code/sync-with-dropbox/beis-mspsds-design-prototype/node_modules/node-sass/lib/binding.js:13
      throw new Error(errors.unsupportedEnvironment());
      ^

Error: Node Sass does not yet support your current environment: OS X 64-bit with Unsupported runtime (83)
For more information on which environments are supported please see:
https://github.com/sass/node-sass/releases/tag/v4.13.1

I've tried bumping to the latest gulp-sass - (which is what I assume is importing node-sass) but it still doesn't work.


[#155623416] Restructure/rename instance_groups to match cf-deployment (PR 2 of 4)

#155623416 Restructure/rename jobs to match cf-deployment
What
Remove and re-order VMs.
How to review
Review in conjunction with https://github.com/alphagov/paas-cf/pull/1263. This branch should be deployed on top that one. We need:

code review
check app and API availability tests pass.
check acceptance tests et cetera pass

This pull request involves removing the global route-emitter while we have cell-local route emitters in place, so we are mainly concerned about availability of routes during the deployment.
Before merging ⚠

review and merge https://github.com/alphagov/paas-cf/pull/1263 . Must be deployed in Prod before the rest
review and merge https://github.com/alphagov/paas-datadog-for-cloudfoundry-boshrelease/pull/23 and replace the temporary commit on this branch with the final release of the datadog-for-cloudfoundry release.
rebase on master

Who can review
Anyone but @keymon or me

I took the liberty and removed the the commits which removed the route_emitter Datadog monitors. It will be deployed in stage 4

I took the liberty and removed the the commits which removed the route_emitter Datadog monitors. It will be deployed in stage 4

Cool, you are right, thank you.


Instructions for confirming new email dissappear

When a user changes their email, they need to confirm that they own the account before the change goes through. At present, the banner informing the user regarding this disappears very quickly, before the user even has time to read it.
Instead, the banner should remain open until either the email is confirmed or the confirmation email expires (assuming it expires? it's not clear from the email whether that's the case).

@sahil839 would you be up for fixing this one?

@sahil839 would you be up for fixing this one?

Sure.
@zulipbot claim


how to make it faster

i installed the latest version and its a cool app but it so slow I'm running the vicuna 7b is there a way to make faster i have an 8gb ram phone and what other models does support and please link me to them

I have a branch that moves more of the processing into native code, I believe it should bring a noticable performance improvement. You can also try 3B models with this version, which should also be much faster. Feel free to try.
Note that the new llama.cpp changes model compatibility, models that used to work with Sherpa probably don't work any more until conversion.
Pull request: https://github.com/Bip-Rep/sherpa/pull/12
apk available: https://github.com/dsd/sherpa/releases/tag/2.2.1-dsd

I have a branch that moves more of the processing into native code, I believe it should bring a noticable performance improvement. You can also try 3B models with this version, which should also be much faster. Feel free to try. Note that the new llama.cpp changes model compatibility, models that used to work with Sherpa probably don't work any more until conversion. Pull request: #12 apk available: https://github.com/dsd/sherpa/releases/tag/2.2.1-dsd2

Hi dsd,  it works with the apk you provided , but I failed to run it from your forked source.
and when I run on my Mac , it shows "Library not loaded: @rpath/libllama.dylib"

It's my first time developing Android apps but feel free to share info about the failure to run from source and I will let you know if I have any ideas.
I did not do any work to retain Mac compatibility but I think this is what needs to be done: https://github.com/Bip-Rep/sherpa/pull/12#issuecomment-1621045871

I have a branch that moves more of the processing into native code, I believe it should bring a noticable performance improvement. You can also try 3B models with this version, which should also be much faster. Feel free to try. Note that the new llama.cpp changes model compatibility, models that used to work with Sherpa probably don't work any more until conversion. Pull request: #12 apk available: https://github.com/dsd/sherpa/releases/tag/2.2.1-dsd2

Is this app using both CPU and GPU of smartphones?
Also, is there any chance to make it run with less RAM like 4gb?

llama.cpp is used as the backend, so you would need to check if llama.cpp supports your GPU, and if it is usable on 4GB RAM with the model you are interested in.

llama.cpp is used as the backend, so you would need to check if llama.cpp supports your GPU, and if it is usable on 4GB RAM with the model you are interested in.

Does it support any mobile gpu like mali or adreno?


Print return value for forge run

Component
Forge
Describe the feature you would like
Print the return values for returns functions when using forge run.
Additional context
An example function:
function greater(uint256 a) view external returns (uint256 gasUsed) {
    uint256 startGas = gasleft();
    require(a > 0, "!a > 0");
    gasUsed = startGas - gasleft();
}

Now if I e.g. run:
forge run --optimize --optimizer-runs 200 --use 0.8.13 --via-ir GasTesting.sol --sig "greater(uint256)" 0x64

it should print the value of gasUsed.

Working on this. What is the preferred printing format? Currently, I'm printing the returned data using
println!("{:?}", returned_data)
[⠘] Compiling...
[⠊] Compiling 1 files with 0.8.13
[⠒] Solc 0.8.13 finished in 100.44ms
Compiler run successful
Script ran successfully.
Gas used: 220
Result: b"\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\x1d"
== Logs ==

Awesome, thx for working on this issue - well I would suggest having something like the following:
[⠘] Compiling...
[⠊] Compiling 1 files with 0.8.13
[⠒] Solc 0.8.13 finished in 100.44ms
Compiler run successful
Script ran successfully.
Gas used: 220
== Result ==
0: <type>: <variable_name> <return_value>
1: <type>: <variable_name> <return_value>
⋮
== Logs ==

So let's say you have a function like that:
function bar(uint256 x) external view returns (bool result, uint256 gasUsed) {
    uint256 startGas = gasleft();
    assembly {
        result := iszero(x)
    }
    gasUsed = startGas - gasleft();
    return (result, gasUsed);
}

The output could look like this (similar to Remix):
[⠘] Compiling...
[⠊] Compiling 1 files with 0.8.13
[⠒] Solc 0.8.13 finished in 100.44ms
Compiler run successful
Script ran successfully.
Gas used: 220
== Result ==
0: bool: result true
1: uint256: gasUsed 18
== Logs ==

@onbjerg what do you think?

So on digging into how the function is getting called, there doesn't seem to be a straight forward way of getting the function's return types, since run is just encoding the calldata using arguments from the command line then passing it to the runtime bytecode. The way this is being done doesn't seem to involve fetching the contract ABI.
One way to enable return data decoding could be to include an option such as --decode (<return_types>) to decode the return data.

@JoshuaTrujillo15 since we compile the contract before running the script, we could simply retrieve the ABI beforehand under the hood using solc Contract.sol --combined-json abi,bin (adding the flag --abi)? The bin code is anyhow used somehow to deploy the contract locally iirc.

So on digging into how the function is getting called, there doesn't seem to be a straight forward way of getting the function's return types, since run is just encoding the calldata using arguments from the command line then passing it to the runtime bytecode. The way this is being done doesn't seem to involve fetching the contract ABI.
One way to enable return data decoding could be to include an option such as --decode (<return_types>) to decode the return data.

If you pass --sig "myFunc(uint256)(uint256)" it will be parsed into a function that takes a uint256 and returns a uint256 - this information should be possible to get in forge run. For raw calldata it will not be possible

I found a way to get the function ABI without adding any additional commands. The Token enum that gets printed below seems to print numbers as hexadecimal, but not in an appropriate hexadecimal format. This seems to be because of how fmt is implemented for Token.
== Result ==
0: bool      true
1: int256    ff
2: uint256   2
3: address<PHONE_NUMBER>000000000000000000000000000003
4: bytes32<PHONE_NUMBER>000000000000000000000000000000000000000000000000000004
5: bytes
6: string    test
7: uint8[]   []

It seems the best way to get this formatted is to run a match against every Token type and append the appropriate prefix, since some types such as string just print naturally, but this may be overkill for a single feature. Thoughts?

I don't think running a match against every Token type and appending the appropriate prefix and converting them appropriately (e.g. hex to int) significantly impacts the overall speed. I don't see another way tbh (changing the implemented fmt approach is not the way to go IMHO). Also, we probably need to test thoroughly every type to assure that e.g. strings are always naturally printed. @onbjerg @gakonst you know much more about the inner workings of Foundry - what would be your recommendation here?

@zk-tarts that's awesome input! just tested it and it works:

do you know by chance what are the output formats within the logs for e.g. string, arrays & co.?

do you know by chance what are the output formats within the logs for e.g. string, arrays & co.?
string outputs as a quoted string, arrays should be output as the the comma seperated values surrounded by square brackets

I don't think running a match against every Token type and appending the appropriate prefix and converting them appropriately (e.g. hex to int) significantly impacts the overall speed. I don't see another way tbh (changing the implemented fmt approach is not the way to go IMHO). Also, we probably need to test thoroughly every type to assure that e.g. strings are always naturally printed. @onbjerg @gakonst you know much more about the inner workings of Foundry - what would be your recommendation here?

Believe that's what Brock did here, it's kinda gross but it works well: https://github.com/foundry-rs/foundry/blob/master/utils/src/lib.rs#L750-L773

As @gakonst said you can use format_token to display the return values etc. You can get the output tokens using this function from ethers-rs

Also, we probably need to test thoroughly every type to assure that e.g. strings are always naturally printed

This is tested in ethers-rs (and ethabi?), not sure if there is much value in duplicating that here.

Awesome, thank you all for your help! I'll have a PR up soon. 🤘

LFG 💥

Fixed in https://github.com/foundry-rs/foundry/pull/1592


Turn on optimizely on comparison matrix, faq, and docs home

At @swatikumar16's request, this PR adds the optimizely script to cockroachdb-in-comparison, frequently-asked-questions, and index.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a20e28f40a39ddd4b3c75eddd2874179e75d3e08/


Adds Gitpod Support

Here it is, a 1-click customizable installation of Filament's Demo project.
It's possible to use it inside the browser and with a local VSCode/Jetbrains. I forgot to do that in the video.
Let me know if a short (edited) YouTube video would be nice.
https://app.usebubbles.com/2Fhk4kzyqMnH1Sf4dg4XL9/filament-demo-on-gitpod

Hey @danharrin
So here it is as spoken. With the correct link, then I can proceed to the next step 😃

Still working?

It will, after merge 😄

Current main branch is missing the .gitpod.yml file.


Fleet configimporter no longer works

Fixing this by pinning the version of Fleet to 1.0.8 for now until I can figure out a way to use fleetctl

I'm having the same problem

I have a fix ready for this on the 1804 branch. I’m hoping to be able to merge to Master tonight

On Jun 25, 2018, at 1:00 PM, Jonas Lejon<EMAIL_ADDRESS>wrote:
I'm having the same problem
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Tested the 1804 branch and it worked :) Thanks!

Fixed in #116


android, ios support

does this run on android? what flutter version should it work?

Yes, this can run on andorid & iOS


Link iTunes-Discord-RP for MacOS users

i am user of 🍎operating system
🍎operating system good operating system
this software good software, but no support 🍎operating system :(
sad day
but i find alternative
others shall also find this alternative if you dear @nint8835 accept my pull request
many thanks
jackathy
p.s. i have office hours

It is scaaaaary


Operation dispatcher err handling

Fixes https://github.com/Kuadrant/wasm-shim/issues/123

This is still not working for me. I run the following configuration:

{
  "services": {
    "limitadorA": {
      "type": "ratelimit",
      "endpoint": "limitador",
      "failureMode": "deny"
    },
    "limitadorB": {
      "type": "ratelimit",
      "endpoint": "unknown",
      "failureMode": "deny"
    }
  },
  "actionSets": [
    {
      "name": "basic",
      "routeRuleConditions": {
        "hostnames": [
          "*.example.com"
        ]
      },
      "actions": [
        {
          "service": "limitadorA",
          "scope": "basic",
          "data": [
            {
              "expression": {
                "key": "a",
                "value": "1"
              }
            }
          ]
        },
        {
          "service": "limitadorB",
          "scope": "basic",
          "data": [
            {
              "expression": {
                "key": "a",
                "value": "1"
              }
            }
          ]
        }
      ]
    }
  ]
}

There are  two actions, both with failureMode to false. And the second one targets a missing cluster. On running a request, the request makes its way to upstream and returns 200 regardless of failureMode.

Now @eguzki ! this is your time to shine !!! (and thanks in advance for the reviews 🙏🏼 )

(I've included the commits from your test, just for easy testing... I could remove them from this PR if you want)

By the way @eguzki, this PR also fixes your https://github.com/Kuadrant/wasm-shim/pull/122/ ;)


build: add another workaround for CMake <3.16

Linking against libraries from the system was broken on Windows with
CMake 3.15.  It would require explicitly adding the -l prefix which is
less than ideal.  This workaround allows building without a dispatch or
Foundation build tree available for Windows.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


⚠ Bofrost.nl has degraded performance

In d55cc95, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1512 ms

Resolved: Bofrost.nl performance has improved in 8579ba2.


Revert "Restored Triple Threat Titan gun"

This reverts commit 6dfe2856e8c4b9c533f0c380abbfd4b6fada3518, i.e. #276
While there is no known issue with the Triple Threat (apart from missing logic where the crosshair wouldn't rotate on ADS), it introduces full compatibility break with older versions that do not have the Triple Threat. This is especially problematic atm as with "Titanfall day" coming up we want as much compatibility as possible between 1.6, 1.7 and maybe a version thereafter.
To make running testing things on main branch less of a hassle I'd suggest removing the Triple Threat from main again for now and re-opening a PR to add it again.
In order to actually release it, I'd suggest an update solely dedicated to it with no other changes. That way server owners can update their servers without any fear of breakage introduced by other features which should hopefully result in a quick update rate.
Probably best to release it in ~2 weeks from now once the potential new crowd of players from "Titanfall day" has gone down a bit again.

@taskinoz

Turned to draft, as simply removing the pre-cache line makes more sense.


Add LibrePGP PreferredEncryptionModes signature subpacket

I decided to implement this in its own dedicated class to express that this is not an official OpenPGP signature subpacket.

This PR now includes changes from #1464


React v15.5 deprecating React.createClass

I was upgrading my React version and now am receiving this error:
Warning: ReactPageClick: React.createClass is deprecated and will be removed in version 16. Use plain JavaScript classes instead. If you're not yet ready to migrate, create-react-class is available on npm as a drop-in replacement.

Need to switch to prop-types and create-react-class. PR welcomed!


feat: add codespell linter job for CI

Feature Request
Add codespell  linter job for CI.
Why

There are many pull requests If you search for "fix typo".

It’s ultimately important to spell quality correctly when your role is in QA. It’s worth being vigilant for errors of any kind, from spelling to dead links to scripting issues. Sometimes, we will bring in bugs even crash because of typo error.

So we can't ignor the "fix typo" pull requests.

Describe Preferred Solution
imo, add codespell linter job for CI will be a good solution.

Fixed in linked PR.

@orizi wow! Glad to see you add workflow in CI so quickly. Thank you very much.


Update documentation references due to partnership changes

Due to changes in the consortium partnership that took effect on 1 April 2020, can we please review the documentation for the Platform and Genetics Portal and make the following changes?

Remove all references and web links to Biogen
Replace all Celgene references and web links with Bristol Myers Squibb and link to their website at https://www.bms.com

We have no references to Biogen in the Open Targets Platform docs.
As per Open Targets Genetics docs, we had two instances referring to Biogen and Celgene:

https://genetics-docs.opentargets.org/#our-partners
https://genetics-docs.opentargets.org/#copyright

Biogen removed. Celgene changes to


HTTP Resource invalidates HEAD requests with no body but Content-Length set

If the advertised Content-Length is bigger than the allowed data_limit the request is cancelled even if there is no response body.

Ready for review.

@jerith thanks, updated.

:+1:


Monitoring for litmus.

Proposed changes
Adds monitoring directory with instructions and grafana dashboards.
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[x] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the commit for DCO to be passed.
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have added necessary documentation (if appropriate)

Dependency

None.

Special notes for your reviewer:
Litmus Monitoring with Grafana and Prometheus.

@ksatchit @umamukkara Please take a look.


[$1000] Web - Clicking on PDF causes NewDot to crash

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to any chat with a pdf attachment
Click on attachment

Expected Result:
Attachment loads
Actual Result:
Clicking on PDF causes NewDot to crash
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.38-3
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/894ae359-d225-47b2-92c8-8f88fd397b4c
Expensify/Expensify Issue URL:
Issue reported by: Dylan Courtney
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688995158435269
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0152bfc273f0c5cb94
Upwork Job ID:<PHONE_NUMBER>875233280
Last Price Increase: 2023-07-19

I am unable to reproduce this issue. The scenarios I tested:

send a pdf from account 2 and open in account 1
send a pdf from account 2 and then send more chats from both sides and then open in account 1
send pdf from account 1 and open in same account
send pdf from account 1 and them messages from both sides and open the pdf in account 1

I'm unable to reproduce this issue as well using the reproduction steps. @dylanexpensify Can you re-open this issue if you're able to reproduce somehow?

@JmillsExpensify One of the contributor experienced crash with PDF file:
Action Performed:
1 User A (android) initiates a chat with User B (web)
2 User A (Android) uploads a PDF file to the chat
3 User B opens the file.
4 User A deletes the PDF file.
5 User B attempts to download the deleted PDF file. Observe, its successful
6 User B refreshes the page. Observe the error
Expected Result: 1 After User A deletes the PDF file, User B should no longer be able to download it.
2 Refreshing the page after PDF deletion should not cause any errors or crashes.
Actual Result: 1 User B is still able to download the deleted PDF file, indicating an inconsistency in the deletion process.
2 Refreshing the page after PDF deletion leads to an error message and page crash, causing inconvenience and potential data loss.
https://github.com/Expensify/App/assets/93399543/e946746c-3d40-435b-b559-c4a9563e1bc9
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689275002964209

Thanks @kbecciv that's helpful! Triaging.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Clicking on Deleted Attachment causes NewDot to crash
What is the root cause of that problem?
We have two problems here

Downloading Deleted Attachment.

With the existing implementation we don't know if the message is deleted or not before downloading.
Implementation is here
https://github.com/Expensify/App/blob/f2259369ed001c77cd6670a8416217f431eb523a/src/components/AttachmentModal.js#L142-L153

Showing Error screen in-case of opening/refreshing Deleted Attachment using URL
Since image is deleted and the image source with the reportAction would be empty.
https://github.com/Expensify/App/blob/4e7c9e04127d6fd20b2b81f7afa87c0f7633f8c2/src/components/AttachmentCarousel/index.js#L192
The above code snippet is throwing error since it wouldn't able to find the attachment and throwing error.

What changes do you think we should make in order to solve the problem?
For Problem 1
Once user open's the attachment I don't think we don't have a reliable way to catch it in realtime but I feel a work around can be done here.
https://github.com/Expensify/App/blob/f2259369ed001c77cd6670a8416217f431eb523a/src/components/AttachmentModal.js#L143-L154
In this function we can add a check before downloading the file.
In the URL we have the reportID and in the source parameter we can parse reportActionID which can be used to detect if the message is deleted or not by following code snippet.
ReportActionsUtils.isDeletedAction(ReportActionsUtils.getReport(reportID, reportActionID))

Which returns true if it's true we don't anything (not doing file download) or maybe we can do some kind of alert saying this file is not downloadable maybe some personalized message as well.
For Problem 2
Detecting the deleted attachment before throwing the error.
We can use the same above method to detect if the attachment is deleted and if it's a deleted attachment we navigate to back.
const reportActionID = source ? source.split('/')[2] : '';

const [isDeleted, setIsDeleted] = useState(false);

useEffect(() => {
    if (!reportID || !reportActionID) return;
    const reportAction = ReportActionsUtils.getReportAction(reportID, reportActionID);
    const isActionDeleted = ReportActionsUtils.isDeletedAction(reportAction);
    setIsDeleted(isActionDeleted);
}, [reportActionID, reportID]);

useEffect(() => {
    if (!isDeleted) return;
    Navigation.goBack();
}, [isDeleted]);

PS: Getting reportActionID can be done differently but I chose split cause it's easy ;). This method won't work on prod or staging env cause of the different source URL this is done for testing purpose in local.

What alternative solutions did you explore? (Optional)
NA
Result

Problem 1
uu.webm

Problem 2
ii.webm

@allroundexperts my proposal is based on this reproduction steps https://github.com/Expensify/App/issues/22624#issuecomment-1635776540 Please check and let me know.

if you're referring to this https://github.com/Expensify/App/issues/22624#issue-1798597942 I don't think it's possible now. If you see the video source url contains s3 URL but right now with latest it used /chat-attachments URL.
And my proposal is working fine on web, might need some tweaks on native platforms but ultimately the way I'm gonna implement is the same.

@allroundexperts Should I need to update my proposal if then what was the bug? can you share the comment to it?

Convo still happening above, though in general we are still open for proposals.

For the first issue, I don't think we should block the download if the image is open already. A while ago, we were automatically hiding the image in the carousel if it was opened. But recently, we changed it such that an opened image remains visible even if its deleted. Following this, I don't think we should disable the download of an image which got deleted while it is open. If you disagree, then we might need some input from the design team as we need to show some sort of alert indicating the reason why the download failed.

I agree with you. We shouldn't disable the download.

For the second issue, Is it fine to just silently navigate back to concierge chat? I assume that we need to show a Not Found page. If that's true, then @b4s36t4 might have to update his proposal.

Also agree with you here. I don't really like that we're silently navigating back to Concierge. You're right that a better pattern is showing a Not Found page.

Thanks @allroundexperts @JmillsExpensify Will update my proposal according to the expected OP.

@allroundexperts Updated the Proposal https://github.com/Expensify/App/issues/22624#issuecomment-1642614433

@allroundexperts It's kind of same what you said above.
This is how it's now.

User A Opens attachment
User B deletes attachment.
User A still able to view/download attachment without any issues.
User A navigates(using carousel) to any other attachment and come back to the deleted attachment it would show NotFoundPage.
Copy the URL and access it in other tab, the same behaviour you're expecting showing the NotFoundPage.

Since carousel navigation causes the props change it would trigger the condition and logic that I have written which is showing NotFoundPage.
I feel this behaviour is pretty much what we should be getting, user will only able to view/download the attachment when he is active with the attachment even if it is deleted. Once he moves away from the attachment ain't it ok to show deleted attachment??
Can you please re-watch the demo I have shared you'll get the gist of it now.
Thanks.

Apologies ya'll. Catching up now.

I agree with @allroundexperts. I'll go ahead and increase the price.

ReportActionsUtils.isDeletedAction(reportAction);

Hey @allroundexperts, so as you explain will the fix also cover the cases when

User use carousel and back to deleted attachment?

If yes or no, what will the user see in this case? Thanks
@JmillsExpensify

@b4s36t4 We don't want to show the not found page when navigating using carousel. It should only show up once you open it via deeplink.

Hey @allroundexperts, so as you explain will the fix also cover the cases when
User use carousel and back to deleted attachment?
If yes or no, what will the user see in this case? Thanks
@JmillsExpensify

@studentofcoding If the user navigates back to the carousel, in that case, the deleted attachment should simply be skipped like it is currently.

Can you elaborate on what skipped means?

Issue not reproducible during KI retests. (First week)

Thanks ya'll!


feat: add intermediate result YAML saving

I added a functionality that regularly saves intermediate fit results, so not all of the progress is lost upon abortion of the fit or a death of the process.
Closes #158

Here are a few thoughts:

We should try to follow the interfaces of keras optimizers
If we write to a format that can be understood by tensorboard, we may be able to use tensorboard to monitor fit progress.
For that we need to write to different sequential log files, see also https://github.com/ComPWA/tensorwaves/pull/154#discussion_r521906639

This would be a perfect time to test our new branching model in action. The idea is to push this code into a new "feature" branch. In this branch the "code quality" are not as strict as in the master branch, but allow quick additions of new features. Then those branches can be refactored and merge with the master later on. @redeboer can tell a bit more about the details

Since this concerns quite a few issues, I created a branch 99-export-fit that is to address #99.

I updated this PR a bit with the new callback syntax (#164). Now it should also close #158, as illustrated in the notebook.
@sebastianJaeger, have a look here for the new implementation
https://github.com/ComPWA/tensorwaves/pull/154/commits/2d621a560ad7b6d13fbf33e4c2c61632c63b7a54?file-filters[]=.py#diff-9f533f5231a0e5fcd84806765d6bbf0282451172465fee983ac5df640128cceeR34-R77

I added some more ideas. But depending on how fast this functionality is needed it can be merged to the epic branch and improved there later on. Hence i approved the PR


Does it work?

I ran across your blog post discussing your intentions with this project. I just wondered if you ever managed to get it functional? I know it hasn't been updated in a while, but I was curious.

I'd like to know as well.

Hi Folks  - I haven't tried it in years, sorry.  The code is pretty straightforward to read (I hope).  Your best bet is to just try it out.


Add api call for HostExternalSettings

Pretty straightforward.

 It's not a paradigm we've used yet anywhere in the codebase.

true, I was only thinking about the internal API, not the HTTP API. I don't think returning both data and an error is a viable option for the API.


Scaling Issues

There is an odd issue with the scaling of the window. I think it has to do with my windows settings having the font at 125% and not 100%.

Hmm, it is not 125% Scaling alone - I do have that configured too. What is your Resolution?

2560x1440

That is also exactly my Resolution ^^

So my scanning is 100%, but I adjusted the windows text size in accessibility to 125%.
So it's possible that because the screen size didn't readjust and only text size did it clipped

Ah ok, that Scaling/Setting!
Did not say it is impossible, just trying to figure out the Conditions ;)


replace switch with routes

close: #419

Can you upgrade react-router-dom with this ?

Can you upgrade react-router-dom with this ?

mean version of react-router-dom?

Yes

Yes

See this what @johnaohara said #418

[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager] Creating an optimized production build...
[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager] Failed to compile.
[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager]
[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager] Attempted import error: 'Routes' is not exported from 'react-router-dom' (imported as 'Routes').

#421

[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager] Attempted import error: 'useHistory' is not exported from 'react-router-dom' (imported as 'useHistory').

[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager] Failed to compile.
[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager]
[INFO] [io.quarkiverse.quinoa.deployment.packagemanager.PackageManager] Attempted import error: 'Routes' is not exported from 'react-router-dom' (imported as 'Routes').

localhost:8080 doesn't render the homepage anymore...