The problem getting on windows debugging, how can solve this issue

Launching lib\main.dart on Windows in debug mode...
Building Windows application...
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:119 (add_subdirectory):
The source directory
C:/Users/DELL/model_clon_pizza_romi_new/build/windows/x64/extracted/firebase_cpp_sdk_windows

does not contain a CMakeLists.txt file.
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
get_target_property() called with non-existent target "firebase_app".
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
string sub-command REPLACE requires at least four arguments.
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
set_target_properties Can not find target to add properties to:
firebase_app
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
get_target_property() called with non-existent target "firebase_auth".
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
string sub-command REPLACE requires at least four arguments.
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
set_target_properties Can not find target to add properties to:
firebase_auth
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
get_target_property() called with non-existent target "firebase_storage".
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
string sub-command REPLACE requires at least four arguments.
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
set_target_properties Can not find target to add properties to:
firebase_storage
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
get_target_property() called with non-existent target "firebase_firestore".
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
string sub-command REPLACE requires at least four arguments.
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
set_target_properties Can not find target to add properties to:
firebase_firestore
Error: Unable to generate build files
I followed these command
flutter clean
flutter pub upgrade firebase core
flutter pub get
flutter build [windows]
but three have no changes still getting error

I encountered the same problem when connecting Firebase and trying to run it on Windows

Seems like a problem with the flutterfire scripts and not firebase-cpp-sdk itself.
 How to fix it?

I have the same issue.
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:119 (add_subdirectory):
  The source directory

    C:/Users/USER/StudioProjects/hexolink4/build/windows/x64/extracted/firebase_cpp_sdk_windows

  does not contain a CMakeLists.txt file.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_app".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_app

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_auth".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_auth

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_storage".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_storage

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_firestore".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_firestore

Error: Unable to generate build files

So, how to fix it?

any update for this problem?

Please delete build/windows/x64/CMakeLists.txt file and CMakeFile folder  then flutter clean command run on your terminal and then flutter run command. I solved this error in this way

Any updates? I've stopped my app and when tried to rerun, get this issue and still same:
Launching lib\main.dart on Windows in debug mode...
Building Windows application...
cmake -E tar: error: ZIP decompression failed (-5)
CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:119 (add_subdirectory):
  The source directory

    E:/Android Project/Tools/anihon/build/windows/x64/extracted/firebase_cpp_sdk_windows

  does not contain a CMakeLists.txt file.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_app".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_app

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_auth".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_auth

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_storage".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_storage

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:125 (get_target_property):
  get_target_property() called with non-existent target "firebase_firestore".

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:126 (string):
  string sub-command REPLACE requires at least four arguments.

CMake Error at flutter/ephemeral/.plugin_symlinks/firebase_core/windows/CMakeLists.txt:127 (set_target_properties):
  set_target_properties Can not find target to add properties to:
  firebase_firestore

Error: Unable to generate build files


Fix custom links colors

child of #323
Check if this is already fixed in the dev branch

done in https://github.com/wp-pwa/saturn-theme/issues/316


Is there a way to use >=18.16.0 in .nvmrc file?

We want to enforce >=18.16.0 because our code is facing issues with prior versions. The string >=18.16.0 doesn't work in .nvmrc file. Nvm throws >=18.16.0 version not found error.
Closest solution is to use lts/hydrogen. At the moment it installs v18.17.1. But this forces/nags users with v18.16.0 to upgrade. How can we handle the situation in .nvmrc file? Is there a way to configure >=18.16.0 condition?

no, we don’t support that - everything in nvm resolves to one single version.
generally I’d expect everyone in an org to be on the exact same version - not just in a range. Why wouldn’t you want everyone on the latest 18?

@ljharb thanks for the clarification!

Why wouldn’t you want everyone on the latest 18?

Because it's an open source project and all contributors are not on same node versions.

Why wouldn’t you want everyone on the latest 18?

I think the problem is not that nvm installs a new version; it's that it doesn't delete the old version, which means without carefully curating the local installation, the number of copies can easily go up with every patch release coming out. If there's a way to tell nvm to "migrate my V18 installation to the newest version" that would also be great.

@Josh-Cena that doesn’t seem relevant here; the stated desire is to avoid installing a new one. You’re talking about #1091.

@OnkarRuikar sure but in that case you’d want your contributors to have every supported version installed anyways, which includes 18.17.

@ljharb Thanks for the pointer; I was trying to make this not an XY problem. For the context I was the one that first complained that >=18.16.0 does not work. For me, I personally don't care if I need to install a new version every time, but I don't want versions to pile up, and nvm migrate and >=18.16.0 will both fix this for me.

The discussion in #1091 explains why the primary use case of nvm is in fact to have versions "pile up", so you can easily switch between multiple versions, and it's just not optimized for using it as a moving target of a single node version.

I think this is a duplicate of #2562


Passing multiple dependencies to NotifierProvider the same as StateNotifierProvider?

I'm using StateNotifierProvider, and my StateNotifier class contains dependencies, which I create as follows.
///provider
final productProvider = StateNotifierProvider.autoDispose<ProductProvider, HomeState>((ref) {
  return ProductProvider (
    menProductService: ref.read(menProductService),
    womenProductService: ref.read(womenProductService),
  );
});

Currently, I've updated Riverpod to version 2.0, and the documentation suggests using Notifier instead of StateNotifier. I've found that NotifierProvider does not have a ref to pass the required dependencies.
final productProvider = NotifierProvider<ProductProvider, HomeState>(() {
  return ProductProvider(
  menProductService: // ref needed,
  womenProductService: //ref needed,
  );
});

How can I pass multiple dependencies to NotifierProvider as we do in StateNotifierProvider?
To Reproduce
StateNotifierProvider provides a ref, whereas NotifierProvider does not provide a ref.
final productProvider = NotifierProvider<ProductProvider, HomeState>(() {
  return ProductProvider(
  menProductService: // ref needed,
  womenProductService: //ref needed,
  );
});

Expected behavior
As suggested by the documentation, I want to use Notifier instead of StateNotifier to achieve the following implementation, which is currently done using StateNotifierProvider by passing dependencies.
class ProductProvider extends StateNotifier<HomeState> {
  final MenProductService menProductService;
  final WomenProductService womenProductService;
  ProductProvider({required this.productService})
      : super(const HomeState.initial());

  Future<void> getMenAllProducts() async {
    ///loading
    state = state.copyWith(allProducts: const AsyncLoading());

    ///call the getAllProducts
    final result = await menProductService.getAllProducts();

    ///check the result for success and failure
    result.when(
      success: (products) {
        state = state.copyWith(
          menAllProducts: AsyncData(products),
        );
      },
      failure: (failure) {
        state = state.copyWith(
          menAllProducts: AsyncError('Something went wrong!', StackTrace.current),
        );
      },
    );
  }

  Future<void> getWomenAllProduct() async {
    ///loading
    state = state.copyWith(allProducts: const AsyncLoading());

    ///call the get all Products
    final result = await womenProductService.getAllProducts();

    ///check the result for success and failure
    result.when(
      success: (products) {
        state = state.copyWith(
          womenAllProducts: AsyncData(products),
        );
      },
      failure: (failure) {
        state = state.copyWith(
          womenAllProducts: AsyncError('Something went wrong!', StackTrace.current),
        );
      },
    );
  }
}

This was answered, so closing


Better function transport

[ ] I have added the tests to cover my changes.

[x] I have updated the documentation, VERSION, and CHANGELOG accordingly.

[x] I have read the CONTRIBUTING document.

Fixes https://github.com/AgnostiqHQ/covalent-slurm-plugin/issues/11 (remote machine shouldn't need Covalent installed)

args and kwargs are now pickled along with the function for transport to the remote machine. Previously, these two inputs were sent as strings, which sometimes failed.

The remote machine will indeed need the Covalent SDK installed as it needs to understand basic Covalent data types -- Electron, Lattice, TransportableObject -- but not the entire dispatcher.

Ah, makes sense! Then I think the covalent docs + README will just need to be updated to prevent friction for new users (#48).


Can't download video from youtube: HTTP Error 429: Too Many Requests

The error occurs when I try to download this link: https://www.youtube.com/watch?v=SFK3giQxVYg
[youtube] SFK3giQxVYg: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I'm also running into this issue at the moment. However you can workaround it by including cookies with the option:
"youtubedl.exe [URL] --cookies cookies.txt"
https://askubuntu.com/questions/1220266/youtube-dl-do-not-working-http-error-429-too-many-requests-how-can-i-solve-this


how can i input more picture to improve the accuracy of identification

face_recognition version:
Python version:
Operating System: Raspbian

Description
how can i input more picture to improve the accuracy of identification

Add multiple images of same person in known people folder with any addition character like Furqan_1,Furqan_2 .... Furqan_n

Add multiple images of same person in known people folder by adding _1,_2 ....n
at end of name.
While adding label in results remove _ and numbers after _ by using this
name=os.path.basename(known_names[i])
end=name.find('')
name=name[:end]

Add multiple images of same person in known people folder by adding _1,2 ....n
at end of name.
While adding label in results remove _ and numbers after _ by using this
name=known_names[i]
end=name.find('')
name=name[:end]

Add multiple images of same person in known people folder by adding _1,2 ....n
at end of name.
While adding label in results remove _ and numbers after _ by using this
name=known_names[i]
end=name.find("")
name=name[:end]

Add multiple images of same person in known people folder by adding _1,_2 ....n
at end of name.
While adding label in results remove _ and numbers after _ by using this
name=known_names[i] end=name.find('_') name=name[:end]

can you please give more clarification? @FurqanMehmood

I was interested in the same point. What i did is:

one folder per person, with multiple images, i.e. ./barak_obama/0.jpg, 1.jpg etc.
I used a db to quickly map filepath and name
Hope it helps

use this tutorial for more details
https://www.pyimagesearch.com/2018/06/18/face-recognition-with-opencv-python-and-deep-learning/

@FurqanMehmood Thanks for the link!
I haven't tried yet! I will.
If I make the dataset of 1k images of one person. Do you think that might help me to improve accuracy?

@lordcenzin Can you please give some example of how you did implement in code?

I was looking into this myself and found that there is a KNN example in this repo worth checking out
https://github.com/ageitgey/face_recognition/blob/master/examples/face_recognition_knn.py

I was wondering if there was a way to do this with the face_recognition library. Perhaps loading n images and then averaging the face_encodings tensors?
For example, I have 7 photos of Amy. I want to store a (128-scalar) tensor for Amy - not 7 of them. Can I combine them/average them?

Well, I added more photos with averaging, and was excited the distance was less on image 2, but adding image 3 made it worse. Still need advice :-)
import face_recognition
import numpy as np
# https://face-recognition.readthedocs.io/en/latest/face_recognition.html

