cmdstan 2.18.1 installed, cannot run sample model on Windows 10

Summary:
I have successfully installed cmdstan (I think) but I tried running the sample model, and get the below error
Description:
When running bernoulli I get about 10 pages of the warnings and errors of this output, followed by the failure at the end:
Reproducible Steps:
I installed on cmdstan 2.18.1 on Windows 10 on my machine, then run the model:
make bernoulli\bernoulli
Output
In file included from :0:0:
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:5: error: stray '\2' in program
In file included from :0:0:
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:984: warning: null character(s) ignored
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:986: warning: null character(s) ignored
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:999: warning: null character(s) ignored
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:1010: warning: null character(s) ignored
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:1021: warning: null character(s) ignored
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:87:1032: warning: null character(s) ignored
In file included from :0:0:
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:1:1: error: expected unqualified-id before '!' token
!
^
In file included from :0:0:
./stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a:12:2: error: 'A' does not name a type
In file included from C:/RBuildTools/Rtools/mingw_64/x86_64-w64-mingw32/include/_mingw.h:282:0,
from C:/RBuildTools/Rtools/mingw_64/x86_64-w64-mingw32/include/crtdefs.h:10,
from C:/RBuildTools/Rtools/mingw_64/x86_64-w64-mingw32/include/stddef.h:7,
from C:/RBuildTools/Rtools/mingw_64/lib/gcc/x86_64-w64-mingw32/4.9.3/include/stddef.h:1,
from C:/RBuildTools/Rtools/mingw_64/x86_64-w64-mingw32/include/c++/cstddef:45,
from stan/lib/stan_math/lib/boost_1.66.0/boost/config/compiler/gcc.hpp:165,
from stan/lib/stan_math/lib/boost_1.66.0/boost/config.hpp:39,
from stan/lib/stan_math/lib/boost_1.66.0/boost/lexical_cast.hpp:21,
from stan/src/stan/callbacks/writer.hpp:4,
from src/cmdstan/arguments/argument.hpp:4,
from src/cmdstan/arguments/argument_parser.hpp:4,
from src/cmdstan/command.hpp:4,
from src/cmdstan/main.cpp:1:
C:/RBuildTools/Rtools/mingw_64/x86_64-w64-mingw32/include/vadefs.h:12:32: error: expected declaration before end of line
#pragma pack(push,_CRT_PACKING)
^
make: *** [make/models:16: examples\bernoulli\bernoulli] Error 1
Current Version:
v2.18.1

Do you know what version of RTools you have installed? Could you perhaps paste the full output after a make clean-all onto something like gist.github.com?

I think I had very similar error messages myself with the unit tests of stan-math under Windows... no idea what the problem was... since I can always move to a different platform I gave up. I wonder what the solution is.

I'm running the most recent version of RTools, 3.5 IIRC.
(I'll try to rerun and put the full output online tomorrow.)

Unfortunately, it has more output that windows allows me to keep in the cmd window.  The very first line is that examples\bernoulli\bernoulli "doesn't match the target pattern" (I scrolled up when it started so I could see the message. Windows is weird.)
Here's the output I have: https://gist.github.com/davidmanheim/44b02610106b976b3097a4e1c69e3d37
If there is a way to suppress some output, I can do that and paste the results.  (I tried make -s and it changes nothing.)

Those are all super weird... If you get a chance, could you try redirecting the output to a file? Basically something like
make clean-all
make examples/bernoulli/bernoulli >out.txt 2>&1

Right. Apologies for somehow forgetting how to do operating systems - 🤷♂.
The gist linked above is now updated with the full output.

I couldn't run the example too. (no error messages during the installation 2.18.1)
(Windows 10, RTools 3.5, cmdstan 2.18.1)
I got exactly the same output as @davidmanheim but for the command (no exe)
make examples\bernoulli\bernoulli
But when I added .exe (which we should do for Windows)
I got this instead
D:\Data-Work\Programs\cmdstan-2.18.1>make examples\bernoulli\bernoulli.exe
--- Translating Stan model to C++ code ---
bin''stanc.exe  examples\bernoulli\bernoulli.stan --o=examples\bernoulli\bernoulli.hpp
Model name=bernoulli_model
Input file=examples\bernoulli\bernoulli.stan
Output file=examples\bernoulli\bernoulli.hpp
--- Linking C++ model ---
g++ -Wall -I . -isystem stan/lib/stan_math/lib/eigen_3.3.3 -isystem stan/lib/stan_math/lib/boost_1.66.0 -isystem stan/lib/stan_math/lib/sundials_3.1.0/include -std=c++1y -DBOOST_RESULT_OF_USE_TR1 -DBOOST_NO_DECLTYPE -DBOOST_DISABLE_ASSERTS -DBOOST_PHOENIX_NO_VARIADIC_EXPRESSION -Wno-unused-function -Wno-uninitialized -I src -isystem stan/src -isystem stan/lib/stan_math/ -DFUSION_MAX_VECTOR_SIZE=12 -Wno-unused-local-typedefs -DEIGEN_NO_DEBUG -m64 -Wno-unused-but-set-variable -DNO_FPRINTF_OUTPUT -pipe   src/cmdstan/main.cpp -O3 -o examples\bernoulli\bernoulli.exe -include examples\bernoulli\bernoulli.hpp -include examples\bernoulli\USER_HEADER.hpp stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_nvecserial.a stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_cvodes.a stan/lib/stan_math/lib/sundials_3.1.0/lib/libsundials_idas.a
In file included from stan/src/stan/model/model_header.hpp:8:0,
from ./examples\bernoulli\bernoulli.hpp:3,
from :0:
stan/src/stan/io/reader.hpp: In member function 'T stan::io::reader::scalar_locscale_constrain(TL, TS)':
stan/src/stan/io/reader.hpp:709:16: error: 'locscale_constrain' is not a member of 'stan::math'
return stan::math::locscale_constrain(scalar(), loc, scale);
^
stan/src/stan/io/reader.hpp: In member function 'T stan::io::reader::scalar_locscale_constrain(TL, TS, T&)':
stan/src/stan/io/reader.hpp:727:16: error: 'locscale_constrain' is not a member of 'stan::math'
return stan::math::locscale_constrain(scalar(), loc, scale, lp);
^
In file included from stan/src/stan/model/model_header.hpp:9:0,
from ./examples\bernoulli\bernoulli.hpp:3,
from :0:
stan/src/stan/io/writer.hpp: In member function 'void stan::io::writer::scalar_locscale_unconstrain(double, double, T&)':
stan/src/stan/io/writer.hpp:179:27: error: 'locscale_free' is not a member of 'stan::math'
data_r_.push_back(stan::math::locscale_free(y, loc, scale));
^
make: *** [makefile:108: examples\bernoulli\bernoulli.exe] Error 1

Per other thread, this works in cmdstan 2.19.
Closing.


PR: Add open collective funding button

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Add the configuration file for generating the Github support button at the top of the repo
Issue(s) Resolved

Fixes #10118
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Steff456

Thanks @steff456! I just activated the button :+1:


How could be deleted an account (and all devices data)?

I log in into my account but I can not find any way to actually delete it.

After deleting the user all the data is deleted. Thank you! Sorry for the inconvenience.


Reduce ceph resources to support single node jobs

This patch limits the default set of ceph services to be run to avoid issues in single node setup.
Changes made in this patch are

Dashbaord service will be disabled by default and can be enabled by updating the parameter 'cifmw_cephadm_dashboard_enabled'

ingress service will be deployed if the edpm nodes > 1

As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

@fmount FYI

Can you point out in the description that this is related to Jira: https://issues.redhat.com/browse/OSPRH-5300

/ok-to-test
@katarimanojk will be part of the group, probably the last manual ok-to-test to date :).

updated documentation here: https://gitlab.cee.redhat.com/ci-framework/docs/-/merge_requests/105

This should be good to go. @katarimanojk did you test the override using a struct like:
cifmw_ceph_daemons_layout:
  rgw_enabled: true
  dashboard_enabled: false
  cephfs_enabled: false

to see how the behavior changes in a single node deployment?
Also, @fultonj do you think we should change the "default" to false and always use this data structure to deploy more than rbd?  I'm tempted to move everything to false and drive everything via overrides.

This should be good to go. @katarimanojk did you test the override using a struct like:
cifmw_ceph_daemons_layout:
  rgw_enabled: true
  dashboard_enabled: false
  cephfs_enabled: false

to see how the behavior changes in a single node deployment? Also, @fultonj do you think we should change the "default" to false and always use this data structure to deploy more than rbd? I'm tempted to move everything to false and drive everything via overrides.

@fmount i tested the override struct in my 3-node VA1 deployment and saw the tasks were skipped accordingly.

This should be good to go. @katarimanojk did you test the override using a struct like:
cifmw_ceph_daemons_layout:
  rgw_enabled: true
  dashboard_enabled: false
  cephfs_enabled: false

to see how the behavior changes in a single node deployment? Also, @fultonj do you think we should change the "default" to false and always use this data structure to deploy more than rbd? I'm tempted to move everything to false and drive everything via overrides.

@fmount i tested the override struct in my 3-node VA1 deployment and saw the tasks were skipped accordingly.

ack as per our conversation this patch is good to go.

/approve


Allow passing RNG instance to splitobs() for shuffle control

splitobs() allows shuffling of data, but the randomization cannot be controlled by passing in a user-specified RNG. shuffleobs() does support this, but this functionality is not exploited by splitobs().

@abhro a PR would be welcome if you have time

Honestly, I'd love to have a crack at it, but right now I might end up needing the feature sooner than I can implement it. Would love for anyone else's attempt at it though :+1: :+1:


Rollback ff upgrade 0.11->0.12 (#594) due to breakage

Codecov Report
Base: 72.31% // Head: 71.95% // Decreases project coverage by -0.36% :warning:

Coverage data is based on head (0778df5) compared to base (28fc020).
Patch coverage: 48.67% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #647      +/-   ##
==========================================
- Coverage   72.31%   71.95%   -0.37%
==========================================
  Files          85       84       -1
  Lines       11577    11453     -124
==========================================
- Hits         8372     8241     -131
- Misses       3205     3212       +7

Impacted Files
Coverage Δ

client/cli/src/main.rs
0.29% <0.00%> (ø)

client/client-lib/src/mint/mod.rs
80.36% <0.00%> (-1.51%)
:arrow_down:

client/client-lib/src/wallet/mod.rs
70.50% <0.00%> (-2.19%)
:arrow_down:

client/clientd/src/main.rs
0.82% <0.00%> (-0.01%)
:arrow_down:

fedimint-api/src/db/mod.rs
79.02% <ø> (ø)

fedimint/src/lib.rs
82.71% <0.00%> (+0.33%)
:arrow_up:

ln-gateway/src/bin/ln_gateway.rs
0.55% <0.00%> (-0.01%)
:arrow_down:

fedimint-sled/src/lib.rs
49.31% <52.00%> (ø)

client/client-lib/src/ln/mod.rs
74.33% <53.84%> (-1.26%)
:arrow_down:

fedimint-rocksdb/src/lib.rs
54.66% <57.69%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Error in browser console - from pattern

Hi there
When using superfields/0.18.2.vaadin14) we get errors in the browser console.
We use the thousandseparator ' (#x2019) and the decimal separator .
This results in the following patterns being set on the entries.
	(^\-?(((\d{0,2}('?\d{3}){0,3}('?\d{0,3})?)|(\d{0,3}('?\d{3}){0,2}('?\d{0,3})?))(\.\d{0,2})?)?$) 	(^\-?(((\d{0,1}('?\d{3}){0,715827881}('?\d{0,3})?)|(\d{0,3}('?\d{3}){0,715827880}('?\d{0,3})?))(\.\d{0,3})?)?$)
