Bad value for optimize in bench.

https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L50
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L51
In both lines it pass (speed(debug0)) as the optimize argument to make-dsb-bencher and  make-dsm-bencher, which then use it to create a declaration (optimize (speed (debug 0))), which is just plain wrong.
The way make-dsb-bencher  and make-dsm-bencher are currently written it can "optimize" only one thing, e.g. (speed  3), To allow it to optimize more things, an '@' character needs to be added in
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L22
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L40
inside  make-dsb-bencher and make-dsm-bencher  so they become
`((declare (optimize ,@optimize)))

and then bench can pass a list of optimizations like it does now.

Oh, thank you, I will fix this when I'm back home.On 21 Nov 2023, at 18:30, Yehouda Harpaz @.> wrote:
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L50
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L51
In both lines it pass (safety (debug0)) as the optimize argument to make-dsb-bencher and  make-dsm-bencher, which then use it to create a declaration (optimize (safety (debug 0))), which is just plain wrong.
The way make-dsb-bencher  and make-dsm-bencher are currently written it can "optimize" only one thing, e.g. (safety 0), To allow it to optimize more things, an '@' character needs to be added in
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L22
https://github.com/tfeb/dsm/blob/08d7f6d616174ef7c3148579dc24cf214e75cb79/bench/bench.lisp#L40
inside  make-dsb-bencher andmake-dsm-bencher  so they become
`((declare (optimize @.)))
and then bench can pass a list of optimization, e.g. ((safety 0) (debug 0)).
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Resolved by 1c1746a I hope: thank you.


Fix function require_splitter

The function returns whether a V4V6 splitter app is required.  Before it checked whether an parameter on-a-stick was a command line argument, but this should not be the case as an on-a-stick setup is also possible when a configuration file has only one instance and one queue.
This caused that the function returned true if passing an on-a-stick argument to the command line, but returned false if running a lwaftr using only a configuration file.
After this fix I'll check whether the code path that not uses a splitter is correct (using vlan tags but different vlan tags), but that's another issue IMHO.

I have pushed the following changes:

If passing an --on-a-stick command line argument, the function that migrates configuration has to overwrite the value of queue.external_interface.device to the value of the v4 device. Use case is a configuration that has a device for an instance and a different one for an external_interface.
Added a function that determines whether to run on-a-stick if:

The configuration file has an instance device but not external_interface device.
Both interfaces are set and have the same value.

If a configuration runs on-a-stick and each interface uses different VLAN tags, we skip the V4V6 splitter and setup two virtual interfaces for each NIC. However in this scenario, it's necessary that both interfaces have a different MAC address.


Returns data as json

Hey, I use your project all the time, congratz!
I'd love to know if you're interested in implementing a new route to return results as JSON to be consumed as an external API (this would be amazing to use inside Google Sheets 😄 ). As far as I can see, today, the project is a static one, and adding such a feature would mean implementing a server (extra costs), but I'd be curious to know if you've considered it.
Thanks

Having an API to calculate salaries and taxes to pay would actually be a great idea. I will consider it for sure!

You have me a good idea, and so I'm currently developing a new project - saldo.
The idea is to calculate taxes and net income for any type of income one might have.
For now, It works for dependent workers only. I'm now implementing features for independent worker as well.
The idea is to eventually make it accessible through an API so we can abstract the calculation logic from the UI.

Amazing! Looking forward to it 😄


linux_mptcp_95: 0.95 -> 0.95.1

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 115178 run on x86_64-linux 1

1 package built:

linux_mptcp (linux_mptcp_95)

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

linux_mptcp:

build log of '/nix/store/dq31gr5lfcwnc2iri9npx2qh42bm9r4c-linux-4.19.126-mptcp_v0.95.1' is not available
error: --- Error --- nix
build log of '/nix/store/dq31gr5lfcwnc2iri9npx2qh42bm9r4c-linux-4.19.126-mptcp_v0.95.1' is not available
When evaluating attribute ‘linux_mptcp’:
warning: unclear-gpl
gpl2 is a deprecated license, check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/os-specific/linux/kernel/manual-config.nix:288:9:
    |
288 |         license = lib.licenses.gpl2;
    |         ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/unclear-gpl.md


MSG: Failed validation of sequence '[unidentified sequence]'. Invalid characters were: _900108945113131010000441

Hi, I am trying to use AliTV to display my gene clusters, when I use the simple mode, which directly input the .fasta files, it worked. However when I use the yaml config file, it throws the error as: 'MSG: Failed validation of sequence '[unidentified sequence]'. Invalid characters were: _900108945113131010000441'. (But I have not found invalid characters in my fasta file.)
I wanna to know how this happens, and how can I solve this problem?
Thanks in advance.
Best,

The config file
genomes:
   -
    name: a
    sequence_files:
      - GCA_900108945.1.fasta
    feature_files:
      nodnif:
             - /home/AliTV/AliTV-perl-interface/data/Rhizobia/GCA_900108945.1.txt
   -
     name: b
     sequence_files:
      - GCA_014201025.1.fasta
     feature_files:
      nodnif:
             - /home/AliTV/AliTV-perl-interface/data/Rhizobia/GCA_014201025.1.txt

features:
  feature1:
    color: "#e7d3e2"
    form: arrow
    height: 30
    visible: 1

Maybe the error is in the feature_files and not the fasta files. Does AliTV run with this yml file, when you remove the feature_files entries?


Use next.js to create simple example structure.

Setup next.js to serve example pages. Pages are yet to be filled with specific
examples in future commits.
Run the dev server using npm run example and access the page on
localhost:3000.

Was already merged as part of another PR


GridViewColumn not localized

Hi,
I have a problem with a localization. When I change the language, the Headers of GridViewColumns aren't localized. Infact appears "key:MyKeyword" etc.
<GridViewColumnCollection x:Key="RuntimeColumns">
   <GridViewColumn Header="{lex:LocText HighTheshold}" Width="75"/>
   <GridViewColumn Header="{lex:LocText LowTheshold}" Width="75"/>
   <GridViewColumn Header="{lex:LocText Time}" Width="75"/>
</GridViewColumnCollection>

Do you have any suggestions?

Did you try with extented or full notation, like:
<GridViewColumn Header="{lex:LocText RessourceName:HighTheshold}" Width="75"/>
or full notation
<GridViewColumn Header="{lex:LocText AssemblyName:RessourceName:HighTheshold}" Width="75"/>

Thank you!


Pyramid level calculation for multiscale is incorrect

🐛 Bug
When using pyramids in Napari, the level chosen appears to be incorrect. This can cause the application to slow down or crash in certain instances.  Specifically, a lower resolution level should often be chosen than what is actual picked.  The main issue appears to be that the algorithm assumes the layer image fills the entire field of view when picking the pyramid level.  I think this may also be a contributing factor into why the multiscale images are slower than they should be in napari.
To Reproduce
Steps to reproduce the behavior:

Create pyramid view (I'm using a 2D pyramid) with multiple levels (say 1kx1k with 4 levels)
Zoom out
Observe the level chosen (it is in the toolbar).  For an image smaller than the display, it will never go below 0 even if you zoom way out.

Issue I'm running into currently that crashes napari:
I have a large mosaic image that is not square.  The image has dimensions of [3072, 70656].  When I zoom out, napari picks the zoom level to be 1 (scale is [2,2]).  This is because the y-axis is being compared to the whole view window, even though it is only taking up a small area of the window. The x-axis at level 1 is now greater than the max texture size and napari crashes.
Expected behavior
Napari should NOT use the display window size by itself when calculating the level, but should calculate the extent of the layer being displayed and use that instead.  Then it can pick the level of the multiscale that most closely matches the what needs to be displayed.  To put in other words, if we zoom out so that the 10kx10k image now only takes up 100px x 100px on the display, we should pick the level that is closest to the 100x100 pixels instead of using the size of the display.
napari/layers/base/base.py Layer._update_draw function:
The parameter shape_threshold needs to be cropped to match the extent of the layer being displayed before being passed to compute_multiscale_level.  It looks like the code crops the size of the layer to only what is being displayed, but then doesn't crop the shape_threshold to the layer as well.
Environment
napari: 0.4.16
Platform: Linux-4.18.0-372.9.1.el8.x86_64-x86_64-with-glibc2.28
Python: 3.9.0 (default, Nov 15 2020, 14:28:56) [GCC 7.3.0]
Qt: 5.15.2
PyQt5: 5.15.6
NumPy: 1.22.3
SciPy: 1.8.0
Dask: 2022.05.0
VisPy: 0.10.0
OpenGL:

GL version: 4.6.0 NVIDIA 515.48.07
MAX_TEXTURE_SIZE: 32768

Screens:

screen 1: resolution 3840x2160, scale 1.0

Plugins:

console: 0.0.4

napari-svg: 0.1.6

scikit-image: 0.4.16

Any other relevant information:

Additional context

Good point @michael-sutherland and I believe I've encountered it before myself. Agree this needs fixing.

Thanks for the write-up here. I'm going to assign myself to this issue because there's some strong overlap with the async slicing redesign I'm working on - we need the non-experimental multi-scale data lookup to work as intended for some of that work to be useful.
I'll start by writing up some test cases based on the great description above.
Thanks @michael-sutherland!

Depending on timelines, it might be nice to patch the potential crashing.  I threw in an ugly check to make sure I'm not picking a level that is displaying more pixels than the max texture size.  I ended up hard coding the texture size for my computer to make it easy for me.  Here's my snippet from layers/utils/layer_utils.py:
def compute_multiscale_level(
    requested_shape, shape_threshold, downsample_factors
):
    """Computed desired level of the multiscale given requested field of view.

    The level of the multiscale should be the lowest resolution such that
    the requested shape is above the shape threshold. By passing a shape
    threshold corresponding to the shape of the canvas on the screen this
    ensures that we have at least one data pixel per screen pixel, but no
    more than we need.

    Parameters
    ----------
    requested_shape : tuple
        Requested shape of field of view in data coordinates
    shape_threshold : tuple
        Maximum size of a displayed tile in pixels.
    downsample_factors : list of tuple
        Downsampling factors for each level of the multiscale. Must be increasing
        for each level of the multiscale.

    Returns
    -------
    level : int
        Level of the multiscale to be viewing.
    """
    # Scale shape by downsample factors
    scaled_shape = requested_shape / downsample_factors

    # HACK: find min level that doesn't exceed size limit
    min_level = 0
    for level in range(scaled_shape.shape[0]):
        if np.any(scaled_shape[level] >= 32768):
            min_level = level + 1

    # Find the highest resolution level allowed
    locations = np.argwhere(np.all(scaled_shape > shape_threshold, axis=1))
    if len(locations) > 0:
        level = locations[-1][0]
    else:
        level = 0

    level = max(min_level, level)   # HACK
    return level

I'm actually surprised by the crashing behavior, as napari has some existing logic to downsample any image that exceeds that the max texture size. Maybe there's at least one case when that doesn't happen? I plan to add to add at least one test case to cover that.
Thanks for the suggested workaround. Ideally, I'd like to have a more complete fix here with some test coverage. I think I almost have a suitable test case to cover the issues here, though it seems like I have to do some special setup to replicate errors in napari-proper.
@pytest.mark.parametrize(
    ('camera_zoom', 'expected_data_level'), ((1, 0), (0.5, 1), (0.25, 2))
)
def test_on_draw_with_multiscale_image_camera_zoom(
    make_napari_viewer, camera_zoom, expected_data_level
):
    viewer = make_napari_viewer()
    qt_viewer = viewer.window._qt_viewer
    shapes = [(1600, 1600), (800, 800), (400, 400)]
    data = [np.empty(s) for s in shapes]
    layer = viewer.add_image(data, multiscale=True)
    qt_viewer.canvas.size = (1000, 1000)
    # This does not get triggered automatically in the test.
    qt_viewer.on_resize(None)
    viewer.reset_view()
    viewer.camera.zoom = camera_zoom

    qt_viewer.on_draw(None)

    assert layer.data_level == expected_data_level

If I don't have a PR that fixes this test by the end of the week, then I'll definitely consider bringing in your suggested workaround, which could make it into the next patch version of napari.

@michael-sutherland : took me a little longer than I said, but I think I have a fix for this in #4842


Karpenter Node always in "Not Ready" status (or) Deamonset( aws-node-xxxx, kube-proxy-xxx) in pending status

Is an existing page relevant?
https://karpenter.sh/v0.25.0/getting-started/migrating-from-cas/
What karpenter features are relevant?
While Migrating from CA to karpenter I have followed the above document. But there was a small mistake which takes me 2-3 days to resolve.
At "Update aws-auth ConfigMap" step, we have to edit  aws-auth configmap with the following rolearn as per doc.
rolearn: arn:aws:iam::${AWS_ACCOUNT_ID}:role/KarpenterInstanceNodeRole
But the role was created with different name in the earlier steps of the doc as KarpenterNodeRole-${CLUSTER_NAME}
How should the docs be improved?
Please replace the role name in the rolearn.
rolearn: arn:aws:iam::${AWS_ACCOUNT_ID}:role/KarpenterNodeRole-${CLUSTER_NAME}
Community Note

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

Thanks for surfacing this! #3518 fixes the problem that you are describing. I'll close this issue when that goes in.

Closing now that #3518 is in


Side menu Integration

I am trying to use the Avatar side menu for my app, but I cant integrate it into my app. What all code and files are needed to be copied?
Could anyone help me out with it please as I need it for my project asap?
Thank you.
P.S: I'm a newbie to ionic

Hey @vishal2develop, thanks for your interest in this repository!
You have to get the HTML code from app.html regarding your desired sidemenu, then its css from app.scss. This shows to me that I have to add a README explaining how to do so for new users. Thanks for your feedback :)
Here's the gist with what you have to use: https://gist.github.com/yannbf/0f2fea2421dee8e388b4d0c5e78ba480
Just change the logic and assets.

Thank you so much for your valuable and prompt reply. Will copying the app.html and app.scss replace the default style of the menu.
I don't want to click on a button to change the  UI of the side menu. I want the avatar mode to be my default.
How am I supposed to do that?

yes, if you integrate that code with yours, the avatar sidemenu will be displayed by default.

I'm gonna close this issue as it seems to be resolved. I provided you all the ingredients, so you just need to bake the cake.
Thanks!


Issue in assigning csv file data into a list - see this working example to process every field

when i try to extract data from product.csv file into a list, i could not assign product.csv to csv_file, how can we assign it?
array_items = []
csv_file = 'product.csv'
load `csv_file` to lines
array_lines = lines.split('\n')
for (n=0; n<array_lines.length; n++)
    array_items[n] = array_lines[n].trim()

Hi @fazlur10 you are very close to the solution already, below is a working example of reading through each field in csv.
demo.tag
load data.csv to lines
array_lines = lines.split('\n')
for (n=0; n<array_lines.length; n++)
    array_items = array_lines[n].split(',')
    for (m=0; m<array_items.length; m++)
        echo `array_items[m]`

data.csv
#,name,country
1,Jenny,US
2,Zack,Africa
3,Lee,China

output
START - automation started - Sat Jun 05 2021 11:41:25 GMT+0800 (+08)

load a.csv to lines
#
name
country
1
Jenny
US
2
Zack
Africa
3
Lee
China

FINISH - automation finished - 1.3s

thank you for the response
On Sat, 5 Jun 2021 at 09:13, Ken Soh @.***> wrote:

Hi @fazlur10 https://github.com/fazlur10 you are very close to the
solution already, below is a working example of reading through each field
in csv.
demo.tag
load data.csv to lines
array_lines = lines.split('\n')
for (n=0; n<array_lines.length; n++)
array_items = array_lines[n].split(',')
for (m=0; m<array_items.length; m++)
echo array_items[m]
data.csv
#,name,country
1,Jenny,US
2,Zack,Africa
3,Lee,China
output
START - automation started - Sat Jun 05 2021 11:41:25 GMT+0800 (+08)
load a.csv to lines

name
country
1
Jenny
US
2
Zack
Africa
3
Lee
China
FINISH - automation finished - 1.3s
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kelaberetiv/TagUI/issues/1049#issuecomment-855177173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBP5K34YGYJZCLIGBYTT6DTRGMNFANCNFSM46CLOSOQ
.


enhance API readme

Signed-off-by: Mike Ng<EMAIL_ADDRESS>See https://github.com/mikeshng/api/blob/64dddfa667e75ad1bf92986234739f7b73eeb998/README.md
for rendered preview.

/assign qiujian16


[Snyk] Security upgrade black from 24.2.0 to 24.3.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

test-requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Issue
Upgrade
Breaking Change
Exploit Maturity

Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-BLACK-6256273
black:  24.2.0 -> 24.3.0
No
Proof of Concept

Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

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

Project coverage is 68.29%. Comparing base (b6ef82d) to head (924c399).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   68.29%   68.29%
=======================================
  Files         102      102
  Lines        8077     8077
=======================================
  Hits         5516     5516
  Misses       2561     2561

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


Loaded PCRE do not match with compiled

