Getting error on install

remote: bower handlebars              extra-resolution Unnecessary resolution: handlebars#>= 2.0.0 < 3.0.0
remote: -----> Caching bower_components directory for future builds
remote: -----> Building Ember CLI application production distribution
remote:        version: 0.1.3
remote:
remote: /tmp/build_c483dcfcb84f7a1c49e134c9576795bc/node_modules/ember-cli-bootstrap-sassy/index.js:34
remote: 			configMessage.push('some JS loaded [+'o.js.join(',')'+]');
remote:        Unexpected identifier
remote:        SyntaxError: Unexpected identifier

points to the o in o.js.join

made a new PR to fix some typos.

this should be fixed now


Add proper types to support .d.ts generation

Make sure all of the types match results.

Howdy,
I'd like to assist here, but I need some pointers on what I need to do to get them generated. I'm assuming we aren't creating the *.d.ts file by hand.
Thanks.

Awesome!
So we add types just like in TypeScript and we use those to generate typedocs from.  Then, when actually building, we strip those types so they don't linger around in the browser at runtime.
Here is an example of adding types to methods
Here is an example of adding typedoc comments to methods
You can test the output using gulp doc-generate from here

By the way this should go without saying but I'm going to say it anyway - this will be an awesome contribution and we appreciate the help a ton!

Hi. I want to do it but have a problem on start.
I have cloned the repo and when i am running gulp doc-generate i am getting some errors:
[11:41:41] C:/Projects/validatejs/dist/aurelia-validatejs.d.ts(3)  Cannot find module 'aurelia-pal'. [11:41:41] C:/Projects/validatejs/dist/aurelia-validatejs.d.ts(6)  Cannot find module 'aurelia-validation'. [11:41:41] C:/Projects/validatejs/dist/aurelia-validatejs.d.ts(9)  Cannot find module 'aurelia-dependency-injection'. [11:41:41] C:/Projects/validatejs/dist/aurelia-validatejs.d.ts(12)  Cannot find module 'aurelia-metadata'. [11:41:41] C:/Projects/validatejs/dist/aurelia-validatejs.d.ts(13)  Cannot find module 'validate.js'. [11:41:41] 'doc-generate' errored after 2.58 s [11:41:41] Error in plugin 'gulp-typedoc' Message:     Failed to generate load TypeDoc project.
What am i doing wrong?

I'm new to this, but would like to help. If just by stating my opinion and asking questions.
Just as issue #67 was being closed I was posting an update. But I would assume in this case a .d.ts file is already created specifically aurelia-validatejs.d.ts. The issue you are getting is because of error with that file, see my comments on #67. It also looks not to be specific to webpack.
To me the gulp doc-generate is not needed if the .d.ts file is already created. My question, now that aurelia-validatejs.d.ts is there what are the maintenance of the file moving forward? The idea of adding types to a .js file, generate docs, then remove them seems odd for maintaining the file. Would we just maintain the .d.ts file now? Is the gulp doc-generate suggestion a starting point? When maintaining other .d.ts files do you go through that exercises of adding types .js files and removing them?
As I pointed out in #67 is using the type any for all things a good solution?
I also understand that once this a .d.ts is built and is set in stone the signature of methods don't change that much and objects types are not added often. Therefore not too many changes to the .d.ts files are needed. Once you get it right from the start there should not be too many changes moving forward.

I have started a branch for adding docs and type hints here:
https://github.com/derkork/validatejs/tree/issue-10/src
I'm not really sure about the property observer (https://github.com/derkork/validatejs/blob/issue-10/src/property-observer.js) from the code this does return nothing, but other functions are calling this and they seem to expect some return type.
I'm also getting the documentation generation error that @da1z got, so I can't create the documentation here locally.

d.ts generation is implemented in the alpha
http://blog.durandal.io/2016/06/14/new-validation-alpha-is-here/

Closing since this has been setup.


I'm a fresher can you tell how to add propagation key??

https://docs.spring.io/spring-cloud-sleuth/docs/current/reference/html/project-features.html#features-baggage

Hii, I'm a fresher i saw your discussion in internet  regarding overriding
trace I'd for me also same problem.i don't know how to use propagation key
and insert it in  application.can you help me to overcome this problem.
On Thu, Nov 3, 2022, 2:46 PM Marcin Grzejszczak @.***>
wrote:

https://docs.spring.io/spring-cloud-sleuth/docs/current/reference/html/project-features.html#features-baggage
—
Reply to this email directly, view it on GitHub
https://github.com/spring-cloud/spring-cloud-sleuth/issues/2225#issuecomment-1301824169,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWBNO4YLJ7M3HEUZIA56KKLWGN7IHANCNFSM6AAAAAARV5GNZY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Launch of Metaquity parachain testnet on Rococo.

Rococo network slot request https://github.com/paritytech/subport/issues/603#issuecomment-1786416949


Add Test suites for Zcb sub extension from Code Size Reduction Extension (Zce)

Description
Code Size Reduction is the superset of the standard Compressed (C) extension. It consists of six sub-extensions:

Zca
Zcf
Zcd
Zcb
Zcmp
Zcmt

Out of these six sub-extensions zca, zcf, zcd contains the instructions which are part of the standard C extension. But zcb, zcmp, zcmt sub extensions add new instructions.
:bulb: Currently, this repository has test suites only for zca instructions.
Changes

This PR adds the test suites for zcb instructions, which is one of six sub-extensions of RISC-V Code Size Reduction.
Added a test macro that uses a single operand.

Related Pull Requests

RISC-V CTG: PR #64 adds the test template and cover group format (cgf) to generate the tests suites for zcb.
RISC-V Config: PR #128 adds the ISA strings for Code Size Reduction extension, which is used by RISCOF.

Related Issues
Issue #334 and #6
Ratified/Unratified Extensions

[x] Ratified

Reference Model Used

[x] Spike

:warning: Support of Code Size Reduction Extension (Zcb, Zcmp, Zcmt) is not yet added to SAIL.
Mandatory Checklist:

[x] All tests are compliant with the test-format spec present in this repo
[x] Ran the new tests on RISCOF with Spike as a reference model successfully?
[x] Ran the new tests on RISCOF in coverage mode
[x] Link to Google-Drive folder containing the new coverage reports.
[x] Link to PR in RISCV-ISAC from which the reports were generated.
[x] Changelog entry created with a minor patch

This needs the following checklist items completed:

Ran the new tests on RISCOF in coverage mode
Link to Google-Drive folder containing the new coverage reports.

I would love to use these tests on CORE-V Wally. It would be easiest when they are integrated into riscv-arch-test.  @Abdulwadoodd, do you have an anticipated schedule about completing the checklist items to finish the PR?

Hello prof. @davidharrishmc , generating coverage reports is a bit challenging for me due to the apparent lack of zcb support in ISAC's internal decoder. There were some alternatives highlighted in one of the sig-arch-test meetings. I will look to look into this and try to make it work by the end of the upcoming week.

Coverage reports for the zcb instruction test suites (32/64) are available here

@allenjbaum ,
With the coverage reports and the corresponding PR in RISCV ISAC by @YazanHussnain-10x, mandatory checklist has been completed and this PR is good to go (unless there is any change required).

hello @allenjbaum, pinging for a review on this PR

Thanks, @allenjbaum for approving the PR and pulling it in.
In addition to "manually updating the version number", it would have also required a "git rebase" to avoid the conflict and make a stable version release through CI. CHANGELOG.md isn't chronologically correct at the moment.

@Abdulwadoodd hello sir , I am trying to understand the working of zcmp extension ( specifically cm.push and cm.pop instruction) i am searching for the config(.cgf) and arch test files , it would be really helpful if you have them and share them
thank you

@Abdulwadoodd @allenjbaum @YazanHussnain-10x hello ,
I am trying to understand zcmp extension ( specifically cm.push and cm.pop instruction ) it would very helpful if you can share the config files and arch test files for the same
thank you


python311Packages.rns: 0.5.9 -> 0.6.0, python311Packages.nomadnet: 0.3.7 -> 0.3.8

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 257220 run on x86_64-linux 1

12 packages built:

python310Packages.lxmf
python310Packages.lxmf.dist
python310Packages.nomadnet
python310Packages.nomadnet.dist
python310Packages.rns
python310Packages.rns.dist
python311Packages.lxmf
python311Packages.lxmf.dist
python311Packages.nomadnet
python311Packages.nomadnet.dist
python311Packages.rns
python311Packages.rns.dist


FR: grpc_cli: channel selection flag

Today, there's an --enable_ssl flag to switch between insecure and secure. However, internal-to-Google, there's also a LOAS channel, and having mutually incompatible flags of --enable_ssl and enabling LOAS is less than ideal.
I'd like to amend --enable_ssl to be instead a --channel enum that selects from { insecure, tls }, and internal-to-google, there'd be also a loas. This also allows us to support alts or other alternatives in the future.
For maintaining existing behavior, the default for --channel would be insecure unless --enable_ssl. I'm not familiar with the deprecation policy for the CLI: would you warn that the flag will be removed at some point, or just never remove it?

This correlates to a portion of https://issuetracker.google.com/issues/111599303.

grpc_cli flags is not considered as our public API, yet I think we would like to support both for a while and maybe warn user first before finally removing it.
Please reference this issue in your future PRs. Thanks.

