[R] Implement bindings for cumsum function

Describe the enhancement requested
There exist cumulative functions in cpp, but those aren't linked to cumsum within R.
Could it be possible to add the link ?
The point is that to compute something that is pretty common in stats, a weighted median, one has to make raw function calls.
Thanks
Component(s)
R

Would you be interested in submitting a PR to add this, @arnaud-feldmann ?

Wonderful, thanks!  Let us know if you have any issues, or anything we can help with.

Tried all the afternoon, seems like it is above my level. Thought it was just a lacking binding, now I understand there's a bigger problem.
Sorry for the bother @thisisnic . Thanks for your work

It's not a bother at all!  Happy to guide if you've got a specific bit you were stuck at, or if you'd rather not, that's totally fine, but mind sharing the bigger problem, so if I (or someone else) has time to pick this up, I can use that as a starting point?
I've not had the chance to look into it at all myself yet, so you're ahead of me on this right now.
Honestly, this stuff can be tricky, and that's coming from someone who's been working on this stuff for 2 years and still gets tripped up all the time!

Right now I do that ugly thing to compute weighted medians :
library(arrow)
#> Some features are not enabled in this build of Arrow. Run `arrow_info()` for more information.
#>
#> Attachement du package : 'arrow'
#> L'objet suivant est masqué depuis 'package:utils':
#>
#>     timestamp
library(dplyr)
#>
#> Attachement du package : 'dplyr'
#> Les objets suivants sont masqués depuis 'package:stats':
#>
#>     filter, lag
#> Les objets suivants sont masqués depuis 'package:base':
#>
#>     intersect, setdiff, setequal, union
tf <- tempfile()
dir.create(tf)
write_dataset(mtcars, tf, partitioning = "cyl")
ds <- open_dataset(tf)

# weighted median of gross horspower by weight
get_median <- function(arrow, var, wt) {
  var <- ensym(var)
  wt <- ensym(wt)

  sel <-
    arrow %>%
    filter(! is.na(!! var) & ! is.na(!! wt)) %>%
    mutate(!! var := as.double(!! var),
           !! wt := as.double(!! wt)) %>%
    arrange(!! var)
  sel_wt <- sel %>% pull(!! wt, as_vector = FALSE)
  sel_val <- sel %>% pull(!! var, as_vector = FALSE)
  sel_cum_poids <- call_function("cumulative_sum_checked",sel_wt)
  sel_sum_wt <- Scalar$create(sel_cum_poids[sel_cum_poids$length()])
  sel_val[sel_cum_poids>  sel_sum_wt/2][1L]$as_vector()
}
get_median(ds, hp, wt)
#> [1] 175

This isn't very pretty. That'd why I searched for a direct binding.
ds %>%
  mutate(cs = cumsum(wt)) %>% collect()
#>     mpg  disp  hp drat    wt  qsec vs am gear carb cyl     cs
#> 1  21.0 160.0 110 3.90 2.620 16.46  0  1    4    4   6  2.620
#> 2  21.0 160.0 110 3.90 2.875 17.02  0  1    4    4   6  5.495
#> 3  21.4 258.0 110 3.08 3.215 19.44  1  0    3    1   6  8.710
#> 4  18.1 225.0 105 2.76 3.460 20.22  1  0    3    1   6 12.170
#> 5  19.2 167.6 123 3.92 3.440 18.30  1  0    4    4   6 15.610
#> 6  17.8 167.6 123 3.92 3.440 18.90  1  0    4    4   6 19.050
#> 7  19.7 145.0 175 3.62 2.770 15.50  0  1    5    6   6 21.820
#> 8  22.8 108.0  93 3.85 2.320 18.61  1  1    4    1   4  2.320
#> 9  24.4 146.7  62 3.69 3.190 20.00  1  0    4    2   4  5.510
#> 10 22.8 140.8  95 3.92 3.150 22.90  1  0    4    2   4  8.660
#> 11 32.4  78.7  66 4.08 2.200 19.47  1  1    4    1   4 10.860
#> 12 30.4  75.7  52 4.93 1.615 18.52  1  1    4    2   4 12.475
#> 13 33.9  71.1  65 4.22 1.835 19.90  1  1    4    1   4 14.310
#> 14 21.5 120.1  97 3.70 2.465 20.01  1  0    3    1   4 16.775
#> 15 27.3  79.0  66 4.08 1.935 18.90  1  1    4    1   4 18.710
#> 16 26.0 120.3  91 4.43 2.140 16.70  0  1    5    2   4 20.850
#> 17 30.4  95.1 113 3.77 1.513 16.90  1  1    5    2   4 22.363
#> 18 21.4 121.0 109 4.11 2.780 18.60  1  1    4    2   4 25.143
#> 19 18.7 360.0 175 3.15 3.440 17.02  0  0    3    2   8  3.440
#> 20 14.3 360.0 245 3.21 3.570 15.84  0  0    3    4   8  7.010
#> 21 16.4 275.8 180 3.07 4.070 17.40  0  0    3    3   8 11.080
#> 22 17.3 275.8 180 3.07 3.730 17.60  0  0    3    3   8 14.810
#> 23 15.2 275.8 180 3.07 3.780 18.00  0  0    3    3   8 18.590
#> 24 10.4 472.0 205 2.93 5.250 17.98  0  0    3    4   8 23.840
#> 25 10.4 460.0 215 3.00 5.424 17.82  0  0    3    4   8 29.264
#> 26 14.7 440.0 230 3.23 5.345 17.42  0  0    3    4   8 34.609
#> 27 15.5 318.0 150 2.76 3.520 16.87  0  0    3    2   8 38.129
#> 28 15.2 304.0 150 3.15 3.435 17.30  0  0    3    2   8 41.564
#> 29 13.3 350.0 245 3.73 3.840 15.41  0  0    3    4   8 45.404
#> 30 19.2 400.0 175 3.08 3.845 17.05  0  0    3    2   8 49.249
#> 31 15.8 351.0 264 4.22 3.170 14.50  0  1    5    4   8 52.419
#> 32 15.0 301.0 335 3.54 3.570 14.60  0  1    5    8   8 55.989

So I tried binding cumulative_sum_checked to cumsum, but I realise that, even with the UDF helper, things starts again at each chunk, so I guess, as cumsum is a array function (not a scalar one), things are more complicated.

Thanks for all the effort there trying to make this work, I can see you've delved right into the deep end of things!

I tried to find a place for Array functions, but I realise that there are none

So this isn't super well-documented, which is most likely why you didn't find it, but there's an R6 class in the package called ArrowDatum.  It's the base class for Array, ChunkedArray, and Scalar objects, and the point of it is basically to allow developers to write a function e.g. my_fun.ArrowDatum(), which will run on object of this class.
For example, the S3 method for the unique() function (which has an identically named C++ equivalent function) looks like this:
#' @export
unique.ArrowDatum <- function(x, incomparables = FALSE, ...) {
  call_function("unique", x)
}

There's a PR here which shows someone implementing exp and sqrt both for Tables and for ArrowDatum objects.
https://github.com/apache/arrow/pull/13517/files
I reckon you could probably take a similar approach for the cumsum function.  It might end up being simpler than you thought (but it's the surrounding complexity, and things being hidden away which make it, reasonably, look complex).

@thisisnic I have put the binding for cumsum. As for the datasets, as far as I understand this isn't possible since we use Scalar Expressions.
Thank you for your help.


Fix namespaces and covers comments.

Context

Summary
While working on https://github.com/google/web-stories-wp/pull/7725 I found a number of places where the php unit tests were wrong. Namespaces, missing comments, covers wrong. Fix PR fixes them.
Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[ ] I have added a matching Type: XYZ label to the PR

Fixes #

Neat! Looking good.
A few warnings remain:
1) Google\Web_Stories\Tests\Media\Media::test_filter_ajax_query_attachments_args
Trying to @cover or @use not existing method "\Google\Web_Stories\Media\Media::get_poster_tax_query".

2) Google\Web_Stories\Tests\Media\Media::test_filter_ajax_query_attachments_args_existing_tax_query
Trying to @cover or @use not existing method "\Google\Web_Stories\Media\Media::get_poster_tax_query".

3) Google\Web_Stories\Tests\Migrations\Add_Media_Source_Video_Optimization::test_migrate
Trying to @cover or @use not existing method "::migrate".


High Azure Network API calls by azure_sd_config causing API throttling

We are using azure_sd_config to discover VM's and VMSS.How ever for around 400 VM+VMSS Prometheus is making 60k+ API calls where as Azure has a limit of 10k API calls.This is causing instability for other application who are making azure API calls
Can we optimize the azure discovery codebase to reduce the number of calls ?
System Information :

We observed this issue in Prometheus 2.15.2 and then upgraded to 2.24.1 but still facing the same issue
Prometheus is Running on Kubernetes 1.16

Thank you, this is a recurring/known limitation of azure, but can you share a bit of your prometheus config so we can see if the discovery configs are used as expected?

