Fix Vector2Angle() not working as expected

This pull request fixes Vector2Angle() not working as expected. (https://github.com/raysan5/raylib/issues/2172, https://github.com/raysan5/raylib/pull/2193)
Here's the animation of a simple example I made with the fixed version of Vector2Angle():

float angle = Vector2Angle((Vector2) { .x = 1 }, Vector2Subtract(GetMousePosition(), SCREEN_CENTER)) * RAD2DEG;

@jdeokkim thank you very much for the review and the visual explanation. 👍🙂


Add workaround for Safari for loop lexical scope bug

Safari raises a syntax error for a let or const declaration in a for loop initalization that shadows a parent function's parameter:
function a(b) {
  for (let b;;);
}

This is valid code, but Safari throws a syntax error. The bug has been reported and since fixed in WebKit, so future versions of Safari will not have this problem.
This modifies the scope tracker's canUseInReferencedScopes method to detect cases where a rename would trigger this bug and use a different name.
Fixes #559
This is my first time dealing with Babel internals, so please let me know if there's a better way to do this!

Thanks for the PR. It looks great. I have added some review inline. I'll try this out tomorrow.

Any eta on this PR going in and being released? Old babili releases have some breaking bugs and this is blocking us from upgrading.

I just encountered this one as well :)


Pre-trained model and the interface

Thank you for your work. I want to use this pre-trained model to get your demo image. How can I make these boxes disappear?

I just want to modify the detector and change the way it marks the image.


Added translation for Marathi Language

Signed-off-by: Tech2IT<EMAIL_ADDRESS>Summary

Closing Issues
Use Fixes #XXXX or Closes #XXXX to autoclose an associated issue.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvements to existing code, such as refactoring or optimizations (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Feel free to reopen if you decide to make the requested changes. But I'm assuming from lack of response and significant incompleteness of the translation at tail end of Hacktoberfest that this is just abandoned. So I'm closing it.


Only destroy an app once. Test for stop on destroy.

A Marionette.Object can call destroy multiple times, but we want it to allow it only once. In some edge cases removeChildApp could be called on a destroyed child app.

Does my test look okay @brentli1?

:+1:


chore: Add GH event on release to send request to website

Description

Sends a GH evennt to the website repo on release so website repo can update any rhum versions

Sweet! Time to do it for all other repos now


Ranged expert and dead eye not functioning with thrown weapons

Hi ! I've come to the discovery that Ranged expert and Dead Eye do not seem to recognize thrown weapons for half of their features.
I'll explain my discovery process :
First up I was levelling an armorer Inventor with the Infiltrator armor type, got the dead eye feat for them and realized it didn't work, after looking around here found that it's a known bug, so I redid the level up and took ranged expert. Now I saw that it didn't work either, I assumed it might have been because it wasn't a "one-handed" weapon per say, so I had my inventor holding a dagger to throw... Didn't work either, only way to get the bonus action to show up is by wielding two daggers but then it's just TWF activating.
In case it was some weird inventor interaction, I went up and made two new character specifically to try it, a lv1 Fighter/lv4 Inventor human (to get the proficiency with hand crossbow) and a lv4 Fighter human, both having dead eye and ranged expert
Through testing I found that :

While wielding a hand crossbow, Ranged expert function perfectly with the artificer (Any first attack, even the lighting launcher, allows for the hand crossbow bonus action) but Dead eye attack debuff and damage bonus only appeared on the hand crossbow shots.
While wielding Thrown weapons, even on the full fighter nothing triggered : Using Two javelin (so TWF wouldn't interfere)  the bonus action never appeared nor did the attack or damage change with dead eye.
I also tested the other features of both feat : Lighting launcher and the thrown weapon ignore the engaged in melee disadvantage, the covers AC bonus and long range disadvantages, as they should.
I also tried swapping load-outs in between action and got the same results.

In conclusion:

Ranged expert bonus action attack doesn't activate while wielding a thrown weapon, despite being in the description
Dead eye Attack debuff and bonus doesn't activate with ranged weapon (this might be on purpose though, as it doesn't specify thrown weapon) but its other features do.

Thank you for your time!

Posting didn't seem to work with the save file included, so i'll try here :
RangedExpertThrownWeapon.zip

Thanks. It’s pretty hard to add thrown functionality to both so will fix ranger mastery description.


Improve the wording of the defintion of a Counter

Link to the wikipedia article on Montonic functions.
Mention that counters can be reset to zero.

:+1:
Missing DCO

Fixed DCO.

Ah sorry - missed that this was merged.


Directory listing failed

Hey mate. First things first, great package!
I've been using fvm for some time now and it's been great. However, I installed v0.0.4 from pub today, and tried to run fvm list. Unfortunately, it ran into some errors.
Here's the output:
▶ fvm list --verbose
[     0.019s] Oops, something went wrong: Directory listing failed
[     0.027s]
Unhandled exception:
type '_StackTrace' is not a subtype of type 'String'
#0      fvmRunner.<anonymous closure> (package:fvm/fvm.dart:39:23)
#1      _RootZone.runBinary (dart:async/zone.dart:1384:54)
#2      _FutureListener.handleError (dart:async/future_impl.dart:151:20)
#3      Future._propagateToListeners.handleError (dart:async/future_impl.dart:690:47)
#4      Future._propagateToListeners (dart:async/future_impl.dart:711:24)
#5      Future._completeError (dart:async/future_impl.dart:530:5)
#6      _AsyncAwaitCompleter.completeError (dart:async-patch/async_patch.dart:36:15)
#7      CommandRunner.runCommand (package:args/command_runner.dart)
<asynchronous suspension>
#8      CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#9      new Future.sync (dart:async/future.dart:224:31)
#10     CommandRunner.run (package:args/command_runner.dart:112:14)
#11     fvmRunner (package:fvm/fvm.dart:33:23)
<asynchronous suspension>
#12     main (file:///home/pitri/.pub-cache/hosted/pub.dartlang.org/fvm-0.0.4/bin/main.dart:4:9)
<asynchronous suspension>
#13     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:303:32)
#14     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

Any ideas on what could be wrong?

Good to hear @FranPitri. This version is under heavy development has I have rewritten from Go to Dart. This issue happens because there are no current flutter sdks installed through fvm. I have updated version 0.0.5. You still get the same error, but I will review that to make sure there are better error messages.
Also I have updated Readme.md to reflect some of the updates


Docker GPU Issues

