UI should intercept/rewrite navigations to http://www.omex-library.org/

When clicking on a linked predecessor metadata value, the user is directed to http://www.omex-library.org/, which is just a placeholder namespace.
The UI should determine if the link corresponds to a valid reference in the biosimulations database and redirect there instead if that is the case.

The interception should be to http://www.omex-library.org/*.omex.

Actually, I think the API should just replace the http://www.omex-library.org/ with the hostname  of the API and edit the URL to point to the download link of the path. That way the metadata URIs will uniquely identify its subject and treating the URI as a URL will redirect to the resource itself.
@jonrkarr would biosimulators-utils need extensive changes to support this for things like validation or project generation?

http://www.omex-library.org/ is a constant in the library. This could be turned into a configurable variable. This would be fairly simple.
For hyperlinks, this can be modified in BioSimulations.


Move tests to the repository root level

I don't thing there's any need to bundle them in the built package. Smaller package sizes == happier devs and planet.
Dev reference: https://hynek.me/articles/testing-packaging/ | https://blog.ionelmc.ro/2014/05/25/python-packaging/#the-structure
Project structure:
src/
╰─ project_name/
   ╰─ ...
tests/
╰─ ...

<configuration files etc>

This sounds good to me!
Don't forget to update https://github.com/wagtail/cookiecutter-wagtail-package/blob/main/wagtail-{{ cookiecutter.project_name_kebab }}/setup.py#L23 to prevent the tests folder being packaged up an installed as a global "tests" app (and it might conflict with the test suite draftjs_exporter which does this).

Test were moved to the root level. Now to move package_name to src/package_name so it is never a top level import


i want show 1399/01/02 instated of 1399/1/2

i want show 1399/01/02 instated of 1399/1/2
i mean 2 character for day and month no 1 character

use jYYYY/jMM/jDD -> 1399/01/02
if you use jYYYY/jM/jD you will get something like this -> 1398/1/2

@matinzd thanks a lot


Add coverage, gha, jenkins server, documentation and forum badges

Signed-off-by: Suraj Singh<EMAIL_ADDRESS>Description
Adds badges to readme for different jobs listed below

GHA gradle check
GHA precommit
GHA validate pull request
Public jenkins gradle check
Documentation
Forum

Issues Resolved
#1323
Related
#850

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Moved badges after logo & change ordering of badges.


Bitcode error on Xcode 8

Hey there, I'm trying to use your library on Xcode 8 and any attempt at archiving an app bundled with it results in a Bitcode error on the libmysqlclient.a binary. The error is as follows:
ld: bitcode bundle could not be generated because '.../Pods/OHMySQL/OHMySQL/lib/ios/libmysqlclient.a(libmysql.o)' was built without full bitcode. All object files and libraries for bitcode must be generated from Xcode Archive or Install build for architecture arm64.

It seems the libmysqlclient.a binary needs to be rebuilt in Xcode >v8, otherwise Archiving will fail. All attempts to disable bitcode in settings have resulted in no changes. I'm currently compiling a new libmysqlclient.a binary with Xcode 8 with the hopes to solve this.
Any suggestions or alternative methods would be appreciated, thank you!

Hi,
I got it. Looks like I have to re-build with extra flags.
I will try to do that ASAP. I believe I will be able to do provide a fix till Monday.
Thanks for letting me know.

Thanks for the quick response!
On April 22, 2017 8:27:39 AM PDT, Oleg<EMAIL_ADDRESS>wrote:

Hi,
I got it. Looks like I have to re-build with extra
flags.
I will try to do that ASAP. I believe I will be able to do provide a
fix till Monday.
Thanks for letting me know.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/oleghnidets/OHMySQL/issues/6#issuecomment-296380763

Hi there,
Hope I fixed that. But before the release, I want to make sure that it works. Develop branch contains the latest changes.
Could you, please, try it out pod 'OHMySQL', :git => 'https://github.com/oleghnidets/OHMySQL', :branch => 'develop' and let me know a result?

Yes, it working perfectly now. Thank you!


Feature: Nested stories

I have a lot of different states of components, a good solution would be nested story directories.
Depth ~ 3-4
Custom Library > BaseComponent > SubComponent > SubComponent with different state

We are tracking this here: https://github.com/kadirahq/react-storybook/issues/151


Datetime control in outlook actionable messages adaptive cards to record back-dated datetime transactions

Hi,
Need a  Date-time control in outlook actionable messages adaptive cards to record back-dated transactions with time.
So that user entered date-time value will be converted in to GMT with reference to client time zone.
Thanks,
Vishnukumar M

@Vishnuvk33 you can use a Date input plus make your own time input using two ChoiceSet (for hours and minutes). Ideally the AC Time input would be supported in Outlook, but at the moment it is not.
As I outlined here, the team is working on a fix that will give you the time zone offset for the date value.
All in all that should allow you to do what you want.

@Vishnuvk33 please do take a look at @dclaux 's response above and reactivate or file a separate one in case this doesn't work. I am dup'ing this to #4186


Add constants overriding due to fast forward argument value

Coverage remained the same at 82.989% when pulling ae7fc36a2a6edf4e9e6169f8b259a03b188d5e0b on Scandie:a552552396982565_fast_forward_internal_test into 143dcfb8ffd63a2269fa76cc241b2e3d098d91a7 on openprocurement:sandbox.


Add credentials_cookie as valid option

Configures the JS API to write a secure cookie on your application domain by adding credentials_cookie: true to the framework initialization script tag:
https://developer-programs.linkedin.com/documents/exchange-jsapi-tokens-rest-api-oauth-tokens
Adding the option to the config map with default value false should be enough.

Thanks for fetching the pull. Is it possible to make a new release of the library, so I can include your repo in my project?


VAC ban date bugged in list

In one of my lists, the VAC ban date is displayed incorrectly:
https://imgur.com/kTUvPzW
https://vacstat.us/list/20806 page 14

You're not giving any info. Please link me the list and profile of the user that is glitched. The list can be private.
For now im closing this.

sorry, I hit enter before filling out the description, but edited it in a minute ago.

I think this issue might be from when I first introduced better vac ban system in the database. I probably made everyone's ban date something like 0000-00-00 00:00:00.
I recently updated the code to not check on the date. I totally forgot about this.

Thanks for your fast reply. Unfortunately, the Vac ban date is still displayed as "Nov 30 -0001" in the list for me.


Using CloudSDK through CAP4 in Kubernetes context

Hello colleagues,
We are building the SAP Graph. It allows customers to bring their heterogenous systems and APIs into one aligned API. The SAAS app runs in a Gardener provisioned Kubernetes cluster on Google Cloud Platform.
Authentication Use-case
Token-Based Authentication with external systems (e.g. S4, C4C) through CAP4, which uses the CloudSDK.
Details
Our application, running in our GCP K8 cluster, should retrieve destinations bound to a SAP CP account and uses the user_token/client_credentials flow to get access to the source Business system (S4, C4C).
Current State
Authentication is working locally as a prototype with help of the app-router. Working source systems are S4 and C4C in DLAB.
For us to continue it'd be amazing if you can guide us through the remaining questions.
Open Questions

How does the XSUAA handshake work when deployed?
Is the app router supposed to be deployed as well?
Documentation is not entirely clear: https://sap.github.io/cloud-sdk/docs/java/guides/cloud-foundry-xsuaa-service vs https://github.wdf.sap.corp/pages/cap/node.js/authentication#jwt
Does/Should the CloudSDK configuration differ when deployed?

Hello @JohannesRu,
before you start a disclaimer, that I do not know the K8s part of cloud foundry really well. It is on our agenda to provide a smooth developer experience also there. Especially since we see increasing demand from various people. So my knowledge is related to the classic cloud foundry but I hope this also applies to the situation you have.

How does the XSUAA handshake work when deployed?
Is the app router supposed to be deployed as well?

I think these two go hand in hand (I am not 100% sure what you mean by handshake but I will write something about the general flow). The approuter is the central entry point for your application. From there all requests are redirected and the auth flow is started with the help of the XSUAA service by the approuter. The XSUAA service knows about the role templates and scopes and this information is added to issue a JWT by the identity provider to which the request is redirected by the approuter. Later the XSUAA is also used to issue service specific token, but more to this in the next section. In the end you have a JWT in your request. The issued JWT contains the tenant, user and scope information.

Documentation is not entirely clear: https://sap.github.io/cloud-sdk/docs/java/guides/cloud-foundry-xsuaa-service vs https://github.wdf.sap.corp/pages/cap/node.js/authentication#jwt

The second link does not work unfortunately. But I will explain the JWT thing a bit. Let's look at the destination case. We start with a JWT issued for the application. The application has a binding to the destination service. Hence the SDK can get an access token for the destination service (here I see a potential issue see below) via the XSUAA service. With this service token the destination is fetched. The destination service takes care of building the necessary tokens (OAuthSamlBearer, OAuthClient, Basic....) to access the target system. The SDK just takes this information and puts it in the request headers or destination object.
The way the SDK gets an access token for the destination service is based on the clientID and clientSecret of the bound destination service. These are available via some environment variables (VCAP_SERVICES). Here I do not know if these are present in your Kubernetes cluster.

Does/Should the CloudSDK configuration differ when deployed?

If you have an approuter which triggers the auth flow and adds the JWT to your request AND the VCAP_SERVICES variables are present it does not matter if you are deployed on CF or local. The destination service is just in the internet and can be reached from everywhere. However, in most cases developers do not have an approuter locally and use some environment variables to have it working locally.
I hope this helps a bit.
Best
Frank

