golang style package management support

Is your feature request related to a problem? Please describe.
vcpkg for now, only allow user to install packages that have a port file in this repo.  For some projects that are not so popular, ports may not be accepted.
Making a private port library/registry takes time and efforts.
Getting to know the correnct package name for a certain github projects takes effort.
Proposed solution
golang's package manager uses github/gitlab repo as a de-centralized registry. They use a full url(like github.com/username/packagename) of the repo as package name, which makes naming conflicts impossible. Packages are automatically supported by package manager as long as it is written following certain rules.
Can we take some of golang's idea and take packages(following certain rules) from github automatically?
This helps both package author and user.
We(vcpkg developer) don't have to maintain a larges port library.
If a package doesn't follow the rules, user can fork and apply patch himself and ultimately make a pull request to the original package author.
Given a github repo, user don't have to take any effort to figure out the correnct package name in vcpkg.
Describe alternatives you've considered
We can keep current port. An alternative would be that we make every fork of the packages and apply patches.

Can you post an example so that we can discuss it better?
Thanks.

Can you post an example so that we can discuss it better?
Thanks.

hi @JackBoosY :
when golang user want to install a package, they type:
go get github.com/davyxu/cellnet

which is quite convenient and versitile.
for now, vcpkg allow us to:
vcpkg create handycpp https://github.com/zhangfuwen/handycpp/archive/refs/heads/main.zip "handycpp.zip"
vcpkg install handycpp

As a package consumer,  I want to write a vcpkg.json in my project:
{
  "name": "gles-demo",
  "version-string": "0.1",
  "dependencies": [
    "github.com/zhangfuwen/handycpp",
    "pthreads",
    "happly"
  ]
}

and then vcpkg install.
vcpkg have enough facility to do this. A small script may easily achieve this goal using vcpkg. But I think we should take a step further and make it standard in vcpkg. This will encourage C/C++ authors to write standard cmake/vcpkg projects and eventually we will have a modern C++ package manager.

I think this issue is more about describing vcpkg create.
In fact, I had a PR to improve the create command: https://github.com/microsoft/vcpkg/pull/9430 but after some discussion, we think it's too complex.
The code in portfile.cmake contains the following parts:

Download and unzip the source code
Set options and link with the feature in vcpkg.json
configure, build and install
Repair the content after installation
Install the license.

We can do 1, 3 and 5, but 2 and 4 are not easy to solve:
For 2: The options for each port may contain complex dependencies and are compiler or system dependent. We need to look at its configuration file in detail before deciding what options to add.
For 4: Some of the generated configuration files may have path or content problems, we should add this part after testing.

Note that different 3rd party libraries may use different configurators such as cmake, meson, qmake or others. Obviously, their usage is inconsistent.
And because of this, the configuration interface is not as unified as golang. This is also the reason for the current situation.

@JackBoosY I agree there is not a solution that will fix all problems. But I think even a solution that will work for some of the 3rd party libraries will make a point. Fow now, there is not even a rule for people to conform to. We should make the rule first and let library author conform to it later. Package managing with C/C++ is all a mess and I found vcpkg the most promising.

@BillyONeal What do you think about this?

I don't expect this to be something we are interested in doing soon, as

it makes transitive dependencies a potentially unbound set, which allows an unknowable amount of third party code to be run by customers without reasonable means for them to control
as proposed it fails the 'no changes means no changes' test, since there is no declaration of which version to build here
very few ports' portfile.cmake contents are automatically discoverable

Golang got to declare in advance a packaging format that their entire ecosystem agreed upon and most of their developers follow. There is no such standard packaging format for C++: that's why we have portfile.cmake in the first place.


Hotfix filtering plotting

Fix access to strand() in plotting. Also added error catch if transcript GOF statistic filtering removes all transcripts.

Codecov Report

Merging #49 into master will decrease coverage by 0.06%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
- Coverage   89.69%   89.62%   -0.07%
==========================================
  Files          15       15
  Lines        1048     1051       +3
==========================================
+ Hits          940      942       +2
- Misses        108      109       +1

Impacted Files
Coverage Δ

R/plot_tq.R
98.23% <100%> (ø)
:arrow_up:

R/transcript_shape_profile-class.R
95.08% <66.66%> (-0.29%)
:arrow_down:

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 fd6a1f7...f574212. Read the comment docs.


Wrong example

this can't be used in arrow functions:
https://github.com/mochajs/mocha/blob/master/docs/index.md#arrow-functions

Hi @falsandtru
Thanks for taking the time to file this.  However, the example was given as something that won't work as described by the preceding text:
For example, the following code will fail:

What might be interesting is if we standardize a clearer way of giving properly formed examples and breaking examples? @ScottFreeCode @boneskull what do you think?  Is there enough cases or gotchyas we can cover by that type of documentation change?  I'm envisioning a 50% split in the code block examples, one with a heading "Works" in green, and the other with "Won't Work" in red?

IMO the best way to do this would be to have "live" code examples, a la AngularJS.  @markowsiak can you make an issue for that?  I don't think it'll get done any time soon, but if someone ever wants to work on the site, that's a big win.

Right, sorry about that.


It is recommended to use Grpc channel.
thanks :)

@koo-taejin
Can you explain the bug in detail to help me recreate?
thanks.

It is also related to the issue #3217
This error may occur even if another unexpected connection is connected to the old channel.

my pp-server version is 1.7.6
this 'invalid packetType received' ended up causing a memory leak both at client and server

2020-08-18 11:39:55 [Pinpoint-Server-Worker(17-2)] WARN  Unexpected exception in the selector loop.
java.lang.OutOfMemoryError: Java heap space
2020-08-18 11:40:08 [Pinpoint-Server-Worker(17-15)] ERROR invalid packetType received. packetType:-4336, channel:[id: 0xb6f0173c, /<IP_ADDRESS>:51732 => /<IP_ADDRESS>:9995]
2020-08-18 11:39:39 [Pinpoint-Server-Worker(6-13)] WARN  Unexpected exception in the selector loop.
java.lang.OutOfMemoryError: Java heap space
2020-08-18 11:39:31 [Pinpoint-Server-Worker(17-12)] WARN  Unexpected exception in the selector loop.
java.lang.OutOfMemoryError: Java heap space
2020-08-18 11:39:31 [Pinpoint-Server-Worker(17-6)] WARN  Unexpected exception in the selector loop.
java.lang.OutOfMemoryError: Java heap space
2020-08-18 11:39:14 [Pinpoint-Server-Worker(6-11)] WARN  Unexpected exception in the selector loop.
java.lang.OutOfMemoryError: Java heap space.

I recommend you to extract and to analyze heap dumps.


Add creation of arbitrary symbolic links at sandbox creation

One idea - bind symlinks to whitelisted (bind mounted paths)
Possibly a field in the whitelist entry, e.g.
, {"path": "${XDG_DOWNLOAD_DIR}/Chrome", "can_create": true, "symlink": "${XDG_DOWNLOAD_DIR"}

Fixed by shared folder features. symlink remains unexposed to normal whitelist items but could be turned on if needed.


Add a "Copy as WorkItem" button

Move buttons underneath the issue title

Fixes #28

@dpoeschl I think I addressed all your feedback.

Looks good. I'm interested to see if we get feedback about the buttons being on a separate line on the individual issue/PR page. It looks especially weird on the individual PR page with only the one button, but I think that will get better when we add more features.
We'll probably need to add some spacing between those buttons too, but this seems fine for now. :+1:


"NoMethodError: undefined method `/' for nil:NilClass"  for certain currencies

