[20.03] curl: update CVE patch hashes for extra digits added to index line

Motivation for this change
The fetched CVE patches for curl changed so the hashes no longer agree.
Things done
I diffed the old and new versions. It seems github just added extra digits to the index lines. For example
diff -U2 /nix/store/4psjdyqqf4wb4cv8ijds35cpqnmyjd5p-CVE-2020-8169.patch /nix/store/rsx49gzgb1fgylyh8cggnd92nx0hl9gw-CVE-2020-8169.patch

--- /nix/store/4psjdyqqf4wb4cv8ijds35cpqnmyjd5p-CVE-2020-8169.patch	1969-12-31 19:00:01.000000000 -0500
+++ /nix/store/rsx49gzgb1fgylyh8cggnd92nx0hl9gw-CVE-2020-8169.patch	1969-12-31 19:00:01.000000000 -0500
@@ -17,5 +17,5 @@

 diff --git a/lib/url.c b/lib/url.c
-index f250f2ff20a..9b8b2bdde64 100644
+index f250f2ff20a3..9b8b2bdde640 100644
 --- a/lib/url.c
 +++ b/lib/url.c
@@ -38,5 +38,5 @@
    }
 diff --git a/tests/data/Makefile.inc b/tests/data/Makefile.inc
-index 004a90b2360..bb6bf0f2fd0 100644
+index 004a90b23608..bb6bf0f2fd06 100644
 --- a/tests/data/Makefile.inc
 +++ b/tests/data/Makefile.inc
@@ -51,5 +51,5 @@
 diff --git a/tests/data/test1168 b/tests/data/test1168
 new file mode 100644
-index<PHONE_NUMBER>0..283e91e0197
+index<PHONE_NUMBER>00..283e91e01974
 --- /dev/null
 +++ b/tests/data/test1168

so I just updated the hashes to match the new versions.
If this causes some sort of mass rebuild, perhaps a better solution would be to just add the old ones directly to nixpkgs to avoid having to changes any hashes. If you would prefer that, let me know and I will change it.

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

[X] 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 nixpkgs-review --run "nixpkgs-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)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Actually, I see the newer versions of the curl file have a big comment right at the top that says

Note: this package is used for bootstrapping fetchurl, and thus cannot use fetchpatch! All mutable patches (generated by GitHub or cgit) that are needed here should be included directly in Nixpkgs as files.

so I'll update this pull request to directly include the patches instead.

Actually, I see the newer versions of the curl file have a big comment right at the top that says

Note: this package is used for bootstrapping fetchurl, and thus cannot use fetchpatch! All mutable patches (generated by GitHub or cgit) that are needed here should be included directly in Nixpkgs as files.

so I'll update this pull request to directly include the patches instead.

Ok. I've updated the pull request to actually include the patch. Unfortunately it seems this still results in a rebuild of curl as, at least, the ultimate store path for a file is different if you fetch it vs you directly reference it as a file path.
CCing @edolstra as this last bit seems a bit strange. I would have thought including a patch vs fetching it should not cause a rebuild as both would be the same fixed output. Fetching it vs referencing it ultimately gives two different paths, respectively
/nix/store/4psjdyqqf4wb4cv8ijds35cpqnmyjd5p-CVE-2020-8169.patch
/nix/store/f1wi4gqnkjxf0sfnc8k4fvgwrc4yir6g-CVE-2020-8169.patch

that are yet clearly the same file
diff -U2 \
  /nix/store/4psjdyqqf4wb4cv8ijds35cpqnmyjd5p-CVE-2020-8169.patch \
  /nix/store/f1wi4gqnkjxf0sfnc8k4fvgwrc4yir6g-CVE-2020-8169.patch \
&& echo the same

the same

Ok. I've updated the pull request to actually include the patch. Unfortunately it seems this still results in a rebuild of curl as, at least, the ultimate store path for a file is different if you fetch it vs you directly reference it as a file path.
CCing @edolstra as this last bit seems a bit strange. I would have thought including a patch vs fetching it should not cause a rebuild as both would be the same fixed output. Fetching it vs referencing it ultimately gives two different paths, respectively
/nix/store/4psjdyqqf4wb4cv8ijds35cpqnmyjd5p-CVE-2020-8169.patch
/nix/store/f1wi4gqnkjxf0sfnc8k4fvgwrc4yir6g-CVE-2020-8169.patch

that are yet clearly the same file
diff -U2 \
  /nix/store/4psjdyqqf4wb4cv8ijds35cpqnmyjd5p-CVE-2020-8169.patch \
  /nix/store/f1wi4gqnkjxf0sfnc8k4fvgwrc4yir6g-CVE-2020-8169.patch \
&& echo the same

the same

The commit message should change.

New commit message is curl: fix hash mismatch issue by directly include CVE patches.

I just noticed this was for 20.03, not 20.09. I'm not sure the staging branch for 20.03 is still merged into release-20.03 regularly?

Sorry. Sounds like I should have put in the request against release-20.03.
I can do a new pull request against release-20.03 if you want?

Nah, I just merged it into release-20.03: 1eea37190739485a01f5036558cd2ff257985678
Just to make sure: You are aware that NixOS 20.03 is end of life for about five months now? So no guarantees about security fixes being backported.

no guarantees about security fixes being backported.

Got it. Thanks! :+1:


Adds "Your First Swift App"

The list looks awesome, thanks for putting it together!
I have a (non-free) book on Swift that I thought might belong on the list, too.

@ashfurrow you got it

:bow:


Improving lab README code blocks

So I actually had a look at the README's in github and some of the code blocks were broken up due to the extra back-ticks from the slides. Didn't notice it in an editor.
Also added back-ticks around lab1 directory for consistency.

Codecov Report

Merging #127 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #127   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           2      2
=====================================
  Hits            2      2

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 ba45373...30cb048. Read the comment docs.


typechecker: Don't infer return type for function blocks

Not specifying a return type for a function block results in that block
returning void. We will still infer the return type for fat arrow
functions.

I'm surprised you didn't have to change more test cases for this one. I guess maybe we don't use return type inference very much?


[FEATURE] CohereEmbeddingModel: implement batching

CohereEmbeddingModel currently sends all TextSegment that it receives in embedAll(List<TextSegment> textSegments) method to the Cohere API.
When the number of segments is more than 96, Cohere API responds with an error:
status code: 400; body: {"message":"invalid request: total number of texts must be at most 96 - received 280"}
We need to add a maxSegmentsPerBatch parameter (with default value 96) in the ctor/builder and batch textSegments.

Hey i would like to work on this issue. Can you let me know where to get started?

Hi, you can take a look at AzureOpenAiEmbeddingModel as an example

Please check and see if the implementation is right.


Output encoded UTF-16 while Response header is UTF-8

I have got UTF-16 strings in the ef core models (nvarchar in SQL Server is UTF-16 encoded by default.
Entity Sets are retrieved in UTF-16 even when requested explicitely by an "Accept-Charset=utf-8" Request Header.
JSON Outputs should be UTF-8 by default, what can I do? Do you think this is an aspnetcore.odata issue?
Regards, Peter

Hi @labackp
Our environment (ja-JP strings in nvarchar column) response utf8  body and the nvarchar string value was escaped.
{
  "@odata.context":"https://localhost:44335/ec/$metadata#Products(Name)/$entity",
  "Name":"\u30b5\u30f3\u30d7\u30eb\u5546\u54c1201"
}

I think it is correct behavior...
Would you tell us expects and actuals with a sample?
p.s.
ODatalizer is very thin wrapper for microsoft.aspnetcore.odata, you known.
May be, I think that it is thire issue or just settings of  microsoft.aspnetcore.odata, efcore, aspnetcore.

Hi @labackp
Our environment (ja-JP strings in nvarchar column) response utf8  body and the nvarchar string value was escaped.
{
  "@odata.context":"https://localhost:44335/ec/$metadata#Products(Name)/$entity",
  "Name":"\u30b5\u30f3\u30d7\u30eb\u5546\u54c1201"
}

I think it is correct behavior...
Would you tell us expects and actuals with a sample?
p.s.
ODatalizer is very thin wrapper for microsoft.aspnetcore.odata, you known.
May be, I think that it is thire issue or just settings of  microsoft.aspnetcore.odata, efcore, aspnetcore.

Hi @iwate
Thank you for your answer! I would have expected the json strings to be escaped with UTF-8 encoding if I request a utf8 encoding, but this is not the case. I just want to make sure that the output is readable by a wide range of browsers an their javascript engines.
SQL Server 2019 has got a new collation group with utf8 encoding, but nvarchar will always be encoded utf16.
So thank you for your help, I will try this my first wabapi odata implemenation "as is" and will let you know if there are issues on the consumer side.
Best regards, Peter

Hi @iwate
Thank you for your answer! I would have expected the json strings to be escaped with UTF-8 encoding if I request a utf8 encoding, but this is not the case. I just want to make sure that the output is readable by a wide range of browsers an their javascript engines.
SQL Server 2019 has got a new collation group with utf8 encoding, but nvarchar will always be encoded utf16.
So thank you for your help, I will try this my first wabapi odata implemenation "as is" and will let you know if there are issues on the consumer side.
Best regards, Peter

Thank you for clarification!

Thank you for clarification!

Note:
Microsoft.OData.Core has own JSON Serializer, don't use System.Text.Json or Newtonsoft.Json
https://github.com/OData/odata.net/tree/master/src/Microsoft.OData.Core/Json
Microsoft.OData.Core.Json has two escape modes.
https://github.com/OData/odata.net/blob/9788e845f5664642f990c02b6a5fec7c7df7febd/src/Microsoft.OData.Core/Json/ODataStringEscapeOption.cs#L12-L24
And default use EscapeNonAscii
https://github.com/OData/odata.net/blob/9788e845f5664642f990c02b6a5fec7c7df7febd/src/Microsoft.OData.Core/ContainerBuilderExtensions.cs#L134
https://github.com/OData/odata.net/blob/9788e845f5664642f990c02b6a5fec7c7df7febd/src/Microsoft.OData.Core/Json/DefaultJsonWriterFactory.cs#L32-L35
I try override the factory in ODatalizer.EFCore.EndpointRouteBuilderExtensions and get unespaced payload.
public static void MapODatalizer(this IEndpointRouteBuilder builder, params ODatalizerEndpoint[] endpoints)
        {
            builder.Select().Expand().Filter().OrderBy().MaxTop(100).Count().SkipToken();
            foreach (var ep in endpoints)
            {
                builder.MapODataRoute(ep.RouteName, ep.RoutePrefix, b => {
                    // ---- add here ------
                    b.AddService<IJsonWriterFactory>(ServiceLifetime.Singleton, sp => new DefaultJsonWriterFactory(ODataStringEscapeOption.EscapeOnlyControls));
                    // --------------------
                    b.AddService(ServiceLifetime.Singleton, sp => ep.EdmModel);
                    b.AddService<ODataBatchHandler>(ServiceLifetime.Singleton, sp => new ODatalizerBatchHandler());
                    b.AddService<ODataDeserializerProvider>(ServiceLifetime.Singleton, sp => new DefaultODataDeserializerProvider(sp));
                    b.AddService<IEnumerable<IODataRoutingConvention>>(ServiceLifetime.Singleton, sp => {
                        var conventions = ODataRoutingConventions.CreateDefaultWithAttributeRouting(ep.RouteName, builder.ServiceProvider);
                        if (string.IsNullOrEmpty(ep.ODatalizerController) == false)
                        {
                            conventions.Add(new ODatalizerDynamicConvention(ep.ODatalizerController.Replace("Controller", string.Empty)));
                        }
                        return conventions;
                    });
                });
            }
        }
    }

