Need help using custom ckpt file from S3

Hello!
I'm trying to use a custom ckpt to deploy to banana. My file is in S3 and I tried setting the CHECKPOINT_URL ARG in the Dockerfile with no luck (looks like the default stability weight got loaded instead of my ckpt in the S3 bucket).
I tried setting MODEL_URL to the s3 location as well, and not seeing too much luck either (it reports a tar error of the ckpt file from s3).
Am I approaching this the wrong way? I digged around and saw there's code to convert ckpt to diffuser format (and use it while building on banana). Would appreciate some guidance, thank you! 🙏🏻

Hey, welcome!
You're totally on the right track... this should be working with CHECKPOINT_URL.  Are you able to provide the full build logs so I can see exactly where it's getting stuck?
Two most common reasons:

It couldn't convert the checkpoint file.  Sometimes diffusers convert script fails on certain checkpoints.  Newer versions are more versatile.  What version of diffusers-api are you using?  There's a link in the readme on how to use the v1 prerelease which will probably work better.

Wrong S3 URL, but since it managed to correctly download with MODEL_URL, don't think this is it.  It's possible the checkpoint download code doesn't have the full S3 support, I need to double check this.

Just before a long international flight but will check in tomorrow.  You can send the log in the meantime if you have it 🙏

Thank you for the response Gadi, hope the flight treated you well :).
Re: 1, I think I am using the v1 diffuser-api (the Dockerfile I'm using is from this repo). A stupid reason I could think of could be difference between the file extension being .pth vs. .ckpt but I tried using both format to no avail
Re:2, I noticed that copying the URI from S3 gets a URI that looks like s3://<bucket-name>/model.pth while the Dockerfile expects something like s3:///<bucket-name>/model.pth (with the difference being the extra /). Is that the right observation? (I noticed the download taking place in MODEL_URL when I added the extra /).
This is the log when I set the CHECKPOINT_URL parameter, but left MODEL_URL flag unset (left it alone per the Dockerfile):
You've triggered a build + deploy on Banana. It may take ~1 hr to complete. Thanks for your patience.
Waiting for logs...

SUCCESS: Git Authorization
SUCCESS: Build Started
SUCCESS: Build Finished... Running optimizations

SUCCESS: Model Registered

Your model was updated and is now deployed!

This is when I set the MODEL_URL to the same s3 path as CHECKPOINT_URL:
You've triggered a build + deploy on Banana. It may take ~1 hr to complete. Thanks for your patience.
Waiting for logs...

SUCCESS: Git Authorization
SUCCESS: Build Started
ERROR: Build Failed - Logs follow:

{"stream":"Step 1/38 : ARG FROM_IMAGE=\"gadicc/diffusers-api\""}
{"stream":"\n"}
{"stream":"Step 2/38 : FROM ${FROM_IMAGE} as base"}
{"stream":"\n"}
{"status":"Pulling from gadicc/diffusers-api","id":"latest"}
{"stream":"Step 31/38 : RUN python3 download.py"}
{"stream":" ---\u003e Running in f48408c6849a\n"}
{"stream":"download_model {'model_url': 's3:///<bucket>/model.pth', 'model_id': 'stabilityai/stable-diffusion-2-1-base', 'model_revision': 'fp16', 'hf_model_id': ''}\n{'normalized_model_id': 'models--stabilityai--stable-diffusion-2-1-base--fp16'}\nself.endpoint_url None\nDownloading s3:///<bucket>/model.pth to /root/.cache/diffusers-api/model.pth...\n"}
{"stream":"\u001b[91m\rDownloading:   0%|          | 0.00/2.13G [00:00\u003c?, ?B/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   0%|          | 262k/2.13G [00:00\u003c1:22:25, 431kB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   0%|          | 1.31M/2.13G [00:00\u003c15:59, 2.22MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   0%|          | 6.03M/2.13G [00:00\u003c03:07, 11.3MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   1%|          | 16.0M/2.13G [00:00\u003c01:08, 30.9MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   2%|▏         | 33.3M/2.13G [00:01\u003c00:32, 65.2MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   2%|▏         | 51.1M/2.13G [00:01\u003c00:22, 91.7MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   3%|▎         | 67.9M/2.13G [00:01\u003c00:19, 105MB/s] \u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   4%|▍         | 80.2M/2.13G [00:01\u003c00:23, 88.9MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   5%|▍         | 99.1M/2.13G [00:01\u003c00:18, 111MB/s] \u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   6%|▌         | 117M/2.13G [00:01\u003c00:15, 128MB/s] \u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   6%|▌         | 132M/2.13G [00:01\u003c00:15, 128MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   7%|▋         | 146M/2.13G [00:01\u003c00:18, 108MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   7%|▋         | 160M/2.13G [00:02\u003c00:17, 114MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   8%|▊         | 176M/2.13G [00:02\u003c00:15, 125MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   9%|▉         | 189M/2.13G [00:02\u003c00:15, 122MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:   9%|▉         | 202M/2.13G [00:02\u003c00:15, 123MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  10%|█         | 215M/2.13G [00:02\u003c00:16, 115MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  11%|█         | 228M/2.13G [00:02\u003c00:16, 117MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  11%|█▏        | 245M/2.13G [00:02\u003c00:14, 129MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  12%|█▏        | 263M/2.13G [00:02\u003c00:13, 142MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  13%|█▎        | 278M/2.13G [00:02\u003c00:14, 129MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  14%|█▎        | 291M/2.13G [00:03\u003c00:17, 107MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  14%|█▍        | 306M/2.13G [00:03\u003c00:15, 118MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  15%|█▌        | 323M/2.13G [00:03\u003c00:13, 130MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  16%|█▌        | 338M/2.13G [00:03\u003c00:13, 135MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  17%|█▋        | 352M/2.13G [00:03\u003c00:16, 107MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  17%|█▋        | 364M/2.13G [00:03\u003c00:17, 103MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  18%|█▊        | 376M/2.13G [00:03\u003c00:17, 103MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  19%|█▊        | 395M/2.13G [00:04\u003c00:13, 124MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  19%|█▉        | 413M/2.13G [00:04\u003c00:12, 133MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  20%|██        | 427M/2.13G [00:04\u003c00:14, 115MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  21%|██        | 439M/2.13G [00:04\u003c00:14, 115MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  21%|██▏       | 454M/2.13G [00:04\u003c00:13, 123MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  22%|██▏       | 472M/2.13G [00:04\u003c00:12, 138MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  23%|██▎       | 486M/2.13G [00:04\u003c00:12, 135MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  23%|██▎       | 500M/2.13G [00:04\u003c00:13, 124MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  24%|██▍       | 513M/2.13G [00:04\u003c00:13, 124MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  25%|██▍       | 533M/2.13G [00:05\u003c00:11, 143MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  26%|██▌       | 550M/2.13G [00:05\u003c00:10, 151MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  27%|██▋       | 566M/2.13G [00:05\u003c00:10, 150MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  27%|██▋       | 581M/2.13G [00:05\u003c00:10, 142MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  28%|██▊       | 599M/2.13G [00:05\u003c00:10, 151MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  29%|██▉       | 616M/2.13G [00:05\u003c00:09, 156MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  29%|██▉       | 616M/2.13G [00:05\u003c00:09, 161MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  30%|██▉       | 633M/2.13G [00:05\u003c00:09, 159MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  30%|███       | 650M/2.13G [00:05\u003c00:09, 160MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  31%|███       | 666M/2.13G [00:05\u003c00:10, 141MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  32%|███▏      | 683M/2.13G [00:06\u003c00:09, 148MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  33%|███▎      | 701M/2.13G [00:06\u003c00:09, 157MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  34%|███▍      | 720M/2.13G [00:06\u003c00:08, 166MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  35%|███▍      | 737M/2.13G [00:06\u003c00:08, 164MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  35%|███▌      | 754M/2.13G [00:06\u003c00:10, 137MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  35%|███▌      | 754M/2.13G [00:06\u003c00:11, 124MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  36%|███▌      | 771M/2.13G [00:06\u003c00:10, 132MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  37%|███▋      | 791M/2.13G [00:06\u003c00:09, 147MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  38%|███▊      | 810M/2.13G [00:06\u003c00:08, 159MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  39%|███▉      | 827M/2.13G [00:07\u003c00:09, 131MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  39%|███▉      | 841M/2.13G [00:07\u003c00:09, 135MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  40%|████      | 856M/2.13G [00:07\u003c00:09, 133MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  41%|████      | 873M/2.13G [00:07\u003c00:08, 141MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  42%|████▏     | 890M/2.13G [00:07\u003c00:08, 149MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  42%|████▏     | 905M/2.13G [00:07\u003c00:08, 148MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  43%|████▎     | 922M/2.13G [00:07\u003c00:07, 153MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  44%|████▍     | 937M/2.13G [00:07\u003c00:07, 150MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  45%|████▍     | 953M/2.13G [00:07\u003c00:07, 150MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  46%|████▌     | 972M/2.13G [00:07\u003c00:07, 161MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  46%|████▋     | 988M/2.13G [00:08\u003c00:07, 162MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  47%|████▋     | 1.00G/2.13G [00:08\u003c00:07, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  48%|████▊     | 1.02G/2.13G [00:08\u003c00:07, 155MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  49%|████▉     | 1.04G/2.13G [00:08\u003c00:06, 166MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  50%|████▉     | 1.06G/2.13G [00:08\u003c00:06, 170MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  50%|█████     | 1.08G/2.13G [00:08\u003c00:06, 153MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  51%|█████     | 1.09G/2.13G [00:08\u003c00:06, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  51%|█████     | 1.09G/2.13G [00:08\u003c00:06, 156MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  52%|█████▏    | 1.11G/2.13G [00:08\u003c00:06, 166MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  53%|█████▎    | 1.13G/2.13G [00:08\u003c00:05, 178MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  54%|█████▍    | 1.15G/2.13G [00:09\u003c00:05, 166MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  55%|█████▍    | 1.17G/2.13G [00:09\u003c00:06, 151MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  56%|█████▌    | 1.18G/2.13G [00:09\u003c00:06, 153MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  56%|█████▌    | 1.18G/2.13G [00:09\u003c00:06, 155MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  56%|█████▋    | 1.20G/2.13G [00:09\u003c00:06, 143MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  57%|█████▋    | 1.22G/2.13G [00:09\u003c00:06, 140MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  58%|█████▊    | 1.23G/2.13G [00:09\u003c00:06, 147MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  59%|█████▊    | 1.25G/2.13G [00:09\u003c00:05, 151MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  59%|█████▉    | 1.27G/2.13G [00:09\u003c00:05, 157MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  60%|██████    | 1.28G/2.13G [00:09\u003c00:05, 145MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  61%|██████    | 1.30G/2.13G [00:10\u003c00:06, 136MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  62%|██████▏   | 1.31G/2.13G [00:10\u003c00:05, 139MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  62%|██████▏   | 1.33G/2.13G [00:10\u003c00:05, 147MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  63%|██████▎   | 1.35G/2.13G [00:10\u003c00:04, 159MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  63%|██████▎   | 1.35G/2.13G [00:10\u003c00:04, 167MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  64%|██████▍   | 1.37G/2.13G [00:10\u003c00:04, 163MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  65%|██████▍   | 1.38G/2.13G [00:10\u003c00:05, 150MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  66%|██████▌   | 1.40G/2.13G [00:10\u003c00:04, 150MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  66%|██████▋   | 1.41G/2.13G [00:10\u003c00:04, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  67%|██████▋   | 1.43G/2.13G [00:10\u003c00:04, 157MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  68%|██████▊   | 1.45G/2.13G [00:11\u003c00:04, 162MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  69%|██████▊   | 1.47G/2.13G [00:11\u003c00:04, 152MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  69%|██████▉   | 1.48G/2.13G [00:11\u003c00:04, 152MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  70%|███████   | 1.50G/2.13G [00:11\u003c00:03, 160MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  71%|███████   | 1.52G/2.13G [00:11\u003c00:03, 157MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  72%|███████▏  | 1.53G/2.13G [00:11\u003c00:03, 156MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  73%|███████▎  | 1.55G/2.13G [00:11\u003c00:03, 160MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  73%|███████▎  | 1.56G/2.13G [00:11\u003c00:03, 160MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  74%|███████▍  | 1.58G/2.13G [00:11\u003c00:03, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  75%|███████▍  | 1.60G/2.13G [00:12\u003c00:03, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  76%|███████▌  | 1.61G/2.13G [00:12\u003c00:03, 161MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  77%|███████▋  | 1.63G/2.13G [00:12\u003c00:03, 162MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  77%|███████▋  | 1.65G/2.13G [00:12\u003c00:02, 162MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  78%|███████▊  | 1.66G/2.13G [00:12\u003c00:03, 153MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  79%|███████▉  | 1.68G/2.13G [00:12\u003c00:03, 144MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  80%|███████▉  | 1.70G/2.13G [00:12\u003c00:02, 159MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  81%|████████  | 1.72G/2.13G [00:12\u003c00:02, 171MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  81%|████████▏ | 1.74G/2.13G [00:12\u003c00:02, 163MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  82%|████████▏ | 1.75G/2.13G [00:13\u003c00:02, 150MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  83%|████████▎ | 1.77G/2.13G [00:13\u003c00:02, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  84%|████████▍ | 1.79G/2.13G [00:13\u003c00:02, 162MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  85%|████████▍ | 1.81G/2.13G [00:13\u003c00:01, 172MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  86%|████████▌ | 1.83G/2.13G [00:13\u003c00:02, 141MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  86%|████████▋ | 1.84G/2.13G [00:13\u003c00:02, 128MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  87%|████████▋ | 1.86G/2.13G [00:13\u003c00:01, 143MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  88%|████████▊ | 1.88G/2.13G [00:13\u003c00:01, 155MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  89%|████████▉ | 1.90G/2.13G [00:13\u003c00:01, 166MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  90%|████████▉ | 1.92G/2.13G [00:14\u003c00:01, 133MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  91%|█████████ | 1.93G/2.13G [00:14\u003c00:01, 134MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  92%|█████████▏| 1.95G/2.13G [00:14\u003c00:01, 146MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  92%|█████████▏| 1.97G/2.13G [00:14\u003c00:01, 156MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  93%|█████████▎| 1.99G/2.13G [00:14\u003c00:00, 148MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  94%|█████████▍| 2.00G/2.13G [00:14\u003c00:00, 138MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  95%|█████████▍| 2.02G/2.13G [00:14\u003c00:00, 148MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  96%|█████████▌| 2.04G/2.13G [00:14\u003c00:00, 154MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  96%|█████████▋| 2.05G/2.13G [00:15\u003c00:00, 156MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  97%|█████████▋| 2.07G/2.13G [00:15\u003c00:00, 148MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  98%|█████████▊| 2.09G/2.13G [00:15\u003c00:00, 160MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading:  99%|█████████▉| 2.11G/2.13G [00:15\u003c00:00, 166MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading: 100%|█████████▉| 2.12G/2.13G [00:15\u003c00:00, 163MB/s]\u001b[0m"}
{"stream":"\u001b[91m\rDownloading: 100%|██████████| 2.13G/2.13G [00:15\u003c00:00, 136MB/s]\n\u001b[0m"}
{"stream":"\u001b[91mzstd: /*stdin*\\: unsupported format \n\u001b[0m"}
{"stream":"\u001b[91mtar: Child returned status 1\ntar: Error is not recoverable: exiting now\n\u001b[0m"}
{"stream":"\u001b[91mTraceback (most recent call last):\n  File \"/api/download.py\", line 175, in \u003cmodule\u003e\n    download_model(\n  File \"/api/download.py\", line 75, in download_model\n\u001b[0m"}
{"stream":"\u001b[91m    subprocess.run(\n  File \"/opt/conda/envs/xformers/lib/python3.9/subprocess.py\", line 528, in run\n\u001b[0m"}
{"stream":"\u001b[91m    raise CalledProcessError(retcode, process.args,\n\u001b[0m"}
{"stream":"\u001b[91msubprocess.CalledProcessError: Command '['tar', '--use-compress-program=unzstd', '-C', '/root/.cache/diffusers-api/models--stabilityai--stable-diffusion-2-1-base--fp16', '-xvf', '/root/.cache/diffusers-api/model.pth']' returned non-zero exit status 2.\n\u001b[0m"}
{"stream":"\u001b[91mERROR conda.cli.main_run:execute(47): `conda run /bin/bash -c python3 download.py` failed. (See above for error)\n\u001b[0m"}
{"errorDetail":{"code":1,"message":"The command '/opt/conda/bin/conda run --no-capture-output -n xformers /bin/bash -c python3 download.py' returned a non-zero code: 1"},"error":"The command '/opt/conda/bin/conda run --no-capture-output -n xformers /bin/bash -c python3 download.py' returned a non-zero code: 1"}

Thank you again for your help 🙏🏻

I might be looking at the wrong place, but is the correct flow in the download.py for the ckpt to go to this line?
Seems like my build is getting stuck here.
I could be looking at older version of the API (or misunderstanding something) but wanted to point it out just in case it's helpful.
Thank you again!

Hey, @DenimMazuki!  Thanks for your patience and detailed reporting and troubleshooting steps.
Ok great, you're indeed on the new v1, the correct repo, and, just all round doing everything right actually :)  And yes, in our S3 URLs we expect a triple slash ("///") in the beginning unless you want to explicitly set an ENDPOINT_URL (which is the much less common case) - there are some very brief docs about that here.
And indeed, you want CHECKPOINT_URL and not MODEL_URL.  I've thought about combining the two, but, since conversion is not necessarily 1:1 if you don't get the configuration correct, I decided against it, even though it would be a little easier and more intuitive in most (but not all) cases.  I did also check that CHECKPOINT_URL is indeed using the updated S3 download code, so not expecting any problems here.
In short, all looks good.  It's a pity banana doesn't give the full build logs on a successful build too, in case anything came up there.  However, I'm recalling now that you said it's downloading the default stability weights...  so here's my game plan:

Could you please try setting the build-arg MODEL_ID to something unique, and see if that makes a difference?
Failing that, could you send the full runtime logs from the call, just so I can see exactly what is happening on this side of the equation?
Lastly, any chance you could give me access somewhere to the checkpoint file, so run the conversion locally and see if there's anything useful in the build output?

We'll get this working and thanks for your patience too over my travels.

Hey @gadicc , thank you for the detailed response! I really appreciate the responses :)
Gameplan 1: I set the MODEL_ID into something unique and got a build error regarding downloading the repo on HF
2023-02-17T14:38:50.000Z You've triggered a build + deploy on Banana. It may take ~1 hr to complete. Thanks for your patience.
Waiting for logs...

