Using flutter create to repair a project with the --empty option deletes test folder

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Run the following commands sequentially:
flutter create my_app --empty
mkdir my_app/test
touch my_app/test/some_test.dart
cat my_app/test/some_test.dart  # prints nothing, finds this file
flutter create my_app --empty  # I want to repair this project, but I don't want to add any boilerplate
cat my_app/test/some_test.dart  # ! No such file or directory

Expected results
some_test.dart shouldn't be deleted since we are just trying to repair this project after its initial creation.
Actual results
The whole test folder is now gone.
Alternatives I've considered
Using flutter create without the --empty option. But that'll add the boilerplate I want to avoid.
Context
I'm trying to set-up my own mason brick, with several files in both lib and test which. flutter create . --empty is handy since it won't generate useless (within this context) files.
Code sample
N.A.
Screenshots or Video
N.A.
Logs
N.A.
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.2, on Ubuntu 22.04.3 LTS 6.2.0-32-generic, locale en_US.UTF-8)
    • Flutter version 3.13.2 on channel stable at /home/venir/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ff5b5b5fa6 (4 weeks ago), 2023-08-24 08:12:28 -0500
    • Engine revision b20183e040
    • Dart version 3.1.0
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc4)
    • Android SDK at /home/venir/Android/Sdk
    • Platform android-33, build-tools 33.0.0-rc4
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build 18.0.2-ea+9-Ubuntu-222.04)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] VS Code (version unknown)
    • VS Code at /snap/code/current
    • Flutter extension version 3.72.0
    ✗ Unable to determine VS Code version.

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.2.0-32-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 117.0.5938.62

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thanks for the report @lucavenir
I followed the steps to replicate the reported behavior but getting below:
dhs@Dhss-MacBook-Pro ncflutter % touch my_test_app/test/some_test.dart
touch: my_test_app/test/some_test.dart: No such file or directory
dhs@Dhss-MacBook-Pro ncflutter %

ie, while trying to run touch to create some_test.dart, it doesn't seem to create the said file and shows the message no such file or directory.

I updated the steps: I forgot to let you create the test folder, first 😄

Thanks for the update. I followed the updated steps but don't seem to replicate the reported behavior. See below for the steps taken:
dhs@Dhss-MacBook-Pro my_test_app % ls
README.md		lib			pubspec.lock
analysis_options.yaml	linux			pubspec.yaml
android			macos			web
ios			my_test_app.iml		windows
dhs@Dhss-MacBook-Pro my_test_app % mkdir my_test_app/test
mkdir: my_test_app: No such file or directory
dhs@Dhss-MacBook-Pro my_test_app % mkdir test
dhs@Dhss-MacBook-Pro my_test_app % ls
README.md		linux			test
analysis_options.yaml	macos			web
android			my_test_app.iml		windows
ios			pubspec.lock
lib			pubspec.yaml
dhs@Dhss-MacBook-Pro my_test_app % touch my_app_test/test/some_test.dart
touch: my_app_test/test/some_test.dart: No such file or directory
dhs@Dhss-MacBook-Pro my_test_app % cd test
dhs@Dhss-MacBook-Pro test % touch some_test.dart
dhs@Dhss-MacBook-Pro test % ls
some_test.dart
dhs@Dhss-MacBook-Pro test % cat some_test.dart
dhs@Dhss-MacBook-Pro test % cd ..
dhs@Dhss-MacBook-Pro my_test_app % flutter create my_test_app --empty
Signing iOS app for device deployment using developer identity: "Apple
Development: Darshan Kawar (AV76JL68FU)"
Creating project my_test_app...
Resolving dependencies in my_test_app... (1.6s)
Got dependencies in my_test_app.
Wrote 128 files.

All done!
You can find general documentation for Flutter at: https://docs.flutter.dev/
Detailed API documentation is available at: https://api.flutter.dev/
If you prefer video documentation, consider:
https://www.youtube.com/c/flutterdev

In order to run your empty application, type:

  $ cd my_test_app
  $ flutter run

Your empty application code is in my_test_app/lib/main.dart.

dhs@Dhss-MacBook-Pro my_test_app % cat my_test_app/test/some_test.dart
cat: my_test_app/test/some_test.dart: No such file or directory
dhs@Dhss-MacBook-Pro my_test_app % ls
README.md		linux			pubspec.yaml
analysis_options.yaml	macos			test
android			my_test_app		web
ios			my_test_app.iml		windows
lib			pubspec.lock
dhs@Dhss-MacBook-Pro my_test_app % cd test
dhs@Dhss-MacBook-Pro test % ls
some_test.dart
dhs@Dhss-MacBook-Pro test % cd

If you see the last section above, it does seem to have test folder and underlying some_test.dart file.
dhs@Dhss-MacBook-Pro my_test_app % cd test
dhs@Dhss-MacBook-Pro test % ls
some_test.dart
dhs@Dhss-MacBook-Pro test % cd

I tried above using latest master version.

Thanks for the update. I do see the same behavior as reported now.
dhs@Dhss-MacBook-Pro ncflutter % flutter create another_app --empty
Signing iOS app for device deployment using developer identity: "Apple
Development: Darshan Kawar (AV76JL68FU)"
Creating project another_app...
Resolving dependencies in another_app... (1.9s)
Got dependencies in another_app.
Wrote 128 files.

All done!
You can find general documentation for Flutter at: https://docs.flutter.dev/
Detailed API documentation is available at: https://api.flutter.dev/
If you prefer video documentation, consider:
https://www.youtube.com/c/flutterdev

In order to run your empty application, type:

  $ cd another_app
  $ flutter run

Your empty application code is in another_app/lib/main.dart.

dhs@Dhss-MacBook-Pro ncflutter % cd another_app
dhs@Dhss-MacBook-Pro another_app % mkdir test
dhs@Dhss-MacBook-Pro another_app % pwd
/Users/dhs/documents/ncflutter/another_app
dhs@Dhss-MacBook-Pro another_app % ls
README.md		lib			test
analysis_options.yaml	linux			web
android			macos			windows
another_app.iml		pubspec.lock
ios			pubspec.yaml
dhs@Dhss-MacBook-Pro another_app % touch test/some_test.dart
dhs@Dhss-MacBook-Pro another_app % ls
README.md		lib			test
analysis_options.yaml	linux			web
android			macos			windows
another_app.iml		pubspec.lock
ios			pubspec.yaml
dhs@Dhss-MacBook-Pro another_app % cd test
dhs@Dhss-MacBook-Pro test % ls
some_test.dart
dhs@Dhss-MacBook-Pro test % cat some_test.dart
dhs@Dhss-MacBook-Pro test % cd ..
dhs@Dhss-MacBook-Pro another_app % cd ..
dhs@Dhss-MacBook-Pro ncflutter % flutter create another_app --empty
Signing iOS app for device deployment using developer identity: "Apple
Development: Darshan Kawar (AV76JL68FU)"
Recreating project another_app...
  another_app/test/widget_test.dart (created)
Resolving dependencies in another_app...
Got dependencies in another_app.
Wrote 2 files.

All done!
You can find general documentation for Flutter at: https://docs.flutter.dev/
Detailed API documentation is available at: https://api.flutter.dev/
If you prefer video documentation, consider:
https://www.youtube.com/c/flutterdev

In order to run your empty application, type:

  $ cd another_app
  $ flutter run

Your empty application code is in another_app/lib/main.dart.

dhs@Dhss-MacBook-Pro ncflutter % cat another_app/test/some_test.dart
cat: another_app/test/some_test.dart: No such file or directory
dhs@Dhss-MacBook-Pro ncflutter % cd another_app
dhs@Dhss-MacBook-Pro another_app % ls
README.md		ios			pubspec.lock
analysis_options.yaml	lib			pubspec.yaml
android			linux			web
another_app.iml		macos			windows
dhs@Dhss-MacBook-Pro another_app %

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.13.4, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.13.4 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 367f9ea16b (6 days ago), 2023-09-12 23:27:53 -0500
    • Engine revision 9064459a8b
    • Dart version 3.1.2
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.14.0-14.0.pre.375, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.14.0-14.0.pre.375 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e300fda767 (20 minutes ago), 2023-09-20 00:51:11 -0400
    • Engine revision 81b93fc4a2
    • Dart version 3.2.0 (build 3.2.0-180.0.dev)
    • DevTools version 2.28.0-dev.8
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I made a PR addressing this issue: https://github.com/flutter/flutter/pull/147160.

@gspencergoog Do you happen to recall why --empty deletes the test directory? Code here:
https://github.com/flutter/flutter/blob/5a63b1df955bba1939bd7eab2bced5fe016ca429/packages/flutter_tools/lib/src/commands/create.dart#L454-L459

Because it was the easiest way to implement it on top of the existing code.  This bug shows that it was also the wrong way to implement it. We should probably just not be modifying/creating the test directory if --empty is specified.


