Issues with jsx language loading after upgrade to prismjs 1.24.1 from 1.22.0

Information:

Prism version: 1.23.0 onwards
Plugins: line-numbers
Environment:  Node nodejs v11.0.0, Webpack 3.7.1, React 16.13.0 using latest version of babel-plugin-prismjs]

Description
in prismjs 1.22.0, this issue not reproducing. Once after upgrade to 1.23.0 or even the latest 1.24.1, i could see the issue, where jsx language is not loaded into Prism object.
Here are the snippets of the code.
Here is my babelrc file looks like:
{
  "presets": ["env", "react", "stage-1"],
  "env": {
    "development": {
      "presets": ["react-hmre"]
    },
    "lib": {
      "plugins": [
        "transform-react-constant-elements",
        "./babel-plugin-import-scss-to-css.js"
      ]
    },
    "production": {
      "plugins": ["transform-react-constant-elements"]
    }
  },
  "plugins": [
    [
      "prismjs",
      {
        "languages": ["javascript", "css", "markup", "jsx"],
        "plugins": ["line-numbers"]
      }
    ]
  ]
}

I haave a js file, where i am using the Prismjs and dynamically passing codes to this file to get it highlighted.
code-highlighter.js
import Prism from 'prismjs/prism';
import Normalizer from 'prismjs/plugins/normalize-whitespace/prism-normalize-whitespace';
const codeNormalizer = new Normalizer({
  'remove-trailing': true,
  'remove-indent': true,
  'left-trim': true,
  'right-trim': true,
  'break-lines': 100,
  'remove-initial-line-feed': true,
  'tabs-to-spaces': 2,
  'spaces-to-tabs': 2,
  indent: 0
});
const simpleNormalizer = new Normalizer({
  'remove-trailing': true,
  'remove-indent': true,
  'left-trim': true,
  'right-trim': true,
  'remove-initial-line-feed': false
});
export const highlighter = (code, lang = 'jsx') =>
  Prism.highlight(codeNormalizer.normalize(code), Prism.languages[lang], lang);
export const normalize = (code, options) => simpleNormalizer.normalize(code, options);

if u note the above code, i need to import Prism by changing the import from 'prismjs' to 'prismjs/prism'
otherwise, i am getting, "Prism is not defined" error.
if i peep into the Prism object in this code, i could not see jsx being loaded in the languages (Prism.languages object)
it seems, 1.23.0 or 1.24.1 (so far i have tested with these version of prismjs in my application) not backward compatible. i couldnt able to find in your docs, what exact changes went in from 1.22.0 to 123.0. Anything, which i need to change in my code, based on the changes went in for 1.23.0
i think still there is support for "jsx" language, but how come it is not showing up in the Prism.languages object.
I dont have working example, also, my project is confidential, i couldnt able to share it here. so, let me see whether i can do a small snippet of code changes to reproduce this one.
Any ide on this one could help.
i am not sure on Any dependency i miss for jsx language. it was working fine when i use prismjs 1.22.0

We haven't changed the dependencies of JSX since v1.22.0.

if u note the above code, i need to import Prism by changing the import from 'prismjs' to 'prismjs/prism'

That's the problem. The Babel plugin searches for imports of prismjs and nothing else, so it won't detect prismjs/prism. prismjs/prism is a bundle that includes HTML, CSS, JS, and C-like.
That's why JSX isn't loaded. You import a bundle that doesn't include JSX and (unintentionally) aren't using the babel plugin.

otherwise, i am getting, "Prism is not defined" error.

That's strange. Does that also happen without the babel plugin?
Also, are you using the latest version of the babel plugin?

Yes. i am using the latest version of babel plugin. for now, to make it work, as you suggested, i have imported jsx manually into the code-highlighting.js. now, it is working.
Regarding the other thing, still i am not sure, why this statement, "import Prism from 'prismjs'" not working. I didnt try without babel plugin. Let me try to get u an working example to reproduce this issue. i mean to say "reference error: Prism is not defined" issue

Let me try to get u an working example

That would be great. Thank you!


underscoreToCamelCase Missmatch

Different underscoreToCamelCase in Shopware\Components\Model\Generator and here. Resulting in error when attributes containing Numbers.
foo_10_bar
Shopware\Components\Model\Generator: foo_10Bar
Shopware\Components\SwagImportExport\DbAdapters\ArticlesDbAdapter: foo10Bar
Probably the same problem in the other DbAdapters as well.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/PT-7985.
Please use this issue to track the state of your pull request.

Thanks for your contribution


Forseti Day 19 schedule as image

Replacing the markdown table in the Forseti Day news post with an image for readability.

@ryanismert here's an example of how to include an image in the markdown of a news post.
https://raw.githubusercontent.com/GoogleCloudPlatform/forseti-security/0f46081ee26b8e63052dc3744f315f2ab28a36b0/_news/2018-05-09-test-post.md

@mirons-google I cut and paste the Jekyl template from the visualization post - do you believe it won't work, or that an  tag is superior?

Apologies, it'll work. If you want the image to be included in the snippet of all news then you'll want to use the excerpt front-matter field.

@ryanismert

This is how Jekyll renders the Markdown in the .md file. Let me know if you want to make any changes  before I push it out.

lgtm, thanks @red2k18


[New] Configure electronic navigational charts

Description
This PR implements Configure electronic navigational charts in Layers category. This is a new sample that replaces the all of the ENC samples. In our case, just Add ENC exchange set was removed.
URL to README: Configure electronic navigational charts
Linked Issue(s)

common-samples/issues/3950

How To Test

Tap on the map to select an ENC feature.
Tap "Display Settings" and uses the pickers to adjust the settings.

Screenshots

Before
After

@philium Could you take a look this when you get a minute? Thank you


sent a payment via paypal.

DEVICE ID: a89911a2125c5c37207fdf99e21f3b78

@kingparks

0d 13h 9m 0spanic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x28 pc=0x608cc9]
goroutine 1 [running]:
github.com/kingparks/cursor-vip/auth.getHash({0x0?, 0xc0002a1bf0?})
/Users/au/dev/my/cursor-vip/auth/client_cursor.go:125 +0x29
github.com/kingparks/cursor-vip/auth.setClient({0x6de4ed?, 0x9e85b?})
/Users/au/dev/my/cursor-vip/auth/auth.go:175 +0x1d1
github.com/kingparks/cursor-vip/auth.Run({0x6de4ed, 0xa}, 0x1)
/Users/au/dev/my/cursor-vip/auth/auth.go:121 +0x9c
main.startServer({0x6de4ed, 0xa}, 0x1)
/Users/au/dev/my/cursor-vip/main.go:31 +0x1f1
main.main()
/Users/au/dev/my/cursor-vip/main.go:16 +0x18
GO routine problem

just update :
bash <(curl -Lk https://github.com/kingparks/cursor-vip/releases/download/latest/i.sh)

DEVICE ID: a89911a2125c5c37207fdf99e21f3b78

https://github.com/kingparks/cursor-vip/issues/68#issuecomment-2556453219

i updated it. also i already paid for the 30$ in your paypal.

Patrick Martiel Franco is you?

yes

Done
exp at 2025-12-22 13:07:17
thx for your support

thank you! im still getting this error:
Authorization successful! Please do not close this window during use
365d 12h 59m 13spanic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x28 pc=0x10f8cc9]
goroutine 1 [running]:
github.com/kingparks/cursor-vip/auth.getHash({0x0?, 0xc000223bf0?})
/Users/au/dev/my/cursor-vip/auth/client_cursor.go:125 +0x29
github.com/kingparks/cursor-vip/auth.setClient({0x11ce4ed?, 0xb8e85b?})
/Users/au/dev/my/cursor-vip/auth/auth.go:175 +0x1d1
github.com/kingparks/cursor-vip/auth.Run({0x11ce4ed, 0xa}, 0x1)
/Users/au/dev/my/cursor-vip/auth/auth.go:121 +0x9c
main.startServer({0x11ce4ed, 0xa}, 0x1)
/Users/au/dev/my/cursor-vip/main.go:31 +0x1f1
main.main()
/Users/au/dev/my/cursor-vip/main.go:16 +0x18
i used the latest and updated it.

 i updated it. also i already paid for the 30$ in your paypal.

get , I will fix now

fixed, up to v2.3.7
bash <(curl -Lk https://github.com/kingparks/cursor-vip/releases/download/latest/i.sh)


doc: fix command name minify:asset

There is no s at the end: https://github.com/sensiolabs/minify-bundle/blob/655f29e6b839aa1df035ed904575ca606ff187d0/src/Command/MinifyAssetCommand.php#L31

Thank you @alexislefebvre !


Fix DBE build

Fix DBE build: upadte DBHandler init parameters and return removed ExecuteDDL function.
Fix segfault on std::filesystem::path destructor by changing gcc version from 8 to 9.

It is possible GCC 9 will break our CUDA builds, so we're doing a bit of investigation there -- otherwise this looks fine.


Skeleton

So this is roughly the kind of structure I had in mind (I coded a HMM example + test so that it would be more explicit).
Essentially split the model logic from the methodological one. This means that the methods can be fairly low level and efficient, while not hindering readability too much.
Happy to experiment on some variations.

Hi @AdrienCorenflos, thanks for taking this first pass! I like the idea of splitting the model from the core functions, but I'd propose a couple changes to the HMM interface. My main concern is that this formulation seems to assume that the HMM has categorical emissions, whereas I'd like to support arbitrary emission models. I think we could handle that level of generality by having core functions like the following:
hmm_filter(log_initial_state_probs, log_transition_matrices, log_likelihoods) -> (log_normalizer, filtered_potentials)

where filtered_potentials is a (T,K) array with filtered state log probs at each time step. That's essentially what this function does in our codebase. (It would be a little more efficient to work with initial_state_probs and transition_matrices instead of their logs, but that's a small change.) You could define similar functions for smoothing, posterior sampling, and Viterbi. All would be applicable to HMMs of arbitrary initial state, transition, and emission distribution.
Then the model is responsible for deriving transition matrices, log likelihoods, etc from its particular parameters and the given data. The model could be derived from a NamedTuple or implemented in a more object oriented way that allows for plugging in different transition and likelihood models build on TFP, etc. I'm somewhat partial to the latter, but I think we can leave that to downstream libraries to decide. If this library just implements the core message passing routines in a JAX-friendly, functional way, then those would be widely useful to us and others.