Hi @roidelapluie : Here is my config
`  - job_name: prometheus
static_configs:
- targets:
- localhost:9090

azure_sd_configs:

authentication_method: ManagedIdentity
environment: AzurePublicCloud
port: 1934
refresh_interval: 300s
subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
authentication_method: ManagedIdentity
environment: AzurePublicCloud
port: 1935
refresh_interval: 300s
subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
authentication_method: ManagedIdentity
environment: AzurePublicCloud
port: 1936
refresh_interval: 300s
subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
authentication_method: ManagedIdentity
environment: AzurePublicCloud
port: 1937
refresh_interval: 300s
subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
job_name: haproxy_exporter
metrics_path: /metrics
relabel_configs:
action: keep
regex: westus2
separator: ;
source_labels:

__meta_azure_machine_location

action: keep
regex: .-haproxy.
separator: ;
source_labels:

__meta_azure_machine_tag_APPS

action: replace
regex: (.*)
replacement: $1
separator: ;
source_labels:

__meta_azure_machine_tag_BUSINESSUNIT
target_label: bu

Hi @roidelapluie : Here is my config

Thank you!! You configuration is correct but there is a way to reduce the calls to the Azure API.
Prometheus is able to reuse the same SD configs, reusing the same API calls for multiple jobs.
The condition for this is that the SD config is exactly the same. In you case, azure_sd_configs should be identical.
That would require you to align your configurations and reuse relabel_configs to change the port. Because the ports are currently different, Prometheus can not reuse them.
In your case, you will also need your configuratio in multiple jobs:
- job_name: prometheus
   static_configs:
   - targets:
     - localhost:9090
- job_name: job1
  azure_sd_configs:
   - authentication_method: ManagedIdentity
     environment: AzurePublicCloud
     port: 80
     refresh_interval: 300s
     subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
  relabel_configs:
  - source_labels: [__address__]
    target_label: __address__
    regex: '(.+):80'
    replacement: '$1:1234'
  - source_labels: [__address__]
    target_label: instance
- job_name: job2
  azure_sd_configs:
   - authentication_method: ManagedIdentity
     environment: AzurePublicCloud
     port: 80
     refresh_interval: 300s
     subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
  relabel_configs:
  - source_labels: [__address__]
    target_label: __address__
    regex: '(.+):80'
    replacement: '$1:9191'
  - source_labels: [__address__]
    target_label: instance

That would divide the calls to the azure API by 4.

Hi @roidelapluie : Here is my config

Thank you!! You configuration is correct but there is a way to reduce the calls to the Azure API.
Prometheus is able to reuse the same SD configs, reusing the same API calls for multiple jobs.
The condition for this is that the SD config is exactly the same. In you case you should make azure_sd_configs identical.
That would require you to align your configurations and reuse relabel_configs to change the port. [Because the ports are currently different in the sd configs, Prometheus can not reuse them (even if the rest is identical)].
In your case, you will also need to split your configuration in multiple jobs (I assume it is the case and you have provided a partial config):
- job_name: prometheus
   static_configs:
   - targets:
     - localhost:9090
- job_name: job1
  azure_sd_configs:
   - authentication_method: ManagedIdentity
     environment: AzurePublicCloud
     port: 80
     refresh_interval: 300s
     subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
  relabel_configs:
  - source_labels: [__meta_azure_machine_tag_privateip]
    regex: (.+)
    replacement: ${1}:9090
    target_label: __address__
- job_name: job2
  azure_sd_configs:
   - authentication_method: ManagedIdentity
     environment: AzurePublicCloud
     port: 80
     refresh_interval: 300s
     subscription_id: xxxxx-xxxxx-xxxxx-xxxx-xxxx
  relabel_configs:
    - source_labels: [__meta_azure_machine_tag_privateip]
      regex: (.+)
      replacement: ${1}:9100
      target_label: __address__

That would divide the calls to the azure API by 4.

Thanks @roidelapluie for the help.The above trick of reusing SD for multiple jobs drastically reduced the number of API calls.We also discovered we had similar 20 more jobs running and optimized those too.I guess this one of the feature which should be well documented in the official prometheus document.


set GroupID w/ cds init for java

Why?
cds-dk uses a default groupID: customer
As it is hard to define a real default it was decided to stick to customer as default, but show in capire how to specify the groupID with the cds init command.
As this is only meant to showcase this option, the proposal is to make the groupID specific to capire and not use any cap oder cds namespace for that.
On the other hand, our Java tutorial (starter scenario) will stick to the default, for the time being, to keep it simple.

Merging this on behalf of René


Remove inactive member from SECURITY.md

Should probably add @Reecepbcups too.

Should probably add @Reecepbcups too.

@JakeHartnell Reece#3370 if you all want me on it :)

I prefer this broad.


Re-indexing issue?

Problem Description:
I attempted to use the Colpali model for PDF document question answering, aiming to index the PDF file only once when the program starts and then reuse this index for retrieval without re-indexing each time. However, I encountered the following issue:
After indexing the PDF file when the program starts, using the RAG.search function for retrieval results in a RuntimeError: torch.cat(): expected a non-empty list of Tensors error.
Problem Analysis:
Through investigation, I found that the issue might stem from the following aspects:
Empty or corrupted index file: The RAG.index function may not have successfully indexed the PDF file, resulting in an empty or corrupted index file.
Empty query vector: The retriever_evaluator.evaluate_colbert function might fail to generate query vectors, leading to an empty scores_batch variable.
Mismatch between index file and query vectors: The index file and query vectors might be generated using different models or parameters, causing a mismatch that prevents successful retrieval.
Solution Attempts:
I attempted the following solutions, but none resolved the issue:
Check the index file: Ensured that the index file exists and its size is reasonable.
Check the query vector generation process: Examined the retriever_evaluator.evaluate_colbert function to ensure it correctly generates query vectors.
Check the consistency between the index file and query vectors: Ensured that the index file and query vectors use the same model and parameters.
logs:
Traceback (most recent call last):
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/gradio/queueing.py", line 536, in process_events
response = await route_utils.call_process_api(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/gradio/route_utils.py", line 322, in call_process_api
output = await app.get_blocks().process_api(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/gradio/blocks.py", line 1935, in process_api
result = await self.call_function(
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/gradio/blocks.py", line 1518, in call_function
prediction = await fn(*processed_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/gradio/utils.py", line 793, in async_wrapper
response = await f(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/mnt/sda/LLM/colpali/colpali_+_qwen2_vl.py", line 78, in process_query
relevant_results = RAG.search(query, k=3)
^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/byaldi/RAGModel.py", line 158, in search
return self.model.search(query, k, return_base64_results)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/byaldi/colpali.py", line 480, in search
scores = self._score(qs)
^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/byaldi/utils.py", line 9, in wrapper
result = func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/byaldi/colpali.py", line 448, in _score
scores = retriever_evaluator.evaluate(qs, self.indexed_embeddings)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/colpali_engine/trainer/retrieval_evaluator.py", line 13, in evaluate
scores = self.evaluate_colbert(qs, ps)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/ubuntu/anaconda3/envs/minicpm/lib/python3.11/site-packages/colpali_engine/trainer/retrieval_evaluator.py", line 62, in evaluate_colbert
scores_batch = torch.cat(scores_batch, dim=1).cpu()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: torch.cat(): expected a non-empty list of Tensors

Hello seems like a Byaldi issue maybe ? Rag.search() is a function in Byadi, if you want, you can use inference code in this repo to try out your documents without the wrapper and see if the bug persists, or you can raise an issue in Byaldi ?

Thank you. Yeah, I will raise a issue to Byaldi team.


Compatibility

All middlewares, compression handlers, view-engines marked as not compatible with AppExpress (0.2.4) would soon need to be marked compatible!
See - https://github.com/ItzNotABug/appexpress/pull/30.

An RC version is already on npm which works as expected on the dev channel i.e. https://appexpressdev.appwrite.global.
Version check - https://appexpressdev.appwrite.global/versions.

Completed by https://github.com/ItzNotABug/appexpress-essentials/pull/8 & https://github.com/ItzNotABug/appexpress-essentials/pull/9.


display(): options > "values" should require at least one field

It's currently possible to only specify a key (but no value) in the options. I can't think of a scenario where this kind of setting would result in a useful chart. Can you?

don't think you meant to label this "blog" ;)

There is a very basic use case for mapping -- just show all the locations of the data without coloring it in any way -- but I wouldn't be upset if value was required

That one makes sense, but then this exception case should be handled for other chart types. And you are right about blog vs bug. Thanks for fixing my defect. I have validated that the correct label is now applied :smiley:

Looks okay in 0.89


Can A PR Review Checklist be automated with GitHub actions?

This repository's issues are reserved for feature requests, and bug reports, and questions.
I'm submitting a ...

[ ] bug report
[x] feature request
[ ] question

Expected Behavior
When someone begins to review a PR, a review checklist should pop up. Ideally the new checklist (to be put in the guides by @Dayvis7 withing issue #184).
This is quite distinct from a pr_template, which is possible already within github.
Here is a relevant discussion by other folks who want this feature. https://github.community/t/pull-request-review-template/912/25
Actual Behavior
When someone begins to review a PR, a blank comment box pops up.
How can this issue be closed?
It seems like this could be possible with GitHub actions, and that other solutions might be in development by github. Please investigate the options and determine a path forward.
This issue can be closed with either:

confirmation that there is no way to do this, or
demonstration that this can be done with github actions or with a pr-review-template file of some kind.

I don't believe there's a way to do this. According to the community's thread, they're been "working on it" for two years now, and even though there are comments from June 2020 requesting the feature there's been no news of it. I've looked through GitHub actions relating to PR reviews, review comments, and review templates, yet there's nothing that can let you create a pre-filled comment for reviews.
My suggestion is to add a link to the PR template. This link will take you to the PR review template text which you can copy and paste. Maybe when you click the link, a text file opens with the review template. I think that could be an option, but for right now it doesn't appear to be possible to add a template to a PR Review.
Since the GitHub team hasn't announced it's possible and there are no Actions made for it, I say there's no way to do this yet.

My suggestion is to add a link to the PR template. This link will take you to the PR review template text which you can copy and paste.

This seems like a good idea. Go for it.

@katyhuff where can I find the file for the template?

Nevermind. Please disregard previous statement.


Autoscroll console

Seems weird that the console output does not automatically scroll down upon appending a new line. Is this intended behavior or can I work on a PR to add this feature?

closing since the site has been rewritten. please file at https://github.com/sveltejs/svelte.dev/issues/new if the issue persists

I have a PR pending over at the new repo for a contributor's guide


Error on Admin Role Validation

Using controller version 5.2.4 with APPGATE_CLIENT_VERSION=13
https://github.com/appgate/terraform-provider-appgatesdp/blob/5de6536fcccff7b8d6d52f185c159227d7772824/appgate/resource_appgate_administrative_role.go#L121
on terraform apply
│ Error: Could not update Administrative role Validation error Validation failed.
│  privileges[4].target RegisteredDevice was not one of [All, Appliance, Condition, CriteriaScript, Entitlement, AdministrativeRole, IdentityProvider, MfaProvider, IpPool, LocalUser, Policy, Site, DeviceScript, EntitlementScript, RingfenceRule, ApplianceCustomization, TrustedCertificate, OtpSeed, Fido2Device, TokenRecord, Blacklist, License, UserLicense, OnBoardedDevice, AllocatedIp, SessionInfo, AuditLog, AdminMessage, GlobalSetting, CaCertificate, File, FailedAuthentication, Stats]
│
│
│   with appgatesdp_administrative_role.REDACTED,
│   on ../admin_roles.tf line 1, in resource "appgatesdp_administrative_role" "REDACTED":
│  214: resource "appgatesdp_administrative_role" "REDACTED" {

when switching to OnBoardedDevice as provided by the server error message
╷
│ Error: type must be on of [All Appliance Condition CriteriaScript Entitlement AdministrativeRole IdentityProvider MfaProvider IpPool LocalUser Policy Site DeviceScript EntitlementScript RingfenceRule ApplianceCustomization OtpSeed TokenRecord Blacklist UserLicense RegisteredDevice AllocatedIp SessionInfo AuditLog AdminMessage GlobalSetting CaCertificate File FailedAuthentication], got OnBoardedDevice
│
│   with appgatesdp_administrative_role.REDACTED,
│   on admin_roles.tf line 1, in resource "appgatesdp_administrative_role" "REDACTED":
│   34:     target = "OnBoardedDevice"
│

Hi @stephengroat-dd , what type is used in combination here? Could you paste a full redacted terraform hcl example of the appgatesdp_administrative_role

Sure!
resource "appgatesdp_administrative_role" "test" {
  name  = "test"
  tags  = ["builtin"]

  privileges {
    type   = "View"
    target = "RegisteredDevice"

    scope {
      all = true
    }
  }

  privileges {
    type   = "Delete"
    target = "RegisteredDevice"

    scope {
      all = true
    }
  }
}

i'm switching RegisteredDevice and OnBoardedDevice. I can't plan with OnBoardedDevice since the validation step fails (OnBoardedDevice isn't part of the validation list), but I can't apply with RegisteredDevice.
I think that the provider needs to take the controller version into account during the validation phase

Closing this, as it has been updated in v0.5.7
Keep in mind that OnBoardedDevice is replaced in 5.3 and upstream to RegisteredDevice


Lag can cause Audio to get behind Video

If there is lag that causes the sound to skip it will desync. The video will end up ahead of the sound. This mostly only happens to me on linux in my vm. But I imagine someone with a weaker pc might have this issue on windows.
It basically needs to keep the Dynamic Sound Interface fed with audio samples. I was trying to increase the buffer size sent to it. But if I sent something other than 32 samples (approx 128 bytes) it caused sound glitches. I'm unsure if there is a way to detect if my audio is behind.
Possible Solutions:
Just increase the amount pre buffered?
FFMPEG audio processing in it's own thread?
Something faster than Mashal.Copy to copy from the byte* to byte[];? //when I switched to using Marshal.Copy to copy to a byte[] then feed it to Dynamic Sound Interface it was noticeably slower. But when I filled a memorystream with the entire uncompressed audio it would eat a lot of ram and that ram wouldn't always release correctly. Maybe there is a middleground.

Well I thought of one optimization while eating dinner. I bypassed a Marshal.Copy. By using an byte array to store the resampled sound data instead of a byte pointer. I didn't realize I could use fixed to point to the array.  fixed (byte* tmp = &_convertedData[0])                 {                         outSamples = ffmpeg.swr_convert(ResampleContext, &tmp, ResampleFrame->nb_samples, null, 0);                 }
So it skips the whole convert byte* to byte[] bit. Might be able to do more of that in some other spots.

On saturday night I did some small optimization. I minimized some coping to buffer. I put all the needed header data into a byte[]. So it's ready to go and just need to send it to ffcc. with the info of where the data is in the data file and the data file's name. On the step where it reads from the file it only reads enough to fill the buffer for ffmpeg and closes the file.

So we got threading last week. And increased the buffer to 100 chunks of samples. That seemed to help. I thought of another change to make when I'm home.
I can track how many samples are sent to the audio buffer, subtract the precached buffer, divide that by sample rate. And that gives us the seconds of audio sent to buffer. I think I can use that to detect if video got ahead of the audio. And basically pause the video till audio catches up.
There might be a need to check number of chunks in the buffer. But I don't know if all chunks will always be 32 samples.
If you resize the window during playback it's an easy way to force this to happen.


Update to the latest Emscripten

Currently the latest tag is sdk-tag-1.38.32-64bit. Emscripten is up to 1.38.42.
Is there any plan to update this image for the latest Emscripten?

Hi @VirtualTim,
Thank you for this report, I'm aware about issue and will work to fix it.
Due vacation I will handle this in ~2w. Cheers!

No worries. I appreciate the ETA.

Is #45 blocking the update to the latest version? (1.38.41)
Let me know if I can help with investigation or testing.

Hi @boardend indeed, those two issues are the same. Due personal reasons my time is very limited. Nonetheless, I'm dedicated in the remaining fraction of time to fix entire docker image pipeline.
Thank you for your offer.

So how's the progress going on this?
Anything I can do to help?

Hi, @VirtualTim I have a working solution with fastcomp compiler, I have one element which is too hacky to be published, as it points on some arbitrary version of binaryen, once this is solved it will be done.

Hi @here,
Fixed all related problems, sorry that it took so long, it was a nice puzzle to solve. At the moment I've started to rebuild all images 1.38.34 upwards. Will take a couple of hours until they land in registry,
At the moment you can check 1.38.40 as I was using it as a test case.
➜ docker run --rm -it trzeci/emscripten:sdk-tag-1.38.40-64bit emcc --version
Unable to find image 'trzeci/emscripten:sdk-tag-1.38.40-64bit' locally
sdk-tag-1.38.40-64bit: Pulling from trzeci/emscripten
9fc222b64b0a: ...
Digest: sha256:839c186ec629fcec7ebb5ad50b9a503415707d98cbdaa5afaf4d7e15555f1e7b
Status: Downloaded newer image for trzeci/emscripten:sdk-tag-1.38.40-64bit
emcc (Emscripten gcc/clang-like replacement) 1.38.40 ((unknown revision))
Copyright (C) 2014 the Emscripten authors (see AUTHORS.txt)
This is free and open source software under the MIT license.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Will keep this issue open for a couple of days to make sure everything works for you.
Cheers,

Awesome. Thanks Piotr. I should get a chance to test these out Monday.

Should be done, please re-open in case of any problems,
Cheers!

I switched over my builds to use 1.38.43 on Monday, and it all seems good.


How to obtain rendered images of MPM simulation results with textures?

Thanks for your work! We noticed that the provided synthetic dataset only contains point cloud files. We want to create our own dataset, but we don't know how to use Blender to render MPM simulated point cloud files, how to obtain and bind textures. Can you provide us with some references and suggestions?

Hi, you can simply do surface reconstruction to obtain meshes from point cloud.

Hi, you can simply do surface reconstruction to obtain meshes from point cloud.

I am still confused. If we reconstruct and render the surface from a point cloud, how can we obtain its color and texture? Is it from the GS kernel? But the Gaussian kernel corresponds to the point cloud reconstructed from the original image, how to calculate the correspondence with the mesh?

During surface reconstruction, you can also compute vertices' color for mesh (such as use the nearest point's color) to get a .ply file. GS was not used when generating the data.

Assuming I have a mesh based model at the beginning, I perform volume sampling on it to obtain particles, and then use particles for MPM simulation. At each time step, I can reconstruct a mesh from the deformed particles and obtain the color of the mesh from the particles. Therefore, when I perform volume sampling at the beginning, I must assign the color to each particle at the same time, such as obtaining it from the texture image corresponding to this particle. Is my understanding correct?

Yes, that's right.


Only update hover execution state if cell is not active.

Currently, prompting for user input with .NET Interactive breaks when mousing over the cell because we always re-focus on the cell when changing the execution state on hover. So if you click run on a cell that prompts for input and mouse anywhere off of the run button, then the input box closes immediately. The hover state is only used to determine if the cell's Run button should be visible, so my fix here is to only fire the hover state change if the cell isn't active, since the Run button is always visible while active anyway.
This PR fixes #18980

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
62 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.003%) to 42.26%

Files with Coverage Reduction
New Missed Lines
%

extensions/machine-learning/src/common/processService.ts
2
81.48%

src/sql/workbench/services/notebook/browser/models/cell.ts
60
72.14%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
27678

Relevant Lines:
61226

💛  - Coveralls

If you plan on bringing this to ask mode don't link it to auto-close here. We want it to stay open until it's ported over to the release branch.

CI failures are from the manage packages test again, which is unrelated to these changes. Going ahead with the merge.


Use higher resolution land-sea mask

As added in https://github.com/CliMA/ClimaArtifacts/pull/49
30 arcseconds is already pretty high resolution
Closes #936
EDIT: This is not ready yet

We can either try using this and see if there is a problem (with our resolution I guess probably not), or use the one @akshaysridhar referenced here, which is already better than what we have now.

We can either try using this and see if there is a problem (with our resolution I guess probably not), or use the one @akshaysridhar referenced here, which is already better than what we have now.

Yeah this reference is in part based on the current AMIP documentation: (we reference that 360x180 dataset, but use a coarser 64x64 one in current main)

We can either try using this and see if there is a problem (with our resolution I guess probably not), or use the one @akshaysridhar referenced here, which is already better than what we have now.

Yeah this reference is in part based on the current AMIP documentation: (we reference that 360x180 dataset, but use a coarser 64x64 one in current main). Thanks @Sbozzolo for updating this to use the latest artifacts.

This PR uses the topography artifact. Could you either check if this is ok, or add the higher resolution land sea mask to ClimaArtifact and update it in the coupler?

I updated this PR to remove ArtifactWrappers and all the artifact infrastructure.
I think this can only be merged after https://github.com/CliMA/ClimaAtmos.jl/pull/3378


Merge remote-tracking branch 'chatterino2' into feature/pronouns

We are now on c19fd0d33239031cb1290a7a60dd5036502cb62a - Notable changes from upstream:
Fixed splits losing filters when closing and reopening them https://github.com/Chatterino/chatterino2/pull/3351
Add "clear cache" button to settings https://github.com/Chatterino/chatterino2/pull/3277
Add copy button for both login & display name inside the user info popout https://github.com/Chatterino/chatterino2/pull/3335
Fixed underlying text from disabled emotes not being colorized after using /me https://github.com/Chatterino/chatterino2/pull/3333
Parse usernames out of /mods and /vips messages: https://github.com/Chatterino/chatterino2/pull/3187
Add chatter count to viewer list popout: https://github.com/Chatterino/chatterino2/pull/3261
Add flags.first_message filter variable: https://github.com/Chatterino/chatterino2/pull/3292
Subcategories in settings are now searchable https://github.com/Chatterino/chatterino2/pull/3157
2021-11-07 edit: I deleted it I don't remember what it said
2021-10-31 edit: We are now on: 4b903d7fcf31156db08f2cc6a566d234c20f3c77 Notable changes from upstream:
2021-10-30 edit: We are now on: b4b745024c1290977d5236f78f81429a8f8eb028 - Notable changes from upstream: nothing special since I opened this PR
We are now on: 4a90f10df905cb03cb7c8673dabc8e72b82ed100 - Notable changes from upstream:
Pull request checklist:

[X] CHANGELOG.md was updated, if applicable

Description
I probably wont end up doing weekly updates unless there's actually important things, (edit: xd) // this was obviously a lie
but for this the actual important change is https://github.com/Chatterino/chatterino2/pull/3293
Currently 'First Message` highlights cannot be disabled, and many of the people who would enjoy having the pronouns extension, would hate not being able to disable this, so I've held off on mentioning it until this could be committed.

