Add ability to bypass ssl certificate verification for Consul HTTPS health checks

Building on top of #388, this adds support for a boolean toggle to skip TLS verification on HTTPS health checks. The prerequisite is the merging of this Consul PR.

Needs rebase and tests fail.

Rebased successfully, however the build will fail until the above mentioned PR in the Consul project gets merged.

Alright then we'll need to wait until that is done.

Hello - just a heads up that the Consul feature that this depends on was merged and released

rebased and ready

This would be great.

In the latest Consul API, TLSSkipVerify was changed from string to bool.
$GOPATH/src/github.com/hashicorp/consul/api/agent.go:
	TLSSkipVerify     bool   `json:",omitempty"`

So the code should be as follow
		if tlsSkipVerify := service.Attrs["check_tls_skip_verify"]; tlsSkipVerify != "" {
			check.TLSSkipVerify = true
		}

@cavemandaveman @josegonzalez How can this PR be progressed so it is merged? We're really keen on this feature.

Could we please get this merged in and a v8 released?  Currently SERVICE_CHECK_HTTPS checks are not supported in v7.

Any progress on this?

bump


Output images all 0 bytes (Ubuntu 10.10)

mbutils compiled without any warnings. mbutil --input file.mbtiles --output dir on a number of valid mbtiles gives me the expected metadata.geojson, but a directory of 0 byte pngs. There is no console output when running the command.

This one is working my mac os x laptop is slow cpu - long delay to produce the png.


customization:  Adding a New Search Tab

followed the README to try to add a new search tab.
Copy pasted the examples in the README (with exception of the icon, I reused an existing one).
result : white screen...

Hi Arno,
can you give us the code snippet you used for adding the new search tab please?

cannot reproduce this


Not support variable batch size?

During graph construction, network should not be able to know the batch size (the first dimension of tensor). I have made some modifications to deal with this problem, not sure if you need to support this feature.

What did you modify to support this?

In netwwork.py

Remove unnecessary reshaping after bias_add
Reshape fc feed_in with shape=[-1, dim]

Then you can feed your network with tf.placeholder(tf.float32, shape=[None, height, width, channel])

Applying your patch, I am able to load LeNet with None in batch_size but failed in testing with caffe-tensorflow/examples/vgg.py
>>> from vgg import VGG16
>>> import tensorflow as tf
>>> x = tf.placeholder(tf.float32, [None, 224, 224, 3])
>>> y = tf.placeholder(tf.float32, [None, 1000])
>>> net = VGG16({'data': x})
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "kaffe/tensorflow/network.py", line 37, in __init__
    self.setup()
  File "vgg.py", line 6, in setup
    .conv(3, 3, 64, 1, 1, name='conv1_1')
  File "kaffe/tensorflow/network.py", line 20, in layer_decorated
    layer_output = op(self, layer_input, *args, **kwargs)
  File "kaffe/tensorflow/network.py", line 108, in conv
    bias = tf.reshape(tf.nn.bias_add(conv, biases), conv.get_shape().as_list())
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_array_ops.py", line 1202, in reshape
    name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/op_def_library.py", line 445, in apply_op
    as_ref=input_arg.is_ref)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 565, in convert_to_tensor
    ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/constant_op.py", line 179, in _constant_tensor_conversion_function
    return constant(v, dtype=dtype, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/constant_op.py", line 162, in constant
    tensor_util.make_tensor_proto(value, dtype=dtype, shape=shape))
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/tensor_util.py", line 343, in make_tensor_proto
    _AssertCompatible(values, dtype)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/tensor_util.py", line 281, in _AssertCompatible
    (dtype.name, repr(mismatch), type(mismatch).__name__))
TypeError: Expected int32, got None of type '_Message' instead.

Thanks for the trick! Do you know which modifications could be made to enable variable image size?

@vodp variable image size will result in non-determined network size, unless you are dealing with fully convolutional neural network. You will need to use pure TensorFlow to define your model.

For posterity: the current version should work fine with variable batch sizes.


Update CHANGELOG.md

Updated with recent content changes

Add new entry to A to Z of NHS health writing for "PMS (premenstrual syndrome)"
Amend entries for "GP surgery" and "seizure or fit" in the A to Z of NHS health writing
Delete mention of wizard behind the curtain from accessibility guidance

Duplicated PR


"ember serve" doesn't work

When following the instructions ember serve doesn't work. I have to do npx ember serve.

Therefore you should install the Ember CLI via npm install -g ember-cli
More info on that on https://cli.emberjs.com/release/

People might be reluctant to install dependencies globally just to try something out. But to be fair it's mentioned in the README that the CLI is required.

Fair enough, I've added the alternative instruction to the readme as well in https://github.com/smessie/FormGenerator/commit/5566d43008c66dda60054e2b088e648f39481a61


Adding CHANGELOG.md

Adding CHANGELOG.md, based on feedback from team.

@AmandaBirmingham Personally, I think everyone on the team should be able to edit the changelog when any change of significance has been made. It's probably a quicker way to get up to speed than going through the closed issues on GitHub.

@AmandaBirmingham, good question! IMOO the highlights of the issues being solved or big decisions that might make an impact in the future.
For example, a release CHANGELOG could become a user email of new features in the release or a simple summary of the big changes.

@charles-cowart, right but note that reviewers can also ask the PR author to add a line in the CHANGELOG; however, for Qiita I normally go over all the closed issued for that given release ...

@antgonza seems like it's a matter of whether you want an continuously updated list, or a summary of changes compiled and published at release. I can go either way; I've done both.

@charles-cowart, I think we should start as a continuously updated list of good to knows, highlights of issues, etc until we have a formal release and then move to changes compiled and published at release but it's your call ...

@antgonza Agreed! I was thinking the same thing, actually.


Fix pthread parallel_for

More correct behaviour. CMake option should work.
Merge after https://github.com/Itseez/opencv/pull/4106

thanks! :+1:


Windows: The input line is too long.

https://github.com/gqmelo/exec-wrappers/blob/98bf5bf17b7f34c5c8a0f5141b2d4e23c0b63319/exec_wrappers/templates/conda/run-in.bat#L10
Maybe break this up.

There is some recursive thing going on!


Using single swagger documentation for different Version of Web Controllers

Hi.
Quite nice and useful solution.
Would it be possible to configurate versioning with swagger to use different API versions But withing one swagger documentation (Open Api specification) ?

Can you elaborate a little on exactly what you mean?
This project doesn't specifically care about or have an specific dependency on Swagger nor any specific implementation libraries such as Swashbuckle or NSwag. What's provided is an API Explorer library that is API version-aware. The API Explorer collates all actions by API version for you. Exactly how that information is used is purely up to you or the consuming library.
The examples provided use Swashbuckle, but that's purely for illustration purposes. If the information isn't collated the way you want, I'd consider alternate arrangements.

Thank you a lot for the feedback.
Sure, will explain in details then:

We configured to use Versioning as header based, so it adds version parameter to all Api calls, like on on configuration below:

services.AddApiVersioning(o => {
	o.ReportApiVersions = true;
	o.AssumeDefaultVersionWhenUnspecified = true;
	o.DefaultApiVersion = new ApiVersion(1, 0);

	o.Conventions.Controller<HomeV1Controller>().HasApiVersion(new ApiVersion(1, 0));
	o.Conventions.Controller<HomeV2Controller>().HasApiVersion(new ApiVersion(2, 0));
});

As understood it will use version parameter to route to the proper method/controller, Right ?
(For example: if passed version 1.0 it will lead to method which marked for version 1.0 etc..)

For the Swagger configuration below will generate swagger documentations (OpenApi specs) for all  versions, Right ?

app.UseSwaggerUI(
        options =>
        {
            foreach ( var description in provider.ApiVersionDescriptions )
            {
                options.SwaggerEndpoint(
                    $"/swagger/{description.GroupName}/swagger.json",
                    description.GroupName.ToUpperInvariant() );
            }
        } );

If Yes, then main question: How to use this version parameter alongside with swagger withing one API endpoint (or Swag. API spec)?
Is it clear or should I add more details ?
Thanks,
Boris
P.S
And yes It may be a question to Swagger also.

Did this get you what you needed or are you still stuck?

@commonsensesoftware Thank you a lot for the feedback. Yeah, a bit stuck. Basically, my goal is combine somehow versioning with only one, single SwaggerDoc.  Exactly like in issue mentioned Here
So, for example methods with V1 and methods with V2 should be listed in one Swagger Documentation.

Great! Feel free to come back if you remain stuck. Thanks.

hi ,
i using asp mvc 5 and api in area difrrent project ,
how to config mulitiapiVersion in swagger

I don't believe there's anything special you need to do if you're mixing UI and APIs together in the same project, even if you're using Areas. To enable Swagger you need the API Explorer package and choose a supported Swagger generator such as Swashbuckle or NSwag. The sample project shows an end-to-end configuration using Swashbuckle.
If you're trying to use nested applications (which is not quite the same as Areas), you might want to peek at #391.
I hope that helps.

Hi @Boriszn
Thank you very much. Could you please provide me with a code?

Hi @Boriszn or @gaoconggit
Can you share you solution on how you managed to display all versions on same swagger document?

@Boriszn I am trying to show two different versions v1 and v2 on same swagger page - having same method name AddRequests().
Can it be possible for you to share your solution to do that?