Hello,
We installed and configure modsec on a new debian wheezy (v8.8).
Everything works like a charm except one thing.
It seems that the version of PCRE installed by debian (8.39) is not the same version as the one in modsec (8.35).
[Mon Jun 19 13:09:24.000370 2017] [:notice] [pid 29605] ModSecurity for Apache/2.8.0 (http://www.modsecurity.org/) configured.

 For more information visit: http://status.modsecurity.org/
[Mon Jun 19 13:09:25.009950 2017] [mpm_prefork:notice] [pid 29608] AH00163: Apache/2.4.10 (Debian) mpm-itk/2.4.7-02 OpenSSL/1.0.1t configured -- resuming normal operations

And with that, modsec blocks requests that use pcre. Here are 5 examples on 5 different rules:
[Mon Jun 19 11:03:52.383438 2017] [:error] [pid 20805] [client xxx.xxx.xxx.xxx] ModSecurity: Rule 7702436f3528 [id "942270"][file "/etc/modsecurity/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"][line "316"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "mail.xxx.org"] [uri "/"] [unique_id "WUeTeIlKzlsAAFCo9jkAAAM"]
[Mon Jun 19 11:03:52.383438 2017] [:error] [pid 20805] [client xxx.xxx.xxx.xxx] ModSecurity: Rule 7702436f3528 [id "942270"][file "/etc/modsecurity/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"][line "316"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "mail.xxx.org"] [uri "/"] [unique_id "WUeTeIlKzlsAAFCo9jkAAAAM"]
[Mon Jun 19 11:03:52.382332 2017] [:error] [pid 20805] [client xxx.xxx.xxx.xxx] ModSecurity: Rule<PHONE_NUMBER>f0 [id "941310"][file "/etc/modsecurity/REQUEST-941-APPLICATION-ATTACK-XSS.conf"][line "702"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "mail.xxx.org"] [uri "/"] [unique_id "WUeTeIlKzlsAAFCo9jkAAAAM"]
[Mon Jun 19 11:03:52.379851 2017] [:error] [pid 20805] [client xxx.xxx.xxx.xxx] ModSecurity: Rule 770243a076b0 [id "941200"][file "/etc/modsecurity/REQUEST-941-APPLICATION-ATTACK-XSS.conf"][line "367"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "mail.xxx.org"] [uri "/"] [unique_id "WUeTeIlKzlsAAFCo9jkAAAAM"]
[Mon Jun 19 11:11:26.701057 2017] [:error] [pid 23672] [client xxx.xxx.xxx.xxx] ModSecurity: Rule 770243bf7be8 [id "932110"][file "/etc/modsecurity/REQUEST-932-APPLICATION-ATTACK-RCE.conf"][line "185"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "mail.xxx.org"] [uri "/"] [unique_id "WUeVPolKzlsAAFxmkvAAAAAB"]

Normally, when pcre fails, modsec doesn't block the request (at least on our other servers). And the logs here don't say "ModSecurity: Access denied with code YYY"
But that's the problem. As soon as we added SecRuleRemoveById xxx on these rule numbers, we could play with roundcube again.
Any idea what's the best solution.

upgrade pcre from modsec to the same version as the one installed on the system? (is that possible?)
downgrade pcre from the system?
no solution with modsec v2, have to install v3? (I'd like that very much, but I need a bit more time before testing v3)

Hi @quenenni,
ModSecurity uses the PCRE that is loaded from your server. You have to compile ModSecurity with the exactly same version that your web server is using.

Thanks for your very quick answer.
It seems it was not a pcre problem, but a high risk country problem.
I'm not sure now what exactly happens with our Pcre version mismatch.
I'll wait to go for the v3 before checking more.
Thanks again.
Have a nice day.


Use Shenandoah for GC

This may also help with the CI failures.

/retest

/retest  jvm-build-cache-on-pull-request

/retest jvm-build-cache-on-pull-request

appstudio bootstrap failed with in-repo
trying again
/retest

unrelated appstudio bootstrap issue nuked this run https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_jvm-build-service/144/pull-ci-redhat-appstudio-jvm-build-service-main-jvm-build-service-in-repo-e2e/1553112748081025024
/retest

This does not seem to have made any difference, the actual fix was to increase CPU/memory


For namespaces, the generated bundle type definitions doesn't declare exactly what the generated module exposes

When using the CLI command generateBundles, the generated type definitions are not exectaly in-sync with the generated JavaScript module.
From the following grammar:
Foo {
}
Bar {
}

Here is what the generated JS module looks like, formatted for clarity:
'use strict';
const ohm = require('ohm-js');
const result = ohm.createNamespace();
result.Foo = ohm.makeRecipe(["grammar", {"source": "Foo {\n}"}, "Foo", null, null, {}]);
result.Bar = ohm.makeRecipe(["grammar", {"source": "Bar {\n}"}, "Bar", null, null, {}]);
module.exports = result;

Here, the module is an object with two properties, Foo and Bar. Said differently, the JavaScript module consists of two named properties.
But here is what the generated types definition looks like:
// AUTOGENERATED FILE
// This file was generated from grammar.ohm by `ohm generateBundles`.

import {
  ActionDict,
  Grammar,
  IterationNode,
  Namespace,
  Node,
  NonterminalNode,
  Semantics,
  TerminalNode
} from 'ohm-js';

export interface FooActionDict<T> extends ActionDict<T> {

}

export interface FooSemantics extends Semantics {
  addOperation<T>(name: string, actionDict: FooActionDict<T>): this;
  extendOperation<T>(name: string, actionDict: FooActionDict<T>): this;
  addAttribute<T>(name: string, actionDict: FooActionDict<T>): this;
  extendAttribute<T>(name: string, actionDict: FooActionDict<T>): this;
}

export interface FooGrammar extends Grammar {
  createSemantics(): FooSemantics;
  extendSemantics(superSemantics: FooSemantics): FooSemantics;
}

export interface BarActionDict<T> extends ActionDict<T> {

}

export interface BarSemantics extends Semantics {
  addOperation<T>(name: string, actionDict: BarActionDict<T>): this;
  extendOperation<T>(name: string, actionDict: BarActionDict<T>): this;
  addAttribute<T>(name: string, actionDict: BarActionDict<T>): this;
  extendAttribute<T>(name: string, actionDict: BarActionDict<T>): this;
}

export interface BarGrammar extends Grammar {
  createSemantics(): BarSemantics;
  extendSemantics(superSemantics: BarSemantics): BarSemantics;
}

declare const ns: {
  Foo: FooGrammar;
  Bar: BarGrammar;
};
export default ns;

This type definitions declare that the generated JavaScript module exports a default object, ns. Hence it declares that the generated module doesn't export named properties.
This enforces all the consuming applications to be compiled with the esModuleInterop TypeScript directive...or to post-process the type definitions to append the legit named properties declarations.

Here, the module is an object with two properties, Foo and Bar. Said differently, the JavaScript module consists of two named properties.

Hmmm, I'm not sure I'd put it that way. The module has a single export, which is a Namespace object. That object has two properties.
The way it's declared was based on the recommendation from the TypeScript documentation on .d.ts files. I didn't realize that it requires the use of esModuleInterop, but I see from the documentation that you're right about that.
I could see a few possible ways forward:

The documentation mentions using the old export = syntax. Would it be preferable for us to use that instead?
Maybe there is little value in Namespace objects, and it would be simpler to just use regular, named exports.


Allow to disable quotes around objects' keys

Thank you for these changes, i finally got around to update the crate in our project :)


import error with jsnext:main

{
 "jsnext:main": "src/InfiniteScroll.js",
}

package.json has jsnext:main feild
but in .npmignore , src has been ignored.

in addition , .babelrc should be ignore in .npmignore
because  "add-module-exports" someone like me ,isnot installed

.babelrc should be ignore in .npmignore

agree.
ref: https://github.com/CassetteRocks/react-infinite-scroller/issues/99

I encountered the same error, I should install add-module-exports to build my project:
yarn add babel-plugin-add-module-exports

Fixed in #279


gRPC can't recover itself after network changed

I'm writing API server that talks to GoogleCloud and underlying calls over grpc protocol, the production deployment is in Compute Engine or App Engine but development is on local laptop; I found a problem of gRPC is it can't recover itself after network changed, like moved over different wifi hotspot, and suspend/resume; it always failed with Error: Deadline Exceeded and I have to restart this Server
[2017-02-07T17:45:30.690Z] ERROR: App/25905 on hostname: unable to format response (err.code=4)
    Error: Deadline Exceeded
        at /node_modules/grpc/src/node/src/client.js:434:17

this is<EMAIL_ADDRESS>from npm used by @google-cloud/bigquery @google-cloud/monitoring and many other @google-cloud Nodejs libraries
Wonder if gRPC has design to overcome network fail and recover

Would you mind trying again with<EMAIL_ADDRESS>There were some changes in that version that may have affected this kind of failure.


Same news repeating again and again

@sumitkolhe Hey, can you help me with how can I get different news every time I call API

News updates periodically, so if there's no new news since you last made a request, you will get same news again

what languages are you using in it? NodeJS? or what

Its written in typescript


Dimension dictionary reduce locking

Description
Following from https://github.com/apache/druid/pull/12105 with 1-20% CPU in DimensionDictionary, there are some improvements to be made to the add() path on the StringDimensionIndexer. The main concern is that the locking is more complicated than necessary.
There are three sub-changes:

Introduce a benchmark
Swap the existing lock implementation to use a synchronized block
Reduce the number of times we have to cross the lock.

Estimate this code reduces CPU by ~5-7% on our indexing jobs overall, with ~30% improvements to the uncontended path.
Alternative solutions / future work
It seems like this code is mostly used only one-at-a-time, so an alternative path that changes the DimensionDictionary would be to move responsibility for synchronization outside of the dictionary and make the caller responsible for synchronization. This is likely to see a good boost to the later compareUnsorted stage but might constrain future work. Specifically it would prevent concurrent use of the dictionary from multiple threads during ingestion.
Related
#12105, #12109,
Key changed/added classes in this PR

Introduce benchmark StringDimensionIndexerProcessBenchmark
DimensionDictionary locking changes and accumulate size during add
StringDimensionIndexer simplification of call

This PR has:

[x] been self-reviewed.

[x] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[x] been tested in a test Druid cluster. -- a solution more like 12105 on top of 0.20 is currently running in Production for many months.

@kfaraz I started a conversation on slack; I think this is in a similar area you've been working recently. CC @maytasm

it would probably be worth checking that query performance for realtime tasks is not negatively impacted too much by this change (especially from multiple concurrent threads). There is only one writer thread but potentially many reader threads, as new dimension selectors are created on top of this dictionary during realtime queries.
For batch ingestion tasks, it might be worth using a separate DimensionDictionary implementation that has no locks. I use a similar thing for the NestedDataColumnIndexer, ComparatorDimensionDictionary, which is basically a copy of DimensionDictionary except uses a Comparator instead of requiring a Comparable, and doesn't have any locks since the nested column indexer doesn't currently use the dictionary it is building for realtime queries.

@clintropolis , thx for comments. I think it's ready to take a look at.
I've included benchmark results for each commit, and shown the progress. Each commit is broken to a small change so if we would prefer distinct PRs we can do that too.

it would probably be worth checking that query performance for realtime tasks is not negatively impacted too much by this change (especially from multiple concurrent threads). There is only one writer thread but potentially many reader threads, as new dimension selectors are created on top of this dictionary during realtime queries.

I've added test explicitly for the contended situation and brought in StampedLock which has much better performance with mixed read workloads.
It looks to me like most of the time it is one-thread-writing with possibly many readers, so I think this is a good blend. We could build a more advanced ArrayList structure that would be lockfree since this is append-only, but i'm not sure it's warranted. Let me know your thoughts.

For batch ingestion tasks, it might be worth using a separate DimensionDictionary implementation that has no locks. I use a similar thing for the NestedDataColumnIndexer,

I included a benchmark for single-writer only and it shows up with similar performance to the contended writer situation.
Overall, for JDK11,
BEFORE
jdk11 -- Benchmark                                                       Mode  Cnt      Score     Error  Units
StringDimensionIndexerProcessBenchmark.parallelReadWrite                 avgt   10  18769.135 ± 705.057  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelReader  avgt   10  23772.439 ± 947.011  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelWriter  avgt   10   3759.221 ±  59.897  us/op
StringDimensionIndexerProcessBenchmark.soloReader                        avgt   10    666.688 ±  12.723  us/op
StringDimensionIndexerProcessBenchmark.soloWriter                        avgt   10   3013.473 ±  13.480  us/op

AFTER
jdk11 - Benchmark                                                        Mode  Cnt     Score    Error  Units
StringDimensionIndexerProcessBenchmark.parallelReadWrite                 avgt   10   873.654 ± 12.358  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelReader  avgt   10   283.315 ±  0.067  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelWriter  avgt   10  2644.672 ± 49.549  us/op
StringDimensionIndexerProcessBenchmark.soloReader                        avgt   10   284.451 ±  0.588  us/op
StringDimensionIndexerProcessBenchmark.soloWriter                        avgt   10  2284.851 ± 51.339  us/op

Overall, both the contended and solo read/write is all improved.

cool, thanks for checking. Btw, I actually had a bug in that lockless dictionary used by nested columns that resulted in it not fully merging dictionaries which is fixed in this PR https://github.com/apache/druid/pull/13714/files#diff-b00489e26a5bff0e308349f4bdf8dc21a55798f8d0f35c995c56599cb6485eb1R93 😅
anyway, I'll try to have a closer look at this PR sometime soon

Made a typo in the old test cases, so I've rebased the lot and reprocessed the tests, which confirmed another opt I had in mind. Updated results:
BEFORE
jdk11 -- Benchmark                                                       Mode  Cnt      Score     Error  Units
StringDimensionIndexerProcessBenchmark.parallelReadWrite                 avgt   10  30471.552 ±  456.716  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelReader  avgt   10  18069.863 ±  327.923  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelWriter  avgt   10  67676.617 ± 2351.311  us/op
StringDimensionIndexerProcessBenchmark.soloReader                        avgt   10   1048.079 ±    1.120  us/op
StringDimensionIndexerProcessBenchmark.soloWriter                        avgt   10   4629.769 ±   29.353  us/op

AFTER
jdk11 - Benchmark                                                        Mode  Cnt      Score    Error  Units
StringDimensionIndexerProcessBenchmark.parallelReadWrite                 avgt   10   2914.859 ± 22.519  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelReader  avgt   10    508.010 ± 14.675  us/op
StringDimensionIndexerProcessBenchmark.parallelReadWrite:parallelWriter  avgt   10  10135.408 ± 82.745  us/op
StringDimensionIndexerProcessBenchmark.soloReader                        avgt   10    205.415 ±  0.158  us/op
StringDimensionIndexerProcessBenchmark.soloWriter                        avgt   10   3098.743 ± 23.603  us/op


After make PVM as a LeafNode, we won't change it's child plan, treat it as a HiveRelation, make code more simple.
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklists
📝 Author Self Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

[ ] Pull request title is okay.
[ ] No license issues.
[ ] Milestone correctly set?
[ ] Test coverage is ok
[ ] Assignees are selected.
[ ] Minimum number of approvals
[ ] No changes are requested

Be nice. Be informative.

@yaooqinn  WDYT about this solution?

Any more suggestion?

please take care of the tests

please take care of the tests

Fixed

ping @yaooqinn GA passed and conflict resolved.

The PR title is for refactoring, but actually what we do here is a bugfix, can we make it more precisely.

The PR title is for refactoring, but actually what we do here is a bugfix, can we make it more precisely.

How about current, desc also full fill.

Thank you, merged to master


CLion setup

@hendrikmuhs

Coverage remained the same at 86.698% when pulling 2280b7c84e381b210ad07c05fd81b85f0965f93e on narekgharibyan:clion_setup into f641487741ed758f61325e68512bd91dd4786f79 on cliqz-oss:master.


consul_agent  in api_z1 missing for OpenStack in docs

name: api_z1
templates:
- name: go-buildpack
release: cf
- name: binary-buildpack
release: cf
... ...

it seems the following two lines are missing, and api_z1 will fail to find blobstore since the endpoint is  https://blobstore.service.cf.internal
- name: consul_agent
release: cf

And also java-buildpack packages are missing on the Openstack stub, without the packages acceptance tests will fail:
- name: api_z1
    templates:
    - name: java-buildpack
      release: cf
    - name: java-offline-buildpack
       release: cf

Ok, there are a few things going on here, and some of this information is stale, so I'm going to try to break it down.
Docs
@shijiaxin, you mention that go-buildpack, binary-buildpack, and consul_agent are missing from the Openstack docs. It looks like they've been added by now. Is there anything else that you needed help with, or was it just those docs?
Templates and stubs
@Infra-Red, you mention that the Java buildpacks are missing from stubs. Which stubs are you referring to? Remember that stubs are per-environment configuration -- generic changes are made to templates that are defined in cf-release. If you take a look at templates/cf.yml, you'll see that the java buildpack is included. My guess is that these have been fixed since this issue was opened back in July. Have you been having more issues with Openstack manifest generation since then?
If all concerns have been taken care of, I'll close the issue.

@dsabeti Java buildpack is still missing in docs about deploying Cloud Foundry on Openstack iaas. Here the link to stub. Also want to note that this document contains different content compared to page source from github. Looks like github version of this example stub already has the fix, since api_z1 job using default templates which, as you have described above, is in templates/cf.yml.

@Infra-Red Great catch.
The docs are generated from the example stubs in cf-release, which override the templates for the api_z1 job. So they bring in cloud_controller_clock and cloud_controller_worker, but they don't bring in either of the java.*buildpack templates. Perhaps even more bizarrely, they don't do the same override for api_z2, so the stub would give you to api vms that are configured pretty differently.
I'm working on a fix to get this to be consistent. It may a bit little while before these changes make it to the docs page (I think they get published when we cut new releases of cf-release), but it should be in the pipeline soon.

Ok, here's the commit that should fix this problem: https://github.com/cloudfoundry/cf-release/commit/7dbb775af116d1dbaed41432f79940d3ca6278fa
@Infra-Red @shijiaxin, does this look like what you need? If so, I'm going to close the issue out at the end of the week.


How to get curcu

Why am I not getting circular profile picture using this code?
  Jimp.read('/profile_picture.png', function(err, profile_picture) {
    Jimp.read('/circle.png', function(err, circle) {
        profile_picture.mask(circle, 0, 0).getBase64(Jimp.MIME_JPEG, function (err, src) {
          var img = document.createElement("img");
          img.setAttribute("src", src);
          document.body.appendChild(img);
        });
      });
    });
  });

