 To use CUDA with multiprocessing, you must use the 'spawn' start method
(VllmWorkerProcess pid=13978) Process VllmWorkerProcess:
(VllmWorkerProcess pid=13978) Traceback (most recent call last):
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
(VllmWorkerProcess pid=13978)     self.run()
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/multiprocessing/process.py", line 108, in run
(VllmWorkerProcess pid=13978)     self._target(*self._args, **self._kwargs)
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_worker_utils.py", line 211, in _run_worker_process
(VllmWorkerProcess pid=13978)     worker = worker_factory()
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/executor/gpu_executor.py", line 68, in _create_worker
(VllmWorkerProcess pid=13978)     wrapper.init_worker(**self._get_worker_kwargs(local_rank, rank,
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/worker/worker_base.py", line 311, in init_worker
(VllmWorkerProcess pid=13978)     self.worker = worker_class(*args, **kwargs)
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/worker/worker.py", line 87, in init
(VllmWorkerProcess pid=13978)     self.model_runner: GPUModelRunnerBase = ModelRunnerClass(
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/worker/model_runner.py", line 196, in init
(VllmWorkerProcess pid=13978)     self.attn_backend = get_attn_backend(
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/attention/selector.py", line 51, in get_attn_backend
(VllmWorkerProcess pid=13978)     backend = which_attn_to_use(num_heads, head_size, num_kv_heads,
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/vllm/attention/selector.py", line 158, in which_attn_to_use
(VllmWorkerProcess pid=13978)     if torch.cuda.get_device_capability()[0] < 8:
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/init.py", line 430, in get_device_capability
(VllmWorkerProcess pid=13978)     prop = get_device_properties(device)
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/init.py", line 444, in get_device_properties
(VllmWorkerProcess pid=13978)     _lazy_init()  # will define _get_device_properties
(VllmWorkerProcess pid=13978)   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/init.py", line 279, in _lazy_init
(VllmWorkerProcess pid=13978)     raise RuntimeError(
(VllmWorkerProcess pid=13978) RuntimeError: Cannot re-initialize CUDA in forked subprocess. To use CUDA with multiprocessing, you must use the 'spawn' start method
(VllmWorkerProcess pid=13979) Process VllmWorkerProcess:
(VllmWorkerProcess pid=13979) Traceback (most recent call last):
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
(VllmWorkerProcess pid=13979)     self.run()
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/multiprocessing/process.py", line 108, in run
(VllmWorkerProcess pid=13979)     self._target(*self._args, **self._kwargs)
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_worker_utils.py", line 211, in _run_worker_process
(VllmWorkerProcess pid=13979)     worker = worker_factory()
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/executor/gpu_executor.py", line 68, in _create_worker
(VllmWorkerProcess pid=13979)     wrapper.init_worker(**self._get_worker_kwargs(local_rank, rank,
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/worker/worker_base.py", line 311, in init_worker
(VllmWorkerProcess pid=13979)     self.worker = worker_class(*args, **kwargs)
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/worker/worker.py", line 87, in init
(VllmWorkerProcess pid=13979)     self.model_runner: GPUModelRunnerBase = ModelRunnerClass(
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/worker/model_runner.py", line 196, in init
(VllmWorkerProcess pid=13979)     self.attn_backend = get_attn_backend(
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/attention/selector.py", line 51, in get_attn_backend
(VllmWorkerProcess pid=13979)     backend = which_attn_to_use(num_heads, head_size, num_kv_heads,
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/vllm/attention/selector.py", line 158, in which_attn_to_use
(VllmWorkerProcess pid=13979)     if torch.cuda.get_device_capability()[0] < 8:
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/init.py", line 430, in get_device_capability
(VllmWorkerProcess pid=13979)     prop = get_device_properties(device)
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/init.py", line 444, in get_device_properties
(VllmWorkerProcess pid=13979)     _lazy_init()  # will define _get_device_properties
(VllmWorkerProcess pid=13979)   File "/opt/conda/lib/python3.8/site-packages/torch/cuda/init.py", line 279, in _lazy_init
(VllmWorkerProcess pid=13979)     raise RuntimeError(
(VllmWorkerProcess pid=13979) RuntimeError: Cannot re-initialize CUDA in forked subprocess. To use CUDA with multiprocessing, you must use the 'spawn' start method
Traceback (most recent call last):
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_worker_utils.py", line 170, in _enqueue_task
self._task_queue.put((task_id, method, args, kwargs))
File "/opt/conda/lib/python3.8/multiprocessing/queues.py", line 82, in put
raise ValueError(f"Queue {self!r} is closed")
ValueError: Queue <multiprocessing.queues.Queue object at 0x7feceed0fbb0> is closed
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/ossfs/workspace/inter_vllm_long_length.py", line 94, in
test_model(path)
File "/ossfs/workspace/inter_vllm_long_length.py", line 64, in test_model
llm = LLM(model=model_path, **kwargs_launcher)
File "/opt/conda/lib/python3.8/site-packages/vllm/entrypoints/llm.py", line 144, in init
self.llm_engine = LLMEngine.from_engine_args(
File "/opt/conda/lib/python3.8/site-packages/vllm/engine/llm_engine.py", line 409, in from_engine_args
engine = cls(
File "/opt/conda/lib/python3.8/site-packages/vllm/engine/llm_engine.py", line 242, in init
self.model_executor = executor_class(
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/distributed_gpu_executor.py", line 25, in init
super().init(*args, **kwargs)
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/executor_base.py", line 41, in init
self._init_executor()
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_gpu_executor.py", line 70, in _init_executor
self._run_workers("init_device")
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_gpu_executor.py", line 112, in _run_workers
worker_outputs = [
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_gpu_executor.py", line 113, in
worker.execute_method(method, *args, **kwargs)
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_worker_utils.py", line 177, in execute_method
self._enqueue_task(future, method, args, kwargs)
File "/opt/conda/lib/python3.8/site-packages/vllm/executor/multiproc_worker_utils.py", line 173, in _enqueue_task
raise ChildProcessError("worker died") from e
ChildProcessError: worker died
Steps to reproduce
No response
Expected Behavior
No response
Logs
No response
Additional Information
No response

Close due to duplicate content with #62.


when running interactive.py occur the following error

_tkinter.TclError: no display name and no $DISPLAY environment variable

I have the same problem. Have you solved it？


SFP does not work on Windows 8.1

I think I managed to follow the installation instructions but when I open the SFP_UI.exe, there are SFP.log and SFP.config files appearing but the UI isn't opening (should it?). And when I go to the tray to look for the icon, it's there for a while but until I move my mouse towards it, the icon disappears.
My OS is Windows 8.1 and Steam version is the latest one available. I am wondering if the issue is related to my OS not being Windows 10 or 11 but let's hope that's not the case.
The log file looks like the following:
2023-06-22 12:49:41.3752|INFO|SFP.Models.Log|Initializing SFP version 0.0.47 on platform win81-x64
2023-06-22 12:49:41.6650|DEBUG|SFP.Models.Log|SelectedSkin is steamui.
2023-06-22 12:49:41.8465|INFO|SFP.Models.Log|Monitoring Steam state
2023-06-22 12:49:41.8465|ERROR|SFP.Models.Log|System.EntryPointNotFoundException: Unable to find an entry point named 'AdjustWindowRectExForDpi' in DLL 'user32.dll'.
   at FluentAvalonia.Interop.Win32Interop.AdjustWindowRectExForDpi(RECT* lpRect, Int32 dwStyle, BOOL bMenu, Int32 dwExStyle, Int32 dpi)
   at FluentAvalonia.UI.Windowing.Win32WindowManager.HandleNCCALCSIZE(WPARAM wParam, LPARAM lParam)
   at FluentAvalonia.UI.Windowing.Win32WindowManager.WndProc(HWND hWnd, UInt32 msg, WPARAM wParam, LPARAM lParam)
   at FluentAvalonia.UI.Windowing.Win32WindowManager.WndProcStatic(IntPtr hwnd, UInt32 msg, IntPtr wParam, IntPtr lParam)

Windows 10 is the minimum Windows version supported by Microsoft, and by the FluentAvalonia library this app depends on.
If anyone would like to submit pull requests fixing issues on older operating systems I'd be happy to accept them, but I will not be providing support myself.

As expected, SFP UI won't work with Windows 7 or 8.1, but SFP skinning itself will still work just fine.


Improved ergonomics for shallow routing

Following up from some feedback related to the #1152 PR, As we explore the ideas around sub-routes, etc — we should re-consider the ergonomics to make this easier.

You can actually accomplish this without us needing a new API:

Manually call setServerProps and use an alternate path, like _search or _filter
Call window.history.pushState() in your client component to update the URL. This does not affect scroll position.
In your server component, look for both params (real search and alternate path). This allows you to present the correct filter values for both initial page loads and realtime RSC updates.

The use case I've seen most often is where merchants will have multiple related products under one "product group" (e.g. /products/tshirt with multiple products pulled in, instead of /products/tshirt-blue, etc…).
The colour swatches are then essentially a navigation — but typically you only want to change part of the page … typically the product images — and for SEO reasons you don't want to split the routes (so … /products/tshirt?color=blue).
Also, in standard liquid themes, any time you change a variant the URL is updated with the ?variant=VARIANT_ID. We should certainly make it easy (and in the demo store, the default) to adopt this behaviour so backlinks work as expected.
It feels like this should either be in the useNavigate or useRouteParams hooks.

What we actually want is to be able to get a fresh RSC response in addition to updating the URL in a way that does not trigger a page scroll.
I think a useRefresh API would probably be our best bet here. The one that exists today in rsc.ts is not working for me, but I bet we could make it work.
Then we could make this sort of use case easier with either a custom hook or by augmenting an existing hook above, e.g. useNavigate().updateParamsAndRscPlease().

What we actually want is to be able to get a fresh RSC response in addition to updating the URL in a way that does not trigger a page scroll.
I think a useRefresh API would probably be our best bet here. The one that exists today in rsc.ts is not working for me, but I bet we could make it work.
Then we could make this sort of use case easier with either a custom hook or by augmenting an existing hook above, e.g. useNavigate().updateParamsAndRscPlease().


Cannot read property 'get' of undefined (cookie)

After installed your module, i had got a fatal error "Cannot read property 'get' of undefined" with getToken method in .nuxt/strapi.js line 148 on all pages.
 getToken () {     return this.$cookies.get(TOKEN_KEY)   }
I don't understand why do you retrieve token in constructor because to register we don't need a jwt-token.
Version
@nuxtjs/strapi: 0.1.7
nuxt: 2.14.6

In fact, it don't found $cookies...

You can use  this.$strapi.$cookies (https://strapi.nuxtjs.org/strapi#cookies)

Not fixed.

@dimst  I had to change the order or the modules I was using, I put strapi first


Add videosink display driver

The videosink package provides a display driver implementing an HTTP
request handler. Client requests get an initial snapshot of the graphics
buffer and are updated further on every change.
Provides the basic implementation for #26. Still to be done are example code and periph/cmd integration.

Codecov Report

Merging #27 (79d06e3) into main (d92a4ed) will increase coverage by 0.8%.
The diff coverage is 87.8%.

@@           Coverage Diff           @@
##            main     #27     +/-   ##
=======================================
+ Coverage   60.3%   61.2%   +0.8%
=======================================
  Files         48      53      +5
  Lines       6453    6666    +213
=======================================
+ Hits        3893    4078    +185
- Misses      2425    2447     +22
- Partials     135     141      +6

Impacted Files
Coverage Δ

videosink/handler.go
83.0% <83.0%> (ø)

videosink/display.go
84.4% <84.4%> (ø)

videosink/writer.go
93.8% <93.8%> (ø)

videosink/encoder.go
100.0% <100.0%> (ø)

videosink/format.go
100.0% <100.0%> (ø)

ads1x15/ads1x15.go
82.5% <0.0%> (-0.7%)
:arrow_down:

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 d92a4ed...79d06e3. Read the comment docs.

I decided to merge, let's continue the discussion if we want to rename things a bit before next release. Even then it's not a big deal if we break it while there isn't users yet.


fix: make redirect option work

I can only assume this is what was intended

I confirmed locally things work after this change and didn't work before the change when config.redirect is set.

Thanks for merging. Do you plan in publishing a new version to npm soon?

yes, will do soon. I'm getting stuck with npm 2FA, I've just enabled it and don't know how to deal with the publishing

Sounds good, don't rush on my account, I have a workaround in place for now and will check every few weeks to try to pull a newer version from npm. Thanks!


Pickling hangs scalac phase typer

Using IntelliJ IDEA 14.1.2 and scala plugin 1.4.15 with scala 2.11.5:
Information:Using javac 1.8.0_05 to compile java sources
Information:24/04/2015 19:12 - Compilation completed with 1 error and 0 warnings in 20h 0m 56s 73ms
Error:scalac: Error: GC overhead limit exceeded
java.lang.OutOfMemoryError: GC overhead limit exceeded

The phase typer hangs while processing a class that pickles an object graph to JSON. I'm relying on automatically generated picklers. The object graph starts with a WrappedArray[UnsealedTrait] and the instances contain values of various types including Array[Any].
I'm using the latest code from Git (commit e7249135ab0821c3c144b09ae9ffd52f996bdd2c). Note that scala-pickling 0.10.0 compiles quickly but loses most of the object graph on pickling. I couldn't pull the 0.10.1-SNAPSHOT from Sonatype and https://oss.sonatype.org/content/repositories/snapshots/org/scala-lang/modules/scala-pickling_2.11/ appears to be empty.
I have configured the Scala compile server with 6144MB and the Scala compiler with 4GB using the command line option -Jmx4G.
My project was originally converted to Scala using the IntelliJ Java to Scala conversion wizard. It contains about 19000 lines (including comments and blank lines) in 255 Scala files.
The compiler continued to run flat out (on up to 8 "cores") even after the error.
I've spent a lot of time tweaking my classes to be more Scala-friendly but it doesn't seem to have helped. Should pickling to JSON be able to deal with my use case?
Thank you

Should pickling to JSON be able to deal with my use case?

Without reproducible tests, it's hard to answer this. One thing you might try is try using it first Java 7 and without IntelliJ. Also I want to clarify that your pickler code compiled fine, and that you're seeing OutOfMemoryError while pickling some data.
We have seen OutOfMemoryError that I suspect is caused by Pickling too, and hopefully we can investigate more on it in the future.

The OutOfMemoryError occurred after 20 hours. Despite the message the pickler code didn't finish compiling. The compiler continued running at between 600% and 800% of CPU (where 100% is one hyper thread running flat out) until I killed it.
I'll see if I get better results with Java 7 and post the result. Thank you.

The code needed a few changes in order to compile and run on Java 7 from a shell script. These changes were: 1. remove dependencies on Java files in the project. 2. Increase the heap memory (8G was enough). 3. Solved runtime errors by removing the private modifier on two val members of a class to allow an Unpickler to be generated.
Incremental compilation via the IDE is reasonable after the first time, however initial compilation is still slow and so is full compilation via a shell script (1974 seconds). Using a shell script scalac -verbose seemed to spend a long time on org.postgis.Geometry.
A lot of the members are either lost or pickled to abstract classes or 0 but I think these relate to known issues with Map, Array, Byte and Java classes.

The answer to my question seems to be "No, scala-pickling to JSON does not currently support your use case. You might have more success by writing custom picklers for the cases where the generated picklers fail."


Bugfix/analyzers web UI

Scope & Purpose
Fixes BTS-608 and BTS-708

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[x] Backport for 3.9: https://github.com/arangodb/arangodb/pull/16558
[ ] Backport for 3.8: (Please link PR)
[ ] Backport for 3.7: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[x] GitHub issue / Jira ticket: See PR description.
[ ] Design document:

Unrelated failure in Jenkins.


[REEF-637] Remove deprecated methods of NCS and mark as @Unstable

This addressed the issue by:

Annotate NetworkConnectionService interface as @Unstable
Remove NetworkConnectionService deprecated methods declarations:
registerId(), unregisterId(), registerConnectionFactory(), and getNetworkConnectionServiceId()
Remove corresponding NetworkConnectionServiceImpl deprecated methods
Remove deprecated "myId" field from NetworkConnectionServiceImpl
Remove deprecated getSrcId() from NetworkConnectionFactory
Substitute calls to getSrcId() with getLocalEndPointID() in NetworkConnection
Remove UnbindNetworkConnectionService.
Rationale: (1) it was deprecated (2) it was not used anywhere in the project
Remove BindNetworkConnectionService.
Rationale: (1) it was deprecated (2) it was not used anywhere in the project
Remove unused imports from NetworkConnectionService

JIRA:
[REEF-637] (https://issues.apache.org/jira/browse/REEF-637)

Sadly, this is a -1 as per the discussion on the JIRA. We should do this after 0.13 is done.
@zerg-zerg Can you please close this PR?


Remove the Require on the socket for the rpm

Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

LGTM. Before this fix, installing rpm on RHEL fails to start docker.service. After this fix, it starts fine.

Before this fix, installs on CentOS7 fail to start with error:
Failed to start docker.service: Unit docker.socket failed to load: No such file or directory

@ernestgwilsonii yes, should be all good after this fix.  thanks for checking

same issue.
what should i do?

@daikaixian this is merged for 1.12.0 which will be out on Thursday.

docker-engine-1.12.0-1.el7.centos.x86_64 still has this issue in it. Could we get a fix?

@bmoo did your unit file contain changes? In that case it's possible that yum/dnf didn't replace it with the unit-file from 1.12

Oops. I was overwriting the file with an ansible script. The RPM is fine

Thanks @bmoo !

Sorry if this is the wrong place, but I notice that docker-machine (v0.8.0) create with the generic driver running against a Centos 7 host is adding back the "Requires=docker.socket" flag...
About to run SSH command:
printf %s "[Unit]
Description=Docker Application Container Engine
After=network.target docker.socket
Requires=docker.socket

[Service]
ExecStart=/usr/bin/docker daemon -H tcp://<IP_ADDRESS>:2376 -H unix:///var/run/docker.sock --storage-driver devicemapper --tlsverify --tlscacert /etc/docker/ca.pem --tlscert /etc/docker/server.pem --tlskey /etc/docker/server-key.pem --label provider=generic
MountFlags=slave
LimitNOFILE=1048576
LimitNPROC=1048576
LimitCORE=infinity
Environment=
" | sudo tee /etc/systemd/system/docker.service

@djleeam can you open an issue in the docker machine issue tracker, and refer to this issue? https://github.com/docker/machine/issues

Might be best to just piggyback on https://github.com/docker/machine/issues/2795#issuecomment-236156423 which I think was originally a slightly different issue but is still open and where I've been tracking this.


java.lang.NoClassDefFoundError: org/slf4j/LoggerFactory (how to add dependency to fatjar)

Describe what you would like to know or do
I'm trying to add the websocket client to a minecraft mod, but for some reason my implementation of the fatjar doesn't add it no matter what I do
Describe the solution you'd considered
multiple different ways of adding a library to a fatjar, both compile and implementation
Additional context
Crash report:
`---- Minecraft Crash Report ----
// Uh... Did I do that?
Time: 9/24/24 1:47 PM
Description: Exception in server tick loop
net.minecraftforge.fml.common.LoaderExceptionModCrash: Caught exception from Advanced Base Security (advancedbasesecurity)
Caused by: java.lang.NoClassDefFoundError: org/slf4j/LoggerFactory
at org.java_websocket.drafts.Draft_6455.(Draft_6455.java:113)
at org.java_websocket.drafts.Draft_6455.(Draft_6455.java:210)
at org.java_websocket.drafts.Draft_6455.(Draft_6455.java:198)
at org.java_websocket.drafts.Draft_6455.(Draft_6455.java:178)
at org.java_websocket.client.WebSocketClient.(WebSocketClient.java:184)
at com.cadergator10.advancedbasesecurity.util.WebsocketHandler.(WebsocketHandler.java:37)
at com.cadergator10.advancedbasesecurity.AdvBaseSecurity.init(AdvBaseSecurity.java:53)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraftforge.fml.common.FMLModContainer.handleModStateEvent(FMLModContainer.java:637)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:91)
at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:150)
at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:76)
at com.google.common.util.concurrent.MoreExecutors$DirectExecutor.execute(MoreExecutors.java:399)
at com.google.common.eventbus.Subscriber.dispatchEvent(Subscriber.java:71)
at com.google.common.eventbus.Dispatcher$PerThreadQueuedDispatcher.dispatch(Dispatcher.java:116)
at com.google.common.eventbus.EventBus.post(EventBus.java:217)
at net.minecraftforge.fml.common.LoadController.sendEventToModContainer(LoadController.java:219)
at net.minecraftforge.fml.common.LoadController.propogateStateMessage(LoadController.java:197)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:91)
at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:150)
at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:76)
at com.google.common.util.concurrent.MoreExecutors$DirectExecutor.execute(MoreExecutors.java:399)
at com.google.common.eventbus.Subscriber.dispatchEvent(Subscriber.java:71)
at com.google.common.eventbus.Dispatcher$PerThreadQueuedDispatcher.dispatch(Dispatcher.java:116)
at com.google.common.eventbus.EventBus.post(EventBus.java:217)
at net.minecraftforge.fml.common.LoadController.distributeStateMessage(LoadController.java:136)
at net.minecraftforge.fml.common.Loader.initializeMods(Loader.java:749)
at net.minecraftforge.fml.server.FMLServerHandler.finishServerLoading(FMLServerHandler.java:108)
at net.minecraftforge.fml.common.FMLCommonHandler.onServerStarted(FMLCommonHandler.java:338)
at net.minecraft.server.dedicated.DedicatedServer.func_71197_b(DedicatedServer.java:219)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:486)
at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.ClassNotFoundException: org.slf4j.LoggerFactory
at net.minecraft.launchwrapper.LaunchClassLoader.findClass(LaunchClassLoader.java:101)
at java.lang.ClassLoader.loadClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
... 43 more
A detailed walkthrough of the error, its code path and all known details is as follows:
-- System Details --
Details:
Minecraft Version: 1.12.2
Operating System: Windows 11 (amd64) version 10.0
Java Version: 1.8.0_401, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory:<PHONE_NUMBER> bytes (1060 MB) /<PHONE_NUMBER> bytes (1203 MB) up to<PHONE_NUMBER> bytes (7227 MB)
JVM Flags: 0 total;
IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
FML: MCP 9.42 Powered by Forge <IP_ADDRESS>60 5 mods loaded, 5 mods active
States: 'U' = Unloaded 'L' = Loaded 'C' = Constructed 'H' = Pre-initialized 'I' = Initialized 'J' = Post-initialized 'A' = Available 'D' = Disabled 'E' = Errored
| State | ID                   | Version      | Source                        | Signature                                |
|:----- |:-------------------- |:------------ |:----------------------------- |:---------------------------------------- |
| LCHI  | minecraft            | 1.12.2       | minecraft.jar                 | None                                     |
| LCHI  | mcp                  | 9.42         | minecraft.jar                 | None                                     |
| LCHI  | FML                  | <IP_ADDRESS>    | forge-1.12.2-<IP_ADDRESS>60.jar | e3c3d50c7c986df74c645c0ac54639741c90a557 |
| LCHI  | forge                | <IP_ADDRESS>60 | forge-1.12.2-<IP_ADDRESS>60.jar | e3c3d50c7c986df74c645c0ac54639741c90a557 |
| LCHE  | advancedbasesecurity | 1.0          | advancedbasesecurity-1.0.jar  | None                                     |

Loaded coremods (and transformers):
Profiler Position: N/A (disabled)
Is Modded: Definitely; Server brand changed to 'fml,forge'
Type: Dedicated Server (map_server.txt)`

The build.gradle:
buildscript {
    repositories {
        maven { url = 'https://maven.minecraftforge.net/' }
        mavenCentral()
    }
    dependencies {
        classpath 'net.minecraftforge.gradle:ForgeGradle:3.+'
    }
}

apply plugin: 'net.minecraftforge.gradle'
// Only edit below this line, the above code adds and enables the necessary things for Forge to be setup.
apply plugin: 'eclipse'
apply plugin: 'maven-publish'

version = '1.0'
group = 'com.cadergator10.advancedbasesecurity' // http://maven.apache.org/guides/mini/guide-naming-conventions.html
archivesBaseName = 'advancedbasesecurity'

sourceCompatibility = targetCompatibility = compileJava.sourceCompatibility = compileJava.targetCompatibility = '1.8' // Need this here so eclipse task generates correctly.

minecraft {
    // The mappings can be changed at any time, and must be in the following format.
    // snapshot_YYYYMMDD   Snapshot are built nightly.
    // stable_#            Stables are built at the discretion of the MCP team.
    // Use non-default mappings at your own risk. they may not always work.
    // Simply re-run your setup task after changing the mappings to update your workspace.
    //mappings channel: 'snapshot', version: '20171003-1.12'
    mappings channel: 'snapshot', version: '20171003-1.12'
    // makeObfSourceJar = false // an Srg named sources jar is made by default. uncomment this to disable.

    // accessTransformer = file('src/main/resources/META-INF/accesstransformer.cfg')

    // Default run configurations.
    // These can be tweaked, removed, or duplicated as needed.
    runs {
        client {
            workingDirectory project.file('run')

            // Recommended logging data for a userdev environment
            property 'forge.logging.markers', 'SCAN,REGISTRIES,REGISTRYDUMP'

            // Recommended logging level for the console
            property 'forge.logging.console.level', 'debug'
        }

        server {

            // Recommended logging data for a userdev environment
            property 'forge.logging.markers', 'SCAN,REGISTRIES,REGISTRYDUMP'

            // Recommended logging level for the console
            property 'forge.logging.console.level', 'debug'
        }
    }
}

configurations {
    // configuration that holds jars to include in the jar
    extraLibs
}

dependencies {
    // Specify the version of Minecraft to use, If this is any group other then 'net.minecraft' it is assumed
    // that the dep is a ForgeGradle 'patcher' dependency. And it's patches will be applied.
    // The userdev artifact is a special name and will get all sorts of transformations applied to it.
    minecraft 'net.minecraftforge:forge:1.12.2-<IP_ADDRESS>60'

    // You may put jars on which you depend on in ./libs or you may define them like so..
    // compile "some.group:artifact:version:classifier"
    // compile "some.group:artifact:version"
    //compile 'org.apache.httpcomponents:httpclient:4.5.14'
    //implementation 'org.java-websocket:Java-WebSocket:1.5.7'
    //implementation group: 'org.json', name: 'json', version: '20240303'

    extraLibs 'org.slf4j:slf4j-api:2.0.6'
    extraLibs 'org.apache.httpcomponents:httpclient:4.5.14'
    extraLibs 'org.java-websocket:Java-WebSocket:1.5.7'
    extraLibs group: 'org.json', name: 'json', version: '20240303'

    configurations.compile.extendsFrom(configurations.extraLibs)

    // Real examples
    // compile 'com.mod-buildcraft:buildcraft:6.0.8:dev'  // adds buildcraft to the dev env
    // compile 'com.googlecode.efficient-java-matrix-library:ejml:0.24' // adds ejml to the dev env

    // The 'provided' configuration is for optional dependencies that exist at compile-time but might not at runtime.
    // provided 'com.mod-buildcraft:buildcraft:6.0.8:dev'

    // These dependencies get remapped to your current MCP mappings
    // deobf 'com.mod-buildcraft:buildcraft:6.0.8:dev'

    // For more info...
    // http://www.gradle.org/docs/current/userguide/artifact_dependencies_tutorial.html
    // http://www.gradle.org/docs/current/userguide/dependency_management.html

}

// Example for how to get properties into the manifest for reading by the runtime..
jar {
    from {
        configurations.extraLibs.collect { it.isDirectory() ? it : zipTree(it) }
    }
    manifest {
        attributes([
            "Specification-Title": "examplemod",
            "Specification-Vendor": "examplemodsareus",
            "Specification-Version": "1", // We are version 1 of ourselves
            "Implementation-Title": project.name,
            "Implementation-Version": "${version}",
            "Implementation-Vendor" :"examplemodsareus",
            "Implementation-Timestamp": new Date().format("yyyy-MM-dd'T'HH:mm:ssZ")
        ])
    }
}

// Example configuration to allow publishing using the maven-publish task
// This is the preferred method to reobfuscate your jar file
jar.finalizedBy('reobfJar')
// However if you are in a multi-project build, dev time needs unobfed jar files, so you can delay the obfuscation until publishing by doing
//publish.dependsOn('reobfJar')

publishing {
    publications {
        mavenJava(MavenPublication) {
            artifact jar
        }
    }
    repositories {
        maven {
            url "file:///${project.projectDir}/mcmodsrepo"
        }
    }
}

Strange, I tried building a reduced version of your build.gradle file, and the jar included slf4j as expected. Can you give link to the full project? Also need to know what Gradle version you're using, since your file uses a lot of deprecated/removed functionality.

Uploading Security-Mod-main.zip…

https://drive.google.com/file/d/1doqnqulFqt5B2jiuc9u0jacdOp0YHdUk/view?usp=sharing

Note: you might need to replace only the build.gradle file with what i pasted above, since it hasn't been updated

I cloned your provided link, replaced build.gradle with version from comment, ran gradle jar and checked resulting jar files. Both of them include org/slf4j/LoggerFactory.class as expected. Looks like this issue is specific to minecraft/forge.
Here are other people experiencing exactly the same issue (slf4j is present in jar but not found by class loader):

https://forums.minecraftforge.net/topic/62968-slf4j-dependency-missing/
https://forums.minecraftforge.net/topic/40890-dependency-problems/

So forge must be doing something stupid with class loading...
You can probably use the "shadow" plugin to remap slf4j within your jar: https://gradleup.com/shadow/configuration/relocation/
But I would recommend asking forge devs what to do in this situation. As a last resort - you can clone this library, delete all uses of "log" and "Logger" (there aren't that many actually) and recompile.
Anyway, closing this issue since it is unrelated to java-websocket, good luck with your project!

Just double checking: the gradle program "publish" works right? That's what I used to get the mod jar

Yes, in fact all 3 jar files are identical

@cadergator10 Out of curiosity I checked and it's pretty easy to remove slf4j from this library, see my fork https://github.com/PhilipRoman/Java-WebSocket/commit/a8d43d1dd2e7447f0ad0440e298427b8884e6d3e
So you have a backup option available

@cadergator10 Out of curiosity I checked and it's pretty easy to remove slf4j from this library, see my fork PhilipRoman@a8d43d1 So you have a backup option available

For some reason, now I'm getting a "[java.lang.ThreadGroup:uncaughtException:1052]: java.lang.NoSuchMethodError: java.nio.ByteBuffer.flip()Ljava/nio/ByteBuffer;" error now, both in the ide and the mod

Yes, this happens because in Java 9 they added overrides to ByteBuffer which return a more specific type. So code compiled with jdk >= 9 will fail to run on java <= 8. To avoid, either compile with older jdk, or pass --release to javac (in Gradle, that should be targetCompatibility setting)


Update README.md

Codecov Report

Merging #37 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   78.94%   78.94%
=======================================
  Files           1        1
  Lines          38       38
  Branches       15       15
=======================================
  Hits           30       30
  Misses          1        1
  Partials        7        7

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 1b55b25...2d34694. Read the comment docs.


[Improve] Flink CONF_DIR improvement

[Improve] Flink CONF_DIR improvement

cc @zhoulii PTAL. thx

LGTM


Fix Swagger bug with long line in sample parameters

There was an issue with long sample request parameters. This PR ensures that they are considered as long strings and no
"\n" gets inserted.
Before:

cc @vikhyatsharma


Review and generalize Project folder to Detail

Review detail structure (e.g config, view builders, transformers).
Review detail-specific components.
Review dynamic vs static props (and ComponentsConfig; is this still required or can it be deleted?).
Review transformers - I didn't get a response vs I can't use this response.
See #193.

Review Notes

config.ts => site-config.ts
projectsEntity => projectEntityConfig
Update outline in projectEntityConfig to group list vs detail config (and create corresponding folders for any external modules).
Move enittyTitle to be entity config-specific?
Create folders and files for footer and header.


fix readme

The readme contains a typo in example :
deleteUsersPermission.is change to > deleteUsersPermission.id

Coverage remained the same at 87.248% when pulling bd5d78d43ad428c2aa6954eebb8f0a9d9ef717f9 on kelvin2go:feature/README_TYPO into fc179d4b784682bd878ae6f0140ae1e7ebc26dc9 on enniel:master.

Coverage remained the same at 87.248% when pulling bd5d78d43ad428c2aa6954eebb8f0a9d9ef717f9 on kelvin2go:feature/README_TYPO into fc179d4b784682bd878ae6f0140ae1e7ebc26dc9 on enniel:master.


enh(vscode): :tada: add proxy support

Add proxy support for vscode-python

Hi, as discussed, it would be better to use environment variables instead of the environment file

superseded by #157


use Visual Studio warnings with SourceLink.Build

In SourceLink.Build 2.0.0-bxxx, when in Visual Studio, it isn't printing all the log statements.

dotnet tooling has completely changed this last year. This is no longer relevant.


Unable to register U2F device in the dashboard

Applicable versions - All Identity Server versions (regular, WUM updated)
1.Log into the WSO2 Identity Server end-user dashboard.
2. Navigate to the My Profile section by clicking the associated View Details button.
3. Click Manage U2F Authentication.
4. Then touch the 'key' icon on the FIDO device and click attach fido token button.
5. U2F device in the dashboard not getting registered
When debugging js we have observed the following error.
Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('chrome-extension://kmendfapggjehodndflmmgagdbamhnfd') does not match the recipient window's origin ('null').
This is caused due to u2f API change please refer [1], [2] and [3]
[1] https://github.com/google/u2f-ref-code/issues/174
[2] https://github.com/google/u2f-ref-code/issues/107
[3] https://groups.google.com/a/fidoalliance.org/forum/#!topic/fido-dev/HeNBVRosRGs

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


igc beta version: pipeline failing at deploy stage

While testing beta toolkit on ocp44 cluster on VPC  with 0.5.10-beta.4 version of igc, pipeline is failing at deploy stage with the following error:

Pipeline is working fine with non-beta version of toolkit & igc. Webhook is working for the template
Have attached the deploy step logs for reference.
logs.txt

You need to dig with oc or UI why is your deployment/template-node-angular not running correctly


Filter by playing moves shows games which don't start by those moves. [BUG]

Hi there and here description.
Describe the bug
Filter by playing moves shows games that don't match the opening.
To Reproduce
Steps to reproduce the behavior:

Click "Game Filters"
Click "Filter By Playing Move"
Use the mouse to create the game as follows: 1.e4 e5 2.Nf3 d6 3.d4 exd4
12 results will display which sixth is irrelevant.

Expected behavior
Search results must not include sixth game which has no same opening as the game filter specified.
Screenshots

Desktop (please complete the following information):

OS: Windows 10 21H2
Browser Edge
Version 112.0.1722.39 (Official Build) (64-bit)

Additional context
There is a warning that says:

This is only a fun project and not everything is properly parsed. Do not trust entirely.

So this is not a problem, but I hope there is an improvement.

Hi, there was invalid PGN in video description. Fixed that


install "www-browser" symlink as well

In the terminal, as with e.g. targets xiwi and cli-extra, tools such as xdg-open can't find a $DISPLAY, so they attempt to execute www-browser rather than x-www-browser. So, that symlink should also exist.

Good catch.  Does croutonclip get properly launched, though, in these situations?  Still worth merging since if you install a console-based browser it will override this.

I've just been running croutonwebsocket at the beginning of my session, which seems to be enough to get this working, although please do advise if it's not the recommended solution. I read #2113 with interest, because it seems plausible that something similar could be done behind the scenes... I haven't yet understood the referenced pipe perm issues. If one wanted to fix that issue, how would one go about it?

Right, that should be sufficient.  Pipe permissions have already been fixed thanks to the merging of the multiuser branch, so if you'd like to take a stab at it, just make sure to copy how croutonwebsocket is launched elsewhere.


NOX-574 Configure OpenApi Request and Response for Custom Route Mapping Targettng a Default Generated end point

@rochar  please see the code updated based on your comments.


RFC: Automatic assert messages

Issue #, if available:
Description of changes:
This demonstrates a technique for making the message/format arguments to
ASSERT_* optional. When a message format is not specified, a default is built
using the expression passed in instead.
I've only converted ASSERT_TRUE and ASSERT_FALSE for now, to see what people
think of this approach before spending too much time on it.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Updated the PR with a full implementation, plus a meta-test for the asserts themselves.

For reference, here's the failure.
$ cppcheck --enable=all --std=c99 --language=c --suppress=unusedFunction --suppress=missingInclude -I include --force --error-exitcode=-1 -USELF_TEST  -UCLOCK_MONOTONIC_RAW --suppress=memleak:tests/hash_table_test.c --suppress=unreadVariable:tests/hash_table_test.c -q ./
[tests/assert_test.c:120]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/assert_test.c:120]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/assert_test.c:102]: (style) The scope of the variable 'nullp1' can be reduced.
[tests/assert_test.c:112]: (style) The scope of the variable 'increment' can be reduced.
[tests/assert_test.c:126]: (style) The scope of the variable 'str_x' can be reduced.
[tests/assert_test.c:235]: (style) The scope of the variable 'rv' can be reduced.
[tests/assert_test.c:135]: (warning) Unnecessary comparison of static strings.
[tests/assert_test.c:138]: (warning) Unnecessary comparison of static strings.
[tests/math_test.c:34]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:34]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:35]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:35]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:36]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:36]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:37]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:37]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:38]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:38]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:40]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:40]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:41]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:41]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:42]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:42]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:43]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:43]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:44]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:44]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:45]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:45]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:46]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:46]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:47]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:47]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:48]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:48]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:49]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:49]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:56]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:56]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:57]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:57]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:58]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:58]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:59]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:59]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:60]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:60]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:62]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:62]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:63]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:63]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:64]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:64]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:65]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:65]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:66]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:66]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:67]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:67]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:68]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:68]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:69]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:69]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:70]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:70]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:71]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:71]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[tests/math_test.c:72]: (warning) %llu in format string (no. 2) requires an unsigned integer given in the argument list.
[tests/math_test.c:72]: (warning) %llu in format string (no. 3) requires an unsigned integer given in the argument list.
[*]: (information) Unmatched suppression: missingInclude


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_20331119-8421-43d1-85ba-6cb253022d19/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_20331119-8421-43d1-85ba-6cb253022d19.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-ServiceAccountCannotAccessRepository] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source has enabled Docs Build service account. It seems that at least one of the accounts, e2ebd1, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master#ops-e2e-ppe) and then retry the build. Don't know who to contact? Refer https://repos.opensource.microsoft.com/VSC-Service-Account/repos/docs-archive-test-source/permissions/ to get admin list of the repo if the repo is in one of the organizations Microsoft owns.

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