Note: This error occurs with 145 of the 178 currencies available.
Money.new(100_00, 'GBP').exchange_to('BYR')
NoMethodError: undefined method /' for nil:NilClass 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/bundler/gems/eu_central_bank-d2cc9ac3c8f5/lib/eu_central_bank.rb:63:in exchange_with'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/money-6.7.0/lib/money/money.rb:434:in exchange_to' 	from (irb):294 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/railties-4.2.1/lib/rails/commands/console.rb:110:in start'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/railties-4.2.1/lib/rails/commands/console.rb:9:in start' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/railties-4.2.1/lib/rails/commands/commands_tasks.rb:68:in console'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/railties-4.2.1/lib/rails/commands/commands_tasks.rb:39:in run_command!' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/railties-4.2.1/lib/rails/commands.rb:17:in <top (required)>'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:274:in require' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:274:in block in require'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:240:in load_dependency' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:274:in require'
from /home/peter/TipShield/workspace/tipshield/bin/rails:9:in <top (required)>' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:268:in load'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:268:in block in load' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:240:in load_dependency'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:268:in load' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/commands/rails.rb:6:in call'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/command_wrapper.rb:38:in call' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application.rb:185:in block in serve'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application.rb:156:in fork' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application.rb:156:in serve'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application.rb:131:in block in run' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application.rb:125:in loop'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application.rb:125:in run' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/spring-1.6.4/lib/spring/application/boot.rb:18:in <top (required)>'
from /home/peter/.rbenv/versions/2.2.2/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in require' 	from /home/peter/.rbenv/versions/2.2.2/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in require'
from -e:1:in `'irb(main):295:0>

Ah, looking at the source code, it seems these are the only currencies available for conversion:
["USD", "JPY", "BGN", "CZK", "DKK", "GBP", "HUF", "ILS", "PLN", "RON", "SEK", "CHF", "NOK", "HRK", "RUB", "TRY", "AUD", "BRL", "CAD", "CNY", "HKD", "IDR", "INR", "KRW", "MXN", "MYR", "NZD", "PHP", "SGD", "THB", "ZAR"]

This list seems to be the most common global currencies which works fine for my purposes. Thanks for the sweet gem. Closing.

I've been running into this problem even with supported currencies:
Money.new(100, 'CAD').exchange_to('EUR')
NoMethodError: undefined method `/' for nil:NilClass
from /Users/alexanderpiercey/.rbenv/versions/2.1.0/lib/ruby/gems/2.1.0/gems/eu_central_bank-1.0.1/lib/eu_central_bank.rb:63:in `exchange_with'

@APiercey it seems like the error is caused by not being able to get a direct rate and falling back to a double exchange via EUR… can you try any other exchanges?

Unfortunately, I get the same result.
Money.new(100, 'USD').exchange_to('EUR')
NoMethodError: undefined method `/' for nil:NilClass
from /Users/alexanderpiercey/.rbenv/versions/2.1.0/lib/ruby/gems/2.1.0/gems/eu_central_bank-1.0.1/lib/eu_central_bank.rb:63:in `exchange_with'

Money.new(100, 'EUR').exchange_to('USD')
NoMethodError: undefined method `/' for nil:NilClass
from /Users/alexanderpiercey/.rbenv/versions/2.1.0/lib/ruby/gems/2.1.0/gems/eu_central_bank-1.0.1/lib/eu_central_bank.rb:63:in `exchange_with'

However, EUR to EUR works fine, it seems.
Money.new(100, 'EUR').exchange_to('EUR')
=> #<Money fractional:100 currency:EUR>

it ends up working after some retries or firing some eu_bank.update_rates, but that feels unsafe.
Is that due to their api ?


ci: use actions for codecov

Signed-off-by: Troy Ronda<EMAIL_ADDRESS>

Bash uploader is now working. Waiting for proper Action. https://github.com/codecov/codecov-action/issues/29#issuecomment-594976661

Codecov Action is now working.


[MM] Label and destination inconsistency

This work should begin after the bugs are fixed.  The target to finish all content updates is the end of this sprint, Friday Aug 28.
Detailed description
The label Infrastructure security goes here: https://www.ibm.com/it-infrastructure/storage/data-protection but in the spreadsheet it says Secure infrastructure and goes here: https://www.ibm.com/it-infrastructure/solutions/security?lnk=hpmps_buit&lnk2=learn
Steps to reproduce the issue

Click Products & Solutions
2.Click Infrastructure security

@amandafbooth not seeing Secure infrastructure in the spreadsheet:

Per chat with Keegan - secure infrastructure will only be in v18, not in the carbon mega menu


API_'Schedules'_Status code 200 (Successful update of schedule) is displayed when User updates a single schedule with wrong 'index' data

Environment: Windows 10 Pro, 64 bit operating system, Postman v9.15.10,
https://whatbackend.azurewebsites.net/index.html?urls.primaryName=WHAT Project API v2.
Reproducible: always.
Build found: last commit from https://github.com/ita-social-projects/WhatBackend (dev branch)
Preconditions
Log in as Administrator to https://whatbackend.azurewebsites.net/index.html?urls.primaryName=WHAT Project API v2
Log in as Administrator to in "Postman" app with collections "WHAT Project API"
Steps to reproduce
Make PUT request to REST endpoint https://whatbackend.azurewebsites.net/api/v2/schedules/eventOccurrences/{eventOccurrenceID} for 'Daily' type of occurrence pattern with following parameters: eventOccurrenceID 198
Request body contains JSON object:
{
"pattern": {
"type": 0,
"interval": 1,
"daysOfWeek": [
0
],
"index": -10,
"dates": [
31
]
},
"range": {
"startDate": "2022-04-03T08:37:09.667Z",
"finishDate": "2022-04-05T08:37:09.667Z"
},
"context": {
"groupID": 1,
"themeID": 1,
"mentorID": 1
}
}
Actual result
Status code '200': Response body:
{
"id"*: 198,
"studentGroupId": 1,
"eventStart": "2022-04-03T08:37:09.667Z",
"eventFinish": "2022-04-05T08:37:09.667Z",
"pattern": 0,
"events": [
{
"id: 39702,
"eventOccuranceId": 198
"studentGroupId": 1,
"themeId": 1,
"mentorId": 1,
"lessonId": null,
"eventStart": "2022-04-05T08:37:09.667Z",
"eventFinish": "2022-04-05T08:37:09"
},
{"id: 39703,
"eventOccuranceId": 198
"studentGroupId": 1,
"themeId": 1,
"mentorId": 1,
"lessonId": null,
"eventStart": "2022-04-05T08:37:09.667Z",
"eventFinish": "2022-04-05T08:37:09"
},
{
"id: 39704,
"eventOccuranceId": 198
"studentGroupId": 1,
"themeId": 1,
"mentorId": 1,
"lessonId": null,
"eventStart": "2022-04-05T08:37:09.667Z",
"eventFinish": "2022-04-05T08:37:09"
}
],
"storage":<PHONE_NUMBER>678805000
}
Expected result:
Status code: 400
(Error, update data is wrong)
Note
According to "Schedule recurring events", even if a field is ignored, it is still validated. If a field has a set list of possible values, using a value outside the allowed set causes an error, even if that field is ignored.
The same bug also is reproduced for the following conditions:
with 'Weekly' type of occurrence pattern
with 'Secretary' role.
User story and test case links
Test case https://jira.softserve.academy/browse/DW-401
SRS https://github.com/ita-social-projects/WhatBackend/wiki/Schedule-service-overview
Schedule recurring events https://docs.microsoft.com/en-us/graph/outlook-schedule-recurring-events
Labels to be added
"Bug", Priority ("Very Low "), Severity ("Minor"), Type "API" .
https://user-images.githubusercontent.com/97450841/161447346-8c59083b-2b23-44eb-9134-f229b1d5f86c.mp4
https://user-images.githubusercontent.com/97450841/161447364-30629d1e-0641-477d-a668-33cd633f0daa.mp4

Bug has been fixed


focus the editor at app startup; restrict focus traversal

focus the editor at app startup
restrict focus traversal out of the editing area

Wrapping the editing area in a new FocusScope will restrict focus traversal out of the widget. In particular, hitting tab will not move the flutter focus to another widget (the tab key - and shift-tab - will still be interpreted by codemirror). This should address https://github.com/dart-lang/dart-pad/issues/2852 and related.

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

I have another similar solution that allows focus to leave the editor with mouse and with Esc in https://github.com/dart-lang/dart-pad/pull/2920.
Happy to land either and iterate from there.

Ah, I missed that PR! I'm not attached this this PR in particular; happy to lane either one.


feat(Cardano): CW-8050 Implement staking methods

Add Cardano staking methods

Register
Delegate
Deregister
Withdraw

Task linked: CW-8050 SDK - Implement Cardano Staking


json dump of past dream challenges

I'm currently refactoring the ROCC schemas. Once I'm done I'll come back to this ticket and review it.

@thomasyu888 I sure do. I'll have to clean it up a bit though, as I kind of just wrote it to get the output quickly :grimacing:
I will mark this as a draft for now.


Add support for windows CPU affinity

This PR adds support for certain CRI fields in OCI runtime spec for supporting CPU affinity for windows containers.
CRI PR : https://github.com/kubernetes/kubernetes/pull/124285
Github issue: https://github.com/kubernetes/kubernetes/issues/125262
KEP: https://github.com/kubernetes/kubernetes/issues/125262

cc @jsturtevant @kevpar @aravindhp @marosset

@AkihiroSuda @thaJeztah could you please take a look? :) thanks!

Is there a compelling reason not to roughly match the naming convention adopted for Linux in #1253?
Also, please don't mark conversations with useful discussion/links as "resolved" especially if the PR itself didn't change as a result of the discussion but the discussion is still useful and potentially something someone else might reasonably ask about or want to discuss -- it makes them much more difficult to discover when reading the PR's comments.

Is there a compelling reason not to roughly match the naming convention adopted for Linux in #1253?

For cpu's, Linux is a simple list where as on windows it is a bitmap with a group number (https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/miniport/ns-miniport-_group_affinity)  and the naming/structure was used to map to those Windows specific concepts
For memory, I think it was the same idea, making sure it was known this is "perferred" but not guarenteed do to the Windows API's.  Depending on outcome of https://github.com/opencontainers/runtime-spec/pull/1258#discussion_r1648430504 we might not have this field

@AkihiroSuda @kevpar addressed review comments. Would you be able to take a look when you have some time please? Thanks!

@tianon @kolyshkin @kevpar @jsturtevant could you please take a look at this PR when you have some time please? Thanks!

/lgtm
for the functionality, I will defer to the reviewers here on the naming discussion

LGTM
@tianon @kevpar @kolyshkin could you give a final look?

A couple really minor nits, but I think this is probably fine.
Maybe we need to update the JSON schema also?
https://github.com/opencontainers/runtime-spec/blob/09fcb39bb7185b46dfb206bc8f3fea914c674779/schema/config-windows.json#L30-L43

Addressed this!

@tianon would you be able to take a look at this PR when you have a chance please? Thanks!


[test] Remove retired system KESCH

kesch[:cn,pn]  was removed from all tests and configuration files, since the system was taken out of production.

Hello @kraushm, Thank you for updating!

In the file cscs-checks/microbenchmarks/cpu/dgemm/dgemm.py:

Line 20:31: E201 whitespace after '['

Do see the ReFrame Coding Style Guide

Codecov Report

Merging #1588 (c8db986) into master (3ad9e85) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1588   +/-   ##
=======================================
  Coverage   87.57%   87.57%
=======================================
  Files          45       45
  Lines        7244     7244
=======================================
  Hits         6344     6344
  Misses        900      900

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 3ad9e85...c8db986. Read the comment docs.

@kraushm Can you also check why we have two failing tests?

@jenkins-cscs retry daint


group routes by self need create normalize route record, but vue-router-next not expose

What problem does this feature solve?
we can create normalize route record on demand and maintain them, this is convenient if normalizeRouteRecord exposed
What does the proposed API look like?
expose normalizeRouteRecord

I can create PR if this is reasonable~

I think this fits better in a library because it disconnect from how routes are really handled


Automatic reduction of nb of threads if they are above the max connections

The number of connections (maxConnections) represents the number of engines that are available, and the number of threads (maxNbThreads) is used to size the number of concurrent pdfs that are processed when running the end2end evaluation.
If maxNbThreds  > maxConnections many PDFs will fail because no enough engines will be available to cover all the parallel pdfs.
Example:
(base) [Luca@falcon grobid]$ ./gradlew jatsEval -Pp2t=/data/workspace/Luca/PMC_sample_1943 -Prun=1
<-------------> 0% INITIALIZING [232ms]
> Evaluating settings
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/home/Luca/.gradle/wrapper/dists/gradle-6.5.1-all/cdund22i8guosqylfo49op4dv/gradle-6.5.1/lib/groovy-all-1.3-2.5.11.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
Configuration(s) specified but the install task does not exist in project :grobid-core.
Configuration(s) specified but the install task does not exist in project :grobid-home.
Configuration(s) specified but the install task does not exist in project :grobid-service.
Configuration(s) specified but the install task does not exist in project :grobid-trainer.

> Task :grobid-trainer:jatsEval
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
>>>>>>>> GROBID_HOME=/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/affiliation-address/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/affiliation-address/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/name/header/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/name/header/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/name/citation/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/name/citation/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/header/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/header/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/date/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/date/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/citation/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/citation/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/fulltext/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/fulltext/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/segmentation/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/segmentation/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/reference-segmenter/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/reference-segmenter/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/figure/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/figure/model.wapiti
[Wapiti] Loading model: "/data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/table/model.wapiti"
Model path: /data/workspace/Luca/github/grobid/grobid-trainer/../grobid-home/models/table/model.wapiti

PDF processing   0% │                      │    0/1943 (0:00:00 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Lett_Appl_Microbiol_2009_Aug_49(2)_191-195/lam0049-0191.pdf
PDF processing   0% │                      │    0/1943 (0:00:01 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/J_Insect_Physiol_2010_May_56(5)_447-454/main.pdf
PDF processing   0% │                      │    0/1943 (0:00:02 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Neuropharmacology_2011_Feb_60(2-3)_488-495/main.pdf
PDF processing   0% │                      │    0/1943 (0:00:03 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Thorax_2011_May_25_66(5)_375-382/thoraxjnl153825.pdf
PDF processing   0% │                      │    0/1943 (0:00:04 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Curr_Atheroscler_Rep_2011_Jun_5_13(3)_277-284/11883_2011_Article_176.pdf
PDF processing   0% │                      │    0/1943 (0:00:05 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Public_Health_Nutr_2008_Feb_11(2)_203-213/S1368980007000304a.pdf
PDF processing   0% │                      │    0/1943 (0:00:06 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Protein_J_2010_Oct_25_29(7)_524-529/10930_2010_Article_9283.pdf
PDF processing   0% │                      │    0/1943 (0:00:07 / ?)           Could not get an engine from the pool within configured time.
Could not process: /data/workspace/Luca/PMC_sample_1943/Cell_Metab_2011_Jan_5_13(1)_92-104/main.pdf

This PR automatically reduces maxNBThreads to maxConnection -1 if maxNbThreds  > maxConnections.
Additionally, I've modified the GrobidPooledFactory to be strongly-typed as it's dealing only with Engine objects.
Note: As of today, this PR is compatible with the PR #706.

Coverage increased (+0.004%) to 39.68% when pulling 27c09eec6747444b03149b59c11b48ac1c866372 on improvement/e2e-automatic-lower-nb-threads into 0078ecd64d60466a15dcb48ff29a32e32fe258aa on master.

Normally everything work fine with the new "yaml-config" branch, now merged, because the configuration is a bit different. There is a global concurrency configuration parameter used generally (by the service, by the end-to-end eval too, and in the future probably by the batch mode too) and the nbThreads parameter is only defined for Wapiti stuff. So these parameters should be more clear and they are not "competing".
About the changes on GrobidPoolingFactory which are not related to the end2end evaluation, I agree that it's more clear with stronger typing, what about opening a dedicated PR for this? Or change this PR?

Sure! I close this and open a new one.


run fail in android bash

arm:
GOARCH=arm GOOS=linux GOARM=7 $(GOBUILD) -o $(BINDIR)/$(NAME)-$@
2020/02/18 08:51:25 failed to connect to target: dial tcp: lookup i.stack.imgur.com on [::1]:53: read udp [::1]:34310->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup i.stack.imgur.com on [::1]:53: read udp [::1]:39516->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup www.google.com on [::1]:53: read udp [::1]:56988->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup i.stack.imgur.com on [::1]:53: read udp [::1]:33594->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup www.google.com on [::1]:53: read udp [::1]:46059->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup i.stack.imgur.com on [::1]:53: read udp [::1]:52322->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup i.stack.imgur.com on [::1]:53: read udp [::1]:52322->[::1]:53: read: connection refused
2020/02/18 08:51:26 failed to connect to target: dial tcp: lookup www.google.com on [::1]:53: read udp [::1]:35978->[::1]:53: read: connection refused
2020/02/18 08:51:27 failed to connect to target: dial tcp: lookup graph.facebook.com on [::1]:53: read udp [::1]:35463->[::1]:53: read: connection refused
===
in adb shell it can reslove the domain
1|root@Hi3798MV100:/data # ping www.google.com
PING www.google.com (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from hkg12s18-in-f4.1e100.net (<IP_ADDRESS>): icmp_seq=1 ttl=55 time=14.5 ms
64 bytes from hkg12s18-in-f4.1e100.net (<IP_ADDRESS>): icmp_seq=2 ttl=55 time=15.6 ms
64 bytes from hkg12s18-in-f4.1e100.net (<IP_ADDRESS>): icmp_seq=3 ttl=55 time=14.3 ms
^C
--- www.google.com ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2003ms
rtt min/avg/max/mdev = 14.357/14.849/15.660/0.594 ms
root@Hi3798MV100:/data #

Please do not use cross compilation. The version compiled with Termux's pkg install golang and then go get is available.


Add "missing count" text

This adds a missing count variable to be used in custom options. I find it more useful to see how many counts I have left to pull instead of having to do the math on the fly.
The way I use it https://i.imgur.com/PKw3BKq.png

Added those :D

Thank you so much, that's super cool! I'll release a new version tomorrow :-)


storage pools: implement rsync.bwlimit property

Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

I think buildd06 went down.

jenkins: test this please

what's the unit of rsync.bwlimit? bytes?

@stgraber, everything that is parseable by ParseByteSizedString().

Perfect, I was just wondering as the tests use 1024 as a limit which seemed a bit low :)


Update TS for OOPers.md

I can see a case for both the before and the after to be honest

Isn't this guide teaching TypeScript?

Yep but TypeScript doesn't run in JavaScript engines - JavaScript does and the original wording is appropriate for that

Alright. I'm still convinced "TypeScript" would be a better choice but I'll close the PR.


Pump frequently switching on/off for 1 second for about an hour around 01:00 hrs every day

Seems like the anti-corrosion functions does something strange. At 0100 hrs the pump switches on and off every second for around an hour. It looks to me that something strange happens in the calculation if the anti corrosion should run or not....

Hi @davidoff62, yes, thank you for the feedback. This got the same message from another user. And I don't know yet what is happening. This switching is not good for the pump, so don't use the switch for now. One way of dealing with this is to disable the corrosion run inside the ESP Home script until it is fixed.
You could help me out, finding the cause if you like. I could create an updated script with some additional logging. Let me know if you like help.

Hi Jeroen,
Altough my guess is that you are a Dutch native speaker (so am I), let's stick to English so the world can follow.
Of course i like to help, no doubt on that! I'm an electrical engineer, so software isn't really my thing (a.k.a. i'm learning), but uploading a modified script is well within my capabilities.
In the mean time some more info: i checked and crosschecked the availability of the NTP server server for the SonOFF (as my tought was that this could be of importance), this is the case.
In your screenshot it clearly shows a continued operation of the pump :) but not at around 01:00 hrs, is this also normal behavior? .... from the documentaion i understood that if the pump did not run in the last 24 hrs because of high water temp the pump will run for 10 min at 01:00 local time (hence the NTP service).
BR David

@davidoff62 Yes, you guessed right, I am a Dutch native speaker. And, yes, we should stick to English, so others can follow the discussion. I studied electrical engineering as well, but my career headed quite fast in the direction of software development. So I did a second study on software engineering. Both worlds are very interesting and are quite connected.
Nice that you like to cooperate! Let's see if we can fix the bug.
By the way, the problem you mention is reported earlier, see this conversation: https://community.home-assistant.io/t/diy-underfloor-heating-pump-controller-in-home-assistant/362954
Are you using the S20 or the S26?
In response to your remarks.

In the mean time some more info: i checked and crosschecked the availability of the NTP server server for the SonOFF (as my tought was that this could be of importance), this is the case.

Yes, I think that something goes wrong with the SNTP section. In the discussion on the home assistant forum, Martin removed the entire section of the corrosion run and moved that to an HA automation. Of course, this solves the problem, but the general idea was to have a 'stand-alone' solution, which could work without HA being online.

In your screenshot it clearly shows a continued operation of the pump :) but not at around 01:00 hrs, is this also normal behavior?

The pump runs when warm water enters. There is a very small spike at one o'clock. That is only one minute. And that's indeed why the NTP service is configured for.

@jeroenvdwaal
I'm using the S26

Thinking of this code;
If the above is the issue, could the following be a solution?
time:
  - platform: sntp
    servers:  [<IP_ADDRESS>]
    on_time:
      # schedule anti corrosion action, once a day
      - hours: 1
        then:
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "true"
          - script.execute: controller

      - hours: 1 minute: 2
        then:
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "false"
          - script.execute: controller

Last night the pump did't freak out :) So i guess we can conclude the bug is in the above passage. Does my proposal makes sense? probably the syntax is faulty, but you get the point.....

That's great to hear. Great that you changed the script successfully!! I'm curious if the servers: line is still needed, or does it work fine without? Could you check that?

Just checked without the servers: line. The automation does not work without the line, best is to keep it in i guess.....
Like said: i'm not a software guy, can you therefore explain why in your revised code you use - before hour, minutes and seconds? i only use it at the first line of the time definition (conform the example:  https://esphome.io/components/time.html
I feel confident now and i connected the pump. Tonight we'll (hopefully) see the code in action :)

I have one option in my sleeve to fix it in a more easy way. It is not important to run the anti-corrosion at one o'clock. A minute every 24 hours, approximately suffices. So instead of using the time section, I just create a loop, which just waits 24 hours and runs for a minute. Which could be any time of the day. Which is fine by this automation. What do you think about that approach?

@jeroenvdwaal
Hi Jeroen,
Thanks for the addition! i guess we have two flafours now;
1: you latest version making sure the pump runs at least once every 24hrs on a undifined time and does not need a NTP server.
2: my version that runs on a pre-defined time, but does need a reachable NTP server.
As a response to your latest remark:
I do run my own NTP server within my IoT network segment. In this way hosts in this network can be (and are!) separated from the internet, but still have access to a NTP server. That's the reason you only see a local IP in my code.
I would like to run the pump at a pre defined time because my pump makes noise (and thus i run it at 10:00, not disburbing my sleep). Im my use case, i'll stick to flavour 2.......in other cases flavour 1 might be more practical!

Hi @davidoff62,
I am sorry for my late response. I posted something earlier, but I think didn't hit the submit button. It's nice to know that there is a preference for running it at a specific point in time. Thanks for the valuable feedback. I have only one question to ask.
  time:
  - platform: sntp
    on_time:
      # schedule anti corrosion action, once a day
      - hours: 1
        minutes: 0
        seconds: 0
        then:
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "true"
          - script.execute: controller
          - delay: 60s
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "false"
          - script.execute: controller

Did you test the script above? Do the minute and second entries solve the issue?

Hi Jeroen,
Been busy lately so dind't had time to check. I was busy fixing my homeassistant installation since the ssd crashed :P ....i'll start this weekend with the pump controller again. When i get ESPhome running again, i give the code a go.....

Thanks!
Op vr 11 mrt. 2022 22:30 schreef davidoff62 @.***>:

Hi Jeroen,
Been busy lately so dind't had time to check. I was busy fixing my
homeassistant installation since the ssd crashed :P ....i'll start this
weekend with the pump controller again. When i get ESPhome running again, i
give the code a go.....
—
Reply to this email directly, view it on GitHub
https://github.com/jeroenvdwaal/underfloor_heating_pump_controller/issues/1#issuecomment-1065544981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALMUMBQHRJBSDWXHL2E6ZYDU7O3OBANCNFSM5NOUIE3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<jeroenvdwaal/underfloor_heating_pump_controller/issues/1/1065544981@
github.com>

Hi @davidoff62,
I am sorry for my late response. I posted something earlier, but I think didn't hit the submit button. It's nice to know that there is a preference for running it at a specific point in time. Thanks for the valuable feedback. I have only one question to ask.
  time:
  - platform: sntp
    on_time:
      # schedule anti corrosion action, once a day
      - hours: 1
        minutes: 0
        seconds: 0
        then:
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "true"
          - script.execute: controller
          - delay: 60s
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "false"
          - script.execute: controller

Did you test the script above? Do the minute and second entries solve the issue?

Yes, the minutes & seconds addition fixes the issue

Hi @davidoff62,
I am sorry for my late response. I posted something earlier, but I think didn't hit the submit button. It's nice to know that there is a preference for running it at a specific point in time. Thanks for the valuable feedback. I have only one question to ask.
  time:
  - platform: sntp
    on_time:
      # schedule anti corrosion action, once a day
      - hours: 1
        minutes: 0
        seconds: 0
        then:
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "true"
          - script.execute: controller
          - delay: 60s
          - globals.set:
              id: g_pump_demand_anti_corr
              value: "false"
          - script.execute: controller

Did you test the script above? Do the minute and second entries solve the issue?

Yes, the minutes & seconds addition fixes the issue

A bit late to the party, but I can confirm I also don't have the issue with the minutes and seconds specified.


Generated ShareX config file doesn't have correct extension (.sxcu)

Hey all,
When on the gallery page and I choose 'Generate ShareX Custom Uploader File' it does generate a file that works but it does not have the correct extension at the end of it. If I manually add the .sxcu at the end, it works flawlessly. I've tried a few times and the result was the same every time.
I don't believe I made any changes that would affect this but if you need any information let me know. Just wanted to report this as an issue.
Thanks!

Just wanted to add that I helped my friend set his Uploader up tonight and he is getting the extension so no idea why mine didn't.

That's interesting. I have a theory as to why this could be: the uploader generates the file name based on the page title you set in your config, and if that includes invalid filename characters, that could cause some strange behavior. Would you mind sharing what you have set for page_title or at least the name of the file that gets downloaded? Is the file name cut off in any way?

Yeah it is cut off which is another thing I found odd. I have not changed my page_title at all and neither did my friend. Yet his works and mine does not.
Mine cuts off like so: sharex_My
That is all that is shown when I download the config file.
Also just found the issue, it seems to only be happening in Firefox for me which would make sense why my friend didn't have an issue with it. When I download it through Chrome it shows correctly but when downloading through Firefox it attempts to download this: https://i.imgur.com/tfMbWaB.png

That's really strange. Firefox probably just has an unusual way of dealing with download names that maybe doesn't support spaces. I'll leave a little PR to just make the file name consistent and safe, since we'll have big problems when someone goes and tries to name their uploader ^@&*!(@:*&$#^;&*/(:!@'"^*(.

Also, I just tried downloading the file in Firefox and I can reproduce this bug, so this is definitely an issue.

Awesome. Just wanted to add that I changed my page_title to website_name and I got the correct file this time through Firefox. I believe you are right that it doesn't like spaces in the page_title.


"Move windows based on their position" (Fancy Zones)

As feared, the new feature did not really solve/simplify things for me.
Latest versions, just 3 custom zones in horizontal arrangement.
✔ Expected results

A window is positioned/opened anywhere on the screen - but not in a zone
Expectation: Pressing Win+Left moves it to the leftmost zone

A window's center is left of the leftmost zone
Expectation: Pressing Win+Left still moves/enlarges it to the leftmost zone

I do not expect any wrap-around for the snap keys at all, it is more annoying than helpful/needed

❌ Actual results

Zone depends on current position

Window appears in rightmost zone (Your code thinks the intent is to wrap around)

wrap around happens

In summary, just turning off the wrap-around would be a big plus. I could live with that.
((The only remaining discussion then would be: if a window is pretty far right, but not in a zone, should Win+Left chose the leftmost zone or a better matching zone. I would still prefer leftmost.))

@nuuSolutions
we offer two behavior for Win + arrows

Regarding 3: we want to keep consistency with Windows Snap experience and removing the feature would affect those that use it.


Expand header_widget to include doc path, citation and help text

Description
What is this PR

[ ] Bug fix
[X] Addition of a new feature
[ ] Other

Why is this PR needed?
The header_widget currently doesn't support some of the required features for existing brainglobe headers like cellfinder and brainrender-napari
What does this PR do?
This PR adds new features to the header_widget:

Makes tutorial links optional
Adds option for documentation links
Adds option for citation doi
Adds option for help text
Adds option for github repo name (if not the same as the package name)

References
For https://github.com/brainglobe/cellfinder/issues/327
How has this PR been tested?
All tests pass locally.
Is this a breaking change?
No
Does this PR require an update to the documentation?
Relevant docstrings have been updated.
Checklist:

[X] The code has been tested locally
[X] Tests have been added to cover all new functionality (unit & integration)
[X] The documentation has been updated to reflect any changes
[X] The code has been formatted with pre-commit

Example usage for cellfinder in this draft PR

All fixed now @willGraham01 ! Hopefully the PEP overlords are satisfied :laughing:. Feel free to merge if you're happy with it


Error when importing the downloaded Query Store Dashboard .rdl file

When you download the .rdl file from the GitHub page and import the .rdl into SQL Server Management Studio you are confronted with and error.

The issue occurred because GitHub would add additional information inside the XML of the .rdl file. Instead of making the .rdl directly available we provide the .rdl inside a ZIP file so GitHub is unable to alter to contents of the .rdl.


Fix Stretched Homepage Discover Cards Image For Express International

Fixes following issues|Adds following features:

Issue/Feature 1

Resolves: MWPW-162755
Before:
https://www.stage.adobe.com/kr/express/

Pages to check for regression and performance:
After:

https://intl-homepage-images--express--adobecom.hlx.page/kr/express/?martech=off

https://intl-homepage-images--express--adobecom.hlx.page/express/?martech=off
No regression for US site

@JingleH I tried removing the width completely but without it set to auto - the aspect ratio of the image is skewed.


fix typo

pull request : issue #15

+100!

@nxhack thanks for this. Can you please sign the CLA? https://github.com/elasticsearch/logstash/blob/master/CONTRIBUTING.md#contribution-steps

@suyograo After I have seen this error message, I was signed CLA soon. :)
Thank you.


Fix riak_test setup race condition

This has been resident since 59f7e32 . Blame me because I was inappropriate.

+1 e658a77

@borshop merge


Unable to start APIM 3.1.0 with JVM 11.0.20 version

Description
When we start the APIM 3.1.0 pack with the latest update level with JVM version 11.0.20, we get OSGI bundle resolving error.
Steps to Reproduce
Get 11.0.20 JVM and set it as the default JVM
Start the APIM 3.1.0 pack and you will see the errors
Affected Component
APIM
Version
3.1.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

The identified issues have already been fixed in the master branch.


#1: add support for horses in database initialization

new support for horse

/paulo - can you please review this pull request?

/paulo - can you check now?


.Net Core is not supported (yet)

Sorry, this isn't quite the right place to ask this, but it might turn out to be a documentation issue, so... :-)
I'm missing a really obvious step.  Axocover is saying 'Hmm.. AxoCover found no unit tests...' in the Tests panel, but I do have some in the Solution - as per screenshot.  Using Xunit, if that makes any odds.

Current release version only supports MSTest, Latest prerelease version supports XUnit. Go to https://github.com/axodox/AxoTools/releases and check latest xNUnitSupport build.

Thanks, Péter.  The latest version recognises the tests and can run them; but 'Cover' gets stuck on 'Initializing test runner...'  Let me know what information I can send you if you need it.

I used version VS2017Support-1.1.57, however no Xunit tests are found.
Note that I'm using a netcoreapp1.1 for the tests.

.Net core is not yet supported. It will come depending on how much free time I have, which is not much as of now.

@craigseymour I have made some significant changes which addressed issues resulting in same behavior as you described. Can you try latest xNUnitSupport release? Maybe it helps you too...

Using xNUnitSupport-1.1.68 in VS2017 no tests are found for this project: https://github.com/aspnet/Microsoft.Data.Sqlite which uses xunit tests.

Json style projects and .Net core is not supported yet, it is planned after next major update.

Currently OpenCover only support full pdbs - https://github.com/OpenCover/opencover/issues/601
We do have wip that supports portable and we are looking at supporting embedded PDBs (once we have the later working then we will look to make a new release)

Great, by that time I should also have the xUnit, NUnit, MSTestV2 changes finished and released, so I can work on the .Net Core support. This will also require changes in AxoCover since the VS API to query these projects is different and I also need to make my test host .Net Core capable.

xUnit stuff is now working great for us (bar some non-fatal errors about not being able to find runner.json files that we're not using).  Thanks, Péter.

I will comment that if it would be interesting. I had problems with an assembly that is .net standard, because it wasn't covered by the tests. This is because in the properties of the project, in build options, in the button of advanced options, the PDBs was set as portable. When I changed that to full, then the assembly was added to the results of the tests.
Perhaps with .net Core it happens the same, that you have to change the PDBs to full.

Ms just released netstandard/core 2 which is much more complete and stable, so hopefully we'll see Core support soon... ☺

I found the same problem because I am using the netcore2.0.
Hope for this festure.

xUnit stuff is now working great for us (bar some non-fatal errors about not being able to find runner.json files that we're not using). Thanks, Péter.

Is that also working for .NET Standard 2 for you? I got this:

Hi, I am sorry to say it, but I decided to hold off the .Net Core work for now.
Unfortunately I really don't have the free time required to support so many test frameworks and platforms by myself while they are keeping changing (along with Visual Studio updates). So I will stick with supporting what I actually need at work and some of the ones people ask which are still fit the timeframe I am willing to put into it.
If you have more time than me, I am willing to give help to support .Net Core in AxoCover. So please tell and I will brief you what has to be done.
There is a chance I will work with .Net Core next year, if that happens I will likely bring support for it at that time (might be half to one year from now).

This is a duplicate of #9. Would recommended to close one of the issues.

Hi @axodox. In reference to your comment, can you describe the detail and scope of the work that needs to be done in order to fully support .NET Core 2.x based on .NET Standard 2.0? If it's inconvenient to do so here, please send to me at dexter[dot]valkyrie[at]outlook.com.
Thanks!

@axodox will this https://github.com/tonerdo/coverlet/ be helpful? Problem with OpenCover is it does not use portable pdb.

Seems like it's not coming 😔


Bootstrap agent in windows environment

related to https://github.com/rancher/rancher/issues/10442
Enhance the agent setup experience in windows environment.

The agent's binary will be download from rancher server
Set host labels and agent IP to windows environment variable when bootstrapping agent.
Setup and start agent service in windows

Merged into Cattle.
Closing as testing will be for overall windows experience and new issues will be filed.


Update PR in CONTRIBUTING.md

I feel confused about "a pull request, or PR" in CONTRIBUTING.md.
I thought they were different things at first, but later I figured out they were the same thing. I suggest updating it to "a pull request (PR)" to avoid ambiguity.

Please sign off your commit according to DCO standards and we can merge this :smile:

@cybette I fixed up the DCO signoff, thank you.


lorri: unstable-2020-01-09 -> 1.0 (backport to release-19.09)

Cherry-picked from commit 42a2668eb07b169264bca841a1ae31a11452b178.

Motivation for this change
Backport of https://github.com/NixOS/nixpkgs/pull/79230. We want lorri 1.0, the first stable version, in NixOS 19.09.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build lorri.tests


Strange warnings in Code::Blocks 17.12, GNU GCC

I'm not sure exactly what's causing it to happen but when I compile my project I often get these warnings:
||=== Build: Debug in Stellar Winds (compiler: GNU GCC Compiler) ===|
C:\C++ Projects\Stellar Winds\include\nlohmann\json.hpp||In instantiation of 'std::__cxx11::string nlohmann::detail::lexer<BasicJsonType>::get_token_string() const [with BasicJsonType = nlohmann::basic_json<>; std::__cxx11::string = std::__cxx11::basic_string<char>]':|
C:\C++ Projects\Stellar Winds\include\nlohmann\json.hpp|4309|required from 'void nlohmann::detail::parser<BasicJsonType>::parse(bool, BasicJsonType&) [with BasicJsonType = nlohmann::basic_json<>]'|
C:\C++ Projects\Stellar Winds\include\nlohmann\json.hpp|16931|required from 'static nlohmann::basic_json<ObjectType, ArrayType, StringType, BooleanType, NumberIntegerType, NumberUnsignedType, NumberFloatType, AllocatorType, JSONSerializer> nlohmann::basic_json<ObjectType, ArrayType, StringType, BooleanType, NumberIntegerType, NumberUnsignedType, NumberFloatType, AllocatorType, JSONSerializer>::parse(IteratorType, IteratorType, nlohmann::basic_json<ObjectType, ArrayType, StringType, BooleanType, NumberIntegerType, NumberUnsignedType, NumberFloatType, AllocatorType, JS|
C:\C++ Projects\Stellar Winds\include\nlohmann\json.hpp|18544|required from here|
C:\C++ Projects\Stellar Winds\include\nlohmann\json.hpp|3366|warning: unknown conversion type character 'h' in format [-Wformat=]|
C:\C++ Projects\Stellar Winds\include\nlohmann\json.hpp|3366|warning: too many arguments for format [-Wformat-extra-args]|
||=== Build finished: 0 error(s), 6 warning(s) (0 minute(s), 3 second(s)) ===|

I'm using the single header version with sfml version 2.5.0, everything else is vanilla C++.
I was able to "fix" this a few times before by removing extra includes in my classes but now it remains more constant. My project compiles and runs fine with the warnings but they're annoying and I'm not sure if they're too important. Is this normal and/or can it be fixed?

Are you using MinGW?

yes

Then I guess MinGW does not implement snprintf properly.

Maybe related: https://stackoverflow.com/questions/10678124/mingw-gcc-unknown-conversion-type-character-h-snprintf

It seems we do not need the h specified in the first place.

Yeah, sorry I forgot to check back here. I'll have a response in about 11 hours

Works great, mingw builds it with no warnings/errors

Thanks for checking!

I'm seeing similar warnings when using MinGW again:
json.hpp:3780:105: warning: unknown conversion type character 'h' in format [-Wformat=]
(std::snprintf)(cr.data(), cr.size(), "%.2hhX", static_cast(element_type));

I'll check.

@jacob-keller Could you please check whether the warning disappears in branch https://github.com/nlohmann/json/tree/issue1192?


Assertion failure when calling reflect with Proxy target

The following code causes an assert:
function shouldBe(actual, expected) {

}

{
    let target = {};
    let handlers = {
        get: function(theTarget, propName, receiver) {

            shouldBe(receiver, 1);
            return 42;
        }
    };
    let proxy = new Proxy(target, handlers);
    shouldBe(Reflect.get(proxy, 0, 0x777777), 42);
}

The incorrectly casted value is never used, so this doesn't cause a crash in release.
ASSERTION 22521: (/home/user/test_everywhere/ChakraCore/lib/Jsrt/../Runtime/Types/RecyclableObject.inl, line 29) Ensure instance is a RecyclableObject
Failure: (Is(aValue))
Illegal instruction (core dumped)
proxy.txt

Confirmed repro in debug build on Windows.


Schematics doesnt show on servers?

So i imported an schematic via nbt file at a server. But it doesnt show a ghost mode where i can watch the building so i can properly rotate and stuff.

Thanks for reporting, I'll need some more details, though.

Have you deployed it already? (info)
Was it given to you by another player?
Is it rendering the boundaries?

-Its not deploying sadly but it says syncing when right clicking and loads for some seconds.
-Made it myself
-Not even boundaries show up

i see some little boundaries. But the schematic is way bigger ^^

How did you create the Schematic, was it with the in-game item or through other means?
Can you track down the file in your instance folder?

yea it was a schematic from another world. On the server there is a new folder called schematics where a nbt file is placed. Exactly the one i uploaded via schematic table.

Yes to all 3 points. I think i found the issue. The schematic i tried to copy is from 1.13 world. Could that be the problem?

Yep that happened. It actually worked now ty


Websocket fallback

Sadly in some cases websocket connections are not possible (eg. corporate proxys). It would be create if a fallback existed.

@th3noname this is not possible. Cloud IDEs rely too much on Websockets since they are the implementation of the WebRTC specs. So no, engineering-wise this is impossible to make a fallback just to support such.
Recommending a close as wontfix. code-server relies on WebRTC and is impossible to support such fallbacks.

Then ask your IT to enable websockets, use a VPN or ssh proxy.

Cool tips: if you have a VPS/dedicated server you could bind OpenVPN on 443 and add a little line in the config file to check whether the call is for him or for your HTTPS webserver. This lets you go through most proxies without sacrificing your website

@sr229 I expected that answer but wanted to try anyway.
Sadly non of the provided workarounds would work in my case. Tried talking to our CEO and gave up. And all HTTPS connections are intercepted and non HTTP traffic is blocked.
Thanks for the answers!

So they are doing MITM. I expect in a few years every https website will break due to this and it is a very big security risk if the machine ever gets compromised.
Did you try https over port 80? Did you check ports in the 65ks? Worst case just use your mobile data.
Anyway I am getting way of topic here.

@th3noname you can add the --allow-http flag to enable http connections.

Going to close this as the browser doesn't provide a great alternative.


Attempt at adding support for cudnnr5 API only (no LSTM/RNN additions

yet).
Tested only on R5 on a subset of tests so far.  Have not yet
tested that it doesn't break R4 -- Jenkins tests should cover
that.

Jason from Stream Executor looked at this and gave his LGTM, so merging.


Update MongoHealthCheck for new Mongo driver version

MongoHealthCheck uses the Mongo com.mongodb.DB which exists in the Mongo 3.x driver but not in the 4.x and higher versions.
It needs to be replaced, which changes the public API of this class, specifically the constructor.

See the following migration guide for the Mongo driver:
https://www.mongodb.com/docs/drivers/java/sync/current/legacy/#api-changes

After some initial investigation, I found that the replacement for DB is MongoDatabase, which doesn't have some of the features the health check uses. Specifically, it does not have an equivalent to the getMongo() method, which we used to obtain the Mongo host/port information:
var host = db.getMongo().getAddress().toString();

We also used the getStats() method which returned a CommandResult object. Neither this method nor this class exist anymore in the new Mongo driver. The getStats() method did this:
public CommandResult getStats() {
  BsonDocument commandDocument = new BsonDocument("dbStats", new BsonInt32(1)).append("scale", new BsonInt32(1));
  return executeCommand(commandDocument);
}

We used it to call the ok() method which returns a boolean to determine if healthy or not. We also used it to obtain the "storageSize" and "dataSize" properties which we included in the details of the health check results. And finally, if there was an error, we called the getErrorMessage() method to obtain an error message.
If we want to retain the same or similar functionality, we need to determine how to replicate this functionality. We can of course copy the code to create the BsonDocument for the dbStats command. And the MongoDatabase class does contain a bunch of overloaded runCommand methods some of which return a Document and some which return a generic type that you specify in an argument.
Basically, we just need to figure out how to execute the dbStats command as described in the Mongo documentation. With this new and "improved" MongoDatabase, it looks like it will be more work and custom code, not less, unless I can find that maybe they just moved all this functionality somewhere else. I am not counting on that, however.

However, after reading more about the dbStats command, I'm not sure if we should replicate all the functionality that we had been using. One concern is this:

The time required to run the command depends on the total size of the database. Because the command must touch all data files, the command may take several seconds to run.

We don't want to be doing this every time this health check is called. If we really want this functionality, then we should probably cache the storage and data size information for some (possibly configurable) amount of time. While the cache is not expired, we simply return the last known values. No idea what a good default would be for the cache expiration, e.g. maybe 10-15 minutes?
Another concern is this regarding "unexpected" shutdowns:

After an unclean shutdown of a mongod using the Wired Tiger storage engine, count and size statistics reported by dbStats may be inaccurate.

So, the stats themselves might not even be accurate.
For the health check to be meaningful, we need to find something to do to validate that the Mongo database is available. We'll need to do more research into alternative commands we could use. A good starting point might be Diagnostic Commands. For example, a few commands we might consider include hostInfo and ping and serverStatus.
@chrisrohr and/or @terezivy - any thoughts on all this?

If all we want is the status of the server, then the ping command should be good.  I agree, we don't want to add process intensive healthchecks.

Here's a reference to running commands in MongoDB:
https://www.mongodb.com/docs/manual/tutorial/use-database-commands/
In the Command Responses section, it only mentions that there is always an "ok" property defined as:

A number that indicates whether the command has succeeded (1) or failed (0).

Here's the result of running a successful ping command:
{
  "operationTime": Timestamp(7088351293026074, 1),
  "ok": 1.0,
}

In the previous Mongo 3.x driver there is a wrapper class, CommandResult, that encapsulates some of the logic to determine success or failure. A few of the key methods are ok() which returns a boolean, and getErrorMessage() which returns a nullable String.
ok() is defined as:
    /**
     * Gets the "ok" field, which is whether this command executed correctly or not.
     *
     * @return true if the command executed without error.
     */
    public boolean ok() {
        Object okValue = get("ok");
        if (okValue instanceof Boolean) {
            return (Boolean) okValue;
        } else if (okValue instanceof Number) {
            return ((Number) okValue).intValue() == 1;
        } else {
            return false;
        }
    }

I'm not really sure why they are checking if the value was an instance of Boolean if the result is always either zero or 1, unless maybe earlier Mongo versions might return different values for ok.
And getErrorMessage() is defined as:
    /**
     * Gets the error message associated with a failed command.
     *
     * @return The error message or null
     */
    @Nullable
    public String getErrorMessage() {
        Object foo = get("errmsg");
        if (foo == null) {
            return null;
        }
        return foo.toString();
    }

By executing various commands in actual Mongo databases, I've observed the following:

ok is a Double with value 0.0 or 1.0
errmsg is a String
when errmsg exists, there are two more properties: code and codeName which describe the error code and name, respectively.

Example of a bad command:
db.runCommand( { crash: 1 })

with result:
{
  "operationTime": Timestamp(7088351293026074, 1),
  "ok": 0.0,
  "errmsg": "no such command: 'crash'",
  "code": 59,
  "codeName": "CommandNotFound"
}

So, our health check can just use the ping command and replicate the logic in the CommandResult class. It can also include the additional code and codeName details if the ping fails.


Add String.number_format() method for more readable large numbers

This is used in the editor to display large numbers such as in the View Information panel.
TextServer.format_number() automatically uses this method with the current language (if no language is specified), which benefits many locations in the editor such as the inspector, performance monitors and the 3D editor View Information panel.
Unit tests have been added for both String.number_format() and TextServer.number_format().

This closes https://github.com/godotengine/godot-proposals/issues/9289.

Preview
English

French

Does not work well in the inspector:
https://github.com/godotengine/godot/assets/2223172/c3d918fb-64a9-4f7c-b2e9-1ebb95e08197
Double click will select parts separately and the commas are treated as decimal separators, so if you add an actual separator, the result is invalid.
In space-separated locales, the decimal part is simply rejected.
https://github.com/godotengine/godot/assets/2223172/c0e04f12-ee53-4926-8628-4ae8996a758b

Related on the editor side I think:

https://github.com/godotengine/godot/pull/24532

Somewhat related on the editor side I think:

https://github.com/godotengine/godot/pull/80541

Does not work well in the inspector:

I recall it working correctly in my testing (the displayed number during editor didn't have any separators). I think it should work this way preferably, but we should make it so EditorSpinSlider and SpinBox accept numbers with various separators. To handle situations where we have both , and . separators in the same string, the decimal separator is the one that comes last in the string.

TextServerAdvanced::_format_number isn't doing anything like this (adding separators), it's only converting western Arabic numerals to the language specific numeral system. So it should be either modified to do so or not use it as fallback.


Admin panel "primary leaderboard" tweaks

The current system is pretty confusing. The default behavior of the "primary leaderboard" field on Projects allows admins to select from any existing leaderboard.
It would be great if it would only allow admins to select from Leaderboards which have "this project" as their project.
Also, probably add a constraint on the database level where primary leaderboards can only be selected if the Leaderboard.project is said project.

Also would like to add this request from our slack thread: https://metaculus.slack.com/archives/C01Q9AQBVHB/p1733951548602969?thread_ts=1733951101.929149&cid=C01Q9AQBVHB
The request is to make it the default behavior that all question series and tournament projects have a leaderboard created by default and show that leaderboard by default.


Test Explorer not showing up when you open 'news'

Environment data

VS Code version: 1.33.0-insiders
Extension version (available under the Extensions sidebar): 2019.3
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Python 3.7.0
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: pytest 4.3.1

Expected behaviour
Beaker for Test Explorer to show up.
Actual behaviour
No beaker.
Steps to reproduce:

Apply https://github.com/Microsoft/vscode-python/pull/4981
Open the news/ directory as your workspace
Discover tests (it will find 14 as denoted in the status bar)

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
N/A

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
console.ts:134 [Extension Host] Python Extension: Cached data exists ActivatedEnvironmentVariables, c:\Users\brcan\Repositories\vscode-python\news (at r.logInformation (C:\Users\brcan\.vscode-insiders\extensions\ms-python.python-2019.3.6139\out\client\extension.js:1:18417))
console.ts:134 [Extension Host] Python Extension: getActivatedEnvironmentVariables, Class name = b, Arg 1: <Uri:c:\Users\brcan\Repositories\vscode-python\news>, Arg 2: undefined, Arg 3: undefined (at r.logInformation (C:\Users\brcan\.vscode-insiders\extensions\ms-python.python-2019.3.6139\out\client\extension.js:1:18417))

@brettcannon
Unable to replicate this issue. I just opened it and that's it, absolutely no changes (using auto selected pipenv, etc...)

Unable to replicate on windows either. Lets try this together.

Are you still experiencing this issue?
Could be some flaky issue related to test discovery in unit test

Problem was the Test Explorer was hidden from the activity bar.


Vue-feathers / handsontable and websockets?

Hi,
I am trying to wire up handsontable with vue. I am using feathers on the backend, and I am using vue-feathers to access my service.
   this.$feathers
      .service("teams")
      .find()
      .then(d => {
        this.users = d.data;
      });

the data is there, but if I put it into hottable:
<HotTable root="root" :data="users" />
It doesn't work. I am new to websockets, is there some lifecycle hook I need to do before I load the websocket data?

If anyone else sees this, it is confusing but the doc says community edition can take an array of objects which it cant. Simple mapping to arrays fixes:

  mounted() {

    let vm = this;
      this.$feathers
        .service("teams")
        .find()
        .then(d => {
          //vm.hotdata = d.data;
          var data = d.data.map((o) => Object.values(o) );
           vm.$refs.testHot.table.loadData(data);
           //vm.$refs.testHot.table.render();
        });
  },


JCUnit fails to compile depending on JDK patch version

(t.b.d.)

Looks like it is still actual - latest code fails to compile on Java 11 latest build on Windows (11.0.1)
Here is the fix (for com.github.dakusui.jcunit8.pipeline.Pipeline):
        @SuppressWarnings("unchecked")         private Parameter toSimpleParameterIfNecessary(Config config, Parameter parameter, List<Constraint> constraints) {             if (!(parameter instanceof Parameter.Simple) && isInvolvedByAnyConstraint(parameter, constraints)) {                 List<Object> values = Stream                         .concat(                                 (Stream<Object>)parameter.getKnownValues().stream(),                                 engine(                                         config, new ParameterSpace.Builder().addParameter(parameter).build())                                         .stream().map(tuple -> tuple.get(parameter.getName())                                 ) // Extraction                         ).collect(toList());                 return Parameter.Simple.Factory.of(                         Utils.unique(                                 values                         ))                         .create(parameter.getName());             }             return parameter;         }

Thanks a lot! I will try this fix on my linux box. Please give me output of  java -version.

Thanks!
Version that I've used for compilation (JDK in IntelliJ Idea project settings) is:
  java version "11.0.1" 2018-10-16 LTS
  Java(TM) SE Runtime Environment 18.9 (build 11.0.1+13-LTS)
  Java HotSpot(TM) 64-Bit Server VM 18.9 (build 11.0.1+13-LTS, mixed mode)

This is Windows x64 version. I have downloaded it yesterday from Oracle.

Sorry for keeping you wait.
I'm a bit busy right now and will be working on this early next week.
Thanks again!

No problems at all: I have added that comment just in case if anybody else runs into same issue...

@mike-d-davydov , I realized that this can only be reproduced with JDK 11, not with JDK 8, which should be used to develop JCUnit8.
The reason to use Java 8 is because JCUnit is intended to be used various software products for various purposes and it should require dependencies and preconditions as less as possible.
Once compiled and shipped, JCUnit should be able to run under Java 11 (if not, file a bug please), but to build it, Java 8 must be used. Otherwise we can introduce dependencies or make assumptions that cannot be met by Java 8 based projects.
That being said, Java8 will not last forever and it will be replaced by Java 9, 10, 11, or more sooner or later. To start discussing such a future transition, I have filed this ticket:  #139 . (Java doesn't guarantee 100% compatibility when major version is incremented, and this sort of things have happened from time to time in history)
If the fix helps you in some way I still don't realize or you have any reason not to be able to use Java 8, please let me know, I will consider integrating the fix into development branch again.
And Thank you again for your feed back!


Combining extensions in stats._boost into one

After gh-15770 I noticed another wheel size increase. With the current codegen it's easy to add new functions, but it looks like a separate new Python extension per function isn't all that sustainable:
$ ls -l build/scipy/stats/_boost/
...
-rwxr-xr-x  1 rgommers  staff  403276 Jul 12 07:13 beta_ufunc.cpython-39-darwin.so
-rwxr-xr-x  1 rgommers  staff  358557 Jul 12 07:13 binom_ufunc.cpython-39-darwin.so
-rwxr-xr-x  1 rgommers  staff  267553 Jul 12 07:13 hypergeom_ufunc.cpython-39-darwin.so
-rwxr-xr-x  1 rgommers  staff  366350 Jul 12 07:13 nbinom_ufunc.cpython-39-darwin.so
-rwxr-xr-x  1 rgommers  staff  337707 Jul 12 07:13 ncf_ufunc.cpython-39-darwin.so
-rwxr-xr-x  1 rgommers  staff  375308 Jul 12 07:13 ncx2_ufunc.cpython-39-darwin.so
# Note; removed directories from output

That's ~350kb per function. A lot of which will be Cython overhead I expect. @mckib2 what do you think about putting them all in a single _ufuncs extension?

As @mckib2 suggests in https://github.com/mckib2/scipy/pull/33#issuecomment-1556097520, it's probably better to get rid of the separate wrapping of Boost functionality in scipy.stats, and get everything needed from scipy.special.

In general, I think this is worth doing. I can take it up. I will have to look into scipy.special first. :-)

As per https://github.com/scipy/scipy/issues/20208#issuecomment-2027547367 (with some decisions to be made on https://github.com/scipy/scipy/issues/20208#issuecomment-2027606448) I think this is worth doing and I will be start doing this from Monday onwards.

I have a question. For the following, why don't we use direct results from here. Mean, skewness, kurtosis and variance have closed form expressions for beta distribution, so why calling into _boost? Can we return the closed form expressions directly?
https://github.com/scipy/scipy/blob/8b22ba9a4495c871dd129722ac164c2c841f6499/scipy/stats/_continuous_distns.py#L702-L707


Meteor 1.3.1 - 'meteor test' with server TAPi18n.__() call fails with "TypeError: Object [object Object] has no method 'en'"

I am working on a Meteor 1.3.1 project, and use meteor test for my unit testing.
I just added the tap:i18n package, but unfortunately in doing so, it causes meteor test to crash if I use the TAPi18n.__() method on the server. The stack trace is shown below.
To reproduce the problem, please see the following repository<EMAIL_ADDRESS>In particular, the line that triggers the problem during testing is in server/main.js:
let appName = TAPi18n.__('app.name');
meteor run works fine, and the translated app name is printed to the server console.
However, meteor test fails with the following stack trace:
TypeError: Object [object Object] has no method 'en'
W20160418-13:36:34.935(10)? (STDERR)     at [object Object].__ (packages/tap_i18n/lib/tap_i18n/tap_i18n-server.coffee:26:16)
W20160418-13:36:34.935(10)? (STDERR)     at myServerFunc (server/main.js:6:27)
W20160418-13:36:34.935(10)? (STDERR)     at server/main.js:14:5
W20160418-13:36:34.935(10)? (STDERR)     at /tmp/meteor-test-runkihh1b/.meteor/local/build/programs/server/boot.js:290:5

Unfortunately this is still an issue after upgrading to meteor <IP_ADDRESS>

just letting you know that I got the same issue and NO solution.
At least after 2 hours waisted in debugging and searching for a solution I finally found this topic and now I know that I am NOT the only one.
it seems like the json files are simply NOT loaded/recognized when being in unit-test-mode.
You can see that NO languages have been loaded by looking at the output:
    console.log('TAPi18n.getLanguages()')
    console.log(TAPi18n.getLanguages())
    console.log('TAPi18n')
    console.log(TAPi18n)
```