This is the circle image I am using and profile_picture is a square picture of the person... both are of the same size...

Why am I not getting circular profile picture using this code?
What I am getting is completely blacked out image...
I might be very wrong using this mask function but if somebody could explain to me how to use jimp to create circular profile picture?

According to the docs here, what Jimp.mask does is:

masks the image with another Jimp image at x, y using average pixel value
So I believe this will not really give you what you want.

You can use simple CSS to achieve what you want, here's an example.


_api/search/suggest?querytext='*' returns empty result using azure AD authention

Iam calling the search api "_api/search/suggest?querytext='*" using httpclient in azure functions. Authentication is done via azure ad application. Application permissions sites.fullcontrol is provided. But when ever i tried to call this suggest api, it returns empty results.
i can able to retreive the results of suggest api using username and password but using the azure ad app auth it gives the empty results.
whether we can able to call this suggest api using application permissions set in azure ad ? kinldy let me know if there are any alternatives

Closing as per https://github.com/SharePoint/sp-dev-docs/issues/5068


feat!: Removed vra7

vra7 is being removed from BTVA, this is removing it from the samples too, so the build will pass

Related Build Tools for Aria PR: https://github.com/vmware/build-tools-for-vmware-aria/pull/453


Create SECURITY.md

Am i ok with it being the way it is?

I would like some feed Feedback and thank you.


Connected with google oauth2 but no moderator privilege