Examples don't seem to work for .net 6 mvc web app

I've created a new .net6 mvc web app using vs2022 & the standard project template.
When I add via nuget the CorrelationId package, and modify program.cs and then run, my first page load shows:
The 'X-Correlation-ID' request header is required, but was not found.

Here's my code ...
           builder.Services.AddDefaultCorrelationId(options =>
            {
                options.CorrelationIdGenerator = () => Guid.NewGuid().ToString();
                options.AddToLoggingScope = true;
                options.EnforceHeader = true;
                options.IgnoreRequestHeader = false;
                options.IncludeInResponse = true;
                options.RequestHeader = "My-Custom-Correlation-Id";
                options.ResponseHeader = "X-Correlation-Id";
                options.UpdateTraceIdentifier = false;
            });
            var app = builder.Build();
            // Configure the HTTP request pipeline.
            if (app.Environment.IsDevelopment())
            {
                app.UseMigrationsEndPoint();
            }
            else
            {
                app.UseExceptionHandler("/Home/Error");
                // The default HSTS value is 30 days. You may want to change this for production scenarios, see https://aka.ms/aspnetcore-hsts.
                app.UseHsts();
            }
            app.UseHttpsRedirection();
            app.UseStaticFiles();
            app.UseCorrelationId();
            app.UseRouting();
            app.UseAuthentication();
            app.UseAuthorization();

