Feature request: Add china and korean regions support

???....

I am not actively working on this project anymore, so consider this will never be implemented.

I am not actively working on this project anymore, so consider this will never be implemented.

Yes.


Feature: Links docs in support

Type of feature
🍕 Feature
Current behavior
Would love to have a link for docs in the support dropdown.

Suggested solution
Link to the docs sites in the list.
Additional context
No response
Code of Conduct

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

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

@BekahHW, what's the best link for our docs and @isabensusan, which icon would you like to use?

This is making link: https://docs.opensauced.pizza. I think that should be a good starting point. I'll add links on that page to the guides for contribs and maintainers.


Add Sentry Stack Traces

for logs containing errors.
Closes #58

POSEIDON-2Y is an example with the new Stack Trace. Overall, it seems to work. I don't like that the Exception Type is used as issue title instead of the event message (as before) :roll_eyes:
Let's try to set the Transaction name as suggested in the Sentry forum.

it feels like the title is somewhat "ignored" due to our custom logging integration

For me, it feels like Sentry does not provide capabilities of setting the title to the SDK. The SDK can just collect information. The Sentry Backend analyses the information i.e. to perform the Issue Grouping. Doing so it "consider[s] the stack trace first, the exception next, and then finally the message." Based on that, it chooses a title for the group.

what changes the AttachStacktrace config option -- will it add the stack trace automatically, without the call to SetException?

Exactly! We have three main methods to send events to Sentry:

CaptureEvent: We have to provide/create the event that will be send to Sentry. We use this variant to set specific attributes (Timestamp, Level, Message, ...).
CaptureMessage: The event is generated from just one message. It will add a Stack Trace when AttachStacktrace is true.
CaptureException: The event is generated from just one error. It will always add a Stack Trace using the SetException function (that we also call).

But, I like using the Configuration option to collect the Stack Traces. Therefore, I refactored (b7b0d86) the Event generation to use EventFromMessage (including the Stack Trace collection).
In 14110c2 we remove the Exceptions (not the Stack Trace) as I would prefer a proper title as in POSEIDON-38 (having a Stack Trace) over POSEIDON-39 (having a Stack Trace via Exception, but not a proper title)!?

For me, it feels like the error currently captured with this PR is unwrapped once too often

I don't think so. When we compare the error without the Stack Trace and with the Stack Trace it's the same (the client wants to stop the runner).
For me, the *errors.errorString text creates the feeling something is wrong. But, this is actually wanted by Sentry as it gathers the type of the passed error (reflect.TypeOf(err).String()). To me, it feels wrong that Sentry sets the Error Type as Title for the Issue group!

Especially the part about cutting stack traces seems relevant to that question

Thanks, I would like it! However, I tried quite some time with different rules including
path:**/go/pkg/mod/github.com/sirupsen/**              ^-group -group
stack.abs_path:**/go/pkg/mod/github.com/sirupsen/**      ^-group -group
stack.abs_path:**/sirupsen/logrus*/**/*.go               ^-group -group
stack.function:(*Entry).Warn                             ^-group -group
stack.abs_path:**/pkg/logging/sentry_hook.go             ^-group -group

and also found one Golang specific example, but the upper Stack Trace Frames did not disappear :disappointed:

We only use the first option, don't we?

Yes, we use the first one to send the event. But, we also use EventFromMessage (which is the main part of CaptureMessage) since b7b0d86.

As far as I know, the issue grouping is only applied for new issues, not for existing ones (just to keep in mind for your experiments). Besides that, I wondered why you are using an exclusion to group and not to app (but I haven't investigated further).

That's right. Sentry states Changing stack trace rules will apply to future events only. But, new events do not have the cut stack trace (neither new issues). Also the added ^-app -app does not succeed. I'm tempted to ask the Sentry Support for help (rather than time intensive debugging)!?

But, new events do not have the cut stack trace (neither new issues)

I was about to try that myself, but did not recognize those issues. Here is a new, custom error with POSEIDON-3D and a well-known one with POSEIDON-3E.
Isn't that how we would like it? I did not make any changes to Sentry, and just used the latest commit of this branch for my testing.

Oh wow, thank you! Yes, this is how we would like it. It seems, I was too impatient when testing the added ^-app -app :see_no_evil:

Alright, great to hear. This error triggered by staging made by the previous commit b7b0d86c looks also fine. Still, I rebuild Poseidon with the latest commit, and reproduced an error. It is also shown as desired, isn't it?
P.S: I rebuild Poseidon manually in order to prevent any influence on your current DNS changes.


Indonesian Translation for Angular CLI

Hi, Can I contribute to translate angular CLI documentation into Indonesian Language?

Why indo programmer don't know english ?

Our documentation website isn't ready yet. When it is you're welcome to contribute.
I'm going to leave this open to come back when we have a proper documentation website.

Heya, a better place to ask is in https://github.com/angular/angular. I will move this issue there.


Add support for opengraph data

Would need to add:
Frontmatter:

ogtype
author
description (rename excerpt?)

Config:

default ogtype
default author

Then modify the page parsing and config structs to match.

Fixed in #11


React-native uses outdated version of connect, which in turn requires package with vulnerability

At the moment react-native dependency on connect is declared like this: "^2.8.3", which won't allow  for the newer versions of connect like 3.6.3 (connect 3.6.3 doesn't depend on compression module, so this eliminates the problem I have).
My dependency tree looks like this:
└─┬<EMAIL_ADDRESS>  │ A framework for building native apps using React
  │<EMAIL_ADDRESS>  │ https://github.com/facebook/react-native#readme
  └─┬<EMAIL_ADDRESS>    │ High performance middleware framework
    │ git+https://github.com/senchalabs/connect.git
    │ https://github.com/senchalabs/connect#readme
    ├─┬<EMAIL_ADDRESS>    │ │ Node.js compression middleware
    │ │ git+https://github.com/expressjs/compression.git
    │ │ https://github.com/expressjs/compression#readme
    │ └─┬<EMAIL_ADDRESS>    │   │ Higher-level content negotiation
    │   │ git+https://github.com/jshttp/accepts.git
    │   │ https://github.com/jshttp/accepts#readme
    │   └──<EMAIL_ADDRESS>    │       HTTP content negotiation
    │       git+https://github.com/jshttp/negotiator.git
    │       https://github.com/jshttp/negotiator#readme

