docs: how to add a new chain-id to download genesis

Closes https://github.com/celestiaorg/celestia-app/issues/2792
Testing
$ ./build/celestia-appd download-genesis
Downloading genesis file for celestia to /Users/rootulp/.celestia-app/config/genesis.json
Downloaded genesis file for celestia to /Users/rootulp/.celestia-app/config/genesis.json
SHA-256 hash verified for celestia

$ cat ~/.celestia-app/config/genesis.json | jq ."chain_id"
"celestia"

$ ./build/celestia-appd download-genesis mocha-4
Downloading genesis file for mocha-4 to /Users/rootulp/.celestia-app/config/genesis.json
Downloaded genesis file for mocha-4 to /Users/rootulp/.celestia-app/config/genesis.json
SHA-256 hash verified for mocha-4

$ cat ~/.celestia-app/config/genesis.json | jq ."chain_id"
"mocha-4"

Codecov Report

Merging #2812 (b7b0fbc) into main (e5e98d8) will decrease coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2812      +/-   ##
==========================================
- Coverage   19.72%   19.44%   -0.28%
==========================================
  Files         140      143       +3
  Lines       17007    17320     +313
==========================================
+ Hits         3354     3368      +14
- Misses      13334    13633     +299
  Partials      319      319

Files
Coverage Δ

cmd/celestia-appd/cmd/download_genesis.go
16.86% <100.00%> (ø)

... and 2 files with indirect coverage changes


LazyTSDBOpener to unblock HTTP handler for metrics and profile when replaying WAL

Closes #37
Other components might need updating as well.
cc @bboreham

Given that conprof uses a noopAppender while starting we actually need to profile conprof startup with another conprof.
HA pair of conprof - anyone? 😂

We should be ok on Polar Signals to try this out as the sampler is separate from the storage (part of which would be restarting).

Generally looking good, but it looks like e2e tests are failing.

The e2e test are failing due to start up of HTTP and TSDB now being concurrent...
I need to take a look again to make it somewhat predictable for the tests.


Investigate Whether CAN_Wrapper_Transmit is Safe to Call from Inside an ISR

I have a strong feeling it may not always be safe to call CAN_Wrapper_Transmit (or transmit_internal) inside an ISR, due to the fact that a call to HAL_CAN_AddTxMessage could be in-progress at the time of the ISR's execution.
This is an important consideration because transmit_internal is being invoked in the HAL_CAN_RxFifo0MsgPendingCallback ISR. If it's unsafe, we may have to find a work-around such as deferring transmissions when inside an ISR.

After some investigating, it definitely can fail and we should find a solution for that.


Replace hir_ty::Lifetime with chalk equivalent

Our Lifetime isn't really used yet so this is a rather simple change

Oh yeah, that makes sense...
bors r+
changelog internal


OrleansTestKit 8.1.0 not available on nuget.org

The latest version on nuget.org is still pointing to 8.0.0.
https://www.nuget.org/packages/OrleansTestKit
Can you please push the latest version to nuget?

Project maintainers have been notified of the publishing issue.
https://github.com/OrleansContrib/OrleansTestKit/issues/160#issuecomment-2068773710

@seniorquico I'm happy to taken on a responsibility of maintaining NuGet packages - could you please add me as a package co-owner in NuGet?

I would like to point out that there is a glaring problem in the 7.0 branch too, with an open PR that would fix the issue provided that a new version from 7.0 branch was published after the PR was merged.
So +1 for giving jkonecki maintaining rights!

I've invited @jkonecki to be a maintainer.

@jkonecki already had maintainer privileges thanks to the OrleansContrib permissions.
Also of note- the 8.1.0 package was released to NuGet last week. Closing this issue.


Add images with Node.js 14

As the new version of Node.js was released, it would be nice to have Cypress docker images with it too.
Related to #294

 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.


Profile must have access to Payment Object to use the Manage Allocations Button