@sarchibald-mdsol Are you including a header My-Custom-Correlation-Id in your request? With EnforceHeader=true, this is the expected behaviour if incoming requests do not include the header. I cannot reproduce an issue when the expected header is present.


Feedback for "Manage Authorizations and Roles"

https://help.sap.com/webcomponents/products/SAP Business Application Studio/9d1db9835307451daa8c930fbd9ab264/01e69c53003c4b0a8a64310a3f08867d.html
The description of the different Roles for Business Application Studio is very generic. The following descriptions:

To allow development using SAP Business Application Studio, the developer must be assigned the Developer role.
To allow administrator operations using SAP Business Application Studio, the relevant user must be assigned the Administrator role.
To allow a developer to create an SAP Business Application Studio extension, the relevant user must be assigned the Extension Deployer role.

Should be explained in more detail. What are "administrator operations"? What covers "development"?
The Security Guide contains no additional details either.

I am following up with my dev team and will get back to you soon. Thanks for your feedback!

There is more detail about each role in the Assigning Permissions section of Manage Authorizations and Roles. But I have moved the description of each role to the beginning of the topic to make it more clear and have added links to topics describing the Admin role's specific capabilities. The changes will appear in our next release (about a week and a half). Thanks again for your valuable feedback!


we should default to dashboard, not metrics def

