fix(ar_tag_based_localizer): added small changes

Description
Multiple small changes added.

Changed Qos for the input image topic from reliable to best_effort.
Changed to dynamically scale covariance according to recognition distance.
Changed camera_frame_ to be taken from the received image topic instead of keeping it as a member variable
Added the sample simulation data from AWSIM.

Tests performed
for deault Autoware (i.e. pose_source:=ndt)

[x] logging_simulator (sample rosbag)
[x] driving_log_replayer (sample rosbag)
[x] AWSIM

for ar_tag_based_localizer (pose_source:=artag)

[x] logging_simulator (sample data 1 / Real world data)
[x] logging_simulator (sample data 2 / AWSIM)

Evaluation Result

Method
Mean Error[m]

Odometry Only
2.766

Odometry + AR-Tag
0.627

Odometry + NDT(Default Autoware)
0.053

Effects on system behavior
If AR tag-based self-location estimation is used, the behavior changes slightly. Otherwise, there is no change.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Thank you for creating the PR!  I have tried your new rosbag.
The position is corrected each time an AR marker is detected, so I think it is functioning as intended.
https://github.com/autowarefoundation/autoware.universe/assets/24854875/b9b04a18-f6ba-4001-9f18-582bba6c2384

@KYabuuchi
Thanks for your review!
The upper limit was revised to 13 m. https://github.com/autowarefoundation/autoware.universe/pull/4885/commits/0d83680a41552c868a899b6a3b43a1e936da1fef
With this change, there is now less lateral shift except for the last tag.
In fact, it would be better if the AR tag placement could be improved to observe within 10 m. In AWSIM, the vehicle is a full-size car, so the AR tag is placed about 12~13 m away from the car.
In the use case assumed by AR tag-based self-localization, such as a small car, it would be possible to place it with such constraints.


Embed and play youtube videos

The app should support embedding and playing youtube videos (to start with).
We can leverage the "comments" column in the "songs" db to define the media url in a property format, for eg:
mediaUrl=https://youtu.be/ijmhcA0W_Bg
The app should render this video in addition to showing the lyrics. The video can be displayed at the bottom in Portrait mode.
NOTE: We can adapt a different layout for landscape mode with the video and lyrics displayed side by side with lyrics taking a larger portion.

Below exception got in android monitor when display song content and youtube view in same view
10-16 22:19:37.677 14857-14857/org.worshipsongs W/YouTubeAndroidPlayerAPI: YouTube video playback stopped due to unauthorized overlay on top of player. The YouTubePlayerView is not contained inside its ancestor com.google.android.youtube.player.YouTubePlayerView{27d9b047 V.E..... ......ID 0,0-0,0}. The distances between the ancestor's edges and that of the YouTubePlayerView is: left: 0, top: 0, right: 0, bottom: 0 (these should all be positive).
10-16 22:19:38.036 14857-14857/org.worshipsongs W/YouTubeAndroidPlayerAPI: YouTube video playback stopped due to unauthorized overlay on top of player. The YouTubePlayerView is obscured by com.getbase.floatingactionbutton.FloatingActionButton{1df709e3 VFED..C. ........ 572,986-720,1134 #7f0d00aa app:id/play_song_fab}. The view is inside the YouTubePlayerView, with the distance in px between each edge of the obscuring view and the YouTubePlayerVi

Format to add media url in comments column.
For e.g This is  https://www.youtube.com/watch?v=UFccvtrP1d8 sample youtube url.
Add mediaurl=UFccvtrP1d8 this is sufficient. No need to add full url.

Maddy, adding the full URL is less error prone. Otherwise the contributors may make mistakes if they were to copy just a portion of it.
Moreover, we may support additional media options like Spotify etc in future.


fix: reduce pod nomination event counts

Description
These aren't useful during large scale-ups, so drop them instead
of loading the API server.  The case where these are useful is
if we believe a pod will schedule, but kube-scheduler won't.  In
this case, the pod nominations will be repeated indefinitely and
will be available on the pod.
How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I could be convinced there is a better way to do this.


Install Error

Hi, @gstecher
Thank your tool at first, but I have some problem while installing.
Can you give  me some advice ?  Here is the log:
(python3) [xiucz@mu01 CloneFinderAPI]$ python setup.py install
running install
running bdist_egg
running egg_info
writing requirements to CloneFinderAPI.egg-info/requires.txt
writing CloneFinderAPI.egg-info/PKG-INFO
writing top-level names to CloneFinderAPI.egg-info/top_level.txt
writing dependency_links to CloneFinderAPI.egg-info/dependency_links.txt
reading manifest file 'CloneFinderAPI.egg-info/SOURCES.txt'
writing manifest file 'CloneFinderAPI.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
warning: install_lib: 'build/lib' does not exist -- no Python modules to install

creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying CloneFinderAPI.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying CloneFinderAPI.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying CloneFinderAPI.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying CloneFinderAPI.egg-info/requires.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying CloneFinderAPI.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
creating 'dist/CloneFinderAPI-0.1-py2.7.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing CloneFinderAPI-0.1-py2.7.egg
Removing ~/anaconda3/envs/python3/lib/python2.7/site-packages/CloneFinderAPI-0.1-py2.7.egg
Copying CloneFinderAPI-0.1-py2.7.egg to ~/anaconda3/envs/python3/lib/python2.7/site-packages
CloneFinderAPI 0.1 is already the active version in easy-install.pth

Installed ~/anaconda3/envs/python3/lib/python2.7/site-packages/CloneFinderAPI-0.1-py2.7.egg
Processing dependencies for CloneFinderAPI==0.1
Searching for foo>=3
Reading https://pypi.org/simple/foo/
No local packages or working download links found for foo>=3
error: Could not find suitable distribution for Requirement.parse('foo>=3')

Thank you very much.
Xiucz

solved and closed


[Bug]: module/pulseaudio does not preserve audio balance on volume change

Checklist

[X] I have read the appropriate section in the contributing guidelines
[X] I believe this issue is a problem with polybar itself and not a misconfiguration on my part
[X] I have searched for other open and closed issues that may have already reported this problem
[X] I have checked the known issues page for this problem.
[X] I have followed the debugging guide to narrow down the problem to a minimal config.

Steps to reproduce
I have polybar-3.7.1 from here. In my .config/pulse/default.pa I have this line to set the audio balance on my speakers: set-sink-volume 0 +0% +35%.
When I start my DE, polybar loads fine and the default balance if fine. Using the mouse wheel on polybar to change the audio volume changes each audio channel by a different percentage. This does not preserve the audio balance.
Minimal config
[bar/DP-2]
monitor = ${env:MONITOR:DP-2}
background = ${colors.background}
foreground = ${colors.foreground}

modules-right = pulseaudio date

font-0 = Noto Sans Mono:size=11:weight=bold;2
font-1 = unifont:fontformat=truetype:size=8:antialias=false;2
font-2 = siji:pixelsize=10;1
font-3 = Font Awesome 6 Free:style=Solid:pixelsize=11;2

override-redirect = true
cursor-click = pointer
cursor-scroll = ns-resize

[module/pulseaudio]
type = internal/pulseaudio
click-right = pavucontrol
label-muted-foreground = #FFBF00
label-volume-foreground = ${root.foreground}

format-volume = <label-volume> <bar-volume>
label-volume =  %percentage%%
label-muted = "   "

Polybar log
No response
Expected behavior
Using the mouse wheel on polybar to change the volume, both audio channels should change by the same amount to preserve the volume balance.
Actual behavior
When I use the mouse wheel on polybar to change the volume, the right audio channel changes by +5% or -5%. But the left channel changes by only +3% or -3%, which means that module/pulseaudio does not preserve the volume balance.
To confirm, using the volume wheel on my keyboard, or using the mouse wheel directly in pulsemixer, does preserve the balance as expected.
Window Manager and Version
labwc-0.7.1 or enlightenment-0.26.0
Linux Distribution
Slackware-current
Polybar version
polybar-3.7.1

Additional Context / Screenshots
No response

So what is currently happening is that when we change the volume, it preserves the proportion of the channels. For example if you are at (50%, 25%) and increase by 10%, you get to (60%, 30%), so the 2-1 proportion is preserved. I suspect the main reason we do it this way is that pulseaudio has an API for this (pa_cvolume_inc and pa_cvolume_dec) but not for changing volume while preserving balance.
I have no idea what the natural/intuitive/correct way to handle this is. But it probably makes sense if we match the behavior of pactl and pulsemixer here.
I don't really have the capacity to work on this, but PRs are welcome.
The code for this is in inc_volume in src/adapters/pulseaudio.cpp


Happyx coin info change logo url

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

this adds a duplicate block for address 35W3RdZ3pKoqxcHkwSfHkRDRerdVUhHBkAK8MK1NY2ZJ instead of modifying the existing one: https://github.com/solana-labs/token-list/pull/13523/checks?check_run_id=4779596894


teste 2

Describe the bug
asasasasdasdasdsadsadasd
Link to the Bolt URL that caused the error
https://bolt.new/~/bolt-shadcn-wj8bka
Steps to reproduce
asdasd
Expected behavior
asdasda
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 77720743,
  "usedJSHeapSize": 67886043,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = luiznova
Chat ID = dc4b460485ba

Additional context
No response

This issue does not have enough information. Please file a new one if you run into any issues.


-Unrelated- Hooking into KernelAPI calls  -🐨 -

Hello koala!
This is completely unrelated to the project but I don't have much of a way to DM for such a thing 😢
I was wondering how I could hook into certain calls, namely, GetSystemTime, GetLocalTime, GetSystemTimeAsFileTime, NtQuerySystemTime, GetSystemTimePreciseAsFileTime, GetSystemTimeAdjustment, GetSystemTimeAdjustmentPrecise, GetSystemTimes, RtlGetSystemTimePrecise , among others.
All in all, just every windows call to give a time value to a program that requests it, what I intend to do is essentially Nirsoft's RunAsDate but open source and more powerful, as it cannot, for instance, spoof the datetime library from Python.
The problem being that I do not have experience using PolyHook2 or any hooking library, and the documentation is just past me, as such I would like to humbly ask for your help with this project, if you accept we can move to discuss this wherever you wish, otherwise you can just close this issue, as I know you have other projects to work on and time is of the essence.
Thank you for your attention and have a good day!

I personally believe that you might need to get comfortable with some of the basic aspects of C++ before you move on to more advanced ones. In other issue you mentioned that

I've yet to make a header file myself.

I mean this is like learning how to fly without learning how to walk first.
Nevertheless, if you feel like you are equipped for the job, then there are plenty of tutorials out there, just google c++ function hooking in general or WinAPI function hooking tutorial in your specific case. First google search result.
The author of the polyhook library has actually written several articles regarding hooking methods which is available here. There are other hooking libraries out there, so you are free to choose any of them. It's just that most of them only support Detour hooks, while polyhook 2 supports 8 different types of hooks. From your use case you can either use Detour, IAT, or EAT hooks. Koalageddon itself has couple of examples, although it might be difficult to understand the relevant part because of the presence of other code. Example with CreateProcessW.

I just remembered that in case of WinAPI, hooking doesn't really require hooking libraries.
Normally you need to execute the overwritten instructions if you hook functions, and the logic that handles that can become quite complex, if you are aiming to cover all possible cases. This is what hooking libraries are trying to take care of - correctly executing the overwritten instruction, if you call the original.  But Microsoft has deliberately reserves the first 4 bytes of function prolog with 2 nops (it's mov edi, edi actually, which is slightly faster than 2 nops, but still occupies 4 bytes). So you can simply overwrite them with a jmp instruction to any address without having to deal with the overwritten instructions, which is what the hooking libraries are made to deal with. And you can safely call the instruction right after the reserved for bytes to execute the original.
Microsoft has reserved those 4 bytes to allow hot-patching security vulnerabilities on machines where rebooting is not possible without preparation. For example you cannot simply reboot a web server since it may bring down one or more websites for quite some time.

I just remembered that Polyhook 2 has a dead-simple example for a standard Detour hook: https://github.com/stevemk14ebr/PolyHook_2_0/blob/25c88bacc317ae1c371887b5d826f513301ce4e4/ExampleProject/ExampleProject/ExampleProject/ExampleProject.cpp

Thank you :D


The official name is SAP Business Application Studio. Please do not use BAS in the documentation.

Issue description

Feedback Type (Optional)
None
Page Title on SAP Help Portal (prefilled)
SAP Business Application Studio
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/SAP_FIORI_tools/17d50220bcd848aa854c9c182d65b699/b0110400b44748d7b844bb5977a657fa.html?q=business application studio

Issue has been fixed and pushed to production


Convert webpage zooming to native

A native solution for webpage zooming is possible, but needs lots of work. The JS version is "hacky" but works really well in testing on iOS 15.
v1.0.1 will feature the JS version of zooming, but a later version update should switch to native zooming.

Well, this was easy to fix!
Native solution implemented in commit c9d5b7e214d15f9afe70dfed136af3817a970e93


Docker Container Available

Hi :),
is there a docker container available to try out the U-Net##?
Best regards

Hello :)
There is no docker container available right now, but I will try to work on creating one. There has also been an update on the model.
All the best.