Hello @JohannesRu,
before you start a disclaimer, that I do not know the K8s part of cloud foundry really well. It is on our agenda to provide a smooth developer experience also there. Especially since we see increasing demand from various people. So my knowledge is related to the classic cloud foundry but I hope this also applies to the situation you have.

How does the XSUAA handshake work when deployed?
Is the app router supposed to be deployed as well?

I think these two go hand in hand (I am not 100% sure what you mean by handshake but I will write something about the general flow). The approuter is the central entry point for your application. From there all requests are redirected and the auth flow is started with the help of the XSUAA service by the approuter. The XSUAA service knows about the role templates and scopes and this information is added to issue a JWT by the identity provider to which the request is redirected by the approuter. Later the XSUAA is also used to issue service specific token, but more to this in the next section. In the end you have a JWT in your request. The issued JWT contains the tenant, user and scope information.

Documentation is not entirely clear: https://sap.github.io/cloud-sdk/docs/java/guides/cloud-foundry-xsuaa-service vs https://github.wdf.sap.corp/pages/cap/node.js/authentication#jwt

The second link does not work unfortunately. But I will explain the JWT thing a bit. Let's look at the destination case. We start with a JWT issued for the application. The application has a binding to the destination service. Hence the SDK can get an access token for the destination service (here I see a potential issue see below) via the XSUAA service. With this service token the destination is fetched. The destination service takes care of building the necessary tokens (OAuthSamlBearer, OAuthClient, Basic....) to access the target system. The SDK just takes this information and puts it in the request headers or destination object.
The way the SDK gets an access token for the destination service is based on the clientID and clientSecret of the bound destination service. These are available via some environment variables (VCAP_SERVICES). Here I do not know if these are present in your Kubernetes cluster.

Does/Should the CloudSDK configuration differ when deployed?

If you have an approuter which triggers the auth flow and adds the JWT to your request AND the VCAP_SERVICES variables are present it does not matter if you are deployed on CF or local. The destination service is just in the internet and can be reached from everywhere. However, in most cases developers do not have an approuter locally and use some environment variables to have it working locally.
I hope this helps a bit.
Best
Frank

Documentation is not entirely clear: https://sap.github.io/cloud-sdk/docs/java/guides/cloud-foundry-xsuaa-service vs https://github.wdf.sap.corp/pages/cap/node.js/authentication#jwt

The second link does not work unfortunately.

The external link to the CAP docs about authentication is here: https://cap.cloud.sap/docs/node.js/authentication#jwt

Documentation is not entirely clear: https://sap.github.io/cloud-sdk/docs/java/guides/cloud-foundry-xsuaa-service vs https://github.wdf.sap.corp/pages/cap/node.js/authentication#jwt

The second link does not work unfortunately.

The external link to the CAP docs about authentication is here: https://cap.cloud.sap/docs/node.js/authentication#jwt

One thing also just came to my mind:

Working source systems are S4 and C4C in DLAB.

I think the DLAB systems are only reachable from the intranet. You can test this if you exit the VPN and your local setup stops working. If this is the case you must use as source systems either proper cloud systems which are exposed in the internet or in case of onPremises systems a cloud connector should be attached.
Also keep in mind that this repo is publicly visible and no SAP internal landscape details should be exposed here e.g. that we have something called DLAB. Some mean people could use these information to get more credibility when they make some social engineering attempt.
Best
Frank

One thing also just came to my mind:

Working source systems are S4 and C4C in DLAB.

I think the DLAB systems are only reachable from the intranet. You can test this if you exit the VPN and your local setup stops working. If this is the case you must use as source systems either proper cloud systems which are exposed in the internet or in case of onPremises systems a cloud connector should be attached.
Also keep in mind that this repo is publicly visible and no SAP internal landscape details should be exposed here e.g. that we have something called DLAB. Some mean people could use these information to get more credibility when they make some social engineering attempt.
Best
Frank

We had a meeting today but for the rest of the world I put some information here:

The app router is not mandatory, but you need a valid JWT including all fields in scope, issuer etc. Note that the mandatory fields depend on the thing you want to do. If you plan to use some user dependent OAuth flow your JWT must be a user specific one => @JohannesRu will compare the working JWT generated by the approuter/xsuaa and see which fields are missing
Currently the colleagues will focus on cloud systems as target. For onPremises systems the connectivity proxy will make things a bit more complicated and we would need most likely also a change in the SDK see ADR
We assume the the @sap/xsenv will also fetch the service credentials in the K8s runtime

We had a meeting today but for the rest of the world I put some information here:

The app router is not mandatory, but you need a valid JWT including all fields in scope, issuer etc. Note that the mandatory fields depend on the thing you want to do. If you plan to use some user dependent OAuth flow your JWT must be a user specific one => @JohannesRu will compare the working JWT generated by the approuter/xsuaa and see which fields are missing
Currently the colleagues will focus on cloud systems as target. For onPremises systems the connectivity proxy will make things a bit more complicated and we would need most likely also a change in the SDK see ADR
We assume the the @sap/xsenv will also fetch the service credentials in the K8s runtime

Hey Frank,

thanks for the summary. Yes, we managed to identify the missing scope "uaa.user" as the missing piece. With that in place CAP+CloudSDK works as expected and we can happily skip AppRouter. Thanks again for helping us here.

Looking at the release notes for Java and reading the lates SAP guides "uaa.user" may be redundant, is there any plan to align accordingly in the JS version? Else we will simply incorporate the requirement into our documentation as well.
https://sap.github.io/cloud-sdk/docs/java/release-notes-sap-cloud-sdk-for-java/#improvements-19

Regarding @sap/xsenv, we are currently evaluating the setup and will come back to you when we have figured it out....
Again, thanks for the speedy and efficient help. Hope we stay in touch on the subject ✌

I hope so too. Great that we could of help and that you also reminded us about the new token grant. I will make an issue in the backlog so that we remember to implement it. Seems like an easy change or additional method.


[engsys] upgrade dependency glob to ^8.0.0

for eslint-plugin-azure-sdk and service-bus
remove unused glob dependencies from core-http

API change check
API changes are not detected in this pull request.


Add tests to demonstrate issue with writing files into symlinked directories

Current master has some regressions when trying to write files into symlinked dirs, e.g
FileUtils.writeStringToFile(new File("symlink-dir/target-file.txt"), "content", "UTF-8")

with
.
├── symlink-dir -> target-dir
└── target-dir
    └── target-file.txt

FileUtils.openOutputStream (used by 2.11 implementation) allows this (per the test - which passes)
PathUtils.newOutputStream and PathUtils.createParentDirectories do not allow this currently (per added tests)
The switch from the former to the latter inside FileUtils seems to have broken copyToFile, copyInputStreamToFile, copyURLToFile, write, writeStringToFile, writeLines, touch compared to 2.11 when working with a target file that would be written inside an existing symlinked dir

As discussed at https://github.com/apache/commons-io/pull/319#issuecomment-1025934935 writing will currently fail with errors like
java.nio.file.FileAlreadyExistsException: /var/folders/3s/x7hkyjrd0y76fsw7417x7j1h0000gn/T/junit1985630074694639589/FileUtilsTest8511279228626771736/symlinked-dir
	at java.base/sun.nio.fs.UnixException.translateToIOException(UnixException.java:94)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
	at java.base/sun.nio.fs.UnixFileSystemProvider.createDirectory(UnixFileSystemProvider.java:388)
	at java.base/java.nio.file.Files.createDirectory(Files.java:694)
	at java.base/java.nio.file.Files.createAndCheckIsDirectory(Files.java:801)
	at java.base/java.nio.file.Files.createDirectories(Files.java:747)
	at org.apache.commons.io.file.PathUtils.createParentDirectories(PathUtils.java:339)
	at org.apache.commons.io.file.PathUtils.newOutputStream(PathUtils.java:1066)
	at org.apache.commons.io.FileUtils.newOutputStream(FileUtils.java:2454)
	at org.apache.commons.io.FileUtils.writeStringToFile(FileUtils.java:3543)
	at org.apache.commons.io.FileUtils.writeStringToFile(FileUtils.java:3528)
	at org.apache.commons.io.FileUtilsTest.testWriteStringToFileIntoSymlinkedDir(FileUtilsTest.java:3096)

I don't know how to address this while still using NIO-based PathUtils.createParentDirectories and not breaking the contract. Users possibly expect PathUtils.createParentDirectories to fail if there is already a symlink to another directory there as this is existing behaviour since 2.9.0.
The current implementation uses Files.createDirectories(dir) underneath which does not seem to allow treating the symlinked dir as a valid dir, as when it checks directories it uses LinkOption.NOFOLLOW_LINKS which is not parameterised to allow user to control it. If we want to keep using NIO for this, I suspect we would need our own version of Files.createDirectories
    private static void createAndCheckIsDirectory(Path dir,
                                                  FileAttribute<?>... attrs)
        throws IOException
    {
        try {
            createDirectory(dir, attrs);
        } catch (FileAlreadyExistsException x) {
            if (!isDirectory(dir, LinkOption.NOFOLLOW_LINKS))
                throw x;
        }
    }

Marking as draft, because it deliberately has failing tests.

Hello @chadlwilson
I've brought in the tests separately and the runtime is better, would you please check your use case, update this PR or create a new one? TY!