has anyone solved this?

@Dejavu333 Yes. However, this is only achievable if you are versioning URL segment, which I don't recommend; it's not RESTful. OpenAPI does not support the concept of versioning directly. All endpoints must have a unique path. The only method of versioning that produces unique paths per version is by URL segment. While that might solve one problem, it will come with a litany of other problems, unless you don't care about REST and then you can do as you please.
Asking or elaborating on your use case in a new discussion might bear more fruit. Merging all documents into a single one for the UI will likely be clunky for all but the smallest of APIs. I suspect your use case is different.


use fa-star instead of fa-star-o if the board is starred

Current

Suggestion
Replace fa-star-o with fa-star if the board is starred and remove the background shadow.

Benefits

more obvious
you can see if the board is starred or not even if the icon is hovered

This is done on the newui branch that you can see on http://git.libreboard.com/libreboard/libreboard

Awesome, thanks!


Inputbar tuning

Three added behaviours:

submit disabled when input is empty (incl. button greyed)
placeholder text added
on assistant selection by picker, focuses back on input at end of text

Note: only mentions are considered a valid, non-empty input--since some assistants may be meaningfully called without any more text (e.g. @summarize_conversation, or just @gpt4 after you called claude to get a second opinion). There is no big downside since most assistants will properly answer when not given input (eg. gpt4: "Hello! It seems like you didn't ask a question or provide any instructions in your last message. How can I assist you further?")

This is probably a WIP but just in case:

placeholder does not disappear on agent selection from dropdown
placeholder does not disappear on copy paste in the input bar (we have a custom logic there where you probably want to hook?)

:pray: on it

@philipperolet
Maybe that
var contentEditableElement = document.querySelector('[contenteditable]');

var observer = new MutationObserver(function(mutationsList, observer) {
  for(let mutation of mutationsList) {
    if (mutation.type === 'childList') {
      console.log('A child node has been added or removed.');
    }
    else if (mutation.type === 'characterData') {
      console.log('The content has been changed.');
    }
  }
});

observer.observe(contentEditableElement, { attributes: true, childList: true, characterData: true });

?

i had given it a try but  didn't work out of the box and added some complexity that i'd prefer to avoid if possible
Trying to get a more simple solution, if I fail I'll revisit

@philipperolet
Maybe that
var contentEditableElement = document.querySelector('[contenteditable]');

var observer = new MutationObserver(function(mutationsList, observer) {
  for(let mutation of mutationsList) {
    if (mutation.type === 'childList') {
      console.log('A child node has been added or removed.');
    }
    else if (mutation.type === 'characterData') {
      console.log('The content has been changed.');
    }
  }
});

observer.observe(contentEditableElement, { attributes: true, childList: true, characterData: true });

?

cannot reproduce this part, on my end the placeholder disappears when I paste
is there some specificity to what you pasted to make it break?

placeholder does not disappear on copy paste in the input bar (we have a custom logic there where you probably want to hook?)

Nope just pasting anything there the placeholder remains super-imposed and visible

Multilines

ok multilines does not trigger an onInput (maybe because text is not pasted as is but as multiple nodes in the contenteditable)
fixing

As discussed IRL, switching to MutationObserver, better since setEmpty is confined to the observer rather than sprinkled a bit everywhere


#<RestClient::BadRequest: 400 Bad Request>

➜  abc git:(master): ruby -v
ruby 2.2.6p322 (2016-04-29 revision 54815) [x86_64-darwin15]
➜  abc git:(master): rails -v
Rails 5.0.2
➜  abc git:(master): gem -v
2.6.4
➜  abc git:(master): bundler -v
Bundler version 1.14.6
➜  abc git:(master): gem list google_url_shortener
*** LOCAL GEMS ***
google_url_shortener (1.1.0)
➜  abc git:(master): rails c
:009 > t = 'https://mdbootstrap.com/components/inputs'
=> "https://mdbootstrap.com/components/inputs"
:010 > st = Google::UrlShortener.shorten! t
RestClient.post "https://www.googleapis.com/urlshortener/v1/url?key=xxxxxxxxxxxxxxxxxxxxx", "{"longUrl":"https://mdbootstrap.com/components/inputs\"}", "Accept"=>"application/json", "Accept-Encoding"=>"gzip, deflate", "Content-Length"=>"55", "Content-Type"=>"application/json", "User-Agent"=>"rest-client/2.0.2 (darwin15.5.0 x86_64) ruby/2.2.6p322"
=> 400 BadRequest | application/json 132 bytes
#<RestClient::BadRequest: 400 Bad Request>
NoMethodError: undefined method []' for nil:NilClass 	from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/google_url_shortener-1.1.0/lib/google/url_shortener/url.rb:27:in encode!'
from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/google_url_shortener-1.1.0/lib/google/url_shortener.rb:13:in shorten!' 	from (irb):10 	from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/railties-5.0.2/lib/rails/commands/console.rb:65:in start'
from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/railties-5.0.2/lib/rails/commands/console_helper.rb:9:in start' 	from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/railties-5.0.2/lib/rails/commands/commands_tasks.rb:78:in console'
from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/railties-5.0.2/lib/rails/commands/commands_tasks.rb:49:in run_command!' 	from /Users/jdm/.rvm/gems/ruby-2.2-head@abc/gems/railties-5.0.2/lib/rails/commands.rb:18:in <top (required)>'
from bin/rails:4:in require' 	from bin/rails:4:in '

would be nice to have an error handler rather than always raising, or at least get a better exception..

Agree completely. I think there are a few nice additions that would make the DSL a bit easier to deal with, happy to accept a PR for this but I'm afraid I don't have time to work on this for now 😢

Could this error be related to a recent announcement by Google?

Absolutely, looks like this gem may become less useful...

@joshnesbitt First of all, my apologies that I haven't looked deeply into your code, hence this question.
How difficult would it be to change the underlying REST call from Shortener API to this Firebase API
Even more so, would it be worth it?

I guess you could look at an approach similar to this gem that supports multiple backends. The gem itself is pretty simple so as long as there was a single request to shorten a URL it would be relatively easy.

I started looking more into Firebase, but unfortunately it is not a viable option for just short links. It needs an app to be created, which could either be an iOS or an Android app. So not particularly useful for web apps.
Thanks again Josh for a wonderful gem, pun intended. I will have a look at your suggestion.

Thank you @darkfishy.

Any luck with this? Can you please suggest some alternative.

I don't have an alternative I'm afraid, might be worth looking at other URL shortener services and seeing if they provide Ruby integrations...

Thanks for your quick response. I switch to https://github.com/robbyrussell/shorturl
Thanks for your wonderful library. I used it for 2 years.

Thank you :)


Optimize gp_*_cov functions

Summary
gp_*_cov functions can be optimized by reordering the work they do, so that memory is accessed in blocks. This way the data in CPU cache can be reused multiple times and fewer transfers between CPU and RAM are needed.
Optimal value of block size depends on the amount of cache available. I experimentally optimized it for my CPU. Although, it is not very sensitive, so the value should be fine on other CPUs as well.
I also reimplemented rev overloads using reverse_pass_callback, reducing the amount of code.
I did not benchmark all of the modified functions, but all I did benchmark were faster. The largest speedup I observed was 90%.
Tests
This is just a refactor. No new tests.
Side Effects
None.
Release notes
gp_*_cov functions are faster, especially for large amount of data.
Checklist

[ ] Math issue #(issue number)

[ ] Copyright holder: Tadej Ciglarič
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[ ] the code is written in idiomatic C++ and changes are documented in the doxygen

[ ] the new changes are tested

Just wanted to check and wrote a test you can add to mix
#include <test/unit/math/test_ad.hpp>
#include <vector>

/**
 supported signatures in stanc3
 gp_dot_prod_cov(array[] real, real) => matrix
 gp_dot_prod_cov(array[] real, array[] real, real) => matrix
 gp_dot_prod_cov(array[] real, array[] real, real) => matrix
 gp_dot_prod_cov(array[] vector, real) => matrix
 gp_dot_prod_cov(array[] vector, array[] vector, real) => matrix
*/
TEST(mathMixMatFun, gp_dot_prod_cov) {
  auto f = [](const auto& x, const auto& sigma) {
    return stan::math::gp_dot_prod_cov(x, sigma);
  };

  auto ff = [](const auto& x1, const auto& x2, const auto& sigma) {
    return stan::math::gp_dot_prod_cov(x1, x2, sigma);
  };

  double sigma = 0.2;
  std::vector<double> x_stdreal{-2, -1, -0.5};
  stan::test::expect_ad(f, x_stdreal, sigma);
  stan::test::expect_ad(ff, x_stdreal, x_stdreal, sigma);
  Eigen::VectorXd x_vec = Eigen::VectorXd::Random(2);
  std::vector<Eigen::VectorXd> x_stdvec{x_vec, x_vec};
  stan::test::expect_ad(f, x_stdvec, sigma);
  stan::test::expect_ad(ff, x_stdvec, x_stdvec, sigma);
}

I think if we make the signatures use require_vector_like_t<> a good few of these can also support eigen matrix inputs

Can you share any benchmarks you did? When @bbbales2 and I tried to fix these up we had a pretty big slowdown at small N sizes

Here is one benchmark I actually saved plot of:

