Check for document.getElementById in main.js

I use a slightly modified index.html page which includes my custom CSS styles and text changes. With the most recent update I ran into an issue with the index template where #url was undefined causing a fatal JavaScript error.
I tracked this down to document.getElementById("url").textContent = url; in main.js
https://github.com/bilde2910/Hauk/blob/master/frontend/main.js#L215
To keep things simple and to not make changes to main.js I just set the color of the URL div to match the background color which works. I think I could also use visibility:hidden instead
Overall,  I do like the new changes providing more information but wondering if we should wrap some of the getElementById statements in a if type !== undefined) checks. If so, which ones?
The main one the effects the layout I've found is #URL so that's why I'm specifically mentioning that one.

Default view:

Showing the URL is still there but hidden:

I can certainly implement this, as it's a pretty quick fix. Maybe it could be an idea to allow some kind of custom HTML setup, but right now I have no idea whatsoever on how to practically implement that.

Fixed in latest commit. Modifying the HTML isn't really officially supported, but at least you shouldn't get errors anymore when doing it


fix: correct GHSA URLs

Previously, all Wolfi vulnerabilities handled by Vunnel were from NVD CVEs. However, now that Wofli also reports GHSA vulnerabilities as well, switch the link URL on the vulnerability prefix, so that we output the correct URLs.

Closed in favor of https://github.com/anchore/vunnel/pull/425, which fixes additional related URL problems and fixes the quality gate as well.


Update scenario viz

Corrects bug where color weight was being incorrectly calculated, and adds a density plot alongside the scenario trajectories.
Also updated the colors to use the preset color scheme.

@Rosejoycrocker feel free to adjust the colors to your liking

Looks good to me, and tests all passing


(Video) Shaders

Add video shaders so you can add CRT effects and stuff.

Welp, shaders work... Need a config/toggle though.

Shader_Crt.zip
From Anata on discord.


cant create an unnamed volume in conjunction with volume-driver

doing the equivalent of
docker run --volume-driver rancher-nfs -v /tmp ...

in cattle will not create a /tmp as a rancher-nfs volume

Tested with v1.2.0-pre4-rc10 and  rancher/storage-nfs:v0.4.0
Able to create a service when volume provide as "/" or "/:" using rancher-nfs volume driver .
volumes gets created with a self generated name like 8d901dc6-9332-43d1-832d-4867c292aa75 .


Align schema and actions with Realtime

What kind of change does this PR introduce?
To more closely align walrus with the SQL used in Realtime:

renames the cdc schema to realtime
removes the truncate action

@olirice can you please double check d069711f819ab18545ed876b19b7daed2e32e3f7? Otherwise, feel free to merge!

@olirice can you please double check d069711? Otherwise, feel free to merge!

looks good. intended to leave those in (except the pydantic model) and leave behavior undefined, but i agree its cleaner to remove all trace


Problem with condition of Metricbeat Value

We have an issue with load average metric of MetricBeat. We run the query and create a condition but doesn't receive the notification, instead write at the log:
"Condition Error for new_watcher_35zcqmjhk: SyntaxError: Unexpected number"
We have ELK version5.4.2
Thanks,
Alejandro

Hi @alejandrobarrios07
In order to help you we need to see your watcher, there's no way anyone can guess what you did.

Hi @lmangani , I'm new reporting issues, sorry.
Luckily I found the solution. When a metric name contains a number, the condition gives the error that remarks above.
My condition was:
Payload.hits.hits [1] ._ source.system.load.1> 2
And the solution I found was to process the metric by logstash and rename it by changing the number by the text, being as follows.
Payload.hits.hits [1] ._ source.system.load.one> 2
Thank you very much,
Alejandro


feat: lightdom css controller

What I did

add new LightdomCSSController which auto-loads lightdom css
remove rh-table-lightdom.css from docs pages
add an --_rhds-lightdom: rh-table brand to the table lightdom stylesheet (required for the controller to work efficiently)

Testing Instructions

Notes to Reviewers
The SSR story has yet TBD, it may be possible to automatically write lightdom sheets as styletags via the controller / element renderer

Doing this causes FOUC on rh-table at load.  I don't see this as a desired effect.
https://github.com/user-attachments/assets/fc182c1d-b5f7-4ee1-bab5-39c0e4e0d4aa
I'm questioning how loading the styles via javascript in this case would be helpful to the end user seems likely it would only reduce the need for an implementer to add a<link href> for for mandatory lightdom css components.
It may also be I need more time to process how we divide up the lightdom.css and lighdom-shim.css files and which get loaded via this controller.

yes, as mentioned in chat, I'm aware of the fouc, and as i wrote in the description i'm working on an ssr solution
lightdom css is part of the element definition. meaning that the element does not work if you don't have it's associated lightdom css. What this PR solves (as discussed at length in many places over several months) is

the case where the user forgot to load the lightdom css - one of our most common support issues
the case where the user did load the lightdom css, but loaded it in the wrong place (i.e. in the document when the element is in a shadow root)

Hope that clarifies


BrushNet fix failed: This action is not allowed with this security level configuration.

BrushNet fix failed: This action is not allowed with this security level configuration.

How did you get this errro? Can you post ComfyUI log?


Main menu crash

I've downloaded and installed modlist step-by-step following readme, but after loading main menu game just crashes without any issue message.
RTX 2070,
AMD Ryzen 5 3600 6-Core Processor 3.59 GHz,
16 GB of RAM,
pagefile ~ 40GB

Savage.
I have seen suggestions to work around similar issues. I have just forgotten them, over time.
I can suggest the Wabbajack discord service, for a fast tailored response.
Wabbajack discord -> Elysium Remastered Support (page)
All the best, and good luck!
J/TheClock69


Use state boundary from api

This PR updates the tools and Location & Boundary related components to use the state boundary vector layer from the api. There is additional cleanup to do in the src/helpers/geocode,js . Proposing to update it after the Extreme Heat and Snowpack tools are migrated so sapper does not complain about missing functions.

@clhenrick I'm done adding the ImageLoader carbon component. Can you take another quick look at it when you have time?

@mukhtyar sure thing, will take a look right now.


BAEL-7008 Full and Partial Text Search in MongoDB

retest this please

@lor6 - this is not correct the PR for this task. I created it first, but it does not follow the correct naming convention for tests that require a live MongoDB instance.
Here's the correct one which is approved & meged - https://github.com/eugenp/tutorials/pull/16623

Thanks @usman-mohyuddin I will close this.


bump rke2 to v1.27.13+rke2r1 and rancher to v2.8.3

Problem:

Solution:

Related Issue:
https://github.com/harvester/harvester/issues/4657, https://github.com/harvester/harvester/issues/5792
Test plan:
Create a cluster and operate VM without error.

@mergifyio backport v1.3


opt Avoid unnecessary copies in the profile thrift.

Proposed changes
need this pr
https://github.com/apache/doris/pull/34693
before
Create report exec status params cost 631.780ms
Create report exec status params cost 629.973ms

now
Create report exec status params cost 149.688ms
Create report exec status params cost 185.920ms

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run cloud_p1

run buildall

run buildall

run buildall


Considering a  file with a single LF is not useful

$ echo "" > c
$ sha1sum c
adc83b19e793491b1c6ea0fd8b46cd9f32e592fc  c

there are many files with just that single LF alright and they should not even be treated seriously in a pipeline and just reported as empty. This could be based on the size being 1 byte for a start.
Otherwise we end up sending things to be matched to the PurlDB for no good reason.
See https://github.com/nexB/purldb/issues/198

