Set user.room for reaction_added events

This addresses 18F/hubot-slack-github-issues#33.
cc: @afeld

Were these modified tests failing without the associated code change? Seems like they may not actually be testing the behavior described in the linked issue.

The updated should produce a SlackTextMessage and should parse the text in the SlackTextMessage tests pass without any changes to SlackBot.message because it was already setting user.room. The should produce a SlackTextMessage for reaction_added events test failed before updating SlackBot.reactionAdded because user.room was previously undefined.
My hypothesis is that setting user.room in SlackBot.reactionAdded addresses 18F/hubot-slack-github-issues#33 because it's the only context I've discovered that could account for a Hubot reply going to the channel that a user last commented in, rather than to the channel in which the reaction_added event took place. In fact...
I just deployed a Charlie with this change, and 18F/handbook#443 confirms that it works.


Compatibility with new versions of ansible

@fizista this has been fixed in #1 but the new release is missing from the @nickjj

This is already implemented as pointed out above, and now it's pushed to v0.2.0 which on the Galaxy. Sorry for the confusion!


getting the version ID after cog push

In situations where cog push is being run in an automated fashion, it's useful to be able to get the id of the published version.
The version id is present in the output (see latest: digest: sha256:7507bda...) below), but maybe there's a nicer way to return it, or make it more obvious what it is.
$ cog push
Building Docker image from environment in cog.yaml as r8.im/zeke/haiku-multi-output...
d38adf39e1dd: Layer already exists
4ed121b04368: Layer already exists
d9d07d703dd5: Layer already exists
...
latest: digest: sha256:7507bdae030fd2a0948b7e63337bd08147a3e137d0ea420612d01f9d838883c6 size: 3269
Image 'r8.im/zeke/haiku-multi-output' pushed

Run your model on Replicate:
    https://replicate.com/zeke/haiku-multi-output

cc @tstramer

When I'm running cog push, I'm not getting the latest in the output:
Building Docker image from environment in cog.yaml as r8.im/rdev/sdtest...
[+] Building 258.8s (18/18) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.78kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 46B                                                                                0.0s
 => resolve image config for docker.io/docker/dockerfile:1.2                                                                                0.1s
 => CACHED docker-image://docker.io/docker/dockerfile:1.2@sha256:e2a8561e419ab1ba6b2fe6cbdf49fd92b95912df1cf7d313c3e2230a333fdbcc           0.0s
 => [internal] load metadata for docker.io/nvidia/cuda:11.6.2-cudnn8-devel-ubuntu20.04                                                      0.2s
 => [internal] load build context                                                                                8.9s
 => => transferring context: 2.32GB                                                                                8.9s
 => [stage-0  1/10] FROM docker.io/nvidia/cuda:11.6.2-cudnn8-devel-ubuntu20.04@sha256:91779650798919050553d6673d04a96bfd7cdf3fa931b6ff321  49.1s
 => => resolve docker.io/nvidia/cuda:11.6.2-cudnn8-devel-ubuntu20.04@sha256:91779650798919050553d6673d04a96bfd7cdf3fa931b6ff32120b7f2eaaa4  0.0s
 => => sha256:bcf9a52c75ac54dc66b5b683fc8dc2bb8a1cc8eef9340433a3efbfe8d6fa3006 187B / 187B                                                  0.0s
 => => sha256:9ce4eceb346b57d4be964a3a57551204b7435b2eae1d8c3154ef672661d710e4 1.12GB / 1.12GB                                             13.7s
 => => sha256:3a665e4036160d33fc32ce19c889271febc4fe4cdb82637cdd8c9ef10ca9541c 23.60MB / 23.60MB                                            0.5s
 => => sha256:624cfe37262e44c78cd50ec7bc8a98e417be3b81bb215eed099bf7da343b1a63 16.20kB / 16.20kB                                            0.0s
 => => sha256:cb69caf25724810d45aec6392c11de6f67c2efde38da9d46402857dedb68aefb 7.93MB / 7.93MB                                              0.1s
 => => sha256:8941157b58ada869bd12299ba8a56cdc5317923a5ce7df8158c5a3b44ff2fb67 6.43kB / 6.43kB                                              0.1s
 => => sha256:91779650798919050553d6673d04a96bfd7cdf3fa931b6ff32120b7f2eaaa4ad 743B / 743B                                                  0.0s
 => => sha256:828017b99ce1b75fe6307d54f97bdd9f224730a1a3f31d935ce85209cb08c93c 2.43kB / 2.43kB                                              0.0s
 => => extracting sha256:cb69caf25724810d45aec6392c11de6f67c2efde38da9d46402857dedb68aefb                                                   0.2s
 => => sha256:b1c91cec61e689319fc5354207f545dc4d065599ae530f8aae92bded993c72e3 62.66kB / 62.66kB                                            0.2s
 => => sha256:01035e520ac2fd77f181410687ecc81e9df074e738f8a4dfea4589572c663bd8 1.44GB / 1.44GB                                             17.4s
 => => extracting sha256:3a665e4036160d33fc32ce19c889271febc4fe4cdb82637cdd8c9ef10ca9541c                                                   0.3s
 => => sha256:246ba1b32462cea77206e04797df75780a0bdebd112b020b9bf11a7c6694eaa1 85.60kB / 85.60kB                                            0.5s
 => => sha256:dc3b3af10cf618c7b3038ca70bbcc422701fa0bf6cb9afae277406df7140445a 1.48GB / 1.48GB                                             19.1s
 => => extracting sha256:bcf9a52c75ac54dc66b5b683fc8dc2bb8a1cc8eef9340433a3efbfe8d6fa3006                                                   0.0s
 => => extracting sha256:8941157b58ada869bd12299ba8a56cdc5317923a5ce7df8158c5a3b44ff2fb67                                                   0.0s
 => => extracting sha256:9ce4eceb346b57d4be964a3a57551204b7435b2eae1d8c3154ef672661d710e4                                                   8.9s
 => => extracting sha256:b1c91cec61e689319fc5354207f545dc4d065599ae530f8aae92bded993c72e3                                                   0.0s
 => => extracting sha256:01035e520ac2fd77f181410687ecc81e9df074e738f8a4dfea4589572c663bd8                                                  13.1s
 => => extracting sha256:246ba1b32462cea77206e04797df75780a0bdebd112b020b9bf11a7c6694eaa1                                                   0.0s
 => => extracting sha256:dc3b3af10cf618c7b3038ca70bbcc422701fa0bf6cb9afae277406df7140445a                                                  13.0s
 => [stage-0  2/10] RUN rm -f /etc/apt/sources.list.d/cuda.list &&     rm -f /etc/apt/sources.list.d/nvidia-ml.list &&     apt-key del 7fa  2.7s
 => [stage-0  3/10] RUN --mount=type=cache,target=/var/cache/apt apt-get update -qq && apt-get install -qqy --no-install-recommends  make  13.9s
 => [stage-0  4/10] RUN curl -s -S -L https://raw.githubusercontent.com/pyenv/pyenv-installer/master/bin/pyenv-installer | bash &&  git c  55.7s
 => [stage-0  5/10] COPY .cog/tmp/build260375280/cog-0.0.1.dev-py3-none-any.whl /tmp/cog-0.0.1.dev-py3-none-any.whl                         0.0s
 => [stage-0  6/10] RUN --mount=type=cache,target=/root/.cache/pip pip install /tmp/cog-0.0.1.dev-py3-none-any.whl                          6.9s
 => [stage-0  7/10] RUN --mount=type=cache,target=/root/.cache/pip pip install   torch==1.12.1 --extra-index-url=https://download.pytorc  106.1s
 => [stage-0  8/10] RUN pip install -qq git+https://github.com/ShivamShrirao/diffusers                                                      8.1s
 => [stage-0  9/10] WORKDIR /src                                                                                0.0s
 => [stage-0 10/10] COPY . /src                                                                                2.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:e13bd98fb352c579857ff1d660ab4e2ce4846d644aeb6ca05a867c8c3ed82fac                                                0.0s
 => => naming to r8.im/rdev/sdtest                                                                                0.0s
 => exporting cache                                                                                0.0s
 => => preparing build cache for export                                                                                0.0s
Adding labels to image...

Pushing image 'r8.im/rdev/sdtest'...
Image 'r8.im/rdev/sdtest' pushed

Run your model on Replicate:
    https://replicate.com/rdev/sdtest

The actual model ID I see in Replicate dashboard is not present anywhere in the logs 🤔

also not seeing the latest: digest: sha256... when pushing from a linux box on ec2. i do see it when running from my mac though.

Implementation thoughts:
@bfirsh suggested that cog push could print logs to STDERR, and meaningful response data to STDOUT.
@evilstreak suggested that the STDOUT response could be a JSON object with the model ID inside.

