Unity will crash on Flutter 1.17 and any beta channel version

Hi,
I got crash on iOS, the message is listed as below
ClassID 100000 (int) conflicts with that of another class (int). Please resolve the conflict.
However, the older stable version 1.12 is fine, does anyone know about this issue ?

UnityUpdateMuteState([audioSession outputVolume] < 0.01f ? 1 : 0);
// Modified by https://github.com/snowballdigital/flutter-unity-view-widget
[[NSNotificationCenter defaultCenter] postNotificationName: @"UnityReady" object:self];

I found root cause that UnityReady event didn't be posted, and startUnity didn't be called, so it will init unity twice, then crash on RegisterAllClasses.
It is also happened on Flutter unity sample project.
However, the flutter 1.12 version with the Unity plugin, everything is normal, so maybe need to take look why startUnity didn't be called.

I fixed the issue by directly call start unity method on UnityAppController.mm.
 [self startUnity:application];
 // [self performSelector: @selector(startUnity:) withObject: application afterDelay: 0];

Then add below before InitUnity inside dispatch_async
if (_isUnityReady) {
  completed();
  return;
}

It can resolve the crash issue, however flutter ui can't show up on UnityView.
=> After insert unityview on FlutterUnityView, it will occupy whole screen, the flutter ui can't be saw.

I fix crash issue by adding

it resolved, but the interaction with the unity view is null, I cannot interact with unity view even with the ui inside the unity app

Where exactly do you need to place this code:
if (_isUnityReady) {
  completed();
  return;
}

I cant seem to find the _isUnityReady variable.

Where exactly do you need to place this code:
if (_isUnityReady) {
  completed();
  return;
}

I cant seem to find the _isUnityReady variable.

Is on UnityUtils.mm file, around  line 156 of the file

Where exactly do you need to place this code:
if (_isUnityReady) {
  completed();
  return;
}

I cant seem to find the _isUnityReady variable.

Is on UnityUtils.mm file, around line 156 of the file