SUCCESS: Git Authorization
SUCCESS: Build Started
ERROR: Build Failed - Logs follow:

{"stream":"Step 1/38 : ARG FROM_IMAGE=\"gadicc/diffusers-api\""}
{"stream":"\n"}
{"stream":"Step 2/38 : FROM ${FROM_IMAGE} as base"}
{"stream":"\n"}
{"status":"Pulling from gadicc/diffusers-api","id":"latest"}
{"status":"Digest: sha256:a98c52386a23f0c68d159f23d9f0b4e88f2cfcd7c0c8f8833151ca7affc3831f"}
{"status":"Status: Image is up to date for gadicc/diffusers-api:latest"}
{"stream":" ---\u003e 8e20dd55e5a8\n"}
{"stream":"Step 3/38 : ENV FROM_IMAGE=${FROM_IMAGE}"}
{"stream":"\n"}
{"stream":" ---\u003e Using cache\n"}
{"stream":" ---\u003e b802fbdfbe20\n"}
{"stream":"Step 4/38 : ARG MODEL_ID=\"something-unique\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 74b835e75dbc\n"}
{"stream":" ---\u003e 3d116f08acc6\n"}
{"stream":"Step 5/38 : ENV MODEL_ID=${MODEL_ID}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in e72f23377dd2\n"}
{"stream":" ---\u003e 7855c26e3dc3\n"}
{"stream":"Step 6/38 : ARG HF_MODEL_ID=\"\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 770b14ef4d74\n"}
{"stream":" ---\u003e 23a6d3c341da\n"}
{"stream":"Step 7/38 : ENV HF_MODEL_ID=${HF_MODEL_ID}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in 1f08822a2b2a\n"}
{"stream":" ---\u003e a772cef0a9d8\n"}
{"stream":"Step 8/38 : ARG MODEL_PRECISION=\"fp16\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 8b32a482186f\n"}
{"stream":" ---\u003e c5c5f054e40c\n"}
{"stream":"Step 9/38 : ENV MODEL_PRECISION=${MODEL_PRECISION}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in ab2ce97ea8df\n"}
{"stream":" ---\u003e 66f46c521857\n"}
{"stream":"Step 10/38 : ARG MODEL_REVISION=\"fp16\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 21a3fd0683b9\n"}
{"stream":" ---\u003e 2a4e359e948b\n"}
{"stream":"Step 11/38 : ENV MODEL_REVISION=${MODEL_REVISION}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in 6d493d6fabd6\n"}
{"stream":" ---\u003e 3626502d2183\n"}
{"stream":"Step 12/38 : ARG MODEL_URL"}
{"stream":"\n"}
{"stream":" ---\u003e Running in 642d7dabba3f\n"}
{"stream":" ---\u003e 8dafdd6c3ce0\n"}
{"stream":"Step 13/38 : ENV MODEL_URL=${MODEL_URL}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in 75c45f9dbfe1\n"}
{"stream":" ---\u003e 44595abc3f5d\n"}
{"stream":"Step 14/38 : ARG CHECKPOINT_URL=\"s3://<bucket>/model.pth\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 226f2a270f69\n"}
{"stream":" ---\u003e 94e6d58b35bb\n"}
{"stream":"Step 15/38 : ENV CHECKPOINT_URL=${CHECKPOINT_URL}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in ec5d6c678ea2\n"}
{"stream":" ---\u003e ce840451980a\n"}
{"stream":"Step 16/38 : ARG CHECKPOINT_CONFIG_URL=\"\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 7a2d17387a33\n"}
{"stream":" ---\u003e 2680223e087e\n"}
{"stream":"Step 17/38 : ENV CHECKPOINT_CONFIG_URL=${CHECKPOINT_CONFIG_URL}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in 9d887618e5a1\n"}
{"stream":" ---\u003e<PHONE_NUMBER>9e\n"}
{"stream":"Step 18/38 : ARG PIPELINE=\"ALL\""}
{"stream":"\n"}
{"stream":" ---\u003e Running in 8e8cb5bf2284\n"}
{"stream":" ---\u003e 9ab472dbbeae\n"}
{"stream":"Step 19/38 : ENV PIPELINE=${PIPELINE}"}
{"stream":"\n"}
{"stream":" ---\u003e Running in f032f4b08235\n"}
{"stream":" ---\u003e 06b307219eb6\n"}
{"stream":"Step 31/38 : RUN python3 download.py"}
{"stream":"\n"}
{"stream":" ---\u003e Running in b542319dca3f\n"}
{"stream":"\u001b[91mTraceback (most recent call last):\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/utils/_errors.py\", line 264, in hf_raise_for_status\n\u001b[0m"}
{"stream":"download_model {'model_url': '', 'model_id': 'something-unique', 'model_revision': 'fp16', 'hf_model_id': ''}\nloadModel {'model_id': 'something-unique', 'load': False, 'precision': 'fp16', 'revision': 'fp16'}\nDownloading model: something-unique (fp16)\nInitializing DPMSolverMultistepScheduler for something-unique...\n"}
{"stream":"\u001b[91m    response.raise_for_status()\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/requests/models.py\", line 1021, in raise_for_status\n\u001b[0m"}
{"stream":"\u001b[91m    raise HTTPError(http_error_msg, response=self)\nrequests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://huggingface.co/something-unique/resolve/main/scheduler/scheduler_config.json\n\nThe above exception was the direct cause of the following exception:\n\nTraceback (most recent call last):\n  File \"/api/diffusers/src/diffusers/configuration_utils.py\", line 326, in load_config\n\u001b[0m"}
{"stream":"\u001b[91m    config_file = hf_hub_download(\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py\", line 124, in _inner_fn\n\u001b[0m"}
{"stream":"\u001b[91m    return fn(*args, **kwargs)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/file_download.py\", line 1105, in hf_hub_download\n\u001b[0m"}
{"stream":"\u001b[91m    metadata = get_hf_file_metadata(\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py\", line 124, in _inner_fn\n\u001b[0m"}
{"stream":"\u001b[91m    return fn(*args, **kwargs)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/file_download.py\", line 1440, in get_hf_file_metadata\n\u001b[0m"}
{"stream":"\u001b[91m    hf_raise_for_status(r)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/utils/_errors.py\", line 306, in hf_raise_for_status\n\u001b[0m"}
{"stream":"\u001b[91m    raise RepositoryNotFoundError(message, response) from e\nhuggingface_hub.utils._errors.RepositoryNotFoundError: 401 Client Error. (Request ID: Root=1-63ef9189-26224b654059849d236317da)\n\nRepository Not Found for url: https://huggingface.co/something-unique/resolve/main/scheduler/scheduler_config.json.\nPlease make sure you specified the correct `repo_id` and `repo_type`.\nIf you are trying to access a private or gated repo, make sure you are authenticated.\nInvalid username or password.\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File \"/api/download.py\", line 175, in \u003cmodule\u003e\n\u001b[0m"}
{"stream":"\u001b[91m    download_model(\n  File \"/api/download.py\", line 148, in download_model\n\u001b[0m"}
{"stream":"\u001b[91m    loadModel(\n  File \"/api/loadModel.py\", line 52, in loadModel\n\u001b[0m"}
{"stream":"\u001b[91m    scheduler = getScheduler(model_id, DEFAULT_SCHEDULER, not load)\n  File \"/api/getScheduler.py\", line 88, in getScheduler\n\u001b[0m"}
{"stream":"\u001b[91m    scheduler = initScheduler(MODEL_ID, scheduler_id, download)\n  File \"/api/getScheduler.py\", line 50, in initScheduler\n\u001b[0m"}
{"stream":"\u001b[91m    inittedScheduler = scheduler.from_pretrained(\n  File \"/api/diffusers/src/diffusers/schedulers/scheduling_utils.py\", line 134, in from_pretrained\n\u001b[0m"}
{"stream":"\u001b[91m    config, kwargs = cls.load_config(\n  File \"/api/diffusers/src/diffusers/configuration_utils.py\", line 341, in load_config\n\u001b[0m"}
{"stream":"\u001b[91m    raise EnvironmentError(\nOSError: something-unique is not a local folder and is not a valid model identifier listed on 'https://huggingface.co/models'\nIf this is a private repository, make sure to pass a token having permission to this repo with `use_auth_token` or log in with `huggingface-cli login`.\n\u001b[0m"}
{"stream":"\u001b[91mERROR conda.cli.main_run:execute(47): `conda run /bin/bash -c python3 download.py` failed. (See above for error)\n\u001b[0m"}
{"errorDetail":{"code":1,"message":"The command '/opt/conda/bin/conda run --no-capture-output -n xformers /bin/bash -c python3 download.py' returned a non-zero code: 1"},"error":"The command '/opt/conda/bin/conda run --no-capture-output -n xformers /bin/bash -c python3 download.py' returned a non-zero code: 1"}