Bluetooth: Audio: fix possible overflow in BASE LTV decode

Consistently process BIS data, codec data and codec metadata. Range check count of data received so that buffer overflow cannot occur.

The code looks good. Still unsure about whether we should truncate the BASE information or not.

If we bail out with an error and storing none of the BASE values we are guarantied not to be able to receive the broadcast. If we keep what we can store there is a fairly good chance that the most important stuff (what is required by the profiles) is there and a Zephyr device can receive the audio. (vendor specific data tends to be placed as the last items.)
None are optimal, but since the broadcaster and broadcast assistant might not be the same device, there is often nothing the broadcast assistant can do if the Zephyr device fails to attach to the broadcast, so I favor the last option and store what we can. And then get the underlying issue fixed - I 100% sure the amount of metadata and codec data will keep increasing - hence a product launched today might work now, but not when a new profile adds more data.
The TLV approach is great for backwards compatibility IF the specification had included a mechanism for the peripheral to report back to a central which T was skipped :-)


feat(breadcrumb): toBeAccessibleBreadcrumb

Describe the feature you'd like:
A matcher for the Breadcrumb pattern as listed in https://www.w3.org/WAI/ARIA/apg/patterns/breadcrumb/

Released in https://github.com/veiko/jest-a11y/releases/tag/v1.10.0


Cannot stop the environment: subject id is empty

Describe the bug
Cannot stop the environment: subject id is empty.
Steps to reproduce

Create three services: db, backend and frontend. The backend depends on the db, and the frontend depends on the backend;
Stop the environment.

Result

Environment

Walrus version: v0.4.0-rc3

Probably related to https://github.com/seal-io/walrus/issues/1470

Cannot reproduce in main


Retain data validation annotation attributes in model generation

Attributes such as [Required], [Range], [StringLength(5)] from the System.ComponentModel.DataAnnotations namespace should be retained in the generated client side models so validation can be performed.

Attributes such as [Range], [Required], [StringLength] are being generated, but I'm not seeing the [Display(Name = "Field Name")] in the generated c# code?

We cannot retain everything via json schema/openapi/swagger because these fields do not really exist there.. some of them are generated as xml docs i think

would it be possible to generate a [System.ComponentModel.DataAnnotations.DisplayName(Name = "")] annotation in the c# httpclient via the xml docs perhaps?
I see [MaxLength(15)] in my dto is able to generate a
[System.ComponentModel.DataAnnotations.StringLength(15, MinimumLength = 1)] in my http generate c# client.
I also notice that if I set the description on the [Display] attribute, it will populate a comment of the value. For example in my dto I set:
[Display(Description = "Total Price")]
public decimal TotalPrice { get; set; }
and in the generated client, it comes across as:
/// Total Price
[Newtonsoft.Json.JsonProperty("totalPrice", Required = Newtonsoft.Json.Required.Always)]
public decimal TotalPrice { get; set; }
it seems the description part of the [Display] is available, could this be used to pass this information down to the generated c# client as a [Display] attribute?

Just something else to add. When adding the Name of the [Display] attribute, looking at the json that get's generated, it sets the value against a title param. For example, in my dto I add
       [Required]
       [Range(1.9, 3.9)]
       [Display(Name = "Total Price", Description = "The total price of the idea")]
       public decimal TotalPrice { get; set; }

which renders the following:

Is there a way to have the C# generated client, use the title value of the property, in this case totalPrice, as a [System.ComponentModel.DataAnnotations.Display(Name = "Total Price")] ]?
I'd be willing to help with this matter if needed, but am pretty new to this kind of thing and not sure how to go about it or get started? I do see some kind of annotation template, but it's empty and not sure how I would use it?

Hi @RicoSuter
Just wondering what you're thoughts are on the items I mentioned above last week and if you think it can work?
Thanks again

@RicoSuter


[Bug] mlc_llm serve throws CUDA: invalid device ordinal

🐛 Bug
I found this repo on Huggingface, kindly publicly shared by @bayley who also provided the commands for serving, but upon using those commands, I get an error: CUDA: invalid device ordinal. I tried using mlc-ai/Mistral-7B-Instruct-v0.3-q4f16_1-MLC instead and it works fine.
Could it be that each version of mlc requires a new MLC model conversion? If that is likely the cause, then:

How would I pip install a version of mlc from 25 days ago when bayley uploaded the model?
Could the error message be made more informative - e.g. "This model was converted with an older version of MLC. Please install that version of MLC with [...] or re-convert the model."

To Reproduce
docker run -it --rm --gpus all nvidia/cuda:12.1.1-devel-ubuntu22.04 bash
apt-get update && apt-get install tmux python3 python3-pip git git-lfs -y
git-lfs install
python3 -m pip install --pre -U -f https://mlc.ai/wheels mlc-llm-nightly-cu122 mlc-ai-nightly-cu122
tmux new -s server
cd workspace
git clone https://huggingface.co/bayley/Midnight-Miqu-70B-v1.5-q4f16_1-MLC
mlc_llm compile Midnight-Miqu-70B-v1.5-q4f16_1-MLC/mlc-chat-config.json --device cuda --overrides "tensor_parallel_shards=2" -o /workspace/Midnight-Miqu-70B-v1.5-q4f16_1-cuda.so
mlc_llm serve /workspace/Midnight-Miqu-70B-v1.5-q4f16_1-MLC --model-lib /workspace/Midnight-Miqu-70B-v1.5-q4f16_1-cuda.so --host <IP_ADDRESS> --mode server --port 8000

Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/usr/local/lib/python3.10/dist-packages/mlc_llm/cli/worker.py", line 54, in <module>
    main()
  File "/usr/local/lib/python3.10/dist-packages/mlc_llm/cli/worker.py", line 49, in main
    worker_func(worker_id, num_workers, reader, writer)  File "tvm/_ffi/_cython/./packed_func.pxi", line 332, in tvm._ffi._cy3.core.PackedFuncBase.__call__  File "tvm/_ffi/_cython/./packed_func.pxi", line 277, in tvm._ffi._cy3.core.FuncCall
  File "tvm/_ffi/_cython/./base.pxi", line 182, in tvm._ffi._cy3.core.CHECK_CALL
  File "/usr/local/lib/python3.10/dist-packages/tvm/_ffi/base.py", line 481, in raise_last_ffi_error
    raise py_err
