RuntimeError: t() expects a tensor with <= 2 dimensions, but self is 3D :getting this error for  better mAP

First of all thanks to all developers who build the best model.
I have trained Yolact Edge on a single object. When I try to inference using a trained model with around 50 mAP then it starts prediction(prediction very bad) but if the model mAP around 80 or high then it through a bellow error.
Traceback (most recent call last):
File "eval.py", line 1246, in
evaluate(net, dataset)
File "eval.py", line 894, in evaluate
evalvideo(net, args.video)
File "eval.py", line 777, in evalvideo
frame_buffer.put(frame['value'].get())
File "/usr/lib/python3.6/multiprocessing/pool.py", line 644, in get
raise self._value
File "/usr/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "eval.py", line 699, in prep_frame
return prep_display(preds, frame, None, None, undo_transform=False, class_color=True)
File "eval.py", line 167, in prep_display
score_threshold   = args.score_threshold)
File "/home/ubuntu_pc//yolact_edge/layers/output_utils.py", line 103, in postprocess
masks = proto_data @ masks.t()
RuntimeError: t() expects a tensor with <= 2 dimensions, but self is 3D
I have changed the dimension of tensor 3D to 2D but getting multiple errors.
If anyone has a solution then please share.

Hi, can you share the command and the config you were using?

Thanks for your support haotian-Iiu
Did changes in config.py file
added my_custom_dataset(...) below the dataset_base = Config({...})
my_custom_dataset = dataset_base.copy({
    'name': 'My Dataset',

     'train_images': '/home/ubuntu_pc/dataset/train',
     'train_info':   '/home/ubuntu_pc/dataset/train/laptop_coco.json',

     'valid_images': '/home/ubuntu_pc/dataset/val',
     'valid_info':   '/home/ubuntu_pc/dataset/val/laptop_coco.json',

    'has_gt': True,
    'class_names': ('laptop',)
})

also changed the value of  'dataset' variable and replaced coco2017_dataset by my_custom_dataset
yolact_base_config = coco_base_config.copy({
    'name': 'yolact_base',

    # Dataset stuff
    #'dataset': coco2017_dataset,
    'dataset': my_custom_dataset,
    'num_classes': len(my_custom_dataset.class_names) + 1, ## 1 + 1
.
.
.
})

The config seems fine. Were you using the TensorRT, and if you are using the latest code base?

yes, TensorRT(<IP_ADDRESS>) and the Latest code

This is weird. Can you print out the shape of the tensors in masks = proto_data @ masks.t() for both TensorRT enabled and disabled?

With TensorRT
Model mAP:78
Image Inference: python3 eval.py --trained_model=weights/yolact_edge_7_544_interrupt.pth --score_threshold=0.3 --top_k=100 --image=/home/ubuntu_pc/dataset/val/frame_5.jpg
print(masks.shape)
Output >> torch.Size([1, 1, 32])
print(masks.t().shape)
Output >> .... RuntimeError: t() expects a tensor with <= 2 dimensions, but self is 3D
Model mAP:45
Image Inference: python3 eval.py --trained_model=weights/yolact_edge_4_76_interrupt.pth --score_threshold=0.3 --top_k=100 --image=/home/ubuntu_pc/dataset/val/frame_5.jpg
print(masks.shape)
Output >> torch.Size([5, 32]) ###  show 5 mask
print(masks.t().shape)
Output >> torch.Size([32, 5])
Without TensorRT
Model mAP:78
Image Inference: python3 eval.py --disable_tensorrt --trained_model=weights/yolact_edge_7_544_interrupt.pth --score_threshold=0.3 --top_k=100 --image=/home/ubuntu_pc/dataset/val/frame_5.jpg
print(masks.shape)
Output >> torch.Size([1, 32])
print(masks.t().shape)
Output >>  torch.Size([32, 1])
Model mAP:45
Image Inference: python3 eval.py --disable_tensorrt --trained_model=weights/yolact_edge_4_76_interrupt.pth --score_threshold=0.3 --top_k=100 --image=/home/ubuntu_pc/dataset/val/frame_5.jpg
print(masks.shape)
Output >> torch.Size([4, 32]) ### show 4 mask
print(masks.t().shape)
Output >> torch.Size([32, 4])
Note: Without TensorRT model(mAP 78) working well with 38 FPS on RTX 2080

I too faced the same error and tried to managed it. Please change output_utils.py#L45 with the content below:
dets[k] = torch.index_select(dets[k], 0, keep)

It should work then.

Thanks for your comment Ravijo, but it gives me the below error >>
 File "/home/ubuntu_pc/yolact_edge/layers/output_utils.py", line 45, in postprocess
    dets[k] = torch.index_select(dets[k], 0, keep)
**RuntimeError: Expected object of scalar type Long but got scalar type Bool for argument #3 'index' in call to _th_index_select**

Ohhh. I am sorry.
I forgot to tell you that you need to convert keep to long by replacing output_utils.py#L41 with the following content:
keep = dets['score'] > score_threshold
keep = keep.long()

Ohhh. I am sorry.
I forgot to tell you that you need to convert keep to long by replacing output_utils.py#L41 with the following content:
keep = dets['score'] > score_threshold
keep = keep.long()

Sorry to say but getting below error:
File "/home/ubuntu_pc/yolact_edge/layers/output_utils.py", line 110, in postprocess
masks = proto_data @ masks.t()
RuntimeError: CUDA error: CUBLAS_STATUS_ALLOC_FAILED when calling cublasCreate(handle)

This is strange. It worked on my side and I did not face this error.

This is strange. It worked on my side and I did not face this error.

May I know your TensorRT version

Sure. Please see here for my environment details.
By the way, I am also facing many issues when using TensorRT and torch2trt module. It seems that torch2trt module has issues (not sure though).

