Suggestion: Gregtech compatibility

I opened an issue on the GT5U Experimental github asking them to make the new Low Grav Processing work with Advanced Rocketry. There are two other potentially useful points of compatibility that I can think of at the moment. Current GT5U configs specify whether an ore can generate on the Moon, Mars, or asteroids in the file WorldGeneration.cfg. It would be useful if Advanced Rocketry could read these config options and use them to return naquadah ore from automatic asteroid mining missions, for example. Also, it would be neat to be able to land on Titan and have Titan generate with limitless Gregtech reserves of light/medium/heavy oil and natural gas, oceans of Buildcraft/Magneticraft/Gregtech oil of all kinds with infinite water mechanics, and GT oilsands ore.

I'd suggest use COG

I could look into adding the ability to specify what liquid the ocean is made of for each planet.  As for the ores, I cannot say if i can get around to it or not, it should be doable with the current settings for AR, bit tedious though.  Unfortunately i currently have a lot on my plate right now so I need to focus more on the things that currently cannot be done or are broken, but I'll consider adding that as a feature when time frees up

It seems that if Advanced Rocketry generated a dimension with "Titan" or similar as its Class Name then it would be simple enough for me or other users to add infinite Gregtech drilling rig "underground fluid" deposits just by tweaking Gregtech's configs. One could even add deep underground deposits of molten cheese to the Moon. 🧀

Generating new classes at runtime is a very difficult and is a rather hacky way to solve a problem.  (It's almost as bad as trying to have your mod generate the code for the mod at runtime from a template).  Does gregtech have no way of specifying a dimension ID to use as a modifier for terrain generation?

Dimension ID works fine too.


Poll: UDLs vs constants

The more I think about it the more I think that UDLs may not be the right tool for the job. Yeah, we used it in std::chrono and it seems nice to write auto v = 120km / 2h;. However, they could be replaced with constants. For example:
namespace si {
inline namespace unit_constants { // is it a good name?

inline constexpr length<metre> m(1);

}
}

Here are a few issues that I see with UDLs:

UDLs are only for compile-time known values. Currently with UDLs:
using namespace units::si;
auto v1 = 120km / 2h;
auto v2 = length<kilometre>(length) / time<hour>(duration);

with constants those 2 cases would look like:
auto v1 = 120 * km / 2 / h;
auto v2 = length * km / duration / h;

Constants treat both cases in a unified way. It is also worth to notice that we work mostly with runtime variables and compile-time known values appear only in physical constants and unit tests.

UDLs for some units may be impossible to achieve in C++. I already found issues with F (farad), J (joule), W (watt), K (kelvin), d (day), l or L (litre), erg, ergps. It is probably still not the complete list here. All of those problems originated from the fact that those numeric symbols are already used in literals (sometimes compiler extensions but still). I am afraid that at for some of those cases we will not be able to fix it or workaround it in the C++ specification. None of those issues affect constants.

UDLs cannot be disambiguated with the namespace name:
using namespace si;
auto d = 1cm;   // quantity<si::dim_length, si::centimetre>

using namespace cgs;
auto d = 1cm;  // quantity<cgs::dim_length, si::centimetre>

using namespace si;
using namespace cgs;
auto d = 1cm;   // FAILS TO COMPILE

With constants it is simple:
using namespace si;
using namespace cgs;
auto d1 = 1 * si::cm;   // quantity<si::dim_length, si::centimetre>
auto d2 = 1 * cgs::cm;  // quantity<cgs::dim_length, si::centimetre>

So maybe we should prefer constants over UDLs? If so should they be put to an inner inlined namespace (like literals)?
Let's vote:
(please vote only once by clicking on the correct bar above)

Related issue #31.

UDLs are simple shorter and neater so get my vote . Ultimately if they have to have slightly longer names then that is not a problem as far as I am concerned.
UDLs is a neat idea that unfortunately causes very short names to appear in global scope due to unqualified lookup only allowed.
Reserving literals operator names without underscores to the standard namespace didnt solve anything anyway since they are so useful that they are already colliding in std namespace :)
A nice solution would be to extend UDL rules  to allow qualified lookup. Here is one possible syntax. I think scope-resolution-operator followed by a number is syntactically available
  auto x1 = si::1mm;

  auto x2 = si :: 1 mm;  // with the scope resolution operator prefix
                         // pretty spaces should be parsable

That would bring them to a similar functionality to inline constants.
Meanwhile the obvious pragmatic workaround is to attach a postfix namespace to avoid collisions which are otherwise inevitable as use grows
auto x2 = 1l_si;

Qualified lookup for UDLs was discussed already in the ISO C++ Committee but if I recall correctly it did not fly (at least so far). But even if it did it would solve only point 3 from the above list. Still, we cannot easily create some UDLs and long names do not help in some cases (i.e. erg or ergps). Also as I noted in point 2 UDLs help only when we have compile-time known constant. Otherwise, we have to decay to the long form of variable initialization.
But yes, I also always hated the "multiply" syntax to form quantities but it seems it actually may have some benefits over UDLs.

There is another option. Just use a temporary as you show above,
I  show again my typedef syntax which works very well and could be used by mpusz units . It contains all the information required in the shortest form for the commonest quantities and coherent units
   length::mm{1}
      // or ...
   length::mm<>(1) // to expose the value_type default to double

Anyway, certainly worth looking at where and how often the use of such constants occurs in real world. How common is it and is it necessary to define constants at global scope versus just temporaries with explicit initialisers as I used to do before UDLs?  (N.B. Discussing quality of my code and how it could be improved is not the point here ;). It is just shown as an example of day to day use. The code could certainly benefit by review, upgrading to UDLs etc,etc)
To answer point1 I would take the UDL where it works and use the temporary where not
Initialisation is a very common programming chore in application development.
Looking through my code I tend to try to use a short local typedef and then a rvalue with explicit number init for a constant ( remembering I used these types for many years before UDLs were available)
https://github.com/kwikius/aerofoil/blob/master/aerofoilDoc.cpp#L11
// mm is typedef in class definition, could be replaced by mpusz quantity
https://github.com/kwikius/aerofoil/blob/master/aerofoilDoc.hpp#L21
I think using 1 * si::mm here would start to get on my nerves!
in for loop you would hope for a terse syntax.
https://github.com/kwikius/quan-trunk/blob/master/quan_matters/examples/capacitor_time_curve.cpp#L55
for ( auto t = 0ms ; t <= timeout; ++t  ){     //nice when it works !
for ( auto t = 0 * si::ms ; t <= timeout; ++t  ){ //  maybe !
for ( auto t = si::0ms ; t <= timeout; ++t  ){  // maybe !

In class constructors generally I found explicit init with an explicit value easiest), so UDL irrelevant
https://github.com/kwikius/mavlink_to_frsky/blob/master/stm32f4/aircraft.hpp#L83
Another answer : Provide both options for now ... :)

Actually I am starting to change my mind. Maybe the multiplier syntax is not so bad  :)

Hehe, I see you are going the same path as I did. My initial opinion was also "yack, it is an awful and old way to do it" ;-)

With constants, we could also consider more extensions like:
inline constexpr auto km = k * m;
inline constexpr auto kmph = km/h;

But as we discussed in another thread that will work against the downcasting facility as the user is never providing a user-friendly name for a target type.

with constants those 2 cases would look like:
auto v1 = 120 * km / 2 * h;
auto v2 = length * km / duration * h;

Won't these expressions produce km×h instead of km/h?

@i-ky You are right, I do not have experience with this syntax and did not notice that. But it seems it is a good point against the "multiply" syntax. I will fix the code sample in polls above, thanks!

Another point against UDLs. Arbitrary expressions are not allowed
   auto x = (1./4)sq_m; // error

But a major point against constants is that they clog up the global namespace with many tiny names , h, s, etc. and that could be a real headache in real scale code .
    auto constexpr v1 =  10 * m/s;

    std::cout << v1 << '\n';

    double constexpr s = 2.0;

    auto v2 =  20 * m / s;

    std::cout << v2 << '\n';

https://godbolt.org/z/idqHM4
That doesnt happen with UDLs'.

Personally I dont think item 1 regarding initialisation of quantitites by runtime numeric values on the list above is a problem in practise. Generally a variable quantity is initialised with a constant , but once initialised, it is the quantity that is being used as the runtime variable.
In light of my point about name hiding in the post above, I think that UDLs are the right way to go. Since it is not possible to use qualified lookup in current c++, the only other option is to  add a namespace to the name itself as in the good old C days.
So for example reserve the Q_ prefix for use by std::quantity literals
   auto v1 = 10.0Q_F;
   auto v2 = 10.0Q_J;
   auto v3 = 10.0Q_W;
   auto v4 = 10.0Q_K;
   auto v5 = 10.0Q_d;
   auto v6 = 10.0Q_l;
   auto v7 = 10.0Q_L;
   auto v8 = 10.0Q_erg;
   auto v9 = 10.0Q_ergps;

That solves all items. I think it is correct that the si units take precedence over the other less complete unit systems. SI is the dominant system for very good reason.   For disambiguating non si units, then just extend the namespace. The extra ugliness is justified:
auto v10 = 10.0Q_imp_in;

I think we need feedback from a wider audience on this.
It might make sense to retain both options.

I think q_ prefix has a lot of sense. I will make this change soon.
Also, during the last ISO discussion it was raised that there should be a dedicated operator:
quantity<Dim, U, Rep> operator*(Rep v, magic_type<Dim, U>);

It will only allow the syntax 3 * s where:
inline cosntexpr magic_type<dim_time, second> s;

It will solve the issue with op/ (as it will be an illegal operation)
We can deduce Rep for quantity from the LHS type of the operator
It is easy to disambiguate different systems

I think that both options seem interesting and thus we will probably provide both for now and get some more field experience with both of them. Do you have any suggestions on how to name a magic_type?

Both options are the way to go. Sometimes you want to encode a constant, others, do apples / h, which I prefer very much over apples / hours{1}.
I like magic_type, so I don't have to write apples / h<int> (like with std::numbers). But you mention that operator/ would be ill-formed. I think everything should be allowed everywhere.
I make a point for having a natural syntax when writing formulas with a units library at https://github.com/nholthaus/units/issues/196. In C++20, you can write 1970y/January/1. A units library shouldn't lose to that.
Looking for formulas at https://en.wikipedia.org/wiki/Formula, we see:

units::pow<3>(cm) should work and be the intuitively expected magic_type specialization.
Some formula look like A = x * u1 * u2 / u3, where ux are units. Maybe I have a quantity a representing x * u2 / u3, so I only need to do A(a * u1) in C++.
I think magic_type should just be a quantity with a library-defined representation type that does the right thing when it is involved in an operation with any other representation type, so that everything else remains unchanged.

On a second thought, there'd be cm3, so units::pow<3>(cm) is too round-about, but feasible still when the library doesn't have such constant. Also, point 2 in the OP still applies to the names of these constants. Another advantage of constants relates to the N*M problem. The library only offers q_m_per_s and q_km_per_h, but maybe I want q_km_per_s. With constants, this isn't a problem.

On a second thought, there'd be cm3, so units::pow<3>(cm) is too round-about, but feasible still when the library doesn't have such constant. Also, point 2 in the OP still applies to the names of these constants.
Another advantage of constants relates to the N*M problem. The library only offers q_m_per_s and q_km_per_h, but maybe I want q_km_per_s. With constants, this isn't a problem.

A major practical problem with global constants is they can be hidden by local constants of the same name :
// some short name global constant