GP 2: Failing the first, here's the runtime logs on the re-deploy (with stability weights)
17 Feb, 11:16:24
if not hasattr(tensorboard, "__version__") or LooseVersion(
17 Feb, 11:16:24
/opt/conda/envs/xformers/lib/python3.9/site-packages/torch/utils/tensorboard/__init__.py:4: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
17 Feb, 11:16:25
np.bool8: (False, True),
17 Feb, 11:16:25
/opt/conda/envs/xformers/lib/python3.9/site-packages/skimage/util/dtype.py:27: DeprecationWarning: `np.bool8` is a deprecated alias for `np.bool_`.   Ensure that you abide to the conditions of the Stable Diffusion license and do not expose unfiltered results in services or applications open to the public. Both the diffusers team and Hugging Face strongly recommend to keep the safety filter enabled in all public facing circumstances, disabling it only for use-cases that involve analyzing network behavior or auditing its results. For more information, please have a look at https://github.com/huggingface/diffusers/pull/254 .
17 Feb, 11:16:44
0%|          | 0/50 [00:00<?, ?it/s]
  2%|▏         | 1/50 [00:00<00:44,  1.11it/s]
  6%|▌         | 3/50 [00:01<00:13,  3.56it/s]
 10%|█         | 5/50 [00:01<00:07,  5.97it/s]
 14%|█▍        | 7/50 [00:01<00:05,  8.22it/s]
 18%|█▊        | 9/50 [00:01<00:04, 10.18it/s]
 22%|██▏       | 11/50 [00:01<00:03, 11.81it/s]
 26%|██▌       | 13/50 [00:01<00:02, 13.11it/s]
 30%|███       | 15/50 [00:01<00:02, 14.12it/s]
 34%|███▍      | 17/50 [00:01<00:02, 14.88it/s]
 38%|███▊      | 19/50 [00:01<00:02, 15.44it/s]
 42%|████▏     | 21/50 [00:02<00:01, 15.84it/s]
 46%|████▌     | 23/50 [00:02<00:01, 16.13it/s]
 50%|█████     | 25/50 [00:02<00:01, 16.32it/s]
 54%|█████▍    | 27/50 [00:02<00:01, 16.44it/s]
 58%|█████▊    | 29/50 [00:02<00:01, 16.54it/s]
 62%|██████▏   | 31/50 [00:02<00:01, 16.59it/s]
 66%|██████▌   | 33/50 [00:02<00:01, 16.66it/s]
 70%|███████   | 35/50 [00:02<00:00, 16.71it/s]
 74%|███████▍  | 37/50 [00:03<00:00, 16.73it/s]
 78%|███████▊  | 39/50 [00:03<00:00, 16.75it/s]
 82%|████████▏ | 41/50 [00:03<00:00, 16.77it/s]
 86%|████████▌ | 43/50 [00:03<00:00, 16.75it/s]
 90%|█████████ | 45/50 [00:03<00:00, 16.73it/s]
 94%|█████████▍| 47/50 [00:03<00:00, 16.68it/s]
 98%|█████████▊| 49/50 [00:03<00:00, 16.71it/s]
100%|██████████| 50/50 [00:03<00:00, 13.05it/s]
17 Feb, 11:16:44
2023-02-17 16:16:44.479397 {'type': 'inference', 'status': 'done', 'container_id': 'denimmazukibananadeployment0027d4a105d53d594b3093eaa817cc9c6625', 'time':<PHONE_NUMBER>479, 't': 18601, 'tsl': 5093, 'payload': {'startRequestId': None}}
17 Feb, 11:16:44
- (sanic.access)[INFO][<IP_ADDRESS>:58246]: POST http://<IP_ADDRESS>:8000/  200 574321

GP 3: Yes! I'll send the path to you privately on discord :)
Thank you again and hope that's helpful!

Hey @DenimMazuki.
Ok great, thanks so much for all this... I can see exactly what's going on now.
Firstly, let me apologize :sweat_smile:  It was a silly issue on my side.   I know you've spent a few days on this and how frustrating that can be.  But we're very close to an official v1 release and your pain will help many others moving forward ;)
I've released a fix to the dev branch.  To try this out, it's as simple as editing the Dockerfile and changing:
ARG FROM_IMAGE="gadicc/diffusers-api"     # <- from this
ARG FROM_IMAGE="gadicc/diffusers-api:dev" # <- to this

(or overriding that build arg variable on the banana dashboard, but last time I checked, you still have to push a commit anyways to get it to use the new values)
The reason why this wasn't working was an faulty assumption that checkpoints would be optimized first and saved to S3.  This will work now regardless, but since we don't support banana's optimization anymore, your cold starts will be slower.  There's info on creating an optimized build at https://forums.kiri.art/t/safetensors-our-own-optimization-faster-model-init/98.
The short of it is, you need a 2nd deployment of the main repo (not build-downloads), which can perform the optimization for you.  It's not too much extra work but is a a little time consuming, especially the first time, so as a thank you for your patience, I'm happy to do the conversion for you if you send the checkpoint file.
(The reason why we need a second deploy is because banana builds happen without GPU, which is required for optimization.  In banana's case, after the first stage of the build, they move the built image to separate "optimization servers" to complete the optimization, but we have no way of hooking into this).
Anyway, hope you find all the extra info interesting, otherwise you can safely ignore for now and just get up and running in the meantime without the optimization.  I should be around to help if you experience any further issues.  And thanks again for your patience :)

Whoa, sorry, one other note... I haven't deployed to banana in quite a while, and things have improved there big time!!!
First, their optimization worked, which I wasn't expecting (it was breaking on docker-diffusers-api for a really long time!)... so even if you don't use our optimization, you'll still get there's.
Secondly, wow, they've really improved things... everything completed in a few minutes instead of the 1hr+ I remember.
Anyways, just wanted to correct my earlier note about things being slow without using "our" optimization.