Argh!!! Github lost nearly all of my comments :(
Let me write them here instead.
I agree with @slinderman  that we should allow for any kind of observatiton model eg a list of distrax distributiions, one per state. I think we can separate out the functional core (which works with any kind of HMM) from the model specific stuff by implementing a function to create the local evidence potentials, and then passing this in to the core. Similarly we can handle time-varying or input-dependent parameters as a form of pre-processing. Something like this:

def filter(model: HMM, obs_seq: T*D, obs_mask: T*1, input_seq: T*M = None):
   obs_pots = _make_obs_pots(model.emission_dist, obs_seq, obs_mask)
   trans_pot_callables = _make_trans_pot_callables(model.transition_fun, input_seq)
   return _filter(obs_pots, trans_pots_callables)

def _filter(obs_pots, trans_pots_callables):
   for t=1:T
      prior_predictive = trans_pots_callables(t) @ beliefs[t-1]
     beliefs[t], Z = normalize(prior_predictive .* obs_pots[t])
    loglik += log(Z)
   next t
   return beliefs, loglik

def make_obs_pots(emission_dist, obs_seq, obs_mask):
  for t=1:T
     if obs_mask(t) = False:
       obs_pots[t,k] = 1 for k=1:K
     else
      obs_pots[t,k] = emission_dist[k].prob(obs_seq[t]) for k=1:J
  return obs_pots

def make_trans_pots(trans_fun, input_seq):
   if input_seq is None:
    def trans_fun(t): return trans_fun
    else
     def trans_fun(t): return trans_fun(t, input_seq[t])
   return trans_fun // closure has access to input_seq (is this safe?)

I also recommend we distinguish posterior sampling from ancestral sampling.
Something like this
(using markov_chain_sample from https://github.com/probml/ssm-jax/blob/main/ssm_jax/hmm/hmm_utils.py#L4):
def hmm_ancestral_sampling(model, T):
  z = markov_chain_sample(model.trans_mat, T)
  for t=1:T
     y[t] = model.emission_dist[z[t]].sample(1)
  return z, y

def hmm_posterior_sampling(model, obs_seq):
  obs_pots = _make_obs_pots(model.emission_dist, obs_seq, obs_mask)
   trans_pot_callables = _make_trans_pot_callables(model.transition_fun, input_seq)
   return _hmm_sampler(obs_pots, trans_pots_callables)

def _hmm_sampler(rng, obs_pots, trans_pots): // forwards filter backwards sampler
    beliefs = _filter(obs_pots, trans_pots)
    for t=T:1
       two_slice_dist = beliefs[t+1] .* trans_pot(t))
       conditional_dist = two_slice_dist[:, z[t+1]]
      z[t] = sample_categorical(rng, conditional_dist, 1)
    return z

I took a stab at a similar organization for the linear Gaussian LDS. Take a look here. Maybe we can converge on a similar framework for both HMM and LGLDS.

@slinderman  I like the LDS skeleton! Here are some comments.

Your filtering implementation is very slick.

https://github.com/probml/ssm-jax/blob/lgssm/ssm_jax/lglds/messages.py#L67
I didn't understand the math in  _condition_on(m, S, C, D, R, u, y).
By my reckoning, the update for the mean should be much simpler:

mu_cond = m + K@(y - C@m - D@u)

https://github.com/probml/ssm-jax/blob/lgssm/ssm_jax/lglds/messages.py#L115
By my recokingin the log likelihood from the one-step-ahead predictive density is

   ll += MVN(Ct @ mu_tm1t + **D** @ ut,
                  Ct @ Sigma_tm1t @ Ct.T + Rt).log_prob(yt)

I think we should agree to use import jax.numpy as jnp not import jax.numpy as np

I think we should use
from distrax import MultivariateNormalFullCovariance as MVN
instead of MVN = tfp.distributions.MultivariateNormalFullCovariance

Should we use lg_ssm instead of lds?

I find J for the precision matrix confusing, since it makes me thing of Jacobians. Is this the standard notation for information form filtering?

I think having the book chapters to accompany and explain the code (including derivation of the math, and perhaps some worked examples) will be very helpful, especially for the more obscure information form smoother.

For the smoothers, there is an issue that the posterior covariances are of size TxDxD which could get big. We may want to make returning them optional.  (For EM we just need to sum the expected sufficient statistics, which takes DxD space.) A similar issue arises with HMMs, where we may want the two-slice joint distributions of size TxKxK. I think for some of the posterior linearization methods also need two-slice joints... (I guess we can deal with this later.)

If we use lax.scan, do we not need to jit the funciton?

We should probably have some unit tests to check indexing eg for non-stationary transition matrices, which get tricky, as you mentioned. In the past I've created explicit joint MVNs for 3 time slices and compared 'brute force' inference with Kalman smoothing.

ancestral_sampling can probably live in the models.py file, since it's not an inference algorithm.

Hi @AdrienCorenflos, thanks for taking this first pass! I like the idea of splitting the model from the core functions, but I'd propose a couple changes to the HMM interface. My main concern is that this formulation seems to assume that the HMM has categorical emissions, whereas I'd like to support arbitrary emission models. I think we could handle that level of generality by having core functions like the following:
hmm_filter(log_initial_state_probs, log_transition_matrices, log_likelihoods) -> (log_normalizer, filtered_potentials)

where filtered_potentials is a (T,K) array with filtered state log probs at each time step. That's essentially what this function does in our codebase. (It would be a little more efficient to work with initial_state_probs and transition_matrices instead of their logs, but that's a small change.) You could define similar functions for smoothing, posterior sampling, and Viterbi. All would be applicable to HMMs of arbitrary initial state, transition, and emission distribution.

This is a valid point. Not sure what the best interface would be but it shouldn't be too messy to do in a principled way.

Then the model is responsible for deriving transition matrices, log likelihoods, etc from its particular parameters and the given data. The model could be derived from a NamedTuple or implemented in a more object oriented way that allows for plugging in different transition and likelihood models build on TFP, etc. I'm somewhat partial to the latter, but I think we can leave that to downstream libraries to decide. If this library just implements the core message passing routines in a JAX-friendly, functional way, then those would be widely useful to us and others.

I am very partial to not doing OOP :). In my opinion frameworks are hard to extend beyond the use case of the people who developed it. TBH I'm not even sure if having an API is a good idea. Best could be to design a framework agnostic library (beside having to use JAX) and let people build the framework they wish around it. I think there has been some success in doing this with, e.g., blackjax.

I think we should use
from distrax import MultivariateNormalFullCovariance as MVN
instead of MVN = tfp.distributions.MultivariateNormalFullCovariance
Actually, I tend to think we should do without these altogether. What happens if the distrax team wants to use our code to define SSMs objects? We'd end up with circular imports just for the sake of computing the log-density of a Gaussian.

I find J for the precision matrix confusing, since it makes me thing of Jacobians. Is this the standard notation for information form filtering?
I'm used to J too, but a more important question is how verbose we want to be in the name of the variables: instead of J should we have prec, precision, etc?

For the smoothers, there is an issue that the posterior covariances are of size TxDxD which could get big. We may want to make returning them optional.  (For EM we just need to sum the expected sufficient statistics, which takes DxD space.) A similar issue arises with HMMs, where we may want the two-slice joint distributions of size TxKxK. I think for some of the posterior linearization methods also need two-slice joints... (I guess we can deal with this later.)
I think (to be checked) that if you don't use them later on in am overall jitted function, the whole scan loop is optimised to not accumulate it, so I wouldn't worry about this kind of low-level optim.

I agree that BlackJax is a gold standard of a nice and simple API. I also think the core methods should be functional. We can wrap OO stuff on top if desired, but this should be separate. If the only thing we need from distrax is Gaussian logprob and sampling, then we can indeed write our own code for that. But I like the look of this kind of thing:
ll += MVN(Ct @ mu_tm1t + **D** @ ut,
                  Ct @ Sigma_tm1t @ Ct.T + Rt).log_prob(yt)

rather than explicitly writing out log(det(...)).

Well, this would go into a "utility" file of sorts, so I don't really see the issue with writing something like this ourselves once and then calling "from .utils.mvn import logpdf" or similar.
For example that is what is done in blackjax at the moment for arbitrary mass matrices in HMC.

Also that would help with being more flexible: we can use the precision form, cholesky form, etc, more easily.

The circular import concern is a good point, but I’m worried this is a slippery slope. We can implement our own MultivariateNormal but if that then turns into DiagonalGaussian, Normal, etc, each essentially a copy of distrax/tfp implementations, at some point we’ll have a whole distributions library within SSM. That has happened to me before and it’s not very satisfying.
Good points about verbosity of variable names — I was inconsistent between the info and standard form with one being J/h the other being long form names. Also good points about returning full expectations vs sums. That could be a kwarg.

Is it not easier to go to say distrax than to come back from it? The main point is to not build complicated distribution objects, but simply to have a couple of maths utilities that only do one thing ever. Then, if there is a need to delegate this to more specialised libraries (for efficiency, scalability, maintenance) it can happen down the line.

I guess my thinking right now is it’s fine to rely on external libraries for distribution implementations rather than copy/paste their code into this repo. If ever there comes a point where distrax or tfp wanted to import our stuff, we could work with them to fix it. It seems a bit premature to guard against that though.