Fixed in #16204.


Integration with AIrsenal

Hi @anguswilliams91 , I finally got round to having a look at this today, and getting some score predictions with ExtendedDixonColesMatchPredictor was really smooth, thanks! 🎉
To get it working in AIrsenal there are 3 functions we use in bpl that are not in bpl-next currently (I think):

BPLModel.add_new_team
BPLModel.score_n_probability
BPLModel.concede_n_probability

I'm happy to help with adding these to bpl-next (to BaseMatchPredictor for the 2nd two?), but may need a hand with add_new_team. We can also implement them on the AIrsenal side instead if you'd prefer not to have them here?

Had a go at the low hanging fruit in #5


Same template for both logging.yml and log4j2.properties

Hi,
https://github.com/elastic/puppet-elasticsearch/blob/6d31554205305d4127acb8d31f4e640466c42f70/manifests/instance.pp#L244
https://github.com/elastic/puppet-elasticsearch/blob/6d31554205305d4127acb8d31f4e640466c42f70/manifests/instance.pp#L245
Is there any reason that only one template is allowed for both logging.yml and log4j2.properties ? When using default values, there are 2 templates (which is correct), but if you override the logging_template variable, it will replace both files, while one is YML, the other is ini like, and both have totally different content.

There's not an outstanding reason I can think of that they need to be that way, there should probably be a parameter to determine which logging file to use. logging.yml is still around in the module since 2.x isn't officially EOL yet, but I think the reasonable approach would be to keep generically-named parameters like logging_template but use parameters like logging_yml_ensure and log4j2_ensure and default to absent and file respectively to default to 5.x/6.x behavior.

It seems the correct combination of options is:
logging_template   => "${module_name}/elasticsearch/logging.yml.erb",
logging_yml_ensure => absent,

I don't see logging_yml_ensure param to be availble anywhere in module
How can control log4j content while not manage logging.yml ?
Thanks


EnableHttpCompression doesn't work correctly

NEST/Elasticsearch.Net version : 2.3.3
Elasticsearch version: 2.3.2
Description of the problem including expected versus actual behavior:
I'm trying to use the compression between Client and Server and getting the following Error:
"System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects".
I have used: "settings.EnableHttpCompression()"; at client and "http.compression:true" at server.
I'm using .net core rc1.
It seems to be a bug, or I'm making something wrong...

  Also, if you can paste the value of DebugInformation on the response here that would also be helpful.

Hi! Here is an Example of my code:
` [Route("testES")]
[HttpGet]
public string testES()
{
var host = "localhost";
var port = "9202";
var node = new Uri("http://" + host + ":" + port + "/indexname/");
var settings = new ConnectionSettings(node);
settings.DisableDirectStreaming();
settings.EnableHttpCompression();
ElasticClient client = new ElasticClient(settings);
        var request = new SearchRequest()
        {
            Query = new TermsQuery() {
                Field = "Test",
                Terms = new string[] { "276055540680082", "528528001853706" },
                Name = "Test"
            }
        };
        var response = client.Search<object>(request);
        return "";
    }`

And the DebugInformation:
`# FailureReason: Unexpected when trying to POST http://localhost:9202/indexname/_search

BadResponse: Node: http://localhost:9202/indexname/ Exception: InvalidOperationException Took: 00:00:00.3078855

Audit exception in step 0 BadResponse:
System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects.
at System.Net.Http.Headers.HttpHeaders.CheckHeaderName(String name)
at System.Net.Http.Headers.HttpHeaders.Add(String name, String value)
at Elasticsearch.Net.HttpConnection.CreateHttpRequestMessage(RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 155
at Elasticsearch.Net.HttpConnection.Request[TReturn](RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 84
at Elasticsearch.Net.RequestPipeline.CallElasticsearch[TReturn](RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Pipeline\RequestPipeline.cs:line 397
at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 68
Inner Exception: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects.
System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects.
at System.Net.Http.Headers.HttpHeaders.CheckHeaderName(String name)
at System.Net.Http.Headers.HttpHeaders.Add(String name, String value)
at Elasticsearch.Net.HttpConnection.CreateHttpRequestMessage(RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 155
at Elasticsearch.Net.HttpConnection.Request[TReturn](RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 84
at Elasticsearch.Net.RequestPipeline.CallElasticsearch[TReturn](RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Pipeline\RequestPipeline.cs:line 397
at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 68
Exception:
Elasticsearch.Net.UnexpectedElasticsearchClientException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects. ---> System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects.
at System.Net.Http.Headers.HttpHeaders.CheckHeaderName(String name)
at System.Net.Http.Headers.HttpHeaders.Add(String name, String value)
at Elasticsearch.Net.HttpConnection.CreateHttpRequestMessage(RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 155
at Elasticsearch.Net.HttpConnection.Request[TReturn](RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 84
at Elasticsearch.Net.RequestPipeline.CallElasticsearch[TReturn](RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Pipeline\RequestPipeline.cs:line 397
at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 68
--- End of inner exception stack trace ---
at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 88
at Elasticsearch.Net.ElasticLowLevelClient.DoRequest[T](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\ElasticLowLevelClient.cs:line 58    at Elasticsearch.Net.ElasticLowLevelClient.Search[T](PostData1 body, Func2 requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\ElasticLowLevelClient.Generated.cs:line 6778    at Nest.LowLevelDispatch.SearchDispatch[T](IRequest1 p, PostData1 body) in c:\code\elasticsearch-net\src\Nest\_Generated\_LowLevelDispatch.generated.cs:line 2575    at Nest.ElasticClient.<>c__DisplayClass486_02.b__0(ISearchRequest p, PostData1 d) in c:\code\elasticsearch-net\src\Nest\Search\Search\ElasticClient-Search.cs:line 73    at Nest.ElasticClient.Nest.IHighLevelToLowLevelDispatcher.Dispatch[TRequest,TQueryString,TResponse](TRequest request, Func3 responseGenerator, Func3 dispatch) in c:\code\elasticsearch-net\src\Nest\ElasticClient.cs:line 56    at Nest.ElasticClient.Nest.IHighLevelToLowLevelDispatcher.Dispatch[TRequest,TQueryString,TResponse](TRequest request, Func3 dispatch) in c:\code\elasticsearch-net\src\Nest\ElasticClient.cs:line 46
at Nest.ElasticClient.Search[T,TResult](ISearchRequest request) in c:\code\elasticsearch-net\src\Nest\Search\Search\ElasticClient-Search.cs:line 71
at Nest.ElasticClient.Search[T](ISearchRequest request) in c:\code\elasticsearch-net\src\Nest\Search\Search\ElasticClient-Search.cs:line 65
at TestProject.Controllers.WebApi.TestController.testES() in C:\Users\TestProject\Controllers\WebApi\TestController.cs:line 157
`

Hi! Here is an Example of my code:
[Route("testES")]
[HttpGet]
public string testES()
{
var host = "localhost";
var port = "9202";
var node = new Uri("http://" + host + ":" + port + "/indexname/");
var settings = new ConnectionSettings(node);
settings.DisableDirectStreaming();
settings.EnableHttpCompression();
ElasticClient client = new ElasticClient(settings);
        var request = new SearchRequest()
        {
            Query = new TermsQuery() {
                Field = "Test",
                Terms = new string[] { "276055540680082", "528528001853706" },
                Name = "Test"
            }
        };
        var response = client.Search<object>(request);
        return "";
    }