I think if we make the signatures use require_vector_like_t<> a good few of these can also support eigen matrix inputs

Adding support for more signatures and more tests is a separate thing and should be a separate PR.

I think if we make the signatures use require_vector_like_t<> a good few of these can also support eigen matrix inputs

Adding support for more signatures and more tests is a separate thing and should be a separate PR.

Yeah that's fine.
If you don't have time I can try to run benchmarks later, but we really need to check small N speed here. That looks like a 10% loss for anything below size 50

I also reran this benchmark multiple times. The slowdown around 50 is just noise, although the slowdown below 5 is real.

What am I calling wrong in the benchmark script below?
./benchmarks/benchmark.py gp_dot_prod_cov --multiplier 2 --max_size 8094 --repeats 10 --csv "gp_dot_prod_cov.csv"

It's giving me back
$ ./benchmarks/benchmark.py gp_dot_prod_cov --multiplier 2 --max_size 8094 --repeats 10 --csv "gp_dot_prod_cov.csv"
make: 'test/expressions/stanc' is up to date.

make ./benchmarks/benchmark
g++ -march=native -mtune=native -std=c++1y -pthread -D_REENTRANT -Wno-sign-compare -Wno-ignored-attributes      -I lib/tbb_2020.3/include   -O3  -I . -I lib/eigen_3.3.9 -I lib/boost_1.75.0 -I lib/sundials_5.7.0/include -I lib/benchmark_1.5.1/googletest/googletest/include -I lib/benchmark_1.5.1/googletest/googletest -I lib/benchmark_1.5.1/include      -DBOOST_DISABLE_ASSERTS        -DGTEST_HAS_PTHREAD=0         -Wl,-L,"stan/origin/math/lib/tbb" -Wl,-rpath,"stan/origin/math/lib/tbb"     -L lib/benchmark_1.5.1/build/src   benchmarks/benchmark.cpp lib/benchmark_1.5.1/googletest/googletest/src/gtest-all.o lib/tbb/libtbb.so.2        -Wl,-L,"stan/origin/math/lib/tbb" -Wl,-rpath,"stan/origin/math/lib/tbb"   -lbenchmark -o benchmarks/benchmark

./benchmarks/benchmark --benchmark_repetitions=10 --benchmark_display_aggregates_only=true --benchmark_out=gp_dot_prod_cov.csv --benchmark_out_format=csv
Failed to match any benchmarks against regex: .

Hmm, that is a bug in bench script. It was incorrectly identifying arrays as having many dimensions, so it skipped them by default. I am pushing a fix.

Also I remembered that I did some changes that could affect performance after the benchmark I posted earlier. So I ran one benchmark and now it looks faster for all sizes.

That's awesome! The code itself looks good let me run the benchmarks locally then I think this is good

I was able to confirm Tadej's findings locally, was just looking at the code. If you say its good I can stop looking :)

Cool cool I'll just check the code then

I don't know what is the issue with your tests. I find the ad tests quite hard to debug. Whatever it is, it does not seem to be from this PR, as the same test is failing on develop. Possibly something is wrong with the test itself.

It could be there's something wrong with the test since the testing framework is not made to handle functions with 4 arguments.

I don't know what is the issue with your tests. I find the ad tests quite hard to debug.

Anything in particular? I've thought about cleanup up a few things there, like for instance I think it would be be nice to have better  debugging style stuff to happen when a test fails like printing out the failing matrices. gtest let's you wait to fail instead of immediately failing which does allow you to collect more debug info before calling the test a failure

Anything in particular?

It is virtually impossible to set a breakpoint so that it breaks only when one (lets say rev) overload is called. Also there is huge amount of code and lambdas, so for me it is hard to follow what certain part is responsible for. Converting arguments into std vectors and back also does not help clarity. Better error messages, such as at which index the error is, would also help.
Now that you mention it, I kinda want to refactor it myself. Possibly merging it with distribution testing framework.

Now that you mention it, I kinda want to refactor it myself. Possibly merging it with distribution testing framework.

Let's open up an issue shopping some ideas
Also sorry for the delay in reviewing this. I just want to check the periodic, rest of the code looks good


New-command: outlook report maillactivityuserdetail

implements #1209

Coverage remained the same at 100.0% when pulling 6aa910bca399a80ae7d8cec1f1876a5288e99e48 on plamber:outlook-report-mailactivityuserdetail into 0e5efd9d93e854d43547e9e1396e619ace74e34d on pnp:dev.

Thank you very much Patrick @plamber ! This is not manually merged in dev.


refactor: admin order API

Moved client order APIs to client.js

@SankhaJay Check the updated files and merge


Tag a new release for Faraday updates

We're still using fcm 0.0.6 because 1.0.2 is locked to Faraday 1.0.0, and we're using Faraday 1.4.1. The gemspec is all updated and it looks like a new release of what's currently on master is all that's required to get fcm working with a current version of Faraday.

Thanks for the prompt all. Released https://github.com/decision-labs/fcm/releases/tag/v1.0.3

This is great. Thank you!


Use docker container (instead of binary) for kubectl and travis cleanup

This fix uses docker container for kubectl. Since Kubernetes docker
image hyperkube has already been downloaded and it consists of
kubectl, there is really no need to download kubectl binary again.
This fix cleans up the Kubernetes related travis setup and removes
unneeded scripts.
This fix also fixes several mismatches of the Kubernetes version used,
so that any changes in version in the future only need to update .travis.yml.

\0/ LGTM

Current coverage is 39.62% (diff: 100%)

Merging #352 into master will not change coverage

@@             master       #352   diff @@
==========================================
  Files           103        103
  Lines          5123       5123
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2030       2030
  Misses         2916       2916
  Partials        177        177

Powered by Codecov. Last update 14dc376...a192010


Fix issue with link from map popup

Map popup does not have a prefix (added by base_path)

Think this is fixed @gilesdring ?

Resolved (temporarily!) by using a relative path, rather than absolute path.
So rather than the popup linking to /place/XYZ, it links to ../place/XYZ
This might break if the tooltip is ever used elsewhere, but works OK for now!


[Bug] get_flops error when cotains carafe op

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (master) or latest version (3.x).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
3.x branch https://github.com/open-mmlab/mmdetection/tree/3.x
Environment
sys.platform: linux
Python: 3.8.16 (default, Mar  2 2023, 03:21:46) [GCC 11.2.0]
CUDA available: True
numpy_random_seed:<PHONE_NUMBER>
GPU 0,1: Tesla P100-PCIE-16GB
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 9.2, V9.2.14
GCC: gcc (GCC) 5.1.0
PyTorch: 1.7.1
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.4-Product Build 20210904 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v1.6.0 (Git Hash 5ef631a030a6f73131c77892041042805a06064f)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 9.2
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.3
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_VULKAN_WRAPPER -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.8.2
OpenCV: 4.7.0
MMEngine: 0.6.0
MMDetection: 3.0.0rc6+5615382
Reproduces the problem - code sample
python tools/analysis_tools/get_flops.py configs/carafe/faster-rcnn_r50_fpn-carafe_1x_coco.py

Reproduces the problem - command or script
python tools/analysis_tools/get_flops.py configs/carafe/faster-rcnn_r50_fpn-carafe_1x_coco.py

Reproduces the problem - error message

Additional information
No response

@zgzhengSEU Thank you for your feedback, this is a known issue and we will fix it as soon as possible.


Use Karpenter on CAPZ

Karpenter can be used on Azure using this upstream repo : https://github.com/Azure/karpenter-provider-azure in self hosted mode.
We should thus adapt the karpenter-bundle app to make it work on CAPZ.

In our GS environment, karpenter is deployed using the karpenter-bundle app which in turns deploy 3 apps :

karpenter-app
capa-karpenter-taint-remover
karpenter-crossplane-resources

So the first step to have it working on CAPZ is to know which of these apps work out-of-the-box in a CAPZ environment (if there are any) and then adapt those that are not accordingly.