clicking on a metrics thing on the left sidebar should go to the dashboard if the dashboard is valid
if it isn't valid, it should go to the metrics def
if it is empty, it should go to the metrics def
we should call the section "DASHBOARDS" + use the little dashboard icon

@AdityaHegde could you take al ook at this? I tried to get it in but the metrics def didn't quite seem to have the right information in the MetricsDefinitionAssets.svelte component. The measures and dimensions seemed to be empty even if they weren't actually empty.


[BUG] infinite recursion

Hello,
The tool runs into infinite recursion, I think i have a good example for that to demonstrate:

The first directory cgi-bin returned 403 and anything after that is 404 which led the tool to assume that it is a directory which is a good technique but the directory .git  returns 403 and everything after that returns 403 also which made the tool assume everything after .git is a directory and finally the process gets killed because it ate up all the memory
My suggestion:
if .git returns 403 then:
    BLACKLIST_STATUS_CODE = https://targets.com/.git/RANDOMSTRING

    if https://target.com/.git/WORDLIST rerurns BLACKLIST_STATUS_CODE then:
        return NOT_VALID_RESULT
    else
        return VALID_RESULT

This can work for both files and directories (because it also happened to me with files and retuned all of the files in the wordlist as valid results in a new discovered directory)

Good morning and thanks for the report!
This is a great use-case for the scan cancel menu.
It looks like your suggestion is essentially the same wildcard test we already perform on each recursed directory, or am i missing something?

