Provide a helpful error message when passing a string to label

I think this LGTM. Another possibility would be to make that function translate it into a keyword list as a shortcut since I imagine changing the label text is a fairly common occurrence

I've debated this and I'd love your opinion (and @dbernheisel) on the implications of that. My reasoning for not allowing that was simply to make it an easier "one option" interface. I'm primarily trying to keep it simple for designers so they can quickly follow it without needing to know the library in depth.

I think converting it to label text falls right in the sweet spot of useful magic because it's hard to use wrong, and easy to understand at first glance.
For example, I wouldn't do something like: label: "my-css-class" because that just looks weird (to me at least). I think most designers would agree. So IMO I think it would be hard to misuses and it seems easy to understand when reading.

If the docs are clear, I could go either way, but I think the extra magic makes sense.

Seems reasonable, updated!


Change character set comparison function in sort-alternatives?

Since we need to be able to sort character sets in sort-alternatives, I implemented a comparison function in #320. However, I recently discovered that this function doesn't sort that way you'd expect.
Example: sort-alternatives wants to swap the following alternatives:
var re = /\b(?:sample|(?:un)?stable)\b/u;

This is quite strange to me because the first characters of sample and (?:un)?stable are [s] and [su] respectively, and I think [s] should come before the character set [su].
Honestly, I don't know whether this is a bug or not. We don't specify how alternatives are sorted, so technically any order is correct. What do you think @ota-meshi?

I think [s] should come before the character set [su].

I agree with you :+1:

Resolved by #423.


frontend: fix nullable node status

nice, thanks for adding this @farodin91. Was this change prompted by some issue you've experienced?

I did a node rotation and the node list was broken during after a new node was created.

@sniok updated.


Use chi.alt instead of x for lowercase chi

This should close https://github.com/google/roboto/issues/18
I decided that to change the name of the "chi.alt" glyph was more
trouble than it's worth.
cc @roozbehp

LGTM


add Givens-based orthonormal matrix transform and inverse plus Jacobian

