fix: Send()-task is now properly awaitable and reacts to exceptions

Previously, exceptions triggered by the closure of the underlying TCP-Socket were not adequately relayed to the caller of Send(...). Consequently, awaiting the outcome of connection.Send(...) was not feasible. This problem was prevalent in all connections where the client lost connectivity due to network issues, as it would not dispatch a close-event.
This update addresses these shortcomings, enabling the following:
try {
    conn.Send("Hello World!")
} catch (Exception e) {
   // pursue appropriate handling
}

Furthermore, the Task returned by Send was always instantly completed, which is now fixed too.

:white_check_mark: Build Fleck 0.0.69-ci completed (commit https://github.com/statianzo/Fleck/commit/487b121e71 by @)

:white_check_mark: Build Fleck 0.0.69-ci completed (commit https://github.com/statianzo/Fleck/commit/487b121e71 by @)

Thanks for the contribution @VinzSpring. Is this something that should be fixed at the SocketWrapper.Send level so it returns a properly awaitable task?

@statianzo yes, fixing it at the SocketWrapper level would be an option too. Do you want me to update my pull request or is it fine as is?

If you could move it to the SocketWrapper, it may help establish a pattern for the other Task producing methods that may be in need of a change.


Allow PreferredMinSize larger than 500x500

There are some use cases where an application is simply not usable in small sizes, and anything other than setting an appropriate PreferredMinSize is just a workaround. (See 1)
In my opinion, style guidelines should be recommendations and not hard-coded limitations. I think this can be a useful feature in those use cases. The current state of affairs is documented here.

I disagree. When developing a UWP app, a big focus should be making it responsive so that it can run on many different display sizes with no problem. I absolutely hate Win32 apps that set a really big min size, which leads to me not being able to put that window side by side with another window on my Surface device. Ideally an app should be able to fit in one rectangle of the 2x2 grid that Windows 10 provides, which is especially useful when multitasking.
Also, that restriction is set by the OS and it may even be different depending on the OS variant. So my suggestion is to fit your app to that restriction instead of lifting it, mainly to provide a better user experience.

@Austin-Lamb can you please find the right owner for this?

@chigy - would you be able to help with this?


unread-counts and "last updated" for connections in feed-view

#1041 resolves the unread counts.
45d252c partly resolves the "last updated" for connections that have been viewed in detail before / for which new events have arrived. However, if the page has been reloaded anywhere else, the events aren't loaded due to selective loading and thus the timestamp can't be determined. We'd need to cache it server-side to correctly display it in all cases.


feat(PageSingleFeature): Abstract to PageSingleFeature

Overview

Add a PageSingleFeature component that abstracts away much of the logic that PageHero. PageHero is a special case of PageSingleFeature with specific media widths set and the heading set to the page heading.

PR Checklist

[x] Related to designs: https://www.figma.com/file/J5yB1jyELiNt3Vb47T9GcE/Job-Readiness?node-id=2%3A43
[x] Related to JIRA ticket: [REACH-1326]
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.97-alpha.2316fe.0@codecademy/gamut-labs@24.1.1-alpha.2316fe.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6154eec31f503d2a4809761f--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.97-alpha.322d9c.0@codecademy/gamut-labs@24.1.1-alpha.322d9c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6155d76fe49b82392b9158c5--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.97-alpha.b7153a.0@codecademy/gamut-labs@24.1.1-alpha.b7153a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://615648bdc9d45b0a5f071aff--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.99-alpha.f393b1.0@codecademy/gamut-labs@24.2.1-alpha.f393b1.0@codecademy/gamut@37.9.0-alpha.f393b1.0@codecademy/monaco@0.4.15-alpha.f393b1.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.5.101-alpha.64f7c7.0@codecademy/gamut-labs@24.3.1-alpha.64f7c7.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://615b6c08a4dce200e48afcd9--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.102-alpha.a8fab5.0@codecademy/gamut-labs@24.3.2-alpha.a8fab5.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://615b74091f5f4a10d83d4e1d--gamut-preview.netlify.appDeploy Logs


Could you provide more details about this project?

Question
Describe your question in ONE SENTENCE.

Can both parties get the intersection itself instead of its cardinality ?
What PSI protocols are implemented in this project?

The result will always be returned to the client, who can choose to share it with the server. If you want to reveal the intersection itself and not only its cardinality, set reveal_intersection = true when constructing client and the server.
We implemented a standard Diffie-Hellman-based PSI protocol, augmented with a bloom filter for more efficient communication. A description can be found here in the source code. We have an upcoming workshop paper describing the protocol, which will have more details.

Our paper on this PSI library can now be found on arxiv. It includes a formal description of our protocols. I hope this answers your second question, let me know if not.

Yes! Thanks for your explanation!


Update web3 to v0.9.0

@e00E Can you rebase this? Preferably also bump to web3 0.10.0 :)

Also, this can be merged once that's done. Since we removed graph dependency from services, we no longer have any motivation to keep the web3 version pinned back.


ITelemetryContext.user is sometimes null - setAuthenticatedUserContext throws

-Package: @microsoft/applicationinsights-web
-Package version: 2.6.1
-Installed using npm, built with webpack
-Using TypeScript
Just upgraded @microsoft/applicationinsights-web from version 2.5.8 to version 2.6.1.
I needed to update our code when I found that ITelemetryContext.user (IUserContext) has added two new methods; setAuthenticatedUserContext and clearAuthenticatedUserContext.
Up till now we have just set ai.context.user to an object literal that implements IUserContext when a user logs in (and set it to null when the user logs out) which worked fine.
Since the IUserContext interface now also requires these two new methods, I changed the code so that instead of setting/clearing the ai.context.user property, I rather call these two new methods ai.context.user.setAuthenticatedUserContext and .clearAuthenticatedUserContext when users log in/out.
The problem I faced is that ai.context.user sometimes is null,  and then an exception is thrown.
I believe that ai.context.user is supposed to never be null even if the user has not yet logged in, because how are you then supposed to set the authenticated user context?
It looks to me as if the ai.context.user is initialized asynchronously a bit too late.
It would expect this to initialized to something other than null immediately after ai.loadAppInsights() has been called.

@MSNev I'm sorry, I found the problem and the issue was actually on my side.
To my excuse - until recently, your TypeScript typings did not expose any other way to set the user context to my knowledge.
IUserContext.setAuthenticatedUserContext or IUserContext.clearAuthenticatedUserContext was missing.
You also mentioned that there is a method ai.setAuthenticatedUserContext in your comments.
If this exists, it is also not exposed through the typings.
Since ITelemetryContext.user is NOT readonly in the typings, and the object I assigned to this property implemented the expected interface (in version 2.5.8), I would argue that it was your typings that led me into doing what you tell me I should never do. According to the typings, what I did was perfectly fine.
I encourage you to tighten up the typings a bit and add readonly to properties/fields in the interfaces when they are meant to be readonly - to prevent others from doing things they should never do.
Thanks for your help!

I encourage you to tighten up the typings a bit and add readonly to properties/fields in the interfaces when they are meant to be readonly - to prevent others from doing things they should never do.

Agree, there is a bunch of things that are slowly being addressed including the previously missing xxxAuthenticatedUserContext  functions on the user object.

missing ai.setAuthenticatedUserContext

Will need to check this as it "should  be" as it's defined here, it's probably got to do with the dynamic nature of the snippet as it can be 1 of 3 different implementations.
I'll create am issue for the typings...

Created #1546

@MSNev Great, thanks 👍😃

v2.6.3 is now fully deployed


Schema dump fails with prefixed tables (specifically migrations)

Laravel Version: 8.17.0
PHP Version: 7.4.10
Database Driver & Version: MySQL 5.7.32

Description:
php artisan schema:dump does not seem to support databases with prefixes set. It throws this error:
vagrant@code-homestead:~/code$ php artisan schema:dump --database=mysql
mysqldump: [Warning] Using a password on the command line interface can be insecure.
mysqldump: [ERROR] unknown variable 'column-statistics=0'
mysqldump: [Warning] Using a password on the command line interface can be insecure.

   Symfony\Component\Process\Exception\ProcessFailedException

  The command "mysqldump  --user="${:LARAVEL_LOAD_USER}" --password="${:LARAVEL_LOAD_PASSWORD}" --host="${:LARAVEL_LOAD_HOST}" --port="${:LARAVEL_LOAD_PORT}" --skip-add-locks --skip-comments --skip-set-charset -
-tz-utc --set-gtid-purged=OFF "${:LARAVEL_LOAD_DATABASE}" migrations --no-create-info --skip-extended-insert --skip-routines --compact" failed.

Exit Code: 6(Unknown error)

Working directory: /home/vagrant/code

Output:
================

Error Output:
================
mysqldump: [Warning] Using a password on the command line interface can be insecure.
mysqldump: Couldn't find table: "migrations"

  at vendor/symfony/process/Process.php:257
    253▕      */
    254▕     public function mustRun(callable $callback = null, array $env = []): self
    255▕     {
    256▕         if (0 !== $this->run($callback, $env)) {
  ➜ 257▕             throw new ProcessFailedException($this);
    258▕         }
    259▕
    260▕         return $this;
    261▕     }

      +18 vendor frames
  19  artisan:37
      Illuminate\Foundation\Console\Kernel::handle()

Updating the config/database.php migrations value with the prefix fixes the issue but wouldn't that affect other areas i.e other areas calling that config variable and prefixing it (again)? I would think I'd only have to prefix the database connection details not the general migrations config variable.
Steps To Reproduce:

Create a new database
Update config/database.php and your MySQL connection being used with a prefix.
Run migrations to create the tables
Run php artisan schema:dump

I also noticed that if I was success to run schema:dump, the dump file isn't used while testing, it's not run, likely due to it exporting from a mysql driver and testing using the sqlite driver, but I don't see anywhere to specify the dump file to migrate or how to force it to use the mysql one.

Regarding your last comment - you need to use MySQL when running tests. You can't import a MySQL dump into SQLite.

@taylorotwell Thanks, good point. I can put in a PR to fix the the migration table/prefix issue?

Sure.

I see withMigrations method was added in commit 110eb15a77f84da0d83ebc2bb123eec08ecc19ca by @taylorotwell. I wonder if adding the prefix within that method is preferred or to just update schemaState in DumpCommand.php?

I think DumpCommand could just be updated at this point.

I think DumpCommand could just be updated at this point.

I've pushed a fix for this.

I've pushed a fix for this.


Missing block-preview.html for v2.0.0RC2

Which Umbraco.Community.BlockPreview version are you using?
v2.0.0RC2
Which Umbraco version are you using? For example: 10.4.0 - don't just write v10
14.1.1
Bug summary
Missing block-preview.html for v2.0.0RC2
Steps to reproduce
Install nuget
Expected result / actual result
See block-preview.html

Hi @parxal - now that Umbraco 14 is using Web Components, the block-preview.html file doesn't exist. Please follow the instructions in the v2 README to get up and running, specifically either one of these configurations:
Program.cs:
+using Umbraco.Community.BlockPreview.Extensions;

builder.CreateUmbracoBuilder()
    .AddBackOffice()
    .AddWebsite()
    .AddDeliveryApi()
    .AddComposers()
+   .AddBlockPreview(options =>
+   {
+       // This will enable the preview for every content type
+       options.BlockGrid.Enabled = true;
+   })
    .Build();

or appsettings.json:
{
  "BlockPreview": {
    "BlockGrid": {
      "Enabled": true
    }
  }
}


OGV governance deploy tooling

This PR updates our deploy tooling so it is able to:

on mainnet build out the proposal and output to and data that is to be copied to gnosis Safe to create a multisig transaction. With the old governance we were able to create a proposal directly on the governance contract, but OGV governance requires the proposer to have 10m veOGV balance. For that reason it is done via 5/8 multisig.
on forked net if governance proposal is already pending/Active/Succeeded/Queued the tooling is able to drive it to the end of its life-cycle and execute it. The only time this would fail is when there would be too much against votes already present on the proposal in forked environment.
on forked net if governance proposal hasn't been submitted yet. The tooling submits its, and does all the intermediate steps to execute it.
local node environment is not supported by this deploy tooling

To test that the tooling works correctly I have

simulated the execution of currently active proposal<PHONE_NUMBER>3407050666554191388123037800510237271029051418887027936281231737485
and executed the #48 deploy script that changes the vaultAdmin. This step would fail if I didn't first execute the first step (currently active proposal) with the error that underlying Timelock transaction has failed. Which is correct since the governance needs to first accept the governance of VaultProxy before it is able to change the VaultAdmin implementation.

I will also be using this tooling to submit the governance proposal to upgrade VaultAdmin to enable the new withdraw/deposit strategy functionality.

Current fork test CI is failing because the active proposal has no been executed yet


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Addition of DKES-related target to STELLOPT

I have been using DKES during optimizations to minimize the monoenergetic transport coefficients $D_{11}^{*}$. An additional target that would be helpful is the quantity $\alpha$, which is the average slope of the $D_{11}^{*}$ vs $\nu^{*}$ curves on a log-log plot. I propose adding a target according to the following pseudo-code (which assumes that a database of DKES outputs is already being calculated for each trial configuration in the optimization):
alphas_over_s = []
for S in DKES_surfaces:
    alphas_over_er = []
    for ER in DKES_radial_electric_field_values:
        alpha = slope of linear trend line that is fit to the (log(DKES VAL) vs log(DKES NU)) data for the given ER
        alphas_over_er.append(alpha)
    alphas_over_s.append(average(alphas_over_er))
return alphas_over_s

Thus, we would have one $\alpha$ value for each flux surface in the DKES calculation.
As a side note, DKES does not technically take $\nu^{*}$ as input or output $D_{11}^{*}$, but the conversions between these quantities and those DKES uses is a matter of normalization. I don't think normalizing the DKES quantities is necessary for the purpose of optimization.

Work has begun.

@leebr48 If you can provide a test input file which works for one surface for DKES, DKES_ALPHA, and DKES_ERDIFF for one iteration so we can include it as a regression test, that would be appreciated.  Single iteration would be fine.

@leebr48 Sorry branch is: feature\DKES_alpha

@lazersos I have done some quick testing with the new feature branch, which I built on Raven. When using any of the DKES-relevant objectives, I get MPI_FREE errors in the log.ext file. I don’t believe this was the case before. The regular DKES objective seems to print correct outputs to the stellopt.ext file despite the MPI_FREE errors. On the other hand, the DKES_ERDIFF objective does not appear to be working at all. It throws a segmentation fault when I try to use it. DKES_ALPHA also does not seem to be working. I notice that it seems to initialize DKES many times at the beginning of the run rather than just once. Then DKES does not seem to run at all - the STELLOPT run completes without another mention of DKES in the log.ext file, and no mention of it appears in the stellopt.ext file either. I have attached input files and log files in case they are helpful for troubleshooting.
input.dkes.txt
log.dkes_erdiff.txt
log.dkes_alpha.txt
log.dkes.txt
input.dkes_erdiff.txt
input.dkes_alpha.txt

@leebr48 Can you do a test for me, I've got raven setup in a production mode and can't rebuild stellopt right now (or for the foreseeable month).  Do you get the MPI_COMM_FREE errors if you just have NEO as a target?  What if you just have a target like ASPECT?  I'm gonna dig through the code this morning.

@leebr48 I can reproduce your errors on COBRA so I'll debug from there.

So there appears to be four issues:

The MPI Error on completion of a sucessful run.
The DKES_ALPHA spitting out the DKES header too many times.
DKES_ERDIFF crashing.
DKES_ALPHA not even running.

@leebr48 OK pull the branch, rebuild and test.  I fixed the above issues and all cases you supplied should now run without error.  I can confirm this for 2 Cobra nodes (80 threads).

@lazersos The issues you outlined appear to be fixed when I run on Raven (1 node, 72 cores). I am now a bit unsure of how to use the namelist inputs for the DKES_ALPHA objective. When I tried to make DKES_ALPHA use the same NU and ER values as the standard DKES objective, it calculated NaN for all the DKES_ALPHA objective values. I have attached the input.ext and stellopt.ext files so you can see what I did. I am almost certain I am just using the namelist incorrectly. I have attached a spreadsheet showing the calculations I was trying to get DKES_ALPHA to do. Would you mind giving me some pointers on how to use the namelist properly? In particular, I am not sure what the "P" and "M" tags indicate in this context.
input.dkes_alpha_2.txt
stellopt.dkes_alpha_2.txt
dkes_alpha_testing.xlsx

@leebr48 OK I should probably update the wiki now to make it clear.  However in short
TARGET_DKES_ALPHA and SIGMA_DKES_ALPHA are indexed over radial grid points as normal.
NUP_DKES_ALPHA, EP_DKES_ALPHA, NUM_DKES_ALPHA and EM_DKES_ALPHA are indexed so you can target multiple pairs on a given flux surface.  So in your case you've got 9 pairs over 3 surfaces for 27 targets.  The M and P are the minus and plus values.  So EP_DKES_ALPHA and EM_DKES_ALPHA should not be the same.  The NUs can be.  I'll update the wiki.


No implemantation of convolve_s8_kernel3_stride1_pad1_fpreq()

Hey, @meenchen
I found that there is no kernel of convolve_s8_kernel3_stride1_pad1_fpreq(),
Would you mind upload this kernel of tiny engine?
Or, How should I use int_forward_op/convolve_s8_kernel3_stride1_pad1.c to implement convolve_s8_kernel3_stride1_pad1_fpreq()?

Hi. @wslong36,
The only difference between fpreq and other ops is how outputs are re-quantized. fpreq means re-quantizing outputs with a fp32 scale (e.g., https://github.com/mit-han-lab/tinyengine/blob/fdc001922898df80fd07901f15a07a9e304ed705/TinyEngine/src/kernels/fp_requantize_op/mat_mul_kernels_fpreq.c#L294), while other ops follow the TFLite approach.
You can also use the fp_requantize flag in code generator to disable it (e.g., https://github.com/mit-han-lab/tinyengine/blob/fdc001922898df80fd07901f15a07a9e304ed705/examples/detection_fpn.py#L63).


feat: update vue-chartjs version to fix tooltip bug [MA-1926]

Summary
https://konghq.atlassian.net/browse/MA-1926
This PR fixed a chartjs internal tooltip-related bug
https://github.com/apertureless/vue-chartjs/pull/1064

Resources

Creating a package docs

@adamdehaven newest version of chartjs, delivered via vue-chartjs
Original bug fix:
https://github.com/chartjs/Chart.js/pull/11596


Facades in ServiceProviders in online documentation

Laravel Version: any
PHP Version: any
Database Driver & Version: any

Description:
In many places inside the documentation facades are used inside service providers. This is like a recommendation for people to do that.
In my experience this is a very bad habit. While facades are great to read, they should be used only in the very upper levels of an application (like in views or if it makes you happy also in Controllers or Commands).
ServiceProviders are something like the opposite of a high level place inside an application. I tend to use only the resolving and afterResolving hooks inside ServiceProviders if I use stuff bound in other or the same ServiceProviders.
If you resolve bindings in ServiceProviders there are three main problems:

You have to know the extact dependency chain of that binding (is the cache system completely bound while i asking for its bindings)
You are resolving classes you do not need. See for example here , If you dont need oauth your service provider will register it anyway.
You are calling Facade::__callStatic(), Facade::getFacadeRoot(), Facade::resolveFacadeInstance(), Application::offsetGet(), Application::make() while you having a reference to the application in $this->app.

I cant count how often my devs resolving Blade and trapped into the cache errors which are occuring because of a dependency chain they do not completely know.
So perhaps because of readability it is not an option to you to use afterResolving or resolving in docs but $this->app->make() would be a good start.
Steps To Reproduce:
Read the docs ;-)

this doesn't belong here. Create a PR in the laravel/docs repo

Ok thanks


Separate primitive mesh data creation

It is difficult to use generation of planes for anything useful, because I can't get a lava::mesh_data_template<T> object that can then by translated, scaled, and rotated. This pr separates creation of primitive meshes from mesh_data objects. It is backwards compatible, so no changes have been made to the sample projects.

👍 "Divide et impera"


Use preCreateChatMessage instead of createChatMessage hook

Using preCreate~ eliminates the need for double update (create+update, I mean), which in turn makes the message handling snappier if there's lag anywhere along the line. It also removes the need for checking permissions for updating the message since it only cares about the creator (only creator gets the preCreate hook).

If I recall correctly, I had to use the current process because table rolls are async but hooks don't allow for async functions within them. Then, I couldn't figure out how to store the message data from the hook without actually creating the message if that makes sense. I'll check this out though and circle back.

Ah, yes, that is indeed a reason for using post create hook. I don't think that has changed, so I'll close this as such.


Differences between gpt4all and OpenLLM?

I've been exploring libraries that provide wrappers for LLM (large language models) and came across both gpt4all and OpenLLM. On a first glance, both seem to offer wrappers for a wide array of LLMs. I'm hoping to get some clarification on the differences between these two repositories.
Functional Differences: Are there any core functional or technical differences between gpt4all and OpenLLM?
Performance and Efficiency: Are there any performance benchmarks comparing the two, especially in terms of inference time or resource usage?
Primary Use Cases: Is one library geared towards a specific use-case or audience compared to the other?
Thank you in advance for your insights!

I believe that gpt4all is a project released early April from the Nomic team, and it built on top of llama.cpp, or at least they provide binding on top of the work at the ggml team.
OpenLLM fits in where you can deploy different model architecture, as well as running on different backend implementation (pytorch, tensorflow, flax, vllm, gguf(wip))

Functional difference

correct me if I'm wrong, but gpt4all focus on local deployment of models (based on llama.cpp), whereas OpenLLM provides nice UX for you to play with a wide range of LLM, and then can bring such AI application to production (k8s, your own VPC, cloud, etc.)
OpenLLM supports serving fine-tuning, loading adapters dynamically, quantization, integrations with langchain, hf agents, and etc.

Performance

I currently don't have bandwidth to do a thorough benchmarking, because gpt4all and openllm are pretty different project. Where gpt4all focus on local development, OpenLLM can also do local development (given that you have a GPU to run for now as CPU is on our roadmap) and production deployment

In the ballpark, we have some additional optimization baked in, such as PagedAttention, FastTransformer, as well as quantization method such as gptq, bitsandbytes (flash attention is on the roadmap)

Use case

Seems like to me that gpt4all doesn't have a SDK per say (again correct me if I'm wrong), whereas OpenLLM is Python-first, and we have SDK for you to build AI application (integration with BentoML, LangChain, LlamaIndex (wip)). What we have seen so far is people trying to build chatbot, and tbh any type of application that you can think of when working with these models
Note that some of the items that we have on the roadmap

AWQ quantization (see here)
More solid support for GPTQ (wip)
CPU support (GGUF and in extension GGML)

Hope this explains a bit better. Interested to hear more about your usecase.


Setting arbitrary project's output path results with unworking site

Works ok:

Does not work:

Does not work:

Somehow when you set the output path you end up with 2 different paths
<OutputPath>bin/Debug</OutputPath>
... further down the .fproj ...
<OutputPath>..\..\out\Debug\MyApp</OutputPath>

If you delete the first one it works fine.
After you need to specify the bin folder and root directory of your app:
do startWebServer defaultConfig (WebSharperAdapter.ToWebPart(Main, "root_here", "bin_here"))

OK, thanks.
About specifying bin and root folders by hand, should it be done automatically? I think it's very unclear and puzzling that changing project's output path results with non working site.

Ok, I've changed output path and removed duplication from the project file. Now it compiles, but does not work - the static part is loaded and then the browser is waiting for something forever.
What I've noticed, the when the output path was bin/Debug\, a folder named _PublishedWebSite (or the like) appeared there. Now it's gone - it does not exist either in bin\Debug nor in the new output folder (..\..\out\Debug\Monitor.Web).
The errors in the browser is as following:

So, when I removed the trailing slash from outputPath, _PublishedWebsites folder appeared, but it's not helped - the site still does not work.

After changing RootFolder everything starts working.


PR for 762

Allow .timeout to take an error instance in 2.x instead of a string as an argument.

Can you update the API docs also?

Ya sure.

ping @petkaantonov

2.10.2


Update eligibility message to differentiate between state and local

https://github.com/techequitycollaborative/tenantprotections2/pull/32


Print degenerate singular values warning only during derivation

As suggested in #89, this PR will implement the degenerate singular value warning such that it is printed only when CTMRG is being differentiated.
This makes sense since degenerate values shouldn't really be problematic in the forward pass but in the reverse pass degenerate singular values will make the SVD adjoint diverge and thereby render the CTMRG gradient useless.

This can be tested e.g. using a product state PEPS which will lead to a singular value spectrum $S = (1, 0, 0, \dots)$:
The forward pass doesn't print any warnings
julia> psi = product_peps(2, 2; noise_amp=0);
julia> leading_boundary(CTMRGEnv(psi, ComplexSpace(16)), psi, CTMRG());
[ Info: CTMRG init:     obj = +1.235387069986e-01 -7.116847020371e-02im err = 1.0000e+00
[ Info: CTMRG conv 66:  obj = +1.000000000000e+00       err = 3.3306690739e-16  time = 0.71 sec

But the reverse pass will warn and fail
fixedpoint(psi, heisenberg_XYZ(InfiniteSquare()), PEPSOptimize(), CTMRGEnv(psi, ComplexSpace(16)))
[ Info: CTMRG init:     obj = +7.495770233902e-02 -1.901630443277e-01im err = 1.0000e+00
[ Info: CTMRG conv 83:  obj = +1.000000000000e+00       err = 2.2204460493e-16  time = 0.23 sec
┌ Warning: degenerate singular values detected:
│   svals =
│    TensorKit.SortedVectorDict{Trivial, Vector{Float64}} with 1 entry:
│      Trivial() => [1.0, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17,…
└ @ PEPSKit ~/repos/PEPSKit.jl/src/algorithms/ctmrg/ctmrg.jl:280
┌ Warning: degenerate singular values detected:
│   svals =
│    TensorKit.SortedVectorDict{Trivial, Vector{Float64}} with 1 entry:
│      Trivial() => [1.0, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17,…
└ @ PEPSKit ~/repos/PEPSKit.jl/src/algorithms/ctmrg/ctmrg.jl:280
┌ Warning: degenerate singular values detected:
│   svals =
│    TensorKit.SortedVectorDict{Trivial, Vector{Float64}} with 1 entry:
│      Trivial() => [1.0, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17,…
└ @ PEPSKit ~/repos/PEPSKit.jl/src/algorithms/ctmrg/ctmrg.jl:280
┌ Warning: degenerate singular values detected:
│   svals =
│    TensorKit.SortedVectorDict{Trivial, Vector{Float64}} with 1 entry:
│      Trivial() => [1.0, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17, 9.99201e-17,…
└ @ PEPSKit ~/repos/PEPSKit.jl/src/algorithms/ctmrg/ctmrg.jl:280
┌ Warning: Invariant subspace of dimension 2 (up to requested tolerance `tol = 1.0e-8`), which is smaller than the number of requested eigenvalues (i.e. `howmany == 16`); setting `howmany = 2`.
└ @ KrylovKit ~/.julia/packages/KrylovKit/Y0zG7/src/eigsolve/arnoldi.jl:350
┌ Warning: Invariant subspace of dimension 2 (up to requested tolerance `tol = 1.0e-8`), which is smaller than the number of requested eigenvalues (i.e. `howmany == 16`); setting `howmany = 2`.
└ @ KrylovKit ~/.julia/packages/KrylovKit/Y0zG7/src/eigsolve/arnoldi.jl:350
┌ Warning: Invariant subspace of dimension 2 (up to requested tolerance `tol = 1.0e-8`), which is smaller than the number of requested eigenvalues (i.e. `howmany == 16`); setting `howmany = 2`.
└ @ KrylovKit ~/.julia/packages/KrylovKit/Y0zG7/src/eigsolve/arnoldi.jl:350
┌ Warning: Invariant subspace of dimension 2 (up to requested tolerance `tol = 1.0e-8`), which is smaller than the number of requested eigenvalues (i.e. `howmany == 16`); setting `howmany = 2`.
└ @ KrylovKit ~/.julia/packages/KrylovKit/Y0zG7/src/eigsolve/arnoldi.jl:350
...


Have a method to get only changed properties

Q
A

Bug report?
no

Feature request?
yes

BC Break report?
no

RFC?
no

The AuditReader::diff is great (and will be event better once #130 will be resolved), but if we wanted to get only changed property, we have to walk onto the returned array and make a if ('' !== $property['same']) each time.
This can be redundant if used on multiple part.
It would be great to have a method returning only changed properties.
I don't know for the name. Maybe AuditReader::revealDiff? Of maybe simply add a boolean parameter to AuditReader::diff to tells which result we want.

Why don't we change it directly? Why do we need the properties that have not changed? Do we have a usecase?

Yes, by yourself! :trollface:
Ref: https://github.com/simplethings/EntityAudit/blob/master/src/SimpleThings/EntityAudit/Resources/views/Audit/compare.html.twig
Also use by sonata admin for the same kind of thing.
I think both are interesting, depending of the use case.

okay :+1:


[AutoPR cost-management/resource-manager] Reports API moved from stable to preview version with addition of documentation of a couple of new APIs

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3640

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2924


my project can not   create   the share.Framework  normally

Kotlin 1.8
xcode  14.2
Android studio Android Studio Electric Eel | 2022.1.1 Patch 1
Compose 1.3.0
when i  add the  plgin    id("org.jetbrains.compose")  in share moudle 's build.gradle.kts , run  podspec task, can not create the share.Framework , in the  build/cocoapods/framework/shared.framework/Headers  is not share.h  , but  placeholder.h  ,and it is empty 。
i  want get  some help  ,thinks

Please, create an issue on YouTrack: https://youtrack.jetbrains.com/issues/KT
It's not a question about the RSS reader


Wave display issue

Very impressive work.


bug(@vtmn/svelte): components cannot be imported on server side

Hello,
On svelte component (example VtmnButton.svelte), the component import the style on the <script> node.
This type of operation can cause side effects on the generated component on the SSR because is tires to interpret the css as script.
The quick solution is to move the css import into a <style> node.
To go further, during the build, inject the script directly into the component to make it totally autonomous (without node imports)

Hi @Tlahey, thanks a lot for reporting this!
As this package is built by the community, we cannot help you more sorry.
Do you have contact that could help you resolve this issue?
@jleveugle @thollander if you have an idea. I thought we tested for SSR the Svelte components :/
Thanks a lot.
Laurent


Add support for Eth mappings

Summary of changes

Changes introduced in this pull request:

Introduce a new ParityDB namespace for storing Ethereum mappings such as Hash to TipsetKey and Hash to message Cid.
This is required by methods like:

EthGetBlockByHash
EthGetBlockTransactionCountByHash
EthGetMessageCidByTransactionHash

Increment Forest version to 0.19.0

Reference issue to close (if applicable)

Closes
Other information and links

Change checklist

[x] I have performed a self-review of my own code,
[x] I have made corresponding changes to the documentation,
[x] I have added tests that prove my fix is effective or that my feature works (if possible),
[x] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Some things to consider: https://github.com/ChainSafe/forest/blob/6f60fedc52a10d8575a6f06950fb278476dedbb0/src/db/parity_db.rs#L126 needs to account for the new column as well as all the places this method is being called.


Could not find package snap-static-pages in known packages

An error in my haskell-snap-stack app:
build-depends:       base
                     , ghcv-snap
                     , snap
                     , snap-core
                     , snap-server
                     , heist
                     , snap-static-pages >= 1.0 && < 1.1

**Could not find package snap-static-pages in known packages
**
And the same thing for this:
build-depends:       base
                     , ghcv-snap
                     , snap
                     , snap-core
                     , snap-server
                     , heist
                     , snap-static-pages

After cloning I can't install it either. Also, how can I it on hosting even if I install it locally?

Yeah, snap-static-pages was a quick and dirty library we threw together specifically for this website.  We didn't want to release it on hackage because it's not up to our normal quality standards.  To build snap-website you should get clone snap-static-pages from github here https://github.com/snapframework/snap-static-pages and add it as a source to your cabal sandbox.


test: Revert "Try to get lima and colima sane with LIMA_SSH_PORT_FORWARDER (#6697)"

The Issue

#6697

Lima 1.0.0 port forwarder broke tests, it was fixed via #6697, but now in 1.0.1 they reverted that, so we can revert the workaround.

This still seems to hang without the override.


Add RowSelection::intersection

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While @Ted-Jiang  was working on https://github.com/apache/arrow-datafusion/pull/3967 he needed code to intersect two row selections:
// combine two `RowSelection` return the intersection
// For example:
// self:     NNYYYYNNY
// other:    NYNNNNNNY
//
// returned: NNNNNNNNY
// set `need_combine` true will combine result: Select(2) + Select(1) + Skip(2) -> Select(3) + Skip(2)

The code and tests are in DataFusion
Describe the solution you'd like
I would like to code and tests brought to arrow-rs
see pub(crate) fn intersect_row_selection( specifically
Describe alternatives you've considered
Leave in DataFusion
Additional context

+1 for moving this down into arrow-rs


shadcn-ui init --yes is not honored

Currently, the init command ignores the --yes option.
promptForConfig should only invoke the prompts method conditionally.
PR to follow.

If I understand well, the --yes option, is just to skip the last confirmation,
I did a simple test here, and it's apparently working as expected check the screenshot below

If I misunderstand something, let me know @sramam

@brunocroh, Thanks for taking a look. I'm sorry I never got to a PR.
I'd hoped this would behave like "npm init --yes"
Using prompts.override ideally.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


response.read?  Loading Illinois data for psql big dedupe example.

When I run python pgsql_big_dedupe_example_init_db.py, I get the error
Traceback (most recent call last):
  File "pgsql_big_dedupe_example_init_db.py", line 38, in <module>
    localFile.write(u.read())
AttributeError: 'Response' object has no attribute 'read'

I've tried with both python 2 and 3.  Looking at the api for requests, I can't see an attribute called read on the response object either.
http://docs.python-requests.org/en/master/api/

That's not right. I haven' t tried it, but I suspect you're looking for
localFile.write(u.contents)

Its line 38 of dedupe-examples/pgsql_big_dedupe_example/pgsql_big_dedupe_example_init_db.py

Do you want to make a PR?
On Wed, Apr 19, 2017 at 11:58 AM, Ryder<EMAIL_ADDRESS>wrote:

Its line 38 of dedupe-examples/pgsql_big_dedupe_example/pgsql_big_
dedupe_example_init_db.py
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dedupeio/dedupe-examples/issues/50#issuecomment-295344906,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgxbRjGn-pr-IMWrhvgTxO63Yu-DWjHks5rxj1SgaJpZM4NB7KJ
.

--
773.888.2718

@fgregg yep

#51


Fix problem with external monitor hints misalignment when laptop screen is closed.

Different magnification between displays seems to cause problems.


cups/dest.c: Look for default printer on network as well

The current code of cupsGetNamedDest() doesn't look up on network for
a default printer. The result is a temporary queue cannot be set as a
default printer.
Fixes #452

@tillkamppeter I wouldn't want to disturb Mike more than it is necessary (my guess he's on vacation), would you mind reviewing the PR?

For me it fully makes sense so that virtual/temporary queues can be used like physical/permanent ones, especially if the latter will not exist any more in CUPS 3.x. Also for the user experience and transparency it is important to have the same behavior. #452 is really a bug.
Michael is actually on vacation and I have no return date.

@zdonal @tillkamppeter I am not on vacation now, but I will be in PWG meetings the next three days.

@tillkamppeter OT: permanent queues will be in CUPS Sharing server module, thus libcups will have to support them as well AFAIK.
Thank you for the review!
@michaelrsweet I took your new avatar and 'busy' in nickname as you are out of the office and enjoying some free time, so I didn't want to disturb :)


[INFRA-588] Use new distributionManagement.url

@reviewbybees & @batmat

🐝

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

:+1:


Don't initialize a group in setUpAll/tearDownAll.

This initialized setUp/tearDown, which aren't used for
setUpAll/tearDownAll, and broke the common case where
setUpAll/tearDownAll are called before setUp/tearDown.

There isn't a test; I'm trying to avoid spending a bunch of resources on scheduled_test since I'm hoping to scrap it for parts sooner than later.


Some question regarding HystrixCommand.Setter

We have a hystrix command group with name "XYZ" suppose. We are creating Setter as below
return Setter
                .withGroupKey(HystrixCommandGroupKey.Factory.asKey(commandGroupName))
                .andCommandKey(HystrixCommandKey.Factory.asKey(commandName))
                .andCommandPropertiesDefaults(
                        HystrixCommandProperties.Setter().withMetricsRollingStatisticalWindowInMilliseconds(200000)
                                .withExecutionTimeoutInMilliseconds(60000))
                .andThreadPoolPropertiesDefaults(HystrixThreadPoolProperties.Setter().withCoreSize(1));

This piece of code returning Setter is called at 4-5 different places from where API request is made.
Will this maintain a single thread pool based on command group or it will generate multiple thread pools because of different instance of Setter

Under the covers, all long-lived objects are cached per-command key.  So all commands with commandKey="XYZ" would share the same threadpool, properties, metrics, etc.

Just for clarification. All command groups or all commands?
On Thu, Aug 25, 2016 at 9:54 PM, Matt Jacobs<EMAIL_ADDRESS>wrote:

Under the covers, all long-lived objects are cached per-command key. So
all commands with commandKey="XYZ" would share the same threadpool,
properties, metrics, etc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Netflix/Hystrix/issues/1329#issuecomment-242450428,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJaTgfoL3VmcETRG1KNCxtnsRz0Lwqhtks5qjcG5gaJpZM4Jq4AE
.

Yeah, thread pools are not precisely like the others, so that deserves clarification.
Thread pools are static and references by name (ThreadPoolKey).  In your example above, you've left off the ThreadPoolKey, so GroupKey is used instead.
A concrete example:
Setter.withGroupKey("AAA").andCommandKey("ZZZ")
Setter.withGroupKey("AAA").andCommandKey("YYY")
would share a single thread pool named "AAA"


sysdig: 0.35.1 -> 0.35.3

Diff: https://github.com/draios/sysdig/compare/0.35.1...0.35.3
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Maybe it's a good idea to add you as a maintainer already? Feel free to remove me in the process (or not if you prefer not to), as I haven't actually used sysdig in a long time.


Non-core-contracts should be verifiable

Expected Behavior
I should be able to easily tell whether non-Celo Core, but audited contracts like ReleaseGold or MetaTransactionWallet correspond to the audit commit
Current Behavior
I can't tell

This is for the most part already one in Valora, so kicking it out of the milestone https://github.com/celo-org/celo-monorepo/blob/master/packages/komencikit/src/verifyWallet.ts

@nambrot this seems like its still needed for ReleaseGold Contract, correct?

@nambrot this seems like its still needed for ReleaseGold Contract, correct?

I'd say this is still valuable, but afaik no one specifically is asking for it. IMO, this would be more valuable for all the MultiSigs that have been recently deployed. Though maybe more primarily to do in Blockscout than via code tooling

I'd say this is still valuable, but afaik no one specifically is asking for it. IMO, this would be more valuable for all the MultiSigs that have been recently deployed. Though maybe more primarily to do in Blockscout than via code tooling

might get this for free with https://github.com/celo-org/celo-monorepo/issues/7439
I read sourcify identifies duplicate bytecode for "factory" contracts


Doom fails to build

After this commit, dooms fails to build with the following error:
Single file ghub-pkg.el does not match package name ghub
builder for '/nix/store/p1g1i94fwf88998sqvvf56g14rpghf5j-emacs-ghub-20220429.1708.drv' failed with exit code 255

The test configuration in this repo builds fine, excluding an issue with codeberg being down recently, but you aren't hitting that. Good luck.


Add EODAG

Insert URLs to the project here:
https://github.com/CS-SI/eodag
In one sentence, explain what the project is about:
Gives you an easy way to access products from more than 10 earth observation providers, with more than 50 different product types (Sentinel 1, Sentinel 2, Sentinel 3, Landsat, etc.) that can be searched and downloaded.

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues marked as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

@sbrunato @adrcad @maximlt


Add location.str to Hexen Deathkings vanilla-engine folder

The presence of /location.str file pointing to the hexen.exe path is a requisite to launch hexendk.exe

Sorry it took a while, but thanks :)


It's very easy for the local.py.example to get outdated

Instead of local.py.example lets have a local.py and a base_local.py. base_local.py has everything we currently put on local.py.example while local.py will import everything from base_local.
README should have instructions for the developer to create local.py and import from base_local.py

@aericson @fjsj any opinion?

looks best IMO


Unhandled Exception: 'int' object has no attribute 'iter_content'

It happen when i download<PHONE_NUMBER>260

This also happened to me for book<PHONE_NUMBER>028
It didn't happen before I applied the change from:
https://github.com/lorenzodifuccia/safaribooks/pull/188
However, without changing that snippet it had the behaviour described in issue https://github.com/lorenzodifuccia/safaribooks/issues/184
Nevertheless after changing this snippet, it does not correctly download the cover page and images (but it does download the actual book content correctly, which is much preferrable to the cover page being pretty).

I have the same problem after applying #188. Does anybody know which changes need to be made so that it also downloads the missing images?

Both #188 and #184 had been merged into master. Can you get the latest version and try again?


Add 'allCookies' method

This method returns all the cookies for the current page, even third party ones.
Based on the Chromium command Network.getAllCookies(): https://bugs.chromium.org/p/chromium/issues/detail?id=668932#c15

Browser is a better place for this API: this should be browser.cookies()

Good point. I'll move and rename it cause it returns all the cookies in the jar. Thanks for the feedback.

@MicheleLevorato unfortunately, the Network domain is not available on the browser target today, so this would require some chromium work. @JoelEinbinder was planning to do something about it.

Ouch. Ok, i'll stick locally with my custom code to retrieve third party cookies, looking forward for a stable solution. Thank you, @aslushnikov!

@MicheleLevorato thanks for the PR!

@MicheleLevorato FYI: filed #645 to track progress on this.

Will this come with a compatible setCookies?

@dht there's a browser.addCookie in the suggestion.


migrate to JUnit5

Not sure if you want to, but why not migrate to JUnit 5 I thought :)

Ideally I would like to do this when we switch over the main Mockito test suite (https://github.com/mockito/mockito/issues/1246) and I don't think we want to move to JUnit 5 just yet. Let's discuss further in the Mockito issue before we merge any PRs related to JUnit 5.

All right, no worries. If you need a helping hand on the other test code, let me know.

@hennr Just curious. What are the advanteges of using junit5 over junit4 (with respect to an updated junit dependency like junit 4.13.2)?

@bohsen That would be the same question for me, but let's move that discussion to the Mockito issue rather than this PR, so that we can keep this PR focused on the implementation. Thanks!

You folks might consider closing this one - it's been a while


Add support of zoom/pan force directed graphs

PTAL: @sohanmurthy
On my machine, zoom perf is pretty good on large and smaller orgs. Pan perf isn't super hot on larger orgs. I can't think of too much we can do there except maybe try out this library for the charge https://github.com/twosixlabs/d3-force-reuse or maybe move to a canvas rendering instead of svg.
Smaller org: https://p-zmf7dq.b0.n0.cdn.getcloudapp.com/items/9ZuXyEew/Screen Recording 2020-09-01 at 05.27.45 PM.mp4
Larger org: https://p-zmf7dq.b0.n0.cdn.getcloudapp.com/items/6queJ5R1/Screen Recording 2020-09-01 at 05.29.39 PM.mp4

This looks so great! Up to you on how deep you want to go with perf improvements. Currently there are only a handful of accounts where rendering the galaxy chart crushes your browser (Uber, Square, Intercom, etc.) but I know folks like Joe and Lauren check them often.

Gotcha. My hardware is pretty new so Square and Uber, while not snappy, seem fine before and after this change. Are you seeing crashes when loading those galaxy charts?

Having just played with it, it works fine on my computer which is a couple years old. All good!


any api like this page？

hello, had any api like this page ：https://github.com/dashboard/discover ？i can't find in developer document

Not sure what you're asking. The app doesn't have a discover feature but GitHub does.

sorry,   I just want to know if  GitHub already open such api or not.
The app doesn't have a discover feature because no such api？
thanks for you reply！

Not sure, it's not what this repository is about. If you want to look further there is documentation about GitHub's api here: https://developer.github.com/


This version of CLI is only compatible with Angular versions ^13.0.0 || ^13.3.0-rc.0, but Angular version 14.0.0 was found instead

🐞 Bug report
Command (mark with an x)

[x] build

Description
After migrating from Angular 13 to Angular 14, it is not possible to build using<EMAIL_ADDRESS>This message is displayed when running ng build:
This version of CLI is only compatible with Angular versions ^13.0.0 || ^13.3.0-rc.0,
but Angular version 14.0.0 was found instead.
🔬 Minimal Reproduction
In an Angular 13 project perform migration to Angular 14.
ng update @angular/core@14 @angular/cli@14
After migration performed and ng s working, run the build:
ng build
Message is displayed:
This version of CLI is only compatible with Angular versions ^13.0.0 || ^13.3.0-rc.0,
but Angular version 14.0.0 was found instead.
Please visit the link below to find instructions on how to update Angular.
https://update.angular.io/
🌍 Your Environment

Angular CLI: 14.0.0
Node: 16.10.0
Package Manager: yarn 1.19.0
OS: linux x64

Angular: 14.0.0
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, material, material-moment-adapter, platform-browser
... platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1303.7
@angular-devkit/build-angular   14.0.0
@angular-devkit/core            14.0.0
@angular-devkit/schematics      14.0.0
@angular/flex-layout            13.0.0-beta.38
@angular/http                   7.2.16
@schematics/angular             14.0.0
rxjs                            7.5.5
typescript                      4.7.2

Can you please provide the output of npm ls @angular-devkit/build-angular?

I ran into the exact same issue. In my case, the issue is a result of using @angular-builders/custom-webpack (v 13.1.0). I got past this error by replacing the relevant "builder" settings in angular.json

I need to use @angular-builders/custom-webpack (now it is v13.1.0). @damilgra How did you solve this incompatible?

Can you please provide the output of yarn ls @angular-devkit/build-angular?

├─┬<EMAIL_ADDRESS> │  └──<EMAIL_ADDRESS> └──<EMAIL_ADDRESS>

Everything indicates update a future module update in progress @angular-builders/custom-webpack -> https://github.com/just-jeb/angular-builders/issues/1219

Closing since this is caused by a 3rd party builder which is not supported by the Angular team.

For those who have module dependency and cannot remove, this solution works -> @abadakhshan https://github.com/just-jeb/angular-builders/issues/1219#issuecomment-1146736928


prompb: Add histograms to remote write/read protobufs

This is only for the sparsehistogram branch!
A first attempt to add histograms to remote write/read protobufs.
@bboreham again cc'ing you to double-check if this makes sense.

Thanks, @codesome . Still waiting for remote write folks to comment.
Note that https://github.com/prometheus/client_model/pull/58 has changes to the exposition format protobuf, adding support for float and gauge histograms, with some possibly enlightening comments.

Also mention / link to the explanation for "float histograms".

I think the already mentioned https://github.com/prometheus/client_model/pull/58 is a good starting point to understand the need for "float histograms". I expect the most common use case for remote write will be to send the result of recording rules to the remote storage. (If you rate a histogram, you inevitably get a float histogram as a result.)

I added two more things:

Utilized oneof (a proto3 feature) to mark the various one-of fields more explicitly. (Note that oneof isn't compatible with repeated, so for count vs delta in Buckets, we still depend on the user to only use one or the other.)
Added the HISTOGRAM chunk encoding, so that this should in principle be ready for chunked remote-read. (Note that STREAMED_XOR_CHUNKS turns out to be a bad name. Given that the Chunk message has an Encoding type field, it was always ready to contain non-XOR chunks, so the name in the enum should be simply STREAMED_CHUNKS. Not sure about the implication of changing it now, so I left the old name in place.)

I have now also added a link to the design doc in the doc comments, including a pointer to the concept of float histograms.
This is just for the sparsehistogram branch, and it got a few looks already. I'm merging it now, but we can still iterate on it. Whoever works on implementing the actual use of these proto messages (myself, @cstyan, @csmarchbanks, maybe even @LeviHarrison), feel free to come back to the proto spec and adjust it to needs and insights found later.


[locale] ru: Capitalize month names

Can you please modify src/locale/ru.js and src/test/locale/ru.js.
@Viktorminator what do you think.

Hi @armenavetisyan  - thanks for your contribution. You need to sign the CLA in order for this to go through.
@Viktorminator - could you confirm this change?


Hosting the Dolly dataset on the Hugging Face Hub

Hi Databricks team, this is a really cool project and great job creating a high quality instruction dataset with a permissive license!
Would you be interested in hosting the Dolly dataset on the Hugging Face Hub (https://huggingface.co/datasets)? The Alpaca dataset is also hosted there (link) and your version would be of wide interest to the community, especially since many people have noted the ToS issues with training on OpenAI model outputs
If that sounds interesting, you just need to create a new dataset repo under the DataBricks org and upload your databricks-dolly-15k.jsonl file through the UI. For more details, you can check out our docs here

I asked, and looks like the plan is to just host it in Github here, for now.


Pipeline active after stopped

tested on
systemVersion: "v2.1.88",
fullSystemVersion: "v2.1.88-1631616917808",


validate the type of children

scenery.Text and scenery.Image have name collisions with window.Text and window.Image respectively.  If you forget to require one of those scenery types, you may see a runtime error like (for example):
Uncaught Error: assertion failed: subtree pickable count should be guaranteed to be >= 0
window.assertions.assertFunction	@	assert.js:22
extend.changePickableCount	@	Node.js?bust=1447883882614:624
extend.insertChild	@	Node.js?bust=1447883882614:330
extend.addChild	@	Node.js?bust=1447883882614:359
RectangularButtonView	@	RectangularButtonView.js?bust=1447883882614:112
RectangularPushButton	@	RectangularPushButton.js?bust=1447883882614:67
SpectrometerNode	@	SpectrometerNode.js?bust=1447883882614:83
ModelsOfTheHydrogenAtomScreenView

It would be nice if this could be detected at an earlier point, and a better error message provided, instead of just failing at some later point that's difficult to grok.
Yes, you the client can trigger an earlier failure by doing things like new Image( ... ).someSceneryFunction(), but that fails immediately with a stack trace that indicates the source line that's in error.  I encounter this with addChild and { children: [ ... ] } methods of building the scene graph, where the problem manifests as a failure deeper in the call stack.
I don't know where the "best" place to check this is in scenery.  But it seems like a well-placed assertion like this would be useful:
assert && assert( child instanceof Node, 'child is not a Node, constructor = ' +
  ( child.constructor ? child.constructor.name : 'no constructor'  )  );

Btw... Since I make this mistake quite frequently, I have learned to recognize the symptoms of this. But since scenery is useful outside of PhET, I'm also thinking of the general scenery user.

Applied the assertion. Testing in the playground:
scene.addChild( new Image() )

assert.js:22 Uncaught Error: Assertion failed: addChild/insertChild requires the child to be a Node. Constructor: HTMLImageElement(…)

Assigning for review, or suggestions of anywhere else this should be added (adding it as a child seems like the best place to me).

I tested this with addChild and options.children, and it provides a useful error message in both cases. Those are the most common methods that I use of setting children, so I'm happy.   And looks like it should also work for other functions that add a child (insertChild, setChildren).   So thanks, :+1: and closing.


How to change the background color?

I would like to know how to change the background color of MessageBox.Avalonia?

Owerride style for control

Owerride 风格的控制

[Binding]An error occurred binding 'OverlayBackground' to '': 'Could not convert 'MainWindowViewModel' (MainWindowViewModel) to 'Avalonia.Media.IBrush'.' (DialogHost #59893061)
[Binding]An error occurred binding 'Background' to '': 'Could not convert 'MainWindowViewModel' (MainWindowViewModel) to 'Avalonia.Media.IBrush'.' (DialogHost #59893061)
In the latest version of Avalonia, the background color of the dialog box has become transparent and can be used normally with error messages.


Implement a user-agent for twilio API

https://www.twilio.com/messaging-api

This issue is for the building block to connect to twitter APIs.  Once you are done with it. You can implement a demo app to use this building block.   The demo app  will be like test-git2.cpp, but it will take a message from the command line and send it to a phone # thru twilio.


Add notebook extension support for .NET Interactive.

List of changes:

Use an entire contributed NotebookController as a notebook kernel, rather than mistakenly using the controller's supportedLanguages as the kernels. Supported languages are instead supposed to be cell-level info (i.e. for the cell language dropdown).

Modified cells to display the current cell language in the lower right of the cell.

Added support for basic VS Code notebook output mime types.

Added a supported languages field to the notebook provider contribution point, since notebook kernels can now have multiple languages in use at one time.

Fixed SQL kernel aliases incorrectly showing up for non-SQL notebooks.

Fixed issues with creating new notebooks through the VS Code extension API. The openNotebookDocument function needed to create a NotebookDocument without opening it in the editor. Interactive then modifies the returned document object to update things like cell languages before opening the document in an editor with the showNotebookDocument function.

Fixed an issue with execute managers so that .NET Interactive is usable in ipynb notebooks. We were retrieving execute managers solely based on uri, rather than uri and provider id, so every extension-contributed notebook provider was using the same execute manager (i.e. jupyter).

Fixed bugs with VS Code notebook conversion around metadata and default fields.

Fixed issues with kernelSpec inconsistencies when using .NET Interactive through Jupyter. The languages and kernel names in the ipynb kernelSpec are different from the actual contributed controller info, so some kernelSpec values have to be temporarily replaced in the kernelSpec to get things to work. The kernelSpec info is then restored when serializing the notebook back to JSON.

Remaining work:

Enable cell edits via the BulkEditService so that .NET Interactive can update cell languages properly.

Allow cell languages to be selectable via a dropdown in the lower right of the cell, rather than just having a language display.

Cell language doesn't update unless cell changes edit state.

When creating a new cell with the .NET Interactive kernel after changing to a different kernel and back, the cell doesn't inherit the correct default language (C#). It instead shows the default Python.

Demo GIF:

Added a demo gif to the description.

@Charles-Gagnon Looks like there's 46 references to the azdata nb namespace in cell.ts alone. Do you mind if I fix all those azdata imports in a separate PR?


Add Include filter for mgmt pipeline

Add management pipeline filter to include instead of exclude for data plane packages.

/azp run js - mgmt - pr

/azp run js - mgmt - pr

/azp run js - mgmt - pr

@praveenkuttappan Can you please update the PR description with a link to the issue being fixed so that reviewers know the reason for this change?
@qiaozha Please review. This affects the mgmt plane pipeline

@qiaozha Please see https://github.com/Azure/azure-sdk-for-js/issues/9814#issuecomment-807219433 for context

@ramya-rao-a Does this mean, we should update the CI/PR mgmt. plane configuration each time we have a new mgmt plane package?

Also. I wonder if it's possible to use regex to express the management package in azure devops ? something like sdk//arm-

Does this mean, we should update the CI/PR mgmt. plane configuration each time we have a new mgmt plane package?

@qiaozha Yes.
For data plane packages, we need to add an entry to rush.json file for every new data plane package.
Until now, we also had to add an entry to the exclude list in this file that is for mgmt plane pipeline.
So, this is an effort to even the playing field so that

every new data plane Track 2 pkgs (and in the future Track 2 mgmt plane packages) will add an entry to rush.json
every new mgmt plane Track 1 pkg will add an entry to the file in this PR

Also. I wonder if it's possible to use regex to express the management package in azure devops ? something like sdk//arm-

Unfortunately the Devops path triggers don't support regex so things have to be listed explicitly which is why I suggested we follow the same pattern that .NET uses and if a new mgmt library is added all they need to do is run a script to get the yml file updated with the new path.

@ramya-rao-a @weshaggard Thanks for the clarification.   Could we update these folder before we merge it ? I found https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/changeanalysis/arm-changeanalysis this one has been newly added, but not in this PR yet.

@qiaozha Added entry for change analysis in https://github.com/Azure/azure-sdk-for-js/pull/14599/commits/1babee1ff1ee69d72dec947b5f0c6c6bd6945e87. Can you review this PR again?

@mikeharder The build on Windows keeps getting cancelled, can you take a look here and have this PR merged as Praveen is out?

There were some agent issues earlier I retriggered the failed leg and it is running now.


Panel: order of panes does not put terminal last

When you run with fresh UI state, the terminal appears before debug console for some reason, I think it is always last normally:

I am still seeing this when I run yarn web.

Fresh user directory?

@sbatten yes. if you just use yarn web you will always get a fresh user data dir because no data is persisted.


Using extras like borderRadius with components

Hi there
Really enjoying using the style functions to unify the look of my site with these! But I'm having a bit of difficulty understanding how the extra props are used with existing components. In addition to importing any of them (i.e. borderRadius) I have included it as a prop in my component (a styled Box using styled-components. But there is no border-radius on my output.
import styled from "styled-components";
import { Box } from "grid-styled";
import { space, borderRadius } from "styled-system";

const PerformanceTime = styled(Box)`
  display: inline-block;
  color: #fff;
  font-size: 12px;
  font-weight: 600;
  ${space};
`;

<PerformanceTime p={1} mb={1} borderRadius={1}>10:00pm</PerformanceTime>

I'm obviously missing something simple!
Thanks

You need to include ${borderRadius} within your PerformanceTime style, like you did with ${space}.

Thanks @gragland
I originally had that as well.
Should it just work or is there anything else I might need to do? All the base ones work great as is!

@babycourageous Ah okay, not sure then as I haven't used borderRadius yet.

@gragland thanks for your eye. Strangely I can't get any of those "extra" configurations to work on my components. Do I need a styled-components Provider with a theme prop by any chance wrapping up the App ?

@babycourageous Just tested borderRadius and it looks like it expects a px/em/rem value. It doesn't read from theme spacing.

Oh thanks for checking into that @gragland. That's too bad - was hoping to take advantage of some consistency there (like with width margin and padding props).
Are you also a maintainer - both the styled-system and rebass docs and examples seem to have references to a radius or radii property in the documentation for theme and "extra props"

Ah yes it looks like it gets values from theme.radii instead of theme.spacing. Maybe creating your own borderRadius style prop would do the trick (haven't tested this):
import { responsiveStyle } from 'styled-system'
const borderRadius = responsiveStyle({
  prop: 'borderRadius',
  cssProperty: 'borderRadius',
  key: 'spacing'
})

Thanks for all your help @gragland --
I do see it working as a pixel value (rather than pulling from the theme object as docs suggest). I'll keep this open for now since rewriting the borderRadius property that already comes with styled-system and rebass is probably not the intention of the library.

I think the solution would be to override the theme using themeProvider and add an array of px values to theme.radii. I think it's unlikely that borderRadius will be changed to read from theme.spacing since that would be a breaking change for existing users.

It sounds like most of the questions have been addressed here, but just adding a note that if you'd like to use something like theme.space for the borderRadius utility, you can always handle that in your own theme definition, e.g.:
const space = [ 0, 4, 8, 16, 32, 64, 256 ]
const radii = space
const theme = {
  space,
  radii
}
export default theme

Feel free to reopen this issue if you have any other questions.

Thanks @jxnblk - That note is very helpful!


Translation langs dropdown does not dismiss when clicking elsewhere

In re-deploying Qiskit Optimzation docs I was checking the language dropdown and went to dismiss it by clicking elsewhere on the page. I expected this to dismiss the dropdown, as this is what normally happens in that type of UI. But it did not and at first I thought my browser (or system) had frozen since nothing seemed to be happening. Turns out I either needed to explicitly click on the dropdown down-arrow toggle or a language to dismiss it. I think the dropdown should be dismissed when I click outside of it as would be expected of such a UI element,

The design in the new Furo-based theme (https://github.com/Qiskit/qiskit_sphinx_theme/pull/425) no longer blocks any content:

With the new design, clicking outside the dropdown will not change the toggle state of the dropdown. But because the dropdown no longer covers content, that is how UI typically would work. The designers liked this setup.
So, closed by switching to the new theme 🚀


Test cases do not work consistenly

Running the test cases does not work consistently for me.
Sometimes some of the subtests work, sometimes they don't.
It seems to be a problem because of the following problem: https://answers.ros.org/question/251194/rospy-subscriber-needs-sleep-some-time-until-the-first-message-is-received/
TLDR:
Sending or receiving messages directly after creating the publisher / subscriber might not work as it is not correctly set up yet.
A rospy.sleep(1) behind creating them is needed for some systems to have it working correctly.

The tests are more stable with the new release 0.1.3.
I still experience sometimes (~1 out 5 times) a problem.
ros@fdfbfc4f61b2:/scratchlocal/creinke/data/study/perception/code/spring/robot_ws/src/pyhri/test$ python test_pyhri.py
[ROSUNIT] Outputting test results to /home/ros/.ros/test_results/pyhri/rosunit-test_pyhri.xml
[Testcase: test_get_bodies] ... ok
[Testcase: test_get_faces] ... ok
[Testcase: test_get_faces_roi] ... ok
[Testcase: test_get_known_persons] ... ok
[Testcase: test_get_tracked_persons] ... ok
[Testcase: test_get_voices] ... ok
[Testcase: test_person_attributes] ... FAILURE!
FAILURE: unexpectedly None
File "/usr/lib/python3.8/unittest/case.py", line 60, in testPartExecutor
yield
File "/usr/lib/python3.8/unittest/case.py", line 676, in run
self._callTestMethod(testMethod)
File "/usr/lib/python3.8/unittest/case.py", line 633, in _callTestMethod
method()
File "test_pyhri.py", line 317, in test_person_attributes
self.assertIsNotNone(p1.face)
File "/usr/lib/python3.8/unittest/case.py", line 1328, in assertIsNotNone
self.fail(self._formatMessage(msg, standardMsg))
File "/usr/lib/python3.8/unittest/case.py", line 753, in fail
raise self.failureException(msg)

SUMMARY:

RESULT: FAIL
TESTS: 7
ERRORS: 0 []
FAILURES: 1 [test_person_attributes]

thanks for the feedback. These timing issues are super fiddly, and depends a lot on each ROS machine. I've pushed a new version with a slightly longer delay. Let me know if it fixes the issue for you. Otherwise, please increase the delays in the test file as required on your machine, and create a PR (so that we find a compromise that work on as many ROS configurations as possible).

Works now fine for.


Overflow on polygon with 256 sides

Setting the sides to 256 in the shape tool and then drawing causes an overflow panic (assuming you are running with debug asserts). This can be fixed by either casing to a larger integer (i.e. u16) or by subtracting 1 rather than adding one (giving the same result)

We're using a u8? Why not a u32 or u64 like most normal numbers?

I can't reproduce this crash.

@Keavon It needs to be compiled with debug assertions (otherwise int overflow is ignored)

@Androxium and I are interested in working on this!

Interested in working on this with @linda-zheng !


Dashboard view alignment issue:   Tables don't align vertically on Go (Both Chrome and IE11)

Dashboard view - Go (Both Chrome and IE11)
Tables don't align vertically when window is resized.

possibly related to issue #475


[Android] ScrollView momentum not stopping when calling scrollTo programmatically

Description
When scrolling down a long ScrollView or FlatList on Android and calling scrollTo programmatically while the ScrollView's momentum is still going, the momentum does not stop and the resulting scrolling is stuttering and simply wrong.
React Native version / Platform
0.65.1 / Android
With 0.63.4, this bug was not yet present.
Steps To Reproduce

Render a ScrollView (or FlatList) with long content and a button with onPress={() => scrollViewRef.scrollTo({ y: 0, animated: true })}.
Scroll quickly down with your finger so that a momentum occurs.
Press the button while the momentum is still going.

Expected Results
The momentum should stop and the ScrollView should scroll to its beginning.
Possible reason
It seems that the reason for this bug lies in the following commit: https://github.com/facebook/react-native/commit/10314fe621e1649654e83df197adf657e0ca8363

Running into the same issue.

React Native 0.70.6. The same issue..

Any update? fk rn

scrollViewRef.scrollTo({ y: xxx, animated: false})
Nor did it move to the specified location, the new target location did not overwrite the old target location

This is also something we are facing, and for a little bit more context:

Scroll down fast
onScroll reports normal scrolling values (ex: 1200, 1220, 1240, etc.)
Trigger a scrollTo to y: 0 with an outside action
onScroll correctly reports a scroll of y: 0 once, but then continues with its momentum (ex: 1240, 0, 1260, 1280, etc.)

This is also something we are facing, and for a little bit more context:

Scroll down fast

onScroll reports normal scrolling values (ex: 1200, 1220, 1240, etc.)

Trigger a scrollTo to y: 0 with an outside action

onScroll correctly reports a scroll of y: 0 once, but then continues with its momentum (ex: 1240, 0, 1260, 1280, etc.)

same issue

The issue still persists on 0.71.2.
I've made a repro: https://github.com/tomekzaw/Issue32235
I will investigate this issue further.


Help

How to make the appearance of the menu in the slideout, as here - https://itchief.ru/demo/demo-navmenu-with-push-effect

Hi @wekrait,
Here is a demo: https://codepen.io/pazguille/pen/mmgMEq


Backport update to email-validator>=2.0.0.post2

Fixes #3905

@tiangolo looks like we're blocked by this: https://github.com/tiangolo/fastapi/blob/8ac8d70d52bb0dd9eb55ba4e22d3e383943da05c/pyproject.toml#L62
Any chance you could cut a FastAPI patch release that removes the upper version pin?

please review


Remove redundant election-related code

WARNING: Do not deploy before the the publication history backfill has been performed.
This code was used in the run up and during the election and is now redundant. Dead code is bad code so let's clean the decks!

:+1:

That script has now been run so this is now good to merge.


Crash when attempting to open 132x50 file

Program crash when attempting to open attached 132x50 file.
board-132x50-level-sectors.zip

Should be fixed.


Case Insensitive Tab Completion, Regex Highlighting

Implements case insensitive tab completion (differs from former PR in that it will preserve case of the other guy's nick upon tab completion), as well as regex based highlighting in addition to your own nick (which is enabled by default even with an empty regex string). Empty regex strings will default to your active nick.
In addition, implemented nick exclusions (which also supports wildcards via *): nicks that match will not highlight even if their message matches highlight's regex.

@andrew2060 thanks!


Integrate kolivas more extensive parsing

So that everyone's parsing needs should be met.
Also:

Normalize line endings and indents
use .editorconfig to codify this
heed edit preferences warnings
unify project and solution files
consolidate parse options

Review @poma and @Zemill

I like that you moved all of the parsing options to a separate class file with defined parsing method options.
What I might do is add another method in there called MediumParsing with the same values as Minimal except ShouldParseDetaileBattleLobby = true

I am having a hard time seeing how to add the updates directly.  So I will just leave it to you.
On the Player.cs file add
On the Statistics.cs file add
public int RegenGlobes { get; set; } = 0;
case "RegenGlobePickedUp":     replay.Players[(int)trackerEvent.Data.dictionary[2].optionalData.array[0].dictionary[1].vInt.Value - 1].ScoreResult.RegenGlobes++;
break;       // {StatGameEvent: {"RegenGlobePickedUp", , [{{"PlayerID"}, 1}], }}
So that would re-base it to Koliva's updates, HotsApi updates, and Heroes Profile updates.
From there we may want to see about pushing that into Barrets main parser

Once you make those changes I will clone this repo into my main parser and try it out

I tested this locally without issue. Having issues with something with my lambda parser with it, so still looking into it.  @poma I will let you make the final decision

Do let us know how things go with the lamba. We should investigage if not 100%

Alright, like I thought it wasn't an issue with the parser.  Between my last update and now AWS deprecated netcode 1.0, so I had to update to 2.1 and took some time to get the configuration right.  Anyway, looks good to me

There is actually an issue with the code.  When running MediumParse, talents are not picked up.  @poma I am assuming it has to do with how the skipping of parsing is working, so I assume you are getting talents through your parses right?


Support wheres

Open for discussion:

[ ] Is this implemented correctly, i.e. select is considered the root/control node?
[ ] Should or/and accept any number of children or stick to 2 (head/tail)?
[ ] Is the main processor aimed at any flavour of SQL (i.e. Postgres)? Need this to implement LIMIT / OFFSET etc.

I've resurrected this project and moved it over here and now this PR is obsolete so I've closed it.


dialog forces mulptiple chunks in webpack prod build

I'm submitting a bug report

Library Version:
2.0.0

Webpack Version:
4.27.0

Language:
TypeScript 3.1.2

I need my project to be bundled into a single .js file. This works until using aurelia-dialog which results in a whole bunch of extra chunks output by webpack.
Steps to reproduce

au new - Create a default TS project using webpack
Adjust the webpack config to not create chunks:

    optimization: {
      runtimeChunk: false,
      splitChunks: {
        cacheGroups: {
          default: false
        }
      }
    }

au build --env prod - Make a prod build and verify there is only one .js file.
Add aurelia dialog to the project

npm install --save aurelia-dialog
Actually use it, e.g. in app.ts:

import { DialogController } from 'aurelia-dialog';
import { autoinject } from 'aurelia-framework';

export class App {
  constructor(readonly controller: DialogController) {  }
}

au build --env prod - Make a prod build and see there's a plethora of .js files

Well, maybe "submitting a bug report" is a bit harsh. Could be this turns out to be a support request. Anyways, would be greateful if this could be resolved.


MaxAuthTries - Citation(s) for baseline choice.

I'm new to Ansible and have got a lot of value from your ansible-ssh-hardening project, thanks!
I did hit one snag with the MaxAuthTries setting of 2, compared to the default of 6. I actually managed to lock myself out of a host due to my ssh agent offering different keys before the correct one, causing a "Too many authentication failures for XXX" disconnection. After discovering the issue I wanted to understand from the baseline why this setting is chosen.
I read the description for this control baseline and am struggling to see if changing this setting offers any tangible benefits for the increased risk of inconvenience (based on the fact the baseline already requires password login disabled).
I wanted to offer a general observation here. The internet is full of varying quality guides for hardening SSH with very little reference to reputable STIG or other similar frameworks for secure configuration. I think the vision for this project is fantastic, it should streamline things for many people but I think it's important to track why baseline settings are chosen, citations for any particular attack vectors and noting some of the tradeoffs for the decision. Otherwise it feels like just an extension of "cargo cult" style blog posts where everyone is offering their chosen secure settings with little critical evaluation on why this setting was chosen.
Are you able to provide some background on this setting?
I'm happy to open a PR for the ansible-ssh-hardening to update the documentation/faq to flag this, unfortunately as Ansible uses SSH as the control channel it's particularly sensitive to these types of issues!

Thank you @monobaila for reaching out and being so honest. We are very happy that you find our resources useful.
What are the benchmarks your rely on?
In general, we only accept controls that are based on trusted sources. In most cases those are guided by STIG or CIS. Historically, we have some rules that originated from NSA guidelines and Deutsche Telekom.
I cannot see where the control originated from?
This is the case for the content that is part of the project since the early days. We started with ServerSpec and it had no capacity to annotate checks when we migrated to InSpec. With newer contributions, we required references to trusted sources. A good example is https://github.com/dev-sec/windows-baseline/blob/master/controls/account_policies.rb#L3-L21
We strive to have those annotations for all controls to provide high-confidence in our profiles and automation. Any PR to add this metadata is very welcome!
Why is MaxAuthTries set to 2
In your case, the CIS guideline says that Ensure SSH MaxAuthTries is set to 4 or less.

One of the key benefits of the DevSec project is its ability for customization. For most values, we set defaults that can be overridden. This is not the case for MaxAuthTries yet, so feel free to open a PR to submit this improvement.

Thanks for the detailed response @chris-rock, I've opened a tentative PR (https://github.com/dev-sec/ssh-baseline/pull/138)  with some improvements - thanks for pointing me to a good reference example.
I've not had much time to put this together so hopefully there are no typos, I'm hoping the automated tests for the PR will blow up if I've done anything stupid!

Spoke too soon, it's already blown up :)
I'll take a look when I have some time.

I've fixed up my PR and the build is now clean.


Kubernetes: Enable ILB probe

What this PR does / why we need it: Enable probe for Kubernetes ILB ARM resource
@juan-lee @khenidak @slack FYI
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Kubernetes: Enable ILB probe

Thanks!

/lgtm
@jackfrancis can we add a test for this somewhere? I have a similar question for #3617. I'd be glad to do that but I could some pointers.

@mboersma We have a basic ILB test in place, what we don't have is a test that "shuts down one or more masters in a multi-master cluster and confirm that ILB continues to balance load appropriately".


My bug report

Steps to reproduce

Expected behavior

Actual behavior

Environment data

<EMAIL_ADDRESS>

kxkgxkyxxgkxy

jfktdktzb8472cifjd

@powercode @theJasonHelmick

Closing due to lack of information. If you have an issue you need to report, please add the required information and then request the issue be reopened. Thanks!


The clinvar submitter removal may not work as intended?

Or Maybe it does... Prove it
https://centrepopgen.slack.com/archives/C035QN8C0DN/p1676524915805259?thread_ts=1676512346.242019&cid=C035QN8C0DN
was this a config problem?

Closing this for now, I think this was an issue (bug?) in how the config file was generated, so the cohort-specific submitters to ignore were left out.
Unsure if this was a config generation issue, or a me not providing the correct argument issue.
Either way, AIP performed correctly for the provided config file, and I've not been able to reproduce


Question: Best practices for passing along cancelled/not cancelled?

From reading the docs, I'm aware that when a job is cancelled, isCancel() will return true. In my use case, the job is responsible for syncing a few different repositories, so I'm not able to check isCancel from within the job frequently enough.
Do you have any recommendations for passing along that state to the repositories? My first thought was to override the "cancel" function in the job, and maintain an AtomicBoolean in the job which is passed into the repos. But, the cancel function is final so I can't override it.
Second thought was forking the repo and removing that pesky final label, but I figure this might be a solved problem.
Thanks!

You can override onCancel(), see https://github.com/evernote/android-job/blob/master/library/src/main/java/com/evernote/android/job/Job.java#L124

Sorry for not double checking the code before posting! Thanks for the help.


Sample usage don't work on fersh python=3.9 env

During call  API = CVAT(username="username", password="password", url="http://localhost:8080")
with proper credetentials
---------------------------------------------------------------------------
JSONDecodeError                           Traceback (most recent call last)
File ~/anaconda3/envs/py39/lib/python3.9/site-packages/requests/models.py:972, in Response.json(self, **kwargs)
    971 try:
--> 972     return complexjson.loads(self.text, **kwargs)
    973 except JSONDecodeError as e:
    974     # Catch JSON-related errors and raise as requests.JSONDecodeError
    975     # This aliases json.JSONDecodeError and simplejson.JSONDecodeError

File ~/anaconda3/envs/py39/lib/python3.9/json/__init__.py:346, in loads(s, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
    343 if (cls is None and object_hook is None and
    344         parse_int is None and parse_float is None and
    345         parse_constant is None and object_pairs_hook is None and not kw):
--> 346     return _default_decoder.decode(s)
    347 if cls is None:

File ~/anaconda3/envs/py39/lib/python3.9/json/decoder.py:337, in JSONDecoder.decode(self, s, _w)
    333 """Return the Python representation of ``s`` (a ``str`` instance
    334 containing a JSON document).
    335
    336 """
--> 337 obj, end = self.raw_decode(s, idx=_w(s, 0).end())
    338 end = _w(s, end).end()

File ~/anaconda3/envs/py39/lib/python3.9/json/decoder.py:355, in JSONDecoder.raw_decode(self, s, idx)
    354 except StopIteration as err:
--> 355     raise JSONDecodeError("Expecting value", s, err.value) from None
    356 return obj, end

JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

JSONDecodeError                           Traceback (most recent call last)
Input In [7], in <cell line: 1>()
----> 1 API = CVAT( username="gytarasov", password="Tar-2022@", url="http://cvat.rcud-rt.ru/")

File ~/anaconda3/envs/py39/lib/python3.9/site-packages/CVAT/__init__.py:37, in CVAT.__init__(self, username, password, url)
     31 body: dict = {
     32     "username": username,
     33     "password": password
     34 }
     35 response: Response = self.session.post(url=f'{self.url}/api/auth/login',
     36                                        json=body)
---> 37 self.session.headers = {"Authorization": f'Token {response.json()["key"]}'}
     38 if response.status_code != 200:
     39     raise ValueError("Bad credentials")

File ~/anaconda3/envs/py39/lib/python3.9/site-packages/requests/models.py:976, in Response.json(self, **kwargs)
    972     return complexjson.loads(self.text, **kwargs)
    973 except JSONDecodeError as e:
    974     # Catch JSON-related errors and raise as requests.JSONDecodeError
    975     # This aliases json.JSONDecodeError and simplejson.JSONDecodeError
--> 976     raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)

JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Hello thank for your feed back.
I think it's because I set the token before checking the status code...
I created a new release v1.0.0 and this thing may be corrected.
Try with the new release and text me back please.
Thank you

Thank you.
This is reolved,
Now gives
ValueError: Bad credentials
I've checked on new private windows pasting there, they are correct,


Update/expense controller

[ ] Add GET requests for timestamps and categories
[ ]  Make JavaScript functions to work with other charts
[ ] Add GET requests for expenses in a year


not able to configure containerd

https://rancher.com/docs/k3s/latest/en/advanced/#configuring-containerd says that config for containerd should be in /var/lib/rancher/k3s/agent/etc/containerd/config.toml.
That's not the case with k3s 1.24.4+k3s1.
I tried to find config in several places but I think it simply doesn't exist anymore. So I'm not able to change any containerd settings.
If that's intentional then documentation is misleading.

That file will be created in that directory after starting k3s, but only if you’re using the containerd built in to k3s. If you’re using your own container runtime endpoint, or using docker, then none of this matters. As the docs say, you need to create a different file in that same directory (config.toml.tmpl, NOT config.toml) if you want to customize the template... so it doesn’t really matter if the toml file exists yet or not.
However, if you want to change the snapshotter, there’s a dedicated flag for that: --snapshotter


Wrong path conversion in url() for imported files

In this filesystem situation:
project/
|- app/... php stuff ...
|- web/
|  |- css/
|  |- img/
|  |  |- bg.png
|  |- less/
|  |  |- main.less
|  |  |- _imported.less

This is main.less:
@import "_imported";

This is _imported.less:
h1 {
  background: url(../img/bg.png);
}

Calling lessc from outside the less directory works:
cd project/
lessc web/less/main.less >web/css/main.css

Results in:
h1 {
  background: url(../img/bg.png);
}

While calling it from the less directory changes the path:
cd project/web/less/
lessc main.less ../css/main.css

Results in:
h1 {
  background: url(img/bg.png);
}

This is a problem for some auto-compile scripts and IDE integration plugins, that often work inside the less directory (as the project structure could be much more complex).
I tried several combinations of -ru and -rp without success.
In general the path conversion should not depend on from where the compiler gets called.
I am using lessc <IP_ADDRESS>.

I have the same problem... It works as expected in <IP_ADDRESS>

Same problem here.
I defined images dir in my vars.less:

@layout_images_dir: "../bundles/app/images/";
@img_main_bg: @layout_images_dir + "bg.png";

vars.less is included in other less file, where I want to use the @img_main_bg variable as an url for the image like this:

background-image: url(@img_main_bg);

The result is:

background-image: url("");

(This worked well with leafo lessphp)
I tried this way too:

background-image: url('../bundles/app/images/bg.png');

The result is:

background-image: url('bundles/app/images/bg.png');

I can confirm the same happens when compiling with PHP.

Same thing here. Thanks @Teyras for the tip.

After some digging I found out that relativeUrls option became false by default in v<IP_ADDRESS>, was true before.


new mutex parameter

Added new parameter 'mutex' - both in manifest and as adhoc (which overrides manifest).
  An empty string is synonymous with no-mutex, allowing for override adhoc ('--mutex=').
Functionality is implemented with JUnique.
JUnique is (at present) served from GitHub.
JUnique is packaged into the JAR.
pom.xml has been updated accordingly.
Code is commented.
Readme is updated.

We have discussed this and find it problematic to add a dependency on JUnique. We also have different views on whether the launcher should be in charge of the app mutex, or if this is an application issue.
I'm leaning towards including app mutex functionality in the launcher, but it has to be a clean room implementation, based on sockets. Would you be willing to investigate the possibility to write something like this from scratch?

I agree that it would be cleaner to have a built-inn solution, rather than one based on JUnique.
Yes, I would like to write a clean room socket-based implementation from scratch. No problem.  Looking forwards to it.  Will start next week.

Perfect! Thank you Terje, this will be a very nice addition to the launcher!

As we have agreed that @terjedahl will make a built in solution not based on JUnique, I am closing this PR

I apologize for notifying you sooner:
I have not yet had a chance to do this.  It would certainly be fun to do, but as my current solution is working satisfactorily for me, it has not been a priority.  I dare not give an estimate of when I will have a new solution in place.

No rush Terje :) Thanks for getting back to us :)


[Feature][Flink] Support custom Yarn user submission of applications

Purpose of the pull request

Brief change log

Verify this pull request

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

Specifies where a yarn user submission task can be verified to be submitted by this user.


"No matching version found for<EMAIL_ADDRESS>with NPM v5+

OS: Mac OS Sierra
Node: 8.0.0
Npm: 5.0.1
Trying to build examples/frontend-only on Mac OS got this error. On Windows and Ubuntu build is successful.
> gradlew build
:npm-preunpack
:npm-configure
:npm-install
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of 'frontend-only'
npm ERR! notarget

npm ERR!      /Users/name/.npm/_logs/2017-06-06T14_54_18_482Z-debug.log
:npm-install FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':npm-install'.
> npm install failed (exit code = 1)

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 5.804 secs

UPD: It seems that problem is not related to OS. The problem caused by newer versions of npm. Npm v3.10.* which comes with Node v6.* is working as expected. But Npm v5.0.* which comes with Node v8.* ignores modules already installed in node_modules folder (in this case kotlin.js library) and tries to download it from npm repository.
The best solution I've found so far is to change
"dependencies": {
  "kotlin": "1.1.2-4"
}

to
"dependencies": {
    "kotlin": "file:./node_modules/kotlin"
}

and then npm install manually.
After that gradlew bundle is working properly. But after clean the above procedure should be repeated.

@mikhalchenko-alexander In which file exactly did you change the dependencies for a workaround?

@vonox7 The /frontend/build directory get generated each time I run gradle, and those values get back to their default values. Any suggestion?

@channguyen Do you run just the frontend run task or more?

Note that on Windows, it's backslashes (ugh).
    "dependencies": {
        ...
        "kotlin": "file:.\\node_modules\\kotlin",
        "kotlin-test-js": "file:.\\node_modules\\kotlin-test-js",
        "kotlinx-html-js": "file:.\\node_modules\\kotlinx-html-js"
    },

In 0.0.20 it should be fixed.
Also there is a workaround: you can specify nodejs version
kotlinFrontend {
    downloadNodeJsVersion = "7.9.0"

Note that specifying nodejs version it will cause bundle downloading at first run

Yes, the plugin v0.0.20 is working as expected with the latest Node v8.2.1 and NPM v5.3.0. Thanks.


order primary key with partition column first

Switches default primary key from (id, tenant_id) to (tenant_id, id) as described online and for increased usability.

Just noticed #68 would achieve the same result. Feel free to close this as a duplicate if you think that one's close to being merged 👍

Hi @zacharywelch,
Thank you for your contribution, I would like to merge your PR and release a new version. Could you sign the CLA so that I can do this?
Thanks


Do not override base user agent

See https://github.com/aetrion/dnsimple-go#26

@jacegu the feature is there, but I honestly have no idea how to test it (and if it's possible).

@jacegu I made some research. What I found is that:

excvr doesn't support it

in all cases I've checked, including httpoison tests theirselves, the common approach is to use mocks. In fact, httpoison includes a mock library
 {:meck, "~> 0.8.2", only: :test},

We have two options here:

We just move on, and we don't test this specific change
We introduce the dependency on the mock library

Preferences?

@weppos I rather not test it than introducing a new dependency.
If I come with a solution to test it without introducing the dependency I'll let you know.

👍