For the deploy_to_devel pipeline we should have a step to ignore empty files from matching.

Instead of looking at the content, I would suggest that we ignore files of size under a few (TBD) Bytes.


Table outline is not hide when using uncover

#slide(title: [k-distinctness], theme-variant: "north")[
  #set text(size: 18pt)

  == Problem Description

  - Given an input $x in [q]^n$, for some $q in "poly"(n)$, decide if there exist distinct $a_1, ..., a_k in [n]$ such that $x_(a_1) = thick dots.c thick = x_a_k$.
  - When $k=2$, it is the collision problem we discuss in class.

  #uncover(2)[
    == Current Results

    #table(
      columns: (auto, auto),
      inset: 15pt,
      [=== Query Complexity], [=== Time Complexity],
      [$O(n^(k\/(k+1)))$ [2003]], [$tilde(O)(n^(k\/(k+1)))$ [2003]],
      [$O(n^(3/4-1/4(2^k-1)))$], [$tilde(O)(n^(5\/7)) (k=3)$; $tilde(O)(n^((k-1)\/k)) (k>3)$ [2013 #super("*")]],
      [$O(n^(3/4-1/4(2^k-1)))$], [$tilde(O)(n^(3/4-1/4(2^k-1)))$ [2023]]
    )
  ]
]

Nasty!
In the meantime, maybe you could try with tablex ? https://github.com/PgBiel/typst-tablex

I confirm, it works with tablex !

Ah Thanks!

Tracking https://github.com/typst/typst/issues/2040


Fix segfault in string_name.cpp

Fixes #96423
Not sure if this is the most optimal way but it surely fixes the issue

Hmm I had found another issue with AnimationPlayer / AnimationMixer. Not sure if its related, but if we try to remove an animation and addd an animation of the same name, certain caches aren't cleared, the animation frame-skips in the first few frames before catching up. I event tried with clear_caches but that didnt do anything, which is why I had to do this "animation counter" thing in the original issue. I will try to see if I can create a Minimal reproduction project with that issue too and want to ask if its related. If so I can attempt fixing both in one go.

According to https://github.com/godotengine/godot/issues/96423#issuecomment-2451646625 this is no longer reproducible, so closing this


Project: simplification

We could simplify Rustup a lot. It was built with some very ambitious goals and it is clear that some of things will never happen, but the code is designed to accommodate them causing unnecessary complexity. It is also backwards compatible with very old versions of Rust and Rustup. There is lots of opportunity to reduce complexity. Some ideas:

remove telemetry (this has never been used and is not that useful in any case)
remove support for v1 manifests (we should still detect and error out with a message to delete things manually or whatever)
support fewer ways to install Rust
use standard logging rather than notifications
merge some of the crates (probably only rustup and rustup-dist)

cc @rust-lang/cargo for any suggestions of things we should/should not do

@nrc @alexcrichton could you give more information regarding "support fewer ways to install Rust" and "simplify filesystem transaction handling code" please?  If it's likely to generate a back-and-forth, then perhaps in its own issue.
Meanwhile I've opened an issue dedicated to the crate merging: https://github.com/rust-lang/rustup.rs/issues/1644

Also, it's probably good to spin out the curl vs reqwest discussion into a thread and come to a conclusion.

support fewer ways to install Rust

iirc, there are a lot of ways to install Rust via Rustup but the only ones anyone ever uses are dist and the windows installer (which is somewhat separate any way). devs do sometimes install from local disk, but I think that might be able to be simplified by  separating it and handling it differently (we can certainly make breaking changes in this case).

cargo toolchain install <toolchain>... is the common case.
For rustc devs they also use cargo toolchain link <toolchain> <path> (as documented also in https://rust-lang.github.io/rustc-guide/how-to-build-and-run.html#creating-a-rustup-toolchain), and I think we should keep.
Other than that rustup help toolchain does say:

but rustup can also install toolchains from the official archives, for alternate host platforms, and from local builds.

But I don't exactly see how or where that happens in the code (I'm looking at rustup_mode::update).

@h-michael @alexcrichton @seanmonstar Created a PR for the removal of the cURL backend. Figure discussion could happen there with respect to @dwijnand 's request for a place to hold one.

remove support for v1 manifests

Were v2 manifests generated for older releases? I think we shouldn’t break the ability for future rustup.rs to install, say, Rust 1.2.0. (I’ve used this to find old regressions.)

cc @brson

Were v2 manifests generated for older releases? I think we shouldn’t break the ability for future rustup.rs to install, say, Rust 1.2.0. (I’ve used this to find old regressions.)

@SimonSapin afaik, no
I'm in agreement with everything in the op, except for possibly removing v1 support.
The multi-backend support was a pet project just because I wanted to see if I could support all the HTTP/HTTPS implementations, particularly to support production usage of rustls. It's fine to strip down to just reqwest, assuming nobody is using the curl backend - I know at some point, somebody needed to explicitly use the curl backend because reqwest didn't work. It's probably reasonable to remove it and see if anybody complains.
multirust and very-old rustup compat code can probably be deleted at this kind, though if possible, emitting an error would prevent rustup from doing something destructive in the case those scenarios do come up.
Re multiple ways for rustup to install rust, the only ways I recall are from the xz-tarballs and gz-tarballs (even on windows). I don't recall rustup having windows installer support, though there is a rustup-win-installer subdirectory that I assume still doesn't do anything useful. That can be deleted.
The gz support can't be removed as long as rustup supports pre-xz toolchains.
I don't have any insight into whether people use rustup toolchain link or not, though I do know I was pretty close to using it the other day for testing custom rust toolchains, but got distracted before I did.

Telemetry is fine to remove, though I still think having telemetry somewhere is a good idea.

I left a comment on the curl-removal PR about the prudence of both making reqwest the default and removing curl in the same release.

very-old rustup compat code

If you could give some more specifics or pointers that would be great.

though there is a rustup-win-installer subdirectory that I assume still doesn't do anything useful. That can be deleted

Oh!
Thanks, @brson!

I don't have any insight into whether people use rustup toolchain link or not, though I do know I was pretty close to using it the other day for testing custom rust toolchains, but got distracted before I did.

It's convenient way to use Rust packaged by Linux distributions. Not like I'm using it often but it comes handy.

I'm using rustup toolchain link system /usr for that exact purpose.

rustup toolchain link is also very useful for rustc development.