Hey,
I actually don't know the first thing in rust xd
It seems like you already run a wildcard test but sometimes it fails hard and considers the whole wordlist as valid results as you see in the above picture, maybe it's because the server is returning a 403 and the wildcard test is only checking for 200? Not sure.
As for the scan cancel menu, first of all, it would be nice to expect a range like 2-10 to remove all tasks in range.
And for my example, every time I cancelled a task, 10 more gets generated so it didn't help.
I've sent you my example URL on your Twitter account if you are interested to test it.
Thanks for the amazing work btw.


Raft protocol version 3 by default

why isn't version 3 enabled in config?

This is a valid issue and should be addressed. A good example how to resolve this is https://github.com/hashicorp/terraform-google-consul/blob/master/modules/run-consul/run-consul#L192. Since we don't have the bandwidth to work on this right now and it's a pretty simple change, PRs are welcome!


Use the EnableConfigurationProperties annotation

As reported by @sdeleuze this is the correct way to configure them, and this will solve one of the issues in https://github.com/spring-projects-experimental/spring-native/issues/600

@jdubois @pascalgrimaud Better to merge it after V7 or this should be included?

@DanielFran I'm trying to have Spring Native support for JHipster 7, but if it's later there's no big issue...

Depending if you are confident or not.
Anyway, it's simple, so I don't think this PR will broke the builds

It's simple and I did some tests, so yes I'm confident.


:sparkles: comment typo fix

Summary
Typo fix in a comment.
This API is imported as logicalcluster, and that was the intention of Mike.

wrong fix location

@vMaroon looks like you have a codegen issue

@clubanderson true, my bad. Ran 'make crds' to propagate the change into generations, anything else?


[0.11] Fixes #2353 - Race condition in scaling app

TaskTracker gets updated
RateLimiter gets notification for errors
TaskUpdate is propagated to other interested parties

LGTM. Especially the fact, that we have 150 less LOCs.

After Merge, can you please cherry pick on master?


it doesn't work on react jsx project and vanilla html/css project on my pc

it doesn't work on react jsx project and vanilla html/css project on my pc

Could you explain more? it should work with vanilla and react


CI-build-adding-MQTT-and-WIFI

Subject: Added MQTT and WIFI to CI
type: User story
git tags: User story, Sprint 1
changes: # Solution or List of Changes Note
notes: Building in pipeline Arduino CI
Resolves user story #20

Directory Error fixed!


🌱 update mockery version on the repo

this was breaking on remote system so this commit updates mockery to latest version.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

you should run unit tests locally before you trigger the CI. I could imagine that it panics

Yes I ran and it passed but I was on go 1.22.1 on my system so wasn't able to catch it.


Cannot convert value of type 'TCPSocketSource' to expected argument type 'SocketSource<_>'