Hi @garydgregory - how do you suggest the PR should be updated? I had added tests to try and ensure we don't keep getting these types of regressions from refactors, but didn't test all possible use cases, so that's about it.
GoCD's regression suite at least passes now against master/SNAPSHOT.
It's a bit too time consuming trying to reconcile what else might need to be done since there doesn't seem to have been a cherry-pick + fix done of my commit, and there are just a sequence of unrelated commits on master without PR or linked JIRA issue. So I'll just close this in lieu of more specific feedback.


First button drawn in wrong position when Camera2D is zoomed and in Forward+/Mobile mode

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Godot v4.1.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - integrated Intel(R) UHD Graphics 620 (Intel Corporation; <IP_ADDRESS>81) - Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz (8 Threads)
Issue description
When on my laptop, I noticed the first button of the buttons in my project was being drawn in totally the wrong location, but only on my laptop. And if I switched it to Compatibility mode, it worked correctly. See the following screenshots of Forward+ and Compatibility mode, respectively.

In playing around with things, I determined that the zoom of the Camera2D is what causes the first button to be drawn in the wrong position. It's not the button itself either. If I switch the order of the two buttons, so the other button is first, it will now happen to the other button instead. Additionally, the text of the button is drawn in the correct position. The rest of the button, however, is what gets drawn in the incorrect position.
My laptop has no dedicated GPU, which I suspect is where the problem lies. It does not happen when using either of my desktops, both of which have a dedicated GPU (2070 SUPER and 3090).
I created a sample project that successfully reproduces this bug and have attached it. I tried to make it match what I had in my original project in terms of layout, though stripped out anything else irrelevant.
Steps to reproduce

Place two buttons as children of a CanvasLayer, positioned some place where it overlaps the camera's viewport.
Make sure the Camera2D has a non-(1,1) zoom.
Run the project and notice the first button is drawn in the wrong position, and seems to be relative to the camera and its zoom.

Minimal reproduction project
ButtonBug.zip

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/58314.


Update Treatment Validation doc with examples

What this PR does / why we need it: Adding some examples to the docs for the Go Template based treatment validation, since it's an area that we often get questions on.

@deadlycoconuts thanks for the super quick review!


Add REST catalog integration tests

Feature Request / Improvement
#1434 created tests/integration/test_rest_catalog.py for integration tests with the REST catalog. Previously REST catalog tests were using mocked requests in tests/catalog/test_rest.py.
Let's add more integration tests!

What's going to be the implementation strategy for this integration test? I'd be happy to use my current momentum with REST Catalog to implement the rest of the tests.
My current approach would be to migrate the test_rest.py unit tests from mock to the integration test, but I'm wondering if there's any changes or additions we could include in our new integration test flow. Let me know your thoughts

My current approach would be to migrate the test_rest.py unit tests from mock to the integration test

I think thats a great first step! Generally, testing via mocked requests is not as resilient as testing against a REST server. Let's move all the mocked tests into the integration rest test.
Would you like to work on this?

Sounds good! Yes, I'll work on it


clarifying async support for older python versions

per https://github.com/Azure/azure-iot-sdk-python/blob/master/azure-iot-device/README.md

@gregman-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cartertinney

@gregman-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cartertinney does this look good now?

#sign-off

@gregman-msft: I'm sorry - only the author of this article, @robinsh, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off This looks great. Thanks @gregman-msft for making the change.


Support dark mode in auth flow

Resolves #723 for the Swift code authentication flow.

I'll check and let ya know

LGTM. Does Xcode 10.x support the new key/values in the sample storyboard file?

So, it opens with no problem. But, some of the constraints and such render oddly. But, it won't build due to the [UIColor labelColor] use. Essentially, due to the iOS 13 SDK not being available in Xcode 10. Thoughts?

I suppose I could use #ifdef __IPHONE_13_0

No, no need to update the code. Let's keep this PR open until we update Xcode on Travis to support building the new IB file.

Is there an ETA to get this merged? Now that iOS 13 has officially launched, not having this is preventing our app from fully supporting dark mode.

@morganchen12 I'm also curious about when it will be merged, now that Xcode 11 is generally available via the Mac App Store.

I plan on merging this week. Bumping the dependency in Travis should take less than a day.

Thanks @cbess for contributing!


Use external id when reconciling routes

The node name used by Kubernetes internally doesn't match what is returned from AWS when querying route tables. This changes the reconciliation step to match against the node's external id as returned by AWS.
I know we're trying to make the Kubernetes node name match what is used internally by AWS, but that seems to be blocked on some other issues where the node name is expected to be resolvable. As such, this code will probably need to be revised again when those changes land.
@justinsb PTAL
This is an attempt to resolve #11979 which was experienced by myself and another user.

ok to test

But I'm pretty sure this breaks cluster deployment on GCE.

@cjcullen yeah, you're probably right. At first glance it seemed like only AWS was using this but I see that's not true at all.
That said, we might get lucky since the default value for external id is set to the node name when empty:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/v1/types.go#L1256

Our gce cloudprovider uses a numeric ID for ExternalID() which unfortunately won't work here.
Would it be possible to make the aws.ListRoutes function fill out the cloudprovider.Route object with the node name using ExternalID()? Then reconcile() would see the routes it expected to see.

@cjcullen Good idea, the instance metadata has a field PrivateDnsName which matches the node name internally. The downside is that we'll make a lot of request to describe the nodes.
I had come up with a far more convoluted approach involving adding a new method to the Routes interface to determine the correct way to access the node identifier. This seems better and stays isolated to AWS. I'll work on it and push again soon.

@iterion I think isolated to aws is easier for now. It'd be better to have an explicit GetWhatMyCloudProviderKnowsMeBy() function, but that's one of many pieces of a big refactor of our cloudprovider libraries.

@cjcullen Pushed your recommended changes. It seems that e2e fail is unrelated, but I'm not sure if that's a flaky test or not. I'm going to test this on AWS and I'll get back to you once I validate this fixes the issue there.

@cjcullen Pushed a small fix. Testing went well on my AWS cluster so this is ready for review.

@k8s-bot test this please

I'm not sure why your updated change would have any effect on the GCE e2es.

This makes sense to me now.  I will test locally.
I agree that we will be making a disappointing number of calls to map instanceid -> name.  We could cache them, and I like your other suggestions of logging all calls.  And I do want to fix it so that name == instanceid again (which I think is how this bug crept in).

@justinsb I updated this to use orEmpty as you suggested. I'm happy to work on improved logging, but I'll probably make a new PR for that. As I mentioned in #11688 I think these issues are all related and that this is likely the actual root cause.
@cjcullen I'm not sure what to do about the failing e2e, to me they seem to be unrelated and I'm not sure how to investigate. If you have any leads I'd gladly try to track them down.

Awesome @iterion ... just running the e2e tests now.  (Had a flake on my last test).
And sorry, I agree that logging etc should be not in this PR.  I meant to say that, but looks like I lost my chain of thought somewhere along the way.
I will LGTM this the minute I get an e2e pass.

E2e's look happy again (definitely wasn't your problem). LGTM. Can you squash commits?

Squashed and pushed.

e2e worked for me on AWS: LGTM!

Thanks for squashing. If it LGT Justin, it LGTM.

Yes, please cherry-pick this – for us, a small K8s evaluation cluster causes enough requests to trip rate-limits that affect production services, so this is quite severe.

Can this be closed now?

I think merged == closed on a PR?
But we have cherry-picked it, so I don't think there is anything outstanding here.


Distinguish between system and normal headers.

Distinguish between system header files (which are less strictly checked by the compiler and tools like clang-tidy) and normal header files. System header files should be used for header files which are not part of the current project/repo.
This allows us to use properly strict compiler settings.

Note to self: See if cfgbuild.cmake is required


Preserve Undefined values in arrays in MarkLogic

Fixes an issue where undefined values were being collapsed in Arrays meaning they wouldn't be as large as expected and cause ProjectIndex to fail in those cases.
Also improve handling of inputs to qscript:length to detect the input type more reliably.
These fixes allow another handful of regression test to pass for MarkLogic.

Dealing with undefined values in arrays is an open problem – we don’t store or return undefined values (well Data allows us to, unfortunately – and I think there’s a comment somewhere stating that NA should only show up in arrays), but at the same time, we can’t just remove their entries like we can in a map (since it changes the indices of any later values).

Nice progress.
A couple of the sticky issues are coming up now, and I think we need to start addressing them.

@mossprescott Finally got the green light, I've shuffled around the mongodb tests and #1642 to track the marklogic field name issue (which I'll solve in a separate PR).


add total velocity to table [CPP-908]

add speed to solutions table.
not an ideal display for the HUD / quickview but includes some additional values.
an idea for the quickview might be to just allow rearranging of entries in table or focusing multiple values when shift clicking etc and will highlight / enlarge. this might be the best option as it does not add additional UI changes.

Frontend and Release Workflow Started here


Move GTFS field names from table loader to entity classes

E.g., now use GtfsStop.STOP_ID_FIELD_NAME instead of GtfsStopTableLoader.STOP_ID_FIELD_NAME.
Motivation

Table loader classes should be only used for loading data tables. They should not be used by validator classes.
Our final goal is to stop generating table loader classes and instead make them usual classes that use a GTFS schema as a parameter. GTFS field names have to be generated, so we move them to entity classes that are generated anyway.

Thanks for reviewing!


Add variable validation to only allow 1 item (cbr_rule) in the list.

Closing this, as this will be for every module.


