Refactor checks about pointers in structs

Fixes #468

Move the check about pointers in structs to only happen for kernel
arguments
Add a new check to prevent recursive struct definitions
New and updated tests

@jrprice I think this should do the trick without being too lax, can you give it a try?

Yes, this works for me, thanks.
Maybe worth adding tests that cover the cases we want to work? I can only see tests that cover the error cases at the moment. The two that would be relevant for the Arm Compute Library are:
struct T {
  global int* ptr;
};

void bar(struct T* t) {
  *t->ptr = 42;
}

kernel void foo(global int* out) {
  struct T t;
  t.ptr = out;
  bar(&t);
}

and
struct T {
  global int* ptr;
};

struct T bar(global int* out) {
  struct T t;
  t.ptr = out;
  return t;
}

kernel void foo(global int* out) {
  struct T t = bar(out);
  t.ptr = 42;
}


Implement automatic Bodhi update notes

Related to https://github.com/packit/packit-service/issues/1518.
Merge after https://github.com/packit/specfile/pull/113.
RELEASE NOTES BEGIN
Packit is now able to generate automatic Bodhi update notes including a changelog diff since the latest stable build of a package.
RELEASE NOTES END

I'm going to merge this manually. Builds and tests fail because https://github.com/packit/specfile/pull/113 hasn't been released yet.

/packit build

/packit-stg build

recheck

definitely needs a rebase :)

definitely needs a rebase :)

Yes, I've just rebased it and I'm going to merge it if the tests pass.


Add interaction tests

Change List

Add InteractionTestRunner to @deck.gl/test-utils
Browser-based interaction tests

Coverage remained the same at 83.036% when pulling 9a4429482f9861d2c99d2176edc786d4821568bd on x/interaction-tests into 82078de8f6fa3111c108a3ecfade028559005307 on master.


assign custom font on the editor does not work

Hi,
Working with text objects right now, noticed I can't assign custom fonts to the text objects on the editor, so now have to initialize the texts and setup fonts by script, can't we have a field to either type the font name as a string or drop the resource font there from the assets tab, like with textures for example?
Version: Desktop 0.95
OS: Windows 10 x64 bit
thanks!

You cannot use the fonts imported to the editor with CSS. I really need to improve the flux with HTML/CSS im currently working on a HTML layer with CSS3D for nunuStudio but still needs a lot of work.


Make verification config truly optional

Depends on: https://github.com/kubewarden/policy-evaluator/pull/108
Fixes: https://github.com/kubewarden/policy-server/issues/220

I just realized I forgot to do a follow up once I merged the last batch of sigstore-related code into policy-server...
I would simply drop the --enable-verification flag and just enable it when the user provides a verification file. I think that would simplify this PR and also the conversation about the default value for the verification file

As an aside, I think the sigstore unit tests take too long for being unit. I would consider them as integration and separate the running of unit tests and integration tests.
I can follow up on that later if you are fine. No rush at all with that, just an idea.

Yes, that's because they are downloading the Sigstore data from a TUF repository. I think there's a way to disable them during the regular runs. We could then create a cron job that runs them at least once a day. I would do that with another PR however

I've created https://github.com/kubewarden/policy-server/issues/225 to keep track of this improvement

I've created https://github.com/kubewarden/policy-server/issues/225 to keep track of this improvement

Thank you 🙌

Can we skip the verification if the --verification-path is not provided? I don't think it make sense to do the verification with the default values (all_of and any_of None)

Yes, this is already the case as we never get through https://github.com/ereslibre/policy-server/blob/b30382c4d837e2e17f0c3538ded18c4499bbbc26/src/policy_downloader.rs#L82 if validation is disabled.

That being said, I think that loop could be converted to three loops (N -> 3*N, but still, I think it's worth it). 1 loop) download policies, 2 loop) sigstore verification, 3 loop) verify integrity of policies.

and 3) might be merged together on the same loop. The 2) loop would be skipped if verification is disabled.

Can we skip the verification if the --verification-path is not provided? I don't think it make sense to do the verification with the default values (all_of and any_of None)

Yes, this is already the case as we never get through https://github.com/ereslibre/policy-server/blob/b30382c4d837e2e17f0c3538ded18c4499bbbc26/src/policy_downloader.rs#L82 if validation is disabled.

ah I see, I got confused with the verification_config default value. But that happens after creating the Downloader, so it is fine.


Improving the footer of the post.

Description
Here in the post where we can comment, it takes most of the space and the white box doesn't look good , I would like to change it and make it a simple underline where the placeholder value will be add a comment.
I would also like to add alignment to the emoji icon , comment and Post button
@narayansoni could i work on this UI enhancement? under the gssoc.
Screenshots

Additional information
No response

Yes I wish to work on this issue @narayan954

great go ahead !

I'd like to work upon this issue if this hasn't been done yet.Pls assign it to me

I'd like to work upon this issue if this hasn't been done yet.Pls assign it to me

Hi @bhoomikhurana it's already assigned, you may check other issues for now or come back if it remains unsolved after 1 week

This issue has been inactive for a long time
I would like to work on this issue
Please assign

This issue has been inactive for a long time
I would like to work on this issue
Please assign

yes please go ahead and I'd like to update the issue a bit... save/favorites button could be on the right.


How can i run it in a realtime image

How can i run it in a realtime image

@stiglioglu We don't have camera APIs, if that is what you want.
I think your use case is similar to a video input? We have a video-inference-visualize pipeline:
https://github.com/voldemortX/pytorch-auto-drive/blob/master/docs/VISUALIZATION.md#segmentation-mask-video

Thank you for your answer. I want to run on ros camera topic. How can I run on snapshot eg image from webcam.

The general method is read the image, make the testing transforms (e.g., resize), then inference, finally apply specific post-processings. For some lane detection methods, they already have a inference() method, which includes the post-process.
An lane example is https://github.com/voldemortX/pytorch-auto-drive/blob/9ec8172481b6d38201bdc2308caadc8a5feb02f5/utils/runners/lane_det_visualizer.py#L153

where do you read the video frames can you help me i couldn't find it

where do you read the video frames can you help me i couldn't find it

We use a torch style dataloader for video reading:
https://github.com/voldemortX/pytorch-auto-drive/blob/master/utils/datasets/video.py


Bug report: unable to remove SharePoint group site

Priority
(Urgent) I can't use the CLI
Description
While removing some useless sites from my tenant, I keep getting an error that the site cannot be removed.

Steps to reproduce
Run spo site remove --url https://contoso.sharepoint.com/sites/Sample on a GROUP#0 site.
Expected results
Both the group & the SharePoint site to be in the recycle bin.
Actual results
Site group still exists in the deleted groups. The site won't be removed.
Diagnostics
Retrieving the group Id of the site https://thejasey.sharepoint.com/sites/GMTest_11cXkpAiV9
Removing Microsoft 365 Group: 7f0fc87c-2c76-4c33-b717-6e7d0f1e6cdb...
Deleting site https://thejasey.sharepoint.com/sites/GMTest_11cXkpAiV9...
Site group doesn't exist. Searching in the Microsoft 365 deleted groups.
Error: Site group still exists in the deleted groups. The site won't be removed.
CLI for Microsoft 365 version
v7.5.0
nodejs version
v20.10.0
Operating system (environment)
Windows
Shell
PowerShell
cli doctor
No response
Additional Info
No response

Yeah, unfortunately this has been known quite a while now... we just haven't gotten round to fixing it:
https://github.com/pnp/cli-microsoft365/issues/5218

Aw, in that case, let's keep that one.


presentations: add ci-operator end-to-end tests

Small addition to the internals page, extracted from the presentation: https://deploy-preview-296--zealous-shirley-0f56b3.netlify.app/docs/architecture/ci-operator-internals/#image-pipeline.

*  External link https://vault.ci.openshift.org/ failed: […] SSL connect error

/retest


Permission denied

Hi,
After a long time of usage without any problem, I finally restarted the running container (after modifying the host hardware), now the image doesn't want to start any more on my previous volume.
btool.log shows the following
08-29-2016 09:18:07.764 ERROR IniFile - Cannot open file=/opt/splunk/etc/system/local/server.conf for parsing: Permission denied
08-29-2016 09:18:07.766 ERROR UsernameMapper - Cannot create username mapping file: /opt/splunk/etc/users/users.ini: Permission denied
08-29-2016 09:18:07.766 ERROR IniFile - Cannot open file=/opt/splunk/etc/users/users.ini for parsing: Permission denied
08-29-2016 09:18:07.766 ERROR UsernameMapper - Error opening username mapping file: /opt/splunk/etc/users/users.ini
08-29-2016 09:18:07.773 ERROR ConfObjectManagerDB - Cannot initialize: /opt/splunk/etc/apps/launcher/metadata/local.meta: Permission denied
08-29-2016 09:18:07.773 ERROR ConfObjectManagerDB - Cannot initialize: /opt/splunk/etc/apps/search/metadata/local.meta: Permission denied
08-29-2016 09:18:07.773 ERROR ConfObjectManagerDB - Cannot initialize: /opt/splunk/etc/apps/splunk_management_console/metadata/local.meta: Permission denied
08-29-2016 09:18:07.773 ERROR ConfObjectManagerDB - Cannot initialize: /opt/splunk/etc/apps/user-prefs/metadata/local.meta: Permission denied
08-29-2016 09:18:07.776 ERROR ConfObjectManagerDB - Cannot initialize: /opt/splunk/etc/system/metadata/local.meta: Permission denied
08-29-2016 09:18:07.776 ERROR ConfObjectManagerDB - Cannot initialize: /opt/splunk/etc/apps/learned/metadata/local.meta: Permission denied
The user rights of the folder look like the following (in the host, then user are not displayed as splunk:splunk) :

I tried to boot the image on a new volume, it seems the problem doesn't occur.
Do you have any idea how I can fix that ?
Thanks !

Ok issue fixed by doing a recusrive chmod on folder. I don't know why the permissions changed.

Possible that you have executed something or tried to restart splunk not with the default user we use in the container (splunk), this is why permission messed up


Fix readme

glide install will not add a dependency to the glide.yaml file.
glide install --help
NAME:
   glide install - Install a project's dependencies

USAGE:
   glide install [command options] [arguments...]

DESCRIPTION:
   This uses the native VCS of each package to install
   the appropriate version. There are two ways a project's dependencies can
   be installed. When there is a glide.yaml file defining the dependencies but
   no lock file (glide.lock) the dependencies are installed using the "update"
   command and a glide.lock file is generated pinning all dependencies. If a
   glide.lock file is already present the dependencies are installed or updated
   from the lock file.