After deploying the capa-karpenter-taint-remover app on glean, I noticed that it worked perfectly without having to update the app itself, so nothing to do here except renaming the app.
Issues are arising when it comes to the karpenter-crossplane-resources app. Indeed this app is deploying AWS specific resources  such as IAM and SQS related ones. Therefore, this app definitely needs to be updated to support CAPZ deployment;
Concerning karpenter-app, as the app relies on a <cluster_id>-crossplane-config configmap to be deployed but crossplane is not deployed on glean, there's no such cm in the cluster and thus I couldn't deploy the app. However by removing the extraConfig field related to this one in the app CR it deployed nicely even though the pod was crashlooping (which was expected as it wasn't running in the adequate environment as of the time of testing).

So the remaining things to do are :

[ ]  Rename capa-karpenter-taint-remover app to capi-karpenter-taint-remover as well as all other CAPA mentions inside the app's repo.
[ ] Update the karpenter-crossplane-resources with the resources required to have Karpenter running on Azure

[ ]  Find which custom resources are needed for karpenter on Azure.

[ ] Deploy the karpenter-bundle app on glean using the standard procedure (nodepool + provisionner creation) and make it work.


Having to explicitly set the 'children' prop for EVERY component?

Hi, I use Styled Components all over my app, nested inside of each other many levels deep. I just upgraded to React 18 and now I have 200+ error messages saying I need to explicitly define the children prop for every single styled component that has a child.  Is there an easy to way to get past this issue?

can you paste an example?

Here is my JSX...
export const HeadlineLink = styled.a
color: ${theme.colors.text};
text-decoration: none;
&:hover {
text-decoration: underline;
}
`;

{title}
`
      and i get the error `Property 'children' does not exist on type 'IntrinsicAttributes & {} & { theme?: any; } & { as?: string | undefined; forwardedAs?: string | undefined; }'.`

What version of styled component?
What version of types react/react-dom?
I had similar issue in a project when I upgraded to react 18

@multivoltage @SteveAtKlover
I encountered the same issue, Did you find a solution
I used "styled-components": "^6.1.11"
import styled from 'styled-components/native';

const Container = styled.View({ flex: 1 });

const Item = styled.View({ flex: 1 });

// Usage:
const APP = ({ data }) => {
  return (
    <Container>
      <Item />
    </Container>
  );
};

An error occurs:
No overload matches this call.
The first overload out of 2, “(props: PolymorphicComponentProps<"native", FastOmit<ViewProps, never>, void, void, {}, {}>): Element”, has the following error.
Type “{ children: Element[]; }” is not assignable to type “IntrinsicAttributes & FastOmit<Substitute<FastOmit<ViewProps, never>, FastOmit<{}, never>>, keyof ExecutionProps> & FastOmit<...> & { ...; }”.
The second overload out of 2, “(props: FastOmit<ViewProps, never>): ReactElement<any, string | JSXElementConstructor>”, has the following error.
Type “{ children: Element[]; }” is not assignable to type “IntrinsicAttributes & FastOmit<ViewProps, never>”.ts(2769)
(alias) const Container: IStyledComponent<"native", FastOmit<ViewProps, never>>

I solved playing with different version of react and types :)

I solved playing with different version of react and types :)

What versions?

@doubleSkinMilk
I'm able to write
const Container = styled.div({ flex: 1 });

const Item = styled.div({ flex: 1 });

// Usage:
const APP = () => {
  return (
    <Container>
      <Item />
    </Container>
  );
};

with:
"styled-components": "^6.1.8"
"typescript": "4.9.5"
"react": "18.2.0",
"react-dom": "18.2.0",
"@types/react": "18.2.0",

I encountered the same issue, Did you find a solution
"styled-components": "^5.2.1"
"typescript": "4.6.3"
"react": "17.0.2",
"react-dom": "17.0.2",
"@types/react": "17.0.2",


OAK-10141: relevance on queries involving dynamic boost sometimes differ from lucene

refactoring of full-text tests to share them between lucene and elastic
fix an issue on elastic where full-text queries wrongly assigned the boost factor on specific fields
fix relevance in Elastic when dynamic boost is enabled. The dynamic terms need to be de-boosted (as it happens in lucene). To properly address the issue, a new field has been created. This is not a breaking change since the queries won't complain in case the field is not yet there. A reindex is required to get the correct relevant results.

@nit0906 the list of tests is not the same as before. While refactoring, I noticed duplicate tests that I removed and added new tests. To easily see what was done in the refactoring, you can have a look at the specific commits in this PR. Every commit has a clear description and you can see the relevant changes in smaller chunks.


Update releases in github to match NPM

Hi, thanks for a great project.
I can see that NPM package for vue2-leaflet  has been updated with a new release of this project, but can't seem to find it here on github?
https://david-dm.org/schlunsen/nuxt-leaflet

@schlunsen you are right I still not have done a github release matching npm, in the pipeline, I am adjusting some travis stuff first

@schlunsen Releases are now on pair And I will endeavour to keep them up


Image or Video sourceType not working as expected

Describe the bug
Using Image or Video as sourceType not rendering expected results.
To Reproduce
Try: arjs="sourceType: image;  sourceUrl: https://cdn.glitch.com/c9111e7d-1d31-41a0-8e15-78b57caa9816%2Fhiro.jpg?v=1569447311795;
and:

Expected behavior
A red box to appear on top of the image.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Chrome, Mac
https://rawgit.com/jeromeetienne/AR.js/master/aframe/build/aframe-ar.min.js

moved to new repository: https://github.com/AR-js-org/AR.js


Fixes #1289 Env swarm parameter lost on the eval help message

My go-fu is non-existent, and toolchain isn't where I want it to be yet, but was able to fix this nagging issue :)

Looks like there's some funny business going on with the integration test PR commits ... I think you need a rebase.

Yeah, pull/merge played a trick on me here, I'll redo and cherry pick, traveling the next couple days

@aperepel If you run git rebase origin/master on your feature branch, and then force push to that branch, it usually fixes it for me if I run into issues like this.  Would be easier than cherry-picking and re-opening a PR.

Sorry, @nathanleclaire something was very wrong with the origin & upstream tracking. I've created a feature branch and opened a new PR since.


[scan] Remove Scan.project platform checks to fix multi-platform targets

Fix regression introduced by: https://github.com/fastlane/fastlane/pull/5159


 So, I was wondering if you'd be interested in making a DELTARUNE engine for GameMaker. You obviously understand how to, so why don't you try it? You'd be the first to ever do it. (btw i was impressed enough to follow you, good job! :)

Yes, I would like that one as well, but TML is determined about waiting for all of the chapters. He could make an engine for chapter one as well. It would at least help my dead fan-game.

related: #75


OPF problem

Describe the bug
The AC OPF is giving an infeasible solution for a case.
To Reproduce

Run an OPF for attached case in MATPOWER and check infeasibility in power balance equations.
Save the case as .MAT file (It is attached as well).
Load it in a python distribution with the command opf.read_case_matpower("file.mat") after importing optimod modules
Run the AC OPF.
Check if solutions match.
Check infeasibility the same way as before.

MATPOWER case file (.m) and Optimod case (.mat)
IEEE69.zip
Expected behavior
If the optimization problem was solved and the optimal solution was found, the solution must be feasible within tolerance. I expected the same solution as MATPOWER.
Additional context
With MATPOWER I run the OPF for a case (69 node Radial network, 1 reference node (node 1), 68 PQ nodes (2-69)) , whose cost function is polynomial (default in MATPOWER) with zero coefficients, except for the first order one (linear cost function). With this setup, we are minimizing indirectly the power in the slack generator, thus the losses.

To verify feasibility, we use the obtained solution and check if the voltages are within limits and if the power balance equations are satisfied (in this case the line power limits are unconstrained).

To check balance equations I use 2 tests:

First, create a zero array (size Nx1) for the active $p_{g}$ and reactive power $q_{g}$ at the slack bus and the demand ($p_{d}, $q_{d}$) for each node, and assign the values from the solution obtained accordingly. Calculate the power losses $p_{loss},q_{loss}$. Then check power balance equations:

$$ \sum_{i=1}^{N} p_{g} - \sum_{i=1}^{N} p_{d} -p_{loss} =0$$
$$ \sum_{i=1}^{N} q_{g} - \sum_{i=1}^{N} q_{d} -q_{loss} =0$$

Choose a power flow model, compute to each corresponding variable its values from the solution and calculate for each node the power balance equation. For instance, if the complex bus injection model is chosen, the complex admittance matrix $Y$, voltage $V_{i}$, generation $S_{i}^{g}$ and load $S_{i}^{d}$ vectors should be constructed.

$$ S_{i}^{g} -S_{i}^{d} = V_{i} \sum_{j=1}^{N} Y_{ij}^{\ast}V_{j}^{\ast} \quad \forall i,j \in N$$

As observed, the MATPOWER solution is feasible, and is often found as reference value in the literature.
Applying the same procedure in python. First, the modules are imported, and the case is loaded. Then the AC OPF is excecuted:

The objective differs from MATPOWER, but the solution found is optimal.

This solution pass the first feasibility check:

However the second feasibility check is failed:

Hi @dimendozao, thanks for raising this. Could you please attach the code necessary to reproduce this result from your input files as a python script? We appreciate you sharing a working example, but transcribing code from screenshots is time consuming and error-prone.
In addition, can you please share the versions of installed packages required to reproduce the issue (python, gurobipy, gurobi-optimods) and the OS you are running?

Hi @simonbowly. Thanks for your reply. I used the following setup:

Windows 11 22H2
Python 3.11
Anaconda 3
Spyder IDE 5.4.3
gurobipy version 10.0.2
gurobi-optimods version 1.1.0
numpy 1.24.3

Here is the .py file.
IEEE69.zip

Hi @dimendozao, sorry for the delay! I could reproduce this with your code, thanks for sharing.
I believe this is linked to the rectangular coordinates representation currently used for the AC formulation. Gurobi doesn't report any large infeasibility in the solution itself, so the formulation would seem to be the issue here. We haven't added a polar form representation (which I believe MATPOWER uses?) to the optimod just yet, but this is planned for early next year.

One thing I noticed is that the line limits are set to 0 in the m file. I would also recommend using the per unit numerical values in the matrices and avoid having the per unit conversion done inside the file.
I'm attaching an updated m file (fixed line limit and per unit issues) that returns an optimal solution with an active power of 3.8921 (optimal objective of 77.842).
I'm also attaching the solution file if that helps.
Archive.zip