The package with vulnerability - negotiator (issue described here https://nodesecurity.io/advisories/106).
p.s. not sure whether this is the right place to enquire regarding the support for the latest version of connect, but all the other resources seemed equally unfit for my issue.

This has already been mentioned in #11637, but that issue has been closed due to inactivity. Since its just a packager issue I have this .nsprc in our project:
{
  "exceptions": [
    "https://nodesecurity.io/advisories/106" // https://github.com/facebook/react-native/issues/11637
  ]
}

Thanks for the quick reply @Strayer.
I understand that the vulnerability might not apply in my case.
Still, I'm interested whether there are plans to update the dependency.

There are no plans to update it. If you would like to go through all the call sites and update the dependency, a PR would be appreciated.


Bug: bad SSR index.html rendering.

Hi!
I get a blank page after build for production and serve.
In dist/server/main.js  the template has

 without ". Any idea?

Sorry a bad html-minifier was the problem.


blueos idx plugin issue

Describe the bug
on lassen:
Error opening plugin file: /usr/WS1/cyrush/2024.01.build_visit/visit/build-blueos/plugins/databases/libMIDXDatabase.so (/usr/WS1/cyrush/2024.01.build_visit/visit/build-blueos/plugins/databases/libMIDXDatabase.so: undefined symbol: ompi_mpi_comm_world)
Running: engine_ser -dv -host <IP_ADDRESS> -port 5600

Looks like this plugin assumes openmpi?
We are using spectrum mpi (ibm's mpi) so openmpi shoudn't be near the equation.

delete after install for temporary victory :-)

this was a previously known issue, but turning off pidx wasn't captured in blueos case of run-build-visit

resolved in #9331


Add deprecation warnings for numbered colors

The new theme is not supporting numbered colors anymore (blue0, green1..).

should be addressed in #164


StableDiffusion / Non-zero status code returned while running Add node. Name:'Add_221'

While trying Basic Stable Diffusion Example of https://www.nuget.org/packages/OnnxStack.StableDiffusion, at the following line in the code:
    // Run Pipleine
    var result = await pipeline.GenerateImageAsync(promptOptions);

Following exception is raised:
Microsoft.ML.OnnxRuntime.OnnxRuntimeException
  HResult=0x80131500
  Message=[ErrorCode:RuntimeException] Non-zero status code returned while running Add node. Name:'Add_221' Status Message: D:\a\_work\1\s\onnxruntime\core\providers\dml\DmlExecutionProvider\src\MLOperatorAuthorImpl.cpp(2482)\onnxruntime.DLL!00007FFC280E7AA5: (caller: 00007FFC280E712D) Exception(3) tid(1b6c) 80004005 Unspecified error

  Source=Microsoft.ML.OnnxRuntime
  StackTrace:
   at Microsoft.ML.OnnxRuntime.NativeApiStatus.VerifySuccess(IntPtr nativeStatus)
   at Microsoft.ML.OnnxRuntime.InferenceSession.<>c__DisplayClass75_0.<RunAsync>b__0(IReadOnlyCollection`1 outputs, IntPtr status)
--- End of stack trace from previous location ---
   at Microsoft.ML.OnnxRuntime.InferenceSession.<RunAsync>d__75.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<EncodePromptTokensAsync>d__39.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<GeneratePromptEmbedsAsync>d__40.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<CreatePromptEmbedsAsync>d__37.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<RunInternalAsync>d__31.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<GenerateImageAsync>d__26.MoveNext()
   at TestOnnxStack.TestStableDiffusion.<Test01>d__1.MoveNext() in ...\TestOnnxStack\TestStableDiffusion.cs:line 42
   at Program.<<Main>$>d__0.MoveNext() in ...i\TestOnnxStack\Program.cs:line 6

To reproduce:

Create .net8 console project
Add nuget package microsoft.ml.onnxruntime.directml (1.17.3) and onnxstack.stablediffusion (0.31.0)
Copy code in Program.cs from Basic Stable Diffusion Example of https://www.nuget.org/packages/OnnxStack.StableDiffusion documentation.
create d:\model folder to git clone https://huggingface.co/runwayml/stable-diffusion-v1-5 -b onnx
change path to model in code e.g. var pipeline = StableDiffusionPipeline.CreatePipeline(@"D:\model\stable-diffusion-v1-5");

Platform

Windows 10
.NET 8.0.200
OnnxStack.StableDiffusion 0.31.0
Microsoft.ML.OnnxRuntime.DirectML Version="1.17.3"
Target CPU x64

That;s an odd looking error coming from deep within
I'll download that model and see if its a regression, been a while since I have used the version of the model
I have this version on disk, and taht seems to work ok following your steps
https://huggingface.co/TensorStack/stable-diffusion-v1-5-onnx
I will check the other model now and update you with what I find

Downloaded the model from https://huggingface.co/TensorStack/stable-diffusion-v1-5-onnx
Now getting slightly different error than above:
Microsoft.ML.OnnxRuntime.OnnxRuntimeException
  HResult=0x80131500
  Message=[ErrorCode:RuntimeException] Non-zero status code returned while running Mul node. Name:'' Status Message: D:\a\_work\1\s\onnxruntime\core\providers\dml\DmlExecutionProvider\src\MLOperatorAuthorImpl.cpp(2482)\onnxruntime.DLL!00007FFC9DD77AA5: (caller: 00007FFC9DD7712D) Exception(3) tid(6c74) 80004005 Unspecified error

  Source=Microsoft.ML.OnnxRuntime
  StackTrace:
   at Microsoft.ML.OnnxRuntime.NativeApiStatus.VerifySuccess(IntPtr nativeStatus)
   at Microsoft.ML.OnnxRuntime.InferenceSession.<>c__DisplayClass75_0.<RunAsync>b__0(IReadOnlyCollection`1 outputs, IntPtr status)
--- End of stack trace from previous location ---
   at Microsoft.ML.OnnxRuntime.InferenceSession.<RunAsync>d__75.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<EncodePromptTokensAsync>d__39.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<GeneratePromptEmbedsAsync>d__40.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<CreatePromptEmbedsAsync>d__37.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<RunInternalAsync>d__31.MoveNext()
   at OnnxStack.StableDiffusion.Pipelines.StableDiffusionPipeline.<GenerateImageAsync>d__26.MoveNext()
   at TestOnnxStack.TestStableDiffusion.<Test01>d__1.MoveNext() in ...\TestOnnxStack\TestStableDiffusion.cs:line 42
   at Program.<<Main>$>d__0.MoveNext() in ...i\TestOnnxStack\Program.cs:line 6

Device/CPU.:

GPU:

Kind of stuck at the moment as just a beginner in this ML.Net/Onnx/directML/etc. space.
Thank you for your support.

Unfortunately you GPU may not have enough VRAM for stable diffusion, and minimum you would need 3GB-4GB for a F16 model
You can switch to CPU mode by using the CPU execution provider and see if that works
var pipeline = StableDiffusionPipeline.CreatePipeline(@"D:\models\test\stable-diffusion-v1-5", executionProvider: ExecutionProvider.Cpu);
Its ok, I'm just learning ML too :)

Tried on another machine with success.
Typical GPU state during image generation:

Dxdiag system:

Dxdiage AMD Radeon GPU:

It takes on average about 8 minutes per image generation.
Now looking into how to reduce the time, which is of course not accetable at the moment.
Would welcome any suggestion in that direction!


Mac OS make failure with set USE_OPENMP = 0

problem
mxnet compile failure with instruction in http://mxnet.readthedocs.io/en/latest//how_to/build.html
the error is dmlc-core make failure of USE_OPENMP in osx.mk seem useless in dmlc-core.
make error:

cd /Users/stupid-coder/Documents/Codes/DMLC/mxnet/dmlc-core; make libdmlc.a
config=/Users/stupid-coder/Documents/Codes/DMLC/mxnet/config.mk; cd /Users/stupid-coder/Documents/Codes/DMLC/mxnet
g++ -c -O3 -Wall -msse2  -Wno-unknown-pragmas -Iinclude  -std=c++0x -fopenmp -fPIC -DDMLC_USE_HDFS=0 -DDMLC_USE_S3=0 -DDMLC_USE_AZURE=0 -o line_split.o src/io/line_split.cc
clang: error: unsupported option '-fopenmp'
make[1]: *** [line_split.o] Error 1

solved
change mxnet/Makefile(17-25):
ifneq ($(USE_OPENMP), 1)
        export NO_OPENMP = 1
endif

# use customized config file
include $(config)
include mshadow/make/mshadow.mk
include $(DMLC_CORE)/make/dmlc.mk
unexport NO_OPENMP

to
include $(config)

ifneq ($(USE_OPENMP), 1)
        export NO_OPENMP = 1
endif

# use customized config file
include mshadow/make/mshadow.mk
include $(DMLC_CORE)/make/dmlc.mk
# unexport NO_OPENMP

mv inlcude $(config) before export NO_OPENMP=1, make sure the USE_OPENMP=0 before the ifneq and comment the unexport NO_OPENMP

i got same problem but it doesn't work even changed the makefile

I counter the same problem, I will try your solution

The Sample Wap:

export NO_OPENMP=1

#unexport NO_OPENMP

before run the make

The compilation problem should be solved by https://github.com/dmlc/mxnet/pull/2407


Stale gRPC stubs used for cloud spanner

https://github.com/GoogleCloudPlatform/google-cloud-java/pull/2989    fails due to an old version of the generated grpc code in use.  Looking on maven, it looks like 1.2.0, which is a few months old at this point.  It would be nice if this could be bumped to 1.10.0
@garrettjonesgoogle would you mind taking a look?

@garrettjonesgoogle friendly ping.   Any chance these could be rolled out in the near future?

We need to change this line: https://github.com/googleapis/gapic-generator/blob/master/build.gradle#L426

Follow-up: this has been released in google-cloud-spanner.


ADDomainTrust: Replace the trust type (string) value 'External' with 'Domain'

The resource is having a validate set with two valid values 'External' and 'Domain'. The value 'External' is converted to 'Domain' in the resource to get the correct context type value when using the the class methods to get a directory context.
The code code be simplified if we switch the value 'External' to 'Domain' in the validate set (and schema.mof).
Is it more logical/natural for a user to write 'External' than 'Domain' in the configuration? If so, then we can close this issue. Appreciate feedback on this.

Closing this due to lack of discussion. Let us leave it as is for now.


Inconsistency between Android and iOS on validation state for CardField

Describe the bug
There is a inconsistency on validation state for CardField (state control which was added by https://github.com/stripe/stripe-react-native/pull/629).
You can see below what I got on onCardChange after typing only the first card number:
iOS:
{
   "brand":"Visa",
   "complete":false,
   "expiryMonth":null,
   "expiryYear":null,
   "last4":"",
   "validCVC":"Incomplete",
   "validExpiryDate":"Incomplete",
   "validNumber":"Incomplete"
}

Android:
{
   "brand":null,
   "complete":false,
   "expiryMonth":null,
   "expiryYear":null,
   "last4":null,
   "validCVC":"Invalid",
   "validExpiryDate":"Invalid",
   "validNumber":"Invalid"
}

To Reproduce
Steps to reproduce the behavior:

Setup a screen with a CardField
Type just the first digit of card number
Log what you receive on onCardChange

Do it for both platforms and check results.
Expected behavior
All platforms should handle the state equally

Hello,
Is this ticket still active? It's marked as closed and merged but labelled 'in progress'.
In any case, I can still see an inconsistency between iOS and Android on the brand name:
iOS
{"brand": "Visa", "complete": false, "expiryMonth": null, "expiryYear": null, "last4": "", "validCVC": "Incomplete", "validExpiryDate": "Incomplete", "validNumber": "Incomplete"}
Android
{"brand": "", "complete": false, "expiryMonth": null, "expiryYear": null, "last4": "", "validCVC": "Incomplete", "validExpiryDate": "Incomplete", "validNumber": "Incomplete"}
I tested on 0.12.0 and 0.13.1.
Thank you

If you are using latest version of stripe-react-native then please add prop to CardField postalCodeEnabled={true} for android and false for ios . It will solve the issue


paymentservice OOMKilled, CrashLoopBackOff

Is there anyone else having a problem with the payment service not properly starting up?
$ kubectl get pods my-otel-demo-paymentservice-758cd56c47-7lxxh
my-otel-demo-paymentservice-758cd56c47-7lxxh          0/1     OOMKilled   4 (56s ago)    108s

After a while:
$ kubectl get pods
NAME                                                  READY   STATUS             RESTARTS         AGE
my-otel-demo-prometheus-server-57789ccfff-8t6wv       0/1     Pending            0                45m
my-otel-demo-emailservice-57d4cf7f8c-r75xl            1/1     Running            0                45m
my-otel-demo-checkoutservice-bf487f555-nv66x          1/1     Running            0                45m
my-otel-demo-jaeger-b54c97c5b-mqr4p                   1/1     Running            0                45m
my-otel-demo-quoteservice-6886646668-s55mv            1/1     Running            0                45m
my-otel-demo-kafka-59bb6fc9c6-djbgr                   1/1     Running            0                45m
my-otel-demo-accountingservice-c89755b9f-lfc2j        1/1     Running            0                45m
my-otel-demo-otelcol-6587f49484-lfrgv                 1/1     Running            0                45m
my-otel-demo-loadgenerator-7d7d8c5cf7-rz6p9           1/1     Running            0                45m
my-otel-demo-ffspostgres-59c75468f6-t9s9w             1/1     Running            0                45m
my-otel-demo-adservice-7fbdf74578-xfc97               1/1     Running            0                45m
my-otel-demo-recommendationservice-7d669c79bb-tn6kk   1/1     Running            0                45m
my-otel-demo-currencyservice-5668f894c5-vwx9p         1/1     Running            0                45m
my-otel-demo-cartservice-5f497c4fc8-qgptv             1/1     Running            0                45m
my-otel-demo-frontend-7f88b74488-xh9vm                1/1     Running            0                45m
my-otel-demo-redis-5b4b84644c-cgskx                   1/1     Running            0                45m
my-otel-demo-productcatalogservice-6cb65cc977-6rlb2   1/1     Running            0                45m
my-otel-demo-shippingservice-654b7c9694-vmfl4         1/1     Running            0                45m
my-otel-demo-frauddetectionservice-6477d4fbff-vpgzc   1/1     Running            0                45m
my-otel-demo-frontendproxy-67796969d6-nvjrx           1/1     Running            0                45m
my-otel-demo-grafana-55c4b589f9-whghq                 1/1     Running            0                45m
my-otel-demo-featureflagservice-7d6cbddd-p6ppp        1/1     Running            2 (45m ago)      45m
my-otel-demo-paymentservice-758cd56c47-ghp4l          0/1     CrashLoopBackOff   13 (3m58s ago)   45m

$ kubectl logs my-otel-demo-paymentservice-758cd56c47-ghp4l

><EMAIL_ADDRESS>start
> node opentelemetry.js

Environment
Ubuntu 22.04.1 LTS on x86
k0s v1.25.4+k0s.0
How to reproduce?
Deploy the demo app helm chart following the Kubernetes deployment guide:
https://github.com/open-telemetry/opentelemetry-demo/blob/main/docs/kubernetes_deployment.md

@mviitane can you try increasing the requests/limits of the paymentservice in your values.yaml?

Thanks @TylerHelmuth! I increased the limits like this. Also, featureflag service required an increase:
components:
  paymentService:
    resources:
      limits:
        memory: 100Mi
  featureflagService:
    resources:
      limits:
        memory: 200Mi

After getting the pods running, the memory consumption wasn't that high:
$ kubectl top pods
NAME                                                  CPU(cores)   MEMORY(bytes)
my-otel-demo-accountingservice-c89755b9f-vbwxb        3m           6Mi
my-otel-demo-adservice-7fbdf74578-bncm8               3m           159Mi
my-otel-demo-cartservice-5f497c4fc8-2f4tw             2m           47Mi
my-otel-demo-checkoutservice-bf487f555-6nh8p          2m           9Mi
my-otel-demo-currencyservice-5668f894c5-crmww         1m           1Mi
my-otel-demo-emailservice-57d4cf7f8c-m6gtj            1m           46Mi
my-otel-demo-featureflagservice-6f64c65dfd-pxrdw      7m           146Mi
my-otel-demo-ffspostgres-59c75468f6-gbcql             1m           49Mi
my-otel-demo-frauddetectionservice-6477d4fbff-jmgj5   9m           132Mi
my-otel-demo-frontend-7f88b74488-zc9rj                32m          94Mi
my-otel-demo-frontendproxy-67796969d6-gc4c7           7m           24Mi
my-otel-demo-grafana-55c4b589f9-lw9g6                 3m           64Mi
my-otel-demo-jaeger-b54c97c5b-dmn6k                   2m           55Mi
my-otel-demo-kafka-59bb6fc9c6-vj6hs                   21m          405Mi
my-otel-demo-loadgenerator-7d7d8c5cf7-98pwq           10m          45Mi
my-otel-demo-otelcol-6587f49484-8qrvq                 12m          113Mi
my-otel-demo-paymentservice-7b6dcf5c4f-tdrkd          1m           48Mi
my-otel-demo-productcatalogservice-6cb65cc977-dtj97   4m           7Mi
my-otel-demo-quoteservice-6886646668-p75xb            1m           26Mi
my-otel-demo-recommendationservice-7d669c79bb-vrgt6   8m           39Mi
my-otel-demo-redis-5b4b84644c-pkk2b                   2m           2Mi
my-otel-demo-shippingservice-654b7c9694-cxrc5         1m           1Mi

Should the default limits be increased anyway to avoid the problem?

@puckpuck what do you think?

See also #595

i had the same issue, increasing the memory limit of paymentService fixed it for me
See also: https://github.com/open-telemetry/opentelemetry-demo/pull/525


Verlinkung in der Hilfe zu den station_ids fehlerhaft

Version of home_assistant
die aktuellste
Version of the custom_component

Describe the bug
Verlinkung in der Hilfe zu den station_ids fehlerhaft
Debug log

Add your logs here.

I think this is the correct information?
https://www.dwd.de/DE/leistungen/met_verfahren_mosmix/mosmix_stationskatalog.cfg;jsessionid=491F2BA487383F46185EB694E65FE5B5.live31094?view=nasPublication&nn=16102
Regards Roli

Yeah, this is my bad. Sorry for this. I will publish an update of the integration in a few weeks, which makes them obsolete. And in preparation for this I updated the files with the station_ids. So your link is the correct one and I will leave this issue open for reference.
Best regards

Obsolete with v2.0.0.


Latest dave

Checklist:

[ ] Read the Code of Conduct and Contributing Guidelines.

[ ] Followed the commit message guidelines.

[ ] Stack adheres to Appsody stack structure.

Modifying an existing stack:

[ ] Updated the stack version in stack.yaml

Contributing a new stack:

Describe how application dependencies are managed:

Explain how Appsody file watcher is utilized:

Describe other Appsody environment variables defined by the stack image:

Describe any limitations and known issues:

Related Issues:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.8 out of 12 committers have signed the CLA.:white_check_mark: uberskigeek:white_check_mark: garypicher:white_check_mark: groeges:white_check_mark: kilnerm:white_check_mark: scottkurz:white_check_mark: djones6:white_check_mark: gireeshpunathil:white_check_mark: ajm01:x: Rick Trotter:x: dettlaff:x: arunavemulapalli:x: davco01aRick Trotter seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Benchmark LattePanda Mu

As the title says... see https://github.com/geerlingguy/sbc-reviews/issues/42
This system uses a single Intel N100 CPU with unlocked TDP thanks to an aluminum active cooler.

Power usage during the benchmark averages 26W, with peaks to 27W.

63.225 Gflops at 26W, giving 2.43 Gflops/W
================================================================================
HPLinpack 2.3  --  High-Performance Linpack benchmark  --   December 2, 2018
Written by A. Petitet and R. Clint Whaley,  Innovative Computing Laboratory, UTK
Modified by Piotr Luszczek, Innovative Computing Laboratory, UTK
Modified by Julien Langou, University of Colorado Denver
================================================================================

An explanation of the input/output parameters follows:
T/V    : Wall time / encoded variant.
N      : The order of the coefficient matrix A.
NB     : The partitioning blocking factor.
P      : The number of process rows.
Q      : The number of process columns.
Time   : Time in seconds to solve the linear system.
Gflops : Rate of execution for solving the linear system.

The following parameter values will be used:

N      :   23314
NB     :     256
PMAP   : Row-major process mapping
P      :       1
Q      :       4
PFACT  :   Right
NBMIN  :       4
NDIV   :       2
RFACT  :   Crout
BCAST  :  1ringM
DEPTH  :       1
SWAP   : Mix (threshold = 64)
L1     : transposed form
U      : transposed form
EQUIL  : yes
ALIGN  : 8 double precision words

--------------------------------------------------------------------------------

- The matrix A is randomly generated for each test.
- The following scaled residual check will be computed:
      ||Ax-b||_oo / ( eps * ( || x ||_oo * || A ||_oo + || b ||_oo ) * N )
- The relative machine precision (eps) is taken to be               1.110223e-16
- Computational tests pass if scaled residuals are less than                16.0

================================================================================
T/V                N    NB     P     Q               Time                 Gflops
--------------------------------------------------------------------------------
WR11C2R4       23314   256     1     4             133.63             6.3225e+01
HPL_pdgesv() start time Sat May  4 20:38:30 2024

HPL_pdgesv() end time   Sat May  4 20:40:44 2024

--------------------------------------------------------------------------------
||Ax-b||_oo/(eps*(||A||_oo*||x||_oo+||b||_oo)*N)=   3.29349238e-03 ...... PASSED
================================================================================

Finished      1 tests with the following results:
              1 tests completed and passed residual checks,
              0 tests completed and failed residual checks,
              0 tests skipped because of illegal input values.
--------------------------------------------------------------------------------

End of Tests.
================================================================================

Re-tested today because I realized I had a 1TB USB flash drive plugged in during the benchmark, sucking down like 1W extra.
It's closer to 25W average, and I got 62 Gflops, across two runs, for 2.48 Gflops/W


[ja]move HostAliases page to tasks

related PR: #28631
fix issue: #28630

Optionally: add a redirect from the old to the new page.

As discussed in https://github.com/kubernetes/website/pull/28730#discussion_r662089797, we decided to not flood the redirect with the localization entries since it's very unlikely someone bookmarks that url.

/assign
/approve
/cc @makocchi-git @kakts @bells17 @inductor

I compared with en page, and also confirmed this PR is the same modification as zh and ko.
/lgtm


[feature] Change the code review note line index, if a line is added or deleted before it

🧩 Feature
If a new line is added, or if a line is deleted, amon the lines before a code review note, change the code review note line index.
Description
In case of code modification (ex : add a new line, delete a line), the Code Review notes currently do not follow the code, but remain at the same line. The goal of the feature is to keep the code review note at the same line as the code considered by the note.
 A clear and concise description of the feature

Any update on this issue, when this fix will be coming. Without this it seems difficult to communicate back and forth (comment/re-comment etc.) between dev and reviewer.

Unfortunately not. I won't be able the next time to focus on this issue and fix it.
Please: feel free to fork and contribute!
I would appreciate it!


"Error installing electron" and "Error extracting ASAR"

For the last couple of versions of BluOS Controller, I was able to download the AppImage files shared on https://support1.bluesound.com/hc/en-us/community/posts/360033533054-BluOS-controller-app-on-Linux, and they seemed to work just fine. Since no AppImage for v3.16 was posted on the forum though, I tried to build it myself following these steps on Linux Mint 20.2, but have encountered a few errors.

Ran "sudo apt install npm"
Running "which npm" gives "/usr/bin/npm"
Running "npm -v" gives "6.14.4"
Running "node -v" gives "v10.19.0"
Downloaded the bs-patch binary
Created a file called config.yaml with the following contents: nodeBinPath: "/usr/bin/"
Ran "./bs-patch", which gives the following results:

INFO[0000] Reading config.yaml
INFO[0000] Finding Latest BluOS Controller
INFO[0003] Downloading Latest BlueOS controller: https://content-bluesound-com.s3.amazonaws.com/uploads/2021/11/BluOS-Controller-3.16.0.dmg
Downloading... 248 MB complete
INFO[0055] Extracting BlueOS Controller DMG
INFO[0055] Extracting ASAR
INFO[0055] Patching electron.js
INFO[0055] Patching app.js - Update Check
INFO[0055] Patching app.js - Update Platform
INFO[0055] Adding electron dependency
ERRO[0055] Error installing electron:
INFO[0055] Adding electron-builder dependency
ERRO[0055] Error installing electron:
INFO[0055] Building Snap and AppImage
ERRO[0055] Error extracting ASAR:
INFO[0055] Cleaning up

Any thoughts on what I may be doing wrong?
Thanks!

Same here. Seems the dmg of later versions cannot be extracted correctly with 7z. Still havent found a way to make the build finish.


Do not conflict between ender and jQuery

also see #30

Thanks for the pull request, but there is an issue with this approach that doesn't guarantee it will work. Namely, ender does not implement noConflict(), so this will only work if jQuery is loaded after the menu plugin loads the ender libraries. #30 intended to remove the use of ender entirely to avoid the potential conflict completely, now fixed in dev.


Remove baking and finalization commission

Purpose
Remove any “baking” words in the frontend and remove finalization commission if the amount is zero.
Task: https://concordium.atlassian.net/browse/CBW-1465
Changes

The terms baker and baking have been removed for anything which is rendered
Finalization commissions are only shown when the amount is non zero. This makes the views backward compatible.

Checklist

[x] My code follows the style of this project.
[ ] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.

I am uncertain if it is a requirement or not, but should the URL for viewing a validator also be updated? Right now it is like e.g. https://testnet.ccdscan.io/staking?dcount=1&dentity=baker&did=3113, and I am wondering if it should be https://testnet.ccdscan.io/staking?dcount=1&dentity=validator&did=3113.

I am uncertain if it is a requirement or not, but should the URL for viewing a validator also be updated? Right now it is like e.g. https://testnet.ccdscan.io/staking?dcount=1&dentity=baker&did=3113, and I am wondering if it should be https://testnet.ccdscan.io/staking?dcount=1&dentity=validator&did=3113.

I would expect it should change. I have updated PR and ask for clarification on this.

I am uncertain if it is a requirement or not, but should the URL for viewing a validator also be updated? Right now it is like e.g. https://testnet.ccdscan.io/staking?dcount=1&dentity=baker&did=3113, and I am wondering if it should be https://testnet.ccdscan.io/staking?dcount=1&dentity=validator&did=3113.

I would expect it should change. I have updated PR and ask for clarification on this.

Yes, it would be good to change this too. Maybe for backwards compatibility, the url could still work with “baker”, but whenever we generate a URL, we do it with “validator”?

Purpose
Remove any “baking” words in the frontend and remove finalization commission if the amount is zero.

I don't mean to be unnecessarily picky, I just want to avoid any confusion: there seems to be a mix-up between finalization rewards and commissions in this description. Finalization rewards should not be displayed if they are 0. The finalization commissions for a pool should not be displayed any more, not conditioned on anything.

I am uncertain if it is a requirement or not, but should the URL for viewing a validator also be updated? Right now it is like e.g. https://testnet.ccdscan.io/staking?dcount=1&dentity=baker&did=3113, and I am wondering if it should be https://testnet.ccdscan.io/staking?dcount=1&dentity=validator&did=3113.

I would expect it should change. I have updated PR and ask for clarification on this.

Yes, it would be good to change this too. Maybe for backwards compatibility, the url could still work with “baker”, but whenever we generate a URL, we do it with “validator”?

Works now with both links. Using old baker will have you routed to url with validator.


I get a typescript error in defineContentScript

See image. I do have a yarn mono repo. I used react template
This is the error: The inferred type of 'default' cannot be named without a reference to '../../node_modules/wxt/dist/index-TYmXqKVq'. This is likely not portable. A type annotation is necessary.ts(2742)

Please provide a minimal reproduction. It can be a zip file of a simple monorepo.

@aklinker1 looks like if I remove "composite: true" (because I have a monorepo) from tsconfig the error goes away

Hmm, I need to do more research into what composite and this error message means. I've never had to use composite for monorepos, but I bet there's a reason to enable it...


Allow reporting status from Manifest function of AgentAddon

When the agent reconciles the target's state in the Manifest function, it might encounter an error, which at this moment doesn't allow it reconcile.
However, there seems to be no proper way to record this, other than returning an error, which will re-trigger the reconciliation.
I think it would be helpful if the agent could contribute to the conditions of the ManagedClusterAddOn resource.

hrm, I think it makes sense. we need to think also whether directly return error make sense since it triggers reconcile and it is highly possible to return error again.

Maybe there can be two kinds of errors: temporary error (which re-trieggers) and permanent error (which does not)


StatFiles.jl integration

as suggested in
https://discourse.julialang.org/t/ann-tableio-simple-reading-and-writing-of-tabular-data-experimental/47427/5

Codecov Report

Merging #8 into master will increase coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   95.18%   95.40%   +0.22%
==========================================
  Files           3        3
  Lines          83       87       +4
==========================================
+ Hits           79       83       +4
  Misses          4        4

Impacted Files
Coverage Δ

src/TableIO.jl
100.00% <100.00%> (ø)

src/file_io.jl
91.66% <100.00%> (+0.17%)
:arrow_up:

Continue to review full report at Codecov.

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


Win32

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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

@conda-forge-admin rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

requirements: host: liblapack >= 3.8 should not contain a space between relational operator and the version, i.e. liblapack >=3.8

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

@conda-forge-admin, rerender

note: a patch may be necessary to resolve the "module" issue
https://github.com/xoviat/flang/blob/windows/tools/driver/Upper.cpp#L846-L852

fixed in xoviat/flang@30b5411f8226386a642fdee0429101ed153c6604; need rebuild

ref conda-forge/flang-feedstock#20

It looks like this is the problem:
flang.exe:f90: src\FFTW3.f95
  F90-F-0004-Unable to open MODULE file iso_c_binding.mod (src\FFTW3.f95: 7)

The FFTW3.f95 file uses this line:
  use, intrinsic :: iso_c_binding

I can't remember the details here, but it is my understanding that this is an F2008(?) extension, and that it is implemented by "almost all" fortran compilers. It looks like "flang.exe" doesn't implement this yet.
When we build the windows wheels, I believe that we are using the mingw gfortran compiler: https://github.com/SHTOOLS/build-shtools/blob/master/appveyor.yml . Perhaps we should try to use mingw with the conda build as well???

I am the author of flang for windows and the numpy blas detection. The solution is to fix flang so that it works correctly.  The numpy blas detection failed because lapack was pinned (cb335fc). flang supports iso_c_binding, and conda build works correctly when run on my local computer. I will try to display the include path so see what is happening.

GetShortPath is used to convert paths. I wonder if that's the problem...

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

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

build fails because the winapi _access('\bld\PYSHTO~1\_BUILD~1\Library/include/flang/iso_c_binding.mod', 0) returns -1, when the file is present:
 Directory of D:\bld\PYSHTO~1\_BUILD~1\Library\include\flang

02/02/2021  11:00 PM    <DIR>          .
02/02/2021  11:00 PM    <DIR>          ..
01/31/2021  08:10 PM            66,630 ieee_arithmetic.mod
01/31/2021  08:10 PM            66,658 ieee_arithmetic_la.mod
01/31/2021  08:10 PM            35,597 ieee_exceptions.mod
01/31/2021  08:10 PM            35,625 ieee_exceptions_la.mod
01/31/2021  08:10 PM             7,626 ieee_features.mod
01/31/2021  08:10 PM            11,666 iso_c_binding.mod

I cannot reproduce this on my local pc, or with conda build. Unclear what the next step is.

see https://github.com/xoviat/flang/commit/7c11324d27349d23feb93309d5710cb28fdda0d8

what's more confusing is that it returned path acess before.

https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=272726&view=logs&j=00f5923e-fdef-5026-5091-0d5a0b3d5a2c&t=3cc4a9ed-60e1-5810-6eb3-5f9cd4a26dba&l=495

@moorepants it's ready.

The changes seem to be adding python setup.py build, removing the pins on fftw and liblapack, adding blas/cblas explicitly, and disabling the test on windows. It imports on windows, but should there be at least a test to show that the bindings to the lower level codes do work?

Numpy is not working in the test for some reason; not sure why. flang also doesn't seem to work inside pip for some reason.

Do the tests run on your personal machine? If you could point me to the error I will take a look at it.

The error is here: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=274630&view=logs&j=e5cdccbf-4751-5a24-7406-185c9d30d021&t=66e4a587-a723-53e9-873c-6c9a2c5971ea
Traceback (most recent call last):
  File "./ClassExample.py", line 5, in <module>
    import numpy as np
  File "D:\bld\pyshtools_1612650140689\_test_env\lib\site-packages\numpy\__init__.py", line 145, in <module>
    from . import core
  File "D:\bld\pyshtools_1612650140689\_test_env\lib\site-packages\numpy\core\__init__.py", line 48, in <module>
    raise ImportError(msg)
Makefile:63: recipe for target 'ClassInterface/ClassExample.py.phony' failed
ImportError:
make: Leaving directory '/cygdrive/d/bld/pyshtools_1612650140689/test_tmp/pyshtools-4.8/examples/python'

IMPORTANT: PLEASE READ THIS FOR ADVICE ON HOW TO SOLVE THIS ISSUE!

Importing the numpy C-extensions failed. This error can happen for
many reasons, often due to issues with your setup or how NumPy was
installed.

not sure what the problem is. I usually can run dependency walker to find the dependencies, but not sure how to do that.

@MarkWieczorek is there a way to download the artifacts?

@conda-forge-admin rerender

@MarkWieczorek not sure which DLL is failing to load. Dependency walker shows the following dependencies:

LIBCBLAS
PYTHON39
KERNEL32
VCRUNTIME140

note: I just noticed that the 3.6 build does pass.

I'm going to close this given that the windows packages are now building....


_fs.readFileSync is not a function

This shows up while following the simple steps in npm documentation

Can you send the entire error log along with the Node.js version you are using?

I have the same Problem. The reason seems to be that clientside js cant use the fs(FileSystem).
Here is the same issue with another package:
https://github.com/SheetJS/sheetjs/issues/418
According to the Answers there, the Problem can be solved by using a html Input tag and reading the file that way. But i have not tried it yet.
I hope that helps.

It works! But the code has to be modified. I added a posibiliy to pass a "type"  argument to the underlying xlsx package.
My code looks like this:
const onChange = (e) => {
    const reader = new FileReader();
    reader.onloadend = function (event) {
      console.log(event.target.result);
      const tmp = excelToJson({ source: event.target.result });
      console.log(tmp);
    };
    reader.readAsArrayBuffer(e.target.files[0]);
  };

<input type="file" onChange={onChange} />

and the change in convert-excel-to-json.js looks like this:
if (_config.source) {
            workbook = XLSX.read(_config.source, {
                type:"buffer",
                sheetStubs: true,
                cellDates: true
            });

Henrik's change in convert-excel-to-json.js also worked for me, with a slight variation on where the array buffer comes from:
      if (this.file.type === 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet') {         this.file.arrayBuffer()           .then(buffer => {             const batchJson = excelToJson({               source: this.file.arrayBuffer(),               header: { rows: 1 },               columnToKey: { '*': '{{columnHeader}}' }             })             alert(batchJson)           })       }

It works! But the code has to be modified. I added a posibiliy to pass a "type" argument to the underlying xlsx package.
My code looks like this:
const onChange = (e) => {
    const reader = new FileReader();
    reader.onloadend = function (event) {
      console.log(event.target.result);
      const tmp = excelToJson({ source: event.target.result });
      console.log(tmp);
    };
    reader.readAsArrayBuffer(e.target.files[0]);
  };

<input type="file" onChange={onChange} />

and the change in convert-excel-to-json.js looks like this:
if (_config.source) {
            workbook = XLSX.read(_config.source, {
                type:"buffer",
                sheetStubs: true,
                cellDates: true
            });

This worked for me as well. Thank you!

Works perfect!!! @DiegoZoracKy Could commit the change pls?


Handling multiple embeds of same video from different sources.

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Hello,
I am writing an extractor for a site that has multiple embeds of the same video in different resolutions from different sites.
As an example consider embeds from site A (360p, 720p) and site B(720p). The extractors for both site A and B already exist
Then what is the correct thing to return from the extractor.

One case can be doing all the filtering within my extractor and then split out an array that has all direct URLs.
Other can be (if youtube-dl supports it) return two extractor objects of A and B and let youtube-dl figure out the urls.

Not sure if the solution in case 2 exists in the current code. I tried looking for it couldn't find it.

it depends, without having enough information, i can only point you to an example, this is from the NBA extractor that combines the information from two sources.


More info on "in a way that sends data to Adobe, such as clicking a link or a video start."`

Issue in help/components/c-variables/c-metrics/metrics-bounces.md
If a customer lands on a page and reads the content for 10 min, then leaves, does that count as a bounce?

@lamek: Yes, it does.


NullPointerException

Hy, I cloned the project and it gives NullPointerException in line                        socket.getChannelByName("MyClassroom").publish(object); .  Can you help me, what to do?

Hi @Hayko985 , I will close the issue if your problem is resolved. Please let me know about it.

Hi @sacOO7 )) Really I already forgot about this))) haha . Sure you can close.

Hey no problem. Feel free to raise issues on official repo.

@mahima-ms Hi. My problem was, because getChannelByName() throw NullPointerException. The problem was that the channel sometime not created and even when I checked if (getChannelByName != null), that throw NullPointerException too.  The only solution was, that I add the getChannelByName() in try-catch block. That helped me.
Something like this.
try{
    socket.getChannelByName("MyClassroom").publish(object);
} catch (NullPointerException n) {
    n.printStackTrace();
}

So the next question is why my channel become null?  Ok let me explain. Sometimes, when screen gone off on phone or connection closed, or something else - socket disconnected after sometime, because
when screen goes off, application calls onStop (as I remember) and socket disconnected, after this my connection become null. I think if I'll add my Socket in Background using WorkManager, this problem will be solved))

Awesome! Understood. Thank you very much @Hayko985 for such a detailed explanation on the issue and sharing the resolution. Appreciate your time :)

@mahima-ms Thank you too. Good luck :)


Upgraded webpack to 1.15.0

Upgraded webpack to 1.15.0 due to the ENOENT error I got on initial installation/start of application: https://github.com/davezuko/react-redux-starter-kit/issues/1197#issuecomment-301264325

Thanks for the PR. It looks like you have added in a few extra changes that don't relate to this fix, however. If you could fixed those plus update the yarn.lock file then we can get this merged. Thanks!

Project has officially migrated to webpack 2: https://github.com/davezuko/react-redux-starter-kit/releases/tag/v3.0.0.


"no-unused-vars" rule false positive in inherited contracts

I get an [ERROR] At line: x, column: y -> Variable 'someAddress' is declared but never used. in the scenario below, in which the variable is used in contracts inheriting from the base but not in the base contract itself.
contract IMyContract {

  address public someAddress;

  function myFunction(address someVariable);
}

import 'IMyContract.sol';
contract MyContract is IMyContract {
  function someOtherFunction(address X)
  {
    someAddress = X;
  }
}

I'm unable to reproduce the error with the code you've posted. In fact, when I comment out someAddress = X;, then it gives me the var-not-used error (as expected), but when its not commented out, there is no error.
Could you tell me which version you're using? (solium --version)

<EMAIL_ADDRESS>

I just installed 0.1.8 and it is still throwing the error

---------------------------------------------------------------------------------------
FILE:	/contracts/IMyContract.sol
---------------------------------------------------------------------------------------

[WARNING] At line: 1, column: 0 -> Contract name 'IMyContract' doesn't follow the CamelCase notation

[ERROR] At line: 3, column: 2 -> Variable 'someAddress' is declared but never used.

Ahh! When I test the whole code in a single file, it doesn't pose any errors.
You must've declared IMyContract in 1 file, and MyContract in another.
At a time, Solium lints only a single file, so it is not informed whether you are using the variable in some other file or not.
This needs fixing.

Putting this link here for my own (future) reference
https://eslint.org/docs/rules/no-unused-vars#exported

this issue no longer persists since solium does not check state variables for usage any more.
feel free to re-open in case you face this (or something similar).


Does it support Chinese?

No Chinese yet: https://snips-nlu.readthedocs.io/en/latest/languages.html
Supported Languages:
German
English
Spanish
French
Japanese
Korean

I am wondering why it supports Japanese and Korean bit not Chinese, technically, those East Asian language are same,

Hi @loveJasmine, tokenization is not really handle right now in the NLU.
The implicit convention is that you input string is already tokenized and joined on space, so yes if your input is already tokenized it does not make much difference.
We're close to have a first version of Snips NLU for Chinese. We've started working on it but haven't had the resources to finish it.
As explained here one of the main step to add a language is to write rustling grammars to correctly support builtin entities. We've done part of this work but not finished it, we'll try to find time soon to release a version of the NLU with a support of Chinese even if it's not perfect.

@ClemDoum Maybe a detailed roadmap to implement&improve the support of new language,will help us join to contribute.


fixes #523

Pull Request Template
Fixes # 523
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

guration
Additional Context (Please include any Screenshots/gifs if relevant)

...
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings
[x] I have tagged the reviewers in a comment below incase my pull request is ready for a review
[x] I have signed the commit message to agree to Developer Certificate of Origin (DCO) (to certify that you wrote or otherwise have the right to submit your contribution to the project.) by adding "--signoff" to my git commit command.

@joewxboy

This commit fixes #523
Not #525

I recommend a rewrite of the commit message so that it follows the Open Horizon convention.

This commit fixes #523 Not #525

my bad. I have fixed the commit message

Better! Thanks.  Can you rewrite the git commit message to be
Issue #523 - style: fix navbar text color


Truncate should set TC flag more often

On #854, @markdingo raised the following:

On a related front, I think the logic behind setting the Truncate flag is a bit simplistic.
The code comment says "According to RFC 2181, the TC bit should only be set if not all of the answer RRs can be included in the response." and the code does precisely this. But that's not what RFC2181 says. It says TC should be set if a "required" RRSet doesn't fit, regardless of whether that required RRSet is in the answer section or not.
Admittedly that whole section in RFC2181 is pretty muddled, but there is at least one use-case where TC should be set when the additional section is truncated. That's when the additional section contains referral/glue RRs for a sub-domain. By way of example, if we imagine a query to an example.net name server for sub.example.net/NS which results in:
;; QUESTION SECTION:
;sub.example.net.               IN      NS
;; AUTHORITY SECTION:
sub.example.net.        172800  IN      NS      c.sub.example.net.
sub.example.net.        172800  IN      NS      a.sub.example.net.
sub.example.net.        172800  IN      NS      b.sub.example.net.
;; ADDITIONAL SECTION:
c.sub.example.net.       172800  IN      A       ...
a.sub.example.net.       172800  IN      A       ...
b.sub.example.net.       172800  IN      A       ...
If just the additional section is truncated by msg.Truncate() then the client will fail to resolve the sub-domain as it won't see TC=1 to re-query with TCP and trying to resolve the sub NS address via the parent doesn't work since the address RRs are glue. You can try this for yourself with one of the TLDs such as .com. Imagine the additional section for google.com/NS is truncated with TC=0. You'll never be able to resolve google.com as you never see the NS address glue and can't get at it directly via the .com servers.
The problem here is that msg.Truncate() really has no idea whether non-Answer RRSets are "required" or not since it's just a generic function that is oblivious to the caller's intentions or role.
I'm not sure what the right approach is. Either the description of msg.Truncate() should accurately reflect that it's setting of TC is only a guide and the caller really has to make the final TC=1 decision. Alternatively it could follow the approach that PowerDNS takes in the next release which is to set TC on any truncation. This doesn't disagree with RFC2181 and it is a safer approach.
And, if you think about it, setting TC=1 only with an Answer truncation vs any truncation is mostly just a corner-case optimisation so forgoing this optimisation is unlikely to be harmful so the powerDNS approach isn't terrible.
Maybe not knowing what is "required" as per RFC2181 explains why some of the main DNS packages force the matter by wiping out all sections and setting TC=1 when any truncation is needed? They never give the client a chance to use a partial answer.
Anyway, there is at least one use-case where relying on msg.Truncate() to set TC correctly seems to be risky.

I’m opening a new issue since that PR has already been merged and comments aren’t always seen.
I’m fine making this change if others agree. The reasoning is sound and setting the TC flag when any record was removed was how I originally planned to implement this.

/cc @miekg and @gibson042 who had insight on the original PR.

I’m opening a new issue since that PR has already been merged and comments aren’t always seen.

Thanks Tom.

[ Quoting<EMAIL_ADDRESS>in "[miekg/dns] Truncate should set TC ..." ]

I'm not sure what the right approach is. Either the description of msg.Truncate() should accurately reflect that it's setting of TC is only a guide and the caller really has to make the final TC=1 decision. Alternatively it could follow the approach that PowerDNS takes in the next release which is to set TC on any truncation. This doesn't disagree with RFC2181 and it is a safer approach.

I like the PowerDNS approach the most as this doesn't require to somehow interpret
the response (i.e. is it a referral?).

I like the PowerDNS approach the most as this doesn't require to somehow interpret the response (i.e. is it a referral?).

Yeah, I wondered about "interpreting" too. Trying to understand whether additional has referral info or something else that is "required" seems like a complexity minefield. It also presupposes that we know about future dependencies that could end up in non-Answer sections such as cryptographic keys or DNSSEC related material or who knows what?
Unfortunately TC is binary but it's trying to serve two masters so taking the PowerDNS approach eliminates the benefits that the current implementation is trying to offer to the first master.
The two masters of course are a) Have Answers been truncated and b) has the DNS packet been truncated.
By far the "Have Answers been truncated" is of interest to the overwhelmingly dominant A/AAAA queries which is what the current msg.Truncate() implementation cares about but we're now saying that the risk to the other master is non-zero so the majority have to suffer.
What we really need are two TC bits. One to say Answers has been truncated and one to say the packet has been truncated but TCP re-queries are cheap these days and larger EDNS0 query support is pervasive so why bother with the complexity?
So yeah, I think PowerDNS is doing the right thing. I also think that even tho most DNS implementations clear all Answers when they set TC=1 (because they expect a re-query) the smart move is what msg.Truncate() does which is to leave as much intact as possible in case a client is incapable of re-querying.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] Truncate should set..." ]

So yeah, I think PowerDNS is doing the right thing. I also think that even tho most DNS implementations clear all Answers when they set TC=1 (because they expect a re-query) the smart move is what msg.Truncate() does which is to leave as much intact as possible in case a client is incapable of re-querying.

"incapable of re-querying" is basically anything that use musl libc - unless
they have implemented DNS over TCP now.
/Miek
--
Miek Gieben

"incapable of re-querying" is basically anything that use musl libc - unless they have implemented DNS over TCP now.

Or if you're forwarding a packet to a limited resolver/proxy. I don't know whether it's still the case, but at some point dnsmasq only supported UDP queries and it's embedded in a lot of home routers and DSL gateways and the like.
In fact dnsmasq used to clear TC=1 on inbound packets because it knew it couldn't answer TCP queries. I think that's wrong, but quite possibly it was to protect against clients which presumed that they could discard their partial UDP response and get a bigger response via TCP before they actually made the re-query.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] Truncate should set..." ]

"incapable of re-querying" is basically anything that use musl libc - unless they have implemented DNS over TCP now.

Or if you're forwarding a packet to a limited resolver/proxy. I don't know whether it's still the case, but at some point dnsmasq only supported UDP queries and it's embedded in a lot of home routers and DSL gateways and the like.
In fact dnsmasq used to clear TC=1 on inbound packets because it knew it couldn't answer TCP queries. I think that's wrong, but quite possibly it was to protect against clients which presumed that they could discard their partial UDP response and get a bigger response via TCP before they actually made the re-query.

Wow...
anyway: a PR implementing this will be merged :)

I like the change but I wonder whether it's missing the fact that the EDNS0 RR is removed at the top of the code and re-appended at the end after the length check? Note the popEdns0() call at the top of the function?
Maybe a test case with a call to SetUDPSize(4096) would catch this?
As an aside, with that change to the semantics, can the truncation of the RR slices be avoided if Truncated is not set? Probably a silly optimisation, but why not?
if dns.Truncated {
dns.Answer = dns.Answer[:numAnswer]
dns.Ns = dns.Ns[:numNS]
dns.Extra = dns.Extra[:numExtra]
}


Add morphometrics algorithms description

Checklist

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[x] I've applied the relevant labels to this PR
[ ] I've added relevant tests for my contribution
[x] I've updated the documentation and/or added correct docstrings
[x] I've assigned a reviewer
[x] I've consulted ADS's internal developer documentation to ensure my contribution is in line with any relevant design decisions

Description

This PR adds some details about the morphometrics computation. In particular, the diameter and eccentricity estimations are detailed. I would also like to eventually describe how the perimeter is computed, but #776 needs to be adressed first.

um the doc build fails due to "excessive memory consumption"? I'll have to look into this tomorrow, not sure what the problem is.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1175

Relevant Lines:
1316

💛  - Coveralls

@mathieuboudreau I'm a bit stumped here. Initially the doc build failed (timed out), but then I re-triggered the tests (which did not re-trigger the rtd build unfortunately) and even the tests failed with coveralls.exception.CoverallsException: Could not submit coverage: 422 Client Error: Unprocessable Entity for url: https://coveralls.io/api/v1/jobs.
Any idea what's going on?

Have you seen this thread @hermancollin ? https://github.com/lemurheavy/coveralls-public/issues/1435
Maybe a token issue or their API has been updated. We could also try removing this project on coveralls and re-adding it. Do you have access?

Have you seen this thread @hermancollin ? lemurheavy/coveralls-public#1435
Maybe a token issue or their API has been updated. We could also try removing this project on coveralls and re-adding it. Do you have access?

I don't think I have access. I saw the thread but why would it fail now? You pushed PRs with changes to the doc this month

No idea why it would fail this morning; a test passed last night!

I just started a separate branch PR to see how it behaves there

I see that on RTD, the error is this: "Error
Build exited due to excessive memory consumption
"
Likely, something changed in one of the dependencies and it messed up things (maybe version resolution?).

Ahh! It works now. Thank you @mathieuboudreau. The new part in the doc can be seen here: https://axondeepseg--777.org.readthedocs.build/en/777/documentation.html#implementation-details


Builing RealSense 2.42.0 with OpenVINO support results in - openvino-helpers.h:189:5: error: expected class-name before ‘{’ token

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 20.04

Kernel Version (Linux Only)
5.8.0-43-generic

Platform
Ubuntu 20.04 VirtualBox VM

SDK Version
2.42.0

Language
C

Segment
Computer Vision

Issue Description
While compiling RealSense 2.42.0 with OpenVINO support from source on either a Raspberry Pi 4 or an Ubuntu 20.04 VM, the build fails with the following error:
[ 63%] Built target rs-distance
[ 64%] Built target rs-post-processing
[ 66%] Built target rs-record-playback
Scanning dependencies of target rs-motion
[ 67%] Building CXX object examples/motion/CMakeFiles/rs-motion.dir/rs-motion.cpp.o
In file included from /home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/base-detection.h:7,
                 from /home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/object-detection.h:6,
                 from /home/jovision/develop/RealSense/librealsense/wrappers/openvino/dnn/rs-dnn-vino.cpp:10:
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/openvino-helpers.h:189:5: error: expected class-name before ‘{’ token
  189 |     {
      |     ^
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/openvino-helpers.h:190:14: error: ‘void openvino_helpers::error_listener::onError(const char*)’ marked ‘override’, but does not override
  190 |         void onError( char const * msg ) noexcept override
      |              ^~~~~~~
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/dnn/rs-dnn-vino.cpp: In function ‘int main(int, char**)’:
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/dnn/rs-dnn-vino.cpp:219:12: error: ‘class InferenceEngine::Core’ has no member named ‘SetLogCallback’
  219 |     engine.SetLogCallback( error_listener );
      |            ^~~~~~~~~~~~~~

.
.
.

make[2]: *** [wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/build.make:63: wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/rs-face-vino.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1229: wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 65%] Building CXX object wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/__/rs-vino/detected-object.cpp.o
make[2]: *** [wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/build.make:76: wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/__/rs-vino/base-detection.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/build.make:63: wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/rs-dnn-vino.cpp.o] Error 1
make[2]: *** [wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/build.make:89: wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/__/rs-vino/object-detection.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1256: wrappers/openvino/dnn/CMakeFiles/rs-dnn-vino.dir/all] Error 2
make: *** [Makefile:130: all] Error 2
 removing possibly corrupted object files
[  1%] Built target fw
[  7%] Built target realsense-file
[ 42%] Built target realsense2
[ 47%] Built target pybackend2
[ 51%] Built target pyrealsense2
[ 51%] Building CXX object wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/rs-face-vino.cpp.o
In file included from /home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/base-detection.h:7,
                 from /home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/object-detection.h:6,
                 from /home/jovision/develop/RealSense/librealsense/wrappers/openvino/face/rs-face-vino.cpp:8:
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/openvino-helpers.h:189:5: error: expected class-name before ‘{’ token
  189 |     {
      |     ^
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/./rs-vino/openvino-helpers.h:190:14: error: ‘void openvino_helpers::error_listener::onError(const char*)’ marked ‘override’, but does not override
  190 |         void onError( char const * msg ) noexcept override
      |              ^~~~~~~
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/face/rs-face-vino.cpp: In function ‘int main(int, char**)’:
/home/jovision/develop/RealSense/librealsense/wrappers/openvino/face/rs-face-vino.cpp:41:12: error: ‘class InferenceEngine::Core’ has no member named ‘SetLogCallback’
   41 |     engine.SetLogCallback( error_listener );
      |            ^~~~~~~~~~~~~~
make[2]: *** [wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/build.make:63: wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/rs-face-vino.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1229: wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/all] Error 2
make: *** [Makefile:130: all] Error 2

The CMake command used is:
cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_EXAMPLES=true -DBUILD_PYTHON_BINDINGS=true -DPYTHON_EXECUTABLE=$(which python3) -DFORCE_LIBUVC=true -DBUILD_OPENVINO_EXAMPLES=true -DOpenCV_DIR=/opt/intel/openvino_2021/opencv/cmake -DINTEL_OPENVINO_DIR=/opt/intel/openvino_2021 ..

Hi @jomoengineer   As a starting point in investigating your case: if you need to use a VM with librealsense then Intel recommend using VMWare Workstation Player instead of Oracle VirtualBox in order to provide proper emulation of the USB 3 controller.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation.md

@MartyG-RealSense So that explains why the build is failing for the Raspberry PI as well?

It might be an influencing factor but I would not like to say for sure without knowing more details about your Pi 4 setup.  For example, when using Ubuntu with Pi instead of Raspbian, the last couple of cases I have handled that have been successful have involved Ubuntu Server.  This subject is discussed in the recent ROS-related case below:
https://github.com/IntelRealSense/realsense-ros/issues/1724#issuecomment-786508280

@MartyG-RealSense
The links you reference are not related to the issue I am seeing.
The Raspberry Pi I am using is running
Raspbian OS 10 (buster)
Kernel: 5.10.11-v7l+
The VirtualBox VM that is hosting Ubuntu 20.04 has USB3 support enabled with the RealSense device filters added.
Note, I can build, install and run both OpenVINO ToolKit 2021.2.185 and RealSense 2.42.0 (Without OpenVINO support) individually on both the Raspbian and Ubuntu VM config and the RealSense camera is indented and usable in both the Raspbian and Ubuntu instances.  The issue I am having, which is what I prefer to focus on in this issue, is that I see the posted error when attempting to build RealSense Release v2.42.0 (or v2.41.0) with OpenVINO support as listed in the RealSense OpenVINO Wrapper page at the bottom under Linux:
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/openvino
I am curious if anyone else is successful in building RealSense with OpenVINO support as shown in the Linux example?

I note that you are using OpenVINO ToolKit 2021.2.185.  Compatibility up to OpenVINO 2020 was added in SDK 2.41.0.
https://github.com/IntelRealSense/librealsense/pull/7851
I would therefore recommend using OpenVINO 2020 with the OpenVINO wrapper examples if it is possible for you to downgrade.

@MartyG-RealSense
Well, this seems to run on Windows 10, so I am not sure why there is a limitation with Linux. I have no interest in downgrading to a 2020 version of OpenVINO and will just write this up in my review as not being supported with Linux and works in Windows only.
Also, I will have a word or two with regards to the Intel support and this product.
You can close this issue. I have no interest in pursuing it any further and will seek another vendor option.

I'm sorry that you have not had a smooth experience with your OpenVINO implementation.  I will feed your comments back to Intel.

Just a note, with OpenVINO 2020.3 on the Raspberry Pi, it gets different error, although I believe this is due to ngraph not being ported on the Raspberry Pi.
```/usr/bin/ld: /opt/intel/openvino/opencv/lib/libopencv_dnn.so.4.3.0: undefined reference to ngraph::Node::get_input_size() const' /usr/bin/ld: /opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine.so: undefined reference to typeinfo for ngraph::op::v0::TensorIterator'
/usr/bin/ld: /opt/intel/openvino/opencv/lib/libopencv_dnn.so.4.3.0: undefined reference to ngraph::as_node_vector(std::vector<ngraph::Output<ngraph::Node>, std::allocator<ngraph::Output<ngraph::Node> > > const&)' /usr/bin/ld: /opt/intel/openvino/opencv/lib/libopencv_dnn.so.4.3.0: undefined reference to ngraph::Node::get_input_element_type(unsigned int) const'
/usr/bin/ld: /opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine.so: undefined reference to typeinfo for ngraph::op::v0::RegionYolo' /usr/bin/ld: /opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine.so: undefined reference to ngraph::op::v0::Tile::Tile(ngraph::Outputngraph::Node const&, ngraph::Outputngraph::Node const&)'
/usr/bin/ld: /opt/intel/openvino/opencv/lib/libopencv_dnn.so.4.3.0: undefined reference to ngraph::Node::description[abi:cxx11]() const' /usr/bin/ld: /opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine.so: undefined reference to typeinfo for ngraph::op::v1::ReduceLogicalOr'
/usr/bin/ld: /opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine.so: undefined reference to typeinfo for ngraph::op::v0::Abs' /usr/bin/ld: //opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine_transformations.so: undefined reference to ngraph::pass::ConstantFolding::construct_constant_range()'
/usr/bin/ld: //opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine_transformations.so: undefined reference to ngraph::op::v1::Pad::get_pads_begin() const' /usr/bin/ld: /opt/intel/openvino/opencv/lib/libopencv_dnn.so.4.3.0: undefined reference to ngraph::Node::m_next_instance_id'
/usr/bin/ld: /opt/intel/openvino/inference_engine/lib/armv7l/libinference_engine.so: undefined reference to `typeinfo for ngraph::op::v0::Sigmoid'
collect2: error: ld returned 1 exit status
make[2]: *** [wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/build.make:164: wrappers/openvino/face/rs-face-vino] Error 1
make[1]: *** [CMakeFiles/Makefile2:372: wrappers/openvino/face/CMakeFiles/rs-face-vino.dir/all] Error 2
make: *** [Makefile:130: all] Error 2

Thanks very much for your patience and the additional information, which will be included in discussion with Intel about your problems in this case.
Given that the RealSense OpenVINO wrapper makes use of pre-trained models from the Open Model Zoo and the link below states that the Zoo needs to be downloaded separately on Raspberry Pi, it may be worth trying that Zoo download.
https://www.intel.co.uk/content/www/uk/en/support/articles/000055510/boards-and-kits/neural-compute-sticks.html

@MartyG-RealSense
Yeah, I am quite aware of Open Model Zoo and have been running through the demos on the Raspberry Pi and Ubuntu 20.04 x86_64.  However, again, I have been using the 2021.2 Release to keep inline with the 2021.2 Release of OpenVINO I have been using.  The link you reference is for 2020.3 release.
The non UK one is:
https://www.intel.com/content/www/us/en/support/articles/000055510/boards-and-kits/neural-compute-sticks.html
These are the links I have been following for OpenVINO and OMZ:
https://docs.openvinotoolkit.org/latest/openvino_docs_install_guides_installing_openvino_raspbian.html
https://docs.openvinotoolkit.org/latest/omz_demos_README.html
I have previously posted an issue with regards to ngraph and bulding the OMZ Python Demos on the Raspberry Pi 4:
https://github.com/openvinotoolkit/open_model_zoo/issues/2135
I'll have to look through the config and see if I can remove references to YOLO and then see if it builds.
Also, I did not see any reference for placement of OMZ for RealSense on a Linux build, so I'll to look for that as well if it is in fact a dependency.

Thanks very much @jomoengineer for the detailed update - I look forward to your next report after the tests described above.  Good luck!

Hi @jomoengineer  Do you have an update about your case that you can provide, please?   Thanks!

@MartyG-RealSense Are you with Intel?  I was hoping that someone from Intel would chime in on this.
I complete my review of the product but would like to find a solution for this issue.  My preference would be to use the 2021.2 version rather than the older 2020 Release since there are differences in the demo naming structure and use.

I am a RealSense support engineer with Intel, yes.
At the present time, support is offered up to OpenVINO 2020.  I do not have information regarding support for newer OpenVINO versions at a future date.

Okay.  You can close this. I have not interest in pursuing this further.  I'll just mark it us unsupported.


Page starts with a quote

Hi, I know that the first sentence on the page is how MS perceives the philosophy of DevOps, but I would prefer to reference quotes back to the originator!
http://donovanbrown.com/post/what-is-devops
I could imagine, that Donavan Brown is ok with it, but I also the perception of people is important. And (me coming from a university) I would prefer an approach to reference it back to the source!

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

ID: 0cf98c4b-9185-00d5-a980-e2adb3e291fd
Version Independent ID: f6005931-c9b5-48d9-e6c3-f835bb9b01b1
Content: What is DevOps? - Azure DevOps
Content Source: docs/what-is-devops.md
Product: devops
Technology: devops-learn
GitHub Login: @SamGu
Microsoft Alias: samgu

Pull request for the suggested change: https://github.com/MicrosoftDocs/devops-learn-pr/pull/8

@SebastianSchuetze Thank you for the feedback, reviewing the PR now.

Started a new PR due to validation errors and submitted for further review.


Feature request: Renaming layers to string after last "/"

Layers get messy when you insert a symbol that is part of a well organized Library...
So for example it would be great to have a command to rename all layers to the string after last "/"
Example:
"UI / Buttons / Primary Button - Active"
becomes: -> "Primary Button - Active"
This would really help make files more usable!
Thank you!

Any thoughts on this @rodi01 ? In general, this is about removing part of the original name. But I understand you probably don't want to overcomplicate the plugin.
The above use case mentioned by @fbmore is actually the most common I run into as well - wanting to remove first part or all parts of "path" in symbol name, i.e. turning the above into "Buttons / Primary Button - Active" or "Primary Button - Active"
Maybe some advanced mode with the ability to add regular expression? Thank you.

@rodi01 I am actually not using the find & replace feature. but maybe just for renaming. and maybe it should be a separate function. but to be honest, there are some other plugins doing the same, e.g. https://github.com/littlebusters/Regexp-Rename and https://github.com/leonardpauli/layer-renamer - so maybe no need to reinvent the wheel. or just copy the code from there or merge the projects - after all, this is all open source cooperation :-)


include "catch" in list of likelihood components

@brianlangseth-NOAA just pointed out that the "catch" likelihood component, which appears in the Report.sso table of likelihoods, is not listed in the manual: https://github.com/nmfs-stock-synthesis/doc/blob/85c95ecd6c035328933ae941d8b30c865e8c7271/11likelihoods.tex#L16-L39.
Perhaps that's due to separating equilibrium catch from catch at some point in the past?
Regardless, we could tune up that section in the future.

So you are suggesting adding a line somewhere, probably at the top of this table, that looks something like this?
$i$  &  fishery $f$  &  Catch  &  normal  \Tstrut\\

I'm not sure of the distribution of catch.

@iantaylor-NOAA said equilibrium catch is log normal and that catch would be log normal


Refactor LZMA and installers targets into individual project builds

Prototype code:
https://github.com/aspnet/Universe/blob/23de03147dc13dc9655882d77f5e21229f5ea168/src/PackageArchive/PackageArchive.targets
https://github.com/aspnet/Universe/tree/23de03147dc13dc9655882d77f5e21229f5ea168/src/Installers
Refactor the targets for building the LZMA, installers (debian, rpm), and tarballs into individual projects instead of repo.targets

This issue was moved to aspnet/AspNetCore#3653


Add alchemy data news

This pull request adds the Alchemy Data News service to the SDK.

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

Merging [#313][cc-pull] into [master][cc-base-branch] will decrease coverage by 1.28%

2 files (not in diff) in .../VisualRecognitionV3 were modified. more

Misses -131
Hits +131

1 files (not in diff) in Source were modified. more

Misses +12
Hits -12

@@             master       #313   diff @@
==========================================
  Files           103        124    +21
  Lines          8268       8552   +284
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6676       6795   +119
- Misses         1592       1757   +165
  Partials          0          0

Powered by Codecov. Last updated by [d5add9f...2bea083][cc-compare]
[cc-base-branch]: https://codecov.io/gh/watson-developer-cloud/ios-sdk/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/watson-developer-cloud/ios-sdk/compare/d5add9f6905abf0e9e8f3da3b552a9d5fbc86c9a...2bea0830e29457b90db6ca454532491b69c4959e
[cc-pull]: https://codecov.io/gh/watson-developer-cloud/ios-sdk/pull/313?src=pr

This is ready to be merged. Just need to resolve the merge conflict.


Fix an issue with monitrc

After setting up a brand new box with tape, and trying to do monit restart all, I was running into this:
/etc/monitrc:290: Include failed -- Success '/etc/monit/conf-enabled/*'

To fix it, I simply commented out line 290 in /etc/monitrc (include /etc/monit/conf-enabled/*).

Brain dump:
I just paired with Brandon on this, and we realized it was a kind of different issue. Ultimately, we're pretty sure the issue was using the custom nginx role needed for letsencrypt. In order to resolve it, we decide it'd be best if that letsencrypt stuff lives inside tape. There's some more work to be done for that. Specifically, we'd like to remove all manual steps required (ie: right now you have to ssh into the server, and go through the blue screen flow). Trellis does something like that here

have you guys looked to see if a role already exists out there?  would this be helpful?  idk, i'll keep looking, too.
https://galaxy.ansible.com/jaywink/letsencrypt/

Just as an update here, I'm working on incorporating the role @dacur linked (jawink/letsencrypt), to do all the letsencrypt configuration. I'm having a little trouble getting it working, and I've opened an issue here. Once that's fixed, I should be really close to done with this.
As a side note, there is no good way to test the letsencrypt stuff. The way letsencrypt works (I think) is you configure your domain to point to your host, then letsencrypt talks to that domain, to make sure your box is actually that domain. Since we can't really set up a domain in docker/travis, it's not going to be possible to automate testing it. So instead, I'm manually deploying to a box, with this new role in place.

I'm kill this PR and pushing up a clean fix on a new branch.


Change ASM compiler for x264 and ffmpeg from yasm to nasm

建议其他环境，用docker编译以及跑SRS：https://github.com/ossrs/srs-docker/tree/centos#usage
Recommend to use docker to build and run SRS: https://github.com/ossrs/srs-docker/tree/centos#usage


Creating draft for new release v4.7

Create a new draft of v4.7.
Mention all the added functionalities , updated functionalities and bugs fixed.

Published draft for v4.7


Placeholder text is offset

In the body, the placeholder text "type something" is below the cursor and does not disappear until some text is typed.

I am using the github version and getting this issue.

@jason-hobbs this should be fixed in the current version 2.4.1.


DontCare Influences Type Inference

Consider the following:
class DontCareTest extends Module {
  class Bun extends Bundle {
    val foo = UInt(8.W)
  }
  val io = IO(new Bundle {
    val uint = Output(UInt())
    val bun = Output(Valid(new Bun))
  })
  val uint = WireInit(UInt(8.W), DontCare)
  io.uint := uint

  val bun = WireInit(Valid(new Bun), DontCare)
  bun.valid := false.B
  io.bun := bun
}

This gives the following error:
[info] Compiling 1 Scala source to /Users/jack.koenig3/ucb-bar/chisel3-raw/target/scala-2.12/classes...
[error] /Users/jack.koenig3/ucb-bar/chisel3-raw/src/main/scala/DontCare.scala:16: value valid is not a member of chisel3.core.Data
[error]   bun.valid := false.B
[error]       ^
[error] one error found

The definition of WireInit.apply being:
def apply[T <: Data](t: T, init: T)(implicit sourceInfo: SourceInfo, compileOptions: CompileOptions): T

Now changing the type of init to Data should fix this particular instance but this problem will pop up all over the place, including in libraries. Is there a way to tell the type system to ignore DontCare for type inference? @ducky64 @sdtwigg

What other instances of this problem are there? := is already untyped, and changing init to type Data makes the WireInit equivalent to a Wire declaration followed by an untyped := connect.
I'd imagine it doesn't make any sense to initialize a register with DontCare, since Chisel doesn't complain about uninitialized Regs?

In library code if you accept both a t and an init it will do the same type inference and you'll get the same issue. That being said I don't think there's a fundamental fix other than "don't write bad library code".


Extract Helper Methods for Combining Middlewares

Current coverage is 100% (diff: 100%)

Merging #15 into master will not change coverage

@@           master   #15   diff @@
===================================
  Files           3     3
  Lines          34    37     +3
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           34    37     +3
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update a3c7b3e...d49e50f


Any dep which installs node will override the node version

https://github.com/sindresorhus/run-node/blob/master/run-node#L4
Due to the combination of this line and the hoisting done on installs, any dep in your tree which installs node will result in all calls via run-node to use it, even if the top level project did not intend this behavior.  The only workaround I can see without changes to this lib is to also install node of your own version at the top level of each project.
I think that parsing json in bash to determine if it is a top level dep is a hard problem best avoided, but IMO this bug makes this module actively harmful in the contexts which I have seen it used.  One idea I had, which would resolve it in the use case I encountered this, is to use npm_node_execpath if it exists as the first check.  This would at least ensure that if it was run in the context of a package.json script with npm it would use the same node version found in the parent process.  I am happy to open this PR if this seems like a good resolution to the problem above.

One idea I had, which would resolve it in the use case I encountered this, is to use npm_node_execpath if it exists as the first check. This would at least ensure that if it was run in the context of a package.json script with npm it would use the same node version found in the parent process. I am happy to open this PR if this seems like a good resolution to the problem above.

👍🏻 That makes sense to do regardless of this issue.


Doesn't play nice with black navigation bar(s)

Hi!
First of all thanks for this great library!
I am using cocoapods 1.4.2 and found out that the transition doesn't work right if the source controller is embedded in an navigation controller with barStyle = .black
Most of the times the transition doesn't happen and when it does, it does with significant delay. If the barStyle is default there is no problem.

Thanks, I am tested now

@magnett I am tested on my iPhone X, and iPhone 6+. All ok... Or maybe I am not understand you. I am record screen, please, see it:

I though it would be better to record a video instead of a gif in order to watch the console output and hear an audible sound when i tap and when the transition occurs. Also, let me add that exactly the same issue happens to my iPad (2017).
YouTube video

@magnett sorry, I don’t see any problem on video

Really? Don't you see that there are delays up to 3 seconds from the moment the button is tapped since the transition begins?

Yes, now I now see the problem. Very strange behaves - if you press twice, the controller will instantly show up. I'm afraid this is not a library issue.

For example if you try
self.present(productContainerVC, animated: true, completion: complection)
You can see delay also. The library is not related to your problem, you need to look for another problem.

You were right. Any kind of modal transition under black navigation bar was having that delay. Just to let you (and anyone might having that issue) know, the solution was:
DispatchQueue.main.asyncAfter(deadline: .now()) {             self.presentAsStork(someViewController)         }


[FLINK-28240][network] Fix the bug that NetworkBufferPool#getRequestedSegmentsUsage may throw ArithmeticException.

What is the purpose of the change
There's a bug that NetworkBufferPool#getRequestedSegmentsUsage may throw ArithmeticException if the pool is initialized without any buffers. It may cause other metrics failing to report.
Brief change log

Return 0 as the requested segments usage if the buffer pool is empty.

Verifying this change
This change added tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

Merged


Can I set the body type to json ?

I just want to know if I can set the body text to json ? Thank you


add CI workflows running tests

Closes https://github.com/rapidsai/graph_dl/issues/638
Adds CI workflows running the unit tests.
Also adds some .pre-commit-config.yaml changes, follow-up to #14 suggested by @eriknw .
Notes for Reviewers
I copied these new scripts in ci/ from the latest state of branch-24.12 in https://github.com/rapidsai/cugraph, then tried to adapt them to be nx-cugraph specific.
What about notebook tests?
Proposing we add notebook tests in a follow-up PR, to keep things smaller and easier to review.

/merge


Error in documentation.  HTTP Verb should be PUT not POST

On Step 11 of the "Upgrading from the older customappsso job to the SCIM job" section, it states that you must make a request to Graph API using the POST verb, as follows:
POST https://graph.microsoft.com/beta/servicePrincipals/[object-id]/synchronization/jobs/[new-job-id]/schema { <your-schema-json-here> }
This is in fact an update, and should be using the PUT Verb.  This is verified from the Graph API Beta documentation:
(https://learn.microsoft.com/en-us/graph/api/synchronization-synchronizationschema-update?view=graph-rest-beta&tabs=http#http-request)

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

ID: 09b8c472-3a4e-dd2c-11df-f05b05e5a642
Version Independent ID: 834bcfe7-2e5c-97ed-eeb8-d5eed616188b
Content: Known issues with System for Cross-Domain Identity Management (SCIM) 2.0 protocol compliance - Azure AD - Microsoft Entra
Content Source: articles/active-directory/app-provisioning/application-provisioning-config-problem-scim-compatibility.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @kenwith
Microsoft Alias: kenwith

@ot-mslater
Thanks for your feedback! We will investigate and update as appropriate.

@ot-mslater
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


left-sidebar: Add max-height to private message container.

Add max-height to PM section so that it doesn't cover the screen when a user has too many conversations. Fixes #5384.
GIFs or Screenshots:
Before:

After:

@timabbott any chance you Close'ed instead of Merge'ed?

@gioragutt we don't use GitHub's "merge" button, because it doesn't allow editing commits very flexibly (in this case, I edited the commit message).  So yes, this was in fact merged; you can see 321b70559198fc0e0925431a89a8f96dc40dfcfd in master.

👍


After backgrounding a process with Ctrl+z, powershell is unable to accept input

Steps to reproduce
Open emacs, vim, or any other process that supports being backgrounded via Ctrl+z.
Press Ctrl+z
Expected behavior
You are returned to PowerShell, the prompt is displayed, and new input is processed.
Actual behavior
You are returned to PowerShell, the prompt is not displayed, and no new input is able to be processed.
Environment data

Name                           Value
----                           -----
PSVersion                      7.0.3
PSEdition                      Core
GitCommitId                    7.0.3
OS                             Darwin 19.6.0 Darwin Kernel Version 19.6.0: Sun…
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

It looks like there are actually two possible behaviors here, depending on the terminal being used.
In the built in Terminal app, pressing Ctrl+z causes the screen to flash to PowerShell, and immediately resume the app.
In iTerm, the behavior is as described above.

Dup #7854.

Thanks. I'm not sure how I missed that one in my search


Improvements

Revert "Revert "Maintenance (#23)""
Upgrade some easier deps

📝 Changed routes:

/news
/search

(view full size)
(view full size)

Commit 8484c12ca7ded16ca9e5a567e665f56fe8b5453d (https://refuge-worldwide-ayj5zyb16-refugeworldwide.vercel.app).


Update theme

Update theme to remove the dark border we have on buttons and update sync button style to fix some inconsistencies when disabled & hovered between "sync" and "sync options dropdown" (see below)

Before

After

The changes to input will remain in EE since adding them to the theme would have meant having to overwrite the style on both SignIn page and in filters

Manuela wanted the border separating sync and the dropdown removed so I did it here https://github.com/weaveworks/weave-gitops/pull/3724
Can we keep that or is that what broke things in light mode?

Yup it looked weird (it basically looks like in the "before" screenshot I attached with a different combination of colors 😅 ). Because it's not a button but a collection of buttons when you hover on top of one the other's state doesn't change. Without the border in between it looks a bit weird I think.
We actually have another similar button on EE, with the border visible