Oatts throws an error while running mocha command

Hi,
As given in the github documentation(https://github.com/google/oatts), following command generates the .js file for mentioned specification.
oatts generate -s ./path/to/openapi.yaml -w ./output/dir
For next steps, while running them using mocha (mocha --recursive (test directory)), this gives me following error for JS file.
"Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves."
Is there a way to resolve this? Which rules(validation) does Oatts uses?
Thanks.

Hi @Hiral-Dadhaniya thanks for using oatts and opening an issue.
oatts uses a rest testing framework called chakram to execute requests and assert on responses. If you take a look at your generated code, you'll notice a return chakram.wait() inside the callback for every it clause. As described here, this is returning a promise for mocha to wait for.
In this scenario it looks like the test timeout used is 2000ms (the default), and the test is being preempted. A couple of possible scenarios might be that the request is taking longer than the allotted timeout to complete, somehow the server is not reachable but the client's timeout for this is longer than the test's timeout, or there could even be an issue with how mocha is attempting to resolve the promise (i believe this would be out of the scope of this code base and we'd need to dig deeper to prove it).
I'd ask you to please raise the timeout by running the tests with a --timeout flag, and to ensure that the server you are targeting in the tests is reachable from the machine you are running the tests on (server is available on the host/port, firewalls aren't blocking, etc.)
Thank you!

Hi @noahdietz than you for quick response.
Yeh, looks like timeout issue. It works with --timeout flag. I was under impression oatts may use own rules to validate json/yaml files, but I think as you mentioned it's the chakram for executing requests and doing assert. So, is it possible to have own rules to validate json/yaml or .js(As it creates the JS files).

OK glad the timeout flag helped.
If I understand the question, you want to know how oatts validates the JS files it creates? Or the spec that it consumes? Or are you asking about the assertions done on the response as part of the tests that are run? Not sure I am following the question can you please explain in more detail? Thanks.

Yes, Sure.
Basically i am interested in how Oatts validates spec that it consumes. But as it creates JS out of it and validates using chakram, am not sure specific validations of spec file for edit/update supported. (For example: json structure, schema(These two i think chakram itself does?), specific rules like 'parameter objects should have a description' or 'no comma in description' etc. )

oatts uses a module called sway to consume & traverse the given specification. sway also validates the schema of the given OAS document, ensuring it conforms. However this is not the goal of oatts. oatts is meant to generate a test suite (of JS files) that would test the contract validity between the OAS document and the server that implements the API. It does not validate the schema of the specification specifically as a test.
I am going to close this issue as we have resolved the issue. Thanks!


Usage with coffeescript and source maps

I may be using webpack wrong but it appears to me that the ng-annotate-loader does not work on coffeescript files if you have source maps enabled. Here are the relevant pieces of my configuration:
devtool: 'eval-cheap-module-source-map',
loaders: [
  { test: /\.coffee$/, loader: 'ng-annotate-loader!coffee-loader', exclude: commonExclude },
]

So I am running the coffeescript loader on the coffeescript files, then running ng-annotate on the resulting javascript files. This results in the error below for every coffeescript file:
ERROR in ../~/raw-loader!../~/ng-annotate-loader/loader.js!../~/coffee-loader!./app/bigscreen/tools/donutChart/donutchart.coffee
Module build failed: Error: "../donutchart.coffee" is not in the SourceMap.
  at SourceMapConsumer_sourceContentFor [as sourceContentFor] (/Users/jason/sift/livesift/node_modules/ng-annotate-loader/node_modules/source-map/lib/source-map/source-map-consumer.js:677:15)
  at SourceMapGenerator.<anonymous> (/Users/jason/sift/livesift/node_modules/ng-annotate-loader/node_modules/source-map/lib/source-map/source-map-generator.js:233:42)
  at Array.forEach (native)
  at SourceMapGenerator_applySourceMap [as applySourceMap] (/Users/jason/sift/livesift/node_modules/ng-annotate-loader/node_modules/source-map/lib/source-map/source-map-generator.js:232:34)
  at Object.module.exports (/Users/jason/sift/livesift/node_modules/ng-annotate-loader/loader.js:47:17)
 @ ../~/script-loader!./app/bigscreen/tools/donutChart/donutchart.coffee 1:80-370

Is there something wrong with my configuration or does this appear to be a bug?

@axelson We had an #6 issue mostly about this. Please try 0.0.6 version, it would help.

Thanks, I didn't realize that issue was related. It's working great now, thanks!


README needs something visual

Hi!
I'd really like some sort of visual representation in the README of either the problem described or the solution that the package is proposed to address.
Can anyone out there help?
@demitri?

Sorry I didn't reply! I don't have any specific answer - it's not an easy thing and I'm definitely not a good artist!

Haha no worries. Just been preparing the repo for the global sprint. You should check it out! Any feedback welcome !
Also are you sprinting this year?


Support for parallel test runners and output files

ember-exam can parallelize an Ember test suite by using multiple Chrome instances. However, because ember-cli-blanket uses the /write-coverage-report endpoint at the end of a test suite with a single output filename, only the last write will win.
If you could do something like specify a function that returns the output filename, and pass it an index as an argument, then you could write coverage-1.lcov, coverage-2.lcov, etc. This would work automatically with services like codecov.io that automatically merge report from parallel runners.

I'm more than happy to accept a PR on this, but most of my future effort is going to be focused on ember-cli-code-coverage - if you would like to help contribute a parallelize feature to that project I'd be happy to work with you on it


Roll engine 12bf95fd49b7..49d6552e76c0 (16 commits)

https://github.com/flutter/engine/compare/12bf95fd49b7...49d6552e76c0
git log 12bf95fd49b7..49d6552e76c0 --first-parent --oneline
2019-12-11<EMAIL_ADDRESS>Verify accounting for loop counts in Gif and WebP assets is consistent. (flutter/engine#14321)
2019-12-11<EMAIL_ADDRESS>[fuchsia] Do not Execute paint tasks when there is no vsync (flutter/engine#14298)
2019-12-11<EMAIL_ADDRESS>Roll src/third_party/dart c74a8ec2c46e..bcd18e67dcae (9 commits) (flutter/engine#14317)
2019-12-11<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from nqJnP... to UdfLO... (flutter/engine#14316)
2019-12-11<EMAIL_ADDRESS>Roll src/third_party/skia 732c49739fa5..5afc7b2af854 (16 commits) (flutter/engine#14315)
2019-12-11<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from 9C6UA... to h4iiT... (flutter/engine#14314)
2019-12-11<EMAIL_ADDRESS>[web] Update build_web_compilers to 2.7.1 (flutter/engine#14305)
2019-12-11<EMAIL_ADDRESS>Cleanup the IO thread GrContext (flutter/engine#14265)
2019-12-10<EMAIL_ADDRESS>Fix for tab not working (flutter/engine#14165)
2019-12-10<EMAIL_ADDRESS>[SkParagraph] Convert the height override flag in text styles (flutter/engine#14283)
2019-12-10<EMAIL_ADDRESS>Roll src/third_party/dart 02a8b015ad..98c13ba18f (5 commits) (flutter/engine#14280)
2019-12-10<EMAIL_ADDRESS>Remove specificity on Android and iOS (flutter/engine#14282)
2019-12-10<EMAIL_ADDRESS>Create separate objects for isolate state and isolate group state (flutter/engine#14268)
2019-12-10<EMAIL_ADDRESS>Roll src/third_party/dart 8b8894648f..02a8b015ad (26 commits) (flutter/engine#14278)
2019-12-10<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from Zkpa_... to nqJnP... (flutter/engine#14274)
2019-12-10<EMAIL_ADDRESS>Roll src/third_party/skia e56cc054dbae..ab26643258ad (3 commits) (flutter/engine#14273)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Trybots failed. These were the failed builds: https://github.com/flutter/flutter/runs/344129990 , https://github.com/flutter/flutter/runs/344129980 , https://github.com/flutter/flutter/runs/344129979 , https://github.com/flutter/flutter/runs/344129932 , https://github.com/flutter/flutter/runs/344129918 , https://github.com/flutter/flutter/runs/344129901 , https://github.com/flutter/flutter/runs/344129859

Commit queue failed; closing this roll.


Rename to_bigint to to_big_int

Since BigInt is made up of two different words, they should be separated with an underscore in methods like to_bigint and to_biguint.

@gsingh93 I'd be severely against it. All frameworks have combined these two words -- besides BigDecimal, BigInteger and relevant class names are used as one word.

Not that my opinion matters, but I agree with @andylamp that the method names should remain the same as they relate directly to the class names.

Why do we choose to go against the conventional Rust naming conventions in just this specific case? Because there's precedent in other language?
In Rust there's precedent for using snake case here. Look at the BinaryHeap struct for example. It's contained in a module called binary_heap, not binaryheap. We're diverging from common Rust practices with this naming convention.

I agree that the general precedent in Rust is to convert CamelCase to snake_case.  But there's also a historical precedent here, since this implementation came from the Rust standard library in the first place! (pre-1.0)
I don't feel strongly either way, but to avoid breaks I'd favor the status quo.  We could change it in a future release bump, but there's no real benefit here, just a slight naming consistency.

FYI, num-bigint is in its own repo now: https://github.com/rust-num/num-bigint
But honestly, I still don't feel motivated to make such a breaking change.


Question/Feature Request: How to replace the build step?

Hi there!
I want to replace the compilation step and do something similar to what tshy so I can rely on running tsc and have more control of the compilation.
Since tshy does some of the work that packemon does, I was thinking of simply having a script to build each dialect. The script is simple and does the following:

creates a tempory package.json in src with appropriate module
symilnks the package root to src/node_modules/[PKG] to support imports
compiles tsconfig.DIALECT.json
deletes package.json and src/node_modules

How hard would it be to customize packemon to call a script or a function I'm exporting to do that?

The build process is very tightly coupled into packemon, especially around folder structure, and how exports are mapped. I don't have plans to change this, as packemon is meant to be used as-is, since it does all the heavy lifting.
You're probably better off forking packemon if you want this kind of custom functionality.

The build process is very tightly coupled into packemon, especially around folder structure, and how exports are mapped. I don't have plans to change this, as packemon is meant to be used as-is, since it does all the heavy lifting.
You're probably better off forking packemon if you want this kind of custom functionality.

I understand. Since packemon optionally supports swc I thought this could fall into that category.
I don't mind forking to customize, but before doing that, I would like to know if there's anything else that packemon depends on for the rest of the work that maybe comes from babel or rollup being the folder structure.
In other words, my idea was that I would replace only the transpilation phase, ensuring the folder structure is exactly what packemon expects

Packemon only supports swc, as it's a drop-in replacement for babel, both of which get fed into rollup. Both babel and rollup have custom plugins to achieve certain functionality.

In other words, my idea was that I would replace only the transpilation phase, ensuring the folder structure is exactly what packemon expects. Would that be enough? What are other possible pitfalls?

This should be enough off the top of my head. Would need to dig through everything in Package and Artifact files to verify.

Got it! I'll take a look and see if it's too much trouble or not.

Oh, btw, are you interested in taking a look at the solution (in case I decide to implement it) to see if a PR makes sense?
Or is that something you don't want to support due to the added complexity?

Yeah, depends on the complexity.


Feature/importing options

Changes info here: https://app.hacknplan.com/p/113092/kanban?categoryId=0&boardId=295774&taskId=78&tabId=basicinfo

Yeah, I'm still thinking about the uuid, taking in account we are using the imported file path hash as uuid these are my concerns:

If you move a file to a different path, and you have a file with the same name in the old path the uuid is going to be the same, when maybe this files are no the same.
If you do the same than above but you change the file uid when this file is moved you will have all the references to it broken.
By doing some random uuid you don't have the first problem, buuuut you have second problem.

I was reading Unity documentation and I found these:

If an asset loses its meta file (for example, if you moved or renamed the asset outside of Unity, without moving/renaming the corresponding .meta file), any reference to that asset will be broken. Unity would generate a new .meta file for the moved/renamed asset as if it were a brand new asset, and delete the old “orphaned” .meta file.

So I think I would make this uid random, but we will need to keep the meta files in the repo, otherwise, any file pointing to these resource will lose its track.
So as summary I think I will make uid random, make the files be store by the uid as unity instead as imported file and have good control over reimporting and keeping the old uid. But we will need to keep the meta file in the repo and don't touche them at all.


Dynamic object attributes / params parsing

For having an agnostic framework, try to make this process generic. For parsing we understand the serialization of the attributes from the xml responses

Developed in #46


Flaky test: ShutdownTestRun

This test fails occasionally with the following error:
System.Exception : dotnet publish exited with exit code : 1
   at Microsoft.AspNetCore.Server.IntegrationTesting.ApplicationDeployer.DotnetPublish(String publishRoot) in /_/src/Microsoft.AspNetCore.Server.IntegrationTesting/Deployers/ApplicationDeployer.cs:line 150
   at Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer.DeployAsync() in /_/src/Microsoft.AspNetCore.Server.IntegrationTesting/Deployers/SelfHostDeployer.cs:line 55
   at Microsoft.AspNetCore.Hosting.FunctionalTests.ShutdownTests.ExecuteShutdownTest(String testName, String shutdownMechanic) in /_/test/Microsoft.AspNetCore.Hosting.FunctionalTests/ShutdownTests.cs:line 70
   at Microsoft.AspNetCore.Hosting.FunctionalTests.ShutdownTests.ShutdownTestRun() in /_/test/Microsoft.AspNetCore.Hosting.FunctionalTests/ShutdownTests.cs:line 33
--- End of stack trace from previous location where exception was thrown ---
------- Stdout: -------
| [0.025s] TestLifetime Information: Starting test ShutdownTestRun at 2018-07-25T01:54:30
| [0.001s] TestLifetime Information: Starting test ExecuteShutdownTest at 2018-07-25T01:54:31
| [0.014s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: Deploying [Variation] :: ServerType=Kestrel, Runtime=CoreClr, Arch=x64, BaseUrlHint=, Publish=True
| [0.023s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: SET ASPNETCORE_ENVIRONMENT=Shutdown
| [0.023s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: Executing command dotnet publish  --output "/mnt/temp/buildTmp/2fddb6a2-127f-4395-b479-db25173f3890" --framework netcoreapp2.2 --configuration Release --no-restore -p:VerifyMatchingImplicitPackageVersion=false
| [0.471s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: dotnet-publish stdout: Microsoft (R) Build Engine version <IP_ADDRESS>72 for .NET Core
| [0.472s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: dotnet-publish stdout: Copyright (C) Microsoft Corporation. All rights reserved.
| [2.208s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: dotnet-publish stdout: /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-10013/build/GenerateAssemblyInfo.targets(25,5): error MSB4062: The "Microsoft.AspNetCore.BuildTools.Sdk_GetAssemblyFileVersion" task could not be loaded from the assembly /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-10013/build/../tools/netcoreapp2.1/Internal.AspNetCore.Sdk.dll. Assembly with same name is already loaded Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/mnt/work/33bdfc1cae7b2a38/modules/Hosting/src/Microsoft.Extensions.Hosting.Abstractions/Microsoft.Extensions.Hosting.Abstractions.csproj]
| [2.226s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: dotnet-publish stdout: /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-10013/build/GenerateAssemblyInfo.targets(25,5): error MSB4062: The "Microsoft.AspNetCore.BuildTools.Sdk_GetAssemblyFileVersion" task could not be loaded from the assembly /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-10013/build/../tools/netcoreapp2.1/Internal.AspNetCore.Sdk.dll. Assembly with same name is already loaded Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/mnt/work/33bdfc1cae7b2a38/modules/Hosting/src/Microsoft.AspNetCore.Hosting.Server.Abstractions/Microsoft.AspNetCore.Hosting.Server.Abstractions.csproj]
| [2.308s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Error: dotnet publish exited with exit code : 1
| [2.312s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Warning: Host process already exited or never started successfully.
| [2.316s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Warning: Failed to delete directory : Could not find a part of the path '/mnt/temp/buildTmp/2fddb6a2-127f-4395-b479-db25173f3890'.
| [2.416s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Warning: Failed to delete directory : Could not find a part of the path '/mnt/temp/buildTmp/2fddb6a2-127f-4395-b479-db25173f3890'.
| [2.517s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Warning: Failed to delete directory : Could not find a part of the path '/mnt/temp/buildTmp/2fddb6a2-127f-4395-b479-db25173f3890'.
| [2.619s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: [Time]: Total time taken for this test variation '2.6044817' seconds
| [2.621s] TestLifetime Information: Finished test ExecuteShutdownTest in 2.6206626s
| [2.712s] TestLifetime Information: Finished test ShutdownTestRun in 2.7011415s

ShutdownTestRun failed again.

ShutdownTestRun failed again.

ShutdownTestRun failed again.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

@Tratcher - can you please take a look at this failing/flaky test?

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

@natemcmaster does this error make sense to you?
[2.399s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: dotnet-publish stdout: /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-20180731.2/build/GenerateAssemblyInfo.targets(25,5): error MSB4062: The "Microsoft.AspNetCore.BuildTools.Sdk_GetAssemblyFileVersion" task could not be loaded from the assembly /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-20180731.2/build/../tools/netcoreapp2.1/Internal.AspNetCore.Sdk.dll. Assembly with same name is already loaded Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/mnt/work/33bdfc1cae7b2a38/modules/Hosting/src/Microsoft.AspNetCore.Hosting.Server.Abstractions/Microsoft.AspNetCore.Hosting.Server.Abstractions.csproj]
[2.518s] Microsoft.AspNetCore.Server.IntegrationTesting.SelfHostDeployer Information: dotnet-publish stdout: /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-20180731.2/build/GenerateAssemblyInfo.targets(25,5): error MSB4062: The "Microsoft.AspNetCore.BuildTools.Sdk_GetAssemblyFileVersion" task could not be loaded from the assembly /mnt/work/33bdfc1cae7b2a38/packages/internal.aspnetcore.sdk/3.0.0-alpha1-20180731.2/build/../tools/netcoreapp2.1/Internal.AspNetCore.Sdk.dll. Assembly with same name is already loaded Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/mnt/work/33bdfc1cae7b2a38/modules/Hosting/src/Microsoft.Extensions.Hosting.Abstractions/Microsoft.Extensions.Hosting.Abstractions.csproj]

It happens when you run dotnet publish without --no-restore in the context of a Universe build. In general, it's better to use --no-restore if you can. It prevents dotnet-publish from re-restoring the .csproj and changing its dependencies.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

--no-restore is being used here.
There's definitely a build tools issue here. The tests pass on Mondays and continue to pass until Universe updates it's BuildTools dependency.
First failure 27 Jul 18 16:32: http://aspnetci/viewLog.html?buildId=517560&tab=buildResultsDiv&buildTypeId=Releases_22xPublic_UbuntuUniverse
This may have been the very first build of BuildTools produced by VSTS. Is it possible there was a regression in BuildTools when transitioned to VSTS?

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

ShutdownTestRun failed with about the same error on master.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

Should be resolved by this update to build tools https://github.com/aspnet/BuildTools/pull/734


All chests generating empty

I made a family server and thought this mod would be a nice addition. Well everytime we try to loot a village, maybe one chest has loot, if that. It also deletes the loot from the ruins mod we have so any structures made by it are missing their cool loot.

Which version of Minecraft and Lootr are you using, and could you give me a list of all of the mods that you have running? I need a bit more information to go off, I'm afraid.

The 1.15 version is no longer supported. Please let me know if you still have issues in 1.16.

The 1.15 version is no longer supported. Please let me know if you still have issues in 1.16.


[INLONG-1421][TubeMQ] Update Go SDK Apache License

Signed-off-by: Zijie Lu<EMAIL_ADDRESS>Title Name: [INLONG-XYZ][component] Title of the pull request
where XYZ should be replaced by the actual issue number.
Fixes #1421
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Codecov Report

Merging #1422 (e8be97b) into INLONG-25 (85aa1e4) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             INLONG-25   #1422   +/-   ##
===========================================
  Coverage         7.51%   7.51%
  Complexity         479     479
===========================================
  Files              267     267
  Lines            29500   29500
  Branches          4843    4843
===========================================
  Hits              2216    2216
  Misses           26810   26810
  Partials           474     474

Continue to review full report at Codecov.

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


Beatufication error

A structure like
    if (conditional) then
        if (end_flag = '0') then

does not beautify correctly.

Turns out the end clause was matching because end did not have a word boundary check afterwards.  Hideous result but fortunately easy fix.


Home Screen Scrolling issue Resolved (#7)

If you still have conflicts then you can let me know.

Changes were merged in another PR


setup_project_user_executor_profile shell function needs fix

Problem
There are useless local declarations in setup_project_user_executor_profile in file sechub-solution/8900-helper.sh line 45-58:
    local "$project"
    local "$user"
    local "$executor_file_name"
    local "$profile"

They define function-local variables dynamically named after the contents of vaious other variables.
This leads to odd error messages like
8900-helper.sh: line 45: local: `test-gosec': not a valid identifier
8900-helper.sh: line 48: local: `pds-gosec': not a valid identifier

Solution
IMHO the lines 45-48 can be removed.

I should have seen it when reviewing the code :smiling_face_with_tear:
@Jeeppler Please check and provide a fix.


start on the investors view

how about the http://www.highcharts.com/demo/pie-basic since we use it already?

done


docs: add a simple local development guide

What this PR does / why we need it:
Simply adds a markdown doc for local development. I keep forgetting what the auth config and registry ENVs when doing development work
Which issue(s) this PR fixes:
No open issues for this PR

Can you please lint all your Markdown files using markdownlint, please.

I suggest following the same layout as the AWS development guide: https://cluster-api-aws.sigs.k8s.io/development/development.html, i.e. start with explaining the prequisites, then set up the environment, then how to commit, etc.

@Djelibeybi ended up moving the doc into the book like you pointed out and expanding upon the developer guide a bit.


Add Hash#select intrinsic

Adds a compiler intrinsic for Hash#select.
Benchmarks
Hash#select with block:

source
interpreted
compiled

stripe/hash_select_baseline.rb
.140
.118

stripe/hash_select.rb
.495
.405

stripe/hash_select.rb - baseline
.355
.287

Hash#select without block (returns an enumerator):

source
interpreted
compiled

stripe/hash_select_baseline.rb
.139
.120

stripe/hash_select_enum.rb
.799
.625

stripe/hash_select_enum.rb - baseline
.660
.505

Motivation
General push for more compiler intrinsics.
Test plan
See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_KemYNkPlKVeQLM
→ https://go/builds/bui_KemYTWf0sal5UM

This looks good to me. Do you happen to have results from running your benchmarks on master?

Good point!
master:

source
interpreted
compiled

stripe/hash_select_baseline.rb
.142
.123

stripe/hash_select.rb
.516
.486

stripe/hash_select.rb - baseline
.374
.363

stripe/hash_select_enum.rb
.699
.668

stripe/hash_select_enum.rb - baseline
.557
.545

PR branch (rebased on same master rev):

source
interpreted
compiled

stripe/hash_select_baseline.rb
.140
.123

stripe/hash_select.rb
.514
.413

stripe/hash_select.rb - baseline
.374
.290

stripe/hash_select_enum.rb
.698
.644

stripe/hash_select_enum.rb - baseline
.558
.521

So actually I guess about a third of the observed speedup on hash_select_enum.rb


Ssh build

Pull Request Test Coverage Report for Build 2522

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 22.808%

Totals

Change from base Build 2498:
0.0%

Covered Lines:
2502

Relevant Lines:
10970

💛  - Coveralls


update documentation

Current coverage is 96.44% (diff: 100%)

Merging #33 into master will not change coverage

@@             master        #33   diff @@
==========================================
  Files            10         10
  Lines           647        647
  Methods         570        570
  Messages          0          0
  Branches         77         77
==========================================
  Hits            624        624
  Misses           23         23
  Partials          0          0

Powered by Codecov. Last update bd92f4a...6ac6688


Select xAxis to draw value.

General Questions
Is it possible to choose the x axis where I want to draw the values? , I need that the bars have 2 different colors and are on 2 different axes
Expected Behavior

Current Behavior

the two series are being drawn on the axis1
Online Example
http://jsfiddle.net/kfbp5j9z/1/
Environment

ECharts version；ECharts 版本: 4.1.0

You can use data: [['axis1', 30]]. See demo here: http://jsfiddle.net/ovilia/b1dmg93v/2/


Toast utility tests

[ ] Unit tests
[ ] Integration tests
[ ] Migrate to Consensys repo

Pushed up to:
https://consensys.github.io/rimble-app-demo/


Revert to not allocating fluid cons vars if fluid is not active?

There are some difficulties getting radiation to work right in all cases if the fluid is off when not allocating fluid cons variables, so I changed the code in #104 to always allocate both fluid prim and cons vars. Maybe in the future we want to think a little more carefully and stop allocating fluid cons vars if we don't have fluid fluxes.

I think it's fine to allocate fluid cons, if we need them, so long as the fluid is frozen and not evolved.


Some of the custom controls don't work with Unity6

https://monosnap.com/file/3q87druoRi7YEcd3FRL0HbeqL4i6v0

The problem is that UNITYMVVMTOOLKIT_TEXTMESHPRO_SUPPORT is not defined.
You should add it into Player Settings like this:

I guess the problem was caused by the reason that package for Text Mesh Pro was removed, because Text Mesh Pro was added built-in com.unity.ugui package


Fix code scanning alert - URL redirection from remote source

Fix code scanning alert - URL redirection from remote source
issue has been resolved.


How I can use dependencies in StackedApp?

How I can use dependencies in StackedApp with my config for ApiService? Can u suggest me how i can make like this not in thirdparty_services_module?
abstract class ThirdPartyServicesModule {
  @lazySingleton
  KeyValueStorageService get keyValueStorageService => KeyValueStorageService.getInstance();

  @LazySingleton(as: ApiConsumer)
  BBHttpClient getApiConsumer(
    KeyValueStorageService keyValueStorageService,
  ) {
    return BBHttpClient(
      baseUrl: '${dotenv.env[BaseUrlKey]}/api/',
      keyValueStorageService: keyValueStorageService,
    );
  }
}

Here is all the dependency registration methods we support.

Here is all the dependency registration methods we support.

Not it's not working for me, because i need provide the params (keyValueStorageService & baseUrl)

Can u help me to connect my dependency?

Did you use the factory parameters as in the docs I linked?

Yep i don't understend how can include BBHttpClient

If it's not in those docs then the feature is not supported the way you want to use it.


changes on readme

[Current coverage][cc-pull] is 92.23%

Merging [#54][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #54   diff @@
==========================================
  Files             1          1
  Lines           206        206
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            190        190
  Misses           16         16
  Partials          0          0

Powered by Codecov. Last updated by [770b09f...8e80108][cc-compare]
[cc-base-branch]: https://codecov.io/gh/inaka/EventSource/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/inaka/EventSource/compare/770b09fa108f2295d45e8f84a7cccb978c72c78d...8e801083eadfe969b9d08578aa962c03b9e7ce43
[cc-pull]: https://codecov.io/gh/inaka/EventSource/pull/54?src=pr


[Feature Request] Add /clearchat command

Hey, I think a /clearchat command would be extremely handy and quite important for a chat plugin. Especially when someone sends messages that should be deleted.
I would make the command look like this: /clearchat [self/all/<channel>] (I think what does what is self-explanatory)

In vanilla Minecraft clients there isn't a way for servers to clear their chat, apart from sending a lot of blank chat messages so the chat history get filled with them.
However, the chat messages would still be displayed in the game's logs, also, some modded clients can still display the messages in the chat.


Better control on how missing fields are treated in the generation

Summary of the new feature / enhancement
I am new to to PlatyPS, and I've been trying to use 1.0.0-Preview1 to generate markdown for the Devolutions.PowerShell module. By default, PlatyPS uses placeholder values that are mean to be manually replaced, like "{{ Fill XYZ }}". The problem is that even if it's good practice to have all the fields (synopsis, inputs/outputs descriptions, related links, etc), this just makes the generated markdown unsuitable for a "good enough" generated static site, one which can still be very useful to navigate through all available cmdlets and parameters.
I don't mind quality checks to list all the missing fields that should be added, but at a bare minimum we should be able to easily control how to treat missing fields: skip them in the generated markdown, or specify the placeholder value to use. Originally, I thought I could work around it by scripting checks for values containing "{{ Fill" to replace them with "", like this:
Import-Module $ModuleName
Get-Command -Module $ModuleName -CommandType Cmdlet | ForEach-Object {
	$CommandHelp = New-CommandHelp $_
    $CommandHelp.RelatedLinks | ForEach-Object { $_.Uri = "/docs/$($_.LinkText)" }
    #$CommandHelp.RelatedLinks += [PSCustomObject]@{ Uri = $ModuleName; LinkText = $ModuleName }
    $CommandHelp.Inputs | Where-Object { $_.Description.Contains("{{ Fill") } | ForEach-Object { $_.Description = "" }
    $CommandHelp.Outputs | Where-Object { $_.Description.Contains("{{ Fill") } | ForEach-Object { $_.Description = "" }
    if ($CommandHelp.Synopsis -and $CommandHelp.Synopsis.Contains('Fill ')) {
        $CommandHelp.Synopsis = ""
    }
	Export-MarkdownCommandHelp -CommandHelp $CommandHelp -OutputFolder $OutputFolder -Force
}

Putting aside the fact that checking for specific values like "{{ Fill" is prone to break if the string changes, I hit another problem: some fields are arrays, where empty arrays of objects result in placeholder text in the output. Since we can't just replace the placeholder value in the object, the only workaround I could find was to post-process the markdown to do a search and replace, which further makes PlatyPS hard to use:
Get-Item "$OutputFolder/$ModuleName/*.md" | ForEach-Object {
    $filePath = $_.FullName
    $content = Get-Content -Path $filePath -Raw
    $content = $content -replace '\{\{Insert list of aliases\}\}', ''
    $content = $content -replace '\{\{ Fill in the related links here \}\}', ''
    Set-Content -Path $filePath -Value $content
}

In the case of related links, if there are none, I'd just like to either skip it, or put the text "None". Post-processing the textual output is very suboptimal and I would like to handle it on the CommandHelp object. I thought of trying to add at least one related link on all cmdlets, but hit another issue: CommandHelp.RelatedLinks is a read-only property, so while I could modify the existing items, I cannot add new items.
My understanding is that the new PlatyPS is designed to make it easier to customize things by scripting modifications to the CommandHelp objects, but from my experience so far (about a day), I've hit several blockers that require hackish workarounds. Is what I'm asking for already supported, but I haven't found the right way to do it, or is it not currently supported? Thanks!
Proposed technical implementation details (optional)
No response

I was struggling with it as well and there's a way to set internal properties using reflection. Take the following example:
$commandHelpObject = [Microsoft.PowerShell.PlatyPS.Model.CommandHelp]::new()

# Set your properties here that can be set

# To set properties that are gettable only, use reflection. For example, to set the parameters, you can use something like this
$list = [System.Collections.Generic.List[Microsoft.PowerShell.PlatyPS.Model.Parameter]]::new()
$parameter = [Microsoft.PowerShell.PlatyPS.Model.Parameter]::new("MyParameter", "System.String")
$list.Add($parameter)

# Now reflect on it
$property = $commandHelpObject.GetType().GetProperty('Parameters')
$property.SetValue($commandHelpObject, $list, $null)

# Run MarkdownToString()
$commandHelpObject.ToMarkdownString()


containerapp: fix create or update containerapp --yaml exposedPort

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

containerapp

@zhoxing-ms
Hi, this PR has been reviewed by hong and zunli, and it needs to be merged before https://github.com/Azure/azure-cli-extensions/pull/6081
please help to review and merge, thanks!


fix: Optional previous element subject

This fix is small, but removed the error of Cannot read property 'ownerDocument' of undefined. This change will change the error to be the built-in Cypress error stating the previous subject must be an element (if supplied). It will then output the subject and previously run command.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Else condition after a newline gets skipped

Description
The else condition gets skipped in case it's written after a newline.
Expected behavior
Either the code executes the else condition or fails with an error.
Current behavior
The else condition does not get executed.
Additional information
Here is a small program to reproduce the issue:
fn main() {
    let a:bool = false
    if a == true {
        outln("True")
    }
    else {
        outln("False")
    }
}

This issue is fixed with commit f4b18b0, thanks for your contribution.


Please update this page for Azure ML v2 version

[Enter feedback here]
Please update this page for Azure ML v2 version.

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

ID: d2607987-8b8e-9785-b8b3-54805fefe6da
Version Independent ID: a47d24c3-d193-98f0-26da-9c49dcecd3c5
Content: Use private Python packages - Azure Machine Learning
Content Source: articles/machine-learning/v1/how-to-use-private-python-packages.md
Service: machine-learning
Sub-service: core
GitHub Login: @rastala
Microsoft Alias: roastala

@krishtanya
Thanks for your feedback! We will investigate and update as appropriate.

@krishtanya
I've delegated this to @rastala , a content author, to review and share their valuable insights.


hello world using hapi.js

What the program do ?
---- Add hello world using HapiJs Framework (NodeJs)
In what programming language it is written?
HapiJs ( NodeJs Framework )
For test just npm start and open with http://localhost:8000/hello
#33 Hello World HappiJS #1

This pull request exeeds the max additions limit. Please fix it and reopen


GazeboControlPrivate::Reset() not called after a /reset_sim service call

Currently working on reinforcement learning with ROS 2 and Gazebo, and I am using gazebo_ros2_control to simulate some effort_controllers and /controller_manager/switch_controllers to call services. When I call /reset_sim, I notice that the controllers freeze and my /switch_controllers service calls are very delayed. I was thinking about creating a custom plugin that inherits from gazebo_ros2_control to restart the controls without service calls, but I noticed that GazeboRosControlPrivate::Reset() doesn't get called when simulation resets. I verified by putting and RCLCPP_INFO() statement in the function call when building gazebo_ros2_control from source. I was wondering what I could add in a custom plugin that inherits from GazeboRosControlPlugin to make sure GazeboRosControlPrivate::Reset() gets called.

This would be desired behaviour from gazebo_ros2_control so please try adding it here to share the fix with the rest of us.


Allowing EXCLUDE_FROM_ALL

Hi @TheLartians ,
Thanks for the library.
When I include a CMake library with CPM.cmake it runs all the install commands in that library even when I don't need these installers with my own library.
When adding the library as a subdirectory, it's possible to use add_subdirectory(libname EXCLUDE_FROM_ALL) to avoid that behavior.
Is it possible to use EXCLUDE_FROM_ALL when downloading a CMake library?
I don't know if that's a question or an enhancement.
Best,

Hey, that's a good question. Currently I don't think there is a good way to achieve this behaviour besides setting the DOWNLOAD_ONLY flag and calling add_subdirectory manually.
I actually wasn't really aware of this option until now, but it does sound like it could be the desired behaviour for most use-cases (given the option to opt-out). Will setting EXCLUDE_FROM_ALL on a directory still allow us to install all targets that are used normally or are there some caveats?

Setting DOWNLOAD_ONLY to TRUE and calling add_subdirectory manually is something I didn't think about. This should be able to achieve something more similar to find_package. It's a great option. What a Columbus' egg. It's not a replacement for find_package in the sense that I'll need some conditionals after that but it's more than enough for what I need. Feel free to close this issue if you want.
If you're still interested in the idea anyway, using EXCLUDE_FROM_ALL could be a good practice when using add_subdirectory as a way to emulate what find_package does. I think that's why they created this option. Because when you use find_package it doesn't mess with your install commands and these imported targets don't get built when you build all. Without EXCLUDE_FROM_ALL, add_subdirectory would never be able to replace find_package properly in any situation because find_package never puts its own targets in your install commands. Some libraries include an option to not build the install target for that reason, but not many libraries do that.
Regarding your question, as far as I understand, EXCLUDE_FROM_ALL does not forbid us to install these imported targets in any way. It just stops these scripts in the subdirectory from messing with the install commands in the main project by default with no way to opt-out. Putting external packages in your own package becomes an option rather than a requirement for all external packages. By default, it behaves a little as if you were using find_package. It's also great not to have lots of useless targets in your "build all" when you just want to use the subdirectory as if it were a package. I'm sorry if I didn't understand your question properly.

Thanks for the response, I'm glad it has resolved your issue!
The option does sound like a good idea. I think I'll do some experimenting with the option on my own and evaluate if it makes sense to include it as a default. Thanks for bringing it up!

+1 on this. I have been encountering these same issues with using other CMake-based projects as sub-projects.
I've seen some CMake projects get around this by wrapping their install(export) commands with a condition to check if it's being built as a sub-project or not (such as https://github.com/catchorg/Catch2), but providing an option in CPM to expose the EXCLUDE_FROM_ALL behaviour is IMHO a better solution (wrapping install commands has some side-effects: https://github.com/catchorg/Catch2/issues/2164)

As you may see from the referenced commit, I have had a go hacking this feature into CPM, with some success. In that commit, I've just set EXCLUDE_FROM_ALL always for targets imported with CPM when FetchContent() is used, however I have been unable to add this as a new option for CPM, nor write any new tests for it or fix the tests that the change broke.
If anyone else wanted to take a look at refining my hack, of course that commit is available as a starting point if needed...

Hey @saxbophone, thanks for bringing the issue up again and creating an example implementation. I've done some testing and believe the best approach for now is to add an optional flag EXCLUDE_FROM_ALL that can be added to the CPMAddPackage call. I've created an example implementation in #198 and published a pre-release for easy testing of this feature.

Since then, I've been using:
FetchContent_Declare(
  googletest
  GIT_REPOSITORY https://github.com/google/googletest.git
  GIT_TAG        release-1.8.0
)
FetchContent_GetProperties(googletest)
if(NOT googletest_POPULATED)
  FetchContent_Populate(googletest)
  add_subdirectory(${googletest_SOURCE_DIR} ${googletest_BINARY_DIR} EXCLUDE_FROM_ALL)
endif()

because without the EXCLUDE_FROM_ALL option, both snippets would be almost as verbose anyway.

Yeah, gotcha there, if you don't need any of the additional CPM.cmake features, this would give you most control. FYI in the current implementation, the API would be as
CPMAddPackage(
  NAME googletest
  GITHUB_REPOSITORY google/googletest
  GIT_TAG release-1.8.0
  EXCLUDE_FROM_ALL YES
)

I prefer the explicit version vs using it as a default for now, as I've discovered that even my own projects rely on implicit installation of added dependencies and this would be a breaking feature. I still have to add a test case, but would love to hear any feedback you have.

Hi @TheLartians thanks for taking a further look and making that pre-release, I'll take a look at using it for my use case and see if it does the job.

I prefer the explicit EXCLUDE_FROM_ALL option vs enabling it as a default for now, as I've discovered that even my own projects rely on implicit installation of added dependencies and this would be a breaking feature. I still have to add a test case, but would love to hear any feedback you have in the meantime.

I completely agree that EXCLUDE_FROM_ALL should be an explicit option rather than enabled by default, as it would be unfortunate to break existing projects that might rely on the current implicit installation and it's likely anyone who needs the new behaviour, will know they need it and not miss out on finding it when documented.

I've just tested the new pre-release v0.29.0-preview-4 and it works just fine for my use case, where I need to find SDL and want to use installed versions where available, or source versions where not. I've tested the pre-release using CPMFindPackage() for this reason:
CPMFindPackage(
    NAME sdl2
    GIT_REPOSITORY https://github.com/libsdl-org/SDL.git
    GIT_TAG main
    VERSION 2.0.15
    OPTIONS "SDL_TEST OFF" "SDL_SHARED OFF" "SDL_STATIC ON"
    EXCLUDE_FROM_ALL YES
)

Note that this call will currently always retrieve SDL from source unless you've compiled your own copy from bleeding-edge, because v2.0.15 hasn't been released yet.

Awesome, thanks for the feedback and testing! I've just released the feature in v0.29.0. 🎉

No problem, thank you very much for cutting a new release with this feature so quickly, much appreciated! 🤝

Wouldn't it be possible to have flags without parameters?
CPMFindPackage(
    NAME sdl2
    GIT_REPOSITORY https://github.com/libsdl-org/SDL.git
    GIT_TAG main
    VERSION 2.0.15
    OPTIONS "SDL_TEST OFF" "SDL_SHARED OFF" "SDL_STATIC ON"
    EXCLUDE_FROM_ALL # YES
)

That would make it compatible with the syntax for add_subdirectory (add_subdirectory(_______ EXCLUDE_FROM_ALL) instead add_subdirectory(_______ EXCLUDE_FROM_ALL TRUE)). And the usual syntax for most cmake functions: (find_package(_______ QUIET) instead of find_package(_______ QUIET TRUE), etc). It seems more intuitive.

I did have that in the first implementation, but decided against the flag syntax, as

it's consistent with current CPM.cmake syntax: all current arguments use a KEY VALUE declaration, including DOWNLOAD_ONLY, GIT_SHALLOW, etc. IMO it would be confusing if some parameters were without value and others with.
it's easier to extend: say for some reason we want to add an additional setting for EXCLUDE_FROM_ALL besides ON/OFF, we could just re-use the existing parameter, e.g. EXCLUDE_FROM_ALL sometimes.

That makes sense. Thanks again for including this option.

FYI, EXCLUDE_FROM_ALL is now the default behaviour for the new shorthand-syntax introduced in v0.31.0, allowing con definitions such as:
<EMAIL_ADDRESS>
instead of the way more verbose previous version
CPMAddPackage(
  NAME cxxopts
  GITHUB_REPOSITORY jarro2783/cxxopts
  VERSION 2.2.0
  OPTIONS
    "CXXOPTS_BUILD_EXAMPLES Off"
    "CXXOPTS_BUILD_TESTS Off"
)

Thanks again for the suggestion, this is a great improvement in usability!

That's beautiful! 😀 It's a single step from a requirements.txt file.
Imagine C++ projects being part of the GitHub dependency graphs.
Unfortunately, GitHub won't pick up the dependencies unless it's in one of the usual formats.

If CPM becomes popular enough, it will be once of the usual formats :smile:

It seems like they're just not really interested in common C++ formats because not even vcpkg and conan files are supported there.
It's a shame because being part of the graph has lots of advantages, like automatically documenting dependencies, inferring how far your project has reached and being recommended for sponsorships. Until they are interested, a file in one of the existing formats might be very useful, instead of reinventing the wheel.
Like CPM already has a package-lock.cmake, I was thinking of writing some script to annotate whatever dependencies I included as a pip requirements.txt/pipfile.lock or a npm package-lock.json/package.json.
A requirements.txt file might be easier because it's just a list of repositories. Just a matter of appending a new line to requirements.txt file whenever we call CPMAddPackages. For instance, if we call CPMAddPackage("gh:catchorg/Catch2@2.5.0"), we would just append a line like
-e<EMAIL_ADDRESS>Catch2

to requirements.txt.
A package-lock.json file might be more complicated because writing a json file from cmake must be very clumsy.
The only reason I haven't moved forward with this is that I would still need to compare the supported formats, and look into the syntactic details of pip requirements.txt.

FYI, EXCLUDE_FROM_ALL is now the default behaviour for the new shorthand-syntax introduced in v0.31.0, allowing concise definitions such as:
<EMAIL_ADDRESS>
instead of the way more verbose previous version
...

Wow! Thank you for the update!

It seems like they're just not really interested in common C++ formats because not even vcpkg and conan files are supported there.
It's a shame because being part of the graph has lots of advantages, like automatically documenting dependencies,
...

I too lament at the lack of inclusion of C++ projects in the dependency tracker. I did respond once to a GH survey about future expansion of the feature. My guess would be it's not on GH's priorities list in part due to focusing more on web-dev popular languages, as well as the fact that C++ unfortunately does not have wide consensus on build systems (although I understand CMake is a popular contender).

Like CPM already has a package-lock.cmake, I was thinking of writing some script to annotate whatever dependencies I included as a pip requirements.txt/pipfile.lock or a npm package-lock.json/package.json.
A requirements.txt file might be easier because it's just a list of repositories. Just a matter of appending a new line to requirements.txt file whenever we call CPMAddPackages. For instance, if we call CPMAddPackage("gh:catchorg/Catch2@2.5.0"), we would just append a line like
...

This is an exciting idea! I wonder if it would be worthwhile choosing a more unique common filename for the requirements file (that way, one could mix C++ and Python in the same project without the requirements files conflicting)?

I wonder if it would be worthwhile choosing a more unique common filename for the requirements file

That's what I would like, but I think GitHub would ignore the file if we did that.
Some alternatives would be (i) telling GitHub to look into another file, (ii) to annotate which dependencies are python or C++ dependencies, even if it's just a comment, (iii) to use a pip syntax to differentiate build modes, or to (iv) leave it to the lock file only.
The pros/cons seem like (i) Maybe there's a way to tell GitHub to look into another file, but I doubt that. (ii) Including it as just a comment is a workaround that doesn't really solve the problem, but it's better than nothing. (iii) I'm not sure pip provides a syntax for doing that outside comments, but I guess it must be possible somehow since even Python projects might have different dependencies depending on how you want to use it. (iv) I'm not sure if leaving it to lock file solves the problem or doesn't cause any conflict with python projects.
The second alternative seems most promising in that case. But that's a problem only assuming the project is mixing C++ and Python, and the project cannot use different directories for these things. It's more like a feature than a problem because even a python project might need two different sets of requirements. It should be ok to not have this feature in the initial implementation.
Another interesting thing is that CPM could annotate itself in every project that uses CPM, so it would also have the benefits of being part of the dependency graph and it would make clear which CPM version that project is using.
I apologize if these implications are going too far from the original issue. We could open a new discussion to compare alternatives and possibilities if you want. Both seem fine since this issue is closed anyway.

I apologize if these implications are going too far from the original issue. We could open a new discussion to compare alternatives and possibilities if you want. Both seem fine since this issue is closed anyway.

Good points all around, I am glad we have this opportunity to discuss these ideas with others who are interested in them. Please invite me to the discussion if it moves elsewhere!
...I wonder if it could be a good idea to see if there's any Github staff member who is enthusiastic about C++ enough that it could be worth tagging them in a conversation about C++ project dependency tracking, maybe with a view to making the case for such a feature to Github?

Yeah, this belongs in discussions. It's a very interesting topic

good idea to see if there's any Github staff member who is enthusiastic about C++ enough

That would be great. If anyone is interested in that, things should be much easier. However, even if no one is interested, I think it's still worth working on this.
So there is the discussion: #221
This would be great.


MessageBox icon missing

Hi, is there a reason for not providing "question mark" icon in MessageBox package (it is in enum MessageBoxImage)? It is avaiable in winforms

From your link

The message box contains a symbol consisting of a question mark in a circle. The question mark message icon is no longer recommended because it does not clearly represent a specific type of message and because the phrasing of a message as a question could apply to any message type. In addition, users can confuse the question mark symbol with a help information symbol. Therefore, do not use this question mark symbol in your message boxes. The system continues to support its inclusion only for backward compatibility.

right, I am used to it in winforms msgboxes :)

bad habit according to Microsoft :)


Can we upload artifact package?

In the standalone version, it is possible to serve a package from a specific location on the disk. There is no such possibility for the public version. We could make a separate function that would allow you to upload the artifact (for example in the form of a zipped file).

@vidyli links for you:
https://repman.io/docs/private-packages/#path
https://repman.io/docs/private-packages/#artifact

Thanks, that  woule be very nice of you.


Remove placeholder TPL imports from ZepToken

Two imports are made in ZepToken exclusively to have Truffle artifacts available. Despite this being supposedly solved, Truffle is not finding the artifacts in the TPL dependency.
Make sure to remove these when TPL is released as a package.

Fixed in https://github.com/zeppelinos/zos/commit/cc4eb539642ff5c255d0bece293d004c9dcbef6b


Created manager class which has same interface, like proposed in #17

I'll refactor once #33 is merged in, because we could use the interface in the commands instead.

When all 3 of these PR's are merged in I will kick off testing.

I'll have to resolve the conflicts when I get home from work.  Looks good tho.  Thanks!

Nah, I will refactor it for you, since you agree with the intention.

Refactor complete, can you review please?


Dependabot groups don't fetch all dependencies latest version

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
npm
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
/client/package.json
dependabot.yml content
version: 2

# Set update schedule for GitHub Actions
updates:

  - package-ecosystem: "github-actions"
    directory: "/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

# Check for NPM packages
  - package-ecosystem: "npm"
    directory: "/client/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"
    open-pull-requests-limit: 10
    # Create a group of dependencies to be updated together in one pull request
    groups:
       # Specify a name for the group, which will be used in pull request titles
       # and branch names
       angular-dependencies:
          patterns:
            - "@angular*"
       types-dependencies:
          patterns:
            - "@types*"

# Check for PIP packages
  - package-ecosystem: "pip"
    directory: "/dataset_manager/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/itf_model_registry/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/lux/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/message_queue_manager/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/metamorph/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/model_wrapper/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/monitoring/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/orm/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/report_manager/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/server/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

  - package-ecosystem: "pip"
    directory: "/test_run_manager/"
    schedule:
      interval: "weekly"
    rebase-strategy: "disabled"

Updated dependency

<p dir="auto" style="box-sizing: border-box; margin-top: 0px !important; margin-bottom: 16px; color: rgb(230, 237, 243); font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", "Noto Sans", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji"; font-size: 14px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: normal; background-color: rgb(13, 17, 23); text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial;">Bumps the angular-dependencies group in /client with 14 updates:

Package | Update
-- | --
@angular-material-components/datetime-picker | 8.0.0 to 16.0.0
@angular/animations | 14.1.3 to 16.1.7
@angular/cdk | 14.1.3 to 15.1.6
@angular/common | 14.1.3 to 16.1.7
@angular/compiler | 14.1.3 to 16.1.7
@angular/core | 14.1.3 to 16.1.7
@angular/forms | 14.1.3 to 16.1.7
@angular/material | 14.1.3 to 15.1.6
@angular/platform-browser | 14.1.3 to 16.1.7
@angular/platform-browser-dynamic | 14.1.3 to 16.1.7
@angular/router | 14.1.3 to 16.1.7
@angular-devkit/build-angular | 14.2.12 to 16.1.6
@angular/cli | 14.2.12 to 15.1.6
@angular/compiler-cli | 14.1.3 to 16.1.7

Bumps the angular-dependencies group in /client with 14 updates:

Package	Update
[@angular-material-components/datetime-picker](https://github.com/h2qutc/angular-material-components)	8.0.0 to 16.0.0
[@angular/animations](https://github.com/angular/angular/tree/HEAD/packages/animations)	14.1.3 to 16.1.7
[@angular/cdk](https://github.com/angular/components)	14.1.3 to 15.1.6
[@angular/common](https://github.com/angular/angular/tree/HEAD/packages/common)	14.1.3 to 16.1.7
[@angular/compiler](https://github.com/angular/angular/tree/HEAD/packages/compiler)	14.1.3 to 16.1.7
[@angular/core](https://github.com/angular/angular/tree/HEAD/packages/core)	14.1.3 to 16.1.7
[@angular/forms](https://github.com/angular/angular/tree/HEAD/packages/forms)	14.1.3 to 16.1.7
[@angular/material](https://github.com/angular/components)	14.1.3 to 15.1.6
[@angular/platform-browser](https://github.com/angular/angular/tree/HEAD/packages/platform-browser)	14.1.3 to 16.1.7
[@angular/platform-browser-dynamic](https://github.com/angular/angular/tree/HEAD/packages/platform-browser-dynamic)	14.1.3 to 16.1.7
[@angular/router](https://github.com/angular/angular/tree/HEAD/packages/router)	14.1.3 to 16.1.7
[@angular-devkit/build-angular](https://github.com/angular/angular-cli)	14.2.12 to 16.1.6
[@angular/cli](https://github.com/angular/angular-cli)	14.2.12 to 15.1.6
[@angular/compiler-cli](https://github.com/angular/angular/tree/HEAD/packages/compiler-cli)	14.1.3 to 16.1.7

What you expected to see, versus what you actually saw
All versions that contain a 16.x.x package to be selected (see @angular/cdk, material, cli that are moved to 15.x.x)
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

I tried to reproduce this but it worked for me: https://github.com/dsp-testing/angular-grouped-update/pull/5
There's a chance there's an ignore rule interfering. When a PR is closed Dependabot automatically adds an ignore. Would you mind providing the Job ID? It appears in the logs as something like job_702189558. I can use that to look up any ignores that might be lurking.

Where should I look up the job id?

 Or I can use a command to remove them on my side, let me know if you'd like me to do that.

You are right - this is now resolved. Reopening the PRs cleared those ignores. Is there a way to view those lists without going over every closed dependabot PR?

Not yet. We're testing out some new functionality which will make the process of unignoring much simpler, and make existing ignore conditions more visible. Once that ships this experience will be much better.


Why do addresses become so frequent?

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Sarama
Kafka
Go

Configuration
What configuration values are you using for Sarama and Kafka?

Logs
When filing an issue please provide logs from Sarama and Kafka if at all
possible. You can set sarama.Logger to a log.Logger to capture Sarama debug
output.
logs: CLICK ME

Problem Description

https://github.com/Shopify/sarama
https://github.com/markrcote/sarama
https://github.com/IBM/sarama

Shopify handed over this project to IBM: https://github.com/IBM/sarama/issues/2461
Therefor the addresses changed.


[Infineon] Lock-app updates, adding multiple users and credentials for CYW30739

[Infineon] Lock-app updates, adding multiple users and credentials for CYW30739
* Fix mCredentials, fix NVM flash saving

* restyle

* cleanup

* remove unused array

* remove TotalCredentials key

* Check mMaxUsers against DOOR_LOCK_MAX_USERS

Fast tracking platform changes, pending comments.


Add support for tiny display

@bryantpwhite Here's some improvements to the script I gave you

This looks great!


Error: swapChain ->GetDevice

Haven't seen this issue anywhere else. Whenever I open the overlay, it says it injects successfully but a different error pops up instead. I have the overlay extracted and VC++ updated, so I'm not sure what's causing the issue. Opening the overlay before/after tekken itself and running it in or out of admin mode also don't make any difference.

Probably fixed in more recent updates?


Support for splatting with commands

Is this:

[ ] feature
[X] bug:

System Details
This should be supported in PS2.0 and above when splatting was added.
Steps to Reproduce

Import the dbatools module
Create a hash of any number of parameters for a given command:

$params = @{ SqlInstance = 'manatarms'; Database='pubs' }
#or just
$params = @{ Database='pubs' }

Run the command Get-DbaDatabase

Get-DbaDatabase $params
# or on second hash
Get-DbaDatabase -SqlInstance MyServer $params

Both of the above options work with native commands in PowerShell. It would be good to be able to support this because splatting supports more gracefully handling variables/parameters being passed in with no value.
As well, support for splatting would allow us to handle variables not being passed in to whatever wrapper script we are building around dbatools.

Heya, hate to break it to you, but splatting works like this:
$params = @{ SqlInstance = 'manatarms'; Database='pubs' }
Get-DbaDatabase @params

Don't miss the @ :)

Oh dang, I totally missed that! I just saw a 'sea of blood' ;)


Guardian: Setup UI race condition verifying hashes

If a guardian receives all config hashes from "verify your guardian" screen, doesn't copy and share their config hash, and hits next:

Then they will arrive at this page with no ability to get their config hash and share it with all other guardians. Rest of the guardians can't finish setup.

Duplicate of #14. Some notes on possible fixes are in there.


Introduce  original_system, original_exec, unbundled_system, and unbundled_exec

What was the end-user problem that led to this PR?
The problem was that the clean_system and clean_exec would print deprecation messages, but there was not alternative for them.
What was your diagnosis of the problem?
My diagnosis was that the helpers are using deprecated behavior and that we should provide non deprecated alternatives.
What is your fix for the problem, implemented in this PR?
My fix is to introduce original_system, original_exec, unbundled_system, and unbundled_exec for consistency with the rest of the helpers, and deprecate clean_system and clean_exec.
Why did you choose this fix out of the possible options?
I chose this fix because while maybe not super pretty names, they offer an alternative consistent with the other helpers.

@bundlerbot r+


dom/is-visible fails focusable children of <canvas>

The <canvas> element may have children for compatibility. Those children, while not rendered, can be focused and tabbed to. But because they're not rendered, they do not necessarily have dimension and are thus considered hidden by dom/is-visible.

fixed in master, since is/visible dropped the deeply flawed dimensions check


Set game, title and tags from c2

What should be added?

Chatterino/chatterino2#1300
Chatterino/chatterino2#2315
Why should it be added?

[x] /settitle
[ ] /setgame
[ ] /settags


Add function vcpkg_configure_make/vcpkg_build_make/vcpkg_install_make/vcpkg_build_nmake/vcpkg_install_nmake

For Windows:

support make using yasm
support nmake to handle makefile will support this in the future.
support nmake to handle makefile.vc

For UNIX:

support make.

Progress:

[X] add vcpkg_configure_make
[X] add vcpkg_build_make
[x] add vcpkg_install_make
[X] add vcpkg_build_nmake
[x] add vcpkg_install_nmake
[x] test in windows(make)
test in windows(nmake makefile) will support this in the future.
[x] test in windows(nmake makefile.vc)
[x] test in linux
[x] update documentation

Add an option to use jom instead of nmake? (I should not be the person to write this since I always have problems with jom on my main pc but whatever....)

@Neumann-A I considered jom, but we already have vcpkg_build_qmake.cmake, so if it is reasonable, I will add it in the future.

looking ahead, you can see an example of the assembly
for example TCL
https://github.com/tcltk/tcl

Blocking in make(linux), seems to require passing target/host /buildin./configure`.
Can anyone help me?

I suddenly realized that mixing configure/makefile with makefile.vc is a bad idea. Because using nmake in windows to handle makefile and makefile.vc will appear ambiguous.
Maybe we should separate vcpkg_build_makefile/vcpkg_install_makefile.
This is indeed over-engineered.

are radical changes planned? or can stamp ports)))
https://github.com/krb5/krb5/tree/master/src/windows

@Voskrese There are fewer ports using make or nmake, so I plan to update these ports later.

Finally, all the work is done (except for the nmake makefile).

I just merged a TCL port, I guess I should have preferred this one...

@cbezault #8402 is upgrading this.

May I ask you what’s the meaning of

support make using yasm
Is yasm supporting makefiles now in a more compatible windows way than nmake? I don’t know. I am wondering because that would be great news?!?
Thanks :)

@cbezault
why isn't the merger happening?
either close or merge and so on with the rest, many edits are simply outdated

Pull requests [138]

@voskrese, I've moved on from this project in an official capacity. I'm just finishing up one more feature which is going to be useful for the ci. I'm also going to be in charge of the macos infrastructure for the foreseeable future. I'm just responding to/working on PRs I'm interested outside of work. @dan-shaw, @grdowns and @vicroms can still help you.

Well, I think this PR is ready.


Find parameters which are specified in the LightningDataModule, only

What does this PR do?
Fixes #4226
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?
[x] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

thanks for the fix, don't forget to update the changelog :)


Delete Jenkins lock directories

The Jenkins init script tries to delete directories as files. This commit lets it delete directories. See #1586

/cc @lachie83 @viglesiasce

Closing in favor of #2747


Bot secure against fraud ?

Hello,
I would like to know if your bot is secure against fraud when trading on CS GO items.
Let me explain: a few days ago I set up a trading bot CS GO developed in Node js with an external API that checked the prices of all items.  The taken from this item was handled on the Steam market and instead of being sold at $ 0.50, it went up to $ 4.
1 / Is your bot able to detect "jumping prices" and manipulation of prices on the Steam market during an exchange?
2 / Does your bot use the market price of Steam or an external API?
3 / If a user wants to exchange with your bot 30 times the same article, is it possible to manage the overstock and limit the quantity of each item during a trading?
Thanks a lot for your answers.

You would have to code all of the features you mentioned yourself.

This framework has no pricing features, you'll have to manage those yourself.  Also #427


Undeprecate the various throws: expectation/requirements

Description
Various methods were deprecated in https://github.com/swiftlang/swift-testing/pull/780. We maintain a number of libraries, usually for the last three Swift versions, so having test methods deprecated is a little tedious as we either need to live with the deprecation warnings or use #if compiler(...) in our tests to use the most appropriate API. It'd be great if the deprecated APIs weren't actually deprecated and lived for few more Swift versions.
Expected behavior
No response
Actual behavior
No response
Steps to reproduce
No response
swift-testing version/commit hash
No response
Swift & OS version (output of swift --version ; uname -a)
No response

Should have been specified as deprecated: 100000 (API_TO_BE_DEPRECATED). My bad.

Will need cherry-pick to 6.1.


Discurses uses way too much RAM

I'm unable to use the program as my ssh host only has 2GB RAM and the kernel always kills discurses every time I try to run it:
[6429792.083077] Out of memory: Kill process 21135 (discurses) score 398 or sacrifice child
[6429792.090599] Killed process 21135 (discurses) total-vm:1115996kB, anon-rss:796892kB, file-rss:0kB, shmem-rss:0kB

This is exactly the reason I cannot use the website version either... IMO 800+ megabytes for a text chat client is unacceptable.
Is there anything that can be done?

Idk, it's not really under development anymore. Try one that isn't written in python maybe...

Oh really? Hmm, it seems most/all of the nice looking ones are now abandoned. Oh well...


Add a GET / route to backend

AWS Load balancer can use this route to check backend server health.
TN: U507-032

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


Add tests for saving models as and loading from .safetensors files

Fixes #158.
I added the .safetensors test into assert_loads_correctly, because it was easy and because it means that we'll test many parameter configurations.

SCUNet: Saving models work, but loading them doesn't because of a bug in safetensors (https://github.com/huggingface/safetensors/issues/442).


chore: bump deps

Local Tableland needed a bump; also closes https://github.com/tablelandnetwork/hardhat-tableland/pull/1, https://github.com/tablelandnetwork/hardhat-tableland/pull/2, https://github.com/tablelandnetwork/hardhat-tableland/pull/25, https://github.com/tablelandnetwork/hardhat-tableland/pull/26, https://github.com/tablelandnetwork/hardhat-tableland/pull/32 (manually updated vs. merging them).

Note: turns out this is a bug that causes contract deployments to fail, specifically, for Hardhat TS projects (but Hardhat JS projects are somehow unaffected).


An example notebook for plotting with marsilea

This PR is for scverse/scanpy#2444
Thanks for your interest in Marsilea. I'm not sure if my explanation in the notebook is simple enough. Feel free to modify and comment.

cc: @grst @flying-sheep

This may also be more of an API thing, but in the dotplot the sizes seem to provided as counts, but the legend says they are fractions. Should: size = counts / cell_counts? Or is some form of normalization implicit? It's unclear to me how the plot would know what to normalize by.

This may also be more of an API thing, but in the dotplot the sizes seem to provided as counts, but the legend says they are fractions. Should: size = counts / cell_counts? Or is some form of normalization implicit? It's unclear to me how the plot would know what to normalize by.

Yes, it should be divided by cell counts. I was just copying the legend title of Scanpy's dot plot. There is no normalization in the plot, it simply just maps the data values to the size range of the circle marker. I will also change this part in the notebook.

I would also suggest:
agg = sc.get.aggregate(pbmc[:, markers], by="louvain", func=["mean", "count_nonzero"])
...
m = ma.Heatmap(
    agg.to_df(layer="mean"),  # Or agg.layers["mean"]
    height=h / 3,
    width=w / 3,
    cmap="Blues",
    linewidth=0.5,
    linecolor=".8",
    label="Expression",
)
...
m = ma.SizedHeatmap(
    size=agg.to_df(layer="count_nonzero") / adata.obs["louvain"].value_counts(),
    color=agg.to_df(layer="mean"),
    cluster_data=agg.to_df(layer="count_nonzero"),
    height=h / 3,
    width=w / 3,
    edgecolor=".8",
    cmap="Blues",
    size_legend_kws=dict(
        colors="#538bbf",
        title="Fraction of cells\nin groups (%)",
        labels=["20%", "40%", "60%", "80%", "100%"],
        show_at=[0.2, 0.4, 0.6, 0.8, 1.0],
    ),
    color_legend_kws=dict(title="Mean expression\nin group"),
)

Which would remove some of the code needed above.

@Mr-Milk please let me know when I should take another look here! I've bumped the build over on scanpy tutorials

@ivirshup, I'm happy with the current notebook. Could you take a look?
For tracks plot and stacked violin, I can implement and include them in Marsilea in the next minor release.

Looks pretty good! I may make some minor changes, but otherwise good. What should your name look like in the release notes?

Sure, that's good for me!

Merged via https://github.com/scverse/scanpy-tutorials/pull/98
Had to make a separate PR since I can't push to Mr-Milk:main


Issue with print function

First of all - awesome project!
I've been playing with your demo and have been having issues with the print function. When I attempt to use it I get this error:
JIT session error: Symbols not found: [ printf ]
Failed to materialize symbols: { (main, { main }) }

Everything else seems to work great. Any ideas what might be going on? Thanks!

This PR attempts to correct the issue.

I'm surprised that the linker didn't handle printf by itself. Thanks for the PR!


opt parallel serialize fragment and add detail schedule profile

Proposed changes

Add detail schedule profile
-  Schedule  Time:  652ms
   -  Fragment  Assign  Time:  57ms
   -  Fragment  Serialize  Time:  70ms
   -  Fragment  RPC  Phase1  Time:  522ms
   -  Fragment  RPC  Phase2  Time:  3ms
   -  Fragment  Compressed  Size:  6.82  MB
   -  Fragment  RPC  Count:  2
-  Wait  and  Fetch  Result  Time:  N/A
   -  Fetch  Result  Time:  0ms
   -  Write  Result  Time:  0ms

Fragment Serialize Time: Time to serialized the fragment thrift
Fragment RPC Phase1 Time: Time to send first fragment RPC
Fragment RPC Phase1 Time: Time to send second fragment RPC
Fragment Compressed Size: Total serialized size for fragment in bytes
Fragment RPC Count: Number of RPC

Separate the serialization of RPC send step
In previous, Doris will serialize and send fragment to each BE
one by one. If fragment is large, it is costy.
In this PR, I separate these 2 steps by first serializing the
fragment in parallel, and than send it one by one.

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

run buildall

run buildall

run buildall

run buildall


username and password to access the server (for the volume). I need to edit a file in a volume

Hi!
I need to edit some values in a config file, which is saved on a volume (On a specific Hetzner cloud server)
what is the way to do this?
When I logon to Hetzner cloud, volumes and click on the cloud server and 'Console', it is asking for an username and password.
These are not the Hetzner cloud credentails.

a reply from their support
As mentioned the login for a cloud server is always "root".

If you created the ssh key after creating the server it is not deposited on the remote system. This means you have to reset the password of your server.

If you have created the server and selected the ssh key during initial setup you can login without a password.

does your script generate the ssh key before or after the creating of the server?
anyway, when I try some combinations (username Root and or without password), none of them works
What am I doing wrong?
Can I reset the password without any problems? the server should just keep working from the cluster

Hi, when you create a cluster you specify your public SSH key in the config file for the cluster. You can use the corresponding private key with the root user to SSH into the instances. The installer doesn't create a new SSH key, it just uses what you specify. You shouldn't need the password. Try with
ssh -i <path of private key> root@<ip address>

But, after re-reading, I am not sure of what you are trying to do exactly. Can you clarify?

Ive applied a nextcloud deployment and it is using the hetzner storage class to put application files on a hetzner volume for persistent storage.
I need to edit a file on the volume
But i dont know how to do that the quickest way

Is the volume attached and mounted to a pod? If yes you can just exec into the pod to access the volume. If it's not attached to a pod you could run a temporary pod with Ubuntu image or something and mount the volume there.

Hi. Yes the volume is in use by the pod.
I can use the exec command to put files on the volume but not view the file right? On my notepad for instance
So i need to view a file put a some lines in it and save it agian
Thats not possible with exec i thought

It depends on the image used for the container in the pod. If it's running as root you may be able to install and run vim or something and edit the file that way.

thanks Vito!


Transforming directory pathh adds extra slashes "\"

I am suing Magic chunks. In my master json file I have one key with value
"ExternalFiles": {
"EncryptionKeyFile": "C:\TotalKeys\key.config.json"
}
In my transformation file transforming it to I have this
"ExternalFiles/EncryptionKeyFile": "C:\TotalKeys\keytest.config.json"
When it transform result is having some extra slashes, dont know why:-
"ExternalFiles": {

}
Any Idea why there are extra "\". it got 4 instead of 2.
Any one aware of any solution
Thanks

@sourcejon, @sunilbindra I wanted to have a look at this, but could not reproduce it.
Would one of you be in a position to provide a minimal example that reproduces the error?

Found it. This happens, when using the DevOps Task and "correctly" escaping \ in json of the transformation.
I.e. when the task is defined like:
- task: MagicChunks@2
  inputs:
    sourcePath: 'test.json'
    fileType: 'Auto'
    targetPathType: 'source'
    transformationType: 'json'
    transformations: |
      {
        "ExternalFiles/EncryptionKeyFile":
              "C:\\TotalKeys\\keytest.config.json"
      }

(notice the double \\ in the transformations) - the error occurs.
However, if the transformation-json is defined ("wrongly") as
- task: MagicChunks@2
  inputs:
    sourcePath: 'test.json'
    fileType: 'Auto'
    targetPathType: 'source'
    transformationType: 'json'
    transformations: |
      {
        "ExternalFiles/EncryptionKeyFile":
              "C:\TotalKeys\keytest.config.json"
      }

(notice the single \ in the transformations) - the error does not occur.
This behavior is the same, regardless whether one is using the "Inline JSON" transformation type or the "JSON File" type.


Specialty: Reinforce action shows Rapid Synthesis II Tooltip

In some situations withing the Simulator the Specialty: Reinforce action shows Rapid Synthesis II Tooltip
(seem to be doing so only on CRP)

Thanks for spotting that! Fixed on beta site. Please force reload.

All good now. Thanks for a prompt fix :)


feat: If the container exists, no pull is performed to speed up program execution

At present, the program will try to download the latest version of the execution image every time it is started. Since we will definitely execute the image parameters during execution, we can directly start the program after checking the existence of the image without repeating the download.

continue in this pr: https://github.com/golang/playground/pull/18


update resources.md

Icons list reorder to alphabetic order
Circum icons library added to the list

Hi @ZoranPandovski can you give it a look at this PR please! Thank you

@Klarr-Agency maybe we can do the same for all items in the list

@ZoranPandovski Thank you Zoran :) Yes, it would nice to do the same for the other lists. I will try to take time for that this week.


Use .jlt instead of .jl for template files

This patch changes the file extension of Julia template files from .jl to .jlt. The motivation for this is tooling that assume these are (valid) julia source files based on the .jl extension.

I think this makes sense.
Most syntax highlighting tools etc (that are ok with broken code) still are not that useful  on the template files.
And the language can always be set manually.
Do you have merge rights?

This CI error is real:
https://github.com/JuliaCI/PkgTemplates.jl/actions/runs/10521345275/job/29151850911?pr=481#step:7:137

This CI error is real:

Fixed, didn't show up locally (on the same Julia version).

Do you have merge rights?

Looks like it.

LGTM
Probably bump as a nonbreaking change so you can tag a release after you merge


Compile failure on M1 Mac: Missing OpenMP

Trying to compile from source on Mac OS X Ventura (M1), getting this error:
❯ make loadable
cmake -B build; make -C build
CMake Error at /opt/homebrew/Cellar/cmake/3.26.4/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)
Call Stack (most recent call first):
  /opt/homebrew/Cellar/cmake/3.26.4/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /opt/homebrew/Cellar/cmake/3.26.4/share/cmake/Modules/FindOpenMP.cmake:577 (find_package_handle_standard_args)
  vendor/faiss/faiss/CMakeLists.txt:253 (find_package)

-- Configuring incomplete, errors occurred!
make[1]: *** No targets specified and no makefile found.  Stop.
make: *** [dist/debug/vector0.dylib] Error 2

Any ideas? I've tried installing libomp using homebrew but that didn't seem to help.

I can build faiss in ./vendor/faiss manually and it works using this command:
LDFLAGS="-L/opt/homebrew/opt/llvm/lib" CPPFLAGS="-I/opt/homebrew/opt/llvm/include" CXX=/opt/homebrew/opt/llvm/bin/clang++ CC=/opt/homebrew/opt/llvm/bin/clang cmake -DFAISS_ENABLE_GPU=OFF -DFAISS_ENABLE_PYTHON=OFF -B build .```

When I try to specify those same environment variables in for `make loadable`, it doesn't work.

Was able to get it to compile, but had to compile faiss separately (and comment out out the add_subdirectory call).
I also had to change this line to omit the avx2 version of the faiss library:
https://github.com/asg017/sqlite-vss/blob/2d0dd01d3dfaf76a1bd4fe7225c4d71c8d011977/CMakeLists.txt#L52
I don't know anything about CMake, but it seems like hard coding an Intel-specific library name would cause problems on ARM macs?

Thanks for the help! I'll try those env vars little later today.
I did try downloading the pre-compiled version, but the archive only contained vector0.dylib, it was missing vss0.dylib.

The latest "alpha' releases should have both vector0.dylib and vss0.dylib, you can try here with sqlite-vss-v0.1.1-alpha.13-loadable-macos-aarch64.tar.gz https://github.com/asg017/sqlite-vss/releases/tag/v0.1.1-alpha.13
I'll be publishing v0.1.1 soon


Response Scoring

Similar to how we have a "did this help" message for the KB, response scoring adds a similar functionality to resolved topics. An email should be sent to the user asking them to rate the quality of their interaction. It should also be shown on the web UI.
In addition, these things should be logged to the database and reporting generated.

Added to commercial version


cinny-desktop: 2.2.6 -> 3.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cinnyapp/cinny-desktop/releases.
meta.description for cinny-desktop is: Yet another matrix client for desktop
meta.homepage for cinny-desktop is: https://github.com/cinnyapp/cinny-desktop
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.0 with grep in /nix/store/n8ixbjjmx8rckl7n7jp56di93kdfzfm9-cinny-desktop-3.0.0
found 3.0.0 in filename of file in /nix/store/n8ixbjjmx8rckl7n7jp56di93kdfzfm9-cinny-desktop-3.0.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cinny-desktop

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n8ixbjjmx8rckl7n7jp56di93kdfzfm9-cinny-desktop-3.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/0852b76df1fbaa457cdefffc2fa9a1fde0ab148a#cinny-desktop

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

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

1 package built:

cinny-desktop

Maintainer pings
cc @aveltras for testing.

seems we'll need to handle missing libraries
/nix/store/n8ixbjjmx8rckl7n7jp56di93kdfzfm9-cinny-desktop-3.0.0/bin/cinny
thread 'main' panicked at 'Failed to load ayatana-appindicator3 or appindicator3 dynamic library
libayatana-appindicator3.so.1: cannot open shared object file: No such file or directory
libappindicator3.so.1: cannot open shared object file: No such file or directory
libayatana-appindicator3.so: cannot open shared object file: No such file or directory
libappindicator3.so: cannot open shared object file: No such file or directory', /build/cinny-desktop-3.0.0-vendor.tar.gz/libappindicator-sys/src/lib.rs:41:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

bump to 3.1.0 in #263790, with libappindicator added

Superseded by #263790.


add integration tests

This pull request adds Integration tests to the project.  The integration test suite is implemented in IntegrationTest.java
The test suite spins up an instance of a Wiremock server and a Zuul server.
The test suite sends HTTP requests to Zuul. Zuul sends the requests to the WireMock server.
Each test case uses WireMock verifications to confirm the request has been delivered.

WDYT? @argha-c


🔧 entity index

close #97
close #96

today I learned blocknumber isn't unique by collectionEnitity

today I learned blocknumber isn't unique by collectionEnitity

Yup you can create more collections in one block


Implement census tract level feedback links

@KameronKerger to insert designs

send_feedback.pdf
contact_census_tract.pdf
email.pdf