how to enable sds on ingress gateway and cert-manager using istioctl

Please provide a command to enable SDS and cert-manager using istioctl. I know we can enable gateway using --set components.ingressGateways.enabled=true but how to add SDS? Is there any document with the details of options available?

As per the link https://istio.io/docs/tasks/traffic-management/ingress/ingress-certmgr/
istioctl manifest apply
--set values.gateways.istio-ingressgateway.sds.enabled=true
--set values.global.k8sIngress.enabled=true
--set values.global.k8sIngress.enableHttps=true
--set values.global.k8sIngress.gatewayName=ingressgateway
Few question -

if we use --set values.gateways.istio-ingressgateway.sds.enabled=true we dont have to use --set components.ingressGateways.enabled=true?

values.gateways.istio-ingressgateway.sds.enabled is helm installation option used in istioctl. Is there any option to enable sds which can be used using components.ingressGateways.enabled.

do we need to use k8sIngress.enabled, k8sIngress.enableHttps and k8sIngress.gatewayName=ingressgateway? When using helm to install isitio we did not use these options. can anyone clarify the significance of k8sIngress?

https://preliminary.istio.io/docs/ops/integrations/certmanager/ may be useful as well
k8sIngress.enabled is needed to generate the Gateway object that makes things work. This is not needed in master but is in 1.5 and older versions

@howardjohn Does values.gateways.istio-ingressgateway.sds.enabled really work? The ingress-gateway does not deploy pod with node-agent-k8s,
Actually following commands do not have any diff:
istioctl manifest generate \
--set values.global.k8sIngress.enabled=true		      \
--set values.gateways.istio-ingressgateway.sds.enabled=true > tmp1

istioctl manifest generate \
--set values.global.k8sIngress.enabled=true		      \
--set values.gateways.istio-ingressgateway.sds.enabled=false > tmp2

No diff between tmp1 and tmp2.
$ diff tmp1 tmp2
$

SDS is always enabled now and does not use a different container, it all runs in istio-proxy

Oh, I see. Thank you. So, --set values.global.k8sIngress.enabled=true do not make sense actually. I think it should be removed from docs. (If so, I am happy to send the PR.)

Questions here are addressed and document so I will close this


[ie/crunchyroll] Make additional audio languages available

Description of your pull request and other information

Fetch additional audio languages for crunchyroll streams if available.
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

@Grub4K

I'm curious: what is the use case for querying the information for up to eight or even more languages a video is available for, when I'm most probably only interested in exactly the video I've selected? Each query is another network request...

I'm curious: what is the use case for querying the information for up to eight or even more languages a video is available for, when I'm most probably only interested in exactly the video I've selected? Each query is another network request...

I'm not quite sure what this PR is trying to accomplish but the current default is disgustingly slow with the amount of requests it makes so I'm using a closed PR from ProDev2 that added language selection so it only requests the language I specify (https://github.com/Jules-A/yt-dlp/tree/cr-langs if you want to try it). Grub4K seems obsessed on keeping such stuff to a useless plugin that no-one will end up using so don't know if language selection will ever officially return.

I'm curious: what is the use case for querying the information for up to eight or even more languages a video is available for, when I'm most probably only interested in exactly the video I've selected?

The link that gets copied from the web app is the one with Japanese audio, and there was no option to change the audio.

Each query is another network request...

I can rewrite it as an extractor argument like the hardsub option, I felt that offering more options available for the Format List was more natural from a user in search of a dub perspective.
But it makes sense reducing the request count.

The link that gets copied from the web app is the one with Japanese audio, and there was no option to change the audio.

Hmm. Not for me.

The link from the calendar is always for a video with the language I choose.
In the player you can change language, it'll redirect to the link for the chosen language.
Crunchyroll save this language setting for this show for your next visit.

The link that gets copied from the web app is the one with Japanese audio, and there was no option to change the audio.

Hmm. Not for me.

Same story here:
$ yt-dlp --login --print language https://www.crunchyroll.com/watch/GK9U3KPKW
ja-JP
$ yt-dlp --login --print language https://www.crunchyroll.com/watch/G0DUNKEKJ
en-US

But that's probably offtopic here.

No, this is a valid concern, and exactly the reason why I did not want to implement this before, since doing so will either break the download archive or, like in this case, playlist filtering. It's a lose-lose in that case.
Problem: I think there were cases where passing different urls (the changed one after language change) would turn out the same language version. If someone can provide an example where it doesnt work as expected, please show it here so we can re-evaluate.
The only way I can think off that will solve this while maintaining compatiblity is this:

Replace the current id with the id for the media stream (so new ids are actually unique, since one url could now point to multiple languages)
Provide the old id as _old_archive_ids (so existing archives don't break)
Propagate language selection from playlist to individual url by using smuggle_url (so that playlist filtering and _RETURN_TYPE = 'video' are still valid)
Create an extractor arg to allow multiple languages to be downloaded (default to downloading only the main selected language)

This has to be overridden by the smuggled selection from the playlist, otherwise filtering would break
If anyone can reproduce the mismatch then above would have to be implemented.

Having said that, I am not completely opposed an extractor arg to force language, given that @pukkandan agrees that it is okay that the download archive breaks when providing an extractor arg. Alternatively, https://github.com/Grub4K/yt-dlp/commit/96b2919146b9b7b06c486d979cf7fe03ce8d7b64 can still be merged to reduce playlist queries without impacting compat

The only way I can think off that will solve this while maintaining compatiblity is this:

Replace the current id with the id for the media stream (so new ids are actually unique, since one url could now point to multiple languages)

Provide the old id as _old_archive_ids (so existing archives don't break)

Propagate language selection from playlist to individual url by using smuggle_url (so that playlist filtering and _RETURN_TYPE = 'video' are still valid)

Create an extractor arg to allow multiple languages to be downloaded (default to downloading only the main selected language)

This has to be overridden by the smuggled selection from the playlist, otherwise filtering would break
If anyone can reproduce the mismatch then above would have to be implemented.

I believe this is the ideal solution. Does this have any drawbacks except for complexity?

I believe this is the ideal solution and is similar to how I had implemented funimation before. Does this have any drawbacks except for complexity?

No, not as far as I can tell, however nobody took the time to properly implement it yet. Since I cannot reproduce, didn't get sent any reproduction cases (cough cough), it's a fair effort, AND will change the ids used I was hesitant to implement it myself


ScaledJob "Target Average Value" prometheus

Discussed in https://github.com/kedacore/keda/discussions/3437

Originally posted by rh-voice July 27, 2022
Hi there,
So I am trying to scale some jobs that run agents for our CI/CD; I can read pipeline demand metrics from prometheus (I have an exporter for that); Now this works fine with ScaledObjects; but I am trying to switch those to jobs.
My question is; what the "Target Average Value" attribute for prometheus ? because Threshold is not working; which is expected; it defines the threshold to start scaling for HPA not for spinning up jobs.

Let's talk about this in the discussion


Fix doc comment line splits for the first paragraph or report issue with tooling

          This is what I mean:

Versus:

Originally posted by @asmello in https://github.com/chanced/jsonptr/pull/56#discussion_r1669268761
...
Strangely enough, vscode gets it right.

Originally posted by @chanced in https://github.com/chanced/jsonptr/pull/56#discussion_r1669278695

I've clarified it with Jon, and as it turns out his remark about the first line having special treatment was about the first line under markdown semantics - delimited by \n\n - so I think this is actually ok. It does seem unintended to have the newlines reproduced raw in the hover docs, but I tested with vscode and I can't reproduce, so it seems that rust-analyzer sends the raw markdown and it's up to the editor to render how they choose. So I guess this is on Helix to fix.
Sorry about the false alarm! Will close out, hope you haven't wasted energy changing this yet.

I dont think this was a false alarm, but rather that some tooling in the ecosystem isn't properly parsing markdown, or at least not rendering it properly.
The screenshots were from neovim right?  Kinda got sucked into the easiness of vscode with vim bindings and just got lazy, i guess.

The screenshots were from neovim right? I wrecked my config for neovim and just haven't had the time to go back and rectify that. I kinda got sucked into the easiness of vscode with vim bindings and just got lazy, I guess.

Actually I use Helix! If you're used to Neovim, Helix is probably not quite mature enough yet for a migration, but since I never used Neovim in the first place I figured I may as well skip straight into the next thing.

Ah, helix is quite nice. I actually prefer the movements in it, the only reason I haven't switched is the lack of a plugin system. Well, that or a file tree (not sure why I'm so dependent on one).


chore: bdd-sbhg-1584638450 to 0.0.1

chore: Promote bdd-sbhg-1584638450 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


The library has no method to list threads

Confirm this is a Node library issue and not an underlying OpenAI API issue

[ ] This is an issue with the Node library

Describe the bug
There is no method in the API to list threads, although technically they can be listed in the dashboard at https://platform.openai.com/threads (has to be enabled in the org setting)
This question was asked in the community as well
https://community.openai.com/t/where-can-i-see-all-the-threads-ive-created-with-the-api/592963/9
You may also want to update the documentation page to warn users that thread information should be maintained in the external system while you fix the API.
To Reproduce
Easy to reproduce: look at official documentation page - https://platform.openai.com/docs/api-reference/threads
Code snippets
No response
OS
macOs
Node version
node 20
Library version
3.0.35

I hope the endpoint is simply not documented yet, nope - API blocks access for API keys
{
    "message": "Your request to GET /v1/threads must be made with a session key (that is, it can only be made from the browser). You made it with the following key type: secret.",
    "type": "invalid_request_error",
    "param": null,
    "code": "missing_scope"
  }

The SDK generally does not support endpoints which are not generally available.

So I got the Grammar correction example and it doesn't work, I'm not sure why in the documentation is available as an option but when you try to use the API it doesn't work. What we can do to make it work?


Issue with ffmpeg 5 on Ubuntu 22.10

Compiling ccextractor with rusty_ffmpeg and ffmpeg 5.0 on Ubuntu 22.10 gives this error, can you check if something is missing:
/usr/bin/ld: ./libccx_rust.a(rsmpeg-d3dd6e1fa3ffc9ea.rsmpeg.d0068580-cgu.10.rcgu.o): in function `<rsmpeg::avfilter::avfilter::AVFilterGraph as core::ops::drop::Drop>::drop':
/home/cfsmp3/.cargo/registry/src/github.com-1ecc6299db9ec823/rsmpeg-0.14.1+ffmpeg.6.0/src/avfilter/avfilter.rs:244: undefined reference to `avfilter_graph_free'

avfilter_graph_free is a fairly basic symbol, and there is a FFmpeg 5.0 compatibility test in rsmpeg's CI(link). I guess the ffmpeg build might be corrupted or something was missed. Any detailed error info?

avfilter_graph_free is a fairly basic symbol, and there is a FFmpeg 5.0 compatibility test in rsmpeg's CI(link). I guess the ffmpeg build might be corrupted or something was missed. Any detailed error info?

See previous comment @ldm0 :-)

Let me install a VirtualBox.

@cfsmp3 @prateekmedia
I reproduced this problem and found that libavfilter is missing on linking. After tweaking linker flag, the compilation succeed.
Here is the PR: https://github.com/CCExtractor/ccextractor/pull/1513


Change Data Capture support SQL Server 2019 Linux??

Is anyone able to enable CDC at table level? At DB level is fine but not able to do it at table level.
SQL Server 2019
Docker
Linux CentOS
"Could not update the metadata that indicates table DBO.TABLENAME is enabled for Change Data Capture. The failure occurred when executing the command '[sys].[sp_cdc_add_job] @job_type = N'capture''. The error returned was 22836: 'Could not update the metadata for database DB to indicate that a Change Data Capture job has been added. The failure occurred when executing the command 'sp_add_jobstep_internal'. The error returned was 14234: 'The specified '@srv' is invalid (valid values are returned by sp_helpserver).'. Use the action and error to determine the cause of the failure and resubmit the request.'. Use the action and error to determine the cause of the failure and resubmit the request."

Hi,
I tried it on my SQL Server 2019 (CTP2.2) container and it worked. Here is my script.
create database test

use test
go
EXEC sys.sp_cdc_enable_db
GO

CREATE TABLE dbo.MyTable (EmployeeID int
PRIMARY KEY CLUSTERED);
go

EXEC sys.sp_cdc_enable_table
@source_schema = N'dbo',
@source_name   = N'MyTable',
@role_name     = N'MyRole',
@filegroup_name = N'Primary',
@supports_net_changes = 1
GO

select name, is_tracked_by_cdc from sys.tables     --check whether a source table has already been enabled for change data capture

Result:

Job 'cdc.test_capture' started successfully.
Job 'cdc.test_cleanup' started successfully.

name                                              is_tracked_by_cdc
------------------------------- -----------------
systranschemas                                    0
change_tables                                     0
ddl_history                                       0
lsn_time_mapping                                  0
captured_columns                                  0
index_columns                                     0
MyTable                                           1
dbo_MyTable_CT                                    0

wow awesome! thx  forkazmoden....I may be missing some thing else.

@forkazmoden: are you using a Linux container right?
Following your steps I got this error and the DB is CDC enabled....
Could not update the metadata that indicates table [dbo].[MyTable] is enabled for Change Data Capture. The failure occurred when executing the command '[sys].[sp_cdc_add_job] @job_type = N'capture''. The error returned was 22836: 'Could not update the metadata for database test to indicate that a Change Data Capture job has been added. The failure occurred when executing the command 'sp_add_jobstep_internal'. The error returned was 14234: 'The specified '@srv' is invalid (valid values are returned by sp_helpserver).'. Use the action and error to determine the cause of the failure and resubmit the request.'. Use the action and error to determine the cause of the failure and resubmit the request.
select name, is_cdc_enabled from sys.databases:
name	is_cdc_enabled
test	            1

Yes, I was using a Linux container.
select @@version

result:
Microsoft SQL Server 2019 (CTP2.2) - 15.0.1200.24 (X64)
	Dec  5 2018 16:51:26
	Copyright (C) 2018 Microsoft Corporation
	Developer Edition (64-bit) on Linux (Ubuntu 16.04.5 LTS) <X64>

(1 row affected)

Thank you @forkazmoden will keep searching for a fix.

I used to see similar error message when try to enable CDC on normal SQL Server instance. It is caused by the mismatch of server name. You can run the following query to check that.
select SERVERPROPERTY('ServerName')
select * from master.dbo.sysservers

You are crack.... it was exactly that... for some reason there were different names in sys.servers table.
So the steps to fix it
select SERVERPROPERTY('ServerName') : Correct_Server_Name
select srvname from master.dbo.sysservers: Old_Server_Name
the fix:
EXEC sp_DROPSERVER 'Old_Server_Name'
EXEC sp_ADDSERVER 'Correct_Server_Name', 'local'
at it works!
Thx a lot @forkazmoden !

Glad to hear that:-) If the issue has been resolved, you can close this thread.