I'm going to close this PR and the other one for a couple reasons.
The main one being due to a discussion in https://github.com/Chatterino/chatterino2/pull/3346 - I've realized how much of a pain this extension is when sending a single request for EVERY SINGLE user. It seems like it would be a billion times easier to support in Chatterino if the information was sent in json form (similar to FFZ and Chatterino badges.)
The minor one being the people who would enjoy this, also would be confused/frustrated with having to download a nightly build and while waiting for chatters to load, would assume it's not working because of how long it takes  - and the restart/refresh after the loading has finished. So they would be liable to just remove it instead of waiting.
As a note: if/when you intend to update this in the future, be careful of https://github.com/Chatterino/chatterino2/pull/3335 as it breaks pronouns sticking in the usercard.


Crashes analytics

if called setOnTabSelectListener app's analytics crashes, be it amplitude, firebase, fabric..

Ok. Problem was in butterknife. sorry.


cannot import name '_is_pairwise' from 'sklearn.base'

What happened: I am trying to use dask ml for my project, I have one scheduler and one worker connected. There is the error: cannot import name '_is_pairwise' from 'sklearn.base'
I have checked that I have the same versions of Dask, Dask ml, Python, scikit-learn for both the scheduler and the worker. I hope that the Dask community can help me.
Thank you!
What you expected to happen: no such error appears, the code can be processed normally
Minimal Complete Verifiable Example:
My code:
import dask_ml.cluster