Collect W8/W9

Per tax law we need to collect a W8/W9 from a user before they exceed $600 in payouts. Below this limit the user filling out the from is completely voluntary. If they don't fill it out before the $600 in payouts we have to stop payouts and cut contracts until they have completed it.
Forms:

W9 - To be completed if they are in the US
W8 - To be completed if they are international

@boshevski Need you to prototype our those forms in the interface.
Todo:

[ ] @boshevski Prototype design W8 form
[ ] @boshevski Prototype design W9 form
[ ] Discuss method to encrypt/send info to Storj Labs

Once this is complete it will be implemented in https://github.com/Storj/driveshare-gui/issues/114


Validation generation for SSDK

Description of changes:
This can be broken up if desired, of course.
This adds:

TypeScript implementations of validators for each constraint trait
Generation of low-level validation methods for every modeled type when the SSDK is enabled.
-- I still don't know how to generate paths to members of sets, so I've punted on the path stuff in the design for now.
An integ test subpackage that generates TS code from a model and then runs tests against it. It works, but is not hooked up to the main build of smithy-typescript.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Forgot to mention -- also did not do type validation. That happens during deserialization in prior art, and generates a very different exception. I'm not sure where it fits in here.

That happens during deserialization

Does it? I'm pretty sure if a thing is modeled as a string it'll just take your word for it. Anywhere else it would have to cast or use methods that don't exist on strings (hopefully)

To summarize offline discussion: we still need the type validation, but it should be a separate process.


admin character panel

About the PR

adds a character panel for admins to use to look up a player's current character's description, records, and event preferences to help with game master events. confidential records can come in another pr
Why / Balance

because currently looking at a player's preferences is pretty difficult due to the requirement of toolshed, this also consolidates most of a current character's information into one panel for an admin to get a quick overview of.
Technical details

to-do: this
Media

to-do: also this
Requirements

[ ] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

for admins: adds the character panel to more easily see details about a player's current character

@dffdff2423 regarding the link of the issue: as i'm working on this, it's currently going towards the panel only displaying the player's current in-round character, not being able to select from all of a player's profiles as is ideal in the issue. is that something that can just be added later, and would the current character data (which should be gotten from the db either way) be good enough for now?

Yep! That is perfectly fine. I don't expect adding the ability to view other profiles will be that hard to add if you are already fetching them from the DB.


1.8.1 whats new bugs

to address
https://github.com/SciTools/scitools.org.uk/pull/109
@ajdawson please may you have a look and let me know if you think this is sufficient
I have picked fixes from:
https://github.com/SciTools/iris/commits/v1.8.1rc1
starting from Apr 15, 2015  when 1.8.0 was tagged

You'll need to change the Release part at the top too I think, to 1.8.1rc1 or whatever you think is best, as well as the date (I think that is how we do it, only list the current bugfix version number and release date).

It'd be better if all the entries didn't start with Fix in. Perhaps these could read more naturally? For example:
The PP loader carefully handles floating point errors in date time conversions to hours.

The new GRIB2 loading code is fixed to enable the loading of various data
representation templates whose unpacking is handled by the GRIB API.

However, this are at your discretion, I don't really mind.

i agree, the language isn't pleasing to read; i've tried a different style

That reads a lot better.
I just noticed that the section heading are not the same as in the 1.7 what's new file (1.7.0 has a higher level bugs fixed section than further bugfix releases, which are subsections of bugs fixed). I don't know if they should be the same or not...

same ordering as last time is fine by me

travis is being very slow today.  Are you willing to merge this without awaiting CI approval, or would you prefer to wait until tomorrow (as I am nearly out of time) and a green travisTick?
many thanks
marqh


#14316: Refactoring moreh_helper function

Ticket
https://github.com/tenstorrent/tt-metal/issues/14316
Problem description
The moreh_helper functions are helper functions that were used back in the TT-eager days.
These functions are all defined in the primary namespace, which causes the functions in ttnn::operations to call the helper functions with unnecessarily long namespaces like the following.
    tt::operations::primary::check_tensor(params_in, "moreh_adam", "params_in");
    tt::operations::primary::check_tensor(grad, "moreh_adam", "grad");
    tt::operations::primary::check_tensor(exp_avg_in, "moreh_adam", "exp_avg_in");
    tt::operations::primary::check_tensor(exp_avg_sq_in, "moreh_adam", "exp_avg_sq_in");

The reason the moreh_helper functions are in the primary namespace is that, during the TT-eager days, the ops were in the primary namespace. Now that we are using ttnn, there is no reason for the moreh_helper functions to remain in the primary namespace.
What's changed

I will move the moreh_helper functions to the ttnn::operations namespace.

I will clean up unnecessary code and modify it as shown below.

change moreh_helper function name to avoid duplicated name.

moreh_helper::split_work_to_cores-> moreh_helper::split_work_to_cores_wt_core_range

    check_tensor(params_in, "moreh_adam", "params_in");
    check_tensor(grad, "moreh_adam", "grad");
    check_tensor(exp_avg_in, "moreh_adam", "exp_avg_in");
    check_tensor(exp_avg_sq_in, "moreh_adam", "exp_avg_sq_in");

I'll add ttnn::operations:: to the function calls for ops still in the primary namespace.

Checklist

[x] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[x] New/Existing tests provide coverage for changes

@SeanNijjar @ntarafdar @sjameelTT
Would you guys review the changes for this file?
ttnn/cpp/ttnn/operations/reduction/prod/device/prod_nc_program_factory.cpp


Added 'NULL' option, it means no resize!

Example: I want user to upload file 128x128 and no other sizes. But method resize() is able to decrease size of file > 0px. Thats problem, i want use the same file as user uploaded, no 127x127. When is photo opened array with sizes is filled with origin values.

I'm not sure if I understand it, but when user upload image smaller which less then 128x128px image will be enlarged.
It's problem?

Incorrect implementation

That's true, anyway thanks for being kind to solve my problem :)


Fix single core coverity violations in FreeRTOS-Kernel.

Fix single core coverity violations in FreeRTOS-Kernel.
Description

Fix single core coverity violations in FreeRTOS-Kernel.
Test Steps

Clone my forked FreeRTOS repo.
cd FreeRTOS/
Update the compiler configuration in Coverity
cov-configure --force --compiler cc --comptype gcc
cov-configure -co /usr/bin/cc -- -m32 -std=gnu90
Create the build files using CMake in a build directory
cmake -B build -S FreeRTOS/Test/Coverity
Go to the build directory and copy the coverity configuration file
cd build/
Build the (pseudo) application
cov-build --emit-complementary-info --dir cov-out make
Go to the Coverity output directory (cov-out) and begin Coverity static analysis
cd cov-out/
cov-analyze --dir . --coding-standard-config ../../FreeRTOS/Test/Coverity/coverity_misra.config
Format the errors in HTML format so that it is more readable while removing the FreeRTOS-Kernel directory from the report

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

There are some changes. This PR should target coverity violation only. Suggest we change the logic in other PR.
!= pdFALSE
=>
== pdTRUE


出现NPU报错

C++ Call Stack: (For framework developers)

mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_data_queue.cc:269 QueryQueueSize
Line of code : 81
File         : mindspore/ccsrc/minddata/dataset/util/task.cc
[ERROR] MD(41,ffe9f2ffd1f0,python):2023-08-21-06:23:16.639.874 [mindspore/ccsrc/minddata/dataset/util/task_manager.cc:223] InterruptMaster] Task is terminated with err msg (more details are in info level logs): Exception thrown from dataset pipeline. Refer to 'Dataset Pipeline Error Message'. Unable to query real-time size of Mbuf channel: eea1e8a8-3fea-11ee-b4c5-6ce874fff76f, error code: 507899

Ascend Error Message:

EL9999: Inner Error!
EL9999  [drv api] halQueueQueryInfo failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueQueryInfo][FILE:npu_driver.cc][LINE:3238]
TraceBack (most recent call last):
rtMemQueueQueryInfo execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rtMemQueueQueryInfo failed, device is 0, qid is 1[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)

运行python tools/train.py --config configs/det/dbnet/db++_r50_icdar15.yaml命令报错，用的ReCTS数据集，已经用rects.md中的命令运行出det_gt.txt
root@manas-train-node01:/data/mindocr# python -c "import mindspore;mindspore.set_context(device_target='Ascend');mindspore.run_check()"
MindSpore version:  2.0.0rc1
The result of multiplication calculation is correct, MindSpore has been installed on platform [Ascend] successfully!

Start training... (The first epoch takes longer, please wait...)
[WARNING] MD(45501,fff200ff91f0,python):2023-08-21-08:14:34.658.871 [mindspore/ccsrc/minddata/dataset/engine/datasetops/data_queue_op.cc:903] DetectPerBatchTime] Bad performance attention, it takes more than 25 seconds to fetch a batch of data from dataset pipeline, which might result GetNext timeout problem. You may test dataset processing performance(with creating dataset iterator) and optimize it.
[ERROR] MD(45501,fff200ff91f0,python):2023-08-21-08:14:40.677.158 [mindspore/ccsrc/minddata/dataset/util/task_manager.cc:223] InterruptMaster] Task is terminated with err msg (more details are in info level logs): Exception thrown from dataset pipeline. Refer to 'Dataset Pipeline Error Message'. Unable to query real-time size of Mbuf channel: 8068338c-3ffa-11ee-8db2-6ce874fff76f, error code: 507899