Closing.


Support resource files added once project opened

Currently, a list of resource files is identified when the package is initialized.
This does not account for resource files added to a project once opened, or existing projects (containing resources) added to the opened solution.

Also, need to support new entries in the RESX file once the project has been loaded.

Investigate IVsSolutionEvents.OnAfterLoadProject for when project added to solution
Investigate IVsRunningDocTableEvents3.OnBeforeSave for when project file saved -- as might have a new resource file in it.

Look at using FileSystemWatcher as per suggestion in #11

Handle changes to

*.sln (as a new project may have been added)
*.csproj & *.vbproj (as a new resource file may have been added)

The above would also handle the removal of projects or files too.


differentiate commits not belong to commits to pull

this https://github.com/sourcegit-scm/sourcegit/commit/14469b13990d9c6d93b447b4b967bd894167ba49 made me a suggestion: should be good differentiate colors about the commits that not belong to the branch (and now is a great thing), but should be good have a difference about the commits that need to be pulled

I've pushed a commit that implements this feature in the way that another well-known git GUI client Fork does.


sad: fix build

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: sad
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: sad
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25
shrinking /nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25/bin/alice
shrinking /nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25/provers/moses
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25/bin
patching script interpreter paths in /nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25
checking for references to /build in /nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25...
/nix/store/18v77x84sjgyh55xcfz88mrqh1jqncv9-system-for-automated-deduction-2.3.25


Expose RenderingServer SSIL quality setter methods

This allows changing SSIL quality at run-time in a project.
In my photo mode add-on, I added a way to maximize SSIL quality thanks to the newly exposed setter:[^1]
Before

After

[^1]: The change isn't pushed to that repository yet.

Thanks!


Option for passing memory-limit to meili:import command