Hi!
I've enabled oidc with google, i can login correctly but i can't give to auth user "moderator" id
I must see something in the screen that identify me as moderator?
i've this in config.js:
userMapping : async ({ peer, roomId, userinfo }) =>
{
peer.addRole(userRoles.MODERATOR);
peer.addRole(userRoles.AUTHENTICATED);
},
thanks!

What does your permissionsFromRoles look like?

Default:
permissionsFromRoles : {
// The role(s) have permission to lock/unlock a room
[CHANGE_ROOM_LOCK] : [ userRoles.MODERATOR ],
// The role(s) have permission to promote a peer from the lobby
[PROMOTE_PEER]     : [ userRoles.NORMAL ],
// The role(s) have permission to send chat messages
[SEND_CHAT]        : [ userRoles.NORMAL ],
// The role(s) have permission to moderate chat
[MODERATE_CHAT]    : [ userRoles.MODERATOR ],
// The role(s) have permission to share screen
[SHARE_SCREEN]     : [ userRoles.NORMAL ],
// The role(s) have permission to produce extra video
[EXTRA_VIDEO]      : [ userRoles.NORMAL ],
// The role(s) have permission to share files
[SHARE_FILE]       : [ userRoles.NORMAL ],
// The role(s) have permission to moderate files
[MODERATE_FILES]   : [ userRoles.MODERATOR ],
// The role(s) have permission to moderate room (e.g. kick user)
[MODERATE_ROOM]    : [ userRoles.MODERATOR ]
},

Do you have the red buttons in the top in the participant list in the side drawer?

Nothing...

@verbavolant  can you please share your config, and please test it with the latest 3.3 version.

@misi i'm using lastest 3.3 git
Here are my configs, thanks a lot!
mm.tar.gz

I see you have userMapping defined twice, please remove one of them.

Ok now it works, thanks a lot.
I need to merge the userMappings if i want to use name ad image from oauth2, so is better to insert in the config.example.js:
    userMapping           : async ({ peer, roomId, userinfo }) =>
    {
            peer.addRole(userRoles.MODERATOR);
            peer.addRole(userRoles.AUTHENTICATED);

           if (userinfo.picture != null)
            {
                    if (!userinfo.picture.match(/^http/g))
                    {
                            peer.picture = `data:image/jpeg;base64, ${userinfo.picture}`;
                    }
                    else
                    {
                            peer.picture = userinfo.picture;
                    }
            }

            if (userinfo.nickname != null)
            {
                    peer.displayName = userinfo.nickname;
            }

            if (userinfo.name != null)
            {
                    peer.displayName = userinfo.name;
            }

            if (userinfo.email != null)
            {
                    peer.email = userinfo.email;
            }
    },


"action" missing from CardHeaderClassKey in CardHeader.d.ts

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

According to the documentation, "action" should be able to be overridden with classes in CardHeader.

https://material-ui-next.com/api/card-header/
Current Behavior

On CardHeader, the action key is supposed to be override-able with classes, however it is missing in the CardHeaderClassKey in CardHeader.d.ts

@chubbsMcfly Do you want to add the missing key? :)

I'm new to this, not sure if i went about it the proper way.
https://github.com/mui-org/material-ui/pull/10379


Angular 4 build and TypeScript throws export alias private using private name error

Your Environment

flatpickr version used: 4.0.6
Browser name and version: Chrome 60
OS and version: Windows 10

Using the latest 4.0.6 version, it's running fine when running development but when I try to build my library, it throws the following errors
Building Angular library

BUILD ERROR
Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/locale.ts:38:22:
Exported type alias 'Locale' has or is using private name 'MonthsWeekDays'.

Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/locale.ts:62:28:
Exported type alias 'CustomLocale' has or is using private name 'MonthsWeekDays'.

Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/instance.ts:51:24:
Exported type alias 'Instance' has or is using private name 'Elements'.

Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/instance.ts:52:3:
Exported type alias 'Instance' has or is using private name 'Formatting'.

Error: Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/locale.ts:38:22: Exported type alias 'Locale' has or is using private name 'MonthsWeekDays'.

Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/locale.ts:62:28:
Exported type alias 'CustomLocale' has or is using private name 'MonthsWeekDays'.

Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/instance.ts:51:24:
Exported type alias 'Instance' has or is using private name 'Elements'.

Error at C:/sourcecode/Angular-Slickgrid/node_modules/flatpickr/src/types/instance.ts:52:3:
Exported type alias 'Instance' has or is using private name 'Formatting'.

    at new UserError (C:\sourcecode\Angular-Slickgrid\node_modules\ng-packagr\node_modules\@angular\tsc-wrapped\src\tsc.js:27:28)
    at check (C:\sourcecode\Angular-Slickgrid\node_modules\ng-packagr\node_modules\@angular\tsc-wrapped\src\tsc.js:93:15)
    at Tsc.typeCheck (C:\sourcecode\Angular-Slickgrid\node_modules\ng-packagr\node_modules\@angular\tsc-wrapped\src\tsc.js:173:9)
    at C:\sourcecode\Angular-Slickgrid\node_modules\ng-packagr\node_modules\@angular\tsc-wrapped\src\main.js:122:23
    at <anonymous>

error Command failed with exit code 111.
ERROR: "packagr" exited with 1.

Hello,
Thanks for reporting this!
The typings are a bit of a pain right now (see also #1100), but everything should be fixed by next release.
I'd suggest finding a workaround, most likely by circumventing type checking if its preventing you from building your project.

Thanks waiting for your next release patiently.

Should be fixed in v4.1 out right now

Thanks for the new version, I had problems with my library with 2 frameworks Aurelia and Angular (I maintain both) and the Aurelia is now fixed with the new way to import your library
import * as flatpickr from 'flatpickr';

I did not have a chance to try with Angular yet (I use it at work), but it's looking promising.
Once I can time to test out with Angular (which the issue was open for), I will report back. Thank you for this great library. By the way, I got into using your lib because of the beautiful UI and different theme available, good job :)

@ghiscoding yes, please report back if it works on angular.
and thanks for the kind words.