Just talked this through with Docker and Cog expert @bfirsh. It may be a bit difficult to get this right without still having a race condition. I'll leave it to @bfirsh to chime in here with details.

In the meantime, here's a Docker-based approximation of cog push that gives you a way to snag the image id using Docker tags. This workflow could be used to run multiple Docker builds in parallel.
Specify a Docker tag when building your image, so you can retrieve it later from the Docker registry:
cog build -t r8.im/zeke/haiku-standard:my-tag

When the build is finished you can retrieve metadata for it using the full image URL, and use a tool like jq to extract the hash from the resulting JSON metadata:
docker inspect r8.im/zeke/haiku-standard:my-tag | jq ".[0].Id"

Then push the image to the registry:
docker push r8.im/zeke/haiku-standard:my-tag


Elasticsearch is trying to access loggedin user's "profile" property, which returns null

Found a bug from the recent develop version, I am getting error from elasticSearch.js from line 22.
Exception while invoking method 'getChartData' TypeError: Cannot read property 'profile' of null
I20151215-14:02:54.605(2)?     at [object Object].Meteor.methods.getChartData (server/methods/elasticSearch.js:22:1)

@jykae please provide some steps to reproduce. Is this when you deploy? What is the context around this error?

@frenchbread is the [chartsLayout template](https://github.com/apinf/api-umbrella-dashboard/blob/develop/client/views/dashboard/charts/charts.js#L67 used in any routes where anonymous users can access? We may need to make sure the user is logged in before the getChartsData method is envoked, otherwise there needs to be a try catch around the failing code.

If i'm recalling correctly, chartsLayout template is not used in any routes. It is used in a dashboard template only. (Did some some search for chartsLayout occurrences within the project folder).
Tested this issue locally by calling Meteor.call("getChartData") and got similar error:
Exception while invoking method 'getChartData' TypeError: Cannot read property 'profile' of null

Seems like when deploying, getChartData gets called for some reason.
Agreed that error handling is needed there. What about checking if user isLoggedIn inside getChartsData method:
if (Meteor.user())
    // continue getting charts data
else
   // terminate

Created a hotfix. Please review & test.
This should fix the problem now.

@frenchbread great work!


Modernize print level semantics

Print level semantics of the BiCGStabSolver differ from the semantics of all other solvers' .
This is more or less a hotfix. Print level semantics are unclear at the current state - at least I could not find any documentation on what the integers represent exactly. I also was unable to reverse this information from the source code. Hence I suggest that the integer input should be at least documented - or even better exchanged with some enum to clarify the semantics. The latter proposal is not part of this PR for the previously mentioned reasons.
With this PR a new mechanism to set the print level is proposed where the previously utilized print_level integer is exchanged with a more clear PrintOptions struct which is populated through the code base. In the same turn the docs for the iterative solver has been updated.
Todos

[x] Improve docs
[x] Exchange print_level with the novel print_options
[x] Provide some "manual" tests
[x] Check doxygen

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2519
@termi-official
@tzanio
@tzanio + @v-dobrev + @pazner
09/13/21
12/07/21
⌛due 12/14/21

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Thanks @termi-official.
Maybe we can use this as an opportunity to clarify the print_level and properly document it.
I could be wrong, but my recollection is that the intention for print_level was:

-1 : no output
0 : print only if there are issues such as no convergence or non positive-definite preconditioner
1 : print each iteration + potential issues (the output of 0)
2 : print just a final summary at the end + potential issues (the output of 0)
other values: special options for the particular solver that need not be exposed to users

I can use cases for all of these (the default is 0) but maybe we are not consistent everywhere.
What do you think?
Tzanio

Hi Tzanio,
sure, I can grab this. However, before proceeding I want to shortly discuss this.
First I have a small follow up question as I still cannot map your description fully to the implementation: What is the intend of print level 3? It seems to be the same as 1 for all linear solvers but SLI.
Regarding the interface I have two proposals to improve the interface.
A) Introduce an enum, where we have an inclusive ordering, with the following semantics:

-1 : suppress output
0 : errors only errors plus warning
1 : same as 0 but with summary at the end
2 : same as 1 but with potential issues
3 : same as 2 but with full iteration information
and where internal custom print flags are set with specialized methods.

B) Alternatively I would suggest that the input could be a flag where

1st bit: print on errors
2nd bit: print summary
3rd bit: print warnings
4th bit: print iteration info
remaining bits: custom info with docs on the corresponding solver

Since both of the proposed changes are breaking I am also fine with just documenting the current state in this PR and to check whether the solvers comply to the semantics as defined.

Regarding the interface I have two proposals to improve the interface.
A) Introduce an enum, where we have an inclusive ordering, with the following semantics:
[…]
B) Alternatively I would suggest that the input could be a flag where
[…]
Since both of the proposed changes are breaking I am also fine with just documenting the current state in this PR and to check whether the solvers comply to the semantics as defined.

I'm supportive of both/either of these suggestions. Note that we could potentially do this in a non-breaking way by introducing a strongly typed enum, and adding an overload to SetPrintLevel that will translate between enums and the integer flags.

My proposal is almost finished. Not a good idea how to handle the function forms of the solvers (CG, PCG, ...). I also added some docs on the iterative solver base class in the process of working out what exactly is going on in each solver. Output behaviour should now be at least consistent through all Mult calls.
I have another thing which I could not resolve properly: Should on error everything printed to mfem::err or should the context information (residuals, iteration count, ...) only be printed to mfem::out?
Finally I want to note that I had no time to check if the doxygen pages are generated correctly.

Doxygen also works now as intended on my local machine. I am not sure how to access the repo-check / documentation output. Other than the open question on the semantics of the error output (see comment above) this should be fine now. I was able to preserve the interface similar to the idea @pazner suggested, however the output formatting is not 1:1.
I am not sure why the style check fails.

I am not sure why the style check fails.

We switched to a new version of AStyle in master. I merged it here, so it should be fixed.

Hi @termi-official, first of all thanks for your work on this PR, I think it will make a very nice improvement! Secondly, I apologize for asking for yet another round of modifications, but we discussed this as a team, and we thought that the following proposal would be the best solution in terms of expressivity/lack of verboseness/etc. See this gist for a proof of concept.
The idea is to return to your original idea of a struct with bool flags, but allow for setting the flags easily using daisy chaining: PrintLevel().Warnings().Errors().Summary(). This would reduce the verbosity of flags like IterativeSolver::PrintLevel::WARNINGS, and would be pleasant to work with within the solvers (if (print_options.warnings) { … }). It's also pretty natural to convert to and from the legacy print level integers.
Also note that @tzanio suggested we shorten iteration_details to iterations, and @v-dobrev brought up the point that many solvers support a "print level 3", which includes the first and last iteration (but not any in between), as well as a summary at the end (convergence factor, etc.). Those changes are reflected in the gist.
What do you think about this idea?

No problem - I have to thank for taking so much time and effort here! I think these proposal are great. I will also propagate the "new" print level 3 through the solvers.

Edit: Do you also had the chance to shortly discuss the output policy regarding mfem::err ?

I think that errors and warnings should go to err, but all other informational messages should go to out.