I added FluentMySQL to my Vapor 3 beta project and now I can't build it because of error
Cannot convert value of type 'TCPSocketSource' to expected argument type 'SocketSource<_>'

Is there any way to fix it?

This has been fixed


Can we have a resizable column when the user drag a particular column

Hi Allen,
Is there any option to resize the column or row when the user do some action related to that
Thanks

it's a duplicated issue, and there's two PR want to support this feature, but we still working. Thanks


feat: add serve config for teacher model in ilab data generate

Currently ilab data generate uses the configuration from the serve section of configuration file to serve the teacher model in the background.
In the case where a user is using a specific model only as the teacher model, the user would have to edit the serve section of the config with the teacher model's settings.
Since the teacher model and it's configuration is static, and the serve section is dynamic, the teacher model and it's configuration should live in the generate section of the configuration.
Giving the teacher model it's own configuration section also prevents the user from accidentally serving the teacher model, if they have not changed the serve section.
This PR is meant to be a starting point. Further work should evolve the configuration of the teacher model in ilab data generate to be similar to how the ilab model evaluate command configures the judge model for MT-Bench.
This PR also includes a commit that adds the --enable-serving-output flag to ilab data generate. This flag serves the same purpose as it does in ilab model evaluate.

Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

Adding the hold label, which will be removed after I'm done testing this PR by hand. Reviews are still welcome.

I guess I just don’t feel this is necessary. Here’s how I think about it: something has to go in the serve config. What is the majority use case that is fair to expect should be in there? My thought: the teacher model config.
An alternative answer to that could be “the target model to be trained, for use with ilab chat to test model behavior.”
Is the latter what you were thinking?
I’ll also say that I don’t like “general inference use cases” as an answer here because I don’t think using ilab makes much sense for that vs running vllm directly (via systemd). I’m not sure what value ilab provides in that case.

@leseb The teacher model can be a separate model that specializes just generating the data from the taxonomy. One model you could use for example is Mixtral. This is similar to how we have a judge model just for MT-Bench in evaluation. The model is prometheus.
The teacher model could have it's own set of configuration, ex: vllm_args.

I guess I just don’t feel this is necessary. Here’s how I think about it: something has to go in the serve config. What is the majority use case that is fair to expect should be in there? My thought: the teacher model config.
An alternative answer to that could be “the target model to be trained, for use with ilab chat to test model behavior.”
Is the latter what you were thinking?
I’ll also say that I don’t like “general inference use cases” as an answer here because I don’t think using ilab makes much sense for that vs running vllm directly (via systemd). I’m not sure what value ilab provides in that case.

The alternative answer along with inference use cases is what I was thinking. Here's how I see the entire ilab workflow along with when models are served:

Download all the necessary models. These are the Base, Teacher, and Judge models.
Chat with the Base model on how much it knows the information you want it to know about. Right now chat reads from the serve config if the model is started up in the background.
Make changes to the taxonomy, adding new skills and knowledge.
Run data generation. Right now the model started up in the background also reads from the serve config.
Train the model. Here MT-Bench is used in multiphase training. I think @JamesKunstle is calling the exact same functions in model/evaluate.py for the base and judge model and not reading anything from the serve config.
Chat with the trained model to see what the model learned. As mentioned in step 2 right now chat reads from the serve config if the model is started up in the background but since it's derived from the base model, I think it's serving configuration should exactly be the same except for the model path.
Run evaluation. For MT-Bench the necessary base and judge model configuration happens within evaluate.py, not in the serve config.

As I mentioned above I think since the teacher model can be viewed as something static similar to the judge model, that's a good reason for it to have it's own config in the generate section.
@leseb @n1hility  do you have anything to add about the inference use case?

OK - I buy that the base model is a better thing to assume stays in the base serve config.  Thanks.

Note: the last patch should be squashed before merging.

@alimaredia looking at the checklist in the description, I think you may want to update changelog file to document the new section.

@alimaredia do we still need the hold label here?

@booxter Ran the branch by hand, and serving worked as expected. I've removed the hold label.

@russellb could you approve this PR so it merges?


production with gunicorn, nginx

What I'm trying to achieve
Trying to start up a gunicorn socket with saleor.wsgi:application
Steps to reproduce the problem

git clone https://github.com/mirumee/saleor-platform.git --recursive --jobs 3
2.create socket and service in gunicorn with a working path of home/clayton/saleor-platform/saleor/saleor

[Unit]
Description=saleor gunicorn daemon
Requires=saleor.socket
After=network.target
[Service]
User=clayton
Group=www-data
WorkingDirectory=/home/clayton/saleor-platform/saleor/saleor
ExecStart=/home/clayton/saleor-platform/env/bin/gunicorn
--access-logfile -
--workers 3
--bind unix:/run/saleor.sock
saleor.wsgi:application
[Install]
WantedBy=multi-user.target
---and---
[Unit]
Description=saleor gunicorn daemon
Requires=saleor.socket
After=network.target
[Service]
User=clayton
Group=www-data
WorkingDirectory=/home/clayton/saleor-platform/
ExecStart=/home/clayton/saleor-platform/env/bin/gunicorn
--access-logfile -
--workers 3
--bind unix:/run/saleor.sock
saleor.wsgi:application
[Install]
WantedBy=multi-user.target
What I expected to happen
starting a gunicorn service, but testing the connection with 'sudo systemctl status saleor'  results in :
(env) clayton@Personal:/etc/nginx/sites-enabled$ sudo systemctl status saleor
● saleor.service - saleor gunicorn daemon
Loaded: loaded (/etc/systemd/system/saleor.service; enabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Tue 2021-01-19 03:16:20 UTC; 1min 34s ago
Main PID: 21965 (code=exited, status=3)
Jan 19 03:16:20 Personal gunicorn[21965]:   File "", line 219, in _call_with_frames_removed
Jan 19 03:16:20 Personal gunicorn[21965]:   File "", line 994, in _gcd_import
Jan 19 03:16:20 Personal gunicorn[21965]:   File "", line 971, in _find_and_load
Jan 19 03:16:20 Personal gunicorn[21965]:   File "", line 953, in _find_and_load_unlocked
Jan 19 03:16:20 Personal gunicorn[21965]: ModuleNotFoundError: No module named 'saleor'
Jan 19 03:16:20 Personal gunicorn[21965]: [2021-01-19 03:16:20 +0000] [21992] [INFO] Worker exiting (pid: 21992)
Jan 19 03:16:20 Personal gunicorn[21965]: [2021-01-19 03:16:20 +0000] [21965] [INFO] Shutting down: Master
Jan 19 03:16:20 Personal gunicorn[21965]: [2021-01-19 03:16:20 +0000] [21965] [INFO] Reason: Worker failed to boot.
Jan 19 03:16:20 Personal systemd[1]: saleor.service: Main process exited, code=exited, status=3/NOTIMPLEMENTED
Jan 19 03:16:20 Personal systemd[1]: saleor.service: Failed with result 'exit-code'.
Screenshots

System information
Ubuntu 18 on digitalocean

From my experience you have three options...

Deploy an unsecured demo/development environment on your local machine.
Deploy an unsecured demo/development environment on your remote machine with Docker.
Pay $950/month to deploy a secured production environment on a remote machine (probably also with Docker which I don't find to be easy to secure, and securing your docker container is not in the scope of Mirumee's monthly fees because docker is not part of Saleor).

I'm working on a bash install script for secure Saleor production deployment on Ubuntu/Nginx and have gotten literally zero help form anyone at Saleor. They want that $950/mo. pretty bad. I don't blame them. This app was a lot of work to build, and remains a lot of work to maintain & update.

From my experience you have three options...

Deploy an unsecured demo/development environment on your local machine.
Deploy an unsecured demo/development environment on your remote machine with Docker.
Pay $950/month to deploy a secured production environment on a remote machine (probably also with Docker which I don't find to be easy to secure, and securing your docker container is not in the scope of Mirumee's monthly fees because docker is not part of Saleor).

I'm working on a bash install script for secure Saleor production deployment on Ubuntu/Nginx and have gotten literally zero help form anyone at Saleor. They want that $950/mo. pretty bad. I don't blame them. This app was a lot of work to build, and remains a lot of work to maintain & update.

From my experience you have three options...

Deploy an unsecured demo/development environment on your local machine.
Deploy an unsecured demo/development environment on your remote machine with Docker.
Pay $950/month to deploy a secured production environment on a remote machine (probably also with Docker which I don't find to be easy to secure, and securing your docker container is not in the scope of Mirumee's monthly fees because docker is not part of Saleor).

I'm working on a bash install script for secure Saleor production deployment on Ubuntu/Nginx and have gotten literally zero help from anyone at Saleor. They want that $950/mo. pretty bad. I don't blame them. This app was a lot of work to build, and remains a lot of work to maintain & update.

Gotcha, I was mostly interested in testing it, but the setup I had wasn't ultra powerful, which made the docker route less than ideal. I do all of my setups on django/gunicorn instances, so uwsgi wasn't a big stretch. I actually got a lot of it figured out, but it's definitely through research that veers heavily outside of the documentation.

From my experience you have three options...

Deploy an unsecured demo/development environment on your local machine.
Deploy an unsecured demo/development environment on your remote machine with Docker.
Pay $950/month to deploy a secured production environment on a remote machine (probably also with Docker which I don't find to be easy to secure, and securing your docker container is not in the scope of Mirumee's monthly fees because docker is not part of Saleor).

I'm working on a bash install script for secure Saleor production deployment on Ubuntu/Nginx and have gotten literally zero help from anyone at Saleor. They want that $950/mo. pretty bad. I don't blame them. This app was a lot of work to build, and remains a lot of work to maintain & update.

Gotcha, I was mostly interested in testing it, but the setup I had wasn't ultra powerful, which made the docker route less than ideal. I do all of my setups on django/gunicorn instances, so uwsgi wasn't a big stretch. I actually got a lot of it figured out, but it's definitely through research that veers heavily outside of the documentation.

Check out this video I found on deploying projects like this to a production environment.
https://www.youtube.com/watch?v=ZpR1W-NWnp4
It's helping me a lot on trying to implement a production deployment of Saleor. Very relevant.

Check out this video I found on deploying projects like this to a production environment.
https://www.youtube.com/watch?v=ZpR1W-NWnp4
It's helping me a lot on trying to implement a production deployment of Saleor. Very relevant.


useCustomCells removed in v6?

Upon attempting to upgrade to v6 I've noticed that useCustomCells is no longer exported from @glideapps/glide-data-grid. I don't see any mention of this removal anywhere, or any indication what to use instead

This change with v6: You can just provide a list of custom cells to use in the customRenderers props. Or you can import all via:
import { allCells } from "@glideapps/glide-data-grid-cells"

 <DataEditor
      # ... other props,
      customRenderers={allCells}
    />

So instead of doing
    const customCellsArgs = useCustomCells([ButtonCell, RangeCell]);

     <DataEditor
       ref={gridRef}
       {...customCellsArgs}
      />

You now just do
     <DataEditor
       ref={gridRef}
       customRenderers={[ButtonCell, RangeCell]}
      />

Yep, I believe that should work. Also, see the comment here on how you can use a custom drawing.

Ok thanks :) I'll give the upgrade another go when I get a chance. v6 looks nice :)