And the DebugInformation:
# FailureReason: Unexpected when trying to POST http://localhost:9202/indexname/_search
BadResponse: Node: http://localhost:9202/indexname/ Exception: InvalidOperationException Took: 00:00:00.3078855 # Audit exception in step 0 BadResponse: System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects. at System.Net.Http.Headers.HttpHeaders.CheckHeaderName(String name) at System.Net.Http.Headers.HttpHeaders.Add(String name, String value) at Elasticsearch.Net.HttpConnection.CreateHttpRequestMessage(RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 155 at Elasticsearch.Net.HttpConnection.RequestTReturn in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 84 at Elasticsearch.Net.RequestPipeline.CallElasticsearchTReturn in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Pipeline\RequestPipeline.cs:line 397 at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 68 # Inner Exception: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects. System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects. at System.Net.Http.Headers.HttpHeaders.CheckHeaderName(String name) at System.Net.Http.Headers.HttpHeaders.Add(String name, String value) at Elasticsearch.Net.HttpConnection.CreateHttpRequestMessage(RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 155 at Elasticsearch.Net.HttpConnection.RequestTReturn in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 84 at Elasticsearch.Net.RequestPipeline.CallElasticsearchTReturn in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Pipeline\RequestPipeline.cs:line 397 at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 68 # Exception: Elasticsearch.Net.UnexpectedElasticsearchClientException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects. ---> System.InvalidOperationException: Misused header name. Make sure request headers are used with HttpRequestMessage, response headers with HttpResponseMessage, and content headers with HttpContent objects. at System.Net.Http.Headers.HttpHeaders.CheckHeaderName(String name) at System.Net.Http.Headers.HttpHeaders.Add(String name, String value) at Elasticsearch.Net.HttpConnection.CreateHttpRequestMessage(RequestData requestData) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 155 at Elasticsearch.Net.HttpConnection.RequestTReturn in c:\code\elasticsearch-net\src\Elasticsearch.Net\Connection\HttpConnection-CoreFx.cs:line 84 at Elasticsearch.Net.RequestPipeline.CallElasticsearchTReturn in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Pipeline\RequestPipeline.cs:line 397 at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 68 --- End of inner exception stack trace --- at Elasticsearch.Net.Transport1.Request[TReturn](HttpMethod method, String path, PostData1 data, IRequestParameters requestParameters) in c:\code\elasticsearch-net\src\Elasticsearch.Net\Transport\Transport.cs:line 88 at Elasticsearch.Net.ElasticLowLevelClient.DoRequestT in c:\code\elasticsearch-net\src\Elasticsearch.Net\ElasticLowLevelClient.cs:line 58 at Elasticsearch.Net.ElasticLowLevelClient.SearchT in c:\code\elasticsearch-net\src\Elasticsearch.Net\ElasticLowLevelClient.Generated.cs:line 6778 at Nest.LowLevelDispatch.SearchDispatchT in c:\code\elasticsearch-net\src\Nest_Generated_LowLevelDispatch.generated.cs:line 2575 at Nest.ElasticClient.<>c__DisplayClass486_02.<Search>b__0(ISearchRequest p, PostData1 d) in c:\code\elasticsearch-net\src\Nest\Search\Search\ElasticClient-Search.cs:line 73 at Nest.ElasticClient.Nest.IHighLevelToLowLevelDispatcher.DispatchTRequest,TQueryString,TResponse in c:\code\elasticsearch-net\src\Nest\ElasticClient.cs:line 56 at Nest.ElasticClient.Nest.IHighLevelToLowLevelDispatcher.DispatchTRequest,TQueryString,TResponse in c:\code\elasticsearch-net\src\Nest\ElasticClient.cs:line 46 at Nest.ElasticClient.SearchT,TResult in c:\code\elasticsearch-net\src\Nest\Search\Search\ElasticClient-Search.cs:line 71 at Nest.ElasticClient.SearchT in c:\code\elasticsearch-net\src\Nest\Search\Search\ElasticClient-Search.cs:line 65 at TestProject.Controllers.WebApi.TestController.testES() in C:\Users\TestProject\Controllers\WebApi\TestController.cs:line 157

And here is the content of elasticsearch.yml :
cluster.name: cluster1
http.port: 9202
transport.tcp.port: 9302
http.compression: true


Sometimes fails to find latest release when using Manifest.fromConfig

If you are using Manifest.fromConfig (e.g. not using a manifest file in the repository), we can fail to find the correct latest release.
This could be because the release was manually tagged and doesn't match with what release-please expects.
See https://github.com/googleapis/release-please/issues/1259#issuecomment-1022603167

@chingor13  this is also related https://github.com/google-github-actions/release-please-action/issues/400


TODO: Add .TakeAction

Add Action for autoassign.
Thanks to @blackgirlbytes ❤

https://github.com/marketplace/actions/contributor-takes-action
Thanks to @bdougie for building this action!

.TakeAction

.take

.take

I want to .take this issue

.take

.take

I want to .take this issue

.take

I want to .take this issues please.

I want to .take this issue please.

I want to .take this issue please.

.take

I want to .take this issue

Can I .take this now?

Now can we .take action

Maybe I can .take it?

Try to .take it again

.take

.take

yay it works!


elfet/php-ssh Not found

Hi,
Please fix composer.json require section in the Deployer 2. It use old repo "elfet/php-ssh", that already droped.

Yeah, sorry about that.

Fixed.

Hi,
It looks like there is still some issue with this. After i updated composer requirements to "deployphp/deployer": "2.0.4"
it shows this error:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - deployer/deployer v2.0.4 requires phpseclib/phpseclib 2.0.*@dev -> satisfiable by phpseclib/phpseclib[2.0.x-dev].
    - deployer/deployer v2.0.4 requires phpseclib/phpseclib 2.0.*@dev -> satisfiable by phpseclib/phpseclib[2.0.x-dev].
    - Removal request for phpseclib/phpseclib == 2.0.9999999.9999999-dev
    - Installation request for deployer/deployer 2.0.4 -> satisfiable by deployer/deployer[v2.0.4].

When i put this in my composer.json it works fine:
"phpseclib/phpseclib": "2.0.*@dev"
Is this requirement to explicitly put phpseclib dependency to composer?

Thus is issue with phpseclib and packagist. I already created issue there.

4 ôåâð. 2015 ã., â 17:29, Tomas Majer<EMAIL_ADDRESS>íàïèñàë(à):
Hi,
It looks like there is still some issue with this. After i updated composer requirements to "deployphp/deployer": "2.0.4"
it shows this error:
Your requirements could not be resolved to an installable set of packages.
Problem 1
- deployer/deployer v2.0.4 requires phpseclib/phpseclib 2.0.@dev -> satisfiable by phpseclib/phpseclib[2.0.x-dev].
- deployer/deployer v2.0.4 requires phpseclib/phpseclib 2.0.@dev -> satisfiable by phpseclib/phpseclib[2.0.x-dev].
- Removal request for phpseclib/phpseclib == 2.0.9999999.9999999-dev
- Installation request for deployer/deployer 2.0.4 -> satisfiable by deployer/deployer[v2.0.4].
When i put this in my composer.json it works fine:
"phpseclib/phpseclib": "2.0.*@dev"
Is this requirement to explicitly put phpseclib dependency to composer?
—
Reply to this email directly or view it on GitHub.

Fixed.

I'm having the same problem whatever version I try to require

Yeeahh. Fix is coming....

:)

I know how to hotfix...

Will be fixed in #182


Issue with parsing URLS

Testing the tokenizer on Bootstrap, and ran into an issue tokenizing the following CSS
@font-face {
  font-family: "Glyphicons Halflings";
  src: url(../fonts/glyphicons-halflings-regular.eot);
  src: url(../fonts/glyphicons-halflings-regular.eot?#iefix)
      format("embedded-opentype"),
    url(../fonts/glyphicons-halflings-regular.woff2) format("woff2"),
    url(../fonts/glyphicons-halflings-regular.woff) format("woff"),
    url(../fonts/glyphicons-halflings-regular.ttf) format("truetype"),
    url(../fonts/glyphicons-halflings-regular.svg#glyphicons_halflingsregular)
      format("svg");
}

Specifically, the issue comes when parsing the section url(../fonts/glyphicons-halflings-regular.eot);, the code loops in the Bad URL remnant recovery process. I assume this means the ) token is being obscured somewhere

Fixed in 70b31c7037711b46a57862699c3ad174afef082c


[nrf fromtree] add GRTC driver test

Migrate GRTC driver test from Zephyr upstream to verify it in NCS context for nRF54L and nRF54H

Twister tests are passing: https://jenkins-ncs.nordicsemi.no/job/latest/job/test-fw-twister-ncs-zephyr/job/customizable/job/main/273/artifact/outcomes/results/sanitycheck_reports/part_1/twister_report.xml


Improve documentation for our templates (mainly readme.md)

Documentation should contain basic information about what is the picked technology: express.
What is the GraphQL technology. Apollo etc. What tools are required before development (codegen, graphback generate) and how they work into complete development workflow.
Current readme has this info but it needs to be extended.

/cc @craicoverflow, @machi1990
Automatically generated comment to notify maintainers

Hi! I'm new to this repo and would like to work on this issue. Can you point me in the right direction? Apart from the mentioned stuff in the issue what all should I include in the README?

@RinkiyaKeDad Hi. Thanks for reaching out.
Our repo templates contains readme:
https://github.com/aerogear/graphback/tree/master/templates
Readme needs to have more information about what technologies are used for testing, code generation etc. Similairly to https://github.com/facebook/create-react-app

Thanks for responding @wtrocki. As far as I understand the READMEs in all four of the folders need to updated. I'll be starting with the 'ts-react-apollo-client' one first. I'm a bit new to this so would like to confirm that I'm to fork the repo, make the changes in the readme and create a pull request right?

Hey @RinkiyaKeDad! Thanks for offering to help us out! Have you used Graphback before? I would first suggest playing around with Graphback and using the templates to get an understanding of how they work.
Yes, you fork the repo, create a new branch and create a PR!

Hi @craicoverflow! I thought of using Graphback but hesitated since I have never used TypeScript before. But I'll follow your suggestion and try again in order to get a better understanding before I begin updating the README.

Have you used JavaScript before?

Yes I have.

That's great - to use the templates JavaScript knowledge should be sufficient - TypeScript is a superset of JavaScript so they are very similar.

What tools are required before development (codegen, graphback generate) and how they work into complete development workflow.

I have added a section of dependencies in the README where I've briefed about GraphQL, Apollo Client and Material UI. But I'm not quite sure what this line means? I see that there is a generate script in the package.json,
 "generate": "graphback generate && graphql codegen"
So from what I understand till now  graphback generate is to regenerate the files after some changes have been made. But what is codegen and graphql codegen?

So from what I understand till now graphback generate is to regenerate the files after some changes have been made. But what is codegen and graphql codegen?

Amazing questions! Absolutely love that you have reached out and helped us as this really shows that.. in fact we need better README.md
For graphback generate see:
https://graphback.dev/docs/cli/graphback-cli
For codegen generate see:
https://graphql-code-generator.com/

Thanks a lot @wtrocki! Have gone through them a created a PR for the README of the React Template. Please suggest me what more can I imporve in the README.

Hey before I start working on the other READMEs, I wanted to ask that if something is similar across the templates is it fine if I use the same descriptions or is it prefered to re-write it? Like if lets say another template also uses React then can I copy over what I wrote for React in the previous README?

You can likely reuse some bits of information from the React template around Graphback, but since all the other templates are backend templates, the content will be pretty different overall.

Got it!

Was going through the mongodb template files. yarn generate there too has the same funtionality as far as I could understand, right?

Yep. It was shortcut to not explain complexities of using 2 tools.

So should I use the same explanation?

Yes you can. The best thing to do is create a PR and we can review whether it needs to be adjusted from there.

Fixed by @RinkiyaKeDad - amazing work!

Thanks @RinkiyaKeDad for the amazing contributions. Looking forward to many more from you <3

Thanks :) Will surely be contributing more.