Ascend Error Message:

EL9999: Inner Error!
EL9999  [drv api] halQueueQueryInfo failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueQueryInfo][FILE:npu_driver.cc][LINE:3238]
TraceBack (most recent call last):
rtMemQueueQueryInfo execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rtMemQueueQueryInfo failed, device is 0, qid is 1[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)

C++ Call Stack: (For framework developers)

mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_data_queue.cc:269 QueryQueueSize
Line of code : 81
File         : mindspore/ccsrc/minddata/dataset/util/task.cc
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.174.718 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:742] GetDumpPath] The environment variable 'MS_OM_PATH' is not set, the files of node dump will save to the process local path, as ./rank_id/node_dump/...
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.204.436 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:1390] DeleteDumpFile] Delete dir /data/mindocr/rank_0/node_dump/GetNext.GetNext-op8<IP_ADDRESS>2604994242540.output.0.DefaultFormat.npy failed!
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.214.124 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:1390] DeleteDumpFile] Delete dir /data/mindocr/rank_0/node_dump/GetNext.GetNext-op8<IP_ADDRESS>2604994580026.output.3.DefaultFormat.npy failed!
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.224.038 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:1390] DeleteDumpFile] Delete dir /data/mindocr/rank_0/node_dump/GetNext.GetNext-op8<IP_ADDRESS>2604994511768.output.2.DefaultFormat.npy failed!
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.233.914 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:1390] DeleteDumpFile] Delete dir /data/mindocr/rank_0/node_dump/GetNext.GetNext-op8<IP_ADDRESS>2604994443519.output.1.DefaultFormat.npy failed!
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.243.362 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:1390] DeleteDumpFile] Delete dir /data/mindocr/rank_0/node_dump/GetNext.GetNext-op8<IP_ADDRESS>2604994650171.output.4.DefaultFormat.npy failed!
[ERROR] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.243.473 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:760] DumpTaskExceptionInfo] Task fail infos task_id: 2, stream_id: 3, tid: 45501, device_id: 0, retcode: 507011 ( model execute failed)
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.409 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:769] DumpTaskExceptionInfo] Dump task error infos (input/output's value) for node:[Default/GetNext-op866], save path: ./rank_0/node_dump,
The function call stack:
In file /usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/train/dataset_helper.py:94/        outputs = self.get_next()/
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.438 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:776] DumpTaskExceptionInfo] GetNext error may be caused by slow data processing (bigger than 20s / batch) or transfer data to device error.
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.453 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:778] DumpTaskExceptionInfo] Suggestion:
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.467 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:779] DumpTaskExceptionInfo]     1) Set the parameter dataset_sink_mode=False of model.train(...) or model.eval(...) and try again.
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.480 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:781] DumpTaskExceptionInfo]     2) Reduce the batch_size in data processing and try again.
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.493 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:782] DumpTaskExceptionInfo]     3) You can create iterator by interface create_dict_iterator() of dataset class to independently verify the performance of data processing without training. Refer to the link for data processing optimization suggestions: https://mindspore.cn/tutorials/experts/zh-CN/master/dataset/optimize.html
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.246.506 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:786] DumpTaskExceptionInfo]     4) If it is a dynamic dataset, please set the input to dynamic through set_inputs, or set sink_size to 1. It is recommended to use the former, because the latter has poor performance.
[WARNING] DEVICE(45501,fff2ab7fe1f0,python):2023-08-21-08:15:14.718.905 [mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_kernel_runtime.cc:1116] RunTask] Destroy tdt channel failed.
Traceback (most recent call last):
File "tools/train.py", line 312, in
main(config)
File "tools/train.py", line 248, in main
initial_epoch=start_epoch,
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/train/model.py", line 1049, in train
initial_epoch=initial_epoch)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/train/model.py", line 100, in wrapper
func(self, *args, **kwargs)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/train/model.py", line 598, in _train
cb_params, sink_size, initial_epoch, valid_infos)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/train/model.py", line 681, in _train_dataset_sink_process
outputs = train_network(*inputs)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/nn/cell.py", line 620, in call
out = self.compile_and_run(*args, **kwargs)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/nn/cell.py", line 942, in compile_and_run
return _cell_graph_executor(self, *new_args, phase=self.phase)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/common/api.py", line 1439, in call
return self.run(obj, *args, phase=phase)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/common/api.py", line 1478, in run
return self._exec_pip(obj, *args, phase=phase_real)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/common/api.py", line 102, in wrapper
results = fn(*arg, **kwargs)
File "/usr/local/python3.7.5/lib/python3.7/site-packages/mindspore/common/api.py", line 1458, in _exec_pip
return self._graph_executor(args, phase)
RuntimeError: Run task for graph:kernel_graph_1 error! The details refer to 'Ascend Error Message'.

Ascend Error Message:

EL9999: Inner Error!
EL9999  [drv api] halQueueDestroy failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueDestroy][FILE:npu_driver.cc][LINE:2996]
TraceBack (most recent call last):
rtMemQueueDestroy execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rts api [rtMemQueueDestroy] failed, retCode is 507899[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)
E39999: Inner Error!
E39999  Aicpu kernel execute failed, device_id=0, stream_id=3, task_id=2, errorCode=91.[FUNC:PrintAicpuErrorInfo][FILE:task.cc][LINE:862]
TraceBack (most recent call last):
Aicpu kernel execute failed, device_id=0, stream_id=3, task_id=2, fault op_name=[FUNC:GetError][FILE:stream.cc][LINE:1133]
rtStreamSynchronize execute failed, reason=[the model stream execute failed][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)
EL9999: Inner Error!
EL9999  [drv api] halQueueQueryInfo failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueQueryInfo][FILE:npu_driver.cc][LINE:3238]
TraceBack (most recent call last):
rtMemQueueQueryInfo execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rtMemQueueQueryInfo failed, device is 0, qid is 1[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)

C++ Call Stack: (For framework developers)

mindspore/ccsrc/plugin/device/ascend/hal/hardware/ascend_graph_executor.cc:256 RunGraph
[WARNING] MD(45501,ffffa20461a0,python):2023-08-21-08:15:21.402.701 [mindspore/ccsrc/minddata/dataset/engine/datasetops/data_queue_op.cc:115] ~DataQueueOp] preprocess_batch: 11; batch_queue: 0, 0, 0, 0, 0, 0, 0, 0, 0, 0; push_start_time: 2023-08-21-08:13:04.696.711, 2023-08-21-08:13:06.260.506, 2023-08-21-08:13:10.447.333, 2023-08-21-08:13:13.348.705, 2023-08-21-08:14:34.658.753, 2023-08-21-08:14:34.897.697, 2023-08-21-08:14:35.244.371, 2023-08-21-08:14:36.841.525, 2023-08-21-08:14:37.574.301, 2023-08-21-08:14:39.307.348; push_end_time: 2023-08-21-08:13:04.732.580, 2023-08-21-08:13:06.297.322, 2023-08-21-08:13:10.485.841, 2023-08-21-08:13:13.386.329, 2023-08-21-08:14:34.714.893, 2023-08-21-08:14:34.937.644, 2023-08-21-08:14:35.290.069, 2023-08-21-08:14:36.880.453, 2023-08-21-08:14:37.611.451, 2023-08-21-08:14:39.345.333.
terminate called after throwing an instance of 'std::runtime_error'
what():  Failed to destroy channel for tdt queue. The details refer to 'Ascend Error Message'.

Ascend Error Message:

EL9999: Inner Error!
EL9999  [drv api] halQueueDestroy failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueDestroy][FILE:npu_driver.cc][LINE:2996]
TraceBack (most recent call last):
rtMemQueueDestroy execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rts api [rtMemQueueDestroy] failed, retCode is 507899[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)
EL9999: Inner Error!
EL9999  [drv api] halQueueDestroy failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueDestroy][FILE:npu_driver.cc][LINE:2996]
TraceBack (most recent call last):
rtMemQueueDestroy execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rts api [rtMemQueueDestroy] failed, retCode is 507899[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)
E39999: Inner Error!
E39999  Aicpu kernel execute failed, device_id=0, stream_id=3, task_id=2, errorCode=91.[FUNC:PrintAicpuErrorInfo][FILE:task.cc][LINE:862]
TraceBack (most recent call last):
Aicpu kernel execute failed, device_id=0, stream_id=3, task_id=2, fault op_name=[FUNC:GetError][FILE:stream.cc][LINE:1133]
rtStreamSynchronize execute failed, reason=[the model stream execute failed][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)
EL9999: Inner Error!
EL9999  [drv api] halQueueQueryInfo failed: deviceId=0, qid=1, drvRetCode=7.[FUNC:MemQueueQueryInfo][FILE:npu_driver.cc][LINE:3238]
TraceBack (most recent call last):
rtMemQueueQueryInfo execute failed, reason=[driver error:internal error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:49]
[Call][Rts]call rtMemQueueQueryInfo failed, device is 0, qid is 1[FUNC:ReportCallError][FILE:log_inner.cpp][LINE:161]
(Please search "Ascend Error Message" at https://www.mindspore.cn for error code description)

C++ Call Stack: (For framework developers)

mindspore/ccsrc/plugin/device/ascend/hal/device/ascend_data_queue.cc:250 ~AscendTdtQueue

您好，由于ReCTS数据集没有validation set，需要将val_while_train 设置成False


feat(charts/dex): Add extraObjects

Overview
This PR add the ability to include extraObjects. Array of extra K8s manifests to deploy
What this PR does / why we need it
Closes #3227
Special notes for your reviewer
Checklist

[x] Change log updated in Chart.yaml (see the contributing guide for details)
[x] Chart version bumped in Chart.yaml (see the contributing guide for details)
[x] Documentation regenerated by running make docs

Would be great :)