a1 = face_recognition.load_image_file('people/amber1.jpg')
a2 = face_recognition.load_image_file('people/amber2.jpg')
a3 = face_recognition.load_image_file('people/amber2.jpg')

a1_encodings = face_recognition.face_encodings(a1)
a2_encodings = face_recognition.face_encodings(a2)
a3_encodings = face_recognition.face_encodings(a3)

# unknown is amber4
unknown_person = face_recognition.load_image_file('people/unknown.jpg')
unknown_encoding = face_recognition.face_encodings(unknown_person)

two_mean_encodings = np.array((a1_encodings[0] + a2_encodings[0])) / 2
three_mean_encodings = np.array(
    (a1_encodings[0] + a2_encodings[0] + a2_encodings[0])) / 3

distance = face_recognition.face_distance(a1_encodings, unknown_encoding[0])
distance2 = face_recognition.face_distance(
    [two_mean_encodings], unknown_encoding[0])
distance3 = face_recognition.face_distance(
    [three_mean_encodings], unknown_encoding[0])

print('one image', distance)
print('two images', distance2)
print('three images', distance3)

results;
one image [0.38789462]
two images [0.35752516]
three images [0.38814902]

Well, I was able to average them and was excited when adding image 2 got a better score; but adding image 3 made the result worse. I need advice;
import face_recognition
import numpy as np
# https://face-recognition.readthedocs.io/en/latest/face_recognition.html

a1 = face_recognition.load_image_file('people/amber1.jpg')
a2 = face_recognition.load_image_file('people/amber2.jpg')
a3 = face_recognition.load_image_file('people/amber3.jpg')

a1_encodings = face_recognition.face_encodings(a1)
a2_encodings = face_recognition.face_encodings(a2)
a3_encodings = face_recognition.face_encodings(a3)

# unknown is amber4
unknown_person = face_recognition.load_image_file('people/unknown.jpg')
unknown_encoding = face_recognition.face_encodings(unknown_person)

two_mean_encodings = np.array((a1_encodings[0] + a2_encodings[0])) / 2
three_mean_encodings = np.array(
    (a1_encodings[0] + a2_encodings[0] + a2_encodings[0])) / 3

distance = face_recognition.face_distance(a1_encodings, unknown_encoding[0])
distance2 = face_recognition.face_distance(
    [two_mean_encodings], unknown_encoding[0])
distance3 = face_recognition.face_distance(
    [three_mean_encodings], unknown_encoding[0])

print('one image', distance)
print('two images', distance2)
print('three images', distance3)

results;
one image [0.38789462]
two images [0.35752516]
three images [0.38814902]

So.. this averaging thing seems to work, but could someone provide an educated opinion?
import face_recognition
import numpy as np
# https://face-recognition.readthedocs.io/en/latest/face_recognition.html

a1 = face_recognition.load_image_file('people/amber1.jpg')
a2 = face_recognition.load_image_file('people/amber2.jpg')
a3 = face_recognition.load_image_file('people/amber3.jpg')

a1_encodings = face_recognition.face_encodings(a1)
a2_encodings = face_recognition.face_encodings(a2)
a3_encodings = face_recognition.face_encodings(a3)

# unknown is amber4
unknown_person = face_recognition.load_image_file('people/unknown.jpg')
unknown_encoding = face_recognition.face_encodings(unknown_person)

two_mean_encodings = np.array((a1_encodings[0] + a2_encodings[0])) / 2
three_mean_encodings = np.array(
    (a1_encodings[0] + a2_encodings[0] + a3_encodings[0])) / 3

distance = face_recognition.face_distance(a1_encodings, unknown_encoding[0])
distance2 = face_recognition.face_distance(
    [two_mean_encodings], unknown_encoding[0])
distance3 = face_recognition.face_distance(
    [three_mean_encodings], unknown_encoding[0])

print('one image', distance)
print('two images', distance2)
print('three images', distance3)

results;
one image [0.38789462]
two images [0.35752516]
three images [0.31829679]

So.. this averaging thing seems to work, but could someone provide an educated opinion?
import face_recognition
import numpy as np
# https://face-recognition.readthedocs.io/en/latest/face_recognition.html

a1 = face_recognition.load_image_file('people/amber1.jpg')
a2 = face_recognition.load_image_file('people/amber2.jpg')
a3 = face_recognition.load_image_file('people/amber3.jpg')

a1_encodings = face_recognition.face_encodings(a1)
a2_encodings = face_recognition.face_encodings(a2)
a3_encodings = face_recognition.face_encodings(a3)

# unknown is amber4
unknown_person = face_recognition.load_image_file('people/unknown.jpg')
unknown_encoding = face_recognition.face_encodings(unknown_person)

two_mean_encodings = np.array((a1_encodings[0] + a2_encodings[0])) / 2
three_mean_encodings = np.array(
    (a1_encodings[0] + a2_encodings[0] + a3_encodings[0])) / 3

distance = face_recognition.face_distance(a1_encodings, unknown_encoding[0])
distance2 = face_recognition.face_distance(
    [two_mean_encodings], unknown_encoding[0])
distance3 = face_recognition.face_distance(
    [three_mean_encodings], unknown_encoding[0])

print('one image', distance)
print('two images', distance2)
print('three images', distance3)

results;
one image [0.38789462]
two images [0.35752516]
three images [0.31829679]

Hi,
did you find a solution for this?


Omitting fetcher function argument is ambiguous?

The README states:

Note that fetcher can be any asynchronous function, so you can use your favorite data-fetching library to handle that part. If ommitted, swrv uses the Fetch api.

But later also:

To only retrieve a swrv cache response without revalidating, you can omit the fetcher function from the useSWRV call.

So, which one is it? :) Or am I misunderstanding what omitting means in the first case?

yes it is ambiguous, thanks for noting! We need to document and maybe add some tests so it's clear. It should be:
- To only retrieve a swrv cache response without revalidating, you can omit the fetcher function from the useSWRV call
+ To only retrieve a swrv cache response without revalidating, you can set the fetcher function to `null` from the useSWRV call

If it is "omitted" (aka undefined) then the Fetch api will be used. https://github.com/Kong/swrv/blob/bd6c7e6de2b8c2e9595b02fe1b010ac1a197cd37/src/use-swrv.ts#L189

Relevant swr issue: https://github.com/vercel/swr/issues/738

This is great, but the typings for it do not allow null, it should be a quick fix to just add null to the type definition.

My PR resolves typing problem.
https://github.com/Kong/swrv/pull/274

Current doc is not ambiguous for me, function calls f(), f(undefined) and f(undefined, undefined) have the same arguments (geek note: except argument length). In JS, omitting argument always means supplying undefined.
Yeah, but I think it's good to clarify the doc.

afaik, the null fetcher behaves like Promise.race([]) or () => new Promise<never>(() => {}) (meaning resolves after infinite minutes), but isVaildating value differs.


Parse comma separated input

What does this PR?
This PR fixes the comma parsing in powerdown screen
Screenshots/Video
https://user-images.githubusercontent.com/48380998/170729426-cb19b8cc-d0be-40ed-8274-2119e1f71279.mp4

@noumantahir Update:
Parsing amount input similar to delegation screen.
Also added keyboard taps persistance and return keytype to done
https://user-images.githubusercontent.com/48380998/170738720-977c3b1d-18a4-4bce-9151-354d33207d49.mp4

@feruzm tested well, I also merged one of my relevant branch with it. you may review before merge....


[Bug] broker cannot connect the bookie when bookieId set hostname

Search before asking

[X] I searched in the issues and found nothing similar.

Version
2.10.2
Minimal reproduce step
With zookeeper deployed, bookieId is set to hostname, and the broker cannot connect to bookie
===============================================================
My preliminary analysis is that in the scenario of Zookeeper deployment, the broker should need to use zk to get the IP of Bookie
In the following function, DNS is directly applied to resolve IP, and there is no support for zk
public BookieSocketAddress(String addr) throws UnknownHostException {
        String[] parts = addr.split(COLON);
        if (parts.length < 2) {
            throw new UnknownHostException(addr);
        }
        this.hostname = parts[0];

        try {
            this.port = Integer.parseInt(parts[1]);
        } catch (NumberFormatException nfe) {
            throw new UnknownHostException(addr);
        }

        if (InetAddresses.isInetAddress(hostname)) {
            socketAddress = Optional.of(new InetSocketAddress(hostname, port));
        } else {
            socketAddress = Optional.empty();
        }
}

What did you expect to see?
broker should connect the bookie when bookieId set with hostname
What did you see instead?
N/A
Anything else?
N/A
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

add the ip hostname config to the /etc/hosts file

yes, this works, I have already tested
But I think that in the deployment mode of zk, the ip should not be obtained through DNS resolve, but should be obtained through zk, this is old feature but missing in the 2.10.2