After setting up Grounded-SAM-2 using the docker container provided and running the demo script:
/home/appuser/Grounded-SAM-2# python grounded_sam2_local_demo.py
I am met with the following warning and error:
UserWarning: Failed to load custom C++ ops. Running on CPU mode Only!
UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at /opt/conda/conda-bld/pytorch_1716905979055/work/aten/src/ATen/native/TensorShape.cpp:3587.)
final text_encoder_type: bert-base-uncased
model.safetensors: 100%|██████████| 440M/440M [00:05<00:00, 56.5MB/s]
UserWarning: The given NumPy array is not writable, and PyTorch does not support non-writable tensors. This means writing to this tensor will result in undefined behavior. You may want to copy the array to protect its data or make it writable before converting it to a tensor. This type of warning will be suppressed for the rest of this program. (Triggered internally at /opt/conda/conda-bld/pytorch_1716905979055/work/torch/csrc/utils/tensor_numpy.cpp:206.)
UserWarning: torch.utils.checkpoint: the use_reentrant parameter should be passed explicitly. In version 2.4 we will raise an exception if use_reentrant is not passed. use_reentrant=False is recommended, but if you need to preserve the current default behavior, you can pass use_reentrant=True. Refer to docs for more details on the differences between the two variants.
UserWarning: None of the inputs have requires_grad=True. Gradients will be None
Traceback (most recent call last):
File "/home/appuser/Grounded-SAM-2/grounded_sam2_local_demo.py", line 58, in
boxes, confidences, labels = predict(
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/util/inference.py", line 68, in predict
outputs = model(image[None], captions=[caption])
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/groundingdino.py", line 327, in forward
hs, reference, hs_enc, ref_enc, init_box_proposal = self.transformer(
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/transformer.py", line 258, in forward
memory, memory_text = self.encoder(
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/transformer.py", line 576, in forward
output = checkpoint.checkpoint(
File "/opt/conda/lib/python3.10/site-packages/torch/_compile.py", line 24, in inner
return torch._dynamo.disable(fn, recursive)(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/_dynamo/eval_frame.py", line 451, in _fn
return fn(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/_dynamo/external_utils.py", line 36, in inner
return fn(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/utils/checkpoint.py", line 487, in checkpoint
return CheckpointFunction.apply(function, preserve, *args)
File "/opt/conda/lib/python3.10/site-packages/torch/autograd/function.py", line 598, in apply
return super().apply(*args, **kwargs)  # type: ignore[misc]
File "/opt/conda/lib/python3.10/site-packages/torch/utils/checkpoint.py", line 262, in forward
outputs = run_function(*args)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/transformer.py", line 785, in forward
src2 = self.self_attn(
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/ms_deform_attn.py", line 338, in forward
output = MultiScaleDeformableAttnFunction.apply(
File "/opt/conda/lib/python3.10/site-packages/torch/autograd/function.py", line 598, in apply
return super().apply(*args, **kwargs)  # type: ignore[misc]
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/ms_deform_attn.py", line 53, in forward
output = _C.ms_deform_attn_forward(
NameError: name '_C' is not defined
The container appears to build with the GPU correctly. Below are the results of test I have ran to find discrepancies:
nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2023 NVIDIA Corporation
Built on Mon_Apr__3_17:16:06_PDT_2023
Cuda compilation tools, release 12.1, V12.1.105
Build cuda_12.1.r12.1/compiler.32688072_0

TORCH_VERSION = ".".join(torch.version.split(".")[:2])
CUDA_VERSION = torch.version.split("+")[-1]
print("torch: ", TORCH_VERSION, "; cuda: ", CUDA_VERSION)
torch:  2.3 ; cuda:  2.3.1

print(torch.cuda.is_available())
True

print(torch.cuda.device_count())
1

print(torch.cuda.current_device())
0

print(torch.cuda.device(0))
<torch.cuda.device object at 0x7a10b3552080>

print(torch.cuda.get_device_name(0))
NVIDIA GeForce RTX 4090

I am running the host machine with Ubuntu 24.04 with CUDA 12.6 if that is of any relevance.
I cannot for the life of me figure out what is causing the issue is or the solution to it. Any support on this issue would be greatly appreciated.

I have tested the dockerfile on another device and recreated the error following the same set up procedure described in the README.
/home/appuser/Grounded-SAM-2# python grounded_sam2_local_demo.py
UserWarning: Flash Attention is disabled as it requires a GPU with Ampere (8.0) CUDA capability.
UserWarning: Failed to load custom C++ ops. Running on CPU mode Only!
UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at /opt/conda/conda-bld/pytorch_1716905979055/work/aten/src/ATen/native/TensorShape.cpp:3587.)
final text_encoder_type: bert-base-uncased
UserWarning: The given NumPy array is not writable, and PyTorch does not support non-writable tensors. This means writing to this tensor will result in undefined behavior. You may want to copy the array to protect its data or make it writable before converting it to a tensor. This type of warning will be suppressed for the rest of this program. (Triggered internally at /opt/conda/conda-bld/pytorch_1716905979055/work/torch/csrc/utils/tensor_numpy.cpp:206.)
UserWarning: torch.utils.checkpoint: the use_reentrant parameter should be passed explicitly. In version 2.4 we will raise an exception if use_reentrant is not passed. use_reentrant=False is recommended, but if you need to preserve the current default behavior, you can pass use_reentrant=True. Refer to docs for more details on the differences between the two variants.
UserWarning: None of the inputs have requires_grad=True. Gradients will be None
Traceback (most recent call last):
File "/home/appuser/Grounded-SAM-2/grounded_sam2_local_demo.py", line 58, in
boxes, confidences, labels = predict(
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/util/inference.py", line 68, in predict
outputs = model(image[None], captions=[caption])
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/groundingdino.py", line 327, in forward
hs, reference, hs_enc, ref_enc, init_box_proposal = self.transformer(
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/transformer.py", line 258, in forward
memory, memory_text = self.encoder(
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/transformer.py", line 576, in forward
output = checkpoint.checkpoint(
File "/opt/conda/lib/python3.10/site-packages/torch/_compile.py", line 24, in inner
return torch._dynamo.disable(fn, recursive)(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/_dynamo/eval_frame.py", line 451, in _fn
return fn(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/_dynamo/external_utils.py", line 36, in inner
return fn(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/utils/checkpoint.py", line 487, in checkpoint
return CheckpointFunction.apply(function, preserve, *args)
File "/opt/conda/lib/python3.10/site-packages/torch/autograd/function.py", line 598, in apply
return super().apply(*args, **kwargs)  # type: ignore[misc]
File "/opt/conda/lib/python3.10/site-packages/torch/utils/checkpoint.py", line 262, in forward
outputs = run_function(*args)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/transformer.py", line 785, in forward
src2 = self.self_attn(
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/ms_deform_attn.py", line 338, in forward
output = MultiScaleDeformableAttnFunction.apply(
File "/opt/conda/lib/python3.10/site-packages/torch/autograd/function.py", line 598, in apply
return super().apply(*args, **kwargs)  # type: ignore[misc]
File "/home/appuser/Grounded-SAM-2/grounding_dino/groundingdino/models/GroundingDINO/ms_deform_attn.py", line 53, in forward
output = _C.ms_deform_attn_forward(
NameError: name '_C' is not defined

TORCH_VERSION = ".".join(torch.version.split(".")[:2])
CUDA_VERSION = torch.version.split("+")[-1]
print("torch: ", TORCH_VERSION, "; cuda: ", CUDA_VERSION)
torch:  2.3 ; cuda:  2.3.1

print(torch.cuda.is_available())
True
print(torch.cuda.device_count())
1
print(torch.cuda.current_device())
0
print(torch.cuda.device(0))
<torch.cuda.device object at 0x735d5cb464d0>
print(torch.cuda.get_device_name(0))
NVIDIA GeForce RTX 2070 SUPER

Again, I am running the host machine with Ubuntu 24.04 with CUDA 12.2 if that is of any relevance.
This issue seems to be known as the Makefile references a similar issue (https://github.com/IDEA-Research/Grounded-Segment-Anything/issues/84) however the references suggests that the problem has been addressed.
Additionally I have tried and tested setting up Grounded-Segment-Anything and receive  the same error, again on both devices. I was initially trying to set up Grounded-Segment-Anything and then tried setting up Grounded-SAM-2 as i thought the issue could have came from the version of cuda being used.

Hi there,
This error typically occurs due to incorrect compilation of custom CUDA/C++ extensions or incompatibilities between PyTorch versions, CUDA, and custom operations. I can confirm that I am using Ubuntu 20.04, and the Docker container works correctly in that environment. If you are running Ubuntu 24.04, I recommend updating the Dockerfile to ensure compatibility with your system.
Best regards!

Hi! I'm using Ubuntu 20.04 with CUDA version 12.1, and I'm facing the same issue. I read this and they suggested setting CUDA_HOME which I also did. I was wondering if there is any updates on this? thanks!

To share some of my observations, I have faced the same error NameError: name '_C' is not defined when running the grounded_sam2_local_demo.py file. However, I can run grounded_sam2_hf_model_demo.py successfully without any errors.
Also the Ubuntu OS version does not seem to matter, I have experimented with both 20.04 and 22.04 and I have the same observations as above.

I also experienced a similar issue while running grounded_sam2_local_demo.py within the docker container.
For whatever reason, when I repeated: /home/appuser/Grounded-SAM-2# python -m pip install --no-build-isolation -e grounding_dino in the command line and within the docker container the issue resolved itself.

I also experienced a similar issue while running grounded_sam2_local_demo.py within the docker container. My digging seemed to confirm the issue was related to the grounding_dino install.
For whatever reason, when I repeated: /home/appuser/Grounded-SAM-2# python -m pip install --no-build-isolation -e grounding_dino in the command line and within the docker container the issue resolved itself.

@amerk12 - Thank you very much for your reply, after testing this on 2 different machines and 3 Operating systems this solution works consistently.
I had previously seen other suggestions to do this and I thought I had testing it to no avail. Clearly I had broken other parts of the system in trying to solve the issue.
@ShuoShenDe - I might be worth adding a remark to the readme adding this as a suggestion to fix this common issue.


Mobile searches stopped working

EDIT:
Well, pretty strange it's working now... I will leave this open in case someone is still experiencing the same issue
Hello, today I started getting this error while running the extension:

Anybody else experiencing the same?

I believe this is caused by a small mistake in utility.js :

            userAgents.mb = ua.chrome.chrome_ios;

should be userAgents.mb = ua.chrome.ios;
EDIT : To be clear, from what I understand this would actually set the mobile useragent to "" which would cause the issue you're getting while thinking the useragent has been updated and shoudl be working.

Thanks you are very correct about the error.

fixed in the latest push, will be published in the next release.


API search in publisher not working correctly for some reserved words

Description:
When the API name is having some internally used words (may be in API definition, etc) the search box will not give the API until we type the exact API name.
Example API names: ResponseCacheAPI, ContextAPI, VersionAPI
Type "Version". If there are many APIs in the publisher, you will not get the "VersionAPI" as a result at all. Ideally it should come as the first entry.
Suggestion to fix: API Name should be prioritized when searching. Results via other attributes should come later.

Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Content search performs a full text search for the API so this can the usual behaviour.


Update refined version to 0.3.7

Current coverage is 80.06%

Merging #258 into master will increase coverage by +0.05% as of 650ce39

Powered by Codecov. Updated on successful CI builds.


Rename error_handler to error_handler_spec to run tests again

Also fix tests that broke after recent migration

Merged with the other PR


Memory leak causes allocation failure for large JSON File import

Reproduction steps (this might vary according to the available RAM):

Open AboutCode Manager
Go to File -> Import JSON File
Load in a ScanCode JSON file ~35 MB on MacOS with 8 GB memory
In the Activity Monitor under the Memory Tab the Electron Process shows memory increasing up to ~1.6 GB and then the app crashes with a Dev Tools error.
DevTools was disconnected from the page. Once the page is reloaded, DevTools will automatically reconnect.

 Also contains the following terminal error
<--- Last few GCs --->

  198214 ms: Mark-sweep 1535.1 (1582.0) -> 1535.1 (1582.0) MB, 701.1 / 0.0 ms [allocation failure] [GC in old space requested].
  198914 ms: Mark-sweep 1535.1 (1582.0) -> 1535.1 (1579.0) MB, 700.5 / 0.5 ms [last resort gc].
  199636 ms: Mark-sweep 1535.1 (1579.0) -> 1535.1 (1579.0) MB, 721.3 / 0.5 ms [last resort gc].

Merged into develop


Rewrite Python 3.11+ linetable calculating code in Rust

Fixes armoha/euddraft#127

Overall additional -5.7% reduced compile time on top of https://github.com/armoha/eudplib/commit/29ed0ef5a50bc78d375b9d09aba27598c83268c9.


Nonlinearity Correction

I could'nt open a second pull request, so here is a patch for the implementation of the nonlinearity correction:
--- ../python-oceanoptics_upstream/oceanoptics/base.py  2015-01-14 10:22:10.573384769 +0100
+++ oceanoptics/base.py 2015-01-16 09:40:37.732968993 +0100
@@ -168,7 +170,7 @@
         only_valid_pixels : bool, optional
             only optical active pixels are returned.
         correct_nonlinearity : bool, optional
-            does nothing yet.
+            corrects for nonlinearity of CCD-Chip
         correct_darkcounts : bool, optional
             does nothing yet.
         correct_saturation : bool, optional
@@ -183,8 +185,14 @@
             data = np.array(self._request_spectrum()[self._valid_pixels], dtype=np.float64)
         else:
             data = np.array(self._request_spectrum(), dtype=np.float64)
+        if correct_nonlinearity:
+            data = data/self._calc_nonlinearity(data)
         return data

+    def _calc_nonlinearity(self, counts):
+        return sum( self._nl_factors[i] * counts**i for i in range(8) )
+
+
     def spectrum(self, raw=False, only_valid_pixels=True,
             correct_nonlinearity=True, correct_darkcounts=True,
             correct_saturation=True):
@@ -197,7 +205,7 @@
         only_valid_pixels : bool, optional
             only optical active pixels are returned.
         correct_nonlinearity : bool, optional
-            does nothing yet.
+            corrects for nonlinearity of CCD-Chip
         correct_darkcounts : bool, optional
             does nothing yet.
         correct_saturation : bool, optional

While your implementation of the nonlinearity correction is correct, you must not apply this correction to a spectrum that was not dark-count-corrected!
So before this can be merged into master, the dark-count-correction needs to be implemented and in the code we need to be sure, that if the non-linearity-correction is enabled that the dark-count-correction is enabled too.

Do you mean that the a dark spectrum has to be substracted before the nonlinearity correction ?
That would mean that for using the nonlinearity one would have to inlcude a framework for saving the spectra into the oceanoptics-module, that could get ugly ...
By the way, I got the formula for the correction from this site: http://www.manualsdir.com/manuals/292553/ocean-optics-ooinlcorrect.html

"Do you mean that the a dark spectrum has to be substracted before the nonlinearity correction ?"
No. The counts of the dark pixels (the ones that are physically blocked on the CCD) need to be subtracted from the spectrum.
This is because otherwise the assumption that doubling the integration time = doubling the intensity which should be doubling the counts is wrong.
I guess in spectra suite if you only enable the non-linearity correction but not the dark count correction, the dark counts are added again, after the nonlinearity correction.

Ah ok I understand, I'll have a look into this next week, should'nt be too hard to implement I guess.

Will be resolved when PR #19 gets merged.


Invalid filtering for multiple token options

Let's consider those scalac options:
  val parallelismOption = new ScalacOption("-Ybackend-parallelism" :: "8" :: Nil)
  val releaseOption = new ScalacOption("-release" :: "8" :: Nil)

when both added to tpolecatScalacOptions ++= Set(parallelismOption, releaseOption) the resulting scalacOptions is invalid:
[info] * -Ybackend-parallelism
[info] * 8
[info] * -release
[info] * -other-option

This is due to the distinct in  https://github.com/typelevel/sbt-tpolecat/blob/05fe604cf12482a66dfbb96f550b6cf58d3cb4ff/src/main/scala/io/github/davidgregory084/TpolecatPlugin.scala#L132
Which is applied on individual ScalacOption token instead of the full option.
The filtering logic in the plugin does not handle well the case of multiple token options.

Yes you are right @RustedBones - the filtering should have been applied on ScalacOption

@RustedBones a fix for this went in on #76. I will try to cut a new release (probably 0.4.0) next week

That was fast :rocket:  Thank you so much !


can i get help?

Please provide more information, thanks.

Hi @customcoops77, I'm closing this issue due to inactivity. If you need further assistance, please open a new issue and provide details of the problem you're encountering. Thanks


Update app tx version when processing action

Description
This is a drive-by fix.
I encountered a bug when using the delta txn version from kafka-delta-ingest. The prior process_action implementation does not update the existing map entry for tx version when re-processing actions. This change resolves that.

good catch 👍 could you also write a simple test?

will do

@houqp - While writing the test, I noticed that process_action was contained within the DeltaTable scope even though it doesn't have a self ref in the sig. Feels more clear to hoist the code into the outer scope to me

I totally agree!


Need unallocated resource view to help customize job resource requirements

What would you like to be added: Unallocated resource view
Why is this needed: Users doesn't know how many resource they could apply. Hence,  users might submit exceeding job, then get failed.
Without this feature, how does the current module work： from YARN page maste_ip:8088/cluster/nodes
Components that may involve changes: rest-server, webportal

does the yarn portal provide such information?

close and track in #2208 .


Default Operation Tags

Server administrators should be able to manage a list of default tags that are copied into new operations. This will help teams reduce tag duplication.
In contrast to custom findings categories, we do not want to offer any default tags within the product.

How would this work? If there are 3 default tags, when they're copied, are they true copies? I.e. in a particular project, if I need to make a modification to one, am I (an operation owner) allowed to do so, or are these only managed by admins? true copies are easier to implement, but you would effectively lose the ability to make customization while gaining the ability to make global changes.

They should be true copies managed by a global admin. Within an operation if a tag needed to be edited for whichever reason we would create a new tag instead.
e.g. Global admin-managed tags copied to new operation:
-Reconnaissance
-Privilege Escalation
-Credentials
1- Hurried user begins typing "creds"
2- typeahead sees Credentials and makes it easier for the user to select with keybindings
3- If hurried user really wants to tag only "creds" they still can within the operation
4 - Hurried user should not be able to rename "Credentials" to "creds" without global admin


Added new resources in learning, challenges and tools section

I added some new resources, keeping "Free for Developers" and "Free Developer Stuff" in the tools section. These two resources are a vast directory of all the free stuff for the developers. I could not find any suitable section other than the tools section. Should I create a new section for these two?
Please review.

Sorry for my silly mistakes. I've fixed broken links and badges


how to load dashboard faster

i added the cache configuration with redis
but the dashboard loading is not evolve a lot
is there any solution for that ?
`CACHE_CONFIG = {
Specify the cache type
'CACHE_TYPE': 'redis',
'CACHE_REDIS_URL': 'redis://localhost:6379/0',
The key prefix for the cache values stored on the server
'CACHE_KEY_PREFIX': 'superset_results'
}`

This cache is a cached between the Dashboard and the datasource(Table in
Superset). When you warm up charts, all data will be provided by redis.
On Tue, Jun 25, 2019 at 5:07 PM stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you
for your contributions. For admin, please label this issue .pinned to
prevent stale bot from closing the issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-superset/issues/7387?email_source=notifications&email_token=AAPMKUQ7PWJGDMZNJQYAXMTP4HN4LA5CNFSM4HITSVYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYPSGKA#issuecomment-505357096,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPMKUTIGYEUSRLEIEPFU7TP4HN4LANCNFSM4HITSVYA
.


updated README.md with releases

Overview

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

We are not compatible with mocha-4 / arabica 10 until we do a new release with #1166

Markdown lint still breaks as we don't have a rc13 release yet


Why are all nodes detecting at the same time?

Hello, I have a question. In orchestrator/raft, why are all nodes detecting at the same time, will they summarize the results of their detection to comprehensively judge MySQL? I think that only the leader can detect it himself.

This is so that if the leader node dies or gets network isolated, the other nodes (and in particular the new leader) will have an independent view of the problem. This is especially oriented towards multi-datacenter setups, where an entire DC can get network isolated. Such scenario will be auto-analyzed by the orchestrator nodes outside the isolated datacenter.
Please consider watching https://www.youtube.com/watch?v=YzhkTu0lSvk

I think there is a possibility of a single node detection error judgment. Assuming that only the orchestrator and the mysql instance network of the local room are jittery, the orchestrator-leader thinks this needs to be failover. It is more reasonable to use the majority to participate in the judgment.

I think there is a possibility of a single node detection error judgment.

Did you happen to read how detection works? https://github.com/github/orchestrator/blob/master/docs/failure-detection.md
There is no "single node detection error judgment" because failovers are not based on single node monitoring.
This logic has very low noise and very high accuracy. I use the works "low" and "high" without providing numbers, I know. Generally speaking I haven't seen neither a false positive nor a false negative in a long time.
Please consider watching https://www.youtube.com/watch?time_continue=11&v=E96ItlGSxN0 or reviewing https://speakerdeck.com/shlominoach/reliable-crash-detection-and-failover-with-orchestrator.
There was also thinking to further add majority analysis, but at this time I don't see that it is needed.
But please do make sure to read the docs first, https://github.com/github/orchestrator/blob/master/docs/failure-detection.md

yes, i kown

The current approach negates single-host errors.

I love orchestrator very much ，very good.

oh, thank you for saying so!

Closed as no further action is needed.


URL not properly formed with diacritics/accents not encoded

Describe the bug
If any of the feed query parameters contains diacritic (accent) characters, they are left as is and not encoded, which will results in some of the clients fail to add the RSS feed with a "URL invalid" error. See: https://stackoverflow.com/questions/33211310/convert-french-accent-to-specific-encoding-in-php
To Reproduce
Steps to reproduce the behavior:

For any chosen bridge which takes a text character parameter, use a string containing diacritic/accent characters (or copy and paste this: ąśćż)
Generate feed URL
Copy that feed to a RSS client of choice (it fails here with TT-RSS at least)
See error

Expected behavior
Diacritics/accents should be properly encoded

i think this is a bug in TT-RSS or your browser. im not sure

i think this is a bug in TT-RSS or your browser. im not sure

Sorry, what bug? Per RFC 3986, section 2.3, the URL should consist of specific character set, which does not contain non-ascii characters, period. Any other characters need to be UTF-8 encoded, per RFC3987.
Meanwhile RSS-Bridge allows those characters to make it to the URL. Sure, modern browsers or some clients will automatically UTF-8 encode such query before they send it outside, but RSS-Bridge should not rely on that and instead generate a feed URL that conforms to the standards.
See also:
https://www.w3schools.com/tags/ref_urlencode.ASP

are you copy pasting url from browser?
are you talking about those urls that are produced inside '` tags?
i was unable to reproduce. using firefox.

Reproducible.
Search and result on reddit with a german umlaut "ä". Similar problem than the accented french characters.

RSS bridge config

Result on dvikans public instance

okay i get it.
it happens when parameters are used in http requests without url encoding them.
in the particular case of RedditBridge a solution is to manually url encode the user input parts.
related: https://github.com/RSS-Bridge/rss-bridge/issues/3091

I like your arguments. Okay let me dwell a bit on  it.

@Bockiii fixed for reddit in https://github.com/RSS-Bridge/rss-bridge/pull/4010

i have discovered that curl will automatically escape the url if needed.
but if curl detects an already escaped url, it will NOT escape.
so this particular error only happens if a url is already partially escaped (as was the case with RedditBridge),

i have discovered that curl will automatically escape the url if needed.
but if curl detects an already escaped url, it will NOT escape.
so this particular error only happens if a url is already partially escaped (as was the case with RedditBridge),

The problem is not with how RSS handles that internally (i.e. the curl lib that it uses), but on the outside, i.e. with the RSS clients that you pass unescaped RSS-Bridge URL to.
In other words, we need to make sure that the URL generated by the RSS Bridge after you click "Generate Feed" on the bridge configuration page needs to be properly formed.

im confused now. can you give an example?

here is a url. its html have urls being properly encoded as you requested. it has always been like this as far as I can tell.
https://rss-bridge.org/bridge01/?action=display&bridge=FilterBridge&url=https%3A%2F%2Florem-rss.herokuapp.com%2Ffeed%3Funit%3Dday&filter=ąśćż&filter_type=permit&target_title=on&length_limit=-1&format=Html
pls give example of a non-encoded url being produced


Dashboard: Add ability to buy more tokens w/ MoonPay

Testing instructions:

Navigate to Wallet dashboard
Click on 'Buy'
Click on 'Buy with MoonPay'

Note: MooPay module only shows up if you are currently in a supported region. VPN works.

@corwinharrell note that it takes 1-2 seconds to check MoonPay support, which is a bit suboptimal. Might want to consider some type of 'Checking MoonPay availability' state for the UI.

@kcole16 @Patrick1904 @jimmy3dita should we add gate.io as well as it's the only way to buy in US right now AFAIK?
We also can probably link to CoinMarketCap page with more exchanges as fallback:
https://coinmarketcap.com/currencies/near-protocol/markets/
However options convenient in some places don't even seem to be listed there, e.g. https://coins.black

@kcole16 @Patrick1904 @jimmy3dita should we add gate.io as well as it's the only way to buy in US right now AFAIK?
We also can probably link to CoinMarketCap page with more exchanges as fallback:
https://coinmarketcap.com/currencies/near-protocol/markets/
However options convenient in some places don't even seem to be listed there, e.g. https://coins.black

This may change based on the selected language too.
I agree that we should add at least gate.io, as it's the only way to buy from the US. Looking into the opportunity to generate a bit of revenues from the referrals

This may change based on the selected language too.

@jimmy3dita Moonpay even giving us geo IP information already, so we can show based on that

@picturepan2 can you please add translations for this when you have a chance.


add The Remote Company to Trickster user list

Signed-off-by: Nikola Milojević<EMAIL_ADDRESS>

 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.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_298b7e0e-0b11-4444-a7c1-be44ce597539/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_298b7e0e-0b11-4444-a7c1-be44ce597539.git

Docs Build status updates of commit 2d751c2:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Refactor and improve st2ctl

Currently, st2ctl is a hard to maintain mess. The problem is that we are keep adding stuff without refactoring it.
We could make the whole script 10 times easier to read and maintain if we just refactored the code a bit and do things:

Use arrays and for loops (get rid of nasty hard-coded ifs)
Reduce 4 levels of if nesting (use return early approach)
Reduce duplicated code

Don't see us getting this done in 0.10. I vote for pushing it out as there is no user impact.

Yeah, that's fine. We can move it to the next release.


Pre publish script, removed lib from repo, dev dependency

Removed lib from repo
Add pre publish script
Moved babel dependency to dev dependecny

Thanks and released as part of v0.9.0. Sorry, my life has taken me away from React Native for the past 2 years…


https://appr.tc  website does not work anymore

You're correct.
As per https://github.com/webrtc/apprtc

appr.tc has been shutdown. Please use the Dockerfile to run your own test/dev instance.


Basic auth merge dev

🤖 Generated by Copilot at ed5dfa4
Summary
🌐🐳🐛

This pull request improves the localization, performance, security, and display of the homarr app. It uses the next-i18next library and the useTranslation hook for translating strings, the dayjs library for date and time manipulation, and the TRPCError class for error handling. It also simplifies the docker image tagging and the GitHub workflow, fixes some bugs and typos, and adds some missing or needed configurations and dependencies. It affects several files, such as .github/workflows/docker_dev.yml, src/components/Dashboard/Modals/EditAppModal/EditAppModal.tsx, src/widgets/calendar/CalendarTile.tsx, and prisma/schema.prisma. It also deletes the unused file src/components/User/Preferences/Language/LanguageSelect.tsx.

We're sailing on the homarr app, aye, aye, aye
We're fixing bugs and adding features, aye, aye, aye
We're using next-i18next and dayjs for localization
And we're heaving on the code with prisma and TRPC

Walkthrough

Remove prefix=test- from docker image tag and analyze job from GitHub workflow (link,link)
Install prisma globally in Dockerfile for database migration and seeding scripts (link)
Add hu language code to next-i18next config file to fix error when switching to Hungarian language (link)
Add linux-musl-arm64-openssl-3.0.x binary target to prisma schema file for prisma client to work on ARM-based devices (link)
Change npx prisma to prisma in run script to avoid unnecessary downloads (link)
Replace hard-coded validation messages with translated strings from next-i18next library in EditAppModal.tsx (link,link,link)
Replace hard-coded labels and messages with translated strings from next-i18next library in AvailableElementsOverview.tsx (link,link,link,link,link)
Add useTranslation hook and replace hard-coded labels and messages with translated strings from next-i18next library in CategoryEditMenu.tsx (link,link,link)
Replace hard-coded labels for keybinds with translated strings from next-i18next library in AboutModal.tsx (link)
Fix typo in display property of image style in 404.tsx (link)
Remove moment library and replace it with dayjs library and its plugins for date and time manipulation in _app.tsx (link,link,link)
Add conditional statement to throw error if edit mode is disabled and change error handling in notebook router (link,link)
Add conditional statement to return false if adapter or createSession method is undefined in constructAuthOptions function (link)
Change Language type definition and values for switching from moment library to dayjs library and removing emoji flags in language.ts (link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link,link)
Find TranscodeSession element from video element elements and change name of currently playing item in plex client (link,link,link)
Replace hard-coded validation messages and labels with translated strings from next-i18next library and remove unused import in BookmarkWidgetTile.tsx (link,link,link,link,link,link,link,link,link)
Remove unused import, add useRouter hook, and replace hard-coded labels with values only in CalendarTile.tsx (link,link,link,link,link,link,link)
Replace moment library with dayjs library and its plugins for date and time manipulation in DateTile.tsx (link,link,link,link,link,link,link,link)
Replace hard-coded message with translated string from next-i18next library in IFrameTile.tsx (link)
Delete LanguageSelect.tsx file (link)

Superseeded by #1227


[GOBBLIN-226] Nested schema support in JsonStringToJsonIntermediateConverter and JsonIntermediateToAvroConverter

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Currently, a nested schema is not supported in source.schema property.
This PR adds support for declaring nested schema which could be used by JsonStringToJsonIntermediateConverter and JsonIntermediateToAvroConverter.
The following changes are made:

JsonStringToJsonIntermediateConverter

Parse JSON input using nested schemas

JsonIntermediateToAvroConverter

Add support for RECORD type.
Add support for NULL type.
Add support for allowing nested members in ARRAY, RECORD and MAP types.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
org.apache.gobblin.converter.avro.JsonIntermediateToAvroConverterTest
org.apache.gobblin.converter.avro.JsonElementConversionFactoryTest
org.apache.gobblin.converter.json.JsonStringToJsonIntermediateConverterTest

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@jack-moseley Can you please help review this?

@jack-moseley suggested changes have been made. Please review

@jack-moseley Please review. Thanks.

@jack-moseley Suggested changes have been made. Please review.

@abti can you merge if it looks okay?

@abti Can you merge this PR? If there is no feedback.


[REQUEST] make things persistent

Is your feature request related to a problem? Please describe.
I cannot explain this without an image:
actually we can change the panel position like that:

and we can make the dropdown menus in a closed state like that:

the problem is that this is not persistent, every time I restart Yana I have to reconfigure those two things again and again
Describe the solution you'd like
I hope you can make this persistent after restarts.
thank you.

Please note that you can remove the sections "Starred items" and "Recent items" from the sidebar altogether by going into the settings, then the tab "Sidebar" and deselecting the options "Show starred items in sidebar" and "Show recently edited items in sidebar", so if you are not using these sections at all, you can just hide them from there.
Persisting the sidebar state (e.g. which folders are open) is actually on my list of todos, so that I'll adress in the future. Thanks for the input.

if you are not using these sections at all, you can just hide them from there.

yes I saw those options, but I find this sections useful. I hope it can persist too.

Persisting the sidebar state (e.g. which folders are open) is actually on my list of todos

that is cool, thank you very much.

Persisting folders and collapse-state of sidebar headers was added.
Fixed in v1.0.6


Unable to read repository  com.palantir.typescript_1.8.0.v20160504-1733.jar Read timed out

Hi There,
When I try to install the Typescript plugin for Eclipse I am getting following error can any body help me to our come this issue.
An error occurred while collecting items to be installed
session context was:(profile=epp.package.jee, phase=org.eclipse.equinox.internal.p2.engine.phases.Collect, operand=, action=).
Unable to read repository at http://eclipse-update.palantir.com/eclipse-typescript/plugins/com.palantir.typescript_1.8.0.v20160504-1733.jar.
Read timed out

Is this still an issue? Are you able to access http://eclipse-update.palantir.com/eclipse-typescript/plugins/com.palantir.typescript_1.8.0.v20160504-1733.jar in your browser?


[BUG] List Column Getting Flattened by KerasSequenceLoader

Describe the bug
Column with list are incorrectly rendered by KerasSequenceLoader. The column values get flattened out and therefore don't match with dimensions of rest of the data.
Steps/Code to reproduce bug
df = pd.DataFrame({'seq':[[0, 0, 0, 1], [0, 0, 1, 2], [1, 2, 3, 4], [0, 1, 5, 6], [9, 2, 3, 4]],
                   'label':[1, 1, 0, 0, 1]})
df.to_parquet('data.parquet', index=False)
train = nvt.Dataset(['./data.parquet'], engine="parquet")
train_ds = KerasSequenceLoader(
    train,
    cat_names=["seq"],
    label_names=["label"],
    batch_size=5,
    shuffle=True,
    buffer_size=0.06,
    parts_per_chunk=1,
)
for batch in train_ds:
    print(batch)

This results in output
({'seq': (<tf.Tensor: shape=(20, 1), dtype=int64, numpy=
array([[0],
       [0],
       [0],
       [1],
       [0],
       [0],
       [1],
       [2],
       [0],
       [1],
       [5],
       [6],
       [9],
       [2],
       [3],
       [4],
       [1],
       [2],
       [3],
       [4]])>, <tf.Tensor: shape=(5, 1), dtype=int32, numpy=
array([[4],
       [4],
       [4],
       [4],
       [4]], dtype=int32)>)}, <tf.Tensor: shape=(5, 1), dtype=int64, numpy=
array([[1],
       [1],
       [0],
       [1],
       [0]])>)

Expected behavior
Expected output is tensor of shape (5, 4) for seq column and (5, 1) for label.
Environment details (please complete the following information):

Environment location: aws ec2 instance
Method of NVTabular install: conda
nvtabular version: 0.9.0

Additional context
Add any other context about the problem here.

I had to add another parameter sparse_names=['seq'] and it worked.


Converting PHP-FastCGI handler from a managed handler to a module mapping

Resolves: https://github.com/PowerShell/xWebAdministration/issues/305

This change is

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@andy1547, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

DSCResources/MSFT_xIisModule/MSFT_xIisModule.psm1, line 161 at r1 (raw file):

                Modules = $ModuleType
                ScriptProcessor = $Path
                ResourceType = 'File'

Why hardcoding this to File?
I suggest leaving it out or adding the parameter $ResourceType.

Comments from Reviewable

I'm about to make bigger changes to IISModule (and other resources), so I have added a comment to Reviewable.
I'd like this issue to be Done before I start.
I can resolve my comment myself too, as the author hasn't been active on this for a few months now.
Is it okay to pass the review then? Asking because that's usually a bad practice and I don't want to just mess up the system.

Just to make sure I'm following you want to basically incorporate this change into your PR? Personally I'm fine with whatever as I don't fully comprehend this change.

t fully compreh
Exactly. The changes the author meant to change was changing "Module" to "Modules" (with an S), which fixes a bug. However, author also added a hardcoded ResourceType, which is kind of dirty and as part of my changes I'd add a Parameter for setting the ResourceType. The question if I could pass the "dirty" change as I'll be soon replacing that anyway.

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

I'm fine with you doing the change in yours and closing this one. Let's give another day or two for @andy1547 to respond before moving forward. If he doesn't respond I'll close this PR.

@Devvox93  I agree that ResourceType should be a string parameter with a ValidationSet of: Unspecified (don't think this is an option in the UI but is in handler documentation?), File, Directory and Either. The IIS UI does not force the user to input this and defaults to File, so I'd suggest using that as the default.
Would also need to add ResourceType to GetScript and validate in TestScript.
@tysonjhayes  I won't be able to do these changes any time soon so by all means supersede them, however for the time being I'd suggest keeping this PR open.

With the UI defaulting to File (I didn't think about that), I suggest this PR to be approved, as it does fix the bug mentioned in issue #305 . The extended functionality of using a Parameter (also in Get and Test) can be added later in my PR.

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

DSCResources/MSFT_xIisModule/MSFT_xIisModule.psm1, line 161 at r1 (raw file):
Previously, Devvox93 wrote…
Why hardcoding this to File?
I suggest leaving it out or adding the parameter $ResourceType.

Default value that UI sets is File, so it's not that ugly. Future change will use a parameter.

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@Devvox93 you solution sounds good. Do you have a PR ready for this change? If that includes this change then I suggest we close this PR. @andy1547 I will set this a abandoned, but if you want to continue the work on this than, then please let me know.

@johlju No PR yet as I've been busy with other work, but in a few days I'll be able to implement as I suggested and create a new PR with a mention of this one.


Info screen

Resolves #26
changes:

Added Hive for local data persistence, introduction to the package available here: Hive on our Wiki
Changed HomePage to StatefulWidget to allow for reading data from Hive
Added AchievementTracker widget to notify of answer and completed trips scores

@darowsluk please take a closer look at the styling of the info screen (fonts, colors etc.) as I was unsure whether what I came up with is consistent with the overall aesthetic 'strategy' for the app.

Could you comment out the firebase code until we need it, since I always forget how to generate the extra file.

I cannot build the code, because hive has dependency, which forces sdk 31:
Warning: The plugin path_provider_android requires Android SDK version 31.
Could you check if it works on your setup?

@knutm92 I cannot build the code, because hive has dependency, which forces sdk 31: Warning: The plugin path_provider_android requires Android SDK version 31. Could you check if it works on your setup?

I checked my setup and I have SDK v 32.1-rc1 installed, this is why I haven't experienced the version error. Are you able to update SDK in your Android Studio?
I have also looked at GetX persistent storage, it is actually a separate package called get_storage. Looking at this introductory article to get_storage the code logic would be similar to what is used in hive (service should be started in main and access is via async functions, so requiring extra code to get values). So I would be more inclined to stick to Hive, as it can offer more once needed. (unless there are any reasons like GetX controller integrations or similar?).
I'm pasting Android Studio packages versions for reference

@darowsluk Thank you for the review, I'm closing this PR ;)


cursor in bad state after executing statement with print statement and really long input

Please first make sure you have looked at:

Documentation: https://github.com/mkleehammer/pyodbc/wiki
Other issues

Environment
To diagnose, we usually need to know the following, including version numbers.  On Windows, be
sure to specify 32-bit Python or 64-bit:

Python: 3.11 64-bit
pyodbc: 4.0.35
OS: Windows 10
DB: SQL Server 2019 (15.0.4123.1)
driver: ODBC Driver 18 for SQL Server (2<IP_ADDRESS>)
SQL.LOG

Issue
The PRINT statement is not getting outputed. (see https://github.com/mkleehammer/pyodbc/wiki/Cursor#messages). It will work if I make input less than 2070 characters.
cnxn = pyodbc.connect(cnxn_str, autocommit=True)
    crsr = cnxn .cursor()
    input = '165,2900,3123,6135,2025,1117,5673,1889,7452,6652,4088,6837,6104,4129,676,6013,5963,4516,2947,1408,255,1859,7205,3747,3553,123,6446,357,1403,1091,2140,3397,3774,5924,3379,3436,6043,1194,5618,1406,1081,4120,2252,6751,3765,826,2645,949,2993,5718,1043,2316,4438,2535,7165,6562,4436,121,7429,4433,2610,663,832,6941,3554,3698,120,6920,3077,5965,2809,527,6566,4548,3929,6724,7228,2098,119,5771,1264,5843,1508,1075,5623,1239,4338,6787,7406,6569,4190,1683,2538,1415,6672,3521,3295,118,6615,1827,6406,7329,6831,7328,3534,849,2002,5746,2742,6549,7439,6892,2082,2042,1667,7446,2192,897,3428,1435,2487,780,117,6418,1858,2016,1193,3566,1237,4525,2085,1068,6369,2049,487,494,4362,846,1646,1588,875,793,6778,6177,1067,5708,712,7414,5973,4074,3832,3571,116,6594,3287,2870,3419,3174,115,4428,4124,1823,2422,3352,540,5863,2228,1714,2143,6195,5977,3894,545,5998,6755,6253,6847,1350,4515,2666,1929,7095,114,7061,3167,6689,6185,5722,7293,6834,6354,4421,6630,113,3565,3551,661,3307,1344,5717,7072,3114,7497,1184,3454,878,468,2188,6085,7071,112,4471,4331,2668,3533,7212,461,6946,7269,3550,6545,1005,3862,716,4205,1341,3976,6074,585,2287,3582,7221,3735,5643,3343,1551,2648,5695,353,3612,1850,1573,7006,1145,6824,6246,6669,703,4081,6070,3350,6067,3135,6184,5940,2927,5857,2581,2390,4295,6157,950,7110,7294,3275,3986,7113,4313,1739,1092,7495,6126,5855,3824,5839,6226,3069,2663,6722,5831,3705,5941,111,3786,1965,6441,6602,856,6140,4158,3610,6242,2170,1887,4429,1108,6404,3327,2251,1141,4100,7344,7342,2858,7343,110,4330,3062,1796,5671,4378,800,2539,6848,109,4150,5850,3090,3362,2385,1623,6927,3763,1263,2334,7167,7203,1487,6025,7411,2102,5644,3730,2722,1778,814,2104,1595,5794,635,4184,5881,3548,3048,6932,1574,6049,2320,3543,1387,5936,7079,108,7408,3497,3959,1389,1262,5574,3098,2341,2165,5680,3485,4430,1730,6329,7451,1757,7196,6179,107,2401,6411,6165,1012,1061,2642,5847,1585,2086,2081,6678,5575,6172,1727,6676,2807,2091,6655,3969,1795,4505,5915,5571,5674,2951,3253,4449,3120,499,105,104,103,4445,102,6805,6467,1177,3480,2670,101,753,475,1111,864,100,3517,5584,5804,1235,5960,6934,2957,1444,3477'
    crsr.execute("PRINT 'HI';SELECT ?", [input])
    print(crsr.messages)

Often it is easiest to describe your issue as "expected behavior" and "observed behavior".

It will work if I make input less than 2070 characters.

The cutoff is actually 2000 characters (4000 bytes), which is where DAE kicks in. So when len(input) > 2000 we get
[]

whereas when len(input) <= 2000 we get
[('[01000] (0)', '[Microsoft][ODBC Driver 17 for SQL Server][SQL Server]HI')]

I ran this again with the SQL Trace and I see the ODBC driver is sending back the PRINT message on line 438 but for some reason, pyodbc is not capturing it.

test_me         3da4-aef8	EXIT  SQLParamData  with return code 1 (SQL_SUCCESS_WITH_INFO)
HSTMT               0x000001ACEBCE1740
PTR *              0x0000001895DEC2F8
DIAG [01000] [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]2023-07-25 19:29:04.1355377 -04:00 (0)

The SQL trace for the below python code is @ SQL.LOG
import logging
import unittest

import pyodbc

log = logging.getLogger(__name__)

class MyTestCase(unittest.TestCase):

    def test_something(self):
        conn_str = "Driver=ODBC Driver 18 for SQL Server;Server=localhost;TrustServerCertificate=yes;Trusted_Connection=yes;APP=tests.utils.TestCase:1022::46840;MARS_Connection=yes;ConnectRetryCount=30"
        conn = pyodbc.connect(conn_str, autocommit=True)
        crsr = conn.cursor()
        input = '165,2900,3123,6135,2025,1117,5673,1889,7452,6652,4088,6837,6104,4129,676,6013,5963,4516,2947,1408,255,1859,7205,3747,3553,123,6446,357,1403,1091,2140,3397,3774,5924,3379,3436,6043,1194,5618,1406,1081,4120,2252,6751,3765,826,2645,949,2993,5718,1043,2316,4438,2535,7165,6562,4436,121,7429,4433,2610,663,832,6941,3554,3698,120,6920,3077,5965,2809,527,6566,4548,3929,6724,7228,2098,119,5771,1264,5843,1508,1075,5623,1239,4338,6787,7406,6569,4190,1683,2538,1415,6672,3521,3295,118,6615,1827,6406,7329,6831,7328,3534,849,2002,5746,2742,6549,7439,6892,2082,2042,1667,7446,2192,897,3428,1435,2487,780,117,6418,1858,2016,1193,3566,1237,4525,2085,1068,6369,2049,487,494,4362,846,1646,1588,875,793,6778,6177,1067,5708,712,7414,5973,4074,3832,3571,116,6594,3287,2870,3419,3174,115,4428,4124,1823,2422,3352,540,5863,2228,1714,2143,6195,5977,3894,545,5998,6755,6253,6847,1350,4515,2666,1929,7095,114,7061,3167,6689,6185,5722,7293,6834,6354,4421,6630,113,3565,3551,661,3307,1344,5717,7072,3114,7497,1184,3454,878,468,2188,6085,7071,112,4471,4331,2668,3533,7212,461,6946,7269,3550,6545,1005,3862,716,4205,1341,3976,6074,585,2287,3582,7221,3735,5643,3343,1551,2648,5695,353,3612,1850,1573,7006,1145,6824,6246,6669,703,4081,6070,3350,6067,3135,6184,5940,2927,5857,2581,2390,4295,6157,950,7110,7294,3275,3986,7113,4313,1739,1092,7495,6126,5855,3824,5839,6226,3069,2663,6722,5831,3705,5941,111,3786,1965,6441,6602,856,6140,4158,3610,6242,2170,1887,4429,1108,6404,3327,2251,1141,4100,7344,7342,2858,7343,110,4330,3062,1796,5671,4378,800,2539,6848,109,4150,5850,3090,3362,2385,1623,6927,3763,1263,2334,7167,7203,1487,6025,7411,2102,5644,3730,2722,1778,814,2104,1595,5794,635,4184,5881,3548,3048,6932,1574,6049,2320,3543,1387,5936,7079,108,7408,3497,3959,1389,1262,5574,3098,2341,2165,5680,3485,4430,1730,6329,7451,1757,7196,6179,107,2401,6411,6165,1012,1061,2642,5847,1585,2086,2081,6678,5575,6172,1727,6676,2807,2091,6655,3969,1795,4505,5915,5571,5674,2951,3253,4449,3120,499,105,104,103,4445,102,6805,6467,1177,3480,2670,101,753,475,1111,864,100,3517,5584,5804,1235,5960,6934,2957,1444,3477'
        crsr.setinputsizes([(pyodbc.SQL_WVARCHAR, 0,)])
        crsr.execute("PRINT SYSDATETIMEOFFSET();SELECT ?", [input])
        print(crsr.messages)
        try:
            crsr.fetchall()
        except Exception:
            pass
        while crsr.nextset():
            print(crsr.messages)
            try:
                crsr.fetchall()
            except Exception:
                pass
            print(crsr.messages)
        print(crsr.messages)

if __name__ == '__main__':
    unittest.main()

Reproduced with pyodbc 5.1.0 and FreeTDS ODBC.
Shorter parameter value:
import pyodbc

cnxn = pyodbc.connect(
    "Driver=FreeTDS;"
    "Server=<IP_ADDRESS>;"
    "Port=1433;"
    "Database=test;"
    "UID=scott;"
    "PWD=tiger^5HHH;"
)
crsr = cnxn.cursor()

num_chars = 1999
crsr.execute("PRINT 'HI!'; SELECT ?", "x" * num_chars)
print(crsr.messages)
# [('[01000] (0)', '[FreeTDS][SQL Server]HI!')]

trace_1999.log
Longer parameter value:
num_chars = 2001
crsr.execute("PRINT 'HI!'; SELECT ?", "x" * num_chars)
print(crsr.messages)
# []

[trace_2001.log](https://github.com/mkleehammer/pyodbc/files/15239007/trace_2001.log)

Compare
		Exit:[SQL_SUCCESS]
			Column Name = []
			Data Type = 0x7fff9f49752a -> -9
			Column Size = 0x7fff9f497530 -> 3998 (64 bits)          <--- nonzero length
			Decimal Digits = 0x7fff9f49752c -> 0
			Nullable = 0x7fff9f49752e -> 1
[ODBC][2763][1715103186.042597][SQLDescribeColW.c][209]

with
		Exit:[SQL_SUCCESS]
			Column Name = []
			Data Type = 0x7ffd4d1e425a -> -9
			Column Size = 0x7ffd4d1e4260 -> 0 (64 bits)             <--- zero length
			Decimal Digits = 0x7ffd4d1e425c -> 0
			Nullable = 0x7ffd4d1e425e -> 1

This indicates a max type, whic it seems pyODBC 5 doesn't understand (compare with the trace posted above on Windows with the msodbcsql driver, where it looks like pyODBC 4 made an attempt to fetch the data.)


Join to a presence channel

Hi, can you help me to Join to a presence channel?
I probe for all ways and always it rejected . I suppose that the request that i receive in laravel dont have the expected format.
I can connect to a public channel without problem.
Can you help me?

please share your error message and code.

I dont recibe an error. I get forgiven.  Thanks

An full example for presence and private channel, with passport and socket.io LaravelApp and NativeScriptApp


Results depend on the ordering of the input fasta

Thanks for the great tool!
https://github.com/phac-nml/mob-suite/pull/159 addresses unstable ordering when sorting contig-link counts.
When assigning contigs to clusters, it feels the priority should be correctness, and then stability. One option is to always sort the fasta before processing begins. While that would provide stability across different-ordered-fastas, it doesn't necessarily provide correctness (what's to say whether ascending or descending sort would yield more correct results). Therefore it seems that for each method of assigning contigs to clusters, the correct ordering might be subtly different. For low linkage contigs, I think it makes sense to prioritize lower link count and use cluster score as a tie-breaker.
In the case of assigning based on high-confidence references, there can also be ties for the reference scores, which can impact cluster assignment. Here are some example reference scores:
             ('JX627737', 103233.0),
             ('NC_019424', 103233.0),
...
             ('JX981514', 52572.0),
             ('NC_022105', 52572.0),
             ('CP011290', 52532.0),
             ('CP011293', 52532.0),

assign_contigs_to_clusters is fairly complex, assigning contigs to clusters based on many different criteria. I'm curious if you've given thought to make assignment correct/stable, regardless of the order of the input fasta. Happy to help here!

Hello,
I am currently working on this issue and contig ordering in the input fasta file has indeed impact on final results such as different number of contigs for the same cluster as the contig assignment to plasmid clusters is iterative and order of contigs matters. The simplest solution is indeed resolve ties by an alphabetical ordering but this rather a simplistic solution
but would not necessarily capitalize on optimal (i.e. "correctness") of the final reconstructed plasmid. Instead we are testing now trying to order input contigs by lengths before contig assignment to clusters. Sorry for delay as I was on vacation and it took some time to figure out the assign_contigs_to_clusters logic and write unit test functions for testing and debugging.


About the inference in ScanNet200

Hi, thanks for your great work. I want to know if I conduct instance segmentation evaluation ont the test split of ScanNet dataset, which includes scene0707_00~scene0806_00, what parameters should I change? I just change the test_mode to test in indoor.yaml, but I got the following error:

Thanks very much for your kind reply.

Hi @KaKa-101
A couple of things to note: It is unfortunately not possible to conduct the instance segmentation evaluation on the test split of the ScanNet dataset using our repository, and by evaluation I mean computing the AP scores. This is because the ground truth instances for the test split of the ScanNet is hidden.
However, it is possible for you to compute the predictions and class assignments on the test scenes. You should use the original indoor.yaml config of our repository - it might be that changing the setting to test_mode in indoor.yaml could cause some errors we are currently not handling in our repository.
One thing to note is that, we are using the eval_on_segments configuration for computing the masks from the Mask3D backbone for the ScanNet validation set. If you do not have access to such segments for the test set (computed from the Segmentator in the original ScanNet repository), you can simply use our alternative script for performing inference on arbitrary scenes. You would need to loop over all scenes from the test set and run this script to export OpenMask3D features and masks. Once you complete this extraction process, you need to run one final script to convert these features to ScanNet class assignments. Our evaluation script does that conversion,  but you would need to comment out the following line: https://github.com/OpenMask3D/openmask3d/blob/3bc3fc52693b25668d0e91d55a2ea714544a4749/openmask3d/evaluation/run_eval_close_vocab_inst_seg.py#L159 as it requires the GT files. Instead, you can simply export the preds dictionary following the ScanNet benchmark submission file structure. Please see this page for details about how to create the submission file, as we do not provide support for that within our codebase at the moment.
Hope this helps!
Best,
Ayca

In order to perform instance segmentation and mask feature computation on the train split of the ScanNet dataset, you can simply change the paths in the ScanNet200 config we have in the repository. Instead of valpaths, you can set the train paths. Everything else can stay the same. However, please note that this will not be a fair evaluation with respect to the mask quality, as the instance segmentation checkpoint we provide was trained on the training set of the ScanNet dataset. However, you can use this evaluation to measure how well the CLIP features from our method are.

Thanks for your reply. And I have changed the path in script to the folder contains train split dataset following the right structure as you have mentioned.
But I got the following printing information, it's still about the val split since there are ~1200 scenes in train split of Scannet dataset. And it seems that the masks I get are also about the val split.

So could you tell me what other configures do I need to change for performming on the train split?

Could you verify whether you have processed all ScanNet scenes including the training scenes (e.g. this step)? Could you please check the label dataset includes also the training scenes, (i.e. SCANNET_LABEL_DB_PATH)?

Yes, I have precessed all  ScanNet scenes and the folder is like this:

And which is the label dataset that I need to check please?

I took a look at the screenshots you sent and you need train_database.yaml to be used instead of validation_database.yaml. Because of the mode parameter in this script, the mask generator uses the validation_database.yaml file. But I wouldn't recommend you to change the mode to train there, because that would result in data augmentations to be applied to the point cloud as if these files are loaded for training. A very hacky way to solve your problem could be to rename the train_database.yaml file temporarily to validation_database.yaml, and validation_database.yaml to validation_database_orig.yaml temporarily while you compute the masks for the training scenes. It's been a while since I worked with this database structure so this is a naive solution I could think of for the time being, but it might not work in the end. Let me know how it goes!

The problem is solved finally and thank you so much! Besides I also want to ask about the model that can be running on an arbitrary scene you provided in this link. How do you get this model and is this model just trainned on Scannet dataset? If so, why can this model be applied to conduct instance segmentation on any scene?
Thanks again.


Formatting diff editor title

Add background color and other formatting to diff editor title to match look of OE. Also changing title for schema compare diff editor to "Compare Details" instead of "Object Definitions"

@kisantia of the two dark theme pics.. which one is after change? this one right?

@udeeshagautam the second one is dark theme and the third one is high contrast black. I updated the comment to say what each of them are.


As an automation-loving SoC professor, I can send auto emails to students with upcoming deadlines

.. so that I can remind them to work on the tasks by the deadline.

Closed as this is an upcoming feature beyond v1.4.


brain-score.org submission (user:386) | (public:False)

Plugin-only changes from brain-score.org submission. Generated via Jenkins job create_github_pr, build 174. Plugins added: models: [eSimCLR_lmda_01_1]

already added


mergeFallback not working for sections

mergeFallback option doesn't seem to work for sections or subkeys.
The subkeys from fallback language are not copied to the current language.

I've prepared a fix for that. Check my fork or wait for it to be merged.


multi: Implement RateSource for exchange rates

https://github.com/crypto-power/cryptopower/assets/57448127/5e4e95b6-00bf-4d35-a47e-a5d6cdf16ae4
Closes #78

Below is the Decred icon to be used for that table, it got changed when i implemented the mobile view for the overview page.
the mobile version has a different icon, while the desktop has a different icon, so let's revert this icon without changing the mobile icon

2023-10-13 16:06:49.807 [INF] DLWL: Loaded 3 wallets
2023-10-13 16:06:49.807 [ERR] CRPW: init assetsManager error: ext.NewCommonRateSource error: New rate source bittrex is not supported

my app crashes when i launch your PR
perhaps that error should just be logged and the app allowed to proceed

Oh, I guess I have to stick to lowercase as we had before now? Unless this change in casing will be an unnecessary pita.

2023-10-13 16:06:49.807 [INF] DLWL: Loaded 3 wallets
2023-10-13 16:06:49.807 [ERR] CRPW: init assetsManager error: ext.NewCommonRateSource error: New rate source bittrex is not supported

my app crashes when i launch your PR
perhaps that error should just be logged and the app allowed to proceed

Oh, I guess I have to stick to lowercase as we had before now? Unless this change in casing will be an unnecessary pita.

what change is casing are you referring to?

what change is casing are you referring to?

Changed the exchange constants value to Title case.

what change is casing are you referring to?

Changed the exchange constants value to Title case.

oh, yeah let's change back to lower case then

Desktop look, pending your feedback on 24H(h) Change vs 1D Change @dreacot.

Desktop look, pending your feedback on 24H(h) Change vs 1D Change @dreacot.

Looks good

https://github.com/crypto-power/cryptopower/pull/177/commits/11af9e481ea5968b2e8ca3757bc11738a8344e56 UI change

also the updated now text keeps updating like every 2 seconds, is that intended?

we have web sockets implemented so all listeners are notified after each ws message. Still, we can improve by notifying after a certain interval(no longer based on when each message is received).

Updated.

also the updated now text keeps updating like every 2 seconds, is that intended?

we have websockets implemented so all listeners are notified after each ws message. Still, we can improve by notifying after a certain interval(no longer based on when each message is received).
EDIT: Increased the notification interval to 5min for ws updates.

Alright sounds, good

fix conficts


Add Elasticsearch_dsl support

Added Elasticsearch Support to FastAPI_Filter
In this pull request, I have introduced Elasticsearch support to the existing FastAPI_Filter library, which already provides search and filtering capabilities using SQLAlchemy and MongoDB. The addition of Elasticsearch support expands the range of options for users seeking efficient search functionality.
Currently, I haven't included tests and documentation for the Elasticsearch implementation. Before proceeding with these additions, I wanted to confirm whether you are interested in officially supporting Elasticsearch as part of the library.
For a practical demonstration of the Elasticsearch support, please refer to the example directory where I have provided sample code. I have also included a requirements.txt file and a docker-compose.yml file  to simplify usage testing.
Please review this initial implementation and let me know if it aligns with your project's goals. If it does, I will proceed with adding comprehensive tests and documentation to ensure a seamless and well-documented contribution.

@arthurio


amplify update auth command should show the existing settings

Is your feature request related to a problem? Please describe.
If I run amplify update auth it provides a complicated series of questions to answer. It’s hard to remember what values were chosen the first time around.
Describe the solution you'd like
It would be really helpful if it showed me the existing value for each one during each step.
Describe alternatives you've considered
Trying to read through my cloudformation files?
Additional context
This came up because I wanted to update my project to include hosted UI info as specified here: https://github.com/aws-amplify/amplify-cli/issues/844#issuecomment-479583969

@mrcoles What questions are you seeing as a part of the amplify update flow? We do retain the values from your previous responses (At least that's the expected behavior), and use it to show the amplify update questions.

@kaustavghosh06 I detailed my experience below.
For "What do you want to do?", I selected, "Walkthrough all the auth configurations" (It seemed like the other approaches might auto-overwrite things I had already chosen).

"Select the authentication/authorization services that you want to use" - no default value

"Allow unauthenticated logins?" - no default value (but I see "allowUnauthenticatedIdentities" in parameters.json)

"Do you want to enable 3rd party authentication providers in your identity pool?" - no default value

"Select the third party identity providers you want to configure for your identity pool" - has previous values as defaults!

"Enter your … ID for your identity pool" - has previous values as defaults

"Multifactor authentication (MFA) user login options" - unclear if this is defaulting to what I chose or not (it matches the first choice)

"Email based user registration/forgot password" - unclear if this is defaulting to what I chose or not (it matches the first choice)

"Please specify an email verification subject" - seems to have default

"Please specify an email verification message" - seems to have default

"Do you want to override the default password policy for this User Pool?" - unclear if I choose "n" if it will keep my previous settings or use the standard defautls?

all password policy overrides are reading from and showing my defaults

"Do you want to specify the user attributes this app can read and write?" - same concern as step 10

all user attribute settings are showing my defaults

"Do you want to use an OAuth flow?" - OK this appears to be the part I actually wanted to edit, this is for the hosted UI

"What domain name prefix you want us to create for you?" - this is tough because I’d really like to just use my existing one that I created manually in the web console, what happens if I specify that here? Also, does the value I enter get modified by multi-env stuff?

"Enter your redirect signin URI" - this is confusing, the web console UI allows multiple URLs here:

Enter your callback URLs below that you will include in your sign in and sign out requests. Each field can contain multiple URLs by entering a comma after each URL.

Also, my URLs don’t end with a trailing slash, but it seems like the console needlessly enforces that URIs must have a trailing slash?

The value must be a valid URI with a trailing forward slash. HTTPS must be used instead of HTTP unless you are using localhost.

At this point, I just gave up and just ctrl+c'ed out of the wizard.

One concept that would work well for me is if I could skip steps and only do steps that mattered to me, because in this case I only wanted to do the "Do you want to use an OAuth flow?" step. In this specific case I got caught up in that step anyways and just bailed. It seems safer to leave my existing working setups than risk messing them up?
Hopefully this info helps!

@mrcoles Thanks a ton for sending over this. This would greatly help us in making the update flow better.

10 on your list is a great catch, I rarely find myself running auth update but it is a pain to make sure the updated password policy matches the existing policy.

any update on this? I still not see user friendly version of existing settings when using command
amplify auth update

Below does not add any value:
% amplify auth update
Please note that certain attributes may not be overwritten if you choose to use defaults settings.
You have configured resources that might depend on this Cognito resource.  Updating this Cognito resource could have unintended side effects.
Using service: Cognito, provided by: awscloudformation
What do you want to do? Walkthrough all the auth configurations
Select the authentication/authorization services that you want to use: I want to learn more.
Amazon Cognito identity pools provide temporary AWS credentials for users who are guests (unauthenticated) and for users who have been authenticated and received a token. An identity pool is a store
of user identity data specific to your account.
If you choose to use the default configuration, this utility will set up both a Userpool and an Identity Pool.
Select the authentication/authorization services that you want to use: (Use arrow keys)
❯ User Sign-Up, Sign-In, connected with AWS IAM controls (Enables per-user Storage features for images or other content, Analytics, and more)
User Sign-Up & Sign-In only (Best used with a cloud API only)
I want to learn more.

any update on this? it's been 3 years... what needs to happen?

@mrcoles write-up is fantastic, covers the pains I'm facing now
I'd like to trigger a lambda function after Cognito Confirmation; this is a simple change but the auth update flow is complicated and doesn't clearly show me what my existing settings are. I also decided to ctrl-c out of the flow because I'm afraid to make breaking changes

@renschler @mrcoles @kaustavghosh06
...I'm sitting with this exact issue. I work on a number of apps using Amplify and can't remember, for the life of me, what the settings were or how they relate to the config files.
I would like to update one setting at a time (Add a new Lambda Trigger) however, I am scared I make breaking changes in the other settings using the config all-or-nothing logic flow this approach uses.
I have resorted to making an individual lambda function and then linking it every time in the console. This is a messy approach and shouldn't be necessary.
This CLI should give a simple "skip this" option

And updates? I am stuck with the exact issue as well, just want to add another Cognito lambda trigger.

Any updates on this? It is a common use case to revise auth settings as the project develops, based on my own experience and the comments above. I actually brick'ed a previous project by pushing a change after running "amplify update auth" (local worked but the remote build started failing after I push'ed), so I'm a bit hesitant to use the command in general. I ended up having to start with a clean install to fix that problem. Thanks for the support.

I have found a temporarily work around for this. All the options chosen is in cli-inputs.json. Once I made the changes on the desired config through amplify auth update, I will try to fill the rest of the field to previous configuration. Then, I will observe changes in cli-inputs.json and only pick changes that is intended before running amplify push
Note: the amplify cli version that we used is 12.0.3. And, I am using this for iOS application.

Any updates ?

Updates?

It is absolutely insane to not be able to choose which part of auth to update and having to go through all options. Very error prone, very time consuming. This should be highly prioritised by the amplify team


Add external links to MARRVEL, DECHIPER, and VarSome

And gnomAD

https://varsome.com/position/hg19/chr19-1223121

Matt - I'd say in order of most important to least, DECIPHER, MARRVEL, VarSome

Matt: I'd say in terms of the order of the links: OMIM, HumanMine, GeneCards, NCBI, DECIPHER, MARRVEL, UniProt, GTex, ProteinAtlas (maybe change the wording here too?)

VarSome:  https://varsome.com/variant/hg19/17-17698535-G>A

Done


Enable FriBiDi Support for RTL Languages

yazi --debug output
Yazi
    Version: 0.3.3 (VERGEN_IDEMPOTENT_OUTPUT 2024-09-23)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.3 (VERGEN_IDEMPOTENT_OUTPUT 2024-09-23)

Emulator
    Emulator.via_env: ("st-256color", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect : Unknown([])

Adapter
    Adapter.matches: X11

Desktop
    XDG_SESSION_TYPE           : None
    WAYLAND_DISPLAY            : None
    DISPLAY                    : Some(":0")
    SWAYSOCK                   : None
    HYPRLAND_INSTANCE_SIGNATURE: None
    WAYFIRE_SOCKET             : None

SSH
    shared.in_ssh_connection: false

WSL
    WSL: false

Variables
    SHELL              : Some("/usr/local/bin/fishlogin")
    EDITOR             : None
    VISUAL             : None
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None

Text Opener
    default: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block  : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : No such file or directory (os error 2)
    ZELLIJ_SESSION_NAME: None
    Zellij version     : No such file or directory (os error 2)

Dependencies
    file             : 5.45
    ueberzugpp       : No such file or directory (os error 2)
    ffmpegthumbnailer: 2.2.2
    magick           : 7.1.1-36
    fzf              : 0.54.3
    fd               : 10.2.0
    rg               : 14.1.0
    chafa            : No such file or directory (os error 2)
    zoxide           : 0.9.4
    7z               : 24.08
    7zz              : 24.08
    jq               : 1.7.1

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/amr/.local/state/yazi" directory.

Please describe the problem you're trying to solve
For simple programs, I can generally pipe their output to fribidi, which does the rendering of Arabic text correctly. However, I can't just do that with a complex program like yazi. There doesn't seem to be a way to extend it with lua to achieve this.
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
Either use FriBiDi headers directly in code, perhaps through c, or a simple pipe to the fribidi executable before rendering text, which is a lot less efficient.
Additional context
No response
Validations

[X] I have searched the existing issues
[X] The latest nightly build of Yazi doesn't already have this feature

I don't know FriBiDi, it would help if you could explain what it is and what Yazi can do here

FriBiDi is GNU's implementation of the standard bidirectional text algorithm. It takes unicode RTL text and renders it properly. If the characters are in the simple latin range, or the slightly wider LTR characters ranger, it does nothing. Hence it's a harmless and useful addition to pipe all text through the algorithm before rendering.
This is important to me because I have Arabic files/directories and Arabic content, so it messes with my directory listings and the content preview.

I think this is a bit beyond the scope of Yazi, as it involves text formatting. Yazi being a TUI app, doesn't handle that directly and relies on the UI library Ratatui.
If you just want to deal with the filename part, you could try rewriting the Entity:highlights() method, where you can access the file name through self._file.name and perform any processing or conversion. Here's a simple example:
function Entity:highlights()
  return ui.Line("You can return anything you want here")
end

However, content preview is still tricky because it's not just a Line like the filename but a Paragraph, which requires the appropriate API from Ratatui. Fortunately, I found an issue discussing this: https://github.com/ratatui/ratatui/issues/756.
Implementing this on the Yazi side would be quite complex; we might need to iterate over each Paragraph and calculate their content size before Ratatui renders them, which would turn the original single render into two renders. Since Yazi doesn't have complete info on each Paragraph, it would need to render them again. So, I'll close this issue as it's beyond Yazi's scope.


Amend OLM catalog IS according to OpenShiftImageRegistryOverrides

What this PR does / why we need it:
Amend the image address used for the imageStreams
used for the OLM catalog source according to the value
of OpenShiftImageRegistryOverrides as genetically used
in the control plane operator in order to handle the disconnected use case.
This will implicitly assume that the images used for
the 4 default OLM catalogs got mirrored to the internal registry
using the original name and tag.
An annotation based escape mechanism is also defined:
the owner of the hostedcluster will be able to inject custom
addresses (only by digest) for the catalog images.
In that case the imageStream mechanism is completely skipped
and the catalog deployments are directly with the values
in the annotations.
It will be completely up to the user to manually refresh
them when the internal mirror will get refreshed.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes https://issues.redhat.com/browse/HOSTEDCP-591
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest

/approve

/test e2e-aws

Updating the title so the Jira ticket is linked and the label is added. This is going to be required soon "DPTP will be adding the requirement of PRs having a jira/valid-reference label for a PR to merge.  Currently PRs will get this label added automatically by referencing a Jira in their title...".
/retitle HOSTEDCP-591: Amend OLM catalog IS according to OpenShiftImageRegistryOverrides


Using Alarm Volume

Before start check this: Volume settings in android
Currently notifee notification using Ring volume
but i want to use Alarm volume is it possible?
note: i don't know its related but i used alarmManager already but no luck.

mmm interesting 🤔 do you have a specific place in the codebase we are setting Ring volume. But, from a brief look, it looks like we could set this on the channel when we create the audioAttributes... and allow this to be customized:
 AudioAttributes audioAttributes =
                  new AudioAttributes.Builder()
                      .setUsage(AudioAttributes.USAGE_NOTIFICATION)
                      .setContentType(AudioAttributes.CONTENT_TYPE_SONIFICATION)
                      .build();

@helenaford i used USAGE_ALARM it's working but ringtone stops immediately when pressing power button of phone.


Fix markdown file formatting, change file name to avoid confusion

Fix bash formatting
Change file name from troubleshooting.md to getting-started-troubleshooting.md

👍

👍


xds: Add xDS delegate feature support to the unified mux

https://github.com/envoyproxy/envoy/pull/22473 and https://github.com/envoyproxy/envoy/pull/22921 added support for the xDS resources delegate in the gRPC Sotw mux. This PR adds support for the xDS resources delegate feature to the unified gRPC Sotw implementation, which is intended to be the default implementation in the future.
Signed-off-by: Ali Beyad<EMAIL_ADDRESS>

/cc @alyssawilk

/retest

@adisuissa friendly ping

@adisuissa any other concerns/feedback on the PR?

/assign @lizan
Alyssa is OOO for the next few days, assigning to Lizan instead

/assign @snowp

@snowp can you do a cross-org /sr maintainer review? @lizan seems to be offline


Adding a built-in function "is_prime"

This function will take a positive integer number as input and returns the output '1' if the number is prime otherwise '0'. ( Issue #13 )

@himanshuvarandani thank you for your contribution!


[sil] Add two helper methods for checking if a block has single successor or predecessor blocks.

Just slicing off some changes from a larger PR.

@swift-ci smoke test

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


hthx example

I am a golang dev. thanks for making this nice and clean golang base.
htmx really looks like an answer to so many gui issues.
would be great if you added a htmx example, so i can see how to join up the front and back.

You're welcome and thank you. I'm glad to hear that you like it.
There are many HTMX examples included. First, please make sure you've read the HTMX section of the README.
The current HTMX examples provided are:

All sidebar navigation links use hx-boost to fetch and replace the body
Click the search bar. A modal will appear which is done with Alpinejs. Type a few characters. hx-get is used to fetch mock search results from the server and render them on the screen.
All forms (contact, login, register, forgot password, etc) either use hx-boost or hx-post.
On the dashboard/homepage, you can page through the mock "posts" using hx-get.
The contact form, when submitted successfully, will fetch markup from the server to replace the form using hx-post.

The HTMX package to handle the request and response headers can be found here. As the README explains, it should be used via the Page
I believe that is it for now. If you have a suggestion for other examples to include or have any questions, please let me know.


Add OAuth2 feature for Config Clients

When OAuth2 token property present, call IDP for Bearer Token.
Add Bearer Token header on config server resource requests.
Replace deprecated Base64Utils

We will also need to add some documentation for this

@ryanjbaxter Can you point me to the repo that contains the Spring Cloud Config that I will need to update?

Docs live here https://github.com/spring-cloud/spring-cloud-config/tree/main/docs/modules/ROOT/pages

Docs live here https://github.com/spring-cloud/spring-cloud-config/tree/main/docs/modules/ROOT/pages

Nervous that I didn't fork correctly. I am in the 4.0.5-SNAPSHOT and I do not see the modules directory under the docs module?
I only see src/main/asciidoc folder?
Also curious how this will go since I see a 4.1.0 release of spring-cloud-starter-config on Dec 6th in Maven Central?

Yes this will actually have to go in a feature release, most likely 4.2.x.  That said its best to use main.

Thanks @ryanjbaxter
Sorry, I am very new to the forking to another repo and merging back.
Can I sync the current fork in my repo or do I need to:

Close this PR
Delete the fork in my repo
Create a new fork into my repo from main which shows 4.1.1-SNAPSHOT.
Create the feature branch off main in my repo.
Move code to new fork.
Then cut new PR back into main on this repo.

Yes I think so

Moving to version 4.1