invoke_device_method doesn't work in the sample called: iothub_registry_manager_method_sample

Context

OS and version used: Windows 10
Python version: 3.9.5
pip version: 21.1.2
list of installed packages: azure-core 1.14.0, azure-iot-device 2.6.0, azure-iot-hub 2.4.0
cloned repo: master

Description of the issue
When I try to execute the example called "iothub_registry_manager_method_sample" on my own environment, it raises an exception that doesn't allow me to receive the correct response.
Code sample exhibiting the issue
iothub_registry_manager_method_sample
Console log of the issue
Unexpected error JSON is invalid: Expecting ',' delimiter: line 1 column 48 (char 47)

Hi @FelixAngelMartinez, can you send what the device's response to the method request is when this occurs?

It works on my device, but my backend shows me that error. I have to control that exception with a try and catch operation to avoid app crash.
The problem is in the library, more specifically in the "devices_operations.py", in the method "invoke_method" and in the part:
"deserialized = self._deserialize("CloudToDeviceMethodResult", response)".

When a device receives a method request, it is supposed to send back a response. This is what I meant by "device's response to the method request". Apologies if that wasn't clear. Could you send an example of a response payload sent by the device that causes this error?

That's rigth. I've checked and the fault is in my code.
In my ESP32 device, more specifically in my c++ code I have the following line to manage the response:
const char *responseMessage =  R"({"foo":"bar"})";
And if I debug the Python SDK call, I receive the following:
<class 'requests.models.Response'>
{"status":200,"payload":{"foo":"bar"}}
Unexpected error JSON is invalid: Expecting ',' delimiter: line 1 column 38 (char 37)
I've tried to leave it empty, to change it several times in different ways, but I still don't know how to fix it.
If you have a quick solution I really appreciate it, if not, don't worry.
Thank you very much.

Hi @FelixAngelMartinez , sorry for the delayed response. Can you send a simplified C++ example of the device code that triggers this error?

Closing due to inactivity. @FelixAngelMartinez , feel free to reopen it you have an update.


Documentation for getting ML models input and output

Hi,
I am very new to Apollo and want to work on Machine Learning model testing. I am able to successfully build and run Apollo dreamview using quickstart demo. However I am unable to understand how can I do module to module testing and in general getting Machine Learning model inputs and outputs and/or inputs and outputs at each step. Could you help me with how can I log the inputs and outputs before and after each model inference. Any documentation would be helpful. I tried using cyber_monitor but it just gives result for a particular module.
I want to:

Get instructions on running each module and each model independently.
Get the structure of input and output using logging.

I also tried using ADEBUG in the code and then run the recording using cyber_recorder play -f demo_3.5.record --loop
Then I checked the /apollo/data/logs folder however no logs for the module were found where I used ADEBUG.
Any documentation regarding this would be helpful. Thanks a lot.

If you want to implement the functions of openai, usually you need to use it in conjunction with the simulator.
If you want to debug the module, you can use the log AINFO or AERROR, If you want to use ADEBUG, you should open the
https://github.com/ApolloAuto/apollo/blob/463fb82f9e979d02dcb25044e60931293ab2dba0/cyber/setup.bash#L41-L42
then enable it use, you can found the logs in data\log

source cyber/setup.bash


JDK 10 is no longer supported on linux

##[warning]JDK 9 and JDK 10 are out of support. Please switch to a later version in your project and pipeline. Attempting to build with JDK 11...

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

ID: 7336a04e-5b38-d02f-8bba-a55445b2f094
Version Independent ID: 783c30b3-63b6-da7c-1336-b000bb8b4c66
Content: Create your first pipeline - Azure Pipelines
Content Source: docs/pipelines/create-first-pipeline.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

Closing because I've created an issue in the sample repo for this. https://github.com/MicrosoftDocs/pipelines-java/issues/89


Effiecient way to upload from a stream without specifying length

Query/Question
Hello. We are looking for an efficient way to upload data rom an inputSteam when content length is unknown.
Is this feature implemented already ? If yes, could you please provide sample code. Thanks!

Hi @dhatri308
You may use:
BlobClient.upload(InputStream)
BlobClient.upload(InputStream, boolean)
and
BlobClient.uploadWithResponse(BlobParallelUploadOptions, Duration, Context)
These all don't require you to pass in a length for the input stream.


Upload Task Id format

Hi,
Thank you for this wonderful package.
I noticed that the upload task id format is inconsistent in android and ios. While android gives us just the UUID, iOS has the package name also attached. My local database column had a length set only to accommodate the UUID so it was failing to insert.
I am using uploader.enqueue(upload);
Android: 405bc0ff-25b8-44ac-b724-26adccc7e7c6
iOS: chillisource.flutter_uploader.upload.background.869B1C87-9766-499E-9B87-E4D80E3BB3F4
Is this expected?

Kind of expected. On iOS the session identifier is part of the taskId.
This might be leftover from when the taskId was not generated as a UUID.

Makes sense. Thanks!


[SPARK-31715][SQL][TEST] Fix flaky SparkSQLEnvSuite that sometimes varies single derby instance standard