I want to have this kind of issues all fixed ASAP, if it is possible, can you share the trained model (mAP=78 which you claimed to have issues) and the sample image (i don't need training set and annotations) to my email lhtliu at ucdavis.edu in the following command (if it is correct)?
python3 eval.py --trained_model=weights/yolact_edge_7_544_interrupt.pth --score_threshold=0.3 --top_k=100 --image=/home/ubuntu_pc/dataset/val/frame_5.jpg

Sent, please have look.

Thanks, @haotian-liu, and @ravijo for helping me.
The above issue can be solved using cammand:
python eval.py --trained_model=./weights/yolact_edge_road_7_544.pth --image=./data/image_files/frame_5.jpg
but when I pass arguments  --score_threshold=0.3 --top_k=100 then it through the same above error.
With the cammand
python eval.py --trained_model=./weights/yolact_edge_road_7_544.pth --video=./data/video_files/test.mp4
give 70 to 72 FPS on RTX 2080 GPU(input video resolution 1280x720 px),
I am planning to deploy it on Xavier AGX, I hope it will work and will give around 30 FPS.

@kashzade
Glad to hear from you.
The default value of the parameter score_threshold is 0 and it can be seen here. Similarly, the default value of the parameter top_k can be seen here and it is equal to 5. Probably, top_k = 100 is a too high value.

@ravijo, As I tested, the issue does not with the value of the top_k parameter, I have tested random values between 1 to 100. The error cause is the score_threshold parameter. It works with only 0 value. tested with 0.01 to 1 but does not work.

@kashzade
You are right. I just realized it too. Right now I have also set score_threshold to 0 but it is not a good choice though. I have shared my observations here.

@kashzade I set the score_threshold to 0.3 and it still works. Can you provide the command you used that throws the error (is that the same image you sent to me?) and maybe also the error message itself.

@kashzade Also, I implemented a TensorRT safe mode (experimental) to securely handle those TensorRT related issues. Please try out haotian-dev branch, run the evaluation with --use_tensorrt_safe_mode, and see if it helps.

@haotian-liu and @kashzade
Thank you so much. I have not tried use_tensorrt_safe_mode in the haotian-dev yet.
At present, in my case, when cc_fast_nms is used and score_threshold is non zero, I get the same error, i.e., CUDA error: CUBLAS_STATUS_ALLOC_FAILED when calling cublasCreate(handle). The complete description is provided here.
Please let me know if you find out any workaround.
Meanwhile, I am going to try use_tensorrt_safe_mode .

use_tensorrt_safe_mode

Using this argument, the model is working with 104 FPS for input 1280x720 px video

@kashzade Thank you for reporting back. Btw what device you are working on? 104 FPS seems higher than what we could expect on Jetson devices


Ability to select which series is displayed in a Single Stat or Gauge cell

This could be a simple dropdown in the top of the options:

Selecting the seriesKey from a dropdown needs some design work to make the very long series key readable.
In order to complete this task currently, we need to do the following:

[X] Update the GaugeViewProperties to store the seriesSelections:

type GaugeViewProperties struct {
   Type              string            `json:"type"`
   Queries           []DashboardQuery  `json:"queries"`
   Prefix            string            `json:"prefix"`
   Suffix            string            `json:"suffix"`
   ViewColors        []ViewColor       `json:"colors"`
   DecimalPlaces     DecimalPlaces     `json:"decimalPlaces"`
   Note              string            `json:"note"`
   ShowNoteWhenEmpty bool              `json:"showNoteWhenEmpty"`
   SeriesSelections  []SeriesSelection `json:"seriesSelections"`
}

// TableSelection represents the series selected for viewing
type SeriesSelection struct {
   Selected  bool   `json:"selected"`
   SeriesKey string `json:"seriesKey"`
}

[X] Update the GaugeChart's lastValue method to select either the last value from the spreadTables or find the lastTime and value for a selected seriesKey
[ ] Update the spreadTables' key creation to ensure seriesDescription['key'] is unique across query tabs.  This might be tricky since the graph is currently only provided the tables and no queryTab info.
[X] Sync updated SeriesSelections on componentUpdate or componentMount to redux state to ensure the GaugeOptions displays the correct series in the dropdown.

However, storing the seriesKey as a string might impact the ease of the design of the dropdown in GaugeOptions, i.e. we may want to store the group key, tableName, columnName, and query tab index separately for rendering of the dropdown. Also, after some discussion with  @chnn we may want to determine the lifetime of such changes if we are going to be changing up query visualization rendering soon and what needs to be stored is going to change quite a bit.

Alex and I did pair on the design for this and found that something like the hover legend on line graphs might suffice.

Closing in favor of more up to date #14121


Unclear wording of ACPERM rule

The rules for ACPERM includes this for RV32:

Clear X-permission if X-permission and R-permission are set, but C-permission and W-permission are not set.

This is a bit ambiguously worded. It could either mean:

if (X && R && !(C || W) then clear X
if (X && R && !(C && W) then clear X

If it's option 2 (which I think is the most logical reading of it) then there's several encodings which are impossible - encodings 2-3 or 6-7 for quadrant 1:

Which is quite a large area of redundant encoding.
However if it's option 1 it's possible to create an a set of permissions that has no encoding. If we have R+C+X+LM permissions (quadrant 1, encoding 2-3) in a capability that we load from memory without LM permission in the authorising cap, then we're supposed to clear the LM permission, which would leave us with R+C+X permissions. If we interpret the above rule as option 1 then we shouldn't clear X, but there is no encoding that exists for R+C+X permissions.

I think the rule needs to look like this:
(X && R && ((LM ^ C) || !(C || W))) then clear X
Not sure how you'd sensibly express that in a sentence though.

The intent of Clear X-permission if X-permission and R-permission are set, but C-permission and W-permission are not set. was that clearing C on from Execute + Cap RO would go to RO in q0 and clearing X on Execute + Data RW goes to Data RW in q0.
@LawrenceEsswood does that match your understanding?

Quadrant 1:
R,W,C,LM,X,ASR -> R,W,C,X,ASR doesn't exist so will need to map to R,W,X or R,C

When clearing LM here, there are no other encodings with ASR, so we have to look at R,W,C,X (almost exists, but that encoding option also has LM), so we have to either clear C to get R,W,X or R,C. We don't have R,W,C (the potential encoding here has an implicit LM due to being in the "elevate" quadrant that implies LM for W).

R,C,LM,X -> R,C,X doesn't exist so will so need to map to R,C
R,W,C,LM,X -> R,W,C,X doesn't exist so will need to map to R,W,X or R,C
Quadrant 3:
R,C,LM -> R,C - this one is ok! R,W,C,LM -> R,W,C doesn't exist so will need to map to R,C
so we have 4 cases where only a single mapping possible (R,C), and two where there is a choice between R,W,X and R,C.

Also agree with all of these.


Roll Engine from 2abe69c60818 to aec855d3963c (21 revisions)

https://github.com/flutter/engine/compare/2abe69c60818...aec855d3963c
2020-09-18<EMAIL_ADDRESS>Roll Skia from c21dc07a78b9 to 31634288fdf3 (5 revisions) (flutter/engine#21260)
2020-09-17<EMAIL_ADDRESS>Roll Dart SDK from fcaedc6d9587 to 2cec6af2652f (1 revision) (flutter/engine#21257)
2020-09-17<EMAIL_ADDRESS>Disable ShellTest.SkipAndSubmitFrame for flakes (flutter/engine#21254)
2020-09-17<EMAIL_ADDRESS>Roll Skia from 45f41b376260 to c21dc07a78b9 (4 revisions) (flutter/engine#21250)
2020-09-17<EMAIL_ADDRESS>Add --enable-isolate-groups to allowed flags in switches.cc (flutter/engine#21149)
2020-09-17<EMAIL_ADDRESS>[web] Remove commented import (flutter/engine#21247)
2020-09-17<EMAIL_ADDRESS>Roll Skia from 12d06a38427d to 45f41b376260 (6 revisions) (flutter/engine#21249)
2020-09-17<EMAIL_ADDRESS>Add support of cross-building the engine for ARM64 Linux Host (flutter/engine#20254)
2020-09-17<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 91U3isvKn... to Qi0ptKLxN... (flutter/engine#21246)
2020-09-17<EMAIL_ADDRESS>[web] run safari desktop tests on luci (flutter/engine#21228)
2020-09-17<EMAIL_ADDRESS>avoid hard coding OS (flutter/engine#21248)
2020-09-17<EMAIL_ADDRESS>Roll Dart SDK from 99970d646dc9 to fcaedc6d9587 (3 revisions) (flutter/engine#21245)
2020-09-17<EMAIL_ADDRESS>[fuchsia] set maxframesinflight to be configurable (flutter/engine#21152)
2020-09-17<EMAIL_ADDRESS>Roll Skia from 51a7f9559ad4 to 12d06a38427d (3 revisions) (flutter/engine#21244)
2020-09-17<EMAIL_ADDRESS>Roll Skia from f9fdf291c567 to 51a7f9559ad4 (5 revisions) (flutter/engine#21242)
2020-09-17<EMAIL_ADDRESS>Roll Dart SDK from eb24e3324908 to 99970d646dc9 (1 revision) (flutter/engine#21239)
2020-09-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 13BzEPO47... to 2nHpC_AEO... (flutter/engine#21238)
2020-09-17<EMAIL_ADDRESS>Roll Skia from bf66ffbbd4ce to f9fdf291c567 (1 revision) (flutter/engine#21235)
2020-09-17<EMAIL_ADDRESS>Roll Dart SDK from e41a6008746d to eb24e3324908 (1 revision) (flutter/engine#21233)
2020-09-17<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from G5UItrFeP... to 91U3isvKn... (flutter/engine#21232)
2020-09-17<EMAIL_ADDRESS>Roll Skia from 2564767d24e5 to bf66ffbbd4ce (27 revisions) (flutter/engine#21230)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from G5UItrFeP96D to Qi0ptKLxNLDk
fuchsia/sdk/core/mac-amd64 from 13BzEPO47Hu7 to 2nHpC_AEOz0o
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.

Commit queue failed; closing this roll.


feat: Check that all rules have the correct CRS tag and version

Add two new features for rules-check.py linter:

check that all rules have a tag with OWASP_CRS
check that all rules have ver action with correct version string

Version can be passed manually with -v/--version. If no argument given, the script tries to get that from output of git describe --tags.
How the version is determined:

eg git describe --tags gives string v4.5.0-6-g872a90ab
-6- means there are 6 commits after 4.6.0, so this is a dev version

in this case we increment MINOR version: 5 -> 6
and version gets -dev

the result will be: 4.6.0-dev (v will be removed from the start position)
if the version string is something like this: v4.5.0-0-abcd01234 then the version will be 4.5.0

In all cases the OWASP_CRS/ prefix will be appended.

This PR closes #3573.

@theseion could you help me what's the issue here. I think there is no git command in workflow environment, or I don't know why splitted string contains only 1 item.

The checkout action may not fetch any tags. To debug, add the outputs of git tag -l and git fetch --tags to your script or to the pipeline.

The checkout action may not fetch any tags. To debug, add the outputs of git tag -l and git fetch --tags to your script or to the pipeline.

Hmmm... Could you take a look at this?
fatal: No tags can describe '3bef193c7d191f8ff9aac25d5548683dd927209e'.
Try --always, or create some tags.
Error: Process completed with exit code 128.

Fixed.

Fixed.

Thanks! Seems like all test has passed.


Reduced MenuItem height after upgrading to 1.12

MenuItem component

1.11
1.12

The tab height is incorrect in this example:


Clippy items after test mod

Addresses a Clippy warning where items are placed after test module.

Although it seems that this lint was added in 1.71, I can only seem to trigger it on nightly.
There's no harm in merging this, I just want to understand why this lint is so hard to trigger...

I just fix the squiglies sir. ;)
I saw it popup and I do agree with the suggestion. For me at least it is unexpected to find items after a test module.
It probably only pops up if you lint for 'Style' group lints.
items_after_test_module lint


Adding to a group now fails with the latest version

Using version  "version": "2023.8.8",
Now adding a whatsapp number to a group the same way as before either returns a "200" but does nothing, or returns a 500:
{   "statusCode": 500,   "timestamp": "2023-08-09T19:31:04.914Z",   "exception": {     "stack": "Error: Evaluation failed: i\n    at ExecutionContext._evaluateInternal (/app/node_modules/whatsapp-web.js/node_modules/puppeteer/lib/cjs/puppeteer/common/ExecutionContext.js:221:19)\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n    at async ExecutionContext.evaluate (/app/node_modules/whatsapp-web.js/node_modules/puppeteer/lib/cjs/puppeteer/common/ExecutionContext.js:110:16)\n    at async GroupChat.addParticipants (/app/node_modules/whatsapp-web.js/src/structures/GroupChat.js:62:16)\n    at async /app/node_modules/@nestjs/core/router/router-execution-context.js:46:28\n    at async /app/node_modules/@nestjs/core/router/router-proxy.js:9:17",     "message": "Evaluation failed: i",     "name": "Error"   },   "request": {     "path": "/api/default/groups/120363026XXXXXXXXXXX%40g.us/participants/add",     "method": "POST",     "body": {       "participants": [         {           "id"<EMAIL_ADDRESS>       }       ]     },     "query": {}   },
Of course I'm checking that the contact is correct beforehand by checking that it exists. Any ideas what this could be? Maybe I'm missing something that I wasn't missing before, but it used to work perfectly with the version from two months ago

Hi! Waiting for engine to support it, some changes have been made in WhatsApp itself. https://github.com/pedroslopez/whatsapp-web.js/pull/2344
As a quick fix - we'd suggest to send invite code (link) to the contact https://waha.devlike.pro/docs/how-to/groups/#invite-code so they can accept group invitation


Vending machines break into old metal

Title

So do arcade machines

Should be closed by #4233, believe the arcade machines are directly Upstream objects.


Extension keeps re-downloading the latest version - every launch

good first issue

is this the same issue? always redownloads latest version but then gets stuck on 100%

is this the same issue? always redownloads latest version but then gets stuck on 100%

The same thing happened on my machine. I was able to work around this issue by Installing clarity-lsp from source.
from the readme:
git clone https://github.com/lgalabru/clarity-lsp.git
cd clarity-lsp
cargo xtask install

Should be addressed with latest.


feat: Benson/Boran: implement skeleton of osd-network-verifier

As a note:
cloudclient.GetClientFor(configv1.AWSPlatformType)
cloudclient.ByoVPCValidator()

were commented out as they weren't pushed to github.com/openshift/osd-network-verifier/pkg/cloudclient

/lgtm

/lgtm


New version: PlutoSliderServer v0.2.7

Registering package: PlutoSliderServer
Repository: https://github.com/JuliaPluto/PlutoSliderServer.jl
Created by: @fonsp
Version: v0.2.7
Commit: 60f5ee0fb6fcfa23a9a7f4cd03c9bcc45338413f
Reviewed by: @fonsp
Reference: https://github.com/JuliaPluto/PlutoSliderServer.jl/commit/60f5ee0fb6fcfa23a9a7f4cd03c9bcc45338413f#commitcomment-58493590
Description: Web server to run just the @bind parts of a Pluto.jl notebook

you can close this PR, thanks!


Modified Milestone Fields

Using good start, terrific start etc. does sound creative (and did come out of my brain when creativity was at its peak 😂), but might not be quite as explanatory and also complicating a little? What do you think? I've modified it to take note of good_first_issue,medium_issue_1... check it out and merge it if you feel right.
I've explained how it blends well with the phrases in a docstring.
I also think that maybe keeping an entire model for these three boolean values is too many models? I'm making a separate PR to combine these models to one. If that is more appropriate according to you too, then close this and merge that one 😂

You're welcome 👀👀


[Runtime Bug]: Render artifacts linked to high xformOp:size values

Describe the bug
When using a small mesh, and the user wants to increase the size of that mesh a lot, using the xformOp:size value, for example a 1.0x1.0x1.0 plane mesh to become 500 x 1.0 x 500, the center of this mesh will introduce noisy black outlines formed as a cube. This artifact is becoming more and more visible as the size of the mesh is increased. At 16 or 32m the artifact is nearly invisible. But as the size value is increased, the artifact becomes more visible. Here's some screenshots:

And here's three screenshots showing how the artifact subsides with distance from the center of the mesh:

The artifact is especially noticeable on semi-transparent materials.
The solution to this problem is to make your mesh as big as you need it prior to inserting/importing it to your scene in Composer, so that you don't need to rely on the xformOp:size values to make it big.
How do you reproduce the bug?
Inside Composer:

Create a simple box or plane mesh
Increase its size to 1000 x 1.0 x 1000
Add a plain, solid color to the mesh
Replace a vanilla game's mesh to bring it ingame
Go to the center of the mesh ingame and look down

What is the expected behavior?
The runtime should...
Version
0.5.1
Logs
No response
Crash dumps
No response
Media
No response

Thanks for reaching out. A few questions:

Would you please provide us with a mod package/set of files that you used to reproduce this test case?
In what game did you encounter this issue?
Does this happen in both the toolkit and the game, or just the game?

Thanks for reaching out. A few questions:

Would you please provide us with a mod package/set of files that you used to reproduce this test case?
In what game did you encounter this issue?
Does this happen in both the toolkit and the game, or just the game?

Hi LL and thanks for taking care of this!

Sure, I'll setup an environment where this bug happens and share all relevant files shortly! But you're going to have to obtain Max Payne 1, but before that, you could always try this in another game first, because I think this bug is related to the Remix renderer and not being game specific.

The game in question is Max Payne 1. But I have reason to believe this happens in more games, or even in any game if the circumstances are right.

Not sure about the Remix app, as I'm using Composer, but it's not happening there, only ingame.

@NV-LL Hi Lindsay! I've prepared a test environment for use on the first level (house prologue) of Max Payne, kindly try it out and see if it's working as supposed. I'll attach the relevant files in this post. You'll need to remove the ".zip" part of the filenames in order to extract.
You'll need to disable Volumetric Lighting under Froxel Radience Cache/Volumetrics But I also included this in the READ ME.
Artifact_debug_test_files.7z.001.zip
Artifact_debug_test_files.7z.002.zip

Hi @PappaSlask - we apologize for such a delay! Thank you for providing those files for repro, and we've opened REMIX-3573 for tracking.

Hi @PappaSlask - we apologize for such a delay! Thank you for providing those files for repro, and we've opened REMIX-3573 for tracking.

Hi Lindsay and thanks for the update! I want to add a few comments regarding those files that I shared: Some people have had troubles extracting their content, so let me be clear: You NEED to remove the ".zip" part of the filenames to extract, so that both files look like this:
DXVK_Bridge_latest_working_versions.7z.001
DXVK_Bridge_latest_working_versions.7z.002
You may or may not need 7zip to extract them, unsure if they work with winrar or Windows built in extractor!
The reason for this is that the file attachment/uploader for Github doesn't support 7z or split archive files, only plain .zip files are accepted, I worked around that limitation by adding a fake .zip suffix to the files ;)


Feature Request: Mark dependencies for individual scss files

Feature Requested
I only want to use checkboxes for my own purposes, but it took me a while to whittle down what is needed for them to work. This repo used to have snippets, which I used up until the latest Obsidian update. Maintaining snippets might be a hassle, but marking dependencies for scss files seems like a good middle ground to me. That way people who just want a small feature can still pick and choose.
I'd imagine keeping track of dependencies for its own sake has other potential benefits for continued development.
In my use-case, just adding @use "../variables/colors"; and @use "../components/checkbox"; to the top of the file makes it so I can execute sass src/scss/features/custom-checkboxes.scss snippet.css and get only what I want.
Relevant Screenshot(s)
No response
Relevance

[X] The feature would be useful to more users than just me.

Style Settings

[X] I have checked that the feature is not already available via the Style Settings plugin.

Hi! I'm not sure I fully understand what you mean by "marking dependencies". Since the theme's been broken down into small modules, it's relatively easy to find certain features and convert the code into css, like you explained. Granted not everyone has the know-how of how to do that.
In case this answers your request, I'm planning on adding proper documentation to the theme (currently exploring Github Pages as an option), where I could add a link to the specific scss modules where each feature is located. I could have a page to explain how to convert said modules into css for use as a snippet (I've written an article over at Obsidian Hub about scss, so it'd only be a matter of linking to it as a resource). Do you think that's suffice?

By marking dependencies, I simply mean having @use statements in each file, so that they can be assembled from places other than the index. I came for checkboxes, and needed trial and error to see what else I need for them to work. With some the @use statements already present and some instruction about SASS, no one would need to guess.
More documentation would make this request less relevant for sure. And I would add to that, that good code documents itself. Having @use statements at the top of each file would be a developer and tooling-friendly way of documenting which snippet requires which other one. Having to read the docs is not a lot to ask from someone, but it's one more step removed from true modularity, and not good enough for good tooling. Ideally, I'd like to be able to execute sass on any module and get a result that works on its own.
I think this would make documenting easier for you too. Instead of explaining each module's dependencies at length in text, you could just tell people to execute sass (or your equivalent command) on any module and all the dependencies would be handled by SASS. Having the dependency explanation in a different place than the code also means they have to be maintained together, which takes more effort and is more error-prone.
On an somewhat unrelated note: does custom-checkboxes.scss even require checkbox.scss? It seemed to work fine without it at a glance. Does it require anything else besides colors.scss?
And one last thing: thanks for the article, and the theme! The article was a good starting point for organizing my existing CSS, and I loved the theme's features for a long while.

custom-checkboxes.scss should only need colors.scss, yeah. It specifically targets "special" checkbox syntax, so it doesn't affect regular checkbox code, which is formatted in checkbox.scss.

Alright, I just went through every scss module, and they should all now work as individual snippets. If you happen to find an error on my part, be sure to report it as a bug.  In any case, they'll still be present in the file, so it'll be easy to know what that file needs in order to correctly build a full snippet.

Isn't it just a problem with relative paths?
I put @use '../variables/colors'; at the beginning of custom-checkboxes uncommented, and npm run build ran fine and produced an identical output.

It is, I just didn't have time to fix it today. 😅I'll troubleshoot it and update every file tomorrow

Hey there! I think you missed updating this.

Hi! I took a few days off to recharge and take care of some things irl. I'll take a look at it soon.

I found a typo in the file, I guess I'll just post it here: https://github.com/jdanielmourao/obsidian-sanctum/blob/34c7803c890645c6adba53f93360f64c14cca9d0/src/scss/features/custom-checkboxes.scss#L424

Hi!
This is still not resolved. Please at least reopen the issue, if you don't have the time for it.


[Spike] Investigate Manage Identity AKS interoperability with CSI

As a:
Operator
I want:
Better Understanding of Kubernetes-Secrets-Store-CSI-Driver integration with Key Vault on AKS
So that:
I can better determine the best secrets solution in an MSI AKS implementation for handling service secrets.
Describe the solution you'd like:
Documentation around benefits of using CSI and comparison with FlexVolume.
Acceptance Criteria:

[ ] CSI vs FlexVolume
[ ] CSI + MSI & AKS
[ ] CSI guidance with Key Vault and accessing secrets through Env Vars
[ ] Terraform implementation of CSI

Describe alternatives you've considered:

Continue with FlexVolume

Additional context:
Related to: https://github.com/microsoft/bedrock/issues/1197
Does this require updates to documentation?:
Yes

@paulbouwer just did a deep dive on CSI-driver with MI and could provide so insight

Have a look at the following:
https://github.com/paulbouwer/experiments/blob/master/aks/install-aadpodidentity-and-secretsstoredriver.md
This guidance assumes AKS with Managed Identity (Kubernetes Cloud Provider Identity). It leverages AAD Pod Identity to assign another Managed Identity (Key Vault Identity) to the Secrets Store CSI Driver. This all works.
If you are looking to spin up Secrets Store CSI Driver with a Managed Identity not controlled by AAD Pod Identity, you will have to wait for Azure/secrets-store-csi-driver-provider-azure PR #46 - Add support for MSI to land.


Translation into Dutch

Hi folks,
I'm about to start translating the documentation into Dutch. Any help is appreciated. Feel free to hit me up if you're willing to help.

Thank you. Please note though, that in order to reduce maintenance debt, we don't officially host such translations anymore. Instead, translations should be forked, hosted (and with it, maintained) externally. Thanks!


micronaut: 1.3.5 -> 1.3.6

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/micronaut-projects/micronaut-core/releases.
meta.description for micronaut is: "Modern, JVM-based, full-stack framework for building microservice applications"
meta.homepage for micronaut is: "https://micronaut.io/"
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.3.6 with grep in /nix/store/iakhmrnrl3jlcy4pp3dfcwr0j9ryh244-micronaut-1.3.6

directory tree listing: https://gist.github.com/74d73dc54d8eee2f69bfaaaf954eddea

du listing: https://gist.github.com/5ba4bc705d94c43ddf55f718e600ced4

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
micronaut

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/iakhmrnrl3jlcy4pp3dfcwr0j9ryh244-micronaut-1.3.6 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A micronaut https://github.com/r-ryantm/nixpkgs/archive/e51b28bfd0e247f6f9f150229dc722f5311665db.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/iakhmrnrl3jlcy4pp3dfcwr0j9ryh244-micronaut-1.3.6
ls -la /nix/store/iakhmrnrl3jlcy4pp3dfcwr0j9ryh244-micronaut-1.3.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- micronaut

Maintainer pings
cc @moaxcp for testing.

The bot has built all dependent packages and the changelog looks good (https://docs.micronaut.io/1.3.6/guide/index.html#whatsNew).


A way to provide info from platform request to application

Describe the problem
There are things on the original platform request that are needed in SvelteKit and can't be accessed today. E.g. some Node middlewares stick things on the request and the user may need to access them, Cloudflare KV and Durable Objects need access to env (https://github.com/sveltejs/kit/issues/3426), etc.
Describe the proposed solution
A way for adapters to copy info from platform request into locals, a new field like meta, some new argument to render, or something else
Alternatives considered
Had considered exposing the whole request object (https://github.com/sveltejs/kit/pull/2359)
Importance
i cannot use SvelteKit without it
Additional Information
No response

I have some questions about this:

Will there be a namespace for the info in locals (like locals.info.foo or will it be at the same level than the others (local.foo)?)
Can any kind of object be passed down?
If it is possible, can the platform request be exposed via locals?
For adapter-node could it be possible to expose the req object via locals (allowing the nextjs middleware style)?

I would like to rise the issue, that this might make it very hard for 3rd-party-libs to support sveltekit (at least with the full feature-set they have in other frameworks). I'm thinking about libs like https://github.com/PabloSzx/graphql-ez or https://github.com/trpc/trpc.

this might make it very hard for 3rd-party-libs to support sveltekit (at least with the full feature-set they have in other frameworks)

Can you explain why it would make it difficult? I'm not familiar with these libraries. graphql-ez claims to already support SvelteKit

@benmccann ATM, sveltekit chooses to only provide a minimal common denominator for request/response. This is an approach which imho arbitrarily takes away options and freedom from users and implementors. Additionally it is a quite different approach to basically everyone else, often resulting in quite some effort to support by 3rd-parties (see f.e. https://github.com/trpc/trpc/releases/tag/v9.3.0).
If we would go down the way with a raw-request-prop, it would be the responsibility of the implementor to check and guarantee what features/props/methods of the request are there and supported. It would make support much easier for 3rd-parties, as they f.e. simply could type-check the object in .rawRequest to determine the platform and, ideally, reuse existing code for this platform.
The road with manually copying props into req.locals has no guarantees about any interface which could be checked against  and would result in the need to manual configure things on a per case basis by the enduser of many libs. The responsibility is split between implementor and enduser with no common anchor and no clear separation of concerns.
Additionally I would fear that it could result in a higher support-burden for 3rd-party-libs, as I would suspect bugs resulting of problems by manually copying props as a prerequisite to use those 3rd-party-libs (to their full extend).

I completely agree with @benbender, the truth is people know which adapter they are going to use. There is minimal real utility to being able to switch between serverless and node without having to rewrite some stuff.  That's not something people need to do often enough to justify constraining to developer this much.

One thing that I haven't seen a solution propose yet is how this would work in local development? Perhaps the adapter can write a sort of polyfill to allow the app to run locally while assuming the adapter specific environment exists

I would like to rise the issue, that this might make it very hard for 3rd-party-libs to support sveltekit (at least with the full feature-set they have in other frameworks). I'm thinking about libs like https://github.com/PabloSzx/graphql-ez or https://github.com/trpc/trpc.

@benbender - if you're interested in tRPC, have a look at trpc-sveltekit. Not a full features-set yet, but it's a start.


Add EKS view policy

PR Checklist:

[x] Describe and explain your intentions for this change
[x] Setup pre-commit and run the validators (info in README.md)

Add the ability to view k8 deployments (port forward) deployments.
cc @BryanFauble

@thomasyu888 Can you close this pull request in favor of:
https://github.com/Sage-Bionetworks-IT/organizations-infra/pull/1191


LoadSceneWhenUserAuthenticated.cs not working properly

[REQUIRED] Please fill in the following fields:
Unity editor version: _____2020.1.0f1
Firebase Unity SDK version: _____Latest
Source you installed the SDK: _____ .unitypackage
Problematic Firebase Component: _____ Database
Other Firebase Components in use: _____ Auth, Database and Analytics
Additional SDKs you are using: _____ Facebook
Platform you are using the Unity editor on: _____ Mac
Platform you are targeting: _____ iOS, Android
Scripting Runtime: _____ IL2CPP
[REQUIRED] Please describe the issue here:
So, I was in doubt if this was indeed a bug or just a question. As I don't know where else to post it, I decided to do it here.
I am stuck with a functionality of this SDK regarding the Scenes and it's integration. Here's how they work:
1st: Loading Scene
Here, I just added the FirebaseInit code EXACTLY as suggested by Patrick, which it's only function is to call the next scene (Login/Register) once everything loads correctly.
2nd: Login/Register Scene
Here I do all the Login AND ALSO the register logic. I set up a button that alternates between the two (Activating the respective parent gameObject within the Canvas). Once the user log's in, the 3rd scene comes into play.
3rd: App's Main Screen Scene
Main Screen of the app, where the user can LOGOUT and return to the Login Scene.
Problem
I added the 'LoadSceneWhenUserAuthenticated.cs' in the 2nd Scene, and it works (kind of).
It actually does what it is supposed to. If I log in, quit the game without loging out, and open it again, it does come back directly to the 3rd scene. BUT some things are happening and they aren't supposed to.
First
When I Sign Up a user, I call the method 'CreateUserWithEmailAndPasswordAsync()'. Once it completes, it should activate the login screen and stay there, waiting for the user to fill in the password, but the 'FirebaseAuth.DefaultInstance.StateChanged' comes into play and forces the 3rd screen to be loaded, skipping several other steps that should be taken (email registration for example).
Second
As I mentioned in the end of number 1 above, if I try to log in to an account that does not have it's email verified, it works! (due to the 'LoadSceneWhenUserAuthenticated.cs' which is added in the scene). Code:
var LoginTask = auth.SignInWithEmailAndPasswordAsync(_email, _password);
      LoginTask.ContinueWithOnMainThread(task =>
       {
         if (task.IsCanceled || task.IsFaulted)
         {
           Firebase.FirebaseException e =
             task.Exception.Flatten().InnerExceptions[0] as Firebase.FirebaseException;
           GetErrorMessage((AuthError)e.ErrorCode, warningLoginText);
           return;
         }
         if (task.IsCompleted)
         {
           User = LoginTask.Result;
            if (User.IsEmailVerified == true)
           {
             UIControllerLogin.instance.MainScreenScene();
           }
           else
           {
             warningLoginText.text = Lean.Localization.LeanLocalization.GetTranslationText($"Login/VerifyEmail");
           }

I know that it's possible to fix this issue by adding an extra scene just before the login scene (as Patrick does in the youtube video) but it doesn't make any sense in my app. It would actually only harm the UX of it.

Hi @NickChittle
Without diving too deep into your code, here are several things which might help you to debug your situation.

When you are switching scene, Unity will destroy all objects which is not marked as DontDestroyOnLoad.  Your FirebaseAuth instance and login status will not be reset, since they are stored internally.  But if you try to store FirebaseAuth as a static reference in a MonoBehavior, that reference will very likely be cleared, if the game object it attaches to is not marked as DontDestroyOnLoad.  What you need to do is just to get the reference again using Firebase.FirebaseAuth.DefaultInstance.
Firebase Auth has a persistence feature.  That is, if the user logged in and did not logout before the app is closed, the login state persists next time when the app is launched. That is, you do not need to call SignInWithEmailAndPasswordAsync() again. If the user logged in before, FirebaseAuth.DefaultInstance.StateChanged will be called once after FirebaseAuth instance is created, since the login state changed after it loads the persistence data. That is one way the SDK tell you that the user has successfully login before.  Another way is to check FirebaseAuth.DefaultInstance.CurrentUser and see if it is null. It is designed to tell you login state whenever you call it.

Here is my recommendation for your 3 scenes

1st: Loading Scene - Firebase Init (CheckAndFixDependenciesAsync and all that jazz).  Once the init is done, check FirebaseAuth.DefaultInstance.CurrentUser.  If null, go to 2nd scene.  If not null, go to 3rd scene.
2nd: Login/Register Scene - Your login and register flow.
3rd: App's Main Screen Scene - Your main screen flow.

Hope this helps.
Shawn

Hi @NickChittle
Without diving too deep into your code, here are several things which might help you to debug your situation.

When you are switching scene, Unity will destroy all objects which is not marked as DontDestroyOnLoad.  Your FirebaseAuth instance and login status will not be reset, since they are stored internally.  But if you try to store FirebaseAuth as a static reference in a MonoBehavior, that reference will very likely be cleared, if the game object it attaches to is not marked as DontDestroyOnLoad.  What you need to do is just to get the reference again using Firebase.FirebaseAuth.DefaultInstance.
Firebase Auth has a persistence feature.  That is, if the user logged in and did not logout before the app is closed, the login state persists next time when the app is launched. That is, you do not need to call SignInWithEmailAndPasswordAsync() again. If the user logged in before, FirebaseAuth.DefaultInstance.StateChanged will be called once after FirebaseAuth instance is created, since the login state changed after it loads the persistence data. That is one way the SDK tell you that the user has successfully login before.  Another way is to check FirebaseAuth.DefaultInstance.CurrentUser and see if it is null. It is designed to tell you login state whenever you call it.

Here is my recommendation for your 3 scenes

1st: Loading Scene - Firebase Init (CheckAndFixDependenciesAsync and all that jazz).  Once the init is done, check FirebaseAuth.DefaultInstance.CurrentUser.  If null, go to 2nd scene.  If not null, go to 3rd scene.
2nd: Login/Register Scene - Your login and register flow.
3rd: App's Main Screen Scene - Your main screen flow.

Hope this helps.
Shawn


Example code target group data structure invalid

Description
Please provide a clear and concise description of the issue you are encountering, and a reproduction of your configuration (see the examples/* directory for references that you can copy+paste and tailor to match your configs if you are unable to copy your exact configuration). The reproduction MUST be executable by running terraform init && terraform apply without any further changes.
If your request is for a new feature, please use the Feature request template.

[x ] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]: 9.7.0

Terraform version:
v1.6.5

Provider version(s):
provider registry.terraform.io/hashicorp/aws v5.39.1

Reproduction Code [Required]

module "alb" {
  source  = "terraform-aws-modules/alb/aws"
  version = "9.7.0"

  name    = "my-alb"
  vpc_id  = "vpc-abcde012"
  subnets = ["subnet-abcde012", "subnet-bcde012a"]

  # Security Group
  security_group_ingress_rules = {
    all_http = {
      from_port   = 80
      to_port     = 80
      ip_protocol = "tcp"
      description = "HTTP web traffic"
      cidr_ipv4   = "<IP_ADDRESS>/0"
    }
    all_https = {
      from_port   = 443
      to_port     = 443
      ip_protocol = "tcp"
      description = "HTTPS web traffic"
      cidr_ipv4   = "<IP_ADDRESS>/0"
    }
  }
  security_group_egress_rules = {
    all = {
      ip_protocol = "-1"
      cidr_ipv4   = "<IP_ADDRESS>/16"
    }
  }

  access_logs = {
    bucket = "my-alb-logs"
  }

  listeners = {
    ex-http-https-redirect = {
      port     = 80
      protocol = "HTTP"
      redirect = {
        port        = "443"
        protocol    = "HTTPS"
        status_code = "HTTP_301"
      }
    }
    ex-https = {
      port            = 443
      protocol        = "HTTPS"
      certificate_arn = "arn:aws:iam::123456789012:server-certificate/test_cert-123456789012"

      forward = {
        target_group_key = "ex-instance"
      }
    }
  }

  target_groups = {
    ex-instance = {
      name_prefix = "h1"
      protocol    = "HTTP"
      port        = 80
      target_type = "instance"
    }
  }

  tags = {
    Environment = "Development"
    Project     = "Example"
  }
}

Steps to reproduce the behavior:
terraform init
terraform plan

Expected behavior
Run without error, or do not require the target_id if I wanna link to an ASG. of course, I will create traffic source from my ASG module.

Actual behavior

Got error like below
╷
│ Error: Unsupported attribute
│
│   on .terraform/modules/alb/main.tf line 537, in resource "aws_lb_target_group_attachment" "this":
│  537:   target_id         = each.value.target_id
│     ├────────────────
│     │ each.value is object with 4 attributes
│
│ This object does not have an attribute named "target_id".

That's due to lack of target_id within target_groups definition. I checked source code from .terraform/modules/alb/main.tf line 537, and the target_id is required, and it's stick to a single instance, or a lambda function. But what if I want to bind target group to an autoscaling group instead of a specific instance_id ?
Terminal Output Screenshot(s)

Additional context

check the docs please, we have documented and provided examples for these scenarios

even without consider the autoscaling group, the sample code just doesn't work.

https://github.com/terraform-aws-modules/terraform-aws-alb/blob/master/docs/patterns.md#target-group-without-attachment
https://github.com/terraform-aws-modules/terraform-aws-ecs/blob/master/examples/ec2-autoscaling/main.tf#L167-L232


Add raw diagram files (excalidraw) to repo

👋, I was going through the Cloud-Optimized Geospatial Formats Guide and noticed that the Excalidraw diagrams used in the guide are not part of the repo. Having these diagrams in the repo would benefit traceability, easy updates, cross-sharing content on other presentations, etc.
Why this is important

Traceability: Keeping the Excalidraw source files in version control allows us to understand the evolution of the diagrams over time.
Collaboration: It would make it easier for contributors to update diagrams without recreating them from scratch.
Reproducibility: Having all assets in one place makes the project more maintainable and more accessible for newcomers to understand.

Suggested Solution

Create a directory named diagrams (or something of the like?) in the project's root.
Add all the Excalidraw source files or other formats in the guide to this directory.
Update the README to mention that the Excalidraw source files can be found in this directory.

@kylebarron @abarciauskas-bgse @wildintellect , could you help me figure out who has the diagram files used for example, here:

cc: @kcarini


Service instance operation page for openstack is not available

[Version] OSCM 17.4.0 2017/10/16
Docker image with TomEE
[How to reproduce]
Enter URL: http://estst1.intern.est.fujitsu.com:8880/oscm-app/controller/?cid=ess.openstack
[Expected result]
Log-in, and then Service instance operation page available
[Observed result]
Error 404 No webpage was found for the web address: http://estst1.intern.est.fujitsu.com:8880/oscm-app/controller/?cid=ess.openstack

Parameter cid is changed to controllerId. This is relevant for documentation.
@DawidCh please communicate this.
@SylviaDehn
http://estst1.intern.est.fujitsu.com:8880/oscm-app/controller/?controllerId=ess.openstack should work.

Is it controllerId or controllerid (in the code it looks like the latter - lowercase i)?

@DawidCh Is it controllerId or controllerid (in the code it looks like the latter - lowercase i)?

controllerid

Modified the doc for container-based OSCM accordingly.

verified


Follow cursor/ cursor chomp stops moving if shark and cursor are on the same level up and down.

This affects all fish in follow cursor mode, not just the shark. Somewhere it's checking if X or Y are within a range from the fish's X,Y coordinates when it should check if both X and Y are within the range.

This is fixed in v1.13.


Socket from DOCKER_HOST not used if default socket exists.

I am running both docker and podman in parallel. When I try to make testcontainers to use podman by setting the DOCKER_HOST environment variable the test still connected to the default /var/run/docker.sock. I had to stop the docker service and remove the socket by sudo rm -fr /var/run/docker.sock, after that the test connected to the socked specified int the environment variable as expected.
The socket from the environment variable should be used always even if /var/run/docker.sock exists.

/kind bug

Hi @matejvasek,
Testcontainers will store previous successful strategy as docker.client.strategy in ~/.testcontainers.properties, and try it first before any other strategy.
I'd assume you had a successful run with Docker before switching to Podman. When you stop Docker, Testcontainers will attempt to connect using the previously stored strategy, and, if it fails (after a rather long timeout that we will change in the next version, see #4386), will continue with the next one (no need to delete /var/run/docker.sock), and then store that other strategy as successful.
If you remove docker.client.strategy then Testcontainers will go through the discovery and use standard priorities, and DOCKER_HOST strategy has a higher priority than the unix socket one.

@bsideup you are correct removing the strategy helped.


Fix script working directory, add gitignore

Modifications
Backup Script

Assigned archive path to $backup_dir similarly to the Restore script
${PWD} will return the user's working directory, not the script's location, use $(dirname "$0") for that
-p Unnecessary in the mkdir, seeing as the directory containing the script exists

Restore Script
Changed ${PWD} to $(dirname "$0") same as backup script
Additions
.gitignore - For the archive not to be committed into the repository

Great work! I'm pleased that you found it useful.


[New] A custom quest for Descent 2nd edition (in French)

Description of Problem or Question
Valkyrie Version :
Hello there,
I tried to follow the tutorials and managed to package a small independent quest for Descent, in order to expand the french resources. Would it be possible to look at it ?
The ini file is here :
https://easyupload.io/aga4an
The valkyrie file is here :
https://easyupload.io/788jct
Let me know if I failed somewhere or if it isn't the proper way to submit a new quest.
Best regards,

Hi, you need to have the .valkyrie, .ini and thumbnail (if you have one) all in the same directly linkable location. This can be done with github, just create a repository and add the files. Doing it this way means that you can then update your project at any time without raising an issue.

@BenoitLeoutre : plus, you need to change the file name. Names such as Editorcenario1.valkyrie are not allowed.
Rename the file with the scenario title, i.e., but please, do not use special characters like ê, ç or ñ

Well, I tried to correct it, and here it is : https://github.com/BenoitLeoutre/FestivalDeSang
Is it better ?

Well, I tried to correct it, and here it is : https://github.com/BenoitLeoutre/FestivalDeSang
Is it better ?

I think it is.  @scrubbless, is everything OK for publishing this Descent'quest?
By the way, I had a look to your quest and it looks great. Could I make the Spanish translation of your scenario? Not right now, but after you get some feedback or make any fixes (a couple of weeks?)

I believe so, I'll add it now

Well, I tried to correct it, and here it is : https://github.com/BenoitLeoutre/FestivalDeSang
Is it better ?

I think it is. @scrubbless, is everything OK for publishing this Descent'quest?
By the way, I had a look to your quest and it looks great. Could I make the Spanish translation of your scenario? Not right now, but after you get some feedback or make any fixes (a couple of weeks?)

Yes, of course I can, I would be honoured. Best regards,


How to add elements to selection?

Is there a way to add item to a selection?
I saw the ".select" method but it does not seems to work as i think:
https://codepen.io/anklud/pen/MWKrXrb?editors=1111

First, .select() takes a selector (or multiple ones), not raw elements. Second it does not fire any events and only appends these elements to the list. Both things are mentioned in the readme :)

selection.select(query:[String]|String) - Manually appends elements to the selection, can be a / an array of queries / elements. Returns actual selected elements as array.

Hi @Simonwep  !
I'm trying to figure out how to select all elements with a click on a button, but same as @alvise , I must be doing something wrong.
I've tried to directly add the .selected class to my .file elements and then run selection.select($('.file')) or selection.select(document.getElementsByClassName("file")) on page load, but no luck.
Here's a test code I'm using. selection_result is returning a bunch of elements, but getSelection() is empty :
  var selection_result = selection.select(".file")
  console.log('SELECTION RESULT', selection_result)
  console.log('SELECTION GET', selection.getSelection())

Console log :
SELECTION RESULT (48) [div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file, div.file]
SELECTION GET []

Am I doing something wrong? 🤔
Great work by the way, this is a beautiful library

@albandum getSelection returns the stored items, you'll have to call selection.keepSelection() to move your selected files to the ones you want to keep. Actually I don't know why they don't get added to the selection directly, I'll take a look at this.

Thanks a lot, adding keepSelection does the job :)

The v2-beta will now automatically fire the move event on .select


Create /auth/cli/download

Fetch latest release from Github API and provide to the frontend for direct download.

What version should it download? latest or should we have some way to encode the compatible version which ideally the latest one should be.
Will GitHub ratelimit api requests? Should we cache the responses? Any possible fallbacks like hard coding urls for download.

What version should it download? latest or should we have some way to encode the compatible version which ideally the latest one should be.

Answer should come from what is the compatibility relation between CLI and Paralus. We currently not decided this relation yet. It's good time to think and come up with compatibility matrix. The latest version of CLI may or may not always compatible with the Paralus version someone is using.
Created separate ticket to decide on compatibility matrix: https://github.com/paralus/paralus/issues/214

Will GitHub ratelimit api requests? Should we cache the responses? Any possible fallbacks like hard coding urls for download.

Currently for unauthenticated user, the GitHub sets rate limit to 60 requests per hour which is fair number for getting-started user or low to mid-size group of users. We may reduce the GitHub calls by caching the binary at back-end or front-end which can be done using ETag or Last-Modified headers.
However for really big organization Paralus should have option to set GitHub token that would increase the rate limit to 5000 requests per hour.
Few other topics to discuss before we start implementation:

Is the Dashboard only client which requires this endpoint? If yes, should we move this complete download facility to Dashboard repo?
Should we provide download for different architectures (x86, x86_64, arm) along with OS and distros?
Will this CLI download API placed behind authentication and authorization?

Closing as we no indent to support this feature.
Reopen if you're interested of having this feature.


chore(main): release 0.3.6

:robot: I have created a release beep boop
0.3.6 (2023-12-06)
Bug Fixes

publish (#48) (aaf9985)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/gouarin/splinart-cpp/releases/tag/v0.3.6 :sunflower:


Calling GetAllAirports() returns an exception - An item with the same key has already been added.

var airportProvider = new OpenFlightsDataAirportProvider(airportCache, new OpenFlightsDataCountryProvider(countriesCache));
Airports = airportProvider.GetAllAirports().ToList();
This results in an exception of : An item with the same key has already been added.

Thank you @thevlf. This is an error in the source data. We have written a fix to address this. Version 1.1.0 will be available shortly.


Search for recently updated PRs

It would be nice to create queries that find recently updated PR's for example if we could make a query for "PR's Updated During The Last 7 Days".
The query syntax does not seem to to allow relative date queries, which seems like a big omission on GitHub.com's part. However the extension could do the math for you with some invented syntax. So "Last 7 Days" does something like updated:>2020-06-15 or whatever date was 7 days ago.

Second this!
A relative date query is also what I was looking for 👍
My specific case was to create a query for PRs merged in the last 7 days. For now I'm using a query for all merged PRs.

GitHub supports this in their query syntax!
  "githubPullRequests.queries": [
    {
      "label": "Updated Last 7 Days",
      "query": "is:open updated:>=2024-12-10"
    }
  ],

@alexr00 That's not quite what we're after. The ask is for relative dates, which would probably need to be a variable resolved within the VS Code extension. Your example is an absolute date that would need manually updated daily to work. It would be great to use something like:
{
	"label": "My Past Reviews",
	"query": "is:closed reviewed-by:${user} updated:>${date -1w}"
}

Understood, thanks for clarifying.


Migrate to @sentry/node

Refactored params to use destructuring with defaults, so the normalization is no longer necessary
Added extras to use Sentry’s scope.setExtra() to enrich the error data
Added captureReturnedErrors option to capture errors from other middlewares that return errors instead of throwing, e.g., graphql-shield https://github.com/maticzav/graphql-shield#custom-errors

Closes #29

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Event Request] Page 104 "Account Schedule" - OnBeforeSetupAccSchedLine

Describe the request
Hello, can you please add a new Event OnBeforeSetupAccSchedLine in Page 104 "Account Schedule"?
    procedure SetupAccSchedLine(var AccSchedLine: Record "Acc. Schedule Line")
    var
        IsHandled: Boolean;
    begin
        AccSchedLine := Rec;
        OnBeforeSetupAccSchedLine(Rec, xRec, AccSchedLine, CurrentSchedName, IsHandled);
        If IsHandled then
            exit;
        if Rec."Line No." = 0 then begin
            AccSchedLine := xRec;
            AccSchedLine.SetRange("Schedule Name", CurrentSchedName);
            if AccSchedLine.Next() = 0 then
                AccSchedLine."Line No." := xRec."Line No." + 10000
            else begin
                if AccSchedLine.FindLast() then
                    AccSchedLine."Line No." += 10000;
                AccSchedLine.SetRange("Schedule Name");
            end;
        end;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeSetupAccSchedLine(var Rec: Record "Acc. Schedule Line"; var xRec: Record "Acc. Schedule Line"; var AccSchedLine: Record "Acc. Schedule Line"; CurrentSchedName: Code[20]; var IsHandled: Boolean)
    begin
    end;

Additional context
An additional event is required in page 104 ("Account Schedule") to customize the insertion of Acc. Schedule Lines. The event needs to be able to override the default behaviour, using an IsHandled flag.Internal work item: AB#494284

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14858.


Parallelization for standalone mode with NCCL

BTW, maybe we can also support parallelization when msg_type = 'evaluation'


Merge release-1.2.1 to develop

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Merge release-1.2.1 to develop
Why
N/A
Known limitations
N/A

Reference Server Preview is available here:https://anchor-ref-pr634.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr634.previews.kube001.services.stellar-ops.com/


Enhancement: Docs upgrade- kit-docs and sveltejs/kit

Description
Upgrades:
@svelteness/kit-docs: 1.1.3
@sveltejs/kit: 1.5.0
Fixes font loading
Routes restructured to work with new kit-docs structure: https://kitdocs.vercel.app/docs/getting-started/quickstart
PLEASE NOTE- Checklist must be complete prior to review.
Checklist

[NA] Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
[x] Check the box that allows repo maintainers to update this PR
[x] Test locally to make sure this feature/fix works
[x] Run yarn check-all to confirm there are not any associated errors
[x] Confirm this PR passes Circle CI checks
[x] Add or update relevant information in the documentation

@Adamj1232 realized I forgot to match the .prettierrc to the original version from the docs- updated and reformatted. Hopefully looks less wonky on the IDE now! LMK what you think!

Maybe a slight regression on the styling of the tabs?
Overall looks good but would be helpful to have @gesquinca review

It also looks like the Unstoppable module has slipped out of the Modules group

@Adamj1232 unstoppable domains has been reintegrated in the new file structure
I should have linked the ticket b/c I did make a comment about there being changes to kitdocs default styling:
"Default styling has changed slightly- working to match to current styling as close as possible, but some features (like the header- as that is compiled/styled by default in kitdocs) may look slightly different"
I have pulled up the current version and the new version side by side and matched as closely as possible. Without making changes to default kitdocs components- like the header. I think final styling tweaks should be a separate ticket but tagging @gesquinca for a review/opinion.

@Adamj1232 unstoppable domains has been reintegrated in the new file structure I should have linked the ticket b/c I did make a comment about there being changes to kitdocs default styling: "Default styling has changed slightly- working to match to current styling as close as possible, but some features (like the header- as that is compiled/styled by default in kitdocs) may look slightly different"
I have pulled up the current version and the new version side by side and matched as closely as possible. Without making changes to default kitdocs components- like the header. I think final styling tweaks should be a separate ticket but tagging @gesquinca for a review/opinion.

Sweet! That sounds good to me! I do remember a note on some styling differences, apologies on forgetting that, Palooza brain got me 🤪
We may want to wait on merging until the styling changes are completed as functionally they are equivalent... @gesquinca thoughts?


azure deploy, add persistent postgres db

Add setup document
Add postgres service or use a persistent/managed disk attached to a ephemeral postgres docker image

Adding a persistent disk can be done in the google cloud, not sure about Azure

In settings, configure persistent file storage

Current demo will not progress as Dataverse is significantly changing


⚠ SceneTime has degraded performance

In c0e3028, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1398 ms

Resolved: SceneTime performance has improved in 65ebf57 after 7 minutes.


Add View and Backend classes

The View class makes the object expected by showView a bit more self-documenting.
The Backend class encapsulates all the details of API requests and response handling so that the user interface functions don't need to manage them any longer.
Replacing calls to cl.xhr with calls to cl.backend was a case where having the end-to-end tests paid off; it caught the fact that cl.userId wasn't getting set properly after getting all of the browser-only tests to pass.

Coverage increased (+0.02%) to 97.969% when pulling 0d47f3c90cdb4e390831fec0b7d07bae6f62ae9f on view-and-backend into d2af85871fb80cca9a09ce32e47c596808a03247 on master.

Coverage increased (+0.02%) to 97.969% when pulling 0d47f3c90cdb4e390831fec0b7d07bae6f62ae9f on view-and-backend into d2af85871fb80cca9a09ce32e47c596808a03247 on master.

Coverage increased (+0.02%) to 97.969% when pulling e4416c2f74e99adc7615d50eddbec5d462bd0901 on view-and-backend into d2af85871fb80cca9a09ce32e47c596808a03247 on master.


New tags on DockerHub ?

Are the new tags (the ones created in #24) available on DockerHub ? They do not appear on https://hub.docker.com/r/phpunit/phpunit/tags/

I create soon an automation, so sorry 👍

perfect, thanks !


ENYO-1163 Fix: Light theme: Icons unexpectedly blend inside Button

Enyo-DCO-1.1-Signed-off-by: Yevgen Sukharenko<EMAIL_ADDRESS>

Closing out since Jira ticket has been closed as will not fix.


applocal.ps1 should check dependencies timestamps

applocal.ps1 only checks if the dependency was previously copied. I believe it should also check if the files are different to avoid keeping and old copy of a port dll in the output directory of a consuming project.
And while you are at it, it could try to hardlink the dll if possible and copy otherwise.

This issue hasn’t been updated in a year, if it is still an issue, please reopen.


Progress bar not drawn when stdout is redirected on Windows

How to reproduce:
bar := pb.New(1000)
bar.Output = os.Stderr
bar.Start()

Run with:
myprogram > output

Expected:
Progress bar is drawn because output goes to Stderr and Stderr is a terminal.
Actual:
No progress bar is drawn, only counters and percentage is shown.

I can reproduce this too on Windows 10 64 bit.
go get gopkg.in/cheggaaa/pb.v1

I think this is fixed in ca37cd7eed45ea7cee15cba4abdf022116a01ce7


using "%s[discord][token]"

str  is
{
   "discord": {
      "token": "XXX"
   },
   "logging": {
      "filename": "YYY",
      "level": debug
   }
}

jscon_scanf(str,
              "%s[discord][token] %s[logging][filename] %s[logging][level]",
              settings->discord.token,
              settings->logging.filename,
              settings->logging.level);

Does this work?

It won't work for "level": debug because of the missing double quotes token for debug. It will attempt to interpret it as another primitive.

It won't work for "level": debug because of the missing double quotes token for debug. It will attempt to interpret it as another primitive.


Does this project support Android?

Actually, we never tested it and plan to do that. We also lack experience in mobile development, so we will not commit to such tasks.
But, we'll be very happy if you are interested in contributing to the project


Get covariates for daymet records

I haven't dealt with long format tables before. I need to add the covariates to the daymet records so that each of the parameters below have their own row. Is there an easy way to add this to the query below?
var.names <- c("Latitude", "Longitude", "airTemp", "airTempLagged1", "airTempLagged2", "prcp", "prcpLagged1", "prcpLagged2", "prcpLagged3", "dOY", "Forest", "Herbacious", "Agriculture", "Developed", "TotDASqKM", "ReachElevationM", "ImpoundmentsAllSqKM", "HydrologicGroupAB", "SurficialCoarseC", "CONUSWetland", "ReachSlopePCNT", "srad", "dayl", "swe")

Query:
  qry_daymet <- tbl(db, 'daymet') %>%
    left_join(select(qry_locations, site, featureid), by='featureid') %>%
    filter(featureid %in% catches) %>%
    mutate(airTemp = (tmax + tmin)/2)

eh yeah, i'm not exactly sure how to do that with SQL. i think you can, but we might need to get the covariates separately, use tidyr to convert from long to wide, and then merge with the daymet table. i'll give it a shot...

Okay, I'll be doing it in small enough chunks that it shouldn't be a big deal collecting to R then tidying then doing a left_join.

something like this? one issue is whether you want local or upstream covariates. does that vary the covariate variable? i just filtered for upstream ones here.
# covariates
qry_covariates <- select(qry_locations, site, featureid) %>%
  left_join(tbl(db, 'covariates'), by='featureid')

df_covariates_long <- collect(qry_covariates)

df_covariates_wide <- tidyr::spread(df_covariates_long, variable, value)

# which zone? upstream or downstream?
df_covariates_upstream <- filter(df_covariates_wide, zone=="upstream")

masterdata <- left_join(tbl_daymet, df_covariates_upstream, by=c('featureid', 'site'))

Houston, we may have a problem. Do we only have these covariates for the catchments with observed data? I'm getting a lot of NA. I need these to do predictions to ALL catchments.

uhh, better ask kyle. i haven't really looked at the data, just uploaded what he gave us.

False alarm. I couldn't get a hold of Kyle but I did another check and I think it's okay. At first I just pull the covariates for the first 100 featureid and all the covariates except area and elevation were NA for almost every catchment. I just did a random sample of 10,000 catchments and there were only 222 with NA for forest and 473 with NA for surfcoarse.
The way  the featureid were indexed must have happened to start with ones from an area without GIS data for the catchments.


goal: actual moveit/copyit log filenames are returned as string

Right now, sipcreator and masscopy have to do some guesswork to figure out what the lognames are.
This is because moveit is called as a subprocess and not as a function that could return a value.
So the goal should be: moveit.main() should return logname.log.

Actually, the log name is pointless here  - just return pass/fail.


MdInputContainer should change colors with the dividerColor directive
What is the current behavior?
Nothing happens when changing the dividerColor, and clicking on it does nothing
What are the steps to reproduce?
The main focus below is in the Username and Password fields
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/o077B6uEiiIgkC0S06dd
Have a custom theme, and use the code below. The example in the docs doesn't work either

What is the use-case or motivation for changing an existing behavior?
I'm not sure what to say other than it doesn't work at all
Which versions of Angular, Material, OS, browsers are affected?
Angular: 2.3.2
AngularMaterial2: 2.0.0-beta.1
Is there anything else we should know?

Are any of the inputs in your screenshot focused? the dividerColor only affects the color when focused. It seems to be working fine to me

@mmalerba The first one is focused. I changed my version from 2.0.0.beta-1 2.0.0.alpha.9-3 and changed the containers to the deprecated md-input and it worked fine. I also tested this on another project and got the same result, meaning it isn't just a weird use case

It seems to be working for me: http://plnkr.co/edit/42trGBbs8gdShrtmZf8U?p=preview you might have CSS styles on your site that are interfering. your inputs look darker than they naturally would be

I've narrowed this down to an issue with custom theming. If you include the css style in the html sheet like you have it works fine, but with custom theming it does not. All other components are functional with the styling sheet being imported with scss that's how I know the import is at least getting through.

It works for me when using custom themes as described in the guide: https://github.com/angular/material2/blob/master/guides/theming.md


:ghost: Change default application inventory tab

Change default tab to analysis

@ibolton336, I have no pb with with this change, I'm just wondering why analysis tab is becoming the default ?


update the correct operator link

The original link of operator pattern is a meaningless page. This commit updates the link to the correct operator pattern page.

I've signed CLA.

Thanks.
/lgtm
/approve


Improve handling of automanaged namespaces

Allow multi-config scenario where single automanaged namespace is reused between tests
What type of PR is this?
/kind bug
What this PR does / why we need it:
Currenly, leaving automanaged namespaces between configs breaks non-first config. This PR handles this use case by adding an explicit tracker of all created automanaged namespaces and by fixing verification in appropriate places.
Which issue(s) this PR fixes:
Fixes #1848

/unhold


Coordination service v1

Реализация клиента к сервесу координации.
И решение базовых сценариев как leader election

Codecov Report
Patch coverage: 54.11% and project coverage change: +0.29 :tada:

Comparison is base (6f8a761) 50.29% compared to head (86a5c4e) 50.59%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop      #91      +/-   ##
=============================================
+ Coverage      50.29%   50.59%   +0.29%
- Complexity       996     1008      +12
=============================================
  Files            150      151       +1
  Lines           6201     6295      +94
  Branches         655      658       +3
=============================================
+ Hits            3119     3185      +66
- Misses          2847     2873      +26
- Partials         235      237       +2

Impacted Files
Coverage Δ

...rc/main/java/tech/ydb/core/grpc/GrpcTransport.java
4.00% <ø> (ø)

...ain/java/tech/ydb/core/impl/BaseGrpcTransport.java
0.00% <0.00%> (ø)

.../tech/ydb/core/impl/FixedCallOptionsTransport.java
0.00% <0.00%> (ø)

...java/tech/ydb/core/impl/MultiChannelTransport.java
0.00% <0.00%> (ø)

...ava/tech/ydb/core/impl/SingleChannelTransport.java
0.00% <0.00%> (ø)

...main/java/tech/ydb/core/impl/YdbTransportImpl.java
0.00% <0.00%> (ø)

...main/java/tech/ydb/core/impl/auth/GrpcAuthRpc.java
0.00% <0.00%> (ø)

...tech/ydb/core/impl/discovery/GrpcDiscoveryRpc.java
0.00% <0.00%> (ø)

...ydb/test/integration/utils/ProxyGrpcTransport.java
0.00% <0.00%> (ø)

...a/tech/ydb/core/impl/operation/OperationUtils.java
25.00% <25.00%> (ø)

... and 5 more

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


'Core' version of the package

Hey Andy
Thanks so much for porting this over to Core, we used it previously on 7 and its been great to work with. One issue we have had recently is that if we want to install the package references into multiple projects there is only 1 NuGet package and it comes with all the app_plugin stuff.
Would it be feasible to have a Core package that can be installed so that other projects in the solution can have the dll references?

Sounds like that would be a good idea yes.  I'll try to make this split when I get a bit of time to work on it.

Hi @LachlannRattray - I've pushed out a new RC release 3.2.0-rc001 to NuGet, that has this split.  Would you be able to try it and see if it's set up as makes sense for your solution please?  And if so I can release a full version.
With the change there's a small namespace change - anything that was Our.Umbraco.PersonalisationGroups.* is not Our.Umbraco.PersonalisationGroups.Core.*.

Hey Andy that's great thank you for doing that. I will get it tested as soon as possible.

Hey Andy just to say we have updated to the new Core package and it seem to work perfectly. thanks very much for doing that!

Good to hear.  I've released this now as 3.2.0.


use requests for now to fetch tails file reliably

Signed-off-by: sklump<EMAIL_ADDRESS>

Tested with tails files up to 100,000 entries
Tested on PWD with a mobile agent with a 10,000 entry tails file

Please don't merge this just yet though - I forgot to touch up the NOTICES file. Will amend in the next few minutes.

... it's in, ready to go.


Option To Disable "Add New Item" In Select Array Elements.

- Do you want to request a feature or report a bug?
Feature
- What is the current behavior?
I want the user to be able to edit array items without adding additional new ones. Currently the user can add new items
- If the current behavior is a bug, please provide the steps to reproduce.
- What is the expected behavior?
The 'Add New Item' Button is removed in selected arrays, & the user can only edit the current items

- Please mention your CMS, node.js, and operating system version.

CMS Ver 1.1.0
- Please link or paste your config.yml below if applicable.

This makes sense, agreed.

Hi! I'd like to take this issue, if that's ok with y'all.

@gazebosx3 Sounds great, let us know if you have any questions!

Hey, I'm a bit confused about the requirements of this issue. Is "Array" referring to the array in the Select Widget? If so, I don't see where, in the CMS, users can add elements to a select array...

The button that we want to be optional is on the "List" widget, sorry that wasn't clarified.

@gazebosx3 feel free to reach out in Gitter if you need help in realtime.


Does it make sense that tokio::codec::Decoder's decode method takes a BytesMut?

While trying to use prost with tokio's framed codec mechanism, I found I couldn't use its decode method because it expects an IntoBuf, which BytesMut isn't (I created an issue on prost about this: https://github.com/danburkert/prost/issues/33).
But it seems to me that it doesn't make a lot of sense that decode is being given a BytesMut to begin with. A decoder probably shouldn't be modifying the underlying bytes of the buffer. Wouldn't passing in Bytes or an IntoBuf make more sense here?

This is basically providing a static guarantee that once the bytes have been extracted you're allowed to mutate and change them as necessary. It's true that you probably don't want to modify what's already there, but I don't think we'd want to change to Bytes as that'd be hindering some possible applications.

BytesMut should probably impl IntoBuf

I believe that the answer is "yes", but that BytesMut needs to improve. #158 represents the requested improvement. I'm going to close this issue now. Thanks for the report.


⚠ Jurnal has degraded performance

In 7907255, Jurnal (https://jurnal.unmer.ac.id) experienced degraded performance:

HTTP code: 200
Response time: 5861 ms

Resolved: Jurnal performance has improved in db0de5c.


ci: :construction_worker: update java version

:white_check_mark: Build TheIdServer 1.0.0-user-ole1-0180 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/b29b403bdd by @aguacongas)


Return the same instance of USBDevice from separate getDevices calls

Changes

Currently each call to getDevices on the webusb interface returns new instances of USBDevice even for the same connected device. This causes multiple different issues.
Instead of creating a new instance every time, cache the WebUSBDevice instances corresponding to the underlying Device instances and return the cached instance to ensure we always return the same USBDevice instance for the same device.
As far as I could see, this is the behaviour in web, and should be the intended behaviour for us as well.

Fixes

#566

Checklist

Have you...

[x] Tested the change acts as expected
[x] Checked the change doesn't remove or change existing functionality
[ ] Considered how the feature impacts the product and tested around it (not just the happy paths)
[ ] Where possible, executed the hardware tests on multiple operating systems

Thanks, your suggested change makes sense. I'll test it out.

Thanks for your suggestions, makes sense! I've made some changes to simplify getWebDevice and loadDevices. In the current structure, isAllowedDevice cannot be moved to getWebDevice, since requestDevice needs a list of unfiltered devices to work. Have a look?


🧪 Refactor nightly/pre-merge tests to follow the new refactor design

Restructure tests to follow the same folder structure as the repo.
Depends on

PR https://github.com/openvinotoolkit/anomalib/pull/66
https://github.com/openvinotoolkit/anomalib/pull/87

Closing this issue as this seems to have been addressed already.


Disable automatic token refreshing for tests

As it causes problems (see #35), also it might actually randomly break tests.

@marcoow your commit 75a73a6 affects our tests. can we make this behavior a config variable rather than hardcoding to if (!Ember.testing)?

@johnnyshields: hm, why would you ever want that to run in tests?

for long-running integration tests which test performance. I think the question is really "why would a library hardcode its behavior to not run in tests, rather than making it configurable?" Its fine if the default config is false for test env, true for non-test so as to affect existing users.

I get your point - changing the behavior of sth. if it's running in test env isn't ideal but at the same time a pretty common pattern in Ember.js as you have to make sure tests are not leaking and run loop events.
I think if you're running long-running integration tests to test performance you shouldn't be running the app in test env anyway though as it will behave differently and might give you a wrong sense of its performance even (for example, ESA will always store the session in memory when running in test mode which of course is much faster than using localStorage or a cookie).

@marcoow can we make this a config?

@johnnyshields: I honestly don't see the need for doing that

could we make a more general config test_mode which replaces all usages of Ember.testing within the library, which by default is set to the value of Ember.testing but can be overridden, so that we are not tightly coupled to Ember itself?

@johnnyshields: I honestly don't see the need for doing that. As I said above - when you're executing long-running integration tests, specifically for performance testing you shouldn't be running the app in test mode anyway.

I pray for the day when the Ember community will learn the merits of loose coupling.


fix(engine) Fix crash with hax_lib::fstar::before in recursive bundles

fixes #1177
The crash was due to a name collision between the item that has the attribute hax_lib::fstar::before and the Quote item created by the TransformHaxLibInline phase. Instead of giving it the exact same name as before, we add a postfix.

I guess this was making a collision because of two items sharing a same name below an anonymous const, is that what it is?
The problem is not about the quote I think, but more about the bundling mechanism not ensure unique names while moving items.
I think the naming improvements we talked about yesterday will solve those issues.

I guess this was making a collision because of two items sharing a same name below an anonymous const, is that what it is? The problem is not about the quote I think, but more about the bundling mechanism not ensure unique names while moving items.
I think the naming improvements we talked about yesterday will solve those issues.

The problem is not only about the new names in the bundle. The collision is a problem with the old names because when we rename we need original names to be unique (otherwise if we have two things with the same name and they get different names in the bundle, when we find an occurrence of the old name and have to rename it, which one of the new names do we choose?).
The naming improvements could help by allowing to define a name for the Quote item that is different from the item it is attached to (and doing so in a nicer way that what is in this PR). But we need to do this at the creation of the Quote item.

We should wait for the naming to be implemented and use a fresh ident.


Appium server indicates a different deviceName from the one used in desiredCapabilities

I have no issue running Android tests, but all of my iOS tests fail. When the tests attempt to run, the Appium server displays the error [simctl] simctl error running 'create': Invalid device type: iOS Emulator and the server logs also indicate that "iOS Emulator" is being used as the deviceName, even though my deviceName is set to "iPhone 11 Pro Max" in desiredCapabilities.

It's all about passing the right set of desired capabilities. Here's the link to a similar question on SO. In your case, the behavior you're getting might be better explained by the next messages in the log:

The platformVersion capability is missing, but XCUI driver expects it.
There's a message saying Simulator udid not provided which means you must pass the udid capability. A quick search I just did returned another SO answer that says you can find the UDID of the simulator you want to use with the instruments -s devices command.

Try searching for some good articles on the proper desired capabilities setup for XCUI.

I'd cloned the opentest repo into my project, and ran mvn install in the opentest/actor/base directory. For whatever reason this resulted in me having an old version of openest. Once openest was updated, the issue no longer occurred.


Add route to test transformation, validations, or RITS from a Record

It would be handy to have a convenient way to test:

transformations
validation scenarios
RITS (record id transformations)

from a single Record page.   It is possible to navigate to these screens and then find the Record in question, but that table is not designed for deep searching, and it can be difficult to know that copying/pasting a Record's db_id is a sure fire way to find it.  Particularly when all those screens allow a ?q=3671328 style syntax for loading those testing pages with a Record already highlighted.

Done - added "Test/Run Scenarios for this Record" tab.  Perhaps not ideal, but placement until better spot determined.


stem fails with gmsh 4.12.0, find out why and create a test which catches this in gmsh_utils

4.11.1 seems to work fine

this should be fixed in the gmsh master, I expect it to work in (future) versions greater than 4.12.2


Cannot read property 'padding' of undefined

Hello,
First of nice work!
Secondly, I have been trying to implement your example StyleByCSS but I am getting this error:
What am I missing? everything works fine, except tooltip.
main.js:1822 Uncaught TypeError: Cannot read property 'padding' of undefined
    at u.Tooltip.init (main.js:1822)
    at new u.Tooltip (main.js:1822)
    at n.value (adviser.entry.js:60548)
    at CallbackQueue.notifyAll (adviser.entry.js:7138)
    at ReactReconcileTransaction.close (adviser.entry.js:17784)
    at ReactReconcileTransaction.closeAll (adviser.entry.js:7796)
    at ReactReconcileTransaction.perform (adviser.entry.js:7743)
    at ReactUpdatesFlushTransaction.perform (adviser.entry.js:7730)
    at ReactUpdatesFlushTransaction.perform (adviser.entry.js:6898)
    at Object.flushBatchedUpdates (adviser.entry.js:6981)

My code:
    <HighchartsChart plotOptions={plotOptions}>
        <Tooltip
          borderWidth={3}
          shadow={false}
          hideDelay={0}
          padding={8}
          headerFormat={'Something'}
          pointFormat={'{series.name}: {point.percentage:.1f}%'}
          valueSuffix={'%'}
        />
        <PieSeries
            id="my-pie"
            name="Asset Allocation"
            data={chartData.series[0].data}
        />
    </HighchartsChart>

Thanks
Note:
If we pass padding inside style prop, it doesn't crash, but pointFormat, valueSuffix, headerFormatare not set.
<Tooltip
          borderWidth={3}
          shadow={false}
          hideDelay={0}
          padding={8}
          headerFormat={'Something'}
          pointFormat={'{series.name}: {point.percentage:.1f}%'}
          valueSuffix={'%'}
          style={{padding: 8}}
/>

Apparently we are looking for padding inside style if I do this:
<Tooltip style={{padding: 8}}/>
If works fine.

I'm not too sure on this one, I'm struggling to reproduce it. Could you provide a bit more information?
Such as;

Highcharts version?
Are you exporting your Chart using withHighcharts?
Are you using server side rendering?

Sorry for the late reply, been on holiday for the past 2 weeks!

Hi @whawker thanks for you reply,

Version
-- 6.0.3
Are you exporting your Chart using withHighcharts?
-- Yes
Are you using server side rendering?
-- No,  Client side

No worries, I gave up.
End up writing a wrapper of my own, it was for a very small proof case.
Thank you.


Update client images to shapella-ready version

Changes:

Update client images to shapella-ready version

Types of changes

[ ] New feature (non-breaking change which adds functionality)

Testing
Requires testing

[x] Yes

Codecov Report
Patch coverage has no change and project coverage change: -0.07 :warning:

Comparison is base (961474b) 85.99% compared to head (27c53d0) 85.92%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #271      +/-   ##
===========================================
- Coverage    85.99%   85.92%   -0.07%
===========================================
  Files           76       76
  Lines         4491     4491
===========================================
- Hits          3862     3859       -3
- Misses         545      548       +3
  Partials        84       84

Flag
Coverage Δ

unittests
85.92% <ø> (-0.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


[QUESTION] How to use MQTTnet with Unity3D ?

Describe your question
Hello. I use MQTTnet on a console C# app. Which is my broker. I want a MQTT client made with Unity3D to be able to discuss with that broker.
I tried to use Nuget For Unity and get the Nuget Package but there is too much depencies, and theses dependencies are conflicted with Unity internal Engine.

What should I do to use MQTTnet in Unity3d ?
Thank you.
Which project is your question related to?

Client

I am no Unity expert so I can only guess...
What I read sometimes is that you need to download the nuget file, extract it and reference the lib directly (also adding some microsoft ones). Then it might work.
Apart from that I have no idea. The project is netstandard and just requires some libraries for web socket communication etc.

You can just build the MQTTNet project yourself and use the Standard2.0 dll.


README user must modifiy AUTHORITY_URL

The README is missing the need to change the AUTHORITY_URL,
from:
AUTHORITY_URL = 'https://login.microsoftonline.com/common'
to:
AUTHORITY_URL = 'https://login.microsoftonline.com/contoso.onmicrosoft.com'
as per:
https://github.com/AzureAD/azure-activedirectory-library-for-python/wiki/ADAL-basics
else the example fails with:
AdalError(return_error_string, error_response) adal.adal_error.AdalError: Get Device Code request returned http error: 400 and server response: {"error":"invalid_request","error_description":"AADSTS50059: No tenant-identifying information found in either the request or implied by any provided credentials....

Closing this issue as we are archiving this repository. Please see https://github.com/microsoftgraph/msgraph-training-pythondjangoapp for a Python sample.


Update setup-gradle.rst and setup-maven.rst

.. because it's outdated.

Remove the configuration related with Jetty ALPN agent

We support Windows, Linux and OS X using netty-tcnative-boringssl-static.

List all module dependencies

Current coverage is 64.94% (diff: 100%)

Merging #345 into master will increase coverage by 0.01%

@@             master       #345   diff @@
==========================================
  Files           327        327
  Lines         13140      13140
  Methods           0          0
  Messages          0          0
  Branches       1948       1948
==========================================
+ Hits           8532       8534     +2
+ Misses         3698       3697     -1
+ Partials        910        909     -1

Powered by Codecov. Last update 390e91c...63736f2

Thanks, @kojilin !


Refactored App tests

Refactored App tests. Additionally added accessors to router, api, storage and block-info (used in integration tests).

The App implementation is not trivial and the file is quite big. The number of tests is also quite impressive, so it is not easy to navigate this file containing code-base and test cases. The second reason is, that extracting the tests shows immediately if there are lacking accessors. The third reason is, that most of the tests in App are integration tests (not unit tests), so in fact they should have been be placed outside the crate in /test directory. But these tests use helper contracts intensively, which unfortunately are all private inside crate, so the current solution is somewhat a compromise  between the ideal way and current code-base state.


Questions and feedback

Hi, first off, awesome work on this!
It is super useful in its current state, and I only really miss few things:

go-to/peek definition - i. e. being able to hover (K) or vim.lsp.buf.definition()
running alongside yaml LSP - not sure how this is possible (I guess the yaml needs to be stripped), however, what I hope to achieve is yaml schema validation
extra: perhaps https://github.com/ngalaiko/tree-sitter-go-template can be adopted for better syntax highlighting

A question though, do you have a wishlist or set of features already?
Edit: Trying not to mix up LSP and TS

I guess we can close this by now. If you have any other feature suggestions feel free to open another issue.
For syntax highlighting: https://github.com/ngalaiko/tree-sitter-go-template/pull/13 is looking promising.