System state covered by MARS agent

I believe the MARS agent option under "Azure VM backup support" should include system state in "What's backed up?"
This is directly stated on the front page for Backup.
https://docs.microsoft.com/en-us/azure/backup/backup-overview#what-can-i-back-up
Azure VMs
Run MARS agent
Back up files, folders, system state.
The MARS agent is also referenced explicitly in the instructions for restoring system state.
https://docs.microsoft.com/en-us/azure/backup/backup-azure-restore-system-state

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

ID: 5f33a676-6bb6-949f-3bf4-bb0f9a8297d0
Version Independent ID: 9a31c4c4-f191-1298-cdbf-be70e59f4578
Content: Azure Backup support matrix
Content Source: articles/backup/backup-support-matrix.md
Service: backup
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@dcbrown16 Thanks for the feedback. We are looking into this and will get back to you shortly.

@dcbrown16 Appreciate your time here! Based on your feedback we've updated the document to reflect MARS agent support for "System State". We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Case insensitive string <-> enum support?

I'm not sure if you support this, but for serialization purposes, it would be nice for case insensitivity when converting from string to enum.
Basically, a string "FOO" should map to an enumerator Foo.

I have this: http://aantron.github.io/better-enums/ApiReference.html#_from_string_nocase
Of course, as it is now, you have to choose to call it – the >> operator uses the case-sensitive function _from_string.

Presuming this is addressed, and closing. Let me know if I'm mistaken :)


Enable ruff linter

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.59%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

circuit_knitting_toolbox/entanglement_forging/entanglement_forging_ground_state_solver.py
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1515

Relevant Lines:
1791

💛  - Coveralls

@caleb-johnson, the remaining problem that ruff discovers is:
circuit_knitting_toolbox/entanglement_forging/cholesky_decomposition.py:332:9: F841 [*] Local variable `num_gammas` is assigned to but never used
Found 1 error.

This is from
https://github.com/Qiskit-Extensions/circuit-knitting-toolbox/blob/389b8b8039b6726ec86996f518c6d8b4cbd82ebb/circuit_knitting_toolbox/entanglement_forging/cholesky_decomposition.py#L325-L333
Should num_gammas be used?  If not, we could assign it to _, I suppose, to get rid of the error.

@caleb-johnson, the remaining problem that ruff discovers is:
circuit_knitting_toolbox/entanglement_forging/cholesky_decomposition.py:332:9: F841 [*] Local variable `num_gammas` is assigned to but never used
Found 1 error.

This is from
https://github.com/Qiskit-Extensions/circuit-knitting-toolbox/blob/389b8b8039b6726ec86996f518c6d8b4cbd82ebb/circuit_knitting_toolbox/entanglement_forging/cholesky_decomposition.py#L325-L333
Should num_gammas be used? If not, we could assign it to _, I suppose, to get rid of the error.

Hey thanks for this PR. I've been using ruff locally too, and all the output has been giving me anxiety hah!
I think assigning it to _ is right. I've seen a few other unused variables in the ruff output as well in my branch. Not sure which of them were unique to my branch and which came from main

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

5 of 8 (62.5%)  changed or added relevant lines in 5 files are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 84.59%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

circuit_knitting_toolbox/entanglement_forging/cholesky_decomposition.py
1
2
50.0%

circuit_knitting_toolbox/entanglement_forging/entanglement_forging_ground_state_solver.py
0
1
0.0%

circuit_knitting_toolbox/utils/orbital_reduction.py
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

circuit_knitting_toolbox/entanglement_forging/entanglement_forging_ground_state_solver.py
8
82.56%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1515

Relevant Lines:
1791

💛  - Coveralls


Split samples and real cloud support code

First iteration, not ready yet, just want let everything run through CI.

So it is mostly done now and ready for a review. The last thing I was going to do is to change the python sample to use the changed tooling.


Detect "history" files as a changelog

Quite a few open source projects use a file named "HISTORY" as a changelog. It's also confirmed by this Wikipedia page on the subject.
This PR updates the file linter to also detect history files.

Thank you!
Especially appreciate the wiki link :smile:


JWTsecret to relative path

Changes:

When not provided, the jwt secret is set as a relative path. This allows users to share their setup more easily.

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[x] Yes
[ ] No

In case you checked yes, did you write tests?

[x] Yes
[ ] No

Comments about testing , should you have some (optional)
Further comments (optional)
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report
Attention: Patch coverage is 71.42857% with 6 lines in your changes missing coverage. Please review.

Project coverage is 24.76%. Comparing base (6e485d7) to head (5b2a894).

Files with missing lines
Patch %
Lines

cli/actions/jwt_secrets.go
72.72%
2 Missing and 1 partial :warning:

cli/generate.go
70.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #484   +/-   ##
========================================
  Coverage    24.75%   24.76%
========================================
  Files          120      120
  Lines        22299    22317   +18
========================================
+ Hits          5521     5527    +6
- Misses       16235    16243    +8
- Partials       543      547    +4

Flag
Coverage Δ

unittests
24.76% <71.42%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Add unit tests

Fixes #8

:rocket: PR was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


add timeouts

Add timeouts so that it is possible to specify things such as "the user has 15 min to accept or the booking is cancelled".

requires pascalpoizat/veca-ide#3


Leptos integration/helpers in Roadster?

Only if the leptos options can be parsed from a regular config file instead of a Cargo.toml. Maybe could open a PR on Leptos if it's not currently possible.

Only if the leptos options can be parsed from a regular config file instead of a Cargo.toml

I think it can for leptos, but I think it needs to be in Cargo.toml for the cargo-leptos command. Maybe can open a PR for cargo-leptos change how/where the config is read from.

Not sure how useful this would be. The integration is already pretty straight-forward, so users can just copy from an example.


No pdf output

@gtheler reports no pdf output, probably missing package.
@gtheler , do you have any error message?

Working on automating this now, basically you need (in format command: package):
#qrencode: qrencode
#base64: coreutils
#zbarimg: install zbar-tools
#gzip: gzip
#gunzip: gzip
#split: coreutils
#dd: coreutils
#truncate: coreutils
#convert: imagemagick
#img2pdf: img2pdf
#zbar-tools: zbar-tools
Installing these on Ubuntu18.04LTS should give you high enough versions for full functionability. Checking for package versions etc is on my todo list, will do ASAP.
Let me know if installing all packages fixes your problem @gtheler :) .

Try
for i in qrencode base64 zbarimg gzip gunzip split dd truncate convert img2pdf zbar-tools; do
 if [ -z "`which $i`" ]; then
   echo "error: $i is not installed"
   exit 1
 fi
done

Yes, working on it, for now going to eat, will fix this tonight :)

@gtheler thanks for your help, now integrated in the package:
https://github.com/jerabaul29/qrdump/blob/39b95b35832df7d0295163721daaf2c834195493/src/qrdump.sh#L8-L12
https://github.com/jerabaul29/qrdump/blob/39b95b35832df7d0295163721daaf2c834195493/src/check_dependencies.sh#L1-L8
I went quickly through the code and checked, that should be all that is needed. But curious to hear if this helps fix the no pdf output problem. Let me know if this is not enough. If this is enough, can you confirm and close? :)

Is this still a problem @gtheler or did the warnings about missing packages allowed you to catch all missing dependencies and get a pdf out? :)

well I installed img2pdf and it worked so I think this can be closed

That's great, thanks, closing.


Deploy model server on new environment and run benchmark to create models.

0.5 release

Should cover the issue https://github.com/sustainable-computing-io/kepler-model-server/issues/94

Now simplify training step here:
https://github.com/sustainable-computing-io/kepler-model-server/tree/main/model_training
Describe the instruction to contribute in contributing page:
https://github.com/sustainable-computing-io/kepler-model-server/blob/main/contributing.md


PP-6709 Only show all service transactions link for live service users

remove feature flag that has been enabled in all environments.
only show all service transactions link if a user has access to live
services that they would have permissions to see.

👍 It uses already existing/ presumably tested methods to do the filtering logic so I figured it was mostly covered.
Passing the value through to the template correctly isn't covered by that.

@stephencdaly thanks for the comments; added that coverage in https://github.com/alphagov/pay-selfservice/pull/2089/commits/6b12baf74be46497533a07da7c5fa04a9f310596


VPA-updater gets OOM killed

How to categorize this issue?

/area scalability
/kind bug
/priority 3
What happened:
On Seed cluster that has VPA enabled, the vpa-updater of a Shoot cluster is OOMKilled, because it recommends to low memory:
    State:          Waiting
      Reason:       CrashLoopBackOff
    Last State:     Terminated
      Reason:       OOMKilled
      Exit Code:    137
      Started:      Wed, 24 Mar 2021 15:39:51 +0200
      Finished:     Wed, 24 Mar 2021 15:40:38 +0200
    Ready:          False
    Restart Count:  59
    Limits:
      cpu:     44m
      memory:  125Mi
    Requests:
      cpu:     11m
      memory:  50Mi

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Gardener version:
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

We have a VPA for it, see https://github.com/gardener/gardener/blob/master/charts/seed-bootstrap/charts/vpa/charts/runtime/templates/vpa-updater.yaml.
As it's not clear how to continue here and as it doesn't seem to be too relevant as of today, let's
/close
for now until it becomes relevant again.


Improve recognition for contributions

Recognition for activity and contributions to ctapipe is something that could be improved upon in at least two ways:

