AJ Schmidt comments

Results 68 comments of


                                            AJ Schmidt

`git_status` deleted includes both unstaged and staged deletions

FWIW, I had a PR below that was rejected which added the `git` granularity from `powerlevel10k` to `starship`. - https://github.com/starship/starship/pull/3276

Feature: Let slots be wrapped in if statements to avoid "must be a child of a component" error.

+1. this would help make my code less verbose in some cases.

Bifurcate Dependency Lists [skip-gpuci]

> Is this really a 22.08 PR? Nope, I will re-target this PR for `22.10`

[REVIEW] Add cuda_event type

Removing `ops-codeowners` from the required reviews since it doesn't seem there are any file changes that we're responsible for. Feel free to add us back if necessary.

[skip gpuci] Add GitHub Actions workflow for PRs

> It seems you would to have slugs as display name for the jobs. I'm not sure to understand why? I totally agree that we need to rethink about the...

Deleting a Node does not remove the associated MQTT topic

Dropping a StackOverflow link here for reference: https://stackoverflow.com/questions/42218056/how-to-remove-a-published-topic I'm not sure if removing a topic is a thing.

Add python benchmarks for `from_geopandas`

> What is the integration repository that you refer to? I will do so when I know. Ah sorry! I had the link in my clipboard and forgot to paste...

ENH Replace gpuci_conda_retry with gpuci_mamba_retry

rerun tests

Add cucim.skimage.feature.match_descriptors

Removing `ops-codeowners` from the required reviews since it doesn't seem there are any file changes that we're responsible for. Feel free to add us back if necessary.

Fix complex type bug with correlate

@awthomp I'm seeing a lot of failures (see screenshot) related to the `test_convolution.py` file which was touched in this PR. can you double-check the tests to ensure that they are...