Description
First of all, thanks for creating this bundle!
I created a test-setup where I imported around 700.000 records into the search-index of Meilisearch. While doing so, I realized that the import requires more PHP memory than regular operations / my regular PHP memory_limit configuration. To successfully import the data into the search-index, I had to increase the memory_limit to 3072M.
I was also testing TNTSearch (https://github.com/teamtnt/tntsearch) with the same data-set and for exporting / creating the index, 512M were sufficient. However, TNTSearch is not using Doctrine.
I will look into the memory consumption in more detail, but I would suggest having the possibility to pass the memory_limit as a parameter to the command. E.g.
php bin/console meili:import --memory-limit=<memory in MB>
And then use this value to ini_set the memory_limit in the execute function:
ini_set('memory_limit', $memoryLimit);
With this approach, regular updates of the index would run with the general memory_limt. And full imports could be run with a specific memory setting passed to the command.

A possible solution could look like this:
src/Command/MeilisearchImportCommand.php
->addOption(
    'memory-limit',
    'm',
    InputOption::VALUE_REQUIRED,
    'Memory limt (in MB) to run the import command',
    trim(ini_get('memory_limit'), 'M')
)

protected function execute(InputInterface $input, OutputInterface $output): int
{
    ini_set('memory_limit', $input->getOption('memory-limit').'M');
    $indexes = $this->getEntitiesFromArgs($input, $output);
    $config = $this->searchService->getConfiguration();
    ...

I will look into the memory consumption in more detail, but I would suggest having the possibility to pass the memory_limit as a parameter to the command. E.g. php bin/console meili:import --memory-limit=<memory in MB>
And then use this value to ini_set the memory_limit in the execute function: ini_set('memory_limit', $memoryLimit);

You can run as php -d memory_limit=128M bin/console meili:import

Thanks for the tip. No need for another parameter in the command...


Using SigProfiler with an heterogeneous dataset

I am currently working on a signatures extraction for a neuroblastoma cancers dataset. The specificity of our dataset is its heterogeneity. Our samples (WGS) come from both untreated and treated patients. Therefore, the profiles I am working with are very different and lead to some theoretical issues.
1. The total number of mutations vary between the samples
Because I am working on untreated and treated samples, the mutational burden varies a lot between the samples. The result is that SigProfiler primarily minimizes the error of the samples with a high number of mutations. When I plot error_by_sample=f(total_number_of_mutations_by_sample), I get a strong correlation suggesting that the mutational burden biases the results. It is not a surprise looking both on the cost function and the update rules.
Potential solution and question: Since it is a work on mutational profiles, I have normalized all the samples in order to remove the mutational burden bias (giving the same number of mutations for each sample). Do you think it is a good idea ? Am I missing a biological aspect by doing this ?
2. The dataset is slightly unbalanced
The dataset I am working on is slightly unbalanced. The result is that it seems that the algorithm is focusing hierarchically on the parts of the dataset that are more represented. In particular, when looking on the signatures obtained for different values of the total number of mutations, it seems that the distribution of the dataset biases the extractions. Therefore, it is possible that, for a given value of K, I overfit a part of the dataset while underfitting the the other part.
Question: Do you think that in such a case, I can trust the optimal value of K ?
3. Is there a parameter that allows to add sparsity during the extraction
Because the dataset is heterogeneous, there are some signatures that are present only in a part of the dataset (the signatures associated tp a type of chemotherapy are only present in the treated samples). The problem is that these signatures modify the signatures that are present in the entire dataset. I observe a clear influence of these signatures when I perform an extraction on the entire dataset and an extraction only on the untreated samples.
Potential solution and question: One solution could be to add sparsity during the extraction (on the exposure).
I have seen that you have created a parameter “penalty” to set up the degree of sparsity in the suggested solution,
If I have understood well your code, this penalty is only applied during the fitting part of the algorithm.
During the extraction, the cost functions minimized are an L2-norm or a KL-divergence without any sparsity penalty.
Question: Am I right when I say that there is no parameter to add sparsity during the extraction ? Is there a way with this implementation to fight against the bleeding of signatures ?

The total number of mutations vary between the samples

The total number of mutations can indeed affect the results of NMF extraction. Samples with higher number of mutations have more weight in the minimization. However, this most likely reflects the reality as we are more confident about the pattern of mutations in samples with higher numbers of mutations. For example, a sample with only a single C>T at TpCpT mutation can be observed simply by chance. In contrast, a sample with 1000 C>T at TpCpT mutations may not have exactly 1000 mutations but (assuming a Poisson distribution of the errors) we have a high confidence that it has between 950 and 1050 mutations. In principle, we do not recommend normalization of samples as this will make samples with low numbers of mutations and samples with high numbers of mutations equally possible. Instead, SigProfiler leverages Poisson resampling in each of its iterations.

The dataset is slightly unbalanced

Indeed, this is a common problem in NMF. SigProfiler has a hierarchal version that allows analysis with underlying structure. Please use the hierarchal option.

Is there a parameter that allows to add sparsity during the extraction

Currently, there is no sparsity as part of the de novo extraction of mutational signatures. Our prior experience with such penalties was that the results can be very dependent on the sparsity parameter. Further, we do not plan to implement such a parameter in the near future. However, we do have a sparsity parameter for assignment of mutational signatures. It reduces/removes the signature bleeding of the decomposed solution (i.e., when the de novo extracted signatures are matched with the known set of consensus signatures).


Window is not defined

I am using version 12.0.4 within a mean stack app.
I was initially having issues with angular recognizing the module and then realized that it was installed globally. After installing it locally I tried to run it with a require statement attached and I received this error in my shell:
C:\myapp\node_modules\ng-file-upload\dist\ng-file-upload-all.js:23
if (!window.FileAPI) {
^
ReferenceError: window is not defined
at C:\myapp\node_modules\ng-file-upload\dist\ng-file-upload-all.js:23:8
at Object. (C:\myapp\node_modules\ng-file-upload\dist\ng-file-upload-all.js:230:3)
at Module._compile (module.js:409:26)
at Object.Module._extensions..js (module.js:416:10)
at Module.load (module.js:343:32)
at Function.Module._load (module.js:300:12)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
at Object. (C:\myapp\node_modules\ng-file-upload\index.js:1:63)
at Module._compile (module.js:409:26)
If there were other cited issues like this and I missed them, please link them!

You probably trying to run it server side.


feat: #1918 #1934 - simplified user preferences food

What

The user preferences/food is simplified
One and only one group is expanded
Onboarding: removed doubled description and confusing refresh button

Screenshot

Fixes bug(s)

Fixes: #1934
Closes: #1918

Why is there no arrow anymore on the extended panel ? Could we not rotate it counter-clock wise ?

Why is there no arrow anymore on the extended panel ? Could we not rotate it counter-clock wise ?

@teolemon The thing is that I assumed (or read) that there was always one and only one item selected. Therefore, if I put a "collapse" button, that means more or less that there's a collapse action. Which is not the case, because there would be no group expanded then.
I'm open to suggestions.

2 suggestions

@teolemon Done

Codecov Report

Merging #1950 (ef8f838) into develop (2ea0da3) will decrease coverage by 0.46%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #1950      +/-   ##
==========================================
- Coverage     8.86%   8.40%   -0.47%
==========================================
  Files          161     176      +15
  Lines         6623    8226    +1603
==========================================
+ Hits           587     691     +104
- Misses        6036    7535    +1499

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
61.64% <0.00%> (-21.34%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
14.10% <0.00%> (-18.21%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
26.47% <0.00%> (-4.96%)
:arrow_down:

packages/smooth_app/lib/main.dart
15.74% <0.00%> (-2.16%)
:arrow_down:

...es/smooth_app/lib/data_models/smooth_it_model.dart
5.00% <0.00%> (-1.25%)
:arrow_down:

...smooth_app/lib/pages/product/new_product_page.dart
0.00% <0.00%> (-0.88%)
:arrow_down:

...rds/knowledge_panels/knowledge_panels_builder.dart
1.88% <0.00%> (-0.29%)
:arrow_down:

...es/smooth_app/lib/cards/data_cards/score_card.dart
4.54% <0.00%> (-0.22%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
2.94% <0.00%> (-0.19%)
:arrow_down:

... and 126 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b32506e...ef8f838. Read the comment docs.

Thank you @M123-dev for the code review!
@teolemon too seems to prefer the animation - feel free to put it back (I'm in no condition to do it right now, github-wise).
Regarding the scroll jump I don't picture what you mean or more exactly what is expected. Please create an issue if needed. I'm sure it would involve a smooth scrolling too :)


Why is the downlink queue being purged on Class C messages?

Hi @brocaar,
I stumbled with instructions in the HandleDownlinkQueueItem function, puring the downlink queue when sending a message to a Class C device.
See the file:  https://github.com/brocaar/lora-app-server/blob/a24ddb41a7b210e465e0deb3008958d2974d28ad/internal/downlink/downlink.go#L68

Why is the downlink queue being purged on Class C messages?
Why are the Class C messages not beeing queued, as normal messages?

In my understanding, there is no concept of a queue in Class-C, as frames are sent immediately to the node. So when sending 10 frames using Class-C, they would all be sent immediately to the node.
Now for confirmed-data (down), when an ACK is received (from the node) you still want to have the last transmitted frame around (to send an ack notification to the user with the reference of that frame).

@brocaar what do think of introducing an optional flag to enqueue (Class C) downlink message, like any other message?
Example:
            {
                "reference": abcd1234,
                "confirmed": false,
                "fPort": 42,
                "data": base64.b64encode("Hello World!"),
                "force-enqueue": true             // default := false
            }

This flag would then allow frames to be sent one after another to a Class C mote.
As my understanding Class C introduces a continuous listening mode, so therefore the mote should be able to receive data all the time, except when it is transmitting. In addition the RX2 downlink only frequency (869.525 Mhz) has a duty cycle of 10%. Thus the NS could send several frames (e.g segmented data) to the mote. So it is useful if the network server (the lora-app-server component) is able to enqueue and schedule the data (tx frame at timestamp+timeOnAir+GuardTime).
A use case could be firmware update over the air. Or exchanging the crypto keys.
Opinions?
Questions?

The only issue I see is with confirmed data down, would LoRa App Server in this case keep broadcasting this frame, until it receives an ACK? Then also some firmwares send an ACK when the node is sending data back (which might be a day later). It is currently undefined in the LoRaWAN specs how a node should handle when it receives a Class-C confirmed data-down.
This makes the Class-C queue a bit problematic in my opinion. When sending data as an application, you expect that the frame is send immediately. In case of a "hanging confirmed-data down item", this would not be the case.


Shared heartbeat for proxied services

Hello,
I have a scenario where I have eureka registered micro-services running behind a proxy.
The proxy is a true microservice , it is UP or DOWN.
The sub-services are virtual and created and registered when the proxy is started.
They should use/share the heartbeat of the proxy microservice. All UP or All DOWN.
Is there a way (without custom code) for this scenario to be possible using the supplied Netflix classes/config options (sub-services share heartbeat of proxy) ?
Is there a way to disable a heartbeat for a service permanently other than setting expiry periods so that a service is always up unless told otherwise ?

If I understand your design correctly, you want something like:

Have the Proxy heartbeat to Eureka only, do not register the sub-services with Eureka at all as they are not supposed to be independently addressable.

In the proxy heartbeat logic, logically AND or logically OR the sub-service statuses depending on the desired behavior, determine the sub-service statuses by hitting their healthcheck endpoint periodically.

If you wish to just always report UP (this is even weaker than doing a logical OR of all sub-service healthcheck statuses), you can set up a no-op callback always saying it's UP (on the proxy) when registering with Eureka.
Hope this makes sense.


Add features landing page

this is looking great. just a couple things I noticed

the design has the individual items inline at desktop width, (e.g name  description  'view doc'). this can break down for smaller screens (just set flex-direction: column and maybe change the gap size)
reduce the padding more on smaller screens in the accordions
the design has all of the features on the same page. can use a hash param and IDs to scroll to the right section from a link


Fix the bug that frequent refreshing of worker info reduces throughput

What changes are proposed in this pull request?
Fix bug that worker info is refreshed frequently.
Why are the changes needed?
We found that worker information is refreshed frequently and unnecessarily. This dramatically decreases the throughput since the hash function in refresh operation takes much time. Therefore, we have to reduce the refresh frequency and now we don't need to update the list of worker addresses if they are not updated in a fixed interval.
Does this PR introduce any user facing changes?
No, it doesn't.

alluxio-bot, merge this please.


Develop multiple-selection content type field

PocketBase: Show error when change type from multiple to single

Directus: It is not important for it. first tries to fetch data from db, if it works with current type, it will set initial values, otherwise it will skip value and update when content saved.
for Block Editor component it stores this JSON

When I change it's type to Checkbox Group, all items are unchecked and after save, value changes to Array of string.
Webflow: It doesn't support Multiple selection mode, but it has (Image, Multiple Image), (Reference, Multi Reference) types.
To have multiple selection field we should create new ContentType and use Multi Reference type.

@pournasserian We can have separate Field type for single/multiple modes.
String
Number
Boolean
Date
Media
Media (Multiple)
Enumeration / Options / Select
Enumeration (Multiple) / Options (Multiple) / Select (Multiple)
Refernece
Reference (Multiple) / Multi Reference / References
JSON --> for complex types we can have json type and show CodeEditor for Content form.


Allow mgmt create via controller if managed by cli

Issue #, if available:
The unification of the controller and CLI requires the ability to create mgmt clusters via the controller. However, we should only allow this creation when it's initiated by the CLI.
Description of changes:
Modifies webhook create validations to return an error for self-managed clusters only if the managed-by-cli annotation is not present.
Testing (if applicable):
Manually created mgmt clusters with CLI (both old and new workflows) and FLC.
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


[Request] .dat file naming suggesting

I know when adding new accounts, it creates each new .dat file as 1.dat, 2.dat, etc.  And the newest one created will always be the latest number.  However I was thinking why not have the .dat named which matches the "nickname"  So you would have ones named Alt 1.dat, Alt 2.dat, Main Account.dat, etc  So you know exactly which one is which.  Because sometimes you want to rearrange them in certain order, and they won't always match up with the original order created.

For me its not even working to log in an other account... no matter which one i choose it always takes the last used one...
Does this work for you?
where can I find the "1.dat" and "2.dat" files you are talking about :)?
thx buddy

The problem with the files is that they actually represent the unique ID of an account. The nickname of the account can be switched anytime and therefore would require remake of the local.dat file

Yeah I was just thinking about that too before I passed out in bed.  The only way is if LB would rename that exact dat file once changes were made.

@underlinex3 Give this a try.  https://www.reddit.com/r/Guildwars2/comments/als6hu/gw2_launchbuddy_release_184_still_standing/efhdih7/

@underlinex3 Give this a try. https://www.reddit.com/r/Guildwars2/comments/als6hu/gw2_launchbuddy_release_184_still_standing/efhdih7/

omg. you my hero <3 thx


Deserialize secrets inplace for api_keys in PineconeEmbeddingRetriever

Loading serialised pipeline which includes PineconeEmbeddingRetriever with PineconeDocumentStore currently fails because loading of api keys is not correctly deserialised into Secret class. I propose changes to fix this.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please solve this bug ASAP as it's preventing me from using serialised haystack pipelines in deployment.

Thanks for the bug report and the PR! Since the correct fix for it was a bit more involved, we've opened another one. I'll close this in favour of the latter.

Released a new version: https://pypi.org/project/pinecone_haystack/0.4.1/

Cheers, thank you!


WASM file size has become much larger than necessary

There's a lot of zeroes inside the file. I think it's due to the buffer used for the fallback allocator.

Code is failing multiple integration tests.


"Link all assemblies" iOS option causes failure to load OData adapter

Described in the comment to the post:
https://vagifabilov.wordpress.com/2014/11/03/resolving-filenotfoundexception-when-using-simple-odata-client-in-ios-applications/comment-page-1/#comment-102
"However, I have trouble using it with third linker options: Link all assemblies.
Maybe the linker stripped some members from some other dependent assembly (e.g. Micrsoft.Data.OData)"
IUnable to load OData adapter from assembly Simple.OData.Client.V3.Adapter —> System.ArgumentNullException: Argument cannot be null. Parameter name: type at System.Reflection.IntrospectionExtensions.GetTypeInfo (System.Type type) [0x0000c] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Reflection/IntrospectionExtensions.cs:38 at Simple.OData.Client.Extensions.TypeExtensions.GetDeclaredConstructors (System.Type type) [0x00000] in :0 at Simple.OData.Client.AdapterFactory.LoadAdapter (System.String adapterAssemblyName, System.String adapterTypeName, System.Object[] ctorParams) [0x00000] in :0 — End of inner exception stack trace — at Simple.OData.Client.AdapterFactory.LoadAdapter (System.String adapterAssemblyName, System.String adapterTypeName, System.Object[] ctorParams) [0x00000] in :0 at Simple.OData.Client.AdapterFactory+c__DisplayClass18.b__16 () [0x00000] in :0 at Simple.OData.Client.AdapterFactory+d__0.MoveNext () [0x00000] in :0 — End of stack trace from previous location where exception was thrown — at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000b] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Runtime.ExceptionServices/ExceptionDispatchInfo.cs:62 at System.Runtime.CompilerServices.TaskAwaiter1[Simple.OData.Client.IODataAdapter].GetResult () [0x00034] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Runtime.CompilerServices/TaskAwaiter_T.cs:59 at Simple.OData.Client.Session+d__c.MoveNext () [0x00000] in :0 — End of stack trace from previous location where exception was thrown — at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000b] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Runtime.ExceptionServices/ExceptionDispatchInfo.cs:62 at System.Runtime.CompilerServices.TaskAwaiter1[Simple.OData.Client.IODataAdapter].GetResult () [0x00034] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Runtime.CompilerServices/TaskAwaiter_T.cs:59 at Simple.OData.Client.ODataClient+d__b6.MoveNext () [0x00000] in :0 } System.InvalidOperationException
Here is the stack trace:
at System.Threading.Tasks.Task1[System.Collections.Generic.IEnumerable1[System.Collections.Generic.IDictionary2[System.String,System.Object]]].get_Result () [0x0003c] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Threading.Tasks/Task_T.cs:52 at Simple.OData.Client.FluentClientBase1+c__DisplayClass4[Ptix.Entities.Show].b__3 (System.Threading.Tasks.Task1 x) [0x00000] in :0 at System.Threading.Tasks.TaskActionInvoker+FuncTaskInvoke2[System.Collections.Generic.IEnumerable1[System.Collections.Generic.IDictionary2[System.String,System.Object]],System.Collections.Generic.IEnumerable1[System.Collections.Generic.IDictionary2[System.String,System.Object]]].Invoke (System.Threading.Tasks.Task owner, System.Object state, System.Threading.Tasks.Task context) [0x00000] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Threading.Tasks/TaskActionInvoker.cs:330
at System.Threading.Tasks.Task.InnerInvoke () [0x00019] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Threading.Tasks/Task.cs:546
at System.Threading.Tasks.Task.ThreadStart () [0x000d0] in /Developer/MonoTouch/Source/mono/mcs/class/corlib/System.Threading.Tasks/Task.cs:438

Hi, any progress on this issue?
I now get the exact identical StackTrace in a Xamarin Class-Library, independent of Device.
It is caused when calling
var commandText = await command.GetCommandTextAsync();
As we have checked in all packages on 15.April (Simple.OData.V3 in version 4.3.0), and have not updated them since, there no changes on this, or its dependencies, could be responsible for the exception.
Only updates we did were Xamarin/SDK, and .NET/Windows.

Hi,
Is it possible for you to build a small sample project so I can reproduce the error? You can send it to vagif.abilov at gmail. Please specify what version of Xamarin tools you are using.

Unable to reproduce, closing.


autoRegister: false doesn't work

LWC Version (/lwc admin version):
Modern LWC 2.1
Server version (/version):
Paper 271 - Minecraft 1.13.1
Short five word summary of issue:
configuration, permissions, plots, auto-protect
Detailed description of problem:
Already, in core.yml autoRegister: false does not work if you leave the autoRegister: private (or public) line for a block
So I deleted the autoRegister: private line from all blocks
Then, the plugin doesn't work with PlotSquared or LuckPerms I don't know
I have disabled lwc.* permission for my plot world and players cannot use the chests they have placed in their plots
The message: The server admin does not allow you to open this (a red message from LWC)

You are reporting more than one issue. I would appreciate if you can make several issues instead of putting them all here. Mixing them makes it really hard to understand what your problem with each of them are.
Using "autoRegister" on a block will cause it to lock automatically when players place it. You can add or remove this from each block to have it work the way you want it to.
LWC should work perfectly fine with LuckPerms. You shouldn't be using a negative wildcard though, or players will not have access to their chests.
I don't know anything about PlotSquared, but it's a fairly popular plugin and no one has reported issues with it working with LWC.
I am closing this issue now. Please make a new one for each problem you still have, if I have not answered them already above. Also, be much more descriptive! Saying "It doesn't work" will not help fix it. Thanks!


[Doc-Improvement][Redirect-files]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/teams-toolkit-fundamentals.md
90
11
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit bc14027:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/toolkit/teams-toolkit-fundamentals.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/teams-toolkit-fundamentals.md
90
11
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 4e03360:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

msteams-platform/toolkit/teams-toolkit-fundamentals.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/teams-toolkit-fundamentals.md
90
11
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 192f1ce:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

msteams-platform/toolkit/teams-toolkit-fundamentals.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Use the defaultChannel for the OADP OLM Subscription

Since a default channel is specified in the package manifest of the OADP operator OLM bundleL
$ oc get packagemanifests redhat-oadp-operator -oyaml |grep defaultChannel
  defaultChannel: stable-1.4

we can use it to get the latest stable version of the latter.  This way we won't have to update the Subscription whenever a new version comes up or a channel we are currently using suddenly disappears.
/cc @tsorya @javipolo

I think it's totally worth a try :+1:


main.45ffd3ab.js:1 Uncaught Error: unable to get client provider context: unable to create client context:

When I try to use the demo , I am not able to update the option and it returns me the following error in browser console.
Error in option passed to client: Unable to load identity: failed to get signing identity: find enrollment cert path failed: read directory failed: open<EMAIL_ADDRESS>no such file or directory

Hi @mohamednoushad,
What option did you try to update?
The error looks like it is coming from the Fabric SDK about the config you used to run your fab3 instance. It seems like this<EMAIL_ADDRESS>file is not there.
What config did you use? And what is your fabric network setup?
Regards,
Swetha

Hi @swetharepakula
I was trying out directly the online demo link and found this error in the console.
I havent tried to set it up manually.

We are not running the live demo anymore. Please try to the demo locally.


spi_ll_stm32 driver does not support stm32f1soc

when I compile the samples/hello_world with the stm32f103ze boards, Two errors appear：
error: ‘SPI_SR_FRE’ undeclared
error: ‘LL_SPI_PROTOCOL_MOTOROLA’ undeclared

I also encountered the same problem.
I plan to use spi with the stm32f107 , but dts or pinmux seem can't configure SPI pins.
however, stm32f4, f3, l4 all works with pinmux except f1.

I have the same problem.
I want to use SPI on stm32_min_dev board (stm32f103)

stm32f1 does not have the SPI_SR_FRE regesiter bit，and not support the LL_SPI_PROTOCOL_MOTOROLA protocol.

I will check it.

Just a quick update.
I started working on the issue at https://github.com/ydamigos/zephyr/tree/stm32f1-spi.
The driver compiles fine but when I use the spi-loopback test, I get a bus fault, pointing to line 245 of spi_contex.h. The error isn't specifically in this part of the code. I will try to find the root of it.
There is a similar issue #3850 for stm32f3 family.

Could you test PR #5413?


Fix download command

This fixes the download command of the celebrity dataset.

this happened to me before

What do you mean? It works with this change.

i mean that dropbox dropped the link before we have to keep the warning. but I noticed your correction. would you update the PR with the new warning?

Aha, ok :) will do that tomorrow!

thank you!!!


Positional encoding?

I can't find any positional encoding in models or examples.

@matthew-jurewicz hey, it's taken care of for you! https://github.com/lucidrains/linear-attention-transformer/blob/master/linear_attention_transformer/linear_attention_transformer.py#L372


Markdown alerts do not seem to display background color

Past Issues Searched

[X] I have searched open and closed issues to make sure that the bug has not yet been reported

Issue is a Bug Report

[X] This is a bug report and not a feature request, nor asking for support

Describe the bug
HTML alerts display with a background color.
Example code:
{{< alert icon="🍅" context="success" text="HTML alert" />}}

Displays as:

However, when I create a markdown alert, it does not seem to come up with a background color.
Example code:
{{% alert icon="🍅" context="success" %}}
Markdown
{{% /alert %}}

Displays as:

I tried the markdown example with the cart icon in your doc as well, but it seems to not show a background color either.
Expected behavior
Have markdown alerts display the selected background color.
Screenshots
No response
Environment
- OS: macOS
- Browser: Chrome
- Browser Version: 116

Having trouble replicating this. Your example:
{{% alert icon="🍅" context="success" %}}
Markdown
{{% /alert %}}

renders as below for me (Chrome/FireFox on Windows/MacOS):

Do you see any warnings, errors, 404's in Chrome's dev tools? Is the alert code nested inside another Shortcode?

@colinwilson It looks like that the markdown alert is not assigned a css class, but HTML class are assigned one.
This is the example for the markdown alert

And this is the HTML alert output:

This shortcode is not used within a shortcode. It is placed right underneath the heading. I was checking the network tab in the Chrome developer tools, but I cannot see any errors. I am using Hugo 117 to build my site. I am not sure if it could be related to the Hugo version?

Ah, Apologies!
This is due to a lack of documentation on my part!
The <!-- raw HTML omitted --> in your screenshot indicates a missing (required) parameter in your hugo.toml config.
Set unsafe = true under [markup.goldmark.renderer] in your hugo.toml config like below:
[markup.goldmark.renderer]
    unsafe = true

Without the unsafe = true setting, raw HTML in a Goldmark-flavored Markdown source file will be replaced with <!-- raw HTML omitted --> in the markdownified destination file. This was a change made in Hugo v0.60.0 when Goldmark became the default library used for Markdown in Hugo. The markdown (%) alert shortcode won't render properly without this setting.

I'll document this ASAP and update the example hugo.toml in the repo.

ah awesome! this worked like a charm. Thank you so much! :)