That's great to hear! :+1:


recaptcha v3 not supported

Hello Joeph,
The package does not work with recaptcha v.3
I get this error.
ERROR for site owner: Invalid key type
recaptcha v.2 is alright though.
How can I fix this?

Sorry for the long time between comments. I'm leveraging someone else's library for recaptcha and I'm no longer maintaining this project so I'm not sure why v3 isn't working.
https://github.com/josephspurrier/gowebapp/blob/master/vendor/app/shared/recaptcha/recaptcha.go#L7

Closing - old issue.


parent is null in JQLite code

Sentry is reporting an error in our app in Angular's source code for JQLite:

The line in question is https://github.com/angular/angular.js/blob/master/src/jqLite.js#L986
It appears under some circumstances parent can be null, in which case this error pops up. It's probably a good idea to guard against unexpected values of parent so this error doesn't occur.
This was seen happening on Angular 1.5.8, on Firefox v26 on Windows XP.

There are many circumstances under which parent can be null. The question is who/what is calling after() on an element without a parent.
That said, jQuery seems to silently ignore a missing parent, so it might be a good idea to have the same behavior in jqLite. Thoughts, @mgol?

I am somehow torn on this issue. On one side, not checking for the parent in after is bad. On the other side, silently ignoring might cause some difficult to debug issues.

This is the jQuery implementation: https://github.com/jquery/jquery/blob/3.1.1/src/manipulation.js#L365-L371:
	after: function() {
		return domManip( this, arguments, function( elem ) {
			if ( this.parentNode ) {
				this.parentNode.insertBefore( elem, this.nextSibling );
			}
		} );
	},

It's specifically guarded and documented:

Prior to jQuery 1.9, .after() would attempt to add or change nodes in the current jQuery set if the first node in the set was not connected to a document, and in those cases return a new jQuery set rather than the original set. The method might or might not have returned a new result depending on the number or connectedness of its arguments! As of jQuery 1.9, .after(), .before(), and .replaceWith() always return the original unmodified set. Attempting to use these methods on a node without a parent has no effect—that is, neither the set nor the nodes it contains are changed.

jqLite should primarily align with jQuery where it doesn't introduce a lot of code so it seems a good idea to fix it.
Generally, I think a good rule if you think some behavior is undesired in jqLite and that behavior is shared with jQuery is to report an issue to the jQuery bug tracker and make the change there first.

Agree with @mgol that we should align with jQuery. That said, when the node
has no parent, should this be logged?
On Mon, Oct 31, 2016 at 1:35 PM, Michał Gołębiowski <
<EMAIL_ADDRESS>wrote:

This is the jQuery implementation: https://github.com/jquery/
jquery/blob/3.1.1/src/manipulation.js#L365-L371:
after: function() {
    return domManip( this, arguments, function( elem ) {
        if ( this.parentNode ) {
            this.parentNode.insertBefore( elem, this.nextSibling );
        }
    } );
},

It's specifically guarded and documented:
Prior to jQuery 1.9, .after() would attempt to add or change nodes in the
current jQuery set if the first node in the set was not connected to a
document, and in those cases return a new jQuery set rather than the
original set. The method might or might not have returned a new result
depending on the number or connectedness of its arguments! As of jQuery
1.9, .after(), .before(), and .replaceWith() always return the original
unmodified set. Attempting to use these methods on a node without a parent
has no effect—that is, neither the set nor the nodes it contains are
changed.
jqLite should primarily align with jQuery where it doesn't introduce a lot
of code so it seems a good idea to fix it.
Generally, I think a good rule if you think some behavior is undesired in
jqLite and that behavior is shared with jQuery is to report an issue to the
jQuery bug tracker and make the change there first.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular.js/issues/15331#issuecomment-257282477,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAX44ukbD8cFJXPvFp1fo03Q6gSO8LTSks5q5eCogaJpZM4Kj_B_
.

If we decide to silently not-do-anything (as jQuery does), then we shouldn't log imo.

I agree with @gkalpak. This seems a non-breaking feature to me so we can land it whenever we want. @adambuczynski would you like to try to create a patch?

@mgol yes sure, will do


force web viewer to render all pages

Is there a way to force the web viewer to render all pages? Is there a setting? If not, can you give a pointer to the section of code I need to change?
Use Case
I want to show all search matches in the scrollbar after the PDF loads similar to how Chrome's CTRL-F works. Right now I use JQuery to find all the matches by CSS class, get positions within document, and translate the positions to the scrollbar. I draw a custom scrollbar too.
Problem
pdf.js only renders a few pages after the initial load so I cannot get the position of all matches. The user would need to scroll to the bottom in order to render all pages.

You would have to do something similar as in the SVG viewer: https://github.com/mozilla/pdf.js/blob/master/examples/svgviewer/viewer.js#L13.

@timvandermeij Okay thanks. Is it possible to render the text layer only for all pages? That would be a good middle ground, but will the text line up with the view layer correctly? In other words, are the two layers completely independent?

I believe that at the moment you have to render the page along with the text layer. You may be able to render only the text layer, but that's out of scope for the PDF.js project and you would have to implement that yourself. For example, use the text extraction example at https://github.com/mozilla/pdf.js/blob/master/examples/node/getinfo.js#L49 to obtain just the text layer.

Maybe it can help : http://stackoverflow.com/questions/31185785/pdf-js-render-pdf-with-scrollbar/41851102#41851102


How to use socket?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
How to use socket?

Could you please provide more details? Are You talking about network sockets? Currently, the engine has no network functionality out of the box. Please specify the code approach in your project. C++ or Angel Script?

你能提供更多细节吗？你在谈论网络套接字吗？目前，该引擎没有开箱即用的网络功能。请在您的项目中指定代码方法。C++ 还是天使脚本？

thanks

I have a branch with the started implementation of network functionality. But nobody requested this until this moment. I believe I need to return to this.


Hugging face

Which task should I use in a pipeline to get same results as in the paper?

Hi Tijana, if you want to reproduce most things you would have to run it in this order:

bash scripts/run.all.sh
bash scripts/run.predict.all.sh
bash scripts/run.evaluate.sh

Note that the results might differ slightly as we didn't release the big subset, as it is proprietary data.


Graceful handling of nested resolver invocations for larger data sets

Summary
https://newrelic.slack.com/archives/G24GM9VRR/p1599855383012600
Setups with nested resolvers can result in resolvers being invoked per item returned at the highest level resolver. For larger datasets, that can result in a ton of data.
At a high level, here are some of the issues that may cause:

Bloat the amount of data we capture per transaction
Crowd the UI for transaction traces and DT traces
Result in high number of broken traces in DT (we only allow 1k spans per minute)
Other?

After doing some initial research, it seems likely the best approach would be to do some sort of merge of the segments/spans while keeping metrics unique. That being said, there's awkwardness there with starting/ending if there's any sort of repeat.
Other options possible to help with this issue or others, depending on what customer needs come up:

Filter out list items: do not capture field resolution for items that are a part of a list

less ideal, seems like you'd filter out a lot of useful sub-resolvers but depends on the setup

Filter out paths including wildcards. For example: libraries.books.*

Could leverage identical logic to attribute filtering (flatten rules to regex, etc.)
High level of flexibility
As a primarily solution, could require a ton of configuration. Might not be valuable if merge or similar solution in place

Once beyond X count group altogether in one span/segment for remainder of list resolver?

Avoids awkward start/stop overlap issue
Allows for seeing some fine-grained details for items ultimately 'merged'.
Possibility of losing fine-grained metric counts unless solve for another way (maybe a way to generate individual metrics when that massive span/segment completes?)

Other?

Will want to consider impacts to metrics to still allow for best possible tracking down of issues on a per-field resolver basis.

Different than the intention of this story but perhaps we want to consider all these things together...
There doesn't seem to be a perfectly obvious way to avoid non-scalar fields but it looks like we can filter out where info.returnType.typeOf does not have the astNode property but does have the name property. Debatable whether or not you'd want to do this automatically or configurable. I don't believe apollo's tracing plugin (that sets the extensions) does any sort of filtering.
Similarly, we could filter down to only top-level resolvers by checking for a parent type name of Query/Mutation/whatvs. You'd prob only want to do this via configuration as you'd potentially hide some critical details.