Note:
Microsoft.OData.Core has own JSON Serializer, don't use System.Text.Json or Newtonsoft.Json
https://github.com/OData/odata.net/tree/master/src/Microsoft.OData.Core/Json
Microsoft.OData.Core.Json has two escape modes.
https://github.com/OData/odata.net/blob/9788e845f5664642f990c02b6a5fec7c7df7febd/src/Microsoft.OData.Core/Json/ODataStringEscapeOption.cs#L12-L24
And default use EscapeNonAscii
https://github.com/OData/odata.net/blob/9788e845f5664642f990c02b6a5fec7c7df7febd/src/Microsoft.OData.Core/ContainerBuilderExtensions.cs#L134
https://github.com/OData/odata.net/blob/9788e845f5664642f990c02b6a5fec7c7df7febd/src/Microsoft.OData.Core/Json/DefaultJsonWriterFactory.cs#L32-L35
I try override the factory in ODatalizer.EFCore.EndpointRouteBuilderExtensions and get unespaced payload.
public static void MapODatalizer(this IEndpointRouteBuilder builder, params ODatalizerEndpoint[] endpoints)
        {
            builder.Select().Expand().Filter().OrderBy().MaxTop(100).Count().SkipToken();
            foreach (var ep in endpoints)
            {
                builder.MapODataRoute(ep.RouteName, ep.RoutePrefix, b => {
                    // ---- add here ------
                    b.AddService<IJsonWriterFactory>(ServiceLifetime.Singleton, sp => new DefaultJsonWriterFactory(ODataStringEscapeOption.EscapeOnlyControls));
                    // --------------------
                    b.AddService(ServiceLifetime.Singleton, sp => ep.EdmModel);
                    b.AddService<ODataBatchHandler>(ServiceLifetime.Singleton, sp => new ODatalizerBatchHandler());
                    b.AddService<ODataDeserializerProvider>(ServiceLifetime.Singleton, sp => new DefaultODataDeserializerProvider(sp));
                    b.AddService<IEnumerable<IODataRoutingConvention>>(ServiceLifetime.Singleton, sp => {
                        var conventions = ODataRoutingConventions.CreateDefaultWithAttributeRouting(ep.RouteName, builder.ServiceProvider);
                        if (string.IsNullOrEmpty(ep.ODatalizerController) == false)
                        {
                            conventions.Add(new ODatalizerDynamicConvention(ep.ODatalizerController.Replace("Controller", string.Empty)));
                        }
                        return conventions;
                    });
                });
            }
        }
    }


The SSN in a demo form

Hello, I am able to run your PyTorch SSN code and I can get it running in all the segments (ssn_train, ssn_test, evaluate_detection_results parts). Also the SSN project should output starting time, ending time, and category for each action instance (of an input video). I want to create a demo based on your code, and for an input video, output the starting time, the ending time, and the label of activity(ies) detected in that video but not sure where I can find this information in the ssn_test.py/eval_detection_results.py scripts? Could anyone maybe help me about this? Many thanks in advance! @yjxiong

@mikeleatila have you succeded with this?

+1


[BUG] Harvester install crashes when assigning Static IP to join existing cluster.

Describe the bug
When trying to join an existing cluster and setting a static IP for the node, the installer will crash.
To Reproduce
Steps to reproduce the behavior:

Select 'Join existing Harvester cluster' from the installer
Select 'Static' for the network
When you type in the static IP address and press Enter, the installer crashes.

Expected behavior
I expected that the node would statically set the IP address and continue instead of crashing the installer.
Support bundle
NA
Environment:

Harvester ISO version: 0.2.0
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):  UniFi Application Server (UAS-XG)

Additional context
NA

looks like a similar issue of https://github.com/harvester/harvester/issues/698, caused by small screen resolution

looks like a similar issue of #698, caused by small screen resolution

I think this is precisely the issue.  It drops to the command line.  I found that typing exit will get me back to the installer, but it then starts over.

Duplicated with https://github.com/harvester/harvester/issues/2937


C# sample code with compilation error

Compilation error at QueryParameters in C# example:
requestConfiguration.QueryParameters.Format = "{format}";

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

ID: 59cfb473-0783-09cb-3c88-96f884d76e06
Version Independent ID: 9992b18b-299d-6863-c06f-8bacb7ff2962
Content: Convert to other formats - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-get-content-format.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @spgraph-docs-team
Microsoft Alias: MSGraphDocsVteam