shows:

I20160522-16:43:01.439(2)? TAPi18n
I20160522-16:43:01.440(2)? { domain: null,
I20160522-16:43:01.440(2)?   _events: {},
I20160522-16:43:01.440(2)?   _maxListeners: 10,
I20160522-16:43:01.440(2)?   _fallback_language: 'en',
I20160522-16:43:01.440(2)?   _language_changed_tracker: { _dependentsById: {} },
I20160522-16:43:01.440(2)?   _loaded_languages: [ 'en' ],
I20160522-16:43:01.440(2)?   conf: null,
I20160522-16:43:01.440(2)?   packages: {},
I20160522-16:43:01.440(2)?   languages_names: {},
I20160522-16:43:01.440(2)?   translations: {},
I20160522-16:43:01.441(2)?   server_translators: {},
I20160522-16:43:01.441(2)?   __: [Function] }
I20160522-16:43:01.444(2)? TAPi18n.getLanguages()
I20160522-16:43:01.444(2)? null

+1 same here, Meteor <IP_ADDRESS> and it stoping me from testing server methods

My app won't even run due to this same issue. Is there an intended fix?

You need to make sure you import your translations file. In (non-full-app) test mode, your translations file won't automatically get imported (i.e. no eager loading).
You could add:
// import this
import '../i18n/en.i18n.json';