Use zenodo to get a DOI (#678)
Utilise Github's CODEOWNERS functionality (#569)

Both of these were suggested in previous issues.
More on codeowners:
https://help.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners
I don't recommend activating the requiring a review from a code owner for a PR to be accepted - I don't think this would be productive for us. But being able to see who is responsible for maintaining particular code could be helpful

added DOI and codeowners (see #1150  and #1151)


Move {En,De}codeImageJPGCoefficients out of codec_jpg

This is a step towards making codec_* not depend on CodecInOut /
ImageBundle.

Oops, build_cleaner, one second.


max_tokens_to_sample: 9016 - Is this the Anthropic API maximum?

This is regarding this line.
We cannot find the 9016 token maximum anywhere in the Anthropic or any third-party documentation. The max_tokens_to_sample is mandatory, though, so we must set something (obviously <<10000).
If there is no hard limit in Anthropic's API, why not explicitly leave it to the script user:
// Config constant for max tokens
const MAX_TOKENS = 9016;

async function handleRequest(request) {

  // ...

  const claudeRequestBody = {
    prompt,
    model: claudeModel,
    temperature,
    stop_sequences: stop,
    stream,

    // Always include max_tokens, using constant as default
    max_tokens_to_sample: MAX_TOKENS
  };

  // ...

}

Either way, we appreciate having found your nifty script.

9016 is an empirical value, because Cloudflare Worker only supports API response time within 10ms for free plan. However, this value does not seem to work recently, Proxy API endpoint deployed using Cloudflare Worker now often have the problem of returning truncated. I have adopted the Docker deployment solution myself.
You are right, this value can be set as a constant for easy modification by those in need, I will extract it.

The value has been extracted.


Variables with old name reappear when renamed after switching sprites

Expected Behavior
Renamed variable should stay renamed
Actual Behavior
There are now 2 variables, one with the old name
Steps to Reproduce

Go to Data. Create a variable.
Drag one of the data blocks to the workspace
Add a new sprite
Drag "set x to" block to workspace
In the dropdown, rename variable. Choose a new name
You can also rename variables by right clicking on them in the flyout and choosing rename

all appears well

Return to the first sprite
New variable with old name has appeared

Operating System and Browser
Windows 10 chrome

Dupe of https://github.com/LLK/scratch-blocks/issues/1183
The fact that it occurs on sprite switch is a result of how scratch-gui uses scratch-blocks, but doesn't make it a different bug.


Broken Links in tutorial

In the "Start a local relay chain" tutorial
https://docs.substrate.io/tutorials/connect-other-chains/relay-chain/
Under the Pre-configured chain spec files section:
The links for the chainspecs are broken:

https://docs.substrate.io/assets/tutorials/cumulus/chain-spec/rococo-custom-2-plain.json/
https://docs.substrate.io/assets/tutorials/cumulus/chain-spec/rococo-custom-2-raw.json/

There are also several image links missing. For example in https://docs.substrate.io/tutorials/connect-other-chains/parachain/#block-finalization :
<Image is missing>

included in #1140


Extendable Clients

Now will generate the extended interface's http method implementation.

adds a parameter to @Client that will disable generation is desired
modify generation to read super client interface methods

@rbygrave plz review and deploy when you get time


Loading indicators

While requests are being made, would be nice if you could tell!

Ian added opacity loading effects earlier this year 👍


ActiveQuery relations via 3 tables

What steps will reproduce the problem?
4 table:
[table1]

id, primaryKey
name, string

[table2]

id, primaryKey
table1_key, int - notNull
name, string

[table3]

id, primaryKey
table2_key, int - notNull
name, string

[table4]

id, primaryKey
table3_key, int - notNull
name, string

Relations:
[table4]
public function getTable3(): ActiveQuery { return $this->hasOne(Table3::class, ['id' => 'table3_key']); }
public function getTable2(): ActiveQuery { return $this->hasOne(Table2::class, ['id' => 'table2_key'])->via('table3'); }
public function getTable1(): ActiveQuery { return $this->hasOne(Table1::class, ['id' => 'table1_key'])->via('table2'); }
SearchModel:
$query = Table4::find()->with(['table3', 'table2', 'table1'])
What is the expected result?
Getting 'Table1' model with data.
What do you get instead?
In debug database tab:

SELECT * FROM table1 WHERE id IN (1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 14, 11, 12, 13, 15, 16, 17, 18, 19)

When I want to echo table1.name getting this error:

Getting unknown property: name

When I var_dump() the model:

null

When I don't use ->with(), it works correctly, but i have a lot of sql query.
What could I have messed up?
Additional info

Q
A

Yii version
2.0.45

PHP version
7.4.26

Operating system
Windows 10 - wampserver

$query = Table4::find()->with('table1')


[7.1.0] Please ignore. This is for testing.

Forked from #463

PR failed to be created.

PR failed to be created.

PR failed to be created.


Create better documentation

I found two interesting libraries for svelte, which will allow creation of far better documentation than the one I started creating.
sveld - will serve to generate typescript definitions
mdsvex - will make it easier to write documentation thanks to md with component embedding

Due to official release documentation of beta version wont be further improved except bugfixes.


Documentation improvement

translation as feature (nearly completely missing)
translation for developers (maybe fix minor issues)
translation as part of the release-management (get started)
translation for translators (misses details and parts)

I know that this issue is under-specified. I will work on this issue in the upcoming time.

on this note, please also have a look at the docs: https://inveniordm.docs.cern.ch/maintenance/documentation/
Moving forward, we would like to standardize documentation in order to introduce a structure. The tutorials provide a great reference point on where to place certain documentation as well as where to put certain details and where to omit them :)


Test the tutorials in Windows

Make sure the tutorials work in a Windows box.

This was done. Sahana ensured our presented examples are using the cross-platform version of React


Genvar not found, related to n-dim array and parameters?

In the following code, the genvar index is not found when used in the initialization of a localparam.
This seems to be related to multi-dimensional arrays, packed structs, and parameters (tried to simplify, but removing any of these aspects does not trigger this bug anymore).
module submodule #(
  parameter logic [0:2][31:0] two_d_logic_array = '{default: 0}
) ();
endmodule

package pkg;
  typedef struct packed {
    logic [0:2][31:0][0:3]  three_d_logic_array;
  } a_packed_struct;
endpackage

module top #(
  parameter pkg::a_packed_struct my_packed_struct = '{ three_d_logic_array: '{default: 0} }
)();
for (genvar index = 0; index < int'(4); index++) begin
    localparam logic [0:2][31:0] my_two_d_logic_array = my_packed_struct.three_d_logic_array[index]; // <-- ERROR: Identifier `\index' is implicitly declared and `default_nettype is set to none.
    submodule #(
      .two_d_logic_array ( my_two_d_logic_array )
    ) sub1 ();
end
endmodule

Debug output:
synlig> read_systemverilog -debug id_not_found.sv

1. Executing SystemVerilog frontend.
[INF:CM0023] Creating log file "/.../slpp_all/surelog.log".
[INF:CP0300] Compilation...
[INF:CP0301] /.../id_not_found.sv:7:1: Compile package "pkg".
[INF:CP0303] /.../id_not_found.sv:2:1: Compile module "work@submodule".
[INF:CP0303] /.../id_not_found.sv:13:1: Compile module "work@top".
[INF:CP0302] Compile class "work@mailbox".
[INF:CP0302] Compile class "work@process".
[INF:CP0302] Compile class "work@semaphore".
[INF:EL0526] Design Elaboration...
[INF:CP0335] /.../id_not_found.sv:16:50: Compile generate block<EMAIL_ADDRESS>[INF:CP0335] /.../id_not_found.sv:16:50: Compile generate block<EMAIL_ADDRESS>[INF:CP0335] /.../id_not_found.sv:16:50: Compile generate block<EMAIL_ADDRESS>[INF:CP0335] /.../id_not_found.sv:16:50: Compile generate block<EMAIL_ADDRESS>[NTE:EL0503] /.../id_not_found.sv:13:1: Top level module "work@top".
[NTE:EL0508] Nb Top level modules: 1.
[NTE:EL0509] Max instance depth: 3.
[NTE:EL0510] Nb instances: 5.
[NTE:EL0511] Nb leaf instances: 4.
[INF:UH0706] Creating UHDM Model...
[INF:UH0707] Elaborating UHDM...
[  FATAL] : 0
[ SYNTAX] : 0
[  ERROR] : 0
[WARNING] : 0
[   NOTE] : 5
Object 'work@top' of type 'design'
  Object 'builtin' of type 'package'
  Object 'pkg' of type 'package'
    Object 'pkg::a_packed_struct' of type 'struct_typespec'
      Object 'three_d_logic_array' of type 'typespec_member'
        Object '' of type 'range'
          Object '' of type 'constant'
          Object '' of type 'constant'
        Object '' of type 'range'
          Object '' of type 'constant'
          Object '' of type 'constant'
        Object '' of type 'range'
          Object '' of type 'constant'
          Object '' of type 'constant'
  Object 'work@top' of type 'module_inst'
    Object 'my_packed_struct' of type 'parameter'
      Object 'pkg::a_packed_struct' of type 'struct_typespec'
        Object 'three_d_logic_array' of type 'typespec_member'
          Object '' of type 'range'
            Object '' of type 'constant'
            Object '' of type 'constant'
          Object '' of type 'range'
            Object '' of type 'constant'
            Object '' of type 'constant'
          Object '' of type 'range'
            Object '' of type 'constant'
            Object '' of type 'constant'
    Object '' of type 'param_assign'
      Object 'my_packed_struct' of type 'parameter'
        Object 'pkg::a_packed_struct' of type 'struct_typespec'
          Object 'three_d_logic_array' of type 'typespec_member'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
      Object '' of type 'operation'
        Object '' of type 'operation'
          Object '' of type 'tagged_pattern'
            Object '' of type 'constant'
    Object 'genblk1[0]' of type 'gen_scope_array'
      Object '' of type 'gen_scope'
        Object 'my_two_d_logic_array' of type 'parameter'
          Object '' of type 'range'
            Object '' of type 'constant'
            Object '' of type 'constant'
          Object '' of type 'range'
            Object '' of type 'constant'
            Object '' of type 'constant'
        Object 'index' of type 'parameter'
        Object '' of type 'param_assign'
          Object 'my_two_d_logic_array' of type 'parameter'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
          Object 'my_packed_struct.three_d_logic_array[index]' of type 'hier_path'
            Object 'my_packed_struct' of type 'ref_obj'
            Object 'three_d_logic_array' of type 'bit_select'
              Object 'index' of type 'ref_obj'
        Object 'sub1' of type 'module_inst'
          Object 'two_d_logic_array' of type 'parameter'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
            Object '' of type 'range'
              Object '' of type 'constant'
              Object '' of type 'constant'
          Object '' of type 'param_assign'
            Object 'two_d_logic_array' of type 'parameter'
              Object '' of type 'range'
                Object '' of type 'constant'
                Object '' of type 'constant'
              Object '' of type 'range'
                Object '' of type 'constant'
                Object '' of type 'constant'
            Object 'my_packed_struct.three_d_logic_array[index]' of type 'hier_path'
              Object 'my_packed_struct' of type 'ref_obj'
              Object 'three_d_logic_array' of type 'bit_select'
                Object 'index' of type 'ref_obj'
/.../id_not_found.sv:17: ERROR: Identifier `\index' is implicitly declared and `default_nettype is set to none.

The essence of the problem laid in simplify_parameter function which it is executed while initial AST tree is being formed.
That function calls synlig_simplify function on some nodes and those calls demand properly set scope.
Because of the fact that in that moment AST tree is not properly formed yet, scope has to be artificially created.
The issue was that Synlig didn't consider adding nodes from currently created subtree to scope.
https://github.com/chipsalliance/synlig/pull/2674 should fix this issue.


Spec compliance: Errors do not propagate to the closest nullable field

How to reproduce:

Define a schema with a nullable field, and a non-nullable sub-field
Have the schema for the non-nullable field return an error

Expected:
The nullable field should be null and the error should point to that field. Specifically, the non-nullable field should not appear in the response because it must never be null - the nullable parent field must be null instead.
From the spec:

Since Non-Null type fields cannot be null, field errors are propagated to be handled by the parent field. If the parent field may be null then it resolves to null, otherwise if it is a Non-Null type, the field error is further propagated to it’s parent field.

http://facebook.github.io/graphql/draft/#sec-Errors-and-Non-Nullability
Actual:
Both the nullable and the non-nullable field will be present, with the non-nullable field being null and the error in the errors array pointing to that error. This is not spec-compliant.

Version: b46637030579abd312c5eea21d36845b6e9e7ca4 (latest)