You're welcome. 👍
I'll close this once I've updated the docs and repo to reflect the config requirements for the theme.


Native Federation: What about SSR?

Is there please a possiibility to configure or plan to support SSR in Native Federation Plugin?
Our stack is:

NG17
default SSR enabled (which is provided by Ng17 cli out of the box by default)

and we are trying this: https://www.angulararchitects.io/en/blog/micro-frontends-with-modern-angular-part-1-standalone-and-esbuild/
but getting this error when running ng serve shell:
Error: Schema validation failed with the following errors:   Data path "" must NOT have additional properties(server).

We currently don't have a sound SSR story. This will change soon with 17.1 or 17.2
Also, you are using the ApplicationBuilder -- this will be supported with 17.1
So far, we are supporting the -esbuild builder.

Thanks for the fast response, looking forward to 17.1 👍


Special character in hosts file is causing dnsmasq to malfunction

There's an entry in the hosts file with a special character, that's preventing dnsmasq from working, and therefore, Pi-Hole.
lifehacĸer.com
The "ĸ" is not an actual "k", is a special character.
"systemctl status dnsmasq" returns this error:
bad name at /etc/pihole/gravity.list line 74173
Which is this line:
lifehacĸer.com
Removing the offending simbol from the file solves the issue and make dnsmasq work again. The problem is that when Pi-Hole updates it comes back. Can you remove this entry from the hosts file?

This is an obvious phish, and we'll be seeing lots more of that, I fear.
Is the Pi-hole project aware of this problem?  https://github.com/pi-hole/pi-hole/issues

I was going to report to them, but since they use your list, they can't really fix the problem. Also, the issue is not in Pi-Hole, it seems that dnsmasq doesn't accept special characters.

