Add an onUnsubscribe hook

Enable adding some connection state tracking on the server code.
Here are the signatures now:

onSubscribe(parsedMessage, params, subId)
onUnsubscribe(graphqlSubId, subId)

By tracking the subId we can now identify if a client is connected or not.

@rricard good idea. I can merge it as soon as you add a test for it 👍

@helfer , will be a good occasion to test some stuff marked as TODO as well

I would just write a simple test that checks that onUnsubscribe actually runs when someone calls unsubscribe.
Oh, and one more thing: You should also think about whether onUnsubscribe should run when the connection just closes without the client calling unsubscribe first.

@helfer that's what I was worried about indeed, after using this in a custom fork today, I think I'll try some other edge cases as well

@rricard do you think this is close to ready to merge? Let me know when you've gotten the tests to pass, and I'll review it again!

@helfer I'm still not completely sure the PR is good. I'm trying to trace a rather large leak that may be caused by this. So I want to be careful here, needs more testing.

@rricard Have you had any chance to follow-up on this? If not I'd be happy to take a look as well!

@kesne Couldn't finally get a reliable way to do that. I went to an activity-based system where I trigger jobs if and only if an user was active in the last hour.
I wonder if the core meteor team that actually worked a lot with websockets could give their feedback on this... I think they solved most of the problems this package is having!

see discussion here - https://github.com/apollostack/graphql-subscriptions/pull/27