Hi @dimendozao & @simonbowly,
after talking with @JaromilNajman over the opf-optimod last week, I tried it myself and came across this issue here.
I was also able to reproduce the issue, although I found 3 slightly different solutions with Gurobi than dimendozao did:
Solution count 3: 80.886 80.9328 80.9432
Maybe this is because I am using Gurobi 11.0.1.
However, I think these different solutions are not the important point here. I think the important point here are the power differences at the buses, which show up for any arbitrary solution when the node powers are calculated from the generator and load powers on the one hand and from the grid equation system with the complex valued voltages on the other hand.
I did some more investigations because I couldn't believe that the use of rectangular coordinates could be the cause of such large deviations.
Long story short: I am pretty sure that the voltage angles are calculated incorrectly.
I didn't elaborate what the function compute_voltage_angles(alldata, result) in grbformulator.py is supposed to do in detail (I think I have a rough idea ...) but I think this function is way to complicated and maybe there is an error in it. However, I commented the according function-call in line 372 of grbformulator.py out and added the following simple line in the for-loop starting in line 345:
resbus["Va"] = math.atan(alldata["LP"]["fvar"][databus].X/alldata["LP"]["evar"][databus].X)
Furthermore, the angle is calculated with ph[i]=np.radians(res_bus[i]['Va']) in dimendozao's OptimodTest.py, but I think it simply needs to be ph[i]=res_bus[i]['Va'].
With both adjustments significantly better (and reasonable) values can be calculated and less differences result at each node.
Can you reproduce the improvements with your code?
P.S.: It is still unsatisfactory that the remaining differences are so large. There is certainly still room for improvement.
From my experience with rectangular coordinates, I would generally recommend not to switch from rectangular coordinates to polar coordinates and back, as the (inverse) trigonometric functions entail numerical inaccuracies.

Hi folks, we believe that we found the issue and can get all violations down to zero. It has to do with using auxiliary variables in the power flow constraints, and this particular instance has numerically large coefficients (1e6) multiplying these variables.
The solution is to have the power flow constraints written using the original variables ((e,f) for complex V).
We are working on an updated model, stay tuned.

Hi everyone,
I am checking the response given by Thomas @100110110101 regarding the computation of phase angles. The calculated infeasibility in the code is indeed substantially reduced by not transforming the angles into radians, as suggested.

This issue could appear naturally if the results are said to be formatted as MATPOWER results, since according to their manual phase angles are displayed in degrees. This raises the question if the solution obtained from optimods delivers the angles in degrees or radians.


SmsAndroid undefined?

Sorry I'm quite new to React Native.
I yarn installed your package and when I try this code
componentDidMount() {
        Sms.autoSend(
            '+111111111',
            'Test message',
            fail => {
                console.log('Failed with this error: ' + fail);
            },
            success => {
                console.log('SMS sent successfully');
            }
        );
    }

I get this error:
00:08:32: TypeError: undefined is not an object (evaluating '_reactNativeGetSmsAndroid2.default.autoSend')

Any idea of what I'm doing wrong?

@briankabiro I think it's Expo not supporting SMS access. I had found a github issue about it but I can't find it back.

I've updated the README accordingly.


Wiki documentation is non-existent

I get we are all busy but when all MS documentation points to MSAL as the way to go for SPA apps, then documentation needs to be a first class citizen.

@aszalacinski  - I apologize that you are not able to find what you are looking for. We do have the documentation and we are currently working on improving it. Could you please  explain why do you say that it's non-existent?

@nehaagrawal the Wiki on this Github Repository does not show any useful information. Every item links back to Home. The only link that works is Register your app with AAD that links you to Microsoft website.

@aszalacinski I have fixed the wiki. Please check.


Faulty PHP DocBlock in Psr.php

Faulty PHP DocBlock:

https://github.com/zendframework/zend-log/blob/5cb1e94d203f3e2c2e848a66ec6888479906d0e8/src/Writer/Psr.php#L58-L59

A possible fix would be:
@param  array|Traversable|PsrLoggerInterface $options
@throws \Zend\Log\Exception\InvalidArgumentException

How to reproduce the issue
Open Psr.php in PhpStorm

i added right use statement to file.

@dennybrandes
Nice, could you also change on line 59 the LoggerInterface to PsrLoggerInterface, as it's declared on line 15?

@KoenigsKind
It is changed.

Fixed by #93


eglChooseConfig failed

Just trying to run the demo project in the Xamarin Android Player.  I receive this error:
[eglCodecCommon] TcpStream::connect() - management hostname is <IP_ADDRESS>
[eglCodecCommon] TcpStream::connect() - connecting host: <IP_ADDRESS>
[AndroidRuntime] FATAL EXCEPTION: GLThread 91
[AndroidRuntime] Process: com.skobbler.sdkdemo, PID: 1216
[AndroidRuntime] java.lang.IllegalArgumentException: eglChooseConfig failed
[AndroidRuntime] 	at com.skobbler.ngx.map.SKMultisampleConfigChooser.chooseConfig(SourceFile:54)
[AndroidRuntime] 	at android.opengl.GLSurfaceView$EglHelper.start(GLSurfaceView.java:1024)
[AndroidRuntime] 	at android.opengl.GLSurfaceView$GLThread.guardedRun(GLSurfaceView.java:1401)
[AndroidRuntime] 	at android.opengl.GLSurfaceView$GLThread.run(GLSurfaceView.java:1240)

Hi SJ,
What device are you emulating in the Android player? Have you tried it on a real device or the emulators provided with the Android SDK?
-TD