This is because special characters need to be converted to a string that your DNS or web server can understand. Luckily Nginx already does this for me.
Take for instance xn--oogle-wmc.com
Put that in a browser and see what it changes to it says "Google.com" but it's not.

Have a look at the list of converted domains on my list that any DNS can understand.
https://github.com/StevenBlack/hosts/pull/248
You can nslookup on any of those funny domain strings and get an answer on them.

When I paste the address in the browser nothing happens.
If I understood correctly, it's possible to avoid the issue by converting the string? If that's the case, it's something that Steven could do?

Weird, I can do nslookups on those funny strings from any linux or Mac OS I've tested on or paste them into any of my browsers and it resolves.

lifehacĸer.com translates to lifehacĸer.com
nslookup xn--lifehacer-1rb.com
xn--lifehacer-1rb.com
	origin = dns1.yandex.net
	mail addr = iskalko.yandex.ru
	serial =<PHONE_NUMBER>
	refresh = 14400
	retry = 900
	expire = 1209600
	minimum = 14400
xn--lifehacer-1rb.com	mail exchanger = 10 mx.yandex.net.
Name:	xn--lifehacer-1rb.com
Address: <IP_ADDRESS>
xn--lifehacer-1rb.com	nameserver = dns2.yandex.net.
xn--lifehacer-1rb.com	text = "v=spf1 redirect=_spf.yandex.net"
xn--lifehacer-1rb.com	nameserver = dns1.yandex.net.

Nslookup works for me, but the address doesn't change in the browser, and also fails to resolve.
I'm using Fedora 25.

Nslookup works for me, but the address doesn't change when I paste it in the browser, and also doesn't resolve.
I'm using Fedora 25.