The same problem when bookieId set with hostname，zookeeper deployed，k8s environment,  version 2.10.2
====================================================================================
The error log:
[pulsar-registration-client-28-1] ERROR org.apache.bookkeeper.client.TopologyAwareEnsemblePlacementPolicy - Cannot resolve bookieId pulsar-k8s-cluster-bookie-2 to a network address, resolving as /default-region/default-rack org.apache.bookkeeper.proto.BookieAddressResolver$BookieIdNotResolvedException: Cannot resolve bookieId pulsar-k8s-cluster-bookie-2, bookie does not exist or it is not running 	at org.apache.bookkeeper.client.DefaultBookieAddressResolver.resolve(DefaultBookieAddressResolver.java:68) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.TopologyAwareEnsemblePlacementPolicy.resolveNetworkLocation(TopologyAwareEnsemblePlacementPolicy.java:804) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.TopologyAwareEnsemblePlacementPolicy.createBookieNode(TopologyAwareEnsemblePlacementPolicy.java:794) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.TopologyAwareEnsemblePlacementPolicy.handleBookiesThatJoined(TopologyAwareEnsemblePlacementPolicy.java:718) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicyImpl.handleBookiesThatJoined(RackawareEnsemblePlacementPolicyImpl.java:80) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicy.handleBookiesThatJoined(RackawareEnsemblePlacementPolicy.java:249) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.TopologyAwareEnsemblePlacementPolicy.onClusterChanged(TopologyAwareEnsemblePlacementPolicy.java:665) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicyImpl.onClusterChanged(RackawareEnsemblePlacementPolicyImpl.java:80) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicy.onClusterChanged(RackawareEnsemblePlacementPolicy.java:92) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.BookieWatcherImpl.processWritableBookiesChanged(BookieWatcherImpl.java:197) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.BookieWatcherImpl.lambda$initialBlockingBookieRead$1(BookieWatcherImpl.java:233) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.pulsar.metadata.bookkeeper.PulsarRegistrationClient.lambda$null$5(PulsarRegistrationClient.java:139) ~[org.apache.pulsar-pulsar-metadata-2.10.2.jar:2.10.2] 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) ~[?:?] 	at java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?] 	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) ~[?:?] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) ~[?:?] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) ~[?:?] 	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) ~[io.netty-netty-common-4.1.77.Final.jar:4.1.77.Final] 	at java.lang.Thread.run(Thread.java:829) ~[?:?] Caused by: java.net.UnknownHostException: pulsar-k8s-cluster-bookie-2 	at org.apache.bookkeeper.net.BookieSocketAddress.<init>(BookieSocketAddress.java:68) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.discover.BookieServiceInfoUtils.buildLegacyBookieServiceInfo(BookieServiceInfoUtils.java:40) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.discover.RegistrationClient.getBookieServiceInfo(RegistrationClient.java:84) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	at org.apache.bookkeeper.client.DefaultBookieAddressResolver.resolve(DefaultBookieAddressResolver.java:43) ~[org.apache.bookkeeper-bookkeeper-server-4.14.5.jar:4.14.5] 	... 18 more
====================================================================================
it's not convenient and flexible to add the "ip hostname" config to the /etc/hosts file in k8s environment.  The pod's ip maybe   change.


Pass product module name into test_rule

This will need to wait until this is merged

Have we solved additions to rules_apple like this in a backwards compatible way in the future or just assume you'll need to bump to be on this change or newer?

Have we solved additions to rules_apple like this in a backwards compatible way in the future or just assume you'll need to bump to be on this change or newer?

@mattrobmattrob I was thinking of looking into some patching for this PR as well but i'm not sure how future-proof that is

@mattrobmattrob @jerrymarino PTAL at how I plan on doing patching. LMK if there is something else we may want to do

Also are there concerns the rules_apple patch will break for people even with landing this? I gave it a stamp assuming we'll fix it - but it might cause an issue.

Have we solved additions to rules_apple like this in a backwards compatible way in the future or just assume you'll need to bump to be on this change or newer?

➕ to @mattrobmattrob comment on this

We already keep a fork ofxctestrunner for our patches from time to time. To better integrate with how we do things, and to integrate with github workflows I'd like to use that instead of patch files:  https://github.com/bazel-ios/xctestrunner - similar for rules_apple as well.
This said - we'll still need to make it backwards compatible.

FWIW, I don't want to block a merge because of backwards compatibility, necessarily. Just want it to be a conversation. 👍 We are essentially on trunk so it shouldn't cause any issues for us.

The "backwards compatibility" issue I encountered was the PR seemed to require me to use the rules_apple version generated by the patch: in other words, if it needs argument module_name which doesn't exist outside of the patches. So if you have a different version of rules_apple than rules_ios gives you it won't work.

@dostrander if you want to add an if statement in a macro - I came up with a mechansim to add this:
https://github.com/bazel-ios/rules_ios/pull/628
load("@rules_ios_cfg//:config.bzl", "using_internal_rules_apple")

if using_internal_rules_apple:
   # some
else:
   # other

We might want this anyhow - but let me know what you're thoughts are. I added some suggestions to simplify rules_apple in that review BTW - both which can be backwards compatible.


Featured images

PR to merge featuredImages to master.

Whoops, sorry this was left out for so long. Don't we want to implement the backend for featured images before we merge into master?


Fix calls to deprecated htsjdk APIs

I'd like to merge this soon unless there are any objections.

Thank you, @tomwhite @lbergelson


WIP: first pass at implementing varargs for ESC commands.  'custom' makes use of this.

Fixes https://github.com/godot-escoria/escoria-issues/issues/106

Exactly what I wanted ;)
Thanks!

Be advised that this also fixes an issue with spaces immediately after (or immediately before) a quotation mark. This affects all ESC commands that can take arguments.

Thanks!


App loads empty after install, error in console

I couldn't get it to work. Below is the install log. The app loads empty after that. There is an error in the console "Uncaught Error: Cannot find module" in module.js:340.
$ npm install
npm WARN deprecated<EMAIL_ADDRESS>electron-settings <3.0.0 is no longer                       supported. Please upgrade :)

><EMAIL_ADDRESS>postinstall C:\Users\Me\Projects\riotjs-electron-exam                      ple-counter\node_modules\electron-prebuilt
> node install.js

Downloading electron-v0.36.12-win32-x64.zip
[============================================>] 100.0% of 50.41 MB (2.49 MB/s)
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fseve                      nts):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@1.                      1.3: wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

added 304 packages in 35.48s

Would you be able to look into it? I tried to follow the source code of this app to build my own simple Electron + Riot "Hello World" but I'm getting a "ReferenceError: riot is not defined" error.

Hi,
I ll take a look into this. I admit I never aim to maintain this , I was just and example by the way it's probably repeated to deprecated library.
But cant be sure about this. Probably it ll be fixed just upgrading version on package , removing node_modules and the install again using npm.

I'm short on successfully running examples of Riot and Electron, hence my digging. I've tried a fresh install of Electron with Riot, so my libs are up to date, but I'm getting a "ReferenceError: riot is not defined" error, so I though I'll try finding something that works first. Thanks for looking into it!


[Bug] msg: Zstd only supports the same type of bytebuffer decompression

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug

Affects Version(s)
master
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Do you know the managedBuffer is not direct buffer when getting memory by netty? @leixm

Do you need to turn on spark.rss.client.off.heap.memory.enable?