Move priming deg

IMPORTANT: Please search among the Pull request before creating one.
Title

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] New feature (non-breaking change which adds functionality)

Description

Add new properties: cellrank.tl.Lineage.priming_{degree,direction}, switch diff. potential (entropy) to priming degree.
Also fixes priming degree for terminal states (was NaN, is 1 now) and some logging errors.
How has this been tested?

I've added 2 new tests for the above mentioned properties.
Closes

closes #500

@Marius1311 - could you check if it works as expected? I've changed also the estimator property (diff_potential -> priming_degree).

Fantastic, thanks Mike, will check today.

This doesn't yet work as expected, I need to look into this a bit more tomorrow to understand what's going wrong here.

Okay, I needed some time to think about this, here's what I came up with:
I think the whole priming degree/diffusion potential thing is difficult enough that we should devote an own method to this at the level of the base estimator class as that's something you can do with lineage probabilities, regardless of how they have been computed. I have in in mind a method .compute_lineage_priming with parameters

method: one of entropy or kl_divergence, where the former refers to palantir-like diffusion potential and the latter refers to STEMNET like priming degree
clusters: a list of "early" clusters that are used to define the "average" lineage probabilities when using kl_divergence. By default, all cells are considered

This method should return a vector, normalised to [0, 1], where 0 corresponds to no priming and 1 corresponds to fully committed. This way, we could unify both mechanisms to compute lineage priming under a single method with unified interface and consistent output, while giving a bit more control to the user. I would then stop automatically computing this when computing absorption probabilities and I would also stop automatically plotting this when plotting absorption probabilities. This would avoid overloading these methods with extra parameters.
In my experience, computing these measures can be a bit tricky and both the priming degree and the diffusion potential worked sometimes and sometimes not - that's why I believe we need to give a bit more control through the optional cluster selection.
We could then have a dedicated plotting method .plot_lineage_priming - however, I think that's an overkill and it's fine to just write to adata.obs, from where this can be plotted conveniently using scanpy/scvelo.
What's your opinion?

@Marius1311 ready for review. Here are some TODOs:

[ ] update docstrings (I again leave this up to you, think you can do it better)
[ ] determine whether early_cells should also be exposed in cellrank.pl.circular_projection
[ ] write tests for BaseEstimator.compute_priming_degree function

Okay, will try to get this done today, thanks already for the great work @michalk8!

Numba 0.53.0 seems to have broken many things, will have to take a look into it (i.e. whether we need to pin it down).

Fantastic, thanks for finishing this off!


Question: How to reference both NuGet packages?

How to reference both NuGet packages ?
My library should work on .NET and JS in the fsproj I have:
  <ItemGroup Condition="$(DefineConstants.Contains('FABLE_COMPILER'))">
    <PackageReference Include="Thoth.Json" Version="10.2.0" />
  </ItemGroup>

  <!-- a Not-Contains-Condition with leading exclamation mark:  -->
  <ItemGroup Condition="!$(DefineConstants.Contains('FABLE_COMPILER'))">
    <PackageReference Include="Thoth.Json.Net" Version="11.0.0" />
  </ItemGroup>

Is there another way to reference both packages?
then in the code :
#if FABLE_COMPILER
    open Thoth.Json
#else
    open Thoth.Json.Net
#endif

    Encode.Auto.toString(..)

When compiled with Fable 4.14 fails with
The value, namespace, type or module 'Encode' is not defined. (code 39)

Hello,
Do you need to use the Auto API or the manual API is enough?
If the manual API is enough, you could use:

Thoth.Json.Core : To include only in the shared project as this is runtime agnostic
Thoth.Json.JavaScript: To include only on the Fable project
Thoth.Json.Newtonsoft: To include only on the .NET project

There package are a major rewrite of Thoth.Json to make it runtime agnostics (there is even a Python one) but I didn't announce it officially yet.
Usage:
// In the Core library
module MyLib

open Thoth.Json.Core

let value =
	Encode.string "maxime"

// In the runtime project
open Thoth.Json.JavaScript
// or
// open Thoth.Json.Newtonsoft

let json =
	Encode.toSring 0 MyLib.value

For the old packages, the error you are getting from Fable compilation is probably because Fable doesn't evaluate MSBuild condition. We have an experimental Project cracker which use MSBuild under the hood, which can be invoked by running dotnet fable --test:MSBuildCracker (it is possible that you need to delete your obj / bin folders to start from a clean state).
It is also possible to do:
  <ItemGroup>
    <PackageReference Include="Thoth.Json" Version="10.2.0" />
  </ItemGroup>

  <!-- a Not-Contains-Condition with leading exclamation mark:  -->
  <ItemGroup Condition="!$(DefineConstants.Contains('FABLE_COMPILER'))">
    <PackageReference Include="Thoth.Json.Net" Version="11.0.0" />
  </ItemGroup>

as for .NET it will be harmless has long as you use:
#if FABLE_COMPILER
    open Thoth.Json
#else
    open Thoth.Json.Net
#endif

Thoth.Json will be included in your project but not used. It is not perfect but should do the jobs.

Thanks, Maxime, I did not know about Thoth.Json.JavaScript.
I would like to use mostly the Auto API.
Using --test:MSBuildCracker it works with MS Build conditions !
Alternatively, just removing all conditions also works. (But has less compile safety.)
Thanks for your fast reply.

Auto API is planned for the new split library but I don't have an estimate for it yet focusing on Glutinum ATM.
Glad that we have a solution for your problem.

Yes , if Glutinum works well it will be a Gamechanger!


As a user, I want to see more context for deserialization errors so that debugging is faster

Is your feature request related to a problem? Please describe.
An example candid deserialization error message is:
"No more values to deserialize".
That can be hard to debug: what I want to know is:

The full context -- that is the full sequence of bytes that were being deserialized.
Why was a value expected? In other words, what part of the candid type is missing?

Describe the solution you'd like
I'd like something like:
"No more value to deserialize. While trying to deserialize  into a , the end of the byte stream was reached, but a  was expected."
Describe alternatives you've considered
I have none.
Additional context

There's a rudimentary debug info here: https://github.com/dfinity/candid/blob/master/rust/candid/src/de.rs#L127. It is in the error structure, but not displayed at the moment. I can recover that before we have a better debug message.

@chenyan-dfinity Sounds great!

@sasa-tomic pointed me to https://github.com/dfinity/bigmap-rs/blob/master/vendor/rust-cdk/src/ic_cdk/src/api/context.rs#L103-L119

CC @bitdivine

@chenyan-dfinity, @hansl and others: can we get this bug prioritized?
For instance, @bitdivine is complaining, with merit I think, that in case of "type mismatch", the error message does not contain the field name that caused the mismatch to happen.

Thank you for pointing out this PR.  Indeed, I have:
+ dfx --identity bootstrap-super-leader canister --network=bootstrap_bootstrap call governance forward_vote '(49, 3, variant{Yes})'
The Replica returned an error: code 5, message: "Canister rrkah-fqaaa-aaaaa-aaaaq-cai trapped explicitly: Panicked at 'Deserialization Failed: "Deserialize error: Type mismatch. Type on the wire: Int; Expected type: Nat64"', /source/source/src/endpoint.rs:34:21"

with a fully up-to-date did file.  I have no idea what the problem is or how to fix it.

Yes, we are improving the error message as we add subtyping check in deserialization.
For @bitdivine's case, it seems dfx is not getting the did file. Is governance.did specified in dfx.json? In case, the did file is missing, you need type annotations for nat64, so the argument would be (49: nat64, 3: nat64, variant{Yes}). The annotation is not needed when dfx can fetch the correct did file.


Windows CLI support

Describe the feature
Running hub commands on Windows does not seem to work. We likely need to add testing on windows.

I tested on a Windows machine. Basic hub commands are working fine.
Upload Code:
from hub import tensor, dataset
import numpy as np

images = tensor.from_array(np.ones((4, 512, 512)))
labels = tensor.from_array(np.ones((4, 512, 512)))

ds = dataset.from_tensors({"images": images, "labels": labels})
ds.store("adi10hero/test")

Terminal Output:
(active) D:\ADI\activeloop-ai>python test_win_upload.py
initialized

On the front end:

Download Code:
import hub

ds = hub.load("adi10hero/test")
data = ds["images"][:].compute()
print(data)

Terminal Output:
(active) D:\ADI\activeloop-ai>python test_win_download.py
[[[1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  ...
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]]

 [[1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  ...
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]]

 [[1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  ...
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]]

 [[1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  ...
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]
  [1. 1. 1. ... 1. 1. 1.]]]

Please let me know which other tests I should run and do you happen to know which commands were giving errors?

@ADI10HERO can you please take a look at discussion https://github.com/activeloopai/Hub/discussions/125, where @SanvirDessai mentions about
Running hub commands on Windows does not seem to work. It could be due to inconsistencies within my Python installation, but I wasn't able to get it running in order to register and login via cli. Fortunately, I was able to use WSL which worked fine.