What changes were proposed in this pull request?
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/122622/testReport/junit/org.apache.spark.sql.hive.thriftserver/SparkSQLEnvSuite/SPARK_29604_external_listeners_should_be_initialized_with_Spark_classloader/history/?start=25
According to the test report history of SparkSQLEnvSuite，this test fails frequently which is caused by single derby restriction.
Caused by: sbt.ForkMain$ForkError: org.apache.derby.iapi.error.StandardException: Another instance of Derby may have already booted the database /home/jenkins/workspace/SparkPullRequestBuilder/sql/hive-thriftserver/metastore_db.
	at org.apache.derby.iapi.error.StandardException.newException(Unknown Source)
	at org.apache.derby.impl.store.raw.data.BaseDataFileFactory.privGetJBMSLockOnDB(Unknown Source)
	at org.apache.derby.impl.store.raw.data.BaseDataFileFactory.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at org.apache.derby.impl.store.raw.data.BaseDataFileFactory.getJBMSLockOnDB(Unknown Source)
	at org.apache.derby.impl.store.raw.data.BaseDataFileFactory.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.TopService.bootModule(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.startModule(Unknown Source)
	at org.apache.derby.iapi.services.monitor.Monitor.bootServiceModule(Unknown Source)
	at org.apache.derby.impl.store.raw.RawStore.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.TopService.bootModule(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.startModule(Unknown Source)
	at org.apache.derby.iapi.services.monitor.Monitor.bootServiceModule(Unknown Source)
	at org.apache.derby.impl.store.access.RAMAccessManager.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.TopService.bootModule(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.startModule(Unknown Source)
	at org.apache.derby.iapi.services.monitor.Monitor.bootServiceModule(Unknown Source)
	at org.apache.derby.impl.db.BasicDatabase.bootStore(Unknown Source)
	at org.apache.derby.impl.db.BasicDatabase.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.boot(Unknown Source)
	at org.apache.derby.impl.services.monitor.TopService.bootModule(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.bootService(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.startProviderService(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.findProviderAndStartService(Unknown Source)
	at org.apache.derby.impl.services.monitor.BaseMonitor.startPersistentService(Unknown Source)
	at org.apache.derby.iapi.services.monitor.Monitor.startPersistentService(Unknown Source)
	... 138 more

This PR adds a separate directory to locate the metastore_db for this test which runs in a dedicated JVM.
Besides, diable the UI for the potential race on spark.ui.port which may also let the test case become flaky.
Why are the changes needed?
test fix
Does this PR introduce any user-facing change?
NO
How was this patch tested?
SparkSQLEnvSuite itself.

Test build #122645 has started for PR 28537 at commit 31d032e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27307/
Test PASSed.

Test build #122645 has finished for PR 28537 at commit 31d032e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122645/
Test FAILed.

Test build #122653 has started for PR 28537 at commit 97597ac.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27315/
Test PASSed.

cc @cloud-fan @HeartSaVioR thanks

Test build #122653 has finished for PR 28537 at commit 97597ac.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122653/
Test PASSed.

thanks, merging to master/3.0!

Late +1.


Caddy seems to not forward the authorization headers

I recently upgraded to Caddy 0.9.5 from 0.9.3 and I notice an odd breakage: Caddy's proxy directive doesn't forward the Authorization header any more. I am doing basic auth on caddy, but also relying on the proxied server getting that authorization, but this broke after the upgrade.
Here's my config:
hostname.com {
    gzip
    log /var/log/caddy/hostname.log
    header / Strict-Transport-Security "max-age=31536000; includeSubdomains";
    basicauth / user1 htpasswd=../../../../etc/caddy/htpasswd
    basicauth / user2 htpasswd=../../../../etc/caddy/htpasswd
    basicauth / user3 htpasswd=../../../../etc/caddy/htpasswd
    proxy / http://localhost:5232/ {
        transparent
    }
}

As I suspected, this was removed in #1325 because double authentication was deemed useless. I'm afraid that was misguided, as my use case is not that I'm doing authentication twice, but that I'm reusing the username of the first authentication for the second.
I like having all my auth done by Caddy, because I can then have a single authentication source, but, since a user already authenticated, I would like to reuse this authentication upstream. However, the change in #1325 breaks this workflow, as it doesn't allow me to use header_upstream to add either the {>Authorization} header or the {remote_user}, thus completely removing my upstream's knowledge of who authenticated.
This is much worse than "double-dipping", because it means that now I must legitimately perform authentication twice, which is very inconvenient for everyone. Given that I don't even have the passwords from my users in a format that I can add to the upstream service, I'm forced to tell everyone that they just can't authenticate to the service any more.

Also, even if you disagree with this, there's no rationale in completely removing the Authorization header so I can't use it (or remote_user) in header_upstream to work around this issue.

I took out the Authorization header upon successful authentication because of https://github.com/mholt/caddy/issues/1324 - but the username is now stored on the context of the request: https://github.com/mholt/caddy/commit/977a3c322631b41b534f079e2466689f9aa3d3f7
Rather than passing the password upstream when it's not needed, and if all you need is the username, we could add a {remote_user} placeholder. Then you could set a header going upstream with that information in your proxy directive. Would that work?

Wasn't that added in #1426? Regardless, I don't think that will work with Radicale (which is what I'm using), because it only expects to parse the username from a custom header, and I doubt Caddy can base64-encode the placeholder :/
Can the Authorization header be removed after the proxy directive has been evaluated, so people can readd it with a header_upstream directive?

Do you have a moment to try something? Can you take out the line of code in basicauth that strips the header, and then use this in your proxy directive:
header_upstream -Authorization

And see if it does indeed prevent the Authorization header from being sent? (It should, I think.)
If so, I suppose I can revert my change, as it was more of a "this would be a good idea but isn't super-critical for security I guess" change...

Sure, let me try to build it and I'll report back. For what it's worth, I agree with you that people who don't trust their backend can just strip the header off (but why aren't people trusting their backends)?

Actually, can I just use an older version? I don't have Go 1.8 anywhere...

@skorokithakis Maybe... I know we've worked on the proxy/header code recently though. (I would test it but I'm on my way out the door for today.)

I tested header_upstream Authorization "", which properly cleared the header. I tried to test -Authorization but 0.9.1 choked, I'll try a more recent version.

0.9.4 doesn't remove it with -Authorization... I'll try building 0.9.5.

Alright, yes, I can confirm that removing that line and removing the auth header does what you'd expect (removes the auth header) with the latest master.

Excellent. In that case, I will remove the line in basicauth that strips the header, essentially reverting my change.

Great, thank you!

When will this feature be in a new release?


Rename-DbaDatabase, round #3

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Reissue of  https://github.com/sqlcollaborative/dbatools/pull/2442, which is a reissue of https://github.com/sqlcollaborative/dbatools/pull/2030 . Please merge ASAP, keeping tracks is starting to be extremely difficult.

Just curious if the -Preview output should look like this...It didn't actual do any changes, but the way that output appears it looks like it did unless you pay close attention to the output:

Shouldn't that be using WhatIf instead of having a parameter for it called -Preview?
Just one more small thing...if an operation requires setting the database offline and that fails I don't think it is best to keep going with the changes and leaving "low hanging fruit" like "You are in charge of moving the files to the new location".

I obviously don't know how to use this one 😝

I think overall it is a good first round but believe it is going to be too easy for folks to dork up their databases with those placeholders. We will likely need a good blog post or something to point folks at starting out. Then as we get more folks using it figure out how (or if we can) fine tune it.

I'm up for merging it but will leave that to @potatoqualitee ... I mean we have to start somewhere and just wait for feedback to come in on it.

yeah, apparently no better ideas came around during code review so we should hear from users pretty soon. The original requested feature-set is in. Lemme fix the license and try to figure out your issue, then I think it can be merged.

I'd love to merge! Then we'll ask for some testers.  @wsmelton - feel free to merge and ping me once you and niph are ready. then i'll add it to psd1

I would say we push this one out as a "beta" type thing, get folks using it in their test environments and then have them offer feedback/suggestions. Especially since we cannot test for every possible scenario of file configurations in a database.

on the "leave the low hanging fruit" theme, unfortunately it's kinda chicken/egg problem: nothing can be moved until the db is put offline (files are locked if db is online), and there's no way to pre-check if a move can be done. Maybe it should try to rollback the changes, but it's not that simple.
@wsmelton : the error shown seems to point out you can't Get-DbaFile -sqlinstance matanarms\sql11 -path 'c:\program files\microsoft sql server\mssql11.sql11\mssql\data' , can you confirm ?

Yesssss thanks @niphlod ! Adding this and alerting Rob Volk who originally requested it.


ERROR ITMS-90056: This bundle is invalid

Hi, Today I try to release my app with SwinjectAutoregistration in dependency (via Carthage) and I receive error like 'ERROR ITMS-90056: This bundle is invalid. The Info.plist file is missing the required key: CFBundleVersion'
In project I found that in CFBundleVersion used parameter $(CURRENT_PROJECT_VERSION). But 'Current Project Version' is not installed in the build settings
Fix please or explain how I should use this.
Thanks.

Thanks @threat70 for submitting the issue. We might need fix like SDWebImage did, though we still need some investigation in our case.

https://github.com/rs/SDWebImage/issues/1780
https://github.com/rs/SDWebImage/pull/1784/files

For submit the application I did the same in Carthage's checkouts folder

@yoichitgy Their fix is redundant, the setup of 'CURRENT_PROJECT_VERSION' is sufficient
diff --git a/SwinjectAutoregistration.xcodeproj/project.pbxproj b/SwinjectAutoregistration.xcodeproj/project.pbxproj
index 4a25656..0bedd36 100644
--- a/SwinjectAutoregistration.xcodeproj/project.pbxproj
+++ b/SwinjectAutoregistration.xcodeproj/project.pbxproj
@@ -1317,6 +1317,7 @@
                        isa = XCBuildConfiguration;
                        baseConfigurationReference = CD5B9E301D832A4F007ED05F /* iOS-Framework.xcconfig */;
                        buildSettings = {
+                               CURRENT_PROJECT_VERSION = 1;
                                INFOPLIST_FILE = "$(SRCROOT)/Sources/Info.plist";
                                PRODUCT_BUNDLE_IDENTIFIER = "com.swinject.SwinjectAutoregistration-iOS";
                        };
@@ -1326,6 +1327,7 @@
                        isa = XCBuildConfiguration;
                        baseConfigurationReference = CD5B9E301D832A4F007ED05F /* iOS-Framework.xcconfig */;
                        buildSettings = {
+                               CURRENT_PROJECT_VERSION = 1;
                                INFOPLIST_FILE = "$(SRCROOT)/Sources/Info.plist";
                                PRODUCT_BUNDLE_IDENTIFIER = "com.swinject.SwinjectAutoregistration-iOS";
                        };

Thanks @threat70👍 I made a pull request.


[Feature Request] Option to disable update checker.

Is your feature request related to a problem? Please describe.
[04:34:25] [Craft Scheduler Thread - 15 - WanderingTrades/INFO]: [WanderingTrades] This server is running a development build of WanderingTrades! (v1.6.7-SNAPSHOT+6d9d6b2)
[04:34:25] [Craft Scheduler Thread - 15 - WanderingTrades/INFO]: [WanderingTrades] The latest official release is v1.6.6

Describe the solution you'd like
A config option to disable the update checker.
Additional context
An option to disable the update checker so WanderingTrades isn't checking for a new version every initialization. Useful if you intentionally stay out of date due to an upstream bug or are testing newer commits. I always prefer to disable update checkers in plugins to reduce http calls & notification spam, as I'm always using the latest dev builds for testing

fe46e8a02f1f8f19003bda0d9649098fe3c63d07


Fixing big endian.

What does this PR do?

Many more tests, with numpy coverage.
Actually test values on disk.

Fixes #321

Fixes # (issue) or description of the problem this PR solves.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


elixir_1_11: init at 1.11.0

Motivation for this change
Set elixir_1_11 as default elixir version.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Hi @mtanzi. Cool thanks.
In 1.11.nix minimumOTPVersion could still be "21".
→ https://hexdocs.pm/elixir/compatibility-and-deprecations.html#compatibility-between-elixir-and-erlang-otp

Hi @mtanzi. Cool thanks.
In 1.11.nix minimumOTPVersion could still be "21".
→ https://hexdocs.pm/elixir/compatibility-and-deprecations.html#compatibility-between-elixir-and-erlang-otp

Thanks @cw789, well spotted! I will fix that 👍

Tested via nixpkgs-review pr 99883.
2 packages built:
elixir elixir_1_10

Execution of elixir --version is fine.
Erlang/OTP 22 [erts-10.7] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] [hipe]

Elixir 1.11.0 (compiled with Erlang/OTP 22)

@mtanzi @cw789 Where did this land? :smile:

@fritzblue I am hoping someone is going to merge it soon 😅 🤞

Hey @havvy @couchemar @ankhers @filalex77.
Is anyone available for review to speedup getting this PR merged?

Result of nixpkgs-review pr 99883 1

1 package built:

elixir (elixir_1_11)

@r-ryantm just openend #99883, which is probably a duplicate of this. Although it upgrades to 1.11.1 instead of 1.11.0, so that might be something you might also want to do here.

@ajs124 I've updated the PR with the latest elixir (v1.11.1) 👍

nixpkgs-review pr 99883 → build elixir (elixir_1_11) ✔
Erlang/OTP 22 [erts-10.7] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] [hipe]

Elixir 1.11.1 (compiled with Erlang/OTP 22)


[MAID-2925, 2926] Remove maintainer field and update doc attributes

r? @Fraser999
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Improve "Community" page

This is a Feature Request
What would you like to be added/enhanced?
There are some sections in Community page that i felt needs some sort of enhancement as below:-

Why is this needed

It will reduce space wastage and will give overall better look of that page.
Enhancement part is optional and if it looked good to reviewers/approvers/owners/maintainers, then it will be applied.

/area web-development

/retitle Improve "Community" page
/priority awaiting-more-evidence

/assign @Ayush9026

i can do this please assign me
If it still not solved.

@Lakshman76  i will solve this please wait if help needed i will ask you.

/triage accepted
Let's see what people can come up with.
Really bored to see this kind of complaints again and again.

/remove-priority awaiting-more-evidence

/priority backlog
All the other issues that we have triaged as accepted are more important than this change.


Nuget publishing task tries to upload .snupkg packages twice

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: NuGet
Environment
account name: trwalke,
team project name: Identity Division IDDP (https://identitydivision.visualstudio.com/IDDP)
build definition name/build number: .NET MSAL V3 Release/ release Id=13717

Agent - Hosted or Private:

Hosted agent on "Hosted VS2017"

Issue Description
We are trying to us the nuget task to push our nuget packages and symbols via the .snupkg. The task is able to successfully upload the .nupkg and the .snupkg packages. However, it tries to upload the .snupkg again after it has already been uploaded resulting in an error.
We are using nuget version <IP_ADDRESS>23 via the nuget tool installer.
We are simply pointing the nuget task to a folder that contains both .nupkg and the .snupkg packages
Task logs
ReleaseLogs_13717.zip
"Nuget Push" logs
Error logs

The nuget command failed with exit code(1) and error(Response status code does not indicate success: 409 (It looks like there is another copy of this symbols package pending validation(s). Please wait for the validation(s) to finish before trying to replace the symbols package.).)

The task documentation says we can use the "Index Sources & Publish Symbols task" for pushing symbols but that task doesnt seem to support .snupkgs.
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/package/nuget?view=azure-devops

any updates on this?

Apologies for the slow response. Please try excluding the .snupkg file using the search pattern in the task input (the field called "Path to NuGet package(s) to publish"). From the logs, it looks like you are setting it to "C:\Nuget*.*nupkg", which will also match .snupkg files; please try removing the second wildcard character. The .snupkg is already automatically pushed by nuget.exe when the .nupkg is pushed; this behavior is mentioned in the third item in these docs.
The task runs nuget.exe push on every file matched by the search pattern. In this case, what's happening is that it's matching both files, running nuget.exe push on the .nupkg (which will also upload the .snupkg), then running it on the .snupkg, and failing.


Start the 0.9.0 release, shipyard phase

Signed-off-by: Miguel Angel Ajo<EMAIL_ADDRESS>

@skitt  :pray:  :-)


fix: user jsonify to set response headers to json

Hello @ngoduykhanh
I tried to make a go client from your swagger spec with go-swagger tool. From the description, all returned data from the API is application/json, but headers were not set and defaulted to text/html.
The client raise an error because it expects json. I updated all json.dumps() to use Flask's jsonify() which correctly sets headers.
If there is any particular reason let me know.
Regards

Yes, using jsonify is better. Merging your PR. Thank you!

Yes, using jsonify is better. Merging your PR. Thank you!


Options for Engines

This is inspired by #2982. It adds an option object to engines, and adds an option for bypassing the filesystem mapping and checks done in view.js.
Alternatively, this option could be used to bypass the if (!view.path) { check in application.js:565, or even to assign view.path = name somewhere around there.
Either way, I think it'd be a good idea to have options for engines.
Put into the 5.0 branch because it's a breaking change.

Can you have different view engine options for different view engines? Or do all engines ends up with the same options?

Well, if no engine options are given, it uses app.get('view engine options') or {} as a fallback.
But I've added a parameter to app.engine which lets you pass options for a specific engine.

Ah. Can you describe this feature a bit more so I can understand it? We will also need to understand it in order to write up the documentation. Is there a use case for having many view engines use the same options?

Basically, this PR changes the following:

Store engines in objects instead of just storing the function. Might be handy in case any other data ever needs to be stored with it.
Add an options parameter to app.option that allows specifying options for the engine.
Add 'view engine options' setting for default engine parameter options.
Add the option "noFileSystem" to allow bypassing checks and calcs intended only for usage with filesystem engines.
Tests

In short, all this really does right now is add the functionality to bypass the filesystem-specific logic associated with engines, so non-filesystem engines (like ones using databases as described in the linked issue) are possible.
This is stored in a general "options" object now associated with engine instances, which can be set when using app.engine or the 'view engine options' setting. More options can thus be added trivially.
The refactoring from storing just the callback to storing an object is just to store the options object with it right now, but it might come in handy at some other point.
The default value setting probably won't see much use for the current option. I added it for consistency and maintainability reasons - I can remove it if you prefer though.

@dougwilson What's the status of this PR?


Finish "action done" notifications

Long story short, for autonomous we have to indicate when actions are completed. Currently this is rather non-existent in Sprocket.
I see two ways of doing this, but there could be more.

Add a return notifier that fires every time something is dequeued.

Create a VI to check if an element is still in the queue (in queue = incomplete, out of queue = complete).

Consider stealing code from C&C to do this.
@gcschmit @dchammond @AlexEastman.

If an action is dequeued, it isn't necessarily complete. Shouldn't the return notifier be fired when the action is complete? (I prefer the push mechanism of firing a notifier or the poll mechanism of checking the queue.)


Add disk_encryption_set_id support to vm module

Issue-id
PR Checklist

[ ] I have added example(s) inside the [./examples/] folder
[ ] I have added the example(s) to the integration test list for normal (~30 minutes) or long runner >30 minutes
[ ] I have checked the coding conventions as per the wiki
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

[ ] YES
[x] NO

Testing

thanks for the PR @onlyillusion!
Terraform is lazy at plan time, so with your statement, when you add a disk encryption set, even not related to the VM, you will get something like that:

We try to have statements that force plan-time calculation to avoid that, this statement is evaluated at plan time, so it doesn't create a know after apply:
disk_encryption_set_id = can(each.value.os_disk.disk_encryption_set_id) ? each.value.os_disk.disk_encryption_set_id : can(each.value.os_disk.disk_encryption_set_key) ? var.disk_encryption_sets[try(each.value.os_disk.lz_key, var.client_config.landingzone_key)][each.value.os_disk.disk_encryption_set_key].id : null

@onlyillusion would you be able to test if the statement proposed above works for you and if so please update the PR for going ahead!
thanks!

checked, everything is fine.
thanks for the advice
lz_key - ok
current level key - ok
id - ok


[bindgen] Failing due to older libstdc++.s0.6 on CentOS 7

Description
Looks like bazel is downloading the bindgen package for a newer version of the libstdc++
bash bazelisk.sh --output_user_root=/home/dbeitel/.opentitan build --jobs 8 //...
which: no bazel in (/home/tools/Xilinx/Vitis_HLS/2022.2/bin:/home/tools/Xilinx/Model_Composer/2022.2/bin:/home/tools/Xilinx/Vivado/2022.2/bin:/home/tools/Xilinx/DocNav:/tools/verible/v0.0-2135-gb534c1fe/bin:/tools/verilator/4.210/bin:/opt/rh/devtoolset-11/root/usr/bin:/usr/lib64/qt-3.3/bin:/home/dbeitel/perl5/bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin:/bin:/sbin:/home/dbeitel/.local/bin:/home/dbeitel/bin)
INFO: Build option --features has changed, discarding analysis cache.
WARNING:root:Closest bitstream to HEAD is 44c75fcc9cc4c4c4daa551d0614ae49b85e54cc4.
WARNING:root:No manifest found. Attempting to generate manifest from legacy file paths.
INFO: Analyzed 30714 targets (641 packages loaded, 94265 targets configured).
INFO: Found 30714 targets...
INFO: From Running Cargo build script libftdi1-sys:
Build Script Warning: Config for libftdi1 not found, falling back to default search path: PKG_CONFIG_ALLOW_SYSTEM_CFLAGS="1" PKG_CONFIG_ALLOW_SYSTEM_LIBS="1" "pkg-config" "--libs" "--cflags" "libftdi1" "libftdi1 >= 1.4" did not exit successfully: exit status: 1
INFO: From Running Cargo build script libftdi1-sys:
Build Script Warning: Config for libftdi1 not found, falling back to default search path: PKG_CONFIG_ALLOW_SYSTEM_CFLAGS="1" PKG_CONFIG_ALLOW_SYSTEM_LIBS="1" "pkg-config" "--libs" "--cflags" "libftdi1" "libftdi1 >= 1.4" did not exit successfully: exit status: 1
ERROR: /mnt/hgfs/dbeitel/workspace/com.rivos.opentitan-integrated/sw/host/opentitanlib/bindgen/BUILD:36:21: Generating bindings for sw/host/opentitanlib/bindgen/difs.h.. failed: (Exit 101): bindgen__bin failed: error executing command bazel-out/k8-opt-exec-2B5CBBC6/bin/external/rules_rust_bindgen__bindgen-0.60.1/bindgen__bin --no-rustfmt-bindings '--allowlist-var=CLKMGR_.BIT' '--allowlist-var=CLKMGR._REG_OFFSET' ... (remaining 48 arguments skipped)
Use --sandbox_debug to see verbose messages from the sandbox
thread 'main' panicked at 'Unable to find libclang: "the libclang shared library at<EMAIL_ADDRESS>could not be opened: /lib64/libstdc++.so.6: version GLIBCXX_3.4.22' not found (required by bazel-out/k8-opt-exec-2B5CBBC6/bin/_solib_k8/_U@bindgen_Uclang_Ulinux_S_S_Clibclang___Ulib/libclang.so)"', external/rules_rust_bindgen__bindgen-0.60.1/src/lib.rs:2172:31 stack backtrace:    0: rust_begin_unwind              at /rustc/c18a5e8a5b1afb0d7a582fe9ebad4c1996c90da3/library/std/src/panicking.rs:575:5    1: core::panicking::panic_fmt              at /rustc/c18a5e8a5b1afb0d7a582fe9ebad4c1996c90da3/library/core/src/panicking.rs:64:14    2: core::result::unwrap_failed              at /rustc/c18a5e8a5b1afb0d7a582fe9ebad4c1996c90da3/library/core/src/result.rs:1790:5    3: std::sys_common::once::futex::Once::call    4: bindgen::clang_version    5: bindgen__bin::main note: Some details are omitted, run with RUST_BACKTRACE=full` for a verbose backtrace.
INFO: Elapsed time: 198.363s, Critical Path: 100.03s
INFO: 607 processes: 235 internal, 372 processwrapper-sandbox.
FAILED: Build did NOT complete successfully

@timothytrippel FYI

@dbeitel-opentitan I had to solve this for our internal infrastructure as well. The way I solved it was two fold:

I built a newer version of libstdc++.so for our internal infrastructure by building GCC from source (I pulled / built release 9.3.0: git clone --branch releases/gcc-9.3.0 --depth 1 git://gcc.gnu.org/git/gcc.git <download to this local dir>)
I patched rules_rust a while ago to enable it to set LD_LIBRARY_PATH during the Bazel build when you set the following envar: export RULES_RUST_BINDGEN_LIBSTDCXX_DIR="/path/to/gcc/lib64" (Note: you won't need to patch rules_rust yourself, the version Bazel pulls in is already patched, you just need to set the aforementioned envar.)

@a-will I'd love to upgrade, but our HW flow is based on CentOS-7 and I don't think we are upgrading anytime soon.  @timothytrippel Yeah, I can do that but I was hoping there was a way to cleanly work in CentOS-7.
Thanks for the info.

Hi @timothytrippel ,
Overview: I am a new user of OpenTitan, and I encounter a similar error shown in snippet below when I run this: bazel test --test_output=streamed //sw/device/tests:uart_smoketest_sim_verilator on Centos 7 even after I have done the following. Please advise. Thank you.

I have built GCC of version 11.1.0 from source and
set the export RULES_RUST_BINDGEN_LIBSTDCXX_DIR="/path/to/gcc/lib64" in my .bashrc and source .bashrc

Environment:

Centos 7
Verilator version: 4.210
GCC version: 11.1.0
LLVM (and CLang) version: 16.0.6

What I have done:

I am only getting started with OpenTitan, and I use the Verilator workflow as recommended in the Getting Started page.
I could run the chip simulation on its own successfully: bazel build //hw:verilator
Among other things, I have tried to explicitly add the gcc library path to the LD_LIBRARY_PATH but to no avail. Regardless, I could not get past the error. Note that for each change in the environment, I ran a bazel clean --expunge to start from a clean slate.

Error Message Snippet:

Sorry late to this one, is this still an issue or can we close this?

It is, but I don't know what we can do about it.  There is a workaround, but its ugly.  CentOS 7 is being EOLed, but its still in use in by a lot of HW projects.

Lets just close it for now.

Hi @dbeitel-opentitan , you mentioned that there is a workaround for this libstdc++.s0.6 issue on CentOS 7, but it is ugly.  Would you mind to share what is the workaround?


Test bug report from the IRC channel

Submitted by: phillipadsmith
From channel: ##prose-bot-testing
At 2015-08-06 05:28:17 UTC

Just testing, please ignore. :)

@phillipadsmith awesome :)


Example not working

I have tried the following as suggested in the main page

    <title>Website</title>
    <link rel="stylesheet" href="bower_components/angular-patternfly/dist/styles/angular-patternfly.min.css"/>

Msg = {{ msg }}

And
var app = angular.module("myapp", [
    'patternfly',
    'patternfly.charts',
]);

app.controller("MainController", function($scope) {
    $scope.msg = "Hello World";
})

But the following error is given:
localhost/:1 GET http://localhost:5000/bower_components/patternfly/components/c3/c3.min.js
localhost/:1 GET http://localhost:5000/bower_components/patternfly/components/d3/d3.min.js
localhost/:1 GET http://localhost:5000/bower_components/patternfly/components/c3/c3.min.js
localhost/:1 GET http://localhost:5000/bower_components/patternfly/components/d3/d3.min.js
angular-patternfly.min.js:1 Uncaught ReferenceError: $ is not defined(anonymous function) @ angular-patternfly.min.js:1(anonymous function) @ angular-patternfly.min.js:1
angular.js:68 Uncaught Error: [$injector:modulerr] Failed to instantiate module myapp due to:
Error: [$injector:modulerr] Failed to instantiate module patternfly due to:
Error: [$injector:modulerr] Failed to instantiate module patternfly.filters due to:
Error: [$injector:modulerr] Failed to instantiate module patternfly.select due to:
Error: [$injector:nomod] Module 'patternfly.select' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.5.5/$injector/nomod?p0=patternfly.select

Two major reasons:

There is no /bower_components/patternfly/components/d3/d3.min.js and for c3. They are in seperate folders.
When I used the non-minified patternfly.js, and corrected the d3 and c3 paths, I see the error is generated  from this line where $ is undefined. var patternflyDefaults = $().c3ChartDefaults();

What to do?

Hi,
What version of Angular-PatternFly are you using?
Thanks,

Dave

I think you might be missing a link to patternfly js file:

Dave

Well, you are right, the correct way to run the example is to include original patternfly.js and JQuery as well.

    <title>Website</title>

    <link rel="stylesheet" href="bower_components/patternfly/dist/css/patternfly.min.css">
    <link rel="stylesheet" href="bower_components/patternfly/dist/css/patternfly-additions.min.css">
    <link rel="stylesheet" href="bower_components/angular-patternfly/dist/styles/angular-patternfly.min.css"/>

Msg = {{ msg }}

Welcome to Angular Patternfly application

          Click for more info

Should I make a pull request for this? Upon reading README.md I assumed it will work with just angular-patternfly.js like ui-bootstrap module which does not have bootstrap.js but only ui-bootstrap.js

Hi, I think this would be great in an Angular-PatternFly QuickStart Guide, similar to PatternFly's QuickStart Guide:  https://github.com/patternfly/patternfly/blob/master/QUICKSTART.md
I could see a PR for that,
Thanks,

Dave

Info is now in https://github.com/patternfly/angular-patternfly/blob/master/README.md


Patch Release Update for lz-vending module

Description
As per https://github.com/Azure/bicep-lz-vending/pull/38 this is adding the release upstream to the registry for consistency.
Updating an existing module

[x] This is a bug fix:

[x] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable. - see associated PR
[x] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[x] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

@shenglol please can we make an exception on the tests here please as this has passed our test framework fine as you can see on checks here: https://github.com/Azure/bicep-lz-vending/pull/38
Also as we have cut a release we want customers to have a consistent experience across consumption methods of this module.
Thanks
Jack


Feature: use .env to configure API Keys etc

building from the great work of @lozanoSergio I added the feature to read private config data from a .env file which gets ignored by git so people don't commit their secret. also it makes it easier to create PR and merge stuff...

@dargmuesli incorporated your changes. thanks for reviewing

Are oyu

@dargmuesli sure, you can find me in the graphile discord, as well as the other usual suspects: nuxtjs, vue land,typescript,vuetify,etc. Same username ;D.


This capability is not yet supported for macOS or iOS

Under part
Sharing encrypted documents with external users
“This capability is not yet supported for macOS or iOS.”
Specific functions and features are now supported on IOS and MAC OS. Refer Support for sensitivity label capabilities in apps https://docs.microsoft.com/en-us/microsoft-365/compliance/sensitivity-labels-office-apps?view=o365-worldwide#support-for-sensitivity-label-capabilities-in-apps

@LeeMin-Z Thank you for your feedback. Could you please provide us with the document which states it is not yet supported for macos or ios?
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @LeeMin-Z Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[WinError 2] The system cannot find the file specified

I have just re-installed a new version, but I my kernels does not work:
It cant detect R kernel and when i create a new notebook, I see
[WinError 2] The system cannot find the file specified
And when i try to open any .ipynd files I get error:
File Load Error for appendixC_computing-derivatives.ipynb
Unhandled error
I am not sure what could cause it. my previous desktop version worked perfectly

solved teh issue with R kernel, but still having issues with loading ipnyb


Improve the docs on backend secured with OAuth2

Description:
In the doc, we say as the following.
You cannot call backend services secured with OAuth through APIs created in the API Publisher. At the moment, you can call only services secured with username/password (Basic Auth/ Digest Auth).
Via an extension point, you can call a backend which is secured via OAuth2. This feature will be available in the product OOB in the next release.
https://apim.docs.wso2.com/en/latest/learn/design-api/endpoints/endpoint-types/
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

@pubudu538 , we need more clarification in this regard. We will reach out to you soon.

This has been updated in 3.0.0 and 3.1.0 docs.
In addition to this, we can point to this article https://medium.com/@athiththan11/wso2-api-manager-oauth2-protected-endpoint-aa51c62f0ad7 in Configuring OAuth2 Protected Endpoints.
WDYT @pubudu538 ?

Looks like this is fixed so I'm closing this.


Map on coderdojo.com geo-fixes

Two main issues;

When country is typed in full country should appear on the map - looks like atm it's going to the first one with the country in the full address
Places without Dojos typed in - a zoomed out version should be shown rather than the nearest Dojo if it is > 100km away

Hi I want to fix this.As this is my first time,please explain what needs to be done.
Thanks.

This is referring to the map on https://coderdojo.com currently it the search on it has the problems stated above. The site is a WordPress with our custom theme, found here. The map file is here
the map should function the same as the one on https://zen.coderdojo.org. that maps code can be found here.
Good luck

Also, if you never participated before, have a look at https://github.com/CoderDojo/cp-local-development to get you started with the environment :)

I have done all the setup. And at localhost:8000 the page is opening but map is not rendering.

look like there is some JavaScript or CSS not working.
In an other issue I just read about the Bower step in the installation. Maybe that is a hint but I am really not sure. Others will know better.

No....there is no error neither in terminal nor browser console...
Only a warning i get in browser console
Google Maps API warning: SensorNotRequired https://developers.google.com/maps/documentation/javascript/error-messages#sensor-not-required

Sorry we are in the process of changing out Dev system and haven't updated
the docs. You need to run gulp Dev in another terminal in the
cp-zen-platform folder
On Mon 29 Aug 2016 at 19:14, Vajahat Kareem<EMAIL_ADDRESS>wrote:

No....there is no error neither in terminal nor browser console...
Only a warning i get in browser console
Google Maps API warning: SensorNotRequired
https://developers.google.com/maps/documentation/javascript/error-messages#sensor-not-required
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CoderDojo/community-platform/issues/1020#issuecomment-243206938,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0-Wgee9lMkPtd3MCdj_KoodK8X5pzsks5qkyFogaJpZM4Jr9lO
.

--
Cian Butler (butlerx)
Redbrick Secretary 2016-2017
Computer Applications and Software Engineering - Year 3

Issue won't be solved this way.
The direction taken is to replace the current map on coderdojo.com by the one on zen.coderdojo.com; using the same logic for search.
I'm keeping the issue open, but the fix is related to https://github.com/CoderDojo/community-platform/issues/1199

We removed the map on coderdojo.com, this issue doesn't apply anymore


Plugins devcontainer

Add a devcontainer for running/debugging plugin source code.
To begin the discussion for devcontainers...
This is a Proof of Concept for running/debugging local plugin source code. I elected to place this in the /plugins folder as there are .devcontainer and .vscode directories in the root. Do we need those?
To test this code out, see if it works for you.

open VS code
Open Folder... navigate to /plugins
If prompted to "Reopen in Container", do so...
Else open your command pallette and select "Dev Containers: Open Folder in Container..."
This should begin building the container.

This will add docker-in-docker (self contained docker environment in the devcontainer) and will install poetry. I have elected to pip install to the default python for the devcontainer. We should do something to make it easier to keep devcontainer in sync with poetry. (The editor windows won't use the poetry virtual environment so it can't find the imports when you are editing the files...)
Included a Run/Debug (that uses poetry!) to start up acapy with the plugins loaded. I just created specific config and compose files for the devcontainer until we decide the path forward. The compose only starts db and proxy.
To test it all out, add a breakpoint to oca_record_list method in traction_innkeeper/v1_0/oca/routes.py, open Run/Debug view, select "Run/Debug Plugin" and start/F5.
This will kick off the docker compose build/up to start db and proxy, then will run acapy + plugins from source.
Open browser to get_oca, Try it out, execute and you should hit your breakpoint in your IDE.
Once we get this stuff squared away, I think we can remove the top level poetry and change the docker images. The image that should be used for "production" can pull the plugins from github (use the PR sha or a tag). It would be nice to get a single devcontainer config that will load up (and debug?) tenant UI as well. But start with the plugins...

I think it's best to keep any devContainer for the project at the root level that is suitable for working on the project as a whole.  It looks like @amanji added a docker-outside-of-docker container for frontend development there.
Just a note on the different docker options.  docker-in-docker provides an isolated docker environment for development.   docker-outside-of-docker provides a connection to the host's docker environemnt to provide a shared docker environment.  Which you use depends whether you need to interact with external services also running in docker.

Just a note on the different docker options. docker-in-docker provides an isolated docker environment for development. docker-outside-of-docker provides a connection to the host's docker environemnt to provide a shared docker environment. Which you use depends whether you need to interact with external services also running in docker.

Is the tradeoff as easy as that - pick one or the other?  If so, I would say go with docker-outside so that docker-based external services can be easily used — von-network, redid, tails server, even a database as needed.  Keep those independent and just have them running, and stop/start Traction as needed. External versions of those services couyld be used, but the mixing and matching might be tough?
Definitely think the two instances of DevContainers need to be rationalized.

The existing devContainer is just for Node for the TenantUI. Really that's just a standalone application that points at a traction instance in it's configuration.
We of course start it up with Traction as part of the whole ecosystem that runs up when you start up the docker compose. But it's its own stack and separate development from the plugins themselves so I don't know what's best practice for combining all those things or not.
Really the Tenant UI could be it's own repository without much difference.
I don't use the devcontainer for TenantUI dev (have the version of node I need locally right now) but it's a good practice to have around so people don't need to have specific Node versions locally to dev. Can do whatever is needed to try out the plugin devcontainer here.

Updated docker compose startup to use askar-only ghcr.io/hyperledger/aries-cloudagent-python:py3.9-0.8.2 image as well

Updating progress on this ticket. Will keep looking into but would be good to see if anyone else on this PR has feedback.
There are a number of fronts here:
Update ACA-Py to 0.8.2, non-Indy image
Local Docker testing
Can start fresh and build/up the compose ecosystem from /scripts
Can do full Traction regression (tenant create, schema/cred create, connect to BC Wallet, issue, revoke, message)
So everything appears all good there
Pull request deployment
I've updated values needed in the helm vals for this and got it to deploy.
I can create a Tenant, make it an issuer, connect to a wallet (or other tenant), make schemas/creds
However, if I issue a cred, then accept it (I am getting the offer pop up in the wallet and can accept it), it is always ending up in abandoned state. I'm not sure if this is related to endorser or something?

Tenant here:
4b750fc9-70c1-4345-afcc-01d1e4d8aa3c
3ee16e18-c16f-4879-9e71-eef0f1d2a1ad
Dev container
Same results below on Windows and with WSL (ubuntu)
Can seem to build up everyhting when starting in container, but on Run/Debug it crashes out  with a DB password error
2023-07-10 21:56:38,447 aries_cloudagent.core.profile INFO Create profile manager: askar
2023-07-10 21:56:38,534 aries_cloudagent.commands.start ERROR Exception during startup:
Traceback (most recent call last):
  File "/workspaces/traction/plugins/.venv/lib/python3.9/site-packages/aries_cloudagent/askar/store.py", line 157, in open_store
    store = await Store.open(
…
aries_askar.error.AskarError: Backend error
Caused by: error returned from database: password authentication failed for user "postgres"

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/workspaces/traction/plugins/.venv/lib/python3.9/site-packages/aries_cloudagent/commands/start.py", line 72, in init
…
    opened = await store_config.open_store(provision=False)
  File "/workspaces/traction/plugins/.venv/lib/python3.9/site-packages/aries_cloudagent/askar/store.py", line 171, in open_store
    raise ProfileError("Error opening store") from err
aries_cloudagent.core.error.ProfileError: Error opening store

Shutting down

I do also see these problems listing and a popup warning me there's errors prior to that when

postgres may need to be updated - try image: postgres:14

also... the plugins/docker plugins/demo stuff is not nearly as configured as the stuff in scripts. so you will probably have to some re-engineering and additions if you want to run it with all those configurations. I'm assuming the addition of an env file and env vars (can that be done in vscode files?) for the startup of launch.json acapy.
maybe we should also take out pylint. those classes should be in the classpath at least on a rebuild of the devcontainer.

Will need to see what to do about this PR. @esune
The 0.8.2 update (actually on 0.8.3pre) has been merged in with additional tweaks from another PR now.
The devcontainer stuff I could not get to work, see comment above I believe from discussing with Sherman there's additional configuration or environment work or something that would need doing to be able to debug a plugin using the instructions documented above. I get the Caused by: error returned from database: password authentication failed for user "postgres" error shown in that comment when trying to run.
@usingtechnology might have other thoughts on additional needs for the dev container. Not sure if worth merging and working on further or if a different approach is needed?

i don't see those messages in the Problems tab, on Mac but shouldn't really be different? also, i can launch acapy + plugins using the "Run/Debug plugin" command (launch.json) and then call the API through swagger in a browser and it acts ok (saves to the db). I don't know how you guys want to develop or what you want out of the devcontainers, so that would be for you to tweak (like if you want tenant UI started in the devcontainer, or if you want to run it externally but against plugins running in the IDE).
If you've got the main branch up to date (python 3.9, acapy 0.8.x) then just torch this PR, start fresh when you have time to figure out what the team wants for developers.

i don't see those messages in the Problems tab, on Mac but shouldn't really be different? also, i can launch acapy + plugins using the "Run/Debug plugin" command (launch.json) and then call the API through swagger in a browser and it acts ok (saves to the db). I don't know how you guys want to develop or what you want out of the devcontainers, so that would be for you to tweak (like if you want tenant UI started in the devcontainer, or if you want to run it externally but against plugins running in the IDE).
If you've got the main branch up to date (python 3.9, acapy 0.8.x) then just torch this PR, start fresh when you have time to figure out what the team wants for developers.

Yeah it's not the problems listing that's blocking, but that if I start out and pull this repo and open everything up following the instructions listed I get that Caused by: error returned from database: password authentication failed for user "postgres" error when trying to launch. But if that works out for you I'm not sure if I'm missing running something else along with it or it it's an issue with my system.

just checked this PR out clean, new location on my machine. reopened in plugins in container, everything setup nicely. launched in debug and proxy, db and acapy stood up fine, able to use swagger at localhost:8032 to create reservations, log in as innkeeper etc. so no db connection issues.
don't know what is going on with your machine, but could try the command palette: "Dev Containers: Rebuild Container without Cache".  maybe that will get you back to clean?
in another vscode window, opened up tenant-ui in a container, again, all built nicely. launch 'backend - run dev', then 'frontend- run dev', then 'frontend - chrome'. was able to set breakpoints in vue and in plugins and all worked fine. able to create reservations, approve etc. so db connectivity was good.
the problems, i guess it depends on what files are open? anyway, if you don't need pylint, just remove it and figure out what helper plugins you want.
given what is in this PR, I can debug plugins and tenant UI vue code with breakpoints in an IDE. be nice if other traction devs could try it out and see if you want to keep the PR and/or what tweaks you want/need to make. but if you guys can't get it to work or it that flow doesn't work, I'd just kill the PR. been around a while and is dead-weight if no one makes use of it.

just checked this PR out clean, new location on my machine. reopened in plugins in container, everything setup nicely. launched in debug and proxy, db and acapy stood up fine, able to use swagger at localhost:8032 to create reservations, log in as innkeeper etc. so no db connection issues.

Ok, thanks @usingtechnology !
Must be something with my setup, I will mess around more when I have time. If you are able to get debugging going from clean that's great.
Yeah the devcontainer is definitely something we'd want to work with IMO so if this can work for people to get debugging going then we should pull it in for sure.
I've rebased in the changes upgrading ACA-py to 0.8.3pre (from the other PR) so the changes in here are good to merge in.


RuntimeError: Error(s) in loading state_dict for GeneralizedRCNN:

❓ Questions and Help
I run with python tools/train_net.py --config-file configs/e2e_ms_rcnn_R_50_FPN_1x.yaml,and get the error:
Traceback (most recent call last):
File "tools/train_net.py", line 171, in
main()
File "tools/train_net.py", line 164, in main
model = train(cfg, args.local_rank, args.distributed)
File "tools/train_net.py", line 53, in train
extra_checkpoint_data = checkpointer.load(cfg.MODEL.WEIGHT)
File "/home/dspuser/maskscoring_rcnn/maskscoring_rcnn/maskrcnn_benchmark/utils/checkpoint.py", line 62, in load
self._load_model(checkpoint)
File "/home/dspuser/maskscoring_rcnn/maskscoring_rcnn/maskrcnn_benchmark/utils/checkpoint.py", line 97, in _load_model
load_state_dict(self.model, checkpoint.pop("model"))
File "/home/dspuser/maskscoring_rcnn/maskscoring_rcnn/maskrcnn_benchmark/utils/model_serialization.py", line 79, in load_state_dict
model.load_state_dict(model_state_dict)
File "/home/dspuser/anaconda3/envs/msrcnn/lib/python3.7/site-packages/torch/nn/modules/module.py", line 777, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for GeneralizedRCNN:
size mismatch for roi_heads.maskiou.predictor.maskiou.weight: copying a param with shape torch.Size([81, 1024]) from checkpoint, the shape in current model is torch.Size([3, 1024]).
size mismatch for roi_heads.maskiou.predictor.maskiou.bias: copying a param with shape torch.Size([81]) from checkpoint, the shape in current model is torch.Size([3]).

I have learned some issues ,just like 430 , 15 and so on, and I try the first method in 430 , and
https://github.com/facebookresearch/maskrcnn-benchmark#finetuning-from-detectron-weights-on-custom-datasets. but still exist this error.  I know the problem may be in last layer, because it's for coco 81classes, however I have just 2class. But I really can not deal with it by using method above.  Another idea in 430 is to make  model surgery, but I do not know which files I should modify.  I am looking forward to  method to solve it . Thanks in advance .

@lingbo666 Did you do the model surgery and saved your model?

@Darshan2701 I just have tried to download a model and delete as #430 ,but it did not work for me. Another method is model surgery, but I can not find witch files I can do it .

I have solved this problem, and run maskscoring-rcnn successfully. this problem is that I do not delete last layer which for coco81. Then I try #430 some methods,  but bug still exist. Finally, I find methods in #430 may not delete all, so I need delete all the weight ,bis.
months:
newdict['model'] = removekey(_d['model'],
['cls_score.bias', 'cls_score.weight', 'bbox_pred.bias', 'bbox_pred.weight','mask_fcn_logits.bias','mask_fcn_logits.weight'])
when remove, like this, remove these and it run successfully for me.
I hope this can help who encounter this problem.If any question, please point out. Thanks!!


Update scheduler passthrough data type

as title

 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.


Allow custom http client on webhooks

This is just a convenience function to allow passing a custom http.Client without breaking backward compatibility which is quite useful for customising client parameters and testing your own functions.
There seems to have been some effort https://github.com/nlopes/slack/pull/230 into allow custom http.Client in the past, but webhooks were not covered by.
I hope this makes sense

LGTM.


No module named 'dataproviders'

2.2.1 A standard WOFOST run
First we make a standard WOFOST run to see the model behaviour without assimilation of satellite data.
When Run it：
In [4]:
1
from dataproviders import parameters, agromanagement, weather
2
wofost = Wofost71_WLP_FD(parameters, weather, agromanagement)
3
wofost.run_till_terminate()
4
df = pd.DataFrame(wofost.get_output()).set_index("day")
Show out Error：

ModuleNotFoundError                       Traceback (most recent call last)
 in
----> 1 from dataproviders import parameters, agromanagement, weather
2 wofost = Wofost71_WLP_FD(parameters, weather, agromanagement)
3 wofost.run_till_terminate()
4 df = pd.DataFrame(wofost.get_output()).set_index("day")
ModuleNotFoundError: No module named 'dataproviders'

It seems that you didn't copy the file 'dataproviders.py' which is part
of the repo.
Allard
On 2019-11-18 03:50, IrriPro wrote:

2.2.1 A standard WOFOST run
First we make a standard WOFOST run to see the model behaviour without
assimilation of satellite data.
When Run it：
In [4]:
1
from dataproviders import parameters, agromanagement, weather
2
wofost = Wofost71_WLP_FD(parameters, weather, agromanagement)
3
wofost.run_till_terminate()
4
df = pd.DataFrame(wofost.get_output()).set_index("day")
Show out Error：
ModuleNotFoundError Traceback (most recent call last)
in
----> 1 from dataproviders import parameters, agromanagement, weather
2 wofost = Wofost71_WLP_FD(parameters, weather, agromanagement)
3 wofost.run_till_terminate()
4 df = pd.DataFrame(wofost.get_output()).set_index("day")
ModuleNotFoundError: No module named 'dataproviders'
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Links:
[1]
https://github.com/ajwdewit/pcse_notebooks/issues/1?email_source=notifications&email_token=ABJYCNTMFGWOTWCYCNRBXRTQUH7H7A5CNFSM4JONKXG2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZ5GAPQ
[2]
https://github.com/notifications/unsubscribe-auth/ABJYCNS7SMNJNLJUCLSOYBTQUH7H7ANCNFSM4JONKXGQ

Thank you,
I has fix it. I forgot to copy the file


CM705: How to Create Better Looking Console Applications With Spectre…

….Console

@cjaynjoku, the project name is very long let's change it to something shorter.
This is preventing the project from being loaded properly on Visual Studio and Jetbrains Rider.


Slim & stateless version of the boilerplate

Cloned the project and didn't need any of the auth/db/models things.
Any thoughts on stripping those out and maybe creating a separate branch?
i.e. Static react app with routes and an empty need maybe, with no authentication and DB. Would be more suitable in a microservices architecture land perhaps.
Happy to write up a PR removing those things (not for master obviously).

@dzotokan Definitely!
This has been bothering me because it took me at least 30 minutes stripping it out that day.

Static react app with routes and an empty need maybe, with no authentication and DB. Would be more suitable in a microservices architecture land perhaps.

Good thinking. Perhaps we only need to keep that branch up to date when there are webpack or config changes then.

I'm going to try to work on this guy. The refactoring here will mainly be in the server/ folder.
The easiest approach I can think of is to:

abstract connecting + authentication + model creation  into a different file - so people can easily comment that out if you wish to switch databases/not use a database.

I'll have a go to see how much work this will take + whether it's a good solution

Closing as issue is tacked in #372


HTTPS link to semver.org

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @coliff!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS


LUIS Programmatic API : Publish error 400

im trying to publish my Luis Model via api ,
but i dont understand how it works ?
the method called :
https://westus.api.cognitive.microsoft.com/luis/v1.0/prog/apps/{appId}/publish
there is the response bodyof my post 400
{
"error": {
"code": "BadArgument",
"message": "Cannot find an application with the specified ID"
}
}
im usind this app id

i'm i wrong ? i just want to update the luis model i just trained ...
thanks in advance .

For luis programatic API you need to use your cognitive services subscription key which is different that Microsoft App Id. You can find further documentation here: https://dev.projectoxford.ai/docs/services/56d95961e597ed0f04b76e58/operations/5739a8c71984550500affdfa

it's not a key problem , im not geting a 401 not autorized , im getting a 400 , Bad argument ...

I looked at the Microsoft App Id and thought you are getting 401. But you need to use luis subscription key and not the Microsoft App Id when calling to Luis programmatic api that is the main reason that you get 400.


refactor - improved error regarding  issue "Better error message #6302"

closes #6302

i cant figure out why my tests are failing, could you please guide me through this?

i cant figure out why my tests are failing, could you please guide me through this?

It looks like the self-tests didn't pass. Since it's important for the rustfmt source code to adhere to the same formatting style that it enforces, the self-test runs rustfmt on its own codebase. You can find more details in the Contributing.md.
To resolve this, you might want to try running cargo run --bin rustfmt /path/to/modified/source.rs to format the modified source code using the rustfmt built from the source. After that, you can check if the tests pass locally by running  cargo test.
Hope this helps!

@ytmimi  i think its ready for review now, thank you

Thanks for continuing to look into this. I've left a few comments inline. Also, I think it would be great if we could add a unit test similar to those defined here, where we call the rustfmt binary and check the content of stdout / stderr.
In this case we can create a new config file in tests/config that will trigger the error, and we can call rustfmt with the --config-path argument and validate that we see our expected error message in stderr.

okay, so we have to write a test case in tests/rustfmt/main.rs that takes the file we create in tests/config which is faulty and we should expect the output as an error, right?

Also, one more note. The rustfmt team strongly discourages merge commits. Please rebase your branch to bring it up to date when you get a chance.

would it be better if i do once we have finalized all the refactoring?

okay, so we have to write a test case in tests/rustfmt/main.rs that takes the file we create in tests/config which is faulty and we should expect the output as an error, right?

Yeah, thats right.

would it be better if i do once we have finalized all the refactoring?

Sure. Just wanted to make you aware of our stance on merge commits.

@ytmimi , why trying to test this issue, i faced two issues

when we raise the error , we get

The file /home/rufevean/shitbox/rustfmt/tests/config/issue-6302.tomlfailed to parse. Error details: invalid type: integer2019, expected string in edition
we get the entire path . so we cant test it that way as the absolute path will change for every person. so i think we might have to refactor the code where it will only present the path from the main project directory?

in the error display

The file `/home/rufevean/shitbox/rustfmt/tests/config/issue-6302.toml` failed to parse.
Error details: invalid type: integer `2019`, expected string
 in `edition`

the  in edition part is forming a new line , to track that issue we might have to tract the {e} directly as of my knowledge, we didnt specify anything that makes it in a new line.

@ytmimi  , i have added the commit, could you please check it and let me know where i am getting it wrong?

Let me know if you'd like to squash the commits / rebase the PR or if you'd rather I just squash the commits on the merge.

i had left a comment regarding Ok(parsed_config.to_parsed_config(style_edition, edition, version, file_path.parent().unwrap_or(Path::new("")))) and i would like to know how do you want me to proceed, and once i have refactored it according to how you think is better. i will rebase it . thank you

@ytmimi i think i did something wrong, i rebased it and squashed  all my commits into 1. but i can see other's commits here .

Yeah, definitely doesn't look right. You should be able to get back to the commit before the rebase if you check the  git reflog

@ytmimi , i tried squashing it again and deleting the commits that i dont want to include but even then there are some conflicts . its better if you may handle it. thank you

hmm weird, when I tried to update your branch GitHub marked me as closing the PR 🤔

reopening


⚠ Bofrost.lu has degraded performance

In ca51241, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 833 ms

Resolved: Bofrost.lu performance has improved in f311905 after 6 minutes.


consider integrating with overleaf.com?

Overleaf is an online latex editor. Perhaps some of the users of osf.io already do some of their writing using overleaf, and maybe being able to launch the editor from osf or being able to list projects within some collection on overleaf on osf could be benefical?
https://www.overleaf.com/devs (dev api "docs")

Thanks for getting in touch @thegreatmichael! This is a great suggestion. We keep track of user feature requests to inform future development internally, so I'll add a note to our issue tracker there. We also welcome community contributions to the code, so if you (or someone you know!) might be interested in building an Overleaf add-on, I can help point you to some documentation and resources.
There is currently a group working on an integration with ShareLaTeX, another latex editor (https://github.com/CenterForOpenScience/osf.io/pull/4597)

ShareLaTeX recently merged with Overleaf https://www.sharelatex.com/blog/2017/07/20/sharelatex-joins-overleaf.html
Great opportunity to build on the past work to push the integration through!


[Docs] Meshery Adapter for App Mesh

Current State
This adapter has minimal docs.
Desired State
https://docs.meshery.io/service-meshes/adapters/app-mesh should have a more complete description akin to that of other adapter pages. Most of the pages are now templated, and so, the skeleton of the page can be copy/pasted.

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@Shreyas220 one for you?

on it

i will do it


Add plugin to track and display stats from your performance in Castle Wars

Keeps track of various stats while playing a Castle Wars game and prints a post-game summary in the chatbox: https://i.imgur.com/NqDwdnJ.png

Cleaned up the PR a bit.

Removed tracking of stuff that required way too much logic and was hard to follow and understand or was working only in specific scenarios (e.g removed all barricade tracking, and freeze/splash tracking, as that one worked mostly only for ice barrage).
Removed unused stuff like checking if player is wearing CW brace etc
Properly encapsulated everything

I will post screenie in a bit, I just need to play some CW game

Updated screenshot:

This is so niche, should it be off by default?

Idea on how to display the stats:
Castle Wars - 24v24 - World 334
Zamorak 0 - 0 Saradomin
Captures: 0 Saves: 0 Deaths: 7 Kills: 9 Stuns: 0 Freezes: 0
Damage dealt: 2460 (Max 62) Damage taken: 1728 (Max 55)

Kills could be removed if it can't be reliably tracked.
Stuns instead of Spears since it's a stun spec, even though OSRS doesn't really have other stun sources.
No need for all the |, ~ and . business.
I kinda like the advanced stats that were in the initial PR. Those would only be useful for more serious CW players, but would provide valuable information to improve one's play. Casting stats not so much.

I don't think so @Adam- , it makes the end of the CW game way more interesting, and with the stuff I left/removed it is not so niche anymore imo.
@Unmoon hmmm that looks a bit better. Yeah I don't think that kills are possible to track. I removed the advanced stats because they were unreliable and added unnecessary complexity.

Updated image after @Unmoon 's requests

As per @Adam- 's request disabled the plugin by default (even though I still disagree)

I don't like this at all. We have much, much less niche plugins that are off by default like the loot tracker. Having a config option for "do more things" is silly. I'm fine with it if 5ea5f204de31c85b0c7e7410758951b2f794ac45 is reverted.

Well yeah but loot tracker works everywhere, this works only in this particular activity


Correctly handle large number of favorites

Could be handled similarly to Niagara, make the list scrollable, and once you get to the bottom you can use the swipe up gesture and vice versa.
Or, calculate how many favorites would fit on the screen and limit the number of favorites to that number. Disable the add-favorite button for all non-favorites once the limit is reached.

doesn't seem possible to connect pointerInput with a nestedScrollConnection, so allowing scroll in the LazyColumn would mean having to reimplement the entire feature.


Not sorting imports in .vue files

Hi, I have a following config:
{
  "importOrder": [
    "^vue/(.*)$",
    "^[^@v](.*)$",
    "^@(.*)$",
    "^[./]"
  ],
  "importOrderSeparation": true,
}

And an App.vue file (already formatted with prettier):
<script>
import Vue from 'vue'
import TheNavBar from '@/components/TheNavBar'
import dayjs from 'dayjs'
</script>

When I copy its content to .ts file, it gets formatted as expected:
import Vue from 'vue'

import dayjs from 'dayjs'

import TheNavBar from '@/components/TheNavBar'

Is it possible to somehow enable .vue files support?

Hey @fbjorn thanks for reporting this. Could you please provide a sample repo, it surely makes investigating this easier.

@byara sure, I created a repo: https://github.com/fbjorn/vue-prettier-plugin-sort-imports-example
It's bare minimal, but demonstrates that plugins sorts .ts file correctly, but does nothing for a .vue

Hi, We do not support the vue as of now. It is in our roadmap for version 4.

I'm looking forward to Vue support! :rocket:

I've added the base support for .vue files; i've successfully applied the patch via yarn patch and working successfully in a large project.


diff_drive_controller/cmd_vel requires twice the value

The size of one section is 180 mm, so it should run one section at once when the cmd_vel is 180mm/s.

ROS Kinetic 1.12.14
Gazebo 7.0.0

solved on 1de891125ec7472fc143ba1baa1b249bd0cc29cb


HDDS-7251. Replace Log4j 1.x with Reload4j

What changes were proposed in this pull request?
Log4j 1.x is EOL.  This PR proposes to replace it with Reload4j, which is a drop-in replacement with security updates.
https://issues.apache.org/jira/browse/HDDS-7251
How was this patch tested?
Built locally.  Only Log4j 2 present in distribution:
$ ls -1 hadoop-ozone/dist/target/ozone-1.3.0-SNAPSHOT/share/ozone/lib/*log4j*
hadoop-ozone/dist/target/ozone-1.3.0-SNAPSHOT/share/ozone/lib/log4j-api-2.17.1.jar
hadoop-ozone/dist/target/ozone-1.3.0-SNAPSHOT/share/ozone/lib/log4j-core-2.17.1.jar

Verified no Log4j or Reload4j classes are present in shaded FS jars.
Full CI:
https://github.com/adoroszlai/hadoop-ozone/actions/runs/3103286435

Merged. Thanks @adoroszlai

Thanks @jojochuang for reviewing and merging this.


tutorial 2 (intro to jax): checkpoint API has changed

** Tutorial **
https://uvadlc-notebooks.readthedocs.io/en/latest/tutorial_notebooks/JAX/tutorial2/Introduction_to_JAX.html
Describe the bug
The checkpoint saving/loading code needs to be changed to the new flax API
https://flax.readthedocs.io/en/latest/guides/orbax_upgrade_guide.html

Thanks for pointing out the API change! It seems to me that the API is still in progress and possible subject to change, so I'll wait a bit longer before changing it everywhere in the tutorials.


Frame Window Timed Out error while trying to renew the token from UserManager

Hi,
I am new to this, so kindly bare with me. We are having the token expiration time of 60 minutes in our angular app. Just before on minute from which the token gets expired, we are triggering a popup to continue the session in the application. There's a button which is triggered at that time. Firstly we are checking if the token is expired or not. If it is not expired, we are calling  this.userManager.signinSilent().then((user: Oidc.User) => {
this.$uibModalInstance.close();
});
But seeing the above code, I am not getting the promise in the response after signinSilent() is called. Im stuck with this. I am getting Frame Window Timed out error in the console. I think that must be regarding iframe. Can anyone help me on this?

Check the logs of the token service -- perhaps there's an error being displayed in the non-visible iframe.

The error is in iFrameWindow.js.

In the above code snippet, in the line

this._timer = window.setTimeout(this._timeout.bind(this), timeout); there is an error while "this._timeout" is called. Then error is thrown saying 'Frame Window Timed out'.

Closing this issue.. Found the error, it was regarding the invalid redirect uris. Actually, the siloent redirect uri was not present in the allowedredirectUri's which caused the error. Thanks @brockallen for suggesting me to check through the log files. Closing this issue.

I have redirect uri set in my database still I am facing this issue. Any Idea?
this.manager.events.addAccessTokenExpiring(() => {
      this.manager.signinSilent()
            .then((user) => {
              debugger;
            })
            .catch((error: Error) => {
              console.log(error);
                this.manager.getUser()
                    .then((user) => {
                      debugger;
                    });
            });
    });

    .catch((error: Error) => {
      console.log(error);
        this.manager.getUser()
            .then((user) => {
              debugger;
            });
    });

});

@brockallen These token service logs that you mentioned earlier, where can I find them assuming I installed oidc-client via npm.

@AlexanderBaggett They would be in your token server server somewhere, unrelated to this client.


Update VTK and Trame sections

Updates the Trame and VTK sections that @jspanchu will lead

Thanks for porting those notebooks over here!
Regarding the pre-commit.ci check failure,

tutorial/06_vtk/e_vtk_next.py:64:1: F401 'vtkmodules.util.numpy_support.numpy_to_vtk' imported but unused

Seems like we can remove that numpy_to_vtk import and leave the vtk_to_numpy as it is.
Otherwise, LGTM.
Right now the pip install --extra-index-url ... command shows a big warning about our gitlab package registry using an improper html5 package. I've reported it but i think it's not a big showstopper.

Thanks for doing this session @jspanchu ! We cannot wait to see how the vtk wasm works.


03-09-extending-gem5-models

[x] probe point explanation
[x] probe point usage
[x] probe point example
[x] probe point example 2

I only did ProbePoints

All slides and materials are done.
What haven't been done

Third party testing and edit pass


docker documentations

Hi,
The documentation for integrations with docker is not updated.  Please fix. (broken url for requirements.. etc)
I'm trying to use: https://superset.apache.org/docs/databases/dockeradddrivers but this is not ok.
Maybe  the best driver is this : https://pypi.org/project/mysql-connector-python/
But I get : mysql+mysqlconnector://root:root@localhost/table
"ERROR: (mysql.connector.errors.DatabaseError) 2004 (HY000): Can't create TCP/IP socket (97)
(Background on this error at: http://sqlalche.me/e/13/4xp6)"
Can you check ?

Hi @jot4p this is helpful feedback, I'll investigate what we can do on the docs side.
At first glance -- it seems like the container running Superset can't connect to your MySQL database. So this could be a networking configuration issue!
One thing I would try is firing up the shell for the docker container running superset_app and see if you can connect to your MySQL database from within that container. If it can't / you ran into a similar error, then you need to configure your network surrounding your MySQL database to accept connections from your computer, VM, etc.


How do I know if cell is thisDay

I would like know highlight cell that is thisDay, just like Apple Calendar.
But dateBelongsTo of CellState haven't thisDay value,
So how do I know if cell is thisDay ?

In the project I'm currently working, I just use simple yyyy.MM.dd date formatter to compare cellState.date to Date().

Dev @myurieff is correct.
If you want to know know if a particular day is Today just compare it.
You can compare it with your calendar instance.
let myCalendar = Calendar.current // You can configure this calendar to your time zone and locale etc
if myCalendar.isDateInToday(myDate) {
   print("This date is Today!")
} else {
   print("This date is not today")
}


Add license to source files

Currently the repository has a license that applies to all IslandMUD files, but it would be clearer/nicer if it was at the top or bottom of each source file.

I don't intend to do this unless it comes to my attention that implementing this as described would be worth the effort.


fix(manual) adds deno_emit and rollup to the list of 'deno bundle' alt

Currently the deprecation warning just suggests ESbuild as an alternative to deno bundle. deno_emit and Rollup are two other valid methods.
This is also inline with the logged warning when calling deno bundle on latest (see: https://github.com/denoland/deno/blob/main/cli/tools/bundle.rs#L35)
Mostly helpful because deno_emit took me awhile to find but is exactly what I needed for a project I'm working on.

 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.

Yes, sorry about that. Signed now.

No problem. Thanks for signing!


[Feature Request] Labelling or Describing Pomodoro work

Hey @eatgrass,
Thank you for creating the almost 'perfect' plugin for my time tracking woes. I would love to see the ability to add a 'task name' or 'description' to a Pomodoro work session or even the ability to change the fact it says 'work'
The idea being that I can better describe what the block is being used for. Leading to reporting on those blocks becoming easier in future.
Just an idea, currently getting around this by using your data view task example.
Thanks again for the excellent addition to my toolkit
b

Hi @benjaminwestern, thank you for sharing your ideas.
Could you let me know which version you're currently using? I've just released a new update today, it includes a task field in the log, this feature is still relatively simple but sets a foundation for future enhancements to track time and tasks more effectively.
I've taken note of your requirements, as soon as I have a clearer idea of implementation, I'll start working on them.

yeah, the editable task description is reasonable for sure, and I'll investigate the bug tomorrow. Thank you once again.

fixed 1.1.1

Hi @benjaminwestern
I've added task tracking in the new update, which should cover most of the cases we mentioned here. I'd be glad to hear any more feedback you have.

Thank you sir, I will keep enjoying your plugin and let you know :) thanks again!
b


enhancement Clarify user feedback messages

This PR attempts to enhance user feedback messages in the start_be, stop_be.sh, start_fe.sh, stop_be.sh commands.

Removes period at the end of some feedback messages for consistency.
Removes the use of "please", good to be polite but better to be clear and concise.
Uses consistent uppercase for PID.
Improves English wording in a few places.

run buildall

I can see there are some actions failing. Probably some failing tests. I tried to look into it but TeamCity requires a login.

I can see there are some actions failing. Probably some failing tests. I tried to look into it but TeamCity requires a login.

run buildall


QnA Maker Regex is problematic and will often fail

Cut and paste can introduce formatting problems (extra spaces.)
Multi-line escaped strings in code are more awkward than useful. We are expecting the programmer to format and escape this somehow?

Overall the feature seems to be more trouble than it is worth. Suggest removing feature.

and for consistency remove from JavaScript implementation as well

@johnataylor did you end up removing this form the code? If not do it.

@Stevenic verify the core was removed from JS https://github.com/Microsoft/botbuilder-js/issues/281


Why Coverlet use custom attribute ExcludeFromCoverage instead of ExcludeFromCodeCoverage

Hi guys!
Why Coverlet use the custom attribute ExcludeFromCoverage from its own namespace instead of using already exists in .NET Core attribute ExcludeFromCodeCoverage from namespace System.Diagnostics.CodeAnalysis?
To Coverlet ignore any class, I need to install the nuget package coverlet.msbuild in all projects with tests, right?

@Valeriy1991 no, you can create a ExcludeFromCoverage attribute and use that. Coverlet uses the name and not the full type so anyone will work.

Is there any reason to not support ExcludeFromCodeCoverage as well? That would make it easier to transition to Coverlet from other products.

Hello! I'm still work on adding support on both attributes: ExcludeFromCoverage from Coverlet and ExcludeFromCodeCoverage from System.Diagnostics.CodeAnalysis in my fork.

In case you're wondering where to make the change it's right here... (in Coverlet.Core.Instrumentation.Instrumentation)
        private void InstrumentModule()
        {
            using (var stream = new FileStream(_module, FileMode.Open, FileAccess.ReadWrite))
            {
                var parameters = new ReaderParameters { ReadSymbols = true };
                ModuleDefinition module = ModuleDefinition.ReadModule(stream, parameters);

                foreach (var type in module.GetTypes())
                {
                    if (type.CustomAttributes.Any(a => a.AttributeType.Name == "ExcludeFromCoverageAttribute" || a.AttributeType.Name == "ExcludeFromCoverage"))
                        continue;

                    foreach (var method in type.Methods)
                    {
                        if (!method.CustomAttributes.Any(a => a.AttributeType.Name == "ExcludeFromCoverageAttribute" || a.AttributeType.Name == "ExcludeFromCoverage"))
                            InstrumentMethod(method);
                    }
                }

                module.Write(stream);
            }
        }

Now available in https://www.nuget.org/packages/coverlet.msbuild/2.0.1

Hi :)
I cannot get this to work. If I include the ExcludeByAttribute property in my *.Tests.csproj file and add a property GeneratedCode to my generated code project then Coverlet still computes coverage for the generated code. Similarly if I add the property ExcludeFromCoverage to my generated code project coverage is again still computed for the project.
Here is my *.Tests.csproj file:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <IsPackable>false</IsPackable>
    <CollectCoverage>true</CollectCoverage>
    <CoverletOutputFormat>lcov</CoverletOutputFormat>
    <CoverletOutput>./Datahub/lcov.info</CoverletOutput>
    <ExcludeByAttribute>GeneratedCode</ExcludeByAttribute>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="coverlet.msbuild" Version="2.8.0">
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
      <PrivateAssets>all</PrivateAssets>
    </PackageReference>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.2.0" />
    <PackageReference Include="xunit" Version="2.4.0" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.0" />
    <PackageReference Include="coverlet.collector" Version="1.0.1" />
    <PackageReference Include="FluentAssertions" Version="5.10.0" />
  </ItemGroup>

  <ItemGroup>
    ... project references ...
  </ItemGroup>
</Project>

and here is my ProtoBuf.csproj file for the generated code:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <GeneratedCode>GeneratedCode</GeneratedCode>
    <ExcludeFromCoverage/>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Google.Protobuf" Version="3.11.2" />
    <PackageReference Include="Grpc" Version="2.26.0" />
    <PackageReference Include="Grpc.Tools" Version="2.26.0" PrivateAssets="All">
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
      <PrivateAssets>all</PrivateAssets>
    </PackageReference>
  </ItemGroup>

  <ItemGroup>
    <Protobuf Include="**/*.proto" />
  </ItemGroup>
</Project>

Am I missing something or what is wrong?
Thanks :)

@svenskmand please open new issue, we'll discuss there.

Ok, thanks :) I have opened a new issue here.


Update catalog for namespace pref

Motivation and Context

Addresses #80 to update prefixes to the catalog
How has this been tested?
Ensure no datasets come under Datasets not mentioned in pipeline:

Checklist

[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Assigned myself to the PR
[ ] Added tests to cover my changes

But for dataset, only the dot syntax would work, not the top-level key right?

Thank you! The context is very helpful to understand the  evolution of the design.


[metadata] add DelayInjectionMetadataStore

Motivation
Currently, we have FaultInjectionMetadataStore which can only inject errors for metadata store.
but there are some race conditions in metadata store, like #13004, and we don't have useful tools to address these.
Modifications
Add DelayInjectionMetadataStore with two key features.

Add a random delay (range can be configured ) in each metadata operation.
Add a condition delay with target operation and path, this type of delay will be removed once fired.

Verifying this change

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

This change added tests and can be verified as follows:

DelayInjectionMetadataStoreTest

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] doc
Doc provided in DelayInjectionMetadataStore.java

/pulsarbot run-failure-checks

@eolivelli Hi~ Please help take a look. Thank you so much.

@merlimat PTAL. I think we can discover more race condition issues in current metadata implementation with this DelayInjectionMetadataStore,


Addon will not start when no internet

Hi.
I have a system i plan to leave running in a location with no internet and use samba-backup to backup to a local server however samba-backup will not start when there is no internet.
It come up with the error Failed to validate addon configuration and in the logs has "Timeout on addons/15d21743_samba_backup/options/validate request"
Does anyone have any ideas on how I can resolve this?

I have the same error but my system is ONLINE. Did you ever figure this out?
I am gettign the same error when trying to save my samba backup configuration. How is it checking for itnernet access?
The sahre I am trying to back up to is onyl reachable via tailscale and this HA instance can reach it just fine.

I also mentioned this here: https://community.home-assistant.io/t/samba-backup-create-and-store-backups-on-a-samba-share/199471/672

Sorted with a complete reboot of the OS, not just restart of HA. Seems HA had a hiccup somewhere thinking it was offline.
Meanwhile, I was remotely connected to it from a different country and was struggling with this nonsense.


SlimerJS usage in headless (windowless) situations

Since there is the call for issues post 2.0 I wanted to raise this one. One of the major differences between SlimerJS and PhantomJS is the fact that SlimerJS is not headless and requires a windowing system of some sort to function. When run under Linux or FreeBSD there are situations where PhantomJS use will function without problem but SlimerJS will exit immediately because there is no X-Windows environment available to run Firefox/XulRunner in. SlimerJS suggests using xvfb-run in instances when under Linux (or OSX).
This situation can be tested for by checking process.env.DISPLAY (BASH variable $DISPLAY) and if it's  empty/null then there isn't Display available to run in.
There are three potential approaches I see to addressing this:

Document the requirement to using SlimerJS as the engine and make no code changes. It's likely this is a small use case (even though it did effect me, mine isn't a typical use scenario).

Document the requirement but also add a check to the DISPLAY variable when under Linux/FreeBSD/OSX  and throw a meaningful error on failure (or fall back on PhantomJS?).

Add in a check on the DISPLAY variable and when null/empty use xvfb-run as the child-process command, pushing the SlimerJS command as the first argument (or second since we would probably want to use the -a switch to have xvfb-run automatically grab the first open server number).

a. This would also require documentation to call out the dependency on xvfb and xvfb-run for the instances where headless operation is desired.
I'm willing to make the implementation attempt for option 3, but wanted to poll opinions before putting too much effort into it.

IMHO if problem can be solved without code change, code should not be changed. I think, adding info to readme will be enougth. PR is welcome. Readme is not in perfect state.

That's seems appropriate. I noticed a couple of (very) minor README errors so I'll add a note to the SlimerJS known issues section along with those and that should address it for the time being. If the need to implement it in becomes important enough we can always add it in later.


/.well-known/openid-configuration being automatically duplicated in configuration authority field

I am coming from using https://www.npmjs.com/package/@axa-fr/react-oidc-context in my CRA React app. I am migrating to Vite and changing the oidc library to this one.
My configuration has always used this format for the authority:
authority: "https://<my-keycloack-idp/realms/test/.well-known/openid-configuration"
Now after migrating, calls to my idp fail because the request is sent with the ending duplicated:
https://<my-keycloack-idp/realms/test/.well-known/openid-configuration/.well-known/openid-configuration
Any clues or knowledge of the source of the issue?

I can see in here that /.well-known/openid-configuration is appended to the OpenIdIssuerUrl
https://github.com/AxaGuilDEv/react-oidc/blob/5e0684f2ab86007abc5e76e0190a2b92982497ab/packages/react/src/oidc/vanilla/requests.ts#L8-L11

a comment would be good. Even better would be to make it configurable