Now that I have tried to recover the "first and last" semantics I am not sure anymore what the difference to the summary should be. The way I implemented it feels like I have misunderstood something (see e.g. https://github.com/mfem/mfem/blob/bicgstab-print-lvl-fix/linalg/solvers.cpp#L652-L667 ).

@v-dobrev, any objections to merge in next??

Should have marked this as don't merge yet, sorry. There are still some errors and inconsistencies which I have to fix, but I could not find time to tackle this yet.

Extending the review deadline to next week

I think all bugs are gone and the output is consistent across the solver categories (i.e. linear solvers have consistent output and nonlinear ones have a slightly different convention).
However, the printing should be refactored at some point (like in the SLBQP where we have a method print_iteration).
Updated manual test for checking the linear solvers:
#include "mfem.hpp"
#include <fstream>
#include <iostream>
#include <vector>

using namespace std;
using namespace mfem;

int main(int argc, char *argv[])
{
    // --------- test setup ---------
   const char *mesh_file = "../data/star.mesh";
   int order = 1;

   Mesh mesh(mesh_file);
   for(int i=0;i<2;i++)mesh.UniformRefinement();

   H1_FECollection fec(order, mesh.Dimension());
   FiniteElementSpace fespace(&mesh, &fec);

   Array<int> boundary_dofs;
   fespace.GetBoundaryTrueDofs(boundary_dofs);

   GridFunction x(&fespace);
   x = 0.0;

   ConstantCoefficient one(1.0);
   LinearForm b(&fespace);
   b.AddDomainIntegrator(new DomainLFIntegrator(one));
   b.Assemble();

   BilinearForm a(&fespace);
   a.AddDomainIntegrator(new DiffusionIntegrator);
   a.Assemble();

   SparseMatrix A;
   Vector B, X;
   a.FormLinearSystem(boundary_dofs, x, b, A, X, B);

   GSSmoother M(A);

    // ---------- start of actual test --------------
   std::vector<std::pair<std::string, IterativeSolver*>> solvers_to_test;
   solvers_to_test.push_back(std::make_pair("CG", new CGSolver()));
   solvers_to_test.push_back(std::make_pair("BiCGSTAB", new BiCGSTABSolver()));
   solvers_to_test.push_back(std::make_pair("GMRES", new GMRESSolver()));
   solvers_to_test.push_back(std::make_pair("MINRES", new MINRESSolver()));
   solvers_to_test.push_back(std::make_pair("FGMRES", new FGMRESSolver()));

   // Some print options.
   constexpr char header_fillchar = '#';
   constexpr char test_sep_fillchar = '-';
   constexpr char inner_sep_fillchar = '~';
   constexpr int half_width = 40;

   for(auto [solver_name, solver] : solvers_to_test)
   {
      std::cout << '\n' << std::setfill(header_fillchar) << std::setw(2*half_width) << header_fillchar << std::endl;
      std::cout << std::setfill(header_fillchar) << std::setw(half_width) << std::right << "Testing "
                  << std::setfill(header_fillchar) << std::setw(half_width) << std::left << solver_name << std::endl;
      std::cout << std::setfill(header_fillchar) << std::setw(2*half_width) << header_fillchar << std::endl << std::endl;

      solver->SetOperator(A);
      solver->SetPreconditioner(M);

      for(int print_level = 0; print_level <= 3; print_level++)
      {
         std::cout << '\n' << std::setfill(test_sep_fillchar) << std::setw(2*half_width) << test_sep_fillchar << std::endl;
         std::cout << std::setfill(test_sep_fillchar) << std::setw(half_width) << std::right << "PRINT LEVEL="
                   << std::setfill(test_sep_fillchar) << std::setw(half_width) << std::left << print_level << std::endl;
         std::cout << std::setfill(test_sep_fillchar) << std::setw(2*half_width) << test_sep_fillchar << std::endl << std::endl;

         solver->SetPrintLevel(print_level);

         // Test "divergence"
         x = 0.0;
         solver->SetMaxIter(3);
         solver->SetRelTol(0.0);
         solver->SetAbsTol(0.0);
         solver->Mult(b, x);

         std::cout << "\n\n"
                   << std::setfill(inner_sep_fillchar) << std::setw(half_width) << std::left << " >>> v Convergence"
                   << std::setfill(inner_sep_fillchar) << std::setw(half_width) << std::right << " Divergence ^ <<<\n" << std::endl;

         // Test convergence
         x = 0.0;
         solver->SetMaxIter(500);
         solver->SetRelTol(1e-3);
         solver->SetAbsTol(1e-3);
         solver->Mult(b, x);
      }
   }

   return 0;
}

Since the nonlinear ones do not have such a complicated output mechanism I have not tested them beyond eyeballing.

Merged in next for testing...

There are a few things that came up in the nightly testing that we need to address:

The variable IterativeSolver::print_level is used in the derived class mfem::KINSolver and in that case it has a meaning as defined by SUNDIALS' function KINSetPrintLevel. Due to this (and other issues with this deprecation attribute), I think it is better to remove the deprecation attribute from IterativeSolver::print_level -- having both print_level and print_options should be fine. One problem is that the base class method IterativeSolver::SetPrintLevel(int) (when used with mfem::KINSolver) is that it will try to set also the print_options which is not correct due to the different meaning of "print level" in that class. We can just override the method SetPrintLevel(int) in the sub-class.
We used to treat "no convergence" conditions as a warnings before, whereas now they are treated as errors. Both make sense based on the context, so one approach could be to allow these different treatments to be set by the user while keeping the default to be "treat no-convergence as a warning". Alternatively, we can always treat "no convergence" as a warning (with message going to mfem::out if warning-output is enabled) and let the user check explicitly for non-convergence if they want to treat this as an error.
The new macros for suppressing the deprecation warning do not work with the Intel compiler. As mentioned above, I think we should just remove the deprecation attribute from IterativeSolver::print_level.

What do you think?

Thanks for the detailed feedback Veselin.
Let me answer to your points with an analogous numbering:

The intent of the deprecation warning was to make users aware of the change. Overriding IterativeSolver::SetPrintLevel is how I intended to set the print level consistently, whereas custom print level options could be introduced by either a different method, e.g. CustomSolver::SetCustomPrintLevel or by providing an overload of IterativeSolver::SetPrintLevel with some custom type. But yes, we can also remove the warning (small discussion is found in point 3.).
I think the second approach makes most sense for everyone. We should not complicate the solver implementations with a too complicated log management - one intent of this PR was to simply the logging anyway (at least long term).
I can either fix this issue (have several version of the Intel compiler accessible and I think it should have an analogous feature to gcc and clang) or I can remove the deprecation warning. I do not have a strong opinion here, just wanted to notify users which implement specialized iterative solvers, that this will become a (very minor) breaking change at some point. However, this should be so easy to fix that the deprecation warning should not really be necessary.

If we agree here I am happy to incooperate the changes.

Sorry @v-dobrev, I missed that your questions were addressed to everyone. My thoughts are:

I am OK with removing the deprecation attribute from IterativeSolver::print_level.
I will prefer to treat "no convergence" as a warning (with message to mfem::out).

Re-merged in next for testing ...

Thank you so much for taking the time and helping us with this nice contribution @termi-official 👍
@v-dobrev and @pazner -- Thank you for reviewing!

I have to thank for the nice discussions and also for your time here!


internal tool error on MacOS Catalina

When I try to run metanome on macOS, I can't see any algorithms.
Logs:
➜  deployment-1.2-SNAPSHOT-package_with_tomcat ./run.sh
This script does not support Microsoft Windows please use the run.bat script.

Adding Context  for frontend
Adding Context  for backend
Oct 13, 2019 9:12:03 PM org.apache.coyote.AbstractProtocol init
INFO: Initializing ProtocolHandler ["http-nio-8080"]
Oct 13, 2019 9:12:03 PM org.apache.coyote.AbstractProtocol init
INFO: Initializing ProtocolHandler ["http-nio-8081"]
Oct 13, 2019 9:12:03 PM org.apache.tomcat.util.net.NioSelectorPool getSharedSelector
INFO: Using a shared selector for servlet write/read
Oct 13, 2019 9:12:03 PM org.apache.catalina.core.StandardService startInternal
INFO: Starting service Tomcat
Oct 13, 2019 9:12:03 PM org.apache.catalina.core.StandardEngine startInternal
INFO: Starting Servlet Engine: Apache Tomcat/8.5.5
Oct 13, 2019 9:12:03 PM org.apache.tomcat.util.net.NioSelectorPool getSharedSelector
INFO: Using a shared selector for servlet write/read
Oct 13, 2019 9:12:03 PM org.apache.catalina.core.StandardService startInternal
INFO: Starting service Tomcat
Oct 13, 2019 9:12:03 PM org.apache.catalina.core.StandardEngine startInternal
INFO: Starting Servlet Engine: Apache Tomcat/8.5.5
Oct 13, 2019 9:12:03 PM org.apache.catalina.startup.ContextConfig getDefaultWebXmlFragment
INFO: No global web.xml found
Oct 13, 2019 9:12:04 PM org.apache.catalina.startup.ContextConfig getDefaultWebXmlFragment
INFO: No global web.xml found
Oct 13, 2019 9:12:04 PM org.apache.jasper.servlet.TldScanner scanJars
INFO: At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
Oct 13, 2019 9:12:04 PM org.apache.coyote.AbstractProtocol start
INFO: Starting ProtocolHandler [http-nio-8080]
Oct 13, 2019 9:12:07 PM org.apache.jasper.servlet.TldScanner scanJars
INFO: At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
[Server@18011049]: [Thread[localhost-startStop-1,5,main]]: checkRunning(false) entered
[Server@18011049]: [Thread[localhost-startStop-1,5,main]]: checkRunning(false) exited
Oct 13, 2019 9:12:09 PM org.hibernate.engine.jdbc.connections.internal.DriverManagerConnectionProviderImpl configure
WARN: HHH000402: Using Hibernate built-in connection pool (not for production use!)

Logs stop here, after >10 min still no message that the server was started. When I open localhost:8080, all requests to localhost:8081 fail with http 500 (doesn't matter if CORS is turned on or off):

How to reproduce:

macOS version 10.15
deployment version: https://hpi.de/fileadmin/user_upload/fachgebiete/naumann/projekte/repeatability/DataProfiling/Metanome/deployment-1.2-SNAPSHOT-package_with_tomcat.zip
follow https://github.com/HPI-Information-Systems/Metanome/wiki/Metanome-User-Guide#operate-metanome

Solved by using a deployment package built by the repository status


OnRest called too many times for Motion

Hi Guys
I am using Motion to do a simple animation of width in my web application.
I want to do a specific action once the animation has completed so I tried using onRest prop.
The issue is that onRest is called too many times. I could not find a onEnd prop like that would get called once animation has completed
Please suggest what I am missing

Also can I dispatch an action when animation completes like inside the onRest prop?

Conceptually, the animation has no end, the onRest callback is simply called when it is idle. If you want the callback to have an action only after the first animation is complete, you should try to do something like this:
class MyComponent extends Component {
  constructor(props) {
    super(props)
    this._hasHandleRestBeenCalled = false
    this.handleRest = this.handleRest.bind(this)
  }
  handleRest() {
    if(this._hasHandleRestBeenCalled) {
      return
    }
    this._hasHandleRestBeenCalled = true
    // handle first onRest
  }
  render() {
    return (
      <Motion defaultStyle={…} style={…} onRest={this.handleRest}>
        {…}
      </Motion>
    )
  }
}

Thank you @bloodyowl for the example
In my case though after animation some dom elements may have been moved due to other dom elements becoming hidden. So I am trying to get new offsets which I will get correctly only after animation ends
So I need to know when the last onRest is called and I would really like to avoid putting a timer for the same


Rafik Gerges adding Compliance/information-risk-management-reference-architecture.md

Kindly reach out to<EMAIL_ADDRESS>in case of any query. Thanks

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@JohanFreelancer9 and @yogkumgit - proposed changes have been committed. Many thanks !

Dear @JohanFreelancer9 , done. Thank you so much !

@robmazz Please merge this PR. Thanks

Hi @robmazz - kindly merge. Thanks

Hi @robmazz , kindly merge. Thanks

@RaGerges - I'm not familiar with this project and these new topics, are these topics part of a plan you've coordinated with members of the compliance content team for DOCs? Who have you been working with on these topics? Please let me know, if it's easier you can ping me on teams.

@yogkumgit - @RaGerges and I have touched base and we'll be discussing this content and the schedule. Adding Do Not Merge label for now until we've discussed and confirmed planning.

@yogkumgit - @RaGerges and I have synced again, closing this request. We are going to review the content offline and determine the best way to surface this information for customers.


Can't sign PSBT with connected Ledger hardware taproot wallet

I'm trying to sign a PSBT in Sparrow with my connected Ledger hardware taproot wallet.
When I click 'sign' with my Ledger in Sparrow, I see no prompt on my Ledger and there is no error messages in the Sparrow log. Nothing happens.
The taproot wallet was created inside Ledger Live and imported to Sparrow.
I have successfully used my Ledger segwit wallet with Sparrow.
Sparrow 1.7.3, Ledger Bitcoin app 2.1.1. Firmware up to date.
The PSBT:
cHNidP8BAF4CAAAAAX27a/e1mkAGq+wqtUtzW6YCn5q5PNYDibpP4Yh64lwxAAAAAAD/////AQDh9QUAAAAAIlEgHyLoQu8pJTiotEfmdgAhn2RTQ+SKtYVGLDf0gKdYKjIAAAAAAAEAXgEAAAABPWUqcfTHaU5Bagp+RsbqwDUMuRl8G5hI9nFBvf8CMrgAAAAAAP3///8BVCUAAAAAAAAiUSAfIuhC7yklOKi0R+Z2ACGfZFND5Iq1hUYsN/SAp1gqMgAAAAABAwSDAAAAAAA=
Thank you

I should add, when I try to create a send txn with the same wallet, I can sign that fine with my ledger.

I believe the failure may either be because there are required fields missing from the PSBT (which I understand to have been created externally to Sparrow), or that Ledger does not support this sighash flag on Taproot signatures.
Here is some debugging output from this PSBT:
Transaction with txid: 69b4a9d519a529e39d9bde5e74373adf23036d06202b1b4364807f1a4a44ed3d version 2 size 94 locktime 0
 Transaction input references txid: 315ce27a88e14fba8903d63cb99a9f02a65b734bb52aecab06409ab5f76bbb7d vout 0 with script
 Transaction output value: 100000000 to addresses [bc1pru3wssh09yjn3295gln8vqppnaj9xsly326c233vxl6gpf6c9geql6sct5] with script hex 51201f22e842ef292538a8b447e67600219f645343e48ab585462c37f480a7582a32 to script OP_1 1f22e842ef292538a8b447e67600219f645343e48ab585462c37f480a7582a32
Found input non witness utxo with txid: 315ce27a88e14fba8903d63cb99a9f02a65b734bb52aecab06409ab5f76bbb7d version 1 size 94 locktime 0
 Transaction input references txid: b83202ffbd4171f648981b7c19b90c35c0eac6467e0a6a414e69c7f4712a653d vout 0 with script
 Transaction output value: 9556 to address bc1pru3wssh09yjn3295gln8vqppnaj9xsly326c233vxl6gpf6c9geql6sct5 with script hex 51201f22e842ef292538a8b447e67600219f645343e48ab585462c37f480a7582a32 to script OP_1 1f22e842ef292538a8b447e67600219f645343e48ab585462c37f480a7582a32
Found input sighash_type Single + Anyone Can Pay

Compare this to the example in https://github.com/bitcoin-core/HWI/issues/608, particularly the lack of the taproot internal key field. The sighash flag is also unusual - you may wish to try signing a PSBT created in Sparrow with this sighash to narrow things down.

I have experienced exactly the same issue. It doesn't seem possible to sign a PBST in Sparrow with connected Ledger taproot wallet.

I have experienced exactly the same issue. It doesn't seem possible to sign a PBST in Sparrow with connected Ledger taproot wallet.

Note this issue specifically addresses a PSBT created externally to Sparrow. You need to upgrade the Bitcoin app on Ledger to 2.1.1 for Taproot support.

I have Bitcoin app 2.1.1 on Ledger. When I use a software Taproot wallet on Sparrow I can sign a PSBT. When I use a Ledger I can't.

I have Bitcoin app 2.1.1 on Ledger. When I use a software Taproot wallet on Sparrow I can sign a PSBT. When I use a Ledger I can't.

Ok - can you create a separate issue with the details?


pywavelet - not clear if needed but if needs to be part of env

I found pywavelet to be super fast wavelet lib better then scipy - but not yet clear if needed
assumtion is the a CWT maybe a good feature extracor

pused to future work


[FLINK-4422] Convert all time interval measurements to System.nanoTime() - flink-connector-kafka-0.9

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

+1 to merge, with #3422.


Proposals fail to load - TypeError: Cannot read property 'proposal' of undefined

Expected behavior
Proposals on https://gov.harmony.one/#/ should load successfully and allow users to view all proposals, and vote on active ones.
Actual behavior
The majority of proposals fail to load.
The Harmony Mainnet, Harmony Testnet, Validator Dao Mainnet, and Validator Dao Testnet spaces load the proposals with no issues.
All others get stuck trying to load the proposals (for active, pending, and closed).

This has been a persistent bug for several months. Sometimes a proposal will load, but the vast majority of the time it gets stuck loading the proposals. Many users have reported in community TG channels that they are having this same issue.
The console logs "TypeError: Cannot read property 'proposal' of undefined".

Steps to reproduce the behavior

Visit a space other than Harmony Mainnet, Harmony Testnet, Validator Dao Mainnet, and Validator Dao Testnet on https://gov.harmony.one/#/
Open any proposal (active, closed, or pending)

thanks for feedback. get some information after detect the error:
sometimes the IPFS request will return 503 when load the proposal(see the image below). will try to fixed it. let it more stable then.

hey @Zil-B . we have optimized the RPC interface of mainnet government. can you help me check whether it is sitll always fail to load. thanks

@jhd2best Working great on my end, and users are reporting the same. Proposals all load within seconds. Good to close?
Thanks

Closing - issue seems to be resolved


add more tables and data

In this phase of the project, I have answered the following questions;
1 Who was the last animal seen by William Tatcher?

2 How many different animals did Stephanie Mendez see?

3 List all vets and their specialties, including vets with no specialties.

4 List all animals that visited Stephanie Mendez between April 1st and August 30th, 2020.

5 What animal has the most visits to vets?

6 Who was Maisy Smith's first visit?

7 Details for most recent visit: animal information, vet information, and date of visit.

8 How many visits were with a vet that did not specialize in that animal's species?

9 What specialty should Maisy Smith consider getting? Look for the species she gets the most.

Thank you
Sent from Yahoo Mail on Android
On Fri, Apr 7, 2023 at 12:13 PM, Kasongo Mukebo @.***> wrote:
@BenMukebo approved this pull request.
Hi @Divinity-dev, 👋
STATUS: Approved 🎉
Great work on making the changes requested by a previous reviewer 🥇
Your project is complete! There is nothing else to say other than... it's time to merge it
Congratulations! 🎉 🎉 🎉
To Highlight 🥇

Your Pull Request has a proper title and description. ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding! 👏 👏 👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me on( @BenMukebo ) in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
In data.sql:

+INSERT INTO specialization(species_id, vets_id)VALUES
+(1, 1),
+(2, 3),
+(1, 3),
+(2, 4);
+--insert data into visits table

[OPTIONAL] This insertion could have been done dynamically where you do not know the vet_id and specie_id.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Table component, a several questions want to know .I hope author can give me alittle init

1 How do set the request type, post or get
2 How do I access the server。 Follow the documentation action ，Did not get the results ，Nothing is displayed

3 Table output  whether can support json arrays
Thank you    I'm sorry to trouble you    Hope to get the answer

Quoting the docs:
<table class="table striped table-border mt-4"
   data-role="table"
   data-source="data/table.json">
</table>

To use external source, you must create JSON data in specific format. Example of JSON file.

{
    header: [...],
    data: [...]
}

You are using data-view-... is to load/save a specific layout for the table's columns, as well described in detail in the docs https://metroui.org.ua/table-component.html#_table_inspector

for data-view-* https://metroui.org.ua/table-component.html#_table_inspector


Create wiki page for Overpass challenges

Create a wiki page that describes how to correctly create a challenge using Overpass, ideally with a "cookbook" of example queries for common scenarios and a "troubleshooting" section addressing frequent mistakes and misconceptions. See #954 and #721
The new wiki page could be referenced as additional documentation during challenge creation to fix #912
Current help on the wiki references a general diary entry on challenge creation, which does contain some details of Overpass: https://www.openstreetmap.org/user/mvexel/diary/46863
Old MR2 wiki entry on challenge creation (since deleted): https://github.com/maproulette/maproulette2/wiki/Introducing-New-MapRoulette---Part-1.-Creating-and-Maintaining-Challenges/d67f308dadd4f736a58ab12314f3302689dcb56e

@Binnette Is wiki access is holding you up? If you want to create a gist I will get the content on to the wiki. Thank you for helping with this!
cc @mvexel

Hi @nrotstan, I still have no write access to the wiki. I can not edit this page:
https://github.com/osmlab/maproulette3/wiki/Using-Overpass-queries-to-create-Challenges

Hi @Binnette, can you create a Gist (see gist.github.com)  and link back to it in this issue, please? We will then merge it into the wiki. Thanks and sorry about the delay responding on my part.

A link to Martijn's new wiki page is now included in the updated challenge forms and deployed to maproulette.org


switchLocalePath does not work in layouts/error.vue

Is it possible to switch language on error page (layout)?
It is known that the route must have a name for switching language.
function switchLocalePath (locale) {   const name = this.getRouteBaseName()   if (!name) {     return ''   } }
After redirect on the layouts/error.vue route does not have name.
How to handle this problem?

It should be possible to support that. Would need some changes in the module.
Or at least the module could attempt to generate a locale-specific path but that wouldn't guarantee that such a route exists.
And it wouldn't work with "custom paths" as we currently need route base name to figure those out.

What are your use-cases exactly?
You want to change the locale on a 404 page or on other kinds of error pages?
This is not really that easy or even possible in some cases. While switching locale using setLocale API is already possible on error pages, switching using NuxtLink is likely not possible in all cases because we can't resend the same error to the route we want to switch to. If it would be just a 404 error then it would more or less work but for other kinds of errors not so much.

What are your use-cases exactly?

Use case is simple and exactly the same as for all other translated pages on the website. If the user clicks a link leading to our page and the page no longer exists, they may want to switch the language of 404 information to the one they understand...

What are your use-cases exactly?
I have a default layout with a footer (that applies to all pages). In that footer is a language switcher. It does nothing on error pages.

As I said, though, it depends on the kind of error if it's even possible to handle that.
If the page errors-out because of a failed axios request then navigating to another locale will re-load the page and that original error will be lost. The newly loaded page might trigger the same error or it might just load correctly.
So the part that I don't like about that is that the language switch could result in information loss.
I've also mentioned that switching locale through the setLocale API should work already. So it's possible to implement error-page-specific language switcher that would work without changing the URL.

I was able to solve this by adding a click handler to my language selector. The click handler calls setLocale, but I also use the :to directive to update the URL (if applicable), using switchLocalePath.
My LocaleMenu component looks like this:
<template>
  <v-menu
    transition="slide-y-transition"
    offset-y
  >
    <template v-slot:activator="{ on, attrs }">
      <v-btn
        depressed
        v-bind="attrs"
        v-on="on"
        v-text="$i18n.localeProperties.name"
      />
    </template>
    <v-list>
      <v-list-item
        v-for="locale in $i18n.locales"
        :key="locale.code"
        :to="switchLocalePath(locale.code)"
        nuxt
        @click="$i18n.setLocale(locale.code)"
      >
        <v-list-item-title>{{ locale.name }}</v-list-item-title>
      </v-list-item>
    </v-list>
  </v-menu>
</template>

<script>
export default {
  name: 'LocaleMenu',
}
</script>

Closing this as v7 is not being actively worked on (critical hotfixes only), if this still applies to v8 and later please open a new issue!


Using JNet 1.5.5 some limits or constraints of JNetReflector are raised

          Using [JNet 1.5.0](https://github.com/masesgroup/JNet/releases/tag/V<IP_ADDRESS>) some limits or constraints of JNetReflector are raised:

A class containing main method is reflected using JVMBridgeMain<TClass>, but fields and methods are not available; plus it is not possible to allocate it to use dynamic behavior to access fields and mehods
Some language keyword, e.g. event, are not replaced with new one if used in the reflected code

Originally posted by @masesdevelopers in https://github.com/masesgroup/KNet/issues/168#issuecomment-1565488034

Other limitations raised are:

A class reported to be Listener does not contains nested classes
In some packages (e.g. org.apache.kafka.connect.storage) the classes are created, but AllPackageClasses.cs not contains all classes
Add a new switch to rename a class which clash with namespace: current feature does not work if the class name clash happens between different JARs or modules
When a namespace is avoided (e.g. org.apache.kafka.common.internals) some methods still reports types from that namespace
Some Iterable classes (e.g. ConsumerRecords) inherits from Java.Lang.Iterable<K, V> which is syntactically wrong, the correct superclass is Java.Lang.Iterable<ConsumerRecord<K, V>>
Some generic return types are not reported in the right way: e.g. KafkaProducer.Record() reports Java.Util.Map, it must be like Java.Util.Map<MetricName, Metric>
Some where clause are applied, but the constraints are related to the interfaces which are not reported in the class definition, but as operator associated to the class: the conditions break the applicability of where clause

Other limitations raised are:
* In some packages (e.g. `org.apache.kafka.connect.storage`) the classes are created, but `AllPackageClasses.cs` not contains all classes

This issue depends on package name shared between multiple JARs; so when latest JAR is analyzed it overwrites the previous AllPackageClasses.cs file, leaving the some orphan class in the folder

Other limitations raised are:
* A class reported to be Listener does not contains nested classes

Waiting for #137 create empty class for listener which accepts nested classes

Other limitations raised are:
* Some where clause are applied, but the constraints are related to the interfaces which are not reported in the class definition, but as operator associated to the class: the conditions break the applicability of where clause

The first step will be: associate a C# interface to any JVM Class reported to be an interface

Other limitations raised are:
* `Iterator<E>` shall be managed like it is done for `Iterable<E>`

* Some Iterable classes (e.g. `ConsumerRecords`) inherits from `Java.Lang.Iterable<K, V>` which is syntactically wrong, the correct superclass is `Java.Lang.Iterable<ConsumerRecord<K, V>>`

The following piece of code
https://github.com/masesgroup/JNet/blob/44acc282db09b92ddad136faa7bd0eacdefdbff4/src/net/JNetReflector/InternalExtensions.cs#LL903C1-L914C22
does not manage generic Java.Lang.Iterable in the right way, shall be analyzed the GenericInterface.

Other limitations raised are:
* When a namespace is avoided (e.g. `org.apache.kafka.common.internals`) some methods still reports types from that namespace

The problem comes from two main reason:

when a base class is retrieved, if it is a super class it is checked, but if the type is defined from the single interface the type is not checked
if ta type is a generic, it is checked only the outer type, not the inner type(s)


Added envoyfilters dependency to cds cache

Fix: #34356
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

/hold the second commit is to test

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-k8s-tests_istio

integ-cni-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-cni-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-k8s-tests_istio

integ-cni-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-cni-k8s-tests_istio

integ-security-multicluster-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test release-notes_istio

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-k8s-tests_istio

integ-cni-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-cni-k8s-tests_istio

integ-security-multicluster-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-k8s-tests_istio

integ-cni-k8s-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-cni-k8s-tests_istio

integ-security-multicluster-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
c247276ff09f89089509c78097d97e89115accd1
link
/test integ-pilot-multicluster-tests_istio

integ-multicluster-k8s-tests_istio
6fc00ad35f9b674802e46cda5ea9a0b5c54080b9
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test all

@howardjohn Ready for review now

/test benchmark_istio
/test integ-assertion-k8s-tests_istio

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

benchmark_istio
0886877d482d3b67299cb6cbe8ed63812bd911a8
link
/test benchmark_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test benchmark_istio

@ramaraochavali @howardjohn ptal, any other comments

In response to a cherrypick label: #34404 failed to apply on top of branch "release-1.11":
Applying: Add envoyfilter dependency to cluster cache
Using index info to reconstruct a base tree...
M	pilot/pkg/model/envoyfilter.go
M	pilot/pkg/networking/core/v1alpha3/cluster.go
M	pilot/pkg/networking/core/v1alpha3/cluster_builder.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster_builder.go
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster.go
Auto-merging pilot/pkg/model/envoyfilter.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Add envoyfilter dependency to cluster cache
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #34503


MissingMethodException

This piece of code fails...
    // METRICS.NET
    Metric.Config.WithReporting(report =>
          report.WithInflux(
               config.InfluxDbTargetDataRepositoryOptions.Host, // Uri
               config.InfluxDbTargetDataRepositoryOptions.UserName,
               config.InfluxDbTargetDataRepositoryOptions.Password,
               TimeSpan.FromSeconds(5)
     ));

with the error message being:
Unhandled Exception: System.MissingMethodException: Method not found: 'Metrics.Reports.MetricsReports Metrics.Reports.MetricsReports.WithRep
ort(Metrics.Reporters.MetricsReport, System.TimeSpan)'.
   at Metrics.NET.InfluxDB.ConfigExtensions.WithInflux(MetricsReports reports, Uri influxdbUri, String username, String password, TimeSpan interval, ConfigOptions config)

Also I wonder how the InfluxDB measurement name is being set.
Can you set it yourself?

What version of Metrics.NET and Metrics.NET.InfluxDB are you using?
What do you mean by measurement name? Have you looked at the MetricNameConverter facility?

I'm using the latest versions as of today.
InfluxDb has a concept of measurements, see Key concepts. It's basically a container where all your information goes to. For example I would like to have a measurement "projectX metrics".

Are you looking for a generic InfluxDB client for .NET?

No of course not. There are plenty of libraries for that :-)
I'm looking for a working InfluxDB integration for Metrics.NET.
If you on the other hand need a InfluxDB client for the Metrics.NET integration I'd go with influxdb-csharp - but use RX for the aggregation instead of the collector project.

I don't think this lib supports your use case, sorry. Each metric is a separate measurement in the same InfluxDB database.

OK. Maybe I'll look into that :-)


Fix colab notebook

Hi. Made some changes to notebook:

fixed RuntimeError #21, #32, that was fixed in #31 and #37, but not for msgnet.ipynb ;
removed unused import torch.nn.functional;
prettified according to pep8;
changed os.system('wget ...') to direct calling !wget ... without importing os module.

Tested in colab (run all), the notebook works as expected without errors.

Thanks a lot!

Thanks a lot!


data-placeholder not working

The data-placeholder attr gets overwritten by options
<medium-editor custom-tag='h2' data-placeholder='Your name' :text='name' :options='options' v-on:edit='applyTextEdit'>
const options = {   placeholder: {     hideOnClick: false   } }
Expected outcome: Placeholder text = 'Your name'
Actual outcome: Placeholder text = 'Type your text' // the editor standard text.
Version:
<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

The data-attribs for medium editor are not supported. Please use the option prop for everything.
For configuring the placeholder, please refer to:
https://github.com/yabwe/medium-editor/blob/master/OPTIONS.md#placeholder-options

Please let me know if you need any additional support for a specific use case :)

I see. Thanks. Using option prop is a possible, but that means a new instance for each editor. Are you considering adding support for data-attributes in the future?

I get your point. I would accept a PR. You would have to add a prop named dataAttributes or something and then use the render function to add thes props to the element.
For now, I will reopen it. It is on the bottom of the TODO list though until the demand increase. First I need to put some time into release 1.1.0.

With #46 we are generating a new instance for each component. Also, I want to keep the code as simple as possible while data-attributes do not seem to be a convention in the VueJS community.
I will close this now and re-open for discussion when there is more demand.


Problem emitting binary files

Hi.
I am using this library, version 2 (socketIO-client 2).
I had trouble emitting binary files in chunks. this is the code:
progress = 0
dumpSize = os.stat(s_dummyDumpPath).st_size
dumpFile = open(s_dummyDumpPath, "r+b")
while True:
    piece = dumpFile.read(1024)
    if not piece:
        # Sending finish message to server
        serverSocket.emit(s_fileSendFin, { })
        break
    progress += len(piece)
    serverSocket.emit(s_fileReq, {"name": s_dummyDumpName,"progress": progress * 100 / dumpSize, "buffer": piece})
f.close()

And I get the following error:
Traceback (most recent call last):
  File "CTedAgent.py", line 40, in <module>
    serverSocket.emit(s_fileReq, {"name": s_dummyDumpName,"progress": progress * 100 / dumpSize, "buffer": piece})
  File "/home/toml/.local/lib/python2.7/site-packages/socketIO_client/__init__.py", line 413, in emit
    self._message(str(socketIO_packet_type) + socketIO_packet_data)
  File "/home/toml/.local/lib/python2.7/site-packages/socketIO_client/__init__.py", line 26, in wrap
    return f(*args, **kw)
  File "/home/toml/.local/lib/python2.7/site-packages/socketIO_client/__init__.py", line 208, in _message
    transport.send_packet(engineIO_packet_type, engineIO_packet_data)
  File "/home/toml/.local/lib/python2.7/site-packages/socketIO_client/transports.py", line 161, in send_packet
    packet = format_packet_text(engineIO_packet_type, engineIO_packet_data)
  File "/home/toml/.local/lib/python2.7/site-packages/socketIO_client/parsers.py", line 170, in format_packet_text
    return encode_string(str(packet_type) + packet_data)
  File "/home/toml/.local/lib/python2.7/site-packages/socketIO_client/symmetries.py", line 32, in encode_string
    return x.encode('utf-8')
UnicodeDecodeError: 'ascii' codec can't decode byte 0x90 in position 30: ordinal not in range(128)

Couldn't find any answer over the web :(

We haven't gotten around to adding binary support yet.
I think @feus4177 has a fork that adds binary support.
https://github.com/feus4177/socketIO-client-2
I'll try to see if we can integrate his changes in January.

Hope in the new year,we will get the binary support by author

Hope in the new year,we will get the binary support by author


unable to convert generated pdf using this library to MS Word format.

Hello, I had used this library for pdf generation, its really awsome.
Now i need to convert generated pdf to word document. Is there any way to do that???

Not an easy task. Maybe you take the PDF.js from mozilla as a renderer and then you use a word file generating js tool and then you "just"  have to write the conersion.
https://mozilla.github.io/pdf.js/
https://github.com/OfficeDev/office-js-docs/blob/master/reference/word/word-add-ins-reference-overview.md
But it will be a poorly result, because you cant rely on the positions of the text in the pdf, etc..


Variable name is displayed in an error message instead of resovling to the string it contains

I got the following error message while running puppeteer:
1. you did not perform an installation before running the script (e.g. `npx puppeteer browsers install ${browserType}`) or
but I believe it should be:
1. you did not perform an installation before running the script (e.g. 'npx puppeteer browsers install chrome') or
The offending line is here:
https://github.com/puppeteer/puppeteer/blob/82034fad620751dc6df57251c479466f14139c36/packages/puppeteer-core/src/node/BrowserLauncher.ts#L460
I think the single quotes were supposed to be backticks and the backticks were supposed to be single quotes.  However the firefox error case has firefox hardcoded so it might be better to just hardcode chrome for consistency.
Tthe firefox error message is here:
https://github.com/puppeteer/puppeteer/blob/82034fad620751dc6df57251c479466f14139c36/packages/puppeteer-core/src/node/BrowserLauncher.ts#L467

There is already a PR in flight to fix it - https://github.com/puppeteer/puppeteer/pull/13119


Storing values

Dear all,
Thank you for this very useful package.
I don't really know where to put this question and it is not really an issue. I am using Pyfixest on a very large dataset. It seems that when running several times pf.feols and storing the output in a list, it increases a lot the memory usage. I fill that the dataset I use is being copied and stored in the list each time I am adding a new pf.feols object.
I would like to know if:

Is it the case or the increase in size is only due to the storing of the fixed effects?
Is there any option to shrink the size of the pf.feols output in order to keep only the information relevant for a regression table?

Best,
Swan

Hi Swan - I've had the exact same problem before and that's why there are two function argument for feols and fepois that allow you to configure how to store the data objects in the Feols / Fepois objects:

store_data:

Whether to store the data in the model object, by default True. If set to False, the data is not stored in the model object, which can improve performance and save memory. However, it will no longer be possible to access the data via the data attribute of the model object. This has impact on post-estimation capabilities that rely on the data, e.g. predict() or vcov().

By setting store_data = False, you lose the capability to update the vcov matrix post estimation - the reason is that you cannot select a cluster variable that is not part of the initial model if the data is not stored fully. Additionally, it is not possible to predict when estimating with fixed effects, as the fixef() method needs access to the input data. etable() will still work - I think this is the option that you want.

copy_data allows you to select out of making an internal copy of the input data - this might help with memory as well but might change data outside of the function call if it is modified inside of the call (which currently shouldn't be the case, but there are no guarantees).

Generally I am wondering if this is a design flaw and we should change the default to not store all the data objects in the Feols object?

Hi s3alfisc,
Thank you for this fast answer. It really fits my needs! Since the main use of PyFixest I have is to be able to run FeOLS on large dataset, I think not storing the values should be a better default variable !
Have a great day!


add like and dislike functionality

No description provided.

can I work on this .

what should be in the env file

Yes you can work 😄 Fork the repository and Do your work

can you assign me here.

Yes you can work 😄 Fork the repository and Do your work
I need the env file

check the readme file I update the environment variable
1.create a env file in server
2. paste the env

Have you deleted all the videos , nothing is seen when I ran your application locally .

check the readme file I update the environment variable 1.create a env file in server 2. paste the env

what to paste in env?

check the readme file I update the environment variable 1.create a env file in server 2. paste the env

what to paste in env?

Have you deleted all the videos , nothing is seen when I ran your application locally .
sorry for this, Please wait 2 days i will solve this problem

check the readme file I update the environment variable 1.create a env file in server 2. paste the env

what to paste in env?

create a env file in server and paste this stuff
MONGO = mongodb+srv://dipaksk11:dipak@youtube.qytcjz0.mongodb.net/?retryWrites=true&w=majority&appName=Youtube
MONGO_DB = "youtube"
JWT = "gopikishan"

in the home page video is not visible that is the problem

Hi, can you add the data inside the mongodb once more , I accidently deleted it .


refactor(sbb-tooltip): optimize positioning and add hideCloseButton prop

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

See Review Guidelines for more information what is checked during review process.
Changes
Changes in this pull request:

optimized positioning on window scrolling and resizing
added hideCloseButton prop (useful, , e.g., for the sbb-datepicker placed inside the tooltip)

Browsers
I tested the build on the following browsers:

[x] Firefox Desktop
[x] Chrome Desktop
[ ] Edge Desktop
[x] Safari Desktop
[ ] Chrome Mobile
[x] Safari Mobile

Screen readers
I tested the build on the following browsers:

[ ] JAWS Firefox Desktop
[ ] JAWS Chrome Desktop
[ ] NVDA Firefox Desktop
[ ] NVDA Chrome Desktop
[ ] VoiceOver Safari Desktop
[ ] VoiceOver Chrome Desktop
[ ] VoiceOver Safari Mobile
[ ] Android Accessibility Suite Chrome Mobile

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Codecov Report

Merging #1553 (03d9ee1) into master (ab917b0) will decrease coverage by 2.75%.
The diff coverage is 60.58%.

@@            Coverage Diff             @@
##           master    #1553      +/-   ##
==========================================
- Coverage   54.85%   52.10%   -2.75%
==========================================
  Files          49       82      +33
  Lines        1659     3585    +1926
  Branches      406      968     +562
==========================================
+ Hits          910     1868     +958
- Misses        671     1583     +912
- Partials       78      134      +56

Impacted Files
Coverage Δ

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

src/components/sbb-clock/sbb-clock.tsx
0.00% <0.00%> (ø)

src/components/sbb-link-list/sbb-link-list.tsx
84.00% <ø> (-4.24%)
:arrow_down:

src/components/sbb-link/sbb-link.tsx
58.00% <ø> (-33.43%)
:arrow_down:

src/components/sbb-logo/sbb-logo.tsx
0.00% <ø> (ø)

src/components/sbb-menu-action/sbb-menu-action.tsx
60.00% <ø> (ø)

src/components/sbb-menu/sbb-menu.tsx
26.54% <ø> (ø)

...ts/sbb-navigation-action/sbb-navigation-action.tsx
66.66% <ø> (ø)

...onents/sbb-navigation-list/sbb-navigation-list.tsx
84.00% <ø> (ø)

...mponents/sbb-checkbox-group/sbb-checkbox-group.tsx
32.00% <32.00%> (ø)

... and 79 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


ES6 code package

This package uses ES6 syntax, this suits most but I'm having to jump through hoops.
This file is causing me pain, with its arrow functions : https://github.com/kauegimenes/jsonexport/blob/master/lib/core/helper.js
When using Typescript, the node_modules folder is ignored for casting files. Files in node_modules folder, for Typescript, are expected to be in distribution format and no transpiling takes place.
I use webpack but not babel and do not wish to install babel just to cast this package.
I'm going to make a pull request to dump this library back down. I hope you will except. I will use the fork till then.
Thank you in kind

pull request #36 created

Completed in #36

I'm not sure if I'm doing something wrong (beginner), but as part of webpack, I'm running UglifyJS for my Angular 4 project at build.  Currently I'm getting an error from this package because the package is using ES6 syntax. If I was reading correctly, I thought ES5 support this was built into the package with the referenced pull request.

Please read the docs: https://www.npmjs.com/package/jsonexport#browser
You most likely have the wrong import statement and need to add "jsonexport/dist"

Thanks so much @AckerApple , you were correct.


feat: add new design for welcome screen

You can view the figma file here.

The design should be responsive.

ℹ We recommend updating the styling of existing components, otherwise you can add extra components if there is a need.

You have to implement dropdown menu (to hide and show the section)

Welcome section

Community section

Resources section

Any changes to the above design are welcome. Please feel free to share your thoughts, and we will discuss them thoroughly.

⚠ Make sure to push all the changes into dev branch

Hi, @Anmol-Baranwal could you please assign this issue to me?

@rupali-codes
Will you work on this?
Please assign him, if you're not planning to do it.

@JoshiUtsav any updates?

Hello @rupali-codes, I've got an exam tomorrow, but right after that, I'll make the quick changes and send in the PR.

Hello @rupali-codes, I've got an exam tomorrow, but right after that, I'll make the quick changes and send in the PR.

okay, thanks


Dependency missing in library.properties file

Hello,
Would it be possible to add the libraries used in the examples as a dependency of this library?
For exemple:

ArduinoJson
NTPClient
LVGL 8.3.x

This would avoid having to manually install the libraries to test the examples.
https://github.com/esp-arduino-libs/ESP32_Display_Panel/blob/289d3575fee397d5ef80e10576b92497299db4a2/library.properties#L11
Regards

Hi @lboue,
The example documentation and .ino file specify the dependent libraries and their versions. And I believe that only the components that the source code under src depends on should be considered fixed dependencies. This follows the principle of minimal dependencies. Otherwise, if future examples rely on more components, adding all of them as fixed dependencies wouldn't be elegant.

OK I understand the reasoning even if I don't find it practical.

Thanks for your understanding! And please feel free to provide your suggestions.


operator reconciler update fails sporadically

The operator Update call fails sporadically at least after the gpu controller has run, but potentially after any of the controllers
E0622 10:15:07.773355       1 reconciler.go:118] intel-device-plugins-manager/controller/gpudeviceplugin "msg"="unable to update DaemonSet" "error"="Operation cannot be fulfilled on daemonsets.apps \"intel-gpu-plugin-fzn2t\": the object has been modified; please apply your changes to the latest version and try again" "name"="intel-gpu-plugin" "namespace"="" "reconciler group"="deviceplugin.intel.com" "reconciler kind"="GpuDevicePlugin" "DaemonSet"={"kind":"DaemonSet","apiVersion":"apps/v1","metadata":{"name":"intel-gpu-plugin-fzn2t","generateName":"intel-gpu-plugin-","namespace":"inteldeviceplugins-system","uid":"e4d331f4-279a-4b7a-b7ae-9d60865acb60","resourceVersion":"294160","generation":35,"creationTimestamp":"2021-06-22T10:05:59Z","labels":{"app":"intel-gpu-plugin"},"annotations":{"deprecated.daemonset.template.generation":"35"},"ownerReferences":[{"apiVersion":"deviceplugin.intel.com/v1","kind":"GpuDevicePlugin","name":"intel-gpu-plugin","uid":"5d80e18a-7319-4ae4-ae0f-a97beddf17c2","controller":true,"blockOwnerDeletion":true}],"managedFields":[{"manager":"intel_deviceplugin_operator","operation":"Update","apiVersion":"apps/v1","time":"2021-06-22T10:05:59Z","fieldsType":"FieldsV1","fieldsV1":{"f:metadata":{"f:annotations":{".":{},"f:deprecated.daemonset.template.generation":{}},"f:generateName":{},"f:labels":{".":{},"f:app":{}},"f:ownerReferences":{".":{},"k:{\"uid\":\"5d80e18a-7319-4ae4-ae0f-a97beddf17c2\"}":{".":{},"f:apiVersion":{},"f:blockOwnerDeletion":{},"f:controller":{},"f:kind":{},"f:name":{},"f:uid":{}}}},"f:spec":{"f:revisionHistoryLimit":{},"f:selector":{},"f:template":{"f:metadata":{"f:labels":{".":{},"f:app":{}}},"f:spec":{"f:containers":{"k:{\"name\":\"intel-gpu-plugin\"}":{".":{},"f:args":{},"f:env":{".":{},"k:{\"name\":\"NODE_NAME\"}":{".":{},"f:name":{},"f:valueFrom":{".":{},"f:fieldRef":{".":{},"f:apiVersion":{},"f:fieldPath":{}}}}},"f:image":{},"f:imagePullPolicy":{},"f:name":{},"f:resources":{},"f:securityContext":{".":{},"f:readOnlyRootFilesystem":{}},"f:terminationMessagePath":{},"f:terminationMessagePolicy":{},"f:volumeMounts":{".":{},"k:{\"mountPath\":\"/dev/dri\"}":{".":{},"f:mountPath":{},"f:name":{},"f:readOnly":{}},"k:{\"mountPath\":\"/sys/class/drm\"}":{".":{},"f:mountPath":{},"f:name":{},"f:readOnly":{}},"k:{\"mountPath\":\"/var/lib/kubelet/device-plugins\"}":{".":{},"f:mountPath":{},"f:name":{}}}}},"f:dnsPolicy":{},"f:initContainers":{".":{},"k:{\"name\":\"intel-gpu-initcontainer\"}":{".":{},"f:image":{},"f:imagePullPolicy":{},"f:name":{},"f:resources":{},"f:securityContext":{".":{},"f:readOnlyRootFilesystem":{}},"f:terminationMessagePath":{},"f:terminationMessagePolicy":{},"f:volumeMounts":{".":{},"k:{\"mountPath\":\"/etc/kubernetes/node-feature-discovery/source.d/\"}":{".":{},"f:mountPath":{},"f:name":{}}}}},"f:nodeSelector":{".":{},"f:kubernetes.io/arch":{}},"f:restartPolicy":{},"f:schedulerName":{},"f:securityContext":{},"f:terminationGracePeriodSeconds":{},"f:volumes":{".":{},"k:{\"name\":\"devfs\"}":{".":{},"f:hostPath":{".":{},"f:path":{},"f:type":{}},"f:name":{}},"k:{\"name\":\"kubeletsockets\"}":{".":{},"f:hostPath":{".":{},"f:path":{},"f:type":{}},"f:name":{}},"k:{\"name\":\"nfd-source-hooks\"}":{".":{},"f:hostPath":{".":{},"f:path":{},"f:type":{}},"f:name":{}},"k:{\"name\":\"sysfs\"}":{".":{},"f:hostPath":{".":{},"f:path":{},"f:type":{}},"f:name":{}}}}},"f:updateStrategy":{"f:rollingUpdate":{".":{},"f:maxUnavailable":{}},"f:type":{}}}}},{"manager":"kube-controller-manager","operation":"Update","apiVersion":"apps/v1","time":"2021-06-22T10:15:01Z","fieldsType":"FieldsV1","fieldsV1":{"f:status":{"f:currentNumberScheduled":{},"f:desiredNumberScheduled":{},"f:numberAvailable":{},"f:numberReady":{},"f:observedGeneration":{},"f:updatedNumberScheduled":{}}}}]},"spec":{"selector":{"matchLabels":{"app":"intel-gpu-plugin"}},"template":{"metadata":{"creationTimestamp":null,"labels":{"app":"intel-gpu-plugin"}},"spec":{"volumes":[{"name":"devfs","hostPath":{"path":"/dev/dri","type":""}},{"name":"sysfs","hostPath":{"path":"/sys/class/drm","type":""}},{"name":"kubeletsockets","hostPath":{"path":"/var/lib/kubelet/device-plugins","type":""}},{"name":"nfd-source-hooks","hostPath":{"path":"/etc/kubernetes/node-feature-discovery/source.d/","type":"DirectoryOrCreate"}}],"initContainers":[{"name":"intel-gpu-initcontainer","image":"registry.local/intel-gpu-initcontainer:1.20.5","resources":{},"volumeMounts":[{"name":"nfd-source-hooks","mountPath":"/etc/kubernetes/node-feature-discovery/source.d/"}],"imagePullPolicy":"IfNotPresent","securityContext":{"readOnlyRootFilesystem":true}}],"containers":[{"name":"intel-gpu-plugin","image":"registry.local/intel-gpu-plugin:1.20.5","args":["-v","2","-shared-dev-num","300"],"env":[{"name":"NODE_NAME","valueFrom":{"fieldRef":{"apiVersion":"v1","fieldPath":"spec.nodeName"}}}],"resources":{},"volumeMounts":[{"name":"devfs","readOnly":true,"mountPath":"/dev/dri"},{"name":"sysfs","readOnly":true,"mountPath":"/sys/class/drm"},{"name":"kubeletsockets","mountPath":"/var/lib/kubelet/device-plugins"}],"terminationMessagePath":"/dev/termination-log","terminationMessagePolicy":"File","imagePullPolicy":"IfNotPresent","securityContext":{"readOnlyRootFilesystem":true}}],"restartPolicy":"Always","terminationGracePeriodSeconds":30,"dnsPolicy":"ClusterFirst","nodeSelector":{"kubernetes.io/arch":"amd64"},"securityContext":{},"schedulerName":"default-scheduler"}},"updateStrategy":{"type":"RollingUpdate","rollingUpdate":{"maxUnavailable":1}},"revisionHistoryLimit":10},"status":{"currentNumberScheduled":1,"numberMisscheduled":0,"desiredNumberScheduled":1,"numberReady":1,"observedGeneration":34,"updatedNumberScheduled":1,"numberAvailable":1}} E0622 10:15:07.773496       1 controller.go:302] intel-device-plugins-manager/controller/gpudeviceplugin "msg"="Reconciler error" "error"="Operation cannot be fulfilled on daemonsets.apps \"intel-gpu-plugin-fzn2t\": the object has been modified; please apply your changes to the latest version and try again" "name"="intel-gpu-plugin" "namespace"="" "reconciler group"="deviceplugin.intel.com" "reconciler kind"="GpuDevicePlugin"
It may take many changes to the custom resource to be executed in order for the problem to arise, this does not happen every time. I was fiddling with the shared-dev-num arg and went back-and-forth between two values.
Further rounds of the reconciler will successfully update the object, as the object is always refreshed. Thus this is mostly a cosmetic issue, but it could perhaps be avoided by using other update mechanisms such as patching.