I'm closing this for now, I don't think this implementation is good. Feel free to get inspiration from it but know it has many shortcomings (we can't really ensure the hook will be called!)


WIP [state] pre-allocate and recycling Node struct

Description
Implement the object pool of the Node structure to reduce the memory allocation overhead of immutable-trie.decodeNode
Changes include

[x] New feature (non-breaking change that adds functionality)

Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it
Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels

Testing

[x] I have tested this code with the official test suite
[x] I have tested this code manually

Manual tests

Codecov Report

Merging #277 (75d2283) into dev (a840ee8) will decrease coverage by 3.07%.
The diff coverage is 40.47%.

@@            Coverage Diff             @@
##              dev     #277      +/-   ##
==========================================
- Coverage   51.21%   48.13%   -3.08%
==========================================
  Files         117      127      +10
  Lines       17525    19514    +1989
==========================================
+ Hits         8975     9393     +418
- Misses       7789     9324    +1535
- Partials      761      797      +36

Impacted Files
Coverage Δ

blockchain/storage/kvstorage/kvstorage.go
54.16% <ø> (+2.46%)
:arrow_up:

consensus/ibft/hooks.go
55.26% <ø> (ø)

consensus/ibft/pos.go
0.00% <0.00%> (ø)

helper/keccak/keccak.go
0.00% <ø> (ø)

helper/keccak/pool.go
0.00% <ø> (ø)

jsonrpc/debug_endpoint.go
36.08% <0.00%> (-0.76%)
:arrow_down:

jsonrpc/net_endpoint.go
0.00% <0.00%> (ø)

network/bootnodes.go
100.00% <ø> (ø)

network/connections.go
98.07% <ø> (ø)

network/server_nonnetwork.go
0.00% <0.00%> (ø)

... and 106 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Decimal does not expose any constructors for non-float values

Decimal class is supposed to be able to represent values outside the number type in JavaScript. However, the only public constructor takes coefficient and exponent as numbers, making it impossible to construct Decimal values where the coefficient does not fit in a JavaScript number.  There should be either string constructors, or BigNum support.
A current workaround is to use Decimal.parse and manually construct a string with both coefficient and exponent, delimited by d.

We decided not to make the longint (bigint wrapper) class part of the public api. Parse should fulfill your needs, we could make a parse from scientific notation function pretty easily.

I believe #354 gets to the heart of this issue, once we have a formal large integer as part of our API, we can expose that for the components of the Decimal API for both constructing values and extracting those properties with full fidelity.

Solved by #469 and #470.


[Bug] Build verification is failing

Is this a new bug in dbt-bigquery?

[X] I believe this is a new bug in dbt-bigquery
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Build verification is failing when running twine check dist/*. This is happening across all supported versions.
Expected Behavior
Build checks should pass.
Steps To Reproduce
https://github.com/dbt-labs/dbt-bigquery/actions/runs/9668100685
Relevant log output
Run twine check dist/*
  twine check dist/*
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.8.18/x64
    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.8.18/x64/lib/pkgconfig
    Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.8.18/x64
    Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.8.18/x64
    Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.8.18/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.8.18/x64/lib
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.8.18/x64/bin/twine", line 5, in <module>
    from twine.__main__ import main
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/twine/__init__.py", line 40, in <module>
    __uri__ = metadata["home-page"]
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/importlib_metadata/_adapters.py", line 54, in __getitem__
    raise KeyError(item)
KeyError: 'home-page'

Environment
- OS: ubuntu 22.04 LTS
- Python: 3.8.18
- dbt-core: N/A
- dbt-bigquery: 1.6 and later

Additional Context
The root cause seems to be https://github.com/pypa/twine/issues/1125.

Resolving as upstream dependencies unborked themselves


Can you create a new release?

Now that #19 is in, could you please tag a new release?

Done! https://github.com/dims/community-images/releases/tag/v0.2.0
Will need 15-20 mins for the artifacts to be uploaded and another 15-30 mins for krew index to be updated.


support TERM=dumb or something like it

I was hoping to get plain text output from starlark-repl, for when it is being invoked by another program (fuzz). This is what happens currently:
$ TERM=dumb starlark-repl -c "x"
thread 'main' panicked at 'failed to emit error: operation not supported by the terminal', /Users/josh/.cargo/registry/src/github.com-1ecc6299db9ec823/codemap-diagnostic-0.1.0/src/emitter.rs:881:27
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Fired off https://github.com/kevinmehall/codemap-diagnostic/pull/5 so that codemap-diagnostic does the right thing for TERM=dumb and ColorConfig::auto.
If it doesn't get merged soon-ish we can either use a git dependency on the fork, or do something similar in our code.


warm-up-r-spatial

test github issue comments on jekyll site...

test markdown

[ ] test...

...

is this operational now?


Use vis.js to create graph

Working as expected. [https://fredrkl.github.io/the-brother-karamazov-characters/](https://fredrkl.github.io/the-brother-karamazov-characters/)


Where are the settings stored?

Where does Terminus (both installed and portable) store the settings on a Windows platform?
It doesn't look like the portable Windows version has portable settings.

See https://github.com/Eugeny/terminus/wiki/Config-file
The portable version doesn't have a portable config file yet


Add release support through gorealeaser

We need to add support for goreleaser for the next release as it not only ease the release process but does provide various other features such as built-in CI tests which supports all the major CI platforms and various release format including, but not limited to binary, deb, rpm, etc.

@MuellerMH Do we really even need this? I find the current release process pretty apt.

For me, the current release process is perfect. In addition, I could not yet integrate the desired version into tk8 in the goreleaser. From my point of view this is not necessary.


getAccountSells() is not working

I am using this method and facing this problem.
Fatal error: Call to undefined method Coinbase\Wallet\Client::getAccountSells()
so, why to send the BTC Coin.

You need to pass an account object to this method, as shown in the README:
$account = $client->getPrimaryAccount(); // or some other account
$sells = $client->getAccountSells($account);


TimeFarm AutoStake issue after update

HI a big update recently and timefarm autostake not working right.
"STATUS: We still have 5 minutes left to wait - sleeping.
CHROME DRIVER DETACHED: It is now safe to exit the script.
Need to wait until 21 August - 05:55 before the next claim attempt. Approximately 5 minutes."
I't run  and wait 5 minutes to claim.

The Normal timefarm also got issue too.

Hi,
Yes looks like some bugs have crept in, a new version will be pushed shortly.
Thanks.

The new files have been uploaded, if you experience any further problems, please reopen the issue.


Emacs fails with libgccjit.so: error invoking gcc driver

Describe the bug
When installing and then running Emacs, Warnings are emitted in Emacs:
⛔ Warning (native-compiler): libgccjit.so: error: error invoking gcc driver
⛔ Warning (native-compiler): /nix/store/ny5j2ay81dzl931yzfk2wds3k83ygxn0-emacs-30.0.92/share/emacs/site-lisp/site-start.el: Error Internal native compiler error: "failed to compile", "/Users/jeremy/.emacs.d/eln-cache/30.0.92-802b27dc/site-start-f0b9874f-8f4312e0.eln", "error invoking gcc driver"

Prior to execution, there is no ~/.emacs.d, it is created on initial start. I ensured there was no ~/.config/emacs as well. Fresh install, simply emacs.
This report includes screen shots of the emacs30 package, but I have tried straight emacs and emacs29 packages with the exact same result.
This is a warning here and emacs will continue to function however as you begin to add packages or attempt to install tree sitter grammar files, the same error occurs and thus emacs is unusable the moment you need a tree sitter grammar file to provide editing capability for a particular language.
Steps To Reproduce
Steps to reproduce the behavior:

Add pkgs.emacs to configuration.nix

Expected behavior
I expect Emacs to start with no warnings
Screenshots

Additional context
This is on a macOS system using nix-darwin and an aarch64 (M3).
Metadata

system: "aarch64-darwin"
host os: Darwin 24.1.0, macOS 15.1.1
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.24.10
channels(root): "nixpkgs"
nixpkgs: /nix/store/7c2g8d7nzs52snmcf5fk5m4nwvnbkqqm-source

Notify maintainers

libgccjit maintainers: @Synthetica9, @vcunat, @Ericson2314
emacs maintainers: @AndersonTorres, @adisbladis, @jwiegley, @lovek323, @matthewbauer

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

I can't reproduce it on Linux, I've tagged it as a Darwin specific issue.

I can't reproduce it on Linux, I've tagged it as a Darwin specific issue.

Just as a note, when discussing the issue on discord, another on macOS user added emacs as a system package and received the same problem. So, very possible it is Darwin.

In my emacs 29 in my setup code so not a simple case.
the error is slightly clearer that it is a Darwin issue. with the -lSystem error.
From Async)native-compile-log
Compiling /Users/mark/.emacs.d/site-lisp/Third-party-modified/lispy-mnemonic.el...
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/mark/.emacs.d/site-lisp/Third-party-modified/lispy-mnemonic.el: Error: Internal native compiler error failed to compile
Compilation finished.

I too am able to reproduce this on Darwin with Nix release 24.11, using emacs-overlay and an M4
Compiling /Users/j/.emacs/fm30/.local/straight/build-30.0.92/inheritenv/inheritenv.el...
Warning (evil-collection): Make sure to set `evil-want-keybinding' to nil before loading evil or evil-collection.

See https://github.com/emacs-evil/evil-collection/issues/60 for more details.
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/inheritenv/inheritenv.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/inheritenv-4086f0b4-cde89279.eln", "error invoking gcc driver"
ld: library not found for -lSystem
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/evil-collection/modes/so-long/evil-collection-so-long.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/evil-collection-so-long-e0dc7c2b-74f4ddb9.eln", "error invoking gcc driver"
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/nix-mode/nix.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/nix-0f61d268-a1f916d0.eln", "error invoking gcc driver"
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/envrc/envrc.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/envrc-aa67a1ba-32efa594.eln", "error invoking gcc driver"
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/magit-section/magit-section.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/magit-section-cbff8b04-b828c58a.eln", "error invoking gcc driver"
Compilation finished.
Compiling /Users/j/.emacs/fm30/.local/straight/build-30.0.92/evil-nerd-commenter/evil-nerd-commenter.el...
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/evil-nerd-commenter/evil-nerd-commenter.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/evil-nerd-commenter-378efc80-57d2759d.eln", "error invoking gcc driver"
Compiling /Users/j/.emacs/fm30/.local/straight/build-30.0.92/smartparens/smartparens-html.el...
Compiling /Users/j/.emacs/fm30/.local/straight/build-30.0.92/evil-nerd-commenter/evil-nerd-commenter-sdk.el...
ld: library not found for -lSystem
ld: library not found for -lSystem
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/smartparens/smartparens-html.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/smartparens-html-1b1cbca5-f96d7805.eln", "error invoking gcc driver"
libgccjit.so: error: error invoking gcc driver
/Users/j/.emacs/fm30/.local/straight/build-30.0.92/evil-nerd-commenter/evil-nerd-commenter-sdk.el: Error Internal native compiler error: "failed to compile", "/Users/j/.emacs/fm30/.local/cache/eln/30.0.92-a3adebf0/evil-nerd-commenter-sdk-529dbf54-2e6052f7.eln", "error invoking gcc driver"

This issue is introduced in this staging cycle: https://github.com/NixOS/nixpkgs/pull/354201.
The commit before the merge of that PR does not have this issue while the merge commit of that PR does have this issue.
CC darwin experts @reckenrode @emilazy

Does it work if SDKROOT is exported from the Emacs wrapper ln Darwin?

Same problem on MacOS:

MacOS Version 14.6.1
M3 processer
Nix release 24.11
nix-darwin 24.11
just added the emacs package to home-manager

https://github.com/NixOS/nixpkgs/pull/361752 should fix the issue. Please test and let me know if it does not.


Warn message for data date range does not work properly

The message discussed in #17 is not working properly. Basically it does not work at all. Maybe somehow related to #108 which says that the data is missing for search results

fixed by #108


adding Austrian postcodes

All of the Austrian files include postcode as well in the PLZ field.

Thanks Al!


unable to connect my azure storage account using Microsoft azure storage explorer. Its keep asking Reauthenticate now

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.34.0
Regression From
no
Architecture
x64
Storage Explorer Build Number
No response
Platform
Windows
OS Version
Version	10.0.22631 Build 22631
Bug Description
unable to connect my azure storage account using Microsoft azure storage explorer. Its keep asking Reauthenticate now
Steps to Reproduce
unable to connect my azure storage account using Microsoft azure storage explorer. Its keep asking Reauthenticate now
Actual Experience
unable to connect my azure storage account using Microsoft azure storage explorer. Its keep asking Reauthenticate now
Expected Experience
unable to connect my azure storage account using Microsoft azure storage explorer. Its keep asking Reauthenticate now
Additional Context
unable to connect my azure storage account using Microsoft azure storage explorer. Its keep asking Reauthenticate now

Can you please provide more information, such as the precise error message you are receiving? Can you also provide the authentication logs (see https://learn.microsoft.com/en-us/troubleshoot/azure/azure-storage/blobs/alerts/storage-explorer-troubleshooting?tabs=Windows#authentication-logs)?

Closing due to inactivity. If this is still a problem, please respond with additional details as requested, and we can review.


Feat/early dns validation

Summary
Earlier checks for DNS TXT upon reaching FormSelect Page
Changes
Shifted DNS Check from onClick to useEffect
Issues
NIL

Closed based on discussions.
tl;dr: Running checks on all forms on load is a bit of an overkill, especially given that leeway (Pre-approvals) has to be given on slower or failing calls.
https://github.com/TradeTrust/document-creator-website/pull/228#discussion_r825542274


Normalize JSON property names

Adjust JSON serializer to use normalized camel cased property names (instead of original ones that may contain spaces)

@kulnor The name normalization is now applied early on, so it applies to all the output formats (including the new iRODS format) 7bbac49b1


Deprecate taxonomy sidebar component

Depends on https://github.com/alphagov/government-frontend/pull/507
Bring taxonomy links in line with related links, leaving a single
pattern that can be iterated in multivariate tests.
In time, uses of this component will be switched to the related links
component or a new alternative.

Coerce taxonomy sidebar into related links component
Use the “More” link to point to taxon pages
Use the “Elsewhere on GOV.UK” section to group taxon links without children
Don’t show taxon descriptions

https://trello.com/c/FN3ppmn2/60-update-taxonomy-related-links-in-right-column-to-match-the-design-used-by-mainstream-browse-related-links

Closing for now, as this might not be the right approach.


Node carries static peer data and transmits on new connections

This is a requirement that was discussed with @georgeee
We need to have application-specific, network-agnostic data about a peer, but it was determined that sending this information for every message would be too heavy. Another option was to have each node carry an application-specific identifier, and other nodes keep state associating those identifiers with this peer data, and I suppose having a way to demand peer data in case it's not known for that identifier.
The solution implemented here complicates the node quite a bit but it should be good. The first lightweight connection to a peer shall always be responsible for sending the peer data. Once the data is sent (send returns) other connections to that peer can begin sending data. The dispatcher therefore shall always know the peer data about every peer which has at least one connection to it before it performs a handshake or runs any application-specific handlers.
Also included in this changes is some metrics/statistics, and an abuse test case, from previous work.

CI passes, that's great :) I've also observed a flat heap in the ping-pong and discovery examples. This one needs very careful review, though. Lots of potential for concurrency-related bugs.


Gradient Boosted Trees prediction provides wrong result if algorithm and result datatypes do not match

Hi!
This is basically the same issue this one only with Gradient Boosted Trees.
Describe the bug
I want to use shallow copies of my own memory for my classification to avoid copying data. For the data input that works fine. To get the predicted results I create a prediction::ResultPtr and set a NumericalTablePtr with my own memory. If the daalDataType and the resultDataType are the same I get the expected results. If the resultDataType differs only the first element in my array is changed. The rest seems to stay the same.
I tried the same procedure with most all the other classifiers and there the results are correct if the datatypes mismatch.
To Reproduce
#include <daal.h>
#include <service.h>

int main() {
    const size_t nClasses = 2;

    typedef int dataType;
    typedef double daalDataType;

    // ## with a mismatching type (compared to daalDataType) only the first entry in vResultsMemory is changed
    typedef int resultDataType;
    // ## works as expected
    // typedef double resultDataType;

    /* creating arrays for shallow data access */
    dataType vData[8] = { 0, 0, 0, 1, 1, 0, 1, 1 };
    dataType vLabels[4] = { 0, 1, 1, 0 };

    dataType vTestData[8] = { 0, 1, 1, 0, 0, 0, 1, 1 };
    dataType vTestLabels[4] = { 1, 1, 0, 0 };

    resultDataType vResultsMemory[4] = { -2, -2, -2, -2 };

    const size_t vPrintLines = 4;

    using namespace daal;
    using namespace daal::algorithms;
    using namespace daal::data_management;
    using namespace daal::algorithms::gbt::classification;

    /* Create Numeric Tables for training data and labels with the data pointer of xtensor */
    NumericTablePtr vTrainData(HomogenNumericTable<dataType>::create(vData, 2, 4));
    NumericTablePtr vTrainGroundTruth(HomogenNumericTable<dataType>::create(vLabels, 1, 4));

    printNumericTables<dataType, dataType>(vTrainData, vTrainGroundTruth, "Train features:", "Train labels:", "Check data:", vPrintLines);

    /* Create an algorithm object to train the decision forest classification model */
    training::Batch<daalDataType> vTrainingBatch(nClasses);

    /* Pass a training data set and dependent values to the algorithm */
    vTrainingBatch.input.set(classifier::training::data, vTrainData);
    vTrainingBatch.input.set(classifier::training::labels, vTrainGroundTruth);

    /* Build the model */
    vTrainingBatch.compute();

    std::cout << "Training Complete" << std::endl;

    /**
     * Predict using the trained model
     */
    NumericTablePtr vNumTestData(HomogenNumericTable<dataType>::create(vTestData, 2, 4));
    NumericTablePtr vNumTestLabels(HomogenNumericTable<dataType>::create(vTestLabels, 1, 4));

    prediction::Batch<daalDataType> vPredictorBatch(nClasses);

    vPredictorBatch.input.set(classifier::prediction::model, vTrainingBatch.getResult()->get(classifier::training::model));
    /* Pass a testing data set and the trained model to the algorithm */
    vPredictorBatch.input.set(classifier::prediction::data, vNumTestData);

    // We want to get the result in c memory, so first we create a shallow NumericTablePtr
    NumericTablePtr vPredictedLabels(HomogenNumericTable<resultDataType>::create(vResultsMemory, 1, 4));
    // we create a new resultPtr
    classifier::prediction::ResultPtr vResultPtr(new classifier::prediction::interface2::Result());

    // and assign our shallow NumericTablePtr to the prediction
    vResultPtr->set(classifier::prediction::prediction, vPredictedLabels);
    vPredictorBatch.setResult(vResultPtr);

    /* Predict the values */
    vPredictorBatch.compute();

    /* Retrieve the algorithm results */
    printNumericTables<int, int>(vNumTestLabels, vPredictorBatch.getResult()->get(classifier::prediction::prediction),
                                 "Expected results", "Prediction results",
                                 "classification sample program results:", vPrintLines);

    // Thanks to our shallow copy we can directly access the results in vPredictedLabels
    for (size_t i = 0; i < 4; i++) {
        std::cout << vResultsMemory[i] << std::endl;
    }
}