Oh boy! Maybe it's good we have 3 lead designers to break ties when it comes to these issues :)
I think @slinderman is speaking from his experience of building and using HMMs and SSMs with non-Gaussian observation models, such as GMM,  Bernoulli or Poisson. IIUC this arises a lot in the neuroscience applications he works on. But it also arises in many other areas that I care about, eg time series forecasting of count data, or state estimation with robust likelihood models.
I agree with @slinderman  that it would be a waste of time to reimplement a mini library to cover all these distributions,  and to implement in an efficient and numerically stable way all the functions we will need, which presumably includes: logprob, sample (in a reparameterizable way!), and entropy (for future VI implementations).  (Any other key functions we need?)
And if we think ahead to inference for the non-linear non-conjugate case, we will definitely want to support some VI methods (eg those discussed in the BayesNewton paper. For this we will likely want to use bijectors so we can implement things like ADVI or normalizing flows. Are we going to reimplement that as well?
The key question is: where does the buck stop? We all agree we are willing to rely on jax primitive's like jax.grad, and not reimplement them. IMHO  distrax is a sufficiently simple, widely used and well supported library that we can and should treat it as a stable primitive as well. This will increase the readability of our code and save  us time. If we need features like MVNs in information or Cholesky form (which we  will definitely need!), we should add them to distrax rather than our library. (I know people at Deepmind (eg. @gpapamak) and can try to twist their arm into accepting our PR :) In the meantime we can make our own distrax-compatible MVNInfo or MVNChol class.)
@AdrienCorenflos  asks "what happens if the distrax team wants to use our code to define SSMs objects? ".  I don't think we need to worry about that now. It's true that they have HMMs but that is derived from my JSL code. In hindsight one could argue that such structured probability distributions should not belong in distrax, and indeed it only seems to be included as an example of how one could extend the library, rather than a core offering.
One compromise is that we could build our own distributions utility library, that provides functional versions of everything, eg mvn_log_prob(mean, cov, data), mvn_diag_log_prob(mean, vars, data) , mvn_info_log_prob(precision_weighted_mean, precision, data), mvn_sample(key, mu, Sigma, nsamples) or mvn_entropy(mu, Sigma). This can be implemented on top of distrax. I don't think it will be too much work: we probably only need ~6 distributions and ~3 functions per distribution, and we can just add more when needed. This would maintain functional purity without having to reinvent the wheel. It also makes it easier to add our own stuff (eg information form) without blocking on DM accepting our PRs. WDYT?

Also, in the spirit of small modular libraries, we might want to write a separate JAX mini-library for Gaussian quadrature, sigma points, etc. We will need this for various nonlinear filters and smoothers, but it may have other applications outside of SSM. @AdrienCorenflos  I think you're the expert on this topic :)

One disadvantage of the 'compromise' I mentioned is that it makes it harder to write generic code that works with any distribution. For example consider this 'preprocessing' function that computes the local evidence before running HMM filtering, smoothing, sampling:
def make_obs_pots(emission_dist, obs_seq, obs_mask):
  for t=1:T
     if obs_mask(t) = False:
       obs_pots[t,k] = 1 for k=1:K
     else
      obs_pots[t,k] = emission_dist[k].prob(obs_seq[t]) for k=1:K
  return obs_pots

If emission_dist is not a container of distribution objects, we would have to provide a container
of pdf evaluation callables, which seems a little less natural.

I probably need to rephrase. I see no issue with using distrax to define models. But I'd rather keep it segregated from the low-level code. This indeed means using functional containers @murphyk, but this would anyway be necessary right?
Maybe it would be good to focus on one simple example to see how everyone's approach would differ and see how we can combine ideas?

Re splitting (sigma points etc) that's a fair point, there may be some ground to discuss this with people at https://github.com/probabilistic-numerics/probnum. I think they have a GSOC project involving JAX too, but not sure what it entails exactly.

It's a good idea to coordinate with probnumerics project. They even list "Filtering and smoothing for (probabilistic) state-space models, mostly variants of Kalman filters." as one of their deliverables. AFAIK they don't yet have any JAX code, so we may just need to roll our own quadrature code to get up and running in the short term.
As for the rest of the API discussion, yes the proof of the pudding is in the eating. Maybe we should each implement HMM and LDS in our preferred styles and swap notes?

I will close this thread, since Scott and I have synced offline about some of these details,
and he has just pushed a first prototype of the functional core:
https://github.com/probml/ssm-jax/commit/0e675a190ff8b620486f1d51d4a746ad00aa3eee


bazel: fix macOS build

It seems that some combination of macOS Monterey and Xcode 13 started
causing a build issue. The problem is that mach/boolean.h adds macros
for TRUE and FALSE which conflicts with some of Envoy's own uses of
these. We never import this directly so it's possible these macros are
not new, but just exposed through headers in a new way.
The core fix here is to update cel-cpp to fix https://github.com/google/cel-cpp/issues/121
The secondary fix is to remove this macro as envoy already did for
windows. This stops any code that imports the platform header from
hitting this in the future.
This fixes: https://github.com/envoyproxy/envoy/issues/18842
Signed-off-by: Keith Smiley<EMAIL_ADDRESS>

Confirm, it does complete on M1-Pro

/retest

@moderation can you LGTM deps, please?

/lgtm deps


Learn React

Individuals working in the react framework will need to get spooled up on how to use it

ERRONEOUSLY CREATED/CLOSED WHILE WORKING OUT ZEN HUB SORRY


execlude body

give the option of exclude bode from request and response.
the default will be false

What does it mean to exclude body? Do you want an option to validate only headers, path params and query params but not the body?
Can you please share the use case?


Fix order/display of IV and CP in Telegram /caught function

Short Description:
When using the /caught function in Telegram, the CP is always 0 (or 1 in case of a perfect iv) and IV holds the CP. This PR fixes that.
It now also retrieves only the needed information from the database in a set order; something I suggest you/we do more often, seeing as you almost always select all columns. (Which might even cause these kinds of bugs in case the order of the columns changes in the table).
Fixes/Resolves/Closes:
None

👍


Add Camunda Community Hub badge

Hello!
Could you please add the appropriate labels for which Camunda product(s) your repo is compatible with? You can find these badges and more information here in our documentation.
Thank you!

Hi Mia
It's done
Regards Pascal
On Tue, Sep 13, 2022 at 8:50 PM Mia Moore @.***> wrote:

Hello!
Could you please add the appropriate labels for which Camunda product(s)
your repo is compatible with? You can find these badges and more
information here in our documentation
https://github.com/Camunda-Community-Hub/community/blob/main/extension-lifecycle.md
.
Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/camunda-community-hub/camunda-dmn-tester/issues/4,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2HKZ2RVKIRV7UHI5LLBC3V6DEGFANCNFSM6AAAAAAQLW4Q24
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Pascal Mengelt
Sonnenweg 23a
6414 Oberarth
079 3170128

Thank you so much!


Extract STM32 discovery book to separate repo and archive

The old STM32 discovery book is based on a board that is end of life. I'd like for the the contents of the f3discovery folder to be moved into a separate repository in the rust-embedded namespace, and for it to be marked archived

https://github.com/rust-embedded/wg/pull/759


Segfault when an Error is thrown in glutMainLoop()

If a PHP error is throw (for example attempting to call an undeclared PHP function) it will result in glut segfaulting.

Glut is no longer part of this extension. All the Glut related code has been removed.
In order to create a window, GL context and capture events you would need an additional PHP extension: SDL.
This example shows how to use PHP-SDL and PHP-OpenGL together.


Getting an error while installing

I see below error when I run "carthage bootstrap"
Failed to read file or folder at /Users/avinashp/Cartfile: Error Domain=NSCocoaErrorDomain Code=260 "The file “Cartfile” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Users/pranavm/Cartfile, NSUnderlyingError=0x7ff08cf55220 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}

Do you have carthage?  Also are you running that command in the project dir?

I am able to run the command now "carthage bootstrap" but I dont know how to build it with Xcode. I never used it before. I just want to use integrated graphics card instead of discrete graphics card for my 2011 MBP  gpu problem.

Thanks for your help.

@avi20072008  If you just want to use the program you don't need to build with xcode, just download from here:
https://codyschrank.github.io/gSwitch/
However there have been some problems with older macbook pro's but I am currently fixing that.

Thank you Cody. I already downloaded gSwitch but when I run the program it says
"gSwitch” is damaged and can’t be opened. You should move it to the Trash."

Hmm.  Please delete anything you have related to gSwitch, then download from this url
https://github.com/CodySchrank/gSwitch/releases/download/1.7.8/gSwitch.zip
open the zip and move the gSwitch.app to your applications folder.  Open the app and it should ask "this is downloaded from the internet do you want to open?" and click open.

I just downloaded from that url and it was fine for me.

I dont know what am I doing wrong. I downloaded a file from your given location. Moved /MacOS/gSwitch to the Applications folder and still the same here.  Here is the screenshot -

No worries.  What you're downloading is an application container, commonly referred to a .app
When you download and extract the zip you should get this:
https://puu.sh/AzFWZ/1540467490.png
Move THAT to your applications folder, there is no need to go into the gSwitch.app and take anything out.

how do you fix 'The file “Cartfile” couldn’t be opened because there is no such file' ?

Hmm. Please delete anything you have related to gSwitch, then download from this url
https://github.com/CodySchrank/gSwitch/releases/download/1.7.8/gSwitch.zip
open the zip and move the gSwitch.app to your applications folder. Open the app and it should ask "this is downloaded from the internet do you want to open?" and click open.

how do you fix 'The file “Cartfile” couldn’t be opened because there is no such file' ?


Support for Bukkit#createCommandSender

CommandAPI version
9.1.0
Minecraft version
1.20.1
Are you shading the CommandAPI?
Yes
What I did
I encountered an issue when trying to use Bukkit#createCommandSender in combination with Bukkit#dispatchCommand.
val myCommandSender = Bukkit.createCommandSender { }
Bukkit.dispatchCommand(myCommandSender, "<anycommandapicommand>")

What actually happened
Executing commands from custom senders generated by Bukkit#createCommandSender results in a NullPointerException (see below) due to incorrect handling by the CommandAPI.
What should have happened
The CommandAPI should support the special CommandSender returned by Bukkit#createCommandSender.
Server logs and CommandAPI config
Cannot invoke "[...].dev.jorel.commandapi.commandsenders.AbstractCommandSender.getSource()"
because "this.val$sender" is null

Other
Given this method, the CommandAPI assumes that CommandSender will only be the types provided by Spigot. However, on Paper, Bukkit#createCommandSender returns io.papermc.paper.commands.FeedbackForwardingSender, which extends CommandSender.

Is this just going to be a case of adding support for io.papermc.paper.commands.FeedbackForwardingSender, or should the CommandAPI export generic CommandSender classes?

In my personal case, adding support for io.papermc.paper.commands.FeedbackForwardingSender would be just fine. It would be greatly appreciated to implement this support first, regardless of whether generic CommandSender implementations are added later just to get support for custom command senders in place as quickly as possible.
As for generic CommandSender support, I’m not sure whether it is necessary or even appropriate. Then again, being able to register custom CommandSender classes seems like a useful niche feature at first glance.

I don't think supporting generic CommandSender implementations would be useful. Implementing the CommandSender class is not supported by Spigot (SPIGOT-7023). You can ignore Spigot and create your own command sender anyway. In my experience, custom command senders do seem to work within the Bukkit CommandMap system. However, the bridge from Bukkit to Brigadier is incompatible with custom command senders. This is related to #477.
VanillaCommandWrapper#getListener is responsible for turning the Bukkit CommandSender into a Vanilla CommandListenerWrapper. This is the code that does that:
public static CommandListenerWrapper getListener(CommandSender sender) {
    if (sender instanceof Entity) {
        if (sender instanceof CommandMinecart) {
            return ((EntityMinecartCommandBlock) ((CraftMinecartCommand) sender).getHandle()).getCommandBlock().createCommandSourceStack();
        }

        return ((CraftEntity) sender).getHandle().createCommandSourceStack();
    }
    if (sender instanceof BlockCommandSender) {
        return ((CraftBlockCommandSender) sender).getWrapper();
    }
    if (sender instanceof RemoteConsoleCommandSender) {
        return ((CraftRemoteConsoleCommandSender) sender).getListener().createCommandSourceStack();
    }
    if (sender instanceof ConsoleCommandSender) {
        return ((CraftServer) sender.getServer()).getServer().createCommandSourceStack();
    }
    if (sender instanceof ProxiedCommandSender) {
        return ((ProxiedNativeCommandSender) sender).getHandle();
    }

    throw new IllegalArgumentException("Cannot make " + sender + " a vanilla command listener");
}

This method only succeeds if the CommandSender is a block, console, entity, or proxy. These checks are actually much stricter, as the sender is usually cast to a CraftBukkit class, throwing a ClassCastException if the sender simply implements the Bukkit-API interfaces. You can still create custom command senders if you work with these restrictions, but I don't recommend that. Regardless, in order for a CommandSender to run Vanilla commands, it has to be one of these specific classes. So, it should be safe for the CommandAPI to assume the CommandSender objects passed into CommandAPIBukkit#wrapCommandSender will only be those that are already being checked for.
This bug only occurs because Paper added special logic to make their FeedbackForwardingSender work with VanillaCommandWrapper#getListener:
diff --git a/src/main/java/org/bukkit/craftbukkit/command/VanillaCommandWrapper.java b/src/main/java/org/bukkit/craftbukkit/command/VanillaCommandWrapper.java
index 45e85f252acd72800956cc2a120564256d2de797..c3742b6c4abea173b38307048091fc56bd5051d7 100644
--- a/src/main/java/org/bukkit/craftbukkit/command/VanillaCommandWrapper.java
+++ b/src/main/java/org/bukkit/craftbukkit/command/VanillaCommandWrapper.java
@@ -83,6 +83,11 @@ public final class VanillaCommandWrapper extends BukkitCommand {
         if (sender instanceof ProxiedCommandSender) {
             return ((ProxiedNativeCommandSender) sender).getHandle();
         }
+        // Paper start
+        if (sender instanceof io.papermc.paper.commands.FeedbackForwardingSender feedback) {
+            return feedback.asVanilla();
+        }
+        // Paper end

         throw new IllegalArgumentException("Cannot make " + sender + " a vanilla command listener");
     }

So yeah, this should just be a case of adding support for FeedbackForwardingSender when running on Paper. Since the CommandAPI generates Vanilla commands, any other custom CommandSender, while technically possible, can't run CommandAPI commands anyway.

@willkroboth Thanks for the insight. Let's go ahead with adding support for FeedbackForwardingSender when running on Paper. Let's see if we can push this out for our next release (9.2.0).

Implemented for 9.2.0's release. I haven't been able to write an automatic test for this, however was able to verify it works with a manual test:

Compile a plugin with these commands:
new CommandAPICommand("sayhi")
    .executes((sender, args) -> {
        Bukkit.broadcastMessage("hi");
    })
    .register();

new CommandAPICommand("mycmd")
    .executes((sender, args) -> {
        CommandSender myCommandSender = Bukkit.createCommandSender(a -> {});
        Bukkit.dispatchCommand(myCommandSender, "sayhi");
    })
    .register();

Run sayhi from the console (verify this prints hi in the console)

Run mycmd from the console (verify that this now no longer crashes)

Implemented in 9.2.0.


Media > Video | Design QA | Small Breakpoint | Caption

Detailed description

Describe in detail the issue you're having.

Caption is 100% width, not 90% per visual spec.
Visual spec >

Is this a feature request (new component, new icon), a bug, or a general
issue?

Bug

Is this issue related to a specific component?

Media > Video

What did you expect to happen? What happened instead? What would you like to
see changed?

Caption is 100% width, not 90% per visual spec.

What browser are you working in?

Chrome Version 80.0.3987.149 (Official Build) (64-bit) on Mac.

What version of the IBM.com Library are you using?

1.5.1
Steps to reproduce the issue

Open Video Storybook
Use Inspect to view Small breakpoint in the caption (320px) (see screenshot below)

Additional information

This is issue is now consolidated into issue #1971.  It is being closed.


☂ Google Maps for Flutter package roadmap

This umbrella issue is to track the individual features that are part of the ongoing work to improve the Google Maps for Flutter plugin. It is a continuation of the 2023 roadmap. The features listed here are based on the most popular feature requests (by 👍 vote and/or comments) and a goal to provide feature parity between the Flutter plugin and the underlying SDKs:

[x] #34657
[x] #26863
[x] #33586
[ ] #117907
[ ] #39810
[ ] #155526

[ ] #130472

[ ] #27550

[ ] #29181

[ ] #28493
[ ] #146920
[ ] #26479
[ ] #41731

Beyond the features listed above, we are also triaging and fixing bugs as we go.

We've done a bug review and added a few bugs to the above roadmap which are high priority or may be unblocked from fixing with a low level of effort.


Add getrecs option in make.short.parameter.estimates.table()

Be nice to have the same option that we have in make.short.parameter.estimates.sens.table().

Note that Table 26 (MLE v MCMC) has 2008 recruitment hardwired, but Tables 32-34 have an option to choose the cohorts to specify.

Also Table 35 for the retros has 2008 still. Aha, just needs the getrecs option added (like the table before it).

Probably better to change the default in make.short.parameter.estimates.sens.table() etc. each year, rather than the function calls (see d954031). I didn't want to without checking the document. That would automatically solve 2 (each year).

All doable (just not right now....).

We talked about having 2010, 2014, and 2016 recruitment in those tables
(not 2008 anymore), except for the retro one because 2016 would be omitted
for several retro runs this year (so left that with 2008 this year).
On Wed, Feb 13, 2019 at 12:01 PM Andrew Edwards<EMAIL_ADDRESS>wrote:

Be nice to have the same option that we have in
make.short.parameter.estimates.sens.table().

Note that Table 26 (MLE v MCMC) has 2008 recruitment hardwired, but Tables
32-34 have an option to choose the cohorts to specify.

Also Table 35 for the retros has 2008 still. Aha, just needs the
getrecs option added (like the table before it).
2.
Probably better to change the default in
make.short.parameter.estimates.sens.table() etc. each year, rather
than the function calls (see d954031
https://github.com/cgrandin/hake-assessment/commit/d954031abcfa22dc29da6484fec2e6cd3456edad).
I didn't want to without checking the document. That would automatically
solve 2 (each year).
All doable (just not right now....).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/561, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AHk_-sxR5cy3wv5Pll8i46GH7Aqf3pabks5vNG8OgaJpZM4a6Kh8
.

--
Aaron M. Berger, Ph.D. Research Mathematical Statistician
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center
2032 SE OSU Dr. Bldg. 955
Newport, OR 97365-5275
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Agree about retros, so leave as is.

But for consistency wouldn't the MLE v MCMC table (Table 26) have 2016 not 2008 )? I'll just leave for now in presentation.

Maybe look at next year....


Scaffold resizeToAvoidBottomInset in App2App scenario

When FlutterViewController has custom padding (does not match screen dimensions) the behaviour of the Scaffold seems to be incorrect. So, the goal is that Scaffold resizes automatically when the keyboard appears (MediaQuery viewInsets changes). This works correctly when the FlutterViewController fits the screen but when I give it a bottom padding the Scaffold still resizes without considering the "correct" area hidden by the keyboard which is now less than the normal scenario.
Is this behaviour intended ? Any suggestion ?
BR,

@BondarenkoStas @HansMuller  Is there any recommendation to workaround this issue ?

Hi @vitor-gyant
Are you still facing this issue? If yes, can you please provide your flutter doctor -v your flutter run -v and a minimal reproducible sample code? if the code is too big you can create a sample project, push it to GitHub and share the link here.
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


ooooo

oooooo this is going to be exiting, I am 100% going to make issues and pull requests on this one!

Lol, I'm just porting some of my better projects over to Github :)
Feel free to make PR's I can't wait to see them!

yea(idk much about js, maybe I'll learn, but I will try to understand it to the best of my abbilites!)


Database: Not selected / Collection - Not selected

Debian 8, installed mongodb single instance running on localhost:27017 no auth, have admin database with various collections. Install adminmongo - cannot see this database! Can create new/other databases in adminmongo... wat?
"Read and write access to data and configuration is unrestricted."
Collections have been created using mongoose in 'admin' by default. Cannot see admin. Can see other dbs.

"Read and write access to data and configuration is unrestricted."

Okay I am guessing that the admin database is hidden. If I specify a database for mongoose (which is fine) such as test in the connection string then it creates everything in there and I can see it using adminmongo. I guess this is expected behaviour. Is there a setting to allow us to see admin?


Incorrectly selecting registers in input_regs

In input_regs https://github.com/draperlaboratory/cbat_tools/blob/master/wp/lib/bap_wp/src/precondition.ml#L423, AArch64 matches the arm target, but we are selecting 32-bit registers instead of 64-bit registers. We should also probably look up registers by their roles as stated in the comment.

Do you need the input registers to be in a specific order? If not, i.e., if you're just looking for the set of call clobbered registers, then you can safely use Theory.Target.regs ~roles:[Theory.Role.Register.caller_saved]. If you need the set of registers that are used to pass arguments to a function, you can use function_argument and function_return roles. But the list of returned registers will be ordered by the register names (which coincides with the proper order in ARM, MIPS, PowerPC, but in general, e.g., in x86, doesn't work). You can, however, use the predefined list for x86, as it is now, and rely on the roles interface for the rest of the targets.

Nope, we don't need the registers to be in a specific order, so using Theory.Target.regs is exactly what we need. Thanks for the insight!


edited data dictionary description

Overview
Closes this Google Season of Docs Task:
There should be a description of what the Data dictionaries sub-page is about before the list
What problem does this address?
This adds descriptions for the Data dictionary sub-headings in the readthedocs to aid clarity for users.
Testing
How did you make sure this worked? How can a reviewer verify this?
I ran the command sphinx-build -b html docs docs/_build/html and checked if my changes reflected correctly locally
@aesharpe @zaneselvans

For testing the documentation output, I think it will usually be preferable to run:
make docs-build

Which rebuilds the documentation from scratch, after removing all dynamically generated outputs, which can occasionally be leftover after a failed docs build.
While running the Sphinx command directly can be faster if there's a single file you're editing and you want to see the results of your edits.  The CI on GitHub will attempt to rebuild that docs from scratch with the make command though.

Okay, noted. I'll apply this to the other pull requests I'll work on later

@Nancy9ice Looks like there are two "line too long" errors in lines 8 and 20 of index.rst. Once they're split, should be good to go!
docs/data_dictionaries/index.rst:8: D001 Line too long
docs/data_dictionaries/index.rst:20: D001 Line too long

@Nancy9ice Looks like there are two "line too long" errors in lines 8 and 20 of index.rst. Once they're split, should be good to go!
docs/data_dictionaries/index.rst:8: D001 Line too long
docs/data_dictionaries/index.rst:20: D001 Line too long

Resolved!

Hmm, docs build still seems to be failing. Did you push?

Maybe out of scope for this PR, but @Nancy9ice  I also think it would be a good idea to change the name of this data dictionary to "Raw FERC Form 1 Data Dictionary" vs. "FERC For 1 Data Dictionary" if it's not too hard.

Hmm, docs build still seems to be failing. Did you push?

Sorry, I just noticed I didn't push. I've pushed it now. Please confirm that the make docs-build test now runs successfully

Maybe out of scope for this PR, but @Nancy9ice I also think it would be a good idea to change the name of this data dictionary to "Raw FERC Form 1 Data Dictionary" vs. "FERC For 1 Data Dictionary" if it's not too hard.

I just added this and pushed it. Thanks


Entities not showing up in the graph

The entity wrapper class is not being applied to the cytoscape graph in the Gaffer UI

Merged into develop


Referencing values with refs

This PR might be broken into other parts if the page is too long!

So, I have decided to make some buttons in Bangla. Maybe there could be more changes, but I only limited the translation to what the page needed for the time being. As far as I have seen, making the whole page Bangla could look really nice. So, that is going to be a thing to decide. Like 'download', 'fork'...everything. Again the PR has gone a bit large. So, take your time and make change requests as much as you can.
@moshfiqrony  @nomandhoni-cs

Thanks @nafistiham bhai. Reviews will be done by tomorrow early evening.

@moshfiqrony vai. Thanks. Fixed the PR with your help!


Countdown (until time)

Like what they do on New Year's Eve.
Command idea:
>> create countdown <h:m:s> [days]

where h:m:s will be the time counting down to. If days is not specified, it will be the next time the time occurs. For instance, using 13:0:0 at 5 pm will start a countdown until 1 pm the next day. If days is specified, the time will be that many days in the future.

Implemented very quickly


Incorporated feedback

fix: changed from port forward to exposed
refactor: updated to point to consul k8s 1.1.2 . Need to clarify if 1.15.3 will be available.
docs: Updated README for reccs to export tokens and deploy HashiCups.
Thanks

LGTM!


Cognito Groups get unrestricted access to S3 by default

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
No response
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
4.52.0
What operating system are you using?
Mac
Amplify Categories
storage
Amplify Commands
update
Describe the bug
7-8 months ago I had created an S3 bucket through the amplify CLI. Recently I wanted to update the settings for the bucket and noticed that the template that was generated had group policies what did not restrict access to /public like they did before. Now they allow access to the entire bucket by default.
Here's an example of a cognito group policy that was generated before (must be 7 months ago or more) vs now:
Old Version:

"orgManagerGroupPolicy": {
            "Type": "AWS::IAM::Policy",
            "Properties": {
                "PolicyName": "orgManager-group-s3-policy",
                "Roles": [
                    {
                        "Fn::Join": [
                            "",
                            [
                                {
                                    "Ref": "authcognitoUserPoolId"
                                },
                                "-orgManagerGroupRole"
                            ]
                        ]
                    }
                ],
                "PolicyDocument": {
                    "Version": "2012-10-17",
                    "Statement": [
                        {
                            "Effect": "Allow",
                            "Action": [
                                "s3:GetObject",
                                "s3:ListBucket"
                            ],
                            "Resource": [
                                {
                                    "Fn::Join": [
                                        "",
                                        [
                                            "arn:aws:s3:::",
                                            {
                                                "Ref": "S3Bucket"
                                            },
                                            "/public/*"
                                        ]
                                    ]
                                }
                            ]
                        },
                        {
                            "Effect": "Allow",
                            "Action": "s3:ListBucket",
                            "Resource": [
                                {
                                    "Fn::Join": [
                                        "",
                                        [
                                            "arn:aws:s3:::",
                                            {
                                                "Ref": "S3Bucket"
                                            }
                                        ]
                                    ]
                                }
                            ]
                        }
                    ]
                }
            }
        },

New Version:

 "orgManagerGroupPolicy": {
            "Type": "AWS::IAM::Policy",
            "Properties": {
                "PolicyName": "orgManager-group-s3-policy",
                "Roles": [
                    {
                        "Fn::Join": [
                            "",
                            [
                                {
                                    "Ref": "authcognitoUserPoolId"
                                },
                                "-orgManagerGroupRole"
                            ]
                        ]
                    }
                ],
                "PolicyDocument": {
                    "Version": "2012-10-17",
                    "Statement": [
                        {
                            "Effect": "Allow",
                            "Action": [
                                "s3:GetObject",
                                "s3:ListBucket"
                            ],
                            "Resource": [
                                {
                                    "Fn::Join": [
                                        "",
                                        [
                                            "arn:aws:s3:::",
                                            {
                                                "Ref": "S3Bucket"
                                            },
                                            "/*"
                                        ]
                                    ]
                                }
                            ]
                        },
                        {
                            "Effect": "Allow",
                            "Action": "s3:ListBucket",
                            "Resource": [
                                {
                                    "Fn::Join": [
                                        "",
                                        [
                                            "arn:aws:s3:::",
                                            {
                                                "Ref": "S3Bucket"
                                            }
                                        ]
                                    ]
                                }
                            ]
                        }
                    ]
                }
            }
        },

The only difference is the removal of "/public" in the resource.
This allows members of the specified Cognito group to be able to read anything in the bucket. My understanding is that the amplify philosophy was prescribing a public private and protected prefixes and by default users were allowed to write under private/cognito:sub. This seems to be broken now and is introducing security risks.
Expected behavior
By default users should not be given access to the entire bucket but only two keys with the prefix of public
Reproduction steps
roll back to a version of the CLI that's 8 months old
create a new project and add an S3 bucket
Update the to latest and modify the S3 bucket configuration through the amplify cli
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

Hi @krikork going back to version 4.26.0 of the CLI (released a year ago), I still saw the "new version" of the policy being generated. In the template you can see there are other policies that restrict the access of authenticated and unauthenticated users to the /private/{sub} and /protected/{sub}. This is the default behavior. Granting bucket access to a user group will grant all users in the group access to all objects in the bucket.

Hi @edwardfoyle, thanks for the response. It's possible that I generated the old template earlier than that.
After I pushed the "new template", I tested to see if i can get keys that start with /private but not with /private/${cognito-identity.amazonaws.com:sub} and I was able to access them without an issue. At this point, I have manually modified the templates and am moving on to other work. If you don't think this is a defect, feel free to close the issue but it appeared suspicious to me and wanted to report it.

@krikork Thanks for reporting the behavior! This is by design but was not obvious from the documentation here. I have submitted a PR to clarify the expected behavior in the documentation.
Additionally, I'm going to mark this as a feature request to support more fine-grained access control when using user pool groups as it seems like what you are trying to do is grant an "admin" role additional permission but only on objects with the /public prefix. Is this a correct assumption of your use case?

Hi @edwardfoyle, what I was trying to do is build a multi tenant system and use attributes for access control but that's far beyond the scope of this issue.
Yet the behavior above still seems odd to me. If I understand correctly, when a new s3 resource is created, authenticated users are allowed to get keys within the bucket with the prefix of /public and /protected. They can also write to keys with the prefix of /private/${cognito-identity.amazonaws.com:sub}. However, if a new group is added, users that belong to that group by default would have access to everything in the bucket. This seems inconsistent.


Depreciate func rule

As there is no functions module, let's depreciate the rule: func.
Instead, I suggest the following rule:
{
  "rule": "webhook",
  "url":"<url>"
}

Added in v0.13.0


New metric tracking rejections at the threadpool-level

We already do this per-command, but not per-threadpool

Closed in #680


Becoming a member

Hi, could I please be added as a member of the organization? I'm a member of the github.com/government organization and have verified my .gov email address on Github. Thank you.

@ternera we ask that people who want to become members of the organization start by contributing data and/or code to one of the projects here. This isn't a government project - most of our contributors are outside government.


feat: support i18n

Codecov Report

Merging #72 into master will decrease coverage by 1.68%.
The diff coverage is 64.24%.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
- Coverage   73.72%   72.03%   -1.69%
==========================================
  Files          27       32       +5
  Lines         826     1005     +179
  Branches      142      175      +33
==========================================
+ Hits          609      724     +115
- Misses        143      185      +42
- Partials       74       96      +22

Impacted Files
Coverage Δ

src/common/types.ts
100% <ø> (ø)
:arrow_up:

src/services/vscodeService.ts
78.43% <0%> (-1.57%)
:arrow_down:

src/language/locale/index.ts
100% <100%> (ø)

...language/locale/localeKeyCompletionItemProvider.ts
25.92% <25.92%> (ø)

src/common/utils.ts
94.44% <50%> (-2.62%)
:arrow_down:

src/language/locale/localeDefinitionProvider.ts
64.51% <64.51%> (ø)

src/services/parser/localeParser.ts
72.88% <72.88%> (ø)

src/services/localeService.ts
73.68% <73.68%> (ø)

... and 3 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 c255b89...d7ec2ac. Read the comment docs.


Support building with GHC 9.10 (base-4.20)

Lifting the upper bound of base does allow build to succeed:
https://github.com/agda/agda2hs/blob/7753783a316de2db07da627f47058ed83bbc9876/agda2hs.cabal#L63

Opened this as working on updating to GHC 9.10.1 in Homebrew (https://github.com/Homebrew/homebrew-core/pull/178590) and noticed the Homebrew agda formula failed to build with GHC 9.10.1 when building agda2hs.

It seems we should get rid of head from our codebase:
src/Agda2Hs/Compile.hs:132:139: error: [GHC-63394] [-Wx-partial, -Werror=x-partial]
    In the use of ‘head’
    (imported from Prelude, but defined in GHC.Internal.List):
    "This is a partial function, it throws an error on empty lists. Use pattern matching, 'Data.List.uncons' or 'Data.Maybe.listToMaybe' instead. Consider refactoring to use "Data.List.NonEmpty"."
    |
132 |         genericDocError =<< vcat (map (\x -> text $ "Cannot generate multiple constructors with the same identifier: " <> Hs.prettyPrint (head x)) duplicateCons)
    |                                                                                ^^^^


Add link to the model for M1's instruction

In the https://huggingface.co/CompVis/stable-diffusion-v-1-4-original page we can dwonload two

sd-v1-4.ckpt
sd-v1-4-full-ema.ckpt

which is the difference between them ?

Do you mind closing and pulling to branch "development" instead? I'm starting to consolidate the day's worth of PRs into the development branch and doing one nightly merge into main after all check sout.
Thanks.

In the https://huggingface.co/CompVis/stable-diffusion-v-1-4-original page we can dwonload two chekpoints

sd-v1-4.ckpt
sd-v1-4-full-ema.ckpt

which is the difference between them ?

My understanding is that the non-EMA version of the weights gives more priority to images learned towards the end of its training session, whereas the EMA version equalizes the weights of early and late images. Some people say that the EMA version gives better results, but I haven't seen a good head-to-head comparison. The EMA version is larger.

Do you mind closing and pulling to branch "development" instead?

You or the submitter can click the "edit" button to the right of the PR name and change the target branch; no need to close this PR and open a new one.

@lstein changed the branched to be merged in, and adapted the changes


"on script generates error" - context missing script and queue

When the event "on script generates error" is called, some errors does not specify the script or any trace of where the error came from. Only <context.message> works, leading to <context.script> and <context.queue> being undefined.
The following error-messages does not show script or queue context:

"Narrated to non-existent or offline player!"
"'something' is an unknown argument!" (this it is usually followed up by a another error which shows the script and queue)
"Entity referenced is null!"
"Tag filling timed out!"

This is probably a problem in the internal code but I'll leave this issue here incase if it somehow could be fixed. It would help a lot through debugging.

That's an odd limitation of D1, is that the system that handles errors doesn't know where it's at very often!
D2 fixes this inherently.
But D1, nothing will cure it :/


Accept orgid in activate commands

We must first register and login in to the dashboard and only later activate it.
The dashboard will have a command that will also have the orgID and the token in it.
This will let us not require a login before we activate.

done


adapter installation without  weather station

At first I had problems with the installation of the Sainlogic adapter. It wasn't until I installed via GITHUB that the installation succeeded. I haven't bought a weather station yet. First I wanted to see if the adapter installation worked. It is unusual for the adapter not to show up in the instances. The adapter appears in the objects. Is this behavior normal for this adapter?

It should appear in instances as well, at least it does for me :D
As fot the installation, I know, trying to get the repo team to accept the latest version, but they added some extra hurdles which take me extra time to figure it out. Will try to work on this during october...

Thanks for answering. Could the reason for not appearing in  instances, the
fact I have no wether station yet?
What is repo team?
Greetings
Volker Ziegler
Am So., 26. Sept. 2021 um 15:15 Uhr schrieb Daniel Dräs <
@.***>:

It should appear in instances as well, at least it does for me :D
As fot the installation, I know, trying to get the repo team to accept the
latest version, but they added some extra hurdles which take me extra time
to figure it out. Will try to work on this during october...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-927305078,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQU7D7PQXMA56LAHA3TUD4MIBANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hello,
there is no message in the log.
I have installed the adapter again via github. Please see attached
installing log.
Grreetings
Volker Ziegler
Am Mo., 27. Sept. 2021 um 23:43 Uhr schrieb Daniel Dräs <
@.***>:

No, that should not be an issue. You need the instance to configure how to
access the weather station.
Can you check if there is any message in the log while/after installing?
There is a ioBroker repository with know adapters which is maintained by a
bespoke team. They control what is allowed in the list of adapters where
you do not need to install from github URLs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-928314313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQXG2Z7YWJDCODNVJHTUEDQQDANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi Volker,
can't see any attachment :/
Daniel

Hi Daniel,
sorry, here is attachment.
Greetings
Volker Ziegler
Am Mi., 29. Sept. 2021 um 11:29 Uhr schrieb Daniel Dräs <
@.***>:

Hi Volker,
can't see any attachment :/
Daniel
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930007917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQQ44UUIBU3E53W4ZG3UELMALANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Seems the attachment doesn't carry over to github :/

sorry I did not understand. Did you get the  attachment ?
Volker Ziegler
Am Mi., 29. Sept. 2021 um 13:31 Uhr schrieb Daniel Dräs <
@.***>:

Seems the attachment doesn't carry over to github :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930091326,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQRTOXXHFIWXMYZW3FDUEL2J7ANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

No... seems github does not store the attachment in your email. Can you copy/paste the content?

September 2021 17:09, "volkerz12" @.*** @.@.>)> schrieb:
sorry I did not understand. Did you get the attachment ?

Volker Ziegler
Am Mi., 29. Sept. 2021 um 13:31 Uhr schrieb Daniel Dräs <
@.***>:

Seems the attachment doesn't carry over to github :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930091326,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQRTOXXHFIWXMYZW3FDUEL2J7ANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

—

You are receiving this because you commented.
Reply to this email directly, view it on GitHub (https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930268711), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ABNXOZUKJH5DMQ7S22POQMLUEMT33ANCNFSM5EYXTQBA).

here is it. I hope the words are big enough to read
[image: image.png]
[image: image.png]
Volker Ziegler
Am Mi., 29. Sept. 2021 um 17:31 Uhr schrieb Daniel Dräs <
@.***>:

No... seems github does not store the attachment in your email. Can you
copy/paste the content?

September 2021 17:09, "volkerz12" @.***
@.@.>)> schrieb:
sorry I did not understand. Did you get the attachment ?