Using device name for the device names, if possible

Uses the existing device info cache to pull out actual device names fro vpn and uplink status. Reverts to the current behavior if for any reason the device can't be found.

On staging, check it out when you like.

staging testing confirmed! 🚀


Turning off shortcuts

Hi,
Could I ask how I can turn off shortcuts, specifically NodeJS > Debug [^ + D]? The NodeJS shortcuts are interfering with VI mode for me
Thanks!
Nolan

Hi @NolanChan ,
I've added new functionality for changing default key bindings. Could you check out the branch issue-62?

Fixed in a new release v1.5.5. Check out the master branch or install the new version from package manager.

Thanks @varp , will do


How to run the samples project?

Is your feature request related to a problem? Please describe.
When I run the AzureMapsControl.Sample project using visualstudio or the dotnet cli I don't understand how to browse to the sites pages.
Is the feature already existing on azure-maps-control or on one of its plugins ?
This relates to the  AzureMapsControl.Sample project.
Describe the solution you'd like
Some guidance on how to run the project would be greatly appreciated.
Additional context
Add any other context or screenshots about the feature request here.

I tweaked the project so that the pages are being served OK.


Automated cherry pick of #3380: fix: local snapshot's ownerId should be same with its disk even if extsnapshot's project has corresponding cloud project

Cherry pick of #3380 on release/2.11.
#3380: fix: local snapshot's ownerId should be same with its disk even if extsnapshot's project has corresponding cloud project

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@9a88408). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #3396   +/-   ##
==============================================
  Coverage                ?   7.13%
==============================================
  Files                   ?     476
  Lines                   ?   77277
  Branches                ?       0
==============================================
  Hits                    ?    5516
  Misses                  ?   71298
  Partials                ?     463

Flag
Coverage Δ

#aFlag
7.13% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/snapshots.go
1.63% <0%> (ø)

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 9a88408...1a51913. Read the comment docs.

/lgtm

/approve


Any plans to support it for Angular 7?

I tested it. Seems to work with Angular 7. Can you confirm this ?

Works great! Already tested it on latest Angular 7. but do have any plans to support latest monaco editor v0.15.6? :)