+1


ALTER TABLE xxx RENAME TO issue with memsql spark connector

Hello Team,
We are trying to rename a table present in memsql database using memsql spark connector as shown below.
But we are not successful. Can you please verify below memsql spark connector code and advise ?
Code:
sparksession
.read
.format("memsql")
.option("ddlEndpoint", connectionURL)
.option("user",username)
.option("Password", password)
.options(Map("query" -> ("ALTER TABLE Table1 RENAME TO Table2"),"database" -> "MyDB"))
.load()
ERROR:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'alter table MyDB.Table1 rename to MyDb.Table2) AS q WHERE 1=0' at line 1

Hi,
Thank you for reaching out and welcome to our community!
For DDL and other operational queries (e.g., update/delete), there is no concept of those in SparkSQL and you must use a direct JDBC connection (or prepared statement) to achieve them. For example:
try {
con = DriverManager.getConnection("mysqlurl", "userName", "password");
try {
Statement st = con.createStatement();
} finally {
if (st != null) close st
}
String sql = "ALTER TABLE Table1 RENAME TO Table2";
st.execute(sql);
st.close()
con.close();
} finally {
if con != null close con
}
Thank you for bringing this question to our attention. We will be updating our MemSQL documentation to make this point clearer, and we are considering releasing future functionality which will make it easier to connect to MemSQL directly via JDBC for such operations that cannot be performed natively through Spark.
Best,
Roxanna

Hi, currently you can use our new feature for running DDL queries directly to MemSQL.
Here's an example:
import com.memsql.spark.SQLHelper.QueryMethods
spark.executeMemsqlQuery("MyDB", "ALTER TABLE Table1 RENAME TO Table2")

Here's the first argument to executeMemsqlQuery is the Database name, and the second argument is the query to execute. You can also run this method with only one argument (without database name), in this case, the query will be executed against the database specified in SparkSession options.
Here's the doc about this feature.


Support of path filtering/monorepos when rendering

Checklist

[x] I've searched the issue queue to verify this is not a duplicate feature request.
[ ] I've pasted the output of kargo version, if applicable.
[ ] I've pasted logs, if applicable.

Proposed Feature
I would like to see a monorepo support (which could be implemented via possibility to perform path filtering when rendering to env. branches)
Motivation
I would like to have a monorepo with multiple microservices, like
svc-1
svc-2
svc-3

and those microservices are supposed to be separate pipelines in Kargo and promoted using rendering to env. branches. Currently Kargo is able to invoke kargo-render which is rendering contents of whole branch according definition in kargo-render.yaml. Kargo-render.yaml contains definition for rendering of all microservices in a repo, therefore in case of svc-1 promotion kargo-render will render manifests also for svc-2 or vice versa. Possibility to filter/specify and pass to kargo-render path (of source manifests) for rendering will make possible work with multiple services in single repository
Suggested Implementation

kargo-render.yaml spec has fields like configManagement.path or appConfig.<name> which could be utilized. Kargo could pass application name to kargo-render and kargo-render could thus determine the path for rendering
This can be configurable as part of gitRepoUpdates.render

@Brightside56 you may need to take a closer look at Kargo Render, because there is already adequate flexibility in the kargo-render.yaml file to enable what you're asking for.
Here's an example:
configVersion: v1alpha1
branchConfigs:
- pattern: svc-1/stage/(\w+)
  appConfigs:
    svc-1:
      configManagement:
        kustomize:
          path: svc-1/stage/${1}
      outputPath: svc-1
- pattern: svc-2/stage/(\w+)
  appConfigs:
    svc-2:
      configManagement:
        kustomize:
          path: svc-2/stage/${1}
      outputPath: svc-2

@krancour flexibility from my point of view is something what allows user to decide how to organize his/her gitops-repo. Example above is more like workaround, which assumes that I need to have a branch prefix for each service for each stage and prescribes me how things should be organized. I wouldn't like to have hundreds of branches in my monorepo. If kargo-render spec contains appConfigs including path in repo, why just not use it and let user decide how to organize his/her repo? I would like to have branch per stage with multiple services for this stage and not branch per service/stage

@Brightside56 I again invite you to take a closer look because there is nothing opinionated about how your repo is organized. Organize it however you like.
Kargo Render's config allows you to spell out the rules for how to construct each branch, either using the branch's explicit name or by matching it to a pattern.
I'm not sure if my use of pattern matching is what's throwing you off here?
The following is equally valid:
configVersion: v1alpha1
branchConfigs:
- name: my-weird-branch
  appConfigs:
    svc-1:
      configManagement:
        kustomize:
          path: any/path/i/want
      outputPath: any/path
...

@krancour I'm trying to have following config
configVersion: v1alpha1
branchConfigs:
- name: my-weird-branch
  appConfigs:
    svc-1:
      configManagement:
        kustomize:
          path: svc-1
      outputPath: svc-1
    svc-2:
      configManagement:
        kustomize:
          path: svc-2
      outputPath: svc-2
...

where svc-1 and svc-2 are my apps with respective Kargo pipelines and my-weird-branch is stage for those pipelines. I would like to have a possibility to render svc-1 independently from svc-2, but if svc-1 and svc-2 are in same branch, kargo-render can only render whole branch, therefore any render of svc-1 will lead to render of svc-2
/ $ kargo-render render --help
Render environment-specific manifests into an environment-specific branch of the remote gitops repo

Usage:
  kargo-render render [flags]

Flags:
      --allow-empty             allow the rendered manifests to be empty; if false this is disallowed as a safeguard against scenarios where a bug of any kind might otherwise cause Kargo Render to wipe out the contents of the target branch in error
  -m, --commit-message string   specify a custom message to be used for the commit to the target branch
  -d, --debug                   display debug output
  -h, --help                    help for render
  -i, --image stringArray       specify a new image to apply to the final result (this flag may be used more than once)
  -o, --output string           specify a format for command output (json or yaml)
  -R, --ref string              specify a branch or a precise commit to render from; if this is not provided, Kargo Render renders from the head of the default branch
  -r, --repo string             the URL of a remote gitops repo (required)
  -p, --repo-password string    password or token for reading from and writing to the remote gitops repo (required; can also be set using the KARGO_RENDER_REPO_PASSWORD environment variable)
  -u, --repo-username string    username for reading from and writing to the remote gitops repo (required; can also be set using the KARGO_RENDER_REPO_USERNAME environment variable)
  -t, --target-branch string    the environment-specific branch to render manifests into (required)

@Brightside56 having two pipelines (svc-1 and svc-2) would seem to indicate that you want to work on and deploy those services independently of one another. If this is the case, why overcomplicate things by requiring them to be in the same branch?

@krancour yes, I want to work with those services independently. However those services are related to same product, team and environments. From functional and security point of view I don't have any reason to isolate them into separate branches.
One set of repository permissions per environment instead of 10, one branch per environment instead 10, possibility to check versions, env. vars and changelog of all services on environment without switching branches and frustrating. I find it convenient, comfortable and less complicated.


Getting error while running Android app

<EMAIL_ADDRESS>(43, 17): Class 'kotlin.Unit' was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.6.0, expected version is 1.4.2.
Note: Could you add  iOS and Android version compatibility in document?

getting same error

try modifying as follows:
`diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index fcb659d..01c2341 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -1,6 +1,6 @@
buildscript {
// Buildscript is evaluated before everything else so we can't use getExtOrDefault

def kotlin_version = "1.4.32"

def kotlin_version = "1.6.0"
repositories {
google()
@@ -120,7 +120,7 @@ repositories {
}
}

-def kotlin_version = "1.4.32"
+def kotlin_version = "1.6.0"
dependencies {
// noinspection GradleDynamicVersion
`

solved it!

not worked


Addon project structure

The previous ember-cli guide shows more information regarding the conventions of the addon structure.

previous

latest

When looking at ember-cli/ember-cli#8342, I wanted to know how test-support and addon-test-support works, but this info is not available in the new ember-cli guide.
I think the missing info should be added in the new guides as well. I can create a PR if someone suggests how it could fit best in the new guide structure.
Some suggestions to start the discussion:

add the missing items into the Addon file structure section.

create an Extended addon file structure section.

cc @jenweber

I am closing this issue because the file structure will likely change with V2 addons.


chore(helm): set default image tag to 1.1.1 for ui

If we go with the default installation and create the deployment with the older release, this PR will set the UI image version to 1.1.1 by default.
https://github.com/eshepelyuk/cmak-docker/pkgs/container/dckr%2Fcmak-<IP_ADDRESS>

Thanks @adiii717
Available since
https://github.com/eshepelyuk/cmak-operator/releases


Send HTML widgets as Show File events to Positron

This change is the R side of https://github.com/posit-dev/positron/pull/4151; see that PR for a detailed explanation of the changes.

reprex is correctly going to the Viewer pane for me, but I prefer it without the IOPub Stream message. Otherwise it conflicts with reprex's actual output

profvis::profvis({Sys.sleep(2)}) experience still seems a bit wonky
https://github.com/user-attachments/assets/2e728b24-7b5c-4305-baad-c428caa5b819