![image](https://github.com@leixm ache/incubator-uniffle/assets/8609142/87008bbe-4005-4d3c-9fda-6145faa3f23e)
@leixm

@leixm

@LuciferYang Do you know more about this point?


Delete ASv1 related packages and SDK code

Description
deleting:

attestation-service package

Tested
make sure all existing tests pass
Related issues

Fixes #9987

Backwards compatibility
No
Documentation
not updated

duplicate of https://github.com/celo-org/celo-monorepo/pull/10146


Division by a non-constant in symmetric cryptography code

Some processors do not have hardware division, and therefore the compiler generates code using other operations (additions, multiplications, shifts) for division by a constant, and calling a toolchain-provided function for division by a variable. In some environments, Mbed TLS is linked into a code segment that doesn't include the division function. Therefore my understanding is that we limit the use of division to asymmetric cryptography (which really needs division), and only for mbedtls_mpi_uint, not for larger types.
However, looking through the code base, I see other uses of division by a non-constant.
grep -n -P ' / *(?![ */0-9A-Z]|sizeof|[^"]*")' library/*.c

shows on 2.24.0 (in addition to many false positives):
library/cmac.c:276:    n = ( ilen + block_size - 1 ) / block_size;
library/ecp.c:1886:        i = 1U << ( j / d );
library/ecp.c:2296:    d = ( grp->nbits + w - 1 ) / w;
library/hkdf.c:110:    n = okm_len / hash_len;
library/memory_buffer_alloc.c:240:    if( n == 0 || size == 0 || len / n != size )
library/psa_crypto.c:4271:            / operation->block_size * operation->block_size;
library/ssl_tls.c:1991:        uint32_t ratio = ssl->conf->hs_timeout_max / ssl->conf->hs_timeout_min + 1;
library/timing.c:256:                                 / hfreq.QuadPart );
library/timing.c:499:    ratio = cycles / millisecs;
library/timing.c:509:        if( cycles / millisecs < ratio - ratio / 5 ||
library/timing.c:510:            cycles / millisecs > ratio + ratio / 5 )

The types of the dividends are:

w in ecp.c: unsigned char
Timeout in ssl_tls.c: uint32_t.
QuadPart in timing.c: doesn't matter, this is Windows code and harware division is available.
millisecs in timing.c: unsigned long. Only in a self-test.
Others: size_t.

This is probably not a problem in practice since nobody has complained. On most platforms, mbedtls_mpi_uint is size_t (or perhaps twice as large if MBEDTLS_NO_UDBL_DIVISION is defined). Since most of the occurrences above are size_t or uint32_t, it would only be a problem in a build without bignum (so either no asymmetric cryptography, or only through alt implementations, and I haven't checked whether the divisions in ecp.c can be avoided with alt implementations).
Should we fix this? Or should we stop worrying about division on a type that's bounded by at least one of uint32_t, size_t, unsigned long or mbedtls_mpi_uint?
Regardless of our decision, we should document the expectations precisely. The only documentation I'm aware of is that of MBEDTLS_NO_UDBL_DIVISION, which doesn't say whether division by size_t is supposed to be acceptable, or whether the use of division is acceptable when the bignum module is disabled.

Note: I think % should count as a division as well.

Or should we stop worrying about division

To be honest, I don't think we've ever consistently worried about division. I know of only one place if the code where we did (the old Lucky 13 counter-measure) and I suspect it was an accident.
I'm not entirely sure about labeling this as a bug, though, IMO it could be considered an enhancement: we currently don't offer an option for division-free builds, we never said we did, and offering such an option would be a new feature. Which of course we should consider only if people are asking for it, and apparently nobody is so far. Considering people already asked for MBEDTLS_NO_UDBL_DIVISION and MBEDTLS_NO_64BIT_MULTIPLICATION, it looks like at least some people know they can ask if needed.
Or do you think it's a documentation bug that it's undocumented? We say we're using standard C, we're not saying we don't use division, and standard C includes division, so while I agree it's not explicitly documented, I'd still say it shouldn't come as a big surprise. Though I agree that it would be good to have better documentation about which features of C we use and which we stay away from (for example, we'll never use VLAs).

asymmetric cryptography (which really needs division)

On a less important note, I'd like to nuance that assertion: with appropriate design choices, I'm positive you can do all of ECC without division. I'm almost sure you can to DHM key exchange and RSA public/private operations without it too. You might need it for DHM parameter generation and RSA key generation, though, I'm not sure about those parts.

we currently don't offer an option for division-free builds

Right, all we offer is a build without “double-width integer division”. Which we don't quite respect since some of the divisions are “double-width” on some platforms. Since we only support 32-bit and 64-bit platforms (we do not support 16-bit platforms and any other size is too exotic for us to have an official position), this means that according to the documentation, with ``MBEDTLS_NO_UDBL_DIVISIONenabled, in practice, division onsize_toruint32_tis ok but division byunsigned long` depends on whether you consider IP32L64 to be too exotic for concern or not.

We say we're using standard C, we're not saying we don't use division

We don't actually say that we don't use floating point, either, but I'd be annoyed if linking my application to a crypto/tls library on a platform without hardware floating point pulled in a floating point library. Integer division is less clear-cut, but it is a practical problem that I've faced before (not in Mbed TLS) where a module that used symmetric crypto and contained code that divided by the block size had a minor refactoring where the compiler suddenly stopped realizing that the block size was a constant, and this caused the linking to fail because the linker was configured with an explicit list of toolchain functions to include. So if we were not using integer division and started to do it, this could break existing applications.

with appropriate design choices, I'm positive you can do all of ECC without division

Well, yes, you can .


Add apiserver

This PR adds k8s manifests for the AppCat API Server.
Checklist

[x] The PR has a meaningful title. It will be used to auto generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.

Applying the manifests from the golden folder fails as there doesn't seem to be a valid certificate.

Applying the manifests from the golden folder fails as there doesn't seem to be a valid certificate.

Applying where? You need to configure manually a valid certificate or use the one from the lab cluster.


Cannot load state dict for optimizer

The following error occurs when I'm trying to resume to train the model from checkpoint.
I used the pretrained model whose backbone is ResNet provided by this repository.
Traceback (most recent call last):
  File "/home/qiming/workspace/pytorch-deeplab-xception/my_train.py", line 318, in <module>
    main()
  File "/home/qiming/workspace/pytorch-deeplab-xception/my_train.py", line 305, in main
    trainer = Trainer(args)
  File "/home/qiming/workspace/pytorch-deeplab-xception/my_train.py", line 95, in __init__
    self.optimizer.load_state_dict(checkpoint['optimizer'])
  File "/home/qiming/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/optim/optimizer.py", line 114, in load_state_dict
    raise ValueError("loaded state dict contains a parameter group "
ValueError: loaded state dict contains a parameter group that doesn't match the size of optimizer's group

When I print
print(self.optimizer.param_groups)

It shows
[{'params': [Parameter containing:
tensor([[[[ 1.4587e-02,  9.5933e-04, -7.4720e-03,  ..., -6.6904e-03,
            7.3701e-04,  9.5040e-03],
          [ 2.6121e-03,  4.9495e-03, -5.4276e-03,  ..., -3.9156e-02,
           -3.4436e-02, -1.4963e-02],
          [ 4.9470e-03,  1.5524e-02,  2.6544e-02,  ...,  1.8064e-02,
            7.9475e-03,  1.7200e-02],
          ...,
          [ 1.2903e-03, -6.0330e-03, -2.8998e-03,  ..., -4.9042e-03,
            6.9914e-03, -2.5362e-04],
          [-3.0935e-03, -1.4736e-02, -5.1809e-03,  ...,  7.1390e-03,
            2.9086e-03, -1.1533e-02],
          [-3.0223e-03, -1.6870e-02, -1.5665e-02,  ..., -5.6725e-03,
           -1.9275e-02, -1.6497e-02]]]],
...,Parameter containing:
tensor([ 0.2031, -0.0163, -0.0131, -0.0167, -0.0157, -0.0165, -0.0108, -0.0070,
        -0.0026, -0.0032, -0.0179, -0.0085, -0.0044, -0.0158, -0.0137,  0.0181,
        -0.0103, -0.0173, -0.0063, -0.0112, -0.0140], device='cuda:0',
       requires_grad=True)], 'lr': 0.035, 'momentum': 0.9, 'dampening': 0, 'weight_decay': 0.0005, 'nesterov': False}]

and print
 print(checkpoint['optimizer']['param_groups'])

shows
[{'lr': 0.0003280832169020921, 'momentum': 0.9, 'dampening': 0, 'weight_decay': 0.0005, 'nesterov': False, 'params': [139745645565704,<PHONE_NUMBER>25560,<PHONE_NUMBER>26928,<PHONE_NUMBER>28296,<PHONE_NUMBER>66784, ...,<PHONE_NUMBER>50024]}, {'lr': 0.003280832169020921, 'momentum': 0.9, 'dampening': 0, 'weight_decay': 0.0005, 'nesterov': False, 'params': [139745620893344,<PHONE_NUMBER>92192,<PHONE_NUMBER>81344,<PHONE_NUMBER>78320,<PHONE_NUMBER>71280,<PHONE_NUMBER>60000,<PHONE_NUMBER>48576,<PHONE_NUMBER>45696,<PHONE_NUMBER>38512,<PHONE_NUMBER>27232,<PHONE_NUMBER>27088]}]

It seems that they really don't match each other, but I have no idea how to solve it.
Could anyone tell me how to deal with the problem?

I've made following change when  loading state for optimizer.
  if not args.ft:
                for i in range(len(checkpoint['optimizer']['param_groups'])):
                    checkpoint['optimizer']['param_groups'][i]['params'] = torch.Tensor(
                        checkpoint['optimizer']['param_groups'][i]['params'])

                state = defaultdict(dict)
                for k, v in checkpoint['optimizer']['state'].items():
                    state[torch.tensor(k)] = v
                checkpoint['optimizer']['state'] = state
                self.optimizer.param_groups = checkpoint['optimizer']['param_groups']

                self.optimizer.load_state_dict(checkpoint['optimizer'])

                self.best_pred = checkpoint['best_pred']
                print("=> loaded checkpoint '{}' (epoch {})"
                      .format(args.resume, checkpoint['epoch']))

After that, it can be run now, whereas the training loss will increase up to 0.4**.
After one epoch, the mIoU declines to 0.7681, the previous best of which is 0.7843.

Is that a normal loss value?

@jfzhang95 Any idea? The issue still isn't solved. After I made the change, the model weight actually cannot be trained. And I don't think what I have done is correct.

@huqiming513  https://github.com/jfzhang95/pytorch-deeplab-xception/issues/83


Finalize daily patching

This PR captures the code changes needed to finalize patching of missing data described in desihub/desispec#2345. Since the changes in the files are already accepted or close to acceptance, this should be a non-controversial PR. A corresponding tag will be created after merging.

I'm ok with the coverage decreasing due to adding "untested" patching code.  Although it hasn't been tested with unit tests, it is a one-off usage that has been extensively checked my multiple people on the real daily files where it matters.

@sbailey, yes, at some point I will increase the test coverage for the entire package, but that requires special techniques: a mock database or something conceptually similar.


structure_from_coords finds too many regions

Hey guys, I'm making a probe planning app, as you can see above.
I use the structure_from_coords method to find through which brain regions the probe goes through. This works mostly fine but occasionally finds too much stuff:
For instance here:

it finds some stuff in the hypothalamus, miles away from the probe. I think that this is a bug with the atlas function and not with my probe planner stuff, but might require some investigation.

I think I have figure it out. If the coordinates are outside of the atlas space, it returns a structure counting the distance from the opposide side (like if you were using negative indices to get an item in a list).
There should be a check that if the coordinates are outside of where the brain outline is it should return None or raise an error

Thanks @FedeClaudi for the report?
Do you have a code snippet do to reproduce? If I just ask for coordinates outside the brain with an offset that matches the atlas shape I actually get an error with my code:
import numpy as np
from bg_atlasapi.bg_atlas import BrainGlobeAtlas

# This will automatically trigger the download of the zebrafish atlas. This atlas has a resolution of 1 um
atlas = BrainGlobeAtlas("mpin_zfish_1um")
atlas.structure_from_coords((500, 150, 300))  # return 833
atlas.structure_from_coords((500+974, 150, 300))  # gives an index error

Mmm that's weird, this example code:

from brainrender import Scene
import numpy as np
s = Scene()

root = s.atlas.get_region('root')
p0 = root.centerOfMass()

x = np.linspace(p0[0], p0[0]+10000, 20)
for p in x:
    point = p0.copy()
    point[0] = p
    print(p, root.mesh.isInside(point), s.atlas.structure_from_coords(point, as_acronym=True, microns=True))

print(p0)

raises an Indexerror, but pretty much the same code in my app didn't, it gave bogus results instead. Weird...

Is is possible that within the program there are some try-except that leave some default values unaltered?

I think you are right, sorry it seems that it's not a bg-atlasapi issue after all. Thanks Luigi


Feat: Add more features to the pinterest module

Describe the feature
I am adding 2 features to the pinterest module:
Feature 1: Search Pins by Keyword
Description:
Add functionality to search for pins containing a specific keyword on Pinterest. Users will be able to provide a keyword, and the script will scrape pins related to that keyword, including details such as pin title, description, link, and image URL.
Feature 2: Get Pin Details
Description:
Implement a feature to fetch details about a specific pin on Pinterest. Users can provide the URL of a pin, and the script will extract information such as pin title, description, number of saves, and number of comments.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

@nikhil25803 please assign this to me

Go ahead @priyankeshh
Note

Please create a separate module for this, as in the folder and project structure (if it is already created, just add your features as functions in the same module).
Do not use the `selenium web driver as it is incompatible with all devices and cloud platforms.
Before making any changes, please check whether the module you want to add exists. If yes, then you can add your functionality as a method only make a separate module and class for it.

All the best 👨💻

Due to some errors, I have closed this PR, I am making a new PR for this issue. Please check that out #999


Add autocomplete for type parameters in 3.12

Currently they don't appear in the autocomplete list. It would be really nice if they did

Thanks for the suggestion.


Add support for custom sub position parent alignment

Closing since this can be achieved by just observing the selection changes and updating the current parent configuration


Adding A New Section "Project Setup" In The Readme File

Issue: There is no section like "Project Setup" in the readme file.
Way to fix: This section will contain the link to the readme files of all the individual projects.
Reason to fix: So that people do not get confused to set up their preferred project.

Hello @phyleria would please assign this task to me?

Ok, go ahead


Memory leak

It looks like pywt has a memory leak. When I process more than 200K small images (<10 KB each) a process takes all 16GB of my memory and stops (or slows down).

Code

whash() function from imagehash lib:
https://github.com/JohannesBuchner/imagehash/pull/23/commits/da9386da76d3d32f6ba41160c7ef284102df36fc
Reproduction code snippet:
res = {}
i = 0
for filename in onlyfiles:
    fname = os.path.join(filedir, filename)
    try:
        image = PIL.Image.open(fname)
        whash = imagehash.whash(image)
        value = whash.hash.flatten()
        res[int(filename[:-4])] = sum(1<<i for i, b in enumerate(value) if b)
    except Exception as e:
        print("Error. File " + fname + ": ", e )

Btw.. imagehash.phash() function doesn't use pywt and doesn't have this issue.

Data

Image subset from Avito competition:
https://www.kaggle.com/c/avito-duplicate-ads-detection

Perhaps it is more helpful to write a test case code which does not use imagehash to show that the issue is indeed in pywt.

Does the memory leak for all kinds of modes/methods or only one in particular?

Candidates could be these two functions, which allocate a Wavelet object, and one should double-check that these are being de-allocated.
wavedec2
https://github.com/PyWavelets/pywt/blob/416c697d612ac75acfc3a216a193b2582701b62b/pywt/_multilevel.py
idwtn
https://github.com/PyWavelets/pywt/blob/416c697d612ac75acfc3a216a193b2582701b62b/pywt/_multidim.py

Reproduction code snippet without imagehash (files - list of input files, 200K+ files):
mode = 'haar'
level = 3
image_scale = 32
ll_max_level = int(numpy.log2(image_scale))
dwt_level = ll_max_level - level

for fname in files:
        image = PIL.Image.open(fname)
        image = image.convert("L").resize((image_scale, image_scale), PIL.Image.ANTIALIAS)
        pixels = numpy.array(image.getdata(), dtype=numpy.float).reshape((image_scale, image_scale))
        pixels /= 255
        coeffs = pywt.wavedec2(pixels, mode, level = dwt_level)
        dwt_low = coeffs[0]
        med = numpy.median(dwt_low)
        diff = dwt_low > med
        # Ignore result

I saw the leaking issue with 'haar' and 'db4' modes.

Can reproduce, will investigate.
import numpy as np
import pywt

for i in range(100000):
    pixels = np.ones((32, 32), dtype=np.float)
    pywt.wavedec2(pixels, 'haar', 5)

Throwing in a gc.collect() doesn't help.

Gotcha. It's these two lines in dwt_axis responsible, I'll figure out what to do about them in the morning.
    output_shape = (<size_t [:data.ndim]> <size_t *> data.shape).copy()
    output_shape[axis] = common.dwt_buffer_length(data.shape[axis], wavelet.dec_len, mode)

Is that a known issue in Cython ?   Could be worth raising it there ?

@stuaxo yes, I will raise it upstream, I have a minimal example here. http://trac.cython.org is dead at the moment though.

Cheers :) only mentioned as it gave me a horrible flashback to a similar bug I had before, but never tracked down in a different library.

@dmpetrov could you double check that this is fixed now?

I've checked the master branch version. Now it works much better. For one specific scenario it uses 400MB instead of 1.2GB of RAM.
However, the memory usage is still increasing. The growth rate is 3-10 times less then it use to be. I'm not sure if it is another issue in the library or just a regular Python behavior.
Anyway now we can process million of images with less then 1Gb of memory (less than 700MB to be precise). So, I think we can close the issue.

That still shouldn't be happening. I'll re-open this so I remember to chase it down.

@dmpetrov, could you try to do gc.collect() in your loop?

I'll try today night.
Sent from my iPhone

On Jul 7, 2016, at 7:02 AM, Johannes Buchner<EMAIL_ADDRESS>wrote:
@dmpetrov, could you try to do gc.collect() in your loop?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

tried gc() in the loop - the same result

I am unable to reproduce the issue with the following script, with 10 000 000 loop iterations memory use stays constant. Perhaps the bug is in PIL.
#!/usr/bin/env python3

import numpy as np
import pywt
import sys

mode = 'haar'
level = 3
image_scale = 32
ll_max_level = int(np.log2(image_scale))
dwt_level = ll_max_level - level

for _ in range(int(sys.argv[1])):
    pixels = np.ones((image_scale, image_scale), dtype="float")
    coeffs = pywt.wavedec2(pixels, mode, level = dwt_level)
    dwt_low = coeffs[0]
    med = np.median(dwt_low)
    diff = dwt_low > med
    # Ignore result

@kwohlfahrt you are right. I replaced image reading part to image copy and now the memory usage is constant.
Do you know where is the correct PIL repository for opening a bug https://github.com/python-pillow/Pillow ?

That looks like the right place (if you are using pillow instead of the old PIL). I haven't used it much to be honest. Closing this now, thanks for reporting and following up :)

Thank you for the fix!

I reported the bug to Pillow: https://github.com/python-pillow/Pillow/issues/2019


Zoom-in square animation

Describe the project you want to add with tech stack

This project contains a zoom in square animation.Can you assign this task to me?
Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 ,
@TusharKesarwani Can you assign this task to me?

@MohitGupta121 ,@TusharKesarwani Can you Please assign this task to me?As Gssoc contribution period is going to end..this issues can level me up ..Please assign this issue to me...Please.


Integrate updated memif API

This updates the API for creating memif sockets per changes required for upstream

memif socket IDs are now allocated opportunistically walking the hash
netns name is part of the socket name

This should superseed #271


How to store OTA certificate without using adding it manually in "aws_ota_codesigner_certificate.h" file

I am currently using NVS to store the OTA certificate. I would like to retrieve it via NVS by using a function. Is there any way to override the variable static const char signingcredentialSIGNING_CERTIFICATE_PEM[] and assign it to the return value of the function or parse an argument when initializing OTA to have custom certificates ?
Any other approach and suggestions are appreciated as well.

Have you considered provisioning the OTA certificate through PKCS #11? There is a pkcs11configOTA_SUPPORTED macro in core_pkcs11_config.h that you can use for this.

@MrMelon1232,
Could you try the provisioning that @yourslab mentioned? Are you looking for any further help from us?

Hi @leegeth @yourslab, I have found a workaround for this issue and will therefore close it.


Sprint Challenge #1

This is my completed sprint challenge #1

Comments:

You can't link to the Contact Page from the Blog Page.
You might also want to change the <title> tags to reflect the different pages as well.

Praises:

Those were the only things I could even find to complain about and it took me a while to find them. This is an awesome submission and you completed all of the MVP along with stretch while keeping your work clean and concise. Great first week, Hamza. 💯
You've also been extremely helpful throughout the week, and that's greatly appreciated as always -- I think there's a positive correlation between teaching and learning and you're proving that with this Sprint 🙌


Backup coded can be enabled through myAccount only when Authenticator App is configured

Describe the issue:

Backup coded can be enabled through myAccount only when Authenticator App is configured.
Otherwise the button would remain disabled.

According this message , backup coded should be abled to configured when atleast one  step is  configured
https://github.com/wso2/product-is/assets/40081059/693cf3bf-08a3-42d6-94f5-fbb2504fc2d5
How to reproduce:

Go to the myAccount =>security
Go to the Additional Authentication section
Configure Passkey and SMS Number

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

This issue occur since, by adding a mobile number, we are not enabling an authenticator.  As mentioned in the image above At least one additional authenticator should be configured to enable backup codes.
Hence, when we add the mobile number, we do not allow the generation of backup codes. Adding authenticator app (totp) is considered as the totp authenticator being enabled here.
Through myaccount is the only way a user can generate and download backup codes. So the user should be allowed to generate backup codes if that user has a chance of encountering smsOtp.
This can be solved by adding a validation to confirm if the user has a mobile number added and if mobile number is available, we can enable backup code generation. But since mobile number is not a claim that is required by default, it can be removed which will result in the removal of an authenticator (if we treat mobile number here as an authenticator). Hence an approach need to be finalised on how to enable backup code for smsOtp.

Feedback from @malithie :
We should not disable backup codes by authenticator at this point. There’s number of scenarios which are not of our control to decide that given we haven’t provided proper governance.
That means this should be always enabled in my account.


Change in ToString from 3.x to 4.0

string x = CBORObject.True.ToString();
results in "21" not in "true"
I just upgraded due to the security issue and am having this problem when doing comparisons on strings produced.  The output of "{1:21}" is really not the same as "{1:true}"

I am not doing the comparison in my program, but rather in the output.  I don't have any problems with doing updates of that string over time as things change, but this is not going to promote uses to understand things.  This string is dumped as part of examples for people in the COSE test suite.  Reading this is easier than reading a hex string.

Released version 4.1 which dealt with this issue.

Released version 4.0.1 which dealt with this issue.  However, note that for now, a more stable solution is to write your own method that converts CBOR objects to strings.  You can use the current implementation given in CBORObject.ToString() as a guide.


Odd manipulation of configmap with Istio

Expected behavior
Working Istio Sidecar Injector
Actual behavior
Istio Sidecar Injector failed due to odd config map manipulation on the side of Ship.
Information

Ship version: 0.44.0
Docker version: 18.06.2-ce
Command line run:

Steps to reproduce the behavior
See step How was Istio installed? in https://github.com/istio/istio/issues/15152
Ship apparently edited initContainers: \n into initContainers: []\n (as one example).
See https://github.com/istio/istio/issues/15152 for more information.

Interesting - I thought we had some code to handle that (specifically empty or comment-filled lines following a :) and it seems to be failing here.
The reason we have such code at all is that quite a few helm charts produce things that are not in fact valid yaml, and kustomize balks when presented with such.
Would you be willing to share the helm config used here?

The following should already produce the error:
ship init https://github.com/istio/istio/tree/1.2.0/install/kubernetes/helm/istio
grep "initContainers:" rendered.yaml

yields }}\n  initContainers: []\n

The reason we have such code at all is that quite a few helm charts produce things that are not in fact valid yaml, and kustomize balks when presented with such. It's really not my favorite piece of ship, [...]

Perhaps it would be better to get Helm to "fix" it? Imo, the only thing that ship should do is play the mediator (helm template -> kustomize).

@Timoses just saw this again - I need to fix my github notification settings
I'll take a look tonight/tomorrow

Oh, this makes a lot more sense after looking at it further
The edit made by ship is to the data within a configmap that has not yet been templated
So there are two ways I can see to fix this
One is to have this find/replace code recognize the contents of strings and avoid editing them, which would be ideal (but may not be practical)
The other (easier IMO) solution would be to have ship recognize the beginning of untemplated strings and accept that as a child item

The "recognize unexecuted template functions" fix has been merged and will be in the next release @Timoses


Run a subset of services

As the nubmer of services grows there may be use cases where only a subset of services should be brought up. It's not clear immediately how to do this in tilt.

Yeah, we should make this more evident.
There are currently two main options:

Run tilt up <service1> [<service2>] [...], which will run your tiltfile and then only deal with the services named on the command line. This is simple to use, but you can't change the service list without restarting Tilt, and it's not as convenient if you have a lot of services and want to have convenient ways to group them.
With the read_json Tiltfile builtin, e.g.:

svcgroup1():
  docker_build(...) // service A
  k8s_yaml(...) // service A

svcgroup2():
  docker_build(...) // service B
  k8s_yaml(...) // service B
  docker_build(...) // service C
  k8s_yaml(...) // service C

service_groups = {
  'svcgroup1': svcgroup1,
  'svcgroup2': svcgroup2
}

# default value = all services
selected_services = read_yaml('tilt_services.yaml', {}).get('services', services.keys())

for name in selected_services:
  services[name]()

and then in tilt_services.yaml put, e.g.:
services:
  - svcgroup1
  - svcgroup2

From there, you can comment/uncomment entries in the yaml to enable/disable them in tilt.
Some customers prefer adding the yaml file to .gitignore so they don't have to worry about committing those changes.
The example above probably has room for improvement, and maybe you just wanna go with tilt up svc1 svc2 svc3 ... anyway.
I'll leave this ticket open since we should have proper docs for this.

Is it possible to use target a subset of services that are read in from a docker_compose()?

hi @tmc -- not sure I understand your specific use case, can you give me an example?

Two things that may be relevant:

tilt DOES support using docker compose directly, if you want Tilt's UI and responsiveness with your existing docker compose file: https://docs.tilt.dev/docker_compose.html
your docker-compose.yml is a yaml file that you can read in as a dict with the read_yaml() builtin -- you can then get the service names out of that dictionary and use them to do what matt did in his example above (i.e. only call set-up functions for those specific services)

Given
docker_compose([
    "./docker-compose.yml",
    "./other-config.yaml"
])

can I write something akin to svcgroup1 above but have that group select a subset of compose-defined services?
At the end of the day I'd like to have closest to tilt up svcgroup1 or tilt up svcgroup1 without touching files on disk.

ah i see. Unfortunately no :(
if tilt up svc1 svc2 svc3 is too much of a pain, you might try:
export SVC_GRP_1='svc1 svc2 svc3' # in .bashrc or similar
tilt up $SVC_GRP_1


Re-add link to dspace for the deposit container.

Add link so that the deposit container can address the dspace container as pass.local

This got missed in the 0.1.0 deposit services integration.


Nightly build fails due to MSRV check

The nightly job fails because the MSRV check fails to execute.

closed via #208


Allow to change service name when creating new Span

At present "service_name" is set during tracer initialization and service name set at tracer level is used as service_name when reporting a span.
It would be better if user can be allowed to change (override) "service_name" when creating a new span.
Why?
It will allow developers to change service_name for spans created to track a remove server call.
E.g. While handling an API call from a service ( "api-service")  if we are making calls to external services like ElasticSearch or Mongo, new spans which are used to track these remote call can specify "service_name"="elasticsearch" or service_name="mongo" instead of using default service_name="api-service".

the name of the remote server should be set on the span as peer.service. Typically each RPC call is represented by two spans, one in the calling service (span.kind=client), and the other in the called service (span.kind=server). The "service name" remains constant in each service and tells which service emitted which span.

Thanks for that explanation, "peer.service" is set as a tag on span? I am setting tag "peer.service" on my span, but I don't see this in Jaeger UI.

Yes, it's one of the standard tags (there are constants for them in opentracing lib). You need to click on the span to expand its details to see the tags

Hmm, interesting I don't see in on my end. BTW, I am using jaegertracing/all-in-one:latest to log and view the traces. I did confirm that peer property on span object has correct values and trace JSON on Jaeger UI is also not showing any tag named peer.service any other custom tags I put from my app are showing there

This could be a bug in the backend. The Python client currently uses zipkin.thrift format to emit spans, which is internally transformed to Jaeger data model, and because Zipkin represents peer service in a different way, the transformation from zipkin to Jaeger format should reconstruct the peer.service tag.

Withdrawing my comment about the bug in the backend - I just checked a few of our internal services that use Go client, which also sends zipkin.thrift at this time, and they all show peer.service tags fine.

Thanks @yurishkuro  for checking and confirming that. I will do some additional debugging on my end.

You may want to run a tcpdump on the UDP port and check what kind of strings are being included in thrift

So I did run 'tcpdump' and confirmed peer.service tag is not getting propagated. I confirmed by switching to zipkin based tracing ( using py_zipkin) that all tags I am setting including ( peer.*) are making their way to zipkin.

It's probably this: https://github.com/uber/jaeger-client-python/blob/master/jaeger_client/span.py#L195
It looks like we do special handling for some opentracing tags in python (ie. we remove them but set the zipkin equivalent on the zipkin endpoint). On the backend, we should be converting the zipkin endpoint service name back into the peer.service tag though. Since you're not seeing that occur, looks like a bug either in the backend when we do the conversion or the client when we send the zipkin thrift to the agent. I'll take a look.

Looks like based on comment on https://github.com/uber/jaeger-client-python/blob/master/jaeger_client/span.py#L239 , this https://github.com/uber/jaeger-client-python/blob/master/jaeger_client/span.py#L201 should be changed to False?

Let me try making above change and testing it again, if it works will submit a PR

Given the way the backend zipkin to jaeger model conversion happens, we will be double reporting the same opentracing tags if we change it to False (granted the bug is fixed).

Ok, confirmed setting it to False does indeed make it work.

Are you setting the span.kind tag by any chance? If not can you try setting it?

No I am not, but I can try to set it.

Ok, if I set span.kind = client it does show up in Jaeger UI. So it seems you are right, setting False might not be the right fix and it must be an issue with Thrift conversion or somewhere else.

https://github.com/uber/jaeger-client-python/blob/master/jaeger_client/thrift.py#L149 If the span doesn't contain span.kind tag, we consider it a local component which according to opentracing semantics cannot have a peer service tag (because all the work occurs within the same service).
So you have to set the span.kind tag for all your incoming and outgoing spans for the peer.service tag to be set. I don't think there's a bug here.

Aah ok, thanks for clarifying that.


Datacite import not in sync with pagination

Describe the bug
The datacite import source is not in sync with the pagination form the external sources page.
To Reproduce
Steps to reproduce the behavior:

go to dspace demo and log in as some demo user.
Import 10.5281/zenodo from datacite (this should import a lot of entries https://api.datacite.org/dois?query=10.5281/zenodo)
Paginate through the results. The request is made for 10 results (see console log), but 25 are returned. The pagination does not show valid informations.

Expected behavior
We expect the pagination of the results to be in sync with the start and count settings
https://github.com/DSpace/DSpace/blob/344d79bfde40190e210ac2c5879c5e99676c595d/dspace-api/src/main/java/org/dspace/importer/external/datacite/DataCiteImportMetadataSourceServiceImpl.java#L88
according to the datacite api pagination . It seems like we have to forgotten to consider this parameters.
Related work
We can provide some fix for this issue.

There is also some issue with calculating the (total) number of records. By now the number of records is restricted to the pagination which default value is 25 (see screenshot). https://github.com/DSpace/DSpace/blob/344d79bfde40190e210ac2c5879c5e99676c595d/dspace-api/src/main/java/org/dspace/importer/external/datacite/DataCiteImportMetadataSourceServiceImpl.java#L76 In the screenshots describing the problem above the number should be around 448
There was some approach getting the total number of results from the request in the early stage of the datacite PR's  https://github.com/DSpace/DSpace/pull/8593/commits/a9cee40a15cdb00b4bf2e5f96bc7129beb57d36d#diff-49e1b00f5c9bbd91b2fb5ab6a67e7b88665f0331e66eb3344e557a15f8d25b95R345 , but it has unfortunately been modified meanwhile.
Since both topics are related to each other (pagination+total number of results) my fix would also include this topic. Or should i open an own issue for this?

@floriangantner : You can keep two related changes in the same ticket/PR.  That's perfectly OK.  We just try to discourage unrelated changes in the same PR, unless they are clearly described in the PR.


Real time data augmentation for color noise

Hi,
May I ask is there any way to do real time data augmentation with color noise?

I notice the parameter list in this file:
https://github.com/dmlc/mxnet/blob/5ff545f2345f9b607b81546a168665bd63d02d9f/R-package/man/mx.io.ImageRecordIter.Rd
It seems that  there is contrast, illumination fill_value factors to do augmentation, the explanation is not so clear. Could you elaborate more on this?
Still, it seems there is no noise generator on colors.


pyroute2 0.7.1

Got a dependency conflict @ Home Assistant

Caused by the release of pyroute2 0.7.1

Which originates from this repository

Will publish a new one later today

No rush 👍

Published 1.1.5
Need to wait for https://github.com/home-assistant/core/pull/75180 to bump it


Standardize on msg vs. message (or support both)

This has been a perennial source of confusion / bugs for me for our handful of APIs where we take a config with a "message" parameter.
E.g. ExceptionHandler takes message, FeedbackService msg, server-side tracking/error/feedback APIs use msg, ToastManager and MessageModel message, TrackService msg.
Given that msg appears to be baked further into the server, it might be most expedient to standardize on that, but I do think it could be worth it to provide for a fallback on the client-side APIs.

The specific issue I ran into right now was TrackService taking msg, so if we had one change to make I would vote for accepting either there, or switching to message and updating any current code.  That could be done as part of #209

atm and I both think 'message' makes the most sense here.
Let's just support msg silently, where necessary.  Best not to document that.

ok, will make sure message is supported everywhere.

We have decided that we will not accept msg in the JS API.  I have removed all instances of it.  The change is in the develop branch.


self signed certificate in certificate chain

Send grid Mail send response {"statusCode":500,"body":"{"message":"self signed certificate in certificate chain","name":"Error","stack":"Error: self signed certificate in certificate chain\n    at TLSSocket. (_tls_wrap.js:1088:38)\n    at emitNone (events.js:86:13)\n    at TLSSocket.emit (events.js:188:7)\n    at TLSSocket._finishInit (_tls_wrap.js:610:8)\n    at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:440:38)"}","headers":{}}

Hello @rekhasinghal08,
This appears to be a non-library/SDK related issue. I believe you have an issue with your certificate configuration.
With Best Regards,
Elmer


fix bugs and auto-import

using registration auto-import
waiting for the agent is registered
fix agent addon permission errors in hosted mode
ignore work manager log cert
build in managed-serviceaccount addon
refactor e2e

/assign @clyang82

/hold

/unhold


Proposal: Enhance User Experience with AOS (Animate On Scroll) Animations

Issue Description
Proposal:
Consider integrating the AOS (Animate On Scroll) library into the project to enhance the user experience with smooth and engaging animations triggered by scrolling.
Rationale:
AOS is a lightweight and user-friendly library that can be used to create eye-catching animations as elements come into view during scrolling. By incorporating AOS, we can add subtle animations to various sections of the website, such as fade-ins, slides, and other visually appealing effects.
Benefits:

Improved User Engagement: Animations can grab users' attention and make the website more engaging.
Enhance Visual Storytelling: AOS can be used to tell a visual story as users scroll through the content.

Implementation Steps:

Include the AOS library in the project by adding the necessary script tags or using package management tools.
Configure AOS with appropriate settings for the desired animations (e.g., duration, easing, offset, etc.).
Identify sections or elements where AOS animations will enhance the user experience, e.g in the landing page to animate the images into view.
Implement AOS animations on these sections/elements according to our design guidelines.

Additional Considerations:

AOS is well-documented, and the official documentation can serve as a valuable resource during implementation.
Collaborators and team members are encouraged to share their thoughts on this proposal and suggest specific areas where AOS animations could be beneficial.

References:

AOS Documentation

Nice idea. looking into docs


Issue following tutorial 'Seed a historical cost dataset with the Exports API'

[Enter feedback here]
Following the steps and using the API playground presents me with
{   "code": "MissingSubscription",   "message": "The request did not have a subscription or a valid tenant level resource provider." }
Is something missing from the provided Request body example?
Also there is no indication here of how far back in time you can seed data for. Is there a limitation?

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

ID: e172f0ad-bfe1-a489-e254-ac45c88c8ada
Version Independent ID: b642b931-db13-be5b-b7c4-b8e44a5ee725
Content: Tutorial - Seed a historical cost dataset with the Exports API - Microsoft Cost Management
Content Source: articles/cost-management-billing/automate/tutorial-seed-historical-cost-dataset-exports-api.md
Service: cost-management-billing
Sub-service: cost-management
GitHub Login: @bandersmsft
Microsoft Alias: banders

@curious-toast
Thanks for your feedback! We will investigate and update as appropriate.


add other cells without checkmark to promoter card

@vidhya-metacell I am assigning this  to you but you don't have the data yet to work on this.
Zoran will ping you with all the information you need to work on this. Thx

@vidhya-metacell the data is uploaded

@zsinnema how do i differentiate between cells that need to be with and without checkmark?

@vidhya-metacell the cells in the field CellsByLineaging are with the cells in field OtherCells are without

@zsinnema thank you


clightning: 23.05.2 -> 23.08

Description of changes
change log: https://github.com/ElementsProject/lightning/releases/tag/v23.08
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.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

CC @jb55 @prusnak

we need to add the clnrest plugin inside the postPatch = because we have some error

Ok, will wait for update of the PR

Need to replace this vanilla python hashbang, not sure how to actually do this in the build: https://github.com/ElementsProject/lightning/blob/master/plugins/clnrest/clnrest.py#L1

shebang updated here https://github.com/NixOS/nixpkgs/pull/253539
clnrest doesn't start, but at least it doesn't take down your whole node now.
we need a better solution for python dependencies.
this seems promising https://github.com/nix-community/poetry2nix
but we'd need to convert clnrest's requirements.txt file into a poetry project
also see: https://github.com/ElementsProject/lightning/issues/6637#issuecomment-1707595266

Merged this PR
Let's deal with the clnrest fixes in https://github.com/NixOS/nixpkgs/pull/253539


Win7 breakage with 0.37.0 Win8 only API PowerDeterminePlatformRoleEx

I'm not sure why this works on Chrome v49 itself but this is a win8 only API.  Maybe the WINVER setting isn't working properly in brightray's gypi

It could be patched by dynamically loading the api w/ load library and get proc address, but it's strange that chrome itself doesn't need to do this.  Maybe it uses a stub of that api somewhere / somehow.

I don't understand how Chrome can get it to work
From Powerbase.h
#if (NTDDI_VERSION >= NTDDI_WIN8)
POWER_PLATFORM_ROLE
WINAPI
PowerDeterminePlatformRoleEx(
    _In_ ULONG Version
    );

#endif

So it seems like Chrome has to compile with NTDDI_VERSION set to at least NTDDI_WIN8 or the code isn't included at all for some reason in Chrome.

I'm testing a patch that fixes this here:
https://github.com/brave/libchromiumcontent/commit/a94f0b71bff03ae49929d9ecdfa0b511968df5f9
If it works then I'll upstream to atom/libchromiumcontent

I would like to find out the root cause first, if we have a bad configuration making electron load with wrong symbols, we might have bigger troubles.

Wouldn't delay loading not solve the problem if the code is hit? Whereas this gracefully handles even if it is called?

This function is only called on Windows >= 8.

OK I guess Chrome will notice first if that's ever not the case.


fix: viewer list error message

This is my hack to showing an error, not sure the best way to address the problem so I'll iterate on this.

Nice work, this looks like some good practice.


On-Prem CRM 2013 IFD needing HomeRealm

I apologize if this is not the forum to apply this question but I'm struggling to use the XRM Framework per the subject provided.
As I attempt to use Get-CrmConnection, which I believe is just part of the SDK DLL's, I'm not able to establish a connection and export a solution via powershell. Here's the script I've written, any direction would be appreciated. I also get the same experience using the example file ExportSolution.ps1 but I'm having to remove the -ExportSales parameter because it seems to cause an issue or i just manually apply $false for -ExportSales but I still can't connect.
I apply the same connection string to Ping.ps1, I'm able to return the GUID. I also just tried it w/out the HomeRealm and ping.ps1 still worked. :/
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser
#Local folder location
Import-Module "C:\Client\Dynamics CRM SDK\Microsoft.Xrm.Data.PowerShell"

#ON-PREM
Connect-CrmOnlineDiscovery -InteractiveMode
#After connecting just select the section of script to run to avoid re-authentication

$xrmCIToolkit =  "C:\Client\Dynamics CRM SDK\XRM_CI\CRM365\CI_DLLs\Xrm.Framework.CI.PowerShell.Cmdlets.dll"
Write-Host "Importing CIToolkit: $xrmCIToolkit"
Import-Module $xrmCIToolkit
Write-Host "Imported CIToolkit"

$CrmConnectionString = "AuthType=IFD;Url=https://xxxxxxxxxxx.xx.dev.lab/ORGXXX;Domain=XX;Username=XXXXXXXX@XX.dev.lab;Password=XXXXXXX;RequireNewInstance=True;HomeRealmUri=https://xxxxxxx.xx.dev.lab/XRMServices/2011/Organization.svc"

#I also tried this just to exlude the XRM
#$cred = Get-Credential
#$CrmConnectionString = Get-CrmConnection -ServerUrl "https://xxxxxxx.xx.dev.lab/ORGXXX" -Credential $cred -OrganizationName "ORGXXX"

$exportedFile = Export-XrmSolution -ConnectionString $CrmConnectionString -IncludeVersionInName $false -UniqueSolutionName "TestSolution" -OutputFolder "C:\Client\CRM Solutions" -Managed $false

ERROR: this occurs when I manually set the -ExportSales param to false or remove it.
Exporting Unmanaged Solution
Export-XrmSolution : Unrecognized request parameter: ExportSales
At C:\client\Dynamics CRM SDK\xrm_ci\crm365\ci_dlls\ExportSolution.ps1:84 char:28
+ ... nagedFile = Export-XrmSolution -ConnectionString "$CrmConnectionStrin ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Export-XrmSolution], FaultException`1
    + FullyQualifiedErrorId : System.ServiceModel.FaultException`1[[Microsoft.Xrm.Sdk.OrganizationServiceFault, Microsoft.Xrm.Sdk, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35]],Xrm.Framework
   .CI.PowerShell.Cmdlets.ExportXrmSolutionCommand

Looks like you are using the Framework version that is not supported for CRM2013.
You could try to build the version from here: https://github.com/WaelHamze/xrm-ci-framework/tree/master/CRM2013/Xrm.Framework.CI


Are lights outside of the camera culled?

Just checking if lights outside the cameras viewpoint are culled, and how far away does that occur?
You might have a light-node that's just outside of the cameras view point where itself is not visible, but its light source should be.

Just checking if lights outside the cameras viewpoint are culled, and how far away does that occur? You might have a light-node that's just outside of the cameras view point where itself is not visible, but its light source should be.

@seivan I think it should be, because of how the occluders and point lights are queried, but I'm not really sure. I'll investigate and let you know.


Rerun reorders the column headers

Because the column headers are generated anew on a rerun, the order of them can be changed between runs.
Options

preffered - read the existing column headers in from the excel sheet, insert extra columns between bibtex parts and keywords if needed
easy - force an order, eg alphabetical, on the bibtex parts as well as the keywords

fixed via option 2 for now.  Alphabetical columns help looking data up anyway.


Execute conformance tests in the browser

I forgot that we also need to validate some of the tests in the browser. This means spawning a headless browser, sending it payloads, and executing those. I'll make sure to implement this.

Thanks for posting this one, yes this is a high priority.

We now have a browser test suite, which I think is one of the main requests here. WASI conformance in the browser is still not on the roadmap for now unfortunately.

That's fantastic news! Yeah, if we consider e.g. wasi:http for the browser again in the future we should run the conformance tests. But that can be tracked as part of any future efforts like that. ^^


Added accuracy validation for CPP part

I've supported ClassificationModels as well. Current tests show that the results diverge for Python and C++ API.

I will look into it


Mapper.MapDepthFrameToColorSpace 'The parameter is incorrect'

depthframe = kinect.get_last_depth_frame()
ptr_depth = numpy.ctypeslib.as_ctypes(depthframe.flatten())
L = kinect._depth_frame_data_capacity.value
S = 1080 * 1920
TYPE_ColorSpacePointArray = PyKinectV2._ColorSpacePoint * S
csps1 = TYPE_ColorSpacePointArray()
x = kinect._mapper.MapDepthFrameToColorSpace(L, ptr_depth, S, deneme)
i have an error => 'The parameter is incorrect'

Hi muharremmandaci, did you solve the issue ?

kinect = PyKinectRuntime.PyKinectRuntime(PyKinectV2.FrameSourceTypes_Color | PyKinectV2.FrameSourceTypes_Depth | PyKinectV2.FrameSourceTypes_Infrared)
color2depth_points_type = _DepthSpacePoint* np.int(19201080)
color2depth_points = ctypes.cast(color2depth_points_type(), ctypes.POINTER(_DepthSpacePoint))
kinect._mapper.MapColorFrameToDepthSpace(ctypes.c_uint(512 * 424), kinect._depth_frame_data, ctypes.c_uint(1920 * 1080), color2depth_points)
i used this code and it is fine ,then what we want is the 'color2depth_points'. if you want to take points from color space to depth_space (such as [255,255] in color space),just use color2depth_points[2551920+255-1].x   and  color2depth_points[255*1920+255-1].y  to take the points in depth space.

@Lancename , first thx for your code. It works without any error!
Though I have some problems right now.
I got a data from your code, but after I don't know how to use that. Can you give me a small help if you don't mind..?


Update rubocop to 0.68.1

Fixes #3197
The latest version of rubocop is version 0.75.0, but version 0.70.0 or above requires ruby 2.3.0 or above. Therefore, this PR is updating rubocop version 0.68.1 that supports ruby 2.2.2.
I put all new cops into .rubocop_todo.yml and disable them for now.

Nice work, thanks!


Support for white-space possible?

Hi, I would just like to know if support for the css prop white-space is technically possible.

Hey @fiws
you should check out https://github.com/bezoerb/text-metrics

Thanks


Ty in advance!

New version has been released


release: Klave #KN-233

Reason for change:

Enable users to :

Invite other developers to join their Organisation
Have access to multiple Organisation
Choose Organisation to access to and deploy to.

The dev is at fff758e4

The dev is at 6ee73b2a

The dev is at 6ee73b2a

LGTM

The dev is at b056ffd8


Not finish to search files when there is a directory named ","

I'm using 0.28.0 and ruby 2.1.5. When there is a directory named ,, rubocop command will not finish. I exclude , directory like below to fix this. The , character in the pattern to search files should be escaped.
AllCops:
  Exclude:
    - ",/**/*"

I cannot reproduce this error:
Code/rubocop-test » tree -a
.
├── ,
│   └── foo.rb
├── .rubocop.yml
└── foo.rb

1 directory, 3 files

Code/rubocop-test » cat .rubocop.yml
AllCops:
  Exclude:
    - ",/**/*"

Code/rubocop-test » rubocop --version
0.28.0

Code/rubocop-test » rubocop
Inspecting 1 file
.

1 file inspected, no offenses detected

Sorry for my poor description. I can reproduce this problem.
% tree .
.
├── ,
│   └── foo.rb
└── foo.rb
% rubocop
(not finished)

Ah, I see. The problem is in TargetFinder#find_files, where the “wanted toplevel dirs” (which includes ,) are joined together into a Dir.glob pattern. And the dirs are joined by commas…
I’ll try to come up with a solution.


Batch state updates to avoid recreation of state object multiple times

When a large table/query with many columns is profiled there are a lot of small state updates that happen in quick succession. Every time immer will create a readonly copy of the state object that our code modifies. If we batch the creation of this readonly copy we can gain some performance improvement on large table/query.

Hamilton Ulmer
May 3, 2022, 8:14 PM
moved this to sprint 12, in case there’s room to do this.
Edit
Delete
Hamilton Ulmer
April 1, 2022, 10:20 PM
It’s possible that we may also be able to reduce some frontend jank here ~ with a large number of histograms,, some slowness will still be present due to the way we are naively drawing things. I think we can file a separate ticket for this.
Edit
Delete


Can we add more column to query SSE key_type?

This is for the requirement
CIS 5.9 Ensure server-side encryption is set to ‘Encrypt with BYOK’ (Manual)
Use case - the bucket is enable with KMS > CMK, when we use CMK, the KMSMasterKeyID get populated with the value. However it does not explicitly mention CMK as provided in screen
select
  name,
  jsonb_pretty(server_side_encryption) from alicloud_oss_bucket where name = 'cis-test2'

+-----------+---------------------------------------------------------------+
| name      | jsonb_pretty                                                  |
+-----------+---------------------------------------------------------------+
| cis-test2 | {                                                             |
|           |     "SSEAlgorithm": "KMS",                                    |
|           |     "KMSMasterKeyID": "484f854b-a629-4cd5-b618-eb02c0eb5146", |
|           |     "KMSDataEncryption": ""                                   |
|           | }                                                             |
+-----------+---------------------------------------------------------------+

When we make the query with the key_id to  alicloud_kms_key, that shows Aliyun_AES_256, hence its difficult to know if CMK.
 select
  key_id,
  arn,
  key_state,key_spec,
  description,
  creation_date,
  region
from
  alicloud_kms_key where key_id = '484f854b-a629-4cd512344545-0eb5146'

The result returns as below

Though we can map the KMSMasterKeyID check to know if its CMK or not, it would be great if we can solve this confusion by adding a value add column.

creator column from alicloud_kms_key table can identify if the type is CMK. When CMK, creator value will be account id.


Add Graphic Converter to supported softwares

Thanks for the pull request, please fill out the cla (see instructions above) and add yourself to the authors list

Thanks!

You’re welcome, thank you!


The build scripts install random stuff on my system and break it.

The build scripts install tools and libraries on my system as if it was alone on it. All dependencies not explicitly installed by the user should be put in the build directory instead.

axmol version: 2.0.0

Steps to Reproduce:
Follow https://github.com/axmolengine/axmol/blob/dev/docs/DevSetup.md

install-pwsh will download .deb files from https://github.com/PowerShell/PowerShell/releases and install them on my system, prompting for sudo rights. IMHO using sudo in a script is bad practice as a broken script may break the user's environment.

Suggested alternative: use CMake's FetchContent to download the required tools in the build directory if they cannot be found on the system. Ask the user to install them.

setup.ps1 runs the following command: sudo apt install --allow-unauthenticated --yes $DEPENDS > /dev/null. Again, there is the sudo problem. There is also a security problem with --allow-unauthenticated, and the output is hidden from the user!

Suggested alternative: use CMake's find_library/find_program functions and ask the user to install de missing dependencies themselves. Don't mess with the user's environment.

Still in setup.ps1, we have an installation of FreeType in /usr: cmake …  '-DCMAKE_INSTALL_PREFIX=/usr'  …', then sudo cmake --build build --config Release --target install. Now the FreeType library and headers I had installed from my distribution's repository are broken, as well as my other development requiring them >:(

Suggested alternative: use CMake facilities. Ask the user to install the dependency, let them pass the search paths to CMake. Do your internal matter in the build directory.

I think you mean dev on linux:

install-pwsh is only for user to easy install powershell, not the only way.
inherited from cocos2d-x
if ubuntu system installed libfreetype-dev, due to freetype source code use include <freetype/xxx.h>, it will include system installed freetype, if version may match axmol required, will trigger compile error, that why I force install it to /usr/ in script setup.ps


zip should use the length of the largest array

Most other libraries use a zip implementation that will apply the length of the largest array for the result.
zip([1, 2, 3], ['a', 'b', 'c', 'd']); // => [[1,"a"],[2,"b"],[3,"c"],[undefined,"d"]]

+1 for R.unzip

@kwijibo: Interested in creating a Pull Request?

@CrossEye I'll give it a go