double  constexpr s = 0.5;

//----------------------------------
#include <iostream>

int main()
{

   constexpr auto a=1.,b=2.,c=3.,s=0.,k=5.;

   double  y = 1;

   auto  z =  y / s ; //<< --- ouch!

   std::cout << z << '\n';

}

Another advantage of constants relates to the N*M problem. The library only offers q_m_per_s and q_km_per_h, but maybe I want q_km_per_s. With constants, this isn't a problem.

In PQS I solved the problem of ad-hoc units by creating a so-called unit_binary_op that models the pqs::unit concept while providing customised output
Here is how an ad-hoc quantity can be created in source code
https://github.com/kwikius/pqs/blob/master/examples/fountain.cpp#L48
output :
PQS fountain power example
Demo of ad-hoc output units
spray height = 0.6 m
volume per s = 32.3202 cm³⁄s  // <----------- Here is the custom output
mass per s = 32.3202 g⁄s      // here is another ad-hoc unit
fountain output power = 0.190043 W
I show si units there but you can also use non-si /si combination of course and also stack them together

There's also the case of linear algebra and other quantity wrappers.
using namespace units::physical::si;

fs_vector<si::length<si::metre>, 3> v = { 1_q_m, 2_q_m, 3_q_m };
fs_vector<si::length<si::metre>, 3> v = fs_vector<int, 3>{ 1, 2, 3 } * 1_q_m;
fs_vector<si::length<si::metre>, 3> v = fs_vector<int, 3>{ 1, 2, 3 } * m;
auto v = fs_vector<int, 3>{ 1, 2, 3 } * 1_q_m;
auto v = fs_vector<int, 3>{ 1, 2, 3 } * m;

si::length<si::metre, fs_vector<int, 3>> v(fs_vector<int, 3>{ 1, 2, 3 });

Do you think unit constants are an improvement here?

A major practical problem with global constants is they can be hidden by local constants of the same name :

This can be mitigated by using the dimension concepts. Another reason to standardize them.


Consider renaming assemblies to follow Microsoft guidelines

This is related to #2
Microsoft's naming guidelines suggest using a format similar to
<Company>.<Component>.dll

when creating assemblies. The current structure of this project doesn't really work for that, as you've separated the interfaces and implementation into two projects. So, I would recommend a two step modification:

Combine the implementation and interface projects, as you're not really gaining anything by having them split, and
Renaming the project to something like Mehdime.Entity.

I can actually do both of these myself on a fork and submit a pull request, if you prefer. Once that is done, I can compile a single .dll for the NuGet package.
Obviously, this would be a breaking change for anyone using the existing code.

:+1:  for a fork + pull request and a nuget package!

@maldworth If you're interested in being a bit of a test subject, you can try the NuGet package I've uploaded here: https://drive.google.com/open?id=0By4iYTLNHcYRMVJ2SG9xSlMxVWM&authuser=0
Extract the 7zip archive. You'll have to create a local NuGet source, and then you can add this package to a project. Everything was moved to a Mehdime.Entity namespace, but other than that no code changes, so it should just work.

@tiesont I just tried it and the nupkg and it works great. I'm curious if the Interfaces should be either broken into a separate namespace, or a separate package. I like to separate my contracts from implementation. What is the standard nuget packaging guidelines for implementation and contract separation?

@maldworth I don't recall that there are NuGet guidelines for namespaces. I can see separating the interfaces into a separate namespace, but what is gained by doing so? The current codebase actually does this, but also keeps them in separate assemblies. I didn't want to deploy them like that as a NuGet package, so I combined them.
I take the stance that assemblies like System.Web, et al, don't separate namespaces or assemblies at that level; you can organize classes and whatnot into folders within the solution & project, but namespaces don't need to reflect that.

@tiesont Yeah, Nothing is gained in this scenario. I'm good with the package as you provided. Let me know when you upload it to Nuget.org!

@maldworth Uploaded to https://www.nuget.org/packages/Mehdime.Entity/ - let me know if you have any issues with the "live" package.

Can you submit a pull request for this work? It's pretty confusing that the nuget package doesn't match the repository.

@ChrisMarinos I suppose I should. I'll see if I can get around to it at some point.
It's worth pointing out that @mehdime never gave a "yay/nay" response to this suggestion. I went ahead and published the NuGet package with my version mostly because it's what I had handy, and I don't like pushing multiple assemblies in one package if I can avoid it.

Yeah, that makes sense. PR or not, it would just be good to have the code that you used to publish the package somewhere on github.

@ChrisMarinos Pushed code to my fork, and pull request created. We'll see what @mehdime says.

Hi guys,
Pull request merged. And apologies for the silence in the last while - I'll try to address all open issues in the next week. I should have a little bit of free time then.
Thanks a lot!

@mehdime Awesome. Did I ever ask if you have a NuGet account? I'd like to add you to the package as an owner. My contact info is public on my profile, if you can send me a quick email.

Sent email.


Disable AdaptiveSampling when using UseApplicationInsights

There is no way to set EnableAdaptiveSampling to false when using the WebHostBuilder extensions.
public static IWebHostBuilder UseApplicationInsights(this IWebHostBuilder webHostBuilder)
{
    webHostBuilder.ConfigureServices(collection =>
    {
        collection.AddApplicationInsightsTelemetry();
    });

    return webHostBuilder;
}

Should this be possible? Right now I'm having to ditch this extension and do it myself:
webHostBuilder.ConfigureServices(collection =>
{
    collection.AddApplicationInsightsTelemetry((options) =>
    {
        options.InstrumentationKey = appInsightsKey;
        options.EnableAdaptiveSampling = false;
    });
});

If there is need to customize, then use the AddApplicationInsightsTelemetry extension method on IServiceCollection.
https://github.com/Microsoft/ApplicationInsights-aspnetcore/wiki/Getting-Started-with-Application-Insights-for-ASP.NET-Core#option-2-addapplicationinsightstelemetry-preferred-approach-if-customizing-application-insights-settings

That's what I'm doing. Just thought it would make sense to be able to do this as part of UseApplicationInsights


Add option timestamp option to OPC UA adapter

Body
A user should have the configuration possibility to use the timestamp of the OPC UA Server as event time.
Currently the timestamp has to be added in the schema editor by clicking Add timestamp.
With this option, the adapter will add a timestamp, but this might differ from the original time of the data source.
StreamPipes Committer

[X] I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

This issue is an outcome of our discussion in #886

HI @tenthe
can you please assign me this issue, I want to work on this and I want to contribute to OPC-UA Browser for the Apache StreamPipes project for GSOC'23.


fix(ci): fetch the correct sha on PR image builds

Description
Please explain the changes you made here.
Which issue(s) does this PR fix

Fixes #?

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/lgtm


Editors need a way to determine whether they are talking to an official TS Server

