Ingress w/ Tye websocket (signalr) support

having trouble switching over to using ingress and tye with websockets, specifically using signalr. is there a gentle work around for me?

Request URL: ws://localhost:10000/api/communications/hub?id=XlXIkUhWY384C56RW_gOwA&access_token=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJuYW1laWQiOiJjNTBkMDg0Ny00NjNjLTQxN2MtYTdkNy1jNWFmMjY0NmQ0MWYiLCJzdWIiOiJib2JAYm9iLmNvbSIsImp0aSI6IjczN2YxNDE5LTk4ZDQtNGRiYi1hM2MxLWRjNDMyNTM0MmQxNyIsInVuaXF1ZV9uYW1lIjoiYm9iQGJvYi5jb20iLCJuYmYiOjE2MTI4NDAzODksImV4cCI6MTYxMjg0Mzk4OSwiaWF0IjoxNjEyODQwMzg5LCJpc3MiOiJsb2NhbGhvc3QiLCJhdWQiOiJib2IifQ.iV9S81ojUBUL3WPWj4vpMcUR9Egs3ix7WV0JywAKc8E
Request Method: GET
Status Code: 101 Switching Protocols
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: http://localhost:51771
Connection: Upgrade
Date: Tue, 09 Feb 2021 03:19:54 GMT
Sec-WebSocket-Accept: 0A6qxW6+1LSPOQg14HS7pzYl24o=
Server: Kestrel
Upgrade: websocket
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9
Cache-Control: no-cache
Connection: Upgrade
Host: localhost:10000
Origin: http://localhost:51771
Pragma: no-cache
Sec-WebSocket-Extensions: permessage-deflate; client_max_window_bits
Sec-WebSocket-Key: pz/nzhtjKWl17XG34ORwyA==
Sec-WebSocket-Version: 13
Upgrade: websocket
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.146 Safari/537.36
id: XlXIkUhWY384C56RW_gOwA
access_token: eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJuYW1laWQiOiJjNTBkMDg0Ny00NjNjLTQxN2MtYTdkNy1jNWFmMjY0NmQ0MWYiLCJzdWIiOiJib2JAYm9iLmNvbSIsImp0aSI6IjczN2YxNDE5LTk4ZDQtNGRiYi1hM2MxLWRjNDMyNTM0MmQxNyIsInVuaXF1ZV9uYW1lIjoiYm9iQGJvYi5jb20iLCJuYmYiOjE2MTI4NDAzODksImV4cCI6MTYxMjg0Mzk4OSwiaWF0IjoxNjEyODQwMzg5LCJpc3MiOiJsb2NhbGhvc3QiLCJhdWQiOiJib2IifQ.iV9S81ojUBUL3WPWj4vpMcUR9Egs3ix7WV0JywAKc8E

Tye settings:

ingress:

name: ingress
bindings:

port: 10000
rules:
path: /api/identity
service: identity
path: /api/members
service: members
path: /api/accounts
service: accounts
path: /api/communications
service: communications

@davidfowl looks like there may be a bug with websockets with the local ingress.

Yea, it doesnt handle it at all. We need to YARPIfy this thing.

More info
[Information]: Request finished in 12.6356ms 200 application/json; charset=utf-8 [Information]: Request starting HTTP/1.1 GET http://localhost:8085/notificator/jobhub?id=v6tE6q_6s-lig8sdqzgb7g [Information]: Executing endpoint 'notificator' [Information]: Executed endpoint 'notificator' [Error]: Connection id "0HM78HHCQB3O4", Request id "0HM78HHCQB3O4:00000001": An unhandled exception was thrown by the application. System.OperationCanceledException: The operation was canceled. at System.Threading.CancellationToken.ThrowOperationCanceledException() at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken) at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.GetResult(Int16 token) at System.IO.Stream.CopyToAsyncInternal(Stream destination, Int32 bufferSize, CancellationToken cancellationToken) at System.Net.Http.HttpConnection.RawConnectionStream.CompleteCopyToAsync(Task copyTask, HttpConnection connection, CancellationToken cancellationToken) at Microsoft.AspNetCore.Proxy.ProxyAdvancedExtensions.CopyProxyHttpResponse(HttpContext context, HttpResponseMessage responseMessage) in /_/src/Microsoft.Tye.Hosting/Infrastructure/ProxyExtensions.cs:line 223 at Microsoft.AspNetCore.Proxy.ProxyAdvancedExtensions.CopyProxyHttpResponse(HttpContext context, HttpResponseMessage responseMessage) in /_/src/Microsoft.Tye.Hosting/Infrastructure/ProxyExtensions.cs:line 223 at Microsoft.AspNetCore.Proxy.ProxyAdvancedExtensions.ProxyRequest(HttpContext context, HttpMessageInvoker invoker, Uri destinationUri) in /_/src/Microsoft.Tye.Hosting/Infrastructure/ProxyExtensions.cs:line 43 at Microsoft.Tye.Hosting.HttpProxyService.<>c__DisplayClass4_1.<<StartAsync>b__3>d.MoveNext() in /_/src/Microsoft.Tye.Hosting/HttpProxyService.cs:line 160 --- End of stack trace from previous location where exception was thrown --- at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger) at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application) [Information]: Request finished in 17018.8777ms 101

@davidfowl Has there been a fix for this?

Hi, I also ran into this problem with Ingress and SignalR websockets. I worked around it by using an nginx service as ingress, with the associated nginx configuration for websockets. I suppose this would also bypass some of the deployment helpers, but it got me progressing with tye.

Hi, I also ran into this problem with Ingress and SignalR websockets. I worked around it by using an nginx service as ingress, with the associated nginx configuration for websockets. I suppose this would also bypass some of the deployment helpers, but it got me progressing with tye.

I'm lazy. Can you post your tye config? :)

I did something inspired by https://github.com/dotnet/tye/tree/main/samples/nginx-ingress.
name: nginx-proxy

services:
- name: backend
  project: "backend/backend.csproj"
  bindings:
  - protocol: http

- name: nginx
  image: nginx
  bindings:
    - protocol: http
      port: 8081
  volumes:
    - source: nginx.conf
      target: /etc/nginx/templates/default.conf.template

And an nginx.conf (using the environment expansion detailed on https://hub.docker.com/_/nginx/):
upstream backend {
    # Using environment variables generated by Tye
    server ${BACKEND_SERVICE_HOST}:${BACKEND_SERVICE_PORT};
}

server {
    listen 80;

    location / {
        proxy_http_version 1.1;
        proxy_set_header   Upgrade $http_upgrade;
        proxy_set_header Connection 'upgrade';
        proxy_set_header   Connection keep-alive;
        proxy_set_header   Host $host;
        proxy_cache_bypass $http_upgrade;
        proxy_set_header   X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header   X-Forwarded-Proto $scheme;

        proxy_pass http://backend/;
    }
}

The environment variables don't get expanded for me.

Hi! Was there a fix/update for this issue?
I'm having a similar problem on an application that does not include docker so I can't use the image for nginx like on the proposed solution from @james-allan-lloyd
In particular I have an .net core hosted blazor app. It's running with an ingress and everything work but the blazor signalr hub, with the following error on the browser console:


tikv-ctl: add command transfer-leader.

The command can be run on a crash tikv node to tell pd do a
transfer-leader on a specified region which causes the crash.

Seem that pd-ctl has already supported this, why do we need it in TiKV?

Because this command can only run on crashed TiKV nodes, which maybe doesn't have pd-ctl binary.

Seem the assumption is not suitable. Our deploy tool will guarantee we can use them together.
The only reason I can see here is that we don't use two tools at the same time.
What is your opinion? /cc @BusyJay

I think it's OK to use pd-ctl here. If the follower is far behind leader's progress, pd-ctl is still needed to be used any way. But for the long term, maybe it's better to provide only one binary to operate the whole system.

Now tikv-ctl call pd-ctl directly. PTAL again @siddontang @BusyJay .

It's a bad style. I will examine how to collect all recovery points to one unified style.


1.2.0rc1 release

Codecov Report

Merging #456 into master will decrease coverage by 0.1%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master     #456      +/-   ##
==========================================
- Coverage   86.04%   85.94%   -0.11%
==========================================
  Files          56       57       +1
  Lines        4429     4474      +45
  Branches      481      488       +7
==========================================
+ Hits         3811     3845      +34
- Misses        465      474       +9
- Partials      153      155       +2

Impacted Files
Coverage Δ

...ge-common/azure/storage/common/_deserialization.py
81.75% <ø> (ø)
:arrow_up:

...torage-blob/azure/storage/blob/_upload_chunking.py
71.42% <0%> (ø)
:arrow_up:

...zure-storage-common/azure/storage/common/models.py
91.32% <100%> (+0.05%)
:arrow_up:

...re-storage-common/azure/storage/common/__init__.py
100% <100%> (ø)
:arrow_up:

...-storage-common/azure/storage/common/_constants.py
100% <100%> (ø)
:arrow_up:

...zure-storage-blob/azure/storage/blob/_constants.py
100% <100%> (ø)
:arrow_up:

...orage-blob/azure/storage/blob/appendblobservice.py
94.38% <100%> (ø)
:arrow_up:

...storage-blob/azure/storage/blob/baseblobservice.py
94.71% <100%> (+0.02%)
:arrow_up:

...storage-blob/azure/storage/blob/pageblobservice.py
91.12% <100%> (ø)
:arrow_up:

...torage-blob/azure/storage/blob/blockblobservice.py
91.01% <100%> (ø)
:arrow_up:

... and 12 more

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 ff76da6...2fd1bc1. Read the comment docs.


Values for Selection Custom Fields via API

Issue type
[X] Feature request
[ ] Bug report
[ ] Documentation

Environment

Python version:  3.6
NetBox version:  2.2.8

Description
As I already told on the mailing list, I would like to fill in the values of selection custom fields via the API.
I expected to provide the actual value of the field, i.e. the text I see in the dropdown of the GUI. But the API expects to get the ID of the option, rather than the value, and there seems to be no way of looking up the available (value,id)-pairs for a given selection custom fields. Also I would like to avoid hardcoding any (value,id)-pairs in my application, as it's not consistent across our different Netbox instances (local, staging, production) and it would be very fragile.
I see two solutions:

When I set the value of such a selection custom field, Netbox accepts the String value and converts it to the respective ID internally.
There is a new endpoint to fetch the option pairs of selection custom fields, similar to the other _options endpoints.

IMO both options could even live in parallel to each other: Whenever an integer is presented as value for a selection custom field, we can assume it's the ID of the value. Whenever it's a string, we can assume it's the text value and first has to be converted to the corresponding integer id.
Allowing to submit text values via the API instead of the actual IDs might imply that we don't allow the same text value on one given selection custom field twice.

I ran into this same issue when trying to update a selection-based custom field I'm using through APIs (and pynetbox). I'm really trying to avoid custom fields whenever I can but I still need a few. In general I'd rather have a custom field with fixed selection options rather than a string to try to limit data diversity.. but without a programmatic way of changing selection values, it's getting a bit challenging..

We need to decide how to expose the custom field choices in the API. Currently, each app has a _choices endpoint which returns the choices for all models in the form "model:field": [choices]:
{
    "device:face": [
        {
            "label": "Front",
            "value": 0
        },
        {
            "label": "Rear",
            "value": 1
        }
    ],
    "device:status": [
        {
            "label": "Active",
            "value": 1
        },
        ...
    ]
}

We can't do model:custom_field_name because it's possible that custom_field_name will collide with the name of a built-in field. We need a way to distinguish custom fields.
One option would be to prepend cf_ to the field name, such that custom field foo on the Device model would become device:cf_foo, but that may not be the cleanest approach. Open to suggestions.

I am also very much interested in this feature.

I feel this should be its own API endpoint. Perhaps /extras/custom-fields/_choices/. This way it can act just like the built-in app choices endpoints, but there can be no name collision or name manipulation. It is also very apparent that these choices apply only to custom fields.

I'd be fine with any way to lookup the '_choices' label using the value. Doesn't have to be pretty, just has to work. :)