Interesting one this, I think it's a browser problem.
Just did some tests as follows:
Ubuntu:
Firefox and Chrome
Paste address lifehacĸer.com without http://
Opens page
nslookup - resolves
MacOS Sierra:
Firefox, Chrome, Safari
Paste address lifehacĸer.com without http://
Opens page
nslookup - resolves
Windows XP:
Firefox, Chrome
Paste address lifehacĸer.com without http://
Opens page
nslookup - resolves
Linux Mint:
Firefox, Chrome
Paste address lifehacĸer.com without http://
Opens page
nslookup - resolves
Android Phone: (here's an issue)
Chrome
See screengrab

Android on Chrome incorrectly translates what I type into the search bar.

Update:
Sent an email from my desktop to my android phone with the links in the email. Clicking either one of the links in the email and they both open in chrome on Android.
The fact that all nslookups resolve should be enough to satisfy that the hosts file will also understand the xn--- formats perfectly.

Pi-hole dev chipping in here! @drakofrost I've just updated my gravity.list to include new changes to this list and I'm not hitting the same error with lifehacĸer.com causing dnsmasq to fall over... what version of dnsmasq do you have?

@mitchellkrogza

lifehacĸer.com translates to lifehacĸer.com

I notice you include both xn--lifehacer-1rb.com and lifehacĸer.com in your list. If the latter just redirects to the former, then lifehacĸer.com is superfluous, surely?

Thanks, fixed in https://github.com/mitchellkrogza/Badd-Boyz-Hosts/commit/5001714c984350c67671e9c1e6806e73328a3126

@mitchellkrogza @PromoDaux @drakofrost thank you, everyone, for your diligence!
I just pushed the latest sources to master.

@mitchellkrogza It may have to do with the fact that I'm running Pi-Hole as a DNS server, maybe Pi-Hole is blocking those addresses.

@PromoFaux dnsmasq version 2.66, I'm running CentOS 7 on DigitalOcean.

dnsmasq version 2.66,
@drakofrost Possibly something to do with it. I know debian repositories have version 2.72, and I have no issues with that, even with the special chars.

Latest version is 2.76, which you can compile from source.

@PromoFaux I left the process of compiling stuff in the 90's :)

Go to Yandex.ru
Search for: lifehacĸer.com
1st result is there.
The ĸ in lifehacĸer is a letter of the Cyrillic script.

This one should be also fixed by #520 in future updates @StevenBlack :+1:

Thanks @funilrys.  Closing.


Show http://localhost:4280 as the main emulator URL

Is your feature request related to a problem? Please describe.
Currently, this is printed on startup:
[swa] Available on:
[swa]     http://<IP_ADDRESS>:4280
[swa]     http://<IP_ADDRESS>:4280

Describe the solution you'd like
Not sure how often those addresses are used. I'd imagine 99% of the time, we want folks to use http://localhost:4280. Consider changing this output to simply:
[swa] Available on: http://localhost:4280

Or remove it altogether and just add the URL to this message:
Azure Static Web Apps emulator started at http://localhost:4280 . Press CTRL+C to exit.

Without this, Azure Active Directory redirects while developing locally won't work at first glance because they don't allow http for anything but localhost. And if I try to use the urls listed in the SWA CLI, then I have to make sure locally I'm running with HTTPS. A customer has to really get AD and the client app to know what to change where.

@diberry for your particular use case, there is a --host and --port options to help you change the CLI default address and port:
swa start --host localhost --port 1234

@manekinekko That is only a fix if the client spins up with HTTPS too. Is there a flag for that? I don't have a certificate, I usually have the .env in React have https=true.
The issue is that if I want AAD to redirect to anything other than the literal "localhost", it has to be https. So an IP address has to be able to handle HTTPS.

You can enable SSL using the --ssl flag, however, for now you need to generate your own SSL certificate. The CLI does ship with its own SSL certificate for localhost (see https://github.com/Azure/static-web-apps-cli/tree/main/scripts). But for now we do required the users have their own SSL.
@anthonychu we may need to improve the SSL experience.

anything other than the literal "localhost"

Does this mean http://localhost works? If that’s true, you can get the CLI to start up at port 80.

... you can get the CLI to start up at port 80.

But then you will have to start the CLI as an administrator (or with root). because that's a limitation of the operating system.


Use nginx as the reverse proxy for firewatch service.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Fixed a bug with deleting a model from changelist inside modal

Summary
Fixes #6495

Coverage remained the same at 78.189% when pulling fab5d78e736f3e1e61355670507c0c597c6086bd on vxsx:bugfix/model-removal into b706cfa1c0d2e85d8c0b09b77c12d8087f9243a3 on divio:release/3.5.x.

@vxsx any idea on when this might get merged?


Is there a posibility to include Nvidia Jetson (ARM64)

Hi, I just started using this awesome project and replace our self-hosted runners,
since you're basically emulating an ARM-cpu and download raspberry images, is there a possibility to also include Jetson images ?

It should be doable, especially if they are really based on ubuntu linux.
This action considers partition 1 is boot and 2 is root, and the glob expression doesn't work with more than 9 partitions.
Apparently, the Jetson image I tried has 11 partitions. Do you know how they are organized and how they should be mounted?
https://github.com/pguyot/arm-runner-action/runs/6679373325?check_suite_focus=true#step:3:157

https://geekymind.de/jetson.png
looks like mmcblk0p1 is the main partition, the rest is just for the bootloader

Thanks. Got it working. Fixed in #39
Can you test it before we can call it a release?
The CI test is here:
https://github.com/pguyot/arm-runner-action/blob/main/.github/workflows/test-partitions.yml

mounts and starts great, but I have a strange issue, networking isn't working (using jetson-nano-image) I'll invenstigate, maybe we only need dhclient or something like that :

2022-06-02T22:43:28.1450723Z ##[group]Run pguyot/arm-runner-action@f204b58258d5b01e297eec6db036a14b11dc9ffa
2022-06-02T22:43:28.1451317Z with:
2022-06-02T22:43:28.1451991Z   base_image: https://developer.download.nvidia.com/embedded/L4T/r32_Release_v6.1/Jetson_Nano_2GB/jetson-nano-2gb-jp46-sd-card-image.zip
2022-06-02T22:43:28.1452697Z   cpu: cortex-a53
2022-06-02T22:43:28.1453038Z   rootpartition: 1
2022-06-02T22:43:28.1453524Z   commands: cat /etc/os-release
uname -a
ping google.de
sudo apt update
sudo apt upgrade

2022-06-02T22:43:28.1454047Z   image_additional_mb: 0
2022-06-02T22:43:28.1454421Z   bind_mount_repository: no
2022-06-02T22:43:28.1454766Z   copy_artifact_dest: .
2022-06-02T22:43:28.1455108Z   optimize_image: yes
2022-06-02T22:43:28.1455470Z   use_systemd_nspawn: no
2022-06-02T22:43:28.1455786Z   shell: /bin/sh
2022-06-02T22:43:28.1456104Z   exit_on_fail: yes
2022-06-02T22:43:28.1456424Z   debug: yes
2022-06-02T22:43:28.1456759Z   import_github_env: no
2022-06-02T22:43:28.1457085Z   export_github_env: no
2022-06-02T22:43:28.1457441Z env:
2022-06-02T22:43:28.1457743Z   BUILD_TYPE: Release
2022-06-02T22:43:28.1458071Z ##[endgroup]
2022-06-02T22:43:28.1713819Z ##[group]Run sudo apt-get update && sudo apt-get install -y qemu qemu-user-static binfmt-support parted wget dosfstools zip
2022-06-02T22:43:28.1714708Z sudo apt-get update && sudo apt-get install -y qemu qemu-user-static binfmt-support parted wget dosfstools zip
2022-06-02T22:43:28.1780681Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-06-02T22:43:28.1781168Z env:
2022-06-02T22:43:28.1781463Z   BUILD_TYPE: Release
2022-06-02T22:43:28.1781788Z ##[endgroup]
2022-06-02T22:43:28.3375105Z Hit:1 http://azure.archive.ubuntu.com/ubuntu focal InRelease
2022-06-02T22:43:28.3377574Z Get:2 http://azure.archive.ubuntu.com/ubuntu focal-updates InRelease [114 kB]
2022-06-02T22:43:28.3386334Z Get:3 http://azure.archive.ubuntu.com/ubuntu focal-backports InRelease [108 kB]
2022-06-02T22:43:28.3410425Z Get:4 http://azure.archive.ubuntu.com/ubuntu focal-security InRelease [114 kB]
2022-06-02T22:43:28.3929991Z Get:5 https://packages.microsoft.com/ubuntu/20.04/prod focal InRelease [10.5 kB]
2022-06-02T22:43:28.5879063Z Hit:6 http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu focal InRelease
2022-06-02T22:43:28.6065525Z Get:7 http://azure.archive.ubuntu.com/ubuntu focal-updates/main amd64 Packages [1833 kB]
2022-06-02T22:43:28.6391408Z Get:8 http://azure.archive.ubuntu.com/ubuntu focal-updates/main Translation-en [337 kB]
2022-06-02T22:43:28.6426357Z Get:9 http://azure.archive.ubuntu.com/ubuntu focal-updates/main amd64 c-n-f Metadata [15.4 kB]
2022-06-02T22:43:28.6463785Z Get:10 http://azure.archive.ubuntu.com/ubuntu focal-updates/restricted amd64 Packages [1010 kB]
2022-06-02T22:43:28.6616828Z Get:11 http://azure.archive.ubuntu.com/ubuntu focal-updates/restricted Translation-en [144 kB]
2022-06-02T22:43:28.6782987Z Get:12 http://azure.archive.ubuntu.com/ubuntu focal-updates/universe amd64 Packages [926 kB]
2022-06-02T22:43:28.6843543Z Get:13 http://azure.archive.ubuntu.com/ubuntu focal-updates/universe Translation-en [208 kB]
2022-06-02T22:43:28.6895651Z Get:14 http://azure.archive.ubuntu.com/ubuntu focal-updates/universe amd64 c-n-f Metadata [20.7 kB]
2022-06-02T22:43:28.7635820Z Get:15 http://azure.archive.ubuntu.com/ubuntu focal-backports/universe amd64 Packages [23.7 kB]
2022-06-02T22:43:28.7652739Z Get:16 http://azure.archive.ubuntu.com/ubuntu focal-backports/universe amd64 c-n-f Metadata [860 B]
2022-06-02T22:43:28.9253114Z Get:17 http://azure.archive.ubuntu.com/ubuntu focal-security/main amd64 Packages [1491 kB]
2022-06-02T22:43:28.9514668Z Get:18 http://azure.archive.ubuntu.com/ubuntu focal-security/main Translation-en [257 kB]
2022-06-02T22:43:28.9519646Z Get:19 http://azure.archive.ubuntu.com/ubuntu focal-security/main amd64 c-n-f Metadata [10.4 kB]
2022-06-02T22:43:28.9545951Z Get:20 http://azure.archive.ubuntu.com/ubuntu focal-security/restricted amd64 Packages [948 kB]
2022-06-02T22:43:28.9675020Z Get:21 http://azure.archive.ubuntu.com/ubuntu focal-security/restricted Translation-en [135 kB]
2022-06-02T22:43:28.9742791Z Get:22 http://azure.archive.ubuntu.com/ubuntu focal-security/universe amd64 Packages [705 kB]
2022-06-02T22:43:28.9867853Z Get:23 http://azure.archive.ubuntu.com/ubuntu focal-security/universe Translation-en [126 kB]
2022-06-02T22:43:28.9879888Z Get:24 http://azure.archive.ubuntu.com/ubuntu focal-security/universe amd64 c-n-f Metadata [14.5 kB]
2022-06-02T22:43:29.0622012Z Get:25 https://packages.microsoft.com/ubuntu/20.04/prod focal/main amd64 Packages [161 kB]
2022-06-02T22:43:35.4390452Z Fetched 8711 kB in 2s (4549 kB/s)
2022-06-02T22:43:36.5074377Z Reading package lists...
2022-06-02T22:43:36.6064760Z Reading package lists...
2022-06-02T22:43:36.8142032Z Building dependency tree...
2022-06-02T22:43:36.8158628Z Reading state information...
2022-06-02T22:43:36.9895304Z dosfstools is already the newest version (4.1-2).
2022-06-02T22:43:36.9896975Z dosfstools set to manually installed.
2022-06-02T22:43:36.9898794Z zip is already the newest version (3.0-11build1).
2022-06-02T22:43:36.9899993Z binfmt-support is already the newest version (2.2.0-2).
2022-06-02T22:43:36.9902309Z binfmt-support set to manually installed.
2022-06-02T22:43:36.9903534Z parted is already the newest version (3.3-4ubuntu<IP_ADDRESS>).
2022-06-02T22:43:36.9904942Z parted set to manually installed.
2022-06-02T22:43:36.9906056Z wget is already the newest version (1.20.3-1ubuntu2).
2022-06-02T22:43:37.0640670Z The following NEW packages will be installed:
2022-06-02T22:43:37.0641700Z   qemu qemu-user-static
2022-06-02T22:43:37.1062661Z 0 upgraded, 2 newly installed, 0 to remove and 22 not upgraded.
2022-06-02T22:43:37.1748944Z Need to get 21.3 MB of archives.
2022-06-02T22:43:37.1750145Z After this operation, 208 MB of additional disk space will be used.
2022-06-02T22:43:37.1751545Z Get:1 http://azure.archive.ubuntu.com/ubuntu focal-updates/main amd64 qemu amd64 1:4.2-3ubuntu6.21 [15.2 kB]
2022-06-02T22:43:37.1959289Z Get:2 http://azure.archive.ubuntu.com/ubuntu focal-updates/universe amd64 qemu-user-static amd64 1:4.2-3ubuntu6.21 [21.2 MB]
2022-06-02T22:43:37.9460128Z Fetched 21.3 MB in 0s (64.7 MB/s)
2022-06-02T22:43:37.9875873Z Selecting previously unselected package qemu.
2022-06-02T22:43:38.0160478Z (Reading database ...
2022-06-02T22:43:38.0160942Z (Reading database ... 5%
2022-06-02T22:43:38.0161769Z (Reading database ... 10%
2022-06-02T22:43:38.0163622Z (Reading database ... 15%
2022-06-02T22:43:38.0179999Z (Reading database ... 20%
2022-06-02T22:43:38.0180320Z (Reading database ... 25%
2022-06-02T22:43:38.0180750Z (Reading database ... 30%
2022-06-02T22:43:38.0181017Z (Reading database ... 35%
2022-06-02T22:43:38.0181268Z (Reading database ... 40%
2022-06-02T22:43:38.0181545Z (Reading database ... 45%
2022-06-02T22:43:38.0181810Z (Reading database ... 50%
2022-06-02T22:43:38.0642575Z (Reading database ... 55%
2022-06-02T22:43:38.1104137Z (Reading database ... 60%
2022-06-02T22:43:38.1506974Z (Reading database ... 65%
2022-06-02T22:43:38.1750250Z (Reading database ... 70%
2022-06-02T22:43:38.2271599Z (Reading database ... 75%
2022-06-02T22:43:38.2842818Z (Reading database ... 80%
2022-06-02T22:43:38.3821147Z (Reading database ... 85%
2022-06-02T22:43:38.4736766Z (Reading database ... 90%
2022-06-02T22:43:38.5330880Z (Reading database ... 95%
2022-06-02T22:43:38.5331262Z (Reading database ... 100%
2022-06-02T22:43:38.5331608Z (Reading database ... 235546 files and directories currently installed.)
2022-06-02T22:43:38.5456452Z Preparing to unpack .../qemu_1%3a4.2-3ubuntu6.21_amd64.deb ...
2022-06-02T22:43:38.5482481Z Unpacking qemu (1:4.2-3ubuntu6.21) ...
2022-06-02T22:43:38.5800421Z Selecting previously unselected package qemu-user-static.
2022-06-02T22:43:38.6072468Z Preparing to unpack .../qemu-user-static_1%3a4.2-3ubuntu6.21_amd64.deb ...
2022-06-02T22:43:38.6096481Z Unpacking qemu-user-static (1:4.2-3ubuntu6.21) ...
2022-06-02T22:43:41.4132971Z Setting up qemu (1:4.2-3ubuntu6.21) ...
2022-06-02T22:43:41.4193358Z Setting up qemu-user-static (1:4.2-3ubuntu6.21) ...
2022-06-02T22:43:41.4612446Z Processing triggers for man-db (2.9.1-1) ...
2022-06-02T22:43:45.0058823Z ##[group]Run case cortex-a53 in
2022-06-02T22:43:45.0059186Z case cortex-a53 in
2022-06-02T22:43:45.0059436Z "")
2022-06-02T22:43:45.0060018Z     cpu="arm1176:cortex-a53"
2022-06-02T22:43:45.0060379Z ;;
2022-06-02T22:43:45.0060833Z arm1176)
2022-06-02T22:43:45.0061093Z     cpu="arm1176:cortex-a53"
2022-06-02T22:43:45.0061331Z ;;
2022-06-02T22:43:45.0061559Z cortex-a7)
2022-06-02T22:43:45.0061833Z     cpu="cortex-a7:cortex-a53"
2022-06-02T22:43:45.0062096Z ;;
2022-06-02T22:43:45.0062308Z cortex-a8)
2022-06-02T22:43:45.0062572Z     cpu="cortex-a8:max"
2022-06-02T22:43:45.0062815Z ;;
2022-06-02T22:43:45.0063028Z cortex-a53)
2022-06-02T22:43:45.0063289Z     cpu="max:cortex-a53"
2022-06-02T22:43:45.0063534Z ;;
2022-06-02T22:43:45.0063729Z *)
2022-06-02T22:43:45.0063966Z     cpu=cortex-a53
2022-06-02T22:43:45.0064207Z esac
2022-06-02T22:43:45.0064429Z arm_cpu=${cpu%:}
2022-06-02T22:43:45.0064690Z aarch64_cpu=${cpu#:}
2022-06-02T22:43:45.0065035Z sudo mv /usr/bin/qemu-arm-static /usr/bin/qemu-arm-static0
2022-06-02T22:43:45.0065448Z sudo mv /usr/bin/qemu-aarch64-static /usr/bin/qemu-aarch64-static0
2022-06-02T22:43:45.0066059Z sudo gcc -static /home/runner/work/_actions/pguyot/arm-runner-action/f204b58258d5b01e297eec6db036a14b11dc9ffa/qemu-wrapper.c -DQEMU_CPU=${arm_cpu} -O3 -s -o /usr/bin/qemu-arm-static
2022-06-02T22:43:45.0066823Z sudo gcc -static /home/runner/work/_actions/pguyot/arm-runner-action/f204b58258d5b01e297eec6db036a14b11dc9ffa/qemu-wrapper.c -DQEMU_CPU=${aarch64_cpu} -O3 -s -o /usr/bin/qemu-aarch64-static
2022-06-02T22:43:45.0067520Z sudo update-binfmts --disable qemu-arm
2022-06-02T22:43:45.0067864Z sudo update-binfmts --disable qemu-aarch64
2022-06-02T22:43:45.0068223Z sudo update-binfmts --enable qemu-arm
2022-06-02T22:43:45.0068569Z sudo update-binfmts --enable qemu-aarch64
2022-06-02T22:43:45.0130041Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-06-02T22:43:45.0130356Z env:
2022-06-02T22:43:45.0130571Z   BUILD_TYPE: Release
2022-06-02T22:43:45.0130815Z ##[endgroup]
2022-06-02T22:43:45.9094910Z ##[group]Run bash /home/runner/work/_actions/pguyot/arm-runner-action/f204b58258d5b01e297eec6db036a14b11dc9ffa/download_image.sh https://developer.download.nvidia.com/embedded/L4T/r32_Release_v6.1/Jetson_Nano_2GB/jetson-nano-2gb-jp46-sd-card-image.zip
2022-06-02T22:43:45.9096001Z bash /home/runner/work/_actions/pguyot/arm-runner-action/f204b58258d5b01e297eec6db036a14b11dc9ffa/download_image.sh https://developer.download.nvidia.com/embedded/L4T/r32_Release_v6.1/Jetson_Nano_2GB/jetson-nano-2gb-jp46-sd-card-image.zip
2022-06-02T22:43:45.9159130Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-06-02T22:43:45.9159472Z env:
2022-06-02T22:43:45.9159692Z   BUILD_TYPE: Release
2022-06-02T22:43:45.9159940Z ##[endgroup]
2022-06-02T22:44:15.1225348Z Archive:  jetson-nano-2gb-jp46-sd-card-image.zip
2022-06-02T22:47:14.9499484Z   inflating: sd-blob.img
2022-06-02T22:47:15.0453157Z ##[group]Run sudo bash /home/runner/work/_actions/pguyot/arm-runner-action/f204b58258d5b01e297eec6db036a14b11dc9ffa/mount_image.sh /home/runner/work/_temp/arm-runner/arm-runner.img 0 no 1
2022-06-02T22:47:15.0454080Z sudo bash /home/runner/work/_actions/pguyot/arm-runner-action/f204b58258d5b01e297eec6db036a14b11dc9ffa/mount_image.sh /home/runner/work/_temp/arm-runner/arm-runner.img 0 no 1
2022-06-02T22:47:15.0521443Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-06-02T22:47:15.0521813Z env:
2022-06-02T22:47:15.0522227Z   BUILD_TYPE: Release
2022-06-02T22:47:15.0522556Z ##[endgroup]
2022-06-02T22:47:15.1209379Z Created loopback device /dev/loop3
2022-06-02T22:47:17.8984715Z /dev/loop3: gpt partitions 2 3 4 5 6 7 8 9 10 11 12 13 14 1
2022-06-02T22:47:17.9551074Z mount: /home/actions/temp/arm-runner/mnt/boot: wrong fs type, bad option, bad superblock on , missing codepage or helper program, or other error.
2022-06-02T22:47:17.9900274Z cp: cannot stat '/home/actions/temp/arm-runner/mnt/etc/resolv.conf': No such file or directory
2022-06-02T22:47:17.9919170Z cp: not writing through dangling symlink '/home/actions/temp/arm-runner/mnt/etc/resolv.conf'
2022-06-02T22:47:18.0574238Z ##[group]Run case "yes" in
2022-06-02T22:47:18.0574834Z case "yes" in
2022-06-02T22:47:18.0575165Z yes|true)
2022-06-02T22:47:18.0575484Z     debug='x'
2022-06-02T22:47:18.0575766Z     set -x
2022-06-02T22:47:18.0576118Z ;;
2022-06-02T22:47:18.0576422Z no|false)
2022-06-02T22:47:18.0576685Z     debug=''
2022-06-02T22:47:18.0576991Z ;;
2022-06-02T22:47:18.0577328Z esac
2022-06-02T22:47:18.0577599Z repository_path=
2022-06-02T22:47:18.0577986Z if [ "${repository_path}x" = "x" ]; then
2022-06-02T22:47:18.0578505Z     repository_name=basename /home/runner/work/Open.HD/Open.HD
2022-06-02T22:47:18.0578908Z     repository_path=/${repository_name}
2022-06-02T22:47:18.0579308Z fi
2022-06-02T22:47:18.0579620Z case "yes" in
2022-06-02T22:47:18.0579890Z yes|true)
2022-06-02T22:47:18.0580216Z     exit_on_fail='e'
2022-06-02T22:47:18.0580785Z ;;
2022-06-02T22:47:18.0581221Z no|false)
2022-06-02T22:47:18.0581549Z     exit_on_fail=''
2022-06-02T22:47:18.0581855Z ;;
2022-06-02T22:47:18.0582293Z esac
2022-06-02T22:47:18.0582543Z shell=/bin/sh
2022-06-02T22:47:18.0582900Z shell_package=
2022-06-02T22:47:18.0583283Z [ -x /home/actions/temp/arm-runner/mnt/${shell} ] || \
2022-06-02T22:47:18.0583714Z shell_path=$(sudo chroot /home/actions/temp/arm-runner/mnt which ${shell}) || \
2022-06-02T22:47:18.0584161Z case ${shell} in
2022-06-02T22:47:18.0584510Z bash|sh)
2022-06-02T22:47:18.0584758Z     :
2022-06-02T22:47:18.0585042Z ;;
2022-06-02T22:47:18.0585346Z *)
2022-06-02T22:47:18.0585739Z     sudo chroot /home/actions/temp/arm-runner/mnt sudo apt-get install -y ${shell_package:-${shell##/}}
2022-06-02T22:47:18.0586211Z ;;
2022-06-02T22:47:18.0586499Z esac
2022-06-02T22:47:18.0586875Z shell_path=${shell_path:-$(sudo chroot /home/actions/temp/arm-runner/mnt which ${shell})}
2022-06-02T22:47:18.0587443Z sudo mkdir -p $(dirname /home/actions/temp/arm-runner/mnt${repository_path})
2022-06-02T22:47:18.0587896Z case no in
2022-06-02T22:47:18.0588239Z yes|true)
2022-06-02T22:47:18.0588593Z     sudo mkdir -p /home/actions/temp/arm-runner/mnt${repository_path}
2022-06-02T22:47:18.0589166Z     sudo mount --bind /home/runner/work/Open.HD/Open.HD /home/actions/temp/arm-runner/mnt${repository_path}
2022-06-02T22:47:18.0589614Z ;;
2022-06-02T22:47:18.0589861Z no|false)
2022-06-02T22:47:18.0590345Z     sudo cp -Rp /home/runner/work/Open.HD/Open.HD /home/actions/temp/arm-runner/mnt${repository_path}
2022-06-02T22:47:18.0590783Z ;;
2022-06-02T22:47:18.0591091Z esac
2022-06-02T22:47:18.0591383Z if [ "nox" != "x" -a "nox" != "nox" ]; then
2022-06-02T22:47:18.0591792Z     chroot_script_dir=/scripts
2022-06-02T22:47:18.0592224Z     script_dir=${RUNNER_TEMP:-/home/actions/temp}/scripts
2022-06-02T22:47:18.0592566Z else
2022-06-02T22:47:18.0592902Z     chroot_script_dir=/tmp
2022-06-02T22:47:18.0593312Z     script_dir=/home/actions/temp/arm-runner/mnt/tmp
2022-06-02T22:47:18.0593658Z fi
2022-06-02T22:47:18.0593984Z mkdir -p ${script_dir}
2022-06-02T22:47:18.0594391Z script_path=${script_dir}/commands.sh
2022-06-02T22:47:18.0594937Z sudo touch ${script_path}
2022-06-02T22:47:18.0595254Z sudo chmod o+wx ${script_path}
2022-06-02T22:47:18.0595676Z echo "#!${shell_path}" > ${script_path}
2022-06-02T22:47:18.0596109Z echo "set -${debug}${exit_on_fail}" >> ${script_path}
2022-06-02T22:47:18.0596437Z case "no" in
2022-06-02T22:47:18.0596833Z yes|true)
2022-06-02T22:47:18.0597416Z     export | sed -e 's/^declare -x //g;s/^[^=]$/\0=""/g;s/='''(.*)'''$/=\1/g' >> ${script_dir}/environment.sh
2022-06-02T22:47:18.0597980Z     echo ". ${chroot_script_dir}/environment.sh" >> ${script_path}
2022-06-02T22:47:18.0598320Z ;;
2022-06-02T22:47:18.0598618Z esac
2022-06-02T22:47:18.0598927Z case "no" in
2022-06-02T22:47:18.0599220Z yes|true)
2022-06-02T22:47:18.0599639Z     echo "GITHUB_ENV=${chroot_script_dir}/github_env.sh" >> ${script_path}
2022-06-02T22:47:18.0600036Z ;;
2022-06-02T22:47:18.0600331Z esac
2022-06-02T22:47:18.0600643Z echo "cd ${repository_path}" >> ${script_path}
2022-06-02T22:47:18.0601127Z cat >> ${script_path} <<"ARM_RUNNER_INPUT_COMMANDS_EOF"
2022-06-02T22:47:18.0601528Z cat /etc/os-release
2022-06-02T22:47:18.0601810Z uname -a
2022-06-02T22:47:18.0602152Z ping google.de
2022-06-02T22:47:18.0602532Z sudo apt update
2022-06-02T22:47:18.0602814Z sudo apt upgrade
2022-06-02T22:47:18.0603131Z
2022-06-02T22:47:18.0603469Z ARM_RUNNER_INPUT_COMMANDS_EOF
2022-06-02T22:47:18.0603805Z if [ "nox" != "x" -a "nox" != "nox" ]; then
2022-06-02T22:47:18.0604352Z     sudo apt-get install -y systemd-container
2022-06-02T22:47:18.0604955Z     sudo systemd-nspawn -q -a --bind=${script_dir}:${chroot_script_dir} -D /home/actions/temp/arm-runner/mnt ${shell_path} ${chroot_script_dir}/commands.sh
2022-06-02T22:47:18.0605478Z else
2022-06-02T22:47:18.0605850Z     sudo chroot /home/actions/temp/arm-runner/mnt ${chroot_script_dir}/commands.sh
2022-06-02T22:47:18.0606276Z fi
2022-06-02T22:47:18.0606602Z rc=$?
2022-06-02T22:47:18.0606905Z [ -f ${script_dir}/github_env.sh ] && \
2022-06-02T22:47:18.0734228Z     cat ${script_dir}/github_env.sh >> $GITHUB_ENV
2022-06-02T22:47:18.0734573Z exit $rc
2022-06-02T22:47:18.0794082Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-06-02T22:47:18.0794393Z env:
2022-06-02T22:47:18.0794624Z   BUILD_TYPE: Release
2022-06-02T22:47:18.0794856Z ##[endgroup]
2022-06-02T22:47:18.0887776Z + repository_path=
2022-06-02T22:47:18.0891136Z + '[' x = x ']'
2022-06-02T22:47:18.0898067Z ++ basename /home/runner/work/Open.HD/Open.HD
2022-06-02T22:47:18.0910098Z + repository_name=Open.HD
2022-06-02T22:47:18.0910613Z + repository_path=/Open.HD
2022-06-02T22:47:18.0914498Z + case "yes" in
2022-06-02T22:47:18.0914931Z + exit_on_fail=e
2022-06-02T22:47:18.0917422Z + shell=/bin/sh
2022-06-02T22:47:18.0917825Z + shell_package=
2022-06-02T22:47:18.0921080Z + '[' -x /home/actions/temp/arm-runner/mnt//bin/sh ']'
2022-06-02T22:47:18.0951693Z ++ sudo chroot /home/actions/temp/arm-runner/mnt which /bin/sh
2022-06-02T22:47:18.2008401Z + shell_path=/bin/sh
2022-06-02T22:47:18.2012331Z ++ dirname /home/actions/temp/arm-runner/mnt/Open.HD
2022-06-02T22:47:18.2026356Z + sudo mkdir -p /home/actions/temp/arm-runner/mnt
2022-06-02T22:47:18.2106380Z + case no in
2022-06-02T22:47:18.2107122Z + sudo cp -Rp /home/runner/work/Open.HD/Open.HD /home/actions/temp/arm-runner/mnt/Open.HD
2022-06-02T22:47:18.2195016Z + '[' nox '!=' x -a nox '!=' nox ']'
2022-06-02T22:47:18.2195523Z + chroot_script_dir=/tmp
2022-06-02T22:47:18.2196031Z + script_dir=/home/actions/temp/arm-runner/mnt/tmp
2022-06-02T22:47:18.2196564Z + mkdir -p /home/actions/temp/arm-runner/mnt/tmp
2022-06-02T22:47:18.2209599Z + script_path=/home/actions/temp/arm-runner/mnt/tmp/commands.sh
2022-06-02T22:47:18.2210614Z + sudo touch /home/actions/temp/arm-runner/mnt/tmp/commands.sh
2022-06-02T22:47:18.2292256Z + sudo chmod o+wx /home/actions/temp/arm-runner/mnt/tmp/commands.sh
2022-06-02T22:47:18.2367716Z + echo '#!/bin/sh'
2022-06-02T22:47:18.2377434Z + echo 'set -xe'
2022-06-02T22:47:18.2377823Z + case "no" in
2022-06-02T22:47:18.2378083Z + case "no" in
2022-06-02T22:47:18.2378385Z + echo 'cd /Open.HD'
2022-06-02T22:47:18.2378608Z + cat
2022-06-02T22:47:18.2394875Z + '[' nox '!=' x -a nox '!=' nox ']'
2022-06-02T22:47:18.2395609Z + sudo chroot /home/actions/temp/arm-runner/mnt /tmp/commands.sh
2022-06-02T22:47:18.2754874Z + cd /Open.HD
2022-06-02T22:47:18.2765292Z + cat /etc/os-release
2022-06-02T22:47:18.3177984Z NAME="Ubuntu"
2022-06-02T22:47:18.3178350Z VERSION="18.04.5 LTS (Bionic Beaver)"
2022-06-02T22:47:18.3178622Z ID=ubuntu
2022-06-02T22:47:18.3178846Z ID_LIKE=debian
2022-06-02T22:47:18.3179099Z PRETTY_NAME="Ubuntu 18.04.5 LTS"
2022-06-02T22:47:18.3179365Z VERSION_ID="18.04"
2022-06-02T22:47:18.3179679Z HOME_URL="https://www.ubuntu.com/"
2022-06-02T22:47:18.3180002Z SUPPORT_URL="https://help.ubuntu.com/"
2022-06-02T22:47:18.3180370Z BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
2022-06-02T22:47:18.3181396Z PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
2022-06-02T22:47:18.3181800Z VERSION_CODENAME=bionic
2022-06-02T22:47:18.3182050Z UBUNTU_CODENAME=bionic
2022-06-02T22:47:18.3216330Z + uname -a
2022-06-02T22:47:18.3560562Z Linux fv-az177-982 5.13.0-1025-azure #29~20.04.1-Ubuntu SMP Thu May 19 14:50:45 UTC 2022 aarch64 aarch64 aarch64 GNU/Linux
2022-06-02T22:47:18.3585908Z + ping google.de
2022-06-02T22:47:18.4281143Z ping: google.de: Name or service not known
2022-06-02T22:47:18.4350989Z ##[error]Process completed with exit code 2.

After few attempts, I was able to get it right. Could you please test it again?
BTW, you probably want to ping with -c 1.

looks great, didn't find any bugs, really big thank you :)

Thanks for testing. Just released that as v2.4.0


ENH: condense decorators e.g. @cython.cfunc(returns=foo, nogil=True, exceptval=-1)

Looking at converting some cython-syntax code to be valid python, what is one line in cython becomes much more verbose in pure-python mode:
cdef bint is_leapyear(int64_t year) nogil:

@cython.nogil
@cython.returns(bint)
@cython.cfunc
def is_leapyear(year: int64_t):

The decorators here could reasonably be condensed to something like @cython.cfunc(nogil=True, returns=bint)
(ideally the @cython.returns(bint) could be specified python-style by -> bint:, but thats a separate issue)

Yeah, the @cython.returns() isn't really needed any more, now that return type annotations can be used.
That reduces the gain to @cython.cfunc(nogil=True). Is that still worth it?

In this example its less of a big deal, but there's also exceptval and potentially wraparound and boundscheck


Tickets/sitcom 1189

Include results from SITCOM-1159 analysis

Everything looks good! I just have one comment.
According to Figure 2, slews have amplitude of at least 50 deg, as seen in Elevation Difference axis. Requirements specified in TS-88-REQ-0052, which are plotted  in the figures, are for slews of amplitudes less than 3.5 deg. For larger amplitudes we have LTS-88-REQ-0053 which is a bit more flexible the specifications.
According to the data used, we are actually analyzing LTS-88-REQ-0053 and not LTS-88-REQ-0052.
Given that you made modificatons to the plots in sections "Slewing analysis - Positioning", "Slewing in Azimuth Analysis - Positioning", and "Position and Rotation Repeatibility for a Specific Target Elevation.",  I propose changing the limits of the requirements in the plots and verify if the difference still out of tolerance intervals, and repeating the analysis for LTS-88-REQ-0052 with other dataset (with just small-amplitude slews) in a new ticket.
Maybe changing the limits can be another ticket by itself. What are @b1quint thoughts of this?
Appart from that, I'd say that the specifications for SITCOM-1189 are met.

I believe that a discussion during our meeting could help clarify things (unfortunately, I will have to miss it). We might want to split the analysis if we have different requirements. However, I would do this work by separating the requirements and analysis in a new PR.

The original analysis detailed in this technical note correspond to SITCOM-797 that doesn't distinguish between short and long slews. In fact I just discover that there is such a distinction (I didn't know about LTS-88 until you mentioned it).
It should definitely be taken into account but I suggest to proceed incrementally and to merge this version of the TN and then to open a separate ticket.
We already know that the piston requirement for long slew in elevation is not met (fig. 1)
Please tell me if you agree and if I go ahead to merge this branch

I completely agree with this, proceed with the merge.


CROSSSPAWN-8303230: Regular Expression Denial of Service (ReDoS) affecting cross-spawn package in versión 11.0.0 of Glob

There is a vulnerability in version 11.0.0 of Glob originating from the cross-spawn 7.0.4 (CVE-2024-21538).
I am requesting a fix, as the cross-spawn library has already addressed this issue in version 7.0.5.
I appreciate your prompt attention to this matter. I look forward to a resolution.

I have a PR open to fix the vuln in the relevant dependency: https://github.com/tapjs/foreground-child/pull/60


Remove main method calls

Update the reference for user.dir and main method from below file.
https://github.com/irmein/common-util/blob/master/src/main/java/com/merker/common/util/app/ExcelWrapper.java

Main mthod removed.

The file invocation is updated as command line arguments.


Added a notification for incoming private messages

Added a notification for incoming private messages. Fixes #7687.

How does this behave with chathistory when world hopping?

This will send notification for every private message from chat history plugin


Add Liquidity analysis section to tearsheet

At first just a simple table of data for every security the algo trades.
3 columns per ticker:

Average Daily Trading Volume over last 3-months
Minimum daily volume
Maximum # of shares the algo ever placed an order for

The volume and transactions tear sheet should support this analysis. There is also an illiquidity exposure plot in the risk tear sheet.


Peer to peer code review

You have done amazing work. The only thing I could suggest is your readme file. "Live demo" link is missing. If you could consider this. Other than that, everything else is good.

@AFakhriddinov Thanks for your suggestions. I will work on that


Response character set error

In file ResponseHandler.cs line 105
response.Response.CharacterSet = response.Response.ResponseHeaders[i].Value.Split(';')[1].ToLower().Replace("charset=", string.Empty).Trim();

On machines which have different locale settings other than english, ToLower() function causes problem like it converts ISO-8859-9 to ıso-8859-9 which is Turkish and not a correct defininition for char set.
Please notice that ıso-8859-9 is not iso-8859-1.
I removed ToLower() like below and it works.
response.Response.CharacterSet = response.Response.ResponseHeaders[i].Value.Split(';')[1].Replace("charset=", string.Empty).Trim();

This has been address in release branch


[SymbolGraph] Put extending declarations in rootmost module

When extending another module's type in your module, serialize declarations in
the extension into the other module's "extension" symbol graph file, including
relationships. This mechanic should continue up to the rootmost module. For
example:
A.AStruct <- B.BStruct < C.CStruct
Both BStruct and CStruct should go in @A symbol graph files because AStruct
owns BStruct and by extension owns CStruct. This is reflected in
documentation curation in some form already.
rdar://60796811

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test OS X Platform

@swift-ci Please smoke test Windows platform

Windows failure is unrelated.


automatically deploy binary releases to github releases upon tag

I couldn't figure out a way to test this locally so there may be some trial and error cutting releases to make sure this works.

related to #245

lgtm


Update rust and dependants

This updates:

rust
rustUnstable
rustRegistry
cargo
rustfmt

PR is tested

I'd like to cherry-pick this to release-15.09 if it fixes transient builds for cargo on i686

@wkennington apparently did most of this stuff already just now, I cannot guarantee that this will fix the cargo failures, but possibly it could, will have to see.

I'll rebase on those changes

What about racer?

Yep! racer WIP, will push onto here in a sec.

@domenkozar https://hydra.glob.in/build/1025/nixlog/2 this seems to fix cargo, didn't try to update dependencies.


Add Serialization property to Kafka Config in Turing API

This PR adds the SerializationFormat property to the Kafka Config processed by the Turing API, which will allow for setting this value on the Turing Router (implemented in PR).
Originally, Kafka result logging was only supported in JSON format. With this change, result logging to Kafka topics can be configured for one of 'json' or 'protobuf' formats.
Note on DB Migration:
When these changes are integrated and deployed, the data in the existing Turing database would need to be updated to set serialization_format: "json" in the Kafka Config, for the router versions using Kafka as the result logger.

I just noticed that the same SerializationFormat constants are defined in engines/router/missionctl/config/config.go in https://github.com/gojek/turing/pull/31/files. Can we consider re-using the same types/constants to avoid any possible discrepancies?

LGTM. I'm not quite clear on that, but shouldn't you add a migration here: https://github.com/gojek/turing/tree/main/api/db-migrations so it's applied in E2E test?

The DB columns remain the same only the value in router_versions.log_config.kafka_config needs to be updated. E2E tests are not affected by this.

I just noticed that the same SerializationFormat constants are defined in engines/router/missionctl/config/config.go in https://github.com/gojek/turing/pull/31/files. Can we consider re-using the same types/constants to avoid any possible discrepancies?

Can. I will make another PR to clean it up along with ResultLogger types and some other mentions of experiment engine types.


Remove Authentication from name of Authentication.cs classes

Is your feature request related to a problem? Please describe.
Related to #174. Turns out we aren't really following the Microsoft convention regarding some of our naming. A lot of our classes have Authentication in their name, for example: BankIdAuthenticationDefaults.cs

Microsoft does not include Authentication for any of their auth classes.
By shortening them it would be easier to read and remove redundant code, the Authentication part is still there but based on the namespace.
What area is it related to
These packages have classes that can be renamed:

ActiveLogin.Authentication.BankId.AspNetCore
ActiveLogin.Authentication.BankId.AspNetCore.Azure
ActiveLogin.Authentication.BankId.AspNetCore.QRCoder
ActiveLogin.Authentication.GrandId.AspNetCore

Describe the solution you'd like
Remove the word Authentication from our classes. So, for example:
BankIdAuthenticationDefaults.cs would become BankIdDefaults.cs
GrandIdAuthenticationDefaults.cs would become GrandIdDefaults.cs
Additional context
It would be a breaking change, but will only affect users that have started to make changes and customizations to Active Login. None of these classes are needed to run Active Login, but can and should be used when customizing. I think it's still worth doing and the sooner the better.

Draft available at #177.


[~] fix: [grains] Grains template now uses u64 type, that can hold expected 2^64-1

Why? Right now there is an overflow situation, that accidently doesn't show up, since vlang treats 2^64 as 0 adding -1 seems to be the same as 2^64-1.

I'm not a V maintainer, but I'm support of this change.
We also use uint64 for this exercise in Go.