In the browser console i then get the error
<input pattern='(^\-?(((\d{1,1}('> cant be checked it is a invalid Regexp: invalid identity escape in regular expression vaadin-bundle-eb9f796d93b2e8f37b0a.cache.js line 9214 > eval:603:11
<input pattern='(^\-?(((\d{1,1}('?\d{3}){0,715827881}('?\d{0,3})?)|(\d{1,3}('?\d{3}){0,715827880}('?\d{0,3})?))(\.\d{0,3})?)?$)'> cant be checked it is a invalid Regexp: invalid identity escape in regular expression
We have currently switched back to org.vaadin.miki:superfields:0.16.0-vaadin14. It workes fine there.
(but we want https://github.com/vaadin-miki/super-fields/issues/471)

hello @thomasharre - this is duplicate of #481, but for vaadin 14; I will close this issue and do a release of 0.18.3 for vaadin 14 some time next week (0.18.3 has the issue fixed for v24)
sorry for the problem and sorry for the delay (holiday season for me ;)

@thomasharre apologies for the delay, but both my holiday and regular work got a bit out of hand, hence fixing the issue took a bit more time than initially expected
anyway, release 0.18.3-vaadin14 is now in maven central - please try it out and report any errors
and thank you for using my components 😃

thanks, the JS error for the pattern have been fixed
but we are now getting other errors
TypeError: undefined is not an object (evaluating 'this.inputElement.inputMode = $0') WARNUNG: failed 3 init attempts for SuperTextField (perhaps it is used in GridPro?) - browser-initiated text selection events may not work correctly
I was not aware that there are restrictions for the superfields

@thomasharre there are no restrictions as such, but there is a known problem with GridPro - see #275 - in short, that particular component does weird things when initialising components and regular approach (used in Grid) did not help
since I have no personal license for GridPro, there is (sadly) very little I can do at the moment, given that SuperFields are my hobby, open-source project that I receive no compensation for (and hence I am unwilling to spend money on a pro license - I am of course open for discussion by email if needed ;)


example how to use actor client in javascript

e.g. in a simple svelte app
with the javascript generated client

example:

show jobs from myjobs from an actor on the server in a html5 page using svelte

@xmonader We are also kinda blocked on that for out work on the cockpit (https://github.com/threefoldtech/tf_cockpit/issues/4), so if we could prioritize this documentation effort that would be lovely.

Code is available here
1-  the package https://github.com/threefoldtech/digitalmeX/tree/development_jumpscale_myjobs_website_example/threebot/packages/myjobs
2- the actor https://github.com/threefoldtech/digitalmeX/blob/development_jumpscale_myjobs_website_example/threebot/packages/myjobs/actors/myjobs.py
3- the HTML application (svelte)  https://github.com/threefoldtech/digitalmeX/blob/development_jumpscale_myjobs_website_example/threebot/packages/myjobs/html/index.html
4- the inclusion of gedis client https://github.com/threefoldtech/digitalmeX/blob/development_jumpscale_myjobs_website_example/threebot/packages/myjobs/html/index.html#L14
5- logic to call gedis client https://github.com/threefoldtech/digitalmeX/blob/development_jumpscale_myjobs_website_example/threebot/packages/myjobs/html/JobsVisualSvelt/src/data.js
6- simple file to run the app https://github.com/threefoldtech/digitalmeX/blob/development_jumpscale_myjobs_website_example/threebot/packages/myjobs/tests/run_package.py
What's remaining is tracked on the issues mainly documenting every single bit.
1- Create UI Visualizer https://github.com/threefoldtech/jumpscaleX/issues/804
2- Create myjobs package https://github.com/threefoldtech/jumpscaleX/issues/805


Add MCV, Voxpopuli and language id example

I will be updating the PR with tests and docs later on. But want to push it online to make sure other people can get the code earlier, if needed

"The base branch requires all commits to be signed."
@Kipok could you address that issue and then let's merge?

Fixed. Sorry, I still didn't find time to do tests and docs. But please let me know if we have some users who'd want to start using this functionality right away and docs/test absence is blocking. I will then try to find some time this week to work on this.


Handle sign of zero correctly in floating-point addition and subtraction

Related to #406.
Todo:

[x] Correct sign of zeros for scalars.
[ ] Correct sign of zeros for arrays.

We should maybe summarize the rules for the different rounding modes for documentation purposes, i.e., in the documentation? Especially since we need to decide how this (and other related things, like the zero rounding in multiplication) should be handled in all non-IEEE modes...

Yes we should. I will fix the sign for the IEEE-modes now, and maybe we can go through the modes together later?

For multiplication and division the sign is currently always the XOR of the operands, for all quantization modes.
For addition and subtraction the sign currently works as described in the tables below. Should I add this information to the documentation now or wait until we have discussed the different modes?

$x + y$
TO_NEG
Other modes

$(+0) + (+0)$
$+0$
$+0$

$(+0) + (-0)$
$-0$
$+0$

$(-0) + (+0)$
$-0$
$+0$

$(-0) + (-0)$
$-0$
$-0$

$x - y$
TO_NEG
Other modes

$(+0) - (+0)$
$-0$
$+0$

$(+0) - (-0)$
$+0$
$+0$

$(-0) - (+0)$
$-0$
$-0$

$(-0) - (-0)$
$-0$
$+0$

$1 - 1$
$-0$
$+0$


⚠ Bofrost.at has degraded performance

In 0fec5d5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2001 ms

Resolved: Bofrost.at performance has improved in ec69384 after 17 minutes.


Add GenCC database

Is your feature request related to a problem? Please describe.
We're adding link-out to GenCC in #82. It would be nice to have GenCC information directly available in VarFish.
Describe the solution you'd like
Create model, add to database downloader, display icon similar to how we do now with ACMG.
Describe alternatives you've considered
N/A
Additional context
N/A

Subsumed by #1300.


 Please help me in making this project more coherent and better to look at
Thank you!!

I think there should also be a link to the repository within the main web page

It should not be named mazegenerator.com since we do not own the rights to that domain

Hi, I can work on this issue if this is still available.

Hello, sure thing! Thank you for participating!


Fix last digit wrap in customDID sequence

Fixes #11736
Status
ready
Description
I found that current algorithm in customDID function creates wrong sequence number when last digit is more than 10 and lasts with number 9, i.e. 19, 1119, 29, etc. This leads to non-sequential sequence number wraps, e.g. (taken from cmsweb-testbed):
    {
      "containerFraction": 1.0,
      "customDID": "/hasan-test10-Neutrino_E-10_gun/Run3Summer21PrePremix-120X_mcRun3_2021_realistic_v6-v2/PREMIX-V19",
      "updateTime":<PHONE_NUMBER>,
      "DN": ""
    },
    {
      "containerFraction": 1.0,
      "customDID": "/hasan-test10-Neutrino_E-10_gun/Run3Summer21PrePremix-120X_mcRun3_2021_realistic_v6-v2/PREMIX-V110",
      "updateTime":<PHONE_NUMBER>,
      "DN": "xxx"
    },

Please see that transition record went from -V19 to -V110. This PR fixes this issue.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
External dependencies / deployment changes

Please test V99 and V100 as well

Please test V99 and V100 as well

I added new unit test for this use-case too, it runs fine in my local area, but jenkins are triggered again.

commits are squashed

Thanks Valentin!


Stage PoC dual-publisher for feedback

I suspect that this'll save about 2400 more LoC in this repo based on some back-of-the-envelope calculations


blog post for MatchLabelKeys in PodAffinity

~This is just a place holder PR for now.~
It's a blog post for https://github.com/kubernetes/enhancements/issues/3633

/hold
pending assignment of publication date (by release comms)
we'll also keep this held until the v1.31 release has happened

Hey @sanposhiho , it's Matteo from the 1.31 Release comms team.
I'm reaching out with a reminder that the blog ready for review deadline is July 26, 2024.
Please let me know how we can help with content, review or anything else 😄

Now it's ready for review.

Blog publish date is currently scheduled for August 16, 2024 @sanposhiho
I'll take a look this or next week for a review :)

@Huang-Wei can you also possibly take a look from KEP reviewer pov?

/approve

@sanposhiho You may want to change "rolling upgrade" to "rolling update" if possible. Updating a Deployment doesn't always translate to "upgrade"; it could be a "downgrade" in some cases.

Good point, fixed.

Preview page

@natalisucks this one is approved and requires a lgtm label :eyes:

/hold
/lgtm
Copying @sftim's warning note, thanks Tim!
🛑 Do not unhold or merge before Kubernetes v1.31 is released 🛑

/unhold
Kubernetes v1.31 has been released.


Capture models define what's editable

(NB this might not be exactly the same as capture models in Madoc)
The user of the manifest editor is able to focus on different resources within the manifest. A canvas, a range, the manifest itself...
What is editable - i.e., what UI the manifest editor shows for the current scope - is defined by a capture model.
An out of the box manifest editor has a set of capture models, one for each possible scope, that allows the user to edit/create IIIF properties and resources in line with the spec, and nothing more.
Sometimes, a deployment of the manifest editor will wish to simplify the interface - to not expose all possible editable things.
Sometimes, a deployment of the manifest editor will wish to extend the interface - add custom behavours, additional custom properties, or specialised editors for defining an external resource.
In both cases, the manifest editor has been configured with a non-default set of capture models. The manifest editor uses the capture model configuration to generate the editing UI for the scope of the user's current focus.

Would a toggle option be possible to go from simple - to complex etc - model 1, 2, 3 .... n ?
Another thought - would this process allow for default values in the defined models?
I want the author to always be me, I want the rights to always be CC0, My manifests are all supported by Grant number 1, etc ....

I think this could be done via templates. As well as supporting sources like the IIIF Cookbook for starting points for Manifests, rather than a blank slate, you can via config add in additional ones - such as a Manifest with provider and other metadata pre-filled.
This also sounds like something that would be useful for per-user config, stored in local storage.
As well as pre-canned templates under File->New, I also get any personal ones I've added.

Although not using capture models yet, the process for defining "Creators" for different types of resources has been implemented.
https://github.com/digirati-co-uk/iiif-manifest-editor/tree/feature/editor-changes/manifest-editor/src/_creators
We have templates for starting points (Manifests) which can be expanded.
There is also an "Overview" panel in the editing side that shows non-empty values together. We don't currently have a way to specify global "Read-only" for a property, but this will be required soon anyway for locking resources that are being edited elsewhere.


Make serde optional

This change makes the serde dependency optional to reduce the total number of them from 57 to 51 using --no-default-features flag.
Also for simpler maintainablity, moved SerializableProof into separate serializable modules, so by placing #[cfg(feature = "serde")] above it's declaration the whole implementations and structures became optional (instead of placing #[cfg(feature = "serde")] above each of them).
Removed usage of serde inside tests for more prefered Debug implementations.

Conflicts with #2 as updated some of the code in the tests

It will be great to add checks and tests with serde feature and without into CI


fix: adding in some loggers to see what is going on in the handler

Adding in additional loggers to troubleshoot the index handler not properly extracting the byte-type output and converting it to json.

Won't merge this into the Defiance repo.


Update README.md to include yarn commmand

On installing and running the app for the first time, directly running yarn start does not work and fails with a error saying that it is missing some dependencies like react-script, hence users first need to run yarn, once inside the directory

Thanks!


Draft: Issue 630 -  Widget Tests

Hey @dayhaysoos -- as I dug into this, I realized that some additional refactoring may be needed.  Because the JavaScript in examples/web5-quickstart-widgets/index.js is not co-located with the HTML (as it is in docs/web5/quickstart.js), there are test failures when event handlers/query selectors attempt to find the DOM elements on which they are operating.
I still need to do a little more digging, but it might make sense for this code that manipulates the DOM to live in the HTML in a script tag, and any pure functions (or functions that otherwise don't touch the DOM directly) can live in index.js.
Would love your thoughts on what would make the most sense here.  Thanks!

Hey @danlourenco, I agree with this approach. I tried messing around with the vite config and adding jsdom for browser APIs but I think that's going to be too much. We just want to test the Web5 stuff for now. You wanna give that a shot and see how it goes?


Unable to Answer Calls with Locked Device

Description:
I am experiencing an issue where I am unable to answer incoming calls when my device is locked. When a call comes in, I am unable to complete the call while the device is locked. The call can only be initiated once the device is unlocked, as this is when Flutter is initialized.
Steps to Reproduce:

Lock the device.
Receive an incoming call.
Attempt to answer the call while the device is still locked.
Observe that the call cannot be completed.
Unlock the device.
Initiate the call successfully after the device is unlocked.

Expected Behavior:
I expect the call answering functionality to behave similarly to applications like WhatsApp, where incoming calls can be answered even when the device is locked. Users should be able to answer calls directly from the lock screen without needing to unlock the device first. This would enhance user experience and provide greater convenience when receiving calls.
Can anybody help me?

Hello @viniciusoliverrs, I'm not exactly sure about your use case, but I recently had this same issue on IOS.
My issue was when answering the call on an unlocked phone, it took me to the app screen as expected. But when the phone is locked and I answer the call, the app wakes, and the call starts in the background.
After troubleshooting and trying different applications' behavior, I found that the issue was with the call type.
If you set the call type as video, and answer the call in any state the call will start in the app in the foreground, you'll still need to unlock the device to see the app UI.

Hi @umairmehar, but on Android too? Where can I configure call settings? In the server?

@viniciusoliverrs in the parameters you send to the showCallkitIncoming there is an int named type.
according to the docs 0 is voice call and 1 is video call.
i had your issue and it was fixed this way.


Static return types are being ignored

Describe the bug
When using find or findOrFail methods within Laravel as such:
$account = Account::findOrFail($request->account_id);

It will be inferred that Illuminate\Database\Eloquent\Collection<Illuminate\Database\Eloquent\Model>|Illuminate\Database\Eloquent\Model is being returned, even though what is actually returned is object of Account type, per declaration within find or findOrFail being * @return \Illuminate\Database\Eloquent\Model|\Illuminate\Database\Eloquent\Collection|static|static[]
This will cause additional errors to be returned like:
ERROR: UndefinedMagicPropertyFetch - app/Http/Controllers/Policy/StoreController.php:36:31 - Magic instance property Illuminate\Database\Eloquent\Collection::$current_turnover is not defined (see https://psalm.dev/218)

or
ERROR: InvalidArgument - app/Services/CoverTransactionService.php:351:72 - Argument 1 of App\Snapshots\AccountSnapshot::__construct expects App\Models\Account, Illuminate\Database\Eloquent\Collection<Illuminate\Database\Eloquent\Model>|Illuminate\Database\Eloquent\Model provided (see https://psalm.dev/004)

It's as if static was not being taken into account when considering returned types.
As a workaround one needs to preamble with @var as such:
/**
 * @var Account $account
 */
 $account = Account::findOrFail($request->account_id);

but I believe the expected behaviour should be inferring that Account model is being returned per static
Impacted Versions
barryvdh/laravel-ide-helper                      v2.9.0             Laravel IDE Helper, generates correct PHPDocs for all Facade classes, to improve auto-completion.
cknow/laravel-money                              v6.1.0             Laravel Money
fruitcake/laravel-cors                           v2.0.3             Adds CORS (Cross-Origin Resource Sharing) headers support in your Laravel application
jessarcher/laravel-castable-data-transfer-object v1.0.0             Automatically cast JSON columns to rich PHP objects in Laravel using Spatie's data-transfer-object class
kreait/laravel-firebase                          2.4.0              A Laravel package for the Firebase PHP Admin SDK
laravel/framework                                v8.30.1            The Laravel Framework.
laravel/horizon                                  v5.7.0             Dashboard and code-driven configuration for Laravel queues.
laravel/tinker                                   v2.6.1             Powerful REPL for the Laravel framework.
psalm/plugin-laravel                             v1.4.2             A Laravel plugin for Psalm
spatie/laravel-activitylog                       3.17.0             A very simple activity logger to monitor the users of your website or application
spatie/laravel-enum                              2.4.0              Laravel Enum support
spatie/laravel-event-sourcing                    4.8.0              The easiest way to get started with event sourcing in Laravel
spatie/laravel-json-api-paginate                 1.10.0             A paginator that plays nice with the JSON API spec
spatie/laravel-query-builder                     3.3.4              Easily build Eloquent queries from API requests
spatie/laravel-schemaless-attributes             1.8.3              Add schemaless attributes to Eloquent models
vimeo/psalm                                      4.6.2              A static analysis tool for finding errors in PHP applications

I wasn't able to reproduce this in the test suite: https://github.com/psalm/psalm-plugin-laravel/commit/997c3e61447c3a05c219e4fc62a50e707fc0a55b
I believe this may be resolved now. Please reopen if you think I'm incorrect!


Nothing happens with showNotificationWithTitle

Hi everyone,
I'm calling:
TSMessage.showNotificationWithTitle("something here", type: TSMessageNotificationType.Message)

But nothing seems to display.  However, calling showNotificationInViewController works wonderfully.
Is it because the current view is in a tabBarController? (with navigation bars)
Is this standard behavior? I'm on latest version of Swift and master branch.

Hi felixchan....  I have the same problem.... I invoke
TSMessage.showNotificationWithTitle("something here", type: TSMessageNotificationType.Message)
in a UIView  and nothing happend.... I work with objetive c .....
In a controller ....showNotificationInViewController work fine....

Not for me ! Same problem in UIViewController in UINavigationController.
Swift 2.2, XCode 7.3.1, OSX 10.11.5


fresh install u8 fails

Describe the bug
on a single clean disk u8 fails to install.
there is an error when creating new Uefi bootvar
this is caused by trying to activate (-a) prior to having a boot variable created.
error located 0n line 366 "functions-unmount.sh"

Ticket moved to the trueos/trueos repository (issue with pc-sysinstall):
https://github.com/trueos/trueos/issues/306


Update Hibernate ORM from 5.0.1 to 5.0.2.

Linux Build 8365 outcome was FAILURE using a merge of 1d31b95dcb6b43ebc730ee7029a377fb65a25f02
Summary: Tests failed: 1 (1 new), passed: 3189, ignored: 374 Build time: 00:56:30
Failed tests
org.jboss.as.test.integration.jpa.secondlevelcache.JPA2LCTestCase.testEvictQueryCache: java.lang.AssertionError: Expected 1 miss in cache(hitCount=1, missCount=0, putCount=0). expected:<1> but was:<0>
	at org.jboss.as.test.integration.jpa.secondlevelcache.JPA2LCTestCase.testEvictQueryCache(JPA2LCTestCase.java:250)
------- Stdout: -------
 [0m23:44:44,445 INFO  [stdout] (pool-352-thread-1) Hibernate: select employee0_.id as id1_0_, employee0_.address as address2_0_, employee0_.name as name3_0_ from Employee employee0_ where employee0_.id>3
 [0m [0m23:44:44,448 INFO  [org.hibernate.engine.internal.StatisticalLoggingSessionEventListener] (pool-352-thread-1) Session Metrics {
    241366 nanoseconds spent acquiring 2 JDBC connections;
    41275 nanoseconds spent releasing 2 JDBC connections;
    524197 nanoseconds spent preparing 1 JDBC statements;
    74245 nanoseconds spent executing 1 JDBC statements;
    0 nanoseconds spent executing 0 JDBC batches;
    363285 nanoseconds spent performing 5 L2C puts;
    28611 nanoseconds spent performing 2 L2C hits;
    47932 nanoseconds spent performing 1 L2C misses;
    75088 nanoseconds spent executing 1 flushes (flushing a total of 4 entities and 0 collections);
    193438 nanoseconds spent executing 2 partial-flushes (flushing a total of 4 entities and 4 collections)
}
 [0m [0m23:44:44,449 INFO  [org.hibernate.engine.internal.StatisticalLoggingSessionEventListener] (pool-352-thread-1) Session Metrics {
    0 nanoseconds spent acquiring 0 JDBC connections;
    0 nanoseconds spent releasing 0 JDBC connections;
    0 nanoseconds spent preparing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC batches;
    0 nanoseconds spent performing 0 L2C puts;
    0 nanoseconds spent performing 0 L2C hits;
    0 nanoseconds spent performing 0 L2C misses;
    0 nanoseconds spent executing 0 flushes (flushing a total of 0 entities and 0 collections);
    0 nanoseconds spent executing 0 partial-flushes (flushing a total of 0 entities and 0 collections)
}
 [0m [0m23:44:45,453 INFO  [org.hibernate.engine.internal.StatisticalLoggingSessionEventListener] (pool-352-thread-1) Session Metrics {
    0 nanoseconds spent acquiring 0 JDBC connections;
    0 nanoseconds spent releasing 0 JDBC connections;
    0 nanoseconds spent preparing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC batches;
    0 nanoseconds spent performing 0 L2C puts;
    758138 nanoseconds spent performing 6 L2C hits;
    0 nanoseconds spent performing 0 L2C misses;
    306506 nanoseconds spent executing 1 flushes (flushing a total of 4 entities and 0 collections);
    13506 nanoseconds spent executing 1 partial-flushes (flushing a total of 0 entities and 0 collections)
}
	at org.jboss.as.test.integration.jpa.secondlevelcache.JPA2LCTestCase.testEvictQueryCache(JPA2LCTestCase.java:250)
	at org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.doRunTestMethod(ArquillianService.java:247)
	at org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.runTestMethod(ArquillianService.java:223)
	at com.sun.jmx.mbeanserver.StandardMBeanIntrospector.invokeM2(StandardMBeanIntrospector.java:112)
	at com.sun.jmx.mbeanserver.StandardMBeanIntrospector.invokeM2(StandardMBeanIntrospector.java:46)
	at com.sun.jmx.mbeanserver.MBeanIntrospector.invokeM(MBeanIntrospector.java:237)
	at com.sun.jmx.mbeanserver.PerInterface.invoke(PerInterface.java:138)
	at com.sun.jmx.mbeanserver.MBeanSupport.invoke(MBeanSupport.java:252)
	at com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.invoke(DefaultMBeanServerInterceptor.java:819)
	at com.sun.jmx.mbeanserver.JmxMBeanServer.invoke(JmxMBeanServer.java:801)
	at org.jboss.as.jmx.PluggableMBeanServerImpl$TcclMBeanServer.invoke(PluggableMBeanServerImpl.java:1503)

Windows Build 3402 outcome was FAILURE using a merge of 1d31b95dcb6b43ebc730ee7029a377fb65a25f02
Summary: Tests failed: 1 (1 new), passed: 3189, ignored: 374 Build time: 00:58:32
Failed tests
org.jboss.as.test.integration.jpa.secondlevelcache.JPA2LCTestCase.testEvictQueryCache: java.lang.AssertionError: Expected 1 miss in cache(hitCount=1, missCount=0, putCount=0). expected:<1> but was:<0>
	at org.jboss.as.test.integration.jpa.secondlevelcache.JPA2LCTestCase.testEvictQueryCache(JPA2LCTestCase.java:250)
------- Stdout: -------
23:53:08,886 INFO  [stdout] (pool-352-thread-1) Hibernate: select employee0_.id as id1_0_, employee0_.address as address2_0_, employee0_.name as name3_0_ from Employee employee0_ where employee0_.id>3

23:53:08,889 INFO  [org.hibernate.engine.internal.StatisticalLoggingSessionEventListener] (pool-352-thread-1) Session Metrics {
    194858 nanoseconds spent acquiring 2 JDBC connections;
    53080 nanoseconds spent releasing 2 JDBC connections;
    312610 nanoseconds spent preparing 1 JDBC statements;
    63276 nanoseconds spent executing 1 JDBC statements;
    0 nanoseconds spent executing 0 JDBC batches;
    288588 nanoseconds spent performing 5 L2C puts;
    25700 nanoseconds spent performing 2 L2C hits;
    71306 nanoseconds spent performing 1 L2C misses;
    80038 nanoseconds spent executing 1 flushes (flushing a total of 4 entities and 0 collections);
    240182 nanoseconds spent executing 2 partial-flushes (flushing a total of 4 entities and 4 collections)
}
23:53:08,889 INFO  [org.hibernate.engine.internal.StatisticalLoggingSessionEventListener] (pool-352-thread-1) Session Metrics {
    0 nanoseconds spent acquiring 0 JDBC connections;
    0 nanoseconds spent releasing 0 JDBC connections;
    0 nanoseconds spent preparing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC batches;
    0 nanoseconds spent performing 0 L2C puts;
    0 nanoseconds spent performing 0 L2C hits;
    0 nanoseconds spent performing 0 L2C misses;
    0 nanoseconds spent executing 0 flushes (flushing a total of 0 entities and 0 collections);
    0 nanoseconds spent executing 0 partial-flushes (flushing a total of 0 entities and 0 collections)
}
23:53:09,891 INFO  [org.hibernate.engine.internal.StatisticalLoggingSessionEventListener] (pool-352-thread-1) Session Metrics {
    0 nanoseconds spent acquiring 0 JDBC connections;
    0 nanoseconds spent releasing 0 JDBC connections;
    0 nanoseconds spent preparing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC statements;
    0 nanoseconds spent executing 0 JDBC batches;
    0 nanoseconds spent performing 0 L2C puts;
    247934 nanoseconds spent performing 6 L2C hits;
    0 nanoseconds spent performing 0 L2C misses;
    127182 nanoseconds spent executing 1 flushes (flushing a total of 4 entities and 0 collections);
    7474 nanoseconds spent executing 1 partial-flushes (flushing a total of 0 entities and 0 collections)
}
	at org.jboss.as.test.integration.jpa.secondlevelcache.JPA2LCTestCase.testEvictQueryCache(JPA2LCTestCase.java:250)
	at org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.doRunTestMethod(ArquillianService.java:247)
	at org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.runTestMethod(ArquillianService.java:223)
	at com.sun.jmx.mbeanserver.StandardMBeanIntrospector.invokeM2(StandardMBeanIntrospector.java:112)
	at com.sun.jmx.mbeanserver.StandardMBeanIntrospector.invokeM2(StandardMBeanIntrospector.java:46)
	at com.sun.jmx.mbeanserver.MBeanIntrospector.invokeM(MBeanIntrospector.java:237)
	at com.sun.jmx.mbeanserver.PerInterface.invoke(PerInterface.java:138)
	at com.sun.jmx.mbeanserver.MBeanSupport.invoke(MBeanSupport.java:252)
	at com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.invoke(DefaultMBeanServerInterceptor.java:819)
	at com.sun.jmx.mbeanserver.JmxMBeanServer.invoke(JmxMBeanServer.java:801)

This error looks related to the upgrade. @scottmarlow any ideas on this?

Not sure at this point if this is a regression in Hibernate ORM 5.0.2 or a problem with the failing test.

This seems likely to be a Hibernate regression but will see.

Just found http://planet.jboss.org/post/hibernate_second_level_cache_improvements where the author says "you should not use eviction, ...".
Perhaps this test should be disabled then?

I sent a private email to the author (Radim Vansa), asking for his help with understanding what has changed that leads to the test failure.

@langfr Eviction should not be used if you're using replicated/distributed cache for entities/collections. If you haven't changed your configuration (local or invalidation most likely) you're safe to use it.

@scottmarlow It seems I have to ask you for trace logs, again. Btw., any chance you could port these tests to ORM so that I can see them failing before you try to do the upgrade? I thought it would be great if there was some 2LC TCK that could be easily used from all 2LC impls, so that nobody boasts implementing 2LC when the impl is not working properly.

@rvansa wouldn't it be easier to run the WildFly test suite once before a Hibernate release (if there are changes to the hibernate-infinispan module)?  I enable TRACE logging for Infinispan and then Hibernate + Infinispan.  logs.zip is at https://www.dropbox.com/s/cl49nmjzjuejl5u/logs.zip?dl=0

the test source is at https://github.com/wildfly/wildfly/blob/master/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/jpa/secondlevelcache/JPA2LCTestCase.java#L238 and the called test bean method is at https://github.com/wildfly/wildfly/blob/master/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/jpa/secondlevelcache/SFSB2LC.java#L313

@scottmarlow Thanks, I'll take a look. I don't think that Steve would fancy the idea of him running WF test suite before release, but I probably should check it out after doing some changes.

@scottmarlow Damned, found the problem, silly mistake. Region used for querying is not used via any RegionAccessStrategy and therefore one initialization step is skipped. Quite easy to fix (I suppose), but I don't see any workaround but calling BaseTransactionalDataRegion.createAccessDelegate(AccessType.TRANSACTIONAL) using reflection.
https://hibernate.atlassian.net/browse/HHH-10163

Regarding running the WildFly testsuite, I don't think Steve is going to run that himself.  It would be more that you could if you wanted to (after making a change to hibernate-infinispan).  You don't need to run the entire testsuite.  It would be more like the following after building ORM locally (./gradlew clean publishToMavenLocal):
cd wildfly_root_folder
./build.sh clean install -Dversion.org.hibernate=5.0.3-SNAPSHOT -DskipTests=true
cd testsuite/integration/basic
mvn clean install -Dtest=JPA2LCTestCase
On my Thinkpad T540 (running Fedora), it took 28 seconds to run the test and just under 3 minutes to build WildFly.

This pull request can be closed, due to the HHH-10163 regression.

I'll close this, lets try again with Hibernate ORM 5.0.3 (if the HHH-10163 issue is addressed by then).

Hmm, I don't have permissions to close, so someone else will need to.

Closing per request.


Arduino OTA integration

I did simple OTA integration on the latest unified sketch.

Thanks for the PR. I'll set up a test environment for this shortly and merge it in.


feat(acme) change default email to<EMAIL_ADDRESS>

Replace acme email address by the private jenkins infra one #2793

Thanks for this contribution @ADITYADAS1999 !


bug: tweak external http requests so that they are not failing too often

External HTTP requests are failing too often. For example downloading binaries from GitHub or keys from repositories such as VirtualBox.

:tada: This issue has been resolved in version 1.13.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


helm2 can't get correct hook result cause install chart failed

helm version:
Client: &version.Version{SemVer:"v2.15.2", GitCommit:"", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.15.2", GitCommit:"", GitTreeState:"clean"}
kubectl version:
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"archive", BuildDate:"2019-11-12T15:46:42Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"archive", BuildDate:"2019-11-12T15:47:23Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
private cloud platform.
issue:
in the chart file we have two hooks, one is preinstall hook, another one is preupgrade hook.
see below information (I replace senstivie information as ***)
$ kubectl get jobs
NAME                        COMPLETIONS   DURATION   AGE
***-preinstalljob   0/1           8s         8s
***-preupgradejob   0/1           46m        46m
[cranuser1@controller-1 aic-vdu]$ kubectl get pod
NAME                              READY   STATUS                       RESTARTS   AGE
***-preinstalljob-6hkts   0/1     Completed            0          20s
***-preupgradejob-dvg2w   0/1     Error                        0          46m
When preupgradejob failed with BackoffLimitExceeded, and we install new same release, the helm will report error immediately. like below:
$ helm install ./ --name  *** -f values-override.yaml  --debug
[debug] SERVER: "***"
[debug] Original chart version: ""
[debug] CHART PATH: ***
Error: Job failed: BackoffLimitExceeded
Actually the pre-install hook is running at this time and finally it exit with 0.

Hi @ddljdd. We are no longer accepting bug reports for Helm 2. At this time, only issues related to security are being considered. See https://helm.sh/blog/helm-v2-deprecation-timeline/ for more information.
If you can reproduce the issue with helm 3, please file a new ticket. Thank you!


react template not loading

Hello,
Im running last version of Visual Studio Community 2022 (17.0.0 RC3) in a updated Windows 10 (21H1 19043.1320) with NPM 8.1.0 and node 16.13. I can create the solution and create a .NET Core project but when I tried to create a React standalone the Visual Studio just freeze, the project is not created and I have end the task manually.
Any ideas? Thanks in advance

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

ID: f5beb7f5-5d28-d5e5-92ba-3e04c75e44b5
Version Independent ID: 867e3a3f-2387-ba46-c911-10822f8c4a81
Content: Create an ASP.NET Core app with React - Visual Studio (Windows)
Content Source: docs/javascript/tutorial-asp-net-core-with-react.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@Sergiodiaz53 -- Sergio, thank you for your feedback.  This repo is solely for documentation issues, and this appears product-related.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

You're welcome to add this here:

Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Incorrect API Call for Contributors' Data on About Page

Describe the bug
The API call for contributors data in About Us page is referring "MAVRICK-1/e-commerce_website" which is other repo.
Expected behavior
The API call for contributors' data should be made to this repo.
Screenshots

About Page :

API Call :

please assign me this issue for ssoc


Requirements for zcsmower not found: ['pillow==9.5.0'].

Hello,
I have updated HA this morning to the august version and I got this error after restarting it.
Logger: homeassistant.setup
Source: setup.py:209
First occurred: 07:26:26 (1 occurrences)
Last logged: 07:26:26
Setup failed for custom integration zcsmower: Requirements for zcsmower not found: ['pillow==9.5.0'].

Reading the HA release note, it seems they have updated pillow to version 10.0.0
Update Pillow to 10.0.0 (@frenck - #96106)

Fix is available in main branch. Version update is available in two hours or less.

@JC38, new version v0.9.7 is available.


Add === for Types

https://github.com/non/spire/issues/423

I believe that this PR is ready for review, please.

One larger comment: when calling the method, let's write this === that instead of ===(that). I think the former is a lot easier for people to visually-parse.

Another thought I just had: do you think we should add a =!= method? otherwise, someone will need to write !(x === y). What do you think?

Another thought I just had: do you think we should add a =!= method?

Definitely. Please let me know any improvements/comments/suggestions/criticisms.
I have to admit that one of my interests in contributing to non/spire is to learn from you especially (and maybe contribute/learn more about non/cats).

Hmm. NumberTest is failing:
[info] - operations *** FAILED ***
[info]<PHONE_NUMBER>854775806 equaled 9.223372036854776E18 (NumberTest.scala:131)

Looking into it.

Hey, sorry I dropped the ball on this. I think we should merge it as-is now and then fix up any problems in a future PR. I'm going to try to fix the merge conflicts locally and merge this now.


Cannot access this repository

Hi,
I hope you are doing good.
I am trying to clone this repository (handoversim) into my local machine using the procedure mentioned in the README of this repo with this command (git clone --recursive git@github.com:NVlabs/handover-sim.git). The problem that my machine raise is :
""
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
""
Can you please fix it as soon as possible?

This is an issue of your GitHub config and not the repo itself.
To fix it, you can either:

add your SSH key to your GitHub account, or

clone the repo using HTTPS:
git clone --recursive https://github.com/NVlabs/handover-sim.git


Can't install with google maps iOS utils 3.x

Question
I want to install react-native-maps but I'm using use_frameworks in my Podfile (and I can't get rid of it). Google Maps iOS utils has now a branch 3.x which allows the use of use_frameworks but react-native-maps dependency asks for the version 2.1.0.
Is it possible to change that so I can use the module directly?
Thanks.

What you can try is to go to node_modules/react-native-maps in your project and change the podspec files yourself and then run in  the  ios  folder pod install
If all works fine maybe make a PR too 😅

Unfortunately, even if I changed the dependency needed, I still get this error:
target has transitive dependencies that include statically linked binaries: (xx/ios/Pods/GoogleMaps/Base/Frameworks/GoogleMapsBase.framework, xx/ios/Pods/GoogleMaps/Maps/Frameworks/GoogleMaps.framework, and xx/ios/Pods/GoogleMaps/Maps/Frameworks/GoogleMapsCore.framework)

So though Google Maps iOS utils allows the use of use_frameworks , react-native-maps doesn't.

I found a workaround just after posting the last comment with this: #1923
pre_install do |installer|
# workaround for CocoaPods/CocoaPods#3289
Pod::Installer::Xcode::TargetValidator.send(:define_method, :verify_no_static_framework_transitive_dependencies) {}
end`

The pods then install fine with no error but now I have either AirMapCoordinate.h not found or RCTConvert+AirMap.h not found. I did add AirMaps and AirGoogleMaps inside my project.

Some

I found a workaround just after posting the last comment with this: #1923
pre_install do |installer|
# workaround for CocoaPods/CocoaPods#3289
Pod::Installer::Xcode::TargetValidator.send(:define_method, :verify_no_static_framework_transitive_dependencies) {}
end`

The pods then install fine with no error but now I have either AirMapCoordinate.h not found or RCTConvert+AirMap.h not found. I did add AirMaps and AirGoogleMaps inside my project.

This causes others bugs FUuu!

Yeah, I've been on this since then and I can't seem to find any solution ... I've tried everything, I've tried to set the header search paths to every folder I know that contain the headers I need but no luck. If anyone has the start of a solution for this, I'll take everything to be honest.

Yeah, I've been on this since then and I can't seem to find any solution ... I've tried everything, I've tried to set the header search paths to every folder I know that contain the headers I need but no luck. If anyone has the start of a solution for this, I'll take everything to be honest.

I tried about 10 ways and it didn't work in any way.  I will describe what I did at the end.

Clear all (build, npm, ..)
a) In project folder run:
rm -rf ./node_modules
b) Enter in ios folder and run:
rm -rf ~/Library/Caches/CocoaPods; rm -rf Pods; rm -rf ~/Library/Developer/Xcode/DerivedData/*; pod deintegrate; pod setup; pod install;
obs: This step i found in -> https://gist.github.com/mbinna/4202236#gistcomment-1814665
c) In XCode clear build

Deintegrate your pod, I heard that the command "use_frameworks!" you can make fun of the premises, in ios folder run:
pod deintegrate

Now reset your machine, this step was very important for me (maybe XCode updates can make fun of open projects), restart your machine.

In your pod filde, do not use use_frameworks! , this can make fun of many libs, I am not aware of the reason but I switched to use_frameworks! :linkage =>: static
In my case what started to hinder me was the onesignal libs, maps, places and text input mask .. The lib text input mask needed use_frameworks! but use_frameworks! bugged maps, places!  so I used the step described above use_frameworks! :linkage =>: static and resolved!
Again, Remove all 'use_frameworks!' and use only 'use_frameworks! :linkage =>:static' (I put it inside the target of my project only).

I use the onesignal frame and with that I had these problems too:
"'sharedApplication' is unavailable: not available on iOS (App Extension)"
If you have this problem too then add this code in your pod (i added outside of project target):
Obs: (Remember to put 'No' and not 'NO', as in the code below. And this step only works accurately if you perform step 2 before)

post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'No'
    end
  end
end

The method it was only useful when used with line command 'react-native run-ios', by xcode it still throws the error, if you want to run through XCode, when the error appears in XCode double click on the project containing the error in the error window and it will be forwarded to all pods, select all, click in in build settings and set
'require only App-Extension-Safe Api' to 'No' for all. Unfortunately I still need to do this whenever I need to run 'pod install' and test the app directly through XCode.

I followed several tutorials that sent edit header path, framework path, add in linked libraries. I just undid everything, left the project as it was without these meaningless edits and did these steps above.

If necessary use:

`pre_install do |installer|
workaround for CocoaPods/CocoaPods#3289
Pod::Installer::Xcode::TargetValidator.send(:define_method, :verify_no_static_framework_transitive_dependencies) {}
end``

In project folder run:
npm install

In ios folder inside your project folder run:
pod install

Prepare a coffee, and try to build it again .. Good luck.

Thanks for your answer. I tried what you said but no luck. I have a few questions:

Did you keep the AirMaps and AirGoogleMaps folders in your iOS targets ?
Did you add the AirMaps library in the library folder?
Did you keep all of that in your Podfile ?

pod 'react-native-maps', path: rn_maps_path
  pod 'react-native-google-maps', path: rn_maps_path  # Uncomment this line if you want to support GoogleMaps on iOS
  pod 'GoogleMaps'  # Uncomment this line if you want to support GoogleMaps on iOS
  pod 'Google-Maps-iOS-Utils', '~> 3.0.2'

Thanks for your answer. I tried what you said but no luck. I have a few questions:

Did you keep the AirMaps and AirGoogleMaps folders in your iOS targets ?
Did you add the AirMaps library in the library folder?
Did you keep all of that in your Podfile ?

pod 'react-native-maps', path: rn_maps_path
  pod 'react-native-google-maps', path: rn_maps_path  # Uncomment this line if you want to support GoogleMaps on iOS
  pod 'GoogleMaps'  # Uncomment this line if you want to support GoogleMaps on iOS
  pod 'Google-Maps-iOS-Utils', '~> 3.0.2'

Look my pod file:
Obs:(I didn't use google maps, I ended up using the native one.)
`platform :ios, '9.0'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'
target 'gorjeta_app' do
Pods for gorjeta_app
pod 'FBLazyVector', :path => "../node_modules/react-native/Libraries/FBLazyVector"
pod 'FBReactNativeSpec', :path => "../node_modules/react-native/Libraries/FBReactNativeSpec"
pod 'RCTRequired', :path => "../node_modules/react-native/Libraries/RCTRequired"
pod 'RCTTypeSafety', :path => "../node_modules/react-native/Libraries/TypeSafety"
pod 'React', :path => '../node_modules/react-native/'
pod 'React-Core', :path => '../node_modules/react-native/'
pod 'React-CoreModules', :path => '../node_modules/react-native/React/CoreModules'
pod 'React-Core/DevSupport', :path => '../node_modules/react-native/'
pod 'React-RCTActionSheet', :path => '../node_modules/react-native/Libraries/ActionSheetIOS'
pod 'React-RCTAnimation', :path => '../node_modules/react-native/Libraries/NativeAnimation'
pod 'React-RCTBlob', :path => '../node_modules/react-native/Libraries/Blob'
pod 'React-RCTImage', :path => '../node_modules/react-native/Libraries/Image'
pod 'React-RCTLinking', :path => '../node_modules/react-native/Libraries/LinkingIOS'
pod 'React-RCTNetwork', :path => '../node_modules/react-native/Libraries/Network'
pod 'React-RCTSettings', :path => '../node_modules/react-native/Libraries/Settings'
pod 'React-RCTText', :path => '../node_modules/react-native/Libraries/Text'
pod 'React-RCTVibration', :path => '../node_modules/react-native/Libraries/Vibration'
pod 'React-Core/RCTWebSocket', :path => '../node_modules/react-native/'
pod 'React-cxxreact', :path => '../node_modules/react-native/ReactCommon/cxxreact'
pod 'React-jsi', :path => '../node_modules/react-native/ReactCommon/jsi'
pod 'React-jsiexecutor', :path => '../node_modules/react-native/ReactCommon/jsiexecutor'
pod 'React-jsinspector', :path => '../node_modules/react-native/ReactCommon/jsinspector'
pod 'ReactCommon/jscallinvoker', :path => "../node_modules/react-native/ReactCommon"
pod 'ReactCommon/turbomodule/core', :path => "../node_modules/react-native/ReactCommon"
pod 'Yoga', :path => '../node_modules/react-native/ReactCommon/yoga'
pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'
pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
pod 'RNCAsyncStorage', :path => '../node_modules/@react-native-community/async-storage'
pod 'RNVectorIcons', :path => '../node_modules/react-native-vector-icons'
pod 'RNI18n', :path => '../node_modules/react-native-i18n'
pod 'react-native-safe-area', :path => '../node_modules/react-native-safe-area'
pod 'RNTextSize', :path => '../node_modules/react-native-text-size/ios/RNTextSize.podspec'
pod 'react-native-google-places', :path => '../node_modules/react-native-google-places'
pod 'react-native-geolocation', :path => '../node_modules/@react-native-community/geolocation'
pod 'react-native-maps', :path => '../node_modules/react-native-maps'
pod 'react-native-onesignal', :path => '../node_modules/react-native-onesignal'
pod 'RNInputMask', :path => '../node_modules/react-native-text-input-mask/ios/InputMask'
target 'gorjeta_appTests' do
inherit! :search_paths
# Pods for testing
end
use_frameworks! :linkage => :static
use_native_modules!
target 'OneSignalNotificationServiceExtension' do
pod 'OneSignal', '>= 2.11.2', '< 3.0'
end
end
post_install do |installer|
installer.pods_project.targets.each do |target|
if target.name == 'react-native-google-places'
target.build_configurations.each do |config|
config.build_settings['CLANG_ENABLE_MODULES'] = 'No'
config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'No'
end
else
target.build_configurations.each do |config|
config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'No'
end
end
if target.name == "React"
target.remove_from_project
end
end
end
target 'gorjeta_app-tvOS' do
Pods for gorjeta_app-tvOS
target 'gorjeta_app-tvOSTests' do
inherit! :search_paths
# Pods for testing
end
end
`

God thank you!!!!!!
It finally works!
I indeed had to use the native maps for iOS, instead of the google maps provider, but at this point, I'll be happy with anything.
Thanks a lot for the time you took to help :)

:D !! Any time!

If someone else stumbles upon this closed issue, this comment worked for me: https://github.com/react-native-maps/react-native-maps/issues/3656#issuecomment-749570974

If someone else stumbles upon this closed issue, this comment worked for me: https://github.com/react-native-maps/react-native-maps/issues/3656#issuecomment-749570974


--prof-startup stop too early

We have made some changes and now the profiler stop too early most likely not catching editor/viewlet restore.

Tested out of dev. Ensured that trace contains:  restore editor view state


Update Event::VoiceStateUpdate / Add Channel::editMessage

Event::VoiceStateUpdate:

use allowVoice()
get rid of the foreach and just check if it has() the member
since we are looping through the channels anyways, why not push him here as well?

Channel::editMessage:

because it's missing

Cheers


v0.7.1: fix a dependency error

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (10553b2) 82.08% compared to head (cf344c2) 82.08%.
Report is 1 commits behind head on v0.7-release.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           v0.7-release     #906   +/-   ##
=============================================
  Coverage         82.08%   82.08%
=============================================
  Files               226      226
  Lines             12478    12478
=============================================
  Hits              10243    10243
  Misses             2235     2235

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


zod/lib/external has no exported member 'infer'

I am having issues using z.infer. However, z.TypeOf (which looking at the types it appears infer is just an alias of TypeOf) is working.
const repaymentSchema = z.object({
  from: z.number(),
  to: z.number(),
  amount: z.string(),
});

export type Repayment = z.infer<typeof repaymentSchema>;

This results in the following error:
TSError: ⨯ Unable to compile TypeScript
src/splitwise/client.ts (11,27): Namespace '"/Users/dmmulroy/repos/ynab-splitwise-sync/node_modules/zod/lib/external"' has no exported member 'infer'. (2694)

Switching to the following works fine:
export type Repayment = z.TypeOf<typeof repaymentSchema>;

In vscode, the editor accurately infers the type when using z.infer but at compilation time it fails.

 I have theexact same issue. And I'm not using ts-node. The error just showed up suddenly, when I renamed a schema inside my schema file and saved. Boom. Suddenly infer was not recognized anymore. Any idea what could be the underlying problem?

Ah I'm sorry, I can't remember what I changed and don't have access to the code anymore.


Nested Models with optional members fail on 1.6.1

Bug
Defining a model with Optional nested recursive submodels is throwing a required value error when validating the model.  The nested models also contain nested submodels that container other recursive models potentially.  This works on 1.6.0 and all versions prior that we have been using the last year. It only breaks when we move to version 1.6.1
Here is a sample of the model that we are using with basic one letter named attributes.
class Spec(BaseModel):
    spec_fields: List[str] = Field(..., alias="fields")
    filter: Optional[str]
    sort: Optional[str]

class PSpec(Spec):
    g: Optional[GSpec]

class GSpec(Spec):
    p: Optional[PSpec]

class Filter(BaseModel):
    g: Optional[GSpec]
    p: Optional[PSpec]

Trying with the following values will generate an error
        data = {"p": {"sort": "some_field:asc", "fields": []}}
        result = Filter(**data)

The error generated is:
E   pydantic.error_wrappers.ValidationError: 1 validation error for Filter
E   p -> g
E     field required (type=value_error.missing)

It seems to be disregarding the Optional attribute for GSpec field in the nested PSpec model.
Same code on 1.6.0 works as expected.
Thanks!
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
pydantic version: 1.6.1
            pydantic compiled: True
                 install path: /Users/..../.local/share/virtualenvs/..../lib/python3.7/site-packages/pydantic
               python version: 3.7.3 (default, Mar 27 2019, 09:23:15)  [Clang 10.0.1 (clang-1<IP_ADDRESS>)]
                     platform: Darwin-19.5.0-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions', 'email-validator']

import pydantic

...

Hi @roycollins
I can't reproduce your issue with the code sample (python 3.7 and 3.8) :/
I just added PSpec.update_forward_refs() as it is impossible to run without it.
from pydantic import BaseModel, Field
from typing import List, Optional

class Spec(BaseModel):
    spec_fields: List[str] = Field(..., alias="fields")
    filter: Optional[str]
    sort: Optional[str]

class PSpec(Spec):
    g: Optional['GSpec']

class GSpec(Spec):
    p: Optional[PSpec]

PSpec.update_forward_refs()

class Filter(BaseModel):
    g: Optional[GSpec]
    p: Optional[PSpec]

data = {"p": {"sort": "some_field:asc", "fields": []}}
print(repr(Filter(**data)))

output
Filter(g=None, p=PSpec(spec_fields=[], filter=None, sort='some_field:asc', g=None))
``

Here is some more details to make it work. It seems if I import the models from another file then I get the error. If I run the models in the same file then I dont have any issues.  Here are the 2 files that I am using.
If I run the files with pydantic 1.6.1 it throws the error above. If I downgrade to the 1.6.0 version then it works.
Archive.zip

Hi @roycollins
I just tried quickly and it indeed didn't crash in v1.6. Moving some code of ModelField.prepare into a dedicated method breaks the code. But I'm not convinced your code is supposed to work when you use GSpec without quotes to make as it is a forward ref.
from __future__ import annotations
from pydantic import BaseModel, Field
from typing import List, Optional

class Spec(BaseModel):
    spec_fields: List[str] = Field(..., alias="fields")
    filter: Optional[str]
    sort: Optional[str]

class PSpec(Spec):
    g: Optional[GSpec]

class GSpec(Spec):
    p: Optional[PSpec]

PSpec.update_forward_refs()

class Filter(BaseModel):
    g: Optional[GSpec]
    p: Optional[PSpec]

data = {"p": {"sort": "some_field:asc", "fields": []}}
print(repr(Filter(**data)))

this crashes indeed!
And in fact when I remove the from __future__ import annotations, the error changes to be
NameError: name 'GSpec' is not defined

If you change class PSpec into
class PSpec(Spec):
    g: Optional['GSpec']

can you confirm it works as expected please?

Yes removing the future for delayed annotations does indeed work if I define the types as a string. I am perfectly fine converting the type annotations to strings to make it work.
I was more concerned that it stopped working on 1.6.1 when it worked on 1.6.0.  It does seem to be a change with the way nested models are instantiated when they are optional only.  If they are not optional it seems to work.
There is supposed to be support for python 3.7 delayed annotations per the last block of the docs:
https://docs.google.com/document/d/1jbHNCcAyMC29-2ooBcLz0ROaA0TOfPX-JL2e9klZJEE/edit#
I would be willing to update that part of the docs if it is not meant to support Optional.
Thanks for taking the time to help get to the bottom of this issue.

You can keep the __future__ import. Just have to change the type into string as it doesn't exist yet.
I'm more concerned on "why it used to work before?" personally x)
I can have a deeper look maybe but I'm afraid it's going to take quite some time for no real benefit.

@roycollins I opened #1752. Feel free to try it out and comment.
Sorry for the regression.

Thank you for the quick response. I tested out the commit and it works perfectly!!


Two bugs

hello iizoo :) ,
thanks for this amazin plugin, it's a great tool for writing.
I have recently found two bugs in the console:
One thing I don't know what it's causing (SRY) but the other bug is a blank mno folder sidebar:

Edit: Sorry for the wrong FR. I changed the title of this issue, it's working with this plugin:
https://github.com/Raphlette/obsidian-new-tab-plus

Thank you for using the plugin and reporting the issues!
Regarding the first error, I am currently investigating it. It likely does not impact the functionality, but I will look into it further.
As for the second error, I have attempted to fix the suspected cause in v0.6.1. The issue was occurring when sorting items chronologically in Folder View if there were two or more subfolders present. Please test the new version and let me know if the problem persists.


OIDC: userinfo_endpoint is RECOMMENDED, but not REQUIRED

According to https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata, the userinfo_endpoint is RECOMMENDED, but not REQUIRED.
So, if the openid-configuration, for whatever reason, does not include a userinfo_endpoint, no userinfo should be requested.
Accordingly,

the sub claim from the id token cannot be verified against the sub property from the userinfo response,
no userinfo is stored in request.context.<oidc_name>.

Are issues that are on a milestone not automatically closed if the linked PR is merged?

Works, comfirmed with an Azure B2C application without an userinfo_endpoint within the configuration.


Disable some log parts

MODSEC-175: It would be handy to have an option to disable logging of some parts of the H part. Typically: Stopwatch, Producer, Server, Sanitised-Request-Headers.
In some cases, we are not interested in these fields. On sites with a lot of alerts, this can spare several GB.

Disable logging of parts of H part was implemented, merged to master and included on 2.9.2:
{dis|en}able-stopwatch-logging: https://github.com/SpiderLabs/ModSecurity/issues/1067
{dis|en}able-server-context-logging: https://github.com/SpiderLabs/ModSecurity/issues/1069
{dis|en}able-dechunk-logging: https://github.com/SpiderLabs/ModSecurity/issues/1068
{dis|en}able-handler-logging: https://github.com/SpiderLabs/ModSecurity/issues/1381
{dis|en}able-collection-delete-problem-logging: https://github.com/SpiderLabs/ModSecurity/issues/1380
{dis|en}able-server-logging:  https://github.com/SpiderLabs/ModSecurity/issues/1070
{dis|en}able-filename-logging: https://github.com/SpiderLabs/ModSecurity/issues/1065
Thanks @marcstern and @zimmerle.


Support for TinyMCE 7 and beyond

We've just noticed that TinyMCE has taken the same approach as CKEditor and made version 7 available under GPLv2 or alternatively a commercial license  (https://github.com/tinymce/tinymce/issues/9453). I couldn't see any mention of this in issues here and was wondering whether you will aim to support TinyMCE 7 and subsequent releases?

We are not in a rush to support the most recent versions of TinyMCE, we just added TinyMCE 6 compatibility in django-tinymce 4.0. But yes, at some point this project will have no other choices than follow TinyMCE development and hence its license policy. As long as some devs are keen to contribute to this package, even in a GPL context, it will continue to support TinyMCE evolution.

Thanks @claudep, that's reassuring. It will be interesting to see whether the change to GPL pushes clients and developers away to different libraries. 🤞
Just noting here for reference that the EOL for open source v6 is 31 October 2024 (1).
I see that the premium version of 6.8 is supported until 6 June 2025 (2) and the community edition of v5.10 received security updates for 7 months after premium support finished in April 2023 (3, 4), so perhaps we can hope that open source 6.8 will keep receiving fixes until June 2025!


Remove incorrect Public IP + UDR information for Batch VNet

@alfpark : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@alfpark thank you for the correction!

#sign-off


Create samskritabharatiusa.txt

@andykamath I am sorry, but the school:

does not have long-term IT-related courses (at least I failed to find them on the website)
seems to be an online school.
JetBrains does not provide students of online and/or distance learning schools with free licenses.
Also, I can not add a school to this repository if it does not offer IT-related long-term courses to the students. That's why I have to close this request without merging. Please reopen and comment if I am mistaken. Thank you for your understanding!


bugfix: fix iptables clean deprecated rules error

Pull Request Description
Describe what this PR does / why we need it

Fix clean iptables deprecated rules error, cause panic while running in a environment without "RAMA-XXX" deprecated chains and rules.
Change alpine image version to 3.12 to avoid DNS resolving error, reference to https://github.com/alpinelinux/docker-alpine/issues/185.

Does this pull request fix one issue?

Describe how you did it
Need to ensure chains exist first and clean all the deprecated chains and rules.
Describe how to verify it
Update to newest image in "Rama" environment
Special notes for reviews

Since this is a broken change, downward compatibility should be be our concern.


make GCC version detection OS-independent

Previous design put GCC version detection inside of OSNAME 'WINNT'.
However, such detections are required for 'Linux' and possibly other
OS'es as well. For example, there is usage of the GCC versions
in Makefile.arm64. When compiling on Linux machine, in the previous
design, Markfile.arm64 will not know the correct GCC version.
The fix is to move GCC version detection into common part, not
wrapped by anything.
Signed-off-by: Guodong Xu<EMAIL_ADDRESS>

Indeed, thanks, this was quite an embarassing mistake in the makefile. Perhaps there should be comments on these kind of nested endifs to make it clearer where each conditional block ends.


⚠ ClamAV has degraded performance

In a4eccde, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: ClamAV performance has improved in 03c13f7 after 11 minutes.


Document installing runner specific dependencies

@m-mohr  quick PR to add some dask/vito dependency related docs
Mainly I want to check changing the "Dask" extra identifier to "dask" (which is the more standard way for install extras)


Invalid Number of Parameters on manageAppointmentAsync Function

On "view appt dialog" and "view appt history" receiving this error message when invoking onClick function.
Error managing appointment Error: Invalid number of parameters. (0x8002000E)

@rknamba - Can you check your implementation again? I've reviewed the Cerner Wiki and the code appears to be correct.
Each function for the appointment action takes the same parameter.

PexSchedulingActions
The PexSchedulingActions object is used to launch various Cerner Scheduling functions. Available since 2015.01.01 and for 2012.01 via exception package 88327.
A PexSchedulingActions object is created by invoking window.external.DiscernObjectFactory("PEXSCHEDULINGACTIONS"). It supports the functions listed below each of which accepts the sch_event.sch_event_id for the appointment as the only parameter and returns a boolean value indicating whether the action was performed. If the return value is true the action was performed. Otherwise it was not. If the action was not performed there is no way to tell whether the user cancelled the action or an error occurred.

https://wiki.cerner.com/display/public/MPDEVWIKI/PEXSCHEDULINGACTIONS


[Feature]: Required Access to resource's OpenAPI attributes whose validation is failing

Suggested Behavior
Hello, I am currently using OpenAPI-core with my Flask app and I am customizing validation error responses as per following comment.
The issue I am facing is that there are some fields in request which holds sensitive information, e.g credentials, Auth-tokens.
When the validation fails on these fields we get an error string which includes value of these sensitive fields.
E.g consider following response,
{
    "errors": [
        {
            "class": "<class 'openapi_core.validation.schemas.exceptions.InvalidSchemaValue'>",
            "status": 400,
            "title": "Value {'flag': 'MyPassword1234'} not valid for schema of type object: (<ValidationError: \"'MyPassword1234' is too short\">,)"
        }
    ]
}

As we can see in above example, the password of user is exposed as part of response.
I do want to add validation for these fields however I don't want the values of these fields to be send as a response.
For that I checked whether there is a flag in OpenAPI which marks a field as sensitive and found this issue which suggested usingx-pii: true field in the yaml
Also I used FlaskOpenAPIErrorsHandler to fetch the error object and see if we get details of the flags which are set for the field where the validation failed.
Following is my Flask code
#!/usr/bin/python3
"""Test server."""

from flask import Flask, request, jsonify
from openapi_core.contrib.flask.decorators import FlaskOpenAPIViewDecorator, FlaskOpenAPIErrorsHandler
from openapi_core import Spec

# Custom Error Handler block
class ErrorHandler(FlaskOpenAPIErrorsHandler):
    """"Custom Error Handler"""
    def handle(self, errors:list):
        return jsonify({
            "causedBy" : [self.handle_error(error) for error in errors]
        }), self.OPENAPI_ERROR_STATUS.get(errors[0].__class__, 400)

    def handle_error(self, error):
        """
        Converts error object into error string message

        :param error: Error object which stores exception message
        :type error: Exception object
        :return: Error message string corresponding to error object
        :rtype: str
        """
        if error.__cause__ is not None:
            error = error.__cause__
        # TODO: If the field in error object has x-pii: true, return a generic string which does not include it's value
        if not (hasattr(error, "value") and hasattr(error, "type") and hasattr(error, "schema_errors")):
            return str(error)
        return f"Value(s) {error.value} not valid for schema of type {error.type} errors: {', '.join([err.message for err in error.schema_errors])}"

SPEC = "test.yaml"
obj = ErrorHandler()
openapi = FlaskOpenAPIViewDecorator.from_spec(Spec.from_file_path(SPEC), openapi_errors_handler=obj)

app = Flask(__name__)

@app.route("/test", methods=["POST"])
@openapi
def read_permission():
    """Test function"""
    return jsonify({
                    "flag stri_normal_json": request.json.get("flag", 1)
                })

if __name__ == "__main__":
    app.run(host="<IP_ADDRESS>", port=345, debug=True)

And following is the Yaml file
openapi: '3.0.2'
info:
  title: Test Title
  version: '1.0'
servers:
  - url: http://localhost:345/
paths:
  /test:
    post:
      requestBody:
        content:
          application/json:
            schema:
              type: object
              required:
                - flag
              properties:
                flag:
                  x-pii: true
                  type: string
                  pattern: "^[\\w.-]*$"
                  minLength: 6
                  maxLength: 20
      responses:
        200:
          description: Sample response
          content:
            application/json:
              schema:
                type: object
                properties:
                  flag stri_json:
                    type: string
                    minLength: 6
                    maxLength: 20

Please check the TODO comment inside handle_error method.
If the validation of field flag fails and if we have access to attributes of flag yaml properties e.g below properties
                flag:
                  x-pii: true
                  type: string
                  pattern: "^[\\w.-]*$"
                  minLength: 6
                  maxLength: 20

Then we would be able to have better customization and control over the response message.
Why is this needed?
Many times we need to have better control over the response generated on the failure of validation.
Current error messages generated by openapi-core exposes the field contents as part of response which makes openapi-core useless if there are sensitive fields in request body which needs to be validated.1
Having access to OpenAPI attributes of any field would be very helpful in generating custom response messages and would help us to perform validation on fields with sensitive data and also not expose the sensitive information to the response.
References
https://github.com/OAI/OpenAPI-Specification/issues/2190
Would you like to implement a feature?
Yes

Hi @rohan-97
There's not need for extension; it's a use case for a password format defined in OpenAPI 3.x
I would see this as a format unmarshaller which returns extended string type (similar to SecretStr in pydantic) and converting to string by default will give us masked string.
As a workaround for now I would catch/override InvalidSchemaValueexception.

Hello @p1c2u ,
I think the control won't hit format unmarshaller.
However I was able to get the resource's openapi attributes by checking attributes of error.schema_errors objects.
schema_errors objects had following attributes
schema: provides detail about OpenAPI attributes
validator: Provides the attributesname on which the validation failed e.g minlength, pattern
absolute_path: path of field name in entire OpenAPI schema
Using above attribute error.schema_errors[0].schema,  I am able to query attributes of the validation which is failing.


Selecting parent node checkbox as intermediate

I want to make parent node checkbox as intermediate selection on the selection of child nodes? I dont see any way to do it. can you please help me with that.

I would imagine in your node renderer, you could run a check that looks through all the children in the node to see if they are selected. If some but not all are selected, you could add a class to the component that marks it as intermediate.
const allSelected = node.children.every((child) => child.isSelected)
const someSelected = node.children.some((child) => child.isSelected)

if (someSelected && !allSelected) {
  // Add your intermediate style
}

If this works for you, will you please close the issue. Thank you and good luck!


No syntax error highlighting

I have installed the Omnisharp package and followed the instructions regarding Kulture and Python, but Omnisharp doesn't seem to work.
I guess the problem resides  in setting Python properly. I installed the 2.7.9 version from the Python website and added its path from here : My Computer > Properties > Advanced > Environment Variables :

Do you have any thorough tutorial on how should Omnisharp be installed ?
Note : I'm on Windows and I use Unity3D ( wth the Unity packages installed)

Have you tried a reboot after changing the path?
On Saturday, 17 January 2015, sonoshee-desu<EMAIL_ADDRESS>wrote:

I have installed the Omnisharp package and followed the instructions
regarding Kulture and Python, but Omnisharp doesn't seem to work.
I guess the problem resides in setting Python properly. I installed it
from the Python website and added its path from here : My Computer >
Properties > Advanced > Environment Variables :
[image: screenshot_1]
https://cloud.githubusercontent.com/assets/10574869/5789521/fcb61fce-9e66-11e4-9a3f-b97ead30abac.png
Do you have any thorough tutorial on how should Omnisharp be installed ?
Note : I'm on Windows and I use Unity3D ( wth the Unity packages installed)
--
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-sublime/issues/107.

Yes I did reboot both Sublime and Windows.
How can I know for sure that Omnisharp is working ? I try to intentionally type a syntax mistake and wait for some kind of feedback, but nothing happens.

Try putting spaces on your lines of code then try a format document from
the menu to see if it does it. You can also see results in View - Show
Console
On Saturday, 17 January 2015, sonoshee-desu<EMAIL_ADDRESS>wrote:

Yes I did reboot both Sublime and Windows.
How can I know for sure that Omnisharp is working ? I try to intentionally
type a syntax mistake and wait for some kind of feedback, but nothing
happens.
--
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-sublime/issues/107#issuecomment-70377516
.

They said in the console that there was an error while formatting the document :

You need to open the folder with the solution file in it. If you have
multiple solutions in the same folder then you need to create a sublime
project file pointing to the right sln file. It's explained more in the
readme
On Saturday, 17 January 2015, sonoshee-desu<EMAIL_ADDRESS>wrote:

They said in the console that there was an error while formatting the
document :
[image: screenshot_1]
https://cloud.githubusercontent.com/assets/10574869/5789889/3319ee1a-9e76-11e4-8ebd-5085189febfd.png
--
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-sublime/issues/107#issuecomment-70378001
.

Ah I followed the README.md instructions ( which I first didn't know exists) and OmniSharp works.
Is there any way of making the syntax highlighting enabled while I'm typing, because by default the syntax is only checked when I save the file.
Nevertheless, I must say it's an extremely helpful package!

Not whilst you type I'm afraid. You could change the source code if you
like so it does that locally for you. So instead of on post save its on
modified
On Saturday, 17 January 2015, sonoshee-desu<EMAIL_ADDRESS>wrote:

Ah I followed the README.md instructions ( which I first didn't know
exists) and OmniSharp works.
Is there any way of making the syntax highlighting enabled while I'm
typing, because by default the syntax is only checked when I save the file.
Nevertheless, I must say it's an extremely helpful package!
--
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-sublime/issues/107#issuecomment-70378943
.

There is a config setting that you can turn on and off now for this

Hi jchannon, you said there was a config setting for checking syntax errors. Where is it?

I think we changed it again. Sorry
On Saturday, 19 September 2015, sonoshee-desu<EMAIL_ADDRESS>wrote:

Hi jchannon, you said there was a config setting for checking syntax
errors. Where is it?
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-sublime/issues/107#issuecomment-141670772
.

Ah okay but is there any solution for me to get syntax error highlighting? I desperately need it :p

No. You only get it when saving the document.
On Saturday, 19 September 2015, sonoshee-desu<EMAIL_ADDRESS>wrote:

Ah okay but is there any solution for me to get syntax error highlighting?
I desperately need it :p
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-sublime/issues/107#issuecomment-141681057
.


Add API to allow macros/modules to trigger line of sight check between tokens

Currently, the API provides functions to allow triggering a LoS check between points.
It would also be useful to be able to trigger this between tokens, as per the new LoS tool - either by an API function which takes two tokens as parameters (e.g. by id) and shows the LoS ruler between them, or by API functions which take a single token as a parameter and display the LoS ruler once both are set.
My use case is an in-person game using Material Plane - an infra-red tracking setup for physical minis which comes with a "pen" that can be used to trigger macros.  While it's possible to treat this as a mouse (and use the LoS tool via the current UX), it's clunky, so it would be much smoother for me to be able to select the tokens using the pen and trigger a macro to show the LoS calculation.
This could also be used to, e.g. provide a macro to allow the user to draw the LoS ruler between their controlled token and a targeted token.

Thanks for the feature request. This has now been added in v0.3.3.


[PROPOSAL] Add an ability to prepare chunk rows from query before processing each row separately

Prerequisites
Versions

PHP version: 7.4.11
Laravel version: 8.10.0
Package version: 3.1.23

Description
Hi, team.
Could you please add new method like prepareChunkRows before calling this logic:
https://github.com/Maatwebsite/Laravel-Excel/blob/3c5cd51d36fa056c5a98c5802ea356424d00afc8/src/Sheet.php#L582
Right now I'm using FromQuery concern and my query it's union of the many tables which returns poor id, and type column values.
After that the query executing through chunk method and I should get real data by poor id and type (I group my poor data like: [type_1 => [id1, id2], type_2 => [id3, id4], ...] to decrease queries. And after that I can process each row.
Example
Possible solution:
Add new concern like 'PrepareQueryChunk' or something else and add before https://github.com/Maatwebsite/Laravel-Excel/blob/3c5cd51d36fa056c5a98c5802ea356424d00afc8/src/Sheet.php#L442 or https://github.com/Maatwebsite/Laravel-Excel/blob/3c5cd51d36fa056c5a98c5802ea356424d00afc8/src/Sheet.php#L582

Why can't you use map() for this? You can return multiple rows in there as well

@patrickbrouwers because map used for each row separately.

I'm wondering if in your cause it wouldn't be better to use FromCollection instead.

I used FromCollection concern before I got the problem about memory limit. Therefore I want some improve my export and use FromQuery concern and I think it will be cool to have method for preparing result of the chunk.

I'm open for a PR which adds a prepareRows(array $rows) method (no need for a concern in this case). Which can be called in appendRows

@patrickbrouwers I think it can be closed. And please make a release with merged PR above. Thanks.

Will be included in next release


Server for templates

Just an idea I had today, a separate repo for templates.
How it works:
Separate GitHub repo with a templates.json in the root. This file would contain a list of the available templates, generating the dropdown. Clicking on a choice fetches the fields from the JSON file on the repo. Updates would be git commits.

Added in ff1feb1ce7e868588ed2fa665047031ed59f0b9b


Switch from color_cycle to prop_cycle in tomography plotting tools

This was a name change in matplotlib 1.5 of the default color cycle, and the old name is now deprecated.
This PR addresses https://github.com/QInfer/qinfer-examples/issues/4#issue-407817702

This matplotlib>=1.5 requirement should be listed somewhere in the project. A good place might be under optional requirements in setup.py

Hmm, it's not actually a requirement; it's imported in try blocks. Maybe we should check the version inside the try block and raise if < 1.5.

Coverage remained the same at 75.631% when pulling 397c98be54e592df893c5963ff87b59200d2bb04 on fix-color_cycle into 8170c84a0be1723f8c6b09e0d3c7a40a886f1fe3 on master.

Coverage remained the same at 75.631% when pulling 397c98be54e592df893c5963ff87b59200d2bb04 on fix-color_cycle into 8170c84a0be1723f8c6b09e0d3c7a40a886f1fe3 on master.

Right, but matplotlib is an optional requirement and can be listed as such.
I also imagine it should be listed in the docs somewhere.
Actually, now that the matplotlib website, 2.2.x is the LTS dual py2/3 release. It appears that 1.5 was the last 1.x release, which is no longer supported. I'd probably still put a check in somewhere, but you might not need to put it in the docs.

I checked travis, its reporting failures on doctest for both 2.7 and 3.5
3.5: Could not import extension matplotlib.sphinxext.only_directives, which was apparently removed in matplotlib 3 https://github.com/matplotlib/matplotlib/pull/11295
2.7: ImportError: cannot import name l_ caused at conf.py#L18

I tried this, but the cycler can't be indexed
line 177, in plot_rebit_prior
    c=pallette[0],
ValueError: Can only use slices with Cycler.__getitem__

Maybe  something like pallette = [x['color'] for x in plt.rcParams['axes.prop_cycle']] instead?


[CPP-386] More getCanonicalQlClass() goodness

The new node types were uncovered by running the following query against torvalds/linux, vim/vim and wireshark/wireshark:
import cpp

from Element e
where not exists(string s | s = e.getCanonicalQLClass() | s != "???")
select e, e.getLocation(), concat(e.getAQlClass(), ", ")

I'm still trying to create test cases for these; the LGTM console is less than helpful.
The merge conflict is with https://github.com/Semmle/ql/pull/1647 and is trivial to resolve.

I haven't been part of the latest "what's canonical" discussions, so there might be additional feedback from those who were. @dave-bartolomeo ?

@geoffw0 We don't want to overrides for MetricFile and its cousins. Every File is a MetricFile.

We don't want to overrides for MetricFile and its cousins. Every File is a MetricFile.

Good point.  I was seeing some Files with no getCanonicalQLClass(), but MetricFile is indeed the wrong choice for them.  I think a File in the snapshot that is neither a HeaderFile, CFile or CppFile should have getCanonicalQLClass() of just "File".

@zlaski-semmle, don't forget to rebase so the tests can run.

Apart from the comment I've just made, my concerns have been addressed.

Something's gone wrong with a merge here.  I would suggest you try a "rebase" instead of merging next time you have a conflict on a PR.

Something's gone wrong with a merge here. I would suggest you try a "rebase" instead of merging next time you have a conflict on a PR.

I thought I did rebase  (I used the git rebase command), but the result surprised me also.  Perhaps you could provide herein the exact incantation to rebase, say , branch zlaski/cpp386?
My mistake aside, is zlaski/cpp386 mergeable as is?

Perhaps you could provide herein the exact incantation to rebase, say , branch zlaski/cpp386?

I'd use git rebase --onto upstream/master zlaski/cpp386~X zlaski/cpp386, where X is the number of commits from the top to rebase (there are shorter forms of this command, but this one should do what you want without surprises).

Perhaps you could provide herein the exact incantation to rebase, say , branch zlaski/cpp386?

I'd use git rebase --onto upstream/master zlaski/cpp386~X zlaski/cpp386, where X is the number of commits from the top to rebase (there are shorter forms of this command, but this one should do what you want without surprises).

Looking at the commits for this PR, I now see that my commits are interspersed with others' commits that I pulled in with my failed rebase

Perhaps you could provide herein the exact incantation to rebase, say , branch zlaski/cpp386?

I'd use git rebase --onto upstream/master zlaski/cpp386~X zlaski/cpp386, where X is the number of commits from the top to rebase (there are shorter forms of this command, but this one should do what you want without surprises).

I tried the the git rebase command you suggested but got a bunch of weird conflicts in C#, Python, etc. (Perhaps I misunderstood what the value `X should actually be.)  So I did a hard rewind followed by 2 cherry-picked commits.

LGTM, but a test is failing. It looks like it's just a matter of accepting the new output.

It's a strange beast.  At any rate, I'm adding a direct link to the offending file here:
https://github.com/zlaski-semmle/ql/blob/zlaski/cpp386/cpp/ql/test/library-tests/ir/ir/PrintAST.expected

All right, I finally squashed the Jenkins test failure.  The PR should be ready to go.

@zlaski-semmle are the changes to docs/language/learn-ql/cpp/introduce-libraries-cpp.rst intended to be here, or has #1658 mistakenly been merged into this PR?

@zlaski-semmle are the changes to docs/language/learn-ql/cpp/introduce-libraries-cpp.rst intended to be here, or has #1658 mistakenly been merged into this PR?

The latter, I'm afraid.  I rebased the #1658 PR (successfully, it would appear) but then that spilled into this PR: somehow.

I made a total mess of things here, so needed to create a new PR:: https://github.com/Semmle/ql/pull/1753/.  Sorry for the inconvenience.


Update KubeFedCluster API optional fields

This updates the optional fields in KubeFedClusterStatus to make them pointers. It also makes KubeFedClusterSpec required as it was inadvertently changed previously.
Additionally, a couple more cluster conditions are added to the shared ValidKubeFedCluster test object (to complete all 4 possible conditions) as a result of changes for this PR.

this LGTM too..
Thanks @font

/lgtm