tvm.error.InternalError: Traceback (most recent call last):  6: tvm::runtime::PackedFuncObj::Extractor<tvm::runtime::PackedFuncSubObj<tvm::runtime::TypedPackedFunc<void (int, int, long, long)>::AssignTypedLambda<void (*)(int,int, long, long)>(void (*)(int, int, long, long), std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)::{lambda(tvm::runtime::TVMArgs const&, tvm::runtime::TVMRetValue*)#1}> >::Call(tvm::runtime::PackedFuncObj const*, tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)  5: tvm::runtime::WorkerProcess(int, int, long, long)
  4: tvm::runtime::DiscoWorker::Impl::MainLoop(tvm::runtime::DiscoWorker*)
  3: tvm::runtime::DiscoWorker::Impl::CallPacked(tvm::runtime::DiscoWorker*, long, tvm::runtime::PackedFunc, tvm::runtime::TVMArgs const&)
  2: tvm::runtime::PackedFuncObj::Extractor<tvm::runtime::PackedFuncSubObj<tvm::runtime::TypedPackedFunc<void (tvm::runtime::ShapeTuple, std::__cxx11::basic_string<cha
r, std::char_traits<char>, std::allocator<char> >)>::AssignTypedLambda<void (*)(tvm::runtime::ShapeTuple, std::__cxx11::basic_string<char, std::char_traits<char>, std:
:allocator<char> >)>(void (*)(tvm::runtime::ShapeTuple, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >), std::__cxx11::basic_string<ch
ar, std::char_traits<char>, std::allocator<char> >)::{lambda(tvm::runtime::TVMArgs const&, tvm::runtime::TVMRetValue*)#1}> >::Call(tvm::runtime::PackedFuncObj const*,
tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)
  1: tvm::runtime::nccl::InitCCLPerWorker(tvm::runtime::ShapeTuple, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)
  0: _ZN3tvm7runtime6deta
  File "/workspace/tvm/src/runtime/disco/nccl/nccl_context.h", line 65
InternalError: Check failed: (e == cudaSuccess || e == cudaErrorCudartUnloading) is false: CUDA: invalid device ordinal

Environment
This Docker image:
nvidia/cuda:12.1.1-devel-ubuntu22.04

Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA): CUDA
Operating system (e.g. Ubuntu/Windows/MacOS/...): Ubuntu 22.04
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...): Dual RTX 4090s (Runpod)
How you installed MLC-LLM (conda, source): See above.
How you installed TVM-Unity (pip, source): See above.
Python version (e.g. 3.10): 3.10.12
GPU driver version (if applicable): 550
CUDA/cuDNN version (if applicable): 12.4 according to nvidia-smi within the Docker container

likely it was due to config being compiled for higher TP(leveraging multiple gpus) but you don't have as enough gpu

yeah the model was compiled for 4x GPU. I'll upload ones for 2x GPU in a bit or you can compile them yourself (it's not hard, just a bit inconvenient for the bigger models)

Ah, thank you both.
@bayley A 2x one would be great if it's not too much trouble! Or can I just change the tensor_parallel_shards values in the config file to 2? I just tried that, and also reduced all the 8192 values (other than hidden_size) down to 4096, but am running into:
TVMError: Check failed: (output_res.IsOk()) is false: Insufficient GPU memory error: The available single GPU memory is 20584.716 MB, which is less than the sum of model weight size (18643.766 MB) and temporary buffer size (2977.048 MB).

which seems a bit strange because there should be ~24GB of memory available in each of the two GPUs. So my guess here is that simply changing the tensor_parallel_shards values in the config file is not a valid approach.

oh hmm, I just realized the model quants are independent of the TP rank outside of the config file, so something else is going on here - @tqchen is --overrides "tensor_parallel_shards=2" the correct flag to set the number of TP shards during compile time?

Is --overrides "tensor_parallel_shards=2" the correct flag to set the number of TP shards during compile time?

@bayley Yes that's true. Sorry for the late response.

The chat config in the HF repo was generated with TP=4, besides the TP override during compile do any other edits need to be made to the config for TP=2?


GN-6404: fix behaviour of embedded editor keymap

Overview
This PR solves issues with the keymap initialization of an embedded editor instance:

The default keymap of an embedded editor is now correctly initialized
An additional shift+Enter handler has been added to the default embedded editor keymap
Only run leave-on-enter modifier when Enter has been pressed without modifier keys (Ctrl, Alt, Shift or Cmd)

connected issues and PRs:
GN-6404
How to test/reproduce
Before this fix

Set the allowed content of a link to inline*
Insert a link with some text inside
Notice that none of the editor shortcuts work inside the link (Mod+b, Enter, Mod+i etc.)

After this fix

Set the allowed content of a link to inline*
Disable the leave-on-enter modifiers in the link component
Insert a link with some text inside
Notice that the editor shortcuts work inside the link (Mod+b, Enter, Mod+i etc.), as well as shift+Enter

Checks PR readiness

[x] UI: works on smaller screen sizes
[x] UI: feedback for any loading/error states
[x] Check cancel/go-back flows
[x] Check database state correct when deleting/updating (especially regarding relationships)
[x] changelog
[x] npm lint
[x] no new deprecations

gonna merge this but some notes:

adding newline at end of node looks like it does nothing the first time, but this is because browsers dont' render a trailing  -> I don't think this is a regression
I misunderstood the fix, my idea was to have leave-on-enter behavior for enter, and hard-break on shift-enter. But I thought this because I thought we added leave-on-enter for variable nodes as well as links, it seems we didn't.


GitHub LWU complete

no

@Rishabh248 please create a new branch and commit a file from that branch. Also, change the base branch from main to the created branch.
Resource


CameraRoll on Android

From the docs:
@param {string} tag On Android, this is a local URI, such as "file:///sdcard/img.png".
Is the camera roll API actually available on Android? I get undefined is not an object (evaluating RCTCameraRollManager.getPhotos) when calling CameraRoll.getPhotos.

Answered in #3743
(searched for camera roll instead of CameraRoll)


Repo sync for protected CLA branch

The pull request is created from master637636092850540086 to master to fix git push error for protected CLA branch

@opbld16 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

See comments in https://github.com/MicrosoftDocs/visualstudio-docs/pull/6906#issuecomment-892071813

See https://github.com/MicrosoftDocs/visualstudio-docs/pull/6906#issuecomment-892071813 for comments that relate do this.

@ShannonLeavitt
I don't think the issue in the visualstudio-docs-pr repo applies here. We don't have automation that automatically merges updates from cpp-docs-pr into cpp-docs. We've always merged these by hand. The automation going the other way also requires manual intervention. We rarely get more than one submission to the public repo in a day, so I don't think we've had issues with squashing the auto-generated PRs yet. The other issue about PRs that qualify for auto merge not being auto merged hasn't been tested yet, so we might have the same problem Gordon describes.


10.1 counting inversions (naive approach in O(n^2))

My code is clear and well understanding
Please assign it to me

Need to add a description and your approach. Thanks, Closing as of now.


Test commit

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Add test commit

Thanks for testing Brendan. No longer needed, closing PR.


Very poor performance

Incredibly poor performance, on simple vim files to big projects when running the latest neovim NVIM v0.5.0
With the plugin enabled: https://imgur.com/W1Ge9w3
With the plugin disabled: https://imgur.com/XvCkAHx
Now there's a huge improvement in not only cursor speed but general vim speed like auto completion etc, which I don't see how it's related but it for some reason is.
Now the file in the video is only a example but disabling the plugin improves speed on every file I've opened so far.
Please do let me know if there is any fix for this, I would love to keep using this plugin.

Turns out that if i disable  "cursorcolumn" in the vim settings aka set cursorculumn it works perfectly, although if i enable it performance decreases.


call hide twice cause hide the same view

add status so that when call hide twice quickly before the first hidden animation finish, we will not hide the same view twice. I think it is because the run loop break the view animation into piece, the main thread will proceed the second hide request before the animation finished

Can you avoid calling hide twice? It will still work, just result in the HUD disappearing immediately instead of animating. To me it sounds like the caller would need to make sure this does not happen.

I guess https://github.com/jdg/MBProgressHUD/pull/89 is relevant here.

no the problem is still there. because hide with animation will perform animation before remove from the super view. if you call another hide, the method simply find the subview and perform hidding action, the subview found maybe the view in previous performing hidding animation. it is the root reason, if you cannot make sure the unique subview called, this problem may cause other bugs.
Sent from my iPhone

On 19 Jun 2016, at 7:13 PM, Matej Bukovinski<EMAIL_ADDRESS>wrote:
Can you avoid calling hide twice? It will still work, just result in the HUD disappearing immediately instead of animating. To me it sounds like the caller would need to make sure this does not happen.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ah, you're referring to hideHUDForView:animated:.


Remember/Sticky/Save timers across restarts

I just noticed that after restarting Windows, I lose all my timers and their titles. This is almost a deal-breaker for me.
I would really, really appreciate the option to remember/stick/save timers, or make certain timers permanent, so I don't have to name and organize them every time I restart my operating system.
Thank you.

@Wisdawn Fixed in https://github.com/i2van/hourglass

Thank you for bringing the feature to my attention, @i2van! I have now ticked the option to "Open saved timers on startup," but how do I save any timer? I don't see the option to save anywhere, and as you can see in the screenshot attached, Hourglass says I have "No saved timers." I confirm that I have set up three timers, though.

@Wisdawn You don't need to save timers explicitly, timer is saved when you close not yet expired one (don't ask me why it was implemented this way). Just close application and run it again without any command-line and all timers you have should be automatically loaded (if the corresponding option is set).
By the way, please use the Hourglass modified by me, old one has a lot of bugs I've fixed already.

@i2van Yes, I just realized that the timers save in many ways, including stopping and then restarting the timer! I restarted my system the other day to see dozens of duplicate timers when I started Hourglass. This is bizarre design.
I just installed your version. It does work much better, but it still lacks a Save button/link and preventing the saving of timers any other way.
Timers shouldn't save by stopping, starting, or in any way. There should simply be a Save button or link for saving, and that's it.
Thank you very much for working on your version. I was wondering why this useful app has been abandoned for so long and why there wasn't someone else working on updating it.

Tracking: https://github.com/i2van/hourglass/issues/24


fix: detect case sensitive filenames

Fix https://github.com/nvim-telescope/telescope-frecency.nvim/issues/207
Now frecency.validate_database detects case sensitive filenames and
delete if it is not found. For example, now it deletes foo.lua even if
Foo.lua still exists.
For performance issue, this function is now an async function, so you
should wrap it with async.void to call.

This problem will be solved in #232


Fix Contract Deployment population of arguments

This is resolved now - the bytecode was not passed correctly to the ContractFactory constructor, which (probably) resulted in an incorrect encoding/appended zeroes.


Feature/qr view customization

I pulled master into the swift2.3 branch and converted the code to Swift 2.3. This adds the customisation delegate to the swift-2.3 branch and syncs the code between the two branches.
The reason for this PR is that I'm using this library in my project but I need to be able to customise the QR code reader view. Since I can't update my project to Swift 3.0 yet, I decided to create a PR and add this functionality to your Swift 2.3 branch.

Oh great, thank you!
You should migrate to Swift 3.0 quickly because Xcode 8.2 will be the last release that will support swift 2.3.

Thanks @yannickl!


[client/network] Update authority discovery after authority-discovery w/ PR#3230 is released

https://github.com/libp2p/rust-libp2p/pull/3230 invalidates the assumptions in Substrate's Authority Discovery.
https://github.com/paritytech/substrate/blob/8d11cec9713a93b9f6bee1b29fde840da5ba56e0/client/authority-discovery/src/lib.rs#L88-L109
"Either you go with the new defaults and update Authority Discovery, or you override the defaults of libp2p."
https://github.com/libp2p/rust-libp2p/pull/3230#discussion_r1051976550
opening an issue so we don't forget