Volker Ziegler
Am Mi., 29. Sept. 2021 um 13:31 Uhr schrieb Daniel Dräs <
@.***>:

Seems the attachment doesn't carry over to github :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930091326
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AKOKRQRTOXXHFIWXMYZW3FDUEL2J7ANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930268711),
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABNXOZUKJH5DMQ7S22POQMLUEMT33ANCNFSM5EYXTQBA
).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-930288549,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQS2KHS3B7Q5F67HWP3UEMWMVANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

doesn't work either ... sorry. All I see is:
[image: image.png]
[image: image.png]
You can try to answer to this issue on the github website and attache the files/images there.

Let us try here
saint.pdf

Hello,
I need you help again.
I have bought a Sainlogic Weatherstation WS3500. I want to connect to
iobroker and import the weather data in iobroker. I need temperatur,
pressure, humidity, windspeed, winddirection
Do I have to use Wunderground or ecowitt?
Thanks
Greetings
Volker Ziegler
Am So., 26. Sept. 2021 um 15:15 Uhr schrieb Daniel Dräs <
@.***>:

It should appear in instances as well, at least it does for me :D
As fot the installation, I know, trying to get the repo team to accept the
latest version, but they added some extra hurdles which take me extra time
to figure it out. Will try to work on this during october...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-927305078,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQU7D7PQXMA56LAHA3TUD4MIBANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi Volker,
good question. Usually the ecowitt has more values it transmits. Would recommend you to test both with debug and see what you get in the logs.
Daniel