OPTIONS:
   --force             If there was a change in the repo or VCS switch to new one. Warning: changes will be lost.
   --strip-vendor, -v  Removes nested vendor and Godeps/_workspace directories.
   --skip-test         Resolve dependencies in test files.

glide get --help
NAME:
   glide get - Install one or more packages into `vendor/` and add dependency to glide.yaml.

USAGE:
   glide get [command options] [arguments...]

DESCRIPTION:
   Gets one or more package (like 'go get') and then adds that file
   to the glide.yaml file. Multiple package names can be specified on one line.

       $ glide get github.com/Masterminds/cookoo/web

   The above will install the project github.com/Masterminds/cookoo and add
   the subpackage 'web'.

   If a fetched dependency has a glide.yaml file, configuration from Godep,
   GPM, GOM, or GB Glide that configuration will be used to find the dependencies
   and versions to fetch. If those are not available the dependent packages will
   be fetched as either a version specified elsewhere or the latest version.

   When adding a new dependency Glide will perform an update to work out the
   the versions for the dependencies of this dependency (transitive ones). This
   will generate an updated glide.lock file with specific locked versions to use.

   The '--strip-vendor' flag will remove any nested 'vendor' folders and
   'Godeps/_workspace' folders after an update (along with undoing any Godep
   import rewriting). Note, The Godeps specific functionality is deprecated and
   will be removed when most Godeps users have migrated to using the vendor
   folder.

OPTIONS:
   --test                   Add test dependencies.
   --insecure               Use http:// rather than https:// to retrieve packages.
   --no-recursive, --quick  Disable updating dependencies' dependencies.
   --force                  If there was a change in the repo or VCS switch to new one. Warning, changes will be lost.
   --all-dependencies       This will resolve all dependencies for all packages, not just those directly used.
   --resolve-current        Resolve dependencies for only the current system rather than all build modes.
   --strip-vendor, -v       Removes nested vendor and Godeps/_workspace directories.
   --non-interactive        Disable interactive prompts.
   --skip-test              Resolve dependencies in test files.

This change is

@caesarxuchao ping

LGTM. Thanks @ash2k. @ericchiang @lavalamp FYI.


International rankings

Reminder to make sure international rankings work

some to check:

[ ] https://staging.datacommons.org/ranking/Amount_EconomicActivity_GrossNationalIncome_PurchasingPowerParity_PerCapita/Country/asia?h=country%2FMYS&unit=%24
[ ] https://staging.datacommons.org/ranking/LifeExpectancy_Person/Country?h=country%2FMYS
[ ] https://staging.datacommons.org/ranking/Amount_Emissions_CarbonDioxide_PerCapita/Country/asia?h=country%2FCHN
[ ] https://staging.datacommons.org/ranking/CumulativeCount_MedicalConditionIncident_COVID_19_PatientDeceased/Country?pc

those are all fixed!


Retry if the reference objects are not properly label with backup

Signed-off-by: Mike Ng<EMAIL_ADDRESS>for https://github.com/stolostron/backlog/issues/22512

/hold
needs approval

/unhold

/lgtm


Add Pundit boring generator

First of all, I don't have any previous experience with Rails generators (this is also a reason, why I wanted to try it).
As you can see, I tried to proceed with the same, but I found a few issues.

I was forced to add Bundler.with_unbundled_env to be able to install the Pundit gem.
~I use the same way to skip tasks, but it does not work (even in tests).~ fixed
I was forced to run the rails generate pundit:install before I include it in the ApplicationController (it would throw uninitialized constant ApplicationController::Pundit). ~I can see (overlooked) that it still does not work.~ fixed

I would be very glad if you can look at it and help me fix the tests.
I've manually tested it and it works.
➜ rails generate boring:pundit:install
Running via Spring preloader in process 21740
Adding Pundit gem
         run  bundle add pundit from "."
.
.
.
Running Pundit Generator
         run  DISABLE_SPRING=1 rails generate pundit:install from "."
      create  app/policies/application_policy.rb
Adding Pundit module into ApplicationController
      insert  app/controllers/application_controller.rb
Force ensuring policies
      insert  app/controllers/application_controller.rb
Add rescue from Pundit::NotAuthorizedError
      insert  app/controllers/application_controller.rb

Please check the `application_controller.rb` file and fix any potential issues

Don't forget, that you can generate policies with
rails g pundit:policy Model

@abhaynikam Thank you! I updated it but I still have issues with one test. It looks like that it ignores the bundle add pundit or it is adding it somewhere else 🤷♂
Could you please look at it? I hope it will take you less time to find the bug there. There must be something a little bit different from the real environment. I was able to easily install it on my test app.
If you don't have time, I will try to find time during the week. I am not happy that I did not "finished" it properly.

Could you please look at it? I hope it will take you less time to find the bug there. There must be something a little bit different from the real environment. I was able to easily install it on my test app.

No worries @CiTroNaK. I'll take a look at it. Thanks for the awesome work ❤

Fixed the issue with the test in master. The template app used for test was using rails 6.1-alpha which was failing in bundle.
Thank you for your contribution 🎉 . Please feel free to take up other features if you like.


amperefaraday test failing on SciPy 1.14.0

The amperefaraday physics test is failing when run with SciPy version 1.14.0 or later (which, for example, the automated tests on GitHub are). The call to interp2d should be replaced with appropriate use of the RectBivariateSpline instead.

Apparently the problem was not in the amperefaraday test, but rather in the routine which evaluates the Braams-Karney conductivity (which is used by the amperefaraday test).


support mvariable dump by prometheus rpc service

support mvarible dump by prometheus rpc service

LGTM
@serverglen 有空也看看

LGTM

LGTM

Thanks for your contributions!


[chip,dv] Correct a testplan item

Similar to commit 25b8f54
The testpoint was given a "tests" field that was supposed to be the "bazel" field.

Thanks for your PR @elliotb-lowrisc.
Sorry for missing it, but I fixed this same issue on this PR

Ah well, at least it got done


Algebra Solver

Summary:
This related to issue #2023  and the pending pull request #2023 . Expose the algebraic function. I'm raising the issue now, because I do not expect the signature for algebra_solver to change. Given this is a higher-order function, I will follow the scheme used for the ODE solvers.
Description:
Allows users to solve systems of algebraic equations, given an initial guess.
Additional Information
If we move to C++11, we could change the signature, notably by removing the x_r and x_i arguments. I expect we'll however still need the parms argument to calculate the Jacobian.
Current Version:
v2.15.0

Following the example of the ODE integrators, I got a working prototype. However, I can't get rid of the argument std::string& solver_name, which is equivalent to std::string& integration_function_name, in the constructor of the algebra_solver structure (.../lang/ast/node/algebra_solver_def.hpp). Since there is only one solver, I do not need this argument, but I get a bug if I don't include it.
It is not a error at compilation, but when I run the test, all the arguments are "misplaced" -- pushed to the left, as though an additional argument were constructed at the beginning. Which prompts several error messages from the parser.
Can I view how the parser attempts to call the function? As far as I can tell, no hpp file gets produced if the parsing fails.

Is there a public branch where I can look at what you've done?
The problem is that if the grammar rule doesn't fire, you won't get any output.  You can instrument the grammar rules themselves with print statements, but it's a huge pain because it requires semantic actions (it'll get easier with lambdas in C++11).  You need to make sure the parser children (right hand side of rules) and the semantic coercion that maps the sequence of terminals into the parent category match.

@bob-carpenter thanks for the pointers. The branch I used is feature/issue-2300-algebra_solver.

Did you make sure to synch the fusion adaptor declarations in term_grammar_def.hpp?
BOOST_FUSION_ADAPT_STRUCT(stan::lang::algebra_solver,
                           (std::string, solver_name_)
                           (std::string, system_function_name_)
                           (stan::lang::expression, x_)
                           (stan::lang::expression, y_)
                           (stan::lang::expression, dat_)
                           (stan::lang::expression, dat_int_) )

BOOST_FUSION_ADAPT_STRUCT(stan::lang::algebra_solver_control,
                           (std::string, solver_name_)
                           (std::string, system_function_name_)
                           (stan::lang::expression, x_)
                           (stan::lang::expression, y_)
                           (stan::lang::expression, dat_)
                           (stan::lang::expression, dat_int_)
                           (stan::lang::expression, rel_tol_)
                           (stan::lang::expression, fun_tol_)
                           (stan::lang::expression, max_num_steps_) )

the rule in the same file,
     algebra_solver_control_r
        %= (string("algebra_solver") >> no_skip[!char_("a-zA-Z0-9_")])
        >> lit('(')              // >> allows backtracking to non-control
        >> identifier_r          // 1) system function name (function only)
        >> lit(',')