Looks like https://github.com/libp2p/rust-libp2p/pull/3230 was not merged. I will reopen this issue if libp2p team decide to reopen & merge it again.


Remove existing implementation of StartupExceptionPage and use the on…

…e in Common
[Fixes #831] Fix startup exception page to handle flattened exceptions
@Eilon @pranavkm

New screen shots, with and without details?

What does this page look like? The diagnostics one?

Here you go
https://jsfiddle.net/xenv6d9s/1/ with ReflectionTypeLoadExceptions
https://jsfiddle.net/446cq538/1/ with exceptions in Configure method of Startup

This startup page looks worse than the one we used to have 😄 .  Will fix.

🕐

:up: :date:

:up: :date:

:shipit:

83e0d4798eb4fd9113d06c54bba9d69faefa3d96


Backend: Bump the python version to 3.11

Description
Updated the Python version to the latest in the backend, as version 3.6 is no longer supported. Also, there is a problem with running 3.6 version on the latest Apple processors (like M2)

Thank you for the contribution and subsequently bringing the upgrade to our attention.
Closing as the base image was finally changed to almalinux, aligning with the rest of Invenio products


Complete integration between the miner and the validator side

[x] Complete integration between the miner and the validator side.
[x] Already tested it with mock, The integration was perfect.
[ ] Still remains configuring timeout in validator forward func.

    create_request_zero_score = await forward_create_request(self, validator_db_manager, miner_uid)
    time.sleep(20)
    update_request_zero_scores = await forward_update_request(self, validator_db_manager, miner_uid)
    time.sleep(30)
    delete_request_zero_score = await forward_delete_request(self, validator_db_manager, miner_uid)
    time.sleep(10)
    read_score = await forward_read_request(self, validator_db_manager, miner_uid)
    time.sleep(30)

[ ] Still have to update logs for perfect visualization.

You should submit this PR to staging branch. 😃


filterType in sap.ui.table.Column documentation

OpenUI5 version: 1.52.13
Browser/version (+device/version): Version 75.0.3770.142 (Official Build) (64-bit)
Any other tested browsers/devices(OK/FAIL): -
URL (minimal example if possible): https://sapui5.hana.ondemand.com/1.52.33/#/api/sap.ui.table.Column/controlProperties
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:
The documentation of sap.ui.table.Column mentions the possibility to pass a function to filterType: As an alternative you can pass a function which does the conversion. The function receives the entered filter value as parameter and returns the proper value for the filter expression..
I want to pass a function to filterType, but I can't seem to find a suitable way of doing this from an XML view.
I thought I could simply add the name of a function similar to how you define a handler for events like dataReceived in Binding (i.e.: filterType = ".getFilterValue"), but this doesn't work.
When I load the app I get the following message:
The filter type is not an instance of sap.ui.model.Type! Ignoring the filter type!
I also tried adding a function to filterType by using method setFilterType; here I don't get any error message but the  defaultComparator from sap.ui.model.Filter doesn't get the value formatter by the custom function passed to filterType.
Would you be kind to provide an example on how you could pass a function to filterType? If this is not possible from an XML view, it would be good to mention this restriction in the documentation.
Cheers,
SePo

Hello @seiupo ,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,
Petya Bogdanova
(UI5 Dispatcher)

Hallo @seiupo ,
Thank you for the ticket. We modified the documentation accordingly and added a simple example for passing a function to the filterType.
Best Regards,
Oliver Stoyanovski

Thank you 👍


PS: disable the "Sync Records"

Play Store version:  disable the "Sync Records" option for the attack records

sync records disabled


Disables on lock screen

When on the lock screen, this extension does not run and the icon returns to the normal, inaccurate version.
As soon as the user logs in, the extension re-enables and shows the updated icon.

Try installing to: /usr/share/gnome-shell/extensions/


Implement proper tail calls.

The following PR implements proper tail calls as demanded by ES6. The are behind the new experimental flag --proper-tail-calls. (Experimental because it necessarily reduces performance of the generated code.)
This probably makes Traceur the first transpiler properly supporting tail calls!
Each function that contains calls in tail position is rewritten. All tests pass with proper tail calls enabled by default. Traceur itself compiled with --proper-tail-calls true also works as expected.
At the moment, the transformer does not touch arrow functions, class declarations and class expressions (so they should be transpiled if their are supposed to be properly tail recursive). Likewise, template literals in tail position are currently not touched, so the corresponding transpilation should be enabled as well.
The changes to runtime.js are far less worse than what the diff shows. I had to wrap most of that code into an extra IIFE and the additional two spaces at the beginning of each line cause the diffs.
It is left as a TODO to check all other transformers that they do not convert tail calls in non tail calls.
Please review this PR; it's hopefully ready to merge.

I was thinking a bit more how to better tests this instead of just hoping we would run out of stack. Maybe we could write a non feature test that inspects the continuation somehow. Another option might be to throw an error and look at the error .stack.

I'll see how I can possibly reformulate the tests. --- I have added quite a number of comments above, please review them.

This is one idea how we can measure the stack size:
Error.stackTraceLimit = Infinity;

var stackSize;
function recordStackSize() {
  try {
    Error.prepareStackTrace = function(error, structuredStackTrace) {
      return structuredStackTrace.length;
    }
    stackSize = new Error().stack;
  } finally {
    delete Error.prepareStackTrace;
  }
}

function assertProperTailCalls(limit = 30) {
  assert.isTrue(stackSize < limit);
}

I have checked that the test environment accounts for about 15 stack frames, so the limit of 30 is very safe when one has only about 40 or 50 tail calls and 100,000.
Could this go in one file of the testing framework in test?

Issues have been taken care of in the new commit.
Tests run much faster now :-)
Although the 100,000 was quite impressive...
P.S.: Who is responsible for updating the table by kangax?

That would be @kangax ;-)

@mnieper please send PR on https://github.com/kangax/compat-table and I'll gladly merge it in! Thanks.

Pushed the corrections.

@kangax Thanks. I forked your project and took a look at it. But I'd like someone else to run the tests because the only browsers I have running are the latest Firefox and the latest Chromium, so this would probably give false results.

LGTM

Committed as d9cc38aa813a9591bb72e8fd8545984d45919498

Cool.

Yes. Extremely cool.

In the end it was easier than I thought. :-)

@mnieper I guess I can just change the result of TCO test from false to true for Traceur and that's it, right?

@kangax Both TCO tests should pass. (There are equivalent tests in Traceur's test suite.)
Apart from that, %GeneratorPrototype%.return also does work now in Traceur.
For ES7, async generators have been implemented. (https://github.com/jhusain/asyncgenerator)
Finally, could you please review the test here: https://github.com/google/traceur-compiler/issues/1730. I think most transpilers are broken in this regard and this could be a test worth adding to your table.

@mnieper thanks, I updated Traceur's ES6 results. For ES7, I don't believe we have a test for async generators (only async functions and async arrow functions). I'll get to that at some point. I'll also take a look at #1730 later if I get a chance — no time at the moment!

@sebmck looks like someone needs to step up their game ;)

@sebmck I have already been asked whether I could implement this work and my work on async generators in 6to5. However, as I am doing everything in my spare time, I'd rather like to do something new than just reimplement something (and I'd have to go through another codebase to understand it).
Nevertheless, I hope you can take a look at this stuff and maybe find another volunteer. In any case, I will readily answer any question.

@kangax I'm the only one working on the 6to5 internals and I do it all in my spare time so I really wish I could "step up my game". Also don't mean to be pedantic but the Traceur TCO tests should be marked as "flag" on compat-table since you have to explicitly enable the properTailCalls option.
@mnieper Awesome work! Will definently look into it when I get the time. What's your preferred way to be contacted? Would love to talk about it.
TCO was disabled because there were issues with nested tail calls returning their tail call instance. TRO is currently enabled by default though and actually makes recursive calls signifcantly faster:
function fact(n) {
  function f(n, acc) { return  n === 0 ? acc : f(n - 1, acc * n); }
  return f(n, 1);
}
fact(20);

Might be worth looking into this to see if it can be implemented in Traceur by default. The main issue we ran into with TCO was it significantly slowed down a lot of calls that didn't need to be, what's the performance impact like?

@sebmck
I'm the only one working on the 6to5 internals and I do it all in my spare time
Wow!
I haven't implemented special optimisations for self-calling functions - in fact, the code hasn't been optimised in any way yet. Running all tests with proper tail calls turned on takes 10 times the normal time (and from the way it has to be implemented in general, one has to expect a factor in that order).
For easy cases like function calling itself, yes, one should add an extra transformation pass to eliminate the recursion altogether. But as JavaScript has a lot of imperative features to write loops, I don't think this will be the majority of use cases for PTCs. I think they are much more important for programming in continuation passing style, combinator libraries and JavaScript as a compilation target for languages relying on PTCs. In this case, however, just optimising self-calling recursive functions won't help much.
For PTC to be really useful, we will wait for the browser engines to implement it (any news on this by the V8 and SpiderMonkey teams?). But with transpilers like Traceur and 6to5 allowing PTCs (albeit at lower performance) it allows library writers to already use this feature so that the libraries are already ready for production when PTC is implemented in the browser engines.
You can either contact me via email<EMAIL_ADDRESS>or via Google Talk/Jabber at the same address.

@sebmck I mostly meant it as a joke — 6to5 is already doing amazing :) We'll take care of flag situation shortly; thanks for noticing!

@kangax Sorry! In hindsight that much was obvious, had only just woken up :stuck_out_tongue_winking_eye:

@kangax How do you handle the flag thing? At the moment, the more options a transpiler offers, the worse seems to be its display on your table, doesn't it?
You would also have to add flags to all ES7 language elements Traceur supports because they are also hidden behind a flag (for the benefit of the user).

@mnieper There's a separate table for ES7: https://kangax.github.io/compat-table/es7/

@sebmck Yes, but all ES7 features of Traceur are behind  flags, which isn't shown on that table.

@sebmck 6to5 is really getting good. Like 6to5, traceur is also all done in our spare time (I try to use my 20% time but it is rare that I actually can do that). @johnjbarton is in a similar position.
@mnieper I work on V8 but PTC has not gotten on our radar yet. We have a lot of other important work to do before we will get to that.