Unfortunately, I can't get it to work when building my Angular 4 library. However I don't have these errors that I mentioned when I opened the issue.
I know that the problem I have is all but related to rollup.js (tree shacking) and because they strictly only allow to import a library this way import flatpickr from 'flatpickr' and they refuse this way import * as flatpickr from 'flatpickr'
I have the same problem with moment.js and had to resolve to a monkey patch to get rollup happy. But I tried this patch with flatpickr and that doesn't work.
Modify the tsconfig.json to allow synthetic default
{
  "compileOnSave": false,
  "compilerOptions": {
    "allowSyntheticDefaultImports": true,
  ...

and try the monkey patch
import * as _flatpickr from 'flatpickr';
const flatpickr: any = (<any>_flatpickr).default || _flatpickr;

I can get the library to compile with the monkey patch, but then using the lib in my Angular App just throws a bunch of unrelated error. If I remove flatpickr and recompile then my App is back to normal, so I know it's related to that.
I spent over an hour trying all kind of different ways to import it without much success. I'll have to put that aside for Angular, but at least it works in Aurelia now.

@ghiscoding :( thats unfortunate. Have you tried using require() to import flatpickr instead?
e.g.
const flatpickr = require("flatpickr")

oh I could give that a try tomorrow at work, will let you know. Thanks

Sorry for the delay, I finally took the time to try this out and... I'm happy to report back that it's works 🎉
However for TypeScript to accept it, I had to write it this way (else require is unknown by TS)
declare function require(name: string);
const flatpickr = require('flatpickr');

I can finally close the issue 😄

I've opened a request to fix issues like these in TypeScript by bringing declaration files to parity with language features. https://github.com/microsoft/TypeScript/issues/35822


odo list fails with SISSEGV when application has no odo managed components

/kind bug

What versions of software are you using?
Operating System: Linux
Output of odo version: 2.0.0
How did you run odo exactly?
It seems in some cases (or always?) when odo can detect application name from non odo managed components it fails with SISSEGV  because it does not expect array of components for app to be empty. (Just my guess) .
Original issue https://github.com/redhat-developer/vscode-openshift-tools/issues/1824.
Actual behavior
Command failed: "/home/denis/p/vscode-openshift-tools/out/tools/linux/odo" list --app app2 --project p1 -o json
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x166add2]
goroutine 1 [running]:
github.com/openshift/odo/pkg/component.getRemoteComponentMetadata(0xc0004729a0, 0xc00073dc20, 0xa, 0x7ffe88816183, 0x4, 0x7ffe88816192, 0x2, 0x101, 0x0, 0x0, ...)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/pkg/component/component.go:1414 +0xd2
github.com/openshift/odo/pkg/component.GetComponent(...)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/pkg/component/component.go:1403
github.com/openshift/odo/pkg/component.List(0xc0004729a0, 0x7ffe88816183, 0x4, 0xc000039420, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/pkg/component/component.go:882 +0x90a
github.com/openshift/odo/pkg/odo/cli/component.(*ListOptions).Run(0xc000753320, 0x0, 0x0)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/pkg/odo/cli/component/list.go:273 +0x15e8
github.com/openshift/odo/pkg/odo/genericclioptions.GenericRun(0x2227fc0, 0xc000753320, 0xc0008e4500, 0xc0000af380, 0x0, 0x6)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/pkg/odo/genericclioptions/runnable.go:31 +0x13c
github.com/openshift/odo/pkg/odo/cli/component.NewCmdList.func1(0xc0008e4500, 0xc0000af380, 0x0, 0x6)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/pkg/odo/cli/component/list.go:338 +0x5e
github.com/spf13/cobra.(*Command).execute(0xc0008e4500, 0xc0000af320, 0x6, 0x6, 0xc0008e4500, 0xc0000af320)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/vendor/github.com/spf13/cobra/command.go:830 +0x2aa
github.com/spf13/cobra.(*Command).ExecuteC(0xc000139680, 0x1fd50c8, 0xc0004cc780, 0x7)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/vendor/github.com/spf13/cobra/command.go:914 +0x2fb
github.com/spf13/cobra.(*Command).Execute(...)
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/vendor/github.com/spf13/cobra/command.go:864
main.main()
	/builddir/build/BUILD/gocode/src/github.com/openshift/odo/cmd/odo/odo.go:67 +0x330

Expected behavior
Returning empty array of components.
Any logs, error output, etc?

I would like to work on this

currently trying to reproduce this on a 3.11 cluster.

opened a PR to resolve it but need verification if its resolving this panic @dgolovin @mohitsuman

@dgolovin can you please have a look.


new: plugin capabilities, dropping get_type API

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature
Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area driver-kmod

/area driver-ebpf

/area libscap
/area libsinsp

/area tests

/area proposals

What this PR does / why we need it:
See #259:
Basically, we want to drop the get_type plugin API because it has no great use cases, and instead does not allow us to add new plugin types without breaking API.
Plugin capabilities notion enters the chat:

a plugin is composed of at least 1 capability
at the moment, we support Event Sourcing and Extraction capabilities
we might support more capabilities in the future
get_type C function is not needed anymore; plugin caps will be automatically detected from exported symbols

Which issue(s) this PR fixes:

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

new: deprecated plugin `get_type` API in favour of a composable set of capabilities, automatically detected given exported symbols

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@FedeDP: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs

build-libs-with-chisels
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-with-chisels

build-libs-minimal
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-minimal

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs

build-libs-with-chisels
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-with-chisels

build-libs-minimal
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-minimal

build-libs-bundled-deps
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-bundled-deps

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs

build-libs-with-chisels
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-with-chisels

build-libs-minimal
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-minimal

build-libs-bundled-deps
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-bundled-deps

build-libs-arm
460d14f46f2d6174e2d695e0b8f51d9dd0e39dc1
link
true
/test build-libs-arm

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash:<PHONE_NUMBER>b5254c1838ef89e6202545a19385a2

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 1b7294f822dc125fa779f9f45ebf7650048cc3a9

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


cog: Fix wrong var in use to add the selected platforms

Also:

graphene: disable neon support on arm 32bits (https://github.com/ebassi/graphene/issues/215). Upstream-status: Reported

Could you elaborate what component uses graphene causing the alignment fault? At least for GStreamer's gltransformation, this should be fixed by https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1321.

Could you elaborate what component uses graphene causing the alignment fault? At least for GStreamer's gltransformation, this should be fixed by https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1321.
I'm running the WPE port from latest WebKit main and recent cog/libwpe/wpebackend-fdo on ARM 32 bit devices and I have not seen this since.

The crash came out by using the gtk4 Cog's platform (cog -P gtk4 https://...) on a raspberrypi3 (32bits) and MACHINE_FEATURES:append = " vc4graphics" (environment created using the instructions detailed in image-with-mesa-vc4-driver-weston-and-wpebackend-fdo)
Core was generated by `cog -P gtk4 https://igalia.com'.
Program terminated with signal SIGBUS, Bus error.
#0  0x6af085c0 in merge_color_matrix_nodes (matrix2=matrix2@entry=0x1c07b98, offset2=offset2@entry=0x1c07bd8, child=child@entry=0x1c760f0) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtksnapshot.c:562

warning: Source file is more recent than executable.
562	  graphene_vec4_t off2 = *offset2;
[Current thread is 1 (LWP 1340)]
warning: File "/home/psaavedra/local/workshop/toolchain_browsers_rpi3/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-gnueabi/usr/lib/libstdc++.so.6.0.30-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load:/home/psaavedra/local/workshop/nwot-arm/veld/1.0.0/sysroots/cortexa9hf-neon-veld-linux-gnueabi/usr/lib/libstdc++.so.6.0.24-gdb.py".
To enable execution of this file add
	add-auto-load-safe-path /home/psaavedra/local/workshop/toolchain_browsers_rpi3/sysroots/cortexa7t2hf-neon-vfpv4-poky-linux-gnueabi/usr/lib/libstdc++.so.6.0.30-gdb.py
line to your configuration file "/home/psaavedra/.gdbinit".
To completely disable this security protection add
	set auto-load safe-path /
line to your configuration file "/home/psaavedra/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
	info "(gdb)Auto-loading safe path"
(gdb) bt
#0  0x6af085c0 in merge_color_matrix_nodes (matrix2=matrix2@entry=0x1c07b98, offset2=offset2@entry=0x1c07bd8, child=child@entry=0x1c760f0) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtksnapshot.c:562
#1  0x6af08a80 in gtk_snapshot_collect_color_matrix (snapshot=<optimized out>, state=0x1c07b78, nodes=<optimized out>, n_nodes=<optimized out>) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtksnapshot.c:600
#2  0x6af09356 in gtk_snapshot_pop_one (snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtksnapshot.c:1374
#3  0x6af09540 in gtk_snapshot_pop_internal (snapshot=snapshot@entry=0x1c84580, is_texture_pop=is_texture_pop@entry=0) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtksnapshot.c:1459
#4  0x6af0a24a in gtk_snapshot_pop (snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtksnapshot.c:1575
#5  0x6afb33c2 in gtk_css_filter_value_pop_snapshot (filter=0x1088790, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkcssfiltervalue.c:992
#6  0x6aee7c8a in gtk_css_style_snapshot_icon_paintable (style=0x1ba5b80, snapshot=0x1c84580, paintable=0x1131930, width=16, height=16) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkrendericon.c:153
#7  0x6b054462 in gdk_paintable_snapshot (paintable=0x106de50, snapshot=0x1c84580, width=16, height=16) at /usr/src/debug/gtk4/4.6.7-r0/gdk/gdkpaintable.c:236
#8  0x6ae8ed58 in gtk_image_snapshot (widget=<optimized out>, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkimage.c:1045
#9  0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10e0258, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#10 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10e0258, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#11 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10e0258, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#12 0x6af88380 in gtk_widget_real_snapshot (widget=0x10e0170, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#13 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10e0170, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#14 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10e0170, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#15 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10e0170, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#16 0x6af88380 in gtk_widget_real_snapshot (widget=0x10eb2e0, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#17 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10eb2e0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#18 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10eb2e0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#19 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10eb2e0, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#20 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10df1a0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#21 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10df1a0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#22 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10df1a0, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#23 0x6af88380 in gtk_widget_real_snapshot (widget=0x10eb100, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#24 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10eb100, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#25 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10eb100, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#26 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10eb100, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#27 0x6af88380 in gtk_widget_real_snapshot (widget=0x1031ad8, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#28 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x1031ad8, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#29 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x1031ad8, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#30 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x1031ad8, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#31 0x6af88380 in gtk_widget_real_snapshot (widget=0x10319e0, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#32 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10319e0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#33 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10319e0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#34 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10319e0, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#35 0x6af88380 in gtk_widget_real_snapshot (widget=0x10e9198, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#36 0x6af7e1c6 in gtk_widget_create_render_node (widget=widget@entry=0x10e9198, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11587
#37 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10e9198, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#38 0x6af88304 in gtk_widget_snapshot_child (widget=<optimized out>, child=0x10e9198, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:12043
#39 0x6af88380 in gtk_widget_real_snapshot (widget=0x10841a0, snapshot=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:734
#40 0x6af7e252 in gtk_widget_create_render_node (widget=widget@entry=0x10841a0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11582
#41 0x6af8000e in gtk_widget_do_snapshot (widget=widget@entry=0x10841a0, snapshot=snapshot@entry=0x1c84580) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11622
#42 0x6af87c78 in gtk_widget_snapshot (snapshot=0x1c84580, widget=0x10841a0) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11644
#43 gtk_widget_render (widget=0x10841a0, surface=0x11380c8, region=0x1ba3148) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwidget.c:11676
#44 0x6af8d53a in surface_render (surface=<optimized out>, region=<optimized out>, widget=<optimized out>) at /usr/src/debug/gtk4/4.6.7-r0/gtk/gtkwindow.c:4733
#45 0x6b0395aa in _gdk_marshal_BOOLEAN__BOXEDv (closure=0x198b878, return_value=0x7e9e4720, instance=<optimized out>, args=..., marshal_data=0x0, n_params=1, param_types=0x1135448) at /usr/src/debug/gtk4/4.6.7-r0/gdk/gdkmarshalers.c:130
#46 0x73e0ff3a in _g_closure_invoke_va (closure=closure@entry=0x198b878, return_value=return_value@entry=0x7e9e4720, instance=instance@entry=0x11380c8, <EMAIL_ADDRESS>n_params=1, param_types=0x1135448)
    at /usr/src/debug/glib-2.0/1_2.72.3-r0/gobject/gclosure.c:893
#47 0x73e2176a in g_signal_emit_valist (instance=0x0, instance@entry=0x11380c8, signal_id=0, detail=detail@entry=2124302308, var_args=<EMAIL_ADDRESS>at /usr/src/debug/glib-2.0/1_2.72.3-r0/gobject/gsignal.c:3406
#48 0x73e2203e in g_signal_emit (instance=instance@entry=0x11380c8, signal_id=<optimized out>, detail=0) at /usr/src/debug/glib-2.0/1_2.72.3-r0/gobject/gsignal.c:3553
#49 0x6b0594c2 in gdk_surface_process_updates_internal (surface=0x11380c8) at /usr/src/debug/gtk4/4.6.7-r0/gdk/gdksurface.c:1348
#50 gdk_surface_paint_on_clock (clock=<optimized out>, data=<optimized out>) at /usr/src/debug/gtk4/4.6.7-r0/gdk/gdksurface.c:1436
#51 0x73e0ff3a in _g_closure_invoke_va (closure=closure@entry=0x11372a0, return_value=return_value@entry=0x0, instance=instance@entry=0x1136908, <EMAIL_ADDRESS>n_params=0, param_types=0x0)
    at /usr/src/debug/glib-2.0/1_2.72.3-r0/gobject/gclosure.c:893
#52 0x73e21e90 in g_signal_emit_valist (instance=0x1b562c8, instance@entry=0x1136908, signal_id=0, detail=detail@entry=2124302700, var_args=<EMAIL_ADDRESS>at /usr/src/debug/glib-2.0/1_2.72.3-r0/gobject/gsignal.c:3406
--Type <RET> for more, q to quit, c to continue without paging--c
#53 0x73e2203e in g_signal_emit (instance=instance@entry=0x1136908, signal_id=<optimized out>, detail=0) at /usr/src/debug/glib-2.0/1_2.72.3-r0/gobject/gsignal.c:3553
#54 0x6b04ced8 in _gdk_frame_clock_emit_paint (frame_clock=frame_clock@entry=0x1136908) at /usr/src/debug/gtk4/4.6.7-r0/gdk/gdkframeclock.c:708
#55 0x6b04d7ec in gdk_frame_clock_paint_idle (data=<error reading variable: value has been optimized out>) at /usr/src/debug/gtk4/4.6.7-r0/gdk/gdkframeclockidle.c:608
#56 0x73d30286 in g_timeout_dispatch (source=0x1581d10, callback=<optimized out>, user_data=<optimized out>) at /usr/src/debug/glib-2.0/1_2.72.3-r0/glib/gmain.c:4971
#57 0x73d2f950 in g_main_dispatch (context=0x100d108) at /usr/src/debug/glib-2.0/1_2.72.3-r0/glib/gmain.c:3417
#58 g_main_context_dispatch (context=context@entry=0x100d108) at /usr/src/debug/glib-2.0/1_2.72.3-r0/glib/gmain.c:4135
#59 0x73d2fc02 in g_main_context_iterate (context=context@entry=0x100d108, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at /usr/src/debug/glib-2.0/1_2.72.3-r0/glib/gmain.c:4211
#60 0x73d2fc68 in g_main_context_iteration (context=context@entry=0x100d108, may_block=may_block@entry=1) at /usr/src/debug/glib-2.0/1_2.72.3-r0/glib/gmain.c:4276
#61 0x73f044aa in g_application_run (application=application@entry=0x10048a8, argc=argc@entry=4, argv=argv@entry=0x7e9e4c14) at /usr/src/debug/glib-2.0/1_2.72.3-r0/gio/gapplication.c:2569

We reached to thissolution through https://github.com/ebassi/graphene/issues/215#issuecomment-1092113902

Ah, I see. Thanks for the explanation.

Is it possible to disable the NEON instructions only if the gtk platform is enabled? I think it's a bit unfortunate that all graphene users get their NEON instructions disabled, even if the affected gtk platform is not even built.

Is it possible to disable the NEON instructions only if the gtk platform is enabled?

I don't see a way of how an flag in one component (cog) could affect to the selected flags for others.  I can only visualize an kind of global setting to be set on the local.conf or in the distro. Something like: ENABLE_COG_GTK4 = "yes" or ENABLE_GTK4_TWEAKS = "yes"

Yes, a global setting, a distro feature (maybe gtk3, gtk4, ..?) or a packageconfig neon for graphene (maybe even upstream in oe-core/meta-oe) that could be disabled for the gtk case sounds good to me.

Yes, a global setting, a distro feature (maybe gtk3, gtk4, ..?) or a packageconfig neon for graphene (maybe even upstream in oe-core/meta-oe) that could be disabled in the local.conf for the gtk case sounds good to me.

Unfortunately, the distro features are more about the stack itself (wayland, x11, 3g ... )  than for a particular widget toolkit. I am going to do an explicit mention here to @kraj  who time to time collaborated in this  repo and it has by far much more experience in poky and meta-oe. For sure he can give a good suggestion.

Is it possible to disable the NEON instructions only if the gtk platform is enabled? I think it's a bit unfortunate that all graphene users get their NEON instructions disabled, even if the affected gtk platform is not even built.

We do not have UI toolkits as distro level knobs, but loosely inferred so perhaps we could do a packageconfig knob for the above recipe something like
PACKAGECONFIG[gtkgui] = "--enable-gui=gtk3,--enable-gui=no,gtk+3"

and then it could be tied to a DISTRO_FEATURE like x11 if needed.
PACKAGECONFIG ??= " \
    ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'gtkgui', '', d)} \

but if we are needing it without X11 too then we might just keep it a packageconfig and turn it on and off from
site config file like local.conf on need bases
PACKAGECONFIG:append:pn-<PN> = " gtkgui"

It landed in poky this commit: https://github.com/yoctoproject/poky/commit/491a323d0866e85d6ebf621baade2924ad474a78
It should be part of Yocto Mickledore (to be released in April 2023).
I don't know if it will be backported to current stable version of Yocto Langdale.
In any case, I guess it is an option to add here a .bbappend with similar contents

It landed in poky this commit: yoctoproject/poky@491a323
It should be part of Yocto Mickledore (to be released in April 2023). I don't know if it will be backported to current stable version of Yocto Langdale.
In any case, I guess it is an option to add here a .bbappend with similar contents

Let's keep this discussion just as reference anyone facing this problem could take the decision by their own.
I've remove the append and I will merge the original commit what actually it is a bug in meta-webkit.


Merge reload with vanilla refresh

As of v1.4.1 there seems to be a reload button in the multiplayer view. While it could be useful in some cases, I don't think it needs a separate visible button at the moment.
How about merging it with the vanilla refresh button so it does both things at once?

I fear that will not be very intuitive.

Then it could be merged with it's function, aka reload NCR's config every time multiplayer server list is loaded by any means.

Well, how is that going to be more intuitive?

E.g. change this in the description

Configuration file is named NoChatReports.json and is located in default config folder.
to
Configuration file is named NoChatReports.json and is located in default config folder. It can be changed while the game is playing and it will take effect as soon as the player loads the multiplayer server list.

That description is only visible for unsafe servers.  Leave it alone.


NeuroKit outdated installation

Hello,I have some question,and need your help
I wanted to use Neuropython but it's still failed
I ran the code,and it made error
error
module 'biosppy.signals.ecg' has no attribute 'correct_rpeaks'
I checked the file 'biosppy.signals.ecg', there has function'correct_rpeaks',and already has attribute
Is file "bio_ecg_preprocessing" wrong or file "biosppy.signals.ecg"?
code

import neurokit as nk
import pandas as pd
import numpy as np
import seaborn as sns
df =  pd.read_csv("C:/Users/User/Desktop/CH1.csv")
df.plot()
bio = nk.ecg_process(ecg=df["ECG1"], rsp=None, sampling_rate=250, filter_type='FIR', filter_band='bandpass', filter_frequency=[3, 45], segmenter='hamilton', quality_model='default', hrv_features=['time', 'frequency'], age=None, sex=None, position=None)
nk.z_score(bio["df"]).plot()

I tried to unistall neurokit ,and upgrade it
but it still wrong>> C:\Users\User\AppData\Local\Temp\pip-j3oj0gb4-build
What can i do?

Related to this issue.
Don't worry, we'll fix this.
First of all, I have some questions:

Did you already use python for anything else?
Does the python you downloaded work, besides neurokit-related stuff (i.e., Can you run import numpy as np and np.mean([1,2,3]) in Spyder?

@SIHHUA I have updated the neuropython version, could you try downloading it and running to see if it works? Here's the link thanks!

@DominiqueMakowski
Did you already use python for anything else?>>No
Does the python you downloaded work, besides neurokit-related stuff (i.e., Can you run import numpy as np and np.mean([1,2,3]) in Spyder? >>Yes，it printed 2.0,and I tested other code which could run.

At what moment does this error appear? Could you provide some reproducible code / steps required to reproduce this error?

I opened WinPython Command Prompt.exe to install NeuroKit, as shown below

But it could install Neuropsydia successfully

@SIHHUA That's strange indeed, what is your python version?
Could you try, from here, installing the neuropython distribution (where neurokit is already installed)? Does this work?

@DominiqueMakowski
Python version is Python 3.6.6
Yes,I downloaded Neuropython.zip

And followed hard steps

It didn't work,because neurokit can't download successfully >>  No module named 'neurokit'


Fix outcome check in indexer

Receipts are an incorrect point of filtering "submit"s to da contract. Same submission could be filtered in chunk on sender shard, and then in chunk on receiver shard. This is avoided checking initial tx itself

Had to re-do status fething since GetExuctionOutcome doesn't reflact status of transactions triggered by indexed one


Haskell Stack swallows RTS flags

I ran into an issue when I tried to get stack to generate .prof files with the +RTS flags.  It seems like the stack/ghc on windows swallows the +RTS flags meant for the application!  This bug means the windows code never even got the -N flag and wasn't even parallelized.  One workaround is to run the benchmarks against the executable directly without calling stack (somewhere in .stack-work\install\xxxxxxx\bin) until this is resolved.

I just discovered a workaround in that thread I linked.  We should change the command used to run skynet to:
stack build && stack exec skynet --RTS -- +RTS -N


Keep IPC sockets in tmp

Fix #86382

If you're not comfortable with it, I'm not comfortable with it. How about I check the length and log a warning if it's too long but leave the logic alone

Yeah that would be already much better than today 🙏


Change the type of time value from integer to number

Codecov Report

Merging #2 (e7957ac) into master (03e2789) will increase coverage by 0.18%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   96.87%   97.05%   +0.18%
==========================================
  Files           1        1
  Lines          64       68       +4
==========================================
+ Hits           62       66       +4
  Misses          2        2

Flag
Coverage Δ

unittests
97.05% <100.00%> (+0.18%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

context.lua
97.05% <100.00%> (+0.18%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Mat Action Row] Missing left margin for buttons other than mat-button

Bug, feature request, or proposal:
Bug
What is the expected behavior?
All buttons types should have a 8px margin left when in a mat-action-row, except for the first one in the row (mat-action-row>button:first-of-type or something like that)
What is the current behavior?
Only mat-button have it, which is set with this rule; .mat-action-row button.mat-button
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 6.0.6
Material 6.3.1
Is there anything else we should know?
Maybe something like mat-button-base shall become the general class added to all buttons and manage styling which must be applied to all buttons regardless of the appearance, like this one. instead of repeating all mat-buttons selectors

There's https://github.com/angular/material2/pull/11961 which adds mat-button-base, but it hasn't been merged yet. After it's in we should be able to use it for the action row.

I'm seeing this with code like this:
    <mat-action-row>
      <button type="submit" mat-raised-button color="primary">Apply Filters</button>
      <button type="button" mat-raised-button (click)="clearFilters()">Clear Filters</button>
    </mat-action-row>

It looks like I can work around it by adding mat-button like this:
    <mat-action-row>
      <button type="submit" mat-button mat-raised-button color="primary">Apply Filters</button>
      <button type="button" mat-button mat-raised-button (click)="clearFilters()">Clear Filters</button>
    </mat-action-row>

Is it safe to do this or could it cause problems mixing mat-button and mat-raised-button?

This has been fixed with https://github.com/angular/components/commit/6a21d947f67dac472cecef02ba49996b2cc81047. Thanks for the issue report!


for me, clear pod cache and modify pod target to arm64, I don't know why pod target change to i386

          for me, clear pod cache and modify pod target to arm64, I don't know why pod target change to i386

Originally posted by @yongbam in https://github.com/dooboolab-community/react-native-iap/issues/2539#issuecomment-1718750305

/Users/apple/Library/Developer/Xcode/DerivedData/jiubApp-bzhczaqfqsovtccvlaqoilqwndut/Build/Intermediates.noindex/jiubApp.build/Debug-iphonesimulator/jiubApp.build/DerivedSources/jiubApp_vers.c module map file '/Users/apple/Library/Developer/Xcode/DerivedData/jiubApp-bzhczaqfqsovtccvlaqoilqwndut/Build/Products/Debug-iphonesimulator/RNIap/RNIap.modulemap' not found


Add support for Vue Single File Components

Related to #3
We should support .vue files, since they are the reason (at least for me) that make vue awesome!
Decide if .vue files can be used with JIT compilation, or only with AOT

Would this help with css implementation? Currently I can use an external style sheet to style my template markup, like this:
<scroll-view>
                <stack-layout>
                    {{msg}}
                    ##{{msg}}##
                    <stack-layout orientation="horizontal">
                        <button text="Foo"></button>
                        <button text="Bar" @tap="onTap"></button>
                        <button text="Baz" style="color: red;"></button>
                    </stack-layout>
                </stack-layout>
            </scroll-view>

But if I want to use an external style sheet wtih .vue files, what would be the procedure?

.vue files are structured like this:
<template></template>

Order doesn't matter, and multiple

Not sure how this will work inside {N} though!

Looks like we ought to give this a test, to see if we can have this external .vue file included in our current setup - which is what we do with external .html files in Angular. Cool stuff!

I attempted to import a separate component:
import OtherComponent from './component.vue';
and them use it as a component:
Components: {
OtherComponent
}
but I think my formatting of the separate vue file is wrong as I get an error that there's an unexpected '<' - it's picking up my simple template markup in the .vue file. Not sure I am formatting my external file properly. If you can tell me how it should look, I will try to import and use it
(btw I also got loops working using v-for) :)

Don't think importing .vue files is possible without transpiling them into js, will probably require a hook

@rigor789 - You should be able to use a .vue file just like .xml or the .html file that angular uses.   Both angular and xml goes through the xml system, by leveraging the xml system you should be able to parse the vue files at runtime and then use them, there is really no magic to how it works...

@NathanaelA I already have the parser! Does ns-angular use a build step (webpack?) to transpile into js? I'm wondering if there's a way to add "loaders" that work at runtime.

No ns-angular does not need to use webpack (recommended for speed, but not needed).   The html load runs it through there own builder that converts things into the proper  ns components.    For .vue files; you would parse them using the built in xml parser; then when it hit

Webpack is mentioned as the way to improve performance for iOS and it can be a more easy solution to make vue -> nativescript transpiled js-bundle. Also many developers use webpack. There is no obvious answer for me why webpack loader can work or not at nativescript runtime, but why is it needed? Can try to use https://github.com/systemjs/systemjs

Hi - member of the Vue team here. I'm not very familiar with NativeScript or its build process, but I definitely recommend using vue-loader with Webpack if it's already being used for Angular integration. Vue's SystemJS support has been suspended for now as very few people were using it and it became a bit of a headache.

Hey @chrisvfritz I'm not very familiar with the internals of vue-loader, wondering if it's tied to vue, or can I replace it with nativescript-vue?

Vue-loader is what Vue uses to transform .vue files to a JS module. I would imagine from there, you'd then take the JS module and send it through nativescript-vue?
https://vue-loader.vuejs.org/en/
Scott

@rigor789 Vue files can be transformed without Webpack, if that's what you mean. The same tools used in the source code could be used here. It'd simply be more work to duplicate what vue-loader already does. Am I understanding your question correctly?

I'm probably overthinking it, I was just thinking if vue-loader is tied to platform specific parts of vue, since in nativescript-vue there are no html elements, just nativescript specific elements.
So If I have an entry file like main.js which requires nativescript-vue and the .vue files, and creates a root Vue instance, I can run it through vue-loader, and it will not complain that vue isn't required?

Ah, I think I see what you mean now. The way Webpack works, vue-loader (and other loaders) are used to specify how to process modules of specific file extensions. Unfortunately, it can't be used independently as a simple function that an entry file as an input. So if you want to process these files without Webpack, you would not use vue-loader, but rather look at its internals to see how it's working and to do something similar within NativeScript's build process.

I think using webpack is fine, don't want to reinvent the wheel, just want to make sure that the output can be consumed properly by nativescript!

Hi!
I got this (mostly) working. All I had to do was:

Add webpack according to {NS} documentation.
Add vue-loader to the webpack configuration.

Had to work through a couple of bumps, but nothing fancy.
I created this NativeScript template that bootstraps a project with webpack and .vue file support.
I think this feature is better placed on a NS template since there was nothing to do in the nativescript-vue lib itself. If we ever need to fork/change vue-loader, that will be its own project.
The


Show song name + length when using -m option

What is the purpose of your issue?

[ ] Bug
[X] Feature Request
[ ] Question
[ ] Other

Description
I personally like to use the -m option with -sf '{artist} - {track_name}' a lot to get the songs I want. However, when I'm presented with the list when going over a large album or playlist, it's sometimes hard to figure out which one is the correct one. I think it would be super useful to print out the name of the artist/song in Spotify as well as the length of what Spotify has to compare lengths with the Youtube links (in the case of mixes and radio edits) before the list of songs to choose from.

Because I don't know if contributing to this would require adding tests or not... I can at least say the solution to this (at least for my situation) was adding the following at the end of def generate_metadata(raw_song): in spotify_tools.py. For anyone else who may want it, assuming it doesn't get added to the repo:
    duration_min = int(meta_tags["duration"] / 60)
    duration_sec = int(meta_tags["duration"] % 60)
    duration = "{}:{}".format(duration_min, duration_sec)
    log.info("Song: {} - {} ; Length: (about) {}".format(meta_tags["artists"][0]["name"], meta_tags["name"], duration))


summary

PR automatically created

coverage: 100.0%. remained the same
when pulling c904ffde282efd1971fa8005e99a342c955a4910 on summary
into 18eb6e777225fed6b52b23e2c2407f03ceb4847b on master.