Description
We would like to have an orthonormal matrix type for constrained variables in Stan.  That requires defining the transform and its inverse plus log Jacobian determinant in the math library first.
Among the applications are factor models and parameterizations of positive definite matrices in terms of eigenvectors.
This would amount to a data type orthonormal[P, Q] in Stan and three functions in the math library, which have roughly this signature (see, e.g., corr_matrix_free.hpp and corr_matrix_constrain.hpp in directory stan/math/prim/fun.   This amounts to two files with a total of three function signatures.
orthonormal_free.hpp
Eigen::Matrix<T, 1, -1> orthonormal_free(const Matrix<T, -1, -1>& y);

orthonormal_constrain.hpp
// w/o Jacobian adjustment
Eigen::Matrix<T, -1, -1> orthonormal_constrain(const Matrix<T, -1, 1>& x, int P, int Q);

// w Jacobian adjustment
Eigen::Matrix<T, -1, -1> orthonormal_constrain(const Matrix<T, -1, 1>& x, int P, int Q, T& lp);

Example
Reference
We can follow the code and transforms laid out here:
Arya A. Pourzanjani, Richard M. Jiang, Brian Mitchell, Paul J. Atzberger, Linda R. Petzold. 2021. Bayesian Inference over the Stiefel Manifold via the Givens Representation. Bayesian Analysis.
Derivative reduction
Algorithm 1 has the constraining transform, where angles are defined by
theta[i, j] = atan2(y[i, j], y[i, i]).

Equation (4.5) lists the Jacobian adjustment, which involves terms that on the log scale are equal to
(j - i - 1) * log(cos(theta[i, j]))

We can exploit the fact that
d/du log cos(u) = -tan(u)

and the definition of theta[i, j] to reduce to
d/d.theta[i, j] (j - i - 1) * log(cos(theta[i, j])) =  (j - i - 1) * y[i, j] / y[i, i].

Auxiliary variable approach
What I don't see how to do immediately is deal with the overparameterization and unit normal "prior" that makes Pourzanjani et al.'s approach feel very much like Marsaglia's approach to unit vectors that we currently employ.
Unconstraining transform
We also need the transform to go from the constrained to the unconstrained spaces (for initialization).  I couldn't find it in the paper.
Original GitHub repo
More details can probably be extracted from Pourzanjani's GitHub repo, TfRotationPca.
Relation to unit vector coding
We should be able to code unit vectors as (p, 1) orthonormal matrices.  The two approaches with auxiliary variables are closely related, and may be identical.
Other repos
We also need to add the doc for the transform to the reference manual (docs repo) and the actual data type to the language and code generate (stanc3 repo).
Unit and negative unit determinants
@bgoodri and Pourzanjani were discussing the fact that the Givens approach leads to solutions which can have positive or negative unit determinants.  Maybe we could constrain to one in the prior, but in higher dimensions, this turns into combinatorial multimodality.
Alternatives
There is more than one way to do this.  Pourzanjani's paper considers the geodesic Monte Carlo (GMC) approach of Byrne and Girolami.  It's also possible to use Cayley transforms.
Expected Output
Invertible transforms with Jacobians so we can sample uniformly overly over Stiefel manifolds.
Current Version:
v4.1.0

Unconstraining transform
We also need the transform to go from the constrained to the unconstrained spaces (for initialization). I couldn't find it in the paper.

I don't think this is technically invertible on the entire real line.  Just like the unit_vector approach isn't really a bijection since the origin is left out (see https://math.stackexchange.com/a/2629480).
@betanalpha commented on the Pourzanjani paper on discourse:

This method does not fully parameterize the space of orthogonal matrices. It instead defines a chart over a Steifel manifold that will necessarily always have a slice of parameters that it cannot describe (i.e. the chart is not isomorphic to the entire space). All of the issues with reflections and reorientations are consequences of this.

from https://discourse.mc-stan.org/t/new-transform-for-orthonormal-matrices-in-stan/2981/2
Is there a clever way to ignore these spaces for invertibility defined only on the subset?

What we do with the unit vector coding is to make the transform f (constrained to unconstrained) one to one and the inverse transform f_inv many to one, making sure that f_inv(f(x)) = x for constrained x.

Upon further review, can this method generate square orthogonal matrices with negative determinants? I can't seem to generate one when doing this in R:
rstan::expose_stan_functions("https://raw.githubusercontent.com/pourzanj/TfRotationPca/master/Stan/UniformStiefel.stan")
sign_det <- 1
counter <- 1
while (sign_det == 1) {
  theta <- runif(3, min = -pi, max = pi)
  W <- area_form_lp(theta, n = 3L, p = 3L)
  sign_det <- sign(det(W))
  counter <- counter + 1
}

What we do with the unit vector coding is to make the transform f (constrained to unconstrained) one to one and the inverse transform f_inv many to one, making sure that f_inv(f(x)) = x for constrained x.

Yea, my point is more semantics rather than practical. I don't think it's called a bijection since it cannot be one-to-one. There's more at https://discourse.mc-stan.org/t/divergence-treedepth-issues-with-unit-vector/8059. In fact that thread suggests using a normal(0, 0.1) rather than standard normal for the unit vector transform.
I think this rather subtle point should be made more clear in the docs for unit vector. Similarly, I'm curious when this Givens-based orthonormal transform shows divergences or treedepth issues, which we can document as well.
Here's stan code for the Cayley method https://github.com/michaeljauch/cayley/blob/master/stiefel_unif.stan. I'm not sure if it's correct. The corresponding paper is https://arxiv.org/abs/1810.02881.

Yea, my point is more semantics rather than practical. I don't think it's called a bijection since it cannot be one-to-one.

Correct, the “embedding methods” are not bijections and we really shouldn’t be using the same naming for them, including in the code.
The embedding space to embedded surface is a projection/surjection, and the inverse “lifting” map isn’t uniquely well-defined.

I think this rather subtle point should be made more clear in the docs for unit vector. Similarly, I'm curious when this Givens-based orthonormal transform shows divergences or treedepth issues, which we can document as well.

If I remember correctly someone experimented with a proper embedding approach in the Forums (not the Givens or Cayley local charts but an actual embedding like the unit vector implementation) but was plagued by intense divergences and other fitting problems. The
The problem in all of these approaches is that the data will inform posterior concentration on only the embedded surface, without no constraints on the behavior transverse to the embedding surface.  Only the default prior that is implicitly added affects those, and the intersection of these two constraints can lead to a lot of cone-like geometries that diverge at the tip.

Thanks, @betanalpha, that's all really helpful.

my point is more semantics rather than practical.

I agree.  We shouldn't call a non-bijective function a bijection!  I think TensorFlow Probability is the one who introduced the "bijector" data structure to encapsulate a transform, an inverse defined over the range of the transform (and possibly extended to a larger domain, hence the non-bijectiveness), and the log absolute Jacobian determinant of the inverse.

In fact that thread suggests using a normal(0, 0.1) rather than standard normal for the unit vector transform.

I take it that makes the actual parameters more unit scale?

I think this rather subtle point should be made more clear in the docs for unit vector

@spinkney If you could open a docs issue with a suggestion on which docs to fix (reference manual or user's guide or both) and what you'd like to see, I can write it.
And thanks for the reference to the other embeddings.
This kind of thing is why we're thinking about adding user-defined transforms to the language.  The only question there is how to make the result clearer than spreading transform logic over three blocks as we currently do.

This kind of thing is why we're thinking about adding user-defined transforms to the language. The only question there is how to make the result clearer than spreading transform logic over three blocks as we currently do.

In my opinion we make the users implement the transformation themselves, which they can already do.
parameters {
unconstrained_type unconstrained_name;
}
transformed parameters {
constrained type constrained_name = constraining_map(unconstrained_name);
}
Zero ambiguity and the constrained implemented wherever it’s most natural in the Stan program.
We can consider implement and explore maps with more direct names, for example “pos_real_to_real” instead of “log”, but that introduces the problem that there are many maps from the positive reals to the reals other than the log.  Users have to know why they’re using log instead of any of the other infinite possibilities, although this is already what the language largely forces users to do.
Personally I think much of the problem here is doing too much to abstract important model structure from users, which just facilitates unconsidered defaults.


The variable $input does not exist

see https://scrutinizer-ci.com/g/AgelxNash/functions/inspections/679cb6b4-e3f3-45c4-91b9-3aa95702c661/issues/files/src/utils.functions.php?status=fixed&orderField=path&order=asc&honorSelectedPaths=0#inspectioncomment-5940590

https://github.com/AgelxNash/functions/commit/8a7d45d7db1ab5bafc1516d76b9c0c4c8cb64625


How to get PR's created in Azure DevOps

Hello Team,
I am using the given instructions, and able to run a scan after downloading the Frogbot. Pipeline based on the shared template gets executed successfully but i dont see any PR's generated. What can be the issue. Logs from pipeline are as per below:
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
 23  2395   23   555    0     0    787      0  0:00:03 --:--:--  0:00:03   787
100  2395  100  2395    0     0   3397      0 --:--:-- --:--:-- --:--:--  3392
Downloading the latest version of Frogbot...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
 30 25.5M   30 7902k    0     0  4375k      0  0:00:05  0:00:01  0:00:04 8232k
100 25.5M  100 25.5M    0     0  11.7M      0  0:00:02  0:00:02 --:--:-- 19.1M
Frogbot downloaded successfully!
15:18:16 [Info] Frogbot version: 2.18.0
15:18:16 [Info] Running Frogbot "scan-repository" command
15:18:17 [Info] Running SCA scan for vulnerable dependencies...
15:18:17 [Info] Detected: npm.
15:18:17 [Info] Calculating npm dependencies...
15:18:26 [Warn] Ecosystem-Usage(Attempt 1) - Failure occurred while sending POST request to https://usage-ecosystem.jfrog.io/api/usage/report: Post "[https://usage-ecosystem.jfrog.io/api/usage/report"](https://usage-ecosystem.jfrog.io/api/usage/report%22): net/http: TLS handshake timeout
15:18:26 [Info] Ecosystem-Usage executor timeout after 0 attempts with 0 milliseconds wait intervals
15:18:27 [Info] Scanning 591 npm dependencies...
15:18:28 [Info] Waiting for scan to complete on JFrog Xray...
15:18:58 [Info] Xray scan completed
15:18:58 [Info] Frogbot "scan-repository" command finished successfully

Hi @jashan05,
Thank you for reporting this issue. Could you please provide more information about this issue? Kindly attach the log with 'debug' information by adding the environment variable 'JFROG_CLI_LOG_LEVEL': 'DEBUG'.
In Azure Repos, we only create pull requests for dependency issues (CVE) that have a fix in a released version. Are you aware of a security issue that we should have discovered and opened a pull request for? If so, could you provide the steps to replicate this issue?
Thank you for your assistance.

Hello @attiasas ,
Thanks for quick response !
By enabling DEBUG mode i see it is skipping the indirect dependency update. Is there any documentation which i can refer for more information on the behaviour of Frogbot.
Please see the logs below:
10:53:10 [Info] Xray scan completed
10:53:10 [Debug] Frogbot will attempt to resolve the following vulnerable dependencies:
 tough-cookie,
formidable
10:53:10 [Debug] Attempting to fix tough-cookie with 4.1.3
10:53:10 [Debug] Creating branch frogbot-tough-cookie-55a2094db87e0b506cfe7698b7536175 ...
10:53:10 [Debug] tough-cookie is an indirect dependency that will not be updated to version 4.1.3.
Fixing indirect dependencies can potentially cause conflicts with other dependencies that depend on the previous version.
Frogbot skips this to avoid potential incompatibilities and breaking changes.
10:53:10 [Debug] Running git checkout to branch: main
10:53:10 [Debug] Attempting to fix formidable with 3.2.4
10:53:10 [Debug] Creating branch frogbot-formidable-69bab9618eba86d7d6a53b9c9b97213f ...
10:53:10 [Debug] **_formidable is an indirect dependency that will not be updated to version 3.2.4.
Fixing indirect dependencies can potentially cause conflicts with other dependencies that depend on the previous version.
Frogbot skips this to avoid potential incompatibilities and breaking changes._**
10:53:10 [Debug] Running git checkout to branch: main
10:53:10 [Info] Frogbot "scan-repository" command finished successfully

Also another question, can it also create PR's if there are new versions detected in case the current used version doesnt have any vulnerabilities.
Best Regards
Jashan

@jashan05,
The Frogbot documentation currently doesn't describe in detail the strategies used for fixing vulnerable depedencies. Those strategies aren't the same across all package managers supported by Frogbot, and we're also refining them over time to produce the best outcome.
As for your second question, Frogbot currently focuses on fixing security vulnerabilities only, but allowing upgrades from versions that aren't vulnerable is something that we're considering for the future.
Please feel free to reach out if you have any more questions or if you need further assistance.

Thanks @eyalbe4 , will keep an eye out from new functionalities. Closing the issue


WIP: Use named_type_derive instead relying intrinsics

Test on stable from now on. Fix README to reflect current state.
The issue with core types #4 is not fixed, so this remains work in progress.
Signed-off-by: Florian Gilcher<EMAIL_ADDRESS>

Thanks a lot. I really think we do need a solution that also works with core/std types.
Ideally, the best way to get the impl of NamedType for the core/std types is to implement this in the upstream create (i.e. in named_type). However, it looks the crate has been abandoned since I checked it earlier this year (the GitHub account linked on creates.io has been deleted). This makes me a bit nervous to add it as a dependency, even if we could add the missing impls via proxy.
Considering the type name itself is currently currently only presented to the user and otherwise not used at all, I would actually propose we feature gate the call to std::intrinsics::type_name and return something like "<unknown>" if compiled with Rust stable.

Closing in favour of #20


VESPA_UNPRIVILEGED set to yes fails to run the vespa docker image

Setting VESPA_UNPRIVILEGED to yes and running the docker with a low privileged user fails due to failure in chmod/chgrp attempts.
To Reproduce
Steps to reproduce the behavior:

Create default-env.txt file with contents as follows:

cat <<EOF > default-env.txt
fallback VESPA_HOME /opt/vespa
override VESPA_USER vespa
override VESPA_UNPRIVILEGED yes
EOF

Run the docker as a vespa user (As of vespaengine/vespa:7.551.16 image, vespa has 999 uid and 996 gid):

docker run --rm -it -u 999:996 --name test-vespa-bug -v "$PWD/default-env.txt":/opt/vespa/conf/vespa/default-env.txt vespaengine/vespa:7.551.16
hostname: you must be root to change the host name
mkdir: cannot create directory 'libexec/vespa/plugins': Permission denied
chmod: cannot access 'libexec/vespa/plugins/qrs': No such file or directory
chmod: changing permissions of 'logs': Operation not permitted
chmod: changing permissions of 'tmp': Operation not permitted
chmod: changing permissions of 'var': Operation not permitted
mkdir: cannot create directory 'var/crash': Permission denied
chmod: cannot access 'var/crash': No such file or directory
mkdir: cannot create directory 'var/jdisc_container': Permission denied
chmod: cannot access 'var/jdisc_container': No such file or directory
mkdir: cannot create directory 'var/run': Permission denied
chmod: cannot access 'var/run': No such file or directory
mkdir: cannot create directory 'var/vespa': Permission denied
chmod: cannot access 'var/vespa': No such file or directory
mkdir: cannot create directory 'var/vespa': Permission denied
chmod: cannot access 'var/vespa/application': No such file or directory
mkdir: cannot create directory 'var/vespa': Permission denied
chmod: cannot access 'var/vespa/bundlecache': No such file or directory
mkdir: cannot create directory 'var/vespa': Permission denied
chmod: cannot access 'var/vespa/bundlecache/configserver': No such file or directory
mkdir: cannot create directory 'var/vespa': Permission denied
chmod: cannot access 'var/vespa/cache/config/': No such file or directory
Running /opt/vespa/libexec/vespa/start-configserver
Creating data directory /opt/vespa/var/zookeeper/version-2
/opt/vespa/libexec/vespa/start-configserver: line 160: /opt/vespa/var/jdisc_container/configserver.properties: No such file or directory
Creating data directory /opt/vespa/var/vespa/bundlecache/configserver
mkdir: cannot create directory '/opt/vespa/var/vespa': Permission denied

Expected behavior
I expect the vespa container to run as it would without any privileges
Screenshots
N/A
Environment (please complete the following information):

OS: Ubuntu 20.04
Infrastructure: Docker Engine (on Linux)
Versions:

Docker Engine: 20.10.8

Vespa version
7.551.16
Additional context
Another interesting aspect is, even when we do not change the 'run as' user, and run with default root user, by setting the VESPA_UNPRIVILEGED , the docker still fails:
docker run --rm -it  --name test-vespa-bug -v "$PWD/default-env.txt":/opt/vespa/conf/vespa/default-env.txt vespaengine/vespa:7.551.16
hostname: you must be root to change the host name
Running /opt/vespa/libexec/vespa/start-configserver
Creating data directory /opt/vespa/var/zookeeper/version-2
could not create pidfile /opt/vespa/var/run/configserver.pid: Permission denied
/opt/vespa/var/run/configserver.pid: Permission denied

(notice the lack of -u 999:996 parameter to docker run command)

The VESPA_UNPRIVILEGED is an option that developers can use when installing and running Vespa in their home directories. This is only used for development and testing at the moment and not documented as a general option to use when running Vespa in containers.

This has been solved for some time and Vespa does not need privileges https://docs.vespa.ai/en/operations/docker-containers.html


Where did MLPNone go?

I was using MLPNone as an activation function on my output layer. I just updated the via CocoaPods and now I am getting an error, Use of undeclared identifier 'MLPNone'. Was this removal on purpose?

@crobertsbmw MLPNone is in 1.0.10. Are you sure you updated it right?

I'm sorry I never returned to this thread. I'm going to close the issue. I don't remember if I had a problem with updating it correctly, or if I manually added it back in. Either way, this is no longer an issue for me.


Accumulo Start JUnit5 conversion

Converts Accumulo-start module from JUnit4 to JUnit5.

It looks like some of the changed tests use both JUnit4 and JUnit5 imports. I think we might have to keep some tests as JUnit4 as long as we are using Powermock. For those tests, it is probably best to not mix imports and just keep those tests as JUnit4. It looks files were created outside of temporary directories (possibly due to the mix of JUnit4 and 5).

Would moving the tests that use power mock to a separate file make sense? One the plus side all power mocks would be isolated and might be easier to identify and only those files would need junit4 - on the negative side testing would be split across multiple files which might require duplication in set-up, or just be confusing.  Something like
org.apache.accumulo.testing.powermock.foo - or whatever make sense

Would moving the tests that use power mock to a separate file make sense?

There's only one class that uses PowerMock in this module, and it's deprecated, to be removed in 3.0. I don't think it's worth spending too much time trying to get it working when it's being removed anyway. If necessary, we can keep this entire module using the vintage engine, and we can migrate to jupiter later. However... from what I've read, if you have both the vintage and jupiter engines on the class path, then the surefire plugin will run both kinds of tests anyway. So, we can keep just the one test class using the vintage stuff, and everything else can be migrated to jupiter. At least, I think that would work.

This is looking good. Just had the one comment. Also, I was only able to get these tests to run with surefire version M6 so if you could merge main (which is on version M6) into this branch that would be good.


Allow setting the contributor on a new ApiLimit

Overview
We couldn't create new ApiLimits via the admin because the contributor
field was read only. The contributor is now read only for editing, and
not for creation.
Connects #1149
Demo

Testing Instructions

Run ./scripts/server
Go to :8081/admin/api/apilimit/
Create a new ApiLimit. You should be able to select a contributor. Save.
 You should not be able to edit the contributor.

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Looking at this now.


First draft for the slack integration

Fix slack
another typo
another typo
Fix slack
Add channels to the form
another typo
Another set of changes
Make code more transparent
Fix tests
Add logging
logging
use logger
import logging
import logging
import logging
add assert
more logging
Fix channels
Fix channels
SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11 into master will decrease coverage by 0.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   71.02%   70.65%   -0.38%
==========================================
  Files         586      402     -184
  Lines       30635    12584   -18051
  Branches     3159     3152       -7
==========================================
- Hits        21758     8891   -12867
+ Misses       8766     3582    -5184
  Partials      111      111

Flag
Coverage Δ

#cypress
53.57% <ø> (-0.07%)
:arrow_down:

#javascript
59.24% <ø> (ø)

#python
?

Impacted Files
Coverage Δ

...rontend/src/SqlLab/components/QueryAutoRefresh.jsx
65.90% <0.00%> (-6.82%)
:arrow_down:

...set-frontend/src/dashboard/util/getDropPosition.js
92.06% <0.00%> (ø)

superset/utils/screenshots.py

superset/db_engine_specs/oracle.py

superset/views/log/api.py

superset/dashboards/commands/delete.py

superset/connectors/sqla/views.py

superset/views/sql_lab.py

superset/views/datasource.py

superset/examples/misc_dashboard.py

... and 173 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #11 into master will decrease coverage by 11.78%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #11       +/-   ##
===========================================
- Coverage   71.02%   59.24%   -11.79%
===========================================
  Files         586      402      -184
  Lines       30635    12584    -18051
  Branches     3159     3159
===========================================
- Hits        21758     7455    -14303
+ Misses       8766     4948     -3818
- Partials      111      181       +70

Flag
Coverage Δ

#cypress
?

#javascript
59.24% <ø> (ø)

#python
?

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


[Feature] Add a glowing parameter (boolean)

Is your feature request related to a problem?
When writing in white, the color does not appear white but gray-ish in the GUI. Working with resource packs that change the sign GUI itself, it can become annoying.
What solution would you like?
Having a new boolean parameter (not mandatory, to avoid breaking any code using older versions of the API) to enable glowing on the sign
Describe alternatives you've considered
—
Any additional comments?
I don't know which obfuscated method needs to be called but with the mappings I've found net.minecraft.world.level.block.entity.SignText#setHasGlowingText(boolean)

Good idea, I hadn't thought about that.
However, I'm currently on vacation but I will see if it's possible and implement it in about 2-3 weeks.

Perfect, enjoy your vacation!

But text is not glowing in the sign UI. This change is visible only on block itself

There is for sure a difference in the render of the text color in the GUI when the sign has the glowing effect. I've only tested this using white color, but there might be a difference with other colors too.

Wow, i tested it and there is really difference. My apologies

The option is implemented and will be pushed with the next update (today or in the next few days)

The new release is published, as soon as it's published on Maven (which takes a few hours) you can use the glowing option!


Default to https

I think it would make sense for the tool to automatically append https:// to the domain, since all websites should be served over https?
Maybe it could report if https was not available and continue to test http:// ?
Thanks for making this awesome tool!

done. Please your version to 0.0.16

Awesome!


chore(other): updated stencil to v4.19.2

updated stencil to v4.19.2
✅ Closes: COMUI-2775

This is as far as I could update stencil with trivial changes.
Stencil v4.20.0 and v4.21.0 will need bigger (and manual) changes in our e2e tests so I will do that in a separate PR.

I'm just waiting for the timemachine branch to be available so I can click around and make sure the app is still working as expected, but code changes look good.


Swiper in swiper doesnt work (Nested Swipers)

OS: Android 10
react-native-swiper v1.6.0
react-native v0.63
Nested swiper is not displayed, but what do I need - https://jsfiddle.net/imoskvin/s0fm20m6/
My code:

Did you find any solution to this?


Firefox support (which extension?)

So, I have this working great with Chrome 👍
But I cannot seem to get it working with what I presume is the Firefox extension mentioned in the README. Anyone had any luck with it?

Thanks for opening this. This should be fixed in v0.0.10. To upgrade, just download/run the new installer. Let me know if this doesn't work.

👍 Confirmed it's working with macOS 10.12, but not having as much success with the 10.13 beta (for either browser) on my home computer. That's unrelated to this, however. I'll look into messing with that later tonight.

Yeah. I just installed High Sierra also and am having problems too. I was about to open an issue, since I'm not sure on the fix yet and someone else might have an idea.

I find a config in firefox nightly.
security.webauth.u2f and security.webauth.u2f_enable_softtoken
All turn to true.
In demo page, It's work. but not have notification.

Firefox has its own internal software U2F implementation, though that is just intended for testing while they add HID support. That is what those config flags enable. If you want to use Soft U2F or any hardware U2F device with Firefox you'll need to install this extension for the time being.

Alas the extension mentioned above isn't compatible with Firefox Quantum (61.0) so is there another workaround?

I'm not aware of any workaround for using U2F in FF without that extension.

@tommythorn @mastahyeti Firefox Quantum includes native U2F support, but according to that article, Mozilla didn't fully implement the FIDO AppID and Facet specification. Perhaps that's I get the following error from the Yubico Demo Site when I try to register Firefox Quantum (63.0b13) as a device:
 Traceback (most recent call last):
  File "/root/python-u2flib-server-demo/examples/yubiauth_server.py", line 161, in __call__
    raise Exception("FIDO Client error: %s" % error)
Exception: FIDO Client error: 1 (OTHER ERROR)


UI Improvements for Projects Search Page

This page needs to be redesigned.

Are you working on this? (Yes/No)
Not right now because first, we need to finalize the design for the page.

HI, I would really like to work on this issue.

Hey, I'm new to open source
I would really like to work on this issue
Can I ?

Can I work on this? @Kartikay304

Can I work on this? @Kartikay304

Sure,but first we need to make the design for the page,you can make the design and share it on #UX channel on slack.

Can I work on this? @Kartikay304

Sure,but first we need to make the design for the page,you can make the design and share it on #UX channel on slack.

Okay @Kartikay304

Hello @Kartikay304
I would like to work on this issue, and I am preparing the sketches; could you please share the slack invite link.

@Kartikay304, I have uploaded the new designs in the #ux channel. Could you please review them once and let me know? I would then proceed with the PR.
Thank you!

is this issue still open , can i work on it ?

Hello @kushalShukla-web sorry but I have already submitted UI designs in the UX channel and it is in process of review.
Thank you!


Added node to the root node is not working

Found a bug when added a top-level node
(New node is not displayed)

This should be fixed

this issue can be closed (solved)


Build passes when it should fail for AzureResourceGroupDeployment@2

Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Environment

Server - Azure Pipelines or TFS on-premises?
Azure devOps

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
Azure Pipelines

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
[Include task name(s), screenshots and any other relevant details]
I tried a build using the old repo and generated a yaml, and it failed as expected but with the new azure devops, when I use the same yaml, the build passes successfully:
Below build fails without yaml interface which is correct. Below is the generated yaml:

While the below build passes on the new azure devops with yaml which is incorrect as the input azuredeploy.json is incorrect and build should fail as seen above:

Expected error:
2018-11-20T16:41:45.6112562Z ##[error]Error: Ensure the Template file ( 'D:\a\1\s\azuredeploy.json' ) is valid. Task failed while parsing with following error: "Unexpected token { in JSON at position 1512" 2018-11-20T16:41:45.6191805Z ##[section]Finishing: Azure Deployment:Create Or Update Resource Group action on test03
Looks like an issue with:
New YAML pipeline creation experience
Please contact me with any other details needed for this issue.
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Error logs
[Insert error from the logs here for a quick overview]

Hey @manojvazirani, thanks for reporting this issue. Could you explain why you expect the build to fail? Are you saying the azuredeploy.json is an invalid json file but that isn't caught?

Hey @manojvazirani
can you share the logs of the passing task in the build with build variable "system.debug" = "true".


ScottPlot 5

ScottPlot 5 is being actively developed (#521) to achieve the following goals:

Figure items (ticks, axis labels, grid, etc.) will implement IRenderable and track state (styling) and handle rendering in the same class. This makes experimental renderables (like new tick systems) easy to implement and test.
Support for an arbitrary number of coordinate planes (allowing plots with two different Y scales)
Improved rendering system

The Plot module will not store Bitmap objects
Existing Bitmaps can be passed-in to render onto, improving performance of user controls
Architecture is designed for eventual support of System.Drawing alternatives like SkiaSharp and ImageSharp

Mouse interactivity of all user controls will be abstracted into a PlotControl module, ensuring all user controls to have the same functionality and options without requiring maintenance of duplicate code for each control.
All plottables will have improved error-checking when data is modified

Strategy  / Timeline

ScottPlot 5 will be developed in a separate folder from ScottPlot 4
A minimum viable mouse-interactive plotting system will be created using the new architecture
Features will be migrated from 4 one-by-one, refactoring to achieve the design goals above
Unit tests will be added for each feature as it is migrated from 4 to 5
After things settle down, work through every item in #412 to ensure nothing was forgotten

progress toward ScottPlot5 was copied to dev/ScottPlot5 so it can be experimented with in parallel with ScottPlot


Updates deps and codegen for v1.9

Signed-off-by: Paul S. Schweigert<EMAIL_ADDRESS>Because https://github.com/knative-sandbox/eventing-redis/pull/380 reorganized the eventing-redis code, we needed to update those import paths to get the updates to work.
/assign @evankanderson @zroubalik

/retest


Archive log file size is not correct for large files.

NLog version: (e.g. 4.6.6)
Platform: .Net 4.8
In target -If we set archiveAboveSize="2048000"

What is the current result?  2000kb
What is the expected result? 2048kb
Did you checked the Internal log? yes

tried this, it works.
Are you sure your not fooled by Explorer? (Windows 10 here)

also please note, we don't split lines.
So if your line is 10,000 bytes long, we can't make an archive of 2,048,000
;)

test case: https://github.com/NLog/NLog/commit/8041617e47e27c0d8ef2b19fd8ab91262f5abe3f


Airdrop Creation endpoint

step 1 validate the data incoming to make sure it matches the expected merkle root
step 2 if it is valid, save individual nodes in the DB using NewAirdropClaim
step 3 if successful, return merkle root to client for uploading to smart contract.

Questions:
There is no guarantee that the merkle root will match in this case
Maybe we need a is confirmed on chain status that matches the merkleroot of the contract with the airdropId that is expected?   Another ticket? Add verified merkleRoot (bool) and contract airdropId (int) to airdrop table.

Shipped release 0.1.6


I would like to add preloader to website.

Describe the Feature you want to end
Add preloader
Expected behavior
Make user experience better if website take more time loading till that pre-loader runs
Screenshots (optional)

Additional context (optional )

Hello @samipak458  and @HridoyHazard, I would like to make a contribution to this issue.

I have raised a pull request(#82 ) against this issue, please verify.


custom dataset collection ?

Hi~
Is this ok to use different nums of camera or multi-lidar?
I want to make my own dataset and deploy this model to real vehicle, how many cameras should i choose ?  And what kind of camera(FOV and resolution) should be choose ?
Thanks a lot.

Hi, we use the nuScenes dataset with 6 cameras (1/1.8" CMOS sensor of 1600x1200 resolution) and 1 LIDAR, which  can be used as a reference.


Oops

@moxie-coder When you can, can you help fix things please? I was just testing 1.4.0-rc1, and a TON of stuff broke. So, yeah. When you can.

i fixed the issue
but im triggered because SOMEHOW three lines of code broke the whole game :skull:

@moxie-coder When you can, can you help fix things please? I was just testing 1.4.0-rc1, and a TON of stuff broke. So, yeah. When you can.

oki


AppState 'change' fires redundant 'active' when app is already active the first time the listener is added on Android iOS Prod

Description
On Android and iOS PROD when the AppState listener 'change' is added the very first time it automatically also fires the listener with nextState as "active" even though the app is already "active".
This issue occurs specifically on Prod and not Dev. On Dev it works as expected where the listener is not fired the first time it's added. listener is only fired the first time when the app goes to the background.
EXPECTED BEHAVIOR:
The behavior should be the same with dev and prod where the listener does not automatically fire with 'active' when it's added the very first time.
Steps to reproduce

Create a new RN app with npx react-native init androidAppStateIssue
Add the AppState 'change' listener on a button click as follows

  const addAppStateListener = () => {
    // read 'AppState.currentState' to register current state on prod
    // comment, uncomment this console.log to see difference on prod
    console.log('INITIAL', AppState.currentState);

    AppState.addEventListener('change', nextAppState => {
      console.log('nextAppState', nextAppState);
    });
  };

return (
...
<Button title="addAppStateListener" onPress={addAppStateListener} />
)

Build on Android/iOS, generate apk/app and install on emulator/simulator (PROD).
Click the addAppStateListener button and notice that the listener is NOT automatically fired with "active" when the listener is added.
Comment the line console.log('INITIAL', AppState.currentState); and rebuild, re-install app.
Click the addAppStateListener button and notice that the listener IS automatically fired with "active" when the listener is added

React Native Version
0.74.3 (current latest)
Affected Platforms
Other (please specify)
Android and iOS PROD
Output of npx react-native info
System:
  OS: macOS 14.5
  CPU: (10) arm64 Apple M1 Pro
  Memory: 89.75 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.20.4
    path: ~/.nvm/versions/node/v18.20.4/bin/node
  Yarn:
    version: 1.22.22
    path: ~/.nvm/versions/node/v18.20.4/bin/yarn
  npm:
    version: 10.7.0
    path: ~/.nvm/versions/node/v18.20.4/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.12.1
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK:
    API Levels:
      - "33"
      - "34"
    Build Tools:
      - 30.0.3
      - 33.0.0
      - 33.0.1
      - 34.0.0
    System Images:
      - android-33 | Google APIs ARM 64 v8a
      - android-34 | Google APIs ARM 64 v8a
      - android-35 | Google Play ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2022.3 AI-223.8836.35.2231.10671973
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.9
    path: /Users/ashwsrir/.jenv/shims/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.3
    wanted: 0.74.3
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Stacktrace or Logs
When listener is added for the first time in dev vs prod. (make sure to comment the console.log('INITIAL', AppState.currentState)on app)
dev
Running "androidAppStateIssue
listener is added

prod
Running "androidAppStateIssue
listener is added
nextAppState', 'active' <== additional event fired

Reproducer
https://github.com/ashwinkumar6/androidAppStateIssue
Screenshots and Videos
No response

@ashwinkumar6
I debugged this issue and here are my findings:
With new architecture enabled I found out that the way NativeModule are being initialised is different in debug and release builds.
In order to debug this I put a log here
https://github.com/facebook/react-native/blob/65a3259f039416394d2b945ec10565d38b3cad9e/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/shell/MainReactPackage.java#L109
For debug build: All the mentioned native modules constructors are being called on app load it self.
In release builds: Only these SoundManager, NativeAnimatedModule, FrescoModule modules are being called on app load.
Now because of this in debug build: AppStateModule constructor is called on app load itself which calls onHostResume there only which calls triggerAppStateChangeEvent but since there are no listener added in RN side we didn’t get any log.
In release builds: When we click on button to add an eventListener then it firstly loads AppStateModule which calls onHostResume which triggerAppStateChangeEvent and since we have a listener added at that time on RN side we get a log.
I checked AppStateModule here we are using TurboModules to register it.
@cortinico I am just curious to know how it works differently in release and debug builds?

@cortinico I am just curious to know how it works differently in release and debug builds?

All the native modules you find listed inside MainReactPackage are available to be used inside JS code.
@shubhamguptadream11 if you add a log on getModule as you pointed out, you're not checking if the module is loaded or not, but you're checking if someone from Javascript is accessing it or not.
I haven't investigated this further, but in Debug we do have more modules loaded (for example LogBoxes and the react-devtools debug overlay) which are not loaded in Release. That's most likely the reason why you see "less" calls to getModule.

Hi @cortinico @shubhamguptadream11,
Appreciate for the quick response !
After some additional testing, I'm able to reproduce this issue on both Android and iOS PROD environments. I've updated the issue desc to reflect the same.
Thank you


bump deps

This supersedes both #69 and #71 because it's difficult to do them separately.
This should reduce the duplicated toc issue in the Daml language tutorial, but not solve it entirely (specifically, this includes the removal of the toc in intro_0, but not the removal of the toc in writing-daml, which will have to wait for tomorrow).


⚠ Segfault when getting input from pipe

When inserting a command like this for example in bash echo hello | ./minishell the program  crashes

It seems that this caused by the readline!! I'll see about it

The minishell actually needs the char   **env arguments for its functioning, it uses it to set up the ENV variable in order to fetch commands paths and much more... . When running ./minishell, it accesses the environment list  using the global variable char **env. If we type in the prompt yielded by ./minishell this given command echo hello | ./minishell, this last word would be treated as a command and hence it gets executed through the execve() syscall, the  last parameter of this call is supposed to be the char **env which was set to NULL beforehand, the thing that causes the SEGV.  By passing the env along till the execve, we prevent this issue and refine the behavior of our program.


CARDS Array

Description
Hi,
I would really like to access the information that the API provides in the CARDS array.
Is there any plan to integrate this?
best regards
API Keys
CARDS

Mahlzeit! - if you could explain me a 'reasonable' use case (why the info from the CARD's array can generate any value) - then I would considder adding it - right now the info is used for the values in the comboxbox 'trx'

Mahlzeit! Thank you for your quick response! ;) The Use Case is: My Wife and me hat a token to unlock the charger, and my son-in-law too. I charge my son-in-law for the kilowatt hours that he charged an my wallbox.
In my old Iobroker, i had a script for that:
var kWhPreis = 0.3697;
on({id: 'go-e.0.unlocked_by', change: "ne"}, async function (obj) {
var value = obj.state.val;
var oldValue = obj.oldState.val;
//console.log(['Unlocked by ',value,' Oldvalue: ',oldValue].join(''));
Jetzt = (new Date().getTime());
var lastchanged = obj.state.lc;
var res = Math.abs(Jetzt - lastchanged) / 1000;
// wenn die letzte Änderung nicht mehr als 2 Minuten alt ist
if (res < 120) {
// Manuel hat entsperrt //oder hatte entsperrt
if (value == '4' )//|| oldValue == '4')
{
var ZaehlerStandKona = getState("go-e.0.rfid.badges.4.consumption").val
var ZaehlerWertKona  = ( (ZaehlerStandKona - 123.5)* kWhPreis + 45.65 ).toFixed(2);
// Sekunden seit lezter Änderung
Jetzt = (new Date().getTime());
var lastchanged = obj.state.lc;
var res = Math.abs(Jetzt - lastchanged) / 1000;
  var messagetext = (['Aktueller Zählerstand:\n',ZaehlerStandKona,' kWh, ',ZaehlerWertKona,' € \n'].join(''));
  if (obj.oldState.val == '4') {messagetext+='Laden beendet'}
  console.log(['Unlocked by ',getState("go-e.0.unlocked_by").val,' Telegram Messagetext: ',messagetext].join(''));
  sendTo('telegram.1', {text: messagetext}, function (res) {
      console.log('Sent to ' + res + ' users');
  });
    // Wenn Manuel lädt dann auf maximale Leistung stellen
    if (getState("go-e.0.unlocked_by").val == '4') {
        setState("go-e.0.amperePV", 16 , false);
    }

}

};
});

Thanks for your use-case - I have to check (mainly internal motivation) if I would assist you with the external/additional REST Sensor setup - or if I will integrate energy counter's per CARD... week just started...

sensor:
  - platform: rest
    name: Go-eCharger Manuel Energy
    resource: http://<IP_DES_CHARGERS>/api/status
    method: GET
    headers:
      Content-Type: application/json
    value_template: >
      {{
        value_json.cards | selectattr("name", "equalto", "<NAME>")
        | map(attribute="energy") | first | default(0) | int
      }}
    unit_of_measurement: "Wh"

Do it so far, but it would certainly still be great if one could find the "cards" in your HA-integration. Even with the increasing popularity of electric cars, I can imagine more “guest” access to which you can offer PV electricity while having coffee and cake.
I suspect it's not as easy with this array as it is with singleValues but hey, I can program a little, but I've found that my skills aren't enough to support you here.

check the https://github.com/marq24/ha-goecharger-api2/releases/tag/2024.11.1

Works perfectly! 💯 Many thanks for your very quick response and support!

JFYI: You 'son-in-law' already contacted me in order to tweak the numbers a bit... and of course I get a little share of his savings :-D -> Looks like a cool business model of the future for me...

I was suspicious of him right from the start when my daughter brought him along!


Get AzureDevOps Server version via the extension

Please check our current Issues to see if someone already reported this https://github.com/Microsoft/azure-pipelines-task-lib/issues
Environment
azure-pipelines-task-lib version: AzureDevOps extension
Issue Description
Need to fetch the AzureDevOps server version in the extension
Expected behaviour
Should be able to report the AzureDevOps server version, the extension is integrated with
Actual behaviour
Unable to find a library to achieve the desired output.

Hey @knowds - the server itself doesn't currently output this info as far as I know. Since we're just a wrapper for tasks, we don't have the ability to surface that information in this library. Since its a more general service issue, please open this request at https://developercommunity.visualstudio.com/spaces/21/index.html.
FWIW I'd guess we probably won't be able to do this in a particularly helpful way since it would require patching all existing versions of AzureDevOps and forcing existing customers to upgrade (which is something we never do). One option you have here that would get around this requirement is to take the version as an input.


Enable plugin support for service discovery

Until Kubernetes and Docker Swarm get native support for IP broadcast/multicast, it makes sense to enable service discovery over another tool.
This is fundamentally at odds with cote's zero configuration approach; but in order to enable cote on limited environment such as public clouds, it's viable to introduce an optional alternative. This will also lead to greater adoption of cote.

Would this be something worth reconsidering? Cote's zero-configuration capabilities are amazing. However, consul is a fairly popular service discovery system. Especially in the docker world. Combining cote with piloted+containerpilot would make for some very fault tolerant applications.
Having cote support a plug-able service discovery system would allow users to reuse existing service discovery methods. This may also help cut down on the chatty-ness of cote.


download an artifact from the same workflow run

essential-features-of-github-actions.md should explain how to download an artifact from the same workflow run

Why:
Closes https://github.com/github/docs/issues/10269

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@achekerylla
Thanks so much for opening a PR! I'll get this triaged for review :zap:

My changes in staging look good to me. https://docs-10268--patch-2.herokuapp.com/en/actions/learn-github-actions/essential-features-of-github-actions

Self review
You should always review your own PR first.
For content changes, make sure that you:

[ ] Confirm that the changes meet the user experience and goals outlined in the content design plan (if there is one).
[ ] Compare your pull request's source changes to staging to confirm that the output matches the source and that everything is rendering as expected. This helps spot issues like typos, content that doesn't follow the style guide, or content that isn't rendering due to versioning problems. Remember that lists and tables can be tricky.
[ ] Review the content for technical accuracy.
[ ] Review the entire pull request using the localization checklist.
[ ] Copy-edit the changes for grammar, spelling, and adherence to the style guide.
[ ] Check new or updated Liquid statements to confirm that versioning is correct.
[ ] If there are any failing checks in your PR, troubleshoot them until they're all passing.

Localization checklist found here. Link from self review directions needs updating.
https://github.com/github/docs/blob/main/contributing/localization-checklist.md

Unintentionally changed name of this PR. Restored to original name.

Merging is blocked

Anything that I can do to help?  @myoazwan95, @Marksmith1987 , @HarryBO881

@achekerylla An acceptable review comes from a member of the docs team and it says member on their profile 🐬  A green check mark appears only when a PR is approved by a docs team member ✅ Until then, you can ignore any comments and/or approvals from anyone else 💖

Hi
https://github.com/github/docs/pull/10268#


vtkLookupTable serializer missing component coloring info?

See https://discourse.paraview.org/t/color-mesh-by-vector-component/11356/7

Fixed in 2.3.2


Add support for executing scripts

As a developer of IDEasy, I do not want to worry what I want to execute with ProcessContext so that like in bash, I can just run any executable command.
However, if the command is a bash script (e.g. catalina.sh for tomcat), then IMHO ProcessBuilder will not be able to run it directly.
If I am not mistaken the problem and solution is explained here:
https://www.baeldung.com/run-shell-command-in-java
We also need to implement a way how to determine if an executable is a (bash)script or not. My assumption would be:

Filename is *.cmd or *.bat -> run using Windows CMD (only if OS is windows)
Filename is *.sh or the first 2 bytes of the file content are #! -> run using Bash - on Windows we also have to find bash. If I just enter bash in a CMD shell, I get C:\Windows\System32\bash.exe that is not our Git-Bash but a very limited (and IMHO kind of buggy Bash implementation from Microsoft that is not working on my computer). So like we did in devonfw-ide, we might need to auto-detect and find where bit-bash is installed (most likely "C:\Program Files\Git\git-bash.exe" but we need a more generic way - see old bash scripts to see how it can be done). Further, we might not want to open a new Git-Bash window but run that inside our existing shell (that may be a git-bash, CMD, or PowerShell - all three standalone or inside Windows-Terminal).

https://github.com/devonfw/ide/blob/master/scripts/src/main/resources/scripts/devon.bat#L38-L69

Done with PR #86


Check for minimum req should be done after the splash screen is shown

Splash screen should come up ASAP.

Merged to the hotfix branch.


Nested responsive components that use refs results in stale elements in Safari

Hello, we use examples provided in Easy Mode section of the doc and faced the issue that reproduces in Safari (13.0.5) on Mac OS.
The problem occurs when the responsive components are nested and there is some child that uses ref. Something like this (sandbox)
const Leaking = ({ children }) => {
  const ref = React.useRef();
  React.useEffect(() => {
    // do smth with ref
    console.log(ref.current.clientWidth);
  });

  return
{children}

}

const ResponsiveChild = () => {
  return (

      <Desktop>
        <Leaking>Desktop</Leaking>
      </Desktop>
      <Mobile>
        <Leaking>Mobile</Leaking>
      </Mobile>

  );
};

export const Issue = () => {
  return (

      <Mobile>
        <ResponsiveChild />
      </Mobile>
      <Default>
        <ResponsiveChild />
      </Default>

  );

After resizing the window back and forward the following warning occurs:

and some elements get duplicated.
We found this issue in 8.0.3 version but it also presents in 7.0.0.

Thanks for the heads up and providing a reproduction - that should help us fix this quickly.
Ping @Tomekmularczyk since this is related to the hook changes.

Thank you and let me share some additional info on it. It reproduces with a MediaQuery component as well (I've updated sandbox with it) and presents in 7.0.0 version that does not use hooks.
The set and order of components matters, playing with it I was able to get different results.
The warning appears regardless of ref usage in a child component but when it is used everything gets broken.

I managed to fix my case.

Comment this line.
Change useMatches (it is needed because MQ object provided by the library stops updating matches property after step 1).

const useMatches = (mediaQuery) => {
  const [ matches, setMatches ] = React.useState(mediaQuery.matches)

  React.useEffect(() => {
    const updateMatches = ({ matches }) => {
      setMatches(matches)
    }
    mediaQuery.addListener(updateMatches)
    setMatches(mediaQuery.matches);

    return () => {
      mediaQuery.removeListener(updateMatches)
    }
  }, [ mediaQuery ])

  return matches
}

Unfortunately, I cannot explain why it is so. I played with my app and got an infinite updates cycle at some moment, so I guess the event fires a bit differently in Chome and Safari so that mediaQuery.matches used in hook gets a wrong value under some circumstances.

Is anyone able to reproduce this issue still? Has been 4yrs and I the library has gone through a lot of improvements, this should be fixed. Will close in 30 days if nobody can repro this.


login_required issue coming every time I make public request.

from instagrapi import Client
cli = Client()
extractdata = ['username','full_name','is_private','profile_pic_url','is_verified','media_count','follower_count','following_count','biography','external_url','account_type','is_business','public_email','business_category_name','category_name']
main_data = []
user_bio_data = cli.user_info_by_username("cristiano").dict()
print(user_bio_data)
for i , j in user_bio_data.items():
if i  in extractdata:
main_data.append({"{}".format(i): user_bio_data[i]})
i am new to this instagrapi.
I am trying to run this and gives me login_requires error from past 3 days and I don't know where to go It gives me response of 1 out of every 100 request of different account.past 3 days sometimes runs sometimes not.
Can you tell me what to do. I have seen that use proxy but I don't know how to use it and where to get i tried one but its also not working.
I have doubt is there request limit and if yes how should i tackle this because i have to run this script for different public user to extract data from their profile
Can you please have look into this and let me know what should i do next.
i am using Instagrapi latest version, Chrome as browser.

You need to login or use the best proxies

but I am using user_info_by_username it will give response without any login required if i am not wrong and it will give output perfectly when i run for first 10 or 20 times without any errors.

@adw0rd Hi, kinda same situation here.
I logged in with 2FA and stored settings.
Restarted everything later and logged in again with load_settings.
Logged in succesfully.
I can get medias by tag without problem.
But when I try to upload with photo_upload() I got the following error:
Photo Upload failed with the following response: <Response [403]> [ERROR]: {"message":"login_required","status":"fail"}.
What should I do differently?

Using ensta's Guest Class:
from ensta import Guest

guest = Guest()
profile = guest.profile("cristiano")

if profile is not None:
    print(profile.followers)
    print(profile.followings)


Release file lock with h5netcdf engine after close

What is your issue?
Even after closing a dataset, the file lock is not released which causes a BlockingIOError. It would be great to release the file lock automatically after closing the dataset connection.

Could you please provide an MVCE? It would also help, if you could add the output of xr.show_versions().

@schwemro Please reopen with a minimal reproducible example when possible. Thanks!


Revert "Implement a lock-free fast path for queue_call_request()"

Reverts grpc/grpc#17401 (as it causes https://github.com/grpc/grpc/issues/17463).
Creating the revert PR in case we don't come up with a fix quickly.
The fix might need to get backported to v1.18.x

CC @sanjaypujare

@jtattermusch all mac tests started to fail last night. Do you think this is related? (since it's the only one that gets merged around that time)


Verify a message without instantiating with a wallet?

I am looking at this example https://github.com/spacebudz/lucid/blob/65d589eeb3872c250b201d6326bb5678dcd0dc25/src/examples/sign_message.ts
and trying to figure out if I should be able to do the verifyMessage step without linking the lucid library to an in browser wallet?
the use case being using lucid to sign some user data in the browser and then sending that signed data to the server and using lucid to verify it was signed by the owner of an address stored in a database?

Use lucid.selectWalletFrom, like this
const address = "...bech32...";
lucid.selectWalletFrom({address: address});


Closing chunked responses early via HttpResponse or HttpContext

Hi, hopefully this is the right place for this question. I'm implementing on-demand streaming of a zip download via chunked transfer (code below) in ASP.NET Core.
How do I close the chunked transfer early in the face of errors, such that a client can understand 'this transfer was interrupted'? I tried HttpContext.Abort and HttpResponse.Body.Dispose but that didn't seem to help. I always seem to have the browser consider the download complete (just corrupt) - testing happens to be on Firefox.
My understanding (and happy to be corrected if I'm wrong) is that the correct thing is to close the connection without sending the final terminating line - but I am not sure how to do that many levels above in ASP.NET. If it helps, our use-case is MVC Core on Kestrel, on top of ASP.NET Core on CentOS & Docker.
Example code inside an MVC action:
// Content-Disposition is an attachment, Content-Type: application/zip
using (var archive = new ZipArchive(new PositionPreservingWriteOnlyStream(_httpContextAccessor.HttpContext.Response.Body), ZipArchiveMode.Create, true))
{
	// If somewhere below throws, how to terminate the chunked transfer so the download is aborted?
	foreach (var source in sources)
	{
		var fileEntry = archive.CreateEntry(source.Path, CompressionLevel.Optimal);
		using (var zipEntryStream = fileEntry.Open())
		{
			await source.StreamInto(zipEntryStream, ct).ConfigureAwait(false);
		}
	}
}

Thanks in advance for assistance.

HttpContext.Abort() is the correct approach, or just throwing an unhandled exception and letting the server abort. Are you sure the client receives the chunk terminator? A missing terminator displays differently in the various browsers. What does Fiddler or a network trace show?
There was a bug in older versions of AspNetCoreModule (IIS) that would mistakenly add the chunked terminator for failed responses.

Clearly I needed to dig deeper before raising this issue 😊. Turns out I had misread the tracing before (too many 0x0d 0x0a values stared at...), and testing in more browsers does show the download as failed. It seems to be a peculiar quirk of Firefox to list the download as completed, but Chrome lists Network Failure explicitly.
Regardless, much appreciated for the fast confirmation that HttpContext.Abort() is the right approach to use. Thanks very much!


[PE-D] Typo under list command in UG: piroity

The UG specifies one of the sorting parameters as piroity I believe it is supposed to be priority.
Screenshot 2022-04-01 at 2.35.43 PM

Labels: severity.VeryLow type.DocumentationBug
original: manu2002g/ped#2

Same as issue #161


fixed no resource crashing dashboard

When the add button was being rendered on dashboard it had a onclick event that was only made for resource cards not the add button causing it crash as their were no id's available. Removed the event and everything is function properly .

Approved, pushing fast because it is a high priority fix.


bigtable-beam-import 1.1.0 jar

Congratulations on releasing 1.1.0! Are there plans to make a 1.1.0 bigtable-beam-import jar available? The latest at http://repo1.maven.org/maven2/com/google/cloud/bigtable/bigtable-beam-import/ appears to be 1.0.0. Thank you!

I thought I hit the Release button earlier in Sonatype, earlier today, but I guess not.  I just pushed it, so hopefully maven should distribute the artifacts over the next few hours.

I just checked repo1, and 1.1.0 was rolled out.  Here's the direct link for the shaded import jar.

Great! Thank you for the quick response.

I am getting below error  error . any idea what is issue
2018-05-23 19:54:55 INFO  MonitoringUtil$LoggingHandler:108 - 2018-05-23T19:54:54.478Z: Workers have started successfully.
2018-05-23 19:55:15 ERROR MonitoringUtil$LoggingHandler:101 - 2018-05-23T19:55:14.257Z: org.apache.hadoop.hbase.client.RetriesExhaustedWithDetailsException: Failed 125 actions: StatusRuntimeException: 125 times,
at com.google.cloud.bigtable.hbase.BigtableBufferedMutator.handleExceptions(BigtableBufferedMutator.java:276)
at com.google.cloud.bigtable.hbase.BigtableBufferedMutator.mutate(BigtableBufferedMutator.java:199)
at com.google.cloud.bigtable.beam.CloudBigtableIO$CloudBigtableSingleTableBufferedWriteFn.processElement(CloudBigtableIO.java:875)

@InfyNord: It's difficult to tell without more information.  I think that the best way forward would be to open a support ticket.


Remove old 2018 edition compatability tests.

Those are not needed anymore since the refactor to 2018 edition.

Codecov Report

Merging #350 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #350   +/-   ##
=======================================
  Coverage   88.32%   88.32%
=======================================
  Files         103      103
  Lines       14922    14922
=======================================
  Hits        13180    13180
  Misses       1742     1742

Continue to review full report at Codecov.

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


Dynamic Energy mix per country

Hello!
I have noticed in Denmark that the national electricity grid manager provides an API to track, among other things, the evolution of the energy mix in realtime: https://www.energidataservice.dk/tso-electricity/DeclarationGridmix
I was thinking of using this data source as a replacement for global_energy_mix.json, when computing CO2 emissions.
Have you already considered a method for people to include different APIs of this kind as plugins? I noticed another related question regarding using ElectricityMap, but it would be interesting to incorporate data from freely accessible  API than rely on commercial ones.
cheers,
Étienne

Hi Etienne, thanks for using CodeCarbon. It would mean a lot of maintenance to handle a different API for each country unless they all have the same standardized way of querying. On improvement I could imagine though would be to use this API to enrich a more detailed static file with evolution of mix across seasons or lets be crazy the hours of the day.
Another option would be to allow any API to provide electricity data but you would need to implement a "middle man" API to ensure standardisation so that code carbon could call it the same way it does the electricitymaps API
There is a  free tier access to ElectricityMaps that can acomodate small use cases.

Hi Luis,
Thanks for the reply. I was indeed thinking of a middle man structure, where one could wrap different APIs into a unified call to code carbon. So I think I'll fork and explore a bit  :)
have a great weekend,
Étienne

Hello, CodeCarbon already support this with co2_signal_api_token , see https://github.com/mlco2/codecarbon/blob/fad87d7acb11d54fa67e9af17c899d0752e3163a/codecarbon/core/co2_signal.py#L12C53-L12C73 for the code.
Indeed, like Luis said, https://www.co2signal.com/ has now moved to a free ElectricityMaps API so it will be nice to switch to it.


Merged project-template v10.1.1

Updated to project-template v10.1.1.

Codecov Report

Merging #667 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #667   +/-   ##
=========================================
  Coverage     31.03%   31.03%
  Complexity     1191     1191
=========================================
  Files           105      105
  Lines          4034     4034
=========================================
  Hits           1252     1252
  Misses         2782     2782

Impacted Files
Coverage Δ
Complexity Δ

webroot/robots.php
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


MAINT: use PyTorch 2.3 in CI, fix CuPy failures, more type annotations for _array_api.py

CircleCI failure is a network issue. Mypy is happy now. I'll put this in - thanks for the reviews @lucascolley and @mdhaber


[gitmoji]

User Story
As a DaaP developer
I want the data lake buckets to be cleared out of any previous iterations of data structures
So that any data in the buckets is relevant to the current state of the catalogue
Definition of Done
Example

[x] Delete items in the data and landing buckets that is versioned with minor versions rather than major versions, i.e. test_product/v1.0/ rather than test_product/v1/
[x] Delete Glue databases in the awsdatacatalog data catalogue that belong to previous versions of Data Products but don't have the suffix _vX

There are old versions of s3 keys in the data buckets. If we want to completely purge data when we delete, we need to modify any deletion methods to also remove versions.
Note: it is not possible to retrieve information about the glue databases with the sandbox account other than using the aws console due to lack of permissions:
botocore.errorfactory.AccessDeniedException:
An error occurred (AccessDeniedException) when calling the GetDatabases operation:
User<EMAIL_ADDRESS>is not authorized to perform:
glue:GetDatabases on resource: arn:aws:glue:eu-west-1:awsdatacatalog:catalog

loop for version deletion:
import re

import boto3

bucket_name = ""
s3 = boto3.resource("s3")
bucket = s3.Bucket(bucket_name)
folder = ""
key_prefix = ""
folder_key = folder + "/" + key_prefix
objects = bucket.object_versions.filter(Prefix=folder_key) # https://boto3.amazonaws.com/v1/documentation/api/1.28.1/reference/services/s3/bucket/object_versions.html
for object in objects: # https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3/objectversion/index.html
    old_v = re.search(r".*\/v\d[.]", object.key)
    if old_v:
        print(old_v.string)
        object.delete()


[CARBONDATA-3609][CARBONDATA-3610] Remove preaggregate and timeseries datamap

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1228/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1238/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1247/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1231/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1241/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1250/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1233/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1243/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1252/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1245/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1254/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1235/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1239/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1249/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1258/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1241/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1251/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1260/

By removing preaggregate we lose preaggregate + streaming feature. But we can implement it on mv (currently doesn't support)
@akashrn5, @kumarvishal09  : please check the changes once. I will also review.

@akashrn5 , @Indhumathi27 : Is all pre-aggreagte and timeseries testcase scenario we have converted to MV test cases?  If not instead of removing pre-aggregate test case, may be we can convert them to mv test case.

@ajantha-bhat I have checked MV test cases. It contains all preaggregate scenarios. So, I think Preaggregate test cases can be removed.

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1278/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1257/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1267/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1259/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1269/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1280/

LGTM


Streams leaking resources

I'm raising this issue and a PR (#2597) to address memory and goroutine leaks in drone server.
The Problem
I've noticed that, when using Streams, Drone was leaking go routines and heap space. I was able to reproduce it by doing the following:

Opening a new tab and access Drone
Refreshing the page multiple times (or opening multiple tabs)
Analysing the resource usage by accessing /api/debug/pprof/, go tool pprof, debug logs and chrome developer tools.

The logs showed multiple connections being opened but never closed, just by refreshing the page

Memory tracking showed that the heap never got garbage collected

Go routines would never terminate

Go routines and heap usage confirmation in pprof endpoint

Idle streams not getting closed:

Analysis
When profiling I found the two main issues to be:
Timers memory leak

The 60 minute timer to close the stream was not actually working. This was due to the fact that a new timer was being created for every iteration of the for loop, which happened every 30 seconds when the ping timeout ran, which meant that new 60 minute timers were created every 60 seconds. This would also prevent the old timers from being garbage collected as Go won't consider them unused until they're triggered.
I solved that issue by creating the timers outside of the for loop and properly reseting them only when data was flowing through the stream.
Stream not being closed

Using drone with a large number of repositories (> 1000) means that the database call to retrieve the repos can generate a map of over 1MB in size. This is done for every stream (this means every tab, every page refresh, etc) and this is not cached and only cleaned up once the stream closes. Since the stream was not closing, the memory usage gets very high very quickly.
The fix for this is to ensure the 1 hour idle timeout actually works and all this data can eventually be garbage collected. I also added some handlers to close the stream based on the gin context.
The Fix
Memory being freed and reused after the 1 hour idle timeout

Connections being properly closed by the server and reopened by the browser after the default 3 seconds

Testing the timers, both the ping and the idle timer work

Thanks for the pull request.  I created a separate branch and Docker image so that we can let others start testing the fix. The image is drone/drone:0.8.9-pr.2597. After a few weeks of successful testing we can re-tag drone/drone:0.8.9-pr.2597 as drone/drone:0.8.10

Thanks for merging this, I'll also start running this version in our dev drone instance.
As a note, you already have the git tag 0.8.10 in the drone/drone repo but no docker image was actually released to docker hub.


Add Assistant Tool

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
We should add an Agent tool in order to support nested assistants in SAML
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise and descriptive.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

duplicate of #731


Documented functional parameters of wrong type

Hi
When I am going through the app-script-samples I found an issue that documentation of functional parameters are of Wrong type. Prefer refer to this link  for reference to make it more sense.
and the same wrong parameters are mentioned in the main site too. Please refer this guides link.

Thanks for reporting!
Feel free to send a Pull Request to fix these comments.

This is fixed. So closing this.


add a .dockerignore to lighten docker image

What this PR does / why we need it:
This PR adds a '.dockerignore' file. This file's purpose is similar to '.gitignore'. Docker will ignore files in the list when building a container image. It can help to lighten the docker image.
reference: https://docs.docker.com/engine/reference/builder/#dockerignore-file
Files that related an item of the following list will be ignored

git, github
CI
document
integration test

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

Additional documentation:

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

@computerphilosopher Thanks for the contribution.
Using .dockerignore seems to prevent unnecessary files from blending into the image. But we are using multi-stage build and only add the binary to the image. I don't think it is necessary to reduce the image size.
Are there any other benefits?

I tested both with and without dockerignore. Image size is not reduced actually as you expected.
I will close PR until I could prove dockerignore's benefit.


You may need an appropriate loader to handle this file type (.css)

Hi everybody, I have this error
ERROR in ./node_modules/react-dates/lib/css/_datepicker.css
Module parse failed: Unexpected token (1:0)
You may need an appropriate loader to handle this file type.
| .PresetDateRangePicker_panel {
|   padding: 0 22px 11px
| }

but my webpack config is very similar to yours, except for .pcss extension (I don't use Sass, but directly postcss). Here my config:
module.exports = {
 devtool: "inline-source-maps",
 resolve: {
   extensions: [".jsx", ".css", ".pcss", ".js", ".json"]
 },
 module: {
   rules: [
     {
       test: /\.(js|jsx)$/,
       loader: "babel-loader",
       exclude: /node_modules/,
       include: path.join(__dirname, "..")
     },
     {
       test: /\.(pcss|css)$/,
       use: [
         "style-loader",
         {
           loader: "css-loader",
           options: {
             localIdentName: "[local]___[hash:base64:5]",
             minimize: true,
             sourceMap: true,
             modules: true,
             importLoaders: 1
           }
         },
         {
           loader: "postcss-loader",
           options: {
             context: path.join(__dirname, ".."),
             ident: "postcss",
             plugins: () => [
               [...]
             ]
           }
         }
       ],
       exclude: /node_modules/,
       include: path.join(__dirname, "..")
     },
     {
       test: /\.svg$/,
       loader: "svg-sprite-loader"
     },
     {
       test: /\.jpe?g$|\.gif$|\.png$|\.woff$|\.ttf$|\.wav$|\.mp3$/,
       loader: "file-loader"
     }
   ]
 }
};

Could anyone help me, please?

do you have autoprefixer in your options.plugins for the postcss-loader?

Here my postcss plugins:
              plugins: () => [
                postcssImport,
                postcssMixins,
                postcssNesting,
                postcssSimpleVars,
                postcssStripUnits,
                postcssCustomProperties,
                postcssColorFunction,
                postcssConditionals,
                postcssPresetEnv,
                cssnano
              ]

I removed autoprefixer plugin some weeks ago and instead I added postcss-preset-envwith this configuration and with an internal autoprefixer management:
const postcssPresetEnv = require("postcss-preset-env")({
  browsers: ["last 1 versions", "not ie <= 11", "not op_mini all", "not dead", "not < 0.5%"],
  stage: 1,
  autoprefixer: {
    grid: true
  },
  insertBefore: {
    "nesting-rules": postcssMixins
  }
});

Do you think autoprefixer breaks this stuff?

i cant say for certain bc im using postcss-loader pretty much out of the box with autoprefixer (the project uses sass), but you should try putting it back in. sorry i cant be more helpful

I was running into the same issue today and the way I've fixed it was including the node_modules
...
test: /\.(s?)css$/,
include: [
    path.resolve(__dirname, "./src"),
    path.resolve(__dirname, "./node_modules")
]
...

Not 100% sure but I think it fixed it because I'm importing the .css file from node_modules that needs to be parsed (wild guess after debugging for a couple of hours, might be wrong and just got lucky)

...

I attemped this approach with webpack 4, wasn't able to remove the error.

Note that the ./node_modules/react-dates/lib/css/_datepicker.css is not a CSS module, so if you're using webpack, you'll need to set the CSS loader modules option to false. You may want to make a separate rules entry just for react-dates in your webpack.config.js file to handle this, since every other library I've used uses CSS modules.
Note to the devs: I feel that the readme / getting started guide should at least have a sentence that mentions this, as I think most libraries these days are assumed to use the module pattern.


dashing don't show any result

Hello,
I get a fresh git clone from dashing tool, change the config/icinga2.json file and start the tool.
The dashlet is reachable with my browser, but without any icinga2 api result data.
The API user credentials are correct, a curl test with the icinga2 api works fine.

Icinga2 looks fine:
[2017-01-25 23:43:16 +0100] information/HttpServerConnection: Request: GET /v1/objects/hosts (from [10.X]:33748, user: dashing)
[2017-01-25 23:43:22 +0100] information/HttpServerConnection: Request: GET /v1/status/IcingaApplication (from [10.X]:33749, user: dashing)
[2017-01-25 23:43:22 +0100] information/HttpServerConnection: Request: GET /v1/objects/hosts (from [10.X]:33750, user: dashing)
[2017-01-25 23:43:26 +0100] information/HttpServerConnection: Request: GET /v1/status/IcingaApplication (from [10.X]:33751, user: dashing)
[2017-01-25 23:43:26 +0100] information/HttpServerConnection: Request: GET /v1/objects/hosts (from [10.X]:33752, user: dashing)
[2017-01-25 23:43:31 +0100] information/HttpServerConnection: Request: GET /v1/objects/services (from [10.X]:33753, user: dashing)
[2017-01-25 23:43:32 +0100] information/HttpServerConnection: Request: GET /v1/status/IcingaApplication (from [10.X]:33754, user: dashing)
[2017-01-25 23:43:32 +0100] information/HttpServerConnection: Request: GET /v1/objects/hosts (from [10.X]:33755, user: dashing)

Any Idea? :)
(we have a few more hosts and services, can this be a problem for the dashing tool?)

Probably the connection was interrupted. Error handling inside the code is far from perfect, might need some additions. Patches welcome :)
The Ruby gem which is used for making API requests: https://github.com/rest-client/rest-client

Looks like it might actually be a problem with the 'thin' webserver: https://github.com/Shopify/dashing/issues/119

I guess this could also be related to #7 ... though it sounds a bit hard to just change thin to puma or any other server.

the base ruby lib for icinga2 can't yet handle a reconnect. :(
i will implement this feature in my icinga2 gem and test this.
then i can create a PullRequest to the used lib.

i think, i found an workaround.
the test in my docker container works well.
now i can build an fix for this icinga2-lib

I've adopted @bodsch 's patch into a cleaned up version which puts error handling in one place.


STORM-694: java.lang.ClassNotFoundException: backtype.storm.daemon.common.SupervisorInfo

Exception in thread "main" java.lang.ClassNotFoundException: backtype.storm.daemon.common.SupervisorInfo, compiling:(supervisor.clj:16:1)

@duke-bartholomew
Does this still a issue?
See  SupervisorInfo has already be imported at :
https://github.com/apache/storm/blob/master/storm-core/src/clj/backtype/storm/daemon/supervisor.clj#L21

@caofangkun
I just tried again on the latest master branch, and sadly enough, yes the issue is still there.
When executing 'mvn package' or 'mvn test' I still get the same error :
Exception in thread "main" java.lang.ClassNotFoundException: backtype.storm.daemon.common.SupervisorInfo, compiling:(supervisor.clj:16:1)
	at clojure.lang.Compiler$InvokeExpr.eval(Compiler.java:3558)
	at clojure.lang.Compiler.compile1(Compiler.java:7226)
	at clojure.lang.Compiler.compile1(Compiler.java:7216)
	at clojure.lang.Compiler.compile(Compiler.java:7292)
	at clojure.lang.RT.compile(RT.java:398)
	at clojure.lang.RT.load(RT.java:438)
	at clojure.lang.RT.load(RT.java:411)
	at clojure.core$load$fn__5066.invoke(core.clj:5641)
	at clojure.core$load.doInvoke(core.clj:5640)
	at clojure.lang.RestFn.invoke(RestFn.java:408)
	at clojure.core$load_one.invoke(core.clj:5446)
	at clojure.core$compile$fn__5071.invoke(core.clj:5652)
	at clojure.core$compile.invoke(core.clj:5651)
	at clojure.lang.Var.invoke(Var.java:379)
	at clojure.lang.Compile.main(Compile.java:

@duke-bartholomew
It works very well  when I try to reproduce this issue:
   git clone https://github.com/apache/storm.git
   cd storm
   mvn package -Dmaven.test.skip=true

@caofangkun
indeed, you are right.  If I start from a new clone, it seems to work.
Thanks for your reply!
It still confuses me though why this is not working when I pull master on my already cloned storm repo.
But that's most probably a problem on my side then.

You're including common twice, once on line 29, and once on line 18.
The real solution is to remove the import of [backtype.storm.daemon.common SupervisorInfo] and use the record constructor (->SupervisorInfo ...) rather than the Java constructor (SupervisorInfo. ...) on line 504.

@knusbaum Thanks for the input :+1:

@duke-bartholomew , Seems STORM-694 was closed because #492 was merged in.  Should this pull request be closed?


capture “the Basic concepts of flexbox”

📖 https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Flexible_Box_Layout/Basic_Concepts_of_Flexbox
our definition of CssBoxAlignment can be coupled with new types, CssFlexDirection and CssFlexWrap, to drive, say, Bulma flexbox helpers (see #5)

this one might have to re-opened when #5 (or its equivalent) is reopened


Generic function does not accept argument of type with union and intersection

Bug Report

🔎 Search Terms

generic function union intersection type optional property
🕗 Version & Regression Information

This changed between versions 3.6.0-dev.20190724 and the last version

⏯ Playground Link

Playground link with relevant code
💻 Code

type A = ({ a: any } | { b: any }) & { c?: any };

declare function testA<Param extends A>(param: Param & A): Param;

declare const s1: A;

const a1 = testA(s1);
/**
 * Argument of type 'A' is not assignable to parameter of type '{ b: any; } & A'.
 *   Type '{ a: any; } & { c?: any; }' is not assignable to type '{ b: any; } & A'.
 *     Type '{ a: any; } & { c?: any; }' is not assignable to type '{ b: any; } & { a: any; } & { c?: any; }'.
 *       Property 'b' is missing in type '{ a: any; } & { c?: any; }' but required in type '{ b: any; }'.(2345)
 */

type B = ({ a: any } | { b: any }) & { c: any };

declare const s2: B;

const a2 = testA(s2); // Type of result is A but must be B

🙁 Actual behavior

The line const a1 = testA(s1); gives an error:
Argument of type 'A' is not assignable to parameter of type '{ b: any; } & A'.
  Type '{ a: any; } & { c?: any; }' is not assignable to type '{ b: any; } & A'.
    Type '{ a: any; } & { c?: any; }' is not assignable to type '{ b: any; } & { a: any; } & { c?: any; }'.
      Property 'b' is missing in type '{ a: any; } & { c?: any; }' but required in type '{ b: any; }'.(2345)

In the line const a2 = testA(s2); the typescript compiler infers the constanta a2 has type A
🙂 Expected behavior

The variable s1 has the type A that means function testA<Param extends A>(param: Param & A): Param accepts s1. The parameter type Param is instantiated to A.
The constant a2 has the type B since the parameter s2 has the type B.
It was a normal behavior up to version 3.6.0-dev.20190723. Since then it leads to errors.

Also, it is worth to note that the compiler made the wrong conclusion in the error message when it reveals { b: any } & A as { b: any; } & { a: any; } & { c?: any; }

What was the intent in writing
declare function testA<Param extends A>(param: Param & A): Param;

instead of
declare function testA<Param extends A>(param: Param): Param;

?

This was an example that simplifies the real-life piece of code exposing the bug. This pattern is useful in a generic function where types are not narrowing properly due to design limitation:
type Animal = { type: 'dog'; forelegs: number } | { type: 'duck'; wings: number };

function calculateForelimbs<A extends Animal>(animal: A): A & { forelimbs: number } {
    return { ...animal, forelimbs: animal.type === 'dog' ? animal.forelegs : animal.wings };
    // won't work due to design limitation
}

It could be fixed by adding & Animal to the type of animal argument:
function calculateForelimbsWorkable<A extends Animal>(animal: A & Animal): A & { forelimbs: number } {
    return { ...animal, forelimbs: animal.type === 'dog' ? animal.forelegs : animal.wings };
    // working like a charm
}

function createSuperAnimal(): Animal {
    const legsOrWings = 100500;
    const typeOfAnimal = Math.random() < 0.5 ? 'dog' : 'duck';

    return typeOfAnimal === 'dog'
        ? { type: typeOfAnimal, forelegs: legsOrWings }
        : { type: typeOfAnimal, wings: legsOrWings }
}

const superAnimalWithLimbsCalculated = calculateForelimbsWorkable(createSuperAnimal());

link to playground
If we make the Animal type more complex:
type Animal = ({ type: 'dog'; forelegs: number } | { type: 'duck'; wings: number }) & { domesticated: boolean };

it still works (link to playground).
When we decide to have property domesticated being optional, it turns out the code is broke:
type Animal = ({ type: 'dog'; forelegs: number } | { type: 'duck'; wings: number }) & { domesticated?: boolean };

link to playground
Now, when version 4.3.2 with improved contextual narrowing for generics has been shipped, it has become less actual. Now we can skip & Animal part.
But the bug is a bug. It may affect the legacy code or be a broken part of a bigger problem.

I found the use case where this pattern would be still useful:
type A = ({ a: any } | { b: any }) & { c?: any };

declare function f<X extends A>(x: X): X

function testA1<Param extends A | undefined>(param: Param): Param {
    return param && f(param)
    // won't work even in typescript 4.3
};

function testA2<Param extends A | undefined>(param: Param & A): Param {
    return param && f(param)
    // working fine
};

declare const s1: A;

const a1 = testA2(s1); // still error

type B = ({ a: any } | { b: any }) & { c: any };

declare const s2: B;

const a2 = testA2(s2); // wrong type A | undefined, must be B

const a3 = testA1(s1); // A, but testA1 would not work on its own

const a4 = testA1(s2); // B, but testA1 would not work on its own

link to playground
It works fine in typescript 3.3.3333: link


chore: Included githubactions in the dependabot config

This should help with keeping the GitHub actions updated on new releases. This will also help with keeping it secure.
Dependabot helps in keeping the supply chain secure https://docs.github.com/en/code-security/dependabot
GitHub actions up to date https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
https://github.com/ossf/scorecard/blob/main/docs/checks.md#dependency-update-tool
Signed-off-by: naveensrinivasan<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


add mouse support

There are several reasons why using a mouse instead of a keyboard can be a good idea:
Enhanced precision: A mouse allows for precise pointing and clicking, which can be beneficial in tasks that require accuracy, such as graphic design or photo editing. It offers fine-grained control for selecting small areas or objects on the screen.

Improved navigation: Navigating through complex interfaces or websites is generally easier with a mouse. It allows for smooth scrolling, quick access to hyperlinks, and intuitive exploration of menus and options.

Versatility: Mice often come with multiple buttons or customizable features, enabling users to perform various actions with a single click. This versatility makes it easier to execute shortcuts, control multimedia playback, or switch between applications effortlessly.

Ergonomics: Mice are designed to fit naturally under the user's hand, providing comfort during prolonged use. This ergonomic design helps prevent repetitive strain injuries and reduces the risk of discomfort or fatigue compared to typing extensively on a keyboard.

Graphic and design work: For tasks involving graphical elements, such as drawing, 3D modeling, or CAD applications, a mouse provides better control and precision over the cursor's movement, allowing for more detailed and accurate creations.

Gaming: Many games are optimized for mouse input, providing intuitive controls for aiming, selecting objects, or moving characters. The mouse's sensitivity and fluid movement make it a preferred choice for gamers, providing a more immersive and responsive experience.

Accessibility: Using a mouse can be particularly advantageous for individuals with certain physical disabilities or limitations, such as limited mobility or dexterity. A mouse can offer an alternative input method, making it easier for these users to interact with computers and software.

While keyboards also have their own advantages, such as text input and quick access to shortcuts, the mouse excels in tasks requiring precision, navigation, and graphical manipulation. Therefore, combining the use of a mouse and a keyboard provides a balanced and efficient approach to interacting with computers and software.

small d energy