// before calling any translations
import { myServerFunc } from './main.js';

Or, probably better, you put your translation in imports/, and explicitly import your translations from main.js.

@tmeasday how can I make
import '../i18n/en.i18n.json';
work?
It throws me an error:  Cannot find module.
Of course I setup proper path to my translation file.

@poltorak that solution doesn't fix the problem in the linked reproduction app (if you add it to server/main.js)?

@tmeasday Yes, this solution fix the problem in linked reproduction but my problem is quite different. I have package (with no traslation, I got only one global translation in i18n dir) and it brokes when I try to test package with command meteor test-packages. Without test (in normal mode) it works fine. Should I import some test translation file? Or maybe there is any other way to resolve that?

@poltorak, yeah if the translations are defined in the app and you run test-packages, there's no way the translations will come across.
In the Todos example we stub out {{_ to help with a similar problem: https://github.com/meteor/todos/blob/master/imports/ui/components/client/lists-show.tests.js#L17-L23
Perhaps that would help? Probably it'd make sense for this package to include a way to "mock" a language for testing.

@tmeasday Thank you very much. I already use this solution for testing on client but on server I use try catch for now to prevent crashing app. I thought maybe there's some other workaround :)


Avoid unnecessary message_edit_form element in messages that are not being edited

message_body.hbs contains the following HTML construction, present for every message regardless of whether it is being edited:

I don't believe there's any plausible reason why we'd need to have two divs in every message for this purpose; we should be able to either move the inner form into the web/templates/message_edit_form.hbs block that gets inserted when a message is edited, or deduplicated.
CZO thread

@zulipbot claim

@zulipbot claim

@prathebaAT I’ll be taking over this issue and will ensure its completion. In the meantime, please feel free to work on other issues labeled https://github.com/zulip/zulip/labels/help wanted or https://github.com/zulip/zulip/labels/good first issue. Thanks!


Update jaeger stable to v1.21.0 on production

Description

v1.21.0 was released on 11/16/2020
update stable on production
updates ci_system_type to use travis-ci-com after .org deprecated
passes trigger builds: https://gitlab.staging.cncf.ci/jaegertracing/jaeger/-/jobs/201868

Issues:
https://github.com/vulk/cncf_ci/issues/341
How has this been tested:

[ ]  Covered by existing integration testing
[ ]  Added integration testing to cover
[x]  Tested with trigger client against

[x]  cidev.cncf.ci
[x]  dev.cncf.ci
[x]  staging.cncf.ci
[ ]  cncf.ci (production)

[x]  Browser tested on staging.cncf.ci
[x]  Have not tested

Types of changes:

[ ]  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)
[x] Version update

Checklist:

[ ]  My change requires a change to the documentation
[ ]  I have updated the documentation accordingly
[x]  No updates required

LGTM on https://staging.cncf.ci/


securityHandlers?

Is there anything that works in a similar way as express openapi security handlers?
I need a solution to apply some kind of granular security

You can add custom OpenAPI property using detail as the following:
new Elysia().get('/', () => {}, {
	detail: {
		security: [
			{
				Bearer: ['bearer']
			}
		]
	}
})


Setting a TravisCI

@GwonGisoo  TravisCI seems not working yet. Could u share the progress on this?

TravisCI is required to '.travis.yml' file on Project root path.
You can see result of TravisCI below:
https://travis-ci.org/naver/ngrinder/builds/124123067
Travis run progress

TravisCI is detect to the new PR or Commit.
If PR or Commit was added, create the build environment to based on 'ROOT/.travis.yml'
When build environment is created, will be execute maven build by 'mvn install -DskipTests=true'.
When maven build is completed, will be execute maven test by 'mvn test'.

※ '.travis.yml' should be create a test evironment for jdk7 and jdk8.

But now occur failed on 'mvn test'
testNotRegisteredDNSLookup(org.ngrinder.dns.LocalManagedDnsTest)  Time elapsed: 0.011 sec  <<< FAILURE!

We need to skip the test.. Just compile.

Fixed


chore: Convert typings to mypy

ref: https://github.com/lyft/amundsen/issues/560.
Summary of Changes
Convert (most) existing typings from comment to annotation style, and add missing type annotations wherever they exist. Just the same as the other Amundsen repos, we now require functions to declare their argument and return types.

I generally favored the minimal change necessary to get tests to pass. I didn't TODO any because I felt the added noise would be worse than too-weak typings
There were some legit bugs: formatting strings that weren't complete, etc. Tried to use my best judgement for these.
There are some spots where adding TypeVars would make the typings more accurate, but it was Any before, and so Any it will stay for now.
There were a lot of missing type signatures in args and return values. For the vast majority, I tracked down the correct type and included it. I didn't add many "bad" Anys
There were a lot of places with incorrect types. For example, there were some places labeled Iterator where callers were expecting to be able to call len, meaning the type signature was incorrect. I fixed these.
I had to add some None guards assertions to tests so that the compiler knows the nullness of some variables and doesn't complain about expected and actual type mismatches
I didn't add any casts, instead I refactored those sections to use try blocks
The BigQuery inheritance chain got a bit mixed up, and needed some surgery, I split that into a commit
There was some dead code that referenced non-existent class members. I removed it. I'm assuming there's not any crazy dynamism going on (even if there was, those methods would have thrown)
I organized the commits for development sanity, not for review. I believe the easiest way to review for both correctness and style is to review the combined diff. But it's a huge diff, and I know it's going to be a huge pain to review carefully. I'm happy to reorganize the commits in any way that makes it easier to review this beast.
I also don't know any way to make this a smaller PR, but open to feedback. I considered structuring it as a progressive sequence with some checks disabled, then enable the check and fix violations. But this was a lot faster, and I felt like the test suite was good enough, and the existing type annotations complete enough, and the failure modes obvious enough, that the combined risk was low enough to merit doing the far-faster option of doing it all at once. But if this is too much to digest, I can break it up differently.

Some things I intend to do immediately after this lands (since they impact so many files, I wanted to do it later to avoid big merge conflicts):

Re-enable unused import option
Remove unused import # noqa: F401 comments
Conversion of remaining comment types into annotations

And some things we should do piece-by-piece but I don't plan to open immediately:

There were some lingering Union[..., None] that can be safely replaced with Optional, but I didn't replace all of them.

Tests

Added mypy to Travis CI
Many existing tests were modified to add typings. I mostly just used Any, since mocks are challenging to type and the benefits are small

Documentation
No documentation changes, the user-facing experience is transparent since mypy is called from make test and CI
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

Rebased onto master, I'm working on getting the test pass in https://github.com/lyft/amundsen/issues/605, will update this once I figure that out.

Codecov Report

Merging #311 into master will increase coverage by 0.03%.
The diff coverage is 82.35%.

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

All tests now pass, after fixing a few lint errors, one error in bigquery caused by the larger-than-typical surgery required, and one error in HttpFailureSkipOnStatus due to me misunderstanding the original intent of the function.
@feng-tao I realized after opening that I actually could have deferred all of the stylistic com2ann stuff until later. I have a few other ideas that could also allow me to break this into a sequence of smaller PRs. I'm more than happy to do that if it would let you land this more easily/safely/faster. But if reviewing in this format works for you, I'll leave it as is.

@dorianj let me know if it is ready to review. Would like to get it shipped first and let others rebase based on the slack conv.

@feng-tao should be ready for review -- let me know if anything is missing

Thanks for the review! I answered the comments, also rebased to make sure it's also tested on Python 3.7 and won't break master when merged. Will push changes tomorrow in response to questions as needed

@dorianj the pr lgtm, could you also rebase the pr with master as I just add github action to fix the pypi upload issue with travis? thanks.

@feng-tao sure thing, rebased onto master

@dorianj  could you add the mypy to github action as well? let me know if you want to do it in this pr or prefer a seperate one.

Thanks @dorianj for making these changes. This is great. Looking forward to your next set of changes :)


