NwbRecording parsing dataset name -- ValueError: invalid literal for int() with base 10

Greetings!  I have a minor little goblin to report/share!
I'm reading events from an NWB file like this:
from open_ephys.analysis import Session
directory = './my_data/nwb'
session = Session(directory)

recording = session.recordnodes[0].recordings[0]
recording.events()

And getting this error message:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/x/miniconda3/envs/oe-python-plugin/lib/python3.10/site-packages/open_ephys/analysis/recording.py", line 87, in events
    self.load_events()
  File "/home/x/miniconda3/envs/oe-python-plugin/lib/python3.10/site-packages/open_ephys/analysis/formats/NwbRecording.py", line 146, in load_events
    processor_id = int(dataset.split('.')[0].split('-')[1])
ValueError: invalid literal for int() with base 10: 'DAQmx'

That DAQmx string comes from the dataset name NI-DAQmx-101.PXIe-6363.TTL.  I think maybe NwbRecording is not expecting a processor name itself to contain a hyphen -, and so picking the wrong string part out of the dataset name.
A possible fix (and/or workaround) might be to look for the last string part instead of the second one?
processor_id = int(dataset.split('.')[0].split('-')[-1])

Thank you for the sweet tools!

Yeah, that should work! I will make that change and push an updated version to PyPI.

I'm using 0.1.8 now, thank you!


Refactor the main class

The main library class (SaltStackClient.java) has grown quite a bit, especially since the recent merge of #94. It could make sense to refactor this class now, moving away the new methods (callSync() and callAsync()) into a separate module of the library for instance.

Can you clarify term module?
The callSync() and callAsync() members could be moved as member functions of calls package taking SaltStackClient as parameter. So for example, WheelCall.java would have a callSync(SaltStackClient client) and similar async function.

Sorry for the late reply, I mean a separate class, while leaving it open how this should be designed. What would be important to me is that we create a nice API for the users of the library that allows to write beautiful and readable code. Thanks for your pull request, I will have a look and give feedback there.

Closing this issue after the merge of #120.


[Request Feature] Hot reload model and texture

Currently I have to exit and open editor to see the change of model and texture

Hot reload model implemented in build #198, reload texture still not supported.


Use table-based YUV-to-RGB conversion.

On HD video (crowd) img_to_rgb takes nearly 22% of the time, second after od_apply_qm. This patch reduces it to 6%.

@tmatth thanks for the review, I've addressed the comments and uploaded a patch to:
https://review.xiph.org/1026/
Closing this since it will be tracked on Reitveld instead.


Specification and Requirements to Run CogVideoX-5B model

I have a linux machine to do the research and testing for CogVideo. Below are the specifications for the linux machine.
GPU Model: NVIDIA L4
VRAM: 24GB
RAM: 16GB
It is enough to run the model?. I got an OOM Error when running just CogVideoX-2B.
ComfyUI Error Report
Error Details

Node Type: CogVideoDecode
Exception Type: torch.OutOfMemoryError
Exception Message: Allocation on device