Hi,
Thanks for your feedback. Unfortunately this repo is for keeping track of issues or new contributions in the Microsoft Graph docs and not to provide help about how to use Microsoft Graph. You should rely on the Microsoft Graph support area (https://developer.microsoft.com/en-us/graph/support) for this kind of questions. I'm going to close this item. Thanks for your understanding.


Add-SPOFile with Checkout option does not update existing files

Updating an existing file with Add-SPOFile command and the Checkout option does not work if the library has the Require Checkout -option set. The cmdLet does not even try to checkout the existing file before updating it.
The file exists -check before the actual checkout fails because the necessary property (Exists) is not loaded through CSOM.
I'll submit a pull request to fix this issue.

The fix has been implemented. I will close this issue now.


WIP: integration: add v2v3 emulation

from #9626.
Run existing integration tests for v2 api (TestV2) on v2v3 emulation to improve the compatibility between them.

Currently tests are broken due to v2v3 emulation doesn't work properly for member list.

Are member APIs are supposed to be supported by v2v3 emulation? Or it just support basic KV operations...?

@yudai the kv operations are the most tested; it still doesn't cover all membership operations. I left stubs for membership calls (see v2v3/cluster.go) with the intent that it would make v3 calls, convert to v2 structures, then let v2http handle the http response.

@heyitsanthony thanks for the pointer. I found that the methods of the api.Cluster interface don't have error in their return values, since the native v2 API uses internal values. However, if we use clientv3, it can cause errors.  I'm wondering if we can change the interface (can breaking backward compatibility) to add error to return values.
TestV2Set is working now anyway.  To make TestV2CreateUpdate, we need TTL support, which currently explicitly marked as errUnsupported for some reason. Is this just a stub?

@yudai

wondering if we can change the interface (can breaking backward compatibility) to add error to return values.

I don't think adding error returns will break much. A lot of those old interfaces should be internal. Worst case, there can be some adapter to get the old behavior back.

if there are other operations happen between the stm.Rev and the commit of the main Tx, it can cause inconsistency.

For every key the txn is writing, the applier calls stm.Rev, which should insert the key into the readset (https://github.com/coreos/etcd/blob/master/clientv3/concurrency/stm.go#L291). If there's a change to any of those keys before the txn commits, STM will have a read conflict, throw away the puts, and retry. Is that not happening or is there some inconsistency it's missing?

@heyitsanthony Thank you for the reference. I overlooked that readset and now I understand the behavior.
It's taking time more than I expected when I started this work. I'll make tests pass one by one anyway.


Level 2

Added Level 2 exercises essentially straight from the readme files of the lessons.
Note how the last commit is the one that contains the static site build. Since its Github pages, we have to build and then commit it, so adding them in on a commit of their own is best. In the case we might need to roll back something, the commit changes to the source files will be segregated from the build commits.

After writing that description, I realized there is a thing called Github Actions... so if any of y'all want to play with that go for it. I don't want to hog all the excitement.


LIMS XML and bug fixes

Change description

Download an XML for LIMS from clicking dropdown that's now part of the download PDF button on the Report tab #60
Add page numbers to PDF, uses different package so env has changed, also adjusted formatting #13
Fix issue where the wrong tab is defaulted to #47
Fix bug where you couldnt reopen a sample fail or enter a fail 2nd check if you're assigned to it, also tidy up of that page because there's lots of different options and there were loads of nested loops in template
Add 'No gaps' or 'No calls' to results page when there are no reportable calls/gaps (both GUI and PDF, not needed for XML), for both DNA and RNA
Fixed visualisation of COSMIC percentages and added columns for COSMIC counts in Depth & Report tabs & PDF (for solid tumour referrals only)
Added 'options' page to link out to various other things such as poly lists (and probably other things in future
Split the poly list into separate build 37 and 38 pages (you'll need to add the two poly lists on your test dev, called build_37_polys and build_38_polys)

Justification for minor update
No effect on processing of data, only how it's visualised
Testing description

Running existing unit tests
LIMS XML not needed yet, will be tested as part of LIMS validation, may be tweaked as we go along, just review the code
COSMIC counts/percents will be tested more in future PR - need test data from Sophie, just review the code
Manual testing of database functionality

XML downloads and data matches report tab
PDF numbers pages correctly, data matches report tab and formatting is okay
No calls/ No gaps shown when no calls in report tab and PDF, instead of empty table
tried all possible combinations for the assigned to buttons on the samples page, shown in the comments of view_samples.html
Tested that text after hash is successfully removed when going onto the view_samples.html page (analyse one run normally and when you're redirected back there should be no hash)
Test adding polys to the build 37 and build 38 lists and make sure that they end up in the correct ones

Test data location
N/A
Verification results

Unit tests all completed successfully
Database testing listed above all performed as expected
Deployment plan:

Simple model change - defaults to null
Notify CS about PDF changes in case there's any weird formatting that needs fixing on the fly
Cant specify output location for PDF/XML due to security issues, but can show CS how to set up browser to save as rather than just download straight into download folder
Conda environment will need updating, test with different env name first as Viglen conda is temperamental

All unit tests pass.

Manual testing: XML

No XML download for failed samples - is this intentional?
Compared XML download and data for 2 passed samples - data identical between database and XML

Manual testing: PDF

Pages number correctly
"No calls" in variants table when no calls
"No hotspot gaps" in gaps table when no gaps
For POLE gaps, all gaps show as POLE(NM_006231.3):c.NA_NA on web app and on download - this is also true of live SVD. Possibly because in the regions, formatting is e.g. POLE(NM006231.3):c.1376_p.(Ser459Phe) rather than POLE(NM006231.3):c.1376_1376_p.(Ser459Phe)
Data matches between download and web app - tested for 3 samples
PDF output formatting is good

Manual Testing: Poly Lists

The 'tag' next to the Build 38 poly list title reads GRCh37
Pages don't load without existing polylist objects - could be helpful to add this to fixtures/readme for dev
Same user cannot check polys - good
Cannot add b38 variants to b38 polylist. Tried b38 with/without chr and b37 just in case. Will investigate in code.

fight-209 not included in the panels_march2023 json as a b37 TSO500 DNA referral (there for b37 RNA, b38 DNA and RNA) so one sample from 37 test data does not upload

XML download for failed samples (EW to discuss with clinical scientists)

Thanks for spotting this, will discuss with CS. XML may go through a few iterations before final version is done

POLE gaps addressed in SS pipeline update

Yes, this is an issue with the bed2hgvs part for the current pipeline, it cant annotate POLE (and a few other genes) so just defaults to NA. New pipeline will resolve this issue

Minor changes to code - mostly aesthetic and 1 variable name correction, see individual comments

Done, replied to individual comments

Minor changes to fixtures to ensure smooth dev - polylists in fixtures, update panels with fight-209 b37 dna and aml

Polylist fixture added, it will just add the objects but no actual polys
I've edited the poly list script and put instructions on how to fill from the poly list text files if polys are needed in the database, also includes a flag to prevent us having to double check them all for dev work
The readme has been updated with this info and made deploy instructions clearer
I havent updated the panels yet as I want to do a cleanup on deploy and do it all in one go, there's a few more panels in main on Viglen that aren't in here too


Incorrect Singularization of Certain Words

The following words are incorrectly singularized:

Word
OperationId
CommandName

Sms
users.authentication.methods_enableSmsSignIn
Enable-UserAuthenticationMethodSmSign

iOS and Statuses
deviceManagement_ListIosUpdateStatuses
Get-DeviceManagementIoUpdateStatuses

Windows
deviceManagement_GetWindowsAutopilotSettings
Get-DeviceManagementWindowAutopilotSetting

Possible Solutions

Support a configurable excludes list in AutoREST config that customers can use to specify words to exclude from singularizing. Rename directives don't scale well when targeting hundreds of command names.

@dolauli , let's discuss design here.

Configuration design
we may add configurations as below to specify singular form for a noun.
WordsOfSpecialSingularization:
  - Pluralization: Sms
    Singularization: Sms
  - Pluralization: Windows
    Singularization: Window

Code change
Need to update following codes to apply the configurations above
https://github.com/Azure/autorest.powershell/blob/9eb9fada924562e1566028d4a926c3f5d01c7f40/powershell/plugins/plugin-create-inline-properties.ts#L17-L24
https://github.com/Azure/autorest.powershell/blob/9eb9fada924562e1566028d4a926c3f5d01c7f40/powershell/plugins/create-commands-v2.ts#L36-L40


Have a netty warning

Hello.
After 2-3 days of starting the server, i got WARN:
2014-02-07 12:42:36  WARN: netty warning: Failed to accept a connection

And after that one of connected devices ceases response. After that history repeats and another device turning off. Etc at the end - all devices does not response.

UPDATE:
You should be using
<entry key='gt06.timeout'>60</entry>

instead of
<entry key='gt06.resetDelay'>60</entry>

resetDelay is deprecated.
#1524

In my case was a 60 connecting to port 5020. This parameter solve an issue affecting the platform in a way of "too many open files" and "connections idle"


recipe: use go-licenses dependency instead of installing it manually

Hello @nicks,
Please review the following commits I made in branch nicks/go-licenses:
bf465a04d952150a37b772c1c340094ccba76944 (2021-01-29 14:57:53 -0500)
recipe: use go-licenses dependency instead of installing it manually
I checked other autogenerated Go projects (like
https://github.com/conda-forge/podman-feedstock/blob/master/recipe/meta.yaml)
and this seems be what they do. But I don't totally understand why conda-smithy
rerender doesn't fix this automatically.
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Routine Release

Prepare a new routine release

Created Release https://github.com/GioF71/squeezelite-docker/releases/tag/2022-09-09


ZEPPELIN-140 Send complete buffer to interpreter.completion

Thanks for the contribution.
Looks good to me.


[Feature request] Expose tokensMap as public

There are a number of useful class properties / methods on the BaseParser / CstParser that aren't exposed via TypeScript, so it's hard to know if they're stable / safe to use. One of those is tokensMap. I'd been creating this exact type of map for use when parsing, so I was surprised to find the same structure on the instance itself when debugging. I'd like to eliminate creating this map, but I'm not sure if tokensMap is safe to use.

In theory it could be changed to a Map instead of a dictionary at some point.
I am not sure its that important to expose it, particularly when it is trivial to create it...
e.g. something like:
import { createToken as orgCreateToken } from "chevrotain";

const tokensMap = {}

function createToken(opts) {
  const newToken = orgCreateToken.call(null, opts);
  tokensMap[opts.name] = newToken
  return newToken;
}

FYI, object key lookups in v8 (and most other engines as well) are multiple times faster than map lookups. Noticed that while experimenting with https://github.com/Chevrotain/chevrotain/pull/1793.


Storm-Pulsar bolt and spout

What is the purpose of the change
Apache Pulsar is a popular streaming and messaging platform.
It provides storm integration as part of the pulsar-adapters subproject but the storm adapter never keeps up with the latest storm changes/builds.
The adapter does not require much knowledge about the pulsar internals but knowledge of Storm internals is something that is more beneficial.
This PR is an attempt to transition pulsar-storm adapter to the storm community .
How was the change tested
Unit/integration test.

Travis build is failing for unrelated reasons. I built and tested as:
mvn clean install -DskipTests -P '!examples,!externals'
cd external/storm-pulsar/
mvn clean install
...
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.pulsar.storm.PulsarSpoutIntegrationTest
[INFO] Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 71.711 s - in org.apache.pulsar.storm.PulsarSpoutIntegrationTest
[INFO] Running org.apache.pulsar.storm.PulsarSpoutTest
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.098 s - in org.apache.pulsar.storm.PulsarSpoutTest
[INFO] Running org.apache.pulsar.storm.PulsarBoltIntegrationTest
[INFO] Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 42.811 s - in org.apache.pulsar.storm.PulsarBoltIntegrationTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 22, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

Travis needs https://github.com/apache/storm/pull/3437  ...

Overall looks good to me. Thanks @dlg99 for the great job. And do you have any updates for @agresch and @bipinprasad comments?


use git submodule instead of git clone when install

I think we should to use git submodule here, because if I use git clone , and add docker folder to my project git repo, when my teammate pulls from  remote, he will faced the No submodule mapping found in .gitmodules for path 'docker' error.

@chungth thanks


Issue 1046: Use the Spring Boot Actuator logger management instead of our own solution

Fixes #1046
I added simple loggers endpoint to the .properties file and modified the security for these endpoints, because the previous does not secure them for not authorized users. Tests that I wrote are simple tests, just to test the endpoints, in the future it should be improved, for now it is okay I believe, if there is anything what should be changed, let me know and I will do it.

Hi @adavid9 , I'll continue work on this task, if you don't  mind :)
FYI @steve-todorov

moved to ant continued in https://github.com/strongbox/strongbox/pull/1440

@fuss86 Hi Przemyslaw, sure go ahead.

@adavid9 Thanks for your work on this! I'm closing it in favor of #1440 :)


Force called IPAM binaries to support unknown args

LGTM - do we need a UT to keep 100% coverage?

Coverage is still 100%.


Opening gs links

@dreampiggy hello friend I'm confused about this simple code:
    SDImageLoadersManager.shared.loaders = [FirebaseUI.StorageImageLoader.shared]
    SDWebImageManager.defaultImageLoader = SDImageLoadersManager.shared
    let ref = Storage.storage().reference().child(imageName)
    print(ref) //gives the gs link
    let url = NSURL.sd_URL(with: ref)! as URL
    return WebImage(url: url)

When I run this I get a Code=-1002 "unsupported URL" error. Could you please help me?

I was trying to follow #70 but I got confused.

This should works. Because it's supported via: https://github.com/firebase/FirebaseUI-iOS/pull/841
Maybe you're using the old version of FirebaseUI ?
Provide all your Pod's version, including:

SDWebImage
SDWebImageSwiftUI
FirebaseUI

Or a demo to reproduce this issue ?

Thanks so much for responding!!!
These are all my pods:
Installing BoringSSL-GRPC (0.0.7)
Installing Disk (0.6.4)
Installing Firebase (7.10.0)
Installing FirebaseAnalytics (7.10.0)
Installing FirebaseCore (7.10.0)
Installing FirebaseCoreDiagnostics (7.10.0)
Installing FirebaseFirestore (7.10.0)
Installing FirebaseFirestoreSwift (7.10.0-beta)
Installing FirebaseInstallations (7.10.0)
Installing FirebaseStorage (7.10.0)
Installing FirebaseUI (10.0.2)
Installing GTMSessionFetcher (1.5.0)
Installing GoogleAppMeasurement (7.10.0)
Installing GoogleDataTransport (8.4.0)
Installing GoogleUtilities (7.3.1)
Installing PromisesObjC (1.2.12)
Installing Resolver (1.4.1)
Installing SDWebImage (5.11.0)
Installing SDWebImageSwiftUI (2.0.2)
Installing abseil (0.20200225.0)
Installing gRPC-C++ (1.28.2)
Installing gRPC-Core (1.28.2)
Installing leveldb-library (1.22.1)
Installing nanopb (2.30908.0)
I'll work on a demo to reproduce the issue :)

I'm seeing on the link that SDWebImage should be 5.6 I'll work to fix this!

Thanks! Here they are from my pod file:

SDWebImage (5.11.0):

SDWebImage/Core (= 5.11.0)

SDWebImageSwiftUI (2.0.2):

SDWebImage (~> 5.10)

FirebaseUI (10.0.2):

FirebaseUI/Anonymous (= 10.0.2)
FirebaseUI/Auth (= 10.0.2)
FirebaseUI/Database (= 10.0.2)
FirebaseUI/Email (= 10.0.2)
FirebaseUI/Facebook (= 10.0.2)
FirebaseUI/Firestore (= 10.0.2)
FirebaseUI/Google (= 10.0.2)
FirebaseUI/OAuth (= 10.0.2)
FirebaseUI/Phone (= 10.0.2)
FirebaseUI/Storage (= 10.0.2)
If this doesn't work I'd be more than happy to code up a demo!

The dependency version looks correct.
Maybe this because of some bugs. Can you provide some Firebase Storage URL (without authoritation) for me to test ?
A little demo is better.

@dreampiggy sounds good I'll set it up in a bit

@dreampiggy done! I invited you to the demo the URL in the demo should work. Demo is WebImageGSDemo

@SnowballAntrobus Seems works.
You should put that SDK initialize code into App.swift, like this

Not the onAppear logic, when the onAppear called, your image request already been sent and since URLSession does not support gs:// URL, it will fail.

I can add this into the readme or wiki about How to use some codec / loaders with SwiftUI.

Done. See: Using with external loaders/caches/coders

Thanks so much my friend!!!


建议Timer 计时器添加震动效果

震动js
var supportsVibrate = "vibrate" in navigator;
if (supportsVibrate) navigator.vibrate(1000);

this feature had been added.


Web - Report is not highlighted even if context menu is open

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Right click on report in LHS
Again right click on the same report
Move cursor anywhere on the web app
Observe it is not highlighted but context menu is open

Expected Result:
The report should be highlighted when context menu is open.
Actual Result:
The report is not highlighted when context menu is open.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?:  y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/1c3839a7-f90f-41ef-9449-65a58eb04826
https://github.com/Expensify/App/assets/93399543/61e97b0e-1437-4179-9388-c696fe04bbb9
**Expensify/Expensify
Issue reported by: @krishna2323
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693512993104379
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011686c8222a068c45
Upwork Job ID:<PHONE_NUMBER>612799488
Last Price Increase: 2023-10-05

I'm confused why this is a bug though. The LHN should always highlight the report that is in the main chat window.

Maybe @getusha can raise follow-up PR fixing this as this is 1 line change?

Looks similar issue, i can raise the PR.

This should have been handled in #25582 as they both have the same root cause and just happen in different places. Maybe @getusha can raise follow-up PR fixing this as this is 1 line change? cc: @abdulrahuman5196 @pecanoro

Ideally, this issue should have been marked dupe and notified on original issue initially. Since we were unaware of this issue, the other PR(https://github.com/Expensify/App/pull/28639) didn't target this place of issue and the other PR is merged.
I am not sure how a follow up PR can pan out now since there is another contributor already raised a formal proposal.
Its fine IMO to raise a followup PR and I can C+ review the PR.
But I would let the BZ confirm/take final call here.

@aimane-chnaif IMO i think it should've been handled here if it is a dupe or not, which was already attempted by @bernhardoj

Agree that @bernhardoj marked this issue as a dupe long ago. I was simply curious why it was a bug, though in any case, it's a dupe so we should close it.


Icon doesn't work on google button

I just tried version 0.5.0 in my project with a google button (dark) and I get no icon and this exception:
I/flutter (24157): ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
I/flutter (24157): The following assertion was thrown resolving an image codec:
I/flutter (24157): Unable to load asset: packages/flutter_auth_buttons/graphics/google-logo.png
I/flutter (24157):
I/flutter (24157): When the exception was thrown, this was the stack:
I/flutter (24157): #0      PlatformAssetBundle.load (package:flutter/src/services/asset_bundle.dart:221:7)
I/flutter (24157):
I/flutter (24157): #1      AssetBundleImageProvider._loadAsync (package:flutter/src/painting/image_provider.dart:464:44)
I/flutter (24157):
I/flutter (24157): #2      AssetBundleImageProvider.load (package:flutter/src/painting/image_provider.dart:449:14)
I/flutter (24157): #3      ImageProvider.resolve... (package:flutter/src/painting/image_provider.dart:315:48)
I/flutter (24157): #4      ImageCache.putIfAbsent (package:flutter/src/painting/image_cache.dart:160:22)
I/flutter (24157): #5      ImageProvider.resolve.. (package:flutter/src/painting/image_provider.dart:315:25)
I/flutter (24157): (elided 13 frames from package dart:async)
I/flutter (24157):
I/flutter (24157): Image provider: AssetImage(bundle: null, name:
I/flutter (24157):   "packages/flutter_auth_buttons/graphics/google-logo.png")
I/flutter (24157): Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#833bd(), name:
I/flutter (24157):   "packages/flutter_auth_buttons/graphics/google-logo.png", scale: 1.0)
I/flutter (24157): ════════════════════════════════════════════════════════════════════════════════════════════════════

Tested 0.4.0 now and there it works fine...

That's an odd one. I can't reproduce at my end.
Also, if you look at the differences between 0.4.0 and 0.5.0, there is nothing related to the Google button.
Have you tried a flutter clean to trigger a fresh build?

Well, you are right, I changed back to 0.5.0 now, the issue is gone. So I will close my issue.


hiksink: init at 1.2.0

Motivation for this change
Tool to convert Hikvision camera events to MQTT
https://github.com/CornerBit/HikSink
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 142008 run on x86_64-linux 1

1 package built:

hiksink


Refactor

This adds a virtualization specific preview vdb.  Each complete view will be available.  The preview query logic has changed to expect a virtualization name, rather than a vdb name.  The ui logic for a preview query should be changed to just issue "select * from view" - no ddl parsing nor usage of the source paths is needed.
This adds a dirty flag to the datavirtualization to let us know when to refresh it on preview.
Once we address allowing views to reference other views, this will get more complicated.

I should clarify that it needs to be a fully qualified preview query:
select * from views.viewname
Because the source tables are not hidden in the dv specific vdb, so they could have an ambigious name.
Note that the views schema name convention is worth discussing.  It will prevent any layering of data vdbs - as there would be schema collision on import.  If we want to eventually allow for that, then we may want to choose a different convention, such as the data virtualization name - but of course with restrictions to make it a valid schema name.

select * from views.viewname
Isn't this single "virtual model" based view?

Because the source tables are not hidden in the dv specific vdb, so they could have an ambiguous >name.
From the UI there is no direct access to the user, so why not classify them as hidden?

Isn't this single "virtual model" based view?

I'm not sure what you are asking.

From the UI there is no direct access to the user, so why not classify them as hidden?

I'd like for the dv specific preview vdb to become the basis for the a generic sql client, such that you can execute any valid sql in the context of the dv - which means that the source tables cannot be hidden.

I am saying that we agreed there is going to be a single virtual model in the vdb, so the resolution always needs to be at the virtual model level.

I'd like for the dv specific preview vdb to become the basis for the a generic sql client, such that you >can execute any valid sql in the context of the dv - which means that the source tables cannot be >hidden.
On this, I would differ. The generic SQL preview should be open to the views exposed through vdbs. We do not have to stop the user from executing the source level queries if they know the full path. as such if we do it right the SQL editor can support auto-compete for these tables.

I am saying that we agreed there is going to be a single virtual model in the vdb, so the resolution always needs to be at the virtual model level.

I'm not entirely sure, but you're probably just communicating the same thing as what I'm expressing - when the ui is responsible for issuing a preview query for a given view, then it should be "select * from views.view".  If the sources were hidden, then it doesn't need to be qualified, but as I'm indicating there's a reason for why you would want a general dv specific preview vdb.

On this, I would differ. The generic SQL preview should be open to the views exposed through vdbs. We do not have to stop the user from executing the source level queries if they know the full path. as such if we do it right the SQL editor can support auto-compete for these tables.

I'm suggesting a mode for a sql client is a workbench for playing around with sql/results until you're ready to create a view from it.  In that mode you should be able to reference other views and sources.
There's another preview mode of just the "views" - which is currently restricted to a drop down list.

This discussion made me double check, and sure enough the source models were not hidden in the service vdb, so that has been updated as well.

So you only suggesting that when drop down queries coming in always qualify them with view. which is static. If we populate them with qualified name to begin with that issue should be solved.
I do agree the ad-hoc mode is good, that is what we wanted to support to begin with, but I guess we choose current one for expediency.

If we populate them with qualified name to begin with that issue should be solved.

The user won't see that.  The ui currently has logic that either tries to parse the view ddl, or tries to construct a select query from the sourcePaths.  I'm saying that the ui should replace all that logic with just "select * from views.view"

yeah, from what I read from your previous comments that need to be taken out anyway, that will not work.


Although defined, some elements are not translated

Hello,
I had created an issue (https://github.com/apache/superset/issues/21493) but it has multiples cause of no translation.
I specialise this issue on elements not translated although defined
It's not just a lack of traduction

Items are :

"Dashboards": ["Tableaux de bord"],
"Charts": ["Graphiques"],
"Datasets": ["Jeux de données"],
"Saved Queries": ["Requêtes sauvegardées"],
"Query History": ["Historiques des requêtes"],
"Action Log": ["Journaux d'actions"],
"Annotation layers": ["Couches d'annotations"],
"CSS template": ["Templates CSS"],
"Row Level Security": ["Sécurité de niveau ligne"],
"Alerts & Reports": ["Alertes et rapports"],
"Refresh": ["Forcer à rafraîchir"],
"Report name": ["Nom du rapport"],

Screenshot :

The french translation are not applied.
Best regard

I use Supserset in version 3.0.0

Hi @xavier-GitHub76
I've found that the menu items are populated from the backend, not the front-end. So the translated values must be in the .mo file, not the .json file.
Hope that helps!

@xavier-GitHub76
I noticed that if you run
pybabel compile -d superset/translations
it will create/update the messages.mo files and then the navbar is correctly translated.

This is related to the issue
There are no messages.mo files in the docker container so no translations related to FAB !

I'm not sure how we ought to handle this, really, but I suspect it's quietly affecting a lot of people that just don't realize it. Should we auto-generate the .mo files again? I'm not sure the advantage/history of why those are currently not included... just file size? I think @mistercrunch knows the history on their removal, but I'm not sure anyone's feelings for or against reintroducing them.
Also curious if people think we should leave this open (since it's kind of a bug) or close this out in favor of the open Discussion.

I'm thinking we should add the .mo and .json file build as part of the docker and pypi builds, and keep these files from the repo.

I wonder if we could even generate them at start time, since (as I recall) they're pretty slow. If we do that, then it's not a breaking change, the repo gets lighter, and the translations will be up to date every time the code changes.

at 10-20 seconds each (pybabel and po2json) I think it's a bit slow to force that on web-server start, plus there are many ways to start the web server, and many people don't care for all or some of the translations...
I think this fits perfectly in a docker build and/or pypi build steps, though it does add a bit of weights to the images/bundles (about 8mb) which I think is totally fine

True... though npm run build already takes approximately forever. 10 more seconds there might not be the end of the world, if it catches more use cases and centralizes the task for docker/pypi/local use cases.

Wait but the built frontend assets are bundled in the python package, meaning if you get the package you don't need to have node/npm, run npm i, npm run build, ... Should be prepackaged.
I guess similarly here we can compile and package .po+.json, and people don't have to have all that extra junk (npm / po2json / pybabel / ...). Also doesn't seem good practice (is it even allowed) to mutate the package in place post install. I think you'd be supposed to do this in home somewhere like ~/.superset/ ...

Right, the npm/webpack built assets are indeed included in the python package... which I thought was done via npm run build anyway... so it'd just be shifting it to that NPM process. This would just have the benefit for people cloning the repo from github, so they could just run npm run build to have the assets in their local builds. Doing so on npm run dev(-server) seems a bit heavy-handed, but also seems useful for those contributing translations. The NPM script might even be able to check if you have translations turned on, and only run it if relevant. Maybe this is a good topic to discuss more synchronously :) It'd be great to get it all straightened out one way or another! :D

My proposal to improve things around translations -> https://github.com/apache/superset/pull/28483


Add a dedicated man page for wLog

@freerdp-bot test.

Hmm TestConnect timeouts quite often lately. We should have a look.

@hardening Maybe also add the perference order: Environment is overwritten by command line arguments and add the command line arguments in the man page

@akallabeth I will add the order. For command line arguments I'd prefer to put them in the freerdp man page as wLog is used can be used in other projects (you see what I mean).

@akallabeth adding wlog man page reference in /log-level and /log-filter forced me to do a big cleanup in man page generation.

@akallabeth I have added a section stating the precedence of arguments on environment variables

Anybody to comment and merge this one ?

@nfedera @bmiklautz Any objections? Otherwise I'll merge this one.


test_vpc_redundant.py: reduce sleep time from 1 hour to 21 mins

Description
The default value of network.gc.wait and network.gc.interval are both 600 (seconds).
in test_vpc_redundant.py, it waits 1 hour which is unnecessary.
I have run it 10 times when I tested #4224, no isues.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian. SL-JID 1374

@blueorangutan test

@weizhouapache a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2182)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35434 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5503-t2182-kvm-centos7.zip
Smoke tests completed. 86 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_hostha_enable_ha_when_host_in_maintenance
Error
303.62
test_hostha_kvm.py

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian Build Failed (tid-2186)

@blueorangutan test

@sureshanaparti a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2196)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31077 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5503-t2196-kvm-centos7.zip
Smoke tests completed. 87 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


⚠ ClamAV has degraded performance

In f7f79d6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 85dca4a.


LiteDB 3.1.3 is no longer signed

Hi,
I've just upgraded nuget package from 3.1.1 to 3.1.3 and it's no longer working due to missing assembly signature (crash at runtime).
I've traced it down to #660 and created PR #695 that brings signature back.

Thanks! I will release next version later.


Respect client color settings

Although not currently respected by the Vanilla server, it'd be nice to respect the client's settings for if the chat should contain color or not.

Currently blocking on on https://github.com/KyoriPowered/adventure/pull/354.


Redirect stdout and stderr to logfile

This is to have panic messages logged, mostly.
See https://github.com/gogits/gogs/issues/3893

And for docker I would expect the opposite, drop the log files and print everything to stdout and stderr.

Anyone work on this?

Nobody claimed it, but as said above I wish we can get rid of all log files for the docker container

@lunny please go ahead, for 1.0.0 I'd like to have a working setup outside of docker, further tweaks (like an option to only log to stdout) would be nice for 1.1.x

Maybe we could resolve all log issues in 1.1

I put this to v1.2 since nobody could send a PR to resolve this.


[Bug]: Price range picker returns only last value.

Expected Behavior
All the picked price ranges should be passed to backend.
Actual Behavior
Currently, in our price picker, we are returning products from the first checked price range instead of products from all checked price ranges.
Possible Solution
Spree expects us to pass the price range as interval [min_price, max_price], so maybe it is a good idea to change this interval price picker to some slider where we can choose minimal and maximal price value.
Steps to reproduce
https://user-images.githubusercontent.com/26672113/150956627-d2d495f8-4fb1-493d-b49f-8e73bc224d9d.mov
What version of Spree integration are you using?
4.3.0
What version of Node.js are you using?
14.18.0
What browser (and version) are you using?
Firefox
What operating system (and version) are you using?
macOS
Relevant log output
No response
Code of Conduct

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

#122 seems to resolve this issue


Fix install

Certain deps aren't transferring over when installing.

Use CPack to build the Debian package

Remove portable packaging, and use build.sh

#18


Upgrade repo-tools and regenerate scaffolding.

Codecov Report

Merging #19 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #19   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines        1396   1396
=====================================
  Hits         1396   1396

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 5c8f422...9040e38. Read the comment docs.


Try re-enabling node 6.3

6.3 is working for me locally now, so trying this here.

Interestingly, this works for me locally now (I got the same break appveyor is showing before, but now i don't get that with my last change).

Depends on https://github.com/nodegit/nodegit/pull/1125
Not comfortable calling this the correct long term fix (not sure if it links to the built in or our own when both are available) but better than not building at all. Thanks to @implausible for helping find this.


NullPointerException in SearchResult.java when getHits(Map.class)

Exception:
java.lang.NullPointerException at io.searchbox.core.SearchResult.getHits(SearchResult.java:80)
Line of code:
obj = ((JsonObject) obj).get(keys[i]);
Suggested cause:
If explain is set to false in the query builder (searchSourceBuilder.explain(false)) there is no hits/hits/_source path element. It stops at the first hits, eg:
{"took":3,"timed_out":false,"_shards":{"total":5,"successful":5,"failed":0},"hits":{"total":0,"max_score":null,"hits":[]}}

Just realised that this is wrong. Closing it and cancelling pull request.


Use embedded Flux manifests for air-gapped bootstrap

This PR allows running bootstrap in air-gapped environments by embedding the Flux manifests in the provider binary.
Description
With embedded_manifests enabled and registry set to an private container registry where the Flux images are copied, users can run bootstrap on air-gapped environments where access to GitHub is denied.

[!TIP]
All users should enable embedded_manifests instead of setting version, this not only reduces the network traffic but also ensures that the Flux deployment matches the provider version.

Motivation and Context
The implementation in #503 has many flaws, the major problem is that customisations no longer work and users have to download the manifest files by hand before running bootstrap.
Fix: #590
Fix: #634
How has this been tested?

[x] Have you added an acceptance test for the functionality being added?

Manually tested with github.com dropped in firewall.
resource "flux_bootstrap_git" "this" {
  depends_on = [github_repository_deploy_key.this]

  path = "clusters/air-gapped"
  embedded_manifests = true
  registry = "docker.io/fluxcd"
  kustomization_override = file("${path.root}/resources/flux-kustomization-patch.yaml")
}

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Documentation

[x] I have updated the documentation (if required) with make docs

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

This whole section should be in the documentation that gets rendered on the Terraform registry.

Which one is that, haven't you deleted all guides? Better to have this here https://fluxcd.io/flux/installation/configuration/air-gapped/


Consider creating additional custom game rules.

~Player starting health adjusted~https://github.com/orendain/DemeoMods/issues/47
Pots have AoE effect
~Sorc has 2 zaps~https://github.com/orendain/DemeoMods/issues/35
non-class cards allowed when mana hits 100
Players start with specific cards.

start w/ chest card.

assassin has reusable arrow card

Universal Player game mode:

Players start with all starting class cards (or random mix).
All players can get all class cards.
Increased card hand count to compensate.

Hunter's paradise game mode:

Only pets + turret cards + hunters mark + lures
~Increased mana gain.~ https://github.com/orendain/DemeoMods/issues/61

Godmode but timelimit:

Players have godmode and/or uber powers.
Game ends if players don't complete game within X rounds.
On each round, print number of rounds left.

Some ideas from https://www.reddit.com/r/Demeo/comments/ou1gss/modding_demeo:

Increase HP for Lure, Behemoths, Ballista, Verochka
Healing ward gets 4 AP per round.
0AP for Wizards zap and repair armour
Prevent enemies opening doors.
Increase HPs replenish by potions and fountains.
Chests give two cards
More chest spawns

Could this be leverage to perhaps Difficulty Settings?
Like a rule set for 'Easy'

No monster spawn
Scaling reduced by 1 (so 4 players is like 3 and so on)

Then 'Normal' is well normal
Then 'Hard'

Scaling increased by 1

This is also another very much requested feature

Hey @Buruko,
Now that a few more rules have been implemented (current list of available rules listed here, which exact rules/values would you suggest exist for each ruleset?

Closing as rules/rulesets have been moved to their own top-level issues.


Allow React 17 in peer dependencies

React 17 should not break this package, this PR simply adds React 17 to the allowed peer dependencies.

I would rather you use the same style that the react-native peer dependency uses. Can you change it to that style please?

react-native peer dependency does not allow for React 16 anymore, so it is actually advisable to keep both here since the library should work just fine with both.

EDIT: I misunderstood your comment, please disregard my reply. However I'm not sure how you expect to change this. Like, including versions between 16 (inclusive) and 18 (exclusive)?

Yes, I think that would be fine for now 🙂 thank you


Check decimal overflow at compile time

Purpose
Fixes #37541

Decimal MAX and MIN are determined based on https://speleotrove.com/decimal/dbover.html

Decimal MAX and MIN are determined based on https://speleotrove.com/decimal/dbover.html

According to this
Decimal Max value contains

34 digits in significand
6144 in exponent(In ballerina spec this value is 6111)
So the following program should be allowed. Isn't it?

public function main() {
    decimal k =<PHONE_NUMBER>999999999999999999999999E6144d;
}

Decimal Min value contains

34 digits in significand
-6143  in exponent(In ballerina spec this value is -6176)

According to that following program should be disallowed.
public function main() {
    decimal k = -9999999999999999999999999999999999E-6178d;
}

decimal z = 9.999999999999999999999999999999999e6144 * 9.999999999999999999999999999999999e6144;

According to the spec https://ballerina.io/spec/lang/master/#section_<IP_ADDRESS>,  this is runtime decimal overflow. Right?
Shall we include the runtime changes as well?

decimal z = 9.999999999999999999999999999999999e6144 * 9.999999999999999999999999999999999e6144;

According to the spec https://ballerina.io/spec/lang/master/#section_<IP_ADDRESS>, this is runtime decimal overflow. Right? Shall we include the runtime checks as well?

Runtime team is working on that

decimal z = 9.999999999999999999999999999999999e6144 * 9.999999999999999999999999999999999e6144;

According to the spec https://ballerina.io/spec/lang/master/#section_<IP_ADDRESS>, this is runtime decimal overflow. Right? Shall we include the runtime checks as well?

Runtime team is working on that

According to this Decimal Max value contains

34 digits in significand
6144 in exponent(In ballerina spec this value is 6111)
So the following program should be allowed. Isn't it?

No, we need to consider about the scientific notation, Otherwise 6144 is not the maximum value, please go through the last paragraphs of pointed documentation.

Decimal Min value contains

34 digits in significand
-6143  in exponent(In ballerina spec this value is -6176)

We are not working on smallest values in this PR.

According to that following program should be disallowed.
public function main() {
    decimal k = -9999999999999999999999999999999999E-6178d;
}

This is incorrect because k = -9.999999999999999999999999999999999E-6145d, which is less than E_min (-6143)


Python: 06-memory-and-embeddings.ipynb -  AzureCognitiveSearchMemoryStore not loading

Describe the bug
In the python/notebooks/06-memory-and-embeddings.ipynb, when using the new SK
from semantic_kernel.connectors.memory.azure_cognitive_search import (
AzureCognitiveSearchMemoryStore
)
the function itself is not loading with the following error
ImportError: cannot import name 'HnswVectorSearchAlgorithmConfiguration' from 'azure.search.documents.indexes.models' ...\sk.venv\Lib\site-packages\azure\search\documents\indexes\models_init_.py)
To Reproduce
Steps to reproduce the behavior:

create .venv with the requirements.txt below
run python/notebooks/06-memory-and-embeddings.ipynb and then get the import error

Expected behavior
library should load without issue
Screenshots

Platform

OS: Windows
IDE: VS code
Language: Python
Source:  pip

Additional context
Requirements.txt has the following
PyPDF2==3.0.1
nltk==3.8.1
pycryptodome==3.15.0
streamlit==1.31.0
azure-search-documents==11.4.0
semantic-kernel==0.5.1.dev0
azure-ai-formrecognizer==3.32
libraries used
import os
import re
import nltk
import yaml
import asyncio
import PyPDF2
import numpy
from dotenv import load_dotenv
import semantic_kernel as sk
import semantic_kernel.connectors.ai.open_ai as sk_oai
from semantic_kernel.connectors.ai.open_ai import (
OpenAIChatCompletion,
OpenAITextEmbedding,
AzureChatCompletion,
AzureTextEmbedding,
)
from semantic_kernel.connectors.openapi import register_openapi_plugin
from semantic_kernel.connectors.ai.open_ai.models.chat.open_ai_chat_message import (
OpenAIChatMessage,
)
from azure.ai.formrecognizer import DocumentAnalysisClient
from azure.core.credentials import AzureKeyCredential
from semantic_kernel.text import text_chunker as tc
from semantic_kernel.connectors.memory.azure_cognitive_search import (
AzureCognitiveSearchMemoryStore
)

Also have tried. azure-search-documents==11.4.0b and also has not worked

works with pip install azure-search-documents==11.4.0b9


[Bug] Error: bind EADDRINUSE <IP_ADDRESS>:9800

What happened?
A bug happened!
Version
2.0.16
Distribution
Desktop
OS
Windows
What browsers are you seeing the problem on?
Chrome
If applicable, What Listener Clients are You Using?
 Broadcasting on Port: 5958
[12/16/2021, 11:53:23 AM] Finished VMix Emulation Setup. Listening for VMix Tally Connections on TCP Port 8099.
[12/16/2021, 11:53:23 AM] Tally Arbiter running on port 4455
[12/16/2021, 11:53:51 AM] Source Added: Carbonite
[12/16/2021, 11:53:51 AM] Source: Carbonite  Creating Ross Carbonite UDP Connection.
[12/16/2021, 11:53:51 AM] Source: Carbonite  Ross Carbonite Server started. Listening for data on UDP Port: 9800
[12/16/2021, 11:53:51 AM] Config file saved to disk.
[12/16/2021, 11:53:51 AM] Caught exception: Error: bind EADDRINUSE <IP_ADDRESS>:9800

Error stacktrace (if applicable)
Error: bind EADDRINUSE <IP_ADDRESS>:9800
    at dgram.js:337:20
    at processTicksAndRejections (internal/process/task_queues.js:81:21)

What else are you running on this machine that’s using port 9800? Can you just pick a different port?

Turns out a restart of the computer fixed it. Imagine that!


Add support for pulling docker images in crank jobs

Crank supports launching executables, building docker images etc we should support pulling docker images directly on the agent.

We have docker build support with the dockerFile crank argument, docker load support with the dockerLoad crank argument, so the action should be to have a docker pull support with a dockerPull crank argument.
The code would go there:
https://github.com/dotnet/crank/blob/main/src/Microsoft.Crank.Agent/Startup.cs#L2082


Resolve AY2122S2-CS2103T-T11-4/tp#11

Let's

Remove /data/ from .gitignore to track the new sample data file
Add sample data which is tailored to a contractor payroll management context
Add the Codecov badge to README.md

Codecov Report

Merging #12 (24baeec) into master (2bae734) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #12   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 2bae734...24baeec. Read the comment docs.


"Command 'mkdocs' not found" after following the installation instructions

I tried to run/build the website locally. All steps went well until mkdocs serve which output:
Command 'mkdocs' not found, but can be installed with:
    sudo apt install mkdocs

The python -m pip install -r requirements.txt did raise some warning that the installed packages were not added to the path but adding the install directory to the path did not solve the issue.
However, running python -m mkdocs serve works - could this be an issue with multiple installations/versions of python ?

Yup this totally looks like your python distribution is not on the PATH.
python -m mkdocs serve

doesn't look like a bad workaround :)

Adding the correct path to the PATH solved the issue (in my case $HOME/.local/bin).


Issue with matching projects between Asana and Track in Asana's list view

OS version: Windows 10
Browser version: Chrome 87.0.4280.66
Extension version: 1.66.10

Relevant integration (if any):
https://app.asana.com
🐛 Describe the bug
The user reported that after importing all their Asana projects into Track the extension will recognize the project it needs to apply automatically when the extension is used in task view but it will not "see" which project to apply in list view in Asana's web app.
Expected behaviour
The extension should match the project by name between Asana and Track (provided it already exists in Track under the same name) both in Task view and List view when the Track extension is used with the Asana web app.
Steps to reproduce
Steps to reproduce the behaviour:

Get the latest version of the Track extension
Enable the Asana integration
Make sure to import projects from Asana (or at least manually create a project with the same exact name as the one you are testing in Asana)
Start a timer from Task view within that project in Asana to see if the project is automatically applied via the extension
Try the same in list view for that project in Asana

Other details or context
Intercom reference
Browser outdated, a possible cause of issue perhaps?

closing this on behalf of #1885


[5.6] Distinct validation rule can optionally ignore empty values

ignore_empty_values parameter allows ignoring the empty values.
Use case
A table of 10 authors where only one is required at least.
Without this flag, the validation will fail because of the other 9 empty email fields.

EDIT: By the client request, all the 10 rows must be visible from the start, which means the 10 email fields exists in the form and will be submitted to the server.

@morloderex
required doesnt help here
this is for optional distinct fields

Just filter your input of empty values before giving it to the validator in the first place.


Accessing MySQL Database remotely.

First of all, we would like to thank and congratulate for a wonderful project. The design and overall flow is very user-friendly and smooth.
We are trying to integrate this SPOJ along with our web application for project purposes. In this regard, we need to connect to the MySQL Database remotely (basically, to register on Aurora SPOJ automatically, when the user registers on our application).
The main idea is to eliminate the double registration process for the user. Can you please help us on how to connect to the database remotely? We want to try the automation of the registration from our end.
We are accessing the docker application without any issues as a stand-alone instance..
Thanks in advance!

First of all, let me reiterate that you should follow our wiki for using this in production. Otherwise, a user will be able to run a program and get away with your user information.
Secondly, you can expose the MySQL port just by adding ports option in the docker-compose file. Read more about it here.

Closing the issue. Reopen if it needs further clarification.


openid logout-url no longer working when update to 2.5.0

I tried updating from 2.3.0 to 2.5.0. Here is my application.yaml:
proxy:
  ...
  authentication: openid
  openid:
    roles-claim: cognito:groups
    auth-url: https://{auth_domain}/oauth2/authorize
    token-url: https://{auth_domain}/oauth2/token
    jwks-url: https://cognito-idp.{region}.amazonaws.com/{user_pool_id}/.well-known/jwks.json
    logout-url: https://{auth_domain}/logout?response_type=code&client_id={client_id}&redirect_uri=https://{host_url}/login/oauth2/code/shinyproxy
    client-id: ...
    client-secret: ...
  ...
server:
    useForwardHeaders: true # won't redirect to http (need for cognito)

When I updated to 2.5.0 I changed useForwardHeaders: true to   forward-headers-strategy: native. Login works fine but when I logout I get redirected to /auth-error whereas with 2.3.0 it would return to the login page.
The logs show the user logged out, proxy released, then this error:
2021-08-14 13:06:39.567 ERROR 1 --- [XNIO-1 task-4] e.o.c.a.i.OpenIDAuthenticationBackend    : org.springframework.security.oauth2.core.OAuth2AuthenticationException: [invalid_request]

org.springframework.security.oauth2.core.OAuth2AuthenticationException: [invalid_request]
        at org.springframework.security.oauth2.client.web.OAuth2LoginAuthenticationFilter.attemptAuthentication(OAuth2LoginAuthenticationFilter.java:156) ~[spring-security-oauth2-client-5.3.4.RELEASE.jar!/:5.3.4
.RELEASE]
...

Any suggestions?

Hi, I think the problem is that after logout you redirect the user back to login/oauth2/code/shinyproxy. This URL is normally used inside the OpenID authentication process. In that process the request to that URL contains some more parameters. However, if those parameters are missing, you'll get an error.
Can you try changing the redirect_uri parameter to something different? You can simply redirect back to https://{host_url}/ if you want that the user gets back on the login page. Alternatively, you could use https://{host_url}/logout-sucess to show a "logout success" message to the user. See https://shinyproxy.io/documentation/configuration/#miscellaneous-settings for more information.

I tried both and I get a redirect_mismatch error from the browser with this (no errors in shinyproxy logs):

I figure out a solution looking here. Basically the redirect_uri approach that AWS allows doesn't seem to work with shinyproxy 2.5.0 but the logout_uri does. I changed it to this:
logout-url: https://{auth_domain}/logout?client_id={client_id}&logout_uri=https://{host_url}

Hi @alexvpickering, thanks for sharing the solution!
Since I think your question is solved, I'm closing this issue, feel free to open a new issue if you experience a problem with ShinyProxy.


Combine/group work on .selector encapsutated media queries

Hi and thx for your work,
i just realised that media-queries are combine if they are defined in a selector, and not if they are the first level of css declaration.
example:
.test {
    @include media('>phone','<tablet'){
        color: blue;
    }
    @include media('<desktop'){
        color: green;
    }
}

result:
@media (min-width: 431px) and (max-width: 767px) {
  .test {
    color: blue;
  }
}

@media (max-width: 1023px) {
  .test {
    color: green;
  }
}

if i use it in multiples files like this:
// Layout.scss
@include media('<desktop'){
    *, *:before, *:after {
      box-sizing: border-box;
    }

    body,
    body.large,
    body.etroit{
        min-width: 100%; // Surcharge des spip.css
    }
   ...
// Navigation.scss
@include media('<desktop'){
    #bando_navigation {
        border-right: 1px solid silver;
        width: 120px;
        min-height: 100%;
        position: fixed;
        top: 0;
        left: 0;
  ...

the grouping is ignore and there is one media-querie by imported files .
does i'm right ?

Hey @mistergraphx,
I think I need a you to help me understand use case.
Your code looks like SCSS code. group-css-media-queries works with CSS files (usually output after SCSS processor). It should work fine (group media queries in correct order) with example of a result you've provided.
If it does not work as you expect, please provide an input CSS file, output and expected output.


Implement key presses

Currently there is no keypress support.
The following loop must be modified:
https://github.com/mratsim/chirp8/blob/ba98bdbd09166958df6aea19b135659fe1e2a17a/src/chirp8.nim#L34-L44
KeyDOWN matches pressing
KeyUp matches release
So something like that can be done
 while gameState.running:
   while sdl2.pollEvent(gameState.event):
     case gameState.event.kind:
     of QuitEvent:
       gameState.running = false
       break
     of KeyDown:
       case event.key.keysym.sym:
       of K_KP0: foo(key0)
       of K_KP1: foo(key1)
       ...
     else:
       break

Keycodes are available here: https://github.com/nim-lang/sdl2/blob/master/src/sdl2/private/keycodes.nim
A new "key" array should be added to store the key states (up or down)
Layout
The original key layout is this:

with 2-4-6-8 corresponding to up left right down.
Ideally Chirp-8 should read mapping from a .cfg to so that people can customize their keyboard mapping:

laptops without a keypad
non-QWERTY keyboard
...

cc @Vindaar

With commits https://github.com/mratsim/chirp8/commit/fa809d48a56567169f92cd434b17b9f7053dcd6f and https://github.com/mratsim/chirp8/commit/1c0d7481c13390b3dc6ab445d72d885c31656db7
basic key input is in place now, including custom keybindings set in config/keybindings.cfg.
The F00A instruction is the only one still missing.


Support specifying a bind address

The current code always binds to <IP_ADDRESS>.  This adds support to specify binding (for example) to <IP_ADDRESS> to ensure the proxy is only accessible via localhost.
Example:
export PW_BIND_ADDRESS=<IP_ADDRESS>;python ./server.py

Thanks @zi0r !  Nice addition.


fixordering exprs should be substituted in the same way as select part

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.62 seconds
stream load tsv:          443 seconds loaded<PHONE_NUMBER>9 Bytes, about 161 MB/s
stream load json:         23 seconds loaded<PHONE_NUMBER> Bytes, about 97 MB/s
stream load orc:          60 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          78.2 seconds inserted 10000000 Rows, about 127K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230526053448_clickbench_pr_151080.html

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.87 seconds
stream load tsv:          446 seconds loaded<PHONE_NUMBER>9 Bytes, about 159 MB/s
stream load json:         22 seconds loaded<PHONE_NUMBER> Bytes, about 102 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          26.0 seconds inserted 10000000 Rows, about 384K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230527024225_clickbench_pr_151735.html


fix(katana/rpc): return events in chunks with continuation token

Fixes: #668
discussion related to this in discord thread: https://discord.com/channels/1062934010722005042/1125377039248142356

@kariy does our discussion on discord resolves rest of open question?

@kariy does our discussion on discord resolves rest of open question?

Yes! thanks for the clarification


Hiding bar shadow

Hello,
The method BottomBar.hideShadow() has apparently been removed in the v2.
How can I achieve the same action now ?
Thank you!

Possible in the upcoming version

Well ... that was quick !
👍


Download link failed,shapenet_part_seg_hdf5_data.zip

Thanks for your great work, the download link in zeroshot_seg/data.py have failed,could you offer a new one? Thanks!
www = 'https://shapenet.cs.stanford.edu/media/shapenet_part_seg_hdf5_data.zip'

@yangyangyang127 @ZrrSkywalker


Doesn't work properly

When I have this plugin enabled, after a while (can't pinpoint it to an action) there is a CodeHelper process stuck at 100% CPU usage. Hasn't happened anymore since I disabled it.
I think it's related to Intellisense because when it happens, usually the hints stop updating (e. g. if I update a JSDoc comment). Any idea what it could be? Only other related plugin I have installed is tslint but that should only be enabled for .ts files.
Is there any way I can enable more verbose logging?

I've been using your version and no problems so far!


Feature: CPU metrics

Possibly relevant WMI class: Win32_PerfRawData_Counters_ProcessorInformation, provides these fields:
Name (socket#+core#)
AverageIdleTime
AverageIdleTime_Base
C1TransitionsPersec
C2TransitionsPersec
C3TransitionsPersec
ClockInterruptsPersec
DPCRate
DPCsQueuedPersec
IdleBreakEventsPersec
InterruptsPersec
ParkingStatus
PercentC1Time
PercentC2Time
PercentC3Time
PercentDPCTime
PercentIdleTime
PercentInterruptTime
PercentofMaximumFrequency
PercentPerformanceLimit
PercentPriorityTime
PercentPrivilegedTime
PercentPrivilegedUtility
PercentPrivilegedUtility_Base
PercentProcessorPerformance
PercentProcessorPerformance_Base
PercentProcessorTime
PercentProcessorUtility
PercentProcessorUtility_Base
PercentUserTime
PerformanceLimitFlags
ProcessorFrequency
ProcessorStateFlags

Haven't found any MSDN documentation of this class, so potentially there might be better ones? It was the only one reported on my test machine, though

Win32_PerfRawData_PerfOS_Processor might also be a candidate

https://msdn.microsoft.com/en-us/library/aa392397(v=vs.85).aspx mentions "Classes derived from Win32_PerfRawData and from Win32_PerfFormattedData must be used with a refresher object.", not sure we currently have ability to probe these using the current wmi lib. (research needed)

The "refresher object" here is Prometheus.

Did we resolve this by merging #26 ?

Yes, I think so


Logo design contribution

Hallo
I am Masud Rana.
Want to male a logo for this projects.if you have any requirements then let me know.its totally free for open source projects.

Thanks for volunteering to make a logo! We've already started work on this internally.

its okey


Use new ESLint API

This PR refactors to use the new API instead of the old ESLint API.
See https://eslint.org/blog/2023/09/preparing-custom-rules-eslint-v9/
They are also present in the minimum version we support.

Code looks good, but I'm a bit worried about updating eslint and @types/eslint. Wouldn't this enable us to use newer API functions than our minimum supported ESLint version? I'm not sure if this will be an issue, but ideally we could use TS to enforce our minimum supported ESLint version. What do you think?

I think we should use<EMAIL_ADDRESS>as you say :+1:
I made sure that<EMAIL_ADDRESS>and<EMAIL_ADDRESS>are used in package-lock.json so that we can use<EMAIL_ADDRESS>in our tests.


Data placed in 'main_' prefixed schema

Whenever I specify a schema, e.g. raw, the data gets placed in a schema prefixed with main_, e.g. main_raw.
Take jaffle_shop_duckdb, if I specify a schema for my seeds to be placed into:
seeds:
  +docs:
    node_color: '#cd7f32'
  +schema: seeds

I'd expect data to land in seeds.raw_customers but instead it lands in main_seeds.raw_customers.

yah this is a standard dbt thing https://docs.getdbt.com/docs/build/custom-schemas


Add a tag and release for version 2.0

Apparently version 2.0 was released on December 7 (f9dc8ecf757d91006a718fefc7a366a4a30d6b73), but it's still missing a tag and a release in the github project. Looking at the starting page and the releases page, it looks like the latest release is 1.9.0, which is very old by now.
Also it would be nice if a short description could be added in the project's "About" field. Currently it just says: No description, website, or topics provided.
Might be sufficient to just use OpenDroneID Android receiver application, maybe with a link to http://www.opendroneid.org?

Good points. Fixed now.

Good points. Fixed now.


ci: frozen lockfile when install in ci

Checklist

[x] Fix linting errors
[x] Tests have been added / updated (or snapshots)

:tada: This PR is included in version 1.0.0-alpha.39 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adding apptoapp.rst page

Documentation to describe the process of using an app to app sign in
flow.

Added the app to app sign in page which @karvell wrote, and started populating the FAQ.


Start of massive refactor to set targets and source properly

TcgStatics changes show the  changes that should be made to delayed effects to set targets and sources properly. I haven't really thought much about setting EffectTypes yet, but will probably work well in addition to the new event management.
The Unbroken Bonds STEALTHY_HOOD change shows how the new event management in the sister PR in the engine can be used to ensure we only block effects from abilities used by the opponent's Pokémon. Currently both resolvedTarget and thisPokemon are checked (all Groovy delayed checks extend AbstractDelayedMoveEffect so currently thisPokemon will work on more abilities this way for now, but once all abilities set their target

@flagrama any updates on this?

This is the sister PR to PR 204 in the engine repo. My questions over there pertain to this, but I'm discussing over there since the solution is going to require changes to the engine.


Make accessible offline

Is your feature request related to a problem? Please describe.
Since there's no real backend functionality, having the app available offline would be awesome.
Describe the solution you'd like
The simplest way to make an AstroJs app a PWA, doesn't need to be SPA, MPA is fine.
We don't want an install app button, just the offline capacity.
Describe alternatives you've considered
NA
Additional context
NA

Having some weird dev issues, made this request: https://discord.com/channels/830184174198718474/1088190877438050344

Using this for the service worker: https://vite-pwa-org.netlify.app/frameworks/astro.html
However, not all files are cached, maybe check this: https://blog.logrocket.com/how-to-build-a-progressive-web-app-pwa-with-node-js/


Fix sorting for diff on delete

Per discussion point raised https://github.com/Azure/AzOps/discussions/878

@hazvk please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree


Fixed behaviour with slice() to return empty array when sliced beyond bounds

Suppose we have an xarray dataset with a coordinate axis {x_0 ... x_N}. When we have a slice for two values a and b xarray should return all values with coordinates between a and b regardless of whether a is less than x_0 or b is greater than x_N. This is working as expected, however it should also be the case that if $[a,b]$ does not intersect with the coordinate set (e.g if both a and b are less than x_0), that an empty xarray dataset should be returned. This behaviour can be seen if a dataset is read from a NetCDF file and we lazily slice the array before loading it. However as eratos-xarray is currently, no check is done before slicing is done using the Eratos API resulting in an error. I've made some adjustments to the indexing method in the backend array and added a small test to check.

LGTM. Just remove that pycache directory please

Done, let me know when it's merged so I can change the version for one of my models

@steven-eratos Could you increment the minor version of the package and we can merge this in to test with your pypi testing.


Fixed unit tests that were broken before 1.13

The unit tests on builds prior to 1.13 were failing because I was patching the wrong function. Prior to 1.13, formatter_for_file didn't exist, but the unit tests would patch that but the function_exported? check in source_file.ex would skip that branch and succeed, which would cause the tests to fail.
The change is simple, check to see if formatter_for_file exists, and if it doesn't, patch the formatter_opts_for_file function

Nice @scohen


Updating docs with new tag version

This should fix #363

@googlebot I signed it!

Thanks for the quick fix! :)


Propagate errors to KameletBinding status

When there is an error in the operator during the reconciliation of a KameletBinding (e.g. a trait raises some error) the KameletBinding should reflect this error state in its status.
The actual behavior is that the KameletBinding is stuck in "Creating" phase and never recovers from that state.
To reproduce this behavior add a KameletBinding with an unknown Kamelet as a source or sink. The KameletBinding status should reflect the error state and provide some proper reason indicator in its condition status.

I had a look at this and I don't think it's a bug but the way it is designed the system so far (we may understand if it makes sense to change it though). Right now, if an Integration is in error state, then, the related KameletBinding is moved in error as well (tested with nightly build).
The scenario illustrated is more generic because we never set an Integration in error when the traits are failing, but it is infinitely reconciled to watch for any change that may be applied to the Integration CR in order to have it fixed. That's the reason why we don't cascade the failure to the KameletBinding.
We may reason if it makes sense to apply some retry logic (as we do for the Build) or at least report the failure in the Integration status, as it is really difficult to understand what's going on when this situation happen (until we check the operator log).
Adding @astefanutti @lburgazzoli @oscerd to the thread for more opinions.

We should review how the re-queue of reconciliation events is handled by controller-runtime.
For errors, in general, we should configure the work-queue to use a rate-limited queue with an exponential failure rate limiter, cap the maximum number of retries, and eventually report the error in the custom resource status and forget the element from the queue. For functional errors, that we know won't be resolved by retrying, we may want to shortcut that mechanism and directly report the error in the custom resource status.


gulp unit not working

$ gulp unit
[14:12:19] Using gulpfile C:\Dev\angular2-seed\gulpfile.js
[14:12:19] Starting 'unit'...
[14:12:19] Starting 'karma'...
14 12 2015 14:12:19.457:WARN [watcher]: Pattern "C:/Dev/angular2-seed/build/js/**/*.js" does not match any file.
14 12 2015 14:12:19.637:INFO [karma]: Karma v0.13.15 server started at http://localhost:9876/
14 12 2015 14:12:19.644:INFO [launcher]: Starting browser Chrome
14 12 2015 14:12:21.250:INFO [Chrome 47.0.2526 (Windows 10 0.0.0)]: Connected on socket qq_LWE1d1mhxfVlYAAAA with id 64167172
14 12 2015 14:12:22.201:WARN [web-server]: 404: /base/build/js/hero/hero.service
Chrome 47.0.2526 (Windows 10 0.0.0) ERROR
  Error: Unable to load script C:/Dev/angular2-seed/build/js/hero/hero.service
      at HTMLScriptElement.error (C:/Dev/angular2-seed/node_modules/systemjs/dist/system.src.js:2506:18)
      at Zone.run (C:/Dev/angular2-seed/node_modules/angular2/bundles/angular2.dev.js:138:17)
      at HTMLScriptElement.zoneBoundFn (C:/Dev/angular2-seed/node_modules/angular2/bundles/angular2.dev.js:111:19)

[14:12:22] 'karma' errored after 3.22 s
[14:12:22] Error: 1
    at formatError (C:\Users\Genad_000\AppData\Roaming\npm\node_modules\gulp-cli\lib\versioned\^4.0.0-alpha.2\formatError.js:20:10)
    at Gulp.<anonymous> (C:\Users\Genad_000\AppData\Roaming\npm\node_modules\gulp-cli\lib\versioned\^4.0.0-alpha.2\log\events.js:26:15)
    at emitOne (events.js:82:20)
    at Gulp.emit (events.js:169:7)
    at Object.error (C:\Dev\angular2-seed\node_modules\undertaker\lib\helpers\createExtensions.js:58:10)
    at handler (C:\Dev\angular2-seed\node_modules\now-and-later\lib\mapSeries.js:43:14)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at done (C:\Dev\angular2-seed\node_modules\async-done\index.js:24:15)
    at removeAllListeners (C:\Dev\angular2-seed\node_modules\karma\lib\server.js:332:7)
    at Server.<anonymous> (C:\Dev\angular2-seed\node_modules\karma\lib\server.js:343:9)
    at Server.g (events.js:260:16)
    at emitNone (events.js:72:20)
    at Server.emit (events.js:166:7)
    at emitCloseNT (net.js:1521:8)
    at doNTCallback1 (node.js:441:9)
[14:12:22] 'unit' errored after 3.23 s
[14:12:22] Error: 1
    at formatError (C:\Users\Genad_000\AppData\Roaming\npm\node_modules\gulp-cli\lib\versioned\^4.0.0-alpha.2\formatError.js:20:10)
    at Gulp.<anonymous> (C:\Users\Genad_000\AppData\Roaming\npm\node_modules\gulp-cli\lib\versioned\^4.0.0-alpha.2\log\events.js:26:15)
    at emitOne (events.js:82:20)
    at Gulp.emit (events.js:169:7)
    at Object.error (C:\Dev\angular2-seed\node_modules\undertaker\lib\helpers\createExtensions.js:58:10)
    at handler (C:\Dev\angular2-seed\node_modules\now-and-later\lib\map.js:46:14)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at done (C:\Dev\angular2-seed\node_modules\async-done\index.js:24:15)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at handler (C:\Dev\angular2-seed\node_modules\now-and-later\lib\mapSeries.js:44:16)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at f (C:\Dev\angular2-seed\node_modules\once\once.js:17:25)
    at done (C:\Dev\angular2-seed\node_modules\async-done\index.js:24:15)
    at removeAllListeners (C:\Dev\angular2-seed\node_modules\karma\lib\server.js:332:7)
    at Server.<anonymous> (C:\Dev\angular2-seed\node_modules\karma\lib\server.js:343:9)

Did you build the project in dev first?

I don't have a Windows env here at the moment but the glob "/build/js/*/.js" looks unusual to me. Did you change it?

No it's not the reason

Was there any file generated under the folder C:/Dev/angular2-seed/build/js?


Capacitor 4 support

Due to Google Play publishing requirements Capacitor plugins must update to the latest version.
Updates the core capacitor plugin to 4.0.1
Android minsdk bumped to 22
iOS platform bumped to 13
Android Branch SDK updated to 5.2.3 which targets 32 as well.
Dependency type changed from implementation to api to keep references consistent and integration simpler.
For testing please refer to https://capacitorjs.com/docs/updating/4-0

Thanks for the reminder. I had intended to commit this PR with the alpha tag but I'll just properly set the release ver here and there to avoid these getting out of sync.


Better utilization of first TXT RR

Making sure the first lookup contains a full datagram.
Added exception to ensure first RR label maches the domain rather than the
spf#.domain pattern.

Thanks for the patches. I see your point. The thing is there was a reason for the original way the records were utilized:
If you were manually changing the TXT records (e.g. via web interface of your DNS hosting), you could easily prepare all the records before changing the final one to point to the new chain. I.e. you could use _spf# records and spf# records alongside each other, always changing those which are not currently in use and by the final change (to spf.your-doma.in) you would make it use the recently changed records.
Does it make sense to leave it the way it is after this explanation?
Maybe my original idea is not important at all when taking into consideration DNS TTL.
What do you think?
Thanks!

Thanks for the explanation.  I still think spf-tools should try get every last drop out of SPF and fill up the first record.  Anyone doing it by hand (and can afford it) can still do the trick.  Since they're doing it manually anyway, they'd give the first record a chain-name like spf0.example.org.
Maybe compromise with a command line option to output all records and not stop at 10?
The current output would remain the same and those of use who need all 10 records full will just ignore the first short record.

Fair enough! Merging in.