@arv Thanks! I always get people thinking that I have some vendetta against Traceur when really it's the opposite and I really appreciate what you guys do and I'm super happy that both projects exist. Truth is that 6to5 (now Babel) and Traceur are targeting completely different ecosystems, Traceur with AtScript/Angular while 6to5 focuses on JSX/React.

@sebmck I didn't mean to imply that. I'm sorry if it came out that way.
I'm also don't feel happy with saying that Traceur is targetting AtScript/Angular. They liked how easy Traceur is to extend so they decided to add the transformation passes they wanted.
To me Traceur was a test bed for harmony features. We implemented a lot of the ES6 features before we brought them to TC39. Today I feel like we are still struggling to make traceur a useful tool for real world projects, something it seems Babel (nee 6to5) is doing much better at.

@arv No, you didn't imply that at all. It's just that people always assume there's some sort of bitter rivalry between projects in the same space.

I also don't feel happy with saying that Traceur is targetting AtScript/Angular. They liked how easy Traceur is to extend so they decided to add the transformation passes they wanted.

I wasn't trying to say that these are in anyways locked into particular frameworks. Both projects can find use anywhere, but Traceur has become most popular within the Angular community as 6to5 has become in the React community.

Today I feel like we are still struggling to make traceur a useful tool for real world projects, something it seems Babel (nee 6to5) is doing much better at.

Thanks, it means a lot! Babel owes a great deal to Traceur and the standards it's created, it wouldn't exist without it and I'm incredibly grateful.

@arv @sebmck
I think the whole developer community can consider itself fortunate that there are at least two excellent ES6 to ES5 transpilers. The more tools there are that are able to prove that some suggested features for ES7, 8, 9, etc. is actually working and will make coding in JavaScript easier, the more likely it is that this feature will be accepted in a new standard of the language.
So I really hope that you both continue your work on your tools so that both will be used a lot in the JavaScript community.
(By way of comparison, if only V8 but no SpiderMonkey, or vice versa, were in daily use, JavaScript and its engines wouldn't have seen such a rapid progress recently. asm.js is one example that readily comes to my mind.)


Force pushing tag seems to trigger CD workflow

The CD workflow is only supposed to trigger if the file 'VERSION' is included in a push.  If you force push a tag move (such as v1) to the latest commit, even if the latest commit is bumping the compose.override.yaml file, the CD workflow is triggered.  This tries to re-release and re-publish for a second time.   The release logic is pretty forgiving and same for Docker image publish.   The issue is actually the update compose.override.yaml update fails.  A quick fix would be to make this operation pass even if no update to the file.
See: https://github.com/JeffersonLab/container-workflows/wiki/Workflow-Tagging

If we use an annotated tag instead of a lightweight tag we can add the [no ci] hint.  This might be easiest as just a procedure change.

Using annotated tag with [no ci] doesn't work.   Updating the CD workflow with:
    tags-ignore:
      - '**'

does work.  This seems like a bug that the paths filter isn't honored and also that the [no ci] isn't honored.  Oh well.


Fix rendering of enum attribute labels

Rendering of enum attributes does not take into account ::form/field-labels form option.
During investigation I found that all field renderers use (form/field-label ...) except for enum attributes. This patch removes custom handling and uses the function instead. Tested on both cardinality :one and :many enum attributes.

Thanks

released in 1.2.8


Fix migration step

To fix the following error while migrating:
SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes (SQL: alter table translations add unique translations_table_name_column_name_foreign_key_locale_unique(table_name, column_name, foreign_key, locale))

SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes

It sounds like your MySQL installation might be older than 8.0? InnoDB is the default storage engine for MySQL 8.0+, older versions use the MyISAM storage engine as default. You can check which storage engine your database uses by running the following SQL. Change DATABASE_NAME to your database, which should return a list of each table and its storage engine.
SELECT TABLE_NAME, ENGINE
FROM INFORMATION_SCHEMA.TABLES
WHERE TABLE_SCHEMA = 'DATABASE_NAME';

You can append and ENGINE = 'MyISAM' if you want to search for a specific engine.
If any table uses MyISAM you can alter the table to use InnoDB using the following SQL. Substitute the TABLE_NAME with the name of the table you want to alter.
ALTER TABLE TABLE_NAME ENGINE=InnoDB;


terror: replace sirupsen/logrus by pingcap/log

What problem does this PR solve?
Apply pingcap/tidb#13459 on this repository.
Fixed some unit test failures on Windows.
What is changed and how it works?
Replaced the only use of sirupsen/logrus in terror by pingcap/log.
Check List
Tests

Unit test

Code changes

Has exported function/method change

The three log-related functions (terror.MustNil, terror.Call and terror.Log) now write logs to the default logger of pingcap/log instead of logrus.

Side effects
Related changes

PTAL @tiancaiamao

LGTM

(CLA broken )


Write unit tests for text formatting

User Stories

As a user, I want to see emails formatted as links.

As a user, I want to see phone numbers formatted with hyphens.

As a user, I want to see the duration of a voicemail in hours, minutes, and seconds.

As a user, I want to see the date formatted by how long ago it was recorded.

Functions and unit tests complete


Handle GCS bucket record deletion

Detect and plumb through to worker for deletion from Datastore Bugs that are no longer present in GCS.
Related changes (#2023, #2029) to the NVD CVE generation cause CVEs no longer being generated (due to changes in the heuristics like that made in #1707) to remain existing in GCS.
This PR addresses cases like this and the need identified in #1467 by adding a deletion phase to the importing of new/updated records. The functionality is flag-protected, it won't go live in Production until a new --delete flag is included so in the execution of the importer.
Incidental changes:

use the GCS bucket directory_path to efficiently filter the blobs returned when listing bucket contents
make blob retrieval resilient to blob generation change between blob listing and blob retrieval (this can happen if combine-to-osv happens to have run in between these two points in time)
fix a behavior inconsistency with schema validation not being performed when ignore_last_import_time is in effect (addressing head scratching TODO from @michaelkedar)
tidy up the existing tests, making them more readable and debuggable
add a slow manual test against live data in staging to validate real-world behavior and run time (this adds ~13 minutes to an import run on just the CVE GCS bucket)

how's the performance of this running on the staging CVE bucket? if this ends up being slow and impacts our freshness SLO, we may need to consider moving this to e.g. a separate "cleanup" cron job instead of sticking this in the importer.

Yeah I think based on my testing so far, while not running on the same sized GKE node as it would in production, running it as part of the regular importing run isn't looking tenable. I think we'll need to have a separate say daily cronjob that runs it in deletion mode. Good thing I've got all of that behind a flag :-)


Fix intermitent test failures due to non-unique entity names

leverage the withTimestamp method in a few more tests
move withTimestamp into common module and remove copies

pg4/1417 in progress

@mdeuser is this abandoned?


Fix preg_match error on 7.4snapshot

Error.

@kubawerlos , can you please elaborate why original code was problematic? thanks

@keradus use the link form first message, here is original warning.

Thank you @kubawerlos.


Use constructor based injection

Part of #2184
Remove setters from themes module

@DavidRG13 as you go through these changes, please ignore the saml-idp module for now. Working on a few changes ATM and I'd like to minimize merge conflicts.

Of course, if you have already done that one it's perfectly fine. We shall merge.

Don't worry mate, working on another module

Excellent. One thing I noticed by the way: recent changes broke master at runtime. When you change type information for generic collections in beans, you must make sure all instances of that bean are changed.
For instance, if the bean is:
public List<Object> someList() {}

Make sure all references to it are also changed:
@Autowired
@Qualifier("someList")
private List<Object> list;

Leaving it like:
@Autowired
@Qualifier("someList")
private List list;

causes breakage.


Change font size by half-steps with Increase-/DecreaseFontSize bindings

(extracted from https://github.com/jwilm/alacritty/issues/1339)
Which operating system does the issue occur on?
Ubuntu 18.04 LTS
If on linux, are you using X11 or Wayland?
X11
I noticed that if I set font size to 7.5 it reports Size(15) in the console. And by using the shortcuts to resize the font I can only achieve odd numbers until I go all the way down to 2. I think this means that I cannot step through all font sizes.
$ alacritty -v
Welcome to Alacritty.
 As jwilm has mentioned that is intended and shouldn't cause any issues.

You do not. It's not possible to increment/decrement by 0.5 using the shortcut. But I agree that it would probably make sense to do that.

Will you reopen this issue as a request for IncreaseFontSize to increase the font by the smallest step possible? Since Alacritty became DPR/DPI aware I would really like to be able to single step the fonts in different terminals.

I'm fine with changing it to step by the .5 instead of 1., so I've reopened this issue. If @jwilm disagrees with this or thinks this is unnecessary, I guess he can just close this issue again. :)

sounds good to me. Not sure why we don't do this already

Hey, I'm new around here but I've been using alacritty and also trying to learn some rust. How can I help with this issue?

Hi, this should be a fairly simple issue, if you want to have a go at it, here are some basic instructions:
The code where font sizes are increased/decreased is here:
https://github.com/jwilm/alacritty/blob/master/src/input.rs#L229-L234
To make the desired change, these would need to be changed to increase/decrease by (-)0.5 instead of (-)1.
This will lead to a few issues with types, because the change_font_size method needs to take an f32 instead of an i8 now, this has to be changed in the input.rs and event.rs file. Changing just https://github.com/jwilm/alacritty/blob/master/src/input.rs#L65 should allow the compiler to throw errors and tell you all the different places where things need to be changed.
Finally the method in term/mod.rs has to be switched, this is where the actual font size change happens:
https://github.com/jwilm/alacritty/blob/master/src/term/mod.rs#L844
Here you just need to change the type of delta to f32 and then replace the f32::from(data) part with just data.
I hope this should get you going if you want to tackle this issue @felippemr. If you want to give it a try just let me know and I'll assign this to you. If you have any further questions feel free to just respond to this issue or ask on irc (freenode/#alacritty). I'll answer as quickly as possible. :)

Nice, thanks for the help! I'll give it a shot later today and will let you know if I have any problems.

I'm working on it, can you assign it to me?

I actually can not, I'm not sure why. But this should suffice as a record that someone else is currently working on it.

I created a pull request. Would you be able to take a look?

Sure! :)

@NickeZ the pr with this change has been merged.


Fix Linting Issues

The current state of the repository is highly linted. As we go along, our goal is to make the repository lint free. In order to solve this issue follow the given steps.