@Grokzen I think that looks good if you want to submit a PR. I like @lampwins' idea of putting the endpoint at /extras/custom-fields/_choices/, but that wouldn't show up in the browsable API, so your way is probably preferable to make it more obvious.

Awesome, i will put it up during the upcomming workweek

Closed in #2941. Thanks @Grokzen!

@Grokzen & @jeremystretch
Great work! This is fan-friggin-tastic, and a much needed endpoint function.


Making pricing definable via environment variables

The hardcoding is unwieldy

Unfortunately, the /new route is a client component and I think is making accessing the env vars more difficult. Probably need to push the interactive stuff into a client component that is imported into the /new page (which should be a server component).


Upgrades Spring to 5.1.2 and Spring Boot to 2.1.0

I had to delete one test in camel-spring-boot: CamelAutoConfigurationWithContextRunnerTest, I could not tell why it was failing or what it tested, so I figured: best not waste time fixing it.
There is a build job for this:
https://builds.apache.org/view/C/view/Apache Camel/job/Camel/job/spring-upgrades/
We should probably wait and see if something is broken with this. I did run tests for camel-spring and camel-spring-boot locally, but let's just make sure.

How are we going with this one?

How are we going with this one?

We can merge this, the camel-spring-cloud-consul tests will fail. We can work on fixing those in a separate issue/PR if that's OK with everyone?

+1 for me.


ci: fixing smoketest runners

Description
Fixes the smoke test workflow to work on public runners, removed a bunch of unused actions, and disabled the Discord token based release notification. We can do it much more safely with webhooks for a specific channel vs using a Discord bot token.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions

Checklist:

[ ] I have added unit tests that prove my fix feature works

This looks good to me, I'm just wondering about one thing. make zetanode step seems to take one additional minute to complete compared to before: https://github.com/zeta-chain/node/actions/runs/7410644201/job/20163410705?pr=1436 https://github.com/zeta-chain/node/actions/runs/7414576117/job/20175927185?pr=1525 Any known reason for this?

Likely do to a change in runners. Our private runners make have had higher performance or a faster network connection but I not sure about that.
We can do testing in another PR and compare the speed across different sized runners.


How additional tools can be set on Home page bot ?

Hi ,
During the Agent creating I got the question about additional tools agent should use :

How to answer here if I want to add the Summary or Key tool , for instance

Ask it " suggest some additional tools for the RAG".
This will give you a list of some tools.
Then you can type "set the include_summarization parameter to True."

heads up @snassimr i meant to add web tools in the initial release but it wasn't quite ready yet. but that was something on the roadmap

Hi ,
Looks like I wanted too much . I thought it's possible to add , i.e. keyword_query_engine (SimpleKeywordTableIndex) and agent_util.py code (create_agent function) to be updated accordingly. It will allow experimentation with tools.
Now when I ask to add tools - I prompted to review/edit the config that just created - makes sense to me.

@jerryjliu hinted that the example for a tool is web search . I think this is something else.
@jerryjliu Would you to take this issue as feature request for web search or I have just to close the issue ?


Collapsable layout for DE and EN FAQ

To Do:
English translation missing
Add bootstrap container

@mairima finally we finished the work you started :) It is now online!


Small documentation suggestions

README.md talks about "subtitle-language-default", but I believe that the actual name is "subtitle-default-language" (default first, then language), similar to "audio-default-language"
Also a quote from README.md:
subtitle-language = same as audio-language but for subtitles. Set to nil to disable copying of subtitles.
I think we should clarify that if left blank, only the existing ones are copied from the source, but of course no additional subtitles can be downloaded, because the script doesn't know in what languages, which in turn leads to the warning "Subtitle language array is empty."

Done


"flv" under "hls" settings?

https://github.com/DIYgod/DPlayer/blob/master/src/DPlayer.js#L79
Seems there should be enablehls = false rather than enableflv = false.
By @RexSkz

Thanks!!!


[RFC] TypeScript migration

Summary
TypeScript is a superset of Javascript – all Javascript is valid Typescript. However, once a project has typescript enabled, you can annotate your code to provide type hints to your users and other developers.
Just as shared styles between Carbon frameworks improves user experience, so to would shared behavior. Typesafety allows us to ship components that perform and behave as expected.
Related issues: #1296, #727, #1103, #580
Motivation
End users, product developers, as well as Carbon developers would all benefit from the addition of type safety to Carbon products

End Users – by shipping more resilient, bug free components end users will encounter fewer unexpected edge cases
Product developers – developers would be guided towards proper API usage and would be alerted to new or changed apis.

Carbon developers and Typescript developers –