Otherwise, they will obliviously forward events from the server to our official telemetry system and we'll have confusing noise in our data (e.g. stacks that don't match our source or asserts that don't exist).
The most secure approach would be to cryptographically hash the official tsserver.js and have the editor only report telemetry for matching files.  This would likely be impractical for, e.g., nightly builds.
An intermediate approach would be to only report telemetry from the in-box implementation (i.e. not if tsserver comes from npm or nuget).  We might lose a lot of data.
Alternatively, our publication process could modify tsserver.js in some way that the editor could detect.  A fork would have to be deliberately malicious to replicate the change (vs now, where they'd have to explicitly suppress events to be good citizens).

FYI @minestarks @mjbvz


WalkUtils: handle unsafe_ref_cast which casts between AnyObject and a class

We need to ignore such casts because otherwise the constructed walking path wouldn't contain the right existential field kind. Fixes a miscompile caused by redundant load elimination.
rdar://132364917

@swift-ci test

@swift-ci benchmark

This test did not report "may alias" before this change. The reason was a mismatch of init- and open existential instructions (unchecked_ref_cast was not considered as potentially "open" the AnyObject existential).
The EscapeInfo walkers push an existential field onto the walking path when walking up an open_existential_ref instruction. And try to pop it when seeing an init_existential_ref. But if there is no such field type in the path, it's considered a path mismatch which aborts the walk and the result is a no-escape.

Thanks. I understand. What is the purpose of creating a path component for init_existential_ref %0 : $X : $X, $AnyObject if it's not because of the type change? Converting to AnyObject from a class can't change the representation right? Then you could continue ignoring unchecked_ref_cast

Yes, ignoring creating AnyObject existentials would be a possibility. Currently we don't treat AnyObject different than other existentials.


Build fails

I could not build the adapter on a new Linux box due to compilation errors.
Also, the build instructions miss some critical steps. I had to debug make not finding the go libraries that glide downloaded because they were not in $HOME/go/src.

Could you provide some more information on the compilation errors you are seeing.
FWIW, Glide isn't supposed to install dependencies in $HOME/go/src. It puts things in the vendor directory.

Hi. I've installed Go 1.9 and Glide on a clean Ubuntu container according to https://github.com/golang/go/wiki/Ubuntu and https://glide.sh/ respectively.
I've set $GOPATH=$HOME/go
I then cloned this repo to $HOME and followed https://github.com/timescale/prometheus-postgresql-adapter#building :

glide install --strip-vendor --strip-vcs output:

root@633482f8518d:~/prometheus-postgresql-adapter# glide install --strip-vendor --strip-vcs
[WARN]	The --strip-vcs flag is deprecated. This now works by default.
[WARN]	The name listed in the config file (github.com/timescale/prometheus-timescaledb-adapter) does not match the current location (.)
[INFO]	Downloading dependencies. Please wait...
[INFO]	--> Found desired version locally github.com/beorn7/perks 4c0e84591b9aa9e6dcfdf3e020114cd81f89d5f9!
[INFO]	--> Found desired version locally github.com/gogo/protobuf 100ba4e885062801d56799d78530b73b178a78f3!
[INFO]	--> Found desired version locally github.com/golang/protobuf 18c9bb3261723cd5401db4d0c9fbc5c3b6c70fe8!
[INFO]	--> Found desired version locally github.com/golang/snappy 553a641470496b2327abcac10b36396bd98e45c9!
[INFO]	--> Found desired version locally github.com/lib/pq 2704adc878c21e1329f46f6e56a1c387d788ff94!
[INFO]	--> Found desired version locally github.com/matttproud/golang_protobuf_extensions c12348ce28de40eed0136aa2b644d0ee0650e56c!
[INFO]	--> Found desired version locally github.com/prometheus/client_golang c5b7fccd204277076155f10851dad72b76a49317!
[INFO]	--> Found desired version locally github.com/prometheus/client_model 6f3806018612930941127f2a7c6c453ba2c527d2!
[INFO]	--> Found desired version locally github.com/prometheus/common 13ba4ddd0caa9c28ca7b7bffe1dfa9ed8d5ef207!
[INFO]	--> Found desired version locally github.com/prometheus/procfs d098ca18df8bc825079013daf7bacefbb1ee877e!
[INFO]	--> Found desired version locally github.com/prometheus/prometheus 4666df502c0e239ed4aa1d80abbbfb54f61b23c3!
[INFO]	--> Found desired version locally github.com/Sirupsen/logrus 10f801ebc38b33738c9d17d50860f484a0988ff5!
[INFO]	--> Found desired version locally github.com/syndtr/goleveldb 8c81ea47d4c41a385645e133e15510fc6a2a74b4!
[INFO]	--> Found desired version locally golang.org/x/net feeb485667d1fdabe727840fe00adc22431bc86e!
[INFO]	--> Found desired version locally golang.org/x/sys 9ccfe848b9db8435a24c424abbc07a921adf1df5!
[INFO]	--> Found desired version locally golang.org/x/time 8be79e1e0910c292df4e79c241bb7e8f7e725959!
[INFO]	--> Found desired version locally gopkg.in/yaml.v2 cd8b52f8269e0feb286dfeef29f8fe4d5b397e0b!
[INFO]	Setting references.
[INFO]	--> Setting version for github.com/prometheus/client_golang to c5b7fccd204277076155f10851dad72b76a49317.
[INFO]	--> Setting version for github.com/prometheus/prometheus to 4666df502c0e239ed4aa1d80abbbfb54f61b23c3.
[INFO]	--> Setting version for github.com/beorn7/perks to 4c0e84591b9aa9e6dcfdf3e020114cd81f89d5f9.
[INFO]	--> Setting version for github.com/golang/protobuf to 18c9bb3261723cd5401db4d0c9fbc5c3b6c70fe8.
[INFO]	--> Setting version for github.com/prometheus/client_model to 6f3806018612930941127f2a7c6c453ba2c527d2.
[INFO]	--> Setting version for github.com/prometheus/common to 13ba4ddd0caa9c28ca7b7bffe1dfa9ed8d5ef207.
[INFO]	--> Setting version for github.com/Sirupsen/logrus to 10f801ebc38b33738c9d17d50860f484a0988ff5.
[INFO]	--> Setting version for github.com/syndtr/goleveldb to 8c81ea47d4c41a385645e133e15510fc6a2a74b4.
[INFO]	--> Setting version for github.com/prometheus/procfs to d098ca18df8bc825079013daf7bacefbb1ee877e.
[INFO]	--> Setting version for github.com/matttproud/golang_protobuf_extensions to c12348ce28de40eed0136aa2b644d0ee0650e56c.
[INFO]	--> Setting version for github.com/golang/snappy to 553a641470496b2327abcac10b36396bd98e45c9.
[INFO]	--> Setting version for github.com/lib/pq to 2704adc878c21e1329f46f6e56a1c387d788ff94.
[INFO]	--> Setting version for github.com/gogo/protobuf to 100ba4e885062801d56799d78530b73b178a78f3.
[INFO]	--> Setting version for gopkg.in/yaml.v2 to cd8b52f8269e0feb286dfeef29f8fe4d5b397e0b.
[INFO]	--> Setting version for golang.org/x/net to feeb485667d1fdabe727840fe00adc22431bc86e.
[INFO]	--> Setting version for golang.org/x/time to 8be79e1e0910c292df4e79c241bb7e8f7e725959.
[INFO]	--> Setting version for golang.org/x/sys to 9ccfe848b9db8435a24c424abbc07a921adf1df5.
[INFO]	Exporting resolved dependencies...

[INFO]	Removing: /root/prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/vendor
[INFO]	Removing: /root/prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/web/ui/static/vendor

make output:

root@633482f8518d:~/prometheus-postgresql-adapter# make
GOOS=linux GOARCH=amd64 CGO_ENABLED=0 go build -a -installsuffix cgo --ldflags '-w' -o prometheus-postgresql-adapter main.go
main.go:29:2: cannot find package "github.com/gogo/protobuf/proto" in any of:
	/usr/lib/go-1.9/src/github.com/gogo/protobuf/proto (from $GOROOT)
	/root/go/src/github.com/gogo/protobuf/proto (from $GOPATH)
main.go:30:2: cannot find package "github.com/golang/snappy" in any of:
	/usr/lib/go-1.9/src/github.com/golang/snappy (from $GOROOT)
	/root/go/src/github.com/golang/snappy (from $GOPATH)
main.go:31:2: cannot find package "github.com/prometheus/client_golang/prometheus" in any of:
	/usr/lib/go-1.9/src/github.com/prometheus/client_golang/prometheus (from $GOROOT)
	/root/go/src/github.com/prometheus/client_golang/prometheus (from $GOPATH)
main.go:32:2: cannot find package "github.com/prometheus/common/log" in any of:
	/usr/lib/go-1.9/src/github.com/prometheus/common/log (from $GOROOT)
	/root/go/src/github.com/prometheus/common/log (from $GOPATH)
main.go:33:2: cannot find package "github.com/prometheus/common/model" in any of:
	/usr/lib/go-1.9/src/github.com/prometheus/common/model (from $GOROOT)
	/root/go/src/github.com/prometheus/common/model (from $GOPATH)
main.go:36:2: cannot find package "github.com/prometheus/prometheus/storage/remote" in any of:
	/usr/lib/go-1.9/src/github.com/prometheus/prometheus/storage/remote (from $GOROOT)
	/root/go/src/github.com/prometheus/prometheus/storage/remote (from $GOPATH)
main.go:34:2: cannot find package "github.com/timescale/prometheus-postgresql-adapter/postgresql" in any of:
	/usr/lib/go-1.9/src/github.com/timescale/prometheus-postgresql-adapter/postgresql (from $GOROOT)
	/root/go/src/github.com/timescale/prometheus-postgresql-adapter/postgresql (from $GOPATH)
Makefile:30: recipe for target 'prometheus-postgresql-adapter' failed
make: *** [prometheus-postgresql-adapter] Error 1

root@633482f8518d:~/prometheus-postgresql-adapter# ls -al ~/go/src/
total 0
drwxr-xr-x 2 root root  6 Dec 10 22:01 .
drwxr-xr-x 4 root root 28 Dec 10 22:01 ..

I created /tmp/go/src and cloned into there which gets past the error above. glide install seems to work, but make then fails.
root@prometheus30517:/tmp/go/src/prometheus-postgresql-adapter-master# make GOOS=linux GOARCH=amd64 CGO_ENABLED=0 go build -a -installsuffix cgo --ldflags '-w' -o prometheus-postgresql-adapter main.go main.go:34:2: cannot find package "github.com/timescale/prometheus-postgresql-adapter/postgresql" in any of: 	/tmp/go/src/prometheus-postgresql-adapter-master/vendor/github.com/timescale/prometheus-postgresql-adapter/postgresql (vendor tree) 	/usr/lib/go-1.7/src/github.com/timescale/prometheus-postgresql-adapter/postgresql (from $GOROOT) 	/tmp/go/src/github.com/timescale/prometheus-postgresql-adapter/postgresql (from $GOPATH) Makefile:30: recipe for target 'prometheus-postgresql-adapter' failed make: *** [prometheus-postgresql-adapter] Error 1

further to my last comment, two warnings in the glide output
[WARN]	The --strip-vcs flag is deprecated. This now works by default.
[WARN]	The name listed in the config file (github.com/timescale/prometheus-timescaledb-adapter) does not match the current location (prometheus-postgresql-adapter-master)

this would explaint the timescale error

...so I took out the first line of glide.yml, removing the timescale reference. seems to build now.

I've set GOPATH=/tmp/go, cloned this repo into /tmp/go/src, deleted the first line of glide.yaml but it still will not compile:
root@2df305cad2e5:/tmp/go/src/prometheus-postgresql-adapter# make
GOOS=linux GOARCH=amd64 CGO_ENABLED=0 go build -a -installsuffix cgo --ldflags '-w' -o prometheus-postgresql-adapter main.go
main.go:34:2: cannot find package "github.com/timescale/prometheus-postgresql-adapter/postgresql" in any of:
	/tmp/go/src/prometheus-postgresql-adapter/vendor/github.com/timescale/prometheus-postgresql-adapter/postgresql (vendor tree)
	/usr/lib/go-1.9/src/github.com/timescale/prometheus-postgresql-adapter/postgresql (from $GOROOT)
	/tmp/go/src/github.com/timescale/prometheus-postgresql-adapter/postgresql (from $GOPATH)
Makefile:30: recipe for target 'prometheus-postgresql-adapter' failed
make: *** [prometheus-postgresql-adapter] Error 1

Glide was complaining that [WARN]	Lock file may be out of date. Hash check of YAML failed. You may need to run 'update', so I ran glide update and then make again and it worked!
Anyway, now I'm running into fatal errors when running the adapter:
root@2df305cad2e5:/tmp/go/src/prometheus-postgresql-adapter# ./prometheus-postgresql-adapter
./prometheus-postgresql-adapter flag redefined: log.level
panic: ./prometheus-postgresql-adapter flag redefined: log.level

goroutine 1 [running]:
flag.(*FlagSet).Var(0xc420056060, 0xc81560, 0xc42000fa20, 0x9b228b, 0x9, 0x9cd410, 0x63)
	/usr/lib/go-1.9/src/flag/flag.go:793 +0x5e1
prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/vendor/github.com/prometheus/common/log.AddFlags(0xc420056060)
	/tmp/go/src/prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/vendor/github.com/prometheus/common/log/log.go:118 +0xc9
prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/vendor/github.com/prometheus/common/log.init.0()
	/tmp/go/src/prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/vendor/github.com/prometheus/common/log/log.go:110 +0x2d
prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/vendor/github.com/prometheus/common/log.init()
	<autogenerated>:1 +0xc0
prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/storage/local.init()
	<autogenerated>:1 +0x64
prometheus-postgresql-adapter/vendor/github.com/prometheus/prometheus/storage/remote.init()
	<autogenerated>:1 +0xbc
prometheus-postgresql-adapter/vendor/github.com/timescale/prometheus-postgresql-adapter/postgresql.init()
	<autogenerated>:1 +0x84
main.init()
	<autogenerated>:1 +0x9e

I'm getting identical as @alexisph

Built with prometheus repo in GOPATH/src/
ran glide install --strip-vendor , glide update, make
attempted to run using envars found in docker container:

PG_PROMETHEUS_ENV_PGDATA
PG_PROMETHEUS_ENV_PG_VERSION
PG_PROMETHEUS_ENV_LANG
PG_PROMETHEUS_PORT_5432_TCP_ADDR
PG_PROMETHEUS_PORT_5432_TCP_PORT
PG_PROMETHEUS_PORT_5432_TCP_PROTO
PG_PROMETHEUS_ENV_PG_PROMETHEUS_VERSION
PG_PROMETHEUS_PORT
PG_PROMETHEUS_ENV_TIMESCALEDB_VERSION
PG_PROMETHEUS_ENV_PG_SHA256
PG_PROMETHEUS_ENV_PG_MAJOR
PG_PROMETHEUS_NAME
PG_PROMETHEUS_PORT_5432_TCP

A new version of the adapter will be released soon (coming days) and it should take care of most of these issues.

@alexisph Just released v0.2. Note the new build instructions and switch to Dep for dependency management. Let me know if you continue to experience build issues.


Addon stops when connection/forwarding fails

We had a power outage today. This lead to a weird constellation in which the autossh-addon wasn't able to connect and stopped after the run script was finished.
I assume there are similar causes for connection issues and I would like to ensure that the addon never stops trying to reconnect.
I see three options to fix this:

Fix my specific issue by setting the following line to "no" - not sure what that will do exactly https://github.com/ThomDietrich/home-assistant-addons/blob/01dce2b79dda9e6488c66bec46826f52f6ff3f57/autossh/run.sh#L83
Put the autossh command into an endless loop - that would certainly help
Make use of supervisor watchdog "will restart the addon when it fails" - I don't know how it works and what fails means. Is it a simple exit 1?

Option 3 is my favorite overall.
What do you think?

I've encountered this issue a few times now so I'm keen to get this resolved.
Option 1 will not do anything. That setting only controls the behavior of the SSH client process when the SSH connection itself succeeded but forwarding the port failed (let's say the remote port is already occupied on the remote host).
In our case the issue is very likely to be networking (the add-on starts trying to connect before the network connection is up) so we wouldn't even get to a stage where that setting becomes relevant anyway, but even if it did, having ssh stay running after failing to forward the port would be useless to us anyway.
Option 2 is the solution.
Option 3 is not a full solution.  The problem is that it has a max number of restarts and/or has a "burst" limit where lots of restarts in a short time will cause the watchdog to give up.

@Rjevski thanks for your response. I favored option 3 under the assumption that this watchdog behavior can be influenced. It did follow an infinite restart pattern some time ago and the "giving up" strategy is changed behavior since then. I wonder if that can be configured per add-on, which would make this the best solution. Long story short: I never looked into the behavior of the watchdog. Do you have any further knowledge?
After we have rules out option 3 as a viable option, I am more than happy to implement option 2 or to accept a PR.

I have opened the PR above - would you be able to test the following:

starting add-on with broken/unavailable SSH server - you should see the add-on continue to run and retry forever - if you fix your SSH server it should automatically connect
stopping the add-on from the Home Assistant add-ons UI - the add-on should still stop properly.

Should be resolved in latest release.


Performance/Detect Cop and Enumerator::Lazy

Hi there. First, many thanks for the great work :)
Performance/Detect doesn't look to handle Enumerator::Lazy instances.
2.2.2 :001 > [1, 2, 3, 4, 5].lazy.select { i % 2 == 0 }.first(2) => [2, 4]
Not sure if this is known and/or planning in supporting it ?
One workaround is to take(n).to_a

My guess is that lazy was not considered when writing the cop. This sounds like a reasonable request to not register an offense when lazy is used. This is probably impacting Performance/Count as well. This should be pretty easy to implement.


Unable to issue / renew cert anymore

I'm hoping this isn't a "me" problem but no matter what I do I am unable to get wildcard to work via porkbun dns / certbot.
It did work but not I get the error:
2021-05-23 14:24:23,496:DEBUG:urllib3.connectionpool:https://porkbun.com:443 "POST /api/json/v3/dns/create/co.uk HTTP/1.1" 400 None
2021-05-23 14:24:23,499:DEBUG:certbot._internal.error_handler:Encountered exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/certbot_dns_porkbun/cert/client.py", line 87, in _perform
self.record_ids[validation] = self._get_porkbun_client().dns_create(root_domain, "TXT", validation,
File "/usr/local/lib/python3.9/dist-packages/pkb_client/client.py", line 115, in dns_create
raise Exception("ERROR: DNS create api call was not successfully\n"
Exception: ERROR: DNS create api call was not successfully
Status code: 400
Message: Invalid domain.
I have checked and checked but I can not work out what's wrong.

The error message indicates that you are trying to create a DNS record for a domain that does not belong to you or does not exists:

Exception: ERROR: DNS create api call was not successfully
Status code: 400
Message: Invalid domain.

The error message also states that you are trying to generate a certificate for the domain co.uk:

/api/json/v3/dns/create/co.uk

I assume that this is not the actual domain but the top and second level domain of your actual domain like example.co.uk.
Could you please provide the certbot command, certbot/certbot_dns_porkbun versions and especially the domain you used there in an anonymized form:

replace any secrets with placeholder like --dns-porkbun-key <my-secret-token>
replace parts of the domain but keep the structure for subdomains, second level and top level domains
examples:

mhisani.co.uk -> example.sld.tld
*.mhisani.co.uk -> *.example.sld.tld
example.mhisani.co.uk -> example.mydomain.sld.tld
mhisani.com -> example.tld

Thankyou very much for your fast reply and time! As requested:

certbot certonly   --non-interactive   --agree-tos   --email<EMAIL_ADDRESS>--preferred-challenges dns   --authenticator dns-porkbun   --dns-porkbun-key   --dns-porkbun-secret  --dns-porkbun-propagation-seconds 60   -d *.someexampledomain.co.uk

Encountered exception during recovery: KeyError: 'SomeLongKeyHere'
ERROR: DNS create api call was not successfully
Status code: 400
Message: Invalid domain.

Thanks for the further details.
It seems the correct processing of arbitrary number of subdomains which are part of the top level domain (example.abc.com, example.xyz.abc.com) does not work and result in a wrong domain name. I have found the cause and will fix it as soon as possible in a hotfix release.


Array data type throws exception with recent changes to highlight-adapter.ts (v0.5.7+)

Description
Array type data being processed for highlighting throws an exception originating from highlight-adapter.ts
"product__tags__ids": [
  67,
  80
],

Expected behavior
No exceptions should be thrown, array data types should be correctly processed.
Current behavior
Array type data being processed for highlighting throws an exception originating from highlight-adapter.ts
Screenshots or Logs
Uncaught (in promise) TypeError: can't access property "replace", _escape__WEBPACK_IMPORTED_MODULE_0__["default"](...) is undefined
    replaceTagsAndEscape escape-highlight.js:21
    recursiveEscape escape-highlight.js:36
    recursiveEscape escape-highlight.js:27
    recursiveEscape escape-highlight.js:26
    recursiveEscape escape-highlight.js:32
    recursiveEscape escape-highlight.js:27
    recursiveEscape escape-highlight.js:26
    hits escape-highlight.js:48
    escapeHits escape-highlight.js:44
    getWidgetRenderState connectInfiniteHits.js:174
    getRenderState connectInfiniteHits.js:142
    render index.js:453
    render index.js:451
    InstantSearch InstantSearch.js:117
    progress defer.js:24
    replaceTagsAndEscape escape-highlight.js:21
    recursiveEscape escape-highlight.js:36
    recursiveEscape escape-highlight.js:27
    recursiveEscape escape-highlight.js:26
    recursiveEscape escape-highlight.js:32
    recursiveEscape escape-highlight.js:27
    recursiveEscape escape-highlight.js:26
    hits escape-highlight.js:48
    escapeHits escape-highlight.js:44
    getWidgetRenderState connectInfiniteHits.js:174
    getRenderState connectInfiniteHits.js:142
    render index.js:453
    render index.js:451
    InstantSearch InstantSearch.js:117
    progress defer.js:24
    promise callback*fn defer.js:16
    init index.js:367
    emit events.js:81
    _dispatchAlgoliaResponse algoliasearch.helper.js:1315
    _dispatchAlgoliaResponse algoliasearch.helper.js:1307
    promise callback*49374/AlgoliaSearchHelper.prototype._search algoliasearch.helper.js:1271
    searchOnlyWithDerivedHelpers algoliasearch.helper.js:154
    search InstantSearch.js:386
    search InstantSearch.js:407
    g events.js:165
    emit events.js:81
    _search algoliasearch.helper.js:1240
    search algoliasearch.helper.js:149
    searchFunction shop-search.vue:632
    search InstantSearch.js:415
    InstantSearch InstantSearch.js:106
    progress defer.js:24
    promise callback*fn defer.js:16
    removeWidgets index.js:271
    handler widget.js:1
    VueJS 12
    searchFunction shop-search.vue:617
    search InstantSearch.js:415
    InstantSearch InstantSearch.js:106
    progress defer.js:24
    promise callback*fn defer.js:16
    start InstantSearch.js:440
    mounted createInstantSearchComponent.js:1
    VueJS 5
    mounted createInstantSearchComponent.js:1
    VueJS 9
escape-highlight.js:21:15

Environment (please complete the following information):

OS: Ubuntu 20.04
Browser: Firefox 94.0b9 (64-bit)
MeiliSearch version: 0.23.1
instant-meilisearch version: 0.5.7 – 0.5.9
instantsearch.js version: 4.33.0

Hey @chamby thanks for raising this error. I will look into it :)

Could you show me how you configurated your instantsearch components for product__tags__ids ?

Actually, I am not making use of product__tags__ids. All attributes are being fetched and highlighted in the query, not explicitly, just by default.
"attributesToHighlight": [
  "*"
],

In the course of processing the response, this exception is thrown.

Thanks @chamby I will look into this. Sorry for the bug. If you have any idea on how to resolve it feel free to make a PR. If not, I will fix it asap

Hey @chamby, I made a fix. It should be released next week if it is approved.
If you want you can already try it out on this branch https://github.com/meilisearch/instant-meilisearch/tree/fix_highlighting_on_undefined


questions from stackoverflow not displaying

After selecting fetch questions on  http://erwaysoftware.com/blaze/# , Questions are not displayed it is giving error as  "Uncaught ReferenceError: isLoggedIn is not defined" as shown in the image below.

Browser:  Google chrome Version 51.0.2704.79 m

@SandeepBiradar Questions haven't had much attention at all, really. I've committed something should fix this, but there may well be other unfixed bugs in the questions view.
Waiting on @Undo1 to deploy this.

Deployed; looks like 52d0944 from Art fixed this.


Updating zero bound check to include document level check

Updating zero bound check to include document level check also. In some cases docInfo is not updated and it ends up getting zero bounds at the doucment level also. In such a case it's better to get bounds using  _getSettingsWithExactBounds
This was squashed and split off of PR #369, so see conversation there.

Looks good. grunt passes. Merging.


Bundle of wmagent patches for 1.1.14.patch2

wmagent version (1st commits) of:
#8653 Added exit code 8033 - FileWriteError - to the list of exceptions
#8651 Reduce the BossAir memory footprint
#8615 Strip the requirement string and proxy for Volunteer jobs
#8650 Limit the number of archiveable jobs to be loaded each cycle

 Fix #8657 - Bundle of wmagent patches for 1.1.14.patch2


Update to Selenium 4.6.0

Plus addition of new libs needed by 4.6.0.

Are you planning to fix the broken CI @rbourga ?

The version of plugin built was 4.5.1 and it's already been published on Maven
See https://github.com/undera/jmeter-plugins-webdriver/pull/21/files#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d8R7

Done


Update UNIXPATH

To prevent infinite back tracking.
This change is a mindless port of logstash-pattern-core which was made to addrses %UNIXPATH can cause RegEx DOS

@daschl can I get an up or down on this pull request?  This issue was discovered during automated vulnerability scanning done by an off-the-shelf scanner.

@daschl can I get an up or down on this pull request?  Thank you for your time.

Thank you for your contribution @froismo!
I am not able to maintain the repository at the moment. While I am planning to do again in the future (no date I can share though), if you need to make these changes for now I suggest forking the repository and working off of your own fork - thanks for your understanding!


[CI] the action just in main branch

The actions need to be executed when they occur in the 'main' branch

the pull request does not correspond, because mix two of them


Docker multiple build

Now we can publish application to multiple arch package like linux/amd64 that run on docker on WSL2 for Windows x64 and linux/arm64 that run on docker on Raspberry Pi and other arm64 arch

Ooow god ! french comments are so ugly, I'm going to change that quickly, not like their government. 🧨


logo src updated

@all-contributors please add @Hilver for infrastructure, tests and code

@all-contributors please add @Hilver for infrastructure, tests and code


Incomplete error message

Hi,
I am using the bioconda installation of ReSeq. When I try to run the illuminaPE command, I get the following error:
>>> 27-07-21 23:39:39: info:  60150316 (15%) were in an unmapped pair without adapters detected
!!! bool reseq::DataStats::SignsOfPairsWithNamesNotIdentical() [/opt/conda/conda-bld/reseq_1607076765868/work/reseq/DataStats.cpp:563]: error: Read names of reads from the same pair are not identical. Cannot continue.
Example position: 36177584:32669
Example read name: 0(� �'�DSXY:2:1165:32380:16986

I am not sure I get understand what this error mean. If two reads have different names then they would be different pairs, right? So, what is happening here? I am quite optimistic that the BAM file is correct and does not have discrepancies in read-names, so this is a bit surprising.
Still, it is great to have a clear error message, thanks for that. However, there is no read at the 36177584 position with 32380 or 16986 in the read name.
11:27 goel@pc-t7-095 goel:~$ samtools view $BAM CUR1G:36177580-36177590 | grep '32380'
11:27 goel@pc-t7-095 goel:~$ samtools view $BAM CUR1G:36177580-36177590 | grep '16989'

Also, the read-name is not properly decoded and shows binary characters. It would be really helpful in solving the issue if the exact position (chromosome:position) and the correct read-name is printed.
Thanks.

Thank you for testing out this error message. I fixed the issue with the non-readable read name and improved the error message. I will push it to the devel branch once the run on a test dataset completes, to check if I introduce new problems. I will see to your installation issue next, so you can actually use the devel branch.
The position tells you the number of the reference sequence and the starting position of the read (both starting from 0). The error occurs because ReSeq could not find the other read from the pair (based on the name). So either they have different names or reads are missing. I would suggest you grep the whole bam file for "DSXY:2:1165:32380:16986" or wait until I fixed your installation issue and you get a better error message.

Since the error message is fixed, I am closing the issue. Feel free to reopen it, if the bam file is correct and ReSeq is still crashing.


Setup Azure Pipelines

cc @alexcrichton this needs enabling Azure Pipelines for this repo

@gnzlbg: no appropriate reviewer found, use r? to override

cc @pietroalbini

It's just sort of how it works but https://github.com/rust-lang/libc/pull/1377 is the branch auto-created by azure pipelines, and it should be configured now. It may or may not require actually using that particular branch, I'm not sure?

The badge is:

@alexcrichton I see an azure job here that appears to properly show errors in the CI script :)

👍
Feel free to cc me if any further configuration is needed.

:umbrella: The latest upstream changes (presumably #1419) made this pull request unmergeable. Please resolve the merge conflicts.

So this is all green except for aarch64-unknown-linux-android, which appears to be missing for some reason under azure pipelines, but AFAICT it is there under travis. cc @alexcrichton @mati865 any idea what might be going on ?

@gnzlbg it should be aarch64-linux-android.

Ok, so this is ready. There are two build jobs failing, Semver Linux&OSX, but these are expected to fail since we did some breaking changes. I've tried to use continueOnError: true to allow them to fail, without much luck, so I've temporarily disable them so that we can land this, and will re-enable them after we do a new release. If anybody knows what the Azure alternative for Travis allow_failures is please let me know.
I think we need to land the PR to rust-central-station for bors to start using azure pipelines for this repo.

Yeah, it's not possible right now https://github.com/microsoft/azure-pipelines-tasks/issues/9302

@mati865 I've tried to deploy your mingw appveyor linking fix without much luck - it also appears that the incorrect rustc version is being picked (1.35.0 as opposed to nightly). Any ideas ?

If toolchain is installed by rustup then C:\Program Files (x86)\Rust\lib\rustlib\%TARGET%\lib won't be valid location. Instead it should be something like %USERPROFILE%\.rustup\toolchains\nightly-%TARGET%\lib\rustlib\%TARGET%\lib.
I don't know Windows cmd well but with bash I'd use this location:
`rustc --print sysroot`/lib/rustlib/$TARGET/lib

It won't work for Rust installed via installer but should be fine for stable/beta/nightly installed by rustup.

D:\a\1\s>xcopy /Y "C:\msys64\mingw64\x86_64-w64-mingw32\lib\crt2.o" "C:\Program Files (x86)\Rust\lib\rustlib\x86_64-pc-windows-gnu\lib"
File not found - crt2.o
0 File(s) copied

Logs from previous builds are not available, they could have location of MSYS2. Maybe it's not installed or installed to different directory.

if %ARCH% EQU i686 choco install mingw --x86 --force part worked fine.
It's just copying from wrong directory to wrong directory:
D:\a\1\s>xcopy /Y "C:\ProgramData\Chocolatey\lib\mingw\tools\install\mingw64\x86_64-w64-mingw32\lib\crt2.o" "C:\Program Files\Rust\.rustup\toolchains\-i686-pc-windows-gnu\lib\rustlib\i686-pc-windows-gnu\lib\crt2.o"
File not found - crt2.o
0 File(s) copied

For i686 that would be C:\ProgramData\Chocolatey\lib\mingw\tools\install\mingw32\i686-w64-mingw32\lib\crt2.o.
C:\Program Files\Rust\.rustup\toolchains\-i686-pc-windows-gnu\lib\rustlib\i686-pc-windows-gnu\lib is just totally wrong.
For rustup toolchains it should be C:\Users\VssAdministrator\.rustup\toolchains\nightly-x86_64-pc-windows-gnu\lib\rustlib\x86_64-pc-windows-gnu\lib. https://github.com/rust-lang/libc/pull/1376#discussion_r301555317

So this is ready and green. We now spawn the jobs hierarchically, so it shouldn't fill the queue either one a single PR is being tested.
When multiple PRs are on flight at the same time, Azure does not appear to have an option to limit the number of concurrent jobs per repository.

Just in case you want to see bash version of mingw script: https://dev.azure.com/mati865/_git/libc?path=%2Fci%2Fazure-install-rust.yml&version=GCcc8997972d5be0cf92a0f348970f2ab005ee6c46&line=49&lineStyle=plain&lineEnd=57&lineStartColumn=1&lineEndColumn=37
It takes path from rustc so there should be no problem if rustup is installed in weird place.
Passed tests:
https://dev.azure.com/mati865/libc/_build/results?buildId=20&view=results

:umbrella: The latest upstream changes (presumably c8aa8ec72d631bc35099bcf5d634cf0a0b841be0) made this pull request unmergeable. Please resolve the merge conflicts.

@bors: r+

:pushpin: Commit e966c5e1cb2dc011cc3ee2f1961b3c872416d5fa has been approved by gnzlbg

:hourglass: Testing commit e966c5e1cb2dc011cc3ee2f1961b3c872416d5fa with merge 78de950c00fd7716a61748c7cdfb8d33c44446cf...

:boom: Test timed out

@bors: retry

:hourglass: Testing commit e966c5e1cb2dc011cc3ee2f1961b3c872416d5fa with merge 8bf72cdd7f134d9152e9847a621a55d350ca4f8b...

@bors: r+

:pushpin: Commit 59234646b7b64517c758c408f5a2d644ab5b28fe has been approved by gnzlbg

:hourglass: Testing commit 59234646b7b64517c758c408f5a2d644ab5b28fe with merge b00bcbc4a5adcfbfd13504999b6481898babad03...

:boom: Test timed out

@pietroalbini this is timing out for some reason.

Oh, I figured out what the problem is. You're running Azure Pipelines only on the auto and try branches, but bors is configured to use auto-libc instead of auto. If you replace the trigger with ["auto-libc", "try"] everything should work.

This is blocked on https://github.com/rust-lang/rust/issues/63115

@bors: r+

:pushpin: Commit f3d0118751bc434ada3dc20895e1ae586444172d has been approved by gnzlbg

:hourglass: Testing commit f3d0118751bc434ada3dc20895e1ae586444172d with merge 62c0d6fc9cb6deec8e06e5b5cf32282778cd0eb3...

:boom: Test timed out

@bors: retry

:hourglass: Testing commit f3d0118751bc434ada3dc20895e1ae586444172d with merge a98ddf5f6cbd0595015b4db58beececfc9636841...

:boom: Test timed out

@bors: retry

:hourglass: Testing commit f3d0118751bc434ada3dc20895e1ae586444172d with merge 73869e11af39403b4216971d3ad8226b8ea3e5d0...

@bors: r+

:pushpin: Commit e63e7d4180f835b7113d9550d9a485aa1858f7eb has been approved by gnzlbg

:hourglass: Testing commit e63e7d4180f835b7113d9550d9a485aa1858f7eb with merge 6a27ab19911ac9c457170687ea4ed65097e2095b...

@bors: r+

:pushpin: Commit bce9075570889b0fc20aad628594db9555fa194a has been approved by gnzlbg

:hourglass: Testing commit bce9075570889b0fc20aad628594db9555fa194a with merge f79ce35f78ff31eb57024a7bd30fd1a4cd580f4e...

:boom: Test timed out

@rust-lang/infra any idea on why bors is continuously timing out ?

@bors delegate=pietroalbini

:v: @pietroalbini can now approve this pull request

@bors try

:hourglass: Trying commit e78222153b8aa93d867fdc9ef65d7f39376fcf25 with merge 37d1bf493f3b75a975b074bfd28ce5116369bb4f...

@bors retry

:hourglass: Trying commit e78222153b8aa93d867fdc9ef65d7f39376fcf25 with merge 13b368c0c4897bdf1b12f5a0467bca2cde0149dd...

Ok, bors successfully recognized Azure Pipelines :tada:

:sunny: Try build successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Build commit: 13b368c0c4897bdf1b12f5a0467bca2cde0149dd

@bors: r+

:pushpin: Commit bce9075570889b0fc20aad628594db9555fa194a has been approved by gnzlbg

:hourglass: Testing commit bce9075570889b0fc20aad628594db9555fa194a with merge b0a19dd0e8a6d5396add591f6c508d8319d61d45...

:boom: Test timed out

This timeout appears to be correct. Azure Pipelines started executing some jobs, but at some point it hanged and stopped starting some of them. Cirrus-CI also appears to have hanged, with some jobs running for an unreasonable amount of time.
I've cancelled the Azure jobs but cannot cancel the Cirrus one, so I'll wait till the queue there is clean and try again.

@bors: retry

:hourglass: Testing commit bce9075570889b0fc20aad628594db9555fa194a with merge 47f056ab9aa27ee31dde71d8fa28637db9c857d6...

Something is wrong with Azure:
401 - Uh oh, you’re locked out.
Your account, <hidden> is not authorized to view this page. Make sure the URL is correct and your account has access. View a list of your organizations here. You may request access by contacting the admin.
08.08.2019 10:18:35 (UTC)
c0e79bdd-e865-496b-95bc-1dc982002779

:boom: Test timed out

@mati865
I think the homu azure job is still running, but it is only running one concurrent task, probably because the i686-pc-windows-gnu task (https://dev.azure.com/rust-lang2/libc/_build/results?buildId=65) failed with:
D:\a\1\s>if "i686" == "i686" choco install mingw --x86 --force
Chocolatey v0.10.15
Installing the following packages:
mingw
By installing you accept licenses for the packages.
mingw v8.1.0 already installed. Forcing reinstall of version '8.1.0'.
 Please use upgrade if you meant to upgrade to a new version.
mingw not installed. An error occurred during installation:
 The remote server returned an error: (503) Server Unavailable. Service Unavailable
This is try 1/3. Retrying after 300 milliseconds.
 Error converted to warning:
 Could not find a part of the path 'C:\ProgramData\chocolatey\lib\mingw\.chocolateyPending'.
This is try 2/3. Retrying after 400 milliseconds.
 Error converted to warning:
 Could not find a part of the path 'C:\ProgramData\chocolatey\lib\mingw\.chocolateyPending'.
Maximum tries of 3 reached. Throwing error.

Chocolatey installed 0/0 packages.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Could not find a part of the path 'C:\ProgramData\chocolatey\lib\mingw\.chocolateyPending'.
##[error]Cmd.exe exited with code '1'.

@bors: retry

:hourglass: Testing commit bce9075570889b0fc20aad628594db9555fa194a with merge 27399bf16d369e94e4773791a4804ebd13ec1765...

I cannot access Azure at all:

@bors: retry

:hourglass: Testing commit bce9075570889b0fc20aad628594db9555fa194a with merge 461868e35e2a7184452c457681df4927ddbcf362...

@mati865 can you try again?

Thanks, I can access it now.

:boom: Test timed out

So Azure succeeded and bors properly registered it: 🎉 .
Cirrus-CI is having issues, so I've canceled the builds for now. They will let us know when these issues are resolved.

The Cirrus-CI issue should be resolved now.
@bors: retry

:hourglass: Testing commit bce9075570889b0fc20aad628594db9555fa194a with merge a9ab0f983a38c99aa24c9011c0fce2255d89132b...

:sunny: Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Approved by: gnzlbg
Pushing a9ab0f983a38c99aa24c9011c0fce2255d89132b to master...


Links for docs

The docs on the following URL have links on the right hand side navigation that redirects me to a different webserver to a generic page (code.facebook.com).
http://facebook.github.io/immutable-js/docs/#/Stack/shift
Is there an updated URL I should be accessing for these docs?

Encountering the same issue, every link but the very first page of documentation leads to code.facebook.com.

@willdevelops @tmikoss I ran into this issue today. I noticed a cdn.jsdelivr.net was being blocked by privacy badger. I switched it to only block cookies and now the docs site is working properly. So if you use any sort of blocking tool, I would look at that first. Hope this helps.

@vinnymac Hey, lookie there, it worked.. Thanks!  👍  Problem resolved.


Tech debt: Validated configuration

Yes, I'd say this is safe. Tech debt here is we eventually want a validated configuration (eg. convict, among many others) but this is good for now.
Originally posted by @wenzowski in https://github.com/button-inc/digital_marketplace/pull/111#discussion_r774116727
See also: https://github.com/button-inc/digital_marketplace/pull/111#discussion_r774111439

See also https://github.com/button-inc/digital_marketplace/issues/113


Pilot /debug/authenticationz incorrect TLSConflictStatus

To reproduce, install Istio 1.5.0 beta 5 w/o mTLS, then set a MeshPolicy for STRICT mTLS:
istioctl manifest apply --set values.global.mtls.enabled=false \
--set values.global.mtls.auto=false
   kubectl apply -f - <<EOF
apiVersion: "authentication.istio.io/v1alpha1"
kind: "MeshPolicy"
metadata:
  name: "default"
spec:
  peers:
  - mtls:
      mode: "STRICT"
EOF

Debugging with istioctl x describe pod, the result from /debug/authenticationz was v2.AuthenticationDebug{Host:"productpage.default.svc.cluster.local", Port:9080, AuthenticationPolicyName:"/default", DestinationRuleName:"None", ServerProtocol:"STRICT", ClientProtocol:"None", TLSConflictStatus:"OK"}
The TLSConflictStatus should NOT be OK.  I had deployed bookinfo, and bookinfo-gateway, and via the IngressGateway got 503 talking to /productpage.
cc @GregHanson
Discovered while working on https://github.com/istio/istio/issues/21711

@diemtvu Will you be able to work on this?

We will need to remove this tool. It's no longer fit with workload-based
APIs. I'll send a PR to remove this.
On Mon, Mar 23, 2020 at 2:08 PM Ed Snible<EMAIL_ADDRESS>wrote:

@diemtvu https://github.com/diemtvu Will you be able to work on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/21821#issuecomment-602856784, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7X24MUORNWVH5OVSRMXELRI7FTXANCNFSM4LBWQQKA
.

--
Diem Vu | Software Engineer |<EMAIL_ADDRESS>| +1<PHONE_NUMBER>

@diemtvu is it possible to fix this API?  We are counting on it for istioctl x describe.  If we cannot trust the output of Istiod /debug/authenticationz then describe will need its own implementation of conflict detection.
cc @GregHanson

Closing in favor of https://github.com/istio/istio/issues/23776


cannot get .caseWithAction to work with my Handler

Hi,
I have the following:
import actionCreatorFactory from 'typescript-fsa'
import { reducerWithInitialState } from 'typescript-fsa-reducers'
import { FormState, IFormState } from '../../shared/shared.store/store.util'
export interface IEmail {
address: string
type: string
meta: IMeta
}
export class Email {
type: string = ''
address: string = ''
meta: IMeta
}
const userAcf = actionCreatorFactory( '[Admin ResetEmail]' )
export const addResetEmailDataAction = userAcf( 'ADD_DATA' )
export function addResetEmailDataHandler( state: IEmail, email: IEmail ): IEmail {
return Object.assign( {}, state, email )
}
export const resetResetEmailDataAction = userAcf( 'RESET_DATA' )
function resetResetEmailDataHandler( state: IEmail ): IEmail {
return Object.assign( {}, state, new Email() )
}
export const resetEmailDataReducer = reducerWithInitialState( new Email() )
.caseWithAction( addResetEmailDataAction, addResetEmailDataHandler,  ) // ERROR with addResetEmailDataHandler
.case( resetResetEmailDataAction, resetResetEmailDataHandler )
}
My  .case works but the .caseWithAction does not.  I need your help to use my addResetEmailDataHandler handler to work in the .caseWithAction above.
Thanks

Hi st-claire-clarke!
The first reason you're getting an error is because your addResetEmailDataAction has no payload. That is, in the line
export const addResetEmailDataAction = userAcf( 'ADD_DATA' )

since you don't specify a payload type, the actions created by this factory are assumed to have no payload. Based on the fact that your addResetEmailDataHandler function takes an IEmail as its second argument, I assume you want to be able to create actions with IEmail as their payload, but note that as written the following is an error:
let email: IEmail
const action = addResetEmailDataAction(email)
// Argument of type IEmail is not assignable to parameter of type 'undefined'.

Because of this, I would guess that what you want instead is
export const addResetEmailDataAction = userAcf<IEmail>( 'ADD_DATA' )

which allows you to call addResetEmailDataAction(email) which is probably what you want.
Secondly, supposing you make that change you'll still have a type error on the .caseWithAction() line because .caseWithAction() expects a handler function which takes (state, action) => state, by contrast to .case() whose handler function is (state, payload) => state. In your example code, the second argument of addResetEmailDataHandler has type IEmail which is the payload type, not the action type, so you actually do want to be using .case here.
Based on the code you posted, I don't see a reason why you would need to be using .caseWithAction() rather than .case()- the former is mainly useful when you need to either read meta information from the action or pass the entire action on unchanged to a different handler, but the example code does neither.
Please let me know if I am misunderstanding or if you have further questions. Thanks!

Hi Phillipson,
Thanks for your quick reply.
The meta: IMeta type in the IEmai has the definition as shown below:
export interface IMeta {
lastEditedBy: string
timeStamp: Date
}
export class Meta {
lastEditedBy: string = ''
timeStamp: Date
}
I would like to access these data when the store is updated using .caseWithAction(). So, I would like to update a log with the meta data. With this information, what would the complete code look like, including the handler?
Cheers

Any reason this doesn't work?
const userAcf = actionCreatorFactory( '[Admin ResetEmail]' )

export const addResetEmailDataAction = userAcf<IEmail>( 'ADD_DATA' )

export function addResetEmailDataHandler( state: IEmail, email: IEmail ): IEmail {
    // In here, you can access the metadata through email.meta.
    return Object.assign( {}, state, email )
}

export const resetResetEmailDataAction = userAcf( 'RESET_DATA' )

function resetResetEmailDataHandler( state: IEmail ): IEmail {
    return Object.assign( {}, state, new Email() )
}

export const resetEmailDataReducer = reducerWithInitialState( new Email() )
    .case( addResetEmailDataAction, addResetEmailDataHandler)
    .case( resetResetEmailDataAction, resetResetEmailDataHandler )
}

// Example of creating an addResetEmailAction
const meta = new Meta();
meta.lastEditedBy = "someGuy";
meta.timestamp = new Date();

const email = new Email();
email.address =<EMAIL_ADDRESS>email.meta = meta;

const myAction = addResetEmailAction(email);

Note that the metadata as you've written it is different from the meta field on the action, because IEmail is not the type of the action, it's the type of the action's payload. If it were the meta field of the action, then the action would appear:
{
    type: "ADD_DATA",
    payload: {
        type: "",
        address: ""
    },
    meta: { lastEditedBy: "", timestamp: <some-date> }
}

but your actions don't look like that, they look like this:
{
    type: "ADD_DATA",
    payload: {
        type: "",
        address: "",
        meta: { lastEditedBy: "", timestamp: <some-date> }
    }
}

Since meta is just another field on your payload, you don't need to use .caseWithAction to read it and can just pull it out of .case.

Closing this for inactivity. Feel free to re-open if you have more questions.


Simplify input in preprocessor

State-of-the-art ASP systems employ sophisticated preprocessing to simplify input programs, thereby improving solving performance. Probably Alpha could benefit from such techniques as well.
Example reference:
Gebser, M., Kaufmann, B., Neumann, A., & Schaub, T. (2008, June). Advanced Preprocessing for Answer Set Solving. In ECAI (Vol. 178, pp. 15-19).

Also note that #70 is concerned with one concrete preprocessor.

Example implementation: asp_preprocessor.cpp in clasp
Preprocessor::preprocessSimple creates solver variables for rule heads and non-trivial supported bodies and atoms, while reusing variables where possible and creating equivalences between atoms and trivial (unit) bodies.


Danganronpa Another Episode: Ultra Despair Girls [PCSE00692]

Game summary

Game name: Danganronpa Another Episode: Ultra Despair Girls
Game serial: PCSE00692

Vita3K summary

Version: 0.01
Build number: 404
Commit hash: https://github.com/vita3k/vita3k/commit/d4cd595

Test environment summary

Tested by: waterflame
OS: Windows 7
CPU: Intel i7-2600
GPU: AMD Radeon HD 6700 Series
RAM: 8 GB

Issues
Crashes when trying to load module testprx.elf. No noticeable regressions against 328-https://github.com/vita3k/vita3k/commit/6525c1c.
Log
vita3k-404.log
Recommended labels

Crash
Module bug

Vita3K summary

Version: v0.1
Build number: 571-70599d3
Commit hash:  https://github.com/Vita3K/Vita3K/commit/70599d3

DanganronpaDespairGirls.log

Vita3K summary

Version: v0.1
Build number: 1315
Commit hash: https://github.com/vita3k/vita3k/commit/5b070aac

DanganronpaDespairGirls.log

Vita3K summary
Version: v0.1.3
Build number: 2270
Commit hash: https://github.com/vita3k/vita3k/commit/3e791b1
Test environment summary

Tested by: NSMHD
OS: Windows 8.1 Pro
CPU: AMD A6 7310
GPU: AMD Radeon R4 Graphics
RAM: 8 GB

Issues
Game crashes after loading screen after "Start game" dialog, background music doesn't restart after returning from load save screen and sound stuttering
Screenshots

Log
PCSE00692 - [Danganronpa Another Episode  Ultra Despair Girls] [Dynarmic].log
PCSE00692 - [Danganronpa Another Episode  Ultra Despair Girls] [Unicorn].log

as of https://github.com/Vita3K/Vita3K/commit/5bd8b3b6aa3f86d6d38cbd0b7a66a852b1bdf17f, the game can go ingame but in Vulkan, the graphics is looking bad, while OpenGL is kinda better but it still has oversaured graphics and glitching on the textures.
Screenshots

Logs
vita3k (OpenGL).zip
vita3k (Vulkan).log
Recommended Labels

Ingame +
graphics bug
vulkan issue


Allow users with AL privileges to execute RESET statements

Summary of the changes / Why this improves CrateDB
They can use SET GLOBAL so it makes sense to be able to RESET settings
as well.
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

@MarkusH fyi

Thank you!

retest this please
unrelated failure:
java.lang.IllegalArgumentException: byte value out of range: 128
	at __randomizedtesting.SeedInfo.seed([45E644A42819611F:773F22490BFF3270]:0)
	at io.crate.types.ByteType.value(ByteType.java:68)
	at io.crate.types.ByteType.value(ByteType.java:30)
	at io.crate.types.DataType.implicitCast(DataType.java:94)
	at io.crate.types.TypeConversionTest.numberConversionTest(TypeConversionTest.java:112)


No matching manifest

I'm getting no matching manifest for linux/arm64/v8 in the manifest list entries when I'm writing docker compose up -d

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

ID: 65f634bf-2280-a093-5897-724310dafa2d
Version Independent ID: f11c0af7-bfaf-6d64-2c75-9015b0eb1f30
Content: Kubernetes on Azure tutorial  - Prepare an application - Azure Kubernetes Service
Content Source: articles/aks/tutorial-kubernetes-prepare-app.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@joeyschen
Thanks for your feedback! We will investigate and update as appropriate.

@joeyschen
Thank you for your feedback!
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)
Thank you for your time and patience throughout this issue.


presenceUpdate issues

Experiencing some inconsistencies with the presenceUpdate event.
All regarding a regular user that is not the bot.

Starting from not playing anything and launching a game seems to work as intended.
Exit game, new user object still has user.game data as if nothing changed.
Launch same game again, presenceUpdate does not fire at all.
Exit game, again seems like oldUser == newUser.

The native Discord client properly reflects these events, and the old presence event from v8 worked fine.

Users don't receive their own presence updates. I think this is what you're experiencing here.

I'm trying with a user that is unrelated to the bot, and the same seems to hold true for other users on the server. I have never seen an update where oldUser has a game and newUser is null since I moved to v9.

#666 describes the issue you're having.


Update backitup to 3.0.19

Please update my adapter ioBroker.backitup to version 3.0.19.
This pull request was created by https://www.iobroker.dev c0726ff.

@mcm1957 As already discussed in advance, this is a hotfix

Ok - was too ago long to remember :-)