Nativescript videoplayer not showing on tns build (ios)

Environment
CLI: 6.1.2
Xcode: 11.1.0
tns-ios: 6.1.0
Describe the bug
When building for ios 13, videos do not show up in the view (but they are there, and they do play, because you can hear the sound of the video). You can get the videos to show up temporarily the following way:
"tns run ios"
When the app opens, and the video isn't displaying, change anything in the file (like on the xml page)
When the app updates and re-opens, the video will be there.
However, any time you close the app manually and reopen it, the video is no longer there (but it plays in the background again).
To Reproduce
build a plain typescript app
add nativescript-videoplayer plugin
add to your xml file:
 <VideoPlayer:Video id="nativeVideoPlayer" controls="false" autoplay="true" height="300" src="https://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4" />
tns run ios
You will see that the video is not there, but you can hear it playing.
Change anything in the file (like on the xml page)
When the app updates and re-opens, the video will be there.
Close the app manually, and reopen
The video (again) is not there, but you can hear it playing.

@hetd54 -
Is this an iOS 13 device?
Which device?
which version of iOS?

Yes this is an iOS 13 device (auto update was unfortunately on)
iPad Air 3
Version 13.1
On Tue, Oct 29, 2019, 1:35 AM Nathanael Anderson<EMAIL_ADDRESS>wrote:

@hetd54 https://github.com/hetd54 -
Is this an iOS 13 device?
Which device?
which version of iOS?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nstudio/nativescript-videoplayer/issues/158?email_source=notifications&email_token=AIP5B2NJTOYRNIPUAPMSRN3QQ7DSTA5CNFSM4JF32ZH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECPJM7Q#issuecomment-547264126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIP5B2N663IFHGKVXBJFCH3QQ7DSTANCNFSM4JF32ZHQ
.

Thank you so much for helping me with this issue. I tried your suggestion,
and I get the same behavior as I described above (it works in debug mode
when you make a change, but when you close and reopen or try to build, it
does not work).
On Tue, Oct 29, 2019 at 12:46 PM Nathanael Anderson <
<EMAIL_ADDRESS>wrote:

Did you try adding a width="300" to your code. Different iOS versions
sometimes won't render things properly without a width...
I just tested natiescript-exoplayer (which basically uses the same code as
this for the iOS side) on a ios 12.4 and ios 13.2 device and the video
player worked on all local and remote url's I threw at it.
However, my app does use width & height on it.
Using NS 6.12
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nstudio/nativescript-videoplayer/issues/158?email_source=notifications&email_token=AIP5B2O7CNF3HM2DDBO3X5LQRBSGXA5CNFSM4JF32ZH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECRHZHA#issuecomment-547519644,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIP5B2OQ6XYRMK5CJDCEJBTQRBSGXANCNFSM4JF32ZHQ
.