users_ts_2d_dask = users_ts_2d_dask.persist()

kmeans = dask_ml.cluster.KMeans(n_clusters=3, init_max_iter=1, oversampling_factor=8)

kmeans.fit(users_ts_2d_dask)

Error:

---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
Input In [23], in <cell line: 1>()
----> 1 import dask_ml.cluster
      3 users_ts_2d_dask = users_ts_2d_dask.persist()
      5 kmeans = dask_ml.cluster.KMeans(n_clusters=3, init_max_iter=1, oversampling_factor=8)

File ~/anaconda3/lib/python3.9/site-packages/dask_ml/__init__.py:4, in <module>
      1 from pkg_resources import DistributionNotFound, get_distribution
      3 # Ensure we always register tokenizers
----> 4 from dask_ml.model_selection import _normalize
      6 __all__ = []
      8 try:

File ~/anaconda3/lib/python3.9/site-packages/dask_ml/model_selection/__init__.py:8, in <module>
      6 from ._hyperband import HyperbandSearchCV
      7 from ._incremental import IncrementalSearchCV, InverseDecaySearchCV
----> 8 from ._search import GridSearchCV, RandomizedSearchCV, check_cv, compute_n_splits
      9 from ._split import KFold, ShuffleSplit, train_test_split
     10 from ._successive_halving import SuccessiveHalvingSearchCV

File ~/anaconda3/lib/python3.9/site-packages/dask_ml/model_selection/_search.py:18, in <module>
     16 from dask.utils import derived_from
     17 from sklearn import model_selection
---> 18 from sklearn.base import (
     19     BaseEstimator,
     20     MetaEstimatorMixin,
     21     _is_pairwise,
     22     clone,
     23     is_classifier,
     24 )
     25 from sklearn.exceptions import NotFittedError
     26 from sklearn.model_selection._search import BaseSearchCV

ImportError: cannot import name '_is_pairwise' from 'sklearn.base' (/home/cs201503/anaconda3/lib/python3.9/site-packages/sklearn/base.py)

Anything else we need to know?:
Environment:

Dask version: 2022.5.0
Dask ml version: 2022.1.22
Python version: 3.9.7
Operating System: Linux
Install method (conda, pip, source): conda

Cluster Dump State:

Thanks for reporting this. I'll make a release today with the fix. See https://github.com/dask/dask-ml/issues/928 & linked issues for more.


Catalog: mysql_send_long_data() is longer than 'max_long_data_size' bytes

Rancher versions:
rancher/server:v1.6.4
rancher/agent:v1.2.5
Infrastructure Stack versions:
healthcheck:v0.3.1
ipsec:v0.11.3
network-services:v0.7.4
scheduler:v0.8.2
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Client:
Version:      17.03.0-ce
API version:  1.26
Go version:   go1.7.5
Git commit:   60ccb22
Built:        Thu Feb 23 11:02:43 2017
OS/Arch:      linux/amd64
Server:
Version:      17.03.0-ce
API version:  1.26 (minimum version 1.12)
Go version:   go1.7.5
Git commit:   60ccb22
Built:        Thu Feb 23 11:02:43 2017
OS/Arch:      linux/amd64
Experimental: false
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Ubuntu 16.04.2 LTS
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
KVM, Bare-metal
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
HA rancher with external DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Activate/refresh catalogs
Results:
(Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes)
[2017-07-16 19:02:27]
ERRO[0013] Failed to perform catalog refresh: community: Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes

(Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes)
[2017-07-16 19:02:33]
ERRO[0020] Failed to perform catalog refresh: community: Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes

I found the above errors every ~6 seconds in "docker logs" and cattle-debug.log. The error appears, I think, after the upgrade from 1.6.2 to 1.6.4. I tried to increase the max_long_data_size from 1048576 to 2097152 but the error appears until I disable the catalogs in admin settings.
mysql> show variables like 'max_long_data_size'\G
Variable_name: max_long_data_size
        Value: 2097152

How can I find out how big is mysql_send_long_data?

Are you sure this started since upgrading to 1.6.4? Can you tell me what database server and what database server version you are running? Try to up max_allowed_packet in the meantime

No I am not totally sure. But after the upgrade to 1.6.2 I did not saw this error.
I use the image mysql:5.5 as database.
I tried to increase the max_allowed_packet but the error still there:
mysql> set global max_allowed_packet=2097152;
root@rancher:~$ docker logs -f --tail=100 rancher_server_1
..
..
(Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes)
[2017-07-17 12:38:59]
ERRO[63408] Failed to perform catalog refresh: community: Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes

(Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes)
[2017-07-17 12:39:21]

(Error 1105: Parameter of prepared statement which is set through mysql_send_long_data() is longer than 'max_long_data_size' bytes)
[2017-07-17 12:39:41]
..
..

Can you try setting it to 16777216 or higher?

Its a bit strange... I set max_long_data_size=16777216 and max_allowed_packet to default and the error not appeared anymore.
Then I removed all the settings so its the default settings of the image, and the error also is gone.
After every change I recreated the database-container.
Maybe the errors is gone because the server could update the requested data once?
I let the container run the next hours to see if the error comes back. I will give feedback.

This should be detectable and/or documented, please report back and thanks for testing.

I recently ran into this error, I had to up the max_allowed_packet on my database server.

been receiving the same error in our HA environment we are using amazon rds with maria db 10.0
this might be some helpful information
max_allowed_packet |   | 1024-1073741824 | true | engine-default | dynamic | integer | This value by default is small, to catch large (possibly incorrect) packets. Must be increased if using large BLOB columns or long strings. As big as largest BLOB.

You'll need to increase the max_allowed_packet. I increased mine to 16777216 to resolve the problem.