Expected behavior
Results I get:
classification sample program results:
Expected resultsPrediction results
1              1
1              -2
0              -2
0              -2

Results I expect (result for matching datatypes):
classification sample program results:
Expected resultsPrediction results
1              1
1              1
0              1
0              1

Environment:

OS: [Ubuntu 18.04]
Compiler: [GCC7.4]
Version: [2020 Update 2]

Hello @Mightrider,
Thank you for your great attention to quality of our product! You was right, this issue has the same nature as the previous one. Fix for this bug has already been prepared and merged into master branch and we are going to merge it into rls/2021-gold-mnt soon. Please, feel free to try it with upcoming 2021.1-beta10 release and contact us if you have any concerns or questions.
Thank you for valuable contribution to oneDAL project!

Thanks a lot @KalyanovD for this immediate action and positive response! Can I ask, we are still using DAAL (planning to upgrade to oneDAL at some point, but not yet). Will the fix go into any DAAL 2020.x series too?

Hello @emmenlau,
Contribution to DAAL 2020.x line has already been finished, last update (2020 Update 3) will be available soon but this fix can't be included into this release. The closest release with this fix would be 2021.1-beta10, it is beta versions of upcoming oneDAL 2021.1 major release.
Sorry for inconvenience!

Thank your for your quick response @KalyanovD. I am happy to help.


Newly installed PowerShell version did not automatically become default profile

Windows Terminal version
1.21.1094.0
Windows build number
No response
Other Software
PowerShell 7
Steps to reproduce

Have a fresh machine (or similar config)
Install WT Canary
Notice that you have the standard profiles of Windows PowerShell, CMD, and ACS
Install PowerShell 7 via winget
Close and reopen WT Canary

Expected Behavior
We have code to automatically select PowerShell 7 as your default profile if it was just installed. That should happen.
Actual Behavior
PowerShell 7 is not the default profile. Windows PowerShell continues to be the default.

FWIW, we never change the user's default profile once they've installed Terminal. Why would we?

Oh. I thought we had code in the PowerShell dynamic profile generator to figure out which PowerShell profile we wanted the user to have and make that the default.

This only applies if pwsh is present at the time of first launch


Sort api functions in alphabetical order

See https://forum.rescript-lang.org/t/how-to-use-js-belt-set-belt-map-dict/432/7?u=ryyppy

This isn't 100% possible at least, if only because the items have a dependency on each other. But we can keep this in mind

It would be possible to sort them just in the docs? (not in the mli file). Reasoning is that the dependency doesn't matter for the consumer of the API. I think it's also more readable if submodules, values/functions and types are separate sections in the docs.

