[BUG]: Calibrate test error

Scenario: Calibrate
Step: 3.4
Link to the project: https://app.staging.terarium.ai/projects/6034fbd3-4cd2-41ac-a8ca-a610041f97dd/workflow/84f2045f-84ff-4597-af4f-42d2d8b819c8?operator=d1e8c14d-dcf0-4b7d-8688-8f74ee4d0d5e

Describe the issue
Im getting a calibrate error when following the steps when I run a calibrate.  Here are the logs:
ERROR

Traceback (most recent call last): File "/usr/local/lib/python3.10/site-packages/pyro/poutine/trace_messenger.py", line 174, in __call__ ret = self.fn(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context return func(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 12, in _context_wrap return fn(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyciemss/interfaces.py", line 499, in wrapped_model full_trajectory = model( File "/usr/local/lib/python3.10/site-packages/pyro/nn/module.py", line 449, in __call__ result = super().__call__(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl return self._call_impl(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl return forward_call(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyciemss/compiled_dynamics.py", line 126, in forward simulate(self.deriv, self.initial_state(), start_time, end_time) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/solver.py", line 109, in _pyro_simulate state, start_time, next_interruption = simulate_to_interruption( File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/handlers/solver.py", line 89, in _pyro_simulate_to_interruption msg["value"] = torchdiffeq_simulate_to_interruption( File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 252, in torchdiffeq_simulate_to_interruption value = simulate_point( File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/handlers/trajectory.py", line 97, in _pyro_simulate_point trajectory: State[T] = simulate_trajectory( File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/handlers/solver.py", line 77, in _pyro_simulate_trajectory msg["value"] = torchdiffeq_simulate_trajectory( File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 169, in torchdiffeq_simulate_trajectory return _torchdiffeq_ode_simulate_inner(dynamics, initial_state, timespan, **kwargs) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 71, in _torchdiffeq_ode_simulate_inner solns = _batched_odeint( # torchdiffeq.odeint( File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 127, in _batched_odeint yt_raw = torchdiffeq.odeint(func, y0_expanded, t, **odeint_kwargs) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/odeint.py", line 79, in odeint solution = solver.integrate(t) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/solvers.py", line 34, in integrate solution[i] = self._advance(t[i]) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/rk_common.py", line 227, in _advance self.rk_state = self._adaptive_step(self.rk_state) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/rk_common.py", line 292, in _adaptive_step y1, f1, y1_error, k = _runge_kutta_step(self.func, y0, f0, t0, dt, t1, tableau=self.tableau) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/rk_common.py", line 78, in _runge_kutta_step f = func(ti, yi, perturb=perturb) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl return self._call_impl(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl return forward_call(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/misc.py", line 193, in forward t = _nextafter(t, t - 1) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/misc.py", line 361, in _nextafter out = torch.nextafter(x1, x2) RuntimeError: "nextafter_cpu" not implemented for 'Long' The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/usr/local/lib/python3.10/site-packages/rq/worker.py", line 1428, in perform_job rv = job.perform() File "/usr/local/lib/python3.10/site-packages/rq/job.py", line 1278, in perform self._result = self._execute() File "/usr/local/lib/python3.10/site-packages/rq/job.py", line 1315, in _execute result = self.func(*self.args, **self.kwargs) File "/service/./execute.py", line 37, in run output = eval(operation_name)(**kwargs) File "/usr/local/lib/python3.10/site-packages/pyciemss/integration_utils/custom_decorators.py", line 29, in wrapped raise e File "/usr/local/lib/python3.10/site-packages/pyciemss/integration_utils/custom_decorators.py", line 10, in wrapped result = function(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyciemss/interfaces.py", line 526, in sample samples = pyro.infer.Predictive( File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl return self._call_impl(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl return forward_call(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyro/infer/predictive.py", line 273, in forward return _predictive( File "/usr/local/lib/python3.10/site-packages/pyro/infer/predictive.py", line 78, in _predictive max_plate_nesting = _guess_max_plate_nesting(model, model_args, model_kwargs) File "/usr/local/lib/python3.10/site-packages/pyro/infer/predictive.py", line 21, in _guess_max_plate_nesting model_trace = poutine.trace(model).get_trace(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/trace_messenger.py", line 198, in get_trace self(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/trace_messenger.py", line 180, in __call__ raise exc from e File "/usr/local/lib/python3.10/site-packages/pyro/poutine/trace_messenger.py", line 174, in __call__ ret = self.fn(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context return func(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 12, in _context_wrap return fn(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyciemss/interfaces.py", line 499, in wrapped_model full_trajectory = model( File "/usr/local/lib/python3.10/site-packages/pyro/nn/module.py", line 449, in __call__ result = super().__call__(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl return self._call_impl(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl return forward_call(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/pyciemss/compiled_dynamics.py", line 126, in forward simulate(self.deriv, self.initial_state(), start_time, end_time) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/solver.py", line 109, in _pyro_simulate state, start_time, next_interruption = simulate_to_interruption( File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/handlers/solver.py", line 89, in _pyro_simulate_to_interruption msg["value"] = torchdiffeq_simulate_to_interruption( File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 252, in torchdiffeq_simulate_to_interruption value = simulate_point( File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/handlers/trajectory.py", line 97, in _pyro_simulate_point trajectory: State[T] = simulate_trajectory( File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 281, in _fn apply_stack(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/runtime.py", line 212, in apply_stack frame._process_message(msg) File "/usr/local/lib/python3.10/site-packages/pyro/poutine/messenger.py", line 162, in _process_message return method(msg) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/handlers/solver.py", line 77, in _pyro_simulate_trajectory msg["value"] = torchdiffeq_simulate_trajectory( File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 169, in torchdiffeq_simulate_trajectory return _torchdiffeq_ode_simulate_inner(dynamics, initial_state, timespan, **kwargs) File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 71, in _torchdiffeq_ode_simulate_inner solns = _batched_odeint( # torchdiffeq.odeint( File "/usr/local/lib/python3.10/site-packages/chirho/dynamical/internals/backends/torchdiffeq.py", line 127, in _batched_odeint yt_raw = torchdiffeq.odeint(func, y0_expanded, t, **odeint_kwargs) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/odeint.py", line 79, in odeint solution = solver.integrate(t) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/solvers.py", line 34, in integrate solution[i] = self._advance(t[i]) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/rk_common.py", line 227, in _advance self.rk_state = self._adaptive_step(self.rk_state) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/rk_common.py", line 292, in _adaptive_step y1, f1, y1_error, k = _runge_kutta_step(self.func, y0, f0, t0, dt, t1, tableau=self.tableau) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/rk_common.py", line 78, in _runge_kutta_step f = func(ti, yi, perturb=perturb) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl return self._call_impl(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl return forward_call(*args, **kwargs) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/misc.py", line 193, in forward t = _nextafter(t, t - 1) File "/usr/local/lib/python3.10/site-packages/torchdiffeq/_impl/misc.py", line 361, in _nextafter out = torch.nextafter(x1, x2) RuntimeError: "nextafter_cpu" not implemented for 'Long' Trace Shapes: Param Sites: Sample Sites: persistent_β dist | value | persistent_γ dist | value |

This is a TA3 issue.


docs: Reorganize structure

Move documents around to have a better structure
Testing
Follow the instructions on https://github.com/inspektor-gadget/website/tree/docusaurus to run the website locally. (be sure to checkout the right branches on both, the ig-website and inspektor-gadget repositories)
TODO (in this or another PR)

[x] Update requirements document
[x] Update arquitecture
[x] Provide core concepts (see #3229)

Follow ups

[ ] Handle reference/ig document
[ ] Sychronize quick start with README.md

ref #1184

I updated the PR to have a minimal quick start guide and move other things to reference/.

I'm merging it now. Since it's targeting a branch different than main it doesn't require an approval to be merged. I'll handle obvious things directly on that branch, and open PRs for non obvious things. We can do another review pass when merging new-docs back to main.


tessia Annual Review- 10/12

Please share any additional details on this topic
tessia Annual Review
Detail what actions or feedback you would like from the TAC
tessia Annual Review
How much time do you need for this topic?
None

Confirmed 10/9

Project renewed  by TAC vote-10/12/2023

Hi @kjijogeorge - Reminder of your annual review on 10/24 - let us know if you have any questions or need to reschedule.

Rescheduling due to TechXchange.  Emailed 9/24/24.

Waiting on response for reschedule - emailed 10/1/24.

Confirmed for 10/10


Match existing formatting

I made these changes so the setting match the existing formatting you're already doing

If you actually prefer the settings the way you had them, I can just update all the files with the new formatting instead. This way is just less invasive.


Fix #1562: consolidate reshape() tests

Based on PR #1778.

Current coverage is 87.89%

Merging #1779 into master will decrease coverage by -0.02% as of 23d7301

@@            master   #1779   diff @@
======================================
  Files          283     283
  Stmts        49317   49299    -18
  Branches      5033    5034     +1
  Methods
======================================
- Hit          43359   43329    -30
- Partial        861     864     +3
- Missed        5097    5106     +9

Review entire Coverage Diff as of 23d7301

Powered by Codecov. Updated on successful CI builds.


GO:0008440    inositol-1,4,5-trisphosphate 3-kinase activity (synonyms)

GO:0008440    inositol-1,4,5-trisphosphate 3-kinase activity
Has synonym "IP3 3-kinase activity"
Narrow (is that correct?)
This is a  IP3 3-kinase activity and presumably there are slightly different specificities, of which this is one?
So it seems broad?

In fact, shouldn't it be exact synonym of
GO:0051766   inositol trisphosphate kinase activity?
(it would be really nice if all of the IPX kinases  had the exact synonym to land you on the parent term, because it's really confusing...)

There are lots of broad and narrow synonyms on the children. It would be better if they were just placed as "exact synonyms" of the term they relate to.

Yes; clearly we have more than one i3p substrate
we have 1,4,5, and 1,3,4
ip3 would definitely be broad, and would definitely fit GO:0051766
As per all the synonyms
These appear to be derived from the EC entry (<IP_ADDRESS>) for inositol-trisphosphate 3-kinase.
1D-myo-inositol-trisphosphate 3-kinase
Inositol 1,4,5-trisphosphate 3-kinase. <<<<<<< would be narrow synonym
Ins(1,4,5)P3 3-kinase.<<<<<<< narrow
IP3 3-kinase <<<<< exact)
IP3K. <<<< broad: does not specify 1,4,5 or 1,3,4
But EC:<IP_ADDRESS> ( note that EC says the ENZYME carries out either adding to 5, or adding to 6.
We two separte activities (two rheas) :GO:0052726 (for addition to 5; RHEA:13253) and GO:0052725 (for addition to 6, RHEA:20940).
The synonyms derive from the EC number (mapped to both GO:0052726 and GO:0052725)
inositol trisphosphate 5/6-kinase.<<<<<<< sort of narrow-ish
Ins(1,3,4)P3 5/6-kinase. <<<<<<< sort of narrow-ish
IP56K. <<<<<< broad because the 1,4,5 or 1,3,4 aren't specified).
GO:0051766 is an broad grouping term made by GO. for the 3 (that is they are all IP3Ks;

In fact, I suggest that  I add the IP3K to the grouping term which currently has no synonyms.  And make sure IP3 is a broad synonym for the children (not narrow).  Or maybe 'related' ?
Then, if someone searched on IP3 they would see there were 3 children.
Again, the synonyms for the children come from the EC entries.

Another note:
for the grouping term: this name is a general name.(ie, inositol trisphosphate kinase (not saying what position gets the phosphate)
At EC, the 1,4,5 is called 'inositol trisphosphate 3-kinase, BUT the reaction is specific for the 1,4,5 substrate  and specifies that it is position 3 that gets modified, hence the separate GO term)

So if I mark the synonyms as above, could that be ok?

Yes, perfect! I really struggle with this branch and I have a had a few papers recently...

@kaxelsen I am trying to clean up term synonyms and match the definitions  with those of RHEA:
In GO many (or all, as far as I can tell) reactions had 2 hydrogens in the products, for example:

GO:0008440 inositol-1,4,5-trisphosphate 3-kinase activity
Catalysis of the reaction: 1D-myo-inositol 1,4,5-trisphosphate + ATP = 1D-myo-inositol 1,3,4,5-tetrakisphosphate + ADP + 2 H+ (RHEA:11020)

However in RHEA there is usually a single hydrogen, except for

GO:0000823   inositol-1,4,5-trisphosphate 6-kinase activity (RHEA:32359)

which has 2 ATP in the substrates and ADP and 2 H+. in the products.
I dont see why this one would be different, just I just wanted to confirm with you before making more edits.
Thanks, Pascale

Just now I only see one output H+ in RHEA:11020, but see two in RHEA:32359. All of this seems consistent with the idea that number of output H+ equals the number of input ATPs consumed in the phosphorylation reaction. And I guess the definition for GO:0008440 needs to be changed in line with this?

GO:0000823   inositol-1,4,5-trisphosphate 6-kinase activity (RHEA:32359)

which has 2 ATP in the substrates and ADP and 2 H+. in the products.
I dont see why this one would be different, just I just wanted to confirm with you before making more edits.
Thanks, Pascale

It is because RHEA:32359 produces 1D-myo-inositol 1,3,4,5,6-pentakisphosphate.
The Rhea reaction corresponding to your current text in the definition: D-myo-inositol 1,4,5-trisphosphate + ATP = D-myo-inositol 1,4,5,6-tetrakisphosphate + ADP + 2 H+. Source: MetaCyc:<IP_ADDRESS>-RXN
is RHEA:17717: 1D-myo-inositol 1,4,5-trisphosphate + ATP = 1D-myo-inositol 1,4,5,6-tetrakisphosphate + ADP + H+
Note that the synonym lpmK from GO:0000823 is linked to EC <IP_ADDRESS>, inositol-polyphosphate multikinase (https://www.enzyme-database.org/query.php?ec=<IP_ADDRESS>), that has RHEA:32359 as overall reaction with the two step reactions RHEA:17717 and RHEA:11856

@sjm41  and @rozaru  could you check, it seems likely that  your work here
https://github.com/geneontology/go-ontology/issues/28385
makes this ticket out of date?

@ValWood I am not sure what the outstanding task(s) is on this ticket. Looks like Harold and Pascale did some clean-up of synonyms in this ticket, while Rossana and I did not review synonyms in #28385 (just the xrefs, defs and relationships).
If you're happy with the current synonyms on these terms, then I think you could close this.

The 2 steps reaction for EC <IP_ADDRESS> is sorted normally now.
It is covered by:
id: GO:0000823
name: inositol-1,4,5-trisphosphate 6-kinase activity
xref: EC:<IP_ADDRESS> BROAD
xref: RHEA:17717 {source="skos:exactMatch"} EXACT
and
id: GO:0000824
name: inositol-1,4,5,6-tetrakisphosphate 3-kinase activity
xref: EC:<IP_ADDRESS> {source="skos:broadMatch"}
xref: RHEA:11856 {source="skos:exactMatch"}

For the IP3K synonym:
The grouping term is now:
id: GO:0051766
name: inositol trisphosphate kinase activity
def: "Catalysis of the reaction: inositol trisphosphate + ATP = inositol tetrakisphosphate + ADP + H+." [GOC:ai]
synonym: "IP3K" BROAD []
id: GO:0008440
name: inositol-1,4,5-trisphosphate 3-kinase activity
namespace: molecular_function
def: "Catalysis of the reaction: 1D-myo-inositol 1,4,5-trisphosphate + ATP = 1D-myo-inositol 1,3,4,5-tetrakisphosphate + ADP + H+." [RHEA:11020]
synonym: "1D-myo-inositol-trisphosphate 3-kinase activity" BROAD []
synonym: "inositol trisphosphate 3-kinase activity" BROAD []
synonym: "inositol-trisphosphate 3-kinase activity" BROAD []
synonym: "Ins(1,4,5)P3 3-kinase activity" RELATED [EC:<IP_ADDRESS>]
synonym: "IP3 3-kinase activity" NARROW [EC:<IP_ADDRESS>]
synonym: "IP3K activity" NARROW [EC:<IP_ADDRESS>]
It looks OK as IP3K activity in the literature is normally used for inositol-1,4,5-trisphosphate 3-kinase activity.
I think you can close the ticket.

Thanks both, closing.


Error: Icon collection not found

Related issue: https://github.com/egoist/tailwindcss-icons/issues/3
First of all thanks for your time in building this awesome library. I'm opening this issue because I'm still having the same issue above in version 1.1. With the error, I can't format the code using prettier.
λ node -v
v20.5.1
λ npm -v
8.3.2

Environment: Windows

Error: Icon collection "heroicons" not found. Please install @iconify-json/heroicons or @iconify/json
    at c:\Users\rafaa\Developer\something\node_modules\@egoist\tailwindcss-icons\dist\index.js:101:17
    at Array.reduce (<anonymous>)
    at getIconCollections (c:\Users\rafaa\Developer\something\node_modules\@egoist\tailwindcss-icons\dist\index.js:98:22)
    at c:\Users\rafaa\Developer\something\tailwind.config.cjs:32:32
    at evalModule (c:\Users\rafaa\Developer\something\node_modules\jiti\dist\jiti.js:1:255202)
    at jiti (c:\Users\rafaa\Developer\something\node_modules\jiti\dist\jiti.js:1:253130)
    at c:\Users\rafaa\Developer\something\node_modules\tailwindcss\lib\lib\load-config.js:37:30
    at loadConfig (c:\Users\rafaa\Developer\something\node_modules\tailwindcss\lib\lib\load-config.js:39:6)
    at CG (file:///c:/Users/rafaa/Developer/something/node_modules/prettier-plugin-tailwindcss/dist/index.mjs:2384:40916)
    at cI (file:///c:/Users/rafaa/Developer/something/node_modules/prettier-plugin-tailwindcss/dist/index.mjs:2384:40215)

Package.Json:
{
    "name": "next-start",
    "version": "13.5.2",
    "private": true,
    "scripts": {
        "dev": "next dev",
        "build": "next build",
        "start": "next start",
        "lint": "next lint",
        "lint:fix": "next lint --fix",
        "format": "prettier --write \"**/*.{js,jsx,ts,tsx,md,mdx}\"",
        "deps": "pnpm dlx npm-check-updates -x typescript --deep --interactive",
        "docker:build": "docker build . --no-cache -t $npm_package_name:$npm_package_version -t $npm_package_name:latest",
        "docker:run": "docker run --rm -it -p 3000:3000 --name $npm_package_name $npm_package_name:latest",
        "docker:shell": "docker exec -it $npm_package_name /bin/sh",
        "docker:logs": "docker logs -f $npm_package_name",
        "docker:stop": "docker stop $npm_package_name"
    },
    "dependencies": {
        "@twistail/react-toast": "^0.1.2",
        "next": "13.4.13",
        "react-dom": "18.2.0",
        "react": "18.2.0"
    },
    "devDependencies": {
        "@egoist/tailwindcss-icons": "^1.1.0",
        "@iconify-json/heroicons": "^1.1.12",
        "@iconify-json/ph": "^1.1.6",
        "@tailwindcss/aspect-ratio": "^0.4.2",
        "@tailwindcss/forms": "^0.5.4",
        "@tailwindcss/typography": "^0.5.9",
        "@types/node": "20.4.10",
        "@types/react-dom": "18.2.7",
        "@types/react": "18.2.20",
        "@typescript-eslint/eslint-plugin": "^6.3.0",
        "autoprefixer": "10.4.14",
        "clsx": "^2.0.0",
        "eslint-config-next": "^13.4.13",
        "eslint-plugin-simple-import-sort": "^10.0.0",
        "eslint": "8.47.0",
        "postcss": "8.4.27",
        "prettier-plugin-tailwindcss": "^0.5.2",
        "prettier": "^3.0.1",
        "tailwind-merge": "^1.14.0",
        "tailwindcss-debug-screens": "^2.2.1",
        "tailwindcss": "3.3.3",
        "typescript": "5.0.4"
    }
}

tailwind.config.cjs:
const colors = require('tailwindcss/colors')
const defaultTheme = require('tailwindcss/defaultTheme')
const { iconsPlugin, getIconCollections } = require('@egoist/tailwindcss-icons')

/** @type {import('tailwindcss').Config} */
const tailwindConfig = {
  content: ['src/**/*!(*.stories|*.spec).{ts,tsx}'],
  // darkMode: ['class'],
  theme: {
    extend: {
      fontFamily: {
        sans: ['Inter', ...defaultTheme.fontFamily.sans],
        mono: ['var(--jetbrains-mono-font)', ...defaultTheme.fontFamily.mono],
      },
      colors: {
        black: '#121314',
        gray: colors.gray,
        primary: colors.blue,
        secondary: colors.indigo,
      },
    },
    debugScreens: {
      position: ['bottom', 'left'],
      ignore: ['dark'],
    },
  },
  plugins: [
    require('@tailwindcss/aspect-ratio'),
    require('@tailwindcss/forms'),
    require('@tailwindcss/typography'),
    require('tailwindcss-debug-screens'),
    iconsPlugin({ collections: getIconCollections(['heroicons', 'ph']) }),
  ],
}

module.exports = tailwindConfig

Reordering the plugin's array solved it for me. Instead of adding the iconsPlugin() as the last item, I did add it as the first one, and the problem was solved.
 plugins: [
    iconsPlugin({ collections: getIconCollections(['heroicons', 'ph']) }),
    require('@tailwindcss/aspect-ratio'),
    require('@tailwindcss/forms'),
    require('@tailwindcss/typography'),
    require('tailwindcss-debug-screens'),
  ],

my previous fix only fixed it on macOS, let me know if it works now


Covalent's executer in API docs RTD

@santoshkumarradha commented on Tue Jan 18 2022

We now have exposed the executers inside covalent, but still do not yet have any API documentation of various parameters for it in https://covalent.docs.agnostiq.ai/en/latest/api/api.html. We need to expose that in RTD

Edit: Moving the preceding subtask to a different issue.

[ ]  https://covalent.docs.agnostiq.ai/en/latest/api/api.html#lattice object has some text not being displayed right new Lattice <covalent._workflow.lattice.Lattice> object.

[x] Covalent cli https://covalent.docs.agnostiq.ai/en/latest/api/api.html#covalent-restart 's doc string is not correctly formatted that options show up weirdly. Click's doctoring does not need to contain args and return value as those are not used by cli.

[ ] Re run covalent in notebooks to change the job-id to uuid everywhere and change it in references like getting started

same as #781


clusteroperator: fix missing rbac

Signed-off-by: hejianpeng<EMAIL_ADDRESS>What type of PR is this?

/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm


Remove redundant tray options.

First of all, thank you for this amazing app. Now I don't have to wait for main repo to get latest version of Discord, or do some wacky tinkering.
Is your feature request related to a problem? Please describe.
Yes, I think.
I close discord to tray all the time. Opening it is major pain because I had to click the icon and move my mouse four lines to the bottom carefully to open discord again. Not to mention, the button are labeled, Toggle instead of Open Webcord. This is counter-intuitive.
I don't quite remember how official discord did it since I can't open them now because they asked for update. But I'm 100% sure it's quick and smooth.
Describe the solution you'd like
One solution is to remove every other options in the tray. Like the About, Documentation, and Report a bug.
And instead, put them to the main window, which is already there under Help menu. Putting them in tray menu just felt like a clutter.
Describe alternatives you've considered
Keep all the options like that, but move "Toggle" to the top, and even better to change it to "Open" or "Restore". It's less clean but at least opening discord from tray will be smoother and less annoying experience.
How other apps did it.
For reference, Hamsket only has 2 options, Show/Hide and Quit. While Ferdium has 3 options: Show, Disable audio, and Quit. (Order are from top to bottom). All of them have Show as first options.

I'm also using tray quite frequently, but got used to that. Not to blame you, but I guess it is quite relative experience for everyone what you find painfull. I also have status bar placed on top. Personally I'm happy with the current tray implementation, maybe would like to see Electron supporting left-click action with libappindicator (we got multiple buttons on mice for a reason – we want do many different actions with these).

Opening it is major pain because I had to click the icon and move my mouse four lines to the bottom carefully to open discord again.

This is only true if status icons are placed on top of the screen. In other cases, the bottom option is easier to access. AFAIK there's currently no way to really know how popup would render.

I don't quite remember how official discord did it since I can't open them now because they asked for update. But I'm 100% sure it's quick and smooth.

Discord has Open Discord option at top, excluding the option with app name that WebCord doesn't have. But as Discord app is likely designed for Windows users, I doubt this is something people would find intuitive.
However, there's an alternative to the menu that should be more intuitive to some. See https://github.com/SpacingBat3/WebCord/issues/226. On Linux, it does only work if GtkStatusIndicator is used for rendering a tray icon/menu. Normally you're probably using libappindicator and in that case, you might wait for this PR that has been opened quite recently: https://github.com/electron/electron/pull/35319.

(...)  and even better to change it to "Open" or "Restore".

There's a reason it is called Toogle, it literally toogles the visibility of the window unlike to Discord that can only make the window visible. Calling it open or restore would mean this action can only make window visible. That *is not what I want from this action to be implemented. For me, it's more intuitive to show Discord only when it isn't focused or visible and otherwise hide it.

There's a reason it is called Toogle, it literally toogles the visibility of the window

Eh is that how it actually works? It doesn't do that in my installation.

Calling it open or restore would mean this action can only make window visible.

That's exactly what Toggle button do with webcord in my installation. Is it because I'm on linux version?
When discord window is minimized to tray, pressing it will make it appear. But when window is not minimized to tray, pressing it will not make it minimized or hide them in any ways.

However, there's an alternative to the menu that should be more intuitive to some. See https://github.com/SpacingBat3/WebCord/issues/226.

It does sound good, and I was thinking of suggesting that, but for every single electron apps I used, none of those is doing that. I assume it's impossible to do. I'm using gnome extension.
While changing label doesn't sound interesting, how about the part about removing redundant tray options like About, Documentation, and Report a bug? It's already there in "Help" menu. Why also put them in tray?
With this, it doesn't matter if the tray is on top panel, bottom panel, middle panel, or even solar panel. Restoring window that are minimized to tray will be smoother on desktop who doesn't have left click tray thing.

That's exactly what Toggle button do with webcord in my installation. Is it because I'm on linux version?

No, but this might be an issue specific to your DE. This is because WebCord also checks for the focus and if WebCord's window is losing focus each time tray menu is being shown, WebCord will interpret window as not focused and try to bring its focus back.
I'm also on Linux (XFCE) and for me Toogle works fine. What's your DE?

See https://github.com/SpacingBat3/WebCord/commit/2f39b40a3bb788b4a4410851509782d503d97a07, it moves Toggle to the top of the menu and might fix an issue with it which you have described.


Can you please be a bit clearer in your bug description.
Hmm, may have to log a bug with github themselves...


Updates to latest Zipkin 1.16.2

Here are the features relevant to sleuth

MySQL support of 128-bit group-by
don't stack overflow on json write bug
more forgiving of IPv4-mapped addresses in json

Should we also update this - https://github.com/spring-cloud-samples/brewery/blob/master/zipkin-server/build.gradle#L13 ?

yeah. there are a few UI fixes since then, and always nicer to have the
versions aligned


V1.0.31 rc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: hyunfa:x: github-actionsgithub-actions seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


ARROW-10040: [Rust] Add slice that realigns Buffer

Has the consequence of removing the alignment limit on bool kernels.
It however comes at the cost of slower buffer manipulation.

@jhorstmann @paddyhoran this is related to the alignment fixes made recently. I noticed while reviewing another PR that we had a limitation on boolean kernels if offsets weren't a multiple of 8. So I've implemented a slice on Buffer that copies the buffer and removes the offsets.
I expect there to be a minor perf impact, and I only use the above slice method when necessary.
It's very likely that my implementation can be improved upon, but I'm at my current limits on bit manipulation. Please see https://godbolt.org/z/bMc5qd for some of the assembly generated. Not happy with the instruction count, and would appreciate some guidance or improvements. Feel free to push any changes directly on this branch.

Hi @nevi-me I guess I have to thank you for pushing this topic. I played around today with a different approach today that uses an iterator over the validity mask and returns bits in chunks of u8/u16/u32/u64. This can then be used together with a chunked iterator over the typed data of buffer to efficiently implement kernels. So far it's more of a proof of concept because it doesn't handle the remainder bits of this chunking yet. It's also largely untested. Generated code for a kernel looks very very nice, the inner loop gets unrolled and it even uses avx512 mask registers if available:
https://rust.godbolt.org/z/qGcrW8
This approach could also be used with explicit vector instructions in the inner loop, and a scalar loop for the remainder. Ensuring no out of bounds reads if the chunk size matches the vector register size.
Big todos are the remainder part and a lot of tests.

My proof of concept is available at https://github.com/jhorstmann/bititer-poc/blob/master/src/lib.rs but I'm not sure when I will have time to integrate it with arrow.

@jhorstmann can I close this PR, and rely on your implementation when ready? Also, do you think we'd be able to use your implementation in parquet, as we might need that for iterating through Boolean arrays when converting from Arrow to Parquet.

@nevi-me can you point me to the part of the parquet code that you have in mind? I found the BitReader used by bit packed encoding but that seems to solve a more general problem of reading arbitrary bit widths and also seems to work on a different parquet::util::memory::Buffer implementation instead of arrow::Buffer.

Hey @jhorstmann, I haven't had time to look, but maybe I'm confused. What I recall is that I needed a way of converting a Buffer to a Vec<bool> when I write to Parquet. It might even be that I don't need to do this though, but I'll see when I get time to work on the Arrow Parquet writer some more.
I'm closing this PR


fix: prevent sentry errors for on premise subscriptions

Changes
Changes the log message to warning to stop sending events to Sentry when an on-premise enterprise subscription is created in Chargebee.
How did you test this code?
Updated unit test.

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

Comparison is base (a0eefdd) 95.64% compared to head (76f1066) 95.64%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2948   +/-   ##
=======================================
  Coverage   95.64%   95.64%
=======================================
  Files        1012     1012
  Lines       29154    29182   +28
=======================================
+ Hits        27883    27911   +28
  Misses       1271     1271

Files
Coverage Δ

api/organisations/tests/test_views.py
100.00% <100.00%> (ø)

api/organisations/views.py
91.39% <100.00%> (ø)

... and 2 files with indirect coverage changes

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


Simplify/Cross-Platform-ize Build

Includes #175
The additional changes here remove the need for InjectModuleInitialize.exe and manually setting up the PostBuild.bats.
All that should be necessary to get the build up-and-running from a fresh repo clone is to copy the necessary dlls into the ThirdParty folder and restore the NuGet packages.
These changes remove the automatic copying of dlls into the RimWorld mods folder directly, they just get copied into _Mod.  What I've done locally for development purposes as a replacement is create symbolic links(1).  This keeps the 'mod' in the RimWorld Mods folder updated without unnecessary copying.  It's also helpful when you have multiple RimWorld versions to test against, as you can have two symlinks point to the same place.
In addition, these changes allow building on OSX in Xamarin Studio (I assume it'll work the same in MonoDevelop, if you have that).  To achieve that, I found a NuGet package that replaces InjectModuleInitializer.exe and I replaced the batch scripts with msbuild commands in the csprojs.
(1)
Symbolic links can be setup with the following console commands:
Windows
In a console window run as Administrator inside the RimWorld Mods directory:
mklink /D "{ModName}" "C:\Path\To\CommunityCoreLibraryRepo\_Mod\{ModType}\{ModName}"
In my case:
mklink /D CCL "C:\Users\etinquis\Projects\CommunityCoreLibrary\_Mod\User Release\Community Core Library"
OSX and very probably Linux
In a console window inside the RimWorld Mods directory:
ln -s "/path/to/CommunityCoreLibraryRepo/_Mod/{ModType}/{ModName}" "{ModName}"
In my case:
ln -s "/Users/etinquis/Projects/CommunityCoreLibrary/_Mod/User Release/Community Core Library" CCL

symlink needed for the "mod tweaks" project, too?

Yeah, any of the mod folders that you want to load in the game.  You would just substitute {ModName} in the examples for Community Core Library - Mod Tweaks or Community Core Library - Vanilla Tweaks.

Are you using Zhentar's forked tool to simplify making these pulls? https://ludeon.com/forums/index.php?topic=16599.msg265091#msg265091

@FudgePoly No, the changes from #175 were made with very little decompilation/analysis at all (https://github.com/RimWorldCCLTeam/CommunityCoreLibrary/pull/175#issuecomment-245957535).
The additional changes in this pull request don't include anything that would benefit from a decompiler; it's just build process changes for the project itself.

The post build process is also doing local copies for individual developers.  It can't be merged into the main build process.  That being said, I'll look into the the module initializer nuget to see what it's doing and if it performs the functions needed.

@ForsakenShell Can you expand on what you mean re: local copies for individual developers?  Did you mean things that aren't in PostBuild.txt?

They are in your local PostBuilt.bat (which is a copy of PostBuild.txt).  Putting any local copies in the .csproj file will mean that it would attempt those copies on all developers machines.  Further, the PostBuild process is there for future expansion and was deliberately set up like it is.  We will not be removing that particular step in the build process.


feat: install command

The goal of this PR is to lay the groundwork for installers: classes that can publish files into a user's project. I've already converted the existing install command to use the new approach. Another example are the auth migrations.
Let's say this PR gets merged, than we could make an auth installer like so:
final readonly class AuthInstaller implements Installer
{
    use PublishesFiles;

    public function getName(): string
    {
        return 'auth';
    }

    public function publishFiles(): void
    {
        $this->publish(
            source: __DIR__ . 'User.php',
            destination: root_path('src/User.php'),
        );

        $this->publish(
            source: __DIR__ . 'UserPermission.php',
            destination: root_path('src/UserPermission.php'),
        );

        $this->publish(
            source: __DIR__ . 'CreateUsersTable.php',
            destination: root_path('src/CreateUsersTable.php'),
        );

        $this->publish(
            source: __DIR__ . 'CreateUserPermissionsTable.php',
            destination: root_path('src/CreateUserPermissionsTable.php'),
        );

        // …
    }
}

And run it like so:
./tempest install auth

Here's a demo of the framework installer:
https://github.com/user-attachments/assets/2d446dd8-4bef-4061-9782-4918ecb973fa
A couple of things to note:

We cannot hardcode src, since some project might have another src directory configured. I think @innocenzi already looked into this, it might even already be available somewhere. We probably need some kind of src_path helper as well.
Currently it only allows for publishing files. Maybe we need more flexibility. @gturpin-dev is working on make commands which will depend on this PR. Maybe you can provide some feedback @gturpin-dev ? What am I missing that you'll need?

Overall, would like the input of @aidan-casey , @innocenzi and @gturpin-dev
TODO:

[x] Add test helpers to make it easy to create testing install directories, and refactor both the FrameworkInstaller and AuthInstaller tests
[x] Add finish hook to allow an installer to eg. run migrations

I like that a lot!
When the kernel boots, it instantiates the \Tempest\Support\Composer class, which holds the main namespace of the user. You can use it to replace the hardcoded src:

https://github.com/tempestphp/tempest-framework/blob/main/src/Tempest/Core/src/Composer.php#L15
https://github.com/tempestphp/tempest-framework/blob/main/src/Tempest/Core/src/ComposerNamespace.php#L11

It should look like that:
root_path($this->composer->mainNamespace->path);

Additionally, we'll still be missing a way to publish files that are supposed to be discovered by Tempest, like migrations. We still need #[DoNotDiscover], so we need to be able to remove it before publishing through ClassManipulator.
While we could do it using the callback, I think it's better to provide a simple API for it in PublishesFiles. But in my opinion we should do that in another pull request, to limit the scope of this one

Additionally, we'll still be missing a way to publish files that are supposed to be discovered by Tempest, like migrations. We still need #[DoNotDiscover], so we need to be able to remove it before publishing through ClassManipulator.

Yes, I remembered this when I was going to bed yesterday as well. We need a way to make changes (remove an attribute, change the namespace). I think it makes sense to have support for that within this PR. Gonna tinker with it

Allow the installer to specify a discovery location?

@aidan-casey I don't see a reason for it. It's already possible by altering the Kernel. We would need a Kernel::BOOTING event, to be able to add discovery locations BEFORE the kernel is booted. But that's outside the scope of this PR.

One thing still missing is the ability to automatically run migrations with the newly added files. I made a separate issue for that: #627

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

165 of 178 (92.7%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 82.315%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Tempest/Console/src/Input/ConsoleArgumentBag.php
0
2
0.0%

src/Tempest/Generation/src/ClassManipulator.php
11
13
84.62%

src/Tempest/Core/src/PublishesFiles.php
16
19
84.21%

src/Tempest/Core/src/Commands/InstallCommand.php
12
18
66.67%

Totals

Change from base Build<PHONE_NUMBER>6:
0.2%

Covered Lines:
6963

Relevant Lines:
8459

💛  - Coveralls

I'm late, but I like this addition 👌
We could refactor the PublishFiles to use the Generator package on a later PR


gitlab: Update to 11.9.0

Motivation for this change
Just a regular version bump.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This seems to be mixed together with https://github.com/NixOS/nixpkgs/pull/57578.
Can you drop the commit?
What about gitlab-rails, which was added to nixos/modules/services/misc/gitlab.nix? I don't see it referenced anywhere, and wonder how this can be used…

@flokli, I've pushed a branch isolating the version bumps and have split out the gitlab-rails change into #58284.


Remove the dependency on the ruby racer.

I love the gem, but instead of having a dependency on therubyracer, we can leave the selection of the JS runtimes up to the user (node, therubyrhino, JavaScriptCore). This convention follows the jshintrb gem and gives more flexibility with no functional loss.
Updated the minor version since some users may depend on this gem to include therubyracer.

This looks good - thanks for your PR.
I'll probably not merge this till #28 is resolved, as they both seem like issues that ought to go in to v2. Hope that's cool with you :smile:

No worries, looking forward to the next version of this gem!


Karpenter Under-Provisioning Nodes

Version

Karpenter Version: v0.18.1

Kubernetes Version: v1.21.14 (EKS)
Expected Behavior
Karpenter should ensure that the provisioned nodes have enough space for both DaemonSets as well as any Deployments and StatefulSets.
Actual Behavior
We are migrating our various EKS clusters to use Karpenter to provision nodes. We have an array of DaemonSets and Deployments we create as part of the initial cluster setup. In one of our clusters, we are seeing that Karpenter is provisioning nodes as small as at3.medium, which cannot accommodate both the DaemonSets and the Deployments. This leads to a loop where:

A new node is spun up.
DaemonSet pods begin to be placed on nodes. In some instances, the node is not even large enough to accomodate all DaemonSets
Deployment pods begin to be placed on nodes. In the case that all DaemonSets have been placed, there is usually not enough room for the deployments.
Because there are only DaemonSets on the node, Karpenter sees that the node is empty and terminates after the  ttlSecondsAfterEmpty period has expired.
Repeat

Sometimes after a while, Karpenter will instead spin up a larger instance and the deployments may finally be placed.
In some cases, we see that a deployment will take priority over one of the DaemonSet pods, which in turn creates a scenario where the DaemonSet pod cannot be placed. This could be caused by priorityClass, but the issue still stands.
To fix this we have had to limit the provisioner from using anything below an xlarge to force it to provision a node that can accommodate all pods.
The language in Scheduling section of the Concepts page in the docs has also been confusing to decipher for us, and we could use some clarification. I have pasted the text below:
Karpenter schedules pods that the Kubernetes scheduler has marked unschedulable.
After solving scheduling constraints and launching capacity, Karpenter creates the Node object
and waits for kube-scheduler to bind the pod. This stateless approach helps to avoid race conditions
and improves performance. If something is wrong with the launched node, Kubernetes will
automatically migrate the pods to a new node.

Initially it states that Karpenter schedules pods, but it seems as if Karpenter provisions nodes based off the calculated capacity of unschedulable pods. Once the node is ready, the kube-scheduler is what is responsible for binding the pod. Additionally, this issue further indicates that Kubernetes does not act as the pod scheduler: https://github.com/aws/karpenter/issues/1097. Is this just a misunderstanding on our end?
If it is true that Karpenter is not scheduling the pods, one of our theories is that Karpenter provisions these under-sized t3/t3a instances because it is aware of their burst capacity, but the kube-scheduler - who is responsible for binding the pods - sees an under-sized node that cannot accomodate the pods.
Steps to Reproduce the Problem

Install Karpenter
Create Provisioner with the Configuration included in the Resource Specs section
Install DaemonSets and Deployments (Simultaneous):

DaemonSets

calico-node
csi-node-driver
datadog
ebs-csi-node
fluentd
kube2iam

Deployments  (Some company specific deployments are not listed)

aws-load-balancer-controller
external-dns
k8s-metrics-server
reloader
datadog-kube-state-metrics

Karpenter will cycle through nodes until a large enough node is provisioned to fit at least one deployment

Resource Specs and Logs
apiVersion: karpenter.sh/v1alpha5
kind: Provisioner
metadata:
  annotations:
    meta.helm.sh/release-name: karpenter-on-demand-provisioner-chart
    meta.helm.sh/release-namespace: default
  labels:
    app.kubernetes.io/managed-by: Helm
spec:
  labels:
    env: karpenter-on-demand
  provider:
    launchTemplate: >-
      launch-template-karpenter-ingress-staging-1-default20220513134204268900000001
    subnetSelector:
      Service: Private
      kubernetes.io/cluster/ingress-staging-1: '*'
  requirements:
    - key: karpenter.sh/capacity-type
      operator: In
      values:
        - spot
    - key: kubernetes.io/arch
      operator: In
      values:
        - amd64
    # The below requirements are still insufficient
    - key: karpenter.k8s.aws/instance-size
      operator: NotIn
      values:
        - nano
        - micro
    - key: node.kubernetes.io/instance-type
      operator: NotIn
      values:
        - t3a.small
        - t3.small
  ttlSecondsAfterEmpty: 30

Karpenter terminating node considered empty because of inability to place Deployment pods:

Unfortunately have lost the screenshot, but the scheduler complains of a node with insufficient CPU, and sometimes of a node with too many pods on it. The latter was very puzzling as no such node existed as far as we could tell.
Community Note

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

Hi @thandleman-r7, can you share the pod specs of the deployments you are trying to provision? Have you set pod requests on all the deployments and DS?

@jonathan-innis Updated the issue with some deployments. Resources are not set for a handful of Deployments and DaemonSets. Will this be our issue?

@thandleman-r7 I don't see the deployments that you updated the issue with. Yes, it would be an issue if your deployments or daemonsets don't have requests. This is what Karpenter uses to determine what size of nodes that it needs to launch and how many nodes that it needs to launch.
If there are no requests for some of your workloads, Karpenter may think it has enough capacity to schedule all the workloads when there is actually not enough capacity for all the workloads to run on all the nodes appropriately. There are best practices around setting resource requests/limits that you can find in a few of the links below:

https://aws.github.io/aws-eks-best-practices/reliability/docs/dataplane/#reserving-resources-for-system-and-kubernetes-daemons
https://aws.github.io/aws-eks-best-practices/reliability/docs/dataplane/#configure-and-size-resource-requestslimits-for-all-workloads


run error

I installed digits 2.0 follow the Installation Instructions. But when I run runme.sh, I meet an error:
cudaRuntimeGetVersion() failed with error #35
Then I run the runtest.sh. The results are as follow:
FAIL: digits.test_webapp.TestModelCreation.test_alltests('GRAY',)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lq/digits-2.0/venv/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/lq/digits-2.0/digits/digits/test_webapp.py", line 652, in check_snapshot_interval_2
    assert self.model_wait_completion(job_id) == 'Done', 'create failed'
AssertionError: create failed
-------------------- >> begin captured logging << --------------------
digits: DEBUG: Train Caffe Model task started.
digits.webapp: INFO: Train Caffe Model task started.
digits.webapp: ERROR: Train Caffe Model: Cannot create Cublas handle. Cublas won't be available.
digits.webapp: ERROR: Train Caffe Model: Cannot create Curand generator. Curand won't be available.
digits.webapp: ERROR: Train Caffe Model: Cannot create cuDNN handle. cuDNN won't be available.
digits.webapp: ERROR: Train Caffe Model: Check failed: error == cudaSuccess (35 vs. 0)  CUDA driver version is insufficient for CUDA runtime version
digits.webapp: ERROR: Train Caffe Model task failed with error code -6
--------------------- >> end captured logging << ---------------------

======================================================================
FAIL: digits.test_webapp.TestModelCreation.test_alltests('GRAY',)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lq/digits-2.0/venv/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/lq/digits-2.0/digits/digits/test_webapp.py", line 661, in check_snapshot_interval_0_5
    assert self.model_wait_completion(job_id) == 'Done', 'create failed'
AssertionError: create failed
-------------------- >> begin captured logging << --------------------
digits: DEBUG: Train Caffe Model task started.
digits.webapp: INFO: Train Caffe Model task started.
digits.webapp: ERROR: Train Caffe Model: Cannot create Cublas handle. Cublas won't be available.
digits.webapp: ERROR: Train Caffe Model: Cannot create Curand generator. Curand won't be available.
digits.webapp: ERROR: Train Caffe Model: Cannot create cuDNN handle. cuDNN won't be available.
digits.webapp: ERROR: Train Caffe Model: Check failed: error == cudaSuccess (35 vs. 0)  CUDA driver version is insufficient for CUDA runtime version
digits.webapp: ERROR: Train Caffe Model task failed with error code -6

By the way, Caffe is working well in my computer. CUDA version is 6.5. GPU model is Tesla K40. It's driver version is 340.29.
According to the test results, I think the CUDA may result in the error. How to solve this problem. Thanks!

Hi @Michael-Guo, the 2.0 web installer comes with CUDA 7.0, which requires driver version 346.xx. Can you update your Nvidia driver?

@Michael-Guo are you happy with the answer? Thanks!


'vColor' : undeclared identifier

When I try to cast shadows a WebGL error says: 'vColor' : undeclared identifier
Here's a codesandbox that shows the error: https://codesandbox.io/p/sandbox/37gxj9
Steps to reproduce:

navigate to the codesandbox link.
await for the preview to load and display the scene.
open CS 'Chrome WebTools for the preview'
select the Console tab.
you should see the mentioned error.

Note that if you comment out lines 42 and 45 (were the color is assigned) and reload, the error disappears.
There's a clue I can't fully understand but if you load the preview without assigning color to instances (with commented lines), the boxes are white, then uncomment the mentioned lines and after hot reload the boxes display the right color, shadows and no error is present in the console.
Maybe a race condition during the shader compilation?
Hope this can help to find a bug if it's there.
Great lib btw 🙌

I found this: https://github.com/agargaro/instanced-mesh/blob/676e7f326a17b8ad80d5994258299f260a9e6597/src/objects/InstancedMesh2.ts#L22
Is it related to this issue?

I made a local fix but not sure if it make sense.
I split this file https://github.com/agargaro/instanced-mesh/blob/master/src/shaders/chunks/instanced_vertex.glsl.ts into two, one with the instanceMatrix assign and another one with the instanceColor. Then I concatenated the instance color chunk to the 'shadowmap_vertex'.
shaderChunk.ts
import instanced_vertex from "./chunks/instanced_vertex.glsl.js";
import instanced_vertex_color from "./chunks/instanced_vertex_color.glsl.js";

...

ShaderChunk["instanced_vertex"] = instanced_vertex;
ShaderChunk["instanced_vertex_color"] = instanced_vertex_color;

...

ShaderChunk["batching_vertex"] = ShaderChunk["batching_vertex"].concat("\n#include <instanced_vertex>"); // TODO fix d.ts
ShaderChunk["shadowmap_vertex"] = ShaderChunk["shadowmap_vertex"].concat("\n#include <instanced_vertex_color>");

What do you think?

Sorry for the delay! I will investigate and let you know asap

@Cosmitar
The problem occurs when patching a material that has no color (such as MeshNormalMaterial, MeshDepthMaterial, etc).
We can avoid patching the color of materials that don't have #include <color_pars_fragment> to fix the bug.
Thank you very much for the report and sorry for the late reply (I've not seen the notification 😅). If you need, you can always find me on the discord channel linked in the README 😄

I've released a new version (0.2.4) to fix this.
This is an example to test the fix:
https://stackblitz.com/edit/three-ez-bug-fix-path-color-material?file=src%2Fmain.ts
PS. you can write it also in this way
obj.color = 'red';

Works fine for me, after updating to 0.2.4 the sandbox shows the shadows as expected with no error. Also tested in local project.
Grazie mille!

Grazie a te! ^^


FOUTA : Flash of unstyled text area

Fix the flash of unstyled <textarea> for input and output on init page load w/a slight transition or animation.

done


Wrong computation for CC_LAST_4

See PluginPaymentDao line 163
It currently reads
final String ccLast4 = ccNumber == null ? null :
        ccNumber.substring(ccNumber.length() - 5, ccNumber.length() - 1);

Which produces for a card like<PHONE_NUMBER>123456 a value of 2345. It should read:
final String ccLast4 = ccNumber == null ? null :
        ccNumber.substring(ccNumber.length() - 4, ccNumber.length());

Closing this issue as it's fixed in #3


Unable to open serial connection

Hi,
I'm trying to connect Datalogic GD4400 scanner with my Android (version 6.0) tablet.
I was able to get pass requestPermission step, but at this point, can't open connection.
Tried using scanner default configuration with cordovarduino defaults (which should be equal), but with no luck.
Not really sure which scanner interface to use:

RS-232 standard or
USB-COM STD (simulates RS-232 standard interface)

`serial.requestPermission({
vid: '05F9',
pid: '4204',
driver: 'ProlificSerialDriver'
},
function success()  {
serial.open(
{dtr: false},
function success() {
alert("Success");
},
function error(evt) {
alert("Error: Unable to open serial connection! ");
});
},
function error(evt) {
alert("Error: " + evt);
});`
Any help would be highly appreciated.

It is very hard for me to debug without the hardware. Please give me more inputs about the serial connexion your scanner provides.

Hi xseignard,
I'm beginner to cordova, I have to read/write data from weighing scale which have rs232 (9 pole) port. I'll connect the device to my mobile using rs232-USB and USB OTG cable. I used the same above code with only VID and PID.
But I could not able to open the port and it gives no error. The serial settings given in device User's manual is: 8-bit ASCII code; 8 data bits; 1 stop bit; no parity bit; baud rate 9600.
Can you please help me to find out what am I missing?

Hello,
Could you please show me your code?
Also, could you please check you can see data being sent to a computer through the arduino serial monitor (http://www.instructables.com/id/HOW-TO-use-the-ARDUINO-SERIAL-MONITOR/)
Regards

Hi, please find the code which I used
function serialDataRead() {
	alert('Start');
	var errorCallback = function(message) {
		alert('Error: ' + message);
	};
	serial.requestPermission({vid: '1a86', pid: '7523'},
		function success(successMessage) {
			alert(successMessage);
			alert('Before Open');
		serial.open(
			{baudRate: 9600},
			function success(successMessage) {
				alert('Port opened');
				serial.read(
					function success(successMessage) {
						alert(successMessage);
					},
				errorCallback);
			},
			errorCallback);
		},
		errorCallback);
}

I didn't use arduino serial monitor, but I can see data being sent to the computer using putty serial connection.
I get the alert message upto 'Before Open', then app quits itself.

Hi, I changed the code as below and now I get this (Error: init failed!) errorCallback message.
...
serial.requestPermission({vid: '1a86', pid: '7523', driver: 'Ch34xSerialDriver'},
...
{baudRate: 9600, dataBits: 8, stopBits: 1, parity: 0},
...

Do you have a link to your rs232/usb adapter? and to the scale?

Sorry I don't have a link to my rs232/usb adapter, here is the link to the scale.

I get below information when I connect the device with Ubuntu (lsusb):

ID 1a86:7523 QinHeng Electronics HL-340 USB-Serial adapter

And found this link.

Hello,
Could you please send me a link to see the technical specs of your rs232/USB adapter? Also the specs of your weighing scale?
Regards

Could you try the following ?
serial.requestPermission({vid: '1a86', pid: '7523', driver: 'ProlificSerialDriver'}

I tried with above driver change, and now it shows this error message:
Error: ControlTransfer with value 0x0 failed: -1

Are you sure about the conf here?
{baudRate: 9600, dataBits: 8, stopBits: 1, parity: 0}

I'm not sure about the conf of adapter, is it possible to configure the adapter with same conf?

unfortunately I'm not sure, when your run putty serial monitor, do you see the data decoded right?
how are your serial settings on putty?

When I run putty, I can see the data decoded correctly and putty serial settings is same as above (baudRate: 9600, Data bits: 8, Stop bits: 1, Parity: 0)

My guess is that the driver of the usb chip (presumably CH340/CH341) on your rs232/usb cable is not working.
That's weird because I commited the following https://github.com/mik3y/usb-serial-for-android/blob/master/usbSerialForAndroid/src/main/java/com/hoho/android/usbserial/driver/Ch34xSerialDriver.java
And tested it with some arduino clones that have a CH340 usb chip and it was working.
Could you please take a picture of the electronics of your cable so I can check what is the chip you have in it?
Maybe you could also try to find a rs232/USB cable with a more reliable usb chip? such as PL203
This one https://www.amazon.com/UGREEN-Adaptateur-Compatible-Windows-Supérieur/dp/B00QUZY4WO/ref=pd_sim_147_3?_encoding=UTF8&psc=1&refRID=4R5A9YG2809PCFQKGYZQ
Regards

PS: I would really love to see this working because RS232 to android through this plugin is a recurrent question.
Unfortunately, I don't have any matching hardware to test it.
Regards

My guess is that the driver of the usb chip (presumably CH340/CH341) on your rs232/usb cable is not working.

You maybe right.

Maybe you could also try to find a rs232/USB cable with a more reliable usb chip? such as PL2303
This one https://www.amazon.com/UGREEN-Adaptateur-Compatible-Windows-Supérieur/dp/B00QUZY4WO/ref=pd_sim_147_3?_encoding=UTF8&psc=1&refRID=4R5A9YG2809PCFQKGYZQ

I will buy and try with a reliable usb chip cable like you referred.
Thank you so much for the support and I hope it should work.

Hi @xseignard
I'm happy to share you that after using rs232/USB adapter with reliable usb chip, I'm able to read data from weighing scale, will continue with write process as well.
Thank you so much for the quick responses and for giving right instructions to achieve my requirement, also for the helpful plugin.

That's great news @sureshkumar-s89 !
Would you mind post a link to the adapter you used? For other people to know what adapter is working?
Regards

Hi @xseignard
Sorry for the delayed response, unfortunately I don't have the link for adapter, it was an old one bought from near by retailer.

Could you at least take some pictures? And if possible write down the references that you can read on the chips of the cable?
Regards

It looks exactly same like this https://www.amazon.in/PruthvikTMUSB-Serial-Adapter-Desktop-Through/dp/B074W2CYCZ/ref=sr_1_8?ie=UTF8&qid=1513331616&sr=8-8&keywords=rs232+usb.
Sorry I do not know how to read references on the chips.

Thanks! I close the issue.
Feel free to reopen it.
Have fun!


React cilent-side scripts directly from server?

Hi React.NET team,
Firstly, thanks very much for developing this great library. The JSX compilation a fantastic feature and I'm using it extensively on my website.
Secondly, apologies if this is more of a question than an issue, but I did some honest Googling and couldn't find any links to forums etc, so here goes!
Yesterday I came across an issue whereby some organisations have blocked Facebook in their workplaces, so the React client-side scripts failed to load from fb.me:
<script src="https://fb.me/react-15.0.1.js"></script>
<script src="https://fb.me/react-dom-15.0.1.js"></script>

I've resolved the issue by switching to unpkg.com, but I was wondering if it's possible to serve the React scripts directly from the server using the same scripts referenced by the server-side compilation feature? Serving these scripts would also ensure there is always parity between server-side and client-side behaviour should the React scripts change.
Thanks again for the great work with React.NET!

Hi @NuclearFishin! You could load React from CDNJS instead, if you like.

serve the React scripts directly from the server using the same scripts referenced by the server-side compilation feature

This is tricky. The scripts used for server-side compilation are stored as embedded resources in the React.Core.dll assembly. Serving them would mean there'd have to be a controller that loads the resource and serves it. A controller to do that could be added, but it's easier to just have a copy of React in your project and reference that directly (or use CDNJS).
I'm going to close this issue since it's a question, but feel free to ask if you have any other questions.

Thanks @Daniel15 for your quick response! I went for unpkg.com because it's the source used in the React getting started guide, but yes CDNJS would work just as fine!
Indeed my requirement for loading server-side may be too esoteric for inclusion in the main libraries, but I am thinking that some kind of mechanism for maintaining parity with my server-side scripts would be a useful feature? The use-case I'm thinking of is that I update my React.NET Nuget packages, and now the server-side React scripts would be out of sync with my client-side ones.
A very minimal way to address this concern would be to expose a property detailing the React version used by the server-side? Then consumers like me could use that in our script references, such as:
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/@(reactVersion)/react.min.js"></script>
What do you think? Does such a property exist?


feat: support podman

This patch adds support for running service using Podman. There are a few notable changes that were required to enable Podman support, described below.
Podman keeps HostIP empty instead of using <IP_ADDRESS>. This behavior required an update to extractIPPort. More information in https://github.com/containers/podman/issues/17780.
Podman prefers to use fully-qualified container images (i.e., example.com/foo/bar:latest) but will perform short-name aliasing for unqualified container images. When an unqualified image such as foo/bar:latest is loaded into a kind cluster it will be stored as localhost/foo/bar:latest.
Using backticks for command substitution isn't POSIX compatible. Switched the backticks to $() instead to support alternative shells (e.g., /bin/fish).

Thank you for this. I will review in the morning. If I have questions I'll reach out.


Return type of bitwise_xxx_agg doesn't match Presto's

Description
Window fuzzer found that the return type of bitwise_xxx_agg in Presto always return BIGINT, while they in Velox return the same type as the input.
Error Reproduction
Executing query plan:
-- Window[partition by [p0, p1] order by [s0 ASC NULLS LAST] w0 := bitwise_xor_agg(ROW["c0"]) RANGE between UNBOUNDED PRECEDING and CURRENT ROW] -> c0:TINYINT, s0:ARRAY<BOOLEAN>, p0:REAL, p1:BIGINT, row_number:BIGINT, w0:TINYINT
  -- Values[1000 rows in 10 vectors] -> c0:TINYINT, s0:ARRAY<BOOLEAN>, p0:REAL, p1:BIGINT, row_number:BIGINT

Relevant logs
I20231219 20:28:21.981456 2392528 WindowFuzzer.cpp:64] ==============================> Started iteration 36 (seed:<PHONE_NUMBER>)
I20231219 20:28:21.987654 2392528 AggregationFuzzerBase.cpp:364] Executing query plan:
-- Window[partition by [p0, p1] order by [s0 ASC NULLS LAST] w0 := bitwise_xor_agg(ROW["c0"]) RANGE between UNBOUNDED PRECEDING and CURRENT ROW] -> c0:TINYINT, s0:ARRAY<BOOLEAN>, p0:REAL, p1:BIGINT, row_number:BIGINT, w0:TINYINT
  -- Values[1000 rows in 10 vectors] -> c0:TINYINT, s0:ARRAY<BOOLEAN>, p0:REAL, p1:BIGINT, row_number:BIGINT
I20231219 20:28:22.011268 2394184 Task.cpp:1062] All drivers (1) finished for task test_cursor 37 after running for 22 ms.
I20231219 20:28:22.011299 2394184 Task.cpp:1746] Terminating task test_cursor 37 with state Finished after running for 22 ms.
I20231219 20:28:22.233783 2392528 Writer.cpp:546] Stripe 0: Flush overhead = 30656, data length = 10744, pre flush mem = 448192, post flush mem = 445056. Closing = true
I20231219 20:28:22.234041 2392528 FileSink.cpp:123] closing file: /var/folders/yl/2mzb3q4s0gn62ffjlfbch0w80000gn/T/PrestoTest2551609741947032008/hive_data/tpch/tmp/fuzzer.dwrf,  total size: 11.62KB
/Users/weihe/Development/velox/velox/exec/tests/utils/QueryAssertions.cpp:1063: Failure
Failed
Types of expected and actual results do not match: ROW(TINYINT, ARRAY, REAL, BIGINT, BIGINT, BIGINT) vs. ROW(TINYINT, ARRAY, REAL, BIGINT, BIGINT, TINYINT)
E20231219 20:28:22.403347 2392528 Exceptions.h:69] Line: /Users/weihe/Development/velox/velox/exec/fuzzer/WindowFuzzer.cpp:179, Function:verifyWindow, Expression: assertEqualResults( expectedResult.value(), plan->outputType(), {resultOrError.result}) Velox and reference DB results don't match, Source: RUNTIME, ErrorCode: INVALID_STATE
libc++abi: terminating due to uncaught exception of type facebook::velox::VeloxRuntimeError: Exception: VeloxRuntimeError
Error Source: RUNTIME
Error Code: INVALID_STATE
Reason: Velox and reference DB results don't match
Retriable: False
Expression: assertEqualResults( expectedResult.value(), plan->outputType(), {resultOrError.result})
Function: verifyWindow
File: /Users/weihe/Development/velox/velox/exec/fuzzer/WindowFuzzer.cpp
Line: 179

@kagamiori The issue here is slightly different. bitwise_xxx functions in Presto have only 1 signature, but in Velox there are multiple signatures. A Fuzzer may create a query plan using bitwise_and_agg(tinyint). When this plan is converted to SQL,  Presto's query planner applies implicit cast to input argument: tynyint -> bigint. Hence, the result in Presto is bigint, not tinyint.
presto:di> show functions like 'bitwise_%_agg';
    Function     | Return Type | Argument Types | Function Type | Deterministic | Description | Variable Arity | Built In | Temporary | Language
-----------------+-------------+----------------+---------------+---------------+-------------+----------------+----------+-----------+----------
 bitwise_and_agg | bigint      | bigint         | aggregate     | true          |             | false          | true     | false     |
 bitwise_or_agg  | bigint      | bigint         | aggregate     | true          |             | false          | true     | false     |
 bitwise_xor_agg | bigint      | bigint         | aggregate     | true          |             | false          | true     | false     |
(3 rows)

vs. Velox
  for (const auto& inputType : {"tinyint", "smallint", "integer", "bigint"}) {
    signatures.push_back(exec::AggregateFunctionSignatureBuilder()
                             .returnType(inputType)
                             .intermediateType(inputType)
                             .argumentType(inputType)
                             .build());
  }

CC: @aditi-pandit @kgpai


Rename navigation not working properly

I tried to rename a file and when trying to use navigation keys, the focus stayed on the explorer instead of the text for renaming the file, so when I pressed the End key it went all the way to the end of my project files.

VSCode Version: 1.19.0-insider
OS Version: win10

Steps to Reproduce:
1.Press F2 in the explorer or try to rename a file.
2.Press the navigation keys (up, down, left or right arrows, end and home).

Reproduces without extensions: Yes

have the same, it's now a nightmare to create/edit files with vim-bindings enabled.

/duplicate of #39211


Not able to type comma (,) in Select component

I don't know whether it's antd doing something fishy or it's us, but I'm not able to type comma (,) in the Select component.
This results in users not able to create new custom number formats with commas.
Screenshots

Environment
Latest master

I have the same issue

Seems to be working fine nowdays :D


Fixing serialization of ports to float

This branch is dedicated to fixing the serialization of ports of class type float.

@joamatab this fix is necessary when the serializer gets a non-standard float type it doesn't understand, i.e. something imported from a C-module...


Improve commands - order networks selection based on the selected account id

This PR is served for #222.

Finished per your review, please recheck.


Select.Value should be string | number

Describe the feature in detail (code, mocks, or screenshots encouraged)
<Select.Item value={g.value}>{g.label}</Select.Item>
if g.value is a number typescript complains. Binding works, but you might want to consider numbers for dropdowns sometime
What type of pull request would this be?
None
Provide relevant links or additional information.
No response

I ran into this issue as well. Anytime I have a drop-down based on an int-keyed database entity I need to have a number value.
If possible, it would be nice to be able to use enums as well.

It would be misleading because HTML form data is transmitted as text in the HTTP request, creating a difference in expectations once it reaches your server.

It would be misleading because HTML form data is transmitted as text in the HTTP request, creating a difference in expectations once it reaches your server.

@huntabyte Do you have any suggestions for the best way to handle this? In my situation I never use form data, only json, so the server is not expecting a string. Technically the server should be able to deserialize the stringified number into a number but I get a lot of typing errors in my svelte project.

Additionally, strings give us the ability to do form autofill and typeahead, which isn't possible with numbers, etc. so it's a trade-off I decided to make while still providing the APIs necessary to create your own custom component that's could accept whatever you wish.
You can use onValueChange to transform the data into whatever you want, whether it's a number, boolean, object, component,
Here's a very convoluted example for "selecting a component" (ideally you'd want to have a map of sorts to map string representations).
<script lang="ts">
	import { Select } from 'bits-ui';
	import { ComponentA, ComponentB, ComponentC } from 'somewhere'

	let selectedComponent = $state<Component>()
</script>

<Select.Root
	onValueChange={(v) => {
		if (v === "a") {
				selectedComponent = ComponentA
		} else if (v === "b") {
				// ...
		}
	})
>
	<!-- ... -->
</Select.Root>

or
<script lang="ts">
	import { Select } from 'bits-ui';
	import { ComponentA, ComponentB } from 'somewhere'

	let value = $state('')
	let SelectedComponent = $derived.by<Component | null>(() => {
			if (value === "a") return ComponentA
			if (value === "b") return ComponentB
	})
</script>

<Select.Root bind:value>
	<!-- ... -->
</Select.Root>

{#if SelectedComponent}
	<SelectedComponent />
{/if

or in the case of numbers
<script lang="ts">
	import { Select } from 'bits-ui';

	let value = $state('')
	let numValue = $derived(`${value}`)
</script>

<Select.Root bind:value>
	<!-- ... -->
</Select.Root>

or using controlledValue/onValueChange
<script lang="ts">
	import { Select } from 'bits-ui';

	let value = $state<number>(0)
</script>

<Select.Root value={`${value}`} controlledValue onValueChange={(v) => value = `${v}`>
	<!-- ... -->
</Select.Root>


workflow: test-images.yaml

fix bad docker images build for test images

https://github.com/confidential-containers/cloud-api-adaptor/actions/runs/8385253126

add test_e2e_libvert label to run/verify Libvirt e2e tests too.


!FindInMap issues while in Fn::ForEach, cannot find resource or parameter

CloudFormation Lint Version
cfn-lint 0.87.0
What operating system are you using?
Ubuntu
Describe the bug
I am now getting:

E1012 Ref AccountId not found as a resource or parameter

When using !FindInMap in a Fn::ForEach
Expected behavior
I expected this to pass as this deploys fine.
Reproduction template
AWSTemplateFormatVersion: '2010-09-09'
Transform: 'AWS::LanguageExtensions'

Description: Configures IAM Identity Center. IAM Identity Center must be enabled before you create this stack.

Parameters:

  IamIdentityCenterInstanceArn:
    Type: String
    Description: 'The ARN for the IAM Identity Center Instance. Found in the settings of IAM Identity Center.'

  IdentityStoreId:
    Type: String
    Description: 'The id for the identity store. Found in the settings of IAM Identity Center.'

Mappings:

  AccountIdsMap:
    # GovCloud
    "101010101010":
      DevSandAccount: "1111111111111"
      ProjectADevAccount: "22222222222222"
      ProejctADemoAccount: "333333333333333"
    # Commercial
    "101010101011":
      DevSandAccount: "1111111111112"
      ProjectADevAccount: "22222222222223"
      ProejctADemoAccount: "333333333333334"

Resources:

  SystemAdminPermissionSet:
    Type: AWS::SSO::PermissionSet
    Properties:
      InstanceArn: !Ref IamIdentityCenterInstanceArn
      Name: 'SystemAdmin'
      Description: 'System Admin Permissions'
      SessionDuration: 'PT8H'
      InlinePolicy:
        Version: '2012-10-17'
        Statement:
          - Effect: Allow
            Action: '*'
            Resource: '*'

  SystemAdminGroup:
    Type: AWS::IdentityStore::Group
    Properties:
      DisplayName: SystemAdmin
      Description: 'System Admin Group'
      IdentityStoreId: !Ref IdentityStoreId

  # Testing Account Assignments
  'Fn::ForEach::SystemAdminAssignments':
    - AccountId
    - - DevSandAccount
      - ProjectADevAccount
      - ProejctADemoAccount
    - 'SystemAdminAssignment${AccountId}':
        Type: 'AWS::SSO::Assignment'
        Properties:
          InstanceArn: !Ref IamIdentityCenterInstanceArn
          PermissionSetArn: !GetAtt SystemAdminPermissionSet.PermissionSetArn
          TargetId: !FindInMap
            - AccountIdsMap
            - !Ref AWS::AccountId
            - !Ref AccountId
          TargetType: AWS_ACCOUNT
          PrincipalId: !GetAtt SystemAdminGroup.GroupId
          PrincipalType: GROUP

Thanks for reporting this.  Built in some logic to clear each of the elements in a FindInMap when have issues resolving it.  Should have this released today.  I'm waiting on the new specs to be released today then I'll do a release.


[AutoRelease] t2-servicebus-2022-09-29-72778(Do not merge)

https://github.com/Azure/sdk-release-request/issues/2985
Generate tests successfully
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1031
BuildTargetingString
azure-mgmt-servicebus
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/2985


MatchConditions headers are improperly applied.

Library name
Azure.Data.Appconfiguration
Describe the bug
The MatchConditions headers should be applied by calling this method. As it currently is, when the headers are applied (e.g. here), the quotes are always omitted from the ETag value. However, when options.IfMatch.Value does not equal ETag.All, the ETag values must be stored with the quotes included. Without the quotes, ETag values cannot be compared.
The client is generated using the following swagger:
Input File :  -  https://github.com/Azure/azure-rest-api-specs/blob/e01d8afe9be7633ed36db014af16d47fec01f737/specification/appconfiguration/data-plane/Microsoft.AppConfiguration/stable/1.0/appconfiguration.json
Expected behavior
ETag values are stored with quotes included.

Actual behavior
ETag values are stored without quotes included.

This is an issue in Azure.Core. We should transfer to azure-sdk-for-net.


Is it possible to know if Chrome-tabs is available?

and if not, use a WebView instead?
Users might not have the latest version of Android/Chrome, and might not even have Chrome installed, so I'd like to use a Webview in all cases that Chrome-Tabs cannot work.
Is it possible?

Yes, it is possible.
Check the ServiceConnectionActivity.java for an example. It uses WebviewActivity.java as a fallback, which is an Activity with a WebView.

Nice. Say, if I call the warmup() and/or mayLaunchUrl() , would it also help if I do it on a previous activity, before starting the new one?
Also, why doesn't the SDK also include the WebView fallback itself, so that it will look about the same on all possible states ?

Is it also always better to call the warmup() and mayLaunchUrl()  functions?
Sometimes (or even usually) I don't see much of a difference between using them and not using them...

I've tried to set minSdkVersion to 14, but the "com.android.support:customtabs" library doesn't allow me to do this. How come? I can use the WebView in case it's not available, anyway...

we do not look at github issues frequently enough, so let's continue this discussion here:
https://code.google.com/p/chromium/issues/detail?id=543546
We are tracking all customtabs related bugs in chromium issue tracker at the moment. This includes examples in this repo, chrome implementation and related android support library issues.

Too bad. Github is an awesome place to share code and also talk about it.
Also, this post was more of questions than an issue to report about .


Training details

good job.
I have some questions about Training details.

In your paper, “We train 200 epochs using a batch size of 1024 on 16 GPUs and an initial learning rate of 0.12”, which type of GPUs are used in your paper? V100?
you compute the Spearman [36] correlation between the similarity ranking and their final performance ranking. Why not use Kendall’s Tau correlation?
Is there any data on search costs?

thanks

Yes, we trained supernet with V100.
We follow UnNAS, so we adapt the Spearman. And, Kendall's Tau is truly a more strict judgment.
I don't understand this question, can you explain it more ?


Building libsass/sass-worker won't work?

Sorry to bother man, I've downloaded this on my machine, updated the git and executed the npm run build script and it won't complete due to some warning:
Warning: Command failed: (cd libsass && /bin/bash ./prepare.sh "3.6.5")
The system cannot find the path specified.
 Use --force to continue.

Same result with npm run build --force. Is there anything I need to do in preparation to running this script?
Also after update I got notified about some 18 vulnerabilities.

Can you create a pr with your changes?

I didn't change a thing, I only:

downloaded the zip,
opened in VisualCode,
ran npm run update to install and update
ran npm run build

Do you have docker installed and tuning on the background?

Nop, is that a thing?

So docker is needed iirc for building emscripten from source

I just completed downloading, installing Docker.Desktop but it won't work because it requires some feature called hardware assisted virtualization, which my machine does not support.

@thednp probably you can modify this so it could run in a GitHub action. But honestly I don't see the reason, the versions of libsass and node sass are the latest ones thus the output won't be different (unless if you plan to play around with the emscripten options)

TBH I was thinking to have have a repository/folder somewhere I can hit npm update & npm run build to get the latest sass compiler.

Hopefully the dart-sass is close to have a native wasm, so all this would be definitely deprecated. (probably you can do that right now if you're willing to fiddle with their build tools: https://github.com/sass/dart-sass/issues/25)

Thank you again for your time, I'm now going to revamp the color picker to work with a regular <input> in a way to make it easy to add a <joomla-color-picker> CE with <slot> later on (I've learned how to do that meanwhile, and I didn't get you back then).
Will let you know on the progress.
Thanks again. Cheers!


Corrected Java and PHP code generation using Docker on Windows

🤔 What's changed?
I corrected the Java and PHP code generation when running using Docker on Windows.
⚡ What's your motivation?
Fixes #146
🏷 What kind of change is this?

:bank: Refactoring/debt/DX (improvement to code design, tooling, documentation etc. without changing behaviour)

♻ Anything particular you want feedback on?
It seems to be related to new lines differences between Unix and Windows.
📋 Checklist:

[x] I agree to respect and uphold the Cucumber Community Code of Conduct
[ ] I've changed the behaviour of the code

[ ] I have added/updated tests to cover my changes.

[ ] My change requires a change to the documentation.

[x] I have updated the documentation accordingly.

[ ] Users should know about my change

[x] I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

LGTM. Didn't test on a *Nix system.


A question for UploadSet.save()

When I use UploadSet instance save to file, It's raise UploadNotAllowed Exception. I find it strange，so, I read your code. finally, I found it, Because I'm Chinese, the filename maybe is Chinese. But, when the filename is Chinese, use the werkzeug.secure_filename(), the result is the extension. For example.
filename = 'boy.jpg'
res = secure_filename(filename)
res is boy.jpg, but when the filename is chinese
filename = '男孩.jpg'
res = secure_filename(filename)
res is jpg.

Please use StackOverflow for questions about how to use the library.


Allow targeting single methods in kotlin tests

Checklist

[ ] I have filed an issue about this change and discussed potential changes with the maintainers.
[ ] I have received the approval from the maintainers to make this change.
[ ] This is not a stylistic, refactoring, or cleanup change.

Please note that the maintainers will not be reviewing this change until all checkboxes are ticked. See
the Contributions section in the README for more
details.
Discussion thread for this change
Issue number: #2301
Description of this change
Fix the longstanding issue of not being able to target single tests in kotlin
--target_filter needs # on method reference

Thanks @perarnet! I checked https://github.com/bazelbuild/intellij/issues/2301 but I was not able to reproduce the problem by following the examples in the issue discussion, can you provide a simple project that shows the problem? You can use this example project as a base https://github.com/bazelbuild/intellij/tree/master/examples/kotlin/simple_project

Thanks @perarnet! I checked #2301 but I was not able to reproduce the problem by following the examples in the issue discussion, can you provide a simple project that shows the problem? You can use this example project as a base https://github.com/bazelbuild/intellij/tree/master/examples/kotlin/simple_project

It makes more sense now. By looking more into the test_filter documentation, it clearly states it's up to the test runner to interpret it. The issue I've been facing is when using junit5. So would probably need to adapt that to allow for . prefix to the method. Sorry for the noise. :)


Development: Dark mode in Apollon Standalone

Motivation and Context
This PR implements the Darkmode feature in Apollon Standalone
Description
The feature is as identical to the one we have in Artemis, where users can now have a button to switch between dark mode and light mode.
Steps for Testing
To test locally:

checkout the enhancement/dark-mode  branch
 yarn install && yarn start
To test in test server:
Goto https://test1.apollon.ase.in.tum.de

Todos:

[x] Implement toggle switch instead of radio button
[x] Fix Codacy Static Code Analysis Issue
[x] Fix styling of File Menu Entries

Looks good in general!
I think there are a few issues left:
Dark Mode popup closes even though it's hovered
The popup auto closes even if I hover it. If I'm not quick enough, I could be unable to click the switch
https://user-images.githubusercontent.com/23171488/168484747-7a5b1e78-4ee7-4f77-aef0-7113a7da343b.mov
Background intensive
The grid background is very intensive in my opinion, especially compared to light mode.
It's also less intensive in Artemis.
But that's just an opinion
Apollon Standalone:

Artemis:

Separator appears black
The separator in the right sidebar seems to be black in dark mode and is barely visible:

Alert is not themed
This Firefox alert seems to be not themed

[ ] Dark Mode popup closes even though it's hovered
[ ] Background intensive
[ ] Separator appears black
[ ] Alert is not themed


Total amount of G$ claimed is incorrect

Description
The total amount of claimed G$ is 0 when open the 'Claim Daily G$' page after a successful claim.
Pre condition
Register a new account
Steps to reproduce

Go to the 'Claim Daily G$' page
-> The statistic is '4 People Claimed 400 G$ Today!'

Successfully claim your 1 G$

Go to the 'Claim Daily G$' page again

Actual result: '5 People Claimed 0 G$ Today!'
https://screencast.com/t/E9rrETHwQ
Expected result: Total claimed G$ should be correct ( I'm not sure if it's correct on the first screen as if 4 people claimed 1 G$, than the total sum should be 4 G$ )
Supporting data
https://screencast.com/t/E9rrETHwQ
Alpha2 https://dapp.gooddollar.org
Chrome 75.0.3770.142

@sirpy
Checked on https://gooddev.netlify.com
iPhone X // iOS 12.4 Safari
The bug is reproduced. The "Total claimed G$" value isn't displayed
https://www.screencast.com/t/8kaSghWphvK0

@qatests1001 the bug is NOT reproduced, you are showing a different bug all together.
if you didnt do any of recreate the steps to reproduce the bug then its not reproduced.

@sirpy
Sorry, wrong link
https://www.screencast.com/t/b0WCHqWosfNO

@AnastasiiaOdnoshevna
There is a new Claim screen without Total amount of claimed G$ - so the initial issue is not actual I guess.
But I noticed on your video, that amount of people claimed today isn't changed, looks like it's a stub value.
I think we need a separate ticket on this.


fix maven error for spring-petclinic

There is a problem when running maven install for spring-petclinic:
'parent.relativePath' of POM org.springframework.samples:spring-petclinic:2.7.3 points at com.alibaba:druid-parent instead of org.springframework.boot:spring-boot-starter-parent, please verify your project structure @ line 8, column 11
It is highly recommended to fix these problems because they threaten the stability of your build.
For this reason, future Maven versions might no longer support building such malformed projects.
Just fix this error.

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


Add ctrie

ctrie needs to support a few updated dependencies before it can start building with the nightlies:
QuickCheck-2.8.1 depended on by:

ctrie-<IP_ADDRESS> (==2.5.*). Michael Schröder<EMAIL_ADDRESS>@mcschroeder. @mcschroeder

atomic-primops-<IP_ADDRESS> depended on by:

ctrie-<IP_ADDRESS> (==0.6.*). Michael Schröder<EMAIL_ADDRESS>@mcschroeder. @mcschroeder

base-<IP_ADDRESS> depended on by:

ctrie-<IP_ADDRESS> (==4.7.*). Michael Schröder<EMAIL_ADDRESS>@mcschroeder. @mcschroeder

cmark-0.5.0 depended on by:

ctrie-<IP_ADDRESS> (==1.0.*). Michael Schröder<EMAIL_ADDRESS>@mcschroeder. @mcschroeder

primitive-<IP_ADDRESS> depended on by:

ctrie-<IP_ADDRESS> (==0.5.*). Michael Schröder<EMAIL_ADDRESS>@mcschroeder. @mcschroeder

Looks like I was a little bit too hasty in submitting the pull request and stackage-curator picked up an older version. The current one, ctrie-<IP_ADDRESS>, has the fixed bounds.

Looks good, thanks!


Added more utilities for File/filename

Constructing filename from entity
Extract subject and session ID from filename/path

@c42f Yeah that sounds handy! I've just added that option.


Multiple Images

Is there a way to add multiple images or attachments to a particular record right now via one of the available features out of the box? If not, can you please describe how we can approach this? Additionally, can we resize these images via motor-admin out of the box yet or should we use something like image-processing which is suggested by the RoR guide for transforming images? Thank you!

@vkhang55 it's possible to add multiple images/attachments via Rails ActiveStorage:
https://edgeguides.rubyonrails.org/active_storage_overview.html#has-many-attached
Also ActiveStorage allows to transform images (image_processing gem should be added to the app Gemfile).
  has_many_attached :images do |attachable|
    attachable.variant :thumb, resize: "100x100"
  end

Thank you for the quick response @omohokcoj!

@omohokcoj I was attempting to handle multiple images via a custom form and had lots of trouble getting it to work properly. Would it be possible to add an example of a working form with multiple images to the demo app?

I was attempting to handle multiple images via a custom form and had lots of trouble getting it to work properly. Would it be possible to add an example of a working form with multiple images to the demo app?

@drewbaumann It has been implemented in the newest version of the gem:
https://rubygems.org/gems/motor-admin/versions/0.1.81
You can check how it works on in the demo app:
https://motor-admin.herokuapp.com/demo/forms/4
Also here is the custom form controller code to save the images (there is a small trick with string encoding to process binary data correctly):
https://github.com/omohokcoj/motor-admin/blob/master/spec/dummy/app/controllers/notes_controller.rb#L17
Also this multiple images form works best with the create action - adding/removing images should be easy to handle via the default associations CRUD:
https://motor-admin.herokuapp.com/demo/data/notes/1/images_attachments


docs: CLI and KYVE

did you run yarn docs to generate the README updates? Thats the easiest way to get a full updated TOC

:tada: This PR is included in version 1.11.0-alpha.3 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: support different architectures when copying images

Fixes #568
Currently we can only copy images for the amd64 architecture. copy supports ArchitectureChoice which is leveraged in this PR. If no Architecture is provided the current behavior applies (uses the platform architecture, which is x86).

Great PR @uncledru. Even I am looking for this feature.

It will be great if @wchaws can look into this

also tagging @rix0rrr

Hi, awesome and useful feature, please make it happen soon. 😄 Thanks.

Is there an update on this?