We ended up with duplicates. The other (https://github.com/newrelic/node-newrelic/issues/576) is on the roadmap so closing this one.


Spring Data Cosmos 3.0.0.M1 is not compatible with Spring Boot 2.1.X

spring-data-cosmosdb 3.0.0.M1 is not compatible with Spring Boot 2.1.X (it seems spring-data-commons 2.1.X is the real culprit).
You can refer to #302 , as the error is the same:
Caused by: java.lang.IllegalStateException: You have defined query method in the repository but you don't have any query lookup strategy defined. The infrastructure apparently does not support query methods!
        at org.springframework.data.repository.core.support.RepositoryFactorySupport$QueryExecutorMethodInterceptor.<init>(RepositoryFactorySupport.java:545) ~[spring-data-commons-2.1.2.RELEASE.jar!/:2.1.2.RELEASE]
        at org.springframework.data.repository.core.support.RepositoryFactorySupport.getRepository(RepositoryFactorySupport.java:324) ~[spring-data-commons-2.1.2.RELEASE.jar!/:2.1.2.RELEASE]
        at org.springframework.data.repository.core.support.RepositoryFactoryBeanSupport.lambda$afterPropertiesSet$5(RepositoryFactoryBeanSupport.java:297) ~[spring-data-commons-2.1.2.RELEASE.jar!/:2.1.2.RELEASE]
        at org.springframework.data.util.Lazy.getNullable(Lazy.java:211) ~[spring-data-commons-2.1.2.RELEASE.jar!/:2.1.2.RELEASE]
        at org.springframework.data.util.Lazy.get(Lazy.java:94) ~[spring-data-commons-2.1.2.RELEASE.jar!/:2.1.2.RELEASE]

This issue has been closed for spring-data-cosmosdb 2.1.X but is still present in 3.0.0.M1.
PS. I have noticed that reactive implementation has been added with #394 on master, does this mean that 3.0.0.X is currently dead and 2.1.3 is supposed to provide the reactive capabilities for spring data cosmosDb repositories?

Do not use 3.0.0.M1 - we're aligning our versions to match the rest of Spring Boot ecosystem.


compare: fix typo

This PR rework a bit silx compare to fix some problems related to RGB images.
Now it supports at least Y,X,RGB using uint8. Other encodage are not properly supported.

fabio is not reading images as RGB, now PIL is used for few file extensons (jpg, png) (for convenience, that's not our main use case anyway)
The DnD was not working, maybe the implementation depends on PyQt version, now the implementation is stronger
Save/restore y axis orientation
Added some HDF5 autogenerated samples for images (intensity, RGBA)
Plus few fixes and typo

Changelog:

Added silx compare support of uint8 dataset as Y,X,RGB layout

This PR was fixed, it's ok to be reviewed.


[22][SR-1421] Starter task to improve sourcekitd-test output

SR-1421: Document the sourcekitd-test tool, which is used to quickly experiment with SourceKit. This task will involve writing a small amount of C++. You'll learn about LLVM's command line interface library and its argument parser, as well as about Swift's SourceKit.

thanks @modocache ! 🙇 😄 🎉


'live' event being called too soon for Flash fallback when Security window opens

I am getting Webcam.on('live') being called back too soon. (Before the user has the chance to click Allow/Deny on the flash popup for security/privacy.)
I believe this has to do with this line in Webcam.as
https://github.com/jhuckaby/webcamjs/blob/d46749d8a5b8237059f0a356a7f357d479e9e16b/flash/Webcam.as#L64
if (flashvars.new_user) {
   Security.showSettings( SecurityPanel.PRIVACY );
}

I think maybe the Security.showSettings above is asynchronous and being returned immediately? (Because a few lines further down are these bits which eventually trigger the Webcam.on('live')
ExternalInterface.call('Webcam.flashNotify', 'flashLoadComplete', true);

Here is my own JS code:
    Webcam.on('live', function() {
       setTimeout(function() {
         \$j('#loading_message').hide();
         \$j("#TakeSnapshot").show(); //A camera button
         \$j(".crop_markers").show();
      }, 5000);
    });

I added the setTimeout in there to see that it actually was getting fired. (I see the crop_markers appear 5 seconds after things load up, even though its waiting for the user to click Accept/Deny for flash privacy.

I can offer this really nasty/complex bit of code I found. I wish there was just a callback from the Security.showSettings! (or at least Adobe should explicitly state if its asynchronous or not!)
https://gist.github.com/Kalinovych/774b092d5230052b41d9

I found this. The first response seems simpler than the gist I previously pasted.
http://stackoverflow.com/questions/2510311/showsettings-callback-in-flex
var mic:Microphone = Microphone.getMicrophone();
mic.addEventListener(StatusEvent.STATUS, onMicStatus);

Looking at Adobe's reference implementation: http://help.adobe.com/en_US/as3/dev/WSfffb011ac560372f3fa68e8912e3ab6b8cb-8000.html#WS5b3ccc516d4fbf351e63e3d118a9b90204-7d37
It seems the Webcam.as is very similar.
My next question is: if the permission settings are checked to always allow, does handleCameraStatus get called?
(Because if it is called, then maybe the 'send "live" event to JS' should go in there? After they have allowed camera access?)

Working on a version that's closer to Adobe's reference implementation:
https://github.com/linagee/webcamjs/blob/master/flash/Webcam.as
(Untested as of this post. Will try to compile and test.)


Rewrite the descriptions of some monitors

Development Task
There are some descriptions of monitors written in Chinese and perhaps we should change it to English. e.g.

I am willing to take this task up.

/assign


Host the plugin on jenkins-ci.org

Nobody wants to build the plugin on their own, everybody wants to simply install it with Jenkins' "Manage Plugins". :)
https://wiki.jenkins-ci.org/display/JENKINS/Hosting+Plugins

I think so, too :smile:  I will send hosting request in this weekend.

Any update on this?

Updates? I don't really fancy installing plugins from any other place.

Hello everyone,
add fork and add hpi file on my repo:
(https://github.com/sisko59/jenkins-favicon-plugin/blob/master/bin/favicon.hpi)

@sisko59 I think you should ask on IRC #jenkins-dev to fork this plugin inside jenkinsci organization as this will also enable you to use CI to build and test it.


[Feature] List of supported optimizers

Problem (Why?)
move load_optimizers to proper place
Solution (What/How?)

[x] show supported optimizers
[x] move optimizers.py -> load_optimizers to __init__.py -> load_optimizer
[x] refactor load_optimizer

Other changes (bug fixes, small refactors)

[x] rename CI/CD pipelines
[x] pylint score to 10.0

Notes
version to 1.1.1

Codecov Report

Merging #60 (a9557a9) into main (c6e3159) will decrease coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   95.99%   95.86%   -0.14%
==========================================
  Files          29       28       -1
  Lines        2095     2029      -66
==========================================
- Hits         2011     1945      -66
  Misses         84       84

Impacted Files
Coverage Δ

pytorch_optimizer/__init__.py
100.00% <100.00%> (ø)

pytorch_optimizer/fp16.py
69.01% <100.00%> (ø)

pytorch_optimizer/lookahead.py
94.52% <100.00%> (-0.08%)
:arrow_down:

pytorch_optimizer/sam.py
94.44% <0.00%> (-0.11%)
:arrow_down:

pytorch_optimizer/base_optimizer.py
97.33% <0.00%> (-0.07%)
:arrow_down:

pytorch_optimizer/nero.py
96.66% <0.00%> (-0.06%)
:arrow_down:

pytorch_optimizer/diffrgrad.py
95.74% <0.00%> (-0.05%)
:arrow_down:

pytorch_optimizer/radam.py
96.51% <0.00%> (-0.05%)
:arrow_down:

pytorch_optimizer/adabelief.py
96.22% <0.00%> (-0.04%)
:arrow_down:

... and 14 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 c6e3159...a9557a9. Read the comment docs.


Small update to README, routes attr to "push_routes"

I think its current implementation, only the route is expected: https://github.com/xhost-cookbooks/openvpn/blob/master/templates/default/server.conf.erb#L28.
override_attributes(
  "openvpn" => {
    "push_routes => [
      "<IP_ADDRESS> <IP_ADDRESS>'"
    ]
  }
)

@senilegenius could you test that out perhaps?
In my dna.json:
 "openvpn": {
   "type":"server",
   "config": {
     "dev":"tun",
     "status":"/var/log/openvpn-status.log"
   },
   "push_routes": [ "<IP_ADDRESS> <IP_ADDRESS>" ]
 }

@flaccid  Indeed. That's what I get for submitting a PR at 3am. Thanks for the feedback.

Fixed via https://github.com/xhost-cookbooks/openvpn/commit/b466847f96fef635f12f713545d2e419997d9684.


runReaderT has the wrong type

Taking the mtl library, using the --hoogle flag, I generates:
newtype ReaderT r (m :: * -> *) a :: * -> (* -> *) -> * -> *
[ReaderT] :: (r -> m a) -> ReaderT r a
[runReaderT] :: ReaderT r a -> r -> m a
runReaderT :: ReaderT r m a -> r -> m a

Observe that the first runReaderT (the one in brackets) is missing the m field from the ReaderT type. Originally reported at https://code.google.com/p/ndmitchell/issues/detail?id=557

I'm not sure what fixed this, but this doesn't appear to be the case anymore. I'll add a regression test.

What does the [runReaderT] mean? Why does it occur twice in the reexport?

Square brackets means that an entity is not exported directly, but rather indirectly through a catch-all export like ReaderT(..). In the test case I added, I export both ReaderT(..) and runReaderT to ensure that both variants would show the correct type signature.
It would be good to actually document this—I'm about to file an issue regarding this.

See https://github.com/haskell/haddock/issues/807.


make rmi messages more compatible with docker

in the case where we rmi an image that has only one reponame, we print
out an untagged reponame message.
$ sudo podman rmi busybox
Untagged: docker.io/library/busybox:latest
Deleted: db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb
Signed-off-by: baude<EMAIL_ADDRESS>

Fixes: #2878

LGTM

Re-ran failed tests

/lgtm
/hold

/lgtm

/lgtm
/hold

Restoring lgtm
/lgtm

/hold cancel


Autowin for radiant team again

It happaned when one of dire team had been disconected during the voting time.
Then he abandoned the game during wave 1 and radiant team got autowin.

fixed


[WIP] Algolia site search

[description]

implement site search with Algolia
implement versioned search based on site cookie

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e3af1a35d7514d9d0f2b6fa5503852909fcca697/

cc: @jseldess @kuanluo @lhirata @Amruta-Ranade @rmloveland
This PR is an MVP of site search with Algolia.  Algolia provides a lot of nice functionality that I would like to explore adding for this initial version.  I'll reach out to the cc'd above for your suggestions early next week.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/8756b8ef4cdedb5e4d5af7106b826774f7c58025/

@j-low, I'm running your branch locally. The search is fast, and it seems to return only content for the currently selected version, which is great.
I'm seeing one main issue right now: When searching a term, the same page is represented multiple times in the search result. Here's an extreme example:

Also, a few question:

What is our plan for the quick results (that display before hitting Enter)?
Are we returning all results or only one page worth? If the latter, what's the 1-page limit?

@jseldess thanks for noting that.  What's happening there is that the index needs to be configured by to ignore the anchored links.  I tried a few options on the front end to fix that but it seems that since the index was compiled by Algolia some other tweak needs to be made.  I've been in contact with their dev as of last Thursday about resolving this particular issue.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/114ee997eb400fb86949549189a46436cd798e50/

Looking good, @j-low. I'm still seeing See Also texts in the results. I know you're working on excluding them. But now direct page hits have a better summary.
A nice-to-have nit: Can we bold the search term(s) in the results summaries?
Post-2.0, it might be nice to replicate what Google does, where the link is to the page but the summary starts with a "Jump to XXX", if there's a section to jump to.

@jseldess
re: "See Also" text: that config has been PR'd but not yet merged so this is expected.
re: highlight search terms: it looks like they provide a nice way to handle that so, yes, this can be worked in for this iteration.
re: "Jump to ... ": this would definitely be nice.  I don't have a clear idea how to do this right off the top of my head but definitely worth exploring.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/63414c801603e9fd0462e4baed345a6a14518645/

Seeing something a bit strange now. Not sure if it was happening before. Some result summaries are only the search keyword, even though on the page the term shows up in the context of a sentence/paragraph.

For example, if you click the first link, UPSERT, you're taking to an example: https://www.cockroachlabs.com/docs/stable/upsert.html?__hstc=181257784.62e02c0a517f58508c195cf17fdd48cc.1489812728556.1521644926081.1521652640628.522&__hssc=181257784.11.1521652640628&__hsfp=1496384207#upsert-that-fails-conflict-on-non-primay-key
The term doesn't actually appear anywhere in that section, but it does appear in the See Also just below. It also appears in a flag description further up the page. So I'm not sure what's happening here.

@jseldess it looks like two separate issues.  I'm debugging them currently and should have some more info by later today.
For reference, here is the PR thread where I am discussing the issue with Algolia: https://github.com/algolia/docsearch-configs/pull/349#issuecomment-374977212

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/8d636657a7bd6c5c617fbfc8c51057f3c42948ff/

LGTM, @j-low. I think this is good to merge, given that we'll continue making improvements. Thanks for all your work!


Make an extra Repo for the Python Version.

Make an extra Repo for the Python Version.
called e.g. "Roblox-assets-extractor-legacy" or "*-python"

Good idea, will do that after merging rust-write into main

Actually, thinking about this more, the python versions pull from the main branch in this repo. It is impossible to change something without things breaking


Master

BabylonJS Toolkit - Support Plugins Source Projects

Lookd like a LOT of files, is it correct? do we need all sources files for dependencies?

Its All the Source For FreeImage... I thought you might people to be able to fully REBUILD that plugin... All these files are compiled and in the Toolkit/Assets/Babylon/Plugins as dlls... That i pre-built from these projects... If you want the WHOLE souces for all the plugins that DID NOT come from nuget (Ones I had to Build From Source)... These are them :)

You can cancel the Pull and we can just go with the prebuilt binaries... And i can put a URL link somewhere in docs that point to where they can download the source for FreeImage... And AjaxMin... I made that Babylon.Terrains Generator Specifically for BabylonJS... Might wanna keep that source project

OK let's close it as I do not want our repo to host other projects than ours:)


the setting ”temperature scalar T to 2“ is not in codes？

what is ”temperature scalar T to 2“ ？
thank you ！

I know.


Vertex path doesn't update properly when new points are added in runtime

When adding new points to the Bezier path during runtime, the vertex path doesn't update unless you first click on the GameObject the PathCreator script is attached to.

an event pathUpdate on PathCreator is running somehow in editor
so , you should call method TriggerPathUpdate on PathCreator each time you change the bezzier path to trigget the event
this way it should be working 😉

Thanks for the response! I actually ended up using VertexPathSettingsChanged on the PathCreatorData, which gets the vertex path again and calls TriggerPathUpdate. TriggerPathUpdate, for some reason, doesn't work when I call it by itself.

TriggerPathUpdate() does nothing for me as well, and frustratingly, VertexPathSettingsChanged does nothing as well! I feel as though I've tried everything, I'm just trying to change the positions of two of the points on the path using a script, but when the path isn't selected in the editor, it doesn't update. Please, any help would be appreciated, I'm at the end of my rope.


 I don't have a crashlog because it's an unexpected issue.

why is this invalid? how is this invalid? its not!

why is this invalid? how is this invalid? its not!

Lack of crash log.

You are doing an amazing job geniiii! Don't let people's rudeness bring you down, take as much time as you need :)

yea!

also, this does not crash on 1.14 fabric

btw, you could just close this as incomplete, not invalid!

btw, you could just close this as incomplete, not invalid!

I think it's best to leave it as invalid, since they did not include any crash logs, and were incredibly rude with the title of their "issue".


Display application number

For users who do not use MacOS, it is more difficult to quickly get the version of Quest currently in use. An entry that displays the current version should be added to the settings or menu.

Done with cc36ee89c8e3dd98963fc37bbd6d55476e6e2069


Flare() trigger bugfix and config changes

I found a bug in classes.py that prevented the code from running.  Also, I have updated validation.py to use the config.py configuration instead of assuming a hard-coded directory structure.  I added a requirements.txt that will be used later for packaging.  It can be used to get the code up and running in a virtual environment like this:
% git clone https://github.com/ktindiana/sphinxval.git
% cd sphinxval
% virtualenv venv
% pip install -r requirements.txt
% python bin/sphinx.py --help

I made a new commit to ignore (with warning) invalid empty string "" noaa_region that COMESEP has been providing.


Update to work with new modular exports

The generated files in tests still need to be updated, it currently doesn't work because the move packges need to be re-published on devnet I think

Looks good, thanks for the PR! I'll improve the test setup later.


Make export to PDF safer...

...by mandating sequencing of the promises. To regain the speed benefits I introduced in #30, perform extensive testing on concurrency and parallelism w.r.t loadPage in pages.ts

Don't accept this; there's an issue on my computer and I don't know whether this causes it.

It does not; this PR is valid


[PLUGIN/CORE/TAGS] Feature request: Merge tags

There are a lot of tags that are similar. This makes the tag list unnecessary long and more difficult to find a tag.
It would be great if it was possible to merge tags under one main tag.
E.g.: 'bird', 'birds' and 'aves' are practically the same. Someone can pick the tag 'birds' as the main-tag (what will be listed in the side-bar). Clicking on it will show all articles with the tags 'bird', 'birds' and 'aves', with the url https://blog.tld/tag/birds.
Bludit version
3.8.1

While someone might make a case for categories, aliased tags do sound quite nice.

I second this. It would be really useful to add tags to new content.


Introduce "Field.Group"

What
Introduce <Field.Group> from the previous versions of the Form.
Why
It is potentially useful for multi-form layouts like we have.
How
Look up the previous local versions.

A pull request has been issued for this feature (#23).

Implemented under #23. Closing.


WKWebView Detected unsupported browser

Hi,
we are trying to use edumeet embed in a iOS APP using WKWebView, when we try to load the URL it says detected unsupported browser. but same is working on safari browser.
Can you please let us know anything we need to change to support WKWebView
Thanks

I added this line of code and it will works fine with me.
Add it in viewDidLoad
myWKWebView.customUserAgent = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.97 Safari/537.36"

Hi @Alex-YddeR
I added this line of code and it will works fine with me.
Add it in viewDidLoad
myWKWebView.customUserAgent = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.97 Safari/537.36"


Flag issue for all interpretation problems

We flag lots of interpretation problems in the issue set of a NeoTaxon.
But this is not yet done consistently for all problems.
Update Issue enum adding missing XXX_INVALID entries and use them during interpretation in AcefInterpreter and DwcInterpreter

We track issues for names and taxa separately in the db but add them to a single set in NeoTaxon during interpretations. Reconsider doing this. Consider to:

add issues directly to NeoTaxon.taxon and NeoTaxon.name as relevant
how many issue types can be applied to both a name and a taxon? Should we have separate NameIssue and TaxonIssue enums? Or indicate for each enum value if its relevant to names and/or taxa?

Clean issue flagging code and restrict issues to name and/or taxon if possible


Support array in test_prefix

In some cases it makes sense to provide multiple entries in test_prefix in the CLT action. Let's add support of arrays in test_prefix.

I've added support for multiple inputs in test_prefix that should follow this format:
test_prefix: test-prefix

or
test_prefix: |-
  prefix1
  prefix2

I've also refactored and updated the flow in ManticoreSearch to group and simplify the CLT tests into a matrix: https://github.com/manticoresoftware/manticoresearch/pull/2075
The version 0.1.23 of CLT also has a comment_mode parameter that can be set to 'always' or 'failures', defaulting to 'always'.


BLEScan::start - blocks when duration is 0 (infinite)

BLEScanResults BLEScan::start(uint32_t duration)
can be called with duration=0, the C function esp_ble_gap_start_scanning will then scan indefinetly.
The callback mechanism however can not be used in this case as the semaphore m_semaphoreScanEnd.wait("start"); will block the function from returning.
Simple use case of duration=0 is when you just listen and want to use the BLE as beacon locator (e.g. RSSI based distance measurement/proximity detection), and you would like to have a callback for tracking the advertised IDs/addresses.
I suggest to introduce start_async() where the function will not return a list of found devices but requires a BLEAdvertisedDeviceCallbacks* pAdvertisedDeviceCallbacks

it is this is what u mention? https://github.com/nkolban/esp32-snippets/issues/496

It was a long time ago. I used instead the C implementation with the
callbacks available.
I did not mean to create a separate task but that is also an option (my
point was only that with duration=0 a callback would be enough to register
using the CPP api).
2018-05-24 2:37 GMT+02:00 beats0126<EMAIL_ADDRESS>
it is this is what u mention? #496
https://github.com/nkolban/esp32-snippets/issues/496
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nkolban/esp32-snippets/issues/410#issuecomment-391547039,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgCm-eOncn5Upu4Psm4FladSG08GPjnKks5t1gC4gaJpZM4SQfkO
.


Accounts should be managed by this provider

Is your feature request related to a problem? Please describe.
We need to create multiple snowflake accounts within a snowflake organization and it would be nice to do this as code in Terraform, similar to how we already manage AWS accounts.
Describe the solution you'd like
There should be a resource such as snowflake_account which wraps around CREATE ACCOUNT. For example:
resource "snowflake_account" "staging" {
  name           = "staging"
  admin_name     = "some person"
  admin_password = "SecretP@ssword"
  email          =<EMAIL_ADDRESS>  edition        = "STANDARD"
}

Describe alternatives you've considered
Just creating accounts by hand... gross.

https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/1492


Fix Ruby 2.4 issues

Solves issues identified in #110

Wow, thanks for finding this so fast and getting a fix in!

Thanks again…released as 1.9.5.
There's a lot of warnings, but I'll clean those up in another PR


Revamp configuration properties

This PR:

removes the names of pre-2.10 property names from the documentation,
adds a note about fuzzy matching of properties and a list of all  prefixes reserved by Log4j,
splits the system properties into sections according to the component  that they apply to,
updates the description of configuration properties based on the way  they are used in the current code.

@vy,
I reformatted the tables as sections. Can you take a look at this again?


validation error: #/definitions/example: failed schema #/properties/definitions/additionalProperties: "description", "links", "title" weren't supplied.

We tried to switch from rubygems version (v0.7.0) to master branch and got this.
I have bisected it down to this merge: https://github.com/interagent/prmd/commit/19553243f2560dae14f8feedbb52ad5fb0d43029

What this validation error means?
Is it a planned breaking change and we need to change our schemas?,
or it is a bug and I should create a PR fixing it? (if I would know the answer to 1st question, I probably could do that :) )?

Just discovered, that adding "$schema": "http://json-schema.org/draft-04/hyper-schema" to our meta.yml fixes the issue.
It affects only one of our schemas, not the other ones. So why one can comply to interagent's schema and other not? (I don't see real major difference between, how they define definitions..)

 It affects all our schemas, but still fixable with $schema value

Hey, I believe this comes from the interagent schema being more strict. We expect those other fields to better be able to build out documentation. In this case I think it stems from it expecting #/definitions/example to contain a resource schema (which I'm presuming it doesn't). If that definition had description, links, and title fields, it would validate. In the very near term, I think you can probably ignore the error and docs/etc would still generate fine. We've had some discussion around this, the validation stuff should probably be changed to be less stringent unless we have reason to believe a particular definition is a resource, but we don't have a good way to do that presently. Perhaps we should nest the known-to-be-resource definitions in something like #/definitions/resources/... so that we can be more or less stringent here. What do you think?

Another option might be to add a new explicit key, ie #/schemata/... or even #/resources. Then definitions would always just be for shared stuff and it would be pretty clear/obvious what this other namespace would contain.


How to use BasicForm?

Hello everyone,
Is anyone has some docs that explain how to use BasicForm and schema?
I appreciate any help!

https://docs.naiveadmin.com/components/form.html


Image Index Not Working

I'm trying to put a certain Dropbox icon on a menu. I've tried both of the following, but the image is always the main program icon, not one of the other ones in the executable. Is there a different way I should do this?

image=image('C:\Program Files (x86)\Dropbox\Client\Dropbox.exe', 20)
image=image.res('C:\Program Files (x86)\Dropbox\Client\Dropbox.exe', 20)

Are you using the debug version?

I'm using 1.8.1 from winget.

please update to 1.9 and test it

Still not working on 1.9. Here's my code:
modify(find='View in Dropbox' menu='Dropbox')
modify(find='Copy Dropbox link' menu='Dropbox')
modify(find='Version history' menu='Dropbox')
menu(title='Dropbox' image=image('C:\Program Files (x86)\Dropbox\Client\Dropbox.exe', 21)) {}

Everything works except it still just uses the main icon instead of the indexed one.

Try this until it fixes
menu(title='Dropbox' image='C:\Program Files (x86)\Dropbox\Client\Dropbox.exe,2')) {}