To understand what is meant by linting checkout this document.
Install ESlint and enable extension in your code editor. Checkout this wiki page for the steps.
Comment on this issue to get assigned a particular folder to work on. Do not start working unless you have been assigned.
Once assigned, checkout all the files in that particular folder. In a given file the lint issues appear as red squiggly lines. On hovering them, a message is displayed stating the reason for the linting issue.
Fix all the issues in the assigned folder. Checkout this commit for sample. Make sure that all the files in the particular folder are covered.
Send a PR and respond to comments.

The folders that we need to update are;

[ ] src/views/AboutUsPage @abhishekshree
[ ] src/views/BlogsPage @abhishekshree
[ ] src/views/ContactUsPage @abhishekshree
[ ] src/views/EventsPage @abhishekshree
[ ] src/views/LandingPage @abhishekshree
[ ] src/views/SingleBlogs
[ ] src/views/TeamPage
[ ] src/views/Vehicles
[ ] src/components/Footers @ritam2358
[ ] src/components/Navbars @ritam2358
[ ] src/components/UI @ritam2358

A request to work on folder src/views/SingleBlogs


图标在状态栏中的位置不能保存

Describe the bug
In mac os 11, the manually opened application will be hidden every time after booting, but before that, I have set that application to not hide, the software can’t remember whether my application is hidden
Steps to reproduce

Open the application with the status bar icon
Set the application to display status
Close the application
Reopen

Expected behaviour
The application should be displayed in the status bar, not hidden
Screenshots

Environment
mac os11 beta7
Hidden bar: 1.6 (10)
All applications

Same question on macOS Big Sur 11.0.1 20B29 x86_64

Fixed in latest MacOS


Fix refresh issues (after login, auto refresh)

Switch from Promise design to async/await that wait for responses instead of synchronously continuing execution before things would be ready.

Looks good to me


✏ Add phone authentication through Supabase

Currently email is the only authentication option we have. It would be nice to allow users to verify their account via a OTP SMS message. It looks like Supabase support this.
Authentication > Providers

❌
Since we no longer have email confirmations, this is not really needed.


⚠ Alchemy Unbound API has degraded performance

In 092abb2, Alchemy Unbound API ($API_URL?token=$API_TOKEN) experienced degraded performance:

HTTP code: 211
Response time: 276 ms

Resolved: Alchemy Unbound API performance has improved in db8127b after 8 minutes.


Adding Nitish Malhotra

This pr fixes issue #195
Signed-off-by: Will Calcote<EMAIL_ADDRESS>

LGTM 😂🙏🏽

Whoo-hoo! 🥳


Add missing tests for sql

As you can see in the following screenshot,
There were some missing sql test using Series, so I added it to Doc test.

could someone check this change maybe if available?
Thanks.

Codecov Report

Merging #733 into master will decrease coverage by 2.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #733      +/-   ##
==========================================
- Coverage    94.3%   91.92%   -2.39%
==========================================
  Files          32       32
  Lines        5570     5570
==========================================
- Hits         5253     5120     -133
- Misses        317      450     +133

Impacted Files
Coverage Δ