Profvis isn't showing up in the Viewer because it is explicitly asking not to, via viewer.suppress (which creates an object with suppress_viewer class instead of the htmlwidget class;)
https://github.com/r-lib/profvis/blob/b8233d3f5295be64aa6b5719db457a94cfd5ebf6/R/profvis.R#L224-L227
    sizingPolicy = htmlwidgets::sizingPolicy(
      padding = 0,
      browser.fill = TRUE,
      viewer.suppress = TRUE,
...


Create a similar site for Wii titles (WiiWare and VC)

Hello,
I know this isn't an issue per se but I couldn't find any other way to contact you.
I've been wondering if it was possible to have a site for Wii Title keys titles just like for the Wii U/3DS (something like wii.titlekeys.gq).
Considering that the Wii uses virtually the same distribution system as the Wii (tmd/tik etc, with the only difference being the common key of course) I believe it should be fairly easy to put in place and would allow people to backup their games before the shop closes in 2019.
Thanks!

Sorry, I am not currently active in hacking scenes. Please try adapting my code for the Wii.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On April 3, 2018 5:34 PM, XorAndZ<EMAIL_ADDRESS>wrote:

Hello,
I know this isn't an issue per se but I couldn't find any other way to contact you.
I've been wondering if it was possible to have a site for Wii Title keys titles just like for the Wii U/3DS (something like wii.titlekeys.gq).
Considering that the Wii uses virtually the same distribution system as the Wii (tmd/tik etc, with the only difference being the common key of course) I believe it should be fairly easy to put in place and would allow people to backup their games before the shop closes in 2019.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This is also valid for DSi.

There is already a DSiWare on 3ds titlekeys
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On April 13, 2018 7:12 PM, mariomadproductions<EMAIL_ADDRESS>wrote:

This is also valid for DSi.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Those are DSiWare ports. They aren't the same thing at all.


Add data-generator and recommendation-app modules

Add data-generator module to easily load up Kafka topics with streams of data.
Add recommendation app directory with files and instructions for running the example

@robobario I've refactored the data generator, can you run it again and see if it runs cleanly for you now?

@robobario I've refactored the data generator, can you run it again and see if it runs cleanly for you now?

Yep, worked for me.


Fix behavior when restoring from a dump with an old format

Scope & Purpose
Fix an error in restore behavior when using an old dump format. Previously, index info was stored under the parameters object in the collection properties in the dump. This handles both the old format and the new one.

[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)

Related Information

[x] There is an internal planning ticket: BTS-84

Testing & Verification
Jenkins: https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/10306/

It seems that we cannot remove the dump json files from the repository.
I saw that the Jenkins run https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/10306/ was blue, so it must have succeeded without all the json files being present.
But running the dump_multiple or hot_backup tests locally without these files being present fails, for both single server and cluster.
I don't know why it works in Jenkins, because it is executing these tests as well as part of PR runs:
https://<IP_ADDRESS>/job/arangodb-matrix-pr-linux/12046/EDITION=enterprise,STORAGE_ENGINE=rocksdb,TEST_SUITE=single,limit=(linux&&test)||gce/artifact/testRuns.html
:thinking:
Anyway, it seems we can't delete these files.


[Feature]

Description
The About page is not so attractive, I can make it more visually appealing by providing proper color and font with appropriate image.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.
[X] I want to work on this issue.

Code of Conduct

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

Hey, allow me to work on this under GSSOC' 2024

Hey there! Allow me to work on this under GSSOC' 2024.

@Anou25 Do the necessary changes and make PR we'll check their.

Ok

Hi @CSEdgeOff  please can u assign it to me under GSSoC'24


How to remove an extension after a failed upload? (Cloud demo)

AL version : 2.0.43900
Server version : 13.0.24630.25242-US
I am trying out the Upload Extension function in a demo tenant on https://businesscentral.dynamics.com/.

My 1st attempt failed due to a compilation error (caused by a difference between the US version of the demo system and the W1 version I used in development).
I downloaded the correct sandbox, resolved the compilation error and recompiled.
My 2nd attempt failed with error "An extension with same App ID and version has already been uploaded. Resolve and deploy again."

Question : how can I uninstall & unpublish the first upload to make way for the recomplied app?
The extension is not listed in the Extensions page and I don't know of a way to use PowerShell on a cloud demo.
For now I have worked around the problem by altering the App ID.
Thank you in advance.

@chris-amco are you using the sandbox? You can reset it by heading to   https://businesscentral.dynamics.com/ and then searching for Sandbox and select SANDBOX ENVIRONMENT (PREVIEW). Then you can reset your sandbox. Later you can try to redeploy with the original app id.
Can you try to provide a clear repro for this issue? When you deploy and the App ID is used but you don't see the extension in the Extensions page.

4134.zip
Thanks for your reply. I was not using the online sandbox. I was uploading to the "production" demo system.
I've attached 2 copies of the same repro app.

Folder "4134_1" : compiled on docker container v13.0.24630.25242-W1.
Folder "4134_2" : compiled on docker container v13.0.24630.25242-US.

I usually use the W1 container for development. I obtained US container from "Container Sandbox Environment (Preview)" -> "Host locally" on the cloud demo system.
The .app file in 4134_1 cannot be uploaded to a US system due to a compilation error. In 4134_2, I made a change to avoid the compilation error.
Repro steps

Log in to a cloud demo environment (US).
Extensions > Upload Extension > choose the .app file in folder 4134_1. It will fail with the following error:

Deployment failed. Errors: App ID : 2854f490-2e7a-4bdf-b5ff-c5d8587674a7 Message : { Extension compilation failed
code/pageextension/Pag-Ext50100.4134_PostedPurchaseInvoice.al(5,18): error AL0248: '"Payment Method Code"' is not found in the target } - Job Id : 0cb2259e-ea40-4dab-a59f-9b6682fe8c03

Try again with the .app file in 4134_2. It will fail with the following error:

Deploy Operation rejected. Error: An extension with same App ID and version has already been uploaded. Resolve and deploy again.; - Job Id : 6504db0e-dd3d-435b-8eb8-21b9eddff14a

But the failed extension does not appear in the Extensions page and I cannot see any other way to remove it.

In order to get around this issue (without changing the AppId) you will have to bump the version number of your extension as the original one is already stored and cannot be 'overridden'.
There is some future work in/around some better management of the packages uploaded in this manner that may include this feature (deletion), but for now you have to be conscious of what you upload.

Okay, understood. It was the first time I had tried uploading a package and I just wanted to make sure that I hadn't missed anything.
Thanks for the advice.


clone into "CUDArt" folder instead of "CUDArt.jl"

Currently the package gets cloned into a  "CUDArt.jl" directory and fails to build.  It turns out the build code looks for a "CUDArt" folder.  Renaming the folder after cloning is enough to allow me to build successfully.

Thanks for the report!
How did you clone it? If you use Pkg.clone(url), I think the .jl gets stripped. If you use the command-line, I think you're right that it's necessary to give it a new name, like git clone url CUDArt. I think that's true for most julia packages, though, because they all (should) end in .jl.

Ah!  I used the command line.  That makes sense.  Thanks!
On Fri, Jan 9, 2015 at 1:34 PM, Tim Holy<EMAIL_ADDRESS>wrote:

Thanks for the report!
How did you clone it? If you use Pkg.clone(url), I think the .jl gets
stripped. If you use the command-line, I think you're right that it's
necessary to give it a new name, like git clone url CUDArt. I think
that's true for most julia packages, though, because they all (should) end
in .jl.
—
Reply to this email directly or view it on GitHub
https://github.com/JuliaGPU/CUDArt.jl/issues/6#issuecomment-69386189.


MSC1700: Improving .well-known discovery

Rendered
Note: This is a summary of my thoughts on the matter of .well-known discovery validation.

This looks very very sane to me.

I'm closing this until I have the motivation to rewrite it.


Error after start

I had error like the image in various instances, when I try to log in or search. I've done the setup following the readme.

An error.log must have been generated during any runtime errors, find them and paste them here. Usually located in /var/log/apache2, if PHP is an apache2 module
validate.php does not require any other schemas than MainDB.
Can you also let me know what PHP is your server running?

Thank you for the attention, I realize that is a problem with my server, because in a test running the system in a local server with xampp and it works fine.


uwsgi doesn't restart on reboot

Maybe related to Ubuntu 18.04 differences.
Figure out what server we're launching this on.

Fixed in install docs. Due to Ubuntu 18.04:
https://www.linuxbabe.com/linux-server/how-to-enable-etcrc-local-with-systemd


Add a Gitter chat badge to README.md

AdaptiveConsulting/ReactiveTrader now has a Chat Room on Gitter
@JakeGinnivan has just created a chat room. You can visit it here: https://gitter.im/AdaptiveConsulting/ReactiveTrader.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Clicked on the wrong repo by accident. Can delete it, let me know


Check inspec tests for docker env

Do we need all those skip_* ??
Should work already: https://github.com/dev-sec/linux-baseline/pull/91

Looks like Inspec doesn't recognize that the tests are running in a container ... maybe the host os (OpenSUSE 15.0) is not recognized?
@artem-sidorenko Please investigate  :-)

@mcgege can't reproduce as I said, sorry :-(

This seems to be a problem in InSpec: https://github.com/inspec/inspec/issues/3819


Getting something went wrong error when trying to navigate into users section

Describe the issue:
Getting something went wrong error when trying to navigate into users section