Fewer Uncaught TypeError(Cannot read x of undefined or spelling errors means more time on real issues.
Fewer undefined refs and instance properties
Reduce reliance on null and undefined guards means more resiliant APIs
– Alerted to misuse of props, state, third-party library, and dom errors while writing your code.

Design
There are currently to main ways to compile typescript into javascript(the typescript compiler, and through babel. Since our project is already using Babel, switching to typescript would be unnecessary.
Namespaces and const enums are not supported by the babel compilation, however, this can be mitigated with compiler configuration.
As we look toward migrating codebases to a mono-repo, typesafety will offer a means to share types between projects and permit large scale refactors with confidence.
Drawbacks

Time spent onboarding carbon devs to typescript

I volunteer as tribute

Friction to carbon contributors new to the project

would empower them to make more meaningful changes
would reduce friction for folks onboarding to product teams

Alternatives

TypeScript via definitely-typed

Types hand written instead of generated
Issues keeping types in parity with new library features and changes

FlowType

call-site type checking is nice
Lots of projects switching to TS(Jest, Expo, Atlassian, Yarn.
Far fewer third party integrations 1/10th of the questions on SO

JSDoc

some bugs around object types
errors in jsdoc means no warnings/errors it just won't get annotated

Adoption strategy
While each of the Carbon javascript projects would benefit from the addition of types. I believe the most valuable library at this time would be carbon components react (CCR).
Since we already have some run-time type checking through prop-types, we already have a starting point. In addition, thanks to prop-types CCR developers are already at least slightly familiar with the constraints imposed by types.
Conversion path

The first pass at converting the project will introuce the .ts/.tsx file names and add babel, storybook, and jest configuration to handle these new file types.

Any objevtive errors uncovered as a result of the conversion will be resolved as part of the same PR.

Default props will be defined at the same level of specificity as the prop-types they are replacing. No unnecessary types added.

The initial config will have strict-mode features disabled. Separate PR's will be made to turn on each of the strict-mode rules separately resolving issues as they come up.

Migration stories from others

hootsuite: Thoughts on migrating to TypeScript
clayallsop: Incrementally-migrating-javascript-to-typescript-565020e49c88 'Incrementally Migrating JavaScript to TypeScript
pleo: Migrating a Babel project to TypeScript
mstsreactconversionguide: TypeScript React Conversion Guide
entria: Incremental Migration to TypeScript on a Flowtype codebase

Thanks @vpicone for writing this up! For full TypeScript conversion I'd like careful observation on the learning curve of our developers/contributors. Though there are extra efforts to get .d.ts and .js in sync as you pointed out, I see .d.ts is a worthwhile alternative for the interim.

@asudoh the issues I see with that approach are:

Maintaining the types would require manual effort as type generation tools are spotty at best. That manual effort would be concentrated on those intimately familiar with typescript as the types would be written from scratch.
We wouldn't benefit from the types internally leaving many of the benefits I listed out.
This approach is much more difficult to implement progressively. With a typescript migration, we can ship types when they are ready, not necessarily all at once.

The DefinitelyTyped types I published for @carbon/elements are a perfect example. After my initial commit, they've fallen out of sync very quickly.

That said I believe Material-UI is using internal .d.ts files, so could be a good example of that the hand written types method.

Wrt learning curve - Another problem I'd like careful look at is getting more people onboard to our development, to better serve our users for the various kind of problems in our backlog. Would like to see a good balance between solving TS problems vs. solving other problems.

With the rules being progressively more strict, solving TS problems would be solving JS problems as well as preventing future issues. For example, the modal trap just recently ended up causing even more problems than it solved. The undefined issue we shipped that broke the modal never would have been prevented with typescript. That issue remained  broken for quite a bit and cost significant dev time for review. That time could have been spent on the backlog.
While we might be introducing friction for contributors unfamiliar with typescript, I think that types would allow contributors to work more confidently in complex components and more quickly orient themselves in the code base. It also allows us to be more confident that their changes aren’t breaking.
In my opinion, the types already exist, they’re just in our mental modal of the project. Typescript allows us to formalize them for new and existing contributors.

thanks @vpicone for drafting this RFC! I'm excited to potentially introduce TS into our project and have been looking to for a bit now. As @asudoh mentioned, we need to carefully weigh the cost and benefits to justify the investment into TS. That being said, I am all for this initiative. prop-types is good but still feels kind of lacking, and in my experience the self-documenting nature of strong static typing should ease the onboarding process for devs unfamiliar with the project (which should also make adoption smoother especially as our project grows). I also think this alleviates a lot of the pressure on the contributors (especially the maintainers) to code defensively. of course this is all anecdotal evidence from me, and I'm interested in hearing the perspectives from others
on a side note I agree that TS would have helped prevent a lot of the headaches that introducing the modal focus trap caused, but I think part of that may have stemmed from adding dependencies without fully vetting them, which is why this RFC and discussion is so crucial!

Couple of initial questions that I have when we think of TypeScript integration:

What happens if a language feature we are using (or want to use) is supported in babel but not in TypeScript?
Would we have to both type our props and offer prop types for teams not using TS? How would we keep these in sync?

@joshblack if the feature changes the syntax significantly, we'd need to use a babel macro until the language feature is supported by typescript (level 3+). Otherwise, all the standard plugins would work. As an example, object rest/spread wasn't typescript supported for quite some time and folks would include the babel plugin.
TS definitely isn't a drop in replacement for prop-types, especially for libraries that are consumed by JS projects like ours, since it disappears at run time. Typescript will throw an error if the prop-types and Prop types don't map:

There's a babel plugin to set prop-types from typescript types. If we wanted to automate the process of generating them.


Unable to set client.captureConsole from karma.conf.js

I'm working on a project that defines its karma options in a karma.conf.js file. When client.captureConsole is defined in the karma config, it gets ignored when I run grunt karma. However, if the option is defined in the Gruntfile instead, it works. I know that client.captureConsole is valid in the config file because I can run karma directly with that config property set, and it works as expected.
It looks like grunt-karma is setting defaults for the client options (here) if they are not set in the Gruntfile options object. My suspicion is that these defaults eventually override the settings from karma.conf.js, causing the behavior I'm seeing.
Is there a reason why the client options are directly being set by grunt-karma? If these are really karma defaults, shouldn't karma take care of setting them?

Looks like this is related to issues #165 and #128. Happy to contribute when I have some time if I can get some guidance on the issue.

Yes this is a bug, which is two fold, one part is in grunt-karma, which shouldn't overwrite anything or change the defaults unless it actually needs to, the other is that there seems to be a bug in passing the client options via the api atm, which is probably happening somewhere in lib/config.js.
I'm happy to help you if you want to try and fix this, just let me know if you have any questions.

I have similar problem with client configs overridden by grunt. It works if I change locally config.js set method from:
  this.set = function (newConfig) {
    Object.keys(newConfig).forEach(function (key) {
      config[key] = newConfig[key]
    })
  }

to:
this.set = function (newConfig) {
    _.merge(config, newConfig);
  }

Is such fix any good?


analytics: Add Google Analytics

Currently we don't have any tracking at all on our website, and this
seemed like the fastest way to get started with it. Default Google
analytics script is used on each page now.

I know Nick was looking at a different analytics provider that didn't track individual users and only provided aggregates for both flowforge.com and nodered.org. Let me see if I can find it before we pull the trigger on this.

Found it, https://plausible.io/ is being loaded in the current flowforge.com
https://github.com/flowforge/website/blob/7166c431dbaef3ae8da6602ad473eaf1361ed489/src/_includes/layouts/base.njk#L24

But I don't see the login details for it on 1Password, so we might have to ask Nick for them.

But I don't see the login details for it on 1Password, so we might have to ask Nick for them.

Will do. Thanks for checking!

I moved to Plausible on a free trial to see how it compared to GA. This includes the Node-RED site as well.
The number of page views we get on the NR site means it costs $50/month. But I think that is worth it.
I have moved the login details to the shared 1password vault.


Windows support

This is the big one...
The bulk of the work would probably be writing Haskell bindings to (a subset of) something like ViGEm.
Once such a binding exists, integration would probably be no more complicated than the existing Linux code.

Given that ViGEm uses C++, hoppy might be the way to create bindings.

Or inline-c-cpp.

Some initial work (for mouse/keyboard emulation, rather than gamepad) on https://github.com/georgefst/monpad/tree/windows.

Some discussion about the difficulties of linking C++ libraries with GHC: https://gitlab.haskell.org/ghc/ghc/-/issues/20010.


on Data records tab of Data page, allow user to create a data set out of the selected data records

On the screenshot, three data records are selected. Add a possibility to turn them into a data set.

Related to this PR https://github.com/nuvla/ui/pull/732


Recipe: Statistics

Crates
std

Section
16.1 Mathematics

Given a collection of integers, calculate the mean, median, mode, standard deviation, maybe Z-score of the highest outlier.

👍


SUBMARINE-890. Extract all resource creation spec

What is this PR for?
Extract all resource creation spec into functions so the code is more readable.
What type of PR is it?
[Refactoring]
Todos

[x] - Make controller be a standalone package
[x] - Extract resource creation spec into functions

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-890
How should this be tested?
Screenshots (if appropriate)
Questions:

Do the license files need updating? No
Are there breaking changes for older versions? No
Does this need new documentation? No

@kevin85421 Can you help me review this PR? Thank you.


Datasource & Publish Filtering / Tests and Issues

While fixing a bug regarding the between-filter-mode of number field in symphony's publish filtering interface I stumbled upon some other issues and started to do a lot of testing and comparing to assure none of them were caused by my changes to the publish filtering interface.
While figuring out the best way to document my test results I stumbled upon this page in the wiki and decided to set up something similar for each field/extension I tested.
To add additional value I not only tried to make the test cases cover most common use cases (I could think of) but also started to run each test not only for Publish Filtering but also for Datasource Filtering:
The results verified that my changes didn't cause any of the issues, but revealed that the filtering system (especially publish filtering) has quite some shortcomings.
So my questions to @nitriques and the community are the following:

Would it help to post a new issue for each single bug I found or should I better let it be? I don't expect most of the bugs to get fixed in the near future and I don't want to flood the issue tracker with publish filtering bugs noone really cares about (not even me).
Would my Symphony Filtering Tests-Repo be a useful addition to symphony's documentation ecosystem? If so I would gladly pass it over to the "symphonycms"-account so that it might get extended, maintained and updated by the community and whoever might stumble upon problems regarding filtering might use this as a reference, starting point and helpful resource for his own tests and extend/update the repo with his results.

Would it help to post a new issue for each single bug I found or should I better let it be?

Yes, I do think so!

Would my Symphony Filtering Tests-Repo be a useful addition to symphony's documentation ecosystem?

Yes it would if there was a way to run the tests automatically... (I'll be working on that...)


getting LocalFileSystem: fail to open "xgboost.model"

Hi there
I was doing a project using R to building xgboost model
it's all goes well in my Windows local laptop in R.
after I put my code in linux server  and install basically all the packages I need,
I keep getting the error that  when I running this code
xgboost(data = dtrain,max.depth = 6, eta = 0.5, nthread = 6, nround = 50,objective = "reg:linear",verbose = 0)

Error in xgb.save(env$bst, sprintf(save_name, env$iteration)) :
[10:46:20] amalgamation/../dmlc-core/src/io/local_filesys.cc:154: Check failed: allow_null  LocalFileSystem: fail to open "xgboost.model"

it is quite rare which I could not find any info that is helpful.

Thanks for reporting!
save_period = 0 setting is currently set as default in xgboost(), which leads to an attempt to save a model at the end of a run. And when you have no write permission for a directory, you would get this error. The default setting has to be changed to save_period = NULL, to be the same as in xgb.train(). I'll do that later. Meanwhile, you can explicitly set it to NULL as a temporary workaround.

I am getting this exception when I call xgb.save with version 0.71.1. All working directories are owned by my user. I did not have this issue with version <IP_ADDRESS>. Any clues on how to get more useful information?

same here


cibuildwheel on Github Actions Windows runner uses GCC by default

Not sure which component to blame here, but this is not something that used to happen with scikit-build -
Build log using scikit-build
--------------------------------------------------------------------------------
  -- Trying 'Ninja (Visual Studio 17 2022 Win32 v143)' generator
  --------------------------------
  ---------------------------
  ----------------------
  -----------------
  ------------
  -------
  --
  Not searching for unused variables given on the command line.
  CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
    Compatibility with CMake < 3.5 will be removed from a future version of
    CMake.

    Update the VERSION argument <min> value or use a ...<max> suffix to tell
    CMake that the project does not need compatibility with older versions.

  -- The C compiler identification is MSVC 19.38.33133.0
...

Full log: https://github.com/planetmarshall/pillow-jpls/actions/runs/7324438660/job/19948011356
Build log using scikit-build-core
 loading initial cache file C:\Users\RUNNER~1\AppData\Local\Temp\tmpr7cco_vs\build\CMakeInit.txt
  Preset CMake variables:

    CMAKE_BUILD_TYPE="Release"
    CMAKE_COMPILE_WARNING_AS_ERROR="ON"
    CMAKE_PROJECT_TOP_LEVEL_INCLUDES="D:/a/pillow-jpls/pillow-jpls/cmake/conan_provider.cmake"

  -- The C compiler identification is GNU 12.2.0
  -- The CXX compiler identification is GNU 12.2.0

Full log: https://github.com/planetmarshall/pillow-jpls/actions/runs/7338592087/job/19981348413
I realise this may be more to do with how Github Runners are setup but as this is probably a common setup and is a change in behaviour from porting from scikit-build, it is probably worth noting.

I believe it's due to
https://github.com/planetmarshall/pillow-jpls/blob/7c98ed7c282c75faed03f7bc9c53fab5089a94f5/.github/workflows/build_deploy.yml#L10-L12
GHA doesn't activate MSVC inside the bash shell automatically, so you have to do it manually if you really want that shell (there are actions to do it for you). Scikit-build classic queries the registry to find MSVC, while scikit-build-core assumes you want to build with whatever is active if MSVC is not (which is GCC for the bash shell). This is required if you are trying to build Fortran code, for example.

Also, I'd highly recommend configuring cibuildwheel via pyproject.toml instead of bunch of CI-based environment variables, will be cleaner and easier to build locally too. :)


!voteythimbs

Add a command !voteythimbs so that the committee/others can have voting on non-urgent issues but without alerting them with pings. I would very much understand if this was closed to no avail, but it would make things easier (albeit more bloated).

Hi, can you give more detail on what you're looking for? I.e. should it just alert with a message or does it need to implement a slack feature?
I would like to work on this. Thanks!

He wants it to replicate the !voteythumbs behaviour that already exists, but with a different keyword. This can be accomplished by adding the decorator @bot.on_command('voteythimbs') above the existing decorator in uqcsbot/uqcsbot/scripts/voteythumbs.py.


MAINT: update dependencies and versions

This is to close https://github.com/fornax-navo/fornax-demo-notebooks/issues/184 and to address the comment in https://github.com/fornax-navo/fornax-demo-notebooks/issues/173#issuecomment-1850792665

The review has been addressed, so I went ahead and merged this.


Image that always freezes on read

I believe I've found an image that freezes the tool every time it is read.
a.jpg.zip
Steps to reproduce:
multi_exiftool_reader = MultiExiftool::Reader.new
multi_exiftool_reader.filenames = [Rails.root.join('a.jpg')]
multi_exiftool_reader.read

I inspected the image with exiftool and I imagine the cause is the huge field "Mask Group Based Corr Mask Masks Dabs".
I've never seen anything like this before, so any ideas how to avoid a freeze are welcomed.

It's a problem with IO buffers. I will have a look at this.

I hope it's fixed in multi_exiftool version 0.18.1. Let me know if it works for you.

Perfect. That is amazing. Tested and fixed. I've been using multi_exiftool since 2010. I really appreciate it!

Thank you for your kind feedback.


Replace sha1 password hashing in initialisation template

The sha1 hashing used here is nowhere near secure enough for modern day applications, and should be changed.
At a minimum sha256 or sha512 should be used, as they are included in node's built-in crypto library, but preferably the Blowfish (bcrypt) algorithm could be included via an npm package without too much effort.
Although most responsible developers will replace this with their own security anyway, the temptation is still there for the default to be left in place, so I think it's worth making it secure by default.
Happy to submit a pull request myself if you agree with the above.

 A PR would be most appreciated pal!

To clarify, I don't ever want to include bcrypt into a project as I've never had out of the box success installing it.

Scheduled for the 0.4.8 maintenance release :+1:

Realised in my half asleep state this morning that this is not a user model per say, if you are using one of the auth plugins (JWT or OAuth this model is created for you using the core user model which uses the equally secure but NIST recommended PBKDF2 with a configurable iteration count https://github.com/Multicolour/multicolour/blob/master/lib/utils.js#L14-L40

Ah, didn't see that helper. I don't think it's worth updating that, PBKDF2 is pretty secure, and would mean a breaking change for little benefit. Might be worth revisiting at the next major release though.
If you're happy then, I'll update the example model to use utils.hash_password with salt instead of the sha1 hash.

Yep, that works for me.  Thanks for your time on this and the report 👍

Thanks for that @molovo, will be published on the 21st (I think that's my next scheduled maintenance release)

Published a couple days late but published in 0.4.8 :+1:
Thanks for your time @molovo! ❤


Show selected profile file

Show selected profile file

After the choose a file, the path of the file is displayed on the terminal, right below the question. It might be a lot nicer if we show it with the colored library without the complete path, just the name of the file.


Question About Blanket: Can Blanket cause a false positive on a Multiplayer server depending on the anti-cheat software being used?

Hey! I have a question about blanket, I hope u don't mind!
I've been a bit on edge about this mod if I where to use it on a Multiplayer server. Could it be possible for a false positive to happen and detect something wrongfully?
Please let me know! Thank you! :D

There is no simple answer for your question.
In Blanket, every tweak/fix is fully configurable, if you disable everything, it should be impossible to detect this mod.
Rendering and UI fixes can not be monitored by the server, using those shouldn't trigger anti-cheat.
Maybe the only thing in this mod what might trigger anti-cheat is minecart rotation tweak. (disabled by default)
It makes the player rotate with the minecart, and the server might not like it.

Alrighty. thanks man!


pull request for e2e test at 11/12/2016 16:45:10

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


The messages are taking too long to load!

The messages are taking too long to load, we can try to catch it to local storage

I would like to improve the performance and implement this @abhijeet007rocks8


Possibly null ref exception after remote tos update

I was working on my app tonight, using a VM that had been open for several days, using an account that had previously been working fine to connect to OneDrive.
Tonight, attempting to authenticate with that account threw a null reference exception. I thought maybe live services was down, but everything was green. I logged in with another account and noticed that there was a mandatory TOS update acceptance screen.
I killed the VM and logged in to the same (problem) account on a fresh VM... and got an updated TOS screen during authentication. Then everything was fine.
Therefore, I'm guessing there's an edge case around this, where maybe an account already has an auth token and it's now invalid due to updated TOS, or another scenario where the new TOS hasn't yet been accepted but the SDK can't handle the rejected auth.

Do you have a call stack on this error?  Happy to dig in if you can give me somewhere to poke at.

I have no other info besides what I wrote and I don't know of any way to simulate this situation since it was tied to a remote server event.

I'll keep my eyes out for the error on SO and over here, if this crops up again I can address it.


Bug: "You already have SDKMAN installed" when installing into an empty directory

Bug report
SDKMAN! does not install into an existing but empty directory.
I was trying to install SDKMAN! into /opt to not pollute my homedir. My user account does not have access to create anything in /opt so I first created a directory, gave myself access, and then tried to install SDKMAN!. That failed with the error message "You already have SDKMAN installed".
For the bug all that matters is that when the installation directory exists (but is empty), SDKMAN! fails to install.
To reproduce

$ mkdir -p sdkmantest
$ export SDKMAN_DIR="$(pwd)/sdkmantest"
$ curl -s "https://get.sdkman.io" | bash

(...)
                                                                 Now attempting installation...

Looking for a previous installation of SDKMAN...
SDKMAN found.

======================================================================================================
 You already have SDKMAN installed.
 SDKMAN was found at:

    /home/you/sdkmantest

System info

Windows 10 + WSL2
zsh 5.8 (x86_64-ubuntu-linux-gnu)
GNU bash, version 5.0.17(1)-release (x86_64-pc-linux-gnu)

On the installation page it says:

For this to work it is vital that your user has full access rights to this folder.It is also important that the folder does not exist as SDKMAN! will attempt to create it.

This seems like a contradiction to me: that a user should have full access rights to a folder that does not exist. What is meant probably is that the user has the rights to create folders in the parent directory for the SDKMAN_DIR.
I would opt that the installer checks the directory and if it exists but is empty, it will treat it as a valid installation directory.

Hi @t9t, this really isn't a bug and wasn't discussed on the user-issues channel on slack as per the contributing guidelines. Feel free to drop in there to ask any such questions.

Thank you very much for taking the time to look at the issue @marc0der.
I see that the "Feature" Github issue template has a note about reading the contribution guidelines, but the "Bug" template does not. So it might be a good idea to add a similar remark to that template as well, to reduce the risk of ignorant people opening an issue on Github when encountering a bug (I don't want to join a Slack group, which is why I chose to point it out here rather than there).
Apologies for opening a bug report issue for something that you don't consider to be a bug.

Apologies mine, I'll make sure the template is updated. But seriously, please hop onto slack. We'd love to help you get the problem solved, and if necessary improve the functionality.

$ export SDKMAN_DIR="$(pwd)/sdkmantest"
$ curl -s "https://get.sdkman.io" | zsh


caps, srm, crossover

🚀 Deployed on https://6629432e84363c27d738bc12--cozy-fox-0defec.netlify.app


Error building with Xcode 14.3

Bug
Unexpected behavior
When building a project using the latest version of this library, the following error occurs.
In file included from <project-paht>/node_modules/react-native-svg/apple/Elements/RNSVGUse.mm:16:
<project-paht>/ios/Pods/Headers/Public/React-Fabric/react/renderer/components/view/conversions.h:111:39: error: 'value' is unavailable: introduced in iOS 12.0
          ? std::optional<Float>(base.value() * floatFromYogaFloat(value.value))

Other libraries have the same error and it seems to be related to Xcode 14.3 see this PR
Possible fix:
Change this line from
s.platforms       = { ios: '11.0', tvos: '11.0' }

to
s.platforms       = { ios: '12.4', tvos: '11.0' }

shameless bump

I want a release :)

When you will release this?

I used patch-package in the meantime btw :)

It's released in v13.10.0 🚀


Broader dependencies versions

Allow more broader dependencies to prevent version conflicts.
https://packaging.python.org/discussions/install-requires-vs-requirements/#install-requires-vs-requirements-files
https://semver.org/

Thanks!


fb-2026 : added test cases with cache type & size for copy

This PR caters to the test cases added for testing copy with cache type & cache size different than the default Pilosa Cache type & size for fld type - fldTypeID, fldTypeIDSet, fldTypeString, fldTypeStringSet

Merged in private repo


Closure Compiler: JSC_UNDEFINED_VARIABLE JSCompiler_renameProperty

Description
Cannot build with Closure Compiler, even in ADVANCED mode with use_types_for_optimization, due to JSCompiler_renameProperty and trustedTypes
Live Demo
See repro here
Steps to Reproduce

Set up an environment with lit-element via npm or yarn (in my case, yarn from bazel)
Write a little sample element that imports lit-element and uses it
Compile it with Closure Compiler in ADVANCED mode, with use_types_for_optimization set to true

Expected Results
Built bundle, with JSCompiler_renameProperty being treated as a special compiler primitive
Actual Results
Instead, it is treated like a regular variable name, which is missing from the compilation unit, yielding a JSC_UNDEFINED_VARIABLE error
Browsers Affected
N/A (Build tooling)
Versions

lit-element: v2.4.0 via NPM
webcomponents: N/A

Full invocation of Bazel with all flags to Closure:
ERROR: /Volumes/KICKSTART/lit-element-repro/BUILD.bazel:12:24: Action example-bundle.js failed (Exit 1): sandbox-exec failed: error executing command
  (cd /private/var/tmp/_bazel_sam.g/ba98e5c3c766625218413987b11d8086/sandbox/darwin-sandbox/4/execroot/lit_element_repro && \
  exec env - \
    BAZEL_NODE_MODULES_ROOT=npm/node_modules \
    COMPILATION_MODE=fastbuild \
    TMPDIR=/var/folders/16/tfkstzwd5478svc3smbwgzbm0000gn/T/ \
  /usr/bin/sandbox-exec -f /private/var/tmp/_bazel_sam.g/ba98e5c3c766625218413987b11d8086/sandbox/darwin-sandbox/4/sandbox.sb /var/tmp/_bazel_sam.g/install/3ac6e4bc67346f686d73708a27e81d33/process-wrapper '--timeout=0' '--kill_delay=15' bazel-out/host/bin/external/npm/google-closure-compiler/bin/google-closure-compiler.sh '--platform=native' '--isolation_mode=IIFE' '--compilation_level=ADVANCED' '--dependency_mode=PRUNE_LEGACY' '--process_common_js_modules=true' '--process_closure_primitives=true' '--module_resolution=NODE' '--env=BROWSER' '--language_in=STABLE' '--language_out=STABLE' '--use_types_for_optimization=true' '--js=node_modules/lit-element/package.json' '--js=node_modules/lit-element/lit-element.js' '--js=node_modules/lit-element/lib/*.js' '--js=node_modules/lit-html/package.json' '--js=node_modules/lit-html/lit-html.js' '--js=node_modules/lit-html/lib/*.js' '--js=./example-element.js' '--entry_point=./example-element.js' '--js_output_file=bazel-out/darwin-fastbuild/bin/example-bundle.js' --debug '--bazel_node_modules_manifest=bazel-out/darwin-fastbuild/bin/_example-bundle.module_mappings.json' --nobazel_patch_module_resolver) sandbox-exec failed: error executing command
  (cd /private/var/tmp/_bazel_sam.g/ba98e5c3c766625218413987b11d8086/sandbox/darwin-sandbox/4/execroot/lit_element_repro && \
  exec env - \
    BAZEL_NODE_MODULES_ROOT=npm/node_modules \
    COMPILATION_MODE=fastbuild \
    TMPDIR=/var/folders/16/tfkstzwd5478svc3smbwgzbm0000gn/T/ \
  /usr/bin/sandbox-exec -f /private/var/tmp/_bazel_sam.g/ba98e5c3c766625218413987b11d8086/sandbox/darwin-sandbox/4/sandbox.sb /var/tmp/_bazel_sam.g/install/3ac6e4bc67346f686d73708a27e81d33/process-wrapper '--timeout=0' '--kill_delay=15' bazel-out/host/bin/external/npm/google-closure-compiler/bin/google-closure-compiler.sh '--platform=native' '--isolation_mode=IIFE' '--compilation_level=ADVANCED' '--dependency_mode=PRUNE_LEGACY' '--process_common_js_modules=true' '--process_closure_primitives=true' '--module_resolution=NODE' '--env=BROWSER' '--language_in=STABLE' '--language_out=STABLE' '--use_types_for_optimization=true' '--js=node_modules/lit-element/package.json' '--js=node_modules/lit-element/lit-element.js' '--js=node_modules/lit-element/lib/*.js' '--js=node_modules/lit-html/package.json' '--js=node_modules/lit-html/lit-html.js' '--js=node_modules/lit-html/lib/*.js' '--js=./example-element.js' '--entry_point=./example-element.js' '--js_output_file=bazel-out/darwin-fastbuild/bin/example-bundle.js' --debug '--bazel_node_modules_manifest=bazel-out/darwin-fastbuild/bin/_example-bundle.module_mappings.json' --nobazel_patch_module_resolver)
(node:9565) ExperimentalWarning: The fs.promises API is experimental
node_modules/lit-element/lib/updating-element.js:118:33:
Originally at:
node_modules/lit-element/src/lib/updating-element.ts:278:12: ERROR - [JSC_UNDEFINED_VARIABLE] variable JSCompiler_renameProperty is undeclared
  278|             JSCompiler_renameProperty('_classProperties', this))) {
                   ^^^^^^^^^^^^^^^^^^^^^^^^^

1 error(s), 0 warning(s)
Target //:example-bundle failed to build
INFO: Elapsed time: 2.714s, Critical Path: 2.12s
INFO: 2 processes: 2 internal.
FAILED: Build did NOT complete successfully

Do you have a repro I can take a look at? I'm unfamiliar with using bazel with closure compiler.
I've confrirmed that this isn't an internal google thing, JSCompiler_renameProperty is in the open source version of Closure Compiler: https://github.com/google/closure-compiler/blob/1bd6e130db57a1020dcd0b7f096ad030b2ae2db2/src/com/google/javascript/jscomp/NodeUtil.java
It looks like it should be processed whenever property renaming is enabled.

@rictic agreed, it's odd - i've seen hints of needing use_types_for_optimization, or process_closure_primitives. both are in use in this case.
i've tried dispatching the Compiler through rules_closure and through google-closure-compiler (npm), no dice either time. the difference in that invocation is:

google-closure-compiler > node > native OSX binary of compiler, or jar, via command line > Compiler
rules_closure > JsCompiler / JsCompilerRunner > Compiler

in both cases, in ADVANCED mode, the RenameProperties pass should be active, but doesn't seem to be.
the repro is here: https://github.com/sgammon/lit-element-repro

so, it's worth mentioning that this is likely not related to Bazel, since the former dispatch method largely dispatches it the same way as other non-google tooling, or a manual CLI invocation.

hi @rictic, does this need to stay open?

@nicolejadeyee still trying to get this to work with bazel for now, so yeah :(


0.2.14

removes some workarounds and patches that should no longer be needed

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/recipe.yaml) and found it was in an excellent condition.


Compilation of Qt moc CMake-generated files should depend on precompiled header

Otherwise, the following error is generated on Windows if the precompiled header changes:
moc_compilation.cpp.obj : error LNK2011: precompiled object not linked in; image may not run

Fixed by 322cbb2913e9e90ba984963b241c0eb72a4c19cc. Only for Makefile-based and Ninja generators. Doesn't check AUTOGEN_BUILD_DIR.


Imou Ranger 2 4Mp Support

Hi,
can you add support also for this camera? When I try integration and fill all requested fields correctly, I am still getting error saying, that username, password and/or IP address are wrong. But I am 100 % sure, that values are right (camera works fine with WebRTC Camera integration with same credentials).
Thank you
Jirka

Hi
Same problem for me
Best regards
Mickaël

Hi,
I firstly installed ONVIF to allow PT control and then, I installed WebRTC. After restart, I added custom card WebRTC, but not see any video (see my issue here: https://github.com/AlexxIT/WebRTC/issues/333). I saw, that H.265 is not supported (but noticed it later, after I posted issue there), so I tried to change to H.264. To change this, I used Dahua's (original camera manufacturer) Config Tool (https://dahuawiki.com/ConfigTool), I logged in using username and password (or security code, if you not set password) and changed codec from H.265 to H.264 and also changed VBR to CBR with maximal quality. The, I restarted camera and HA and video was visible. Now, I am fighting with sound - althrough I set to use PCMA (A-law), sound is outputted only on one of my indetical cameras in HA. But second one is working well after that change. I will try to play with Windows, as I changed it using Mac version. But video is working fine and PT also.

Hi
it seems complicated but I will try.
Thank you
Michael
Le mar. 2 août 2022 à 08:28, Betacart @.***> a écrit :

Hi,
I firstly installed ONVIF to allow PT control and then, I installed
WebRTC. After restart, I added custom card WebRTC, but not see any video
(see my issue here: AlexxIT/WebRTC#333
https://github.com/AlexxIT/WebRTC/issues/333). I saw, that H.265 is not
supported (but noticed it later, after I posted issue there), so I tried to
change to H.264. To change this, I used Dahua's (original camera
manufacturer) Config Tool (https://dahuawiki.com/ConfigTool), I logged in
using username and password (or security code, if you not set password) and
changed codec from H.265 to H.264 and also changed VBR to CBR with maximal
quality. The, I restarted camera and HA and video was visible. Now, I am
fighting with sound - althrough I set to use PCMA (A-law), sound is
outputted only on one of my indetical cameras in HA. But second one is
working well after that change. I will try to play with Windows, as I
changed it using Mac version. But video is working fine and PT also.
—
Reply to this email directly, view it on GitHub
https://github.com/rroller/dahua/issues/208#issuecomment-1202071413, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4MALVS7FX3MZLJ4GTPNUTVXC5Y3ANCNFSM547VEJSA
.
You are receiving this because you commented.Message ID:
@.***>

Hi,
it looks like, but it is not:

install ONVIF
install WebRTC
if you have 4K camera, use ConfigTool to change codec from H.265 to H.264
add Custom Card: WebRTC with camera RTSP address and you should have a picture

hello
I am blocked from the first step, (onvif installation), can I ask you for
your help?
good evening
Michael
Le mer. 3 août 2022 à 07:33, Betacart @.***> a écrit :

Hi,
it looks like, but it is not:

install ONVIF
install WebRTC
if you have 4K camera, use ConfigTool to change codec from H.265 to
H.264
add Custom Card: WebRTC with camera RTSP address and you should have
a picture

—
Reply to this email directly, view it on GitHub
https://github.com/rroller/dahua/issues/208#issuecomment-1203504000, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4MALSI7ZBEW4ZU3KR3NSDVXIAD5ANCNFSM547VEJSA
.
You are receiving this because you commented.Message ID:
@.***>

No problem:

go to Settings/Devices
search for ONVIF in Integrations
enter Name (any name you want for device)
enter Host (it is IP address of your camera - recommending assign static IPs for all your cameras)
leave Port as 80
enter User Name admin
enter password - you can set it in Immou Life app, if not set, then you need to enter security code at the bottom of your camera

DONE :)

Thank you very much for your help
When I try to log in, I get a connection error message.
Le ven. 5 août 2022 à 21:06, Betacart @.***> a écrit :

No problem:

go to Settings/Devices
search for ONVIF in Integrations
enter Name (any name you want for device)
enter Host (it is IP address of your camera - recommending assign
static IPs for all your cameras)
leave Port as 80
enter User Name admin
enter password - you can set it in Immou Life app, if not set, then
you need to enter security code at the bottom of your camera

DONE :)
—
Reply to this email directly, view it on GitHub
https://github.com/rroller/dahua/issues/208#issuecomment-1206774728, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4MALSQRZQ4MK6XTYCPASTVXVQ2VANCNFSM547VEJSA
.
You are receiving this because you commented.Message ID:
@.***>

Hi,
this is because you either entered wrong password (you need to set it first in camera's settings - you will need security code, which is at the bottom of the camera on the sticker) or you not enter security code (in case you not set your own password). User name is always admin.

Thanks a lot I managed to configure onvif and change the codec to 264. I
have picture and sound in a frigate card. Now I will look for how to save
bye Mike
Le mer. 3 août 2022 à 07:33, Betacart @.***> a écrit :

Hi,
it looks like, but it is not:

install ONVIF
install WebRTC
if you have 4K camera, use ConfigTool to change codec from H.265 to
H.264
add Custom Card: WebRTC with camera RTSP address and you should have
a picture

—
Reply to this email directly, view it on GitHub
https://github.com/rroller/dahua/issues/208#issuecomment-1203504000, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4MALSI7ZBEW4ZU3KR3NSDVXIAD5ANCNFSM547VEJSA
.
You are receiving this because you commented.Message ID:
@.***>

Welcome, I am glad to help you!
Anyway, for developer of addon - it will be nice to support also this model (see my original post at the top).

Hi,
can you add support also for this camera? When I try integration and fill all requested fields correctly, I am still getting error saying, that username, password and/or IP address are wrong. But I am 100 % sure, that values are right (camera works fine with WebRTC Camera integration with same credentials).
Thank you Jirka

I'm anxiously waiting for it.
I have 2 IPC-A43 and I can use only 1 with Onvif (https://github.com/home-assistant/core/issues/70056). The other one I use with RTSP, but without PTZ ☹

Do you guys have your sensors working?
Well, with this camera it seems very unstable, the integration, and sensors not working since the begginning. Anything to do?

Hi, I am not using any of this sensors - just use it for live view of picture. But I definitely agree, that integration needs to be improved!

Imou does not support native Dahua API, use the Imou Life cloud integration instead https://github.com/user2684/imou_life

Imou does not support native Dahua API, use the Imou Life cloud integration instead https://github.com/user2684/imou_life

I also got a Range2C and that's what I did at first but, it's really infuriating to be dependant on tens of differents sellers Clouds and/or applications for each devices bought...


Update link of Public Domain

SonarQube analysis reported 1 issue

 1 minor

Watch the comments in this conversation to review them.

Coverage increased (+0.05%) to 84.321% when pulling 4b05fcc127880537c81451046c04baf73c99b2f9 on bugfix-data-licenses into 2c1709dd38b34d0469564b0131b72135069a633c on master.


Bump version to "v12.1.5" with changelog.

Today’s release should be 12.2

Based on the change set, I think we should skip this patch release.
Also, the change log appears to be unformatted (validate check is failing).

Alright, dropping this release and moving on to the 12.2.0 release.


"Copy HTML" button broken in Safari

Hi!
Not sure if you care about browser compat bugs for this, but figured I'd report it just in case.
In Safari browser, clicking the "Copy HTML" button doesn't copy to the clipboard, and doesn't show the butterbar with "copy successful" at the bottom.
Each time the button is clicked, it produces the error Unhandled Promise Rejection: SyntaxError: The string did not match the expected pattern. in the browser console.
Tested in Safari v16.2. Please let me know if you need any more info!

Weirdly, it seems to break on webkit pseudo-selectors:
> element.matches(".mdc-text-field__input:-webkit-calendar-picker-indicator")
< syntaxerror: The string did not match the expected pattern.

I guess it might be worth wrapping this in a try/catch?


Absolute path

If you rename geckolib-forge-.jar isn't working any more and mc tells you to install gecklib again. Which means somewhere there is static path or static name.

 Sounds like an issue with one of your mods that requires geckolib however, not on our side.


[dmlc-core] Add new port

Dmlc-core only supports static build. When building dynamic, it will shows:
Expected Debug,Dynamic crt linkage, but the following libs had invalid crt linkage:

   F:/update/vcpkg/packages/dmlc-core_x86-windows/debug/lib/dmlc.lib: Debug,Static

To inspect the lib files, use:
   dumpbin.exe /directives mylibfile.lib
Expected Release,Dynamic crt linkage, but the following libs had invalid crt linkage:

   F:/update/vcpkg/packages/dmlc-core_x86-windows/lib/dmlc.lib: Release,Static

To inspect the lib files, use:
   dumpbin.exe /directives mylibfile.lib
Found 2 error(s). Please correct the portfile:
   F:\update\vcpkg\ports\dmlc-core\portfile.cmake
-- Performing post-build validation done
Error: Building package dmlc-core:x86-windows failed with: POST_BUILD_CHECKS_FAILED

So I added vcpkg_check_linkage(ONLY_STATIC_LIBRARY ONLY_STATIC_CRT) in portfile.cmake.

/azp run

/azp run

I noticed that it failed on x86-windows, due to

C:/vsts/_work/1/s/buildtrees/dmlc-core/src/5547c00bf1-c597496dc4/include/dmlc/build_config.h
CMake Error at CMakeLists.txt:106 (configure_file):
configure_file Problem configuring file
-- Could NOT find PythonInterp (missing: PYTHON_EXECUTABLE)
-- Configuring incomplete, errors occurred!

But it can be built successfully on my local. So I'm confused about this.

/azp run


Spotify Hifi quality coming

Spotify announced that it will soon introduce lossless, hifi quality streaming. Current maximum is 320 kbps, which is already supported by Balena Sound. It would be great if the new lossless feature could be used as well with Balena Sound. Is there an already existing way to ensure it, or does it require some development?

Hi @Szubszidiaritas, do you have a link to share about that announcement? According to docs 320kbps is high quality already, is hifi a new higher bitrate?
# Bitrate, one of 96 (low quality), 160 (default quality), or 320 (high quality)

In any case, if this is treated as a new bitrate then support for it should be automatically added once the upstream package we use for spotify adds support for it. It should be done in the same way we currently support 96, 160 and 320 kbps, by setting SOUND_SPOTIFY_BITRATE to the new bitrate.

Hi @tmigone,
Thank you for your reply! Here is one source with some details on changes in bitrate:
https://www.whathifi.com/features/spotify-hifi-is-missing-something-but-will-it-matter
The bottom line is, Spotify HiFi will enable 1411 kbps streaming in contrast with the current highest quality option, 320 kbps.
Hopefully, the solution you assume will do the trick (setting bitrate to 1411), but I am not at all knowledgeable about the technical details, ie. whether or not it will be treated as a new bitrate, or they will do it some other way. I guess we'll see how they plan to implement it when it is already launched. :)


[bootstrap] Add support for --test-filter.

This is useful for running a small set of tests via bootstrap test.

@swift-ci please smoke test

@swift-ci please smoke test


how can i fix this error

Wow it's a freaky error haha!
It's not supposed to happen at all though...
Does it happen often or just once?
Does it appear since you changed something in your code?

Hey, can you try running node -v in your console ?

Okay so the flatMap method has arrived in nodejs v14, so try to update your version :)


Add an options to flow the incoming distributed tracing headers

Currently YARP would remove the distributed tracing headers from an incoming requests, and the new updated headers is being added to it by SocketsHttpHandler. These new headers counts YARP as a SPAN, and if we look at the distributed tracing in a tool like Zipkin, the hierarchy would be [client -> YARP -> Destination]. Although This would happen only when all three applications are logging to a distributed tracing system.
There might be some scenarios when YARP is not configured to be in part of distributed tracing system or users don't want to count YARP as a SPAN in distributed tracing system. In this case I think the hierarchy between client and destination would be lost because of the missing link (YARP). In this case an option for clusters to just flow the incoming distributed tracing headers is helpful.

This is exactly what the PassThroughPropagator was designed for.
DistributedContextPropagator.Current = DistributedContextPropagator.CreatePassThroughPropagator();

// Or
services
    .AddReverseProxy()
    .ConfigureHttpClient((context, handler) =>
    {
        handler.ActivityHeadersPropagator = DistributedContextPropagator.CreatePassThroughPropagator();
    });

Cool, I didn't know about this option.
Now should this be an option in HttpClientConfig? It might be useful when having YARP as an exe or docker, so this could be set via config instead of code.

This is what could be added. I will send a PR if you agree with the design.
public enum DistributedContextPropagator
{
    Default = 0, // CreateDefaultPropagator()
    PassThrough = 1, // CreatePassThroughPropagator()
    NoOutput = 2, // CreateNoOutputPropagator()
    Disable = 3 // null , (I'm not sure if there's any behavior different between setting null and CreateNoOutputPropagator)
}

public class HttpClientConfig
{
+    public DistributedContextPropagator? DistributedContextPropagator  { get; init; }
}

This would only be for 6.0+, right?

This would only be for 6.0+, right?

Yes, I believe so.

It would be 6.0+ only, but nothing stops the user from implementing what they need on top of something like the sample DiagnosticsHandlerFactory

I'm not sure if there's any behavior different between setting null and CreateNoOutputPropagator

NoOutputPropagator means HttpClient will still raise diagnostic events, but won't add headers.
null means no events and no headers - the handler won't be included at all.
In the context of Yarp, I think the following options would make sense:

remove headers
keep the original headers (passthrough, 5.0 default)
maintain the trace and create a new span (6.0 default)

Only the first two should ever be useful to set from the config - for the third you would want code instrumentation.

Triage: We think default will work for most people. If we have more upvotes, we can add a config.
Leaving it open for a while to collect more feedback.

I'm a bit confused as to what needs to be done here: it sounds like adding the passthrough propagator solves the issue. If so, we should add to the samples/docs.

Triage: We have it doc'ed a bit, so no need to do more until we have more upvotes.


Create workspace renaming endpoint

Creates a new endpoint that re-enables workspace renaming functionality by making a new WorkspaceRenameSerializer and using it in a new update() function in the WorkspaceViewset class.
Works in conjunction with the rename-workspace branches of multinetjs and multinet-client.
Resolves #39

I think we should add a test here, to both test renaming functionality as well as permission handling. @eagw Lmk if you want some guidance here.

I think we should add a test here, to both test renaming functionality as well as permission handling. @eagw Lmk if you want some guidance here.

@AlmightyYakob How does 25d5a22 look for a test?


ImageWorks manifests are broken

Manifests can't be built for ImageWorks. This is the error raised:
F, [2017-04-20T10:00:03.952613 #7136] FATAL -- : NoMethodError (undefined method `pdf_type' for #<ImageWorkShowPresenter:0x007f07af04d108>):
F, [2017-04-20T10:00:03.952765 #7136] FATAL -- :
F, [2017-04-20T10:00:03.952819 #7136] FATAL -- : app/services/manifest_builder/pdf_link_builder.rb:12:in `apply'
app/services/manifest_builder/composite_builder.rb:11:in `block in apply'
app/services/manifest_builder/composite_builder.rb:10:in `each'
app/services/manifest_builder/composite_builder.rb:10:in `apply'
app/services/manifest_builder.rb:9:in `initialize'
app/services/polymorphic_manifest_builder.rb:7:in `new'
app/services/polymorphic_manifest_builder.rb:7:in `new'
app/controllers/concerns/hyrax/manifest.rb:39:in `block in manifest_builder'
app/controllers/concerns/hyrax/manifest.rb:38:in `manifest_builder'
app/controllers/concerns/hyrax/manifest.rb:8:in `block (2 levels) in manifest'
app/controllers/concerns/hyrax/manifest.rb:6:in `manifest'

Probably this new line added to handle pdf_type of "none": https://github.com/pulibrary/plum/blob/master/app/services/manifest_builder/pdf_link_builder.rb#L12

Yeah, just needed to add pdf_type to the show presenter.


Task 20

Read the excel (consists of Lower Letters), and type into website (Vajrang.org - Uppercase Task) and get the result and write into one excel.

@Bhavyajetty2
Task – 19
Code verified.
Task – 20
Code verified.


Valid genesis file rejected due to capitalization difference

In the process of building a pyethapp Dockerfile for hive, I discovered that pyethapp will reject a perfectly valid genesis json if the capitalization of the keys is different than expected. (Specifically: mixHash vs. mixhash)
Checklist

[ ] installed pyethapp inside a virtualenv (used Docker instead)

System/environment

OS: Ubuntu 16.04
python version: 2.7.12
pyethapp version: 1.5.0/linux2/py2.7.12
installation method (pip & pypi/git & setup.py): git and setup.py
pyethapp branch (if applicable): develop

Expected behaviour
A successful import of a standard genesis.json file. I used this .json from hive.
Observed behaviour
It fails like the following
/usr/local/lib/python2.7/dist-packages/pycryptodome-3.4-py2.7-linux-x86_64.egg/Crypto/Hash/keccak.py:139: UserWarning: implicit cast from 'char *' to a different pointer type: will be forbidden in the future (check that the types are as you expect; use an explicit ffi.cast() if they are correct)
  c_size_t(self.digest_size))

Failed to import scrypt. This is not a fatal error but does
mean that you cannot create or decrypt privkey jsons that use
scrypt

INFO:config setup default config path=/root/.config/pyethapp
INFO:config writing config path=/root/.config/pyethapp/config.yaml
INFO:app    using data in path=/root/.config/pyethapp
INFO:config loading config path=/root/.config/pyethapp
Traceback (most recent call last):
  File "/usr/local/bin/pyethapp", line 11, in <module>
    load_entry_point('pyethapp', 'console_scripts', 'pyethapp')()
  File "/usr/local/lib/python2.7/dist-packages/click-6.6-py2.7.egg/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click-6.6-py2.7.egg/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/dist-packages/click-6.6-py2.7.egg/click/core.py", line 1057, in invoke
    Command.invoke(self, ctx)
  File "/usr/local/lib/python2.7/dist-packages/click-6.6-py2.7.egg/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/dist-packages/click-6.6-py2.7.egg/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click-6.6-py2.7.egg/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/pyethapp/pyethapp/app.py", line 148, in app
    genesis_json_filename_or_dict=config['eth']['genesis'])
  File "/pyethapp/pyethapp/config.py", line 207, in update_config_from_genesis_json
    raise ValueError('genesis_dict contains invalid keys.')
ValueError: genesis_dict contains invalid keys.

Steps to reproduce
Use that .json up there, or just change the capitalization of mixHash in a working .json.
FYI, these are the offending lines. I'd fix it myself, but I'm not sure what you'd want done. Is it supposed to be case-insensitive? Is pyethapp right in this instance, and it's the other implementations that are wrong?

In addition, a genesis.json will be rejected if contains an 0x in an address in the allocation. This file has both issues, for easy testing.


Cannot use <script type="importmap"> in dev mode.

Describe the bug
Cannot use <script type="importmap"> in dev mode. Cannot redirect as map when use <script type="module">.
But when use http-server to serve index.html, browser can download this file defined in importmap.
Reproduction
https://stackblitz.com/edit/vitejs-vite-db7u22?file=index.html&terminal=dev
Steps to reproduce
No response
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (8) x64 Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz
    Memory: 7.14 GB / 31.79 GB
  Binaries:
    Node: 14.18.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.4 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.14.15 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 110.0.5481.177
    Edge: Spartan (44.22621.1265.0), Chromium (110.0.1587.56)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    vite: ^4.1.4 => 4.1.4

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

duplicated with https://github.com/vitejs/vite/issues/2483. There's a workaround provided by Evan.


feature Support to acquire md5 digest of the file to download

Proposed changes
Issue Number: close #xxx

run buildall


Use SANE Ember-cli and Sails by default

I think we could re-think the strategy of using SANE vs. global ember-cli (and sails).  Right now, there is a significant amount of code to simply determine what is installed and what the version numbers are.  This causes issues in Windows (see https://github.com/artificialio/sane/pull/129) and adds a lot of time to starting up a new SANE project.
This could likely be streamlined by using npm-which and defaulting to a set behavior.  For example, by default SANE could use its own ember-cli unless a --global-ember variable is passed.
I'd like to at least start a discussion around this, and find out what people think is the best way to handle ember-cli (and sails.js) versioning.  Do you like being given the option, even if it is at expense of sane new performance?  Would a default behavior (e.g. using the SANE versions) which also alerts you to the version numbers being used be sufficient?

Yeah you are right the current way slows creating a new project on Mac a few seconds and I believe more on Windows. A flag would certainly reduce that overhead because then we don't have to make any guesses and can immediately take the one or the other, I am just always careful with be flags because it is something to think about and one more thing for new developers.
But I agree it would be good to streamline the behaviour and have a discussion what members of the community think.

I guess for other commands (at least on Linux / Mac) the questions becomes easier because we always now there is a sails and ember - cli already in the node_modules.

I think it should default to the SANE CLI as the average user will probably be unaware of the option, and thus probably not need it. Plus the SANE CLI is, I assume, optimized for the SANE stack so it'd be the superior option in the first place.

One thing to keep in mind is that sane new is used once per project. If it were something I was running constantly I might be worried about a few seconds worth of performance but since it is not, I don't mind the wait.
And I prefer the choice of the version. I might have different projects going that might not all be updated to the latest versions or something and having the choice is best I believe.  I could also see adding a flag to make the choice ahead of time but not just defaulting to something one way or another.


Null selected on taggable throws exception.

I'm getting an exception, that occurs when I pass down from parent selected, which is null.
Multiselect is taggable.
He're the exception:
app-cd1a3cff48.js:49506 [Vue warn]: Error when evaluating expression "
function () {
    var t = this.hideSelected ? this.options.filter(this.isNotSelected) : this.options;
    return t = this.$options.filters.filterBy(t, this.search), this.taggable && this.search.length > 0 && !this.isExistingOption(this.search) && t.unshift({ isTag: !0, label: this.search }), t;
        }
": TypeError: Cannot read property 'length' of null (found in component: <multiselect>)

Pull request fixes the issue.

Merged and additional made small adjustments myself. Thanks a lot! :)


Merged PR 267: v1.14.0 - New Game team selection from QBJ tournament

Support choosing teams from a QBJ Tournament file when starting a new game (#190)
Bump version to 1.14.0

I still need to make the site that generates these files, but the format isn't too complicated


Weight of iCub(s)

iCubGenova01
iCubGenova02 (v. 2.5) with battery or dummy
iCubGenova03
iCubDarmstadt01

30.3
33.1
27.5
30.4

The iCubs were weighted without the skin under the feet's soles. In the case it is installed, add to the above weights 0.2 Kg (see #56)
For the sake of completeness, we intend as:
iCubGenova01:  head v2, arms v1, SEA legs (without SEA and feet v2)
iCubGenova03:  head v2, arms v2, legs v1 , feet v2,
@traversaro @francesco-romano @maggia80 @fabrizio69 @super-ste @Mick3Lozzo

Finally I know what exact numbers to give when guests will asking for those weights!
Thanks @DanielePucci

:dancers:

Please remove version tag (except from iCubGenova02) they're misleading.
The only way to classify our robot is givin' out the exact description of the way they're made.
e.g. for iCubGenova01
iCub V1 Body
head V2
SEA legs (without SEA)
:sweat_smile:

Ack.

@DanielePucci it would be nice if you modify your first post according to @super-ste 's suggestions :+1:

Done, checks are welcome.

Hi Daniele,
What about putting all this weight information in the official
documentation of iCub ? I think that it might be easier to find there.
Le 12/10/2015 15:01, Daniele a écrit :

Done, checks are welcome.
—
Reply to this email directly or view it on GitHub
https://github.com/robotology/QA/issues/61#issuecomment-147391367.

Hi @dogoepp,
nice idea. Volunteers are welcome.

For the weights on iCubGenova01 and iCubGenova03 the photos were shot on 30/04/2014, so they reflect the situation of the robots at that point in time.

For consistency checks, it would be awesome if our friends in Paris @rlober @mingxing-liu could weight their robots :)

Added iCubDarmstadt01 as in https://github.com/robotology/icub-support/issues/118

:100:

@DanielePucci
Next time I am down in the cave I will check our iCubs' body mass indexes!

:+1:

@spada1978

The dummy has been re-designed. Then, the weight is changed accordingly.

For further information on the previous dummy weight, see #56

@DanielePucci what is the status of the models commited in the repositories?

I sincerely don't remember, but I have flashes of a conversation with you where I asked you to add additional weight on the dummy. We shall check @traversaro @francesco-romano

Last week I weighed the green iCub (iCubGenova04, v2.5 with battery pack) without the skin under the feet's soles and this is the result:

i.e., the same the weight of the purple (as expected)

iCubGenova04 on the scale.
Total weight = 33.33kg
@DanielePucci

Nice @fjandrad. Can we however update the table on the first https://github.com/robotology/QA/issues/61#issue-110842856?


Add support for quadruple bonds in SMILES

The OpenSMILES specification allows $ to stand for quadruple bonds, with "[Rh-](Cl)(Cl)(Cl)(Cl)$[Rh-](Cl)(Cl)(Cl)Cl" as an example.  From testing locally it seems that Open Babel and the CDK parse this as intended and I believe OpenEye does as well.
Is there a reason not to support this in RDKit?

This change is already in on the non-tetrahedral stereo branch, but given that it's going to take a while to finish that and get it ready for merging, it probably makes sense to just merge that as a discrete change onto master.

Thanks @greglandrum for the speedy response!


Expose RichTextLabel::push_customfx to GDScript

This makes it possible to push custom text fx to a RichTextLabel without having to go through bbcode, like the other push_* methods.
Moves set_process_internal(true) down to push_customfx to ensure process is enabled for the label when a custom effect is used in this way.

Thanks!


feat: Add CLI extensions so users can run custom inspectors & classifiers without forking.

Description:
Currently, if someone wants to write custom inspectors for Brontes, they would need to fork the project and add their code directly. This approach isn't maintainable in the long term due to potential merge conflicts and the need to modify the database schema for storing newly classified data.
To improve the extensibility of Brontes, we want to add support for custom inspectors that can be defined and installed by users without requiring them to fork the project. This will enable users to create and use their own inspectors while keeping their codebase separate from the main Brontes repository.
Requirements

Implement a mechanism for users to define custom inspectors in their own repositories by implementing the existing Inspector trait.
Provide a way to install and load custom inspectors dynamically in Brontes.
Ensure that custom inspectors can be run alongside the built-in inspectors.
Modify the inspector initialization and execution logic to incorporate custom inspectors.
Maintain backward compatibility with the existing inspector system.

Additional Context|
See reth's CLI extension reth cli ext
LibEVM reth cli extension guide

can i take on this ?


Fixed issue 6623 by raising errors for negative delay durations

Summary
Fixes #6623
Details and comments
Negative durations for the delay gate were erroneously accepted as valid inputs. This was resolved by adding conditional statements to the validate_parameter method for Delay, which raise CircuitErrors if the provided integer ('dt') or float ('ns', 'ps', etc.) durations are negative.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Caleb Clothier 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.

@kdk I implemented your suggestions in the most recent commit, let me know if any further changes are needed.


Limitations of OSV DB API

Hello All, I am creating this issue to raise a question on the limitations of OSV DB API.

For example, GitHub APIs have restrictions on the number of request calls that can be made with a specific User access token in an hour.
If there are restrictions, what should the user do to overcome such limits? Whom do you contact regarding this?
I observed that https://api.osv.dev/v1/querybatch API call with multiple queries, returns only a list of GitHub advisories, should it not return the complete vulnerability object with CVE, impact, severity and other infos? Are we missing something here?

Please find my curl sample, as you can see below, the result is just a flat list of GitHub advisories
curl --location --request POST 'https://api.osv.dev/v1/querybatch' \
--header 'Content-Type: application/json' \
--data-raw '{
   "queries":[
        {
            "version":"2.12.6",
            "package": {
                "name": "com.fasterxml.jackson.core:jackson-databind",
                "ecosystem": "Maven"
            }
        },
        {
            "version": "1.59",
            "package":{
               "name":"org.bouncycastle:bcprov-jdk15on",
               "ecosystem":"Maven"
        }
      }
   ]
}'

Output:
{
    "results": [
        {
            "vulns": [
                {
                    "id": "GHSA-57j2-w4cx-62h2",
                    "modified": "2022-12-14T05:10:13.978383Z"
                },
                {
                    "id": "GHSA-jjjh-jjxp-wpff",
                    "modified": "2022-12-14T05:29:48.906253Z"
                },
                {
                    "id": "GHSA-rgv9-q543-rqg4",
                    "modified": "2022-12-14T05:06:42.513025Z"
                }
            ]
        },
        {
            "vulns": [
                {
                    "id": "GHSA-6xx3-rg99-gc3p",
                    "modified": "2022-12-14T04:57:02.505254Z"
                }
            ]
        }
    ]
}

Please let me know if this is the right forum to post the questions.
Thanks
Sourabh

Hi!
There are currently no rate limits for our API.
Re the querybatch question: The responses are not hydrated -- you would need to make separate GET requests to retrieve the full vulnerability objects. Our auto-generated documentation unfortunately lacks this detail -- this will be solved as part of  https://github.com/google/osv.dev/issues/573


RMSProp optimizer.

Is there any particular reason for using RMSPropOptimizer @shashankg7  ?
I am just curious. I hope discussing it here is fine?

Hi. No particular reason. I think this is what is used in the paper. I am not sure though. ADAM was not out back when this paper was published, SGD + Momentum, RMSProp were more common back then. Again I am not sure of this. But this is what I can recollect.


'{"result":"error","error":{"name":"Exception"}}'

I followed the instructions to the letter, everything went fine until “Google Apps Script” gave me this error message 'The script completed but did not return anything.' when I pressed “SEND” from my local website. The code run without any bug.
Normally, the information I put in, i.e. the email address and the name, should then enter the google sheet, except that it doesn't...

OS: Windows 11
Browser: Chrome

I have the same error... Not sure if this is a new Google thing...

actually there are answers available in other similar issues raised. Managed to get to work.

		<meta charset="utf-8" />
		<title>Test Form Gsheet</title>

Test Form Gsheet

		<form name="submit-to-google-sheet">
			<input
				name="email"
				type="email"
				placeholder="Email"
				required
			/>
			<input
				name="name"
				type="text"
				placeholder="Name"
				required
			/>
			<textarea
				name="message"
				type="text"
				placeholder="Name"
				required
			/></textarea>
			<button type="submit">Send</button>
		</form>

Use this code for the html form
set the script to run doPost not initialSetup
make sure you deploy to a new version
make sure const sheetName is the name as your sheet name
make sure your sheet has the column name for all submitted input in your form

in my example above I have email, name, message
my sheet has column email,name,message


chore: upgrade dataloader to v5

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   10.99%   10.99%
=======================================
  Files           8        8
  Lines         373      373
=======================================
  Hits           41       41
  Misses        332      332

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 f29d24d...aa01973. Read the comment docs.


Remove double to string convertion in FormData

These convertions introduced by 2 different PRs
https://github.com/facebook/react-native/commit/5061fde3173e6228d544b5df1ce8ce4e2a7dd2a4
https://github.com/facebook/react-native/commit/de392b5351494675220e0ef5176d8a8f3d473477

Make sure it works for booleans, etc... Not every non-string is an object. (And { ...value } crashes when value is a boolean.)

@olegbl would you mind taking a look at this pull request? It's been a while since the last commit was reviewed.

@olegbl @satya164
Do you mean cases like formData.append('foo', new Boolean(true))?
No, there is no crashes. Just value is empty in formData.getParts() cause no iteratable props in new Boolean().
I can rewrite this https://github.com/facebook/react-native/blob/bef175af61d387084fd9161e495cac3e1b414556/Libraries/Network/FormData.js#L79 to something like if (isPlainObject(value)) { ... }. But I think it's overhead.
In this PR i've just removed some duplicate code

value casting at line 69: value = '' + value and line 89: value: String(value)
checking value type at line 68: if (typeof value !== 'object') and line 79: if (typeof value === 'object')).

I've removed first conversion (value = '' + value) cause String(value) is more obviously in my opinion.

I meant formData.append('foo', true). Looks like it should be fine since "if (typeof value === 'object') {" got added some time back.

Yep, it should be fine and will be casted to foo=true in HTTP request

Can you rebase against master so that the tests pass?

i've rebased my branch

@doochik do you have any updates for this pull request? It's been a while so we wanted to check in and see if you've looked at the requested changes.

@facebook-github-bot shipit


Create boost_business

Add new dataset on the Boost Your Business program from Commerce

Sure thing, I think it was just the spacing on the description elements.


Twig go-to navigation does not support Zed twig templates

Tested on: PhpStorm 2022.2.1, Build #PS-222.3739.61
OS: Windows 10
Plugin version: 0.7.3

When testing go-to navigation via Twig for extends, widget and other twig tags for some kind of dependency, it seems they work for Yves only. When I have something in Zed twig template:
{% extends<EMAIL_ADDRESS>%}

...

{% embed<EMAIL_ADDRESS>%}

...

{% extends '@Spryker:ZedUi/Example/example.twig' %}

and try to use click action for go-to navigation, it fails with basic error Cannot find declaration to go to.
Based on docs, it's not written it does support only Yves, so in case it's also supposed to work with Zed it might be considered as a bug. If it's supposed to work on Yves only, then please raise it as a suggestion enhacement. It might be really useful to have go-to navigation working for Zed twig templates as well.

Hi @senor-dingdong,
thank you! Working perfectly! :)


fix(indexedDb): add cleanup function on db ready to remove old settings

:tada: This PR is included in version 1.0.0-alpha.63 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


SVM conic

06/svm-conic.ipynb does not run. The link to a "previous SVM model" in Chapter 4 is broken (I assume it's now in Chapter 5). Class LinearSVM is not defined (the one from Chapter 5 works). Function scatter4 is not defined, we use the following:
def scatter4(X_test, y_test, y_pred):
    fig, ax = plt.subplots(1, 2, figsize=(10, 4))
    scatter_labeled_data(
        X_test,
        y_test,
        labels=["genuine", "counterfeit"],
        ax=ax[0],
        title="Test Set",
    )
    scatter_labeled_data(
        X_test,
        y_pred,
        labels=["genuine", "counterfeit"],
        ax=ax[1],
        title="Prediction",
)

And a technical question. My Python installation is via apt. Can I install Mosek this way, or do I have to reinstall Python via Anaconda?

Thanks for opening this issue.  The link has been updated to reflect the final organization of chapters and notebook filenames. There was an obvious breakdown in my testing that allowed the missing utilities to get through. svm-conic should now be working for you.
I don't see why it wouldn't work with Python installed via apt, provided pip is installing things in a manner consistent with the installation.  You can add the location the executable in pyomo.SolverFactory if mosek is installed somewhere off the search path.


Remote desktop streaming

The remote desktop client can be implemented in Javascript inside the dashboard. VNC is preferred.

You mean stream the desktop ALVR is running on?

I didn't define the details. Ideally you would be able to toggle the monitors. Or let the VNC server create a new virtual monitor and stream that one. For the VNC client I thought of this library: https://novnc.com/info.html. That client should be compatible with any third party VNC server.

Do we really want this? It feels like scope creep to me and looking at the SteamVR desktop view works well enough for me.

I agree: I don't see why alvr would need to do more than providing a n openVR/openXR driver.

I also agree. This issue was in response to multiple feature requests we had in the past.

Yep, you can just use the steam version of VRDesktop if you need more than the default steam desktop view


How to use protected meilisearch instance?

Description
ISSUE_INDEXER_CONN_STR = "http://xxxx.7700"
when my meilisearch is set master key, gitea can't startup successfully.I want to set api key in app.ini，but i don't know where should i set api key.
meilisearch doc suggest request with auth like this:
curl
-X GET 'http://localhost:7700/keys'
-H 'Authorization: Bearer MASTER_KEY'
Unable to issueIndexer.Init with connection http://<IP_ADDRESS>:7700 Error: unaccepted status code found: 401 expected: [202], MeilisearchApiError Message: The Authorization header is missing. It must use the bearer authorization method., Code: missing_authorization_header, Type: auth, Link: https://docs.meilisearch.com/errors#missing_authorization_header (path "POST /indexes" with method "CreateIndex")
Gitea Version
1.20.2
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
run from docker
Database
MySQL

Under ISSUE_INDEXER_CONN_STR, you would have http://:apikey@localhost:7700 as it works with HTTP basic auth (note the : right before the API key in the URL`

Under ISSUE_INDEXER_CONN_STR, you would have http://:apikey@localhost:7700 as it works with HTTP basic auth (note the : right before the API key in the URL`

Thinks.


Better multi window resize animation

The multi window resize animation is very annoying, but on single window, it performs very well.
My hyprland.conf:
---SNIP---
animations {
    enabled = yes
    animation = windowsOut, 1, 7, default, popin 80%
    animation = border, 1, 10, default
    animation = borderangle, 1, 8, default
    bezier = overshot,0.13,0.99,0.29,1.1
    animation = windows,1,5,overshot,slide
    animation = windowsMove, 1, 6, overshot,
    animation = fade,1,10,default
    animation = workspaces,1,8,overshot,slidevert
}
---SNIP---
# Resize window with Alt + arrow keys
binde = ALT, left, resizeactive, -40 0
binde = ALT, right, resizeactive, 40 0
binde = ALT, up, resizeactive, 0 -40
binde = ALT, down, resizeactive, 0 40
---SNIP---

GIF:

It should be like this:

falls under misc:animate_manual_resizes, but shouldn't.

No use qwq

If this is master then I think it's been fixed.

nope, still the same as before
 System/Version info
Hyprland, built from branch  at commit 024d4ddc74c9bd7945c4075c972575f20bc5b9bd  ().
Date: 2024-03-08
Tag:

flags: (if any)

System Information:
System name: Linux
Node name: imxyy-nix
Release: 6.7.6-zen1
Version: #1-NixOS ZEN SMP PREEMPT_DYNAMIC Tue Jan  1 00:00:00 UTC 1980

GPU information:
03:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 14 [Radeon RX 5500/5500M / Pro 5500M] [1002:7340] (rev c5) (prog-if 00 [VGA controller])

os-release: ANSI_COLOR="1;34"
BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
BUILD_ID="24.05.20240306.9df3e30"
DOCUMENTATION_URL="https://nixos.org/learn.html"
HOME_URL="https://nixos.org/"
ID=nixos
IMAGE_ID=""
IMAGE_VERSION=""
LOGO="nix-snowflake"
NAME=NixOS
PRETTY_NAME="NixOS 24.05 (Uakari)"
SUPPORT_URL="https://nixos.org/community.html"
VERSION="24.05 (Uakari)"
VERSION_CODENAME=uakari
VERSION_ID="24.05"

plugins:

Did you set misc:animate_manual_resizes = true?

Did you set misc:animate_manual_resizes = true?

thanks, it works


adding docs and update python examples.

@vibhatha  please send this PR to https://github.com/cylondata/cylon/tree/0.2.0