As an update, I was able to install and run successfully on a device with
ios 12.4.1 installed on it. Therefore, it seems like this only is an issue
with ios 13.
On Wed, Oct 30, 2019 at 4:01 PM Heather Tarr<EMAIL_ADDRESS>wrote:

Thank you so much for helping me with this issue. I tried your suggestion,
and I get the same behavior as I described above (it works in debug mode
when you make a change, but when you close and reopen or try to build, it
does not work).
On Tue, Oct 29, 2019 at 12:46 PM Nathanael Anderson <
<EMAIL_ADDRESS>wrote:

Did you try adding a width="300" to your code. Different iOS versions
sometimes won't render things properly without a width...
I just tested natiescript-exoplayer (which basically uses the same code
as this for the iOS side) on a ios 12.4 and ios 13.2 device and the video
player worked on all local and remote url's I threw at it.
However, my app does use width & height on it.
Using NS 6.12
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nstudio/nativescript-videoplayer/issues/158?email_source=notifications&email_token=AIP5B2O7CNF3HM2DDBO3X5LQRBSGXA5CNFSM4JF32ZH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECRHZHA#issuecomment-547519644,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIP5B2OQ6XYRMK5CJDCEJBTQRBSGXANCNFSM4JF32ZHQ
.

Have you tried the Exoplayer; it does use a slightly different code base.   We also very recently rewrote some of the setup functions to solve a black screen issue.