[Hub Thread](https://powerofus.force.com/s/question/0D51E00004xzTSfSAM/non-admin-users-are-getting-error-when-trying-to-manage-allocation-is-anyone-else-having-this-issue)
Issue: User is unable to utilize the Manage Allocations button on an Opportunity record unless Payment full CRED access is provided.
Likely introduced with this PR.
Steps to repeat:

On any custom profile, remove CRED access to Payment Object. (Assuming GAU feature is already setup for the profile).
Navigate to existing Opty, or create a new one.
On the GAU Allocations related list, click the Manage Allocations button.

Expected Behaviour:
Manage Allocations Visualforce page should open, allowing the user to modify GAU Allocations related to the Opty.
Actual Behaviour:
GACK error is produced: "npsp__util_pagemessages: system.security.NoAccessException: You do not have sufficient privileges to access the controller: UTIL_HtmlOutput_CTRL"
Issues to resolve:

There shouldn't be a dependency on the Payment Object to use the Manage Allocations button when the Payment Allocations feature is not enabled.
Permissions error should be more specific to what object/permissions are required.

Further details will be added to resulting work item (debug log, etc.)

**lurch: add

Tracking W-035127

**lurch: add

Tracking W-035495


[BIOMAGE-1785] - Fix overflowing embedding and heatmap popup boxes

Description
Background
The popup boxes in the Embedding and Heatmap plots in Data Exploration can not be seen if the cell that is chosen is too close to the top-right or bottom-right part of the panel. This is because the panel is desplayed to the right and bottom of the pointer.
Solution
Invert the popup to display to the top-left or bottom-left of the pointer if the location of the cell is close ( < width of the pop box) to the edge.
Details
URL to issue
https://biomage.atlassian.net/browse/BIOMAGE-1785
Staging link: https://ui-agi-pokey-dane.scp-staging.biomage.net/
Link to staging deployment URL (or set N/A)
N/A
Links to any PRs or resources related to this PR
N/A
Integration test branch
master
Merge checklist
Your changes will be ready for merging after all of the steps below have been completed.
Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format.

Manual/unit testing

[ ] Tested changes using InfraMock locally or no tests required for change, e.g. Kubernetes chart updates.
[ ] Validated that current unit tests for code work as expected and are sufficient for code coverage or no unit tests required for change, e.g. documentation update.
[ ] Unit tests written or no unit tests required for change, e.g. documentation update.

Integration testing
You must check the box below to run integration tests on the latest commit on your PR branch.
Integration tests have to pass before the PR can be merged. Without checking the box, your PR
will not pass the required status checks for merging.

[ ] Started end-to-end tests on the latest commit.

Documentation updates

[ ] Relevant Github READMEs updated or no GitHub README updates required.
[ ] Relevant Wiki pages created/updated or no Wiki updates required.

Optional

[ ] Staging environment is unstaged before merging.
[ ] Photo of a cute animal attached to this PR.

There's a lot of unnecessary white space at the right side of the text box. Can it be removed? See example screenshots.

Right now we have 3 components to show the same UI popup: CellInfo.jsx, HeatmapCellinfo.jsx and HeatmapTracksCellinfo.jsx. The only difference between them is the content that they show. I think these components can be combined into one and refactored such that the parent element is responsible for the generation of the content of the popup. This way, the concern of the component would be just to render the content. Do you think it would be a good idea? @ivababukova

Right now we have 3 components to show the same UI popup: CellInfo.jsx, HeatmapCellinfo.jsx and HeatmapTracksCellinfo.jsx. The only difference between them is the content that they show. I think these components can be combined and refactored into one component.
I'm also thinking of moving the generation of the popup text into the parent component. This way, the concern of the component would be just to render the content. This would also make it easier to address the width issue because the component will be passed the final form of the content, so we can just check which of the text has the longest length.
Do you think this refactor would be a good idea? @ivababukova

I remember that we evaluated this when we were creating those popups. We decided to have them as separate components, because although they are mostly similar, they have some differences which when generalised, make the component much harder to understand and use. The generalisations that we attempted were leading to leaky abstractions. This, combined with the fact that we rarely use and modify these components makes me on the opinion that there is little/no point in refactoring the cellinfo components into one for now. Our time is better spent elsewhere in my opinion


⚠ Bofrost.be has degraded performance

In feb9e4f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1225 ms

Resolved: Bofrost.be performance has improved in 5a7eb12 after 48 minutes.


which tensorlfow version support for arduino TensorFlowLite_ESP32 1.0.0

Hey i am trying to run person detection esp32 cam after converting into c source file of image classification mobilenet v1 with 0.25 but getting reboot error. so please ket me know which version of tf should i chose to convert from tf model to c source

I may be late for the party but here you go.

first, check if brownout detector is disabled. If not, disable it.

WRITE_PERI_REG(RTC_CNTL_BROWN_OUT_REG, 0);  //disable brownout detector

if your model is a big one, then it will use a lot of Internal RAM causing ESP32 to reboot. you can allocate memory on PSRAM for your model and TensorArena. This may solve your issue.

in .ino file:
if (tensor_arena == NULL) {     //allocate memory for TensorArena on PSRAM     tensor_arena = (uint8_t *) ps_malloc(kTensorArenaSize);   }
in model.cc file:
// allocate memory in PSRAM for the model alignas(8) const unsigned char g_person_detect_model_data[] __attribute__((section(".psram.data"))) = {     0x1c, 0x00, 0x00, 0x00, 0x54, 0x46, 0x4c, 0x33, 0x00, 0x00, 0x00, 0x00, 0x00,     0x00, 0x0e, 0x00, 0x18, 0x00, 0x04, 0x00, 0x08, 0x00, 0x0c, 0x00, 0x10, 0x00, .....}

If still not working, you may have to train your model to be smaller. You can use weights of MobileNet models trained on greyscale images to create your base model and do transfer learning. Using greyscale images may lower RAM usage during inference.

here is  the Git repo of greyscale MobileNet weights: https://github.com/Navodplayer1/MobileNet_96x96_greyscale_weights


Hierarchical pipeline example notebook

Before submitting (must do checklist)

[ ] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Proposed Changes

Closing issues

closes #1057

Codecov Report

:exclamation: No coverage uploaded for pull request base (hierarchical_pipeline@edd801d). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                   @@
##             hierarchical_pipeline   #1070   +/-   ##
=======================================================
  Coverage                         ?   8.25%
=======================================================
  Files                            ?     169
  Lines                            ?    8976
  Branches                         ?       0
=======================================================
  Hits                             ?     741
  Misses                           ?    8235
  Partials                         ?       0

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


Patches from v4.3.0 release candidates

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


Iconics missing a few feats

There are a few feats missing on the Iconics:
Chk-Chk (level 1) doesn't have the BG skill feat (Holy Talisman)
Dae (level 5) doesn't have the level 4 class feat (Reactive Strike)
Navasi (level 5) doesn't have the level 1 class feat (Acquire Asset)

Chk-Chk had the feat, but as a bonus feat. It's been moved to the BG slot now. Navasi has been fixed.
Dae has reactive strike. The 2nd level feat (shattering strike) doesn't exist, and that has already been reported to Paizo.


Subreddit feeds showing generic icon

App Store Connect report:
“Adding Reddit subreddits shows generic Reddit icon. Unlike macOS production release, which shows specific subreddit icon”

This person probably hasn't enabled their Reddit extension on iOS. API access is required to get the correct subreddit icon.


accessibility fix part 1

fix the accessibility bug for asdk-installer

Tested the following flow

Prepare Environment (boot into CloudBuilder)
Install ASDK flow, able to reach to final screen to generate the deployment script.
Reboot flow (reboot back into windows server)


Ad possibility to restart pod through fluxctl

Describe the feature
I want possibility to initiate flux controller restart
Expected behavior
I want new command in fluxctl restart that will allow me to restart flux controller (deployment or daemonset)
Workaround
Now in order to restart I have to add some annotation into controller manifest, then commit file to git. If I want to do restart immediately I apply controller manifest to kubernetes manually. Sometimes I observed flux did update one more time during synchronizing with git. It's not convenient.

Could you just do kubectl delete pod -l name=flux instead?

I'm sorry. I initially wrote "initiate kubernetes deployment restart". But in flulx terms deployments and daemonsets are flux controllers. So my aim is to initiate graceful kubernetes deployment or daemonset restart.
kubectl delete pod is not a good option by many reasons. Even if deployment has single pod it can have maxUnavailable: 0, maxSurge: 1 - in that case new pod must be created and started before removing old one.

Maybe I didn't fully understand the motivation for the feature request, but I don't think restarting workloads (at least per-se) falls into Flux's responsibilities.
Can you clarify why restarting a workload is related to what Flux does?

Ok, sure
So we need to restart kubernetes deployment or daemonset. This is common thing and there are many reasons to do that. In our case common scenario is - we store configuration on some http resource, app loads configuration and in order to update config properties that are not hot reloadable we do app restart.
Now - how deployment or daemonset can be restarted in kubernetes. Gracefully, with respect to rollingUpdate policy. Currently kubernetes doesn't have native restart support. As far as I found one way to do that is to add some meaningless annotation - like "restart: date_time". So that can be done with pure kubernetes. It's possible to do that with flux as well - just add annotation into git file and that will initiate restart. Ok. But there are some caveats with that:

there is some lag between git commit and flux apply updates - so what if I want to apply changes immediately
there is no information about update process

So what I'm suggesting is to fix that two problems. One solution is to do that with flux - just add restart command that will do the thing. Another solution can be "apply git changes" flux command that applies changes from git immediately and reports in status progress.

There is fluxctl sync command that can be used to synchronize the cluster with the git repository, now. So applying annotation to manifest file + applying fluxctl sync solves the issue


Simplify keyring

When using numeric keys, the numbering of the keys themselves is a perfectly reasonable way to order them: the created_at can be omitted. Alternately, the id itself could be left out, and we can use the key timestamp as the key identity.

Numeric ids came in 3f6c1ea0a69c8846071f90bf8f273423365029e0. There's some additional work to test the now-legacy uuid path and to provide a migration mechanism, but this is set.


Dead Sample?

Drive API documentation still links here as a sample project. But looks like Authentication with code no longer works.

Have a PR (#12) to fix this and update to latest client & auth libraries.


Keywords in enum values

I have a program with a debug output option, so it can be called with --format debug. Output options are defined as enum:
enum OutputFormat
{
    silent,
    debug_,
}

Obviously, I cannot define debug as enum member since it is a keyword. My current workaround looks like this:
import argparse;
import std.meta;
import std.traits;
import std.conv;
import std.algorithm;

enum OutputFormat
{
    silent,
    debug_,
}

private enum string allowedOutputFormat(OutputFormat Member) =
    Member.to!string.strip('_');
private enum string[] allowedOutputFormats = [
    staticMap!(allowedOutputFormat, EnumMembers!OutputFormat)
];

private OutputFormat parseOutputFormat(string input)
{
    switch (input)
    {
        case "debug":
            return OutputFormat.debug_;
        case "silent":
            return OutputFormat.silent;
        default:
            throw new Exception("");
    }
}

static struct Basic
{
    @(NamedArgument
            .AllowedValues!allowedOutputFormats
            .PreValidation!((string x) => true)
            .Parse!parseOutputFormat
            .Validation!((OutputFormat x) => true)
    )
    OutputFormat format;
}

mixin CLI!Basic.main!((args)
{
    return 0;
});

I don't mind writing some logic for such edge cases, but it would be nice if it were a bit easier. Defining PreValidation and Validation for this looks strange, but the parser doesn't work without it.

I have this in my TODO list. My idea is to have something like this:
enum OutputFormat
{
    silent,
    @ArgumentValue("debug")
    debug_,
}

@belka-ew Implementation will allow you to do exactly this:
enum OutputFormat
{
    silent,
    @ArgumentValue("debug")
    debug_,
}


fix kosmos-scheduler miss lease permission

What type of PR is this?

fix: fix bug

What does this PR do?

add parameter for kosmos-scheduler
--leader-elect-resource-name=kosmos-scheduler
--leader-elect-resource-namespace=kosmos-system
kosmos-scheduler add lease permission
Which issue(s) does this PR fix?

Fixes #https://github.com/kosmos-io/kosmos/issues/117
Special notes for your reviewer:

Screenshots (if applicable):

Additional Comments:

fix bug kosmos-scheduler compete for lease -- kube-system/kube-scheduler

/cc @duanmengkk

/cc @wuyingjun-lucky

/assign @hanweisen

/lgtm

/assign @wuyingjun-lucky

/approve


NIC Bridged: Use of network argument when detecting parent

Also improves host_name state reporting when using VM with lxd-agent running.

Testsuite passed


Add --help flag and usage message

Add an initial usage and --help for the cps-config utility. The goal is to give potential contributors to this repository some idea about what the utility is for based on only looking at a --help message.

@lunacd @dcbaker cxxopts is a nice library, but I'm expecting we'll want some more powerful features sooner or later:

subcommands
an epilog

examples
support

If you want me to open an issue first, let me know. Otherwise, don't be surprised if I revisit the UX for cps-config from the command line in a future PR. I'd still keep a simple way to serve as a drop-in replacement for pkg-config of course. Ideally, if we see argv[0] is pkg-config, that's the default operating mode. pkgconf does this and I think it's nice.

This PR was inspired by me running cps-config --help and getting a raw parsing error message instead of what you'd expect. We should also catch unexpected arguments more elegantly, but I'm thinking that's another PR (see https://github.com/cps-org/cps-config/pull/48#issuecomment-1988171269).


httpCall() proxy options support

Title: httpCall() proxy options support
Description:
I want to generate a new request via HTTP proxy to the https site. In this case, envoy should send a request with the CONNECT method after the TLS channel is established then an HTTP request should be sent.
Now I want to use the form
-- Alternative function signature.
local headers, body = handle:httpCall(cluster, headers, body, options)

And specify proxy in the options array-like
-- Create a fire-and-forget HTTP call.
local request_options = {["proxy"] = "https://exmple.com:3129/"}

Could you implement "proxy" options of httpCall().

Looks like Http11ProxyUpstreamTransport maybe used to relay request via HTTP proxy.
I will try to prepare the envoy config.

cc: @alyssawilk who has been doing CONNECT work lately

as a heads up that transport must (currently) work in conjunction with an HTTP filter which puts metadata in the stream info.  See test/integration/filters/header_to_proxy_filter.cc for a test only filter example of this.  I think it would be lovely to upstream an envoy non-test filter which did this, but the only use case I had was Envoy Mobile.

I am not a Go developer and do not understand how to use his info.
Maybe you have the YAML config file?

I go to relay the request to the upstream envoy forwarding proxy.
For https links I will a script for casting HTTP requests between envoys servers.


Don't use ZGC on win7

Would be good if you explained the motivation behind it in the commit message.

will this change affect Linux users?

Zgc is only supported on windows builds greater than win 10 1803 due to older versions lacking a newer windows memory API.
It does not affect Linux users because they don't use install4j and manage their own vmoptions


can't pass alertmanager url to prometheus 2.0.0

What did you do?
Per: https://prometheus.io/docs/operating/configuration/#
<alertmanager_config>
Sep 21 01:00:54 prometheus prometheus[1515]: time="2017-09-21T01:00:54Z" level=error msg="Error loading config: couldn't load configuration (--config.file=prometheus.yml): unknown fields in config: alertmanager_config" source="main.go:279"
And other docs
[root@prometheus prometheus]# ./prometheus --alertmanager.url
prometheus: error: unknown long flag '--alertmanager.url'
[root@prometheus prometheus]# ./prometheus -alertmanager.url
prometheus: error: unknown short flag '-a'
What did you expect to see?
a successful connection to the alertmanager
There does not seem to be a way to connect prometheus to alertmanager.
Environment

System information:

[root@prometheus prometheus]# uname -srm
Linux 3.10.0-693.2.2.el7.x86_64 x86_64

Prometheus version:
insert output of prometheus -version here
[root@prometheus prometheus]# ./prometheus -version
prometheus: error: unknown short flag '-v'

Alertmanager version:

[root@prometheus alertmanager]# ./alertmanager -version
alertmanager, version 0.8.0 (branch: HEAD, revision: 74e7e48d24bddd2e2a80c7840af9b2de271cc74c)
build user:       root@439065dc2905
build date:       20170720-14:14:06
go version:       go1.8.3

Prometheus configuration file:

# my global config
global:
#  scrape_interval:     15s # Set the scrape interval to every 15 seconds. Default is every 1 minute.
#  evaluation_interval: 15s # Evaluate rules every 15 seconds. The default is every 1 minute.
  # scrape_timeout is set to the global default (10s).

  # Attach these labels to any time series or alerts when communicating with
  # external systems (federation, remote storage, Alertmanager).
  external_labels:
      monitor: 'miner-monitor'

# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
    - miner.rules

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'prometheus'

    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.

    static_configs:
      - targets: ['localhost:9090']

  - job_name: 'miner-monitor'
    static_configs:
      - targets: ['localhost:7979']

alertmanager_config:
  static_configs:
    targets: ['localhost:9093']

Alertmanager configuration file:

[root@prometheus alertmanager]# cat simple.yml | grep -v '#' | grep -v '^$'
global:
  smtp_smarthost: 'localhost:25'
  smtp_from<EMAIL_ADDRESS>  smtp_auth_username: 'alertmanager'
  smtp_auth_password: 'password'
  hipchat_auth_token: '1234556789'
  hipchat_url: 'https://hipchat.foobar.org/'
templates:
- '/etc/alertmanager/template/*.tmpl'
route:
  group_by: ['alertname', 'cluster', 'service']
  group_wait: 30s
  group_interval: 5m
  repeat_interval: 3h
  receiver: kane
  routes:
inhibit_rules:
- source_match:
    severity: 'critical'
  target_match:
    severity: 'warning'
  equal: ['alertname', 'cluster', 'service']
receivers:
- name: 'kane'
  email_configs:
  - to<EMAIL_ADDRESS>- name: 'team-X-mails'
  email_configs:
  - to<EMAIL_ADDRESS>- name: 'team-X-pager'
  email_configs:
  - to<EMAIL_ADDRESS>  pagerduty_configs:
  - service_key: <team-X-key>
- name: 'team-Y-mails'
  email_configs:
  - to<EMAIL_ADDRESS>

Logs:

insert Prometheus and Alertmanager logs relevant to the issue here

In version 2 the urls towards your alertmanager(s) have moved from a flag towards the config file. In our case it looks like this:
    # alert managers
    alerting:
      alertmanagers:
      - scheme: http
        static_configs:
        - targets:
          - "alertmanager-0.alertmanager-headless:9093"
          - "alertmanager-1.alertmanager-headless:9093"
          - "alertmanager-2.alertmanager-headless:9093"

What @JorritSalverda. The flag has already been deprecated in 1.x for a while now and for 2.0 we removed it entirely.


benchmark.h file is missing

I can't compile the project because the benchmark.h file is missing in the repository.

I was trying to compile as well and had the same issue.

@jcwangxp me too

Sorry, I missed your messages, just fixed it.


DeeperSpeed eb7f5cff version branch cannot be found

I followed the tutorial to install the requirements/requirements.txt, but DeeperSpeed eb7f5cff version branch cannot be found, can you provide a correct DeeperSpeed branch version information?

It is in fact right here so I'm not sure why you'd be having problems with it. Try unpinning the commit and seeing what happens.

Thank you very much. This is my mistake. But unfortunately, it raise an out of memory error during I try to finetune 20B model in my machine 8 x A40 (44G). I follow the configure configs/20B.yml in repo, only different is train_micro_batch_size_per_gpu=1.
I'm not sure if this machine can run it? If so, what parameters should I adjust to reduce its GPU memory usage? And why is it that when I set zero_optimization to level 2 or 3, it lacks more memory(when it crashes) than when level 1?

It is in fact right here so I'm not sure why you'd be having problems with it. Try unpinning the commit and seeing what happens.

I am trying install deepspeed with the command: sh install.sh in the branch eb7f5cff as you gived, but errors ocurrs as following:
    from ..op_builder import CPUAdmBuilder
ModuleNotFoundError: No module named 'deepspeed.ops.op_builder'

I am closing this issue as the original problem is solved and the new problem is duplicative of #797. Please continue this conversation there.


Adding a README

Title and Issue number
Adding README.md #4229
Close #4229
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@pranjay-poddar @MohitGupta121 @TusharKesarwani Please review my PR. Thanks

@MohitGupta121 @pranjay-poddar @TusharKesarwani Please review this PR. please.


python312Packages.manifold3d: 3.0.0 -> 3.0.1

Updated manifold from v3.0.0 to v3.0.1 with some bug fixes and performance improvements, same as #369309.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

please fix the format error :)

please fix the format error :)

Fixed.

Hmmm, for some reason the formatter I am using is producing different output. Maybe I should just manually format it.


Fixes User name on image upload

Closes #765
Using real username instead of Sherlouk 😆

lol

😞 Sorry

Nvm. Every one does!!
Sent with GitHawk

Lol if I had a nickel....
Sent with GitHawk


Support all float dtypes in F.roi_pooling_2d

This PR follows #4582 to support all float dtypes in F.roi_pooling_2d.

Large error in Py2 FP16.

Jenkins, test this please.

Jenkins CI test (for commit 004e80e8d85ac2c3467510851cd370a4db467877, target branch master) succeeded!

LGTM!


Create a Galaxy tool for reconciling taxon names

The tool should be able to take text strings and reconcile them with external identifier providers e.g., Catalogue of Life+ IDs, GBIF's Backbone Taxonomy, and others.

Out of scope for the SYNTHESYS+ project but remains a valid/required tool for future DiSSCo services.


Lower vsphere concurrent tests to 60

Description of changes:
Lower concurrent test count for vsphere to 60
/approve
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve

/cherry-pick release-0.21


Added ProcessLauncherWhitelist

@mikehall-ms, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

+1

Do we need to have a localize version?

:+1:


Ensure NEW_NODE notifications aren't errantly collected

Along with https://github.com/pcmanus/ccm/pull/291, this should
make sure that we don't collect NEW_NODE events from the cluster
starting up.  Sadly, I don't see any simple alternatives to adding the sleep()
line.

+1


Core/cli recover

Tracking issue
_https://github.com/flyteorg/flyte/issues/5049_
Closes #5049

Why are the changes needed?
Enable the recovery of executions in pyflyte. This functionality has already been implemented in flyteCtl and flyteconsole.

What changes were proposed in this pull request?
Add a command group execute, which can invoke a recover subcommand to recover executions.

How was this patch tested?

make test
adding a unit_test

Setup process
Usage:
pyflyte execution --execution-id {execution_id} -p {project} -d {domain} recover
Screenshots

Check all the applicable boxes

[x] I updated the documentation accordingly.
[x] All new and existing tests passed.
[x] All commits are signed-off.

Related PRs

Docs link

@novahow , thanks for the PR!
Can you also add support for relaunch?

@eapolinario Sure, relaunch support is added.


add provider information in samples index

Signed-off-by: Stephanie<EMAIL_ADDRESS>What does this PR do?:
Summarize the changes. Are any stacks or samples added or updated?
This PR updates the  extraDevfileEntries.yaml to include provider information in samples index, for ocd to get and show in their frontend
Which issue(s) this PR fixes:
Link to github issue(s)
part of https://github.com/devfile/api/issues/658
PR acceptance criteria:

[ ] Contributing guide
Have you read the devfile registry contributing guide and followed its instructions?
[ ] Test automation
Does this repository's tests pass with your changes?
[ ] Documentation
Does any documentation need to be updated with your changes?

How to test changes / Special notes to the reviewer:

/test v4.7-registry-test

OpenShift CI issues have been resolved, some I’m gonna retest.
/retest


Make the releases available as a Python package

This will greatly facilitate installation

Use __main__.py as well to manipulate commands from module with python -m. This will make the use more comfortable on  Windows.


Syntax error in init.sh on Ubuntu 16.10

I tried to rebuild the image with Ubuntu 16.10, and when I run docker start I got the error:
/init.sh: 7: /init.sh: Syntax error: "(" unexpected (expecting "fi")
I removed the functions from init.sh  (i.e. stop_all() {...}) and I was able to rebuild and start the container, however I noticed that those were added in ed0498a for Fedora.
If this doesn't work on Fedora without the function, is there a way to make it work or should we just remove fedora support (why do we even have it?)?  FWIW I also tried the syntax function stop_all {...} (without ()) but it doesn't work.  Replacing the shebang with /bin/bash might fix the problem for both, but I haven't tried.

I'll leave this open until I'll open a followup I've mentioned in https://github.com/python/docker-bpo/pull/6#issuecomment-285594739

init.sh has changed significantly in the past 5 years, and this seems to be no longer relevant.


Share the slides you presented at KDD 2017?

Hi,
Is it possible to share the slides you presented at KDD 2017?
If so, I would be highly appreciated.
Thanks in advance.
JM

Hi,
No problem. Uploaded to the folder "slides".
Chong

Get it and many thanks :)
Best,
JM