That worked with a small typo fixed:
menu(title='Dropbox' image=image('C:\Program Files (x86)\Dropbox\Client\Dropbox.exe,2')) {}
Thanks!


File download not working in production

The JSON file with the data works in local, but not in production.

Thanks @manthanabc !!

you are welcome : )

Hey, @manthanabc could you help in https://github.com/Fonta22/iss-locator/issues/30? It is happening the same again and don't really know what to do.

sure


Add support for multi-qubit-gates to CSPLayout

Summary
Fixes #7155. Added support for multi-qubit-gates to CSPLayout. CSPLayout now assumes for every 3+-qubit-gate we need a full connection between all involved qubits.
Details and comments
I've implemented the version suggested in the issue, assuming all qubits involved in the gate, need a full connection. As this is my first issue and I'm not too familiar with the library yet, I'm wondering about a few things:

Should I add a test case for this fix?
Do we need to take into account direction? Should we require a bidirectional full connection?
This approach does improve CSVLayout, but is the assumption actually fair? For Toffoli it holds, but can we say every 3+-qubit-gate does require a full connection?

[x] Possibly add testcase
[x] Add release notes
[ ] Check how/if we need to consider direction
[ ] Check assumption

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

Hey guys,
what's the right way for me to get some feedback on this?

Removing the [WIP] is the usual way to signal that :-)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 7 (71.43%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 83.378%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/layout/csp_layout.py
5
7
71.43%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
52239

Relevant Lines:
62653

💛  - Coveralls

very well :D

Hi @Silrem thank-you for your contribution and apologies for the slow response. Yes please do add a test case for this and a bugfix release note. You can check out the Contributing Guidelines for details on this 😄

Hi @javabster, thanks for your guidance. I've now added a test case and release notes.

Hey @Silrem it looks like you've got a linting error, you can run tox -eblack to fix this

blacking in 096a4a00d14296ab0336f1002ee8636b843f0800

Do we need to take into account direction? Should we require a bidirectional full connection?
Thanks a complicated one. I think is better not to do it for now.

@javabster thanks for your hint. I've learned some new syntax there. committed as you suggested.

@Silrem Hey, just want to check in to see if you are still working on the PR. Can you please add the tests as suggested by Luciano?

@HuangJunye At the moment, I'm not working on this PR anymore. Over the last months, other duties have taken over, unfortunately. I might be back in the future, but for now, feel free to let someone else complete this if they want.

Closing this PR as contributor isn't able to complete right now. We can re-open at a later date if contributor returns 😄


allow using udp parameter for spatial/temporal extents

pull request to allow passing udp Parameter objects

@HansVRP  @VincentVerelst  in the frame of adding worldcereal UDP's to apex catalog, can someone take over this PR and check what is still relevant and needed today?
It's not super urgent, but we probably want to move this forward.


Synchronous read timeout

Hi,
Currently there is no way to call the synchronous read_message() method with a timeout. I think it is important to have support for a read timeout because the websocket can stay connected without receiving any data for a while and we cannot always rely on ping/pong messages to interrupt the blocking read.

Hi,
That's indeed a valid point.
With the async (i.e. ASIO) client, that's already possible, see https://github.com/rbeeli/websocketclient-cpp/blob/main/examples/ex_reconnect_asio.cpp#L99
For bare TCP (unencrypted) socket, I have extended the POSIX socket wrapper to be able to set read/write timeouts: https://github.com/rbeeli/websocketclient-cpp/blob/main/examples/ex_hello_ws.cpp#L43
Only for the built-in synchronous OpenSslSocket (WSS) case there is currently no timeout example, which is something to look into, e.g. via poll/select.

It makes sense to have a better handling of timeouts. For example if a timeout happens before a full frame is read, just buffer the data received so far and return an empty reply without flagging an error. Then upon next read request check if the buffered data + any newly read data are sufficient for a new frame and if so, return the frame.

I'll look into a solution for the built-in OpenSslSocket implementation. Whether it will be possible to recover from a timeout needs to be seen. Commonly, the client is closed and reconnected after experiencing a timeout.

Hi @MeanSquaredError,
I have extended the codebase to allow for timeouts for the client functions handshake (previously called init), read_message, send_message, send_pong_frame, and close.
This allows the application to avoid eternal hanging on any of those functions.
Please note that the client must be closed on a timeout error and recreated, the client cannot recover from a timeout.
As always, the examples are a good reference: examples/builtin/

@rbeeli
Thank you for the update. I think that I need a version that would be able to recover from timeouts. Basically I want to do the following in a loop:

Read the websocket with a timeout of 1 second. If data was read successfully (no timeout), proces the data. If there was a timeout, just continue with step 2.
Check if there was no data for 30 seconds, if so send a PING frame.
Go to 1.

So I am trying to use timeouts as a poor-man's version of ASIO/coroutines without actually using either of these.
Here is some background information to explain why I need that.
I am trying to read data from a websocket+STOMP connection provided by a crypto exchange. Currently I am using a custom WS implementation + custom STOMP implementation, but I am looking for a good 3rd party websocket library to switch to. Earlier I tried using boost::beast::websocket, but their implementation is buggy and limited, so I had to switch temporarily to my own implementation.
My program connects to the crypto exchange's WS+STOMP stream and then subscribes to the trades channel for a crypto pair. Once my code subscribes to the stream, the exchange sends a WS/STOMP frame with the last 100 trades for the crypto pair and after that new trades (if any) are being streamed.
However after the initial block with 100 trades, it is possible that the exchange has no new data to send for 60 seconds. In such case after the 60 second timeout elapses, the exchange simply closes the TCP connection, without any websocket close frame. I would like to avoid re-connecting, because the reconnect takes time and upon the reconnect the exchange will send me once again the 100 last trades, and their receiving/processing takes additional time too.
So I would like to try sending PING frames (either WS PING or STOMP heart-beat, whichever works), every 30 seconds or so, in the hope that the exchange would not close the connection if it sees the WS PING/STOMP heartbeat.
However in order to do that, I need blocking reads with timeouts that do not require the websocket to be destroyed and created again, because that would imply a reconnect to the exchange's data stream.
So I wonder what would be the easiest way to achieve that. Maybe use SSL_poll on the underlying SSL connection and only continue with the blocking read if there is some data. In 99% of the cases it will probably mean that there is a websocket frame to be read. Or maybe just try to switch to ASIO/coroutines, something that I really try to avoid. Do you have any suggestions/advice?

Hi @MeanSquaredError,
I understand your requirement, in fact, I have had the same, just in ASIO it's easier to implement by using your own timer next to the read_message call. May I ask what the reason is to not use ASIO in your case? It would solve your problem.
The only way I see to recover from a read_message timeout is if the very first read call inside read_message times out, hence no data has been read yet, then it is possible to read again and not be in a corrupt/undefined state.
Once data has been read and it times out inside the same call to read_message, recovering is not easily possible without rewriting the whole read part and do considerable bookkeeping, which I do not plan to implement.
If we want to recover from a read_message timeout, we might need two timeout parameters for the read_message function. The first would be a message timeout, i.e. how long do we want to wait for a message before we time out, and the second is a read timeout, i.e. how long should it max. take to read a message once we start reading bytes. It would be possible to recover from the first timeout, but not from the second. This way you could periodically send ping etc. on message timeouts. Also, a message timeout could be a dedicated return type, while a read timeout is a normal error.
Another reason to have two timeouts is that we don't want to necessarily time out while reading a message. For example, let's assume we have only one timeout parameter and it's set to 30 sec. Now, after 29.5 sec, a message comes in and it takes 1 sec to read it from the network. At 30 sec, it would time out in the middle of reading the message, even if normally reading a message within 1 sec would be considered ok. To avoid this, the message timeout could be set to 30 s, and the read timeout to 5 s, so the max. time the method could take is 35 s.

Hi @rbeeli

May I ask what the reason is to not use ASIO in your case? It would solve your problem.
The reason why I try to avoid ASIO is that its programming model is pretty intrusive and the whole program would have to be written around it. Essentially the whole processing would happen inside io_context::run()

If we want to recover from a read_message timeout, we might need two timeout parameters for the read_message function. The first would be a message timeout, i.e. how long do we want to wait for a message before we time out, and the second is a read timeout, i.e. how long should it max. take to read a message once we start reading bytes. It would be possible to recover from the first timeout, but not from the second.
I understand that. I really need just the first timeout, which is the timeout until data becomes available. I don't really need the second timeout, because as I mentioned in my previous message, in 99% of the cases, once data starts flowing in, the whole frame should arrive real fast.

I think I will just create a simple wrapper around WebSocketClient::read_message() that will do an initial SSL_poll() wait until data becomes available or a timeout occurs and then if data is available will just forward the call to WebSocketClient::read_message().
By the way, I was able to workaround my problem with the exchange that times out after 60 seconds. If the STOMP connection to the server is configured so that

The server sends a heartbeat to the client every 30 seconds.
The client does not send any hearbeats.

then the server does not close the WS+STOMP stream even if there is no real data to be sent.
So the only thing that my code has to do is to ignore the received server-side heartbeats. This means that at this point I don't have to think of workarounds to send PING frames or other keepalive message.
If further down the road a similar problem arises, that cannot be solved through blocking reads/writes, I guess I will have to switch to ASIO, but for now I guess I will stay with blocking reads/writes.
Anyway, thanks for the help. I think that the problem is resolved, so we could probably close the issue?

Hi @MeanSquaredError
I have added the following function to the blocking WebSocketClient:
    /**
     * Waits until a message is available to read from the WebSocket connection.
     * Note that this includes control frames like PING, PONG, and CLOSE frames.
     *
     * Note that the timeout does not cause an error, but returns `false` if it expires.
     *
     * @returns `true` if a message is available to read, `false` if timeout expires.
     */
    [[nodiscard]] inline expected<bool, WSError> wait_message(std::chrono::milliseconds timeout_ms
    ) noexcept
{ }

I believe this function should help to resolve your issue.
See the corresponding example here: examples/builtin/ex_wait_message_wss_builtin.cpp.
This way, you can periodically perform actions while waiting for a message/frame to arrive, without having to deal with errors or recreating the whole client.

@rbeeli
Great, wait_message() is exactly what I need. Thanks!


[Proto] Implement a State result type

Must allow the handling of:

[x] State vectors as Tensor
[x] Density matrices as Tensor for noisy simulations and functionalities in #367

@dominikandreasseitz @jpmoutinho I will move this one to the new block system project.

Closing as moved elsewhere.


Roll Flutter Engine from bf5c003085fd to 7a58dac0b1c8 (2 revisions)

https://github.com/flutter/engine/compare/bf5c003085fd...7a58dac0b1c8
2024-02-21<EMAIL_ADDRESS>Tweak verbose log messages in ImageReaderSurfaceProducer (flutter/engine#50831)
2024-02-21<EMAIL_ADDRESS>Add a throw statement for imgtestAdd non 0 exit codes. (flutter/engine#50829)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update .gitignore

Description

Ignore the nextpnr-fabulous build output since you likely won't revision control it

No longer required because nextpnr not required for fabulous as a submodule anymore


Remove no-server build arg

Proposing this change since no-server is now enabled by default, and I got a warning that this flag is being ignored during the build.
For context, I got the warning when building on cloud shell with Graal <IP_ADDRESS>-r17.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

Thank you for the fix! Good to know that it is now present by default.

We might need to rename the commit message and PR heading to docs(sample): remove no-server build arg to address the linter issue.


PLT-5905: Proof-reading specification

This PR fixes minor issues in the Marlowe specification document. In addition it proposes some changes in chapter 1.

Just a few corrections are needed.

Thanks a lot @bwbush


Add filter option to run script

It would be great if someone with access to VCS could try out this PR.