databricks/koalas/sql.py
95.83% <ø> (+1.04%)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
77.5% <0%> (-7.5%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/frame.py
93.13% <0%> (-1.74%)
:arrow_down:

databricks/koalas/namespace.py
92.15% <0%> (-1.57%)
:arrow_down:

databricks/koalas/plot.py
93.78% <0%> (-1.13%)
:arrow_down:

databricks/koalas/utils.py
97.31% <0%> (-0.68%)
:arrow_down:

databricks/koalas/testing/utils.py
77.35% <0%> (-0.63%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


Error: Unexpected token target in parse_target

The following minimal example:
#[macro_use]
extern crate delegate;
extern crate petgraph;
use petgraph::{Graph, Directed, graph::DefaultIx};

pub struct DirectedAcyclicGraph<N, E>(Graph<N, E, Directed, DefaultIx>);

impl<N, E> DirectedAcyclicGraph<N, E> {
    delegate! {
        target self.0 {
            /// Return the total number of nodes in the graph.
            pub fn node_count(&self) -> usize;
        }
    }
}

Fails to compile with the error:
error: no rules expected the token `target`
  --> src/main.rs:10:9
   |
10 |         target self.0 {
   |         ^^^^^^

error: ParseError! Unexpected token `target` in `parse_target`. This is possibly a bug in the delegate library itself. Please file an issue with the following debug information:

{ items : [  ] , action : delegate__expand }

  --> src/main.rs:9:5
   |
9  | /     delegate! {
10 | |         target self.0 {
11 | |             /// Return the total number of nodes in the graph.
12 | |             pub fn node_count(&self) -> usize;
13 | |         }
14 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

This is rustc 1.26.0 on stable-x86_64-apple-darwin.

Good catch! Fixed in 0.1.2, thanks for the bug report!

Thanks for the quick fix!


Fix data_files when passing data_dir

This code should not return empty data files
from datasets import load_dataset_builder

revision = "3d406e70bc21c3ca92a9a229b4c6fc3ed88279fd"
b = load_dataset_builder("bigcode/the-stack-v2-dedup", data_dir="data/Dockerfile", revision=revision)
print(b.config.data_files)

Previously it would return no data files because it would apply the YAML data_files: data/**/train-* pattern to this directory
cc @anton-l

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Add HUC VN Domain

Website Url: http://huc.edu.vn/
IT-related course: http://flis.huc.edu.vn/?page_id=19

@TanNNSE02371 Pull request merged. Thank you!

When will I can send the student license request again? I just tried few minutes ago but no lucks.


Update telegram to 3.3.2

split https://github.com/ioBroker/ioBroker.repositories/pull/3679

RE-CHECK!


feat(gke): Add GKE checks

Checklist

[x] GCP_GKE_001: GKE Control Plane is Regional (HA)
[x] GCP_GKE_002: Workload Identity is enabled
[ ] GCP_GKE_003: GKE Control Plane has a private endpoint
[ ] Add/Generate tests

Codecov Report

Merging #21 (dd6a20a) into main (cab38d2) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   16.41%   16.41%
=======================================
  Files          10       10
  Lines         402      402
=======================================
  Hits           66       66
  Misses        333      333
  Partials        3        3

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


(chore): Update speedcrunch to allow persisting history

Settings still can't be persisted due to the hardlink overwritten by the app
Closes #11801

[x ] I have read the Contributing Guide.

/verify


hashProducer doesn't respect p.partitions

Need to use % instead of / to avoid returning partitions > p.partitions

Pull request seem to be stale, assigning @PragmaticCypher as the responsible developer.


SteamVR fails with error -202, compositor not fullscreen

The error says
Headset Error
Make sure your headset's display is in fullscreen mode

Which doesn't make sense to me since steamvr is using direct mode.
This doesn't happen with RADV.

The behavior has changed with Q4.1.
The error no longer occurs, but in the HMD all i can see is static white rectangles on a black background.
Should I close this issue and create a new one?

Please create a new issue if there are still other problems


Recent mechanize update breaks script

PixivDownloader2 version 20170220
https://github.com/Nandaka/PixivUtil2/releases
Donate at https://bit.ly/PixivUtilDonation
Reading /opt/pixiv-downloader-20170220/config.ini ...
done.
Traceback (most recent call last):
  File "PixivUtil2.py", line 1902, in <module>
    main()
  File "PixivUtil2.py", line 1779, in main
    __br__ = PixivBrowserFactory.getBrowser(config=__config__)
  File "/opt/pixiv-downloader-20170220/PixivBrowserFactory.py", line 515, in getBrowser
    _browser = PixivBrowser(defaultConfig, defaultCookieJar)
  File "/opt/pixiv-downloader-20170220/PixivBrowserFactory.py", line 35, in __init__
    mechanize.Browser.__init__(self, factory=mechanize.RobustFactory())
AttributeError: 'module' object has no attribute 'RobustFactory'

The script fails to start as per the error above. Tested with a fresh repository clone.
I run a Linux system and looking through my update log, I see that mechanize was recently updated from 0.2.5 to 0.3.1. Downgrading this package enabled the script to start up and log me in again. Haven't had a chance to test if the script can successfully download something after downgrading; there were a few other dependencies which recently received minor updates as well (numpy and imageio spring to mind).
Edit: Downloading still works after downgrading mechanize, so it's just mechanize that's the problem.

From their github:
Yes, as per the Changelog, mechanize.Browser does not accept the factory argument anymore. All HTML parsing is done using html5lib.

If you are using v3.x, you need to manually remove the  factory parameter from https://github.com/Nandaka/PixivUtil2/blob/master/PixivBrowserFactory.py#L35
I need to check the compatibiliy before I've update the libs.


My data factory has no package.json.

My data factory has no package.json after creating via the web authoring and Azure Repos.  Where is this package.json supposed to be?

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

ID: 126c00f8-e9d2-4af1-65b1-67a1e5516197
Version Independent ID: 30660b73-1e11-3e1f-f2c8-1daa094a410a
Content: Automated publishing for continuous integration and delivery - Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment-improvements.md
Service: data-factory
Sub-service: ci-cd
GitHub Login: @nabhishek
Microsoft Alias: abnarain

Thanks for the ask  . At this time we are reviewing the ask and we will reply back as soon as we can .

@kvsanders - My understanding is that you will have to create the package in the Devops with the below code mentioned in the article :
{
    "scripts":{
        "build":"node node_modules/@microsoft/azure-data-factory-utilities/lib/index"
    },
    "dependencies":{
        "@microsoft/azure-data-factory-utilities":"^0.1.5"
    }
}

Can you please help me in understanding if this is not your ask ?


Does NLog have flood protection?

Does NLog have flood protection?
There are various scenarios where the rate of events to be logged becomes too high.
Too high will, of course, depend on circumstances, but in most cases there will be a threshhold of system resource usage (CPU, memory, buses, disks, etc.) beyond which logging should not go. One nighmare scenario is when a piece of exception logging code throws an exception that it ends up trying to handle, resulting in an infinite, uncontrolled loop generating events at an extremely high rate.
I would kinda expect a logging package such as NLog to have some in-built protection mechanism against flooding, but I see nothing in the documentation.
If NLog doesn't have any flood protection, then I suggest it should become a feature request.
I've seen that, with the use of batches (and async logging?), events can be dropped. That's good, but there's a lot more to flood protection than that. It is not an easy or trivial piece of functionality to get right.

I could understand if the logging framework was the agressor that initiated the logging, then it would be natural that the logging framework was introducing the throttle-/backoff-algorithm.
But since the aggressive logging is just a sifde-effect of the actual issue, then I think the actual issue should be handled, instead of trying to protect against some of the symptoms. I guess other symptoms could be overloading service-communication-infrastructure (ex. cloud-network-traffic) or spinning up many instances (ex. cloud-cpu-usage), thus causing expensive bills from the cloud-provider (independent of the amount of logging).
I could also guess an evil loop could be created without involving any logging, that caused both infrastructure and cpu-usage to sky-rocket, and it would be impossible for logging-framework to prevent this, since not involved in the loop.

Closing as answered, due to inactivity.


Return the jobTs over edgeId as it is no-longer available.

After getting the error described in the issue and checking the montecarlo API docs.. there is no field edgeId to query.
The only suitable field appears to be jobTs.
Closes #28
Example

Checklist

[x] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[x] Includes tests or only affects documentation.
[x] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[x] Includes screenshots of documentation updates.

Run mkdocs serve to view documentation locally.

[x] Summarizes PR's changes in CHANGELOG.md

hi @rudeb0y - apologies for the delay here.
this looks good to me - could you update the changelog? then I can merge and push out a release

Hi - sorry I thought I had
Fixed

create_or_replace_lineage_edge now returns the jobTs field as edgeId is not available.

hey @rudeb0y - sorry i should have caught that we'd need to update the mocks as well
I went ahead and opened a PR that should supersede this one, so I'll close this
feel free to review (I did pretty much exactly what you did) here #31


Tests don't pass, babel-jscs released a breaking change with 5.7.0

Original post on the JSCS repo here: https://github.com/jscs-dev/node-jscs/issues/1567
When I run babel-jscs on my non es6 code, it fails while parsing every file.
Digging a little bit deeper...
index#attachComments is called with >900 tokens, and fails on this line:
} else if (firstComment.start < tokens[0].range[0]) {

If I print tokens[0], I get
{ type: 'String',
  value: 'use strict',
  start: 24,
  end: 36,
  loc: { start: { line: 3, column: 0 }, end: { line: 3, column: 12 } } }

It doesn't have a range, which is why it fails when trying to read index 0 from the undefined property.

It looks like babel-core updated from 5.6 to 5.7 and had breaking changes. Looking further...

Reading their github comparision between 5.6.14 and 5.7.0 is impossible because all the files were moved around. I'm not really sure how you want to proceed with this.

Since I'm working with babel-eslint as well I will probably see what breaks from that repo - @sebmck just changed a few things with the parser that may break some stuff so just need to make similar changes - this time related to the range property.

Thanks @hzoo. This is holding up the 2.0 release. So landing #19 seems like the best move for now.

Ok made 2.0.3 with fixes - I just pinned babel-core to ~5.8.3 for now although we could probably just do ^ like before


Add 2014 copyright notice

fixed by https://github.com/metamx/bytebuffer-collections/commit/0186bb3014d261a87ec4c697a31997e39a6b684c


Doubts clarification and possible documentation update

Hi, I played around with Java & Kotlin SDKs. They are a bit different. In Java you have GBFeaturesRepository which is responsible for fetching & refreshing features. Then you create a context and pass attributes.
Comparing to Kotlin SDK, there is no repository and only GrowthBookSDK has possibility to invoke refreshCache but it doesn't make sense as I need to create a new GrowthBookSDK so I will fetch features on my own and then pass it to the builder.
I experimenting with the SDK for a Kotlin server-side project.
Can you confirm there is no possibility to fetch features using your SDK? So basically do something similar as Java SDK allows.

Yep, it seems like GBFeaturesRepository should be added to Kotlin SDK

I experimenting with the SDK for a Kotlin server-side project.

You can also use the Java SDK with a Kotlin backend. Here is an example for Ktor.

Thanks all 👍  I ended up using Java SDK. Let's close the issue for now.


Does lazysizes also load display:none elements?

Hi, I have some elements that, in small screens don't have images, while in large screens they will.
So I would like lazysizes not to load images until the large breakpoint is reached.
Probably I will use display: none on elements inner images > will lazysizes respect the fact that images are hidden? Or will it load them anyway?
And/or, what's the best practice for achieving this result (loading images only on large breakpoints?)
Also, maybe i will need to do it with backgrounds instead of img tags, and thus I'll use bgset (for tech reasons, I cannot use object-fit for the moment).

will lazysizes respect the fact that images are hidden?

lazysizes will not load elements that are hidden with display: none so you can use this technique.

And/or, what's the best practice for achieving this result (loading images only on large breakpoints?)

The picture element

for tech reasons, I cannot use object-fit for the moment
Why? We have a polyfill for this!!!

And/or, what's the best practice for achieving this result (loading images only on large breakpoints?)
The picture element

And/or, what's the best practice for achieving this result (loading images only on large breakpoints?)
The picture element

Why do you think it's a best practice to use The picture element ?
In Firefox, the width and height given to the source element do not work.
With that in mind, if we can't use object-fit, we shouldn't use The picture element and apply display: none;, so my view is that it's on a case-by-case basis....


ParameterExtractingExpressionVisitor::GetValue

EFCore: master
Could you change
private object GetValue(Expression expression, out string parameterName)
to
protected virtual object GetValue(Expression expression, out string parameterName)
?
My Data Provider for EF needs in overriding of this method for preparing (transforming) expression before local evaluation.
Thanks.

Can you describe in detail why you need to override that method?

I already wrote here about this :)
For equal calculation of expression at server and at client side.
For example, string.Substring not allows startIndex>=length_of_sting, but DBMS allows and returns empty string.
DBMS allows calculation the length of NULL-string, string.Length - no.
And other changes.
I remap "problem" nodes of expression (in overrided GetValue) to my functions, which execute by DBMS-rules.
 protected override object GetValue(Expression expression,
                                    out string parameterName)
 {
#if TRACE
  Core.Core_Trace.Method
   ("LcpiOleDb__ParameterExtractingExpressionVisitor::GetValue\n"
   +"  expression: {0}",
    expression);
#endif

  if(!Object.ReferenceEquals(expression,null))
  {
   var prepareSvc
    =Core.Engines.EngineSvcUtils.QuerySvc<Core.Engines.EngineSvc__ExpressionVisitor>
      (m_CnOptions,
       Core.Engines.EngineSvcID.EngineSvc__ExpressionTree__PrepareForLocalEvaluation);

   expression
    =prepareSvc.Visit(expression);

   Debug.Assert(!Object.ReferenceEquals(expression,null));
  }//if

  return base.GetValue
          (expression,
           out parameterName);
 }//GetValue

Hello,
When I write the linq-expression, I expect that all it parts will be executed with equal rules.
For example:
//r.VarCharCol1 (string) contains "qwerty"
//r.VarCharCol2 (string) contains null
string vv1="qwerty";
string vv2=null;
var recs=db.testTable.Where(r => (vv1+vv2).Length==null && (r.VarCharCol1+r.VarCharCol2).Length==null);

For me, "vv1+vv2" and "r.VarCharCol1+r.VarCharCol2" - must returns equal result: NULL.

Though, I am not able to understand how above differences requires interaction from provider side in ParameterExtraction. Can you write query and what is current parameterization and what is parameterization you are trying to get and why?

string vv1="qwerty";
string vv2=null;
var recs=db.testTable.Where(r => (vv1+vv2).Length==null && r.TEST_ID==testID);

By default, EFCore calculates "(vv1+vv2)==null" as FALSE. As result, this query always returns zero number of records.
By DBMS rules "(vv1+vv2)==null" is TRUE. As result, this query returns records.
Evaluation (vv1+vv2) performed in ParameterExtractingExpressionVisitor::GetValue.

I may copy code of ParameterExtractingExpressionVisitor to my provider. But I try to minimize similar imports.

Based on definition string.Length returns integer type so comparing SomeString.Length == null is always going to be false. It is true that in DBMS vv1 + vv2 will give null as result because one of the argument is null, but at the same time vv1 + vv2 = null is also null result. We need to generate (vv1 + vv2) is NULL to actually get true result. That is what relational null semantics do in EF Core. Database would generate NULL result for any null argument which C# wouldn't hence we generate special IS NULL statements to compensate for that. With having whole pipeline to deal with that, our philosophy has been that we would actually try to match C# semantics rather than database. And compensate for that when generating SQL. Trying to dictate that parameterization should work like database is counter-intuitive to that philosophy, confusing to users and creates more mismatch in server eval vs client eval.

@smitpatel
Ok, I will continue to use modified sources of EFCore.
At finish (if i get to him) I will return to this problem :)

Triage: no decision made about behavior or implementation at this stage, but leaving this open to consider in the context of query guiding principals for 3.0 as tracked by #12795


Labels Wrap not working as expected in doughnut charts

Expected Behavior
Labels should come in newline
Current Behavior
Labels are separated in ","
Steps to Reproduce
var ctx = document.getElementById("myChart").getContext("2d");
var doughnut = {
  type: "doughnut",
  options: {
    responsive: true,
    responsiveAnimationDuration: 0.0,
    aspectRatio: 1.4,
    maintainAspectRatio: true,
    title: {
      display: true,
      position: 2,
      fontSize: 50.0,
      fontColor: "#666",
      fontStyle: "bold",
      padding: 20.0,
      lineHeight: 1.2,
      text: "Yesterday Cost Analysis",
    },
    animation: { Duration: 0.0 },
    layout: { Padding: { Left: 0.0, Right: 0.0, Top: 0.0, Bottom: 0.0 } },
    legend: {
      display: true,
      position: "bottom",
      align: "center",
      fullWidth: true,
      reverse: false,
      labels: {
        boxWidth: 80.0,
        fontSize: 25.0,
        fontStyle: "bold",
        fontColor: "#666",
        padding: 50.0,
      },
    },
    scales: null,
  },
  data: {
    labels: [
      ["Compute", "Cloud", "Compute"],
      ["Load", "Balancing"],
      ["Relational", "Database", "Service"],
    ],
    datasets: [
      {
        label: "",
        data: [61.13, 0.0, 0.0],
        backgroundColor: [
          "rgba(255, 99, 132, 0.8)",
          "rgba(255, 206, 86, 0.8)",
          "rgba(54, 162, 235, 0.8)",
          "rgba(75, 192, 192, 0.8)",
          "rgba(153, 102, 255, 0.8)",
          "rgba(255, 159, 64, 0.8)",
        ],
        borderColor: [
          "rgba(255, 99, 132, 0.8)",
          "rgba(255, 206, 86, 0.8)",
          "rgba(54, 162, 235, 0.8)",
          "rgba(75, 192, 192, 0.8)",
          "rgba(153, 102, 255, 0.8)",
          "rgba(255, 159, 64, 0.8)",
        ],
        borderWidth: 1.0,
      },
    ],
  },
};

var chart = new Chart(ctx, doughnut);

Context

Environment

Chart.js version: 2.9.3
Browser name and version: edgium  86.0.622.38

This is the output
however if I change chart type to chart to bar, then it works as expected

@ankur198 the multiline label is available on axis (like in the example of bar chart) but not in the legend, afaik.
If you want to remove commas, maybe you can use generateLabels callback to labels node of legend(works for doughnut chart):
 generateLabels: function(chart){
     var items = Chart.defaults.doughnut.legend.labels.generateLabels.call(this, chart);
     items.forEach(item => item.text = chart.data.labels[item.index].join(' '));
     return items;
 }

CodePen: https://codepen.io/stockinail/pen/ExygqpJ

Thanks @stockiNail for responding. Will close this issue.


Add variant field to OrderLine type

Order details page in saleor-storefront relies on variant field in the OrderLine type, which must have been removed recently. This PR adds this field back to the schema.
Screenshots

Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] Database migration files are up to date.
[x] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[x] GraphQL schema and type definitions are up to date.
[x] Changes are mentioned in the changelog.

As mentioned in https://github.com/mirumee/saleor/issues/3760, the only issue with Orderline's variant is it maybe deleted after order is created.


fullscreen

hi
i'm use from easy-video-player
call player into fragment , but no work full screen

player.setAutoFullscreen(true);

Hi, Yes am also facing this issue.


Logging mysqls and pqsqls queries

In api.db.mysql.js and api.db.pqsql.js all queries are logged to debug log.
    connection.on('query', (err, res, fields, query) => {
      ...
      application.log.debug(query.sql);
    });

Just not removed after the tests? Or for what this feature in production?

 But you can disable it just removing debug in config/log.js

Ok, thanx.


redeem p2pkh adddress balance

Hi I have a bitcoin address redeem script p2pkh, I don know how to go about redeeming the unspend balance from my address, I have read into the process, but still don/t understand, can send you my transaction   Fee
0.00019447 BTC
(70.206 sat/B - 25.388 sat/WU - 277 bytes)
(101.286 sat/vByte - 192 virtual bytes)
0.14669553 BTC
Hash
f0ff6231e0a9ba3fc5d0ae5f618850e0a57a161fd69007bfbdfba6e1c6b33e5c
2020-05-11 10:56
3GmS7i3Pm8xA4Jb9fwYCbT7UkJWDuNiWAX
0.14689000 BTC
bc1qk7h6nq2j4whztak03puk05t9zh85aup330gu4utal5n0d5k25usq0zwgh3
0.06567816 BTC
1D97VBoikYqWzt29AZ3jXeYqstYnq1CyhJ
0.08101737 BTC
This transaction was first broadcast to the Bitcoin network on May 11, 2020 at 10:56 AM MST.  The transaction currently has 123,375 confirmations on the network.  At the time of this transaction, 0.14669553 BTC was sent with a value of $1,246.08. The current value of this transaction is now $3,133.21.  Learn more about how transactions work.
Details
Hash
f0ff6231e0a9ba3fc5d0ae5f618850e0a57a161fd69007bfbdfba6e1c6b33e5c
Status
Confirmed
Received Time
2020-05-11 10:56
Size
277 bytes
Weight
766
Included in Block
629993
Confirmations
123,375
Total Input
0.14689000 BTC
Total Output
0.14669553 BTC
Fees
0.00019447 BTC
Fee per byte
70.206 sat/B
Fee per vbyte
101.286 sat/vByte
Fee per weight unit
25.388 sat/WU
Value when transacted
$1,246.08
Inputs
HEX
ASM
Index
0
Details
Output
Address
3GmS7i3Pm8xA4Jb9fwYCbT7UkJWDuNiWAX
Value
0.14689000 BTC
Pkscript
OP_HASH160
a55fbed7d8fe42bb11b66f9ae9bfe9ebfcbe3dfa
OP_EQUAL
Sigscript
0020abde5e3877501ef34dcda0f42aa3f52673b395a866ad61f3660d2d036a7c5fc4
Witness
3044022001843fae362c47df2a1f678ae96c5c559c50ea2a110fd3dc2a0f8fd801afa0210220225e61b9d0340b516d54d454b838204fa9e34d75547f7e5a7b468eb6186fb9c501
5121035bef36265999cb6534ebfc7b54c334faaf50f54a2a46d89703f51468d9674e8551ae
Outputs
Index
0
Details
Spent
Address
bc1qk7h6nq2j4whztak03puk05t9zh85aup330gu4utal5n0d5k25usq0zwgh3
Value
0.06567816 BTC
Pkscript
OP_0
b7afa98152abae25f6cf887967d16515cf4ef0318bd1caf17dfd26f6d2caa720
Index
1
Details
Unspent
Address
1D97VBoikYqWzt29AZ3jXeYqstYnq1CyhJ
Value
0.08101737 BTC
Pkscript
OP_DUP
OP_HASH160
852a405633df80b64299a23284d9bf1ec7dd0d75
OP_EQUALVERIFY
OP_CHECKSIG
alternating coins
You’ve thought a

I am having a lot of trouble understanding what the issue. An example of how to spend a P2PKH can be found here:
https://github.com/karask/python-bitcoin-utils/blob/master/examples/p2pkh_transaction.py


Implement REST API config file to support multi-tenancy: restapi_config.json

At the moment user credentials are set using the command line flags: --user and --password. I would like to implement an additional configuration file: /opt/hoster-core/config/restapi_config.json, that will be parsed during the hoster api start execution, and if it exists - will override the command line flags.
I will also implement password hashing in order to protect the REST API server from credential leaks.
This will make the system more robust, and will allow Hoster users to have multiple REST API cluster admins (without sharing a common username/password): a separate HA (--ha-mode) user, a separate Hoster Control Center user, etc. As a side effect it will allow cluster admins to specify the HTTPs/SSL certificate paths, and some other settings.

Config file has been implemented and will be tested shortly. All command line switches have been removed (like --password, --port, etc), to keep the configuration universal.
In case anything else pops up, I'll open a new issue.


How to parse multi object json from file?

https://github.com/nlohmann/json/issues/367

In order to asses your issue, we need the following information:

What is the issue you have?

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

What is the expected behavior?

And what is the actual behavior instead?

Which compiler and operating system are you using? Is it a supported compiler?

Did you use a released version of the library or the version from the develop branch?

If you experience a compilation error: can you compile and run the unit tests?

Each call of fs >> j; lead to reading one JSON from file and leaving stream at where the parsing has ended. When the parser is called for the third time, there is no JSON available in the rest of file contents and that's causes an exception as there is no valid JSON.
I believe this is an expected behavior. This is basically same as having an input file containing only white-spaces. Your solution to continue reading and break on exception seems fine. Is there any reason to avoid it?
One other solution is to manually skip over the whitespace characters until you reach an non-space character or end of file.


[AutoDiff] Diagnose duplicate @differentiable attributes with same parameters.

@differentiable attributes are uniqued by their wrt-parameters. Reject duplicates.
Resolves TF-163.

@swift-ci please test tensorflow

@swift-ci please test tensorflow Linux

@swift-ci please test tensorflow Linux


How to configure DNS records?

Hello.
Is the domain supposed to use the server as its NS, or just point d.domain.com to the IP? I'm a bit confused because it seems as if I should point NS to the server IP, but the install guide doesn't mention configuring a zone file.

Hello,
thanks for using my little side project. I can describe the setup I use for pboehm.de, which should be applicable for your case. pboehm.de is registered at DomainFactory which supports nameserver delegation for subdomains. I have created an A-record dns.pboehm.de pointing to the VM ddns is running on. Then I have created an NS-record for d.pboehm.de pointing to dns.pboehm.de. This delegates the subdomain d.pboehm.de to my ddns instance which handles this subdomain and answers DNS-queries for hosts in this domain.
The backend of ddns is then executed with the following parameters:
ddns --soa_fqdn=dns.pboehm.de --domain=d.pboehm.de backend

I hope this helps you setting up ddns.

Thank you kindly for the quick reply!


F-Droid build crash

v5.9.0 has some kind of build error.
Something in build gradle about File! being deprecated in Java.

Hm, looks like it is trying to compile with Java 11 but TS uses Java 17. I'll investigate

Here's the issue on F-Droid to track it: https://gitlab.com/fdroid/fdroiddata/-/issues/3222

Looks like F-Droid is in the process of upgrading their servers and build failures are expected right now. I don't know what their timeline is but they said to be patient and the build will be retried once their upgrade is complete: https://gitlab.com/fdroid/fdroiddata/-/issues/3222#note_1828212755

The build has succeeded now, just needs to be indexed by F-Droid to appear in their repo (the normal process). So I think this will be all set. If you don't see the build within a week, let me know (I'll also keep an eye on it).


Add new-contributors.md

Small document to provide some introductionlinks to related projects.

Thanks updated

Updated.

Sigh - Now even with fixed indentation.


MM-10095: Fix for invalid wildcard suffix pattern.

Summary
Should only match * character when it's preceded by a word character.
Ticket Link
MM-10095
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)

I may actually change the regex used here and add some more test so please hold off on review.


Add instructions on how to deploy the api (copied from onflow/wallet-api)

From @hichana

I realize the wallet-api is not production-ready yet, but it would be great to have some instructions on how to deploy the api on a cloud provider like GCP. The wallet-api seems pretty powerful so for some it might be wise to start preparing for using it in a real-world app.
Also, how where in the repo is the actual code that starts the server to run the api? I'm not familiar with docker so maybe this is what I'm missing here. I guess the docker image has all of this and is building and running it? Some help/guidance in the readme for this would be appreciated too. Thanks!

some instructions to deploy on Kubernetes, or even something easier (if possible) like Google Cloud Run would be nice

From @psiemens:

Thanks for the feedback, @hichana! I agree that there should be instructions for how to deploy to a cloud environment.

Closing as duplicate of #150