Commented there, would try to simulate a similar environment and test the command again


visual feedback on being hit

Options:

Tint player red for a bit
Show mini health bar decreasing
Show damage icon (like a cross or something) on player

Added with sound effects


Add "is_boxable" trait to allow non-primitive types to be used with claws::box

This idea came up in #40

The idea is rather to be able to detect which types can be boxed without causing hard compiler errors, I'll edit the issue title.


[Framework] Event bindings

All views expose basic pointer events (click, drag lifecycle, enter/leave, etc.) and some expose other custom events like CheckBox.Change. There needs to be a way to consume these in view bindings.
We should assume API users won't have access to types like ButtonEventArgs or even IView, so the literal event signature of object? sender, ClickEventArgs e) and the like do the user no good at all. Instead, the binding system should have some mechanism for passing arguments.
Why argument-passing? Mainly because of context narrowing and walking (see #3). If we consider a simple case of a repeated list of items, e.g.
<lane orientation="vertical">
  <image *repeat={{Items}} sprite={{ItemData}} click={{ToggleItem(...)}} />
</lane>

and bound to a context such as:
class ViewData
{
  public string GroupId { get; set; }
  public IList<MyItem> Items { get; set; }
}

What goes in the ... in the click params above? There are two likely scenarios here. Either:

ToggleItem is on the context bound to the image, and therefore probably needs access to something in the parent context. We might have ToggleItem(^GroupId) for this; MyItem.ToggleItem doesn't need an explicit item ID, since it already knows its own ID, but needs to know the parent's group ID.

Or, and more likely, ToggleItem is a method on ViewData and needs to know which item, which is what we'd normally use the object? sender for. This requires the inverse syntax, ^ToggleItem(ItemId), given that ViewData.ToggleItem knows its own group ID but doesn't know the item ID.

Both designs are valid and the overall spec is the same regardless of which is used, but it means there has to be a way for the view to pass arguments that the event handler would have a hard time receiving/deciphering. (We could pass an IImmutableStack<object> or something like that for all parent contexts, but it's bad ergonomics.)
Sidebar to the sidebar: We do have Tags, which is in fact just a typesafe generic container, and that serializes pretty well over an API boundary and is trivially easy to copy. So a way to assign tags in the markup could be helpful as well, and we make that parameter always available to event handlers. However, it's also not the most ergonomic, and I'm not sure if it solves the entire set of problems that need to be solved.
There is also a question of how, precisely, to distinguish between literal values vs. binding values in event args. While literal values are going to be less common, there is still very much a use for them.
Separately, the markup element needs to be specified. Above we have e.g. click={{ToggleItem(...)}}, which will probably work - I am only concerned about all the brace/paren noise, and think it is a bit hard to read. However, using quoted events, while clearly more readable, would introduce the same ambiguity from #2 about what is a binding and what is a literal.
Current ideas are:

event={{Handler(...)}} (used above)
event="Handler(...)" (easier to read, but confusing since it's not a literal)
event=Handler(...) (much cleaner, but even more confusing from a larger perspective)

Whatever the grammar is, the main tasks are:

Implement the grammar in lexer/parser
Update view binder to distinguish between properties and events (descriptor code currently does not look at events at all)
Map bound event args according to their binding type (literal vs. context etc.) and value type
Attach surrogate event handlers to the real views, which call the bound handlers
Add any bells and whistles like Tags as special arguments.

None of the original syntax ideas have grown on me since I wrote this up, and with the lexer and parser now being fairly stable and, if not exactly elegant then at least pretty clean, I'm also not liking how all of the original ideas are going to require a bunch of hacks to parse.
Event expressions are not like property expressions; even if a dot syntax gets implement for properties at some point, allowing arbitrary function calls is far too much of a footgun and event delegates look like arbitrary function calls even though they aren't. In fact, there has to be an unambiguous way to mix literal expressions and binding expressions within the event arguments. This almost certainly calls for a unique event syntax, not only to avoid user confusion but to make the lexer/parser side anything other than a mess.
While it's a bit ugly, I think pipes are the least ambiguous choice here:
<foo click=|DoThing("foo", Bar, ^Baz)| />
Some things to note:

Single pipe feels slightly less oppressive on the eyes than two pipes, despite the inconsistency with double braces (maybe we should change to single braces for regular bindings anyway; there's no real need for double braces outside of a quoted string).
Literal arguments are quoted. This implies that only strings are supported for literal arguments, and that is completely correct, same as input attributes; since we aren't evaluating arbitrary expressions and the entire markup is a string, then literals start as strings by definition and there has to be a conversion to the requested argument type.
Anything not quoted is a binding argument. Binding modifiers like < > don't make sense here, everything is by definition input and read-only, but context redirects like ^ or ~Xyz do make sense and need to be supported.
Arguments are comma-separated and we can ignore whitespace like everywhere else in the markup.

This needs a new lexer mode, I think, but that goes with the new pipe syntax vs. the existing brace syntax used for other bindings. Unfortunately it might need two or even three new lexer modes due to the complexity of argument parsing, but the state transitions are obvious so it's not a terrible hack.
The actual method, DoThing, should probably support context redirects too, e.g. ^DoThing(...). Other than that, it needs to be on the context somewhere. This means its lifetime will be associated with the context and we can do some basic caching. When the context changes, we'll read the signature and create value sources for all the params, and do whatever validation is necessary. Then, we can defer updating any of those value sources until the event actually happens.
Probably an EventBinding and factory will be required as a counterpart to AttributeBinding and factory. There are many similarities but all the access patterns are different.


Splunk prerequisites list for parsing of attack data

I would like to suggest as improvement to add details ( or a file ) with prerequisites for ingesting the attack data in a new Splunk instance. If the data is ingested in the UI using the Add data wizard, the data is not parsed, in order for Sysmon for Windows telemetry to be parsed the Add-on  "Splunk Add-on for Sysmon" ( https://splunkbase.splunk.com/app/5709/ ) must be installed.
And attack data like https://github.com/splunk/attack_data/tree/master/datasets/malware/cyclopsblink requires "Add-on for Linux Sysmon" ( https://splunkbase.splunk.com/app/6176/ )
This becomes even more complicated since some people might be confused by other add-ons in the Splunk store which are not supported anymore, but may be still found and downloaded from the store.
I think it would make this open source project more accessible if the prerequisites for running the attack data in a freshly installed instance of Splunk it would be specified.

@ionsor : I think it's a great idea and it would certainly make easier! we will have evaluate how to programmatically add these fields.
With that said, we currently use this attack data to test our detections in security_content using these TA's

Hi guys @ionsor @patel-bhavin , I am trying to use some of the sysmon logs found here and I am running into an issue where even though I choose the right data source of XmlWinEventLog:Microsoft-Windows-Sysmon/Operational when uploading the data using the GUI to Splunk I get an error saying "Not Found" see the attached picture.
While doing some search I also came across this  https://docs.splunk.com/Documentation/Splunk/9.1.2/Data/Uploaddata which states that "The Splunk Add-On for Sysmon is not supported for use with data loaded using the Upload Data functionality. For best results, use one of the supported options to collect Windows Sysmon events as described in the Splunk Add-On for Sysmon manual.
" Does that mean that the datasets here coming from sysmon will not work?


iCub Wiring section

Added the iCub Wiring section (porting from the old page http://wiki.icub.org/wiki/Electrical_wiring)
This is just a first step forward...

@MrAndrea  @emilianob80  this is just a first step. We need to add more info


Consistently store pid, tid and probe id as int32_t

Previously, they were mostly 64-bit but had a few places where they were treated as 32-bit.
This saves some memory and may marginally improve performance when pushing usyms onto the ring buffer, etc., but it's mainly just a nice little clean up for consistency. Maybe it even eliminates some undiscovered lurking bugs.
Checklist

[x] Language changes are updated in man/adoc/bpftrace.adoc
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

Changes look good.
I'm wondering if the changelog entry should be in our new "Breaking" section though. Consider this contrived example:
BEGIN { $a = (pid, tid); $a = ((uint64)2, (uint64)3); exit(); }

These tuples are compatible at the moment but after this change they will not be and surface a semantic_analyser error. There might be other cases like this that I'm not considering.

I wouldn't want to overwhelm the "breaking" section with too many entries so we should probably clarify what kinds of entries should go in there. For instance, this one does break compatibility but it can be also viewed as a "fixed bug" as pid/tid are int32 in the kernel.

I wouldn't want to overwhelm the "breaking" section with too many entries so we should probably clarify what kinds of entries should go in there.

Clarification would be good but right now we have 0 entries in the "breaking" section (which doesn't exist yet either) and the only other one that I see going in there soon is the lexical/block scoping PR. I think this PR is a good candidate for the "breaking" section but I don't feel strongly enough to fight beyond this comment 😄

I wouldn't want to overwhelm the "breaking" section with too many entries so we should probably clarify what kinds of entries should go in there.

Clarification would be good but right now we have 0 entries in the "breaking" section (which doesn't exist yet either) and the only other one that I see going in there soon is the lexical/block scoping PR. I think this PR is a good candidate for the "breaking" section but I don't feel strongly enough to fight beyond this comment 😄

Sure, pretty much same feeling here, I just wanted to say that we don't need to block this PR on it.
Let's discuss properly during the next office hours.

The previous example of a broken script with tuples no longer breaks with all of @jordalgo's tuple and integer size improvements. Here's an updated example of a broken script:
$a = pid;
$a =<PHONE_NUMBER>0;

It may now fail to compile with the following error:
ERROR: Type mismatch for $a: trying to assign value '10000000000' which does not fit into the variable of type 'uint32'

To restore the old behaviour, simply cast the value of pid to a uint64:
$a = (uint64)pid;
$a =<PHONE_NUMBER>0;

I'm going to merge this PR without checking in this example because I didn't fully understand what the plan was and I'm running out of time before I'm off 😅

@ajor Ack. I'm fine with merging.


Make the package DebugOnly

I can't see why anyone would want factories in production, but I may be wrong.
Is there a reason to not add debugOnly: true?

Hey @fermuch the main reason is that debugOnly doesn't allow you to export symbols, so you would have to reference it like Package['dburles:factory'].Factory which isn't great.
Aside from that you would also have to define your collections and factories within packages, factories would be a debugOnly (along with Factory if it were so).
To me that's rather convoluted and doesn't seem worth the extra effort in most cases.
But if there's some other approach that I'm not aware of I'd love to hear about it.

You're right, it may be too bothersome.
Right now I'm doing this: I made an internal package, marked it as debugOnly and it depends on dburles:factory. Inside this package, I define all the factories I created.

Yeah that sounds like a decent solution


fix md syntax of configuration quote on ExpressRoute NAT configuration example

Hi,
I've fixed a markdown syntax of configuration quote to offer more readability.
Could you please merge it?
Thanks,
Kazuki

Hi @kazubu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@cherylmc - Cheryl, could you please review this update from @kazubu? Thx.

@ganesr , can you please review this? Thanks!

@cherylmc & @ ganesr - second notice  on public PR

Please accept PR

#sign-off

Thanks!

@kazubu - Thanks for letting us know!


min distance from screen not configurable for each tooltip

current this value can be passed in Overlay Layout, so it's can't be configured for each individual item
as shown in below video, I can't specify the value for any specific tooltip
The idea is to move this to ToolTipConfig
Screen_recording_20231205_092016.webm

I don't understand why the user would want this

that can be a common usecase right
eg if an user want to have a offset of 12 for top coachmark but 8 for bottom one, usecase can be anything like there's a button behind and they want to show a poriton of it
also always better to give flexibility if we can

But using distance-from-screen-edge as a general way to control tooltip size will give very different results on screens of different sizes, or portrait vs. landscape.
And the user can already avoid hiding other UI elements by using a different setting for placement - top/bottom/start/end.
I did intend to suggest some more direct way to influence tooltip width: setting a max width (to put it simply) so that we can get a reasonably narrow, multiline tooltip instead of a very wide, single-line tooltip (especially for tablets, but maybe also large phones in landscape orientation).  But that could probably be a single setting for all tooltips.


Docs: add error hint for golang lint

Signed-off-by: Kaige Ye<EMAIL_ADDRESS>
What have you changed? (mandatory)
Add an error hint to go client guide.
What are the type of the changes? (mandatory)
Document improvement
How has this PR been tested? (mandatory)
N/A
Does this PR affect documentation (docs) or release note? (mandatory)
https://github.com/tikv/tikv/pull/4868
Does this PR affect tidb-ansible update? (mandatory)
No
Refer to a related PR or issue link (optional)
https://github.com/golang/lint/issues/446#issuecomment-483638233

PTAL @disksing

/run-unit-test


Grouping feature groups result set from another tab what leads to an error

Description
https://github.com/user-attachments/assets/b842282b-7dec-487c-a8ed-5f013edb5612
DBeaver Version
Community Edition 24.2.1-202409092322
Operating System
Windows 10
Database and driver
PostgreSQL 16
PostgreSQL JDBC Driver 42.7.4
Steps to reproduce
No response
Additional context
No response

thanks for the report


FancyZones not working with some specific applications if not run as admin

ℹ Computer information

PowerToys version: v0.25.0
PowerToy Utility: FancyZones
Running PowerToys as Admin: No
Windows build number: 19042.572

📝 Provide detailed reproduction steps (if any)

Create Zones for Desktop
Install and launch TeamSpeak3 (The application I recognized the error with)
Drag window (with shift)

✔ Expected result
Show the zones and snap app to it
❌ Actual result
Zones are not shown. Cannot snap application to them.
When running PowerToys as admin it works. Its just not that userfriendly to get the elevation request with every boot.

Is "TeamSpeak3" running as admin?

No it is not. But its a Qt Application. Maybe thats a problem.

@skaesdorf
it's a known issue https://github.com/microsoft/PowerToys/issues/1466 TeamSpeaks3 need PowerToys to run elevated.


#1543 Map.computeIfPresent() & Map.computeIfAbsent

Current coverage is 96.16% (diff: 100%)

Merging #1617 into master will increase coverage by <.01%

@@             master      #1617   diff @@
==========================================
  Files            89         89
  Lines         10862      10881    +19
  Methods           0          0
  Messages          0          0
  Branches       1826       1828     +2
==========================================
+ Hits          10445      10464    +19
  Misses          274        274
  Partials        143        143

Powered by Codecov. Last update 41bb914...591a10c

Thank you!!

Fixes #1261


Are color modes actually supported on native?

The readme mentions that dark mode / color modes are supported, but I can't seem to make it work. Dripsy doesn't export a useColorMode hook and initialColorModeName and useColorSchemeMediaQuery seem to have no effect. Am I doing something wrong?
I think full support should be doable by now, using the Appearance API.

Hey thanks for the issue. I believe this is supported but not documented well yet. Could you try this?
import { useDripsyTheme } from 'dripsy'

const { setColorMode, colorMode } = useDripsyTheme()

@nandorojo Hi, how do you implement that in a class component?

I would wrap it with a function component, put the hook in the function component, and pass the color mode functions down as props.
You can find an example here.

@nandorojo can you check? I'm getting undefined, sorry I'm new to hooks  https://snack.expo.io/@coffeeprince/dripsy-dark-mode

The hook has to be nested within the provider. Here you put the hook outside of it.

@nandorojo it's undefined even inside the functional component, check the console log https://snack.expo.io/@coffeeprince/dripsy-dark-mode

The hook can only be called in a component that is the child of DripsyProvider. Here, you're putting the hook in the same component as the provider.

Hello, I have same/similar issue:
<DripsyProvider theme={theme}>  <App /> </DripsyProvider>
Inside the app component I have:
const { setColorMode, colorMode, theme } = useDripsyTheme();
console.log(setColorMode, colorMode, theme);
That console log outputs: undefined, undefined, {...mythemeobject...}
In this case app is direct child for dripsyprovider component, I get same thing when it's not direct child.

Does your theme have multiple color modes

Does it need to have?
export const theme = {
colors: {
text: '#0F0',
background: '#fff',
modes: {
dark: {
text: '#FFF'
}
}
},
text: {
a: { color: '#00F', fontSize: 16, margin: 10 },
link: { color: '#F00', fontSize: 16, margin: 10 }
}
};

@cmaycumber have you used color modes in your app? I haven't actually. Maybe there's a localstorage flag to disable or something?

Hm... I see in dripsy code:
export { useThemeUI as useDripsyTheme } from 'theme-ui'
But here: https://theme-ui.com/color-modes, they are saying:
const [colorMode, setColorMode] = useColorMode()
Dripsy doesn't have useColorMode in its code?

You can try importing that from theme ui directly and see if it works.

Error: [useColorMode] requires the ColorModeProvider component


Implemented success response with an empty array

Pulled out default success response data to a field
Added fluent method for setting default success response data
Added tests for usage of new method
Added missing tests for null values in respondWithSuccess, respondCreated, respondNoContent and fixed problems

null values were allowed both in types and phpdocs, yet exceptions were thrown when passed, this is fixed and null values are returned as empty arrays

Added missing tests for Arrayable, JsonSerializable and Collection and their empty variants
Strengthened tests to use strict assertions

This should solve everything discussed in #13.
It should also be compatible with #15, with possibly minor changes.
I tried explaining anything, if something is hard to understand or needs fixing please ask.
@ultrono Please close conversations where you have no questions and run workflows if all is ok.

Great PR and explanation @JacekAndrzejewski, much appreciated.  Have finally got my schedule under control and will look at getting this merged asap.

After merging this in (everything looks great from an initial look) I think a small update the readme would be beneficial too.

@ultrono By the way, could you label this with hacktoberfest-accepted? I want to plant a tree :D


"Ghimicești" village is not found in database

ENVIRONMENT:
SOLR: http://zimbor.go.ro/solr/#/romania/query?q=Ghimicești&q.op=OR&indent=true&rows=182
browser: Chrome
STEPS TO REPRODUCE:
1.open SOLR in browser
2.in q section type "Ghimicești"
3.click on Execute Query button
EXPECTED RESULTS:
In the requirements document with diacritics there is "Ghimicești" village in "Fitionești" township
ACTUAL RESULTS:
0 results; there is no "Ghimicești" village of "Fitionești" township in database

Este comuna Fitionesti cu satul Ghimicesti.


[FIX] Use stack based delimiter stack instead of pointer based delimiter stack

This fixes #5. Basically I rewrote the delimiter stack to use a local stack instead of a malloc'd stack. So every time a delimiter is opened, a copy of the token struct is given to the stack instead of its address. This is done to avoid getting the same address that new_tok function returns.
 For now, I'll just leave a fixed size stack of tokens for delimiters. When I consider using functional programming for this language, I might consider fixing this properly.

Also, I added better error handling for lexing, provided file's full path.


[material-ui] Issue while installing it on Linux

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
System configurations: Ubuntu 22.04.3 LTS, node : v18.17.1, npm : 9.6.7
Steps:

Run command: npm install @mui/material @emotion/react @emotion/styled

Current behavior 😯
Error:
No matching version found for<EMAIL_ADDRESS>Expected behavior 🤔
Correct installation of MUI.
Context 🔦
Issue while trying to install MUI using command:
npm install @mui/material @emotion/react @emotion/styled
Your environment 🌎

npx @mui/envinfo
  Browser: Chrome

  System:
    OS: Linux 6.2 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
  Binaries:
    Node: 18.17.1 - ~/.nvm/versions/node/v18.17.1/bin/node
    Yarn: Not Found
    npm: 9.6.7 - ~/.nvm/versions/node/v18.17.1/bin/npm
  Browsers:
    Chrome: 119.0.6045.159

The error is not clear. Are you trying to install @emotion/cache? The veresion you are specifying exists, see: https://www.npmjs.com/package/@emotion/cache


I want the button of the page that choose the theory are not squared

What is the change?
I want the button of the page that choose the theory are not fall squared, I don't want the angle of the squared be 90ﾟ, maybe change it to quite round.
Why do you want this change?
Because I think squared would not look good, maybe this change would made the people playing this game more comfortable.
How can me make this change?
I'm not sure how to do this. But I think there is a option of change the image of the game.

#14 ?

I will close this issue as Annie has found a solution for it, and changed the program.
Please add more information or re-open the issue if you feel more changes need to be made.
This is how it looks now:

Since it is changed for now, I will close the issue.


Create test.pythowo

Hi there, can you please give a description of the files you've added and what they do, thanks.

I’m just messing around with them


grade-school: remove potential test pollution

The grade-school exercise creates a single instance of School and uses it through the entire test suite.  The suite has deftly avoided test pollution by leaning on deterministic test ordering and using different grades in each test.  This sets a bad example of how to organize tests: each test should be isolated from the others so as to avoid false negatives (test failures when there is no corresponding defect in the code) and false positives (tests passing when a defect exists but is covered up by side-effects of earlier tests).
Please move the construction of School into a setUp() method.

Hi jtigger, I am learning Java now and I am just starting to contribute to GitHub projects, but if you don't mind I would like to take a shot at this.

I'm delighted you're digging in, @superklamer.  Your PR looks good (included some rebase help, there).  Thanks for your support!


Change glass pane recipe to match the Vanilla one

This has the same problem just in reverse. The vanilla recipe you can put in either the top 2 or bottom 2 rows.


Cannot customise borders of years section in showDatePicker

Steps to reproduce
The years section in the showDatePicker has a border top and bottom that cannot be customised and doesn't respond to any changes applied in the Theme. I would expect that the dividerColor is also applied to this border, or that any other colours options in the ColorScheme could impact these borders
Expected results
The years section border top and bottom colour take the same colour as the dividerColor in the first screen

Actual results
The years section has a border top and bottom whose colours aren't affected by the custom styles

Code sample

Code sample
showDatePicker(
                                builder: (BuildContext context, Widget? child) {
                                  return Theme(
                                    data: Theme.of(context).copyWith(
                                      inputDecorationTheme:
                                          InputDecorationTheme(
                                              border: UnderlineInputBorder(
                                        borderSide: BorderSide(
                                          width: 0.5,
                                          color: ColorRes.colorPrimary,
                                        ),
                                      )),
                                      colorScheme: ColorScheme.light(
                                          onPrimary: ColorRes.white,
                                          primary: ColorRes.colorPrimary,
                                          background: ColorRes.white),
                                      datePickerTheme: DatePickerThemeData(
                                        headerBackgroundColor:
                                            ColorRes.colorPrimary,
                                        backgroundColor: ColorRes.white,
                                        headerForegroundColor: ColorRes.white,
                                        surfaceTintColor: ColorRes.white,
                                        dividerColor: ColorRes.colorPrimary,
                                      ),
                                    ),
                                    child: child!,
                                  );
                                })

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.16.5, on macOS 14.1.1 23B81 darwin-arm64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.2)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

in color scheme use property outlinevariant to change the color of border

Awesome, that worked, thank you!

Closing this since it seems like the issue has been solved.
Thank you


Disabled Elites cause beefy normal enemies to spawn

This is an enemy that spawned almost immediately when using honor (also happens without honor, this is just an easy way to test)

(this is boostDamage and boostHp respectively)

makes sense


Strava doesnt go Dark Mode on Samsung S23

Strava doesn't go Dark Mode on Samsung S23 whereas on OnePlus7 the inbuild darkmode doe it well, can you please fix it.

Not a DarQ issue.

Sad but fair.
We are already pushing Strava Dev Team to add dark mode support.
Thanks for building this app though


Get response size

This is how I get the document object:
var document = await _context.OpenAsync(PAGE_URL);
Is there a way to get content length off the response header?
Thanks

Not with the way how you do it. Only the requester knows this, so either wrap the existing one or implement your own (IRequester). I think the one in AngleSharp.Io can be easy customized w/o doing much work.
Hope this helps!

Thanks, I saw the example code from Unit Test, which looks like:
var httpClient = new HttpClient();
var requester = new HttpClientRequester(httpClient);
var configuration = Configuration.Default.With(requester).WithDefaultLoader();
var context = BrowsingContext.New(configuration);
var request = DocumentRequest.Get(Url.Create("http://httpbin.org/html"));

var response = await context.GetService<IDocumentLoader>().FetchAsync(request).Task;
var contentLength = int.Parse(response.Headers["Content-Length"]);

That works! But keep in mind that no document has been processed yet (only the response has been delivered w/o processing the body).

Yup, saw in the code there is a separate line to process the document. Thx


Open in Merge Editor button shall always remove the conflict markers

Testing #161833

Create conflicts
Open conflicts file
Open in Merge editor (file becomes dirty)
Close the original conflicts file (do not save)
Open the conflicts file again
Open in Merge editor
🐛 Conflict markers are not removed. I would expect Merge editor opens as in 3rd step

Step 4 will restore the file, thus effectively re-introduce the conflict markers.
Step 6 will just reveal the merge editor and not remove the conflict markers, as the merge editor is still opened.
@bpasero @jrieken ideas? :)

Yeah, I wonder if opening the merge editor should simply close the textual merge editor. We already save it (as far as I know), so we could also close it. In other words, the 3-way merge editor would replace the text merge editor. Note that this is only possible if the text merge editor is not dirty, so a bit of proper async code is needed, but should work.
Obviously people could still trigger this, but maybe less likely.

I wonder if opening the merge editor should simply close the textual merge editor.

😨 I fear rage when we take the text editor away, esp when folks do experimental-clicking

Step 4 will restore the file, thus effectively re-introduce the conflict markers.

@hediet Can't we detect that the file has changed outside of the 3wm-editor and handle that special? Like re-start merge in the case of external modifications? That is maybe a little drastic but I believe the scenario is a corner-case and restarting isn't the worst

(this was brought up in the standup): I can also investigate if we can prevent the dialog from asking given the file is still opened in the merge editor. though it would probably not fix all the issues, because a user could still "File > Revert".
maybe the merge editor could detect this and show a in-editor notification with an option to re-merge and remove the markers again.

both ideas sound good!


WIP: fixes for tomo provenance

Codecov Report

Merging #172 into master will decrease coverage by 0.07%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
- Coverage   93.67%   93.59%   -0.08%
==========================================
  Files          15       15
  Lines        1691     1718      +27
  Branches      291      295       +4
==========================================
+ Hits         1584     1608      +24
- Misses         71       72       +1
- Partials       36       38       +2

Impacted Files
Coverage Δ

shed/doc_gen.py
100% <100%> (ø)
:arrow_up:

shed/replay.py
81.81% <77.77%> (-1.17%)
:arrow_down:

shed/translation.py
92.72% <94.11%> (+0.17%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0486935...650eb89. Read the comment docs.

needs news


[功能请求] 支持节点的域名伪装来实现免流

功能描述
伪装域名
这个功能的必要性
实现免流域名伪装就能充分利用5g网络速率优势
当前可用的替代方案
No response
补充
No response

没接触过免流， 能提供一些文章或者资料介绍一下免流时运营商是如何识别「免流域名」的吗？
举个例子， 如果运营商是看 SNI 里的域名来判断是否免流的话， 那理论上只要关掉 obfs ， 协议用 udp ， 并且 server_name 设置成「免流域名」就行了。
然后需要注意的是 hysteria 使用基于 UDP 的 HTTP/3 ， 所以像 v2ray 那样设置成明文 HTTP 然后用 Host 请求头的操作是不行的。

v2ray是改请求头的host来实现的,协议一般是tcp或ws，比如电信停机这样伪装就可以上网

Originally Telecom will limit internet speed, after using the method, now it is quick 😮
Jan 15, 2023 12:01:45 egwang186 @.***>:

v2ray是改请求头的host来实现的,协议一般是tcp或ws，比如电信停机这样伪装就可以上网
[https://user-images.githubusercontent.com/113794160/212522277-7bcd364a-0f5e-4b47-a81b-ce1953fb4d45.png][Screenshot_20230115-115914][https://user-images.githubusercontent.com/113794160/212522277-7bcd364a-0f5e-4b47-a81b-ce1953fb4d45.png]
—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1383049590], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYG42O3MDRZP4BYNEB3WSNZCRANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYH2BMDFEC2JXIBJ573WSNZCRA5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSN6UXM.gif]

Maybe the way redefine the package transmit priority.
We will have a try next time when global optical fiber get congestion.
Jan 15, 2023 13:31:06 Nanyu @.***>:

Originally Telecom will limit internet speed, after using the method, now it is quick 😮
Jan 15, 2023 12:01:45 egwang186 @.***>:

v2ray是改请求头的host来实现的,协议一般是tcp或ws，比如电信停机这样伪装就可以上网
[https://user-images.githubusercontent.com/113794160/212522277-7bcd364a-0f5e-4b47-a81b-ce1953fb4d45.png][Screenshot_20230115-115914][https://user-images.githubusercontent.com/113794160/212522277-7bcd364a-0f5e-4b47-a81b-ce1953fb4d45.png]
—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1383049590], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYG42O3MDRZP4BYNEB3WSNZCRANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYH2BMDFEC2JXIBJ573WSNZCRA5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSN6UXM.gif]

—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1383061041], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYDDZKFKVMKQZPFK4JLWSODRVANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYA3WLDZKKM3DMLHMQLWSODRVA5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSN7LDC.gif]

Speed comparison ratio between after and before maybe near three times.
Jan 15, 2023 13:54:54 Nanyu @.***>:

Maybe the way redefine the package transmit priority.
We will have a try next time when global optical fiber get congestion.
Jan 15, 2023 13:31:06 Nanyu @.***>:

Originally Telecom will limit internet speed, after using the method, now it is quick 😮
Jan 15, 2023 12:01:45 egwang186 @.***>:

v2ray是改请求头的host来实现的,协议一般是tcp或ws，比如电信停机这样伪装就可以上网
[https://user-images.githubusercontent.com/113794160/212522277-7bcd364a-0f5e-4b47-a81b-ce1953fb4d45.png][Screenshot_20230115-115914][https://user-images.githubusercontent.com/113794160/212522277-7bcd364a-0f5e-4b47-a81b-ce1953fb4d45.png]
—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1383049590], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYG42O3MDRZP4BYNEB3WSNZCRANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYH2BMDFEC2JXIBJ573WSNZCRA5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSN6UXM.gif]

—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1383061041], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYDDZKFKVMKQZPFK4JLWSODRVANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYA3WLDZKKM3DMLHMQLWSODRVA5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSN7LDC.gif]