I'm interested in this feature. I think it wouldn't be as hard as it seems because there's already some reasonable buffer management going on.
The general rule that needs to be enforced is that if the current field has any non-nullable sub-fields, it can't flush its output until after its non-nullable sub-fields have succeeded. To avoid premature optimization, I plan to make every field with non-nullable sub-fields wait for all sub-fields to finish, buffer the output, and then decide to: 1. flush it on success, 2. flush a null if it's nullable or 3. do nothing and expect the parent to flush a null eventually
The parent will inspect Request.Errs after executing the child fields to detect if any non-nullable children failed and react accordingly.
Does this make sense? Any objections before I start?

@felixfbecker yes, #296 addresses this issue.


Run processes in the background

It would be nice to have a way of running processes in the background, similar to docker-compose up -d.
Maybe devenv up -d would be an intuitive choice?

That's implemented in #745

Hi @domenkozar ! I tried reading the link you mentioned, it took me to a python rewrite and then to a rust rewrite. I'm using devenv as a flake but when I tried running the command devenv up -d as the original author suggested it still runs as normal with a TUI and everything. How could I run services in a detached state?

It does not work with flakes yet.

Hi! I found a solution for my use case and I'm sharing it here in case someone else ends here. I wanted to run "devenv up" in CI from a github action to automatically setup postgress and other background services of my project. Turns out the default TUI of process-compose was the problem so searching on the documentation I managed to add the following configuration:
default = devenv.lib.mkShell {
          inherit inputs pkgs;
          modules = [
            {
              process = {
                process-compose = pkgs.lib.mkOptionDefault {
                  tui = "false";
                };
              };

              services.postgres = {
                enable = true;
                listen_addresses = postgresHost;
                port = postgresPort;
                initialScript = dbInitFile;
              };
            }
          ];
        };

pkgs.lib.mkOptionDefault overrides only the field tui inside the configuration that process-compose uses.


TextFlowContainer includes spaces in word layout

This breaks centered text.

I attempted to fix this but have no idea how we can make this work. At first I was thinking we should just not include the spaces in words themselves (and rely on FillFlowContainer.Spacing) but this doesn't work well with multiple TextSizes in the same FillFlowContainer.

Moving this out of milestones because it requires some inspiration to solve.


cilium: allow encryption/decryption to coexist with bpf_host logic

We have added logic in the bpf_host that is attached to egress/ingress network facing device. This is the same device that is used by bpf_network for encryption. The result is bpf_network is loaded by bpf_init.sh and then bpf_host based programs are loaded by golang bpf loader replacing the bpf_network program. This results in traffic drops because encryption rules exist in xfrm datapath, but BPF side is not correctly implementing them. To fix make bpf_host do correct decrypt logic.
Additionally, workaround a kernel regression by simplifing the encryption/decryption rules in the encrypt-node case. Kernels 4.19 and earlier appear to work correctly with the current setup, but newer kernels (at least 5.4) is broken with the current setup. The new cleaner approach appears to work on all kernels testsed

First patch pulled into this PR https://github.com/cilium/cilium/pull/13241 once we get that merged I'll pop the first patch off this series.

retest-4.9

@qmonnet should we remove this from the checkpatch tests as well?
WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#156:
new file mode 100644

Also for now I've kept ipsec blocked by bandwidth manager, because I haven't walked through the details yet to see if it is also working. But, it should be getting closer now.

@qmonnet should we remove this from the checkpatch tests as well?

Agreed, it's on the todo list (see related issue). I'll try to free some cycles to work on this soon.

test-me-please

Consulted data around failure the failure in kernel-netnext seems to be happening in multiple places, perhaps a flake?
Suite-k8s-1.12.K8sPolicyTest Multi-node policy test validates ingress CIDR-dependent L4 With host policy Connectivity is restored after importing ingress policy

@pchaigno I assume for bandwidth manager you had to disable the check to not allow it with encryption. But, host firewall do we need a check?

Consulted data around failure the failure in kernel-netnext seems to be happening in multiple places, perhaps a flake?
Suite-k8s-1.12.K8sPolicyTest Multi-node policy test validates ingress CIDR-dependent L4 With host policy Connectivity is restored after importing ingress policy

I suspect this might be the same underlying bug as is affecting #12596. The two tests are very similar, just the above tests a host policy instead of "pod policy".

@pchaigno I assume for bandwidth manager you had to disable the check to not allow it with encryption. But, host firewall do we need a check?

Yes, we have check for each of these when combined with IPSec. I removed them to be able to test.

So unless I'm mistaken we have a checkpatch warning thats not needed, WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? and a flake? So we should be OK to merge.

Would it now be possible to merge bpf_network.c logic into bpf_host.c and remove the former entirely?

@borkmann I believe so thats the next step, but first need to fix above error


On aws with nvme the script always adds after reboot wrong disk on lvm

On m5 instances, with nvme:

configure docker-storage-setup with the real disk name (nvme1n1 in this case)
run container-storage-setup. Everything is fine for now
add more ebs voluyme to instance
reboot instance
now another disk is nvme1n1, and the original one is, for example, nvme5n1
container-storage-setup adds the new disk also to the lvm, by wiping the disk completely. Of course, all data is lost now on that disk

# cat /etc/sysconfig/docker-storage-setup
DEVS=nvme1n1
GROWPART=true
ROOT_SIZE=+100%FREE
STORAGE_DRIVER=overlay2
VG=docker
CONTAINER_ROOT_LV_NAME=pool00
CONTAINER_ROOT_LV_SIZE=100%FREE
CONTAINER_ROOT_LV_MOUNT_PATH=/var/lib/docker
POOL_META_SIZE=1G
WIPE_SIGNATURES=true
EXTRA_STORAGE_OPTIONS="--storage-opt overlay2.size=20G"

Is there any way to resolve this?

problem still exists


Update keycloak-admin-client, ... to 19.0.0

Updates

org.keycloak:keycloak-admin-client
org.keycloak:keycloak-core

from 18.0.2 to 19.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.keycloak" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.keycloak" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #319.


Added User Testimonials and Achievements

#190
https://www.loom.com/share/14779fe1c0a84c3ba9fe90e205b1d37a?sid=62382537-2c8a-4284-b3d8-448f2a71ba3a
screenshot

cab you fix the conflicts

@rishicds i have resolved the conflicts, please review it and merge it as soon as possible as the open source going to end soon

@rishicds i have resolved the conflicts, please review it and merge it as soon as possible as the open source going to end soon

@ananyag309 sorry, but the homepage is totally messed up, after merging this branch,
You have merged incorrectly the conflicting code,
Closing tags without opening.......
Refer the below images.

@rishicds please always try to take a followup video of UI after any kind of merge conflicts

@Shariq2003 its running on my side of the code ...hows that possible?

Is it due to I clicked on resolved conflicts? Cuz the code changes I made is perfectly good and working

@Shariq2003 its running on my side of the code ...hows that possible?

I have raised a PR for the fixes, that have merged.

#334

@Shariq2003 its running on my side of the code ...hows that possible?

#276


Add the 'About Us' section in the home page

Please assign me this issue

Hey there's been a change in the structure. The About Us page is now changed to two different pages for doctor and patient. If u see the home page there are  two cards namely for doctor and patient.  The 'learn more' button takes u to a another page I.e. the About Us page. I have already created those 2 pages in the AboutUs folder. There's only styling and content left to do. If u want I'll assign you the issue but then u have to work on both the pages.


going modular: keep executing train.py with no output!

when I run train.py by : !python going_modular/train.py
it keeps running and gives no output , what's the problem?
I also changed the train_dir and test_dir :
train_dir = "../data/pizza_steak_sushi/train"
test_dir = "../data/pizza_steak_sushi/test"
and it raised below error:
FileNotFoundError: [WinError 3] The system cannot find the path specified: '../data/pizza_steak_sushi/train'
I use Jupyter notebook(locally) not google colab

Maby you should search for how to give full directory in python in jupyter .cues it means system can not find the directory at all. Try to work with directories in python in Jupyter

Hi @mam1680 ,
Do the following directories exist on your local machine?
train_dir = "../data/pizza_steak_sushi/train"
test_dir = "../data/pizza_steak_sushi/test"
If not, you may have to create them.
Also where are you running the !python going_modular/train.py command?
Is it inside a notebook?
Does this notebook have access to the right paths (listed above)?


Basic question about using promises with syncronous methods

Hello! I'm trying to understand how to organize my code but the docs leave me scratching my head. I have a lot of code which is synchronous but takes a long time to process. Is it possible to do something like this?
firstly {
  asyncOpThatReturnsAPromise()
}.then {
  syncOpThatDoesntReturnAPromise()
  return Promise.resolved()
}.done {
  displaySomethingToUser()
}

What is your goal? I think we may be able to suggest some options for you if we know what you're trying to achieve.

Something like this presumably is what you want.
firstly {
  asyncOpThatReturnsAPromise()
}.done(on: myBgQueue) {
  syncOpThatDoesntReturnAPromise()
}.done {
  displaySomethingToUser()
}

Thanks, that's very helpful! I wasn't sure if the chain help up and I didn't know I could chain multiple done()s... I have another place where I'd like to do something similar earlier in the chain...
This is currently what I have:
  func displayImagesAsync(_ images: [UIImage]) -> Guarantee<Void> {
    return
      firstly { self.clearImagesAsync() }
      .then { ImageStore.createAsync(images: images) }
      .done { store in
        self.frameImageStore = store
        self.animatedImageView.image = store.frameImage()
      }
  }

but it seems a bit silly since clearImagesAsync() is just doing this:
  func clearImagesAsync() -> Guarantee<Void> {
    return Guarantee { resolver in
      DispatchQueue.main.async {
        self.animatedImageView.image = nil
        self.frameImageStore = nil
        resolver(())
      }
    }
  }

I was wondering if I could do something more like this:
  func displayImagesAsync(_ images: [UIImage]) -> Guarantee<Void> {
    return
      firstly {
        self.frameImageStore = store
        self.animatedImageView.image = store.frameImage()
        return Guarantee.resolve()
      }
      .then { ImageStore.createAsync(images: images) }
      .done { store in
        self.frameImageStore = store
        self.animatedImageView.image = store.frameImage()
      }
  }

This?
func displayImagesAsync(_ images: [UIImage]) -> Guarantee<Void> {
    self.animatedImageView.image = nil
    self.frameImageStore = nil

    return firstly {
        ImageStore.createAsync(images: images)
    }.done { store in
        self.frameImageStore = store
        self.animatedImageView.image = store.frameImage()
    }
}

@mxcl I don't think that works because displayImagesAsync is called called from a background thread and since  self.animatedImageView.image = nil effects the UI it needs to be called from the main thread.

wow, thank you! that never would have occurred to me.


Fix ndarray pretty printer