Make it usable on non-SSE platforms

For Yamagi Quake2 we had a bugreport that it doesn't build on ARM due to HMM, so I looked into the issue and (hopefully!) fixed it

at one place HANDMADE_NO_SSE instead of HANDMADE_MATH_NO_SSE was used
introduce HANDMADE_MATH__USE_SSE for the SSE #ifdefs throughout code

use #ifdef HANDMADE_MATH_NO_SSE at only one place

only use SSE (#define HANDMADE_MATH__USE_SSE) if the targetplatform
actually supports it
=> users don't have to #define HANDMADE_MATH_NO_SSE on ARM etc
at one place HMM_SqrtF instead of HMM_SquareRootF was used

Looks good to me, but we'll see what @StrangeZak thinks. I'm just confused why the tests didn't catch the typo with HMM_SqrtF.

my assumption would be that you have no tests without SSE?

Oh yeah, that's right...my mental todo list is coming back to me now :P

I'll get those test cases added to make sure this stuff doesn't come up again.

Great!
Ideal would be tests (or at least builds) for non-x86 to make sure it builds at all without SSE (ideally without having to manually specify HANDMADE_MATH_NO_SSE; my patch should allow that).
But I don't know if travis or another free service offers other platforms (ARM?)

I looked at my own code and was surprised to see that it was already testing with and without SSE...only to find that it's on the SSE branch which has been idle for a long time. I'll pull over the relevant testing fixes for now and @StrangeZak's other SSE stuff can come later whenever he decides to finish it.
Unfortunately I am already testing with both compilers available to me on Travis. It seems Travis's support for C and C++ is pretty limited compared to their other platforms. Might there be some more gcc and g++ options that could test different architectures? I would guess it wouldn't handle different includes anyway...

just gcc/g++ options won't suffice, but maybe there are crosscompilers available, e.g. Ubuntu has the gcc-5-arm-linux-gnueabi package.
Note that I consider the change from HANDMADE_MATH_NO_SSE throughout the code to only checking it once and then use HANDMADE_MATH__USE_SSE instead important - for once you don't have to specify anything on non-x86 and it still works, but this also allows adding other kinds of SIMD code (e.g. ARM NEON) in the code (with sth like HANDMADE_MATH__USE_NEON) later without going insane - I mean, how would the #ifdefs look like then?
#ifdef HANDMADE_MATH_NO_SSE
#if defined(__arm__) && !defined(HANDMADE_MATH_NO_NEON)
// neon code
#else
// C code
#endif // __arm__ or whatever
#else // not HANDMADE_MATH_NO_SSE
// SSE code
#endif

Yeah, I've questioned the use of SSE-by-default in this repo before. I think the USE_SSE approach makes more sense.
I just remembered that Handmade Network projects get Gitlab projects, and Gitlab has very robust CI, so we might be able to do much more robust testing over there if we ever migrated the repo. Github is convenient and good for visibility though.

TBH, if this project were on some gitlab server somewhere and I'd have to create an account there to do a PR, I'd be less likely to do that.
Here on github it's easy (as I already have an account), but otherwise not sure I'd bother.
(Of course you could just mirror it there and keep it in sync)

Super good change grabbing it now!


Bug fix and include id field for insurance

Update version number in MainApp.
Add life insurance command now gives proper feedback to the user after successful execution.
Every insurance now contains an id field corresponding to the key used in UniqueLifeInsuranceMap.
Renamed InsuranceProfile to InsuranceCard to avoid confusion.

Changes Unknown when pulling e711547a9e27a269cf8add3cc71b0c8616b8c1f4 on OscarWang114:collate-portfolio into ** on CS2103AUG2017-T13-B1:master**.


Add a builtins parameter

Hello,
I am working on databricks notebooks and I use VScode to edit the content of the notebooks.
I make extensive use of builtin databricks functions such as dbutils or spark. However I find Pylance raises an UndefinedVariable on those functions as they are not locally declared.
I would like to be able to specify some builtins functions that pylance won't raise an undefined error on. I imagine this would be similar to flake8's approach on the subject, see here
Even better : I would like to be able to specify a databricks profile where all databricks' builtin functions will be already defined (and autocompletion compatible). But not sure that fits in the philosophy of the package.

Pylance doesn't hard-code the definition of "builtins". Rather, it uses the type information from the builtins.pyi stub file from typeshed. You can download your own copy of the typeshed stdlib stubs and use the python.analysis.typeshedPaths setting to point to their path. You can then modify your local copy of builtins.pyi file and add additional symbols.

We've received this request from a number of users, so I've implemented a less-cumbersome way to extend the builtins namespace. For details, refer to this documentation. This new capability will be included in next week's release of pylance.

This issue has been fixed in version 2022.2.1, which we've just released. You can find the changelog here: CHANGELOG.md

So, I have a question about using the builtins.pyi file. In the example from databricks, the builtins are not functions, but actually entire modules with submodules, classes and functions. How would I go about 'mocking' that structure with the builtins.pyi file?

You should be able to import those other modules in __builtins__.pyi. To make these modules appear as "re-exported", you'll need to use a redundant form of import.
import foo as foo

You should be able to import those other modules in builtins.pyi.

So if the modules are not actually available outside of the databricks runtime, I would need to write a mock up module with the necessary submodules and classes and then import them into the __builtins__.pyi, is that correct?


utils: let select_proxy not raise an exception when url has no hostname

Fixes #3131.

While we're here, we should probably also handle the possibility that there's no scheme in the URL as well.

scheme defaults to '', though. Well, more or less:
The scheme argument gives the default addressing scheme, to be used only if the URL does not specify one. It should be the same type (text or bytes) as urlstring, except that the default value '' is always allowed, and is automatically converted to b'' if appropriate.

Urgh, urlparse is such an inconsistent function. Good spot.

Thanks! :sparkles: :cake: :sparkles:


⚠ Bofrost.fr has degraded performance

In 5d9acf3, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1066 ms

Resolved: Bofrost.fr performance has improved in 8e06ce7.


Add sort by "Next Episode"

Renamed the column "Next episode" to "Next Episode"
Sort by "Next Episode" is now functional.

Thanks !!
🚢


optional arg first is confusing in get_workspaces fn

https://github.com/axodotdev/axoproject/blob/main/src/lib.rs#L335

We fixed this a while ago


(just a question) Plan to merge with JUnit?

JUnit has Parameterize now: https://github.com/junit-team/junit/wiki/Parameterized-tests
but I like this JUnitParams way better, using an annotation. (You can look at the data and the implementation in 1 screenful. I already liked the similar feature in py.test, the Python test framework which uses decorators to parameterize.)
Is there a plan to get JUnitParams into JUnit, or will this remain as a independent plugin?

I don't think there is any plan for merging currently. JUnit is going for major version change to 5.0. And there are no such plans in near future AFAIK. @kcooney @marcphilipp am I correct?

We don't have any immediate plans on the JUnit side, but longer term I would love to have an extension API in JUnit that would allow you to customize how test methods are defined and run for a given method. One use case would be supporting things like JUnitParams without a special runner.
Anyone on the Pragmatists team willing to work with us on that?

More generally, I would like JUnit to create extension points that are better (read: more flexible, combinable)  so extension developers can write better extensions and be wildly successful.
Often, other projects can iterate much faster than the JUnit team can, so adding JUnitParams into JUnit proper wouldn't necessarily be the best thing for users of JUnitParams.

Good stuff — love the idea of helping devs be "wildly successful — is there an issue for creating these extension points?

@hangtwenty we don't have an issue tracking this currently. Feel free to add one if this is something you would like to see. I just don't want to design it in a vacuum, so if someone on the Pragmatists wants to help us design an extension point, that would help.
Someone could probably look at this pull and come up with a possible API for a MethodInvoker:
https://github.com/junit-team/junit/pull/1138

Ah, I see. OK, so it's pretty much blocked by https://github.com/junit-team/junit/issues/1161, "JUnit needs a better story for extensibility." Then JUnitParams could provide its functionality in some way besides a whole different Runner.
https://github.com/junit-team/junit/pull/1138 would also take care of it but unfortunately I'm not skilled enough with Java yet to take care of it.
Thanks for all the context!


npm ERR! peer dep missing<EMAIL_ADDRESS>|| ^0.5.0, required by<EMAIL_ADDRESS>

Operating system and version:
Linux Manjaro 5.4.72
wget -qO- https://raw.githubusercontent.com/nvm-sh/nvm/v0.37.0/install.sh | bash output:
nvm is already installed in /home/nicolas/.nvm, trying to update using git
=> => Compressing and cleaning up git repository
=> nvm source string already in /home/nicolas/.bashrc
=> bash_completion source string already in /home/nicolas/.bashrc
npm ERR! peer dep missing<EMAIL_ADDRESS>|| ^0.5.0, required by<EMAIL_ADDRESS>=> You currently have modules installed globally with npm. These will no
=> longer be linked to the active version of Node when you install a new node
=> with nvm; and they may (depending on how you construct your $PATH)
=> override the binaries of modules installed with nvm:
/usr/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>=> If you wish to uninstall them at a later point (or re-install them under your
=> nvm Nodes), you can remove them from the system Node as follows:
 $ nvm use system
 $ npm uninstall -g a_module

=> Close and reopen your terminal to start using nvm or run the following to use it now:
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && . "$NVM_DIR/bash_completion"  # This loads nvm bash_completion
Installed package
npm view marked
<EMAIL_ADDRESS>| MIT | deps: none | versions: 69
A markdown parser built for speed
https://marked.js.org
keywords: markdown, markup, html
bin: marked
dist
.tarball: https://registry.npmjs.org/marked/-/marked-1.2.3.tgz
.shasum: 58817ba348a7c9398cb94d40d12e0d08df83af57
.integrity: sha512-RQuL2i6I6Gn+9n81IDNGbL0VHnta4a+8ZhqvryXEniTb/hQNtf3i26hi1XWUhzb9BgVyWHKR3UO8MaHtKoYibw==
.unpackedSize: 262.9 kB
maintainers:

chjj<EMAIL_ADDRESS>styfle<EMAIL_ADDRESS>amidknight<EMAIL_ADDRESS>tonybrix<EMAIL_ADDRESS>
dist-tags:
latest: 1.2.3
published 5 days ago by tonybrix<EMAIL_ADDRESS>

sory wrong github page

The real issue is that you need to manually specify peer deps, so your npm graph is valid.


Difference between primary endpoint and URL unclear

There are options for two of them. Copy URL is a context menu item, always available .
It is also not so clear what the URL does/helps. Following the link leads to a resource not found errer:

This will be clearer when we get SDK support.


已忽略未知表头值 $DIMTXTDIRECTION 第 422 行

hello,if i use netdxf  to do  this "dxf.Save(@"D:\桌面\AAA.dxf");"……and then i will get  the  "已忽略未知表头值 $DIMTXTDIRECTION 第 422 行。"……how can  i remove  this  use the netdxf?…………my  next  application（not  autocad）  cannot  handle this……thank  you very much！

If your program does not recognize a valid header variable do what it says and just ignore it. You cannot remove it without modifying the code. All dimension header variables are written to avoid the appearance of the override style in the dimension styles list.
Write in English and do not understand Chinese.

thank you very much！


Markdown syntax highlighting doesn't work well

The syntax highlighter for markdown files seems to be having some issues.

[ ] Italicization is done spuriously

[ ] Second level nested lists aren't highlighted correctly

If this involves regex (I guess it does) and if I am pointed to the correct files that specifies the rules, I probably may be of help.

please discuss with the syntax file maintainer. Look at the header of the file $VIMRUNTIME/syntax/markdown.vim  Oh and check, that no plugin is interfering (check running  vim -u NONE -N)

The markdown syntax plugin is maintained on github.  Here is the link of the repository.

Italicization is done spuriously

Without a specific example of what's wrong, it's hard to fix the issue.
On the repository mentioned earlier, there are currently 3 issues where the keyword italic is mentioned in the title:

Inappropriate Italic
Ending an italic section with an escaped asterisk isn't parsed properly
Embedded HTML should not italic text between underscores

Second level nested lists aren't highlighted correctly

Again, without a specific example, it's hard to fix the issue.
On the repository mentioned earlier, there are currently 2 issues where the keywords nested list are mentioned in the title:

Link in Nested List
Missing conceal for code block in nested list

Ok. Thanks for all the great information! It's been so long now and I don't particularly remember the reason I opened this myself as things have changed. I'll just close this now and open specific issues in the appropriate repo if I see any issues in future. Thanks :)


Add match CLI option to run only matching tests

Add a match option for running a specific set of tests with titles matching the given pattern. Leverages matcher internally, so ! and * is supported.
ava --match='foo*' --match='!*bar' /test/*.js

# shorthand
ava -m='foo*' -m='!*bar' /test/*.js

This would run all tests inside /test with a title starting with foo and not ending with bar.
The match option can also be configured in the ava section of package.json.
{
  "ava": {
    "match": [
      "foo*"
    ]
  }
}

Fixes #476

I :heart: it! Simple code which does its job. Thank you!
There's one little thing though, could you please add some minimal docs to the readme along with a few examples?

There's one little thing though, could you please add some minimal docs to the readme along with a few examples?

Absolutely. I suppose somewhere in the cli section would make sense? Need to step away from open source for a couple of hours. Sometime tonight! :smile:

Yep, in cli section ;) Sure, no pressure, great job!

Hm. Not sure why the build is failing, ran just fine™ on my machine.
Node v5.7.0
Darwin v15.3.0
  options.match will skip tests with non-matching titles
  not ok TypeError: Cannot read property 'runOnlyExclusive' of undefined

All versions failed, so it must be something :)

Try pulling latest changes (#556 just got merged in).

Nope. Still no sauce unfortunately. It does point in the direction of #566.
Could it be that either match or runOnlyExclusive didn't add an entry in runner#chainableMethods?
Stabbing in the dark here : )

this appears to be the failing line. Somehow I'm not passing options to the run method.

@kasperlewau nice work. However it doesn't actually work end-to-end. I've left some inline comments as to why. Could you add a test to test/cli to exercise the feature end-to-end?

@vdemedes I've added docs for the --match flag (in the cli segment). We could probably shave off a sentence/example or two.
I'm still on the fence regarding some wording.

match titles ending with 'foo' [vs] match titles that end with 'foo'

Let me know what you guys think.

@novemberborn Wuh-oh! I'll see what I can do about that. Cheers!

Tests without a title will not run if any match pattern is supplied.

Unfortunately the runner does not have access to the full test title. It's later derived from the function name, for instance.
Maybe that title logic needs to be moved to the runner. @vdemedes?

@novemberborn

We're now passing through the match option from api -> runner.
We're guarding against an empty match value.
We're guarding against non-test types in the 'apply-match-logic' if block.

The one thing I haven't done is setting up an end-to-end test in test/cli to ensure it all works™. Do you have any suggestions/pointers I can take with me on that voyage? :smile:

@novemberborn It can't be moved to Runner, because we display full test titles only when there are multiple files and only CLI knows how many test files we have. And plus, it would be out of Runner's scope imho.

@novemberborn @vdemedes
I could reverse the behaviour by checking that title is not null, but to me as a user it'd feel weird to see tests without a title running when I've asked for a specific set of titles.
The one time I feel a non-titled test should run, would be with either no match pattern supplied, or --match=*. But I don't see why one would ever want to supply such a pattern.

@vdemedes

It can't be moved to Runner, because we display full test titles only when there are multiple files and only CLI knows how many test files we have.

This isn't about test title prefixes. The runner calls TestCollection#add() which changes the title beyond what was determined in the runner, even though the runner has access to the same arguments.

The one thing I haven't done is setting up an end-to-end test in test/cli to ensure it all works™. Do you have any suggestions/pointers I can take with me on that voyage?

See the "watcher works" test, though it does a few more things than are required here.

@novemberborn Oh I see, I mixed that up. In that case, @kasperlewau's logic seems very reasonable:

Tests without a title will not run if any match pattern is supplied.

it'd feel weird to see tests without a title running when I've asked for a specific set of titles.

And still title manipulation feels out of runner's scope, so I'd leave it in test collection.

My point is that the runner calls another method which figures out an implicit title. Those are not matched. I guess it's OK to say that match only applies to explicit titles but it shouldn't be too hard to apply it to implicit ones too.

Ok, I see your point. What about merging this PR as it is and opening a new issue to extend this functionality? I see some refactoring work involved here.

Ok, I believe that's the e2e test for --match in the bag. Fingers crossed Travis doesn't kick me in the groin : )

I don't think it makes sense to match on implicit titles. It should IMHO only match on what people write out in their test file.

What about merging this PR as it is and opening a new issue to extend this functionality? I see some refactoring work involved here.

Yes that's fair.

I don't think it makes sense to match on implicit titles. It should IMHO only match on what people write out in their test file.

That's fair too. Not sure we want to encourage test(function myTestTitle (t) { style tests anyway.
Apologies for the distraction :smiley:

I realised just now I took care of @sindresorhus's comment about repeatabilty - but I did so in the 'wrong' place :smile:
I'll take care of that on my way to work!

Should this use exclusive rather than skipped? That is, set exclusive for matching tests. Perhaps in the Api report an error when no tests match. That makes --match similar to adding .only in the test files.

Should this use exclusive rather than skipped? That is, set exclusive for matching tests.

Right off the bat that sounds better to me @novemberborn. I can't really comment further than that - I'm not familiar enough with the difference(s) between skip and exclusive when put into the context of say --watch or something of the sort. I bet you do though : )
If anything it would change the conditional statement to something slightly more pretty.

I'm not familiar enough with the difference(s) between skip and exclusive when put into the context of say --watch or something of the sort. I bet you do though : )

It'll probably be OK, but we can make whatever improvements are needed once this lands.

What I mean to say is that the semantics of match fit inclusion better than exclusion in my mind. Albeit that's a very fine line.

Here's another angle. If I have a .only test and I invoke AVA using a --match, what is the expected behavior? IMO --match should win (just like CLI flags override package.json config). That would be easier to achieve if the match implementation changes the exclusive field of the test metadata. Non-matching exclusive tests need to have that field set to false.

If I have a .only test and I invoke AVA using a --match, what is the expected behavior? IMO --match should win.

Agreed. It's seldom that I pass CLI args to my test runner, and when I do pass them I expect them to be king of the hill.
I'm not so fussed about the easier way (oh gosh, nitpicking...) to achieve the end goal, but more so that the produced code is simpler. If switching to exclusive rather than skip to achieve CLI dominance would generate simpler code in the end, I'm all for it :+1:

I think it might make it easier to reason about the side-effects, e.g. in reporter output. Both --match and .only can cause a lot of tests not to run that would have run otherwise, unlike .skip which only causes skipped tests not to run.

@novemberborn Does this still apply https://github.com/sindresorhus/ava/pull/592#discussion_r54730961 ?

@novemberborn Is the exclusive change something you want to do in this PR or can it be a follow-up?

@sindresorhus,

Does this still apply #592 (comment) ?

That's been resolved.

Is the exclusive change something you want to do in this PR or can it be a follow-up?

I think we should do it here, shouldn't be too much work. As the PR stands now, if you have a .only test and use --match it will only be applied to .only tests. I think that's a bug.

if you have a .only test and use --match it will only be applied to .only tests. I think that's a bug.

Agreed.

I've changed some stuff around to add the exclusive flag onto the given test if its title matches --match, leaving the skipped flag for .skip.
As expected, non-matching tests no longer add to testCount, nor skipCount. That could open up for some confusion, as it's a fine line and a very personal opinion on whether a match pattern excludes or includes, but perhaps that is outside the scope what we want to achieve.
Should we introduce another counter to keep track of the tests that were not run due to the --match flag? ignoreCount?
Sounds a lot like skipCount to me :unamused:, plus the fact that now we'd have to juggle even more words around; skip, only, match, exclusive, ignore. Gut feeling says that could be simplified, I'm just not quite sure how.

Also, consider the following:
ava --match=*oo
test.skip('moo', t => {
  t.pass();
});

test('foo', t => {
  t.pass();
});

If --match takes precedence over .only, should it also override .skip?

As expected, non-matching tests no longer add to testCount, nor skipCount. That could open up for some confusion, as it's a fine line and a very personal opinion on whether a match pattern excludes or includes, but perhaps that sort of balancing act is outside the scope of what we're trying to achieve here.
Should we introduce another counter to keep track of the tests that were not run due to the --match flag? ignoreCount?
Sounds a lot like skipCount to me , plus the fact that now we'd have to juggle even more words around; skip, only, match, exclusive, ignore. Gut feeling says that could be simplified, I'm just not quite sure how.

#471 covers this. Hopefully we can solve it in one fell swoop for all cases.

If --match takes precedence over .only, should it also override .skip?

I'd argue no. You'd use .skip because a test is broken. You'd use .only or --match to select specific tests.

I've changed some stuff around to add the exclusive flag onto the given test if its title matches --match, leaving the skipped flag for .skip.

 No worries if you were waiting for feedback on your questions, but figured you might have forgotten ;-)

#471 covers this. Hopefully we can solve it in one fell swoop for all cases.

Gotcha. I mean something like that could become a mess quite quickly. Glad it's on the roadmap :)

I'd argue no. You'd use .skip because a test is broken. You'd use .only or --match to select specific tests.

I suppose you're right about that one. I was halfway through an explanation on why I feel it should be so, but halfway through I realised I ended up at the same conclusion - no.

Doesn't look like you pushed any commits yet though? No worries if you were waiting for feedback on your questions, but figured you might have forgotten ;-)

I'd say that was a 50/50. I was curious as to if it made sense from those point of views, and my net connection died on the train back home as I was about to push. Hadn't gotten around to it yet :smile:

Rebased and good to go from my end. Not sure I nailed the exclusive flag, so I would love to get another checkup on this PR.
Just waiting for those CI's to finish their business.

LGTM! I'm in the midst of overhauling the readme (see #597) so don't worry too much about those changes.

@novemberborn I agree with the changes you proposed to the Readme. Would you like for me to handle those, or defer to #597?

I agree with the changes you proposed to the Readme. Would you like for me to handle those, or defer to #597?

Seems fine as it is now. I'll give it another read within the context of the changes in #597 and adjust as necessary once this is in master.
Not sure why Travis failed in Node 5.

Not sure why Travis failed in Node 5.

Beats me :confused: Can we trigger a rerun and see what's up?

Can we trigger a rerun and see what's up?

Just did. Didn't realize I had that power :muscle:

Just did. Didn't realize I had that power :muscle:

And with that, I will calmly await any further feedback/instructions.
Aside; I was halfway past throwing my MBP right out the window in my previous PR. Not having the power to trigger a re-run had me shouting in agony, being someone who loves to push with -f and pull with --rebase.
:beer:

Not having the power to trigger a re-run had me shouting in agony

Just amend the latest commit without any new changes and force push and you trigger a rerun ;)

Just amend the latest commit without any new changes and force push and you trigger a rerun ;)

See I'm quite sure I did just that, but it simply wouldn't rebuild on Travis/AppVeyor, which struck me as odd as I am an avid wielder of the --force. Nevertheless, we be green now :smile:

@sindresorhus @vdemedes @jamestalmage LGTY?

Please make sure to rebase on master to fix a linting issue and run the latest XO configuration against your PR. Thanks!

Please make sure to rebase on master to fix a linting issue and run the latest XO configuration against your PR. Thanks!

Done!

LGTM2. Landed.
Superb work on this @kasperisager, and thanks for persevering through our extensive review. This turned out really good! :)

My pleasure! And thanks :)
Sidenote; Didn't get no notification. As it turns out, there actually could be too many Kasper's around @novemberborn @sindresorhus :smiley:!


Test

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

server-2013/migrate-response-groups.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


⚠ Bofrost.de has degraded performance

In 9cf9873, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1280 ms

Resolved: Bofrost.de performance has improved in e2f85c1.


fix byte slice column type

close https://github.com/go-gorm/gen/issues/496

[X] Do only one thing
[ ] Non breaking API changes (not sure about this)
[x] Tested

What did this pull request do?
gen.FieldType("...", "[]byte") will generate query field field.NewBytes ([]byte) instead of field.NewField  (driver.Valuer)

https://github.com/go-gorm/gen/blob/37a8cdeb26285e884b7fd7f678ed3551f9851f87/internal/model/base.go#L173-L177
bytes is not a go built-in type.
User Case Description

fix in #499


Fix rendering of workflow import snippet

See https://github.com/galaxyproject/training-material/pull/3163#issuecomment-1042072859

@bgruening can you approve if you have a spare minute?

Thanks @hexylena!


Uncaught TypeError: Cannot read property 'yadcfState' of null

I see the following with 0.9.4-beta.28 (via yarn).
jquery.dataTables.yadcf.js:4275 Uncaught TypeError: Cannot read property 'yadcfState' of null
    at saveTextKeyUpState (jquery.dataTables.yadcf.js:4275)
    at keyUp (jquery.dataTables.yadcf.js:4261)
    at jquery.dataTables.yadcf.js:4269
    at jquery.dataTables.yadcf.js:901

This happens when I type into a column search field.
This is defined as:
      {column_number: 0,
       filter_type: 'text',
       filter_default_label: 'Filter Vuln IDs',
       regex_check_box: true,
       filter_delay: 1500
      },

Not sure if relevant, the datatables column has a render property:
      {
        data: 'vulnerability__vuln_id',
        render: (data, _type, row, _meta) =>
          `
<a href="${location.origin}/airshield/findings/${row.id}/">
${data}
</a>
`
      },

Sorry, if this is not a bug in yadcf.

I just upgraded to beta.40 by hand (28 is available on yarn, so I preferred that earlier) and can reproduce the behaviour there as well.

In fact, the error disappears when I remove the render function.

Uhm, when I reinstall (beta 28 or 40, doesn't matter) this behaviour disappears. Maybe I had the wrong beta (possibly 5, from npm) installed here.

Ugh, it just reappeared again, unfortunately. I'll see what actually reproduces this on my part.

testcafe reproduces this reliably, but I've seen this error in the actual browser console as well... still investigating...

(I hope you don't mind if I'm keeping notes on this closed issue. I'll reopen it when I'm sure that it's a bug in yadcf as opposed to my own code or testcafe. Testcafe is probably out of the picture because I could at some point reproduce it in the browser by hand.)
Firefox (Quantum 68.4.2esr (64-Bit)) gives a different error (the above was with chromium ( 80.0.3987.87 (openSUSE Build) (64-Bit))):
JavaScript error details:
oTable.fnSettings(...).oLoadedState is null:
saveTextKeyUpState@http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:40200:68
keyUp@http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:40186:5
textKeyUP/<@http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:40194:6
airshield</</</yadcf</yadcfDelay</</timer<@http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:36860:6

I think that's because firefox doesn't correctly parse the source-map (no idea why). From the line numbers, it looks like it's this function:
		function saveTextKeyUpState(oTable, table_selector_jq_friendly, column_number, regex_check_box, null_checked, exclude) {
			if (oTable.fnSettings().oFeatures.bStateSave === true && oTable.fnSettings().oLoadedState.yadcfState) {
				if (oTable.fnSettings().oLoadedState.yadcfState !== undefined && oTable.fnSettings().oLoadedState.yadcfState[table_selector_jq_friendly] !== undefined) {
					oTable.fnSettings().oLoadedState.yadcfState[table_selector_jq_friendly][column_number] = {
						regex_check_box: regex_check_box,
						null_checked: null_checked,
						exclude_checked: exclude
					};
				} else {
					yadcfState = {};
					yadcfState[table_selector_jq_friendly] = [];
					yadcfState[table_selector_jq_friendly][column_number] = {
						regex_check_box: regex_check_box,
						exclude_checked: exclude,
						null_checked: null_checked
					};
					oTable.fnSettings().oLoadedState.yadcfState = yadcfState;
				}
				oTable.fnSettings().oApi._fnSaveState(oTable.fnSettings());
			}
		}

The problem in chromium is in the same line of code, the phrasing is just different
JavaScript error details:
TypeError: Cannot read property 'yadcfState' of null
    at saveTextKeyUpState (http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:40200:93)
    at keyUp (http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:40186:5)
    at http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:40194:6
    at http://<IP_ADDRESS>:8003/airshield/static/js/airshield.bundle.js:36860:6

So for some reason oTable.fnSettings() returns a null value.
From the way it behaves (sometimes the error is thrown after the click, sometimes on keyboard input) I would suspect it's related to timing.

I'm confused now. fnSettings is legacy Datatables api isn't it? There's a function getSettingsObjFromTable which smartly dispatches to get the right thing, but in several parts of the code, fnSettings is still used.
I'm using datatables 1.10.19 (could try upgrading to 1.10.20 if that helps).

(Not sure that this is a bug in yadcf yet, but maybe you can clarify things.)

I'm using datatables 1.10.19 (could try upgrading to 1.10.20 if that helps).

Unfortunately the minor datatables upgrade does not help.

fixed by https://github.com/vedmack/yadcf/pull/615


Prefill Diablo IV and World of Warcraft error out

Describe the bug
Running prefill both Diablo IV and World of Warcraft error near the end of the download
Additional Details

Windows 11
Elerunne
Region : Australia

Screenshots

unsure if the error occurs for other games in Battlenet. Also could the Beta of World of Warcraft, and the Mists of Padaria remix impact the caching options?

Also could the Beta of World of Warcraft, and the Mists of Pandaria remix impact the caching options?

Neither the Beta nor Pandaria should have any affect on this.  With the way that Battlenet stores files on their CDN there is a lot of sharing between different "versions" or "branches" such that their files are one in the same.

Problem resolved there was an issue with my NAS throwing errors. Resolved those errors by running xfs-repair against a drive, and running a parity check. thanks


celery error????? help me

CELERY ERROR WHATS UP?

Looks like celery is having a hard time starting. Looking at the celery logs would be the best thing to do. They are located in source/resources/logs. There is a stderr, stdout, and worker log. The first place I would look would be in the stderr log.

Support and development for this project has been halted. We appreciate the help in clearing up these issues, and they will be solved when development is resumed.
Thanks.

I discovered a similar issue, when I looked at source/resources/logs in my deployment I found errors for celery in trying to locate java.
I installed open jdk 1.7 and celery is now staying up, but have other issues I'm also working to resolve
sudo apt-get install openjdk-7-jre

thank you
Date: Thu, 28 Apr 2016 22:11:16 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [memex-explorer/memex-explorer] celery error????? help me (#767)
I discovered a similar issue, when I looked at source/resources/logs in my deployment I found errors for celery in trying to locate java.
I installed open jdk 1.7 and celery is now staying up, but have other issues I'm also working to resolve
sudo apt-get install openjdk-7-jre
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


Implement reset

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets

Documentation

License
MIT

What's in this PR?
Add the possiblity to reset the client
Why?
To avoid side effects when using the mock on multiple tests
Example Usage
$client = new Client();

$response = $this->createMock('Psr\Http\Message\ResponseInterface');
$client->addResponse($response);

$client->reset();
// responses and exceptions stacks are empty now

To Do

[ ] Documentation needed ?

I made some changes, it should be better now

thank you!


UCB staff e-mail address compilation (for upcoming e-mail changes)

o	OAC voro accounts (~43 library.berkeley.edu accounts)
o	CSphere Registry accounts (6 accounts)
o	Mailchimp (~175 library.berkeley.edu subscribers)
o	Freshdesk (63 library.berkeley.edu subscribers Contacts)

Compilation spreadsheet in progress at https://docs.google.com/spreadsheets/d/19sgxKElj7qzBASKr3sQKdh-Ud9guT1qQI0DCSvDqxHk/edit?usp=sharing

@aturner I think we've zapped non-active users in all relevant apps


[Website update] Hero banner carousel

Describe the solution you'd like
Place at the top of the front page to allow a rotating carousel of linked images (or text), highlighting blog posts, upcoming events, news, etc.
Enable the project to draw attention to new and interesting content pieces when the community visits the site

meeting notes:
2022-1104 - Hannah - has the examples - will work with Patti to get to templates for blog posts, release anncmnt, events/comm meetings.
2022-1025 - @nateynateynate - will start on code. Hannah Uselman will create a few examples. (Image / Hero banner @krisfreedain used before is based on OpenSearchCon image in this post) Kris & Nate talk with the hero banner code example
Example dimensions: 1128x191

@jhmcintyre , Patti , Hannah - working on final decisions on content wishlist with a vendor to develop templates. Meeting set for next week.

@pajuric - we should have image templates early next week.

@nateynateynate will also look into making the carousel start on a random image rather the 1st - therefore changing the experience each time someone visits.

@nateynateynate  currently working on this.
@Uselmh  - working on images
we'll have a max 3 to 5 in rotation

Amazing work done by @nateynateynate!
Complete with https://github.com/opensearch-project/project-website/pull/1301


tickets/PREOPS-4606: Add schedview-snapshot service

This is an initial attempt to add the schedview-snapshot survice, so far untested. But, at least some version needs to be checked into main before I can test and debug it, so this pull request is intended to get enough on main that I can begin testing and debugging.

I'm wrong about .helmignore, which is now part of our templates.  That's fine.


Update namespace

As decided in php-http/httplug#98

Http\CurlClient\Client?

Http\Client\Curl\Client

Done.

Nice :+1:

:+1:


metrics not available through fleet-manager endpoint

Attempts to read the metrics currently fail within the fleet-manager.  This occurs because there is no observability component in the kas-installer installed environment.
Internally, fleet-manager fails like this:
E0715 10:17:40.368875       1 api.go:262] error from metric Post "/api/metrics/v1/test/api/v1/query": can't request metrics without auth
E0715 10:17:40.368900       1 metrics.go:55] error getting metrics: KAFKAS-MGMT-9: failed to retrieve metrics

and returns 500 back to the caller.

Discussions with @pb82 and @JameelB initially had us explored configuring fleetmanager to point directly at the prometheus end-point, however, this doesn't work:

the Observatorium tenant is appended to the URL. the URL Observatorium specific and not compatible with raw Prometheus

https://github.com/pb82/kas-fleet-manager/blob/main/pkg/client/observatorium/client.go#L52

@MikeEdgar pointed out to me that kas-installer does have the capability to install observatorium.  I'll give this a go and experiment with wiring fleetmanager to it.


Add version: GoLang.Go.1.17 version 1.17.8

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run


fix #2343

A similar PR may already be submitted!
Please search among the Pull request before creating one.
Thanks for submitting a pull request! Please provide enough information so that others can review your pull request:
Before opening a pull request, please make sure all the tests pass locally by running npm test.
Summary

This PR fixes/implements the following bugs/features

[x] https://github.com/niklasvh/html2canvas/issues/2343

Explain the motivation for making this change. What existing problem does the pull request solve?

Test plan (required)
Demonstrate how the issue/feature can be replicated. For most cases, simply adding an appropriate html/css template into the reftests should be sufficient. Please see other tests there for reference.
Code formatting
Please make sure that code adheres to the project code formatting. Running npm run format will automatically format your code correctly.
Closing issues

Fixes #

@wengcan Can you please publish your fork to npm so that I can use it instead?

@wengcan Thanks for the fix! My Safari users say the fix works.


ng-show, ng-hide

Document in Readme

@LiamDotPro, note that this is much different from:
## ng-if

```jsx

  { foo == "bar" ?
show me if foo's value is "bar"!
 : null }

Because the elements are actually on the dom, but CSS is in charge of showing/hiding them.
I'm sure you've written jQuery code like $('.my-class').addClass('hide'); $('.my-class').removeClass('hide'); (obviously $.fn.hide and $.fn.show are more approriate, but you get what I mean).


[DS-2779] Fix versioning due to service api

This pull request has one controversial change in it, it makes the DSpaceObject, Handle relation a OneToMany.
If versioning is enabled a single item can have multiple handles. I have kept the getHandle() method on DSpaceObject for now since it has over 600 usages.
The getHandle() will always attempt to return the non versioned handle, if ever required a getHandles() method is also available.

This breaks several unit tests in SupervisedItemTest (see Travis CI for more info). At a minimum those would need to be fixed (and this change might warrant additional unit tests).

Added an additional commit containing additional unit tests for the handles. It brought to light some things that weren't working as expected, so it is great to have them.
I haven't been able to fix the unit tests just yet (I don't understand why they are breaking ? The handle fix is unrelated to workspace items....)

Before we decide about this PR I kindly ask to discuss my PRs against DS-1348 and the ticket itself. I see some problems with the concept of "non versioned handles". For more details please take a look in DS-1348 and its comments.

For backward compatability we do not have a choice here. If anyone used the versioning feature as implemented since DSpace 4 we will need this patch.
I rebased this PR and added some more changes: The config/spring/api/versioning-service.xml had a minor error and the JSPUI had problems deleting Versions. I created a PR with those changes against Kevin's branch. In case it is more handy to merge this one first and create another PR then, I can do that as well.

I was able to fix the unit tests. I'm not quite pleased with the solution as it requires a context.complete() to trigger the sql that links the groups to the workspace items.
I have no idea how my versioning fixes influenced this behavior.
It shouldn't cause any issues in the code since there will be little need to first add & then retrieve by query .

Thanks @KevinVdV for taking care of this. Would be great if we could merge this asap!
At the DINI-Jahrestagung (conference in the last two days) I talked to someone who resolved DS-1349 for XMLUI. I would like to ask him to create a PR, but this would be much simpler if this PR gets merged first.
+1

+1 We aren't using versioning anywhere, but if people are satisfied with this, then it sounds good to me.

+1 by inspection.  We aren't using versioning yet, but what I see looks sound.


Add mitosis oversampling via class-aware sampling during training

This adds minority mitosis oversampling via class-aware sampling in
which we sampling even numbers of mitosis and normal patches for each
mini-batch, sampling with replacement from the mitosis samples until we
run out of normal samples.

@feihugis can you review this?  not sure if it is helping yet, but I think it at least works correctly.


FileDownloadOutOfSpaceException

com.liulishuo.filedownloader.exception.FileDownloadOutOfSpaceException: The file is too large to store, breakpoint in bytes:  0, required space in bytes: 3485930, but free space in bytes: 1305600
at com.liulishuo.filedownloader.download.DownloadLaunchRunnable.handlePreAllocate(DownloadLaunchRunnable.java:759)
at com.liulishuo.filedownloader.download.DownloadLaunchRunnable.run(DownloadLaunchRunnable.java:271)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
at java.lang.Thread.run(Thread.java:818)

The description of the exception is clear, your storage space is not enough, so you need to remove some files you don't want to save your storage space.


TypeError when rolling dice

Hiya, this is my first issue on github because I program as hobby and generally I find workarounds or fixes but this one has me stumped. I tried doing everything I could think of.
I have a react app running only the most recent version of MUI. I have installed dice-box, set up the assets folder, configured the DiceBox and then initialized it through useEffect. Whenever I try to call Dice.roll() I get the following three errors:
ERROR
Cannot read properties of undefined (reading 'diceAvailable')
TypeError: Cannot read properties of undefined (reading 'diceAvailable')
    at vl.roll (http://localhost:3000/static/js/bundle.js:65236:68)
    at onClick (http://localhost:3000/static/js/bundle.js:943:26)
    at HTMLUnknownElement.callCallback (http://localhost:3000/static/js/bundle.js:31004:18)
    at Object.invokeGuardedCallbackDev (http://localhost:3000/static/js/bundle.js:31048:20)
    at invokeGuardedCallback (http://localhost:3000/static/js/bundle.js:31105:35)
    at invokeGuardedCallbackAndCatchFirstError (http://localhost:3000/static/js/bundle.js:31119:29)
    at executeDispatch (http://localhost:3000/static/js/bundle.js:35262:7)
    at processDispatchQueueItemsInOrder (http://localhost:3000/static/js/bundle.js:35288:11)
    at processDispatchQueue (http://localhost:3000/static/js/bundle.js:35299:9)
    at dispatchEventsForPlugins (http://localhost:3000/static/js/bundle.js:35308:7)
    at http://localhost:3000/static/js/bundle.js:35468:16
    at batchedUpdates$1 (http://localhost:3000/static/js/bundle.js:49886:16)
    at batchedUpdates (http://localhost:3000/static/js/bundle.js:30852:16)
    at dispatchEventForPluginEventSystem (http://localhost:3000/static/js/bundle.js:35467:7)
    at dispatchEventWithEnableCapturePhaseSelectiveHydrationWithoutDiscreteEventReplay (http://localhost:3000/static/js/bundle.js:32974:9)
    at dispatchEvent (http://localhost:3000/static/js/bundle.js:32968:9)
    at dispatchDiscreteEvent (http://localhost:3000/static/js/bundle.js:32945:9)

ERROR
Cannot read properties of undefined (reading 'diceAvailable')
TypeError: Cannot read properties of undefined (reading 'diceAvailable')
    at vl.roll (http://localhost:3000/static/js/bundle.js:65236:68)
    at onClick (http://localhost:3000/static/js/bundle.js:943:26)
    at HTMLUnknownElement.callCallback (http://localhost:3000/static/js/bundle.js:31004:18)
    at Object.invokeGuardedCallbackDev (http://localhost:3000/static/js/bundle.js:31048:20)
    at invokeGuardedCallback (http://localhost:3000/static/js/bundle.js:31105:35)
    at invokeGuardedCallbackAndCatchFirstError (http://localhost:3000/static/js/bundle.js:31119:29)
    at executeDispatch (http://localhost:3000/static/js/bundle.js:35262:7)
    at processDispatchQueueItemsInOrder (http://localhost:3000/static/js/bundle.js:35288:11)
    at processDispatchQueue (http://localhost:3000/static/js/bundle.js:35299:9)
    at dispatchEventsForPlugins (http://localhost:3000/static/js/bundle.js:35308:7)
    at http://localhost:3000/static/js/bundle.js:35468:16
    at batchedUpdates$1 (http://localhost:3000/static/js/bundle.js:49886:16)
    at batchedUpdates (http://localhost:3000/static/js/bundle.js:30852:16)
    at dispatchEventForPluginEventSystem (http://localhost:3000/static/js/bundle.js:35467:7)
    at dispatchEventWithEnableCapturePhaseSelectiveHydrationWithoutDiscreteEventReplay (http://localhost:3000/static/js/bundle.js:32974:9)
    at dispatchEvent (http://localhost:3000/static/js/bundle.js:32968:9)
    at dispatchDiscreteEvent (http://localhost:3000/static/js/bundle.js:32945:9)

ERROR
Uncaught TypeError: Cannot read properties of undefined (reading 'byteLength')
    at handleError (http://localhost:3000/static/js/bundle.js:61652:58)
    at http://localhost:3000/static/js/bundle.js:61671:7

Here is what my code looks like:
import DiceBox from '@3d-dice/dice-box'

const Dice = new DiceBox({
  id: 'dice-box',
  assetPath: './assets/dice-box/'
})

function Component(props) {

  useEffect(()=>{
    Dice.init()
  })

  return(
  <div>
      <Button onClick={()=>{Dice.roll('2d10')}}></Button>
  </div>
  )
}

Missed this message from the console:
ede42b55-3d8b-4a07-b18f-ca7c1ace6a7a:1
Uncaught (in promise) TypeError: Failed to execute 'fetch' on 'WorkerGlobalScope': Failed to parse URL from http://localhost:3000./assets/dice-box/ammo/ammo.wasm.wasm
    at ede42b55-3d8b-4a07-b18f-ca7c1ace6a7a:1:5355
    at ede42b55-3d8b-4a07-b18f-ca7c1ace6a7a:1:5578
    at new vl (ede42b55-3d8b-4a07-b18f-ca7c1ace6a7a:1:5585)
    at Dl (ede42b55-3d8b-4a07-b18f-ca7c1ace6a7a:1:130645)
    at self.onmessage (ede42b55-3d8b-4a07-b18f-ca7c1ace6a7a:1:129185)

Makes me think it is something to do with the pathing to the assets folder, but the one time I managed to make it work was using the CDN version, but then that started to give me the same error. And going back to the CDN version is also a no-go as in the end this app will run offline.

Update: Turns out it was a webassembly issue (something I don't know the first thing about, yay!) and there are a few issues closed that mention the exact problem I'm facing (though the fixes they mention sound like arcane cyphers to me). Tried migrating my whole app to Vite to no avail. Tried installing vite-plugin-wasm and adding it to vite.config.js, also did nothing. I'm on the verge of giving up... been trying to fix this for days now and it feels like it's something super simple but I'm too much of a rookie to know how to fix.
37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1  wasm streaming compile failed: TypeError: Failed to execute 'compile' on 'WebAssembly': Incorrect response MIME type.  Build with -s ASSERTIONS=1 for more info.
    at Nt (37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1:3010)
    at 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1:4706
(anonymous) @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
Promise.catch
n @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
(anonymous) @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
Promise.then
(anonymous) @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
Promise.then
(anonymous) @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
(anonymous) @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
vl @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
Dl @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
self.onmessage @ 37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1
37fb5aa4-77a8-4a78-80f9-88bb23e55fce:1  Ammo file 'http://localhost:5173/assets/dice-box/ammo/ammo.wasm.wasm' not found. This is required to run the physics simulation. Is your 'assetPath' config option correct and is there an 'ammo' folder with the 'ammo.wasm.wasm' file?   Build with -s ASSERTIONS=1 for more info.
    at Nt (1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1:3010)
    at 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1:4706
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Promise.catch
n @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Promise.then
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Promise.then
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
vl @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Dl @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
self.onmessage @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1  Ammo file 'http://localhost:5173/assets/dice-box/ammo/ammo.wasm.wasm' not found. This is required to run the physics simulation. Is your 'assetPath' config option correct and is there an 'ammo' folder with the 'ammo.wasm.wasm' file? Ensure your server has the proper mime type for '.wasm' files.
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Promise.catch
n @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Promise.then
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Promise.then
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
(anonymous) @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
vl @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
Dl @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
self.onmessage @ 1b2ddada-4897-4bfa-9ef7-4ee33d22ba27:1
145f764b-44b0-4b35-b427-a223a93fc6ec:14 Babylon.js v5.57.1 - WebGL2 - Parallel shader compilation

Hey, I'll see if I can help you out. Incorrect MIME type means your server needs some configuring. What are you using for your server setup?
You also mention running that app in offline mode, which leads me to believe you're trying to get around a fetch call.
Issue https://github.com/3d-dice/dice-box/issues/102 mentions using the dice box in an electron app. Is that something you're trying to do?

Hey! Thanks for taking the time to respond and thanks for such a cool project!
I've never really configured anything, just created a vite app with react and run it with npm run dev. That's pretty much it. As for offline mode, I'm seeing more and more problems with that going forward for urnelated reasons, but in the end I'll be converting the app to also run on mobile through a web emulator for offline TTRPG playing. So I don't plan on using electron, no. But then again, seeing more and more issues with that as I move forward.

I just added a demo project at https://github.com/3d-dice/dice-box/tree/main/demo/react-project. Try running that. All you need to do is npm i then npm run dev and it should work.

That worked! Merged my app with that demo project and it totally works now! To someone like me that is beyond sorcery. Thanks a bunch! <3

Awesome. Glad it worked. Sometimes, getting the initialization order correct is the trick. You'll notice that the DiceBox class is created outside the App component but init runs inside the component. I think that's the key to mixing this vanilla JS module with React.
If you're interested in seeing other React demos I also have these:
https://codesandbox.io/p/sandbox/react-roller-attributes-v1-1-3-65uqhv?file=%2Fsrc%2FApp.js
https://codesandbox.io/p/sandbox/react-roller-advanced-notation-v1-1-3-rz0nmr?file=%2Fsrc%2FApp.js