Component import in for loop not working

Hi,
I want to use a component in a for loop, but I get several errors.
The errors I get are that i and v are not used ( in MemberList for loop) and MemberRows.Render undefined (see screenshot)

Here's the code if it helps:
package view

import "techscene.de/gymsoft/model"
import "fmt"

templ MemberList(m []model.Member, offset int, limit int) {

Mitglieder List

	{ fmt.Sprint(len(m)) }
	for i, v := range m {
		@memberRows(v, offset + limit, limit, i == len(m)-1 )
	}
}

templ memberRows(m model.Member, offset int, limit int, triggerLoadMore bool) {

{ m.FirstName } { m.LastName }

}

This could very well be a skill issue on my part, but I couldn't find anything in the docs regarding components in loops. Help would be appreciated!
Edit: not sure if this is related, but templ format has a weird formatting for the component import with the function call parens on the next line

Ok, sorry for bothering you. I tried it again and now it seems to work. I don't know (yet) what I did differently, but the important thing is, it works


Radio Button within a table with Cdk Table or Material Design

Bug, feature request, or proposal:
I'm actually trying to insert a radio button group within a table component of  Cdk Table /material design
What is the expected behavior?
Group radio button
What is the current behavior?
not found
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
requirement
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
NA
Is there anything else we should know?

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.
(this is a canned response)