yes that's something i'd like to have soon. The only thing needed to make this work is adding the sorting logic in the extract–indices script ((here)(https://github.com/reason-association/rescript-lang.org/blob/master/scripts/extract-indices.js))
just a matter of opening your local docs, adjust the logic, run the script and check the docs again

Fixed with #161

Fixed with #161


mightFail.any's error type is inaccurate

I had to manually cast the error type to AggregateError here:
async function performSomethingWhileTryCheckForOtherStuff<T>(callback: () => Promise<Either<T>>): Promise<Either<T>> {
  const either: Either<T> = (await mightFail.any([
    callback(),
    new Promise((_, reject) => {
      return checkForNewRecordsAndMarkAsUpdated().finally(() => reject());
    }),
  ])) as Either<T>;

  if (either.error) {
    const aggregateError = either.error as AggregateError; // I shouldn't have to do this myself
    return Fail(aggregateError.errors[0]);
  }

  return Might(either.result as any); // Also, this is already addressed and I am encountering it again. https://github.com/might-fail/ts/pull/19#issuecomment-2451114060
}

And regarding the last comment, I would just get rid of the NotUndefined thing because it's creating additional complexity just for the sake of "semantics". It removes the flexibility of the users while enforcing a best practice in a flawed language called JavaScript.

does your last pr fix all the issues, or are you still manually casting that error type?

No this is a different issue. I was making a function that wrapped two promises in an mightFail.any and the typing were messed up.
It gave me a headache that's why I'm still procrastinating on fixing it

I will get back to this, I promise

@meech-ward I've added an API for passing custom Errors to the Either type to fix the issue of having to cast AggregateError to the error of mightFail.any. It seems like a core addition since it added a generic type to the Either types.
And, about the issue of having to cast the either variable in the second line:
 const either: Either<T> = (await mightFail.any([

I don't think it's something we can fix because the callback() causes extra complexity as it also returns Either. But, I'll think about it some more and maybe figure out something. But before that, I kind of wanted the current PR to be merged as I didn't want to change too much in one PR.
There are other changes that are irrelevant to this issue that are mentioned in the PR description itself.

PR done, hopefully it fixes all this?

Yeah, I think it should be fine. The main issue was the Error casting that was annoying.


[277] Support for custom html on slots

#cs: @santosguillamot
#supporter: @DAreRodz
https://frontity.pipedrive.com/deal/712
Site pageviews: (0.91)
Extra group sites: (0)
Extra group pageviews: (0)
Development size: (3)
Blocking: no frontity without it (x15)
Environment: integration is finishing (x10)
Reusability potential: broad (x3)
PRIORITY: [277]

We'll create a new package that saves the HTML in the database and uses dangerouslyHTML to inject it.

Use Custom Iframe as inspiration.

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Directory import 'PATH_TO_PRJECT/.output/server/node_modules/@heroicons/vue/outline' is not supported resolving ES modules imported from PATH_TO_PRJECT/.output/server/chunks/app/server.mjs

Hello,
I'm testing NuxtJS 3 with @heroicons/vue/outline.
(Version : "@heroicons/vue": "^1.0.5")
When I build the project to test it on firebase emulator (hosting, cloud functions)
I have the following error:
Directory import 'PATH_TO_PRJECT/.output/server/node_modules/@heroicons/vue/outline' is not supported resolving ES modules imported from PATH_TO_PRJECT/.output/server/chunks/app/server.mjs
> Did you mean to import @heroicons/vue/outline/index.js?
> at new NodeError (node:internal/errors:371:5)
> at finalizeResolution (node:internal/modules/esm/resolve:412:17)
> at moduleResolve (node:internal/modules/esm/resolve:932:10)
> at defaultResolve (node:internal/modules/esm/resolve:1044:11)
> at ESMLoader.resolve (node:internal/modules/esm/loader:422:30)
> at ESMLoader.getModuleJob (node:internal/modules/esm/loader:222:40)
> at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
> at link (node:internal/modules/esm/module_job:75:36)

I use this library in a .vue file as follows:
<template>
<div>
// code...
<MenuIcon class="h-6 w-6" aria-hidden="true" />
// code...
</div>
</template>

<script setup lang="ts">
import { MenuIcon, XIcon } from "@heroicons/vue/outline";
</script>

Did you have this problem ?
Thanks

Hey, this looks like a duplicate of #564. Closing in favor of that issue.
That said, it looks like a workaround is to append /index.js to the end of the import:
import { MenuIcon, XIcon } from<EMAIL_ADDRESS>


[PowerShell@2]: Task reports success if PowerShell.exe is killed externally

Required Information
Question, Bug, or Feature?
Type: Bug/Feature
Enter Task Name: Powershell@2
Environment

Server - Azure Pipelines

Agent - Private; v2.166.3 agent on Windows Server Core 10.0.19042

Issue Description
Background: our org has mandated a 3rd party antimalware product on all boxes & VMs, including our onprem build agents. Part of its policy enforcement targets certain uses of PowerShell; if it sees something it doesn't like, it kills the PowerShell process. Our dev team is not in a position to change the product usage or the antimalware policies, which are pretty broad and trap an undocumented list of (sometimes useful) functionality.
Where this affects the PowerShell@2 task is that if a script invokes something on the blacklist, the killing of the PowerShell process is detected by the agent as successful completion of the script step. This means we remain unaware that script actions were silently not performed, making any subsequent issues in our software take longer to investigate (particularly if this happens on a deployment environment rather than a build agent).
We could add Write-Host logging throughout our scripts to track whether they completed properly or not, but this would become a manual task to inspect rather than seeing the failure light up immediately. Given that the task first writes the PowerShell script to a temporary file before execution, perhaps it could add something (eg. a temporary file created before the ps1 executes and a completion marker written into it by the ps1 as the final step) to detect whether PowerShell.exe actually fully executed the script vs "abnormally" terminating?
Task logs
Added a Write-Host before the suspected problem lines and another at the end of our script to announce completion. Only the first reached the task log.
Ps-killed-log.zip
Troubleshooting
Error logs
Issue is that no error is logged and task reports success. Debug log shows externally killed PowerShell process returned exit code 0.

Hello @PST2,
Thank you for logging the ticket.
Unfortunately, I can't repro this issue. As I see the killing of processes started by task PowerShell@2 causes returning of exit code '-1' and failure of the task.
Could you please share YAML of the affected pipeline or a YAML with a minimal reproducible sample to allow me repro this on my side?

I suspect this one might be difficult to locally rep without having the same (3rd party) antimalware installed. This is enough to do it on our onprem agents:
- task: PowerShell@2
  displayName: Should Not Pass But Does
  inputs:
    targetType: inline
    script: |
      Write-Host 'Begin step...'
      $conn = new-object system.data.sqlclient.sqlconnection('server=<snip>;Database=<snip>;Integrated Security=true')
      $conn.Open()
      Write-Host '...End step.'

As the end of the diagnostics-enabled log shows, we never make it to the second Write-Host, PowerShell.exe reports exit code 0, and the step gets green-ticked despite not executing everything sent to it:
========================== Starting Command Output ===========================
##[debug]Entering Invoke-VstsTool.
##[debug] Arguments: '-NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'C:\<snip>\_work\_temp\668a788b-f63e-4870-9dbd-06121f5bf1e4.ps1'"'
##[debug] FileName: 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe'
##[debug] WorkingDirectory: 'C:\<snip>\_work\1\s'
"C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'C:\<snip>\_work\_temp\668a788b-f63e-4870-9dbd-06121f5bf1e4.ps1'"
Begin step...
##[debug]Exit code: 0
##[debug]Leaving Invoke-VstsTool.
##[debug]Leaving C:\<snip>\_work\_tasks\PowerShell_e213ff0f-5d5c-4791-802d-52ea3e7be1f1\2.190.0\powershell.ps1.
Finishing: Should Not Pass But Does

A slightly different way of illustrating what's going on here is replicating the behaviour by hand in a CMD session. The antimalware sees the attempt to make a DB connection from within PowerShell and somehow instantly kicks us back to the CMD prompt with a zero exit code for the PowerShell process:
D:\>echo %errorlevel%
0

D:\>powershell
Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.

Try the new cross-platform PowerShell https://aka.ms/pscore6

PS D:\> $conn = new-object system.data.sqlclient.sqlconnection('server=<snip>;database=<snip>;Integrated Security=true;')
PS D:\> $conn.open()

D:\>echo %errorlevel%
0

D:\>

@PST2 Thank you for providing me these details. Please provide the PowerShell version installed on used agent also. It can be got by command (Get-Host).version

Sure:
PS C:\> (Get-Host).Version

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      19041  1237

@PST2, thank you for providing the info.
Unfortunately, we are still not able to reproduce your issue.
Could you please confirm that CMD output above was got from the same agent? Did you see the same issue on other machines/agents?
How frequently has this issue occurs? Did you see any correlations between reproes of the issue and any additional factors (network quality, agent load, etc.)?

@PST2, thank you for providing info.
As I understand from all info about issue, it's not related to PowerShell task, it's related to compatibility issue for PowerShell shell tool and used antimalware suite. And adding of check for finish of PowerShell script won't solve this issue. All scripts with activities like described in the issue will fail after this possible fix (in accordance with info that this behavior happens 100% of the time).
So I would suggest to check configuration of used antimalware suite to resolve root cause of the issue. Also I think that you can ask support team of used antimalware suite about compatibility with PowerShell tool, it's strange behavior to block PowerShell script but allow this to finish with exit code 0.

Yes, I filed this issue after taking the matter up with IT on our side. Altering the blocking behaviour is a non-starter -- all requests to do so get rejected -- hence this request to detect such an occurrence.
Just to reiterate, this issue is not about the fact these steps get blocked; I agree that is outside the scope of the Task's ability to fix. Rather, it's that the step doesn't result in a failure when the script isn't executed all the way through due to the process being terminated in this manner -- in our opinion, this should fail the build job (as it would if one of the lines in the script itself threw an exception, or left lastexitcode nonzero at the end).
It would be enough for us to just get a ❌ and error message saying something like "Abnormal script termination" from the step when this happens.
A similar workaround would be for us to use a logging command at the end to set a pipeline variable and check the result in a subsequent step, but as with other detection workarounds we'd have to remember to add the required YAML/PS every time (with the resulting increase in job step counts as well).

@PST2 , root cause of this issue is strange behavior of PowerShell tool after blocking this by antimalware tool. I checked many possible cases with blocking or killing of PowerShell process by Stop-Process command, "kill process" feature in Task Manager, Windows Defender and Firewall tools. But each of the cases finished with returning of non-zero exit code. And all these cases are successfully handled by PowerShell task. I didn't find any case with killing PowerShell process and returing exit code 0.
So it's too hard to repro this issue correctly and it's better to fix root cause of the issue.

Awaiting vendor response.


add mysql port config

you can change mysql port ( 3306 by default )

Looks good for merge. Thank you @chocholand


feat: Allow opening a window with various repair features

This window contains various common repair tasks like disabling all but core mods, deleting FlightCore persistent store, etc.
Note that for this PR I purposefully only moved a single one of the currently existing repair buttons to the new view in order to keep the diff small.
The rest will be moved in a separate PR after this one has been merged.
The button to open the repair window will be moved from the DeveloperView to the settings page in a separate PR once the rest of the functions have been moved ^^
Rendered:

For some reason I cannot get the window to launch on a specific view >.<

For some reason I cannot get the window to launch on a specific view >.<

Turns out cause of the router it needs to be prefixed with /#, e.g. /#/changelog

Okay I cannot get the top nav bar to not be visible when launching new window. Trying to hid it with CSS in RepairView.vue via
<style scoped>
#fc_menu-bar {
    visibility: hidden !important;
}
</style>

and removing the scoped keyword just means it now affects all views and it's hidden everywhere...

This is how it looks currently

Naturally the menu bar shouldn't be in the new window that exists just for repair stuff.

Currently, repair window does not close when closing main window; I think it should. What are your thoughts about this?

Agreed. I didn't account for that scenario. Will investigate ^^

Closing all windows is now implemented in 20f1db85b838ad8cd4f920b571245bda2850246c


anvil misreports custom errors as no reason reverts

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
No response
Operating System
None
Describe the bug
Anvil should report "reverted with a custom error" when unable to pass a revert as a string. Instead it reports "without a reason string"

good point
the custom error decoding fn I quickly hacked together probably also needs some work
https://github.com/foundry-rs/foundry/blob/135ee3ec8befec3654507478f1e88b80e2b89c5c/evm/src/decode.rs#L222-L225

Would like to try this one, will add PR soon

oops got distracted by shitcoins, will get back on this asap


Invalid workflow file:  You have an error in your yaml syntax on line 77 (and line 70, same bit of code)

https://github.com/42Willow/upptime/actions
I'm not sure if I'm doing something wrong, but I'm getting an invalid syntax error for these lines:
https://github.com/upptime/upptime/blob/0a984cdfc1276d5d7a3e4003584a79f165969425/.github/workflows/setup.yml#L77
https://github.com/upptime/upptime/blob/0a984cdfc1276d5d7a3e4003584a79f165969425/.github/workflows/response-time.yml#L70

I'm seeing this same error on a brand deployment of this earlier this morning.

Yes, this is in a newline
https://github.com/upptime/upptime/blob/0a984cdfc1276d5d7a3e4003584a79f165969425/.github/workflows/setup.yml#L77

name: Install WARP to support IPv6
introduced in a recent change
https://github.com/upptime/upptime/commit/c1a25aff868599938bcbbdd5a4fdcc6bad38c7b9

@AnandChowdhary can we get a fix for this? this is a catastrophic bug as it prevents setup.

I just shipped a fix! :) Can you please run "Update Template CI" and once it's updated try running "Setup CI" and see if it works for you?


Links on windows, create shadow copys, not links

If you attempt to create a soft link on windows server 2012, you get a shadow copy, not a symlink.
link "c:\\foo" do
  to "d:\\bar"
end

Notice how if I right click on the file, it only gives me the option to 'restore previous version'

I did some digging here and could reproduce easily enough.  I'm concerned that there isn't much of a resolution available.  The file provider delegates all of the link creation logic directly to the win32 API. Obviously this could be fixed by, instead, utilizing mklink under the hood but my chef newbie senses tell me that's not looked highly upon

I only tested this when using links across drive letters.
Did this also happen when making a link on the same drive?
Unfortunately, I don't see a work around because you can't make a hard link across drives.

looks like the same issue is present when both files are on the same drive

I'm out of time for the night, but the last bit of testing I can apply here is to try to load up the win32 API in powershell or a c# exe and create a symlink with the same API call to rule out anything suspicious with the ruby win32 API hooks (I don't think this is likely, but worth investigating).
If that shows up negative and the issue is in the win32 api I'm not sure if there's much that can be done without calling out to cmd or powershell.

I'm not sure if this list is publicly taggable, but wth.
@chef/client-windows I've yet to test the win32 api implementation in ruby but I'd be shocked to find that the problem is there and not the way the win32 API treats symlinks, but the current soft-link implementation is pretty useless. The softlinks produced by the cmd shell mklink utility however work pretty much like you'd expect.
Once I validate that this is a win32 api limitation and not the chef code, the only real resolution would be to shell-out to mklink rather than utilize the win32 api just for softlink creation.  Would that even be considered? It seems like it could be a code smell at the least.

It'd be nice to figure out why the win32 api does not work. mklink should be calling those APIs. We should avoid shelling out if we can, but if that's the only way, it would be ok.
cc @chef/client-windows

Weird thing: when I create the link with chef, the target field is missing under the properties of the file. But the link still works

procmon does not seem to show mklink calling CreateSymbolicLink. There's a CreateFile operation and a FileSystemControl operation. It intertubes seem to say this means something about reparse points:
http://xxcopy.com/xxtb_050.htm

That's pretty interesting. Procmon shows a lot more activity for mklink /H.  The NTFS reparse points are considered hard links, so mklink is using classic hard links for soft links and I'm not quite sure what's up with the hard links.

functionally it looks like the hard link is just  a duplicate pointer to the same HDD address. (#CaptainObvious).  I'll try to slap together a pull request that does the soft link with the same API calls as mklink

It looks like the windows cookbook has a resource called windows_shortcut. Would be nice to have that merged into core.
https://github.com/opscode-cookbooks/windows/blob/master/README.md#windows_shortcut
Here is the resource
https://github.com/opscode-cookbooks/windows/blob/master/providers/shortcut.rb

The link resource is doing the right thing.  For local file system links between files, a symlink is created, for links to directories or across volumes, a junction is created.  How Explorer treats them is a UI idiosyncracy and they are different from shortcuts.


command line tool did not work at all

Using mac with mavericks.  no problems using any other gems

What is your setup? What have you tried? Are there any stack trace errors? Try bundle exec octopress deploy --trace and post what you're getting. I need some more information to help out here.

I can't help you if this is all you can tell me.


abnormal meiotic telomere clustering

Consider adding:
abnormal meiotic telomere clustering
DEF: any anomaly in the cell cycle process by which the dynamic reorganization of telomeres occurs in early meiotic prophase, during which meiotic chromosome ends are gathered in a bouquet arrangement at the inner surface of the nuclear envelope proximal to the spindle pole body; this process plays an important role in progression through meiosis and precedes synapsis
SYN: abnormal meiotic bouquet biosynthesis
SYN: abnormal meiotic bouquet formation
J:122602
Definition based on GO:0045141
Anna

Added:
abnormal meiotic telomere clustering MP:0030023
Anna


Layer doesn't check validity of scissor rectangles for vkCmdSetScissor

Section 25.2 of the spec says the following about scissor rectangles:

The x and y members of offset must be greater than or equal to 0
Evaluation of (offset.x + extent.width) must not cause a signed integer addition overflow
Evaluation of (offset.y + extent.height) must not cause a signed integer addition overflow

parameter_validation_vkCmdSetScissor could do with checking these. Particularly the first of the three, which can cause some peculiar behaviour on some tiling hardware.

Should do viewport at the same time.

partially fixed by 7572b6f915a6b3f8b5992a42d04ad20788c44b1d

SetViewport part is fixed by 79a99540d19bd9a809eed9af86463eabe79261eb


Just need a little help with something...

So I'm trying to implement some new features that I found in the excellent update recently.
I'm trying to implement the adding of a new theme using the 'love' on as a test. Bellow is my complete code for the ResourceDictionary. I've tried it like this <ImageBrush Opacity=".1" ImageSource="/Assets/background.love.jpg" Stretch="UniformToFill" />
and like this <ImageBrush Opacity=".1" ImageSource="/CTaCInformationSystem;component/Assets/background.love.jpg" Stretch="UniformToFill" />
both give me this error: Provide value on 'System.Windows.Baml2006.TypeConverterMarkupExtension during the AppearanceManager.Current.ThemeSource = value.Source; method that is called on line 148 of the AppereanceViewModel.cs.  The error put's it right at the Opacity='.1' in the code below.
The images is located in the Assets/background.love.jpg section in the source code.
Any ideas on how to fix this?
<ResourceDictionary xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
                    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
                    xmlns:mui="http://firstfloorsoftware.com/ModernUI">

    <ResourceDictionary.MergedDictionaries>
        <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.Dark.xaml" />
    </ResourceDictionary.MergedDictionaries>

    <Color x:Key="AccentColor">#e51400</Color>

    <!-- background image taken from http://good-wallpapers.com/ -->
    <Rectangle x:Key="WindowBackgroundContent" x:Shared="false">
        <Rectangle.Fill>
            <ImageBrush Opacity=".1" ImageSource="/CTaCInformationSystem;component/Assets/background.love.jpg" Stretch="UniformToFill" />
        </Rectangle.Fill>
    </Rectangle>

</ResourceDictionary>```

Found the issue. Needed to set the property of the image to Copy Always in the output directory for the Properties. Wow....that's what you get for self teaching c# :package:

Got in the same issue more than once when making themes >_<

Thanks for sharing the solution


Delete-commit doesn't clean up stats commit correctly

What happened?:
Running normal openCV demo with stats:true for edges pipeline. Deleting commits seems to have left some hanging commits (probably stats commits) that have provenance that no longer exists.
REPO   ID                               PARENT STARTED           DURATION           SIZE
images bb43143a866a41489d1851dfa3d7e428 <none> About an hour ago Less than a second 714.2KiB
JosephZckersMBP:demo joeyzwicker$ pc list-commit edges
REPO  ID                               PARENT                           STARTED           DURATION  SIZE
edges 2b4391dd71e6468ea2aa8e0d99aab860 <none>                           About an hour ago 1 second  390.3KiB
edges a84e2a21c07642edb9edc87893ddb3a2 <none>                           About an hour ago 1 second  1.089MiB
edges 8ebfbcf2cdf642c5a6d04bf37623827a <none>                           About an hour ago -         -
edges 58bb97ce81814878b406192a1892d608 5a62771d2f204a9786d869f0c6c9e83d About an hour ago -         -
edges 389d1acdb61845e38906da79b5b0c61f <none>                           About an hour ago 4 seconds 390.3KiB
edges 5a62771d2f204a9786d869f0c6c9e83d <none>                           About an hour ago 4 seconds 1.091MiB
JosephZckersMBP:demo joeyzwicker$ pc inspect-commit edges 8ebfbcf2cdf642c5a6d04bf37623827a --raw
{
  "commit": {
    "repo": {
      "name": "edges"
    },
    "id": "8ebfbcf2cdf642c5a6d04bf37623827a"
  },
  "started": "2018-10-16T00:01:17.628980417Z",
  "provenance": [
    {
      "repo": {
        "name": "edges"
      },
      "id": "118e2ff3cb14437fb9a4870b53dabba5"
    }
  ],
  "branchProvenance": [
    {
      "repo": {
        "name": "edges"
      },
      "name": "master"
    }
  ]

There's only 1 commit in images, because I added a bunch of commits, called update-pipeline -f edges a few times (w/ and w/o reprocess), and then started deleting commits from images. There shouldn't be so many commits in edges. Some of them reference other edges commits that no longer exist and are missing a tree, but the commit-info still exists.
What you expected to happen?:
All stats commits to be properly deleted when their provence is deleted.
How to reproduce it (as minimally and precisely as possible)?:
I don't have 100% nailed down repro steps yet, but something along these lines:

set up opencv demo and create multiple input commits. Make sure stats is turned on for edges.
Call update pipeline (w/ and w/o reprocess, i'm not sure if that makes a difference) on edges.
add a few more input commits
start deleting commits from images, especially not the HEAD commit (e.g. delete-commit images master^3)

Anything else we need to know?:
Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI version (use pachctl version):
pachyderm version 1.7.8
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others:

@JoeyZwicker I believe this has been fixed by #3585. My suspicion is that by improperly computing provenance in the subset in the propagateCommit step this enabled commits to hang around after commits they were provenant on had been deleted.

Let me know if there is anything else you'd like to try or would me to verify before closing this issue out

If you were able to repro initially and couldn't any longer after the fix, feel free to close. I can test it further when it get released

Yup, repro'd before and after, closing.


Chapter 12: multi output custom layer example

Description:
Adding multi-output custom layer example in chapter 12.
Note: To minimize the code changes, I have executed only the cells required for this PR, so cellblocks numbers in the notebook are not in the order

Thanks for the PR! 👍


Integrate New OneSearch with LibGuides

(Two weeks) before go-live, we would like to have all content in LibGuides that currently integrate with OneSearch to be switched to the new OneSearch.

[ ] Convert and test all scoped searches on test site.

[ ] Create a list of Summon assets/links that need to be redirected to EDS. (Summon links include "summon", "go.exlibris", "rp8jq9jy4s")

[ ] Update all scoped searches and assets.

[ ] Figure out workflows/recommendations for creating links to resources and scoped searches in the New OneSearch

We also need to investigate permanent linking options. See this EBSCO's link configuration article for more details

also see SES #7

LibGuides is now loaded as a separate collection.


fix: input boxes in dark theme

This PR adds the new variables for input boxes introduced in Home Assistant 2022.3.
This closes https://github.com/Kibibit/hass-kibibit-theme/issues/121 https://github.com/Kibibit/hass-kibibit-theme/issues/120 and https://github.com/Kibibit/hass-kibibit-theme/issues/119

Edited the title to satisfy semantic pull requests

I used VS Code to update the kibibit yaml in my HA; the fix works, thanks @diademiemi


Windows stuck at boot

When I install the new beta, Windows won't start anymore and gets stuck in the boot screen.
I could recover from that by going into Safe Mode (with networking support) and uninstall OpenVPN.
I'm blaming the DCO driver, as that appears to be the only new and related thing.
Also the ntbtlog.txt (Boot Log) ends with BOOTLOG_LOADED \SystemRoot\System32\drivers\ovpn-dco.sys

OpenVPN version: 2.6_beta1
OS: Windows 10 22H2 Build 19045.2311 (publicly available "Update Preview")
Possible conflicting drivers: a whole bunch that accumulated during the last 3 years as a sysadmin... WSL stuff, npcap, wireguard, cisco vpn, forticlient vpn, ...

Thanks for reporting. I have seen dco driver caused BSODs, but not this kind of behavior. I assume it works fine if you deselect dco driver in installer dialog? Could you by any chance boot with kernel debugger attached and see if there is anything interesting in logs?

Hi, I have same issue and lost two full days until I realised what the cause was,  did you resolve this?

No, we are unable to reproduce this unfortunately. Can you describe what is happening? Do you get any error message?

Can you post your boot log? You may need to enable it first in Start -> Run -> msconfig -> Boot -> Bootlog.

I also came across this issue whilst upgrading OpenVPN from 2.5.8 to 2.6.0, wondering why my laptop spent 10 or so minutes in the manufacturer logo when usually not even a minute were needed to boot to Windows. Also can confirm when ovpn-dco.sys is not loaded, laptop booted fine.
Here's what i did:

Install openvpn 2.6.0 with ovpn-dco.sys selected / enabled it through Autoruns, also enable bootlog.
Reboot to see laptop stuck at manufacturer logo, wait for about 5 minutes, then force shutdown by holding power button.
Repeat step 2 until i boot to WinRE, boot to safe mode (basically this)
Whilst in safe mode, disable ovpn-dco.sys using Autoruns.
Reboot and see that the laptop successfully boot to Windows. Disable bootlog and copy ntbtlog.txt somewhere safe.

Unfortunately i cannot found BOOTLOG_LOADED \SystemRoot\System32\drivers\ovpn-dco.sys line in it like OP, but here it is anyway. Windows 10 version is 22H2 Build 19045.2486.

@ArinoJenynof thank you.
Do you happen to have anything special installed on your laptop? Special in terms of drivers / hardware. Could you please post an output of systeminfo?

Just a typical entry-level gaming laptop, i think (Acer AN515-54). As far as i know there is nothing special in the configuration, maybe Nvidia Optimus, but that's common in laptop like this. I do notice an "Airplane Mode Driver" that came from Acer themselves, instead of the big three (Intel, Nvidia, AMD) vendors. Here's systeminfo.txt

@ArinoJenynof Good news, my colleague was able to reproduce this exact issue on the same laptop. Now I just need to get this model and fix it :)

@Tabiskabis @bunglex which hardware do you use? Is this also Acer laptop?

@Tabiskabis @bunglex which hardware do you use? Is this also Acer laptop?
 Features required for Hyper-V will not be displayed.

Another list of problematic models:
Model: XPS 15 9500, Windows Version: 10.0.19045.2486
Model: XPS 13 7390, Windows Version: 10.0.19045.2486
Model: XPS 13 9305, Windows Version: 10.0.19045.2486
Model: XPS 13 7390, Windows Version : 10.0.19045.2364

Note that on another XPS 7390 everything works just fine.

Dell Latiude 7400 Win10 different builds also affected

Since I am unable to reproduce this on our machine, I would like to ask for assistance in figuring out the cause. I have prepared a new version of the driver with enhanced logging which hopefully sheds some light on what is happening.
ovpn-dco-0.8.4.zip

Extract this archive and run install.bat from Administrator command prompt. This will install (or update) the driver to the version with improved logging. This will also enable driver logging at boot phase (AutoLogger session)
Reboot and experience the problem (manufacturer logo and spinning circle, boot not progressing)
Reboot in Safe Mode
You should have driver logs in C:\Windows\System32\LogFiles\WMI\ovpn-dco.etl.001 (002, 003 etc after each reboot)
Paste those files here. They do not contain any private info, just ovpn-dco driver logs. If you have no network in Safe Mode, disable ovpn-dco driver via Device Manager and reboot (you might want to disable the driver in any case)

@lstipakov, will devcon remove ovpn-dco.inf suffice to get rid of the driver again?

@lstipakov, will devcon remove ovpn-dco.inf suffice to get rid of the driver again?

Should be suffice, yes.

@lstipakov ovpn-dco.etl.001.zip

@Tabiskabis thanks. Unfortunately the driver log is empty. Just to be sure - is this the log from problematic boot?
I found something interesting in your ntbtlog -
12  5 2022 22:12:40.500
BOOTLOG_LOADED \SystemRoot\system32\ntoskrnl.exe
...
BOOTLOG_LOADED \SystemRoot\System32\drivers\ovpn-dco.sys

Looks like Windows managed to load DCO driver in last May. Have you used some beta client at that time?

also @Tabiskabis can you look into Event Log and see if there is anything about dco driver?

and do you happen to have ovpn-dco.etl?

Traces from problematic machine:
ovpn-dco.zip
There seem to be no warnings/errors.

DCO in May? I can't remember, but you're probably right, I must have been trying a beta, then.
No ovpn-dco.etl, but ovpn-dco.etl.002.zip from a later boot.
Event viewer is full of errors complaining about services that could not be startet. Last log before these start is The vmms service terminated with the following error: Ran out of memory

Close when I004 is out.

@Tabiskabis @Gulaschcowboy @ArinoJenynof This bug should be fixed in upcoming release. Could you give this build a try? https://github.com/OpenVPN/openvpn-build/suites/10769949141/artifacts/541018299

Will test the 3550 on monday, but the new version is looking good on a Precision 3560. Except the connected interface always shows received packets: 0 - is that normal?
Meanwhile, i've tested a Precision 3560 with the same issue. Removed as many drivers as possible, then installed ovpn-dco and rebooted. It worked. Windows Update reinstalled most of them and the next reboot failed.
Then tried a binary search (removing fewer and fewer drivers). In the end i thought i had nailed it down to either Intel Thunderbolt, Management Engine, or Wireless Manageability drivers. But either it's a certain driver combination, or the issue is a heisenbug.

This bug should be fixed in upcoming release. Could you give this build a try? https://github.com/OpenVPN/openvpn-build/suites/10769949141/artifacts/541018299

Thanks, that build works. No more hanging on my laptop.

Thanks, I assume you are (still) able to connect with DCO driver? Lähetetty iPhonestaArino Jenynof @.***> kirjoitti 5.2.2023 kello 17.53:
This bug should be fixed in upcoming release. Could you give this build a try? https://github.com/OpenVPN/openvpn-build/suites/10769949141/artifacts/541018299
Thanks, that build works. No more hanging on my laptop.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

Yes, also tried that. No issues connecting.

This bug should be fixed in upcoming release. Could you give this build a try? https://github.com/OpenVPN/openvpn-build/suites/10769949141/artifacts/541018299

Thanks, that build works. No more hanging on my laptop.

Do you mind resharing the link? It doesn't seem to work anymore.

Hi,
Posted on forum:
https://forums.openvpn.net/viewtopic.php?p=110487
;)

And here is direct link - https://build.openvpn.net/downloads/releases/OpenVPN-2.6.0-I004-amd64.msi

Thank you all. Can confirm that it is working.

Fixed in 2.6.0-I004.


Not resolving to less specific language

Scenario:
const acceptLanguage = require('accept-language');
acceptLanguage.languages([ 'en',  'ja',  'zh',  'zh-TW',  'zh-HK',  'zh-CN',  'zh-MO',  'zh-HanS',  'zh-HanT',  'ko', 'fr' ]);
acceptLanguage.get('zh-HanS-TW'); // returns 'en', expected 'zh-HanS'
acceptLanguage.get('zh-SG'); // returns 'en', expected 'zh'
acceptLanguage.get('fr-BE'); // also returns 'en', expected 'fr'

As you can see in this example accept language doesn't fallback to the closest match. I'm not sure if this is a bug or the intended way of working. From your tests it appears you support the other way around - less specific get returns the closest defined language.

I agree.
If we propose ['en', 'fr'] in our list, 'fr-BE' or 'fr-FR' should match with 'fr' and not default to 'en'.

hm, I probably misinterpretted RFC 4647 https://tools.ietf.org/html/rfc4647#section-3.4.
I thought, that it shouldn't match a more specific fr-BE with a more broader one fr. I have done the opposite. If you support zh-CN and the user requests zhit will return zh-CN, but not the other way around.
But after having read through the docs. It seems I've done wrong. I will read it more carefully later and maybe correct this.

@tinganho any feedback on this yet?

@micky2be Sorry for the delay. I will try to take a closer look in the very close future :)
But it seems like we should only revert the matching algo.

This should be now fixed and following RFC 4647 https://tools.ietf.org/html/rfc4647#section-3.4.


Set TicketDataFormat, or else it doesn't work with Identity

Workaround https://github.com/aspnet/Identity/issues/1321

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

@HaoK Can you please review this one?

@HaoK  please review

@fujiy  Thanks for the fix. Security updates are doubly appreciated.


Store all transport messages.

Add the ability to easily store all transport messages as they leave or enter the transport.

This needs #205 to land first and is going to be difficult to get done by 0.4 but it's important so it's staying in the v0.4 milestone for now.


[LLVMGPU] Convert maximumf/minimumf to max/min for ROCM

AMDGPU does not support the former

@dcaballe FYI for you regarding the changes upstream for Nan propagation.... Seems like AMDGPU have issues as well.

@kuhar @MaheshRavishankar I dont have write access so please merge this when you get the chance

@nirvedhmeshram done


Change in TableId behaviour for non default BigQueryClient project 1.44 -> 1.45

When using the getTable method of the BigQuery client the project of the TableId is overwritten. TableId would previously ignore this if a project had been set. Table Id has been changed such that the project is always overwritten. There is now no way to get a table for a project other than the one the client was created for.
TableId tableId = TableId.of(project, schema, tableName);
Table table = client.getTable(tableId);
// table returns null if project != client.getOptions().getProjectId()

this is due to the change in TableId
In BigQueryImpl:
public Table getTable(TableId tableId, TableOption... options) {
    final TableId completeTableId = tableId.setProjectId(getOptions().getProjectId());
    ...

TableId old code:
TableId setProjectId(String projectId) {
  return getProject() != null ? this : TableId.of(projectId, getDataset(), getTable());
}

TableId new code:
TableId setProjectId(String projectId) {
  Preconditions
    .checkArgument(!Strings.isNullOrEmpty(projectId), "Provided projectId is null or empty");
  return TableId.of(projectId, getDataset(), getTable());
}

@MMMarcy that would work for us. It is more suitable to have this logic in BigQueryImpl.
Thanks

Perfect. Waiting for the owners thoughts now. If positive, I can open a PR

Actually, thinking more about this using the proposed solution I mentioned hides too much of the behavior to the caller. Maybe overloading the getTable method with
public Table getTable(TableId tableId, ProjectId projectId, TableOption... options) {
...
}

seems a better alternative.

TableId already has a factory method including the project:
public static TableId of(String project, String dataset, String table)
I would suggest that if the project is set in TableId, the developer knows the project he wants.

Unless I've missed something obvious folks, this issue should be reopened. The PR only fixed the getTable() method, but the same bug exists for other methods like delete(TableId tableId) and update(TableInfo tableInfo). Other methods like create() and update() also appear to have the same issue.

The TableId changes also effect all code where TableInfo.setProjectId is referenced too.

@MMMarcy Can this issue be reopened and all occurrences be fixed or the original change reverted?

@JesseLovelace PTAL

I think there's way more callsites of setProjectId that aren't fixed - including insertAll that are still broken. IntelliJ shows 15 usages of which 3 are fixed now.

@pmakani, it looks like there are more places that need to change.

@pmakani Did #4196 actually fix all the cases?

@chingor13 I was about to ask the same. He's all the references I see to setProject() in my IDEA..

Hi, we believe this is still broken as of today (1.65.0). As in the description of this ticket, still There is now no way to get a table for a project other than the one the client was created for.
Specifically, in https://github.com/googleapis/google-cloud-java/blob/master/google-cloud-clients/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/BigQueryImpl.java#L635-L639 , the project in my tableId will always be overridden with the project of the service, if the service has a project.
Maybe the intention of that code was to override the project only if it is not set in tableId? Then the code would look like
          tableId.setProjectId(
              Strings.isNullOrEmpty(tableId.getProjectId())
                  ? serviceOptions.getProjectId());
                  : tableId.getProject()

Or maybe, the confusion arose between TableId and TableInfo. The code above calls TableId::setProject, which will always perform the change. See: https://github.com/googleapis/google-cloud-java/blob/master/google-cloud-clients/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableId.java#L100
Thanks!


Add cable trace button to front/rear ports

Proposed Functionality
View trace on patchcable between patchpanels.
Use Case
When patching through multiple patch panels it is tricky to view the end device using the patch. Would it be possible to add a link to the trace of the entire path in the cable view?

@jeremystretch thanks for implementing!


Feature Request: Banned from Flea, Penetration, Damage, Armor Damage?

Hey Refringe,
Any way we could get the ability to see things like whether or not the item(s) are banned from flea, the penetration, damage, and armor damage stats of ammo, and those sorts of things into the ID's?
item.props..penetrationPower; .armorDamage; .Damage; etc

needed DMG and Pen stats for ammo myself for a project and made a pull request with a new language file that includes this values and a script that grabs them from the SPT ID Finder API to easily update them.
--> #5


Error while installing Widevine update

This is a repost from https://github.com/Sandmann79/xbmc/issues/156 as I was told the error is related to this module
I tried to play videos using the Amazon VOD Kodi plugin, but I always get the message that a Widevine update is required. However, after accepting the EULA, the update crashes with the following error message:
20:19:11.561 T:1590670192 WARNING: CPythonInvoker(16, /storage/.kodi/addons/plugin.video.amazon-test/default.py): the python script "/storage/.kodi/addons/plugin.video.amazon-test/default.py" has left several classes in memory that we couldn't clean up. The classes include: N9XBMCAddon7xbmcgui6DialogE,N9XBMCAddon9xbmcaddon5AddonE
20:19:11.572 T:1939529744   ERROR: Playlist Player: skipping unplayable item: 0, path [plugin://plugin.video.amazon-test/?asin=B015NZ83EG&mode=PlayVideo&adult=0&name=1+-+Pilot&trailer=0&selbitrate=0]
20:20:10.319 T:1590670192   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.OSError'>
                                            Error Contents: [Errno 12] Cannot allocate memory
                                            Traceback (most recent call last):
                                              File "/storage/.kodi/addons/plugin.video.amazon-test/default.py", line 5, in <module>
                                                EntryPoint()
                                              File "/storage/.kodi/addons/plugin.video.amazon-test/resources/lib/startup.py", line 68, in EntryPoint
                                                PlayVideo(args.get('name', ''), args.get('asin'), args.get('adult', '0'), int(args.get('trailer', '0')), int(args.get('selbitrate', '0')))
                                              File "/storage/.kodi/addons/plugin.video.amazon-test/resources/lib/playback.py", line 579, in PlayVideo
                                                playable = _IStreamPlayback(asin, name, trailer, isAdult, extern)
                                              File "/storage/.kodi/addons/plugin.video.amazon-test/resources/lib/playback.py", line 459, in _IStreamPlayback
                                                if not inputstream_helper.check_inputstream():
                                              File "/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper.py", line 763, in check_inputstream
                                                return self._check_drm()
                                              File "/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper.py", line 741, in _check_drm
                                                return self._check_widevine()
                                              File "/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper.py", line 655, in _check_widevine
                                                return self._install_widevine()
                                              File "/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper.py", line 542, in _install_widevine
                                                return self._install_widevine_arm()
                                              File "/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper.py", line 485, in _install_widevine_arm
                                                if not self._cmd_exists('fdisk') and not self._cmd_exists('parted'):
                                              File "/storage/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper.py", line 164, in _cmd_exists
                                                return subprocess.call('type ' + cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE) == 0
                                              File "/usr/lib/python2.7/subprocess.py", line 172, in call
                                              File "/usr/lib/python2.7/subprocess.py", line 394, in __init__
                                              File "/usr/lib/python2.7/subprocess.py", line 938, in _execute_child
                                            OSError: [Errno 12] Cannot allocate memory
                                            -->End of Python script error report<--
20:20:10.486 T:1590670192 WARNING: CPythonInvoker(17, /storage/.kodi/addons/plugin.video.amazon-test/default.py): the python script "/storage/.kodi/addons/plugin.video.amazon-test/default.py" has left several classes in memory that we couldn't clean up. The classes include: N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon7xbmcgui6DialogE
20:20:10.499 T:1939529744   ERROR: Playlist Player: skipping unplayable item: 0, path [plugin://plugin.video.amazon-test/?asin=B015X5GVZO&mode=PlayVideo&adult=0&name=2+-+A+Stray+Howl&trailer=0&selbitrate=0]

I'm using LibreElec 9 on a RPi 3 if that helps somehow.

It took me a while to get back to this, but your comment helped.
The manual update of inputstreamhelper to the current master did the trick. Now it was able to download the image and is currently in the process of installing the update.

Aaand its working. Thanks a lot!


NVS: add support for i16, u64, i64 types to NVS partition generation tool (IDFGH-2227)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@bbinet Thanks for the contribution.

up?

Hi @bbinet, sorry for the delay. The commits have been cherry-picked into master, and should show up on Github soon. Will update once again when this happens.

Thank you @igrr

Hi @bbinet
The changes are up on Github.
https://github.com/espressif/esp-idf/commit/cab895f45996e312e7594fcd8bf47731c29336c3#diff-cb3dd88d88d6fd7bc7482a58b07995eb
https://github.com/espressif/esp-idf/commit/70840158844131c1dd2afa5b5ec3361ea85ca0c0#diff-cb3dd88d88d6fd7bc7482a58b07995eb
Thanks

@bbinet Thanks for contribution.


Feature: add cni-plugin to kubernetes rootfs

What is the problem this feature will solve?
this app need cni-plugin

multus-cni
flannel
kube-router
calico

we can use sealos scp copy cni-plugin binary to all nodes during cluster install, but when run sealos add nodes this action not run again.
If you have solution，please describe it
download from
https://github.com/containernetworking/plugins/releases
What alternatives have you considered?
The official kubeadm based installation method will install cni-plugin to the/opt/cni/bin directory by default.:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/install-kubeadm/
apt install -y kubeadm

There are many places where cni-plugin needs to be used. Should it be consistent with the official default behavior
root@ubuntu:~# ls /opt/cni/bin/
bandwidth  calico       dhcp   firewall  host-device  install  loopback  portmap  sbr     tuning  vrf
bridge     calico-ipam  dummy  flannel   host-local   ipvlan   macvlan   ptp      static  vlan
root@ubuntu:~#

Try using patch type cloudimage

https://github.com/labring-actions/cluster-image/blob/42f1523a2cfca8348b96ec8fc8ad28d5cee1d5fc/applications/flannel/v0.19.0/manifests/kube-flannel.yml#L136
It may need to be modified separately, or a separate install-cni ds

https://github.com/labring-actions/cluster-image/issues/295
using this image install cni plugins


Selection options

What changes are proposed in this pull request?
A new menu that allows some basic operations to be performed from the UI

Clearing is new (and could only be done previously by manually unselecting everything). The other two can also be done by creating stages from Python. Note that they are implemented by creating view stages for simplicity.
Closes #391
Related: #113
How is this patch tested? If it is not, please explain why.
Local testing
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[X] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

Added the ability to clear, select, and exclude the selected samples from the app
What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

A couple other comments related to Select/Exclude fields that would be nice to address before advertising a feature that enables easy creation of Select/Exclude stages:

In the App backlog, there is a feature request to adopt a more compact rendering strategy for Select and Exclude, which can be very long. My proposal is to render <first-few-charcters>... when it gets too long (maybe always, even when one sample is selected, since the IDs are not really meant for visual consumption). When the user clicks into the stage, the full text would be expanded

There seems to be a problem with creating Select/Exclude fields with > 1 ID in them. The intention is to accept input via list,of,ids rather than JSON-style ["list", "of", "ids"] (which is how these stages used to accept input, I believe). When a stage is created programmatically, one sees list,of,ids in the view bar, but one cannot actually input multiple IDs in that way (or in JSON-style input)

Let's not advertise a feature that will cause view bar to look like this :)

(1) When selection/exclusion is applied, you need to refresh more of the App. The display options stats and Viewing XXX samples aren't updated, for example. I did a session.refresh() manually to get those updated

The relevant routes are using a decorator that is very similar to _update_state (what refresh() uses), and is certainly intended to do the same thing as refresh(): https://github.com/voxel51/fiftyone/pull/509/files#diff-72c8f530eb29b913a6aa6dfaf40522bcR275
I would need someone more familiar with the backend to help figure out why this isn't updating stats.

After doing this, the two samples that I had originally selected and then deselected became selected again. Expected behavior is that, since they were deselected prior to deleting the Select stage, they should remain deselected afterwards.

As far as I can tell, this is an issue with the view bar, which caches the entire session state (including selection).

Let's not advertise a feature that will cause view bar to look like this :)

The functionality itself is useful (and as mentioned above, the ability to clear selections is new), so I think that this is worth mentioning regardless of the view bar issues that you pointed out in https://github.com/voxel51/fiftyone/pull/509#issuecomment-688954004. I will look into addressing these, although it's worth noting that they were present and known before this PR (and could have been reproduced by creating a large Select stage from Python, for instance). There are several details that would need to be addressed with those, and I don't feel like most of them should be addressed in this PR. For instance:

My proposal is to render ... when it gets too long (maybe always, even when one sample is selected, since the IDs are not really meant for visual consumption).

This would involve special behavior for certain view stages - I'm assuming you don't want long Limit stages truncated like this, for instance. I don't know if view stages currently support special rendering behaviors based on their types.

@lethosor I hear you. Some of my comments aren't directly related to this PR, but they are directly related to the feature that we're going to announce

(1) When selection/exclusion is applied, you need to refresh more of the App. The display options stats and Viewing XXX samples aren't updated, for example. I did a session.refresh() manually to get those updated

I adopted the approach used by the view bar to fix this. This required discarding the selected samples when applying a selection, though (i.e. you'll have to select them again to perform another operation). I think this is fine because selection and exclusion can be used interchangeably - if you have 9 samples, deselecting one and Selecting the remaining 8 produces the same result as selecting one and Excludeing it.

One final suggestion on design: for the "0 samples selected" state, how about just rendering that in plain text with no disabled dropdown around it? My reasoning is that this would match visually with the "Viewing XXX samples" on the right, and it avoids rendering a dropdown that is not actually clickable. I currently see a "not active" cursor, which is quite distracting when I hover over it with no samples selected

I had experimented with a couple options earlier, and went with the "not allowed" cursor because a couple other things use that (e.g. unsupported fields).
I decided on this one:

We can only easily align the "0" with either the left edge of the grid or the position of the "1" when a sample is selected (which is shifted to the right a bit due to the button), not both. I went with the former.

As for remaining issues:

There seems to be a problem with creating Select/Exclude fields with > 1 ID in them. The intention is to accept input via list,of,ids rather than JSON-style ["list", "of", "ids"] (which is how these stages used to accept input, I believe). When a stage is created programmatically, one sees list,of,ids in the view bar, but one cannot actually input multiple IDs in that way (or in JSON-style input)

@benjaminpkane plans to address this, I believe in #486.

In the App backlog, there is a feature request to adopt a more compact rendering strategy for Select and Exclude, which can be very long. My proposal is to render <first-few-charcters>... when it gets too long (maybe always, even when one sample is selected, since the IDs are not really meant for visual consumption). When the user clicks into the stage, the full text would be expanded

Unsure what the plan is on this (see issues mentioned above) but we might be able to improve at least the scrolling behavior for 0.5.4.

As for remaining issues:

There seems to be a problem with creating Select/Exclude fields with > 1 ID in them. The intention is to accept input via list,of,ids rather than JSON-style ["list", "of", "ids"] (which is how these stages used to accept input, I believe). When a stage is created programmatically, one sees list,of,ids in the view bar, but one cannot actually input multiple IDs in that way (or in JSON-style input)

@benjaminpkane plans to address this, I believe in #486.

In the App backlog, there is a feature request to adopt a more compact rendering strategy for Select and Exclude, which can be very long. My proposal is to render <first-few-charcters>... when it gets too long (maybe always, even when one sample is selected, since the IDs are not really meant for visual consumption). When the user clicks into the stage, the full text would be expanded

Unsure what the plan is on this (see issues mentioned above) but we might be able to improve at least the scrolling behavior for 0.5.4.

I'll have these addressed in #486 tonight.

Screenshot after #486 was merged in, for reference:


Develop

@atz - given the config changes and a possible bump in this gem version, you might have to update some SDR code.

@darrenleeweber - not sure what you mean.  There's no config changes here, except for Travis.


CheckPublishable If Statement Incorrect

Build Example
This If Statement isn't picking up the release branch correctly.
I'm sure it's something obvious, but the check is eluding me.

This is the problem, right?  I think the logic is flawed:
 elif [[ $TRAVIS_BRANCH != "master" || $TRAVIS_BRANCH != "release-"* ]]; then

Won't that compound expression will always return true?

Input
Expression 1
Expression 2
Overall

x
true
not evaluated
true

master
false
true
true

release-0.16.x
true
not evaluated
true

Shell scripting isn't one of my strengths, but exiting from a function feels awkward.  Might be better to return 1 or 0 and exit from the caller based on return value.  Like isPublishableTo "foo" || exit 0
or:
if ! isPublishableTo "foo"; then
    exit 0
fi

(Syntax above not tested; I might have botched something.)
Switching to a return value, probably allows for simpler expressions that are easier to reason about.  (Probably just me, but reasoning about NOT A OR NOT B makes my brain hurt.)

Yup, should probably be
elif [[ $TRAVIS_BRANCH == "master" || $TRAVIS_BRANCH == "release-"* ]]; then
     currentElse
else
    currentElif
fi


Initial upload of doi markdown

ASCT and Ref Organ markdown files for DOI

Initial DOI file upload

CC: @katyb

@mccalluc I'm merging now. Let us know when it can be pushed to portal. Thanks!

Let us know when it can be pushed to portal.

Release schedule is on the README:

Changes made by Wednesday 5pm will be
deployed to DEV/TEST/STAGE Monday after 5pm,
and, if QA passes, on production by Tuesday afternoon.

Chuck, that release schedule conflicts with what needs to happen. We need these files to launch today. We have a whole other website that can't launch because of dependencies. Today was the due date for getting this all up and working. There must be a way to perform a manual push.

From: Chuck McCallum @.>
Sent: Wednesday, March 17, 2021 12:55 PM
To: hubmapconsortium/portal-docs @.>
Cc: Cross, Leonard Edward @.>; Author @.>
Subject: Re: [hubmapconsortium/portal-docs] Initial upload of doi markdown (#55)
@bherr2https://github.com/bherr2 :
Let us know when it can be pushed to portal.
Release schedule is on the README:
Changes made by Wednesday 5pm will be
deployed to DEV/TEST/STAGE Monday after 5pm,
and, if QA passes, on production by Tuesday afternoon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hubmapconsortium/portal-docs/pull/55#issuecomment-801379255, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIWOIHHKXLKYVLCPBZ2QE3TEECJTANCNFSM4ZKQJZCA.

Sure. I've asked on slack to confirm that you don't need any other changes before we redeploy. I only check notifications here a couple times a day.

And as luck would have it, there now are changes. So don't push what you have. I'll be revising today. Katy will review again tonight and make any other changes she wants. You'll likely hear from me tomorrow.
I didn't see any message via Slack. What account are you sending to?

From: Chuck McCallum @.>
Sent: Thursday, March 18, 2021 6:04 AM
To: hubmapconsortium/portal-docs @.>
Cc: Cross, Leonard Edward @.>; Author @.>
Subject: Re: [hubmapconsortium/portal-docs] Initial upload of doi markdown (#55)
Sure. I've asked on slack to confirm that you don't need any other changes before we redeploy. I only check notifications here a couple times a day.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hubmapconsortium/portal-docs/pull/55#issuecomment-801909828, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIWOIBUGS6LDXOS3HGCDJLTEH24TANCNFSM4ZKQJZCA.

I hope it will not be a problem if the current state goes to production?
On slack, there is an @Leonard; I put the note in the #vitessce-viv-ccf-portal-integration channel


Image Curation Error

While I'm trying  to generate  embedding of my images getting below error. Kindly help me
2024-12-06 11:28:45,053 - distributed.worker - WARNING - Compute Failed
Key:       ('to-parquet-158a046691b90af5b8fd472b1642cd8a', 3)
State:     executing
Function:  subgraph_callable-ffac75d9b369c9e0ea212e7fd4d4e9f8
args:      ((3,), Empty DataFrame
Columns: [null_dask_index, output, key, image_embedding]
Index: [], '_run_inference-74e7702fbe349fda107a20f175646924', {'number': 3, 'division': None}, ['/home/sagemaker-user/llama-3.2-vlm/nemo-curator/xpo/00000.tar', '/home/sagemaker-user/llama-3.2-vlm/nemo-curator/xpo/00001.tar', '/home/sagemaker-user/llama-3.2-vlm/nemo-curator/xpo/00002.tar', '/home/sagemaker-user/llama-3.2-vlm/nemo-curator/xpo/00003.tar'], 'key', '_sort_partition-77aaef54c7e20b17298822d14bbe2ab4', 'read-parquet-471c0b903c5984bf871fa2f36a120c6e', {'piece': ('/home/sagemaker-user/llama-3.2-vlm/nemo-curator/xpo/00003.parquet', None, None)})
kwargs:    {}
Exception: 'RuntimeError('Critical error when building pipelin[e:\nError](file:///E://nError) in CPU operator nvidia.dali.fn.readers.webdataset,\nwhich was used in the pipeline definition with the following tracebac[k:\n\n](file:///K://n//n)  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/nemo_curator/image/embedders/timm.py", line 73, in webdataset_pipeline\n    img_raw, text, json = fn.readers.webdataset(\n\nencountere[d:\n\nUnderful](file:///D://n//nUnderful) sample detected at tar file at "/home/sagemaker-user/llama-3.2-vlm/nemo-curator/xpo/00003.tar"\nC++ context: [/opt/dali/dali/operators/reader/loader/webdataset_loader.cc:479] \nCurrent pipeline object is no longer valid[.](https://bz7k93asteypxs6.studio.us-east-1.sagemaker.aws/jupyterlab/default/lab/tree/llama-3.2-vlm/nemo-curator/)')'
Traceback: '  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/optimization.py", line 1001, in call\n    return core.get(self.dsk, self.outkey, dict(zip(self.inkeys, args)))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 157, in get\n    result = _execute_task(task, cache)\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in _execute_task\n    return func((_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in \n    return func((_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in _execute_task\n    return func((_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in \n    return func((_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in _execute_task\n    return func((_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in \n    return func((_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 121, in _execute_task\n    return [_execute_task(a, cache) for a in arg]\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 121, in \n    return [_execute_task(a, cache) for a in arg]\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/core.py", line 127, in _execute_task\n    return func(*(_execute_task(a, cache) for a in args))\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/utils.py", line 78, in apply\n    return func(*args, **kwargs)\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/dataframe/core.py", line 7164, in apply_and_enforce\n    df = func(*args, **kwargs)\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/dask/dataframe/core.py", line 7073, in func\n    return orig_func(*args, **kwargs, partition_info=partition_info)\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/nemo_curator/image/embedders/base.py", line 82, in _run_inference\n    for batch, metadata in datase[t:\n](file:///T:/n)  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/nemo_curator/image/embedders/timm.py", line 87, in load_dataset_shard\n    pipeline.build()\n  File "/home/sagemaker-user/.local/lib/python3.10/site-packages/nvidia/dali/pipeline.py", line 1060, in build\n    self._pipe.Build(self._generate_build_args())\n'
Perform the following steps
from nemo_curator.image.embedders import TimmImageEmbedder
embedding_model = TimmImageEmbedder(
"vit_large_patch14_clip_quickgelu_224.openai",
pretrained=True,
batch_size=256,
num_threads_per_worker=16,
normalize_embeddings=True,
autocast=False,
)
Dask is lazy, so this will not compute embeddings
dataset = embedding_model(dataset)
dataset.save_metadata() "Here I'm getting the above error"

Got the issue, image format should be in jpg and I was using png files.


Update blast entries

To include customizable output format
Change default of  -max_target_seqs to 500
Consider implementing: -subject_besthit

Have defaults be: "6 qseqid sseqid slen length pident nident mismatch gapopen gaps qstart qend sstart send evalue"

Done


Update CSS Styling

-Make Dark Mode Darker
-Add A Placeholder To The Search Bar In Supported Devices
-Match Search Bar Styling to Nav Search Bar in Supported Devices
-Make Danger Tip Have Better Contrast
-Make Nav Items Larger for better Accessibility

Is it possible to bypass Lint checks for this PR as, it is only a CSS update, and not added content

Checks dont block merge

the CSS changes here overall reduce the contrast between elements in dark mode, making things harder to see. I can't say I'm a fan at all.

Is this a reference to the search bar or the website as a whole?

Is this a reference to the search bar or the website as a whole?

both. What is the goal of the CSS changes in this PR?

Closing because author declared absence in Discord