Nested query is not parsed correctly

The current SQL grammar fails to recognize nested query under FROM.
For example, the following query:
select
         w_warehouse_name
 	,w_warehouse_sq_ft
 	,w_city
 	,w_county
 	,w_state
 	,w_country
        ,ship_carriers
        ,year
 from (
    select
 	w_warehouse_name
 	,w_warehouse_sq_ft
 	,w_city
 	,w_county
 	,w_state
 	,w_country
 	,concat('DIAMOND', ',', 'AIRBORNE') as ship_carriers
        ,d_year as year
 	,sum(case when d_moy = 1
 		then ws_sales_price* ws_quantity else 0 end) as jan_sales
 	,sum(case when d_moy = 2
 		then ws_sales_price* ws_quantity else 0 end) as feb_sales
 	,sum(case when d_moy = 3
 		then ws_sales_price* ws_quantity else 0 end) as mar_sales
 	,sum(case when d_moy = 4
 		then ws_sales_price* ws_quantity else 0 end) as apr_sales
 	,sum(case when d_moy = 5
 		then ws_sales_price* ws_quantity else 0 end) as may_sales
 	,sum(case when d_moy = 6
 		then ws_sales_price* ws_quantity else 0 end) as jun_sales
 	,sum(case when d_moy = 7
 		then ws_sales_price* ws_quantity else 0 end) as jul_sales
 	,sum(case when d_moy = 8
 		then ws_sales_price* ws_quantity else 0 end) as aug_sales
 	,sum(case when d_moy = 9
 		then ws_sales_price* ws_quantity else 0 end) as sep_sales
 	,sum(case when d_moy = 10
 		then ws_sales_price* ws_quantity else 0 end) as oct_sales
 	,sum(case when d_moy = 11
 		then ws_sales_price* ws_quantity else 0 end) as nov_sales
 	,sum(case when d_moy = 12
 		then ws_sales_price* ws_quantity else 0 end) as dec_sales
 	,sum(case when d_moy = 1
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as jan_net
 	,sum(case when d_moy = 2
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as feb_net
 	,sum(case when d_moy = 3
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as mar_net
 	,sum(case when d_moy = 4
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as apr_net
 	,sum(case when d_moy = 5
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as may_net
 	,sum(case when d_moy = 6
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as jun_net
 	,sum(case when d_moy = 7
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as jul_net
 	,sum(case when d_moy = 8
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as aug_net
 	,sum(case when d_moy = 9
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as sep_net
 	,sum(case when d_moy = 10
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as oct_net
 	,sum(case when d_moy = 11
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as nov_net
 	,sum(case when d_moy = 12
 		then ws_net_paid_inc_tax * ws_quantity else 0 end) as dec_net
     from
          web_sales
         ,warehouse
         ,date_dim
         ,time_dim
 	  ,ship_mode
     where
            web_sales.ws_warehouse_sk =  warehouse.w_warehouse_sk
        and web_sales.ws_sold_date_sk = date_dim.d_date_sk
        and web_sales.ws_sold_time_sk = time_dim.t_time_sk
 	and web_sales.ws_ship_mode_sk = ship_mode.sm_ship_mode_sk
        and d_year = 2002
 	and t_time between 49530 and 49530+28800
 	and sm_carrier in ('DIAMOND','AIRBORNE')
     group by
        w_warehouse_name
 	,w_warehouse_sq_ft
 	,w_city
 	,w_county
 	,w_state
 	,w_country
       ,d_year
 ) x

The first "from ( select" gets ignored, which results in the the parse tree with one large list of select elements. We need to fix the current SQL grammar for ANTLR somehow to resolve this issue.

Pull request #125 fixes this issue. I will close this issue once the patch gets merged.


hv.DynamicMap does not return hv.HoloMap

Hi all!
I have a Graph object which its hv.Nodes are connected to hv.DynamicMap which returns a hv.HoloMap and HoloMap does not show up.
Ideal funcionality:
When selecting a Node from the hierarchy tree it should appear a HoloMap underneath it with its dropdown/ slider widget to switch from the set of plots (Overlay Objects) I have for that particular Node being Cliked.
The error:
Only renders the Graph Object, and shows a warning message such as
HoloMap can't not be displayed using Bokeh backend.
hv.DynamicMap works if returning an Overlay:
I have the hv.DynamicMap working if returning a single Overlay Object. "the first item in the dictionary basically". Each Overlay Object in the dictionary is something like this : area1area2lineline2line3.
What I am doing is :
df_tree =  {'A': [1,2,3,4] ,
'B': [5,6,7,8] }
overlay_dict = {'A': [{ 0: Overlay, 1: Overlay, 2: Overlay, 3: Overlay }] ,
'B': [{ 0: Overlay, 1: Overlay, 2: Overlay, 3: Overlay }] }

I pass the index of the Node inside hv.Dynamic map using Select1D .
I look up the name of the Node selected in df_tree e.g. 'A'.
I  filter the overlay dict by  column overlay_dict['A'].
I return hv.HoloMap(   overlay_dict['A']  )    <  Doesn't work

But when returning
return hv.HoloMap(   overlay_dict['A'][0]  )      < It does work
What am I doing wrong? and how could I get this working?
Any example would be very much appreciated!
Thanks for the help and for putting together this amazing library!
Elia

DynamicMaps will never be able to return HoloMaps. It's part of the core design of HoloViews. If you need to dynamically display HoloMaps I'd suggest you look at panel (https://panel.pyviz.org).

Hi Philip!
Thanks for your quick response!
Would it be possible to update the hv.DynamicMap with two streams?
The first one being the Selection1D from the Node’ Graph object, and the second one being a pointerXY which would capture the second key inside the dict so hv.Dynamic would return an Overlay instead of a hv.HoloMap({0:Overlay, 1:Overlay...}) ??
If not I’ll have a look at panel anyway.
Thank you so much for your help,
Very much appreciated!
Elia

Would it be possible to update the hv.DynamicMap with two streams?

Yes, DynamicMaps can have as many streams as you want.


Feature requests: Immediately render the linedash of ol.style.Stroke in WebGL mode

Hello guys,
The line-dash feature for ol.style.Stroke works very well in Canvas-rendering mode.
Cloud you please make the line-dash feature for ol.style.Stroke in WebGL mode available as well?

I have discovered that the ol.style.Stroke configuration parameters "strokeStyleLineCap", "strokeStyleLineDash", "strokeStyleLineDashOffset" and "strokeStyleLineJoin" are ignored in the JS file "ol-v4.3.1/ol/ol/render/webgl/linestringreplay.js".
Why?

These are not implemented for the WebGL renderer.

Will this option be implemented into the WebGL Renderer in the future?

In v6, there is a new WebGL renderer for points only.

Will it support lineDash on a LineString Stroke as well?


Simulator Naming Convention

Just to clean up the rest of the package with a consistent approach:
Following the guidelines, our current simulators should be named as:

SplishSplash
GROMACS
OpenFOAM
DualSPHysics
Fenicsx
SWASH
Xbeach
Simsopt

I think it looks good, perhaps with the exception of XBeach (instead of Xbeach), because X is some capital letter they use as a prefix and then Beach is a word. Again "luckily" in this case it also coincides with what they use officially.

If I followed the guidelines, I would've arrived at SIMSOPT instead of Simsopt, as it seems to be an acronym for Simons Stellarator Optimizer, at least according to their github.

This simulator names are stretching the python guidelines to the limits! LOL For simsopt they never use the capitalised version anywhere in their website, so they don't quite treat it as an acronym, but I agree that this is one seems yet another level of gray area. Let's keep it like this for now. I will send an email to the team regarding the process to make these decisions.

They seem to use it: https://joss.theoj.org/papers/10.21105/joss.03525

Oh, ok, good catch, so then it becomes more clear.


Add the ability to manually pass in source files

The pruddy-error project is currently using AJAX or fs to read files that are in the error stacktrace so it can pin-point the location of the errors. There are environments where we cannot use these methods, so we should supply a way to pass in source files manually.
assume.source('filename', 'var balbalb = content of the filename');

This is actually possible right now.


Not compatible with current InteractiveViewer

Thats because the whole purpose of interactive viewer 2 is for widgets that should render bigger than the viewport, so they cannot be constrained and have to set their own constraints (like a canvas for a painting app with specified size) (you could say constrained is false by default and boundaryMargin is infinty in all directions by default (if I remember it correctly).) In your case it should work by just letting these two options unset.
Let me know if it worked.
Merry Christmas


AttributeError: module 'numpy' has no attribute 'typeDict'

 1 # Create the LSTM model

----> 2 from keras.models import Sequential
3 from keras.layers import LSTM, Dense
4
5 model = Sequential()
~/opt/anaconda3/lib/python3.8/site-packages/keras/init.py in
18 keras.io.
19 """
---> 20 from keras import distribute
21 from keras import models
22 from keras.engine.input_layer import Input
~/opt/anaconda3/lib/python3.8/site-packages/keras/distribute/init.py in
16
17
---> 18 from keras.distribute import sidecar_evaluator
~/opt/anaconda3/lib/python3.8/site-packages/keras/distribute/sidecar_evaluator.py in
15 """Python module for evaluation loop."""
16
---> 17 import tensorflow.compat.v2 as tf
...
--> 320         raise AttributeError("module {!r} has no attribute "
321                              "{!r}".format(name, attr))
322
AttributeError: module 'numpy' has no attribute 'typeDict'

This is not a TuriCreate issue. This error is coming from Keras. It looks like a problem with your environment. The version of Keras you have installed is not compatible with the version of numpy you have installed.


Prettier: increase readability with lower the value of printWidth

Some lines in source code don't fit in my screen by far, could we change prettier's settings for printWidth from 250 to 120?
Prettier itself recommends 80 https://prettier.io/docs/en/options.html#print-width
I'm on 24 inches monitor fullscreen, and still, this line is killing me :) https://github.com/dotansimha/graphql-code-generator/blob/master/packages/plugins/typescript/operations/src/visitor.ts#L67

We use our own prettier configuration in the repo combined with code-gen's hooks support:
hooks:
  afterAllFileWrite:
    - npm run prettier:fix

This will format the files based on our npm prettier:fix script.  That will then follow whatever rules you have setup for your own project.

Thank you for the response!
But I am telling about  graphql-code-generator repo. This line to be precise: https://github.com/dotansimha/graphql-code-generator/blob/master/.prettierrc#L4
I want to contribute, but reading the code with the horizontal scroll is not so easy.

@standy - ah, sorry my mistake! That's what I get for skimming.

Updated in master to 120. Thanks @standy ! and thank you for your contributions ;)


Send output current mode when changed

Fixes #229

So should we not be sending the full list of modes in the first place?

We already do that in wl_output_send_to_resource.

I mean, is that wrong?

Oh, not at all: when a client first connects, we send all modes, and if the current mode changes, we only resend that mode. Only sending the current mode when a client connects would make the WL_OUTPUT_MODE_CURRENT flag useless. As the spec says:

The mode event describes an available mode for the output.

So if a mode is available, we should send it.

Aight


nakamoto can not estimate fee after restarts

Nakamoto has an ongoing PR https://github.com/cloudhead/nakamoto/pull/139 that is not able to keep persistent the fee estimated, maybe because I missed something in the nakamoto architecture
But, at the end of the day after the restart for now nakamoto needs to run a little bit to be able to estimate all the fee that core lightning need (up to 100 blocks), but I think the solution is that nakamoto need to be able to store the fee in some cache with the block information.

Tracking issue https://github.com/cloudhead/nakamoto/issues/141

Currently, we are using esplora, but we can use also the neutrino API, but for now, esplora works nicely because we have to fetch also the information about the utxo (but nakamoto can do it too we need just to support it)


Create 'Ask a question' template

Create ask-a-question.yaml in the .github/ISSUE_TEMPLATE folder.

I did it!!


[Bug]: Error: Station: command CMD_SDINFO_EX failed with error: undefined (146)

Adapter version
0.9.10
JS-Controller version
4.0.24
Node version
18.13.0
Operating System type
Linux
Operating system version
Ubuntu 22.10
Describe the bug
Every 10min i get the error "Station: xxx command CMD_SDINFO_EX failed with error: undefined (146)"
To reproduce

Go to '...'
Click on '...'
Scroll down to '....'
See error

Screenshots & Logfiles
No response
Additional context
No response

Same problem here. I don't have any SDs installed, maybe that's the problem.

Could you please provide which station you are using?

T8400 Solo Indoor Cam C24

The error is thrown here. I think, @bropat needs to have a look on it (if we need to change anything in the storageinfo-implementation, give me a sign).

Any update on this issue? It keeps flooding my log with 2 messages every minute......

@PhilippEngler
There are 2 different commands for getting the SD card / storage data.
I think that not all stations support the command CMD_SDINFO_EX. You would have to check if the other command CMD_SDINFO should be used here.
@harryslapalot
Which station reports this error? Model?

Hi, since the topic is closed, what is the solution? My log is full of theses error messages!


Python subclassing doesn't inherit from pyclass on py3.6 or 3.7

🐛 Bug Reports
In retworkx I'm trying to update to use the latest pyo3 version in https://github.com/Qiskit/retworkx/pull/224 ci jobs on python 3.6 and 3.7: https://github.com/Qiskit/retworkx/pull/224/checks?check_run_id=1650749331#step:6:16
Several of the tests use a python subclass of a rust class (it's just a backwards compat alias so there are no changes besides the name it's defined here: https://github.com/Qiskit/retworkx/blob/master/retworkx/init.py#L15-L83 ) and those tests are all failing on python 3.6 and 3.7 (both locally and in CI) with a TypeError on method calls (the python subclass type isn't valid for the rust method) or on the constructor (saying no constructor is defined for the subclas). The issue looks like the changes in subclassing for 0.13.0 don't work with python 3.6 or 3.7 as the 3.8 and 3.9 jobs work fine.
🌍 Environment

Your operating system and version: linux x86_64
Your python version: 3.6.12 and 3.7.9
How did you install python (e.g. apt or pyenv)? Did you use a virtualenv?: distro package with virtualenv
Your Rust version (rustc --version): 1.49.0
Your PyO3 version: 0.13.0
Have you tried using latest PyO3 master (replace version = "0.x.y" with git = "https://github.com/PyO3/pyo3")?: no

💥 Reproducing
I tried to make a minimum reproduce case with:
#[pyclass(module = "mymodule", subclass)]
pub struct MyClass {
    pub value: bool,
}

#[pymethods]
impl MyClass {
    #[new]
    fn new() -> Self {
        MyClass {
            value: false,
        }
    }

    fn get_value(&self) -> bool {
        self.value
    }
}

import mymodule
class MySubClass(mymodule.MyClass):
    pass

obj = MySubClass()
print(obj.get_value())

but this worked fine locally (both in lib.rs and importing the class definition from another rust module). So I'm not sure what I'm doing differently in retworkx that could be causing this. But the same exact code worked fine with 0.12.x and the upgrade notes only called out using the subclass argument on the pyclass macro which I already was using.

Thanks for reporting. As far as I can tell this is a bug in PyO3 0.13 which is caused by the change in https://bugs.python.org/issue35810 being applied to all Python versions, when it should only have been applied in Python 3.8 and up.
https://docs.python.org/3/whatsnew/3.8.html#changes-in-the-c-api
I'm preparing a PR now to fix this in PyO3. Worthy of a 0.13.1 patch release for sure! 😬

Thanks for reporting. As far as I can tell this is a bug in PyO3 0.13 which is caused by the change in https://bugs.python.org/issue35810 being applied to all Python versions, when it should only have been applied in Python 3.8 and up.
https://docs.python.org/3/whatsnew/3.8.html#changes-in-the-c-api
I'm preparing a PR now to fix this in PyO3. Worthy of a 0.13.1 patch release for sure! 😬

Good catch, thank you!
👍🏼  for releasing 0.13.1.

Good catch, thank you!
👍🏼  for releasing 0.13.1.


Is there a way to combine multiple PDF files in the same container

No.
For future refenerence: When opening an issue, you need to provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md

No.
For future reference: When opening an issue, you need to provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md


Move glow color off ScrollBehavior

Instead, it should be an inherited widget you use to configure overscroll indicator.

The widget would have to be based on a builder, I guess, (as opposed to a solid Color), so that it could read the current Theme from the context of the OverscrollIndicator... I'm not sure it's worth it unless we have a real problem.

I think I see an easy way to do this.


Right Half Pyramid Pattern (Using *)

Contributions are invited for the code of the given Right Half Pyramid Pattern (Using *)

Added the code
Issue closed


Amir/inheritance

separated data between decorated classes, and between decorated classes and instantiated classes

fixes #10


added telegram shortlink

I've added the handy link for your telegram account

ohh sorry, my linter added some extra new lines in another snippets


ERR wrong number of arguments for 'set' command

e := redisClient.Set("key", "test", 0).Err()
// e == nil

e := redisClient.Set("key", "test", time.Hour).Err()
// e.Error() == "ERR wrong number of arguments for 'set' command"

I can't find anything different between what I'm doing and the examples

I guess you need to upgrade your Redis Server.

That did seem to be the issue. Is there any way to support the older version? I can't change the redis instance I need to send data to.

You have to use pipeline of 2 commands with old Redis Server:
Set(key, value, 0)
Expire(key, time.Hour)

Theoretically it is possible to do that automatically in the client, but it does not look worth it...


Setting a name and changing my birthday wipes out my spell list when making an oracle

As the title says, all my spells disappear when making an oracle and changing my birthday, and sometimes my name. Start the game with no spells.

This also happens on making a magus, so its not just an oracle thing. It doesnt happen 100% of the time, but most of the time. Cant replicate it with the mod disabled.

Can you give me a more specific reproduction case, I am not able to reproduce.

I cant get it to consistently happen, it seems random. I just go through the normal process of making my character, but then once I change my name (it seems to be linked to name rather than birthday as I originally thought) it just removes the spell selection entirely. I tried to replicate it three times in a row choosing all the same options and it didnt happen, but then it heppened on the 4th time, so I was like 'well i'll do it again and record it this time' but then it didnt happen on the 5th or 6th tries, so idk what the issue is

Do you have toy box? Cause this happens with toy box, usually only when gestalting.


[FEATURE] landing page and job page should tell if a job has been saved or applied to

Follow up for #28, this PR disables the "Save Job" or "Quick Apply" buttons if the user previously saved or applied for the job.
To achieve this I've introduced a small BookmarksByJobId helper function in the bookmark repository, which is loaded onto every page we show "Save Job" or "Quick Apply" buttons (landing.html, job.html).
The diffs are getting quite messy any time we have to introduce a new repository that's used across many handlers. I'm separately considering introducing handler context to the project, as mentioned in the "Receivers" section in this Sharing Values With Go Handlers guide – but that's a bigger change, to be worked on separately.
Screenshots

Scenario
Screenshot

Not saved, not applied.

Unauthenticated
(same as above)

Saved, not applied

Applied

(Layout changed in that last screenshot due to the smaller buttons, and small job description. Job descriptions will be much longer on production)
Solves #122
/claim #122

thank you @rjackson - great work - will test on prod now


QuarterWidgetPlugin crash Qt Designer with Segmentation fault when build with Qt6

I detect this for Quarter plugin when updating package and trying switch to Qt6 from Qt5 on Windows.
Currently we still use Qt5.

Thanks for the report, I'll have a look into it.

Currently we only build 64-bit packages with gcc and clang compilers. And just for note: designer.exe (Qt5), but designer-qt6.exe (Qt6).

Just released version v1.2.1 that fixes the issue. Thanks for your ongoing contribution to Coin development. It is highly appreciated.

Thanks for quick fix! Newlest package in the repo. Binary packages pending.


cucco didn`t install requirements like yaml

I installed pyyaml by myself but then i have the error
module cucco has no attribute regex

Installing pyyaml helped me get through this:
pip3 install pyyaml


Use CodeMirror instance as receive pane for render buffering

What's this PR do?
Brings in a CodeMirror instance to hold the Chrome IDE Receive Pane. This should better handle the output text to improve performance on Chromebooks.
Where should the reviewer start?
npm run build
How should this be manually tested?
Load a file that will quickly loop data. Run it on a BASIC Stamp from a Chromebook. Check performance when line counts get large, > 1000.
What are the relevant tickets?
References #178

While testing this branch against a chromebook, it was discovered that the CodeMirror implementation did not improve performance in this case. Closing this PR for now, we will revisit with a lighter-weight implementation of this concept in the future.

Thanks.  I'm not surprised because I have experience with this before.  The receive pane system as a whole needs to be light-weight in order to process things swiftly regardless of the serial speed or amount of data being received.
As I did in the BSE software, I suggest that the receive pane be implemented as a lower-level entity where:

The data is received and processed into a non-visual buffer (a circular array of line; strings) which is ready for pure copy to a display buffer as needed,
The visible Receive Pane is actively handled to show only the visible window into the circular array line buffer (a relatively easy copy and paste operation),
The scroll bars, pane size, keyboard, and other user actions are actively monitored to adjust the view port noted in the previous bullet.

The most notable advantage of this is that there is never any time wasted by a rendering system that would otherwise be forced to process a large cumulative set of data for which it is only going to display a small slice (represented by the view port); ie: no browser rendering engine touches the entire dataset in the non-visual buffer.  String manipulation can happen swiftly and line recycling at the tail/head collision point is nearly instant as it doesn't affect the visible window until it happens to be withing the view port region.


Add test for nonspecific current

Since NEURON's ODE solver is not extremely accurate in the intermediate region where the jump is present, we only compare the final values (without the fix for the conductances, the result of v is a bunch of nans).
This requires updating the references again: https://github.com/BlueBrain/nmodl-references/pull/9

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

Project coverage is 86.55%. Comparing base (593f38e) to head (cde5083).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1235   +/-   ##
=======================================
  Coverage   86.55%   86.55%
=======================================
  Files         176      176
  Lines       13071    13071
=======================================
  Hits        11313    11313
  Misses       1758     1758

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

I would have preferred to not replace the test, but to have both: the existing one to check accuracy of v(t) and the new one to check stability. To have both it might be better to make the constant 0.005 a parameter that we can set from the Python script. Then one can use the same MOD file for both cases.

To have both it might be better to make the constant 0.005 a parameter that we can set from the Python script.

Since the current workflow is nrnivmodl -> python simulate.py, I don't see a way to set the parameters in the mod file in the Python script itself (that is, unless we add another script beforehand which modifies the AST of the mod file).

Added new test (simulate2.py with mod file leonhard2).

You can have global or range variables in the MOD file:

UNITS {
        (mA) = (milliamp)
}

NEURON {
        SUFFIX leonhard2
        NONSPECIFIC_CURRENT il
        RANGE c
}

ASSIGNED {
        il (mA/cm2)
}

BREAKPOINT {
        il = c * (v - 1.5)
}

(or similar). And then set from Python:
s.leonhard_c = 0.05
# s.c_leonhard = 0.05

(or some variant thereof.)

You can have global or range variables in the MOD file:
...

Huh, that's neat, updated!

It's now a bit hard to understand how the two tests are very different. One is about checking the accuracy of v(t). The other is about checking the stability of the numerical method, e.g. that the method is fully implicit.
We check the one my using small time-steps and checking all the value along the way. While, to me, checking stability is about taking a (stupidly) large timestep and checking it doesn't explode. I understand that we can check both by simply cranking c to be a higher value, under the assumption that dt doesn't automatically shrink. However, I can't judge if 0.5 is large enough compared to the sane value 0.005.
The worry is that future maintenance would remove the second compare(0.5) because it's "redundant".


further fix for absolute paths

we removed the explicit rejection, but still expanded the path in a way
that's unsuitable for paths that are already absolute.

Pull Request Test Coverage Report for Build 489

0 of 13 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 35.375%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/core/object_store.go
0
13
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/core/object_store.go
2
0.0%

Totals

Change from base Build 487:
-0.2%

Covered Lines:
722

Relevant Lines:
2041

💛  - Coveralls


Implement remaining cards from The Red Wedding

Dacey Mormont won't work currently as a result of strength X cards now being defined as 'null' instead of '0' in the JSON data (Beric Dondarrion has the same issue) needs #1115.

Improved Fortifications needs #1108 to function.

Let me know if you want me to convert the trigger of Breaker of Chains to the upcoming onChallengeInitiated event.

I'm going to go ahead and merge despite needing #1108 and #1115. This PR doesn't introduce any new bugs, just reveals existing ones fixed by those PRs. Contrast with the PR changing cannot-stand-during-stand-phase which would break those effects entirely if merged without the corresponding fix to effect recalculation.


[WIP] Allow full expression in interpolation. Fix Microsoft/vscode#44331

This breaks a test here
testCompletionFor('@for $i from 1 through 3 { .item-#{|$i} { width: 2em * $i; } }', {
	items: [
		{ label: '$i' }
	]
});

However I'm wondering why we have test for this
@for $i from 1 through 3 {
  .item-#{|$i} { width: 2em * $i; }
}

but not these, which are far more common completion than the first one
@for $i from 1 through 3 {
  .item-#{|} { width: 2em * $i; }
}

@for $i from 1 through 3 {
  .item-#{$|} { width: 2em * $i; }
}

Change looks good. But it would be interesting to figure out why the variable is no longer suggested.
Likely some other suggestions are now showing preventing the interpolation suggestions to be evaluated.


Eliminate implementation mutation

Mutating the implementation.files hash has caused problems with unrelated solution files being erroneously served to students. See: https://github.com/exercism/x-api/pull/153#issuecomment-303578409
The initial fix for this was to fix trackler to duplicate the files hash when duplicating the implementation: https://github.com/exercism/trackler/pull/51
But a better solution would be to eliminate the mutation of implementation completely.
This can be achieved by using a different view when restoring that separates the the files to be restored from the implementation.
Once this is merged it would be desirable to remove the Trakler::Implementation#merge_files method, so as to discourage mutating Implementation in the future.
I believe this is the only place where it was used.
The un-squashed refactoring commits can be seen at: https://github.com/Insti/x-api/pull/2

@kytrinyx you probably want to urgently look into this, since people are being served completed solution files with exercism fetch.


Use integer keys

This PR
Makes Table.Key always an integer type, removing the need for the compare_keys function.
Rationale:
Removes unnecessary extra comparisons performed by std.math.order when we need to check only one condition, e.g. a > b instead of std.math.order(a, b) == .gt.
Future benefits may be explored when dealing with integer keys.
Results:

Binary size decreased from 11757K to 10934K. Removing the inline function resulted in a 823K smaller binary.

Slightly better performance for both sequential and random IDs:

BeforeAfter

Sequential IDs

Benchmarking...
info: Benchmark running against { <IP_ADDRESS>:44605 }
info(message_bus): connected to replica 0
1226 batches in 59.76 s
load offered = 1000000 tx/s
load accepted = 167324 tx/s
batch latency p00 = 0 ms
batch latency p10 = 13 ms
batch latency p20 = 14 ms
batch latency p30 = 19 ms
batch latency p40 = 19 ms
batch latency p50 = 20 ms
batch latency p60 = 23 ms
batch latency p70 = 23 ms
batch latency p80 = 24 ms
batch latency p90 = 25 ms
batch latency p100 = 2494 ms
transfer latency p00 = 0 ms
transfer latency p10 = 3242 ms
transfer latency p20 = 6370 ms
transfer latency p30 = 10733 ms
transfer latency p40 = 15543 ms
transfer latency p50 = 20723 ms
transfer latency p60 = 26259 ms
transfer latency p70 = 31955 ms
transfer latency p80 = 38095 ms
transfer latency p90 = 44121 ms
transfer latency p100 = 49773 ms

Benchmarking...
info: Benchmark running against { <IP_ADDRESS>:47107 }
info(message_bus): connected to replica 0
1226 batches in 58.06 s
load offered = 1000000 tx/s
load accepted = 172247 tx/s
batch latency p00 = 0 ms
batch latency p10 = 12 ms
batch latency p20 = 13 ms
batch latency p30 = 15 ms
batch latency p40 = 19 ms
batch latency p50 = 20 ms
batch latency p60 = 22 ms
batch latency p70 = 23 ms
batch latency p80 = 24 ms
batch latency p90 = 25 ms
batch latency p100 = 2684 ms
transfer latency p00 = 0 ms
transfer latency p10 = 2675 ms
transfer latency p20 = 5603 ms
transfer latency p30 = 9744 ms
transfer latency p40 = 14511 ms
transfer latency p50 = 19412 ms
transfer latency p60 = 25255 ms
transfer latency p70 = 30772 ms
transfer latency p80 = 36665 ms
transfer latency p90 = 42643 ms
transfer latency p100 = 48067 ms

Random IDs

Benchmarking...
info: Benchmark running against { <IP_ADDRESS>:36915 }
info(message_bus): connected to replica 0
1226 batches in 68.28 s
load offered = 1000000 tx/s
load accepted = 146456 tx/s
batch latency p00 = 0 ms
batch latency p10 = 18 ms
batch latency p20 = 19 ms
batch latency p30 = 22 ms
batch latency p40 = 26 ms
batch latency p50 = 28 ms
batch latency p60 = 30 ms
batch latency p70 = 31 ms
batch latency p80 = 32 ms
batch latency p90 = 34 ms
batch latency p100 = 2824 ms
transfer latency p00 = 0 ms
transfer latency p10 = 3396 ms
transfer latency p20 = 6709 ms
transfer latency p30 = 12640 ms
transfer latency p40 = 18089 ms
transfer latency p50 = 24672 ms
transfer latency p60 = 30580 ms
transfer latency p70 = 37002 ms
transfer latency p80 = 44130 ms
transfer latency p90 = 51227 ms
transfer latency p100 = 58288 ms

Benchmarking...
info: Benchmark running against { <IP_ADDRESS>:45447 }
info(message_bus): connected to replica 0
1226 batches in 66.87 s
load offered = 1000000 tx/s
load accepted = 149544 tx/s
batch latency p00 = 0 ms
batch latency p10 = 17 ms
batch latency p20 = 19 ms
batch latency p30 = 23 ms
batch latency p40 = 26 ms
batch latency p50 = 28 ms
batch latency p60 = 29 ms
batch latency p70 = 31 ms
batch latency p80 = 32 ms
batch latency p90 = 34 ms
batch latency p100 = 2506 ms
transfer latency p00 = 0 ms
transfer latency p10 = 3035 ms
transfer latency p20 = 6561 ms
transfer latency p30 = 11519 ms
transfer latency p40 = 16549 ms
transfer latency p50 = 23144 ms
transfer latency p60 = 29478 ms
transfer latency p70 = 35383 ms
transfer latency p80 = 42422 ms
transfer latency p90 = 49794 ms
transfer latency p100 = 56878 ms

Sorry @matklad, just added some tests for the CompositeKey logic.


Release 0.4.0 with support for new Kubernetes CRI features

Expected Behavior
https://kubernetes.io/blog/2024/08/21/cri-cgroup-driver-lookup-now-beta/
Actual Behavior
Currently there is a mix between Kubernetes API v0.27.8 and CRI API v0.25.8/v0.27.8
Steps to Reproduce the Problem

Specifications

Version: 0.3.14
Platform:
Subsystem:

https://github.com/Mirantis/cri-dockerd/issues/222
https://github.com/Mirantis/cri-dockerd/issues/333
https://github.com/Mirantis/cri-dockerd/pull/350
https://github.com/Mirantis/cri-dockerd/pull/187

Any new minor release would get v27.0.2 (to be upgraded to v27.0.3), since that is what is on master
For the legacy branch, it would be possible to downgrade from v24.0.9 to v23.0.15 (EOL 2025-05-19)?
Since that is the version that Mirantis supports (for MCR 23.0.1 that, not Docker):
https://github.com/moby/moby/blob/master/project/BRANCHES-AND-TAGS.md

This one is slightly out-of-date as well: https://github.com/Mirantis/cri-dockerd/milestones

As docker engine v24.x is EOL, would it be possible to get a new release that uses v27.x as a library?

It has now been updated to v25.x for the 0.3 branch


Bug 1947798: Use CRD v1 for images.config and ICSPs

CRD v1beta1 is going to be removed, so we need to migrate to v1.

/assign @sttts


Cache for Svg Image

Cache didn't work SVG image. Using a network SVG image but get Load every time from the network rather than from cache.
child: UniversalImage(
                                                          bookData[
                                                                  'SvgCover'] ,
                                                          fit: BoxFit.fill,

                                                          enableMemoryCache:
                                                              true,
                                                          cache: true,
                                                          placeholder: SizedBox(
                                                            width: 50,
                                                            height: 50,
                                                            child: Center(
                                                              child:
                                                                  CircularProgressIndicator(),
                                                            ),
                                                          ),
                                                        ),

Sorry for late, please update universal_image to 0.1.1.
I just add cacheColorFilter property.
flutter_svg just updates the color filter cache recently, and this library depends on it.
You can check it here
From the doc:
Whether to cache the picture with the [colorFilter] applied or not.
This value should be set to true if the same SVG will be rendered with
multiple colors, but false if it will always (or almost always) be
rendered with the same [colorFilter].
If [Svg.cacheColorFilterOverride] is not null, it will override this value
for all widgets, regardless of what is specified for an individual widget.
This defaults to false and must not be null.

Thanks for getting back at this. But the issue wasn't about SVG colors. In my case, the SVG will always render with the same color.

I am using the latest version of flutter_universal_image

My issue is that SVG isn't getting cached. So if I open the app with the internet SVG will load but after an initial load if I open the app again without internet the svgs don't load implying svgs are not saved in cache but are getting fetched from the network every time.
In the case of png/jpg images did get load without internet through cache, it's just the SVGs that are not working.
I request you to look for that, all I need is to save SVG images of my app in Cache.
Best Wishes.

hi @rdcoder33 I got your point, but this feature isn't supported by flutter_svg, and flutter_universal_image is just a wrapper of flutter_svg, it can only do what flutter_svg can do, I have no idea to support you that at the moment.
If you need svg cache feature, please fire a request on flutter_svg, or you can find a way to that, don't hesitate to create a pull request for them & add a comment here, I think it's something on picture_provider
Or you can try another lib for svg https://pub.dev/packages/svg_cached_network_image
Thanks!


Smoke test for Docfx v3

Docs Build status updates of commit cb42682:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/SmokeTest.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Bug Report] ServerDescription is blank

Have you read the Important information text above

[X] Yes i did

Current behavior
The server description set in default.env does not work.
SERVER_DESCRIPTION=Exp,WorkSpd,DropItem,PalCap,EggHatch Modified
Desired behavior
The server description must be recorded in the ini file.
However, the description is empty.
ServerDescription=""

Links to screenshots
cat default.env

cat game/Pal/Saved/Config/LinuxServer/PalWorldSettings.ini

To Reproduce
Steps to reproduce the behavior:

download https://raw.githubusercontent.com/jammsen/docker-palworld-dedicated-server/develop/default.env
edit env file
docker-compose up -d
docker logs -f [docker name] or cat game/Pal/Saved/Config/LinuxServer/PalWorldSettings.ini
blank ServerDescription

Software setup

OS: ubuntu 22.04 LTS
Docker: Docker version 24.0.5, build 24.0.5-0ubuntu1~22.04.1 | docker-compose version 1.29.2, build unknown

Hardware setup

vCPU: 4
RAM: 24GB
Disk: 15GB

Additional context
There seem to be a few things wrong.
PLAYER_STOMACH_DECREASE_RATE (PLAYER_STOMACH_DECREACE_RATE)
BASE_CAMP_WORKER_MAXNUM (BASE_CAMP_WORKER_MAX_NUM maybe?)
docker logs

I don't know why, but I tried it again and it worked.
It seems that the versions of the env file and the Docker server were different.
When I received a new env file, there was webhook content, but it was not in the env file I used. I deleted the env file, downloaded it again, and set it up, and it worked fine.

Yeah we pushed a few updates, my just be that you had old cache of docker image.
Make sure to remove the image before and pull the newest version and check for changes in the docker-compose.yml and default.env file.
Changelog is coming, that might make things easier.

Im closing this issue as resolved, feel free to reach out again if you need help.
If you like this project, please consider giving this repo and the docker-hub-repo a Star.

Im closing this issue as resolved, feel free to reach out again if you need help.
If you like this project, please consider giving this repo and the docker-hub-repo a Star.

I already clicked on the star. 😀
I have one question. Sorry for asking a question again on a closed issue.
When backing up, it is broadcasted and output twice.

Are my settings wrong?

@arithxp  Thats an already known and fixed bug,  deployed to develop, will be later coming to master/prod/latest.


Add Record to from Typescript guide in Sorbet

This adds a row to the docs which explains that Record<string, Value>  is T::Hash[String, Value] in Sorbet.
r? @jez-stripe

Closing in favor of something a little more explanatory
https://github.com/sorbet/sorbet/pull/8087


Can't break Glowstone in survival

Blocks like Glowstone don't have a specific tool, so you can't use the wand to break these in survival, but other blocks like that ex. Glass break just fine.

you can add any block not tool specific to wands.json so it can be broken with pickaxe or other vanilla tools, you can also deny a spcific block in str_denied
"str_pickaxe_allowed": [
"minecraft:sea_lantern",
"minecraft:glowstone"
],
str_axe_allowed": [],
"str_shovel_allowed": [],
"str_hoe_allowed": [
"minecraft:moss_block"
],
"str_shears_allowed": [
"minecraft:moss_block"
],
"str_denied": []


Host Keptn Release Docker Images on multiple container registries/repositories

As described in https://github.com/keptn/keptn/issues/2294, DockerHub has introduced a rate-limit for pulling Docker Images.
This is especially problematic for end-users, when they try to install Keptn and use other Docker Images at the same time (e.g., when setting up Keptn + hipster shop).
To overcome this issue, we should host Keptn Docker Images on multiple registries and/or repositories, e.g., Google Cloud, Red Hat Container Registry, Quay.io, etc...
Definition of Done

[ ] CI Job that pushes Images to multiple registries for releases only
[ ] Existing Images for 0.8.x releases are available on multiple
[ ] Add in the README.md the list of registries where the images are pushed

I just ran into this problem locally when trying to install it on K3s (behind a corporate firewall).
$ kubectl get pods -n keptn
NAME                                     READY   STATUS             RESTARTS   AGE
secret-service-7dcd64b7d7-zvcw9          1/1     Running            0          4m9s
api-gateway-nginx-57f58849-72vgw         0/1     ImagePullBackOff   0          4m9s
configuration-service-57f6f5d84d-xw2bd   0/1     ImagePullBackOff   0          4m9s
mongodb-5865888d4f-pvgc2                 1/1     Running            0          4m8s
api-service-57665578b9-nrscx             0/2     ImagePullBackOff   0          4m9s
approval-service-76485b57bc-8k9pd        1/2     ImagePullBackOff   0          4m9s
keptn-nats-cluster-0                     0/3     ImagePullBackOff   0          4m9s
shipyard-controller-5fbcfcdfd4-zkqpp     0/2     ImagePullBackOff   0          4m9s
remediation-service-57d675bd46-sf558     0/2     ImagePullBackOff   0          4m9s
lighthouse-service-586b4cf57-kc6vv       0/2     ErrImagePull       0          4m9s
mongodb-datastore-67f9bd5cb6-kzvcf       0/2     ImagePullBackOff   0          4m9s
bridge-687f477c4d-nchf5                  0/1     ErrImagePull       0          4m9s
statistics-service-f4ccb7497-ss6ks       0/2     ErrImagePull       0          4m9s

To troubleshoot it, I did the following:
$ kubectl -n keptn get events | grep "429 Too Many Requests"

4m4s        Warning   Failed                  pod/shipyard-controller-5fbcfcdfd4-zkqpp      Failed to pull image "docker.io/keptn/shipyard-controller:0.8.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/keptn/shipyard-controller:0.8.1": failed to copy: httpReaderSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/keptn/shipyard-controller/manifests/sha256:e03f2072a5410da3266564e2695fc8e4d6e78bdb3f2090c66104a23da36ef79f: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
4m          Warning   Failed                  pod/lighthouse-service-586b4cf57-kc6vv        Failed to pull image "docker.io/keptn/lighthouse-service:0.8.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/keptn/lighthouse-service:0.8.1": failed to copy: httpReaderSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/keptn/lighthouse-service/manifests/sha256:6e178b04231d04fbbcf7ed01c8f453c866da747be14125757780ef65dc87a07a: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
2m49s       Warning   Failed                  pod/api-service-57665578b9-nrscx              Failed to pull image "docker.io/keptn/api:0.8.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/keptn/api:0.8.1": failed to copy: httpReaderSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/keptn/api/manifests/sha256:dc8ba98bbba5b6ff0e72834672c21db342264333c4fd125ea08b9486ebceddbc: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
2m39s       Warning   Failed                  pod/approval-service-76485b57bc-8k9pd         Failed to pull image "docker.io/keptn/distributor:0.8.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/keptn/distributor:0.8.1": failed to copy: httpReaderSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/keptn/distributor/manifests/sha256:b9263d5400d774ef0240acc5b883ae373ed5db453a13ba9bbef84f52057e1970: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
2m32s       Warning   Failed                  pod/api-service-57665578b9-nrscx              Failed to pull image "docker.io/keptn/distributor:0.8.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/keptn/distributor:0.8.1": failed to copy: httpReaderSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/keptn/distributor/manifests/sha256:b9263d5400d774ef0240acc5b883ae373ed5db453a13ba9bbef84f52057e1970: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

Please also research how other opensource projects provide their images. If all use DockerHub, I would propose to stay with DockerHub.

Images will be hosted additionally on GitHub and on Quay.io


use of %*% with two vectors has a strange behavior with unhelpful warning message

The manual says that doing matrix multiplication with two vectors should work but creates a 1x1 matrix. One would think one can therefore do this:
code <- nimbleCode({
y[1,1] <- A[1:3]%*%x[1:3]
})
m = nimbleModel(code)
cm = compileNimble(m)

But it doesn't work and the warning is hard to parse:
> cm = compileNimble(m)
Compiling
  [Note] This may take a minute.
  [Note] Use 'showCompilerOutput = TRUE' to see C++ compilation details.
Warning, in eigenizing model_y[1, 1] the [ is still there but nDim is not 0 (not a scalar).
Error: Failed to create the shared library. Run 'printErrors()' to see the compilation errors.

One would think that if nDim is not 0, then having [ is what one would want.

Actually, I think we basically cover this in "Understanding dimensions and sizes from linear algebra" in the manual. One needs to do:
y <- (A[1:3]%*%x[1:3])[1,1]

One reason it may be confusing is that this works fine, and I'm not sure we ever explain why/how it works in the manual.
y[1:3] <- A[1:3,1:3]%*%x[1:3]

without additional subscripting.
This also works, and is more consistent with our discussion in the manual (and is given in the linear predictor example on the website).
y[1:3] <- (A[1:3,1:3]%*%x[1:3])[1:3,1]


Potentially smarter CIRE via cire-rotate option

By default disabled as AFAICT it needs (block shape) tuning to deliver a performance improvement
Also needs more experimentation by more people
Early local results are encouraging though
The PR, in general, does improve the code quality in various places. Tests shipped.

@georgebisbas

Are there any particular examples we can try on several platforms?

any code exploiting CIRE would work, so TTI for example. Just add cire-rotate: True to the Operator and it will do the trick. Instructions in the updated performance notebook. Thanks for looking into this

@georgebisbas

Are there any particular examples we can try on several platforms?

any code exploiting CIRE would work, so TTI for example. Just add cire-rotate: True to the Operator and it will do the trick. Instructions in the updated performance notebook. Thanks for looking into this

@georgebisbas

Are there any particular examples we can try on several platforms?

any code exploiting CIRE would work, so TTI for example. Just add cire-rotate: True to the Operator and it will do the trick. Instructions in the updated performance notebook. Thanks for looking into this

As a general comment. it would be good to check that all these opt mode do not break any of the test/examples. Could for example pass a random combination of it to any of the tests (like adjoint/....) so that we know still pass with it.

@mloubout

As a general comment. it would be good to check that all these opt mode do not break any of the test/examples. Could for example pass a random combination of it to any of the tests (like adjoint/....) so that we know still pass with it.

This is a good point. I tried my best to test virtually all possible combinations. There are 800 line changes in test_dse.py; I've basically parametrized all of the CIRE tests to experiment with and without cire-rotate. We could add an option to also test TTI via the examples, but I can't pass a performance option (cire-rotate) from the command line yet, so... TestAliases, today, covers all of the critical cases (ie it also tests operators akin to what we have in our own TTI and TTi adjoint)


Does this package shows JavaScript stack trace in HockeyApp?

Hello,
Thanks for the package.
I'm building my iOS app with React Native and I already integrated HockeyApp SDK and simulated a crash and it appears to work.
The problem is that I can't see any relevant data to the crash (JS Error, stack trace etc) I can only see the Objective-C information.
Does this package sends JS stack trace and other data?

No it does not. Would you like to try to implement this?

@martincik I'm sorry but I'm not able to implement it since we're busy releasing our app.
We switched to Sentry which already supports tracking JS errors :)

@ranyefet I really need the JS stack traces too. I thought this project might have done it.
Did you do this:(https://docs.getsentry.com/hosted/clients/javascript/integrations/react-native/)?  It seems extremely cumbersome since you have to upload a new source map every time.  Are you doing anything better than that or just doing that each time you deploy a new app version?

@pickhardt well yeah it's a little cumbersome but It make sense, else how would they know to show a stack trace from the minified JS file?
I wrote a little script that create a release, push it to client using CodePush and Upload bundle and source map to Sentry. Works pretty well.

Guess I'll just write a similar script too.


CI Test for WebDataset

Stack from ghstack:

-> #378
#377

Only created to run CI, please ignore

Closed after testing

Closed after testing


Child classes without constructor are ignored

If I have the following files:
// src/A.js

/**
 * A beautiful class
 */
export default class A {

  /**
   * Beautifully initialises everything.
   */
  constructor() {}
}

// src/B.js
import A from './A.js';

/**
 * Amazing class.
 * @extends A
 */
export default class B extends A {

  /**
   * Determines how amazing this class is.
   */
  someMethod() {}

}

And then run jsdoc src -r -d docs, only A will be in the generated documentaion. If I add a constructor to class B and then run the above command again, B will already be in the generated documentation. Is this a know issue or am I doing anything wrong? Thank you!

There have been a lot of changes to how JSDoc handles ES2015 classes and modules. The following examples work okay for me on GitHub master (which should closely match JSDoc 3.5.1):
// src/A.js

/** @module A */

/**
 * A beautiful class
 */
export default class A {

  /**
   * Beautifully initialises everything.
   */
  constructor() {}
}

// src/B.js

/** @module B */

import A from './A.js';

/**
 * Amazing class.
 * @extends A
 */
export default class B extends A {

  /**
   * Determines how amazing this class is.
   */
  someMethod() {}

}


Use Constants in ChildReferences

Changes

Prior to this commit, the APIVersion and Kind fields used different variables depending on where they were used. In this cleanup, they use the same constants everywhere.
/kind cleanup
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[n/a] Docs included if any changes are user facing
[n/a] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in (if there are no user facing changes, use release note "NONE")

Release Notes

NONE

cc @abayer @dibyom please take a look


HTML entities ( # <'> & ") in MAP display are not decoded.

Hello, with 4.3 there is a problem with Html Entities not decoded in map display.
this happens with Device names and with geo areas (and probably with all other things that appear as layer on display)
I applied already the fixes found on
https://github.com/traccar/traccar-web/commit/04a92f0fbae8cee0e88b56e736a6c4178d39ca35
I attach some screen shots

What other things are you talking about?

Any other text that user inputs and that can appear on the map
For now i only verified device names and geofences names, but i do not know
if drivers appear as i never used them.
If it's only devices and geofences, then better, less fixes to do ;-)
On Thu, Apr 25, 2019 at 12:07 AM Anton Tananaev<EMAIL_ADDRESS>wrote:

What other things are you talking about?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/traccar/traccar-web/issues/737#issuecomment-486445898,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL3TFMJOQ3E23QXR7EOBI3TPSDKYRANCNFSM4HIIB77A
.


Implement wall time measurement for arbitrary function

Implement a function that accepts any function and measures wall execution time

implemented in https://github.com/embedded-dev-research/itlab_2023/pull/9


Remove controllers from ThirdParty Kernels

Codecov Report

Merging #10884 (584d36a) into main (f5db0f3) will increase coverage by 0%.
The diff coverage is 80%.

@@          Coverage Diff           @@
##            main   #10884   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        486      489    +3
  Lines      33784    33872   +88
  Branches    5502     5509    +7
======================================
+ Hits       21354    21435   +81
- Misses     10387    10397   +10
+ Partials    2043     2040    -3

Impacted Files
Coverage Δ

src/interactive-window/interactiveWindow.ts
74% <ø> (ø)

src/kernels/errors/kernelErrorHandler.ts
61% <0%> (-1%)
:arrow_down:

src/kernels/types.ts
100% <ø> (ø)

.../notebooks/controllers/vscodeNotebookController.ts
80% <ø> (ø)

src/notebooks/debugger/debuggingManagerBase.ts
74% <ø> (+5%)
:arrow_up:

src/notebooks/notebookCommandListener.ts
52% <ø> (ø)

src/notebooks/controllers/kernelConnector.ts
80% <40%> (-3%)
:arrow_down:

src/standalone/api/kernelApi.ts
68% <66%> (-5%)
:arrow_down:

src/kernels/execution/kernelExecution.ts
69% <74%> (-1%)
:arrow_down:

src/kernels/kernel.ts
81% <91%> (-1%)
:arrow_down:

... and 42 more


Tibialyzer not working with the last update

The curved bars and also the summary@ notification is not loading and not updating properly.

Update your memory addresses, that should fix the problem.

How you do that?

Click the Download New Addresses button on the main tab.

q,nkhk


workflows, linters, housekeep

Summary of changes

updated pre-release workflow
moved pylint config, reduced disabled list, and moved disable flags to specific areas where applicable instead of over entire files
add typing for linalg + some tests
fixed grammar and semantics in readme

Codecov Report
Attention: Patch coverage is 92.53731% with 5 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

pyqasm/entrypoint.py
83.33%
5 Missing :warning:

:loudspeaker: Thoughts on this report? Let us know!


Sentry could not resolve the “abort” exception,I have uploaded the Debug Symbols Successful (flutter packages pub run sentry_dart_plugin)

Platform
Flutter Mobile
Obfuscation
Enabled
Debug Info
Enabled
Doctor
Flutter (Channel stable, 3.3.1, on macOS 12.6 21G115 darwin-arm (Rosetta),
Version
7.7.0
Steps to Reproduce
Run flutter build apk before executing the sentry_dart_plugin plugin.
refer to: https://docs.sentry.io/platforms/flutter/upload-debug/
Expected Result
flutter Correct stack
Actual Result
libc++
+0x04a11c

libc++
+0x04a25c

libc++
+0x04afa0

libc++
+0x04af24
std::terminate
libc++
+0x0a4770
std::__1::thread::~thread
libc
+0x0f9124
__cxa_finalize
libc
+0x0eb4b8
exit
libflutter
+0x63174c

libflutter
+0x633348

libflutter
+0x6eff00

libflutter
+0x6efc94

libapp
+0x317084
_kDartVmSnapshotInstructions
libapp
+0x5a7860
_kDartIsolateSnapshotInstructions
libapp
+0x5a780c
_kDartIsolateSnapshotInstructions
libapp
+0x5a75e4
_kDartIsolateSnapshotInstructions
libapp
+0x868da8
_kDartIsolateSnapshotInstructions
libapp
+0x6b1ebc
_kDartIsolateSnapshotInstructions
libapp
+0x63cf30
_kDartIsolateSnapshotInstructions
libapp
+0x7e53f4
_kDartIsolateSnapshotInstructions
libapp
+0x749254
_kDartIsolateSnapshotInstructions
libapp
+0x7a174c
_kDartIsolateSnapshotInstructions
libapp
+0x447398
_kDartIsolateSnapshotInstructions
libapp
+0x447124
_kDartIsolateSnapshotInstructions
libapp
+0x360bf0
_kDartIsolateSnapshotInstructions
libapp
+0x3607cc
_kDartIsolateSnapshotInstructions
libapp
+0x3606c4
_kDartIsolateSnapshotInstructions
libapp
+0x360208
_kDartIsolateSnapshotInstructions
libapp
+0x4ab680
_kDartIsolateSnapshotInstructions
libapp
+0x4ab56c
_kDartIsolateSnapshotInstructions
libapp
+0x32ffc4
_kDartIsolateSnapshotInstructions
libapp
+0x32fef8
_kDartIsolateSnapshotInstructions
libapp
+0x32fe70
_kDartIsolateSnapshotInstructions
libapp
+0x316d7c
_kDartVmSnapshotInstructions
libflutter
+0x6cbd14

libflutter
+0x804908

libflutter
+0x5a0000

libflutter
+0x5a22ec

libflutter
+0x5ae734

libflutter
+0x3460cc

libflutter
+0x349658

libutils
+0x0181a8
android::Looper::pollInner
libutils
+0x017d1c
android::Looper::pollOnce
libandroid
+0x018598
ALooper_pollOnce
libflutter
+0x349764

libflutter
+0x347d8c

libc
+0x0f55c8

libc
+0x08efbc

Are you willing to submit a PR?
None

Hi,
thank you for the message, we will take a look into it.

okey  krystofwoldrich
If there is any progress, please feel free to contact me.
thank you!

@WangQiangQuan Could you share a code snippet on how to reproduce the error? That would help us with debugging.

sorry,
I don't know how to reproduce this problem. The purpose of using sentry is to find out what the problem is and solve it

@WangQiangQuan can you share your SDK init code snippet and the command you use to build your app and run the dart plugin?
Also, can you share the sentry config in your pubspec file? remove the tokens etc of course :)
Also, is this Android your iOS? look at this as well https://docs.sentry.io/platforms/flutter/troubleshooting/#native-symbolication-on-iosmacos

@marandaneto
I share a list of your questions below：
1: My SDK init code is like this
void main() {
const String flutterDSN = “my DSN”;
String env = "Dev";
await SentryFlutter.init(
(options) {
options.dsn = flutterDSN;
options.environment = env;
options.sampleRate = 1.0;
options.attachScreenshot = true;
options.screenshotQuality = SentryScreenshotQuality.medium;
options.attachViewHierarchy = true;
},
appRunner: () {
...
},
);
}
2 : the command I use to build My app and run the dart plugin is
flutter build apk
And
flutter packages pub run sentry_dart_plugin
3: pubspec file is
sentry:
upload_debug_symbols: true
upload_source_maps: false
upload_sources: false
project: home-edu
org: sentry
auth_token:  (my token)
url: http://sentry.staff.xdf.cn
wait_for_processing: false
log_level: error
commits: auto
ignore_missing: true
4: this problem is found on android device

@WangQiangQuan https://docs.sentry.io/platforms/flutter/upload-debug/#when-to-upload
You don't even need to upload debug symbols if you are not using split-debug-info.
Not sure whats your problem is about, can you share a link to an issue on sentry.io so I can take a look?

@marandaneto
https://sentry.staff.xdf.cn/organizations/sentry/issues/81000/?project=51&query=is%3Aunresolved&statsPeriod=14d

sentry.staff.xdf.cn/organizations/sentry/issues/81000/?project=51&query=is%3Aunresolved&statsPeriod=14d

I don't have access to your self-hosted version.
Can you try to do the very same thing on sentry.io (you can create an organization test).

i will try later
Can you spot the problem from this screenshot
Magic功名
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "getsentry/sentry-dart"                                                                                @.>;
发送时间: 2023年7月7日(星期五) 下午5:21
@.>;
<EMAIL_ADDRESS>主题: Re: [getsentry/sentry-dart] Sentry could not resolve the “abort” exception,I have uploaded the Debug Symbols Successful (flutter packages pub run sentry_dart_plugin) (Issue #1539)
sentry.staff.xdf.cn/organizations/sentry/issues/81000/?project=51&query=is%3Aunresolved&statsPeriod=14d
I don't have access to your self-hosted version.
Can you try to do the very same thing on sentry.io (you can create an organization test).
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@WangQiangQuan The screenshot didn't upload via the email response.

@krystofwoldrich Please give me an email address and I'll send the screenshot

@WangQiangQuan Can you upload it here? That would be better to keep all the context in one place.

ok，This is the error message “ abort

”  on the sentry

@WangQiangQuan sorry the information you've given isn't enough to debug this, your sentry access is self-hosted so we cannot check the metadata, can you provide a minimal reproducible example? otherwise, we can't reproduce it.

I don't know how to reproduce this, can you tell me how to get the metadata, and I'll get it from the self-hosted and give it to you

@WangQiangQuan https://github.com/getsentry/sentry-dart/issues/1539#issuecomment-1625119462
Please try this feature on sentry.io first, so I have access to the metadata.

I haven't scheduled to put it on sentry.io recently, is there any other way to solve this problem？

I haven't scheduled to put it on sentry.io recently, is there any other way to solve this problem？

I haven't scheduled to put it on sentry.io recently, is there any other way to solve this problem？

No, I really need to dig into the metadata to check if this is something on our end.
Please provide a sentry.io issue link or a minimal reproducible example.


Report code coverage with SonarCloud Scanner

Report code coverage reports using SonarCloud Scanner since that has now been setup.

I'll ignore the other warnings about unused variables & commented code as I assume you'll be fixing these shortly.
In respect to the build/test file, SonarCloud recommends these steps which look to be later versions of your commands, however, I think yours will work:
name: SonarCloud
on:
  push:
    branches:
      - main
  pull_request:
    types: [opened, synchronize, reopened]
jobs:
  build:
    name: Build and analyze
    runs-on: windows-latest
    steps:
      - name: Set up JDK 11
        uses: actions/setup-java@v3
        with:
          java-version: 11
          distribution: 'zulu' # Alternative distribution options are available.
      - uses: actions/checkout@v3
        with:
          fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
      - name: Cache SonarCloud packages
        uses: actions/cache@v3
        with:
          path: ~\sonar\cache
          key: ${{ runner.os }}-sonar
          restore-keys: ${{ runner.os }}-sonar
      - name: Cache SonarCloud scanner
        id: cache-sonar-scanner
        uses: actions/cache@v3
        with:
          path: .\.sonar\scanner
          key: ${{ runner.os }}-sonar-scanner
          restore-keys: ${{ runner.os }}-sonar-scanner
      - name: Install SonarCloud scanner
        if: steps.cache-sonar-scanner.outputs.cache-hit != 'true'
        shell: powershell
        run: |
          New-Item -Path .\.sonar\scanner -ItemType Directory
          dotnet tool update dotnet-sonarscanner --tool-path .\.sonar\scanner
      - name: Build and analyze
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}  # Needed to get PR information, if any
          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
        shell: powershell
        run: |
          .\.sonar\scanner\dotnet-sonarscanner begin /k:"DFE-Digital_school-account" /o:"dfe-digital" /d:sonar.login="${{ secrets.SONAR_TOKEN }}" /d:sonar.host.url="https://sonarcloud.io"
          <insert_your_clean_build_command>
          .\.sonar\scanner\dotnet-sonarscanner end /d:sonar.login="${{ secrets.SONAR_TOKEN }}"


Greg sgid roads update

new post + added link to data transportation page.
note: sorry for the long post url..  I'll shorten it next time.. removing the redundant date

The only two issues i found were you need a newline above lists and you do not put the .md extension on links to posts.

ahh...  makes sense. thanks for the feedback.
On Wed, Aug 31, 2016 at 3:16 PM, steveoh<EMAIL_ADDRESS>wrote:

The only two issues i found were you need a newline above lists and you do
not put the .md extension on links to posts.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/agrc/agrc.github.io/pull/216#issuecomment-243904650,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQH4THgrGINlEWJ1EoIMtbZSnaQfrF83ks5qle8qgaJpZM4JyD5-
.

--
Greg Bunce
Utah AGRC http://gis.utah.gov/
801.349.0039
<EMAIL_ADDRESS>


[UX] No clear difference between read only and editable fields

Description:
$subject

Suggested Labels:
improvement
Affected Product Version:
2.1.0
Steps to reproduce:

log into carbon console
goto configure-> Multitenancy->view tenants-> click on edit
there are non-editable (read-only) fields as well as editable fields but its hard to recognize the difference

Suggestion

Closing the issue as this is older than 3 years. Please reopen if the issue is valid still.


PR634 1.17 backport

Problem
Summary of Changes
Fixes #

Codecov Report
Attention: Patch coverage is 96.98795% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 81.6%. Comparing base (57ea214) to head (e56a2f2).

Additional details and impacted files
@@           Coverage Diff            @@
##            v1.17     #640    +/-   ##
========================================
  Coverage    81.6%    81.6%
========================================
  Files         806      806
  Lines      219284   219436   +152
========================================
+ Hits       179002   179167   +165
+ Misses      40282    40269    -13

v1.17 is EOL


[Bug] Correct SimConfig syntax.

Closes #408.

Hey @Roland-djee , did you check if it fixes the ZNE test cases 😅. Do you think we should keep a test case on qadence for future updates that might happen on the pulser end?

Hey @Roland-djee , did you check if it fixes the ZNE test cases 😅.

I am on it now.
Tested with this fix locally, It works fine

Tested with this fix locally, It works fine

Because you've got a local Pulserv0.18.0 haven't you ?

Because you've got a local Pulserv0.18.0 haven't you ?
Yes, I do, I checked that its the latest release, so it should be fine right ? 😅

Yes, I do, I checked that its the latest release, so it should be fine right ? 😅

For your local setting yes. However, to be effective for all users, Qadence should support Pulserv0.18.0 which might cause some upgrade issues. I am on that now.

@dominikandreasseitz Is it fine with you ? If so, I'll merge and work on Renovate since it didn't pick up latest deps versions like Pulser.

@dominikandreasseitz Is it fine with you ? If so, I'll merge and work on Renovate since it didn't pick up latest deps versions like Pulser.

@Roland-djee sure, but should we add some tests which ensure that this is actually functional across updates? same for your previous MR related to noise

@Roland-djee sure, but should we add some tests which ensure that this is actually functional across updates? same for your previous MR related to noise

Well, that should be Renovate's job, if configured correctly.

@Roland-djee i meant adding a unit test which tests this particular piece of code, meaning the simconfig in this MR, and the passing of the noise to sample in the previous one

@Roland-djee i meant adding a unit test which tests this particular piece of code, meaning the simconfig in this MR, and the passing of the noise to sample in the previous one

This can be done with Pulserv0.18.0 since it complies with new API. So I can add that test once Qadence supports it.

not sure about the details on the pulser side here tbh but lgtm

Simply the string keys to the simconfig have changed from depolarization_prob -> depolarization_rate.


Meaning of Info during training?

What does the info show me during training?
Loss XY 	[0.0642155707] --> Which loss is this?
Loss WH 	[0.493789375]  --> Which loss is this?
Loss Conf 	[0.0898802057] -->Which loss is this?
Loss Class 	[1.10027122]  -->Is this the loss for the classification prediction?
Total Loss 	[1.74815643] --> Total loss of what? Classification and Detection?
Current Recall 	[0.99999994] --> This stays the whole time at 0.99, why?
Average Recall 	[0.993055463]

@bexx1963 Please the YOLO paper. The name of the variable is self-explanatory.

Which loss is most important for indicating quality of detection network?

It's the current recall.

I really don't know but it should approach zero initially during warmup. It will then go up when the network starts to be able to pick up the object.

And when there is no warmup? Maybe i can display another key figure such as validation accuracy, how would i calculate it? Or is the keras metrics option in model.fit sufficient?

I do print recall.


Allow MuTect2 users to change thresholds for determining active regions

@chandrans commented on Thu Aug 18 2016
Feature request
Tool(s) involved
MuTect2
Description
Right now, users cannot set the normalLod and nonRefInNormal thresholds for determining active regions in MuTect2. The user should be allowed to set how much evidence is allowed in the normal sample vs the tumor sample before an active region is not triggered. This is a follow-up to #1449.

@davidbenjamin commented on Fri Aug 26 2016
@chandrans FYI as of next week we are porting Mutect2 to GATK4

@vdauwera commented on Fri Aug 26 2016
Great news!

@vdauwera commented on Mon Nov 14 2016
Moving this to GATK4

We'll keep this.  It has been suggested that M2's atrocious runtime on genomes may be due to very liberal active region determination.  It is worth adding this knob.

Replacing with #801.


Chapter1/Homework5

Implemented array operations:

Sort array in ascending order
Inverse array
Remove first element from array
Remove last element from array
Remove element at a given index from array
Insert element at the start of array
Insert element at the end of array
Insert element at specified index of array

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter1/ArrayOperations.cs  8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- Src/BootCamp.Chapter1/ArrayOperations.cs  6

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity increasing per file
==============================
- Src/BootCamp.Chapter1/ArrayOperations.cs  6

See the complete overview on Codacy

VS got me a git push error:
Error encountered while pushing to the remote repository: rejected Updates were rejected because the tip of your current branch is behind its remote counterpart. Integrate the remote changes  before pushing again.
but pull said nothing to pull, so I did a push force.


Subsecond values discarded

The parser incorrectly discards resolutions smaller than a second. Numbers in JSON are (double-precision) floating-points. Also RFC 7519 explicitly mentions the applicability with "non-integer values can be represented". The rounding causes "nbf" violation.

Good catch. I believe this will be resolved with the switch to a smarter type for time values in v4.

This is resolved on v4 branch.

The NewTime float64 conversion is still imprecise and complicated @dgrijalva. Please have a look at https://github.com/pascaldekloe/jwt/blob/master/jwt.go#L277.

Yeah. That’s better. Thanks.
-dave

On Apr 9, 2019, at 4:38 PM, Pascal S. de Kloe<EMAIL_ADDRESS>wrote:
The NewTime float64 conversion is still imprecise and complicated @dgrijalva. Please have a look at https://github.com/pascaldekloe/jwt/blob/master/jwt.go#L277.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Still there, please reopen the issue so that this won't be lost.

just got back to my computer. landed the change


frauddetectionservice pod keeps crashing with chart 0.18

I upgraded today my demo setup on EKS and I found that the frauddetectionservice keeps crashing with this error:
Picked up JAVA_TOOL_OPTIONS: -javaagent:/app/opentelemetry-javaagent.jar
Error opening zip file or JAR manifest missing : /app/opentelemetry-javaagent.jar
Error occurred during initialization of VM
agent library failed to init: instrument

The values section for that service is up-to-date, it's possible that something is wrong with the image (ghcr.io/open-telemetry/demo:1.3.0-frauddetectionservice)?
PS: one change I did, even with the previous chart version/values, is to run the services as a normal user since I cannot run pods as root... so other than limits and env var I added a security context like this:
    securityContext:
      runAsUser: 1000
      runAsGroup: 1000
      runAsNonRoot: true

I can verify that the /app/opentelemetry-javaagent.jar actually exists inside of the container image. This might be a permission issue.

I think this PR may be the culprit. It commented out a chmod 644 command for the agent jar.
If I created and published a test frauddetectionservice image, can you test using that image?

I created this image if you want to test it. I added a --chmod=644 flag to the ADD command for the otel agent.
puckpuck/otel-demo:issue634-frauddetectionservice

No changes:
Picked up JAVA_TOOL_OPTIONS: -javaagent:/app/opentelemetry-javaagent.jar
Error opening zip file or JAR manifest missing : /app/opentelemetry-javaagent.jar
Error occurred during initialization of VM
agent library failed to init: instrument

  Type     Reason     Age                 From               Message
  ----     ------     ----                ----               -------
  Normal   Scheduled  2m7s                default-scheduler  Successfully assigned observability-poc/opentelemetry-demo-frauddetectionservice-749c75657b-5jmx5 to ip-X-X-X-X.yyyyyyy.compute.internal
  Normal   Pulling    2m6s                kubelet            Pulling image "puckpuck/otel-demo:issue634-frauddetectionservice"
  Normal   Pulled     2m1s                kubelet            Successfully pulled image "puckpuck/otel-demo:issue634-frauddetectionservice" in 5.122829888s
  Normal   Created    28s (x5 over 2m1s)  kubelet            Created container frauddetectionservice
  Normal   Pulled     28s (x4 over 118s)  kubelet            Container image "puckpuck/otel-demo:issue634-frauddetectionservice" already present on machine
  Normal   Started    27s (x5 over 2m)    kubelet            Started container frauddetectionservice
  Warning  BackOff    12s (x9 over 117s)  kubelet            Back-off restarting failed container

With just this change to the values:
  frauddetectionService:
    imageOverride:
      repository: puckpuck/otel-demo
      tag: issue634-frauddetectionservice

PS: also running it locally but in a k3d cluster fails with the same error (using the chart that is).

I tried the same thing, but the images are based on a distroless base image. This is insane. We are talking about a demo here.

I tried the same thing, but the images are based on a distroless base image. We are talking about a demo here.

In this case the distroless base image is used to reduce build tile and image footprint.
The demo is quite large and both of these have been a concern for some time.

It's wrong and bad engineering.

There should be some alternative debug image available always, for example with a suffix -debug. I will make a PR in the demo repository to build multiple images.

I wonder if you reverted the Dockerfile changes on this PR and used that as the image to see if it would work.

I spent some time trying to do that (including waiting a full docker compose build) with no luck. Just running docker build in the frauddetectionservice doesn't work. I admit I was strictly time bound on this...

Trying again with 1.4.0 and latest chart but until now:
frauddetectionservice Picked up JAVA_TOOL_OPTIONS: -javaagent:/app/opentelemetry-javaagent.jar
frauddetectionservice Error opening zip file or JAR manifest missing : /app/opentelemetry-javaagent.jar
frauddetectionservice Error occurred during initialization of VM
frauddetectionservice agent library failed to init: instrument
wait-for-kafka opentelemetry-demo-kafka (<IP_ADDRESS>:9092) open
(frauddetectionservice)

Small update: with 1.7.2 and my modified values file specifying the same securityContext I get the same error, latest chart version as well.


Add flags to language selector

Current language menu is not intuitive, because it's "text only", please add country flags to menu! @knicos

This was removed deliberately because countries and languages are not the same thing. Generally, the advice is not to use flags.

Really? So reason is accessibility? Ok.

Combined with political correctness. Why should a US citizen have to choose a British flag and why is Finnish using the Finnish flag when Finland has 3 official languages.


java.lang.NoClassDefFoundError: com.daimajia.easing.Skill

My app crashes with this error whenever I use this on my DescreetSeekbar view.
error message:
java.lang.NoClassDefFoundError: com.daimajia.easing.Skill

@srujan7 Please make sure that you have already import the com.daimajia.easing:library:1.0.2@aar

@daimajia  I already have imported!

@daimajia I already have imported!


feat: inject environment setup into session

This PR adds an entrypoint to the base image that copies the contents of the files found in the user's renku-env repository in the gitlab instance. It takes the oauth token from the environment and uses it to clone the private renku-env repo. If the repo isn't there it fails silently. The contents of all the files in the renku-env repository are appended to the corresponding files in $HOME.
This requires the user to set up a private renku-env repository and it should be documented in the main docs.
To test, you may either launch an environment from https://renkulab.io/projects/1464/ or use the image rrrrrok/singleuser:entrypoint in your own project.
closes #64

Just to be clear -- this is intended for things like settings in .bashrc (i.e. emacs config?)

The contents of all the files in the renku-env repository are appended to the corresponding files in $HOME.

The way I'm reading this -- renku-env should just have env related files (i.e. most of the default template stuff should be cleared out of the repo if it's created via renku)?

I tested in the following way & functionality seems to work:

Created private renku-env project, added a .bashrc with one line of contents:

alias helloworld="echo 'hello world'"

Created public use-renku-env project, updated the FROM line in the Dockerfile to:

FROM rrrrrok/singleuser:entrypoint

Started a jupyterlab from use-renku-env, opened a terminal:

$ helloworld
hello world

@emmjab thanks for having a look - yes, the renku-env project should not be a renku project - alternatively, we could have a template for this? Not sure how the UI wants to handle it, one suggestion was to offer some UI elements to standardize the process.

I think the way it should work is to copy over, recursively, any folders that are defined in the repository. The appending to existing file behavior works well for certain files (like .bashrc), but might no work well for everything. I think we need to clarify that aspect and maybe make it a bit more precise (e.g., maybe it should only apply to bash dot files and a different way of resolving conflicts should apply elsewhere).

It's hard to see how to make some generic behavior to support all use cases. One option that comes to mind is to use something like the manifest.yml file we have in the templates. It could have sections to define specific behaviors. Something like:
copy:
  - name: scripts
     dst: ${HOME}/my_scripts
append:
  - .bashrc

This would copy scripts to ${HOME}/my_scripts and append the contents of .bashrc to the existing one.
The other thing that would be nice is to exclude the renku_env repo from the project listing in the UI.
This would copy


Fix LockRecursionException when doing FindById

Fixes #2471, https://github.com/mbdavid/LiteDB/issues/2435, https://github.com/mbdavid/LiteDB/issues/2483
Problem was, that the enumerator did not release the readlock when doing FindById. When the Checkpoint then tried to aquire a writelock, the semaphore threw an exception, because there was still a readlock pending.
I only fixed that case, because i don't know if adding trycatch to all of the method would have downsides.

Ok just found out it was already fixed by https://github.com/mbdavid/LiteDB/pull/2436 but this fix here has a easier git comparison and tests.


chore:Fix-Structured the lerna.json file

the lerna.json by default has a particular structure, and i think we should follow the same

this needs to be rebased tho, 27.5.0 is a few weeks old 🙂

closing this one, will pull a new one


How to consume postgresql metrics in Azure

Hello,
I am referring url: https://docs.microsoft.com/en-us/rest/api/postgresql/operations/list#operation
Can you please specify the endpoint for fetching metrics for below mentioned two metrics:
{
              "name": "cpu_percent",
              "displayName": "CPU percent",
              "displayDescription": "CPU percent",
              "unit": "Percent",
              "aggregationType": "Average",
              "fillGapWithZero": true
            },
            {
              "name": "memory_percent",
              "displayName": "Memory percent",
              "displayDescription": "Memory percent",
              "unit": "Percent",
              "aggregationType": "Average",
              "fillGapWithZero": true
            }

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

ID: 93da39f3-c5d3-cce9-0101-784415455e26
Version Independent ID: f91645ce-b33f-f43e-f5af-69aeb967ae92
Content: Azure Monitor data platform
Content Source: articles/azure-monitor/platform/data-platform.md
Service: monitoring
GitHub Login: @bwren
Microsoft Alias: bwren

@arindam-b
Thanks for your feedback! We will investigate and update as appropriate.

Thank you @MarileeTurscak-MSFT
@femsulu - Can you share the URL to fetch it. I tried to guess it as
/subscriptions/58f5b54c-85ab-4bcb-91a1-c642c701a31b/resourceGroups/NagiosInterconect/providers/Microsoft.DBforPostgreSQL/servers/atos-postgresql/providers/microsoft.insights/metrics?metricnames=cpu_percent&api-version=2017-12-01&aggregation=Average&interval=PT1H
But it is not Ok. Hence need the URL.
It is for one of the most prestigious clients of Azure and if you could share the details fast it would help a lot as I need to demonstrate it to them as per their request.
Thanks.

@arindam-b apologies for the delay. Azure technical support will be the best channel to assist you - guidance here. If you do not have a support plan, send mail to<EMAIL_ADDRESS>and include your subscription ID and we will gladly assist you with engaging technical support.

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

did we ever come up with the answer to this question?

@nosretep Instructions to have a support request created have been sent in response to your email. Please respond to that email with the SR # that is created. Thank you!

@Mike-Ubezzi-MSFT none of the sub-types in the drop down are applicable.

The summary is going to be Metrics API for Microsoft.DBforPostgreSQL server only returns "cpu_percent", is that a known issue?

@nosretep Adding your analysis of root cause to this issue for reference purposes:

Thanks, I found the answer by going to portal.azure.com and then inspecting the network requests. I found the “batch” API call where the metrics API was being called indirectly. I think that I tried “metricnames” on Friday, but I think that I did not have it in tandem with the other query string parameters, so it would just always return “cpu_percent” (just like not using “metricnames”). I would have thought it would just return an error message, or an empty result set.

Additional information can be found:
Retrieve metric values


Reduce "The number of foldable regions is limited to a maximum of 5000" pop up

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

VS Code Version: 1.64
OS Version: MacOS

https://code.visualstudio.com/updates/v1_64#_editor-folding-limit
Since updating to version 1.64 I'm getting notifications constantly about folded regions. The files I have open are not that big so I don't know why this is coming up or what it wants me to do. 5000 is already a lot so I don't need to increase it.
VSCode has enough pop-ups as it is and this is just adding to the noise. What is the call-to-action here? I find it confusing and I'm not sure why its asking me to increase the foldable regions.

We had issues from users that found that folding ranges were missing and understand why.
But yes, I'm also not a fan of notifications with information that is not relevant to me.
@jrieken What would you say, is the language indicator a better place for this?

@jrieken What would you say, is the language indicator a better place for this?

Yeah, that might be fair use of it. Either a message specific about folding or a more generic large file info (in case this also affect after features)

@jasonwilliams Am I right to assume that the file you're opening when you get this message is well over 10,000 lines long? If not it shouldn't be possible to have that many foldable regions.
It should be ok to increase the limit, after clicking on 'Folding Maximum Regions', to 20,000 or more. The 5,000 limit is fairly conservative.

I started getting a Debug Terminal open when I opened the terminal.  Now I'm getting this message.  Are these related?

I get this popup if I open a package-lock.json file, which the current one I'm looking at is 42,415 lines. I'm not going to edit it, but I was in vscode and just wanted to see the top few lines that have the file version.

In the meantime, can this warning be removed again? It pops up very frequently. It might save some time for users who were confused by the missing folding ranges, but it wastes a lot of time for users who don't care about folding.

@mstange Have you tried bumping that limit to 20,000? Or if you never care about folding at all then you could turn it off by opening settings ( Ctrl+, ), type "folding", and uncheck the top option there.
@aeschli Also, how about bumping that default to 10,000? Or, if you don't mind reviewing some code rework to make that area faster, bump the default to 20,000 or more?

@mstange Have you tried bumping that limit to 20,000?

It helps for my 60000 line plain text file, thanks. It probably won't help for the other cases where I encountered it: When I pipe long command line output into code -. I think that case would be improved by not attempting code folding on editors that haven't been saved as files (and where the "language" has been inferred from the contents rather than from the file extension).

Or if you never care about folding at all then you could turn it off

I care about code folding in code. But I also use VS Code on files that are not code: Plain text files and logs. Especially logs are the ones that are likely to be very big. I don't need code folding in those files.

@MattGrdinic This is happening because a "folding range provider" is returning a large number of foldable regions. Those are marked with the downward chevron (like a wide "v") in the margin just left of the text when you hover the cursor there.
If you have one or more extensions which provide folding ranges for a particular language then that's where they are coming from. Some of those are built-in, e.g. for Typescript. If there's no extension present which provides folding ranges for a language then they're coming from the default indent-based provider. That provider creates a foldable range at every line in the source where the next line has a larger indent.
You can disable folding range provider(s) for a particular language or all languages via "editor folding" in vscode settings.
@aeschli Do you think this issue could be resolved by changing that message, about too many folding regions, to be an information level message in the problems panel? I've looked around for how to use that panel in vscode and haven't figured it out yet. If you'd like it moved there a pointer on how to do it would help.

@PieterBranderhorst Agree that the notification is too prominent. The idea is to use the language indicator instead:
E.g. JSON already uses the language indicator API:

@aeschli I was thinking more of adding a message in the problems panel:

@PieterBranderhorst We currently use the problems view only for resource based diagnostics, so I don't think we should put it there.

PMJI with a solution - I found this thread specifically with a search for this new issue...
Exactly as noted by @mstange - I'm getting this in a project where I'm looking at huge .log files.
I see editor.foldingMaximumRegions was added a few months ago. There's actually a little oversight/bug in this implementation that's key to this issue. This feature was implemented to be language-specific. Example:
"[log]": {
   "editor.foldingMaximumRegions": 11,
   "something.else": value
},
"[php]": {
   "editor.foldingMaximumRegions": 22,
   "unrelated.setting": value
}

If you set two languages like that you'll see the warnings change with the correct maximum. However, click on the warning where it offers to change the config, and the setting gets saved outside of the scope of the language. So you don't get the above, you get something like this:
// attempt to evade the nag
"editor.foldingMaximumRegions": 40000,
"[log]": {
   "something.else": value
},
"[php]": {
   "unrelated.setting": value
}

That's a new issue for a different ticket. What's relevant here is that there is another setting editor.folding, and that's also language-specific. So to disable folding for .log files, while preserving folding for .php files with a reasonable warning value, this is the settings.json detail:
"[log]": {
   // no need to set max regions
   "editor.folding": false,
},
"[php]": {
   // be sure folding is active for this language
   "editor.folding": true,
   // reasonable warning trigger
   "editor.foldingMaximumRegions": 10000,
},
// increase warning default above 5000 for all languages except as specified
"editor.foldingMaximumRegions": 30000,

With this, the warning serves the intended purpose, and doesn't get triggered outside of that scope.
HTH

I pushed a change. We're now using the language indicator to inform about limited folding, outline and color symbols. No more notifications.

Thank you!


adding tile to beginning of array

Is there a way to add a new tile on load to the beginning of the grid vs the end?
in the add tile example you dynamically add tiles to the end of the grid, i would like to on load put them at the front.
we have a flag for newly added tiles and i want to prepend them on the grid for increased user visibility and want to know its possible without breaking the tool

FYI:
if you concat a new object/array if will write over it and position. so if your new items have positioning 0,0 it will iterate over and place them in the first position and then push down the next one and repeat creating a vertical column of new tiles in the first column. if you want them to place horizontally via the first row, a map function on the new object and then setting the x coordinate = index will have them placed horizontally and iterate.
hope that helps anyone else :)


Add Monir-Zaman to Contributors list

Thanks for a wonderful git tutorial

@zamanuap, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

Hello Roshan
Hope you are doing well. As I mentioned earlier this is an excellent tutorial to learn about Git  and Gtihub. I have spent hours to learn Git and Github from youtube and other online sites. But I didn't get the whole thing rather I had a vague concept how Git works. But with this tutorial I got a clear concept and now I'm confident that I can work with Git and Github.
Thank you once again for this wonderful tutorial. I will recommend this to everyone who are facing  trouble to learn Git.

@zamanuap, I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


UI preventing org repo creation as well if user org limit is reached

Scenario

On our instance we have MAX_CREATION_LIMIT = 0 to prevent random new users from spamming our instance
There is an org which has a few members, which is being used for a collaborative project. These members should only be able to create repos inside this org, but not under their user account.
Users have been put into a team Dev which has Create repositories enabled

Problem

On the repo create page, the submit button Create Repository is disabled in case CanCreateRepo == false: https://github.com/go-gitea/gitea/blob/5c91d7920f4aff08768e274269e211e926aa3d36/templates/repo/create.tmpl#L212
CanCreateRepo checks whether the user already has too many personal repos: https://github.com/go-gitea/gitea/blob/5c91d7920f4aff08768e274269e211e926aa3d36/models/user/user.go#L240-L251
But the submit button will be disabled for all available repo owners; Personal and also Orgs!
This means the UI will not allow users to create org repos even though they have the Create repositories permission in their org!

Solutions

This is only a UI bug, the backend validation works as expected (I have tested this by removing the previously mentioned disable check from the template to force the button to be always enabled)
The CanCreateRepo value should only influence creation of a personal repo
Repo limit of the org is only checked after the form is submitted, this would be a good quick-fix for this issue (removing the disabled button check and the error banner from the template)
For a long term solution we could hide the personal user account in the owners list the same way orgs are hidden when their limit is reached. This would make the disabled submit button and the banner completely obsolete, as the user is forced to select an owner that is allowed to have more repos created

Note that I was not able to reproduce the issue on the demo instance, because there is no repo count limit and also it seems like you can't create orgs there..
Screenshots
Trying to create a new repo testrepo45 under testorg organization, note the banner telling me there is a limit:

Submit button is disabled:

Repo create works anyway when enabling the submit button by hand:

Gitea Version
v1.21.9

I has try fix it in https://github.com/go-gitea/gitea/pull/15924, but sadly looks no one think it's a bug :( . maybe need a beter design about how to limit repository number for org. or repo number limit will looks meaningless if all user can create orgs.

This is definitely a bug! The validation on the frontend does not match the validation in the backend!
If the behaviour of the ui is actually wanted (which would be really weird), the backend validation would be wrong, which would not only still be a bug, but also a security problem!!!