I have the same problem

problem solved: it does not work if registerElement is put in main.ts, but it work if it is put in the page controller

Hello @NathanaelA I did try Exoplayer and had the same issue.
I spent a LOT of time debugging and I think there were several issues. The problem is mainly with the way NativeScript Layouts-- they seem to work differently with ios 13? It may also be an issue with how I understand NativeScript Layouts -- but previously the way I did it worked fine prior to recently.
I had to stop nesting Layouts within other Layouts in order to get it to work!
Previously, my layout was this:
<Page class="page" loaded="pageLoaded" xmlns="http://schemas.nativescript.org/tns.xsd" xmlns:VideoPlayer="nativescript-videoplayer">

    <ActionBar title="IDS Tablet Study">
    <StackLayout orientation="horizontal" ios:horizontalAlignment="center"
            android:horizontalAlignment="left">
            <Label text="Brown Language and Thought Lab   " class="action-label"></Label>
            <Image src="~/website_browser-icon.png" class="action-image"></Image>
        </StackLayout>
    </ActionBar>
        <FlexboxLayout class="slide" justifyContent="center">
          <StackLayout orientation="vertical" horizontalAlignment="left" verticalAlignment="left">
            <StackLayout class="form" id="instructions" orientation="vertical"
                horizontalAlignment="center" verticalAlignment="center"
                visibility="visible">
                <!--Add your page content here-->
                <Label textWrap="true" textAlignment="center" textDecoration="underline"
                    text="Welcome to the IDS tablet study!