I'm using the Nexus 4 KitKat (v 4.4.2).  I haven't tried it on a real device (don't have one yet -doh!).  I'll try an emulator provided with Android SDK.  Do you know of any of the emulators in the Xamarin Android Player that should work?
thanks!
sam

I tried a nexus 4 4.4.2 on the packaged emulator with android SDK, and it gives me a different error:
[Choreographer] Skipped 44 frames!  The application may be doing too much work on its main thread.
[] HostConnection::get() New Host Connection established 0xb7ec2e90, tid 2538
[] HostConnection::get() New Host Connection established 0xb86014b0, tid 2579
[SKMaps] MapRenderer----@onSurfaceCreated
[Settings] Setting always_finish_activities has moved from android.provider.Settings.System to android.provider.Settings.Global, returning read-only value.
[SKMaps] MapRenderer----@onSurfaceCreated LinInitialized = true
[SKMaps] MapRenderer---- @initGLView
[SkobblerNG] set density JNI  - before NG_SetScreenScale
[SkobblerNG] set density JNI  - after NG_SetScreenScale
[SKMaps] MapRenderer----@onGLInitializationError called main
[SkobblerNG] NG_SetCCPIcon(12)
[SKMaps] MapRenderer----onSurfaceChanged width = 768 height= 1084
[SKMaps] MapRenderer----@onSurfaceCreated called main
[SKMaps] SKMapSurfaceView----Set map follower mode NONE
[mono-rt]
[mono-rt] =================================================================
[mono-rt] Got a SIGSEGV while executing native code. This usually indicates
[mono-rt] a fatal error in the mono runtime or one of the native libraries
[mono-rt] used by your application.
[mono-rt] =================================================================
[mono-rt]
[libc] Fatal signal 11 (SIGSEGV) at 0x00000000 (code=1), thread 2579 (Thread-93)

Do you know of a specific emulator version that it should work with?  I'll keep trying other stuff...

Hi Sj. I'm posting from my phone at the moment since I'm without a PC to verify all this so forgive me if its not totally correct.
I think the problem has to do with Xamarin Android Player being more of a simulator than a true emulator. Similarly if you check "use host GPU" when setting up your SDK emulator, you will get a more responsive emulator but you will run into problems like this when rendering glsurfaces.
My advice would be to use a real device or use the SDK emulators without any optimisations. Unfortunately they're slow but it's the only way to get accurate results without spending money on a device.
I'll be able to have a proper look this weekend and hopefully next week after I get my PC.
TD


Azure Stack Admin REST API Swagger Specs

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
I have checked all below.  I know some of the criteria are not met this is because the Admin side of the RPs have a specific implementation which does not allow for compliance.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@deathly809,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Boss informed me that I should close.  Apparently, someone told him we needed to do something extra.  Oops?


Buffer improvements

Adds additional warnings when trying to use the transform manager without a valid transform buffer
Adds an optional parameter to the transform manager constructor to pass in a transform buffer when constructing the manager to set up the entire system

If the goal here is to allow flexibility and make tf_manager easier to use, I think that's achieved. People with more advanced use cases can pass in their own tf_buf if they are worried about having multiple buffers and hindering perfromance.
If the goal is to keep a similar API to ROS1, I think we should change remove the Initialize function call from the constructor since ROS1 still required you to call the initialize function after creating a tf_manager. However, I'm not sure why Initialize can't (or wasn't) be done in the constructor.

I also looked at how we could check if the listener was set up correctly. I haven't delved deeply into tf2's source before, but I think we could change https://github.com/ros2/geometry2/blob/ros2/tf2/src/buffer_core.cpp so that it returns the cache length with the cache's getListLength() call, and that would tell you if it ever has received something in the transform cache.


Seq2seq output evaluation

Seq2seq can and does change the input word which is not taken into account at boundary prediction evaluation. How should I handle this? @e9t

Compute precision and recall on morphemes.
Questions:

should I average over all words or add true/false positive/negative counts on all and then compute precision and recall?
should I use bag-of-morphemes models or take cardinality into account? This issue probably affects a very small number of words.


Add the SAP Launchpad Service

Tutorials: https://developers.sap.com/tutorials/btp-app-launchpad-service.html
Write here how you think we can improve the tutorial ...
Hi,
I was following the tutorial to build for my own project in a production environment. The steps which I had successfully done were:-

Created a CAP application
Added a Fiori App to the CAP application
Added Authentication to the CAP application
Deployed the Same on the cloud and the Entity for which I had enabled security. It gave an "Unauthorized" error.

While going to the next step
The srv and db are staged, however the app is not being staged and giving an error.
This is the error which I am getting:-

Uploading the logs as well
circtwo-srv.log
OPERATION.log
circtwo-app-content_circtwo-html5-repo-host.log
circtwo-db-deployer.log
circtwo-destinations_circtwo-destination.log

Hi @nikegup,
Thanks for your feedback!
These GitHub issues are to resolve issues (i.e., problems/errors) with the tutorials. Your question is more general since it involves deployment to a production landscape and not documented or suggested app modifications. As such, it would likely get more visibility and a better answer in the SAP Community's Questions & Answers.
Best regards,
Svetoslav


x/build: TryBots consistently failing on misc-compile-android

CL 205798 added misc-compile TryBots for android (#25963), and a coordinator containing that CL has now been deployed.
Unfortunately, misc-compile-android is consistently failing on the android/386 configuration:
https://storage.googleapis.com/go-build-log/5451ccaa/misc-compile-android_aac075f2.log
I tried to git bisect the failure locally, but as far as I can tell this build configuration has never worked. We should either get it working ASAP or roll back the TryBots for the failing configurations.
CC @bradfitz @dmitshur @cagedmantis @tklauser

Don't know if it helps, but I ran into this today, and tried my CL with with a gomote push and it worked successfully. Perhaps the difference in the trybot setup from whatever gomote uses would be helpful to investigate.

Nah, it's 100% reproducible locally too.
cd $(go env GOROOT)/src && GO_BUILDER_NAME=misc-compile-android GOOS=android GOARCH=386 ./buildall.bash


Improve UI of  scroll-to-top button

Fixes #103
This PR changes the button Icon and also makes it a little smaller and centers the icon.
Also, changes the div to button for a11y and cursor while hovering.
Before:

After:

Looks awesome! Thank you 😊

@all-contibutors please add @Deveesh-Shetty for code, design

@all-contibutors please add @Deveesh-Shetty for code and design

@all-contributors please add @Deveesh-Shetty for code and design


Data of bar graph is same for all levels

Select district based on the selection bar chart is not showing block data on the chart
usage progress(courses)

Re-tested it is working as expected


fixed the footer issue #21

fixed the footer by adding hover features and icons with effects
pr for issue#21

@Mayank2808sharma , request to label the PR as according to the announcements in discord server of gssoc  its necessary

@Mayank2808sharma

Please add gssoc and level @Mayank2808sharma as it is.not counting on the leaderboard


auth: get rid of deadlocking channel passing scheme in simpleTokenTTL

Just use the mutex instead.
Fixes #7471

LGTM. Is it easy to get a test covering the bad case?
/cc @davissp14 do you mind giving this a quick test?

@xiang90 Sadly I am out tonight, but will be able to test tomorrow morning.

lgtm

@heyitsanthony thanks a lot for the fix! The changes look good to me. But I could reproduce the test failure on my local env, too:
--- FAIL: TestCtlV3UserDelete (1.19s)
	ctl_v3_user_test.go:93: userDelTest #1: ctlV3User error (read /dev/ptmx: input/output error)
	ctl_v3_test.go:150: error closing etcd processes (exit status 2)
=== RUN   TestCtlV3UserPasswd
--- FAIL: TestCtlV3UserPasswd (0.69s)
	ctl_v3_user_test.go:116: userPasswdTest #1: ctlV3User error (read /dev/ptmx: input/output error)
	ctl_v3_test.go:150: error closing etcd processes (exit status 2)

I'll take a look at the test cases.

Tested out the changes and wasn't able to reproduce the deadlock.  Nice work!

@mitake the crash was caused by calling UserDelete when auth is disabled; the ttl keeper was nil so trying to acquire the lock in invalidateUser would crash it.

@heyitsanthony I see. lgtm after CIs can be passed, thanks!

Codecov Report

Merging #7492 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7492      +/-   ##
==========================================
- Coverage   70.73%   70.65%   -0.09%
==========================================
  Files         245      245
  Lines       21486    21487       +1
==========================================
- Hits        15198    15181      -17
- Misses       5156     5176      +20
+ Partials     1132     1130       -2

Impacted Files
Coverage Δ

auth/simple_token.go
94.73% <100%> (+8.01%)
:white_check_mark:

pkg/adt/interval_tree.go
78.71% <0%> (-6.09%)
:x:

lease/leasehttp/http.go
59.54% <0%> (-3.06%)
:x:

pkg/netutil/netutil.go
77.94% <0%> (-2.95%)
:x:

rafthttp/msgappv2_codec.go
69.56% <0%> (-1.74%)
:x:

etcdserver/server.go
78.45% <0%> (-1.29%)
:x:

etcdserver/v3_server.go
61.84% <0%> (-1.1%)
:x:

raft/node.go
89.73% <0%> (-0.45%)
:x:

rafthttp/transport.go
81.92% <0%> (ø)
:white_check_mark:

rafthttp/stream.go
89.7% <0%> (+0.66%)
:white_check_mark:

... and 6 more

Continue to review full report at Codecov.

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

Will this be added to 3.1.4?

@davissp14 yes, it's tagged for backporting


why not make mirror support a private v2 registry until now?

It's very easy to change the logic of endpoints in the function "lookupV2Endpoints" to put the mirror (remoteurl points to a private registry) in the final endpoint list even if the image is not in the hub, and in fact I find that mirrors can support private v2 registries after I rewrite few lines of codes. @stevvooe @RichardScothern

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@Kaffa-MY I'm going to close this issue for now, because I think it's better to discuss this in the #docker-distribution IRC channel first. Feel free to comment here though, after I've closed. Also, interested to hear what the outcome is of that discussion :+1:

@Kaffa-MY Without going into detail, the main issue here is trust and provenance. Basically, the registry controls the naming, so once you point the daemon at another registry, you can no longer trust the content if layers collide. We avoid these issues by disallowing arbitrary mirroring.
This is a situation we are working on resolving but it will take some effort to appropriately address.

@stevvooe I see I actually forgot to close, but do you want this stay opened, or is this tracked already in your repo?

@thaJeztah This is described in the road map, I believe. @RichardScothern Do we have this documented anywhere?

It is now: https://github.com/docker/distribution/pull/1136
On Tue, Oct 27, 2015 at 3:48 PM, Stephen Day<EMAIL_ADDRESS>wrote:

@thaJeztah https://github.com/thaJeztah This is described in the road
map, I believe. @RichardScothern https://github.com/RichardScothern Do
we have this documented anywhere?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/16974#issuecomment-151667238.

thanks all, I'll close this one!

@stevvooe thanks for your reply


Invalid key for deCONZ after HA restart

Until the latest update (4.1) this addon has been working perfectly.
With the update to 4.1 I experienced the described issue with no access to the Gateway (I guess this was related to the ingress-support). I did remove the integration, restarted Home Assistant and added the integration.
The autodiscovered integration was added but without prompting me to enable the authentication from the addon webinterface. The integration seemed successful without any errors, but with no discovered devices.
Adding the integration by manually selection deConz from HA integrations flow list was normal including the expected authentication prompting and all my devices were discovered. However: Each time I restart Home Assistant I get the following error in the HA-log:
"Invalid key for deCONZ at core-deconz.local.hass.io".
The only solution for this seems to be removing the integration and add it again.

Same experience here, upgraded from 3.9 to 4.1

Thanks, @KKlitgaard Your detailed explanation actually gave me a hint! 👍
Diving back in.

@frenck I just updated to 103.4 and issue is still there, logg shows me "Invalid key for deCONZ at core-deconz". I have to remove the integration and add it again for it to work until next restart of HA.

@frenck I just updated to 103.4 and issue is still there, logg shows me "Invalid key for deCONZ at core-deconz". I have to remove the integration and add it again for it to work until next restart of HA.

I can confirm this. I updated to 103.4 and the integration stayed functional, but after a restart O again get the same error in the log.

Home Assistant 0.103.4 is part of the fix. The add-on needs updates too, which is expected soon.


table Operation Thunder

Operation Thunder (Gottlieb 1992)

Done.
On Wed, Sep 11, 2024 at 4:12 PM lilalien @.***> wrote:

Hey man...
Let's keep all the "wheel" images to the standard round wheel type going
forward, at least until we have the template ready to replace ALL of them
with a unified design.
Replace the Operation Thunder one with:
https://mega.nz/folder/p3hREDgB#pEWMJvVQ7t3Sv_TmrxdU-w/folder/52AzHIDR
Search "operation thunder" and you should see it.
Also, I noticed the very last commit adds "old coney island" for some
reason to the main index, which was already taken care of on kaotic's end..
Was there a reason for that commit?
Thanks man!
—
Reply to this email directly, view it on GitHub
https://github.com/LegendsUnchained/vpx-standalone-alp4k/pull/471#issuecomment-2344614634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADX57UITZIWZUYVHDWNIA73ZWCPZDAVCNFSM6AAAAABN77ZR7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBUGYYTINRTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


files.watcherExclude does not exclude paths that include symlinks

VSCode Version: 1.12.1
OS Version: OSX 10.12.4

If the files.watcherExclude preference includes a pattern such as **/mysymlink/**, and the target of that symlink does not contain an actual directory named mysymlink, then the files will not be excluded.
This, in combination with the way the Bazel build tool works, results in my org's builds occasionally failing on OSX if VSCode is open, because the OS has hits its limit on how many files can be open. In our build, Bazel created thousands of directories in its working directories, and I had tried to tell VSCode's watcher to ignore those directories, but it didn't, since they were actually symlinks, not directories.
Like I said, we use Bazel; Bazel creates several symlinks with names like bazel-out, bazel-bin, etc. On OSX, those map to /private/var/tmp/_bazel_<username>/...'
Steps to Reproduce:

Download and expand example.zip. This is a small sample project that builds with Bazel.

Download and install the Bazel build tool.

Start Visual Studio Code, pointing at this directory.

Notice that .vscode/settings.json is configured with:
 "files.watcherExclude": {
     ...,
     "**/bazel-*/**": true
 }

This is because, as mentioned above, Bazel will create several symlinks in the root with names that start with bazel-*, and will put all of the build artifacts into those directories.

Save a list of all file handles that VSCode's Code Helper program has open: lsof | grep '^Code' > lsof-before

From the command line, compile the project: bazel build //...

Now repeat step 5, saving the new list of file handles: lsof | grep '^Code' > lsof-after

Compare the two lists of files handles. You will notice that the second list of file handles is longer, and that it includes several handles for files/directories of the form /private/var/tmp/_bazel_<username>/....

Let's fix it. Tell it to ignore those files:
 "files.watcherExclude": {
     ...,
     "**/_bazel_*/**": true
 }

Exit VSCode

bazel clean

Repeat the test.

In the above scenario, since this is a tiny sample project, you will only see a small number of newly-opened directories, so it's not a problem. But in my company's real build system, before compiling, Code Helper had 921 handles open, and after, it had 6,767, so OSX hit its handle limit and my build failed.
In general, I think I would argue that people would expect a pattern like **/mysymlink/** to work, and that they wouldn't want to have to think about the name of the directory that is pointed to by the symlink.

P.S. To clarify: In step 6, bazel build //... is actually what you type in order to a build. The ellipsis is part of the command.

I forgot to include this — in this run, process 15521 was the one that had all the open handles (mosty to directories), so I did a ps to see what that was, and here is the result:
$ ps aux | grep 15521
mikemorearty     15717   0.0  0.0  2442020    816 s000  S+    2:00PM   0:00.00 grep 15521
mikemorearty     15521   0.0  0.2  3312124  37180   ??  S     1:58PM   0:01.09 /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper.app/Contents/MacOS/Code Helper /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap --type=watcherService

Here are the lsof results I got before and after my Bazel build:

lsof-before.txt
lsof-after.txt

I'm facing this issue too. The CPU just goes crazy. Has anyone figured out some workaround?

I have the same problem on Debian 9 and the latest version of vscode 1.36.1.
Even this config, where I listed all the base- * folders, does not help.All of these folders are symbolic links.
Every time I open vscode, it rebuilds the index using all my CPU resources and as a result takes 1.7 GB. Therefore, I think that for the remaining 1800 source files of my bazel project, vscode does too much work and consumes too much of a resources.
"files.watcherExclude": {
    "bazel-bin/**": true,
    "bazel-genfiles/**": true,
    "bazel-out/**": true,
    "bazel-testlogs/**": true,
    "bazel-astra/**": true
},
"files.exclude": {
    "bazel-bin/**": true,
    "bazel-genfiles/**": true,
    "bazel-out/**": true,
    "bazel-testlogs/**": true,
    "bazel-astra/**": true
},
"search.exclude": {
    "bazel-bin/**": true,
    "bazel-genfiles/**": true,
    "bazel-out/**": true,
    "bazel-testlogs/**": true,
    "bazel-astra/**": true
},

the handling of this seems to have changed to be more what the issue reporters expect with 1.39's new version of chokidar (i actually found the previous behavior more desirable because it was easier to exclude NFS mounts since it matched on the realpath of the exclude).  do any of the reporters see something similar?

I am more thinking in the future that we will drops support for symbolic links entirely. See https://github.com/microsoft/vscode/issues/25547 last comment.


fasdfxczvcxvcvvzcvc

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>
as

Closing because this is empty.


Using default settings (100-node matrix), unable to achieve satisfactory results?

Hello,
I am able to clone this repository, install the needed dependencies, yet I am unable to achieve satisfactory results?
By running the code and using the default test.json file, given that the sample matrix has 100 nodes and 450 edges, I would expect that running this code would result in a solution that is close to that.
However, using the default settings and running the code, I am only arriving at number_of_edges around the 260-270 range, far short of expected results.

Are these the best results we can expect with your method, or is there possibly an error in my code or approach somewhere?

How did you get that result? I cant any result...(I tried a lot of things and searched on the internet)😭 I'm sorry about many questions...
@robliou


backup for pull request web hook

Thank you for the contribution.
Can you rebase onto current master and try to add some tests for this behaviour?


question about displaying output of pose estimation, action recognition

hello, I'm student studying about computer vision and deep learning.
I'm in trouble about displaying output.
I could get log text of eval program.
but I want to display pose into image and action labeling ....
I searched Issues but I couldn't solve...

I wanna see a 3d coordinate system like in the paper as well.  Maybe using a 3d tool like Open3D or PCL? I am not sure what tool the author used.


Make caller responsible for providing the tile server

Closes #84
Caller should have ability to choose and specify a tile server of their choice whether through a service or self hosted.

Adds tileServer prop
Updates demo-app examples to provide tileServer
Updates docs
Reverts peerDependencies change. build.js already excludes grommet-leaflet dependencies from the bundle. Need to test this will still work for projects using React 17.

Looks like we lost the CSS overrides for Popup at some point. There is extra padding and the anchor has the leaflet treatment.

A general question -- should we be uploading the dist folder to main or should that be in the .gitignore?

Let's try it. I think we needed it at one point, but the set up has since been modified. Added.

Looks like we lost the CSS overrides for Popup at some point. There is extra padding and the anchor has the leaflet treatment.

I think we are going to need to revisit the approach to how the CSS reset is being applied to caller apps.


[incubator/cockroachdb] deprecate missing chart

See https://github.com/helm/charts/issues/23128

/lgtm


[BUG] Fault occurred on robot server, exiting ... And the arm moves unexpectedly

Version information

RDK: v0.8
Robot software: v2.10.5
OS: Ubuntu 20.04

Describe the bug
Start the program implementing my own controller using torque streaming API. The joint 6 suddenly swings down. While the RDK prints out:
[2023-09-07 14:52:00.937] [error] periodicTask: Fault occurred on robot server, exiting ...
[2023-09-07 14:52:00.937] [info] [flexiv::Scheduler] Stopped all tasks
[2023-09-07 14:52:01.937] [info] [flexiv::Robot] Network service closed
[2023-09-07 14:52:01.938] [info] >>>>> Flexiv RDK exited <<<<<
Steps to reproduce
Cannot be reproduced easily.
Expected behavior
The arm will stay at its initial location, and will not move.
Screenshots
NA
Additional context
Please tell us a method to find out what fault is on the server side.

@acf986 Thanks for bring up the issue to us. In the upcoming (late September) v0.9 release, the server-side error message will be automatically transmitted to RDK client so you know exactly what error happened on the robot server.

Hi, for now, is there any way we can see the possible error from you QNX system? or at least the possible type of errors that will print out this message.

@acf986 Yes, you can always keep the UI tablet (Flexiv Elements) connected while using RDK. Once any error occurs, you will see a pop-up error message from the UI tablet. What we will do in RDK v0.9 is to also copy this pop-up error message to RDK client and get it printed in the console in case the UI tablet is not connected. See https://rdk.flexiv.com/manual/troubleshooting.html#fault-handling-and-error-messages


Remove client from repo, use ndt7-client-go instead

This change is

Pull Request Test Coverage Report for Build 1039

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+4.4%) to 79.513%

Files with Coverage Reduction
New Missed Lines
%

ndt7/closer/closer.go
3
70.0%

Totals

Change from base Build 1037:
4.4%

Covered Lines:
1568

Relevant Lines:
1972

💛  - Coveralls


fails to install in atom 1.3.2

I'm running into the following error trying to install autocomplete-ctags in Atom v 1.3.2 in Ubuntu 14.04
I'm able to install other packages (I'm currently using atom-ctags until I can get this issue figured out)
I tried reinstalling node, npm, and atom to no avail.  autocomplete-ctags was working fine as of about a week ago, but I decided to update the package, and it failed with the error.  I uninstalled autocomplete-ctags to see if I could clean-install the new version, but I'm seeing the same error.  Unfortunately that npm-debug log doesn't exist in the directory specified in the warning.
><EMAIL_ADDRESS>install /tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/pathwatcher/node_modules/runas
> node-gyp rebuild

make: Entering directory `/tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/pathwatcher/node_modules/runas/build'
  CXX(target) Release/obj.target/runas/src/main.o
make: Leaving directory `/tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/pathwatcher/node_modules/runas/build'

><EMAIL_ADDRESS>install /tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/ctags
> node-gyp rebuild

make: Entering directory `/tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/ctags/build'
  CC(target) Release/obj.target/ctags/src/readtags.o
  CXX(target) Release/obj.target/ctags/src/tags.o
  CXX(target) Release/obj.target/ctags/src/tag-finder.o
  CXX(target) Release/obj.target/ctags/src/tag-reader.o
  SOLINK_MODULE(target) Release/obj.target/ctags.node
  COPY Release/ctags.node
make: Leaving directory `/tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/ctags/build'

../src/main.cc:1:17: fatal error: nan.h: No such file or directory
 #include "nan.h"
                 ^
compilation terminated.
The bug is not reproducible, so it is likely a hardware or OS problem.
make: *** [Release/obj.target/runas/src/main.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/share/atom/resources/app/apm/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Linux 3.16.0-55-generic
gyp ERR! command "node" "/usr/share/atom/resources/app/apm/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /tmp/apm-install-dir-1151121-29493-g4dwll/node_modules/autocomplete-ctags/node_modules/pathwatcher/node_modules/runas
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Linux 3.16.0-55-generic
npm ERR! argv "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/home/garretts/.atom/.apm/.apmrc" "--userconfig" "/home/garretts/.atom/.apmrc" "install" "/tmp/d-1151121-29493-1j4hb1d/package.tgz" "--target=0.34.3" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the runas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls runas
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/apm-install-dir-1151121-29493-g4dwll/npm-debug.log

similar issue
https://github.com/mulesoft/api-workbench/issues/25

Looks like their solution was to remove some native dependencies which were causing issues.    I'm using ripper-tags to generate ctags externally for my ruby application.  Is there a possibility you could remove the ctags dependency?  Or is that what is actually used for the tag traversal?

Use the node-ctags for the parse of the tags file.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 7327e62, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 7504 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 81e540c.


IsResultObsolete in Microsoft.FSharp.Compiler.SourceCodeServices appears to be internal in released package

I'm looking to update FCS and IsResultObsolete seems to be private in the released package:

Although in the code it appears to be not internal but the .fsi doesn't expose it:
https://github.com/fsharp/FSharp.Compiler.Service/blob/12209eb36a85e5049d2b501403ffd7a9370c848a/src/fsharp/vs/service.fsi
Am I right that this is an unintended public API change?

Note that in VFPT this is used only once, and I'm trying to pass a raw lambda instead, that value then doesn't seem to be used at any callsite.

@smoothdeveloper This was (silently) deprecated in FCS 10.0.1 because it has been replaced either by the passing of a CancellationToken for all calling APIs, or by the use of the cancellation token propagated by the returned Async  computation
Apologies for the silent deprecation, we should have added an Obsolete on this type and kept it public.

I'll close this though I wouldn't be surprised if other people hit the same issue during migration.

Thanks for the notice @dsyme.
Maybe it is worth, if others face the issue, to put simple migration direction (showing before / after code).
In case of VFPT the sole call which used that was to ParseAndCheckFileInProject https://github.com/fsharp/FSharp.Compiler.Service/blob/c28509d14367314ad7038f2b2c5effa237abb1b3/src/fsharp/vs/service.fsi#L520 although the doc mentions that value will be passed to hasTextChangedSinceLastTypecheck I don't see that argument defined for that method.
Is there something else that needs to be changed in ParseAndCheckFileInProject documentation or a missing parameter?

@smoothdeveloper The migration would just be to switch the calling code to use the relevant cancellation token rather than relying on callbacks to IsResutObsolete.  However I think people have always just been ignoring this parameter or passing (fun _ -> false).  If you have trouble migrating please link to the issue here and let's take a look, thanks


Trim Envoy log file prefix

https://github.com/envoyproxy/envoy/pull/10693 improved the logging format. Once we pull that change in we should update the log format

done


Add a method to make the gui not closeable by client

It would be nice to make a method that does not allow the client to close the gui, until the plugin closes it.

I don't think that's very do-able, afaik there's no way to check if the inventory was closed by a player or plugin.

are u sure?

Pretty sure that's a paper only thing.

umhh now I check

I did check. It is.

First is spigot, second is paper.

You can implement that on your end though
https://github.com/TriumphTeam/triumph-gui/blob/7b5817595acdfc150c2426267818503b48acb122/core/src/main/java/dev/triumphteam/gui/guis/BaseGui.java#L409-L417

oh okay, thanks


Tesseract skips recognizing some lines of text.  More and more lines are skipped while increasing DPI of the input image.

Environment

Tesseract Version:  4.1
Platform:  Windows  64-bit

Current Behavior: Accuracy getting worse while increasing DPI of the input image:
When I render pages of the source document deu+eng.print.pdf for diffrerent DPIs and then process it by Tesseract 4.1 I get up to a maximum of 59% recognised characters despite of 98% for older Tesseract 3.05 at 600DPI.
Tesseract 4.1 does recognise all rows in the document at 72DPI but the letters are often wrong.
While increasing the DPI the number of succesfully recognized letters within a row grows but number of recognized rows decreases!

shows the first page rendered at 600DPI. When you process it by tesseract.exe 1.jpg -l deu+eng --psm 1  pdf
you get 1.pdf
When you open the result in a browser and select all the text, you can see that 16 lines of the text does not have any letter recognized at all while remaining 29 lines have recognized all characters despite the font and letters are the same on  both types of rows
The other problem is that the dots "...." are very often recongnised as a bunch of different characters (snuec...) despite all of them are the same.
When I change --psm command line parameter to different values I get slightly different results but the overall accuracy is still very bad comaping to Tesseract 3.05.
Expected Behavior:  Accuracy getting better while increasing DPI of input image like in the older version of Tesseract.
I expect number of recognised rows to grow instead of decrease with growing DPI.
Suggested Fix:

You should test this with txt/hocr config options instead of pdf.
The issue with the dots recognition is a know issue, already discussed in other issues. Basicilly, the trained model data was not trained on similar images that contains a large sequence of dots.
If suggest that you upload another image with the dpi issue but without the dots issue.

No feedback from the OP.

I had the same issue and I solved it by reducing the file width and height. For large width and height OCR skips lines. Hope this helps


Compilation explaining

Hi,
I'm currently using Druid with big satisfaction and I'm really interested to your extension.
Can someone explain how to install these on Druid?
I'm currently running compiled version of Druid, how should I add your code?
Thanks
Maurizio

Adding the jars and respective dependencies to the class path should be enough, thought !
Which extension are you planing to use, like that i can provide more help on how to use it.

I'm planning to use metric-collector in order to enqueue Druid Metrics on Kafka.
Are somewhere already compiled jar file or does I need to do it?
Thanks
Maurizio

Sorry, we publish jars only internally at Yahoo. You would need to create the jars your self.
for instance mvn clean test dependency:copy-dependencies

by the way the best way to do it is to include the extension as a maven module, thus you can get all the jars in the same time in addition you can pin your self the versions. I hope this helps.


REDIS_REPLY_ARRAY reply->len / reply->elements

Howdy!
According to the README.md in the repo, the number of elements of an array result is stated to be stored in redisReply->elements. This does not line up with observation in which it appears that the number of elements returned in being placed into redisReply->len.

Ok, clearly I've built hiredis with some mystical voodoo monkey curse. There is a problem with the allocator perhaps because I'm getting wildly different results with successive runs of my test.


Extractor status flag

changes the return value of ImageExtractors to include an is_valid flag, which is also added to the DL1 data model.

In most extractors (that have no failure modes), the return value is just hard-coded to True
in TwoPassWindowSum, where the second pass can fail, it returns False if the second pass failed (meaning the resulting images are from the first-pass only), unless it was requested to only run the first pass.

This is needed since those events where TwoPass fails should be filtered.   Protopipe already implements this, but it should move to ctapipe.

I was mistaken about DL1ImageContainer, it's actually DL1CameraContainer that has the fields filled here directly and the parameters subcontainer.
So maybe we should split this into a structure like this:
class DL1CameraContainer(Container):
    pixels = DL1PixelContainer()
    parameters = DL1ParametersContainer()

class DL1PixelContainer(Container):
    image = ...
    peak_time = ...
    cleaning_mask = ...

?
That would make the DL1a / DL1b split explicit in the container structure.

Yes, I actually thought the same thing when writing this.  Containers should be filled all at once, so having the image mask and parameters in there doesn't make sense.
Technically the mask should be grouped with the parameters and not the other images (even though it is also an image), so it would be better to have a containers for:

extracted images (output of an ImageExtractor)
cleaning_mask + parameters (output of ImageProcessor?)

Anyhow, maybe let's make that a separate issue - this one is getting too large, and has too many changes

It would be  better to have something like:
class DL1CameraContainer:   # (maybe rename DL1ImageContainer)
     image = ...
     peak_time = ...

class DL1ParametersContainer:
    image_mask = ...
    parmeters = ImageParametersContainer

In any case, right now everything still works, this just changed where the DL1CameraContainer was constructed, so it's no slower or worse than before.  We should open another PR or issue to clean up the containers as above, however