Hi Daniel,
I have tried ecowitt. But in iobroker is no data displayed.
please see attachement.
greetings
Volker Ziegler
Am Mo., 13. Dez. 2021 um 08:43 Uhr schrieb Daniel Dräs <
@.***>:

Hi Volker,
good question. Usually the ecowitt has more values it transmits. Would
recommend you to test both with debug and see what you get in the logs.
Daniel
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-992191418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQUFBHORH5SRX3NU3C3UQWPZDANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

here is the  attachement
Volker Ziegler
Am Mo., 13. Dez. 2021 um 18:40 Uhr schrieb Familie Ziegler <
@.***>:

Hi Daniel,
I have tried ecowitt. But in iobroker is no data displayed.
please see attachement.
greetings
Volker Ziegler
Am Mo., 13. Dez. 2021 um 08:43 Uhr schrieb Daniel Dräs <
@.***>:

Hi Volker,
good question. Usually the ecowitt has more values it transmits. Would
recommend you to test both with debug and see what you get in the logs.
Daniel
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-992191418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQUFBHORH5SRX3NU3C3UQWPZDANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

HI Daniel,
do you have an answer?
Greetings
Volker
Am Mo., 13. Dez. 2021 um 18:42 Uhr schrieb Familie Ziegler <
@.***>:

here is the  attachement
Volker Ziegler
Am Mo., 13. Dez. 2021 um 18:40 Uhr schrieb Familie Ziegler <
@.***>:

Hi Daniel,
I have tried ecowitt. But in iobroker is no data displayed.
please see attachement.
greetings
Volker Ziegler
Am Mo., 13. Dez. 2021 um 08:43 Uhr schrieb Daniel Dräs <
@.***>:

Hi Volker,
good question. Usually the ecowitt has more values it transmits. Would
recommend you to test both with debug and see what you get in the logs.
Daniel
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-992191418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQUFBHORH5SRX3NU3C3UQWPZDANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hm.. attachment is missing again.

Hi Daniel,
sorry again. Here is it.
Greetings
Volker
Am Fr., 17. Dez. 2021 um 16:10 Uhr schrieb Daniel Dräs <
@.***>:

Hm.. attachment is missing again.
—
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-996795502,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQWWFDEDOJLO4O23OJLURNHFPANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

no, it's not. not sure if it works via email...

Ok, please try the link below
https://gofile.io/d/qXNNKF
Greetings
Volker Ziegler
Am Mo., 20. Dez. 2021 um 14:41 Uhr schrieb Daniel Dräs <
@.***>:

no, it's not. not sure if it works via email...
—
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-997934774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQRZMFCVKABF2SSOOWTUR4XBXANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Did you get the attachment?
https://gofile.io/d/qXNNKF
Greetings
Volker Ziegler
Am Mo., 20. Dez. 2021 um 15:04 Uhr schrieb Familie Ziegler <
@.***>:

Ok, please try the link below
https://gofile.io/d/qXNNKF
Greetings
Volker Ziegler
Am Mo., 20. Dez. 2021 um 14:41 Uhr schrieb Daniel Dräs <
@.***>:

no, it's not. not sure if it works via email...
—
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-997934774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQRZMFCVKABF2SSOOWTUR4XBXANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, link works. thank you.
The protocol-section is only relevant for the listener. This would mean your weatherstation needs to be configured to send it's data to ioBroker. In my station I can do that with WSView and custom setup. Not sure this works for you.
This is how it looks on my WSView settings:

As for the scheduler, it will only be able to pull some information. If you activate debug I might be able to decode it... not fully sure though. So far I was lucky to find a documentation how to parse the content.

Ok,
thanks for helping. This will not work. I send weatherstatoin  back to
amazon.
Greetings
Volker Ziegler
Am Di., 21. Dez. 2021 um 17:46 Uhr schrieb Daniel Dräs <
@.***>:

Yes, link works. thank you.
The protocol-section is only relevant for the listener. This would mean
your weatherstation needs to be configured to send it's data to ioBroker.
In my station I can do that with WSView and custom setup. Not sure this
works for you.
This is how it looks on my WSView settings:
[image: image]
https://user-images.githubusercontent.com/5994342/146967221-0c222848-7d0d-4639-b0c2-2243ed30f2d5.png
As for the scheduler, it will only be able to pull some information. If
you activate debug I might be able to decode it... not fully sure though.
So far I was lucky to find a documentation how to parse the content.
—
Reply to this email directly, view it on GitHub
https://github.com/phifogg/ioBroker.sainlogic/issues/96#issuecomment-998933824,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOKRQT5C6DFMCEE4BEI3PTUSCVOJANCNFSM5EYXTQBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Which one did you use, might be interesting for other as well.

He mentioned a Sainlogic Weatherstation WS3500 some posts back.
@volkerz12 : Sorry to hear that. Sainlogic has so many of these stations and it seems not all are equal.
Closing this issue for now. Feel free to open a new one when needed.


scim individual users not mapping correct groups when generating tokens

What version of UAA are you running?
77.16.0
How are you deploying the UAA?
I am deploying the UAA

using a bosh release I downloaded from bosh.io
using cf-deployment v43.6.0

What did you do?
Using ops files, insert sections to create/import specific accounts in LDAP to give specific permissions when used.
Here's an extract from a uaa.yml
scim:
  users:
  - service_account1||service_account1|||cloud_controller.admin,scim.read,scim.write,doppler.firehose,routing.router_groups.read,routing.router_groups.write,routing.routes.read,routing.routes.write|ldap
  - service_account2||service_account2|||cloud_controller.admin_read_only,scim.read,doppler.firehose|ldap

What did you expect to see? What goal are you trying to achieve with the UAA?
On uaa 77.15.0 (the previous release), the correct groups are found when logging in.
[2024-09-30T06:19:13.093476Z] uaa - 11 [https-jsse-nio-8443-exec-4] - [redacted,redacted] ....  INFO --- Audit: TokenIssuedEvent ('["openid","routing.router_groups.write","scim.read","cloud_controller.admin","uaa.user","routing.router_groups.read","cloud_controller.read","password.write","cloud_controller.write","doppler.firehose","scim.write"]'): principal=redacted, origin=[client=cf, user=service_account1], identityZoneId=[uaa]
This is the correct behaviour and results in service_account1 being able to see all the orgs/spaces, etc in CF as they have the cloud_controller.admin group assigned. Same goes for service_account2, it has read only access to all orgs/spaces. Logs not shared for service_account2.
What did you see instead?
On uaa 77.16.0, different groups are allocated, not the ones specified in the uaa.yml
[2024-09-30T06:13:13.470329Z] uaa - 12 [https-jsse-nio-8443-exec-4] - [redacted,redacted] ....  INFO --- Audit: TokenIssuedEvent ('["openid","uaa.user","cloud_controller.read","password.write","cloud_controller.write"]'): principal=redacted, origin=[client=cf, user=service_account1], identityZoneId=[uaa]
This results in the user not being able to see all the orgs and spaces that we could see in previous versions. I assume because the cloud_controller.admin group is not being used in the token generation.

you create tokens via cf login ?

yeah it's just doing cf login. those logs are from /var/vcap/sys/logs/uaa/uaa.log

ok, then we can check if https://github.com/cloudfoundry/uaa/pull/3033 is the causing PR for your change.
do a "cf login xxx --origin ldap" or set defaultIdentityProvider: ldap into uaa.yaml
Because I think you problem is the mix of uaa / ldap origin... the groups are now assigned to ldap origin but the token creation is done with origin uaa ( I guess ) and then it would explain the missing groups. / authorities in your token

Using cf cli v8.7.10, I've done cf login xxx --origin ldap and the uaa logs still appear to say that the groups allocated are incorrect. I also cannot see the orgs/spaces that someone with cloud_controller.admin should be able to see.

are you familiar with uaac ?
If yes, please call: uaac group get cloud_controller.admin

Apologies for lateness. I'm not well at the moment so working odd hours.
uaac group get cloud_controller.admin
  id: redacted
  meta
    version: 0
    created: 2024-08-27T07:37:07.256Z
    lastmodified: 2024-08-27T07:37:07.256Z
  members:
  -
    origin: ldap
    type: USER
    value: redacted-73fc8d54b811
  -
    origin: ldap
    type: USER
    value: redacted-5eeb9bafb311
  -
    origin: uaa
    type: USER
    value: redacted-559c70fb1e11
  -
    origin: uaa
    type: USER
    value: redacted-6926f2e3a411
  -
    origin: uaa
    type: USER
    value: redacted-704ef09b2111
  -
    origin: uaa
    type: USER
    value: redacted-7b2400599011
  -
    origin: uaa
    type: USER
    value: redacted-0ed5c4754011
  -
    origin: uaa
    type: USER
    value: redacted-3840d0e7b411
  schemas: urn:scim:schemas:core:1.0
  displayname: cloud_controller.admin
  zoneid: uaa

A little thing extra. One of the oddities with this feature is that if you add the user to an LDAP group, and you configure UAA to say that any members of LDAP group are a member of the cloud_controller_admin group, it honours this. This functionality is the same between uaa 77.15 and 77.16.
It only appears to be named LDAP accounts with specific permissions provisioned directly to the account which are not honoured.

Thanks. but your token is created with origin uaa ?
So I have to align with the others about a revert / partial revert of https://github.com/cloudfoundry/uaa/pull/3033
Are you able to consume a uaa-release without integrated into cf-deployment ? because then we can provide it to you and you can verify that it works again