go-mysql-driver defaults to 16777216 (https://github.com/go-sql-driver/mysql/blob/0b58b37b664c21f3010e836f1b931e1d0b0b0685/const.go#L13)
Defaults on max_allowed_packet, based on Dockerhub images and RDS

Type
Version
max_allowed_packet

mysql
5.5
1048576

mysql
5.6
4194304

mysql
5.7
4194304

mysql
8.0
4194304

mariadb
5.5
16777216

mariadb
10.0
16777216

RDS
5.7
4194304

If using a compose file, you can pass in the config via "command":
version: 2
services:
    db:
        image: mysql:5.5.53
        environment:
            MYSQL_DATABASE: ${MYSQL_DATABASE}
            MYSQL_ROOT_PASSWORD: ${MYSQL_ROOT_PASSWORD}
            MYSQL_USER: ${MYSQL_USER}
            MYSQL_PASSWORD: ${MYSQL_PASSWORD}
        command:
            - --max_allowed_packet=16M

We are updating our docs to recommend using a max_packet_size of 32M or so.
The record for the catalog has (2) Medium blob fields each can be a max of 16MB. Right now, it appears a 4MB max_packet_size will work for the existing entries. Still  the possibility of this error exists even in those setups with 16M.


⚠ Bofrost.at has degraded performance

In 892031f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2252 ms

Resolved: Bofrost.at performance has improved in 2d10ce6.


The key to becoming a future-oriented flutter

Use case

Proposal
How to make it intuitive
You can write your code in gui.

@wikiworldlang It's not in the flutter roadmap. Closing, as this is not an issue report. If you insist, please make a detailed proposal about the change, otherwise it's not helpful at all.
If you want to share your idea, please check https://flutter.dev/community for support options.


[Glitch] Server slows down somehow when mod is installed.

Legacy4J Version
1.21-1.7.2
Mod Loader Version
Fabric loader 0.16.0
Bug/Glitch Occurrence Context
When the mod is installed on the server, the server itself begins to lag/ping a lot. But this for people who just a too away from the host. (Me from Russia to Belarussia host.)
Yeah, it's lag with Radmin Vpn. But without mod installed - everything so much better and fine.
Crash Report
No response
What happened wrong?
I made a 4 minute video to demonstrate this issue!
(Plz watch if you can.)

Could you help me with this issue?
If you can, test version 1.6.7 of the mod to see if this happens (in this case you would have to make a backup of the server in 1.20.6), so I would already know the solution for this

So. I create a new fabric server on 1.20.6. Put l4j, fabric api, and that depend mod + wing's modpack.
Started with server.properties from 1.21 server.
I log in and it seems to situation is better, but i still have ping spikes while playing (mostly when i flying with elytra). I can interact with world immediately and i didn't get shulker lag when just opening them.
There a 3 minute non-cut video of demonstration.
We tried it without mod, and situation is just better, i don't get 200-300 ping when just flying/playing.

Could you show footage of you flying in vanilla in the same way?
I think it's very unlikely to have anything related to Legacy4J in versions <=1.6.7, since there is nothing in these versions that could cause server slowdowns

Hmm. Maybe you right. Because in one moment, without mod, when i teleported to 30000000 cords i get 700~ ping. (It's 101% is radmin now.
But there i just flying without mod.

Fixed in version 1.7.3, thanks for reporting.


Dynamically extend fd table as needed

Now, all tasks can have hold at most CONFIG_NFILE_DESCRIPTORS file handles. But the complexity for each task is totally different, so it's better to extend the file table size at runtime to match the different requirement.

From top-level TODO list (under " UNIFIED DESCRIPTOR REPRESENTATION"):
           Another example of how the current implementation limits
           functionality:  I recently started to implement of the FILEMAX
           (using pctl() instead sysctl()).  My objective was to be able
           to control the number of available file descriptors on a task-
           by-task basis.  The complexity due to the partitioning of
           descriptor space into a range for file descriptors and a range
           for socket descriptors made this feature nearly impossible to
           implement.

Are you considering implementation of the FILEMAX sysctl() call?  That is one of the non-standard OS interfaces that at least has a precedent.  The Linux call is global, I believe.  prctl() would be a better location for the per-task control of the number of file descriptors.
What happens when a task is spawned?  Does the FILEMAX size propagate too?  I think that it has to be inherited or else you could not propagate the open file descriptors.

We plan to address this issue first: https://github.com/apache/incubator-nuttx/issues/2833, the rough idea is that:

Keep the kernel psock API as before
Implement a new file_operation g_sock_fileops
socket api allocate handle from filelist and associate with g_sock_fileops
Common API(e.g. dup, close, ioctl, read, write) is handled by new g_sock_fileops
g_sock_fileops forward common API to the related psock API
Remove socklist from task_group and NSOCKET_DESCRIPTORS option

so, we get an unified representation of file and socket.

It would be great to support this as it is very common for me to start building app my project logic when it eventually fails to due opening to many FDs. It is the kind of setting that requires guesswork, which is not ideal.
Do you think it could affect performance? I imagine it would require traversing a list for every file operation. Or would you reallocate an array?

It would be great to support this as it is very common for me to start building app my project logic when it eventually fails to due opening to many FDs. It is the kind of setting that requires guesswork, which is not ideal.
Do you think it could affect performance? I imagine it would require traversing a list for every file operation. Or would you reallocate an array?

Yes, fd->file conversion become to O(n) operation if we switch to list, but the reallocation array isn't perfect too, because:

All file pointers will be invalid after reallocation, we has to hold the lock in the entire fs call.
It's very hard to find the big continuous free memory to hold the fd array

so we propose a two layer array structure(like STL deque):

We can get both benefit from list and array:

Convert fd to file in O(1)
Allocated file struct never become invalid
Don't require the bigger and bigger continuous free memory
Avoid the housekeeping overhead of list(each file ~12 bytes)


Do not re-administer non-parameterized questions in a survey that includes parameterized and non-parameterized questions

If only some of my survey questions have scenarios, the non-parameterized questions do not need to be repeated but they currently are. Example code to reproduce this:
from edsl import QuestionMultipleChoice, QuestionFreeText
from edsl import Agent, Scenario, Survey, Model

q1 = QuestionMultipleChoice(
    question_name = "q1",
    question_text = "What size {{ item }} are you wearing?",
    question_options = ["Small", "Medium", "Large"]
)
q2 = QuestionFreeText(
    question_name = "q2",
    question_text = "Who will win the election?"
)
items = ["shirt", "gloves"]
scenarios = [Scenario({"item":i}) for i in items]

agents = [
    Agent(traits = {"persona":"You are a retired baby."}),
    Agent(traits = {"persona":"You are a youthful senior."})
]
model = Model('gpt-4-1106-preview')
survey = Survey([q1,q2])

results = survey.by(scenarios).by(agents).by(model).run()
results.select("persona", "item", "q1", "q2").print()

See multiple responses to the same question by the same agent--should be avoided as unnecessary and potentially confusing:

Given that the cache will make answering these questions super-fast and it would add a lot of complexity to check, I'm going to not build this.


The npm version in docker is too low

I got an error from npm when deploying hexo
this is the error log：
https://github.com/byg-software/byg-software-Official-website/runs/8010043706?check_suite_focus=true
After looking through the sources, I think the cause of the problem may be the version of npm
Because the npm version I am using is 7.5.2

To make the tool work for most people, I followed the version requirements on the official website.
If you need to use a lower version, you can consider fork project to modify the docker base image to the lower node version, or it is recommended to upgrade the local node environment to a version that meets the official requirements.
FROM node:7-slim

# other statement...


cmd/go: list of .Deps shows a mix golang.org and golang_org items

What version of Go are you using (go version)?
go 1.10.2
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
darwin amd64
What did you do?
I'm running this command to find dependencies of my program that are located in vendor/:
go list -f '{{join .Deps "\n"}}' ./... | grep -Eo 'vendor/.*' | sort | uniq

(Output of this command is listed below.)
In this list, there's a mix of both vendor/golang_org/... (note the underscore), and vendor/golang.org/.... But in reality, there's only vendor/golang.org/... on my filesystem.
vendor/golang.org/x/text/language
vendor/golang.org/x/text/runes
vendor/golang.org/x/text/secure/bidirule
vendor/golang.org/x/text/transform
vendor/golang.org/x/text/unicode/bidi
vendor/golang.org/x/text/unicode/norm
vendor/golang.org/x/time/rate
vendor/golang_org/x/crypto/chacha20poly1305
vendor/golang_org/x/crypto/chacha20poly1305/internal/chacha20
vendor/golang_org/x/crypto/cryptobyte
vendor/golang_org/x/crypto/cryptobyte/asn1
vendor/golang_org/x/crypto/curve25519
vendor/golang_org/x/crypto/poly1305
vendor/golang_org/x/net/http2/hpack
vendor/golang_org/x/net/idna
vendor/golang_org/x/net/lex/httplex
vendor/golang_org/x/net/proxy
vendor/golang_org/x/net/route
vendor/golang_org/x/text/secure/bidirule
vendor/golang_org/x/text/transform
vendor/golang_org/x/text/unicode/bidi
vendor/golang_org/x/text/unicode/norm

Looks like some special treatment of string golang_org/x/ in the Go source tree:

https://github.com/golang/go/blob/4258b43afe3d3c48010aeeae7c438e5a88c6b5ab/src/cmd/api/goapi.go#L440
https://github.com/golang/tools/blob/a5b4c53f6e8bdcafa95a94671bf2d1203365858b/cmd/bundle/main.go#L109

I can't tell why it is taking effect but it is confusing, and probably should not happen.
What did you expect to see?

vendor/golang_org/... items in the {{.Deps}} should be listed as vendor/golang.org/... which is how they appear in the filesystem.

What did you see instead?

Dependencies listed as vendor/golang_org/... instead ofvendor/golang.org/... for some reason I cannot figure out.

The references to vendor/golang_org are coming from the standard library, where the name golang_org is used because of #16333.  Since you are asking for .Deps of all packages, which includes the standard library, I think this is correct behavior and that there is no bug here.
The reference to golang_org in cmd/api is irrelevant here.  That code is not being run by cmd/go.

That makes sense, we can close this. Feel free to keep it open if you'd like to use this for tracking.


Dependency error when attempt to build Hudi from git source ..

Hi,
Followed the steps to get the hudi soruce from git and build it and getting the build failure error as mentioned below:
Please help.
[INFO] Hudi ............................................... SUCCESS [  7.032 s]
[INFO] hudi-common ........................................ SUCCESS [ 31.071 s]
[INFO] hudi-timeline-service .............................. SUCCESS [  4.266 s]
[INFO] hudi-client ........................................ SUCCESS [  0.173 s]
[INFO] hudi-client-common ................................. SUCCESS [ 19.735 s]
[INFO] hudi-hadoop-mr ..................................... FAILURE [  1.005 s]
[INFO] hudi-spark-client .................................. SKIPPED
[INFO] hudi-sync-common ................................... SKIPPED
[INFO] hudi-hive-sync ..................................... SKIPPED
[INFO] hudi-spark-datasource .............................. SKIPPED
[INFO] hudi-spark-common .................................. SKIPPED
[INFO] hudi-spark2_2.12 ................................... SKIPPED
[INFO] hudi-spark3_2.12 ................................... SKIPPED
[INFO] hudi-spark_2.12 .................................... SKIPPED
[INFO] hudi-utilities_2.12 ................................ SKIPPED
[INFO] hudi-utilities-bundle_2.12 ......................... SKIPPED
[INFO] hudi-cli ........................................... SKIPPED
[INFO] hudi-java-client ................................... SKIPPED
[INFO] hudi-flink-client .................................. SKIPPED
[INFO] hudi-dla-sync ...................................... SKIPPED
[INFO] hudi-sync .......................................... SKIPPED
[INFO] hudi-hadoop-mr-bundle .............................. SKIPPED
[INFO] hudi-hive-sync-bundle .............................. SKIPPED
[INFO] hudi-spark-bundle_2.12 ............................. SKIPPED
[INFO] hudi-presto-bundle ................................. SKIPPED
[INFO] hudi-timeline-server-bundle ........................ SKIPPED
[INFO] hudi-hadoop-docker ................................. SKIPPED
[INFO] hudi-hadoop-base-docker ............................ SKIPPED
[INFO] hudi-hadoop-namenode-docker ........................ SKIPPED
[INFO] hudi-hadoop-datanode-docker ........................ SKIPPED
[INFO] hudi-hadoop-history-docker ......................... SKIPPED
[INFO] hudi-hadoop-hive-docker ............................ SKIPPED
[INFO] hudi-hadoop-sparkbase-docker ....................... SKIPPED
[INFO] hudi-hadoop-sparkmaster-docker ..................... SKIPPED
[INFO] hudi-hadoop-sparkworker-docker ..................... SKIPPED
[INFO] hudi-hadoop-sparkadhoc-docker ...................... SKIPPED
[INFO] hudi-hadoop-presto-docker .......................... SKIPPED
[INFO] hudi-integ-test .................................... SKIPPED
[INFO] hudi-integ-test-bundle ............................. SKIPPED
[INFO] hudi-examples ...................................... SKIPPED
[INFO] hudi-flink_2.12 .................................... SKIPPED
[INFO] hudi-flink-bundle_2.12 ............................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  56.107 s
[INFO] Finished at: 2021-05-29T14:04:47Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project hudi-hadoop-mr: Could not resolve dependencies for project org.apache.hudi
:hudi-hadoop-mr:jar:0.8.0: Failed to collect dependencies at org.apache.hive:hive-exec:jar:core:2.3.1 -> org.apache.
calcite:calcite-core:jar:1.10.0 -> org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Failed to read artifac
t descriptor for org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Could not transfer artifact org.pentaho:
pentaho-aggdesigner-algorithm:pom:5.1.5-jhyde from/to maven-default-http-blocker (http://<IP_ADDRESS>/): Blocked mirror f
or repositories: [datanucleus (http://www.datanucleus.org/downloads/maven2, default, releases), glassfish-repository
(http://maven.glassfish.org/content/groups/glassfish, default, disabled), glassfish-repo-archive (http://maven.glas
sfish.org/content/groups/glassfish, default, disabled), apache.snapshots (http://repository.apache.org/snapshots, de
fault, snapshots), central (http://repo.maven.apache.org/maven2, default, releases), conjars (http://conjars.org/rep
o, default, releases+snapshots)] -> [Help 1]
[ERROR]

Hi,
Even after cloning the https://github.com/vinothchandar/hudi/tree/remove-spring-deps and attempt to mvn build, getting the same error.
Please help.
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project hudi-hadoop-mr: Could not resolve dependencies for project org.apache.hudi
:hudi-hadoop-mr:jar:0.9.0-SNAPSHOT: Failed to collect dependencies at org.apache.hive:hive-exec:jar:core:2.3.1 -> or
g.apache.calcite:calcite-core:jar:1.10.0 -> org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Failed to rea
d artifact descriptor for org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Could not transfer artifact org
.pentaho:pentaho-aggdesigner-algorithm:pom:5.1.5-jhyde from/to maven-default-http-blocker (http://<IP_ADDRESS>/): Blocked
mirror for repositories: [datanucleus (http://www.datanucleus.org/downloads/maven2, default, releases), glassfish-r
epository (http://maven.glassfish.org/content/groups/glassfish, default, disabled), glassfish-repo-archive (http://m
aven.glassfish.org/content/groups/glassfish, default, disabled), apache.snapshots (http://repository.apache.org/snap
shots, default, snapshots), central (http://repo.maven.apache.org/maven2, default, releases), conjars (http://conjar
s.org/repo, default, releases+snapshots)] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :hudi-hadoop-mr

I met the same question and how I solved this question is to change the settings of maven by changing the mirrors config.But you may still have some  jar files which could not be supported.I downloaded them by myself.

@Doorwood Thanks for chiming in. Have you found any fixes that can address this permanently ?

Hi, The mentioned error is observed in hudi-hadoop-mr module. We can download the jar for hudi-hadoop-mr.
To resolve this error where to keep the jar and what config changes need to be made for successfully mvn build?
Anyway having perm fix or/and hudi-cli binary download would help.

settings.md
You can try this maven configuration file.

Hi, Tried to mvn build using the settings.xml and getting the same error as pasted below.
[INFO] Building hudi-hadoop-mr 0.8.0                                     [6/42]
[INFO] --------------------------------[ jar ]---------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Hudi 0.8.0:
[INFO]
[INFO] Hudi ............................................... SUCCESS [02:09 min]
[INFO] hudi-common ........................................ SUCCESS [01:00 min]
[INFO] hudi-timeline-service .............................. SUCCESS [  5.223 s]
[INFO] hudi-client ........................................ SUCCESS [  0.229 s]
[INFO] hudi-client-common ................................. SUCCESS [ 31.635 s]
[INFO] hudi-hadoop-mr ..................................... FAILURE [  0.327 s]
[INFO] hudi-spark-client .................................. SKIPPED
[INFO] hudi-sync-common ................................... SKIPPED
[INFO] hudi-hive-sync ..................................... SKIPPED
-----------do ----------------------------------------------skipped
[INFO] hudi-flink_2.12 .................................... SKIPPED
[INFO] hudi-flink-bundle_2.12 ............................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  03:48 min
[INFO] Finished at: 2021-06-02T10:53:56Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project hudi-hadoop-mr: Could not resolve dependencies for project org.apache.hudi
:hudi-hadoop-mr:jar:0.8.0: Failed to collect dependencies at org.apache.hive:hive-exec:jar:core:2.3.1 -> org.apache.
calcite:calcite-core:jar:1.10.0 -> org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Failed to read artifac
t descriptor for org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Could not transfer artifact org.pentaho:
pentaho-aggdesigner-algorithm:pom:5.1.5-jhyde from/to maven-default-http-blocker (http://<IP_ADDRESS>/): Blocked mirror f
or repositories: [datanucleus (http://www.datanucleus.org/downloads/maven2, default, releases), glassfish-repository
(http://maven.glassfish.org/content/groups/glassfish, default, disabled), glassfish-repo-archive (http://maven.glas
sfish.org/content/groups/glassfish, default, disabled), apache.snapshots (http://repository.apache.org/snapshots, de
fault, snapshots), conjars (http://conjars.org/repo, default, releases+snapshots)] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :hudi-hadoop-mr

@arun990 A fix for this was landed today -> https://github.com/apache/hudi/pull/3034 Can you check if the master works for you now ?

Hi, checked it and the build was completed without the reported error.
Thank you.
Closing the issue.


Bug of format on type

Using coc.nvim as LSP client with coc-rust-analyzer extension and enable format on type:  "coc.preferences.formatOnType": true
main.rs
use std::io;

use rand::Rng;

fn main() {
    println!("The secret_number is: |")

    let mut guess = String::new();
}

| is cursor position, type { then the code will become:
use std::io;

use rand::Rng;

fn main() {
    println!("The secret_number is: {}")

    let }mut guess = String::new();
}

It will insert } in the line   let mut guess = String::new(); =>  let }mut guess = String::new();
If change the secret_number to other words like number and it works fine.
log:
[Trace - 下午9:39:16] Sending request 'textDocument/onTypeFormatting - (8)'.
Params: {
    "textDocument": {
        "uri": "file:///Users/aioiyuuko/develop/test/hello-rust/src/main.rs",
        "version": 4
    },
    "position": {
        "line": 5,
        "character": 37
    },
    "ch": "{",
    "options": {
        "tabSize": 4,
        "insertSpaces": true
    }
}

[Trace - 下午9:39:16] Received response 'textDocument/onTypeFormatting - (8)' in 4ms.
Result: [
    {
        "range": {
            "start": {
                "line": 7,
                "character": 8
            },
            "end": {
                "line": 7,
                "character": 8
            }
        },
        "newText": "}"
    }
]

Seems like only the length of the string like The secret_number is:  will have the issue, so change to same length of dddddddddddddddddddddd will have the issue too.

This doesn't seem to be macro related, just ran into this as well.
fn check_with(ra_fixture: &str, expect: Expect) {
    let base = r#"
enum E { T(), R$0, C }
use self::E::X;
const Z: E = E::C;
mod m {}

static FOO: E = E::C;
struct Bar { f: u32 }
macro_rules! foo {}
"#;
    let actual = completion_list(&format!("{}\n{}", base, ra_fixture));
    expect.assert_eq(&actual)
}

Typing { at $0 results in the static FOO: E = E::C; line turning into s}tatic FOO: E = E::C;.


Special variable for current window in statusline function

It is currently not easy to have a global statusline function, that knows about if the window it is handling (called for) is the active one.
A workaround is tracking WinEnter/WinLeave (and BufWinEnter) events, and setting up a new function for windows then, but that is clumsy.
It would be much easier, if there was a special variable (e.g. v:statusline_winid) that would provide the winid of the window this statusline is for, allowing to just do let active = winid() == v:statusline_winid then.
Note that using an expression has limitations (e.g. https://github.com/vim/vim/issues/2829), and does not allow for returning highlight groups (%#Foo# - which get used as text).
(There is g:actual_curbuf already, but it is only available for expressions (https://github.com/vim/vim/issues/3299) (and also not useful when you want to know if the window is the active one).)

It is currently not easy to have a global statusline function, that knows
about if the window it is handling (called for) is the active one.
A workaround is tracking WinEnter/WinLeave (and BufWinEnter) events, and
setting up a new function for windows then, but that is clumsy.

Why not use a local 'statusline' option value?  Hmm, is the problem that you
don't know if the window that statusline belongs to is the active window?

It would be much easier, if there was a special variable (e.g.
v:statusline_winid) that would provide the winid of the window this
statusline is for, allowing to just do let active = winid() == v:statusline_winid then.
Note that using an expression has limitations (e.g.
https://github.com/vim/vim/issues/2829), and does not allow for returning
highlight groups (%#Foo# - which get used as text).
(There is g:actual_curbuf already, but it is only available for expressions
(https://github.com/vim/vim/issues/3299) (and also not useful when you want
to know if the window is the active one).)

--
hundred-and-one symptoms of being an internet addict:
16. You step out of your room and realize that your parents have moved and
you don't have a clue when it happened.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Why not use a local 'statusline' option value? Hmm, is the problem that you don't know if the window that statusline belongs to is the active window?

A location 'statusline' option value can be used as a workaround, but it needs to get updated to pass in the active state via it.
I.e. using something like the following (which needs to handle special cases / be more robust IIRC, and adds overhead to every movement between windows in this simplified form):
function! s:set_stl() abort
  let cur = winnr()
  for w in range(1, winnr('$'))
    let active = w == cur
    call setwinvar(w, '&stl', '%!NeomakeStatusLine('.w.', '.active.')')
  endfor
endfunction

augroup my_neomake_stl
  au!
  autocmd WinEnter,BufWinEnter * call s:set_stl()
augroup END

For the record, status lines know when to redraw themselves, so autocommands are never necessary (stay away from what status line plugins do!). In particular, it is not necessary to use autocommands to track the currently drawn status line. You may use something like this:
fun! SetupStl(nr)
  return get(extend(w:, { "is_active": (winnr() == a:nr) }), "", "")
endf

fun! BuildStatusLine(nr)
  return '%{SetupStl(' . a:nr . ')} %{w:["is_active"] ? "active" : "inactive"}'
endf

" winnr() here is always the number of the *active* window
set statusline=%!BuildStatusLine(winnr())

This works because, in a %{} context (i.e., when SetupStl() is invoked), winnr() refers to the window to which the status line being drawn belongs.
Anyway, the patch is welcome and will simplify this task. Looking forward to trying it!


Change OADP operator to be namespace-scoped instead of cluster-scoped

There is no need for OADP operator to be a cluster-scoped operator. This prevents us from fully supporting use cases needed for MTC integration as well as other team's use cases. We should change our operator to only watch the current namespace for CRs.

@dymurray FYI this will break if/when we have to accommodate OLMv1

TL;DR: OLM v1 cannot feasibly support multi-tenancy or any feature that assumes multi-tenancy.


"Unable To Register At This Time"

I have tried registering four different times with four different usernames and each time I get this error, even though Airbitz application clearly accepts my user name because it suddenly becomes unavailable the next time. If I try logging in with my user name, it tells me "Failed to login with PIN."
I have tried using Chrome and I get the errors. I tried using Microsoft Edge, but it loads indefinitely and never goes anywhere.

@paullinator any idea what's going on here?

Yes. We had one last breaking API change which required a server side change. We have it fixed on augur.airbitz.co and will be submitting a pull request this weekend to fix the augur code base.


Non nominal cases for account delete

The API service provided for the test doesn't seems to have the expected behaviour on delete account for non nominal case.

When the account doesn't exists, the returned code is 204 and not 404
When the account does exists but the version doesn't match, a 404 is return (instead of a 409)

This is not a major issue, but it can be confusing when implementing integration tests cases for account deletion.
The tag of the image configured in the docker-compose at the moment of my test is v1.0.0-4-g63cf8434, but the behaviour seems to be the same on latest.

Verified the reported behaviour.
Fixing in internal repo.

Verified the reported behaviour.
Fixing in internal repo.

Closing as fix was merged


Add check for pre-specified YACC to yacc tool's exist()

Fix exist()
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[x] I have updated the appropriate documentation

LGTM

Merging.


Added encoding to spec_name and tags

Already implemented with my latest commit

Already implemented with my latest commit

Already implemented with my latest commit

Sorry I didn't notice

Already implemented with my latest commit

Sorry I didn't notice

Already implemented with my latest commit

Sorry I didn't notice

Already implemented with my latest commit

Sorry I didn't notice

Don't apologize for helping, thanks for the PR!

Don't apologize for helping, thanks for the PR!


PostgreSQL nextval()

PostgreSQL reversed engineer schema includes nextval as a function, like below. Is it possible to support this?
CREATE TABLE pagila_dev.actor (
actor_id integer DEFAULT nextval('pagila_dev.actor_actor_id_seq'::regclass) NOT NULL,
first_name text NOT NULL,
last_name text NOT NULL,
last_update timestamp with time zone DEFAULT now() NOT NULL
)

PRs are welcome ...

PRs are welcome ...


feat: Add support for managing aws-auth configmap using new kubernetes_config_map_v1_data resource

Description

Add support for managing aws-auth configmap using new kubernetes_config_map_v1_data resource

New variables have been added to support enabling/disabling the configmap management by the module, to add additional IAM role ARNs, roles, users, and accounts to the configmap
A kubernetes_config_map is added with a variable var.create_aws_auth_configmap for scenarios where the configmap does not exist and will need to be created (i.e. - using self managed node groups only)

Motivation and Context

Closes #1901 and solves a lot of heartache for many 😅
This new resource properly handles patching an existing configmap resource without any additional hacks. This brings back comparable functionality that was available in v17.x which unfortunately had various issues for supporting (including the use of the forked http provider). I have validated all examples and this is working beautifully thanks to https://github.com/hashicorp/terraform-provider-kubernetes/pull/1671

Breaking Changes

No

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

eh, need to take a deeper look at this again

ok good to go - had to add in a kubernetes_config_map to create a configmap when one doesn't exist

This PR is included in version 18.20.0 :tada:

What was the reason to rely on the aws cli instead of the aws_eks_cluster_auth data source?
e.g.
data "aws_eks_cluster_auth" "cluster" {
  name = module.eks.cluster_id
}

provider "kubernetes" {
  host                   = module.eks.cluster_endpoint
  cluster_ca_certificate = base64decode(module.eks.cluster_certificate_authority_data)
  token                  = data.aws_eks_cluster_auth.cluster.token
  load_config_file       = false
}

Same question here, we are running all our Terraform code in a CI and we'd like to avoid having to install additional tools depending on the Terraform code we have in our manifests. Depending only on the Terraform binary and plugin automatically installed from the registry would be great.

if you have a fargate only cluster, you will also have to create the map , just like the self managed groups

if you have a fargate only cluster, you will also have to create the map , just like the self managed groups

No, when using Fargate profiles and EKS self managed node groups, the EKS service will automatically update the configmap to ensure the roles are added. Self-managed node groups are the only ones who do not update the configmap automatically

if you have a fargate only cluster, you will also have to create the map , just like the self managed groups

No, when using Fargate profiles and EKS self managed node groups, the EKS service will automatically update the configmap to ensure the roles are added. Self-managed node groups are the only ones who do not update the configmap automatically

@bryantbiggs , you meant to say "when using Fargate profiles and EKS managed node groups..."?

Ehe, nice catch.
Anyway, I tested this yesterday with a fargate only cluster and worked as expected.
Was only hit with another annoying issue about the kubernets provider not being able to use data sources, so I had to change it to exec aws instead. Very annoying.

@tanvp112 thanks for the catch - yes EKS managed node groups. updated the comment above so as to not confuse anyone who lands here

How does this new method work for multiple EKS clusters?

Each cluster has its own configmap

Would it make sense to update https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/docs/UPGRADE-18.0.md? It currently has:

Support for managing aws-auth configmap has been removed.

@stevo-f3 your comment is spot on!! Super confusing having the central documentation for the upgrade give essentially false misleading information. Wasted a large portion of my day attempting to figure out what the variables I see do and why they are supposedly unsupported even though they're not really

@bryantbiggs thanks for reintroducing management of aws-auth ConfigMap. Would it make sense to also re-introduce same labels as before in v17 https://github.com/terraform-aws-modules/terraform-aws-eks/blob/v17.24.0/aws_auth.tf#L69-L75 ?


Bug: 菜单徽标变成黑色，checkbox的方框变成了圆圈。

Version
Vben Admin V5
Describe the bug?
菜单徽标变成黑色，checkbox的方框变成了圆圈。
Reproduction
1
System Info
1

Relevant log output
1

Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

徽标问题解决了，checkbox方框变圆圈麻烦给给截图

受到圆角的影响
https://antdv.com/theme-editor

设置borderRadius过大就会变圆角，不算问题
可以通过偏好设置调整


Added test for loading Blackrock file version 2.1

To ensure Blackrock files with version 2.1 are loading correctly I added tests in test_blackrockio.py and test_blackrockrawio.py that compare the files loaded by Blackrock(Raw)IO to the contents of a matlab file that was generated from the test data using Blackrock Microsystems' NMPK toolbox (scripts are located on g-node like the files themselves).
Apart from that I added a second test_input to test_blackrockio.py that tests loading file version 2.1 with different parameters similar to version 2.3.
Note that this cannot work until Blackrock files of version 2.1 can be loaded in BlackrockIO, see pull request #435.

Coverage decreased (-0.2%) to 48.834% when pulling dd6eb01784631b16e7ba9aed5ce99ceb3d0e5748 on bjoern1001001:add_blackrock_21_tests into ab720e75e2c947385a2eb977ee2002770d16dde2 on NeuralEnsemble:master.

Updated and adapted to removal of time_slices in read_block

OK for me.


store options (especially regexps) as JSON

The problem to solve
I use sanitize-html in an MEAN (Mongo Express Angular Node) application, and the sanitize-html options are flexible and powerful. For example:
sanitizeHtml : {
  allowedStyles : {
    '*' : {
      'text-align'    : [ /^(left|center|right)$/ ],
      'float'         : [ /^(none|left|right|initial|inherit)$/ ],
    }
  }
}

I want to load options at run time rather than build time, so they can be edited (e.g., to tweak tags, attributes, and iframe hostnames) without rebuilding.
Proposed solution
Before, I used a typescript file (config.ts) which included sanitizeHtml options but was loaded at build time. Now, I use a JSON file (config.json5) in the assets folder so it can be edited and loaded at runtime. The main problem I've found is regular expressions, which JSON must store as strings, AFAIK:
sanitizeHtml : {
  allowedStyles : {
    '*' : {
      'text-align'    : [ '/^(left|center|right)$/' ],
      'float'         : [ '/^(none|left|right|initial|inherit)$/' ],
    }
  }
}

However, now the application needs to find and convert each regexp. I think this would involving walking the data structure, looking for strings that start with "/^" and end with "$/", and converting each to a regexp. Is this something that would be generally useful to others? Are there better ways to load options at runtime?
Thank you for a great package and your feedback!

This JSON5 issue has good discussion & references, explaining why this gets complicated:
https://github.com/json5/json5/issues/91 [Consider serializing RegExp objects to strings]
This SO post describes how to customize JSON's stringify() with replacer and parse() with reviver arguments:
https://stackoverflow.com/questions/12075927/serialization-of-regexp
In my situation, I need to convert regexp strings to regexps, every regexp starts with /^ and ends with $/, and no other strings match this pattern. In other contexts, it might be better to require a unique prefix (e.g. __REGEXP__) as in the SO post.
/**
 * Revive function for JSON.parse() that converts each regexp string to RegExp.
 * @param val   JSON value to revive.
 * @returns     Value with converted regexp strings
 */
function reviveRE(val : any) : any {
  if (Array.isArray(val)) { return val.map(reviveRE); }
  if (typeof val === 'string') {
    const m = val.match(/\/(\^.*\$)\//); // FIXME: adjust regexp for your situation
    if (2 == m?.length) { return new RegExp(m[1]); }
  }
  return val;
}

const cfg   = JSON5.parse(txt, (_k : string, v : any) => reviveRE(v));

As pointed out by @jordanbtucker, JSON5 already handles arrays, so the first line of reviveRE is unnecessary.

@kussmaul You should probably update your regex to /^\/(\^.*\$)\/$/, since your version will find any strings that contain /^ and $/ anywhere within them (e.g. 'This string contains a matching regex /^abc$/ but it is not a matching regex itself.')
If you're looking for a more robust RegExp reviver function, including one that supports flags, see my regexp-reviver.js gist. It was written for JSON5, but it works just as well with JSON.


Update Link

Fix broken link from legacy Technet redirect.

I concur with the suggested changes. Even if the pages are only available in English (en-us) at this time, it is usually a good idea to keep the links language agnostic, just in case some of them are (or will become) available in other languages (except in the event that any of the links become broken again if the /en-us/ part is removed).

Looks like this PR needs a rewrite to add back the technet link replacement for [Netsh Commands for BranchCache in Windows Server 2008 R2].

@ItzLevvie @mikeamoss  I believe since you added the change, you must use the Commit suggestion button for each of your suggestions. I can't merge anything until your suggestions are part of the actual PR. This must be done in the next 30 days or it will become necessary to close this PR.

As mentioned in the other PRs, I am willing to open a new PR to have the changes incorporated in a new PR to get the necessary changes added to the source.

@illfated  Closing this item. Please open a new PR so we can go ahead and accept these changes.

This PR has already been resolved outside of this conversation (see also closed PR #3428 ).


Full SSBO support for all targets

This will involve SSBOs with more than one member and the last member being an unsized array to a RWByteAddressBuffer for non spirv targets.
See also https://github.com/shader-slang/slang/issues/3348

Close this as won't fix, since we no longer plan to provide full cross-platform support for glsl input.


Could not inject payload into Dock.app. Error: -1743. Catalina 10.15.7.

Hi.
I got the error: scripting-addition either failed to load or could not inject payload into Dock.app! Error: -1743.
OSX Catalina 10.15.7.
Yabai error log:
==> /usr/local/var/log/yabai/yabai.err.log <==
'window_opacity_duration' cannot be changed on macOS Catalina because of an Apple bug in the WindowServer
could not locate display with arrangement index '2'.
could not locate display with arrangement index '2'.
could not locate display with arrangement index '2'.
could not locate display with arrangement index '2'.
could not locate display with arrangement index '3'.
yabai: scripting-addition either failed to load or could not inject payload into Dock.app! Error: -1743
'window_opacity_duration' cannot be changed on macOS Catalina because of an Apple bug in the WindowServer
yabai: scripting-addition either failed to load or could not inject payload into Dock.app! Error: -1743
'window_opacity_duration' cannot be changed on macOS Catalina because of an Apple bug in the WindowServer

SIP is disabled completely:
$ csrutil status
System Integrity Protection status: disabled.

I tried csrutil enable --without debug --withou fs, it didn't work either. Uninstall-sa && install-sa doesn't help. On install-sa, error -1743 appears again in the log.
Security settings:

Yabai version:
$ yabai --version
yabai-v3.3.0

Do you have an idea, what to check?

Fixed.
The problem was, yabai didn't have the specific permission to use Dock.app. Although it was in a list of apps allowed to control my computer (screenshot above).
The fix:

Wipe TCC DB: tccutil reset All. I believe, it's possible to remove yabai related records only, but I haven't found the right command for that. Feel free to post it in comment below.
Reinstall yabai and reboot. Yabai will ask for permissions twice (Catalina 10.15.7). First, to allow to controll your computer. Second, to allow to use Dock.app.


Us77220 telemetry

Has:

[x] event on launch of change image UI
[x] event on selection of an image in the change image UI
[x] event on lazy load of next page of images
[x] event on press of the upload button

Needs still:

[x] event on search
[ ] add user information to the events
[ ] de-hard-code the telemetry URL so that we can also switch to prod when necessary

Just a note, rather than merging master into your branch, rebase your branch on master (git rebase master), this is often a bit more work, but helps keep the commit log nice and readable. (Merging is sometimes ok for super ugly diffs though)

For the constant telemetryUrl I'm under the impression (from @ryantmer) you'll need to set a config variable in the LMS, similar to what's done with d2l.Tools.Quizzes.TelemetryEndpoint. I'm not sure how this part works but you'll have to set it up so that is uses the dev-telemetryUrl for dev instances and the prod-telemetryUrl for prod instances.
With the telemetryUrl as a config variable you can pass it to the d2l-my-courses FRA as a parameter. You can access it in the project from d2l-my-courses.html and add it as a property. From there you'll have to cascade it down to the components that use telemetry.

Now you need a new branch in the core for this: https://git.dev.d2l/projects/CORE/repos/le/pull-requests/6115/overview
So close to finishing this story...so...close...

The merge instead of rebase was because of a confusing conflict.

Code LGTM,
If you wanted, you could take the functions in image-selector/d2l-basic-image-selector.html and throw them in their own behaviour so that they're hidden away a bit, but that's not really necessary
(At some point we should do a bit of a refactoring sweep and put a bunch of stuff into behaviours/separate components)

I volunteer to test this tomorrow morning

Oh whoops - I have actually already checked this and the LE PR out and done a build - had an offline discussion about it with @cgalvind2l. I believe there's a small change still coming, but all that was here tested a-okay.

Hokay! LP PR for this LGTM, but unfortunately I'm at home and that's causing auth issues, so I can't E2E test :(

E2E check done - I'd say this is good to merge! (Although the LP one should probably be merged first)


Should related reactive properties update each other

Here's code that a user might write:
runtime.add(draggable, view);
ReactiveProperty.of(view, CENTER).subscribe( value -> {
  log(value);
})

This wouldn't work because Draggable writes to the view's TRANSLATION instead of CENTER. However, the user may expect this to work.

In Swift we document which properties our view-based interactions affect.

Would be great if android followed the same convention as iOS. I don't have to know anything about CENTER or TRANSLATION options.
runtime.get(block.layer).position.subscribeToValue {
            (p: CGPoint) in
                   log(p.x)
}

iOS has the same underlying problem, unfortunately. For example:
var position = runtime.get(block.layer).position
var positionX = runtime.get(block.layer).positionX

position.subscribeToValue { (p: CGPoint) in
  log(p.x)
}
positionX.write(0.5) // Error: Nothing is logged even though position's value has changed.

The iOS example you provided works because you happen to subscribe to the same property (position) that Draggable modifies (position). If you subscribed to a different property (positionX), then you would run into the same issues even though positionX and position are related. Like Jeff said, documentation is the easiest way to solve this issue on both platforms. A more involved solution would involve creating a graph of properties that are related to each other.