Thanks! Main issue was locating the file itself. I was looking in the UnityExport folder. :(

is on flutter-unity-view-widget\ios\Classes

it resolved, but the interaction with the unity view is null, I cannot interact with unity view even with the ui inside the unity app

please check the https://github.com/flutter/flutter/issues/56660
it can resolve flutter ui issue with Unity.
The main reason platform uiview need to declare at initWithFrame method
Put the following code after [_channel setMethodCallHandler]
_uView = [[FlutterUnityView alloc] init];
if ([UnityUtils isUnityReady]) {
    [_uView setUnityView: (UIView*)[GetAppController() unityView]];
} else {
    [UnityUtils createPlayer:^{
         [_uView setUnityView: (UIView*)[GetAppController() unityView]];
    }];
    [GetAppController() setUnityMessageHandler: ^(const char* message)
    {
         [_channel invokeMethod:@"onUnityMessage" arguments:[NSString stringWithUTF8String:message]];
     }];
 }

Then only return uiview
- (UIView*)view {
    return _uView;
}

@skypu3 thanks, yes it resolve the issue. Thanks for the tip.

it resolved, but the interaction with the unity view is null, I cannot interact with unity view even with the ui inside the unity app

please check the flutter/flutter#56660
it can resolve flutter ui issue with Unity.
The main reason platform uiview need to declare at initWithFrame method
Put the following code after [_channel setMethodCallHandler]
_uView = [[FlutterUnityView alloc] init];
if ([UnityUtils isUnityReady]) {
    [_uView setUnityView: (UIView*)[GetAppController() unityView]];
} else {
    [UnityUtils createPlayer:^{
         [_uView setUnityView: (UIView*)[GetAppController() unityView]];
    }];
    [GetAppController() setUnityMessageHandler: ^(const char* message)
    {
         [_channel invokeMethod:@"onUnityMessage" arguments:[NSString stringWithUTF8String:message]];
     }];
 }

Then only return uiview
- (UIView*)view {
    return _uView;
}

Thanks for all of your work on this! Can you be more specific on where to put this code as i cant get it to work unfortunately :-(

Thanks for all of your work on this! Can you be more specific on where to put this code as i cant get it to work unfortunately :-(

Spoke too soon... i have it working! Remembering to refresh the pubspec.yaml !

it resolved, but the interaction with the unity view is null, I cannot interact with unity view even with the ui inside the unity app

please check the flutter/flutter#56660
it can resolve flutter ui issue with Unity.
The main reason platform uiview need to declare at initWithFrame method
Put the following code after [_channel setMethodCallHandler]
_uView = [[FlutterUnityView alloc] init];
if ([UnityUtils isUnityReady]) {
    [_uView setUnityView: (UIView*)[GetAppController() unityView]];
} else {
    [UnityUtils createPlayer:^{
         [_uView setUnityView: (UIView*)[GetAppController() unityView]];
    }];
    [GetAppController() setUnityMessageHandler: ^(const char* message)
    {
         [_channel invokeMethod:@"onUnityMessage" arguments:[NSString stringWithUTF8String:message]];
     }];
 }

Then only return uiview
- (UIView*)view {
    return _uView;
}

This work for me but the answers its difficult,you have to move the first code below [_channel setMethodCallHandler],and then replace the method to only return the uView.
Thank you very much!!


Preloader added

Description
I have added the preloader.Can it be merged now?
Please provide a brief description of the changes introduced in this pull request. Explain the problem you are solving or the feature you are adding.
Pull Request Checklist
Please make sure that your PR meets the following requirements:

[x] The app is working correctly after applying the changes.
[] The app is responsive and displays properly on different screen sizes and devices.
[ ] The changes are thoroughly documented in the appropriate files or documentation sections.
[ ] The coding conventions and style guidelines are followed consistently.

Checklist
Please check all applicable boxes and provide any additional information if needed.

[ ] I have tested the changes locally and ensured that the app is functioning as expected.
[ ] I have verified that the app is responsive and looks good on various screen sizes and devices.
[ ] I have reviewed the documentation and ensured that it accurately reflects the changes made.
[ ] I have followed the coding conventions and style guidelines defined in the project.
[ ] I have run the automated tests and all of them pass.
[ ] I have considered the performance implications of the changes made.
[ ] I have considered the security implications of the changes made.
[ ] I have checked for any potential regressions or side effects caused by the changes.
[ ] I have performed a self-review of my code.
[ ] Starred the repository.
[ ] Added My Name to the Contributor's List.
[ ] Followed the repository's Contributing Guidelines.
[ ] I ran the app and tested it locally to verify that it works as expected.

Screenshots - If Any (Optional)
Additional Notes(Optional)
Please provide any additional information or context that may be helpful for reviewing the pull request.
Related Issues
Please list any related issues or pull requests that are associated with this pull request.
Reviewers
Please tag the relevant team members or reviewers who should review this pull request.

https://github.com/amupedia2021/Project-Amupedia/assets/88332977/d40e2ca9-e5d0-43ae-bb8c-f0ff900e4114
It is not optimized!! Please look at....

Ya, I'll look at it

I have done some change.Can it be merged now?

I have done some change.Can it be merged now?

@Shreyans2004 I see the site page being rendered twice. In simple terms, After footer I can see navbar and whole thing repeating. Please take a look in this.


Update TCBlobDownloader.m

fix the progress when restart

I love the TCBlob domo， but I find some bugs in your domo

Sorry it took so long. Thank you, I will polish this and publish a new version.
I did not write the multiple file downloads demo, but if there is any bug, then please, I would be happy to accept another PR :)


Cannot generate op.h when build with mkl

os : windows 10 64bits
compiler : vc2015 64bits update 3
mxnet : 1.3.1

I write down the steps of building at stackoverflow, please read it until step 14 if you want to know the details.
I use python download from here to generate op.h, but it throw error message
2>  "Running: OpWrapperGenerator.py"
2>  Traceback (most recent call last):
2>    File "OpWrapperGenerator.py", line 432, in <module>
2>      raise(e)
2>    File "OpWrapperGenerator.py", line 426, in <module>
2>      f.write(patternStr % ParseAllOps())
2>    File "OpWrapperGenerator.py", line 320, in ParseAllOps
2>      cdll.libmxnet = cdll.LoadLibrary(sys.argv[1])
2>    File "C:\Users\yyyy\AppData\Local\Programs\Python\Python37-32\lib\ctypes\__init__.py", line 434, in LoadLibrary
2>      return self._dlltype(name)
2>    File "C:\Users\yyyy\AppData\Local\Programs\Python\Python37-32\lib\ctypes\__init__.py", line 356, in __init__
2>      self._handle = _dlopen(self._name, mode)
2>  OSError: [WinError 193] %1 is not a valid Win32 application

Dependency of libmxnet.dll(by dumpbin)
mkl_rt.dll
KERNEL32.dll
VCOMP140.DLL

Dependency of mkl_rt.dll(by dumpbin)
KERNEL32.dll
I put both of the dll, mkl_rt.dll and libmxnet.dll into a folder could be found by the os, try to generate op.h again, but it still give me the same error, how could I solve this issue? Thanks
ps : Use Anaconda3 can generate op.h, but it cause another issue " Intel MKL FATAL ERROR: Cannot load mkl_intel_thread.dll."

Thanks, I see you raise the issue in the gluon forum.
@xinyu-intel is looking for the issue and will reply to you soon.

@mxnet-label-bot Add [MKL, Build]

@xinyu-intel is looking for the issue and will reply to you soon.

Thanks, by the way, I build mxnet 1.3.1 with gpu and find out I can use gpu to do the inference task(with mxnet v1.3.1), but when I switch to cpu, it throw the same error again " Intel MKL FATAL ERROR: Cannot load mkl_intel_thread.dll."
Not sure which call in the forward function need this dll yet it can't find it.

Hi,
you need include the right folder of the Intel MKL software package on your PATH env. var. It is probably "C:\Program Files (x86)\IntelSWTools\compilers_and_libraries_2019.2.190\windows\redist\intel64\mkl" .
I do not have access to my develop PC to check.
Also, if you have configured OpenCV, you need to add the folder containing the binaries of this project to the PATH. Same for CUDNN support.
Good luck
By the way, are you able to compile c++ program with op.h included without errors?

"C:\Program Files (x86)\IntelSWTools\compilers_and_libraries_2019.2.190\windows\redist\intel64\mkl" .

No use, I copied every dll in that folder and put them into the same folder with the exe, but it still give me the same error

if you have configured OpenCV

No, I don' need to build im2rec by myself, gluoncv installed by pip got that exe

Same for CUDNN support.

CUDNN works fine, but cpu with mkl do not

By the way, are you able to compile c++ program with op.h included without errors?

It got lots of error because many arguments(learning rate and weigth decay of each functions) do not declare the type, I make them as mx_float manually.
Thanks for your helps, you could study the post I put at stackoverflow, it mention how do I build the mxnet1.3.1 and every solutions I have tried.
I guess this problem wouldn't be able to solve unless someone dig into the codes

The libs I link to in my small program are opencv_world342.lib and libmxnet.lib, do I need to link with more lib?
ps : I build the mxnet without opencv, but my program use opencv to read the image

If you use opencv to read the image, you might put the release folder of opencv in your PATH.
I got 1.3.0 without CUDA support but with MKL running on Windows 10 with CPP_PACKAGE.
Also I got 1.3.1 compiled (same errors) and running with both CUDA and MKL support a but my own program crashes without any warnings/log messages when I call NDArray::WAitall after calling the executor.
I did not attempt to use MKL_DNN or MKLML. As the CMake file has the condition IF NOT MSVC. Although the doc I found, stated that these can be enabled.
When I am back on Monday on the office, I will look at the notes I made for CUDA/MKL.
I am working 2 years with mxnet . With python everything is fine. But getting it compiled and running on Windows/CPP_Package is a real struggle. Sparsely documented, often new build errors.

If you use opencv to read the image, you might need the release folder of opencv in your PATH.

Yes, I already put them into PATH. My solution is create a global_dll folder and add it into my PATH, this way I could switch different dll easier without adding too many PATH. Add so many PATH could cause some nasty troubles.

I did not attempt to use MKL_DNN or MKLML

I did not use MKLML either, because the CMake file of mshadow will forced to use openBLAS

I will look at the notes I made for CUDA/MKL.

Thanks

getting it compiled and running on Windows/CPP_Package is a real struggle. Sparsely/flaky documented, often new build errors.

Know your pain, guess I should give 1.3.0 a try. If things are like this since two years ago, that means windows/CPP_Package is not their major concern. 1.3.1 can't build with openBLAS(maybe it is because I update vc2015) and got runtime issues of cpu version. 1.4.0 can't compile on windows, many bugs.

With python everything is fine.

mxnet installed by pip works well on my pc too, api of gluoncv and mxnet are well designed, but python is not the right tool when I want to deploy my app, I need c++ for this kind of issues, unless most of my customers ask me to make the app able to run on windows, windows still dominate 90% of desktop market.
dnn module of opencv4 planning to support cuda and cudnn, hope it can success so we could have a tool easier to build and use.

I guess I found the issue, you need to enable MKL_USE_ILP64(I am surprise it can compile even my pc only install MKL with 64 bits), else the mxnet will try to build with 32bits library, but the problem is when you enable that options, you will get tons of compile time errors.

is there a workaround for this issue? Will it work without mkl? Or is c++ build completely broken?

For me it works (version 1.3.0) with CUDA 9.2, CUDNN 7, OPENCV 3.4.4 with VS2017 and Windows 10. OpenCV was complied without MKL support.
Version 1.4.x compiles with same config (except opencv3.4.5), but I did not link it yet to my application.
On [https://zhuanlan.zhihu.com/p/58918998] you find a quite good guide I guess, my script is almost the same. Maybe you have to translate to English (I did).

A note on gtest and gtest_main projects, I set option "Treat warnings as errors" on "no" (compiler option /WX-)
Otherwise the projects do not compile.

I was able to build properly 1.x and 2.0 on windows 10 with cmake - @szha I think we can close this one as outdated - @stereomatchingkiss if you don't agree feel free to reopen and ping me


Cmake support snappy

Windows CMake builds do not include SNAPPY compression library support, thus the resulting applications are not able to parse TensorFlow models that were compressed with SNAPPY.
This pull request adds SNAPPY as ExternalProject, just as it's done with all other 3rd party libraries.
Unfortunately I cannot verify this on a Linux machine right now, so option is disabled for Linux by default.
There is a relevant Stackoverflow question: https://stackoverflow.com/questions/45304986/cant-read-saved-tensorflow-model-failed-to-seek-to-header-entry
This is my first contribution, so I apologize in advance for any guidelines I might have violated. Feedback is appreciated 👍

Can one of the admins verify this patch?

Seems like I used different emails for my commits and for the Contributor License Agreement. Let me close this pull request and create another one, it'll be easier.


agent: add startup watchdog logging missing components

adding a startup watchdog that logs every 5 seconds what we are missing for agent to start (felixconfig, bgpspec)

Closed in favor of #505


[DOCS] Fix a typo in Encoder.clsTag

What changes were proposed in this pull request?
Fixes a typo: and -> an
How was this patch tested?
Not at all.

Can one of the admins verify this patch?

Yes, but we generally ask people to do a more comprehensive sweep for typos if they're going to bother. It's a lot of overhead to merge a bunch of these

ping @mineo

Merged this to close it, but we'd generally discourage this


release-0.14: bump Helm chart README for v0.14.1

What this PR does / why we need it:
Update the Helm chart README from v0.14.1 :)
Release note:
NONE

/kind documentation
/area deploy
/assign @meyskens

/lgtm


adding create assign and create table features

This PR introduces a minimal implementation of object and table structures, designed as slice helpers for the Starfx store. This is aimed at preserving the modus operandi users are accustomed to when using Robodux.
Key Changes
Created a new directory in root, named tables.
Introduced fxCreateAsign to manage object-like structures.
Introduced fxCreateTable to manage table-like structures.
Established initial state setup through createQueryState and preserve-it by creating and export of it on createStore.
Usage of  ts-pattern library for action matching operations.
Added a set of actions for manipulating data, including add, remove, update, etc.
Incorporated a few tests to validate the implementation's functionality.

Closing this PR, as we are heading to a more mature approach on createAssign / createTable


Wrong path removing files from file storage

Docker image version: 1.4.13
My configuration:
  FILE_RESULT_STORAGE_BASE_DIR: /data/results
  FILE_RESULT_STORAGE_EXPIRATION: 48h
  FILE_RESULT_STORAGE_EXPIRATION_SECONDS: "172800"
  FILE_STORAGE_BASE_DIR: /data/storage
  FILE_STORAGE_EXPIRATION: 48h
  FILE_STORAGE_EXPIRATION_SECONDS: "172800"
  IMAGOR_RESULT_STORAGE_PATH_STYLE: digest
  IMAGOR_STORAGE_PATH_STYLE: digest
  IMAGOR_UNSAFE: "1"
  PORT: "8000"
  PROMETHEUS_BIND: :7000
  PROMETHEUS_PATH: /metrics

Ignore "*_STORAGE_EXPIRATION_SECONDS" vars. They are used for a cleaning storage cronjob.
So imagor is using hashed paths for saving in file storage but it's trying to remove the files using the origin encoded url in the path and obviously no file there. Some logs:
2024-08-23 10:02:12.669Z imagor {"level":"warn","ts":1724407332.6698532,"caller":"imagor/imagor.go:586","msg":"delete","key":"https://s3.amazonaws.com/bucketname/hotel/0a80b01d-beed-4341-b99c-0e5bd57c2dae","error":"remove /data/storage/https%3A/s3.amazonaws.com/bucketname/hotel/0a80b01d-beed-4341-b99c-0e5bd57c2dae: no such file or directory"}

https://github.com/cshum/imagor/pkgs/container/imagor/262909827?tag=master
You may test the docker build at master branch:
ghcr.io/cshum/imagor@sha256:039522d05b9e0bcd45e7749833dec36fc3988e2ec9d24515612a6cb547ee0634

Available in v1.4.15


run cannot recognize alias

I have a function:
function error_msg_func()
{
  local filename="${1}"
  local lineno=${2}
  echo "error xxx (${filename}:${lineno})"
}

For convenient:
shopt -s expand_aliases
alias error_msg='error_msg_func "${BASH_SOURCE[0]}" ${LINENO}'

So I can easy to use:
error_msg

and no need to type "${BASH_SOURCE[0]}" and ${LINENO} every time
When I try to test it:
@test "error_msg" {
  run error_msg
  [ "${output}" = <expected result> ]
}

I got bats/libexec/bats-exec-test: line 58: error_msg: command not found
however, the following one is okay:
@test "error_msg_func" {
  run error_msg_func "${BASH_SOURCE[0]}" ${LINENO}
  [ "${output}" = <expected result> ]
}

If you use Bash 4, you can use builtin BASH_ALIASES (refer to man page for Bash) variable to emulate aliases;
Say that you want to use myAlias in test cases, then you can define a function as follows in your test helper script:
In test_helper.bash
myAlias() {
  $( echo ${BASH_ALIASES[myAlias]} $@ )
}

In your test case:
load test_helper

@test "test" {
    run myAlias hello world
    [ $status -eq 0 ]
}

But if run command natively supports aliases, I think it would be much more great and convenient;

Why not just declare a function instead of an alias? They're more reliable and don't require escaping strings:
error_msg() {
  error_msg_func "${BASH_SOURCE[0]}" ${LINENO}
}

Of course, we all use functions. We implement all functionalities within functions, but

They usually end up with lots of parameters and

Also function names should avoid possible name collision with other command names: (So you may use long names or just use extra underscores...)
  _core_func foo var baz hello world true false

This is good. But except one or two params, users may rarely use the rest of the params, which may happen in real world. Also, some params may be just for internal usage, which users don't have to know in the first place.
Also, making the users type extra characters (like underscores) for your function names might be annoying from the user's perspective.
One workaround is, we use aliases like so:
alias al='_core_func foo var baz hello world true false'

So users just type less characters and they don't have to focus on extra details until they seriously need to access those extra details.
Of course, you may use simpler function names and wrapper functions with less params to wrap your core functions, which may look a lot user-friendly.
But still, name collisions may happen when you stick to functions as your outermost interface.
Of course, you need to provide users a simple mechanism to replace the default alias names with whatever names suit them best.
Please, see my file search tool lf.sh for more detailed usage
Or refer to z which is a more renowned project which I referenced to build lf.sh


Observable API

Reconstruction of @martijnwalraven's PR #97, on top of current master.

@martijnwalraven I think I've reconstructed your whole PR on top of the two other features I merged -- let me know if I missed anything, but it looks like all the tests pass now!

@martijnwalraven @stubailo https://github.com/reactjs/redux/pull/1632

@jbaxleyiii that would be an awesome way to rebuild this functionality once it ships! I'm glad that observables and Redux are something that are going to be officially together soon :]

Rebased on master

Filed an issue about the Redux observable thing: https://github.com/apollostack/apollo-client/issues/123


HIP build failure with GCC 10

I'm trying to build CuPy 10.0.0b2 with HIP (ROCm 4.2.0) and GCC 10 on Linux, and hit the following error. The issue goes away if I downgrade to GCC 9.
Some searches on Google seem to indicate this is an issue with failing to include stdexception.
gcc -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -D_FORCE_INLINES=1 -DCUPY_USE_HIP=1 -D__HIP_PLATFORM_AMD__=1 -D__HIP_PLATFORM_HCC__=1 -DCUPY_CUB_VERSION_CODE=201009 -I/opt/rocm-4.2.0/hipfft/include -I/opt/rocm-4.2.0/include/hipcub -I/tmp/pip-install-n00zntfn/cupy_8fe2c3678294417ab36bc570b68e91c1/install/../cupy/_core/include -I/opt/rocm-4.2.0/include -I/opt/rocm-4.2.0/include/hip -I/opt/rocm-4.2.0/include/rocrand -I/opt/rocm-4.2.0/include/hiprand -I/opt/rocm-4.2.0/include/roctracer -I/autofs/nccs-svm1_home1/eslaught/spock/spinifel/setup/conda/envs/myenv/include/python3.8 -c cupy_backends/cuda/libs/cublas.cpp -o build/temp.linux-x86_64-3.8/cupy_backends/cuda/libs/cublas.o -std=c++11
cc1plus: warning: command-line option '-Wstrict-prototypes' is valid for C/ObjC but not for C++
In file included from cupy_backends/cuda/libs/../../cupy_blas.h:6,
                 from cupy_backends/cuda/libs/cublas.cpp:706:
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h: In function 'hipblasFillMode_t convert_hipblasFillMode_t(cublasFillMode_t)':
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h:35:29: error: 'runtime_error' is not a member of 'std'
   35 |         default: throw std::runtime_error("unrecognized mode");
      |                             ^~~~~~~~~~~~~
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h: In function 'hipblasDatatype_t convert_hipblasDatatype_t(cudaDataType_t)':
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h:59:29: error: 'runtime_error' is not a member of 'std'
   59 |         default: throw std::runtime_error("unrecognized type");
      |                             ^~~~~~~~~~~~~
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h:59:9: warning: control reaches end of non-void function [-Wreturn-type]
   59 |         default: throw std::runtime_error("unrecognized type");
      |         ^~~~~~~
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h: In function 'hipblasFillMode_t convert_hipblasFillMode_t(cublasFillMode_t)':
cupy_backends/cuda/libs/../../hip/cupy_hipblas.h:35:9: warning: control reaches end of non-void function [-Wreturn-type]
   35 |         default: throw std::runtime_error("unrecognized mode");
      |         ^~~~~~~

Command to reproduce:
pip install --no-cache-dir --pre --no-clean cupy

Conditions:

CuPy version: 10.0.0b2
OS/Platform: Linux
HIP version: ROCm 4.2.0
Compiler: GCC 10

Failing compiler:
$ gcc --version
gcc (GCC) 10.3.0 20210408 (Cray Inc.)
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Working compiler:
$ gcc --version
gcc (GCC) 9.3.0 20200312 (Cray Inc.)
Copyright (C) 2019 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

could you test https://github.com/cupy/cupy/pull/5769 ?
Thanks

Thanks for reporting!
Besides that we should handle exceptions gracefully... 😅
>>> cupy.cuda.cusparse.createDnMat(0, 0, 0, 0, 0, 0)
terminate called after throwing an instance of 'std::runtime_error'
  what():  unrecognized type
zsh: abort (core dumped)  python

Confirmed on my end that the PR works. Thanks!


Resolve partitioning error

When parsing the Aurora Justice Court .pdf file, the script is encountering an error, ``partition': wrong argument type nil (expected Regexp) (TypeError)`
Upon investigation, the .pdf file is empty. The script should be revised to handle this logic.

This error also applies for the Alta Justice Court

This error also applies for the Enterprise City Justice Court


Desktop background not recognized

Use configuration of: #295
Use latest Server from SVN and change following in LaunchCommand.java
if (projx == null || projx.isDestroyed() || !projx.isConnected())
->
if (projx == null || !projx.isDestroyed() || !projx.isConnected())
With this code, the application won't do an automatic logout after session expires.
If you press Browser Reload -> logout will be done and login will appear, but the desktop image is wrong. Also after login, no desktop image.
Not sure if this a real use-case, but we should check this case!

After the browser reload, I only receive the DesktopPanel and the shadow icon at the top. The usual image is missing in the response.

This was a problem on server-side. The UI was changed during sending of response and not before.


Ports in node header

Is it somehow possible to have an input/output port in the header of a node, like here, for example:

This would be useful if you want to create visual scripting applications that have an execution flow and (optionally) a separate data flow along the nodes of the graph.  The author of the tool in the above screenshot simply used a USS stylesheet cheat with negative vertical offset. But maybe this could be more cleanly done, as I think it could be a potentially valuable use-case.

Ok, I found a way to do this, but it requires the methods AddPort() and InsertPort() in the BaseNodeView class to be virtual and overridable.
Could you make this update in the NGP code, so developers can override adding/inserting ports?


Plots titles. Version 3.6.4

Hi Robyn team,
Regarding the plots titles, in order to make easy the stakeholders interpretation...

Geometric Adstock: Isn't it proper something like "Fixed prolonged effect rate over time" instead of "Fixed decay rate over the time"?

Initial vs Optimized mean response: Could you correct the labels? they are the same of "Initial vs Optimised budget allocation"

Thanks for the great job !

Hi thanks for the feedback - I think plot 1 the title we currently have is accurate since we are looking at decay rates specifically with adstock, but can you elaborate on the issue with title 2? Is there a bug going on with the labels? Thank you!

Thanks for your feedback @josetigo I've implemented both changes: removed "decay" from the title to have "Geometric Adstock: Fixed Rate Over Time", and fixed the labels to "Initial Mean Response" and "Optimised Mean Response".


Discover package's submodules

When using ::: mypackage, we should automatically discover and build the documentation data for every submodule of mypackage.
Something like
import pkgutil

# this is the package we are inspecting -- for example 'email' from stdlib
import email

package = email
for importer, modname, ispkg in pkgutil.iter_modules(package.__path__):
    print "Found submodule %s (is a package: %s)" % (modname, ispkg)

from https://stackoverflow.com/questions/1707709/list-all-the-modules-that-are-part-of-a-python-package

Done in 231062a


[feature request] block screenshots

Hy there, and thanks a lot for your app and your enthusiasm.
My first test on trying your great app was, what happens if I view a password and simply leave the app.
The result was, that I could read my password over the screenshot (inside the recent apps screen), even after a device reboot.
So I though it would be great, if you could add the same functionality as the open source SMS/MMS app Silence already has: the menu-point is called "screen security" and blocks screenshots in the recents list and inside the app itself.
Thanks in advance for any response and have a great time!

Oh, didn't notice that . I'll fix that soon. Thanks for your advise!


Added solution for zig zag conversion

Pull Request Template
What have you Changed(must)
Added ClimbingStairs.java file with solutions. Also added zigzag.java but it can be deleted if it is assigned to someone else
Issue no.(must) - #1175
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@Aarav545 please go through the Contribution Guidelines

@Aarav545 Please make the necessary changes within a week else this PR will be closed.


Bug in README.md under Queue Security

In README.md, under Queue Security, the validate rule under rules/queue/tasks contains
newData.hasChildren(['_state', '_state_changed', '_progress'])

Notice the presence of the _progress property. However, in queue_worker.js, in _resetTask, we have:
if (task._state === self.inProgressState) {
  task._state = self.startState;
  task._state_changed = Firebase.ServerValue.TIMESTAMP;
  task._owner = null;
  task._progress = null;
  task._error_details = null;
  return task;
}

So, if the validate rule does contain the _progress property, the transaction in queue_worker will keep failing because _progress is set to null. We should either update the doc or update queue worker.

Is think that this issue is still applicable:
https://github.com/firebase/firebase-queue/blob/master/src/lib/queue_worker.js#L126


test: update yarn packages, prettier

Resolves #516
This PR pass conventional commits, and upgrades & locks requested yarn packages on the issue, this PR does not override prettier config for any of other files than initially was sice it's not clarified which other files we want to prettify, request changes accordingly

Not sure if "latest" is valid and you didn't finish implementing the spec. Lastly, unfortunately the rules are that we allow for 7 days of inactivity before allowing another bounty hunter to submit their solution. As we have more developers in the DevPool we may consider shortening this time period.

Not sure if "latest" is valid and you didn't finish implementing the spec.

I update to latest as the originally posted on the issue, you may request the changes
 "devDependencies": {
    "cspell": "latest",
    "prettier": "latest",
    "prettier-plugin-solidity": "latest"
  }

https://github.com/ubiquity/ubiquity-dollar/issues/516
Yes, I agree, it's just looked like the issue was not moving forward, just let me know!

Lastly, unfortunately the rules are that we allow for 7 days of inactivity before allowing another bounty hunter to submit their solution. As we have more developers in the DevPool we may consider shortening this time period.

Yes, I agree, it's just looked like the issue was not moving forward, just let me know!

I appreciate your eagerness to close issues out. Tag me in a couple of days and I can credit you if the active assignee neglects to finish.

Not sure if "latest" is valid and you didn't finish implementing the spec. Lastly, unfortunately the rules are that we allow for 7 days of inactivity before allowing another bounty hunter to submit their solution. As we have more developers in the DevPool we may consider shortening this time period.

yes "latest" it's a valid for npm/yarn packages

You need to rename the "prettier" script to "format" in the package.json and address the bit about .prettierrc comment in the specification. @rndquu do you think you can make sure the spec is followed with @molecula451's implementation

provide more context for the .prettierrc file @rndquu input would be great

provide more context for the .prettierrc file @rndquu input would be great

2 - update the overrides section of the .prettierrc in the root folder

.prettierrc may handle different styles according to a taste, can you guys agree on one? so I shall commit it

We use prettier as a solidity linter. Lint-staged also calls prettier on commit.
The original issue describes a moment in time when we used forge fmt as a linter but we moved to prettier so this issue should have been closed as "not planned".
Regarding this PR: we shouldn't use latest because it introduces dependencies updated with major versions which may contain breaking changes.
So this PR basically brings us:

script rename prettier:all => format:all
some foundry config optimisations

We can either close this issue as "not planned" either merge this PR (but remove the latest versions in package.json).

Why not planned? you guys agree on a prettier style and we'll do it

Why not planned? you guys agree on a prettier style and we'll do it

I don't see any issues with the current prettier setup for solidity

Why not planned? you guys agree on a prettier style and we'll do it

I don't see any issues with the current prettier setup for solidity

same to, @pavlovcik you may leave the current prettier config, you may propose a change later, this PR is done

Why not planned? you guys agree on a prettier style and we'll do it

I don't see any issues with the current prettier setup for solidity

i don't either, @pavlovcik you may leave the current prettier config, you may propose a change later, this PR is done
Yeah, updating the packages as rndquu request

pls also update this one

bumped https://github.com/ubiquity/ubiquity-dollar/pull/685/commits/2aff1a80da17624f87e8325cc9b5b9ce2c560678

let's finish this one

i don't either, @pavlovcik you may leave the current prettier config, you may propose a change later, this PR is done

We use prettier for everything, including for the UI code under packages/dapp/ and other codebases. Prettier can also format the Solidity code so the Forge formatter is not necessary.


Copy StringSubstitute from UICore/apache-2.0

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


Only require 1 elasticsearch master node on the dev vm

In 1cfda1339ca819fdeb80f8c027173e565c6e38dd we changed the default for
minimum_master_nodes to be 2.  This makes sense everywhere except on the
dev vm where we still want it to be 1.  If it remains as 2 then the dev
vm cluster is unhealthy and reports:
{"error":"MasterNotDiscoveredException[waited for [30s]]","status":503}
for curl -XGET 'localhost:9200/_cluster/health?pretty'.  This in turn
means we cannot run replication scripts.  Setting it to 1 makes things
work again.

👍 great, thanks for fixing this.

👍  nice catch. By extension this fix is necessary to run Rummager tests, use the unified_search API in dev - anything relying on ES basically.


WIP: Try revert "Update CI to CUDA 11.7.1 (#16123)"

This reverts commit 3326e65bb2f7de2040bdf300dca062c627cdddb1 (#16123)
WIP: For debugging, we attempt to revert these changes to determine whether they were the cause of the Horovod failures we see on master.

The horovod failures show even though we are using the previous pytorchlightning/pytorch_lightning:base-cuda-py3.9-torch1.13-cuda11.6.1 here. So we can rule this out as the source for the issue. Closing.
cc @carmocca @akihironitta @Borda


preserve original code and refactor

getGuildMessage is annotated Nullable

Seems like I did not get a notification for this and hence kinda missed it until now... Either way, changes like this should probably be sent to the main JDA repository instead of this fork. For the change itself though, it is true that getGuildMessageChannel is nullable. However the channel will never be null here as we first check that the channel is actually from a guild, which guarantees that the GuildMessageChannel will be non-null.

I have seen that, it still bothers me though to this day that it theoretically is nullable. I'm more on the compilers side here.

Also, i opened a pull request here because at the time where this was relevant, your pull request was not accepted to the main repo.

Sure, but if a change like that were to be made it should probably be done everywhere instead of just in one place. Refactoring by checking against explicit channel types is also not really ideal though as future channel types without permission containers would make the original bug resurface (forum channels would most likely do this).


Change rewards earned to Accured rewards

@Zeugh-eth can you further describe this task once again, for me, please?


Update example in README.md

I think it is safe to assume that visitors to the IronPython GitHub project page know what Python is. So the example to show how to print "Hello World!" in plain Python is kind of pointless. Also, they probably know C# and .NET, so the only reason for comparing IronPython code to C# would be to give an idea for people wondering "well, I know how to do .NET stuff in C#, how would that work in IronPython?"
I've come up here with two simple but-not-totally-trival examples: how to access .NET from IronPython and how to access IronPython form C#. Comments welcome.

I think these are good examples, definitely an improvement over the print example which doesn't really highlight why IronPython is interesting.


Python-based Music Handler

After too many frustrations over the Music module which is currently in pure JavaScript, it is evident we had to use another language for Music. Python was foresaw as it had more better performance at music-related requests.
TODO

[ ] a purely python music handler.
[ ] An Abstraction or "Bridge" for the handler to interact with.

rip my efforts
On 2 Aug 2017 01:52, "Capuccino"<EMAIL_ADDRESS>wrote:

After too many frustrations over the Music module which is currently in
pure JavaScript, it is evident we had to use another language for Music.
Python was foresaw as it had more better performance at music-related
requests.
TODO

An Abstraction or "Bridge" for the handler to interact with.
a purely python music handler.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ClaraIO/Clara/issues/88, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARyjNboOUuju5vQ80m1EUP75Hn3_Dm02ks5sT0nWgaJpZM4Op7u9
.

THose who volunteered during dicussion are added to assignees (except for ovy)

What clear advantages does this have over a native music system?

probably more optimized than our current one @Ovyerus

I don't really think this is a good idea. Node has been shown to be faster than Python. Also I'm not sure how'd it be handled inb4 IPC or RPC, and it would just be easier to optimise our current implementation.

Node is faster, but generally requires better hardware to work properly, which is why I'm assuming we'll want to use Python.

Yeah, but we're already using Node to begin with.

Reason behind this is the handler we currently have is very unoptimized and buggy so its better we use a host lang like Py and provide a IPC bridge instead

Fixing and optimisation exists /shrug

@Ovyerus it's stil lunoptimized :v

Time™

Closing due to negotations


mstest /testsettings runsettings config file format

Moving issue from https://github.com/MicrosoftDocs/visualstudio-docs.de-de/issues/89 as it's feedback for English content.
Hi
you defined the Format for the runsettings file in here:
https://docs.microsoft.com/en-us/visualstudio/test/configure-unit-tests-by-using-a-dot-runsettings-file?view=vs-2019
This leads to (Just all structures made empty):
The file '...\mstest\default.runsettings' has unknown format and cannot be converted to the current version.
There is no information about "Local.runsettings". I just would like to find a simple empty Format or Format description for that content of that file. The error message is totally useless for that purpose.
Why do you not provide a download for an easy Local.runsettings and Maybe a complex one. (I just want to add one single Parameter into the TestContext.Properties to configure a set of unit Tests. And this Parameter shall be defined in the Local.runsettings. Where else?)
Best regards
fi

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

ID: e27af73d-196c-4653-aa07-acf7e8b425e3
Version Independent ID: b8957d80-ffa0-1c42-d3f0-6e4d00bf7a1e
Content: VSTest.Console.exe command-line options - Visual Studio
Content Source: docs/test/vstest-console-options.md
Product: visual-studio-windows
Technology: vs-ide-test
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Mikejo5000 commented on Jul 17 •
@fifromrs Local.Runsettings is just a filename for a .runsettings file, normally placed in the project root. I can clarify this in the doc. I tested TestContext.Properties using a basic .runsettings file like the following, which seems to work fine when I test it.

.\TestResults

and this code:
private string docsUrl;
public TestContext TestContext { get; set; }
[TestMethod]
public void TestMethod1()
{
docsUrl = TestContext.Properties["docsUrl"].ToString();
}
@Mikejo5000 Mikejo5000 mentioned this issue on Jul 17
https://github.com/MicrosoftDocs/visualstudio-docs-pr/pull/6874
Merged
Mikejo5000 commented on Jul 17
Updating doc

@Mikejo5000 I moved the issue from https://github.com/MicrosoftDocs/visualstudio-docs.de-de/issues/89, but it looks like you may have already addressed this issue. Please go ahead to close if already resolved.

Hello Mr. Jones,
I just add a sample of the results, I got from you, to this mail. And really, mstest /testsettings does not work with it. If you configure it in Visual Studio with Visual Studio Unit Testing, the RunSettings file works. But the test is started using mstest within a  Jenkins job…
With that command line:
mstest /testcontainer:My.dll /test:MyClass.MyFunction /testsettings:"c:\temp\TestRunSettings.runsettings"
I get the following error:
Loading c:\temp\TestRunSettings.runsettings...
c:\temp\TestRunSettings.runsettings
The file ' c:\temp\TestRunSettings.runsettings ' has unknown format and cannot be converted to the current version.
If the path is invalid, the system returns, the path for the settings file is invalid. But here, the format is wrong. But I used the format, as you have shown in your examples, attached to this mail.
And sadly, there is no better hint. How to get a correctly formatted file for mstest /testsettings command line argument??? I would like to get the data as described in our mail communication within this chat…
Best regards,
Dirk Fischer
From: Mike Jones<EMAIL_ADDRESS>Sent: Friday, October 16, 2020 11:20 PM
To: MicrosoftDocs/visualstudio-docs<EMAIL_ADDRESS>Cc: Fischer Dirk GF-ES3<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: EXT [Newsletter] Re: [MicrosoftDocs/visualstudio-docs] mstest /testsettings runsettings config file format (#5885)
Yep, thanks. Fixed this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/visualstudio-docs/issues/5885#issuecomment-710649071, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALXJQLBG3536M7B6AKY5WNLSLC2ILANCNFSM4RRIKZBA.
Content provided within this e-mail including any attachments, is for the use of the intended recipients and may contain Rohde & Schwarz company restricted information. Any unauthorized use, disclosure, or distribution of this communication in whole or in part is strictly prohibited. If you are not the intended recipient, please notify the sender by reply email or by telephone and delete the communication in its entirety.

@pavelhorak @AbhitejJohn Are there any odd issues with using .runsettings along with Jenkins? I have not been able to repro this issue, but I wasn't using Jenkins.

ah, mstest.exe is a legacy runner which does not understand runsettings. You'd want to use vstest.console.exe instead from a VS install directory (\Common7\IDE\Extensions\TestPlatform\vstest.console.exe) or this package.
/cc: @nohwnd, @pavelhorak

Thanks! That makes sense; in my own tests, I was using vstest.console.exe, per the instructions in the docs. Closing this for now, unless another issue comes up.

Hi Mr. John, Mr. Jones,
Thanks for your responses!
I would like to use vstest.console.exe, but there is no Jenkins plugin, which can analyse and interpret vstest.console.exe results output. It is available, even if not very well done, for mstest.
Thererfore, what is the format of the /testsettings file. Is there any (easy) example?
Best regards
Dirk Fischer
From: Abhitej John<EMAIL_ADDRESS>Sent: Friday, October 30, 2020 7:18 PM
To: MicrosoftDocs/visualstudio-docs<EMAIL_ADDRESS>Cc: Fischer Dirk GF-ES3<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: EXT [Newsletter] Re: [MicrosoftDocs/visualstudio-docs] mstest /testsettings runsettings config file format (#5885)
ah, mstest.exe is a legacy runner which does not understand runsettings. You'd want to use vstest.console.exe instead from a VS install directory (\Common7\IDE\Extensions\TestPlatform\vstest.console.exe) or thishttps://www.nuget.org/packages/Microsoft.TestPlatform package.
/cc: @nohwndhttps://github.com/nohwnd, @pavelhorakhttps://github.com/pavelhorak
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/visualstudio-docs/issues/5885#issuecomment-719718626, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALXJQLAQ5AHJVOF4FJXFC7TSNL7ONANCNFSM4RRIKZBA.
Content provided within this e-mail including any attachments, is for the use of the intended recipients and may contain Rohde & Schwarz company restricted information. Any unauthorized use, disclosure, or distribution of this communication in whole or in part is strictly prohibited. If you are not the intended recipient, please notify the sender by reply email or by telephone and delete the communication in its entirety.

@fifromrs  can you use https://plugins.jenkins.io/vstestrunner/ Jenkins plugin with VSTest.Console.exe.

no need

Hi Mr. Horak,
sadly, vstest.console.exe is not as stable as expected…. and as mstest.exe is…
<EMAIL_ADDRESS>I have a deadlock when using vstest.console.exe with the scenario above.
Therefore, I would really like to know, … what is the format of mstest’s file “mysettings.testsettings” for parameter “/testsettings:mysettings.testsettings”?
Sorry for my seemingly never ending requests…
Best regards,
Dirk Fischer
From: Pavel Horak<EMAIL_ADDRESS>Sent: Monday, November 2, 2020 11:10 AM
To: MicrosoftDocs/visualstudio-docs<EMAIL_ADDRESS>Cc: Fischer Dirk GF-ES3<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: EXT [Newsletter] Re: [MicrosoftDocs/visualstudio-docs] mstest /testsettings runsettings config file format (#5885)
@fifromrshttps://github.com/fifromrs can you use https://plugins.jenkins.io/vstestrunner/ Jenkins plugin with VSTest.Console.exe.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/visualstudio-docs/issues/5885#issuecomment-720373302, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALXJQLF6VQDU53KRU2GPCD3SN2AODANCNFSM4RRIKZBA.


Multiple generic_utils modules were used when loading model, which caused problems like "Unknown optimizer"

To reproduce:

Keras==2.4.3
tensorflow==2.5.0

import keras

class RenamedOpt(keras.optimizers.Adam):
    pass

inputs = keras.Input(shape=(3,))
outputs = keras.layers.Dense(5)(inputs)
model = keras.models.Model(inputs, outputs)
model.compile(optimizer=RenamedOpt(), loss='mse')
keras.models.load_model('temp.h5', custom_objects={'RenamedOpt': RenamedOpt})

It will raise:
ValueError: Unknown optimizer: RenamedOpt.
When loading the optimizer, the custom objects were passed to the global dict in keras.utils.generic_utils:
https://github.com/keras-team/keras/blob/95c2966d9dd8b5ba67c9473bb1099a407f3b9bd7/keras/saving/saving_utils.py#L204-L206
But then it used the deserialize method in tensorflow.python.keras, which tried to find the global custom objects from tensorflow.python.keras.utils.generic_utils.
One possible quick fix is to explicitly pass the custom_objects argument to deserialize in line 206.
For keras users, one workaround is to explicitly invoke the function in tensorflow.python.keras:
from tensorflow.python.keras.utils.generic_utils import CustomObjectScope
    with CustomObjectScope({'RenamedOpt': RenamedOpt}):
        keras.models.load_model('temp.h5')

@CyberZHG Sorry for the late response! Could you please let us know if it is still an issue in TF latest version 2.7.0 ? Please try to use Use tf.keras.optimizers.Adam(learning_rate) instead of keras.optimizers.Adam(learning_rate) and refer to the link .Please let us know if it helps? Thanks!


Adding solutions of quiz and different difficulty level questions

After answering the questions of the quiz, solution should be given.

3 difficulty level of questions should be present - Easy, medium and hard.

Each difficulty level should have different time limit.

@rohitalal see what exactly I was thinking is, after the quiz then the solution should be present. and as u said different questions will have different time.

can I work on this issue


macOS : Error #2123: Security sandbox violation

Hello,
I get security error when calling capture() on macOS?
SecurityError: Error #2123: Security sandbox violation: BitmapData.draw: app:/com.rainbowcreatures.FWEncoderANE cannot access null. No policy files granted access.
at flash.display::BitmapData/draw()
at com.rainbowcreatures::FWVideoEncoder/capture()[/Users/vmmac/Documents/FlashyWrappers/AS3/com/rainbowcreatures/FWVideoEncoder.as:2199]

This security error occur when capturing StageVideo.

This has nothing to do with FW, its policy issue, trying to capture a Bitmap from a different domain probably (another remote possibility might be trying to use the Flash SWC where you should use the ANE, those don't mix well).
Anyway I believe we discussed exactly this issue over e-mail already, so I'll close this.

Closing


PBS Executor

Description
Add an executor plugin which interfaces with the OpenPBS job scheduler used in HPC environments.  This executor should be similar in format and style to the Slurm executor.  For details on how to translate Slurm commands to PBS commands, see resources here and here.  For a more complete view of documentation around PBS, see Altair's PBS User Guide.
Acceptance Criteria

[ ] Executor plugin lives in its own repository
[ ] PBSExecutor is a subclass of Covalent's AsyncBaseExecutor and all calls use async def
[ ] Users are able to submit, query, and cancel jobs on a remote system managed by PBS
[ ] When both covalent and the PBS plugin are installed, users can run from covalent.executor import PBSExecutor or something similar without error.
[ ] Unit test coverage of at least 60%.

Has this executor been developed?
Is there a timeline for its release?

Has this executor been developed? Is there a timeline for its release?

@Vespertili0 we currently have an unreleased version of this plugin.  If you are interested in testing and providing feedback please let me know.

@wjcunningham7 I am interested in using covalent in workflows similar to your advanced tutorial on quantum chemistry (https://covalent.readthedocs.io/en/stable/tutorials/3_QuantumChemistry/nitrogen_copper_interaction/source.html).
The HPC cluster I have access to runs with PBS, so, I am happy to test the plugin.


refactor: Leverage Context's Client to make requests

Original request: https://github.com/posit-dev/posit-sdk-py/pull/352#pullrequestreview-2502579569
I like it. So I updated as many as I could where I didn't have to guess at the intent. Sessions is still used by other classes that use .params.
Given all Paginator instances could use context, I switched those as well.

Given all Paginator instances could use context, I switched those as well.

Thank you!


yazi: icons and desktop entry

Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[X] 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

[X] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

Hi, thanks for contributing! Just wanted to note there is an existing PR open for this :) #301060

Ah, beat me to it :)


mountinfo: parse empty strings in source

The source of mount in /proc/self/mountinfo can be an empty string. Many mountinfo parsers (even in util-linux or runc, but also in Cilium) are failing when the source is empty. Example from util-linux:
$ sudo mount -t tmpfs "" /tmp/bb
$ mount
mount: /proc/self/mountinfo: parse error at line 64 -- ignored
$ mountpoint /tmp/bb
/tmp/bb is not a mountpoint

Patch to util-linux: https://www.spinics.net/lists/linux-fsdevel/msg128896.html

Nevermind, Cilium is resistant for that issue. I forgot that doing Split(" ") in Golang doesn't trim the emty strings aroused by spaces. It's all good, Cilium works perfectly with mounts with empty sources.


Is the error reported as shown in the figure a configuration problem?

There may be many reasons why active nodes cannot be found based on the error message for the Query node. The information in the figure is not enough to tell. You can look at the relevant logs or try to reinstall the TARS framework.

根据报错信息为query节点无法找到active的节点，可能原因很多，图中的信息不足以判断，可以看看相关日志，或者尝试重新安装TARS框架。


Intellisense not working after update to vscode 1.35

Issue Type: Bug

open vscode with dotnet core sln
get warning notification that some projects couldn't load
see intellisense not able to provide methods, classes, etc. It can only give me suggestions of things I've already typed.

I tried to uninstall and reinstall the extension. I had previously installed a beta version in order to fix an issue the extension had with the latest version of the linux kernel or something like that.
Here's some logging:
Starting OmniSharp server at 6/7/2019, 10:57:37 AM
    Target: /home/seanmccay/repos/tamus.livestock/Livestock.sln

OmniSharp server started.
    Path: /home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/run
    PID: 3231

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on ubuntu 19.4 (x64)
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 1 MSBuild instance(s)
            1: StandAlone 15.0 - "/home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        MSBUILD_EXE_PATH environment variable set to '/home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin/MSBuild.dll'
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: StandAlone 15.0 - "/home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin"
            MSBuildExtensionsPath = /home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild
            BypassFrameworkInstallChecks = true
            CscToolPath = /home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin/Roslyn
            CscToolExe = csc.exe
            MSBuildToolsPath = /home/seanmccay/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in '/home/seanmccay/repos/tamus.livestock'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.WorkspaceInitializer
        Project system 'OmniSharp.DotNet.DotNetProjectSystem' is disabled in the configuration.
[info]: OmniSharp.MSBuild.ProjectSystem
        Detecting projects in '/home/seanmccay/repos/tamus.livestock/Livestock.sln'.
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/home/seanmccay/repos/tamus.livestock/Livestock.Core/Livestock.Core.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/home/seanmccay/repos/tamus.livestock/Livestock.Infrastructure/Livestock.Infrastructure.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/home/seanmccay/repos/tamus.livestock/Livestock.UnitTests/Livestock.UnitTests.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/home/seanmccay/repos/tamus.livestock/Livestock.Web.API/Livestock.Web.API.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/home/seanmccay/repos/tamus.livestock/Livestock.Web.UI/Livestock.Web.UI.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/home/seanmccay/repos/tamus.livestock/Livestock.Infrastructure.Database/Livestock.Infrastructure.Database.csproj'
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in '/home/seanmccay/repos/tamus.livestock'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: /home/seanmccay/repos/tamus.livestock/Livestock.Core/Livestock.Core.csproj
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider
[info]: OmniSharp.WorkspaceInitializer
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location '/home/seanmccay/repos/tamus.livestock' on host 2695.

[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file '/home/seanmccay/repos/tamus.livestock/Livestock.Core/Livestock.Core.csproj'.
/home/seanmccay/repos/tamus.livestock/Livestock.Core/Livestock.Core.csproj(1,1)
Microsoft.Build.Exceptions.InvalidProjectFileException: The SDK 'Microsoft.NET.Sdk' specified could not be found.  /home/seanmccay/repos/tamus.livestock/Livestock.Core/Livestock.Core.csproj

that occurs for each project and then:
[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: /home/seanmccay/repos/tamus.livestock/Livestock.Core/Livestock.Core.csproj
[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: /home/seanmccay/repos/tamus.livestock/Livestock.Infrastructure/Livestock.Infrastructure.csproj
[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: /home/seanmccay/repos/tamus.livestock/Livestock.UnitTests/Livestock.UnitTests.csproj
[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: /home/seanmccay/repos/tamus.livestock/Livestock.Web.API/Livestock.Web.API.csproj
[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: /home/seanmccay/repos/tamus.livestock/Livestock.Web.UI/Livestock.Web.UI.csproj
[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: /home/seanmccay/repos/tamus.livestock/Livestock.Infrastructure.Database/Livestock.Infrastructure.Database.csproj

Extension version: 1.19.1
VS Code version: Code 1.35.0 (553cfb2c2205db5f15f3ee8395bbd5cf066d357d, 2019-06-04T01:13:26.790Z)
OS version: Linux x64 5.0.0-15-generic

System Info

Item
Value

CPUs
AMD Ryzen 7 1700 Eight-Core Processor (16 x 1501)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwaresurface_synchronization: enabled_onvideo_decode: unavailable_offwebgl: enabledwebgl2: enabled

Load (avg)
0, 0, 0

Memory (System)
31.40GB (26.97GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

@seanmccay Could you try setting "omnisharp.useGlobalMono": never and then restarting vscode

We have often seen similar issues when mono is upgraded on linux/mac systems and the new version doesnt have certain files that are needed by omnisharp. When you set the above option, you instruct omnisharp to use the mono version shipped with itself which has all the necessary files.

Ok, updated my user settings and restarted vscode + omnisharp and still get the same result. Is there another step I should take to make sure the setting is taking effect?

@akshita31 I just rebooted my machine as well. I'm not sure if there might be something cached somewhere... Still pretty new to Linux. I still have the same results though.
It looks like I get Intellisense for stuff outside of my own code like in Microsoft/System namespaces at least but omnisharp still can't load my projects.

@akshita31 Tried out extension version 1.20.0-beta1 and no changes. So I'm guessing Mono isn't the issue? Unless the useGlobal setting isn't taking effect... Not sure how to tell.

I reinstalled dotnet core following the instructions on the .net core site and somehow went from version 2.2.3 to 2.2.204, but my intellisense is operational again :man_shrugging:


A few questions

Hi there
Can you please help me get clarity on.
Q1. Assuming your collection is defined from an array of objects, how do I get an index of a complete object?
There are some examples that are not clear for example wrap and unwrap? Can you make more clarifying examples that will perhaps show differences between what was originally passed and the output.

I'm not completely sure what you are asking.
You can get an item by index with the get method.
With collect.js you never (at least rarely) look up something by index, that's kind of the point of the library, to give the developer a better experience iteracting with arrays and objects in JavaScript.
I've put together a few examples for you here 👇
https://runkit.com/embed/5k2djkphs7aw
Wrap will turn an array into a collection. If you pass a collection you will get the collection returned.
Documentation: https://collect.js.org/api/wrap.html
Source: https://github.com/ecrmnn/collect.js/blob/master/src/methods/wrap.js
Unwrap will return all items by calling all under the hood if the passed value is a collection.
If it's not a collection the passed value will simply be returned.
Documentation: https://collect.js.org/api/unwrap.html
Source: https://github.com/ecrmnn/collect.js/blob/master/src/methods/unwrap.js
To be fair I've never seen the need for these methods either. I've never used them, but they're implemented because collect.js should adhere to the API of Laravel Collections. The same methods should be available.


Importing config file still not working as of 24.08.30

Describe the bug
Still getting "Invoke-WPFPresets" error when trying to import a config file
To Reproduce
Steps to reproduce the behavior:

Go to import freshly exported config file.

Screenshots
Screenshot 2024-09-04 133734
Screenshot 2024-09-04 133838
Screenshot 2024-09-04 134049
Is working in 24.08.16
Additional context
I use the .exe version of the program but I tested it with the powershell script as well. Same results.

Hello @FinalDemise and thanks for reporting this issue
This was reported in #2657 issue and fixed in #2658 PR, the fix will be available in the next release of WinUtil.
as there's no further discussion to be made on this topic, I'll be closing this issue...
/close


feat(v2): siteMetadata

🚀 Feature
Currently, as far as I know, a site can't know which docusaurus version it is currently using
It would be useful to be able to get some metadata about the framework.
Algolia/DocSearch devs have expressed the need to be able to track which version of Docusaurus uses which version of DocSearch.
I think this does not belong to the site config: it is a new docusaurus context item that we should create.
const {siteMetadata} = useDocusaurusContext();

For now we could keep it simple and just add the version, as it's been asked by Algolia.
{
  docusaurusVersion: "2.0.0-alpha.58"
}

This could be enhanced later, and include all plugins versions, the site version, and other useful things that we'd find useful.
We could also add these data to docusaurus-plugin-debug , @SamChou19815 do you think of additional data that could be useful to display?

We could also add these data to docusaurus-plugin-debug , @SamChou19815 do you think of additional data that could be useful to display?

I think it can be useful. The version information can also be included in all official plugins, and the debug plugin can help detect out-of-sync versions (e.g. core at alpha-58, theme-classic at alpha-56)

Yes that's what I think too

The version information can also be included in all official plugins

For official plugins or not, I think we could do this in a generic way, to add plugin versions to this siteMetadata file:

if plugin is in node modules, read the version from its package.json
if plugin is a local js file, maybe fallback to the package.json version of the site (or null if no version specified)

The file could look like
{
  docusaurusVersion: "2.0.0-alpha.58"
  siteVersion: "1.0.0",
  pluginVersions: {
    "docusaurus-plugin-pwa": "2.0.0-alpha.58",
    "custom-site-plugin": "1.0.0"
  }
}

I think we should use plugin name like docusaurus-plugin-pwa, and not plugin package name @docusaurus/plugin-pwa because local plugins don't necessarily have a package name, yet we could enforce them to have a name attribute.
Does someone wants to work on this?

if plugin is in node modules, read the version from its package.json

Detecting existence of node_modules in plugin's resolved path might not be a good idea, since it might give surprising result for monorepo setup. For example, this is the resolved path for all plugins in docusaurus repo:
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-plugin-content-docs/lib/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-plugin-content-blog/lib/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-plugin-content-pages/lib/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-plugin-debug/lib/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-theme-classic/src/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-theme-search-algolia/src/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-plugin-client-redirects/lib/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-plugin-ideal-image/lib/index.js
/Users/sam/Desktop/workplace/docusaurus/packages/docusaurus-theme-live-codeblock/src/index.js

There are no node_modules in the path, but they clearly belong to some package.
Therefore, I think all we can reliably do is to recursively walk up along the plugin resolved path until we hit a directory that contains package.json, and try to resolve version info from there.
Then there comes to the question of local plugins. I think it will almost always be the case that they share the same package.json with the root project. If we detect this is the case, we probably should just say it's local without giving a confusing version.
I will probably try to work on it tomorrow.


Link profiles to indexers

Type of change

[x] Refactor
[x] New feature
[ ] Bug fix
[x] Optimization
[x] Documentation Update

Description
Multiple changes with the goal to adopt the updated version the metricsEndpoints API in the configuration file and link metrics, alerts (supporting multiple files now) and measurements to indexers:
With this change the configuration file now looks like:
metricsEndpoints:
- prometheusURL: http://localhost:9090
  token: blablabla
  indexer:
    type: opensearch
    esServers: ["https://es-instance.com:9200"]
    defaultIndex: kube-burner
  metrics: [metrics-profile.yaml]
  alerts: [alert-profile.yaml]
- prometheusURL: http://localhost:9090
  token: blablabla
  indexer:
    type: local
  metrics: [metrics-profile.yaml]
  alerts: [alert-profile.yaml]

Another important feature is that now It's possible to "split" the measurement metrics and send them to different indexers thanks to the new fields timeseriesIndexer and quantilesIndexer. i.e:
metricsEndpoints:
- indexer:
    type: local
- indexer:
    type: opensearch
    defaultIndex: kube-burner
    esServers: ["https://opensearch.domain:9200"]
global:
  measurements:
  - name: podLatency
    timeseriesIndexer: 1
    quantilesIndexer: 2

With the snippet above, the measurement podLatency would use the local indexer for timeseries metrics and opensearch for the quantile metrics. cc: @afcollins
Related Tickets & Documents

Related Issue #
Closes #

So timeseriesIndexer: 1 is the index ordinal of the indexers defined under endpoints starting with 1 (not 0) ?
I like being able to specify different metrics profiles for each indexer, so that then we can push report mode to elastic and capture the timeseries locally, for metrics profile metrics.

So timeseriesIndexer: 1 is the index ordinal of the indexers defined under endpoints starting with 1 (not 0) ?
I like being able to specify different metrics profiles for each indexer, so that then we can push report mode to elastic and capture the timeseries locally, for metrics profile metrics.

Yeah, that number points to the indexer to be used. The reason not to use 0 was that 0 is the "null" value of an integer variable, so in case of not specifying timeseriesIndexer 0 would be assumed. Open to other ideas though; Im not super-fan of this either.

Except for the build and test failures, overall idea looks good to me.

So timeseriesIndexer: 1 is the index ordinal of the indexers defined under endpoints starting with 1 (not 0) ?
I like being able to specify different metrics profiles for each indexer, so that then we can push report mode to elastic and capture the timeseries locally, for metrics profile metrics.

Yeah, that number points to the indexer to be used. The reason not to use 0 was that 0 is the "null" value of an integer variable, so in case of not specifying timeseriesIndexer 0 would be assumed. Open to other ideas though; Im not super-fan of this either.

Instead of numbering here, can we have a name filed in each metricsEndpoint that can be used to reference as an alias for indexer? that way I feel it will be more intuitive for an end user.

So timeseriesIndexer: 1 is the index ordinal of the indexers defined under endpoints starting with 1 (not 0) ?
I like being able to specify different metrics profiles for each indexer, so that then we can push report mode to elastic and capture the timeseries locally, for metrics profile metrics.

Yeah, that number points to the indexer to be used. The reason not to use 0 was that 0 is the "null" value of an integer variable, so in case of not specifying timeseriesIndexer 0 would be assumed. Open to other ideas though; Im not super-fan of this either.

Instead of numbering here, can we have a name filed in each metricsEndpoint that can be used to reference as an alias for indexer? that way I feel it will be more intuitive for an end user.

Few more thoughts to add on top of it.

Currently Prometheus metrics are being indexed to all the indexers. Can we have an option to restrict it to one specific indexer?
As an extension of above thought I think it is good to have 3 fields for indexing, timeSeriesIndexer (for prom metrics), latenciesIndexer (for pod latencies etc) and quantilesIndexer (for pod latency quantiles) at global (i.e outside measurements config) and at each job level. Meaning indexers at job level will override the ones at global level if a user wants to.
Default values of global indexers should be all until unless user explicitly specifies index aliases.

So timeseriesIndexer: 1 is the index ordinal of the indexers defined under endpoints starting with 1 (not 0) ?
I like being able to specify different metrics profiles for each indexer, so that then we can push report mode to elastic and capture the timeseries locally, for metrics profile metrics.

Yeah, that number points to the indexer to be used. The reason not to use 0 was that 0 is the "null" value of an integer variable, so in case of not specifying timeseriesIndexer 0 would be assumed. Open to other ideas though; Im not super-fan of this either.

Instead of numbering here, can we have a name filed in each metricsEndpoint that can be used to reference as an alias for indexer? that way I feel it will be more intuitive for an end user.

Few more thoughts to add on top of it.
* Currently Prometheus metrics are being indexed to all the indexers. Can we have an option to restrict it to one specific indexer?

They are not, the idea of this PR is to link metricsProfiles to indexers so we can restrict what we send to each indexer, if not working like this there must be a bug

So timeseriesIndexer: 1 is the index ordinal of the indexers defined under endpoints starting with 1 (not 0) ?
I like being able to specify different metrics profiles for each indexer, so that then we can push report mode to elastic and capture the timeseries locally, for metrics profile metrics.

Yeah, that number points to the indexer to be used. The reason not to use 0 was that 0 is the "null" value of an integer variable, so in case of not specifying timeseriesIndexer 0 would be assumed. Open to other ideas though; Im not super-fan of this either.

Update, I've updated the code to use indexer aliases rather than integers pointing to an indexer


Method threw 'java.lang.NullPointerException' exception. Cannot evaluate okhttp3.HttpUrl$Builder.toString()

Method threw 'java.lang.NullPointerException' exception. Cannot evaluate okhttp3.HttpUrl$Builder.toString()
why？  I used in Retorfit

Include the entire stacktrace or a failing test please.

Retrofit ret = new Retrofit.Builder()                 .baseUrl("http://api.igootoo.com/")                 .addConverterFactory(GsonConverterFactory.create())                 .build();
it's just wrong in baseUrl
@JakeWharton

That is not enough information to reproduce or identify the problem.

I get this error where I used baseUrl(...) @JakeWharton
can i talk to you in some chat tools

No. Provide the full stacktrace or a sample which reproduces the problem.
On Fri, Oct 6, 2017 at 2:09 PM juntity<EMAIL_ADDRESS>wrote:

I get this error where I used baseUrl(...) @JakeWharton
https://github.com/jakewharton
can i talk to you in some chat tools
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/3639#issuecomment-334830023, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEETDEB1mbWxj6AGvOEpj02gbKiIulks5spmzPgaJpZM4Pw074
.


how to process mutli-channels signal

Hello! We have 256 channels now. But we want to process them together. How can we do it?

I'm sorry, we (the maintainers) have extremely limited bandwidth to work on this package, and can't answer questions like these.


Defined new revisions for golang dep: github.com/bazelbuild/bazel-gazelle

Ran component through the scanners since I noticed it had not been harvested yet. PR matches curation site. Closing.


This Windows version (6.3.9600) does not support the required bluetooth API.

🐛 Bug
Running napari from windows server 2012R2, I have got the following error:

'This Windows version (6.3.9600) does not support the required bluetooth API.'
Looking on others projects, it looks to be a dependency (PyQT) issue (see similar error in https://github.com/GIS4WRF/gis4wrf/issues/103 ).
It looks like it could be avoid if
from PyQt5 import Qt
is not done (import Qt imports all the modules including bluetooth which has issues on Win8.1 + PyQt5 >= 5.10)
(see https://github.com/spyder-ide/qtpy/issues/166 )
To Reproduce
Steps to reproduce the behavior:

Install Napari
run it

Expected behavior
Napari should start
Environment
Windows server 2012R2
Python 3.8.10
Additional context
Renaming the file D:\bin\napari_conda_env\Lib\site-packages\PyQt5\Qt5\bin\Qt5Bluetooth.dll to _Qt5Bluetooth.dll temporary fixed the issue.

We don't import from PyQt directly, instead use qtpy which appears to have resolved this in https://github.com/spyder-ide/qtpy/pull/143
Closing -- if this is still occurring, please open a new issue and provide napari --info output.


[#4751] Update library - ehcache-core 2.6.9 -> 2.6.11

#4751
ehcache-core 2.6.9 -> 2.6.11

Codecov Report

Merging #4776 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4776      +/-   ##
==========================================
- Coverage   40.45%   40.45%   -0.01%
==========================================
  Files        2570     2570
  Lines       78297    78297
  Branches    10643    10643
==========================================
- Hits        31678    31677       -1
  Misses      43734    43734
- Partials     2885     2886       +1

Impacted Files
Coverage Δ

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
52% <0%> (-16%)
:arrow_down:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
84.93% <0%> (-4.8%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75% <0%> (-3.13%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
64.62% <0%> (-0.69%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.51% <0%> (+0.69%)
:arrow_up:

...point/rpc/client/DefaultPinpointClientHandler.java
72.29% <0%> (+1.27%)
:arrow_up:

...pinpoint/profiler/instrument/AccessorAnalyzer.java
65.71% <0%> (+8.57%)
:arrow_up:

...p/pinpoint/rpc/client/WriteFailFutureListener.java
75% <0%> (+18.75%)
:arrow_up:

Continue to review full report at Codecov.

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


Update sql doc with interactive session defaults

Fixes #2397.
Related code changes in
https://github.com/cockroachdb/cockroach/pull/20805.
Summary of changes:

Add a new section, 'Interactive Sessions', to the 'Use the built-in
SQL client' page.  In this section we describe the changed defaults at
a high level, with links to the flags for details. (This required
adding anchors to a number of the specific flag descriptions.)

Update descriptions of the affected flags' defaults.

Where possible, made references to flags/options into links in a few
places (now that said flags/options have anchors).

Unrelated edit: Made it so that all flag descriptions use the same
"Default: foo" syntax for easier scanning by readers.

Unrelated edit: Made a list of subhead links in the 'Flags' section
since the previous version was missing the 'Client Connection'
subsection.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d760aac5874e480c4b9832b36f31b8accef58042/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c30ec7821a4e8bf2ed71340dfff9d47d9ae74d67/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/63c8de6ef29e9853053023d1fe425d3310e41eec/


Update PULL_REQUEST_TEMPLATE.md

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
There is no use in requesting people to check certain things before making a pr if you do not define what the result of the check should be.

These are obvious points and don't make any sense to be mentioned.


Comparison of target path and configuration directory on WIN32

In file collections/collections.py the target directory and conf directory are compared to evaluate if the link is created in the right directory. This fails on windows due to comparing configured python paths with os.path.realpath ones.
Assume you must also compare with the os.path.realpath of the target in Line 94:
        if not target.startswith(os.path.realpath(app.confdir)):             raise CollectionsException(                 'Target path is not part of documentation folder\n'
        if not os.path.realpath(target).startswith(os.path.realpath(app.confdir)):             raise CollectionsException(                 'Target path is not part of documentation folder\n'

Thanks for the bug report and the detailed analysis :+1:
A fix will need some time, as I'm currently quite busy.
If you like and as you already proposed a solution, you can create a PR and add yourself to the AUTHORS file (ohh this is not there yet, but we should create it :) )
And if there are any questions about this, I'm will be happy to answer them.

I will provide a fix cause the suggested solution has issues with clean/incremental build cause "realpath" of the target behaves different if symbolic link exists or not.
To provide a pull request: How do i run the tests in the package? I am not so familiar with python packages....


Update Llama3 training test case with Trainium on SMHP

This PR updates the test case to make it compatible with Neuron SDK 2.19
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Review approved. Only comment is why are 16 nodes required, is it possible with 4?

Thanks @nghtm! It requires at least 8 instances I believe, 16 instances are prepared just to showcase 3D parallelism, with data parallel degree of 2.


reverted change of submodule accidentially committed by me within #9485

reverted change of submodule accidentially committed by me within #9485
Orignal change:
diff --git a/homeassistant/components/frontend/www_static/home-assistant-polymer b/homeassistant/components/frontend/www_static/home-assistant-polymer
index 73289bca6..07d5d6e8a 160000
--- a/homeassistant/components/frontend/www_static/home-assistant-polymer
+++ b/homeassistant/components/frontend/www_static/home-assistant-polymer
@@ -1 +1 @@
-Subproject commit 73289bca6d4e326de4484e991019e10f69a351ed
+Subproject commit 07d5d6e8a9205f77f83f68615695bcbc73cf83e3

The workflow is revert the wrong PR and merge the fixed one. Please open a fixed version of the origin PR


change features.* -> devcontainer-features.*

context: https://github.com/microsoft/vscode-remote-release/issues/5980

@joshspicer Found a few more: https://github.com/microsoft/vscode-dev-containers/commit/ac43b05bece5ef685929d102cf074f75da907d4f (v0.215.0 failed, but I just saw the release, so I won't rerun it now.)


[bug] Uncaught TypeError: Cannot assign to read only property 'chosen' of object '#' since upgrade to v6.1.0

Describe the bug
After upgrate the react-sortablejs version from 6.0.3 to 6.1.0, I got this error everytime I click on my sortable list:
Uncaught TypeError: Cannot assign to read only property 'chosen' of object '#<Object>'

To Reproduce
Steps to reproduce the behavior:

Go to my repo: https://github.com/Saul-Mirone/vite-plugin-book
Upgrade the react-sortablejs version to 6.1.0 in packages/vite-plugin-book/package.json
Run pnpm install, pnpm dev and pnpm doc in root
Click on the sidebar and see error.

Expected behavior
Just work normally as v6.0.3
Information
Uncaught TypeError: Cannot assign to read only property 'chosen' of object '#'
Versions - Look in your package.json for this information:
react-sortable = ^6.0.3
react = ^x.x.x
Additional context
I use nested sortable list.

Update:
I found it broken since v6.0.4 and I believe it's caused by this PR:
https://github.com/SortableJS/react-sortablejs/pull/226

It's the only feature added in 6.1

Still don't know why but things work fine after I made this change:
https://github.com/Saul-Mirone/vite-plugin-book/commit/b949839983e2f8530c104a223d839b6c4d7f461e#diff-811aee71034d5086fa8e1e23d182c2f20de005829cb071a0c1a0cf2cf4476a02R58

Still don't know why but things work fine after I made this change: Saul-Mirone/vite-plugin-book@b949839#diff-811aee71034d5086fa8e1e23d182c2f20de005829cb071a0c1a0cf2cf4476a02R58

I think it is because if you destructure the object, even if it was frozen by immer or something like that, will send a copy that can be modified. From what I saw the object is being mutated here:
// make all state false because we can't change sortable unless a mouse gesture is made.
const newList = [...props.list].map(item => Object.assign(item, {
  chosen: false,
  selected: false
}));

If I for example would make a shallow copy of item like this it wouldn't modify the original frozen object:
const newList = [...props.list].map(item => Object.assign({...item}, {
  chosen: false,
  selected: false
}));

What do you think about updating this method here?
I would be happy to submit a pull request.

Any fix for this yet?


Is there a way to have multiple concurrent and independent progress bars?

Description
I have 2 concurrent processes (threads really) performing operations that are independent.
I would like to have 2 progress bars (as 2 lines) to show the progress of each. The first one is bound (number of tasks known), the second one is not (number of tasks unknown)
Is that possible?
I didn't see anything obvious in the examples

There's a simpler version available called the multi-bar: https://github.com/WoLpH/python-progressbar/pull/208
But the full feature is still a work in progress: https://github.com/WoLpH/python-progressbar/issues/189

Yes, I'm aware of the multi-bar (I already use it for something else), but here the 2 processes are (almost) completely independent and of different nature, so the multi-bar wouldn't work.
Any idea when you think the full feature might become available?

Well... I've been thinking about the feature for a few years now with little time to work on it. But I've probably been thinking of a much too difficult solution for the issue :)
I've got a workaround that isn't full support, but I think it works rather well :)
import sys
import time

import progressbar

class LineOffsetStreamWrapper:
    UP = '\033[F'
    DOWN = '\033[B'

    def __init__(self, lines=0, stream=sys.stderr):
        self.stream = stream
        self.lines = lines

    def write(self, data):
        self.stream.write(self.UP * self.lines)
        self.stream.write(data)
        self.stream.write(self.DOWN * self.lines)
        self.stream.flush()

    def __getattr__(self, name):
        return getattr(self.stream, name)

bars = []
for i in range(5):
    bars.append(progressbar.ProgressBar(
        fd=LineOffsetStreamWrapper(i),
        max_value=1000,
    ))

    if i:
        print('Reserve a line for the progressbar')

for i in range(100):
    for j, bar in enumerate(bars, 1):
        bar.update(i * j)
    time.sleep(0.1)

Would this work for you? :)

I will test this and get back to you.

I've properly added the feature now. I'm still testing with it but I believe it works quite well. Usage: https://github.com/wolph/python-progressbar#multiple-threaded-progressbars
import random
import threading
import time

import progressbar

BARS = 5
N = 50

def do_something(bar):
    for i in bar(range(N)):
        # Sleep up to 0.1 seconds
        time.sleep(random.random() * 0.1)

        # print messages at random intervals to show how extra output works
        if random.random() > 0.9:
            bar.print('random message for bar', bar, i)

with progressbar.MultiBar() as multibar:
    for i in range(BARS):
        # Get a progressbar
        bar = multibar[f'Thread label here {i}']
        # Create a thread and pass the progressbar
        threading.Thread(target=do_something, args=(bar,)).start()

You can add your custom progressbars to a MultiBar instance like this:
multi['Some label'] = your_progress

Thanks.
I'm afraid I've now changed job, and the project I was planning on using that on is now dead...
Hopefully I get to make use of it in the near future in a new project

No worries :)
If you ever need it in the future you know where to find it ;)


Language detection worker

based on https://github.com/CodinGame/monaco-vscode-api/pull/151

Is this ready for review? And if so, please release a next version!, too! 🙂

Is this ready for review? And if so, please release a next version!, too! slightly_smiling_face

Yes it is! :)
And as I said the currently published version if already based on it (just a few commits behind)

And as I said the currently published version if already based on it (just a few commits behind)

Ah, I see, languageDetectionWorker was already in there. Good, then I can already integrate that into monaco-languageclient.

@kaisalmen will you have the time to look at it? at the end there is some fixes here I forgot about (terminals and audioCue are not working)

Can you please publish a next release with all changes. Thank you.

Can you please publish a next release with all changes. Thank you.

sure, done as<EMAIL_ADDRESS>


Error in production - Fatal Exception: NSInvalidArgumentException

im using  v2.0.1-rc.2
Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x25809b0b __exceptionPreprocess
1  libobjc.A.dylib                0x24fc6dff objc_exception_throw
2  CoreFoundation                 0x2572243b -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]
3  CoreFoundation                 0x257222bf +[NSDictionary dictionaryWithObjects:forKeys:count:]
4  SetbeatNative                  0x389aa7 __46-[RNFSManager downloadFile:resolver:rejecter:]_block_invoke
5  SetbeatNative                  0x38cf17 -[RNFSDownloader URLSession:downloadTask:didFinishDownloadingToURL:]
6  CFNetwork                      0x25df7681 __82-[NSURLSession delegate_downloadTask:didFinishDownloadingToURL:completionHandler:]_block_invoke
7  Foundation                     0x2601e84d __NSBLOCKOPERATION_IS_CALLING_OUT_TO_A_BLOCK__
8  Foundation                     0x25f7f87f -[NSBlockOperation main]
9  Foundation                     0x25f71f8f -[__NSOperationInternal _start:]
10 Foundation                     0x26020b0d __NSOQSchedule_f
11 libdispatch.dylib              0x253a5e7f _dispatch_queue_drain
12 libdispatch.dylib              0x2539ee17 _dispatch_queue_invoke
13 libdispatch.dylib              0x253a720d _dispatch_root_queue_drain
14 libdispatch.dylib              0x253a707b _dispatch_worker_thread3
15 libsystem_pthread.dylib        0x2553ae0d _pthread_wqthread
16 libsystem_pthread.dylib        0x2553a9fc start_wqthread

I have same problem, any updates? jerson, did you resolved it?

A PR for this is welcome - I have no time currently to fix things.

@moby18 im just do a single react native module using TCBlobDownloader
https://gist.github.com/jerson/bea9bf34c2d5ab410d59538f2ce8cf6e

This seems to occur when the response body is empty. Haven't had the time to dive into it deep into. Anyone made a fix? My objective c is a bit rusty

This might be the fix: https://github.com/johanneslumpe/react-native-fs/pull/189


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #122.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #122

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Introduce analyzer and mapping

This patch introduces:

mapping, first one is based on Openstack Neutron, retrieving TenantID
and VNI.
analayzer, which is basically forwarding flow form agent to storage
using mapping to complete flow structure.
config package, used currently in neutron mapping
logging package, which is basically a helper

The neutron mapping is using a local in-memory cache that is updated in
an async way.
This patch fixes the concept of flow computing a unique flow id based on
headers.

LGFM ; Just the flow timestamp granuality need to be changed

Agreed, I'll reduce the precision of the timestamp


dependencies is ignored in bower list --json meta if processed.

This is an inquiry to confirm if this is a bug or in-design behavior.
Project.prototype._restoreNode = function (node, flattened, jsonKey, processed) {
    var deps;
    node.dependencies = node.dependencies || {};
    node.dependants = node.dependants || {};
    processed = processed || {};

    // Only process deps that are not yet processed
    deps = mout.object.filter(node.pkgMeta[jsonKey], function (value, key) {
        return !processed[node.name + ':' + key];
    });

    mout.object.forOwn(deps,...);

If we install jquery-cookie with foundation(foundation has jquery-cookie as dependency), when running bower list --json, in the output(https://gist.github.com/pwang2/79c716ef79d319b31022), jquery-cookie under foundation's dependencies has dependencies: {}; actually it should be
{
    "jquery":{
       "pkgMeta":{},
       "dependencies":{}
    }
}

I think it is kinda reasonable to have a full meta file format with all dependencies. If you agree this is a bug, I could send PR to fix that. I was doing some work with bower list meta information(see https://github.com/pwang2/brower), this will also require me to add a processed cache to get a full self-contain meta structure without ignore.  Just want to make it more reasonable for all cases.
Thanks

@sheerun Can I bother you with this?

I guess it is to prevent duplication or something? I don't think it's by design in this case, it looks wrong. If you could fix it, but also make sure it works with circular dependencies, I think we could merge fix to Bower.

@pwang2 Thank you for reporting it, by the way :)

@sheerun , You are absolutely right.  Circular is the problem here to solve.
The current implementation may be a shortcut to solve the problem. But It's unexpectedly breaks non circular case like what I mentioned above.
What we could improve is to create a circular cache to break the ring ONLY when needed.  If a dependency is identified also on existing dependency path(aka a pkg is the descendant of itself), keep the fact in object representation but break the ring when we need to make a JSON serialization.
I will do some a further review on the user case in current code and plan to make a PR.

@sheerun , could you brief me the de-circular strategy in bower?
Even in the bundler case like webpack, source code of circulated dependencies need to be arranged to certain pattern  to work.  in the <script> tag loading case, it is definitely not working at all.
so my extended question is is it valuable to push this to a package manager like bower here?

I never really went into de-circular strategy in Bower. I'd be glad if someone experimented with it and documented it.


Bump misc dependencies

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
ab289bb1f839ad5c8ff4bec637554dceaa4955cf
link
true
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


Improve Interoperability with AWS Endpoints for Login

Enhance getAWSLogin to get JWT Token, based on Session Info. Get Access Token, and update getDevicesAWS to use Access Token. Related to #1

Very cool. I'll take a look as soon as I can (vacation).