That'd be super thanks.
I'd need it to be available on bosh.io/releases or somewhere else then I can use an ops file to consume the new release. it'd need to be part of cf-deployment but I can merge it in to the manifest without anyone upstream changing cf-deployment.yml
I should be able to test pretty quick tho if that's helpful

but your token is created with origin uaa ?

Would their token be created with origin ldap, since they said: "I've done cf login xxx --origin ldap"?

Ok here is my local reproduction.
UAA 3069 repro.txt
TLDR: The interesting difference is that in the latest UAA, the ldap user's groups bootstrapped by UAA.yml get wiped away after a login (via password grant).

Ok here is my local reproduction. UAA 3069 repro.txt
TLDR: The interesting difference is that in the latest UAA, the ldap user's groups* bootstrapped by UAA.yml get wiped away after a login (via password grant). I don't yet know the connection between this behavior and #3033.
*the ldap user's groups as returned by /Users endpoint (aka uaac user get).

Thanks a lot because this helped me to understand howto test and with it I created https://github.com/cloudfoundry/uaa/pull/3072 which now returns back the scopes (groups from scim setup).
I splitted the createUser into the case where user is created from uaa.yml and from shadowUser so that we dont revert everything from https://github.com/cloudfoundry/uaa/pull/3033 but if you bootstrap users withing uaa.yml it fixes the problem from here.

@thelangley What I would recommend to grant external LDAP users group is via external group mapping rather than pre-creating those shadow LDAP users (which I am not sure is a supported / intended feature). Would the former work for you?

It isn't a core requirement to create shadow LDAP users. But it is a core requirement to have an LDAP user who is not a member of any LDAP groups to have a certain level of access. We do want to map these particular LDAP users to groups (like cloud_controller.admin) so they have permissions to do CF things. These singular accounts are for robots to only do CF things as part of pipeline tasks so don't need to be a part of any LDAP group.
Can you perform external group mapping but for instead of groups, map specific users in LDAP who are not in any LDAP groups?
We already use external group mappings for real people who are members of LDAP groups. Permissions continue to work fine for them between 77.15 and 77.16

@strehle I continued to investigate why #3033 has resulted in my local reproduction and my conclusion is that the behavior described in the local reproduction (previously-bootstrapped group memberships of an external user get wiped away after an external login by the said user) and this issue is the original intended design of UAA.
UAA has always intended to "refresh" the external user's group membership by clearing out the user's previous group membership upon an ExternalGroupAuthorizationEvent event. Before #3033, this intended behavior was not working because UAA is querying the external user's previous group membership with an origin filter; however, we now know that before #3033, the origin value in the group_membership was all incorrect. Now #3033 fixes that and this intended behavior is working again.

@thelangley

Can you perform external group mapping but for instead of groups, map specific users in LDAP who are not in any LDAP groups?

Currently, I don't see how to do that.
The simplest way to accomplish your goal would be to add your "singular accounts...for robots to only do CF things as part of pipeline tasks" to those LDAP groups (that are mapped to CF scopes/groups) just like the human accounts, since these service accounts and human accounts need to have similar power in CF.
If you don't wanna add the service accounts to the existing LDAP groups (because maybe those groups are for human), you could add a new LDAP group just for service accounts. And then add an additional external group mapping to UAA to map that new LDAP group.

Security want us to use LDAP accounts for service accounts so we'll need to continue to start using them. Service accounts and puny humans cannot co-exist in the same LDAP groups.
So it looks like we'll need to work around this new uaa functionality by creating new LDAP groups. I work in Enterprise so it'll take ages to get this done.
We've used this functionality for a good few years now and it's the first time that we've encountered this feature.
I'll kick off the process to create new LDAP groups, add the service accounts into these LDAP groups then map said groups to the uaa groups via extra external group mappings.


Package timeout reach

Hello,
I am using your default example (which does nothing).
Everything is working fine, until the server logs "Package reach timeout".
Logs:
Client
pi@raspberrypi:~ $ node raspi.js
DEPRECATION: Please use require('serialport') instead of require('serialport').SerialPort
CLIENT: Sp connecting...
CLIENT: Sp connected
CLIENT: Tcp connecting...
CLIENT: Tcp connected
CLIENT: Tcp data received  <Buffer 24 4d 3c 00 01 64 00 65>
CLIENT: Tcp package unserialized 1 100
CLIENT: Sp write data 1 100 <Buffer 24 4d 3c 00 64 64>
CLIENT: Tcp data received  <Buffer 24 4d 3c 00 02 65 00 67>
CLIENT: Tcp package unserialized 2 101
CLIENT: Timeout reached 1
CLIENT: Sp write data 2 101 <Buffer 24 4d 3c 00 65 65>
CLIENT: Timeout reached 2
CLIENT: Tcp reconnecting...
Server
alexandre@Alex-Linux:~/Documents/RaspiDrone$ sudo node desktop.js
DEPRECATION: Please use require('serialport') instead of require('serialport').SerialPort
SERVER: Server listening...
GENERAL TCP 1000 BOUND
SERVER: Client connecting...
/home/alexandre/node_modules/fibers/future.js:313
throw(ex);
^
Package timeout reach (#1/100)
Thank you for your help.

Figured out it was a misconfiguration.
Follow THIS if you have similar problems on Raspi: http://raspberrypi.stackexchange.com/a/54766


Sync (Rebase) Command added

In relation to #31055

@excerebrose,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@excerebrose, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Please consider merging this pull request. It would be great that UIs like Visual Studio Code accommodate rebasing to allow teams to keep a linear history rather than the mess of automatic merges that the normal git pull command creates.

Thanks! 🍻


cve-check-tool: Use anonymous HTTPS as GIT transport rather than SSH

Fetching of cve-check-tool from git repo using ssh protocol as
transport isn't feasible for publicly accessible repositories
such as github. Fetches will fail unless ssh private key is available.
Use anonymous HTTPS as transport for GIT.
Signed-off-by: Sergey Popovich<EMAIL_ADDRESS>

Can one of the admins verify this patch?


VSCODE flickers in full screen mode

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: latest version , i uninstall it , and backto 1.94.2
OS Version: mac os sonoma 14.4.1 macBook air

Steps to Reproduce:

Open two VSCode editors.
Move one editor to the secondary screen.
Double click headerbar
Select a file or ctrl +p
The screen shaking....

It seems to perform particularly like these related issues.
https://github.com/microsoft/vscode/issues/41513
https://github.com/microsoft/vscode/issues/15211

/gifPlease

I updated my computer system, and this bug can no longer be reproduced.


The Datasets view is serving the previous dataset version

Collection: https://cellxgene.cziscience.com/collections/9700c831-f97e-49f0-98a1-10812e302428
If I download B cells -- C_V19 infection, vaccination and HC from the Collection detail page, then the latest version is served - https://datasets.cellxgene.cziscience.com/72fcdc42-c37f-4945-881f-a41baf3d1b95.h5ad
If I download B cells -- CV19 infection, vaccination and HC from the Datasets page, then the previous version is served
https://datasets.cellxgene.cziscience.com/b46df5a9-094f-4c03-a790-1957204aa467.h5ad
Using the Discover API to return dataset versions, *b95 is the current version and *467 is the previous version:
'B cells -- CV19 infection, vaccination and HC'
'id: 1a7e99fe-d5b6-44d7-aa50-c6ef6ab29761'
'version_id: 72fcdc42-c37f-4945-881f-a41baf3d1b95'
'published at: 2023-12-13T20:51:08.676653+00:00'

'B cells -- CV19 infection, vaccination and HC'
'id: 1a7e99fe-d5b6-44d7-aa50-c6ef6ab29761'
'version_id: b46df5a9-094f-4c03-a790-1957204aa467'
'published at: 2023-12-04T22:41:04.125989+00:00'

This may have been a caching issue. Not reproducing.


Create a generic robot class

Add comments to this issue about things that every robot you build can do. We're going to write out some function stubs and then write the functions as our first commit

should be solved by #5


sandbox for azure

Describe the bug
Yesterday I created Azure free account and started the azure course on docs.microsoft.com because i decided to learn it during the quarantine. And I can't use the sandbox, I just see the message that the sandbox is prohibited for me because I violated the Terms of Use. But how can I  violate it if I've just created free Azure account and do nothing yet?
Now I can see this message: "We have reviewed your appeal and determined that your account has violated our Terms of Use. Your sandbox access will not be reinstated."
How it could be, if I just tried take the training course and did nothing?
Really disappointing bug.
To Reproduce
Steps to reproduce the behavior:

Started the course
Sow the message
Stuck

Expected behavior
Continue the course.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Browser Firefox
Version latest

@ordinaire1 Please report feedback at https://docs.microsoft.com/en-us/learn/support/troubleshooting#report-feedback


chore(react): remove react wrappers

Related Ticket(s)
Closes #11501
Description
This PR removes React wrapper build scripts, React as a dependency, and the @storybook/addon-knob dep as it is not used.
Changelog
Removed

react wrapper build script files
@storybook/addon-knobs dependency

@IgnacioBecerra nice! I was just about to do this 😂. Can we also update the docs?
Change these 2 sections:

https://github.com/carbon-design-system/carbon-for-ibm-dotcom/feat/cwc-storybook-7-vite/packages/carbon-web-components/README.md?plain=1#L263-L272
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/docs/welcome.mdx#L114-L123

To maybe something like:
This package can also be used within other JavaScript frameworks such as Angular and
Vue. This is achievable since web components are the modern browser standard, and
work well with other front-end frameworks that exist in the application. In
turn, this also comes with the benefits of encapsulation within the Shadow DOM:

Remove these files:

https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/src/components/implementation-practice.md
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/tests/integration/build/react-ssr_steps.js
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/tests/integration/build/react_steps.js
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/tests/integration/build/ie_steps.js

Remove these entire sections:

https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/README.md?plain=1#L309-L349
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/docs/form.md?plain=1#L30-L63
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/docs/form.mdx#L38-L40

@kennylam knew i was missing something here or there, updated!