How to reproduce:

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [https://wso2.org/jenkins/job/products/job/product-is/4871/]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

The issue arises from a missing configuration. The problem has been addressed in the following pull requests:

https://github.com/wso2/identity-apps/pull/4235
https://github.com/wso2/carbon-identity-framework/pull/5014

This issue is fixed from above PRs.


Remove dropping/renaming limitation for 20.1.1 upgrade

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d81fc19422921f7cf3bebfb28e5bbfd7e117b286/
Edited pages:

v20.1/upgrade-cockroach-version.md


fix: inconsistent API response where sometimes it returns quantity as a

NOTE: Please delete placeholder texts that are not relevant. Including this
NOTE: Make sure your commit messages are semantic
Description
Tips:

Fixes # (issue)
What is the summary of the change and which issue is fixed?
What is the relevant motivation and context?
Is there any dependencies that are required for this change?

How has this been tested?
Tips:

Please describe the tests that you ran to verify your changes.
How can we reproduce the issue?
What are the relevant details for your test configuration:?

https://lorenzopeppoloni.com/tabledriventestspy/


Use padding in table head only in multi-select mode

Details
The left and right padding in the table head is only needed in multi-select mode where the checkbox and the right arrow are rendered
Fixed Issues

$ https://github.com/Expensify/App/issues/42066
PROPOSAL:
Tests

Precondition:

Workspace is connected to QBO
Classes and Customers/Projects are imported from Import settings

Go to workspace settings
Go to Tags
Click Settings
Toggle on "Members must tag all spend" and go back
Verify that "Required" is shown and aligned correctly under Status column

[X] Verify that no errors appear in the JS console

Offline tests

QA Steps

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[X] I verified that all the inputs inside a form are aligned with each other.
[X] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

Networking issue

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

cc @rlinoz

cc @Expensify/design

sorry @thesahindia , @ikevin127 reviewed the original PR

♻ Reviewing and completing the reviewer checklist as we speak!

🚀 Deployed to production by https://github.com/chiragsalian in version: 1.4.74-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Cannot use JsonValueSerializer for types without a public, parameterless constructor

Description
A Producer cannot use JsonValueSerializer<T> if the type  does not have a public, parameterless ctor. This is quite limiting as it prevents using existing libraries such as CloudEvents:
https://github.com/cloudevents/sdk-csharp
See also: https://github.com/cloudevents/sdk-csharp/blob/master/src/CloudNative.CloudEvents.Kafka/KafkaCloudEventMessage.cs
How to reproduce
When constructing a Producer using the ProducerBuilder, use JsonValueSerializer<T>() where T does not meet the new() constraint.
public class CloudEvent {
  public string Id { get; set; }

  public CloudEvent(string id) {
    Id = id;
  }
}

var producer = new ProducerBuilder<string, CloudEvent>(producerConfig)
                .SetValueSerializer(new JsonSerializer<CloudEvent>(schemaRegistryClient, jsonSerializerConfig))
                .Build();

AvroSerializer<T> does not have this constraint. Also, while ProtobufSerializer<T> does, it has an additional constraint on IMessage<T>, which also implies it could not support IMessage<T, U>.
Workaround
It seems the only workaround would be to use Avro or to write a custom implementation of IAsyncSerializer<T>.
Checklist
[1.5.2] Confluent.Kafka version

thanks @ahernandez-allegion - I just merged a change by @appleseed7even to fix this to master, so it'll be in 1.6.


Fix check of node type

Closes #137
Seems that enzyme-adapter-react-16 was the one that broke the type checks. So this also bumps those versions to make sure the tests passes both before and after the bump.

Hi @Benjaminsson, thanks for your PR.
Can you add a test for the case you fix ?

@VincentLanglet I added a test case. I don't know how much value it gives. As you can see the test coverage was already 100% and it did actually break when updating the adapter. But might be a bit more future proof now? I'm not super experienced in testing but this should do the job right?

@Benjaminsson It's a good way for me to be sure the PR does the job.
And it avoid futur regressions.
Thanks !

Relating to the original issue #137, I'm wondering what will be printed in the snapshot? React.memo in place of Component? Shouldn't it be Memo(ComponentDisplayName)?


Config files - add frames per second per channel

Depending on Intel realsense depth camera we can pick difference FPS for different channels. I propose we redesign configuration file and make it nested. In addition, to standard fields, I propose to have: FPS, buffer length,
Here is an example of a section of config file:
channels:

  - #<-this is first entry in the channels list
    type: depth
    fps: 30
    buffer_length: 30

  - #<-this is second entry in the channels list
    type: color
    fps: 30
    buffer_length: 30

  -
    type: infrared
    fps: 30
    buffer_length: 30

  -
    type: accel
    fps: 250
    buffer_length: 30000

  -
    type: gyro
    fps: 400
    buffer_length: 30000

Let us discuss.
I propose we split each stream(channel) in a different entry in the configuration file and assign different circular buffers and different FPS to each channel. It will provide more flexibility in future development. Though, it would add a little bit more coding.
in addition, it will allows to configure L515 camera differently from D435i camera based on config file. For example, there is no "infrared" channel in D435i or D455 cameras but it is present in L515. Adding channels to configuration file would provide needed flexibility.
What do you think? @AbdelRahmanNasser20

I can start working on that but I have a question. How does the d435i not have a infrared channel but it can display a infrared image? is it mixed with another channel?

I can start working on that but I have a question. How does the d435i not have a infrared channel but it can display a infrared image? is it mixed with another channel?

I did not know it has infrared image. I am curious to see how it looks. Last time I tried, I could not start device class for D455 camera because of gyroscope or accelerometer settings. I have hard-coded FPS into configuration file, but later I learned that two cameras have different framerate for gyro(accel).

All you have to do connect the camera and change serial number in the config file  and run the live stream test and you will be able to see it.

I don't have a D455 camera. Can you please confirm that it has a infrared channel.


Problem while understanding the context if it has the entity words but they are not relevant to the context

Rasa NLU version:
Operating system (windows, osx, ...): windows 7
Content of model configuration file: 0.12.3

Issue:
I have trained my model and if I am asking some questions like: "Help me buy liquor". Then the chatbot should ask : "What is your age?". But the problem in my case is that, even if I say: "drinking liquor is injurious to health" , it simply moves to the next question and asks me: "What is your age?", which should not happen. It should understand that though the sentence contains the entities but the context is not relevant. Could somebody help me here?

please can you list the intents used to train the data?? and this is possible if you use node-red i guess where a reply or an action is set for special cases

You should define that sentence as a different intent if you would like different behaviour


use git username and email for getLicense by default

Currently,
import getLicense from "license";
getLicense("MIT")

Will generate a license with empty username and emailname. Why not insert username and email from git by default? That is, if user name is foo and email is<EMAIL_ADDRESS>getLicense("MIT") will be like getLicense("MIT", { author: "foo", year: "2020" }).

Will do this as a part of #13


Error during publishing Azure Function

I already worked with diferent versions of Company Communicator and I customized and publish those change on Azure services with Visual Studio, now I tried to deploy the last version, without any code modification. At first, I updated SDKs and Nuget package to indicated version on project, next I configure the dependencies to publish as I did in another deployments of this appication. I published the Web Service project (Microsft.Teams.Apps.CompanyComunicator) and works correcly. The problem it's when I try to publish the Azure Functions, in this three cases the publish fail in Zip Deployment step.

Thanks a lots

Hi @cesarorosa ,
Thanks for raising the issue. We will get back to  you with proper response soon.

Hi @cesarorosa, Post SDK and NuGet update are you able to build successfully your Azure function in release mode?

Hi @cesarorosa, Post SDK and NuGet update are you able to build successfully your Azure function in release mode?

Hi, thanks for response. The SDK version of .netCore that i user it's the same that appears in global.json ("version": "3.1.108"). For NuGet package I have the following versions:

Thanks a lot for help me.

Hi @cesarorosa ,
Could you please verify whether locally build is working fine when it is kept in release mode. Keep us posted.

Hi @cesarorosa , Could you please verify whether locally build is working fine when it is kept in release mode. Keep us posted.

Hi @gsv022 I builded and tried to publish function in Release mode, but the error still happend

@cesarorosa Please let us know if you still need any further assistance on the issue logged.


crash upon opening

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 1903 18362.175]
App version: [v0.11.0]

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?
Log file
Please upload the log file here so that we can understand your issue better. You can access it from Settings->About->Open log location. You can also find this file by going to %localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState, you should see a file in this directory called debug.txt or/and debug_fulltrust.txt.

@SteveP757 We need a little more information from you in order to investigate this issue, please reopen this issue once you have filled out the issue template.

click the icon, splash screen appears, crashes soon after. Windows 10 build 21313. Surface Studio 2. I don't know where the log file is.

@SteveP757 You can find the log files by going to %localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState, you should see a file in this directory called debug.txt or/and debug_fulltrust.txt.


Link syntax is mangled

See, for instance, https://gitlab.staging.haskell.org/ghc/ghc/issues/16155.

The problem is that
[[https://wiki.haskell.org/Core_Libraries_Committee|Core Libraries Committee]]

is parsed to
Link "https://wiki.haskell.org/Core_Libraries_Committee|Core Libraries Committee" ["https://wiki.haskell.org/Core_Libraries_Committee|Core","Libraries","Committee"]

It looks like the parse order for links with and without descriptions was inverted.

Via 8009db99ec1baca7ddac21934aa48aa63e18ea6d.