implement testing for kubernetes metadata enrichment

test kubernetes metadata fetching in minikube environment
it has been tested via a mock http server providing the sample json data, but not in actual production-like environment

Tested on minikube to be working with fixes provided in PR #20


[docs] pnpm order #42699

[x] I have followed (at least) the PR section of the contributing guide.

https://github.com/mui/material-ui/pull/42712


Test readthedocs/actions/preview since new message template

To examine the new message.

📚 Documentation preview (see all builds) 📚

Looks good.


[OSX] DirAccessUnix.current_is_dir return false after changing dir

Operating system or device - Godot version:
OSX, Godot master git version
Issue description (what happened, and what was expected):
Steps to reproduce:
String dir = "/Applications/";
DirAccess *da = DirAccess::create(DirAccess::ACCESS_FILESYSTEM);
  if (da->change_dir(dir)==OK){
    print_line(da->current_is_dir() ? "DIR" : "NOT DIR"); // NOT_DIR
}

Link to minimal example project (optional but very welcome):

That's not what current_is_dir() does. From the documentation:

Return whether the current item processed with the last get_next call is a directory (. and .. are considered directories).

So this function is meant to be using when you listing the directory contents with list_dir_begin() and get_next().

Ups, thanks! Close issue


Coerce to union missing type

When writing a tutorial, I encountered several time an issue such as this one:
MethodError: no method matching coerce(::Array{Union{Missing, Int64},1}, ::Type{Union{Missing, Continuous}}; verbosity=1)
Closest candidates are:
  coerce(::AbstractArray{#s26,1} where #s26<:Union{Missing, Real}, !Matched::Type{Continuous}; verbosity) at /Users/tlienart/.julia/dev/ScientificTypes/src/conventions/mlj/mlj.jl:54
  coerce(::AbstractArray{T,1} where T, !Matched::Type{Continuous}; verbosity) at /Users/tlienart/.julia/dev/ScientificTypes/src/conventions/mlj/mlj.jl:65
  coerce(::AbstractArray{#s26,1} where #s26<:Union{Missing, Integer}, !Matched::Type{Count}; verbosity) at /Users/tlienart/.julia/dev/ScientificTypes/src/conventions/mlj/mlj.jl:82
  ...
_coerce_col(::DataFrame, ::Symbol, ::Dict{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}; args::Base.Iterators.Pairs{Symbol,Int64,Tuple{Symbol},NamedTuple{(:verbosity,),Tuple{Int64}}}) at tables.jl:41
_coerce_col at tables.jl:38 [inlined]
#49 at none:0 [inlined]
iterate at generator.jl:47 [inlined]
collect_to!(::Array{AbstractArray{T,1} where T,1}, ::Base.Generator{NTuple{24,Symbol},ScientificTypes.var"#49#50"{Int64,DataFrame,Tuple{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}}}}, ::Int64, ::Int64) at array.jl:681
collect_to!(::Array{CategoricalArray{T,1,UInt8,Int64,CategoricalValue{Int64,UInt8},U} where U where T,1}, ::Base.Generator{NTuple{24,Symbol},ScientificTypes.var"#49#50"{Int64,DataFrame,Tuple{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}}}}, ::Int64, ::Int64) at array.jl:689
collect_to!(::Array{CategoricalArray{Union{Missing, Int64},1,UInt8,Int64,CategoricalValue{Int64,UInt8},Missing},1}, ::Base.Generator{NTuple{24,Symbol},ScientificTypes.var"#49#50"{Int64,DataFrame,Tuple{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}}}}, ::Int64, ::Int64) at array.jl:689
collect_to_with_first!(::Array{CategoricalArray{Union{Missing, Int64},1,UInt8,Int64,CategoricalValue{Int64,UInt8},Missing},1}, ::CategoricalArray{Union{Missing, Int64},1,UInt8,Int64,CategoricalValue{Int64,UInt8},Missing}, ::Base.Generator{NTuple{24,Symbol},ScientificTypes.var"#49#50"{Int64,DataFrame,Tuple{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}}}}, ::Int64) at array.jl:660
collect(::Base.Generator{NTuple{24,Symbol},ScientificTypes.var"#49#50"{Int64,DataFrame,Tuple{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}}}}) at array.jl:641
_totuple at tuple.jl:269 [inlined]
Tuple at tuple.jl:241 [inlined]
NamedTuple{(:surgery, :age, :rectal_temperature, :pulse, :respiratory_rate, :temperature_extremities, :peripheral_pulse, :mucous_membranes, :capillary_refill_time, :pain, :peristalsis, :abdominal_distension, :nasogastric_tube, :nasogastric_reflux, :nasogastric_reflux_ph, :feces, :abdomen, :packed_cell_volume, :total_protein, :abdomcentesis_appearance, :abdomcentesis_total_protein, :outcome, :surgical_lesion, :cp_data),T} where T<:Tuple(::Base.Generator{NTuple{24,Symbol},ScientificTypes.var"#49#50"{Int64,DataFrame,Tuple{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}}}}) at namedtuple.jl:86
coerce(::DataFrame, ::Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}, ::Vararg{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},N} where N; verbosity::Int64) at tables.jl:74
coerce(::DataFrame, ::Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}, ::Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}, ::Vararg{Pair{Symbol,Type{#s16} where #s16<:Union{Missing, Found}},N} where N) at tables.jl:71
#coerce#51 at tables.jl:79 [inlined]
coerce(::DataFrame, ::Dict{Symbol,Type{#s16} where #s16<:Union{Missing, Found}}) at tables.jl:79
top-level scope at EX-horse.jl:81

would be good to  investigate, it seems that effectively what  happens is a coerce where the target type is already a union missing (which should be fine  afaik); worth investigating. Maybe by allowing the coerce to take a union target type (otherwise the method is not defined);
So maybe  go from
function coerce(y::AbstractVector{<:Union{Missing,AbstractFloat}}, T::Type{Continuous};

to
function coerce(y::AbstractVector{<:Union{Missing,AbstractFloat}}, T::Type{<:Union{Missing,Continuous}};

This is addressed in #32


Keinerlei Funktion

Keinerlei Funktion
Egal was man für eine ip eingibt, es funktioniert nicht.
MQTT_CLIENT: There are no transports valid, stop mqtt client, config scheme = http

hi elzer, would you please paste your code here?

ou.. i down't know. That was too long ago. I solved it differently. Just test it yourself. Then you see for yourself that it doesn't work.


Look at using Word2vec

I speculate that it might be helpful to use Word2vec as a text pre-processor. Caveat: I barely know what I'm talking about.

I've been exploring this a bit today. It's a fine little program, quite speedy, but feeding it unstructured text yields unstructured results. Another problem is that, at present, I'm only giving it a single source of data. Once we're feeding it multiple dictionaries, it should be able to extract better results.
Again, the text needs to be pre-processed (per #10), and we've really got to gin up some method of isolating each term and its definition, instead of blurring them together with the prior and following term and definition.


"Invalid Entry" in InputBox

Hello Krish
Look like somehow InputBox stopped working in this last update:

Let me know for your fix update..
And Thanks again for this great project.

Hello
This was Fixed with the Update on Jan-28-2022..!!
All good


Recycle cells during render phase on VirtualizedList component

This prevents view creation which is expensive, thus improves performance.
Test Plan:

Begin with index 0
Increment index by 2 every 3s, then call method scrollToIndex(index) to scroll list, and measure the time taken to update the list component
When index reaches 60, end the interval
Calculate the mean time

before  507ms
after 484ms
Changelog:
[General] [Added] - Recycle cells during render phase on VirtualizedList component

Hello @yun77op. Do you mind providing the answers to the questions by @sahrens? I'm afraid without them I won't be able to merge this PR :(

Closing because of inactivity.


Generate Markdown versions of "Consolidated" worksheet

Create a very simple markdown file for each record under:

content/markdown/<worksheet_name>/<record_name>.md - one per record

Create a title for the record, markdown formatted.
Push them to github and show @charliepascoe

Finished this, md files are very simple and are only created for the consolidated texts currently. Will need to speak to @charliepascoe to discuss how they should look / what they should contain.

Finished this, md files are very simple and are only created for the consolidated texts currently. Will need to speak to @charliepascoe to discuss how they should look / what they should contain.


force usage of F16 textures

"tfjs-core: Make it possible to force usage of F16 textures. #1902"
https://github.com/tensorflow/tfjs/pull/1902
appears in tf-core 1.2.9 release (only f16 textures yet, not math!).

can you (we :) add and test f16 textures?
they added in "To Do": "Add flag (defaults to false) that would turn on mediump precision." Can we confirm, that mediump precision is useful in real aplications? (than they will add the flag more quickly :)
E.g. image classification/detection + colab + NVIDIA GPU?

So you are asking me to upgrade the tfjs-core version to latest?

not sure about detailes (I'm from webgl-dev-list :)

if you just add
tf.webgl.forceHalfFloat()
you will half GPU memory footprint and get ~10% acceleration.
with mediump precision you can get x2 acceleration but for some reason e.g. SSD model works wrong with fp16 math (for TFjs team). Can we optimize any useful model for fp16 math (NVIDIA can :)?

These flags you are talking about can be enabled via tf. face-api.js doesn't deal with such kind of things, it just utilizes tfjs as a backend. So you can simply enable those flags via tf and the models of face-api.js will make use of them.

with mediump precision you can get x2 acceleration but for some reason e.g. SSD model works wrong with fp16 math (for TFjs team). Can we optimize any useful model for fp16 math (NVIDIA can :)? More detailes?

I am not familar with such kind of optimizations. Again such optimizations are probably something that have to be done in the tfjs backend and not face-api.js, but correct me if I am wrong.


feat: add privacy policy page

@ankitwww, do you know what could be the reason? I can't see the logs

Yes, here are the logs. You can see them locally by building the project. 'npm run build'

@pedroagcf let me know if you need any further details for error

looks better now! let me know if It is ok for you @ankitwww

Pretty good! Thank @pedroagcf

could you please add the Hacktoberfest label for this PR? @ankitwww

I guess we don't need to tag PR as the whole repo is tagged - so basically every PR is automatically accepted in hacktoberfest.
Still, no worries, I will add accepted tag now


LastPass doesn't prompt to save Secret Key

As mentioned in article, LastPass might require elements to be hidden using DOM tricks, opacity: 0, rather than display none.
https://hiddedevries.nl/en/blog/2018-01-13-making-password-managers-play-ball-with-your-login-form

It's worth waiting until we decide what to do with a final screen before attempting to fix this one, as that may in itself be the fix.


Changes to DriverInterface

#446 Fix some type inconsistencies and replace calls to IOKitCallStructMethod with IOKitCallScalarMethod.

Thanks!


ensure new branches are created with unique names if the action is re-run

It seems the branch naming system is stopping at suffix _0 in
retype-663826198_1
retype-663826198_1_0
Instead of going ahead with
retype-663826198_1_1
retype-663826198_1_2
...and so on. Ensure this is working; may require doing artificial changes to the documentation for it to actually have something to push along the branch.

It looks right. The following action was run several times, and incrementing number of branch names were created:
Action: https://github.com/fabriciomurta/ragha-tests/actions/runs/687028252
Branch names created:
  remotes/origin/gh-subpages
  remotes/origin/gh-subpages-687028252_33
  remotes/origin/gh-subpages-687028252_33_0
  remotes/origin/gh-subpages-687028252_33_1
  remotes/origin/gh-subpages-687028252_33_2

This issue has been transferred to retypeapp/action-github#5.


Recursively search for test files when test target is a directory

As a StarkNet smart contract developer, I want the test command to recursively look for my test files in the provided target directory.
This will help me better organize my tests in sub-directories.

I believe Protostar already does it.
https://user-images.githubusercontent.com/9629577/166728260-89bc9640-1322-4a6c-a134-3113c0920f3d.mov

@kasperski95 you are right, I opened this issue in the first place because I thought protostar could not recursively search for tests, but it was actually due to a misuse.
All our test files were in fact named test.cairo which is not detected by protostar.
Thank you @AnthonyBuisset for pointing this out!


Example site

Similar to exampleSite in Hugo themes with bare-bones templates and “Hello, World” post. A little kick-start would be very helpful in understanding how everything works.

Hey @commagray
As I said in the README, this project is mostly for personal usage than for others. A lot of the code is tailored specifically for what I want it to do and would not work for everyone. Besides, I only started using the new version on Monday, so I'm not yet ready to "officially" announce it. Once I post an article on my website about it, I may add an example site here with the minimum required templates so people can try it out.

I'm closing this as I have no intent of supporting other people's use cases for now. As stated in the README, if there is anything that I may be able to extract in a single module, please let me know.


My RealScoreBoard say no API

Hello My RealScoreBoard say no API,
plz can you help me
here is a code:
[Server thread/WARN]: Plugin RealScoreboard v8-2-2020 Build 1 does not specify an api-version.

@MxMasterX Hi there!
Can you give me some details about the server?

@JoseGamerPT
Oh sorry, ^^ i use:   PaperMc 1.14.4 at V-Server.

Can you help me about api

Does this happen on Spigot?


Support for TypeDoc 0.27

Hi @krisztianb! TypeDoc 0.27 is now available, and it would be great if typedoc-plugin-replace-text could support it.
Thanks in advance and thank you for all your work on this plugin!

Hi Jan. I'm glad that the plugin is helpful for you. I should have time during the following week to update the plugin to support TypeDoc 0.27.

I published a new version 4.1.0 on NPM which works with TypeDoc 0.27.x 🎉
My tests are passing. Please report back if something is not working as expected. Thank you.


Understanding The Modernisation Platforms Carbon Footprint

User Story
As a product manager I want to know what the carbon footprint of the modernisation platform is.
We have been provided with a few open source providers of tools that could measure the footprint.
https://www.apptio.com/products/cloudability/ (IBM)
https://greenpixie.com/  Amazon recommend them over Cloudablity
https://sopht.com/en/
This ticket is to look at https://greenpixie.com/   in the first instance and understand:
What it would report on
what it would not work for
what we would need to do to make use of it
It may be useful to compare it to the other suggestions if time allows
This is to cover AWS elements of modernisation platform - including applications
Value / Purpose
By knowing what the carbon footprint is we can then look at what we can do to reduce it in a measurable way. The plan is we would then use this as a metric for the team.
Reducing the carbon footprint is likely to save money, however we should clearly be looking to reduce carbon footprint anyway
Useful Contacts
<EMAIL_ADDRESS>Additional Information
No response
Definition of Done

[ ] Document produced outlining points above
[ ] Document presented to the team
[ ] New tickets created if we want to proceed or
[ ] netw tickets created to look at other options if necessary

Done some background reading into Green Ops and the above solutions.
I have emailed green pixie and sopht for more details and potential demo that i can present to the team for both solutions.
The IBM solution is for Fin Ops and not Green Ops

Document with findings - https://docs.google.com/document/d/17wdbMib1Oc0Pg-UfUtL1I_dyKSjIV_0oHmaVZL-5k5Y/edit?tab=t.0

Have a meetings with reps from green pixie and sopht today.
Also had a chat with Julien Nioche and he sent me the links below to have a read
https://www.linkedin.com/pulse/environmental-impact-cloud-common-crawl-case-study-julien-nioche-at8xf
https://podcasts.castplus.fm/e/m849x5pn-7-cameron-casher-and-benjamin-davy-cloud-carbon-footprint-sustainability-aws-gcp-azure-vs-aws-azure-and-gcp-sustainability-dashboards
https://www.cloudcarbonfootprint.org/
https://learn.greensoftware.foundation/
https://podcasts.castplus.fm/e/m849x5pn-7-cameron-casher-and-benjamin-davy-cloud-carbon-footprint-sustainability-aws-gcp-azure-vs-aws-azure-and-gcp-sustainability-dashboards
https://www.cloudcarbonfootprint.org/
https://learn.greensoftware.foundation/

Turns out i was wrong. IBM does does provide Greenops services. I have a meeting with them today

About Sopht 2024.pdf

Greenpixie vs AWS

I have raised ticket #8463 as an output this ticket. Simon has confirmed that ticket can now be closed
https://mojdt.slack.com/archives/C013RM6MFFW/p1731337566881729?thread_ts=1731323183.737909&cid=C013RM6MFFW


Documentation: Using PnP.PowerShell in Remote Containers

Is your feature request related to a problem? Please describe.
Not a big problem, but when I set up a new project that is using PnP.PowerShell, if I want to use Visual Studio remote containers, I need to come up with a new text of devcontainer.json. There seems no good description of the procedure and devcontainer.json to use.
Describe the solution you'd like
It would be probably great to have an article, similar to https://pnp.github.io/powershell/articles/azurecloudshell.html, called something like "Using PnP PowerShell in Visual Studio Remote Containers"
Describe alternatives you've considered
Someone could post it in their blogs, gists etc.
Additional context

@shurick81 Please use the Bug Report template when reporting bugs. You can do this by clicking New Issue > Get Started (next to Bug Report).

@CallumCrowley - This is not a bug, and @KoenZomers already added the documentation label.

@veronicageek apologies, I'm not sure how I missed that! 😆

hi @shurick81 , any update on this one as well ? Would be great to have this in our docs :)

For me it would be nice to contribute consecutively, after #2008

hi @shurick81, you are our docker expert,so wanted your opinion :)
Would it make sense to have devcontainer.json kinda thing for PnP PowerShell ? If possible ?
I looked at the CLI M365 code, and they are using it as well. Check this -->
https://github.com/pnp/cli-microsoft365/blob/main/.devcontainer/devcontainer.json
Is it possible to have something like this for PnP PowerShell ?

Yes, I think we should also have something like this. For example, here's the docu that  I made for the SharePoint Framework: https://github.com/shurick81/docker-spfx/blob/feature/Remote-Containers-Docu/DevelopmentContainers.md

Or maybe you mean using dev containers for the those who want to contribute to PnP.PowerShell? Yes, that is another improvement that I would suggest. If so, let's create a separate issue for that?

Both? If possible? Can you create them as enhancements? Sorry, am not even sure what to write for this. Would be a great improvement I think

@gautamdsheth when I wanted to register this new enhancement suggestion, I stumbled into the fact that I know very little about how does it usually works for contributors to implement changes in the PnP.PowerShell module: what environment they need for building and debugging. Would you recommend any person to discuss this with?

Hi @gautamdsheth, @veronicageek, could someone help me show some practices of contributing to new versions of this module? Then I could register a new issue for documenting how to use development containers.

hi @shurick81 , can you elaborate more , sorry not getting this. The repo is totally open-source and we welcome all contributions. We dont have any such defined practices because every person who wants to contribute is different and uses different environment.
I use Visual Studio, where after cloning the repo, I just need to do F5.
Erwin/Koen prefer VS code for development and Veronique uses Mac.
Other contributors may use different stuff as well.
The code works wherever PowerShell works :)
For this issue/feature, maybe just send a PR whenever you are ready. You can showcase whatever you have used for containers, it is totally fine as long as it works :)
Let me know if you need more info.