Stack Trace
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 323, in execute
    output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)

  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 198, in get_output_data
    return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)

  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 169, in _map_node_over_list
    process_inputs(input_dict, i)

  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 158, in process_inputs
    results.append(getattr(obj, func)(**inputs))

  File "/home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-CogVideoXWrapper/nodes.py", line 987, in decode
    frames = vae.decode(latents).sample

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/utils/accelerate_utils.py", line 46, in wrapper
    return method(self, *args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 1278, in decode
    decoded = self._decode(z).sample

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 1249, in _decode
    z_intermediate, conv_cache = self.decoder(z_intermediate, conv_cache=conv_cache)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 970, in forward
    hidden_states, new_conv_cache[conv_cache_key] = up_block(

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 647, in forward
    hidden_states, new_conv_cache[conv_cache_key] = resnet(

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 291, in forward
    hidden_states, new_conv_cache["norm1"] = self.norm1(hidden_states, zq, conv_cache=conv_cache.get("norm1"))

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)

  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 187, in forward
    new_f = norm_f * conv_y + conv_b

System Information

ComfyUI Version: v0.2.7-6-g2865f91
Arguments: main.py
OS: posix
Python Version: 3.10.15 (main, Sep  7 2024, 18:35:33) [GCC 9.4.0]
Embedded Python: false
PyTorch Version: 2.5.1+cu121

Devices

Name: cuda:0 NVIDIA L4 : cudaMallocAsync

Type: cuda
VRAM Total:<PHONE_NUMBER>0
VRAM Free:<PHONE_NUMBER>6
Torch VRAM Total: 503316480
Torch VRAM Free: 61382186

Logs
2024-11-13 08:45:42,072 - root - INFO - Total VRAM 22491 MB, total RAM 15986 MB
2024-11-13 08:45:42,072 - root - INFO - pytorch version: 2.5.1+cu121
2024-11-13 08:45:42,073 - root - INFO - Set vram state to: NORMAL_VRAM
2024-11-13 08:45:42,073 - root - INFO - Device: cuda:0 NVIDIA L4 : cudaMallocAsync
2024-11-13 08:45:44,450 - root - INFO - Using pytorch cross attention
2024-11-13 08:45:46,596 - root - INFO - [Prompt Server] web root: /home/fahmie/comfyvm/ComfyUI/web
2024-11-13 08:45:48,122 - ComfyUI-CogVideoXWrapper.custom_cogvideox_transformer_3d - INFO - sageattn not found, using sdpa
2024-11-13 08:45:48,140 - ComfyUI-CogVideoXWrapper.cogvideox_fun.transformer_3d - INFO - sageattn not found, using sdpa
2024-11-13 08:45:48,141 - ComfyUI-CogVideoXWrapper.cogvideox_fun.fun_pab_transformer_3d - INFO - sageattn not found, using sdpa
2024-11-13 08:45:48,155 - root - INFO - Total VRAM 22491 MB, total RAM 15986 MB
2024-11-13 08:45:48,155 - root - INFO - pytorch version: 2.5.1+cu121
2024-11-13 08:45:48,155 - root - INFO - Set vram state to: NORMAL_VRAM
2024-11-13 08:45:48,155 - root - INFO - Device: cuda:0 NVIDIA L4 : cudaMallocAsync
2024-11-13 08:45:48,526 - root - INFO -
Import times for custom nodes:
2024-11-13 08:45:48,527 - root - INFO -    0.0 seconds: /home/fahmie/comfyvm/ComfyUI/custom_nodes/websocket_image_save.py
2024-11-13 08:45:48,528 - root - INFO -    0.0 seconds: /home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-KJNodes
2024-11-13 08:45:48,528 - root - INFO -    0.0 seconds: /home/fahmie/comfyvm/ComfyUI/custom_nodes/rgthree-comfy
2024-11-13 08:45:48,528 - root - INFO -    0.1 seconds: /home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-Manager
2024-11-13 08:45:48,529 - root - INFO -    0.3 seconds: /home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-VideoHelperSuite
2024-11-13 08:45:48,529 - root - INFO -    0.7 seconds: /home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-CogVideoXWrapper
2024-11-13 08:45:48,529 - root - INFO -
2024-11-13 08:45:48,542 - root - INFO - Starting server

2024-11-13 08:45:48,544 - root - INFO - To see the GUI go to: http://<IP_ADDRESS>:8188
2024-11-13 08:48:14,328 - root - INFO - got prompt
2024-11-13 08:48:14,959 - root - INFO - Requested to load SD3ClipModel_
2024-11-13 08:48:14,960 - root - INFO - Loading 1 new model
2024-11-13 08:48:14,974 - root - INFO - loaded completely 0.0 4541.693359375 True
2024-11-13 08:49:17,174 - ComfyUI-CogVideoXWrapper.pipeline_cogvideox - INFO - Temporal tiling and context schedule disabled
2024-11-13 08:49:23,326 - root - ERROR - !!! Exception during processing !!! backend='inductor' raised:
CalledProcessError: Command '['/usr/bin/gcc', '/tmp/tmpclz_9v3o/main.c', '-O3', '-shared', '-fPIC', '-o', '/tmp/tmpclz_9v3o/cuda_utils.cpython-310-x86_64-linux-gnu.so', '-lcuda', '-L/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/lib', '-L/lib/x86_64-linux-gnu', '-I/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/include', '-I/tmp/tmpclz_9v3o', '-I/usr/include/python3.10']' returned non-zero exit status 1.

Set TORCH_LOGS="+dynamo" and TORCHDYNAMO_VERBOSE=1 for more information

You can suppress this exception and fall back to eager by setting:
    import torch._dynamo
    torch._dynamo.config.suppress_errors = True

2024-11-13 08:49:23,331 - root - ERROR - Traceback (most recent call last):
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/output_graph.py", line 1446, in _call_user_compiler
    compiled_fn = compiler_fn(gm, self.example_inputs())
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/repro/after_dynamo.py", line 129, in __call__
    compiled_gm = compiler_fn(gm, example_inputs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/__init__.py", line 2234, in __call__
    return compile_fx(model_, inputs_, config_patches=self.config)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 1521, in compile_fx
    return aot_autograd(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/backends/common.py", line 72, in __call__
    cg = aot_module_simplified(gm, example_inputs, **self.kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_functorch/aot_autograd.py", line 1071, in aot_module_simplified
    compiled_fn = dispatch_and_compile()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_functorch/aot_autograd.py", line 1056, in dispatch_and_compile
    compiled_fn, _ = create_aot_dispatcher_function(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_functorch/aot_autograd.py", line 522, in create_aot_dispatcher_function
    return _create_aot_dispatcher_function(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_functorch/aot_autograd.py", line 759, in _create_aot_dispatcher_function
    compiled_fn, fw_metadata = compiler_fn(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/jit_compile_runtime_wrappers.py", line 179, in aot_dispatch_base
    compiled_fw = compiler(fw_module, updated_flat_args)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 1350, in fw_compiler_base
    return _fw_compiler_base(model, example_inputs, is_inference)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 1421, in _fw_compiler_base
    return inner_compile(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 475, in compile_fx_inner
    return wrap_compiler_debug(_compile_fx_inner, compiler_name="inductor")(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/repro/after_aot.py", line 85, in debug_wrapper
    inner_compiled_fn = compiler_fn(gm, example_inputs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 661, in _compile_fx_inner
    compiled_graph = FxGraphCache.load(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/codecache.py", line 1334, in load
    compiled_graph = compile_fx_fn(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 570, in codegen_and_compile
    compiled_graph = fx_codegen_and_compile(gm, example_inputs, **fx_kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/compile_fx.py", line 878, in fx_codegen_and_compile
    compiled_fn = graph.compile_to_fn()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/graph.py", line 1913, in compile_to_fn
    return self.compile_to_module().call
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/graph.py", line 1839, in compile_to_module
    return self._compile_to_module()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/graph.py", line 1845, in _compile_to_module
    self.codegen_with_cpp_wrapper() if self.cpp_wrapper else self.codegen()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/graph.py", line 1784, in codegen
    self.scheduler.codegen()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/scheduler.py", line 3383, in codegen
    return self._codegen()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/scheduler.py", line 3461, in _codegen
    self.get_backend(device).codegen_node(node)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/codegen/cuda_combined_scheduling.py", line 80, in codegen_node
    return self._triton_scheduling.codegen_node(node)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/codegen/simd.py", line 1155, in codegen_node
    return self.codegen_node_schedule(node_schedule, buf_accesses, numel, rnumel)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/codegen/simd.py", line 1364, in codegen_node_schedule
    src_code = kernel.codegen_kernel()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/codegen/triton.py", line 2661, in codegen_kernel
    **self.inductor_meta_common(),
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_inductor/codegen/triton.py", line 2532, in inductor_meta_common
    "backend_hash": torch.utils._triton.triton_hash_with_backend(),
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/utils/_triton.py", line 53, in triton_hash_with_backend
    backend = triton_backend()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/utils/_triton.py", line 45, in triton_backend
    target = driver.active.get_current_target()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/runtime/driver.py", line 23, in __getattr__
    self._initialize_obj()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/runtime/driver.py", line 20, in _initialize_obj
    self._obj = self._init_fn()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/runtime/driver.py", line 9, in _create_driver
    return actives[0]()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/driver.py", line 371, in __init__
    self.utils = CudaUtils()  # TODO: make static
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/driver.py", line 80, in __init__
    mod = compile_module_from_src(Path(os.path.join(dirname, "driver.c")).read_text(), "cuda_utils")
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/driver.py", line 57, in compile_module_from_src
    so = _build(name, src_path, tmpdir, library_dirs(), include_dir, libraries)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/runtime/build.py", line 48, in _build
    ret = subprocess.check_call(cc_cmd)
  File "/usr/lib/python3.10/subprocess.py", line 369, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/usr/bin/gcc', '/tmp/tmpclz_9v3o/main.c', '-O3', '-shared', '-fPIC', '-o', '/tmp/tmpclz_9v3o/cuda_utils.cpython-310-x86_64-linux-gnu.so', '-lcuda', '-L/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/lib', '-L/lib/x86_64-linux-gnu', '-I/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/include', '-I/tmp/tmpclz_9v3o', '-I/usr/include/python3.10']' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 323, in execute
    output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 198, in get_output_data
    return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 169, in _map_node_over_list
    process_inputs(input_dict, i)
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 158, in process_inputs
    results.append(getattr(obj, func)(**inputs))
  File "/home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-CogVideoXWrapper/nodes.py", line 866, in process
    latents = pipeline["pipe"](
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-CogVideoXWrapper/pipeline_cogvideox.py", line 875, in __call__
    noise_pred = self.transformer(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-CogVideoXWrapper/custom_cogvideox_transformer_3d.py", line 723, in forward
    hidden_states, encoder_hidden_states = block(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/eval_frame.py", line 465, in _fn
    return fn(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 1269, in __call__
    return self._torchdynamo_orig_callable(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 1064, in __call__
    result = self._inner_convert(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 526, in __call__
    return _compile(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 924, in _compile
    guarded_code = compile_inner(code, one_graph, hooks, transform)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 666, in compile_inner
    return _compile_inner(code, one_graph, hooks, transform)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_utils_internal.py", line 87, in wrapper_function
    return function(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 699, in _compile_inner
    out_code = transform_code_object(code, transform)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/bytecode_transformation.py", line 1322, in transform_code_object
    transformations(instructions, code_options)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 219, in _fn
    return fn(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/convert_frame.py", line 634, in transform
    tracer.run()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/symbolic_convert.py", line 2796, in run
    super().run()
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/symbolic_convert.py", line 983, in run
    while self.step():
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/symbolic_convert.py", line 895, in step
    self.dispatch_table[inst.opcode](self, inst)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/symbolic_convert.py", line 580, in wrapper
    return handle_graph_break(self, inst, speculation.reason)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/symbolic_convert.py", line 649, in handle_graph_break
    self.output.compile_subgraph(self, reason=reason)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/output_graph.py", line 1142, in compile_subgraph
    self.compile_and_call_fx_graph(tx, pass2.graph_output_vars(), root)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/output_graph.py", line 1369, in compile_and_call_fx_graph
    compiled_fn = self.call_user_compiler(gm)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/output_graph.py", line 1416, in call_user_compiler
    return self._call_user_compiler(gm)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/_dynamo/output_graph.py", line 1465, in _call_user_compiler
    raise BackendCompilerFailed(self.compiler_fn, e) from e
torch._dynamo.exc.BackendCompilerFailed: backend='inductor' raised:
CalledProcessError: Command '['/usr/bin/gcc', '/tmp/tmpclz_9v3o/main.c', '-O3', '-shared', '-fPIC', '-o', '/tmp/tmpclz_9v3o/cuda_utils.cpython-310-x86_64-linux-gnu.so', '-lcuda', '-L/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/lib', '-L/lib/x86_64-linux-gnu', '-I/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/triton/backends/nvidia/include', '-I/tmp/tmpclz_9v3o', '-I/usr/include/python3.10']' returned non-zero exit status 1.

Set TORCH_LOGS="+dynamo" and TORCHDYNAMO_VERBOSE=1 for more information

You can suppress this exception and fall back to eager by setting:
    import torch._dynamo
    torch._dynamo.config.suppress_errors = True

2024-11-13 08:49:23,333 - root - INFO - Prompt executed in 68.99 seconds
2024-11-13 08:49:47,912 - root - INFO - got prompt
2024-11-13 08:50:11,802 - ComfyUI-CogVideoXWrapper.pipeline_cogvideox - INFO - Temporal tiling and context schedule disabled
2024-11-13 08:55:25,214 - root - ERROR - !!! Exception during processing !!! Allocation on device
2024-11-13 08:55:25,221 - root - ERROR - Traceback (most recent call last):
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 323, in execute
    output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 198, in get_output_data
    return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 169, in _map_node_over_list
    process_inputs(input_dict, i)
  File "/home/fahmie/comfyvm/ComfyUI/execution.py", line 158, in process_inputs
    results.append(getattr(obj, func)(**inputs))
  File "/home/fahmie/comfyvm/ComfyUI/custom_nodes/ComfyUI-CogVideoXWrapper/nodes.py", line 987, in decode
    frames = vae.decode(latents).sample
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/utils/accelerate_utils.py", line 46, in wrapper
    return method(self, *args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 1278, in decode
    decoded = self._decode(z).sample
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 1249, in _decode
    z_intermediate, conv_cache = self.decoder(z_intermediate, conv_cache=conv_cache)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 970, in forward
    hidden_states, new_conv_cache[conv_cache_key] = up_block(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 647, in forward
    hidden_states, new_conv_cache[conv_cache_key] = resnet(
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 291, in forward
    hidden_states, new_conv_cache["norm1"] = self.norm1(hidden_states, zq, conv_cache=conv_cache.get("norm1"))
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/fahmie/comfyvm/ComfyUI/venv/lib/python3.10/site-packages/diffusers/models/autoencoders/autoencoder_kl_cogvideox.py", line 187, in forward
    new_f = norm_f * conv_y + conv_b
torch.OutOfMemoryError: Allocation on device

2024-11-13 08:55:25,222 - root - ERROR - Got an OOM, unloading all loaded models.
2024-11-13 08:55:25,437 - root - INFO - Prompt executed in 337.51 seconds

Attached Workflow
Please make sure that workflow does not contain any sensitive information such as API keys or passwords.
{"last_node_id":34,"last_link_id":64,"nodes":[{"id":30,"type":"CogVideoTextEncode","pos":{"0":500,"1":308},"size":{"0":413.2752380371094,"1":248.78025817871094},"flags":{},"order":2,"mode":0,"inputs":[{"name":"clip","type":"CLIP","link":54}],"outputs":[{"name":"conditioning","type":"CONDITIONING","links":[61],"slot_index":0,"shape":3},{"name":"clip","type":"CLIP","links":null}],"properties":{"Node name for S&R":"CogVideoTextEncode"},"widgets_values":["A golden retriever, sporting sleek black sunglasses, with its lengthy fur flowing in the breeze, sprints playfully across a rooftop terrace, recently refreshed by a light rain. The scene unfolds from a distance, the dog's energetic bounds growing larger as it approaches the camera, its tail wagging with unrestrained joy, while droplets of water glisten on the concrete behind it. The overcast sky provides a dramatic backdrop, emphasizing the vibrant golden coat of the canine as it dashes towards the viewer.\n\n",1,true]},{"id":31,"type":"CogVideoTextEncode","pos":{"0":501,"1":598},"size":{"0":411.2016296386719,"1":244.37738037109375},"flags":{},"order":3,"mode":0,"inputs":[{"name":"clip","type":"CLIP","link":56}],"outputs":[{"name":"conditioning","type":"CONDITIONING","links":[62],"slot_index":0,"shape":3},{"name":"clip","type":"CLIP","links":null}],"properties":{"Node name for S&R":"CogVideoTextEncode"},"widgets_values":["blur, watermark. ",1,true]},{"id":33,"type":"VHS_VideoCombine","pos":{"0":1320,"1":204},"size":[504.50164794921875,310],"flags":{},"order":6,"mode":0,"inputs":[{"name":"images","type":"IMAGE","link":59},{"name":"audio","type":"AUDIO","link":null,"shape":7},{"name":"meta_batch","type":"VHS_BatchManager","link":null,"shape":7},{"name":"vae","type":"VAE","link":null,"shape":7}],"outputs":[{"name":"Filenames","type":"VHS_FILENAMES","links":null,"shape":3}],"properties":{"Node name for S&R":"VHS_VideoCombine"},"widgets_values":{"frame_rate":8,"loop_count":0,"filename_prefix":"CogVideoX5B","format":"video/h264-mp4","pix_fmt":"yuv420p","crf":19,"save_metadata":true,"pingpong":false,"save_output":false,"videopreview":{"hidden":false,"paused":false,"params":{"filename":"CogVideoX5B_00002.mp4","subfolder":"","type":"temp","format":"video/h264-mp4","frame_rate":8},"muted":false}}},{"id":20,"type":"CLIPLoader","pos":{"0":101,"1":399},"size":{"0":352.55279541015625,"1":82},"flags":{},"order":0,"mode":0,"inputs":[],"outputs":[{"name":"CLIP","type":"CLIP","links":[54,56],"slot_index":0,"shape":3}],"properties":{"Node name for S&R":"CLIPLoader"},"widgets_values":["t5xxl_fp8_e4m3fn.safetensors","sd3"]},{"id":34,"type":"CogVideoSampler","pos":{"0":969,"1":113},"size":{"0":315.8404846191406,"1":410},"flags":{},"order":4,"mode":0,"inputs":[{"name":"pipeline","type":"COGVIDEOPIPE","link":60},{"name":"positive","type":"CONDITIONING","link":61},{"name":"negative","type":"CONDITIONING","link":62},{"name":"samples","type":"LATENT","link":null,"shape":7},{"name":"image_cond_latents","type":"LATENT","link":null,"shape":7},{"name":"context_options","type":"COGCONTEXT","link":null,"shape":7},{"name":"controlnet","type":"COGVIDECONTROLNET","link":null,"shape":7},{"name":"tora_trajectory","type":"TORAFEATURES","link":null,"shape":7},{"name":"fastercache","type":"FASTERCACHEARGS","link":null,"shape":7}],"outputs":[{"name":"cogvideo_pipe","type":"COGVIDEOPIPE","links":[63],"shape":3},{"name":"samples","type":"LATENT","links":[64],"shape":3}],"properties":{"Node name for S&R":"CogVideoSampler"},"widgets_values":[480,720,49,50,6,806286757407563,"fixed","CogVideoXDDIM",1]},{"id":11,"type":"CogVideoDecode","pos":{"0":981,"1":623},"size":{"0":300.396484375,"1":198},"flags":{},"order":5,"mode":0,"inputs":[{"name":"pipeline","type":"COGVIDEOPIPE","link":63},{"name":"samples","type":"LATENT","link":64}],"outputs":[{"name":"images","type":"IMAGE","links":[59],"slot_index":0,"shape":3}],"properties":{"Node name for S&R":"CogVideoDecode"},"widgets_values":[false,120,360,0.2,0.2,true]},{"id":1,"type":"DownloadAndLoadCogVideoModel","pos":{"0":594,"1":42},"size":{"0":315,"1":214},"flags":{},"order":1,"mode":0,"inputs":[{"name":"pab_config","type":"PAB_CONFIG","link":null,"shape":7},{"name":"block_edit","type":"TRANSFORMERBLOCKS","link":null,"shape":7},{"name":"lora","type":"COGLORA","link":null,"shape":7},{"name":"compile_args","type":"COMPILEARGS","link":null,"shape":7}],"outputs":[{"name":"cogvideo_pipe","type":"COGVIDEOPIPE","links":[60],"slot_index":0,"shape":3}],"properties":{"Node name for S&R":"DownloadAndLoadCogVideoModel"},"widgets_values":["THUDM/CogVideoX-2b","fp16","disabled","disabled",false]}],"links":[[54,20,0,30,0,"CLIP"],[56,20,0,31,0,"CLIP"],[59,11,0,33,0,"IMAGE"],[60,1,0,34,0,"COGVIDEOPIPE"],[61,30,0,34,1,"CONDITIONING"],[62,31,0,34,2,"CONDITIONING"],[63,34,0,11,0,"COGVIDEOPIPE"],[64,34,1,11,1,"LATENT"]],"groups":[],"config":{},"extra":{"ds":{"scale":0.6830134553650705,"offset":[-157.53971682235178,-5.75114406010195]}},"version":0.4}

Additional Context
(Please add any additional context or steps to reproduce the error here)

That amount of RAM may not be enough, to avoid running out of memory with the VAE you should enable the tiling option.

That amount of RAM may not be enough, to avoid running out of memory with the VAE you should enable the tiling option.

ouhhh, so its mean i need to increase my RAM more? not Vram?

@Fahmie23 runs fine on 24 GB VRAM usually. Exceptions might be misleading cause might be caused by preceding nodes.
Your workflow seems lacked "Empty Latent Image". Also just in case Fix nodes (re-create), then reconnect missing links.
Just make sure use ComfyUI Manager to: 1. Update ComfyUI ➡ 2. Update All ➡ 3. Restart ComfyUI
I've updated workflow for you:
229_workflow.json


DOC: write down a spec for data format

learn from https://asdf-standard.readthedocs.io/en/latest/
consider also (from https://janclemenslab.org/das/technical/data_formats.html in DAS docs):
https://cyrille.rossant.net/moving-away-hdf5/
https://cyrille.rossant.net/should-you-use-hdf5/
https://github.com/bastibe/jbof

And furthermore
https://jams.readthedocs.io/en/stable/jams_structure.html


Check if input is handled before handling it

If any _input() method marked input as handled, it's not propagated further to _gui_input.
If any Viewport marked input as handled, no other Viewport will handle it.

I'm not sure but this could fix #6194


Sync with development branch

Coverage increased (+21.6%) to 45.865% when pulling 594f1e86f5bd0196ed79495bdaa4190a70ddd6f1 on alvarolopez:master into ee96428a5424e60907259efd400ff3237e2dbca0 on IFCA:master.

Coverage increased (+21.6%) to 45.865% when pulling ef3e8ee72b3b1d83e05f13ddc882c8ac7a6e30b9 on alvarolopez:master into ee96428a5424e60907259efd400ff3237e2dbca0 on IFCA:master.

Coverage increased (+21.6%) to 45.865% when pulling d1c80ed9c2156ffa8992b232504bc34bdfc9c00d on alvarolopez:master into ee96428a5424e60907259efd400ff3237e2dbca0 on IFCA:master.

Coverage increased (+21.6%) to 45.865% when pulling d1c80ed9c2156ffa8992b232504bc34bdfc9c00d on alvarolopez:master into ee96428a5424e60907259efd400ff3237e2dbca0 on IFCA:master.

Coverage increased (+19.6%) to 43.828% when pulling 586484d44da97c2ac6b8673f62b2f76835bed8a7 on alvarolopez:master into ee96428a5424e60907259efd400ff3237e2dbca0 on IFCA:master.

Coverage increased (+19.6%) to 43.828% when pulling 178ee509be3713a0ea033d6f4763cdf5a42b1b5f on alvarolopez:master into ee96428a5424e60907259efd400ff3237e2dbca0 on IFCA:master.


[Bug]: Buscript fails to load when world-container is modified

/mv version -p output
https://paste.gg/p/anonymous/521ec08fc96e476581ac2c349eeee9bc
Server logs
https://paste.gg/p/anonymous/4a7131cdd0524532bb0c3ca971ad0648
Server Version
[19:04:08 INFO]: Current: git-Purpur-2149 (MC: 1.20.4)*
Previous: git-Purpur-2143 (MC: 1.20.4)
* You are running the latest version

Bug Description
Buscript fails to load if the world container is anything but the server's root directory.
The beggining of my bukkit.yml:
settings: world-container: worlds
My world container structure:

Steps to reproduce

Add settings: world-container: worlds  to bukkit.yml
Relocate worlds to that container
Start the server (with Multiverse-Core enabled)

Agreements

[X] I have searched for and ensured there isn't already an open or resolved issue(s) regarding this.
[x] I was able to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.

I know it sounds silly… can you confirm that you actually want to use buscript. It’s quite old now and will most likely be removed soon. You can disable the warning in config if that is what you need


Consume

Codecov Report
Base: 99.52% // Head: 99.52% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (b78e2a1) compared to base (6a2a8df).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #212   +/-   ##
=======================================
  Coverage   99.52%   99.52%
=======================================
  Files          50       51    +1
  Lines        1674     1697   +23
  Branches      347      350    +3
=======================================
+ Hits         1666     1689   +23
  Misses          3        3
  Partials        5        5

Impacted Files
Coverage Δ

src/index.ts
100.00% <100.00%> (ø)

src/ops/consume.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


MethodActuator: fixed issue where complete() would call method with uninitialized values if not updated at least once before stopping

By the way, the following code demonstrates how to reproduce this issue:
var shape = new Shape();
shape.graphics.beginFill(0xff0000);
shape.graphics.drawCircle(20.0, 20.0, 20.0);
shape.graphics.endFill();
addChild(shape);

var tween = Actuate.update((x) -> {
	trace(x);
	shape.x = x;
}, 5.0, [0], [100]);
Actuate.stop(tween);

The function passed to Actuate.update() is incorrectly called with null as the argument value.


Some changes in KEY_MAPS for Studio

Studio's PR - https://github.com/DevExpress/testcafe-studio/pull/3122

Space key property updated for IE
reversedKeyProperty added

@AlexKamaev
Ok, I'll organize consts and yes, @miherlosev said that I'll need a rebase after some fix.


fix(certs): revoke Kafka certificate even when the Kafka is not assigned to a data plane yet

Description

Revoke Kafka certificate even when the Kafka is not assigned to a data plane yet.
This could typically happen when a Kafka is created and deleted right away (before the accepted kafka reconciler runs) when KFM is running in auto mode.
Verification Steps

Code review and Unit test passing
Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

Thanks for the review @miguelsorianod


Moving upload coverage report logic to createcoveragereport.sh

@jskeet PTAL, I've addressed your comments.
Also, I exported the CODECOV_TOKEN, something that we missed yesterday.
Thanks!

@jskeet PTAL (again :)) Thanks!

Thanks!


terragrunt providers mirror doesn't work

The terraform providers mirror dir command doesn't work in terragrunt terraform providers mirror dir

Hello,
will be helpful to get more details on how is intended to use terragrunt in this case - it is to be used in an "offline" environment, without internet access?

I have my own use case that is why terraform has this feature.. Terragrunt is advertised that all terraform sub commands work so please fix this sub command so it works

Hello, I'm also curious about this feature. I need to use Terragrunt in an envrionment with very strict security posturing which blocks most external hosts including hashicorp provider sources.

Make sure to specify an absolute directory, otherwise, it will mirror the providers into the Terragrunt cache directory.


Not support angular 2.2.1

Hello guys,
I am working with angular 2.
package.json:
"dependencies": {
    "@angular/common": "~2.2.1",
    "@angular/compiler": "~2.2.1",
    "@angular/compiler-cli": "~2.2.1",
    "@angular/core": "~2.2.1",
    "@angular/forms": "~2.2.1",
    "@angular/http": "~2.2.1",
    "@angular/platform-browser": "~2.2.1",
    "@angular/platform-browser-dynamic": "~2.2.1",
    "@angular/platform-server": "~2.2.1",
    "@angular/router": "~3.2.1",
    "@angular/upgrade": "~2.2.1",
    "angular2-in-memory-web-api": "0.0.21",
    "bootstrap": "^3.3.7",
    "core-js": "^2.4.1",
    "ie-shim": "^0.1.0",
    "ng2-smart-table": "^0.4.0-8",
    "reflect-metadata": "^0.1.3",
    "rxjs": "5.0.0-beta.12",
    "zone.js": "~0.6.26"
  }

Trying to install ng2-smart-table and its installed successfully.
but when I trying to run my project it gives me an error.
Error:
Module parse failed: /home/tushar/Yugma-Web/node_modules/ng2-smart-table/build/src/ng2-smart-table/components/cell/cell.scss Unexpected token (1:6)
You may need an appropriate loader to handle this file type.
| input {
|   width: 100%;
|   line-height: normal;
 @ ./~/ng2-smart-table/build/src/ng2-smart-table/components/cell/cell.component.js 89:21-43
 @ ./~/ng2-smart-table/build/src/ng2-smart-table.module.js
 @ ./~/ng2-smart-table/build/ng2-smart-table.js
 @ ./src/app/app.module.ts
 @ ./src/main.browser.ts
 @ multi main

Please solve my problem as soon as possible.
Thank you

Hey @Tusharbalar, you can install and setup sass-loader if you use webpack.


Release 1.3.5

Coverage remained the same at 90.06% when pulling bd98c41ea163a6e283fb98d4c5be10e7fcd508b1 on release/1.3.5 into 8bd1b1c211138e58ca4771f8a4d6ad878b32d4d3 on master.


PowerShell Core support?

Hi, thanks for this brilliant project. Example works well, all looks good and straightforward.
Wonder if there are plans to support PowerShell Core.
Could not get it running with PS Core 6.2, not sure why - didn't dig deeper.
Name                           Value
----                           -----
PSVersion                      6.2.3
PSEdition                      Core
GitCommitId                    6.2.3
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Hi, thanks for feedback 😊
I would definitely like to add support for PowerShell Core. This is one of my favorite projects and I think it deserves some maintenance(it's been some time since I last commited here :disappointed:). Give me some time, I'll hopefully manage to add that support and I will notify you. Until then, have a nice day and happy coding :thumbsup:

Noted, thanks @TomasBouda

Hi @avishnyakov,
good news, I have version of PoShLog that works on PowerShell 7 :wink:
Once I'm done with polishing/testing/documenting I will release it. Hopefully till the end of next week :smirk:
If you are intrested you can download preview with:
Install-Module -Name PoShLog -AllowPrerelease

and try quick example:
Import-Module PoShLog

# Create and start new file and console logger - new short version
Start-Logger -FilePath 'C:\Data\my_awesome.log' -Console

Write-InfoLog 'Hurrray, my first log message!'

Close-Logger

or you can check the code in dev branch.
If you have any questions or problems running the code, please let me know :relaxed:
Have a nice day and happy coding :rocket:

Hi @avishnyakov,
if you are still intrested, I just published new version of PoShLog - https://www.powershellgallery.com/packages/PoShLog/2.0.0
Please let me know if it works for you. Otherwise I will close this issue after one week.
Happy coding 🚀

@TomasBouda noted, thanks for sharing this.


Deprecate support for<EMAIL_ADDRESS>

In the package.json we note that we support<EMAIL_ADDRESS>Running the tests for the repo under that node version yields errors.

I say we deprecate it since it will not longer be supported soon. Plus since it currently fails tests, we can either make the code pass, or just remove it as a supported version.

Yeah, 0.10 falls out of maintenance in October so I don't see any reason to support it. Related, node 0.12 is out of active LTS as of April and falls out of maintenance in December so I wouldn't bother with that either.
I say just jump straight to 6. 👌

Do we want to refactor to use es6 features, too?

I say yeah

4.x LTS will be supported thru April 2018, maybe we continue supporting that for a bit before purposefully rewriting working code to drop support...
Totally on board with dropping support for < 4.x. 👍


Portman GT3000 tracker support

Hello,
is there support for Portman GT3000 trackers?
Packet format is here: http://www.aggsoft.com/gps-tracker-data-logger/gps-tracker-portman-gprs.htm
Regards

It's not supported yet, as far as I can tell. If you want to add support, please send me an email to discuss options.


[YUNIKORN-1431] "Run Flink Jobs" unresolved link

What is this PR for?
I changed the invalid URL to "Overview" in page  "Run Flink Jobs"
What type of PR is it?

[x] - Task

What is the Jira issue?

Open an issue on Jira https://issues.apache.org/jira/browse/YUNIKORN-1431/

How should this be tested?
./local-build.sh run
Screenshots (if appropriate)

Change this as well as [Overview](./workload_overview.md). to see the [overview](./workload_overview.md) for the label specification.

Thank for your review. I have modified it.


chore(ci): Add Node 22 to CI and fix dependabot PRs

Description

Add Node 22 to CI
Don't run push events on Dependabot PRs because it can break things
Add CodeQL scanning

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.55%. Comparing base (d77f379) to head (1114ef1).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #282      +/-   ##
==========================================
- Coverage   83.62%   83.55%   -0.07%
==========================================
  Files          14       12       -2
  Lines         519      517       -2
==========================================
- Hits          434      432       -2
  Misses         85       85

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Documentation enhancement: Explain how to open calendar view

I installed the plugin, no errors were reported - but I'm unable to switch to any calendar view.
There is no content menu entry on the daily notes folder, there does not seem to be a hotkey.
Would be great if the documentation not only showed the plugin in action but also how to actually use it.
(I'm running Obsidian 0.9.10.)

@ralfw Hi, by default the calendar view will appear in the right sidebar, so make sure you have that expanded.

I will make some updates to the documentation to make it more clear, as well as adding an option to the command palette to show it.

Ah, great! Yes, this works.
Would be cool to get a small hint at  that in the documentation ;-)


ocamlPackages.uecc: init at 0.3

Motivation for this change
This is a continuation of PR #129444, #129509, #129511, and #129513 which are prerequisites to make LIGO build under nix.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

@vbgl this PR is now updated to use fetchFromGitLab


Remove confirmation steps during install

apt-get -y etc

it could have taken 2 sec to add it to the script rather than creating this.
why was this created?
for documentation purposes???

I did not have the time to do it so I created an issue in order to fix this properly. Otherwise this is how you get a buggy software.

This way it will take a Month to write a script

https://github.com/angristan/wireguard-install/pull/46

Fixed by https://github.com/angristan/wireguard-install/commit/b9da4d03cdac26430737ed1cbaed37bfb66e8c9f


slow maps on visualize PROD

Describe the bug
For some odd reason on visualize PROD the same GraphQL triggering the same SPARQL query has much longer response time.
To Reproduce
Steps to reproduce the behavior:

Go to https://visualize.admin.ch/de/v/lpFHNCtYBCaE?dataSource=Prod&flag__debug=true
Open debug panel, open GeoShapesByDimensionIri, see first SPARQL query
See it takes ~10s, while the same dataset viewed trough visualize TEST or INT triggering the same SPARQL query takes < 1s

Expected behavior
Equally fast response time as TEST or INT - or an explanation why timing would be biased on PROD...
Screenshots or video

Environment (please complete the following information):

Visualize environment and version: PROD v3.22.9 (a09cdc3)

Additional context
the very same SPARQL query is fired to the same geo endpoint. direct execution behaves fast. (<1s)
SPARQL query on geo endpoint (identically fired from TEST, INT and PROD)
@bprusinowski any plausible explanation about this being related to visualize code? i.e. based on how the query time is measured ...

Hi @Rdataflow, I checked the file where we query geo shapes and the last change (that improved performance a lot) was done in this commit: https://github.com/visualize-admin/visualization-tool/commit/249d1d530b14a97e90ca0a02e2d064d30ec46f12, which was included in 3.21.1 version, so it's also there on PROD.
I did not find anything related to the code / way of measuring the time that could case such huge decrease in performance. I guess we could see if things improve once we have a new deployment to PROD; but it also might be worth to check the infrastructure somehow. But it's strange that's it's only a single query that's affected 🤔

For the records:

Der Server war am Swappen.


Hotfix/docker development

@tothtamas28 This has produced a non issue image . I'd like to run through a formal run and determine if we can

Pushing to internal access registry on the left is an attempt to test if there is a caching issue with the internal pull through / caching server

The --no-cache will not allow buildkit to use existing container layers to build the image; after a test I feel more confident this couldbe the issue; I was unable to reproduce the problem with our internal registry reliably to find an alternative solution.
Docker does not change images after build and during push so this does not 100% explain the issue of working on the built machine but fails once pulled from dockerhub remote. I have noticed the image layers are not the same post push and pull.
Why this is, is unclear; and unable to reproduce currently without using the remote dockerhub (internal registry did not do this.)
--no-cache produced a valid image without permission issues after pull down on a client. Using no cache will at least make sure each build uses fresh layers and does not rely on any remote or locally corrupted layers that could be causing this issue.


Could not find info regarding possible exceptions

[It would be very useful to see what exceptions can be thrown. If it exist, maybe it should be easier to find. I think this is relevant infos...]

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

ID: c5330d01-b6f6-80b1-d93a-5bc538261379
Version Independent ID: df047e90-b65b-b835-02f8-ed59c613c636
Content: BlobServiceClient Class (Azure.Storage.Blobs) - Azure for .NET Developers
Content Source: xml/Azure.Storage.Blobs/BlobServiceClient.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@UmbrellaCorpAgent Most operations raise a RequestFailedException and the references for the methods mention the same in the remarks section like this. You should be able to identify the exact problem by cross referencing the error code with the ones documented.


Add extenders API

Extenders are functions that are part of the pop that extend the pop in a specific way when called.
For example:
row = pop {
  defaults.panels = [ ];
  extenders.addPannel = panel: self: super: {
    panels = super.panels ++ [ panel ];
  };
}
(row.addPanel {}).addPanel {}

Would return:
{ __meta__ = ...; panels = [ {} {} ]; addPanel = <lambda>; }

I also added an unpop function to reside within a pop, so you can do p.__unpop__ instead of lib.unpop p.
TODO

[ ] Type comments for new and changed functions

I like the general idea, but there are many small issues I'd like to discuss.

rebased and updated with your feedback, I still have to write tests for these changes

Existing code allows for this already, so no need for this
pop {
  defaults.value = 1;
  extension = self: super: {
    incValue = num: extendPop self (self: super: { value = super.value + num; });
  };
}


Update versions.support and versions.paging, make the sample compilable.

The code in this project was previously updated to use support library version 27.1.0 and paging library 1.0.0-alpha6. However the versions.gradle still referenced older versions of those libraries, making the project uncompilable.

I signed it!

I had to close this pull request, as the commit was not done with my email address. I created a new proper pull request.


Cell overflow caused by contract's specific state layout

In some very specific case the compiler produces code that fails on-chain with exit code 8 "Cell overflow" (in emulator as well).
import "@stdlib/deploy";

message SetCost {
    cost: Int? as coins;
}

struct Struct2 {
    c: String;
    d: String;
    e: String;
    f: String;
}

contract SampleTactContract with Deployable {
    owner: Address;
    seqno: Int as uint256 = 0;
    a: Int as uint256 = 0;
    b: Int as uint256 = 0;

    struct2: Struct2;

    author3: Address?;

    cost: Int? as coins;
    address: Address?;
    price: Int? as coins;

    init(owner: Address, struct2: Struct2) {
        self.owner = owner;
        self.struct2 = struct2;
    }

    receive(msg: SetCost) {
        self.cost = msg.cost;
        self.reply("Cost is updated".asComment());
    }
}

Repository with a minimal reproducible setup: https://github.com/ak3n/cell-overflow-bug

+1

Thanks for such a helpful bug report with a repro! Really appreciate it.
We just released a new dev version of Tact<EMAIL_ADDRESS>with this issue fixed. And we prepared a wiki page with the recipe how to switch to a dev version of Tact: https://github.com/tact-lang/tact/wiki/Using-unreleased-versions-of-tools-early#dev-versions-of-tact-with-blueprint. Hope it helps. If you could try it out before our official 1.4.1 release I'd be very grateful


qgis: 3.10.4 -> 3.10.7

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 91283 1

2 packages built:
- qgis
- qgis-unwrapped


Error when creating

When trying set up the page the first time (calling simply 'hugo') I receive the following error:
ERROR 2023/04/27 11:56:39 render of "taxonomy" failed: "/home/mhaelsig/QGIS/QGIS_Server/HTML/rwpl/arberia/themes/arberia/layouts/_default/baseof.html:8:4": execute of template failed: template: categories/terms.html:8:4: executing "categories/terms.html" at <partial "footer" .>: error calling partial: "/home/mhaelsig/QGIS/QGIS_Server/HTML/rwpl/arberia/themes/arberia/layouts/partials/footer.html:5:17": execute of template failed: template: partials/footer.html:5:17: executing "partials/footer.html" at <len .Site.Menus.footer>: error calling len: reflect: call of reflect.Value.Type on zero Value
I am new to Hugo, therefore I am not able to fix this :--((
I am using hugo v0.111.3.

Sorry - typical beginners issue.


Boost libraries not found.

Building on CentOS 7.
I've built Boost 1.66 and set BOOST_ROOT and BOOST_LIBRARYDIR environment variables, as well as OPENPOSE_ROOT appropriately.
What do I need to change to get rid of this error?
--- start ---
CMake Error at /usr/local/share/cmake-3.10/Modules/FindBoost.cmake:1928 (message):
Unable to find the requested Boost libraries.
Boost version: 1.53.0
Boost include path: /usr/include
Could not find the following Boost libraries:
      boost_python-py27

Some (but not all) of the required Boost libraries were found.  You may
need to install these additional Boost libraries.  Alternatively, set
BOOST_LIBRARYDIR to the directory containing Boost libraries or BOOST_ROOT
to the location of Boost.
--- end ---
Thank you.

Is boost built with python2.7 support? the missing library should be in the boost installation folder or the system folder if you made a system wide install.

Yes, Python 2.7 is installed and is the default Python installation.
I did "./bootstrap.sh --prefix=/usr/local/boost166" and then "./b2 install" to build and install Boost, and the /usr/local/boost166 folder contains the "include" and "lib" folders.

If the libboost_python.so is built correctly try to change the python-py27 in the find_package line for boost in the CMakeLists.txt to "python"
By the way I have not tested PyOpenpose with boost166 so let me know if it works!

Changing "python-py27" to "python" worked.
Now it can't find "-lcaffe".
I have /etc/ld.so.conf.d/caffe-x86_64.conf with the proper path to the lib folder, but that's not helping.
How to resolve this error?
Thank you.

Caffe installation folders are expected to be under OPENPOSE_ROOT. Check lines 70 and 71 in the CMakeLists.txt.
Either change the paths to the installation folder for your caffe or set the OPENPOSE_ROOT as needed.

I changed those two lines, pointing directly at the Caffe install. (The
OpenPose folks suggested that I build Caffe separately, not within
OpenPose. Your CMake rules don't take that into account.)
It built successfully.
Thank you.
Best,
Stephen
On Wed, Mar 7, 2018 at 1:33 PM, Panteleris Paschalis <
<EMAIL_ADDRESS>wrote:

Caffe installation folders are expected to be under OPENPOSE_ROOT. Check
lines 70 and 71 in the CMakeLists.txt.
Either change the paths to the installation folder for your caffe or set
the OPENPOSE_ROOT as needed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FORTH-ModelBasedTracker/PyOpenPose/issues/37#issuecomment-371292491,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVJt_0ikZGSuoQ_bbAktxVhh1GLSitmAks5tcFIRgaJpZM4Sg2A0
.

--
Stephen Spencer
e-mail<EMAIL_ADDRESS>


Beamer: undefined control sequence

I'm getting the same error as #99, but the version of the eisvogel template I have was downloaded after that was fixed. The exact version of the template is here.
Any ideas as to what's going on?
Test file:
---
title: test
beamer: true
---

# this is a test

some content

Command invocation and output:
$ pandoc --template eisvogel --listings -F pandoc-crossref --citeproc --csl=$HOME/.local/share/pandoc/template/ieee.csl -o test.pdf test.md
Error producing PDF.
! Undefined control sequence.
<recently read> \setbeamertemplate

l.55 \setbeamertemplate

Pandoc version:
$ pandoc -v
pandoc <IP_ADDRESS>

change line 81 to
]{$if(beamer)$beamer$else$$if(book)$scrbook$else$scrartcl$endif$$endif$}


63 changing the automated navigation to a manual navigation

Can you tell me if you like the way I've organized the different sections?

The main navigation should follow the following order:
Overview
Protocol
Nodes
Tools
Libraries
Computing
Guides
Guides could be renamed Tutorials :thinking:

Okay! Do you want me to wait for the confirmation of Mike before pushing the changes?


Add AMSR2 snow depth DA to NoahMP.4.0.1

Description
LIS DA plugin updated to support AMSR2 snow depth DA and NoahMP401. GCOMW_AMSR2L3SND reader also updated to fix a unit mismatch between NoahMP401 and AMRS2. Use of MOD10C1 collection 5 has been replaced with collection 6 to support DA through the present.
Resolves #758

Testcase
Testcase available within EIS Freshwater project space
/discover/nobackup/project/eis_freshwater/mwrzesie/DA_AMSR2_snowdepth

Hi @mlwrzesien, the code looks good and I'm able to reproduce your output so I will go ahead and merge. I'm including your note about these changes for context when we review changes for merger into the master branch after the EIS pilot concludes:

One concern is that I replaced the MOD10C1 collection 5 reader with one for MOD10C1 collection 6 within the AMSR2 code since collection 5 support ended a few years ago. however, I wasn't sure the best way to keep both C5 and C6 since I think it would have required an update to the lis.config file for AMSR2 DA.


docs: update spec/asyncapi.md

title: "update spec/asyncapi.md"

Related issue(s):
N/A

correct article agreement
correct punctuation
correct grammar
reduce verbiage
use active voice vs. passive voice to increase readability
correct spelling

@alequetzalli hey, can I ask you for a favor? can you have a look at the proposed changes? I think they are correct and make sense but I would like expert opinion to make sure these changes do not imply different meaning of changed text 😅

@vladdoster can you have a look at suggestion from @alequetzalli  and solve conflicts and then we can merge

@vladdoster are you willing to continue with this PR?
As @derberg mentioned there are some action items for this PR to proceed further:

@vladdoster can you have a look at suggestion from @alequetzalli and solve conflicts and then we can merge

@char0n Yes, I will apply the suggested change and squash commits. Sorry for delay.

@vladdoster cool. Along with suggested change, conflicts needs to resolved as well.

@vladdoster do you want to continue with this one? there are some important questions from @dalelane

@vladdoster do you plan to continue with this one or shall we ask other contributors to take over?

Since @vladdoster does not answer anymore, I plan to close this PR and instead open an issue so someone can pick it up from where it was left. Maybe marked as a good first issue btw.
WDYT @alequetzalli @derberg @dalelane ?

Since @vladdoster does not answer anymore, I plan to close this PR and instead open an issue so someone can pick it up from where it was left. Maybe marked as a good first issue btw.
WDYT @alequetzalli @derberg @dalelane ?

Makes sense to me 👍

I see you @derberg closed this PR. Are you planning to open a new issue as I suggested we could do?


@ionic/core is not transpiled to ES5

Bug Report
Ionic Info
Ionic:
ionic (Ionic CLI)          : 4.1.1
Ionic Framework            : @ionic/angular 4.0.0-beta.6
@angular-devkit/core       : 0.7.5
@angular-devkit/schematics : 0.7.5
@angular/cli               : 6.1.5
@ionic/ng-toolkit          : 1.0.7
@ionic/schematics-angular  : 1.0.5
Cordova:
cordova (Cordova CLI) : 8.0.0
Cordova Platforms     : not available
Cordova Plugins       : not available
insert the output from ionic info here

Describe the Bug
@ionic/core library is not transpiled to es5 (it is es2017 maybe?)
Steps to Reproduce
ionic build

Check www/vendor.js
Expected Behavior
Ionic is an only library not transpiled to es5.
I don't suspect that ionic v4 will work with old webview but white page only application is not acceptable.
I could use ie https://www.npmjs.com/package/cordova-plugin-webview-checker to detect webview version and redirect to Google Play page if it is too old, but... it won't work because of ionic is not es5 only :(

yes, same issue

#15438

Closing as a duplicate of #15438


Consider removing multi_json dependency

Hi,
JSON has been part of the ruby stdlib since 1.9.0 (see https://github.com/ruby/ruby/commit/af1c4167), and the latest version of poltegeist requires ruby 1.9.3 or better.
Would you consider removing multi_json as a dependency, and just using require 'json' instead? Rails recently made a similar change (https://github.com/rails/rails/pull/23453), which has prompted other libraries to consider the same issue (like simplecov and stripe).
See also this recent blog post.
Thanks for a great library!

Sounds good! Mind opening a PR?

Sure, I've opened #734

Thank you!


workspace pyproject.toml iterator finds project files in .venv

Steps to Reproduce

add a dependency that pulls in pyproject.toml somewhere in its installation  (jupyterlab~=3.6)
Make the project a workspace
Repeated rye sync meets an error, pasted below

Note: rye sync -f is a workaround.
Expected Result
rye sync should be fine.
I think it looks like the pyproject.toml iterator descends down into .venv and it should probably not do that.
Actual Result
Error: could not write production lockfile for workspace

Caused by:
    project from '.../jupyterlab/.venv/lib/python3.11/site-packages/jupyterlab/tests/mock_packages/test_no_hyphens' has no name

Version Info
rye 0.4.0
commit: 0.4.0 (cdc5c37bc 2023-05-29)
platform: linux (x86_64)
self-python<EMAIL_ADDRESS>symlink support: true
Stacktrace
Caused by:
    project from '.../jupyterlab/.venv/lib/python3.11/site-packages/jupyterlab/tests/mock_packages/test_no_hyphens' has no name

Stack backtrace:
   0: rye::pyproject::PyProject::normalized_name
   1: rye::lock::format_project_extras
   2: rye::lock::update_workspace_lockfile
   3: rye::sync::sync
   4: rye::cli::sync::execute
   5: rye::cli::execute
   6: rye::main

Indeed. Your observation is correct and most likely the correct course of action is to at least ignore .venv but probably even go further and ignore some other locations as well.

I am curious how you feel about the approach cargo takes and force members to be explicitly defined (similarly to cargo)? It reduces mental load for people coming to a project, having to understand what parts of the configuration are implicit and what are explicit.
It can be a bit annoying to manage but realistically members change rarely and as long as we detect we are inside a workspace and error out if our project is not listed (similarly to cargo).


Fails to install with Prettier 2.3.0

Hi there,
Now Prettier 2.3.0 is out, there's an issue installing this. In https://github.com/trivago/prettier-plugin-sort-imports/blob/master/package.json Prettier is listed both as a peerDependency with a version specifier of ^2.2.1 which permits Prettier 2.3.0, but it's also listed as a dependency with an exact version of 2.2.1. I suspect this is a mistake, and it should only be listed as a peerDependency, but right now this causes Yarn to fail to install this as it can't find a version of Prettier to link. Fixing Prettier to 2.2.1 in our package.json when importing this fixes it, but I think the duplication needs to be resolved here too.
Thanks in advance for the help :)

I'm facing the same issue 😞

Same

Same issue :(

Same issue

same - but the fix helped me thanks @cnorthwood cnorthwood

Same issue here, how can we resolve it without updating the plugin in node_modules ?

Another possible way to fix, but its just another workarround...
add resolutions section like
  "resolutions": {
    "prettier": "2.2.1"
  },

But the best solution is remove the dependency and just left the peerDependency, right?


Abstract Editor

A problem
I've run into the following obstacles trying to build a product with custom editor support:

Code Mirror widgets have too much responsibilities:

handling code mirror editors, e.g. rendering and reacting on user input
handling general widget logic, e.g. applying standard jupyter styles
application logic, e.g. model synchronization, computation of edge location

Editor Widget Plugin assumes that Code Mirror is used:

import of code mirror styles
providing code mirror related commands
providing code mirror related menu items

Notebook Plugin is not extensible/contractible, particularly it is not possible:

provide a kernel or a notebook model aware cell editors
disable/enable the inspector depending on a kernel and a cell editor type
disable/enable tab based completions depending on a kernel and a cell editor type

An approach
This PR addresses to the aforementioned issues.
Editor MVP
Model-view-presenter pattern is used to separate (1) general widget logic and (2) application logic from a concrete (code mirror) widget.
Views
A view is an abstract UI component which is not aware of application model or logic and only responsible for rendering and dispatching user input to a proper application service. Functionality built on the top of views are reusable among different concrete editors. Views can be easily mocked for testing application logic.
IEditorView is an abstract editor interface, as mentioned before it is agnostic to a particular application model whether it is IDocumentContext, ICellModel or any other. It has several extensions:

IStandaloneEditorView is an abstract interface for a standalone editor used to open an enitre document
ICellEditorView is an abstract interface for a cell editor used inside a notebook
ICompletableEditorView is an abstract interface for a editor providing tab based completions

Decorators
A decorator is an extension of a view which is aware of widgets and responsible for applying general widget logic. It can be reused among different concrete editors.

DefaultStandaloneEditorWidgetDecorator applies default jp styles and updates a widget label
DefaultCellEditorWidgetDecorator applies default jp styles

Presenters
A presenter implements application logic, it is aware of a view and an application model. It can be reused among different concrete editors.

StandaloneEditorPresenter handles synchronization between IDocumentContext and IStandaloneEditorView.
CellEditorPresenter handles synchronization between ICellModel and ICellEditorView, updates an edge location.

Widgets
A widget is responsible for handling a concrete editor, i.e. displaying it and dispatching user input to a particular application service (presenter).
There are following code mirror widgets corresponding to views: CodeMirrorEditorWidget, CodeMirroStandaloneEditorWidget, CodeMirrorCellEditorWidget and CompletableCodeMirrorCellEditorWidget.
In the future CompletableCodeMirrorCellEditorWidget can be replaced by CompletableCodeMirrorEditorWidgetDecorator in the case if completion logic should be reused for non cell aware code mirror widget, e.g for a standalone.
Editor Widget Factory
EditorWidget.Factory is responsible for:

creation and tracking standalone editor widgets
registering commands related to concrete editor, e.g. enabling vim mode for code mirror or applying code mirror themes
registering corresponding menu items

Notebook Widget Factory
NotebookWidgetFactory is responsible for creation and tracking of notebook widgets.
A concrete factory could decide:

which kind of cell editors should be created
which information should be available to cell editors, e.g. whether cell editor presenter should be aware of kernels and a notebook model
whether tab completion should be used and when
whether inspections should be used and when
and so on

A general note
I've (1) followed the style guide, (2) documented everything, (3) fixed tests and (4) done manual UI testing.

@akosyakov Wow, awesome. Thanks for the epic PR! I'm pinging some folks who might want to take a look. One totally minor thing, please end files with a line break character, I think your editor is stripping them out, perhaps.
cc: @blink1073 @jasongrout @ellisonbg @sccolbert

This is a pretty big change. We're not opposed to the concept in theory, but I want to go through it and see if we can cut down on the size of the PR, and/or bring it in in more manageable chunks.
Thanks so much for working on this! Being editor agnostic is a good goal, and we are wanting to support Monaco in the future. This could be a way to get us there.

We actually do all this because we are using monaco (instead of code mirror).
We can contribute that part in another PR, too, if you are interested.

Wow, awesome, excited to dig into this. This type of abstraction work is really helpful at this point. We want JLab to work for other editors - but we recognize that we have to start some where (CodeMirror).
Some questions:

Do you think it is feasible to break this PR up into smaller atomic bits that can be reviewed more easily?
How has your experience been with monaco from a development perspective so far? (I am using VSCode daily).

@ellisonbg It should be feasible to extract Notebook Widget Factory to another PR. It is not related to Abstract Editor.
EditorWidget and CellEditor depend on CodeMirror widget. This PR basically extracts code mirror independent reusable logic from CodeMirror widget to new modules like decorators and presenters, such change affect EditorWidget and CellEditor, since they should be reconfigured to use new modules.

Hi guys, can you rebase?

@ellisonbg Hey! I've rebased it and here structured clarification for PR: https://github.com/TypeFox/RIDE/wiki/Abstract-Editor-Review

Could anybody spend time on reviewing this? We know it's a big one and promise to make them much smaller in future :)

Could anybody spend time on reviewing this? We know it's a big one and promise to make them much smaller in future :)

I'm looking at it right now.

@svenefftinge, @akosyakov - again, thanks very much for this massive amount of work, and the offer to contribute more like the monaco integration. Thanks as well for working with us (jumping on chat, or writing the wiki page to help a review). As you can imagine, it's quite a lot to review with our limited resources, and introduces a number of new patterns into the codebase, so thanks for being patient in the contribution and review process.
We discussed this in our dev meeting today, and we think it would be best to (a) take a step back and discuss the design to make sure we're all on the same page, and then (b) trim down the PR to the absolute bare minimum to implement the design. I think your summary at the top of the PR is a good start for a design discussion. Can you open an issue to start the design discussion?
I think a good thing in a minimal PR would be implementing a very minimal editor using an html textarea. That would help us see the bare minimum required to implement a new editor.

@jasongrout Hi, thx for the replay, eventually found time to open the issue: https://github.com/jupyter/jupyterlab/issues/875

Thanks!

Closing in favor of #890


undefined symbol: _ZNK11btMultiBody12hasFixedBaseEv

Hello,
After installing all the required dependencies and modify the Makefile in DeepMimicCore/, I tried to run the python DeepMimic.py --arg_file args/run_humanoid3d_spinkick_args.txt in Ubuntu 18.04, which led to the issue undefined symbol: _ZN11btMultiBody22addJointTorqueMultiDofEiPKF (Issue #82).
After solving that issue with this solution, I got this new error: undefined symbol: _ZNK11btMultiBody12hasFixedBaseEv

Traceback (most recent call last):
File "DeepMimic.py", line 9, in
from env.deepmimic_env import DeepMimicEnv
File "/home/tung/pj/DeepMimic/env/deepmimic_env.py", line 3, in
from DeepMimicCore import DeepMimicCore
File "/home/tung/pj/DeepMimic/DeepMimicCore/DeepMimicCore.py", line 13, in
from . import _DeepMimicCore
ImportError: /home/tung/pj/DeepMimic/DeepMimicCore/_DeepMimicCore.so: undefined symbol: _ZNK11btMultiBody12hasFixedBaseEv

The solution to previous problem undefined symbol: _ZN11btMultiBody22addJointTorqueMultiDofEiPKF:
https://github.com/xbpeng/DeepMimic/issues/82#issuecomment-534513934

solved by
wget https://github.com/bulletphysics/bullet3/archive/2.88.tar.gz
mv 2.88.tar.gz bullet3-2.88.tar.gz
tar xvzf bullet3-2.88.tar.gz
cd bullet3-2.88
sed -i 's/-DUSE_DOUBLE_PRECISION=ON/-DUSE_DOUBLE_PRECISION=OFF/g'
build_cmake_pybullet_double.sh
./build_cmake_pybullet_double.sh
cd build_cmake
make install
cd ../..

I had the same issue. This helped for me:
https://github.com/xbpeng/DeepMimic/issues/108
Turned out I had two different Bullet versions installed in two different folders.

Try the new build script added in master few days ago.

Well, the same problem. And I also have another undefined reference that is "_ZNK11btMultiBody15isBaseKinematicEv".

I had the same issue. This helped for me: #108 (comment)
Turned out I had two different Bullet versions installed in two different folders.

Thanks, but I think there's a little bit difference between the 2 problems. And I'm sorry that the solution doesn't work for me.

I had the same issue. This helped for me: #108 (comment)
Turned out I had two different Bullet versions installed in two different folders.

Thanks, but I think there's a little bit difference between the 2 problems. And I'm sorry that the solution doesn't work for me.

sorry for the bother.Have you fixed this problem?I met the same problem of this issue too. I have tried the solution provided by other issue, but still doesn't work.

maybe you need to delete all files generated using the last version? To run "make clean" or "rm * .o" might help

maybe you need to delete all files generated using the last version? To run "make clean" or "rm * .o" might help

Thank you for your reply. I have fixed this problem by deleting all files generated using "make python", and running the new build script to automatically install all the package needed, it works finally.


Global constants in firmware

Right now the firmware verification has a lot of kludges to deal with global constants, and it's a major pain point (for example, here). I'd like to see if we can do better.
The constraints here are:

protect against a global constant being overwritten with a local value (so GLOBAL = x; f(GLOBAL) should call f(x), not f(<literal value of the constant>))
produce C code that uses the global constant values, not arguments/literals

I'm not sure how to resolve these constraints in a nice way. The solution we currently have is pretty roundabout and hacky (a typeclass that states what constants exist and instances of the typeclass that give string, variable, literal, Z, and word values of the constants for different contexts; string/variable in the implementations, word in the proofs, and literals for the compiler).
Ideally, I think I would want to write the functions so they didn't take any extra arguments (although I'd be okay with a list specifying what global values they require) and reference the globals as variables. Then in the proofs, I'd want to start with a locals map that had the global values initialized as part of the locals map (again, would be okay with needing to write out their values). For the compiler I'd want to be able to specify particular Z values, or store literals somewhere in memory in the main function.
@samuelgruetter , do you think there are potential upstream changes that  we could make to improve this situation? For instance, could we make a version of WeakestPrecondition.call that would takes an initial map of global values to start with?
CC @dayeol

Noted. This is too technical for me to have an opinion on what should be
done or how, but it does seem like a pain-point
worth addressing/prioritizing.
[image: Google Logo]
Satnam Singh
Software Engineer, Google Research
@.***
On Mon, Jun 7, 2021 at 9:22 AM jadephilipoom @.***>
wrote:

Right now the firmware verification has a lot of kludges to deal with
global constants, and it's a major pain point (for example, here
https://github.com/project-oak/silveroak/pull/815#discussion_r646733217).
I'd like to see if we can do better.
The constraints here are:

protect against a global constant being overwritten with a local
value (so GLOBAL = x; f(GLOBAL) should call f(x), not f())
produce C code that uses the global constant values, not
arguments/literals

I'm not sure how to resolve these constraints in a nice way. The solution
we currently have is pretty roundabout and hacky (a typeclass that states
what constants exist and instances of the typeclass that give string,
variable, literal, Z, and word values of the constants for different
contexts; string/variable in the implementations, word in the proofs, and
literals for the compiler).
Ideally, I think I would want to write the functions so they didn't take
any extra arguments (although I'd be okay with a list specifying what
global values they require) and reference the globals as variables. Then in
the proofs, I'd want to start with a locals map that had the global values
initialized as part of the locals map (again, would be okay with needing to
write out their values). For the compiler I'd want to be able to specify
particular Z values, or store literals somewhere in memory in the main
function.
@samuelgruetter https://github.com/samuelgruetter , do you think there
are potential upstream changes that we could make to improve this
situation? For instance, could we make a version of
WeakestPrecondition.call that would takes an initial map of global values
to start with?
CC @dayeol https://github.com/dayeol
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/project-oak/silveroak/issues/817, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPA642WPKKUX5L7ALC7KRLTRTW3XANCNFSM46IAOQVQ
.

I'm trying out an approach where the source code section in Aes.v is parameterized over a Context {constant_vars : aes_constants expr}, whose fields are then instantiated to expr.literal SOME_Z_CONST for proofs and for compilation using the bedrock2 compiler, and instantiated to expr.var STRING_NAME_OF_CONST for pretty-printing to C code, and the pretty printer gets a list of varnames that it should treat as globals.
There might be a PR soon, depending on how many more word flag shift sideconditions etc need updating... :wink:


playlist tests require working DNS

Describe the bug
Two playlist tests are currently failing in the nixpkgs tests (ignore the cipher tests, I opened a second PR to fix those) as they require DNS to resolve which fails in a sandboxed environment (such as the nix build environment).
Since all other tests pass just fine and there's mocking in place I figure that these tests specifically are doing some additional requests which somehow bypass the mocking, which would mean that a) tests suites may currently cause requests against the live YouTube API which may have adverse effects on people running the test suite locally (like being treated as bots by YouTube and being unable to watch without an account like many others) and b) the tests may not be pure and fail depending on upstream changes which runs counter to the intention of those tests.

code that was used that resulted in the bug
The logs above contain a full trace, however it seems that the pagination bypasses the mock:
https://github.com/JuanBindez/pytubefix/blob/5303b640c3513b734c9edc9294ff0530dda16602/pytubefix/contrib/playlist.py#L325-L327
And is triggered in these two tests:
FAILED tests/contrib/test_playlist.py::test_playlist_failed_pagination - urllib.error.URLError: <urlopen error [Errno -3] Temporary failure in name ...
FAILED tests/contrib/test_playlist.py::test_playlist_pagination - urllib.error.URLError: <urlopen error [Errno -3] Temporary failure in name ...

Expected behavior
Tests should pass even when DNS resolution is not available.
Tests should not make requests to the YouTube API.

Desktop (please complete the following information):

OS: NixOS unstable
Python Version: 3.11.10
Pytubefix Version: main

Additional context
I'd proceed to disable those tests downstream for now, and it's not terribly important, so this is more of a nice to have, and if those tests are made to work without DNS I'd be happy to enable them again.

Oh my, it seems I've entirely missed that a large chunk of tests have already been disabled as they require network access. My bad.
It'd be cool if they could run without network access, but given that there are already many disabled tests I see no urgency whatsoever.


Feature - Admin - Content Management

Description
Is your feature request related to a problem? Please describe.
Right now content changes/entry is extremely slow and cumbersome because it requires us to push new code changes to our production environment. This means that they generally get dictated by our feature releases, which inhibits our ability to make quick, efficient changes to enhance or test alternative content.
Describe the solution you'd like
Ideally it would be very useful for the project team to be able to manage content independently from the codebase. The ability to test, alter, improve copy that exists on the UI right now would be helpful for them, but also alleviate some of the asks on the dev team to modify the lang files as they are now. Content management/entry could totally skip the dev team.
Considerations:

translation
maybe we implement the CMS for only new features, and then retrofit existing pages/tools at a later date
server configuration

Describe alternatives you've considered
After some quick discussion, there are React based CMS implementations that might be possible.

https://strapi.io/

So from a dev perspective, options would be:
Homebrew something ourselves in Laravel and/or React
This would give us the most flexibility but would be the most awful in terms of effort and maintenance. Not recommended unless we really can't find another useful solution.
Use a Laravel-based solution
Something like Twill looks like it could fit our use case pretty well. Upside being we could maintain the benefits of the server side rendered pages (Twig). Downside being since we aren't server side rendering the bulk of our React pages we'd either need to query the information during the bundle generation or add more API routes that the frontend will have to hit to populate the pages on the client side.
Use a headless solution
As Josh mentioned, Strapi or Cosmic look like interesting solutions, but we first will need to know if we can host these alongside our app on a subdomain or another url entirely. The major downside of headless is waiting for both written copy and data to load from API requests on the client side, potentially drastically reducing user experience of the site. We would also need to adopt loading states for the entire page like skeleton loading states unless we can figure out a way to populate written copy during bundle generation.

@Khristophor was this a spike?


[WIP]support elasticsearch

Ⅰ. Describe what this PR did
初步完成了Seata对ElasticSearch的支持
Ⅱ. Does this pull request fix one issue?

#fixes#3746
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Please change the title to :  [WIP]feature:support elasticsearch .
when you finish the code, remote [WIP] , we will start review the code carefully.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 15 committers have signed the CLA.:x: x-shadow-man:x: springlayer:x: elrond-g:x: wangliang181230:x: jsbxyyx:x: lvekee:x: a364176773:x: sustly:x: l81893521:x: coconut4j:x: LiWenGu:x: Rubbernecker:x: UmmizzZ:x: slievrly:x: lightClouds917You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 15 committers have signed the CLA.:white_check_mark: wangliang181230:x: UmmizzZ:x: slievrly:x: jsbxyyx:x: lvekee:x: elrond-g:x: lightClouds917:x: sustly:x: a364176773:x: Rubbernecker:x: l81893521:x: LiWenGu:x: springlayer:x: x-shadow-man:x: coconut4jYou have signed the CLA already but the status is still pending? Let us recheck it.

Due to code conflicts and age, if you are still interested in participating in the community, please resubmit this pr to the 2.x branch after resolving the conflict.
由于代码冲突以及年代久远，如果你还有兴趣参与到社区，请解决完冲突后重新请提交这份pr至2.x分支中


added cast/to_string

Closes #385

I have added very very simple (but supposedly fast) parsing for now

should I add the +00:00 format and missing timezone to be more flexible?

This looks great. I wouldn't add +00:00 nor timezones but I would also parse yyyy-mm-dd hh:mm:ss in the datetime.
Also, should we add a simple validation (just check they are in the proper boundary) or no?

I am still unsure about +00:00 since it has the same meaning as Z so it would fit in ISO8061 UTC definition. Also adding it costs basically nothing. I am adding the timezone-less format yyyy-mm-dd hh:mm:ss as well as hh:mm:ssZ for time.
Adding validation like month <= 12, day <= 31, etc would be trivial and can be done in guards. I would not validate the month/day combination however. What do you think?

Let's leave timezone out until someone asks. :) Agreed with everything else
though.
I also just realized that the datetime cast could just call the date and
time ones once we split them. Not sure if it helps implementation wise. :)
--
José Valimwww.plataformatec.com.br
http://www.plataformatec.com.br/Founder and Lead Developer

basic validation is added and time without a timezone can also be read

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


fix: improve messaging when cloning a repo runs out of hard drive space

I am open to suggestions here, but I thought this may be a good idea.
Basically what we would do before is enable LFS autofetch immediately if the user requested and respect this setting essentially during the whole script that actually clones the repo. So when the session runs out of space and the cloning script fails then the user gets an empty repo without a message as to why things went wrong.
In order to quickly alleviate this without fully revamping our repo cloning script I have done the following:

for as long as possible the repo cloning script avoids fetching LFS objects
all the way at the end and if the user requested it the script will setup so that LFS objects are fetched by default and it will try to fetch the LFS objects
if after fetching the LFS object the disk is 100% used the the local clone of the repo is deleted and a single file called INSUFFICIENT_HARD_DRIVE_SPACE is created in order to communicate the problem to the user

An alternative is to remove the part that removes the local copy of the repo and creates the file if the hard drive is full. This way if the hard drive fills up the user will get a (seemingly) full normal repo and a full hard drive. But some of the LFS objects will not all be there and as soon as the user tries to do something they will hopefully realize that they are out of hard drive space.
/deploy

You can access the deployment of this PR at https://renku-ci-nb-725.dev.renku.ch

A few notes on alternatives (that are actually not viable):

Using the information that gitlab reports at the top of a project that shows how much storage is used is not possible because this is the total of all storage files across all branches (I confirmed with a dummy project). So after the repository grows sufficiently this number is bound to become large and it is not a good representation for the real storage needs.
Digging through the gitlab documentation on the API and the python SDK for gitlab I could not find anything about how to get the size of LFS objects for a specific branch or commit. There is only a way to get the total size of all LFS objects in the whole project just as I mentioned above, but this is not very useful.

I think to properly resolve this going forward we can use the git-sidecar and run a command in there that will list all LFS objects or simply check how much hard drive space is available.
Another alternative is that we revamp the bash script that clones the repository to use python. Then we can have the cloning script annotate the pod or jupyterserver resource in k8s with something that will indicate that there is insufficient hard drive space. The UI can then relay this information to the user.

Would doing a checkout without checking out the LFS objects and then inspecting the pointer files in the tree work? The pointer files include the object size. You could parse the .gitattributes file to see which paths you need to check - I believe there is code already in renku-python that does this but maybe check with @Panaetius

Would doing a checkout without checking out the LFS objects and then inspecting the pointer files in the tree work? The pointer files include the object size. You could parse the .gitattributes file to see which paths you need to check - I believe there is code already in renku-python that does this but maybe check with @Panaetius

@rokroskar once I checkout the repo (even without fetch the LFS objects) I can get the size of all LFS files by doing git lfs ls-files -s. It would be nice if I did not have to do this to see how much size is required by the LFS objects.
The other problem is that once I know this in the init script that clones the repo, how do I communicate it to the notebook service. Or are you proposing that I should clone the repo without fetching the LFS objects in the notebook service itself when someone tries to start a session?

Another alternative is that we revamp the bash script that clones the repository to use python. Then we can have the cloning script annotate the pod or jupyterserver resource in k8s with something that will indicate that there is insufficient hard drive space. The UI can then relay this information to the user.

I can envision a very nice UX around having this information in the annotations (therefore in the GET /notebooks API) :slightly_smiling_face:

I can envision a very nice UX around having this information in the annotations
It would be pretty easy to give the init container that clones the repo a simple k8s role that allows it to annotate the notebooks. So it would add an annotation to indicate that the session has run out of space.

Perhaps a better alternative is to open an endpoint on the notebook service that would result in annotating a user session. Then the git clone script could call that endpoint when it sees that the session runs out of space.

Ok I added an issue to resolve this for the long term in #727. However for the time being we can provide some rudimentary info to users with this PR. All this does is that if the session runs out of space when the repo is being cloned then the whole repo is removed and a single filed called INSUFFICIENT_HARD_DRIVE_SPACE is created in the repository.
I think this is better than getting an empty repo with nothing in it which happens now when cloning the repo results in a full hard drive.