—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1383064175], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYAZOENNECRGPO43GCTWSOGK3ANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYH5QOJU7LDF32I3TXLWSOGK3A5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSN7RG6.gif]

Hi, what about conclusion?
Jan 18, 2023 15:29:19 egwang186 @.***>:

Closed #544[https://github.com/apernet/hysteria/issues/544] as completed.
—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#event-8252863688], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYDR2MKG7M4GWCNPNFLWS6LU5ANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you commented.[Tracking image][https://github.com/notifications/beacon/AKGBAYFKZQAMDR3NMKE45DTWS6LU5A5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNWJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XKUY3PNVWWK3TUL5UWJTYAAAAAD27IWTEA.gif]

hysteria can't change the  http request host, continue to use vmess.

תודה אתה
0  ︶ 0
Jan 18, 2023 15:44:00 egwang186 @.***>:

hysteria can't change the http request host, continue to use vmess.
—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/544#issuecomment-1386618911], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYEBSNRZ4FQZOTNRY5DWS6NL5ANCNFSM6AAAAAAT3KJHTM].
You are receiving this because you commented.[Tracking image][https://github.com/notifications/beacon/AKGBAYAS4E56X4IG7QMXOT3WS6NL5A5CNFSM6AAAAAAT3KJHTOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSSUYQB6.gif]

udp2raw的faketcp方案是什么
用raw socket给udp协议直接加上伪造的tcp包头，把udp伪装成tcp；本质上还是udp，不需要经过系统的tcp协议栈
https://github.com/wangyu-/udp2raw/wiki/faketcp方案和常见的udp-over-tcp、udp-in-tcp方案的区别

如果能支持免流就牛了


A way to self host nadekobot 24/7

I don't know if there is a way to self host nadeko in my computer and let her run 24/7. I mean, everytime I turn off my computer it goes offline, right?
So there is a way to make it run, even after I turn off my computer?
Thank you.

You can better ask these types of questions in then #help in the Nadekolog server.
If you want to have it 24/7 you could use a vps (a server/computer/virtualmachine in the cloud/external). Think kwoth has a link where you get 5$ Extra if you use his link. He also gets a tip from it what is nice.


fix(ci): fixed download matrixes step for pages and relase-body-driver.

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area CI
Does this PR require a change in the driver versions?
What this PR does / why we need it:
I always forget that actions/download-artifact does not support regexes.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/milestone 0.14.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Please publish latest version on powershellgallery

https://github.com/dahlbyk/posh-git/issues/224 is still reproducible.

I just pushed a new version, but got a warning that suggests #242 may not have been sufficient. Give it a try?

WARNING: This module 'C:\Users\Keith\AppData\Local\Temp\1744033178\posh-git\posh-git.psd1' has exported functions. As a best practice, include exported functions in the module manifest file(.psd1). You can run Update-ModuleManifest -FunctionsToExport to update the manifest with ExportedFunctions field.

It looks like Test-ModuleManifest doesn't pick up the exported functions or aliases properly (except Invoke-NullCoalescing, oddly), but actually importing the module processes everything properly.
Moving exported functions from Export-ModuleMember to being directly declared in the manifest causes them to appear properly with Test-ModuleManifest so I'd guess that would fix the warning.

The warning message should be improved.  Using wildcards results in poor performance on V3 and beyond when using module auto-loading (so could be any command).  PowerShell has to analyze the script/dll/whatever when you use wildcards, but if you avoid wildcards, PowerShell skips that potentially slow work.
So best practice is to use:
FunctionsToExport = @('Get-Thing1', 'Get-Thing2')
# no aliases to export, be explicit, don't omit this
AliasesToExport = @()
# no cmdlets to export, be explicit, don't omit this
CmdletsToExport = @()

Instead of:
# Don't use wildcards
FunctionsToExport = "*"
# Don't omit any of the *ToExport entries
## CmdletsToExport = "*"
# Don't use $null, it's handled just like it was omitted
AliasesToExport = $null

So can we drop https://github.com/dahlbyk/posh-git/blob/5f8e87858db093497029e4ddb70bcb67629f9efa/posh-git.psm1#L30-L48 altogether or do we need to maintain the export list in two places?

By default, all functions, cmdlets, and aliases are exported, so if that's what you're doing, it's not necessary.  But if you have have internal functions or aliases, then maintaining the list in both places is unfortunately necessary for the best performance.

By default, all functions, cmdlets, and aliases are exported, so if that's what you're doing, it's not necessary.

Does this include exporting function private:Do-Something? If not, we could make a sweep through to privatize (lol) anything that's not be exported.

I had a feeling private wouldn't have any effect, but had to test to be sure.  And indeed, private functions get exported.  That is probably an oversight, but it doesn't help you if it gets fixed, so you should stick with Export-ModuleMember.

I'd like to clean up the PS Gallery versioning process as part of the push to 1.0: https://github.com/dahlbyk/posh-git/issues/328

I'm closing this since the latest version is posted to the PSGallery.  And the current posh-git.psd1 no longer uses *.  In fact, the only warning it generates is for using ModuleToProcess instead of RootModule.


Bump the pvsite_datamodel dependency

Do the writing to the PV sites database ourselves

Still TODO

[x] Merge https://github.com/openclimatefix/pv-site-datamodel/pull/50
[x] Bump the dependency to newest pv-site-datamodel
[x] Fix the tests in main (See #51 )

Codecov Report

Merging #52 (dec4f7d) into main (188b7bd) will increase coverage by 0.17%.
The diff coverage is 33.33%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #52      +/-   ##
==========================================
+ Coverage   48.81%   48.98%   +0.17%
==========================================
  Files           6        6
  Lines         295      296       +1
==========================================
+ Hits          144      145       +1
  Misses        151      151

Impacted Files
Coverage Δ

pvconsumer/app.py
34.52% <0.00%> (ø)

pvconsumer/solar_sheffield_passiv.py
24.56% <ø> (ø)

pvconsumer/save.py
47.82% <50.00%> (+2.37%)
:arrow_up:

pvconsumer/__init__.py
100.00% <100.00%> (ø)

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


[bdwgc] New port

After having finished this new port, I found that there were already two issues requesting bdwgc pointing to existing branches on other repos (#5471 and #6052). Feel free to pick either one of those or this one.

 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.:x: NNemec sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @NNemec, thanks for the new port. Here are the test results from the current CI system:

x64-windows-static
master
test
notes

bdwgc

Fail
New Port

x64-osx
master
test
notes

bdwgc

Fail
New Port

x64-linux
master
test
notes

bdwgc

Fail
New Port

x64-uwp
master
test
notes

bdwgc

Fail
New Port

arm-uwp
master
test
notes

bdwgc

Fail
New Port

failureLogs.zip

As of right now, the port builds successfully on Windows and Linux.
I haven't investigated what causes it to fail on UWP and MacOS X.
I'm a bit sceptical that the Linux build works properly, so I'll do some manual testing when I have time.

Great! I got UWP building locally once and will try to integrate it within the next days.

I introduced the rename to gc.lib because that was what an existing FindGC.cmake was searching for. I'll have another close look at this with my current understanding.

@NNemec
I have local changes to the CMakeLists.txt to produce binaries with the names gc.lib and gc.dll.
I couldn't find a FindGC.cmake file in the source code, and there's also no FindGC.cmake module that ships with CMake. Is that file specific to your project?
I also noticed that when building on Linux (using the steps on their README), you get libgc.so and libgc.a binaries. So that leads me to believe that <prefix>gc<suffix> is the correct naming convention for the library.
Can I ask, how did you build the binaries before?
Are the names produced different than the ones produced by CMake?

The FindGC.cmake is checked in to the project I'm working on. In no way authoritative, just one example of what is out there...
I never built GC before the approach I submitted here.

/AzurePipelines run

/azp run

I modified the CMakeLists.txt file to produce binaries with the gc name.
These names should match the ones in the debian package for bdwgc.

@NNemec
Whenever you have time can you test the latest commit?
I think that your FindGC.cmake file should work with the added patch 001-install-libraries.patch.
I still haven't gotten bdwgc to work with UPW or on Mac OS.
But as an initial port, Windows and Linux support look OK.

Before I can try this out, I will first have to sort out some stuff in my local workspace. I will try over the weekend.
To get UWP working, the most straightforward approach is to deactivate the build of 'cord' - with your changes that should simply mean to drop BDWGC_INSTALL_TOOLS from the portfile.cmake.
As I understand it, cord is in no way essential for the libraries. It seems to be an interactive text editor using an advanced string class exploiting the garbage collector in some clever way. I doubt that anyone uses that code and I really see no reason to struggle with windows UI libraries when the goal is to get a low-level library compiled.

Trying out the latest changes, I find that the include files are now installed as include/bdwgc/gc.h - why was that extra subdirectory introduced? The FindGC.cmake the the project that I am porting searches for include/gc.h which worked before, but now it breaks. Trivial to fix on either side, but it should be clarified what the proper way is.

I'm modeling the port after the distributed Debian package, so the correct thing to do would be to remove the bdwgc folder and install directly to include, and that would match your FindGC.cmake.

Submitted PR upstream (https://github.com/ivmai/bdwgc/pull/281).
With the hope to make these changes official if they make sense.

/azp run

As it stands, the current package works for all supported triplets.
I'm inclined to merging it right now, and update the port once the patches have been merged upstream.
@NNemec have you had time to test whether this port works for your project?

My original submission works fine, I have not tried out any of the later versions and won't have a chance to do so for the next two weeks. Go ahead with merging. If I encounter any problems, it can still be fixed...


Replace U with S in Strikethrough Icon

The strikethrough icon is a bit visually confusing since it uses the character U with a line across the vertical center. S seems more standard?
Google Docs

Notion

Dropbox Paper

+1


Create a discord Badge for the Readme

The current discord badge is somehow broken but the link to the server works. Need to create another badge so that the link to the discord server is visible.

I want to work on this issue under JWOC.

Well okay I'm assigning you this issue


rdfs:label, rdfs:comment vs sh:name and sh:description

Is there a reason that NodeShapes have rdfs:label and rdfs:comment instead of sh:name and sh:description` as described in the specification?

I believe it was swapped in the early stages of development for the sake of trying out attributes from different naming conventions and its availability. Other than that, there is no reason for the name and description to be swapped, so I will change it back to the original state. Thank you for the reminder.

Issue solved by returning to sh:name and sh:description in the generator. Committed and pushed. Closing the issue.


Get_normalized_expression for specific genes

Hi,
When using the get_normalized_expression function, I've been trying to reduce the memory usage by specifying only specific genes with the gene_list argument (https://discourse.scvi-tools.org/t/get-normalized-expression-function-arguments/140). However, I receive an error:
ValueError: Value passed for key 'denoised_rna' is of incorrect shape. Values of layers must match dimensions (0, 1) of parent. Value had shape (92009, 3) while it should have had (92009, 4000).
Is there any idea how this could be fixed? Thanks in advance.

Ok actually I misinterpreted this. Is your gene list of length 3? If so, the RNA data cannot be stored back inside the anndata in a layer, you'd have to use obsm


home-assistant: make hass cloud integration work

Motivation for this change
I'd like to use my "Amazon ~spy-device~ Echo" to control home assistant.
I haven't actually tried to use it to control HA yet but at least it's connecting.
Should I separate out the pycryptodome override into its own file?
Cc: @dotlambda @FRidh
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: home-assistant
Partial log (click to expand)
tests/components/test_introduction.py .                                  [ 73%]
tests/components/test_logger.py ....                                     [ 75%]
tests/components/test_script.py .....                                    [ 78%]
tests/components/test_shell_command.py ......                            [ 82%]
tests/components/test_system_log.py ...............                      [ 91%]
tests/components/test_websocket_api.py ..............                    [100%]

============================== 1 tests deselected ==============================
================== 163 passed, 1 deselected in 27.12 seconds ===================
/nix/store/d07xmkz56gzzhjfb7m5bhn8qwf5j3agx-homeassistant-0.66.1

No need to use staging anymore, has been merged into master.

@GrahamcOfBorg build python2.pkgs.envs python3.pkgs.envs

Success on aarch64-linux (full log)
Attempted: python2.pkgs.envs, python3.pkgs.envs
Partial log (click to expand)
test_tuple_invalid (envs.tests.EnvTestCase) ... ok
test_tuple_valid (envs.tests.EnvTestCase) ... ok
test_wrong_var_type (envs.tests.EnvTestCase) ... ok

----------------------------------------------------------------------
Ran 18 tests in 0.016s

OK
/nix/store/042li0gv31bkpfkcdwin8hc1pziap63q-python2.7-envs-1.2.4
/nix/store/f2xb0i06mr5v7xpcjmb6mknwz4lcfqw3-python3.6-envs-1.2.4

@GrahamcOfBorg build python2.pkgs.warrant python3.pkgs.warrant

Success on aarch64-linux (full log)
Attempted: python2.pkgs.warrant, python3.pkgs.warrant
Partial log (click to expand)
Successfully installed warrant-0.6.1
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/km8yii01860ixjmjc2bi7nqcjcv5wqq9-python3.6-warrant-0.6.1
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/km8yii01860ixjmjc2bi7nqcjcv5wqq9-python3.6-warrant-0.6.1/lib
patching script interpreter paths in /nix/store/km8yii01860ixjmjc2bi7nqcjcv5wqq9-python3.6-warrant-0.6.1
checking for references to /build in /nix/store/km8yii01860ixjmjc2bi7nqcjcv5wqq9-python3.6-warrant-0.6.1...
/nix/store/79kyfmrpacjihq1ha58vvbj68k4zmxaw-python2.7-warrant-0.6.1
/nix/store/km8yii01860ixjmjc2bi7nqcjcv5wqq9-python3.6-warrant-0.6.1

Success on aarch64-linux (full log)
Attempted: home-assistant
Partial log (click to expand)
copying path '/nix/store/0sm95q8xlaj9lmvjvdyixwm1g439bs2k-python3.6-home-assistant-frontend-20180404.0' from 'https://cache.nixos.org'...
copying path '/nix/store/2k375nm374jh9j6057wizf87npav4y44-python3.6-yarl-1.1.1' from 'https://cache.nixos.org'...
copying path '/nix/store/9nv569x04z3axnh6gv3zdx78id8za59a-python3.6-pyOpenSSL-17.5.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/2ngk9cs02lb8raj704rk4yaf756p7hz7-python3.6-urllib3-1.22' from 'https://cache.nixos.org'...
copying path '/nix/store/7v6jhcj9v9gm2ryjjysnm1pdxr00lrxj-python3.6-aiohttp-3.1.1' from 'https://cache.nixos.org'...
copying path '/nix/store/769yyh95j92307fg5zbpk9dpfbqkbjix-python3.6-requests-2.18.4-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/h9p5vqb1gxn7p066fvwg5plnmbvjsdab-python3.6-astral-1.6' from 'https://cache.nixos.org'...
copying path '/nix/store/a7wz8k6dnaqynl5nnbiylj5j6f1vlf6c-python3.6-aiohttp-cors-0.7.0' from 'https://cache.nixos.org'...
copying path '/nix/store/122psj6iz4n42dzskg1n0rcbsl9nrqa8-homeassistant-0.67.0' from 'https://cache.nixos.org'...
/nix/store/122psj6iz4n42dzskg1n0rcbsl9nrqa8-homeassistant-0.67.0


testing for geometry.angle_between_two_lines()

This MR:

starts fine-grained testing in sgeop, which will (hopefully) make #7 easier done the line.
resolves #23

geometry.angle_between_two_lines() is deep in the code base. Leaving the warning at stacklevel=2 for now, but probably should update later.

It may actually pop at different stacklevels do 2 is good.

It may actually pop at different stacklevels do 2 is good.

So are we good to merge or do you still need to review?


Pool meta with 254/255 character "description" not displayed in Daedalus

The description length inside Daedalus appears to be lower than allowed by cardano-cli metadata verification as such it allows for a transaction that looks complete and correct. However the Daedalus wallet will not display the pool ticker / information tile until you reach the allowed limit inside of Daedalus.
cardano-cli validates the metadata file as long as you are under the 255 limit however if the description contains the full 255 ( or 254) characters even though the metadata is allowed and validates the pool will not be displayed inside Daedalus unless the description is further reduced.
cardano-cli shelley stake-pool metadata-hash --pool-metadata-file metatdata.json
If you go above the limit the command below will show:
Shelley command failed: stake-pool metadata-hash  Error: Error validating stake pool metadata: Error in $: "description" must have at most 255 characters, but it has 258 characters.
If you reduce the description to 254 pass verification and submit a re-registration on chain for the pool the pool will still not display until you further reduce the description and process another re-registration transaction.

@KtorZ @disassembler can you please comment on this?

@gnpf @ManusMcCole @mioriohk please check this on Monday 🙏

Note: when fetching from pool servers directly (and not from a smash instance), cardano-wallet is being quite defensive on what it fetches. In particular, it won't fetch more than 512 bytes of data from the remote server (which helps circumvent the gentlemen who uploaded a 3GB metadata file for instance...)
512 bytes is already quite conservative as an upper-bound as we expect roughly:

A name (at most 50 UTF-8 bytes)
An optional description (at most 250 UTF-8 bytes)
A ticker (at most 5 UTF-8 bytes)
A homepage (at most 100 UTF-8

Thus it could be problematic if:
a) The metadata contains extra fields which eat up space.
b) Name, ticker or homepage are using "special" characters which need 2 or 4 UTF-8 bytes per character (like emojis for instance).
c) The file is oddly formatted with a lot of spaces and newlines (because they'll count towards the limit fixed by the wallet).
I am not sure about how SMASH handles that and whether it is applying more lenient or more strict constraints; but it may behave slightly differently when a SMASH server is configured.

Thanks @KtorZ <3


Fixed Issue #18

Responsiveness in this application for mobile view has been improved. If further changes is needed please comment on it

Star the repo. Thank you for contributing.


gzdoom: 4.8.2 -> 4.10.0

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Maybe it would be better if we move ZMusic into a separate package? Alpine did that.

sounds like a good idea

Result of nixpkgs-review pr 205750 run on x86_64-linux 1

2 packages built:

gzdoom
zmusic

@Lassulus done. I've also enabled building for aarch64-linux, so let's wait for ofborg to do it's thing first.


ATB-108: Session Store from Redis to Dynamo DB

Proposed changes
Replaced the session store in the Account Management Frontend from Redis to Dynamo DB.
ATB-108
What changed

Swapped out connect-redis with connect-dynamodb.
Replaced subject-session-index-middleware as index is now a DynamoDB table index that is maintained automatically.
Replaced subject-session-index with session-store which provides methods to get the session store and provides a method to delete a user's sessions which is utilised by the global-logout and delete-account controllers.
Local dev does not need explicit provisioning anymore - it now provisions tables and data on container startup through docker compose up.
Removed old github actions as they have been duplicated with new actions.
Updated the Build & Test Application action to use a provisioned localstack.
Removed SSM runtime dependency as app no longer needs to acquire them - was only being used to get Redis host, port.

Why did it change
see 0004-switch-from-redis-to-dynamodb.md
Issue tracking
JIRAs
ATB-108
ATB-13
ATB-109
ATB-150
ADRs
0004-switch-from-redis-to-dynamodb.md
Checklists
Environment variables or secrets
Added:
SESSION_STORE_TABLE_NAME - populated via SAM template
Removed
REDIS_KEY - SSM parameter key no longer required

[x] Documented in the README
[ ] Added to deployment steps
[x] Added to local startup config

Other considerations

[x] Demo to a BA, TA, and the team.
[ ] Recorded demo shared on #govuk-accounts-tech
[x] Update README with any new instructions or tasks

Superseeded by new PR


Order Id and other fields can't be mapped

In the documentation it suggests that Order Id be used as an alternate key, but this is currently impossible because Order Id (aka OrderNumber)  can't not be mapped.
There is a "design limitation" that prevents a number of fields from being mapped (see support ticket<PHONE_NUMBER>005262).
Besides changing the example, there should be separate section noting this limitation.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 402b18e7-5542-37ae-08dd-f526c2d2b9c7
Version Independent ID: c0ce55d5-81d3-04b5-170d-c3ca2fec48a2
Content: Field mapping considerations for standard dataflows
Content Source: powerquery-docs/dataflows/get-best-of-standard-dataflows.md
Service: dataflows
GitHub Login: @bensack
Microsoft Alias: bensack

Please ignore (I'm using bot automation to add labels). #please-open

Hi @xairrick
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. In addition, the support ticket no longer appears to be accessible. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Everytime upgrading yabai requires updating privacy settings.

Installed with brew.
I'm suprised that there is no issue about it (or It may be only me experiencing this issue).
Is there any workaround for this problem?
// I wish I can just turn off all the security things in mac.

From the README:

Code Signing
When building from source (or installing from HEAD), it is recommended to codesign the binary so it retains its accessibility and automation privileges when updated or rebuilt.

Sorry missed it. Thanks @dominiklohmann
https://github.com/koekeishiya/yabai/wiki/Installing-yabai-(from-HEAD)


Program not opening

Hi! The compiled binary seems to close itself right after opening it.

我这边也说打不开, 双击后没有打开

this is run from the cmd, isnt a program you launch.


Update websocket documentation

Description:

Related issue(s):
Fixes #1190
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Patch and project coverage have no change.

Comparison is base (f347dd0) 74.64% compared to head (dfaf5d5) 74.64%.

:exclamation: Current head dfaf5d5 differs from pull request most recent head 6ef23e9. Consider uploading reports for the commit 6ef23e9 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1193   +/-   ##
=======================================
  Coverage   74.64%   74.64%
=======================================
  Files          31       31
  Lines        2213     2213
  Branches      436      436
=======================================
  Hits         1652     1652
  Misses        438      438
  Partials      123      123

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@Ivo-Yankov should also update the configurations and main README with info or links so devs know how to not only consume but run their won WS-server

@Ivo-Yankov should also update the configurations and main README with info or links so devs know how to not only consume but run their won WS-server

After #1168 is merged running both the relay and websocket server will be done with docker compose up, so I don't think we need to update that part of the docs.