No, that fixes the print, but not the original problem.  Without any print statement, I'm getting
==19729== Conditional jump or move depends on uninitialised value(s)
==19729==    at 0x18C90196: __lower_bound<double const*, double, __gnu_cxx::__ops::_Iter_less_val> (stl_algobase.h:959)
==19729==    by 0x18C90196: lower_bound<double const*, double> (stl_algobase.h:993)
==19729==    by 0x18C90196: searchsorted<double, (anonymous namespace)::pythonic::types::ndarray<double, (anonymous namespace)::pythonic::types::pshape<long int> > > (searchsorted.hpp:27)
==19729==    by 0x18C90196: operator()<(anonymous namespace)::pythonic::types::ndarray<double, (anonymous namespace)::pythonic::types::pshape<long int> >, double&> (searchsorted.hpp:27)
==19729==    by 0x18C90196: __pythran__empirical_prob::empirical::type<(anonymous namespace)::pythonic::types::ndarray<double, (anonymous namespace)::pythonic::types::pshape<long> >&, double&>::result_type __pythran__empirical_prob::empirical::operator()<(anonymous namespace)::pythonic::types::ndarray<double, (anonymous namespace)::pythonic::types::pshape<long> >&, double&>((anonymous namespace)::pythonic::types::ndarray<double, (anonymous namespace)::pythonic::types::pshape<long> >&, double&) const [clone .isra.248] (tmpghl7d4_c.cpp:68)
==19729==    by 0x18C91C77: empirical0 (tmpghl7d4_c.cpp:78)
==19729==    by 0x18C91C77: __pythran_wrap_empirical0 (tmpghl7d4_c.cpp:98)
==19729==    by 0x18C91C77: operator() (tmpghl7d4_c.cpp:110)
==19729==    by 0x18C91C77: handle_python_exception<__pythran_wrapall_empirical(PyObject*, PyObject*, PyObject*)::<lambda()> > (exception_handler.hpp:17)
==19729==    by 0x18C91C77: __pythran_wrapall_empirical(_object*, _object*, _object*) (tmpghl7d4_c.cpp:107)

...
Traceback (most recent call last):
  File "./grow.py", line 410, in <module>
    run()
  File "./grow.py", line 335, in run
    original_pts, best_circles, best_Us, best_Vs, best_Ws, best_clean_cnt = test2(circle_maker=circle_maker, alpha=opt.alpha, plot=opt.plot, trials=opt.trials, epochs=opt.epochs, npoints=opt.npoints, size=opt.size, randomize='mincircle' in opt.randomize, remove_redundant=opt.remove_redundant, do_cleanup=opt.cleanup, short=opt.short,animate=opt.animate)
  File "./grow.py", line 167, in test2
    U, V, W, center, radius = grow (P, pts, 1, circle_maker, alpha=alpha, randomize=randomize, short=short, do_animate=animate)
  File "./grow.py", line 68, in grow
    V = np.append (V, U[selected])
IndexError: index 7 is out of bounds for axis 0 with size 7

The program crashes with indexerror, which is impossible.  I' believe that this uninitialized value is causing the problem.  Putting in a print statement make the problem disappear (and the program doesn't crash).

Can you provide a minimal reproducer (I need a simple call site)

I think I know what was happening here.
We have fractions = d_i / \sum(d_i),
thresholds = cumsum(fractions).
This means thesholds[-1] == 1.  But due to floating inprecision, thesholds[-1] may not be exactly -1.
This is causing searchsorted to misbehave if the value returned by ranf is close enough to 1.
For now I'm not using pythran's ranf, but passing in my own random value as an argument, which I really wanted anyway, because I need reproducible results.
But there still seems to be a problem with searchsorted reading an undefined memory value.
So far I haven't been able to find a small test case to reproduce.
BTW, same results with clang instead of gcc

Further work on this.  I was wrong.  This code is buggy as hell.  Let's look at the current version:
#pythran export empirical(float[:], float, float)
def empirical(ds, alpha, x):
    sds = np.sort(ds)
    ds_to_the_alpha = sds**alpha
    fractions = ds_to_the_alpha #/ sum (ds_to_the_alpha)
    thresholds = np.cumsum(fractions)
    thresholds /= thresholds[-1]
    i = find_first (thresholds, lambda u: x < u)
    if i is None:
        #print (ds, fractions, thresholds, x)
        print (ds, x)
    #i = np.searchsorted (thresholds, x)
    return i
#pthran export find_first(float[:], bool (float))
def find_first (seq, pred):
    for i,x in enumerate (seq):
        if pred(x):
            return i
    return None

In this code, if the print statement is uncommented, then the program completes normally.  If the print statement is commented, the program crashes.  It seems the possibility that the values of 'fractions' and/or 'thresholds' might be used causes a difference in behavior.  Are you familiar with quantum mechanics?


crnk-jpa error, application unable to start