and the AST in lang/ast/node/algebra_solver.hpp:
    struct algebra_solver {

      /**
       * Name of solver.
       */
       std::string solver_name_;

If you do, I don't see a problem with getting rid of the name.
On the other hand, you should probably leave it in to cover the condition where we add new solvers as we did with ODE integrators.

@bob-carpenter : I edited the files you pointed out (in addition to algebra_solver_def.hpp and expression_visgen.hpp), but this produces the error I mentioned in my previous post. Namely, for the following call to the function:
functions {
  vector algebra_system (vector x,
                         vector y,
                         real[] dat,
                         int[] dat_int) {
    vector[2] f_x;
    f_x[1] = x[1] - y[1];
    f_x[2] = x[2] - y[2];
    return f_x;
  }
}

data {
}

transformed data {
  vector[2] x;
  vector[2] y;
  real dat[0];
  int dat_int[0];
  vector[2] theta;

  theta = algebra_solver(algebra_system, x, y, dat, dat_int);
}

The error message is:
C++ exception with description "PROGRAM ERROR, MESSAGE(S) FROM PARSER:
first argument to algebra_solver must be the name of a function with signature (vector, vector, real[], int[]) : vector second argument to algebra_solver must have type vector for initial guess; found type = ill formed. fourth argument to algebra_solver must have type real[] for real data; found type = vector. fifth argument to algebra_solver must have type int[] for integer data; found type = real[].
" thrown in the test body.

My guess is the first argument (the name of the function, which would have type "ill formed") gets passed as the second argument, the second as the third, etc. So indeed the fourth argument is read in as the third argument, and is a vector instead of a real[], and the fifth argument a real[] instead of an int[]. It is as though a "rogue" first argument, which I gather corresponds to the name of the function, still got passed.
The key is to find where this rogue argument gets passed. That was why I wanted to see what the code parser actually wrote.
Note: I only made changes to the algebraic_solver (not algebraic_solver_control). The changes, which cause the bug, are on GitHub.

The issue is probably in the test condition or signature here. The alg_fun must be getting passed because you're finding alg_fun.system_function_name_ rather than failing to compile.  Are you sure the name you're getting for that is the name of a function with the right signature?  You can grab the available function signatures and print them.
You're trying to find a function that returns a vector, taking (vector, vector, real[], int[]) arguments.
    template <class T>
    void validate_algebra_solver_non_control_args(const T& alg_fun,
                                                  const variable_map& var_map,
                                                  bool& pass,
                                                  std::ostream& error_msgs) {
      pass = true;
      expr_type sys_result_type(VECTOR_T, 0);
      std::vector<expr_type> sys_arg_types;
      sys_arg_types.push_back(expr_type(VECTOR_T, 0));  // x
      sys_arg_types.push_back(expr_type(VECTOR_T, 0));  // y
      sys_arg_types.push_back(expr_type(DOUBLE_T, 1));  // dat
      sys_arg_types.push_back(expr_type(INT_T, 1));  // dat_int
      function_signature_t system_signature(sys_result_type, sys_arg_types);
      if (!function_signatures::instance()
          .is_defined(alg_fun.system_function_name_, system_signature)) {
        error_msgs << "first argument to "
                   << "algebra_solver"
                   << " must be the name of a function with signature"
                   << " (vector, vector, real[], int[]) : vector ";
        pass = false;
      }

Are you sure the name you're getting for that is the name of a function with the right signature?

Yes, the function that gets passed has the right signature. However, the first element of alg_fun doesn't seem to be the name of that function, rather the name of the solver ("algebra_solver") -- I do not know this for sure, this is based on the "rogue argument" theory.
In term_grammar_def.hpp, what do the following blocks of code do:
(1)
algebra_solver_control_r
        %= (string("algebra_solver") >> no_skip[!char_("a-zA-Z0-9_")])
        > ...

In particular what is the no_skip[!char_("a-zA-Z0-9_")] for ?
(2)
      factor_r.name("expression");
      factor_r =
        integrate_ode_control_r(_r1)[assign_lhs_f(_val, _1)]
        | integrate_ode_r(_r1)[assign_lhs_f(_val, _1)]
        | algebra_solver_control_r(_r1)[assign_lhs_f(_val, _1)]
        | algebra_solver_r(_r1)[assign_lhs_f(_val, _1)]
        | (fun_r(_r1)[assign_lhs_f(_b, _1)]

Is it possible they determine the first element of alg_fun. For the ODE integrator, what determines the first element of integration_function_name?

You probably want lit("algebra_solver") rather than string("algebra_solver").  With the latter, the string itself gets passed as a semantic argument.  That could be leading to an extra argument.
Unpacking no_skip[!char_("a-zA-Z0-9_")]:

no_skip prohbits whitespace before matching the argument

! causes failure if the argument is matched

char_(...) matches any single character specified in the argument

a-zA-Z0-9_ matches any character in the range a to z, A to Z, and 0 to 9, and also matches a single underscore

Together, it stops you from finding any characters after the match;  e.g., algebra_solverJUNK would not allow algebra_solver to match.

You probably want lit("algebra_solver") rather than string("algebra_solver"). With the latter, the string itself gets passed as a semantic argument. That could be leading to an extra argument.

Yep, that was the culprit. Well, that takes care of the grammar files :) I'll write the "bad model" unit tests and submit a pull-request.

In parser_generator_test.cpp: As far as I can tell, the test looks at the models under test-models/good/parser-generator/. For each model there, there is an accompanying .hpp file, which corresponds to the C++ translation of the stan file. Should this hpp file be supplied manually, or should the test create it?
I'm asking because just putting the .stan file didn't work, but I fail to see the point of the test if we supply the .hpp file manually.

make automatically parses necessary .stan files into .hpp files
before compiling and running the tests.
See https://github.com/stan-dev/stan/blob/develop/make/tests https://github.com/stan-dev/stan/blob/develop/make/tests.

On Jun 26, 2017, at 10:00 AM, Charles Margossian<EMAIL_ADDRESS>wrote:
In parser_generator_test.cpp: As far as I can tell, the test looks at the models under test-models/good/parser-generator/. For each model there, there is an accompanying .hpp file, which corresponds to the C++ translation of the stan file. Should this hpp file be supplied manually, or should the test create it?
I'm asking because just putting the .stan file didn't work, but I fail to see the point of the test if we supply the .hpp file manually.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/stan-dev/stan/issues/2300#issuecomment-311067756, or mute the thread https://github.com/notifications/unsubscribe-auth/ABdNlmBkJJbQ4TRIu8BrqONbyzYPsDZIks5sH7l_gaJpZM4NRORq.

I think for most of these cases, the .hpp is generated as a string by the compiler and then tested against expected output.  See the actual tests.

Thank you for the explanation and the pointers. Looks like I had an error in my model, which why the .hpp file wasn't created. I opened a pull-request, but for the automatic tests to pass, we'll first need to add the algebraic solver to the math repo (pull-request #516).

The link is to the math repo, so you need the full path, https://github.com/stan-dev/math/pull/516

@charlesm93 can you add to the stan-reference documentation which algorithm is used in the solver and a reference to the which library is used? Couldn't find these in the latest develop branch documentation.

I'm closing this issue.  If I'm doing this in error @charlesm93, please re-open.

@bob-carpenter yup, all good to close.


feat(pixels): add my new pixel (28, 39)

Checklist

[x] I ran npm test locally and it passed without errors.
[x] I only edited the _data/pixels.json file.
[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

Hey @dkundel this has been done here #677


[SPARK-14225][SQL] Cap the length of toCommentSafeString at 128 chars

What changes were proposed in this pull request?
Builds on https://github.com/apache/spark/pull/12022 and (a) appends "..." to truncated comment strings and (b) fixes indentation in lines after the commented strings if they happen to have a (, {, ) or }
How was this patch tested?
Manually examined the generated code.

cc @rxin

Test build #54455 has started for PR 12044 at commit 2e7fdb9.

Test build #54460 has started for PR 12044 at commit 8464149.

Test build #54466 has started for PR 12044 at commit 978e932.

LGTM pending jenkins.

Test build #54460 has finished for PR 12044 at commit 8464149.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54460/
Test PASSed.

Test build #54466 has finished for PR 12044 at commit 978e932.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54466/
Test PASSed.

I've merged this. Thanks.


WIP - refactoring containers and header

Try out Leather build b5d3bd7 — Extension build, Test report, Storybook, Chromatic

This is a POC of how we can refactor our code to make it easier to add headers to the approval flow.
In the screenshot below you can see a double header:

one in a ContainerLayout wrapper
one in the Approver component

This is just a POC. I'm thinking about how best to do this and also to include solving https://github.com/leather-wallet/extension/issues/5181


Make sure summed probs indexes match alts

We weren't removing the choosers layer of the summed probs MultiIndex when probability_mode='single_chooser', so the summed probs in that mode would have a weird MultiIndex.
Fixed it to drop the chooser ID layer of the summed probs index when in "single chooser" mode.

Giving it a try!

Changes Unknown when pulling 43905f55c4b96ee27077b353caf6ebf63a7b5cd4 on summed-probs-fix into ** on master**.

Any luck?

Yes, it seems to be working ;)


Query: get-block-at-era-switch

Query chain for switch block information against a node's JSON-RPC API's chain_get_block endpoint.  Requires either polling until a switch block is detected, or consuming event stream and awaiting next switch block.

Implemented in issue/83


Fix errors in saddle-point handling

I have seen the "Failed to connect to last point" warning. I trust you on the fix ;)


Fix project.pbxproj file name and edit the xcode dependency to be backwards compatible.

When we parse the project file in parseProjectFile we should pass project.pbxproj instead of project.pbxproject.
Also since there is a breaking change in the xcode dependency - pbxNativeTarget is renamed to pbxNativeTargetSection we should edit our local copy to be backwards compatible by adding pbxNativeTarget method.

👍

LGTM :+1:


Error on step 2 of section: Delete an Azure Databricks service

Ref: Delete an Azure Databricks service
In the above referenced document, for going to Delete button, step 2 reads: Click your username in the top bar of the workspace, and select Manage Account from the drop down.. But this step does not take you to the page that has Delete button. You don't actually go to the workspace to delete Azure Databricks service.
Step 2 should read as: In Azure Portal, search for your Databricks workspace by typing Databricks. In the search results click on your Databricks workspace name.
Step3: In the Azure Databricks service page, click Delete and then OK.

@srazasyed
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

@AjayBathini-MSFT Per your request, I have added the reference of the document on the first line of the original post above. Thank you.

@srazasyed
I did not find any Microsoft doc link in the post. Can you share in the comment section. So, it helps me to sort out the issue you are facing.
Thank you!

@AjayBathini-MSFT I may not have clicked the UPDATE button on my post above. I re-entered the first line of the post that references the document you requested the link for. I'm adding it here, as well

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

Hi @srazasyed
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.

@srazasyed
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


ATM Data Generator not emitting right transactions

Hi Robin,
I'm following your demo. I got issue with Gess. As you are using UK ATM transaction data. I have downloaded new Zealand ATM data from open street map. As per instruction I have generate csv file and then I copied atmnz_export.csv under data directory and update gess.conf file accordingly.
Restart the container and verity gess is emitting NZ ATM transaction data by this command
nc -v -u -l 6900

but when I tried push same transaction in new topic using following command
nc -v -u -l 6900 | \
docker run --interactive --rm --network ksqlatmfrauddetection_default confluentinc/cp-kafkacat \
kafkacat -b kafka:29092 -P -t atm_txns_akl

I got UK ATM transaction in topic. If I stop and rerun the same command it start pushing NZ data. it is not consistent.
I don't know how its picking UK ATM transaction data even I have deleted all files from gess data directory but it still emitting UK ATM data.  Any thoughts ?

How are you running gess? From the docker image referenced in the docker-compose? or standalone?

Thanks for reply. I'm using docker-compose to setup everything. I have copied my csv file in docker container and update conf file as well and restart the container. Tot verify the change if I run the nc-v -I 6900 I can see NZ ATM transaction but if stop and rerun the same command its start showing UK ATM transaction. Even if I run second command mentioned above to push message in new topic sometime it show NZ and sometime UK one. I'm facing another issue, if I use UK file to generate ATM transaction and push new message in same topic ATM_TXNS_GESS I can see new transaction data in KSQL but its not being replicated in Elasticsearch. I have check the status of JDBC connector which is  up and running but somehow it's not working. Sorry for dump questions I have recently start playing with ELK and kafka.

I'd probably look at building my own version of the gess image, there's the source repo and Dockerfile here: https://github.com/rmoff/gess/blob/master/Dockerfile


Pr scmsite

This pull request switches from using the github site plugin to maven-scm-publish-plugin as there was no current way to upload the whole site without triggering the abuse mechanisms
The scm-publish strategy relies on git scm provider which also has a problem for large changes as it tries to "git add" all changed files in one command line and this is way too big.
The end result is to use site-stage to push each individual projects site to one "global" directory (currently found using session.executionRootDirectory) and scm-publish immediately after each site is staged. This minimises the number of files changed on each operation. A final site stage and scm-publish is called in order to pickup any aggregate reports that would only have been completed at the end of the site build (cobertura aggregate)
As I was interested in how the cobertura coverage was change (and why), Ive made the site upload to a folder called ${project.version} and disabled any deletion of file through scm-publish. This should mean you can maintain a site at gh-pages without the maven site overwriting it.
The use of the git provider, means authentication is still done with the github token but it is recommended to add the following environment variables to travis:
GIT_AUTHOR_NAME
GIT_COMMITTER_NAME
and these with secure settings applied so they dont show in the logs:
GIT_AUTHOR_EMAIL
GIT_COMMITTER_EMAIL
The email settings are used by github to determine what is displayed on github.
Git provider versions before 1.9.4 showed the whole token in the logs, so the version is set as a dependency with a fixed version
Ive added a required environment variable (SITE_BRANCH_NAME) that designates which branch should initiate the site build - this was a convenience method for me to test outside the main project. It should also mean no site will be built until you set this in travis to the appropriate branch
Ive also added a profile to turn on verbose logging on pitest (off by default)
There is an example of the complete build including site deploy with this branch at:
https://travis-ci.org/abcops/jmxtrans/builds/64401278
and the built site can be seen at:
http://abcops.github.io/jmxtrans/251-SNAPSHOT/
Im not sure what will be produced when a non-feature branch is built.

[ ]   File: jmxtrans-output/jmxtrans-output-core/pom.xml:L41-51
 Why did you add this dependency to commons-logging? It seems that there is no code using it directly...
[ ]   File: pom.xml:L652-696
 Why using version range instead of fixed version number ? I'm not an expert, but it seems to me that [1.9.4] is equivalent to just 1.9.4. Did I miss something ?
[ ]   File: pom.xml:L780-805
 Why do we need those excludes ? There might be a good reason, in which case an inline comment would be welcomed.
[ ]   File: .travis.yml:L6-13
 Not sure what this $SITE_BRANCH_NAME variable is. I would expect to publish maven site only for master, as feature branches typically contain code that is in progress, so should probably not be published (we do not publish snapshots to Sonatype either). I also do not see where this variable is set.
[ ]   File: travis-script.sh:L40-47
 Why do you need to install the artifacts from a feature branch ? Is it related to the $SITE_BRANCH_NAME variable used in .travis.yml ? Is there a requirement to have the artifacts deployed locally to create the maven site ? If that's the case, we need to decide if we want to publish maven site for feature branches.
[x]   General Comment
 Sorry for not reacting before. This looks good! I need to find some quite time to setup the correct variables and merge this...
Thanks a lot!
 Thanks a lot ! This looks good ! The aggregated reports is also something I wanted to do for a long time, but never took the time to actually do it.

[x]   File: pom.xml:L1066-1076
 pitest now has a report goal ! Great !
:+1:

This pull request is now managed with CodeReviewHub. Refresh.

Sorry for not reacting before. This looks good! I need to find some quite time to setup the correct variables and merge this...
Thanks a lot!

Thanks a lot ! This looks good ! The aggregated reports is also something I wanted to do for a long time, but never took the time to actually do it.

General question: do you think it is a good idea to extract GIT_AUTHOR_NAME and other such variables from the last git commit (which should identify the person who merged the code) or do you think we should configure a static user / committer ?

@zonArt If you still have too much time at night, this PR would welcome some of your love...


How onGestureStart, onGestureEnd and onPointSelected work?!

Could someone please edit the documentation or explain how to make it work?
I spent 3 days and still can't figure it out. The person who made this package is not cool.
const Test = ({ setPriceTitle }) => {
  const currentPrice = 0;

  const onGestureStart = useCallback(() => {
    hapticFeedback("impactLight");
  }, []);

  const onGestureEnd = useCallback(() => {
    setPriceTitle(currentPrice);
  }, [currentPrice, setPriceTitle]);

  const onPointSelected = useCallback((p) => {
    setPriceTitle(p);
  }, []);

  return (
    <LineGraph
      points={somePoints}
      color={someColors}
      style={someStyles}
      animated={true}
      onGestureStart={onGestureStart}
      onPointSelected={onPointSelected}
      onGestureEnd={onGestureEnd}
    />
  );
};

export default Test;

What exactly is not clear? onPointSelected takes a callback and passes the current point. From there you can set it via state and do whatever you want


Removing trailing whitespace in some files.

This PR removes trailing whitespace in some files to add clarity to an imminent PR related to #60, #63, and #51.

Thanks @pshriwise


Genotype filtering training labels and cutoff optimization

Adds MakeGqRecalibratorTrainingSetFromPacBio workflow, which generates consensus training labels from Vapor and PacBio calls (pbsv, pav, sniffles). These labels are stored as a json file that is consumed by CrossValidateGqRecalibrator as the truth_json input for training a GqRecalibrator model. This json can also be consumed by FilterGenotypes optimizing SL cutoffs (see below).
Incorporates missing scripts from FilterGenotypes needed for SL cutoff optimization. By providing the truth_json instead of sl_filter_args input, the workflow will automatically generate and apply genotype filtering cutoffs that optimize sensitivity and precision, specifically by maximizing the F-score score for each variant class (e.g. small DEL, medium DEL, INS, etc.). The F-score beta parameter is specified by fmax_beta, which is now set to a default of 0.4 that conferred a reasonable balance of sensitivity and precision across 6 different cohorts. This enables users to QC and potentially improve call set quality when training labels are available.
Workflows were tested on the "hgdp" cohort using the AoU filtering model and yielded results consistent with previous runs, with 8,720 SV/genome median and 5.5% de novo rate. Output files can be found in gs://gatk-sv-hgdp/mw-train-genotype-filtering/.

Note this also includes some dockerfile changes that were necessary for rebuilds, including #646.


WIP milestones

We utilize milestones pretty heavily internally and have had to build one off utilities in order to be able to do it automatically, it'd be great if we could just do this in hub.
The goal is basically to provide functionality for all of the API calls found at https://developer.github.com/v3/issues/milestones/
Relates to:

https://github.com/github/hub/issues/1343

Work to be done:

Add commands to:

[ ] create milestones
[ ] list milestones
[ ] update milestones
[ ] delete milestones
[ ] get a single milestone

UX still needs to be fleshed out but I this right now is just a starting point
If this functionality is not desired let me know and I'll happily close the PR.
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>

Hi, thanks for proposing and starting work on this! While I'm certainly feeling confident that you can fully flesh out this feature, milestones is not something we want to invest in hub right now, since there are many other features on our roadmap that take priority. Because every added feature requires further maintenance for many years to come, lately I'm being conservative with adding new features lately. See https://github.com/github/hub/issues/1593#issuecomment-336528950


Need to get "define" for enum type

<struct name="SimpleStruct">
        <cluster code="0x050F"/>
        <item name="a" type="UINT8U" optional="false"/>
        <item name="b" type="BOOLEAN" optional="false"/>
        <item name="c" define="SimpleEnum" type="ENUM8" optional="false"/>

DROP TABLE IF EXISTS "STRUCT_ITEM";
CREATE TABLE IF NOT EXISTS "STRUCT_ITEM" (
  "STRUCT_REF" integer,
  "FIELD_IDENTIFIER" integer,
  "NAME" text,
  "TYPE" text,
  "IS_ARRAY" integer,
  "MIN_LENGTH" integer,
  "MAX_LENGTH" integer,
  "IS_WRITABLE" integer,
  foreign key (STRUCT_REF) references STRUCT(STRUCT_ID)
);

looks like currently zap don't support struct item with define ?
for enum type, we would like to get "define"
then I can write  {{asType label}} = static_cast<{{define}}>(v);

<struct name="SimpleStruct">
        <cluster code="0x050F"/>
        <item name="a" type="UINT8U" optional="false"/>
        <item name="b" type="BOOLEAN" optional="false"/>
        <item name="c" define="SimpleEnum" type="ENUM8" optional="false"/>

DROP TABLE IF EXISTS "STRUCT_ITEM";
CREATE TABLE IF NOT EXISTS "STRUCT_ITEM" (
  "STRUCT_REF" integer,
  "FIELD_IDENTIFIER" integer,
  "NAME" text,
  "TYPE" text,
  "IS_ARRAY" integer,
  "MIN_LENGTH" integer,
  "MAX_LENGTH" integer,
  "IS_WRITABLE" integer,
  foreign key (STRUCT_REF) references STRUCT(STRUCT_ID)
);

looks like currently zap don't support struct item with define ?
for enum type, we would like to get "define"
then I can write  {{asType label}} = static_cast<{{define}}>(v);

An alternate way to do this with the existing version can be {{asType label}} = static_cast<{{as_delimited_macro name}}>(v);

it does not work as expected if using {{asType label}} = static_cast<{{as_delimited_macro name}}>(v), which returns             c = static_cast(v);

Could you give me the example template on how you are using this?

I'm a bit confused with this, @yunhanw-google :
why would we not simply do:
<item name="c"  type="SimpleEnum" optional="false"/>

instead of:
<item name="c" define="SimpleEnum" type="ENUM8" optional="false"/>

Could you give me the example template on how you are using this?

        {{#zcl_struct_items}}
        case k{{asType label}}FieldId:
            {{#if (isEnum type) }}
            uint8_t v;
            ReturnErrorOnFailure(ClusterObjectUtils::Decode(reader, v));
            //zap issue 231 to return right enum type
            {{asType label}} = static_cast<{{define}}>(v);
            break;
            {{else if isArray}}
            return CHIP_ERROR_NOT_IMPLEMENTED;
            {{else}}
            ReturnErrorOnFailure(ClusterObjectUtils::Decode(reader, {{asType label}}));
            break;
            {{/if}}
        {{/zcl_struct_items}}

another alternative is that, we add one additional flag, isEnum similar to isArray for struct item

There is this helper which may be helpful.
{{#if_is_enum type}}
type is enum
{{else}}
type is not enum
{{/if_is_enum}}
Regarding  "static_cast<{{as_delimited_macro name}}>(v), which returns c = static_cast(v);" I am not sure why this happened. Is this for a particular struct item or all of them?

There is this helper which may be helpful.
{{#if_is_enum type}}
type is enum
{{else}}
type is not enum
{{/if_is_enum}}

Just tried, it cannot detect SimpleEnum is enum.
The issue here is that the internal zap don't have knowledge about the customized enum type currently. Either zap know how to parse define, or parse isEnum or some other way zap get knowledge from xml to know it is enum

So if I do this, does this meet your meeds:
1.) Add the IS_ENUM field in the DB.
2.) Allow for isEnum="true" in the XML.
3.) load and populate these flags.
4.) Give you a helper is_enum_type or something like that that will allow to modify how these types are generated.

Closed in PR 235


question or feature request

I like your window manager it does everything I need.   The only thing is i want to change from using the ALT key to using the WIN key for all functions (including move/resize).   When i change the keycode from 64 to 133, i can get the WIN key to launch my programs however it will not move and resize windows.   I changed the XCB_MOD_MASK_1 to XCB_MOD_MASK_4 but still didn't work.   I guess i don't understand how you are grabbing the WIN key in the code.   I can make it all work using the CONTROL key however I prefer to use the WIN key as it leaves the CNTRL and ALT available to other programs.
Can you explain how you are grabbing the WIN key in your code?  and how I might disable the maximize/delete behavior and use it fully in place of the ALT key?

Move xcb_grab_button out of the loop & only grab WIN modifier
If you don't care for fullscreen/close click behavior, update
case 1:goto*(my==XCB_MOD_MASK_1?&&mvsz:&&full);
case 3:goto*(my==XCB_MOD_MASK_1?&&mvsz:&&shut);

to
case 1:case 3:goto mvsz;

Thanks for the reply!   I tried as you suggested but the move/resize function did not work.
How is the best way to only grab the WIN modifier?

xcb_grab_button(d,1,rt,XCB_EVENT_MASK_BUTTON_PRESS,XCB_GRAB_MODE_ASYNC,XCB_GRAB_MODE_ASYNC,XCB_NONE,XCB_NONE,XCB_GRAB_ANY,XCB_MOD_MASK_4);

Thanks! that worked.  And thanks for the great minimal window manager!


[DOC] Developer guide (extension guidelines and scitype templates)

From my comments in #420:
I think we very much need to write up template designs, type conventions, extension conventions. This information is typically not suitably covered by an auto-generated API reference, as it is distributed across classes - the generic template, and the conventions implemented in the concrete desendants. Sth like in sklearn dev guidelines could work, but I'd actually prefer to also have "formal" template definitions collected in one place. While the typing makes things clearer, it also makes the necessity to document the interface cleanly greater.
Opinions, @mloning, @TonyBagnall?
I'm not sure what the "final form" should be here, but I feel this needs to be manually written and curated, with any modification to template patterns and conventions.
Could live on the wiki, linked from user docs, or directly in the official docs.
Happy to contribute to this, but let's discuss first where it should live.

Hi @utsavcoding, this may be interesting to you too

Hi @fkiraly, I agree, we need better extension guidelines. I think we should write a "developer guide" in addition to the planned "user guide" and the scitype template should be part of that.
It should live in the docs. The wiki isn't very good for suggesting and reviewing changes collaboratively. I'm trying to slowly move away from the wiki for most things.
We should merge in the user guide (#377) first though, even with only some basic information and place holders, that way other people can see it and may be interested in contributing. Having a work-in-progress warning at the top of the page should be fine. Similarly, I'm happy to merge in a basic structure of the developer guide. I believe some sections from CONTRIBUTING.md should eventually also be moved into the developer guide.

@mloning, writing on the extension guidelines.
Quick question on forecasting extension pattern.
Can you confirm the following:
To implement a new forecaster, one should inherit from _SktimeForecaster and precisely one of the two [...]ForecastingHorizonMixin-s.
The extender needs to implement:

fit, writing to self
_predict (not predict!), reading from self
update (if the model can be run in update mode)
update_predict (if the model can be run in update mode)

Optional implements:

compute_pred_int for prediction intervals
_update_predict_single for single-step update/predict
get_fitted_params for fitted parameter inspection

noting that some of these methods are implemented in _SktimeForecaster and will hence not raise NotImplementedError?

@mloning, @big-o, can you please confirm?
Also pinging @aiwalter who went through the extension workflow recently (is the above what you did in #503?).

I would say it's better to implement _compute_pred_err() instead for prediction intervals. compute_pred_int() is just a wrapper for the private method that does some input validation and generic output formatting.

@fkiraly I have another issue that would be great to add in the developer guide here: https://www.sktime.org/en/latest/contributing.html#id18
When adding a new softdependency, the file check_soft_dependencies.py for the azure build has to be updatet accordingly.

@mloning, forgot to ping the draft here:
https://github.com/alan-turing-institute/sktime/wiki/Scitype-interface-specification:-forecaster
You gave me feedback earlier and said that conceptual model & specs should be preceded by a quick overview.
I've added that now - have a look. Is that useful?

@aiwalter, @big-o, feel free to look as well - I haven't added much detail about _SktimeForecaster yet because of #510.
On a side note: what I think is often find very helpful is a template class in the "code template" sense - i.e., an example class where all that is missing is the core logic, and comment blocks tell you what to fill in. Do we have sth like that?

Thanks @fkiraly, in terms of exposition, I'd would have a very concise description at the top, perhaps even a code skeleton, and then discuss key points in more detail below.
For example:
class MyForecaster(BaseForecaster, <forecasting-horizon-mixin>):

    def __init__(self, <hyper-parameters>):
        ...

    def fit(self, y, X=None, fh=None):
        ...

@mloning, would the following help:

create the "fill-in code template" as suggested - I can take a look at this if you suggest the best current python file to "convert". the NaiveForecaster?
use the code template to create the preamble.

Regarding concise description: I added the checklist which I thought would satisfy that requirement. Do you not think this is suitable as such? If not, why is this not good?

Yes, the NaiveForecaster should be fine but bear in mind that it inherits from the BaseWindowForecaster, so best to make the inherited functionality explicit if relevant for the code skeleton.
Yes, the check list is suitable for the concise description!

Yes, the check list is suitable for the concise description!

Hm, so why did you say it was missing - did you not see it?
I can add a template code block if you want. E.g., example template and checklist.

I didn't mean to say it was missing, my comment mainly referred to the structure rather than content. Do you want to create a PR to include this on the website? I can set up the structure for the developer guide, similar to the user guide.

sure, can make a PR once there is a location where to.
Also, please note that some parts are still missing, e.g., on input check conventions. I can give it a try, but I don't know what these actually are.


How do I install Ubuntu?

Hello,
How do I download and install on Ubuntu?
Thank you!

You can install through Node.js, which runs on Ubuntu:
https://github.com/photonstorm/phaser#bower--npm
Or you could simply download the phaser.js file from the cdn.

I compiled it.
How do I open the application?

See this tutorial: http://phaser.io/tutorials/getting-started

Ahhh, thanks !!


security.wrappers should check if file exists

Issue description
The suid wrappers configured via security.wrappers are not checking if a referenced file exists which leads to:
$ kcheckpass
kcheckpass: cannot run `/nix/store/gdfndzkmz221lnhfkfa1z7c2mdr0yqc7-plasma-workspace-5.13.5-bin/lib/libexec/kcheckpass': No such file or directory

For the case above, it's fixed in #47838 but we really should assert on file presence.
Technical details

system: "x86_64-linux"
host os: Linux 4.18.10, NixOS, 19.03.git.1e7c1a82c6cM (Koi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.2
channels(root): "nixos-18.03pre121670.4cbbecc85db"

I'll get around to it.

keep

Implemented in https://github.com/NixOS/nixpkgs/pull/126289


SMA

Note,
without the alpha, tensorflow throws an error with inf tensor

I did not see this error before using SMA? why the alpha fix this issue?
The CI fails because of the format.


STS pod stuck pending until deleted

Version

Karpenter Version: v0.22.1

Kubernetes Version: v1.22.16-eks-ffeb93d
Expected Behavior
Karpenter should provision a new node so that the pod is able to be scheduled.
Actual Behavior
Karpenter is ignoring the pod all together. There's no karpenter events associated with the pod, and restarting Karpenter doesn't help—the pod must be deleted.
Steps to Reproduce the Problem
I haven't been able to narrow down the exact reproduction yet, but it seems to only be affecting STS and we've experienced it a half dozen or so times over the past 2-3 weeks. I'm hoping that the logs provided can indicate the smoking gun causing this.
The only thing that stood out to me is that the pod had nominatedNodeName: ip-10-0-45-138.ap-northeast-3.compute.internal in its status, but that node no longer existed.
Along with this issue https://github.com/aws/karpenter/pull/1051, it seemed like the most likely cause.
Resource Specs and Logs
The stuck pod was 27h old at the time this was grabbed:
Pod events:
Events:
  Type     Reason            Age                    From               Message
  ----     ------            ----                   ----               -------
  Warning  FailedScheduling  37m (x38 over 80m)     default-scheduler  0/59 nodes are available: 2 Insufficient memory, 3 Insufficient cpu, 56 node(s) didn't match Pod's node affinity/selector.
  Warning  FailedScheduling  28m                    default-scheduler  0/59 nodes are available: 1 node(s) had taint {node.kubernetes.io/unreachable: }, that the pod didn't tolerate, 2 Insufficient memory, 3 Insufficient cpu, 55 node(s) didn't match Pod's node affinity/selector.
  Warning  FailedScheduling  13m                    default-scheduler  0/44 nodes are available: 2 Insufficient memory, 3 Insufficient cpu, 41 node(s) didn't match Pod's node affinity/selector.
  Warning  FailedScheduling  8m40s (x64 over 26h)   default-scheduler  (combined from similar events): 0/39 nodes are available: 2 Insufficient memory, 3 Insufficient cpu, 36 node(s) didn't match Pod's node affinity/selector.
  Warning  FailedScheduling  2m40s (x2 over 4m10s)  default-scheduler  0/37 nodes are available: 1 node(s) had taint {node.kubernetes.io/unreachable: }, that the pod didn't tolerate, 2 Insufficient memory, 3 Insufficient cpu, 33 node(s) didn't match Pod's node affinity/selector.

Pod conditions / status:
Status:               Pending
Conditions:
  Type           Status
  PodScheduled   False

Long form status:
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2023-02-17T12:51:34Z"
    message: '0/35 nodes are available: 2 Insufficient memory, 3 Insufficient cpu,
      32 node(s) didn''t match Pod''s node affinity/selector.'
    reason: Unschedulable
    status: "False"
    type: PodScheduled
  nominatedNodeName: ip-10-0-45-138.ap-northeast-3.compute.internal
  phase: Pending
  qosClass: Guaranteed

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Can you show the pod metadata? It might be stuck deleting.

Some additional context:
Karpenter consolidation is enabled in this cluster, and this pod was running at some point (before stuck Pending). It seems like there's a weird interplay between this STS that has a priority class higher than other services, karpenter consolidation, and this service's podAntiAffinity which forbids replicas from running on the same node.
In this stuck state, there's only 2 nodes that the STS is allowed to run on (via node selector), but a target of 3 replicas. So during consolidation Karpenter got rid of one essential node (probably the pod priority trying to evict other stuff on a node that Karpenter chose for deprovisioning).

Browsing through the karpenter-core code seems to indicate this could happen in a multi-pass consolidation scenario.
Since simulateScheduling in pkg/controllers/deprovisioning/helpers.go uses provisioner.GetPendingPods (which ignores nodes with nominatedNodeName, I think the reproduction would be the following:

Start with a cluster in steady state: STS with 3 replicas and podAntiAffinity forbidding co-location. There are four non-homogeneous nodes in the cluster because other deployment workloads exist.
An unrelated deployment workload is rolled and uses maxSurge: 1, maxUnavailable: 0, resulting in final pod placements being a bit different.
Karpenter decides it can consolidate a node (due to the moved pods).
Karpenter begins deprovisioning a node that contain a replica of the STS.
The Kubernetes scheduler nominates one of the other nodes (3 remaining that are Ready, 1 cordoned/draining) to get the STS workload w/ a high priority class back online.
The STS pod does not initialize quickly enough, and a second pass of Karpenter consolidation now thinks it can consolidate again since it was excluded from the provisioner.GetPendingPods results.
Karpenter drains the node that the previously interrupted STS pod was nominated to move to, and we're left in the OP reported state.

Looking at pod & node ages, this theory seems plausible:

I've seen these STS pods sometimes be delayed to initialize due to the EBS CSI driver struggling to keep up with node rotations (~4-5 minute delay, i.e. longer than it takes a new node to become Ready).

Thanks for giving this all a read! I'll continue to try to come up with a more concrete reproduction if y'all think it'd be helpful.

FWIW seems like the stuck nominatedNodeName was a bug https://github.com/kubernetes/kubernetes/issues/85677. It was fixed in 1.24 and backported to 1.23 (nominatedNodeName), but our clusters are on 1.22.

Karpenter is ignoring the pod since it has a nominated node name so we expect it to schedule. We trust the scheduler to be correct, and in this case it's not. If you can upgrade to v1.23, that should resolve it.

Would it be reasonable to have consolidation ignore the presence nominatedNodeName? It feels like pending pods should prevent consolidation from occurring.. but perhaps that's another discussion.
While unrelated to this OP issue, we've also seen Karpenter consolidate & have to re-provision nodes JIT for pods stuck in CrashLoopBackoff.
It feels like having "to be scheduled" pods considered in (preventing) consolidation is reasonable.

Should I test if Kubernetes can get itself unstuck by manually adding a node (but not manually clearing nominatedNodeName / deleting pod)?

I looked at a few options for solving this, but none of them are great and open up possibilities for other problems (either under-provisioning or over-provisioning).  Just ignoring nominated node name when provisioning can cause over-provisioning during pod evictions.
We don't treat CrashLoopBackoff pods any differently that I'm aware of.  If you run into issues with this, file another issue and include Karpenter logs.

Thanks for all the time spent looking at this.
I'll close this since we've narrowed down the issue to be a Kubernetes bug (& without a great way for Karpenter to work around it).


Fix parser crash when parsing a single #

Fixes #44829
Fixes #45114

#45114 seems related

Oh yeah I forgot Daniel opened a new issue when he figured out the repro

Also didn’t see @a-tarasyuk’s fix, which is better than this one.


Correcting photo subtitles

I just noticed now that all photos in the AWS Tutorial are with the same subtitles.

Docs for 3fcc7b82726adba13ef8c44d4f4c36e48dbd6617 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/448/docs/artifacts/html/index.html

Docs for ea4817b1b5c3935d4936f8133ec4744b5ffb5321 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/449/docs/artifacts/html/index.html

Docs for 877f8be8188a8d90dfac296a3c3dc81466e38912 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/450/docs/artifacts/html/index.html

Docs for e7ef3be76b8cd083961d9e542b8be9e2a1856bc2 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/452/docs/artifacts/html/index.html

Docs for 7dc55c57a214b110b179160174a0d9d0db6345c5 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/453/docs/artifacts/html/index.html

LGTM, can you rebase this on master?

Docs for 74c9cb2a6ecdec1ee7b54d11ed752cb0eebb5652 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/489/docs/artifacts/html/index.html


Image size in RST documents

When viewing the RST docs in GitHub, screenshots are often too small to see without clicking on them.  We can remedy this by removing the :width and :height directives.  We may also want to improve any PNG by trimming it vertically to only the interesting parts.  Lastly, it would be good to run optipng on any png to make it smaller without affecting how it looks.

The original width and height directives may have helped generate PDFs from the rst documents.  Since most people will probably view them online, this is probably not desired.


Ability to shrink yourself in the CM dimension

Basically like morph, shrink to less than one block size.
Might be a good upgrade once the upgrade system arrives?

that would be great, especially on servers that dont have morph. Could also be something that works outside of a CM using the power of the PSD perhaps?

Currently working on a 1.10.2 version, the 1.7.10 version is not being maintained anymore, sorry.
Closing all issues not relevant for the new version.


[stable/datadog] ensure init scripts run in correct order

Signed-off-by: Joe Hohertz<EMAIL_ADDRESS>@hkaj @irabinovitch @CharlyF @mfpierre @clamoriniere @xlucas
What this PR does / why we need it:
The init-config container is running a list of scripts prefixed with numbers indicating the order in which they should run. The invocation of these scripts however is NOT sorting the list of scripts to run, resulting in non-deterministic behaviour.
This was found when we tried switching our OS image, and found different behaviour on different images, and was traced down to this flaw.
IE: an example where we modified it to also print the scripts as it run, yielded this before (meanting an empty datadog.yaml was created initially, where this is meant to only happen if all prior steps did not create one):
$ kubectl -n monitoring logs datadog-5klhk init-config
/etc/cont-init.d/01-check-apikey.sh
/etc/cont-init.d/59-defaults.sh
/etc/cont-init.d/60-network-check.sh
/etc/cont-init.d/50-cri.sh
/etc/cont-init.d/50-ecs.sh
/etc/cont-init.d/50-kubernetes.sh
Disabling the apiserver check as leader election is disabled
/etc/cont-init.d/89-copy-customfiles.sh
/etc/cont-init.d/50-mesos.sh
/etc/cont-init.d/51-docker.sh

And this fix results in the correct order:
$ kubectl -n monitoring logs datadog-hgj6g init-config
/etc/cont-init.d/01-check-apikey.sh
/etc/cont-init.d/50-cri.sh
/etc/cont-init.d/50-ecs.sh
/etc/cont-init.d/50-kubernetes.sh
Disabling the apiserver check as leader election is disabled
/etc/cont-init.d/50-mesos.sh
/etc/cont-init.d/51-docker.sh
/etc/cont-init.d/59-defaults.sh
/etc/cont-init.d/60-network-check.sh
/etc/cont-init.d/89-copy-customfiles.sh

Which issue this PR fixes
Unknown if any existing PRs are impacted by this.
Special notes for your reviewer:
None
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

Re-based and re-bumped the chart version

/retest

/lgtm

@jhohertz thanks for the fix! 👍


New version: RealNeuralNetworks v1.0.1

Registering package: RealNeuralNetworks
Repository: https://github.com/seung-lab/RealNeuralNetworks.jl
Created by: @jingpengw
Version: v1.0.1
Commit: 02f6bf33b3d09bec2244acd5d45ea2fae91752ec
Reviewed by: @jingpengw
Reference: https://github.com/seung-lab/RealNeuralNetworks.jl/issues/73#issuecomment-524911367

yep. there is a conflict with my tags.
Jingpeng Wu
Postdoc in Princeton Neuroscience Institute, Princeton University, NJ
website: jingpengw.com
On Mon, Aug 26, 2019 at 3:46 PM Stefan Karpinski<EMAIL_ADDRESS>wrote:

Is skipping version 1.0.0 intentional?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JuliaRegistries/General/pull/2947?email_source=notifications&email_token=AB2MB5LMR5EFVLOIVF6PBSDQGQXKRA5CNFSM4IPQ7RU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5FOV3Y#issuecomment-525003503,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB2MB5P3AVTVRRG5X6OFKWDQGQXKRANCNFSM4IPQ7RUQ
.


quartodoc build: option to build only single object (or section)?

Currently, quartodoc build will rebuild the whole site. While it is careful to only save a file if it has changed, sometimes it's helpful to just rebuild a single file (so you can tweak something and check the result quickly).
One big challenge is that the first time you run quarto preview all of the docs have to be built.

cc @cscheid, would love to discuss Friday! Basically quartodoc has a few moves it could pull in order to potentially reduce the number of pages built when quarto preview runs.
It can...

produce a sidebar config with fewer entries for the API folder (e.g. only put the function the user is currently developing on the sidebar)
Link out to a subset of doc pages from the index

But I can't remember how quarto decides which .qmd files to render. If we remove links to .qmd files, and don't include them on the sidebar / any nav, will they get rendered?

Pairing w/ Carlos, It seems that quarto preview --when re-run--is only re-rendering files that have changed.
I'll go back through the pyshiny-site build to sanity check everything, and make sure quartodoc is writing only when needed.
Note also, freeze does not control running pandoc, only knitr or ipynb, so would not help us.

From @wch "every time I make a change to the renderer and then re-build I feel like I'm going to go insane [waiting for it to re-render]"
Let's add additional CLI pos args so that you can build pages corresponding to specific objects, rather than the whole thing:
quartodoc build ui.nav ui.page_nav --verbose

Note that I had to use a --filter flag, since quartodoc build can optionally accept the name of a _quarto.yml file as its first argument.

--filter should work in v0.3

https://machow.github.io/quartodoc/get-started/basic-building.html#selectively-building-doc-pages


Update forms.md

When select is multiple it does not return array in its value property. To select all options you should check them manually.
Possible solution is:
handleChange(event) {
this.setState(
{
value: Array.from(event.target.children).filter(
children => children.selected
).map(
children => children.value
)
}
);
}

Hello @AndryBethpalko!
As the examples in the latest of react.dev, included here contain the advise on how to read and format the received values I will close this PR as outdated.
Thank you!


Bridge client connection manager

Summary
Implement connection manager for bridge client. See issue comment https://github.com/movementlabsxyz/movement/pull/153#discussion_r1684345723
Changelog
Newest commits

Attempted to configure with deadpool, getting trait bound errors
Update: Liam has mentioned deadpool is not safe to use. So probably need to go back to the drawing board.

Old commits:

Add ConnectionManager struct
Add ManageConnection trait for ConnectionManager
Add async runtime with tokio
Debugging syntax errors (work in progress)

Testing
Currently getting various errors mainly involving mismatch of lifetimes and methods in bb8. In the process of debugging.
Outstanding issues
Tbd, still debugging this PR

Closing this because @0xmovses and @cde8 have implemented a much more elegant solution taking advantage of the fact that the needed clients already implement Clone.


shell: Fix uart_rx_handle byte processing when ring buffer full

In case when shell ring buffer gets full the data may still be taken
from the UART fifo, byte by byte, and accepted by the SMP back-end,
if the back-end has been enabled.
Unfortunately the uart_rx_handle failed to check if it has been
successfull in reading a byte from the fifo, before passing it to
the SMP for processing, which could lead to processing random stack data
as a part of an SMP frame.
Additinally, when the SMP would have accepted the byte,
the uart_rx_handle would fail to signal shell_thread, that the SMP has
internally buffered data that could be processed.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>

@mniestroj Thank you for comments and review!

I merged it in my head with #27657
LGTM

@nvlsianpu Thanks for reviewing!


[Runtime DS] Add REST API Design for API Rate Limits

Purpose

Add the REST API design for API rate limits
Related Issue: https://github.com/wso2/apk/issues/849

Done via https://github.com/wso2/apk/pull/880


Fix no images loading causing a 0% success rate

this works locally, but I don't entirely know if it works if its merged because of how content policies work and stuff

this works locally, but I don't entirely know if it works if its merged because of how content policies work and stuff

I know nothing about coding but really like this site. Is there a way for me to apply this fix if I'm browsing on Android?

I rewrote the site and added another method of loading the images which you can use.
You can find it here: https://unlimigur.herokuapp.com
The servers aren't strong so it will fail a few requests. If it doesn't load anything refresh your page and it'll hopefully work again. I'm working on making a server less version but that has stagnated because of my partial-hatred of JavaScript and web development and the inability to get the client side stuff to work.

this works locally, but I don't entirely know if it works if its merged because of how content policies work and stuff

I know nothing about coding but really like this site. Is there a way for me to apply this fix if I'm browsing on Android?

I rewrote the site and added another method of loading the images which you can use.
You can find it here: https://unlimigur.herokuapp.com
The servers aren't strong so it will fail a few requests. If it doesn't load anything refresh your page and it'll hopefully work again. I'm working on making a server less version but that has stagnated because of my partial-hatred of JavaScript and web development and the inability to get the client side stuff to work.

Thank you.


feat: add step for unregistering data stores in arcgis server

Good catch. DDACTS had some registered folders. Did you go through and clean up after everything?

Yes, I cleaned up DDACTS, fiberverification and one other one that I can't remember at the moment. They were all dead project.


Telemetry headers

code
Make sure SMBA is logging correctly

Requests made to botbuilder team:
https://github.com/microsoft/botbuilder-js/issues/4552
https://github.com/microsoft/botbuilder-dotnet/issues/6701
https://github.com/microsoft/botbuilder-python/issues/2033

currently following up with SMBA on our side to make sure they can ingest custom headers we set. BF team should be able to start this work in the JS SDK early next year.


!uqfinal improvements

Now gives average scores needed when more than one assessment missing.
Accepts "%" in scores.
Gives requirements for all passing grades.
Reiterates inputs for both decimals and percentages.
Names final assessment.
Lists assessment if no marks given.

Also, the handle method desperately needs to be broken up into helper methods, if you're looking for things to do.


Modules with Unknown IO Widths Inferred Incorrectly

Related to the discussion in ucb-bar/chisel3#142
If you have a module with unspecified widths and instantiate it such that the different instantiations should result in different widths, the module is inferred to have the max of the two widths. This is functionally correct but is wasteful in the hardware generated. Consider the example below.
circuit Test :
  module WidthUnknown :
    input a : UInt
    input b : UInt
    output z : UInt
    z <= add(a, b)

  module Test :
    input a : UInt<16>
    input b : UInt<16>
    input c : UInt<32>
    input d : UInt<32>
    output x : UInt
    output y : UInt

    inst foo of WidthUnknown
    foo.a <= a
    foo.b <= b
    x <= foo.z

    inst bar of WidthUnknown
    bar.a <= c
    bar.b <= d
    y <= bar.z

Which results in:
circuit Test :
  module WidthUnknown :
    input a : UInt<32>
    input b : UInt<32>
    output z : UInt<33>
    z <= add(a, b)

  module Test :
    input a : UInt<16>
    input b : UInt<16>
    input c : UInt<32>
    input d : UInt<32>
    output x : UInt<33>
    output y : UInt<33>
    inst foo of WidthUnknown
    inst bar of WidthUnknown
    foo.a <= a
    foo.b <= b
    bar.a <= c
    bar.b <= d
    x <= foo.z
    y <= bar.z

This is the intended behavior (it isn't a bug). As to whether this behavior should be changed, I don't see any alternative solutions other than duplicating the module and inferring separate widths. WidthUnknown's ports need to be inferred to something, and the only legal value (if we don't duplicate the module) is the max of the two widths.

I suppose saying "Inferred Incorrectly" was itself incorrect since this is the intended behavior.
That being said, I think this behavior kind of undermines the point of inferring the widths on modules. My understanding is that the main argument is for code reuse, but in seems that doing so with the current behavior will result in unnecessary hardware. I think in the case where a module has unspecified widths in makes perfect sense to duplicate the module, infer separately, and then recombine if possible (ie. the inferred widths match).

In any case, it doesn't match the expected behavior for Chisel, where two instances of the same module shouldn't be coupled in their implementation.  We'd have to emit different code for each instance of a module in the current FIRRTL regime, which seems to me like the wrong approach.  I'd advocate for changing the spec and duplicating modules with inferred input widths prior to width inference.

I don't know if I agree - when I think of instantiating two instances of the same module, I think its pretty intuitive that the instances should be identical. That being said, Chisel has to emit separate modules anyways for each instance because there isn't any guarantee that the module constructor, when called twice, will emit identical modules, so this really isn't an undue burden on Chisel. Similarly, there can certainly be a Firrtl pass that merges Modules if they happen to be identical. That being said, I'm not diametrically opposed to the idea of changing the spec, and you make a good point. I think it has more to do with what is Firrtl - does it allow for some generator-like behavior (i.e. generating different modules for different widths) or is it more the straight RTL structural representation (i.e. modules are more like black boxes that you stamp out and thus coupling the implementations).

I do see what you mean.
I'd still advocate for changing the spec.  The coupled implementations don't, in my opinion, follow the principle of least surprise.  You might make some subsystem that instantiates a module with inferred input widths, then down the road, instantiate that same module in an unrelated subsystem--and in so doing change the behavior of the first subsystem.  This also precludes unit-testing the individual subsystems, as the generated hardware for one differs in the absence of the other.
It's tangential, but I remain concerned about global width inference becoming a performance bottleneck.  Of course, the implementation can always take advantage of modules that happen to have fully-specified input widths and solve a local problem opportunistically.  It's just a little extra complexity.

Solved similarly to https://github.com/ucb-bar/chisel3/issues/142


Support dependencies with multiple integrity values

Fixes #34
This PR fixes the scenario where a package-lock.json file may include dependencies with multiple integrity values, as it would cause an error on the OBS side when building. The solution involves selecting the first defined integrity and checking against that, and ignoring any other one.

I've squashed these commits and merged them now. Thanks!


When using the vector database Milvus, it is not possible to use Chinese names for the knowledge base.

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
MacOS(M1, M2...)
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[X] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
CPU
macOS M2 Pro
RAM 16G
Models information
LLM_MODEL=spark_proxyllm
What happened
When creating a knowledge base, users often assume that they can input Chinese names by default. However, the Chinese name will be used as the space_name in the vector database, and Milvus does not support Chinese characters. Therefore, it is necessary to add an additional field that serves as the actual name for backend usage, instead of relying on the value entered by the user on the web page.
#1212  This commit failed to address the underlying issue.
What you expected to happen
can use chinese
How to reproduce

update .env
VECTOR_STORE_TYPE=Milvus MILVUS_URL=<IP_ADDRESS> MILVUS_PORT=19530

create  knowledge

use  chinese name

log will show  error

Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

hi, @jsRuner , what's your space name, now it support only Chinese Character,  but not support half Chinese Character and half English Character. it will be fix soon.

@Aries-ckt   Indeed. Previously, I entered 'China11' which couldn't establish a connection. Now, I entered 'China' and it works fine.


[bug]:

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
No response
What version did you experience this issue on?
2.3.5
What happened?
When installing on windows using the provided .bat file, it keeps stopping out of nowhere. I keep getting this error on each attempt:
INFO: This is taking longer than usual. You might need to provide the dependency resolver with stricter constraints to reduce runtime. See https://pip.pypa.io/warnings/backtracking for guidance. If you want to abort this run, press Ctrl + C.
I don't know why it is halting so much, but it never finishes even if I leave the computer on for days. This is attempt 5 but it feels like 500....
What can I do to make this finish installing??? I'm about ready to tear my hair out. Can I make it use only relevant versions instead of checking literally every single one exhaustively? If so, can I do that without having to edit a file that is bigger than my curriculum vitae?
I did enable long path support, even tried running the .bat file with explicit admin privileges, and I ate a taco while waiting once hoping it would bring good luck. I think my computer wants to drop kick me across the room and idk if we can make-up this time.
Screenshots

Additional context
Probably happens because, 1) everything I touch dies, 2) I don't know jack about python (running version 3.10 FYI) and 3) my tummy hurts :C
Contact Details
No response

Seems to be a problem with Python itself. I fixed the issue, not entirely sure how, but I had to uninstall EVERYTHING related to python and I reinstalled the recommended version. However, I am unable to maintain other versions of python at the moment.
Problem: On windows with all CUDA deps met, the installer .bat file stops somewhere before config prompt, and seeks out too many versions of dependencies/libraries, and freezes (with task manager showing signs of memory error.) Installer will not continue, and remain stuck. Likely as a result of conflicting python versions.
Solution: Uninstall all python versions, RESTART, re-install python (only recommended version, no others) and re-install InvokeAI and everything works as planned.
Thanks for taking the time to make this software, it will be interesting to train it and see where things go.


Orphaned Private Teams Channel

The above article says "If a private channel owner leaves your organization or if they are removed from the Office 365 group associated with the team, a member of the private channel is automatically promoted to be the private channel owner."
However, this does not appear to add an owner and I get an error trying to add users to the channel in Teams Admin, "We can't add the member(s) that were selected.". I am the only user in the private team and I have not been made owner of the private channel.

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

ID: ce34b12b-aa50-608b-bbfa-a1fb4bb79f8d
Version Independent ID: 9ddeb6f4-f30a-1e72-fc94-ae66def91f40
Content: Private channels in Microsoft Teams - Microsoft Teams
Content Source: Teams/private-channels.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@eschrader thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

You can close it if you want, but I am pointing out a bug in the documentation. This happens on multiple tenants for my clients and my demo environment.


Please implement jumpTo(index) method

The scrollTo method is doing fine, but I have a very large list and it has about 4000 items and it takes forever to scroll. also, the height of the items is dynamic so I can't use jumpTo(value) method. so is there any way that I can use jumpTo(index)?

Unfortunately, there is currently no way to implement jumpTo(index) because flutter can not get the size of widget before the widget is layout. So we can not get the offset directly. Currently, we animate to the cached item and then the ListView will cache the next more items. And that's how we implement scrollTo(index).


MultiMarkdown Cross References

CommonMark has yet to implement the popular MultiMarkdown v2+ feature, "Cross References."
One of the things I've struggled with is creating unique id's on elements, and incorporating them into Tables of Contents. I know markdown-it-anchor and markdown-it-table-of-contents exist, but I'd prefer to have the Markdown source file be a little more platform-independent.
So, when I want to create a TOC, I have to use HTML. I'm not opposed to that, but it really doesn't look as nice as Markdown.
Consider the following Markdown:
## Table of Contents

* [This Is a Header][]
* [This Is a Subhead][subhead]

# This Is a Header

This is a paragraph.

## This Is a Subhead [subhead]

This is another paragraph.

This is a [link][This Is a Header] to the header.

This is a [link][subhead] to the subhead, using an alternate reference link instead of \_the\_actual\_header\_title\_.

The resultant HTML would be:

Table of Contents

This Is a Header

This Is a Subhead

This Is a Header

This is a paragraph.

This Is a Subhead

This is another paragraph.

This is a link to the header.

This is a link to the subhead, using an alternate reference link instead of _the_actual_header_title_.

So instead of reverting to the HTML parlance of <h1 id="..., you can just use slightly modified Markdown link syntax. You can see how MultiMarkdown makes it easy to link to headers in the document---without having to insert icky HTML into the source document.
Just something I've been thinking about suggesting for a while :)

I think you need to write your suggestions at commonmark forum. This repo is for one of implemantation, not about spec.

I'm not so concerned with what CommonMark is doing---I was suggesting that markdown-it incorporate this directly.
Didn't you say in a CommonMark forum post that you're kind of doing your own thing anyways? Or did I misunderstand?

This project implements commonmark spec + allows extentions.
All suggestions about markup change should be discussed at commonmark forum & accepted to spec first. http://commonmark.org/


PEP440 version identification

Did we want to adopt the PEP440 version identification standard for ESMValTool or more of a semantic versioning approach?
PEP440 makes sense to me, and if there is consensus did we want to also consider use of versioneer?

We need a decision on this for the upcoming pre version 2 releases. For now I've used semantic versioning (i.e. with an -alpha and -beta suffix) for the projects, but this can of course easily be changed to something PEP440 compatible if that is preferred.
I'm not sure if using a tool like versioneer adds much to this project, as there are very few releases so far.

@bouweandela I'm also not particularly fussed about versioneer, just thought I'd ask... but it was sooo long ago, I can't recall even asking :wink:
With regards to PEP440 vs semantic... I don't think that there are any restrictions with regards to whether conda or PyPI (or other packaging tools that we care about?) can parse both version formats.
We've recently moved from semantic to PEP440, so if we're having a vote, I'd opt for PEP440 as a standard.
But I agree with @bouweandela. The core devs should make a call on this asap.

We had a short discussion and decided to go for PEP440.
@ESMValGroup/esmvaltool-coreteam
@bouweandela , feel free to update the project names and close this.

In that case we will use the following version names for version 2.0:
2.0a0   # First alpha release
2.0b0   # First beta release
2.0     # Final version 2.0 release

In case we need multiple alpha or beta releases for version 2.0, the number behind the a/b will be incremented

So, then this will be closed.


docs: add azure-function plugin blog

Changes:
add azure-function plugin blog

Thanks for @SylviaBABY's review! And we could use insert suggestions 😄


Rocket particles

Updated the rendering system to more easily support custom shading.
Added a simple particle system which spawns thrust particles according to the thrust level of the rocket.

Very cool looking! The effect is closer to SRB engines than upper stage H2 - LOX, but it fits nicely with our cartoony aesthetic.


Plugin Errors Propagating as segfault

As exposed in https://github.com/dylibso/modsurfer/issues/64, errors that are occurring in Extism plugins running within a Rust SDK @ v0.5.4 are propagating to the host as segfaults instead of their actual underlying error.
An Extism error can be triggered using Modsurfer as described in the above issue.

Corrupt a wasm binary (you can delete a character in the middle of the file)
Try to generate a checkfile for the corrupt binary using the modsurfer CLI: cargo run -- generate -p /path/to/corrupt.wasm

Expected:
This message sent to stdout should contain some text describing an EOF: unexpected end-of-file
Actual:
The message is:
[1]    89489 segmentation fault (core dumped)  cargo run -- generate -p ~/wasm/cowsay.instr.wasm

Keep for testing prior to 1.0 launch


Align with changes in latest ORM (unreleased)

Superseeded by https://github.com/hibernate/hibernate-reactive/pull/1644


Issue with the Latest Alpine Image

Which version and edition of Flyway are you using?
latest-alpine image
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using? (Type & version)
Postgresql-14
Which operating system are you using?
Alpine
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
The flyway migrate/repair command is broken in the latest push -> flyway/flyway:latest-alpine.
XML_SetReparseDeferralEnabled: symbol not found is the error that I'm seeing.
I think this is something related to python3.11 changes. Please take a look

Could you please provide us with more logs to help locate the source of this issue? For example, running your commands with the -X debug option and providing us with the stacktrace information

Just to follow up — does this ticket still need attention?

Hmm, maybe it is because of the STS Token that is received(for the password - would've contained spurious characters), which might not be related to flyway.
Closing this, will reopen it if I see any issue.


[lite] how to build example label_image for ios (library+binary)?

https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/examples/label_image/label_image.md

this project provide an example for ubuntu+macos+android(32+64), only lack ios(library+binary).

@laohur label_image is a command line program. You probably want to start from the iOS classification example first.

@laohur label_image is a command line program. You probably want to start from the iOS classification example first.

that means that you cannot build ios(lib) project from connand line ?

@laohur Nope, it's possible to run command line program on iOS, but it's complicated. And surely you can share C++ source code.

Here is my code:
build

cc_binary(
name = "libProphet.so",
srcs = ["Prophet.cc",],
copts = tflite_copts(),
linkopts = tflite_linkopts() + select({
"//tensorflow:android": [
"-pie",  # Android 5.0 and later supports only PIE
"-lm",  # some builtin ops, e.g., tanh, need -lm
],
"//conditions:default": [],
}),
deps = [
"//tensorflow/lite:framework",
"//tensorflow/lite/c:c_api_internal",
"//tensorflow/lite/kernels:builtin_ops",
"//tensorflow/lite:builtin_op_data",
"//tensorflow/lite/schema:schema_fbs",
],
linkshared = True,
)
.cc:
extern "C"{
int init(const char* modelBuffer,int bufferSize){
std::unique_ptrtflite::FlatBufferModel model=tflite::FlatBufferModel::VerifyAndBuildFromBuffer(_modelBuffer,modelSize);
return 0;
}
int predict(float* example, int size ){
return -1;
}
}

I just want to get a lib of android & ios.
" https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/examples/label_image/BUILD " is easy to build android library.  How about test this example?
And what is the  elegant way to build ios library ?

it quiet different between android and xcode. thanks!


DataLogger: use List<Coordinates> instead of IList<Coordinates> and document Data.RemoveRange()

DataLogger does not allow to limit its points which means its points will increase forever.
Added two functions allowing to limit by X or length manually.

Hi @Fruchtzwerg94, thanks for this suggestion! I will modify this PR to adapt a similar but alternative implementation of this functionality, and will also document its use in the cookbook 👍
// setup a logger that will grow as data is added
var logger = myPlot.Add.DataLogger();

// add ten values
logger.Add(Generate.RandomSample(10));

// remove the oldest five values
for (int i=0; i<5; i++)
{
    logger.Data.Coordinates.RemoveAt(0);
}

Hi @swharden ,
awesome, thanks for bringing it in so fast. A minor thing: Since DataLogger.HasNewData is private, it can't be set after items changed. Adding a way to set it would be nice as well. I can prepare a PR for it but I assume its faster for you just to bring in a minor change here without a PR.
Thanks,
Philipp

I can prepare a PR for it but I assume its faster for you just to bring in a minor change here without a PR

Actually if you're up for making a PR, I'd appreciate it! I can merge that from the browser (or even my phone) and don't have to keep switching branches on my local desktop setup 😄

See: #4470

How is one supposed to initialize a DataLogger with a CircularBuffer after this change? Is the CircularBuffer now obsolete?

How is one supposed to initialize a DataLogger with a CircularBuffer after this change? Is the CircularBuffer now obsolete?

This is a good question! I opened up a new issue #4507 to track discussion of this topic


nodebuilder: print node version

I'm assuming you mean log node version on start? Like as we do the network, node type etc?

Yes