Not happening anymore -> closing.


cmd/cue: generate Go code from CUE

Originally opened by @mpvl in https://github.com/cuelang/cue/issues/6
Or really any programming language, but Go is a good first target.

Generate Go structs (with comments) from CUE definitions
Generate validation code given the CUE constraints defined on these definitions (well, its the same thing in CUE, but for Go it needs to be split out)

I have started playing with cue and I like it, my idea was to first describe the schema of my inputs as a cue file and from there use it to generate to go struct and then validate the input & fill the go structure but I cannot manage to generate go struct from my cue code.
My first resource was https://cuelang.org/docs/integrations/go/#generate-go-code which is not that helpful since I have no clue what instance is and how to produce it, I made a few attempts but they all fail and there is no documentation about this I could find.
I like the concept of cue but it generated so much frustration that I think I will keep away from it for now until it matures.

This is still a desired feature. It is unclear how the general concept would look with multiple language targets. For example, how does one express OOP vs Go type relations, or relations in the context of databases. What about public vs private visibility? The rules are quite varied across the languages and many have concepts that are not representable in CUE without at least attributes.
I have been using text/template with schemas. It is a little more verbose, i.e. not using CUE's builtin constraints, you have to specify them in the schema.
Some links to what I have been doing:

https://cuetorials.com/first-steps/generate-all-the-things/
https://github.com/hofstadter-io/hof (CUE powered code gen tool)
https://github.com/hofstadter-io/hofmod-server ( hof gen for a Golang server, including types )

In the doc you referenced, an instance is a cue.Value. The example will generate a function to validate an existing Go type based on a CUE value. It is likely not what you are looking for if you want to generate Go types directly from CUE values.

Thanks, I will have a look but if this is not currently supported out of the box it should be mentioned explicitly in the documentation, the documentation page I linked is not that.