Hey, I'm using Spring Boot 2. When I add crnk-setup-spring-boot2 application is able to start. However when I add crnk-jpa dependency i get this during application start:
Error starting Tomcat context. Exception: org.springframework.beans.factory.UnsatisfiedDependencyException. Message: Error creating bean with name 'crnkFilter' defined in class path resource [io/crnk/spring/setup/boot/core/CrnkCoreAutoConfiguration.class]: Unsatisfied dependency expressed through method 'crnkFilter' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'crnkBoot' defined in class path resource [io/crnk/spring/setup/boot/core/CrnkCoreAutoConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [io.crnk.core.boot.CrnkBoot]: Factory method 'crnkBoot' threw exception; nested exception is java.lang.IllegalStateException: java.util.NoSuchElementException: No value present
I've tried to solve this but ran out of ideas. Please help. :( Feel free to ask questions. I aware I might have not provided enough amount of needed information.

do you nevertheless have that stacktrace? maybe to improve the error message


Is this the correct behaviour?

My code got formatted like this..

Is this correct behaviour?
I thought it should be formatted like this..

P.S: I'm using standard-format plugin on sublime text

Can you re-open this issue on the standard-format repo? All formatting issues go there: https://github.com/maxogden/standard-format


Necessity of librt

I am building fiat on Fugaku which is admittedly an unusual setup - x86 login nodes compiling for aarch64 compute nodes. The librt.so found by CMake is the x86 one so I have to manually execute the final link for libfiat.so but pointing at the aarch64 instead. This isn't a big deal, and I don't think this cross-compilation issue should necessarily be accounted for by fiat. However, I also noticed that when I just deleted this line, libfiat.so still built just fine. So, do we actually need to link against librt.so anymore?

@samhatfield  there is a way to specify which library make should use.
Pass this in the config step:
-DRT_LIB=/path/to/the/aarch64/library/librt.so

We usually only use librt to use async io. Not sure fckit is using it or not. eckit does need it.

@samhatfield I don't remember exactly which platform or compiler required librt, but it was definitely required for one.
It is not an absolute requirement for some platforms, and ${RT_LIB} may as well be an empty value.
Instead of specifying RT_LIB as you do you may also try -DCMAKE_DISABLE_FIND_PACKAGE_RealTime=ON.
(see https://cmake.org/cmake/help/latest/variable/CMAKE_DISABLE_FIND_PACKAGE_PackageName.html)
Providing the correct path to RT_LIB is something that could be added to a fugaku toolchain file.

Yes, if in Fugal you are using toolchain file then that is the right place to put it

Thanks, I'm trying it with a toolchain file now. Here is my file:
# Set Fujitsu compilers
set(CMAKE_Fortran_COMPILER /opt/FJSVxtclanga/tcsds-mpi-latest/bin/mpifrtpx)
set(CMAKE_C_COMPILER /opt/FJSVxtclanga/tcsds-mpi-latest/bin/mpifccpx)
set(CMAKE_CXX_COMPILER /opt/FJSVxtclanga/tcsds-mpi-latest/bin/mpiFCCpx)

# Location of aarch64 version of Real Time library
set(RT_LIB "/FJSVxos/devkit/aarch64/rfs/usr/lib64/librt.so")

However CMake interprets librt.so as a target now, and fails:
Scanning dependencies of target fiat
make[2]: *** No rule to make target '/FJSVxos/devkit/aarch64/rfs/usr/lib64/librt.so', needed by 'lib/libfiat.so'.  Stop.
make[1]: *** [CMakeFiles/Makefile2:1054: src/fiat/CMakeFiles/fiat.dir/all] Error 2
make: *** [Makefile:166: all] Error 2

Could a CMake wizard point out what I'm missing?

Nevermind, wrong path!


Sync desktop notes with mobile (not the other way around)

It appears that we can now sync special mobile notes with the Desktop.
However, I think most users want the other way around. Most of us already have notes on the desktop (already synced with Dropbox), and we want to be able to see (an edit) these notes on mobile.

Hi @fladd , thanks for your opening issue!
I want to see other's opinion.

I agree. I was sorely disappointed when I tried out this new feature. My ideal use-case is that I can jot things down on my phone when I am on the go, and they will sync back to my computer for editing later. Being unable to create or event edit notes on mobile makes this feel like a half-baked feature.
That said, I appreciate the progress that the boostnote team is making, and I hope that it will continue so that notes can be synced both ways.

I love boostnote, and agree the @jarhill0 's opinion;

My ideal use-case is that I can jot things down on my phone when I am on the go, and they will sync back to my computer for editing later.

As a new user, I was going to assume that this was how mobile sync would work.
As others have said, creating and editing notes that sync back to the desktop is much more useful. I do think syncing needs to be bidirectional to allow this to work properly: view up to date notes on mobile, note an idea down and have it synced to desktop to edit and refine. The next day the refined notes are on your phone to review.
How do we make this happen?

as @jarhill0 said:

My ideal use-case is that I can jot things down on my phone when I am on the go, and they will sync back to my computer for editing later.

Thanks @jarhill0 @skyshk @drmason13 @dgoosens !
We plan to develop a feature of create and edit notes in mobile and sync it by Dropbox API.
PRs are welcome!

@kohei-takata While it's great news a proper sync feature is planned, could I kindly request that this not be dependent on Dropbox as the only syncing option? Adopting an open standard such as WebDAV syncing would allow people to sync from Nextcloud/ownCloud installations, and not tie them to using one particular proprietary service. I'd imagine BoostNote users, at least some of whom will have been attracted to the app due to being open source among other reasons - as I was, would find this feature particularly important.

hey @r0bbie checkout #63

@dgoosens Looks promising, thanks!

@kazup01 has boosted this issue with $200. Visit this issue on Issuehunt

Created from Boostnote Mac App:
createdAt: "2018-07-05T15:15:01.701Z" updatedAt: "2018-07-05T15:16:35.067Z" type: "SNIPPET_NOTE" folder: "b17645945be9a63b61f9" title: "test" description: "test" snippets: [   {     name: ""     mode: "text"     content: ""   } ] tags: [] isStarred: false isTrashed: true
Created from Boostnote Android App:
'type': 'MARKDOWN_NOTE' 'folder': 'b17645945be9a63b61f9' 'title': 'demoText' 'content': 'demoText' 'tags': [] 'isStarred': false 'isTrashed': false 'createdAt': '2018-07-05T15:08:49.003Z' 'updatedAt': '2018-07-05T15:09:27.915Z'
Why is the metadata of different type??
Boostnote android app cannot read notes created from that of macOS app and vice-a-versa

@kazup01 cancelled funding, $200, of this issue. Visit this issue on Issuehunt


[Improvement][Helm] using helm-docs to generate docs automatically

Purpose of the pull request

close #14720

Generate the chart's README.md file using helm-docs. When the values.yaml file is modified, the README.md file needs to be regenerated.
./mvnw validate -P helm-doc -pl :dolphinscheduler

CI validation has also been added.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

I noticed that a deadline check CI has failed. However, I believe this error can be ignored because it's checking the newly added helm charts' README.md file.

I don't know why the OWASP Dependency Check has failed.

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

Comparison is base (0c470ff) 38.17% compared to head (848cb7f) 38.16%.

:exclamation: Current head 848cb7f differs from pull request most recent head 4398434. Consider uploading reports for the commit 4398434 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15299      +/-   ##
============================================
- Coverage     38.17%   38.16%   -0.01%
+ Complexity     4667     4663       -4
============================================
  Files          1278     1278
  Lines         44482    44442      -40
  Branches       4783     4761      -22
============================================
- Hits          16979    16960      -19
+ Misses        25641    25621      -20
+ Partials       1862     1861       -1

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

I don't know why the OWASP Dependency Check has failed.

Cause we have security issue and we do not handle it, it is unrelated to you patch

I noticed that a deadline check CI has failed. However, I believe this error can be ignored because it's checking the newly added helm charts' README.md file.

Yeah, you are right, but we have to pass the docs check to merge the patch into the dev branch. So can we try to add disable for current line
[DolphinScheduler Helm Charts](https://github.com/apache/dolphinscheduler/blob/dev/deploy/kubernetes/dolphinscheduler/README.md) <!-- markdown-link-check-disable-line -->

others LGTM

I noticed that a deadline check CI has failed. However, I believe this error can be ignored because it's checking the newly added helm charts' README.md file.

Yeah, you are right, but we have to pass the docs check to merge the patch into the dev branch. So can we try to add disable for current line
[DolphinScheduler Helm Charts](https://github.com/apache/dolphinscheduler/blob/dev/deploy/kubernetes/dolphinscheduler/README.md) <!-- markdown-link-check-disable-line -->

Done. @zhongjiajie


Integration tests seem to be leaking file descriptors somewhere

as per https://github.com/WallarooLabs/wallaroo/pull/2567#pullrequestreview-166358580
something is leaking file descriptors in tests using the integration test harness with larger cluster sizes.
cc @slfritchie who noticed this earlier.

Closing in favour of https://github.com/WallarooLabs/wallaroo/issues/2568


Dependency on Microsoft.Kiota.Abstractions >= v1.9.1 causes issues

Hi! I think this is a similar issue to #256. In src/Microsoft.Kiota.Http.HttpClientLibrary.csproj, there's a dependency on Microsoft.Kiota.Abstractions Version="[1.9.1, 2.0.0)".
However, when running the Kiota CLI (v1.15.0) kiota info -l CSharp --json, the dependency given is
{
      "name": "Microsoft.Kiota.Abstractions",
      "version": "1.9.0"
},

Since our project treats warnings as errors, this breaks us at build, but otherwise would break at runtime I believe. Example output of building a .csproj that installs recommended packages based upon that kiota info output:
/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/src/GitHub.Octokit.SDK.csproj : error NU1605: Warning As Error: Detected package downgrade: Microsoft.Kiota.Abstractions from 1.9.1 to 1.9.0. Reference the package directly from the project to select a different version.  [/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/GitHub.Octokit.sln]
/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/src/GitHub.Octokit.SDK.csproj : error NU1605:  GitHub.Octokit.SDK -> Microsoft.Kiota.Http.HttpClientLibrary 1.4.3 -> Microsoft.Kiota.Abstractions (>= 1.9.1 && < 2.0.0)  [/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/GitHub.Octokit.sln]
/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/src/GitHub.Octokit.SDK.csproj : error NU1605:  GitHub.Octokit.SDK -> Microsoft.Kiota.Abstractions (>= 1.9.0) [/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/GitHub.Octokit.sln]
/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/test/Tests.csproj : error NU1605: Warning As Error: Detected package downgrade: Microsoft.Kiota.Abstractions from 1.9.1 to 1.9.0. Reference the package directly from the project to select a different version.  [/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/GitHub.Octokit.sln]
/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/test/Tests.csproj : error NU1605:  Tests -> GitHub.Octokit.SDK -> Microsoft.Kiota.Http.HttpClientLibrary 1.4.3 -> Microsoft.Kiota.Abstractions (>= 1.9.1 && < 2.0.0)  [/home/kfcampbell/github/dev/source-generator/stage/dotnet-sdk/GitHub.Octokit.sln]

Please let me know if you'd like more information!

Thanks for raising this @kfcampbell
For now, I would suggest updating abstractions to the latest version to resolve this (v1.9.6) as the Kiota info command in 1.5.0 is showing 1.9.0 due a misaligned configuration (see https://github.com/microsoft/kiota/issues/4825). We've since resolved this, and future versions should show the correct version.
Please try this out and let us know if this is helpful!

Unfortunately this isn't easily done, as our tooling treats the kiota info command as a source of truth and parses the results of that command to install dependencies. Right now as a workaround we're pinning to Kiota v1.14.0.

Thanks for the additional information. If you use the last preview version it should solve your issue.
Also, since this has "already been solved" and only needs to be released on a mainline release, we're unlikely to make any additional change at this point.

Okay! It sounds like the best course of action for us then is to stay pinned on v1.14.0 for now, skip v1.15.0, and go straight to v1.16.0 when it releases next month.

I'm going to go ahead and close this issue since there's no further action on kiota's side. Let us know if you need anything else!


Cant choose KEYS

its not possible so select the different keys wich i want send.
ip is correct and the token is set in config.
NodeREDLog write:
May  1 00:08:33 homematic-raspi daemon.err node-red: TV is avaliable

GET http://192.168.x.x/samsungtv/keys 404 (Not Found)

Found the Problem, change in samsung.html on line 79 :
            $.ajax({
                url: 'samsungtv/keys',

Remove the leading absolute path  "/"  - now relative installed instances can work too.


feat: support x-html2md-img-src header

Please ensure your pull request adheres to the following guidelines:

[x] make sure to link the related issues in this description
[x] when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues

#268

Thanks for contributing!

:tada: This PR is included in version 2.1.0 :tada:
The release is available on:

v2.1.0
v2.1.0
GitHub release

Your semantic-release bot :package::rocket:


UI-Grid CSS misbehaves

Hi guys
Please help, I have been trying to add CSS to the ui-grid but no luck. I have applied the default css
{ .grid { width: 700px; height: 450px;}} but still no luck. I have created my own css but I am having a problem with the vertical and horizontal scroll bar if I add more data to the grid.
Please see screen shots for more details.

Is this still occurring?  I have seen problems when people override the core grid css, and it's not clear to me what use case people have that needs that override.

Yes and I am using the same CSS  that's on the mockups.
On 15 Jan 2015 00:17, "Paul"<EMAIL_ADDRESS>wrote:

Is this still occurring? I have seen problems when people override the
core grid css, and it's not clear to me what use case people have that
needs that override.
--
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ng-grid/issues/2124#issuecomment-70003556.

It's not clear to me what is different, but when I put that same content into a standard plunker: http://plnkr.co/edit/tB85JVwXqr6sMcb8BaQb?p=preview it seems to work fine.  Could you please retest?

Solution suggested, no further comment for 2 weeks, assuming solution has resolved the problem.

Having the same prob.


Supermium 64-bit crashing on Windows Vista Service Pack 2 GetMyOS

It's been a while since I was using this favorite old browser until It became a crashing problem for Windows Vista Service Pack 2. anything you could help me fix this?

Were any crash dumps created in %LOCALAPPDATA%\Supermium\User Data\Crashpad\reports?

Maybe, however, we're not sure about Windows Vista Service Pack 2 for the VirtualBox. I might test again and see what's going on.

It's that random junk when opened to Notepad, and I don't know which error happens...

The one I got from GetMyOS, crashes if I don't do anything for 2-4 seconds...

In some ISO's for Windows Vista. You may have a chance to crash Supermium due to some files missing and oddly broken.

p.s. sry, rly

That's okay, but for the next update, maybe we'll try harder by doing the virtual box. Then you get the Windows Vista Service Pack 2 GetMyOS and install it there. You'll be able to see what I'm saying.

Do you ever use Vista on a real machine or just the VM? How many CPU cores and RAM were allocated for the OS? If it's 1, try it to set to 2 because in some older issues that was an issue (how repetitive, huh 😅 )

Try using Windows Vista Fully Updated 2017 ISO or Try updating Windows Vista with Platform Update, Supplement Update, SHA-2 Update.


CUDA Azure Bugfix: Only convert CTest output to JUnit if testing is enabled

Fixes #1261

Well it got rid of the error after the build... but the warning is still there under the test section and on the main page. =/

We could drop an empty/fake Text.xml file to shut it up when tests aren't being run...


Use Source Maps to remap Coverage

Remap the coverage information back to the typescript sources. Requires, not yet merged changes to istanbuljs/istanbul-lib-instrument#23, deepsweet/istanbul-instrumenter-loader#28 and SitePen/remap-istanbul#87
closes #10

Coverage decreased (-2.2%) to 88.315% when pulling e7c1f51837c03074900dfb450e5f0319789e7134 on webpack-coverage into aa92996162aece0dcbb85597986b04f90e2b5ff1 on master.


“Commenting is not available” blurb contains instructions for posting a comment

Just saw this at the bottom of the comments for responsive web design—if an article’s closed, should we strike the “Got Something to Say? / Discuss this article….” bits?

This needs to be corrected for the column and blog templates as well, I'll create a new issue.


Step 7

Step 7 was very vague. I was following both the youtube and the written instructions. Couldn't get my app to run instantly when i click "Start without debugging". Turns out I explicitly need to go to the app drawer and find App1.

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

ID: 64faeef1-f3b9-df53-33d2-d050ef1d6a98
Version Independent ID: 17bf9211-d2a6-e3bb-8cfc-357828d84e3d
Content: Build your first Xamarin.Forms app - Xamarin
Content Source: docs/xamarin-forms/get-started/first-app/index.md
Product: xamarin
GitHub Login: @conceptdev
Microsoft Alias: crdun

Thanks for your feedback @zopagaduanjr. Step 7 is very brief - we will expand it to be more helpful.
Regarding your experience, if you choose either Start/Run or Debug, the application should automatically appear in the emulator or on your device. If you find that does not work, perhaps check Developer Community for possible causes and fixes.


Should we merge scripts into lamassu-server?

Merging active scripts into lamassu-server should simplify installation and upgrade. Any reason not to do this? @naconner

The update cycle of lamassu-server is shorter than that of lamassu-scripts, so it's not an issue to me.
However, if these same scripts are included in the next lamassu-server release, then npm installation will break when it finds they already exist in /usr/local/bin/.

No longer relevant with v3.


Asynchronous Requests

Fixes # 9.
Changes proposed in this pull request:

Support for asyncio for Python 3.4 and above.
Support for twisted for all supported Python versions.
BREAKING: Invoking a consumer method now builds and executes the request,
removing the extra step of calling the execute method.

Coverage increased (+0.8%) to 96.255% when pulling 5425fe89d8a1f0260d08f05990a4bf2c2a59d3c3 on feature/async-requests into b11a86a8a2de32ab482f47b100abf32e9c859dc3 on master.