So I think rustup toolchain link is important to keep; certainly I wouldn't advocate removing it.
I do wonder if there might be something even more nice we could do for supporting system-managed toolchains (e.g. converting components, targets, etc, into package names and reporting useful commands such as apt install rust-rls or somesuch should a proxy be run which isn't installed for the system toolchain.
Of course, this doesn't quite fall under the topic of simplification, so I'll shut up now :D

Agreed on rustup toolchain link; I'd even go as far and possibly better integrate the local workflow.
I know I wanted to rustup component add XYZ in the past. This could possibly run ./x.py dist XYZ under the hood or we could change the way these components are built inside Rust repo so that rustup can understand the resulting artifacts and be able to use them for rustup component add purposes.

Can there be +nightly in rustup to keep it similar to cargo rather than using --toolchain=nightly?


Couldn't get the SHARK web ui running

Hi, can you help me with the error below? I've tried different models but the error is similar:
Found device AMD Radeon RX 6800 XT. Using target triple rdna2-unknown-windows.
Using tuned models for stabilityai/stable-diffusion-2-1/fp16/vulkan://00000000-0a00-0000-0000-000000000000.
Downloading (…)cheduler_config.json: 100%|████████████████████████████████████████████████████| 345/345 [00:00<?, ?B/s]
loading existing vmfb from: D:\username\Downloads\shark\euler_scale_model_input_1_512_512_vulkan_fp16.vmfb
loading existing vmfb from: D:\username\Downloads\shark\euler_step_1_512_512_vulkan_fp16.vmfb
use_tuned? sharkify: True
_1_64_512_512_fp16_tuned_stable-diffusion-2-1-base
Traceback (most recent call last):
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 71, in __init__
  File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
  File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
    resolved_vocab_files[file_id] = cached_file(
                                    ^^^^^^^^^^^^
  File "transformers\utils\hub.py", line 409, in cached_file
    resolved_file = hf_hub_download(
                    ^^^^^^^^^^^^^^^^
  File "huggingface_hub\utils\_validators.py", line 120, in _inner_fn
  File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
  File "filelock\_api.py", line 260, in __enter__
  File "filelock\_api.py", line 217, in acquire
  File "filelock\_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\\Users\\username/.cache\\huggingface\\hub\\models--stabilityai--stable-diffusion-2-1\\blobs\\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "gradio\routes.py", line 401, in run_predict
  File "gradio\blocks.py", line 1302, in process_api
  File "gradio\blocks.py", line 1039, in call_function
  File "anyio\to_thread.py", line 31, in run_sync
  File "anyio\_backends\_asyncio.py", line 937, in run_sync_in_worker_thread
  File "anyio\_backends\_asyncio.py", line 867, in run
  File "gradio\utils.py", line 491, in async_iteration
  File "ui\txt2img_ui.py", line 146, in txt2img_inf
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 338, in from_pretrained
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 43, in __init__
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 75, in __init__
  File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
  File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
    resolved_vocab_files[file_id] = cached_file(
                                    ^^^^^^^^^^^^
  File "transformers\utils\hub.py", line 409, in cached_file
    resolved_file = hf_hub_download(
                    ^^^^^^^^^^^^^^^^
  File "huggingface_hub\utils\_validators.py", line 120, in _inner_fn
  File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
  File "filelock\_api.py", line 260, in __enter__
  File "filelock\_api.py", line 217, in acquire
  File "filelock\_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\\Users\\username/.cache\\huggingface\\hub\\models--stabilityai--stable-diffusion-2-1\\blobs\\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'

Found device AMD Radeon RX 6800 XT. Using target triple rdna2-unknown-windows.
Using tuned models for CompVis/stable-diffusion-v1-4/fp16/vulkan://00000000-0a00-0000-0000-000000000000.
Downloading (…)cheduler_config.json: 100%|████████████████████████████████████████████████████| 313/313 [00:00<?, ?B/s]
loading existing vmfb from: D:\username\Downloads\shark\euler_scale_model_input_1_512_512_vulkan_fp16.vmfb
loading existing vmfb from: D:\username\Downloads\shark\euler_step_1_512_512_vulkan_fp16.vmfb
use_tuned? sharkify: True
_1_64_512_512_fp16_tuned_stable-diffusion-v1-4
Downloading (…)ain/unet/config.json: 100%|█████████████████████████████████████████████| 743/743 [00:00<00:00, 739kB/s]
Downloading (…)ch_model.safetensors: 100%|████████████████████████████████████████| 3.44G/3.44G [01:55<00:00, 29.7MB/s]
mat1 and mat2 shapes cannot be multiplied (128x1024 and 768x320)
Retrying with a different base model configuration
Loading Winograd config file from  C:\Users\username\.local/shark_tank/configs\unet_winograd_vulkan.json
100%|███████████████████████████████████████████████████████████████████████████████████| 107/107 [00:00<00:00, 386B/s]
100%|█████████████████████████████████████████████████████████████████████████████████| 107/107 [00:00<00:00, 16.4kB/s]
Loading lowering config file from  C:\Users\username\.local/shark_tank/configs\unet_v1_4_fp16_vulkan_rdna2.json
100%|█████████████████████████████████████████████████████████████████████████████| 24.6k/24.6k [00:00<00:00, 64.4kB/s]
100%|█████████████████████████████████████████████████████████████████████████████| 24.6k/24.6k [00:00<00:00, 1.97MB/s]
Applying tuned configs on unet_1_64_512_512_fp16_tuned_stable-diffusion-v1-4_vulkan
No vmfb found. Compiling and saving to D:\username\Downloads\shark\unet_1_64_512_512_fp16_tuned_stable-diffusion-v1-4_vulkan.vmfb
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\username\Downloads\shark\unet_1_64_512_512_fp16_tuned_stable-diffusion-v1-4_vulkan.vmfb.
Traceback (most recent call last):
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 71, in __init__
  File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
  File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
    resolved_vocab_files[file_id] = cached_file(
                                    ^^^^^^^^^^^^
  File "transformers\utils\hub.py", line 409, in cached_file
    resolved_file = hf_hub_download(
                    ^^^^^^^^^^^^^^^^
  File "huggingface_hub\utils\_validators.py", line 120, in _inner_fn
  File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
  File "filelock\_api.py", line 260, in __enter__
  File "filelock\_api.py", line 217, in acquire
  File "filelock\_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\\Users\\username/.cache\\huggingface\\hub\\models--CompVis--stable-diffusion-v1-4\\blobs\\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "gradio\routes.py", line 401, in run_predict
  File "gradio\blocks.py", line 1302, in process_api
  File "gradio\blocks.py", line 1039, in call_function
  File "anyio\to_thread.py", line 31, in run_sync
  File "anyio\_backends\_asyncio.py", line 937, in run_sync_in_worker_thread
  File "anyio\_backends\_asyncio.py", line 867, in run
  File "gradio\utils.py", line 491, in async_iteration
  File "ui\txt2img_ui.py", line 146, in txt2img_inf
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 338, in from_pretrained
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 43, in __init__
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 75, in __init__
  File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
  File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
    resolved_vocab_files[file_id] = cached_file(
                                    ^^^^^^^^^^^^
  File "transformers\utils\hub.py", line 409, in cached_file
    resolved_file = hf_hub_download(
                    ^^^^^^^^^^^^^^^^
  File "huggingface_hub\utils\_validators.py", line 120, in _inner_fn
  File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
  File "filelock\_api.py", line 260, in __enter__
  File "filelock\_api.py", line 217, in acquire
  File "filelock\_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\\Users\\username/.cache\\huggingface\\hub\\models--CompVis--stable-diffusion-v1-4\\blobs\\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'

Thanks!

Looks like some 🤗 issue. Can you please try to clear your HF cache

hi @powderluv ! yes i've cleared the cache (deleted this C:\\Users\\username/.cache folder) couple of times, restarting the process but the results are the same 🤔 So i'm not sure what the issue is.

Hi, some update from me: it seems the advanced installation is working just fine for me:

So wondering if the problem is within the release version itself. Thanks!

I'm having the same issue here. 5800X and 7900 XTX. Deleting the cache folder did not help either. Hope this helps in solving the problem!
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
Traceback (most recent call last):
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 71, in __init__
  File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
  File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
    resolved_vocab_files[file_id] = cached_file(
                                    ^^^^^^^^^^^^
  File "transformers\utils\hub.py", line 409, in cached_file
    resolved_file = hf_hub_download(
                    ^^^^^^^^^^^^^^^^
  File "huggingface_hub\utils\_validators.py", line 120, in _inner_fn
  File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
  File "filelock\_api.py", line 260, in __enter__
  File "filelock\_api.py", line 217, in acquire
  File "filelock\_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\\Users\\Spencer/.cache\\huggingface\\hub\\models--CompVis--stable-diffusion-v1-4\\blobs\\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "gradio\routes.py", line 401, in run_predict
  File "gradio\blocks.py", line 1302, in process_api
  File "gradio\blocks.py", line 1039, in call_function
  File "anyio\to_thread.py", line 31, in run_sync
  File "anyio\_backends\_asyncio.py", line 937, in run_sync_in_worker_thread
  File "anyio\_backends\_asyncio.py", line 867, in run
  File "gradio\utils.py", line 491, in async_iteration
  File "ui\txt2img_ui.py", line 146, in txt2img_inf
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 338, in from_pretrained
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 43, in __init__
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 75, in __init__
  File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
  File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
    resolved_vocab_files[file_id] = cached_file(
                                    ^^^^^^^^^^^^
  File "transformers\utils\hub.py", line 409, in cached_file
    resolved_file = hf_hub_download(
                    ^^^^^^^^^^^^^^^^
  File "huggingface_hub\utils\_validators.py", line 120, in _inner_fn
  File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
  File "filelock\_api.py", line 260, in __enter__
  File "filelock\_api.py", line 217, in acquire
  File "filelock\_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\\Users\\Spencer/.cache\\huggingface\\hub\\models--CompVis--stable-diffusion-v1-4\\blobs\\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'

Same issue here with the release WebUI and CLI .exe's. Clearing the .cache folder doesn't change anything. With the WebUI, I can launch the web interface, but I get the error as soon as I try to generate an image.
WebUI error:
Found device NVIDIA GeForce RTX 2080. Using target triple turing-rtx2080-windows. Tuned models are currently not supported for this setting. Downloading (…)cheduler_config.json: 100%|█████████████████████████████████████████████| 345/345 [00:00<00:00, 345kB/s] huggingface_hub\file_download.py:133: UserWarning: huggingface_hubcache-system uses symlinks by default to efficiently store duplicated files but your machine does not support them in C:\Users\mattb\.cache\huggingface\hub. Caching files will still work but in a degraded version that might require more space on your disk. This warning can be disabled by setting theHF_HUB_DISABLE_SYMLINKS_WARNING` environment variable. For more details, see https://huggingface.co/docs/huggingface_hub/how-to-cache#limitations.
To support symlinks on Windows, you either need to activate Developer Mode or to run Python as an administrator. In order to see activate developer mode, see this article: https://docs.microsoft.com/en-us/windows/apps/get-started/enable-your-device-for-development
failed to download model, falling back and using import_mlir
loading existing vmfb from: C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark\euler_scale_model_input_1_512_512_vulkan_fp16.vmfb
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
loading existing vmfb from: C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark\euler_step_1_512_512_vulkan_fp16.vmfb
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
use_tuned? sharkify: False
_1_64_512_512_fp16_stable-diffusion-2-1-base
safetensors\torch.py:99: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
Traceback (most recent call last):
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 71, in init
File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
resolved_vocab_files[file_id] = cached_file(
^^^^^^^^^^^^
File "transformers\utils\hub.py", line 409, in cached_file
resolved_file = hf_hub_download(
^^^^^^^^^^^^^^^^
File "huggingface_hub\utils_validators.py", line 120, in _inner_fn
File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
File "filelock_api.py", line 260, in enter
File "filelock_api.py", line 217, in acquire
File "filelock_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\Users\mattb/.cache\huggingface\hub\models--stabilityai--stable-diffusion-2-1\blobs\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "gradio\routes.py", line 401, in run_predict
File "gradio\blocks.py", line 1302, in process_api
File "gradio\blocks.py", line 1039, in call_function
File "anyio\to_thread.py", line 31, in run_sync
File "anyio_backends_asyncio.py", line 937, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 867, in run
File "gradio\utils.py", line 491, in async_iteration
File "ui\txt2img_ui.py", line 146, in txt2img_inf
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 338, in from_pretrained
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 43, in init
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 75, in init
File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
resolved_vocab_files[file_id] = cached_file(
^^^^^^^^^^^^
File "transformers\utils\hub.py", line 409, in cached_file
resolved_file = hf_hub_download(
^^^^^^^^^^^^^^^^
File "huggingface_hub\utils_validators.py", line 120, in _inner_fn
File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
File "filelock_api.py", line 260, in enter
File "filelock_api.py", line 217, in acquire
File "filelock_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\Users\mattb/.cache\huggingface\hub\models--stabilityai--stable-diffusion-2-1\blobs\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'`
CLI error:
`Found device NVIDIA GeForce RTX 2080. Using target triple turing-rtx2080-windows.
Tuned models are currently not supported for this setting.
failed to download model, falling back and using import_mlir
loading existing vmfb from: C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark\euler_scale_model_input_1_512_512_vulkan_fp16.vmfb
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
loading existing vmfb from: C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark\euler_step_1_512_512_vulkan_fp16.vmfb
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
use_tuned? sharkify: False
_1_64_512_512_fp16_stable-diffusion-2-1-base
safetensors\torch.py:99: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
Traceback (most recent call last):
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 71, in init
File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
File "transformers\utils\hub.py", line 409, in cached_file
File "huggingface_hub\utils_validators.py", line 120, in _inner_fn
File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
File "filelock_api.py", line 260, in enter
File "filelock_api.py", line 217, in acquire
File "filelock_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\Users\mattb/.cache\huggingface\hub\models--stabilityai--stable-diffusion-2-1-base\blobs\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "apps\stable_diffusion\scripts\main.py", line 11, in
File "apps\stable_diffusion\scripts\txt2img.py", line 25, in main
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 338, in from_pretrained
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 43, in init
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 75, in init
File "apps\stable_diffusion\src\models\opt_params.py", line 120, in get_tokenizer
File "transformers\tokenization_utils_base.py", line 1770, in from_pretrained
File "transformers\utils\hub.py", line 409, in cached_file
File "huggingface_hub\utils_validators.py", line 120, in _inner_fn
File "huggingface_hub\file_download.py", line 1302, in hf_hub_download
File "filelock_api.py", line 260, in enter
File "filelock_api.py", line 217, in acquire
File "filelock_windows.py", line 25, in _acquire
OSError: [Errno 22] Invalid argument: 'C:\Users\mattb/.cache\huggingface\hub\models--stabilityai--stable-diffusion-2-1-base\blobs\W/"469be27c5c010538f845f518c4f5e8574c78f7c8.lock'
[24600] Failed to execute script 'main' due to unhandled exception!`
Unlike alexanderv21, however, I couldn't get the advanced installation working either. Error when I try to run index.py:
python .\index.py Traceback (most recent call last):   File "C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark_Advanced\apps\stable_diffusion\web\index.py", line 11, in <module>     from apps.stable_diffusion.src import args, clear_all   File "C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark_Advanced\apps\stable_diffusion\src\__init__.py", line 1, in <module>     from apps.stable_diffusion.src.utils import (   File "C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark_Advanced\apps\stable_diffusion\src\utils\__init__.py", line 12, in <module>     from apps.stable_diffusion.src.utils.sd_annotation import sd_model_annotation   File "C:\GitHub_Sync\PugetBench-for-Stable-Diffusion\Shark_Advanced\apps\stable_diffusion\src\utils\sd_annotation.py", line 3, in <module>     from shark.model_annotation import model_annotation, create_context ModuleNotFoundError: No module named 'shark'


Remove unused site plugin declaration

Type of change

Refactoring

Description
Remove unused plugin so we do not have to maintain it via dependabot
Note that it seems to be detected as an active plugin by the enforcer despite apparently contributing no invocations to a maven install or deploy, so we exclude it from the enforcement.

Do we need something similar in the runtime repo?

yep, have created the equivalent over in the kroxy repo https://github.com/kroxylicious/kroxylicious/pull/1569


PR: Fix error when Editor's dockwidget is not yet initialized

Hello @ccordoba12! Thanks for submitting the PR.

There are no PEP8 issues in the file spyder/plugins/editor/plugin.py !


AIPscan is missing an API endpoint for listing Storage Service locations

All of the API endpoints in AIPscan require a storage_service_id, but there isn't a way through the API to discover what Storage Services are available for reporting and what their IDs are.
We may want to add an API endpoint that lists select information about the available Storage Service locations, such as:

Name
ID
Possibly URL (we would need to consider whether there is a security risk to exposing the Storage Service URLs, particularly while we don't yet have authentication for the API)

We'll leave the URL out of the response for now.

PR ready for review: https://github.com/artefactual-labs/AIPscan/pull/269

PR merged.


How to catch component's event?

Timeline just have piana or router's event.

thanks for your answer.


Users can't contact admin

Hello Again
I send a member (tester) a message as admin and when he tried to reply to the admin via message he couldn't, instead said to use contact us.
Can that be changed

This is done on purpose. Admins cannot handle messages.
Otherwise, you would have lots of messages from users, etc. (you don't want that; You need to save your time as much as possible and not shooting the breeze).
Admins aren't there to communicate with users, but they can send messages to them to inform about something (the message should be specific enough, so users won't have to reply to you).
If you want to chat with users, just create a user profile for that or email them (you have the email address of each user on the User Browse Page.
FYI, I added a mailto link on each email, so admins can email users even easier (https://github.com/pH7Software/pH7-Social-Dating-CMS/commit/3f237aa9917e22f7effc0b32b77609cd6fdbdec8). This will be available in the next release.


Document intended workflow with imgpkg

Describe the problem/challenge you have
While integration with imgpkg is mentioned (specifically that one can generate a kind: ImagesLock), the user is left to infer the intended workflow. There are details (e.g. how meta is handled, how references are relocated, explicitly that this lockfile is .imgpkg/images.yml) that are no documented anywhere.
Describe the solution you'd like
Document the end-to-end workflow:
kbld --imgpkg-lock-output=.imgpkg/images.yml ... ==> imgpkg copy ... ==> imgpkg pull ==> kbld -f .imgpkg/images.yml ...
Anything else you would like to add:

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

i believe such documentation already is under imgpkg. if you think it's appropriate to link to it from kbld docs, that's probably good, but i would be a bit on the fence to introduce similar content in kbld directly.

An idea for improvement is to link to the imgpkg documentation on 'Generating resolution imgpkg lock output'. I would even go one step further and say it may deserve its own page to make it more discoverable.

Perfect.


[formatter] Indent docstrings using configured indent_style

Summary
Ruff currently always formats docstrings using spaces for indentation. However, when indent-style is set to "tab", this leads to E101 (mixed indentation) linter errors.
There are a few approaches to improving the situation, some of which are discussed in https://github.com/astral-sh/ruff/issues/8430. One option which is likely the simplest is to recommend turning off E101 when using the formatter. If this is the preferred route, then Ruff should probably also respect the indent-width when expanding tabs in docstrings to avoid the issue described here.
This PR implements an alternative solution which indents docstrings using the configured indent-style. In cases where docstrings have indents that aren't a multiple of the indent-width, this may mean E101 is still triggered in the formatted code. However, this should be very uncommon. I did a cursory check of the few codebases across GitHub that use Ruff with tabs and could not find any cases where docstrings would be negatively affected by this change.
Note that this means that Ruff will format code with tabs differently than Black, which always expands tabs to 8 spaces. I think this is acceptable in this case, because Black doesn't support tabs to begin with (so there's no real compatibility issue).
There is also the question of how other tooling treats tabs in docstrings. As of Python 3.13, docstrings will be dedented to save memory. However, tabs will also unfortunately be expanded to 8 spaces by the interpreter, which will negate the memory usage savings, but this difference should also be mostly inconsequential (and this may potentially be fixed in CPython in the future.)

Thank you for this PR and pushing for another decison.
I don't feel comfortable making this decision right now because I don't have a good enough understanding of docstring handling in Python and I want to avoid the situation where we change the definition only to later change it again. The PR also seems to change docstring formatting when using indent-style=space, meaning this PR reduces black compatibiliity. This doesn't mean we shouldn't move forward with this PR, it's just that I need to find time time to go through the issues and familiarize myself with Python's docstring handling. I'm happy to move forward with this PR if someone else from the team has enough context and feels comfortable making the decision.
Note: We need to double check how this affects code block formatting inside docstrings (@BurntSushi)

I'm now looking into this and plan to make a decision tomorrow.

I understand that changing tabs in docstrings to spaces can be surprising if a specific documentation convention uses them. Unfortunately, changing spaces to a tab isn't safe when the spaces are used for alignment.
I believe the proper fix is to add support for docstring formatting (that supports the different conventions). This way, we avoid the risk of breaking alignment and match the expectation of using tabs in these cases.
See https://github.com/astral-sh/ruff/issues/8430#issuecomment-1934209625 for an in-depth explanation


Question - Multi-region support future plans (if any)

Let us know the feedback or general question
We're currently using the Hub and Spoke network topology and we have a requirement to roll out (mutually independent) workloads across up to 6-8 regions.
I see that there has been some work done to support 2 regions, so we can currently deploy workloads in 2 regions as spokes to their respective hubs.
Are we expecting support for more than 2 regions in the future or can we use the current functionality to deploy across as many regions as we want?
For example, could we make 2-region deployments per region pair:

eastus2 - centralus
northeurope - westeurope
etc

How would you advise we proceed if we had to implement this right now?
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Hi @cspring86, great question! Yes, this is something we are actively working on as part of the Azure Verified Modules (AVM) Refactor (#791). The goal is to enable the creation of multiple hub networks, each with its own peering settings.
We’re aiming to have this implemented in the early months of the new year. In the meantime, if you’d like to get started, you can use the same module we plan to leverage for this functionality: Hub Networking Module.


Have an Articulate Theme tree to allow devs to edit theme files in the back office

It would also be handy to be able to copy themes

This is working now


Q: SignalR ServiceFabric

I'm wondering if SignalR could be hosted from inside a Stateless Service ?
and if so how could/is this done ?

Like any other regular ASP.NET Core service that runs in service fabric.

but what about StatelessService can SignalR be used outside of ASP.NET Core service ? or has it a dependency of kestrel etc. in ASP.NET Core.  sorry I'm not fully up todate with channels/pipline etc. and this may be a numpty question ;) loz

but what about StatelessService can SignalR be used outside of ASP.NET Core service

That's not a thing. SignalR runs on ASP.NET Core.

Ok, that's interesting
I'm trying to use ASP.NET Core 2.0 Preview-3 within ServiceFabric;  I'm getting the following error installing nugget packages ? do I need to target a different framework, other than 4.6.1 which the ServiceFabric templates use as default ?

I'm using VS2017 v15.3.0 Preview 3.0 to create a ServiceFabric project using Stateless ASP.NET Core - Web Application targeting ASP.NET 2.0; this template produces a project which targets the .NET Framework 4.6.1 which is .NET Standard 2.0 compliant? however when attempting to reference ASP.NET Core nuget components marked as .NETStandard 2.0 these fail and rollback ?
any ideas ?

in fact this is happening for any ASP.NET Core 2.0 - (.NET Framework) application ?

@davidfowl ok got more info the nuget package of SignalR 1..0.0-preview3-25967 seams to work ok, but the 1.0.0-preview3-25972 and above are giving me issues? and rolls back?
Restoring packages for ...\Web2\Web2.csproj...
GET https://api.nuget.org/v3-flatcontainer/system.runtime.compilerservices.unsafe/index.json
OK https://api.nuget.org/v3-flatcontainer/system.runtime.compilerservices.unsafe/index.json 475ms
GET https://dotnetmyget.blob.core.windows.net/artifacts/aspnetcore-dev/nuget/v3/flatcontainer/system.runtime.compilerservices.unsafe/index.json
OK https://dotnetmyget.blob.core.windows.net/artifacts/aspnetcore-dev/nuget/v3/flatcontainer/system.runtime.compilerservices.unsafe/index.json 688ms
Package restore failed. Rolling back package changes for 'Web2'.
Time Elapsed: 00:00:02.0552791
========== Finished ==========
BTW I'm using the https://dotnet.myget.org/F/aspnetcore-dev/api/v3/index.json nuget feed ?

Closing as this was resolved

@davidfowl thanks to all the team, great work, beers on me 😁


feat: allow handlers to return a 'null' response

I have a use case where I want response handlers to be able to return null. Using return null! works, so there is nothing in the library preventing null responses except the nullable metadata. I know changing the nullable metadata is breaking change, but I thought I would open this PR for consideration anyway so that the nullability metadata can be made accurate.
Sorry if there is an existing issue for this. I did search, but did not find one.

I vote against this as it would give me a lot of work.

I think you could implement the NULL pattern in any response. The null pattern will help you to cover all the scenarios. NULL values are bad options most of the time.

Yes, I think you'd be better off implementing the Null/Maybe pattern in this case. I've never had a need to return null because I always expect some response.


Nova feature

Description

A clear and concise summary of the change and which issue (if any) it fixes. Should also include relevant motivation and context.
Resolved or fixed issue:
Affirmation

[ ] My code follows the CONTRIBUTING.md guidelines

sorry guys, just testing


Fixed VM package importation.

 Thought I'd help others.

This is the correct code for the master branch. For the current released version look in the stable branch.


If search results overflow off screen, it is difficult to scroll down

Checklist

[x] I have read the tutorials and know the correct effect of the functional design.
[x] There are no similar reports on existing issues (including closed ones).
[x] I found the bug on the latest code of master branch.

Describe the bug
The .search_result_wrapper uses position: fixed;, which helps situate the results at the top of the page. But there is a problem is you have many results: if the results overflow from the screen, then 1) an extra scrollbar shows up, and 2) it's awkward to scroll down to see the remaining results.
To Reproduce
Steps to reproduce the behavior:

You need a site that has more posts than the current theme preview site.
Search for a term that is used on all (or many) of the pages, so that you have many results.
Result: The results overflow off the screen, but it is difficult to get to them.

Expected behavior
I would expect to be able to scroll. This can be achieved with position: absolute; .search_result_wrapper, and also adding height: 0; to the .hidden class. But it changes the page design a little bit. It's not perfect (which is why I didn't make a PR with it, because you may have other preferences).
Additional Issues with the search-loader!
I am tacking on two small search-loader issues, in case you find them compelling: It would be good to have conditionals on the {categories} and {tags} in the search-loader, so if a post does not have them, then those tags (and icons) do not show up in the results. Right now, if you have a post that doesn't have a tag, it literally says "{tags}" in the search results. I had some trouble getting a conditional to function within that capture.
Thanks!
Screenshots

Software

Ruby version:
Gem version:
Bundler version:
Jekyll version:

Desktop

Smartphone

Hi @RyanTG,
Both issues have been fixed, thx for the report.


Recipes exist for unbuildable buildings

After spending 30 minutes and a bit of resources to build the Electric Arc Furnace, only to find it is not buildable, I must admit I was frustrated. This and the rolling machine both have recipes that show up in JEI and indicate they are constructable, as are all of their components (such as the bricks), but the main brick to complete them is unavailable. It would be best to blacklist these recipes entirely if possible.

I'll add tooltip notifications they arent buildable for the blocks.


Fix displaying third party webpack plugins errors

Some third party webpack plugins can generate custom single-line compiler errors without stacktrace.
For example with html-webpack-plugin I got following error as formatMessage argument:
'Template execution failed: TypeError: Cannot read property \'main\' of undefined'

As fast solution I propose to add empty line to avoid big changes in formatMessage logic.

Can you please give us a reproducing case that triggers this exact error?

@Timer
I have ejected react-scripts and now I try to upgrade to 2.x version.
I use custom html template where I use this line:
var scriptSrc = '<%= htmlWebpackPlugin.files.chunks.main.entry %>';

By some reasons files.chunks is not defined, but I can't see this error, instead I got following error:
  if (lines[1].indexOf(moduleWarningPrefix) === 0) {
               ^

TypeError: Cannot read property 'indexOf' of undefined
    at formatMessage (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/react-dev-utils/formatWebpackMessages.js:44:16)
    at /Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/react-dev-utils/formatWebpackMessages.js:169:12
    at Array.map (<anonymous>)
    at formatWebpackMessages (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/react-dev-utils/formatWebpackMessages.js:168:37)
    at compiler.run (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/scripts/build.js:126:24)
    at finalCallback (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:204:39)
    at hooks.done.callAsync.err (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:253:14)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/tapable/lib/Hook.js:154:20)
    at emitRecords.err (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:251:22)
    at Compiler.emitRecords (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:366:39)
    at emitAssets.err (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:245:10)
    at hooks.afterEmit.callAsync.err (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:352:14)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/tapable/lib/Hook.js:154:20)
    at asyncLib.forEach.err (/Users/vhavrylin/dev/git.ceb.loc/ch-web-widget/node_modules/webpack/lib/Compiler.js:349:27)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Excellent. Thank you.


modules and decorators

Trigger registers twice when using decorator in a module.
Test code
modules/dec_test.py:
def startup_with_note(note):

    def inner_dec(func):
        nonlocal note

        @time_trigger('startup')
        def inner(*args, **kwargs):
            nonlocal note
            log.info(f"NOTE: {note}")
            return func(*args, **kwargs)

        return inner

    return inner_dec

script file:
import dec_test

@dec_test.startup_with_note('this is the note 1')
def at_startup():
    log.info('startup 1')

logs:
2020-12-26 07:51:48 INFO (MainThread) [custom_components.pyscript.global_ctx] Reloaded /config/pyscript/scripts/test/dec_test_script.py
2020-12-26 07:51:48 INFO (MainThread) [custom_components.pyscript.modules.dec_test.inner] NOTE: this is the note 1
2020-12-26 07:51:48 INFO (MainThread) [custom_components.pyscript.modules.dec_test.inner] startup 1
2020-12-26 07:51:48 INFO (MainThread) [custom_components.pyscript.modules.dec_test.inner] NOTE: this is the note 1
2020-12-26 07:51:48 INFO (MainThread) [custom_components.pyscript.modules.dec_test.inner] startup 1

Moving the function out of the module and into the script file makes the log lines only show once.

This should be fixed now.

Tested. This works perfectly. Thanks!


Set runner group for runners with enterprise scope

so far, runner group parameter is only set for runners with org scope
now set group for enterprise runners as well
removed null check for org scope as either org or enterprise will be set

... just noticed wrong base, will propose this directly to upstream ...


FLINK-3327: ExecutionConfig to JobGraph.

This makes the ExecutionConfig available to the Task. In a nutshell, the ExecutionConfig is attached to the JobGraph which is sent to the JobManager. The JobManager passes it to the ExecutionGraph, and, later on, to the TaskDeploymentDescriptor, and the TaskManager puts it into the Environment, which is visible to the AbstractInvocable.

Looks good at a first glance. I'll try to go through it and see if I can merge it...

Thanks a lot @StephanEwen

Please review the new pull request.
This pull request is the first step for the one about FLINK-2523.
Thanks a lot!

Hello! Just rebased to the new master. Please review.

we could probably heavily reduce the diff by providing constructors with/without the executionconfig argument. (if none is given, we just instantiate a new one)

Made a few remarks, but didn't find any issue that isn't a minor one. Good job!

Hello! I just rebased to the new master.
Please review this new PR.

Hello!
I just rebased to the new master.
Please review this new PR.

Please Review.

Changes look good. Do you know what happened to the ExecutionStateProgressTest diff?

The problem is the CRLF vs LF characters in the committed versions of the files.

Please Review this PR.

Just rebased. Please review this PR.

@StephanEwen , @zentol  I just rebased to the latest master.
It passes all tests on my Travis (https://travis-ci.org/kl0u/flink/builds/114228524).
Please review because  the PR touches a lot of classes.

looks like something went wrong while rebasing, a commit by aljoscha suddenly popped up.
but hey, that can be fixed while merging, +1 from me.

Thanks a lot @zentol

merging this.

Thanks a lot!


chore: invoke deprecated methods in default supports method for backwards compatibility

Invoke deprecated methods in default supports method to ensure that extensions are not broken, but one day when those are removed it will return only true
Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[X] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Code here looks fine but I don't know any of the context around this change.

@StevenMassaro the new support method was introduced so specially NoSQL databases can check for other supports as Collections without changing Database class and adding a supportsCollections() method.
In core all the locations were supportsSequences() existed were changed to supports(Sequence) , and all *Database classes had this new method implemented. But extensions don't have it, so they could return the wrong value. For instance, BigQuery doesn't support sequences but would start to return true to core.
With this fix we continue to return what is expected.
I'm adding this info to the PR description, thanks!


Complete migrations

This won't pass CI etc. yet, so merging it into a temporary branch, planning one more PR where I setup the seed data so that we can now automatically have a test database setup.
I change the history of migrations in this PR, but when it gets merged into swe-final-project it should come with the seed as well so you can setup a test database from scratch without any dumps.
This completes the migrations so they can run on an empty database and setup our database structure. It's both useful for developer experience, but especially also for database unit tests, so we can easily setup temporary databases for testing

Also, I just remembered that I forked this off the #364 branch so wait with reviewing this until #364 is merged

Sounds good @emilgoldsmith. Just ping me whenever it is ready for review.


Get circle ci running

Description
A circle ci config is added but it is not running. We should get it running and make it merge blocking. This is something only an OP team member can do
Tasks

[ ] Add lint build and test to circle ci config
[ ] Add this repo to circle ci through circle ci config
[ ] #52

Closing we will just move mocktimism to monorepo


Difficulty decrypting songs

Bug in deezer service
Sorry, songs are not decrypted correctly, please check
Songs play at high speeds and are not recognizable

Deezer implementation is still incomplete, and I currently don't have time to fix it currently.


FINERACT-1724: fix sql injection pattern check

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

Failing mysql and mariadb integ test check is overruled, those errors to be fixed by this PR: https://github.com/apache/fineract/pull/3345


Tweak pre-commit autoupdate CI workflow

This PR makes a few tweaks to the new pre-commit CI workflow. This workflow now skips the push and PR-creating steps when no changes are present (i.e., when pre-commit is already up-to-date). Also, PRs created via this action will automatically be assigned reviewers and a ci label.

Checklist

[x] Testing comment in the PR documents testing used to verify the changes

Testing
I tested this pretty extensively on my own fork of mache. I had some issues understanding the syntax related to the GITHUB_ENV variables. I ended up testing two scenarios:

There are no updates to pre-commit when the workflow runs. This causes the push and pr-create steps to be skipped as expected.
There are updates to pre-commit when the workflow runs. This causes all steps to run, creates a PR, and adds labels and reviewers as expected.
@xylar let me know if you can think of any other scenarios that might be useful to test.

Your testing sounds fine.  I can't think of anything else to try out.


Register in log who made the change

1st I don't know if this is possible by HA guidelines.
Currently i'm using NFC tags to disarm alarms (alarmo entities)
I wrote as tag_id, each user PIN.
So I can send notifications like this: "Alarm disarmed by {{changed_by}}"
but this could be improved if the entity saved that information in the log. because that's persistent information that we can check at any time in the future, while notifications are volatile.
Currently its saved like: "Home changed to Disarmed by service alarm_control_panel.alarm_disarm"
Probably the Alarmo users must be linked to HA users...
Another option can be a special service from Alarm to do this...

I have no idea how to make the logbook entries look pretty.
AFAIK this is handled by HA internally.
I agree that it would be nice, also to be able to say that Alarmo triggered the siren etc. 👍

Just to give you some props:

This is now possible with HA automations.
My car auto disarms garage alarm on arrival

There's some ongoing improvements for this, see also here: https://github.com/home-assistant/architecture/issues/473# and here: https://github.com/home-assistant/frontend/pull/8231
Still from the backend-side I don't really know how to implement this.
I assume you just made an automations for this job?
The HA code of automations is really not that transparent, I took a look before but couldn't make much sense of it.

Yes, It was an automation, that was triggered by de "device" Honda, and disarmed the Alarmo

Once again I tried to pick this up, but no success.
self.hass.async_create_task(
    self.hass.services.async_call(
        "input_boolean", "turn_off", {"entity_id": "input_boolean.my_switch"}, context=Context(id="6698f2533feb32dc9c1dc99f61a02301")
    )
)

No matter which variation of context input I try, it always shows up in the logbook like this:

The code used by HA for automations is very complex to understand, so I am stuck at this point.
I also asked for help on the discord channel of HA, but no response.


[Question] Remove backup manually

We had to delete the volume backup files manually on the disk during an incident that required us to need disk space very urgently.
This has resulted in Longhorn backups being stuck, with no option to remove them. When we try via the UI, this happens:
fail to delete snapshot: error deleting snapshot 'backup-92130cae': Failed to execute: /var/lib/rancher/longhorn/engine-binaries/longhornio-longhorn-engine-v0.7.0/longhorn [--url <IP_ADDRESS>:10001 snapshot rm backup-92130cae], output , stderr, Failed to delete backup-92130cae: failed to mark disk backup-92130cae as removed for replica <IP_ADDRESS>:10030: rpc error: code = Unknown desc = Fail to mark disk volume-snap-backup-92130cae.img as removed: Cannot find disk file volume-snap-backup-92130cae.img time="2020-04-09T16:16:53Z" level=fatal msg="Error running rm snapshot command: failed to mark disk backup-92130cae as removed for replica <IP_ADDRESS>:10030: rpc error: code = Unknown desc = Fail to mark disk volume-snap-backup-92130cae.img as removed: Cannot find disk file volume-snap-backup-92130cae.img" , error exit status 1

How can we manually remove them from Longhorn?

What have you deleted exactly? The files in the replica directories are not backups. It contains the current volume data.

@yasker the volume-snap-* files basically.
I have removed the volumes and started again anyway, so don't worry too much about that data.
Just need a way to remove the old backups from the Longhorn UI/System.

Just to explain what happened.
Backups were misconfigured and were stacking up way too much. Noticed too late, and k8s started to kill pods (including longhorn) due to disk pressure. This meant I could not remove backups via the UI, so I had to do something, and removing those files manually helped. Things ran fine but I removed the volumes entirely and started again just in case.

@scottrobertson those are the volume data, so if you remove them, I will not expect volume to be functional.
There is a difference between snapshots and backups in Longhorn. Snapshots are stored as a part of volume data inside the cluster, while the backups are the extra copies of data stored somewhere else, e.g. S3 server. Sound like you're talking about snapshots.
If you want to remove snapshots without UI, you can follow the instruction at https://github.com/longhorn/longhorn/issues/1010#issuecomment-573899942 (we should document it)

@yasker thanks. Unfortunately, they do the same. The commands run, but nothing happens:
lhexec pvc-7cd1a662-a85f-42ed-89ef-4b5e5cfef66a snapshot ls
ID
520e6731-e937-49b1-aed8-c0b6c67f5fa3
backup-92130cae
backup-6165d3a4

lhexec pvc-a008f4bf-ee9d-40a2-82e7-62c8c6cb95f0 snapshot rm backup-92130cae
lhexec pvc-a008f4bf-ee9d-40a2-82e7-62c8c6cb95f0 snapshot rm backup-6165d3a4
lhexec pvc-7cd1a662-a85f-42ed-89ef-4b5e5cfef66a snapshot ls

ID
520e6731-e937-49b1-aed8-c0b6c67f5fa3
backup-92130cae
backup-6165d3a4

Plus, for the volumes I have removed, I cannot delete via this method as it just returns that the volume is not found.

@yasker thank you.
Unfortunately doing via CLI fails too:
lh pvc-7cd1a662-a85f-42ed-89ef-4b5e5cfef66a snapshot rm backup-92130cae

Failed to delete backup-92130cae: failed to mark disk backup-92130cae as removed for replica <IP_ADDRESS>:10030: rpc error: code = Unknown desc = Fail to mark disk volume-snap-backup-92130cae.img as removed: Cannot find disk file volume-snap-backup-92130cae.img
FATA[0000] Error running rm snapshot command: failed to mark disk backup-92130cae as removed for replica <IP_ADDRESS>:10030: rpc error: code = Unknown desc = Fail to mark disk volume-snap-backup-92130cae.img as removed: Cannot find disk file volume-snap-backup-92130cae.img
command terminated with exit code 1


Add breaking_description

Is your feature request related to a problem? Please describe.
A new release of git_conventionnal was just published who add breaking_description
Describe the solution you'd like
Add breaking_description to the available meta
Additional context
Could be used to generate Breaking Changes section in template easily.

It will be available starting from the next release. (0.2.5+) Thank you!


Cannot See Actual Metric Names

Since my Grafana 7.0 update, I cannot see the metric names, even when the option is enabled in panel settings. Instead of the metric name, the "Value" text is shown. I've tried using the updated version of this plugin but without success.
Thank you for any tips on how to resolve this issue.
Expected Behavior
Show alias as the metric name.
Current Behavior
Showing "Value" as metric name.
This is current behaviour:

This is behaviour before Grafana update:

Have the same problem! So the plugin is useless without this information.

Same issue here.

Same issue, what version of plugin got installed? I am noticing that I have 0.0.10-dev installed which was default. Where is v0.1.0 per changelog?

Grafana 7.0.3, Discrete Panel 0.0.10-dev as well.

Same issue here ...
Grafana 7.0.5, Discrete Panel 0.0.10-dev

For me the same issue but only @0.0.10-dev, 0.0.9 is working for me on Grafana 7.0.5.

For me the same issue but only @0.0.10-dev, 0.0.9 is working for me on Grafana 7.0.5.

 Thanks for the tip, it really works!

So to sound so naive, but how do you downgrade or install a specific version?

@raydog153 https://github.com/agross/ansible-raspi/commit/bc8e4a25be7f83969064aa7a1a06dfab3fb6af30


would be good to update all rather than just the ones with ^

could you fix it to update all versions?  Even ones with <> and no ^  ?

I created this based off your code.  https://github.com/sgehrman/pubspec_updater

Hi @sgehrman,
I wanted to include the < > from the first version but they are actually a bit more tricky than they seem. Let's have a discussion on how we can approach this.
According to this article: https://dart.dev/tools/pub/dependencies
We can have these possibilities:

=1.2.3
In this case we can easily update the version if we assume that the API will always give a bigger or equal version than the current one (e.g. from >=1.2.3 to >=4.5.6).

1.2.3
Same as the previous one.

<=1.2.3
According to the article, this case is not typically used.
<1.2.3
This syntax is useful in case you want to set an upper bound in case a dependency introduced a breaking change in a given version.
For those that start with <, the library should not update the version since the user explicitly set an upper bound that should not be increased (correct me if I'm wrong).
There are some other more complex situations, for example:

=1.2.3 <2.0.0

In that case if the latest version of that dependency is 1.5.6 then we should update to:

=1.5.6 <2.0.0

But if the latest version is 2.7.8 we should leave it as it is:

=1.2.3 <2.0.0

Regarding the caret syntax, the current version of the library has a bug because it really doesn't respect the current version in the pubspec. For example, the library could update a dependency from version ^1.2.3 to ^2.5.0, which is not correct.
Let me know what you think.

I understand what you you are saying, but I was wanting a tool that just updated the versions to the latest.
I know I could just leave the version blank for this, but I like seeing version numbers.  I would assume that in most cases you want the latest version unless there was a bug in the latest version or you are unwilling to want to fix your code to conform to the latest version.
When I code in JavaScript I use this tool "npm-check-updates" to bump everything to the latest versions.  I then test it and if everything is good, I stick with that. I guess the tool could have options to 'update everything', or do something more intelligent like you proposed.  I'm fairly new to Flutter, so I wonder if using < > <= etc are commonly needed?  I was also thinking of having an option to get rid of the ^ since in JavaScript that has sometimes caused problems as minor updates can sometimes cause bugs.
Let me know if what you think.  Thanks for writing this tool.

Hi @sgehrman,
I think I get your point but for the library to be semantically correct I need to respect all the possible symbols that the dependencies are declared with (^, <, <=, >, >=).
The priority now is to fix the bug with the caret. I created another ticket for that (#2).
Then I will focus on allowing the library to update dependencies with other version constraints.
Unfortunately, I'm currently overwhelmed with other projects so I won't be able to fix these issues any time soon. If you subscribe to them you will receive a notification once I mange to fix them.