Could you explain how i can run ngx-monaco-editor on latest Angular?

        { "glob": "**/*", "input": "../node_modules/ngx-monaco-editor/assets/monaco", "output": "./assets/monaco/" }
      ],```

this is not working in Angular 7

I'm having an issue with an Angular 7 project, it might be because of the Angular version. When the language is css I get the following error in the console:

Anyone help
@jesusvallez isn't node_modules in the same folder as angular.json?
This works for me:
{
                "glob": "**/*",
                "input": "node_modules/ngx-monaco-editor/assets/monaco",
                "output": "./assets/monaco/"
}

No ../ before node_modules

@mhintzke13  mine was also not working on Angular 7, until I changed the config in angular.json
 "glob": "**/*",
  "input": "../node_modules/ngx-monaco-editor/assets/monaco",
  "output": "./assets/monaco/"

to
  "glob": "**/*",
    "input": "node_modules/ngx-monaco-editor/assets/monaco",
     output": "./assets/monaco/"

try it.


Support 0d arrays in cuda

Feature request
I'm curious what the intent behind these lines is:
https://github.com/numba/numba/blob/0aaf62d830e7a9d1dc5ffbf710a7cd38caa52d1a/numba/cuda/cudadrv/devicearray.py#L681-L686
Converting 0d arrays to 1d arrays often makes it harder to reason about the behavior of array code.
Following on from https://github.com/numba/numba/pull/5343#discussion_r387971072, where I spotted this accidentally - supposedly it is tested for in  TestCudaNDArray.test_device_array_interface.

IIRC, the reason is related to 0D array has no ary.ctypes.

Not entirely sure what you mean by that:
In [10]: np.array(1).ctypes
Out[10]: <numpy.core._internal._ctypes at 0x1bd01a8d520>

Fixed in #5359


Converting LeafVersion into an enum

The original LeafVersion implementation was just a newtype around u8. I think that having enum explicitly listing consensus script implementation rules may be more beneficial in terms of both code readibility and future use of multiple script types, where LeafVersion may operate as a context object provided to Script to specify interpretation rules for particular op codes.

I actually agree to switch back into *_u8 since it is not a byte in fact: it will be modified with parity flag

concept ACK. I also prefer _u8 over byte.

@apoelstra @sanket1729 @Kixunil rebased and changed to *_u8

Concept ACK
In general, we do well to lean further into Rust's excellent enum system.

Note: let's not forget to add this to non_exhaustive TODO after it's merged.

Addressed all comments. @Kixunil added to non-exhaustive todo (as a part of your comment https://github.com/rust-bitcoin/rust-bitcoin/issues/510#issuecomment-888989487).
@apoelstra needs re-ACK

@Kixunil I did as you have suggested. Interesting enough this breaks unit tests, since they include deserialization of an invalid leaf version. I assume this must not error and deserialize correctly, otherwise taproot would be a hard fork and not a soft fork. Basically, we need to allow processing (but not constructing?) invalid control blocks.
I propose to add third variant (LeafVersion::Invalid(InvalidLeafVersion))) which will be produced only during deserialization and can´t be constructed manually.  @apoelstra what do you think?

I don't understand why rejecting an invalid control block would be hard fork and not soft fork. Anyway, this is outside of my knowledge. If accepting invalid versions is truly needed then I'm not against having another variant/type but not currently entirely sure it's the best way.

If our tests assume you can deserialize invalid control blocks then the tests are wrong. There is no reason to support this.
And no, disallowing invalid data does not make Taproot a hardfork.

@apoelstra from what I understand, pre-taproot block may contain witness v1 output spent by anybody with arbitrary script (before taproot activation) including use of invalid control block.
The test which is failing is taken from bitcoin core, so probably we should double-check what it is... I will try to figure this out

@dr-orlovsky maybe the shouldn't be parsed as control blocks pre-taproot activation?

We don't need to support pre-Taproot Taproot-like spends in this library.

From README:

This library must not be used for consensus code

and

patches to fix specific consensus incompatibilities are welcome.

Suggest that it's up to PR author to decide.
IMO it'd be faster to just skip it now, releasing Taproot is more important.
If someone wants to do it (later), I'd prefer using some flag controlling whether Taproot is activated and avoiding parsing control blocks completely if it isn't. Similarly to how Core does it.

Ok, rebased & removed failed tests with invalid control blocks. Ready for re-review

Interestingly enough disallowing invalid leaf versions breaks unit tests, since they include deserialization of invalid control blocks. I assume this must be allowed, otherwise taproot would be a hard fork and not a soft fork. Basically, we need to allow processing (but not constructing?) invalid leaf versions?!

@dr-orlovsky @apoelstra The current implementation is master correct. As I mentioned in the review, I think the futureleafversion is incorrect. I had those test vectors from cross-testing with bitcoin core.
From BIP 341:

In addition, in order to support some forms of static analysis that rely on being able to identify script spends without access to the output being spent, it is recommended  ....

There can be an update to use leaf version, 0x02, but BIP341 does not recommend doing it. But an update would be consensus accepted. Thus, we must parse those leaf versions correctly.

Addresses all your addressable requests in #761


Add XChaCha20Engine and XChaCha20Poly1305

This PR adds an implementation of IETF draft XChaCha20 and XChaCha20Poly1305 (https://datatracker.ietf.org/doc/html/draft-irtf-cfrg-xchacha). The implementation was inspired by the XSalsa20 implementation.
Fixes #631

What is the current progress of this PR? We really need XChaCha20


I can use BPMN group elements for modeling

Why do we need this?
Group artifacts can help a lot to structure models and make them more understandable - just like text annotations.
People ask for them in trainings and I think they should be available in the modeler. Also, you guys promise to provide full BPMN coverage in the modeler ;)
https://docs.camunda.org/manual/7.5/modeler/camunda-modeler/bpmn/#bpmn-2-0-coverage
Cheers, Ben

What about using colors for semantic grouping instead?

Needs to be addressed via https://github.com/bpmn-io/bpmn-js/issues/343.

I wouldn't like using colors as a replacement for groups since they are not part of the standard. Also colors could be used for different concepts in the same model, e.g. use a light shaded color for the happy path and use groups to structure the process by stages/places/involved external services etc.

Got it. Thanks for your feedback.

Groups are one of three basic artifats of BPMN standard. They are not exotic at all. To have no support of Groups is to not conform the standard.
Additionally, most of alternative BPMN modeling tools (e.g. wide spreaded free Bizagi Modeler) do have Gropups and some additional means (like colors or pool milestones) for designating general structure of process and these means don't seem to be excessive or inconsistent.
With such environment Comunda BPMN Modeller is looking slightly faded.

Thanks for your feedback. We'll support groups eventually, I'm pretty sure of that.

Any update on this?  I am looking to use this to supoort adding security rules on Bpmn execution. So someone can create a group over multiple elements and write security rules such as "who ever starts the first task of this group must be the user who does all the tasks". Or "the user who does task 1 cannot be the user who does any other task in the group"
This would be implemented as a parse listener on the engine show de that would find the groups and find the tasks in the groups and apply the configs to the Bpmn based on the group properties/extensions.

We are currently planning to implement groups in our next milestone. Exactly today we started to define the tasks on this topic 🙂

Closed via https://github.com/camunda/camunda-modeler/pull/1404.
:tada:

❤

@nikku is there any docs on how this was implemented and intended to be used?  Does a group has the ability to detect/know what objects are within it?  Does Camunda engine have this parsing capability ?

Groups are visual only at the moment. We do not maintain categorized elements at the moment.
Please open an additional feature request, clearly making your case for that enhancement if you would like us to look into it, some day maybe.

@nikku I originally did this in the original issue as above.... https://github.com/camunda/camunda-modeler/issues/464#issuecomment-476661202


Ambiguous Mob Killing

Is there a way for the Kill Mob Objective, to make it so I can have my players need to kill 5 of ANY mob, instead of 5 of EVERY mob.
I have tried over and over again, and looked in many places for the answer, and Cannot find it.
If anyone has the answer, please respond asap.

No, there is not. The solution is to have a module created for that purpose. See the last section of this article for more information:
https://github.com/PikaMug/Quests/wiki/Casual-‐-Modules#need-support-for-other-plugins
If you would like to tackle this yourself, see Example 3 of the Objective API:
https://github.com/PikaMug/Quests/wiki/Master-‐-Custom-Quest-API#objectives-api


vue-runtime-helpers Reference Added

This may not be an issue, so feel free to just close this, but just so it's documented in case anyone else runs into it:
Commit c837b26 added a reference to vue-runtime-helpers, but it didn't get pulled in as a dependency when running npm install, so I had to add a reference to vue-runtime-helpers in my package.json.
The specific error that popped up in my CI pipeline was:
This dependency was not found:
* vue-runtime-helpers/dist/normalize-component.js in ./node_modules/vue-ckeditor2/dist/vue-ckeditor2.esm.js

This is maybe my vault when I upgrade version of bili. I will fix soon. Thanks @pplusawork

Hi @pplusawork
The dependency vue-runtime-helpers was removed.


Some version alias rules cause the insertion of a space in versions.properties

Describe the bug
As per subject, sometimes the rule interpreter outputs:
 version.packagename (with a leading whitespace) rather than the expected version.packagename (with no leading space)
To Reproduce
Use the following custom rule:
com.uchuhimo:konf(-*)
             ^^^^

Apply the following dependencies:
dependencies {
    implementation("com.uchuhimo:konf:_")
    implementation("com.uchuhimo:konf-yaml:_")
}

Run ./gradlew refreshV and observe versions.properties.
Expected behavior
As per any package, there is no space before version.
Any change to the rule I've tested works normally:
com.uchuhimo:konf(-*)
   ^         ^^^^

produces version..konf
com.uchuhimo:konf(-*)
            ^^^^^

produces version:konf, and so on.
I thought it could be related to selecting the initial part of the artifact name as custom key, but the same approach works flawlessly for a number of other packages:
com.github.cb372:scalacache(-*)
                 ^^^^^^^^^^

org.antlr:antlr4(-*)
          ^^^^^^

org.junit.jupiter:junit-jupiter(-*)
                  ^^^^^^^^^^^^^

They work. I'm puzzled.

You are missing the "Actual behavior" part. That said, I think you're referring to the intentional leading whitespace that is there when the version key is too short. It's there to keep the # available comments aligned for easy upgrade.

I see. I did not know of this behaviour. So, the minimum alias length is 6 chars. Good to know.

Nope, the alias can be shorter, the spaces are not an issue (or did you witness otherwise?).

Not for refreshVersions I'd say.
It's an issue for the current version of UpGradle, but if you can reassure me that this behavior is to stay, I can support it easily.

I wrote the rationale above:
https://github.com/jmfayard/refreshVersions/issues/200#issuecomment-651403722
so I don't see why this would change.

Great. Adding support to UpGradle, then.


Bold durations in CLI messages

I guess the .bold() on the preceding text causes the .dimmed() to... stop?
But you can compare before and after:

Closes #4817.

Can someone test this in another terminal too?


Incorrect image size calculation in k4a_image_create function for K4A_IMAGE_FORMAT_COLOR_NV12 format

Image size calculation inside k4a_image_create function is incorrect in case of K4A_IMAGE_FORMAT_COLOR_NV12 format:
k4a_image_create(K4A_IMAGE_FORMAT_COLOR_NV12, 1280, 720, 1280, &image);

creates image with buffer size 921600 bytes (I guess it is calculated as sride * height which gives 1280 * 720 = 921600 in our case).
But it is not correct size for NV12 image 1280x720. Correct size must be 3 * stride * height / 2  that is 1382400 bytes.
You can say that this is by design and I have to specify different stride for such NV12 image. But let's start color streaming from device in K4A_IMAGE_FORMAT_COLOR_NV12 format and K4A_COLOR_RESOLUTION_720P resolution, then wait for the first capture and ask color image from it using k4a_capture_get_color_image. As a result we'll get image with the following parameters:

format:  K4A_IMAGE_FORMAT_COLOR_NV12
width: 1280 pixels
height: 720 pixels
stride: 1280 bytes (exactly as in our call to k4a_image_create)
size: 1382400 bytes (but k4a_image_create returns image with size = 921600 bytes)

That is, at least we have some inconsistency between k4a_image_create and k4a_capture_get_color_image function for K4A_IMAGE_FORMAT_COLOR_NV12 images.
SDK Version:  1.1.1

Thanks @bibigone, unfortunately k4a_image_create makes a simplified assumption about the size of the image so it doesn't work for NV12. You can use k4a_image_create_from_buffer, although it's a bit more complex.
I think we need to introduce a new API (perhaps k4a_image_create_with_size) to handle the NV12 case without the user having to provide their own buffer.

unfortunately k4a_image_create makes a simplified assumption about the size of the image so it doesn't work for NV12

I see. But there're at least two better solutions than simply create incorrect image:

Adding the following special case processing to image size calculation:

if (format == K4A_IMAGE_FORMAT_COLOR_NV12) {
    size = 3 * size / 2;
}

Returning K4A_RESULT_FAILED from k4a_image_create if format is equal to K4A_IMAGE_FORMAT_COLOR_NV12 (with additional message to log).

I would prefer the first solution, but the second one is also better than current behavior.

I think we need to introduce a new API (perhaps k4a_image_create_with_size)

This is really good idea independent on issue with K4A_IMAGE_FORMAT_COLOR_NV12 format.


Move migrations out of PFS

We want to move to Postgres as a datastore for all services in pachd, but currently the migration code runs in the PFS master. This moves the definition of all the migrations to src/server/pkg/clusterstate and moves the database migration logic to the pachd main so it runs before any services are started.
Right now every pachd will attempt to run the migrations and block on acquiring a lock on the migrations table. This seems fine for now but long-term we may want to move this logic to the deploy process instead.

I think we were running the migration in the PFS master (master.go).  That can be removed, and the blocking call in the constructordriver.go can go as well.  The change you made to WithRealEnv should take care of it.

I think we were running the migration in the PFS master (master.go).  That can be removed, and the blocking call in the constructordriver.go can go as well.  The change you made to WithRealEnv should take care of it.

I've reverted everything but the bare minimum back to the way it was, should be good for another pass once CI is green.

I've reverted everything but the bare minimum back to the way it was, should be good for another pass once CI is green.